Vim9: need str2nr() to compare g:actual_curwin to window ID

Describe the bug
In Vim9 script, we cannot directly compare g:actual_curwin (nor g:actual_curbuf) to a window ID (or to a buffer number).
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    vim9
    set laststatus=2 statusline=%{CurrentFlags()}
    def g:CurrentFlags(): string
        return (g:actual_curwin == win_getid() ? '[curwin]' : '[not curwin]')
          .. (g:actual_curbuf == bufnr() ? '[curbuf]' : '[not curbuf]')
    enddef
EOF
)

E1030 is raised:
E1030: Using a String as a Number: "1000"

Expected behavior
No error is raised, because we most probably want to use these variables in numeric comparisons, not string comparisons.
Environment

Vim version: 8.2 Included patches: 1-2269
OS: Ubuntu 16.04.7 LTS
Terminal: xterm(363)

Additional context
Workaround: pass the variables to str2nr():
vim9script
set laststatus=2 statusline=%{CurrentFlags()}
def g:CurrentFlags(): string
    return (g:actual_curwin->str2nr() == win_getid() ? '[curwin]' : '[not curwin]')
      .. (g:actual_curbuf->str2nr() == bufnr() ? '[curbuf]' : '[not curbuf]')
enddef

Still, I think people would prefer that the variables are recast as numbers; and it makes sense for a window ID and a buffer number to be stored in number variables rather than string variables.  I don't think it would cause issues in legacy scripts where a number can be automatically coerced into a string if necessary.

Patch adding a test:
diff --git a/src/testdir/test_vim9_script.vim b/src/testdir/test_vim9_script.vim
index d9aa71e26..28391337b 100644
--- a/src/testdir/test_vim9_script.vim
+++ b/src/testdir/test_vim9_script.vim
@@ -732,6 +732,19 @@ def Test_statusline_syntax()
       set laststatus statusline=
   END
   CheckScriptSuccess(lines)
+
+  # g:actual_curwin and g:actual_curbuf are numbers, not strings
+  lines =<< trim END
+    vim9script
+    set laststatus=2 statusline=%{CurrentFlags()}
+    def g:CurrentFlags(): string
+      return (g:actual_curwin == win_getid() ? '[curwin]' : '[not curwin]')
+        .. (g:actual_curbuf == bufnr() ? '[curbuf]' : '[not curbuf]')
+    enddef
+    redrawstatus
+    set laststatus statusline=
+  END
+  CheckScriptSuccess(lines)
 enddef

 def Test_list_vimscript()

Patch adding a test:

Sorry, I didn't reset 'laststatus' properly at the end.  That's because I started from a copy of the previous test where the option is incorrectly reset.  New patch:
diff --git a/src/testdir/test_vim9_script.vim b/src/testdir/test_vim9_script.vim
index d9aa71e26..3fb11af8f 100644
--- a/src/testdir/test_vim9_script.vim
+++ b/src/testdir/test_vim9_script.vim
@@ -729,7 +729,20 @@ def Test_statusline_syntax()
       endfunc
       set laststatus=2 statusline=%!Status()
       redrawstatus
-      set laststatus statusline=
+      set laststatus&vim statusline=
+  END
+  CheckScriptSuccess(lines)
+
+  # g:actual_curwin and g:actual_curbuf are numbers, not strings
+  lines =<< trim END
+    vim9script
+    set laststatus=2 statusline=%{CurrentFlags()}
+    def g:CurrentFlags(): string
+      return (g:actual_curwin == win_getid() ? '[curwin]' : '[not curwin]')
+        .. (g:actual_curbuf == bufnr() ? '[curbuf]' : '[not curbuf]')
+    enddef
+    redrawstatus
+    set laststatus&vim statusline=
   END
   CheckScriptSuccess(lines)
 enddef

Patch adding a test:

Sorry, I didn't reset 'laststatus' properly at the end.  That's because I started from a copy of the previous test where the option is incorrectly reset.  New patch:
diff --git a/src/testdir/test_vim9_script.vim b/src/testdir/test_vim9_script.vim
index d9aa71e26..3fb11af8f 100644
--- a/src/testdir/test_vim9_script.vim
+++ b/src/testdir/test_vim9_script.vim
@@ -729,7 +729,20 @@ def Test_statusline_syntax()
       endfunc
       set laststatus=2 statusline=%!Status()
       redrawstatus
-      set laststatus statusline=
+      set laststatus&vim statusline=
+  END
+  CheckScriptSuccess(lines)
+
+  # g:actual_curwin and g:actual_curbuf are numbers, not strings
+  lines =<< trim END
+    vim9script
+    set laststatus=2 statusline=%{CurrentFlags()}
+    def g:CurrentFlags(): string
+      return (g:actual_curwin == win_getid() ? '[curwin]' : '[not curwin]')
+        .. (g:actual_curbuf == bufnr() ? '[curbuf]' : '[not curbuf]')
+    enddef
+    redrawstatus
+    set laststatus&vim statusline=
   END
   CheckScriptSuccess(lines)
 enddef

Well, the help explicitly states "These values are strings".  Changing that would not be backwards compatible.
As you mention, str2nr() can be used.  I don't think we should do something else, such as providing other variables that are type number, or detecting that a Vim9 function is used.  Usin str2nr() is simple enough.

Well, the help explicitly states "These values are strings".  Changing that would not be backwards compatible.
As you mention, str2nr() can be used.  I don't think we should do something else, such as providing other variables that are type number, or detecting that a Vim9 function is used.  Usin str2nr() is simple enough.


Fix MonitoringTest by using PillarboxExoPlayer directly

Pull request
Description
The MonitoringTest was creating its own Monitoring instance to try to replicate PillarboxExoPlayer internals. This PR changes that to reuse the instance from PillarboxExoPlayer directly.
Changes made

Self-explanatory.

Checklist

[x] Your branch has been rebased onto the main branch.
[ ] APIs have been properly documented (if relevant).
[ ] The documentation has been updated (if relevant).
[ ] New unit tests have been written (if relevant).
[ ] The demo has been updated (if relevant).
[x] All pull request status checks pass.

It looks like I was lucky the first couple of times I ran the tests on the CI 😅
I'll keep investigating this flakiness.

It should be good for review now @StaehliJ


Round2+_DeciderTimer:  After the decider from round 1 became a player in round 2, they never saw a timer on their screen when it was supposed to appear.

iphone6s was the first decider in round 1. They made the desktop the new decider in round 2. When the timer first appeared in the brainstorm phase, 6s didn’t see a timer at all, but received the correct actionbox text and templates. No errors occurred, it was just hidden from view/not shown.
I am almost positive this occurred on the desktop player when it got to round 3, not showing timers on their screen even though they went from player to decider, though I can’t verify this part 100%.

I made a fix for this.

confirmed fixed


Fix issues with Windows 7 notifications and improve design

@RocketChat/core

Closes #297
Closes #305
Closes #255

Improve notification design for Windows 7 and below
Fix issues with notifications being displayed behind other windows
Fix issue with multiple windows being created per notification
Fix issue in regular notifications when clicking it doesn't open app when minimised
Added ability to click on notifications so it takes you to the correct message

@Sing-Li Can you test this on your windows 7?

Gladly. For 64 bit win 7 anyways.  @alexbrazier. Do you have an installer build for testing this PR?  Thx

@Sing-Li You can download the installer from here https://ci.appveyor.com/project/RocketChat/rocket-chat-electron/build/1.0.175/artifacts
I have tested it on Windows 7 x32 with no issues.

Thanks - @alexbrazier !

Tested on Windows 7  64 bit Home Premium.   Worked as described - including message context when notification clicked.    @rodrigok

@Sing-Li @alexbrazier awesome, thanks


Remove items from repo does not remove them from Github LFS server

I reverted some changes I had made to a repo. The LFS files seem to remain on the LFS server since its telling me I'm still using .94GB. Is there anyway to delete the files from the LFS server?

This really depends on the behavior on the LFS server, and has nothing to do with the open source project. Contact https://github.com/contact, and I'll help you out there :)


Why adjustmentLayer fail to be exported as images?

I used the following code for creating images of group layers. Something went wrong when Adjustment Layer were exporting. Like Curves and Color Balance were fail to be exported, so that the final exported image was viewed differently from it is in PSD. Could you please help me to take a look?
im = groupLayer.compase()
im.save('test.png')

test.psd
code exported

photoshop exported

@fpgdev Adjustment layers are not supported as of now

@fpgdev Adjustment layers are not supported as of now
Thank you for your reply, do you think there are other ways to achieve this? For example PLI

One obvious approach is to use Photoshop scripting

One obvious approach is to use Photoshop scripting

@kyamagu  got it, thanks

@fpgdev Good. FYI, there is a python interface


Feat request: Destroy the EKS cluster with "terraform apply" instead of "terraform destroy"

Requested feature:
Right now only terraform destroy in the plan, deletes all resoruces that terraform-aws-eks module has created. But people use to have a terraform pipeline that does always "terraform init" and "terraform apply". So it brings us to have to run manually a terraform destroy sikpping the pipeline execution.

Hi all,
I'm having some issues regarding this scenario also.
From what I understood, if we want to remove the EKS cluster, we can't remove the provider because it will be needed to destroy resources created inside the EKS cluster.
I want to implement a gitops approach and easily do this. By removing the EKS module, the terraform would remove the cluster with one PR. From what I have read, this doesn't seem possible to do. Has anyone tried to achieve this?
Maybe we can use terraform CDK for these specific use cases.

Here is my workaround:
data "aws_eks_cluster" "cluster" {
  count = var.create_eks || contains(data.aws_eks_clusters.clusters.names, var.cluster_name) ? 1 : 0
  name = try(module.eks.cluster_id, var.cluster_name)
}

data "aws_eks_cluster_auth" "cluster" {
  count = var.create_eks || contains(data.aws_eks_clusters.clusters.names, var.cluster_name) ? 1 : 0
  name = try(module.eks.cluster_id, var.cluster_name)
}

This way, the datasource will be correctly configured to destroy resources that exists in the state but not anymore in the manifests.

Closing this for now with the latest release of v18.x. Please let me know if your issue still persists after updating to the latest version and we can revisit

Confirming that this issue still exists in v18.21.0.
So what is the solution, how to remove cluster with code in CI/CD without using destroy ?

create = false


feat: spanner NUMERIC support

TODOs:

Revert the generated code commit.

Thanks for the super quick review @jskeet. I was mostly putting this PR up to make sure I was on the right track. We won't be able to merge this in for a few weeks since backend support is still not available. So I will put this back into draft mode. In the meantime I will look into the TODOs that I listed in the first comment.

