SqlAG/SqlAGReplica: NT AUTHORITY\SYSTEM does not exist on localized operating systems

Details of the scenario you tried and the problem that is occurring:
On for example Swedish operating system, the 'NT AUTHORITY\SYSTEM' is not named that. but instead named 'NT instans\SYSTEM'. This will make the resource that are hard-coded to that name fail.

The DSC configuration that is using the resource (as detailed as possible):
n/a
Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
n/a
What module (SqlServer or SQLPS) and which version of the module the DSC Target Node is running:
n/a
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
Dev

@johlju, Is this only a problem with accounts beginning with NT AUTHORITY? Or does this problem extent to accounts beginning with NT SERVICE? Or is it just well known accounts that are localized?

Not sure. I will try to verify that this weekend. Not sure if I have that server left in my lab, so might need to provision a new one, maybe I will setup a Spanish-server and verify it there.

@randomnote1 I having trouble to replicate this on a Spanish. 🤔 Bad of me not writing more information in the issue. 😞 The image above is from the Swedish OS, but I thought that Spanish OS might be more used together with SQL Server. Swedish Server OS is rarely used, unless for Remote Desktop Server.
This is how it looks like on a Spanish OS.

I'm gonna try another shot to reproduce this. But gonna take a week or so.

I had found an article on Friday with a listing of several localized NT AUTHORITY\SYSTEM variants, and now I can't find it! I did find a similar one here. It appears the Spanish version is the same as English! It looks like German and French are some other options for testing.

@randomnote1 I will try this on a German OS and SQL instance then :)

Finally I had time to reproduce this on a German Windows Server 2016 with German SQL Server 2017.
It seems NT AUTHORITY\SYSTEM is the only account affected. NT SERVICE is named the same on the German OS. But as you can see below, the NT AUTHORITY\SYSTEM is named NT-AUTORITÄT\SYSTEM


Error detection with klog binary

Using the language server, parse output from klog to display errors in documents.
Blocked by #7.

Progress on this feature in branch feature/language-server, but progress is hindered by https://github.com/jotaen/klog/issues/64.
I could possible work around this by saving data to temporary file, and execute klog with that temporary file.

As a workaround for above problem, in c4b031b206f50bf2a823113349216d812ae24478 the document text it put in a temporary file, which is used with klog json <temp_file>. This means error reporting is implemented.

Only code cleanup is remaining.
WIP: klog binary is hard-coded as of now.

This is mostly finished. It is only the larger structuring of the language server which is left (settings, etc).
I am closing this, as a working error detection is implemented in the branch feature/language-server. Still blocked by #7.


Godot Wont Open

Godot version
4.0 alpha3
System information
Windows 10
Issue description
i have gt 530 gpu
and amd procesor
Problem
Godot Engine v4.0.alpha3.official.256069eaf - https://godotengine.org
vkEnumerateInstanceVersion not available, assuming Vulkan 1.0.
it does not say i have to upgrade my driver its just stuck
Minimal reproduction project
No response

here

this is related to the driver, not the engine, try updating them
https://developer.nvidia.com/vulkan-driver

NVIDIA Fermi GPUs do not support Vulkan (updating the driver won't fix that). Until an OpenGL renderer is added to Godot 4.x, you'll have to stick to Godot 3.x.


fix respawn bug

Fix respawn bug by resetting CBRN_chemicalDamageHandler to -1 on respawn.

Published. Thank you


Speeding/parallelising CF creation and initialization

Is the an existing scheme create/initialize entire projections? Say I have 10 projections, is there a way to create them in parallel if the don't depend on each other?

That's always been on the road map but we never did get around to it. The main issue was reproduce ability because the weights should be the same no matter the instantiation order. That issue is now solved so in theory it should now be straightforward to implement but no one has tried to do so yet.

I see. That would be by parallelising _create_cfs()?


fail to start due to some errors

error:
Warning: Failed searching manifest: Failed downloading https://nvim-neorocks.github.io/rocks-binaries/dev/manifest-5.1 - failed downloading https://nvim-neorocks.github.io/rocks-binaries/dev/manifest-5.1

        Error: No results matching query were found for Lua 5.1.
        To check if it is available for other Lua versions, use --check-lua-versions.
        Failed installing lsp-ai.nvim with `luarocks`.

        --------------------------------------------------------------------------------

        Trying to build from source.
        Missing dependencies for lsp-ai.nvim scm-1:
           nvim-lspconfig >= 0.1.6 (not installed)

        lsp-ai.nvim scm-1 depends on lua >= 5.1 (5.1-1 provided by VM: success)
        lsp-ai.nvim scm-1 depends on nvim-lspconfig >= 0.1.6 (not installed)
        Installing https://luarocks.org/nvim-lspconfig-0.1.8-1.src.rock

        nvim-lspconfig 0.1.8-1 depends on lua >= 5.1 (5.1-1 provided by VM: success)
        No existing manifest. Attempting to rebuild...

Looks like lua version is hardcoded to 5.1. But I use 5.4.
And this link :https://nvim-neorocks.github.io/rocks-binaries/dev/manifest-5.1 is a 404 page.
Looks like you are using luarocks, a package manager. what benifits it brings?

It does try build with lua 5.4 but I’ve found the fairies is not actually related to lua, but that the rocksoec refers to the ‘doc’ folder which doesn’t exist.
Removing this line (or creating the folder) sorts the build. You can do it manually from the downloaded package.

acknowledged, will add empty doc dir. thank @nickdedekind


Provide Product Information CSV with stable URL for latest data

Thank you for providing the CSV . It would be great if there were a stable URL that we could use to download this file automatically and get the latest updates. The current URL has the release date in the file.
Providing a URL like: https://download.microsoft.com/download/.../Product names and service plan identifiers for licensing vLatest.csv and including the Last-Modified header in the response would allow applications to cache and update this data automatically. Otherwise, administrators still need to check this page to see if it has been updated.

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

ID: 7c245452-161d-89a1-c026-99a0721cf071
Version Independent ID: 16dec59a-eb83-1837-4efd-4645e7aeaed2
Content: Product names and service plan identifiers for licensing - Azure AD
Content Source: articles/active-directory/enterprise-users/licensing-service-plan-reference.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @Nicholak-MS
Microsoft Alias: nicholak

Hi @iinuwa the next time we rev this file we can evaluate stabilizing the URL to not include the version. Not sure when that will be since the updates are not predictable, but will address this in next round of updates. Thank you for the feedback.
Thanks
Nick


Upgrade to terraform 0.12

Hello there!
I hope PRs are welcomed :)
Currently, the Terraform code is written using version 0.11 and is therefore not compatible with newer versions of Terraform. This PR refactors the code to work with 0.12.

Hi @eana, PRs are welcome! I haven't yet migrated to v0.12 yet, but I will review this very soon.

@isaaguilar Is there anything I can do to get this moved forward? Thanks!

Thanks @eana looks good. Had to find some time to catch up on v0.12 since we're still not using where I work.

@isaaguilar Thank you for reviewing and merging the PR.


Crash: unterminated string constant

