Fetch failing with cache errors...

I'm seeing strange issues with govendor fetch +missing where packages in the cache directory are not being found:
% govendor fetch +missing
Error: Failed to fetch package "github.com/coreos/etcd/client": open /tmp/v/.cache/govendor/github.com/coreos/etcd/clien: no such file or directory
[1]    61628 exit 2     govendor fetch +missing
% go get -u github.com/coreos/etcd
% govendor fetch +missing
Error: Failed to fetch package "github.com/aws/aws-sdk-go/private/protocol/rest": open /tmp/v/.cache/govendor/github.com/aws/aws-sdk-go/private/protocol/res: no such file or directory
[1]    62548 exit 2     govendor fetch +missing
% go get -u github.com/aws/aws-sdk-go/private/protocol/rest
% go get -u github.com/aws/aws-sdk-go
% govendor fetch +missing
Error: Failed to fetch package "github.com/hashicorp/hcl/hcl/ast": open /tmp/v/.cache/govendor/github.com/hashicorp/hcl/hcl/as: no such file or directory
[1]    63426 exit 2     govendor fetch +missing
[etc]

It looks like some path length is being exceeded because the right path does exist on the filesystem:
% govendor fetch +missing
Error: Failed to fetch package "golang.org/x/net/context": open /tmp/v/.cache/govendor/golang.org/x/net/contex: no such file or directory
[1]    64110 exit 2     govendor fetch +missing
% ls -lA /tmp/v/.cache/govendor/golang.org/x/net/context
total 72
-rw-r--r--  1 sean  wheel  14129 Mar 18 17:39 context.go
-rw-r--r--  1 sean  wheel  14942 Mar 18 17:39 context_test.go
drwxr-xr-x  6 sean  wheel    204 Mar 18 17:39 ctxhttp
-rw-r--r--  1 sean  wheel    677 Mar 18 17:39 withtimeout_test.go

Using go get on the missing package works around the cache and prevents the problem, however it's rather tedious.

Quick update, it looks like there are incomplete entries in vendor.go from earlier operations.  A fresh vendor.go file from govendor init seems to recreate these incomplete entries:
                {
                        "path": "github.com/aws/aws-sdk-go/aws/reques",
                        "revision": ""
                },

I'm unsure how these are related, but FYI.

I suspect there are two separate issues. I'm working on the vendor file corruption issue, (which I have confirmed and I'm correcting right now). I'll look into this separately.

I'm confused on this one. Could you provide exact steps to reproduce?
I'm assuming the `go get -u

Here are the exact steps required to reproduce this issue:
% mkdir /tmp/t
% cd /tmp/t
% setenv GOPATH $PWD
% go get github.com/hashicorp/vault
% cd src/github.com/hashicorp/vault/
% rm -rf Godeps vendor
% govendor init
% cat <<EOF > vendor/vendor.json
{
	"comment": "",
	"ignore": "test appengine",
	"package": [],
	"rootPath": "github.com/hashicorp/vault"
}
EOF
% govendor fetch +missing
Error: Failed to fetch package "github.com/coreos/etcd/client": open /private/tmp/t/.cache/govendor/github.com/coreos/etcd/clien: no such file or directory
Exit 2

If you want another project to use as a regression test, try Terraform.

Sweet. I'll resolve this shortly.
On Fri, Mar 18, 2016, 20:43 Sean Chittenden<EMAIL_ADDRESS>wrote:

Here are the exact steps required to reproduce this issue:
% mkdir /tmp/t
% cd /tmp/t
% setenv GOPATH $PWD
% go get github.com/hashicorp/vault
% cd src/github.com/hashicorp/vault/
% rm -rf Godeps vendor
% govendor init
% cat < vendor/vendor.json
{
"comment": "",
"ignore": "test appengine",
"package": [],
"rootPath": "github.com/hashicorp/vault"
}
EOF
% govendor fetch +missing
Error: Failed to fetch package "github.com/coreos/etcd/client": open /private/tmp/t/.cache/govendor/github.com/coreos/etcd/clien: no such file or directory
Exit 2
If you want another project to use as a regression test, try Terraform
https://github.com/hashicorp/terraform.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/kardianos/govendor/issues/110#issuecomment-198627210


getting an nginx 403 forbidden page after docker-compose up

solved! issue was i was pointing in a wrong host directory path of my pagekit core install


Inconsistent timeout behavior makes this plugin unusable

Describe the bug
The timeout functionality on this plugin is inconsistent across browser, iOS, and Android making it unusable for my project.  I previously used cordova-advanced-http on Cordova and it worked really well.  This is not a good replacement.
To Reproduce
Steps to reproduce the behavior:
Download my project at https://github.com/funkyvisions/capacitor-demo
Expected behavior
Run the project.  It will call 3 different endpoints with a read and connection timeout of 10 seconds.

https://httpstat.us/200?sleep=1000
https://httpstat.us/200?sleep=15000
https://thisendpointdoesnotexist.com

On the browser:

Works fine. Gets 200 response.
Doesn't timeout after 10 seconds like it should.  Returns response after 15 seconds.
Gets an error like it should.

On iOS:

Works fine. Gets 200 response.
Never returns.
Never returns.

On Android:

Works fine.  Gets 200 response.
Works fine.  Times out after 10 seconds.
Gets an error like it should.

Additional context
This is the code in question:
    async function callHttp(url) {
      alert(`Calling ${url}`)
      let response
      try {
        response = await Http.get({
          url: url,
          connectTimeout: 10000,
          readTimeout: 10000
        })
      } catch(e) {
        response = {
          message: e.message
        }
      }
      alert(JSON.stringify(response))
    }

    await callHttp('https://httpstat.us/200?sleep=1000')
    await callHttp('https://httpstat.us/200?sleep=15000')
    await callHttp('https://thisendpointdoesnotexist.com')

I did figure out how to get cordova-advanced-http to work, so that may be my backup plan until this plugin works a little better.

Sweet.  It looks like https://github.com/capacitor-community/http/issues/224 handles the iOS timeout and unknown endpoint issues.  So now I just need to get timeout to work on browser (not as critical).


Set dragging boundary

You can now pass into makeDraggable the element which should be the boundary for dragging.
Needed to change chapterDOM a bit to make it work - are these changes ok like that?
If you drag the element fast it stops some way before the border - if you move it slow it stops exactly at border.
Here a preview of our game with boundary:

Github says the video can't be played because the file is corrupt, but the code looks okay to me

Strange - How about now? Have change from mp4 format to gif. If still not possible you can pull this branch, run npx webpack and try it out locally.

Github says the video can't be played because the file is corrupt, but the code looks okay to me

Strange - How about now? Have changed from mp4 format to gif. If still not possible you can pull this branch, run npx webpack and try it out locally.

Now i can see video. It looks fine to me, good job!


[Flutter Web] Explanation about FontManifest.json

When running the hello_world example of flutter web using chrome, one will encounter the errors regarding font manifest, including:
(1) an error, a failed network request (404) to http://localhost:8080/assets/FontManifest.json 404
(2) a warning, about a missing font manifest "Font manifest does not exist at assets/FontManifest.json – ignoring.".
I couldn't find a section about it in the official flutter site, although I did understood some of it from this site: https://www.didierboelens.com/2019/05/flutter-for-the-web/
Resolution
Possible resolutions:
(1) Not sure if it's a good idea, but perhaps some bool parameter in the build configuration so that FontManifest.json will not even be attempted to load. For now it seems an app can be used without needing it, and as far as I know the only way of not showing a network error on the chrome console is by not requesting it to begin with.
(2) Give a link with more information, preferably from the official Flutter site.

I don't get either of those errors when running the flutter/flutter hello world. Can you please share how you are running these examples?

I just followed the Readme in flutter web github, specifically the part about "Getting Started", up until and including "Run the hello_world example": https://github.com/flutter/flutter_web#run-the-hello_world-example
How are you not getting the errors though? I even found the source file where it's from, and it make sense; Unless you provided the FonttManifest.json file ?
p.s.
Just to clarify, I'm seeing those errors in the web console, not in my console where I'm running webdev serve

Ahh that is the forked flutter web, I thought you were talking about the flutter/flutter example.
The plan is to upstream/merge the repositories (already under way) - the font manifest and all asset logic is handled by the flutter tool, just the same as with android or iOS applications