@jskeet this is ready for another round of reviews. Since you last looked at it, I've added:

Integration tests. Note that they can only run be against the staging backend for now.
Type conversions from integral and floating-point numeric types to Numeric, and associated tests.

I'm on vacation today, but I'll look at this and the integration-tests-against-emulator PR on Monday.

No rush @jskeet. Please take your time.

Sorry about that. The specific commit to review is https://github.com/googleapis/google-cloud-dotnet/pull/5057/commits/405dd3fbeebcafe6237c870adaaaa4a8904e0746. Nothing else has changed since you last reviewed the PR (sorry about the rebasing, I understand it makes code reviews difficult for incremental work).
If you're happy with the change, then after we merge this in, if you could help us cut a new release, that would be wonderful.


missing swagger/openapi endpoint definitions for autocomplete tags

whoops reported by @ndrwrbgs who is experimenting

Closed by #61


NextJS: Upgrade to version 14

Overview
While NextJS 13 will be supported for the forseeable future, we should consider upgrading to NextJS 14 to leverage NextJS's newest features and 14's stronger support for bug fixes and security patches.
Action Items

[ ] Upgrade to Node 18: https://github.com/chaynHQ/bloom-frontend/issues/703 to unblock.
[ ] Consider if this issue should be completed before or after upgrading NextJS: https://github.com/chaynHQ/bloom-frontend/issues/702
[ ] Follow this guide to upgrade to 14: NextJS Docs: Upgrading to 14
[ ] Run all tests and report any errors here. If there are many or highly complex errors, we will open new issues for them. Otherwise, please resolve these errors.

Resources/Instructions

NextJS Blog: Introducing 14
Medium blog post: Should I update my app to Next 14?

Assign this issue to me .

@jackdraks You are assigned!

Hey @jackdraks! It's been 1 month since any updates.
Would you like to keep working on this issue?
Totally understand if you need more time or if things came up and you cannot complete this issue anymore.
If you have any questions, please let us know here and we're happy to help!

Hi @kyleecodes this issue can be closed as it is already solved


Identify renamed lemmas

If a theorem disappeared and a new of the same type appeared, then show as "abc renamed to cde".

This should already happen, do you have an explicit example where it doesn't?

You're right, it identifies renames. However, in a more difficult case (renamed, restored the old lemma as a deprecated alias) it randomly reports new or old lemma as added.


Adding project reference step causes compilation problems.

> Add a project reference from the UITest project to the Xamarin.Android project
On VS 2017 15.7.4 this step causes the test project to not get past the nuget restore phase of the build with the following error(s):
NU1201	Project Android2.Android is not compatible with net461 (.NETFramework,Version=v4.6.1) / win-x64. Project Android2.Android supports: monoandroid71 (MonoAndroid,Version=v7.1)

NU1201	Project Android2.Android is not compatible with net461 (.NETFramework,Version=v4.6.1) / win-x86. Project Android2.Android supports: monoandroid71 (MonoAndroid,Version=v7.1)

NU1201	Project Android2.Android is not compatible with net461 (.NETFramework,Version=v4.6.1) / win. Project Android2.Android supports: monoandroid71 (MonoAndroid,Version=v7.1)

NU1201	Project Android2.Android is not compatible with net461 (.NETFramework,Version=v4.6.1). Project Android2.Android supports: monoandroid71 (MonoAndroid,Version=v7.1)

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

ID: b2f6b353-936e-2536-37cb-b8190aea7743
Version Independent ID: 4f27ec32-a6f7-3de5-2635-87bacf04e724
Content: Automating Xamarin.Forms Testing with App Center - Xamarin
Content Source: docs/xamarin-forms/deploy-test/uitest-and-test-cloud.md
Product: xamarin
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @KevM - I just tried this out  with VS2017 15.7.5, and I didn't get the same error.  However, my solution was created fairly recently (in the past week).  This doesn't sound like an issue with the docs, but perhaps a problem that existed in VS2017 15.7.4 and/or possibly project files that were created with an out-of-date template.
I'd suggest updating to 15.7.5 and seeing if this happens again.  If it does, then the next best course of action would be to file a bug on Developer Community.

I too am having the same exact issue, even with vs 15.7.5. I assume, there is no way to run UITest offline anymore?

I also had this issue. It ended up working after i removed the references to android and ios project (only reference the shared project)

I also having the same problem.
My visual studio is 15.8.1, Shared project, Android and iOS is using .NET Standard 2.
@bytemi Can you please tell me how are you able to run the UI Test offine without referencing the android and ios project?

I am still getting this error.  When creating a new Xamarin.Forms project and then creating a new "Xamarin.UITest Cross-Platform Test Project."  I am not the only one.  There is a stackoverflow issue on this.  (https://stackoverflow.com/questions/51296837/xamarin-uitesting-nu1201-error-android-8-1-is-incompatible-with-netframework-4)  They are recommending that you just install the app and run it with the bundle id.
Here is the error message.
Severity	Code	Description	Project	File	Line	Suppression State
Error	NU1201	Project Automated.Testing.Android is not compatible with net461 (.NETFramework,Version=v4.6.1). Project Automated.Testing.Android supports: monoandroid81 (MonoAndroid,Version=v8.1)	Automated.Testing.Test	C:\Users\seans\source\repos\Automated.Testing\Automated.Testing.Test\Automated.Testing.Test.csproj	1
Here is the visual studio information:
Microsoft Visual Studio Enterprise 2017
Version 15.8.7
VisualStudio.15.Release/15.8.7+28010.2046
Microsoft .NET Framework
Version 4.7.03056
Installed Version: Enterprise
Visual C++ 2017   00369-60000-00001-AA052
Microsoft Visual C++ 2017
Application Insights Tools for Visual Studio Package   8.13.10627.1
Application Insights Tools for Visual Studio
ASP.NET and Web Tools 2017   15.8.05085.0
ASP.NET and Web Tools 2017
ASP.NET Core Razor Language Services   15.8.31590
Provides languages services for ASP.NET Core Razor.
ASP.NET Web Frameworks and Tools 2012   4.0.30625.0
For additional information, visit https://www.asp.net/
ASP.NET Web Frameworks and Tools 2017   5.2.60618.0
For additional information, visit https://www.asp.net/
Azure App Service Tools v3.0.0   15.8.05023.0
Azure App Service Tools v3.0.0
Azure Data Lake Node   1.0
This package contains the Data Lake integration nodes for Server Explorer.
Azure Data Lake Tools for Visual Studio   2.3.3000.2
Microsoft Azure Data Lake Tools for Visual Studio
Azure Functions and Web Jobs Tools   15.8.05023.0
Azure Functions and Web Jobs Tools
Azure Stream Analytics Tools for Visual Studio   2.3.3000.2
Microsoft Azure Stream Analytics Tools for Visual Studio
Bundler & Minifier   2.8.396
Adds support for bundling and minifying JavaScript, CSS and HTML files in any project.
C# Tools   2.9.0-beta8-63208-01
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
Extensibility Message Bus   1.1.49 (remotes/origin/d15-8@ee674f3)
Provides common messaging-based MEF services for loosely coupled Visual Studio extension components communication and integration.
Fabric.DiagnosticEvents   1.0
Fabric Diagnostic Events
JavaScript Language Service   2.0
JavaScript Language Service
JavaScript Project System   2.0
JavaScript Project System
JustDecompilePackage2017 Extension   1.0
JustDecompilePackage2017 Visual Studio Extension Detailed Info
Microsoft Azure HDInsight Azure Node   2.3.3000.2
HDInsight Node under Azure Node
Microsoft Azure Hive Query Language Service   2.3.3000.2
Language service for Hive query
Microsoft Azure Service Fabric Tools for Visual Studio   2.3
Microsoft Azure Service Fabric Tools for Visual Studio
Microsoft Azure Stream Analytics Language Service   2.3.3000.2
Language service for Azure Stream Analytics
Microsoft Azure Stream Analytics Node   1.0
Azure Stream Analytics Node under Azure Node
Microsoft Azure Tools   2.9
Microsoft Azure Tools for Microsoft Visual Studio 2017 - v2.9.10730.2
Microsoft Continuous Delivery Tools for Visual Studio   0.4
Simplifying the configuration of Azure DevOps pipelines from within the Visual Studio IDE.
Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines
Microsoft Library Manager   1.0
Install client-side libraries easily to any web project
Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers
Microsoft Visual C++ Wizards   1.0
Microsoft Visual C++ Wizards
Microsoft Visual Studio Tools for Containers   1.1
Develop, run, validate your ASP.NET Core applications in the target environment. F5 your application directly into a container with debugging, or CTRL + F5 to edit & refresh your app without having to rebuild the container.
Microsoft Visual Studio VC Package   1.0
Microsoft Visual Studio VC Package
MLGen Package Extension   1.0
MLGen Package Visual Studio Extension Detailed Info
Mono Debugging for Visual Studio   4.11.11-pre (8fb558f)
Support for debugging Mono processes with Visual Studio.
Node.js Tools   1.4.20907.4 Commit Hash:f048d52e417a8902dea9e42828b7fdab92d4aaaa
Adds support for developing and debugging Node.js apps in Visual Studio
NuGet Package Manager   4.6.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit http://docs.nuget.org/.
ProjectServicesPackage Extension   1.0
ProjectServicesPackage Visual Studio Extension Detailed Info
Redgate ReadyRoll   1.17.18155.10346
Extend DevOps processes to your SQL Server databases and safely automate database deployments.
Visit https://www.red-gate.com/readyroll for more information.
Copyright (C) 2011 Red Gate Software Ltd. All rights reserved.
This software contains components from Component Owl.
SQL Server is a registered trademark of Microsoft Corporation.
Visual Studio is a registered trademark of Microsoft Corporation.
ReadyRoll contains code from the following open source software:
NuGet https://www.nuget.org/
SQL LocalDB Wrapper https://github.com/martincostello/sqllocaldb
Autofac https://autofac.org/
Json.NET https://json.net/
MahApps.Metro http://mahapps.com/
SemVer https://github.com/maxhauser/semver
Log4Net http://logging.apache.org/log4net/
StringTemplate https://github.com/antlr/stringtemplate4
Extended WPF Toolkit https://wpftoolkit.codeplex.com/
Code InfoBox VSX http://www.codeproject.com/Articles/55196/Code-InfoBox-Visual-Studio-Extension-VSX
OctoPack https://github.com/OctopusDeploy/OctoPack
SQLite https://sqlite.org/
This product contains icons from http://www.visualpharm.com distributed under a free backlink license.
For license details or other notices relating to the above software, please see NOTICE.TXT and EULA.rtf in the ReadyRoll application folder.
Redgate SQL Prompt   <IP_ADDRESS>01
Write, format, and refactor SQL effortlessly
ResourcePackage Extension   1.0
ResourcePackage Visual Studio Extension Detailed Info
ResourcePackage Extension   1.0
ResourcePackage Visual Studio Extension Detailed Info
Snapshot Debugging Extension   1.0
Snapshot Debugging Visual Studio Extension Detailed Info
SQL Server Data Tools   15.1.61808.07020
Microsoft SQL Server Data Tools
ToolWindow Extension   1.0
ToolWindow Visual Stuido Extension Detailed Info
ToolWindowHostedEditor   1.0
Hosting json editor into a tool window
TypeScript Tools   15.8.20822.2001
TypeScript Tools for Microsoft Visual Studio
Visual Basic Tools   2.9.0-beta8-63208-01
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Visual F# Tools 10.2 for F# 4.5   <IP_ADDRESS>.  Commit Hash: 55a3dc3231c95c77f81ee53f7d29152029da7408.
Microsoft Visual F# Tools 10.2 for F# 4.5
Visual Studio Code Debug Adapter Host Package   1.0
Interop layer for hosting Visual Studio Code debug adapters in Visual Studio
Visual Studio Tools for Containers   1.0
Visual Studio Tools for Containers
Visual Studio Tools for Universal Windows Apps   15.0.28010.2046
The Visual Studio Tools for Universal Windows apps allow you to build a single universal app experience that can reach every device running Windows 10: phone, tablet, PC, and more. It includes the Microsoft Windows 10 Software Development Kit.
VisualStudio.Mac   1.0
Mac Extension for Visual Studio
Xamarin   <IP_ADDRESS>6 (d15-8@1ae9b59d7)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.
Xamarin Designer   4.15.9 (d000f568b)
Visual Studio extension to enable Xamarin Designer tools in Visual Studio.
Xamarin Live Reload   0.3.27 (remotes/origin/dev@207abf7)
Provides live reload capabilities for Xamarin XAML.
Xamarin Templates   1.1.116 (9619170)
Templates for building iOS, Android, and Windows apps with Xamarin and Xamarin.Forms.
Xamarin.Android SDK   <IP_ADDRESS> (HEAD/a8a3b0ec7)
Xamarin.Android Reference Assemblies and MSBuild support.
Xamarin.iOS and Xamarin.Mac SDK   <IP_ADDRESS> (84552a4)
Xamarin.iOS and Xamarin.Mac Reference Assemblies and MSBuild support.


Domain activation is Failed - One or more of the CNAMEs you provided are already associated with a different resource

App Id: d1setlyy3u4krt
Region: us-east-1
Step: Domain activation
Status: Failed

Note:
there is only one CNAME entry for partner.serviceally.co

Issue/question
A clear and concise description of what the issue/question is.
Error message
If there is an error message, please include it here.
Error message: One or more of the CNAMEs you provided are already associated with a different resource.
*Additional information

Were you able to get yours resolved? @beto

Looks like this appId has since been deleted.  Going to resolve this ticket.
If you are still experiencing issues you can try following these steps:

Is this domain connected to a different Amplify App that you own? If so, make sure you are not trying to reuse one of the hostnames.
If you are using www.domain.com on the other app, you cannot use www.domain.com on this app
You can use other hostnames, like blog.domain.com
If you had this domain successfully connected to Amplify and then recently (within the last hour) deleted it, please wait and try again.
Check the Cloudfront console to see if you have this domain deployed to any distributions
If you are positive no CloudFront distribution exists (including in other accounts) using this domain, and only if it would not be disruptive to any currently running services, try deleting and recreating the hosted zone


Channel is existed and is a voice channel , but the bot say doesn't exist or isn't a voice channel

Question as the title says
Wolf狼狼 — 今天 21:21
!enter 999
The Bot
機器人
— 今天 21:21
The channel #999 doesn't exist or isn't a voice channel.

Use discord.js v12 not v13

or use the channel ID

Sadly there is no fix and v12 doesn’t work


Next release version

Should we finally make the leap to 1.0.0 for the next release?

Makes sense to me, given #421

Sounds reasonable to me too. Anything in particular we'd like to mark as milestones for v1.0 ?

Wasn't the original goal of the project to offer a PuppetDB backed dashboard to replace the old Ruby dashboard?
I think that goal was met.

I agree. It is way better even.
The only feature I miss from the original Puppet Dashboard is the CSV export, but I know someone who's looking into this :)

My suggested milestone list for 1.0.0:

#30  (related to #428  )
#283
#333
#360
#361
an update to jQuery 3.x and Semantic UI 2.x
new screenshots

#283 and #360 are pretty much the same.
And a general reminder, if someone would like to remove Python 2.6 support from release 1.0.0, remember that 2.6 is still the Python shipped on EL 6. Since containers can't run properly on the EL 6 Kernel, only alternative would be to install a newer Python from Software Collections.

I picked those two because they were already planned for 0.3.0 at the time.

I suggest using new script for sorting tables, as current sorter cannot handle ip adresses, time (days, hours) and memory

Any suggestions for a better sorter, @emlun007 ? :)

