Upgrade or Re-Install of Chocolatey Fails from Puppet

I'm new here so sorry ahead of time if my posting is not conforming to customs... I am replying inline to Rob :)

I get errors when updating Chocolatey via Puppet. I've tried using the puppet-chocolatey provider to upgrade to v<IP_ADDRESS>, and re-installing chocolatey from Puppet with command line or powershell (v<IP_ADDRESS>).

Rob: So do none of those cases work or just the case with Puppet?

Chocolatey update: from v<IP_ADDRESS> (ChocolateyInstall=C:/ProgramData/chocolatey) to v<IP_ADDRESS>
My 1st try was to upgrade from old version to new using puppet-chocolatey provider, then from command line using "choco -f update chocolatey -version '<IP_ADDRESS>' -source '<our private repo>' " and both methods failed. Of course the command line will be different now with the 'upgrade' command instead. (note: the command works if done manually, but not from puppet)
2nd I tried to just re-install over previous chocolatey installation by executing a batch file from puppet with the standard install command from command line...
@powershell -NoProfile -ExecutionPolicy unrestricted -Command "iex ((new-object net.webclient).DownloadString('<our private repo>\chocoInstall.ps1'))" && SET PATH=%PATH%;%ALLUSERSPROFILE%\chocolatey\bin
3rd try executes the chocolatey Install.ps1 directly with the puppet powershell provider...

My suspicion has been that the ChocolateyInstall variable is null, based on the install log ([DEBUG] Running 'Install-ChocolateyEnvironmentVariable' with variableName:'ChocolateyInstall' and variableValue:''). Possibly because puppet is running as the System user (the variable is set globally).

Rob: That is likely removing a value - unfortunately the message probably also needs to state the scope of the change - user versus system. We attempt to remove the user version if we are setting it globally, which for administrative install we are going for the global variable.

Makes sense. Does running as SYSTEM vs. an Administrator (which is technically a user running as admin -like sudo) get similar environment variables or different... I suspect a different environment, but I haven't looked into this yet.
The error in Puppets run messages:
message: "Chocolatey is unpacking required files for use. Overwriting? True"
message: "The OS handle's position is not what FileStream expected. Do not use a handle simultaneously in one FileStream and in Win32 code or another FileStream. This may cause data loss."

Rob: This seems a puppet-chocolatey issue, specifically https://github.com/chocolatey/puppet-chocolatey/issues/18.

Yes. I was looking at this one... possibly related, but effects more recent version of chocolatey.

Rob: This means something went wrong with the package install/upgrade, in this case the upgrade of Chocolatey itself.
It seems you are likely on a version less than 0.9.9 and upgrading to the latest.

Rob: Some questions to help us get a better understanding:
What version of chocolatey are you upgrading from?

v<IP_ADDRESS> to v<IP_ADDRESS>

Rob: What version of the puppet provider are you on?

puppet agent v3.7.2

@trekytech7 I specifically meant can you do a puppet module list and print what version of the chocolatey module you have installed?

Rob: This seems a puppet-chocolatey issue, specifically #18.

Yes. I was looking at this one... possibly related, but effects more recent version of chocolatey.

Not necessarily, this appears to be happening with <IP_ADDRESS>. You are not running the latest version until your upgrade is successful.

My 1st try was to upgrade from old version to new using puppet-chocolatey provider, then from command line using "choco -f update chocolatey -version '<IP_ADDRESS>' -source '' " and both methods failed.

The command line was due to fail based on an empty source - you probably need to point it to the default source or provide a valid location.

Nevermind on empty source, looked at your comments again and fixed the markdwon.

puppet module list --environment "windows" --modulepath "/etc/puppet/environments/windows/modules"
/etc/puppet/environments/windows/modules
├── badgerious-windows_env (v2.2.1)
├── chocolatey-chocolatey (v0.5.2)
├── liamjbennett-win_facts (v0.0.2)
├── puppet-windows_firewall (v1.0.0)
├── puppetlabs-acl (v1.1.0)
├── puppetlabs-powershell (v1.0.4)
├── puppetlabs-reboot (v1.0.0)
├── puppetlabs-registry (v1.1.0)
├── puppetlabs-stdlib (v4.6.0)
As my upgrade was failing I tried the re-install option from cmd or pwrshl with puppet. The intent being to completely re-install (clobber) the previous version. However, this is where I get the error like #18 in puppet reports. Plus the log at "C:\Windows\Temp\chocolatey\chocInstall\tools\chocolateyInstall\chocolateyInstall.log" has the [variableName:'ChocolateyInstall' and variableValue:''] line. This indicates to me that the ChocolateyInstall variable may be null, causing it to fail. However, I'm running into the limits of my knowledge concerning how chocolatey and puppet works. :(
I'd be happy with a successful re-install or upgrade at this point. However it has to work from puppet over the previous years deployment.
The upgrade option puppet report output looks like this:
message: "C:\x5cwindows\x5csystem32>choco update -source 'http://REPO/' "
message: "Updating chocolatey from <IP_ADDRESS> to <IP_ADDRESS>"
...
message: "Chocolatey is unpacking required files for use. Overwriting? True"
message: "The pipeline has been stopped."
message: "out-lineoutput : The OS handle's position is not what FileStream expected. Do n"
message: "ot use a handle simultaneously in one FileStream and in Win32 code or another F"
message: "ileStream. This may cause data loss."
message: "    + CategoryInfo          : NotSpecified: (:) [out-lineoutput], IOException"
message: "    + FullyQualifiedErrorId : System.IO.IOException,Microsoft.PowerShell.Comma "
message: "   nds.OutLineOutputCommand"
message: "Failed to call refresh: cmd.exe /c C:\x5cAdmin\x5cECEchocoUpgrade.bat returned 1 instead of one of [0]"

The error you are getting is because there is an error in the upgrade. But you have to go to the actual log file to investigate. I'm thinking it's possible you need to not do the upgrade with chocolatey. The problems I see now are that you are using your own internal stuff to do the upgrade and it is hard to know what might be wrong based on that.

Rob: The error you are getting is because there is an error in the upgrade. But you have to go to the actual log file to investigate. I'm thinking it's possible you need to not do the upgrade with chocolatey.

message: "C:\x5cwindows\x5csystem32>choco update -source 'http://REPO/' "

I agree, the output after executing the command above, causes me to conclude that upgrading will be too troublesome. Instead I will only be pursuing a re-intall method. I think using pwrshl from puppet to do it, makes the most sense. If I really have to, I could do a remote pwrshl, but I've found puppet to be more reliable (sometimes remote pwrshl connections fail, but puppet always runs). Hopefully the 'upgrade' method with the new chocolatey will work well next year.

3rd try executes the chocolatey Install.ps1 directly with the puppet powershell provider
Rob: The problems I see now are that you are using your own internal stuff to do the upgrade and it is hard to know what might be wrong based on that.

I've simply pulled 'https://chocolatey.org/install.ps1' and changed line 20 & 47 to use our local/private source (conserves bandwidth). The chocolatey nuget will pull .NET installers locally, but otherwise there's NO change. We deploy Windows with MDT and install .NETv4.5.2 up front, so the .NET sources in the 'Install-DotNet4IfMissing' function, are unlikely to be used anyways. This simply keeps everything in-house and locks our versions for the school term.
Here's the puppet manifest exec to install/update chocolatey:
  exec {'chocoInstall':
	subscribe   => File['C:/Admin/chocoInstall.ps1'],
	refreshonly => true,
	path        => $::path,
	command     => 'C:\Admin\chocoInstall.ps1',
#	unless      => 'if (test for choco version <IP_ADDRESS>) { exit 1}',
	provider    => powershell,
	logoutput   => true,
  }

PS. On a side note: Is there a quick pwrshl one-liner method to check the chocolatey version? ;)
Here's the puppet message output after running the script:
     message: "Downloading http://REPO/Installer/chocolatey.<IP_ADDRESS>.nupkg to C:\x5cwindows\x5cTEMP\x5cchocolatey\x5cchocInstall\x5cchocolatey.zip"
      message: "Download 7Zip commandline tool"
      message: "Downloading http://REPO/Installer/7za.exe to C:\x5cwindows\x5cTEMP\x5cchocolatey\x5cchocInstall\x5c7za.exe"
      message: "Extracting C:\x5cwindows\x5cTEMP\x5cchocolatey\x5cchocInstall\x5cchocolatey.zip to C:\x5cwindows\x5cTEMP\x5cchocolatey\x5cchocInstall..."
      message: ""
      message: "7-Zip (A) 9.20  Copyright (c) 1999-2010 Igor Pavlov  2010-11-18"
      message: ""
      message: "Processing archive: C:\x5cwindows\x5cTEMP\x5cchocolatey\x5cchocInstall\x5cchocolatey.zip"
      message: ""
      message: "Extracting  _rels\x5c.rels"
      message: "Extracting  chocolatey.nuspec"
      message: "Extracting  tools\x5cchocolateyInstall\x5cchoco.exe"
      message: "Extracting  tools\x5cchocolateyInstall\x5cchoco.exe.ignore"
      message: "Extracting  Readme.txt"
      message: "Extracting  tools\x5cchocolateyInstall.ps1"
      message: "Extracting  tools\x5cchocolateysetup.psm1"
      message: "Extracting  tools\x5cinit.ps1"
      message: "Extracting  package\x5cservices\x5cmetadata\x5ccore-properties\x5cf14020bf99604cd9adc4e9f3ed31a355.psmdcp"
      message: "Extracting  [Content_Types].xml"
      message: ""
      message: "Everything is Ok"
      message: ""
      message: "Files: 10"
      message: "Size:       3722464"
      message: "Compressed: 1255790"
      message: "Write-Host : The OS handle's position is not what FileStream expected. Do not u"
      message: "se a handle simultaneously in one FileStream and in Win32 code or another FileS"
      message: "tream. This may cause data loss."
      message: "At C:\x5cAdmin\x5cchocoInstall.ps1:76 char:11"
      message: "+ Write-Host <<<<  \x22Installing chocolatey on this machine\x22"
      message: "    + CategoryInfo          : NotSpecified: (:) [Write-Host], IOException"
      message: "    + FullyQualifiedErrorId : System.IO.IOException,Microsoft.PowerShell.Comma "
      message: "   nds.WriteHostCommand"
      message: " "
      message: "Chocolatey v<IP_ADDRESS>"
      message: "Chocolatey is unpacking required files for use. Overwriting? True"
      message: "The OS handle's position is not what FileStream expected. Do not use a handle simultaneously in one FileStream and in Win32 code or another FileStream. This may cause data loss."
      message: "C:\x5cAdmin\x5cchocoInstall.ps1 returned 1 instead of one of [0]"
      message: "change from notrun to 0 failed: C:\x5cAdmin\x5cchocoInstall.ps1 returned 1 instead of one of [0]"
      message: "Dependency Exec[ECEchocoUpgrade] has failures: true"

And a couple logs:
C:\Windows\Temp\chocolatey\chocInstall\tools\chocolateyInstall\logs\chocolatey.log
2015-05-12 10:26:42,420 [INFO ] - ============================================================
2015-05-12 10:26:42,467 [INFO ] - Chocolatey v<IP_ADDRESS>
2015-05-12 10:26:42,560 [INFO ] - Chocolatey is unpacking required files for use. Overwriting? True

