Currently not possible to paralellize on samples in pandora compare

We were parallelizing on samples in compare (since it receives many samples) and on reads in map (since we receive 1 sample only, but with many reads). Parallelizing on samples in compare makes it run fast, and one of the loops we were parallelizing is:
https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/compare_main.cpp#L310-L384
But this parallelization was wrong: there is an iteration dependency that I missed. I then falled back to parallelize on reads also in compare (each sample is treated one at a time, but the reads of a sample are processed in a multithreaded way) in https://github.com/leoisl/pandora/commit/91ad421ce5579bb8218800d4706966cedc2b2017 . This is 2-5 times slower (in the 16 samples dataset at 20x), but we generate then correct results. The only dependency that is not allowing us to parallelize on samples is the e_rate (error rate) variable:
https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/compare_main.cpp#L104
It is given an initial value, depending on the parameters given to compare. The thing is: after processing the first sample, the error rate might get updated at
https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/compare_main.cpp#L348
The updated error rate is then used to map the second sample at
https://github.com/rmcolq/pandora/blob/5356466d645172171df24009612153b65716176c/src/compare_main.cpp#L326-L331
Then it might get updated again, and will be used in the mapping of the third sample and so on. So this is an iteration dependency: the processing of the i-th iteration depends on a value defined in the (i-1)-th iteration, and thus we can't parallelize the loop as a whole (only some parts of it). I am wondering if this should be kept like this or if there is a way around it which will allow us to fully parallelize on samples in pandora compare. Not a big issue though, since this does not seem to be the major bottleneck in pandora compare, and the heaviest part of this loop is processing the reads, which is currently parallelized.

The error rate is dataset/fastq dependent, and ought to be remembered per-sample @leoisl

Agreed. Was a shortcut and not a good one
Rachel
Sent from my Samsung Galaxy smartphone.
-------- Original message --------
From: Zamin Iqbal<EMAIL_ADDRESS>Date: 07/06/2019 11:56 (GMT+00:00)
To: rmcolq/pandora<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [rmcolq/pandora] Currently not possible to paralellize on samples in pandora compare (#138)
The error rate is dataset/fastq dependent, and ought to be remembered per-sample @leoislhttps://github.com/leoisl
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/rmcolq/pandora/issues/138?email_source=notifications&email_token=ACLIWO4BPBOLNJDAAYRVNRDPZI5EHA5CNFSM4HVVK33KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXFQBDA#issuecomment-499843212, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACLIWOYFYM7MPC2I4XRFOZ3PZI5EHANCNFSM4HVVK33A.

Just for remembering what we discussed on Slack:  I will keep the code as it is, because we can then compare with the previous version. Once we are sure the new version is stable, then we  proceed to fix this bug.

Update: multithreading on samples will be slightly faster but will potentially use a lot more RAM, so I think it is not worth it - we are more constrained on RAM usage when processing a lot of samples in pandora compare. So I'd vote to keep multithreading on reads, as it is now.
So, from this issue remains only solving the minor bug on how the error rate is computed.

I guess this will also close https://github.com/rmcolq/pandora/issues/93


Replace image proxy, add auto-fetch, add Dockerfile

I was interested in using this, and changed/added a few things that may make it easier and/or more usable for others.

Open Library has a public API that can supply cover images. Since the Google Books API provides an ISBN, we can use Open Library to fetch the cover image, rather than using your image proxy. This lets us remove one of the security concerns listed in the Readme.
Speaking of security concerns, I've added an environment variable that can toggle the auto-update functionality. Against my instincts, I kept it on by default, although I'd much prefer it being opt-in rather than opt-out.
I added a way for the app to automatically fetch on a specified interval, rather than requiring the /fetch endpoint to be called. The AUTO_FETCH_INTERVAL environment variable specifies how long to wait (in ms) between automatically checking for new pages.
There is also a new Dockerfile, so people can easily host this locally or wherever they want. Instructions for this were added to the Readme.

Some of these changes would also require changes to the guide you have as well.

Nice! Thank you for these changes, I will merge them when I've rewritten the guide (shouldn't take more than a couple of hours).
Regarding the auto-update functionality - you are right about this and I felt uneasy when implementing it, but I needed a way to deploy updates to servers on Replit not under my control because there were a lot of API degradations that I had to address in the past few weeks. So what I'll do is make it so that it only notifies when there's a new version available, perhaps through the notion page itself (by editing the description maybe). This would mitigate the risk.

I think I have a better auto-update functionality, awaiting testing. This will create a page in the notion database notifying users that there's an update, and they can just tick a checkbox in order to pull the new code. And it also respects the environment variable in regards to checking for updates.

I think I have a better auto-update functionality, awaiting testing. This will create a page in the notion database notifying users that there's an update, and they can just tick a checkbox in order to pull the new code. And it also respects the environment variable in regards to checking for updates.

Oh that's smart!


Getting Nan in matmul gradient