What do you mean by "the forked flutter web" ? I don't see any fork text on github, and I understood flutter web is maintained in a different repo until merged, which doesn't seem as done yet (even if it's under way).
"the font manifest and all asset logic is handled by the flutter tool"
Can you explain that too? is there some tool I was supposed to use?

By "unforked" we mean that you'd be using this: https://flutter.dev/docs/get-started/web
Specifically for fonts, these instructions should work: https://flutter.dev/docs/cookbook/design/fonts
Please, give it a shot, and feel free to reopen this issue if it's still there.


Zsh compatibility and some minor fixes

Set PATH earlier (for get-yarn-cluster-id)
Zsh compatible path extraction
Replacing err command (what is the err command?)

lgtm thx!


Moscow incorrect timezone select

Europe/Moscow should be UTC + 3

Yeah, what's up? Why is almost every one offset by 1?


Add tsconfig moduleResolution and newLine support

I have pushed https://github.com/jrieken/gulp-tsb/commit/e28375af4f7375fb79bbc4d5e0d4f9d94ddb56ab which make handling compiler options a business of TypeScript again. Should make these kind of changes obsolete.


Blocks main thread on potentially slow file reads

Picking a file potentially means downloading it from the internet in the case of e.g. an iCloud Drive remote file. This can be slow when the file is large or the network is slow. Unfortunately this plugin processes everything on the UI thread, so this hangs all of Flutter as well.
(Tested on iOS)

Android is already using coroutines so it doesn't have this problem.


Yosys modulus operator has remainder semantics

As per Wikipedia, usually modulus has the sign of the divisor, while remainder has the sign of the dividend. In languages that only have one % operator, you can get either behavior. https://en.wikipedia.org/wiki/Modulo_operation#In_programming_languages
However, VHDL is a language that has both a mod and a rem operator, where rem has the behavior of Yosys/Verilog $mod/%, and there is no way to represent actual VHDL mod.
Below is a program that when synthesized with GHDL currently throws an error. When using Verific, both statements result in a $mod operator, which is clearly incorrect.
library ieee;
use ieee.std_logic_1164.all;
use ieee.numeric_std.all;

entity lut2 is
    port (a: in integer;
          b: in integer;
          y: out integer;
          z: out integer);
end lut2;

architecture synth of lut2 is
begin
  y <= a mod b;
  z <= a rem b;
end synth;

Here is the offending Verific code:
https://github.com/YosysHQ/yosys/blob/db2268703f657fb977e1aa5506748683fbeb6cb1/frontends/verific/verific.cc#L483-L487

I have the same problem in nMigen, which follows Python semantics and so uses a modulo operator. IMO the $mod cell should be renamed to $rem, and a new $mod (or perhaps $mod2) cell should be added with the correct semantics, so that it can be used by VHDL and nMigen.


Fix new clang-tidy warnings

It seems that the newer version of clang-tidy is finding some more issues with the code base. One of those, misc-include-cleaner, is a bit overzealous and PRs are being spammed with comments from the clang-tidy bot about them, so I've just suppressed it.
Besides that, I've fixed up other various clang-tidy warnings, some with auto-fixes and some manually.
Fixes #432.

Right, this should be working now.
I've also turned off that "LGTM" message that the bot sends, because it's annoying.


get_metadata_schema in BaseImagingExtractorInterface needs a small refactor

Accessing the private attribute sampling frequency and casting to float in get_metadata_schema is not a proper place to have anymore, I think it should be removed.
https://github.com/catalystneuro/nwb-conversion-tools/blob/07e958d7564eb12d00a5ff5fabe661677d023efa/src/nwb_conversion_tools/datainterfaces/ophys/baseimagingextractorinterface.py#L27-L29
@CodyCBakerPhD what do you think?

@CodyCBakerPhD How would you do it? I was thinking to add a decorator (yeah, I know...) that checks the return value of the get_sampling_frequency function:
def check_get_sampling_frequency_return_dtype(func):
    """ A decorator to check the return value of a function is of float type. """

    @wraps(func)
    def wrapper(*args, **kwargs):
        return_value = func(*args, **kwargs)
        if not isinstance(return_value, float):
            return_value = float(return_value)
        return return_value

    return wrapper

We are already using typehints:
https://github.com/catalystneuro/roiextractors/blob/2361597b8234e988e6d70e661b7b20428ff32c99/src/roiextractors/imagingextractor.py#L38-L40

@CodyCBakerPhD How would you do it? I was thinking to add a decorator (yeah, I know...)

I would simply write a test on the ROIExtractors side to ensure the output of each extractor .get_sampling_frequency is a float type.
The conversion tools shouldn't do too much magic like this on their own, they should ideally just expect particular forms of inptus/outputs and not behavior correctly if that is violated.
I did look into Pydantic type stuff for this a while back but the huge downside to that is they are not compatible with numpy data typing, and complicated unions of things can throw it off.

yes, well put, Cody. Magic like this is difficult to maintain and tests are much simpler and just as effective.

The only place in the pipeline where we need this was changed recently and now does a casting over there:
https://github.com/catalystneuro/nwb-conversion-tools/blob/07e958d7564eb12d00a5ff5fabe661677d023efa/src/nwb_conversion_tools/tools/roiextractors/roiextractors.py#L110
I just tested removing this and it works as it is.

Concerning checking for sampling rate equal being a float in imaging extractor, check:
https://github.com/catalystneuro/roiextractors/pull/164

This is removed now in #559.
Should we close the issue @weiglszonja ?

Yeah its fine, I’ll close it.


test: add needs for backport job

Description
test: add needs for backport job

@aqua-bot backport release/v0.52

@aqua-bot backport release/v0.52


Awkward display of lengthy entries that force a line break

Is it an actual bug?
It's what I would consider to be a visual bug.
Did you check if the bug has already been reported?
Yes.
Describe the bug
Title and screeshot below pretty much say it all. It happens in normal view as well as in compact view.
Expected behavior
I expected the display to be in line with single-line entries, that is circle*, priority icon and context/project "pills" + due/rec icons on the same line. Or to be more precise, not have an extra line dedicated to each.
I don't see any benefit of the current behavior. You could fit several more entries in that same space without it being visual overload.
*Why a circle -- that can be confused with a radio button-- and not a check box? (For recurring tasks even some variation on that  check box might be good idea.)
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Version of sleek: 1.1.5
Source: Windows Store + portable .zip from Github

@matterharz this is simply a css bug not a wanted behaviour and it should work in this pre release: https://github.com/ransome1/sleek/releases/tag/v1.1.6-rc.3

Much better, thanks.
It still seems though as if the prio icons have their own line for no reason (unlike single-lline entries). Also, the remaining icons and pills are sort of piled up at the end of line which looks untidy and incoherent:

This would make sense if they were arranged in a column at the end of each row across all the entries (though problematic for very short ones) but each entry, multi-line or not, seems to place them in a random spot. Adding them all to the last line, directly behind the last word of each entry seems more reasonable to me. It also saves space since you don't have rather early line-breaks like in the screenshot above.

This would make sense if they were arranged in a column at the end of each row across all the entries (though problematic for very short ones) but each entry, multi-line or not, seems to place them in a random spot. Adding them all to the last line, directly behind the last word of each entry seems more reasonable to me. It also saves space since you don't have rather early line-breaks like in the screenshot above.

Implementing your proposal as a general rule would only benefit longer todos with lots of tags. Users who use short entries and few tags could be disadvantaged by this as every todo would then occupy more height. If I find some time anytime soon, I can check if there is a CSS way to straighten this out if a todos needs more space. If you by any chance have knowledge on CSS you could contribute to sleek by solving this.

Implementing your proposal as a general rule would only benefit longer todos with lots of tags. Users who use short entries and few tags could be disadvantaged by this as every todo would then occupy more height.

I agree. Just noticed (the structure of) my post was ambiguous. My proposal would simply be this: add icons and pills to the end of each entry as if they were just regular text.
Unfortunately, I can't help you out with this since I have no knowledge of CSS (or any mark-up or programming language) whatsoever.


cert create command is moved or broken

Womp, this does seem like an actual bug
In the blog post I referred to in the other issue, there is a line to setup the certs
hashi-up cert create --host <IP_ADDRESS> --host vault.example.com
This functionality seems to have been moved or deleted
Error: unknown command "cert" for "hashi-up"
Run 'hashi-up --help' for usage.
unknown command "cert" for "hashi-up"

Browsed through the CLI manual a bit but didn't see anything explicitly cert related.

@rgbrizzlehizzle You must use
hashi-up tls cert create ...
instead of
hashi-up cert create ...

Hi @rgbrizzlehizzle,
You were right, the cert subcommand was incorrectly removed when releasing version 0.10.0.
The functionality is recovered in the latest release.


Create 2019-06-19-Future-of-CRDs-Structural-Schemas.md

Follow-up of the 1.15 release blog post.

/cc @castrojo

@sftim merged your fixes

Ok, looks like 1.15 is going today for sure, can you modify to publish tomorrow (6/20)? Sorry for the inconvenience Stefan!

@castrojo done

/lgtm
/hold
All set, thanks! We'll unhold and merge on friday, thanks for your contribution!

/approve

/hold cancel


Replace poison with jason

Or may be made it configurable

What's the advantage of using jason over poison?

I now understand it's becoming the default for the Elixir community, but we'll need to remove poison completely to switch to jason. That means removing any other references to it and swapping out the dependencies.


password text with fresh install is not editable field

The change that was made to unify the lock/unlock button probably triggered this.  On a new wallet without a set password, the password field is not editable so the initial password cannot be set.

Disregard that, not paying attention.


CSS queries by full name rather than family name

I'm currently looking to somewhat replicating the font lookup a browser does, and I stumbled upon this crate. However it seems to only be able to match against the family name rather than the full name. Is this something that you would consider adding, or that I could do a Pull Request for?

What do you mean by full name? A font can have a family name, subfamily name, typographic family name and typographic subfamily name. There are also special name generation algorithms like this one.
So the problem is not to change the current logic, but to understand what result we actually need.

What do you mean by full name? A font can have a family name, subfamily name, typographic family name and typographic subfamily name. There are also special name generation algorithms like this one.
So the problem is not to change the current logic, but to understand what result we actually need.

I believe I've seen it be referred to that in Firefox's source code? I'm not even sure anymore, I'm so confused at this point. Basically browsers can (sometimes?) match on "family name + typographic subfamily" or alternatively "family name + font variation instance" (Firefox doesn't support the latter). I'd like to match against both possibilities.

I believe I've seen it be referred to that in Firefox's source code? I'm not even sure anymore, I'm so confused at this point. Basically browsers can (sometimes?) match on "family name + typographic subfamily" or alternatively "family name + font variation instance" (Firefox doesn't support the latter). I'd like to match against both possibilities.

However this is also what most Windows application choose fonts by

Yes, but this is not the only way. fontdb works with font family name (id1). Basically, to get the output you see on your screenshot, you have to generate it manually on your side and then deconstruct it for frondb. Because Cascadia Mono Light is not a family-name, Cascadia Mono is. And you can match it just fine with fontdb.
The problem is that font's creator can set font family to any value. So for example, NotoSansDevanagari-CondensedMedium.ttf contains:

Family
Noto Sans Devanagari Cn Md

Subfamily
Regular

Typographic Family
Noto Sans Devanagari

Typographic Subfamily
Condensed Medium

PostScript
NotoSansDevanagari-CondensedMedium

Which one do you need and in what combination?
PS: ideally, we should use Typographic Family, but it's often missing and/or contains unexpected data.

However this is also what most Windows application choose fonts by

Yes, but this is not the only way. fontdb works with font family name (id1). Basically, to get the output you see on your screenshot, you have to generate it manually on your side and then deconstruct it for frondb. Because Cascadia Mono Light is not a family-name, Cascadia Mono is. And you can match it just fine with fontdb.
The problem is that font's creator can set font family to any value. So for example, NotoSansDevanagari-CondensedMedium.ttf contains:

Family
Noto Sans Devanagari Cn Md

Subfamily
Regular

Typographic Family
Noto Sans Devanagari

Typographic Subfamily
Condensed Medium

PostScript
NotoSansDevanagari-CondensedMedium

Which one do you need and in what combination?
PS: ideally, we should use Typographic Family, but it's often missing and/or contains unexpected data.

Basically I'm getting my font names from more or less basically the same list as Paint. Which seems to always be (after having looked at it a lot more now) "typographic family" (or just family, not sure, shouldn't be too hard to figure out) + "variation instance name", so I'd need to somehow take this string and find the correct font to load for it. I'm guessing it might make sense if I just pop off pieces at the end of the font until I hit a font family that exists and then look for a named variation instance with the rest? (+ maybe a PR to ttf-parser to query for the named variation instances)

Basically I'm getting my font names from more or less basically the same list as Paint. Which seems to always be (after having looked at it a lot more now) "typographic family" (or just family, not sure, shouldn't be too hard to figure out) + "variation instance name", so I'd need to somehow take this string and find the correct font to load for it. I'm guessing it might make sense if I just pop off pieces at the end of the font until I hit a font family that exists and then look for a named variation instance with the rest? (+ maybe a PR to ttf-parser to query for the named variation instances)

Yes, I think this should be handled by the caller. Nothing we can do on the fontdb side.

maybe a PR to ttf-parser to query for the named variation instances

You mean STAT?

Yes, I think this should be handled by the caller. Nothing we can do on the fontdb side.

maybe a PR to ttf-parser to query for the named variation instances

You mean STAT?

fvar: https://docs.microsoft.com/en-us/typography/opentype/spec/fvar
There's a named instance array at the end

fvar: https://docs.microsoft.com/en-us/typography/opentype/spec/fvar
There's a named instance array at the end

Oh right, they store the same value.

Oh right, they store the same value.

Seems like this is agains the CSS spec, so we would not support it either. A fontdb::Family is strictly a Font Family (ID 1).

Mmh sounds like the spec needs to be updated to reflect reality then. I wonder how easy it is to get that change in.

By the CSS, you're suppose to use font-weight and font-style instead. CSS doesn't care about full typographic family names.
I'm not sure about origin of family names, but this is definitely a huge mess. There are so many ways to define and combine them that this is not even a joke.

Wait, are you even sure that this "is against the CSS spec"?

Note: The precise way a set of fonts are grouped into font families varies depending upon the platform font management APIs.

On various platforms, different names are used to search for a font. For example, with the Windows GDI CreateIndirectFont API, either a family or fullname can be used to lookup a face, while on Mac OS X the CTFontCreateWithName API call is used to lookup a given face using the fullname and Postscript name. Under Linux, the fontconfig API allows fonts to be searched using any of these names. In situations where platform API’s automatically substitute other font choices, it may be necessary to verify a returned font matches a given name.

So not only does the spec seem to mention that full names and co. are usually used for the family names, but also all the OS APIs and browsers do it that way. So only supporting ID 1 isn't going to be a great user experience even if it was in the spec like that, which apparently it isn't even? (Or maybe I'm missing something)
Anyways, I'm not sure to what degree I'm still even affected by this issue, so I'm fine if it's closed.

I did some tests on macos and Chrome and Firefox ignored family names with Regular, Bold, Light suffixes. Safari didn't mind. This is the web inconsistency for you.
There are actually two issues here. First, I'm not sure if this is allowed by the spec to begin with. It's not really clear. And second, getting/generating those names are far from trivial and error-prone.
Here are my results for:
<text id="text1" x="32" y="100" font-family="Arial Bold" font-size="64">Text</text>
<text id="text2" x="32" y="150" font-family="Arial" font-weight="bold" font-size="64">Text</text>

As you can see, only Safari is able to detect both. And since fondb is web-first library, I must obey the Chrome.

That's... surprising. I'll look into it more later. Also Merry Christmas :)

Thanks! 😃 You too.

Not sure if this is necessarily the original problem, but it seems like the problem is rather opposite in that Arial Black, Calibri Light and co. are not considered to be part of Arial, Calibri and co. respectively.

I also tried it with cosmic-text and it absolutely runs into this problem and can't handle these fonts.

Strange. I will check Windows later, but macos fonts do not include Typographic Family. But they do have Full Name sometimes.
This is one giant mess.

Just compare MS docs about Typographic Family ID 16 and Apple one.
MS:

16: Typographic Family name: The typographic family grouping doesn’t impose any constraints on the number of faces within it, in contrast with the 4-style family grouping (ID 1), which is present both for historical reasons and to express style linking groups. If name ID 16 is absent, then name ID 1 is considered to be the typographic family name. (In earlier versions of the specification, name ID 16 was known as “Preferred Family”.)

Apple:

16: Preferred Family. In Windows, the Family name is displayed in the font menu, and the Subfamily name is presented as the Style name. For historical reasons, font families have contained a maximum of four styles, but font designers may group more than four fonts to a single family. The Preferred Family and Preferred Subfamily IDs allow font designers to include the preferred family/subfamily groupings. These IDs are only present if they are different from IDs 1 and 2.

Not only Apple calls it differently, but also Apple's name makes more sense. If it's preferred, then we should prefer it.
The issue is that it will not fix your issue anyway, because it's still not a Full Name.

Yet another example: /System/Library/Fonts/MuktaMahee.ttc

Family: MuktaMahee Regular
Full Name: MuktaMahee Regular
Typographic Family: Mukta Mahee

FontBook shows it as Mukta Mahee. So #35

The issue is that it will not fix your issue anyway, because it's still not a Full Name.

As shown here, I don't think Full Name is used by Chrome anymore and it's these three instead:

I'm fine with only the first and third being supported I think.

Family + Subfamily = Full Name. Not sure if chrome generates one automatically or gets one from the file. Will have to dive into its source code to find out.
Using Typographic Name as preferred is an easy fix, but I'm not sure it will improve the situation much. Unless the Windows Calibri font has "Calibri Light" Typographic Name.


[libspatialindex] Update to 1.9.3

When updating the port, we need to update the CONTROL file and portfile.cmake

Anything I can do?

When updating the port, we need to update the CONTROL file and portfile.cmake

Anything I can do?

Please update CONTROL file

Update with what exactly?

Update with what exactly?

Please run the command "./vcpkg x-add-version libspatialindex --overwrite-version" and submit again

Thanks for the hint, PR updated

Hi Matthias,
you must first commit all changes in port/*, then do the x-add-version stuff in a separate commit.
Best regards, Kai

@m-kuhn , @dg0yt gave you a good suggestion

Thanks Kai, I hope ci is happy now.

Thanks Kai, I hope ci is happy now.

Not yet.
Re-run vcpkg x-add-version --overwrite-version libspatialindex. Then add the changes in a separate commit.
The point is that this adds a reference to a particular git tree element (versions/* refering to a particular state of ports/*). If you squash the changes, you would need to know the resulting reference in advance, but this is impossible.
Normally I would also suggest to first run vcpkg format-manifest ports/libspatialindex/CONTROL in order to convert away from the deprecated CONTROL format, but I don't want to ask too much at the moment ;-)

Please add commits as requested, and push normally.  Force-pushing a single commit is going to fail. The last commit must be the change from vcpkg x-add-version.

I think the culprit here is static builds

DLLs should not be present in a static build, but the following DLLs were found:
D:/packages/libspatialindex_x64-windows-static-md/bin/spatialindex-64.dll
D:/packages/libspatialindex_x64-windows-static-md/bin/spatialindex_c-64.dll
D:/packages/libspatialindex_x64-windows-static-md/debug/bin/spatialindex-64d.dll
D:/packages/libspatialindex_x64-windows-static-md/debug/bin/spatialindex_c-64d.dll

There should be no bin\ directory in a static build, but D:\packages\libspatialindex_x64-windows-static-md\bin is present.
There should be no debug\bin\ directory in a static build, but D:\packages\libspatialindex_x64-windows-static-md\debug\bin is present.
If the creation of bin\ and/or debug\bin\ cannot be disabled, use this in the portfile to remove them

@m-kuhn ,Thanks for your pr, Duplicate of #17506


Fix ddim

Fix for issue #81 (the selected alphas were slightly wrong)

thank you for this!

no problem!


Design and implement mechanism to show users compliance analysis findings

Kiryl, I'm assigning this to you for the future even though we don't yet have enough data logged. The idea is to look at a large amount of logged privacy flag data, and from that data devise some method of giving the user a discrete and simple recommendation about their data- i.e., "we think this site sells your data" or "we think this site is complying". Warnings about selling data should not happen all the time, there should be some threshold- otherwise the user would have no reason to heed constant warnings on their favorite sites.
I think the 'algorithm' here should probably be pretty simple, even maybe as simple as coming up with some numerical threshold/percentage for allowed number of unset (falsy) data privacy flags.

Warnings about selling data should not happen all the time, there should be some threshold- otherwise the user would have no reason to heed constant warnings on their favorite sites.

A few possibilities and comments.

Log all (potential) violations to a background page. In addition, show the result for the current page in the popup --- violation or not. That way, it is up to the user whether they want to see what is going on (or go back to the logged violations in the background page).
Alternatively/in addition, change the little green circle for the OptMeowt icon @kalicki1 introduced to yellow when we see a violation. E.g., currently it is turning briefly full green when visiting a site with .well-known (e.g., try for the NYTimes). Similarly, the icon could be briefly turning full yellow (red and green would not be ideal colors because they look the same grey for color-blind people).
There is the usability issue of altert fatigue you mention, @stanleymarkman. Maybe the above could be some ways to address that. Another usability issue is that we may be sometimes not correct, e.g., we incorrectly detect a flag. We will have to figure out how often that happens. But the easiest way to resolve that point is to just include an explanation in the settings page or elsewhere that the analysis may be incorrect sometimes.

@stanleymarkman, I renamed this issue. Feel free to change with something that captures your idea better.

Sounds good- I think the final UI will end up being very similar to the ideas we discussed in lab meeting:
-First layer: simple color notification in the icon (with colorblind support)
-Second layer: a 1-2 sentence description in the Optmeowt dropdown, with a link to the background/settings page
-Third layer: Detailed breakdown of the privacy flags with specific resources linking to each legislation/implementation. For example: "This website ran 8 ad requests to Google AdSense without [Google's RDP setting]{https://support.google.com/adsense/answer/9598414?hl=en}, despite the GPC cookie being set."
There's two challenges I'll work on here: first, getting the UI to the point where it's giving the user useful info without constantly bothering them, which will require some testing; and second familiarizing myself with the Chrome/Firefox extension UI APIs.

Excellent, @stanleymarkman!
As this plan will likely require quite a number of changes, consider first bringing what you currently have into the main branch (issue #163). Feel free to decide however you see fit, though.

@kbeliauski suggested that the compliance analysis functionality works the way that a user can enter one or multiple websites to analyze, hit a start button, OptMeowt will then silently do the analysis (via headless browsing; is that possible?), and generate a report (say, a CSV file) with the analysis results. This change in functionality also necessitates a different approach to the UI.
I think we would need to further evolve the UI from what @stanleymarkman described. Essentially, the third layer could be what goes into the CSV file. The UI would be a button to start the analysis functionality, e.g., in the settings, some notification that the analysis is finished, an button for exporting the results (the functionality and design could be similar to the import and export of the domain list).
Maybe, this could be a separate Compliance Analysis Settings page (in addition to Settings, Domain List, and About). This Compliance Analysis Settings page would only be present in the Firefox version of OptMeowt (and, in terms of the codebase, is a separate Firefox module).

I think that the way to go is, like @SebastianZimmeck said, to have a separate tab in the settings page for the multiple page analysis mode, but also keep a simpler version in the popup. The simpler version will just do what @stanleymarkman described and only for the website the user is currently on.

We can revive our mockup folder and put the mockups there.

I drew up a quick sketch of a popup design specifically for the US privacy string analysis functionality. This is very flexible and final decisions are TBD but let's use this as a starting point. A lot more has been mentioned above on more intricate design ideas and for more flags, especially with a settings page, so this is definitely a small starting point.

As a small note, I drew this up with a "browsing" use case in mind, where one site is visited and an analysis is done without any other input, so this may be reason to change the designs as well.

Nice!

I like the analysis and protection labels and their symbols in Proposal 1. (As a side note, I feel the more prominently we advertise the analysis mode, the more flawless the transition from and to it has to be as it will be used more frequently.)
Can users click on the analysis and protection labels? If so, what happens when they do?
It seems you envision reusing in the Analysis Breakdown a good amount of structure and code that we already have in place for the approximately corresponding Domain List. That is great!
Not sure if we need a separate Analysis Settings label/button. Maybe, this can be done through the normal top right settings icon. Maybe, if you are in the analysis popup, the link goes to the analysis settings page (or analysis section of  the settings page). And if you are in the protection popup, the link goes to the protection settings page (or protection section of the settings page).
We also still need a mockup for the analysis page, which I think can be an extended or just bigger version of your Analysis Breakdown.

Additionally, if 3rd party sites are counted in the CCPA compliant ration (4/15 in the mockup), it could artificially inflate the denominator. This issue could be circumnavigated by using a modified counting code or removing the ratio altogether.

@kalicki1 or @OliverWang13, do you have suggestions or a mockup for the analysis page? I imagine something along the lines of the domain list page.

Here is my quick, roughly drawn mockup. I like the idea of having the domains with a drop down list that shows the specific information that is shown in the popup. @kalicki1 should also feel free to upload his own version.

@OliverWang13, that is a good! As you are showing it, we can reuse the code from the popup. On the left side, in addition to Settings and About, we would have Domain List and maybe Compliance List or something like that.

@OliverWang13, I am a fan of your options page mockup! I think we should shoot for something along those lines.
@SebastianZimmeck, about your original thoughts on my popup mockup,

Can users click on the analysis and protection labels? If so, what happens when they do?

I imagine them as buttons yes, and then assuming we have a perfect implementation, we can switch back and forth between analysis and protection mode as we discussed in our call.

Not sure if we need a separate Analysis Settings label/button.

I agree with what you said for the most part, I thought that the Analysis Settings button could link to the analysis-specific options page as you said. This button could be renamed something along the lines of "more detailed breakdown," implying that users can get more info about that specific domain's analysis and others as well (which is what we will do according to @OliverWang13's mockup).
@OliverWang13, about your points on my popup mockup,

Another idea is that possibly after one of the step fails, all following steps could become grayed out to illustrate the order

I like this idea, we could also make sure to only generate our actions responsively until a step fails, then load the conclusion.

I imagine them as buttons yes, and then assuming we have a perfect implementation, we can switch back and forth between analysis and protection mode ...

OK, yes, that would be great, indeed. Though, if it turns out to be too tricky, I am personally also fine with omitting the buttons and functionality. But, yes, give it a shot, and see how it goes.

I agree with what you said for the most part, I thought that the Analysis Settings button could link to the analysis-specific options page as you said.

OK, to keep it in the same style, in protection mode we should also have a button to the protection-specific options page (or domain list if we do not have a protection-specific options page).

I think that having the analysis and protection labels being buttons could be a little complicated. Depending on our implementation, I am unsure how smoothly we could make the transition between them. Additionally, a portion of our users will have no use or interest for the analysis mode, so perhaps the way to switch could be a little less convenient, if that makes sense. I could see some users accidentally switching modes without knowing the difference and disrupting the functionality of the extension.

Let's discuss this later today.

In terms of the switching from analysis to protection mode or vice versa, let's not implement the buttons and functionality immediately. Maybe, we will do it later.
In principle, the UI is good as is as shown in the mockup. Maybe, add some tooltips or do some smaller modifications.


move xds cache metrics to interface

To simplify if else conditions and only register metrics if they are enabled.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[X] Does not have any user-facing changes.  Let me change.

I will close this and open a separate PR in that repo first.


Incorrect type inference in 'match' expression

Reproducing the behavior
Considering the following program
type Box:
  Box {x: Unit}

type Unit:
  Unit

Erase: Box -> None
(Erase (Box/Box x)) = (Erase x)

main: None
main = (Erase (Box/Box Unit/Unit))

This should generate a type error in the Erase function, since it expects a Box value but we're giving it a Unit.
Based on the match expression, the type of Erase should be Box -> var, while according to the call it should be Unit -> var. When unifying these two we expect a type error, which never happens.
The final infered type is Box -> var, which specializes to Box -> None when checking against the annotation.
System Settings
Bend commit: 4d6f461d54e43c4aa9bfde14fdc611e235184129
Additional context
No response

The issue was that the type constructors inside of constructor fields were not being resolved and were kept as variables. So, even though the type of the Box/Box constructor was being updated to Unit -> Box with Unit as a constructor, the type of x was still Unit as a type variable.
This means that all fields inside of match expressions were being inferred variable types for the nullary constructors in their types.

Fixed by #728


[Translation] Russian translations

Thanks @Vermond125 :)

Thank you for this plugin, @flosch-hb ! =)


[suport esm client due to new node ssr develop mod and ts 4.7 native esm support]

Thansk to vite, we can easily using plugin like https://github.com/axe-me/vite-plugin-node to realize a HMR dev and test mod .
we can using vite ssr to realize:

dev mod have HMR.(by vite ssr)
test mod have HMR.(by vitest)
use vite plugin ecosystem to do something
prod mod have HMR(not yet)

but there are one caveat:
only support esm due to ssr
It's true we have some work around here,but it is better to have a native esm module for prisma.

Main issue for this: https://github.com/prisma/prisma/issues/5030


[Bug] failed to parse ABI-generated tokens into a Solidity AST: expected ,.

Component
Other (please provide more details)
What version of Alloy are you on?
forge 0.2.0 (0079a11 2024-09-10T00:20:21.629283000Z)
Operating System
macOS (Apple Silicon)
Describe the bug
I get this error:
Generating bindings for 201 contracts
Error:
failed to parse ABI-generated tokens into a Solidity AST: expected `,`.
This is a bug. We would appreciate a bug report: https://github.com/alloy-rs/core/issues/new/choose

When running forge bind in the root folder of this repo: https://github.com/erc7579/smartsessions/tree/main

https://github.com/alloy-rs/core/pull/747 fixes this bug, but the ABI will generate invalid Rust code because of https://github.com/alloy-rs/core/issues/744


Check new SSA code to see if sha256 regression bug is still present

Problem
PR #1340 contains a bug that is present in the current SSA. We should check in the new SSA code to see if this is also present.
Happy Case
.
Alternatives Considered
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

Closing as both sha256 test case and test from PR #1340 are passing.


HNC: prevent required children from being removed once added

Per the Phase 1 UJ, required children should be "permanent" - that is, once a namespace has been created as a child namespace, it cannot be removed (until we add Phase 2 features).
This might be a bit tricky since we basically need to compare the old and proposed version of the object in the validating webhook, which we haven't done yet.
This is a continuation of #90

Now that we have real authz checks, we don't need this anymore.


Add support for AssetMapper

The following issues on the eckinox/tinymce-bundle seem to be happening in projects that use AssetMapper:

https://github.com/eckinox/tinymce-bundle/issues/15
https://github.com/eckinox/tinymce-bundle/issues/13

The assets loaded in templates/form/tinymce_type.html.twig are referenced as follows:

{{ asset('bundles/tinymce/ext/tinymce/tinymce.min.js') }}
{{ asset('bundles/tinymce/ext/tinymce-webcomponent.js') }}

When AssetMapper is used, these might not actually point to the right path/URL anymore due to the bundling/versioning that is taking place, which might cause 404 or other errors.

So the main issue with AssetMapper I found with usage is that it adds versions to the JS filenames, but TinyMCE doesn't use standard imports, and instead tries to find the files itself based on the directory of the main script. Seeing as it doesn't know about the versioned filenames, it fails to find the right files and that leads to lots of 404s.
A potential solution would be to:

Check if AssetMapper is present and in use.
If so, exclude the assets from this bundle using the following config:framework:
    asset_mapper:
        excluded_patterns:
            - '*/bundles/tinymce/*'


JSX in an array does not format correctly.

Description
The results of beautification are not what I expect.
Code:
import React, {Component} from 'react';
import {Link} from 'react-router';
import {connect} from 'react-redux';

class Header extends Component {
    renderLinks() {
        if (this.props.authenticated) {
            return (

                    <Link className="nav-link" to="/signout">SignOut</Link>

            )

        } else {
            return [
 <Link className="nav-link" to="/signin">Sign in</Link>
,

<Link className="nav-link" to="/signup ">Sign Up</Link>
]

            }
        }
    }
    render() {
        return (

                <Link to="/" className="navbar-brand" href="">

                            T-STORE

                </Link>

                    {this.renderLinks()}

        )
    }
}

function mapStateToProps(state) {
    return {authenticated: state.auth.authenticated}
}

export default connect(mapStateToProps)(Header);

Expected Results
The beautified code should have looked like:
import React, {Component} from 'react';
import {Link} from 'react-router';
import {connect} from 'react-redux';

class Header extends Component {
    renderLinks() {
        if (this.props.authenticated) {
            return (

                    <Link className="nav-link" to="/signout">SignOut</Link>

            )

        } else {
            return [

                     <Link className="nav-link" to="/signin">Sign in</Link>

,

                     <Link className="nav-link" to="/signup">Sign Up</Link>

             ]
            }
        }
    }
    render() {
        return (

                <Link to="/" className="navbar-brand" href="">

                            T-STORE

                </Link>

                    {this.renderLinks()}

        )
    }
}

function mapStateToProps(state) {
    return {authenticated: state.auth.authenticated}
}

export default connect(mapStateToProps)(Header);

Steps to Reproduce

Add code to Atom editor
Save code (As we have setting beautify on save)
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist: https://gist.github.com/sujandthakare/dddd2438b5261b176c2d301058cf17d4

This is a Pretty Diff defect that I can reproduce.  I am working it now.

The defect is at https://github.com/prettydiff/prettydiff/blob/master/lib/jspretty.js#L2084
if (token[d] === "return" || types[d] === "operator" || token[d] === "(") {

should become
if (token[d] === "return" || types[d] === "operator" || types[d] === "start" || types[d] === "separator") {

This will open up JSX syntax to wider interpretation inside the parser.  This opens up some beautification regression that I am looking into now.  Even with this code fix the above code will still fail because of line:

Pretty Diff considers white space between the opening delimiter and tag name to be invalid XML and instead interprets the < as a JavaScript less than operator.

I also encountered a weird issue, if you type JSX tag and hit save without completing the tag sometimes it adds text at the end of the file. I am trying to reproduce exact case .

+1
I am trying to get AtomBeautify work for JSX for some days now. Unfortunately, these kind of bugs prevent me from using it for JSX :-/ Yet, I am addicted to this package, I don't have any clue on how to proceed now.

The latest version of Pretty Diff recently landed in Atom Beautify which should resolve this issue.  If the defect remains please reopen this issue.


Moving marker to click location

I'm trying to move a single marker on the map to where the user clicks on the map but I'm struggling. With every click (st_folium) the marker is moving to the previous click location instead of the current click location. What I am doing wrong here?
# State variables
if 'center' not in st.session_state:
    st.session_state.center = [45.503032, -73.566424]

if 'zoom' not in st.session_state:
    st.session_state.zoom = 15

if 'location' not in st.session_state:
    st.session_state.location = folium.Marker(st.session_state.center)

# Map creation
m = folium.Map(location=st.session_state.center, zoom_start=st.session_state.zoom)
fg = folium.FeatureGroup(name="Markers")
fg.add_child(st.session_state.location)

# When the user interacts with the map
map_state_change = st_folium(
      m,
      feature_group_to_add=fg,
      height=400,
      width='100%',
      returned_objects=['last_clicked', 'zoom', 'bounds', 'center'],
)

# If the interaction includes a click
if map_state_change['last_clicked']:
    loc = map_state_change['last_clicked']
    st.session_state.location = folium.Marker([loc['lat'], loc['lng']])

I am currently struggling with the same issue.
Adding a button, that forces a zoom change is a work-around.
Were you able to find a better solution? @adolmajian


HDDS-1803. shellcheck.sh does not work on Mac

What changes were proposed in this pull request?

Filter for file permission on Mac.
Merge two separate find calls to avoid overwriting output (and eliminate code duplication).

https://issues.apache.org/jira/browse/HDDS-1803
How was this patch tested?
$ hadoop-ozone/dev-support/checks/shellcheck.sh | wc
     133     600    6065

$ wc target/shell-problems.txt
     133     600    6065 target/shell-problems.txt

/label ozone

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
71
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
0
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
492
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
843
branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1
mvninstall
444
the patch passed

+1
mvnsite
0
the patch passed

+1
shellcheck
1
There were no new shellcheck issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
739
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
unit
115
hadoop-hdds in the patch passed.

+1
unit
193
hadoop-ozone in the patch passed.

+1
asflicense
44
The patch does not generate ASF License warnings.

3141

Subsystem
Report/Notes

Docker
Client=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1102/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1102

Optional Tests
dupname asflicense mvnsite unit shellcheck shelldocs

uname
Linux 773ae44f6140 4.15.0-48-generic #51-Ubuntu SMP Wed Apr 3 08:28:49 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / f77d54c

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1102/1/testReport/

Max. process+thread count
307 (vs. ulimit of 5500)

modules
C: hadoop-ozone U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1102/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.4.6

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
74
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
0
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
516
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
888
branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1
mvninstall
497
the patch passed

+1
mvnsite
0
the patch passed

+1
shellcheck
0
There were no new shellcheck issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
772
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
unit
107
hadoop-hdds in the patch passed.

+1
unit
185
hadoop-ozone in the patch passed.

+1
asflicense
44
The patch does not generate ASF License warnings.

3285

Subsystem
Report/Notes

Docker
Client=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1102/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1102

Optional Tests
dupname asflicense mvnsite unit shellcheck shelldocs

uname
Linux 2c17343b881a 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / c58e11b

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1102/2/testReport/

Max. process+thread count
306 (vs. ulimit of 5500)

modules
C: hadoop-ozone U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1102/2/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.4.6

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
37
Docker mode activated.

_ Prechecks _

+1
dupname
1
No case conflicting files found.

0
shelldocs
1
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
551
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
777
branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1
mvninstall
471
the patch passed

+1
mvnsite
0
the patch passed

+1
shellcheck
0
There were no new shellcheck issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
673
patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1
unit
98
hadoop-hdds in the patch passed.

+1
unit
170
hadoop-ozone in the patch passed.

+1
asflicense
40
The patch does not generate ASF License warnings.

2989

Subsystem
Report/Notes

Docker
Client=18.09.8 Server=18.09.8 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1102/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1102

Optional Tests
dupname asflicense mvnsite unit shellcheck shelldocs

uname
Linux 054075b12d03 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 4e66cb9

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1102/3/testReport/

Max. process+thread count
412 (vs. ulimit of 5500)

modules
C: hadoop-ozone U: hadoop-ozone

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1102/3/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.4.6

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

Thank you for the patch. I have committed this to trunk and ozone-0.4.1


0.2.18 new version can not working in apline linux

alpine linux 3.19
0.2.18 gnu and musl error log as below
0.2.16 gnu also have same problem but musl working
==============================================================
./tun2proxy_musl -s -p "socks5://<IP_ADDRESS>:10808" -t tun --dns over-tcp --dns-add
r <IP_ADDRESS>-b x.x.x.x
thread 'tokio-runtime-worker' panicked at /cargo/registry/src/index.crates.io-6f17d22bba15001f/tproxy-config-5.0.1/src/linux.rs:147:67:
called `Result::unwrap()` on an `Err` value: AddrParseError(Ip)
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
[2024-05-08T09:41:08Z ERROR tun2proxy] main_entry error task 3 panicked
==============================================================
./tun2proxy_gnu -s -p "socks5://<IP_ADDRESS>:10808" -t tun --dns over-tcp --dns-add
r <IP_ADDRESS> -b x.x.x.x
-ash: ./tun2proxy_gnu: not found

post your resulets here with this comands:
sudo ip -4 route show

and
sudo ip -6 route show

There is a space missing before -b.

ip -4 route show
default via <IP_ADDRESS> dev eth0  metric 1 onlink
<IP_ADDRESS>/24 dev eth1 scope link  src <IP_ADDRESS>
<IP_ADDRESS>/24 dev eth0 scope link  src <IP_ADDRESS>
ip -6 route show
fe80::/64 dev eth1  metric 256
fe80::/64 dev eth0  metric 256
multicast ff00::/8 dev eth1  metric 256
multicast ff00::/8 dev eth0  metric 256
===================================
sorry -b is my typing problem

post your resulets here with this commands:
sudo ip -4 route show

and
sudo ip -6 route show

ip -4 route show
default via <IP_ADDRESS> dev eth0  metric 1 onlink
<IP_ADDRESS>/24 dev eth1 scope link  src <IP_ADDRESS>
<IP_ADDRESS>/24 dev eth0 scope link  src <IP_ADDRESS>
ip -6 route show
fe80::/64 dev eth1  metric 256
fe80::/64 dev eth0  metric 256
multicast ff00::/8 dev eth1  metric 256
multicast ff00::/8 dev eth0  metric 256

There is a space missing before -b.

my typing problem in github

Testing the new version 0.2.19

Sorry, I have closed this too early.

still can not working. can u test alpine 3.19?
only 0.2.16 musl work
alipine default ash:
./tun2proxy-musl.0.2.19 -p socks5://<IP_ADDRESS>:10808 -tun tun -s --dns virtual -b xxx.xxx.xxx.xxx
-ash: ./tun2proxy-musl.0.2.19: not found
./tun2proxy-gnu.0.2.19 -p socks5://<IP_ADDRESS>:10808 -tun tun -s --dns virtual -b xxx.xxx.xxx.xxx
-ash: ./tun2proxy-musl.0.2.19: not found
bash:
./tun2proxy-musl.0.2.19 -p socks5://<IP_ADDRESS>:10808 -tun tun -s --dns virtual -b xxx.xxx.xxx.xxx
error: the following required arguments were not provided:
--unshare
Usage: tun2proxy-musl.0.2.19 --proxy  --unshare --tun  --setup --dns  --bypass <IP/CIDR> <ADMIN_COMMAND>...
For more information, try '--help'.
./tun2proxy-gnu.0.2.19 -p socks5://<IP_ADDRESS>:10808 -tun tun -s --dns virtual -b xxx.xxx.xxx.xxx
bash: ./tun2proxy-gnu.0.2.19: cannot execute: required file not found

This time, it is probably really due to a typing mistake. The --tun argument is missing a second dash.

This time, it is probably really due to a typing mistake. The --tun argument is missing a second dash.

yes I change to --tun work now ,thanks
======================================
Options:
-p, --proxy             Proxy URL in the form proto://[username[:password]@]host:port, where proto is one of socks4,
socks5, http. For example: socks5://myname:password@<IP_ADDRESS>:1080
-t, --tun              Name of the tun interface, such as tun0, utun4, etc. If this option is not provided, the OS will
generate a random one


policy_type variable undefined

Describe the bug
When I run run.sh script, task Create policy for Access Control shows error The error was: 'policy_type' is undefined
To Reproduce

Prepare a cluster by Minikube with docker driver
Prepare Vault by Helm
Follow instruction to setup build folder
Build docker image and run container

Expected behavior
Go to the next step
Screenshots

Environment (please complete the following information):

OS: Ubuntu server with public ip
Version 22.04 LTS

Additional context
I follow the instruction in the tutorial folder to setup environment

As the log says, the policy_templates missing policy_type as below
policy_templates:         quorum: quorum-policy.tpl         besu: besu-policy.tpl         cordaos: cordaos-policy.tpl         fabric: fabric-policy.tpl         r3-corda-ent: r3-corda-ent-policy.tpl
This config doesn't contain policy_type field, I also cannot find the fabric-policy.tmp file in anywhere

the policy_type variable is defined in the calling role. In your case, I assume you trying to deploy Fabric network. The policy_type is defined here
https://github.com/hyperledger/bevel/blob/487f659b7e3637db3f98383b50ebe16b158392c8/platforms/hyperledger-fabric/configuration/deploy-network.yaml#L62

What branch/tag are you using ?

I saw policy_type in this file before, but the log says it is not defined, so how do I fix this error? Before the latest release, I have never seen this error 😗

What branch/tag are you using ?

I use develop branch with the latest code

I just check the code in main branch and everthing is okay, the issue only appears at develop branch

Looks like valid issue on dev branch

Today I try to run network with minikube driver=none, which means I use directly the host to make a cluster and the issue apprears after this config, and I also use the latest code at main branch.
@suvajit-sarkar

I am working on this


can't load xy / xye files that contain meta data in headers

I think the gui can only load xrd files that have no header lines. Many files exported from lab instruments or from anywhere else contain some metadata in the header, which is often quite useful to keep.
Thanks for this very useful tool!
Till

diffpy.utils is a useful package to handle different dataformats. It has a loadData() function which might be useful.

can you send me an example file so I can look into it?

is the metadata in your file marked with a '#'?

It looks like that:
[Diffraction_object]
name = Absorption corrected input_data: SU101_40-1_BiMOF_500umSpezGlass_accum_0
wavelength = 0.71
scat_quantity = x-ray
mud = 2.1792017503005123
anode_type = Mo
wavelength = 0.71
output_directory = C:\Users\tills\PDF\bea\bimof\data\lab\su101_500umSpezGlass_corrected.xy
xtype = tth
method = polynomial_interpolation
username = till
email =<EMAIL_ADDRESS>z_scan_file = C:\Users\tills\PDF\bea\bimof\data\lab\SU101_40-1_BiMOF_500um_spec_glass_exported.xy
package_info = {'diffpy.labpdfproc': '0.2.0', 'diffpy.utils': '3.5.0'}
input_directory = C:\Users\tills\PDF\bea\bimof\data\lab\SU101_40-1_BiMOF_500umSpezGlass_accum_0.xy
creation_time = 2024-11-08 09:50:33.820075
start data
2.000099897384640002e+00 5.779828003216881598e+02
2.020537221804259875e+00 5.662578146175662823e+02
2.040974546223880193e+00 5.571583334011457964e+02

The start data has #### in front of it

if I mark all meta infromation with # it will work?

I've just used np.loadtxt(filename, unpack=True, comments='#'). So if every line of metadata starts with '#', including the axis names, it should work.

Oh okay I haven't realized that. Perfect then I'll just do that. thanks!

Do you have a lot of files? Will it be annoying to modify all of them?


[DNM] Test job hash issue

Related-Issue: OSPRH-7480
Depends-On: https://github.com/openstack-k8s-operators/ovn-operator/pull/302

recheck with fix

just for test


[BUG] Unit tests ToPrettyStringSuite FAILED on spark-3.5.0

Describe the bug
Unit tests ToPrettyStringSuite FAILED on spark-3.5.0, details:
 ToPrettyStringSuite:
 - test show() on booleans
 - test show() on bytes
 - test show() on shorts
 - test show() on ints
 - test show() on longs
 - test show() on floats *** FAILED ***
   Expected exception org.scalatest.exceptions.TestFailedException to be thrown, but no exception was thrown (ToPrettyStringSuite.scala:83)
 - test show() on doubles *** FAILED ***
   Expected exception org.scalatest.exceptions.TestFailedException to be thrown, but no exception was thrown (ToPrettyStringSuite.scala:89)
 - test show() on strings

With https://github.com/NVIDIA/spark-rapids/issues/4204 being fixed. The test has started to fail, we just need to remove the assertThrows.


Update API.md

Summary
Update API.md examples.
Test plan
use drinkFlavor method instead of 'drink' on 'toThrow'/toThrowError/toThrowErrorMatchingSnapshot

Current coverage is 87.77% (diff: 100%)

Merging #1976 into master will not change coverage

@@             master      #1976   diff @@
==========================================
  Files            38         38
  Lines          1252       1252
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1099       1099
  Misses          153        153
  Partials          0          0

Powered by Codecov. Last update 891ee5a...8b9b6c7

thanks!


Add link to live demo (CDN)

I'm not pretty sure about the link text, should it be "See it in action!" or "Live demo" or "Demo" or something else?
cc @jouni

This change is

“Live demo” is probably the thing my eyes would be looking for.

“Live demo” is probably the thing my eyes would be looking for.

Done.


Add Universal Link support

This is dependent on https://github.com/5calls/5calls/issues/75 to request issue details for a specific issue id.

(this is currently blocked on a server feature)

Working on this, it's pretty straightforward


Bug Abuse

Hey there Is a mighty flaw in that whole script. Cops could just open their own lockers everywhere filled with like 100 armors and would never run outta ammo

True.
Would probably be best if I implement that cops can only access the locker when they're in the station? (or wherever defined in config)
Or I can add banned items that can't be placed in a locked?
I might go for the first one.

Yea maybe add like a target that opens a qb-input menu where you enter the ID for the locker or like you said only in specific areas

Will work on both options


Links need distiguishing

What would you like to share?
On the homepage there are two text links: Eddie Jaoude and instructions. However, they are not visually marked as links aside from color which means people who don't perceive color can't tell that they are links. The links should stand out from the rest of the text either with underlines or bold text or some other styling that indicates that these are clickable.
Additional information
No response

Yes, a little bit of focus needs to find out that they are links for normal people too.


Pre commit hook failure

When I want to commit, I receive the message

.git/hooks/pre-commit: line 13: /nix/store/8n7a65jpm6nbz992jkplv2qi2xhbnp1m-python3.10-pre-commit-2.20.0/bin/pre-commit: No such file or directory

To reproduce
$ touch foo
$ git add foo
$ git commit

Environment

Void linux, with the nix pkg manager
Master branch, after 5808c9f4992344019741d627a0cd292473eb1653

Just to be sure that we reset the whole thing, can you:
$ rm .git/hooks/*
$ rm .pre-commit-config.yaml
$ direnv reload

(or leave and enter nix develop again if you don't rely on direnv)
and then try again?

I think it is actually a known problem of cachix/pre-commit-hooks.nix: it is good at installing hooks, but not so good at uninstalling them, so you end up in inconsistent states if you're not in a world managed by cachix/pre-commit-hooks.nix.


Feature/#313 autosave local storage step1

Closed #313

closed to do a new one, with vnext updated


fix: use only one thread for MELT

Coverage: 84.273%. Remained the same when pulling 6bb91c22fafc514b73e5664d6e435045de4a5575 on fix-melt-only-one-thread into 13ee84b60bdf3e168d8d3dea55425314decf59f6 on main.


Fix target_path in drioless mode

afl-showmap show the wrong target_path in drioless mode.

I signed it!

Looks good, thanks for the fix!


Expand tests for CLI options

Given a specific config,
We can guarantee that we want a specific command passed to the command line (e.g. phantomjs --arg1 url widths etc etc).
This is fundamental to the working of Wraith so we should have plenty of tests testing the different expected command line arguments.

Completed in release 3.1.0


Add HCS storage APIs

Add bindings for the HCS storage APIs from computestorage.dll

Signed-off-by: Daniel Canter<EMAIL_ADDRESS>

Can you add information on what this is used for and how the bindings will be used in the future?

@katiewasnothere Yep. Truth be told I opened the PR and went on an errand 😃

For the immediate future these bindings will be used to create a cache of scratch layers of different sizes to be used if a user asks for a scratch layer of a different size than the default that is made by the graph driver APIs (20GB). If the size requested is not in the cache, we will make a new scratch layer of that size using these binds as well.
For the future, we may move containerd off of the old graphdriver (wclayer) calls and onto these for the windows snapshotter. No scenarios come to mind, but it gives more freedom in how layers are managed and especially how the writable layers are handled.

One small suggestion but LGTM otherwise!

Are these not in internal so that the snapshotter can use them?

Are these not in internal so that the snapshotter can use them?

Correct


Upgrade to Rebar3, additions to publish to Hex.pm

Transferred from accense/eini#3 I believe that a new Hex release after this should fix erlcloud/erlcloud#307
Uses rebar3 as the build tool, but with a rebar.config.script for
rebar2 backwards-compatibility.
A few bits filled in on the .app.src file for Hex.pm publishing.

going to merge this and tag 1.2.2

oh...i see it's a development branch and not master. @waisbrot please merge/FF to master to it and tag it so to be used in hex and finally in erlcloud.

I made master the main branch and deleted development.
I made the 1.2.2 tag and put it on Hex. Trying to fetch it into erlcoud, it looks OK to me (it doesn't grab the test deps).
I've added @motobob to the Hex package owner list.


Material-ui 4.11.4 - Grid: items in column require container direction='row' to display correctly (??)

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
I want to have several div's in column, centered on the page axis and stretched in width as much as the div width allows it. I am using the <Grid> component (both container and item) of material-ui 4.11.4 (latest version):

In the previous image all Grid items have xs={12}, while "page body" has xs={10}.
The container should align & stretch its children vertically through the prop direction='column'.
See the desired layout + CSS params in this snippet from FlexyBoxes.com:

I'd expect that the following container would do the job:
<Grid container direction='column' justify='center' alignItems='stretch'>
    <Grid item xs={12}>TOP NAVIGATION</Grid>
    <Grid item xs={12}>HEADER</Grid>
    <Grid item xs={10}>PAGE BODY</Grid>
    <Grid item xs={12}>FOOTER</Grid>
</Grid>

This is also consistent with the Grid interactive demo
Current Behavior 😯
However, if I want the effect shown in the first image, I have to give the container a property direction='row' (?!?!?).
If I give direction='column', all grid items are aligned on the left side of the screen:

Steps to Reproduce 🕹
Codesandbox is here:
To make things worse, I am unable to obtain any thinner <Grid item xs={10}> in the sandbox.
Context 🔦
I wish to have a page where all content is vertically ordered and centered+stretched along the central vertical axis without using style and only using Grid and its properties.

If a <div> is 100% wide, I put <Grid item xs={12}> and it shall take the whole width.
If a <div> is less than 100% wide, I put <Grid item xs={10}> (or another value less than 12) and it shall center+stretch+align with the central vertical axis of the page.

Here is the wished result again:

Your Environment 🌎

`npx @material-ui/envinfo`
  System:
    OS: Windows 10 10.0.18363
  Binaries:
    Node: 14.15.4 - C:\Program Files\nodejs\node.EXE
    Yarn: Not Found
    npm: 6.14.10 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 88.0.4324.104 <--- USING THIS!!
    Edge: Spartan (44.18362.449.0)
  npmPackages:
    @emotion/styled:  10.0.27
    @hot-loader/react-dom:  16.14.0
    @material-ui/core: ^4.10.2 => 4.11.3
    @material-ui/icons: ^4.9.1 => 4.11.2
    @material-ui/lab: ^4.0.0-alpha.56 => 4.0.0-alpha.57
    @material-ui/pickers: ^3.2.10 => 3.2.10
    @material-ui/styles:  4.11.3
    @material-ui/system:  4.11.3
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.11.2
    @types/react:  17.0.0
    react: ^16.13.1 => 16.14.0

This is a bug, but whatever...


[tlgen.py] ERROR: get_s1n_if_exist hits DEVICE type (unexpected) for node: flash_ctrl

Description
connections: {
ibex_corei: ["flash_ctrl"],
ibex_cored: ["sram_ctrl", "flash_ctrl", "adc_ctrl", "dac_ctrl", "i2c_slv"],
i2c_mas: ["sram_ctrl", "flash_ctrl", "adc_ctrl", "dac_ctrl", "i2c_slv"]
},
ERROR: get_s1n_if_exist hits DEVICE type (unexpected)
If only one device in the connections, the aforementioned ERROR occurs.

This is a duplicate and is already tracked here #2337.
We will likely be able to fix this soon.


Please add the no sleep, and laptop lid settings

I've been trying to transplant the settings i have from the schneegans xml i made that makes the computer not go to sleep and keeps the monitor on, and changes the lid settings to do nothing.
I can't for the life of me get this to not error out when i try to add it to your standard install.
just a simple powercfg script to get added.

Powercfg /Change monitor-timeout-ac 60
Powercfg /Change monitor-timeout-dc 0
Powercfg /Change standby-timeout-ac 0
Powercfg /Change standby-timeout-dc 0

Have you been able to fix this issue?
In v2.0.0 linked below, I have reworked the Power Settings part of the file https://github.com/memstechtips/UnattendedWinstall/releases/tag/v2.0.0
Feel free to reopen the issue if it's not fixed.


[Feature request]: Add Readme file to NuGet package

Is your feature request related to a specific problem? Or an existing feature?
Add Readme file to NuGet package
Describe the solution you'd like
Add Readme file to NuGet package
Additional context
No response

Resolved at #1


[Gandiva] How to add custom native functions?

Does gandiva support user-defined functions? How to add functions like
sqrt(float), abs(float), mean(list), first(list) without modify gandiva itself?

I've started a thread around this on the mailing list so that information is easily available/searchable. Please let us continue the discussion on the same.

Where can I get further information?


To close all activities opened during test you need to invoke finishOpenedActivities() twice.

From<EMAIL_ADDRESS>on February 07, 2012 05:23:03
What steps will reproduce the problem? 1. Make a simple layout with just EditText in it.

Create a class extended from ActivityInstrumentationTestCase2<> in which you create a test method which is just clear the EditText:
solo.clearEditText(0);
This is required to show soft keyboard.

In your tearDown() method invoke:
solo.finishOpenedActivities(); What is the expected output? What do you see instead? Expected output: all opened during test activities need to be finished.

Actual result: The soft keyboard hides. In order to close all opened activities you need to invoke the finishOpenedActivities() once again. What version of the product are you using? On what operating system? ICS 4.0.3 Please provide any additional information below.
Original issue: http://code.google.com/p/robotium/issues/detail?id=224

From<EMAIL_ADDRESS>on February 07, 2012 21:37:48
Unfortunately there is nothing to do about this. Robotium does not have access to the keyboard and therefore it does not know  if it is up or not. Therefore it can not change behaviour based on if the keyboard is up or not. You will need to use solo.goBack() or similar to make the keyboard dissappear and then use finishOpenedActivities().
Status: WontFix


Meaningful and shorter names for EnergySolvers

Current names are too long. The extreme being EnergySolver_ThresholdGoalLeaning.
I think we can have sth like Basic_ES on one side and Threshold_ES on the other side. I don't think EnergySolver is needed everywhere as all names are imported from the module energy_solver anyway.
Suggestions welcomed.
This would break some work in other repos using the heuristics already. Shall we introduce some aliases?

Let's merge GoalLeaning and GoalLeaningThreshold into 1.
Suggested names:

BasicES
GoalLeaningES

Closed by a7150b8


vctl pull mono

Any hints ?
Seans-Mac-Pro:~ Sean__Wong$ vctl system start
Preparing storage...
Container storage has been prepared successfully under /Users/Sean__Wong/.vctl/storage
Preparing container network, you may be prompted to input password for administrative operations...
Password:
Container network has been prepared successfully using vmnet: vmnet9
Launching container runtime...
Container runtime has been started.
Seans-Mac-Pro:~ Sean__Wong$ vctl pull mono
INFO Pulling from index.docker.io/library/mono:latest
───                                                                                ──────   ────────
REF                                                                                STATUS   PROGRESS
───                                                                                ──────   ────────
index-sha256:e6e25d424d464706fdaeb058dae2a9d7690ec2773168d9fd2a1aa689b169be4b      Done     100% (1431/1431)
manifest-sha256:18384bd31a4d16c228a893267a7943b7aea1f263f75ed23cd9a139e5cd844b1b   Done     100% (1164/1164)
layer-sha256:68ccfe204fbb0bc2d09a50ebeab14fc143f44a9a8fbf474784b0bb3b83f805a6      Done     100% (130288603/130288603)
layer-sha256:bf59529304463f62efa7179fa1a32718a611528cc4ce9f30c0d1bbc6724ec3fb      Done     100% (27092121/27092121)
config-sha256:ecda9bfba7e60c3f947af76aa7f409fcb0ba0016af425a2c9469882c4cf09f2c     Done     100% (3350/3350)
layer-sha256:1b84988e9412eb7cdedbd0cd734d319d6e7694cf8b8c6b50383a7d9d0bd65408      Done     100% (67092668/67092668)
layer-sha256:f9430cf70b26259617fdf6dea56de0841e6a2d05e7996d61d9b457d837624f2c      Done     100% (255895/255895)
INFO Unpacking mono:latest...
ERROR failed to prepare extraction snapshot "extract-865500000-ZU8W sha256:d0f104dc0a1f9c744b65b23b3fd4d4d3236b4656e67f776fe13f8ad8423b955c": failed to create temp dmg: exit status 1: unknown

Might be that the vctl version with Fusion 11.5.x currently has a bug, I am on the fusion 12 version and it's working no probs.
What do your containerd logs say?  (~/.vctl/containerd.log)
mike@OctoBook  ~  vctl pull mono
INFO Pulling from index.docker.io/library/mono:latest
───                                                                                ──────   ────────
REF                                                                                STATUS   PROGRESS
───                                                                                ──────   ────────
index-sha256:e6e25d424d464706fdaeb058dae2a9d7690ec2773168d9fd2a1aa689b169be4b      Done     100% (1431/1431)
manifest-sha256:18384bd31a4d16c228a893267a7943b7aea1f263f75ed23cd9a139e5cd844b1b   Done     100% (1164/1164)
layer-sha256:68ccfe204fbb0bc2d09a50ebeab14fc143f44a9a8fbf474784b0bb3b83f805a6      Done     100% (130288603/130288603)
layer-sha256:bf59529304463f62efa7179fa1a32718a611528cc4ce9f30c0d1bbc6724ec3fb      Done     100% (27092121/27092121)
layer-sha256:f9430cf70b26259617fdf6dea56de0841e6a2d05e7996d61d9b457d837624f2c      Done     100% (255895/255895)
config-sha256:ecda9bfba7e60c3f947af76aa7f409fcb0ba0016af425a2c9469882c4cf09f2c     Done     100% (3350/3350)
layer-sha256:1b84988e9412eb7cdedbd0cd734d319d6e7694cf8b8c6b50383a7d9d0bd65408      Done     100% (67092668/67092668)
INFO Unpacking mono:latest...
INFO done
mike@OctoBook  ~  vctl run -n myMono -t -d mono:latest
INFO container myMono started and detached from current session
mike@OctoBook  ~  vctl ps
────     ─────         ───────   ──               ─────   ──────    ─────────────
NAME     IMAGE         COMMAND   IP               PORTS   STATUS    CREATION TIME
────     ─────         ───────   ──               ─────   ──────    ─────────────
myMono   mono:latest   bash      <IP_ADDRESS>   n/a     running   2020-08-31T17:11:01-07:00
mike@OctoBook  ~ 

@mikeroySoft  Ireally appreciate your reply, I gave up and went back to Docker. I'll look at it again ITF.


Can't specify schema on tables for redshift

Hello, I'm trying to use the following config file to run the diff between two tables on different schemas on our Redshift cluster but I can't find the proper way to do this. I tried the table with the schema and specifying the schema on the database definition:
This returns psycopg2.ProgrammingError: invalid dsn: invalid connection option "schema"
[database.test_redshift_source]
driver = "redshift"
user = "XXXX"
password = "XXXX"
database = "XXXXX"
host = "XXXXX"
port = 5439
schema="schema_one"

[database.test_redshift_target]
driver = "redshift"
user = "XXXX"
password = "XXXX"
database = "XXXXX"
host = "XXXXX"
port = 5439
schema="schema_two"

[run.default]
update_column = "updated_at"
verbose = true
max_age = "3days"
min_age = "3days"

[run.test_diff]
1.database = "test_redshift_source"
1.table = "table"
2.database = "test_redshift_target"
2.table = "table"

This returns Table 'schema_two.table' does not exist, or has no columns
[database.test_redshift_source]
driver = "redshift"
user = "XXXX"
password = "XXXX"
database = "XXXXX"
host = "XXXXX"
port = 5439

[database.test_redshift_target]
driver = "redshift"
user = "XXXX"
password = "XXXX"
database = "XXXXX"
host = "XXXXX"
port = 5439

[run.default]
update_column = "updated_at"
verbose = true
max_age = "3days"
min_age = "3days"

[run.test_diff]
1.database = "test_redshift_source"
1.table = "schema_one.table"
2.database = "test_redshift_target"
2.table = "schema_two.table"

I tried looking at the docs, the code and the Python API reference but I couldn't figure out how to make it work :(

Can you please try again, but instead of
schema="schema_one"

Write
[database.test_redshift_source]
database="schema_one"
...

[database.test_redshift_target]
database="schema_one"
...

I know it's counter-intuitive, but I think that's the keyword that psycopg2 expects.

@erezsh I tried but this:
  File "/Users/pedrovidotti/.pyenv/versions/3.7.5/lib/python3.7/concurrent/futures/thread.py", line 69, in _worker
    initializer(*initargs)
  File "/Users/pedrovidotti/.pyenv/versions/3.7.5/envs/data-diff/lib/python3.7/site-packages/data_diff/databases/base.py", line 274, in set_conn
    self.thread_local.conn = self.create_connection()
  File "/Users/pedrovidotti/.pyenv/versions/3.7.5/envs/data-diff/lib/python3.7/site-packages/data_diff/databases/postgresql.py", line 62, in create_connection
    raise ConnectError(*e.args) from e
data_diff.databases.base.ConnectError: FATAL:  database "schema_one" does not exist

@pedrosmv Sorry for the delay.
I tested this exact file with the latest data-diff version, and it connects to the database successfully. (says Table 'table' does not exist).
Are you sure schema_one exists in your database instance?
If so, it would really help if you could try to connect to it directly using the psycopg2 package. If that works, we'll know the problem is in data-diff.