server::extension: Excempt system database "template1" from requires.

The system database template1 is also always present and does not need management by this module.

Hi @olifre,
According to the official documentation here the template1 is still used by default installation of Postgresql on the CREATE DATABASE command, therefore having the possibility to manage this would allow you to define the templates you need in your next database creations. As a result of this fact we sadly have to decline this PR for now.
Thank you for your contributions and we are sorry for any inconvenience caused.

Hi @carabasdaniel ,
I see! Just to clarify, this change was not about removing the possibility to manage template1, but just about removing the enforcement of having to manage the template1 database with this module to be able to define a postgresql::server::extension resource using the template1 database.
I do not completely follow why the database itself, which is guaranteed to exist (as documentation still states), must also be managed by the module to be allowed to enable an extension on it (while this is not the case for other builtin databases such as postgres).
In any case, if somebody stumbles upon this at some point, the workaround is simple — just add this to your catalogue:
        postgresql::server::database { 'template1':
                istemplate      => true,
        }

and then postgresql::server::extension will also work with the template1 database.
Cheers and thanks,
Oliver


Switch from Zenhub to Github Project

Action is working:

Project all setup, and added all issues as well. Closing out.


Create magnet tet mesh via Cubit and mbconvert

Creates tetrahedral mesh of magnet coils via Cubit, exporting in Exodus format and subsequently converting to H5M via MOAB's mbconvert executable.

Since I can't test this can someone confirm that this is working?

Or someone else can test/review/approve/merge

I tested and it is working

Perhaps @Edgar-21 can take a second look?

Its working for me


Rudimentary table support

This PR adds an imperfect workaround for adding table support by adding two spans: ClickableTableSpan and DrawTableLinkSpan.
These spans provide limited table support by reconstructing the HTML and passing it to an app-defined implementation of ClickTableSpan (which could possibly forward it to a WebView similar to what's shown in the example).
The DrawTableSpan does not need to be implemented, but we expose it so that applications can display text other than "[tap for table]", which is useful for localization.
This PR is imperfect because once we start keeping track of a table tag, we are only able to keep track of other tags which android doesn't handle by default, so the raw html passed to the ClickableTableSpan won't necessarily be correct.

If this looks good to merge, then I'll add a commit to update the README to reflect table support.

Also if it looks good, I'll add the proper Apache headers. I just realized I left them off.

Yes looks okay, I think that the usefulness is limited but I am willing to merge :)
Please add the license headers and README additions as you proposed.

@dschuermann Applied the changes we discussed; I also increased the version in the build and README to 1.4 but I didn't update the tag because I see it's at v1.4 so I'm not sure what kind of system we've got going on there.

@dschuermann What should happen if the HtmlTextView encounters a table but the developer doesn't implement a ClickableTableSpan?
ATM I provide a default implementation for DrawTableLinkSpan which means <table>...</table> gets replaced with [tap for table] (unless the dev provides their own implementation), but tapping the text does nothing.
There's two routes we could go about this:

Provide a default implementation for ClickableTableSpan which crashes if the HtmlTextView comes across a table and the user/dev clicks the replacement text.
Crash in HtmltextView.setHtmlFromString() which is pretty early so that it developers don't miss it.

I like the idea of crashing early so that devs don't get caught with their pants down, but I can also do what you do with image getters and just log an error. Thoughts?

I think currently, every tag that is not supported is simply not displayed. I would propose we do the same for tables if no ClickableTableSpan has been given and simply omit the whole table, even not showing [tap for table].

@dschuermann Updated the PR and the default behavior is to not render the tables.

thanks!

and released to jcenter...


On click microflow is not passing the object in Mx6.5.1

When I set an On click microflow on the chart the object is not passed to the microflow in Mx6.5.1. When you set a debug point on the microflow the parameter object is totally missing from the variables and hence all the other objects I try to retrieve based on that parameter object will be empty.
I will try to reproduce in the test project of this widget.
Regards,
Ronald

_Test__ChartJS.zip

Test project in Mx5. It is not working there also :) Click on the home page on the bar chart. It will invoke the IVK_Microflow.


Html tag a(href=$"") not working

it says: The object constructor 'a' takes 0 arguments but is here given 1.

It works fine if href is a static string.

Hi, @pkese ! Glad you asked, please vote for the related issue https://github.com/dotnet/fsharp/issues/16696

I don't mind keeping it open, since there are not so many open tickets anyway :)


[Nox.Generator] UpdateCommand save to Localized table

relates to #907
depends on #936

Task

In the API Put, get the request language from the header, fall back to default language in Nox.Solution if not found, see https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Accept-Language, see also #937
Change the Update<Entity>Command request to accept a CultureCode
Update the command to:

Add new translations to Localized Table
Set the translation in the Entity Fields only if is the Default Culture Code Defined in Nox

https://regusit.atlassian.net/browse/MP-9660


Containerization design doc

This is the review request for the overall design doc for containerization

Pull Request Test Coverage Report for Build 5603

130 of 258 (50.39%)  changed or added relevant lines in 10 files are covered.
8 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.07%) to 42.44%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

azkaban-web-server/src/main/java/azkaban/webapp/servlet/LoginAbstractAzkabanServlet.java
0
3
0.0%

az-core/src/main/java/azkaban/metrics/MetricsManager.java
0
4
0.0%

azkaban-web-server/src/main/java/azkaban/webapp/AzkabanWebServer.java
0
4
0.0%

azkaban-common/src/main/java/azkaban/executor/ExecutorManager.java
8
13
61.54%

azkaban-common/src/main/java/azkaban/executor/ExecutionController.java
18
24
75.0%

azkaban-exec-server/src/main/java/azkaban/execapp/FlowRunner.java
16
23
69.57%

azkaban-common/src/main/java/azkaban/executor/AbstractExecutorManagerAdapter.java
68
167
40.72%

Files with Coverage Reduction
New Missed Lines
%

azkaban-common/src/main/java/azkaban/executor/ExecutionController.java
1
57.14%

azkaban-web-server/src/main/java/azkaban/webapp/AzkabanWebServer.java
1
18.49%

azkaban-common/src/main/java/azkaban/executor/ExecutorManager.java
2
64.59%

azkaban-exec-server/src/main/java/azkaban/execapp/JobRunner.java
4
77.03%

Totals

Change from base Build 5527:
0.07%

Covered Lines:
14155

Relevant Lines:
33353

💛  - Coveralls


tuple output of j and k matrix from same function. also moved the form_JK function outside basic_mod

Looks good


Need to replace the header instead of adding header in outbound security support

On configuring outbound-all in security settings and then trying to override inbound basic authentication by providing username/password in outbound request, the basic authentication header values are added to inbound values instead of replacing them. Which is causing the outbound request to fail.
Environment Details

Helidon Version: 2.3.3
Helidon MP
JDK version: jdk-11.0.13
OS: Windows 10

Problem Description
Using below configurations for security-
`security:
providers:

http-basic-auth:
realm: helidon
users:

login: system
password: system
roles:

Global

login: devops
password: devops
roles:

DevOps

login: sqlops
password: sqlops
roles:

sqlops
outbound:
name: "propagate-all"
`

Utilizing rest client to make outbound call by overriding basic authentication via HttpAuthenticationFeature , the service call fails with error logged - Provider io.helidon.security.providers.httpauth.HttpBasicAuthProvider. Message: Basic authentication header with invalid content - not base64 encoded
Cause - The encoded value of basic authentication is appended to the inbound value and the final value becomes like - ZGV2b3BzOmRldm9wcw==,basic c3lzdGVtOnN5c3RlbQ==
But if the HttpAuthenticationFeature is not used and the outbound authentication is overridden using
builder.property(HttpBasicAuthProvider.EP_PROPERTY_OUTBOUND_USER, "devops"); builder.property(HttpBasicAuthProvider.EP_PROPERTY_OUTBOUND_PASSWORD, "devops");
then the authentication of target service is successful.
Steps to reproduce
The authentication yaml is mentioned about.
Logging properties-
handlers=io.helidon.common.HelidonConsoleHandler

java.util.logging.SimpleFormatter.format=%1$tY.%1$tm.%1$td %1$tH:%1$tM:%1$tS %4$s %3$s !thread!: %5$s%6$s%n

io.helidon.security.level=DEBUG

io.netty.level=DEBUG

.level=INFO

AUDIT.level=FINEST

Rest Client code to call target service-
RestClientBuilder builder = RestClientBuilder.newBuilder();

if(!propagate) {

	HttpAuthenticationFeature authenticationFeature = HttpAuthenticationFeature.basic("devops", "devops");

	builder.register(authenticationFeature);

	//		builder.property(HttpBasicAuthProvider.EP_PROPERTY_OUTBOUND_USER, "devops");

	//		builder.property(HttpBasicAuthProvider.EP_PROPERTY_OUTBOUND_PASSWORD, "devops");

}

URI uri = URI.create("http://localhost:8080/auth-svc/rest");

builder.baseUri(uri);

TargetRestClient restClient = builder.build(TargetRestClient.class);

try {

	try(restClient) {

		return restClient.username();

	}

} catch (Exception e) {

	e.printStackTrace();

}

return null;

This should only occur in case there is another Authorization header added as part of the outbound request.
We may add a feature to set the header instead of adding it (as is the current behavior).


Create additional steps in wiki.sh

Script should add HTML to:

Add favicon
Back redirect script to codelabs

during claat rebuild.  This can be done with grep + piping.
Should also eventually add redirect.md matching existing doc (or leave blank if new doc).

Unnecessary for the moment


Investigate committing overrides back to OpenLayers

We have several overrides to ol.render.canvas.PolygonReplay in src/os/ol/mixin/render.js.
Investigate the possibility of committing those changes back to OpenLayers and removing the mixin.

My PR was merged under OpenLayers 5.x, so removing the mixin is now blocked by #245.

Upgrading OpenLayers is going to be a significant change with their move to ES6/TypeScript, so I don't think we need to keep this open. We'll have to hit all mixins in that process anyway. Closing.


feat: Add rescuable to static a token

Adds Rescuable to the static a token contract.
Creates a inheriting contract specific for the new revision, with the specific method to re initialise

Moved changes here: https://github.com/aave-dao/aave-v3-origin/pull/29


SSL SYSCALL error