C:\Windows\Temp\chocolatey\chocInstall\tools\chocolateyInstall\chocolateyInstall.log does not exist.
C:\ProgramData\chocolatey\chocolateyinstall\chocolateyInstall.log
20150512-10:26:36 [DEBUG] Running 'Append-InstallLog' with chocoInstallLog:''
20150512-10:26:36 [CHOCO] ################################################################################
20150512-10:26:36 [CHOCO] #                  Going Chocolatey on 2015-05-12 10:26:36Z                    #
20150512-10:26:36 [CHOCO] ################################################################################
20150512-10:26:36 [DEBUG] Arguments: $command = ''|$packageNames=''|$source=''|$version=''|$allVersions=False|$InstallArguments=''|$overrideArguments=False|$force=False|$prerelease=False|$localonly=False|$verbosity=True|$debug=False|$quiet=False|$name=''|$ignoreDependencies=False|$forceX86=False|$packageParameters=''|PowerShellVersion=2.0
20150512-10:26:36 [CHOCO] Please run chocolatey /? or chocolatey help - chocolatey v<IP_ADDRESS>
20150512-10:26:37 [DEBUG] Running 'Append-InstallLog' with chocoInstallLog:''
20150512-10:26:37 [CHOCO] ################################################################################
20150512-10:26:37 [CHOCO] #                  Going Chocolatey on 2015-05-12 10:26:37Z                    #
20150512-10:26:37 [CHOCO] ################################################################################
20150512-10:26:37 [DEBUG] Arguments: $command = 'list'|$packageNames=''|$source=''|$version=''|$allVersions=False|$InstallArguments=''|$overrideArguments=False|$force=False|$prerelease=False|$localonly=True|$verbosity=False|$debug=False|$quiet=False|$name=''|$ignoreDependencies=False|$forceX86=False|$packageParameters=''|PowerShellVersion=2.0
20150512-10:26:37 [DEBUG] Running 'Chocolatey-List' with selector: '', source:''
20150512-10:26:37 [DEBUG] Showing prerelease versions of packages
20150512-10:26:37 [DEBUG] Executing command ["C:\ProgramData\chocolatey\chocolateyinstall\nuget.exe" list -Prerelease -NonInteractive -Source "C:\ProgramData\chocolatey\lib"]
20150512-10:26:38 [DEBUG] Command ["C:\ProgramData\chocolatey\chocolateyinstall\nuget.exe" list -Prerelease -NonInteractive -Source "C:\ProgramData\chocolatey\lib"] exited with '0'.

Now running the script directly as admin with powershell (it works as expected):
PS C:\Admin> .\ECEchocoUpgrade.ps1
Downloading http://REPO/Installer/chocolatey.<IP_ADDRESS>.nupkg to C:\Users\admin\AppData\Local\Temp\choc
olatey\chocInstall\chocolatey.zip
Download 7Zip commandline tool
Downloading http://REPO/Installer/7za.exe to C:\Users\admin\AppData\Local\Temp\chocolatey\chocInstal
l\7za.exe
Extracting C:\Users\admin\AppData\Local\Temp\chocolatey\chocInstall\chocolatey.zip to C:\Users\admin\AppData\Local
\Temp\chocolatey\chocInstall...

7-Zip (A) 9.20  Copyright (c) 1999-2010 Igor Pavlov  2010-11-18

Processing archive: C:\Users\admin\AppData\Local\Temp\chocolatey\chocInstall\chocolatey.zip

Extracting  _rels\.rels
Extracting  chocolatey.nuspec
Extracting  tools\chocolateyInstall\choco.exe
Extracting  tools\chocolateyInstall\choco.exe.ignore
Extracting  Readme.txt
Extracting  tools\chocolateyInstall.ps1
Extracting  tools\chocolateysetup.psm1
Extracting  tools\init.ps1
Extracting  package\services\metadata\core-properties\f14020bf99604cd9adc4e9f3ed31a355.psmdcp
Extracting  [Content_Types].xml

Everything is Ok

Files: 10
Size:       3722464
Compressed: 1255790
Installing chocolatey on this machine
Chocolatey v<IP_ADDRESS>
Chocolatey is unpacking required files for use. Overwriting? True
Creating ChocolateyInstall as an environment variable (targeting 'Machine')
  Setting ChocolateyInstall to 'C:\ProgramData\chocolatey'
WARNING: It's very likely you will need to close and reopen your shell
  before you can use choco.
Adding Modify permission for current user to 'C:\ProgramData\chocolatey'
We are setting up the Chocolatey package repository.
The packages themselves go to 'C:\ProgramData\chocolatey\lib'
  (i.e. C:\ProgramData\chocolatey\lib\yourPackageName).
A shim file for the command line goes to 'C:\ProgramData\chocolatey\bin'
  and points to an executable in 'C:\ProgramData\chocolatey\lib\yourPackageName'.

Creating Chocolatey folders if they do not already exist.

WARNING: You can safely ignore errors related to missing log files when
  upgrading from a version of Chocolatey less than 0.9.9.
  'Batch file could not be found' is also safe to ignore.
  'The system cannot find the file specified' - also safe.
Chocolatey v<IP_ADDRESS>
Chocolatey is unpacking required files for use. Overwriting? True
Ensuring 'C:\ProgramData\chocolatey\lib\chocolatey' exists.
Chocolatey (choco.exe) is now ready.
You can call choco from anywhere, command line or powershell by typing choco.
Run choco /? for a list of functions.
You may need to shut down and restart powershell and/or consoles
 first prior to using choco.
Ensuring chocolatey commands are on the path

C:\Users\admin\AppData\Local\Temp\chocolatey\chocInstall\tools\chocolateyInstall\chocolateyInstall.log
20150511-10:53:48 [DEBUG] Test-ProcessAdminRights: returning True
20150511-10:53:48 [DEBUG] Administrator installing so using Machine environment variable target instead of User.
20150511-10:53:48 [CHOCO] Creating ChocolateyInstall as an Environment variable (targeting 'Machine') and setting it to 'C:\ProgramData\chocolatey'
20150511-10:53:48 [DEBUG] Running 'Install-ChocolateyEnvironmentVariable' with variableName:'ChocolateyInstall' and variableValue:'C:\ProgramData\chocolatey'
20150511-10:53:48 [DEBUG] Test-ProcessAdminRights: returning True
20150511-10:53:48 [CHOCO] We are setting up the Chocolatey repository for NuGet packages that should be at the machine level. Think executables/application packages, not library packages.
That is what Chocolatey NuGet goodness is for. The repository is set up at 'C:\ProgramData\chocolatey'.
The packages themselves go to 'C:\ProgramData\chocolatey\lib' (i.e. C:\ProgramData\chocolatey\lib\yourPackageName).
A shim file for the command line goes to 'C:\ProgramData\chocolatey\bin' and points to an executable in 'C:\ProgramData\chocolatey\lib\yourPackageName'.

... and so on (standard install log)

My suspicion has been that the ChocolateyInstall variable is null, based on the install log ([DEBUG] Running 'Install-ChocolateyEnvironmentVariable' with variableName:'ChocolateyInstall' and variableValue:''). Possibly because puppet is running as the System user (the variable is set globally).

Rob: That is likely removing a value - unfortunately the message probably also needs to state the scope of the change - user versus system. We attempt to remove the user version if we are setting it globally, which for administrative install we are going for the global variable.

Env variable is set globally... I ran "Get-ChildItem Env:ChocolateyInstall" from puppet just to confirm if it is there when running from system user, and it's set correctly to "C:\ProgramData\chocolatey".
There is one other difference I see: it looks like chocolatey may be partially installed. Some, but not all, of the folders and so forth are present in the ChocolateyInstall location after Chocolatey Install script has run.

Hopefully the 'upgrade' method with the new chocolatey will work well next year.
Actually the new choco doesn't have any issues with upgrade, unlike the older version.

You may be running into many things when you are attempting an upgrade from the old version to the new, most likely is the move from c:\Chocolatey to C:\ProgramData\Chocolatey, which ends up looking like an error but has numerous messages indicating otherwise. It just has to remove the remnants of C:\Chocolatey from the file system and from the environment variables.

I've simply pulled 'https://chocolatey.org/install.ps1' and changed line 20 & 47 to use our local/private source (conserves bandwidth). The chocolatey nuget will pull .NET installers locally, but otherwise there's NO change. We deploy Windows with MDT and install .NETv4.5.2 up front, so the .NET sources in the 'Install-DotNet4IfMissing' function, are unlikely to be used anyways. This simply keeps everything in-house and locks our versions for the school term.

Makes sense. Keeping it all in house that is.

PS. On a side note: Is there a quick pwrshl one-liner method to check the chocolatey version? ;)

We do it in the provider - see https://github.com/chocolatey/puppet-chocolatey/blob/master/lib/puppet/provider/package/chocolatey.rb#L43-L45

Env variable is set globally... I ran "Get-ChildItem Env:ChocolateyInstall" from puppet just to confirm if it is there when running from system user, and it's set correctly to "C:\ProgramData\chocolatey".

:+1:
Is it there when it is done?

There is one other difference I see: it looks like chocolatey may be partially installed. Some, but not all, of the folders and so forth are present in the ChocolateyInstall location after Chocolatey Install script has run.

Is this when Puppet runs it versus when you run it? I'm guessing so.

message: "Write-Host : The OS handle's position is not what FileStream expected. Do not u"
message: "se a handle simultaneously in one FileStream and in Win32 code or another FileS"
message: "tream. This may cause data loss."

So here's the problem and why you are seeing this message. Puppet redirects stdout / stderr while running processes and because of a bug in PowerShell, you can't write out to anything more than one stream (Write-Host vs Write-Warning vs Write-Error). I would tend to state you should capture the exit code, but send everything as Write-Host instead of Write-Warning / Write-Error. I built a package that tests this brokenness. Take a look at https://github.com/chocolatey/puppet-chocolatey/issues/18#issuecomment-35432020
The good news is that choco 0.9.9+ does not suffer from this problem, the bad news is that old choco (anything under <IP_ADDRESS>) and anything PowerShell itself still suffers from this problem.

https://github.com/chocolatey/puppet-chocolatey/issues/18#issuecomment-35440191
https://github.com/chocolatey/puppet-chocolatey/issues/18#issuecomment-42114148

You may be running into many things when you are attempting an upgrade from the old version to the new, most likely is the move from c:\Chocolatey to C:\ProgramData\Chocolatey, which ends up looking like an error but has numerous messages indicating otherwise. It just has to remove the remnants of C:\Chocolatey from the file system and from the environment variables.
I anticipated this and made sure ChocolateyInstall was set to "C:\ProgramData\chocolatey". As I recall, I tried Boxstarter and had it set in during its install.

There is one other difference I see: it looks like chocolatey may be partially installed. Some, but not all, of the folders and so forth are present in the ChocolateyInstall location after Chocolatey Install script has run.

Is this when Puppet runs it versus when you run it? I'm guessing so.

Yes. Puppet runs fail, but manually running chocolatey installation works.

The good news is that choco 0.9.9+ does not suffer from this problem, the bad news is that old choco (anything under <IP_ADDRESS>) and anything PowerShell itself still suffers from this problem.

ugh. rismoney is right that's ugly.
Executing this via Puppet succeeds. From tkelman https://github.com/chocolatey/puppet-chocolatey/issues/18#issuecomment-56116072
(iex ((new-object net.webclient).DownloadString('https://chocolatey.org/install.ps1'))) >$null 2>&1

PS. On a side note: Is there a quick pwrshl one-liner method to check the chocolatey version? ;)

We do it in the provider - see https://github.com/chocolatey/puppet-chocolatey/blob/master/lib/puppet/provider/package/chocolatey.rb#L43-L45