@larsnaesbye , https://github.com/christianbach/tablesorter
Should be able to sort ip and time

Or maybe we should simply make a 1.0 release now?

@mterzo - any things you think holds a 1.0 release back?

@larsnaesbye why do you care so much. It's just a number.
Also there are so many issues open right now. Fix something instead of bothering about version number over and over again.
I don't mean to be rude but I honestly feel like @mterzo is getting spammed lately.
Also @emlun007 if you open a PR it should be working or at least semi-working, if it is semi-working describe what works and what doesn't.

@kirkins, PR #429 (button for saving csv) works. I have been using the function for 2 months now. I am just waiting for code to be merged.

@emlun007 you have a point, I have had PR open for a long time too.
I'm frustrated with this repo because I get so notifications but there are never any updates.
Someone else should be added to the project to sort through everything...

@larsnaesbye sorry if that was harsh. I'm having a rough morning. Cheers

@kirkins, I have code for new PRs but I don't want to spam too much, so I'm just waiting for the first one to be merged. It's been three months soon 🐌 🐌 🐌

@kirkins no problem mate. I'll go fix one of the things alright :-)
I was trying to figure out a kind of priority list before we can re-pull some of the good changes already in. It came out wrong.

Hi all. Is it possible, by any chance, to RELEASE THE KRAKEN ?
I am only able to 'puppet install ..' , so a new release would be extremely helpful.

@florinbesleaga Now the codename has to be 1.0 Kraken :-D

@wyardley Thank you very much for the pull request merge. Do you think it's possible to make a release from the current trunk, as version 0.3.4? It would help some of us quite a bit.

Agreed! Would be great!

@larsnaesbye I haven’t worked on this project much so probably someone else should cut the release. Maybe post in the Vox channel on IRC and see if someone can do that.

Somewhat, yes.

I have now joined the Freenode IRC channel #voxpupuli - here's hoping we can find some guidance there :-)

having any new release whatsoever would be a good start. ;)

Pushing the issue on #voxpupuli IRC channel like I did might help. It did make some waves...

@bastelfreak , could you perhaps tell us what to do? I'm sure many of us would like to help, but the steps to a new release are somewhat unclear....

@mterzo It would be really great if someone with the rights to do so could enable the people who try to help here to actually move the project forward. I think there is mainly an issue around the release process? Every other month something here flairs up and there is once again hope it will be revived.
There is clearly the need of an open source dashboard that uses the awesome power of the PuppetDB. In a company I work for we currently use Puppetboard as it is the only one available with an acceptable feature set, but it has some serious scaleability issues and there are other features we would like to implement. It is clear that we need a solution and at the moment we are asking ourselves if it is even worth to attempt to fix it if the project is abandoned anyway or if we should just start over with yet another dashboard project.

@ZeroPointEnergy if you have a company using this free software project that is avaialble for free, maybe you should consider getting involved in funding its development instead of dropping it for another one, or starting your own (see also this article for context).
that said, i am not sure the problem is enabling people as much as just sitting down and just doing the work. this is a Python project, so I think doing a release probably involves simply:

lay a tag
upload the release to PyPI

From what i can tell, the people that can do the former are @mterzo and anyone in the @voxpupuli team (which is quite a lot of people, possibly). The latter is similar, the PyPI maintainers include @mterzo @corey-hammerton  @daenney and, again, all of @voxpupuli.
Obviously, the problem is we need someone to step up and actually own up to manage this project, which is harder to do than just drop comments on github issues, unfortunately. :p

@anarcat The last time I had looked, I don't think voxpupuli was a contributer on the PyPI project. But also, I don't think "all" of voxpupuli directly has access -- I'm not sure who has access to the voxpupuliorg account in PyPI, but maybe someone else knows. I am pretty confident that this is not an account that any vox contributor has access to.

Look again :wink:

The Voxpupuli is probably owned by the PMC, so at the very least they can add additional people there if that's what's holding this back.
If someone wants to build a milestone for 1.0, spawn the necessary tickets, changelog updates etc. I'm happy to do the actual uploading to PyPi.

Yes, I see that it's there now, I just meant that I didn't remember it being there when I looked a while back. Could be wrong though.

@daenney , I guess there is a consensus that 1.0 is pretty much ready (?), so only the changelog and perhaps updated screenshots are needed.

@anarcat I'm not sure why you got the impression I ask someone to work for free?
What I actually said is that the project looks abandoned and therefor it would be hard to justify to get time to work on it. I had the impression from previous comments that voxpupuli can't release to pypi and that is the actual blocker.

On 2019-05-23 22:58:28, Andreas Zuber wrote:

@anarcat I'm not sure why you got the impression I ask someone to work for free?

Are you offering to pay?

Hi people. Can we stop the snapping at each other please? I'd like to get back to the topic on hand, releasing a new version.

Hi people. Can we stop the snapping at each other please? I'd like to get back to the topic on hand, releasing a new version.

Sure, my apologies.

Nope, they didn’t And it certainly can wait until after a release :)

by the way, i must say i truly appreciate that an actual unicorn came in here to calm me down. thanks @daenney :)

#502 is my attempt at a pull request for a simple changelog and version update.

Great to see you'll release a new version.
I have some open PRs that could be merged in before 1.0.
https://github.com/voxpupuli/puppetboard/pull/441
https://github.com/voxpupuli/puppetboard/pull/438
https://github.com/voxpupuli/puppetboard/pull/439
I'm available if these require any additional changes.