The attached code tricks the preprocessor into producing a file with an unterminated string constant, which results in a null pointer access in Parser.stringLiteral. The macro on line 6 only contains a \r between the closing paren and vers - be careful saving it with a text editor since it may "helpfully" do newline conversion for you and prevent the crash (filename ends in .txt since Github won't let me use a .c extension)
unterminated string.txt
arocc -E output:
void foo ( void ) { "
}

clang treats \r, \n, and \r\n as possible line endings (Lexer::LexTokenInternal in clang/lib/Lex/Lexer.cpp). Updating Tokenizer to behave the same way makes this crash go away but I'm not sure if that's the right solution or just hiding the problem.

\r was the line ending in a bunch of old sytems so adding it is valid, though I highly doubt it will ever be encountered in a real world situation.

Regressed by #65.

This no longer crashes.


Update: Movielens 1m posters&urls dataset now available

Hi Thomas,
I've used and updated your code to scrape all the Movielens posters and urls for the one million dataset (find here: https://grouplens.org/datasets/movielens/1m/). I used pandas for reading the original movies data from the dataset (movies.dat), but other than that the libraries used are the same.
Thanks and kind regards,
Louis
code and datasets are available here:
ml-1m.zip

Hi Thomas,
I've used and updated your code to scrape all the Movielens posters and urls for the one million dataset (find here: https://grouplens.org/datasets/movielens/1m/). I used pandas for reading the original movies data from the dataset (movies.dat), but other than that the libraries used are the same.
Thanks and kind regards,
Louis
code and datasets are available here:
ml-1m.zip

Thank you! It is very useful for my current research! @LouisdeBruijn @babu-thomas


404 not found on initial install/redirect

I think I've followed the instructions for the basic setup, but when i go to http://localhost:8080 it redirects me to http://localhost:8080/authorized/cases/path/ and that gives a 404 error.
Should I be running something alongside this to serve cases or similar?

Hi, as noted in the main readme:

We are currently unable to maintain compatibility of this deployment example. We are preparing advanced architecture, viewer features and deployment options.

The URL you are seeing is actually a valid URL for the new case viewer we've developed. I will try to update the demo as soon as I can, hovewer, I am afraid our priorities are for now elsewhere. I am actually surprised you are getting the redirection, that should not happen - I can at least try to figure out where that is comming from.
If I can help with anything regarding the deployment, please, let me know.

Also, if you would like to try out the new infrastructure, we have a public testing deployment available. If you are interested in trying that, I can arrange a setup - just PM me. Since we are unable to provide an open demo for now here, and since you are showing an interest in our work ❤

whatever WSI server you are using

We don't have a server preference at the moment - which is why I'd gone for the whole package deployment over just the viewer. I'd actually tried building your server and your viewer in my own docker-compose additions to your php version before discovering this repo.
I had to steal the tip on using a lower version of minizip-ng and the cmake settings from your compose file, but interestingly that's giving a different error - instead booting to a data not found screen:

I'll ping you a PM though, as a demo would be great.

Actually - I tried to implement your iipsrv branch, and have just noticed your WSI-Service - that looks like a great option if it'll pair with the xopat viewer?

Yes, the newer server we try to maintain should be much more feature-rich option. Please check here https://github.com/RationAI/xopat/issues/54.
The 'error' you are seeing is just the viewer complaining that you did not say what to view. To do so, you have to

provide the session configuration manually, e.g. via dev_setup
use a different app that will generate these sessions for you

e.g. build a script that generates a HTML page with links that contain the data specification

This is actually the only blocking chain piece that I am missing in deploying a V2 version of the docker compose demo. We do have a viewer able to open xOpat sessions, but it cannot talk to the WSI Service yet.

I've had a look at /dev_setup and while I've added my images to /images/ in the docker container and have /images/1.ndpi as my test case I'm not sure quite how to change the json to get it pointing at my example folder or file? The below doesn't work:
{     "params": {         "customBlending": true     },     "data": [],     "background": [         {             "dataReference": 1,             "lossless": false         }     ],     "visualizations": [         {             "name": "A visualization setup 1",             "lossless": true,             "shaders": {                 "shader_id_1": {                     "name": "Layer 1",                     "type": "identity",                     "visible": 1,                     "fixed": false,                     "dataReferences": ["/images/1.ndpi"],                     "params": { }                 }             }         }     ] }
it says:
JSON parametrization of the visualizer requires valid dataReference for each background layer. Invalid data reference value '1'. Available data: []
I think perhaps I'm swimming in water too deep for me, and I'll need to wait for an idiots guide. I got the Mirax server running from the code here - so if there's a way to upload and then view based on that?

You are saying that the background image should be a data of index 1. You provide data "data": [].
Correct JSON:
{
    "params": {
        "customBlending": true
    },
    "data": ["images/1.ndpi"],
    "background": [
        {
            "dataReference": 0,
            "lossless": false
        }
    ]
}

If you want to create visualization layer, you can do:
{
    "params": {
        "customBlending": true
    },
    "data": ["images/1.ndpi"],
    "visualizations": [
        {
            "name": "A visualization setup 1",
            "lossless": true,
            "shaders": {
                "shader_id_1": {
                    "name": "Layer 1",
                    "type": "identity",
                    "visible": 1,
                    "fixed": false,
                    "dataReferences": [0]
                }
            }
        }
    ]
}

However, this supposes that

the viewer is set up correctly, e.g. env.json knows how to ask the server for data (WSI Service has the module enabled and configured)
the servier is capable of NDPI

This is meant for programs as API, not people to create manually. I've sent demo link to your email.


Theme: Navigation menu custom links

I was wondering if you might be able to give us the option to add our own links on the left menu? Right now, you give us the option to "Show Netdata realtime monitoring link" and include a custom URL.
I would like the ability to add others along with custom URL's.
For example, I have implemented GoAccess (https://goaccess.io/) to gather some stats on my website. I would like the ability to have a link that takes me directly to my GoStats URL built into Webmin's left menu displayed right under my Netdata link.

Hi,
Yes, I could do it.

Oh, GoAccess seems really cool!

There's already Custom Links, for just this purpose, in Virtualmin. It's in System Customization->Custom Links.
Do you need it in Webmin without Virtualmin? That's do-able, I think. The only reason it was done in Virtualmin was that when it was introduced (menu years ago), Webmin didn't have a left menu and Virtualmin did. ;-)

@qooob - GoAccess is pretty neat! I'm personally having troubles with "Real Time" reporting, but I'll (hopefully) get it figured out eventually.
@swelljoe - I personally need the link in Webmin as I do not use Virtualmin.
And to take it a step further, how difficult would it be to have the option to have these links open the URL inside the main body of Webmin as an iframe? Not sure if it would look right still having Webmin's left menu, but the idea just came to mind.

@myopenflixr
Webmin Configuration/Edit Categories lets you create your own category and then put existing modules in there. In order to do that, you would need to mimic the module to have the link.
I think, it's doable to make the theme inject custom links into Webmin menu. It will create few extra options in theme's configuration though.
About opening it inside of the main body, it would be difficult to make and I mean only the links with the same refere. It's theoretically possible to inject iframe on the fly and insert content there but it's a step back and I wouldn't want to have all of us headed to this direction. In case it's the different location, it will not work out of the box ever.

As a temporary solution, I used the theme's Show HTML Snippet to insert a link to my stats page. Hopefully you can come up with a solution that allows us to insert links right below Realtime Monitoring. As an added bonus, it would be nice to also be able to assign an icon (Font Awesome) to display in front of our link :)

Hi,
I hope it's still valid. Latest commit brings such functionality.
Please give it a try. Search for Extra entries under Navigation menu options in Theme Configuration.
There is little help balloon with explanations.

Hi @rostovtsev ,
I like the addition! I do have a question though. When setting the icon image for the link, what icon library are we pulling from? Is this based off of font awesome icon library? Just wondering what icon images I have to choose from.
Thanks for the addition!!!

It's custom built of icons. I don't have it published yet.
You are welcome!

@myopenflixr
Mike, how about this:
https://authentic-theme.com/kit/
:smiley_cat:

It's already possible. Theme has extensions for CSS and JS. Simply add there whatever you need, and it will accessible for you and survive updates.
By the way, it would be possible to inject nee links into navigation menu, just by using JS extensions, with writing couple lines of code.

I have added an option/feature, where you can define at which login mode a link would be displayed, by specifying the level, in comma separated list:
0 - administrator login;
1 - reseller login;
2 - server owner login;
3 - user login;
4 - system owner login.
Example:
{
   "extra":[
      {
         "title":"Google Mail",
         "link":"https://gmail.com/",
         "icon":"google",
         "level":"2,3"
      },
      {
         "title":"BIND DNS Server",
         "link":"/bind8",
         "icon":"server"
      }
   ]
}

Quick question: If I want an extra link, pointing to the same host the Wembin is running on, but a different port... how can I do without knowing the full URL?
I.e. "link":"https://%myhost%:8443" and having %myhost% filled with my hostname
Is it ever possible? It would be great as the wembin host is running other web services in my installation

Is it ever possible?

I don't think it's possible.

Could we have a different type of link, adding a port property? Say "link":"/relative", "port":8443
We'd need the theme to process it and compose the proper URL
I guess I'm not the only one having additional web servers running in the same machine as the webmin
In any case, this work is absolutely wonderful, thank you very much

There you go.
Example:
{
   "title":"Usermin",
   "link":"/",
   "icon":"envelope",
   "port":"20000",
   "target":"_blank"
}

Absolutely fantastic, thank you very much!
I'll go test it as soon as I can

Hello,
Is it possible to make a forwarding on a port but defining also if the link is in http or https protocol?
for example:
{
   "title":"Usermin",
   "link":"/",
   "icon":"envelope",
   "port":"20000",
   "target":"_blank"
   "protocol":"http"
}

Yes, you can just remove "port":"20000", and for the link just set the full URL you prefer.

Thanks for the quick feedback, but my goal would be to be able to do something like cconde had asked
I.e. "link": "https://%myhost%:8443" and having %myhost% filled with my hostname
I am not in localhost and the server address on which the webmin and my 2nd service (Duplicati) is running is not on fix IP because it will be put in service at different customer sites.

Then check the latest patch I already made to address it. It will allow setting host and protocol as well. Let me know if it works for you.

It works perfectly! Thank you very much

Glad to hear!


Support for partials

We could really use this, if it's possible to do.
Thanks for the help!

Found a couple of hours to hack on this.
An initial implementation is on this branch.
https://github.com/xxgreg/mustache/tree/partials
This allows the end user to provide a closure (a PartialResolver) that will return the correct template given a string name. This allows partials to be used in the browser, but also for a server side client to provide a resolver which looks up templates using dart:io to access the file system.

wow, thanks!

On pub in version 0.2


Missing client-side testing

We are missing testing on the client-side. React testing etc.

yeah that would be great! any suggestions to implement this?

@thecodingwhale https://discuss.reactjs.org/t/whats-the-prefered-way-to-test-react-js-components/26/20
http://reactkungfu.com/2015/07/approaches-to-testing-react-components-an-overview/
Seems like it's between: ShallowRendering or mocha/jsdom while ShallowRendering has some limitations
Jsdom Example https://github.com/robertknight/react-testing
Do you want do it?

ohh what a great link thanks for sharing! btw I'm still learning the proper way :)

Let's have a look also at protractor:

https://github.com/angular/protractor
http://www.joelotter.com/2015/04/18/protractor-reactjs.html

and nightwatchjs: http://nightwatchjs.org/

npm test now invokes nightwatch. The postinstall npm script also installs the selenium jar. A few test cases are provided. Adding a new one is relatively easy. Contribution welcomed.


Hugo/turtlesim example

Adding turtlesim example and making foxglove launch optional

Changed this PR so that it only includes optional foxglove launching for now (that part LGTM). We can open a separate PR for the turtlesim inclusion and talk about it there.


"windows authentication" (kerberos) does not work on MacOS

Azure Data Studio Version: 1.24.0

Steps to Reproduce:

Download, install ADS.
Start Electron, add a new connection using "Windows Authentication" for Authentication Type.
Get an error: "Connection failed due to Kerberos Error" (details below)

I have been able to use the mssql-tools package on Mac using the "trusted authentication" options (eg, "sqlcmd -E", "bcp -T") to authenticate using kerberos.  I have confirmed that these continue to work now that I am having trouble with ADS.
I have (re)run kinit, and have verified that I have a valid ticket using klist.
Microsoft.Data.SqlClient.SqlException (0x80131904): Cannot authenticate using Kerberos. Ensure Kerberos has been initialized on the client with 'kinit' and a Service Principal Name has been registered for the SQL Server to allow Kerberos authentication.
ErrorCode=InternalError, Exception=Interop+NetSecurityNative+GssApiException: GSSAPI operation failed with error -  An invalid status code was supplied (SPNEGO called got ISC call one too many).
at System.Net.Security.NegotiateStreamPal.GssInitSecurityContext(SafeGssContextHandle& context, SafeGssCredHandle credential, Boolean isNtlm, SafeGssNameHandle targetName, GssFlags inFlags, Byte[] buffer, Byte[]& outputBuffer, UInt32& outFlags, Int32& isNtlmUsed)
at System.Net.Security.NegotiateStreamPal.EstablishSecurityContext(SafeFreeNegoCredentials credential, SafeDeleteContext& context, String targetName, ContextFlagsPal inFlags, SecurityBuffer inputBuffer, SecurityBuffer outputBuffer, ContextFlagsPal& outFlags)
at Microsoft.Data.SqlClient.SNI.SNIProxy.GenSspiClientContext(SspiClientContextStatus sspiClientContextStatus, Byte[] receivedBuff, Byte[]& sendBuff, Byte[] serverName)
at Microsoft.Data.SqlClient.SNI.TdsParserStateObjectManaged.GenerateSspiClientContext(Byte[] receivedBuff, UInt32 receivedLength, Byte[]& sendBuff, UInt32& sendLength, Byte[] _sniSpnBuffer)
at Microsoft.Data.SqlClient.TdsParser.SNISSPIData(Byte[] receivedBuff, UInt32 receivedLength, Byte[]& sendBuff, UInt32& sendLength)
at Microsoft.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action1 wrapCloseInAction)    at Microsoft.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)    at Microsoft.Data.SqlClient.TdsParser.SSPIError(String error, String procedure)    at Microsoft.Data.SqlClient.TdsParser.SNISSPIData(Byte[] receivedBuff, UInt32 receivedLength, Byte[]& sendBuff, UInt32& sendLength)    at Microsoft.Data.SqlClient.TdsParser.SSPIData(Byte[] receivedBuff, UInt32 receivedLength, Byte[]& sendBuff, UInt32& sendLength)    at Microsoft.Data.SqlClient.TdsParser.ProcessSSPI(Int32 receivedLength)    at Microsoft.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)    at Microsoft.Data.SqlClient.TdsParser.Run(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.CompleteLogin(Boolean enlistOK)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.AttemptOneLogin(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, Boolean ignoreSniOpenTimeout, TimeoutTimer timeout, Boolean withFailover)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.LoginNoFailover(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString connectionOptions, SqlCredential credential, TimeoutTimer timeout)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.OpenLoginEnlist(TimeoutTimer timeout, SqlConnectionString connectionOptions, SqlCredential credential, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, SqlCredential credential, Object providerInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling, String accessToken, DbConnectionPool pool)    at Microsoft.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)    at Microsoft.Data.ProviderBase.DbConnectionFactory.CreateNonPooledConnection(DbConnection owningConnection, DbConnectionPoolGroup poolGroup, DbConnectionOptions userOptions)    at Microsoft.Data.ProviderBase.DbConnectionFactory.<>c__DisplayClass45_0.<TryGetConnection>b__1(Task1 _)
at System.Threading.Tasks.ContinuationResultTaskFromResultTask`2.InnerInvoke()
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot, Thread threadPoolThread)
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.SqlTools.ServiceLayer.Connection.ReliableConnection.ReliableSqlConnection.<>c__DisplayClass30_0.<b__0>d.MoveNext() in D:\a\1\s\src\Microsoft.SqlTools.ManagedBatchParser\ReliableConnection\ReliableSqlConnection.cs:line 314
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.SqlTools.ServiceLayer.Connection.ConnectionService.TryOpenConnection(ConnectionInfo connectionInfo, ConnectParams connectionParams) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Connection\ConnectionService.cs:line 550
ClientConnectionId:18885def-ef45-4cdb-8266-a4ddaa97930f

Hi @jcwoods
It's been a while, but would like to request you to relook at Kerberos on Mac OS with latest ADS v1.39 and let us know if you continue to experience the exception described here.
Thank you for taking time!

Would be happy to. I'll give it a try tomorrow.
On Thu, Aug 25, 2022, 8:03 PM Cheena Malhotra @.***>
wrote:

Hi @jcwoods https://github.com/jcwoods
It's been a while, but would like to request you to relook at Kerberos on
Mac OS with latest ADS v1.39 and let us know if you continue to experience
the exception described here.
Thank you for taking time!
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azuredatastudio/issues/13404#issuecomment-1227869963,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWSCXYSC5Z4UUVHM4QYQF3V3ACTNANCNFSM4TU7ILIQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

I was able to successfully authenticate and connect with both "Windows Authentication" and "Azure AD + MFA".
I had a slight problem (triggering an exception) with the server certificate, but flipped the "TrustServerCertificate=True" setting under the advanced settings and everything works.  I'll look into this separately.

Thanks so much for confirming, I will proceed to close the issue :)


lm_head and v_head, why re-initialize and why dropout?

First off, thank you for building this! 3 questions regarding the two heads of the policy model:

why re-initialize the weights in the language model head in

class GPT2HeadWithValueModel

     self.lm_head = nn.Linear(config.n_embd, config.vocab_size, bias=False)

when a trained lm_head already exist in GPT2LMHeadModel?

why does the model still speak coherently before training even though the lm_head weights of the model are random?

from 01-gpt2-with-value-head.ipynb
My most favourite movie is Captain America: Civil War, which moved into the
My least favourite movie is Jon Favreau's Log Horizon, complete with psychedelic

Why use dropout on your value ? value is not like the entire layer of a neural network where you dont want the model to reply too heavily on one activate, value is the one and only signal you get for that layer, so why drop it out?

  (v_head): ValueHead(
    (summary): Linear(in_features=768, out_features=1, bias=True)
    (activation): Identity()
    (first_dropout): Dropout(p=0.1, inplace=False)
    (last_dropout): Identity()
    (flatten): Flatten(start_dim=1, end_dim=-1)
  )

Thanks again!

So I did some research on my own and basically my first 2 questions can be answered by looking at the huggingface transformers repository: https://github.com/huggingface/transformers/blob/main/src/transformers/modeling_utils.py

So I did some research on my own and basically my first 2 questions can be answered by looking at the huggingface transformers repository: https://github.com/huggingface/transformers/blob/main/src/transformers/modeling_utils.py

Hi @clam004, do you mind explaining your answer/understanding on why they do it? Thanks!

@danjohnvelasco as long as you use the same name self.lm_head, when you load the pretrained model from the dictionary of parameters, these linear parameters will be replaced with the trained ones. So thats why the model still works (question 2).  Also regarding question 3, I suspect somehow it doesnt matter, although Im not sure why, cause when I run this repo without the dropout layer, as expected,  it behaves the same.

Regarding 3 I agree and we moved the dropout before the linear layer in https://github.com/lvwerra/trl/pull/70.


getting Error: The brew link step did not complete successfully on CI when running brew upgrade

brew gist-logs <formula> link OR brew config AND brew doctor output
HOMEBREW_VERSION: 4.0.1
ORIGIN: https://github.com/Homebrew/brew
HEAD: 17c872fb5275d87922a56416587cb439a5064354
Last commit: 4 days ago
Core tap origin: https://github.com/Homebrew/homebrew-core
Core tap HEAD: bb3c5b576fb0b13d844fc5395a46e9411ba86e37
Core tap last commit: 45 minutes ago
Core tap branch: master
Core tap JSON: 20 Feb 11:57 UTC
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 4
Homebrew Ruby: 2.6.8 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: quad-core 64-bit ivybridge
Clang: 13.1.6 build 1316
Git: 2.36.1 => /usr/local/bin/git
Curl: 7.79.1 => /usr/bin/curl
macOS: 12.3.1-x86_64
CLT: <IP_ADDRESS>.1.1651278267
Xcode: 13.4.1 => /Applications/Xcode-13.4.1.app/Contents/Developer

Warning: You have unlinked kegs in your Cellar.
Leaving kegs unlinked can lead to build-trouble and cause formulae that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
<EMAIL_ADDRESS>static:~ distiller$ Warning: You have unlinked kegs in your Cellar.

Verification

[X] My "brew doctor output" says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
run brew upgrade on my circleci machine
What happened (include all command output)?
==> Pouring<EMAIL_ADDRESS>Error: The `brew link` step did not complete successfully
The formula built, but is not symlinked into /usr/local
Could not symlink share/man/man1/python3.1
Target /usr/local/share/man/man1/python3.1
is a symlink belonging to<EMAIL_ADDRESS>You can unlink it:
  brew unlink<EMAIL_ADDRESS>
To force the link and overwrite all conflicting files:
  brew link --overwrite<EMAIL_ADDRESS>
To list all files that would be deleted:
  brew link --overwrite --dry-run<EMAIL_ADDRESS>
Possible conflicting files are:
/usr/local/share/man/man1/python3.1 -><EMAIL_ADDRESS>/usr/local/lib/pkgconfig/python3-embed.pc -><EMAIL_ADDRESS>/usr/local/lib/pkgconfig/python3.pc -><EMAIL_ADDRESS>/usr/local/Frameworks/Python.framework/Headers -><EMAIL_ADDRESS>/usr/local/Frameworks/Python.framework/Python -><EMAIL_ADDRESS>/usr/local/Frameworks/Python.framework/Resources -><EMAIL_ADDRESS>/usr/local/Frameworks/Python.framework/Versions/Current -><EMAIL_ADDRESS>==><EMAIL_ADDRESS>-m ensurepip
==><EMAIL_ADDRESS>-m pip install -v --no

What did you expect to happen?
Brew upgrade to be done correctly
Step-by-step reproduction instructions (by running brew commands)
1. brew upgrade

thx @carlocab for the quick response!
Actually fixed it by just upgrading my executor (circleci) to a newer xcode version (from 13.3.0 to 14.2.0)

Oh oops, we must have missed link_overwrite. Will treat this as high priority given it affects Python.

Oh oops, we must have missed link_overwrite. Will treat this as high priority given it affects Python.

While we're at it I guess we want skip_clean too?

I think skip_clean only applies to build-from-source if I'm not mistaken.


Enhance/user input parsing

Relevant Issues
Closes #22
Description
Uses shlex standard python library to implement a Stdin and Sys argument argument parser.
Screenshots

Running the code in Python REPL

In Piston CLI

WIP: Need to add a custom error handler here

Did you:

[x] Join the Piston CLI Discord Community?
[x] Lint your code (pipenv run lint)?

This has been fixed by commit 88a2a6e8fe11b3bc5942e4f25c1d6f258f6d35d4.


Trying to run

First of all, great script. I'm trying to do a npm run but it's not working. I get the message below.
import { AMM } from '@oselezi/solana-volume-sdk';
         ^^^
SyntaxError: Named export 'AMM' not found. The requested module '@oselezi/solana-volume-sdk' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@oselezi/solana-volume-sdk';
const { AMM } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:132:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:214:5)
    at async ModuleLoader.import (node:internal/modules/esm/loader:329:24)
    at async loadESM (node:internal/process/esm_loader:28:7)
    at async handleMainPromise (node:internal/modules/run_main:113:12)

Node.js v21.2.0

Hi @libinysolutions ,
Thank you for reaching out and for your kind words about the script! I'm glad you're finding it useful.
I understand you're encountering the following error when running your script:
import { AMM } from '@oselezi/solana-volume-sdk';
         ^^^
SyntaxError: Named export 'AMM' not found. The requested module '@oselezi/solana-volume-sdk' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@oselezi/solana-volume-sdk';
const { AMM } = pkg;

This error occurs because the SDK is currently exported as a CommonJS module. When using ES module syntax (i.e., import statements), you might run into issues importing named exports from CommonJS modules.
Here's how you can fix the issue:
Your tsconfig.json can be like this:
{
  "compilerOptions": {
    "target": "ES2020",
    "module": "ESNext",
    "moduleResolution": "NodeNext",
    "esModuleInterop": true,
    "strict": true,
    "forceConsistentCasingInFileNames": true,
    "skipLibCheck": true,
    "allowSyntheticDefaultImports": true
  }
}

And you have to include type: "module"  in your package.json file.
instead of
import { AMM } from '@oselezi/solana-volume-sdk';

please try
import { createRequire } from 'module';
const require = createRequire(import.meta.url);

const SDK = require('@oselezi/solana-volume-sdk');
const { AMM } = SDK;

Let me know if you need further assistance or if you encounter any other issues.

Thanks for the help. I'm not getting this error:
`❯ node index.ts
node:internal/modules/esm/get_format:160
throw new ERR_UNKNOWN_FILE_EXTENSION(ext, filepath);
^
TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts" for /Users/lucasguilherme/Meme/solana-volume-sdk/src/index.ts
at Object.getFileProtocolModuleFormat [as file:] (node:internal/modules/esm/get_format:160:9)
at defaultGetFormat (node:internal/modules/esm/get_format:203:36)
at defaultLoad (node:internal/modules/esm/load:143:22)
at async ModuleLoader.load (node:internal/modules/esm/loader:409:7)
at async ModuleLoader.moduleProvider (node:internal/modules/esm/loader:291:45)
at async link (node:internal/modules/esm/module_job:76:21) {
code: 'ERR_UNKNOWN_FILE_EXTENSION'
}
Node.js v21.2.0`

You can check the example folder for the usecase.


Set Up a Workflow With Extended Employee Onboarding

Tutorial URL: https://developers.sap.com/tutorials/cp-starter-ibpm-employeeonboarding-3-workflow.html
Please specify the step you are referring to
In step 2.3 The screenshot needs to be updated. The file WorkflowManagementFLP.tar should not be there.
Expected:
1st screnshot

2nd screenshot

Actual:
1st screenshot

2nd screenhot

Labels:
Landscape Labels: trail1, trail2,trail3
Sprint Label: SCPTest-2010A
Scenario Label: SCPTest-eboard
Priority Label: Low

Hello Akansha,
There could be different view of the Business Application workspace for different users. They may or may not have projects in their workspace and so the screenshot will be different. All that matters is they have to drag and drop the unzipped project under "project" node in the pop-up. So,  both the screen shots are correct. No change is needed.
@MichaelCzcz - please close the ticket.

Hello @ArchanaShukla,
Noted.


WinError 126: Error Loading fbgemm.dll or one of it's dependencies

Hello, I have built this out on several systems without fail.
Starting last week I am getting the following error on multiple systems:
This is a development version of GIMP.  Debug messages may appear here.
(gimp-2.99.exe:9112): GLib-GIO-WARNING **: 13:59:19.793: win32 session dbus binary not found
set device 'System Aggregated Pointer' to mode: disabled
Skipping potential plug-in 'C:\Users\TestLab\GIMP\gimpenv3\Lib\site-packages\gimpopenvino\plugins\plugin_utils.py': plug-ins must be installed in subdirectories.
Skipping potential plug-in 'C:\Users\TestLab\GIMP\gimpenv3\Lib\site-packages\gimpopenvino\plugins_init_.py': plug-ins must be installed in subdirectories.
Error in set_translation_domain. This is expected if running GIMP 2.99.11 or later
Error in set_translation_domain. This is expected if running GIMP 2.99.11 or later
Error in set_translation_domain. This is expected if running GIMP 2.99.11 or later
C:\Users\TestLab\openvino-ai-plugins-gimp\weights..\gimp_openvino_run_sd.json not found, loading defaults
No stable-diffusion model server found to kill
Traceback (most recent call last):
File "C:\Users\TestLab\GIMP\gimpenv3\Lib\site-packages\gimpopenvino\tools\stable-diffusion-ov-server.py", line 13, in
import torch
File "C:\Users\TestLab\GIMP\gimpenv3\lib\site-packages\torch_init_.py", line 148, in
raise err
OSError: [WinError 126] The specified module could not be found. Error loading "C:\Users\TestLab\GIMP\gimpenv3\lib\site-packages\torch\lib\fbgemm.dll" or one of its dependencies.
Any help is much appreciated.

I was able to resolve this by installing VS Community, pulling libomp140.x86_x64.dll from Windows\System32 and placing it in ...GIMP\gimpenv3\Lib\site-packages\torch\lib.

interesting. i did notice this today, and was able to resolve it by modifying the torch revisions to be torch>=2.2.0,<2.4.0.


remove mut requirement for RedisPool, + some lint fixes

This removes the requirement for a mut pool (which did not make sense, and severely limited use through a web framework)
NOTE: FYI in general there are some more warnings (via lints), which indicates some of the types require Serialize and are not Send, but I left it as-is for now.

Agreed on mut prefix. I think that was a relic of the previous type. And I like the inclusion of #[must_use] .  There are a few must_use failures in CI and one code change that is required. Thoughts on making incrby / decrby not needing let _ = x?

Yea I guess I can exclude those, let me revet back and see how its done in other cases

@film42 fixed the lints, now the counter functions reflect the intent (incr, decr do not return values). The reason that they return value is because of compare-and-swap (returning the previous value), a semantic which isn't needed in this case.

Looks great. I'll get this published shortly. Thanks again, @jondot !


My luarocks is (incorrectly?) diagnosed as out of date

When I upload a rock, I get:
$ luarocks-5.2 upload --force lrexlib-pcre-2.8.0-1.rockspec
/home/rrt/.local/etc/luarocks/upload_config.lua
Sending lrexlib-pcre-2.8.0-1.rockspec ...
Warning: Your LuaRocks is out of date, consider upgrading.

I am using luarocks 2.2.0, installed from source (not as a rock itself).

The server reports a version of 1.0.0 whereas luarocks 2.2.0's tool_version is 0.0.1 (and the same in current git), so the error is definitely OK, but there's nothing I can do!

Yes, please ignore this message for now. It's was a dissonance between the old moonrocks upload tool and its port into LuaRocks as the luarocks upload command. The version I ported identified itself as 0.0.1 but afterwards @leafo bumped it up to 1.0.0 and I missed that. We ended up not deciding then if we'll use this version to identify the actual LuaRocks version or the upload API version... I think I should at least bump the tool_version in Git to 1.0.0.

No, wait, tool_version in git is 1.0.0 already!
https://github.com/keplerproject/luarocks/commit/a35dd43752f0ee10d65c27200299176b29db45ef

Apologies, I was looking at the wrong source.


2020-09-07 tc119_M1SD optimizekel routine fails with mct$OPTIMIZEKEL==1, optimization fails criteria with mct$OPTIMIZEKEL==2

tested with commit cca452d
all data are loaded to one drive

The test case has 6 executions

mct$OPTIMIZEKEL==0 mct$FLGNOCMAX==1
mct$OPTIMIZEKEL==0 mct$FLGNOCMAX==1
mct$OPTIMIZEKEL==1 mct$FLGNOCMAX==1 - guided optimization method
mct$OPTIMIZEKEL==1 mct$FLGNOCMAX==1 - guided optimization method
mct$OPTIMIZEKEL==2 mct$FLGNOCMAX==1 - brute force optimization method
mct$OPTIMIZEKEL==2 mct$FLGNOCMAX==1 - brute force optimization method

there are two execution directories to review
tc119_M1SD_cca452d
FAILS with mct$OPTIMIZEKEL==1
error message:
openNCA computation engine version 3.0.0
Computation Run Date/Time: 2020-09-07 17:47:35
Error in value[[3L]](cond) :
  Error in if (kel_tmp > kel_val) {: missing value where TRUE/FALSE needed
For SDEID 161019312
In addition: Warning messages:
1: In validate_timeconc_data(map, data, flag, verbose = verbose) :
  No tau information provided in 'map'.
2: In validate_timeconc_data(map, data, flag, verbose = verbose) :
  No told information provided in 'map'.
3: In run_computation(data = d, map = mct, flag = flags, parameterset = parameterset) :
  'TAUs and TOLDs values are not provided via 'flag'! Using values provided via 'data' instead (if present)!
4: In run_M1_SD_computation(data = merged_data, map = map_data, method = method,  :
  Flag 'FLGACCEPTTAU' cannot be computed if 'TAUi' is not provided
5: In run_M1_SD_computation(data = merged_data, map = map_data, method = method,  :
  Flag 'FLGEMESIS' is not present in the dataset

tc119_M1SD_cca452d_noopt1
setup to remove all optimization runs with mct$OPTIMIZEKEL==1 to illustrate how the test case will operate once it can run successfully.

As of commit 53d87f1 this issue no longer occurs.


fix(change repository url): change repository url

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add FIPS 140-3 indicator API

FIPS 140-3 requires an indication on whether an operation was using approved services or not. The FIPS 140-3 implementation guidelines (section 2.4.C) provide two basic approaches to doing this: implicit indicators, and explicit indicators.
Implicit indicators are basically the concept of "if the operation passes, it was approved" (in the Example Scenarios table given in FIPS 140-3 IG, that's either "Return code from an approved security service call" in scenario 1 or "A status code indicating the completion of service" in scenario 2). At Red Hat, we were originally aiming for implicit indicators in our copy of OpenSSL.
However, this proved to be a problem, because we wanted to certify a signature service, and FIPS 140-3 requires that a signature service computes the digest to be signed within the boundaries of the FIPS module (i.e., using the OSSL_FUNC_SIGNATURE_DIGEST_SIGN* provider functions, not the OSSL_FUNC_DIGEST* functions followed by OSSL_FUNC_SIGNATURE_SIGN). Since we were planning to certify fips.so only, this means that EVP_PKEY_sign/EVP_PKEY_verify would have to be blocked. Unfortunately, EVP_SignFinal uses EVP_PKEY_sign internally, but outside of fips.so and thus outside of the FIPS module boundary. This means that using implicit indicators in combination with certifying only fips.so would require blocking both EVP_PKEY_sign and EVP_SignFinal, which are the two APIs currently used by most users of OpenSSL for signatures.
EVP_DigestSign would be acceptable, but has only been added in 3.0 and is thus not yet widely used.
A workaround for this catch-22 would be explicit indicators so that EVP_PKEY_sign and EVP_SignFinal can continue to work for now, but FIPS-aware applications can query the explicit indicator to check whether the operation was approved.
It would be nice if such an API was available, but unfortunately it seems to be quite complicated to get this introduced through the entire stack from the provider up to the EVP APIs, especially without breaking ABI.
I've made a rough attempt at sketching out an implementation of EVP_MD_fips_indicator() in https://github.com/neverpanic/openssl/commit/a25ca79e9f8ae832fb2b68929aa00b40de718e4d, which could potentially be expanded into the other EVP_* structs. We're also considering a symbol that can be dlopen(3)/dlsym(3)'d from fips.so in https://gitlab.com/redhat/centos-stream/rpms/openssl/-/merge_requests/71, but that's more a short-term hack than a long-term solution. See also https://bugzilla.mozilla.org/show_bug.cgi?id=1710773#c0 on how NSS implemented FIPS indicators.

Cc: @beldmit.

I should have added that EVP_PKEY_sign can of course be claimed as a raw signature API, and that can be tested and approved. Unfortunately it seems that for RSA, NIST does not provide test vectors (and thus no certification) of the corresponding raw verification operation.

Hm, maybe I checked the wrong file on that; I think I checked util/libcrypto.num, but that has 3_0_0 for all symbols. Regardless of how long they've been available, unfortunately it seems like they're not being used widely – at least not widely enough that making EVP_PKEY_sign() return an error when used with the FIPS provider would not break many applications.

IMO this does not make any sense to me - these existing applications would not be compliant anyway so what would be the point of the indicators? The apps can just switch to non-FIPS mode or be fixed to use DigestSign* APIs.

You're right that they won't be compliant – however, the fallout is a bit unexpected, e.g., ssh no longer works. I think breaking those functions leads to bad user experience, while having indicators would allow for a smoother transition off the old APIs.

@t8m this is a new FIPS-140-3 concept. The FIPS-compliant applications should have to deal with it. It's a long shot but future FIPS certification will require such an API.

EVP_DigestSign would be acceptable, but has only been added in 3.0 and is thus not yet widely used.

EVP_DigestSignInit/Update/Final have been available for a long time (since 1.0.0). Wouldn't they be acceptable? Not sure when the one-shot EVP_DigestSign was added but it appears to be before 3.0. It is present in the 1.1.1 man pages.

Since it goes through to the dedicated digest_sign_update provider op, it seems like it shoould work from the policy perspective.  But that doesn't resolve the "existing software doesn't actually use it" issue.

Since it goes through to the dedicated digest_sign_update provider op, it seems like it shoould work from the policy perspective. But that doesn't resolve the "existing software doesn't actually use it" issue.

Which leads to the obvious question: how far should we bend over backwards to support recent changes in the FIPS guidelines?
Expecting an application to just work when the FIPS standards update is rather optimistic.  Us having to add a hack to support such applications also seems a bit out there.

@paulidale indicators are not a hack, but an explicit requirement of the new FIPS-140-3 standard. We kind of invented the concept of implicit indicators, but having explicit ones is the more standards complaint one.
Note that indicators do not have to be on the API themselves. Indicators identify services, so it can be a completely separate facility whereby the OpenSSL FIPS provider declare what "services" it offers and provide indication about them.
However it is much simpler to tie an indicator to an operation (as represented by an API), as that is the most flexible way going forward.

@t8m @paulidale
Just to move the discussion along this is an example of what indicators could look like:
https://github.com/simo5/openssl/commit/f38ca9f089a17b85a9ab54b39cfc4cb935cef15b
Key aspects:

it allows to confine FIPS checks to the FIPS Indicator parameter processing
does not require to block operations that are not fips approved because just some of the parameters are not of NIST liking, or just because NIST does not have an ACVP test
does not affect applications that do not care about FIPS compliance
allows applications that care to check and decide by themselves if they want to block or allow the operation (the app may know better)

But the most important aspect is that this adds no new APIs, just a parameter that can be define for all functions once and then the already existing EVP_PKEY_CTX_get_params()/EVP_PKEY_CTX_gettable_params() handles it all.
We could do this as a downstream patch, but I rather have this available for all vendors so that applications do not need to be patched differently for each vendor's way of implementing FIPS-140-3 restrictions.

EVP_DigestSignInit/Update/Final have been available for a long time (since 1.0.0). Wouldn't they be acceptable? Not sure when the one-shot EVP_DigestSign was added but it appears to be before 3.0. It is present in the 1.1.1 man pages.

Yes, but these functions don't seem to be available in fips.so:
$ strings fips.so | grep EVP_Digest | sort | uniq
EVP_Digest
EVP_DigestFinal
EVP_DigestFinal_ex
EVP_DigestFinalXOF
EVP_DigestInit
EVP_DigestInit_ex
EVP_DigestInit_ex2
EVP_DigestSignUpdate
EVP_DigestUpdate
EVP_DigestVerifyUpdate

Only the DigestSignUpdate and DigestVerifyUpdate are included, for some reason?

marking as inactive, to be closed at the end of 3.4 dev barring further input

@slontis is making slow progress towards implementing indicators.
Not sure this should be closed unless things are being tracked elsewhere.

@paulidale you're right, this was mislabeled on my part this should be marked for a future minor version

@paulidale is this progress public somewhere?
I will be a user of this work early on and I would like to make sure the solution will be compatible with my needs and provide feedback should it not be.

Best to ask @slontis the status.

https://github.com/openssl/openssl/pull/23609

https://github.com/openssl/openssl/pull/23609 is the design doc..
https://github.com/openssl/openssl/pull/23890 shows how to apply it in code.. I need to break this PR up into parts so that it can be merged back in.


snapshotWorkingDir doesn't update in all situations

When removing a dynamic lookup spec by sending a "DELETE" action to the respective coordinator endpoint, the historicals update their lookupSnapshot.json snapshot accordingly.
However, when posting a complete lookup spec to the /druid/coordinator/v1/lookups endpoint, the lookupSnapshot.json files do not get updated. There are also no log statements contained in the historical logs that would indicate that updating the file wasn't successful.
Posting updates to individual lookups also doesn't update the snapshot files.
BTW: sending DELETE or POST actions to the coordinator endpoints for a single lookup definition works in so far as it performs the action, but the REST calls hang. All other rest endpoints work for me and do not hang.

correction: the DELETE or POST actions do not actually hang. this was unrelated to Druid, but the snapshot files still only get updated in some situations.


Generating generators

New CLI command coming!
This new command sublayer generate generator drops you into a walkthrough for generating a new sublayer generator for your project.
Seeing a longer prompt like this definitely makes me think we need a better abstraction for prompts - this could probably be a good place to look at and experiment with for #75 I think. There are some clear sections that I'd like to break out into methods to build the prompt, a mix of background information, few shot (1 for now) examples, and instructions.
I also had an issue where I was using the same heredoc delimiter in the example and in the code so it blew up when I tried to just inline everything. Gotta be a better way to handle this.
Also ran into an issue trying to integration test this command as running the CLI on a separate thread wouldn't allow VCR to hook into and record the requests. Would love a better solution there to actually testing these commands. Probably need to come up with a mock provider for things like this.
Anywho, this sets the stage pretty well for doing the same thing with Agents and Actions which should come through a lot quicker and be ready for a release very shortly!

Fixes #89

Related to #63


Vulkan SDK path should be infered from environment variables

The README file says one has to edit the vulkan SDK path in the CMakeFile.txt before compiling.
However, the Vulkan SDK already defines an environment variable called VULKAN_SDK
Would it be possible to let cmake detect the SDK path from the environment variable?

I think that VULKAN_SDK is defined only if you run "setup-env.sh" beforehand. I did not mention this anywhere in the README before so it should not be defined at this point


Dataloader example Launcher

In the readme file, under the "Extracted data for FOL model training and testing" section, I believe there is a typo.
--log_config >> --load_config
From
python dataloader_example.py --log_config config/config_example.yaml
To
python dataloader_example.py --load_config config/config_example.yaml

@apbraga You are right, I just fixed it. Sorry about that!

Great! Thanks for the quick reply.


3.4 Place Bid

Bid placement should be working, user needs to be joined to the auction and be present in the database.


feat: per-buffer sign toggle

Allows for per-buffer and global toggling of signs. WIP. The global toggle will override buffer local sign settings - if a buffer has signs disabled, but you globally toggle signs on, that will turn signs on for that buffer too.
closes #21

@andrewferrier can you test this PR out? I don't use sign toggles at all, but maybe you can confirm that this at least fits in with your intended workflow.


#1: add support for horses in database initialization

/paulo - can you please review this pull request?

replaced


is it possible to fill like pie?

from 0 to 360 degrees (full circle)?

Sure. Here is quick example how to do it:
https://gist.github.com/AEugene/60447196fbf503f1a641
Is that what you are looking for?


neighbour information not updated/missing

Hi
Looks like the neighbour info (heard us / we heard) has not been updated for several days.
Is it disabled now ?
Will you consider enabling again ? Very usefull feature, on my point of view.

Hi, this was disabled on the official MQTT server by the Meshtastic developers. You could run your own server to get around this new limitation.

Thanks. Can you be a little more specific? What has been deactivated, and why?
I don't understand what the module in the firmware is for, if the data can't be retrieved?

I don't have a link to the discussion on me right now, but in the official Meshtastic Discord server you should be able to find more about it in the #mqtt channel.
The neighbour info is still useful and can be collected by running your own node that is connected to your own MQTT server, it's just no longer broadcasted by the public Meshtastic MQTT server.
This will of course only allow you to collect neighbour info from your area, and not the entire world.
I believe this was being done to minimise the amount of traffic being sent/received.

Ok, thks for the infos.
I will try to run your map software on my own MQTT server, then.


Add compatibility support for MySQL "tx_isolation" in docker config

This replaces this PR: https://github.com/uber/cadence/pull/2493
Here’s some context: https://forum.pydio.com/t/solved-mysql8-error-when-running-pydio-cells-docker-failed-to-initialize-storage-failed-to-open-migrations-creating-dex-migration-table-error-1193-unknown-system-variable-tx-isolation/2129

the problem are are changes made in MySQL 5.7 and 8 - ‘tx_isolation’ has been depreciated in 5.7.20 and an alias has been added called ’transaction_isolation. In MySQL 8 tx_isolation has been removed entirely leaving the former alias.
The means that cells will not run with MySQL 8 currently. Code changes are needed to allow both, pre 5.7.20 (tx_isolation) and 5.7.20 and later (transaction_isolation). It should however not be a big task I guess, as this variable is rarely used. An option would be by to use the “set transaction” statement instead of setting the isolation level as connection parameter. “set transaction” is supported in all versions and will not cause compatibility issues - it will, however, add a slight overhead due to the additional statement needed.

This change isn't ideal, but it quickly enables folks (like myself) who are trying to deploy Cadence via Docker to use an existing Aurora MySQL instance, which still uses the tx_isolation parameter.

Coverage increased (+0.02%) to 66.852% when pulling c9e6f3c5fff89705e0d908db76186ce48b33c624 on barakalon:barak-tx-isolation-compat into 4200e936fdf2b1c9027a00aa775b3e0c3af4f421 on uber:master.

I'm unable to view the buildkite failures.
Can anyone provide more information on what went wrong?

I'm unable to view the buildkite failures.
Can anyone provide more information on what went wrong?

Sure, I am looking at it


Add simple abi checker util

Thanks, this is going to be helpful!  I tried running the script, and the "git checkout ${libver_old}" clobbered the script when it checkout out the old version in place. How about having the script download a zip from the release tag and compare that to the current repo, so no automatic checking out of anything?

This is a great idea.

oh, haha, i tested this with it uncommitted and didn't even think about the fact it would nuke the script checking out an old version. These are scripts I normally just have in my source root, but seemed like they might be helpful for people wanting to play with this stuff since you all were asking about it in a previous TSC meeting. Yes, makes way more sense to have it shallow clone or download a release tag (I presumed we would want to test against a thing before it is released, so may still need to clone).

So, I do this for some internal projects, and what I do is actually check into git the ABI dump of the standard I'm trying to hit. Then I compare against that reference when I do the ABI test (and obviously, update the reference standard when I know I'm purposely altering the ABI).
The downside is that the reference abi dump is only valid for one build configuration out of your CI matrix, but so what, I only do the ABI check on that one and assume that it's unlikely that I'll break ABI only for one compiler or whatever.


[WIP] preliminary incremental buffer updates

This is a re-start of an old bit of vimscript I wrote to avoid sending the entire buffer between oni and nvim.
I don't think I have all of the code I need to get behavior 100% normal, but this is a start

@extr0py: thoughts?

The overall strategy looks good to me! Long-term, we might be able to replace it with the LiveUpdate PR in progress in Neovim. In the short-term, though, only sending over the current line when in insert mode would be a big perf boost, especially for large files. We only send the current line over to the plugins, anyway, so in that insert mode case, there isn't a need to send over the entire file. I think this could help a lot with typing responsiveness in insert mode!
A couple things we should look at before merging:

Fix the error causing Neovim to block load
Remove extra stuff around last_cursor_line and last_change_tick

Thanks @bert88sta for looking into this!

I just tried again - I'm still seeing the completion issues.
Here's a gif of the expected behavior on master:

Note that as the value is typed from w -> wi -> win, the set of completions is changed.
Here's the behavior I'm seeing on this PR:

This appears to be a regression from this change, so we'll need to fix this.
I see that a failure in the getCompletions  code in the typescript language service, in this block:
        while (col >= 0) {
            var currentCharacter = currentLine[col];
            if (!currentCharacter.match(/[_a-z]/i)) {
                break;
            }
            currentPrefix = currentCharacter + currentPrefix;
            col--;
        }

It seems like the values in the current line are out of sync with the column value (so somehow the current line isn't making it all the way here, or something), and the col ends up indexing a value outside of the current line. Likely this is occurring further up, or their is some timing change that is impacting this code path.

I'm honestly not sure why this is happening. the only difference between
the prior implementation and this implementation is lastBufferLines. Other
than that, this uses the same methods. I'll give it a look later tonight or
tomorrow since I have off work
On Sun, Jul 16, 2017 at 12:01 PM extr0py<EMAIL_ADDRESS>wrote:

I just tried again - I'm still seeing the completion issues.
Here's a gif of the expected behavior on master:
[image: inc-completion-current]
https://user-images.githubusercontent.com/13532591/28249129-f705741e-6a04-11e7-9a9d-26e2a2a08e25.gif
Note that as the value is typed from w -> wi -> win, the set of
completions is changed.
Here's the behavior I'm seeing on this PR:
[image: inc-completion-539]
https://user-images.githubusercontent.com/13532591/28249136-0c2dead8-6a05-11e7-90aa-1e1bb8c6e3f6.gif
This appears to be a regression from this change, so we'll need to fix
this.
I see that a failure in the getCompletions code in the typescript
language service, in this block:
    while (col >= 0) {
        var currentCharacter = currentLine[col];
        if (!currentCharacter.match(/[_a-z]/i)) {
            break;
        }
        currentPrefix = currentCharacter + currentPrefix;
        col--;
    }

It seems like the values in the current line are out of sync with the
column value (so somehow the current line isn't making it all the way here,
or something), and the col ends up indexing a value outside of the
current line. Likely this is occurring further up, or their is some timing
change that is impacting this code path.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/extr0py/oni/pull/539#issuecomment-315618785, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AICBCSOOHU9UMrPY4CVck7USzYGyOVhKks5sOjPtgaJpZM4OYO88
.

Actually, this bit from the index.js:
    Oni.on("buffer-update", (args) => {
        const fullText = args.bufferLines.join(os.EOL)
        const words = fullText.split(/\W+/)

        lastBuffer = args.bufferLines

        currentWords = Object.keys(words.reduce((prev, cur) => {
            prev[cur] = cur
            return prev
        }, {}))
            .filter(w => w.length >= 3)
    })

    const getCompletions = (textDocumentPosition) => {
        if (textDocumentPosition.column <= 1)
            return Promise.resolve({
                completions: []
            })

        let currentLine = lastBuffer[textDocumentPosition.line - 1];
        let col = textDocumentPosition.column - 2
        let currentPrefix = "";

        while (col >= 0) {
            const currentCharacter = currentLine[col]

            if (!currentCharacter.match(/[_a-z]/i))
                break

            currentPrefix = currentCharacter + currentPrefix
            col--
        }

I'm going to take a wild guess and say that lastBuffer is supposed to be the whole thing, but is getting assigned to the single line that gets sent over from neovim. I don't know if this would throw an error or not

I'm going to take a wild guess and say that lastBuffer is supposed to be the whole thing, but is getting assigned to the single line that gets sent over from neovim. I don't know if this would throw an error or not

Ya, good catch! I think this logic isn't quite right in BufferUpdate.ts:
        this._neovimInstance.on("buffer-update-incremental", (args: Oni.EventContext, bufferLine: string, lineNumber: number) => {
            this._lastArgs = args
            this._lastBufferLines = [bufferLine]
            this._lastBufferVersion = args.version

            this._pluginManager.notifyBufferUpdateIncremental(args, lineNumber, bufferLine)
        })

Seems like this._lastBufferLines should be updated as:
this._lastBufferLines[lineNumber - 1] = bufferLine

Kind of like how it is updated in the buffer-update case above.

I changed that, but I still can't seem to get completions for webkit*
On Mon, Jul 17, 2017 at 8:18 AM, extr0py<EMAIL_ADDRESS>wrote:

I'm going to take a wild guess and say that lastBuffer is supposed to be
the whole thing, but is getting assigned to the single line that gets sent
over from neovim. I don't know if this would throw an error or not
Ya, good catch! I think this logic isn't quite right in BufferUpdate.ts:
    this._neovimInstance.on("buffer-update-incremental", (args: Oni.EventContext, bufferLine: string, lineNumber: number) => {
        this._lastArgs = args
        this._lastBufferLines = [bufferLine]
        this._lastBufferVersion = args.version

        this._pluginManager.notifyBufferUpdateIncremental(args, lineNumber, bufferLine)
    })

Seems like this._lastBufferLines should be updated as:
this._lastBufferLines[lineNumber - 1] = bufferLine
Kind of like how it is updated in the buffer-update case above.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/extr0py/oni/pull/539#issuecomment-315775711, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AICBCf8m10gO7Eg9wO1GCgpGjxrm71pmks5sO3KIgaJpZM4OYO88
.

Changes look good! Thanks for fixing it up. Once the conflict is resolved, I'll bring it in.

Aha! fixed it. Now we'll just see if I linted my code well enough :/

Thanks for making the fixes, @bert88sta ! Looks like it's working for me, and it certainly improves perf in insert mode for large files. Nice work, appreciate the contribution!


Option to customise font used for IntelliSense

VSCode Version: 1.41.1
OS Version: macOS 10.15.2

Steps to Reproduce:

Open a file, such as the settings.json document
Type something that will make the fill suggestions appear

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

The text in the editor is correct based on my OpenType settings, but the fill suggestions are still the default. (Note differences in i and l)

Are you talking about the font?

Of course, it is about the font.

Umm... I don't understand the new title. I just thought that when OpenType customization was implemented for the editor, the fill suggestions were overlooked.

Notice the i in the editor does not match the i in the fill suggestion list.

Umm... I don't understand the new title. I just thought that when OpenType customization was implemented for the editor, the fill suggestions were overlooked.

Forgot a word, updates the title again

The IntelliSense just needs to automatically match the editor.

Well, turns out we actually do apply the editor font in the suggest widget. I guess this is a ligature issue. @AlsoScratch Please share your settings with me.

It's a font called Iosevka and it uses what I may call custom OpenType tags to modify individual characters like g, i, l, y, etc.

The settings.json file
{
    "editor.fontFamily": "'Iosevka Term', 'IBM Plex Mono', Menlo, Monaco, 'Courier New', monospace",
    "editor.fontLigatures": "'cv01', 'cv05', 'cv11', 'cv19', 'cv21', 'cv23', 'cv26', 'cv27', 'cv36', 'cv38', 'cv45', 'cv46', 'cv49', 'cv51', 'cv52', 'cv65'",
    "editor.lineHeight": 22
}

Ok, I am starting to understand now (sorry not a font expert). I have pushed a change that makes sure that we use the same font features in the suggest widget as we do in the editor. Tomorrow's insider build should contains this change, please report back if it then works for you.

I don't use insider, I only trust official releases.

Insiders is an official release.

Insiders is an official release.

Isn't it the one that always has the latest commit, something like that?

It's a daily build (usually from 8am CET) which contains the latest changes and allows you to see what will come in VS Code.

Oh, I just realized that the word 'settings' in the title is misspelled.

@jrieken Your change is applied to the details widget, but I think the OP is asking for the suggest item list to share font feature setting.
I don't think we want to apply the font feature settings to the details widget, as it uses system's default sans-serif font (such as San Francisco on macOS), and font features that apply to monospace coding font (such as his choice, Ioveska) might cause weird rendering in details widget.
In short, I suggest taking https://github.com/microsoft/vscode/commit/6f428b15103d05bd2660ae26b029f4417eaedd79 and removing https://github.com/microsoft/vscode/commit/bf2dbfc3412c69bea9a5f1c893deb046316bb7b0.

@jrieken Lemme know if I can remove https://github.com/microsoft/vscode/commit/bf2dbfc3412c69bea9a5f1c893deb046316bb7b0


Minimal Relay example

How can I use relay to populate my data? Will it work automatically? or any plans on how to add support for it?

Any success?

I have yet to try it... mostly today?

Im new too
On Oct 29, 2016 3:39 AM, "pyros2097"<EMAIL_ADDRESS>wrote:

I have yet to try it... mostly today?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zeit/next.js/issues/106#issuecomment-257077113, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AWD9FxgmSSDYtTRZzNyBzDjQBkKTiUXzks5q4vhOgaJpZM4Kh991
.

Or rather in general, how to add a GraphQL client like Apollo (which is fully GraphQL compliant)? 0.5.0 just came out.

I created a higher order component to wrap routes that take data from apollo:
import React from 'react'
import 'isomorphic-fetch'
import ApolloClient, { createNetworkInterface } from 'apollo-client'
import { getDataFromTree } from "react-apollo/server"
import { ApolloProvider } from 'react-apollo'
import { createStore, combineReducers, applyMiddleware, compose } from 'redux';

const getReducer = ( client ) => {
  return combineReducers({
    apollo: client.reducer( ),
    someReducer: ( state = {}, action = null ) => action && action.state
      ? action.state
      : state
  });
}

const getClient = ( headers ) => {
  return new ApolloClient({
    networkInterface: createNetworkInterface({ uri: 'http://mygraphqlserver.local/graphql' }),
    ssrMode: true,
    headers: headers
  })
}

export const initClientAndStore = ( initialState, isServer, headers ) => {
  if ( isServer && typeof window === 'undefined' ) {
    const client = getClient( headers )
    return {
      client,
      store: createStore(getReducer( client ), initialState, applyMiddleware(client.middleware( )))
    }
  } else {
    if ( !window.clientAndStore ) {
      const client = getClient( headers )
      window.clientAndStore = {
        client,
        store: createStore(getReducer( client ), initialState, compose( applyMiddleware(client.middleware( )), window.devToolsExtension
          ? window.devToolsExtension( )
          : f => f ))
      }
    }
    return window.clientAndStore
  }
}

const getRootComponent = ( {
  client,
  store
}, Component ) => (
  <ApolloProvider client={client} store={store}>
    <Component prop={'test'}/>
  </ApolloProvider>
)

export default( ComposedComponent ) => class Main extends React.Component {
  static async getInitialProps({ req }) {
    const isServer = !!req
    const headers = req
      ? req.headers
      : {}
    const clientAndStore = initClientAndStore( {}, isServer, headers );

    await getDataFromTree(getRootComponent( clientAndStore, ComposedComponent ))

    return { initialState: clientAndStore.store.getState( ), isServer, headers }
  }

  constructor( props ) {
    super( props )
    const clientAndStore = initClientAndStore( props.initialState, props.isServer, props.headers );
    this.clientAndStore = clientAndStore;
  }

  render( ) {
    return getRootComponent( this.clientAndStore, ComposedComponent )
  }
}

it mostly works, but im getting a strange warning about the markup checksum:

next-dev.bundle.js:8985 Warning: React attempted to reuse markup in a container but the checksum was invalid. This generally means that you are using server rendering and the markup generated on the server was not what the client was expecting. React injected new markup to compensate which works but you have lost many of the benefits of server rendering. Instead, figure out why the markup being generated is different on the client or server:
(client) <div data-css-1mbax
(server) <div data-css-1mbax

@thorstenweber83 that looks like a react Apollo SSR bug we are tracking and hope to have fixed soon! This looks pretty great. I'd love to create a react-apollo-nextjs package. Would you be up for that?

A general package for Apollo (with SSR) + next.js would be awesome indeed, I've been sketching a little myself on something similar but didn't get very far yet. I'd be glad to just help out testing if you guys need any assistance.

Duo.  So stuck on this.  Can I get a friend or what?  Not another animal.  🦄🐒
Get Outlook for Android
On Sun, Oct 30, 2016 at 5:01 AM -0700, "zth"<EMAIL_ADDRESS>wrote:
A general package for Apollo (with SSR) + next.js would be awesome indeed, I've been sketching a little myself on something similar but didn't get very far yet. I'd be glad to just help out testing if you guys need any assistance.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks @thorstenweber83 for the HOC, works well 👍 except when reloading the page, I get window is undefined. Maybe the conditional should be changed to if (isServer || typeof window === 'undefined')?

@thorstenweber83 I also haven't figured out how I can configure the endpoint URI through an environment variable. Client-side, it seems the constructor calls initClientAndStore before it gets that info from the server?

Hi Mr.  Dubois.  I love your name!  Are you French?
Get Outlook for Android
On Sun, Oct 30, 2016 at 11:05 AM -0700, "Sébastien Dubois"<EMAIL_ADDRESS>wrote:
@thorstenweber83 I also haven't figured out how I can configure the endpoint URI through an environment variable. Client-side, it seems the constructor calls initClientAndStore before it gets that info from the server?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I took @thorstenweber83's code and wrapped to take care of the graphql higher-order component as well and also display loading and error states (but still no idea how to pass the endpoint URI as environment variable):
https://github.com/sedubois/realate/blob/master/containers/Apollo.js
Apologies if there are any mistakes, but at least it works so far. Would be great if newcomers didn't have to figure out all of that by themselves.
/cc @stubailo

It's really surprising to me that it's this hard to set up a global container, like Redux, with this framework. Is there an official answer for how to use Redux with Next.js?

Ah, I found the wiki page, I guess it suggests something similar.
Perhaps someone can maintain a package like next-apollo or similar to do this whole singleton integration thing?

@stubailo I think that's what @jbaxleyiii had in mind, I think it would be great too.
Maybe things could also be further simplified with https://github.com/zeit/next.js/issues/50.

In the first example I gave above, the data is only rendered client-side. But how to render Apollo server-side?
@thorstenweber83, above you render server-side with <Component prop={'test'} />, not sure what you intended by that?
@stubailo In your GitHunt example, you do SSR using react-router which passes the props for rendering, but I don't know how to do it without react-router. Help appreciated 😊

OK I'm starting this getDataFromTree(), we first need to inject the component rendered with dummy data so that Apollo knows which data to query to populate the store.
So I did that but then got server-side Network error: fetch is not defined. Shouldn't Apollo make sure to have what it needs to make HTTP requests server-side?

Related issue: https://github.com/apollostack/apollo-client/issues/645

OK as for lacking SSR, my problem came from having wrapped in a higher-order component and not re-calling getInitialProps (as described here). Sorry for the spam 😄

Has anyone been able to put together a working example of next.js and apollo-client?

Yes @ads1018, here! https://github.com/relatecommunity/relate

Sweet! Thanks @sedubois. I wonder if there's a way to pass the Apollo store to the root component without having to wrap all pages in a HOC.

@ads1018 actually new things could already be possible thanks to the merge of https://github.com/zeit/next.js/pull/301 but haven't looked into that yet (and that code isn't released yet).

Guys, Next.js had a pretty big issue with https://github.com/zeit/next.js/issues/253.
Basically we couldn't share JS modules between next.js pages.
With now v1.2.x we fixed it. So Apollo, Redux, Relay and any other data binding tool works without by default without any additional work.
Anyway, we should have some example showcasing that.

@arunoda amazing news! I've been itching to use next.js for my next project but have been holding off until it played nice with apollo-client.

Really looking forward to see some examples with apollo-client.

I posted an example with apollo in #387 - thought I'd create a new issue since it's not about relay.

I thought this thread is about Relay support! For Apollo there are a few other issues opened (#99, #387), so maybe all apollo related discussions should be moved there?

One problem is that when using isomorphic-relay prepareInitialRender is async, which means the component immediately mounts on the client without enough info to render properly. As suggested in that issue you can get around this by faking initialReadyState, e.g:
const initialReadyState = {
  aborted: false,
  done: true,
  error: null,
  events: [
    {
      type: "STORE_FOUND_ALL",
    }
  ],
  ready: true,
  stale: false,
};

And then passing that into IsomorphicRelay.Renderer:
export default class extends React.Component {
  static async getInitialProps () {
    if (isServer) {
      const { data, props } = await IsomorphicRelay.prepareData(rootContainerProps, networkLayer);
      return { serverProps: props, data };
    }
  }

  constructor(props) {
    super(props)
    if (isServer) {
      // initial server-side render
      this.state = { example: props.serverProps };
    } else if (props.data) {
      // inject data on client side after server render
      IsomorphicRelay.injectPreparedData(environment, props.data);
      // pass in fake initialReadyState
      this.state = { example: { ...rootContainerProps, environment, initialReadyState } };
    } else {
      // client only render
      this.state = { example: { ...rootContainerProps, environment } }
    }
  }

  render () {
    return (
      <IsomorphicRelay.Renderer {...this.state.example} />
    );
  }
}

But as the author of isomorphic-relay said this isn’t really ideal.
Another thing to keep in mind is that if you are thinking of using a hosted graphql service like graphcool then it might not be worth going the relay server rendering route anyway, you’re looking at ~1 sec waiting for a response and a subsequent initial render.

@thisbejim I agree, I only went with Graphcool because it was easy to setup and wouldn't litter the repo with graphql schema related files and dependencies.
I'd like to finish the next.js example, though I don't know how to properly initialize the Relay enviornment, if someone has an idea or interest in it, here's the repo:
https://github.com/Gregoor/next.js-relay-example
And my current issue:
https://github.com/denvned/isomorphic-relay/issues/61#issuecomment-273464691

FWIW @ads1018 found a pretty neat way to get Apollo working with almost no boilerplate in #387. See the example in the wiki.

I wish Relay's API was isomorphic out of the box, but that just doesn't seem to be a priority for FB atm.

I wonder if we should keep this open or not. Maybe we should just document the reason why using Apollo over Relay is better in the case of isomorphic rendering.

I'd definitely keep this open, imo apollo-client and Relay are too far apart to consider them just a replacements of each other. While apollo might get more primitives right, there are things I really like about Relay they that apollo gets wrong:

fetching diffs
pagination
fragments

Maybe I'll find the time to figure out how exactly the Relay API would have to change, to make the integration with next simpler.

I don't have a strong opinion on whether or not to close this issue (if I had to choose, I would say keep it open), but I just wanted to address the points made by @Gregoor, because I think they aren't valid (to be fair, the fragment one used to be valid). While neither is a drop-in replacement of the other, I think it's worth pointing out that with Apollo you can do everything that you can do with Relay, but not the other way around.
To address the specific points made:

Fetching diffs: I think this point is based on Relay advertising query diffing as a feature in the past, when in fact it was a mistake. Apollo intentionally doesn't do query diffing because we recognized the importance of static queries. The Relay team has recognized that query diffing was a mistake, so they explicitly said that when Relay 2 comes out it will have static queries like Apollo does now.

Cursor-based pagination: I think this is a fair point, but I think it's important to put it in context: Apollo lets you do any type of pagination. Relay makes pagination with connections easier, but it doesn't work with any other type of pagination. I'll be the first to admit that if you absolutely know that you want to use connections everywhere in your schema for the rest of time, and it's the only thing you care about, Relay is the way to go until Apollo has the same or better user experience for pagination with connections.
If you're not 100% sure however, consider that a paginated query in Relay has to be at least as complicated as this:

query {
  viewer {
    post(id: "34def7cb-9702-426d-9883-07beaf2d8240") {
      comments {
        edges(first: 10, after: "ba2f4523-93f2-4f3b-9135-073dc465779c"){
          cursor
          node {
            text
          }
        }
      }
    }
  }
}

while in Apollo it can be simply this:
query {
  post(id: 502){
    comments(offset: 10, limit: 10){
      text
    }
  }
}

Which one you prefer might be a matter of taste, but I'm willing to bet that most people will find the second query much easier to understand and aesthetically more pleasing. If you want to use the first  style of query, you can still do that in Apollo, but you cannot use the second style of query in Relay.
note: I intentionally used long ids in the Relay example, because Relay requires all ids to be unique across your schema.
PS: we're planning to add special support to make cursor-based pagination super-easy in one of the upcoming versions of Apollo.

Fragments: I assume that by this you mean co-locating data requirements with the component by using fragments. Apollo lets you do that as well. It's also worth pointing out that in Relay's you need to set up an extra build step with babel and use Relay.QL, while Apollo works out of the box and uses plain old GraphQL.

Now that said, I agree 100% with you that neither should be considered a replacement of the other, and that each client has its own strengths and weaknesses. Personally I think it would be great if Relay was easier to use and integrated better with other libraries,
@Gregoor I would love to know what things you wish Apollo did/had that Relay currently does/has. We're always striving to make Apollo better, so your input would be much appreciated and valued! I also hope that once Relay 2 finally comes out it will be possible to use it with next.js. I'm also hoping that there will be some really innovative features in it that we can adopt in Apollo as well 😀

Thanks @helfer, intriguing points!

fetching diffs:

The Relay team has recognized that query diffing was a mistake

Dang, that's a bit sad. Related Thread is here btw, for anyone that's interested. I hope it gets reimplemented at some later point.

Cursor-based navigation: neat example, but it would be even neater if you'd include fetching further results. For me that's where Relay wins, as it is a simple setVariables call to fetch more whereas the apollo variant looked a bit more obscure (I'd be happy to be proven wrong :) ).

fragments: To include a fragment in Relay, you simply embed a getFragment-call of a component. In Apollo you have to embed it AND spread it at the right place (taking care that the name matches). I think that API is at least twice as complex :upside_down_face:
Also I haven't yet figured out how you'd set variables inside of a fragment. Is that possible?

In Apollo you have to embed it AND spread the right name at the right place. I think that API is at least twice as complex

I would be surprised if you don't have to do that in Relay 2 because it makes static analysis quite a bit more difficult. Perhaps they require you to use a variable name that's identical to the fragment name or something.

Also I haven't yet figured out how you'd set variables inside of a fragment. Is that possible?

That's also a Relay 1-specific feature, I don't think Relay 2 has it because of the move to static queries.

Yaiks, I hope it doesn't turn out that way, then Relay 2 would be quite the downgrade for me. I really do appreciate the isolation of variable setting in Relay and if Relay 2 is more apollo like that would make it a lot more complex wouldn't it? I feel like I'm missing something here.

I think the general Apollo vs. Relay debate is very interesting but a bit off-topic concerning Next.js 😄
If we agree Relay 1 can't practically be integrated with Next.js, I'd just close the issue. And we can reopen a new one when Relay 2 is a thing. @timneutkens ?

We might have gotten a bit sidetracked true :grin:
I added another commit to https://github.com/Gregoor/next.js-relay-example which makes it kinda work. The kinda part is, that there is just no proper way to rehydrate Relay, so the first client-side render is empty (and thereby inequal to the server's render) but without refetching the client renders again correctly. Atm I don't see any API in Relay that would allow completely fixing it, so closing this is fine from my side.

Relay 2 is a thing now.

Closing in favor of #1757


[FAQ] Existing Stripe Customers Not Syncing

Bug report

Extension name: firestore-stripe-subscriptions

Describe the bug
I already have firebase & stripe customers that were synced with a custom webhook.
I am trying to replace that piece of infrastructure with this extension, but it doesn't seem there is anyway to force a sync for the existing customers.
Expected behavior
Some path to syncing existing customers, possibly by their user email.
Thank youuuuu :)

Duplicate of https://github.com/stripe/stripe-firebase-extensions/issues/93#issuecomment-723879491
Leaving this open as a reminder that I need to make it clearer that the extension creates customer objects as needed on the fly and there is no need to backfill existing users.

@thorsten-stripe thanks for your reply! I'm still a little confused about this... I understand it creates new customers in stripe for new firebase users, but all my existing firebase users and stripe customers are not syncing.
The problem is, I have gated content that checks if a users plan is active in the collections created by this extension. So for new users it works fine, but pre-existing users don't have those documents/collections generated, does that make sense?
Thanks in advance!! It's so great to be able to have an even leaner infrastructure by outsourcing the stripe functionality from a custom api to this extension!

@thisisvilla are you saying you have existing customers in Stripe with existing subscriptions? The extension is not able to sync these, but rather you will need to add your existing Stripe customer ID to the user doc in your Cloud Firestore:

As long as you have the stripeId set in your user doc, the extension will use this customer ID for that user. Otherwise it will create a new Stripe customer object.

@thorsten-stripe Thanks for your reply! Yeah, I found that work-around, but I also had to add the 'firebaseUID' meta key to the customer in stripe, as well as do some arbitrary update to the user's subscription. Then everything synced just fine! Thanks!! Keep up the great work! 🎉

@thisisvilla glad you got it working. The firebaseUID metadata should not be required, it's just there to make searching in the dashboard more convenient.


Accept string as valid collection type in lodash’s _.size

Per lodash size docs string is a valid type for the collection argument

@sdg9 Closing as the lodash defs have change quite a lot since this opened. The current definition is still missing string as a valid collection type, so if you're still interested in adding it, a new PR would be mucho appreciated!


terraformPlan fails every other time (with single lambda resource)

I'm using the lambda integration from the README, it creates a resource and its the only resource in the whole project.
> Task :my-root-project:my-terraform-project:terraformMainPlan FAILED
╷ IDLE
│ Error: Reference to undeclared input variable
│
│   on project-rbc.tf line 98, in resource "aws_lambda_function" "my_lambda":
│   98:   filename = "${var.resources.myLambda_jar.path}"
│
│ An input variable with the name "resources" has not been declared. This
│ variable can be declared with a variable "resources" {} block.
╵

FAILURE: Build failed with an exception.

I get this failure every 1st build, then I run the same command again (terraformPlan) and it succeeds.
Might be something to do with the ordering of when the resources are created vs when my tf file is accessed?
Also in the IDE, the line itself is always declared as an "unresolved" reference.

Will have a look

could you share a repro project?

Created a simple reproduction project https://github.com/blundell/TerraformGradlePluginPlayground/tree/lambda_not_found_repro/backend however it no longer fails every other time (it succeeds every time). But the IDE still shows it as red/missing:

Indeed the TF IJ plugin does not support yet this setup. I will try to support it as soon as I can.


fix broken link to slim readme

Thanks for putting this course up! I have been reading through it. I got a 404 with the current link to the Slim readme. This just fixes the broken link.

@brandonmbanks I appreciate your attention.


Obsoleted payment schemas

Fixes #2973
Added:
[Obsolete("Bot Framework no longer supports payments.")]

To all payment classes. Also removed references to the payment actions and removed a few unnecessary using directives.
Questions

Is that message fine?

In ConnectorAPI.json, should I deprecate all of the properties along with the classes?

Note: I'll update the remaining Sagger files once above questions are answered.

Pull Request Test Coverage Report for Build 93279

0 of 0   changed or added relevant lines in 0 files are covered.
23 unchanged lines in 18 files lost coverage.
Overall coverage increased (+0.2%) to 74.24%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder/Adapters/TestFlow.cs
1
77.17%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/EmitEvent.cs
1
65.22%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/IfCondition.cs
1
88.89%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/SwitchCondition.cs
1
91.49%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/DialogManager.cs
1
80.77%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Generators/ActivityChecker.cs
1
86.76%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Testing/TestActions/AssertReplyActivity.cs
1
80.77%

/libraries/Microsoft.Bot.Builder.Dialogs/Prompts/OAuthPrompt.cs
1
54.8%

/libraries/Microsoft.Bot.Builder/Inspection/InspectionMiddleware.cs
1
78.89%

/libraries/Microsoft.Bot.Builder.LanguageGeneration/Expander.cs
1
81.43%

Totals

Change from base Build 93249:
0.2%

Covered Lines:
14315

Relevant Lines:
19282

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.ApplicationInsights.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.AspNet.Core.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Testing.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.6.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Streaming.dll compared against version 4.6.3

This all looks correct to me. @jeffders, any thoughts?


Added an hackbar

Please add "hacktoberfest-accepted" when merging

@Fra146 Sure

Thanks @Fra146 for contributing :)


Subsetting: Comment out tabular view button before build 55 release

This button was made visible for early qa, but it isn't ready for public release.

Button no longer appears on qa sites.


Rename STIMMY to SER Token

I'm submitting a ...
[ ] bug report
[x] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
Quick token rename. I submitted the original PR.

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)
https://github.com/solana-labs/token-list/pull/20139

merged


Use an event bus for handling all events in JSkat

The interface JSkatView has grown over the years into a complicated beast with too much methods. Implementations of JSkatView like NullView must implement all those methods without doing anything. It would be better to use an event bus for all events in JSkat and a view can be a subscriber of some or all of them.
This would help to solve issue #19 too.

Event bus was introduced with issue #30.