could not pass parameters into pyradiomics [FEAT EXTRACTION]

Describe the bug
I was trying to pass the force2D keyword arguments into pyradiomics. But it seems not making any difference in the output. I still see all the 3D shape features and don't see the 2D shape features.
i tried both hard-code and using a parameter file follow the code from https://www.radiomics.io/pyradiomicsnotebook.html
neither worked.
my guess was the parameter was not passed into featureextractor. In fact, i wasn't able to print the default settings. you can see that in the screen grab below.

Version (please complete the following information):

OS: windows 7
Python version: 3.6.8
PyRadiomics version 2.2.0

Thanks.

@Mathematinho,
That notebook represents a very old version of PyRadiomics (check the output, it containts the version: 1.0-post17)
As to the enabled settings, that variable is now called settings, similarly, enabled filters is now enabledImagetypes. Enabled features/classes are still in enabledFeatures.
Finally, setting force2D=True does not automatically switch your shape features to 2D, it is intended to deal with voxel anisotropy (by ignoring offsets in the slice direction, requiring only isotropic size in-plane).
For extracting in 2D (single slice, including 2D shape features): disable class shape and enable class shape2D, set force2D=True. Note that this only works if your segmentation also represents just 1 slice.
Finally, the settings that were enabled are also part of the standard diagnostic features in the output: diagnostics_Configuration_Settings

thank you @JoostJM. problem solved.


[ORA-1406] Handle error cases for SendCoinsFrom

The various SendCoins functions throw errors deep within the cosmos-sdk bank keeper. We should always be catching those errors and doing something correct to handle them.
This code changes the behavior of the payment of rewards functions in the case of an error: before it would try to pay participants in a topic up until an error occured then every participant in that topic processed after the error would not get paid
now if there's an error trying to send someone funds, they get no rewards but the next participant in the chain gets their rewards.

should I write a unit test that tests that if there is an error the next participant still gets paid? probably
do I want to? no


Prevent a user with a different email from accepting the team invite

Changes

Fixes the case where a logged in user can accept an email invitation even if their email address does not match the address in the invitation

What's the reason behind this change?
Suppose:

A user uses email<EMAIL_ADDRESS>They gets an invitation
They changes its email to<EMAIL_ADDRESS>They wants to accept the invitation

I guess it is expected behavior?

What's the reason behind this change?

This change fixes the case where a user is logged into a different account than expected and accepts the invitation. Since the invitation is issued for an explicit email, this check should be enforced.

I guess it is expected behavior?

I think the expected behavior is for the user to get a new invitation with the updated email. This allows the invitation to be more explicit about the invitation recipient's address.