Glad to see progress on this!
Me too I have #477 that I'd like to be included if possible. I'll take care of resolving the rebase conflicts tomorrow (sorry, don't have access right now)

I'll adjust the Changelog if more PRs get merged, of course.
But I'm thinking that we shouldn't wait too long with this already delayed release, so maybe we could ask whoever makes the 1.0.0 tag to prepare a 1.0.1 tag too for a release not long after?

@alexjfisher : Sorry to ask you yet again, but do think the PRs #438 #439 #441 and #477 could get merged?

Sorry for commenting and maybe trigger some people again, but why not just give @larsnaesbye commit and publish rights?

That's not what we're here to discuss right now and there's a process to go through to do that. Lets stay on topic please. Feel free to file a separate issue and tag the PMC on it.

i agree that we shouldn't get lost in "what possible pending PR we could merge before release problem". if the PR isn't merged, it's not going to get merged before the release. we can always make new releases later.
if you want your PR to get merged, get it reviewed: that can be done by someone else on your team or any other user of GitHub. right now, there is zero PR that has a positive review and that passes test (apart from the changelog one), so it's unlikely anyone here will have time to fix that before the release.
let's just get this show on the road!

It looks like I caused some confusion there. My response was to they "why don't we just give X commit access".
W.r.t the pending PRs, if people can commit time in the next week to get those reviewed and rebased I think we can wait for that. But if it's going to take more time this risks stalling again which would be a shame. I'm entirely happy to accept external/first-time reviewers on any of the pending PRs, none look particularly complex.

@anarcat one of my PRs is fixing the graph on the node page.
It wasn't merged because someone was going to do it a different way. That was 2 years ago.
I guess I'll come back in another 2 years and see if he had the time.

I guess I'll come back in another 2 years and see if he had the time.

Snark isn't helping. I know it's annoying that PRs linger and for that I'm sorry. However, people have lives too and that can take priority over things. You can use the mailing lists or reach out to the PMC directly if you'd like to bring attention to this problem.

@anarcat as for your rant on not getting paid for this you had above.
I wouldn't pay you a dime. This project had potential and it's been completely neglected. On top of that you haven't even contributed to this project.
For you to come here are start talking about money is an insult to those of us who have donated our time to fixing bugs but were completely ignored.

Oy. I've asked people to stop snapping at each other. So stop. I'm not going to ask again. Whatever issues you might have with each other, this is not the place. @anarcat was kind enough to recognize that and leave it for what it was. I expect you to do the same @kirkins.

One other question — is it intentional or accidental that Vox contributors don’t have approval or merge access on this repo anymore? I know there was some team shuffling so maybe accidental?

Also, totally agree with shipping first, then merging more changes vs vice-versa.

Accidental-intentional.
B/c pypuppetdb and puppetboard are very different beasts from most of the other things:

Not Ruby
Not a Puppet module

We early on created a separate team to gate access to these repos. Unfortunately the two maintainers appear to have lost interest or be otherwise occupied which caused this to block.

@daenney Hi!

voxpupuliorg is probably owned by the PMC, so at the very least they can add additional people there if that's what's holding this back.

That's indeed correct.  https://github.com/voxpupuli/puppetboard/pull/498#issuecomment-492152720
Looks like someone came through and added the accounts.

Yeah they did, thanks @alexjfisher

Hi people. I had a discussion with @mterzo a few days ago. We added our PyPi account voxpupuliorg to both projects, so we can now do releases as an org and not only from individual accounts. Both projects have a PR open to automate the release process with travis. Anyone that feels confident with python can review them:

https://github.com/voxpupuli/pypuppetdb/pull/147
https://github.com/voxpupuli/puppetboard/pull/503

Let us know if one of you wants to spend more time on one/both projects and we can grant merge permissions here.


docs: add explicit await to clarify sendRedirect is async

🔗 Linked issue

❓ Type of change

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

📚 Description
made the redirect async/await

Thank you ❤


Template path wrong on my Android Install

Great directive, thanks for sharing.
In my tests on Android, it ends up adding plugin.js code in my HTML instead of the tutorial HTML template !
Debugging it seems that in ng-walkthrough.js the line

currentScriptPath.replace(new RegExp("ng-walkthrough.js.*"), 'ng-walkthrough.html');

returns

"file:///android_asset/www/cordova_plugins.js"
instead of the template file

I could fix it hardcoding the templates and icons path as follows but I guess it's not a clean fix.

var templateUrl = "lib/ng-walkthrough/ng-walkthrough.html";
var iconsUrl = "lib/ng-walkthrough/icons/";

Hi, thanks for the update. actually, as a directive, I think it might be cleaner if there was no dependency on an HTML file, so next version I'll embed the template into the js.
Thanks

Hi @Elpetro , new version has been released, can you please update me if the issue was solved?
Thanks


Modify HugeCTR export_ensemble function to work with example

@jershi425

Example: https://github.com/NVIDIA-Merlin/NVTabular/blob/main/examples/getting-started-movielens/04-Triton-Inference-with-HugeCTR.ipynb


fill image to object fit in container

Hi, I want to have an openseadragon container with a height of 90vh.
When i set the height in css this is working fine.
However, I want my image to fill up the space. I think it needs zo zoom in to achieve this.
Is there an easy way to do this?

@shyamkumaryadav Those are some good ideas, but actually we do have a feature for this... @liekeheijne Try using homeFillsViewer: true in your Viewer options.


fix asdf latest python

Fixes #140
This contains bug fixes related to shorthand PR #139.
This makes the partial version lookup compatible with GNU grep and adds a "-latest" suffix to shorthands. asdf list-all python  broke asdf latest python without that suffix but asdf knows to ignore versions with -latest.
Note this means that asdf install python 3.7 won't work anymore (but that change was only in for less than 24 hours). We'll need to use asdf install install python 3.7-latest.

Tested the following on mac and linux and seems to work:
asdf install python 3.9
asdf local python 3.9
python -V
asdf latest python 3.9
asdf latest python

Note that this won't break people's installs that used the 3.9 style shorthands but they won't be able to use that shorthand for new installs after this is merged.

@danhper sorry to bother you but it would be really good if you could review this PR. Right now asdf latest python isn't working but this will fix it.

This PR is not quite right:
asdf latest python 3.9 --> 3.9.13
but
asdf latest python 3.1 --> 3.10.6
was expecting 3.1.5.  I wonder if the decimal point is getting quoted in your regular expression?  For instance, in PR #142, I use ${version//./\\.}.  And although,
asdf install python 3.1-latest -->  downloads 3.1.5
the -latest suffix should not be necessary.
Finally, please try to use POSIX syntax where possible.  asdf-vm is used by systems for which sort and grep are not GNU.  Again,  please see PR #142.

asdf latest python 3.1 is in fact broken but it doesn't appear to ever have worked in this plugin. It's not something either of my PRs caused.

Finally, please try to use POSIX syntax where possible. asdf-vm is used by systems for which sort and grep are not GNU. Again, please see PR https://github.com/danhper/asdf-python/pull/142.

That's fair, I'm not familiar with writing POSIX compatible scripts or how to make sure I'm not using tools that aren't generally available. If you have any tips on methods or tools I could use to check my code for that I'd be happy to use them.

It is critical. That's the primary fix in here as I explain in the description. Without appending the -latest suffix then asdf latest python 3.10 won't work.

Please try PR #142.  gh pr checkout 142 and verify that both asdf latest python 3.10 works as well as asdf install python 3.10.  What am I missing?  Though I see that 3.1 is indeed still broken here:  asdf latest python 3.1 --> 3.10.6

I'm not familiar with writing POSIX compatible scripts or how to make sure I'm not using tools that aren't generally available. If you have any tips on methods or tools I could use to check my code for that I'd be happy to use them.

It might be worth bookmarking SUSv4.  But generally, I'd recommend  going GNU if you can get away with it :)

Removed non-POSIX compatible usage of sort -V. I believe this is entirely POSIX compatible now.

Looks good.  It might be nice to support trailing decimal point, but not essential.

Looks good. It might be nice to support trailing decimal point, but not essential.

I'd be happy to but it seems to work?
❯ asdf latest python 3.9.
3.9.13

The only place it doesn't work would be here, but I'm not sure this is a realistic use-case:
❯ asdf install python 3.10.-latest

Is there a use-case I'm not considering?

Testing shell code is hard :(  I've closed PR #142.  Thank you for your work!

Finally, please try to use POSIX syntax where possible. asdf-vm is used by systems for which sort and grep are not GNU. Again, please see PR #142.

Technically speaking, [[:digit:]] is POSIX syntax. See the POSIX standard for regular expressions, particularly item 6 of section 9.3.5.:

All character classes specified in the current locale shall be recognized. A character class expression is expressed as a character class name enclosed within bracket-  ( "[:" and ":]" ) delimiters.
The following character class expressions shall be supported in all locales:
[:alnum:]   [:cntrl:]   [:lower:]   [:space:]
[:alpha:]   [:digit:]   [:print:]   [:upper:]
[:blank:]   [:graph:]   [:punct:]   [:xdigit:]

That said, if we are assuming that only ASCII 0-9 can form digits of version numbers (which I think is a reasonable assumption), then I have no objection to using [0-9]. Any regex syntax I can think of supports that range notation, so it is probably more portable than the POSIX [[:digit:]].

Since character classes are POSIX, I believe [[:digit:]] generally matches only the ASCII range [0-9].  The Python docs, in particular, point to the Unicode standard, which indicate that this is the case for Python as well.  My objection was more to the use of \d in PR #139 which unfortunately isn't supported by *BSD, for instance.  My personal preference for [0-9] over [[:digit:]] is that it's shorter :)

Since character classes are POSIX, I believe [[:digit:]] generally matches only the ASCII range [0-9]. The Python docs, in particular, point to the Unicode standard, which indicate that this is the case for Python as well. My objection was more to the use of \d in PR #139 which unfortunately isn't supported by *BSD, for instance. My personal preference for [0-9] over [[:digit:]] is that it's shorter :)

👍 I agree that [0-9] is shorter and more clearly expresses the correct intent here; I do think it is the correct way to go in this context. I thought you were responding to the use of [[:digit:]] to replace \d, which is a more apples-to-apples replacement, since that change had already been made by the time you joined the conversation.
Technically, Python \d matches Unicode category Nd, which includes much more than [0-9], unless you supply the ASCII flag or provide the (?a) pattern inline when building the regex. The point about POSIX [[:digit:]] being locale-aware means that it could also match things other than [0-9]. In the case of Python version numbers, I think we just want [0-9] for digits though. I'm not aware of anyone using Bengali or Thai digits in Python version numbers, for example. :)

Hi, and thanks for the fix.
I feel like this adds a bit too many inconsistencies, so I'd suggest we try to get latest:3.7 or so to work with .tool-versions in the main asdf repo rather than introducing another way of doing the same thing here.
I reverted the previous PR to avoid leaving asdf latest broken.


Masked dotplot looks for genes in adata.raw

Description

I am trying to use this plotting tool provided with spapros in order to summarise some example adata.
sp.pl.masked_dotplot(adata, selector, save=True)
I attached a text file of the error:
masked_dotplot_bug.txt
The problem seems to be that the plotting function used requires the gene ids to be in adata.raw.var_names when I know I have them in adata.var_names. IT must be some parameter that can easily force the plotting functions within the spapros customized one to use the names from adata.var_names (use_raw=False?).
Or can you suggest some other idea on how to circumvent this bug?
This issue seems to be common when people load their data from an "h5ad" object.
To Reproduce
Steps to reproduce the behavior:

Loaded file from with the following:

adata = sc.read_h5ad("data.h5ad")

I run the selector function to get marker genes

selector.select_probeset()

run the following plotting function:

sp.pl.masked_dotplot(adata, selector, save=True)
Expected behavior

System [Windows 10]:

Language Version: [e.g. Python 3.9.0]
Virtual environment: [e.g. Conda]

Additional context

I see: If raw is present scanpy's plotting functions look there by default. Unfortunately the sp.pl.masked_dotplot currently doesn't have enabled the use_raw argument or in general **kwargs for scanpy's dotplot. That's something that

[ ]  should be added.

Currently the issue should be solved with del adata.raw - would that work for you?

Tried it but I get some other error:
masked_dotplot_bug.txt

Ah save expects either False or a str (i.e. the path where the img is saved e.g. ./masked_dotplot.png)

That fixed it. Thanks a lot :)

