AssertionError

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Modin version (modin.__version__): 0.10.0
Python version: 3.8.10 64-bit
Code we can use to reproduce:

To run the test, clone the repository, install the dependencies, activate the installed virtual environment, and run the command:
  set MODIN_EXPERIMENTAL=1
  set MODIN_ENGINE=Python
  python -m pytest --simulate-cloud=normal modin/pandas/test/dataframe/test_map_metadata.py::test_drop_duplicates[None-last-no_duplicates]

Describe the problem
Occurs in 47 tests.
modin/pandas/test/dataframe/test_map_metadata.py::test_drop_duplicates[None-last-no_duplicates]
modin/pandas/test/dataframe/test_map_metadata.py::test_drop_duplicates[None-last-all_duplicates]
modin/pandas/test/dataframe/test_map_metadata.py::test_drop_duplicates[None-last-some_duplicates]
And also in 44 tests.
Source code / logs
Part's traceback:
    modin\pandas\test\dataframe\test_map_metadata.py:728: in test_drop_duplicates
        df_equals(
    modin\pandas\test\utils.py:527: in df_equals
        assert_frame_equal(
    E   AssertionError

The error is not reproduced in modin 0.10.1. All the specified tests are successfully executed.
The tracker is closed.


Calling mkFlake iteratively

Hello, I'm trying to refactor my flake into was is effectively subflakes (though with shared lock/inputs). I'm basically just trying not to have to deal wth multiple git repositories if I can avoid it, due to the added maintainance overhead that brings. I planned to effectively make these "stages" such that I have 00-base, and 01-nixos. My plan was to do something like this (pseudocode):
let
  base = mkFlake { inherit inputs; } (import ./01-base);
  nixos = mkFlake { inputs = inputs // { inherit base; }; } (import ./02-nixos);
in
  mkFlake { inputs = inputs // { inherit base nixos; }; } (import ./config)

However, this very quickly leads to infinite recursion. Another thing that's stumped me, is that while I can do { home-manager, ... }: {}  as a module in all 3 mkFlake invocations, I can't do { base, ... }: {}.
I've gone through the issues here that deal with dogfooding, and while that's pretty similar, I would really like to be able to produce multiple stages to make the dependencies much more explicit (and hopefully enforced). I'm however unable to get something like this to work, so I need some guidance.

Somewhat off topic, but relevant is to avoid import

 (import ./01-base)

It's better to do just pass ./01-base, because import drops the file name. Errors will be a little better when you don't call import. If you need to pass extra arguments, call flake-parts-lib.importApply file args instead; that also preserves the file name.
As for the real problem, I don't really see it.
I probably wouldn't bother with the staged approach, but I don't see a fundamental problem with it.
Kind of hard to see what's wrong without the code, but maybe the error trace reveals something. The module system tends to add some relevant context via custom trace entries. (And Nix 2.24 shows those unconditionally.)
If you also pass --show-trace it gets a lot more verbose, but occassionally that's necessary.

Sorry, my head is still not able to fully wrap around the concept of fixed points and calling functions with their own output. I think the issue I had was that all invocations to mkFlake were using the same outputs, but I think I've managed to create something that solves that. I've created a pretty small poc and would be really appreciative of a sanity-check: https://github.com/Alxandr/nix-flake-poc.
If this is a reasonable approach, maybe it makes sense to add something similar to flake-parts?


Java objects in Designer

Accessing underlying java objects will soon be deprecated. Update these sections.

Right now there is a scroll bar bug in the table of block trials. Basically, anytime you update information in the table, the scroll bar jumps back to the top of the table, which is very annoying when you're trying to edit conditions further down. We get around this by accessing the scroll bar's java object, getting its position, then manually resetting it each time the table updates.
According to mathworks documentation here this bug can only be avoided by using their newer uifigure tools. It has instructions on how to convert figure based GUI applications to uifigure based applications. I will start working on making this conversion, but it'll be a longer term thing, as it will take a while, and I'm sure there are functionalities we use now that will be different or unavailble with uifigure. I think uifigure was available when I started this application, though it had less functionality than it does now in the current release. I remember that we specificially decided against using it but I don't remember why. Sooner or later it will have to be done though, as javaframe and other java properties will be deprecated.

Completed pending testing. I'm afraid with the new UIFigure based application, the tables render more slowly and might be frustrating for the user, but I'm not sure. This issue is noticeable on my laptop, but not as noticeable on the PC. Laura is going to do some testing with it and let me know how it does. Researching ways to speed up the table rendering in the meantime.

Not moved to master branch yet, going to use it for a while first and make sure no more bugs pop up, but it is finished and working.


Add double quant example

Type of Change
feature
Description
based on 3.x api
"--double_quant BNB" is used to enable double quant algorithms,  choices: ['GGML_TYPE_Q4_K', 'BNB']
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
local test

model/double_quant
None
BNB/GGML
2.x

gpt-6b-j
xx
xx
0.6823

Llama2-hf
x
xx
0.7264

opt-125
xx
xx
x

Dependency Change?
any library dependency introduced or removed

Offline synced with @Kaihui-intel and left some discussion notes.


[Bug] compile error

Hi guys, I just start using this package and thank you for your works.
I have compile error and this is the compiler log:
This is pdfTeX, Version 3.141592653-2.6-1.40.24 (TeX Live 2022) (preloaded format=pdflatex 2022.7.25)  2 AUG 2022 14:44

Package: simpleicons 2022/07/29 7.5.0 Simple Icons 7.5.0 Logos

! LaTeX Error: File `simpleiconsglyphs-pdftex.tex' not found.

Type X to quit or <RETURN> to proceed,
or enter new name. (Default extension: tex)

Hi @mjavadhpour! Thank you for opening an issue on this repository and sharing your error. May I know more about your environment, how the package was installed and a MWE if possible please?

The project that I tried to used this package is public and is dockerized so let me know if it would be helpful to share it with you.

Thanks for the info! MWE stands for "Minimal Reproducible Example" which could help get the same error as you.
If I'm not wrong, would it be this repo - https://github.com/mjavadhpour/resm?
Please give me a while before I get some time to investigate and get back to you. I'm no LaTeX expert but I'll do my best!

Yes it is.
Thank you for the description about MWE.

I've found out that the archive tlmgr uses to install the package somehow doesn't include that file.
https://mirror.las.iastate.edu/tex-archive/systems/texlive/tlnet/archive/simpleicons.tar.xz
Please bear with me while I try and resolve this.

Hi @mjavadhpour,
My sincere apologies for the delay. I had to get in contact with the installers on these mirrors since there was a misunderstanding, about the file to be part of the doc branch and not runtime, that needed to be cleared up.
The file is there on the archive now, and I would like you to ask you to reinstall the package and confirm that it is now working on your end (and close the issue). 🙂

Hi @ineshbose, Thank you for following up the problem, I test it and it works as intended. 👍


Dailogflow - Status(StatusCode=DeadlineExceeded, Detail="Deadline Exceeded")

I am trying to invoke detect intent method and getting this particular exception.
When I invoke/run the same code from visual studio on local machine - the code works perfectly fine.
When I deploy the application on server, then I am getting the above exception.
From visual studio, I have published the application and uploaded all published files using FTP to server's virtual directory
string jsonFilePath ="JSON FilePath";
Environment.SetEnvironmentVariable("GOOGLE_APPLICATION_CREDENTIALS", jsonFilePath);
SessionsClient _sessionsClient = SessionsClient.Create();

// Initialize request argument(s)
SessionName _sessionName = new SessionName("ProjectId", conversationModel.RequestMessage.SenderId);
QueryInput queryInput = new QueryInput();
var queryText = new TextInput();
queryText.Text = "Hi";
queryText.LanguageCode = "en-us";
queryInput.Text = queryText;

// Make the request
DetectIntentResponse response = _sessionsClient.DetectIntent(_sessionName, queryInput);

I have verified that, the JSON File path is correct. My Program is able to find and read the file.
I have verified that, the environment variable get set. I have read it and printed it.

Could you give more details about your server environment? I suspect the problem is that your server can't reach the Dialogflow server using gRPC, but it's hard to tell from just this.
Are you able to run console applications on your server? That would make it easier to diagnose what's going on, as we could create a tiny console app that just tried to make a call - to various APIs - and then diagnose what's going on from that. It's typically easier than diagnosing web applications.

That is shared hosting server with IIS access only. So I can run the web application only on that server. That is Plesk hosting server.
If you want, I can quickly modify the code and give the output to you, if required.

Okay, if you can try using a different API, that would make it clearer whether this is a Dialogflow issue or a general networking issue.
If you could:

Add a dependency to Google.Cloud.Language.V1 version 1.3.0
Make sure you've got the Natural Language API enabled
Include something like the following code in your project:

using Google.Cloud.Language.V1;
using System;

namespace Issue3780
{
    class Program
    {
        static void Main(string[] args)
        {
            var client = LanguageServiceClient.Create();
            var response = client.AnalyzeSyntax(Document.FromPlainText("This is a simple sentence"));
            Console.WriteLine(response);
        }
    }
}

So you might want to add an extra controller method that returns the JSON, or add some logging - anything that just shows whether that request has worked or not.

I am facing some issue while adding payment method to enable Natural language processing API.
When I my card, the money get debited, but browser shows - error response. I tried with two different cards but not worked. Will try after sometime.
Meanwhile, any other code, I can try ?

Ah, I assumed you already had a billing account in order to use Dialogflow. You can try the same request without enabling the API - it's just that it will definitely fail. It'll still give us information though, in terms of whether it fails due to DeadlineExceeded or an "API was not enabled" exception.
Note that you still need the environment variable set to point at your service account credentials.

I have got the following exception -
Status(StatusCode=PermissionDenied, Detail="Cloud Natural Language API has not been used in project<PHONE_NUMBER>11 before or it is disabled

Okay, so that suggests it is managing to make the gRPC call. I don't know why Dialogflow would fail.
For the Dialogflow call, could you add an extra final argument to the call:
CallSettings.FromCallTiming(CallTiming.FromTimeout(TimeSpan.FromMinutes(5)))

That should give the call 5 minutes to work - I'd be astonished if that weren't long enough. You'll need to import the Google.Api.Gax.Grpc namespace. (This is just for diagnostic purposes - I wouldn't expect you to leave that in permanently.)
If you could add timing to the call so you can see how long it takes before it fails, that would help.
Also, could you give us information about exactly which version of the Google.Cloud.Dialogflow.V2 package you're using?

Adding the timeout as given by, solved the issue. Now I am getting response from the dialogflow.
An observation is - when IIS is reset, then for the first request - it took around 2 min to get response.
Here is print I added before DetectIntent statement and after it.
24-22-2019 02:22:58 966: Sending req to dailogflow:
24-24-2019 02:24:32 920: Response received from Dailogflow:
After the consecutive calls takes less time to execute.
24-26-2019 02:26:23 951: Sending req to dailogflow:
24-26-2019 02:26:24 185: Response received from Dailogflow:

That's very interesting. Are you able to add timing for the Language API call we tried earlier? I wonder whether it is something in the networking, but in a "gets better" way. I'd really like to understand this better.

Here are the timings for Language API -
After IIS reset - First request
24-31-2019 04:31:12 387: Before sending req
24-32-2019 04:32:46 575: Status(StatusCode=PermissionDenied
Second request
24-33-2019 04:33:17 075: Before sending req
24-33-2019 04:33:17 122: Status(StatusCode=PermissionDenied

It's possible this is related to https://github.com/googleapis/google-api-dotnet-client/issues/1475, where the person who raised the issue did some more research which led them to believe this was about their hosting provider not supporting HTTP/2.0 properly. Are you able to raise this with your hosting company?

Sure. I will raise this with hosting company once.
Thank you very much for all your support. I appreciate it.
I am closing this issue.


Tile Palette Selection Not Reset When Palette Rearranged

Summary
It was observed that, when the tile palette is rearranged during window resizing, the selection rectangle is not reset, even when the tiles within the selection are changed due to the resize. When attempting to place tiles using the palette afterwards, it uses a different set of tiles than the once previously used.
How to reproduce

Select some tiles in the tile palette on the right side of the palette
Resize the palette such that some or all of the tiles within the selection are moved to better fit the new window.
Place more tiles. Observe that the selected tiles have changed.

Workaround
Don't resize the tile palette while using it to place tiles.
Analysis
When resizing, the location of the tiles change. Since the selection doesn't reset in this instance, it gets the tile that would now follow the end of the selection.
e.g. if your selection now selects (0, x+1), where x is the end of the available tiles, and x+1 is blank due to the resize, the tile palette uses (1,0), looping back around to the first column of the next row.
One of two things should happen to fix this:

Naively remove the selection whenever the window is resized.
When resizing, scan for whether tiles in the selection are affect, and remove the selection when they are.

Attachments

Is this issue still open? or related to #689 and resolved

This is still an issue. I don't believe any fixes have been made regarding
this.
On Thu, Oct 25, 2018 at 12:28 PM Adam<EMAIL_ADDRESS>wrote:

Is this issue still open? or related to #689
https://github.com/AdamsLair/duality/issues/689 and resolved
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AdamsLair/duality/issues/690#issuecomment-433137770,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQDL5THYs2qtt20kozJOooLOJpXIA94xks5uofSogaJpZM4XLVwI
.

--
Nathan Heller
Software Engineer
Abbott Labs
1 St Jude Medical Dr, St Paul, MN 55117
(715) 551-3074

Where should I look first to start resolving this issue @ilexp ?

The selection logic is done in the tile palette, most of the code for this is probably going to be there. View only behavior is derived from the tileset view.

Hey all, I'm new here.  I've taken a quick look at this particular issue, and I believe I have enough understanding to provide a quick fix.  I'm validating things locally right now and making adjustments.
Given that I'm able to repro the issue in source from master, and don't see any activity here since Dec, I'm assuming this work is still up for grabs?

Yes, the issue still persists and it's currently up for grabs.
The code part you mentioned does look like it should take care of the issue, but it seems like this isn't the case. Maybe you can look into why exactly that is?

I just took a look into this issue and found that I could not reproduce the bug. Sorry to ask for the third time in this thread but is this issue still open? Are there any other additional steps that might be needed to reproduce this? Hopefully this gif will be visible enough to see that the same bug does not occur on my screen.
This behaviour still seems undesirable though. I think it still makes sense to modify it in one of the ways outlined above:

1.Naively remove the selection whenever the window is resized.
2. When resizing, scan for whether tiles in the selection are affect, and remove the selection when they are.

Let me know if it is still needed that one of these two fixes happens, and if one is strongly more desirable than the other.

I just took a look into this issue and found that I could not reproduce the bug. Sorry to ask for the third time in this thread but is this issue still open?

I just checked with the latest master and the issue is still there.

Are there any other additional steps that might be needed to reproduce this?

None that I know of.

Run the tilemaps sample.
Dragdrop the tilemap into the tilemap editor, or create a tilemap by any other means.
Edit tiles as shown in this gif.

As far as I can tell, that's already what you're doing, only with a different tileset. Can you check if you can reproduce the behavior with the sample tileset?

Let me know if it is still needed that one of these two fixes happens, and if one is strongly more desirable than the other.

The second solution sounds a bit nicer, since it is non-destructive. The first one might negatively affect regular workflows.

Thanks for the reply. Is the tilemap sample somewhere in the repository or online? I'm new to Duality so I'm not sure where to find things.

If you check out the repository and build the Duality solution, you should find a Tilemaps sample project under the samples folder. Set it as a startup project and hit Debug - the Duality editor should open up with the Tilemaps sample.

Didn't notice the sample inside the repo! I could reproduce with that for whatever reason. Think the check added in #722 solves the issue. I'm sure if it's the proper place to put that logic, but it seemed reasonable to me.

Fixed by @cowmanjoe  in PR #722. Closing this.


AzCopy --decompress has inconsistent and unexpected behavior

When AzCopy uses --decompress flag to download a blob whose content encoding is gzip, it's supposed to decompress the content and remove the .gz/.zz/.gzip file extension at the destination.
Here is the description for --decompress flag from the --help output:

Automatically decompress files when downloading, if their content-encoding indicates that they are compressed. The supported content-encoding values are 'gzip' and 'deflate'. File extensions of '.gz'/'.gzip' or '.zz' aren't necessary, but will be removed if present.

However, I've seen inconsistent behavior of AzCopy 10.27.0.
On windows, I am seeing AzCopy decompressing the file and renaming the file but the final name is unexpected. If I download a blob named "compressed.tar.gz" with the --decompress flag, the downloaded file is renamed to "compressed.gz". I expected to be renamed to "compressed.tar" instead.
On mac, I am seeing AzCopy decompressing the file but it doesn't rename the file. If I download a blob named "compressed.tar.gz" with the --decompress flag, the downloaded file is still named "compressed.tar.gz".
To reproduce the issue, you can use the following steps:

Create a gzip compressed file and upload as a block blob to a blob container with name "compressed.tar.gz". In the blob properties, set the content-encoding to "gzip".
Use AzCopy to download the blob with the --decompress flag
Check what the downloaded file's name is. Also check the size of the file to see if it's properly decompressed.

Naming the destination path (in the AzCopy command) to include the tar.gz extension takes precedence for the local destination file name as well. To obtain the desired end result for the --decompress flag, the destination should include only top-level directory.


[W5][T08-3]Sean Yap

Added sort functionality that sorts all the people in the address book into ascending order according to their full name. Only in ascending order for now.

Overall a good working implementation of the added functionality with correct coding style!
Great update to I/O & JUnit Tests!
Just some minor comments below:
https://github.com/nusCS2113-AY1819S2/addressbook-level2/blob/e95fec5d9eeef3f99259ea46b369b531efe21ba5/src/seedu/addressbook/commands/SortCommand.java#L1-L8
Consider updating the comments to indicate the new behavior that you have incorporated in this class.
https://github.com/nusCS2113-AY1819S2/addressbook-level2/blob/e95fec5d9eeef3f99259ea46b369b531efe21ba5/test/java/seedu/addressbook/data/AddressBookTest.java#L126-L132
I noticed that you implemented the sort() test in Addressbook Class. Since sort is a command, maybe it is better to create a SortCommandTest under commands instead? Nevertheless, this test still suffices in checking for functionality of the sort function implemented.

Overall a good working implementation of the added functionality with correct coding style!
Great update to I/O & JUnit Tests!
Just some minor comments below:
addressbook-level2/src/seedu/addressbook/commands/SortCommand.java
Lines 1 to 8 in e95fec5
package seedu.addressbook.commands;
/**

Lists all persons in the address book to the user.
*/
public class SortCommand extends Command {

Consider updating the comments to indicate the new behavior that you have incorporated in this class.

Oh yes, thanks for informing this, a clumsy mistake on my part ! This has been edited in the latest commit.

addressbook-level2/test/java/seedu/addressbook/data/AddressBookTest.java
Lines 126 to 132 in e95fec5
@Test
public void sort() throws Exception {
UniquePersonList unsortedPersons = defaultAddressBook.getAllPersons();
UniquePersonList personsToCheck = new UniquePersonList(aliceBetsy, bobChaplin, charlieDouglas, davidElliot);
unsortedPersons.sort();
assertTrue(isIdentical(unsortedPersons, personsToCheck));
}
I noticed that you implemented the sort() test in Addressbook Class. Since sort is a command, maybe it is better to create a SortCommandTest under commands instead? Nevertheless, this test still suffices in checking for functionality of the sort function implemented.

I chose not to do this as similarly with the list command, these commands are not the same as with the others, there is no change in output and no exception/error messages thrown regardless of the state of the program. But it is true that it is still a command on its own. This could be done in the future.
the list command was too not implemented the same way as with other commands.

See my comments for suggestions. Please close this PR after reading comments.


Update: Add multi-datasource report builder support

Resolves #665

Description
Updating report addon to support multi-datasource
Proposed Changes

Update serializer so normalizing adds namespace for the transforms.
Update base-transform to support namespaces.
Update report builder model fetch to pass down the datasource.
Add datasource to csv downloads and get-api
Support for cloning, save-as and bulk dimension value import
Add new mirage models for testing, as well as test support to mock multiple datasources that's quite a bit different config wise than default datasource.

License
I confirm that this contribution is made under an MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

#727


Allow overriding the port configuration for Concierge

Is your feature request related to a problem? Please describe.
For environments that are running Pinniped (Concierge/Supervisor) using hostNetwork: true (eg. because of a lack of a CNI) the listen ports being hardcoded in the tls.Listen() directives can cause issues.
Supervisor:

https://github.com/vmware-tanzu/pinniped/blob/0d285ce993c03ec822c2c29df4b8d6247e145da0/internal/supervisor/server/server.go#L388 (OIDC endpoints served by the supervisor)
https://github.com/vmware-tanzu/pinniped/blob/0d285ce993c03ec822c2c29df4b8d6247e145da0/internal/supervisor/server/server.go#L395-L396 (healthz endpoints/default listener)

Concierge:

Aggregated API configuration: https://github.com/vmware-tanzu/pinniped/blob/0d285ce993c03ec822c2c29df4b8d6247e145da0/internal/concierge/server/server.go#L210

This leads to potential port conflicts in such environments with no way out other than forking and patching the ports.
Describe the solution you'd like
It would be nice to be able to override these via the Config structs already used for the Supervisor (https://github.com/vmware-tanzu/pinniped/blob/88fd9e5c5eae3807fcd60df1188ba2db7a70c95b/internal/config/supervisor/types.go#L9)  and Concierge (https://github.com/vmware-tanzu/pinniped/blob/d780bf64bc8bd159066ffd34a3401aa778c91e86/internal/config/concierge/types.go#L9) so that they can be set in the ConfigMaps mounted on the pods.
Describe alternatives you've considered
Forking is always an option, but is a bit of a maintenance issue.
Are you considering submitting a PR for this feature?
Yes. I have a topic branch with some patches, working on validating it and adding tests.

How will this project improvement be tested?

Should be testable with unit tests/a kind deployment, though I also plan to inject the same binaries into my hostNetwork env for validation.

How does this change the current architecture?

I don't think there are any significant changes to the architecture.

How will this change be backwards compatible?

This change only introduces a new field, appropriate defaults will be used to keep it backwards compatible.

How will this feature be documented?

I'm not entirely sure this is a feature that needs to be documented, but if folks have preferences, I'm happy to update docs as needed.
Additional context
I'm not considering updating the local user authenticator to do this since it's not really meant for production use anyway.

In addition to the above, to support deploying Pinniped in firewalld environments, the listen port for Concierge will be defaulting to 10250 which is also the port on which Kubelet listens,  instead of the Current default of 8443.

We should discuss if we want to change the supervisor's default to 10250 as well.  The use case would be if we ever wanted to host an aggregated API server or admission webhook or similar on the supervisor.  Maybe we would do that?  Maybe we would do that but use a different service targeted at some other port?  Just something to think about.

Related, on the concierge pods, we should set the sidecar.istio.io/inject label to false per the istio docs.  In clusters where istio's sidecar injection is enabled by default, the concierge gets broken by the sidecar messing with its networking.

Note that this issue was split into two. This one now represents the Concierge ports, and #891 was created to handle the Supervisor ports. This was to make it easier to address them as separate PRs.

For people using EKS with a custom CNI plugin like Cilium running overlay network, hostNetwork: true and custom ports are also needed and enabled by the linked PRs.
Errors might look like this:

Error: could not complete Concierge credential exchange: could not login: the server is currently unable to handle the request (post tokencredentialrequests.login.concierge.pinniped.dev)


TiKV BR Development Tasks

TiKV BR Development Tasks
Close: #66
RFC: https://github.com/tikv/rfcs/pull/86
Tasks

[ ] TiKV-BR

[ ] Fix CI error (ref: #57, find CI error here)
[ ] Cherrypick https://github.com/pingcap/tidb/pull/32612
[ ] Support RawKV API V2
[ ] Support migration from RawKV API V1/TTL to V2

[ ] TiKV

[ ] Write development proposal & discuss with DP team
[ ] Support RawKV API V2
[ ] Support migration from RawKV API V1/TTL to V2

Reference
TiKV components related to backup & restore:

components/backup: for backup
components/sst_importer: for restore
split_region interface: for pre-splitting regions

Some merged PR related:

https://github.com/tikv/tikv/issues/10940
https://github.com/tikv/tikv/issues/11041
https://github.com/tikv/tikv/pull/11664

@zz-jason @haojinming PTAL~

All sub tasks are done.


Bring up 2 msg machines. Delete code for cs machines

https://github.com/Shippable/pm/issues/10360

this is done closing


Typically the corrupt file contains only a part of the measurements
{
    "SecretRegistry.registerSecret": 45757,
    "TokenNetwork.closeChannel": 111236,
    "TokenNetwork.openChannel": 97555,
    "TokenNetwork.setTotalDeposit": 44509,
    "TokenNetwork.settleChannel": 123338,
    "TokenNetwork.unlock 1 locks": 32128,
    "TokenNetwork.unlock 6 locks": 66029,
    "TokenNetwork.updateNonClosingBalanceProof": 93752
}

This issue keeps track of, at least, detecting this in the CI and block merge with incomplete gas measurements.

This was fixed in 12961bea391e74aacd9f1193bfd08d7a74d349e2


adding secondary public pool configured by macstadium

found a secondary public block configured on the firewall that was not documented by provider. Adding in currently unused datacenter in case we have the need at some point.

Thank you


[Pods] Fleshing out PodDebugTabView

⚠ Requires #1071 to be merged Merged!

This PR implements the DebugTab view. It shows the time of the last change, and it also extracts last termination information according to this spec: https://github.com/mesosphere/marathon/blob/feature/pods/docs/docs/rest-api/public/api/v2/types/podStatus.raml#L217

I'd suggest moving the Containers row to its own section, so that the the table can fill the width of the page, rather than the column it's in currently. Here's my markdown representation of the idea:
Last Changes
Configuration   Wed...

Last Terminations
Terminated at...
Instance ID     pod...
Message         Pod...
Started At      Wed...

Containers
Containers table goes here

Note that containers is an array on the last termination (that on it's an array on it's own).

Regarding https://github.com/dcos/dcos-ui/pull/1082#issuecomment-246751759, I am in a discussion with @leemunroe to find out a nice solution to this.
For now I moved the table at the end of the list:

Let's wait for @leemunroe's approval

After a discussion with lee yesterday we ended up with something like this:

No results: Title Last Terminations, text (No data)
One result: Title Last Termination (?? ago)
More than one result: Append title Terminated at ??/??/?? (?? ago) for every other item

I get this error when I enter the debug tab of a pod that has no tasks:

Should we handle the error I pointed out, or is this expected?
Could you elaborate a little on the process to come to fix this?
@MatApple Do you know something I don't?

@mlunoe I tested it and didn't get the error.

I get the error when I use fixtures and enter the debug tab. @wavesoft could you check it out?

@mlunoe , @MatApple this seems to be unrelated. It looks like the fixtures have not reflected the latest API advancements. If you test it against a live cluster it works fine.

Right, so we should update the fixtures!


How to using implicit flow keycloak with react native?

I need to get access_token from keycloak.
Currently, I am getting grant_type = client_credentials and client_id, client_secret;
But due to some system requirements we need to use implicit flow. But I can't get the password so I can't get the token with grant_type = password.
I use react native and am using curl to get tokens. Is there any solution that can help me? Please?

Try to do do something like this and tell me if it works:
initOptions={{ flow: 'implicit' }}


Is this a cssnext-loader bug?

Here: ~~https://github.com/webpack/webpack/issues/722#issuecomment-71627471~~ Whoops I meant here https://github.com/webpack/webpack/issues/725#issuecomment-71614133

have you tried to add the .css extension
@import "./base.css";

Doesn't seem to help:
ERROR in ./~/css-loader!./~/cssnext!./lib/index.css
Module build failed: Error: <css input>:3:1: Failed to find './base.css' from /Users/jasonkuhrt/projects/cloud-app-cloud-control
    in [
        /Users/jasonkuhrt/projects/cloud-app-cloud-control
    ]

./lib/base works but that doesn't make any sense at the file level. It seems that cssnext is poorly configured such that it doesn't have the right in value (see error above).
I wonder if the problem relates to the issues rework-loader had to overcome, wherein they say not to use css-loader:
https://github.com/aaronj1335/rework-webpack-loader

Maybe this version needs to be bumped? https://github.com/cssnext/cssnext/blob/master/package.json#L49

See resolution here https://github.com/webpack/webpack/issues/725#issuecomment-71663236

This look like a bug (cssnext-loader or postcss-import).
If you only have one path, please use root option (root: "./lib").
That being said, relative import without path or root options should work since cssnext-loader should provide the postcss from option.

Good to know that cssnext works natively with webpack :D


List of bindings is not updated

After activating a transient module binding the list of bindings shown in the "Cloud" view is not updated automatically. Requires pressing F5.

Fixed by #93, closing


Create Branching Strategy for automatic nuget publish

There should be a branch for merging changes for PRs and then the master branch should have a GitHub action that runs the test suite and pushes the newly created version number to nuget automatically.

This was completed with the creation of the pre-release branch and these three GitHub Actions.
https://github.com/Cryptographic-API-Services/cas-dotnet-sdk/tree/main/.github/workflows
First successful publish to nuget was 1.1.38.


RFC: add support for evaluating log1pexp(x)

Checklist

Please ensure the following tasks are completed before submitting a feature request.

[x] Read and understood the Code of Conduct.
[x] Searched for existing issues and pull requests.
[x] The issue name begins with RFC:.

Description

Description of the feature request.

This RFC proposes to add support for evaluating ln( 1 + exp(x) ) in a more numerically stable manner. The conventional approach is subject to overflow when x is large.
The proposed implementation would seek to divide the domain and provide appropriate approximations depending on the regime. In particular,
log1pexp(x) = exp(x); when x <= -37
log1pexp(x) = log1p( exp(x) ); when -37 < x <= 18
log1pexp(x) = x + exp(-x); when 18 < x <= 33.3
log1pexp(x) = x; when x > 33.3

Package: @stdlib/math/base/special/log1pexp
Alias: log1pexp
Related Issues

Does this feature request have any related issues?

No.
Questions

Any questions for reviewers?

No.
Other

Any other information relevant to this feature request? This may include screenshots, references, sample output, and/or implementation notes.

For further background, see associated numerical analysis.
For example usage in stdlib, see computing the logcdf for the logistic distribution.

I'll try my hand at doing this.

@rei2hu Awesome! Thanks for working on this!


nats_Open and nats_Close can't run simultaneously

The following code often fails in my Linux (Ubuntu 16.04) environment.
#include <nats.h>
#include <thread>
#include <iostream>

static void sub() {
    natsStatus s = nats_Open(-1);
    if (s != NATS_OK) {
        std::cout << "Open failed: " << natsStatus_GetText(s) << std::endl;
    } else {
        nats_Close();
    }
}

int main() {
    std::thread th(sub);
    sub();
    th.join();

    return 0;
}

Error message will be:
% g++ -std=c++11 -Wall -I ../src foo.cpp -Lsrc -lnats_static -lpthread  &&  ./a.out
Open failed: Initialization Failed

Guarding both nats_Open and nats_Close with mutex lock fixes the issue.
Is that expected? If mutex lock is necessary by spec, the document should be updated so.

@hidai It does not make sense to me to be calling nats_Open and/or nats_Close concurrently.
Think about it. You would have code running trying to open the library, do some NATS operations, while another part of your code would be closing it? What does that say about the application?
Adding a lock in the library would mean that  the lock would need to survive the close, which then would be reported as memory in use (which some will report as memory leak, which is not). So if need be I will then document that these should not be called concurrently.

Now I think that my problem should be addressed not by the library side but by the user side.
But let me explain what I wanted to do here since this might be useful for other users who  have similar use-case with me.
The problem I was trying to deal with is Case 3 in the figure below.
Case 1
client A      A--------->|
client B            B----------->|
opened?       |<---------------->|
              ^                  ^
              |                  |
              nats_Open          nats_Close

Case 2
client A      A--------->|
client B                      B----------->|
opened?       |<-------->|    |<---------->|
              ^          ^    ^            ^
              |          |    |            |
              nats_Open  |    nats_Open    |
                         nats_Close        nats_Close

Case 3
client A      A--------->|
client B                 B----------->|
opened?       |<-------->|<---------->|
              ^          ^
              |          |
              nats_Open  !!race condition!!
                         nats_Close and nats_Open are called concurrently

The race condition can be properly handled by user side using an integer counter and a mutex like below.
class Nats {
public:
    Nats() {
        std::lock_guard<std::mutex> lock(m);
        nats_Open(-1);
        ++c;
    }

    ~Nats() {
        std::lock_guard<std::mutex> lock(m);
        if (c == 1) {
            nats_Close();
        }
        --c;
    }

private:
    static int c = 0;
    static std::mutex m;
};


Fix content-type of error responses to text/plain

Because the body is not HTML, so it shouldn't be text/html.

thanks


Validate API-level tests against Swagger spec

The goal is to include validation of requests and responses (maybe only responses) in *ApiTests which are mostly located under whisk/core/controller package in the test suite. This validation is already in place for the tests in system.basic package.
Provide the expected results and outputs:
The validator should check that request and response conforms to the Swagger spec in core/controller/src/main/resources/apiv1swagger.json
Additional information you deem important:
Some test from the system.basic test suite might be later moved to the API-level tests as long as we have the swagger spec validation in place.

@mgencur @markusthoemmes how much more is left on this - can we document that here if any for future reference?


User specific abapGit settings

#1316
Rearrange the settings page. There are now two separated sections "Global settings" and "User specific settings"

User specific settings are stored in the user specific config.

And removed from gloabl storage.

There's also a migration mechanism, migrating existing list size and ADT jump settings to personnel settings of all abapGit users.

code looks good, I'll test it sometime during the weekend

sorry, I have not had much time, does anyone else have time to do a short test?

Hi,
In case there are no global settings, ie. the default are always used, abapGit cannot be started after applying the change,

fixed


Update direct-forwarding.md

@sharkAndshark Could you please explain why do we need this change? Is there something broken right now?

Could you please explain why we need this change?

The C# is currently its own paragraph and not the language specifier for the markdown code block


catkin build error 1: invalid conversion from signed to unsigned int

The catkin build was unable to find the pybind11 config file by cmake .

Fixed by the commit 01c03d6298117f09cf7ef0f729de7b1603c93301

The following error persists

Fixed the issue with commit fa6408ad631e130e5c8574e0e1ebee2be46bdcef

fixed issue with fa6408ad631e130e5c8574e0e1ebee2be46bdcef


how to make the circle thicker

Hello,
Is there a way to make the border of the circle thicker / larger ?
Thanks.

Current there is no support that you can set it up when initiating a new loading. But If you want to apply to all the loading, let override the CSS
Refer to jquery-loading.css
This is by default with 3px border. You can increase it to match your need. For example 5px or even 10px.
.js-loading-indicator { //the green one
  border: 3px solid transparent;
 }

.js-loading-indicator.double::before { // the red one
  border: 3px solid transparent;
}

.js-loading-indicator.triple::before, .js-loading-indicator.triple::after { //the yellow one
  border: 3px solid transparent; }


Add TODO form representing incomplete programs

A "shed" is a region of a program that is not yet written, in which the programmer can write whatever they want. The intention behind sheds is that they be used for fancy interactive editing features. Haskell has a version of sheds, which they call "typed holes", that don't allow any contents. These ones are more similar to those found in Epigram or Agda.
At present, a shed macroexpands to a program that throws an error that lists the expected type. However, they also add a syntax property that tools such as the todo list can use to provide a list of sheds and their goal types.
Future enhancements can include diffing the type synthesized for the shed contents against the type expected for the shed itself, and adding the typing context to the message and/or goal. Sheds in types would also be nice. But this feature is, I think already useful as it currently is.

This is cool. A few comments.

Is the “shed” terminology really common? It seem specific to Epigram, whereas Haskell and Agda both call them “holes”.

This sort of thing seems like it maybe ought to replace undefined!. Keeping error! around makes some sense, since it’s sometimes used for signaling when “the impossible happened”, but undefined! is really just for leaving parts of the program unimplemented.

Whatever the interface is, it should be documented.

Thoughts?

Perhaps "shed" is indeed too obscure :-)
How about uniting undefined! and sheds, calling the form TODO or todo instead? That seems to communicate the purpose of the form quite clearly, while at the same time not seeming particularly exotic.  At the same time, it might help those newer to working with types figure out the difference between undefined and error.
In Scala, ??? is the equivalent of undefined, which also seems to be a fine name for those who are more punctuation-inclined.
Whatever name you like best, I'm happy to use it.

I added docs and renamed it to TODO. I didn't delete undefined!, because TODO is a special form rather than a value that works at any type.
What do you think?

Sorry for my lack of comments on this. I’ve been very busy with work, and I’d hoped to get some time to look at this over the holiday, but truthfully, I’ve been exhausted, so I ended up spending basically the whole time catching up on sleep and relaxing.
The truth of the matter is that I’m still conflicted about a few things.

I don’t know if this should replace undefined! or not. It seems like it largely can by making TODO also work as an identifier macro that works like undefined! currently does.

Honestly, I find the fact that you can put arbitrary code inside it that’s completely ignored a bit… weird? The idea of eventually inferring and subsequently diffing the types makes some sense, but how does that work with global, unification-based type inference? What happens if the code inside the hole is just ill-typed? Currently, it seems like this would be equally well served with a standalone TODO followed by a #; expression comment.

If I were building this feature myself, I’d probably do two things. I’d change the name to todo!, replace undefined! with it, and turn it into an identifier macro. When used in function position, i.e. (todo! ...), I would make it so you could specify a string literal inside the parentheses that includes a message that would show up in your DrRacket plugin.
However, this is something with a purpose that you understand better than I do. Maybe there’s some workflow this feature enables that I’m not currently aware of that would motivate it a little better in my mind?

I hope that you feel under no obligation whatsoever to do free coding/reviewing work for me. Thanks for taking time now that you're recovered a bit!
The idea of a shed/hole is that it's a place for the user to stash things that are not yet ready for the type checker. Eventually, interactive commands can use those stashed things to help write the program. For instance, in agda-mode for Emacs, one can write a name in the hole and then issue the "refine" command, which unifies the type of the name that was written with the type of the hole, constructing a suitable number of holes for arguments. It is also possible to attempt to synthesize a type for whatever is in the hole and then see that synthesized type next to the type that the hole must have, which can provide insight. If the types match, then the hole wrapping an expression can be removed. For doing type-driven synthesis, a list of hints of names to use can be put in the hole.
These modes of interaction require that the shed/hole can hold basically arbitrary expressions. If a task name string is what's needed, then I'd want to provide that with a keyword argument to the macro.
You know better than I whether this should replace undefined!.

In agda-mode for Emacs, one can write a name in the hole and then issue the "refine" command, which unifies the type of the name that was written with the type of the hole, constructing a suitable number of holes for arguments. It is also possible to attempt to synthesize a type for whatever is in the hole and then see that synthesized type next to the type that the hole must have, which can provide insight.

Do you think this is feasible in a language with Haskell-style type inference? Agda has rather different type inference properties than Haskell/Hackett.

I think it's completely feasible. When showing the difference, the type of an expression in the hole should be such that some subset of its \forall-quantified variables can be instantiated to make it alpha-equivalent to the type expected for the hole (and such that type class constraints have a solution). In other words, it should be a subtype.
Example:
(: (TODO show) (-> Integer String))

might say:
At foo.rkt line 30 column 3:
  The expression
    show
  has type
    (forall (a) (Show a) => (-> a String))
  where the type
    (-> Integer String)
  is expected.  These types are compatible.

modulo curly-brace infix and the like. The right-click menu could then support "Merge expression" to delete the surrounding TODO form.

I think that the version in which the contents of the TODO are delayed until after other type inference, with all the scary non-local effects that you're describing, is exactly the right one for an ML or Haskell-style type system, and would be quite useful in practice.

I think that makes sense. That means that type errors would currently prevent holes from producing a result, which is a problem, but that needs to be fixed in general anyway (making it possible for programs to produce multiple type errors).
I’ll look into merging this when I get a free moment.

Finally merged. Thanks for this improvement! 😄


Unable to deploy to a pre-configured existing server

I already have a Ubuntu server at AWS that already configured before and contains the landing page for my app. And when i tried to deploy to it, nothing happened. The app is not accessible on host:port or root_url:port.
{
  "servers": [
    {
      "host": "xx.xxx.xx.xxx",
      "username": "ubuntu",
      "pem": "~/path/to/eat_local.pem"
    }
  ],
  "setupMongo": true,
  "setupNode": true,
  "nodeVersion": "0.10.36",
  "setupPhantom": true,
  "appName": "EatLocal",
  "app": "/path/to/eat-local",
  "env": {
	"PORT": 8080,
    "ROOT_URL": "http://app.ayoeatlocal.com/",
  },
  "deployCheckWaitTime": 30
}

The deployment seems to be successful. No errors were reported.
Meteor Up: Production Quality Meteor Deployments
------------------------------------------------

Building Started: /Users/rizky/Mobile/eat-local

Started TaskList: Deploy app 'EatLocal' (linux)
[xx.xxx.xx.xxx - Uploading bundle
[xx.xxx.xx.xxx ✔ Uploading bundle: SUCCESS
[xx.xxx.xx.xxx - Setting up Environment Variables
[xx.xxx.xx.xxx ✔ Setting up Environment Variables: SUCCESS
[xx.xxx.xx.xxx - Invoking deployment process
[xx.xxx.xx.xxx ✔ Invoking deployment process: SUCCESS
Completed TaskList: Deploy app 'EatLocal' (linux)

Then i try again on an empty DigitalOcean account using a similar config (Ubuntu 14.04). But this time it works.
{
  "servers": [
    {
      "host": "xxx.xxx.xxx.xx",
      "username": "root",
      "password": "xxxxxxx"
    }
  ],
  "setupMongo": true,
  "setupNode": true,
  "nodeVersion": "0.10.36",
  "setupPhantom": true,
  "appName": "EatLocal",
  "app": "/path/to/eat-local",
  "env": {
        "PORT": 8080,
    "ROOT_URL": "http://xxx.xxx.xxx.xx/"
  },
  "deployCheckWaitTime": 30
}

Really appreciate it if anyone can enlighten me on what's wrong. If you need more info just let me know.
Thx.

 I'm glad I could help. :P


Affine Image Transformation

Hey @ermig1979 , following from our convo I'm opening this issue for implementing a generalized affine image transformation API. This function will be capable of expressing an entire family of geometric image transformations. The downside is that it would typically result in unordered memory accesses, and might not be trivially parallelizable. I'll update this issue with a proposal API, and will create a fork with a simple (un-parallelized) C/C++ implementation. Looking forward to what we can do on this!

Ok. I'm waiting for API and scalar implementation.

I added function Simd::WarpAffine.

@ermig1979 thanks for this. We ended up going with a very naive implementation, and turns out clang did a good job at auto-vectorizing with -O3. I'm looking forward to profiling your implementation. I truly cannot thank you enough for this project; we're massive fans at NatML.


Create a Helm Chart for Rafter and its services

Description
Changes proposed in this pull request:
Create a Helm Chart for Rafter and his services:

create chart for Rafter
create chart for Rafter Upload service
create chart for Rafter Metadata service
create chart for Rafter AsyncAPI service
create for all charts Readme.md with description of all exposed values

Stretch:
Thinking about adding possibility to use custom values in values.yaml, something like:
envs:
  upload:
    externalEndpoint:
      value: "{{ if .Values.global.minioExternalEndpoint }}{{ .Values.global.minioExternalEndpoint }}{{ else }}'https://minio.{{ .Values.global.ingress.domainName }}'{{ end }}"

See issue for problem described above: https://github.com/helm/helm/issues/2492
Solution found on postgresql https://github.com/helm/charts/tree/master/stable/postgresql
Related issue(s)
https://github.com/kyma-project/rafter/issues/2

@michal-hudy
Please add comments to values.yaml
I will, but can I add it after merging this PR? It will have to be checked by TWs and I don't want to block this PR.
Helm should wait for pods to be in the running state
Must I add for all helm installation command --wait flag? Or did you thought about something different?
Upload Service buckets are not migrated when installing with rafter chart
I created migration-job template without checking it, my fault. But now, I checked the template condition for creating resources, and it is {{- if not .Values.minio.persistence.enabled }} and in script for migration, we had (here https://github.com/kyma-project/kyma/blob/master/resources/assetstore/templates/_helper_migrate_buckets.txt#L68):
isPersistance() {
  if grep -q "true" "/export/persistance.enabled"; then
    return 0
  fi

  return 1
}

if ! isPersistance; then
  echo "Persistance is not enabled, skipping..."
  exit 0
fi

MinIO credentials are not generated
If you mean about secretKey and accessKey, they are generated, otherwise creating resources would be impossible (I created sample AssetGroup for each installation and and it worked every time).

@magicmatatjahu

Please add comments to values.yaml
I will, but could I add it after merging this PR? It will have to be checked by TWs and I don't want to block this PR for @aerfio and @derberg.

I'm OK with it 😄

Minio localhost:9000 address shouldn't be used
I'm thinking about leaving this parameter empty...

If only it works with empty, then OK, but I'm not sure about it (simply don't remember)...

MinIO credentials are not generated
If you mean about secretKey and accessKey, they are generated, otherwise creating resources would be impossible (I created sample AssetGroup for each installation and and it worked every time).

Unfortunately, they are not generated, the default value from MinIO chart is used.

Upload Service buckets are not migrated when installing with rafter chart
I created migration-job template without checking it, my fault. But now, I checked the template condition for creating resources, and it is {{- if not .Values.minio.persistence.enabled }} and in script for migration, we had (here https://github.com/kyma-project/kyma/blob/master/resources/assetstore/templates/_helper_migrate_buckets.txt#L68):

It doesn't work, because when you switch to the Gateway mode Upload Service chart is not touched, so from its point of view, it is not updated. I think we should move this script to the umbrella chart.

Great work!

Merging with admin rights as pipelines don't work, because they required changes from https://github.com/kyma-project/rafter/pull/20


use object (key is type safe) to choice alert message rather than switch

How about to use object (hash map) to choice alert message?
I think this is same with using switch statement from the perspective of type safety.

HashMap based state transition is a one of my favor too. I agree to use this way here 👍
(BTW, in TypeScript layer, it has same meanings. However thinking in JavaScript, it might be different for making error possibilities 😇 )


Update dependencies and add new plugin for publish

Features:

Dependencies update.
Automatic close and release on publishing.

Cool. Thanks!


minimumResultsForSearch support

I guess this is not supported, right? This is great option for selects that have just few options. Showing search input field in such cases is kinda dumb (from UX perspective). It would be marvelous to be able to set this.

Closing as a dupe of #140


(fleet) store package installation in a bbolt db

Adds a DB that stores installed packages
Add is-installed

/merge


Angle Differences DXF & DWG

While working on DXF & DWG files in which the content is the same, I've noticed that the DWG reader reads angle values in radians (as documented)  I have a branch which shows these differences in MText, TextEntities, Arcs & Ellipses.
TextEntities show degrees on the DXF & Arc start and end angles.  All others are radians.  I was just going to make a PR, but I thought it would be worth discussing.  the DWG documentation rightfully states that the angle values will be in radians but the DXF is a little less explicit by stating "angle" for several types.
netDXF handles this by ensuring all the angles are in degrees, but it seems that most of the angles in the files are actually in radians.  Do we want to just convert everything to make sure it's in radians during the reading process?
Angle DXF & DWG written in AutoCAD 2010 formats.
angles.zip
A side note is that the MText rotation is not being read by either.  I'll open a separate issue for this.
Output of the dwg-dxf-angles branch:
DXF TextEntity Rotation: 45
DWG TextEntity Rotation: 0.7853981633974483
DXF Ellipse StartParameter: 1.020388469764473
DWG Ellipse StartParameter: 1.0203884697644738
DXF Ellipse StartParameter: 1.570796326794896
DWG Ellipse StartParameter: 1.5707963267948966
DXF Arc StartAngle: 45
DWG Arc StartAngle: 0.7853981633974483
DXF Arc EndAngle: 90
DWG Arc EndAngle: 1.5707963267948966
DXF MText Rotation: 0
DWG MText Rotation: 0

I'll keep them as radians, C# works with them so it's easier to work with System.Math functions if the angles are in radians.

My priority right now is to finish the first alpha for the DwgWriter I can open a branch dedicated to this issue but I'll not addressed for a while.

Excellent.  I'll issue a PR for this.


Fix awk field separator not working on non-GNU awk

This changes the way awk is used to separate fields so that it works on non-GNU awk.
This changes the field separator from '| ' to '|' and use gsub to strip whitespace.
The actual code, when used on OSX to provision deis on openstack result in the following error:
awk: illegal primary in regular expression |  at
 input record number 1, file
 source line number 1

The change was tested on OS X and CentOS 6. The risk of it breaking critical stuff is very low IMO.
(first PR so I've tried to follow the contribution guidelines to the best of my understanding. please let me know if there's anything that should be changed)

Thanks for the contribution! Please ensure your commits follow our style guide. This code will be tested once a Deis maintainer reviews it.
Your Pull Request can earn you credit at DigitalOcean! See http://docs.deis.io/en/latest/contributing/hacking/#digitalocean-credit for details.

(first PR so I've tried to follow the contribution guidelines to the best of my understanding. please let me know if there's anything that should be changed)

Looks perfect!

Congratulations @jfchevrette! You've earned $75 of DigitalOcean credit, enough to run a Deis cluster for over a month.
Please email<EMAIL_ADDRESS>to confirm, and we will reply with your DigitalOcean voucher codes.
Thanks again for contributing to Deis!


VS2012 .fsproj files upgraded to VS2013 don't work in Xamarin Studio

From @ovatsus on November 25, 2013 22:36
Xamarin Studio 4.1 didn't work with VS2012 .fsproj files upgraded to VS2012, but https://github.com/mono/monodevelop/pull/407 fixed that, and XS 4.2.x and 4.1.13 include that fix, but unfortunately it still doesn't work. If you build MonoDevelop from source from the equivalent tags, it does work, though. What can we do to try to fix this?
Now that VS2013 RTM is out, people are starting to hit this problem (see for example https://github.com/dungpa/fantomas/issues/104, and there have also been a couple of twitter threads about this)
Note: projects first created in VS2013 have an additional problem (the ToolsVersion=12), the fix for that is https://github.com/mono/monodevelop/pull/447 but it hasn't been merged yet
Copied from original issue: fsharp/fsharpbinding#249

From @ghost on November 27, 2013 10:45
Yes, tt is really important to fix this

From @ovatsus on February 12, 2014 18:23
There was a reply from @slluis saying it was fixed in master, but I don't know if it has been released or not, XS stopped working in my machine at all for some time now... Hangs at startup

From @ghost on February 12, 2014 18:20
@ovatsus do you know the current status of this? thanks.

From @rneatherway on February 12, 2014 18:47
I found it didn't work a few weeks ago. I will try building from master again tomorrow and double-check.

Closing due to age, this was fixed 4.x I think


Added Submission::link and Comment::comment_link

The method names are confusing.
Why does Submission::permalink return a relative path instead of an absolute path?
Why does Comment::link return the link to the submission instead of a link to the comment itself?

So I don't think it's my fault that travis can't build. It is having trouble installing jruby.

Maybe change to other link-function to url as well?

Thank you. I will merge it into my fork (https://github.com/StephanMeijer/redd/)

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @cfliao, @harshit-raj, @n1to-bit, @washujiang, @alfaindia, @kilianmh, @marcofavorito


Testnet

Hello .
How can i enable testnet ?

you can initial the client instance and setting the base_url to the Client:

client = Client(key, secret, base_url="https://testnet.binancefuture.com")

Then you could call most of the methods in the library, but the request will be sent to testnet.
An example file that shows how to set the base_url:
https://github.com/Binance-docs/binance-futures-connector-python/blob/main/examples/futures/trade/new_order.py#L12


Immediately saving card when  adding thru presentPaymentSheet

On the checkout page, after the following promises:
await fetchpaymentsheetparams()
await initPaymentSheet()
once the promises resolves, customer has an option to choose payment option where they can enter a new payment method e.g. 4242 4242 4242 4242
If the customer leaves the checkout page back to the cart to add or remove and item, there's no way to attach this payment method to the payment intent even though i am reusing the same payment intent id when the page reloads with the updated cart accounting
Describe the solution you'd like
presentPaymentSheet to attach a new payment method to the related customer,
also would be nice to use the same presentPaymentSheet for customer payment method CRUD

@m-elgamal payment methods are attached into the customer object instead of paymentIntent, so you need to make sure that you provide the same customerId in initPaymentSheet method. This scenario can be properly reproduced in our example app, see PaymentsUICompleteScreen.tsx

@m-elgamal payment methods are attached into the customer object instead of paymentIntent, so you need to make sure that you provide the same customerId in initPaymentSheet method. This scenario can be properly reproduced in our example app, see PaymentsUICompleteScreen.tsx https://github.com/stripe/stripe-react-native/blob/master/example/src/screens/PaymentsUICompleteScreen.tsx

So I revisited this issue again, seems there's some confusion about how the stripe payment sheet is currently designed
So, when the customer selects the payment methods like below

and enteres a credit card and selects save this card

and goes away from the checkout page then comes back to it that card is gone even though the the page is using the same payment intent and the same customer (both retrieved from the backend which retrieved them from stripe api for the second visit)

when i checked the dashbaord, the customer is created, but there's no cards attached to  it

Any updates?

I have the same issue, even if I'm using the same customer id and then I finalise the payment . all good except that the payment method is never saved to the customer

The payment sheet currently doesn't save the card details until after the payment is finalized. If you're hoping to save payment info for a future payment, have you considered using SetupIntents?
Otherwise, please let me know what the use case is for this :)

Closing since we didn't get a response- if you'd like to save payment methods for later use, I suggest using Setup Intents- https://stripe.com/docs/payments/save-and-reuse


Add a flag to compress files in-place

I.e. unpacking, compressing, and then repacking and overwriting the original file, as opposed to writing to a new directory.

Please check the update. I still need to update the README though.
Usage:
python compress_comics.py -o
or
python compress_comics.py --overwrite

It should work, but I didn't want to add it at first since it's risky to overwrite stuff

thanks for the update. I figured you didn't want it to be destructive, but this could be useful for testing or singular converts.


Changes for ATC 8.0.1 release

This PR is not related to any issues. It includes all the changes required for a release

Which Traffic Control components are affected by this PR?

Traffic Ops
Traffic Router
Traffic Portal
Version

What is the best way to verify this PR?

N/A
If this is a bugfix, which Traffic Control versions contained the bug?

N/A
PR submission checklist

[ ] This PR has tests
[ ] This PR has documentation
[x] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

I had to close this PR to cherry pick the commits to update go version and node modules from master branch. Refer to this PR to see the updates.


Import/Export admin buttons not showing up if another mixin is loaded first

Howdy,
I'm facing an issue where the import/export buttons don't show up in the admin page (Django 1.9) if I have another mixin loaded first.
The mixin being loaded first is from django-advanced-filters: https://github.com/modlinltd/django-advanced-filters/blob/develop/advanced_filters/admin.py#L57
eg.
class WhateverAdmin(AdminAdvancedFiltersMixin, ImportExportMixin, admin.ModelAdmin):
    pass

Anyone ever experienced that?
Thanks.

No longer an issue, thanks.

@hermanocabral I am having the same issue, how did you resolve it?

@taut-and-yare Please can you try with the 3.0.0 release beta version and if it is still an issue, can you provide steps to reproduce?

@matthewhegarty to reproduce: I am trying to use a django-advanced-filters  mixin together with ImportExportActionModelAdmin like so:
@admin.register(Contact)
class ContactAdmin(AdminAdvancedFiltersMixin, ImportExportActionModelAdmin):
    # other stuff

The template used is: admin/advanced_filters.html

@matthewhegarty FYI, the problem persists with 3.0.0
I managed to get it to work by adding one include tag in the template mentioned above:
admin/advanced_filters.html:
{% block object-tools-items %}
  {% include "admin/import_export/change_list_import_item.html" %}
  {{ block.super }}
  {# Add a link to the end of the tool items #}
  {% if advanced_filters %}

		{% trans "Advanced Filter" %}{% if '_afilter' in request.GET %}{% trans "Edit" %}{% endif %}

  {% endif %}
{% endblock object-tools-items %}

@taut-and-yare It was so long ago that I barely remember what I did, but as far as I recall there was something missing in the template and I just had to add it there to work.


Imp(ics23): Add conversion checks

Closes: #1108
Description
Add conversion checks for ProofSpec, LeafOp, and InnerSpec structs.

PR author checklist:

[ ] Added changelog entry, using unclog.
[x] Added tests.
[x] Linked to GitHub issue.
[x] Updated code comments and documentation (e.g., docs/).
[x] Tagged one reviewer who will be the one responsible for shepherding this PR.

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

/cc @Farhad-Shabani @rnbguy

hey. we had a discussion about the error scenarios for prefix_length and depth. we decided to go with only non-negative numbers.
So, the error conditions for any bound [max, min] should be (0 < max && 0 < min && max < min) || max < 0 || min < 0.
sorry for any confusion from my earlier comments.

can you give me permission to edit your branch? I want to make some changes before we merge.

Ah, I just noticed that LeafOp is mentioned but you didn't add its conversion checks.

can you give me permission to edit your PR? I want to make some changes before we merge.

https://github.com/isaacs/github/issues/1681 Yeah I guess I couldn't enable this, not able to find the option in my own PR. Could you kindly suggest the changes and I would resolve it asap.

Ah, I just noticed that LeafOp is mentioned but you didn't add its conversion checks.

I think there's no need to add conversion check for LeafOp because LeafOp's i32 fields hold enum value, thus limiting their value.

Could you kindly suggest the changes and I would resolve it asap.

Ah, I wasn't aware. I pushed changes to my branch rano/pr/1160. It's rebased from your fork - so you can just git pull <remote for cosmos/ibc-rs> rano/pr/1160

LeafOp

The enum values should be validated. I added the respective changes at my branch.


Currently "editon" is required in combination with ipfts-pack

As edition is currently required it would be great to mention this in the README. I wasn't aware of it - and then all filenames are corrupted after running the pack command.
Another solution would be to use the map index instead of the edition field. Is there any downside?

So would you like to use the ipfs pack without the previous steps/generation?
Because I think I don't understand how you get the files for the ipfs pack without the edition field.

@juliancwirko I created 10K pngs/jsons without the edition field - that seems not an issue.

But have you configured a custom metadata structure without 'edition' then?
The assumption was that even if you modify the metadata output, all fields should be used, but it would be good to focus more on how to make some of them optional and not break the whole flow. Of course, making all optional doesn't make sense because at least the ipfs cid for the image/asset should be there. I'll look at this PR and overall logic. Maybe most of the fields could be optional.

I haven't looked too much into the generation of the png/json files but it seems on the creation part the edition is not required. So yes I was running the script with an overwritten schema (to fit OpenSea) and did not include any edition. The generator worked but while packing instead of filename 1.png [object object].png would be generated.

Understood, I haven't had much time, but I will for sure look into how to make the fields optional on each step. I'll try today.

@juliancwirko I think the main issue on the ipfts-pack is that the script we included works but not on a large collection base. 10K items with ~14GB will silently fail on some point.
I'll try to extract the metadata adjustment based on a given IPFS location. NFT UP seems to handle large uploads very well but this requires the upload in 2 separate steps.

@dohomi, do you mean the uploading or packing process? Because you wrote about ipfs-pack.
We could check what 'nft up' does with it. It is built with Electron, so the source code regarding packing and uploading should be ready to copy. I haven't had much time lately, but I will try to test it on a more extensive set.

They don't use ipfs-car. If the problem was with packing using this lib, the nft-art-maker should probably pack the CID files using nft.storage, I am not sure yet, but I think the whole process is here: https://github.com/nftstorage/nftup/blob/main/public/electron.js
The problem is that it will need some more work because the nft-art-maker pack does some additional job updating the metadata files. Maybe it won't be challenging to switch to nft. storage, I will try to sit and think about it today evening.

Also there are similar issues in ipfs-car repo: https://github.com/web3-storage/ipfs-car/issues/114

The current pack command creates *.car files of images and metadata and after trying to upload these with the upload command the script never succeeded on the images (probably to the upload size). The main issue was, that there is no process indication on how much data is already sent from the script.
Secondly I was using NFT UP of the images and that succeeded. Now - as the CID changed I would need to re-run a script to change the CID of all metadata files.
I am open to any suggestion - the nftup is very vague as they state you can drop metadata and image in one go - but that would end up with wrong CID's again on the metadata.

Ok, so I think there are two problems:

We need an additional command to update the CIDs on demand by providing the proper CID (primary metadata and each file metadata JSON files). We need to loop through the files and change fields. This would be an option outside the normal flow when you don't want to use the nft-art-maker pack functionality.
We need to research nft up and check where it is different. I think it should also work through CLI. Maybe ipfs-car produces different .car files or something. Nft up also packs the files into .car, but they don't use the ipfs-car lib. For now, this is the only difference I found, but I need to dig deeper.

I split it into three separate tasks:

https://github.com/juliancwirko/nft-art-maker/issues/22
https://github.com/juliancwirko/nft-art-maker/issues/23
https://github.com/juliancwirko/nft-art-maker/issues/24

@juliancwirko I can look into #22

@juliancwirko great improvement for the upload functionality! I will close this issue now because we handle all subtasks already in tickets or PRs


WIndows 7 directory scanning error

s.js:520
  return binding.lstat(pathModule._makeLong(path));
                 ^
Error: EPERM, operation not permitted 'D:\'
    at Object.fs.lstatSync (fs.js:520:18)
    at start (fs.js:1017:10)
    at Object.realpathSync (fs.js:1005:3)
    at tryFile (module.js:142:15)
    at Function.Module._findPath (module.js:181:18)
    at Function.Module._resolveFilename (module.js:336:25)
    at Function.Module._load (module.js:280:25)
    at Module.runMain (module.js:492:10)
    at process.startup.processNextTick.process._tickCallback (node.js:244:9)

silent mode doesn't solve the issue.

Can you share the program that's raising this error?  It doesn't look like node-glob is anywhere in that stack trace.

No longer an issue on v9


Refactor etcd deployment

How to categorize this PR?

/area dev-productivity quality robustness
/kind technical-debt test
/priority normal
/topology seed
/platform all
/exp intermediate
What this PR does / why we need it:
This PR refactors the deployment for the etcd and adopts the component concept. Consequently, it eliminates the Helm chart and adds full unit tests, instead. This improves several aspects and the overall code quality.
Which issue(s) this PR fixes:
Part of #2754
/cc @shreyas-s-rao @amshuman-kr
ℹ Similar refactoring for etcd-druid: #3099
Release note:

NONE

Thanks @danielfoehrKn for the review! I've addressed the comments and pushed a dedicated commit so that it's easier for you to check.

the field backup.store.prefix value is missing the "/etcd-main" suffix

That's a nice catch, thank you a lot!

the field etcd.quota of 8Gi is missing

Yes, actually it was wrong how it was before: https://github.com/gardener/gardener/blob/master/charts/seed-controlplane/charts/etcd/templates/etcd.yaml#L60, the .spec.backup.quota field does not exist, see https://github.com/gardener/etcd-druid/blob/master/api/v1alpha1/etcd_types.go#L69-L100. However, the .spec.etcd.quota field exists (see https://github.com/gardener/etcd-druid/blob/master/api/v1alpha1/etcd_types.go#L103-L106) and I guessed that it was intended to set this one, so I fixed it in this PR. /cc @shreyas-s-rao @amshuman-kr

I deleted all my network policies and let it reconcile and now it works. I must have had a network policy preventing the traffic. Perhaps we should add a network policy that will specifically allow traffic from prometheus to etcd? I see that etcd doesn't have the label networking.gardener.cloud/from-prometheus: allowed

Thanks @wyb1, I've found the problem in the NetworkPolicy and fixed it (see dedicated commit for it).
In this PR I don't want to introduce changes, I just want to refactor the way how etcd is being deployed. I agree that all these network policies are not optimal yet and that we should refactor them as well at a later point in time.

@wyb1 Well, in our chat you mentioned that the dashboards don't need to be adapted, so I didn't even look after them 😉 Anyways, I've made the changes now, please check.

Thanks @ialidzhikov, done!


Corrected the logic to consider extras when using requirements

Purpose of This Pull Request

[x] Bug fix

Overview of Changes
Bug: When using requirements, irrespective of the arguments given, only requirements.txt is considered.
Fix: if no arguments are passed, only then consider requirements.txt. If arguments are passed then consider only them.

Thanks so much for the fix!
Goes to show I need to write more unit tests to protect against these kinda bugs. I'll merge now and release when I next get some time
Cheers 🙂


[SPARK-35283][SQL] Support query some DDL with CTES

What changes were proposed in this pull request?
Some command used to display metadata, such as: SHOW TABLES, SHOW TABLE EXTENDED,SHOW TBLPROPERTIES and so no.
If the output rows much than screen height, the output very unfriendly to developers.
So we should have a way to filter the output like the behavior of
WITH s AS (SHOW NAMESPACES) SELECT * FROM s WHERE namespace = 'query_ddl_namespace';
https://github.com/apache/spark/pull/32513 replace commands with something like local relation that produces unsafe rows, so can be more plan nodes above commands.
Why are the changes needed?
This PR provides a better way to display DDL when output rows much than screen height.
Does this PR introduce any user-facing change?
'Yes'. A new syntax.
How was this patch tested?
New tests.

Test build #139606 has started for PR 32852 at commit 716a50c.

Test build #139606 has finished for PR 32852 at commit 716a50c.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139606/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44134/

retest this please

Test build #139610 has started for PR 32852 at commit 716a50c.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44134/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44138/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44134/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44138/

Test build #139610 has finished for PR 32852 at commit 716a50c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44138/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139610/

Test build #139623 has started for PR 32852 at commit dc47094.

This is MySQL syntax:
mysql> SHOW CHARACTER SET WHERE `Default collation` LIKE '%japanese%';
+---------+---------------------------+---------------------+--------+
| Charset | Description               | Default collation   | Maxlen |
+---------+---------------------------+---------------------+--------+
| ujis    | EUC-JP Japanese           | ujis_japanese_ci    |      3 |
| sjis    | Shift-JIS Japanese        | sjis_japanese_ci    |      2 |
| cp932   | SJIS for Windows Japanese | cp932_japanese_ci   |      2 |
| eucjpms | UJIS for Windows Japanese | eucjpms_japanese_ci |      3 |
+---------+---------------------------+---------------------+--------+

https://dev.mysql.com/doc/refman/8.0/en/extended-show.html

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44150/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44150/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44150/

https://dev.mysql.com/doc/refman/8.0/en/extended-show.html
@wangyum There is a discussion below.
https://github.com/apache/spark/pull/31548#issuecomment-814170427

Test build #139623 has finished for PR 32852 at commit dc47094.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139623/

Test build #139638 has started for PR 32852 at commit ae77a6b.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44165/

Test build #139638 has finished for PR 32852 at commit ae77a6b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44165/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44165/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139638/

Test build #139647 has started for PR 32852 at commit 243af5e.

Test build #139647 has finished for PR 32852 at commit 243af5e.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139647/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44175/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44175/

retest this please

Test build #139668 has started for PR 32852 at commit 243af5e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44196/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44196/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44196/

Test build #139668 has finished for PR 32852 at commit 243af5e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

ping @cloud-fan @viirya @wangyum @maropu

Looks OK to me.  cc  @cloud-fan

Test build #139965 has started for PR 32852 at commit f5dd033.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44492/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44492/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44492/

Test build #139965 has finished for PR 32852 at commit f5dd033.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139965/


switch to using S3 instead of Google Drive

For reference, here are some other locations we've used S3 uploads in crystal:

https://github.com/PlaceOS/init#backup-container
https://github.com/PlaceOS/init/blob/master/src/utils/s3.cr#L3

@jeremy-west fyi

closed by #4


How to recieve data from page back to node via callback?

Why don't you write a description? Instead, I have to guess as to what you are saying. It doesn't help me help you. Please update the description and follow the guidelines to create new issues.
Before creating new issues search first. I promise answers are easier to find than ask.
If it is just using callback then search for callback to find this.


Install fails at step go install . ./cmd/...

I tried many different sytems, osx, aws ubuntu instance, and always get this, following
https://github.com/lightningnetwork/lnd/blob/master/docs/INSTALL.md
at step:
go install . ./cmd/...
output :

I'm getting the same error on my machines too.

What are your GOROOT and GOPATH set to?
Did you run glide install while in the lnd folder?

also, what does go version say?

GOROOT isn't set, not surprising since there is nothing in the installation instruction that sets GOROOT to anything, right ?
I strictly followed the instructions so I did as in the user manual :
cd $GOPATH/src/github.com/lightningnetwork/lnd
glide install
so yea it is ran from the lnd folder ...
go version says :
go1.9.4 darwin/amd64

Also, what does
git rev-parse HEAD
say when performed in the lnd directory?

65d6d77ef409741c7ad8562cb9781dd5894a5117

When you strictly follow the steps in https://github.com/lightningnetwork/lnd/blob/master/docs/INSTALL.md
can't you reproduce this error ? I get it on every single machine and I simply follow line by line everything in the install.md file

Looks like you either don't have your GOPATH set up properly, or didn't do glide install. I'd also check that your GOPATH and GOROOT aren't over lapping.

Hi, I did :
export GOPATH=~/gocode
export PATH=$PATH:$GOPATH/bin
as instructed...
GOROOT isn't set up to anything since there is no instruction about setting up GOROOT.
How to set GOPATH and GOROOT properly ?
Thanks

@xorq I have got the same issue. Just like you I followed the instruction steps to the letter. No mention of GOROOT there.

If there are any LN geniuses that have managed to get this working. Can you please improve the INSTALL.md file to bring down the barriers to entry. Otherwise most people are simply not going to be able to get this working and not bother asking for help.

I literally followed the guide in a fresh VM with Ubuntu stock and everything is working fine with the commit posted, as well as current master.
I would try MacOS in a VM, but Tim Cook hates me.

@dabura667 which Ubuntu version have you used for your VM? I'm on 16.04.3 LTS (without success).

What version of glide are y'all using?

It seems only a very narrow set of users have this issue, and it's typically related to a dated version of glide.

Tried to reproduce this o a fresh machine as well, but it works w/o issue.

Fine for me on ubuntu16.04 (not a VM), tried a few days ago. (glide 0.13.2-dev)

YES !
Guys ! following step by step never works but I just tried leaving the go path to ~/go and it works ...
So maybe update the INSTALL.md file without the line changing the go path to ~/gocode ?
Could everyone who made it work confirm that they didn't set ~/gocode as go path ?

@xorq In my experience, this usually means there's an issue with your git setup, as glide/go get uses git internally. Could you try to do go get manually on the failing repos, and also git clone?

I put together instructions on how to setup LND from scratch and tested them step-by-step on multiple systems (Ununtu, Debina, and Trisquel).
Go path is covered with more precision then the official LND instructions:
https://github.com/alevchuk/pstm/tree/master/lnd-e2e-testing#build-go

Seems like this is solved just by following the official golang installation instructions (which we link to in INSTALL.md).

Hopefully this is a related question: Can we expect to build in Eclipse (with Goclipse plugin)? Can build per instruction outside of Eclipse but struggling building after creating a new Go project and specifying location of existing code. If Eclipse is not the recommended IDE, is there an IDE that is?

@jrmillerwork Not sure about that. Would recommend trying VSCode together with the golang package if you need an IDE :)

@halseth, thanks. What I am wondering is how people are set up and in particular how code is debugged or any other tips.

@jrmillerwork I would believe it is not very different from other go projects, so I would just look at what is a recommended setup for go. Personally I don't use an IDE, just execute what I need from the commandline. :)

I was having this issue and my error was executing "sudo make" instead of just "make".  Presumably, by using 'sudo', I was using root's $PATH instead of the current user's $PATH, which did not include the /usr/local/go/bin directory.  Hope that helps someone!


deleting a report shows incomplete table

the metrics are not shown, perhaps because they're for an event forecast

@lboeman ?

I guess I would have expected that it would list the forecast/obs pairs too. but that's probably another issue.

The object pairs only exist in the API response as an object with uuids, so I haven't loaded the metadata to display them before. It could definitely be done if that's useful.


Feat tinyarray

Codecov Report
Base: 52.81% // Head: 56.25% // Increases project coverage by +3.43% :tada:

Coverage data is based on head (3d860d4) compared to base (e8807f6).
Patch coverage: 62.01% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   52.81%   56.25%   +3.43%
==========================================
  Files          13       14       +1
  Lines         977     1136     +159
==========================================
+ Hits          516      639     +123
- Misses        461      497      +36

Impacted Files
Coverage Δ

src/arena/unique.rs
66.66% <0.00%> (ø)

src/ir/data_flow.rs
0.00% <0.00%> (ø)

src/ir/instruction.rs
0.00% <0.00%> (ø)

src/arena/secondary.rs
78.23% <30.76%> (-1.23%)
:arrow_down:

src/utility/tiny.rs
70.92% <70.92%> (ø)

src/arena/key.rs
81.25% <100.00%> (+2.67%)
:arrow_up:

src/arena/map.rs
100.00% <100.00%> (+21.21%)
:arrow_up:

src/arena/mod.rs
100.00% <100.00%> (+83.33%)
:arrow_up:

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.


white Background with "Gerry Dark/Darcula" IntelliJ

Hello, the Gerry Dark and Gerry Darcula themes have recently been given a white background in the code field (see images). I loved Gerry Dark but now it is no longer usable.
Greetings Jarno

Gerry Darcula

Gerry Dark

Duplicate to https://github.com/gerryhjs/gerry-themes/issues/8
Workaround:
This is an issue caused by IDE Setting Sync, it will apply a wrong color scheme in some cases(IDEA-325237).
Please check the Settings | Editor | Color Scheme window and select the corresponding color scheme as the workaround.

Duplicate to https://github.com/gerryhjs/gerry-themes/issues/8
This is an issue caused by IDE, it may apply a incompatible color scheme in some cases(IDEA-325237).
Please check the Settings | Editor | Color Scheme window and select the corresponding color scheme. (Switch to Gerry Dark/Darcula)

Thanks, that worked