External Database - Faulting module name: KERNELBASE.dll

Description
I want to create a BC 23.3 container with external database on local sql server 2019. The scripts fails when trying to start the bc server instance. When I create the container without an external database it works. I also tried combinations of windows and NAVUserPassword authentication and process and hyperv isolation.
The database is created successfully on my local sql server.
I tried using PS 7.42 and PS 5.1
Scripts used to create container and cause the issue
$licenseFile = "C:\Sources\German Developer License BC24.bclicense"
$auth = "NavUserPassword"
$credential = Get-Credential
$artifactUrl = Get-BcArtifactUrl -country de -version '23.3'
$artifactUrl
$sqlPassword = ConvertTo-SecureString "<password>" -AsPlainText -Force
$sqlUser = 'sa'

$containerName = "bccontainer"

$databaseParams = @{
    "databaseServer" = 'host.containerhelper.internal'
    "databaseInstance" = ''
    "databasePrefix" = "$containerName-"
    "databaseName" = 'alpaca'
    "databaseCredential" = New-Object pscredential $sqlUser, $sqlPassword
    "multitenant" = $false
}

New-BcContainer @databaseParams -replaceExternalDatabases `
    -accept_eula `
    -containerName $containerName `
    -credential $credential `
    -auth $auth `
    -artifactUrl $artifactUrl `
    -licenseFile $licenseFile `
    -updateHosts `
    -alwaysPull `
    -forceRebuild `
    -isolation hyperv

Full output of scripts
New-BcContainer @databaseParams -replaceExternalDatabases `
    -accept_eula `
    -containerName $containerName `
    -credential $credential `
    -auth $auth `
    -artifactUrl $artifactUrl `
    -licenseFile $licenseFile `
    -updateHosts `
    -alwaysPull `
    -forceRebuild `
    -isolation hyperv
BcContainerHelper is version 6.0.20-preview1214
BcContainerHelper is running as administrator
HyperV is Disabled
Host is Microsoft Windows 11 Enterprise - 10.0.22631.3880
UsePsSession is True
UsePwshForBc24 is True
UseWinRmSession is never
UseSslForWinRmSession is True
Docker Client Version is 27.0.3
Docker Server Version is 24.0.5
Removing Session bccontainer
Removing container bccontainer
Removing entries from hosts
Removing bccontainer from container hosts file
Removing bccontainer-* from container hosts file
Removing Desktop shortcuts
Removing C:\ProgramData\BcContainerHelper\Extensions\bccontainer
Setting database bccontainer-alpaca offline
Removing database bccontainer-alpaca
Starting Database Restore job from https://bcartifacts-exdbf9fwegejdqak.b02.azurefd.net/sandbox/23.3.14876.22140/de
C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\6.0.20\Import-BcContainerHelper.ps1
Fetching all docker images
Fetching all docker volumes
Pulling image mcr.microsoft.com/businesscentral:ltsc2022-dev
ltsc2022-dev: Pulling from businesscentral
Digest: sha256:4f18913f58dafe2af1e083627559ed74aabaf919b7b28d10e148cdbaf2ec49c0
Status: Image is up to date for mcr.microsoft.com/businesscentral:ltsc2022-dev
Using image mcr.microsoft.com/businesscentral:ltsc2022-dev
Creating Container bccontainer
Style: sandbox
Multitenant: No
Version: 23.3.14876.22140
Platform: 23.0.22115.0
Generic Tag: <IP_ADDRESS>-dev
Container OS Version: 10.0.20348.2582 (ltsc2022)
Host OS Version: 10.0.22631.3880 (23H2)
Using hyperv isolation
Using locale de-DE
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file C:\Sources\German Developer License BC24.bclicense
Files in C:\ProgramData\BcContainerHelper\Extensions\bccontainer\my:
- AdditionalOutput.ps1
- HelperFunctions.ps1
- license.bclicense
- MainLoop.ps1
- SetupDatabase.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container bccontainer from image mcr.microsoft.com/businesscentral:ltsc2022-dev
b55a1dcea355e327a8b4a594f542d12781aab5761dfbe7f4b3aacf782dd10a28
Waiting for container bccontainer to be ready
Using artifactUrl https://bcartifacts-exdbf9fwegejdqak.b02.azurefd.net/sandbox/23.3.14876.22140/de
Using installer from C:\Run\210-new
Installing Business Central: multitenant=False, installOnly=False, filesOnly=False, includeTestToolkit=False, includeTestLibrariesOnly=False, includeTestFrameworkOnly=False, includePerformanceToolkit=False, appArtifactPath=c:\dl\sandbox\23.3.14876.22140\de, platformArtifactPath=c:\dl\sandbox\23.3.14876.22140\platform, databasePath=, licenseFilePath=c:\dl\sandbox\23.3.14876.22140\de\Cronus.bclicense, rebootContainer=True
Installing from artifacts
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
c:\dl\sandbox\23.3.14876.22140\platform\ServiceTier\Program Files
c:\dl\sandbox\23.3.14876.22140\platform\ServiceTier\System64Folder
Copying PowerShell Scripts
c:\dl\sandbox\23.3.14876.22140\platform\WindowsPowerShellScripts\Cloud\NAVAdministration
c:\dl\sandbox\23.3.14876.22140\platform\WindowsPowerShellScripts\WebSearch
Copying Web Client Files
c:\dl\sandbox\23.3.14876.22140\platform\WebClient\Microsoft Dynamics NAV
Copying ModernDev Files
c:\dl\sandbox\23.3.14876.22140\platform
c:\dl\sandbox\23.3.14876.22140\platform\ModernDev\program files\Microsoft Dynamics NAV
Copying additional files
Copying ConfigurationPackages
C:\dl\sandbox\23.3.14876.22140\de\ConfigurationPackages
Copying Test Assemblies
C:\dl\sandbox\23.3.14876.22140\platform\Test Assemblies
Copying Extensions
C:\dl\sandbox\23.3.14876.22140\de\Extensions
Copying Applications
C:\dl\sandbox\23.3.14876.22140\platform\Applications
Copying Applications.DE
C:\dl\sandbox\23.3.14876.22140\de\Applications.DE
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Skipping restore of Cronus database
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Installation took 17 seconds
Installation complete
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is bccontainer
PublicDnsName is bccontainer
WARNING: Container starts with TimeZone = Mitteleurop├ñische Zeit, which is not recognized in the list of TimeZones.
Using NavUserPassword Authentication
Success
Import Encryption Key
Cannot establish a connection to the SQL Server/Database.

This could be due to one of the following reasons:

* SQL Server is not started.
* The database does not exist.
* Proper permissions have not been given to the NAV Server Account.

Try again later or contact your system administrator.
at <ScriptBlock>, C:\run\setupDatabase.ps1: line 152
at <ScriptBlock>, C:\Run\my\SetupDatabase.ps1: line 2
at <ScriptBlock>, C:\Run\navstart.ps1: line 141
at <ScriptBlock>, C:\Run\start.ps1: line 390
at <ScriptBlock>, <No file>: line 1Error
Using artifactUrl https://bcartifacts-exdbf9fwegejdqak.b02.azurefd.net/sandbox/23.3.14876.22140/de
Using installer from C:\Run\210-new
Installing Business Central: multitenant=False, installOnly=False, filesOnly=False, includeTestToolkit=False, includeTestLibrariesOnly=False, includeTestFrameworkOnly=False, includePerformanceToolkit=False, appArtifactPath=c:\dl\sandbox\23.3.14876.22140\de, platformArtifactPath=c:\dl\sandbox\23.3.14876.22140\platform, databasePath=, licenseFilePath=c:\dl\sandbox\23.3.14876.22140\de\Cronus.bclicense, rebootContainer=True
Installing from artifacts
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
c:\dl\sandbox\23.3.14876.22140\platform\ServiceTier\Program Files
c:\dl\sandbox\23.3.14876.22140\platform\ServiceTier\System64Folder
Copying PowerShell Scripts
c:\dl\sandbox\23.3.14876.22140\platform\WindowsPowerShellScripts\Cloud\NAVAdministration
c:\dl\sandbox\23.3.14876.22140\platform\WindowsPowerShellScripts\WebSearch
Copying Web Client Files
c:\dl\sandbox\23.3.14876.22140\platform\WebClient\Microsoft Dynamics NAV
Copying ModernDev Files
c:\dl\sandbox\23.3.14876.22140\platform
c:\dl\sandbox\23.3.14876.22140\platform\ModernDev\program files\Microsoft Dynamics NAV
Copying additional files
Copying ConfigurationPackages
C:\dl\sandbox\23.3.14876.22140\de\ConfigurationPackages
Copying Test Assemblies
C:\dl\sandbox\23.3.14876.22140\platform\Test Assemblies
Copying Extensions
C:\dl\sandbox\23.3.14876.22140\de\Extensions
Copying Applications
C:\dl\sandbox\23.3.14876.22140\platform\Applications
Copying Applications.DE
C:\dl\sandbox\23.3.14876.22140\de\Applications.DE
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Skipping restore of Cronus database
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Installation took 17 seconds
Installation complete
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is bccontainer
PublicDnsName is bccontainer
WARNING: Container starts with TimeZone = Mitteleurop├ñische Zeit, which is not recognized in the list of TimeZones.
Using NavUserPassword Authentication
Success
Import Encryption Key
Cannot establish a connection to the SQL Server/Database.

This could be due to one of the following reasons:

* SQL Server is not started.
* The database does not exist.
* Proper permissions have not been given to the NAV Server Account.

Try again later or contact your system administrator.
at <ScriptBlock>, C:\run\setupDatabase.ps1: line 152
at <ScriptBlock>, C:\Run\my\SetupDatabase.ps1: line 2
at <ScriptBlock>, C:\Run\navstart.ps1: line 141
at <ScriptBlock>, C:\Run\start.ps1: line 390
at <ScriptBlock>, <No file>: line 1
New-BcContainer Telemetry Correlation Id: 3ebf9c5c-9ed7-4653-9a7b-469de5963a0b
Initialization of container bccontainer failed
At C:\Program
Files\WindowsPowerShell\Modules\BcContainerHelper\6.0.20\ContainerHandling\Wait-NavContainerReady.ps1:42 char:17
+ ...             throw "Initialization of container $containerName failed" ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Initialization ...ontainer failed:String) [], RuntimeException
    + FullyQualifiedErrorId : Initialization of container bccontainer failed

