snowflake_grant_privileges_to_role

Is your feature request related to a problem? Please describe.
We are often giving a role access to future and existing objects like tables, views, functions, etc. We have to define 2 different resources for these scenarios. One for the future and another for the current objects. If you are in a situation where you are iterating across every schema in every database and applying future and current grants, this spawns a ton of resources, which leads to slower plans, drift detection, etc.
Describe the solution you'd like
It would be great if we could include multiple operations in the 1 resource for scenarios as described above. I could see the usage being something like this:
resource "snowflake_grant_privileges_to_role" "grant_schema_read_usage" {
  privileges = ["USAGE"]

  role_name  = "ROLE_NAME"

  on_schema {
    future_schemas_in_database = var.database_name
    schema_name = "\"${var.database_name}\".\"${var.schema_name}\""
  }
}

or
resource "snowflake_grant_privileges_to_role" "grant_schema_read_tables" {
  privileges = ["SELECT"]
  role_name  = "ROLE_NAME"

  on_schema_object {
    future {
      object_type_plural = "TABLES"
      in_schema          = "\"${var.database_name}\".\"${var.schema_name}\""
    }

    all {
      object_type_plural = "TABLES"
      in_schema          = "\"${var.database_name}\".\"${var.schema_name}\""
    }
  }
}

Describe alternatives you've considered
The alternative is to keep doing this as separate resources.

Similarly, we often find ourselves with a need to declare one of these for each user. Having an array of role_names would be exceptionally useful and cut down on very large amounts of code
resource "snowflake_grant_privileges_to_role" "grant_schema_read_usage" {
  privileges = ["USAGE"]

  # role_name  = "ROLE_NAME"
  role_names  = ["ROLE_NAME_A", "ROLE_NAME_B"]

  on_schema {
    future_schemas_in_database = var.database_name
    schema_name = "\"${var.database_name}\".\"${var.schema_name}\""
  }
}

Hey @Bryan-Meier. Thanks for reaching out to us.
We are in the process of redesigning the existing resources and adding the missing functionalities as part of https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/ROADMAP.md#supporting-all-snowflake-ga-features). I'll add it to our list of things we would like to address/discuss.

@Josh-a-e This would definitely be very useful as we also would like to grant multiple roles to the same object. @sfc-gh-jcieslak Could this be a feature in the roadmap or is there limitations to this?

Hey @lukeyz1
It may, we'll talk about it. There are limitations to those proposals, but I didn't analyze them well enough to say if they'll end up in the provider or not. There are certainly some questions or cases I can think of we would struggle with (grant resources are already complex with one role and one action), but because it's a common issue with grant resources (large number of them == slower apply times) we have it on todo list to analyze/improve.

@sfc-gh-jcieslak Do you know if Snowflake has considered simplifying the privilege and role schema? The approach that Snowflake adopted from the beginning is verbose to the extreme.
I have been around a lot of databases systems in my career and I gotta say (IMO) Microsoft's privilege hierarchical approach to privileges in products like SQL Server is hands down the easiest to manage and requires way less manual maintenance. I would guess that I could apply all the privileges I would need in more than half the SQL statements on a privilege system like SQL Server. It would also make this TF project much more simplified. 😄

Hey 👋 , I'm not aware of Snowflake's design decisions on their grants model, but I agree they're tricky to work with sometimes 😓. Thanks for the feedback though, I'll forward it and maybe get back to you if there is anything worth sharing.


Change strategy of output.

Use a different strategy to retrieve the content of STDOUT and STDERR.
The ShellOutError structure has been changed but the API remains the same.
The structure now contains both STDOUT and STDERR as Data and we can
retrieve the equivalent String using the previous properties message
and output.
From my memories, any non 0 terminationStatus should be considered as
an error.
This also allows to introduce a parameter printOutput which will
output the received data to STDOUT and STDERR.

Hi @yageek! Thanks for this PR! Could you please outline what the target usecase is for this change - that is - why is it better than the current implementation? 🙂