Good. I don't actually know ruby though. I'm relatively familiar with puppet... more to learn in my copious time. Is there something, like a variable, that puppet agent can check against?
What I'm seeing is that "choco_exe?" is true or false, determining if it's C# chocolatey. So that would determine, at least for this year, whether choco has updated on the client. Of course, it shouldn't miss upgrading because puppet subscribes to the pwrshl script, but we know this doesn't always happen in practice. I don't, currently, know how to get this info in the chocolatey puppet manifest (or use it as the case may be).

Basically you need to call choco -v and determine if the output contains Please run chocolatey - you can do that in PowerShell in maybe one line, probably a small script instead. That will allow you to make the determination of whether you are on the c# choco or not. And next year you won't need the script, you can just use the package resource to upgrade chocolatey to whatever version you want it to be without issues.

Also, you could look for the existence of a file that would only be in the PowerShell version of chocolatey. Like https://github.com/chocolatey/chocolatey/blob/master/src/chocolatey.ps1, that file no longer exists and would be a big determination.

Also, you could look for the existence of a file that would only be in the PowerShell version of chocolatey. Like https://github.com/chocolatey/chocolatey/blob/master/src/chocolatey.ps1, that file no longer exists and would be a big determination.

I like that: quick and easy one-liner... And the EXEC becomes :)
  exec {'chocoInstall':
	subscribe   => File['C:/Admin/chocoInstall.ps1'],
#	refreshonly => true,
	path        => $::path,
	command     => 'C:\Admin\chocoInstall.ps1',
	unless      => 'if (Test-Path "C:\ProgramData\chocolatey\chocolateyinstall\chocolatey.ps1") { exit 1}',
	provider    => powershell,
	logoutput   => true,
	before      => {the rest of your packages/modules},
  }

If you can make it, I'd go for <IP_ADDRESS>.

@ferventcoder - FWIW, I ran into this issue and, after reading all of this, crafted a fix that you may want to share. This is used to get Chocolatey on my hosts so that this module can then maintain it:
# lint:ignore:80chars
exec { 'remove-old-chocolatey':
  provider => powershell,
  onlyif   => 'if (Test-Path -Path "C:/ProgramData/chocolatey/chocolateyinstall/chocolatey.ps1") { exit 0 } else { exit 1 }',
  command  => 'Remove-Item -Recurse -Force C:/ProgramData/chocolatey',
  before   => Class['chocolatey_sw'],
}
# lint:endignore

# module: ceritsc/chocolatey_sw
class { 'chocolatey_sw':
  creates => ['C:/ProgramData/chocolatey'], # check for install dir
}

This ensures that versions with the old chocolatey.ps1 get deleted totally instead of the upgrade process running (and failing).

@genebean Not bad, except I don't want the deployment to attempt to install programs again.
  command  => 'Remove-Item -Recurse -Force C:/ProgramData/chocolatey',

This would need to do every folder underneath, except for 'lib'. ;)

@trekytech7 - I can do that (and am glad you pointed it out). I'll post back soon with a revised version.