This is now solved with commit ddd5cb7ddc72244eff7ee8e032501314b42fd475.
The keyword use_raw was added and is set to False by default.
Quick example to show the behaviour:
import spapros as sp
adata = sp.ut.get_processed_pbmc_data()
selector = sp.se.ProbesetSelector(
    adata, n_pca_genes=0, n=20, celltype_key="celltype", verbosity=1, save_dir=False
)
selector.select_probeset()

adata2 = adata.copy()
adata2.X = adata2.X.toarray()
adata2.raw = adata2.copy()
adata2.X += 1

sp.pl.masked_dotplot(adata2,selector) # Doesn't use raw counts --> expect full circles (due to +1)
sp.pl.masked_dotplot(adata2,selector,use_raw=True) # Uses raw counts --> expect unchanged circles


update viap.xml

please checkin

Could you please add the "../" back in?  That is required in the comp setup.

Adressed by PR #59


Error handling for menu API requests

Currently we do that:
https://github.com/drunomics/nuxtjs-drupal-ce/blob/1.x/lib/plugin.mjs#L135:
      } catch (error) {
        // Ignore the error and only log it, so the site stays functional.
        // eslint-disable-next-line no-console
        console.log(error)

Planned solution:

Add general axios error handler to drupal-ce axios instance
Have general axios error handler be used for menu API requests - thus we stop ignoring the error
The default should throw the error and handle the typical cases like getPage (response error, no response error, other error)
We need make sure a menu-API errors are giving enough context and tell us which API request is broken

Resolved in #78


Close session to release conn pool and fix typo

Close session to release the connection pool when calling method VerifyConnectivity and fix type on driver comment

Nice work!
You need to sign our CLA for your PR to be mergable:
https://neo4j.com/developer/cla/

https://neo4j.com/developer/cla/

Hi @2hdddg
I have already signed the CLA.
thank you for your response

Hi @2hdddg
How about the status of this PR?

Hi @dynastymasra
Still having failures with the CLA in our internal builds. This has been fixed on 4.0 branch and that will soon be released so I'm closing this PR now as there are no more planned 1.8 releases.
Thank you very much for finding this and submitting the PR.


Styled Components conflict

I use styled components in my Gatsby site site noticed the work on the feedback widget so I thought I would try it = nice
Using styled-components and emotion/styled, will not work together nicely
emotion error
A little more study I saw this Injection Order
So I replaced
import styled from "@emotion/styled"
With the below in buttons.js
import styled, { css } from 'styled-components'
And then
import styled from 'styled-components'
for the other files calling emotion styled.
And it works perfect using styled components with no conflict.

Nice! We're planning to implement this into gatsbyjs.org, which uses Emotion, but this is really helpful for any future Googlers. 😄

If you all are building for Googlers then you might think about, "in code structured micro-data", = no sudo json LD.
Comments, posts, adds, reviews, products ect... need to have a structure for search engines.. Unless the  Googlers could find our site and page's what good is Googling for not indexed content, we need to have some kind of usable structured microdata for Google search and other spiders, to add any real website site and others so Google search spiders and other spiders can index them.
Now this is not true for Gatsbyjs as you all have 0 microdata in any of your sites and that's ok because of all the people searching for Gatsby stuff. So you are positioned by search hits, but the regular user with a web site need's to have all the help he can get to have his content found so the Googlers will even know his site and pages are there.
I will add this to Gatsby's repo comments to see what the community thinks.
Thanks
Donald Boulton


Complete CLI steps using Azure DevOps extension

With the new Azure DevOps extension to Azure CLI, the document to create repository using CLI should be self sufficient without any portal interaction and the above document reflects that.

@geverghe : Thanks for your contribution! The author, @steved0x, has been notified to review your proposed change.

@steved0x - I still don't see the update in the documentation - is there something else that needs to be done?


Relation between .NET Standard and ECMA 335 Partition IV

ECMA 335 continues to establish uniformity for .NET implementation behavior, but there's no similar spec for the .NET Base Class Libraries (BCL) for .NET library implementations.

^ This sentence confuses me to no end.
BCL is one of the 9 CLI standard libraries specified in ECMA 335 Partition IV, so why does it say that there's no similar spec for it? If there's already a spec for CLI libraries (not just the BCL), then what is the purpose of .NET Standard?
I'm not suggesting that .NET Standard is pointless. I see that it covers a lot more APIs than ECMA 335, and takes a very different approach to assemblies (i.e. the clever use of netstandard.dll as explained here). But I'd really appreciate some clarifications on .NET Standard's relation to ECMA 335 Partition IV, especially since this page already mentions ECMA 335.
Is .NET Standard trying to side step ECMA 335 Partition IV to be a more extensive and flexible spec?
Should ECMA 335 Partition IV be considered a legacy spec with diminishing relevance?

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

ID: 9eb8b774-c6f0-8235-6964-15219baf2ea7
Version Independent ID: 6819a4a7-7527-d8c7-e132-767f7344e00e
Content: .NET Standard
Content Source: docs/standard/net-standard.md
Product: dotnet
Technology: dotnet-standard
GitHub Login: @mairaw
Microsoft Alias: mairaw

ping @CarolEidt.  Can you address this question?

@cfbao - can you tell me where the sentence came from that you quote above?

BCL is one of the 9 CLI standard libraries specified in ECMA 335 Partition IV, so why does it say that there's no similar spec for it?

The BCL specified in ECMA 335 Partition IV is a minimal set, while .NET Standard covers a broader set of APIs. It would be great to see the set of APIs specified in the  ECMA standard broadened, and I would hope to see that happen in future.

Is .NET Standard trying to side step ECMA 335 Partition IV to be a more extensive and flexible spec?

I would say that "side step" is a bit strong, but certainly .NET Standard is taking a more agile approach to specifying and promoting a broadly usable API set.

@CarolEidt Thanks for the reply.
The quoted sentence is the last sentence of the first paragraph on this page, or this document (as linked in "Content Source" in my original post).
I realize that .NET Standard is much broader than (not only BCL but) all of Standard Libraries in ECMA 335. I think it'd be more appropriate to say that there's no spec for the broader (but still widely used) .NET libraries/APIs, instead of "BCL" (or even Standard Libraries). The latter phrasing isn't technically correct, hence my confusion.
I said "side step" because ECMA 335 Partition IV specifies that most classes/APIs (e.g. System.Object) should be in mscorlib.dll, whereas if an app/library is to target .NET Standard, it should reference netstandard.dll for most classes (including System.Object).
A .NET implementation can support/conform to both standards by means of type forwarding, but an app can only choose one. Whether this is "side stepping" or not, I don't really mind either way.
The main point I'd like to make, again, is the distinction of

.NET Standard being the first ever spec for .NET libraries (as implied in that quoted sentence, incorrectly, I think)
vs.
.NET Standard being a new spec that encompasses a broader range of .NET APIs, compared to ECMA 335

It does seem that the statement in this article about there being "no spec for the BCL" is erroneous since the ECMA page does have a zip download of the XML ref that defines the standard. It's similar to these specs for .NET Standard: https://github.com/dotnet/standard/tree/master/docs/versions

I agree with the comments - I think the page should be edited to reflect something along the lines of @cfbao's second bullet above, e.g. replace:

but there's no similar spec for the .NET Base Class Libraries (BCL) for .NET library implementations

with something like:

and while ECMA 335 specifies a small set of standard libraries, .NET Standard encompasses a broader range of .NET APIs.

Does that seem reasonable?

That looks good to me 👍

@BillWagner - who would be the right person to ping to make this kind of change?

@CarolEidt I'll take care of it.  Thanks for helping us resolve the language.


Start rebranding component

The name of the component can confuse some people. Indeed, they stop reading at its name. "Oh I don’t have any Tahoma hub, that’s not for me."
Like already discussed  within a Core PR, I suggest Overkiz (by Somfy) and only rebrand the README.md for now, or perhaps do you want already change the name displayed in HACS (not the integration name).

Is there any way to have aliases so that a google search of Home Assistant Rexel would lead straight to here? I don't think that Overkiz is very well known, although it's the common ground of all those devices...

What’s Rexel, that’s also a supported hub? I agree Overkiz is not well known. I don’t know what could be the best for our visibility.

What’s Rexel, that’s also a supported hub?

I took the least obvious name in our Readme list of supported devices 😝

Damned, I didn’t scrool enough to see it. We can add the name of the hub as tag to our repo. It should help I guess.

What’s Rexel, that’s also a supported hub? I agree Overkiz is not well known. I don’t know what could be the best for our visibility.

And there are even more... See https://github.com/iMicknl/ha-tahoma/pull/418/files. I would love to rebrand this whole integration to ha-overkiz as a breaking change and also implement changes like that one.

I'm okay with it then, provided we use Overkiz by Somfy as a title.


Adjusting label layout

Hi,
I'm trying to modify your code to work with Avery L7157 labels.
These are A4 sheets with 11x3 (33) labels 64.0mm x 24.3mm
I'm stuck trying to work out where you set the actual label height. It's always 1 inch.
Thanks for helping.
Pete

I don't think it's too easy to modify that script because I didn't really design it with adjustability in mind.
The '1 inch height' is in line 21.
But changing that will most likely break everything else alignment-related.
What you would have to change is:

The StickerRect __init__ function (lines 18-22). This one defines the boundaries of one sticker.
The draw_resistor_sticker function (lines 307-361). This one defines the layout of one sticker, e.g. where to draw which text and which image on the sticker.
The render_stickers call in main (lines 390-401), as the L7157 probably doesn't have the same number of stickers as the original 5260

Have fun and feel free to ask further questions :)