The thing I wanted to add is still not working after some tests (last bullet) :(. The elements I wanted to improve:

[X] Any non 0 output should be considered as an error, not only when something has been written to STDERR.
[X] Possibility to retrieve any Data and the termination status when an ShellOutError is thrown.
[ ] As soon as characters are received by the Process, I would like to stream them directly out instead of waiting the completion to finish. In a CI script, it could be useful to stream the received elements as long as the compilation is going forward.

After taking a look more closely, this is working in deed. I have a created a compilation script that behaves like travis and I am able to see the output as it is streamed. This behaviour could be used also for marathon run.

@yageek Thanks again for this, I love the idea - it's going to make a big improvement to Marathon, since you don't have to wait for the full output 👍 I put some comments inline, and also some tests are needed before this can be merged in 🙂

I have updated the API and added the missing tests. The only question remaining is to allow/disallow access to the raw Data in case of error.

I renamed the arguments. Your proposition made more sens. About the error, according to me, me should give the opportunity to have access to the data. Here when we are converting to Stringwe assume we are using an utf8 encoding. May be some programs outputs text in a different encoding. Providing an access to the internal buffers will allow any transformation.


Spreadsheet: Overhaul colors

Follow a color scheme similar to the one for our houses sheet. I haven't landed in the exact color set yet. My housemate is a designer and helping me narrow it down a bit to something more consistent and pleasant. The aim with the colors and fonts is to make the whole thing feel more gentle and accessible.

Status: @catherio requested input from @beshaya and @doctordobes before proceeding.


Touch Event not handle correct on android

i'm not sure this proplem caused by InvertibleScrollView or this component,touch inverted on android.

yes it's related to the invert, RN issue https://github.com/facebook/react-native/issues/3557
I recommend to use inverted={false} in Android until it's fixed in RN core

It will be fixed in RN 0.17
https://github.com/exponentjs/react-native-invertible-scroll-view/issues/16


Document API V2 converts infrastructure exceptions to 5xx

Example where 405 is converted to 500. That should not be done.


Using SmObjectFactory in WebAPI project

When I use SmObjectFactory in my WebAPI project I get this error:
type StructureMapHttpControllerActivator must derive from IHttpControllerActivator. Parameter name: service
I also used SmObjectFactory in WebApiConfig.cs this way:
var container = SmObjectFactory.Container;
GlobalConfiguration.Configuration.Services.Replace(
     typeof(IHttpControllerActivator), new StructureMapHttpControllerActivator(container));
// Web API routes
config.MapHttpAttributeRoutes();
config.Routes.MapHttpRoute(
     name: "DefaultApi",
     routeTemplate: "api/{controller}/{id}",
     defaults: new { id = RouteParameter.Optional }
);

Any idea?
Thanks.

StructureMapHttpControllerActivator is derived from IHttpControllerActivator.
Also you can find its improved version using SmObjectFactory  here: DI08 (It's tested and works)


Update hiredis to 1.2.0

Open the door to https://github.com/redis-rb/redis-client/issues/133
https://github.com/redis/hiredis/issues/1059 wasn't fixed so I had to re-apply my patches.

The upgrade cause the client to deadlock in some situations, I'll have to investigate this.


using vitruvius library

i am already working on a project with kinect and now i want to use your library in that project. I am new to kinect so can you guide me through steps need to be done to add your library in my project.

I believe the best way to add Vitruvius to a project is to use the compiled assemblies via a NuGet package, however for first trying it out you'd better use "download zip" option from GitHub to get the source and the samples and try them

big thnx for such quick help ,I tried again and finally it worked. i will be using it in my project and will be asking more questions about it in future.......hehe


🐛(SEO) fix opengraph og:image meta

The og:image head meta need to have an absolute URL.
Currently on demo we have this value http://edx.richie.education//dpaz9rouqy0e2.cloudfront.net/static/richie/images/logo.bc96dea393cb.png when a social network scraps the page it won't found the image because the URL is bad formed.
Richie is prefixing the og:image meta content value with the site name. But when static assets of the richie site is behind a CDN the URL of the og:image has an incorrect value.

@igobranco we are finalizing PR https://github.com/openfun/richie/pull/1390 which is refactoring OpenGraph tags. Can you check if the subject of your PR is covered or make a PR directly on the add-schema-metadata-courses branch to improve it?

@sampaccoud the PR still makes sense to include on the add-schema-metadata-courses branch. So I'm going to create a new PR to that branch.


generate message if OS/CPU isn't supported by product (the packager)

Instead of 404-ing, we should tell the user upfront the platform is not supported, and where to file a bug.   We have all the information to do this.

this is working except for ARM


qtav: new package

This adds a recipe for QtAV - a cross-platform multimedia framework
based on Qt and FFmpeg.

Thanks
Just a tiny nit: Please replace 'DESCRIPTION' by 'SUMMARY'

Thanks
Just a tiny nit: Please replace 'DESCRIPTION' by 'SUMMARY'

Done + added HOMEPAGE and AUTHOR variables too.

Thanks / cherry picked it to dunfell branch


DropDown value

not available value selected

I think the drop down menu is not showing what is selected
When a value is selected the current value does not appear in the UI and how to set the value from the API

You should use ComboBox for that

Thank you sorry

You should use ComboBox for that

I am sorry thank you for helping


FIX: demande devient déclaration

close jira #147

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


Ask the user if they want to create a log file on the desktop with the error message if it's something that I don't know of yet and tell them to report the bug on this page.

Actually, I'm going to ask them to write down the error message and type, along with what they were doing when the error occurred.

It would probably be better to just make a log file and have them attach it to a GitHub Issue so I can read it easier. Place on the desktop as well.

I just asked them to write it down. Making a log file sounds confusing.
https://docs.microsoft.com/en-us/dotnet/standard/io/how-to-open-and-append-to-a-log-file

I'm now actually implementing a logging mechanism with that MSDN article above, but I want it to write to the AppData folder instead of the app's current directory (Program Files can't be written to easily) so I searched how to do it and here's what I found:
https://stackoverflow.com/a/6634146

Log files are now being written to %LocalAppData%\UXL_Launcher.

Copied from #90:
"Actually, I'm not going to implement logging completely. If you want to see how I did logging in my app launcher, I have a branch I won't merge or delete."


Implement editor indent settings per project (type and size)

Solves #80763.

Already covered more completely in:

https://github.com/godotengine/godot/pull/69012

Already covered more completely in:
* [Allow to override editor settings per project #69012](https://github.com/godotengine/godot/pull/69012)

Maybe overcomplicated since it hasn't been merged in 2 years? We need a tool.

Not really overcomplicated, just a major feature, a tool for what?

a tool for what?

For programming. Try to work in multiple project with different indent settings.
Editor settings are related to user. Project settings are related to project.

I don't think it's a good idea to merge half-solutions and temporary things

It's not a half solution. It is a complete solution to stop Indent Wars, and stop repeating issues with formatting in multiple project...

The solution is to make all setting configurable on a per-project level, not just some specific settings with ad-hoc solutions

The solution is to make all setting configurable

We have different goals. We (all developers) need a tool for work.
The rest / most of editor settings are user-oriented. There is little point in setting them up for a project. Moreover, they can introduce a bunch of a new issues (imagine that somebody will change YOUR preference about a particular text or gizmo color).

Converted to draft because of possible vulnerabilities (settings not exposed in GUI can be written to the file).
See: https://github.com/godotengine/godot/pull/69012#issuecomment-2417897377


dep init error on windows when drive letter is lowercase in GOPATH

Go Version: go version go1.7.5 windows/amd64
OS Information: Windows 10 Pro
Processor Architecture: x64
What did I do:  ran dep init in project folder
What did I expect: for dep init to run properly
What did I see: the command returned project not in a GOPATH
Explanation of issue:
When running dep init on windows you get project not in a GOPATH when the drive letter is lowercase in the GOPATH environment variable.
The non working GOPATH: c:\bryan\dev\go
The updated working GOPATH: C:\bryan\dev\go
Thoughts about the issue:
I am not really sure if this is an issue that needs to be specifically fixed, or just stated that windows users need to make sure to have the drive letter capitalized on windows.

I think I'm fine with normalizing the case of the drive letter so that the user doesn't have to care about this detail.
However, if there's anything analogous to this kind of problem in the go toolchain (I have no idea if there is), I'd prefer to follow the pattern there. @adg, any idea?

AFAIK, go toolchain should works fine for case-insensitive.

As long as we're using filepath.HasPrefix then it should just work.


on windows, using gitbash/mintty, fzf reverse history search (ctrl-r) hangs the terminal indefinitely

Checklist

[X] I have read through the manual page (man fzf)
[X] I have searched through the existing issues
[X] For bug reports, I have checked if the bug is reproducible in the latest version of fzf

Output of fzf --version
0.54.3 (af4917d)
OS

[ ] Linux
[ ] macOS
[X] Windows
[ ] Etc.

Shell

[X] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
on windows, fzf reverse history search hangs the gitbash mintty terminal indefinitely, causing it to become unresponsive and have to be forced closed.
expected behavior
the fzf reverse history search allows user input to search through history.
actual behavior
the fzf reverse history search hangs the terminal program indefinitely, user input no longer works, signals can't be sent via ctrl-c or otherwise, and the terminal has to be forcibly closed.
steps to reproduce

on windows 11, git/bash 2.4.0 installed via choco
install fzf via choco or winget
open the gitbash terminal (mintty)
invoke fzf directly, verify it responds to input and works fine
source the fzf setup (eval "$(fzf --bash)") via .bashrc or manually in session
press ctrl+r

Oh, it's v.2.45.2.windows.1. Does updating from 2.40 solve this issue?

Pardon, 2.4.0 (seems rather old).

upgrading to 2.46,.0 the latest release of git for windows, fixed the problem :)


Shri/c iterative example

Pull request for C example with iterative API.

There was a number of new warnings that showed up on the gcc 4.9 build in travis.  Those will need to be cleaned up before we can merge.

Fixed. Does gcc4.9 not allow mixing declarations and code? I've seen this error with Windows build before, not for gcc.

I think what is going on is that gcc 5.0 switch the default C standard to c11 which is probably why we don't see these warnings in gcc 6 or clang builds.  GCC 4.9 still had the c99 standard which when combined with the -pedantic  probably generates these warnings.  My guess is we could add a -std=XXX something to the C builds which would remove the warnings.  I would be a little hesitant with that since we want the C library to be compatible with somewhat older versions but I am not sure how far back we want to go.

I'd like to understand more of the iterate_if_needed flag. What happens in the following scenario for both federates:
Federate 1: Determines that it has not locally converged, publishes its value, and calls HelicsTimeIterativeRequest with iterate_if_needed flag.
Federate 2:  Determines it has converged locally, does not want to publish its value, and calls HelicsTimeIterateRequest with iterate_if_needed flag.
Do both the federates continue iterating in this case?
This paper has a few simple examples testing different co-simulation interaction protocols (Gauss-Seidel, single-iteration Gauss-Seidel, and Newton). I'm planning to implement one of the examples using HELICS. This would be a good test of iterative API with numerical integration. What do you think?


pf: Aliases exclusions (supress)

Hi!
pf supports exclusions in tables (eg { <IP_ADDRESS>/24, !<IP_ADDRESS> })
(for now AliasContentField.php and alias.py do not allow to add addresses with exclamation mark)
the idea is to create an Alias (Network(s) type) with subnets to exclude from other Aliases (blacklists) and create a merged table (Network group type alias) consisting of blacklist URLTable(IPs) and Exclude alias (eg use FireHOL level1 list "cutting" some private and "bulletproof " subnets from it).
I tried (add "!" sign processing in address validation in php and py files) and it works.
of course, there are some more subtleties: the order of records in the final table (exclusion shoud go first imo) , references check stops working in the GUI (pfctl -test works just fine)
but imho it would be great to add this feature to Aliases.
Thanks!!

just checked. yes, if exclusions is on botton of merged table they must be subsets of networks in blacklist or they will be ignored when loading in pf.
if exclusions is on top of merged table file they completly overcome networks in blacklist.
so to be shure that exclusions completly suppress networks from main table we need to sort merged table file ascending before load in pf.
(just to more strings in update_tables.py)

upd2: find_table_references.py find references of ip in tables by reading all tables and testing each line "if ip in IPNetwork"
why not to use perfect pfctl command "test"? and its exclusions aware

because it wouldn't perform due to context switches between your script and the pfctl command. (by the way our feature request template is https://github.com/opnsense/core/issues/new?assignees=&labels=&template=feature_request.md , which we really do prefer)

thanks again for quick responses!
oops. sorry, I believed that configd allows to do this..
about request template - sorry. im new at github. i used this template, read carefully all text and just delete it.
should I remake my request and not delete anyting from template? just paste text in relevant part?

we'll let it slip this time, just make sure you use the template next time.
It's easier to read if details are more structured (going from "what you want to achieve" to the proposed solution), some times there are other options as well which is easier to validate when the functional question is clear.

got it. thank you!

I misread your previous comment, my mistake, the function to check if a (one) address is matched can indeed be refactored like this, using test in the update tables likely will lead to performance issues.
Just open a PR and we'll work on it from there, looks like a good addition in my opinion.

Great!)
Sorry, what shoud i put in PR? just find_table_references.py with -Ttest or or all my experiments for aliases exclusions?

just start with a PR for find_table_references, since it's a single function call without a relation to other components. If you would like to work on the other piece as well, better do that in a separate PR.

https://github.com/opnsense/core/pull/4320
I don't think I should PR for the rest of the files. These are just test changes that works for me.
But I'm sure you will do it faster and more cleanly. If you suddenly decide to look at my changes, I can always show them here for a quick review.

@kulikov-a no worries, thanks for the PR

@kulikov-a https://github.com/opnsense/core/commit/fe25f69a07ec8175048640f880c7f5dfa5e2e45a should do the trick, maybe you want to improve our docs on the subject?
https://docs.opnsense.org/manual/aliases.html https://github.com/opnsense/docs/blob/6fa0e8da483f2c1e12067999f4cb1db1d718451c/source/manual/aliases.rst

cool! for https://github.com/opnsense/core/commit/fe25f69a07ec8175048640f880c7f5dfa5e2e45a
in php: i also changed validateHost function on my tests. isn't it necessary?
in py: i also allow "!" in address ranges. for now it will throw exception i think.
I got the idea. I'll try to add it to the docks)

well, I'm not sure if it adds anything, maybe it's easier to explain that this option is only available for networks (using either a netmask or a single host). just post a PR there, thanks in advance!

well, Im already using host exclusion for my remote branch offices public IPs (for bulletproofing). I suppose that might be useful.
Yes, host may be added in networks type Alias. I just guess that since Hosts and Networks Alias types allready there, it will be more intuitive.
im not using address ranges, but I also can imagine a situation when it might be useful (eg exclude remote branch public ip subnet but not isp routers or something like that).
in .php i just cutting "!" in the beginning of loop in two functions. in .py im cutting "!" in the beginning of parsing and check "isexclaim" variable whein yielding (and paste it back if its true)
gone to see into rst-format ..
thanks!

The use of the feature is easier to explain from the context of a network entry (all of <IP_ADDRESS>/24, without <IP_ADDRESS> for example). From a host perspective itself it doesn't really make sense, in which case you're forced to explain the relation with alias nesting.

Yeah) But Idea is indead in using nesting: big blocklist (eg firehol level1 which includes bogons) + excluded networks (to exclude some bogons) + excluded hosts (to protect against accidental blacklisting) so we can put blocking rule with this nested alias on top.
i was going to add text to Alias types descitpion, add some additional info with link on pf man and add example to Nesting paragraph

Hm. just realized our misunderstanding. yes, idea IS in nesting. not only an ability tо exclude subnet in same table.
that's why I talked about hosts and ranges with "!"

my PR for docs (for the case with nesting, hosts and subnets Aliases support): https://github.com/opnsense/docs/pull/283

@kulikov-a I'll add the validation to the hosts as well and close this issue, thanks for working on the docs!

great!
sorry. one more thing. as i mentioned in second comment, when pfctl load table (pfctl -Treplace) order of exclusions is vital (if exclusion goes after included matching subent, eg {<IP_ADDRESS>/24, !<IP_ADDRESS>/24}) pfctl will drop exclusion record (in this case exclusion subnet must be a subset of included network). so merged (nesting) table shoud be ordered asc and exclusion records will be at the top of file. if i understand ir right update_table.py does not guarantee this for now (am I right?). so for my test i just call
aliasfile = ('/var/db/aliastables/%s.txt' % alias_name)  sp = subprocess.run(['sort', aliasfile, '-o', aliasfile])
before calling  'pfctl -Treplace'.
but im not shure that this is right method to sort table file before loading.

@kulikov-a are you sure the order is relevant? It didn't look like it when testing on my end (didn't see it in the docs either), if it does matter, we probably need to change the sorting to descending here (or use a lambda to push all ! to bottom):
https://github.com/opnsense/core/blob/abcb23703fd2d534e6b5ba6f9c538760e35a3e3e/src/opnsense/scripts/filter/update_tables.py#L141
(alias_content  contains all entries for the alias including the ones resolved via nesting, so there's no real difference between a nested alias or a single one from pf's perspective)
Can you check if the current state really doesn't work on your end?