@ferventcoder Encountered another hitch while testing prior to turning puppet management back on in 8 undergrad labs. This issue may occur from time to time; if the Chocolatey install fails, for some reason or other :(. The unless statement below will not determine that the choco installation is incomplete (or whatever -I've NO time to look into it).
    unless      => 'if (Test-Path "C:\ProgramData\chocolatey\chocolateyinstall\chocolatey.ps1") { exit 1}',

My other mistake here was to order a puppet agent upgrade before chocolatey upgrade, so it didn't even try to re-install chocolatey (& wouldn't) if the puppet upgrade failed... Ahh! I'm now changing the chocolatey to be before the puppet upgrade, including reboots.
I may change the 'unless' to use a script instead, so I can validate the choco install too. Or perhaps, have the script check the install right after and then re-try if there was an issue. I'm currently undecided which method to use, but it should definitely be more robust (& automated, so every client fixes itself).

@trekytech7 I haven't posted back for this very reason. I like your idea of having the exec also do the new chocolatey install since it is what kills the old one.

@trekytech7 What about this? The lib folder is untouched and Chocolatey is installed / reinstalled unless choco.exe is there.
# lint:ignore:80chars
exec { 'remove-old-chocolatey':
  provider => powershell,
  onlyif   => 'if (Test-Path -Path "C:/ProgramData/chocolatey/chocolateyinstall/chocolatey.ps1") { exit 0 } else { exit 1 }',
  command  => 'Get-ChildItem -Path C:/ProgramData/chocolatey | Where-Object {$_.Name -NotMatch "^lib$" } | Remove-Item -Recurse -Force',
  before   => Class['chocolatey_sw'],
}
# lint:endignore

# module: ceritsc/chocolatey_sw
class { 'chocolatey_sw':
  creates => 'C:/ProgramData/chocolatey/choco.exe',
}

Slight revision
# lint:ignore:80chars
exec { 'remove-old-chocolatey':
  provider => powershell,
  onlyif   => 'if (Test-Path -Path "C:/ProgramData/chocolatey/chocolateyinstall/chocolatey.ps1") { exit 0 } else { exit 1 }',
  command  => 'Get-ChildItem -Path C:/ProgramData/chocolatey | Where-Object {$_.Name -NotMatch "^lib$" } | Remove-Item -Recurse -Force',
  before   => Class['chocolatey_sw'],
}
# lint:endignore

# module: ceritsc/chocolatey_sw
class { 'chocolatey_sw':
  creates => 'C:/ProgramData/chocolatey/choco.exe',
  before  => Package['chocolatey'],
}

@genebean I like your revised 'remove-old-chocolatey' oneliner.
  command  => 'Get-ChildItem -Path C:/ProgramData/chocolatey | Where-Object {$_.Name -NotMatch "^lib$" } | Remove-Item -Recurse -Force',

Removing the old version would fix the failure I more recently experienced, where the chocolatey installation is incomplete & fails, even manually, to be re-installed (mentioned above). The issue probably desires a new issue #. I manually deleted the install folders, except 'lib', and was able to manually re-install.
The item at C:\ProgramData\chocolatey\bin\ has children and the Recurse parameter was not specified. If you continue,
all children will be removed with the item. Are you sure you want to continue?
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"):
WARNING: Was not able to replace 'C:\ProgramData\chocolatey\bin\' with ''.

@ferventcoder If you would like the install files for the above 'NEW' issue, then I did keep a copy. However, I'm running out of time and must move onto other things, such as fixing our MDT deployment server ;).

Adding this as a documentation issue.

#65 is related

in <IP_ADDRESS> i believe there is are some minor bugs in the installer.  I figured out a way to instrument it, but
i am trying to hunt it down.
all i got in the log-
Notice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Missing closing '}' in statemen
t block.
Notice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Missing closing '}' in statemen
t block.
Notice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Missing closing '}' in statemen
t block.
Notice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Missing closing '}' in statemen
t block.
Notice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Missing closing '}' in statemen
t block.
Notice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Ensuring chocolatey commands ar
e on the path
Notice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: Ensuring chocolatey.nupkg is in
 the lib folder
Notice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: executed successfully�
        Install\tools\chocolateyInstall.ps1:10 char:28Notice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: +remove-module <<<<  $moduleName -ErrorAction SilentlyContinueNotice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns: ;Notice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:     + CategoryInfo          : ResourceUnavailable: (:) [Remove-Module], InvalidOperationExceptionNotice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:     + FullyQualifiedErrorId : Modules_NoModulesRemoved,Microsoft.PowerShell.Commands.RemoveModuleCommandNotice: /Stage[protozoic]/Chocolatey::Install/Exec[install_chocolatey_official]/returns:Notice: /Stage[proto�

I think the errors in stderr are benign indicating a missing closing }.  I searched curly braces for an hour...ugh. then off to google.  This I believe was fixed in WMF 4 and above.
see:
https://connect.microsoft.com/PowerShell/feedback/details/371321/benign-errors-for-missing-closing-are-left-in-the-error-variable
I have a solution to these upgrading/installing FileStream errors and I plan to confine the solution to PSv2.
CAUTION: The drawback of my solution is that is that if a nested function within install_chocolatey_official (if ($error -ne $null)�), we will not fail the resource (which is what is causing the filestream error, but instead just report on it. I believe this to be better, because at least you can see the error and troubleshoot, instead of the error on stderr handling.
I would expect a true failed install (as I have seen in several scenarios), the provider/packages will have issues anyway and force subsequent resources skipping anyway.
If there is truly a problem, one will have to investigate the debug log.
Alternatives-

hide stderr altogether from view, and make it always pretty.
log stderr to $env:\temp\chocoerror.log


small question

I have a server.
It is a 4 Core 1GHZ CPU with 1GB RAM
and approx 1 MBPS Network UP
2 MBPS Network Down
Will this hold an Multi-Ogar Edited server for maybe around 30 players?
If not what is the reccommened hardware specifications...
Because my high end current server cannot be used at the moment...

My bad 31 MBPS download

idk

@Niall7459 it depends on your hoster. If you port forward then 31MBPS could be a happy way to start it.
If you're talking about FPS then that should handle it to - my potato laptop gets average 45 - 60fps off it :P

If it's VPS, processor time is shared between virtual servers. That means in fact server can use only part of 1 GHz@4cores, but 10% is still very much. Seems, thats network speed is weak point. You can meausure used connection of one client, and then calculate it. If you do it, please share results

I measured data rate on server with 40 bots. My result:
server to one client:
40 KByte/s=320KBit/s when connected, but not playing
1 KByte/s=8KBit/s on game
50 KByte/s=400KBit/s on spectator mode
one client to server:
0,5 KByte/s=4Kbit/s in game
0 when not playing
Internet and server providers meausers link in KBit/s
Your server can use about 34Kbit/s from one client to server and 68KBit/s from server to one client

If server is self-hosted maybe your router uses feature, that maximize priority of video traffic and real link is not 1 and 2 MBPS

So there is more upload used than download.
yes it is self hosted

Do you have problem with performance or you only have interest? Seems, that 31MBPS link is ok. upload from who? Player or server?

Upload from server is approx 1 MBPS
Occasionally the game just freezes, this is at my school, so wether its the schools network I dont know. because Theres alot of people play

My verdict - ploblem is in 1MBPS link from server to client. Only 4 connected, but playing users can freeze your server(4*320=1280>1024 KBPS).
Suggest to close issue

OH NO

Ok but will my CPU be okay???
1 GHZ  x4 Core ARM Cortex - A53
Running  alongside PHP, NGINX, MYSQL

And memory = 1 GB!

@Niall7459 Yes. I don't calculated this, but seems that it is big values

@cyclopentan Wait to those specs will be fine for about 10 players?

Will it be slowed down if:
-- Internet ----->    My Server    ---Proxy ----> Other Server (With multiogar)
|
--> Website
Will this slow it down?

It's unclear what you asking. What do you mean, when say "Connection between servers"?
Scheme is Client - Client's ISP - Internet - Server's ISP or hoster - Server
Servers can't connect directly to each other.


Add no-output-named-after-standard-event to tslint.json

🚀 Feature request
Command

[X] new

Description
There is a rule no-output-named-after-standard-event in codelyzer, I think it should be enabled as a default setting because we can not remember ALL the native event name.
When we use the standard event name, that could be easily causing an unexpected problem.
So I recommend to enable it when we just start a new project.
Describe the solution you'd like
Add "no-output-named-after-standard-event": true to tslint.json as default when using ng new to create a new Angular project.

Being addressed in https://github.com/angular/angular-cli/pull/13428

Closing as it was addressed in #13428


HLS live streaming gets 4032 codec error

Hi team!
I have tested some hls streams, got the 4032 error in Chrome, etc (except Safari).
Log:
Dropping variant - video not compatible with platform type=video codecs=avc1.4d0028,mp4a.40.2 bandwidth=undefined frameRate=24 width=1920 height=800
If I check through canPlayType, so the codes are really not supported.
const result = document.createElement('video') result.canPlayType('video/mp2t;codecs="avc1.4d0028,mp4a.40.2"')
So, why other players can play the same stream in Chrome?
https://bitmovin.com/demo
https://videojs.github.io/videojs-contrib-hls
Can you please help me with understanding?
thanks in advance)

We don't use video.canPlayType since we aren't using src= playback.  You need to use MediaSource.isTypeSupported.  Chrome doesn't support playback of MP2-TS content, so we need to transmux it to MP4.  We do this using mux.js, but you need to include it separately.  For example:
https://github.com/google/shaka-player/blob/d4d0c4f8880038f6df7371b87b6bc4d36c4aa570/demo/index.html#L40
Does the stream work in the hosted demo?  Can you post your manifest (or send it to shaka-player-issues@google.com)?

Muchas gracias!
It works, thanks for the quick response 👍


Which version will work on WD PR4100

Can you please tell me which version of telly will work on my WD PR4100, and can you please give me some simple instructions? Thanks

Looks like you're able to use Docker on that NAS.
https://hub.docker.com/r/tombowditch/telly/

@sneakypete73 -
You need to find a way to daemonize telly on that NAS.
As @Codehhh said - Docker may work on the NAS. We build telly so you can pull it easily on Dockerhub @ tombowditch/telly, you should just be able to pull that.
T

Following those instructions.
I entered:
./telly<EMAIL_ADDRESS>-uktv
And got:
Unable to read m3u file
Unable to open playlist file
From: Tom Bowditch<EMAIL_ADDRESS>Sent: 03 July 2018 16:03
To: tombowditch/telly<EMAIL_ADDRESS>Cc: sneakypete73<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tombowditch/telly] Which version will work on WD PR4100 (#126)
@sneakypete73 https://github.com/sneakypete73  -
You need to find a way to daemonize telly on that NAS.
As @Codehhh https://github.com/Codehhh  said - Docker may work on the NAS. We build telly so you can pull it easily on Dockerhub @ tombowditch/telly, you should just be able to pull that.
T
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tombowditch/telly/issues/126#issuecomment-402189247 , or mute the thread https://github.com/notifications/unsubscribe-auth/AlfzNtsxR1uycFe-ji29eXUg1LfafHvBks5uC4fAgaJpZM4U_lbU .  https://github.com/notifications/beacon/AlfzNjN88xKJjw2vrhYGQ4pz-ELvvf_Xks5uC4fAgaJpZM4U_lbU.gif

@sneakypete73
Firstly, PLEASE CHANGE YOUR PASSWORD TO YOUR IPTV ACCOUNT - you've just gave a few hundred people your username and password.
Secondly, that command is failing because you have a / just before http
Change  –playlist=/http:// to  –playlist=http://
T

Ooops.
Now. I do indeed have docker – although I have never used it.
What is the first step?
Is it docker run, docker compose, or setup as the first step – and are all commands entered in Putty?
Thanks Tom
From: Tom Bowditch<EMAIL_ADDRESS>Sent: 04 July 2018 11:45
To: tombowditch/telly<EMAIL_ADDRESS>Cc: sneakypete73<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tombowditch/telly] Which version will work on WD PR4100 (#126)
@sneakypete73 https://github.com/sneakypete73
Firstly, PLEASE CHANGE YOUR PASSWORD TO YOUR IPTV ACCOUNT - you've just gave a few hundred people your username and password.
Secondly, that command is failing because you have a / just before http
Change –playlist=/http:// to –playlist=http://
T
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tombowditch/telly/issues/126#issuecomment-402441716 , or mute the thread https://github.com/notifications/unsubscribe-auth/AlfzNhnWRbeiqEf2-5iwD5ZPmfuK88vBks5uDJzBgaJpZM4U_lbU .  https://github.com/notifications/beacon/AlfzNtRGww0mbxgjMuTq4bR6xDmrd4XQks5uDJzBgaJpZM4U_lbU.gif

Take a look at the readme.
Do I have to enter both the docker run and docker compose text? And in what order? Any is this entered through Putty?
From: Tom Bowditch<EMAIL_ADDRESS>Sent: 04 July 2018 12:10
To: tombowditch/telly<EMAIL_ADDRESS>Cc: sneakypete73<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tombowditch/telly] Which version will work on WD PR4100 (#126)
Take a look at the readme. Run something like -
docker run -d
--name='telly'
--net='bridge'
-e TZ="Europe/Amsterdam"
-e 'PLAYLIST'='/home/github/myiptv.m3u'
-e STREAMS=1
-e USEREGEX='.UK.'
-p '6077:6077/tcp'
-v '/tmp/telly':'/tmp':'rw'
tombowditch/telly -base=localhost:6077
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tombowditch/telly/issues/126#issuecomment-402447185 , or mute the thread https://github.com/notifications/unsubscribe-auth/AlfzNuVDCFkQHjGyaK8ZWVsCDG0YF4tzks5uDKKRgaJpZM4U_lbU .  https://github.com/notifications/beacon/AlfzNgkEwsG4580i_w9REjyJwuUg1OnWks5uDKKRgaJpZM4U_lbU.gif

@sneakypete73 To use docker on your NAS: https://community.wd.com/t/docker-and-apps/170177/2
You should only run the docker run command.

Tom I keep getting this error - can you please offer any assistance:
Unable to find image 'tombowditch/telly:latest' locally
latest: Pulling from tombowditch/telly
6763caf1f9ee: Pull complete
4417fed44ce9: Pull complete
Digest: sha256:dc661e56656217642db91f0a4c6ad53e048b41a4173057d7b20bef0bbe8fe570
Status: Downloaded newer image for tombowditch/telly:latest
docker: Error response from daemon: Conflict. The container name "/telly" is already in use by container "3974e747a3f69749f9cd847baa3ebf0a775ec6278ec2e8b166256b562e1faa49". You have to remove (or rename) that container to be able to reuse that name.

@sneakypete73 You already have a docker named telly. You have to either remove (or rename) the container.
docker stop telly && docker rm telly will stop the current telly instance and remove it. Then you can proceed with pulling the new container.

Thanks. I since stopped telly and purged all containers.
 But Plex doesn’t find anything.
Am I missing something?
From: Codehhh<EMAIL_ADDRESS>Sent: 14 August 2018 14:52
To: tombowditch/telly<EMAIL_ADDRESS>Cc: sneakypete73<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tombowditch/telly] Which version will work on WD PR4100 (#126)
@sneakypete73 https://github.com/sneakypete73  You already have a docker named telly. You have to either remove (or rename) the container.
docker stop telly && docker rm telly will stop the current telly instance and remove it. Then you can proceed with pulling the new container.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tombowditch/telly/issues/126#issuecomment-412878880 , or mute the thread https://github.com/notifications/unsubscribe-auth/AlfzNgnnu9cj5BrQPOV-jQlbYGwTr1qUks5uQtYFgaJpZM4U_lbU .  https://github.com/notifications/beacon/AlfzNh1yqu9o7bEnjpJjdMJ6AkTMwAgzks5uQtYFgaJpZM4U_lbU.gif

@sneakypete73 Try this:
docker run --rm --detach \ --name=telly \ --net=host \ -e TZ="Europe/Amsterdam" \ -e 'TELLY_IPTV_PLAYLIST'='/home/github/myiptv.m3u' \  -e TELLY_IPTV_STREAMS=1 \ -e TELLY_FILTER_REGEX='.*UK.*' \ tombowditch/telly --listen.base-address=<IP_ADDRESS>:6077

Thanks but no good:
docker: invalid reference format.
If anyone can offer a simply step-by-step guide for Putty I’d be eternally grateful. I’m using a http address for the m3u
Assume I know nothing…. sorry
From: slawekm<EMAIL_ADDRESS>Sent: 14 August 2018 15:32
To: tombowditch/telly<EMAIL_ADDRESS>Cc: sneakypete73<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tombowditch/telly] Which version will work on WD PR4100 (#126)
@sneakypete73 https://github.com/sneakypete73  Try this:
docker run --rm --detach \ --name=telly \ --net=host \ -e TZ="Europe/Amsterdam" \ -e 'TELLY_IPTV_PLAYLIST'='/home/github/myiptv.m3u' \ -e TELLY_IPTV_STREAMS=1 \ -e TELLY_FILTER_REGEX='.UK.' \ tombowditch/telly --listen.base-address=<IP_ADDRESS>:6077
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tombowditch/telly/issues/126#issuecomment-412892795 , or mute the thread https://github.com/notifications/unsubscribe-auth/AlfzNt3xJVB-Gk4hQzFRM35jApsREB_xks5uQt9ogaJpZM4U_lbU .  https://github.com/notifications/beacon/AlfzNqzomI9QnUoR0jOrAe6aim3xYQiTks5uQt9ogaJpZM4U_lbU.gif

@sneakypete73 my bad, didn't test it locally..
How about this one?
docker run --detach --rm --name telly --net=host\
 -e 'TZ=Europe/Amsterdam'\
 -e 'TELLY_IPTV_PLAYLIST=/home/github/myiptv.m3u'\
 -e 'TELLY_FILTER_REGEX=.*UK.*'\
 tombowditch/telly --web.base-address=<IP_ADDRESS>:6077

(I've dropped TELLY_IPTV_STREAMS which seems to be redundant as it's set to default value).
You need to provide real playlist file in that location too (or mount via -v switch from host).
I've also noticed that you can't specify <IP_ADDRESS>:6077 address - which should be desirable when you want to expose service outside the local machine. I'm getting this error instead:
time="2018-08-14T14:54:50Z" level=panic msg="base URL is set to <IP_ADDRESS>, this will not work. please use the --web.base-address option and set it to the (local) ip address telly is running on."
panic: (*logrus.Entry) (0xaf0d60,0xc42007c870)

@sneakypete73 To see the docker logs after starting the container, run docker logs telly

Thanks
How do I remove filtering?
From: Codehhh<EMAIL_ADDRESS>Sent: 16 August 2018 14:38
To: tombowditch/telly<EMAIL_ADDRESS>Cc: sneakypete73<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tombowditch/telly] Which version will work on WD PR4100 (#126)
@sneakypete73 https://github.com/sneakypete73  To see the docker logs after starting the container, run docker logs telly
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tombowditch/telly/issues/126#issuecomment-413548207 , or mute the thread https://github.com/notifications/unsubscribe-auth/AlfzNtfpJK5Ke8DG3TW9JJn9cIGzX6Kvks5uRXWxgaJpZM4U_lbU .  https://github.com/notifications/beacon/AlfzNonLlN6_aUKABcKpQhB91MapM657ks5uRXWxgaJpZM4U_lbU.gif

@sneakypete73 If you remove TELLY_FILTER_REGEX you'll want to add TELLY_REGEX_INCLUSIVE.  The default regex behavior is now "blacklist" so running with no regex means "exclude everything".  Adding the inclusive flag flips that to "include everything".

My problem seems to be the categories. If I filer with UK. or UK I don’t get UK channel – everything but UK. If I filter with UK Entertainment. I get UK channels but I 4880 channels. I can’t seem to narrow it down to just the 100 or so UK channels I want.
From: Chaz Larson<EMAIL_ADDRESS>Sent: 17 August 2018 17:57
To: tellytv/telly<EMAIL_ADDRESS>Cc: sneakypete73<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tellytv/telly] Which version will work on WD PR4100 (#126)
@sneakypete73 https://github.com/sneakypete73  If you remove TELLY_FILTER_REGEX you'll want to add TELLY_REGEX_INCLUSIVE. The default regex behavior is now "blacklist" so running with no regex means "exclude everything". Adding the inclusive flag flips that to "include everything".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tellytv/telly/issues/126#issuecomment-413926680 , or mute the thread https://github.com/notifications/unsubscribe-auth/AlfzNt3KPVBX22vPOiu8jIx0sJR2kiaAks5uRvXPgaJpZM4U_lbU .  https://github.com/notifications/beacon/AlfzNnkTnv5c2AAqLvk5bO5fKMhW_QQLks5uRvXPgaJpZM4U_lbU.gif

The regex behavior defaults to working as a blacklist.
This:
docker run --detach --rm --name telly --net=host \
-e 'TZ=Europe/Amsterdam' \
-e 'TELLY_IPTV_PLAYLIST=http://tv.hdtv4u.cc:2052/get.php?username=********&password=********&type=m3u_plus&output=ts'\
-e 'TELLY_FILTER_REGEX=.*UK.*' \
tombowditch/telly --web.base-address=<IP_ADDRESS>:6077

Is stripping channels that match the regex.

It works at last – thanks all for your help
From: Chaz Larson<EMAIL_ADDRESS>Sent: 18 August 2018 14:44
To: tellytv/telly<EMAIL_ADDRESS>Cc: sneakypete73<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [tellytv/telly] Which version will work on WD PR4100 (#126)
The regex behavior defaults to working as a blacklist.
This:
docker run --detach --rm --name telly --net=host
-e 'TZ=Europe/Amsterdam'
-e 'TELLY_IPTV_PLAYLIST=http://tv.hdtv4u.cc:2052/get.php?username=&password=&type=m3u_plus&output=ts'
-e 'TELLY_FILTER_REGEX=.UK.'
tombowditch/telly --web.base-address=<IP_ADDRESS>:6077
Is stripping channels that match the regex.
This:
docker run --detach --rm --name telly --net=host
-e 'TZ=Europe/Amsterdam'
-e 'TELLY_IPTV_PLAYLIST=http://tv.hdtv4u.cc:2052/get.php?username=&password=&type=m3u_plus&output=ts'
-e 'TELLY_FILTER_REGEX=.UK.'
-e 'TELLY_FILTER_REGEX_MODE=TRUE'
tombowditch/telly --web.base-address=<IP_ADDRESS>:6077
will flip that and strip everything EXCEPT those that match the regex.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tellytv/telly/issues/126#issuecomment-414058901 , or mute the thread https://github.com/notifications/unsubscribe-auth/AlfzNtUvTOFoYQ54lUVvOD3JvuOMOlnLks5uSBoFgaJpZM4U_lbU .  https://github.com/notifications/beacon/AlfzNj_RzKEoeAjgQ5i2443agrV1N2Fjks5uSBoFgaJpZM4U_lbU.gif


Make use of ChangeNotifier

Ensure that registered listeners on the BottomBarController are called.
_bottomBarController = BottomBarController(
      vsync: this,
      snap: true,
      dragLength: 200,
    )..addListener(() => print('This fires up anytime ButtomBar is opened or closed '));

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Travis jammy

Python 3.6 is after end of life. A new Ubuntu 22.04 LTS Jammy Jellyfish is an option on Travis.
I test frequently also with Python 3.11 beta in tox locally.

FYI, you can test in travis yourself if you just push your development branch to this repo, since you're an admin.
If you follow your normal workflow, you can just add another remote that points here, and push it after you make the PR. The UI is smart enough to just look at the most recent build.
An alternative would be pushing your branch here before creating the PR from that directly.

Thanks. I tried it. That is very useful.


Core DB missing from Delivery Server on Sitecore 9.1

With the change to federeated authentication, delivery server by default don't have a <database id="core"...> tag anymore (or - more specifically, it's disabled by role in the Sitecore.config file). This leads to an error with the package deployer.
Are there any plans to change this?

Hi Michael,
It's something I'm going to be looking into.
Thanks
Charlie


[Do not merge] CBCC - Contributors Notifications

@artoodeeto @ertankara @byronluk @abavish
Gitter is way too unreliable for notifications, so I made this PR as a place for us to notify and discuss (I'm still checking Gitter periodically, though).  I will mark this PR invalid, obviously 😄
I will use this original post to write down our to-do list and update it frequently.

To-dos:

[ ] Something... (assigned to @huyenltnguyen)
[ ] Something... (assigned to @artoodeeto)
[ ] ...

So how is everyone? Have you successfully setup fCC locally? Here is the guide, if you need it: https://github.com/freeCodeCamp/freeCodeCamp/blob/master/docs/how-to-setup-freecodecamp-locally.md

@huyenltnguyen should i pull from the next branch?


Update Stake Pool Operations Doc

I've updated the docs for v1.25.1. I've also included a commit in preparation for Mary :)
With regards to calculating the current epoch, I've included an EKG query as slotNo / epochLength does not produce the correct epoch.
If you need any updates, or removal of the additional commits, please let me know.

Thanks, subsumed by: https://github.com/input-output-hk/cardano-node/pull/2435


Incompatibility with EXAoperation 6.0?

I just installed and configured this for all of our clusters.
Among them is a single cluster that's already on version 6.0.
It's also the only cluster that hosts more than one database.
The "Free DB space" service check fails for both databases on this cluster, with the ominous error:
WARNING - internal error
<Fault -1: 'Volume could not be found'>

Not sure if that's because it's version 6 or because there's two databases.
The database performance data is gathered correctly, so I'm leaning towards v6.
The volumes shown in EXAoperation under "EXASolution" for each database match what is shown under "EXAStorage" and the monitoring user has read-only access to those volumes and has the supervisor role.
(Is this the right venue to report issues with this?)

Did you change the name of the system tag of your data volume (like adding a prefix)? We tried to reproduce the problem and it only occurs if the system given tag has been changed.
When you set up the permissions on your data and archive volumes in EXAoperation you don't need to set it on temporary volumes too. Those volumes will inherit their permission from their data volumes. If you try to change the permission on temporary volumes you will encounter strange error messages.

Ahh, that did the trick.
I had indeed changed the EXAstorage labels from test_persistent to test_persistent_v6 during the upgrade to version 6.0, because as of now, EXAoperation has no way of showing which volumes use the new HBD on-disk storage format and which don't.
So I assigned appropriate names to avoid accidentally deleting the new volumes instead of the old ones, after the upgrade process was finished.
I renamed them back to test_persistent and ci_persistent, respectively, and Nagios recognized them correctly on the next service check.
I guess parsing text modifiable by the end user to determine storage volume type isn't perfectly robust. ;-)
(Must be on the XML-RPC side of things within EXAoperation, the Nagios check only calls https://github.com/EXASOL/nagios-monitoring/blob/master/opt/exasol/monitoring/check_db_diskspace.py#L116 and goes from there.)


Psalm level 6 4

Applies 4 more rules:

InvalidPropertyFetch
InvalidReturnStatement
InvalidTemplateParam
NullArgument

@carlos-granados I think things might be easier to track if we were enabling the level 6 and the config and disabling its rules (and then gradually removing the disabling, until we have the full level 6) instead of having to check when the manual enabling of rules matches the config of that level. With disabling, knowing when we are done is easy: the list of disabled rules is empty.

@carlos-granados I think things might be easier to track if we were enabling the level 6 and the config and disabling its rules (and then gradually removing the disabling, until we have the full level 6) instead of having to check when the manual enabling of rules matches the config of that level. With disabling, knowing when we are done is easy: the list of disabled rules is empty.

Can do that but there is only one step left after this one to reach level 6, do you think it is worth it?


Role text covering input text.

Similar to issue #66. This was fixed by #133 which was merged as 03aa617 but came back recently, input window now has (USER) text on the right side of the window covering input that's long enough.
Screenshot:

Steps to reproduce:

:ChatGPT
press and hold a

same here

same here


Add Faker::Name.middle_name

Adding middle_name for Faker::Name.
Returning first_name only as usually middle name is first_name of parent only.

Due to inactive we're closing this PR.


Small updates

This PR:

Suppresses hydration warning caused by different timestamps
Upgrades to the latest React alpha release
Adds a Suspense wrapper to fix the render error
Makes sure we run the setRenderComplete function only once

LGTM!


State union should merge Argument validators.

class A1State(State):
    v1 = Argument(is_integer)

class A2State(State):
    v1 = Argument(greater_than_7)

NewState = A1State & A2State

The v1 argument in the NewState would be both an integer and > 7.
Checks:

success
validation error in one condition (error message shows in what state it was defined)
in composition of thee layers

Superseded by #722


ENH: Add license checks

This adds checks to avoid mirroring agpl or unlicensed conda packages
internally.  cc @parente @mariusvniekerk

Current coverage is 92.92% (diff: 100%)

Merging #7 into master will increase coverage by 2.02%

@@             master         #7   diff @@
==========================================
  Files             2          2
  Lines            77         99    +22
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits             70         92    +22
  Misses            7          7
  Partials          0          0

Powered by Codecov. Last update 7fe5590...74e538f

LGTM


'toggle' is missing in props validation

🐛 Bug Report
Some deps must have added or changed some ESLint rules. In addition to creating a lot of warnings, there is an error that blocks any build pipeline: 'toggle' is missing in props validation react/prop-types.

I added this annotation in the Modal.tsx
/* eslint react/prop-types: 0 */

Is that a rule we need? Otherwise, you can disable it globally with
  "rules": {
    "react/prop-types": 0
  }

For now, I have made this a warning.


Cleaning up the project structure

add .gitignore
update cmake file (add project name, add gmock_main lib)
Add how-to to readme file
Add "Course assignment" to doc directory
fix typo parce->parse
fix typo test->tests
Remove test-code from main function
update unit-testing structure for get possibility run all tests by one command

Good

Good

Good. Small addition, we need to update Cmake requirements into 3.11 version(because of using "FetchContent").

Thanks, updated

One more thing, when I parse test file, int structure stdout_mode always equal false, so something wrong with yaml_parce.cpp


[Feature] Simple Downtimes, a la Crash

Solved by Crash Integration. Don't want to reinvent the wheel.


docs typo

Typo in dl_build.md:

mapped from a file. than For running

@jkarneges open a PR?

@jkarneges Which line are we talking about and what branch? Can't find ... a file. than For ....

https://github.com/coreos/etcd/blob/v3.2.5/Documentation/dl_build.md
Currently live here: https://coreos.com/etcd/docs/latest/dl_build.html
Looks like this was fixed in master. Sorry for the noise!

Just fixed https://github.com/coreos/etcd/blob/release-3.2/Documentation/dl_build.md.
Thanks for point it out!


Update Composer dependencies (2021-01-07-00-11)

Visual regression test passed!
View the visual regression test report

Visual regression test passed!
View the visual regression test report


BAU: Check internal and external links

This PR introduces checking for internal links (within the GDS Way's own content) and external links (content on other sites).
We don't want to have broken links in the GDS Way as this is a bad and confusing user experience. This was highlighted in issue #700.
This PR builds on existing work in https://github.com/alphagov/tech-docs-template/pull/187 and would aim to be folded into the tech docs template eventually.
This PR is also a place for discussion around:

updated content and new external links prompted by the link checker
how to handle rate-limiting by GitHub when checking links to repos, of which there are many in the project

How much do we care about hashes if the target page at least loads?
For instance, we're not catching the fact that this link:
https://github.com/alphagov/gds-way/blob/db16b146f1469d7538497f30845e0a4f5704edf1/source/manuals/programming-languages/html.html.md.erb#L15 has updated to https://github.com/alphagov/govuk-frontend#browser-and-assistive-technology-support
Looks like we could enable check_external_hash (and check_internal_hash?) to catch these.

@domoscargin That's a very good shout.
According to the HTMLProofer docs, the default value for both of those flags is true, however I will add them to the configuration for the sake of explicitness and not relying on default behaviour that may change.


startCollapsed does not work if I have more than one aCollapTable in a page

If I have more than one aCollapTable in a page and set it to be startCollapsed. When tested with 2 table, only the last table is collapsed.

Have you tried to use id instead of class?
$('#table-id').aCollapTable();

I am using id currently. I have to set the first one with startCollapsed: false and the second one with startCollapsed: true

@ktllo Please try with version 1.0.4


[fetch-configlet] update script to match latest version from upstream

Review requested by @ErikSchierboom or @iHiD as @exercism/erlang is effectively just me, and I can not review myself.

Just realising that I can not do a merge commit, I do not mind squashing this commit, as its only a single commit anyway, though I'd prefer to have a merge commit for future merges. Can you please re-enable "merge commits" and "rebase and merge"?

Done.


Error occurred when executing DZ_Face_Detailer

Error occurred when executing DZ_Face_Detailer:
OpenCV(4.7.0) D:\a\opencv-python\opencv-python\opencv\modules\imgproc\src\morph.dispatch.cpp:1010: error: (-215:Assertion failed) !src.empty() in function 'cv::erode'
File "C:\Users\Daedar\Documents\GitHub\AI\webui\extensions\sd-webui-comfyui\ComfyUI\execution.py", line 144, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "C:\Users\Daedar\Documents\GitHub\AI\webui\extensions\sd-webui-comfyui\ComfyUI\execution.py", line 74, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "C:\Users\Daedar\Documents\GitHub\AI\webui\extensions\sd-webui-comfyui\ComfyUI\execution.py", line 67, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "C:\Users\Daedar\Documents\GitHub\AI\webui\extensions\sd-webui-comfyui\ComfyUI\custom_nodes\facedetailer\FaceDetailer.py", line 71, in detailer
final_mask = dilate_mask(final_mask, dilate_mask_value)
File "C:\Users\Daedar\Documents\GitHub\AI\webui\extensions\sd-webui-comfyui\ComfyUI\custom_nodes\facedetailer\FaceDetailer.py", line 184, in dilate_mask
eroded_mask = cv2.erode(mask, kernel, iterations=1)

are you using comfyui inside auto11111?

I'm getting the same error when running it with IPAdapterPlus Face SD 1.5 as the IPAdapter model.

Are you using only comfyui or comfyui extension for auto1111?

Are you using only comfyui or comfyui extension for auto1111?

I never heard about any ComfyUI extension for A1111? It's just ComfyUI.

It turns out the problem has nothing to do with IPAdapter. It just happens in general depending on if the image itself is large or has a face that takes up most of the frame.

The other facedetailer has the same issue. But the advantage this one seems to have is it works entirely in latent space which means when it works it tends to work better. If you could fix this problem & give it feature parity with the other facedetailer. This project would be able to easily stand by itself. But judging by how you haven't looked into the problem much beyond asking a unrelated question doesn't give me or anyone else who has this issue much confidence in the project itself.

Sorry for the confusion, I thought you were using the extension that allows you to use ComfyUI inside Automatic1111 UI, I am currently researching about IPAdapter and looking for the solution to the problem, thank you very much for informing me about it. in first place the problems seems to be on erode-dilate functions.

The issue isn't related to IPAdapter. I disabled it & had the same problem. There's also this problem when trying to convert the mask to an image after disabling the erode-dilate functions.

Yes you're right, i tested IPAdater and is working well, but now i'm getting the same error as you in google colab except on my local pc. i'm working to find the error, is weird because the latent shape is correct when I debug the output of the mask.

Yes you're right, i tested IPAdater and is working well, but now i'm getting the same error as you in google colab except on my local pc. i'm working to find the error, is weird because the latent shape is correct when I debug the output of the mask.

I don't use Google Colab. This is my local system.

I switched to linux and discovered that the problem was in the routes. In windows it worked because of the way it handles routes, but in linux it is different.
I didn't configure the try-exception properly and that's why it threw the error "list index out of range" error which was not accurate because of that, the original error is the one in the image below. Now it works.

Please remove all folders created in "models" called "DZ-FaceDetailer", update the node and run comfy again.

I've the same error described before. I'm using comfyui extension. The solution is the same? txs!


Can't correctly embed font name

I realise that this issue has already been solved through the use of -regexprep etc. but unfortunately I can't quite figure it out for my case.
I am using set(gca,'fontname','Libertinus Sans'), which I later like to edit in Illustrator, which sees the embedded font name as "LibertinusSans". How do I ensure that the space is included when export_fig produces the file?

You can try export_fig(..., '-regexprep', 'LibertinusSans', 'Libertinus Sans')
This is untested so use at your own risk...


Handle object destructing

Current coverage is 100.00%

Merging #25 into master will increase coverage by +0.25% as of 36d170f

@@            master     #25   diff @@
======================================
  Files           80      80
  Stmts          821     822     +1
  Branches       104     105     +1
  Methods          0       0
======================================
+ Hit            819     822     +3
  Partial          0       0
+ Missed           2       0     -2

Review entire Coverage Diff as of 36d170f

Powered by Codecov. Updated on successful CI builds.


Update deletecontact diagram

Codecov Report

Merging #451 (f624c2f) into master (1ef0cb3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #451   +/-   ##
=========================================
  Coverage     79.89%   79.89%
  Complexity     1069     1069
=========================================
  Files           136      136
  Lines          3208     3208
  Branches        430      430
=========================================
  Hits           2563     2563
  Misses          573      573
  Partials         72       72

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 1ef0cb3...f624c2f. Read the comment docs.


Performance: draw all lines in a single buffer geometry

Is your feature request related to a problem? Please describe.
drawing graphs with many links has poor performance on constrained devices
Describe the solution you'd like
option to render all lines as part of a single buffer geometry
Describe alternatives you've considered
not drawing the lines at all :crying_cat_face:
Additional context
example of many line segments in a buffer geometry
each is 1500 line segments
https://github.com/mrdoob/three.js/blob/master/examples/webgl_lines_sphere.html#L112-L128
primary challenge seems to be the expectation of 1 link matches 1 three object

heres the extracted BufferGeometry usage
function createGeometry () {
  const geometry = new THREE.BufferGeometry();
  const vertices = [];
  // populate vertices
  // start
  // vertices.push( vertex.x, vertex.y, vertex.z );
  // end
  // vertices.push( vertex.x, vertex.y, vertex.z );

  geometry.setAttribute( 'position', new THREE.Float32BufferAttribute( vertices, 3 ) );
  return geometry;
}

const material = new THREE.LineBasicMaterial({
  // color: p[ 1 ],
  // opacity: p[ 2 ]
});
const line = new THREE.LineSegments( createGeometry(), material );
scene.add( line );

// update geometry after some change
line.geometry.dispose();
line.geometry = createGeometry();

ok got it working. performance improvement is significant!
    const graph = new ThreeForceGraph()
      .graphData(packageData)
      .linkThreeObject((link) => {
        // create dummy object
        return new Object3D()
      })
      .linkPositionUpdate((linkObject, { start, end }, link) => {
        lineController.setLine(link.id, [start.x, start.y, start.z], [end.x, end.y, end.z])
        // override link position update
        return true
      })

LineController.js
import { LineSegments, LineBasicMaterial, BufferGeometry, Float32BufferAttribute } from 'three'

export class LineSegmentsController {
  constructor ({ lineCapacity = 1000, color = '#f0f0f0', opacity = 0.2 } = {}) {
    this.lineToIndex = new Map()
    this.material = new LineBasicMaterial({
      color,
      opacity,
      transparent: opacity < 1,
      depthWrite: opacity >= 1
    })
    this.geometry = new BufferGeometry()
    this.lineCapacity = 0
    this.vertices = new Float32Array(0)
    this.setLineCapacity(lineCapacity)
    this.object = new LineSegments(this.geometry, this.material)
    this.object.renderOrder = 10
  }

  setLine (id, start, end) {
    if (this.lineToIndex.has(id)) {
      const positions = this.geometry.attributes.position.array
      // existing line, update entry
      const index = this.lineToIndex.get(id)
      positions.set(start, index)
      positions.set(end, index + 3)
    } else {
      // new line, add new entry
      const lineCount = this.lineToIndex.size
      const index = lineCount * 2 * 3
      this.lineToIndex.set(id, index)
      // double line capacity if expired
      if (lineCount >= this.lineCapacity) {
        this.setLineCapacity(this.lineCapacity * 2)
      }
      const positions = this.geometry.attributes.position.array
      positions.set(start, index)
      positions.set(end, index + 3)
      this.geometry.setDrawRange(0, (lineCount + 1) * 2)
    }
    // mark for update
    this.geometry.attributes.position.needsUpdate = true
    // this.geometry.computeBoundingBox()
    // this.geometry.computeBoundingSphere()
  }

  setLineCapacity (newLineCapacity) {
    // lineCapacity * 2 point/line * 3 floats/point
    const newVertices = new Float32Array(newLineCapacity * 2 * 3)
    // only run this when its not the first time
    if (this.lineCapacity > 0) {
      const positions = this.geometry.attributes.position.array
      newVertices.set(positions)
    }
    this.vertices = newVertices
    this.geometry.setAttribute('position', new Float32BufferAttribute(this.vertices, 3))
    this.lineCapacity = newLineCapacity
  }
}

with this change alone i went 3 fps -> 15 fps on quest 2 vr headset


defaults lookup before falling back on default ami based on platform

Description
This addresses the missing lookup to provided worker defaults before falling back on platform based ami default.
#754

Thanks @ck3mp3r for your PR.
Please update the changelog ?

Just added some info on the changelog.

Great.  @ck3mp3r  can you update your branch please so I can merge your PR ?

Great. @ck3mp3r can you update your branch please so I can merge your PR ?

Done :)


zos call would not list minimal proxies

Is it by design? Still I would want to call method even on minimal proxies.

I believe it is by design, as it's explicitly set in the code. @spalladino may know better though. Is this is a bug, it's super easy to fix.

I'm afraid it's a bug, zos call and zos send-tx should both accept minimal proxies as well. There is no reason not to.


Gotftid/promises numeric n

What happened?
Both examples in the promises folder throw the same error:
invalid 'n' - must be numeric, possibly NA.

Solution
I suggest to make number of rows explicitly numeric to fix this - also won't hurt even if it works as is on some other setup.
Apart from that I added library(shiny) calls to make examples self-contained (looks like these lines were initially there but were removed by accident).
Session info
R version 4.3.0 (2023-04-21)
Platform: aarch64-apple-darwin22.4.0 (64-bit)
Running under: macOS Ventura 13.4.1

Matrix products: default
BLAS:   /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /opt/homebrew/Cellar/r/4.3.0_1/lib/R/lib/libRlapack.dylib;  LAPACK version 3.11.0

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

time zone: Europe/Belgrade
tzcode source: internal

attached base packages:
[1] stats     graphics  grDevices datasets  utils     methods   base

other attached packages:
[1] shiny_1.7.4      future_1.32.0    promises_<IP_ADDRESS>

loaded via a namespace (and not attached):
 [1] crayon_1.5.2      cli_3.6.1         rlang_1.1.1       renv_0.17.3       jsonlite_1.8.5    xtable_1.8-4      listenv_0.9.0
 [8] htmltools_0.5.5   httpuv_1.6.11     sass_0.4.6        jquerylib_0.1.4   ellipsis_0.3.2    fontawesome_0.5.1 fastmap_1.1.1
[15] lifecycle_1.0.3   memoise_2.0.1     compiler_4.3.0    codetools_0.2-19  Rcpp_1.0.10       later_1.3.1       digest_0.6.31
[22] R6_2.5.1          parallelly_1.36.0 parallel_4.3.0    magrittr_2.0.3    bslib_0.5.0       tools_4.3.0       mime_0.12
[29] globals_0.16.2    cachem_1.0.8

Thanks for reporting this @Gotfrid! I wonder how input$rows can end up as non-numeric though? What package or system difference could cause that? 🤔
I agree that forcing it to be numeric won't hurt anyway


FIPS module init

Placeholder until we get more clarity from FIPS certifiability assessment

Confirming my access to post comments on this repo.


New LSP-powered version does not activate when csharp.inlayHints.* settings are present in a VSCode profile

Environment data
dotnet --info output: SDK 7.0.302
VS Code version: 1.78.2 (user setup)
C# Extension version: 2.0.206 (pre-release)
The Extension cannot be activated when installed into a non-default profile:
2023-06-07 08:08:42.744 [error] Activating extension ms-dotnettools.csharp failed due to an error:
2023-06-07 08:08:42.744 [error] TypeError: Cannot read properties of undefined (reading 'scope')
    at z.Hb (vscode-file://vscode-app/c:/Users/Marek/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2266:73376)
    at async Promise.all (index 0)
    at async Object.q [as settled] (vscode-file://vscode-app/c:/Users/Marek/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:97:21381)
    at async z.updateValue (vscode-file://vscode-app/c:/Users/Marek/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2266:61461)

When I switch my active profile to Default, the extension works correctly.

After further investigation, it seems that this only happens when I try to use the extension on a profile that existed before the pre-release version was released.
This led me to review the VSCode settings stored in the affected profile. After removing all the csharp.* and omnisharp.* settings form the JSON file, the extension loaded correctly in the profile as well.
So I delved further, and I think the culprit is these settings: csharp.inlayHints.*. I had a whole bunch of them set up. When the new extension loads, it seems to attempt to migrate these old settings to a new name dotnet.inlayHints.*. Unfortunately, the process seems to crash after migrating exactly 1 setting from the namespace, which causes the extension to not load properly.
After I migrated all the old settings (by repeatedly reloading the VSCode window), I ended up with:

At this point, the extension loaded correctly on first try.

Tried to repro this locally and it happens when executing
https://github.com/dotnet/vscode-csharp/blob/ea41fa6d43349a4bd1404f4d8e3d68ee77abea32/src/shared/MigrateOptions.ts#L55
(remove the old settings from the options)
I see the old configuration is removed. But it throws an exception and interrupts the start-up of the extension.
It feels like either a misuse of the vs code API on our side/bug in vscode.

I hope his get solved soon :)

This turns out to be a VSCode bug, and they fixed it in https://github.com/microsoft/vscode/pull/186975


Add CHANGELOG

Please, a CHANGELOG would be very useful.

@mondeja I put it in Releases.
https://github.com/jaywcjlove/hotkeys/releases

Ah thanks!


Continuous build adds significant overhead

Steps to reproduce:

Clone https://github.com/alexey-tsvetkov/gradle-continuous-build-overhead
Run ./gradlew clean assemble. On my computer it finishes in 5-8 seconds depending on whether the daemon is hot or not.
Change any Kotlin file. For example change project_7/src/main/kotlin/utils_14.kt from:

fun util_14() = 14

to:
fun util_14() = 7

Run ./gradlew assemble. On my computer it finishes in 1 second.
Run ./gradlew assemble -t and change the same file again.

I'm seeing ~40-50 seconds (!) builds for any small change.
According to --profile completely up-to-date projects somehow take 5-6s to run:
reports.zip
Non continuous build scans:

Clean assemble build scan https://scans.gradle.com/s/bymn5u56553ig
Incremental assemble build scan https://scans.gradle.com/s/gighdoyy3uj76

The issue is in the Kotlin plugin, more specifically in inspectClassesForKotlinIC. Instead of using a class directory as an input, it is given each class individually. I.e. instead of having build/classes/kotlin/main as an input, it instead has build/classes/kotlin/main/Util1.class, build/classes/kotlin/main/Util2.class, build/classes/kotlin/main/Util3.class. This leads to an explosion of registered watchpoints the more classes you add to a Kotlin project.
It should use SourceSet#getClassesDirs instead of trying to filter (and flattening) SourceSet#getOutput.
I'm sure we could make watchpoint calculation more efficient, but fixing the Kotlin plugin will have a much bigger impact.

I think I've found a simple fix on our end, but I still encourage you to improve the modeling in the Kotlin plugin. Using file trees instead of flat lists will be more efficient for many other purposes as well.

@oehme thanks for looking into the issue!
We will fix the problem on our side
https://youtrack.jetbrains.com/issue/KT-26208


Do a better job of maintaining this repo

I apologize to everyone for my lack of attention on this repo.  Personal life/work/etc interferes.
So I would like to know who would wish to be a collaborator on this repo?  The following users have been active in the Issues or have submitted PRs in the recent past:  @skyl, @Scribilicious , @ThierryDeruyttere, @TalhaAsmal , @cdgriffith
If you are interested please reply in this issue.  Others not mentioned above are also welcome.

Also interested in making contributions.

Hi Eric
I have a kinded of busy schedule too , but im interressed in the topic, id
be glad to help you on lower importance topics
Simon
Le 2 nov. 2017 13:43, "Eric Somdahl"<EMAIL_ADDRESS>a écrit :

I apologize to everyone for my lack of attention on this repo. Personal
life/work/etc interferes.
So I would like to know who would wish to be a collaborator on this repo?
The following users have been active in the Issues or have submitted PRs in
the recent past: @skyl https://github.com/skyl, @Scribilicious
https://github.com/scribilicious , @ThierryDeruyttere
https://github.com/thierryderuyttere, @TalhaAsmal
https://github.com/talhaasmal , @cdgriffith
https://github.com/cdgriffith
If you are interested please reply in this issue. Others not mentioned
above are also welcome.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ericsomdahl/python-bittrex/issues/79, or mute the
thread
https://github.com/notifications/unsubscribe-auth/APFiwJPCHWQd5BrX9mG3CJmzmR9t2A_Gks5sybjbgaJpZM4QPn8D
.

Hi Eric,
I plan to use this API for a script so I would be excited to collaborate, within the limits of my skills.
Sémy

We're kinda' in limbo until Bittrex officially documents their 2.0 API. I'm interested in contributing to this library. But, I'm also hoping that liquidity on decentralized exchanges improves ;)

Hi Eric,
I'm using this library extensively in my projects, would be happy to help and contribute to the library, if it's possible within my skills.

Hi Everyone,
I'm happy to continue helping as necessary.

@skyl Any news on when 2.0 might be released officially?

I'd be interested in making this async.

I'm closing this issue for now -- I am happy to report that @skyl and @TalhaAsmal have come on board as collaborators.
The community around this API is growing and I am hoping that we can do some fun and interesting (and profitable?) things together


feat: analytics stories frame

This pr does the following:

Adds featureFlag for access to analytics page
Adds a paginated frame for published story cards

To test:

Add ?experimentalAnalyticsPage=true to the base URL
Create a story and publish
Navigate to the Analytics tab and see your published stories 🚀

Questions for @pradel :

regarding the line here, I'm just wondering what your thoughts are on if i was to memoize the value here with useMemo? just thinking about if the user has a lot of articles. Or do you feel it doesn't matter too much?

General notes:

The width can be ignored for now as it should be corrected when adding the referrers alongside it :)

Closes #450

@gregogun can you merge main into the branch to fix the CI ? :)


Normalize Path Names in Sourcekit Diagnostics

Always attempt to resolve/normalize the file name so that we only
produce a diagnostic for the resolved path rather than both the given
and resolved paths.
This mostly applies on Windows where a path like C:\my\regular\path gets normalized to \\?\C:\my\regular\path and a diagnostic gets produced for both of them.

Closing in favor of https://github.com/apple/swift/pull/26702


[BUG] S3BlobStoreRepositoryTests.testRequestStats is flaky

https://build.ci.opensearch.org/job/gradle-check/28343/#showFailuresLink

Have a reproducible seed here https://build.ci.opensearch.org/job/gradle-check/28369/testReport/junit/org.opensearch.repositories.s3/S3BlobStoreRepositoryTests/testRequestStats/


⚠ ClamAV has degraded performance

In d4d178d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 227 ms

Resolved: ClamAV performance has improved in 3049f39.


Rename apps

See: #1509
Please review. I rebase all commits into one, when all is done

OK.

You can rebase.


how about the compatibility?

I find the image as below in main page. is it right?

Looks like the build is failing for credentials to saucelabs, which is the source of the compatibility chart... they're all red because of this.


Quake (2310) uses wrong basedir by default

When using luxtorpeda to play the original Quake it tries to use the id1 folder in the game folder, but Steam installs this folder as Id1, with an uppercase I, and that means any filesystem that is case-dependent would cause it to instantly crash. Just renaming the folder manually is enough to make it work.

Also issues related to engines should be created in this repository.

I've made the changes in these commits. Next time you launch it should work if it's Id1 or id1.
https://github.com/luxtorpeda-dev/packages/commit/19396e49d7886f256fdf6276dab9043994fa938b
https://github.com/luxtorpeda-dev/packages/commit/8f01720b7c8b701bc547edb9610a4a230a2ddcf4
https://github.com/luxtorpeda-dev/packages/commit/f9333b6e5deae97a9769f372dbd11ffa38cf56e8
https://github.com/luxtorpeda-dev/packages/commit/d23e11389d19cd8a0be9f8a027bc332cfd79e044


Fix a bug where goros spawned by a job might run forever after the job has been canceled

Good call with this, LGTM.


Roll engine 74975a4d1716..0602dbb27547 (1 commits)

https://github.com/flutter/engine/compare/74975a4d1716...0602dbb27547
git log 74975a4d17162f23bea92ac298d1b96c2995a529..0602dbb27547cd03d8f8e816b7995cace854f7e6 --no-merges --oneline
0602dbb27 Roll src/third_party/dart 9dcb026b26..6e0d978505 (72 commits)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (bmparr@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Add s390x provisions to Dockerfile

The alpine image has progressed significantly since I last looked into running the apigateway image on System Z.  Now all logic can be executed within the Docker image.  Two major differences in the resulting image:

LuaJIT is pre-built from a slightly different version pre-created for LInux on Z (see https://github.com/linux-on-ibm-z/docs/wiki/Building-LuaJIT ).
PCRE JIT is disabled because it doesn't support the s390x processors.  Looking into fixing that, but for now we'll accept the performance hit.

-J
(P.S.  I have an Apache CLA.)

Looks like this needs #268 before being mergeable. Otherwise, looks good!

Yep.  Doing the merge and re-commit now.  Hopefully that gives us a clean build.

I think it's ready for final review and merge now.


Program with multiple clauses and guards fails to compile

data Color = Red | Blue | Green | Color Int Int Int

equalsColor :: Color -> Color -> Bool
equalsColor Red Red = True
equalsColor Blue Blue = True
equalsColor Green Green = True
equlasColor (Color r1 g1 b1) (Color r2 g2 b2)
  | r1 == r2 && g1 == g2 && b1 == b2 = True
equalsColor _ _ = False

This program fails to compile with the "Multiple declarations with names differing only in case." error. @pparkkin That looks like a renamer error you introduced - can you take a look?
For some reason, the compiler thinks the first couple clauses and the last clause are distinct.

I'll take a look.

There's a typo in the (Color ...) (Color ...) line. It says equlasColor.

The error message "names differing only in case" shouldn't really be there, though. I get the regular "Multiple declarations" error too, which is the correct one here.


Validation issue with 567 753 358 (PL)

Please read the "guidelines for contributing" (linked above) and fill in the template below.
Country/region affected (e.g., "US"): PL
Example number(s) affected ("+1<PHONE_NUMBER>"): 567 753 358
The phone number range(s) to which the issue applies ("+1 555 555-XXXX"):
The type of the number(s) ("fixed-line", "mobile", "short code", etc.):
The cost, if applicable ("toll-free", "premium rate", "shared cost"):
Supporting evidence (for example, national numbering plan, announcement from mobile carrier, news article): IMPORTANT - anything posted here is made public. Read the guidelines first!
link to demo

Duplicate of https://github.com/googlei18n/libphonenumber/issues/1317
Please take care not to file exisiting issues :)


[UX] Pane with tabs don't really need a panel bar

Spyder's pane can be improved to take less space.

For exemple pysharm pane don't add a pane bar:

Versions

Spyder version: 3.6
Operating System name/version:  MacOS

@ccordoba12 related to https://github.com/spyder-ide/spyder/issues/2855

Sure, we can say this is a duplicate of #2855.
@jnsebgosselin, do you plan to work on that one eventually?

Yes... but I really don't know when. If someone else could do it, like Daniel for example, that would be great. He already did the work to undock the plugins in new window, so it would be a continuity to his work...
Otherwise it will have to wait for Beta3 I think, after I finish the multi-cursor extension for the editor. This one is hard, so I don't know how long it will take and I want to help you guy make the Beta2 more stable with smaller PR that fix specific issues.

Don't worry, for beta3 is fine. Unfortunately Daniel is quite busy right now implementing an ipdb kernel for our consoles.

This is all good then. It is going to be done eventually then :D

No problem, I see it as a friendly reminder that this needs to be done... :)


Allow anything that implements the DateTimeInterface

I'm using immutable dates in my application (https://laravel-news.com/carbon-2-laravel), so the Stats page doesn't load (the API returns a 500 error).
This will allow anyone that is using any class that implements the DateTimeInterface as their default to use Canvas without errors.

As an aside, I would suggest not using Carbon directly. Laravel's Date facade wraps whichever version of DateTimeInterface the developer wants so it will be consistent throughout. I only see (I think) two more places where Carbon is used instead of Date: \Canvas\Post and \Canvas\Http\Middleware\Session. I could either add those to this PR or do a separate PR for them.

Thanks for the proposed fix @mikemand! Good catch on this.
If you could go ahead and add this change both to the Session middleware and the Post class, I'll check it out and get it merged in.

Hi @austintoddj,
There you go. Let me know if you have any changes you need me to do. 😃


pull request for e2e test at 01/08/2017 19:15:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Error Domain=jetsam Code=2 "memory-highwater(2)"

This error coming on flutter iOS side while doing widget capture and social share at a time.

I upload the new version to solve this pugs.


wrong time if change device time zone

Is there any solution if change time zone by user?

@bhanuka96 which version are you using?

2.0.0 version. return time also change if change time zone from the device

I am also the same. I tried it on a real device.

Same issue here. Changing the timezone on the emulator/ device somehow changes the ntp time. The result is that "now = await NTP.now();" returns local time instead of utc.

Same here.
I see in ntp.dart that the local time is used to make the query request.
Someone could add a way to use the NTP without local time and ask for a pull request?
I will try my own and let you know

For those like me who need a specific time that's not depending on the user device, you can just use this free api : https://worldtimeapi.org/pages/faqs#future-plans
Example of use :
`    final Response response = await get(Uri.http("worldtimeapi.org", "/api/timezone/Europe/Paris"));
final Map<String, dynamic> dataWorldTimeApi = jsonDecode(response.body);
  parisTimeNow = DateTime.parse(dataWorldTimeApi["datetime"].toString().replaceFirst("+02:00", ""));`

Anybody got a fix for this?

You simply have to convert the NTP to UTC. This will always return the same time regardless of the timezone.
final now = (await NTP.now()).toUtc()


Can you use docker-compose on GCE?

I saw Docker has some GCE-specific docs, but didn't dig into this further;
If you look at our Azure instructions they are now much much easier than the GCE instructions, thanks to the use of Docker-compose. Would that be possible for the GCE instructions as well?

Hi @michielbdejong, thanks for creating this issue - when I initially created these instructions for Google Cloud, the "use a Docker image in GCE" was a beta feature that required special access from Google. I think that feature has gone GA now, so we should be able to make this guide much simpler. When I get some free time, I'll investigate more.


:infinity: Fix CI

Signed-off-by: Julio Jimenez<EMAIL_ADDRESS>Fixes #
Proposed Changes

Fix CI

The problem is in this line:
:perform (test-op (o c) (symbol-call :fiveam '#:run! 'clickhouse-test::all-tests)))