Hi I am trying to train your ViT model with a different dataset, ( a video of shape [B * 300 * 75] where 300 is the no of frames. So I changed the to_embedding as below to match my data
self.to_embedding = nn.Sequential(
            nn.Linear(self.num_features, dim),
        )

unfortunately I am getting nan after some epoch. I have added the backward trace. Could you kindly tell what might be the problem

if you are working with video, try https://github.com/lucidrains/STAM-pytorch or https://github.com/lucidrains/TimeSformer-pytorch

https://arxiv.org/abs/2103.15691 further corroborates the conclusions of the two papers above. in fact STAM is just model 2 in this paper, and TimeSformer model 3

if you are working with video, try https://github.com/lucidrains/STAM-pytorch or https://github.com/lucidrains/TimeSformer-pytorch

Will give it a try. Thanks


Addition: FedGeoDay

FedGeoDay might also interest your communities. Dedicated to bringing more OS into our federal gov.
https://www.fedgeo.us/
April 30, Washington DC.

This is great @mcawley, I will add it to the list.

Done! Thanks for your contribution @mcawley.


Add optional endpoint_url to s3 keywords

These changes make it possible to use an s3 mock implementaion like https://hub.docker.com/r/adobe/s3mock

@teaglebuilt Would it be possbile to get this merged?

@teaglebuilt Is there anything wrong with the PR? Could you please give me more information why this can't be merged?

@adrianyorke @Jakob-em is this ready to be merged?

@teaglebuilt I added the region parameter back to keep the compatibility with existing scripts.
Would be happy to get this merged 🙂

Thanks for the PR and region fix @Jakob-em.

@Jakob-em do you plan a release with this PR ?


Show playlist under the Beat Local

Hi there,
It would be really handy to be able to see under the Beat Local list the playlists a song is in using the playlist thumbnails I guess like the way you did for the difficulties.
That way, it would be quite easy to spot songs that are not in a playlist yet

Hello, yes, it is planned. 😄

I would love to see this as well! Thanks for this awesome utility!


Remove --legacy-peer-deps flag from npm install in frontend

In the frontend we use the @mui/styles package, version 5.9.2. This package is not yet released with official React 18 support, so it expects<EMAIL_ADDRESS>It works with React 18 regardless, so for the moment we use the --legacy-peer-deps flag in the npm install command in the frontend Dockerfile.
Once this package has official React 18 support, we should remove this flag again.
Tracked issue in question: https://github.com/mui/material-ui/issues/29844

Fixed in v1.1.1


Polygon enlargement

@heitzmann : Our chip design workflow is based on gdspy 0.7 and is built on gdspy geometry objects and gdspy boolean operations. There is one feature, however, that requires us to use additional packages to complete the design cycle. In our mask layouts, it is critical to have the ability to introduce CD bias to account for the linewidth reduction during the fabrication process. In mathematical terms, one needs to have the ability to enlarge polygons by a predefined amount in the direction that is normal to the polygon surface... thus a request for this feature. Thank you.

@rosivda, please take a look at the offset function.
I believe it does exactly what you are looking for.


Option to upload value of HEX

Is there option to upload value of HEX file instead of give the path of the hex file to the tycmd?

Hi, and sorry for the delay.
I've added the possibility of uploading from stdin in the latest build, I don't know if that answers your need?
It only works with Intel HEX (*.hex) files.
You can find development builds on my BinTray account: https://bintray.com/koromix/tytools/tytools/view/files#files/ (right now it's TyTools-0.8.8-16-g396b7b9)
To do so, use "tycmd upload -f ihex -" and the firmware will be read from standard input.


FlowListView not appering Xamarin C#

I am working on demo project for my application, in this i added a page

public class Page1 : ContentPage
{
    public Page1()
    {
        ObservableCollection<ItemModel> List = new ObservableCollection<ItemModel>();
        string[] images = {
            "https://farm9.staticflickr.com/8625/15806486058_7005d77438.jpg",
            "https://farm5.staticflickr.com/4011/4308181244_5ac3f8239b.jpg",
            "https://farm8.staticflickr.com/7423/8729135907_79599de8d8.jpg",
            "https://farm3.staticflickr.com/2475/4058009019_ecf305f546.jpg",
            "https://farm6.staticflickr.com/5117/14045101350_113edbe20b.jpg",
            "https://farm2.staticflickr.com/1227/1116750115_b66dc3830e.jpg",
            "https://farm8.staticflickr.com/7351/16355627795_204bf423e9.jpg",
            "https://farm1.staticflickr.com/44/117598011_250aa8ffb1.jpg",
            "https://farm8.staticflickr.com/7524/15620725287_3357e9db03.jpg",
            "https://farm9.staticflickr.com/8351/8299022203_de0cb894b0.jpg",
        };

        int number = 0;
        for (int n = 0; n < 20; n++)
        {
            for (int i = 0; i < images.Length; i++)
            {
                number++;
                var item = new ItemModel()
                {
                    ImageUrl = images[i],
                    FileName = string.Format("image_{0}.jpg", number),
                };

                List.Add(item);
            }
        }

        FlowListView listView = new FlowListView()
        {
            FlowColumnTemplate = new DataTemplate(typeof(ListCell)),
            SeparatorVisibility = SeparatorVisibility.None,
            HasUnevenRows = true,
            FlowColumnMinWidth = 110,
            FlowItemsSource = List,
        };
        listView.FlowItemTapped += (s, e) =>
        {
            var item = (ItemModel)e.Item;
            if (item != null)
            {
                App.Current.MainPage.DisplayAlert("Alert", "Tapped {0} =" + item.FileName, "Cancel");
            }
        };

        Content = new StackLayout
        {
            Children = {
               listView
            }
        };
    }
}

the model winch used seems like this

public class ItemModel
    {
        public string ImageUrl { get; set; }
        public string FileName { get; set; }

    }

and the template for FlowListView..

public class ListCell : View
{
public ListCell()
{
CachedImage IconImage = new CachedImage
{
HeightRequest = 100,
Aspect = Aspect.Fill,
DownsampleHeight = 100,
DownsampleUseDipUnits = false,
LoadingPlaceholder = "image_loading.png",
ErrorPlaceholder = "image_error.png"
};
IconImage.SetBinding(CachedImage.SourceProperty, "ImageUrl");
        Label NameLabel = new Label
        {
            Opacity = 0.5,
            HorizontalOptions = LayoutOptions.Fill,
            HorizontalTextAlignment = TextAlignment.Center,
            VerticalOptions = LayoutOptions.End,
        };
        NameLabel.SetBinding(Label.TextProperty, "FileName");
        Grid grd = new Grid
        {
            Padding = 3,
            ColumnDefinitions = {
                new ColumnDefinition { Width = new GridLength (1, GridUnitType.Star) },
            },
            RowDefinitions = {
                new RowDefinition { Height=GridLength.Star},
            },
        };
        grd.Children.Add(IconImage,0,0);
        grd.Children.Add(NameLabel, 0, 1);
    }
}

What i am facing that it doesn't showing List which i'm rendering on page could you please help me with this..

Thank You,

Change ListCell  to inherit from ContentView than View
Add View = grd; after grd.Children.Add(NameLabel, 0, 1);


jsonnet: add grafana-mixin

Description
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Add grafana-mixin.
~This change requires jsonnet v0.18.0 which isn't released at the time of writing this PR, hence I am marking this as draft.~
Type of change
What type of changes does your code introduce to the kube-prometheus? Put an x in the box that apply.

[ ] CHANGE (fix or feature that would cause existing functionality to not work as expected)
[x] FEATURE (non-breaking change which adds functionality)
[ ] BUGFIX (non-breaking change which fixes an issue)
[ ] ENHANCEMENT (non-breaking change which improves existing functionality)
[ ] NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Changelog entry
Please put a one-line changelog entry below. Later this will be copied to the changelog file.

Add grafana-mixin

This is ready to go. Jsonnet 0.18.0 is no longer needed as grafana mixin is now using plain jsonnet instead of YAML files.


Updating to Bootstrap 3

The current version is made for Bootstrap 2.
I'm trying to have it working with bootstrap 3 - so far, only the modal of insertImage and insertLink window seems to be of trouble.

FYI I edited your project and the changes are now Bootstrap 3 compatible. Also a suggestion on usage it might be beneficial to let users know if they are using ajax to upload the BASE64 image they need to do it the right way like this
data: {image: image},
and not like the old school way, it will break
data: 'image='+image+

Thanks a lot! Is there a PR with this change?

lol I don't do GitHub much, what is PR?

If you want to create a pull request between the patch-3 branch of your fork and zoharad's base, go to
https://github.com/zohararad/wysihtml5n/compare/master...cbielich:patch-3

Ok its done, you should see it

ok to close this one?

Closing ! #6  Thanks @cbielich for the pull and @zohararad for the merge !


"git describe" impact the build on travis-ci

On the forked build the command "git describe" will return an error:
fatal: No names found, cannot describe anything.

close it as fixed.


Marker examples not working

http://tombatossals.github.io/angular-leaflet-directive/examples/0000-viewer.html#/markers/simple-example

Thanks @juanmanavarro, working everything again. I'm doing a complete rework of the code to make it compatible with Leaflet 1.x, and some things gets broken.


Twistie does not change on click

Start debugging a simple javascript

console.log('hello');
debugger;

Type console in the repl and evaluate it
Expand it and some of its children. Notice how the twisties are not changing on click. They are constantly in the collapsed state (even though the object actually gets expanded)

This is related to dynamic heights. First bad commit: https://github.com/Microsoft/vscode/commit/2b5beb356ce872b1ecf59f9dbce3ed12fb59c16c


Fix filamat internal state tracking

in a few places filamat loses the value of targetApi and targetLanguage
and attempt to guess it from other values, this lead to inconsistencies.
we now keep both targetApi and targetLanguage and use them appropriately.
in particular we don't use the optimization level to "guess" what
targetLangage we're on.
In the end this resolve a prior unit test failure and allows us to not
have to use "spirv" rules when generating only GLSL.

reviewers, the first commit is a reformat so it's easier to ignore it during the reivew.


Failed to initialize the log file - Permission error

"Failed opening file C:\ProgramFiles(x86)\Koalageddon\logs\Integration.EpicGamesLauncher.log for writting: Permission denied"
This happens on All User install, this is due to the Program Files folder being protected, you can't get away from Program File UAC bugs can you? ;)

Basic patch-y solution is to remove the "Read only" flag from just the Koalageddon folder, you could either prompt for UAC everytime someone uses the injector, or bypass UAC entirely, though that would make this even more of a "virus" that it already is 😝 .
Here's some code that may help

#include <Windows.h>
#include <iostream>

// Window just goes insane after proper elevation for whatever reason
// To be used as stepping stone :D

int main() {
    wchar_t szPath[MAX_PATH];
    GetModuleFileName(NULL, szPath, ARRAYSIZE(szPath));
        // Launch itself as admin
        SHELLEXECUTEINFO sei = { sizeof(sei) };
        sei.lpVerb = L"runas";
        sei.lpFile = szPath;
        sei.hwnd = NULL;
        sei.nShow = SW_NORMAL;
        if (!ShellExecuteEx(&sei))
        {
            DWORD dwError = GetLastError();
            if (dwError == ERROR_CANCELLED)
            {
                // The user refused to allow privileges elevation.
                std::cout << "User did not allow elevation" << std::endl;
            }
        }
        else
        {
            // The user allowed the privileged elevation
            std::cout << "User gave elevation" << std::endl;
        }
    return 0;
}

That is why I wrote in the Installation instructions:

Run the installer and complete the installation. It is recommended to pick a folder other than Program Files since these folders typically require admin access for read/write operations.

Oh, sorry then!


Getting an error "starting instance: nil"

Summary
I have a Golang plugin, which handles a custom API authentication. I'm also using Kong-ingress in Kubernetes in order to deploy it.
At some point, w/o any obvious reasons I'm getting an error starting instance: nil and it totally breaks Kong and the only way to fix it is to restart kong-ingress pod.
Please find logs below.
Additional Details & Logs

Kong version (kong:alpine)

2020/10/05 14:39:16 [error] 29#0: *10525438 [kong] go.lua:429 [jwt-b2c] starting instance: nil, client: ****, server: kong, request: "GET /accounts HTTP/1.1", host: "****"
2020/10/05 14:39:16 [error] 29#0: *10525438 lua coroutine: runtime error: unknown reason
stack traceback:
coroutine 0:
	[C]: in function 'error'
	/usr/local/share/lua/5.1/kong/db/dao/plugins/go.lua:432: in function 'reset_and_get_instance'
	/usr/local/share/lua/5.1/kong/db/dao/plugins/go.lua:501: in function </usr/local/share/lua/5.1/kong/db/dao/plugins/go.lua:497>
coroutine 1:
	[C]: in function 'resume'
	coroutine.wrap:21: in function <coroutine.wrap:21>
	/usr/local/share/lua/5.1/kong/init.lua:756: in function 'access'
	access_by_lua(nginx-kong.conf:87):2: in main chunk, client: ****, server: kong, request: "GET /accounts HTTP/1.1", host: "****"
<IP_ADDRESS> - - [05/Oct/2020:14:39:16 +0000] "GET /accounts HTTP/1.1" 500 76 "-" "PostmanRuntime/7.26.5"
2020/10/05 14:39:16 [notice] 29#0: *10525438 [kong] handler.lua:?:56 "ACL" nil, client: ****, server: kong, request: "GET /accounts HTTP/1.1", host: "****"
2020/10/05 14:39:16 [notice] 29#0: *10525438 [kong] handler.lua:?:57 "Configuration collection is" "accounts", client: ****, server: kong, request: "GET /accounts HTTP/1.1", host: "****"

It looks like it happens when a lot of requests come in, however, neither pod wasn't restarted nor additional ingress logs added.
The issue is also appearing with image kong:2.1.4-ubuntu

Eventually, I found a bug in my plugin source code, those were fixed and the issue doesn't appear anymore. Seems to be Kong didn't kill the process because there were some open threads in my Golang plugin (e.g. opened UDP connection for remote logging system). Though, I've spent a couple of days zigging and zagging around Kong source code in order to find an issue cause.
Would be better to improve logs, or add more metrics whose will be showing stuck processes and etc.

@YegorMedvedev I have the same problem with you.See this link. https://github.com/Kong/kong/issues/6500

@bungle , it's a blocker issue, please have a look...

@javierguerragiraldez / @gszr, does this ring any bells (I think you have a way better understanding of our Go support than I have)?

yes, working on it

@javierguerragiraldez , @bungle , should I be waiting for the official release, or there is a chance to get the Docker image earlier?

Hi @YegorMedvedev ,
can you give me your email, please?
i want to ask some things about go plugin
thanks.

@tuanpm90 , fee free to use a temporary chat https://tempchat.in/c/kong-go-pl

@javierguerragiraldez , @gszr, @bungle guys, unfortunately, the issue isn't fixed. After a week of running it on dev, I've got the same issue again... please find logs below:
2020/12/08 10:13:06 [error] 30#0: *39049961 [kong] go.lua:436 [jwt-b2c] starting instance: no data, client: ****, server: kong, request: "PATCH /accounts HTTP/1.1", host: "****"
2020/12/08 10:13:06 [error] 30#0: *39049961 [kong] init.lua:255 [jwt-b2c] /usr/local/share/lua/5.1/kong/db/dao/plugins/go.lua:439: no data, client: ****, server: kong, request: "PATCH /accounts HTTP/1.1", host: "****"

I'm using the kong:2.2.1-ubuntu Docker image and Kubernetes ingress.

this error is happens to me as well, is there any work done on it?

for Kong 2.3 we've rewritten several parts of this code, keeping an eye on this issue and other similar ones.  Please try the beta we released yesterday.

thank you :) also is upgrading go-pluginserver from 0.4.0 to 0.6.0 might solve it also?