What's the reason behind this change?
Suppose:
1. A user uses email [my-a<EMAIL_ADDRESS>
2. They gets an invitation

3. They changes its email to [my-b<EMAIL_ADDRESS>
4. They wants to accept the invitation

I guess it is expected behavior?

Since the email is sent to the original email address, we should limit the link just for that email address. If he changed his email, a new invitation link should be sent.

Agree to revert.


Rails 4 image_tag doesn't work on heroku

I have a ruby on rails 4 app. In my app/views/static_pages/home.html.erb I have an image tag which displays a logo when I run it locally on my own computer:
<%= image_tag "logo.png", width: "250px", height: "90px" %>

but when I run the code on heroku the image is missing. I tried all different kinds of solutions posted on the web, but found only two solutions. First using
config.assets.compile = true

which is not a good solution because it uses more memory and performs more poorly. Second using
<img src="<%= asset_path('logo.png') %>" width="250px" height="90px">

which sometimes work, but sometimes doesn't. Why is this happening? Is there a way to get the <%= image_tag ... %> working on heroku?
Thanks.

@rostaamm were you ever able to resolve this?

Would love to know myself

Having the same issue and haven't found a solution after hours of research...


callback for dismissing

Hi, author and contributors. Do you think it's possible that tooltip provides a callback invoked when it got dismissed? it's quite important when I try to combine it with redux and make it driven by state.

just realize it's duplicated with #3


feat: third party resource checksum

Check upload resources sha256 hash.

Hi @dashmage, I update the PR to address your comments. Can I have your review again?

Apart from the check sum -> checksum changes, I have a question on whether os_platform.py itself is necessary or an overkill. We could achieve the same functionality using distro.info() combined with a simple dictionary of ubuntu series mappings for the comparison of supported_series and the series of the machine being run on.

@jneo8 I've resolved my conversations, thanks for checking them out. If you could fix some of those typos brought up in the other reviews, it's a +1 from me.

Apart from the check sum -> checksum changes, I have a question on whether os_platform.py itself is necessary or an overkill. We could achieve the same functionality using distro.info() combined with a simple dictionary of ubuntu series mappings for the comparison of supported_series and the series of the machine being run on.

That is the implementation I try to avoid. Try to use declarative way, using enum to define UbuntuSeries, is really helpful to make new comer understand the code easily, make them more easy to add more support versions and avoid the possible error. Also os_platform provide a flexible to add more distro relate functions in the future. I mean compare the way right now to multiple if-else syntax + distro dictionary, I don't see benefit from it.

I left some inline comments about some typos.
Needs information: Do you think we should make use of the rest of the information in ToolVersionInfo? The validate_checksum function only uses sha256_checksum and supported_series data, maybe we should provide logging message about where to find the appropriate resource ?

Other information in ToolVersionInfo are version, link, and desc.
Logging about those information maybe a noise from my point of view.
Instead of logging, it will be nice if we can have those information in the document instead of logging, it's also the reason why I insist we should maintain our document in the repo before.

I left some inline comments about some typos.
Needs information: Do you think we should make use of the rest of the information in ToolVersionInfo? The validate_checksum function only uses sha256_checksum and supported_series data, maybe we should provide logging message about where to find the appropriate resource ?

Other information in ToolVersionInfo are version, link, and desc. Logging about those information maybe a noise from my point of view. Instead of logging, it will be nice if we can have those information in the document instead of logging, it's also the reason why I insist we should maintain our document in the repo before.

There will be no if-else statements. I agree with using enums for series information, you can disregard me mentioning a dict. However, can we declare the UbuntuSeries enum in checksum.py and replace this line with

Other information in ToolVersionInfo are version, link, and desc. Logging about those information maybe a noise from my point of view. Instead of logging, it will be nice if we can have those information in the document instead of logging, it's also the reason why I insist we should maintain our document in the repo before.

Are those information really necessary in the code if it's not used anywhere? It sound like those are supplementary information that can be put in a doc / discourse post, and then in the code put a link to the doc / discourse post.

I left some inline comments about some typos.
Needs information: Do you think we should make use of the rest of the information in ToolVersionInfo? The validate_checksum function only uses sha256_checksum and supported_series data, maybe we should provide logging message about where to find the appropriate resource ?

Other information in ToolVersionInfo are version, link, and desc. Logging about those information maybe a noise from my point of view. Instead of logging, it will be nice if we can have those information in the document instead of logging, it's also the reason why I insist we should maintain our document in the repo before.

There will be no if-else statements. I agree with using enums for series information, you can disregard me mentioning a dict. However, can we declare the UbuntuSeries enum in checksum.py and replace this line with something like -
distro.info()['version'] in info.supported_series (with appropriate formatting, if necessary). My main point of confusion is do we really need the OSPlatform class and get_os_platform function when we can directly get the series information from a simple function call distro.info()?

Other information in ToolVersionInfo are version, link, and desc. Logging about those information maybe a noise from my point of view. Instead of logging, it will be nice if we can have those information in the document instead of logging, it's also the reason why I insist we should maintain our document in the repo before.

Are those information really necessary in the code if it's not used anywhere? It sound like those are supplementary information that can be put in a doc / discourse post, and then in the code put a link to the doc / discourse post.

I will say yes otherwise you can't easily review the checksum & version information in the PR. Imagine you get a new PR which only include the checksum hash without those information.

I left some inline comments about some typos.
Needs information: Do you think we should make use of the rest of the information in ToolVersionInfo? The validate_checksum function only uses sha256_checksum and supported_series data, maybe we should provide logging message about where to find the appropriate resource ?

Other information in ToolVersionInfo are version, link, and desc. Logging about those information maybe a noise from my point of view. Instead of logging, it will be nice if we can have those information in the document instead of logging, it's also the reason why I insist we should maintain our document in the repo before.

There will be no if-else statements. I agree with using enums for series information, you can disregard me mentioning a dict. However, can we declare the UbuntuSeries enum in checksum.py and replace this line with something like - distro.info()['version'] in info.supported_series (with appropriate formatting/modifications, if necessary). My main point of confusion is do we really need the OSPlatform class and get_os_platform function when we can directly get the series information from a simple function call distro.info()?

We also using the platform package to get the information. I suppose you can't easily add those logic in your example.

I will say yes otherwise you can't easily review the checksum & version information in the PR. Imagine you get a new PR which only include the checksum hash without those information and how many trouble you will get if you are the maintainer.

Maybe the info would work as comments then? If it's not actively being used by the code, I don't think we should have those vars since it could be misleading.
And for the PR example, we can assume that ideally they should include the other supplementary info with the PR so that reviewers can verify.

I will say yes otherwise you can't easily review the checksum & version information in the PR. Imagine you get a new PR which only include the checksum hash without those information and how many trouble you will get if you are the maintainer.

Yeah, it will be convenient for the maintainer. But the difference is only whether you can see the version information on the same page v.s. you need to open another tab for the version information. It's not a blocker, I am okay with your current approach though.
I am trying to see if we can simplify the data structure that can improve the performance a bit. For example, without version information. We can simply have a map from series to list_of_supported_hashes, and we can avoid the for-loop. (Though, the loop is likely to be very small).

We also use the platform package to get the information. I suppose you can't easily add those logic in your example.

Yes, that is part of my point. If we drop the logic to get system info using platform package and just go with distro.info without checking whether it's an ubuntu system or not(because the charmhub page already mentions that the charm is supposed to be run on ubuntu), do we lose anything of value?

The reason I insist to put os platform as a single module also include I expected it will get more and more complex over time.

I do agree that it becomes more extendable as a module though.

Maybe the info would work as comments then? If it's not actively being used by the code, I don't think we should have those vars since it could be misleading.

And for the PR example, we can assume that ideally they should include the other supplementary info with the PR so that reviewers can verify.

I try to force contributor to provide related information in the PR.

Yes, that is part of my point. If we drop the logic to get system info using platform package and just go with distro.info without checking whether it's an ubuntu system or not(because the charmhub page already mentions that the charm is supposed to be run on ubuntu), do we lose anything of value?

That's the code try to support Ubuntu derivatives, I don't know why we don't support them.

Yes, that is part of my point. If we drop the logic to get system info using platform package and just go with distro.info without checking whether it's an ubuntu system or not(because the charmhub page already mentions that the charm is supposed to be run on ubuntu), do we lose anything of value?

One more to add:
Example here: https://github.com/canonical/charmcraft/blob/05b18a035237a2412ef19dab6f3f676632cc220b/charmcraft/utils/platform.py#L46
We maybe need the information to get the architecture once we need to support ARM64.

Hi @Pjack, the newest commit already address your comments.


FCM token refresh every ~10 min, losing notifications

[REQUIRED] Step 1: Describe your environment

Xcode version: 11.5
Firebase SDK version: 6.27.0
Firebase Component: Messaging
Component version: 6.27.0
Installation method: CocoaPods

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
This is likely a configuration or APNS issue, but my FCM tokens get refreshed every 10 minutes or so, and I lose some notifications in between refreshing. I created the key with apple, running the application in a release build on a physical device, and have all the appropriate plist files in my project. Haven't found much online regarding this issue, but was hoping I could get some resources to figure this out.
Relevant Code:
All config, not an issue with code as far as I can tell.

Are you receiving a new APNS token once every ten minutes? Also, is this consistent across all devices that you're sending notifications (are you able to isolate the buggy behavior by device/OS version)?

Can you also share with us the debugLog and the implementation under:application(_:didRegisterForRemoteNotificationsWithDeviceToken:)?

I revoked my APNS key and created a new one, seems to be working now so I'll go ahead and close this. Thanks!


Yorick uses sh but some pre-Elizabethan scripts used bash features

Should we use bash or try to stay inside the sh feature set? @mor1's port is broken by using bash's string substituion feature # for XENIMG.

Whoops, sorry-- only tested on my site which set this explicitly...


Configurable 'use strict' on generator JS files

It would be nice to allow generating files with the 'use strict' directive within functions. It would be useful to allow a choice during initial setup to generate files with or without the 'use strict' directive.

You can always just override the templates that are generated to include 'use strict'

Yes that's very true. I thought it might be handy to allow an option for this as opposed to having to edit the config file and find the existing templates and adding strict to them.  It also prompts the user to think if they want to use strict (which is better practice). It also helps avoiding making the global scope strict which is another common bad practice (especially when the build concatenates all the files together automatically).

On Jan 12, 2015, at 4:13 PM, Corey Wilson<EMAIL_ADDRESS>wrote:
You can always just override the templates that are generated to include 'use strict'
—
Reply to this email directly or view it on GitHub.

+1


[BUG]: Wrong column names in migration while renaming a column in SQLite

Report hasn't been filed before.

[X] I have verified that the bug I'm about to report hasn't been filed before.

What version of drizzle-orm are you using?
0.36.4
What version of drizzle-kit are you using?
0.28.1
Other packages
No response
Describe the Bug
I renamed a column in SQLite
old column name: service_id
new column name: client_id
it generated a migration like this
PRAGMA foreign_keys=OFF;--> statement-breakpoint
CREATE TABLE `__new_service_links` (
	`client_id` text NOT NULL,
	`provider_id` text NOT NULL,
	PRIMARY KEY(`client_id`, `provider_id`),
	FOREIGN KEY (`client_id`) REFERENCES `services`(`id`) ON UPDATE restrict ON DELETE cascade,
	FOREIGN KEY (`provider_id`) REFERENCES `services`(`id`) ON UPDATE restrict ON DELETE cascade
);
--> statement-breakpoint
INSERT INTO `__new_service_links`("client_id", "provider_id") SELECT "client_id", "provider_id" FROM `service_links`;--> statement-breakpoint
DROP TABLE `service_links`;--> statement-breakpoint
ALTER TABLE `__new_service_links` RENAME TO `service_links`;--> statement-breakpoint
PRAGMA foreign_keys=ON;

when I tried to run the migrations, it throws this error
SqliteError: no such column: "client_id" - should this be a string literal in single-quotes?

Then I checked the code to see what's wrong and noticed column names were wrong in the 10th line.
Then fixed it like this: (notice "service_id" next to SELECT)
INSERT INTO `__new_service_links`("client_id", "provider_id") SELECT "service_id", "provider_id" FROM `service_links`;

btw I recently started using casing: "snake_case" and deleted column names from the schema definitions. But I'm not sure if that's the case.

I have the same problem after I started using snake casing. To prevent errors I manually name the columns with multiple words.
export const addressTable = sqliteTable('addresses', {
	id: integer({ mode: 'number' }).primaryKey({ autoIncrement: true }),
	postalCode: text('postal_code').notNull(),
	houseNumber: text('house_number').notNull(),
	addition: text(),
	street: text().notNull(),
	city: text().notNull(),
});


[firebase_dynamic_links] OnLink only called every other time

Describe the bug
With the app minimized, the first time I click on a shared link, the onLink callback works fine, the second it doesn't get called, then the third time it works again, and so forth.
To Reproduce
Steps to reproduce the behavior:

With the app minimized, click on a shared link.
(1st try) - As a result, the app maximizes, and calls onLink onSuccess (perfect).
Then, minimize the app again, then click on the same link.
(2nd try) - As a result, the app maximizes, but it calls onLink onError
Then, minimize the app again, then click on the same link.
(3rd try) - As a result, the app maximizes, and calls onLink onSuccess (perfect).

Expected behavior
I would expect the onLink onSuccess to be called every time, but for some reason, it only works on odd number of tries.
Additional context
Here is my code.
`initDynamicLinks() async {
final PendingDynamicLinkData data = await FirebaseDynamicLinks.instance
.getInitialLink();
final Uri deepLink = data?.link;

if (deepLink != null) {

  bool isPartida = deepLink.pathSegments.contains("partida");
  if(isPartida){
    var partidaId = deepLink.queryParameters["partidaId"];
    if(partidaId != null){
      Navigator.of(context).push(
          MaterialPageRoute(builder: (context) => Partida(partidaId: partidaId)));
    }}}

FirebaseDynamicLinks.instance.onLink(
    onSuccess: (PendingDynamicLinkData dynamicLink) async {

      final Uri deepLink = dynamicLink?.link;

      if (deepLink != null) {

        var isPartida = deepLink.pathSegments.contains("partida");
        if(isPartida){
            var partidaId = deepLink.queryParameters["partidaId"];

          if(partidaId != null){
            Navigator.of(context).push(
                MaterialPageRoute(builder: (context) => Partida(partidaId: partidaId)));
          }}}},
    onError: (OnLinkErrorException e) async {
      print(e.message);
    });}`

Flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.17.4, on Mac OS X 10.15.4 19E287, locale en-US)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
[✓] Android Studio (version 3.6)
[✓] Connected device (1 available)

Im going crazy with the same issue,but sometimes you can get a row of three success message but then the pattern starts again.

Same issue here, only works in odd numbers of tries.
[✓] Flutter (Channel stable, 1.22.0, on Mac OS X 10.15.6 19G2021, locale en-HK)
    • Flutter version 1.22.0 at /Users/leochiu/Documents/Sources/flutter
    • Framework revision d408d302e2 (5 weeks ago), 2020-09-29 11:49:17 -0700
    • Engine revision 5babba6c4d
    • Dart version 2.10.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /Users/leochiu/Library/Android/sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0, Build version 12A7209
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 50.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[!] VS Code (version 1.50.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    ✗ Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (2 available)
    • SM A515F (mobile)        • R58N9318TRB                              • android-arm64 • Android 10 (API 29)
    • Chi Ho’s iPhone (mobile) • 6e805e3b7ea518ced638b5b6eafddf06cf289318 • ios           • iOS 14.0.1
    ! Error: Chi Ho’s iPhone is busy: Copying cache files from device. Xcode will continue when Chi Ho’s iPhone is finished. (code -10)

Any idea on this?
meet the same issue for iOS.

Any idea on this?
meet the same issue for iOS.

Hey @guihackmann, Dynamic Links has been reworked and as such the API has changed quite a bit.
Please upgrade to the latest version and create a new issue if you discover a problem. Thanks.


Replace stdlib on Slang API with CoreModule

This is a breaking change in a way that the Slang API function names are changed. All of them are commented as "experimental" and we wouldn't provide a back-ward compatibility for them.
Following functions are renamed:

compileStdLib() -> compileCoreModule()
loadStdLib() -> loadCoreModule()
saveStdLib() -> saveCoreModule()
slang_createGlobalSessionWithoutStdLib() -> slang_createGlobalSessionWithoutCoreModule()
slang_getEmbeddedStdLib() -> slang_getEmbeddedCoreModule()
hasDeferredStdLib() -> hasDeferredCoreModule()

Following command-line arguments are renamed:

"-load-stdlib" -> "-load-core-module"
"-save-stdlib" -> "-save-core-module"
"-save-stdlib-bin-source" -> "-save-core-module-bin-source"
"-compile-stdlib" -> "-compile-core-module"

-load-stdlib should be -load-core-module.
All current mention of stdlib in the compiler codebase should be changed to core-module. There should be no mention of standard modules in any of our source code files because everything we have is jsut the core module.

-load-stdlib should be -load-core-module.
All current mention of stdlib in the compiler codebase should be changed to core-module. There should be no mention of standard modules in any of our source code files because everything we have is jsut the core module.

I will change as suggested.

Also need to update docs/build_reference.ps1.

This PR is for https://github.com/shader-slang/slang/issues/5335 but it doesn't close.
This PR is limited to "breaking changes".
I still need to work more on the issue.


QueryVariable: No values populated in the variable using InfluxDB

Hi,
In our scenes app, we are trying to create a new QueryVariable based on InfluxDB using the query SHOW TAG VALUES WITH KEY = "<string>". This same query is running on one of our production dashboards and we are able to see the results there. But we don't see any data while using scenes (as seen below)

Also, in the networks tab, we don't see any query url being hit (like it does in SceneQueryRunner)
Here's the code that we are using, @grafana/scenes version is 0.19.0
const <variable_name> = new QueryVariable({
    name: '<variable_name>',
    label: 'QueryVariable',
    datasource: INFLUX_REF,
    query: {
      query: 'SHOW TAG VALUES WITH KEY = "<string>"'
    }
  });

export const INFLUX_REF = {
  uid: 'InfluxDB',
  type: 'influxdb'
}

Thanks!

Change
query: {
      query: 'SHOW TAG VALUES WITH KEY = "<string>"'
    }

to
query 'SHOW TAG VALUES WITH KEY = "<string>"'

@torkelo Thanks! That worked.
Maybe you can update the documentation here, if this is not something specific to Influx

@tanayshah23 there is a note relevant to that right under that example:

The datasource used in the preceding code block refers to the core Grafana Prometheus data source plugin. Make sure your Grafana stack has this plugin installed and configured. The query property is the same one that you would see in typical dashboard template variables when you view the dashboard JSON in the dashboard settings.

But can be made more clear


ConfusedFish

请更新，谢谢。

您好 什么时候可以更新合集里的头像

<EMAIL_ADDRESS>

邮件中没有收到附件呢 辛苦重新check一下吧～
------------------ 原始邮件 ------------------
发件人:                                                                                "okx/ordinals-collections"                                                                                @.>;
发送时间: 2023年12月7日(星期四) 下午5:35
@.>;
<EMAIL_ADDRESS>主题: Re: [okx/ordinals-collections] ConfusedFish (PR #2)
Logo、顶部背景图及简介一起改了吧。
简介：
It's not that I'm not working hard, it's just that the world is changing too fast. Hey, what happened just now?
Logo和背景图见附件，谢谢！
发自我的iPhone
------------------ Original ------------------
From: Ayepecr @.>
Date: Thu,Dec 7,2023 5:31 PM
To: okx/ordinals-collections @.>
Cc: yipianhuang @.>, Author @.>
Subject: Re: [okx/ordinals-collections] ConfusedFish (PR #2)
@.***
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.>

有了吗 刚才我把图片插入到正文
发自我的iPhone
------------------ Original ------------------
From: Demon-devil @.>
Date: Thu,Dec 7,2023 5:54 PM
To: okx/ordinals-collections @.>
Cc: yipianhuang @.>, Author @.>
Subject: Re: [okx/ordinals-collections] ConfusedFish (PR #2)
也是一样没有图，邮件也没有携带附件的标识
------------------ 原始邮件 ------------------
发件人:                                                                                "okx/ordinals-collections"                                                                                @.>;
发送时间: 2023年12月7日(星期四) 下午5:35
@.>;
@.@.>;
主题: Re: [okx/ordinals-collections] ConfusedFish (PR #2)
Logo、顶部背景图及简介一起改了吧。
简介：
It's not that I'm not working hard, it's just that the world is changing too fast. Hey, what happened just now?
Logo和背景图见附件，谢谢！
发自我的iPhone
------------------ Original ------------------
From: Ayepecr @.&gt;
Date: Thu,Dec 7,2023 5:31 PM
To: okx/ordinals-collections @.&gt;
Cc: yipianhuang @.&gt;, Author @.&gt;
Subject: Re: [okx/ordinals-collections] ConfusedFish (PR #2)
@.***
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.&gt;
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

换了另一个邮箱发
发自我的iPhone
------------------ Original ------------------
From: Demon-devil @.>
Date: Thu,Dec 7,2023 6:12 PM
To: okx/ordinals-collections @.>
Cc: yipianhuang @.>, Author @.>
Subject: Re: [okx/ordinals-collections] ConfusedFish (PR #2)
还是没有展示 可以试试不在这里回复，而是直接发一份单读的邮件到上面的邮箱呢
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

nope~ 直接评论发一下呢


Bug 🪲

Hey contributors is any one can resolve this issue?
Footer section in android device is to much larger and not have extra space between the headlines of footer section.
Resolve this issue as soon as possible

@vanshchauhan21
please assign this to me I can resolve it

@vanshchauhan21 please assign this to me I can resolve it

Oky do it

there is this extra space between the headlines of the footer page do u want to remove them ?

@vanshchauhan21 there is this extra space between the headlines of the footer page do u want to remove them ?

Yes you can remove them


Fix engine.ResultStack.Pop()

Synchronize changes https://github.com/neo-project/neo-vm/pull/333

Wait for https://github.com/neo-project/neo/pull/1713


[Feature Request] Store location of test plan inside manifest

According to the documentation you currently need to supply both the test plan and manifest when executing the tester.
Have you considered having the location of the target test plan live within the manifest so only one YAML document (the manifest) would need to be supplied? Perhaps we could even auto-detect if the manifest exists within the directory sample-tester is run from and simply execute with that. WDYT?
$ cd samples/v1
$ sample-tester gen-manifest --env php --bin php *.php --output samples.manifest.yaml
$ sample-tester

$ sample-tester <--- I would love this to work

https://github.com/googleapis/sample-tester/issues/52 (for the $ sample-tester with no arguments)

both the test plan and manifest when executing the tester

The test plans are language-agnostic and reside in googleapis/googleapis alongside the .proto files
The additional YAML file (the manifest) merely provides an index of how to invoke samples in a particular language by their region tag name.
When samples are generated in a library repo, the language-agnostic test plan is also copied into the client library repo. It does not know anything about the specific language. If you execute sample-tester without the manifest, the test plan tells sample-tester to execute the "language_analyze_sentiment" sample, but it doesn't know How to run that test.
Each language is responsible for providing its own file to map region tags to executions.
In dynamic languages, the mapping is usually directly to 1 source file, e.g. language_analyze_sentiment.js or language_analyze_sentiment.rb (although this is not always the case and many non-generated samples are bundled together in the same source files)
For static languages, this is more critical. The file needs to map the dotnet or mvn commands for how to run each code sample, so the test plan can execute them.

Thanks, understood!
Since it seems like the manifest is required (please correct me if I've got that wrong), my thought was that when we generate the manifest with sample-tester it is smart enough to pick up if there is a test plan in the directory, and if so, add its path into the manifest when it generates it so we only need one input file to the tool.

we generate the manifest with sample-tester

We are generating it with sample-tester, simply using a separate command.  And another command runs tests. They have different responsibilities.

it is smart enough to pick up if there is a test plan in the directory

If we wanted it to pick up the manifest / test plans without specifying then YAML files directly, we could request the following feature, WFTY?

[Feature Request] Auto-discover .manifest.yaml files
If no files with .manifest.yaml in their names are passed to sample-tester, search the current directory (recursively) for files named *.manifest.yaml and load them automatically.
And a similar feature idea (based on this common feature in many languages' test runners!)

[Feature Request] Auto-discover .test.yaml files
If no files with .test.yaml in their names are passed to sample-tester, search the current directory (recursively) for files named *.test.yaml and load them automatically as test plans.
Might be nice to standardize on .test.yaml and .manifest.yaml for explicitness. The tool already requires that manifest files be named x.manifest.yaml but test plans can currently be named anything.

I like the thoughts that have come out of this :), going to close this out and we can file these other two feature requests as separate issues.


Change manifest discovery to fetch from top-level collection URIs instead of hard-coded JSON

Instead of copying the collection to a JSON file hosted on the server, have a file with a list of top-level collection URIs. This way we can easily add new collections, and existing collections will be up-to-date. I don't know how significantly this will affect load time. Possibly the fetched collections could be cached locally for a set time, if fetching them anew each time slows things down a lot.

+1, SCTA collections update fairly often. Manifest editor discovery will become out of date very quickly if results are not periodically refreshed.

The reason why I did it this way, besides performance, is that some of the providers do not work when I fetch the collection URI because of CORS issues. Here is a complete list of which providers I added and which ones work and don't work:

Biblissima: does not work (CORS issue)
Bodleian: works
Villanova: works
e-codices: works
SCTA: does not work (CORS issue)
Shared Shelf Commons: works
Stanford University Libraries: does not work (CORS issue)
Wellcome Library: works
Yale University: works
Yale Center for British Art: works

So I will either have to remove Biblissima, SCTA and Stanford or look into other ways of making the request without getting blocked or have those 3 content providers at CORS headers to their collection URLs.
@jeffreycwitt The error I get for SCTA, for example, looks like this:
Fetch API cannot load http://scta.info/iiif/collection/scta. Redirect from 'http://scta.info/iiif/collection/scta' to 'http://scta.info/iiif/scta/collection' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:3000' is therefore not allowed access.

@jeffreycwitt I just noticed that when I update the collection URI for SCTA to http://scta.info/iiif/scta/collection (instead of http://scta.info/iiif/collection/scta) it works perfectly!
I got the wrong URL from https://github.com/ryanfb/iiif-universe/blob/gh-pages/iiif-universe.json

I created a PR on IIIF Universe to update the collection URIs for SCTA and e-codices:
https://github.com/ryanfb/iiif-universe/pull/11

@emmastanford The top-level collections for all content providers are now fetched directly from the collection URIs instead of hard-coded. Sadly, I had to remove Biblissima and Stanford for this. But I am confident that we can bring them back before too long... Could you test these changes using the develop branch and let me know if all looks fine to you? I will then merge into master.

@schwemmer cc @emmastanford The CORS issue on the top-level collection URL should be fixed now: http://biblissima.fr/iiif/collection/top
Thanks for the feedback, and do not hesitate to tell me if anything wrong in the IIIF collections or anywhere else...
As an alternative to the current Biblissima logo (yellow on a blue background), which is very small due to the baseline text, you could use one of this two images instead, if you want: http://static.biblissima.fr/images/logo-biblissima-300w.png (transparent png) or http://static.biblissima.fr/images/logo-biblissima-350w.jpg (black text on white background)

Thank you @regisrob!

@emmastanford I think we can merge this to master

Wonderful!

Merged to master and deployed to the Heroku instance:
https://morning-journey-27147.herokuapp.com

We'll be deploying this in the Bodleian instance next week as part of a larger set of updates.


Training with self processed SUNRGBD dataset

I want to evaluate 3D detection, so I preprocessed SUNRGBD as stated in the README. While training the logs say that
180/10450 missing samples
80/5014 missing samples
Did I make a mistake while preprocessing or is this expected?
Thanks :)

Hi @mertkiray,
This is actually expected.
When trying the processing code of Total3D, we find that the processed results have fewer samples than the downloaded pre-processed data from Total3D.
Since the pre-processing code from Total3D filters out some of the scenes based on several rules, we suspect that their code might have some change after the release which caused more scenes to be ignored.
Since our pre-processing code is based on Total3D's released code, we inherit their train/test set of the dataset processed with it.
The missing samples should not cause a large difference in the evaluation results.

Hi @chengzhag,
Thank you for the detailed answer. I am asking because the model I trained seems to differ on 3D Object detection results from the paper.

Top row is from the paper while the bottom row is from the output of training.
Also I have similar issues with the visualizations differing from the visualizations from paper results.

I wanted to make sure this is not because of preprocessing. Do you have any idea why these results are different as well as visualizations?
Also how can I get our test results you provided in the paper like Layout IoU, cam pitch, cam roll and chamfer distance.
Thank you so much 😊

Hi @mertkiray,
The LDIF does seem to break on some of the objects with thin structures, especially when generalizing from Pix3D to SUN RGB-D.
The other metrics can be checked after the training from the log file in the experiment folder or can be better shown from wandb experiment page (you can create a report with tables or graphs like below)
Scene understanding:

Single object reconstruction:

Would you please check the training and testing curve to see if there is any problem?

Thank you @chengzhag for sharing other metrics,
I am sharing my training and testing curves below.
This is LEN training/testing curves :

This is  LIEN + LDIF:

This is SGCN:

This is joint:

Do you see any problems with this curves?

Hi, @mertkiray,
I presume that the rest of the metrics logged with wandb are OK?
I see that your curves are quite similar to what I have got.
From my observation, It does require some luck (or several attempts) to achieve the results of our provided checkpoint, since there might be some randomness during the training process and the checkpoint we provided is the best we can get.

In LEN logs:
Test loss (layout_iou): 0.620150 Test loss (cam_pitch_err): 3.728792 Test loss (cam_roll_err): 2.606081
IN LIEN+LDIF logs:
Test loss (Avg_Chamfer): 0.008465 {'bed': 0.005958032906252605, 'wardrobe': 0.0031370156852062792, 'bookcase': 0.004260495355346292, 'chair': 0.007310174998500854, 'sofa': 0.003096780832186188, 'table': 0.017010838347444775, 'desk': 0.013083522971386888, 'misc': 0.02592532049439957, 'tool': 0.0029951145195148206} Test loss (chamfer_woICP): 8.669688 {'bed': 6.097681875626751, 'wardrobe': 3.2926102414548657, 'bookcase': 4.573465430221525, 'chair': 7.50823779989532, 'sofa': 3.34066853316409, 'table': 17.324360070516153, 'desk': 12.981401550831366, 'misc': 26.215719928701777, 'tool': 3.0616522062591085} Test loss (chamfer_wICP): 7.467241 {'bed': 4.969611945433475, 'wardrobe': 3.026142153369496, 'bookcase': 3.9906250437764443, 'chair': 6.427196312469815, 'sofa': 3.6129484184597493, 'table': 14.570997367973707, 'desk': 10.58210799721211, 'misc': 22.70249547140817, 'tool': 2.7223403830129893}
In SGCN logs:
Test loss (layout_iou): 0.637629 Test loss (cam_pitch_err): 3.061840 Test loss (cam_roll_err): 2.257464 Test loss (iou_3d): 0.198359 Test loss (iou_2d): 0.666520
In JOINT logs:
Test loss (layout_iou): 0.633157 Test loss (cam_pitch_err): 3.076085 Test loss (cam_roll_err): 2.250714 Test loss (iou_3d): 0.208100 Test loss (iou_2d): 0.676601 Test loss (Lg): 1.353826
I think they are close but still bad from the paper results. :) This is my first research paper that I tried to duplicate results and thank you for the amazing guidance :)
Do we have a chance to get the pretrained checkpoints for LEN and LIEN+LDIF?

Hi @mertkiray
Your pretrained LEN is actually quite close to mine:
Test loss (layout_iou): 0.613996 Test loss (cam_pitch_err): 3.652064 Test loss (cam_roll_err): 2.517752
, which is trained with the parameters provided by Total3D.
Since Total3D already provides a pretrained model with similar testing results of LEN:
Test loss (layout_iou): 0.599689 Test loss (cam_pitch_err): 3.613924 Test loss (cam_roll_err): 2.455977 (even better on camera pose estimation, may be loaded as an alternative)
, there seems no need to provide another LEN pretrained model.
Since the LIEN+LDIF is not finetuned during the joint training, the weights in our provided final model should be identical to the pretrained weights.
You can load it to test the Chamfer loss.

Hi @chengzhag  thank you again :)
So there is nothing wrong I assume with my setup and the differences with the paper is just randomness?

Hi @mertkiray,
I think yes.
On your demand, I also updated a link to download pretrained LIEN+LDIF model.
Hope this can help you.

Thank you so much @chengzhag


SQLite and history question

Hi,Can you please tell me how to stick to the future data only and not load any history data.My requirement is to get data from sqlite and show employee one month data here.Thank you in advance

Below is some sample codes from readme.
    adapter = new CustomAdapter(this, blockListener);
    adapter.enableHeader();//load history, you can also call disableHeader()----default

do not call adapter.enableHeader();
I think this may help you.


Create documentation for UnifiedStorageQuota v1.0 visualization API

Create documentation for UnifiedStorageQuota v1.0 visualization API

@rohit-arora05 , this is the public mirror for external contributions. For new API docs, we work in the private authoring repo. Please open a PR in the microsoft-graph-docs repo and follow our regular workflow for content review and publishing. I'm going to close this PR. Thanks.


Don't drop blocks on foreign functions

A code like
extern "C" {
    fn f() {
        fn g() {}
    }
}

is incorrect and does not compile. Today rustfmt formats this in a way
that is correct:
extern "C" {
    fn f();
}

But this loses information, and doesn't have to be done because we know
the content of the block if it is present. During development I don't
think rustfmt should drop the block in this context.
Closes #4313

My assumption is that this was an unintentional side effect vs. rustfmt deliberately attempting to convert the input to compilable code so I'm a :+1: on this change but will defer to @topecongiro given the nature of the change.
This may also be something we'd want to sneak into the 2.0 release given the "change" in formatting.


[NFC] Simplify Forbidden Prefix API

Rephrase it in terms of DeclBaseName and fix up some callsites.

@swift-ci please smoke test and merge


Migration react-router-dom 5 to 6

Migration react-router-dom 5 to v6
fixes: #723

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

let see build passed?

@johnaohara I have first priority for this instead -> #982

@johnaohara If you have any suggetion on this problem then let me know?

@johnaohara rebased


Improve Connection Failure Handling in NodeLeader

What current issue(s) does this address, or what feature is it adding?
Addresses #915
How did you solve this problem?
I had this same unhandled exception occur on my node. This is the solution I implemented.
How did you make sure your solution works?
Manual Testing
Are there any special changes in the code that we should be aware of?
No
Please check the following, if applicable:

[ ] Did you add any tests?
[x] Did you run make lint?
[x] Did you run make test?
[x] Are you making a PR to a feature branch or development rather than master?
[x] Did you add an entry to CHANGELOG.rst? (if not, please do)

Coverage decreased (-0.02%) to 86.719% when pulling 8c9fd38e87cee7bd395cc713fbd3fa31e114f11e on jseagrave21:fix-#915 into 3b8bfb6105e876d2484e389958c5dbffefb82e3e on CityOfZion:development.


Peers crashed after several hours on longevity stand

See #1714 for longevity stand description.
Load: 2 TPS
Peers crashed after approximately 9 hours on longevity stand. The stand script tried to bring them back online after that for a while unsuccessfully - as their block stores were corrupted.
Reason for a crash needs investigation. Logs need to be checked.

Crash log for peer 4 - https://drive.google.com/file/d/1Vd_RKeMsBVac2lawfOj8fDKd05pW1mCT/view?usp=sharing

It seems that at first something forcefully restarted the peer - making it corrupt the Kura store.
Then after that peer always failed because it could not load the Kura storage.
So the issue that needs investigation is why a peer was restarted, when it has not crashed.

Node started at: 2022-01-19T21:47:40.827656Z
First restart at: 2022-01-20T09:51:13.582132Z

After some investigation and discussion with DevOps we came to conclusion that the first time the peer was restarted due to reaching its memory limit, which was set to 500 MB.

The longevity stand was restarted with peers having more memory. But it is worrying that 500MB were used in just 10 hours.
Also as can be seen from the charts, the CPU usage is increasing. This also needs investigation. Probably also needs a separate issue.

Unfortunately, the RAM usage is in line with what I measured in the first week of January. The CPU usage increase needs investigation.


Nick: Watch heatersexpert.com

Nick requests the watch of the watch_keyword heatersexpert\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

heatersexpert\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


16.03 dropbear initrd not seeing keys

Issue description
I have an encrypted root system.  I am using grub to boot it. I tried to enable remote unlocking.  I feel like I am missing something simple but I can't figure it out for the life of me.
my config
boot.initrd.network.enable= true;
boot.initrd.network.ssh.enable = true;
boot.initrd.network.ssh.port=2000;
boot.initrd.network.ssh.authorizedKeys=[ "my public rsa key" ];
boot.initrd.network.ssh.hostRSAKey = "/etc/ssh/dropbear_rsa_host_key";
boot.initrd.network.ssh.hostECDSAKey = "/etc/ssh/dropbear_ecdsa_host_key";
boot.initrd.network.ssh.hostDSSKey = "/etc/dropbear/dropbear_dss_host_key";

There are keys there but the build initrd doesnt seem to do anything with this, when i boot it says that no key was at those three locations, meaing i assume they werent copied into the initrd
Steps to reproduce
install grub, build an initrd with this config, reboot to see error message.
Technical details
16.03.1242.c5cbda2 (Emu)
nix-env (Nix) 1.11.2
"16.03.1242.c5cbda2"

if you specify the key paths without quoting things, it will copy the keys into /nix/store and treat them as normal dependencies

Unfortunately, you need the keys in store (because we need to pack them into initrd), so you need to use unquoted paths.
We need to check that the keys is really there in initrd, will do later..

passing without quotes did copy them into the nix store, but I won't be able to do a reboot till i get home today, i dont fully trust that it rebuilt the initrd and will work. Thanks for advice, I could have swarn I tried without quotes, but maybe I just had the path wrong.

I just unpacked the initrd it looks like it did the job right. I will verify when I reboot from home later. Thanks for all help

You are welcome! I've pushed a fix -- we now verify that keys have really got into initrd or fail during build.

I just encountered the same Issue, dropbear states that no host key was found, but the build succeeds. This is my config:
boot.initrd.availableKernelModules = [ "igb" ];
  boot.initrd.network = {
    enable = true;
    ssh = {
      enable = true;
      port = 2222;
      authorizedKeys = [ "ecdsa-sha2-nistp256 <key>" ];
      hostECDSAKey = /etc/nixos/host_ecdsa_key;
    };
    postCommands = ''
      echo 'cryptsetup-askpass' >> /root/.profile
    '';
  };

And the host key exists:
~# ll /etc/nixos/host_ecdsa_key
-rw------- 1 root root 141 Aug 11 10:30 /etc/nixos/host_ecdsa_key

Should I open a new issue for this, or is it a simple error on my end?


No underline for the Region-based implementation?

I ran the app on both a Nexus 5X and on a emulator (API 23) and got the following result without any error in the log. Is it as the expected behavior?

same I didn't get region underline for Moto E 2nd Gen, Marshmallow

replacing underlineRegion.getBoundaryPath(mUnderline); to mUnderline.addPath(underlineRegion.getBoundaryPath()); did the trick for me
Path: https://github.com/romainguy/elegant-underline/blob/master/app/src/main/java/com/android/elegantunderline/UnderlineView.java#L143


minor vulnerability in clean-css <4.1.11

Please update clean-css to 4.1.11 or higher due to a minor vulnerability detailed here:
https://snyk.io/vuln/npm:clean-css:20180306

Thank you for bringing this to my attention. Will be fixed today and I'll post back here once it's published up.

@GCDeveloper fixed with the 3.9.3 release


Do you have a plan to support timezone?

Currently, it's fixed as UTC. do you have a plan to support timezone?
I am willing to create a PR, but it's better to confirm before I do that.

Hey banyan, thanks for bringing this up. I think this would be an essential feature for this library. An option to specify different timezones or an option like useLocalTime... Please make a PR if you have time for it 🙏
For implementation I would add a property to the Calver class such as timezone or useLocalTime and then we would do:
import calver from 'calver';

calver.timezone = 'UTC'
// or
calver.useLocalTime = true // should be off by default for backward compatibility

if that makes sense to you also...

Thanks. Both API makes sense to me. Let me try with that.

awesome contribution @banyan thank you very much!

 Thanks too!


BigNumber

@nickgammon
Firstly, thanks for your BigNumber library.

This is the code
#include <math.h>
#include "BigNumber.h"
BigNumber latitude="35.95568352";
BigNumber longitude="120.1484011";
void setup (){
Serial.begin(9600);
BigNumber::begin (15);
BigNumber pi ("3.1415926535897");
BigNumber a ("180");
BigNumber latitude_rad = latitudepi/a; // latitude in rad
BigNumber longitude_rad = longitudepi/a; // longitude in rad
Serial.println();
Serial.println();
Serial.println(latitude_rad);
Serial.println(longitude_rad);
}
void loop(){}

 BigNumber latitude_rad = latitudepi/a; // latitude in rad

That code doesn't even compile. Is that your problem?
/tmp/arduino_modified_sketch_443397/sketch_aug17a.ino: In function 'void setup()':
sketch_aug17a:13: error: 'latitudepi' was not declared in this scope
 BigNumber latitude_rad = latitudepi/a; // latitude in rad
                          ^

If not, please post your code that you are actually using.

sorry，there is mistake happened when i copy and paste the code.The *  is missing，I do not know how it happened.
I shall write these two line
"BigNumber latitude="35.95568352";
BigNumber longitude="120.1484011";
after  " BigNumber::begin (15);"
Thanks.


Documentation for resolving dependencies by tag

This is the documentation for ticket #8 and PR #39.
I also added a super simple example to the example project. Nothing complex but should show one possible way to use it.

Again, thanks a million for your contribution!


Windows Activation URL

I have seen on multiple pages that the requirements for Windows activation always link to a support article on support.microsoft.com how to resolve an issue with activation. Whilst this information is correct, it should have its own page listing the URLs and port numbers on docs.microsoft.com so customers can refer to articles of the same 'type'

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

ID: 1aafc065-6ee6-e3cf-31ba-ab3dd277fb23
Version Independent ID: 35d3102e-4ff7-9408-8dbd-513b7dfff5c0
Content: Windows Autopilot requirements - Windows Deployment
Content Source: windows/deployment/windows-autopilot/windows-autopilot-requirements.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

Similar to the link for the O365 URLs and IP addresses

Hello, @AndrewStalkerAU
The issue is under investigation and you will be notified with any changes on the way.
Thank you for providing feedback.

@AndrewStalkerAU  - thank you for engaging with the community here for the docs.
I know the product teams are always looking for user feedback and how to make support documents better. They keep a close tab on user feedback and it would be awesome if you could add your voice in User Voice https://sharepoint.uservoice.com/.
You might find others have made a similar request that you can add to, or you can open a new one. Let us know here if you do so we can up-vote your request.
If there is any other improvements required in this specific document, please let me know.
Thank you for your contribution to make the docs better! Much appreciated!
Imran.


Homebridge.local doesn’t load with latest Config UI update

I didn’t install any new plugins. Just recently I’ve been prompted to update to the latest Homebridge UI plugins, rebooted, loads just fine.
Until after a few minutes, homebridge.local doesn’t load for me anymore. It’s stuck at loading. I’ve tried several different device, same issue. Home app loads all Homebridge accessories just fine, its the Config UI that fails to load.
To Reproduce:
Update to the latest Config UI, with Homebridge version 1.2.3 installed on Raspberry Pi 4b then reboot Homebridge
Expected behavior:
After for a few minutes/hours, config UI no longer loads. No error message, just a blank page. After rebooting, it loads fine again. Until it wasn’t again.

Hi @ahmadnazirafiq,
I think you forgot to attach the logs?

Hi @ahmadnazirafiq,
I think you forgot to attach the logs?

I’m not sure how to obtain it without access to the config UI. How do I get the logs?

Can you tell me about your setup? Without this information I can't even tell you how to get the logs 😄
**Environment:**

* **Node.js Version**: <!-- node -v -->
* **NPM Version**: <!-- npm -v -->
* **Homebridge Version**: <!-- homebridge -V -->
* **Homebridge Config UI X Version**:
* **Operating System**: Raspbian / Ubuntu / Debian / Windows / macOS / Docker
* **Process Supervisor**: Docker / Systemd / init.d / pm2 / launchctl / hb-service / other / none

Ok looking though your issue history on GitHub I can see you are running the Homebridge Rasoberry Pi Image?

Also, I assume you've tried connecting via IP address? Connections via homebridge.local are not guaranteed to work - the homebridge raspberry Pi image setup guide says as much.

**Environment:**

* **Node.js Version**: <!-- v12.18.3 -->
* **NPM Version**: <!-- v6.14.6 -->
* **Homebridge Version**: <!-- homebridge -V -->
* **Homebridge Config UI X Version**: v4.32.0
* **Operating System**: Raspbian
* **Process Supervisor**: hb-service

Ok looking though your issue history on GitHub I can see you are running the Homebridge Raspberry Pi Image?
If so; see https://github.com/homebridge/homebridge-raspbian-image/wiki#useful-commands for the log viewing command.

This is silly but how I do access the Pi through my Mac via Terminal? Both devices are on the same network.

Before bothering with that, can you access the UI by going to the IP address of your Pi in a browser?
EXAMPLE: http://<IP_ADDRESS>

Before bothering with that, can you access the UI by going to the IP address of your Pi in a browser?
EXAMPLE: http://<IP_ADDRESS>

Yes, I can access it via IP just fine. Until after a few hours, it wasn’t accessible for no reason although accessories on Home app loads just fine. I’m not sure what’s causing it.

The "homebridge.local" dns functionality is not anything to do with the UI (or the UI update) - that's taken care of by the Linux operating system Homebridge is running on top of.
Why it would stop working randomly, I have no idea sorry.

By chance, are you running another instance of homebridge on another device somewhere in your network?

By chance, are you running another instance of homebridge on another device somewhere in your network?

No. Not running Homebridge anywhere other than Pi

Thanks, I'm still not sure what would cause this behaviour then.
But so long as you're able to access the UI via the Pi's IP address; it sounds like homebridge-config-ui-x is working as expected.


Changed connect dataset text by new dataset

That's it.

And happily fixes #3009

Frontend tests were OK :+1: (details)


ListDatabases and ListCollections should allow filtering by name

What should be done?
Backend.ListDatabases and Database.ListCollections should allow getting a database/collection by exact name. Essentially, we are combining "list" and "get" methods in one method.
Some code that currently does the filtering by name itself should be simplified.
Where?
See references to this issue.
Definition of Done

all backends updated;
handler updated;
unit tests added/updated;
integration/compatibility tests added/updated;
spot refactorings done.

@sachinpuranik there is another place that needs updating now


Switch to Bootstrap 4

This is a WIP; do NOT merge until it's ready.

[x] Migrate all frontend code to v4
[x] Make sure tests are passing
[x] Make sure the breakpoint for showing the ads is the right one
[ ] Show the bootstrap bundle since we already host it and use it?

@jmervine: can you help out with the test failures?

Yeah. I can take a look.

I don’t have a strong opinion, but my 2 cents is we leave the UI as is, only updating the underlying bootstrap framework.  Baby steps, you know?
The Try It buttons will need to be moved from the  bootswatch page to the bootswatch4 page. !85 otherwise the rest could remain the same.

Yeah but it's the only way Bootswatch will work otherwise it will be broken.
Feel free to push here
On Dec 2, 2017 00:51, "Joshua Mervine"<EMAIL_ADDRESS>wrote:

I don’t have a strong opinion, but my 2 cents is we leave the UI as is,
only updating the underlying bootstrap framework. Baby steps, you know?
The Try It buttons will need to be moved from the bootswatch page to the
bootswatch4 page. !85 otherwise the rest could remain the same.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/MaxCDN/bootstrap-cdn/pull/889#issuecomment-348635406,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVVtTsB5dTfyLo664bc-yPRweQDpn4Mks5s8IMNgaJpZM4QyDjv
.

OK, tests fixed. Apparently the sri pug variable is needed in the pug files even though it's seemingly unused, otherwise tests fail...

Looks like the try it now buttons aren’t working.  At least not on my phone.

True. I'll fix that and a couple more issues I see on mobile.
On Dec 2, 2017 18:36, "Joshua Mervine"<EMAIL_ADDRESS>wrote:

Looks like the try it now buttons aren’t working. At least not on my phone.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/MaxCDN/bootstrap-cdn/pull/889#issuecomment-348703387,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVVtfCjFNgaW2ktpKCTdnycD0rFch2Uks5s8XyGgaJpZM4QyDjv
.

OK, @jdorfman @jmervine: now this is ready for review; just confirm and I will merge when I believe it's 100% ready.
If you find any issues let me know.

I added the JS bundle in the frontend. We might want to tweak tests and stuff though.


Bumps OS X Deployment Target to 10.9

This P.R. adds OHHTTPStubsSwift.swift to the Mac Framework and bumps the OSX Deployment Target to 10.9 (i.e. Mavericks).

Seems that you forgot to bump the version requirements in the Podspecs
Might want to add an entry in the CHANGELOG too! :wink:

I bumped the version requirements and added an entry in the Changelog. (Note that the link to 4.7.1 is inactive, I'm sort of assuming that it would be a 4.7.1. release)
Thanks!

Merged, sorry it too so long ;-)


Child Trigger and Code Coverage Error - Current Company Not Set

FFA V15
Apex Test Failures
Class Name
Method Name
Error Message
dlrs_c2godaCashEntryLineItemTest	testTrigger	System.DmlException: Insert failed. First exception on row 0; first error: FIELD_CUSTOM_VALIDATION_EXCEPTION, Object validation has failed. Cash Entry Line Item: Object validation has failed. Current company not set.: []
Stack Trace: Class.dlrs.RollupService.testHandler: line 277, column 1 Class.dlrs_c2godaCashEntryLineItemTest.testTrigger: line 11, column 1

Please refer to this page and also the API documentation for that package, it has a sample Apex test that will help you setup the required context needed to pass the validation this object requires.


Microsoft Edge is ANGLE?

Microsoft Edge is not use ANGLE. However, ANGLE check is Yes!
I think it is better to add the following code.
    function getAngle(gl) {
        var lineWidthRange = describeRange(gl.getParameter(gl.ALIASED_LINE_WIDTH_RANGE));

        // Heuristic: ANGLE is only on Windows, not in IE, and does not implement line width greater than one.
        var angle = ((navigator.platform === 'Win32') || (navigator.platform === 'Win64')) &&
            (gl.getParameter(gl.RENDERER) !== 'Internet Explorer') &&
            (gl.getParameter(gl.RENDERER) !== 'Microsoft Edge') && // Add
            (lineWidthRange === describeRange([1,1]));

The reason is because the renderer has been changed from "Internet Explorer" to "Microsoft Edge" at Windows 10 Insider Preview Build 14352.

Thanks for the fix!


Self-hosting is broken (for osmnames, at least, because :host config is ignored)

~BEFORE POSTING AN ISSUE, PLEASE MAKE SURE THE PROBLEM IS NOT ADDRESSED IN THE README!~ Done
Expected behavior
Specifying a :host in Geocoder.configure(...) should result in an instance of Geocoder::Lookup::Osmnames making requests to the given host instead of to 'geocoder.tilehosting.com'.
Indeed, the documentation at https://github.com/alexreisner/geocoder/blob/master/README_API_GUIDE.md#osm-names-osmnames states:

To use self-hosted service, set the :host option in Geocoder.configure.

Actual behavior
Api requests still go to 'geocoder.tilehosting.com'.
Indeed, if you look at the the definition of Geocoder.config_for_lookup (geocoder-1.6.7/lib/geocoder/configuration.rb), you'll see the following, which throws away the :host specification because it's not in Geocoder::Configuration::OPTIONS.
def self.config_for_lookup(lookup_name)
  data = config.clone
  data.reject!{ |key,value| !Configuration::OPTIONS.include?(key) }
  if config.has_key?(lookup_name)
    data.merge!(config[lookup_name])
  end
  data
end

Steps to reproduce
In a Rails project, rails generate geocoder:config, then modify the file generated at config/initializers/geocoder.rb to read:
Geocoder.configure(
  lookup: :osmnames,
  host: 'localhost:8888',
  key: '',
)

Then in the rails console, run:
puts Geocoder::Configuration.instance.data[:host] # Verify that `:host` IS present and speciried as `localhost:8888`
puts Geocoder.config_for_lookup(:osmnames) # Verify that `:host` is not one of the keys
puts Geocoder::Configuration::OPTIONS # Verify that `:host` is not in the list
Geocoder.search [24.7763, 121.02] # It should be apparent that this didn't go to localhost:8888

Environment info

Geocoder version: 1.6.7
Rails version: 6.1.3
Database (if applicable):
Lookup (if applicable): osmnames

What a clear and complete issue report! Thanks very much. Just fixed.


MudList does not fire SelectedValueChanged when embedded inside MudMenu

Bug type
Component
Component name
MudList
What happened?
When a MudList is embedded inside a MudMenu, the SelectedValueChanged callback is only invoked when I select the MudMenu, not an item on the MudList.
Expected behavior
I'd expect the MudList callback to still function normally.
Reproduction link
https://try.mudblazor.com/snippet/wOcwPEmMqMhHwEdG
Reproduction steps

Open menu
Select an item
Check to see if anything had been added to debug output

Relevant log output
No response
Version (bug)
6.0.17
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Also, just had my company @petabridge start sponsoring you. Great project!

Also, I looked at https://github.com/MudBlazor/MudBlazor/issues/2949 but I think this is a different issue.

We have a PR about reworked MudList at #5150 and #5368. We can handle this issue with them.

In the short run, are there any recommended work-arounds?

If you don't have specific reasons to use MudMenu, try MudPopover instead.

I actually do need a menu for this

i upgrade from MudBlazor 6.x to 7.x and the event does not fire regardless. I feel this requires further investigation.


fix(clerk-js): Fix padding for active device item in UP

Description
Before

After

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[X] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

!preview

Hey @anagstef, your preview is currently deploying and should be available shortly.

Status
Preview
Updated (UTC)

:cook: Deploying...
Inspect
Feb 16, 2024 03:43 PM


Einzelne Missionen aus der Verfügbarkeitsliste löschen ohne sie anzunehmen -> Mission verweigern
Per Knopfdruck Missonen neu laden -> es werden neue Missionen mit zb. anderen Feldfrüchten generiert
Pappel-Missionen
Bonus-System wenn man für den selben Besitzer mehrere Aufträge abschliesst
Google-translated

Hey, I have a few suggestions for improving missions:

Deactivate wood missions / Activate/deactivate the different mission types in the settings menu.
Delete individual missions from the availability list without accepting them -> refuse mission
Reload missions at the touch of a button -> new missions with other crops, for example, are generated
Poplar missions
Bonus system if you complete multiple orders for the same owner

All great ideas!

Deactivate wood missions / Activate/deactivate the different mission types in the settings menu.

Coming in the next release.

Delete individual missions from the availability list without accepting them -> refuse mission

Interesting concept, but should be doable - I'll add this to the feature request list

Reload missions at the touch of a button -> new missions with other crops, for example, are generated

Reloading missions is on my development list, but it may not be possible to refuse a mission, then re-generate, and not see the same mission again. the "other crops" would only be based on the farmlands on the map and the crops they have in them, I wouldn't be able to change the crop type already in a field.

Poplar missions

I think this would fall under baling missions - assuming that the whole system was setup for that. this one would probably need to be a separate mod.

Bonus system if you complete multiple orders for the same owner

Gamification of the mission system was something that was already a feature of the "Better Contracts" mod, and I would be "stealing their idea" if i added it to my mod also.  i'll let them handle it

Hi @GMNGjoy - thanks for the hint to my BetterContracts mod 😉 .

Delete individual missions from the availability list without accepting them -> refuse mission

This can be achieved by accept-then-cancel the mission.  Even tree transport missions don't call up penalties if cancelled immediately.

Reload missions at the touch of a button -> new missions with other crops, for example, are generated

see FS25_BetterContracts

Bonus system if you complete multiple orders for the same owner

I'm currently working on that

Hey
Currently i use the V<IP_ADDRESS>
Unfortunatley the Setting customMaxPerType in the default Config works not for me. i set deadwood and treetransport first to 0 then to 1 and test it in the game. I accept & delete all of them and check if they come back. unfortunately in any case i have 5 of both types all the time,

Should be fixed in <IP_ADDRESS>


Switching domain name leads to “DNS rebind attack” warning

I switched my domain name and immediately received this page:

Can this be better handeled? It may also be caused due to me not having set up a TLS certificate yet.

You need to configure the proper dns in the software or add alternate hostnames, see

Should I just put the hostname into that box? (I don’t really understand this option.)
When I change the domain name (not the hostname) in OPNsense, the router should be able to handle this appropriately (maybe redirect afterwards etc), or do I miss something here?

if you add the full hostname (including domain), it should work, same of you change the host and domain name of the machine itself.
If you want to known what this is about, please read https://en.wikipedia.org/wiki/DNS_rebinding

Well, that’s something OPNsense if completely aware of, when I change either the host name or the domain name. Can’t OPNsense do this automatically?


feat: MQTT Broker supplements the Subscription Options test case and fix a bug

What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!

fix: cache_manager.is_new_sub in retain::try_send_retain_message always return false
feat: MQTT Broker supplements the Subscription Options test case

Checklist

[x]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.
[x]  This PR does not require documentation updates.

Refer to a related PR or issue link (optional)
close https://github.com/robustmq/robustmq/issues/360

@CHLK Modify the description of the PR to:  relate #360  => close #360.
This allows us to close issues while we merge.

@socutes I've implemented your suggestion, please take a look when you have a chance.


ARRAY comparison not supported for arrays with null elements

col_a is of type array(varchar)
the column can contain nulls
consider following query:
SELECT id,
        MIN(DISTINCT col_a)  FILTER (WHERE col_a is not null) AS col_a
FROM table
GROUP BY id

This query gives:

ARRAY comparison not supported for arrays with null elements

This seems like a bug. The FILTER should be applied before MIN()
To mention :
SELECT id,
        array_agg(DISTINCT col_a)  FILTER (WHERE col_a is not null) AS col_a
FROM table
GROUP BY id

Works but it converts col_a to be array(array(varchar)) which is not desired.

The above filter isn't applied to each element. It makes sure whether  col_a is null (≒contains).
You may want to use filter array function.

filter(array(T), function(T, boolean)) -> array(T)
Constructs an array from those elements of array for which function returns true:

SELECT id,
        MIN(DISTINCT filter(col_a, x -> x IS NOT NULL)) AS col_a
FROM table
GROUP BY id

I understand how you solution will work.
But I don't understand why my original solution doesn't work.
as you said it makes sure col_a doesn't have nulls then the values are distinct and then MIN() is added.

@RosterIn what is your actual query?
In your original query, you can replace MIN(DISTINCT col_a)  with MIN(col_a) to achieve the same result.
Does this help?

count(col_a) FILTER (WHERE col_a >5)

The query won't work because the comparison is array(integer) vs integer.

Wouldn't the count consider only values where col_a> 5? what is the difference?

The type is difference.

@RosterIn what is your actual query?
In your original query, you can replace MIN(DISTINCT col_a)  with MIN(col_a) to achieve the same result.
Does this help?
@findepi
No. It still gives the same error. This is my actual query with many more columns and complex FROM clause but at the end I group by id and all dimensions needs to be under aggregation function.

count(col_a) FILTER (WHERE col_a >5)

The query won't work because the comparison is array(integer) vs integer.

Wouldn't the count consider only values where col_a> 5? what is the difference?

The expected type and actual type are different. By the way, I would invite you to join the community Slack if you haven't yet joined. https://prestosql.io/slack.html

@ebyhr  I asked about count(col_a) FILTER (WHERE col_a >5) regardless of this type example. to better explain:
Why when x is integer and i do:  count(x) FILTER (WHERE x >5)  filter works but in my case it doesn't work.
I just don't understand what exactly happens when I use FILTER the way I did. obviously NULL values are being evaluated hence the error what I miss to fully understand this is why the NULL values weren't excluded in the FILTER.

@RosterIn consider:
SELECT ARRAY[42], ARRAY[NULL], CAST(NULL AS array(integer));

first constructs array with a number
second constructs array with NULL value
third constructs NULL array
When you do MIN(DISTINCT col_a)  FILTER (WHERE col_a is not null), you filter out the third case, but not the second.

@findepi  thx now I understand. so I just missed condition in my query.
Thus
MIN(DISTINCT col_a) FILTER (WHERE col_a is not null or col_a = ARRAY[NULL] )
Should work and equivalent to:
 MIN(DISTINCT filter(col_a, x -> x IS NOT NULL)) AS col_a
BTW is there a difference in terms of efficiency?

It's not equivalent in some cases because
MIN(DISTINCT col_a) FILTER (WHERE col_a is not null or col_a <> ARRAY[NULL] ) doesn't iterate all elements in an array.

OK I think I get it. In any case it needs to look into the elements with in the array


Decentralized search

I used this slide in my JSConf Asia 2014 talk today:

It would be awesome if we could crack the decentralized search problem.
cc @missinglink

From the p2p-hackers mailing list:

There's a really neat solution to this problem called Hyperspace Hashing which has you model your data in a set of different dimensions (i.e. a euclidian hyperspace), and locate a point within the hyperspace based on the hashes of the inputs. You then break the hyperspace down into a finite number of subspaces, and partition those among the available servers. The familiar hash ring can be seen as the one dimensional version of this concept.

It doesn't quite fit the full text search scenario you described, but it does help expressing more powerful queries than just k/v lookup:

http://www.cs.cornell.edu/people/egs/papers/hyperdex-sigcomm.pdf
http://www.slideshare.net/DECK36/c36-hyperdexmeetupdec2013v2nogfx

Tony Arcieri

perhaps if we relax one requirement for a de-centralized search, like immediacy of answers, then we can solve the problem. Here is a sketch for an algorithm:
Take a look at the decentralized search idea by Bill Gross (Idealab). The essence of it is:

index data on each machine/node, and keep index there
broadcast a search query
have users manually confirm the return of search results
Broadcast can be done using Ethereum's Whisper or equivalent DHT-based mechanism and can be optimized by each node publishing the topics that it allows searches for.
Manual confirmation for searches can be saved with user's permissions and can be used later without bring it to user's attention.

I was trying to build a chrome app to browse content with webtorrent (Each file is some content, with a collection of magnet links). So the user can navigate from one file to another.
But seems DHT is not supported in browser? which makes it not much different than traditional content serving from a server.
Any plan to support DHT in the browser?
Thanks!

What if Webtorrent had a built in version of Norch ? Part of it is already browserified search-index.

@niczy Even without DHT support in the browser, the content still gets delivered P2P which is very different than serving the content from a server.

This is a cool topic, but out-of-scope for WebTorrent. Closing issue, but feel free to continue discussion.

Also, Tribler seems to have tackled this issue: http://dl.acm.org/citation.cfm?id=2072433

Another approach seems to use the blockchain to store magnet links: http://mercbay.com/#faq

http://btdigg.org/about/ seems interesting on that topic, sadly it doesn't seem to be opensource

This is off the wall but you could create a flowing side bar.  If we can stream video we could centrally snag info.  Basically a scrolling window where if someone see's something they like it will display the hash and the file name.  The webtorrent will come and go depending if the browser(s) are open or not.  Could certainly dump this info to a small text file and make hash's search able.

A local index sounds good, but letting a user choose cwhich categories to "host", i.e to limit the local index to contenttypes of interrest.


Some Git status icons are missing

Issue Type: Bug
It seems that deleted, renamed, and added status icons are missing in the SCM sidebar:

VS Code version: Code - Insiders 1.34.0-insider (d234937aedcbc265d2272ba5861125293239f9c7, 2019-04-05T05:18:43.251Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz (8 x 4008)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.93GB (20.09GB free)

Process Argv
c:\Users\Eric\code\vscode-gitlens

Screen Reader
no

VM
0%

@jrieken Anything changed lately wrt resource decorations?

Nope, didn't change anything. @eamodio Are then icons also missing from the explorer? @joaomoreno Those letters/colors in the SCM viewlet should just travel as part of the SCMRawResourceSplices, the decoration service may add extra information, like compile info, later

I can't reproduce this with the latest insiders.


Clarify local development login flow

Ticket #
Description
Improves the new developer experience my making the standard login flow a bit clearer. It does this by:

Creating generic admin and staff users in the DB seed script
Making the logs for the generated login URL stand out more
Update the docker setup documentation to walk through standard login flow in dev
Update the gmail setup documentation with a warning that you generally shouldn't need it

(This PR is based on the discussion here: https://github.com/usdigitalresponse/usdr-gost/pull/2524/files#r1470332892)
Screenshots / Demo Video

Testing
Manually ran the DB seed script and login flow for the admin and staff users
Automated and Unit Tests

[ ] Added Unit tests

Manual tests for Reviewer

[ ] Added steps to test feature/functionality manually

Checklist

[ ] Provided ticket and description
[ ] Provided screenshots/demo
[ ] Provided testing information
[ ] Provided adequate test coverage for all new code
[ ] Added PR reviewers

I've updated the seed admin and staff emails to use example.com for domain so we don't accidentally email all USDR staff. (Whoops! Since I had email sending via gmail set up still, it seems I actually emailed a login email to a real USDR google group address.)

Addressed comments, and ready for re-review!


Antivirus Warning - "Trojan-Downloader.Win32.Alien.gnj" Found

Ill hope this is a false positive,
my Antivirus (CheckPoint Endpoint Security) found in the download of the Windows exe Package the trojan and deletes the File.
Tested with the newest Signature Update for the AV.
Signature Version:
202207260355
AV Version: E84.50.7526

Hi @ChristianMoritz,
It is a false positive. To prove it, you can upload both the installer and the executable file into virustotal.
I have done that myself, but sending a screenshot wouldn't guarantee you that I am not sending an edited pic


Add initial telemetry for schema compare

Add telemetry for the buttons in the schema compare extension

  });

So this is one of the places where people would be starting comparisons but you wouldn't be getting telemetry.
Also don't you want to track failures in the compare as well? So it seems like the telemetry for comparison should be moved inside this method

Refers to: extensions/schema-compare/src/schemaCompareResult.ts:535 in 55547ea. [](commit_id = 55547ead10439b2a3ae237d75edeb4e682839d7e, deletion_comment = False)

From my experience in general, it's better to always add some more data along with the events such as    "SchemaCompareApplyFailed" with data supporting why it failed, so that it can be rectified in future. With that in mind, be careful in adding files or file names/paths as they can contain PII


Kernel updates for 2023-09-19

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@TredwellGit @NeQuissimus something weird in this update: 5.10.186-rt is just gone from the FTP. Might be worth asking around?

6.5.4 builds/runs.


Deploy to Azure Cannot find module 'angular2-universal-polyfills'

Not quite sure if this is an error related to the template or not, but I've created a test app with the aspnetcore-spa yo template with Angular 2 and tried deploying it to Azure App Services.
I get the following error:
Exception: Call to Node module failed with error: Prerendering failed because of error: Error: Cannot find module 'angular2-universal-polyfills'
at Function.Module._resolveFilename (module.js:469:15)
at Function.Module._load (module.js:417:25)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object.<anonymous> (D:\home\site\wwwroot\ClientApp\dist\main-server.js:87:19)
at __webpack_require__ (D:\home\site\wwwroot\ClientApp\dist\main-server.js:20:30)
at Object.<anonymous> (D:\home\site\wwwroot\ClientApp\dist\main-server.js:48:2)
at __webpack_require__ (D:\home\site\wwwroot\ClientApp\dist\main-server.js:20:30)
at D:\home\site\wwwroot\ClientApp\dist\main-server.js:40:18
at Object.<anonymous> (D:\home\site\wwwroot\ClientApp\dist\main-server.js:43:10)
Current directory is: D:\home\site\wwwroot

The code works fine in Visual Studio as well as locally and I'm not sure what I'm missing to make it work once published.

That's strange. The publishing certainly works as far as I'm aware.
Is it possible you forgot to specify a Node version when setting up your Azure website? See: http://blog.stevensanderson.com/2016/10/04/angular2-template-for-visual-studio/#now-lets-deploy
The reason I suggest this is that, if you don't specify a Node version, then it falls back on an old version with NPM 2.x, which doesn't correctly restore all Angular NPM packages. That would at least be consistent with the error message you're getting.
Can you provide repro steps based on deploying to a completely new Azure website? (Don't try re-deploying to your broken one, as you might now have a corrupted node_modules directory that continues to break further deployments)

I've been testing Azure deployment from several angles today, and didn't hit any issue like this. Since we don't have any information on how to reproduce it, I'm going to put it down to something unique to your scenario (e.g., a network failure during NPM restoration) and close this.
If you are able to describe how to reproduce the issue, please reopen with details!

@SteveSanderson Thanks for testing, I'm sorry I didn't have time these weeks to do further testing. I checked quickly that the node version as you suggested was indeed correctly set in the application settings in Azure, so I don't think that might be the issue I was facing.
I've deleted everything on Azure and will try again from scratch, writing down everything I'm doing and will reopen an issue with all the details if I face the issue again.
The main difference between what I did to publish and what you did in your blog post was that I'm using the dotnet publish command from a docker image (I'm using Atlassian Bitbucket Pipelines for CI/CD) and then copying everything in the publish repository to the publish FTP folder in Azure, so it's kind of reverse of what you do as far as I can tell.


JS/TS syntax highlighting breaks with regex v mode and ? character

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

VS Code Version: 1.89.1
OS Version: OSX 14.5

Steps to Reproduce:
Syntax highlighting breaks in (at least) three ways:

Named capture groups: /(?<name>http)/v
Greedy quantifier: /https?/v

Here's small demos. It breaks much worse than shown...

https://github.com/microsoft/TypeScript-TmLanguage/issues/1021
https://github.com/microsoft/TypeScript-TmLanguage/issues/1006


SW-1: As a developer, I want to create a PoC

Closing this issue since the base project has been created. Need to expand on more stories.


Required json format change

We need to add a level to at least

integrals
ranges
peaks

This level will contain:

options
values (the corresponding current array, integrals, ranges or peaks)

https://github.com/cheminfo/nmr-displayer/blob/9beafe7c2e235a351115c364b26b79d57a71da11/src/data/data1d/Datum1D.js#L321-L327

it's done

You could explain why we have a copy of ranges (_ranges) in the React Range Panel ?

@lpatiny
can you point to code line?

Seems you removed https://github.com/cheminfo/nmr-displayer/blob/master/src/component/panels/RangesTablePanel.jsx#L47-L66
We have this _data (same in peaks)

@lpatiny
https://github.com/cheminfo/nmr-displayer/issues/255#issuecomment-572498229


select config for volume

Closes #2474

Coverage increased (+0.003%) to 71.663% when pulling d35f6d2ab96bd9c3fceabebf1e918a528a5a1683 on na3d:volume-select-config into 6d6032bc9cef2e74395e3cb60f233c4d8e1d1c94 on linode:develop.

Coverage decreased (-0.3%) to 71.359% when pulling 89338056ce64e8d3c048974af27f50da647c92d0 on na3d:volume-select-config into 6d6032bc9cef2e74395e3cb60f233c4d8e1d1c94 on linode:develop.

There are two places that need to have support for this: when adding a Volume and when attaching a volume. The issue you referenced is for the latter problem, this PR add support for the former. :)

Coverage increased (+0.05%) to 71.12% when pulling dbf6256f8e0a2a7efaafc9c2b2db60b6f4b8442e on na3d:volume-select-config into 1b0ccebb502afdb8ea61c90bcf9d5e4bb78dfa92 on linode:develop.

Coverage increased (+0.03%) to 71.099% when pulling c206ef575fbfd6dd521b7fbd84db5af0916ea400 on na3d:volume-select-config into 1b0ccebb502afdb8ea61c90bcf9d5e4bb78dfa92 on linode:develop.


running MetaMorpheus is command line

When I unzip "MetaMorpheusDotNetFrameworkAppVeyor.zip" it looks like an empty folder, the file itself is only 22 bytes. Also not sure how running CMD.exe -t would work, isn't CMD.exe the command line editor. Sorry, a bit of a newb here.

We're working on that right now. Please use the previous version until we can make a new release. This is only a command line issue and there have been no really big changes that would alter your results. There are some small bug fixes and addition of new features. Let us know if you have trouble to get the previous release.

We're having an issue building the command-line version via our pipeline, sorry about that. I have built+uploaded 0.0.302. Please let us know if you encounter any issues.