of clickhouse-test.asd.
At them moment, when lisp loads the asd file, there is no clickhouse-test package yet. That is why reader complains the package is not found.
Try to replace 'clickhouse-test::all-tests with (find-symbol "ALL-TESTS" "CLICKHOUSE-TEST").

The problem is in this line:
:perform (test-op (o c) (symbol-call :fiveam '#:run! 'clickhouse-test::all-tests)))

of clickhouse-test.asd.
At them moment, when lisp loads the asd file, there is no clickhouse-test package yet. That is why reader complains the package is not found.
Try to replace 'clickhouse-test::all-tests with (find-symbol "ALL-TESTS" "CLICKHOUSE-TEST").

You must've commented on this around the same time I found it on StackOverflow 🤣 Thank you!
I'm still having the issue of System "lexer" not found, I am pretty horrible at Common Lisp systems and packaging 😮💨

I'm still having the issue of System "lexer" not found, I am pretty horrible at Common Lisp systems and packaging 😮💨

I see all checks are passing now. What is the problem with lexer?

I see all checks are passing now. What is the problem with lexer?

Locally the tests run smoothly, but the CI workflow fails.


Checking if audio is paused/playing?

Hello
I searched the struct methods and didn't find something that tells me if a source/generator are paused, I need this for something, Is this not added yet/not planned to be added, Or did I mis it somewhere?