thank you :) also is upgrading go-pluginserver from 0.4.0 to 0.6.0 might solve it also?

maybe,  see https://github.com/Kong/go-pluginserver/pull/38


server: improve region heartbeat

What problem does this PR solve?
For now, both GetBytesReadRate and GetBytesWriteRate use a read lock, and GetRegionNamespace decodes the same key twice.
What is changed and how it works?
This PR replaces GetBytesReadRate and GetBytesWriteRate with GetBytesRate to reduce lock contention and replace IsMeta with MetaOrTable to reduce one decoding.
before:
2019/04/01 19:41:36 store 67 finish heartbeat, cost time: 5.772357802s
2019/04/01 19:41:36 store 116 finish heartbeat, cost time: 5.808891822s
2019/04/01 19:41:36 store 32 finish heartbeat, cost time: 5.813313253s
2019/04/01 19:41:36 store 4 finish heartbeat, cost time: 5.96003107s
2019/04/01 19:41:36 store 102 finish heartbeat, cost time: 5.880387345s
2019/04/01 19:41:37 store 156 finish heartbeat, cost time: 6.775514196s
2019/04/01 19:41:37 store 84 finish heartbeat, cost time: 7.029659414s
2019/04/01 19:41:38 store 61 finish heartbeat, cost time: 7.145771456s
2019/04/01 19:41:38 store 82 finish heartbeat, cost time: 7.29960913s
2019/04/01 19:41:38 store 5 finish heartbeat, cost time: 7.43298794s
2019/04/01 19:41:39 store 96 finish heartbeat, cost time: 8.178854224s
2019/04/01 19:41:39 store 1 finish heartbeat, cost time: 8.991736446s

after:
2019/04/01 20:19:35 store 14 finish heartbeat, cost time: 3.747103794s
2019/04/01 20:19:35 store 29 finish heartbeat, cost time: 3.758183656s
2019/04/01 20:19:35 store 94 finish heartbeat, cost time: 3.753249054s
2019/04/01 20:19:35 store 126 finish heartbeat, cost time: 3.744539406s
2019/04/01 20:19:35 store 99 finish heartbeat, cost time: 3.7437802s
2019/04/01 20:19:35 store 179 finish heartbeat, cost time: 4.208658087s
2019/04/01 20:19:35 store 5 finish heartbeat, cost time: 4.351112311s
2019/04/01 20:19:36 store 2 finish heartbeat, cost time: 4.472732085s
2019/04/01 20:19:36 store 1 finish heartbeat, cost time: 4.488936655s
2019/04/01 20:19:36 store 77 finish heartbeat, cost time: 4.5184613s
2019/04/01 20:19:37 store 30 finish heartbeat, cost time: 6.089952119s
2019/04/01 20:19:37 store 166 finish heartbeat, cost time: 6.313127002s

Check List
Tests

Unit test

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@56f4100). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #1489   +/-   ##
=========================================
  Coverage          ?   67.26%
=========================================
  Files             ?      158
  Lines             ?    15387
  Branches          ?        0
=========================================
  Hits              ?    10350
  Misses            ?     4090
  Partials          ?      947

Impacted Files
Coverage Δ

server/core/store.go
80.29% <100%> (ø)

table/namespace_classifier.go
81.77% <100%> (ø)

table/codec.go
91.66% <100%> (ø)

Continue to review full report at Codecov.

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


Remove profiling

This PR removes support for the profiling option, which added additional logic flow and was no longer needed.
This is a breaking change.

Codecov Report

Merging #20 (79a0358) into master (34ac616) will increase coverage by 1.34%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
+ Coverage   96.08%   97.43%   +1.34%
==========================================
  Files           5        5
  Lines         485      467      -18
==========================================
- Hits          466      455      -11
+ Misses         19       12       -7

Impacted Files
Coverage Δ

grandiso/test_grandiso.py
100.00% <ø> (ø)

setup.py
0.00% <ø> (ø)

grandiso/__init__.py
93.22% <100.00%> (+4.67%)
:arrow_up:

Continue to review full report at Codecov.

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


DELIA-42792 SystemServices: use strings in firmwareUpdateState

Right now firmwareUpdateState returns update state as int (ranging
from 0 to 6), this patch changes it so it returns state as string
like "DOWNLOADING_COMPLETE" or "PREPARING_TO_REBOOT".
Signed-off-by: Michał Łyszczek<EMAIL_ADDRESS>

Old PR. Closing it since change is not required


tsconfig.json options not applied if files section doesn't use extensions

VSCode Version: 1.5.3 (5be4091987a98e3870d89d630eb87be6d9bafd27)
OS Version: macOS Sierra 10.12

Steps to Reproduce:

Create an empty directory
Run npm init and then npm i --save-dev typescript
Run tsc --init (or ./node_modules/.bin/tsc --init)
Create test.ts and add code that's using Map and Set:

const foo = new Map();
const bar = new Set();

Edit tsconfig.json, set target to be es2015, and add files section with the test script (test) without its extension:

{
    "compilerOptions": {
        "module": "commonjs",
        "target": "es2015",
        "noImplicitAny": false,
        "sourceMap": false
    },
    "files": [ "test" ]
}

Run tsc (or ./node_modules/.bin/tsc), see that test.js is created with exactly the same code, as expected, and no warnings or errors have been seen.

At this point, tsc compiled the file test while adding the default .ts extension when resolving.
Now adding VSCode into the picture:

Run code . to open VSCode with the current testing directory loaded. Either that, or from GUI go to Files > Open and loaded the current directory.
Open test.ts file. At this point VSCode may ask about TypeScript version, if TypeScript 2.0 was installed by npm. It doesn't matter whether you choose TS 2.0 or TS 1.8, both produce the same behavior.
Observe Map and Set being unrecognized:

The observed behavior is inconsistent with tsc.
Now, to "fix" it:

Open tsconfig.json and change the files section to have the full file-name with its extension test.ts.
Reload TypeScript project or reload VSCode, and open test.ts. Everything is fine now.

However, I would expect vscode behavior in this to be the same as of tsc.

@nomaed thank you for opening this issue and for providing a thorough issue description.
@dbaeumer I have verified this issue with Insiders. Please advise next steps.

@waderyan the errors are produced by the tsserver. I would ask the TS team what the expect in this case. There is API that we can define the ScriptKind for files that don't have an extension however test.ts does have one and is clearly treated as a TS file inside the tsserver. However the tsconfig.json is not applied which I think is a bug in the tsserver itself.

ping @mhegazy
We have a question about this issue. See @dbaeumer's comment above and the OP's description of the problem.

This looks like a tsserver bug.

@mhegazy recommend moving it over?

yes please.

Closed and migrated to https://github.com/Microsoft/TypeScript/issues/11194


Remove more nondeterminism in MCMC test

Addresses, but by itself does not fix, #143.  Currently, the test fails deterministically.
@dellaert please take a look and make sure the test is correct.

Bump @dellaert

Bump @dellaert

Master no longer compiles on my Mac (Xcode 11.7 on Catalina). What's the intel on how to make this work?

@dellaert Probably you need the latest XCode beta: https://github.com/borglab/SwiftFusion/pull/181#issuecomment-693108106

So @dabrahams, the test seems correct. And I investigated the difference between the built-in random generator and the ARC4 one, and I can't see a meaningful difference of random-sampling from [-1,1]:

Yet the test fails for every seed I tried with ARC4, with a value for around -1 for the mean rather than zero. I'm stumped for now: brainstorm what could be wrong?

I think I know why. There is a copy operation in the constructor, so the algorithm is bifurcating a random number generator without changing the seed. I can try push a fix...

Yeah, just add a commit to the PR. I have no outstanding changes in the branch.

I think this is a problem about the container type AnyRNG. It is a type erased container, but it creates a copy, and it does not allow dereferencing the inner object...


Issue No. #5 [BUG FIXED] In the footer section Services, Projects and About Me links are not working.

closes #5

Hii @Iamdivyak please add a hacktoberfest-accepted label.


fix: updated sponsors

Description

Changed Platinum sponsors to new mention sponsor images

Related issue(s)

Resolves #528
Screenshot

That was fast! Do you think you can make Postman logo a bit bigger? It looks smaller than the rest.

That was fast! Do you think you can make Postman logo a bit bigger? It looks smaller than the rest.

Sure will do that

@fmvilas  its it good ?

Looks great on wide screens but not on mobile. Can you check it?

Looks great on wide screens but not on mobile. Can you check it?

sure..will do it

@fmvilas made the changes kindly check them

/rtm

@all-contributors please add @ritik307 for code


Feature Request: Additional hover tip info for variables

Would it be possible to add which file a file is defined in in the hover tip, having to ctrl + click to find out is a bit annoying

Nice idea. I will look into it!

Looks like we only do it for procedures at the moment. Should be simple to add to others.
https://github.com/halcyon-tech/vscode-rpgle/blob/main/server/src/providers/hover.ts#L53

the File and type definition definitely appear (using fixed format code)

Oh I see, thought you meant it was ONLY done for prototypes and procedures

I have made a change to show the @file property on all hover situations when the definition is an external file. This applies to all variables and procedures.
When defined in the same source:

When defined externally:


dashboard owner is not able remove themselves from dashboard owners

using reference in new issue feature
1. If you are dashboard owner - you can add chart from chart explore to dashboard directly. This works.
2. If you are NOT dashboard owner but are CREATOR of the dashboard (for example I created a dashboard, and then I asked an admin to remove me as owner, but I was still the creator of that dashboard) - I could still edit the dashboard (when on dashboard) and add charts, but I wasn't able to see this dashboard in the dropdown on chart explore, so I actually couldn't add chart directly from chart explore. --> assuming creator always keeps edit rights, this potentially might be a small bug, that creator can't add from chart explore directly (but simple workaround to add form dashboard)
3. And if I was not owner or creator, I couldn't edit dashboard and couldn't add chart into dashboard from chart explore. This is expected.
Unrelated thing I noticed, is that user (not admin) who is dashboard owner is not able remove themselves from dashboard owners (even if there are additional owners). This could be a bug. 🐛
Originally posted by @zuzana-vej in https://github.com/apache/incubator-superset/issues/11662#issuecomment-730014498

I believe this has been resolved, but if not, let's freshen up the issue with current context and repro steps.


build:all not working when /scripts contains a JS file with only a comment in it

Acceptance Criteria

?

Notes
To reproduce, create:
scripts/global.js
inside global.js add:
// testing
Run gulp scripts:build:all - task fails.