Yes, line 21 does really mess things up lol.I'll have a go at lines 307-361 tomorrow. I think this is where the changes will need to be made, now you have explained why line 21 messes everything up.Interestingly, adding an extra row of resistor values actually works without changing any of your code.Thanks for your quick reply. I'll let you know how I go.On 18 Jun 2021 20:48, Martin Stumpf @.***> wrote:
I don't think it's too easy to modify that script because I didn't really design it with adjustability in mind.
The '1 inch height' is in line 21.
But changing that will most likely break everything else alignment-related.
What you would have to change is:
The StickerRect init function (lines 18-22). This one defines the boundaries of one sticker.The draw_resistor_sticker function (lines 307-361). This one defines the layout of one sticker, e.g. where to draw which text and which image on the sticker.The render_stickers call in main (lines 390-401), as the L7157 probably doesn't have the same number of stickers as the original 5260
Have fun and feel free to ask further questions :)
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.

After a little research, it seems that the L7157 has almost the same sticker size as the 5260, so it didn't actually require any changes in the draw_resistor_sticker function.
I added a config value to the main() function with which you can switch between the two papers.
Let me know if the numbers I entered for L7157 work, as I don't have L7157 paper here to test it.

It works perfectly :-)
I use Ubuntu - the name of the font "Arial Bold" is Arial_Bold.ttf
That's all I had to change, maybe add a note in the README?
Thank you for the speedy update.

Fixed in https://github.com/Finomnis/ResistorLabels/commit/9e892e8488c2c5950fb4753f7c2f5a8bbc04ade5

Is it ok if I close this issue? Unless you have further questions, of course.

Please close it. You have done excellent work
Thanks


Create tag for release on github

Debian project will appreciate if you tag the release on github.
Thank you

Done in new major version.


[Feature]: Lower number of ahorn/loenn(?) plugins

Request Description
Hi so basically I was wondering if it would be possible to change the Portal (Direction, Color) format to just Portal (Direction) because now that there are 16 different portal colors across 5 different directions, there are 80 plugins for almost identical entities, which I feel is a little bit unnecessary and clutters an already very overfilled interface. My idea is to simply make one plugin for each direction, since you can already change the color in the entity settings.

Target Date
No response

Fixed in 1.7.2, thanks for reporting!


Add digest_md5

Closes #71
This also updates the other digest function to use a 32 KiB buffer instead of the 4 KiB one. This should improve hashing speed. This specific number is taken from Go stdlib, where io.Copy uses a 32 KiB buffer if needed. The perfect buffer size for a Linux system varies, but this should be good enough, and better than before.

@anaulin all your comments should now be addressed. Will wait for another review like from @benhylau before merging.


[Sprint 52] 1.2.x only XD-3125, XD-3126 Refactor Codec configuration.

This is a redo of https://github.com/spring-projects/spring-xd/pull/1717 addressing @markpollack review comments.

Do we want this also for the 1.2.x branch?

Only for 1.2 branch

but also on master?
Take a look at https://github.com/markpollack/spring-xd/tree/dturanski-XD-3125b and the rework in AbstractKryoMultiTypeCodec to see if you are ok with it.

I thought master was for 2.0. All this will move to spring-bus
On Thu, Jul 2, 2015 at 11:59 AM Mark Pollack<EMAIL_ADDRESS>wrote:

but also on master?
Take a look at
https://github.com/markpollack/spring-xd/tree/dturanski-XD-3125b and the
rework in AbstractKryoMultiTypeCodec to see if you are ok with it.
—
Reply to this email directly or view it on GitHub
https://github.com/spring-projects/spring-xd/pull/1726#issuecomment-118077358
.

merged on 1.2.x as bc15d3861732a618d61c8e96508469e9afcd47ae


Throw correct exception for failed GUID conversion

Converter interface indicates a ConversionException for such an error

This should fix some crawler errors that we've been seeing on TeamCity


Is it possible to treat a primary key as an Int64?

Given a query that will run in an SqlBackend environment, is it possible to cast, say product ^. ProductId to an SqlExpr of Value Int64? My initial approach to fmap over both Value a and SqlExpr but had to find out it could not deduce a Functor SqlExpr.
For example, this does not work:
-- ...
                [ E.when_ (p E.^. ProductKind E.==. (E.val $ BookProductType))
                    E.then_ $ do
                        ((fromSqlKey <$>) <$>) <$> do
                            mBook E.?. BookProductId
                ]
-- ...

as it causes:
-- causes:
    • Could not deduce (Functor SqlExpr) arising from a use of ‘<$>’
      from the context: (BaseBackend backend ~ SqlBackend,
                         PersistUniqueRead backend,
                         PersistQueryRead backend,
                         IsPersistBackend backend,
                         Database.Esqueleto.Internal.Sql.SqlSelect a r,
                         MonadIO m)
        bound by the inferred type of
                 q :: (BaseBackend backend ~ SqlBackend, PersistUniqueRead backend,
                       PersistQueryRead backend, IsPersistBackend backend,
                       Database.Esqueleto.Internal.Sql.SqlSelect a r, MonadIO m) =>
                      ((SqlExpr (Entity Product), SqlExpr (E.Value (Maybe Int64)))
                       -> SqlQuery a)
                      -> ReaderT backend m [r]
    • In a stmt of a 'do' block:
        ((fromSqlKey <$>) <$>) <$> do { mBook ?. BookProductId }
      In the second argument of ‘($)’, namely
        ‘do { ((fromSqlKey <$>) <$>) <$> do { mBook ?. BookProductId } }’
      In the expression:
        when_ (p ^. ProductKind E.==. (val $ BookProductType)) then_
        $ do { ((fromSqlKey <$>) <$>) <$> do { mBook ?. BookProductId } }

The use case here is to be able to return a list of primary keys which may come from different tables. My models look similar to:
Product
    sku Text
    kind ProductType
    UniqueSku sku
    deriving Show Typeable

BookProduct
    productId ProductId
    isbn Text Maybe
    UniqueProductId productId
    deriving Show Typeable

Is this possible at all or am I doing things completely wrong?

-- Must use this qualified lookup, else will get `Couldn't match type ‘BaseBackend backend’ with ‘SqlBackend'`
import qualified Database.Esqueleto.Internal.Sql as EIS

-- ...

    E.case_
        [ E.when_ (p E.^. ProductKind E.==. (E.val $ BookProductType))
            E.then_ $ EIS.veryUnsafeCoerceSqlExprValue $ mBook E.?. BookProductId
        ]
        (E.else_ E.nothing)

-- ...


Planner

Pull request for matchmaker and planner

Some considerations global considerations (I cannot comment in some files):

Why are you using a Makefile? (planner/core/Makefile)
Why do you have a outdated code folder? (planner/core/old/)

Right comments. Thank you. I will close this pull request and make an updated version that takes into account your suggestions.

@szenzaro This is not necessary, if you push new commits to the branch where the PR is done, the PR is updated as well. Could you close the new one and reopen this PR, so that the conversation is not lost?

ok, I did not know that

Travis CI cannot build because there are missing license headers. If some file doesn't need it just mark it as excluded in the pom.xml
You can check the file rat.txt in the target folder after performing a mvn clean install which contains the status of the license verification for all the files
[ERROR] Failed to execute goal org.apache.rat:apache-rat-plugin:0.11:check (default) on project platform: Too many files with unapproved license: 1 See RAT report in: /home/travis/build/SeaCloudsEU/SeaCloudsPlatform/target/rat.txt -> [Help 1]

