Azure Web App build issues with framework 4.5.2

We started having an issue with multiple Azure Web Apps this morning that are utilizing Kudu to deploy from repositories in BitBucket.  To replicate, I did the following things:

In VS2013, File -> New Project -> ASP.Net Web Application -> WebApi
Opened solution, changed target framework to 4.5.2
Committed to BitBucket.

You can see the repository here:
https://bitbucket.org/veatcha/kudu-test
Here is the beginning of the build log - it starts normally, and then produces a warning for every project reference, similar to the last line included here:
2015-08-24T20:24:38  Updating branch '8e0dcb00be285faece9d9d3c88ce8b20dad147d3'.
2015-08-24T20:24:38  Updating submodules.
2015-08-24T20:24:38  Preparing deployment for commit id '8e0dcb00be'.
2015-08-24T20:24:38  Generating deployment script.
2015-08-24T20:24:39    Using cached version of deployment script (command: 'azure -y --no-dot-deployment -r "D:\home\site\repository" -o "D:\home\site\deployments\tools" --aspWAP "D:\home\site\repository\WebApplication1\WebApplication1.csproj" --solutionFile "D:\home\site\repository\WebApplication1.sln"').
2015-08-24T20:24:39  Running deployment command...
2015-08-24T20:24:39    Command: "D:\home\site\deployments\tools\deploy.cmd"
2015-08-24T20:24:39    Handling .NET Web Application deployment.
2015-08-24T20:24:40    All packages listed in packages.config are already installed.
2015-08-24T20:24:41    D:\Windows\Microsoft.NET\Framework\v4.0.30319\Microsoft.Common.targets(1605,5): warning MSB3267: The primary reference "Microsoft.CSharp", which is a framework assembly, could not be resolved in the currently targeted framework. ".NETFramework,Version=v4.5.2". To resolve this problem, either remove the reference "Microsoft.CSharp" or retarget your application to a framework version which contains "Microsoft.CSharp". [D:\home\site\repository\WebApplication1\WebApplication1.csproj]

After that, it's a series of build errors due to the references not being present.
I've confirmed the identical code (see branch framework4.5) builds and deploys correctly when targetting 4.5.
I don't have any other kudu app setting overrides set - just a fresh Azure Web App.
Thanks,
Andrew

Can you share the name of a sample site that's broken in this way? This will help us investigate. Thanks!

Sure:  http://framework-test.azurewebsites.net

Thanks, we are investigating.

Thanks David.

Well, we broke something, and we are working to fix it. We should be able to do this before tomorrow morning. Sorry about that! :(

We have deployed the fix to your VM. The rest will be deployed later.

Thanks David, that's great. When you say later, do you have sense of the timing?  That VM just has a few dev/test things on it, but we've got a couple production sites that are impacted.

Would tomorrow morning work for you? Say before noon PST. Maybe be able to get it done a bit earlier than that.

Hmmm, looking at your subscription, I would not have expected those other sites to be affected. Are you sure that they are? If so, can you share the name of any site in the same App Service Plan as the affected site? You can created a dummy site for that purpose if you like.

One that I'm aware of that was having the problem in that same subscription yesterday was East-1-Dev - I'm going to verify a couple in our other subscription now.

I'm not seeing a site called East-1-Dev. Maybe not quite the right name?

East-1-Dev was the App Service Plan. commons-assignments-internal-dev is an example of a site in it with the problem.

Ah yes, this App Service Plan does have this issue at this time. We'll get that fixed this morning.

The issue should now be fixed for all your East US sites. Can you please verify?

I'm encountering an almost identical issue with .NET 4.6. However it only occurs for me when adding Unity DI into the project. There is a sample repo here: https://bitbucket.org/jimi312/unity46. Let me know if you'd prefer me to open a separate issue

@JimiSmith it's probably a distinct issue as original one is now fully fixed. So new issue would be best. Thanks!
Also, make sure to mention in new issue whether this is something that used to worked and is now broken, or that maybe you have not tried before.

For reference, @JimiSmith's issue is tracked by #1675, and is indeed distinct from this one.


Add ability to solve on tabs

Summary
Add a "tabs" submodel to account for solving the electrical problem on the tabs or extend the current collector models to include the tabs.

Can this be closed? @rtimms @Scottmar93

Yeah, I don't think we have any plans to add this. Can open a new issue later if anyone is interested.


initialize DateRangePicker with end date

Hi borax12,
When can we use this feature?

@Debjyotinath - i will be merging the pending PRs today . So dont worry , its coming asap

Thank you.

@paralen @Debjyotinath merged :) :+1:

@paralen Could you implement the same for the Time Dialog?


DO NOT MERGE! Fast Forward stable to 5bae502af7492b59d0a59d7f2e89bd95d1088efe.

This PR will be merged automatically once checks are successful.

This change is

Jenkins job mixer/postsubmit passed

Jenkins job mixer/postsubmit passed

Jenkins job mixer/e2e-suite passed


Maps

Hi,.
Just found your app and testing it. So much better than what BMW provide!
The issue I am finding with Amazon Music HD is that when playing individual track the coverart loads fine, but when playing one of their provided playlist no coverart is shown for each track.
Tested with Deezer and that works fine.
I assume searching for tracks, albums and playlist is not supported for these two apps?
Thanks Alan.

Opened in error.


SpectrumWidget: Reset axes upon when changing units

Normally the spectrum is automatically rescaled upon various changes, e.g. when changing energy units, or loading a different workflow.
However, when you change the scales by interacting with the spectrum, this no longer happens, and one has to manually click the reset button. That get very annoying especially when changing the units. Arguably, we should also always rescale when changing the workflow.
This might however be tricky to do, perhaps we will need to set the scales manually in Python as it seems there is not simple Python API to reset, and the JS solution might not be reliable.
https://github.com/bokeh/bokeh/issues/5071
https://github.com/bokeh/bokeh/pull/7757
https://github.com/microscopium/microscopium/issues/124
https://stackoverflow.com/questions/29294957/how-can-i-accomplish-set-xlim-or-set-ylim-in-bokeh
https://github.com/microscopium/microscopium/pull/130/files

I am actually not sure if this is a good idea, closing for now, can revisit later if users complain.


Update network-watcher-create.md

@mattreatMSFT,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@mattreatMSFT : Thanks for your contribution to the Azure documentation! The author, @georgewallace, has been notified to review your proposed change.

#sign-off


Add backchannel auth for devicemgt app

Please do not merge until tested thoroughly.

Hi Dilhasha,
Thank you very much for the PR. Can you restructure your PR into new PR structure including the details of what this PR is for, what it does...etc.
And also added comments for PR please review them as well.

This is obsolete and hence closing.


Why does ncdump work for this URL in older versions of the netcdf-c, but not for 4.7.x

This is a follow on to https://github.com/Unidata/netcdf4-python/issues/1041
For 4.7.x, ncdump -h http://iridl.ldeo.columbia.edu/SOURCES/.Indices/.soi/.c8110/.anomaly/T/%28Jan%201979%29/VALUE/dods fails
See https://github.com/Unidata/netcdf4-python/issues/1041#issuecomment-687642215
For older versions, there is no problem.
ncdump -h http://iridl.ldeo.columbia.edu/SOURCES/.Indices/.soi/.c8110/.anomaly/T/%28Jan%201979%29/VALUE/dods
netcdf dods {
dimensions:
	T = 1 ;
variables:
	float T(T) ;
		T:standard_name = "time" ;
		T:pointwidth = 1.f ;
		T:expires =<PHONE_NUMBER> ;
		T:calendar = "360" ;
		T:gridtype = 0 ;
		T:units = "months since 1960-01-01" ;
	float anomaly(T) ;
		anomaly:file_missing_value = -999.9f ;
		anomaly:standard_name = "air_pressure_anomaly" ;
		anomaly:history = "Indices Tahiti slp c8110 standardized\n",
			"Indices Tahiti slp c8110 anomaly\n",
			"Averaged over T2[1981, 2010] minimum 0.0% data present\n",
			"sqrt [   total ( { Indices Tahiti slp c8110 anomaly } squared )   / 360. ]\n",
			"  total [ ( Indices Tahiti slp c8110 anomaly ) squared ]   / 360.\n",
			"Averaged over T2[1981, 2010] minimum 0.0% data present\n",
			"Averaged over T[Jan 1981, Dec 2010] minimum 0.0% data present\n",
			"Indices Darwin slp c8110 standardized\n",
			"Indices Darwin slp c8110 anomaly\n",
			"Averaged over T2[1981, 2010] minimum 0.0% data present\n",
			"sqrt [   total ( { Indices Darwin slp c8110 anomaly } squared )   / 360. ]\n",
			"  total [ ( Indices Darwin slp c8110 anomaly ) squared ]   / 360.\n",
			"Averaged over T2[1981, 2010] minimum 0.0% data present\n",
			"Averaged over T[Jan 1981, Dec 2010] minimum 0.0% data present" ;
		anomaly:units = "unitless" ;
		anomaly:expires =<PHONE_NUMBER> ;
		anomaly:long_name = "sea level pressure anomaly" ;
		anomaly:missing_value = -999.9f ;

// global attributes:
		:Conventions = "IRIDL" ;
}

This has to do with url encoding. If you use this URL, it should work (if quoted):

'http://iridl.ldeo.columbia.edu/SOURCES/.Indices/.soi/.c8110/.anomaly/T/(Jan 1979)/VALUE/dods'
The problem is that netcdf assumes that the specified URL is not URL encoded,
so this case it is doubly encoding and the server cannot deal with that.
If I recall, this changed because of the way Apache started enforcing proper encoding.

I think that I understand.  It would seem that having the URL encoded should always be ok.
This one without URL encoding does not work
ncdump -h "http://iridl.ldeo.columbia.edu/SOURCES/.Indices/.soi/.c8110/.anomaly/T/(Jan 1979)/VALUE/T/(days since 1960-01-01)streamgridunitconvert/dods"
but maybe it's server side. I don't what URL is actually being sent to the server..

It appears that this most recent problem has to do with handling of
blanks. I was replacing blanks with '+', which works ok with most servers.
Apparently that is not the case here. So I changed the code to use
%20 and that seems to fix it. I am testing now to see if it causes
other problems. I also need to add some notest to NUG/DAP2.dox about this.