Cannot reproduce


animation flicker when opening modal

Bug description:
If the 'animation' parameter is set to false and there are any elements with ng-show="false" these elements are very briefly shown before being hidden. Adding ng-cloak directive does not help as suggested here. Neither does the fix suggested under issue #5183. Only way to avoid flicker is to set the 'animation' parameter to true.
Link to minimally-working plunker that reproduces the issue:
http://plnkr.co/edit/u75Qt0PLevcgQ1DQVoHa?p=preview
Version of Angular, UIBS, and Bootstrap
Angular: 1.5.5
UIBS:1.3.2
Bootstrap:3.3.6

@wesleycho, this issue sounds familiar and i thought we'd addressed it either with code or with an explanation.  do recall which?

Use ng-if.

Works for me.  Thanks @robjacobs.


Open PRs

I see a few attempts to contribute which are in limbo. I think this could be a great project, but think especially at the beginning it's important to merge almost every PR that's offered.
For more background on this concept, I recommend reading and to be more bold to adopt https://rfc.zeromq.org/spec/42/

I totally agree and have no idea why GitHub doesn't include these PRs in my notifications that I go threw religiously every day. Will figure out and be more responsive going forward!

Right on, I'm happy to co-maintain as long as not a lot of review is required.

Oh, cool! If you review PRs and approve them, I'll probably merge them quickly. If you are interested in this, feel free to comment on your vision. You want more details or less? Or other? I find it's quickly getting complex.

Sure, my vision is to follow C4 protocol as linked above.
The way it works is we do basic checks, like make sure it's not obviously broken and addresses a valid problem, and then we merge it. I went looking for the best defined, lightweight, easy process I could find for FOSS and found that. In my experience it works well and scales wonderfully.

I think there is no need to formalize anything at this point. All merged PRs were made by Danny who I paid to research the data. I'd happily merge more data if there was interest but so far I don't see that emerging.


missing isBlank and other is... functions

I just ran across this.  I was trying to remove blank results from a Wikidata query.  Please add them, as they should be easy to add.

It's on our list. To do it efficiently, we depend on a few other PRs currently in the pipeline. Once they are done, it will indeed be relatively straightforward.
In the meantime, you can use STRSTARTS as a workaround. This is relatively slow (for QLever's standards), so best to put the part of the query you want to filter into { ... }:
PREFIX wdt: <http://www.wikidata.org/prop/direct/>
SELECT ?class WHERE {
  { { ?item wdt:P31 ?class . }
  MINUS { ?class wdt:P279 ?super } }
  FILTER (! STRSTARTS(STR(?class), "_:u_"))
}

https://qlever.cs.uni-freiburg.de/wikidata/xTLBHK

This was already fixed by #1184, (see above), however the closing did not work as expected.

Thanks.


Hide problems when file is removed unconventionally

With the VS Code extension, if a Luau file has a warning and I delete it normally, the problem disappears from the Problems widget. But if I instead use git stash, the file is removed but the warning remains in Problems until I reopen VS Code.

This is an upstream bug: https://github.com/microsoft/vscode-languageserver-node/issues/1052


PIXI fails to create a Filter from custom shader code

I am struggling to get a custom Filter with my own shader code working with this error message:
VertexArrayObject.js:171 Uncaught TypeError: Cannot read property 'location' of undefined
    at VertexArrayObject.addAttribute (VM1006 pixi.js:2348)
    at Quad.initVao (VM1006 pixi.js:19874)
    at FilterManager.applyFilter (VM1006 pixi.js:18947)
    at Filter.apply (VM1006 pixi.js:18420)
    at FilterManager.popFilter (VM1006 pixi.js:18877)
    at Container.renderAdvancedWebGL (VM1006 pixi.js:9423)
    at Container.renderWebGL (VM1006 pixi.js:9360)
    at Container.renderWebGL (VM1006 pixi.js:9366)
    at WebGLRenderer.render (VM1006 pixi.js:17563)
    at Application.render (VM1006 pixi.js:8043)

Even the official example is failing with this bug:
http://pixijs.io/examples/#/filters/filter-mouse.js
This error message has been related to some compiler optimizations where glslify removed some unused uniforms, which pixi still tried to access. But this even occurs with a completely static fragShader without any uniforms.  gl_FragColor = vec4(1.0, 1.0, 0.0, 1.0)

The example works for me, what browser are you testing you in?

Google Chrome Version 62.0.3202.94 (Official Build) (64-bit)

Firefox Quantum 57.0 (64-bit)

OS: Ubuntu 17.10 artful

@doebi nice catch! Yeah, optimization removed something (I bet its sampler) and we forgot to check it.
For now, are you sure that you need filter without usage of sampler? May be its better to make renderer plugin like https://github.com/pixijs/pixi-plugin-example ?

To be honest, i haven't really dug deep into this topic yet. Just wanted to experiment with that feature the other day. I will do some testing with the renderer plugin approach. Thanks for the hint.
Is there already some ETA for v5?

@ivanpopelyshev Following your tutorial, i ran into the same issue.
I applied a really hacky patch to pixi.js to make it work for me.
diff --git a/js/pixi.js b/js/pixi.js
index 363f09c..d0a321b 100644
--- a/js/pixi.js
+++ b/js/pixi.js
@@ -2344,6 +2344,9 @@ VertexArrayObject.prototype.activate = function()
  */
 VertexArrayObject.prototype.addAttribute = function(buffer, attribute, type, normalized, stride, start)
 {
+  if (!attribute) {
+    return this;
+  }
     this.attributes.push({
         buffer:     buffer,
         attribute:  attribute,

I know this is not a solution, but it enables me to play with shader codes in v4 until v5 is out. (with possibly a better fix) :)

That's a good hack!

I got the same problem. Also with Ubuntu 17.10 artful. Are shaders at all stable in Pixi.js? Can I use them in production? How do I import a pixel shader?

Either as filter either as renderer plugin. They are stable but require serious knowledge both about webgl and about pixi architecture.
https://github.com/pixijs/pixi.js/wiki/v4-Creating-Filters
https://github.com/pixijs/pixi-plugin-example/

const filterCode = `void main(){
   gl_FragColor = vec4(1.0, 1.0, 1.0, 1.0);
}`;
const filter = new PIXI.Filter(null, filterCode);
someSprite.filters = [filter];

This much code lead to this error. Maybe I should go for a different constructor of the Filter? Or should I substitute null with something?

For a filter, you have to use texture coord and sampler , otherwise FilterManager fails to get location of attribute.
Renderer plugin doesnt have that requirement, try it.

You say it need texture coords and "sampler". Was sample or sampler? A typo?
I managed to make it working with the help of example. Thank you but. Why doesn't this page mention these details?
http://pixijs.io/examples/#/basics/custom-filter.js

Sampler. However its uniform and I dont think somebody will miss it. Our problem is about attributes.
Thanks to you, it does :) https://github.com/pixijs/pixi.js/wiki/v4-Creating-Filters#cannot-read-property-location-of-undefined

Oh, so that's where it was. I missed the wiki.

Thanks to you, it does
I've added it just now.

Also added notice in filter-mouse demo. We cant fix the issue in v4. We'll do something about it in v5.

That appears when some of properties that are required by FilterManager are not used in the shader.
How about to list these properties in there?

only the attribute, aTextureCoord -> vTextureCoord

This is getting very strange.
This code works fine:
varying vec2 vTextureCoord;
varying vec4 vColor;

uniform sampler2D uSampler;
uniform vec4 uTextureClamp;
uniform vec4 uColor;

void main(void)
{
    gl_FragColor = texture2D(uSampler, vTextureCoord);
    gl_FragColor.r = 0.0;
    gl_FragColor.g = 0.0;
    gl_FragColor.b = 0.0;
}

But if I add gl_FragColor.a = 0.0; to the end then it says Cannot read property 'location' of undefined. It seems as though I can only change 3 coords out of 4 at most. What is this? What is wrong?

@germansokolov13 this behaviour makes total sense. Just think about it:
As i mentioned in the original post this error occurs due to glsify optimizing the shader code.
If you add gl_FragColor.a = 0.0; it can optimize your call to texture2D, cause it gets fully overwritten, but until then it takes alpha channel from texture2D
If glsify optimizes the call to texture2D, webgl does not allocate memory for uSampler, hence when pixi wants to upload data to that uniform it fails, cause there is simply no space for that allocated.
You got some options:

Apply the same hack as i did (for now), which basically checks if an attribute is allocated before uploading data to it.
Use uSampler, so glsify doesn't optimize it away.
Or just wait for v5.

Hopefully, this helps.

Or use renderer plugin.

Oh, now I get it! Maybe we should add Doebi's patch to the next minor release of Pixi.js? Should I create a PR?

Yep, PR, but different place.
https://github.com/pixijs/pixi.js/blob/dev/src/core/renderers/webgl/utils/Quad.js#L93
https://github.com/pixijs/pixi.js/blob/dev/src/core/renderers/webgl/managers/FilterManager.js#L242
I suggest extra class which overrides method and that is used in FilterManager.

Hi there,
I'm experiencing a similar problem with my shader code. It is a variation of the default BlurYFilter shader code just with a bigger kernel and a little variation on the gaussian variable, so I'm not entirely sure why it's not working.
It's working fine on Windows, MacOS and Android as far as I tested I only have issues in iOS.
Here's the shader code:
vertex:
attribute vec2 aVertexPosition;
attribute vec2 aTextureCoord;
uniform float strength;
uniform mat3 projectionMatrix;
varying vec2 vBlurTexCoords[15];
void main(void){
   gl_Position = vec4((projectionMatrix * vec3((aVertexPosition), 1.0)).xy, 0.0, 1.0);
   vBlurTexCoords[0] = aTextureCoord + vec2(0.0, -7.0 * strength);
   vBlurTexCoords[1] = aTextureCoord + vec2(0.0, -6.0 * strength);
   vBlurTexCoords[2] = aTextureCoord + vec2(0.0, -5.0 * strength);
   vBlurTexCoords[3] = aTextureCoord + vec2(0.0, -4.0 * strength);
   vBlurTexCoords[4] = aTextureCoord + vec2(0.0, -3.0 * strength);
   vBlurTexCoords[5] = aTextureCoord + vec2(0.0, -2.0 * strength);
   vBlurTexCoords[6] = aTextureCoord + vec2(0.0, -1.0 * strength);
   vBlurTexCoords[7] = aTextureCoord + vec2(0.0, 0.0 * strength);
   vBlurTexCoords[8] = aTextureCoord + vec2(0.0, 1.0 * strength);
   vBlurTexCoords[9] = aTextureCoord + vec2(0.0, 2.0 * strength);
   vBlurTexCoords[10] = aTextureCoord + vec2(0.0, 3.0 * strength);
   vBlurTexCoords[11] = aTextureCoord + vec2(0.0, 4.0 * strength);
   vBlurTexCoords[12] = aTextureCoord + vec2(0.0, 5.0 * strength);
   vBlurTexCoords[13] = aTextureCoord + vec2(0.0, 6.0 * strength);
   vBlurTexCoords[14] = aTextureCoord + vec2(0.0, 7.0 * strength);
}

fragment:
varying vec2 vBlurTexCoords[15];
uniform sampler2D uSampler;
void main(void){
   gl_FragColor = vec4(0.0);
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[0]) * 0.013068780984604511;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[1]) * 0.013907007172070673;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[2]) * 0.017439264394216315;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[3]) * 0.028762309061254498;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[4]) * 0.05603114255667656;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[5]) * 0.10421702583793174;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[6]) * 0.163461199220823;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[7]) * 0.2062265415448454;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[8]) * 0.163461199220823;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[9]) * 0.10421702583793174;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[10]) * 0.05603114255667656;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[11]) * 0.028762309061254498;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[12]) * 0.017439264394216315;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[13]) * 0.013907007172070673;
   gl_FragColor += texture2D(uSampler, vBlurTexCoords[14]) * 0.013068780984604511;
}

