Cloud story

Deploy to cloud the first version completed.

Codecov Report

Merging #55 (b951b85) into master (3f99cae) will decrease coverage by 0.24%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master      #55      +/-   ##
==========================================
- Coverage   97.74%   97.49%   -0.25%
==========================================
  Files           6        7       +1
  Lines         310      319       +9
==========================================
+ Hits          303      311       +8
- Misses          7        8       +1

Impacted Files
Coverage Δ

service/config.py
100.00% <ø> (ø)

service/routes.py
98.80% <66.66%> (-1.20%)
:arrow_down:

service/__init__.py
84.21% <100.00%> (+0.87%)
:arrow_up:

Continue to review full report at Codecov.

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


Use new version of s3 library, Fixes #276

This more up-to-date library is missing the GetKey function on a bucket. So I used the GetReader and the error value to test for a Key that exists. I'm not sure how much data is returned this way, so it may be slower than the other library that did a "HEAD" request.
I did take a look at the Amazon library/SDK but the interface looks to be very different. I'll investigate further at another time.

Thanks for your work!
Please ignore build errors for darwin on travis, these are broken at the moment (see #283), the other errors: please fix ;)

I've restarted the travis build, should pass now.

LGTM! :+1:


fix: publicRuntimeConfig

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


FatalError when no attached rules

Is this a really good behaviour?
I have an array of text fields for UIViewController and I iterate over all and run validate() and merging the result of the validation... But if there is no attached rules fatal error is thrown...
Personally, I would suggest returning .valid if there is no rules attached to ValidatableInterfaceElement instead of throwing fatal error. What do you think guys?

Perhaps. I think at the time of writing I considered it a programmer error to validate a field that hasn't any attached validators. You're probably right, .valid is probably a better bet. Perhaps a print message in DEBUG mode?


[BUG] why use RequiredDuringSchedulingIgnoredDuringExecution rather than PreferredDuringSchedulingIgnoredDuringExecution

https://github.com/fluid-cloudnative/fluid/blob/34c48abdad1f99fb76b695d0fad24f6dad3bc558/pkg/webhook/plugins/nodeaffinitywithcache/node_affinity_with_cache.go#L160
why use RequiredDuringSchedulingIgnoredDuringExecution rather than PreferredDuringSchedulingIgnoredDuringExecution? Is this a bug?

dumb question, PreferredDuringSchedulingIgnoredDuringExecution term has been set in https://github.com/fluid-cloudnative/fluid/blob/34c48abdad1f99fb76b695d0fad24f6dad3bc558/pkg/webhook/plugins/nodeaffinitywithcache/node_affinity_with_cache.go#L154
/close


Docs in wordepress

Fewer Jekylls, moar Wordepress.

This is a purely mechanical conversion of the existing docs into the wordepress site format - over to @abuehrle.

I don't mind losing the jekyll stuff, as long as it is replaced by some other reasonably convenient way to preview doc changes.  Can you suggest how that might work?  Ideally it would be some canned mechanism that can be invoked through make site or similar (perhaps using your dockerized wordpress?).

@dpw the plugin is really simple to use and it's very fast.  Changes can be easily viewed on the dev server.  Or yes, you can download the containerized version and build the site locally on your machine.  But, I don't think it's necessary to do that since we do have a dev server setup that lets you preview changes before they get pushed to the live server.
The plugin can be used on the command line and has a pretty extensive list of commands. The only thing you need is a WP login and then API credentials for the plugin, which are generated from your WP user profile, as described here:
https://github.com/weaveworks/wordepress
If you're not building the local version, you can ignore, "Configuring Wordpress" section.

Changes can be easily viewed on the dev server.

For us, yes, but not for any prospective contributors. Though that would be in the "nice problem to have" category at this point.

For us, yes

Not even that, really.  make site comes up quickly enough that editing the docs is only at a small remove from WYSIWYG, and the docs can be previewed in mid-edit.  That seems unlikely to be the case for the dev server.

To be clear, this PR is a WIP to give you early visibility. Some issues:

I didn't rewrite intra-site page links yet to conform to what the wordepress tool expects (an alternative would be to fix weaveworks/wordepress#8)
{:toc} isn't supported by wordepress
Setting up wordpress.local as a solution for local preview is currently an enormous faff - there are some issues (weaveworks/wordpress.local#1, weaveworks/wordpress.local#2) to address this, however the use of commercial plugins means we can't share it externally

In terms of invoking the tool to publish release docs, you're free to do that however you like. Net and Scope will be using CircleCI, you can run it manually if you prefer.

make site comes up quickly enough that editing the docs is only at a small remove from WYSIWYG, and the docs can be previewed in mid-edit. That seems unlikely to be the case for the dev server.

The manual publish I did for you to the dev server took 30s:
$ wordepress --url https://dev-weavewww.pantheonsite.io ... --product flux --version 0.2.0 publish site
2016/04/25 20:07:23 Loading 9 markdown files from site
2016/04/25 20:07:23 Loading page 1 from https://dev-weavewww.pantheonsite.io/wp-json/wp/v2/documentation
2016/04/25 20:07:24 Uploading document: flux-0-2-0-daemon
2016/04/25 20:07:26 Uploading document: flux-0-2-0-edgebal
2016/04/25 20:07:28 Uploading document: flux-0-2-0-fluxctl
2016/04/25 20:07:30 Uploading document: flux-0-2-0-getstarted
2016/04/25 20:07:32 Uploading document: flux-0-2-0-index
2016/04/25 20:07:34 Uploading document: flux-0-2-0-overview
2016/04/25 20:07:36 Uploading document: flux-0-2-0-prometheus
2016/04/25 20:07:38 Uploading document: flux-0-2-0-swarm
2016/04/25 20:07:40 Uploading document: flux-0-2-0-web
2016/04/25 20:07:42 Uploading image: 787e2cea1343facce26cfb4058d240c9bf6b70a7.png
2016/04/25 20:07:45 Uploading image: 96888487dd382b37d445ff632410c3024d7b1b4c.png
2016/04/25 20:07:48 Uploading image: b3e327b721b1ebf9970e1d45fdc782cf9e830ba5.jpg
2016/04/25 20:07:51 Uploading image: 284ec2f6f89c8e2b6166f21c88e2b0e41f568b1d.jpg
2016/04/25 20:07:53 Uploading image: 34937ec1ecf318dbe69b69b63dd9fdb0f0347ec4.jpg

Publish is differential though - here's what happens when I republish to the dev server without making changes to the source:
$ wordepress --url https://dev-weavewww.pantheonsite.io ... --product flux --version 0.2.0 publish site
2016/04/26 11:11:09 Loading 9 markdown files from site
2016/04/26 11:11:09 Loading page 1 from https://dev-weavewww.pantheonsite.io/wp-json/wp/v2/documentation
2016/04/26 11:11:11 Loading page 2 from https://dev-weavewww.pantheonsite.io/wp-json/wp/v2/documentation
2016/04/26 11:11:12 Skipping document: flux-0-2-0-daemon
2016/04/26 11:11:12 Skipping document: flux-0-2-0-edgebal
2016/04/26 11:11:12 Skipping document: flux-0-2-0-fluxctl
2016/04/26 11:11:12 Skipping document: flux-0-2-0-getstarted
2016/04/26 11:11:12 Skipping document: flux-0-2-0-index
2016/04/26 11:11:12 Skipping document: flux-0-2-0-overview
2016/04/26 11:11:12 Skipping document: flux-0-2-0-prometheus
2016/04/26 11:11:12 Updating document: flux-0-2-0-swarm
2016/04/26 11:11:13 Skipping document: flux-0-2-0-web
2016/04/26 11:11:13 Skipping image: 787e2cea1343facce26cfb4058d240c9bf6b70a7.png
2016/04/26 11:11:13 Skipping image: 96888487dd382b37d445ff632410c3024d7b1b4c.png
2016/04/26 11:11:14 Skipping image: b3e327b721b1ebf9970e1d45fdc782cf9e830ba5.jpg
2016/04/26 11:11:14 Skipping image: 284ec2f6f89c8e2b6166f21c88e2b0e41f568b1d.jpg
2016/04/26 11:11:14 Skipping image: 34937ec1ecf318dbe69b69b63dd9fdb0f0347ec4.jpg

weaveworks/wordepress#24 is another possibility for preview that would also satisfy the external contributor requirement.

weaveworks/wordepress#25

Why would an external contributor need to preview the wordpress site?  I just made some PRs to Google and I was only able to preview them in github.   I would think a contributor to our docs would go through a similar process, make a PR, we review it, then it gets merged, and uploaded to the live site from github.  What were you envisioning that's different from this?
I would not be comfortable giving an external contributor any sort of access to our website anyway.

Why would an external contributor need to preview the wordpress site?

@abuehrle Whom are you asking? I would like an external contributor to be able to preview changes they have made to documentation without depending on our installation of wordpress, to be clear.

Can't they do that in Github at the moment? As I said over at google, that's the way it's done. I don't see the final result on their website, just in the github file.

Can't they do that in Github at the moment?

We've tried to make is easy for people (whether ourselves or outside contributors) to build and test code changes, so that it is easy to develop and contribute high-quality code changes.
In a similar way, it should be easy for people to preview doc changes, with a reasonable degree of fidelity, so that it is easy to develop and contribute high-quality doc changes.
The internal vs outside contributors things is a bit of a red herring, in that sense.

@dpw @squaremo some questions:

Is a 5s incremental publish to dev-weavewww.pantheonsite.io a good enough replacement for make site in the short term to get this moving forward?
Net/Scope will have automated publishing of master and feature branches to dev-weavewww and of release docs to weave.works via Circle. What are your thoughts on automated publishing for Flux?

Is a 5s incremental publish to dev-weavewww.pantheonsite.io a good enough replacement for make site in the short term

It might do as a stopgap, but has the drawback of only allowing one person to work on the docs at once, no?

to get this moving forward?

You're talking as if this is your responsibility.  I'm not sure how that became the case, or whether it is beneficial.  If you've provided adequate documentation for wordepress and a working example of its use in the form of weave net, why shouldn't it be up to those working on flux to move things forward?

Net/Scope will have automated publishing of master and feature branches to dev-weavewww and of release docs to weave.works via Circle. What are your thoughts on automated publishing for Flux?

There should be an automated means of publishing the flux docs, in the sense of a command that does it (when supplied with the necessary credentials).  I don't think how that command gets invoked is something to get hung up on (it might be by hand, or as part of a release script, or as part of some CI system).

@squaremo Very minor edits were made here.   I guess I need to rebase to resolve the conflicts and then we can merge.


add expo

[x] rename package.json main

:tada: This PR is included in version 1.58.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


License clarification

Hey, am looking to package this up for fedora so as to update to the latest version of compass (which pulls this in as a test dep). I did not see any reference to license here or in the gem, could you clarify what this is relesed under. Thank you.

Fixed in f683373


why grpc_csharp_plugin generate service code is abstract class, not a interface?

Hi,
grpc_csharp_plugin default  generate service code  is abstract class,.
if i want do some AOP program like this
internal class GrpcInterceptionService<T> : RealProxy
    {
        public GrpcInterceptionService(T instance)
            : base(typeof(T))
        {
            _instance = instance;
        }
    .....

        public TService Create<TService>(TService instance) where TService:class
        {
            var proxy = new GrpcInterceptionService<TService>(instance);
            return proxy.GetTransparentProxy() as TService;
        }
       ....

because of the generate service type is abstract class, will get a runtime error: the type must a interface or extends from MarshalByRef.
Is there any way can generate service as interface?

I found a new point to intercept, wapper the handle when ServerServiceDefinition.Builder AddMethod.


Conversion error in T2Star when calculating p0

Informations

Qiskit Experiments version: main
Python version: 3.8
Operating system: Linux

What is the current behavior?
The bug occurs when the conversion factor is not 1 (for example when the unit is ms), and the user did not set p0 (i.e., user_p0 is None). In _run_analysis we have the lines:
        si_xdata = xdata * conversion_factor
        t2star_estimate = np.mean(si_xdata)

And later in _t2star_default_params we have:
 t2star = t2star_input * conversion_factor

So we've multiplied twice by conversion_factor, resulting in an incorrect p0 and subsequently a failing fit.
Steps to reproduce the problem
Write tests for this case.
What is the expected behavior?
Suggested solutions

Done in #83


Map right alt-gr key

Not really an issue, but just a quick addition didn't seem worth creating a fork and a PR just for this! Since the Palm is US layout, I found it useful to add a mapping at "Space2" for AltGr so I can use the InternationalUS layout to be able to type, for example "£" (Shift-AltGr-4)
€ etc.
US International layout
-#ifdef GERMAN_LAYOUT
+#if defined(GERMAN_LAYOUT)
key_map[0b00110111] = '`';  // "Space 2"
+#elif defined(ALTGR_LAYOUT)

key_map[0b00110111] = KEY_RIGHT_ALT;  // "Space 2"
#else
key_map[0b00110111] = ' ';  // "Space 2"
#endif

There is an actual German version of the PPK, and the AltGr is not on the Space key's location. The GERMAN_LAYOUT macro in my code is to support that keyboard. See the pic attached.
I guess your purpose is to type German characters using a US layout. You can make the change on your side, but I'd prefer not include it in the repo. Sorry...
Another suggestion: when you want to type a space with your right thumb, there is a high chance that it lands on the “Space/New” key due to its position, so mapping it to other keys may hurdle your typing. A better solution would be mapping the Done/Cancel key to AltGr (it was originally mapped to Insert). Then map Fn+ Del to be Insert.

no worries, yes since there wasn't a UK version this is using a US layout keyboard so making use of the "US international" layout makes sense. As for the lcoation, on intl keyboards it's normal for Alt-Gr to be directly to the right of the spacebar so once you have been using the Alt-Gr combos it feels natural being there.
happy to keep the changes local.
I'm having some "lag/double key" issues at the moment to diagnosee, not sure whats going on.


Import bacpac failed - Object reference not set to an instance of object

When trying to import .bacpac file to local SQL server, the process always fails with the following message:
Import bacpac failed  . | MyDb  Error: Object reference not set to an instance of object.
There are no additional details, just this message. I've tried with two different bacpac files, both created from Azure SQL server, both failed with the same error message.
Current workaround: Use SSMS to import bacpac file.

Extension: SQL Server Dacpac (microsoft.dacpac)
Extension version: 1.2.0
Azure Data Studio version: azuredatastudio 1.15.1 (83544eba2504f564d785534cc780d2073feec2cc, 2020-02-15T05:03:16.545Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz (8 x 3408)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.79GB (9.50GB free)

Process Argv

Screen Reader
no

VM
0%

Can you try exporting using the .NET Core version SqlPackage.exe and use the /df parameter to get the log file and please share it? This uses the same backend that ADS uses to do DacFx operations.
Example export command:
sqlpackage.exe /Action:Export /ssn:[server] /sdn:[database] /su:[user] /sp:[password] /tf:"C:\Temp\database.bacpac" /df:sqlpackage-export.log

Hi @kisantia,
The error occurred when importing a bacpac file into SQL server running locally previously exported from Azure SQL, and not when exporting one.
I've thus run the following command:
sqlpackage.exe /a:Import /df:"bacpac.log" /sf:"1-DbAbc.bacpac" /tcs:"Server=.;Database=1-DbAbc;Integrated Security=true"

The log file is attached. The error in the log file:
Microsoft.Data.Tools.Diagnostics.Tracer Error: 0 : 2020-03-23T11:55:13 : SqlPackage failed unexpectedly
 Exception: System.NullReferenceException: Object reference not set to an instance of an object.
   at
[bacpac.log](https://github.com/microsoft/azuredatastudio/files/4369147/bacpac.log)
Microsoft.Data.Tools.Schema.Sql.Deployment.SqlDeploymentPlanGenerator.DeploymentScriptDomGenerator.GenerateFragment(Int32 operation, IModelElement element) in F:\B\16846\6200\Sources\Product\Source\SchemaSql\Deployment\SqlDeploymentPlanGenerator.DeploymentScriptDomGenerator.cs:line 408
   at Microsoft.Data.Tools.Schema.Sql.Deployment.SqlDeploymentPlanGenerator.DeploymentScriptDomGenerator.GenerateSteps(Int32 operation, IModelElement element) in F:\B\16846\6200\Sources\Product\Source\SchemaSql\Deployment\SqlDeploymentPlanGenerator.DeploymentScriptDomGenerator.cs:line 297
   at Microsoft.Data.Tools.Schema.Sql.Deployment.Analyzers.PlanMediator.BuildDependencyOrderedSteps(Int32 operation, List`1 classOrder, List`1 operationOrder, Dictionary`2 changes, Boolean preserveGraphs, Dictionary`2& relating, Dictionary`2& related) in F:\B\16846\6200\Sources\Product\Source\SchemaSql\Deployment\Analyzers\PlanMediator.cs:line 1365
   at Microsoft.Data.Tools.Schema.Sql.Deployment.Analyzers.PlanMediator.BuildOperations() in F:\B\16846\6200\Sources\Product\Source\SchemaSql\Deployment\Analyzers\PlanMediator.cs:line 803
   at Microsoft.Data.Tools.Schema.Sql.Deployment.SqlDeploymentPlanGenerator.OnGeneratePlan() in F:\B\16846\6200\Sources\Product\Source\SchemaSql\Deployment\SqlDeploymentPlanGenerator.cs:line 340
   at Microsoft.Data.Tools.Schema.Sql.Deployment.SqlDeploymentPlanGenerator.GeneratePlan(List`1 drops) in F:\B\16846\6200\Sources\Product\Source\SchemaSql\Deployment\SqlDeploymentPlanGenerator.cs:line 890
   at Microsoft.Data.Tools.Schema.Sql.Deployment.SqlDeployment.BuildPlan() in F:\B\16846\6200\Sources\Product\Source\SchemaSql\Deployment\SqlDeployment.cs:line 1746
   at Microsoft.Data.Tools.Schema.Sql.Deployment.SqlDeployment.Controller.CreatePlan() in F:\B\16846\6200\Sources\Product\Source\SchemaSql\Deployment\SqlDeployment.Controller.cs:line 126
   at Microsoft.SqlServer.Dac.DeployOperation.<>c__DisplayClass16_1.<CreatePlanInitializationOperation>b__1() in F:\B\16846\6200\Sources\Product\Source\DeploymentApi\DeployOperation.cs:line 151
   at Microsoft.Data.Tools.Schema.Sql.Dac.OperationLogger.Capture(Action action) in F:\B\16846\6200\Sources\Product\Source\SchemaSql\Dac\Logging\OperationLogger.cs:line 48
   at Microsoft.SqlServer.Dac.DeployOperation.<>c__DisplayClass16_0.<CreatePlanInitializationOperation>b__0(Object operation, CancellationToken token) in F:\B\16846\6200\Sources\Product\Source\DeploymentApi\DeployOperation.cs:line 104
   at Microsoft.SqlServer.Dac.Operation.Microsoft.SqlServer.Dac.IOperation.Run(OperationContext context) in F:\B\16846\6200\Sources\Product\Source\DeploymentApi\Operation.cs:line 72
   at Microsoft.SqlServer.Dac.ReportMessageOperation.Microsoft.SqlServer.Dac.IOperation.Run(OperationContext context) in F:\B\16846\6200\Sources\Product\Source\DeploymentApi\ReportMessageOperation.cs:line 44
   at Microsoft.SqlServer.Dac.OperationExtension.CompositeOperation.Microsoft.SqlServer.Dac.IOperation.Run(OperationContext context) in F:\B\16846\6200\Sources\Product\Source\DeploymentApi\IOperation.cs:line 63
   at Microsoft.SqlServer.Dac.OperationExtension.CompositeOperation.Microsoft.SqlServer.Dac.IOperation.Run(OperationContext context) in F:\B\16846\6200\Sources\Product\Source\DeploymentApi\IOperation.cs:line 63
   at Microsoft.SqlServer.Dac.DeployOperation.Microsoft.SqlServer.Dac.IOperation.Run(OperationContext context) in F:\B\16846\6200\Sources\Product\Source\DeploymentApi\DeployOperation.cs:line 470
   at Microsoft.SqlServer.Dac.OperationExtension.Execute(IOperation operation, DacLoggingContext loggingContext, CancellationToken cancellationToken) in F:\B\16846\6200\Sources\Product\Source\DeploymentApi\IOperation.cs:line 26
   at Microsoft.SqlServer.Dac.DacServices.InternalDeploy(IPackageSource packageSource, Boolean isDacpac, String targetDatabaseName, DacDeployOptions options, CancellationToken cancellationToken, DacLoggingContext loggingContext, Action`3 reportPlanOperation, Boolean executePlan) in F:\B\16846\6200\Sources\Product\Source\DeploymentApi\DacServices.cs:line 826
   at Microsoft.SqlServer.Dac.DacServices.ImportBacpac(BacPackage package, String targetDatabaseName, DacImportOptions importOptions, Nullable`1 cancellationToken) in F:\B\16846\6200\Sources\Product\Source\DeploymentApi\DacServices.cs:line 2677
   at Microsoft.Data.Tools.Schema.CommandLineTool.Program.DoImportBacpacOperation(CommandLineArguments parsedArgs) in F:\B\16846\6200\Sources\Product\Source\SqlPackage\Program.cs:line 850
   at Microsoft.Data.Tools.Schema.CommandLineTool.Program.PerformAction(CommandLineArguments parsedArgs) in F:\B\16846\6200\Sources\Product\Source\SqlPackage\Program.cs:line 212
   at Microsoft.Data.Tools.Schema.CommandLineTool.Program.Run(String[] args) in F:\B\16846\6200\Sources\Product\Source\SqlPackage\Program.cs:line 156
   at Microsoft.Data.Tools.Schema.CommandLineTool.Program.Main(String[] args) in F:\B\16846\6200\Sources\Product\Source\SqlPackage\Program.cs:line 50

We had a regression in the 18.4.1 release where there's the error “Object reference not set to an instance of an object.” when importing a bacpac with a user with an external login. Can you try importing using SqlPackage 18.4? If that doesn't work, we'll need a bacpac that repros this to debug.

I had the same issue and downgrading to 18.4 worked for me.

Please try upgrading ADS and the dacpac extension. The fix was included in the ADS 1.16 release.

I believe I'm on the latest version and it still fails.
Azure Data Studio:

Version: 1.16.1
Commit: 16801c522df68e63218b39ccb5b4a7431f9bf667
Date: 2020-03-23T22:09:40.250Z
VS Code: 1.42.0
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.3.0-42-generic

Extension:

Name: SQL Server Dacpac
Id: microsoft.dacpac
Description: Manage data-tier applications
Version: 1.3.0
Publisher: Microsoft

@markhobson Looks like it only got fixed for deploying/importing to Azure. We will also fix this for on prem. Thanks!

first fix didn't work, so now targeting to get this fixed for June release.

I don't know if this helps, but we had a similar problem importing a DB from SQL Azure to localdb, and needed to pass in the following parameter into SqlPackage /p:ExcludeObjectTypes="Logins;ServerRoleMembership;ServerRoles;Credentials;Rolemembership;DatabaseRoles;Users"
Our problem was that we have AzureAD roles in the database, and SqlPackage hit an NRE when trying to import it

@MatthewSteeples glad to hear you were able to use a workaround. We have a fix and it will be in the June ADS release and the next sqlpackage release. Previously, sqlpackage would convert AAD users to users without logins when importing to a platform that didn't external logins, but that got regressed late last year.

I don't know if this helps, but we had a similar problem importing a DB from SQL Azure to localdb, and needed to pass in the following parameter into SqlPackage /p:ExcludeObjectTypes="Logins;ServerRoleMembership;ServerRoles;Credentials;Rolemembership;DatabaseRoles;Users"
Our problem was that we have AzureAD roles in the database, and SqlPackage hit an NRE when trying to import it

@MatthewSteeples how did you pass the command into SqlPackage?  I get an error "'ExcludeObjectTypes' is not a valid argument for the 'Import' action." when trying to import my bacpac from an Azure SQL export.
SqlPackage.exe /Action:Import /SourceFile:"[bacpacfile]" /TargetDatabaseName:"[dbname]" /TargetServerName:"[targetserver]" /p:ExcludeObjectTypes="Logins;ServerRoleMembership;ServerRoles;Credentials;Rolemembership;DatabaseRoles;Users"

We're publishing a dacpac file, rather than importing a bacpac
SqlPackage.exe /action:publish /sf:"db.dacpac" /tcs:"Data Source=(localdb)\mssqllocaldb;Initial Catalog=Ledgerscope;Integrated Security=true;" /p:ExcludeObjectTypes="Logins;ServerRoleMembership;ServerRoles;Credentials;Rolemembership;DatabaseRoles;Users"
Hadn't twigged that this wasn't the same use case, but could well be the same issue

We're publishing a dacpac file, rather than importing a bacpac
SqlPackage.exe /action:publish /sf:"db.dacpac" /tcs:"Data Source=(localdb)\mssqllocaldb;Initial Catalog=Ledgerscope;Integrated Security=true;" /p:ExcludeObjectTypes="Logins;ServerRoleMembership;ServerRoles;Credentials;Rolemembership;DatabaseRoles;Users"
Hadn't twigged that this wasn't the same use case, but could well be the same issue

Thanks @MatthewSteeples for the clarification. In my particular scenario of importing the bacpac I ended up downgrading my SqlPackage to v18.4 and the standard import command worked fine.
SqlPackage.exe /Action:Import /SourceFile:"[bacpacfile]" /TargetDatabaseName:"[dbname]" /TargetServerName:"[targetserver]"


Re-enable editor-service tests

I was not able to let them fail locally, so let's see what CI says.
Fixes #170

It's only very occasionally flaky. We might have turned on retries here though, in which case this should be fine.

Superseded by #429


Test for Transcription Locked By Different User

I believe this should take care of the subject locking issue that was first identified in #158
I believe null values were being interpreted as a transcription being locked. The check was:

See if the current transcription's locked_by equals the current user
If it does not equal, show the locked modal

What if locked_by is null, meaning it is available/unlocked?

Check if null equals the current user
It isn't equal, show the locked modal

This doesn't make sense!
This PR should rectify that issue by checking instead if the transcription is locked by a different user and responding accordingly. I'm not positive it should work, as I wasn't able to replicate the issue, but it should take care of what I believe the issue to be.

@shaunanoordin Thanks for finding the issue here! Turns out the fix was easier than I imagined. In the future, will have to remember we have both the login and display_name properties, the latter of which can be changed by the user.

I believe the only item that needs changing is in the checkIfLocked function, so I went ahead and made the switch. I don't think you would be able to trigger that naturally unless someone came in and accessed the transcription while your browser was asleep for some time and your lock timed out.


🐛 Bug fixing: recurring results of the scorecard fuzzing check for go built-in fuzzers

What kind of change does this PR introduce?
A bug fix: issue #2098

[x] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
Under some specific circumstances, the scorecard fuzzing check returns extra recurring fuzzer results for Go built-in fuzzers. See issues #2060 and #2098.
What is the new behavior (if this is a feature change)?**
I am using a hashmap to drop potential duplicated prominent languages (theoretically there won't be any), so it is expected the fuzzing results will be correctly reported without any recurring contents.

[x] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes
Fixes #2098

Special notes for your reviewer
Does this PR introduce a user-facing change?
No.

NONE

@laurentsimon @oliverchang

Could you add some unit tests?

done, please see https://github.com/ossf/scorecard/pull/2101/files#diff-97593777ce7b8e3b0e3c50d4e8ea916cb608d7c8dae398b083bfb737fcd76d1f and https://github.com/ossf/scorecard/pull/2101/files#diff-04e59c883164f265602b7660801c8c9663e4d7787b7b3d649d0114cc6655a1c2


Add New SSGs and New Themes

Add Docurausus (new SSG) and Docusaurus Theme No Style (Theme)
Add Storyblok (new CMS) and Storyblok Gridsome Boilerplate Moon (new Theme)

@slavakurilyak Hi, thank you for the submission and the extra effort to add the new SSG And CMS, appreciate it! While I did merge your PR i've since had to place the theme you submitted into draft mode. This is my mistake, I should not have merged the PR before leaving this feedback.
I feel the submission is a bit overtly commercial in it's current state. Would you be able to make the following changes to the theme in a new PR?
The title should be: "Moon Gridsome Storyblok"
The demo link presents a bit of a problem. This looks like your products main demo page, and it seems likely the content will change over time. It doesnt feel entirely appropriate to link here. Is it possible to link this theme to a demo url specific to the theme?
Please remove the following from the markdown content

Gridsome Website by Storyblok

Check the website [here](https://demo.storyblok.com/).

and

Powered by Storyblok


CU-85zruzxe4_Implement-Project-Summary-Page

Fetch changes on branches:
CU-85zt3h03j - Refactor the Project logic and CU-85zt3h050 - Refactor the Issue logic

Task linked: CU-85zruzxe4 Implement Project Summary Page


Pull request coverage summary

https://github.com/jpb06/coverage-badges-action/issues/5#issuecomment-2171692242

Hi, I'm interested to know if there are any plans to add a feature that allows us to see the change in coverage, comparing coverage in a PR to coverage in the base branch

For more context, something like a percentage increase or decrease is what i mean. It can be probably another table column


Add upstream patch to kubevirt to force hp-volume- pod to respect blockdevices (IcM 467224770)

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
This PR is required because of an IcM (https://portal.microsofticm.com/imp/v3/incidents/incident/467224770/summary) titled KubeVirt hotplug "addvolume" failures
Under moderate to heavy load, KubeVirt hotplug volume attachment event fail. We believe this is the result of a problem with how pods are rendered, there is an upstream fix in the latest version to fix this - https://github.com/cwedgwood/kubevirt/commit/af25759a3205ac3cdf42254923e3e20cf336a4a0.patch
Bug 6839 Nexus Cloud; KubeVirt virt-operator has persistent volume attachment problems with hotplug (azure.com)
Change Log

Add patch for render hp-volume- pod to respect blockdevices (IcM 467224770)

Does this affect the toolchain?

NO
Associated issues

Links to CVEs
Test Methodology

Pipeline run: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=502082&view=results

LGTM


Add - Subtitles / Closed Captions

Guys, "Subtitles / Closed Captions" is missing.

Regards
Ricardo Belo

same problem as #6 and #41

Yes yes i need this feature, pls update


Snap ML C++ API examples

Added three examples for how to use the Snap ML C++ API for tree-based models.

Please add me.


Upgrade Dataproc initialization to latest TF version and TonY version

Upgrade to TF 1.15
Upgrade to latest Tony Client

@bradmiro

@bradmiro may you take a look into this?

Sure, I'll do this this week :)

Fix PR: #724

Fixed in #865


Should we add handling for BigInt or BigNumber?

Pros:

Would solve a set of interesting use-cases
Would fix floating point precision issues

Cons

Implementations will need 3rd party libraries
Big Number and Big Int cannot be represented in JSON. Would need conversion functions to/from string
Would require a new suite of purpose-built operators and functions
The set of use cases it solves likely does not justify the implementation cost and added complexity

Could we implement Big Number math using existing formulas?
Some limited support seems possible.
Here's a sample that registers a function to add two numbers (represented as strings) by:

converting the number strings to arrays using split()
adding the arrays
reducing the arrays to handle the carry using reduce()
convert the arrays back to a string using join():

register("bigAdd",
&(
  split(@[0], "").reverse(@) +
  split(@[1], "").reverse(@)
).reduce(@,
 &{
   result: accumulated.result,
   d: accumulated.carry + current
  }.{
   result: result ~ if(d>=10, d-10, d),
   carry: if (d >= 10, 1, 0)
  },
  {result: `[]`, carry: 0}
).if(carry, result ~ 1, result)
.reverse(@)
.join(@, "")
)

bigAdd(["12345", "6789"]) => "19134"
bigAdd(["2147483647", "97852516353"]) => "100000000000"
A brute force multiplication function could be registered by leveraging bigAdd():
register("bigMultiply",
&{
  a1: rept(@[0] & ",", length(@[1])).split(@, ","),
  a2: @[1].split(@, "").reverse(@)
}.zip(a1,a2)
.map(@, &rept(@[0] & ",", @[1]).split(@, ",")[?@])
.map(@, &reduce(@, &bigAdd([accumulated, current]), "0"))
.reduce(@, &bigAdd([accumulated, current & rept("0", index)]), "0"))

Then bigMultiply(["12345", "6789"]) => "83810205"

Received for information.


equirectangular video does not play properly

Describe the bug
I am using the library for rendering 360 equirectangular videos, unfortunately, I faced some issues, would like to receive a hint from the community.  2 different issues so far I realized.
at first,  when 360 equirectangular video rendered it starts on muted mode. it keeps replaying after it finishes.  Secondly not able to move in the video, only possible with cursors on the left-up side.
Expected behavior
I am expecting to get output like the one in the link.
https://fluidplayer-exads.firebaseapp.com/html/vr.html

Screenshots or Links
If applicable, add screenshots or links to help explain your problem. DO NOT include links to NSFW webpages. Preferably create code to reproduce using https://jsfiddle.net
Desktop (please complete the following information if relevant):

OS: Linux Debian Buster
Browser Chrome
Version [e.g. 22]

Affected version
I am using version 3.0.0

So couple of things here, in order

You are hard linking to v3.0.0 - it's a buggy version and you should use the latest current - http://cdn.fluidplayer.com/v3/current/fluidplayer.min.js. Unless you are absolutely sure you need a specific version, you should never link against it.
You do not need to manually link three.js or panolens - Fluid Player will not use the ones you linked unless loaded before Fluid Player. All the dependencies are dealt with automatically and manual overrides are not recommended for compatibility.
It keeps replaying after it finishes - this issue was fixed with 3.0.2 and does not happen in current.
not able to move in the video, only possible with cursors on the left-up side - how exactly you expecting to move? Please provide more details here. Unless you tell me what you are doing I can not reproduce it.
it starts on muted mode. - I was able to reproduce this, seems to be caused by panolens however. I will look into this. As a temporary workaround, you can use the code bellow. It is important to add the event listener before initializing Fluid Player, it will not work reliably otherwise. This should unmute the video on when player is ready to play. Consider that this will override mute settings from user on page reload, so you should remove this code once an update has been released (I will notify you in this issue).

    var video = document.getElementById('fluid-player-e2e-case');
    video.addEventListener('canplay', function () {
        video.muted = false;
    });
    var instance = fluidPlayer('fluid-player-e2e-case', {
        layoutControls: {
            showCardBoardView: true,
            showCardBoardJoystick: true
        },
    });

3. 3.0.2

Thank you so much sir for your detailed information and your help! I highly appreciate, taking your time helping me out. I am amazed actually :)

I changed version to v3/current it works perfectly, issue gone.

I was doing some workaround with panolens.js, forgot to remove them.

Yes, exactly it does not happen in the current v3 version.

Could you please check this link below, if you play the video you will be able to change your directions, easily move up, down left right or 360 turn around the while video is playing. Once you click you can easily drag-cursor 360.

https://fluidplayer-exads.firebaseapp.com/html/vr.html
But, using the library, it operates differently, what it does stops the video or replays. You cant even drag-cursor unfortunately.
5)Umm, well apparently  panolens.js,  was not causing this. I removed  panolens.js  library from my template still video mode was muted. The code snippet covers the issue right now, it is cool! Thank you for minding for features.
If you can help me out with the fourth statement, it would be grateful. Thanks again!!

I am sorry I still do not follow what you mean with the navigation part. I can drag the video with the cursor just fine on Chrome and FF. Can you elaborate what exactly does not work for you for the navigation part?
What I meant is this is due to Fluid Player and Panolens integration, there is nothing you can do about it. It's a bug and will need to be investigated.

I have recorded a video about the current issue:
https://streamable.com/efwr3k
If you realized I am able to drag with the mouse in the first 20 seconds, after my custom implementation with the library it is only possible with cursors not with mouse. I hope this clarifies my issue
The link where I was able to move with mouse:
https://fluidplayer-exads.firebaseapp.com/html/vr.html

@sabuhish - can you please confirm if the issues is persisting or resolved

Closing this issue since there is no confirmation regarding the issue being reproducible


Try to unlock the keyring collection when it is locked

On KDE plasma, the secret service is initially locked even though the kwallet is unlocked. Currently if spot is the first application to require the wallet it will fail to access it. My current workaround is to open another app that unlock the wallet and then opening spot.
With this PR, spot will try to unlock the keyring collection if it is locked, resulting on a prompt asking the user for the keyring password, if the keyring unlocking fail, the behavior should be the same as before.
It should help #274 at least on the KDE side. I tested this PR on KDE, can you check if no regression are present on GNOME ?

works perfectly, gnome shell shows a prompt if the keyring is locked. its a bit surprising perhaps, but better than what we have now :) i assume it should be possible to customize the prompt, i'll look into it someday


wingetcreate update fails while there is a version number in the zip archive

Brief description of your issue
The zip archive file tssh_0.1.8_windows_x86_64.zip having tssh_0.1.8_windows_x86_64\tssh.exe in it.
The version number 0.1.8 causes wingetcreate update to fail.
Steps to reproduce
wingetcreate update --urls `
https://github.com/trzsz/trzsz-ssh/releases/download/v0.1.8/tssh_0.1.8_windows_x86_64.zip `
https://github.com/trzsz/trzsz-ssh/releases/download/v0.1.8/tssh_0.1.8_windows_aarch64.zip `
https://github.com/trzsz/trzsz-ssh/releases/download/v0.1.8/tssh_0.1.8_windows_i386.zip `
--version 0.1.8 trzsz.tssh

Expected behavior
wingetcreate update success.
Actual behavior
Retrieving latest manifest for trzsz.tssh
Downloading and parsing: https://github.com/trzsz/trzsz-ssh/releases/download/v0.1.8/tssh_0.1.8_windows_x86_64.zip...
Nested installer not found in zip archive: tssh_0.1.7_windows_x86_64\tssh.exe

Environment
Windows Package Manager Manifest Creator v<IP_ADDRESS>

Windows Package Manager v1.5.1881
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.1992
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.20.1881.0

Winget Directories
-------------------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Enabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

@lonnywong,
It looks like the format of the zip installer changed from v0.1.7 to v0.1.8. When I download both, the v0.1.8 has an extra folder enclosing the nested installer. Wingetcreate uses the relative file path of the existing manifest (/tssh.exe) to try to locate the nested installer, but in this case it should be /tssh_0.1.7_windows_x86_64/tssh.exe (enclosing folder changes depending on the installer). As we currently have no way to override the relative file path, I would use the new command to create a new manifest with the correct relative filepath for this package. Then, in the future, the updates should work to point to the correct relative file path (assuming that the format of your zip does not change).

@ryfu-msft Thanks for your help.

Resolved in WinGet-Create <IP_ADDRESS> in PR #415
cc @denelon / @ryfu-msft if this can be marked as completed

@mdanish-kh Thanks.
wingetcreate <IP_ADDRESS> works.


 I'll make this a draft until someone can tell me it works.
Context:
https://github.com/raysan5/raylib/pull/4380#issuecomment-2415575757

@Not-Nik Please, could you take a look at this build.zig improvement?

Both plain build (zig build) and raylib-zig build (as a .zon dependency) work on my MacBook 👍

@sagehane Hi, I tried using your fix; while I am able to compile raylib by running zig build directly in the repository, it is not working for me as a dependency. For some reason, srcDir() seems to repeat the dependency's directory.
Here is a log showing what happens on my machine:
/tmp/raylib-zig-dep-bug % zig build
install
└─ install raylib-zig-dep-bug
   └─ zig build-exe raylib-zig-dep-bug Debug native
      └─ WriteFile raylib.h failure
error: FileNotFound: /tmp/raylib-zig-dep-bug/raylib/raylib/src/raylib.h

Unfortunately, I do not know enough about Zig to help with fixing this problem. I apologize that I cannot do more.
If you would like to test this yourself, I have provided a repo containing this sample project here. I am using Zig version 0.12.1; my build.zig is not configured to work with 0.13.
Please let me know if there is any additional information I could provide. Thanks.

@Yerdun I have a similar repo here configured for 0.13.0. I'm just sharing it here again for more testing.
https://github.com/rstefanic/raylib-zig-build-bug

I knew my incompetence would get me again. @Yerdun, thanks for the feedback. You're using b.dependency unlike us two, which explains why you're hitting an error I overlooked. This is very interesting but I also honestly don't know the appropriate fix here. I don't even know if this is intended behaviour by upstream.
Regardless, I'm starting to think that the use of @src logic is misguided in the first place... I was trying to fix compile errors without having to change the logic left by the people who previously edited the src/build.zig file but it clearly isn't working. If I can't think of a good way to overcome this issue (which I doubt I will), I'll probably consider a patch that removes src/build.zig and merges the content with build.zig. I honestly don't see why two are needed anyways.
The other solution is to keep src/build.zig and let it be "broken" when compiling from src/. (But why?)

I think Ray opposed that some time ago. IIRC we first had a single build.zig and then it was split into two.

@Not-Nik Why build.zig was split into two? Wasn't src/build.zig enough, same as src/Makefile?

After looking at this project some more, if I'm understanding this correctly, it looks like Raygui support is currently attempted by expecting the user to have cloned the Raylib and Raygui repo with a shared parent directory. With what the Zig build system is capable of, this looks like a terrible hack. I should look into that too.

The config option also looks broken at a glance as attempting zig build -Dconfig="-Werror" causes a compile error but something like zig build -Dconfig="-Werror -Wall" doesn't cause a compile error. My guess is that the logic is broken such that it will only work if only 1 cflag is supplied.

Another weird quirk:
https://github.com/raysan5/raylib/blob/99ff770edce0cf025a77f8bf1923b6a072f8a44e/src/build.zig#L88-L110
This implies that the values of config.h would not be passed as cflags if users don't specify any custom build flags.
I'm not too familiar with C but is this intended behaviour? config.h should be sourced anyways, so if cflags aren't capable of overriding them, there's no point. If cflags can override them, I don't see the need for this code.

@Yerdun, I think I figured it out. It's the logic here:
https://github.com/raysan5/raylib/blob/99ff770edce0cf025a77f8bf1923b6a072f8a44e/src/build.zig#L89-L90
readFileAlloc is agnostic of the build system so, given a relative path, it will always try that literal path.
Something like const file = b.pathFromRoot("src/config.h"); worked on my branch where I moved src/build.zig to build.zig.
/// This is low-level implementation details of the build system, not meant to
/// be called by users' build scripts. Even in the build system itself it is a
/// code smell to call this function.
pub fn pathFromRoot(b: *Build, sub_path: []const u8) []u8 {
    return b.pathResolve(&.{ b.build_root.path orelse ".", sub_path });
}

This does suggest that pathFromRoot should be avoided so I'll try to see if there's a better way.

The convention in Zig projects is to have a build.zig in the root dir. All tooling depends on build.zig at root and would be much harder for the package manager to work otherwise.
In the case of big monorepos with multiple components, one reasonable way to handle that would probably be to have a build.zig in the root of each component and have them exposed as packages to the build.zig in the root dir.
$ tree
.
├── a
│   └── build.zig
├── b
│   └── build.zig
└── build.zig

Something like this is perfectly legal and fine, but it seems overkill for the state of raylib as it seems to just be one library.

Well we also have the examples, so it does make sense to split the build file.

Oh, I totally didn't realise that. Then I'll also try to get this PR to a properly working state in parallel with the other one so it's easier to decide. I think it might involve using newer Zig features and hence require bumping the minimum supported version of Zig to 0.13.0, depending on how things go.

@Not-Nik @sagehane Supporting Zig 0.13.0 as the minimum version sounds good to me, it has been out for some time. When is 0.14.0 released?
An unrelated question: can I use build.zig on Windows to cross-build raylin and a C project for Linux and macOS?

When is 0.14.0 released?

Zig's release schedule currently is meant to match that of LLVM (usually lagging by a few months), which releases twice a year.

An unrelated question: can I use build.zig on Windows to cross-build raylin and a C project for Linux and macOS?

Cross-compiling using Zig is usually very easy. One issue for cross-compiling is that, at least on Linux, Raylib depends on libraries that are normally dynamically linked like libX11, OpenGL, Wayland, etc. I'm not quite sure how this works and it's hard for me to test on NixOS as it's not FHS-compliant, meaning all the dynamic libraries Raylib might expect aren't where they normally would be in other Linux distros.

So, I came up with https://github.com/sagehane/raylib/tree/zig_patch_3 for another branch with a different way of fixing build.zig. It uses some comptime magic with a new function in src/build.zig called addRaylibType to circumvent a dumb compile error I've been having when trying to use raylib with b.dependency from other projects. There must be a better way but it'll do for now.
After thinking about it, I've concluded that:

I think it makes more sense to remove the src/build.zig file. The one in the examples dir should stay though.
The addRaylib function seems misguided to me. The idiomatic way to add dependencies using the package manager should be via b.dependency like from the example repo by @Yerdun. It only makes it slightly easier to depend on Raygui.

It's currently useful as a helper function internal to the project, but I don't think it needs to be exposed with pub.
As for Raygui support, I have no idea how to test it as I've only used a small subset of what Raylib offers so far.

@rstefanic

@Yerdun I have a similar repo here configured for 0.13.0. I'm just sharing it here again for more testing.

Thanks for sharing. Can confirm I get the same results when following the instructions in your readme.

@raysan5

An unrelated question: can I use build.zig on Windows to cross-build raylin and a C project for Linux and macOS?

You should be able to. I've been using Zig as my build system specifically to cross compile. That said, I have not tried from Windows -> Linux before, only the other way around.
From my experience, it works quite well, aside from some problems specific to my use cases. For example, I can't get custom cflags to be recognized when cross compiling. I plan on looking into this more and eventually open an issue about it.

@sagehane Thanks a lot for looking into all this. I was actually going to post an issue regarding the "can only use one cflag through build.zig" problem, but you noticed it before I could say anything, haha.
My sample project was also intended to demonstrate the single cflag problem. I actually haven't checked if the program actually works yet, but after I do that, I'll try out the changes proposed in zig_patch_3 and #4393, and see how they go. Will also test if cross compilation takes into account custom cflags passed through Options.config.

@Not-Nik @sagehane Supporting Zig 0.13.0 as the minimum version sounds good to me, it has been out for some time. When is 0.14.0 released?
An unrelated question: can I use build.zig on Windows to cross-build raylin and a C project for Linux and macOS?

Zig is what I use for cross compiling, but I'm on Linux, so it will be cross compiling to Windows.  That may be an easier path compared to the other way around.  I have not tried it for Mac, so I have no experience there.  Although, while the build.zig seems to be influx, I haven't had successful builds as of yet, but normally it does work well.

@sagehane @Not-Nik thanks for the review, merged!

 It doesn't let me set a PR back to draft mode without first deleting it and making a new PR. This PR wasn't truly ready to be merged because it still resulted in a small logic error when supplying -Dconfig (which I found out after undrafting it...), not to mention that there was some broken logic there before I started looking into it.
That said, I think this commit does solve the primary/immediate compile error that users would run into.


CallRinging event not firing

When we do the outgoing call onConnected event is firing without firing the onRinging event thats why call is not adding to the active calls list and CallDisconnect , hold or other function which are using callSid are not working..
showing activecalls :0 .. Anyone other facing the same issue?

Its happening in trial account only not n upgraded account ..


How to activate Local Only mode ? Make it explicit

Link
> URL of the page / pages on which you found the mistake
Description
I read the doc and have no idea how to activate local only mode & can 't find it

Hey @MarArMar. You can find the instructions on how to change to local-only mode in the next section. We'll try to update it to make it more obvious.

Hey @MarArMar. You can find the instructions on how to change to local-only mode in the next section. We'll try to update it to make it more obvious.

Hey @MarArMar. We've added step by step instructions just for local only mode here. There should now be no confusion on how to switch.


Add stylelint-disable note in README.md

I was trying to think of a better example for a rule to break, but as I'm new to stylelint I couldn't find a good one yet, if anyone has a better example feel free to suggest that.
Especially using the media queries (though it is an example in the docs) could be a bit confusing I guess, but yeah, as I'm new to stylelint it was just hard to think of good examples.
It'd be neat to have in here though so no one else submits erroneous issues like I did :).

Coverage remained the same at 99.333% when pulling 283aab61c998f0bf5082c57caa7630f29b9bab71 on emilgoldsmith:patch-1 into 3c43e97f5678ece590cd6a284b0e943fc69073af on styled-components:master.

Especially using the media queries (though it is an example in the docs) could be a bit confusing I guess, but yeah, as I'm new to stylelint it was just hard to think of good examples.

Yeah you're right, I think the media queries might make it too complicated. You could just do this:
Wrong
/* stylelint-disable color-named */
import React from 'react';
import styled from 'styled-components';

const Wrapper = styled.div`
  background-color: red;
`;

Right
import React from 'react';
import styled from 'styled-components';

const Wrapper = styled.div`
  /* stylelint-disable color-named */
  background-color: red;
`;

Also stylelint supports different types of configuration comments from within the css. So it's probably best to change the  'stylelint-disable' heading to 'Turning rules off from within your CSS', as that's what it's called in the stylelint docs. Also it'd be good to add a link to this section of the stylelint docs: https://stylelint.io/user-guide/configuration/#turning-rules-off-from-within-your-css

It'd be neat to have in here though so no one else submits erroneous issues like I did :).

No worries, issues with the docs are also valid issues! And prs like this are definitely appreciated 👍

Just committed changes. I really like your example, both very simple and seems pretty realistic at the same time!
Seems the URL hash doesn't work in my browser anyway which is a bit sad, but people will just have to figure that out I guess.
And yeah maybe I'll find a bit of time outside of work to help with a few of the other issues, especially the ones we encounter while picking up Styled Components for our MVP here at my startup Gnarles :).

Coverage remained the same at 99.333% when pulling 5fa5d72af984e3f618e96282de8b20ab16e65cc9 on emilgoldsmith:patch-1 into 3c43e97f5678ece590cd6a284b0e943fc69073af on styled-components:master.

Seems the URL hash doesn't work in my browser anyway which is a bit sad, but people will just have to figure that out I guess.

Yeah same here.

And yeah maybe I'll find a bit of time outside of work to help with a few of the other issues, especially the ones we encounter while picking up Styled Components for our MVP here at my startup Gnarles :).

That would be awesome!

Thanks @emilgoldsmith for the PR!

No problem @ismay, happy to help!


Latex formatting issue

Latex commands embedded in Lean comments don't always display properly. In particular, issues arise sometimes when using the underscore for Latex subscripts. An example of the issue can be seen in the first level in "Limits" in the following build:
https://wwwf.imperial.ac.uk/~buzzard/xena/rng_20200412/
To reproduce the issue, clone the repository: https://github.com/ImperialCollegeLondon/real-number-game.git. In the src/game/limits, the file L01defs_completeLatex.lean which showcases the issue should be used (instead of the L01defs.lean in which the Latex commands causing the issue were cleared). I noticed the issue when building the Lean-game-maker with both Python 3.7.2 and 3.8.0. The mathlib library was version 3.8.0; npm version 6.14.4.

Thanks! Fixed in the last commit.


dependency about akka in livy-server/livy-croe

The original question is when refreshing page in notebook, an existing session in livy-server won't be killed, even close the page, and this happened in firefox e.t.c(exclude chrome).
I have just fixed this bug with heartbeat mechanism, while merging my code, i found this:

org.scalatra
scalatra_${scala.binary.version}

com.typesafe.akka
akka-actor_${scala.binary.version}

Actually, i used akka, so the true question is if i can't use akka?

Livy was moved to https://github.com/cloudera/livy


Port/Support for Oracle Solaris 11.4 and 11.next

Applies to: Oracle Solaris 11.4 (SunOS 5.11) & 11.next
When running make and sudo make install on Oracle Solaris 11.4 (SunOS 5.11), it gives the error sh: update-inetd: not found [No such file or directory] without trying to use Service Management Facility.
The /etc/inetd.conf is no longer directly used to configure inetd.  Service Management Facility should be used instead.
Information on SMF is available on Oracle Solaris at man smf or online at https://docs.oracle.com/cd/E88353_01/html/E37853/smf-7.html#scrolltoc, and service templates provided by Oracle Corp. are available on Oracle Solaris at man smf_template or online at https://docs.oracle.com/cd/E88353_01/html/E37853/smf-template-7.html#scrolltoc.

It's highly unlikely that we will be able to support SMF, unfortunately, but PR's are much welcome. I don't know about @hb9kns but I know 0 about Solaris.
However, I can tell you that this is a sort-of-bug. sh: update-inetd: not found [No such file or directory]
I'll make a PR to fix it, and you can try that. You won't be able to run gophernicus through SMF, but you can run it directly and leave it in a tmux/screen, for example, or run it in the background.

I don't know about @hb9kns but I know 0 about Solaris.
Feel free to ask me about any OS-spefic issues you encounter with the fix

I'm just going to make a PR to fix it, and write documentation for how to run it with SMF (If possible).

Since Solaris isn't really even supported, I'm changing this to a porting issue.

just a quick question: what is supported?

Based on new information, this issue will be closed.

New information? What new information?

It was just "isn't really supported", If it can still be ported, then I suppose the issue could be re-opened

@AntonMcClure  assigned you ask per request

Name changed to reflect development and testing that will be done on 11.4 and the 11.x beta versions.

@AntonMcClure progress at all?

@AntonMcClure progress at all?

Hi,
I did some changes (https://github.com/augfab/gophernicus/commit/533975afe69e95ea298954679fd1ca6d00b81016) in order to “port” gophernicus to OmniOS, an OpenSolaris-derived system that uses SMF.
I don't have an Oracle Solaris box where I could test these changes, but I guess it could work there without much additional effort.
SunOS omnios.augfab.dev 5.11 omnios-r151034-b3d6e2addc i86pc i386 i86pc

gopher://omnios.augfab.dev serves the default gophermap, /caps.txt, and /server-status.txt:
$ nc omnios.augfab.dev 70 <<</
iWelcome to Gophernicus!	TITLE	null.host	1
i _______               __                      __		null.host	1
i|     __|.-----.-----.|  |--.-----.----.-----.|__|.----.--.--.-----		null.host	1
i|    |  ||  _  |  _  ||     |  -__|   _|     ||  ||  __|  |  |__ --		null.host	1
i|_______||_____|   __||__|__|_____|__| |__|__||__||____|_____|_____		null.host	1
i               |__|		null.host	1
iIf you can see this, it means that the installation of Gophernicus		null.host	1
ion this system was successful. You may now add content to this		null.host	1
idirectory and replace this page.		null.host	1
i		null.host	1
iGeneric information:		null.host	1
i    your ip address: <IP_ADDRESS>		null.host	1
i    server time....: Thu Sep  3 20:42:22 UTC 2020		null.host	1
i    server uptime..: 1 day(s)		null.host	1
i    server version.: Gophernicus/3.1 "Dungeon Edition"		null.host	1
i    server platform: SunOS/5.11 i86pc		null.host	1
i    description....: 		null.host	1
i		null.host	1
iServer configuration:		null.host	1
i    config file....: /etc/inetd.conf		null.host	1
i    server hostname: omnios.augfab.dev		null.host	1
i    root directory.: /var/gopher		null.host	1
i    running as user: nobody		null.host	1
i    output charset.: UTF-8		null.host	1
i    output width...: 67 characters		null.host	1
i		null.host	1
i___________________________________________________________________		null.host	1
i                   Gophered by Gophernicus/3.1 on SunOS/5.11 i86pc		null.host	1
.

$ nc omnios.augfab.dev 70 <<</caps.txt
CAPS

##
## This is an automatically generated caps file.
##

CapsVersion=1
ExpireCapsAfter=1800

PathDelimeter=/
PathIdentity=.
PathParent=..
PathParentDouble=FALSE
PathKeepPreDelimeter=FALSE
ServerSupportsStdinScripts=TRUE
ServerDefaultEncoding=UTF-8
ServerTLSPort=0

ServerSoftware=Gophernicus
ServerSoftwareVersion=3.1 "Dungeon Edition"
ServerArchitecture=SunOS/5.11 i86pc

$ nc omnios.augfab.dev 70 <<</server-status.txt
Total Accesses: 9
Total kBytes: 5
Uptime: 274
ReqPerSec: 0.033
BytesPerSec: 18
BytesPerReq: 512
BusyServers: 1
IdleServers: 0
CPULoad: 0.00
Total Sessions: 1

Sure! Glad you're interested 😃
Are your changes for the build system rewrite visible somewhere?
It's still a bit of a work in progress, I need to test on a freshly installed machine.
cc doesn't exist in OmniOs, so I need to select an other compiler, by running
$ make CC=gcc

And I needed to add gopher to the list of known protocols:
if ! grep -q gopher /etc/inet/services; then
    echo 'gopher 70/tcp' >>/etc/inet/services
fi

Are your changes for the build system rewrite visible somewhere?

Not quite yet, it isn't complete yet, however I do plan to push it to a branch as soon as it has feature parity.

I forgot about this.  I haven't used Solaris for a while now, and I don't really see any use for it, as there are many (better) and fully free operating systems out there which support this program already.  Closing issue.

Can this get re-labeled with "wontfix"?

Can this get re-labeled with "wontfix"?

Thanks for your input @AntonMcClure , but I'm thinking, and recommending that, we leave this open since sometime else might come along so we can bring this to conclusion.
I'm still running on Solaris 10, so these issues don't doesn't affect me personally at this time, and if I eventually move to Solaris 11+ i might try to tackle this.
But ultimately, it's up to @fosslinux to decide.
⛵
.


Added a simple ViewDefinition edit environment

Description of what I changed

Fixes #995
Please note that only the second commit belongs to this PR; the other commit is from another PR under review.
E2E test

TESTED:
Ran the /view functionality in the browser and tested it for some samples. Here is an snapshot generated using the patient_flat ViewDefinition:

Checklist: I completed these to help reviewers :)

[x] I have read and will follow the review process.

[x] I am familiar with Google Style Guides for the language I have coded in.
No? Please take some time and review Java and Python style guides.

[x] My IDE is configured to follow the Google code styles.
No? Unsure? -> configure your IDE.

[x] I have added tests to cover my changes. (If you refactored existing code that was well tested you do not have to add tests)

[x] I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.

[x] All new and existing tests passed.

[ ] My pull request is based on the latest changes of the master branch.
No? Unsure? -> execute command git pull --rebase upstream master

Codecov Report
Attention: Patch coverage is 49.38272% with 41 lines in your changes are missing coverage. Please review.

Project coverage is 50.36%. Comparing base (65cc128) to head (8b23d7f).

Files
Patch %
Lines

.../java/com/google/fhir/analytics/ApiController.java
0.00%
26 Missing :warning:

...com/google/fhir/analytics/view/ViewApplicator.java
86.04%
4 Missing and 2 partials :warning:

...com/google/fhir/analytics/view/ViewDefinition.java
33.33%
2 Missing and 2 partials :warning:

...java/com/google/fhir/analytics/MainController.java
0.00%
4 Missing :warning:

...ava/com/google/fhir/analytics/PipelineManager.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #996      +/-   ##
============================================
+ Coverage     49.56%   50.36%   +0.79%
- Complexity      631      644      +13
============================================
  Files            87       87
  Lines          5201     5266      +65
  Branches        664      669       +5
============================================
+ Hits           2578     2652      +74
- Misses         2362     2366       +4
+ Partials        261      248      -13

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Improve login view

Let me know if there is anything else to improve?

No, looks great! Much better than our current placeholder UI. Thanks for doing that!
I may replace the logo with the official one later though.


two header files missing

I had to add two header files to get BasicXrApp_win32 to compile:
In samples/BasicXrApp/CubeGraphics.cpp:
#include <list>
In samples/BasicXrApp/OpenXrProgram.cpp:
#include <unordered_map>

Addressing this issue in #7

Resolved in #7


Migrate to spine-base dependency instead of spine-client-core + ThrowableMessage

This PR addresses a few things.

The plugins no longer depend on spine-client-core. Instead, they depend spine-base.
The test code depends on spine-testutil-base instead of spine-testutil-core.
ThrowableMessage is used for code generation instead of FailureThrowable.
The generation folder for Spine failures is now set to ... /generated/ (main|test) /spine instead of  ... /generated/ (main or test) /java.
The dependency on tools module is set to 0.9.25-SNAPSHOT (the one, where there is no model compiler already).

The version for base modules is increased to 0.9.29-SNAPSHOT.

@alexander-yevsyukov PTAL.


None interactive openssl cert generation

This is an alternative setup to #11 which still serves the Jupyter notebook over HTTPS but does not require to user to provide the certificate details.

Cool.. I am working on a branch that adds a little.. I will merge yours
first, there should be no conflicts as I am only working on deps
On 1/17/17 4:20 PM, Jonathan J. Helmus wrote:

This is an alternative setup to #11
https://github.com/openradar/AMS_radar_in_the_cloud/pull/11 which
still serves the Jupyter notebook over HTTPS but does not require to
user to provide the certificate details.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openradar/AMS_radar_in_the_cloud/pull/12#issuecomment-273319152,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAyYB446p_0upPhiVtmU4hpD7342oUoSks5rTT65gaJpZM4LmPGw.

Either #11 or this PR should be selected, not both.

I'll have to roll back #11..
I assume you think #12 is superior?
On 1/17/17 4:47 PM, Jonathan J. Helmus wrote:

Either #11
https://github.com/openradar/AMS_radar_in_the_cloud/pull/11 or this
PR should be selected, not both.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/openradar/AMS_radar_in_the_cloud/pull/12#issuecomment-273325549,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAyYB9RDFZJjcCEJ9BsCO0_2W3o2Hf1Jks5rTUUXgaJpZM4LmPGw.

I think #11 will cause less headaches at the course but this PR is more secure but will be harder to get working on some browsers.
See discussion in #10

We can create a second config file which mirrors #11 that we can fall
back on.
I favor https as it is DoE Policy that all sites be HTTPS, although
strictly this is not a DoE site..
On 1/17/17 4:51 PM, Jonathan J. Helmus wrote:

I think #11
https://github.com/openradar/AMS_radar_in_the_cloud/pull/11 will
cause less headaches at the course but this PR is more secure but will
be harder to get working on some browsers.
See discussion in #10
https://github.com/openradar/AMS_radar_in_the_cloud/pull/10
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/openradar/AMS_radar_in_the_cloud/pull/12#issuecomment-273326419,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAyYB8h5vBK7aFfjo7Jf6mHhSrmfTbd-ks5rTUXxgaJpZM4LmPGw.


Enable Metrics status

Enable "status" for metrics of Kruize API and DB methods.
Below is one of the metric which shows both success and failure status.
kruizeAPI_count{api="listExperiments",application="Kruize",method="GET",status="failure",} 7.0
kruizeAPI_sum{api="listExperiments",application="Kruize",method="GET",status="failure",} 0.015359167
kruizeAPI_count{api="listExperiments",application="Kruize",method="GET",status="success",} 3.0
kruizeAPI_sum{api="listExperiments",application="Kruize",method="GET",status="success",} 0.003072515

@kusumachalasani - I couldn't find the metrics API in monitoringAPI.md, can you please point me to the documentation. Also ensure the documentation is updated with these changes and update the tests to check for status.

I will be updating the tests related to this as part of PR #835


chore: add .envrc with nix for dev env

Adds a nix devenv to this grammar, it notably makes it much easier to have autocompletion in scanner.c on macOS (and Linux, even if that was less of a problem)

Thanks!
Just for my reference, is the lockfile required? I merged it but would prefer to ignore it if that works (just so it doesn't wind up out of date if I don't use it).

Not absolutely required but it means reproducibility would be hellish since you wouldn't know what version of tree-sitter people are using

Makes enough sense, thanks for the follow up.


JSoup encodes code fragments

There is no way to turn off EscapeMode when getting HTML out of JSoup. JSoup automatically interprets ASP or JSP code fragments as bad HTML and encodes them. This disables the code so the application wont run. Could there be an exception added to ignore all code fragments for HTML encoding?
eg This ASP <%Response.Write("This is a test") %>
will be changed to <%Response.Write(&quote;This is a test&quote;)%>

Any chance this can be done? I would just like to know either way. Thanks.

(Closing out old, dormant bugs. If you are still impacted by this, please reopen & vote.)


append_nulls and append_trusted_len_iter for PrimitiveBuilder

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I have cases where I've identified a range of numbers to add to a builder. Or a number of nulls to add. My only options with the existing PrimitiveBuilder interfaces is to either (a) add each value/null individually or (b) create a vector containing the numbers/nulls I wish to add, and then use append_values or append_slice.
Both of these extra work than is strictly necessary -- in one case, capacity checks after every value and in the other materializing an intermediate vector.
Describe the solution you'd like

pub fn append_nulls(&mut self, num_nulls: usize) -> Result<(), ArrowError>. Same as calling append_nulls num_nulls times.
pub fn append_trusted_len_iter_values(&mut self, iter: impl IntoIterator<T::Native>) -> Result<(), ArrowError>. Behaves similarly to collecting the items into a vector and then calling append_slice(&vector).
Could also add a version for Option<T::Native>.

Describe alternatives you've considered
As noted in the description -- multiple calls to append_value/append_null (not ideal due to checking the capacity after each call) or creating a vector and then calling append_values or append_slice (not ideal due to the need to collect/materialize values first).

I have a prototype of these methods, that improved the performance of a 2-way merge kernel 20-30% in some 100k element microbenchmarks.


Chore: Add run configurations for tests

Added run configurations for unit and e2e tests, that can directly be accessed from within Android Studio
Updated the readme
Restructured gitignore + add run configurations to git

I wasn't able to run E2E tests, some of them were failing, although the docker container was running fine, but that's not in connection with this PR, I just wanted to mention it.

So you were able to run the E2E tests, but some of them were failing? If so, this is expected, I will fix this in a separate PR.

Is this a general issue or a problem on my side? I'm not exactly sure.

Have you installed the Docker plugin in Android Studio? I think this should fix it :)
Settings > Plugins > Search for "Docker"


bytecode: Refactor tests

Add assumption that x = x at the end of every test,
this cuts down on duplicated assertions and makes
tests easier to read.

I've added a do-not-merge label because this PR has the commit of the last PR in it and will need to be rebased. It's out of date anyway so needs rebasing for that, but I just want to make sure the other commit gets removed.

I've addressed the feedback but now the linter is failing because the test code is duplicating too much. I wrote up a patch to fix this but reading back on @juliaogris previous comment on testing methodology I wanted to discuss before applying it to this PR.

test.patch
I've addressed the feedback but now the linter is failing because the test code is duplicating too much. I wrote up a patch to fix this but reading back on @juliaogris previous comment on testing methodology I wanted to discuss before applying it to this PR.

Ok, sorry it took so long to respond.
I've reworked the tests the way I'd approach it:
https://github.com/evylang/evy/commit/1dfb1fa42067dbbfa328c240500dfe45587fdd47?diff=unified&w=1
Feel free to take this as is without committer reference. However, if this doesn't work for
you at all, maybe just go with whatever you want.
Rationale for the changes made:
I still stand by my previous comment though I understand that others might disagree.
What's tested in another package should not require detailed testing here (parser errors), however, what's essential to this package should be tested in a linear way
so it's easy to spot the root cause.
I've tried all kinds of setup and other breakages and tried to make sure that it is still reasonably easy to point to the root cause with the changes made here.
Let me know what you think. @camh- I hope this represents well enough what we discussed, too.

I've reworked the tests the way I'd approach it:

This lgtm apart from the assert.Equal on the bytecode which is not very readable when instructions are incorrect (which happens often):
        vm_test.go:293: want != got
            &bytecode.Bytecode{Constants:[]bytecode.value{1, 2, 3, 4, 5, 6}, Instructions:bytecode.Instructions{0x0, 0x0, 0x0, 0x0, 0x0, 0x1, 0x3, 0x0, 0x0, 0x2, 0x0, 0x0, 0x3, 0x5, 0x0, 0x0, 0x4, 0x6, 0x7, 0x4, 0x2, 0x0, 0x0}}
            &bytecode.Bytecode{Constants:[]bytecode.value{1, 2, 3, 4, 5, 6}, Instructions:bytecode.Instructions{0x0, 0x0, 0x0, 0x0, 0x0, 0x1, 0x3, 0x0, 0x0, 0x2, 0x0, 0x0, 0x3, 0x5, 0x0, 0x0, 0x4, 0x6, 0x0, 0x0, 0x5, 0x7, 0x4, 0x2, 0x0, 0x0}}

Adding an extra message to the assert:
assert.Equal(t, tt.wantBytecode, bytecode, "\nwant instructions:\n%s\ngot instructions:\n%s", tt.wantBytecode.Instructions, bytecode.Instructions)

creates a more readable error message:
vm_test.go:293: want != got
            &bytecode.Bytecode{Constants:[]bytecode.value{1, 2, 3, 4, 5, 6}, Instructions:bytecode.Instructions{0x0, 0x0, 0x0, 0x0, 0x0, 0x1, 0x3, 0x0, 0x0, 0x2, 0x0, 0x0, 0x3, 0x5, 0x0, 0x0, 0x4, 0x6, 0x7, 0x4, 0x2, 0x0, 0x0}}
            &bytecode.Bytecode{Constants:[]bytecode.value{1, 2, 3, 4, 5, 6}, Instructions:bytecode.Instructions{0x0, 0x0, 0x0, 0x0, 0x0, 0x1, 0x3, 0x0, 0x0, 0x2, 0x0, 0x0, 0x3, 0x5, 0x0, 0x0, 0x4, 0x6, 0x0, 0x0, 0x5, 0x7, 0x4, 0x2, 0x0, 0x0}}

            want instructions:
            0000 OpConstant 0
            0003 OpConstant 1
            0006 OpAdd
            0007 OpConstant 2
            0010 OpConstant 3
            0013 OpMultiply
            0014 OpConstant 4
            0017 OpDivide
            0018 OpModulo
            0019 OpSubtract
            0020 OpSetGlobal 0

            got instructions:
            0000 OpConstant 0
            0003 OpConstant 1
            0006 OpAdd
            0007 OpConstant 2
            0010 OpConstant 3
            0013 OpMultiply
            0014 OpConstant 4
            0017 OpDivide
            0018 OpConstant 5
            0021 OpModulo
            0022 OpSubtract
            0023 OpSetGlobal 0

Can you fixup and then we merge your branch into this one and then merge down to main?

Ok, cool you could take this. I have changed the branch as suggested, but I pulled the
change into a separate assertBytecode - I hope that works for you (made it a bit more readable for me).
BTW, this also explains why you had wantConstants and wantInstructions separately before,
@camh- and I debated this a fair bit: for the reader who comes to it with less prior
knowledge we feel it is easier to understand what's happening if you pull it together
into the wantBytecode field.
🙏

assert.Equal(t, tt.wantBytecode, bytecode, "\nwant instructions:\n%s\ngot instructions:\n%s", tt.wantBytecode.Instructions, bytecode.Instructions)


Add support for comparing complex types (lexicographic comparison)

JAX supports GT/GE/LT/LE comparison for complex types. Since JAX is upstream we need to support it too.

Superseded by #560.


Add a specific step for new Cloud6-7 upgrade

While having this separated, we can still use current cloudupgrade for the original way of upgrade(even for 6-7 path).
This replaces https://github.com/SUSE-Cloud/automation/pull/1223

+1

So its just a different flow in the cloudupgrade function - why duplicating it to another function then?
I added my proposal as PR #1237

:-1:   please do not merge until we decided with PR we take #1234   or  #1237


spec(Document ESM Import Usage)

Document ESM Import Usage
With the introduction of modules in ES6, we have a new and better path to import modules into JS. Jquery-csv should work with this currently but the documentation is lacking a good example for users.
Goals

Document the usage
Provide an useful example that users can use to write their own code

Milestones

[ ] Update README.md - Use Cases to include an ESM Import section
[ ] Add a 'esm-usage' example to /examples

Details
The following code should work to import jquery-csv.
import {$,jQuery} from 'jquery';
import {$.csv ,jQuery.csv } from 'jquery-csv';

References

readme improvement on nodejs - #68

Hello Evan! I am interested in your project and would like to make my first open source contribution to this issue.
For the 'esm-usage' file, should it be similar to the node-usage.js file where example code is simply shown? Or should it be similar to the .html files where some example is demonstrated. I'm not too sure what an example of importing would ultimately demonstrate in a file similar to the html examples.


ElastAlert Kibana Plugin

Kibana is a fantastic frontend for ElasticSearch, yet ElastAlert still runs trough file editors. We have built a plugin for Kibana, so you can edit, run, test and save rules in a single interface.

It also builds a Docker container that runs ElastAlert and a RESTful wrapper.

I have tried to install this for kibana 5.x as well and have not had success. Does this only work for kibana 4.6?

Hi All,
I am getting following error while opening elastalert in kibana 5.3.
Fatal Error
Uncaught ReferenceError: require is not defined (http://XX.XXX.XX.XXX:5601/bundles/elastalert.bundle.js?v=14823:12)
Version: 5.3.0
Build: 14823
Error: Uncaught ReferenceError: require is not defined (http://XX.XXX.XX.XXX:5601/bundles/elastalert.bundle.js?v=14823:12)
at window.onerror (http://XX.XXX.XX.XXX:5601/bundles/commons.bundle.js?v=14823:69:13503)
is there any workaround for it?

Any solution for the above problem ?

Can you fix this plugin for kibana 5.4.0?

Please open an issue on https://github.com/bitsensor/elastalert

Does anyone have an idea when this will be fixed?

We have made the CI in such a way that only Kibana 4 plugins are outputted. We apply a patch that does this. Feel free to open a PR that generates multiple artifacts in CI.
A new release of the Kibana plugin is also underway.

I don't exactly understand what I can do to make it work? I built on top of this fork to make it work with my kibana 5.4.1 instance: https://github.com/Baoban/elastalert-kibana-plugin
However, I'm still new to docker. I guess I need to build on top of the official kibana 5.4.1 docker image with my own Dockerfile that the installs the plugin and creates an image from it. I would like to have such an image available on the default public docker repository so I don't need to build it myself everytime.
Then, on the other side, the nodejs server part is needed. For this, I need to clone your repo and build the docker container myself. I also would like to have this available on the default docker repositories. Maybe you could automate the release of a new feature version of the elastalert-server part for the kibana plugin, everytime a new version of ElastAlert is released?


update capi update metric docs

update capi update metric docs
show 'isVerified' as an option in request body for metric update endpoint
ghstack-source-id: ed27824ac781d187bd2c2ee62d022864011689a2
Pull Request resolved: https://github.com/statsig-io/docs/pull/1260

🚀 Deployed on https://65c55d25945b3c2acad80eea--cozy-fox-0defec.netlify.app


thank you

bro how ı can run a tool . do you can reply my message thank you

You can use your hosting if have purchased or you can use any free hosting like infinityfree and all.


Rebuilding the index takes a long time to wait

Reconstruction of the index need to wait too long, can take some optimization measures

I'm sorry, wrong place should O1


tensorflow.python.framework.errors_impl.NotFoundError: /usr/local/lib/python3.6/dist-packages/delta/layers/ops/x_ops.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _ZN4absl14ascii_internal13kPropertyBitsE

import delta.compat as tf
2021-01-18 02:39:57.852065: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/python/compat/v2_compat.py:96: disable_resource_variables (from tensorflow.python.ops.variable_scope) is deprecated and will be removed in a future version.
Instructions for updating:
non-resource variables are not supported in the long term
from delta.models.base_model import Model
from delta.models.base_model import Model
from delta.layers.utils import get_seg_mask_from_token_idx
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.6/dist-packages/delta/layers/init.py", line 27, in
from delta.layers.common_layers import *
File "/usr/local/lib/python3.6/dist-packages/delta/layers/common_layers.py", line 21, in
from delta.data.feat import speech_ops
File "/usr/local/lib/python3.6/dist-packages/delta/data/feat/init.py", line 17, in
from delta.data.feat import speech_ops
File "/usr/local/lib/python3.6/dist-packages/delta/data/feat/speech_ops.py", line 26, in
from delta.layers.ops import py_x_ops
File "/usr/local/lib/python3.6/dist-packages/delta/layers/ops/py_x_ops.py", line 38, in
gen_x_ops = tf.load_op_library(path)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/load_library.py", line 58, in load_op_library
lib_handle = py_tf.TF_LoadLibrary(library_filename)
tensorflow.python.framework.errors_impl.NotFoundError: /usr/local/lib/python3.6/dist-packages/delta/layers/ops/x_ops.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _ZN4absl14ascii_internal13kPropertyBitsE

Ubuntu 18.04.5 LTS, dokcer, python-3.6.1, tensorflow-gpu-2.3.1, cuda-10.1, no cudnn

I changed
from delta.layers.ops import py_x_ops
to
from delta.core.ops import py_x_ops
and copied core directory from github source.
Also I ran build bash file in core/ops directory.


How can I build the codes and access in browser using ide in real time?

Why IDE and build tool do you use? thanks.

My Mind is an old app that does not really use any build tool. Its "source" files are simply concatenated. Just run make in your terminal to build the app (and use any text editor to edit the code).


Compiling smart contract not getting done

I have a setup of multi-node of 4 peers blockchain. Blockchain is running, few days before I deployed the smart contract and using it to save data on the blockchain. Now, I did some changes to the smart contract by changing the multi-index table name. Trying to compile and deploy this smart contract again. On the local machine, it gets compiled successfully. But on the server-side, the compilation is not getting finished with no error. It looks stuck (blinking cursor). What may be the reason? Let me know if anybody needs more information. Compilation command:
eosio-cpp --abigen commoncontract.cpp -o commoncontract.wasm

And
EOSIO.CDT 1.6.3

Smart contracts are in hpp files saved in /usr/opt/eosio.cdt/1.6.3/include/contracts/ and included in smart contract as header files. Smart contract is:
#include "contracts/group.hpp"
#include "contracts/user.hpp"
#include "contracts/document.hpp"

extern "C" {

        [[eosio::wasm_entry]]
                void apply(uint64_t receiver, uint64_t code, uint64_t action) {
                        if (code == receiver) {
                                switch (action) {
                                        EOSIO_DISPATCH_HELPER(group, (upsertgroup)(erasegroup))
                                        EOSIO_DISPATCH_HELPER(user, (upsertuser)(eraseuser))
                                        EOSIO_DISPATCH_HELPER(document, (upsertdoc)(erasedoc))
                                }
                        }
                }
}

Also when I tried multiple times, temporary wasm files get generated like commoncontract.wasm.tmp3f5048b. Please help.

Can you please share the OS version, memory size, and # of cores?

OS: Ubuntu 18.04, Memory : 7.79 GiB, CPU Cores : 2


Use Official Highcharts NPM Package

Looks like HighSlide Software is adding Highcharts to NPM: https://github.com/highslide-software/highcharts.com/issues/3616
The package is here, but appears to still have some issues. Once resolved, it would be great if react-highcharts could use the official package instead of bundling Highcharts.
I'd suggest making Highcharts a peer dependency.

+1
Let me know if you want to follow up on that and do a pull request, otherwise I'll do it in the one of the upcoming weekends.

@kirjs - I'm unlikely to find the time to make a PR, but I'll let you know if that changes.
As I mentioned, the NPM release isn't ready for prime-time yet. Hopefully they'll fix it quickly, but we'll see.

I'm actually using browserify, which could probably make it work with AMD too, will take a look if I'll have some time this weekend

Resolved by #20


Support for .drawio files with multiple pages

Hi! I love being able to see my flowcharts from Obsidian, however, I generally tend to use a single .drawio for each project, and have multiple pages inside that .drawio file for different stuff.
Right now it doesn't seem possible to view another page other than the first one in the .drawio file.
Is support for this planned? Or is it already there and I just can't find it? Thanks

if multiple page/tabs exist in a draw.io document, after a first read from the plugin, only the first page/tab is saved. all others pages disappeared.


preview website when linking

I don't know how to test this without commit the changes

Personally I've been attempting to setup a preview project with: https://heroku.com with realtime deployment from Github source.
Setup Guide: https://www.freecodecamp.org/news/how-to-deploy-a-nodejs-app-to-heroku-from-github-without-installing-heroku-on-your-machine-433bec770efe/
Unfortunately I came across an issue that may be related to not specifying the Node.js version on build?
https://devcenter.heroku.com/articles/nodejs-support#specifying-a-node-js-version

Thanks for trying to test. Anyway, it is a pull that doesn't introduce harmful changes. I will commit it and do life testing.

It seems to work:


Unclickable window

Description of the new feature / enhancement
A feature that allow user to select a windows to make it transparent/unclickable. Maybe implementing a control panel to re-enable clicking the window.
Scenario when this would be used?
A feature that allow user to select a windows to make it transparent/unclickable would be a good implementation to complement AoT feature (Ex. by playing games that requires mouse and while playing AoT videos)
Supporting information
Many users are requesting this feature on browser forums like chrome or mozilla. There are some alternatives for this feature but are difficult to implement in most cases:

https://jacks-autohotkey-blog.com/2016/07/22/the-winset-exstyle-command-for-mouse-click-transparent-windows-intermediate-autohotkey-tip/
https://superuser.com/questions/1614211/win-10-how-can-i-make-an-window-be-always-on-top-but-non-interactive-clicks-go
https://www.reddit.com/r/AutoHotkey/comments/ajx7og/help_make_a_window_on_top_opaque_and_not_clickable/

This is equivalent to another issue, and I believe Clint has chosen not to build this feature.
/dup #15767


[Errno 32] Broken pipe

Expected Behavior
Bot running without errors.
Actual Behavior
Bot seem to be running fine but after some time i get [Errno 32] Broken pipe
Your FULL config.json
Tryed latest map config file still same error.
Output when issue occurred
random lines with the error message:
[2016-09-02 00:47:00] [MainThread] [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Gyarados appeared! [CP 1438] [NCP 0.54] [Potential 0.87] [A/D/S 15/11/13]
[2016-09-02 00:47:03] [MainThread] [PokemonCatchWorker] [INFO] [vip_pokemon] This is a VIP pokemon. Catch!!!
[2016-09-02 00:47:06] [MainThread] [PokemonCatchWorker] [INFO] [threw_berry] Threw a Razz Berry! Catch rate with Pokeball is now: 10.30
[2016-09-02 00:47:11] [MainThread] [PokemonCatchWorker] [INFO] [threw_pokeball] Great Curveball throw! Used Ultraball, with chance 19.90 (6 left)
[2016-09-02 00:47:12] [MainThread] [PokemonCatchWorker] [INFO] [pokemon_capture_failed] Gyarados capture failed.. trying again!
[Errno 32] Broken pipe
[Errno 32] Broken pipe
[Errno 32] Broken pipe
[2016-09-02 00:47:25] [MainThread] [PokemonCatchWorker] [INFO] [threw_pokeball] Great Curveball throw! Used Ultraball, with chance 13.27 (5 left)
[2016-09-02 00:47:25] [MainThread] [PokemonCatchWorker] [INFO] [pokemon_capture_failed] Gyarados capture failed.. trying again!
[2016-09-02 00:47:36] [MainThread] [PokemonCatchWorker] [INFO] [threw_pokeball] Great throw! Used Ultraball, with chance 13.27 (4 left)
[2016-09-02 00:47:37] [MainThread] [PokemonCatchWorker] [INFO] [pokemon_capture_failed] Gyarados capture failed.. trying again!
[2016-09-02 00:47:46] [MainThread] [PokemonCatchWorker] [INFO] [threw_pokeball] Nice Curveball throw! Used Ultraball, with chance 13.27 (3 left)
[2016-09-02 00:47:46] [MainThread] [PokemonCatchWorker] [INFO] [pokemon_caught] Captured Gyarados! [CP 1438] [NCP 0.54] [Potential 0.87] [15/11/13] (258/800) [+120 exp]
[2016-09-02 00:47:46] [MainThread] [PokemonCatchWorker] [INFO] [gained_candy] You now have 602 Magikarp candy!
Steps to Reproduce
Run bot for some time with movetomap and social enabled.
Other Information
OS: Mac OSX
Branch: 7f53d653
Git Commit: dev

I can confirm, I am seeing the same behavior in dev and master. I've done a fresh clone, as well as submodule update.  I've been using the bot since the early days so i've done it manually, as well as using . setup -r.  I'm 100% confident the bot is set up correctly
Python 2.7.9
CONFIG.JSON -> http://mediaengine.org/PokemonGo-Bot/web/config.json
Error info:
`[Errno 32] Broken pipe
<IP_ADDRESS> - - [02/Sep/2016 13:36:15] code 400, message Bad request syntax ('\x81\xfe\x03\xbfc\xc70\xbdW\xf5k\x9f\x01\xa8D\x87\x01\xb5_\xdc\x07\xa4Q\xce\x17\xe5\x1c\x9d\x18\xe5Q\xde\x00\xa8E\xd3\x17\xe5')
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/eventlet/greenpool.py", line 82, in _spawn_n_impl
func(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/eventlet/wsgi.py", line 719, in process_request
proto.init(sock, address, self)
File "/usr/lib/python2.7/SocketServer.py", line 655, in init
self.handle()
File "/usr/lib/python2.7/BaseHTTPServer.py", line 342, in handle
self.handle_one_request()
File "/usr/local/lib/python2.7/dist-packages/eventlet/wsgi.py", line 351, in handle_one_request
if not self.parse_request():
File "/usr/lib/python2.7/BaseHTTPServer.py", line 286, in parse_request
self.send_error(400, "Bad request syntax (%r)" % requestline)
File "/usr/lib/python2.7/BaseHTTPServer.py", line 368, in send_error
self.send_response(code, message)
File "/usr/lib/python2.7/BaseHTTPServer.py", line 385, in send_response
self.log_request(code)
File "/usr/lib/python2.7/BaseHTTPServer.py", line 422, in log_request
self.requestline, str(code), str(size))
File "/usr/lib/python2.7/BaseHTTPServer.py", line 458, in log_message
format%args))
File "/usr/lib/python2.7/codecs.py", line 351, in write
data, consumed = self.encode(object, self.errors)
File "/usr/lib/python2.7/encodings/utf_8.py", line 16, in decode
return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode byte 0x81 in position 38: invalid start byte
The bot continues to run, but does no work. It continues to do this until stopped and restarted, which at that point it is alright again.`

I am getting this also. With default configuration file. This is on rasbian.

@mitcht what does FollowCluster exactly?

I am still having this problem

This is the error I am getting ..
[Errno 32] Broken pipe
[Errno 104] Connection reset by peer
Any ideas?

Same problem here, also running on raspian.

@mitcht I don't have errorno 32, but "[Errno 104] Connection reset by peer", the rest of the error message is very similar to yours: https://github.com/PokemonGoF/PokemonGo-Bot/issues/4718#issuecomment-244913275
Bot is able to run but I wonder what is causing these errors

After the error occurs sometime the bot will freeze or just walk


How can I learn the program design of this project?

Hi, I am developing my own pictionary game, I have made little progress but I found my code are a bit messy and hard to proceed to develop. I think the main reason is my weakness in designing the program and arrange the modules, therefore I am trying to find others work for reference, so that I can figure out how to design my code better and continue to develop the game.Here I find your great work!
Might I ask you some question about how you start your project and how your design your code?  The questions might be to big to answer...😂 Anyway, any sharing of ideas or resources would be appreciated.

Sure you can ask me your questions