Hi there so i am trying  django-simple-history + django 1.7 and i have a table with about 300.000 records. When i run populate_history i get an error:
(xxx)vagrant@vagrant:/vagrant$ python manage.py populate_history --auto --settings=xxx.settings.local
INFO 2015-07-31 14:54:39,572 base 1713<PHONE_NUMBER>66016 Raven is not configured (logging is disabled). Please see the documentation for more information.
Existing history found, skipping model <class 'support_tables.models.ParishCouncil'>
Existing history found, skipping model <class 'support_tables.models.Occupation'>
Saving historical records for <class 'support_tables.models.PostalCode'>
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 385, in execute_from_command_line
    utility.execute()
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 377, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/core/management/base.py", line 288, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/core/management/base.py", line 338, in execute
    output = self.handle(*args, **options)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/simple_history/management/commands/populate_history.py", line 61, in handle
    self._process(to_process)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/simple_history/management/commands/populate_history.py", line 106, in _process
    utils.bulk_history_create(model, history_model)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/simple_history/management/commands/_populate_utils.py", line 27, in bulk_history_create
    history_model.objects.bulk_create(historical_instances)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/db/models/manager.py", line 92, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/db/models/query.py", line 409, in bulk_create
    self._batched_insert(objs_without_pk, fields, batch_size)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/db/models/query.py", line 938, in _batched_insert
    using=self.db)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/db/models/manager.py", line 92, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/db/models/query.py", line 921, in _insert
    return query.get_compiler(using=using).execute_sql(return_id)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/db/models/sql/compiler.py", line 921, in execute_sql
    cursor.execute(sql, params)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 81, in execute
    return super(CursorDebugWrapper, self).execute(sql, params)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 65, in execute
    return self.cursor.execute(sql, params)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/db/utils.py", line 94, in __exit__
    six.reraise(dj_exc_type, dj_exc_value, traceback)
  File "/home/vagrant/.virtualenvs/xxx/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 65, in execute
    return self.cursor.execute(sql, params)
django.db.utils.DatabaseError: SSL SYSCALL error: EOF detected

Any ideas? Is this because of the number of entries i have? i am using a machine with single core and 2GB

@psychok7, what DB service are you using?

@macro1 i am using PostgreSQL 9.3

A little googling says it's a generic connection error which could be raised by a timeout, so the amount of data may be the issue. In the short term you can look at the code for the management command and modify it to paginate and add it to one of your own apps.
If you have time, we could probably add a batch size to the command that would allow it to build the initial history layer in chunks. Currently the command would do all 300,000 entries in one query, which is probably not a great idea.


cover user show action with tests

Fixes #52
Proposed Changes

Add test coverage for the show action in the user controller

Pull Request Test Coverage Report for Build 49

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.2%) to 73.118%

Totals

Change from base Build 45:
2.2%

Covered Lines:
68

Relevant Lines:
93

💛  - Coveralls


Using bullet-core-1.0.0

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Can't merge btw


Make functions in Data.List stack-safe

I've noticed, that some of the functions which certainly can be tail recursive aren't, so I've fixed the most obvious. But before going further I thought it would be wise to ask if there is some reason why they weren't tail recursive before?

The foldl case is a compiler bug - purescript/purescript#1413. As for the others, yeah I think it's a mistake, they should be implemented as tail recursive.

:+1: Looks great.

I am trying to finish it up and can't think of a nice way to implement foldr, is there one?

No I don't think so, not without trampolining.

What about implementing it in terms of foldl?
foldr f a bs = foldl (\g b x -> g (f b x)) id bs a

@garyb Hmm. I don't remember if I tried that, but looks promising. I'll try soon.

I think you'll just build up a tower of functions and blow the stack that way instead.

Yeah, you could be right, I didn't try it aside from type checking.

:+1: Thanks!


amulet-map-editor: Add version <IP_ADDRESS>

From lukesampson/scoop-extras#5282
Closes lukesampson/scoop-extras#5281

Should the manifest be named amulet or amulet-editor instead of amulet-map-editor?


"vlock -ans" as a user fails because of permission

As root it works fine, but as a user (in the vlock group) I get:
vlock -as
vlock-nosysrq: could not open '/proc/sys/kernel/sysrq': Permission denied

and
vlock -n
vlock-new: cannot open virtual console: Permission denied

Debian once had the same problem, but I've not been able to distil their solution:
<EMAIL_ADDRESS>@aszlig uses "vlock -ans" on NixOS, so it is possible:
https://github.com/openlab-aux/vuizvui/blob/master/modules/user/aszlig/services/vlock/default.nix#L50

The reason this works on my machine is because it is started as root user. And it's probably not a good idea to allow access to /proc/sys/kernel/sysrq to normal users directly.
It works on Debian however, because the vlock-main binary is setuid-root:
-rwsr-xr-x root/root     31384 2014-07-15 23:11 ./usr/sbin/vlock-main

@aszlig OK, thanks.
Did I understand correctly that you need to type the root password when you want to lock?
I'm sorry I can't follow your code...

I'm passing the USER environment variable to vlock, so it's prompting for the user passwd:
https://github.com/openlab-aux/vuizvui/blob/master/modules/user/aszlig/services/vlock/default.nix#L44-L45

@aszlig Alright, I'll stick with i3lock for now.  Thanks!


[Bug] app.asar" does not exist

Describe the bug
Application entry file "dist-electron\main\index.js" in the "I:\wm\web\dev\electron\pwlive-client\dist\win-unpacked\resources\app.asar" does not exist. Seems like a wrong configuration.  failedTask=build stackTrace=Error: Application entry file "dist-electron\main\index.js" in the "I:\wm\web\dev\electron\pwlive-client\dist\win-unpacked\resources\app.asar" does not exist. Seems like a wrong configuration.
无法打包，前几天还可以用，这几天重新打包报这个错误。几台机器看了都有这个问题？麻烦看下原因，谢谢

你当我是神？


Provide revision 2 art concepts

We want to complete our round of initial concept art by doing a pass using our art guidelines:

Strong, bold shapes
High contrast
Monochrome (black, white, & red)

Should we consider this task done? Can someone put the art here as a reference?

Art is done.


tests.buildRustCrate: Fix after hashing method change

Motivation for this change
As @lopsided98 points out in #105305, we're probably better off not asserting exact hashes, but that is a more involved fix.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

This only fixes the tests for x86_64-linux, every other platform is still broken.

Oh good point, didn't think of that.

@andir that sounds good.

We can probably just exchange this for a loop in which we run state against the given expected file name (which can contain wildcards)?

I liked the idea of running diff because it's easiest to diagnose in the failure case. So i didn't really know what to do. But then I realized it was easy: just normalize before diffing.
This should now be ready to go.

Mhm, hydra reports the job as begin removed now: https://hydra.nixos.org/eval/1636253#tabs-removed

@andir see first commit it's got a new attr path.


Create normalizer for Borrow type

Create a normalizer function that given a Borrow type, it turn it into a base level object.
Required Testings:

Failure Tests:

[ ] A

Success Tests:

[ ] B

already solved


CentralPackageVersions: Microsoft.AspNetCore.All should be excluded

Microsoft.AspNetCore.All is a package which isn't implicit but also should not have a version specified.
In the .NET Core SDK if you try to specify a version for this package you get:
warning NETSDK1071: A PackageReference to 'Microsoft.AspNetCore.All' specified a Version of `[2.2.2]`. Specifying the version of this package is not recommended. For more information, see https://aka.ms/sdkimplicitrefs

But then if you remove the version, you get this from CentralPackageVersions:
error : The package reference 'Microsoft.AspNetCore.All' must have a version defined in '<redacted>\Packages.props'

Almost certainly needs the same fix as: https://github.com/microsoft/MSBuildSdks/commit/0347e81a9aaac5321bbbf2b7e29627eb1a7de833

This is fixed in Microsoft.Build.CentralPackageVersions 2.0.46


Automatically ingest intermediate files into RGD

This PR adds code to automatically ingest intermediate files (i.e. everything except the 3D tiles and the .geon files I mentioned in https://github.com/girder/Danesfield/issues/4#issuecomment-985012320) into the corresponding RGD models.
Screenshot of mesh being viewed through RGD

Are these files viewable through the RGD SSR pages after ingestion?

Are these files viewable through the RGD SSR pages after ingestion?

The obj and ply files are through the rgd_3d ssr pages, but the image files need to be added to an ImageSet to be viewable. I'll update this so an ImageSet is created too.

the image files need to be added to an ImageSet to be viewable. I'll update this so an ImageSet is created too.

This is all set.
Viewing some tiff files from a DatasetRun:


Allow user to change search engine

Operation System: MacOS
Beaker version: 0.8.0-prerelease.6
I could not find a place to change search engine, and I find that duckduckGO is hard coded in the program. I would be happy to give a PR if you approve.

Yeah this has been on my todo list. I'd be happy to hit it myself, or to accept a PR.


Add the memory timeline profiling support through the PyTorch profiler.

This PR adds the memory timeline profiling support through the PyTorch profiler.
To enable it, the following flags in the profiler config must be all to set to true.
  torch_prof_with_stack: true
  torch_prof_record_shapes: true
  torch_prof_profile_memory: true

By default, the general timeline profiling is also enabled and produces a json trace. To produce memory timeline only,
explicitly set torch_prof_filename: to empty.

The default plotting of the memory timeline from PyTorch is not very informative, to use our custom plotting function (with adjusted axis tickers and grids), set torch_prof_memory_custom_plot: true

Thanks for this Cheng! Can we add a mcli run with a memory timeline trace in the PR description for an mpt-7b fsdp model?
Also can we add two tests, one for ensuring the profiler uploads memory html init /file upload correctly and another to validate the custom plot function you wrote
https://github.com/mosaicml/composer/blob/dev/tests/profiler/test_profiler.py#L46-L47.


Add support for generic structs/enums/etc.

opg does not support generic structs.
#[derive(Debug, serde::Serialize, opg::OpgModel)]
struct TestStruct<T: std::fmt::Debug> {
    inner: T,
}

#[derive(Debug, serde::Serialize, opg::OpgModel)]
struct TestSubStruct1 {
    a: i32,
    b: i64,
}

#[derive(Debug, serde::Serialize, opg::OpgModel)]
struct TestSubStruct2<'a> {
    c: &'a i32,
    d: String,
}

describe_api!(
    info: {
        title: "title",
        description: "description",
        version: "0.1.0",
    },
    paths: {
        ("some" / "path" / "one"): {
            GET: {
                description: "Get smth 1",
                200: TestStruct<TestSubStruct1>,
            },
        },
        ("some" / "path" / "two"): {
            GET: {
                description: "Get smth 1",
                200: TestStruct<TestSubStruct2<'static>>,
            },
        },
    }
)

Converts to this:
---
openapi: 3.0.3
info:
  title: title
  description: description
  version: 0.1.0
paths:
  /some/path/one:
    get:
      description: Get smth 1
      responses:
        200:
          description: OK
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/TestStruct"
  /some/path/two:
    get:
      description: Get smth 1
      responses:
        200:
          description: OK
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/TestStruct"
components:
  schemas:
    TestStruct:
      type: object
      properties:
        inner:
          $ref: "#/components/schemas/TestSubStruct1"
      required:
        - inner
    TestSubStruct1:
      type: object
      properties:
        a:
          type: integer
          format: int32
        b:
          type: integer
          format: int64
      required:
        - a
        - b

There is the only first TestStruct<TestSubStruct1> type.