for comparision here are the shaders of the default BlurYFilter:
vert:
attribute vec2 aVertexPosition;
attribute vec2 aTextureCoord;
uniform float strength;
uniform mat3 projectionMatrix;
varying vec2 vBlurTexCoords[5];
void main(void)
{
  gl_Position = vec4((projectionMatrix * vec3((aVertexPosition), 1.0)).xy, 0.0, 1.0);
  vBlurTexCoords[0] = aTextureCoord + vec2(0.0, -2.0 * strength);
  vBlurTexCoords[1] = aTextureCoord + vec2(0.0, -1.0 * strength);
  vBlurTexCoords[2] = aTextureCoord + vec2(0.0, 0.0 * strength);
  vBlurTexCoords[3] = aTextureCoord + vec2(0.0, 1.0 * strength);
  vBlurTexCoords[4] = aTextureCoord + vec2(0.0, 2.0 * strength);

}

frag:
varying vec2 vBlurTexCoords[5];
uniform sampler2D uSampler;
void main(void)
{
    gl_FragColor = vec4(0.0);
    gl_FragColor += texture2D(uSampler, vBlurTexCoords[0]) * 0.153388;
    gl_FragColor += texture2D(uSampler, vBlurTexCoords[1]) * 0.221461;
    gl_FragColor += texture2D(uSampler, vBlurTexCoords[2]) * 0.250301;
    gl_FragColor += texture2D(uSampler, vBlurTexCoords[3]) * 0.221461;
    gl_FragColor += texture2D(uSampler, vBlurTexCoords[4]) * 0.153388;

}

Does Anybody have any ideas why this could go wrong?

@tyleet are you sure its the same issue? What do you see in the console?

Hi,
sorry for the late response, I was not able to make some screenshots over the weekend. It looks to me as if it fails on the same line. "attribute" is undefined. Here are the screenshots from the remote console:

@ivanpopelyshev shall we close this issue to prevent others from warming it up with similar issues?
I wouldn't consider it an issue anymore, since i got your promise, that it will be fixed in v5.

@doebi closing.

@tyleet the number of samples taken from texture can be limited on different systems.


running portainer code

Description

Steps to reproduce the issue:
1.how to run portainers code on my computer
2.
3.
Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
Technical details:

Portainer version:
Target Docker version (the host/cluster you manage):
Platform (windows/linux):
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):
Target Swarm version (if applicable):
Browser:

Have a look at our documentation on how to contribute here: http://portainer.readthedocs.io/en/stable/contribute.html


Member expression refinement

Follow-up to #7597
Fixes #7475 again
Fixes #2990 again

I'm probably missing bunch of refinements with TestPropT

/cc @mvitousek looks like I missed some things in recent PR

Hmm, this breaks one of tests

Closing this as the examples in the two linked issues now work.


Added dictionary manipulation methods

closes #153

Implemented the following methods in the source code:
add_item: Adds a new key-value pair to the dictionary.
update_item: Updates the value for an existing key in the dictionary.
remove_item: Removes a key-value pair from the dictionary.
check_for_key: Verifies if a specific key exists in the dictionary.
iterate_keys: Loops through all the keys in the dictionary.
iterate_values: Loops through all the values in the dictionary.
merge_dictionaries: Combines two dictionaries into one, updating values as needed.
copy_dictionary: Creates a shallow copy of the dictionary.
clear_dictionary: Removes all items from the dictionary.

Added relevant test cases in the test file.

@UTSAVS26 : please review and go ahead with it.

@UTSAVS26 : are you active on this repo?


allow local storage of multiple drawings

keep array of multiple images in local storage. have a menu for opening, saving, and deleting drawings. this way we can go back to old drawings. menu can have "import" in the future.

:santa:


Fix SetTokenAndUserInfoAsync type

Story details: https://app.shortcut.com/cartoteam/story/184705/fix-settokenanduserinfoasyn-types

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 74.407%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
695

Relevant Lines:
866

💛  - Coveralls


Add direct installation links to README

Assigning to you since you should own the publishing steps.

Resolved with PR #115


Fixed Typo

Thanks for pointing out this link broke! I'm going to close this as we just merged a PR to remove it from the list since it's no longer open :)


Thumbsticks not mapped for Windows Mixed Reality Motion Controllers

Hey, I commented on your youtube video showing off the VR joystick about adding support for the WMR controllers. I've attached a picture of the debug using the double joystick script
 looks like the sticks are showing up as Axis 2 and go from ~0 to ~1/-1 at the extremes for the X and Y.
If you need more information just let me know!
Thanks again for looking into this.

Sorry for not looking into it for so long. I might have some time for it this weekend. I'm thinking of bigger redesign, to allow configure of any function to any key. But in meantime you could tell me the preferable axis for the WMR joysticks.

No problem at all! Thanks for taking time to work on this. So I guess I was wondering if they could be mapped to act as a hatswitch instead of a joystick? Example being like position ~0:~1 could equal button 20 in vJoy or something along those lines.

Hey mdovgialo, was wondering if you had gotten a chance to take a look at this. No big deal if you haven't just wondering 😁

¯_(ツ)_/¯


Support BQ clustering with value provider

Fixes #30821

R: @liferoad
R: @chamikaramj (for the change to BigQueryIOTranslation)

@ahmedabu98 @chamikaramj @Abacn where are we on this PR? Does this need to be a release blocker?

This is requested by one customer. What is blocking this PR?


Add method addTopLevelLabel() to EcsEncoder and EcsLayout (#41)

Thanks for the PR! As soon as you signed the CLA, I can merge it in 🙂

I've signed the CLA. You can merge it. :-)


Wrong CUDA ImportError message in tf-gpu 1.14

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version: 1.14
Python version: 3.6
Installed using virtualenv? pip? conda?: conda
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the problem
Provide the exact sequence of commands / steps that you executed before running into the problem
tensorflow-gpu 1.14 missing CUDA error tells to download CUDA 10.0 but links to CUDA 9.0:
ImportError: Could not find 'cudart64_100.dll'. TensorFlow requires that this DLL be installed in a directory that is named in your %PATH% environment variable. Download and install CUDA 10.0 from this URL: https://developer.nvidia.com/cuda-90-download-archive
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@matthiasseibold
Any update on this issue please. Thanks!

As TF1.14 will no longer be updated, can you please switch to 1.15 or 2.1 and tell us if the issue is still present?

@matthiasseibold Please let us know if the above comment helps resolve the issue.

It's not an issue for me, I just wanted to let you know that the error message might be irritating because the link is wrong


Suggestion: Add combined size as size on folder

As of right now, when you hover over a folder, it just shows 000000 under size. It would be cool if it instead displayed the combined size of every item in the folder so you could tell how much space a certain thing takes up (E.G. Appvars).

No, too much extra parsing. I'll just remove the entry.

Closed with 8af4d670642b56f581dc48960ed3cd0fb04ac7e8


Clear Call List After No Login

I rule and have completed some work on Case Manager that's ready for review!
Clears user's call list if they don't sign in for 2 weeks
This pull request makes the following changes:

Adds constant TIME_BEFORE_INACTIVE to User model
Inside User#clear_call_list clears patient list if User#last_sign_in_at is 2 weeks or longer
Test new code in the user model test file

It relates to the following issue #s:

Fixes #666

Current coverage is 98.72% (diff: 100%)

Merging #673 into master will increase coverage by <.01%

@@             master       #673   diff @@
==========================================
  Files            27         27
  Lines           546        549     +3
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            539        542     +3
  Misses            7          7
  Partials          0          0

Powered by Codecov.  Thanks a ton for knocking this out, bud. Love how clean and readable it is!
I'll pull down and test locally tomorrow before merge, but I can't see anything that should hold this up -- looks great to me.

Pounding the button -- thanks again @malively !


remote images for timeline - suggestion/workaround

I am allowing external images in my timeline, then {{site.baseurl}} must not be prepended. My workaround is not the most elegant (that would be testing the variable "image" for "http(s)://") ... but it works. For remote images, I define a {{event.imageremote}} in _config.yml
events:
  - image: "https://media.giphy.com....gif"
    imageremote: True

and insert a test into career.html
<img class="img-circle img-responsive" src="{% unless event.imageremote %}{{ site.baseurl }}{%endunless%}{{ event.image }}" alt="">

so all the other (=local) images do get {{ site.baseurl }} prepended.
It works, so I thought, why not share it with you guys.
Many many thanks for all the work you put into this. My new site looks great!

I see, but I really believe that the images should be local, because if for example the remote image suddenly changes it can mess up your website, if for example is not a square :wink:
Avoid taking dependencies, as always :wink:
Thanks for the feedback!
I had no idea that the repo would be used by so many people, and that's great!
Let's make the web a better place :wink:

Thanks a lot! Yes, I get your point. Still, I want to link to external content. And already built that extension described above, anyways.

I had no idea that the repo would be used by so many people, and that's great!

Yes. Before I started, I compared a lot of different themes. And yours was closest to what I wanted to build. And beautiful, too.
Here, have a look: My new website is launched - http://altsheets.ddns.net/assetgraphs/v2/products/ - using your theme as the basis, and then going wild on modifying it.
Thank you so much for your theme. I worked a lot on it, to adapt it to my needs, and fix some problems - but your underlying design was already so great, that it looks quite nice, I think.

Let's make the web a better place

Yes. And a happy place :-)