Thanks! I admit to being a little confused with who is doing the URL encoding and who should be doing it. I tend to use the URL encoded strings since Ingrid provides them in the webpage (e.g.,http://iridl.ldeo.columbia.edu/SOURCES/.Indices/.soi/.c8110/.anomaly/T/%28Jan%201979%29/VALUE/T/%28days%20since%201960-01-01%29streamgridunitconvert/dods) and they work (usually) across applications.
If you identify an issue with the Ingrid server (iridl), I can contact the developers.

I think in general, the client should encode portions of the URL that it programatically constructs by default (say, when preparing to make a REST API call) and the server should decode before passing information related to the request to the server-side application. However, the backend Ingrid service does not appear to be able to handle enocoded URLs properly (specifically the query portion).
https://github.com/Unidata/thredds/issues/1144#issuecomment-435960241
I can't find much information about Ingrid, but my guess is that they've implemented their own web server and it does not fully handle the rules in RFC3986...and that guess is based on language in this entry on the "software that uses netCDF page:

Ingrid is currently running as a WWW daemon that can be accessed through [...]

The headers indicate they have a Squid caching proxy setup, so decoding issues might lie above the Ingrid service (note: it's a dangerously old version of Squid, and I cannot find much information about how it handles encoded URLs).
The best approach I've found with the iridl.ldeo.columbia.edu site is to turn off any URL encoding done by the client, leave anything I've added to the query unencoded, and encode the path as best I can by hand. For example, somewhere along the communication chain one or more servers know enough about percent encoding to handle %20 (' '), %28 ((), and %29 ())in the path of the URL:
curl -G -v http://iridl.ldeo.columbia.edu/SOURCES/.Indices/.soi/.c8110/.anomaly/T/%28Jan%201979%29/VALUE/T/%28days%20since%201960-01-01%29streamgridunitconvert/dods.dds
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to iridl.ldeo.columbia.edu (<IP_ADDRESS>) port 80 (#0)
> GET /SOURCES/.Indices/.soi/.c8110/.anomaly/T/%28Jan%201979%29/VALUE/T/%28days%20since%201960-01-01%29streamgridunitconvert/dods.dds HTTP/1.1
> Host: iridl.ldeo.columbia.edu
> User-Agent: curl/7.58.0
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Mon, 07 Sep 2020 13:15:17 GMT
< Expires: Tue, 06 Oct 2020 00:19:00 GMT
< XDODS-Server: Ingrid/2.15
< Server: Ingrid 0.9
< Mime-Version: 1.0
< Access-Control-Allow-Origin: *
< Cache-Control: public
< Content-Description: dods-dds
< Content-Type: text/plain
< Last-Modified: Sun, 06 Sep 2020 06:49:02 GMT
< Content-Length: 150
< X-Cache: MISS from gfs2mon2.ldeo.columbia.edu
< X-Cache-Lookup: MISS from gfs2mon2.ldeo.columbia.edu:3128
< X-Origin-Date: Mon, 07 Sep 2020 12:56:17 GMT
< X-Origin-Expires: Tue, 06 Oct 2020 00:00:00 GMT
< X-Cache-Age: 1140
< X-Cache: HIT from iridls2.ldeo.columbia.edu
< X-Cache-Lookup: HIT from iridls2.ldeo.columbia.edu:80
< Via: 1.0 gfs2mon2.ldeo.columbia.edu:3128 (squid/2.7.STABLE9-20110824), 1.0 iridls2.ldeo.columbia.edu:80 (squid/2.7.STABLE9)
<
Dataset {
    Float32 T[T = 1];
    Grid {
     ARRAY:
        Float32 anomaly[T = 1];
     MAPS:
        Float32 T[T = 1];
    } anomaly;
} anomaly;

but as pointed out in the github issue I linked to above, there is a chance the request will fail if you encode the query portion of the URL. I've had mixed results on the ability of this server to handle percent encoding in the query, however (sometimes it works, sometimes it does not). My default when interacting with this particular site is to leave the query unencoded.
@DennisHeimbigner - is there a way to tell netCDF-C to not handle URL encoding? Perhaps an option in the .dodsrc file, or even better, a runtime option?

Currently there is no way to tell the DAP2 client library code to not encode something.
My rule has been that all URL encoding should be handled by the netcdf-c DAP2
code. Primarily because the URL being sent is a mix of the users' URL plus
internally generated constraints to minimize the amount of data requested of the server.
I already have some hacked exceptions in the DAP2 code to handle other Columbia
server foibles. I suppose we could add yet another.

I have attached a zip file containing a new version of libdispatch/ncuri.c.
Please try it and see if it solves the problem.
fixblanks.zip

Thanks! I don't know exactly how to do that but I will try compiling it later. I got here from an error in xarray, and I'm a little over my head.

If you installed netcdf using a package manager (apt, yum, conda, etc) then
you cannot test the change. So you may have to wait a while for a release
containing this fix. Sorry.

Fixed in #1835


Add support for --remote when pushing configs to Heroku

Heroku also supports a --remote flag, which targets the specified git remote.  This pull request provides support for that flag

This looks great. You're only missing one very important step.

Thank you, much better! :clap: :trophy: :tada:

I'll release a minor version shortly.

Version 1.1.0 released! :confetti_ball:


Missing OSX native libraries in couchbase-lite-java-sqlcipher-1.4.1.jar

Need to ensure these are included in 1.4.4

fixed in 1.4.4


API - Salesforce Composite API Endpoint

Goal: Implement an API endpoint to allow Salesforce composite API requests. This endpoint should manage the Salesforce access token, refreshing it as necessary.
Background: We often need to perform multiple related Salesforce operations in a single transaction. The Salesforce composite API allows us to do this, but it requires careful handling of authentication tokens.
Tasks:

[ ] Implement a new API endpoint to handle Salesforce composite requests from our application's front-end.
[ ] The endpoint should add the current Salesforce access token to the request before sending it.
[ ] If a request fails due to an expired access token, the endpoint should automatically refresh the token and retry the request once.
[ ] Implement error handling for other error cases, such as invalid requests or server errors.
[ ] Write tests for the new endpoint, including tests of the token refresh handling.

This task is required for the completion of user story:

TrueSparrowSystems/AI-SalesBridge-iOS-App#8
TrueSparrowSystems/AI-SalesBridge-iOS-App#11

We can make use of Interceptor Pattern to reuse the common code. eg:

Fetch Access Token
Decrypt with KMS
Exception Handling
Handle oauth access token Expiry: refresh token and retry salesforce api


operators hive-operator (1.2.3552-133480f)

Update Hive community operator channel(s) [['alpha']] to 1.2.3552-133480f
/hold

/unhold

Closing due to #880 still being open.

This update will supersede #880.

/merge possible

/merge possible


Add Oracle support and SQLite

@mickeypearce I don't have access to push to your repository, but here is your PR with the updated refs.
This PR will close #51, #13 and #44.
Also it contains commits from #88 .

Hi @mickeypearce . I  really don't know what to do right now about bundling all this together. Any thoughts or suggestions?
I'm thinks about having SQLTools with default pure JS libs as we current have and allow user to change to a bin path, and maybe show raw results. It would allow user to use their already installed cli's, or either usql if they want to.

Do you mean by optionally switching extension into using "usql" for connecting to db with a configuration parameter, and then leaving to the user the task of installating the correct usql binary?

Actually I was thinking about using the oracle binary.
That's the same strategy I've used to created https://github.com/mtxr/SQLTools. I run queries directly using the database cli binaries. We could show the raw output for now, like SQLTools for sublime. But I really appreciate the UI we currently have.

I get the "raw output" part now. In the Oracle case that would mean running queries with SQL Plus and printing results. a bit :disappointed:
The latest node-oracledb driver now includes pre-built binaries for different Node versions and OS architectures (node-oracledb v3.1.0 Release Notes). Unfortunatelly it doesn't solve the problem for using it inside VS Code as that requires an electron rebuild.
I use node-oracledb in oradew (mickeypearce/oradew-vscode#15) from a spawn process (gulp) - outside of "vscode" environment. That way it doesn't even require an extra step for electron rebuilding. Maybe consider that as possibility if manully rebuilding with electron-rebuild is a no-go as was suggested initially in original PR.

I'm finishing this weekend I believe. I found a way. We actually don't need to build against electron, but against to VSCode mode version since we are using a language server. Much easier, much maintainable.
It's happening dude. It's happening..

@mickeypearce can you test it tomorrow? I have no oracle db, also I could not setup it locally during the wekeend, but I got it fully running for SQLite, I believe it will also work for Oracle.
I'm wrapping up some stuff at work, I'll commit the weekend changes tonight, ~1h probably.
If you could test and get it working with oracle, I wouldd appreciate a lot

Just did. Don't forget to run yarn run watch before launching the extension. Also, maybe you can need to reinstall all deps.

Good news , but I won't be at my desk until start of the next week...

Do you want me to pack it for you to test?
If it's an option, I can do
On Tue, Feb 19, 2019, 06:24 mickeypearce<EMAIL_ADDRESS>wrote:

Good news , but I won't be at my desk until start of the next week...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mtxr/vscode-sqltools/pull/120#issuecomment-465054073,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArL6dXpb3ECjiTrrvJhb-ySFEMqv98Vks5vO8LYgaJpZM4aT_Ag
.

Yes , It would be better.

https://transfernow.net/8e16g353bx2y @mickeypearce

@mickeypearce any updates? I'm excited to hear your opinion. I've read the docs, but wasn't able to test.
I hope we can get it done soon
Again, thank you so much!

I tried to connect (any db) and I get "Error selecting connection Launching server using runtime node.cmd failed."
From SQLTools Language Server output:
[Error - 10:01:55] Starting client failed
Launching server using runtime node.cmd failed.

Any idea?

It's not finding node binary in your path.
On Thu, Feb 21, 2019, 06:17 mickeypearce<EMAIL_ADDRESS>wrote:

I tried to connect (any db) and I get "Error selecting connection
Launching server using runtime node.cmd failed."
From SQLTools Language Server output:
[Error - 10:01:55] Starting client failed
Launching server using runtime node.cmd failed.
Any idea?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mtxr/vscode-sqltools/pull/120#issuecomment-465922512,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArL6b9W6sXS5ihcMl4hI3sllzOnzkIgks5vPmQSgaJpZM4aT_Ag
.

It got me a bit further:
Error selecting connection Request connection/openConnection failed with message: Cannot read property 'deps' of undefined

Interestingly sqllite got me to "Install dependency" screen but then failed while installing package with:
Received request to install deps: {"dialect":"SQLite"}
Will install sqlite3 package
Registered exception:  { Error: spawn npm ENOENT
    at _errnoException (util.js:1022:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:190:19)
    at onErrorNT (internal/child_process.js:372:16)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickCallback (internal/process/next_tick.js:180:9)
  code: 'ENOENT',
  errno: 'ENOENT',
  syscall: 'spawn npm',
  path: 'npm',
  spawnargs: [ 'install'<EMAIL_ADDRESS>] } { meta: { type: 'uncaughtException' } }
Uncaught exception received.

https://stackoverflow.com/questions/27688804/how-do-i-debug-error-spawn-enoent-on-node-js
It's the same. Couldn't find the binaries on windows. I'll need to run more tests on windows. I might get a VM for that

Great, I can see you are doing a lot of changes, I'm at work now, but once I finish I'll take a look and test those changes, also setup WIN VM. Thanks @mickeypearce !

Hey @mickeypearce , any updates on oracle testing?

Error selecting connection Request connection/openConnection failed with message: Cannot read property 'deps' of undefined

No, this error was the farest I got. I can try again tomorrow with your updates.

This is happenning probably because the class name for Oracle Database is name as Oracle only.
I believe switching to OracleDB as in the package.json would fix this.
Matheus Teixeira
On Feb 26 2019, at 12:35 pm, mickeypearce<EMAIL_ADDRESS>wrote:

Error selecting connection Request connection/openConnection failed with message: Cannot read property 'deps' of undefined

No, this error was the farest I got. I can try again tomorrow with your updates.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (https://link.getmailspring.com/link/9AA65C89-B79B-430B-8E34-BE7AE1D89EE4@getmailspring.com/0?redirect=https%3A%2F%2Fgithub.com%2Fmtxr%2Fvscode-sqltools%2Fpull%2F120%23issuecomment-467486184&recipient=cmVwbHkrMDAwYWNiZTk3OWUyNDBiOGMwMjM1YjY2NjEzMzNlZjJlN2JhNjMxY2VmZDg2ZmY0OTJjZjAwMDAwMDAxMTg4ZDFlYjc5MmExNjljZTE4MGI3ODc4QHJlcGx5LmdpdGh1Yi5jb20%3D), or mute the thread (https://link.getmailspring.com/link/9AA65C89-B79B-430B-8E34-BE7AE1D89EE4@getmailspring.com/1?redirect=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAArL6Q8wTHdrAhCq3emBV2_DdKD69ovIks5vRVQ3gaJpZM4aT_Ag&recipient=cmVwbHkrMDAwYWNiZTk3OWUyNDBiOGMwMjM1YjY2NjEzMzNlZjJlN2JhNjMxY2VmZDg2ZmY0OTJjZjAwMDAwMDAxMTg4ZDFlYjc5MmExNjljZTE4MGI3ODc4QHJlcGx5LmdpdGh1Yi5jb20%3D).

Please, see my comments and if it's working, I'll merge this tonight into master.
After this I need to do some polishing, add docs and troubleshooting before this release.
I would appreciate if you could help me with oracle docs too.

I added aliases to connection pools so that now each database (Oracle) uses its own pool for retrivieng connections. Otherwise using two or more different databases at the same time resulted in an error...
I tested having simultaneously opened connections to two different database and it works for me now, on Windows at least.

@mtxr , One more thing.
When config (settings.json) gets changed and reloaded, _poolCreated gets discarded but pool persists. Then when you try to connect it fails with "pool already exists". Don't know if this could be solved with proper release of oracledb, but for now I would just add:
try {
      await this.lib.getPool(this.poolName);
    } catch {
      await this.lib.createPool({
        connectString,
        password: this.credentials.password,
        user: this.credentials.username,
        poolAlias: this.poolName
      });
    }

to check if pool exists before creating one. Can I push to master or what do you think?

Why not to create poolName using getDbId function?

I could use getDbId, but after config reload (changes not related to sqltool for example), connection explorer gets disconnected and connected automatically back, it tries creating pool with the same name again.


Fixes #1045 - Correctly resolve mapped drive on Windows

Description
fs.realpath.native in NodeJS uses the Win32 API function GetFinalPathNameByHandle() on Windows hosts,
therefore a given path must follow the guidelines for Win32 API file functions.
Drive letters on Windows need to end on a backslash according to the Win32 File Naming Conventions (https://docs.microsoft.com/en-us/windows/win32/fileio/naming-a-file)
Omitting the backslash results in Windows treating the remaining path components as a relative path starting from the current directory on the specified disk (https://docs.microsoft.com/en-us/windows/win32/fileio/naming-a-file#fully-qualified-vs-relative-paths)
I tested this implementation for several repositories on Windows 10 using:

[x] Repositories on UNC path
[x] Repositories on mapped drive
[x] Repositories on local drive

Checklist

[x] I have followed the guidelines in the Contributing document
[x] My changes follow the coding style of this project
[x] My changes build without any errors or warnings
[x] My changes have been formatted and linted
[x] My changes include any required corresponding changes to the documentation
[x] My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
[x] My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

Incredible investigation! Thank you so much for getting to the bottom of this!

Incredible investigation! Thank you so much for getting to the bottom of this!

You're welcome 😄

You're welcome 😄


Control AUX outputs directly

Hi. We have posted a topic on discuss.px4.io. We also wrote to slack, but did not get answer.
The topic is: http://discuss.px4.io/t/create-software-to-control-camera-gimbal/7621
We want to develop a software, that able to control camera and gimbal. We are able to control only yaw and pitch now using vmount and dronecore sdk.
We want to control zoom and auto tracking also, but we are not able to do that. The questions are:
The main question is how to control AUX outputs directly. E.g. use aux4 output to control zoom of the camera.

Can we use more than 2 aux outputs to control camera & gimbal (by pwm signal)
What do you mean under "We ONLY support trigger on pins 5 and 6 when simultanously using AUX for actuator output" ?
How can we send a command to desired aux output ? e.g. Use aux4 output to control zoom of the camera.
It is very important to get quick reply, we rely on your support !
Thanks !

I have attempted to answer this in your discussion board post


eas update does not run postPublish or postExport hooks

Build/Submit details page URL
No response
Summary
I am trying to use sentry-expo and you are supposed to add a postPublish hook as instructed here: https://docs.expo.dev/guides/using-sentry/. This is so you can upload your source maps to sentry after releasing new code.
This doesn't seem to do anything when running eas update.
Managed or bare?
Bare
Environment
expo-env-info 1.0.5 environment info:
System:
OS: macOS 12.6
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 14.18.1 - ~/.nvm/versions/node/v14.18.1/bin/node
Yarn: 1.22.19 - ~/Code/glasspassapp/node_modules/.bin/yarn
npm: 8.9.0 - ~/.nvm/versions/node/v14.18.1/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /opt/homebrew/lib/ruby/gems/3.0.0/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.1, iOS 16.1, macOS 13.0, tvOS 16.1, watchOS 9.1
IDEs:
Android Studio: 2021.2 AI-212.5712.43.2112.8609683
Xcode: 14.1/14B47b - /usr/bin/xcodebuild
npmPackages:
expo: ^46.0.0 => 46.0.17
react: 18.0.0 => 18.0.0
react-dom: 18.0.0 => 18.0.0
react-native: 0.69.6 => 0.69.6
npmGlobalPackages:
eas-cli: 2.6.0
expo-cli: 6.0.8
Expo Workflow: bare
expo doctor
Expected package<EMAIL_ADDRESS>Found invalid:
<EMAIL_ADDRESS>(for more info, run: npm why @expo/config-plugins)
Error output
No response
Reproducible demo or steps to reproduce from a blank project
Create new project
Install sentry-expo
Add the post publish hook
See that it doesn't run

@bentburton Is your issue still running?
I have the same feeling using Expo with EAS in a managed workflow
"expo": "~47.0.3",
"@sentry/react-native": "4.9.0",
"sentry-expo": "~6.0.0",

This issue should be retitled to something like Run postExport after EAS update. This is not a bug but a feature request as it's currently not implemented.
Relevant file and approximate line of where to add the hook: https://github.com/expo/eas-cli/blob/ea553a10253f050036a4b49f24c303a24cdc6ad3/packages/eas-cli/src/commands/update/index.ts#L441
If I have time I can look into making a PR in the future to add functionality.

@MrChip53 Thank you for the precision it was not crystal clear from my understanding 😄

Any updates on this feature?

Any updates here?

hey @bentburton sorry for the delay here, we've released SDK 50 and have improved our integration to Sentry. A couple things to note:

Integration should be as simple as eas update && npx sentry-expo-upload-sourcemaps dist now
EAS Update doesnt support postPublish, but it should not be required anymore.

https://expo.dev/changelog/2024/01-18-sdk-50#first-class-expo-support-in-sentry

to add on to @quinlanj's point here - the reason i introduced postPublish many years ago was because with classic updates, expo-cli publish did not export a dist directory and i decided that instead of adding that, i would pass the manifest, bundle, and sourcemaps through a hook. this was actually not a great idea after all, and when we built EAS Update we made it output the manifest, bundle, and sourcemaps to the dist directory instead to make it more explicit and easier to inspect. so, now there is no need for a hook that passes this data along.


Setup OAuth

So that users can authorize other services to access their data. Should be done in the context of https://github.com/JabRef/JabRefOnline/issues/635.
Implementation: https://www.npmjs.com/package/oidc-provider

Also allow the access token to be submitted via http header (cookie for browser-based requests, header for requests say from severs)

In particular, we may use the "Device code" flow for JabRef Desktop.

General docs: https://datatracker.ietf.org/doc/html/rfc8628
Use verification_uri_complete and open it when user clicks "login in browser". Then user has to only check that the code is indeed the same (to double check the device)
Discussed in more detail here: https://thomaslevesque.com/2020/03/28/using-the-oauth-2-0-device-flow-to-authenticate-users-in-desktop-apps/

Blocked by https://github.com/JabRef/JabRefOnline/issues/635.


Unable to login after system restart

Hello
I have installed caravel on my system using the installation guide. All database login credentials and everything is in place. However whenever I restart my system and start caravel server, I am unable to login.
Whenever I enter login credentials and hit enter, a message appears which says Invalid Login. I have tried this on two different machines (one Ubuntu, and other Mac), and on both the machine, I experienced similar behaviour. As a result, I have to setup caravel again.
Is this a bug, or am I missing something ?
Thanks

By default caravel places the caravel.db database in your tmp folder which is cleared on restart. If you use config.py to change the location of this database then the issue should be resolved.


new forge release?

hi,
can you create a new puppet forge release?
It is still on version 0.1.35 which for example does not yet support debian 9 (stretch)

Hi,
I've manually uploaded the latest version: 0.1.41 but as long as I cannot see any way (other than travis which I don't like) to automate releases this is still a manual process. I would suggest to use Puppetfiles with tags:
https://docs.puppet.com/pe/3.8/r10k_puppetfile.html
mod 'systemd',
  :git => 'https://github.com/NTTCom-MS/eyp-systemd.git',
  :tag => '0.1.41'

Please bear in mind that applying this using r10k will delete any other puppet module that is not included in the Puppetfile and it will not install dependencies
I have a script that uses r10k to install puppet modules from git repositories that I'm using for testing and module development: puppetmasterless
regards,


allow empty accounts

Description
allow empty accounts
Fixes CCXDEV-10888
Type of change
Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

Testing steps
Please describe how the change was tested locally. If, for some reason, the testing was not done or not done fully, please describe what are the testing steps.
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.31 :warning:

Comparison is base (3d2501b) 86.86% compared to head (f84ebb9) 86.56%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #62      +/-   ##
==========================================
- Coverage   86.86%   86.56%   -0.31%
==========================================
  Files          21       21
  Lines         571      573       +2
==========================================
  Hits          496      496
- Misses         75       77       +2

Impacted Files
Coverage Δ

..._messaging/publishers/rule_processing_publisher.py
94.28% <100.00%> (-5.72%)
:arrow_down:

...x_messaging/publishers/workloads_info_publisher.py
94.44% <100.00%> (+0.15%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


fix(test): allow more time between tests to allow che and jenkins to …

…go idle

Len, if you want to run tests against idled che/jenkins you can just call "oc idle che-host" before running the tests. (I guess this approach will work for jenkins too).

This won't work for Jenkins atm as Idler will wake Jenkins up if it fits into a time span where it should be running after build finished...but this behaviour might be something we want to change..


Bug: | MissingPropertyError > core | A required property is missing

What happened?
Look at logs
Version
latest (ghcr.io/diced/zipline or ghcr.io/diced/zipline:latest)
What browser(s) are you seeing the problem on?
Chromium-based (Chrome, Edge, Brave, Opera, mobile chrome/chromium based, etc)
Zipline Logs
| MissingPropertyError > core
  |   A required property is missing

   Collecting page data  ..2023-12-18 12:38:08,015 AM error [config] Config is invalid, see below:
2023-12-18 12:38:08,017 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

   Collecting page data  ...2023-12-18 12:38:08,251 AM error [config] Config is invalid, see below:
2023-12-18 12:38:08,257 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

2023-12-18 12:38:08,269 AM error [config] Config is invalid, see below:
2023-12-18 12:38:08,271 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

   Collecting page data  .2023-12-18 12:38:08,374 AM error [config] Config is invalid, see below:
2023-12-18 12:38:08,375 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

2023-12-18 12:38:08,428 AM error [config] Config is invalid, see below:
2023-12-18 12:38:08,430 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

   Collecting page data  ..2023-12-18 12:38:09,811 AM error [config] Config is invalid, see below:
2023-12-18 12:38:09,813 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

   Collecting page data  ...2023-12-18 12:38:10,019 AM error [config] Config is invalid, see below:
2023-12-18 12:38:10,021 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

2023-12-18 12:38:10,059 AM error [config] Config is invalid, see below:
2023-12-18 12:38:10,071 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

2023-12-18 12:38:10,131 AM error [config] Config is invalid, see below:
2023-12-18 12:38:10,143 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

   Collecting page data  .2023-12-18 12:38:10,308 AM error [config] Config is invalid, see below:
2023-12-18 12:38:10,310 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

   Collecting page data  ..2023-12-18 12:38:10,473 AM error [config] Config is invalid, see below:
2023-12-18 12:38:10,476 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

   Collecting page data  ..2023-12-18 12:38:11,137 AM error [config] Config is invalid, see below:
2023-12-18 12:38:11,141 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

   Collecting page data  ...2023-12-18 12:38:11,235 AM error [config] Config is invalid, see below:
2023-12-18 12:38:11,237 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

   Collecting page data  ...2023-12-18 12:38:11,880 AM error [config] Config is invalid, see below:
2023-12-18 12:38:11,882 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

2023-12-18 12:38:11,951 AM error [config] Config is invalid, see below:
2023-12-18 12:38:11,954 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

2023-12-18 12:38:11,969 AM error [config] Config is invalid, see below:
2023-12-18 12:38:11,973 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

   Collecting page data  .2023-12-18 12:38:12,704 AM error [config] Config is invalid, see below:
2023-12-18 12:38:12,719 AM error [config] CombinedPropertyError (1)
  Received one or more errors

  input.core
  | MissingPropertyError > core
  |   A required property is missing

> Build error occurred
Error: Jest worker encountered 4 child process exceptions, exceeding retry limit
    at ChildProcessWorker.initialize (/home/container/node_modules/next/dist/compiled/jest-worker/index.js:1:11580)
    at ChildProcessWorker._onExit (/home/container/node_modules/next/dist/compiled/jest-worker/index.js:1:12545)
    at ChildProcess.emit (node:events:519:28)
    at ChildProcess.emit (node:domain:488:12)
    at ChildProcess._handle.onexit (node:internal/child_process:294:12) {
  type: 'WorkerError'
}
   Collecting page data  .ERROR: "build:next" exited with 1.

Browser Logs
No response
Additional Info
No response

+1, WSL <IP_ADDRESS> Ubuntu 20.04 LTS

It could be that you haven't set your config with a .env.local from .env.local.example if you have the original error when starting.

https://github.com/diced/zipline/issues/404#issuecomment-1546767837

Are you sure that it contains the necessary variables: CORE_SECRET and CORE_DATABASE_URL?

Are you sure that it contains the necessary variables: CORE_SECRET and CORE_DATABASE_URL?

Yes

CORE_RETURN_HTTPS=true
CORE_SECRET="actual secret"
CORE_HOST=real hostname.com
CORE_PORT=REAL PORT
<EMAIL_ADDRESS>CORE_LOGGER=false
CORE_STATS_INTERVAL=1800
CORE_INVITES_INTERVAL=1800
CORE_THUMBNAILS_INTERVAL=600
If you want the full file let me know

I fixed the issue by deleting the files and recreating them.


add bookmarks page

This PR will add code for the design of the Bookmarks page
In this PR, 599 additions and 45 deletions lines of code

many files are just icons
some files contain only 5 to 6 lines of code

Please review this PR and give your feedback

Live Preview

Thank you for reviewing,


Extend StorageExample to show how to add ACLs to blobs and buckets

This PR adds examples to StorageExample that show how to add different types of ACLs to existing blobs/buckets. This fixes #1012.

Coverage remained the same at 86.034% when pulling f3d3be215f55ce54be28a073abc3b010a9efd57a on mziccard:storage-example into 11f457382068ce5b5266028000bc4ee51c0521ef on GoogleCloudPlatform:master.

/cc @lesv

Coverage decreased (-0.007%) to 86.028% when pulling 6e859c945bb4122131aa1b69d50faaf13206a980 on mziccard:storage-example into 11f457382068ce5b5266028000bc4ee51c0521ef on GoogleCloudPlatform:master.

Where do you expect this to be used?
If it's our formal docs, you are missing // [START sectionName] and // [END sectionName] tags.
Putting them in threads doesn't seem super clear, but that probably started before your time.

Where do you expect this to be used?
If it's our formal docs, you are missing // [START sectionName] and // [END sectionName] tags.

 We just reference them in our READMEs (see here for example).

Putting them in threads doesn't seem super clear, but that probably started before your time.

What do you mean with this?

I think you'll find that they will start being more in formal docs quickly.

I think there are a couple of AI's you agreed to above, do them, and it's ok to merge.  I'll figure out what I want as next steps and write an issue for it.

Coverage increased (+0.03%) to 86.065% when pulling 44cb081e08f5297ce91777210ad85e92e6516d2f on mziccard:storage-example into 11f457382068ce5b5266028000bc4ee51c0521ef on GoogleCloudPlatform:master.


Fix adversarial catchup: Download breadcrumbs when catching up

Right now, we catchup to any given state blindly. The snark doesn't say anything about the validity of the staged ledger so we need to know it's correct somehow.
We can fix this by either (1) waiting until we know this particular state is valid (when it becomes the new locked-tip) or (2) requesting backwards for external-transitions until we can connect to a path we have from the locked-tip.
We are leaning towards (2) right now because there could be an attack scenario if an adversarial manages even a very short partition.

Invalidated by the new transition frontier


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_9dkxxp5bd1/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_9dkxxp5bd1/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_9dkxxp5bd1/clone/recipe/meta.yaml:26: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_9dkxxp5bd1/clone/recipe/meta.yaml:53: missing_description: The recipe is missing a description
ERROR: /tmp/abs_9dkxxp5bd1/clone/recipe/meta.yaml:53: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_9dkxxp5bd1/clone/recipe/meta.yaml:53: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_9dkxxp5bd1/clone/recipe/meta.yaml:53: missing_dev_url: The recipe is missing a dev_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_0449yq5f9p/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_0449yq5f9p/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_0449yq5f9p/clone/recipe/meta.yaml:26: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_0449yq5f9p/clone/recipe/meta.yaml:53: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_0449yq5f9p/clone/recipe/meta.yaml:53: missing_description: The recipe is missing a description
ERROR: /tmp/abs_0449yq5f9p/clone/recipe/meta.yaml:53: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_0449yq5f9p/clone/recipe/meta.yaml:53: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


feat!: rename request_max_block_number

Follow up from #5490

Docs Preview
Hey there! 👋 You can check your preview at https://6616d9732e1b4a33f56eb459--aztec-docs-dev.netlify.app

Benchmark results
Metrics with a significant change:

batch_insert_into_append_only_tree_32_depth_ms (128): 72.0 (-37%)
batch_insert_into_append_only_tree_32_depth_ms (2048): 871 (-61%)
batch_insert_into_append_only_tree_32_depth_hash_ms (128): 0.444 (-38%)
batch_insert_into_append_only_tree_32_depth_hash_ms (2048): 0.414 (-61%)
batch_insert_into_indexed_tree_20_depth_ms (128): 107 (-31%)
batch_insert_into_indexed_tree_20_depth_ms (2048): 1,313 (-25%)
batch_insert_into_indexed_tree_20_depth_hash_ms (128): 0.482 (-31%)
l2_block_processing_time_in_ms (32): 4,813 (-26%)
note_successful_decrypting_time_in_ms (8): 231 (-19%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
676
676
676

l1_rollup_calldata_gas
6,412
6,400
6,388

l1_rollup_execution_gas
585,745
585,733
585,721

l2_block_processing_time_in_ms
1,304 (-1%)
:warning: 4,813 (-26%)
9,309 (+1%)

note_successful_decrypting_time_in_ms
:warning: 231 (-19%)
610 (+14%)
1,043 (+10%)

note_trial_decrypting_time_in_ms
34.2 (-62%)
19.1 (-57%)
66.6 (+10%)

l2_block_building_time_in_ms
10,453
37,409 (-3%)
73,111 (+1%)

l2_block_rollup_simulation_time_in_ms
10,338
37,010 (-3%)
72,344 (+1%)

l2_block_public_tx_process_time_in_ms
4,945 (-1%)
19,283 (-7%)
38,550 (+1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,985 (-1%)
26,297 (-1%)

note_history_successful_decrypting_time_in_ms
1,206 (-2%)
2,222 (-4%)

note_history_trial_decrypting_time_in_ms
43.8 (-21%)
87.0 (-8%)

node_database_size_in_bytes
18,628,688
34,869,328

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
143 (-3%)
44,377
26,164

private-kernel-inner
153 (-3%)
71,744
26,164

private-kernel-ordering
107 (-5%)
50,830
39,026

base-parity
3,916
128
311

root-parity
1,087 (-1%)
1,244
311

base-rollup
13,309
110,464
861

root-rollup
49.3
4,359
725

public-kernel-app-logic
79.5 (-1%)
47,695
40,661

public-kernel-tail
111 (-3%)
53,372
7,125

merge-rollup
11.1 (+8%)
2,568
861

public-kernel-teardown
76.4
47,695
40,661

public-kernel-setup
75.3
47,695
40,661

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.1 (-10%)
16.2 (-9%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.585 (-10%)
0.499 (-9%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
45.9 (-1%)
:warning: 72.0 (-37%)
231
444 (-1%)
:warning: 871 (-61%)
1,742 (+1%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.470 (-1%)
:warning: 0.444 (-38%)
0.420 (-1%)
0.415 (-1%)
:warning: 0.414 (-61%)
0.416 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
54.3 (-1%)
:warning: 107 (-31%)
338 (+1%)
659 (-1%)
:warning: 1,313 (-25%)
2,636 (+2%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
105
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.479 (-1%)
:warning: 0.482 (-31%)
0.459
0.454 (-1%)
0.454 (-24%)
0.455 (+1%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.1 (-1%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.533 (-2%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
40,372
496,534

Transaction size based on fee payment method

Metric
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
905
1,161
1,377

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,185
822 (-4%)
3,998 (-1%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
420 (-1%)
281 (-1%)
732
368
1,198
435


A: hello!Flask how to solve this problem

http://denizeren.net/2014/07/14/flask-file-descriptor-inheritance-problem/

Seems like that page contains the fix they used.  However, you'll need to provide a Minimal, Complete, and Verifiable Example that demonstrates the issue you're having in order for us to take a look at it.


Resample rule greater than a day raises NotImplementedError

If I try to do df.resample("5H") I'll get the error:

NotImplementedError: Resampling frequency <5 * Hours> that does not evenly divide a day is not implemented

Which I understand.
But, if I try df.resample("2D") I also get the same error.
Although I did not understand the internals completely, I would expect that if df.resample("1D") is possible, then df.resample("2D")should also be (easily?) possible. But maybe it's not that simple!
Is this really that case that  resampling frequencies greater than a day are not supported and difficult to implement?
Or were they simply not considered at implementation time?
I could attempt a PR but some guidance would be required...

I am closing this in favor of #6271, but `df.resample("2D") should definitely work, so if you are still experiencing this issue, please reopen.


Add support for passing common args via environment variables

RAI_ENGINE
RAI_DATABASE
RAI_PROFILE
...

Precedence would be:

command line > sticky settings > environment variables.

Note: This issue has been migrated to https://relationalai.atlassian.net/browse/RAI-7186.
This link is only accessible to employees of RelationalAI.


neuralweights extracts coefficients in wrong order from nnet package

Hi. It seems to me that using neuralweights to  extract weights from a network from the nnet package results in the weights being placed in the wrong order.  The result is that if you use these weights for manual prediction, the predictions will be wrong.  (Note that I believe this issue is exacerbated by poor documentation on nnet's part.)
I've included a reproducible example below.
library(nnet)

library(NeuralNetTools)

# use the iris dataset. It's small and has 3 outcomes
dat <- iris

# train the model
f <- multinom(Species ~ ., data = dat)

plotnet(f)

# make a prediction with predict() as a baseline
p_f <- predict(f, dat, type = "prob")

# extract weights with neuralweights()
w_nnt <- neuralweights(f)

# do a manual prediction with w_nnt and softmax model
p_f_nnt <- cbind(1, 1, as.matrix(dat[, -5])) %*% t(do.call(rbind, w_nnt[[2]]))

p_f_nnt <- exp(p_f_nnt) / rowSums(p_f_nnt)

# we get very different predictions
# compare with the correlation coefficient
for (j in 1:ncol(p_f)) {
  print(cor(p_f[, j], p_f_nnt[, j]))
}

# extract the weights using coef
# coefficients for the first outcome are zero for all variables as a baseline
# not sure why they did it this way. This is the first time I've ever seen it
# done this way.
w_f <- rbind(0, coef(f))

rownames(w_f)[1] <- "setosa"

# get prediction and
# apply the softmax function to get probabilities
p_f_manual <- cbind(1, as.matrix(dat[, -5])) %*% t(w_f)

p_f_manual <- exp(p_f_manual) / rowSums(exp(p_f_manual))

# we get the same predictions
# compare with the correlation coefficient
for (j in 1:ncol(p_f))
  print(cor(p_f[, j], p_f_manual[, j]))

@TommyJones Thanks for the reprex.  The documentation for the nnet package is a bit sparse.
Looks like you are using a multinomial model, via nnet.  I haven't tested the package with these models at all, so the order of the weights might be incorrect.  Have you checked the order without using a classification model?  Could just be that I need to add a disclaimer that the functions don't work for multinomial models.

I have not tried it with a non-multinomial classification model. But thank
you for the tool. I was quite stuck and this package got me unstuck, even
if the weights weren't in the right place.
On Mon, Apr 6, 2020 at 8:17 AM Marcus W Beck<EMAIL_ADDRESS>wrote:

@TommyJones https://github.com/TommyJones Thanks for the reprex. The
documentation for the nnet package is a bit sparse.
Looks like you are using a multinomial model, via nnet. I haven't tested
the package with these models at all, so the order of the weights might be
incorrect. Have you checked the order without using a classification model?
Could just be that I need to add a disclaimer that the functions don't work
for multinomial models.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fawda123/NeuralNetTools/issues/22#issuecomment-609758233,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGCQAXI3BUHIUQ327OSTK3RLHB4DANCNFSM4LPKRYAQ
.

If it helps, you can see how the weights are extracted here: https://github.com/fawda123/NeuralNetTools/blob/eb2f60ab7c16bb57a2a7bf006279f4a4bf6c4e4b/R/NeuralNetTools_utils.R#L104
It's super hacky, but it gives the correct order needed for the rest of the functions in the package.  Model predictions are another question...


Default styles?

The default styles are really stock

No insets
Initial UITextView font
No placeholder
Placeholder color is black

Would anyone want some basic stock styles? Maybe just a nicer first-run experience for those that don't want to fiddle w/ too many customizations.

It could be interesting to move these types of things to a Layout protocol, and provide 2-3 layouts with the sdk
Aside from that, I think at a minimum there should be:

Reasonable insets, 12pt around?
Button text "Send"
Placeholder "Message..."


Remove auto open of the browser on gulp serve

It's a real design choice here. I just don't want anymore that my browser take focus after the gulp serve. Often I run serve several times and get multiple tabs opened.

👍

:+1: but IMO 'open: false'  is better

Yeah, that was too quick. browser: [] works but open: false is better. In addition, it seems that the way I put it, it's not working.


Make it possible to use different cache and session stores than redis

Right now it seems that some parts of the package require a cache store with tagging functionality. We will need to generalize more.

Probably fixed with #13


Missing background images when sending test email using vscode through mailjet

When sending a test email through mailjet using the vscode extension, images other than background images show up OK, but background images are blank.

Try to send it with putsmail.com if it works fine then open a ticket on Mailjet side, not here

Try to send it with putsmail.com if it works fine then open a ticket on Mailjet side, not here


[Rule request] disable /* */ comments

In a codebase, it' can be easy to accidentally type doc comments with a leading /*. If you use //-style comments everywhere else, this lint could catch cases where you meant to write a doc comment.

Happy to take a PR for a new rule for this.

PR for this at #4008

This has been merged


Unable to compile

Hello,
I'm trying to follow this tutorial and I am getting this error trying to compile haskell-foreign-rust:
foreign-rust   > configure
foreign-rust   > Configuring foreign-rust-0.1.0...
foreign-rust   > build
foreign-rust   > Preprocessing library for foreign-rust-0.1.0..
foreign-rust   > Building library for foreign-rust-0.1.0..
foreign-rust   > [ 1 of 17] Compiling Data.Structured
foreign-rust   >
foreign-rust   > /tmp/stack-d01f5ffda1e3e54a/foreign-rust-0.1.0/src/Data/Structured.hs:505:16: error:
foreign-rust   >     Not in scope: type constructor or class ‘Aeson.KeyMap.Key’
foreign-rust   >     Perhaps you meant ‘Aeson.KeyMap.KeyMap’ (imported from Data.Aeson.KeyMap)
foreign-rust   >     Module ‘Data.Aeson.KeyMap’ does not export ‘Key’.
foreign-rust   >     |
foreign-rust   > 505 |     goField :: Aeson.KeyMap.Key -> NonEmpty Builder -> NonEmpty Builder
foreign-rust   >     |                ^^^^^^^^^^^^^^^^

This is due to a missing lower bound on aeson, sorry. You will need aeson version 2.1 or higher; it should also not be too difficult to modify foreign-rust to support other versions; PR welcome! :-)

I was able to compile it through Stack by pinning aeson, borsh, vector and generically.
However, c2hs is failing at processing the example:
$ c2hs ../rust-wrapper/rust_wrapper.h GettingStarted.chs
In file included from GettingStarted.chs.h:2:
../rust-wrapper/rust_wrapper.h:7:10: fatal error: cstdarg: No such file or directory
    7 | #include <cstdarg>
      |          ^~~~~~~~~
compilation terminated.
c2hs: Error during preprocessing custom header file

This is due to the fact that the default for the generated header is C++, not C, and you need to override that. I could have sworn I mentioned this in the blog post but I didn't -- apologies. I've submitted a PR against the blog post to fix that, but meanwhile, here's the relevant new paragraph:

Finally, we need to configure cbindgen and tell it what kind of header file we want; we don't need to call it manually (cargo cbuild will do that for us), but we do need to tell it that we want a C header file, not a C++ header file:
include_guard = "RUST_WRAPPER_H"
include_version = false
language = "C"

Incidentally, the code for the blog post is at https://github.com/well-typed/blogpost-purgatory , and you can find it there, too. (That's no excuse, I should have mentioned it in the blog post for sure, but it might help you get unstuck.)
Thanks for your questions!

I was able to compile and use the library using stack.
I have a few more questions but it's no longer related to compiling the code, is there a better way to post the questions?

Feel free to post them here, or else email me.


Improve notifications page

Add help on how to acknowledge notifications
Add message for if there are no pending notifications
Set charset to UTF-8

Rendered:

Re-rendered help message:


TypeError: ONNX node of type RNN is not supported.

I want to convert a simple lstm model to caffe model, and I got the error.

@RORO11  did solve the issue


chore: release 1.1.0

release notes

Mostly because I want to use this right now and I can't 😆

wait wait wait

ok go again 😸

Lgtm
On Fri, Jun 8, 2018, 5:57 PM Justin Beckwith<EMAIL_ADDRESS>wrote:

ok go again 😸
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/ofrobots/require-so-slow/pull/15#issuecomment-395927650,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE0qaZbYADGHqQRlCpRKM30S2cbH73nks5t6x1rgaJpZM4UhE9L
.


List of further datasets

aiSEGCell: Nucleus segmentation in brightfield (I think the data itself is not published yet)
CryoEM Data Portal: collection of Cryo EM datasets with annotations
Cell Tracking Reference Dataset: Cell tracking in fluorescence LM

Organoid Segmentation (from Sushmita's talk, will add links later)

OrgaSegment

Compares specific measurements on organoids vs. fluorescence

Orga Extractor

Morphology analysis for organoids derived from segmentation

OrganoID

Segmentation + Tracking

Publication from Alves lab: CellPose + Stardist (is dataset public??)

(double check which ones publish the training data)

Histopathology:

https://zenodo.org/records/10940194 - Melanoma dataset with Nuclei and Tissue segmentations.

EM:

https://www.nature.com/articles/s41467-024-48272-7 - Segmentation of mitochondria, ER and other organelles and structures from rat liver in FIB-SEM (very large dataset, and pretty cool paper)


Angular SSR: ReferenceError: Element is not defined

Describe the bug
I'm trying to enable SSR on my Angular project with SAP Spartacus, but when i instance with 'new AdyenCheckout(config)' and enable Angular Universal SSR, then crashes with message:
ReferenceError: Element is not defined
To Reproduce
Implement Angular Universal SSR on project.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

If uncomment the line 151, then crashes.
Desktop (please complete the following information):

OS: Windows 10 x64
Browser: Chrome 94
Node Version: 12.22.1
Npm Version: 6.14.12
Package Version: "@adyen/adyen-web": "^4.7.3",
Angular Version: 10.2.3

Additional context
I tryed to isolate with isPlatformBrowser() from '@angular/common' and make a conditional import of component, but no success.

Hi @drafenous
Thanks for reaching out. Apparently the issue is that Angular Universal is executed in the server, therefore some of the browser APIs and capabilities might be missing there, like the window, document, navigator, etc. And adyen-web interacts with some of these API's. This can be the reason why you get this error.
You can read more about it here:
https://angular.io/guide/universal#working-around-the-browser-apis
As it states in the docs, Angular provides some abstractions for Location and Document browser API's. But you still need to go around and adapt it for the window object. There is a blog post here that might be useful: http://michaelborisov.com/blog
I'll close this ticket, but feel free to reopen if you think this is still an issue related to this library.
Hope this helps!

@drafenous, did you try to force the load of module/component that uses the library on the client side instead of loading it in the server?
I am aware that you are using Angular SSR, but for example, using Next.js framework you can force the load of a specific module on the client side (https://nextjs.org/docs/advanced-features/dynamic-import#with-no-ssr). Can you try a similar approach in your Angular app?
There were other issues in the past related to SSR which were fixed by dynamic imports. Please have a look, they might help you. #902 #233

This semantics is not possible in Angular 10.
However, we took a new approach to dynamic import that we found ended up working.
Here is our validation if it is of interest to you:
          if (this.isBrowser) {
            import('@adyen/adyen-web').then(module => {
              this.checkout = new module.default(this.configuration);
              this.changePaymentMethod(null, 'creditcard');
            });
          }

I am aware that Next.js syntax isn't possible in Angular 10, but I meant about the approach that they did, using dynamic imports.
I'm glad that dynamic imports worked for you, and thanks for sharing your solution. It might help others.

Hey @ribeiroguilherme - totally understandable that Adyen wouldn't choose to support SSR for this library. But I would say that this is one of the very few libraries that crash loudly just when importing on the server-side, instead of gracefully recognising a server environment and halting any execution - or failing when actually using the library. Dynamic importing is a doable workaround, but this complicates things when using TypeScript.
I've no idea of the complexity of implementing such a thing, but it would be significantly easier if we could import AdyenCheckout then check if window is defined (or similar check) before instantiating.

@jonlambert thanks for your input. That is a very good suggestion, and we would like to have a look into it.
Do you know by any chance examples of libraries that halt the execution in the server environment gracefully? So we can investigate properly how they perform those checks and try to find a good fit for our case.

No worries! That would be great.
So you could early-exit and expose some stub methods for code running in SSR, but that's probably a sledgehammer to crack a nut. The problem can usually be avoided by removing any code that expects a browser environment, executing at the top level of a file (outside classes or functions). I can't test this as the problem doesn't seem to appear when building this library locally, but this line looks like it's trying to access Element.prototype:
https://github.com/Adyen/adyen-web/blob/9430ca6ed9aeff69f15d03258088fd57a1ae041c/packages/lib/src/polyfills.ts#L26
...which might explain the error we're getting above.
Quick fix might be to wrap that in a conditional, checking for the presence of Element, CharacterData and DocumentType!

@jonlambert this file (polyfills.ts) is going to be removed on #1343, which is about to be merged quite soon. It will most likely fix the error that was pointed out on this issue 👍

@ribeiroguilherme Ah that's great to hear 🙂 I managed to get a dev setup working and debugged the issue further. Turns out there are a few other parts of the codebase that access window at the top level. I've updated #1384 to add presence checks before using, and can now include the library on my machine without SSR blowing it up 🙂

Same problem for Nuxt with Vue 2.
@drafenous Solution worked.
await can't be used tho, only the .then syntax.


There is a typo.

Please change the word "compony" to "Company".
Under "Birth Of Containerization ( For Software Industory )" 2nd line.

Please change the word "compony" to "Company".
Under "Birth Of Containerization ( For Software Industory )" 2nd line.

ps: I'm creating issue for the first time, please let me know if any mistakes.

Let me fix it! Thanks 😊 keep it coming all typo mistakes

@kspnec https://discord.gg/rEvr7vq join community! And post there any issues.

@kspnec resoloved


Fix text layer being deselected after typing when going to edit its Properties panel, making it disappear immediately

Closes #2127

Introduced the active_panel field in DocumentMessageHandler to track the currently active interface panel.
Added a match expression in set_editing fn to verify if the active panel is the Properties Panel, preventing its deselection during edits.

https://github.com/user-attachments/assets/dd34f593-b3e0-4f54-b65a-f8cde3af87ff

@adamgerhant could you please review this in regards to the introduced ActivePanel code, to make sure it doesn't conflict with your existing implementation of the related concept of the active panel used for showing whether the graph's subnetwork or Layers panel is active?

@adamgerhant the text tool code can't access the portfolio message handler.

It could be passed as a parameter, but I think it makes sense for the active panel to be stored on a per document basis instead of the entire portfolio/application. This means switching tabs will store independent active panel states. In this case the active panel should be removed from the portfolio message handler.

Why not just leave the text layer as selected when editing ends?

I haven't worked much with design tools much, but doesn't keeping text layers selected after editing deviates from standard design tool patterns where edits typically end in deselection.
Persistent selection might create risk for accidental modifications - users may unintentionally resize/move/style the text layer when they meant to work on something else.
In this implementation the layer while in editing is deselected when clicked anywhere else expect properties panel

@Sidharth-Singh10 I like where you're going with that thought process, although I believe for consistency with other drawing tools in Graphite, it would be best to keep it selected after clicking out. If you draw with any other tool like Rectangle or Ellipse, you'll notice if you look in the Layers panel, that layer remains selected after finishing drawing it. So it'd be best if the layer remains selected if you click anywhere in the viewport, or anywhere else in the UI, after interactively editing the text. Thanks :)

!build

Can I have you add another little improvement to this PR while I'm thinking about it? If the user is editing the interactive text in the viewport, and hits escape, it should delete the entire text layer if escape was pressed when the text was empty. So if a user creates text in the wrong spot but doesn't type anything (or types then deletes it all), escape will delete the layer so it doesn't get invisibly stuck behind.

!build

!build

It looks like Escape is working but right-click isn't for canceling an empty text layer for it to get deleted. Can you please look into making it work when it's empty and the user right-clicks?
(Also note that I rebased, so you'll need to pull before continuing to code on it.)

Hey @Keavon, I added an entry for MouseRight with action_dispatch set to TextToolMessage::CommitText in input_mapping.rs, but it doesn't seem to have any effect.
To confirm I was editing the correct location, I commented out the MouseLeft entry with action_dispatch set to TextToolMessage::Interact. This prevented placing or editing text, which suggests my changes are indeed in the right place.
Could you nudge me in the right direction?

I'll ping @0HyperCube to see if he can answer your question about that.

@Sidharth-Singh10
The actions that are advertised to the input mapper system don't include the commit text when the text is not being edited:
fn actions(&self) -> ActionList {
	match self.fsm_state {
		TextToolFsmState::Ready => actions!(TextToolMessageDiscriminant;
			Interact,
		),
		TextToolFsmState::Editing => actions!(TextToolMessageDiscriminant;
			Interact,
			Abort,
			CommitText,
		),
	}
}

Any time any mouse button is pressed that is not inside the text input, input.ts:175 sends editor.handle.onChangeText which sets editing to false.

Thanks for pointing this out!

!build

@Sidharth-Singh10 your solution to emulate an escape keypress from the frontend TS code is a hack that works but isn't very clean. Can you please take into account what Hypercube wrote and implement a solution in the backend Rust code? In other words, you need to have something run before the tool state leaves Editing and returns to Ready to check if that TS code he mentioned is causing it to return to Ready.

!build


Google Home Demo: Modernize application and update dependencies.

This sample code seems to be somewhat outdated. Code changes are needed to bring it up to date. But that just makes sense if this samples is still relevant which I don't have the context to know.
@meteatamel , what do you think? Look at the renovate PRs that I'm closing pointing to this issue for the dependencies that need updating. Thanks!

@amanda-tarafa AFAIK, I don't think this sample is being used from any docs or anywhere else. I don't have plans to keep it up to date. My suggestion is to either keep the sample as is, without any library updates.

OK, I'll think about it. Specially if they are not being referenced by any docs, I'm more inclined to delete it. We can always bring it back to live if we decide we want to include it somewhere.

Closed by #2248


canDeactivate is not being called.

I have a simple route config in which I am registering functions for canActivate and canDeactivate. Can activate is working fine as expected but canDeactivate is not getting called when I am navigation through. When I tried to debug I found canDeactivate function was not registered in _canDeact.
Is there anything I am missing while registering route or in route config?
My route option contains:
{
useHash: true,
hashPrefix: '',
trailingSlash: true,
defaultRoute: 'dashboard.listings-my',
autoCleanUp: true
}

Just found that we need to use Router.canDeactivate api instead of providing canDeactivate function in route config.
Just want to understand, why its not being registered in same fashion as canActivate with route config i.e like
{
name: "",
path: "",
canActivate: fn,
canDeactivate: fn // I think support for this is not there.
}
Or did I missed something.


Compiler error when passing named parameters to attributes

Consider the following attribute definition:
type FooAttribute(value : int) =
    inherit System.Attribute()
    new () = new FooAttribute(-1)

I can easily create instances of the attribute like so:
let attr = new FooAttribute(value = 42)

However when adding the same attribute to a type definition:
[<FooAttribute(value = 42)>]
type Bar = class end

The following compile error occurs:
error FS0041: A unique overload for method 'FooAttribute' could not be determined based on type information prior to this program point. A type annotation may be needed. Candidates: new : unit -> FooAttribute, new : value:int -> FooAttribute

The error goes away when removing the parameter name from the attribute:
[<FooAttribute(42)>]
type Bar = class end

I thought named parameters to attributes were always property names. Is that not the case?

In admittedly rare situations, this could improve readability a lot.
@panesofglass No, they are parameter names. Like in C#:
MyMethod(1,2 myOptionalParam: 3)
{
   MyProperty = 4
}

In C#, named arguments to attributes are named properties (See https://msdn.microsoft.com/en-us/library/bfz783fz(v=vs.110).aspx)

Positional parameters are required and must come before any named parameters; they correspond to the parameters of one of the attribute's constructors. Named parameters are optional and correspond to read/write properties of the attribute.

This seems to be the same in F#, as the following code compiles:
type FooAttribute () =
    inherit System.Attribute()
    member val Value = -1 with get, set

[<FooAttribute(Value = 42)>]
type Bar() = class end

In C# Attributes, named parameters are denoted with a param: value, where as properties are specified with Property=Value.
e.g.:
public class TestAttribute : Attribute
{
    public string Prop { get; set; }
    public TestAttribute(int arg) { }
}

[Test(arg: 1, Prop = "2")]^

For F#, I read the MSDN spec as defining that named parameters always refer to properties, whereas positional parameters are ctor parameters:
https://msdn.microsoft.com/en-us/library/dd233179.aspx
e.g.:
[<Test(1, Prop="2")>

This is in contrast to methods, where the = specifies either a named parameter OR a property initialization.

This seems to be caused by this line:
https://github.com/Microsoft/visualfsharp/blob/f08fcd4245256166781e658ef3c53552147ec9c6/src/fsharp/MethodCalls.fs#L297
Which does something special for attributes. Note the comment:
// the assignment of names to properties is substantially for attribute specifications
// permits bindings of names to non-mutable fields and properties, so we do that using the old
// reliable code for this later on.

I don't understand that comment to begin with - "permits bindings to non-mutable fields and properties"? Surely that isn't the case.
Anyway, unfortunately, in this case, the "old reliable code" is not even triggered, because since the unassignedNamedItem remains erroneously empty, the empty constructor is not filtered out in ResolveOverloading here:  https://github.com/Microsoft/visualfsharp/blob/08c6ebcf9fe1de06054793e4e4aa102a695be566/src/fsharp/ConstraintSolver.fs#L2125
IsCandidate tries to figure out if a caller assigns all named arguments for the candidate; since in the attribute case this is empty, neither candidates can be filtered and the code causes a compiler error a bit further down.
In the normal constructor call, the default ctor of FooAttribute gets filtered out and code proceeds as normally.
So I guess my question is - do we still need the "old reliable code"? I'm not sure if there is still any difference, and if so what it would be.

@kurtschelfthout Apologies I didn't answer when you wrote your comment above.  You were exactly right - the code there needed to be adjusted, and I fixed the garbled comment
See https://github.com/Microsoft/visualfsharp/pull/3251
Thanks!

No worries, thanks for fixing!


increase the delete cluster timeout

A few Jenkins jobs failed because of the short timeout of the delete cluster with the following error:
20:06:28  ods-ci-logger: INFO     ocm cluster with name xxxx exists!
20:07:24  ods-ci-logger: ERROR    xxxx not deleted even after an hour. EXITING
increasing the timeout from 60 minutes to 90 minutes
related to jira ticket: RHOAIENG-3917

Are we sure that the timeout extension helps? Does the cluster deletion really can take that much time? :thinking:

Are we sure that the timeout extension helps? Does the cluster deletion really can take that much time? 🤔

Please look at the Jira ticket
you can see links to 3 jenkins jobs that failed with the same timeout error
when I list the clusters they are no longer there.
btw it was on production(not staging)


Scoverage coverageMininum 80.5

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

[x] I acknowledge that all my contributions will be made under the project's license.

Codecov Report

Merging #640 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #640   +/-   ##
=======================================
  Coverage   80.82%   80.82%
=======================================
  Files          78       78
  Lines        5085     5085
  Branches      130      130
=======================================
  Hits         4110     4110
  Misses        975      975

Continue to review full report at Codecov.

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


Test Issue

Issue Title

XYZ title

Issue Description

provide a brief description of the issue

Expected Behavior

what is the expected behavior?

Actual Behavior

what is the actual behavior?

Steps to Reproduce

provide relevant steps and/or prerequisites required to reproduce this issue.

Version Information

Provide the version information.

Hey! I would like to work on this!


Selecting calendar not works

System Information:

Version: 1.5.1
Language: cs
Timezone Offset: +01:00
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/38.0.2125.104 Safari/537.36

What steps will reproduce the problem?

I look to this app :-D

What is the expected result?
I see my calendar :-)
What happens instead?
I don't. I see other's calendars and selected other, but not mine :-(
Screenshot URL:

My calendar: http://www.dark-net.cz/files/cal01.png
My settings: http://www.dark-net.cz/files/cal02.png

As you can see, my next thing is "Míša AJ", but in calendar it is not.
Logs:

Closing this for lack of a response from the original reporter.


Algokit can't find chocolatey and node.js

node: Command not found!
Node.js is required for some Node.js-based templates;
install via algokit project bootstrap within project directory, or via:
https://nodejs.dev/en/learn/how-to-install-nodejs/
npm: Command not found!
chocolatey: Command not found!
If you are experiencing a problem with AlgoKit, feel free to submit an issue via:
https://github.com/algorandfoundation/algokit-cli/issues/new
Please include this output, if you want to populate this message in your clipboard, run algokit doctor -c
that's the error I am getting, I tried installing via powershell then manually still the same thing

Chocolatey is just a warning for debugging purposes so you can ignore that.
Node.js is needed if you are doing a dapp or you want to write typescript smart contracts. If that's the case, and you want/need Node.js - is it installed? What happens when you execute npm in your console?

@CodedX-cyber closing due to inactivity, please reopen if you hit issues with chocolatey otherwise we recommend using winget which is soon to be a recommended way of installing binary version of algokit cli on windows


JWT claim in the form of DateTime get extra double-quote when converted to claims

Hi,
After an upgrade from 4.x to 5.5 of System.IdentityModel.Tokens.Jwt, we are noticing that if we read a Jwt that has a claim as a DateTime, then the value of Claim object is surrounded with double-quote.
Repro:
run this code:
var backendToken = "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJjbGFpbV9hc19kYXRldGltZSI6IjIwMTktMTEtMTVUMTQ6MzE6MjEuNjEwMTMyNloifQ.yYcHSl-rNT2nHe8Nb0aWe6Qu3E0ZOn2_OUidpxuw0wk";

            var handler = new JwtSecurityTokenHandler();

            var token = handler.ReadToken(backendToken) as JwtSecurityToken;
            var dateClaim = token.Claims.First();

            var claimValue = dateClaim.Value;
            Console.WriteLine(claimValue);

Notice that the value of claimValue is "2019-11-15T14:31:21.6101326Z" instead  of 2019-11-15T14:31:21.6101326Z (extra double-quote).
From looking into the code a little it seems that this related to the use of JsonConvert.DeserializeObject without specifying a DateFormatHandling property.
Thanks

Thank you for reporting this issue @oliviervaillancourt.
The fix will be released in v5.6.0.


feat: Together AI Embeddings Support

feat: Together AI Embeddings Support
docs: Updated docs
docs: Fixed an error in example

#66  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tazarov and the rest of your teammates on  Graphite


Remove obsolete shadowsocks code

It seems this code was migrated to https://github.com/greatfire/envoy/blob/master/android/envoy/src/main/kotlin/org/greatfire/envoy/ShadowsocksService.kt at some point in the past, and hasn't been used for a while

@mnbogner just tested a build off this branch, so I'm going to merge


Object can end with curly braket

Ex:
turf
	start {icon = 'forest.dmi'}

Stale?

Stale?

Stale


Multiple Index Errors

Those error messages appear when running the script. Switching through animations with the joystick doesn't work. Tested on Raspberry Pi 3 B+.
pi@raspberrypi:~/pi-hole-visualizer $ sudo ./dns_stats.py
Initiating connection with server.
Successful connection with server.
Traceback (most recent call last):
File "./dns_stats.py", line 436, in
main()
File "./dns_stats.py", line 432, in main
event_loop(args, pw_hash)
File "./dns_stats.py", line 361, in event_loop
args.randomize)
File "./dns_stats.py", line 97, in bar_chart_vertical
domain_min = interval_data[0][0]
IndexError: list index out of range

Took a while, now it works fine.


Use AWS tags to discover ASGs instead of command line arguments

The minion-manager currently uses the --scaling-groups command line argument to find the list of ASGs on which to operate. Everytime the list has to be updated, the minion-manager deployment has to be updated and restarted. This is cumbersome and error-prone.
Instead, the minion-manager should take an AWS tag name and tag value pair as argument and "discover" the ASGs to operate upon. If the user wants to disable use of spot-instances, the user can simply modify the tags in AWS and the minion-manager pod should factor that in.
This is similar to the way the cluster-autoscaler pod runs.

Requirements:
• It should be possible to run minion-manager without requiring explicit list of ASGs
• Minion-manager should "discover" the ASGs to use spot-instances on
• Minion-manager should "discover" the ASGs to not use spot-instances on
• It should be possible to switch between Spot and On-demand instances with the minion-manager
Design:
• Ask users to explicitly tag ASGs with the key "k8s-minion-manager". Only ASGs which have the "k8s-minion-manager" tag will be considered by the minion-manager. Other ASGs will be left alone.
• Minion-manager will query AWS for ASGs with these tags every "--refresh-interval" ... default to 5 minutes
• The "k8s-minion-manager" key can have two possible values:
•  "use-spot": This will make the minion-manager intelligently use spot instances in the ASG
• "no-spot": This will make the minion-manager always use on-demand instances in the ASG. This is useful when someone wants to temporarily switch to on-demand instances and at a later point switch to "use-spot"
What happens when:
1. User runs k8s-minion-manager without any ASG having the "k8s-minion-manager" tag?
k8s-minion-manager should ignore all ASGs. It will simply continue to keep polling AWS for the tags every "refresh-interval" seconds.
2. User runs k8s-minion-manager and add the "k8s-minion-manager" key but does not have correct values?
k8s-minion-manager should log an appropriate message.
3. User runs k8s-minion-manager, adds the "k8s-minion-manager" key and the "use-spot" value to start with. But later wants to not use spot instances.
User should then change the key from "use-spot" to "no-spot". This will indicate to the k8s-minion-manager that the ASG should have all on-demand instances and it will make sure of that
4. User runs k8s-minion-manager, adds the "k8s-minion-manager" key and the "use-spot" value to start with. But later simply removes the tag and the value.
Once the tag is removed, k8s-minion-manager simply considers the ASG to be off-limits and does not act upon it. The ASG will remain in whatever condition it is in.
5. User runs k8s-minion-manager, adds the "k8s-minion-manager" key and the "no-spot" value to start with. But later simply removes the tag and the value.
Same as above.  The ASG will remain in whatever condition it is in.
6. User is running k8s-minion-manager and using spot instances. But now wants to stop using instances forever.
This will be a multi-step process:

Change the value of the "k8-minion-manager" tag to "no-spot".
Wait for the minion-manager to react to this and switch the instances to on-demand. Look at the AWS console for verifying that all instances are on-demand.
After the above, remove the "k8s-minion-manager" tag.
Delete the "k8s-minion-manager" deployment.


minDate is broken when using also time values in input

Usecase:
I want to set minimum datetime that the user can select in the picker. In my case it's equal to current timestamp so I set minDate = new Date()
Problem/example:

minDate is set to 2016-04-09 17:48
User selects some future date in the picker and modifies the time to be something lower. For example: 2016-04-10 15:00
Now users wants to click on the todays date in calendar but nothing happens when he does that. There's no feedback that the datetime is before minDate constraint.
In my usecase I would expect that when user click on the date in calendar then picker would automatically set the datetime to minimum allowed value for that time (if there is such a constraint).

...
I'm not sure why did you close the last issue as the problem is/was the same: https://github.com/Eonasdan/bootstrap-datetimepicker/issues/1302#issuecomment-207766799
You could have started a new issue from it if you wanted to separate them not just ignore the problem ...

@Renkas try to add else if statement before line 880 similar to isValid, but date will be actual moment
} else if (targetMoment.isSame(moment(), 'd')) {
    date = moment();
     ...
}

Excuse me for duplication. Couldn't get a proper codestyle.
So now i've finally got it and made a pull request #1610 to fix this issue.


Migrate to prop-types package.

Fix PropTypes deprecated warning of the lastest React version.

please put prop-types in peer dependencies as well, then we can merge this

    Cannot find module 'react-test-renderer/shallow' from 'react-compat.js'

      at Resolver.resolveModule (node_modules/jest-resolve/build/index.js:151:17)
      at node_modules/enzyme/build/react-compat.js:151:34
      at Object.<anonymous> (node_modules/enzyme/build/react-compat.js:219:5)
Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        2.47s
Ran all test suites.
console.error node_modules/enzyme/build/react-compat.js:158
    react-test-renderer is an implicit dependency in order to support<EMAIL_ADDRESS>Please add the appropriate version to your devDependencies. See https://github.com/airbnb/enzyme#installation

I have a little confuse about peer dependencies. The instruction of prop-types recommend putting it in dependencies for both apps and librabries (https://www.npmjs.com/package/prop-types). So what does peer dependency do?

peer means it has to be installed on the app level by the end user and is not fetched by the library
even though they recommend that, I don’t see any reason to bundle multiple versions what so ever

On Apr 12, 2017, at 1:27 PM, Quốc Khánh<EMAIL_ADDRESS>wrote:
I have a little confuse about peer dependencies. The instruction of prop-types recommend putting it in dependencies for both apps and librabries (https://www.npmjs.com/package/prop-types https://www.npmjs.com/package/prop-types). So what does peer dependency do?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/makeomatic/redux-connect/pull/112#issuecomment-293697411, or mute the thread https://github.com/notifications/unsubscribe-auth/ABol0XpuV9ZBvqxidMpIYE0qLXwBhncsks5rvTObgaJpZM4M70-n.

Published in 5.1.0

@AVVS Why do you think this should be peer dependency? I use typescript so I don't PropTypes. But since I upgrade redux-connect from 5.0.0 to 5.1.0, I need to install prop-types as peer dependency although I'm not using it in my project.

prop-types module is used in the production level code and, therefore, it would either go normal deps or dev + peer. I opted in for peer, because it makes the end user aware of this package being required, while not pinning it to specific version


Allowed Bucket Name Patterns

The allowed pattern does not allow periods to exist in bucket names - even though periods are valid in S3 bucket creation. Perhaps this could be changed to [a-zA-Z][a-zA-Z0-9-.]* to allow periods.
(https://github.com/awslabs/serverless-image-handler/blob/master/deployment/serverless-image-handler.template#L12)[https://github.com/awslabs/serverless-image-handler/blob/master/deployment/serverless-image-handler.template#L12]

I ran into this too. Needed to remove the AllowedPattern to make this work.

Thanks for the comment. I incorporated support for periods into the template and this should get pushed out in the next update.

Would you test out this template, and verify that the new constraint works for your buckets?
https://s3.amazonaws.com/solutions-test-reference/serverless-image-handler/latest/serverless-image-handler.template

Just tested, and there were no problems specifying a bucket that had periods in it.

Template updated. Closing ticket.


WHAM support, command line options and VCF additions for filtering

Marghoob;
This pull request supports work to improve sensitivity filtering with the new inputs we've introduced (Lumpy, Manta, CNVkit and WHAM). The support for WHAM is new in this pull request, along with additions and fixes that don't change MetaSV defaults but allow more flexibility:

It adds a 'NUM_SVTOOLS' INFO field value to know how many tools support a call. For multi-method, overlapping approaches this was not straightforward to use for post-call filtering with the current targets.
Updates the template VCF to include additional metrics that were present in the VCF but not the header (so bcftools could not use them as filter targets).
There are two new command line arguments to control breakpoint assembly: --assembly_max_tools controls when to do reassembly. It previously did this for calls with support from 1 tool, which is still the default. However I've found assembly useful for filtering calls with support for more than 1 caller so this makes it configurable. --assembly_pad specifies the padding bases to use for assembly and breakpoint resolution. I've found increasing the pad to 1000 gives more resolution for filtering so added this a command line target. It defaults to the previous version of 500.

With these changes and post-call filtering I'm able to improve MetaSV precision without a large loss in sensitivity. Here is the current validation against DREAM synthetic 4 deletions:

versus the previous version:

Thanks much for looking at this. Please let me know if you have any questions about any of the additions.

Thanks, Brad. This is very helpful. It looks like MetaSV becomes more effective beyond 2000bp--the precision is quite low before that. The precision has improved but it still seems low.

Marghoob;
Agreed, I did the most work on trying to improve low and high but they're still not great. I'm going to spend some time with lumpy filters to see if I can improve those, which might help the overall input to MetaSV but couldn't find any other tweaking parameters from the breakend assembly to improve it beyond what I've got so far. Definitely still a work in progress but improving.

Thanks for helping improve MetaSV. Yeah, we are definitely getting the accuracy towards the right direction. I will go ahead and merge this one as it looks okay.


create new short link for Heroku app for flier

bit.ly can customize the name
opt: bit.ly/recycling
or something like that

http://bit.ly/recyclingrequest


I want to unregister

Hello
I want to unregister Hacker plan.
But I forgot my username and password.
How can I unregister?
thank you

Reseting password page does not work.


Add smoketest MVP

Added smoketest that pulls the keyvault sdks from vcpkg and should run it along side the other samples.
The smoketest creates one of each clients and calls a single api from each sdk to ensure all is working properly.
#closes 3035
Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[x] C++ Guidelines
[x] Doxygen docs
[x] Unit tests
[x] No unwanted commits/changes
[x] Descriptive title/description

[x] PR is single purpose
[x] Related issue listed

[x] Comments in source
[x] No typos
[x] Update changelog
[x] Not work-in-progress
[x] External references or docs updated
[x] Self review of PR done
[x] Any breaking changes?

/azp run cpp - core


vs-code creates a second file when called by less

less invokes the editor with this syntax: $EDITOR +n target-file. When EDITOR is code, this +n is interpreted as a separate file. This new "file" is always opened as if it was changed already. Closing it calls a dialog to save this file.

VSCode Version: 1.52.1
OS Version: Fedora 32

Steps to Reproduce:

export EDITOR=code VISUAL=code
less some file
press v

Does this issue occur when all extensions are disabled?: Yes/No
This is not related to extensions

Possibly report this against less to not add this +n? What is VSCode to fix here?

"n" in "+n" is actually the line number where the file should open.
This appears to be a standard syntax among unix editors. See examples below.
vim -h displays:

               Start at end of file

+              Start at line
nano -h displays:
To place the cursor on a specific line of a file, put the line number with
a '+' before the filename.  The column number can be added after a comma.
When a filename is '-', nano reads data from standard input.
emacs --help displays:
FILE                    visit FILE using find-file
+LINE                   go to line LINE in next FILE
+LINE:COLUMN            go to line LINE, column COLUMN, in next FILE
micro -h displays:
[FILE]:LINE:COL (if the parsecursor option is enabled)
+LINE:COL
Specify a line and column to start the cursor at when opening a
buffer
Hope this helps,
Paulo.
On Fri, Dec 18, 2020 at 1:12 PM Benjamin Pasero<EMAIL_ADDRESS>wrote:

Possibly report this against less to not add this +n? What is VSCode to
fix here?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/112806#issuecomment-748076919,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEHIWRNLJPRAD3W47I4DHJTSVNINHANCNFSM4VA7LZWA
.

In order to open a file on a specific line/col with VSCode, you have to add -g (or --goto).