Is there any news about it? It looks pretty useful


Implement ISigningKeyService

Hey
I am facing the same problem you are - trying to decide between CloudHSM and Azure KeyVault as the store for the private key. I saw the work you did, and it look like a great start, but the main thing that missing here, and that I am not sure how to implement is the ISigningKeyService. The main problem is the dependency in X509Certificate2. Look like you can generate a X509Certificate2 from 'JsonWebKey' (which is what you can export from the key vault, as far as I know) but this look like very ugly code to me.
I was wondering if you have any thought regarding this problem.
Thanks,
Omer

I've also opened an issue in identity server repo: IdentityServer/IdentityServer3#2727

I'm not sure how to resolve this one myself. I think you're right: on startup, we fetch the public key from Key Vault, use it to build an X509Certificate2 instance and pass it back to IdentityServer3/4 that way, but seeing as a public key != a certificate, it would be rather messy. I'll poke around the X509Certificate2 implementation in .NET and see what I can do. Watch this space :)

I am working on it myself right now, it is not easy and the code is pretty much a mess. It is also very ugly because of the dependency between the ITokenSigningService and ISigningKeyService - especially the need to use the same kid. If I will have something that I feel good with, I will share it...

Sounds awesome. I do apologize if my code seems in a bit of a state, it's still very much a work in progress at this stage. Let me know if it gets in the way and I'll do what I can to help out.

This is really fine :) I thought that's the situation...
I am also facing the same problem right now, so I might contribute some PR. My main problem is that I am looking for a generic solution that will allow me to switch between keyvault and PKCS11 easily.

Here's hoping your solution in https://github.com/IdentityServer/IdentityServer3/pull/2732 goes through, it looks like this would solve a lot of our problems. Frankly, I've spent the past three hours banging my head against the IdentityServer, .NET and Key Vault APIs and I cannot see a clean way to do this beyond just changing IdentityServer to be more friendly with JWKs instead of requiring X509s.

I was thinking the same, there is a way but the code become so ugly that I preferred to propose a PR to identity server and hope it will be accepted...

Hey,
I've published a small adapter as a nuget package. This might help you with this...

Oh, that looks fantastic! I'll get to work on using this soon.

Thank you! Hope it will be usefull :)


Support "fine grained" scaling.

Currently myriad supports scaling at "Node Manager" level. i.e. it launches a Node Manager instance for every "resource offer"  it receives.
We would like to support scaling at "YARN's container" level. i.e. launch a YARN container per "mesos resource offer".

PR #91 submitted for review.


Check based environment in CI and test .yml files in test_misc.py

Mirrors what was done in GeoUtils to:

Check that the package can be imported with the base environment provided in environment.yml in CI,
Update the environment using dev-environment.yml in CI,
Add a test in test_misc.py to ensure that all packages listed in environment.yml are also listed in dev-environment.yml (which cannot be checked by CI), so that the developer environment file works.

See more details in:
https://github.com/GlacioHack/geoutils/pull/297
https://github.com/GlacioHack/geoutils/pull/301

Ready to merge!


Color Picker shows in comments for github issue numbers

Are we able to tell if the HEX code is located inside a comment? If so, could we not show the color picker in that case?

Also shows for private class variables:


Why not generics?

Why Songbird::get have generics (<G: Into<GuildId>>), but Songbird::get_or_insert doesn't have generics?
And, why Call methods doesn't have generics?

This is an oversight, sorry! If you want to remedy this, I'd be happy to take a PR on songbird:next.


Duplicate note commitment causing failure when syncing view service

Describe the bug
When running pcli view balance, I get an error:
~/c/p/penumbra ((031-autonoe)|✔) $ cargo run --release --bin pcli -- view balance
Scanning blocks from last sync height 13544 to latest height 27114
[0s] ██████████████████████████████████████████████████   13570/13570   14873/s ETA: 0s
Error: status: Internal, message: "Worker failed: error returned from database: UNIQUE constraint failed: notes.note_commitment", details: [], metadata: MetadataMap { headers: {"content-type": "application/grpc"} }

There are two separate problems here:

Something is creating duplicate notes, which indicates a bug -- the blinding factor in a note is intended to ensure that note commitments are globally unique, and that's not happening here.
The view service should not fail on duplicate note commitments.  Although note commitments are verified, in the sense that they must be created from the actual note contents, they're still attacker-controlled (an attacker could deliberately craft two identical notes with identical blinding factors, to obtain duplicate note commitments), so the view service cannot assume that notes are constructed with unique blinding factors.

This is actually a security bug, because it violates infallibility of scanning -- an attacker could use this to take a targeted address's view service offline, and thus learn about that user's network location (by watching which client goes offline).
To Reproduce
My testnet FVK is
penumbrafullviewingkey10meak7pngg580r57fpvclrrzvaf8da56nayejxssv50w85anngfy2sct8myyjy8fgxxs7whrrpew5ncyzma76ern06ueevump8ae7zqd2wd7w

and this can probably be used to reproduce the problem.

To fix this, it seems like we'll either have to decide to:

Relax the UNIQUE constraint on the notes table, sacrificing uniqueness of note-by-commitment lookups. This relaxation would have to propagate through the whole API, because now a note commitment corresponds to a set of NoteRecords.
Reject duplicate notes during scanning, so that duplicated notes are not accepted by the user (and thus become unspendable without a special software implementation). This acts as a forcing function to ensure that client software doesn't create duplicate notes (because the recipients won't accept them).

My sense is that (2) is probably a better option, since duplicate note commitments "shouldn't" happen, and that way we avoid contorting our API to accomodate malformed data.

Currently, pviewd reports this error at INFO level instead of ERROR level, but doesn't crash on this instance, unlike pcli which is stopped by this.

The problem block is 15098, where pcli crashes, and running cargo run --release --bin pcli q shielded-pool compact-block 15098 provides a list of potentially duplicate note_commitments which @plaidfinch helped me pull out, though we haven't yet been able to identify which in particular is already present in storage.
"89538d12e30cbccc5a647a79307376b417a471d981b0b42e4866397e1ac8a608" "858b29518234c445f021b89374bdfdbeeaaf8ab71222a4a4c9c716683c53330c" "3b9240300f59dbbfe44cf4308c491619cf7589cc34224a5ccaa7834d691c540e" "3c42bd6134534fd84b1846558835b8cee37708c68f8b04c50791741ac754080a" "5a2f9072f551809e87b205680c43c274cb7f693f6befe762bb7dea7e5197d10a" "99adcd65858aadfb6f59dcd2eb6eb8568cfcac27c44038891499658e88a39d06" "7907141044c754c8935bf577e1d39e71105f251f1b86b66fecba848b2b487209" "d9db537f53b99b2ae9f77ceb6c69cd13da8e37bd0ced5a390692344d0f07950d" "b2ac0a8e7003f28b7614509c3ad0a37f8de6ceb6ed1f57b60c7dc29f4e73300c" "5ab1ff6d69c85aa83d04f04985bcc2e96153214fcc0556c05d9ebc4747e96501" "4b2939ae54c54ad3df4dfade25fe35ef325a8ce98e14c3b27c4826048354770f" "9ddbbc4b81eb629d62ae5d0b1a9f703dad29536886bd62efe3afeece3e0e8612" "9e9aa6e92fb501123cb75a81c417ccd1060baa431ab1b4d27e2f9b40ed41190f" "ee8dddde3ff3b21893aafea64a143721dcd08961a49f46bd905fded3566e6808" "8b1ea865bbba0b55a607a151a8a3742019b8cd90190fc2e2868db91e37434508" "926a3daa4429e0c1be7e0475f42524bfa82a56c78bb98d88df2dedd7030ff409" "a853fc6e8acde7105f636da047a3a8daf153020594d1d3bdcd1edfcd9db5870e" "cd827f8725291f6093d958236d414520a28dc6c52a440ed2eac4e0b21a160812" "7f47256b0567b90db120edbf191047100b16b29696bf3beada3ecc7c65dfa006" "b67c211a2d7b993a1fe44cd26740ad4c8f1845e25d5514eb5230ac186ffba005" "92392845ba6a960a3df195fd6b3dd062deb6def04b8dd23dd25cc0ae04267910" "65c860e3e80ddaf8a93256ef67a0ed88618bf69526b203271d8508f2e82c340d" "b84e0106a83e894c6b42c3cb66c42071f9ac9bdd7128ca3f6be1944e72cde407" "b484722bf6410ad916cb6a89aeaf1b13413a3296ce3de20ee2119d13be790e0e" "66f50bcc8ae3631e4793a89e901e3b21a2b9634fe6160a9031a106d75dbf7d11" "770f78db0dda4dbd29df3212c214131c07dfab40192053288fd40c7a664e8a08" "a26b34c1afc757890be028be8c02da0c4c171fc2dfd95250cbda60a62727d104" "929d27c5a13fd88a22279b082b084892084fcecd6d5d0ef09f4f3bc3cf535a04" "540cebfb4286c0139ccfb2b5d956785d877776889d7efbec112631e671c1a00f" "ca1fedc09846ee1b45f1604433cf275e4780a1ed2d28468fea8752053b1f2f05" "f3021c0542c0ed5fd0e2155342a1e73f9ed6f848ae83fe630230a65cd39f2a04"