It seems that there are conflicts with the current master. This is probably caused because the planner branch got out-dated from the recent master changes. You probably need to rebase your branch on top of the master to solve this: (https://github.com/edx/edx-platform/wiki/How-to-Rebase-a-Pull-Request)

Ciao Simone, the usage of jdk8 is impossible at this stage as the continuous integration chain is configured for jdk7 and all other components use jdk7. Basically, if you keep the jdk8 constraint, your component, even in the case it gets in the main repository, cannot be built and packaged with the automatic mechanisms. I'm not sure if this blocks also the packaging of the other components or not, but it certainly creates issues to the seamless creation and update of the whole platform that includes your component. Do you have in mind any solution to this problem?


Fix the sugared logger usage

Without w the logged messages are quite useless and annoying.
/lint
/cc @evankanderson

/hold cancel


Separate suffixes for date/month/year in short date + make delimiters optional

What?
These changes allow

users to set suffixes for the short date format and
turn off delimiters for short date format

Suffixes, different from delimiters, appear immediately after the corresponding numeric value (whether that's date, month, or year), and can be separately set to different strings.
Fixes #189
Why?
This is a requested enhancement feature and a currently open issue: #189. Plus, it better enhances the user experience for non-English speakers.
How?

RE: turning delimiters optional. In the original usage example, there are no delimiters, thus they are turned into optional features that can be toggled on/off. There is now a enableDelimiter setting inside settings.ts to keep track of this. The html page for date also checks whether delimiters are enabled before displaying them.
Suffixes are implemented in the same way as delimiters, though 1 enableSuffix setting controls the visibility of all 3 suffixes (dateSuffix, monthSuffix, and yearSuffix). It has the same logic of display, dimming, and change in setting as delimiters code-wise.

Testing?

Successful local dev build
Manual testing through clicking and toggling

Screenshots
As shown, users can set their own suffixes for each of year, month, and date. These suffixes can be dimmed as well. The screenshot displays the usage example outlined in #189.

Delimiters can coexist with suffixes, so that users can use them to better separate the numbers with - or / , or just set it as a space.

Anything Else?

The current character cap on suffixes is 3. More research may be needed to decide on a better cap to encompass all language use cases.
Translation is needed for the numerous blurbs that appears in the "date" panel:

suffix
dimDateDlelimiter
dimDateSuffix
enableDelimiter
enableSuffix

Nice catch thank you so much! My latest commit should fix that -- it checks if the date/month/year is enabled as a part of the ngIf


equality modulo unique id's

for many cases and test cases: we need equality over bexpr modulo unique id's, i.e. neglecting difference in unique id numbers.

Let me try to get the requirements straight. This is about equality of behavior expressions (BExpr) only, or are there any other structures we want to compare modulo unique id? @CarstenRuetz: are there any other places where unique identifiers will be added, or only at Ident?
In the first case I would just define a function
(~~) :: BExpr -> BExpr -> Bool
bexp0 ~~ bexp1 = zero (bexp0) == zero (bexp1)

where zero takes a behavior expression and set all the unique identifiers to zero. My guess is that due to lazy evaluation only one traversal will be needed. I can benchmark this against the standard equality between BExpr (==), if only one traversal is done we should see only one constant increment in the execution time.
If there is a performance penalty the question is whether we want to sacrifice a simple design in favor of more performance. This depends on how often such a ~~ operator would be needed.

@CarstenRuetz for a more dynamic discussion it'd be nice if you could join our gitter channel.

@capitanbatata To keep it in one place, I'll keep it here for now
Requirement is to determine the equality of two ProcDefs modulo unids.
There is indeed the BExpr, but also (again) ChanId and VarId at the ProcDefs level.
Performance-wise I don't mind: there will be a couple of tests but it's not executed on the fly during test generation.
@tretmans Do you need it for s.th. else than tests of TorXakis itself? Does performance play a role?

Ok, so if @tretmans does not have any objections I can implement in the way I described above.


Reproducible threading

Make threaded MC reproducible.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

44 of 46 (95.65%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 90.004%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/strategies_and_params/threadingstrategy.jl
26
28
92.86%

Files with Coverage Reduction
New Missed Lines
%

src/DictVectors/abstractdvec.jl
1
94.78%

src/strategies_and_params/threadingstrategy.jl
2
92.92%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
4475

Relevant Lines:
4972

💛  - Coveralls

I'm closing this as this version of threading is slow and inefficient. The work will be superseded by #183.


test: Create cross engine integration tests

Currently all integration tests validate an engine with itself. This is good for proving things like data type and function coverage but doesn't ensure end comparisons are good.
There are many engine combinations which would generate too many integrations tests. Based on the assumption that BigQuery is our most mature engine support then we propose to add cross engine comparisons for all other engines to BigQuery. If they all match BigQuery then they should all match each other.
We'll use our CORE_DVT_TYPES test table for these comparisons. There will be a requirement for some other cross engine tests, for example for engine specific data types that are not covered by our CORE_DVT_TYPES test table. These can come later via different issues.

Reminder list.

BigQuery BIGNUMERIC is coming back as Decimal(38,9) which is incorrect. Need to look into this.

Reminder list for test_oracle.py:

BigQuery BIGNUMERIC is coming back as Decimal(38,9) which is incorrect. Need to create issue and edit test_oracle.py.
Oracle tstz column is losing time zone and failing column and row comparisons. Needs a new issue or may be fixed by https://github.com/GoogleCloudPlatform/professional-services-data-validator/issues/706. Edit test_oracle.py as appropriate.
Oracle binary_float and binary_double are being converted to string in scientific notation which does not match other engines. Need to create issue and edit test_oracle.py.

Reminder list for test_hive.py:

Same BIGNUMERIC issue as Oracle. After create issue edit test_hive.py.
Column validation failing for col_char, for some reason we're losing the trailing space. Need to create issue and edit test_hive.py.
Hive float and double are being converted to string in scientific notation which does not match other engines. After create Oracle/Hive issue edit test_hive.py.
test_sql_server.py

Reminder list for test_sql_server.py:

SQL Server tstz has time zone but BigQuery doesn't. Should be fixed by https://github.com/GoogleCloudPlatform/professional-services-data-validator/issues/706. Update 706 to request update of test_sql_server.py.
column validation --sum on text column fails with FAILED tests/system/data_sources/test_sql_server.py::test_column_validation_core_types - sqlalchemy.exc.ProgrammingError: (pyodbc.ProgrammingError) ('42000', '[42000] [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]Argument data type text is invalid for argument 1 of len function..... Need to create issue and edit test_sql_server.py.
column validation failing for decimal columns, they are all being cast to float64 and we're getting a mismatch in result. Need to Need to create issue and edit test_sql_server.py. May be related to https://github.com/GoogleCloudPlatform/professional-services-data-validator/issues/763.
SQL Server float and double are being converted to string in scientific notation which does not match BigQuery. After create Oracle/Hive issue edit test_sql_server.py.

Reminder list for test_teradata.py:

Teradata NUMBER is being read into DVT as float64, this is not correct. Created issue 838

Reminder list for test_postgres.py:

PostgreSQL tstz has time zone but BigQuery doesn't. Should be fixed by issue [706].


ENYO-5418: ExpandableItem handle PageUp and PageDown

Issue Resolved / Feature Added
Added support for page up and page down keypresses for ExpandableItem.
When autoClose is true page up will close ExpandableItem and page down will spot the last item of the list (if lockBottom is true).
Links
ENYO-5418
Comments
Enact-DCO-1.0-Signed-off-by: Teck Liew<EMAIL_ADDRESS>

Current behavior was acceptable by UX team.  Closing.


Add readmes for UNet/ICNet trained on CamVid

@yssemaev I'd like to remove fp32 from model name. Name icnet-fp32-onnx-0001 should be icnet-onnx-0001 because it can be as fp32 and as fp16 IR in model package

BTW, if "icnet-fp32-onnx-0001" is the original model, why not naming it like others public models, for instance, ssd512, ssd300?

@yssemaev this model distributed in IR form, so naming convention follow to other such models

@vshampor please note, there is merge conflict

Jenkins please retry a build

Jenkins please retry a build


Populating your repository with open source content from external systems - shar...

Populating your repository with open source content from external systems - share experience or aspirations with methods, APIs, local features
Suggested by: Linda Newman
Presenter: Linda Newman + hopefully others with stories to share
Format: Panel followed by discussion
Audience: All

More comment from Linda: I can speak as a project manager / dev team leader about my research trying to sort out the landscape with various platforms and APIs (some commercial / some open source) that offer potential for automatically discovering and perhaps ingesting content. This may relate well to the suggestion from Rick Johnson and Don Brower titled "Fedora Research Object Model and Hydra APIs." This could be a breakout session or panel (or even lightning talk) - I'm interested in hearing what experiences others have to share if they have tested using things like the Elsevier APIs (Scopus and Science Direct), the JISC Router, FigShare, Symplectic Elements, and/or the Open Science Framework, to even partially automate the discovery and ingest of open source content into your local hydra repository. I'm also interested if others have found ways to work with platforms such as Academia.edu and ResearchGate.

Breakout, if all else fail, lightning

PANEL & DESCRIPTION NEEDED

A review of available APIs and services for identifying, and either manually or automatically loading, open source online content into a local IR.  Grey areas, policy questions, challenges and opportunities.
I do not yet have any co-presenters, but I will put a call out on the Hydra community elist – unless you advise against this.
I am willing to be recorded.


feat: GitHub Workflow automation & updated deps

This runs golangci-lint and go tests on each commit.
If you create a release object and tag it, the release.yml workflow
will build and push container images to ghcr.io and also builds and pushes a Linux/amd64 binary tagged to the release.
I could build and push binaries for other architectures as well if you'd like.
Oh and this workflow and ko make the Dockerfile unnecessary.

@shayne I might have to revert the Go toolchain, tailscale.com, and gvisor package version updates. It's getting quite a few mysterious compiler errors.

Feel free.

Fantastic, thanks!


Selenium Grid support (webdriver-manager)

Currently, there is no way to start a Selenium Grid session using webdriver-manager. I believe it would be useful if start accepted a "--role" argument, which is how one specifies whether or not the machine is a Hub or a Node in a Selenium Grid network.
Here's how it works manually:
$ java -jar selenium-server-standalone-2.47.1.jar -role hub

Conceivably, webdriver-manager could accept something similar:
$ webdriver-manager start --role=hub

Our general thought has been that webdriver-manager is a quick setup for folks who want to get selenium/webdriver started quickly. We don't want to replicate every advanced option. If you're making a grid, the assumption is that you know what you're doing and don't need webdriver-manager to help. Is there a particular reason that this option would be helpful to include?

I'm aggressively closing issues that we are unable to reproduce, are support issues specific to one test, or deal with integrations with other projects that we don't directly support. If you feel you have a question, please see https://github.com/angular/protractor/blob/master/CONTRIBUTING.md#questions. If you still have a bug report or feature request, please open a new issue. Thank you!

Hi @juliemr! As I understood we couldn't use webdriver-manager as (selenium grid). Only one test. Am I right?


Like to add Famous and important Knapsack problem in Dp section

Hello @AryanBarsaiyan, I was reviewing your dp section and I found 0-1 Knapsack missing and you will also be knowing that how important and helpful that algorithm is. So I would like to add self explanatory code for that, may you assign me this.
And My query is I can directly send the cpp file or I have to make a different html file. Thanks

send java file or make a different HTML file


Unable to set custom option

I'm using the latest version (1.3.3) of this gradle plugin. But, if I try to set the cveValidForHours as documented here, I get the following error:
Could not find property 'CVE_CHECK_VALID_FOR_HOURS' on task ':dependencyCheck'.

The reason I am trying to set this value to something greater than the default is since we are trying to reduce our build times, and this seems like a low hanging fruit with high ROI in our usecase.

Sorry about this. I was able to replicate the behavior using the currently published plugin. Apparently I have already fixed this bug in the code base as the current snapshot works without any changes. I will push a release within the next day or two.

@jeremylong Using the latest version (currently 1.3.6) I am experiencing the very same issue again.
The [Update.groovy] (https://github.com/jeremylong/dependency-check-gradle/blob/master/src/main/groovy/org/owasp/dependencycheck/gradle/tasks/Update.groovy#L106) file is using CVE_CHECK_VALID_FOR_HOURS but lacks the static import from org.owasp.dependencycheck.utils.Settings.
Would you mind having a look at this issue?

Thanks for pointing that out. I'll update my release testing to ensure this get tested correctly prior to the next release (which should be soon).


Normalized fluxes or absolute flux calibration

Be sure data headers indicate the correct units or, in some cases, say if the spectrum is normalized (as many SpeX spectra on SPLAT) together with a label about if flux is in F_lambda or F_nu units.

Yes, this is generally part of the process of ingesting and converting spectra.


When IPv4 and IPv6 is configured, the IPv4 listener is not working

When IPv4 and IPv6 is configured, the IPv4 listener is not working:
TCP    <IP_ADDRESS>:9443           <IP_ADDRESS>:0              LISTENING - not OK
TCP    [::]:9443              [::]:0                 LISTENING - OK
http://localhost:9443/metrics - OK
http://172.x.y.z:9443/metrics - not OK
http://[Hostname]:9443/metrics - OK but via IPv6
How to make http://172.x.y.z:9443/metrics work?

Note related to WMI Exporter


WIP: Build and run tests with webpack

Adds the option to build and run tests with webpack using the command npm run test:webpack. (Similar to npm run test:manual in that it spins up a local server)

@humphd Is this the kind of thing you were talking about?

Codecov Report

Merging #770 (05ea9e7) into master (c0a41d6) will increase coverage by 0.16%.
The diff coverage is 96.66%.

@@            Coverage Diff             @@
##           master     #770      +/-   ##
==========================================
+ Coverage   86.89%   87.06%   +0.16%
==========================================
  Files          16       20       +4
  Lines        1740     1770      +30
==========================================
+ Hits         1512     1541      +29
- Misses        228      229       +1

Impacted Files
Coverage Δ

shims/fs.js
95.83% <95.83%> (ø)

shims/buffer.js
100.00% <100.00%> (ø)

shims/path.js
100.00% <100.00%> (ø)

shims/providers/default.js
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Ping me when you want reviews on these.

No problem 👍

Ping me when you want reviews on these.

I will come back to these after #771 is merged as I think it makes sense to use the webpack plugin to build these tests. Also, if we move the plugin to it's own repo (as discussed in #771) would you want to move these tests to that repo or leave them here?

I think doing it in this repo is probably the best idea, since it has CI setup already, and is so closely related to the other code.  I'm more wondering about reducing the bundle size for Filer for cases where people aren't using the webpack plugin.  We could write a separate set of build/test scripts to be run with npm-run-all or something that builds it beside vs. as part of Filer.


HTML Files Not Displaying in Canvas Mode

Firstly, thank you for the fantastic add-on! It works great in most areas of Obsidian, but I noticed that HTML files do not display in Canvas notes.
Is this a known limitation or something that should work in the latest Obsidian updates?
If it's not currently supported, I would like to request it as a new feature.
Additionally, in flow view, the window displaying HTML is currently wide and short. It would be useful if this could be resizable or presented in a more standard size for easier viewing.
Thanks again for your hard work and this great tool!

Firstly, thank you for the fantastic add-on! It works great in most areas of Obsidian, but I noticed that HTML files do not display in Canvas notes.
Is this a known limitation or something that should work in the latest Obsidian updates?
If it's not currently supported, I would like to request it as a new feature.

It seems that the Canvas Mode only support the "note" file types of official Obsidian app, other file types supported by 3rd party plugins are not treated as note files, so they cannot display in Canvas notes (e.g. HTML, epub, csv, etc.).

Additionally, in flow view, the window displaying HTML is currently wide and short. It would be useful if this could be resizable or presented in a more standard size for easier viewing.

Hmm, I am not familiar with flow view, could you please take a snapshot and re-explain with it for your issue?


How to specify advanced-instructions when placing orders

I would like to specify advanced-instructions, particularly strict-position-effect-validation: true when placing orders. However, I could not find anywhere in the documentation or the source code how I can include this when calling NewOrder.

I'm afraid you may be on your own on that one! You can try experimenting with the fields to see what happens, but I have no idea how it works to be honest. I'll leave this issue open in case anyone else has already done this and has some insights.

I was about to ask about this as well. I tested with dry-run True to avoid opening a position and the PlaceOrder Response under warnings say the following:
code='closing_with_no_position', message='You cannot route a closing order without an existing position to close. This order will be updated to a buy to open order when routed.', preflight_id=None)
I'm thinking of embedding the advanced-instructions into the JSON attributes of the order object somehow. I'm gonna play around with it a bit to see if it works

** Temporary FIX **
This may not be ideal but does the trick, please test first and make sure you do so with dry_run=True
Go to the order.py file and find the NewOrder class around the 214~224 line.  At the end of that class, right after the rules variable paste this line advanced_instructions: str = {"strict-position-effect-validation":True}
Let me know what message you receive. On my end it was successful

Interesting, could you submit a PR adding that as an optional field?

Will do !

Hey Graeme,
I don't really know how to open that Pull Request. If you can give me some
guidance on it that would be great.
On a separate note, do you know how to implement the heartbeat feature to
keep the session alive?
Thx

Andres Usma
On Fri, Dec 6, 2024 at 7:41 PM Graeme Holliday @.***>
wrote:

** Temporary FIX **
This may not be ideal but does the trick, please test first and make sure
you do so with dry_run=True
Go to the order.py file and find the NewOrder class around the 214~224
line. At the end of that class, right after the rules variable paste this
line advanced_instructions: str =
{"strict-position-effect-validation":True}
Let me know what message you receive. On my end it was successful
Interesting, could you submit a PR adding that as an optional field?
—
Reply to this email directly, view it on GitHub
https://github.com/tastyware/tastytrade/issues/178#issuecomment-2524709246,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANFECFPXYICEQVGT5PCCVED2EI75LAVCNFSM6AAAAABS6O7G7SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRUG4YDSMRUGY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Btw, could someone explain what strict position effect validation does/means? Not immediately clear to me

If you put a closing order but for whatever reason the position you want to close is no longer opened then the API will turn that closing order into an order to open.
By adding that variable the API will not simply discard the closing order.


Add a blurry loading page

@arpit456jain  you asked me to raise a PR in this repo instead of "Amazing JS" Repo

will you please allow me to work on this issue


Move the documentation over to Read The Docs

Describe the Task
Move the documentation over to Read The Docs.
Time Estimate
1 to 3 days of work
Sub-Issues
Consider breaking the task down into sub-issues.

[ ] Add a checkbox for each sub-issue here.

Relevant Deadlines
ASAP
Funding Source
Define the Metadata
Assignee

[x] Select engineer(s) or no engineer required
[ ] Select scientist(s) or no scientist required

Labels

[x] Select component(s)
[x] Select priority
[x] Select requestor(s)

Projects and Milestone

[x] Review projects and select relevant Repository and Organization ones or add "alert:NEED PROJECT ASSIGNMENT" label
[x] Select milestone to next major version milestone or "Future Versions"

Define Related Issue(s)
Consider the impact to the other METplus components.

[x] METplus, MET, METdatadb, METviewer, METexpress, METcalcpy, METplotpy

Task Checklist
See the METplus Workflow for details.

[ ] Complete the issue definition above, including the Time Estimate and Funding Source.
[ ] Fork this repository or create a branch of develop.
Branch name: feature_<Issue Number>_<Description>
[ ] Complete the development and test your changes.
[ ] Add/update log messages for easier debugging.
[ ] Add/update unit tests.
[ ] Add/update documentation.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into develop.
Pull request: feature <Issue Number> <Description>
[ ] Define the pull request metadata, as permissions allow.
Select: Reviewer(s), Project(s), Milestone, and Linked issues
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

This work is complete.  Closing.


fix: infinite daprun callback loop for non-background tasks with prob…

Using preLaunchTask with a non-background task that has a problem matcher, e.g.
{
    "label": "Build",
    "type": "shell",
    "command": "make",
    "problemMatcher": {
        "base": "$gcc",
    },
}

causes infinite loop of daprun callbacks because it triggers both on_result and on_complete. The fix is to run only the first callback that is fired and it works for my case, but I'm not sure if this is always correct. Is on_result guaranteed to run only after the task complets?

Yeah, on_result will only be called either during a background task or after the task is completed (but before on_complete). This should work, thanks!


Planning module not working

System information

OS Platform and Distribution (e.g., Linux Ubuntu 20.04):
Apollo installed from (source or binary):
Apollo version (6.0):

Steps to reproduce the issue:

start Apollo Dreamview..Then select relevant modules...Set Routing....:
** Planning module does not activate**

SIMILAR TO ISSUE #13300

could you manually start planning module and post the error message here

could you manually start planning module and post the error message here

i changed my apollo version 5.5..Its working fine now

i changed my apollo version 5.5..Its working fine now

@ntutangyun Thank you for your quick response....In future...if I get any issues...will surely bring to notice

@ntutangyun Thank you for your quick response....In future...if I get any issues...will surely bring to notice


when selecting entire eth balance, deduct expected tx cost from the input amount

swap.bancor.network
from token selection - ETH
click the eth balance
(this is only true for eth)
expected - input field will be populated with the (entire balance - expected tx gas cost)

What do you want to base gas price off?

I'm not sure how to deliver this given it will break should the user mutate the gas price unless we over estimate what the user would go for gas price wise and leave the remaining as dust.
Otherwise we update the UI to prompt for the gasPrice and gasAmount for the TX and then pass that data into the wallet, so it's already inline with what the user wants. @ashachaf

can you estimate the transaction cost and deduct this amount from the eth balance when the user clicks "max" (or his balance)?
possible process:
go to swap.bancor.network
click to enter your full balance (only when. you click the "max" or balance,  this process works)
calculate transaction gas estimation
deduct this amount from the balance
input the calculated value into the input field.

With this we'd have to guess what the user wants to set as gas price and any extra gas unused would be left in their account by doing. Could get a bit messy.
Unless we bring up a special modal which offers a decent gas price, decent gas estimate and let the user approve of this before triggering the wallet transaction?

How is this handled by other platforms in the space?

Looks like Uniswap doesn't let the user spend their last .01 ETH

Looks like Uniswap doesn't let the user spend their last .01 ETH

we can have the same setup than.
when clicking on the ETH balance in the swap window, deduct 0.01eth from the amount you input the field.

we can have the same setup than.
when clicking on the ETH balance in the swap window, deduct 0.01eth from the amount you input the field.

Fixed in https://github.com/bancorprotocol/webapp/commit/5cfae1aaad16d1a71af9dea66f4193f4e857e9f4

Fixed in https://github.com/bancorprotocol/webapp/commit/5cfae1aaad16d1a71af9dea66f4193f4e857e9f4


error in prettyCoOncoplot

Error in (function (maf_df, onco_matrix_path, genes, include_noncoding = NULL,  :
formal argument "maf_df" matched by multiple actual arguments

Oups, I was actually providing maf argument twice!