The credits are totally fine like this, don't worry :wink:


Bump to 0.3.12

Includes #71

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@NicoZweifel Hm odd. Can you see the failed Storybook UI tests?

Yea I was afraid this might happen at some point. I can prepare a PR. It is because of navigator.clipboard which is missing in tests. Ideally we mock the clipboard.

I created a Hotfix PR that targets this release branch.
The tests pass in the runner locally now and they should pass in the action.
I added a mock for navigator.clipboard with the help of spyOn.
It works and was kind of tricky to figure out since old solutions all dont work but I am not 100% sure if this is the right way since jest in storybook seems to be a bit different.
Is there a place to put testing utils? I have some duplicated code in the tests and we could maybe organize the mocks better (if that is possible with storybook-jest).

@NicoZweifel How generic are the testing utils? We can create a package if it's generic enough for other people to benefit from it. 👀 If you want complete ownership over it you can even publish your own repo/npm package for it 👀

@hahnbeelee I think these conflicts might be because of squashing


[FLINK-18607][build] Give the maven module a human readable name

What is the purpose of the change
Make the build output more human readable for the developers van various situations.
Brief change log
Add or change the <name> of each maven module (i.e. in every pom.xml)
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
The simplest way to test is by doing a mvn clean and verify the list of names at the end looks good.
Also looking at the change set verify that the names for each module are correct.
In some exceptional cases I made the name reflect a bit more the purpose like in the case of
flink-examples/flink-examples-build-helper where the description indicates This is a utility module for building example jars to be used in flink-dist. so I named it Flink : Examples : Dist :
Fun fact: I noticed that the ordering of the modules during the build is changed a lot by maven because apparently the ordering of the modules in the pom.xml does not meet the configured dependencies between the modules. So maven reorders the build by itself.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

Note that all 'pom'  modules have a name that ends with <space>:<space> because that makes the ordering in the Maven overview in IntelliJ better.
Also note that the naming convention I show here is based upon my own preference: what I find easy to read.

@nielsbasjes Thanks for the update, I will merge this once tests passed.


Make npm run dev exit on quit

This makes npm run dev quit the app, rather than get stuck waiting for a ^C.  Unfortunately, nuxtron isn't quite flexible enough to do so…
At this point though we're basically not really using much of nuxtron; we might want to consider just dropping it entirely and switching to (hopefully better organized) build & dev scripts instead.  So this PR is more as a start of discussion than anything.
So our choices are:

Leave things alone, and just close this PR with no follow-up.
Merge this PR.
Morph this PR to drop nuxtron completely instead.

Let me know what seems like a better path to go down.

@mook-as feel free to merge yourself when ready.


Fix the scheduler plugin manifests

Fix minor mistakes that found their way in the scheduler plugin manifests, and align to how the default kube scheduler behaves.

/lgtm


es.test.mode "info" is sending results that doesn't contain errors

I'm testing on jmeter 4.0 and setting es.test.mode="info" is still sending messages to ElasticSearch even when there isn't an error.
I'm using ElasticSearch 6.2.4

Kibana Result without filter:

Filtering my messages based on ErrorCount:1:

Hey @hanaleung ,
I'll be glad to look into the issue. May I know which realease of the plugin you are currently using?
Thanks,

Version 2.2.5

Using version 2.2.5 myself,
I am unable to reproduce the issue on a single JMeter execution. I'll have to try on a remote execution later tonight.

Would it be possible for you to supply me with the JSON data of the samplers for which the request/response body should not appear? (don't forget to blur the information related to the app you are testing)
Thanks,

As defined in the documentation though:

Info mode should be used when the tests are being executed in a production context. This mode will only send the details of the responses upon failed requests. This allows you to be able to locate the errors when visualizing the tests through either Kibana/Grafana.

By "details" what we mean is that only the samplers which are unsuccessful will have their request/response body sent to ElasticSearch. It doesn't mean that only the errors will be sent to ElasticSearch.
It could however, be a good idea to introduce such a mode.
Thanks,

Hi,
i may be misinterpretting what is a failed request, but here is what i expect should be sent to ElasticSearch under "info" mode which i do see on elasticSearch and can identify via ErrorCount field.
{   "_index": "stockholm_capacity",   "_type": "SampleResult",   "_id": "a4bDt2MBDfEYHFPh-WVH",   "_version": 1,   "_score": null,   "_source": {     "EndTimeInMs":<PHONE_NUMBER>910,     "ContentType": "",     "EndTime": "2018-05-31T15:54:23.910-0400",     "IdleTime": 0,     "ElapsedTime": "Jan 31, 2018 12:36:53 AM",     "ErrorCount": 1,     "Timestamp": "2018-05-31T15:54:23.910-0400",     "URL": "_removed_",     "Bytes": 1158,     "SentBytes": 0,     "ElapsedTimeInMs": -556,     "AllThreads": 118,     "DataType": "text",     "ResponseTime": 0,     "SampleCount": 1,     "ConnectTime": 0,     "RequestHeaders": "",     "RequestBody": "_removed_",     "StartTimeInMs":<PHONE_NUMBER>910,     "ResponseCode": "Non HTTP response code: java.net.URISyntaxException",     "StartTime": "2018-05-31T15:54:23.910-0400",     "ResponseMessage": "Non HTTP response message: Illegal character in path at index 104: _removed_",     "ResponseHeaders": "",     "AssertionResults": [       {         "failure": true,         "name": "json_assertion",         "failureMessage": "Expected to find an object with property ['catalogGroupView'] in path $ but found 'java.lang.String'. This is not a json object according to the JsonProvider: 'com.jayway.jsonpath.spi.json.JsonSmartJsonProvider'."       }     ],     "Latency": 0,     "ResponseBody": "",     "GrpThreads": 118,     "BuildNumber": 0,     "BodySize": 0,     "ThreadName": "guest_shopper_api 1-5",     "SampleLabel": "_removed_"   },   "fields": {     "EndTime": [       "2018-05-31T19:54:23.910Z"     ],     "StartTime": [       "2018-05-31T19:54:23.910Z"     ],     "Timestamp": [       "2018-05-31T19:54:23.910Z"     ]   },   "sort": [<PHONE_NUMBER>910   ] }
And then here is what I don't expect to be logged but is currently being logged anyway:
{   "_index": "stockholm_capacity",   "_type": "SampleResult",   "_id": "X5DTt2MBDfEYHFPhYpk4",   "_version": 1,   "_score": null,   "_source": {     "EndTimeInMs":<PHONE_NUMBER>836,     "ContentType": "application/json",     "EndTime": "2018-05-31T16:11:13.836-0400",     "IdleTime": 0,     "ElapsedTime": "Jan 31, 2018 12:53:43 AM",     "ErrorCount": 0,     "Timestamp": "2018-05-31T16:11:13.765-0400",     "URL": "_removed_",     "Bytes": 1025,     "SentBytes": 293,     "ElapsedTimeInMs": -489,     "AllThreads": 150,     "DataType": "text",     "ResponseTime": 71,     "SampleCount": 1,     "ConnectTime": 0,     "StartTimeInMs":<PHONE_NUMBER>765,     "ResponseCode": "200",     "StartTime": "2018-05-31T16:11:13.765-0400",     "AssertionResults": [       {         "failure": false,         "name": "json_assertion",         "failureMessage": ""       }     ],     "Latency": 71,     "GrpThreads": 150,     "BuildNumber": 0,     "BodySize": 727,     "ThreadName": "guest_shopper_api 1-137",     "SampleLabel": "_removed_"   },   "fields": {     "EndTime": [       "2018-05-31T20:11:13.836Z"     ],     "StartTime": [       "2018-05-31T20:11:13.765Z"     ],     "Timestamp": [       "2018-05-31T20:11:13.765Z"     ]   },   "sort": [<PHONE_NUMBER>765   ] }

Hey @hanaleung,
I released version 2.3.0 a few minutes ago : https://github.com/delirius325/jmeter-elasticsearch-backend-listener/releases/tag/2.3.0
Feel free to manually upgrade and use the "error" test mode. The "error" test mode will only send the samplers that failed to the ElasticSearch engine :) !
Waiting for the JAR file to be on Maven's central repository before pushing the new version to JMeter-Plugins
Closing the issue :)
Cheers,

Thanks for clearing up my misunderstanding and your prompt response.
it seems error mode is what i was looking for and i had thought info mode was that behavior but now i see the difference.
I'll pull the latest and I'll give the error mode a try
Cheers!

@hanaleung : You're very welcome, don't forget to star the repo so it gets noticed :) !
As a side note, PR created on Jmeter-plugins
https://github.com/undera/jmeter-plugins/pull/257


Update Flags.cs

MetnoLicense type is mistaken - should be string instead of List

Ooh, nice catch. Thanks!

You’re welcome!
Great project, I’m heading to using it in my own weather app for Windows Phone 8.1!
I hope you don’t mind to be mentioned in the About page.
From: Jerome Cheng<EMAIL_ADDRESS>Sent: 18 януари 2015 г. 09:48 ч.
To: jcheng31/ForecastPCL
Cc: FourTonMantis
Subject: Re: [ForecastPCL] Update Flags.cs (#4)
Ooh, nice catch. Thanks!
—
Reply to this email directly or view it on GitHub https://github.com/jcheng31/ForecastPCL/pull/4#issuecomment-70399517 .  https://github.com/notifications/beacon/AKFqMjsKMXPC1dO0V46SFOTIyMQa181mks5ni1yWgaJpZM4DTyYG.gif

Thanks! Glad you find it useful :)


Implementation of 0x3a GASPRICE

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: 97
What is the new behavior?

0x3a GASPRICE is implemented

Other information

Closing this PR as it is blocked and > 15 days old. Still available to reopen it and explore further solutions for GASPRICE opcode:)!


New Project: Microsoft FlightSim

Perhaps it is a few different entries in Project Explorer

