Doesn't work on M1 Mac

% uname -a
Darwin Phuong-Mac-M1-Pro.local 20.5.0 Darwin Kernel Version 20.5.0: Sat May  8 05:10:31 PDT 2021; root:xnu-7195.121.3~9/RELEASE_ARM64_T8101 arm64
% asdf list crystal
No versions installed
% asdf install crystal 1.1.0
\e[31mFail:\e[m Unsupported architecture

works on this mac arm64, at least for newer versions like 1.5.1 :
crystal -v
Crystal 1.5.1 [2d2b7799c] (2022-09-08)

LLVM: 10.0.0
Default target: aarch64-apple-darwin


yaml配置格式错误，导致获取到空配置

描述bug
使用apollo配置yaml，如果yaml格式错误，比如缩进有问题。导致应用程序启动时拉取到空的配置。
复现
通过如下步骤可以复现：

配置有格式错误的yaml
拉取配置内容

期望
将异常抛出给应用程序，由应用程序自己处理。

There is a syntax check button for yaml namespace, and it will show the error if the configuration is invalid.

@gp0014 You may use the ConfigFile interface to retrieve and transform the yaml configurations.
  ConfigFile config = ConfigService.getConfigFile("application", ConfigFileFormat.YAML);
  String content = config.getContent();
  // do your own transform logic

Thanks for your reply.
That's could obtain original content as expected. The problem is converting yaml to properties when construction YamlConfigFile object, because of parsing error, an empty object will be created without any exception throwing.

@gp0014 You may obtain the original content and transform it to properties by yourself, in which case, you could catch the parse exception.


Lambda Websocket Authorizer not working

Hi,
There are two examples in the documentation for configuring a lambda authorizer for web sockets.
The first at https://docs.sst.dev/constructs/WebSocketApi#authorization
new WebSocketApi(stack, "Api", {
  authorizer: {
    type: "lambda",
    function: new Function(stack, "Authorizer", {
      handler: "test/lambda.handler",
    }),
  },
});

returns the error when building:
Type 'WebSocketApi' is missing the following properties from type 'Function': _isLiveDevEnabled, localId, createUrl, currentVersion, and 60 more.
The second at https://docs.sst.dev/constructs/WebSocketApi#authorizer
new WebSocketApi(stack, "Api", {
  authorizer: {
    type: "lambda",
    function: new Function(this, "Authorizer", {
      handler: "src/authorizer.main",
    }),
  },
  routes: {
    $connect: "src/connect.main",
    $default: "src/default.main",
    $disconnect: "src/disconnect.main",
  },
});

doesn't get invoked, but the web socket connection is denied.

Many thanks,

Hi @niallel, sorry for super late response. Did you figure out what the issue was?

Sorry, didn’t find out a solution, then changed my code to use client side polling.On 12 Jan 2023, at 21:48, Frank @.***> wrote:
Hi @niallel, sorry for super late response. Did you figure out what the issue was?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@fwang I have the same issue, have you looked into it?
Here is my code:
const api = new WebSocketApi(stack, 'ws', {
  authorizer: {
    type: 'lambda',
    identitySource: ['route.request.querystring.jwt'],
    function: new Function(stack, 'Authorizer', {
      handler: 'packages/functions/src/authorizer.handler',
    },
  },
  routes: {
    '$connect':    'packages/functions/src/ws/connect.handler',
    '$disconnect': 'packages/functions/src/ws/disconnect.handler',
    'sendMessage': 'packages/functions/src/ws/sendMessage.handler',
  }
});

The authorizer gets called, but returning { isAuthorized: true } in it, does not let the request reach connect.handler for the connect event.

Following up here https://discord.com/channels/983865673656705025/1103635174568243210


Glyph

SteamDB app page links to a few games using this
https://steamdb.info/depot/845241/
Brief explanation of the change
Add support for Glyph Launcher. The link above has no information on the Glyph depot itself, but it lists games that use it. many of them are free, so the depot can easily be checked by installing one of them.

Unfortunately due to an issue in steamdb backend this currently won't match: the depot you link has no files (because the app it comes from isn't public, and backend skips updating depots from 'depotfromapp')

Does the link HAVE to lead directly to a game that uses the depot? They are all listed right there. The link is just for you to check the depot to approve the pull request, isn't it?

As you can see from the depot link, it lists no files, so the rule won't match.

What are you talking about? The page lists 10 whole games that use Glyph launcher.

This detector works by looking at the files.

Yeah, the files in the code, not the files i linked to in the pull request!

Bruh.

The issue remains, this rule won't work because SteamDB isn't tracking the names of files in that depot currently due to the problem xPaw mentioned.

What are you talking about?

JUST. CLICK. ON. THEEEEEEEEEEEEEEEEEEEEEEESSSSSSSSSSSE!

Change this to only detect (?:^|/)GlyphClient\.cfg$.
Other files won't be found due to the issue I described in my first comment. (and we don't need to detect multiple files for the same thing anyway).


Fix iBuffalo SNES Controller mappings (Windows)

Confused about the previous values. This controller doesn't have triggers and the shoulder button values didn't match. X and Y as well as A and B were swapped, now matching Xbox button layout.

A number of vendors including Retro Bit and Retro Fighters make Saturn-style devices that spoof the iBuffalo SNES style pad. If you compare closely you’ll notice the mapping you overwrote is a superset of the one you submitted, laid out in the “Sega” format from the README. The face button pairs are swapped on those devices as well — if you use gamepad type identification in your software designating the device ID as “like Saturn” and use an appropriate button label to input lookup, the other mapping will work correctly for all devices instead of just one.
It’s not ideal, but I think it’s an acceptable compromise. PRs from other DB users in the past have suggested we always defer to whichever device has more buttons which I don’t necessarily agree with that, but unless one layout represents a device with a vastly larger market share whose mapping is entirely different, I think this kind of compromise is best.


Add tests

This package can use some automated tests, but I'm unsure how to handle that right now. Any help is appreciated 🙂

:tada: In progress!


Message batch ingestion lag fix

We saw a lot of logs for the FreshnessBasedConsumptionStatusChecker that show the servers are not aware of the freshness of incoming events
Through some debugging, we determined the issue was the stream ingestion interface changed a few months ago, and the backwards compatible code actually has a bug.

"getStreamMessage" default implementation fails to provide the metadata for the message
The new stream message metadata interface doesn’t have a 1 arg constructor anymore

Tested: We deployed this change and we see the consumption checker now has freshness
Before:
INFO [FreshnessBasedConsumptionStatusChecker] [XXXX] Segment XXXX with freshness -9223370349721398798ms has not caught up within min freshness 10000.But the current ingested offset is equal to the latest available offset 25327986.
After:
INFO [FreshnessBasedConsumptionStatusChecker] [XXXX] Segment XXXX with freshness 3621938ms has not caught up within min freshness 10000. At offset 42853810. Latest offset 42853811.

Codecov Report

Merging #10983 (ef346b5) into master (2d30e28) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #10983      +/-   ##
==========================================
- Coverage    0.11%    0.11%   -0.01%
==========================================
  Files        2190     2190
  Lines      118056   118058       +2
  Branches    17873    17873
==========================================
  Hits          137      137
- Misses     117899   117901       +2
  Partials       20       20

Flag
Coverage Δ

integration1temurin11
?

integration1temurin17
0.00% <0.00%> (ø)

integration1temurin20
0.00% <0.00%> (ø)

integration2temurin11
?

integration2temurin17
?

integration2temurin20
?

unittests1temurin11
0.00% <0.00%> (ø)

unittests1temurin17
0.00% <0.00%> (ø)

unittests1temurin20
0.00% <0.00%> (ø)

unittests2temurin11
0.11% <0.00%> (ø)

unittests2temurin17
0.11% <0.00%> (ø)

unittests2temurin20
0.11% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...java/org/apache/pinot/spi/stream/MessageBatch.java
0.00% <0.00%> (ø)

...ava/org/apache/pinot/spi/stream/StreamMessage.java
0.00% <0.00%> (ø)

...apache/pinot/spi/stream/StreamMessageMetadata.java
0.00% <0.00%> (ø)

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

@walterddr Can you rerun the failed checks, they seem like random errors?


Backend question for: Get the whole document from an add-in for Word or PowerPoint

Issue
I am having trouble merging Slices together. I follow these docs, using btoa() to encode each slice, the server then b64 decodes and concatenates the slices together but, like @nguruprasanna in this issue, I get a large string starting with:
80,75,3,4,20,0,6,0,8,0,0,0,33,0,50,100,51,3,232,1,0,0,29,12,0,0,19,0,8,2,91,67,111,110,116,101,110,116,95,84,121,112,101,115,93,46,120,109,108,32,162,4,2,40,160,0,2,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,

I write the above data to a .docx file but it is corrupted when trying to open in Word.
It's unclear from the docs what the correct way to decode/merge these slices is in the backend.
The link mentioned in the article is dead (https://support.microsoft.com/en-us/help/318425/how-to-send-and-receive-binary-documents-by-using-an-asp-net-web-servi)
Article
https://docs.microsoft.com/en-us/office/dev/add-ins/word/get-the-whole-document-from-an-add-in-for-word

@DiegoSalazar Thanks for raising this issue. As you may have seen in the related issue, we have an internal item tracking this work. However, we don't have any timeline yet for when this request may be created and published.
Thanks.

I figured out the issue on my end. I was incorrectly packing/unpacking the byte array.
In Ruby, I was doing bytes = Base64.decode64(slice.data).split(?,).map(&:to_i) and then writing those bytes directly to a file. But I was missing this step: bytes.pack('C*')


Adds i term2 and alfred themes

Greetings!
Really loving the theme, so I spent some time making Alfred and iTerm2 versions and I'm quite happy with the results. Thought I'd give you a chance to decide if they're up to par.
iTerm2 (w/ Fish shell)
Noctis

Noctis Azureus

Noctis Lux

Noctis Minimus

Noctis Obscuro

Noctis Sereno

Noctis Uva

Noctis Viola

Alfred
Noctis

Noctis Azureus

Noctis Lux

Noctis Minimus

Noctis Obscuro

Noctis Sereno

Noctis Uva

Noctis Viola

Hey Ethan!
I really dig what you did here. Let  me have a closer look and I'll come back with some feedback.
Cheers! :)

I really love how Alfred theme turned out. The only thing I'm hesitant about would be the grey text.
What would you think about replacing the grey text with the color used for comments in each theme? It's just something I was wondering how it would feel/look like.
Eg.: for Noctis, Noctis Sereno and Noctis Obscuro the color would be Horizon #5b858b, similar to the attached screenshot (I changed only one line and although the text is bold the font weight should be normal).

For Fish Shell/Iterm2 I attached a screenshot which is the reflection of the way I would have used the colors. I don't know if the suggested changes can be implemented and/or If you will like them. I just thought you will not mind if I share some of my ideas.

Please let me know how do you feel about it. 🙂

Thanks for the great feedback!
I like all the suggestions, and will update them real quick this evening then post some new screenshots.

Hey! Didn't have time to work on it last night but was able to take another look this evening. Unfortunately, some of the iTerm2 suggestions aren't going to be doable due to the limitations of the terminal color support. I'll reply back with some more information tomorrow

No worries mate!

@ethanfann Just wanted to say this looks great!

@chibicode I still have plans to get these updated with some of @liviuschera's suggestions! I've just been pretty bad at getting back around to it

I came here looking for an iterm2! Thank you very much!

It's awesome, exactly what I was looking for. It's a shame it was abandoned 😔

Oh wow this is a blast from the past! I'm still using Noctis but have since swapped back my terminal to a more stock theme. Looks like this would need some minor color updates as well as support for some of the newer variants.
No promises but I'll try to take another look. (I know I said the same like 5 years ago)

Might help land this if the Alfred and iTerm2 themes were split into separate PRs. I could possibly try to finish the iTerm2 themes up if a separate PR would be accepted.


Support for PS Move?

Best utility, solves libusb incompatibility issues, but no compatibility with Move controllers? at least no documentation for such. Please consider addressing this. Thank you.

It's literally in the README, the motion controller is not considered in the design at all. I can leave this open here as a reminder if we ever get to think about implementing it.


Make fields length and hashes mandatory on MetadataDescription

The spec says they can be left out at times, but we might as well include them and bolt them down to be required and match in all cases. This would add a small amount of additional security during checks, but it would add additional security to the internals of the lib by preventing misusing the struct to not include fields when it should.
I can't think of a reason not to do this (other than saving bytes in transit). @trishankkarthik, any thoughts?

Sorry, where does the spec say that they can be left out?
For the targets metadata, the "custom" attribute is optional, but the "target" attribute (containing the length and hashes) are not.
I am assuming you mean the snapshot metadata? For targets metadata files, we require only the version numbers. I think we additionally require the hashes and length of only the root metadata file in there. We don't need the length in there, because that can be fixed in the updater client code. @JustinCappos, do you think we need the hashes of the root metadata in there? What attacks can happen if they are not there?
Cc: @vladimir-v-diaz

https://github.com/theupdateframework/tuf/blob/develop/docs/tuf-spec.txt#L677-L681

The HASHES and LENGTH are the hashes and length of the file, both of which are only specified for the root file.  VERSION is listed for the root file and all other roles available on the repository.  LENGTH is an integer. HASHES is a dictionary that specifies one or more hashes, including the cryptographic hash function.  For example: { "sha256": HASH, ... }.

This is metadata about other metadata, not targets.

Thanks for the clarification! Here's my TLDR summary:

You need to list only the version numbers of all targets metadata files, including the top-level targets role.

You do not need to list the hashes of the targets metadata files. This will increase bandwidth costs, and is useful only in the threat model where you are worried about malicious mirrors being able to substitute some targets metadata files (because they somehow have the private signing keys to these files), but not the snapshot metadata file.

More details will be published soon in the Mercury paper next week.
I'll let Justin and Vlad answer whether we need to list in the snapshot metadata file the hashes of the root metadata file.

@heartsucker
We used to list the hashes of all delegated role files in Snapshot, but discovered that it added a large bandwidth cost to community repositories like PyPI.  The Mercury paper shows that including the hashes of all metadata (and their version numbers) in Snapshot incurs an initial cost of 360% and a recurring cost of 24%.  The cost is the overhead of adding TUF metadata to PyPI.
In contrast, only listing version numbers results in an initial cost of 48% and recurring cost of 24%.

Ok, so then this goes back to the edge case of the snapshot metadata needing to include the hashes / length of the root from some other issues / email / discussion somewhere. I'm not sure what edge case that solves then? Isn't that just the case where the snapshot is compromised and is intentionally falsely reporting the root? What attacked would do that? Wouldn't they try to cover their tracks by just reporting the root correctly?

I think @JustinCappos recommended that we list the hash of Root in Snapshot to help recover from fast-forward attacks, but this was before any of the changes proposed in the TAPs (specifically, TAP 5?)

Adding to my previous reply...
Adding Root's hash was also done before CoreOS's contribution to Root key rotations.

to help recover from fast-forward attacks

A fast forward on which role?

@heartsucker
Suppose an attacker launches a fast forward attack, where they list a maliciously high version number for Root (and no hash is listed for it).  For instance, the actual version number for Root is 4, but the attacker says it is now 666.
The repository detects the attack and attempts to release a new version of Root to revoke Snapshot's keys.  The new Root file has a version of 5.  Unfortunately, a client would reject the Root file because version 5 < 666.  If Root's hash were instead listed in Snapshot, the client could reset its cache of currently-trusted version numbers and use it to recover from the fast-forward attack.
At least, that's how it would have been done before we introduced Root key rotations and TAP 5...
(I hope my explanation is clear and correct.  So many things have changed over time, including switching back-and-forth between listing and not listing Root hashes, that I might have missed a reason for a change.  The TAP process actually stems from this...)

I knew what fastforward attacks were in general, but in a previous discussion, it was said to blow everything away after a root rotation and start over. New root implies new keys for the snapshot role which means the evil snapshot would be out of the question and the client would just grab the newest snapshot and start trusting that, no?

That is correct, but this wasn't always the case because Root rotations did not exist at the time.  The update process was slightly different, in that Root would only be downloaded if any of the other top-level roles either could not be refreshed (due to a key revocation, for example) or Snapshot indicated a new Root version number.
We previously refreshed top-level metadata in the following order: Timestamp->Snapshot->Root (if it has changed)-> Targets-> remaining delegated metadata.  Please note that Root could be updated and jump from version 1 to 8 in one refresh call.

So then if I understand this correctly, given the new root-rotation-wipe-it-all approach, there is no benefit to listing the hash of the root because it should never be inaccurate, and the only case it would be is if an attacker is intentionally trying to make themselves known.

I agree, there doesn't appear to be any benefit to listing Root's hash in Snapshot given the new changes.


Unable to install recent version of asciidoctor-kroki in a maven build

I'm using asciidoctor-kroki to write my diagrams for my conferences
Since asciidoctor-kroki is available as a ruby gem, I have to use torquebox maven plugin to install asciidoctor-kroki.
So I have this dependency definition
		<dependency>
			<groupId>rubygems</groupId>
			<artifactId>asciidoctor-kroki</artifactId>
			<version>0.17.0</version>
			<type>gem</type>

			<exclusions>
				<exclusion>
					<groupId>rubygems</groupId>
					<artifactId>asciidoctor</artifactId>
				</exclusion>
				<exclusion>
					<groupId>rubygems</groupId>
					<artifactId>thread_safe</artifactId>
				</exclusion>
				<exclusion>
					<groupId>rubygems</groupId>
					<artifactId>concurrent-ruby</artifactId>
				</exclusion>
			</exclusions>
		</dependency>

And this configuration for the gem-maven-plugin
			<plugin>
				<groupId>de.saumya.mojo</groupId>
				<artifactId>gem-maven-plugin</artifactId>
				<version>2.0.1</version>
				<configuration>
					<gemHome>${project.build.directory}/gems</gemHome>
					<gemPath>${project.build.directory}/gems</gemPath>
				</configuration>
				<executions>

					<execution>
						<id>install-gems</id>
						<goals>
							<goal>initialize</goal>
						</goals>
						<phase>initialize</phase>
					</execution>
				</executions>
			</plugin>

Unfortunatly, my build always fails with this error
[INFO] --- gem-maven-plugin:2.0.1:initialize (install-gems) @ conferences ---
[WARNING] ERROR:  While executing gem ... (ArgumentError)
    " html, b" is not an octal string

I've started a small debug session, revealing that gem-maven-plugin in fact invokes the Java executable
load('jar:file:C:\Users\nicolas-delsaux\.m2\repository\org\jruby\jruby-complete\<IP_ADDRESS>\jruby-complete-<IP_ADDRESS>.jar!/META-INF/jruby.home/bin/gem') install --ignore-dependencies --no-user-install --no-verbose --no-document C:\Users\nicolas-delsaux\.m2\repository\rubygems\asciidoctor-kroki\0.17.0\asciidoctor-kroki-0.17.0.gem --bindir C:\Users\nicolas-delsaux\Documents\Zenika\conferences\target\gems\bin

Do you have ay idea how I could fix that ?

I don't really know as I've never encountered this error... 🤔
It seems related to rubygems, I found a similar issue (with the same error message): https://github.com/rubygems/rubygems/issues/3050 (not sure if it's related or not).
Do you know when this issue appeared? (As you said "a recent version" which implies that it previously worked?)

Seems like I have an other thing to check:
when opening the asciidoctor-kroki file (obtained from the rubygems maven mriror at torquebox), I have the following content (which is neither zip *nor tar)
<!DOCTYPE html>

        <meta charset="utf-8">

        <meta content="NOW" name="expires">
        <meta content="index, follow, all" name="GOOGLEBOT">
        <meta content="index, follow, all" name="robots">

        <meta content="width=device-width; initial-scale=1.0; maximum-scale=1.0; user-scalable=0;" name="viewport">

I may be wrong, but it seems like http://rubygems-proxy.torquebox.org/releases is no more a good url to fetch gems for maven projects ...

Thanks to Twitter I've found an alternative proxy!
https://twitter.com/jruby/status/1216832621000282112

Well, that's weird, when trying to get asciidoctor-kroki through the http://mavengems.jruby.org/releases repository, I get the same file, which means that "new" url directs to the same content.

Let's try to give more visibility to that question ... I've asked it on StackOverflow.

Fixed by switching to another rubygem download maven plugin : the mavengem download plugin dynamically converts rubygem repository structure to maven one, allowing download of rubygems in amven without relying upon a translating proxy.

without relying upon a translating proxy.

I think that's the right approach. We should probably recommend it on the Asciidoctor documentation. I don't mind adding instruction on how to install the Asciidoctor Kroki extension using Maven (and potentially Gradle) in the README. Could you please share a minimal pom.xml that converts an AsciiDoc file to HTML with the Asciidoctor Kroki extension enabled?


fix bug in installtools

executor takes an extra parameter which the type of executor doesnt
handle
relates to  :https://github.com/Jumpscale/home/issues/8

The bug was due to a poor implementation of the execute command in our SSHClient.
Commit f7d5f94571b00d3c457b07c789ec8b9f7f5e6064 improve the implementation and fix the issue.
Sandboxing works now also when using SSH executor.

tested it works


Click on page service ， display blank

Please answer these questions before submitting your issue.

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

Question

What do you want to know?

Click on page service ， display blank，Other page clicks can be displayed normally。Want to know what is the cause of it?

Bug

Which version of SkyWalking, OS and JRE?
5.0.0 beta 。centos7.2  JDK1.8

Which company or project?

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

When it was originally deployed, it could be clicked normally. But the number of services is relatively small. Now there are more than 150 statistics. I don't know if this is the reason.
The log is as follows:
2018-06-07 15:18:02,642 - org.apache.skywalking.apm.collector.ui.jetty.handler.GraphQLHandler -78391585 [qtp1163157114-7013] INFO  [] - Execution result is ExecutionResultImpl{data={searchService=[{key=-6, label=<IP_ADDRESS>:9110.com.hsjry.p2p.userbiz.bill.facade.AccountSearchBillBizFacade.ifUserLoginIn(null)}, {key=9, label=<IP_ADDRESS>:9110.com.hsjry.p2p.userbiz.facade.AccountBasicSearchBizFacade.queryBindCardInfo(null)}, {key=-10, label=<IP_ADDRESS>:9101.com.hsjry.p2p.usercore.bill.facade.UserImBillFacade.queryUserImBillById(null)}, {key=-8, label=<IP_ADDRESS>:9101.com.hsjry.p2p.usercore.facade.UserBindCardFacade.queryBindCardInfo(null)}, {key=7, label=/notice/pullnotice.json}, {key=-15, label=<IP_ADDRESS>:9121.com.hsjry.p2p.trade.purchase.facade.service.PurchaseQueryFacade.QueryPurchaseCreditBank(null)}, {key=-12, label=<IP_ADDRESS>:9110.com.hsjry.p2p.userbiz.bill.facade.UserImInfoFacade.queryUserImInfo(null)}, {key=-14, label=/notice/pullnotice}, {key=-23, label=<IP_ADDRESS>:9121.com.hsjry.p2p.trade.purchase.facade.service.CommonQueryFacade.queryLastDealOrder(null)}, {key=19, label=<IP_ADDRESS>:9104.com.hsjry.p2p.platform.facade.HelpCenterFacade.queryHelpCenterTree(null)}]}, errors=[], dataPresent=true, extensions=null}
2018-06-07 15:18:02,687 - org.apache.skywalking.apm.collector.ui.jetty.handler.GraphQLHandler -78391630 [qtp1163157114-7115] INFO  [] - Execution result is ExecutionResultImpl{data={getServiceResponseTimeTrend={trendList=[3, 4, 4, 4, 4, 4, 3, 4, 4, 4, 3, 3, 3, 4, 4, 4]}, getServiceThroughputTrend={trendList=[2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 73, 203, 2, 2, 2]}, getServiceSLATrend={trendList=[10000, 10000, 10000, 10000, 10000, 10000, 10000, 10000, 10000, 10000, 10000, 10000, 10000, 10000, 10000, 10000]}, getServiceTopology={nodes=[{id=-6, name=<IP_ADDRESS>:9110.com.hsjry.p2p.userbiz.bill.facade.AccountSearchBillBizFacade.ifUserLoginIn(null), type=null, sla=10000, calls=304, numOfServiceAlarm=0}, {id=6, name=Jedis/get, type=null, sla=10000, calls=304, numOfServiceAlarm=0}, {id=1, name=User, type=USER}], calls=[{source=7, target=-6, isAlert=false, callType=tiny-rpc, cpm=2, avgResponseTime=4}, {source=1, target=-6, isAlert=false, callType=tiny-rpc, cpm=17, avgResponseTime=3}, {source=-6, target=6, isAlert=false, callType=tiny-rpc, cpm=19, avgResponseTime=0}]}}, errors=[], dataPresent=true, extensions=null}

Requirement or improvement

Please describe about your requirements or improvement suggestions.

@hanamily You may can help about this. Maybe some invalid data.

Just trying to reboot collectorService and webappService, It can be shown normally. In a few days, see if the same problem will happen again. Thank you！

Duplicate of #141

Sorry, Duplicate of https://github.com/apache/incubator-skywalking-ui/issues/141


Research large format printing for feasibility

(from user group 2018)

To clarify: Map image resolution degrades when the map image in a printable map is larger than 20" in width or height.


Define Initial Content Strategy

See sub-tasks for details.

📆 November status: on schedule

📆 December status: on schedule

all sub-tasks completed and presented on 1/25 https://drive.google.com/drive/folders/100957yVH-7l4vPqmoZVQpOb0m-76kK94?usp=sharing


Feature request: option to insert space before argument list

My preferred formatting style is
def foo (x : Int) : Int
foo (42)

instead of the more common
def foo(x : Int) : Int
foo(42)

The preserveSpaceBeforeArguments option provides some support for spaces before argument lists, but only if they are already there.  I don't see a way with the current options to get the spaces I want inserted, so can we have that as a new feature?

At the present time looks like the closest one can get is:
def foo(x : Int) : Int
foo (42)

preserveSpaceBeforeArguments=true
spaceBeforeColon=true


Blocking Error

Hej, I'm getting this error, what do I do?
C:\Program Files\dotnet\sdk\5.0.302\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.TargetFrameworkInference.targets(141,5): error NETSDK1045: The current .NET SDK does not support targeting .NET 6.0.  Either target .NET 5.0 or lower, or use a version of the .NET SDK that supports .NET 6.0.

It seems you're missing dotnet 6.0 sdk
https://dotnet.microsoft.com/en-us/download
grab the x64 sdk for 6.0

Thanks!


TypeError: _this._afterConnect is not a function

Line 87
         this._afterConnect(callback, result);
When running light.js from Grow-Iot Quickstart, this line causes this error
Grow server connection established.
/home/cloud/my-plant-light/node_modules/Grow.js/dist/Grow.umd.js:469
          _this._afterConnect(callback, result);
                ^

TypeError: _this._afterConnect is not a function
    at /home/cloud/my-plant-light/node_modules/Grow.js/dist/Grow.umd.js:469:17
    at self._callbacks.(anonymous function) (/home/cloud/my-plant-light/node_modules/Grow.js/node_modules/ddp/lib/ddp-client.js:434:16)
    at DDPClient._message (/home/cloud/my-plant-light/node_modules/Grow.js/node_modules/ddp/lib/ddp-client.js:156:7)
    at null.<anonymous> (/home/cloud/my-plant-light/node_modules/Grow.js/node_modules/ddp/lib/ddp-client.js:96:10)
    at emitOne (events.js:77:13)
    at emit (events.js:169:7)
    at EventTarget.dispatchEvent (/home/cloud/my-plant-light/node_modules/Grow.js/node_modules/ddp/node_modules/faye-websocket/lib/faye/websocket/api/event_target.js:24:10)
    at instance._receiveMessage (/home/cloud/my-plant-light/node_modules/Grow.js/node_modules/ddp/node_modules/faye-websocket/lib/faye/websocket/api.js:142:10)
    at null.<anonymous> (/home/cloud/my-plant-light/node_modules/Grow.js/node_modules/ddp/node_modules/faye-websocket/lib/faye/websocket/api.js:34:49)
    at emitOne (events.js:82:20)

https://github.com/CommonGarden/Grow-IoT/issues/163

Is anyone else having this problem? I'm having trouble reproducing this...

http://www.darul.io/post/2016-01-07_what-is-babel-js-doing

The code is not being compiled to ES2015 correctly... should resolve this soon.


Create initial website mockup

Use Figma to design a few possible options for main pages (homepage, search, detail page?). Follow MIMO for inspiration.

First draft of a potential faceted search, made by @chetbae and me:
VIM-1.pdf

Looks great!
Thank you!!

On Jul 21, 2023, at 1:00 PM, Taz Scott-Talib @.***> wrote:
First draft of a potential faceted search, made by @chetbae and me:
VIM-1.pdf
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Taz's patented histogram timeline search ✨

#44


Update MCW dataset to improve the standardization of PHI datasets

Depends on #24

I will need more details on what "standardization" means .


Add Zellij to Tools and Plugins

Checklist for submitting a pull request to Terminals Are Sexy:

[ x] I have carefully read and comply with the Contributing Guidelines of this repo.
[ x] I have searched the PRs of this repo and believe that this is not a duplicate.

Zellij:
Link:
Tools and Plugins:
A terminal workspace with batteries included:

Also add 'lsd'.
This project is a rewrite of GNU ls with lot of added features like colors, icons, tree-view, more formatting options etc. The project is heavily inspired by the super colorls project.
Title: lsd
Link: https://github.com/Peltoche/lsd
Category: Tools and Plugins
Description: The next gen ls command

Also add 'fd'.
fd is a program to find entries in your filesystem. It is a simple, fast and user-friendly alternative to find. While it does not aim to support all of find's powerful functionality, it provides sensible (opinionated) defaults for a majority of use cases.
Title: fd
Link: https://github.com/sharkdp/fd
Category: Tools and Plugins
Description: A simple, fast and user-friendly alternative to 'find'


weekli demo (partial)

Hey guys,
Added a partial 100-LOC implementation [1] (source [2]) of weekli [3]. It only covers the interesting parts with events and delegation. I didn't do any mediaquery listeners to flip it to a mobile layout or do anything like divs w/ display: table toggling. It does have an exposed api though, which it uses to pick some random hours on init.
Pretty happy with how the delegation semantics worked out. I was worried not being able to bind multiple mousedown handlers would be problematic, but turned out to be easily worked around as well as more optimal for perf.
[1] http://leeoniya.github.io/domvm/demos/weekli.html
[2] https://github.com/leeoniya/domvm/blob/master/demos/weekli.html
[3] http://collnwalkr.github.io/weekli/
--
@lawrence-dol @iamjohnlong @collnwalkr @barneycarroll @yosbelms

Passing the hash of selectors to bind events as:
var onMousedown = {
    "*":		function(e, node) { mouseIsDown = true; },
    ".day":		function(e, node) { toggleDay(node.data); },
    ".hour":	function(e, node) { toggleHour(node.data); },
    ".dayhour":	function(e, node) { setDayHour.apply(null, node.data); setSelected = node.props.class.indexOf("sel") == -1; },
};

Is this documented?  Nice work man.

yeah, it's buried in the tpl reference, that thing is dense with goodies :)
[1] https://github.com/leeoniya/domvm/blob/master/README.md#template-reference

Agree with @iamjohnlong.


Feature borders

To satisfy some art requirements I needed to add a border to the empty circle.
This PR adds a border setting that sets the thickness of the border (it defaults to 0).

Hi!
Thanks for writing tests and documentation - looks pretty good.
The only thing is that border is not so popular among users - you're the first person who is asking to implement it. So I can't just merge it.
However, I had a request to give more control over "empty circle" thickness. And this would solve your problem too, as I understand (I saw your borders implementation).
Probably I'll implement it soon...

No worries, I made this for myself, so it doesn't bother me if it's not merged in.
I think the empty circle thickness would do the same thing as my borders, just in a slightly different (perhaps better) way. I'll keep an eye on the project.

Thanks and sorry for declining your pull-request, I really appreciate any effort people make to improve my work ;)