Container Event Log
PS C:\Sources\BGH\BGH_DE\Script> enter-BcContainer bccontainer
[BCCONTAINER]: PS5 C:\Run> get-eventLog -LogName Application -EntryType Error -Newest 10 | format-table -wrap
Exception calling "ReadLine" with "2" argument(s): "Specified method is not supported."
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : NotSupportedException

   Index Time          EntryType   Source                 InstanceID Message
   ----- ----          ---------   ------                 ---------- -------
    6410 Jul 23 11:34  Error       Software Protection<PHONE_NUMBER> License Activation (slui.exe) failed with the
                                   Platform Service                  following error code:
                                                                     hr=0x8007232B
                                                                     Command-line arguments:
                                                                     RuleId=eeba1977-569e-4571-b639-7623d8bfecc0;Actio
                                                                     n=AutoActivate;AppId=55c92734-d682-4d71-983e-d6ec
                                                                     3f16059f;SkuId=ef6cfc9f-8c5d-44ac-9aad-de6a2ea0ae
                                                                     03;NotificationInterval=1440;Trigger=TimerEvent
    6407 Jul 23 11:34  Error       Software Protection<PHONE_NUMBER> Failed to collect hardware data. Error code
                                   Platform Service                  0xD0000034.
    6400 Jul 23 11:33  Error       Application Error            1000 Faulting application name:
                                                                     Microsoft.Dynamics.Nav.Server.exe, version:
                                                                     23.0.22115.0, time stamp: 0x666c0000
                                                                     Faulting module name: KERNELBASE.dll, version:
                                                                     10.0.20348.2582, time stamp: 0xf8072450
                                                                     Exception code: 0xe0434352
                                                                     Fault offset: 0x000000000003f14c
                                                                     Faulting process id: 0x9e8
                                                                     Faulting application start time:
                                                                     0x01dadce35a6de320
                                                                     Faulting application path: C:\Program
                                                                     Files\Microsoft Dynamics
                                                                     NAV\230\Service\Microsoft.Dynamics.Nav.Server.exe
                                                                     Faulting module path:
                                                                     C:\Windows\System32\KERNELBASE.dll
                                                                     Report Id: c48c6e9e-fb15-4152-89f0-62a1155e5ba4
                                                                     Faulting package full name:
                                                                     Faulting package-relative application ID:
    6399 Jul 23 11:33  Error       .NET Runtime                 1026 Application: Microsoft.Dynamics.Nav.Server.exe
                                                                     CoreCLR Version: 6.0.3224.31407
                                                                     .NET Version: 6.0.32
                                                                     Description: The process was terminated due to
                                                                     an unhandled exception.
                                                                     Exception Info: System.NullReferenceException:
                                                                     Object reference not set to an instance of an
                                                                     object.
                                                                        at Microsoft.Dynamics.Nav.Types.ServerUserSett
                                                                     ings.LoadProperties(Object target,
                                                                     SettingsProvider provider)
                                                                        at Microsoft.Dynamics.Nav.Types.ServerUserSett
                                                                     ings.LoadSettings()
                                                                        at Microsoft.Dynamics.Nav.Types.ServerUserSett
                                                                     ings.LoadAndValidateSettings()
                                                                        at Microsoft.Dynamics.Nav.Types.ServerUserSett
                                                                     ings.CreateInstance(SettingsProvider
                                                                     settingsProvider, ConcurrentDictionary`2
                                                                     appSpecificSettings)
                                                                        at Microsoft.Dynamics.Nav.Types.ServerUserSett
                                                                     ings.CreateInstance(SettingsProvider
                                                                     settingsProvider, Boolean forceRefresh)
                                                                        at Microsoft.Dynamics.Nav.Types.ServerUserSett
                                                                     ings.CreateInstance(String fileName)
                                                                        at Microsoft.Dynamics.Nav.Types.ServerUserSett
                                                                     ings.get_Instance()
                                                                        at Microsoft.Dynamics.Nav.Types.DiagnosticsExc
                                                                     eptionsAllowList.SafeGetAllowedExceptionsForTelem
                                                                     etry()
                                                                        at Microsoft.Dynamics.Nav.Types.DiagnosticsExc
                                                                     eptionsAllowList.GenerateAllowedExceptionsSet()
                                                                        at
                                                                     System.Lazy`1.ViaFactory(LazyThreadSafetyMode
                                                                     mode)
                                                                     --- End of stack trace from previous location ---
                                                                        at System.Lazy`1.CreateValue()
                                                                        at System.Lazy`1.get_Value()
                                                                        at Microsoft.Dynamics.Nav.Types.DiagnosticsExc
                                                                     eptionsAllowList.IsExceptionAllowed(Exception
                                                                     exception)
                                                                        at Microsoft.Dynamics.Nav.Diagnostic.NavDiagno
                                                                     stics.ExceptionTelemetryMessages.AppendException(
                                                                     Exception exception, String diagnosticsContext,
                                                                     String callStack, Boolean&
                                                                     shouldEmitTransientTelemetry)
                                                                        at Microsoft.Dynamics.Nav.Diagnostic.NavDiagno
                                                                     stics.ComposeExceptionTelemetryMessages(Exception
                                                                      exception, String diagnosticsContext,
                                                                     DiagnosticParameter message, Boolean&
                                                                     shouldEmitTransientTelemetry)
                                                                        at Microsoft.Dynamics.Nav.Diagnostic.NavDiagno
                                                                     stics.ComposeAndSendExceptionTraceTag(String
                                                                     tag, Category category, Exception exception,
                                                                     DiagnosticParameter message, Verbosity
                                                                     verbosity, Guid clientSessionId, Guid
                                                                     clientActivityId, String gatewayCorrelationId,
                                                                     String diagnosticsContext, Boolean forceLog,
                                                                     ValueTuple`2[] customDimensions)
                                                                        at Microsoft.Dynamics.Nav.Diagnostic.NavDiagno
                                                                     stics.SendExceptionTag(String tag, Category
                                                                     category, Exception exception, Verbosity
                                                                     verbosity, DiagnosticParameter message, String
                                                                     diagnosticsContext, Boolean forceLog,
                                                                     ValueTuple`2[] customDimensions)
                                                                        at Microsoft.Dynamics.Nav.WindowsServices.Dyna
                                                                     micsNavServer.CurrentDomain_UnhandledException(Ob
                                                                     ject sender, UnhandledExceptionEventArgs e)

      35 Jul 22 10:35  Error       Software Protection<PHONE_NUMBER> Failed to collect hardware data. Error code
                                   Platform Service                  0xD0000034.
      30 Jul 22 10:34  Error       Software Protection<PHONE_NUMBER> Failed to collect hardware data. Error code
                                   Platform Service                  0xD0000034.
      19 Jul 22 10:30  Error       Software Protection<PHONE_NUMBER> License Activation (slui.exe) failed with the
                                   Platform Service                  following error code:
                                                                     hr=0x8007232B
                                                                     Command-line arguments:
                                                                     RuleId=eeba1977-569e-4571-b639-7623d8bfecc0;Actio
                                                                     n=AutoActivate;AppId=55c92734-d682-4d71-983e-d6ec
                                                                     3f16059f;SkuId=ef6cfc9f-8c5d-44ac-9aad-de6a2ea0ae
                                                                     03;NotificationInterval=1440;Trigger=TimerEvent
      14 Jul 22 10:30  Error       Software Protection<PHONE_NUMBER> Failed to collect hardware data. Error code
                                   Platform Service                  0xD0000034.

Edit 1: corrected full output of script, becouse it was from a previous attempt with another version

Is that output a result from that script?

vs.

I am sorry. It seems that I copied the output from a previous attempt. I tried using different versions in case 23.3 was corrupted. I got the same error no matter what version I used.

Thank you very much for your fast feedback. I will try it and report back asap.

@freddydk Thank you for your Feedback
I tried it with the following image and it worked and created the Container
-artifactUrl "https://bcartifacts-exdbf9fwegejdqak.b02.azurefd.net/onprem/24.3.21374.21517/ch"
So its a problem with the Sandbox image from Microsoft?

I was finally able to try it and it worked like a charm. Thank you very much for your assistance.

@freddydk so are we supposed to use the onprem artifacts now? this breaks our scripts...
Will there be a proper fix for this?

I don't know what happened here - but could see that this problem persists with version 23 as well as 24.
Due to that, I have to prioritize this lower as a lot of the other things I have.
If somebody can find out why this happens, I am open to review a PR or create a bug on the responsible team - else it will have to wait until I have the time.

I don't know what happened here - but could see that this problem persists with version 23 as well as 24. Due to that, I have to prioritize this lower as a lot of the other things I have. If somebody can find out why this happens, I am open to review a PR or create a bug on the responsible team - else it will have to wait until I have the time.

Hi Freddy - since this ticket was closed, I had created a new one, but am not seeing any responses over there 3605. I have more details in that issue, but the point in the process that I am seeing differences and that usually it creates 3 databases(prefix-default, prefix-CRONUS and prefix-tenant) - it is no longer creating the prefix-tenant database, but still writing the log of 'Success' to the file databasescreated.txt(with the containername in front). Then it goes on to try to start the BC service and fails. I have also tried with a number of versions of bccontainerhelper(and preview versions), even older known working ones cannot create a sandbox container.
This issue has completely diminished our ability to work effectively. I have a team of 8 developers that cannot work on issues related to customer data. We use these bccontainerhelper tool extensively and have automated processes that happen when a ticket comes in - our tools generate a bacpac export, then it creates a docker container for their version of BC and mounts the customers database to the container. Many of these customers have databases greater than 10gb and cannot use SQL Express. Since all of them are on Cloud, they are all on v24, and no version of v24 currently creates a container successfully when using an external sql server.

@kevindstanley - you should respond to the open issue.


Taiga improvements + Bitbucket2 fix

Is the story here that Taiga just totally redid their API?

Taiga changed their API in at least 2 places. Crucial places for us (e.g. owner full name) so it required to update all of the payloads - I decided to update whole payloads because its more safe.

Cool, makes sense.  Let me know when you're ready for a review.

@timabbott Now it is - the same story like with jira (a few commits to squash)

Looks great, merged, thanks @TomaszKolek!


check for networking setup?

I had networking issues on the guest and could not connect to anything. I could not find anything wrong, and tried to set up the networking again, and then it worked.
Problem is, I could find no indication in the getall and other setup list commands of anything wrong.
Please either add a check or some way to see if / how 'net' is set.
The solution is mentioned in https://github.com/pr1ntf/iohyve preflight checklist, but a way to figure out that that is the problem would be helpful.
[root@freenas /iohyve]# iohyve setup net=igb0
bridge0 is already enabled on this machine...
Setting up correct sysctl value...
net.link.tap.up_on_open: 0 -> 1
[root@freenas /iohyve]# iohyve setup net=igb0
bridge0 is already enabled on this machine...
sysctl already setup properly as well...

I don't understand the question? I understand that you were having network issues but you resolved them?

I'm saying I want to use a iohyve status command or something to see if what the value of 'net' is set to. I didn't realize I had forgotten to iohyve_enable="YES" and iohyve_flags="kmod=1 net=em0" so the networking was not set up. At present I see no way to figure this out or check this except for guessing.

A check of ifconfig on the host gives a rundown of all the network interfaces on the host.

Ah OK. The tap0 probably didn't exist when I booted the guest the first time. OK, then would it be possible to change
$ sudo iohyve getall www-bhyve
Getting www-bhyve iohyve properties...
...
tap            tap0
...

...to say something like tap   tap0 (inactive) or something similar if the interface isn't set up as needed? Then that points directly to the problem and saves debugging time.

The interface tap0 is created when the guest is started. Your problem was that your bridge0 interface was not setup properly, or the correct sysctl's were not in place. There are many ways to setup networking, so it's hard to say "your setup is wrong, try xyz..." (Not everyone uses iohyve setup to set networking.) We are going to have to assume that the user has followed instructions and properly setup the networking. I think this is better solved in documentation, rather than adding logic to the actual scripting.

Hm, but I have tap0 set as the tap, shouldn't there be an error when booting that points to tap0 not being created since something is wrong?
If that's not doable / sensible, feel free to close the issue as is, but it would be nice to have some hint of what's wrong. I was playing with the interfaces in the guest, dhclient, and routing in the guest to try and solve it before I looked at the host and tried iohyve setup net=igb0 again.
Thanks for your help!

Again, it's hard to warn for things like this since there are so many different ways to set up networking besides the default iohyve way. In the future, you can debug the networking by simply checking ifconfig and making sure that your interface (igb0) and your tap interfaces (ex: tap0) are members of the bridge (usually bridge0).
Thanks for the insight though!


add link from metallium guide to multichip programming guide

Ticket
Link to Github Issue
Problem description
Provide context for the problem.
What's changed
Describe the approach used to solve the problem.
Summarize the changes made and its impact.
Checklist

[ ] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

Would be good to add the diagram of the Ethernet stack on Eth core
from top to bottom we have 3 levels:

EDM
Ethernet APIs (sending packets, handshaking)
Ethernet transfer command queue APIs (direct interfacing to the transfer command queue register interface)

And then have section for each of these, you already have the text , would be good to have it structured in that way

fyi @cfjchu

annotate Gateway WH that are not user visible
which should annotate our virtual device coordinates , where is (0,0), where is (7,3) (row, col)
text should say 12.5 GB/s , 25 GB/s - -- GBps is close to Gbps and could be mis-read, it's usually used only for Gbps

And then have section for each of these, you already have the text , would be good to have it structured in that way

You're right this was something that I wanted to add earlier but never got around to. I can add it.

Add fused Matmul+AllGather to the OP list

lead with the stack picture EDM/Eth API/Transfer cmd queue

then with the host side API CCL example

then with the list of supported OPs

then micro-bench

then all the impl details

Updating here: https://github.com/tenstorrent/tt-metal/pull/12363
(The PR is separate because it's from a direct in browser edit ).

The CCL doc diagram for TG has been saved for broader use/sharding.
@cfjchu if you need to expand on it or build off it, you can pick it up here
https://docs.google.com/document/d/10QKmbfQVBDN0hbOs3KQ_rmOiOrTJk8RBF7HF3G4NRpM/edit?usp=drive_web


User specified fonts are not applied

Under Atom v0.176.0 at least, the syntax theme enforces a particular font. Any font specified by the user in Settings is ignored.

+1, I am on v0.178.0 on OS X Yosemite.

Love this theme, but this is a show-stopper.

This should now be fixed in v0.21.0 (via https://github.com/jglovier/fizzy/commit/913dac34285ebce7017b13750411430e92f2a212). Sorry for the long delay on getting this fixed. :heart:


Settings issue

http://prntscr.com/7whij1
I still have this issue, any chance someone could help me with it please instead of randomly closing my thread.
Thanks

If you do not know what that means, that you probably shouldn't be messing
with SteamBot.
2015-07-24 17:21 GMT+01:00 coolxlfc289<EMAIL_ADDRESS>
http://prntscr.com/7whij1
I still have this issue, any chance someone could help me with it please
instead of randomly closing my friend.
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/Jessecar96/SteamBot/issues/780.

Think positively when trying to do new things instead of thinking that someone can not do something. If you are so smart, why not help me out? Or is posting comments like that more helpful than teaching someone? I wonder...

This is an issue tracker, not a help forum.  You were directed to the help forum in the previous issue #779.


Total Proxies 0 ?

Hi,
I did everything and they told me Total Proxies: 0
¿what i need to do and where?

THey told: Generated "proxy-list.txt" but i can't see it and i can put my proxies :S

It seems like the Free Proxy Lists https://www.freeproxylists.net website is inaccessible at the moment. As the proxy.py script is using that site for getting proxies, hence the script is generating 0 proxies.
I will update the proxy script soon. Meanwhile, one can directly make a proxy-list.txt file and add their collected proxies to it in the following structure : <IP address>:<port> per line
example : proxy-list.txt
<IP_ADDRESS>:3677
<IP_ADDRESS>:3523
<IP_ADDRESS>:987

Fixed in a7bc153 commit. New proxy provider added.


Switch to official Fleet API

Upgrade to version 1.1.0 of the Fleet API gem
Ditch arbitrary sleep delay for smart polling against Fleet API when deleting and loading units
Remove the use of docker rm and docker kill from unit files

:+1:


TensorFlowEstimator.restore Error: tensorflow.python.pywrap_tensorflow.StatusNotOK

Restore is still not working for the text_classification.py example.  I am getting the following exception:
I tensorflow/core/common_runtime/local_device.cc:40] Local device intra op parallelism threads: 4
I tensorflow/core/common_runtime/direct_session.cc:58] Direct session inter op parallelism threads: 4
Traceback (most recent call last):
  File "/Users/harsimranb/PycharmProjects/TensorFlowTest/TextRNN.py", line 103, in <module>
    runner.run_classification("amigo what time you close?")
  File "/Users/harsimranb/PycharmProjects/TensorFlowTest/TextRNN.py", line 56, in run_classification
    classifier = skflow.TensorFlowEstimator.restore(self.trained_model_path)
  File "/Users/harsimranb/Library/Python/2.7/lib/python/site-packages/skflow/__init__.py", line 332, in restore
    estimator._restore(path)
  File "/Users/harsimranb/Library/Python/2.7/lib/python/site-packages/skflow/__init__.py", line 314, in _restore
    self._saver.restore(self._session, checkpoint_path)
  File "/Users/harsimranb/Library/Python/2.7/lib/python/site-packages/tensorflow/python/training/saver.py", line 891, in restore
    sess.run([self._restore_op_name], {self._filename_tensor_name: save_path})
  File "/Users/harsimranb/Library/Python/2.7/lib/python/site-packages/tensorflow/python/client/session.py", line 368, in run
    results = self._do_run(target_list, unique_fetch_targets, feed_dict_string)
  File "/Users/harsimranb/Library/Python/2.7/lib/python/site-packages/tensorflow/python/client/session.py", line 446, in _do_run
    six.reraise(e_type, e_value, e_traceback)
  File "/Users/harsimranb/Library/Python/2.7/lib/python/site-packages/tensorflow/python/client/session.py", line 428, in _do_run
    target_list)