It's not added to Synthizer itself on purpose.  Like with reading properties, it is not possible to do this accurately and efficiently, because the underlying architecture has a bunch of queues.  You need to track that yourself with an additional boolean somewhere.
Do not use Synthizer to store state.  It will not work out for you in any way whatsoever.


feat(segmented-control): add machine

Add segmented control machine

Thanks for working on this @anubra266.
I've just refactored most of the code in your PR and updated it to use a new approach I created in the main branch


Fix errors that were crashing doc build

GH-#115 removed surfer.io.read_label but did not change the example
that was referencing it. I'm surprised that got through Travis--
does Travis not build the docs?
While fixing this I also noticed a couple of things that had gotten
out of date and fixed an oversight from the config file removal.

Travis does not build the docs, no. Feel free to open a PR for that :) It could be another worker condition.

LGTM, +1 for merge once the CIs are happy

Coverage increased (+0.11%) to 72.46% when pulling 8bf48a88561787e031213ab40e66b68fe5b53e39 on mwaskom:fix_docs into 634cd6d02119394c602cf229f39d8356fec94848 on nipy:master.


Can i add logistic regression technique for this.

As @akondas repeatedly told in a few related issues, PHP-ML is open to PRs.
However, LogisticRegression is an algorithm that already exists in the PHP-ML. Are you aware of this or do you propose an improvement on the current implementation? If so, you better elaborate the details before implementing something.