3DS Max export, https://github.com/AsoboStudio/FlightSim-glTF-exporter (and https://github.com/AsoboStudio/FlightSim-3dsMax-tools?)
Blender exporter, https://www.fsdeveloper.com/wiki/index.php?title=Blender2MSFS
Importers?
FlightSim runtime itself

CC @bghgary for suggestions or corrections.

I don't actually know how any of this works :-)
@lpierabella may be able to answer better.

this one:   https://github.com/AsoboStudio/FlightSim-glTF-exporter   is a fork of the Babylon 3ds max exporter, it could be interesting for the community to see how ASOBO extensions have been implemented
the second one https://github.com/AsoboStudio/FlightSim-3dsMax-tools is a bunch of 3ds max script to help artist developing content for MSFS.
Even if some of them are related to glTF, I do not think it worth any mention in the project explorer
The Blender exporter speaks for himself, it could be useful to see how ASOBO extensions are implemented in Blender..however, it is not maintained by Asobo.
MSFS uses tinyglTF as a glTF import library  (with some adaptations)...this is not public.
So I suggest mentioning only:
https://github.com/AsoboStudio/FlightSim-glTF-exporter
https://www.fsdeveloper.com/wiki/index.php?title=Blender2MSFS
and FlightSim itself.

@elpie89 really appreciate the prompt and detailed response.  It is amazing what you have achieved with MSFS and with glTF.


Best way to update TextView text

I spent some time trying to figure this out today. I have a custom class that generates a LinearLayout: (trimmed down)
impl MyLovelyClass {
    fn new() -> Self {
        Self {
            text_message: TextView::new(""),
        }
    }

    fn make_view(&self) -> LinearLayout {
        LinearLayout::horizontal()
            .child(self.text_message
                .min_width(35)
                .max_height(1))
    }

    fn update_data(&mut self, text: &str) {
        self.text_message.set_content(text);
    }
}

Now, this obviously isn't going to work because .child() consumes the TextView. This is an ownership problem but the big question is: How can I update a TextView from a second thread? I'll have the data triggered outside the user interface here, so the events that would hand it back to me aren't going to be overly helpful, and I don't think passing a reference to the main Cursive instance for each of my custom objects so that I can use find_id() makes a whole lot of sense.
The ProgressBar has an Arc<Counter>, but I have no such luck for TextViews.
Thanks!

Take a look at https://docs.rs/cursive/0.7.5/cursive/struct.Cursive.html#method.cb_sink and read the note for it.
To access your element you can call with_id on it to assign an ID on creation and afterwards you can call https://docs.rs/cursive/0.7.5/cursive/struct.Cursive.html#method.call_on_id
That should help, else wise look at the examples, they are really good.

Hello, and thanks for the report!
In cursive's ownership model, you have to give ownership of the View. This means cursive is the only owner and manager of the views used in the Layout.
From there, you usually have 2 solutions:

For simple projects, you can have the views own their data, and use the cursive tree as your data holder. This means MyLovelyClass cannot own the TextView, or even keep a reference to it.
What you can do is ask cursive to update this view with a new value now and then.

To update the new value, you will need to wrap your view with with_id, and call Cursive::call_on_id, or Cursive::find_id. This requires access to a &mut Cursive; you can call that in most event callbacks.
If the request to change the value doesn't come from a UI event but from a separate thread, you won't have access to a &mut Cursive there. Instead, you can use Cursive::cb_sink to get a channel, on which you can send callbacks from a different thread (and this callback will have access to a &mut Cursive).

When this is not possible or desirable, you need to separate the data from the view itself. This often means implementing View on your own class, and using internally an Arc<Mutex<T>> to access the data.

The second solution is closer to the Model/View separation often advocated, and I agree that it would be easier with some extra views in library (like a view displaying an Arc<Mutex<String>>, or the same for lists, ...).

Thanks for the ideas! I’ll mull it over. This feels very jQueryish from the web world which is going to be a revisit after using Angular for so long.
Hypothetically, is there any time displayed data might be modified? Thinking about employing an immutable reference / RwLock might be a fun exercise, but I’m a fringe edge case. I’ve done some non-ownership friendly things with Urwid in Python so again, brain adjusting needed. Both this and that project are fantastic wrappers around curses and is very much appreciated!
I really like the channel idea as I think it’ll require the least bootstrapping and fits my data pathway well. I’ll report back on my battle against my own thought processes tomorrow!

I just pushed 799ec812e4f4b54e17300a21ac4bbc9e011b5afb, which adds TextContent, to allow managing the content separately from the view. Examples incoming; in the meanwhile, here are a few samples:
let content = TextContent::new("");
siv.add_layer(TextView::new_with_content(content.clone()));

// You can also retrieve the shared content with `TextView::get_shared_content`

// Could be updated remotely
content.set_content("bob");


Hilbert Tutorial

This isn't working on my end, though I believe it should work. Would love if someone could try out these notebooks and see if they work, or if they need to be modified. Fixes https://github.com/SupertechLabs/client-superstaq/issues/465

Btw -- when submitting via the client is it still necessary to set your options with the cq token ? @vtomole

Btw -- when submitting via the client (css specifically) is it still necessary to set your options with the cq token ? @vtomole

I think the answer is no, but I can check and I'll take a first pass at the notebooks

@SalahedeenIssa Just want to make sure I'm running into the same issues as you:

I can run the qss notebook fine
I'm able to compile for the css notebook but not submit (I did not set options)

@stephanielee9 The qss notebook works for me too, and I can also compile using the css notebook, but regardless of whether I have options or not, I am unable to submit using css it seems. @vtomole Could you try submitting using css to see if this is just a me problem?

@SalahedeenIssa In addition to the notebook comments, it would be good to update:

the cq.rst file to have a brief description (see https://superstaq.readthedocs.io/en/latest/optimizations/ibm/ibm.html) - doesn't need references but if there's a good cite, could be useful to the user
It may be good to update all the instances of cq / ColdQuanta to Hilbert

Had no issue running any of the notebooks :)

@SalahedeenIssa A few things:

Need to update docs/source/index.rst to switch the cq files to hilbert files
Delete the cq directory from the optimizations directory
Also my bad (I gave you the markdown syntax before): update the link in hilbert.rst to rst syntax: https://sublime-and-sphinx-guide.readthedocs.io/en/latest/references.html#links-to-external-web-pages
I think the install instructions are a little out of sync with the other tutorials (e.g., for qss it should be pip install -q qiskit-superstaq[examples])
There seems to be description missing under the single and multiple circuit compilation
The Binder and Colab links seem to still not be working for me


Allow goog.date.relative.format to format relative dates with second precision

In our application it would be helpful to get a relative date format (edit: from goog.date.relative.format) with second precision, e.g. "in 5 seconds", "35 seconds ago", e.t.c.
Is it likely that a PR for this functionality would be accepted? I'm happy to put it together if the idea is acceptable. One thing I'm not too sure about is backwards compatibility. Is it acceptable for the return value of times less than a minute from now to change from (for example) "1 minute ago" to "37 seconds ago"?

@fabalbon Don't we already have something like this?

@joeltine We have closure/goog/date/relative.js, but it has minute precision. If I'm not mistaken, what @danielcompton is proposing is to add second precision to this API. Regarding backwards compatibility I think it might be a problem because users might be expecting minute precision so I wouldn't change the default behavior.

If I'm not mistaken, what @danielcompton is proposing is to add second precision to this API.

Yep, that's correct.

Regarding backwards compatibility I think it might be a problem because users might be expecting minute precision so I wouldn't change the default behavior.

I can see two possibilities to preserve backwards compatibility:

Provide an extra parameter to goog.date.relative{format, formatDay, formatPast, ...}, as to be consistent, all of the relative formatters should be able to return the same format.
Make setTimeDeltaFormatter a publicly usable function and let users override it. However this doesn't feel like a very good approach.
Add another function to toggle if relative date formats include seconds.

I'm happy to put together a PR with some guidance on which option (or another) would be preferred. Not sure if I'm in love with any of the options. Instantiating and using a formatter feels like a better option. There's a comment from 2013 suggesting that this be moved to goog.i18n.DateTimeFormat once a new version of CLDR comes out.

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


Is there a function to get the area of a face in the mesh, or do I just calculate the determinant of the Jacobian?

Hi @rmvega1025 ,
In order to get the surface of a face in a mesh you indeed need to use the determinant of the Jacobian, I am not aware of a function that computes the surface of a face but the following code should be close to what you need:
double ComputeFaceSurface(const FaceElementTransformations &Trans, const IntegrationRule &ir)
{
   double res = 0.0;
   // We integrate over quadrature points in case of a curved mesh
   for (int p = 0; p < ir.GetNPoints(); p++)
   {
      const IntegrationPoint &ip = ir.IntPoint(p);
      Trans.SetAllIntPoints(&ip);
      if (dim == 1)
      {
         const IntegrationPoint &eip1 = Trans.GetElement1IntPoint();
         nor(0) = 2*eip1.x - 1.0;
      }
      else
      {
         CalcOrtho(Trans.Jacobian(), nor);
      }
      res += ip.weight * nor.Norml2();
   }
   return res;
}

Tell me if this works for you,
-Yohann

One simplification: you can simply use Trans.Weight() instead nor.Norml2() -- no need to compute nor explicitly this way.
One option for choosing ir is to use:
   const IntegrationRule &ir = IntRules.Get(Trans.GetGeometryType(),
                                            Trans.OrderJ());


Make testNamePattern and testPathPattern play better together

This is part of set of polish PRs for the typeaheads #2595
Summary
This adds the ability to clear the "active filters" and it also makes them work better together.
Test plan
Added some tests

This PR still has a UI issue that needs to be resolved... The test summary also shows the active filters... what do you think that it is the best way to move forward with this?

What should be the correct UI for this?

Please make sure to use the right prettier configuration (run with yarn prettier, for example, see package.json).
How about we change the summary in watch mode to show "Active filters: …" instead of "Ran all test suites"? That way you don't have to repeat yourself when typing "w" and it doesn't add any more text to the watch mode.

But if we just change the test summary copy, how would it work when the screen has been cleared?

Hmm, that's a good point :(

Codecov Report

Merging #3327 into master will increase coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #3327      +/-   ##
=========================================
+ Coverage   64.12%   64.2%   +0.08%
=========================================
  Files         176     176
  Lines        6531    6546      +15
  Branches        4       4
=========================================
+ Hits         4188    4203      +15
  Misses       2342    2342
  Partials        1       1