tensorflow.python.pywrap_tensorflow.StatusNotOK: Internal: Unable to get element from the feed.

Related to #40

Hm, it works in the case of just text_classification.py - I created an example in text_classification_save_restore.py.
Are you doing anything custom in the model that is different?

I think it's the same issue as #43 - you were using relative path and TF doesn't work currently very well with that. I made a workaround by upping to absolute path when you call save/restore. Please try again and let me know if it doesn't work!

That was it, thanks!


Rename PY_ARRAY_UNIQUE_SYMBOL to make it unique

This makes the symbol unique across all tensorflow parts, thus allowing to link tensoflow and tensorflow/lite into single binary.

Could you elaborate why this change is needed? This question is just for understanding the circumstances around this change.

Unfortunately, I can not disclosure the whole story, but the basic idea is to link two python modules into single binary file.
After this, two PY_ARRAY_UNIQUE_SYMBOLS become not so unique and this make interpreter to segfault.


[Tool] Add git setting to CI

What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[x] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

@Mergifyio backport branch-3.0 branch-2.5 branch-2.4

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

@Mergify backport branch-2.2

https://github.com/Mergifyio backport branch-2.3

@Mergifyio backport branch-2.3


Error in suggesting nonlocal

See https://twitter.com/andre_roberge/status/1456434342855684097

The bug is fixed but the explanation could be improved as mentioned in the dicussion on Twitter.


Leaking memory

Hello,
I use 2 instances of mtg on my VPS with 1gb of memory. The MTG stop transferring previews from Youtube after 24 hours of working. Other messages and pictures works fine. The restart fixed the problem. Before restart instance of the mtg took 300mb of memory.
Version: "0.16-111-g19efcc9 (go version go1.13.4 linux / amd64)" (latest docker image)
I can provide any debug information.

Yep. Both run with the same adtag.

It looks like telegram has stopped to respond to that socket for some reason. Since there was no reconnect, it also means that it didn't ask to close the connection. I've seen such behavior in the past but failed to catch it and debug.
As mitigation, I plan to do some force reconnects until I'll find a real reason for such behavior. At this certain moment you can run this proxy without adtag. Direct mode has no such flaw.

But all other stuff still working. Or it's using another socket?

Telegram has a notion of datacenters. Every time when a client wants to send an RPC request, it specifies a DC to connect to. Some DCs are responsible for getting blobs (when you send some files), some - profile information, some - pictures, etc. Every client also makes about 3-6 connections to the proxy depending on the workload. If you do not have any media, downloads etc, then client usually requires about 2 connections to different DCs.


compiled source matches convenience binary where expected

The JAXB generated class files in daffodil-debugger jar are different when I build from source. It's not clear to me what is different
(javap shows no differences) but this makes reproducible builds difficult to verify and is a bit concerning. I don't know why these
files would be different. Even if generated from JAXB, I would expect it to be deterministic and create the same .class files. Please look into this for the next major release, reproducible builds are important.

UUID like element gets randomly generated at each build, which would cause the above behavior. Timestamps are another common culprit.

Source of this ticket is approval VOTE email from @stevedlawrence for v1.3.1-rc2.

There is a timestamp placed in the header of each generated file - will try to remove them and compare outputs between runs.