Allow private channels to be used in sendtoroute

Background
Currently sendtoroute takes as an argument a set of hops that assume the channels specified reference channels known to the channel graph.
Completely specifying all the route details including public keys of node hops is not possible, which precludes the ability to use sendtoroute for things like sending to private recipients.
Expected behaviour
Ideally sendtoroute would accept public keys in its arguments and follow crafted routes as specified without needing to consult the channel graph for edge and node lookups.

#1944


Allow for SFX and Music volume and mute controls

Allow for the user to set the sound effects and music levels, with the possibility of muting them.

Closed by d596730a295bf01d672f91cef92d30b9ef0ebf08.


[Request-For-External] Report 594 "Get Item Ledger Entries" - Global Variables

Hi, could you please change the protection level of the IntrastatJnlBatch variable on report 594 (Get Item Ledger Entries) to a protected?
    protected var
        IntrastatJnlBatch: Record "Intrastat Jnl. Batch";

https://github.com/microsoft/ALAppExtensions/issues/15396
Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


NarrativeContent Previous & Next

Add previous and next attributes to allow ordering of NarrativeContent

@BSnoeijerCD @dih-cdisc please see attached the proposed changes to the CT deliverable for this ticket:
DDF 334.xlsx

@EMuhlbradt @czwickl : looks good. No comments.