Impacted Files
Coverage Δ

packages/jest-cli/src/constants.js
100% <ø> (ø)
:arrow_up:

packages/jest-cli/src/TestNamePatternPrompt.js
94.54% <100%> (+0.2%)
:arrow_up:

packages/jest-cli/src/TestPathPatternPrompt.js
98.14% <100%> (+0.07%)
:arrow_up:

packages/jest-cli/src/watch.js
76.29% <100%> (+2.1%)
:arrow_up:

Continue to review full report at Codecov.

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


YGRoundToPixelGrid has accumulated error?

every time call YGNodeCalculateLayout function, cached node call 'YGRoundToPixelGrid' function  has accumulated lead to value's error (iOS).

Can you give repro steps, or perhaps write a test case? Thanks!

@AugustRush I think this is fixed via b378a6. It would be great if you could give a short heads up, if this is the error you perceived.

This issue still exsit in my project, if i ignore this everytiing will be OK.

Could you please provide a deeper explanation? I didn't get the problem yet. As @emilsjolander suggested it would be highly appreciated if you could provide the necessary steps to reproduce the issue.

In my project, i use Yoga to layout all subviews on UITableViewCell , there has some buttons , every time i tapped the button , i need to mark the button's yoga property to 'dirty', it will re-layout and then this issue happened!

I'm still not 100% sure if I understand the problem correctly. But it sounds like yoga doesn't preserve the origin and shifts the layout around. Have a look at applyLayoutPreservingOrigin if this solves your problem. Otherwise you'll need to provide a code example to be able to assist you with that.

There have been a lot of fixes lately regarding accumulation errors in rounding. If you still see an issue, please reopen.


Add sops decrypt to helm-controller

Usecase: helm chart with encrypted values files in app repository or packaged and uploaded to internal chartmuseum, then helm-controller decrypts them before installing/upgrading helm chart, in the same way as kustomize-controller do.
I went through documentation on using Kustomize and secretGenerator for handling secrets, and that works pretty well with upstream helm charts where  chart came from some public repo and values/secrets came from flux config repo. But using that approach with custom in-house helm charts just complicates setup, especially if secrets are located in app's repo and not separately in config repo.
I guess official approach is "just use kustomize instead of helm" 😀 especially since HelmRelease does not really reconcile resources, only installed helm chart - same way helm works. But maybe it's worth at least thinking about?

The controller was designed specifically to only deal with charts and features as Helm knows them, and it would take a lot to convince us to head into another direction. This means that as long as Helm does not have an integration with SOPS, you will not find it in this controller either.
To solve your problem, there are several approaches available:

Make the kustomize-controller apply the secret data, use a reference to the created object in your HelmRelease's values.
Make use of Sealed-secrets instead of SOPS. As this uses a Custom Resource type, your chart can include this (encrypted) resource and Helm will apply it just fine.

Thanks for the reply!
I've just been struggling with migration from deploying via gitlab-ci to FluxCD  for some time and I still can't find a simple solution for keeping both helm chart and env's configs in app's repo and use them in FluxCD (combination of GitRepository+HelmRelease is great for git tags but doesn't really work for 'main' branch, plus still need Kustomization for secrets decryption).  Guess I was looking for an easy way out :) I will keep this issue open for while, just in case someone has better arguments/ideas.
Are there perhaps any other official or semi-official examples besides https://github.com/fluxcd/flux2-kustomize-helm-example so I can poke around?


GPT4 being used when not called in code

Hello, an error keeps happening when gpt4 api is being used when im setting the model to 3.5. here is the zip for the .py programs

Related to #125, and specifically, I think this is what is going on:
https://github.com/microsoft/autogen/issues/125#issuecomment-1754004740


Translation into Russian - first PR

This is the first PR translation of the cookbook into Russian.
Any native speaker from the HF team can do the review. An earlier review was done by @MKhalusova.

#92

@merveenoyan @stevhliu Hello, have a good day)
A few questions:

How do I get a review here?
Earlier documentation was built on a "demo server" and you could see if everything was correct (like here). Is there any way to do that now (To make sure I don't break anything with my PR)?

@stevhliu Hello Steven. Can you please tell me who can do a review?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Hi, thanks for the translation!

You can ping Merve or I for a review, but please be patient :)
The preview is automatically generated when we approve and run the workflow.
I would check the forums or Discord to see if there is another contributor who can read Russian to review your translation.

@stevhliu Greate Thanks. But i dont see Russian in docs PR preview:

And I dont undestand why?

@stevhliu I think I found the problem:

I have not added the language code to the documentation build files (YAML files).

@stevhliu How we can rebuild doc preview?

@stevhliu HuggingFaceDocBuilderDev still does not show RU in preview. I've made all the necessary edits, but it just doesn't seem to be rebuilding the documentation.

How do I initiate a rebuild of the preview? If of course I can do that.

It should work now :)

@stevhliu @merveenoyan Good day. Anyway, I haven't found anyone who could do a review for Russian. I made it myself - by myself. Can I run the documentation preview build again? I want to see how the edits I made will look like.

@stevhliu @merveenoyan
I can help with the review (my native language is Russian). For other languages I can help with formatting and other small things like that. But I have never done it before. But I think - I will learn, the main thing is to try).

Thanks for your translation, we appreciate your effort and work! However, to ensure the Cookbook's quality, we really should have a review from someone else who can also read Russian just to ensure there are no typos and it translates well. Hope you understand! 🤗


feat(chart): Added  SriLanka country map

SUMMARY
The Sri Lanka country map is not included in the chart country map plugin. This commit adds Sri Lanka map to the chart country map plugin.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS
cd superset/superset-frontend
npm run dev-server

ADDITIONAL INFORMATION

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

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

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

/testenv up

The test environment isn't working for some reason. Sigh.
The pre-commit hook is complaining about formatting. I made a comment above about a hunch I have there. If you could run that and commit the changes, it should allow me to spin up another ephemeral environment for testing. 🤞

The test environment isn't working for some reason. Sigh.
The pre-commit hook is complaining about formatting. I made a comment above about a hunch I have there. If you could run that and commit the changes, it should allow me to spin up another ephemeral environment for testing. 🤞

I have fixed the linting issues, the tests should pass now.

@rusackas I think the PR is good to merge

I'm trying to use this now, and I can't seem to get it to work. The ISO 3166-2 codes don't seem to map to the values I'm using, whether they're provinces (LK-1 through LK-9) or Districts (LK-11 through LK-92). Can you share your sample data that's rendering on the map?
Also,  I'm curious if we should change the country name from "Srilanka" to "Sri Lanka"


How to restore keybase device from backup? (Windows)

I accidentally formatted the disk of my last keybase device before adding a new device. I also forget where the paper key I stores. So I could not add new device now. I have the disk image backup of the formatted disk. My question is: how to restore the keybase device from the backup (except the full disk restore)?

Thanks, I found issue #9439 is exactly what I need. Just copy the old %userprofile%\AppData\Local\Keybase from the backup to the same path and the account private key is back.
But I got a new problem now. I need to rekey some files in the new setup, type keybase rekey paper got a serious of errors:
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING Unable to Home#pollOnce: API network error: doRetry failed, attempts: 1, timeout 5s, last err: context deadline exceeded [tags:HULT=71yl4smjxXrl]
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
It seems like a network issue, but other keybase command are OK. Any ideas?

Thanks, I found issue #9439 is exactly what I need. Just copy the old %userprofile%\AppData\Local\Keybase from the backup to the same path and the account private key is back.
But I got a new problem now. I need to rekey some files in the new setup, type keybase rekey paper got a series of errors:
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING Unable to Home#pollOnce: API network error: doRetry failed, attempts: 1, timeout 5s, last err: context deadline exceeded [tags:HULT=71yl4smjxXrl]
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
WARNING (CONN gregor f4be2028) Connection: error dialing transport: dial tcp <IP_ADDRESS>:443: i/o timeout
It seems like a network issue, but other keybase command are OK. Any ideas?

Same here: #19663
Reinstall Keybase helped.


OpMulExtended: workaround Intel GPU issue

OpMulExtended only works on Intel GPU with 32bit types.
Workaround it with a new option 'hack-mul-extended' which prevents
clspv from using OpMulExtended.
Also change the SRem implementation not to use OpMulExtended all the
time.

@alan-baker, I did not make this merge branch commit. Is it something done by kokoro?


[WIP] Downgrade cloud-init to 23.3 via extended version 23.4.1+really23.3

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

cloud-init 23.4 introduced a change which requires rfc3442 support. Currently, Mariner's dhclient does not fully support rfc3442. This leads to failure in establishing global routing. Downgrade cloud-init to remove rfc3442 requirement.
Note that version is 23.3 and not 23.3.3 mentioned in source url is intentional
Change Log

Downgrade to 23.3 using 23.4.1+really23.3
Add back overrideDatasourceDetection.patch
Remove ci-Pin-pytest-8.0.0.patch as pytest is pinned to pytest<=7.3.1
Remove Retain-exit-code-in-cloud-init-status-for-recoverabl.patch

Does this affect the toolchain?

NO
Associated issues

https://microsoft.visualstudio.com/OS/_workitems/edit/49661506

Test Methodology

Pipeline build id w/ tests (100% pass): https://dev.azure.com/mariner-org/mariner/_build/results?buildId=543324&view=results
Built locally within toolkit
Built locally within container w/ tests (100% pass:)
=========== 4861 passed, 8 skipped, 1 xfailed, 3 warnings in 55.74s ============"
Full build (in progress - different branch because pipeline doesn't like "+"): https://dev.azure.com/mariner-org/mariner/_build/results?buildId=543418&view=results

Our pipelines complain about "+" in the branch name. Opened PR using different branch but exact same commit: https://github.com/microsoft/azurelinux/pull/8705


Feedback upon path completion

Students find completion feedback lacking when a path is completed. Here a student comment. I think this is legit, and we should look into it.
**Could an email notification be sent out once the Path is accessible, so we don't have to keep refreshing the page (and given that the due date is tomorrow)? Thanks!
Martin's answer: You should complete path P1 before tomorrow, but note that it is not connected to a CS1101S assessment component. Path P1 is meant as an ungraded self-assessment.
We will look into improving the feedback upon path completion. Thanks for the suggestion.
-martin**

Implementation will have to be on the backend. Frontend runs on the client-side and is not suitable for this task.
/move to cadet