This is a weird one... Adding the -no-header flag to xjc seems to resolve this... Waiting on confirmation from Steve to make sure it's not just on my machine. Completely unsure as to why a comment header would result in changes to a .class file, but I don't see any other diffs. I also decompiled two 'differing' class files, and they looked identical to me, so maybe some weird flag in the binary?
Even with all of that, the generated JAXB .java files still differ a little bit between runs. The order of {@link ...} comments are different in a handful of places. The files where this occurs also changes between runs. However, these comment differences do not appear to affect the .class or .jar files.
We should add the -no-header flag on our side, but I will leave this issue open until Steve confirms this is fixed and/or we switch over to scalaxb.

Since we won't make the Daffodil 3.6.0 release with the JAXB changes, we may want to include this. If we care about the .vsix file being binary-identical between builds and if this is the only reason it isn't, we can include this change. If we don't care about binary-identical .vsix files, there's no reason to include this change. Once we start using the Daffodil version with the JAXB bindings, any changes made for this will be moved into the Daffodil repository.


Add member count to the README.md document

Currently, we only state that we're a large community,  but the real scale isn't properly inferred there.
I'd like to back that up with a number (because it's cooler), and that number being how many members we have.
This is possible by making an endpoint in our site project that'll query the number of members we have based on our server's invite link.
Including the with_counts  query parameter will do that as defined here.
Then in the markdown, we'll just have a hyperlink that'll use that endpoint.
If there are other suggestions to do this, i'd be interested to find out more