Based on similar duplicates being found by @redshiftzero it appears this issue arises from the refactor to using a notes table alongside the spendable_notes and quarantined_notes table - so when a note is removed from quarantined_notes, the associated entry in the notes table is not removed, which is causing an error when it is attempted to add the note_commitment to the notes table.
@redshiftzero from discord:
2. At height 13170, it is also added to the quarantined_notes table. (I think at some point it is un-quarantined, and removed from the quarantined_notes table since there are no entries in that table for height 13170 after syncing, but haven't verified this or tracked this down further).
3. Later, we add the same note commitment to the notes table at height 15098. But it was already added in height 13170, so we hit the unique constraint.
The denom for this note is upenumbra.
a while ago I also hit a duplicate note commitment when I was doing the recent changes to the integration tests, and my best guess at that time was it was something related to quarantine```


TEST_TWO_PAGES_CYCLE: Update freezeyt module for pass

PR contains two commits. Resolve issue: https://github.com/encukou/freezeyt/issues/15

first commit is written to pass test_two_pages_cycle
second has been trying improve our code for better efficient with type set

Merged in 22fa3ffdc83462431b4ab0e276e9f25fcd537a3d


Nudge towards bandcamp where possible?

Was thinking of seeing if it's possible to match artist/album against bandcamp and nudging people towards purchasing on that platform where available.
Especially with https://isitbandcampfriday.com/ where artists get 100% of the revenue from a sale.
Thoughts?

I'd love to point people towards places to buy music where possible - and especially places like Bandcamp which are better at supporting artists. So yes, definitely keen for this!
Challenges to work through:

Do we crowdsource Bandcamp matches, or use an API/crawler to get pretty accurate matches?
Where else can we direct people to buy from if an artist/band isn't on Bandcamp?
How best to normalise the artists table - as currently Last.fm and Spotify values are separated (I avoided wrangling this because some bands have the same name, though it is very rare) 🤔

I was getting bad flashbacks to the goodfilms days. Music and movies are such difficult data sets to work with, especially if you want to link to online store fronts. At least books have ISBNs.
That said, looks like musicbrainz could be a go-er. It has an API, and it links to purchase locations: here is an example release.

Okay, dashboard and emails will now display links from MusicBrainz if that has any information. It's not super-pretty, but I'll do for now.
There seems there's a lot of holes in the MB data though, so I'll look into Odesli soon for additional links. Granted, I will definitely need an API key for their service, because otherwise they rate limit at 10 requests per minute 😅

Okay, I've added in Odesli as well (and they kindly gave me an API key for a better rate limit), but there's still a tonne of missing links. Also, Odesli doesn't support Bandcamp (who I've emailed to see if they have any undocumented APIs for finding artists/albums).
So: plenty of room for improvement, but this is a solid enough start that I'm happy to close this issue for now :)


add complimentary colour example

add complimentary colour nn to examples

Wow, this is so cool! I wonder if it might make sense for you to maintain this example in your GitHub and link to it from here?


The "go" and "go mod" icons are very small compared to the others

Tried making them as big as I can, their logo has some extra elements on the side that limit its size unlike other logos:


Remove glauth and accounts services

Todo:

[ ] Remove remaining mentions of accounts from docs (or create issues)
[ ] Remove remaining mentions of glauth from docs (or create issues)

if we drop glauth we would need a way to set up an ocis instance with the built in IDP thet reads the users from an existing owncloud instance. We could add a cs3 backend to LibreIDM, or even an owncloudsql or maybe even generic sql backend.

if we drop glauth we would need a way to set up an ocis instance with the built in IDP thet reads the users from an existing owncloud instance. We could add a cs3 backend to LibreIDM, or even an owncloudsql or maybe even generic sql backend.

Actually our IDP (lico) actually even has a  libregraph identity backend (https://github.com/libregraph/lico/blob/master/identifier/backends/libregraph/libregraph.go), might be possible to enhance that to work with our graph service.


about opencv error

ubuntu@ubuntu-HP-15-Notebook-PC:~/Desktop/EmotionDetection_OpenCV-master/code$ g++ facialEmotion.cpp -o fe -I/usr/local/include/opencv -I/usr/local/include  -L/usr/local/lib -lopencv_core -lopencv_imageproc -lopencv_highgui -lopencv_ml -lopencv_video -lopencv_features2d -lopencv_calib3d -lopencv_objdetect -lopencv_contrib -lopencv_legacy -lopencv_flann
/usr/bin/ld: cannot find -lopencv_imageproc
collect2: error: ld returned 1 exit status
Can u please see to it?

Sorry haven't been quite active with this project. Feel free to use the code. If I do any tinkering with this will get back. Thanks for tinkering with this though.


Use zhmcclient 1.17.0 and stomp-py 8.1.1

Increase minimum version of zhmcclient to 1.17.0 to pick up fixes (also related to notifications), and the additional mock support to fix issue #413
Increase minimum version of stomp-py to the version used by zhmcclient 1.17.0 (i.e. stomp-py 8.1.1) to pick up fixes.

Cannot be rolled back to 1.10.x because of the requirement for Python>=3.8.


Create ConfigKeyAppResourceLoader

we want to create a test and show how to add the appResource to ViewLocator then we can add it to searchLoaders like here https://github.com/yakworks/view-tools/blob/master/test-projects/app/grails-app/conf/spring/resources.groovy
GitHub
yakworks/view-tools
view-tools - Grails plugin to help locate views in the spring mvc context
snimavat [8:02 PM]
so any directory configured as a resource directory can be added to search location, thts what u mean, right?
joshua
[8:03 PM]
the use case is for views.
snimavat [8:04 PM]
yes
joshua
[8:04 PM]
make sense?
snimavat [8:04 PM]
Yes. so view directory can be configured as resource directory
[8:05]
there is one thing though
[8:07]
do we use convention for that, and always add a particular resource key at the end of search paths, or it has to be added manually by app when viewLocator is configured ? (edited)
[8:07]
I guess app will do it manually through viewLocation bean
joshua [8:07 PM]
right
snimavat [8:08 PM]
however, in tht case, shall we need some prefix like: resources:xxxx
joshua [8:08 PM]
will need to be thought out.
snimavat [8:09 PM]
it has to be some thing like
searchPaths = [
//external file path that may be used for production overrides
“file:./view-templates”,
“resources:xxxx” //xxxx is key under resources
]
joshua [8:10 PM]
won’t be in searchPaths
[8:11]
would be added to searchLoaders
snimavat [8:12 PM]
ok, so we need another custom resoruce loader
joshua
[8:13 PM]
probably
snimavat [8:13 PM]
the resource loader will have a property tht says which resource config key is used for base directory udner which to look for views
joshua
[8:17 PM]
that would make it generic?
snimavat [8:17 PM]
it should
[8:17]
so app can configure whtever resource key they want to be used as root for views
[8:18]
if at all
joshua
[8:19 PM]
so we have a ConfigKeyResourceLoader right?
[8:19]
thats what you are thinking?
snimavat [8:19 PM]
AppResourceKeyResourceLoader !
joshua [8:20 PM]
ConfigKeyAppResourceLoader

Done


[PackageLoading] Print warnings when loading manifests

We were supressing the warnings while loading the manifests because we
considered any type of output from the compiler as a sign of error. This
changes the error state to depend on the exit code. The warnings will be
printed when loading a resolved package graph and not during dependency
resolution. This makes sense because the manifest we look at during
dependency resolution might not be used in the final resolved graph.
rdar://problem/41180566 SwiftPM should not suppress compiler warnings in the manifest file

@swift-ci smoke test


Faster Typechecking - It's Finished Edition

Closes #587
I wanted this to get in before I nuke DreamPath. Credit to @ZeWaka for doing basically all the work.

This is horribly broken and I'm not really sure why.

Going to bully zewaka to deal with it


Fetch the llvm-spirv component automatically if no llvm-spirv specified.

Add the public llvm-spirv component to the Triton XPU backend.
Support to build the XPU backend friendly.

https://github.com/intel/intel-xpu-backend-for-triton/blob/llvm-target/scripts/compile-triton.sh can be modified to no longer clone the SPIRV-LLVM translator after this change. Created https://github.com/intel/intel-xpu-backend-for-triton/pull/189.


Feature cache optimizations

Consider this still a Work In Progress, and just here to enable discussion.
This adds features requested in #2786 and #2825

Any chance you might be able to separate this into two separate pull requests? 1 for adding support for existing caching to the core_area_layout, core_scrapbook_display and core_stack_display blocks, and 1 for new functionality for the query on GET parameters? They both have merit, but we're much more likely to get the former in quickly – the latter has value but I think it needs more work to be easily understandable and tested.

Yup - I can split it up. I'm taking a small pause on this as I'm working on a new site, however, the new site is actually using this system so that should help me flush out any quirks in the system.

sound good! thanks
On Thu, Aug 6, 2015 at 10:02 AM, TimDix<EMAIL_ADDRESS>wrote:

Yup - I can split it up. I'm taking a small pause on this as I'm working
on a new site, however, the new site is actually using this system so that
should help me flush out any quirks in the system.
—
Reply to this email directly or view it on GitHub
https://github.com/concrete5/concrete5/pull/2851#issuecomment-128444396.


FeatureFetchException: CONFIGURATION_ERROR savedGroupsJsonElement fetch error code

After upgrading to the newest version some of applications throw the next errors:
FeatureFetchException: CONFIGURATION_ERROR savedGroupsJsonElement fetch error code

FeatureFetchException: CONFIGURATION_ERROR encryptedSavedGroupsJsonElement fetch error code: encryptionKey provided but endpoint not encrypted

Probably this commit to be claimed - https://github.com/growthbook/growthbook-sdk-java/commit/a40b08813470b4053a61df39995e58c7d256e245#diff-27b41c802a65672520c674fa86e847a00271031eb193634b13cc7b18aeeae1a3R415-R421

Hello @vasilii-bolgar-tr!
Is the SDK literally throwing an exception or printing errors to logs?

Hi @Bohdan-Kim,
No it's just printing errors (like once per second). I tested and the functionality of GrowthBook is not impaired.

The pull request was created according to this issue


add weighted softmax loss

Hello, My dataset is imbalanced, which means the number of positive class is much less than that of negative class. I want to make the positive loss has weight. I know that I can repeat sample the positive loss, but I want to add weight. How can make it?
Thanks so much.

@mxnet-label-bot add [question]

You can look at several implementations. For example: https://stackoverflow.com/a/48874517 https://github.com/apache/incubator-mxnet/blob/v1.0.0/example/sparse/weighted_softmax_ce.py

@vuvko : Thanks for the reply.
@haibo-qiu : Would the above examples help you?

@mxnet-label-bot add [pending-requester-info]

@haibo-qiu Pinging to know if the examples were able to answer your question.  If they did , requesting you to close the issue


Fix File.expand_path when HOME ends with /

Oops, forgot to test this properly. At least we have specs for this and the root case now.

Looks good, thank you!


Test rewrite selective check python

TEST of PR

Hey @edithturn  - > do you want to continue on that one? I think I might want to take a stab on it (also to simplify the logic maybe ) )

@potiuk of course! I would like it! 👏🏼 😃
I hope to finish "reuse of build image in python" today (I will send an update in some hours).
Things are more clear for me now and I think I can re-take "selective check", please let me know how it would be "simplified"

Things are more clear for me now and I think I can re-take "selective check", please let me know how it would be "simplified"

Not sure yet :).


fix(dashboard): Return columns and verbose_map for groupby values of Pivot Table v2 [ID-7]

SUMMARY
Columns used by Pivot Table v2 were not processed correctly by /dashboard/{id}/datasets endpoint, resulting in chart not displaying custom column labels when added to dashboard. This PR fixes the issue.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:

After:

TESTING INSTRUCTIONS

Create a pivot table v2 chart using columns with custom labels
Add the chart to dashboard
Verify that custom labels are used instead of column_names

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

CC @junlincc

/testenv up

🏷 2021.46


Do you have any plans to release Checkpoints?

Thank you for your excellent work. Do you have any plans to release Checkpoints?

Same question here! Thanks for sharing the code!

Also looking forward to the checkpoint.

Hi, we plan to release the checkpoint but it might take some time for that. However, it is not that hard to train the ia3 checkpoints you can use the latest version of the huggingface peft library which has ia3 built in. The training should be pretty fast depending on the dataset and the model size. However, for the vision experiments we used the checkpoints provided by the task-arithmetic paper. These can be used for the vision experiments.
I will post on this issue once I get back to school and I am able to release the checkpoints we used for NLP experiments.

I have released the IA3 checkpoints. They can be found in the Readme!


Upgrading CLI rc1 to 1 (Angular 2.4 to 4.0) getting below error for .spec.ts

windows 10 (64bit) and Visual studio Code
C:/ ... src/app/about/about.component.spec.ts (7,3): Module '"C:/ ... node_modules/@angular/core/testing/index"' has no exported member 'beforeEach'.
C:/ ... src/app/about/about.component.spec.ts (7,15): Module '"C:/ ... node_modules/@angular/core/testing/index"' has no exported member 'beforeEachProviders'.
C:/ ... src/app/about/about.component.spec.ts (8,3): Module '"C:/ ... node_modules/@angular/core/testing/index"' has no exported member 'describe'.
C:/ ... src/app/about/about.component.spec.ts (8,13): Module '"C:/ ... node_modules/@angular/core/testing/index"' has no exported member 'xdescribe'.
C:/ ... src/app/about/about.component.spec.ts (9,3): Module '"C:/ ... node_modules/@angular/core/testing/index"' has no exported member 'expect'.
C:/ ... src/app/about/about.component.spec.ts (9,11): Module '"C:/ ... node_modules/@angular/core/testing/index"' has no exported member 'it'.
C:/ ... src/app/about/about.component.spec.ts (9,15): Module '"C:/ ... node_modules/@angular/core/testing/index"' has no exported member 'xit'.

What version of @types/jasmine do you have? 2.5.46 does this.

Running into the same problem. I have 2.5.38 and have tried it with 2.5.47 and got the same type of errors.
Except mine are:
Namespace 'jasmine' has no exported member 'CustomMatcher'

When updating to 1.0 final, these are the versions that should be used: https://github.com/angular/angular-cli/wiki/stories-1.0-update#packagejson


Create a distillery release

Add distillery to the project and run through the process of manually creating a release. Automate this into the CI pipeline so that when a v0.x.y tag is created, a new distillery release is created for Linux, Mac, and Windows and attached to the release as a .tar.gz file, e.g. wasmcloud-host-x86_linux.tar.gz.
Let's also figure out a way to automate the generation of the aarch64 image.

Finished items

[x] Automation of creating the wasmcloud_host-v0.20.0-x86_64-linux-gnu.tar.gz artifact
[x] Automation of creating and pushing the wasmcloud_host:0.20.0 Docker image (hosted on wasmcloud.azurecr.io and docker.io)

In Progress

[ ] Automation of creating the wasmcloud_host-v0.20.0-x86_64-macos.tar.gz artifact using macos github runner

Need to be done

[ ] Automate the creation of release with a pushed tag that includes the above artifacts
[ ] Automation of creating the wasmcloud_host-v0.20.0-x86_64-windows.tar.gz artifact
[ ] Automation of creating the wasmcloud_host-v0.20.0-aarch64-linux-gnu.tar.gz artifact

Optional / Post MVP

[ ] Automation of creating the wasmcloud_host-v0.20.0-aarch64-macos.tar.gz artifact

@billyoung I'd be happy to pull up for a quick chat to go over the work that I've done, but the TL;DR is that the elixir runtime is packaged with a distillery release. This creates a standalone .tar.gz which can be unzipped and then ran using ./install_dir/wasmcloud_host foreground or ./install_dir/wasmcloud_host start. This, by itself, is an awesome system, but there are 2 parts that make it difficult.

The elixir runtime system (ERTS) is bundled with the application by default. This is compiled for an operating system/architecture, so it has to match. We have the option to not include the elixir runtime, but if we do that then the user must install the exact same version of elixir and erlang that we built it with or the project will fail to run.
We use Native Implemented Functions (NIFs) to bring in some of our Rust functionality as callable C functions. These must be compiled for an OS/ARCH, so even if we don't bundle the ERTS the NIFs will still have to match the host machine.

In order to tackle this, my first approach was to create a generic docker image where I used a different image for building the elixir runtime, rust NIFs, and for the distributed docker image I just copied and ran the application with a basic image (I used debian:buster-slim). This worked great for x86 linux, but started to fall apart for different architectures and OSes due to the complexity of cross compiling elixir and Rust, and no generally available docker images for MacOS. I still use this Dockerfile and the Dockerfile.release for the x86 linux build because it's efficient and creates our docker image for us, and I think it's generic enough that it could be used for aarch64 linux and maybe even Windows if you know how to set it up (I couldn't).
With the difficulty of macos and windows and cross compilation builds, I decided to go with using the github macos / windows runner for the next part. The current state of this effort is I have most of the macos build scaffolded out, just dealing with caching the erlang installation so we don't spend 30 minutes installing it every time. Once I finish that up, I'd love to handoff to you (Bill) to help drive this effort home!

@brooksmtownsend is this issue ready to close?

Closing in favor of #137 for clarity around what work has been done and what still needs to be implemented


Data license advice - City of Tshwane

I want to request address data from City of Tshwane for contribution into Openaddress. Will this agreement be suitable for Openaddress, if not kindly advise on issues and I will try to address them with the City of Tshwane.
CM_CoT Spatial Agreement (Rev2).pdf

Looks like it prohibits distribution, so it wouldn't be suitable for us.

The APPLICANT may only make copies of the INFORMATION for backup purposes. No other copies are allowed.

I think points 1 and 2 are also issues:

non-transferable doesn’t work for us since we aggregate and redistribute.
we normalize data into an OA data scheme (we don’t redistribute the raw
data, only the processed data)

On Wed, May 10, 2017 at 5:28 AM, Ian Dees<EMAIL_ADDRESS>wrote:

Looks like it prohibits distribution, so it wouldn't be suitable for us.
The APPLICANT may only make copies of the INFORMATION for backup purposes.
No other copies are allowed.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openaddresses/openaddresses/issues/2989#issuecomment-300466645,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0EO275rZ58NVa9Bo5ass0hAX6l53Dlks5r4a1vgaJpZM4NWfCT
.


Ed25519Signature could use stronghold/crypto types

Description
pub struct Ed25519Signature {
    public_key: [u8; Self::PUBLIC_KEY_LENGTH],
    #[cfg_attr(feature = "serde", serde(with = "serde_big_array::BigArray"))]
    signature: [u8; Self::SIGNATURE_LENGTH],
}

This could potentially use the PublicKey and Signature types from stronghold/crypto.

But these are actually define with the crypto ones so what do you mean? You'd just remove them?

Ah alright, thought you meant the constants

It would be immensely helpful if the crypto types would derive basic things like Copy, Clone, Eq, PartialEq, etc. Otherwise we'll have to work around that. I'll create a PR on crypto for this.


refactor(dependency): ocaml -> 4.11

Gotta get Onivim caught up with: https://github.com/revery-ui/revery/pull/1066
This also will let us start using memtrace (since 4.11 has the Gc.Memprof apis) which will be really helpful for perf: https://github.com/janestreet/memtrace

@bryphe why not a 4.12? 4.11 doesn't support macOS ARM64 while 4.12 and 4.10.2 do.
And at least for building OCaml the differences between native ARM64 and Rosetta emulation is huge.

@bryphe why not a 4.12? 4.11 doesn't support macOS ARM64 while 4.12 and 4.10.2 do.

Good question @EduardoRFS - primarily to minimize risk - I like to take baby steps with critical dependencies like OCaml. Sometimes there can be surprises (a perf regression... a change in GC behavior that exposes a bug in our C FFI interop... packaging gets broken... etc).
I'll start a PR for 4.12 after this is in. Having the ARM build (https://github.com/onivim/oni2/issues/2708) is a good reason to not wait too long to bring in 4.12 :+1: I didn't realize 4.11 doesn't support macOS ARM64...


Add BrightestPixelsPeakWindowSum extractor

This adds a new extractor, similar to GlobalPeakWindowSum but instead of averaging all pixels, only the brightest (as determined by the maximum) pixel_fraction * n_pixels pixels are averaged.
This should be useful for e.g. Muon Events, where we expect a synchronous signal in a small fraction of the pixels.

why not implementing this within GlobalPeakWindowSum adding a new variable set by default to 1, if it is 1 you use the old behaviour (to make it faster), when smaller the new behaviour? This will allow to make the change with much smaller code addition (and one less method to maintain/test in the future)

@jsitarek Good point, will do.
The idea was to use a different name so we can add this code to lstchain already, but of course adding the option to GlobalPeakWindowSum here does not prevent us from adding the class with a different name in lstchain

@jsitarek Currently none of the extractors support two gains (they raise a broadcasting error).

@jsitarek I added the comments in the last commit

ctapipe/image/tests/test_extractor.py::test_two_gains[FullWaveformSum] PASSED                                                          [ 12%]
ctapipe/image/tests/test_extractor.py::test_two_gains[FixedWindowSum] FAILED                                                           [ 25%]
ctapipe/image/tests/test_extractor.py::test_two_gains[GlobalPeakWindowSum] FAILED                                                      [ 37%]
ctapipe/image/tests/test_extractor.py::test_two_gains[LocalPeakWindowSum] FAILED                                                       [ 50%]
ctapipe/image/tests/test_extractor.py::test_two_gains[SlidingWindowMaxSum] FAILED                                                      [ 62%]
ctapipe/image/tests/test_extractor.py::test_two_gains[NeighborPeakWindowSum] Fatal Python error: Segmentation fault

Current thread 0x00007f72223c61c0 (most recent call first):
  File "/home/maxnoe/Uni/CTA/ctapipe/ctapipe/image/extractor.py", line 760 in __call__
  File "/home/maxnoe/Uni/CTA/ctapipe/ctapipe/image/tests/test_extractor.py", line 574 in test_two_gains
  File "/home/maxnoe/.local/anaconda/envs/cta-dev/lib/python3.7/site-packages/_pytest/python.py", line 183 in pytest_pyfun[2]    27047 segmentation fault (core dumped)  pytest ctapipe/image/tests/test_extractor.py::test_two_gains

It seems only the FullWaveformSum still supports two gains. Neihborpeak event segfaults...

Yes, I believe two-gain support was removed when the gain-selection process was introduced, which allowed a lot to be simplified.  We could add it back in if it is necessary, but that should be a separate feature request.  Right now, you can still do benchmarks and debugging of 2-gain data by using the ManualGainSelector and running the analysis two times (once for each channel).

@kosack Yes, we should probably reconsider this, as we need to support 2 gain channels for calibration events.
I mean this could happen via the the road you proposed, but it would probably be more efficient for many of the extractors to directly vectorize over the two gains and not do a python loop with the ManualGainSelector

Yes, to support calibration events, I agree it would not be good to have to run two times as they are interleaved in the data.  So we should just open a feature request. I think it shouldn't be hard to do, just needs to ignore the GainSelector's mask when dealing with a particular event type.


Make a rotate planet page navigation

We want to have a planet as in the bottom center of the image. and on rotating that planet page navigation can be achieved

assign me this

@Tejaswika you can start workingg


FEATURE: Auto create accounts and set network codeword

Issue To Be Solved
I was playing around the idea of "auto create accounts" on demand. So instead of getting error on non-existing account with some flag we can force emulator to create the account on demand.  ( I made a POC on https://github.com/bluesign/flow-emulator/tree/emulator_auto_accounts ) which creates account with the details of service account (pubkeys etc)
But I want to take this a bit further and curious about your ideas about this.
What about if we also let emulator to use any network codeword, especially testnet? ( with deploying contracts to that correct network addresses )
Also this would allow to write tests for testnet only and also can run them in emulator. I am not sure about feature parity between testnet and emulator, but seems like this can make life a bit easier for developers. Usually main problem with testnet is resetting account state, this can remove the need for that, also can push people to use emulator more.
Also this will be a backup for the times where testnet is down.
This can alternatively be implemented with using account keys data from X network, but keeping state internally, but it is a bit more confusing.

Nice! This is an interesting idea, I had the time to just glance I will circle back to this. Maybe until then, should accounts just be generated once they error as non-existing or should you provide a whitelist? I'm thinking what is the benefit for creating accounts on demand instead of beforehand since they would probably be known to the developer no? Why I'm asking this is because there might be cases where you would also want to test non-existing accounts and I see the benefit of creating them on demand just on the first run then each subsequential run would create the same state anyhow.
We had discussions on how to bring testnet closer to the emulator and give it the ability to run state from testnet on emulator and I feel this is a nice mock. Sorry that I didn't have the time sooner.

This is very good point. Potentially there can be an option creating accounts only defined in flow.json in network section that we are trying to emulate.
flow emulator --emulate testnet can create accounts defined in testnet section of the flow.json and launch emulator with testnet codeword , (optionally deploy contracts etc)
My idea to create on demand was actually came out from trying to use blocto wallet also on emulator. ( so payer etc will be created on demand when needed ) So technically you can use blocto testnet account (even mainnet) with emulator.
Main problem currently I see is: people get in to testnet prematurely, then they are having a lot of issues with contract updates/deletes and state of the test accounts etc. Somehow pushing people in to emulator but with the real toolset ( like blocto wallet testnet used on emulator etc ) can help them.

Yes I think the direction you are setting here is great and if we can leverage the testnet tools on the emulator or at least provide enough emulation of the testnet locally that you can use it for testing or debugging that would be amazing. I want to offer any help you will need with this.
I feel providing accounts in flow.json would work perfectly. Maybe there can be an environment section or something like that further defining attributes of the environment, maybe even allowing you to either provide a whitelist of accounts or just enable on-demand creation.
What would be super cool is ability to transfer the state to emulator from the testnet to inspect a bug, do some kind of debugging but I guess that then also needs to be more than just account creation but also storage migration. But maybe CLI can hide the complexity of that by inspecting the state for you and just making sure the same state is established (but this is I guess conditioned by Cadence adding API for state inspection - I believe that is coming soon or later).

I like the idea of the emulator creating accounts defined in flow.json for the emulator chain.
For testnet or mainnet there could be a way to fetch the data needed on demand.
The way that would work is that when cadence wants to read a register which is forwarded to the fvm and then forwarded to the storage layer. the storage layer would read this register in three steps:

Check local storage if register exists. If it does just read it
Check a local testnet cache if the register exists, if it does read it
Go to some service X and get the register (and all registers from the same account) and put them in the cache

When writing a register it would just be written to the local storage.
The service X would be a service that exposes an endpoint where you send it an address and a block height and it gives you all the registers for that address for that block height. The reason data is fetched per account instead of per register, is to have less requests otherwise composite objects that are split over multiple registers might cause a lot of traffic. The DPS could probably offer an endpoint like this.
One more thing that is probably needed is that if you are requesting a keys register, instead of looking it up you just substitute it with some know keys.

@janezpodhostnik thanks for the suggestion, but I think it can be very tricky ( many roundtrips ) with new atree implementation. what do you think?

@janezpodhostnik good suggestion for debugging, but I am more hesitant if we allow this, then development will stuck at testnet, emulator will be used only for debugging.
What I am trying to manage is, if we can bring up emulator with feature parity with testnet, people can delay deploying to testnet. What I see commonly is, people skip emulator, and jump into testnet. And then having a lot of issues with contract updates etc.
But this idea of yours gave me another idea:
if we have some DPS endpoint like this, probably we can simulate a transaction before running, and get a summary of state changes.
So if I am buying an NFT:
Transaction analyzer can tell me:

My flow balance will decrease X amount
TopShot Moment Y with details (blah blah) will be added to My TopShot Collection

I think with debugger this can be really nice feature.n So technically you can debug your testnet contract ( even maybe mainnet )

The DPS could probably offer an endpoint like this.
@janezpodhostnik with whom we can discuss this ?

I started implementing needed parts for this.

We added chainID support here, and I guess with new snapshot, there is no much need to create accounts in the start.  I think we can close this, what do you think @sideninja ?


more explicit logs

Codecov Report

Merging #43 (4285e02) into main (4306769) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
+ Coverage   73.59%   73.63%   +0.03%
==========================================
  Files          11       11
  Lines         784      785       +1
==========================================
+ Hits          577      578       +1
  Misses        160      160
  Partials       47       47

Impacted Files
Coverage Δ

pkg/job.go
81.28% <100.00%> (+0.11%)
:arrow_up:

Continue to review full report at Codecov.

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


set neovim-ruby-host path in runtime/init.vim

For the integration tests to work, neovim must be able to find
'neovim-ruby-host'. This is fine in most cases, but when one wants to
try and run tests without actually installing the gem, neovim cannot
find the executable.
Setting g:ruby_host_prog fixes this.
Note: I am not 100% sure this is the best way to do this, but it works here during a clean package build / test in chroots.

This is a good catch, but I think we need to set the Ruby load path as well, so g:ruby_host_prog would be something like ruby -I ./lib ./bin/neovim-ruby-host. Somehow this isn't needed to get the tests passing, but if I run ./bin/neovim-ruby-host -V it fails with cannot load such file -- neovim/host, so I think setting the load path is more correct.

Also, it seems like bundle and then bundle exec ... should fix this error, since it builds the local gem and then prepends it to the load path.

I can't get it to work with bundle.
let s:ruby_host_path = getcwd() . '/../../bin/neovim-ruby-host'
let g:ruby_host_prog = "bundle exec " . s:ruby_host_path

jason ~/neovim-ruby [host_path*] rake -v
/home/jason/.rbenv/versions/2.4.0/bin/ruby -I/home/jason/.rbenv/versions/2.4.0/lib/ruby/gems/2.4.0/gems/rspec-support-3.6.0/lib:/home/jason/.rbenv/versions/2.4.0/lib/ruby/gems/2.4.0/gems/rspec-core-3.6.0/lib /home/jason/.rbenv/versions/2.4.0/lib/ruby/gems/2.4.0/gems/rspec-core-3.6.0/exe/rspec --pattern spec/\*\*\{,/\*/\*\*\}/\*_spec.rb
Run options: exclude {:embedded=>true}

Randomized with seed 31110
.........................................function provider#ruby#Call[8]..remote#host#Require[10]..provider#ruby#Require, line 10
Vim(if):E475: Invalid argument: Channel id must be a positive integer
Failed to load ruby host. You can try to see what happened by starting nvim with $NVIM_RUBY_LOG_FILE set and opening the generated log file. Also, the host stderr is available in messages.

But running bundle exec ../../bin/neovim-ruby-host in the spec/integration folder works just fine (well, it prints the error message that I can't run the script interactively).

Sorry, I wasn't saying g:ruby_host_prog should use bundle, I was just saying it is a potential workaround to get tests passing without the gem installed. I think g:ruby_host_prog should actually be ruby -I ./lib ./bin/neovim-ruby-host.

I cannot get it working with calling this (Failed to load ruby host)
let s:lib_path = getcwd() . "/../../lib/"
let s:bin_path = getcwd() . "/../../bin/neovim-ruby-host"
let g:ruby_host_prog = printf("ruby -I %s %s", s:lib_path, s:bin_path)

I tried hardcoding paths as well:
let g:ruby_host_prog = "ruby -I /home/jason/neovim-ruby/lib /home/jason/neovim-ruby/bin/neovim-ruby-host"
Does it work for you if you try?

This does actually work for me, as far as I can tell. Without the change it fails with Cannot find the neovim RubyGem. Try :CheckHealth, and with the change the tests pass. Are you getting the same failure?

It works with the commit that I pushed here,  but with "ruby -I [...]" I am getting this:
jason ~/neovim-ruby [host_path*] rake spec
/home/jason/.rbenv/versions/2.4.0/bin/ruby -I/home/jason/.rbenv/versions/2.4.0/lib/ruby/gems/2.4.0/gems/rspec-support-3.6.0/lib:/home/jason/.rbenv/versions/2.4.0/lib/ruby/gems/2.4.0/gems/rspec-core-3.6.0/lib /home/jason/.rbenv/versions/2.4.0/lib/ruby/gems/2.4.0/gems/rspec-core-3.6.0/exe/rspec --pattern spec/\*\*\{,/\*/\*\*\}/\*_spec.rb
Run options: exclude {:embedded=>true}

Randomized with seed 10564
..............................................................................................................function provider#ruby#Call[8]..remote#host#Require[10]..provider#ruby#Require, line 10
Vim(if):E475: Invalid argument: Channel id must be a positive integer
Failed to load ruby host. You can try to see what happened by starting nvim with $NVIM_RUBY_LOG_FILE set and opening the generated log file. Also, the host stderr is available in messages.function provider#ruby#Call[8]..remote#host#Require[10]..provider#ruby#Require, line 10

I did set $NVIM_RUBY_LOG_FILE, the file is created but empty.

The last thing I can think of is also setting NVIM_RUBY_LOG_LEVEL=debug. If you still don't get any logging then the host process really never started up. Otherwise you'll get a LOT of logs, you can attach to a comment.

Okay strange.
Now it works fine, without changing anything. I pushed the latest version, feel free to edit if necessary.

Even better!


merge from konsum-frontend-svelte

package.json

chore(deps): add ^10.15.2 remove ^10.15.1 (devDependencies.npm-pkgbuild)

.github/workflows/browser_ci.yml

chore: add ${{ matrix.os }} (jobs.test.runs-on)

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

npm package (@latest dist-tag)
v1.34.11
GitHub release

Your semantic-release bot :package::rocket:


Electron 7: macOS zip does not seem proper

The contents appear as:

That extra file seems it should be removed.

https://github.com/electron/electron/pull/21447

Seems fixed now.


composite key in model causing invalid SQL

if in modelBuilder there is a composite key present and the object is in multiple relatinships
e.g. A contains Many B, B contains many C with fully defined relationships
then fetching A with dependents results in invalid column ID in query, if relationship to C is commented out, it works fine
modelBuilder.Entity<A>(entity=> {                 entity.HasKey(e => new{ e.EventId, e.EventCloseDate});             });
more detail logged here http://stackoverflow.com/questions/36586487/entity-framework-7-generating-invalid-column-name/36587278

Using model from stackoverflow question, while I am trying to generate model, I am hitting following exception:
The key {'EventId1'} contains properties in shadow state and is referenced by a relationship from 'Event.Sessions' to 'Session.Event'. Configure a non-shadow principal key for this relationship.
The conflicting lines in the model causing this error are:
modelBuilder.Entity<Event>()
        .HasMany(s => s.Sessions)
        .WithOne(e => e.Event).HasForeignKey(s => s.EventId);
modelBuilder.Entity<Event>().HasKey(e => new{ e.EventId, e.EventCloseDate});

First line of code defines one to many relationship between sessions and event which uses EventId (in Session class as foreign key property. By default this FK targets the PK of principal type - Event which is EventId by convention. When second line tries to set composite PK on Event class, the relationship defined before cannot use that PK any longer since relationship has only 1 property as FK. Therefore conventions will create a shadow property from the user-configured FK property and define key on shadow property to be used as principal key for relationship. Since EventId property already exists, the shadow property created has name EventId1. Since shadow keys are not allowed in the model, above exception is thrown during model validation phase. As exception message says, user is trying to set single property FK targeting principal type with composite key therefore user has to specify the principal key explicitly.
It seems that RC1 packages was not making such checks therefore the generated query contained EventId1 column.
Since above 2 statements in model building is causing this issue to arise, removing either of those (not configuring relationship explicitly or not defining composite PK) resolves the issue. Or other work-around would be define principal key to use in explicit relationship configuration.

I worked around by removing the composite key (and using primary only) -
issue arose because i used scaffolding to generate the models and it
inherited the properties form the existing table
On Tue, 19 Apr 2016 at 05:40 Smit Patel<EMAIL_ADDRESS>wrote:

Using model from stackoverflow question, while I am trying to generate
model, I am hitting following exception:
The key {'EventId1'} contains properties in shadow state and is referenced
by a relationship from 'Event.Sessions' to 'Session.Event'. Configure a
non-shadow principal key for this relationship.
The conflicting lines in the model causing this error are:
modelBuilder.Entity()
.HasMany(s => s.Sessions)
.WithOne(e => e.Event).HasForeignKey(s => s.EventId);
modelBuilder.Entity().HasKey(e => new{ e.EventId, e.EventCloseDate});
First line of code defines one to many relationship between sessions and
event which uses EventId (in Session class as foreign key property. By
default this FK targets the PK of principal type - Event which is EventId
by convention. When second line tries to set composite PK on Event class,
the relationship defined before cannot use that PK any longer since
relationship has only 1 property as FK. Therefore conventions will create a
shadow property from the user-configured FK property and define key on
shadow property to be used as principal key for relationship. Since
EventId property already exists, the shadow property created has name
EventId1. Since shadow keys are not allowed in the model, above exception
is thrown during model validation phase. As exception message says, user is
trying to set single property FK targeting principal type with composite
key therefore user has to specify the pri ncipal k ey explicitly.
It seems that RC1 packages was not making such checks therefore the
generated query contained EventId1 column.
Since above 2 statements in model building is causing this issue to arise,
removing either of those (not configuring relationship explicitly or not
defining composite PK) resolves the issue. Or other work-around would be
define principal key to use in explicit relationship configuration.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/aspnet/EntityFramework/issues/5047#issuecomment-211544563

@rowanmiller - Is there anything actionable from our side here?

Don't think so, if you are happy we understand it and there is nothing to change then feel free to close 😄


docs(logging): DOC-532 add page on logging

Resolves Jira: DOC-532

Created an Observability section and moved metrics/logging docs there
Added "draft: true" to existing Integrate Splunk with Armory Enterprise page so it's not compiled; the new logging page comes up when users search for "splunk"
Created a new Configure Monitoring using the Observability Plugin page with content from the Monitor with Prometheus page

@kinnonyee-armory  the new logging file is content/en/armory-enterprise/armory-admin/observe/integrations-logging.md
Deploy preview:  https://deploy-preview-1232--armory-docs.netlify.app/armory-enterprise/armory-admin/observe/integrations-logging/

I believe there were other dashboards that @chrisjeggo created.  Were there reasons why only two of them were added?

Whoop.s. One more thing @aimeeu
The HOSTNAME:PORT can be HTTPS, which would remove the need for the insecure TOKEN. Consult your specific data aggregator provider’s documentation for how to configure an HTTPS endpoint.
That portion should probably say SERVICE_URL:PORT


Conductor bugfixes

What Changed

Many minor bugfixes

Issues

Fixes #443
Fixes #441
Fixes #440

NOTE We will still have to go in and change a bit of the data to actually fix #441 and get it to show up again.

Sick LGTM :+1:


Pandoc permission error when adding a note to a zotero item

I am receiving a pandoc permission error when adding noted to zotero. The note is still added correctly and all its content is there, so I am not sure where the error stems from. Here is the output from running zotcli -v add-note <id>
DEBUG:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): api.zotero.org
DEBUG:requests.packages.urllib3.connectionpool:https://api.zotero.org:443 "GET /items/new?itemType=note HTTP/1.1" 200 82
DEBUG:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): api.zotero.org
DEBUG:requests.packages.urllib3.connectionpool:https://api.zotero.org:443 "GET /users/1905432/items?limit=1&format=json HTTP/1.1" 200 701
Found /home/joel/anaconda3/bin/pandoc, but not using it because of an error:
[Errno 13] Permission denied
DEBUG:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): api.zotero.org
DEBUG:requests.packages.urllib3.connectionpool:https://api.zotero.org:443 "POST /users/1905432/items HTTP/1.1" 200 564
DEBUG:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): api.zotero.org
DEBUG:requests.packages.urllib3.connectionpool:https://api.zotero.org:443 "PATCH /users/1905432/items/DRVZJ8MT HTTP/1.1" 204 0

The permissions of /home/joel/anaconda3/bin/pandoc are
-rwxrwxr-x 2 joel users 53173152 Nov 15 06:49 pandoc*

That's interesting, since the binary has execute permissions for all users... Can you run /home/joel/anaconda3/bin/pandoc as user joel from outside of Python?

II think this was an issue with the pandoc I was using locally. I changed it so that my distribution's pandoc binary comes before the one provided with anaconda on my PATH, and I haven't seen the problem since. Sorry, I forgot to update here.


Parse always includes "_rperm" in the Query

Env
Parse Server 2.2.17
Problem
I don't know if this is an issue or not but after I enable Profiling (level 1) on MongoDB to check slow queries, I saw that alomost all queries are including _rperm
'queryMask': '{"$query":{"_rperm":{"$in":[]},"approved":"<val>"}}',
Although this class is public read, and the _rperm is being passed empty (by Parse Server? by SDK ?),
The same result can be obtained by
'queryMask': '{"$query":{"approved":"<val>"}}',
and in the SDK (here iOS) we don't pass any ACL argument to the query, and the queried class is public read
- why is this an issue?
because this would make the obvious {"approved": 1} index like this completely useless and must create this index {"approved": 1, "_rperm": 1} instead

@benishak can you work on a PR for that?

@flovilmart yes I will. I will update the DatabaseController

@benishak did you make any progress on that?

@benishak ping?%

Although this class is public read  yeah but ACL's per object can say the opposite, as remember ACL override the CLP. There is no way to know ahead of time how to optimize that.


Add NewJarManifest function to retrieve MANIFEST from a JAR file

Summary
Adding a function to retrieve the MANIFEST from a JAR file. This is needed in Support multiple JAR files by picking the first found Main-Class #118
Use Cases
See https://github.com/paketo-buildpacks/executable-jar/pull/118
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Thank you for the feedback @dmikusa-pivotal. I applied your suggestions.


(Manifest) by The Weakerthans unsupported.

Check if issue with swaglyrics or whether song lyrics unavailable on Genius.

stripper -> The-Weakerthans-

done


az storage account network-rule remove does not delete all the rules for storage account

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command
az storage account network-rule remove --account-name
Describe the bug

Its not deleting all the rules from storage account
To Reproduce

az storage account network-rule remove --account-name
Expected behavior

It has to remove all the IPs added in firewall rule
Environment summary

bash
Additional context

@evelyn-ys for awareness

Hi @onenessboy, it's not the correct usage. You need to provide --ip-address or --subnet to delete network rules one by one. We don't support remove all.


@graphql-yoga/plugin-prometheus breaks subscriptions over graphql-ws

Describe the bug
On installing the prometheus plugin, subscriptions fail with the error:
65 |             return undefined;
66 |         },
67 |         onParse() {
68 |             return ({ result: document, context: { params, request } }) => {
69 |                 const operationAST = getOperationAST(document, params.operationName);
70 |                 paramsByRequest.set(request, {
                                     ^
TypeError: WeakMap keys must be objects or non-registered symbols
      at /Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/node_modules/@graphql-yoga/plugin-prometheus/esm/index.js:70:33
      at /Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/node_modules/@envelop/core/esm/orchestrator.js:111:17
      at /Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/src/index.ts:71:17
      at onSubscribe (/Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/src/index.ts:60:23)
      at /Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/node_modules/graphql-ws/lib/server.mjs:146:124
      at onMessage (/Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/node_modules/graphql-ws/lib/server.mjs:51:61)

If we comment out the plugin, subscriptions work again.
If we use the plugins from @envelop/prometheus, the subscriptions work again, but http://localhost:4000/metrics gives a 404.
What is the correct way to use the plugin with graphql-ws subscriptions?
I can also open a PR but will need guidance on what the fix is.
Your Example Website or App
https://github.com/snigdha920/bun-graphql-yoga-prometheus/tree/main
Steps to Reproduce the Bug or Issue

Run  bun dev
Go to http://localhost:4000/graphql
Run the subscription:

subscription MySubscription {
  dynamicLoading(loadTime: 10)
}

You will see there is no data loading:

and this error in the console:
65 |             return undefined;
66 |         },
67 |         onParse() {
68 |             return ({ result: document, context: { params, request } }) => {
69 |                 const operationAST = getOperationAST(document, params.operationName);
70 |                 paramsByRequest.set(request, {
                                     ^
TypeError: WeakMap keys must be objects or non-registered symbols
      at /Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/node_modules/@graphql-yoga/plugin-prometheus/esm/index.js:70:33
      at /Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/node_modules/@envelop/core/esm/orchestrator.js:111:17
      at /Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/src/index.ts:71:17
      at onSubscribe (/Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/src/index.ts:60:23)
      at /Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/node_modules/graphql-ws/lib/server.mjs:146:124
      at onMessage (/Users/snigdhasingh/Development/bun-graphql-yoga-prometheus/node_modules/graphql-ws/lib/server.mjs:51:61)

Comment out the usePrometheus plugin, run the subscription again, works as expected

Expected behavior
I expected the subscription to return data
Screenshots or Videos
No response
Platform

OS: macOS
Bun: 1.1.13
@graphql-yoga/* version(s): 5.6.1

Additional context
No response

The error is here: https://github.com/dotansimha/graphql-yoga/blob/855600cd6f39102f8bdfd5d9f54bf62c9a26bc0f/packages/plugins/prometheus/src/index.ts#L107, when using subscriptions there is no request in the context, and we still try to use it as a key for setting params, adding something like:
// When parsing subscriptions, there is no request
if (!request) {
    return;
}
paramsByRequest.set(request, {
    document,
    operationName: operationAST?.name?.value,
    operationType: operationAST?.operation
});

fixes the issue.
But I'm also not sure why we're using request as a key, in the original @envelop/prometheus they use the entire context, which already takes care of this condition:
https://github.com/n1ru4l/envelop/blob/737f250a32517560d813ed479d671e1412822ec6/packages/plugins/prometheus/src/index.ts#L270
So perhaps the best fix is to use the context as the key?

Right now I'm using the custom minimal metrics plugin, because I only really need the /metrics endpoint: https://github.com/snigdha920/bun-graphql-yoga-prometheus/commit/c2a6dd21f1023b00a8e1ba925d6bc98cc805c405


[fix] Disabled, unimplemented, and unknown opcodes should throw an error

[x] issue #109
[x] follows contribution guide
[x] code change includes tests

Added the necessary logic to return errors without executing opcodes inside a if statement when is falase

Nice work! Thanks for the quick changes.

🫡🫡🫡


Chrome white screen of death

When i start chrome myself it is working fine, I can browse web normally, but when tagui starts chrome it's a white window, sometimes without title bar. It happen suddenly, it was ok yesterday. I run flows daily.
What I tried that didn't work:

Uninstall and reinstall chrome.
delete chrome user data.

Specs:
Google Chrome is up to date Version 89.0.4389.82 (Official Build) (64-bit)
Windows 10 Home
Version	20H2
Installed on	28 Jul 2020
OS build	19042.804
Experience	Windows Feature Experience Pack 120.2212.551.0

Hello @eienkisu
This is an unusual scenario. Can you paste the logs for us to take a look? The log file can be found here: tagui/src/tagui_chrome.log
Also, is this your company laptop? It could be a change in company network policy that blocks the TagUI access to chrome, which was the case for one of our users - https://github.com/kelaberetiv/TagUI/issues/921.

Hello @eienkisu
This is an unusual scenario. Can you paste the logs for us to take a look? The log file can be found here: tagui/src/tagui_chrome.log
Also, is this your company laptop? It could be a change in company network policy that blocks the TagUI access to chrome, which was the case for one of our users - #921.

Hi Ruth
log attached.
strange enough when using work laptop it works fine, using vpn or not. I'm a nus staff.
tagui_chrome for 1_google.log

Hello @eienkisu
The nus laptop is slightly different from your personal laptop specs, but this shouldn't be an issue. Do you have any anti-virus software installed on the laptop?
I've taken a quick look at the logs you attached, seems like TagUI does not throw an error upon visiting Google.com although the webpage is not responding and continues with the next step and is able to find an element with id of "q". For this sample, if I've not mistaken, TagUI would have matched the element by it's name and not id...
I also noticed another strange thing from your screenshot, the output "c:\tagui\src\tee.exe: Reading stdin..." appears twice when running the flow, which should not be the case.
My suggestion is, you may reinstall TagUI by deleting the tagui folder from your C drive, and unzipping a fresh copy. Please follow the steps at https://tagui.readthedocs.io/en/latest/setup.html.
Do let us know if the issue persists upon reinstalling. Thank you!

Hi Ruth
deleting tagui folder and reinstalling tagui works. Thanks!

From: Ruth @.>
Sent: 12 March 2021 11:55
To: kelaberetiv/TagUI @.>
Cc: eienkisu @.>; Mention @.>
Subject: Re: [kelaberetiv/TagUI] Chrome white screen of death (#963)
Hello @eienkisuhttps://github.com/eienkisu
The nus laptop is slightly different from your personal laptop specs, but this shouldn't be an issue. Do you have any anti-virus software installed on the laptop?
I've taken a quick look at the logs you attached, seems like TagUI does not throw an error upon visiting Google.com although the webpage is not responding and continues with the next step and is able to find an element with id of "q". For this sample, if I've not mistaken, TagUI would have matched the element by it's name and not id...
I also noticed another strange thing from your screenshot, the output "c:\tagui\src\tee.exe: Reading stdin..." appears twice when running the flow, which should not be the case.
My suggestion is, you may reinstall TagUI by deleting the tagui folder from your C drive, and unzipping a fresh copy. Please follow the steps at https://tagui.readthedocs.io/en/latest/setup.html.
Do let us know if the issue persists upon reinstalling. Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/kelaberetiv/TagUI/issues/963#issuecomment-797217270, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGMEIPYBCAJYRYVH6SVLZYLTDGGEBANCNFSM4Y72CFFA.

Awesome @eienkisu! Closing this issue, have fun automating with TagUI :-)

Thanks @ruthtxh ! - I've updated issue title with the solution (for this case is reinstalling)


Implement more Effect Size metrics

Other effect size metrics could be useful, for example Hedges' g.
A switch (e.g. --metric hedges) can be added to change what metric is computed by fast-cohen.
Perhaps also a change of names would be appropriate at this point.

Superseeded by gene-ranker


Configurable Healthcheck Expose Port

Healthcheck port (exposed one) is hard coded to 3999. This is a problem when doing parallel deployments and might result in this error
ERROR (SSHKit::Command::Failed): Exception while executing on host plus-6fc20fca3f82fa64049fec08bac4b533.svc: docker exit status: 125
--
  | docker stdout: fec19f2cf5310d4900e21ed48681ce9e896901185f656cfd1797d109af9bf251
  | docker stderr: docker: Error response from daemon: driver failed programming external connectivity on endpoint healthcheck-xxx (92d063a6607cbf7525970c123a75598dcde5dae34862ad9e436c05e9bcb82406): Bind for <IP_ADDRESS>:3999 failed: port is already allocated.

This PR adds an option to healthcheck configuration and allow to pass exposed_port via configuration. Default value stays the same and is still 3999.

Can you document this in kamal-site? Otherwise I will try to do it when I have some time


[locale] bn: Improve bn translation

According to "Bangla Academy", the change are applied. I am working a govt project for Bangladesh where I need to use the locale data and I noticed these. Recently "Bangla Academy" declare of how we should use these English month names when we write it in Bengali letters.
reference:

https://github.com/date-fns/date-fns/blob/v2.0.0-alpha.27/src/locale/bn/_lib/localize/index.js
https://www.prothomalo.com [Bengali renowned Newspaper]

@kaushikgandhi I did some modifications. Please take a look.

Coverage decreased (-0.01%) to 88.398% when pulling 2176ef751e9772a2013f9383c2583f268326048e on mgorabbani:develop into d0f0dd8f40be987593767a3dc5e27c45c6a3fa49 on moment:develop.

Please revert the spelling of january. Other changes seems ok.

@mgorabbani please send the link where we can check the spellings of the month.

that's how January is written.

check here. https://www.prothomalo.com/archive

then click on the calender icon

though both spellings are correct, Indian Bengali version seems to be using the old January spelling, We still do, But Bengali Academy of Bangladesh seems to have approved this so I am accepting this.

I am waiting to use this change!

You can just use your own repo for that time.
On Wed, Mar 27, 2019, 4:12 PM Golam Rabbani<EMAIL_ADDRESS>wrote:

I am waiting to use this change!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/moment/moment/pull/5055#issuecomment-477090298, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACETDJoX4JHs0it5CGTahR2IpYYVnldyks5va0swgaJpZM4cNTem
.

@marwahaha - Can you please have a look at the PR. Thanks

Thanks for the change and your persistence. When I have some time to make a release this will be included!


Added DuplicatedAbstractByteBuf that can provide fast-path for _get* …

…and _set* methods
Motivation:
DuplicatedByteBuf can be used for any ByteBuf implementations and so can not do any optimizations that could be done
when AbstractByteBuf is duplicated.
Modifications:

Add DuplicatedAbstractByteBuf that can eliminate range and reference count checks for _get* and _set* methods.

Result:
Faster DuplicatedByteBuf implementations for AbstractByteBuf sub-classes.

@Scottmitch @nmittler please check

LGTM

Cherry-picked into 4.0 (b8c73a48061d1bb3c12b8c4fb35e5f882780e0b7), 4.1 (8c93f4b1efbbb6e75038a6a7a18cf005d3792c84) and master (10e47f16dfb96d9733847ecbef766b0fb6e9ef3d)


Added types (colors) to Switch component

I thought a type prop could be useful on b-switch so here it is ... 😃
If it's not a good idea just close this.

@Tirke Yes, it could be useful! Thanks


pm2 start  status stopped

Node version 12.21.0
Using PM2 to start the process status is stopped。
The system is windows。

pm2 logs

iuuses：https://github.com/nuxt/nuxt.js/issues/8704


feat(webui): reverse log order to display newest first

The applyLogStyling function in log.html has been updated to reverse the array of log lines.
The reason is because usually we need the most recent logs, not the old ones.

Thanks!

https://github.com/AlexxIT/go2rtc/releases/tag/v1.9.0


Research api.github.com

Make some command line “curl” requests to begin understanding how to make requests and what responses look like. Everyone should should understand this.

You can find details on the GitHub API here!

See #7, we've elected to use a GitHub API Java Library, which will allow us to adapt to new API versions with less code alteration and interact with GitHub through a much more "friendly" api.


Update nf-winerror-hresult_severity.md

Fix HTML escaping bug

Other pages also have the same issue, such as HRESULT_SEVERITY, SCODE_CODE, etc.


Problem with "{{> callTemplate}}"!

I use  Blaze of Meteor JS.
I have problem with {{> callTemplate}} format.
Please help me.

From the main menu could you do:
Packages -> Atom Beautify -> Debug
And then paste the results in a gist.  We require more details to move forward.

Please follow the issue template that automatically appeared when you created this issue.
For your convenience, here is the template: https://github.com/Glavin001/atom-beautify/blob/master/ISSUE_TEMPLATE.md

I also have an issue with Blaze autoformat,
`
resolutions

        {{> resolutions}}

`
The template tag is autoformatted to:
`
Title
`

We don't have enough details to investigate this old option.  Please reopen this issue with more information.


fix issue with filename

Should fix the issue mentioned in #8
Could you check if that's ok?
Thanks!

This has been fixed and merged in #19