yes, based on my test. i manualy placed "!" strings on top or on bottom of table-file and load tables with pfctl -Treplace. pfctl shows how many strings imported and total strings count. also in both variant after table loading i tested IPs from excluded networks via pfctl -Ttest.  Results was different (fully matched strings ignored regardless of "!". pfctl load first and ignores second). This and https://www.freebsd.org/cgi/man.cgi?pfctl (search for "duplicated " word on page) makes me think that pfctl -Treaplce reads table-file sting by string an do some inside magic (check conflict, check duplicates) and in this case order is very relevant.
I tested original update_tables.py and "!" is always on top. thats what we need but i could not understand why it happends..
I thought that the "alias_content_txt = '\n'.join(sorted(alias_content))" sort only current table in loop and join it to the existing content (previosly added related aliases). so result  will depend on aliases order in ("for alias in aliases"). and i just add final file sorting.
now you point me, i look at file again, see "for related_alias_name in aliases.get_alias_deps(alias_name)" loop and understand my mistake.
Sorry!!

@kulikov-a no problem, better to check again than to find out our new feature is only partially functional. Thanks again for your help.


⚠ Bofrost.es has degraded performance

In 1caf768, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1926 ms

Resolved: Bofrost.es performance has improved in 2d44519 after 6 minutes.


Tests pick advertised services from the network and fail expectations

E.g. here
discoverableServices.findServices [ '_ssh._tcp' ], 6000, (error, services) ->
expect(services.length).to.equal(2)

it appears that bonjour picks up the local ResinOS Rpi3 and my macbook as well, so services.length is not the expected 2.

Update for accuracy, I'm not sure if it picked the rpi on the network, but it surely does pick other advertised services from the same workstation, i.e. my macbook:
Found services (3):
 [ { host: 'macbook', service: '_ssh._tcp', name: 'Private SSH' },
  { host: 'macbook', service: '_ssh._tcp', name: 'Main SSH' },
  { host: 'macbook-2.local', service: '_ssh._tcp', name: 'macbook (2)' } ]


Update test versions & add try/finally to docs

Also added some fancy python syntax highlighting to the docs which of course you can see here: https://github.com/MontmereLimited/xvfbwrapper

Thanks.. this looks great! :+1:


CAMEL-20127: Add messageAs function to simple language

Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

test failure not related to this PR


feat: Update presentation Preview and credential shared screen

https://user-images.githubusercontent.com/7862595/209066292-d4a0c320-6bee-4c86-8ec4-216ffb4c7182.mp4
https://user-images.githubusercontent.com/7862595/209066314-653edbc4-7580-49f9-a60f-394e196a8af6.mp4
Signed-off-by: Talwinder kaur<EMAIL_ADDRESS>

Codecov Report
Base: 91.08% // Head: 91.08% // No change to project coverage :thumbsup:

Coverage data is based on head (dc2dd1e) compared to base (e285cb8).
 Consider uploading reports for the commit 6b378db to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #130   +/-   ##
=======================================
  Coverage   91.08%   91.08%
=======================================
  Files          36       36
  Lines        1200     1200
=======================================
  Hits         1093     1093
  Misses         64       64
  Partials       43       43

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Gitkraken doesn't use the macOS setting for double-clicking a window title bar

Version: 7.7.2
Platform: macOS 11.4

When "Double click a window's title bar to..." is set to "Minimize" in macOS, double clicking the gitkraken title bar does not minimize the window - instead it resizes it. This is particularly jarring when every other window minimizes.

Ideally, gitkraken should respect the macOS setting, or allow the behaviour to be set independently in the gitkraken preferences.

Wrong bug tracker, I assume?


Less dropped followers

Scope & Purpose
Remove a case in which followers were dropped unnecessarily in streaming transactions that replicated to the same follower for multiple shards.
This is a backport of https://github.com/arangodb/arangodb/pull/13136

[x] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Backports:

[x] Backports required for: 3.6

Testing & Verification

[x] The behavior in this PR was manually tested
[x] This PR adds tests that were used to verify all changes:

[x] Added new integration tests (e.g. in shell_server / shell_server_aql)

Link to Jenkins PR run:
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/12978/

Obsoleted by https://github.com/arangodb/arangodb/pull/12987


SuperTextField does not seem to call buildTextSpan. How to add a widget inline?

A TextEditingController can be subclassed to override buildTextSpan in order to build a custom TextSpan (i.e. icon before or after the text).
When I subclassed AttributedTextEditingController and added the instance to SuperTextField, I noticed that buildTextSpan was not called. When looking closer into the code, it does not seem that AttributedTextEditingController calls buildTextSpan at all.
How should one go about prepending/appending a widget inline with SuperTextField?

Are we definitely talking about SuperTextField? We had a recent discussion about this behavior with text components in an editor. So you're looking for inline widgets in SuperEditor and SuperTextField?

@matthew-carroll Yes, these are two separate requests.
With regard to SuperTextField, it was determined that when AttributedText is extended with an instance passed into AttributedTextEditingController, then computeTextSpan of extended AttributedText can be overriden.
class MyAttributedText extends AttributedText ....

MyAttributedText attributedText = MyAttributedText('MyText');
AttributedTextEditingController controller = AttributedTextEditingController(text: attributedText)


ddev drush needs to support php 8.1 to avoid Deprecated: preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated in /var/www/html/vendor/consolidation/site-alias/src/SiteSpecParser.php on line 144

Is there an existing issue for this?

[X] I have searched the existing issues