Changes as suggested for #766

changes as suggested for #766 by @ebfe
remove split of destdir into two params
use ln -f option to clobber old binaries rather than unnecessary rm

Thanks!
However now PREFIX is removed :)
Sorry for being annoying on this one but DESTDIR/PREFIX are common make
variables that are used for specific purposes, and I think we should use them
as such (see for example here for an explanation).
Basically have PREFIX ?= /usr/local and DESTDIR empty and install files into
$(DESTDIR)$(PREFIX).

Okay, just a clarifying question: In the link provided the prefix is /usr (absolute path) and the destdir is debian/tmp (relative path). Should it be $(DESTDIR)$(PREFIX) or $(PREFIX)$(DESTDIR)?

$(DESTDIR)$(PREFIX)

okay, that should be resolved then.

Thank


use websockets for measurements

Feature-request
Description
The measurements for the power consumption and the thermal production of the cpu are requested over the REST-API at the moment. To increase performance, the values should be pushed over the websockets. With this it would also be possible for live-updates of the data. For this there also is a redraw of the d3-output necessary.

moved to https://github.com/kitsudaiki/Hanami-AI/issues/59


Fix ex_data memory leak

Code was added in commit b3c31a6 that overwrote the last ex_data value
using CRYPTO_dup_ex_data() causing a memory leak, and potentially
confusing the ex_data dup() callback.
In ssl_session_dup(), fix error handling (properly reference and up-ref
shared data) and new-up the ex_data before calling CRYPTO_dup_ex_data();
all other structures that dup ex_data have the destination ex_data new'd
before the dup.
Fix up some of the ex_data documentation.
Add another type of ex_data use to the unit tests
Checklist

[X] documentation is added or updated
[X] tests are added or updated

Update unit tests to show two different methods of ex_data use.

Moved the no-dtls to it's own PR #3344

Rebased.

So looks like that issue affects master and 1.1.0?  1.0.2 is safe?

Yes, it was committed to 1.0.2 as commit 62f488d31

Ping?

This also needs to be put into 1.0.2 (at least a version of it)

A much reduced version of this needs to be backported to 1.0.2 (PR #3568), and 1.1.0 also.

@mattcaswell does this look good?

This would need to go into 1.1.0 as well, not sure how clean the cherry-pick would be...

1.0.2 was committed.

thanks for the nudge; merged at 1ee2125

Ping me if you need a 1.1.0 version.