You can do this without an extra endpoint. Take a look at https://github.com/Ukraine-Relief-Efforts
In the README I have:
[![Discord](https://img.shields.io/discord/946624673200893953.svg?label=TechFor&logo=discord)](https://discord.gg/4bQvrgXbxW)
Which renders as

That only fetches the online count though, not the total number of members, does it ?


In the admin panel, changing an embedded json field in a relationship causes all the fields below that to get deleted

Describe the bug
I have the following action:
    update :manage_templates do
      accept []
      argument :templates, {:array, :map}

      change manage_relationship(:templates, type: :direct_control)
    end

My Template resource has an attribute like this:
    attribute :config, :map, allow_nil?: true

If I have several templates, for example, template 1, 2, 3, and template 2 has some keys/values in its config map, then editing those values in the manage_templates action using the tree editor causes template 3 to be deleted.
Expected behavior
It allows me to edit template 2's config without deleting the other templates.
Runtime

Elixir version: 1.15
Erlang version: 25
OS: Mac OS
Ash version: 2.13.3
any related extension versions: ash_admin: 0.9.4

I believe this is resolved in recent versions of ash_admin. Please let me know if not.


HIVE-11376: removed code which would be useless assuming MAPREDUCE-1597 is present

Can you update testing done?

So the existing code has a code path(the code path that I've removed here) which was causing CombineHiveInputFormat to fall back on HiveInputFormat. That code path is guarded by a config check. Turns out, setting hive.hadoop.supports.splittable.combineinputformat=true will prevent the code from going into that code path.
We had a situation where this code path was causing a launch of 300K mappers for a job. I set hive.hadoop.supports.splittable.combineinputformat=true and launched the same job. This time it launched 325 mappers. The job completed successfully. Since the job is successful when the specific code block is prevented from executing, it means that the code block is not essential for correctness. Hence, removing that code block will also have correctness. I've already mentioned about performance gain.

Closing as we are going with configuration change.


portalocker::Lock unexpected keyword argument 'truncate'

$ virtualenv --python=python3 venv
$ source venv/bin/activate
$ pip install -Ur requirements_test.txt
$ make test
python setup.py clean
running clean
removed './smother.egg-info/PKG-INFO'
removed './smother.egg-info/SOURCES.txt'
removed './smother.egg-info/requires.txt'
removed './smother.egg-info/top_level.txt'
removed './smother.egg-info/dependency_links.txt'
removed './smother.egg-info/entry_points.txt'
removed directory './smother.egg-info'
find smother -type d -name "__pycache__" -exec rm -fr "{}" +
find smother -type d -name ".cache" -exec rm -fr "{}" +
find smother -type f -name '*.pyc' -delete
find smother -type f -name '*y.class' -delete
rm -f nosetests.xml
rm -f memory_usage.txt
python setup.py develop
running develop
running egg_info
creating smother.egg-info
writing requirements to smother.egg-info/requires.txt
writing smother.egg-info/PKG-INFO
writing top-level names to smother.egg-info/top_level.txt
writing entry points to smother.egg-info/entry_points.txt
writing dependency_links to smother.egg-info/dependency_links.txt
writing manifest file 'smother.egg-info/SOURCES.txt'
reading manifest file 'smother.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'smother.egg-info/SOURCES.txt'
running build_ext
Creating /tmp/smother/venv/lib/python3.5/site-packages/smother.egg-link (link to .)
Adding smother 0.2 to easy-install.pth file
Installing smother script to /tmp/smother/venv/bin

Installed /tmp/smother
Processing dependencies for smother==0.2
Searching for unidiff
Reading https://pypi.python.org/simple/unidiff/
Downloading https://pypi.python.org/packages/c5/a0/0e0edabce6abece92bc9eb20dd7b60c0dea8d86c6ab6a7a4359b20c517e5/unidiff-0.5.2.tar.gz#md5=20dd70ba5a35bc95bf869322d6852227
Best match: unidiff 0.5.2
Processing unidiff-0.5.2.tar.gz
Writing /tmp/easy_install-dfkb1min/unidiff-0.5.2/setup.cfg
Running unidiff-0.5.2/setup.py -q bdist_egg --dist-dir /tmp/easy_install-dfkb1min/unidiff-0.5.2/egg-dist-tmp-gw9tylvp
zip_safe flag not set; analyzing archive contents...
tests.__pycache__.test_parser.cpython-35: module references __file__
creating /tmp/smother/venv/lib/python3.5/site-packages/unidiff-0.5.2-py3.5.egg
Extracting unidiff-0.5.2-py3.5.egg to /tmp/smother/venv/lib/python3.5/site-packages
Adding unidiff 0.5.2 to easy-install.pth file
Installing unidiff script to /tmp/smother/venv/bin

Installed /tmp/smother/venv/lib/python3.5/site-packages/unidiff-0.5.2-py3.5.egg
Searching for portalocker>=0.4
Reading https://pypi.python.org/simple/portalocker/
Downloading https://pypi.python.org/packages/a5/8e/558a0c16fb766b79b122e608481be3037827766f8a54b9eebcfc43d4500e/portalocker-1.0.0.tar.gz#md5=638d163b17a506b951ff2cfaf12fcdc3
Best match: portalocker 1.0.0
Processing portalocker-1.0.0.tar.gz
Writing /tmp/easy_install-om4v9pbb/portalocker-1.0.0/setup.cfg
Running portalocker-1.0.0/setup.py -q bdist_egg --dist-dir /tmp/easy_install-om4v9pbb/portalocker-1.0.0/egg-dist-tmp-_ljxgnrj
zip_safe flag not set; analyzing archive contents...

Installed /tmp/easy_install-om4v9pbb/portalocker-1.0.0/.eggs/pytest_runner-2.9-py3.5.egg
warning: no files found matching 'CHANGELOG'
zip_safe flag not set; analyzing archive contents...
Moving portalocker-1.0.0-py3.5.egg to /tmp/smother/venv/lib/python3.5/site-packages
Adding portalocker 1.0.0 to easy-install.pth file

Installed /tmp/smother/venv/lib/python3.5/site-packages/portalocker-1.0.0-py3.5.egg
Searching for more_itertools
Reading https://pypi.python.org/simple/more_itertools/
Downloading https://pypi.python.org/packages/11/f0/7dc530a028b2b18366974960b26c7125872bdba65e703164617003bbf9fc/more-itertools-2.4.1.tar.gz#md5=f6bbfe38a6d0ee1a743adb2c1a4fb6fc
Best match: more-itertools 2.4.1
Processing more-itertools-2.4.1.tar.gz
Writing /tmp/easy_install-gimci_hv/more-itertools-2.4.1/setup.cfg
Running more-itertools-2.4.1/setup.py -q bdist_egg --dist-dir /tmp/easy_install-gimci_hv/more-itertools-2.4.1/egg-dist-tmp-ljx6ghie
warning: no files found matching 'fabfile.py'
zip_safe flag not set; analyzing archive contents...
Moving more_itertools-2.4.1-py3.5.egg to /tmp/smother/venv/lib/python3.5/site-packages
Adding more-itertools 2.4.1 to easy-install.pth file

Installed /tmp/smother/venv/lib/python3.5/site-packages/more_itertools-2.4.1-py3.5.egg
Searching for click
Reading https://pypi.python.org/simple/click/
Downloading https://pypi.python.org/packages/7a/00/c14926d8232b36b08218067bcd5853caefb4737cda3f0a47437151344792/click-6.6.tar.gz#md5=d0b09582123605220ad6977175f3e51d
Best match: click 6.6
Processing click-6.6.tar.gz
Writing /tmp/easy_install-i_9eoawj/click-6.6/setup.cfg
Running click-6.6/setup.py -q bdist_egg --dist-dir /tmp/easy_install-i_9eoawj/click-6.6/egg-dist-tmp-pqwz8mn4
warning: no previously-included files matching '*.pyc' found under directory 'docs'
warning: no previously-included files matching '*.pyo' found under directory 'docs'
warning: no previously-included files matching '*.pyc' found under directory 'tests'
warning: no previously-included files matching '*.pyo' found under directory 'tests'
warning: no previously-included files matching '*.pyc' found under directory 'examples'
warning: no previously-included files matching '*.pyo' found under directory 'examples'
no previously-included directories found matching 'docs/_build'
zip_safe flag not set; analyzing archive contents...
click.__pycache__.core.cpython-35: module references __file__
creating /tmp/smother/venv/lib/python3.5/site-packages/click-6.6-py3.5.egg
Extracting click-6.6-py3.5.egg to /tmp/smother/venv/lib/python3.5/site-packages
Adding click 6.6 to easy-install.pth file

Installed /tmp/smother/venv/lib/python3.5/site-packages/click-6.6-py3.5.egg
Searching for six==1.10.0
Best match: six 1.10.0
Adding six 1.10.0 to easy-install.pth file

Using /tmp/smother/venv/lib/python3.5/site-packages
Searching for coverage==4.0.3
Best match: coverage 4.0.3
Adding coverage 4.0.3 to easy-install.pth file
Installing coverage script to /tmp/smother/venv/bin
Installing coverage-3.5 script to /tmp/smother/venv/bin
Installing coverage3 script to /tmp/smother/venv/bin

Using /tmp/smother/venv/lib/python3.5/site-packages
Finished processing dependencies for smother==0.2
smother erase
py.test --smother=smother --smother-cover --smother-append smother
================================================= test session starts ==================================================
platform linux -- Python 3.5.2, pytest-3.0.5, py-1.4.32, pluggy-0.4.0
rootdir: /tmp/smother, inifile:
plugins: cov-2.4.0, smother-0.2
...
  File "/tmp/smother/smother/control.py", line 120, in write
    fail_when_locked=False
TypeError: __init__() got an unexpected keyword argument 'truncate'
Makefile:47: recipe for target 'test' failed
make: *** [test] Error 1

It passes with the following:

diff --git a/Makefile b/Makefile
index 0688e64..ccc90ee 100644
--- a/Makefile
+++ b/Makefile
@@ -13,7 +13,7 @@ help:
 	@echo "                 This removes ALL files not tracked by git including untracked"
 	@echo "                 files and temporary files made by text editors"

-WITH_ENV:=source venv/bin/activate;
+#WITH_ENV:=source venv/bin/activate;

 venv: venv/bin/activate
 venv/bin/activate: requirements_test.txt
diff --git a/smother/control.py b/smother/control.py
index f6598ab..0fc0805 100644
--- a/smother/control.py
+++ b/smother/control.py
@@ -115,7 +115,6 @@ class Smother(object):

             outfile = Lock(
                 file_or_path, mode='a+',
-                truncate=None,
                 timeout=timeout,
                 fail_when_locked=False
             )

py.test --smother=smother --smother-cover --smother-append smother
================================================= test session starts ==================================================
platform linux -- Python 3.5.2, pytest-3.0.5, py-1.4.32, pluggy-0.4.0
rootdir: /tmp/smother, inifile:
plugins: cov-2.4.0, smother-0.2
collected 42 items

smother/tests/test_cli.py ........
smother/tests/test_controller.py .....
smother/tests/test_diff.py ....
smother/tests/test_git.py ......
smother/tests/test_interval.py .....
smother/tests/test_plugins.py ..
smother/tests/test_python.py ............

================================================ pytest-warning summary ================================================
WC1 /tmp/smother/smother/tests/test_diff.py cannot collect test class 'TestDiffReporter' because it has a __init__ constructor
===================================== 42 passed, 1 pytest-warnings in 0.65 seconds =====================================
coverage html
coverage report --show-missing
Name                  Stmts   Miss  Cover   Missing
---------------------------------------------------
smother/__init__.py       0      0   100%
smother/cli.py           61     10    84%   63-65, 93-94, 118-119, 128-130
smother/control.py      139     10    93%   64-65, 68, 76, 122, 196-197, 208, 234-235
smother/diff.py          43      5    88%   40-41, 62, 69, 76
smother/git.py           41      4    90%   58, 64, 71, 77
smother/interval.py      48      6    88%   47-52, 121
smother/python.py       102      6    94%   132-133, 173, 193, 205, 220
---------------------------------------------------
TOTAL                   434     41    91%


fix: use faster-whisper's turbo model

https://github.com/SYSTRAN/faster-whisper/issues/1030#issuecomment-2440067655

Thanks. Yes, this makes sense when the new version of faster-whisper is released.

Thanks!


Fix issue #3568: allow sharing of activation function parameters along specified axes

A general fix for all appropriate activation functions for this issue https://github.com/fchollet/keras/issues/3568
Now with tests included!

Bumping because this is important for replicating any convolutional models that use activation functions with free parameters.

Hesitant to make any API changes at the moment. Also it feel like a bit of a niche use case, and one for which you could roll out your own layers. But I'll leave it open and consider later on.

I tried to make it only non-breaking additions that won't change how any existing code functions, but let me know what could be better. Also I don't think its really that niche, any use of parameterized nonlinearities applied to convolutional feature maps are subject to the original issue https://github.com/fchollet/keras/issues/3568 which is kind of a big deal I thought.

PReLU is needed for many state of the art vision tasks.
https://github.com/fchollet/keras/issues/4452
In this very simple MNIST example, the first PReLU will have 18432 parameters when it should be 32.  I will have to use TF/caffe for my application because of this.

Ok, agreed that we should add this feature. I'm wondering if we could avoid the use of pattern_broadcast which is a Theano-specific feature. A simple reshape wouldn't be inefficient since we'd only be adding unit dimensions. Also, param_share_axes isn't a very good name, the equivalent argument in Numpy is simply axis. We could consider reduce_axes or reduction_axes. Does this argument exist in other APIs out there, and what is it called?

I think in a lot of other APIs, there's no option for PReLU to have a tensor of slope parameters across more than one dimension, so your only choices are to have one shared slope parameter for all neurons in an entire layer, or to have N different slope parameters, where N is the dimension of one of the axes (usually the channels axis in a convnet, or the features axis for dense layers). I might be misinterpreting the Torch source though, so I could be wrong.
See:
http://caffe.berkeleyvision.org/doxygen/classcaffe_1_1PReLULayer.html
https://github.com/torch/nn/blob/master/PReLU.lua
As for the parameter name, reduction_axes or possibly unshared_axes?

Thanks for the feedback. I wrote this to have the maximum flexibility in terms of sharing parameters across different axes, but I agree param_share_axes isn't the best name. I could change to channels_shared as @Rocketknight1 suggests. Thoughts?
To avoid having to keep track of, reshape and permute dimensions given this flexibility, broadcasting seemed like the right solution. I used pattern_broadcast because Theano doesn't automatically broadcast unit dimensions but Tensorflow does, so the new backend function only actually does something for Theano. It seemed cleanest to make the Keras broadcasting API behave like Theano, and it seemed useful to have broadcasting in the backend as well. Everything else seemed even more ugly. But I'm of course open to suggestions.

@the-moliver @fchollet
I am re-writing the activation functions in dl4j (deep learning framework in java) to support such functionality and have been working on similar concepts. I came up with the term sharded dimension and would like to suggest the same.
Shared axis is technically not correct since when the alpha when per channel is shared across a subset of dimensions (rows and columns). Sharded seems more correct to me since the alpha is being duplicated across a dimension.

I'll leave it to our benevolent dictator to decide the best combination and order of one of {channels, axes, dimensions} with with one of {shared, sharded, unshared}. My feelings aren't particularly strong on this.

I'd lean towards reduction_axes or shared_axes. As long as the docstring is explicit about what it does, either name seems fine to me. reduction_axes is probably the best.

Made requested changes. I chose shared_axes because reduction_axes sounds to me like a reduction operation is being applied along certain axes, which is not the case.

Tests are failing.

Tests updated to use shared_axes name.

LGTM, thanks


Fusion Blacklist\Whitelist and less than a bucket left

I was curious if there is a way to blacklist/whitelist fluid fuels in the fusion reactor gui to a specific fuel type(s) to specific input tanks, so that if one fuel runs out, the other wont fill an empty slot.
I need to check this one when I get home (I could be wrong on how this behaves):
If there is less than a bucket of fuel left (say Hydrogen) Im pretty sure you cannot load another fuel because there is not enough fuel complete a cycle and empty the tank. I propose a button on the gui that toggles the ability for the fusion reactor to be able to either void partial buckets of fuel at the end of a reaction, or the ability to process partial buckets for a percentage of process time based on how much fluid is left.
Just starting out I found these two reactions produced Hydrogen and Helium3 inversely, so I was able to sustain temperatures (and over time increase efficiency) by swapping the products out and using the Neutron fluid to produce great amounts of Tritium. This is not net gain; Hydrogen, Lithium and/or Boron would need to be created/processed. The idea is that you would only need a single reactor to alternate between T-H and T-He3 combos if set up correctly and it would take considerably less reserve fuels get a reactor to desired efficiency.
This could also help when using a less costly fuel to "pre-heat" a reactor and automatically swap over to another fluid after these fuels run out.
These things cannot currently be done because fuels in the tanks cannot be controlled/separated automatically.

Confirmed that leaving less than a bucket in the tank of any liquid stops alternative fuels from being loaded.

When I originally read this, I didn't imagine it would be possible wihtout some trouble, but I think I have managed to implement it in a reasonably intuitive way, pretty much as you described above. Every time a process gets completed, if this voiding option is on, then if it is not possible to initiate a new process, the input fluids will be voided - this will be available in the next release, which should be tonight :)

Epic, I cant wait to see this tank clearing in action. I am curious if tank white-listing has also been added to manage fuel control more effectively as well. I also quite enjoyed the fission video that covered everything in one go. Keep up the awesome

Fluid whitelisting is quite tricky do do on the machine's side of things - could you not set up some redstone-controlled filtered input pipes?

Not really because the internal tanks don't care if you have multiple fuels. If your trying to load T-He3 (after completing T-H for example), It will input Tritium into the second tank causing a T-T reaction. This would not be an issue if there were dedicated input ports for each internal tank on the 3x3x3 or some method of white-list per tank.

Have you tried using the ‘Prevent Input Overflow’ mode? This will prevent one fuel type filling both input tanks.

Is this the function of the button on the upper right corner of the interface without text label? I must have missed the info on that in an update and was actually wondering what the mode change did.
My apologies; I have not tried this. If this functions as described it should work fine for alternating between T-H and T-He3 combos. The tank will empty with the help of the new (less than a bucket cleanup) toggle button, and then the alternate fuel can load while Tritium remains in the first tank and will not load to the second.
The only thing I can think of being an issue is when trying to intentionally load the same fuel (possibly for pre-heat) and then wanting to swap to primary fuels, but you can use Li7-B11 for this as an alternate method than Li7-Li7 or B11-B11 which all have similar higher Heat Variables.
I appreciate the help and wanted to state again how epic this mod is, and like your vision for its progress and functionality. Please keep up the good work. My hope is that this remains a power generation option in the popular FTB packs.


JBPM-5441 - When a timer is triggered the user unknown does not have …

…permission to access asset which requires a specific role
@ibek if you find a minute...

looks great, thank you for fixing it Maciej
so generally every async activity bypass the security right? Since it's always planned by model (not requested by any user) it should be alright to do so.

that is correct, async operations will bypass security check if user info can't be retrieved, that's why the check is at the end of the security verification.


Add kwargs to arguments

Summary
Add kwargs to low levels functions, this will allow passing a dictionary to them to avoid attribute repetitions in the top level function
Checklists

This pull request meets the following requirements:

[x] installable with all dependencies recorded
[x] runs without error
[x] follows PEP8 and project specific conventions
[x] appropriate use of comments, for example no descriptive comments
[ ] functions documented using Numpy style docstings
[x] assumptions and decisions log considered and updated if appropriate
[x] unit tests have been updated to cover essential functionality for a reasonable range of inputs and conditions
[x] other forms of testing such as end-to-end and user-interface testing have been considered and updated as required
[x] tests suite passes (locally as a minimum)
[ ] peer reviewed with review recorded

If you feel some of these conditions do not apply for this pull request, please
add a comment to explain why.
Closes #25

All tests and pre-commit hooks run successfully. Outside of src/flag_and_count_matched_pairs.py, all functions with kwargs seem to be missing kwargs in their docstrings - should this be included?

All tests and pre-commit hooks run successfully. Outside of src/flag_and_count_matched_pairs.py, all functions with kwargs seem to be missing kwargs in their docstrings - should this be included?

You are right @lhubbardONS, i have committed the docstrings.

@lhubbardONS Thanks for spotting these :)


Question - how to get app URL from CLI?

Hi Brian,
I have another question for you, this time about how you get the correct app URL when in the CLI. Our app sends emails from shell scripts , and these emails contain links to pages in the app.

First of all, it checks if $_SERVER["HTTP_HOST"] is set. If not, it does some further file_exists() calls to look for files which identify which server it's on (because I hadn't heard of environment variables when I wrote this!). Based on the results of that, HTTP_HOST is then set. i.e. if file_exists("this_is_production") returns true then $_SERVER["HTTP_HOST"] is set to the production URL.
Then follows a switch block which switches on the value of $_SERVER["HTTP_HOST"] and sets environment vars for each possible URL.
Finally, it checks if FULL_BASE_URL is defined (app is still on Cake 2.2 before this was deprecated), and if not, defines it as being equal to $_SERVER["HTTP_HOST"]. This gives the app access to the correct app URL whether running as HTTP or CLI.
How would you handle this with your method? The only thing I can think of is to set the URL with the Configure class in app-{$env}?

Another good question. Yeah, this gets sticky too, but I've found the env-awareness can help here. I've had to do the same kind of thing in a nightly Shell that was generating emails and needed the base URL to be correct for the given instance of the app (dev, staging, production, etc.)
The way we approached this is to let the env you set up also define a default domain that should be associated with it. So in our core.php we have something like this:
/**
 * Default Site Configuration
 *
 * Any time you'd be tempted to type one of these strings directly into
 * a file, call this Configure var instead.
 */
Configure::write('Defaults', array(
	'short_name' => 'ACME',
	'long_name' => 'ACME Inc Application',
	'domain' => 'acme.com',
	'meta_description' => 'Maker of great widgets.',
	'meta_keywords' => 'acme, widgets, great',
));

...which of course you can override for each environment.
Then in our Shell's initialize() method:
	/**
	 * Startup for the EmailShell
	 *
	 * This could also even go in your application's AppShell to let every Shell inherit it.
	 */
	public function initialize() {
		parent::initialize();
		if (!defined('FULL_BASE_URL')) {
			define('FULL_BASE_URL', 'http://' . Configure::read('Defaults.domain'));
		}
		// ...
	}

The domain isn't used for anything else from the web browser end of things, since Cake's Router class properly uses whatever domain it's currently being served from in that context. This approach let's you customize the output for each environment, but also only engage that customization from the command line, and let's you do it cleanly. It makes the domain a prescriptive value that you tell the app, instead of an passive value that the app obtains from somewhere else.
I have yet to come with a better or simpler method, but as usual if anyone can improve on this I'll happily switch myself.

On the file_exists() point too, I want to mention that this is actually a great alternative to environment variables. In fact, in some instances it's far easier. PHP running under FPM for example, does not get provided with any Apache SetEnv directives, so it becomes hard to provide an environment variable to the PHP runtime. Using a "marker file" is a great alternative.
As I said in my talk, it really doesn't matter what mechanism you use to detect the environment, as long as you're only having to check for it once and your app is set up to "to the right thing" even if that detection fails. Of course, the simpler and less error-prone the checking process is, the better. I like Apache SetEnv because I pretty much always have to set up a virtual host anyway so it's not so much an extra step as it is a part of an existing step to include the SetEnv APP_ENV whatever line.
That said, in some situations you just have to do the best you can with what you've got, but you should still strive to keep it as bulletproof as possible. For example, if I was going to use the "check a file" approach, my code would probably look something like this:
// in Config/core.php

// Replace this:
// $env = getenv('APP_ENV');

// With this:
$env = trim(@file_get_contents(dirname(__FILE__) . DS . 'APP_ENV.txt'));

// Still do the same config loading.
if (is_readable(dirname(__FILE__) . DS . "core-{$env}.php")) {
	Configure::load("core-{$env}");
}

Why this way?

I'd add Config/APP_ENV.txt to my .gitignore file to keep it out of source control, so it doesn't get committed with a fixed value that might get deployed to other environments and cause problems. This forces someone to create the correct files for each other environment.

I want to always check the same file name (because there's less chance of something going wrong, typos, etc.) and because it doesn't require if/elseif/else statements for every possible environment. Just read the same file, and get the correct env from inside. This mirrors reading the same env var, and getting the correct value from inside of it.

I use the @ suppression because I don't care if the file doesn't exist (maybe you do though) and I want to keep going assuming we're on production if reading the file fails.

I use trim() to take care of stray whitespace in the file, which is more likely to happen accidentally than in an environment variable and cause problems matching environments.

It's still just that one single line of code that makes the whole thing work! I like the elegance of that.

All in all, I think this minimizes the amount of things that can go wrong, and the amount of work I have to do manually to maintain each environment. It's a single unique word typed in the same file file for every environment.

Thanks for the detailed responses, I appreciate you taking the time. Glad to hear my file_exists() solution is not so hacky after all!
I like your proposed solution for the CLI URL. I would assume that one should set a default protocol as well in the config, e.g.
/**
 * Default Site Configuration
 *
 * Any time you'd be tempted to type one of these strings directly into
 * a file, call this Configure var instead.
 */
Configure::write('Defaults', array(
    'short_name' => 'ACME',
    'long_name' => 'ACME Inc Application',
    'domain' => 'acme.com',
    'protocol' => 'http',
    'meta_description' => 'Maker of great widgets.',
    'meta_keywords' => 'acme, widgets, great',
));

Then in AppShell:
    /**
     * Startup for the EmailShell
     *
     * This could also even go in your application's AppShell to let every Shell inherit it.
     */
    public function initialize() {
        parent::initialize();
        if (!defined('FULL_BASE_URL')) {
            define('FULL_BASE_URL', Configure::read('Defaults.protocol') . '://' . Configure::read('Defaults.domain'));
        }
        // ...
    }

Yeah, that's an improvement in my opinion. Alternatively, you could just include the scheme in the existing var (and rename it for clarity/accuracy) if this was the only way it was ever going to be used.
Configure::write('Defaults.shellBaseUrl', 'https://acme.com');

Then there's a whole other discussion on the best way to "enforce" SSL in the environments that require it (like production) but gracefully leave it disabled (or optional) in dev or testing environments...


the style decoder take the offset as input, does it mean the offset to the parent or the last moment position

the style decoder take the offset as input, does it mean the offset to the parent or the last moment position

Hi, in the style transfer project the style encoder takes the joint positions (relative to the root position) as input. (But the style transfer paper never mentioned "offset" or "style decoder". Which project are you referring to?)

Thank you, and I am so sorry to make a mistake in my question. I do mean the encode


Restrictions on column names?

Are there any restrictions on column names in Parquet/Arrow that we could check for in the schema?
Are these for example all valid?

"\r\n"
"a\tb"
"😀"
"äöüß"
...

Column names correspond to field identifiers in the Thrift IDL.  See Identifier here: https://github.com/apache/thrift/blob/v0.17.0/doc/specs/idl.md#identifier
So we could require some pattern like ^([A-Z]|[a-z]|_)([A-Z]|[a-z]|[0-9]|\.|_)*$, but I'm not sure that is a good idea.  There may be implementations that accept more than these characters (I know there are implementations that accept fewer).  And maybe there will be some future version that accepts a wider range of identifiers.
If anything, I think a good validator would assert that the geometry column names match an existing top-level field name, but I think it might be more trouble than value to add JSON schema validation around the identifiers.


Vitest runner: add to old version

Vitest runner: add e2e test for the oldest supported vitest version (feel free to pick a 2y old version en pin that as the min peer dep version)

Fixed with #4043


dist/vite.js contains hardcoded absolute path to user computer

TypeError [ERR_INVALID_ARG_VALUE]: The argument 'filename' must be a file URL object, file URL string, or absolute path string. Received 'file:///home/mohamed/Desktop/projects/svelte/unplugin-svelte-components/tsup.config.ts'

interesting...can you share your code or way you used the package ?
i just create a live demo and i didn't get the same error, so it will be helpful if you share how did you step up it in your project.

I have installed the package and imported it in vite.config.js as import Components from 'unplugin-svelte-components/vite.
It ended up in this error, and when I checked the dist/vite.js file in the unplugin-svelte-components package the first two line has hardcoded into your computer I guess! (i.e., file:///home/mohamed/Desktop/projects/svelte/unplugin-svelte-components/tsup.config.ts)

update to the latest version, demo

Sorry, I updated to the latest version
"svelte": "^3.54.0",
"vite": "^4.0.0"
"unplugin-svelte-components": "^0.2.1",

But received the following error
Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only URLs with a scheme in: file, data are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'd:'
    at new NodeError (node:internal/errors:387:5)
    at throwIfUnsupportedURLScheme (node:internal/modules/esm/resolve:1075:11)
    at defaultResolve (node:internal/modules/esm/resolve:1155:3)
    at nextResolve (node:internal/modules/esm/loader:173:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:852:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:439:18)
    at ESMLoader.import (node:internal/modules/esm/loader:536:22)
    at importModuleDynamically (node:internal/modules/esm/translators:110:35)
    at importModuleDynamicallyCallback (node:internal/process/esm_loader:35:14)
    at configResolved (file:///D:/github_projects/felion_gui_v4/node_modules/unplugin-svelte-components/dist/chunk-EZRYSAWV.mjs:25651:21)

this fixed it though
changing configfile to URL using pathToFileURL
const configFile = path.join(config.root, "./svelte.config.js")
const pkg = await import(configFile)

to
import { pathToFileURL } from 'url'
...

const configFile = path.join(config.root, "./svelte.config.js")
const pkg = await import(pathToFileURL(configFile ))

what version of node are you using ?

Node version ^18

this fixed it though
changing configfile to URL using pathToFileURL
const configFile = path.join(config.root, "./svelte.config.js")
const pkg = await import(configFile)

to
import { pathToFileURL } from 'url'
...

const configFile = path.join(config.root, "./svelte.config.js")
const pkg = await import(pathToFileURL(configFile ))

the problem with the pathToFileURL is breaking in Linux, i think you are using windows, right ?
DTS Build start
src/core/unplugin.ts(45,30): error TS7036: Dynamic import's specifier must be of type 'string', but here has type 'URL'.

Error: error occured in dts build
    at Worker.<anonymous> (/home/mohamed/Desktop/projects/svelte/unplugin-svelte-components/node_modules/.pnpm/tsup@6.2.3_typescript@4.8.4/node_modules/tsup/dist/index.js:2157:26)
    at Worker.emit (node:events:513:28)
    at MessagePort.<anonymous> (node:internal/worker:233:53)
    at [nodejs.internal.kHybridDispatch] (node:internal/event_target:731:20)
    at exports.emitMessage (node:internal/per_context/messageport:23:28)
Error: Failed to compile. Check the logs above.
    at error (/home/mohamed/Desktop/projects/svelte/unplugin-svelte-components/node_modules/.pnpm/rollup@2.79.1/node_modules/rollup/dist/shared/rollup.js:198:30)
    at throwPluginError (/home/mohamed/Desktop/projects/svelte/unplugin-svelte-components/node_modules/.pnpm/rollup@2.79.1/node_modules/rollup/dist/shared/rollup.js:21718:12)
    at Object.error (/home/mohamed/Desktop/projects/svelte/unplugin-svelte-components/node_modules/.pnpm/rollup@2.79.1/node_modules/rollup/dist/shared/rollup.js:22672:20)
    at Object.error (/home/mohamed/Desktop/projects/svelte/unplugin-svelte-components/node_modules/.pnpm/rollup@2.79.1/node_modules/rollup/dist/shared/rollup.js:21895:42)
    at Object.transform (/home/mohamed/Desktop/projects/svelte/unplugin-svelte-components/node_modules/.pnpm/tsup@6.2.3_typescript@4.8.4/node_modules/tsup/dist/rollup.js:7217:20)
    at /home/mohamed/Desktop/projects/svelte/unplugin-svelte-components/node_modules/.pnpm/rollup@2.79.1/node_modules/rollup/dist/shared/rollup.js:22879:40
DTS Build error

i will try to find work around

Yes I'm using windows. I checked some other repos issues, in general, looks like only windows have this issue.

did the other repos resolved this issue ?

it was discussed here
https://github.com/okonet/lint-staged/issues/1054#issue-1059422989

Since URL type is the issue in Linux, can the following method work?
const pkg = await import(pathToFileURL(configFile).toString());

i just released new version, please check it.

Great, thanks it is working.


Added percent

i added the same percent option as in the speechrecognition script. instead of editing two existing jsons seperately it makes two jsons with the names train.json and test.json. the test.json has a given percent of files(default 10 percent of the files are put into test.json)

really good idea!


[10.x] Throws an exception in case of passing an invalid key name 🐛

I was working on a project and I needed to set a default value for an optional parameter, so I used the defaults route method, but I found an unexpected approach. I found that this method set the given value to whatever key I pass, knowing that this key maybe not exist in the route parameters. So, this PR solves this issue by throwing an exception in case we pass an invalid key.

First, you are missing an import for the exception, that is why tests are currently failing
Also, I am not sure if this method should constrain usage to named parameters.
The Route@defaults method is used in the Router@redirect method to bind some parameters that might not be part of the route's URL.
Actually, documentation states they are reserved parameters and should not be used in redirects: https://laravel.com/docs/10.x/routing#redirect-routes
So, in redirect's case you could have named parameters, and the actually bound route would have two additional parameters bound to it that are not named routes. I am not sure how you can handle those cases.
https://github.com/laravel/framework/blob/4b3bd091d7a4e8a9c205cf528d3f08c345c8c1be/src/Illuminate/Routing/Router.php#L248-L253
You can check how these parameters are used on the Illuminate\Routing\RedirectController class.


WebSocket sample not working for wss connections

Got a hub hosted in IIS. It uses https.
Connecting with a modified https://github.com/aspnet/SignalR/blob/dev/samples/ClientSample/HubSample.cs
everything works as expected.
Using the websocket sample
https://github.com/aspnet/SignalR/blob/dev/samples/WebSocketSample/Program.cs
to connect to wss://route/to/hub/ws
gives me an Exception stating it is unable to connect to the remote server. Am I missing something?
var ws = new ClientWebSocket();
ws.Options.RemoteCertificateValidationCallback = (sender, certificate, chain, errors) => true;
await ws.ConnectAsync(new Uri("wss://route/to/hub/ws"), CancellationToken.None); --> Exception

Can you collect the information requested in the issue template? https://github.com/aspnet/SignalR/blob/dev/.github/ISSUE_TEMPLATE.md
Most specifically, it would be helpful to have client and server logs.
Are you able to hit the server with plain https? (i.e. not using a WebSocket?)

try             {                 await ws.ConnectAsync(new Uri("wss://route/to/hub/ws"), CancellationToken.None);             } catch (Exception e)             {                 Console.WriteLine(e.Message);                 Console.WriteLine(e.StackTrace);             }
gives me following output:
Unable to connect to the remote server
at System.Net.WebSockets.WebSocketHandle.ConnectAsyncCore(Uri uri, CancellationToken cancellationToken, ClientWebSocketOptions options)
at System.Net.WebSockets.ClientWebSocket.ConnectAsyncCore(Uri uri, CancellationToken cancellationToken)
at ConsoleApp3.Program.RunWebSockets() in C:\Dev\workspace\webonds\ConsoleApp3\Program.cs:line 22
There are no client logs, since it is a plain console app.
I dont' know where to find the logs of the server app from IIS.
I configured it using
 WebHost.CreateDefaultBuilder(args)                 .UseIISIntegration()                 .UseKestrel()                 .ConfigureLogging((hostingContext, logging) =>                 {                     logging.AddConfiguration(hostingContext.Configuration.GetSection("Logging"));                     logging.AddConsole();                     logging.AddDebug();                 })                 .UseStartup<Startup>();

I also tried to run the server app locally.
Content root path: C:\Dev\workspace\webonds\WebApplication2\bin\Release\netcoreapp2.1\publish Now listening on: http://localhost:5000 Now listening on: https://localhost:5001
Now, again, connecting via HubConnection works as expected, using the connection to the plain websocket with
await ws.ConnectAsync(new Uri("wss://localhost:5001/bonds/ws"), CancellationToken.None);  or
await ws.ConnectAsync(new Uri("wss://localhost:5000/bonds/ws"), CancellationToken.None);
does not work where 'bonds' is the route to my hub.
I use the SDK Microsoft.NETCore.App (2.1.0-preview2-26406-04) dependency on both client console apps.
The HubConnection app uses SignalR.Client (1.0.0-preview2-final) dependency.
The server app uses the same SignalR and NETCore.App dependencies and additionally it uses AspNetCore.App (2.1.0-preview2-final).
I hope this helps a bit more.

It was await ws.ConnectAsync(new Uri("ws://localhost:5000/bonds/ws"), CancellationToken.None);

Remove /ws from the end of your Uri

Yeah, the endpoints were modified to not have /ws on the end, but the WebSocketSample is out of date. I'll send a PR to fix it.

Also, please note that if you are connecting to a SignalR hub, the WebSocketSample is not sufficient. You are responsible for reimplementing the entire SignalR Hub Protocol. The WebSocketSample is for connecting to a ConnectionHandler such as the MessagesConnectionHandler in the SignalRSamples project
If you just want to connect to a Hub using WebSockets directly, you should use HubConnection from Microsoft.AspNetCore.SignalR.Client but set the transport option to WebSockets rather than using the default, which automatically negotiates a transport for you.

Ok. That seems to work now. Since the sample was reworked in the meantime, I think this issue can be closed.
I wanted to use the websocket connection to use my backend hub with frameworks that have no client library (speaking of Dart and Flutter). The previous sample code suggested this possibility without reimplementing the hub protocol.


chore: upgrade charts

updated chart jxgh/lighthouse-webui-plugin from 0.1.6 to 0.1.7

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign ccojocar
You can assign the PR to them by writing /assign @ccojocar in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Dynamically changing account ID  after App init

Hi,
In my current setup, we don't know the Account (tracking id ) until we make a request to get environment info. So I need to wait to set the Account until after the app initializes,  ( I do not have access to $http in a 'config' )
ie:
import analytics from 'angular-google-analytics'
let app = angular
    .module('app', [
        'angular-google-analytics',
    ])

app.config(function (AnalyticsProvider) {
   //**cant do this here**
    //AnalyticsProvider.setAccount('xxx')
    AnalyticsProvider
      .trackPages(true)
      .setPageEvent('$stateChangeSuccess')
      .setDomainName('none')
      .enterDebugMode(false)
      .startOffline(false)

})

So I need to be able to do something like this below. (but it won't work because 'setAccount' is not a function of Analytics) And if I try to manipulate the Analytics.configuration.accounts, it still gives me the  #error
"No account id set to create analytics script tag" and analytics does not work
This is essentially what Im trying to achieve
  app.run([ 'someService', 'Analytics', function(someService, Analytics) {
        someService.getAccount(resp => {
            **set up my Account tracker ID ,etc.**
    }])

Thanks for any advice!

FYI i was able to do something like this to  change the account info.
app.config(function (AnalyticsProvider) {
    AnalyticsProvider.setAccount('UA-XXX-dummyID')

    AnalyticsProvider
      .delayScriptTag(true)
      .ignoreFirstPageLoad(true)
      .trackPages(true)
      .trackUrlParams(true)
      .setPageEvent('$stateChangeSuccess')
      .logAllCalls(false)
      .setDomainName('none')
})

and in my 'run'
app.run([ 'someService', 'Analytics', function(someService, Analytics) {
        someService.getAccount(resp => {
            Analytics.configuration.accounts[0].tracker = resp.acct
             Analytics.createAnalyticsScriptTag()
    }])

proabably not the best way to do it but it works :)

@29er Yeah, you figured out the way to do it that circumvents the config stage. This is purposefully mentioned in the README as unsupported because it should change in the future to prevent such monkeying with the configuration object post config phase.
The suggested way to do this is to pre-load the configuration from the server prior to starting angular.
I do see the merit though in exposing some configuration methods during the run phase if the 'delayScriptTag' was set to true. These extra configuration methods would then produce exceptions if called after the 'createAnalyticsScriptTag' was called.

@justinsa  So, for now if I wanna resolve Analytics account in run-time, I need:
AnalyticsProvider.setAccount('UA-XXX-defaultID');
AnalyticsProvider.delayScriptTag(true); // Can I chain it with previous call?

app.run([ 'someService', 'Analytics', function(someService, Analytics) {
        someService.getAccount(resp => {
            Analytics.configuration.accounts[0].tracker = resp.acct
             Analytics.createAnalyticsScriptTag()
    }])

If you are doing manual bootstrapping of your application you have access to the injector and therefore $http before your application bootstraps.
angular.element(document).ready(() => {
    const $http = angular.injector(['ng']).get('$http');
    const settingsRequest = { ... };

    $http(settingsRequest)
        .then((response) => {
            settings = response.data;
            angular.module('app').constant('yourAppSettings', settings);
            angular.bootstrap(document, ['app'], { strictDi: true });
        });
});

...

app.config(function (AnalyticsProvider, yourAppSettings) {
    // You can do this now :)
    AnalyticsProvider
      .setAccount(yourAppSettings.accountId)
      .trackPages(true)
      .setPageEvent('$stateChangeSuccess')
      .setDomainName('none')
      .enterDebugMode(false)
      .startOffline(false);
});


refactor(ui): change label style to make the value more visible

Fixes #18

Current experiment:


Support udp2raw/stunnel application to baypass blocking in countries

I'm a user in one of countries where i can't use both actually wireguard and openvpn because my ISP is blocking both protocols
So it would be nice idea to add support for application like udp2raw or really any other application to unblock udp protocol
heck! even if you want i could submit an PR to add it

Submit submit submit!

Sure! i already got guides link just allow me some time to translate it into script and merge it
PS: my exams are near (24 so yup you need to wish me luck and allow me some time buddy!) but wish me luck i'm serious i did not study well


Bump aborted runs down in cluster run summary, by falling back to sta…

…rt time when end time is missing

👍

+1


./configure modifies build_config.bzl which is version-controlled

Configure modifies build_config.bzl that is tracked by version control. This causes following annoyances:

When you do git pull, it'll overwrite build_config.bzl after which you need to do a ./configure and rebuild. I sometimes do testing of the same change on Mac and Unix host, and use git pull to sync them, which will overwrite my machine-specific config with default value.
You can't use git commit -a because that'll include build_config.bzl

Current work-around for 1. is to do following when pulling:
git stash
git pull
git stash pop

If you merge failed message, do
git checkout --theirs -- tensorflow/core/platform/default/build_config.bzl

For 2. it's possible to use skip-worktree
@aselle what do you thinking about adding build_config.bzl to .gitignore?

Such trivial change may be better discussed in a PR, so closing for now


How to create the node permutation?

Hi, @acbull. thank you so much for sharing the code. I have a question on the graph preprocessing. May I ask how to determinate the order (permutation) of nodes in a sampled subgraph? What's the criterion that we refer to? -:)

It's a good question that I haven't do a thorough analysis yet.
Currently, I just use my own sampling method to determine the node order, by firstly randomly picking a batch of nodes, and layer-wise expand the neighborhood set. You can refer to data.py for more details.
Ideally, if we know the exact node sequence (such as temporal sequence) to generate the node, it would be better.

I see. Thank you so much for a quick response.


CommissionerSessionId starts from 0

Certificate test 9.2.2 sets CommissionerSessionId to 0 for testing. This is needed for passing 9.2.2.
Not quite the reason that start from 1, so using a separate PR.

FYI, the build failure was introduced in #641.  I just merged #650 which moves the failing test back into the XFAIL list.  You'll need to rebase on top of master to get the checks to pass.

I'm not sure why the test spec requires setting the Session ID to 0.  I could not find any such statement in the Thread Specification itself.  I'll investigate why this check is in place.

Current coverage is 65.75% (diff: 66.66%)

Merging #655 into master will increase coverage by 0.98%

@@             master       #655   diff @@
==========================================
  Files            99         95      -4
  Lines         14455      12852   -1603
  Methods        1983       1789    -194
  Messages          0          0
  Branches       1706       1496    -210
==========================================
- Hits           9363       8451    -912
+ Misses         4565       3953    -612
+ Partials        527        448     -79

Powered by Codecov. Last update a853ffc...eaf29fd


Add support for Pickup shipping method.

Is there any possibility of adding support of pickup?
An overview:

A product can exist in a warehouse. after that, it goes to a store location, how can someone see the store location? There's a Location model here, it has a .stocks that's exactly the same as the relation between Warehouse and Stock
We add Pickup as a shipping method and modify logic as needed.

This also introduces a Location feature, so in the dashboard you can enter where you locations lie, which is I think an essential features because not all products are in warehouses and there must be some kind of tracker for this.

Work on that is already in progress.
Warehouses will have additional flag "use for pickup". Feature is at refining phase so cant give any ETA

Using warehouses gives us:

stock management (users will be able to check if product is in the store before payment
we are reusing existing interfaces for stock management
store manager will be able to configure if pulling stock from other warehouses is possible

@krzysztofwolski
I understand, but we have 2 stocks, you're talking about the normal stock but I'm talking about the store stock, so a pickup is from the store, this means that the item is in the store currently, my whole talk is about store locations more of a pickup, and it will allow pickup logically, The stock should be not only in a warehouse, but also a store, how do we know that item X is in store Y
atm? this is the question I'm trying to answer. Correct me or discuss it with me, I'm willing to help with PRs.


Fix test in Login.js

Problem
Please troubleshoot the test on line 91 of Login.test.js. You can replace the .skip snippet at the beginning of the text declaration to .only, and run the test using npm run test Login  Test should ensure that signing up makes the app reroute to the gardens page. I'm not sure why it isn't working! Please take a look and see if you can figure it out.
Additional Resources
You may find it helpful to read the docs on testing Apollo/GraphQL.

Can you review this @capndave


2421/no main args

#2421

as of right now the CI hasn't run... not sure why, but I imagine it might come up with some stuff.

Aha, first time contribs need to have the CI approved by a contributor.  So hopefully it's running now.

I couldn't set up the negative test probably, is there a guide or should I look the tests in the repo and find my way around ?

We do perform other validations on main() functions here, do you think it would be better to call your check there?
For posterity and documentation, we will eventually support this: #845. Perhaps you could link to the issue in the error message?

Oh, thanks for the detailed explanation, I will add this as soon as I can

Yeah kinda wrestling with it, but I believe if we perform the check on validate_root we should wrap it in ok otherwise the tests start to break up. Following some traces but kinda lost 😄 Also sorry about the comments, fixing those too.

I'm trying to run some tests on my local machine but getting
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Response errors; TransactionCreateBytecodeWitnessIndex', test/src/e2e_vm_tests/harness.rs:28:10

when I try to run tests under the e2e_vm_tests/test_programs/should_pass/test_contracts.  Any suggestions ?

I'm trying to run some tests on my local machine but getting
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Response errors; TransactionCreateBytecodeWitnessIndex', test/src/e2e_vm_tests/harness.rs:28:10

when I try to run tests under the e2e_vm_tests/test_programs/should_pass/test_contracts. Any suggestions ?

Hmm, never seen that error before. Something to keep in mind though, you have to restart the Fuel node between test runs.
I guess you are already running a node to get that error, but if you aren't, you can launch it by cloning the fuel-core repo and doing: cargo run --release --bin fuel-core -- --db-type in-memory

same error remains, any other suggestion ? thanks for your help

Hm, you could try deleting your ~/.fuel directory.
ok() represents either an okay state, or at least a state that recovered (Some value + errors means it recovered). In the case of recovery, the errors will be reported but the compiler can continue on.
So, yes, it is okay to return non-empty errs with ok(), this is actually how we do recovery after a compile error. If an error is non-recoverable, then we use err() which has no value at all.
Therefore, we should be using ok() here and reporting the errors back to the user, but continuing the compilation process.

Therefore, we should be using ok() here and reporting the errors back to the user, but continuing the compilation process

Yep, that's what I thought, except I didn't think we were catching them in this case.  But double checking I see that we do capture the errors when calling validate_root(), so ok() is correct.  Thanks.

Thanks for detailed description first of all, the problem remains but I believe it's not due to my changes because the master branch fails with the same error also. The negative test passes for my part, do you want me to push the changes ? I'm open to any other suggestion also

The test runner in CI is failing for two reasons. One, an outdated lockfile. Secondarily, the filecheck string you provided in test.toml (})  is not being found.
See this filecheck string as an example. You'll want to put a string that matches the location of the error.

As for the transaction error, that's not really relevant to this change and could be caused by a version mismatch. Feel free to filter out that test for now by running cargo run -- <FILTER_STRING> in test. E.g. cargo run -- main_should_not_have_args

I know I have been a headache for the last couple of days but I really don't want to spend any more of your CI minutes 😅 . I have merged the master to my branch and on v0.19.2 , but Forc.lock still has some GitHub  link, with version 0.19.2. I have run the test explicitly on my local with cargo run --locked --bin test -- main_should_not_have_args --locked on my local machine and it passes. Is it going to break CI again, if so what are the steps I should take ? Thanks for your time again, sorry for your CI minutes.

Not a headache at all, we love any and all contributions :) Also github CI minutes are free for public respositories.
If you just delete the Forc.lock file in your test, then run forc build again, it should regenerate the lockfile. Then, commit the regenerated file.


Update template to require rails_helper instead of spec_helper for Rspec >= 3.x

https://github.com/rspec/rspec-rails/blob/master/Changelog.md#300--2014-06-01

@casebook @amarshall how do you y'all feel about this?

@casebook bump ^


Stp 831 refactor model in typescript part3

Description
Ticket STP-831 - Refactor existing Javascript code into Typescript (Part 3)

Large files with multiple classes were broken down (one class per file).
Adjust folder structure to contextualize codebase.
Typescriptify.
More unit tests.
Replace default exports with named to keep API consistent on userland.
Rename classes to improve developer understanding of their functionality when used outside the model workspace.
Types in types.ts to follow TS best practice.
API is exposed via index.ts to facilitate context understanding and control API exposure.
Improve Error messages in AbstractCondition class.
Refactor condition-values registration to allow better file separation.
Rename /condition/valueFrom to reduce ambiguity with /values/valuesFrom.

Type of change

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[X] I have added tests that prove my changes are effective.
[X] New and existing unit tests pass locally with my changes

Checklist:

[X] My changes do not introduce any new linting errors
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[N/A] I have made corresponding changes to the documentation and versioning
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] I have rebased onto master and there are no code conflicts

nice!!! 🚀