Run a Diagnostic and Paste Link Here
ddev version v1.21.3
robdavisprojects@Workstation-W64:~/Work/probonoclients/internationalgospelchoir.uk/sites/ligc-dev$ ddev debug test
Running bash [-c /tmp/test_ddev.sh]
OS Information: Linux Workstation-W64 4.19.104-microsoft-standard #1 SMP Wed Feb 19 06:37:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
User information: uid=1000(robdavisprojects) gid=1000(robdavisprojects) groups=1000(robdavisprojects),4(adm),20(dialout),24(cdrom),25(floppy),27(sudo),29(audio),30(dip),44(video),46(plugdev),117(netdev),1001(docker)
DDEV version:  ITEM             VALUE
DDEV version     v1.21.3
architecture     amd64
db               drud/ddev-dbserver-mariadb-10.4:v1.21.3
dba              phpmyadmin:5
ddev-ssh-agent   drud/ddev-ssh-agent:v1.21.3
docker           20.10.21
docker-compose   v2.10.2
docker-platform  docker-desktop
mutagen          0.16.0
os               linux
router           drud/ddev-router:v1.21.3
web              drud/ddev-webserver:v1.21.3
PROXY settings: HTTP_PROXY='' HTTPS_PROXY='' http_proxy='' NO_PROXY=''
======= DDEV global info =========
Global configuration:
instrumentation-opt-in=true
omit-containers=[]
mutagen-enabled=false
nfs-mount-enabled=false
router-bind-all-interfaces=false
internet-detection-timeout-ms=3000
disable-http2=false
use-letsencrypt=false
letsencrypt-email=
table-style=default
simple-formatting=false
auto-restart-containers=false
use-hardened-images=false
fail-on-hook-fail=false
required-docker-compose-version=
use-docker-compose-from-path=false
no-bind-mounts=false
project-tld=
======= DOCKER info =========
docker location: lrwxrwxrwx 1 root root 48 Nov 14 11:46 /usr/bin/docker -> /mnt/wsl/docker-desktop/cli-tools/usr/bin/docker
docker version:
Client: Docker Engine - Community
Cloud integration: v1.0.29
Version:           20.10.21
API version:       1.41
Go version:        go1.18.7
Git commit:        baeda1f
Built:             Tue Oct 25 18:02:28 2022
OS/Arch:           linux/amd64
Context:           default
Experimental:      true
Server: Docker Desktop
Engine:
Version:          20.10.21
API version:      1.41 (minimum version 1.12)
Go version:       go1.18.7
Git commit:       3056208
Built:            Tue Oct 25 18:00:19 2022
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.6.9
GitCommit:        1c90a442489720eec95342e1789ee8a5e1b9536f
runc:
Version:          1.1.4
GitCommit:        v1.1.4-0-g5fd4c4d
docker-init:
Version:          0.19.0
GitCommit:        de40ad0
DOCKER_DEFAULT_PLATFORM=notset
======= Mutagen Info =========
======= Docker Info =========
Docker platform: docker-desktop
Using docker context: default (unix:///var/run/docker.sock)
docker-compose: v2.10.2
Using DOCKER_HOST=unix:///var/run/docker.sock
Docker version: 20.10.21
Able to run simple container that mounts a volume.
Able to use internet inside container.
Docker disk space:
Filesystem                Size      Used Available Use% Mounted on
overlay                 251.0G     12.7G    225.4G   5% /
Container ddev-ligc-develop-dba  Removed
Container ddev-ligc-develop-db  Removed
Container ddev-ligc-develop-web  Removed
Network ddev-ligc-develop_default  Removed
Project ligc-develop has been stopped.
The ddev-ssh-agent container has been removed. When you start it again you will have to use 'ddev auth ssh' to provide key authentication again.
Existing docker containers:
CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES
Creating a new ddev project config in the current directory (/home/robdavisprojects/Work/probonoclients/internationalgospelchoir.uk/sites/tryddevproject-5266)
Once completed, your configuration will be written to /home/robdavisprojects/Work/probonoclients/internationalgospelchoir.uk/sites/tryddevproject-5266/.ddev/config.yaml
Configuring unrecognized codebase as project type 'php' at /home/robdavisprojects/Work/probonoclients/internationalgospelchoir.uk/sites/tryddevproject-5266/web
Configuration complete. You may now run 'ddev start'.
Network ddev_default created
Starting tryddevproject-5266...
Container ddev-ssh-agent  Started
ssh-agent container is running: If you want to add authentication to the ssh-agent container, run 'ddev auth ssh' to enable your keys.
 You will want to delete this project with 'ddev delete -Oy tryddevproject-5266'
Thanks for running the diagnostic. It was successful.
Please provide the output of this script in a new gist at gist.github.com
Running ddev launch in 5 seconds
If you're brave and you have jq you can delete all tryddevproject instances with this one-liner:
ddev delete -Oy $(ddev list -j |jq -r .raw[].name | grep tryddevproject)
In the future ddev debug test will also provide this option.
Please delete this project after debugging with 'ddev delete -Oy tryddevproject-5266'
Expected Behavior
ddev drush works as normal
Actual Behavior
ddev drush gives fatal error
Deprecated: preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated in /var/www/html/vendor/consolidation/site-alias/src/SiteSpecParser.php on line 144
PHP Deprecated:  preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated in /var/www/html/vendor/consolidation/site-alias/src/SiteSpecParser.php on line 144
Steps To Reproduce
Run ddev drush in usual ddev setup
Anything else?
I fixed this manually, making use of the solution here: https://github.com/consolidation/site-alias/issues/48

But the ideal solution is that this needs to be fixed in the drush that runs inside ddev docker containers.

This can also be solved by deleting .ddev folder and also deleting the ddev project and then running ddev config.
After which, I edited config.yml to have php at version 8.1. Which is the same version as on Ubuntu 22.04 by default which is my production server.

This is not a DDEV issue.
DDEV does not install or manage Drush.
Steps

Configure DDEV
ddev config --project-type=drupal9 --docroot=web --create-docroot --php-version=8.1

Install Drupal
ddev composer create "drupal/recommended-project" -y

Install Drush
ddev composer require drush/drush --dev

Use Drush to configure Drupal
ddev drush si --site-name=drush-ddev-debug -y

Confirm site works

You most likely had the wrong project type, ddev config detected the correct project type.

composer update drush/drush -W fixed it for me.

composer update drush/drush -W
did not make any difference
still having the same issue

Hi @la558 - Please use ddev composer.  And make sure your .ddev/config.yaml has php 8.1 in it.

Thanks for your response.
I've installed ddev :
ddev version v1.21.6
Just, so you know, I'm NOT running my Drupal site inside Docker.
Does your recommendation still apply?
When you say to run:

ddev composer

you mean:

ddev composer update consolidation/* --with-dependencies

Best,
--
Laura Addiego Prospero  |  Software Application Developer - Software
Engineer
GEORGETOWN LAW  | @.***
600 New Jersey Avenue, N.W.  |  Washington, D.C. 20001
Phone:<PHONE_NUMBER>
Website http://www.law.georgetown.edu/ |  Facebook
https://www.facebook.com/georgetownlaw/  |  Twitter
https://twitter.com/georgetownlaw  |  YouTube
https://www.youtube.com/user/GeorgetownLawChannel  |  LinkedIn
https://www.linkedin.com/edu/georgetown-university-law-center-18052  |
Instagram https://www.instagram.com/gtownlaw/
On Wed, Mar 29, 2023 at 10:47 AM Randy Fay @.***> wrote:

Hi @la558 https://github.com/la558 - Please use ddev composer. And make
sure your .ddev/config.yaml has php 8.1 in it.
—
Reply to this email directly, view it on GitHub
https://github.com/ddev/ddev/issues/4382#issuecomment-1488769744, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHWQ2W7X2VFRTOZ4OCZU6DTW6RDRVANCNFSM6AAAAAASANNUFQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

FYI - I'm running consolidation/site-alias version 4.0.0
"name": "consolidation/site-alias",
"version": "4.0.0",
This is what my current function looks like, and it's giving me the error:
[image: Screen Shot 2023-03-29 at 11.30.24 AM.png]
Best,
--
Laura Addiego Prospero  |  Software Application Developer - Software
Engineer
GEORGETOWN LAW  | @.***
600 New Jersey Avenue, N.W.  |  Washington, D.C. 20001
Phone:<PHONE_NUMBER>
Website http://www.law.georgetown.edu/ |  Facebook
https://www.facebook.com/georgetownlaw/  |  Twitter
https://twitter.com/georgetownlaw  |  YouTube
https://www.youtube.com/user/GeorgetownLawChannel  |  LinkedIn
https://www.linkedin.com/edu/georgetown-university-law-center-18052  |
Instagram https://www.instagram.com/gtownlaw/
On Wed, Mar 29, 2023 at 11:19 AM Laura @.***> wrote:

Thanks for your response.
I've installed ddev :
ddev version v1.21.6
Just, so you know, I'm NOT running my Drupal site inside Docker.
Does your recommendation still apply?
When you say to run:

ddev composer

you mean:

ddev composer update consolidation/* --with-dependencies

Best,
--
Laura Addiego Prospero  |  Software Application Developer - Software
Engineer
GEORGETOWN LAW  | @.***
600 New Jersey Avenue, N.W.  |  Washington, D.C. 20001
Phone:<PHONE_NUMBER>
Website http://www.law.georgetown.edu/ |  Facebook
https://www.facebook.com/georgetownlaw/  |  Twitter
https://twitter.com/georgetownlaw  |  YouTube
https://www.youtube.com/user/GeorgetownLawChannel  |  LinkedIn
https://www.linkedin.com/edu/georgetown-university-law-center-18052  |
Instagram https://www.instagram.com/gtownlaw/
On Wed, Mar 29, 2023 at 10:47 AM Randy Fay @.***>
wrote:

Hi @la558 https://github.com/la558 - Please use ddev composer. And
make sure your .ddev/config.yaml has php 8.1 in it.
—
Reply to this email directly, view it on GitHub
https://github.com/ddev/ddev/issues/4382#issuecomment-1488769744, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHWQ2W7X2VFRTOZ4OCZU6DTW6RDRVANCNFSM6AAAAAASANNUFQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

If you're not using DDEV, you're in the wrong place. If you're running Drupal somewhere other than in ddev/docker, why are you asking here?

After updating to PHP 8.1.17,
Whenever I run any drush command, I get:
PHP Deprecated:  preg_match(): Passing null to parameter #2 ($subject) of
type string is deprecated in
<path_to_drupal_site>/vendor/consolidation/site-alias/src/SiteSpecParser.php
on line 144
in the 'composer.lock' file states the url for support > issues
"support": {

"issues": "https://github.com/consolidation/site-alias/issues",
"source": "https://github.com/consolidation/s"
},

Best,
--
Laura Addiego Prospero  |  Software Application Developer - Software
Engineer
GEORGETOWN LAW  | @.***
600 New Jersey Avenue, N.W.  |  Washington, D.C. 20001
Phone:<PHONE_NUMBER>
Website http://www.law.georgetown.edu/ |  Facebook
https://www.facebook.com/georgetownlaw/  |  Twitter
https://twitter.com/georgetownlaw  |  YouTube
https://www.youtube.com/user/GeorgetownLawChannel  |  LinkedIn
https://www.linkedin.com/edu/georgetown-university-law-center-18052  |
Instagram https://www.instagram.com/gtownlaw/
On Wed, Mar 29, 2023 at 11:43 AM Randy Fay @.***> wrote:

If you're not using DDEV, you're in the wrong place. If you're running
Drupal somewhere other than in ddev/docker, why are you asking here?
—
Reply to this email directly, view it on GitHub
https://github.com/ddev/ddev/issues/4382#issuecomment-1488867298, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHWQ2W4P7ZZ5XY6D4J3HHYDW6RKB5ANCNFSM6AAAAAASANNUFQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Please go to Drupal.org or wherever you want, but if you're not using DDEV to run Drupal, you're in the wrong place.


[Fix](Lateral View) Error view includes lateral view

Proposed changes
Issue Number: close #9529
Problem Summary:
Fixed #9529
When the lateral view based on a inline view which belongs to a view,
Doris could not resolve the column of lateral view in query.
When a query uses a view, it mainly refers to the string representation of the view.
That is, if the view's string representation is wrong, the view is wrong.
The string representation of the inline view lacks the handling of the lateral view.
This leads to query errors when using such views.
This PR mainly fixes the string representation of inline views.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

fe CheckStyle error need to be fixed


Add a getter for DataReader on DefaultExtractorInput

Add a getter for DataReader on DefaultExtractorInput

I'm going to send this for internal review now. You may see some more commits being added as I make changes in response to review feedback. Please refrain from pushing any more substantive changes as it will complicate the internal review - thanks!


Challange questions ID  can be given every qestions in the set and it is not unique

Affected: 5.8.0-Alpha2
Severity: Major
Priority: Normal
Component: User Mgt
Type: Bug
According to the document [1] Challenge Question Id field allows you to enter the question Id of the first question in the set. But here its allows to enter question ID in every questions and can duplicate the Question ID within the set.
[1] https://docs.wso2.com/display/IS580/Managing+Challenge+Questions#ManagingChallengeQuestions-Addingachallengequestionset

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Add help to keep fork in sync with upstream

I added few lines in the README.md to help with keeping in sync with the original repo.

Thank you

@all-contributors please add @Edmund1645 for code


just a couple of functions I would love

This package is great, just a couple of things I miss:

console should have timestamps of when an operation occurred

settings should allow for the console to automatically open during an operation. alternatively, there should be some statusbar message with a longer duration. it feels weird to have no feedback at all when I glimpse down.

(extra) this is probably just me, I just couldn't find info about keyboard shortcuts? would love being able to upload with a keystroke.

this is all, as I said a great package, fantastic work.

Hi,
In reverse:
3) for shortcuts and other info checkout https://github.com/NoxArt/SublimeText2-FTPSync/wiki/Key-bindings and other Wiki entries

perhaps, not sure Sublime's API allows it but even if so, I can imagine incessant opening of the console would become very annoying ... FTPSync does output some info right into the small status bar, though it's not persistent

this sounds like something that could be handled by Sublime itself...

... but true, the plugin could do it too, I'm very sorry though as I no longer have time to improve FTPSync, and it's not that fantastic, should have been better, but hopefully you will find it useful in the current version anyway


[CLOSED] Add leading slash to slugs when sending to Rummager

 Issue by jennyd
Thursday 13 August 2015 at 13:24 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/537

This adds the leading slash to the slug for the id and link fields when formatting manuals, manual sections and specialist documents to send to Rummager (finders already use the full base_path) and adds a script to migrate the existing documents in Rummager to the new format.
See the commit messages for the full details.
A temporary workaround for the broken links due to missing leading slashes has recently been deployed in Rummager, but we need to fix the data at source. This issue is fixed for HMRC manuals in a PR on the API.

jennyd included the following code: https://github.com/alphagov/specialist-publisher/pull/537/commits

 Comment by mikejustdoit
Friday 14 August 2015 at 10:52 GMT

Excellent pull request


Feat: add scheduling problems

Add Job Shop problems with optimal value, based on the makespan metric, and valid/invalid solutions.
The branch in based on the val one.

Hi @arbimo,
Is it ok for you to proceed with the merge?


i18n Postal

Hello Folks from Postal ... This App is Awesome!
Can we make an effort to translate Postal? Do you have some guidelines to do this with your App?
Our team can translate Postal, but we want to know your plans ....
Very Very Thanks!!!

@mikecmpbll  @adamcooke Hello Guys make i18n will take a Lot of time ... can I make an pull for each module or one pull after all ?

One pull request to move the English language fragments to that locale file and then each additional language as its own pull request is probably enough separation in this case.
The initial moving of the language fragments would be the biggest one and it looks like there are a few unnecessary terms kept in the current en locale file.


Arc meter customizations sometimes render incorrectly

Some arcs in Figma that are setup with the arc meter customization with our Dials and Gauges plugin is rendered incorrectly. This seems to happen with flipped axes, negative sweeps, or a combination of both.

Could you provide reproduction steps? Perhaps a small Figma file that shows the issue?

I don't know how to reproduce this exactly, but it was happening in our Cluster design. I suspect it has to do with negative sweeps and flipped axes because that's what was happening in the Cluster, but this never happened in any of the arcs in the dial & gauges test doc. The reproduction steps are to enable and arc for meter customization and change the sweep parameters and flip the axes until this happens.

I believe this only happens when the arc has a fill. When there is a fill and it has a meter customization, the code takes a different path where it manually draws the outline of the arc with a bunch of arcs and bezier curves.


rom-sql 1.3.2 regression

It seems that rom-sql 1.3.2 is a lot slower than the previous version 1.3.1.
We have a project that has around 1500 spec scenarios which usually take around 1 and a half minute to complete, but since we updated to 1.3.2, it now takes 6-7 minutes 😢
I've put an example here which is not as bad as our specs but you can still see that it's slower.
#!/usr/bin/env ruby
require 'bundler/inline'
require 'benchmark'

gemfile do
  source 'https://rubygems.org'
  gem 'rom'
  gem 'rom-sql', "=1.3.1" #change this to 1.3.2
  gem 'rom-repository'
  gem 'dry-types'
  gem 'sqlite3'
end

require 'rom-sql'
require 'rom-repository'
require 'dry-types'

module Types
  include Dry::Types.module
end

rom = ROM.container(:sql, 'sqlite::memory') do |c|
  c.gateways[:default].create_table :employees do
    primary_key :id, Integer
    column :name, String
  end

  c.gateways[:default].create_table :positions do
    primary_key :id, Integer
    column :manager_id, Integer
    column :participant_id, Integer
  end

  c.relation(:employees) do
    schema(:employees, infer: false) do
      attribute :id, Types::Int.meta(primary_key: true)
      attribute :name, Types::String

      associations do
        has_many :employees, as: :subordinates, through: :positions, foreign_key: :participant_id
      end
    end
  end

  c.relation(:positions) do
    schema(:positions, infer: false) do
      attribute :id, Types::Int.meta(primary_key: true)
      attribute :manager_id, Types::Int.meta(foreign_key: true)
      attribute :participant_id, Types::Int.meta(foreign_key: true)
    end
  end
end

jane = rom.relations[:employees].insert(name: "Jane")
fred = rom.relations[:employees].insert(name: "Fred")

class Positions < ROM::Repository[:positions]
  commands :create, :delete
end

position_repo = Positions.new(rom)
bench_result = Benchmark.measure do
  1000.times do
    20.times do
      position_repo.create(manager_id: jane, participant_id: fred)
      position_repo.delete
    end
  end
end

puts bench_result

Results
11.390000   0.050000  11.440000 ( 11.548515) - rom-sql 1.3.1
14.410000   0.110000  14.520000 ( 15.049306) - rom-sql 1.3.2

@gmartsenkov thanks for spotting this out, looks like I accidentally removed some optimizations. Could you run your app specs against this branch https://github.com/rom-rb/rom-sql/tree/fix-ds-regression?

@flash-gordon I work with @gmartsenkov and can confirm that this appears to have fixed things.
Finished in 1 minute 34.22 seconds (files took 4.3 seconds to load)
1570 examples, 0 failures, 3 pending

@mrship great, I'll push a new version by Monday

1.3.3 is on rubygems

cheers man!


Incorporate offsite items into our location model

Superseded by https://github.com/wellcomecollection/catalogue-api/issues/770


Allow UMAP to accept data with nan or inf values

Based on our discussion about nan-values and custom metrics, I finally had time to (hopefully) properly address the issue.
I exposed the 'force_all_finite' parameters of sklearn's check array functions to UMAP's data handling functions. The default is "force_all_finite=True" which means that this change shouldn't break anything. I also included a test case to test data inputs in respect to accepting/dismising nan's or inf's.
However, I DON'T check if the selected metric is fine with nan or inf values!

Sorry for taking so long to get to this. Thanks!

Don't worry ;)


Fix bug: we should start conversion webhook before initiating Longhorn datastore

longhorn/longhorn#8578
More details are at https://github.com/longhorn/longhorn/issues/8578#issuecomment-2121405795

Putting this PR in draft until doing some more testing and have better code style

@derekbit Can you review the PR again? It actually still needs a k8s client. Just do not need the Longhorn datastore.
The new modification will create 2 clients:

One without Longhorn datastore
One with Longhorn datastore.

I think it should not create a high memory usage issue since the one without Longhorn datastore doesn't have any cache. However, we should still double-check the test  related to multiple clients in Longhorn manager. I forgot the ticket number, do you remember it?

@derekbit Can you review the PR again? It actually still needs a k8s client. Just do not need the Longhorn datastore.
The new modification will create 2 clients:

One without Longhorn datastore
One with Longhorn datastore.

I think it should not create a high memory usage issue since the one without Longhorn datastore doesn't have any cache. However, we should still double-check the test related to multiple clients in Longhorn manager. I forgot the ticket number, do you remember it?

longhorn#6866?

LGTM
@PhanLe1010 do you mean this one? https://github.com/longhorn/longhorn/issues/6936

Yes, it is the test plan at https://github.com/longhorn/longhorn/issues/6936#issuecomment-1775396141
Thank you @ChanYiLin and @derekbit !

Testing looks good. It is ready to be merged after the reviewers approve

Looks like there is an new race condition arise after this PR. Please hold on merging

@PhanLe1010 is this still draft?

It turned out that we not only have problem with upgrade stuck but also the problem with uninstall stuck because we delete the longhorn-manager daemonset (AKA the conversion webhook server) then list and the the BackupTarget which is still in v1beta1 version. This flow will fail. To solve this problem:

A quick hack is https://github.com/longhorn/longhorn-manager/pull/2812/commits/ad321772848e34cc3074af69157bbe1f3eb15260
A better solution would be to migrate the API versions for all Longhorn resources in the upgrade path (thus eliminate the need for conversion webhook for future Longhorn upgrade) OR separating the conversion webhook from longhorn-manager

Due to time pressure, I think we can go with the first approach for now and do the 2nd approach in v1.7.0 . WYDT cc @derekbit @innobead @ChanYiLin @ejweber

Sounds good to me.

@mergify backport v1.6.x v1.5.x


helmfile --version does not report version

I installed helmfile with go get (the first recommended installation method), and when I run helmfile --version
I get version as version.
$ helmfile --version
helmfile version

Good catch!
We use ldflags to embed the version number into go binary, so in theory the go get command should include ldflasg like `go get -u -ldflags "-X github.com/roboll/helmfile.Version=x.y.z".
However I'm not sure how it should work on master...


[sql lab]Add autoSelect

In sql editor database dropdown list, we want to auto-select the first available option.
fixes https://github.com/airbnb/superset/issues/2463

LGTM!


Link to a non-gx package fails

(Edit: I'm rewriting this issue as I did a poor attempt at explaining it the last time.)
When trying to link to a package that doesn't support gx (i.e., it doesn't contain a package.json) the gx-go link command will fail while trying to install it (with gx install).
cd $GOPATH/src/github.com/ipfs/go-ipfs
make install

go get github.com/dgraph-io/badger
export BADGER_HASH=QmdKhi5wUQyV9i3GcTyfUmpfTntWjXu8DcyT9HyNbznYrn
gx-go link $BADGER_HASH
# ERROR: Error: error during gx install: exit status 1

# Temporary workaround: copy `package.json` from the gx release to the cloned repo
wget https://ipfs.io/ipfs/$BADGER_HASH/badger/package.json -O $GOPATH/src/github.com/dgraph-io/badger/package.json
gx-go link $BADGER_HASH

I'm seeing that when dependencies are installed with gx in IPFS even the external not-gx-aware ones have a package.json, so the issue here may be that some other intermediary step may be missing between the go get call and gx install.

ooooh, yeah, thats annoying... Maybe we should have gx-go link detect that the target package doesnt have the package.json and perform that step for you automatically.


Ensure isHidden is set on main thread

I noticed the progress could be updated from a background thread. Toggling isHidden without switching to the main thread will throw warnings in the console.

Should we document that all the properties can be called/assigned in any thread? That might not be obvious.

Should we document that all the properties can be called/assigned in any thread? That might not be obvious.

🤔 I haven't tested all properties. I think the only property that we should take responsibility over is .progress as it relies on .fractionCompleted from the assigned Progress object.
We could add a note that it can be called from a background thread.

Yes, document that.


standalone creation of riz signatures

Is it possible / this is a request for enhancement / to create riz signatures without having to load up IDA?
Basically the idea is to be able to create many signatures on systems without having to go the way to singlehandedly every file.
e.g. create riz-signatures of all my linux so files.

Short answer is no, rizzo can't be used without IDA; it relies on IDA to do all the disassembly, function identification, xrefs, etc. that it needs to create signatures. Certainly this could be done with some other disassembly library (e.g., the Capstone library), but that would be a completely different tool.
However, you can run IDA in batch mode for automated processing of many files. In batch mode, you can tell IDA to run an IDAPython script, and that script can automatically invoke rizzo, something like this for example:
import idc
import idaapi
import rizzo

# Wait for IDA to finish auto-analysis
idaapi.autoWait()
# Generate rizzo signatures, save to /tmp.output.riz
rizzo.RizzoBuild('/tmp/output.riz')
# Close IDA
idc.Exit(0)

Typically, you'd invoke IDA on the command line with something like:
$ idal -A -S myscript.py libuClibc.so

You can then just have a shell script loop through all the files you want to generate signatures for.
For more information on running IDA in batch mode, see:
https://www.hex-rays.com/products/ida/support/idadoc/417.shtml
http://www.hexblog.com/?p=53
http://www.codejury.com/idas-batch-mode/


BM In Silico Mix

Changelog:

Very small update to add In Silico Mix tab

 Add In Silico Mix to COLO829 Benchmarking page


914 [Feature] - "Add remaining CRUD endpoints to SST"

Added remaining trainspace CRUD endpoints to SST
What user problem are we solving?
Trainspace creation, deletion, management, and deletion of user uploaded datset.
What solution does this PR provide?
Allowing users to create new, retrieve specific, retrieve all, delete specific trainspace, and delete specific user uploaded dataset using AWS Lambda.
Testing Methodology
Tested API endpoints on locally with postman using a developer account.
Created new trainspace, retrieved it, created several new trainspaces, retrieved all, deleted trainspace, attempted (and successfully failed) to retrieve previously deleted trainspace, and deleted user uploaded data successfully.
Any other considerations

More thorough testing may be needed for deleting datasets for S3 buckets, haven't checked false negative case. No exceptions were thrown however.
May need to improve exception messages, and exception detection.

Sept 23rd. Latest

also, can you attach a screen recording to show that the api endpoint works and data is updated on trainspace dynamodb correctly?

https://youtu.be/aMiK-I8cpqs

@alantao912 if you've addressed all the PR comments and tested your changes using the method outlined in your video, let me know and I can mark your PR as approved to unblock

@alantao912 there are some merge conflicts to be resolved?

@alantao912 can you fix the merge conflicts and also test the endpoints?

@alantao912 can you fix the merge conflicts and also test the endpoints?

No merge conflicts. Fast-forwarded and pushed.

@alantao912 can you fix the merge conflicts and also test the endpoints?

No merge conflicts. Fast-forwarded and pushed.

did you test the endpoints @alantao912

@karkir0003 Hey, narrowed permissions on the SST endpoints and tested with the same method as before. Should be good to go.

I think it would be better if each trainspace type was handled by it's own crud api endpoints rather than having a conglomerate trainspace crud endpoints. Our current way of approaching this seems to be leading to the massive switch statement, giant functions, etc, and from looking at the current implementation, it seems like each trainspace type seems to have fundamentally different needs when it comes to implementing the CRUD endpoints for them. I think a CRUD api endpoint set each for tabular, image, classical, etc would be better. @karkir0003 what do you think

fair point @dwu359 but what would the diff between these crud endpoints look like?

I think it would be better if each trainspace type was handled by it's own crud api endpoints rather than having a conglomerate trainspace crud endpoints. Our current way of approaching this seems to be leading to the massive switch statement, giant functions, etc, and from looking at the current implementation, it seems like each trainspace type seems to have fundamentally different needs when it comes to implementing the CRUD endpoints for them. I think a CRUD api endpoint set each for tabular, image, classical, etc would be better. @karkir0003 what do you think

fair point @dwu359 but what would the diff between these crud endpoints look like?

A set of CRUD endpoints for tabular, a set of CRUD endpoints for image, a set of CRUD endpoints for classical ml. Basically just a separation of concerns

like treat trainspace types differently? dont we already segregate tabular train from say image train

@alantao912 thoughts?

what makes img trainspace different from say a tabular trainspace?

Yes. Although all trainspaces are still stored in the same trainspace table. We would still need a get all trainspaces endpoint for the dashboard and a delete trainspace by id for deleting from the dashboard. So I guess it's just the create endpoints that need to get separated then

what makes img trainspace different from say a tabular trainspace?

Mainly different json structures, which may require different ways of handling such data

@alantao912 thoughts?

I don't think there would need to be a complete set of new CRUD endpoints for each type of trainspace. There only need to be different handling for creating new trainspaces. Deletion, reading, should not vary between data sources.

You're quite behind the nextjs branch. Just fix the conflicts and you should be good to go 👍 @alantao912


ONNX export: Add Crop, Deconvolution and fix the default stride of Pooling to 1

Description
Add operators to ONNX exporter:

Crop
Deconvolution
Fix pooling default stride to 1.

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@ptrendx Did you get a chance to test these operators with ONNX? onnx_backend_test.py

@ptrendx Thanks for submitting this PR. :)
You should add 'test_ConvTranspose2d' in "Basic Model Tests" to make sure "Deconvolution" operator is running fine in this file. https://github.com/apache/incubator-mxnet/blob/master/tests/python-pytest/onnx/export/onnx_backend_test.py

@ptrendx - ping :-)

ONNX 1.3 has been merged. Please rebase this PR. Thanks.

@ptrendx Can you please address review comments and rebase this PR? Thanks.

Hi, I'm sorry for lack of communication on my part. Unfortunately I got pulled into other high priority work and will be able to revisit this PR afterwards.

Hi @ptrendx , were you able to get some time on this PR? Do you need any help to continue with this?

Hi, I'm sorry for the long delay in answering the comments. It's rebased onto the latest master.

@ptrendx Can you address the review comments so that we can merge this PR? :)

@ptrendx Can you please rebase this PR?


Slicing with broadcastable subarray fails

Currently slicing with a single array, which could be broadcasted fails. If this array is broadcasted, things work without issues
What happened:
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)
<ipython-input-4-48d1ef76a37c> in <module>
----> 1 a[b]

~/Developer/dask/dask/array/core.py in __getitem__(self, index)
   1721             self, index2 = slice_with_int_dask_array(self, index2)
   1722         if any(isinstance(i, Array) and i.dtype == bool for i in index2):
-> 1723             self, index2 = slice_with_bool_dask_array(self, index2)
   1724
   1725         if all(isinstance(i, slice) and i == slice(None) for i in index2):

~/Developer/dask/dask/array/slicing.py in slice_with_bool_dask_array(x, index)
   1159         isinstance(ind, Array) and ind.dtype == bool and ind.ndim != 1 for ind in index
   1160     ):
-> 1161         raise NotImplementedError(
   1162             "Slicing with dask.array of bools only permitted when "
   1163             "the indexer has only one dimension or when "

NotImplementedError: Slicing with dask.array of bools only permitted when the indexer has only one dimension or when it has the same dimension as the sliced array

What you expected to happen:
Broadcasting is handled for us
Minimal Complete Verifiable Example:
import dask
import dask.array as da

import numpy as np

s = (2, 2, 4, 5)
a = da.arange(int(np.prod(s))).reshape(s)
b = da.array([[True, False], [True, True]])

a[da.broadcast_to(b[..., None, None], shape=a.shape)]  # <--- works

a[b]                                                   # <--- fails

Anything else we need to know?:
Derived from comment ( https://github.com/data-apis/array-api/issues/84#issuecomment-818289677 )
cc @asmeurer
Environment:

Dask version: 2021.4.0
Python version: 3.8.8
Operating System: macOS
Install method (conda, pip, source): Conda

Thought we had an issue like this previously that was resolved, but am not finding it atm. This to say it may be a dup or regression potentially (though I could be wrong about that)

An aside, it's confusing to me at least when both you and dask call indexing "slicing" when the index in question isn't a slice. The error message here is also written confusingly, so that it's hard to tell if it's referring to a or b, for a[b].


having problem in inference quad

there are no offset_h and offset_w in utils.preprocess_image but they  used in inference_quad.py?!!

just remove them

@northeastsquare nothings happened when i remove them my result its like there is no NMS on image

there are no offset_h and offset_w in utils.preprocess_image but they used in inference_quad.py?!!

Did you figure this out? I am having this problem.


FR: Support query operations that other Firebase libraries support

[REQUIRED] Step 2: Describe your environment

Library version: 7.1.1
Firebase Product: firestore

[REQUIRED] Step 3: Describe the problem
I don't seem too have access to all of the same firestore query syntax  as I have in other languages. i.e. I can't do a not equals query, but firestore definitely supports this.

It is supported in https://github.com/googleapis/java-firestore which means we do too. You might have to upgrade the google-cloud-firestore dependency installed into your project/app.


CxxNativeModule::invoke is copying parameters that are meant to be moved

Description:
The CxxNativeModule::invoke function makes use of a lambda to run a native module function on the message queue thread. However the lambda isn't marked as mutable -- this prevents the params capture from being movable when calling the method itself. If the params argument is expensive to copy, this can impact performance.
React Native version:
This can be seen in v0.62.
Steps to reproduce (and expected results) are not applicable (unless running with a profiler).

Thanks for bringing this to our attention. Do you think you can send a PR to address this?

I was going to, but unfortunately getting the CLA signed is too high a bar :(

Oh, sorry to hear that. Are there any particular concerns?

I could look into this @hramos

@hramos just that I'd need to get it signed on behalf of the company I work for, which means lawyers and all that. It's been talked about before so I'm sure we'll get it done at some point
@maschad thanks! 👍


added fix for smoke run

Signed-off-by: Tarun Kumar<EMAIL_ADDRESS>

https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/1166/console

Hello @tarukumar, I'm wondering why this keyword now receives two arguments?
dashboard_title vs dashboard_titles seems a bit confusing to me ... ?
nb: I understand the reason of the fix, I had this failure as well


featureCount --minOverlap seems not functioning

From the IGV, I have seen one read only has 1bp overlap with the feature I am interested in. So, without setup any integer value to the --minOverlap, the output count report 1 read for that feature. However, even after I change the --minOverlap to 5 or 10, the output count file still reported 1 read for that feature, which should be filtered out.
Does this meant that the --minOverlap in featureCount is not functioning.
Thank you!
Yiyi

Yiyi;
Thanks much for the report. Can you provide a bit more context about how you're making use of featureCounts in bcbio and tweaking the minOverlap parameter? Your description sounds right in that is what --minOverlap is meant to do, but it strikes me that maybe the parameter is not applied in bcbio as intended depending on how specify it.

Closing, as this is stale and there wasn't any followup.


Upgrade ETH and uint256 dependencies

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1398  👈
#1397
#1396
main

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

Join @lazynina and the rest of your teammates on  Graphite


GPL Dependency

In version 9.3.1, flyway-core introduces a dependency on com.electronwill.night-config:toml. This is a GPL (v3) dependency, which contaminates the license dependency tree. Our project is licensed as Apache 2 and part of the FINOS foundation for open source projects in financial services, so we absolutely cannot introduce GPL dependencies. Other open source projects releasing under a permissive license will have similar issues due to license incompatibility.
Is this a dependency you can remove and replace in a future version? For now we have stuck on 9.3, but our compliance requirements mean we cannot stick on old versions for very long. If this is something you can fix soon we can stay on 9.3 until the fix comes, otherwise sadly we'll have to look at using another tool, which would be a shame because Flyway is a great fit for us and really easy to use.
Last good version POM: https://search.maven.org/artifact/org.flywaydb/flyway-core/9.3.0/jar
First bad version POM: https://search.maven.org/artifact/org.flywaydb/flyway-core/9.3.1/jar
Any info will be most helpful!
Which version and edition of Flyway are you using?
9.3.1 and later has the issue, so we are using 9.3.0!
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Issue still exists in 9.7.0
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
N/A
Which database are you using? (Type & version)
N/A
Which operating system are you using?
N/A
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
Updated to latest stable
What did you expect to see?
Expected license compliance checks to pass
What did you see instead?
License compliance checks fail, because of a GPL dependency from com.electronwill.night-config:toml

The license for com.electronwill.night-config is only LGPL and we don't modify the source code for it, so it shouldn't affect the license of anything that depends on it. Either way, there are plans to move from com.electronwill.night-config to jackson-dataformat in the near future.

@martin-traverse sorry to ping you here, but can I ask the direct concern of yours about LGPLv3? Management levels above me also concerned here about the usage, but to be honest I read the license a couple of times and I don't see how it would enforce you to do ANY of the following

make your sources open
put your product under LGPLv3.

I think only these are the concerns a company would have against a license, but that's GPL and not LGPL where you would have to do such things. I spent a lot of time on checking licenses to be used for some small thing 10 years ago and even that time I was "aware" that one license which is not problematic is LGPL as long as I don't make a modification and I use library linking. I want to understand small details which might matter here, but I don't get too much feedback from legal anyways what is their problem with the licensing... I tried to search people who had similar concerns and get an answer to see another point of view.
And sorry to spam this issue, not gonna reopen it or whatsoever.


Compile tools for unix systems

unix-compile.sh (Compile.bat alternative for Unix-based systems)
unix-coonfig.rsb (Windows.rsb alternative for Unix-based systems)

I suggest not to use .bat files and use only Bash..

User can install cygwin to compile your project.

It's not a good idea to create additional dependencies when you can do without them. A separate instruction for building and running on Linux will be added in the near future.

Personally, I'm not afraid of additional dependencies. But if we follow your logic, then the entire current project does not make sense, since the user can install Python on his PC with all the necessary libraries and run the original DeOldify. Moreover, all the components used there have been officially supporting Windows since Windows 7 for several years. Moreover, I successfully launched the original DeOldify on my Windows 7 PC. The meaning of this (and other ColorfulSoft's projects) is to simplify the process of using and developing neural networks under Windows and other operating systems as much as possible. For compilation DeOldify.NET in Windows, the tools that are installed in the system by default are sufficient. You are proposing to deliberately complicate the lives of users.


Added DDP related checks

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Shouldn't we also check is_world_process_zero in:
* `_stop_run_if_exists`

* `_initialize_run`

* and all kinds of `_ensure_run*` functions?

We assume that these private methods were properly called (in the proper state). It should be fine except that user may call .run or get_run but this should be documented (I've already added it to specification + Sabine should be aware of it).


Published Documentation Appears to Be Incorrect

Using https://docs.rubocop.org/rubocop-rspec/cops_rspec.html#rspecexamplelength as an example, a CountsAsOne param is indicated as supported. This isn't true in the current release, but rather only on the master branch. As a user of the library I'd expect the documentation at the URL to reflect the current release only.

The site has a dropdown where you can choose to see documentation for v1.44, v2.0, v2.1, v2.2 or master.
I don’t know if we can change what version to show by default.

Ah! I didn't realize that dropdown was there. Very handy.
I think you can change this value (https://github.com/rubocop/rubocop-rspec/blob/master/docs/antora.yml#L3) to match the release version?

Sorry for the confusion.
Doc version would be one more thing to remember for our releases and will open opportunities for mismatches.
RuboCop itself publishes docs for master as well (https://github.com/rubocop/rubocop/blob/master/docs/antora.yml#L3), I think we'll follow this, at least until there's an option to be always point to the latest released version.
There's not much in the changelong, but still - maybe it's time for a release?

Sounds good. Thank you for your courteous and prompt response.
On Tue, Apr 27, 2021, 17:03 Phil Pirozhkov @.***> wrote:

Closed #1145 https://github.com/rubocop/rubocop-rspec/issues/1145.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rubocop/rubocop-rspec/issues/1145#event-4653170768,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAZZHYVUO6PY525RAB3CLTK4RDJANCNFSM43VH7ANA
.


feat(docs): Some SBOM policies

This improves and adds some new policies, including:

SBOM is present
all components have license
no banned licenses
no banned packages
required packages
SBOM freshness (not older than xxxx)

All of them will output an individual message for each violation (for example, the list of packages without licenses)
Closes #1124

@jiparis any reason you are holding the merge? Maybe review from @javirln?

@jiparis any reason you are holding the merge? Maybe review from @javirln?

I'll add your suggestions and merge it right away.


feat: add <header> element for a11y

And move NAVIGATION_ITEMS to a file. It will be needed soon to create another way of navigating when not all tabs fit in screen.

#534  👈
main

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

Join @davidlj95 and the rest of your teammates on  Graphite


Combat Fabbers don't build mines

Queller only uses Combat Fabbers to repair not to build mines

You cannot have the AI build mines and repair, it either has to do one or the other and it has to do so across all personalities. To avoid breaking vanilla this behaviour was left untouched.


Plugins 'hyperterm-paste' and 'hyperterm-safepaste' refuse to work

[x] I am on the latest Hyper.app version

[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: macOS 10.14.4

Hyper.app version: 2.1.2 / 3.0.0-canary.8

Link of a Gist with the contents of your .hyper.js: link

Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere): See below

The issue is reproducible in vanilla Hyper.app: Yes

Issue
Plugins hyperterm-paste and hyperterm-safepaste (which modify pasting behavior) each do not work (I install them separately, not together), although they are loaded.
That is, pasting behavior is not modified in any way, e.g., trailing newlines are not removed.
On stable, dev tools say:
hyperterm-paste:
Plugin hyperterm-paste (1.1.2) loaded.

hyperterm-safepaste:
Plugin hyperterm-safepaste (1.0.5) loaded.

On canary, dev tools say:
hyperterm-paste:
Plugin hyperterm-paste (1.1.2) loaded.

hyperterm-safepaste:
[Notification] Plugin load error: "hyperterm-safepaste" failed to load in the renderer process. Check Developer Tools for details.
bundle.js:1 EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self' 'unsafe-inline'".

get the authors to update them :)


Multisig creation. Signatory selector - can't enter any wallet address

Description
Can't enter any wallet address into second signatory
https://github.com/user-attachments/assets/feaac19c-1320-47e3-8bd6-af01c4cab200
Environment
https://github.com/novasamatech/nova-spektr/commit/2c4fce1386e54459188aa5381f2ebde21bb427d7

Status verification: ✅


Remove hyphen from coffee-script

CoffeeScript is switching from coffee-script to coffeescript as the primary module for various reasons.
Refs https://github.com/atom/atom/pull/13731#issuecomment-340002601
cc @nathansobo

I suggest we stick with coffee-script to avoid having to whitelist coffeescript twice in the snapshot module code.

Yeah... this is dumb.


Introduce Vite

This PR introduces Vite as the build system for RGBDS Live.
I tried to do minimal changes to the code, so currently there’s still lots of globals and stuff, but the PR got big enough as it is. This already does some nice optimizations in that it bundles all the JavaScript and CSS files and minifies them, which should make the site load faster.
These are the changes introduced by this PR:

Move website code to top-level so that the vite can put the bundled and optimized build into www.
Make this a “proper” JS project by using package.json to import vite and to declare npm run build and npm run dev build scripts.
add -s EXPORT_ES6=1 to Wasm builds so that the output is a ES6 module. This makes it easier to use the Wasm builds with vite.
add -s ENVIRONMENT=web to rgbds builds so that the JS files don’t contain references to Node runtimes.
type="module" which allows us to use ESM imports/exports in the code and will allow us to use top-level await in the future.
Explicitly use globalThis to declare globals as ES6 modules have an isolated global scope by default.
Remove script includes for Wasm builds and import their ES6 modules where they are used instead.

Calling ./build.sh is still the right thing to do, exactly so that CI keeps working. Is it broken?

build.sh is not called at all atm in the CI, as all the final files would end up in the www folder anyway, so we need another step after the wasm builds to call build.sh


Fixes #3139 - Do not restart unchanged apps

Cherry-pick to release branch after merge!
I will check if I can also write an integration test for this quickly but I'd already appreciate feedback.

LGTM

added an integration test as well

LGTM too – waiting for tests

cherry-picked onto releases/0.15


fix(ListingManagement): Table view does not work if the owner display…

…name is blank #403

@hoganmaps @blaino

@wski

:+1:


Single Docker - use Terrarium public image

Use public Terrarium image for single.Docker(multistage build)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.EugeneP seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update link to airspeed-velocity

The current link was dead. The updated link redirects to https://asv.readthedocs.io

thanks @peterbell10


Error while trying to export collection to Postman

I trying to export collection as Postman collection but getting following error (tried export different Bruno collections)
Bruno version: portable, Windows, 1.12.3
Oops! Something went wrong
TypeError: Cannot read properties of undefined (reading 'tests')
TypeError: Cannot read properties of undefined (reading 'tests')
at generateEventSection (file:///[path redacted]/Bruno/resources/app.asar/web/_next/static/chunks/951-d020733b4cdea97a.js:1:305839)
at file:///[path redacted]/Bruno/resources/app.asar/web/_next/static/chunks/951-d020733b4cdea97a.js:1:307419
at arrayMap (file:///[path redacted]/Bruno/resources/app.asar/web/_next/static/chunks/pages/_app-25eaa9987eeec13f.js:11:58444)
at map (file:///[path redacted]/Bruno/resources/app.asar/web/_next/static/chunks/pages/_app-25eaa9987eeec13f.js:11:198057)
at generateItemSection (file:///[path redacted]/Bruno/resources/app.asar/web/_next/static/chunks/951-d020733b4cdea97a.js:1:307295)
at exporters_postman_collection (file:///[path redacted]/Bruno/resources/app.asar/web/_next/static/chunks/951-d020733b4cdea97a.js:1:307488)
at onClick (file:///[path redacted]/Bruno/resources/app.asar/web/_next/static/chunks/951-d020733b4cdea97a.js:1:308449)
at Object.Nb (file:///[path redacted]/Bruno/resources/app.asar/web/_next/static/chunks/framework-80ea8c0f440c6a32.js:9:14966)
at Tb (file:///[path redacted]/Bruno/resources/app.asar/web/_next/static/chunks/framework-80ea8c0f440c6a32.js:9:15120)
at file:///[path redacted]/Bruno/resources/app.asar/web/_next/static/chunks/framework-80ea8c0f440c6a32.js:9:33456

This also happens in Bruno version:  Mac Apple Silicon 1.12.3

Duplicate of #1943

This issue is fixed and will go out in the 1.13.0 release. Please subscribe to Discord announcement channel to stay notified on the releases.


v4 Service Bus Connection String with Managed Identity

Using Typescript and trying to create a Function App that listens for incoming messages from Service Bus Queue - I cannot find the combination to support Managed Identity. All advice given for .NET does not seem to work. The connection string is demanding a SAS pair and I do not want to use SAS.
The connection string used for an Service Bus client must specify the Service Bus namespace host and either a Shared Access Key (both the name and value) OR a Shared Access Signature to be valid. (Parameter 'connectionString')

Hi @redhatva I don't think there's anything really unique to Node.js in terms of managed identity. I coincidentally tried this out last week and it was working for me, although I might've had a slightly different setup than you. Here are the instructions I would recommend you follow - is this the same as what you were using?

General tutorial: https://learn.microsoft.com/en-us/azure/azure-functions/functions-identity-based-connections-tutorial-2
Additional information on service bus settings: https://learn.microsoft.com/en-us/azure/azure-functions/functions-bindings-service-bus-trigger?pivots=programming-language-javascript#identity-based-connections

One callout is make sure you are on the latest extension bundle in your host.json. I think you are okay with v3 or v4, but v2 is too old. This probably isn't mentioned in the .NET docs.

Closing due to lack of activity


Labels are duplicated on a pie chart

I have a pie chart that is displaying months as labels. Each month is repeated twice. The brunel below reproduces the problem:
data('https://dwotrends.mybluemix.net/projectcommits?projectid=39892981') stack polar bar y(commits) color(month) label(month,commits) legends(auto) title('developerWorks Open: Commits for hyperleger-archives/fabirc  over the last six months') style('{font-family:Helvetica;}') tooltip(#all)

Cannot replicate this; can you attach a picture? And maybe the generated HTML if that is possible

Picture is above
here is the jade template that is rendered
doctype html
html
head
link(rel='stylesheet' type='text/css' href='//brunelvis.org/js/brunel.2.0.css' charset='utf-8')
link(rel='stylesheet' type='text/css' href='//code.jquery.com/ui/1.11.4/themes/smoothness/jquery-ui.css' charset='utf-8')
link(rel='stylesheet' type='text/css' href='//brunelvis.org/js/sumoselect.css' charset='utf-8')
//    D3 Copyright ï¿½ 2012, Michael Bostock
//    jq1102 Copyright ï¿½ 2010 by The jq1102 Project
//    sumoselect Copyright ï¿½ 2014 Hemant Negi
//
script(src='//cdnjs.cloudflare.com/ajax/libs/d3/4.2.1/d3.min.js' charset='utf-8')
script(src='//d3js.org/topojson.v1.min.js')
script(src='//code.jquery.com/jquery-1.10.2.js' charset='utf-8')
script(src='//code.jquery.com/ui/1.11.4/jquery-ui.js' charset='utf-8')
script(src='//brunelvis.org/js/brunel.2.0.min.js' charset='utf-8')
script(src='//brunelvis.org/js/brunel.controls.2.0.min.js' charset='utf-8')

body
include menu.jade

svg(id="visualization" width="800" height="768")

div.brunel(id="controls")
h2 Commits Chart
script.
var project="#{project}";
var data="#{data}";
var fields="#{fields}";
var project="#{project}";
var chart="#{chart}";
data=data.replace(/;/g,'\n');
data=data.replace(/[ ]/g,'-');
//alert(data);
data=fields+'\n'+data;
//alert(data);
var styleElement;               // Element to put our custom Brunel styles into
//Dynamically sets the CSS for use by the Brunel visualization
function setCSS(css) {
if (!styleElement) {
styleElement = document.createElement("style");
document.getElementsByTagName("head")[0].appendChild(styleElement);
}
styleElement.innerHTML = css;
}
//Dynamically load JS given the brunel and the data as CSV
function loadBrunelJS(brunel_src, width, height, visid, csv) {
var queryParam = {
src: brunel_src,	//Brunel script
width: width,		//Output width
height: height,		//Output height
visid: visid		//id for  tag
};
var url = "https://brunel.mybluemix.net/brunel/interpret/d3?" + $.param(queryParam);
$.ajax({
type: "POST",
headers: {
'Accept': 'application/json',
'Content-Type': 'text/plain'
},
data: csv,	          //The data as CSV
url: url,
success: function (result) {
execBrunel(result);
},
error: function (xhr, textStatus, errorThrown) {
console.log(xhr.responseText);
}
});
}
//Execute the returned Javascript and set the CSS that Brunel requires
function execBrunel(brunelJSON) {
setCSS(brunelJSON.css);
eval(brunelJSON.js);
}
var charts=["com", "pcom", "ncm", "cobm", "chord"];
var found=10;
for (c in charts) {
if (chart == charts[c]){
found=c;
break;
}
}
  //alert("before calling dwotrends");
  //var brunel = "bar x(month) y(commits) sort(commits) title('developerWorks Open: Commits for "+project+" over the last six months') style('{font-family:Helvetica;}') tooltip(#all)  style('fill:red')";
  //var brunel = "bubfoundble color(month) size(commits) label(month,commits) legends(all) title('developerWorks Open: Commits for "+project+" over the last six months') style('{font-family:Helvetica;}') tooltip(#all)";
  //var brunel = "bubble color(month) size(commits) label(month,commits) legends(all) title('developerWorks Open: Commits for "+project+" over the last six months') style('{font-family:Helvetica;}') tooltip(#all)";
  var brunels = [
  "line x(month) y(commits) axes(x,y:grid) tooltip(#all) + point x(month) y(commits) axes(x:grid,y:grid) tooltip(#all) title('Commits Over Months for project "+project+"')",
  "bar y(commits) tooltip(#all) stack polar percent(commits) color(month) label(month, commits) legends(auto) title('Proportion of Commits Over Months for project "+project+"')",
  "bubble x(month) color(month) size(commits) label(month) sum(commits) legends(none) title('Number of commits per months for "+project+"')",
  "bar x(month) y(commits) axes(x,y:grid) tooltip(#all) sort(commits) transpose title('Number of Commits Ordered by Month for preoject "+project+"')",
  "chord x(month) y(commits) color(commits) tooltip(#all) title('Commits for "+project+" over the last six months')"
  ];
  var brunel=brunels[found];

  //Show the chart
  //var response=mos+counts
  loadBrunelJS(brunel, 600, 600, "visualization", data);

Got it now. The data you are using has the pie "category" as a date, which is treated as a numeric variable when we are showing it as a color, so the scale is a continuous one. For this case, you really want it to be categorical so we get a single color for each distinct time point, rather than interpolating the dates!
We do have an outstanding feature to allow you to override the default Brunel allocation to numeric or categorical, but that is not in there yet (if you are using a notebook, it WILL work, as you can use the notebook data frame to set the type, and Brunel respects that). So I think you'll need a work around, the simplest way of which is to make the field not readable as a date -- maybe reformat it that way.


opscode-manage fails to install on RHEL6

A customer faces issues installing the management console on RHEL6…They downloaded and installed CHEF 12 Server, then ran the command chef-server-ctl install opscode-manage.  Here is the output:
rpm -Uvh chef-server-core-12.0.8-1.el6.x86_64.              rpm
attention : chef-server-core-12.0.8-1.el6.x86_64.rpm: Entête V4 DSA/SHA1 Signatu              re, clé ID 83ef826a: NOKEY
Préparation...                       ################################# [100%]
Mise à jour / installation...
   1:chef-server-core-12.0.8-1.el6    ################################# [100%]
Thank you for installing Chef Server!
[root@frrdchefd Téléchargements]# chef-server-ctl install opscode-manage

Starting Chef Client, version 12.0.3
resolving cookbooks for run list: ["private-chef::add_ons_wrapper"]
Synchronizing Cookbooks:
  - private-chef
  - enterprise
  - apt
  - yum
  - runit
  - build-essential
  - yum-epel
Compiling Cookbooks...
Converging 3 resources
Recipe: private-chef::add_ons_wrapper
  * ruby_block[addon_install_notification_opscode-manage] action nothing (skippe              d due to action :nothing)
Recipe: private-chef::add_ons_remote
  * yum_repository[chef-stable] action create
    * template[/etc/yum.repos.d/chef-stable.repo] action create
      - create new file /etc/yum.repos.d/chef-stable.repo
      - update content in file /etc/yum.repos.d/chef-stable.repo from none to 5d              f5e2
      --- /etc/yum.repos.d/chef-stable.repo     2015-06-04 00:38:06.279757300 +0              200
      +++ /tmp/chef-rendered-template20150604-40747-nlqlo8      2015-06-04 00:38              :06.279757300 +0200
      @@ -1 +1,12 @@
      +# This file was generated by Chef
      +# Do NOT modify this file by hand.
      +
      +[chef-stable]
      +name=Chef Stable Repo
      +baseurl=https://packagecloud.io/chef/stable/el/7/$basearch
      +enabled=1
      +gpgcheck=1
      +gpgkey=file:///opt/opscode/embedded/keys/packages-chef-io-public.key
      +sslcacert=/etc/pki/tls/certs/ca-bundle.crt
      +sslverify=true
      - change mode from '' to '0644'
      - restore selinux security context
    * execute[yum-makecache-chef-stable] action run
      - execute yum -q makecache --disablerepo=* --enablerepo=chef-stable
    * ruby_block[yum-cache-reload-chef-stable] action create
      - execute the ruby block yum-cache-reload-chef-stable
    * execute[yum-makecache-chef-stable] action nothing (skipped due to action :              nothing)
    * ruby_block[yum-cache-reload-chef-stable] action nothing (skipped due to ac              tion :nothing)

  * yum_package[opscode-manage] action install
    * No version specified, and no candidate version available for opscode-manag              e
    ============================================================================              ====
    Error executing action `install` on resource 'yum_package[opscode-manage]'
    ============================================================================              ====

    Chef::Exceptions::Package
    -------------------------
    No version specified, and no candidate version available for opscode-manage

    Resource Declaration:
    ---------------------
    # In /opt/opscode/embedded/cookbooks/cache/cookbooks/private-chef/recipes/ad              d_ons_remote.rb

     48:   package pkg do
     49:     notifies :create, "ruby_block[addon_install_notification_#{pkg}]",               :immediate
     50:   end
    51: end

    Compiled Resource:
    ------------------
    # Declared in /opt/opscode/embedded/cookbooks/cache/cookbooks/private-chef/r              ecipes/add_ons_remote.rb:48:in `block in from_file'

    yum_package("opscode-manage") do
      action :install
      retries 0
      retry_delay 2
      default_guard_interpreter :default
      package_name "opscode-manage"
      timeout 900
      flush_cache {:before=>false, :after=>false}
      declared_type :package
      cookbook_name "private-chef"
      recipe_name "add_ons_remote"
    end

Running handlers:
  - #<Class:0x00000004e831a0>::AddonInstallHandler
Running handlers complete
[2015-06-04T00:38:46+02:00] FATAL: Stacktrace dumped to /opt/opscode/embedded/co              okbooks/cache/chef-stacktrace.out
Chef Client failed. 4 resources updated in 70.0411058 seconds
[2015-06-04T00:38:47+02:00] FATAL: Chef::Exceptions::Package: yum_package[opscod              e-manage] (private-chef::add_ons_remote line 48) had an error: Chef::Exceptions:              :Package: No version specified, and no candidate version available for opscode-m              anage

Sure it's RHEL 6? they're getting detected as 7:
      +baseurl=https://packagecloud.io/chef/stable/el/7/$basearch

Going to close this out, since no further feedback has come in and as far as we currently know everything is working as expected.


Migrate from ISODateTime to GraphQLDateTime

https://github.com/opencollective/opencollective-frontend/pull/6511

@Betree It's definitely a breaking as we had to update frontend code. Not sure if other projects are also relying on this.

The PDF service is, I've made a similar PR in https://github.com/opencollective/opencollective-pdf/pull/691

@nicolo-ribaudo I'm sorry about it, we hoped it would be only used in our codebase and we missed that you and others were relying on it.
We're generally avoiding breaking changes here but this is a good lesson. We'll think about a mechanism to publish important changes to our API.