Unable to setup TLS connection to external docker server - [Errno 2] No such file or directory

Hi
Great HACS addon, been using it for a while running non-encrypted socket connection to docker but recently i changed that. Since then I have been having issues getting it to connect. I think the issue is the with the certpath. I've set this as
certpath: /config/custom_components/monitor_docker/ but also tried certpath: /config/custom_components/monitor_docker/docker.pem
Is this cert path looking for the full path to cert file, or is it just looking for folder path with both the server docker cert and the ca.pem cert?
Any help would be greatly appreciated
Addon Error log:

Traceback (most recent call last):
File "/config/custom_components/monitor_docker/helpers.py", line 162, in init
self._api = aiodocker.Docker(url=url)
File "/usr/local/lib/python3.9/site-packages/aiodocker/docker.py", line 113, in init
ssl_context = self._docker_machine_ssl_context()
File "/usr/local/lib/python3.9/site-packages/aiodocker/docker.py", line 374, in _docker_machine_ssl_context
context.load_cert_chain(
FileNotFoundError: [Errno 2] No such file or directory

The certificate names should be:
cert.pem
key.pem

Thanks! That helped, I could not find it documented anyway but it turned out the ca cert for docker needed to list both the root level cert and intermediate cert.


(maint) Add CHANGELOG and update for 0.2.0 release

Add a CHANGELOG for the strings module and update it to reflect the changes made for 0.2.0.

:+1:


Packer ansible provisioner fails in handshake

Describe the bug
Packer fails with ansible provisioner fails handshake with below mentioned configuration.
To Reproduce
Steps to reproduce the behavior:
Execute the packer configuration with a ansible provisioner  as:
build {
 sources = [
  "source.ibmcloud-vpc.centos"
 ]

 provisioner "ansible" {
  playbook_file = "provisioner/centos-playbook.yml"
 }
}

Expected behavior
Provisioning should succeed with builder succeeding.
Screenshots
If applicable, add screenshots to help explain your problem.
Failure response :
==> ibmcloud-vpc.centos: failed to handshake
  ibmcloud-vpc.centos: fatal: [default]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh: Unable to negotiate with <IP_ADDRESS> port 38917: no matching host key type found. Their offer: ssh-rsa", "unreachable": true}
  ibmcloud-vpc.centos:
  ibmcloud-vpc.centos: PLAY RECAP *********************************************************************
  ibmcloud-vpc.centos: default

This issue is with the auth and works with below changes as reported by user:
provisioner "ansible" {
  ansible_env_vars = [
   "ANSIBLE_SSH_ARGS='-o PubkeyAcceptedKeyTypes=+ssh-rsa -o HostkeyAlgorithms=+ssh-rsa'"
  ]
  playbook_file = "provisioner/centos-playbook.yml"
 }


[TF Lite] TfLiteGpuDelegate Init: CONV_2D: Unsupported data type for float32 tensor

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 20.04
TensorFlow installed from (source or binary): pip install tensorflow
TensorFlow version (or github SHA if from source): 2.2.0
TensorFlow Lite version: org.tensorflow:tensorflow-lite:2.2.0, org.tensorflow:tensorflow-lite-gpu:2.2.0

Command used to run the converter or code if you’re using the Python API
The code used to run the converter can be found in this notebook: https://github.com/franksacco/where-is-wally/blob/master/converter.ipynb.
In particular, the part of interest is:
from tensorflow import lite

converter = lite.TFLiteConverter.from_keras_model(model)
converter.optimizations = [lite.Optimize.DEFAULT]
tflite_model = converter.convert()

with open('models/unet_v2.f1lo-b14-e60-lr0.001.44.optimized.tflite', 'wb') as f:
    f.write(tflite_model)

The output from the converter invocation
The optimized TFLite model is successfully generated.
Also, please include a link to the saved model or GraphDef

Trained model: https://github.com/franksacco/where-is-wally/raw/master/models/unet_v2.f1lo-b14-e60-lr0.001.44.hdf5
Optimized TFLite Model: https://github.com/franksacco/where-is-wally/raw/master/models/unet_v2.f1lo-b14-e60-lr0.001.44.optimized.tflite

The model uses this implementation of the U-Net and takes a 256x256x3 image and outputs a 256x256x1 grayscale mask.
Failure details
The model without optimizations works perfectly.
The model with optimizations works correctly when executed on the CPU.
However, when I try to initialize the TFLite interpreter with the GPU delegate using the following code, I get the error java.lang.IllegalArgumentException: Internal error: Failed to apply delegate: TfLiteGpuDelegate Init: CONV_2D: Unsupported data type for float32 tensor.
MappedByteBuffer model = FileUtil.loadMappedFile(
    activity, "unet_v2.f1lo-b14-e60-lr0.001.44.optimized.tflite"
);

Interpreter.Options options = new Interpreter.Options();
options.addDelegate(new GpuDelegate());

Interpreter interpreter = new Interpreter(model, options);

Any other info / logs
Traceback of the exception:
I/tflite: Created TensorFlow Lite delegate for GPU.
I/tflite: Initialized TensorFlow Lite runtime.
W/System.err: java.lang.IllegalArgumentException: Internal error: Failed to apply delegate: TfLiteGpuDelegate Init: CONV_2D: Unsupported data type for float32 tensor
W/System.err: TfLiteGpuDelegate Prepare: delegate is not initialized
    Node number 36 (TfLiteGpuDelegateV2) failed to prepare.
    Restored previous execution plan after delegate application failure.
W/System.err:     at org.tensorflow.lite.NativeInterpreterWrapper.applyDelegate(Native Method)
        at org.tensorflow.lite.NativeInterpreterWrapper.applyDelegates(NativeInterpreterWrapper.java:318)
W/System.err:     at org.tensorflow.lite.NativeInterpreterWrapper.init(NativeInterpreterWrapper.java:82)
        at org.tensorflow.lite.NativeInterpreterWrapper.<init>(NativeInterpreterWrapper.java:63)
        at org.tensorflow.lite.Interpreter.<init>(Interpreter.java:237)
        at it.unipr.advmobdev.whereiswally.ModelExecutor.loadInterpreter(ModelExecutor.java:245)
        at it.unipr.advmobdev.whereiswally.ModelExecutor.run(ModelExecutor.java:160)

I noticed that in this commit was added the setQuantizedModelsAllowed() method in the GpuDelegate.Options class. This method is not available in my current version of the library.
Is it possible that the optimized / quantized model cannot be run with the GPU delegate? I was unable to find this information in the documentation.

Could you try nightly? It seems that your model works well with GPU on master.
dependencies {
    implementation 'org.tensorflow:tensorflow-lite:0.0.0-nightly'
    implementation 'org.tensorflow:tensorflow-lite-gpu:0.0.0-nightly'
}

I changed the version of TensorFlow Lite dependencies, now they are:

org.tensorflow:tensorflow-lite:0.0.0-nightly
org.tensorflow:tensorflow-lite-gpu:0.0.0-nightly

With this update, the model works properly with the GPU but performance is worse: latency with GPU is remained pretty much the same but the time required on the CPU is more than twenty times greater than before.
I report you the latency measured for the execution of the model on a 256x256 image.

Model
TFLite version
CPU
GPU

Normal (8.6 MB)
2.2.0
1.790 s
0.404 s

Optimized (3.4 MB)
2.2.0
1.023 s
Not working

Normal
0.0.0-nightly
37.828 s
0.412 s

Optimized
0.0.0-nightly
27.643 s
0.423 s

Why isn't the latency of the optimized model running on GPU better than that of the normal model? And most importantly, why is CPU latency with the 0.0.0-nightly version worse?

You can use benchmark tool with "--enable_op_profiling=true" to check how it's executed.
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/tools/benchmark
It seems that there is a regression on nightly. (I didn't see it when I tested)
What you said optimized model should be integer quantization.
https://www.tensorflow.org/lite/performance/post_training_quantization
Integer quantization is mainly for CPU since CPU calculates integer better than float number. When you run it on GPU, it converts integer to float internally. So there is no much benefit except the size of model.

Thank you for your answer and sorry for the late reply.

Integer quantization is mainly for CPU since CPU calculates integer better than float number. When you run it on GPU, it converts integer to float internally. So there is no much benefit except the size of model.

I didn't know this detail. From now, I won't execute an optimized model on a GPU.
Since I think the problem is solved, I close the issue.
PS: I think I will use the normal model with the 2.2.0 version of TFLite in my application because I obtained better results with it.


Transaction id wrapped in txId

After a contract call, the tx result in onFinish is txid.txid
It is unclear why when this changed, but my app breaks now.

I fixed this, so it will be included in our release this coming week.

It was fixed in this PR and merged, but not release yet: https://github.com/hirosystems/stacks-wallet-web/pull/2159/files#r791218440

Is there a test for that? That this doesn't happen in the future again :-)

@timstackblock was a test for this added by chance with your new integration tests? If not, I can make an issue to make sure we add one.

I do not believe we are specifically looking for txid.txid a test to verify that assertion is still needed.

Added issue here: https://github.com/hirosystems/stacks-wallet-web/issues/2222


Clear lap and reset button should only work when we have atleast one lap/record in the list.

Before Changes
https://user-images.githubusercontent.com/82964553/195353520-a365b95a-8e7b-4e47-87c2-96c5b2f43184.mp4
After changes
https://user-images.githubusercontent.com/82964553/195353327-3ff32b6d-021a-4f1c-bb12-8196f9d56b05.mp4

Please Assign this issue to me, I have made the Changes.
Ready to make a pull request.


[e2e]: go schedule main config-ldflags-main-dir slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5175936692
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Trigger: schedule
Branch: main
Date: Mon Jun  5 10:19:24 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5175936692
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Trigger: schedule
Branch: main
Date: Tue Jun  6 01:48:27 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5187322221
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Trigger: schedule
Branch: main
Date: Tue Jun  6 10:25:05 UTC 2023
Tests are passing now. Closing this issue.


Unable to get either report to connect to MSDN Azure subscription

Hi, this maybe a noob issue but I'm unable to get either report to connect to my MSDN Azure subscription to test them out.
The Governance Dashboard will only connect to the free Azure trial subscription I created against my organizational account.  The same account is added as a guest user to my MSDN subscription, but I don't see a way of "redirecting" the report to pick up that subscription?
The Infra dashboard, I enter the tenant name and Global, but it won't connect either (Something went wrong, object reference not set to an instance of an object).
I've been through the setup, ensured that the correct permissions are in place and the troubleshooting guide. Even tried on 2 different machines with the same result.  Any help would be appreciated

Update.  The Infrastructure repot now works, I didn't make any changes but it magically started to work, however the Azure Regions with Resource Usage map is entirely incorrect, highlighted regions all over the globe where I have no resources!
The Govenance report still will not work:
Steps Taken - Open freshly download Azure Givernance Dashboard file - Prompts to select AzureKind - Global
Click Load
Prompt - The credentials provided are invalid - choose sign in as different user - Enter credentials - hit connect
Something went wrong - Object reference not set to an instance of an object
Press cancel approx 15 times
The 2 main errors are
OLE DB or ODBC error: Exception from HRESULT: 0x80040E4E
OLE DB or ODBC error: Object reference not set to an instance of an object..
In Data source settings I only see Connect CCO Dashboard to your Azure subscription and https://management.azure.com
Tried modifying the Privacy settings for each of those to Organisational - already set the global privacy setting to ignore privacy settings
Some screen shots

If I cancel out of each box and refresh manually and choose to apply query changes it loads some more, but still fails

Doing the same again, it seems to get a bit further then fails (doesn't get any further than this)

Update.  The Infrastructure repot now works, I didn't make any changes but it magically started to work, however the Azure Regions with Resource Usage map is entirely incorrect, highlighted regions all over the globe where I have no resources!
The Govenance report still will not work:
Steps Taken - Open freshly download Azure Givernance Dashboard file - Prompts to select AzureKind - Global
Click Load
Prompt - The credentials provided are invalid - choose sign in as different user - Enter credentials - hit connect
Something went wrong - Object reference not set to an instance of an object
Press cancel approx 15 times
The 2 main errors are
OLE DB or ODBC error: Exception from HRESULT: 0x80040E4E
OLE DB or ODBC error: Object reference not set to an instance of an object..
In Data source settings I only see Connect CCO Dashboard to your Azure subscription and https://management.azure.com
Tried modifying the Privacy settings for each of those to Organisational - already set the global privacy setting to ignore privacy settings
Some screen shots

If I cancel out of each box and refresh manually and choose to apply query changes it loads some more, but still fails

Doing the same again, it seems to get a bit further then fails (doesn't get any further than this)

Hi @wayno4
Regarding the Infra Dashboard, can you check if there's a relationship ** *:1 ** between the tables ComputeNetStorageUsage and Azure regions for Usage? Because if it's showing all the regions maybe the relationship is missing.
Here my example:

Hi @wayno4
Regarding the Infra Dashboard, can you check if there's a relationship ** *:1 ** between the tables ComputeNetStorageUsage and Azure regions for Usage? Because if it's showing all the regions maybe the relationship is missing.
Here my example:

If we talk about the "Something went wrong" error,  honestly we have no explanation for that. I only know that is produced by how PBI is recognizing the custom connector (we're in process to certify the connector) if you take a look to the tables are still being loaded.
It appearerd randomly since the last PBI update. In my case is not happening but, one of my colleagues told me that after checking the requirements again, forcing the update from the MS Store and canceling the error, the dashboard finally works. Sadly there's nothing we can do from our side. Sorry for the inconvenience.

If we talk about the "Something went wrong" error,  honestly we have no explanation for that. I only know that is produced by how PBI is recognizing the custom connector (we're in process to certify the connector) if you take a look to the tables are still being loaded.
It appearerd randomly since the last PBI update. In my case is not happening but, one of my colleagues told me that after checking the requirements again, forcing the update from the MS Store and canceling the error, the dashboard finally works. Sadly there's nothing we can do from our side. Sorry for the inconvenience.

Regarding the Infra Dashboard, there is no relationship between the tables ComputeNetStorageUsage and Azure regions for Usage.  Which fields should the relationship be mapped against?
The dashboard when it's loaded is pulling data from the Azure Regions and Resources tables, which seems completely different to your screenshot?!?!

Ref the Governance dashboard - the error prevents any data from loading at all.  I cancel out and there is no data loaded into any table.  I've updated PowerBI previously to no avail

Regarding the Infra Dashboard, there is no relationship between the tables ComputeNetStorageUsage and Azure regions for Usage.  Which fields should the relationship be mapped against?
The dashboard when it's loaded is pulling data from the Azure Regions and Resources tables, which seems completely different to your screenshot?!?!

Ref the Governance dashboard - the error prevents any data from loading at all.  I cancel out and there is no data loaded into any table.  I've updated PowerBI previously to no avail

Check the last PR(https://github.com/Azure/ccodashboard/pull/145) pending of approval. This problem is fixed

Check the last PR(https://github.com/Azure/ccodashboard/pull/145) pending of approval. This problem is fixed

Great the Infrastructure Dashboard now works due to the latest update :-)
The Governance Dashboard, I've had a look in more detail and appears to be failing on the Blueprints with a 404 error
[{"Id":"1.1","Query":"Blueprints","Step":"blueprints","Category":"Evaluation","DataSourceKind":null,"Operation":"Evaluation","StartTime":"2021-02-01T01:02:05.3122305Z","EndTime":"2021-02-01T01:02:05.4171921Z","ExclusiveDurationPct":0.0,"ExclusiveDuration":"00:00:00","Resource":null,"DataSourceQuery":null,"AdditionalInfo":null,"RowCount":null,"ContentLength":null,"IsUserQuery":true,"Path":"0","GroupId":0,"PartitionKey":null},{"Id":"1.1","Query":"Blueprints","Step":"blueprints","Category":"Document Evaluator","DataSourceKind":null,"Operation":"Document evaluation","StartTime":"2021-02-01T01:02:05.3122305Z","EndTime":"2021-02-01T01:02:05.4166088Z","ExclusiveDurationPct":0.26390222710019662,"ExclusiveDuration":"00:00:00.0276996","Resource":null,"DataSourceQuery":null,"AdditionalInfo":{"Identity":null},"RowCount":null,"ContentLength":null,"IsUserQuery":true,"Path":"0/1","GroupId":1,"PartitionKey":null},{"Id":"1.1","Query":"Blueprints","Step":"blueprints","Category":"Evaluator","DataSourceKind":null,"Operation":"Query evaluation","StartTime":"2021-02-01T01:02:05.3399301Z","EndTime":"2021-02-01T01:02:05.4169376Z","ExclusiveDurationPct":0.038634129052910783,"ExclusiveDuration":"00:00:00.0040551","Resource":null,"DataSourceQuery":null,"AdditionalInfo":null,"RowCount":null,"ContentLength":null,"IsUserQuery":true,"Path":"0/2","GroupId":2,"PartitionKey":null},{"Id":"1.1","Query":"Blueprints","Step":"blueprints","Category":"Evaluator","DataSourceKind":null,"Operation":"Query evaluation","StartTime":"2021-02-01T01:02:05.3439718Z","EndTime":"2021-02-01T01:02:05.4167686Z","ExclusiveDurationPct":0.1581778478986601,"ExclusiveDuration":"00:00:00.0166026","Resource":null,"DataSourceQuery":null,"AdditionalInfo":null,"RowCount":null,"ContentLength":null,"IsUserQuery":true,"Path":"0/2/3","GroupId":3,"PartitionKey":"Blueprints/blueprints"},{"Id":"1.1","Query":"Blueprints","Step":"blueprints","Category":"Data Source","DataSourceKind":"CcoDashboardAzureConnector","Operation":"Get Response","StartTime":"2021-02-01T01:02:05.3599814Z","EndTime":"2021-02-01T01:02:05.399072Z","ExclusiveDurationPct":0.37242763067636164,"ExclusiveDuration":"00:00:00.0390906","Resource":"{"azureKind":"global"}","DataSourceQuery":"Request:\r\nGET https://management.azure.com/tenants?api-version=2019-11-01 HTTP/1.1\r\n\r\n\r\n\r\n\r\nResponse:\r\nHTTP/1.1 200 OK\r\nContent-Length: 286\r\n\r\n\r\n","AdditionalInfo":{"Use Cache":"True","Use Buffer":"True"},"RowCount":null,"ContentLength":286,"IsUserQuery":true,"Path":"0/2/3/4","GroupId":4,"PartitionKey":"Blueprints/blueprints"},{"Id":"1.1","Query":"Blueprints","Step":"blueprints","Category":"Preview","DataSourceKind":null,"Operation":"Generate preview","StartTime":"2021-02-01T01:02:05.399665Z","EndTime":"2021-02-01T01:02:05.4169242Z","ExclusiveDurationPct":0.01847342266123992,"ExclusiveDuration":"00:00:00.0019390","Resource":null,"DataSourceQuery":null,"AdditionalInfo":null,"RowCount":null,"ContentLength":null,"IsUserQuery":true,"Path":"0/2/5","GroupId":5,"PartitionKey":null},{"Id":"1.1","Query":"Blueprints","Step":"blueprints","Category":"Data Source","DataSourceKind":"CcoDashboardAzureConnector","Operation":"Get Response","StartTime":"2021-02-01T01:02:05.4010415Z","EndTime":"2021-02-01T01:02:05.4163617Z","ExclusiveDurationPct":0.14367159037209798,"ExclusiveDuration":"00:00:00.0150800","Resource":"{"azureKind":"global"}","DataSourceQuery":"Request:\r\nPOST https://management.azure.com/providers/Microsoft.Management/getEntities?api-version=2018-03-01-preview&%24filter=name eq '2dd87c2e-b472-46a8-92f4-f69efd66f0ba' HTTP/1.1\r\n\r\n\r\n\r\n\r\nResponse:\r\n\r\n\r\n","AdditionalInfo":{"Use Cache":"True","Use Buffer":"True","Exception":"Exception:\r\nExceptionType: System.Net.WebException, System, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089\r\nMessage: The remote server returned an error: (404) Not Found.\r\nStackTrace:\n   at Microsoft.Mashup.Engine1.Library.Common.WrappingHttpWebRequest.WrapResponse(Func1 getResponse)\r\n   at Microsoft.Mashup.Engine1.Library.Common.WrappingHttpWebRequest.GetResponse()\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.GetWebResponse(WebRequest webRequest)\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.CreateResponse(ResourceCredentialCollection credentials)\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.GetResponseCore(ResourceCredentialCollection credentials)\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.<>c__DisplayClass109_0.<GetResponse>b__0()\r\n   at Microsoft.Mashup.Engine1.Library.Common.RetryPolicy.Execute[TResult](IEngineHost host, Func1 func)\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.GetResponse(ResourceCredentialCollection credentials, RetryPolicy retryPolicy, SecurityExceptionCreator securityExceptionCreator, Boolean tokenRefreshed)\r\n\r\n\r\n"},"RowCount":null,"ContentLength":null,"IsUserQuery":true,"Path":"0/2/5/6","GroupId":6,"PartitionKey":"Blueprints/blueprints"},{"Id":"1.1","Query":"Blueprints","Step":"blueprints","Category":"Other","DataSourceKind":null,"Operation":"Library: Common: RetryPolicy: TryExecuteAction","StartTime":"2021-02-01T01:02:05.4159244Z","EndTime":"2021-02-01T01:02:05.4160871Z","ExclusiveDurationPct":0.0015500906998368927,"ExclusiveDuration":"00:00:00.0001627","Resource":null,"DataSourceQuery":null,"AdditionalInfo":{"Exception":"Exception:\r\nExceptionType: System.Net.WebException, System, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089\r\nMessage: The remote server returned an error: (404) Not Found.\r\nStackTrace:\n   at Microsoft.Mashup.Engine1.Library.Common.WrappingHttpWebRequest.WrapResponse(Func1 getResponse)\r\n   at Microsoft.Mashup.Engine1.Library.Common.WrappingHttpWebRequest.GetResponse()\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.GetWebResponse(WebRequest webRequest)\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.CreateResponse(ResourceCredentialCollection credentials)\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.GetResponseCore(ResourceCredentialCollection credentials)\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.<>c__DisplayClass109_0.<GetResponse>b__0()\r\n   at Microsoft.Mashup.Engine1.Library.Common.RetryPolicy.Execute[TResult](IEngineHost host, Func1 func)\r\n\r\n\r\n"},"RowCount":null,"ContentLength":null,"IsUserQuery":true,"Path":"0/2/5/6/7","GroupId":7,"PartitionKey":"Blueprints/blueprints"},{"Id":"1.1","Query":"Blueprints","Step":"blueprints","Category":"Evaluator","DataSourceKind":null,"Operation":"Error ","StartTime":"2021-02-01T01:02:05.4161122Z","EndTime":"2021-02-01T01:02:05.4171921Z","ExclusiveDurationPct":0.00087174738189966616,"ExclusiveDuration":"00:00:00.0000915","Resource":null,"DataSourceQuery":null,"AdditionalInfo":{"Exception":"Exception:\r\nExceptionType: System.Net.WebException, System, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089\r\nMessage: The remote server returned an error: (404) Not Found.\r\nStackTrace:\n   at Microsoft.Mashup.Engine1.Library.Common.WrappingHttpWebRequest.WrapResponse(Func1 getResponse)\r\n   at Microsoft.Mashup.Engine1.Library.Common.WrappingHttpWebRequest.GetResponse()\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.GetWebResponse(WebRequest webRequest)\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.CreateResponse(ResourceCredentialCollection credentials)\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.GetResponseCore(ResourceCredentialCollection credentials)\r\n   at Microsoft.Mashup.Engine1.Library.Web.Request.<>c__DisplayClass109_0.<GetResponse>b__0()\r\n   at Microsoft.Mashup.Engine1.Library.Common.RetryPolicy.Execute[TResult](IEngineHost host, Func1 func)\r\n\r\n\r\n"},"RowCount":null,"ContentLength":null,"IsUserQuery":true,"Path":"0/8","GroupId":8,"PartitionKey":"Blueprints/blueprints"},{"Id":"1.1","Query":"Blueprints","Step":"blueprints","Category":"Trace Gaps","DataSourceKind":null,"Operation":"Trace Gaps","StartTime":"2021-02-01T01:02:05.4169376Z","EndTime":"2021-02-01T01:02:05.4171781Z","ExclusiveDurationPct":0.0022913141567963906,"ExclusiveDuration":"00:00:00.0002405","Resource":null,"DataSourceQuery":null,"AdditionalInfo":null,"RowCount":null,"ContentLength":null,"IsUserQuery":true,"Path":"0/8/9","GroupId":0,"PartitionKey":null}]

Just had another look at this and if I disable loading of the CC Get Entities and the Blueprints, it works.  However, of course there is a lot of data missing from the report - Hierarchy Overview and Azure Blueprints tabs are blank


coc -> Rename doesn't work across files

There's an issue where a coc-rename over multiple files fails with [coc.nvim] Error on applyEdits: Cannot read property 'uri' of null
To reproduce, create the following files:
// MyFile1.res

let a = 1

// MyFile2.res

let b = MyFile1.a

Then try to coc-rename MyFile1.a to MyFile1.something, and it will first ask to load 1 file from disk, and then fail with the error message mentioned above.
My wild guess is that it's an issue with how coc interprets the rename command result. Either our LSP is not sticking to certain protocol conventions, or coc-vim is at fault. Hard to debug.

It seems it happens because coc doesn't handle . (current path) in URIs. I think per standard it's okay to have . in uri and it should be fixed in coc rathar than our side (not sure tho).
to confirm this you can do a little hack in server to see if it works or not.
after else block in this line insert this code to remove . from URIs
    locations = locations.map(x => ({
      ...x,
      uri: x.uri
        .split('/')
        .filter(c => c !== '.')
        .join('/'),
    }));

Yeah that worked... okay, gonna report that one on the coc-vim repo then.

Okay, just so we know what we are talking about, this is how the paths look like for my specific rename query:
[
    {
        "uri": "file:///Users/ryyppy/Projects/rescript-association/vim-rescript/examples/rescript-project/././src/polySyntax.res",
        "range": {
            "start": {
                "line": 8,
                "character": 28
            },
            "end": {
                "line": 8,
                "character": 32
            }
        }
    },
    {
        "uri": "file:///Users/ryyppy/Projects/rescript-association/vim-rescript/examples/rescript-project/./src/recordExample.res",
        "range": {
            "start": {
                "line": 9,
                "character": 11
            },
            "end": {
                "line": 9,
                "character": 15
            }
        }
    },
    {
        "uri": "file:///Users/ryyppy/Projects/rescript-association/vim-rescript/examples/rescript-project/./src/recordExample.res",
        "range": {
            "start": {
                "line": 1,
                "character": 12
            },
            "end": {
                "line": 1,
                "character": 16
            }
        }
    }
]

The first uri looks weird though: ././src/....

Yeah that worked... okay, gonna report that one on the coc-vim repo then.

I've opened an issue https://github.com/neoclide/coc.nvim/issues/3104

The first uri looks weird though: ././src/....

Yeah, worth checking what's happening in analysis binary

This is fixed on coc.nvim's master.

question is when this will be shipped in a new release.. not sure how to use that master version via Plug, and if we should provide some instructions on how to do that for the meantime.

this installs coc from master
Plug 'neoclide/coc.nvim', {'branch': 'master', 'do': 'yarn install --frozen-lockfile'}

I suggest to wait until it get released, they release regulary so it should be shipped soon.

This is released on release branch

okay nice, just tried it. Works like a charm.


fix: very_good test doesn't like [ in file names

Description
very_good test fails when files have [] in their filenames (as seen in Dart Frog dynamic route files).
Steps To Reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected Behavior
A clear and concise description of what you expected to happen.
Screenshots

Additional Context
Add any other context about the problem here.

Given that this is a P1 (highest priority), anyone taking this must be ready to respond to feedback quickly to make sure it gets pushed.

I was not able to reproduce the error under latest version (0.11.0+1) not (0.10.0) with the following minimal sample.
@KyleFin do you have a minimal sample that reproduces this error?

Reproduced issue with minimal sample code under version 0.11.0+1 .

@KyleFin the issue seems to be within the optimisation process. In the meantime you can use:
very_good test --no-optimization

I've looked into very_good_test_runner which runs the flutter test command _flutterTest and the test succeeds when there is are [ and ] characters in the name.  Although the optimisation step shows to be completing successfully its generation dislikes the fact that a test file has [ or ] characters in test file names.


Add : possibility to get hover result in preview window

This add an option g:LSP_Hover_Reply_In_Preview which permit to see hover result in a preview window.

I have modified option for show hover reply in preview window for use the new lspOptions.

though looks a bit tradition way :-)
but if so previous discussed pr#4 which adding 'option' on hover popup now is not req, preview perhaps looks more clean.

line#341 :       if !reply.result.contents.kind->empty()
// E715: Dictionary required

and seems syntax would not immediately update when if ft (in/of this preview) was 'markdown' .........

line#341 :  if !reply.result.contents.kind->empty()
// E715: Dictionary required

I try to solve this with #25


sync with main branch

Docs Build status updates of commit 83cf345:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-7.0.0/release-notes-azureps.md
:bulb:Suggestion
View (azps-7.0.0)
Details

docs-conceptual/azps-7.0.0/migrate-az-7.0.0.md
:white_check_mark:Succeeded
View (azps-7.0.0)

docs-conceptual/azps-7.0.0/release-notes-azureps.md

Line 100, Column 152: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'string' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

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 docs.microsoft.com contributor guidesPost your question in the Docs support channel


PSR2 standard fixes

PSR2 standard fixes

To fix PSR-2 coding style automatically, I think it can integrate the StyleCI service, PHP_CodeSniffer or PHP-CS-Fixer coding style check tools.
References

https://styleci.io
https://github.com/squizlabs/PHP_CodeSniffer
https://github.com/FriendsOfPHP/PHP-CS-Fixer


Cannot complete commissioning flow when running apps on macOS

Problem
CHIP Tool cannot finish commissioning flow on macOS:
https://gist.github.com/erjiaqing/158f8546b7d5563f95aaf96df48183aa

@bzbarsky-apple @pan-apple PTAL, thanks.

After running under debugger, seems this is caused by failure of setting up BLE advertisement on macOS, maybe we should disable it by default?

So this is for the case when the client app:

Was not compiled with chip_config_network_layer_ble=false.
Is attempting to be commissioned via on-network commissioning.

right?
Ideally our client example apps would support being commissioned via either the IP network or BLE in their "linux" configuration (which is what's being used here).

So this is for the case when the client app:

Was not compiled with chip_config_network_layer_ble=false.
Is attempting to be commissioned via on-network commissioning.

right?

Right

Ideally our client example apps would support being commissioned via either the IP network or BLE in their "linux" configuration (which is what's being used here).

However, when it failed to setup BLEAdvertisement, it will close commissioning window. I think either a failure asseration or continue with IP commissioning is acceptable.

The thing is, if it's compiled with chip_config_network_layer_ble=true then for me it succeeds in setting up the BLE advertisement, at first glance.... Which is why all the "compile and run locally" bits use chip_config_network_layer_ble=false.

This seems to be working now


[stdlib] Better diagnosis for unavailable APIs

WARNING: DO NOT MERGE THIS before #3040
What's in this pull request?
Added test cases for expected-error and fix-its.
Changes to stdlib code:

Add arguments signature regardless that is the same as before.
Because the error message looks more natural.
e.g. "makeIterator" => "makeIterator()",
"replaceSubrange" => "replaceSubrange(_:with:)"
compare:
'generate()' has been renamed to `makeIterator`
'generate()' has been renamed to `makeIterator()`

Any${ExistentialCollection}.underestimateCount was a method, not computed property.

LazySequenceType has been renamed to LazySequenceProtocol, but not
LazyCollectionProtocol

Streamable.writeTo(_:) had no argument label.

Fixed typo in print() and debugPrint() error message (not working for now)

Repeated.init(): renamed: "..." to message: "..." because the argument
order has changed.

Marked public for some unavailable method on Sequence

Sequence.split(_:maxSplit:allowEmptySlices) has been replaced with
split(separator:maxSplit:omittingEmptySubsequences:),
not split(separator:omittingEmptySubsequences:isSeparator:)

Sequence.startsWith(_:isEquivalent:) or startsWith(_:) had no label on
the first argument.

transcode(_:_:_:stopOnError), not transcode(_:_:_:stoppingOnError)

Remove mutating methods from UnsafePointer.
alloc(_:), dealloc(_:), setter:memory, initialize(_:), destroy(),
and destroy(_:)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

@swift-ci Please smoke test

#3040 fixes these issues.
/Users/buildnode/jenkins/workspace/swift-PR-osx-smoke-test/swift/test/1_stdlib/Renames.swift:74:129: error: expected no fix-its; actual fix-it seen: {{5-16=sort}} {{17-17=isOrderedBefore: }}
  c.sortInPlace { _, _ in true } // expected-error {{'sortInPlace' has been renamed to 'sort(isOrderedBefore:)'}} {{5-16=sort}} {{none}}
                                                                                ^~~~~~~
                                                                                {{5-16=sort}} {{17-17=isOrderedBefore: }}
/Users/buildnode/jenkins/workspace/swift-PR-osx-smoke-test/swift/test/1_stdlib/Renames.swift:88:114: error: expected no fix-its; actual fix-it seen: {{9-16=index}} {{17-17=where: }}
  _ = c.indexOf { _ in true } // expected-error {{'indexOf' has been renamed to 'index(where:)'}} {{9-16=index}} {{none}}
                                                                                ^~~~~~~
                                                                                {{9-16=index}} {{17-17=where: }}
/Users/buildnode/jenkins/workspace/swift-PR-osx-smoke-test/swift/test/1_stdlib/Renames.swift:388:129: error: expected no fix-its; actual fix-it seen: {{9-19=min}} {{20-20=isOrderedBefore: }}
  _ = x.minElement { _, _ in true } // expected-error {{'minElement' has been renamed to 'min(isOrderedBefore:)'}} {{9-19=min}} {{none}}
                                                                                ^~~~~~~
                                                                                {{9-19=min}} {{20-20=isOrderedBefore: }}
/Users/buildnode/jenkins/workspace/swift-PR-osx-smoke-test/swift/test/1_stdlib/Renames.swift:389:129: error: expected no fix-its; actual fix-it seen: {{9-19=max}} {{20-20=isOrderedBefore: }}
  _ = x.maxElement { _, _ in true } // expected-error {{'maxElement' has been renamed to 'max(isOrderedBefore:)'}} {{9-19=max}} {{none}}
                                                                                ^~~~~~~
                                                                                {{9-19=max}} {{20-20=isOrderedBefore: }}
/Users/buildnode/jenkins/workspace/swift-PR-osx-smoke-test/swift/test/1_stdlib/Renames.swift:391:155: error: expected fix-it not seen; actual fix-its: {{9-19=starts}}
  _ = x.startsWith([]) { _ in true } // expected-error {{'startsWith(_:isEquivalent:)' has been renamed to 'starts(with:isEquivalent:)'}} {{9-19=starts}} {{20-20=with: }} {{none}}
                                                                                ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
                                                                                {{9-19=starts}}

--

@rintaro Thank you very much for the fixes and the tests!  Please merge when all the pieces are ready.

#3040  landed.
@swift-ci Please test

@swift-ci Please test


VSSDK.IDE.12 12.0.4

Type: Incomplete
Summary:
VSSDK.IDE.12 12.0.4
Details:
Declaring Other
Resolution:
Nuspec file in package:
This is a metadata-only package used as a dependency for reference packages and extensions that target Visual Studio 2013 and newer.
Affected definitions:

VSSDK.IDE.12 12.0.4

@AE49 I'm not seeing license or source information. Can you point me to it? If there isn't any, should we update the PR to NONE?

@capfei I updated to NONE (probably the better option given no actual license file although it may be covered under VS 2013 and newer).

@AE49 Thank you for the update!


Limit "Select the inputfields where the extra fields should be attached" to appropriate types

I think it would be nicer if only inputfields relevant to images (and perhaps fields if they worked) were listed, rather than all inputfield types.

I agree but I have no idea how to achieve this. At the moment I loop over all modules, grep all with the word Inputfield. All I now about a module is: autoload, singular and file. I could exclude all known modules (long, long list) to minimize the list, but there will be some custom modules I don't recognized as well as new ones.

Instead of what you have:
foreach (wire('modules') as $module) {
    if (preg_match('/^Inputfield/', $module)) {
         $field->addOption($module->name, $module->name);
    }
}

Do this instead:
foreach(wire('fields') as $fieldoption) {
     // filter out incompatible field types
     if($fieldoption->type instanceof FieldtypeImage)  $f->addOption($fieldoption->name);
 }

Nope, this doesn't work, custom fieldtypes like CroppableImage are missing and the name is just "images" but I need "InputfieldImage".

Sorry, my mistake, try this:
    foreach(wire('fields') as $fieldoption) {
      if($fieldoption->type instanceof FieldtypeImage)  $field->addOption($fieldoption->inputfieldClass);
    }

That gets me InputfieldImage, InputfieldCropImage, and InputfieldCroppableImage so it think it should do what you need.
However, I personally wonder whether it would be better to limit by the field names. I don't actually want ImageExtra applied to all fields of a particular InputfieldType. I usually want to limit it to just specific fields. The first approach I listed above would allow that, but obviously you would need to tweak:
$fieldNameFields = wire('fields')->find('inputfieldClass=' . implode('|', $this->selectedInputfields));

to get the fields via selected names, rather than inputfieldtypes.
Hope that helps.

PS My first approach returns CroppableImage fields for me

My fault. I didn't get you right. You want to change the list from general inputfield types to specific existing fields. For that scenario your first approach will work for sure. I thought the goal is to limit the list of inputfield types so it only displays "InputfieldImage" and "InputfieldCroppableImage" as option for example.
Some questions on this. Why do you want to change this behaviour? If I add the optional extra fields for all InputfieldImage fields there are probably some fields which won't need them. But it doesn't bother me because it's just an collapsed fieldset which doesn't effect the field if I keep the fields empty. On the other side it would annoy me if I had to go to the ImageExtra module settings every time after I created a new image field where I want the extra fields to be attached.
One idea is to combine both approaches. So every user can decide on his own whether to use the inputfield type approach or the single field one. I would combine them in an OR way. Doing it this way means also that an update of the module doesn't break anything.
What do you think?

You're probably right - it's only a couple of extra lines in the db table for each of the image fields and a collapsed fieldset.
I am just trying to make your module look cleaner - it's weird that currently you can select "InputfieldDateTime" for example - that just doesn't make any sense.
I would at least go for:
    foreach(wire('fields') as $fieldoption) {
      if($fieldoption->type instanceof FieldtypeImage)  $field->addOption($fieldoption->inputfieldClass);
    }

But then again, if you don't think it matters having the ImageExtra collapsed fieldset on the Input tab of all Image fields, then why even give users an option here - why not just add it to all image related inputfields?
Anyway, not a big deal for me either way :)

I'm completely with you to limit the list of available inputs because most of them doesn't make any sense.
The problem is, how do I get all available / installed modules which are an instance of FieldtypeImage. If I use your approach I just get the inputfield types for which a field already exists. If I've installed CroppableImage and didn't create a field of this type so far, CroppableImage woudn't be listed there. Maybe this is just an edge case...
But you're right, maybe it would be the best to remove the module settings and add it to all related inputfields...

Ah - yes, I didn't think about wire('fields') not working if there isn't already a field defined with the Inputfield type.
Anyway, it's not a huge deal, so will bow out now and leave it to your best judgement.

The extra fields (collapsed fieldset) will be added to all related fields (instances of FieldtypeImage) by default. It's no longer required to add the inputfields manually. Thanks!


[PVA] getting started blade should be updated to reflect PVA capabilities

KEEP: edit what your bot says
REMOVE: train your language model
REMOVE: publish to dev ops
SHOW: enable insights

@srinaath I thought that this was already taken care of but it looks like a regression. Can you confirm?

So fix: Empty Webchat inspector text and Disabling items in PVA context by srinaath · Pull Request #7241 · microsoft/BotFramework-Composer (github.com) I removed Add Connections, Add Packages and Enable Insights from the Get Started section. I could remove these items as well.
I didn't realize "Train your language model" and "Publish to devops" need to be removed from GetStarted section in PVA context. Can we not train the model with the PVA Default Recognizer?

@srinaath Yeah I remember that as well. The current ask is to hide those items and show the insights step.

I believe the POR was the disable the elements, not remove them entirely

Before we do this let's sync with the person who put together the ask

@cwhitten I reached out to @vivekkshankar  and @benbrown on this. On the Get Started section it is to actually hide the lements and not disable

fix: Resolve PVA check logic missing by srinaath · Pull Request #7369 · microsoft/BotFramework-Composer (github.com) Updated the PR description here. I had them hidden in the PR yesterday. Sorry.


Show years in player rating history

The player rating history shows dates below the graph which can get pretty useless if the history spans multiple years.

I would like to add that right now X is not a function of time, but a function of number of games. This is partly why it looks this strange.
While that is ok I guess, I would much rather prefer it being a function of time, or atleast have the option of doing it that way.
I want to see my rating being a straight line if I am not playing any games :D

This is basically the same as #935


Provide a way to extract an arbitrary subsequence of a sequence

Given a sequence [x1, ..., xn], we need a way to extract a subsequence of the elements at indices [i1, ..., ik].

Nice :+1:

Actually, I'm thinking that this exact operation could be used as an efficient basis for many other operations like drop, take and so on. That would require the sequences to be finite and to know their length at compile-time, but this might be a reasonable assumption to make throughout the library anyway. I'll research this further when I have more time.


PISA carl loss become very large then NaN at first several steps

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
Training PISA using large batch (512) across 64 gpu.
The carl loss become very large at the first step, then turns to NaN. And the whole loss is exploded.
Reproduction

What command or script did you run?

GPUS=64 ./tools/slurm_train.sh compute pisa_512 configs/pisa/pisa_mask_rcnn_r50_fpn_1x_coco.py my_workdir

Did you make any modifications on the code or config? Did you understand what you have modified?
set samples_per_gpu=16, workers_per_gpu=4.

What dataset did you use?
coco2017

Environment
Error traceback
If applicable, paste the error trackback here.
2020-10-08 07:54:08,867 - mmdet - INFO - Epoch [1][50/230]      lr: 1.978e-03, eta: 3:23:08, time: 4.498, data_time: 1.876, memory: 26963, loss_rpn_cls: 0.5290, loss_rpn_bbox: 0.0810, loss_cls: 1.5482, acc: 86.6658, loss_bbox: 3.5414, loss_carl: 70.6273, loss_mask: 0.7663, loss: 77.0933
2020-10-08 07:56:43,024 - mmdet - INFO - Epoch [1][100/230]     lr: 3.976e-03, eta: 2:48:01, time: 3.083, data_time: 0.344, memory: 26963, loss_rpn_cls: nan, loss_rpn_bbox: nan, loss_cls: nan, acc: 78.3295, loss_bbox: nan, loss_carl: nan, loss_mask: 2.5743, loss: nan
2020-10-08 07:58:54,010 - mmdet - INFO - Epoch [1][150/230]     lr: 5.974e-03, eta: 2:27:54, time: 2.621, data_time: 0.352, memory: 26963, loss_rpn_cls: nan, loss_rpn_bbox: nan, loss_cls: nan, acc: 30.3752, loss_bbox: nan, loss_carl: nan, loss_mask: 0.6888, loss: nan

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

Resolved, not related to codebase.

@yl-to Hello, how you solved this problem?

@yl-to Hello, how you solved this problem?

Check carefully for your parameters.

Hello, I have same problem, how did you solve it？


I wanna use Regift at OS X!

Hi, @matthewpalmer.
I add target for OSX and check the system version.

This is awesome! Thanks very much. One last thing—would you be able to update the podspec to say that it's available on iOS and Mac?
It's probably something like
  s.ios.deployment_target = '8.0'
  s.osx.deployment_target = '10.10'

Here's an example if needed: https://github.com/matthewpalmer/Locksmith/blob/master/Locksmith.podspec
Thanks very much :+1:

Nice, thanks! Will merge this in now


Table alignments are not working

According https://silverbullet.md/Markdown/Extensions I would expect tables alignment  to be supported by SilverBullet but it looks like they don't (is always left aligned regardless your MD text):
| Not specified alignment| Centered aligned | Aligned to the right |
| --- | :---: | ---: |
| Test1 | Test1 | Test1 |
| Test2 | Test2 | Test2 |

SilverBullet (docker installation, latest release available 0.7.6)

Same code, another MD editor (snap from obsidian):

Indeed, this has not been implemented yet.

I'm looking at this one, I found another issue with this example where missing | at the start/end of a line should still be parsed as a table.
Will open a PR to address the following:

[ ] parseRow does not handle colons : to indicate align HTML properties for table columns
[ ] parseRow does not support a mismatched number of pipes in data rows


Support recursion across all archive types

Recursion supported across all archive types by making crawl.Log4jIdentifier type no longer holds any concepts of specific archives or archive types and then reusing pretty much the same approach that was used before.
Instead, the following fields have been added to it:
ParseArchiveFormat func(string) (archive.FormatType, bool)
ArchiveWalkers     func(archive.FormatType) (archive.WalkerProvider, bool)

When a file or path is encountered, the ParseArchiveFormat function is used to determine if the file is an archive, and the archive type is passed along to the ArchiveWalkers function to retrieve an archive.WalkerProvider.
A new archive.WalkerProvider should support creating a WalkFn from either a file or a reader. This is handy because during the first walk, we have access to files on disk and so do not have to load anything into memory. When we are recursing into archives we do not have access to disk as everything is currently in memory, so a WalkerProvider must also provide a FromReader method to create one from an io.Reader.  Reading into memory is only required by the archive.ZipWalkers provider currently, which still has a configurable memory limit, as it did before this PR (this situation will be improved in https://github.com/palantir/log4j-sniffer/issues/40).

Needs changelog and conflict resolution


Plugins

An email requesting a feature outside the intended scope of Voice Notify gave me the idea for plugins.
Basically, as I think of it now, it takes scripts that can be edited directly in the app or placed in a plugins folder.
Those scripts can use basic functions such as system clock or timers to speak dynamic strings.
For example, announcing when the clock hits the next hour or, as the email suggested, announcing every 5 minutes that passes.
Please comment if you have any other ideas for plugins that you would like to see possible.
Plugins would likely be affected by VN's settings the same way notifications are affected.

I'd like the ability to have more intricate notification ignoring and rewriting.
Also your features (regex rewrite, quiet hours, etc) implemented as plugins would give you more flexibility to tweak those without worrying about introducing bugs in the main code. This would make implementing features and fixes easier for others and pull requests easier to inspect and test as well.
Examples

Ignoring notifications from a single chat user.
Ignoring all notifications from all but one user.
Rewriting a messages such as email subjects based on the source application, time, format, etc all at once.


Fix typo "self.localScreens.splce(idx, 1)" for the .splice function usage

there's a typo in the latest master branch library code:
self.localScreens = self.localScreens.splce(idx, 1)

There's a typo for .splce function, it should be .splice.
Corrected code:
self.localScreens = self.localScreens.splice(idx, 1);

fixed and published as 1.1.2 -- thanks!


The /admin route gives a 404 error on V3.0

Welcome to Ghost's GitHub repo! 👋🎉
We use GitHub only for bug reports 🐛
Anything else should be posted to https://forum.ghost.org 👫
🚨For support, help & questions use https://forum.ghost.org/c/help
💡For feature requests & ideas you can post and vote on https://forum.ghost.org/c/Ideas
If your issue is with Ghost CLI, please report it on the CLI repo ➡ https://github.com/TryGhost/Ghost-CLI/issues/new.
Issue Summary
A summary of the issue and the browser/OS environment in which it occurs.
To Reproduce

This is the first step
This is the second step, etc.

Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
Technical details:

Ghost Version:
Node Version:
Browser/OS:
Database:

Hey! This is expected behaviour in 3.x :grinning:
Previously the /admin route was a redirect to /ghost - which is where the admin has always lived - we've since removed the redirect as we're working toward a separation of the admin and the frontend.
You can still administer your site from /ghost :relaxed:

Fair enough

I can't reach the webhooks api, but site is working, is it related ? I use /ghost/api/v3/admin/webhooks and /ghost/api/v3/admin/site:
public interface GhostBlogAdminApiService {

    @Headers({
            "Accept: application/json; charset=utf-8",
            "Content-Type: application/json; charset=utf-8",
    })
    @GET("site")
    Call<GhostResponse.Admin.Site> getSite();

    @Headers({
            "Accept: application/json; charset=utf-8",
            "Content-Type: application/json; charset=utf-8",
    })
    @GET("webooks")
    Call<GhostResponse.Admin.Webhooks> getWebhooks();
}


3 Improvements

Hi,
I did 3 improvements here.

Adjusted the directive declaration to function with strict-di set;
Added a third function to the menuOptions array to disable items on the fly. If no function is present, the item is enabled by default;
Added a tag attribute to help people (like me) that use this context menu outside a ngRepeat context.

For people updating the plugin, there will be NO necessary changes to old code.
Hope you enjoy!

The second commit is for simplicity sake of item 2 above.
Instead of a function to DISable the menu item, a function to ENable.
This simple stuff can prevent a lot of headaches...

Good catch, oversight on my part.
Awesome!
Uhh... not sure this one is needed. If you are not in an ng-repeat, couldn't you just access it within the $scope variable of the controller like normal. It should be in scope. Unless I'm misunderstanding this.

Not really.
Let me show you my scenario:
I have 2 menus (collections) in my page.
One is loaded dynamically in angular (does not need the tag) and have its isolated scope.
The other is loaded on the server side and is written directly into the template (no ngRepeat).
In my application, they both have the same key to the item (an ID property). So I added the tag to both contexts and use the same item collection.

As the second list is written in the template, all contexts have the same scope, making impossible to identify the item.
The alternative would be to use menuOptions(item) instead, but then I find the tag option more explicit and elegant.

Sorry forgot to reply. I'm accepting this pull request, just going to be renaming contextMenuTag to model. Other than it will probably go in as is. Busy at work so will merge the pull request Tuesday and update the bower package.
Thanks again for writing this pull request, just wanted to give an update.

Sure, no problem. I just dont think model is a good name for it. We simple dont handle it as a model (binds and all). But its up to you.

@SuricateCan : Thanks, the feature for disabled entry was just what I was looking for.
I did some local modifications though: In the disabled case you apply no click handler, thus the default click handlers trigger when clicking on the disabled menu option, which was undesirable in my case.
Thus I added the click handler with preventDefault for the disabled case. Maybe you can amend that to your PR? I don't see how I can modify it.

Seems like creating a PR for your fork is/was the way to go? I'm still not sure :smile:
Anyway, here is what I tried to describe: https://github.com/SuricateCan/ui.bootstrap.contextMenu/pull/1

A pull request for a pull request. :laughing: I'll be putting this in tonight after some testing.
Thanks @kasoban!

Hey @Templarian I merged @kasoban changes.
You are good to go! :D

Thanks @kasoban!

Everything is merged and synced up. The latest version in bower is now 0.9.3.
Thanks again! These additions should be very useful for others.

Also, @SuricateCan I updated the demo with an example of the model for those not using ngRepeat. The demo is now on codepen.io.

Very nice! Thanks!


Revert the missing fix and add some logs

Cc @sebavan as it seems the iridescence tests are failing

Cc @sebavan as it seems the iridescence tests are failing

Can you try to merge master in ? I do not understand why master is not failing :-(

I updated it to see, still no explanations so far...

@RaananW would be great if you could have a look, I wonder why the config was from master but none of the iridescence code was available ? but at least it is all in :-)

@sebavan - I will look into that. The config is taken from the branch it's in, nothing should be taken from master.


fix: Check manifest.json exists before dllReferencePlugin

An error occurs when I ran webpack first time or the vendor has been changed:
× ｢wdm｣: Error: ENOENT: no such file or directory, open 'C:\my\project\node_modules.cache\autodll-webpack-plugin\development_instance_xxxxxxxx\vendor.manifest.json'

I'm having this issue as well


Split transition frontier persistence worker into separate process

The transition frontier persistence worker is designed to be in another process, and the architecture enables pulling it out already, but it's still in process right now. An RPC parallel worker should be created to wrap the existing code. For the most part, it will just work, but there are two new and important considerations: notifying the parent process of synchronization, and moving the ownership of the rocksdb connection to another process. For notifying the parent process that synchronization is complete, we could just have the RPC worker not return a response until it's done, but this could rub up against RPC parallel timeouts, and it keeps an open connection, so another solution may be preferable there. Moving the ownership of the rocksdb connection out to the other process is easy enough, but the daemon also will need to open and close it sometimes (only times when the worker is not running though). There should be some logic in code to help enforce this (like an exn that gets thrown when you try to create a db connection if the process hasn't closed yet).

We need this to not have death async cycles

Seeing 30-50 sec long async cycles currently when flushing 32 diffs at once. Doing this is now critical, since that is an unacceptable length for a long async cycle (even one that is triggered intermittently). Possibly, we should do https://github.com/CodaProtocol/coda/issues/3738 as well, but splitting this work off of the main process will be the larger win.


JSON button not working

@baimingze the jason button is not working properly. When I clicked the button nothing happen.

#69

About the JSON button. We found it's hard to do the on-the-fly transfer from original JSON data to the downloadable JSON/XML/TAB files, because of the transfering is limited to ASCII charactors, rather than the Unicode ones.
One alternative way is doing the transfer in the backend system, who give an URL to the frontend system to download.

@baimingze feel free to add the methods in the web service. the XML and the JSON one

done


Remove usage of sp package

Getting this note related to the ongoing deprecation of some spatial R packages:
The legacy packages maptools, rgdal, and rgeos, underpinning the sp package,
which was just loaded, will retire in October 2023.
Please refer to R-spatial evolution reports for details, especially
https://r-spatial.org/r/2023/05/15/evolution4.html.
It may be desirable to make the sf package available;
package maintainers should consider adding sf to Suggests:.
The sp package is now running under evolution status 2
     (status 2 uses the sf package in place of rgdal)

Looks like maybe just leaflet is using sp, but the current version on CRAN no longer depends on it.  renv::update("leaflet") might be all that needs to happen (hope for no breaking changes)


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/120464

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/120801


add custom server params

This PR add support to openai server with custom params: min_p and repetition_penalty. Although those are not official params, but i think it's useful to add those as extended params.

@zhuohan123 What do you think about adding this kind of non-OpenAI features to the OpenAI server?

Can we please make it happen? Repetition penalty is a very useful parameter to have and supporting it in the OpenAI server makes life a lot easier


Number of genes detected by htseq-count added to QC metrics

names changed to "no_genes_cufflinks" and "no_genes_htseq"

Looks good - can you please just change:
table = infile.split("/")[-1][:-len(".load")]
to:
table = P.toTable(infile)


Gracefully handle when CMakeCache.txt contains some non-utf8 chars

It seems CMakeCache.txt can contain non utf8 characters sometimes, as I realised when building libssh2-sys crate on an OSX 10.11 machine.
This patch ignores non-utf8 characters when parsing it. This shouldn't be an issue given the kind of logic we're doing on the content of CMakeCache.txt

Huh weird, I had no idea that could be the case...
Thanks though!