Adding unit tests for computed properties.

Noticed that the options property wasn't being updated, so I wrote some unit tests to figure out why.

@homu r+

:pushpin: Commit e644e41 has been approved by gmurphey

:hourglass: Testing commit e644e41 with merge 4a9b156...

:sunny: Test successful - status


Mouse Without Borders blocks on Task Manager window

Microsoft PowerToys version
0.70.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Mouse Without Borders
Steps to reproduce

One the secondary PC, right click task bar and select "Task Manager"
Task Manager opens up

✔ Expected Behavior
Mouse and keyboard to continue working as usual
❌ Actual Behavior

Now the keyboard and mouse no longer work.
I have to go to the second PC keyboard/ mouse to close Task Manager
Then the Mouse and Keyboard of primary PC are working on secondary PC via "Mouse Without Borders"

Other Software
None
PowerToysReport_2023-05-25-15-48-27.zip

Hi, that's because Task Manager is a system window that requires app to have higher permissions in order to change it. Are you running as admin in both machines? Have you tried installing the service so that it can manipulate elevated apps?
/needinfo

Excellent - thank you!!  Running as admin on BOTH did the trick
Regards
Greg

From: Jaime Bernardo @.>
Sent: 26 May 2023 11:14 AM
To: microsoft/PowerToys @.>
Cc: ghb67 @.>; Author @.>
Subject: Re: [microsoft/PowerToys] Mouse Without Borders Crash (Issue #26326)
Hi, that's because Task Manager is a system window that requires app to have higher permissions in order to change it. Are you running as admin in both machines? Have you tried installing the service so that it can manipulate elevated apps?
/needinfo
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/26326#issuecomment-1564544329, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BADJAQGLXDXH5BRVSNVJFUDXIDCGRANCNFSM6AAAAAAYPJ56FM.
You are receiving this because you authored the thread.Message ID: @.***>


Sessions do no longer auto-save on exit

@glepnir Sessions do no longer auto-save when closing out of neovim and a session already exists for the current directory. It would be nice to either have a setup option to re-enable this functionality natively or for you to expose the project_name function in the session module so that users could easily implement this functionality themselves. As of now, to find the correct session one would have to copy your code to get the session file name in order to check if it exists for the cwd and depending on that call "session_save" on closing out of neovim.

can provide a function to check the current file in session or not then do something.

can provide a function to check the current file in session or not then do something.

That would be perfect. Thank you!

when have time do this :)

when have time do this :)

Do you want me to implement this? I won't have time today, but could do it tomorrow and create a PR if you like.

I have an existing session and session_auto_save_on_exit set to true, but it doesn't auto save on quit. When I run :SessionLoad it opens the file I first run :SessionSave on and not the file I last quit.
My settings:

@Leo310 I have implemented the feature so that it will only auto-save a session if there is more than one file open. If that's not the case, the existing session file will not be overwritten.

@Leo310 whenever you reopen the directory, you will first have to load the session to enable auto-saving. If it still doesn't work for you, the issue has to lie somewhere else.  I'm using this feature on a daily basis and it works just fine for me.

Very weird:
Small Gif I made showing that I think I am doing everythink right

:SessionSave fileA
:SessionLoad -> opens fileA
navigating to a different fileB and quitting nvim (should auto-save now)
:SessionLoad -> should open fileB but still opens fileA

Any idea why It still doesn't work?

@Leo310 I found the issue. Apparently sessions are not saved when creating buffers through telescope. If you open files through NvimTree or using the standard :e ur_file.lua command, everything works as intended. This is something @glepnir will have to look into but it looks like telescope is the culprit here. Nice catch though :smile:. I usually use NvimTree to open buffers, so I would have probably never noticed this. I recorded a quick video that demonstrates the issue.
https://user-images.githubusercontent.com/55194243/210408880-4dceeb54-899f-4498-a309-b80af2837e53.mp4

Hmmm I think we have slightly different issues:

Also not working with Nvim-Tree
My :SessionSave does save the session but the auto_save is not overwriting it (as seen in my video)

Maybe reopen this issue?

@Leo310 I mean the auto-save feature does work for me as you can see in my video as long as I don't use telescope and the auto-save functionality is based on the existing session save method. You could try to verify that the custom auto-command callback does in fact get called, by simply commenting out the line pcall(vim.cmd, "NvimTreeClose") and opening NvimTree before saving the session and closing out of nvim. On reloading the session, you should see an empty NvimTree Buffer when the line is commented out, which should not be present if you put it back in. If the callback function actually gets called, the auto-save method does it's thing but simply fails to save your buffer list for some other reason (potentially due to telescope). Did you try to open files not using telescope? Does the issue persist?
In any case I would suggest you open a new issue for this, since auto-saving itself works for me.

@LeonHeidelbach Ok then I will propably open a new issue.
What I am trying to say is that the :SessionLoad command always restores the state of the last :SessionSave, but not the one of the last auto-save. And the auto-save callback is also not called. I even tried without Telescope (only opened files with Nvim-Tree), but it didnt work.

@Leo310 Oh, then your problem could be that you define the custom auto command within the plugin config function of your package manager. You should define the custom auto command in your main initialization lua. Give that a try and let me know if that fixes it.

@LeonHeidelbach Sadly, also doesnt work :(

@Leo310 Well, I really cannot say what's the issue then. You could try to execute the following commands and let me know what the outputs are:
:lua print(require('dashboard').session_auto_save_on_exit) --> This should print 'true' if your config has been applied correctly
:lua print(require('dashboard.session').should_auto_save()) --> This should print 'true' if the first command prints 'true' and you have loaded or saved the current session
:lua vim.api.nvim_exec_autocmds('User', { pattern = 'DBSessionSavePre', modeline = false }) --> This should close your NvimTree buffer if you have one open

Really Appreciate your help man, but everything works as expected 🤔
I checked the plugin code and for now I also couldnt find anything.

@Leo310 Sure thing. I mean I implemented this feature, so it might have been a mistake on my side, but I couldn't find anything wrong with the plugin code. Let me know if you find out what the issue was.

fixed it by commenting out these lines.

seems like maybe db.session_auto_save_on_exit wasnt set yet?

@Leo310 That is very strange. But you said that all of the commands I listed  printed 'true' right? That means that the variable should've also been set correctly if this returns 'true':
:lua print(require('dashboard').session_auto_save_on_exit)

Maybe it get's reset somewhere else?

@LeonHeidelbach yeah it returns true. But I wouldnt know where this should be reset. and there is no possibility that the  this variable is set after this plugin is initialized?

@Leo310 I mean it only ever gets assigned a value in the default config of the plugin here, so only changing it from outside, somewhere within your config could cause this.

that is the only place I set it. Maybe something with the Lazy.nvim package manager and its lazy loading?

so weird

Yeah, well that could be. I personally still use Packer, since it's been around for quite some time and it's mostly stable. I need my package manager to work reliably and lazy.nvim probably still has quite some issues since it's just been released.

I have no experience in building plugins or something like that so no idea what exactly could cause this in Lazy.nvim.
So then I think I will just use my hardcoded version for now and hope it is fixed one day haha

I have no idea how Lazy.nvim actually works internally, but it could be a caching issue. Maybe Lazy is pre-computing some plugin tables to make things faster. I remember reading in its documentation, that it also natively implements similar functionality to impatient.nvim. But if it's a caching issue your other config values should not work either. It's a strange issue... You could maybe try to print the value of session_auto_save_on_exit where you commented out the if statement to see if it actually is false or nil.

yeah very strange. and it's actually false where I commented out the if statement

@Leo310 Okay, how about you try and change the default value here to true. If that works then we know it has to be some issue with Lazy.nvim.

Yeah that works and is what I am doing atm.
lazy = false didnt do the trick
Would you open an issue on lazy for that or would you chill and wait?
I just have no idea where  the Problem could be

I would open an issue with Lazy.nvim for that since it's a pretty big problem. It's also still unclear what is causing this exactly. If you want to be sure and take the time you could try switching your package manager to Packer and see if you are still getting this config issue with your setup. If not, there is no doubt that it really is a Lazy issue.

Hi! I just saw this issue.
Is it expected that a user's config is set before sourcing dashboard's plugin files?

I just pushed an update that might fix this. @Leo310 would be great if you could confirm

Hey @folke! Nice to see a direct response to this random issue from you here, how did you even find this? :smile: Dashboard does not have a dedicated setup function, instead users are supposed to directly change values within the "db" settings table located here like so: require('dashboard').session_auto_save_on_exit = true. Dashboards init.lua sets default values to all settings on sourcing the plugin files, so the user should be able to change those values after the defaults have been set, which seemd to have worked for @Leo310. Checking the settings values during runtime like this :lua print(require('dashboard').session_auto_save_on_exit) showed the correct values. However, the settings seemed to have been reset to it's default value by the time we reach this if statement. It almost seemed as if the init.lua had been re-run which resets all values to their defaults.

The problem was that dashboard/plugin is ran before the user's config() method. I've now switched that around.

maybe can use event = 'VimEnter or something else to make plugin folder not load at startup.

@folke yeah it is working now 👌. Also wondering how you find this issue?

I'm searching Github every now and then to find any issues that might be caused by lazy ;)

real sick man. i was a bit to lazy to open a new issue, so love your effort :)

@folke maybe instead of changing when Lazy calls the config function you could introduce another setup field that can be used to set up plugins like dashboard. I am sure there are also other plugins that would require the same way of setup, so having this would make Lazy more compatible without the need of changing plugin code. Especially since Packer supports this way of setting up plugins it would make the transition for people easier without the need to tell other plugin devs to change their setup method.

I would like to keep it simple. config runs when the plugin was loaded, meaning that everything related to the plugin is loaded and available to be used in the config function. This is also how native packages with Pakadd would work.
@glepnir 's suggestion and what i commented is an easy fix and makes more sense.

I see that this issue is closed. Well I just hit this - no autosaving. I'm using Telescope like a billion other people.


Github integration - Refresh login token

Currently the login token gets invalidated after little time. It should get refreshed immediately when it happens, transparently for the user.

Found out that Github doesn't support refresh tokens. Therefore I just fixed this by increasing the expiration time of the cookie.


tcp: not calling watermark callbacks in delay close

Risk Level: low
Testing: new unit test
Docs Changes: n/a
Release Notes: n/a
Fixes #6872

We could arguably future-proof a little better by moving all the callbacks to delay_close_callbacks, and only  doing raiseEvent for disconnects on those.  I figured this was both simpler and safer so went with this for now.

Per https://github.com/envoyproxy/envoy/issues/6872#issuecomment-494522845 I agree with you that I'm not sure this is the actual problem. Should we hold on this until I can look at an actual core dump and debug a bit more?

I picked it up anyway, because even if it's not causing seg-faults I feel like after the connection has been delay-closed we shouldn't be using the callbacks for anything other than raising the actual close.  If there were something buffered in the HCM (like the h2 codec had its own control frame buffer) I don't think we'd want to be draining more at the L7 layer.  If you disagree I'm happy to close this out though.

If you disagree I'm happy to close this out though.

I could go either way on this and don't feel strongly about it. One one hand, maybe it makes sense for the connection owner to be notified of buffer drain even during delayed closing processing, on the other hand, I agree this is probably fine and simple. Although, I don't think going about high watermark should ever happen during delay close, right? Should we cover that case also?

We can close it out until we have a handle on that crash


Iotex_Son_Avatar

Token Information

Project official website: http://iotexsons.io/

Project official social: http://...

Verification of Token

[ ] verify of your token smart contract on iotexscan. io1yt3seeywrckl78f8zxvehs6mk6dv36cnvljex3
[ ] add your token address to your website and list here: https://iotexscout.io/address/io1yt3seeywrckl78f8zxvehs6mk6dv36cnvljex3#code

The logo image only support JPG/PNG format image currently.


LoginWith * Redirects do not work (Chrom Extension)

For some reason I can't make Facebook or Twitter Login to work on my chrome extension
        this.ddp = new Asteroid("localhost:3000");

        this.ddp.on('connected', =>
            console.log 'Connected'

            this.ddp.resumeLoginPromise.then( ->
                console.log ('ok')
            ).fail( =>
                console.log 'fail, now log in'

                this.ddp.subscribe("meteor.loginServiceConfiguration").ready.then =>
                    #this.ddp.loginWithTwitter()
                    this.ddp.loginWithFacebook()
            )
        )

This sends me to an endlessly loading new tab.
The following images are the callback urls for the facebook and twitter apps.

Any idea on why is this happening?
Thanks.

I'm getting this error from the server.

Hi Bernat,
Sorry for the late reply, I've had some busy days. :-)
I think I saw that error before, but I can't remember exactly when or where. How about a hangout to figure it out? (I'm finding google hangouts a much quicker way to solve issues, it takes me forever to type :-D )
Let me know
Cheers!

Fixed via hangout iirc. Reopen if necessary. :smiley:

@pscanf hello Paolo. I was wondering if we could have another hangout to dicuss this. I've had some time to look into it again.
Thanks in advance for your time!
Bernat.

@pscanf @bernatfortet Hello Guys.
I am trying to use this mixin to make google login in my extension that connects with my meteor app. I was able to get the login popup. After I select an account, the popup goes blank. Can you guys please help me?


DPC++FPGA DB example: dataset for FPGA target is not included

Summary
In the Reference Designs of DPC++FPGA, the DB design README.md file states that the FPGA binary should be started with: ./db.fpga --dbroot=../data/sf1 --test, however, the repository doesn't provide ../data/sf1.
Version
Tip of the repository and earlier releases.
Steps to reproduce
Build and execute the example from the repository using:
./db.fpga --dbroot=../data/sf1 --test
Observed behavior
On the other hand ../data/sf0.01 is provided. By documentation, this dataset should be used with the FPGA_emu binary.
I tried to use that dataset but it throws an error:
LineItem table size has 60175 rows when it should have 6001215
Orders table size has 15000 rows when it should have 1500000
Parts table size has 2000 rows when it should have 200000
Supplier table size has 100 rows when it should have 10000
PartSupplier table size has 8000 rows when it should have 800000
ERROR: could not validate the scale factor of the parsed database files

Expected behavior
Either provide dataset sf1 or allow sf0.01 to work with the FPGA target binary.

The files are intentionally not included: the files would be over 1GB in size and would really bloat the repo.
From the DB README:

In the data/ directory, you will find database files for a scale factor of 0.01. These files were generated manually and can be used to verify the queries in emulation. However, these files are too small to showcase the true performance of the FPGA hardware.
To generate larger database files to run on the hardware, you can use TPC's dbgen tool. Instructions for downloading, building and running the dbgen tool can be found on the TPC-H website. Note that this reference design currently only supports databases with scale factors of 0.01 or 1.

A alternative approach would be to write a table generator in C++ and include it in the design itself -- which I may already have ready 👀

@mdbtucker what do you think about my alternative approach? I don't think adding these files to the repo is acceptable; but writing a data generator in C++ is reasonable, but will require some effort.

I think the data generator is the ideal solution, assuming it is minimal effort.
If it's a lot of effort, I think the current solution of pointing the user at an easily available data generator is fine.  If that is the plan, then I think the README instructions for running the fpga code should have a note added to emphasize the fact that sf1 is NOT included in the repo and must be generated by the user as outlined here

In order to create the data set you need to get this tool: https://github.com/electrum/tpch-dbgen
Then you simply run “make” and the executable “dbgen” will be created.
With this tool you can create a new scale factor 1 data set by doing “./dbgen -s 1”.
You can then copy those into the subdirectory db/data/sf1/ and launch the ./db.fpga executable in it.
These instructions need to be added to the db sample README

Implemented in https://github.com/oneapi-src/oneAPI-samples/pull/1080


feat(params): Add customisable committer params

Customised Case:

Default Case:

Feature on behalf of https://github.com/dailydotdev/action-devcard/issues/367

@omBratteng, I've applied changes


[tests] fix yaml load warning

According https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation , this fix python warnings when running tests.
Python 3.8
PyYaml 5.4

From PyYaml 6, Loader argument is required.

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

jenkins test this please


Just some dummy files to get navigation going

Use the tabs now if you want, pulled them into a partial.  Cleared out the junk from the routes file.

Merge Me :sheep: :dash: Please

THANK YOU!!!! :heart: :heart: :heart: (default routes comment bugs me)
I'd personally like to see our resources scoped to just the actions we actually use (like resources :orders, only: [:index, :new, :create], or whatever it would be), but that could always come later.


Cannot take back handed in timesheets

I just inadvertently handed in my timesheet because I confused the "New Work Day" with the "Hand In" button. Now that my timesheet is handed in, I cannot do anything anymore. It should be possible to cancel the hand-in process and maybe there should also be a warning if you try to hand in your timesheet in the middle of the month.

This might be fixed already in the new version. Can we deploy the current dev to the production system @derari @chrisma ?

This looks good! :+1:
Are there tests for this feature yet?


WIP Vendorize

close #109
Blocked by much of issues with bowler.
https://github.com/facebookincubator/Bowler/issues/90
https://github.com/facebookincubator/Bowler/issues/91
https://github.com/facebookincubator/Bowler/issues/92
https://github.com/facebookincubator/Bowler/issues/93

Another "Ugh": Bowler doesn't support Python 3.5. Just for symbol annotations, I think. So, I'll fork it, thanks to MIT license

Ok, everything works. On the way:

Documentation
Bowler-py35 release. Maybe, fissix too, if my PR won't be accepted: https://github.com/jreese/fissix/pull/4

CI failed because of a bug in the old version of dephell. Let's merge it and release to fix it


Thumbnail slightly cut off on top/bottom

The Issue
If you compare the thumbs on Android vs the desktop app, you'll notice they are slightly cut off on the top and button.
System Configuration

Device:
Android version:

Anything Else

Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of the user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

Thumbnail aspect ratios have been adjusted for better display.


Manually adjustable config columns after auto resize

Is your feature request related to a problem? Please describe.
I want to be able to manually adjust the width of the config columns after they were auto sized.
Describe the solution you'd like
I want to be able to manually adjust the width of the config columns after they were auto sized.
Additional context
Add on to #1536

First waiting for feedack on #1536


don't do empty commits

I was walking through the data on gitlab and saw that:
https://gitlab.com/bdrc-data/persons/commit/e3b23ab7e993aa58384fc4866579d62942420aff
which is a commit by xmltold with no difference. I didn't know there was such a thing as an empty commit (and that's the assumption made by xmltold too). Turns out there is! We should make sure that commits by xlmtold are not empty, that will make our data much easier to navigate

GitHelpers has been modified so it doesn't commit when there are no changes. See https://github.com/buda-base/bdrc-libraries/commit/edcb32ea5b45220cf3f742dad77540bf1ed03685


WIP: Mock API Gateway response

Mock API Gateway
Motivation:
It would be convenient to mock API Gateway events (and perhaps a few other) during local development/testing.
Modifications:

definition of DataMapping protocol (rename?)
public initializers for APIGateway.Request
implementation of APIGatewayMapping
dependency of AWSLambdaRuntimeCore on AWSLambdaEvents// TODO: remove
add request/data mappings in LocalLambda.Server:

        let server = LocalLambda.Server(invocationEndpoint: invocationEndpoint, dataMappings: [
            "/api": APIGatewayMapping(),
            "/api2": APIGatewayV2Mapping(),
        ])

Questions/issues:

dependency of AWSLambdaRuntimeCore on AWSLambdaEvents - we could either expose configuration of the LocalLambda.Server or make a copy of the APIGateway structs
should all the "mappings" be handled in the same channel (and on the same port), alternatively we could have a different channel (and port) for each

Result:
HTTP requests to LocalLambda.Server will be mapped to APIGateway.Request.
Example using HTTPie:
$ http "http://localhost:7000/api/path?name=Joe"
HTTP/1.1 200 OK
X-Test: test
content-length: 9

hello Joe

the Lambda:
struct APIGatewayProxyLambda: EventLoopLambdaHandler {
    public typealias In = APIGateway.Request
    public typealias Out = APIGateway.Response

    public func handle(context: Lambda.Context, event: In) -> EventLoopFuture<Out> {
        let name = event.queryStringParameters?["name"] ?? "stranger"
        let response = Out(statusCode: .ok,
                           headers: ["X-Test": "test"],
                           body: "hello \(name)")
        return context.eventLoop.makeSucceededFuture(response)
    }
}

I missed https://github.com/swift-server/swift-aws-lambda-runtime/pull/83 which defines LocalLambdaInvocationProxy and implements InvokeProxy.

@pokryfka the work on this pr has stopped, since we chose to go a different route with configuring the LocalServer. Since the LocalServer is now automatically started if the ENV variable is set, we need to find a different way. One idea is to provide a bootstrap method like in swift-logging to configure the LocalServer before actual startup. Does this make sense?

@pokryfka should we pursue this further in the context of #83?

@tomerd @fabianfett please let me know your thoughts on this, a few comments/questions inline (above)

hi @parkera if you want to keep this around, could you please update this PR to be against the main branch so we can delete the old master branch


How to reproduce the refcoco results

I'm currently diving into your fascinating work, but I'm running into a bit of trouble while trying to replicate the results on the RefCOCO dataset using the provided model checkpoint. My tests are showing an<EMAIL_ADDRESS>of 0.75 on the RefCOCO val set.
Environment

mmcv 2.0.1
mmdet 3.3.0
mmengine 0.8.4
Other libraries installed as the requirements.txt.

Approach
I have used the mdetr-provided COCO format annotations for RefCOCO, which align with the mm-grounding DINO. The evaluation involves organizing annotations into a list and running a simple loop. During each iteration, I replace g_dino_caption and image_path with corresponding RefCOCO elements and conduct inference. I'm using an evaluator similar to mm-grounding DINO to save and compute the results.
I also tried both prompt templates mentioned in the paper, and the results remain the same.
'Please detect the {g_dino_caption} in this image.' and 'What is {g_dino_caption} in this image? Please output object location.'
Could you please confirm whether the provided checkpoint is the final version for RefCOCO? Additionally, any insights or suggestions on potential issues in my evaluation approach would be highly appreciated.
Thank you for your assistance and for providing the resources for replication!

Thanks for your attention! Yes, the checkpoint is related to paper's result. Have you set your box_threshold to a high value like 0.7? In our exp, we set it to 0.3 to make sure there is around one box per image. This value is refer to GroundingDINO's inference setting.
Moreover, have you get the right result through chat.py successfully? It should be same with the res we show in paper.
If it doesn't solve your problem, please feel free to leave a message here and we'll get back to you as soon as we see it!

Thank you for your quick response! I have set the threshold to 0, and in the refcoco validation set, only 118/10834 samples have scores below 0.3, which should not significantly impact the results. The previous results were based on Top-1 accuracy, with the complete results as follows:

Precision @ 1
Precision @ 5
Precision @ 10

0.752353701
0.972217094
0.985231678

I also attempted to include accuracy for all box_score > 0.3, and the result is 0.87.
Here is my evaluator, using 0.3 as the threshold. I call evaluator.process(outputs, data["image_id"]) to save the results at the end of each iteration and finally use compute_metrics to calculate the overall results.:
class RefEvalThr:
    def __init__(self,
                 ann_file: Optional[str] = None,
                 metric: str = 'bbox',
                 topk=(1, 5, 10),
                 iou_thrs: float = 0.5,
                 save_dir: Optional[str] = None,
                 **kwargs) -> None:
        # super().__init__(**kwargs)
        self.results = []
        self.metric = metric
        self.topk = topk
        self.iou_thrs = iou_thrs
        self.save_dir = save_dir

        self.coco = COCO(ann_file)

    def process(self, data_samples: Sequence, id) -> None:
        for data_sample in data_samples:
            result = dict()
            pred = data_sample.pred_instances
            result['img_id'] = id
            result['bboxes'] = pred.bboxes.cpu().numpy().copy()
            result['scores'] = pred.scores.cpu().numpy().copy()
            self.results.append(result)

    def compute_metrics(self, results: list):
        if self.save_dir is not None:
            np.save(self.save_dir, results)
        dataset2score = {k: 0.0 for k in self.topk}
        dataset2count = 0
        count_low_score = 0

        for result in results:
            img_id = result['img_id']
            ann_ids = self.coco.getAnnIds(imgIds=img_id)
            assert len(ann_ids) == 1
            img_info = self.coco.loadImgs(img_id)[0]
            target = self.coco.loadAnns(ann_ids[0])

            target_bbox = target[0]['bbox']
            converted_bbox = [
                target_bbox[0],
                target_bbox[1],
                target_bbox[2] + target_bbox[0],
                target_bbox[3] + target_bbox[1],
            ]
            iou = mmdet.evaluation.bbox_overlaps(result['bboxes'],
                                np.array(converted_bbox).reshape(-1, 4)).reshape(-1)
            # iou = torchvision.ops.box_iou(torch.as_tensor(result['bboxes']),
            #                            torch.as_tensor(converted_bbox).reshape(-1, 4)).numpy().reshape(-1)
            # giou = torchvision.ops.generalized_box_iou(torch.as_tensor(result['bboxes']),
            #                            torch.as_tensor(converted_bbox).reshape(-1, 4)).numpy().reshape(-1)
            scores = result['scores']
            filtered_iou = iou[scores >= 0.3]
            if len(filtered_iou) > 0 and max(filtered_iou) >= self.iou_thrs:
                dataset2score[1] += 1.0
            dataset2count += 1.0
            # if result['scores'][0] < 0.3:
            #     count_low_score += 1
        print(f"count positive: {dataset2count}")
        # print(f"count low_score: {count_low_score}")

        for k in self.topk:
            try:
                dataset2score[k] /= dataset2count
            except Exception as e:
                print(e)
        print(dataset2score)

Here are some examples of wrong predictions:

I wonder if my evaluation approach is flawed, or if there might be inconsistencies in the mmdet version, or if there are issues with hyperparameters, randomness, or other factors. Your insights on these matters would be greatly appreciated. Thank you once again for your swift response!

Hi, here is a sample:
caption: chair that the boy sit on
prompt: Please detect the chair that the boy sit on in this image.
image: COCO_train2014_000000083508.jpg
res:

pred_bboxes_filt: [[ 39.1294,  49.1208, 217.3541, 402.2947]]
pred_scores_filt: [0.7888]
before filtered by threshold(0.3), top 20 pred_bboxes are:
[[ 39.1294,  49.1208, 256.4835, 451.4155],
[386.3545, 222.0057, 640.0000, 452.6325],
[502.6339, 106.8788, 639.1548, 243.7754],
[ 33.5774,  47.3136, 640.0000, 453.1940],
[179.8883,  94.7124, 412.8659, 451.1689],
[378.2805, 106.8258, 640.0000, 452.1395],
[250.4331,  96.8331, 381.4485, 274.8452],
[372.9280, 106.1171, 640.0000, 453.8893],
[374.7803, 106.9710, 640.0000, 452.8337],
[249.4045,  95.0679, 640.0000, 260.4560],
[ 36.7581,  49.5404, 417.0672, 454.6150],
[367.1216, 105.4114, 640.0000, 456.2697],
[250.8099,  91.4537, 640.0000, 262.8784],
[165.9880,  97.8375, 640.0000, 454.9725],
[170.9866,  97.8001, 640.0000, 456.0018],
[458.9583, 256.8557, 638.2808, 327.3876],
[248.9486,  95.4559, 640.0000, 259.1696],
[ 36.0579,  45.2150, 640.0000, 304.8564],
[320.7637, 104.3464, 640.0000, 452.3668],
[487.3306,   0.7818, 640.0000, 251.5550]]

top 20 pred_scores are:
[0.7888, 0.0627, 0.0613, 0.0261, 0.0121, 0.0079, 0.0065, 0.0051, 0.0043,
0.0039, 0.0036, 0.0034, 0.0031, 0.0030, 0.0030, 0.0028, 0.0028, 0.0021,
0.0021, 0.0020]

You can use this sample to check whether there is a problem with the mmdet version or some parameters. Or you can provide the image name of your test examples, we will test on them~

top 20 pred_bboxes：
[[39.1192, 49.1206, 256.4813, 451.4343],
[386.4013, 222.0148, 640.0, 452.6575],
[502.6575, 106.8746, 639.1531, 243.784],
[8.2782, 48.1111, 640.0, 454.3926],
[180.2233, 94.68, 412.8537, 451.0923],
[379.0567, 106.5491, 640.0, 453.942],
[250.7527, 96.8303, 382.9833, 273.9555],
[386.8084, 106.4516, 640.0, 453.771],
[378.004, 105.6164, 640.0, 457.0],
[36.7511, 49.5608, 416.94, 454.7256],
[36.405, 47.306, 640.0, 454.4246],
[207.8284, 232.7865, 416.9753, 455.4554],
[171.7513, 95.6881, 640.0, 455.5508],
[250.8095, 91.4642, 640.0, 262.4736],
[383.1513, 106.5066, 640.0, 452.8468],
[250.5402, 95.1247, 640.0, 255.6561],
[166.0703, 97.7855, 640.0, 454.9286],
[379.0126, 107.7026, 640.0, 453.3336],
[170.4569, 98.0551, 640.0, 456.0147],
[387.941, 105.8021, 639.6733, 313.0541]]

top 20 pred_scores：
[0.7895, 0.0625, 0.0622, 0.0231, 0.0116, 0.0069, 0.0058, 0.0051, 0.0038, 0.0037, 0.0035, 0.0034, 0.0033, 0.0033, 0.0031, 0.0031, 0.003, 0.0029, 0.0027, 0.0025]

The results are similar, possibly due to the randomness by the environment. Thank you for your reply and excellent work!


Transition between slides

I'm grateful to the author for this marvelous Bootstrap plugin for Angular. I implemented carousel slides in my project but one thing: there is no transition between images. Is there any other variables should be defined? Thanks in advance.

@yaoshuqi as soon as ng2 animate will became a bit more stable
animation option will be added to a lot of things here

Is there any news about this ability?

@valorkin there is no transition between images, any other way to transition possible?


Embedded font displayed as random characters

I made a document using Scribus 1.5.0svn with only the text "This should be readable" in it using the embedded Arial Regular font. While it displays fine in Adobe Acrobat Reader DC, PDF.js displays random characters when using Windows 10 x64 and Firefox 43.0.4.
Initially I thought my issues with the Scribus PDF exports were fixed by #6068. That issue is still fixed, so this is not a regression, but something must have changed in the meantime (either in Scribus or in my system fonts) because, as far as I can remember, this new PDF is generated in the exact same way as the one in that issue, yet this one is broken and the other one is not.
Here is a reduced test case for inclusion in the PR that fixes this issue:
example.pdf
Using Brendan's PDF browser I see that this font also has names of the form uniXXX in the differences array, which might indicate a similarity with #6090. @Snuffleupagus Do you perhaps have an idea if the fix for this issue is similar to the one in #6090, but for a different code path?

[...] but something must have changed in the meantime (either in Scribus or in my system fonts)

The embedded TrueType fonts are not entirely identical, since the 'post' table1 format differs:
in #6068 it's 2, and here it's 3. Note how properties.glyphNames is undefined in the latter case2.
Also, it seems that Scribus PDF includes a lot of unnecessary junk in the files it generates, so I figured it couldn't hurt to "reduce" the PDF file even more :)

[1] https://developer.apple.com/fonts/TrueType-Reference-Manual/RM06/Chap6post.html
[2] https://github.com/mozilla/pdf.js/blob/master/src/core/fonts.js#L3748-L3815


Illustration: Needs content written regarding usage

Which page is this about?
Illustration: https://cfpb.github.io/design-system/foundation/illustration
Describe your issue

Write content regarding when we should we use illustrations in our work. When doesn't a product need an illustration? When should we use photography instead of illustration? Etc.
Any info we can add to better help folks understand how to design within our illustration style and incorporate them into our publications?
Link to components that require illustration so you can access those specs easily.

This issue was created when we migrated the Design Manual to our current Design System site. During that time, we reviewed all pages to identify potential improvements. These are nice ideas, but they have not been reported by our team's designers or developers as being critical to implement. Because of this, we've decided to close this issue in favor of prioritizing issues that have been reported by our designers and developers as being blockers to their work. Thanks!


SymbolInformation support

This PR adds support for SymbolInformation symbols and simplifies the context update logic.
SymbolInformation symbols are parsed using the existing logic, then augmented with hierarchical information by examining their positional relationships (e.g., if symbol B is within the bounds of symbol A, B is a child of A).
This has been manually tested with the svelte, html, cssls, tsserver, and sumneko language servers. Svelte, html, and cssls all use SymbolInformation, while tsserver and sumneko use DocumentSymbol. So far I haven't seen any issues.

This would resolve #39

@SmiteshP can this be merged in?

@SmiteshP can this be merged in?

Can confirm works well with powershell_es also!
Very happy to see this working
Currently running Jasons branch

Hey guys! Sorry I haven't been able to look at this for so long... I am traveling and don't have my laptop. Will review this soon.

Closing in favor of #55


Document that table styles will be inherited by nested tables

Using v4.0.0 alpha3
The problem appears, when inside any <th> in <thead class="thead-inverse"> is another table inserted.
Therefore thead-inverse th rule is also applied on the inner table. Wouldn't it be better to have more specific thead-inverse > tr > th rule? Or am I missing something?
http://codepen.io/simPod/pen/wWRgzN

Hi @simPod!
You appear to have posted a live example (http://codepen.io/simPod/pen/wWRgzN.html), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 21, column 26 thru line 22, column 24: th start tag in table body.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

this bot is nice. Fixed

So, we need to document this better, but this is working as intended. Per http://v4-alpha.getbootstrap.com/migration/#tables (emphasis added):

Nearly all instances of the > selector have been removed, meaning nested tables will now automatically inherit styles from their parents. This greatly simplifies our selectors and potential customizations.

Very well then. So the proper usage is to put thead-default on the inner table, right?

How are we supposed to reset an inner table styles? The opt-in approach is great and I'd love to opt-in to sub tables instead of them being the same as the parent. This kills any datepicker inside a table.


Issue #786 Integration: Install operator from marketplace user story

retest this please.

retest this please

Right now the CentosCI is being killed by the timout of 90m in minishift-ci-jobs here. Should we increase it to ~120? We might have to decide on a subset of tests for PR checks... 2h is quite a bit of time. WDYT?

Right now the CentosCI is being killed by the timeout of 90m in minishift-ci-jobs here. Should we increase it to ~120?

@jsliacan Sure, go ahead with that.

retest this please


Include Orlando DevOps in Community Groups

https://orlandodevs.com/about/ does not include Orlando DevOps. Would be cooler if it did.

User Story
As a member of the Orlando DevOps group (an official subset of Orlando Devs), I would like representation on the Orlando Devs website so others can discover the DevOps group.

Related to https://github.com/snohio/odevs/issues/4

Related to https://github.com/snohio/odevs/issues/7


Sounds are not working

Requirements:
Please go through this checklist before opening a new issue

[x] Review the documentation
[x] Search for existing issues in: viromedia/viro & ViroCommunity/viro
[x] Use the latest ViroReact release

Environment
Please provide the following information about your environment:

Macbook Pro
Device OS & Version: iPhone11 iOS 15.3
ViroReact: 2.22.0
React Native: 0.65.1

Description

[ ] ViroSpatialSound crashes with invariant violation: Unrecognized selector sent to instance
[ ] ViroSoundField crashes with invariant violation: Unrecognized selector sent to instance

Reproducible Demo

⚠ NOTE: I've installed viro react from a local download of the viro react to make changes to the viro react source code. All of these issues have been replicated with<EMAIL_ADDRESS>
ViroSpatialSound
https://github.com/NS-BOBBY-C/starter-kit/blob/4d93e7dd3ae05d089573f07f20964e2258a1933d/screens/viro_tests/ViroSpatialSound.tsx#L1-L48
ViroSoundField
https://github.com/NS-BOBBY-C/starter-kit/blob/4d93e7dd3ae05d089573f07f20964e2258a1933d/screens/viro_tests/ViroSoundField.tsx#L1-L48

Are you doing your tests with the latest version from NPM or GitHub?

Both. I was originally using the master branch from GitHub, but then switched back to NPM to confirm it wasn't an issue with my local viroreact repo

Just confirmed @doranteseduardo that this was still an issue. My process:

Remove node_modules from my fork of starter-kit
Remove ios/podfile.lock, yarn.lock, package-lock.json
Update package.json to use "^2.22.0" instead of "../viro"
yarn install
Create demo for starter-kit https://github.com/NS-BOBBY-C/starter-kit/blob/101d6bf7c9995c16fb3c6ba7bc1b3c9d5f05d447/screens/github_issues/Issue74.tsx
See that it isn't working (screen recording)

https://user-images.githubusercontent.com/29097404/150527588-c81a6f70-18dd-489b-969e-28fed2f2c07e.MP4

Just confirmed this as an issue on an android phone as well. Google Pixel XL, android 10

@mondriy you can use ViroSound, which works as expected.

I tried ViroSound, but in my project I need in this feature because volume of sound depends of space.
Do you know how to solve this problem? May be you planed to fix this in future upd?
Thanks.


Multiple users report roundabout quick-setup not working

I've asked for more info about what specifically does "not working" mean.

If they ignore signs - increase simulation accuracy to high+
I feel like adding that options only increased number of reports. I'll recheck conditions, but currently anything below "high" makes priority signs useless and will be even worse when I push fix for issue with segment length.
For roundabouts.. hard to say what is wrong since we didn't really change anything. I'll try to test that later today

Also. Seeing this. Let me know if I can help.

@aagavin Yup, please give a detailed explanation of what "not working" means :)

For me the roundabout setting don't apply at all. Here is a video https://youtu.be/84nm0kF__7s

Wait.. didn't you actually show it works, at the end of the clip?😂 You need to hover the segment, not the node.

I can show it again, none of the roundabout rules are being applied.

@aagavin we'd got another similar reports but it turned you that user did not enable any roundabout-specific settings in the mod options screen. Could you check if all settings you want to apply are enabled after loading the savegame and if so, which of them does not work for you?
I've tested that feature yesterday and didn't notice any issues, everything seem to be working as advertised, but... if you disable all of them, it will do literally nothing after click.

@krzychu124 ok can confirm! That was the issue. This is not a new install, so maybe this got wiped somehow.
Thanks!

Yes, one of hotfixes to the Stable version accidentally introduced a bug which prevented saving of the options so they were reset after loading that savegame. We've fixed it quickly but... who saved game using that faulty version has lost mod options configuration by overwriting everything with default settings 😕
Thank you for confirmation that was the problem. Have a nice day, I'm closing this issue 😉

Will be fixed by PR #1455 which ensures pre-requisite features are enabled for any roundabout / priority road setup options.


Restore code returning calendar-specific subclasses of cftime.datetime

This restores the old behavior for cftime.num2date, cftime.datetime.__add__, and cftime.datetime.__sub__, in that they return calendar-specific sub-classes of cftime.datetime instead of calendar-aware cftime.datetime objects.
xref: https://github.com/Unidata/cftime/pull/227#issuecomment-771153314

Oh snap.  I can merge this and release 1.4.1 tomorrow.  Is this a bug in cftime, or is it just that xarray is relying on the sub-class type to infer the calendar?

Thanks @jswhit!  Really sorry I didn't catch this yesterday.

Is this a bug in cftime, or is it just that xarray is relying on the sub-class type to infer the calendar?

It's exactly the latter (i.e. purely xarray relying on the sub-class type to infer the calendar).


Name and what it does is wrong

Called 2x and says it does 2x upsampling. But it's not.. It's 4x... It doubles both the horizontal and the vertical. giving you 4 times the original res.
If you feed it a 500x500 image (250,000 pixels)
and set it to 2x
then you will get a 1000x1000 (1,000,000 pixels) image in return.
4 times the resolution of the original image. Should be called waifu4x really

Highly doubt we're doing this after 4,000 stars.

Probably not but it's still wrong. The 2x button on http://waifu2x.udp.jp/ should at least say 4x

That is copied from 2xSAI, scale2x and hq2x.
2x means scaling factor is 2.

I agree with Nagadomi. By the way, nobody didn't say that is 2x for all the pixels, it is 2x for one side and 2x for other side. Scale is 2x.

So making an image 4 times bigger is called scaling 2x because it's the 2x of the vertical and the horizontal? Even though doubling both creates an image 4x bigger? scaling each twice makes it 4x the original scale. four 500x500 images can fit in one 1000x1000 frame.

Yes, it's because 2x of the vertical and 2x of the horizontal (or in other words because scale factor is 2x for both of image sides). Yes, it makes image 4x bigger, I think everebody understand it and there's no need to change title, because waifu4x title make people think about 16 times bigger.


add installation instructions to docs

add installation instruction to docs (pypi, anaconda cloud, and source) as well as for developers

Done now.


OK

OK
https://github.com/xgqfrms/gitHub-secrets-all-in-one/actions/runs/396607600

uses + with / env
https://github.com/xgqfrms/gitHub-secrets-all-in-one/blob/main/.github/workflows/main.yml
https://github.com/xgqfrms/hello-github-actions

https://github.com/actions/virtual-environments/issues/1816
bug
https://github.com/xgqfrms/gitHub-secrets-all-in-one/runs/1487829653?check_suite_focus=true


Update for Pandoc 2.x

Hi @wei2912 , I saw your question on #hakyll after you had already left, and this seemed like the easiest way to respond.  Your errors are caused by upgrading from Pandoc 1.x to Pandoc 2.x.  On the following line you want to change S.insert to enableExtension:
https://github.com/wei2912/blog_src/blob/03b5190a9e1c729820e51017641080cf622a3d55/src/site.hs#L58
I think that will be sufficient but I am not 100% sure.  Feel free to ping me if not.  I recently upgraded my own site similarly so we should be able to figure it out.

This has been resolved, thanks!


Resolve grammatical error in Mouse Highlighter description

A simple PR to fix a grammatical error in one of the UI strings for mouse utilities.

Please create an issue for this so we can properly track it. This is an item we do require for a PR like that as it does touch a resource file in the app. The issues help us keep track of what is in a release

This PR resolves #23822.
My apologies for submitting out of order.


editor.actions.removeBrackets doesn't play well with Undo stack

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.77.0 Insiders
OS Version: Windows 11

Steps to Reproduce:

With the cursor on a bracket, press Ctrl+Alt+Backspace to remove the bracket and its matching bracket.
Ctrl+Z to undo the change.

No change, it takes two Ctrl+Z to actually undo and replace the brackets.
On earlier Insiders builds, it took even more cycles before the brackets came back

Doesn't repro with an empty profile.
It seems to be related to Auto Save. I require two Ctrl+Z if I have "files.autoSave": "afterDelay".
I'm testing with the gulpfile.js in the vscode-docs repo.


Type of INBOX_ID column in inbox table is too narrow

JdbcInboxStorage fails to insert a new message to inbox due to a too narrow type of INBOX_ID column:
com.mysql.cj.jdbc.exceptions.MysqlDataTruncation: Data truncation: Data too long for column 'INBOX_ID' at row 1.

The possible workaround can be specifying of a custom TypeMapping that effectively substitutes all VARCHAR(255) column types to VARCHAR(512):
TypeMapping typeMapping = TypeMappingBuilder.basicBuilder()
        .add(Type.STRING_255, "VARCHAR(512)")
        .build();
JdbcStorageFactory factory = JdbcStorageFactory.newBuilder()
        .setTypeMapping(typeMapping)
        .setDataSource(dataSource)
        .build();

Addressed in 1.x via #168.


How to give  ws.receive() a  timeout value  and default value?

I am not sure why you would want to do this but I do believe gevent provides a workaround for situations like this.
For example you could do the following to achieve your timeout of 1 second:
`
msg = None  # be aware that None is also returned when the websocket connection has closed
give the code within the with statement 1 second to run before it is forced to stop
with Timeout(1, False):
msg = ws.receive()
check if we timed out without receiving a value
if msg is None:
msg = "bob"
continue with your code
`
If you could explain why you want to do this I might be able to offer some better solutions for what you want to do.

our web client  send data not a regular time, but our web server must send data every 1 second.  so, if I receive it,  I cant send data every 1 second ,  if I  do not receive it , I cant catch  the msg.
so now , I used 2 threading to solve this，one receive msg  one send data every 1 second.  bug it is not pythonic.


Vulnerability Roundup 12

Here are all the vulnerabilities from https://lwn.net/Vulnerabilities
since our last roundup
cc: @FRidh @fpletz @vcunat @phanimahesh.
Note: The list of people CC'd on this issue participated in the last
roundup. If you participate on this roundup, I'll cc you on the next
one. If you don't participate in the next one, you won't be CC'd on
the one after that. If you would like to be CC'd on the next roundup.
If you would like to be CC'd on the next roundup, add a comment to the
most recent vulnerability roundup. If you would like to be CC'd on
all roundups, leave a comment and tell @grahamc so.
Permanent CC's: @joepie91, @NixOS/security-notifications
(if you no longer want to be CC'd, ask to be removed from this list)
Notes on the list

The reports have been roughly grouped by the package name. This
isn't perfect, but is intended to help identify if a whole group
of reports is resolved already.
Some issues will be duplicated, because it affects multiple packages.
For example, there are sometimes problems that impact thunderbird,
and firefox. LWN might report in one vulnerability "thunderbird
firefox". These names have been split to make sure both packages get
addressed.
By each issue is a link to code search for the package name, and
a Github search by filename. These are to help, but may not return
results when we do in fact package the software. If a search
doesn't turn up, please try altering the search criteria or
looking in nixpkgs manually before asserting we don't have it.
This issue is created by https://github.com/NixOS/security

Instructions:

Triage a report: If we don't have the software or our version isn't
vulnerable, tick the box or add a comment with the report number,
stating it isn't vulnerable.
Fix the issue: If we do have the software and it is vulnerable,
either leave a comment on this issue saying so, even open a pull
request with the fix. If you open a PR, make sure to tag this
issue so we can coordinate.
When an entire section is completed, move the section to the
"Triaged and Resolved Issues" details block below.

Upon Completion ...

[ ] Run the issue through reformat one last time
[ ] Review commits since last roundup for backport candidates
[ ] Update https://github.com/NixOS/nixpkgs/issues/13515 with a
summary.
[ ] Update the database at https://github.com/NixOS/security

Without further ado...
Assorted (27 issues)

[ ] #708243 (search, files)  GraphicsMagick: non-null null pointer
[x] #639393 (search, files)  arj: multiple vulnerabilities
[x] #708152 (search, files)  busybox: two vulnerabilities
[x] #707997 (search, files)  calamares: encryption bypass
[ ] #708137 (search, files)  chromium: multiple vulnerabilities
[x] #598856 (search, files)  cifs-utils: code execution
[ ] #568668 (search, files)  davfs2: privilege escalation
[x] #665921 (search, files)  dpkg: code execution
[x] #707838 (search, files)  firefox: code execution
[x] #708239 (search, files)  gstreamer1-plugins-good: buffer overflow
[ ] #707696 (search, files)  hdf5: multiple vulnerabilities
[ ] #707705 (search, files)  jenkins-remoting: code execution
[ ] #707363 (search, files)  kvm: denial of service
[ ] #708138 (search, files)  libdwarf: multiple vulnerabilities
[ ] #664646 (search, files)  libsndfile: buffer overflow
[ ] #708000 (search, files)  libctnative: SSL improvements
[x] #669403 (search, files)  nghttp2: code execution
[ ] #708140 (search, files)  openafs: information leak
[ ] #707698 (search, files)  p7zip: denial of service
[x] #708154 (search, files)  patch: denial of service
[ ] #708148 (search, files)  phpMyAdmin: multiple vulnerabilities
[ ] #668130 (search, files)  pygments: shell injection
[ ] #707700 (search, files)  teeworlds: code execution
[x] #624610 (search, files)  util-linux: command injection
[ ] #707703 (search, files)  vagrant: nfs export insertion
[ ] #708242 (search, files)  virtualbox: code execution
[ ] #708149 (search, files)  xen: multiple vulnerabilities

firefox (3 issues)

[x] #708241 (search, files)  mozilla: file overwrites
[x] #707838 (search, files)  firefox: code execution
[x] #707854 (search, files)  firefox: same-origin bypass

imagemagick (2 issues)

[x] #708243 (search, files)  GraphicsMagick: non-null null pointer
[x] #707857 (search, files)  imagemagick: code execution

kernel (4 issues)

[ ] #708246 (search, files)  kernel: code execution
[x] #708245 (search, files)  kernel: denial of service
[x] #708240 (search, files)  kernel: denial of service
[x] #707859 (search, files)  kernel: privilege escalation

thunderbird (2 issues)

[x] #708241 (search, files)  mozilla: file overwrites
[x] #707838 (search, files)  firefox: code execution

ImageMagick fixed in my rollup.

We should just delete Calamares: "LUKS accepts every WRONG password after install".

gstreamer needs updating / backporting to 1.10.2 and is ABI-safe according to https://abi-laboratory.pro/tracker/, see also https://github.com/NixOS/nixpkgs/commit/7a6185d9a1eb18fa09892fdef01a5193968cf704 for info on how.

I have an incoming patch for gstreamer

I do not think there are patched kernels for the remaining issue. Only the 4.9 RCs seem to have the commit so far.

The util-linux problem was reported in 2014 and our version is from 2016 (2.28.1 both on master and 16.09), so I don't think it's vulnerable anymore.

Our unarj is meta.broken with notes about vulnerability, so I'm ticking that as well.

libsndfile: the LWN link says versions <= 1.0.25 suffer from that vulnerability but we have 1.0.27 both on 16.09 and master.

libdwarf: fixed up in master and 16.09, except that darwin.dtrace probably contains an embedded copy in it's code which I don't feel like digging in.

Wow, great work, @vcunat. Thank you!

Backported jenkins from 9575eeae108ca23f7af49f0a1566706feef83cb4 in 3573b6c.

It isn't clear to me if virtualbox needs updating ...

hdf5 backported from 16eb67ac9ca52d56ff13fb3cda2e4644df9f387b in b60ae78

@NeQuissimus can you clarify? all the Kernel tasks are still marked not done. Do they all need to wait on releases from kernel.org?

:raised_hands: they're all completed this week, except for the kernel tasks (see previous comment.) I'm letting my patch branch build for a few hours before merging in to master / release. Some bigger builds in there I'm not 100% sure about.

How was arj solved? I was starting to fix the broken build?

Re: the final kernel issue, grsec users are unaffected :)

Oh, sorry, the first one, d'oh

OK, that is weird, I definitely checked off three of them yesterday... Anyways... According to SuSE we are good on that one as well, everything > 4.8.8 (or equivalent 4.4.x, etc.) is safe.

Thank you, @NeQuissimus! I'll merge these patches soon and close this out.

Wow. The roundup is done before I even noticed! That's awesome.
Too many github notifications to plough through. :(  I'll probably have to setup a mobile notification to not miss roundups.
To avoid repeated metoo comments on the roundups I miss, @grahamc, can you please add me to permanent ccs?

:D Done! Thank you!


BUG: CRDs in 0.0.8 with Dapr 1.13.2 not configuring properly

Expected Behavior
Dapr CRDs are granted access to various resources within OpenShift such as ClusterRole admin and all users of OpenShift.
Current Behavior
After deploying 1.13.2 via Dapr Operator 0.0.8, the Dapr CRDs no longer grant anyone but administrators access to the CRDs.
Possible Solution
Unknown
Steps to Reproduce

Uninstall any previous version of Dapr Operator (including cleaning up all CRDs and CRs)
Install Dapr Operator 0.0.8
Create a new DaprInstance with the following configuration (see below)
Monitors the CRDs until they are created
Attempt to access the CRDs (read and write) via namespace users and ServiceAccounts
Both types of accounts will receive access denied

# DaprInstance
apiVersion: operator.dapr.io/v1alpha1
kind: DaprInstance
metadata:
  name: dapr-instance
  namespace: openshift-operators
spec:
  values:
    dapr_operator:
      livenessProbe:
        initialDelaySeconds: 10
      readinessProbe:
        initialDelaySeconds: 10
    dapr_placement:
      cluster:
        forceInMemoryLog: true
    global:
      imagePullSecrets: dapr-pull-secret
      registry: internal-repo/daprio
  chart:
    version: 1.13.2

Environment
OpenShift: RedHad OpenShift Container Platform 4.12
Dapr Operator: 0.0.8 with 1.13.2 Dapr components
Workaround

Cluster admins have created temporary roles for ServiceAccounts to be able to access the dapr components.
Cluster admins have also manually given access to namespace administrators for the various CRDs

As a side note, this may be due to #135. I suspect with the dapr-control-plane crashing that it doesn't get a chance to complete it's work but I am not sure that matters as I would expect the dapr-operator to be creating the CRDs and publishing. Thoughts?

is the same use be able to get other CRDs ? The operator/olm does not set up any permissions for the CRDs except for itself

Doesn't appear that it's related to #135. Now that we have stablized the dapr-control-plane, the dapr-operator doesn't seem to have configured the permissions. I am not sure if this is related to this operator or not.

The operator does not set any permission, except providing a ClusterRole and ClusterRoleBinding for being able to read such CRDs and CRs.


Running within a Podman unprivileged pod

I'm trying to test maeve-csms under Podman. While I was able to start all containers, I still fail to authenticate with occp-1.6 and I'm wondering if I do not misunderstood the login process.
Under podman/pod

containers share a unique network addr (hostname=cscms-host=<IP_ADDRESS>)
containers share a unique config volume
this imposes minor change on config params/values

Provisioning a charger-id with manager/api works
I may provision and verify that charging station was created
curl http://csms-host:9410/api/v0/cs/Tux-Basic -H 'content-type: application/json' -d '{"securityProfile":0, "base64SHA256Password":"c25vb3B5Cg=="}}'
curl http://csms-host:9410/api/v0/cs/Tux-Basic/auth
 > {"base64SHA256Password":"c25vb3B5Cg==","securityProfile":0}

Then I try to log with ocpp-rpc client
I already use ocpp-rpc client with other server, also I known the protocol works. In fact checking here after wireshark image, we may see that the request/authen is clean. Even if the password encoding is probably wrong.
const { RPCClient } = require('ocpp-rpc');
const cli = new RPCClient({
    endpoint: 'ws://csms-host:9310/ws', // the OCPP endpoint URL
    identity: 'Tux-Basic',
    password: 'c25vb3B5Cg==',            // the OCPP identity
    protocols: ['ocpp1.6'],          // client understands ocpp1.6 subprotocol
    strictMode: true,                // enable strict validation of requests & responses
});

Question:

how should I encode my password. When I look to wireshark the encoding does not seems to match ?
would someone have a working sample with a charing-station provisioning and a wireshark dump to check what should be  sent

Here after my logs & startup script
Gateway log
2023/10/12 18:12:54 ERROR GET /ws/{id} duration=445.212µs http.scheme=ws http.method=GET http.url=/ws/Tux-Basic http.route=/ws/{id} csId=Tux-Basic ocpp.security_profile=0 auth.failure_reason="invalid password" http.status_code=401

Manager log
time=2023-10-12T18:12:16.140Z level=INFO msg="GET /api/v0/cs/Tux-B/auth" remote_addr=<IP_ADDRESS>:55506 status=404 bytes=23 duration=16.174µs
time=2023-10-12T18:12:54.321Z level=INFO msg="GET /api/v0/cs/Tux-Basic/auth" remote_addr=<IP_ADDRESS>:47434 status=200 bytes=60 duration=15.514µs

My Podman startup script
#!/bin/bash

# Script to start maeve-cms inside a podman 'pod'
# author: Fulup Ar Foll (iot.bzh)
# Licence: Apache-v2

if test -z "$ROOTDIR"; then
    ROOTDIR=`dirname $0`
fi
echo "Container rootdir=$ROOTDIR"

if test -z "$PODIP"; then
    PODIP=<IP_ADDRESS>
fi
echo "Container POD IP=$PODIP"

# restart from stratch
podman pod rm -f csms-pod

# rebuild local-image
for DOCKERFILE in  $ROOTDIR/*/Dockerfile; do
   podman build -f $DOCKERFILE
   if test $? != 0; then
    echo "Fail to rebuild $DOCKERFILE"
   fi
done

#podman pod create --name=csms-pod --hostname=csms-host  -p 9410:9410 -p 9411:9411 -v $ROOTDIR/config:/config:Z   # unprivileged
podman pod create --name=csms-pod --hostname=csms-host --ip=$PODIP --network=podman  -v $ROOTDIR/config:/config:Z # privileged
podman create --pod csms-pod --name=csms_debug --interactive -u 10000:10000 alpine
podman create --pod csms-pod --name=csms_mqtt -u 10000:10000 eclipse-mosquitto:2 "/usr/sbin/mosquitto" "-c" "/config/mosquitto/mosquitto.conf"
podman create --pod csms-pod --name=csms_lb --net csms --network-alias lb envoyproxy/envoy:v1.26-latest
podman create --pod csms-pod  --name=csms_firestore google/cloud-sdk gcloud emulators firestore start --host-port=<IP_ADDRESS>:8080
podman create --pod csms-pod --name=csms_manager --requires=csms_mqtt -u 10000:10000 localhost/maeve-csms_manager serve -c /config/manager/config.toml
podman create --pod csms-pod --name=csms_gateway --requires=csms_mqtt,csms_manager localhost/maeve-csms_gateway serve --ws-addr :9310 --wss-addr :9311 --status-addr :9312 --tls-server-cert /config/certificates/csms.pem --tls-server-key /config/certificates/csms.key --tls-trust-cert /config/certificates/trust.pem --mqtt-addr mqtt://csms-host:1883 --manager-api-addr http://csms-host:9410

for CONTAINER in csms_debug csms_firestore csms_mqtt csms_manager csms_gateway; do
 echo starting: $CONTAINER
 podman start $CONTAINER
 if test $? != 0; then
   echo "fail to start container: $CONTAINER"
   exit
 fi
done

Hoops I found my error.
My password was effectively not encoded correctly. Following operation store correctly the passwd for 'profile-0'. Note that Everest profile-0 uses unscripted websocket without any authentication/password. Such a permissive profile would be nice to have for testing purpose.
PWD_ASCII="snoopy"
PWD_BASE64=`echo -n $PWD_ASCII | openssl dgst -sha256 -binary | openssl base64`
curl http://$PODIP:9410/api/v0/cs/Tux-Basic -H 'content-type: application/json' -d '{"securityProfile":0, "base64SHA256Password":"$PWD_BASE64"}}'


[Bug] Vaal Disc inocrrectly grants bonus ES

In version 1.3.13, adding Vaal discipline to an skill group and marking it as "enabled" grants bonus ES proportional to the level of Vaal Discipline. According to the 2.6.0 patch notes, Vaal discipline no longer gives bonus energy shield.

Most of the skill changes for 2.6 haven't been made yet; I need the exact numbers and stats, and the only way I'll get those is from the game data files once the patch is out. I will remove VDisc's energy shield now though, since it's a major change.


Spawn Sphere not showing using any planks

When I put any planks on armor stand, it does not showing spawn sphere. i looked at the code, and it show
"v.spawn_show = query.get_equipped_item_name(0) == 'planks';",
i tried to change it to 'oak_planks' and it worked now.
is this because of item id update? shouldn't this pack updated too?

I think you are grabbing an old version check releases
On Thu, Apr 11, 2024, 5:59 AM Ahmad Farhan Sibghotalloh <
@.***> wrote:

When I put any planks on armor stand, it does not showing spawn sphere. i
looked at the code, and it show
"v.spawn_show = query.get_equipped_item_name(0) == 'planks';",
i tried to change it to 'oak_planks' and it worked now.
is this because of item id update? shouldn't this pack updated too?
—
Reply to this email directly, view it on GitHub
https://github.com/RavinMaddHatter/Bedrock-Technical-Resource-Pack/issues/10,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUC5FSI5Y7RH5J7TNCSFDY42CMNAVCNFSM6AAAAABGCIJZ2CVHI2DSMVQWIX3LMV43ASLTON2WKOZSGIZTONRXGI4DKNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Placeholder for reboot of the developer quickstart

Docker versus host versus Vagrant?
SSC + Stream Enrich compile and assemble?
NSQ quickstart?

Closing as dupe of the better https://github.com/snowplow/snowplow/issues/3530


SetPreferCoordGen generates strange coordinates

Configuration:

RDKit Version: '2020.03.3' and '2020.09.1dev1'
Operating system: Ubuntu 18.04
Python version (if relevant): 3.7
Are you using conda? yes
If you are using conda, which channel did you install the rdkit from? I installed rdkit from rdkit channel
If you are not using conda: how did you install the RDKit?

Description:

I couldn't render molecule which has many rings when I set SetPreferCoordGen(True).
I tried to render Adamantane with the same option, it worked well.
Any advice, suggestions are greatly appreciated.

from rdkit import Chem
from rdkit.Chem import rdDepictor
from rdkit.Chem.Draw import IPythonConsole
# following SMILES seems good without SetPreferCoordGen option.
smi = 'C12CC3CC(CC3C1)C2'
ring = Chem.MolFromSmiles(smi)
rdDepictor.SetPreferCoordGen(False)
ring

ring = Chem.MolFromSmiles(smi)
rdDepictor.SetPreferCoordGen(False)
ring

Thanks for reporting this. What do you mean "you couldn't render molecule?" - like, no image was produced or there was a traceback or something? Could you provide an example of a molecule for which it didn't work well?

Hi @iwatobipen, I'm unable to reproduce this with either v2020.09.1:

or with v2020.03.6:

Hi Greg,
Thanks for your reply. Yes, Adamantan worked well but I still have trouble about following ring system ''C12CC3CC(CC3C1)C2''.
I uploaded the example code on my gist.
https://gist.github.com/iwatobipen/4984f397dfb4e027632cc7ee3bdc2122
Could you please check the code if you have time?
Thanks,

@iwatobipen : sorry I didn't look at the molecule closely enough.
Your script is fine: coordgen is generating bad coordinates for this molecule.
@d-b-w : I also confirmed that this isn't due to the fact that the RDKit is using the coarse minimizer parameter settings, when I do:
ps = rdCoordGen.CoordGenParams()
ps.minimizerPrecision = ps.sketcherBestPrecision
rdCoordGen.AddCoords(m,ps)

the coordinates are the same.

Hi,
I have the same problem when drawing molecules with fused rings.
For example:
This code with rdDepictor.SetPreferCoordGen(False) gives the correct structure

 from rdkit.Chem import rdDepictor
 from rdkit.Chem import rdCoordGen

def get_image(smiles):
 mol = Chem.MolFromSmiles(smiles, sanitize=True)
 rdDepictor.SetPreferCoordGen(False)
 rdCoordGen.AddCoords(mol)
 rdDepictor.Compute2DCoords(mol)
 return mol

molec = 'C1=CC=C(N(C2=CC=CC=C2)C2=CC3=C4B(C5=CC=CC=C5N3C3=CC=CC=C3)C3=C(C=C5C(=C3)B3C6=C(C=CC=C6)N(C6=CC=CC=C6)C6=CC(N(C7=CC=CC=C7)C7=CC=CC=C7)=CC(=C36)N5C3=CC=CC=C3)N(C3=CC=CC=C3)C4=C2)C=C1'
m = get_image(molec)
m

while rdDepictor.SetPreferCoordGen(True) gives this:

However, I cannot use always the rdDepictor.SetPreferCoordGen(False) option because other structures (with flexible bonds) are drawn wrong, like this one:

which comes right when we have rdDepictor.SetPreferCoordGen(True) :

Is there any global solution for both kinds of structures?
Thank you very much.

ugh, yeah, these are pretty bad. Thanks for reporting these.
I've cloned this issue over to the schrodinger/coordgenlibs as https://github.com/schrodinger/coordgenlibs/issues/83


Add hybrid collaborative filtering recommender

Goal
Integrate experimental hybrid collaborative filtering recommender. See https://github.com/Pocket/dl-metaflow-jobs/blob/main/home_personalization_jobs/HybridCFModelFlow.py
Todos

[x] Deploy on dev
[ ] Deploy in prod under the feature flag to test manually
[ ] Make sure the logic satisfies our expectations
[ ] More tests if needed
[x] Load test

Reference
Tickets:

Link to JIRA tickets

Implementation Decisions

use our regular S3 bucket for model storage instead of Metaflow one. Metaflow lib is very heavy and brings too many dependencies
implemented a custom slate provider to not mess with already overwhelmed abstraction in ForYou provider

I ran a load test on dev with 20RPS and I don't see a big difference with the default home. I do see a higher CPU utilization (~60%) and ECS starts scaling the service. Our real load in production is way lower (~20RPM), so I don't anticipate issues here, at least until we land all users on home. I also increased memory to accommodate model reloading.


matplotlib widgets in napari

🚀 Feature
Following some guidance from @story645 and @tacaswell we were able to get a matplotlib widget into napari using some instructions from the matplotlib documentation https://matplotlib.org/3.1.1/gallery/user_interfaces/embedding_in_qt_sgskip.html
If you have matplotlib installed then the following code will add a widget:

import numpy as np
import napari
from skimage import data
from matplotlib.backends.backend_qt5agg import FigureCanvas
from matplotlib.figure import Figure

with napari.gui_qt():
    blobs_raw = data.binary_blobs(length=64, n_dim=2, volume_fraction=0.1)
    viewer = napari.view_image(blobs_raw)

    static_canvas = FigureCanvas(Figure(figsize=(5, 3)))
    axes = static_canvas.figure.subplots()
    axes.plot(blobs_raw.mean(axis=0))
    viewer.window.add_dock_widget(static_canvas, area='bottom', name='matplotlib figure')

We were also discussing how to make a new figure manager so that users could just do plt.plot and have it pop up in the dock widget. I was pointed to these two parts of matplotlib
https://github.com/matplotlib/matplotlib/blob/8ac3ea170fee9ab7b78767b9a7463332a5970880/lib/matplotlib/backends/backend_qt5.py#L502
and
https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/backends/backend_qt5agg.py#L98
Will be great for a plugin! But putting it here for others to see too :-) Thanks to the matplotlib team for the help!!

this looks awesome! thinking bigger-picture, do you feel like this kind of functionality is generally the right approach for stuff like 1d plotting in napari?

@d-v-b I certainly see this as a good option and making matplotlib plugin. As to inside napari directly, I think it still makes sense to think about some of the 1D plotting stuff @tlambert03 was working on in #823

Thinking about this again, the chain of calls here is :
plt.plot() -> plt.gca() -> plt.gcf() -> {return currently active figure or create a new figure via plt.figure()} ->  plt.figure() -> plt.new_figure_manager() -> _backend_module.new_figure manager -> _backend_mod.new_figure_manager_given_figure -> creates a FigureCanvas and  FigureManager instances.
The FigureCanvas is the Qt widget that is bound to the Figure object.  It is implemented as a widget where we have replace the paint methods with "copy the RGBA buffer of the rendered figure to the screen".  The FigureManager is the QtWidget.MainWindow which has as children the FigureCanvas widget and the toolbar.  In addition to holding the widgets, the Manager owns the logic for opening the save figure dialog, toggling full screen, passing through to the window title, and some hooks for how to manage the destruction of the window / stopping the event loop.
I think the path through this is

write a viewer / model pair that manages (tabbed?) figures in a way that will play nice with napari, have tool bars etc which will involve implementing a FigureManagerNapari for each tab and a figure manager manager to be the outer dock widget
wrap that up in a function to look like new_figure_manager
wrap that in a custom backend that when you create the manager-manager and add it is a dock widget so that plt.xyz "just works"

If you get 1) done I think there is still great value to this as fig, ax = view.get_the_doc_widget.subplots() etc will get the user as Axes and Figure they can start to work with.  2 and 3 are nice-to-haves as they let users who are used to pyplot have it "just work", but requires a bit of configuration jiggery-pokery.  I am thinking about ways to make that less brittle on the Matplotlib side, but we should not count on that getting done.

I'll make a quick note here as I am doing some tests on this. The example in the matplotlib qt website works perfectly (even the dynamic one). One thing that might help some people is that after updating the canvas in a function that is not init (e.g., using a button), you have to manually refresh the canvas by calling the draw function.
From my side, i am still considering the advantages of using matplotlib and pyqtgraph. One obvious one for matplotlib is the direct translation from a jupyter notebook prototyping to an integrated plugin in napari.

fig.canvas.draw_idle is better than calling fig.canvas.draw as it will defer actually rendering until the next time the canvas is going to repaint itself so it is efficient to call many times between screen re-draws.

This is pretty cool. So the idea @sofroniewn would be to have a default "plots" dockwidget, correct?
Would love to work on this :)

So the idea @sofroniewn would be to have a default "plots" dockwidget, correct?
Do we want this to be a core feature of napari, or a plugin? (even if developed internally as a plugin)
Would love to work on this :)

one of the reasons we paused on #823, and the NapariPlotWidget idea in #675 was in part because the scope of a "default plotting" widget quickly got out of control.  I think the general feeling at this point is that this should start as a plugin, and if something emerges as an "indispensable" tool, we could merge it into core.
in that vein, @lukasz-migas  has been doing some amazing work on a napari plotting plugin prototype over at https://github.com/lukasz-migas/napari-1d

Would love to work on this :)

@goanpeca that's great! As @tlambert03 said, I'd definitely encourage reaching out to and potentially teaming up with @lukasz-migas around his 1D plotting which I believe uses vispy under the hood.
Additionally I think making it as easy and smooth to do matplotlib widgets in napari makes sense - that could actually just be a really good tutorial
finally i do think it would be good if we started trying to add some additional 1D plots to napari itself, like value histograms and profiles, maybe using the 1D stuff from either a plugin or directly in napari. it's been a while since those last convos, and i think we've learnt a lot as a team on how to move work forward since then

👋 I thought I'd drop by and mention that I've started work on a napari-matplotlib plugin: https://github.com/dstansby/napari-matplotlib. Currently the elevator pitch and scope are "napari-matplotlib is a bridge between napari and matplotlib, making it easy to create publication quality Matplotlib plots based on the data loaded in napari layers.". I'd be super interested in any feedback or suggestions folks on this thread have for the plugin, including the scope I settled on for it 😄

Awesome @dstansby I think a lot of folks will be interested in that - maybe you could talk about in one of the community meetings? https://napari.org/community/meeting_schedule.html


graceful shutdown

Previously, shutdown with sigint just killed the app in the middle of whatever it was doing (e.g. writing out json, responding to http client, etc.).  None of the waitgroups or anything were being used, but that's just as well because they weren't balanced and would have panicked.
This catches SIGINT and gracefully shuts everything down.
^C2020/12/24 16:51:43 Shutting down...
2020/12/24 16:51:43 Crawler done.
2020/12/24 16:51:44 HTTP server done.
2020/12/24 16:51:44 134 nodes saved to /home/dcrseeder/.dcrseeder/testnet3/nodes.json
2020/12/24 16:51:44 Address manager done.
2020/12/24 16:51:44 Bye!

With this code we can be reasonably sure that a peers data file will be written on shutdown. As a result we can probably increase dumpAddressInterval - 30 seconds is quite aggressive, how about 5 or 10 minutes?
Also, I notice that you are updating Copyright from 2018 to 2020. Probably needs another change, so it reads 2018-2021

With this code we can be reasonably sure that a peers data file will be written on shutdown. As a result we can probably increase dumpAddressInterval - 30 seconds is quite aggressive, how about 5 or 10 minutes?
Also, I notice that you are updating Copyright from 2018 to 2020. Probably needs another change, so it reads 2018-2021


Automated cherry pick of #4212: guestnetworks: ignore empty preferIfname

Cherry pick of #4212 on release/2.10.0.
#4212: guestnetworks: ignore empty preferIfname

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@8909cba). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #4216   +/-   ##
================================================
  Coverage                  ?   7.47%
================================================
  Files                     ?     418
  Lines                     ?   66661
  Branches                  ?       0
================================================
  Hits                      ?    4984
  Misses                    ?   61251
  Partials                  ?     426

Flag
Coverage Δ

#aFlag
7.47% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/guestnetworks.go
0.22% <0%> (ø)

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 8909cba...16fadcd. Read the comment docs.

/lgtm

/approve


INFO-3 Subscription channels remain open after token expiration

This fixes that a subscription channel stays open after the token of the subscriber has expired. We need to decide if we want to fix this. Currently it is only fixed for ChangeSubscription which the new API uses. If we want to do it for the old query subscription as well it can  be done similiar. To test it take a jwt token of kuksa-val/jwt/ and use a timestamp that expires in the near future -> go to https://www.epochconverter.com/ for timestamps ;)
It is not that pretty in code I think so do not now if we want to fix this in this way/at all. One observation I made was that even if the channel is kept open somehow if you update the token new updates do not come in in the databroker-cli again. Haven't figured out why yet.

and I opted for closing if a new notification would be sent out so no general repeated check. This means if you do not update soem things channel can stay open a while.

Code looks fine to me, tested with python sdk and the channel closes correclty

I just thought that it's maybe not the ideal way code wise. But since @argerus had nothing to complain yet :) From my side we can merge.

Databroker has been migrated to https://github.com/eclipse-kuksa/kuksa-databroker.
Please open a new pull request in that repo.


Docker container

Do you consider packaging pageres(-cli) in a working docker container? It would be a lot easier for me to add it to our workflow :).

You could easily do this yourself and publish the container to Docker Hub. We are not considering doing this.

I took the time to do this: https://hub.docker.com/r/jojomi/pageres/
Example usage
docker run --rm --mount type=bind,source="$(pwd)/data",destination=/data jojomi/pageres-cli pageres [ wikipedia.org 800x600 1024x768 1600x900 1920x1080 ]


I can not launch after previous spyder upgare

I upgraded spyder. Everything seems to be ok. After I reboot my comp I can not lauch it.
Well, I created new virtual env
python3 -m venv virtenv/
pip3 install --pre -U spyder

btw it's the same error.
spyder-4.0.0-py3
Distributor ID: Ubuntu
Description:    Ubuntu 18.04.3 LTS
Release:        18.04
Codename:       bionic

btw, previous version
pip3 install spyder==4.0.0rc2
helped me a lot. It works fine

I'm sorry but we don't have the time nor the resources to help people with pip installations. So our only suggestion is for you to switch to Anaconda.


Framework support for [FRAMEWORK_NAME]

Describe the solution you'd like:

Provide a link to a github repo containing an example of the framework:
Additional context:

Closing this issue as it's unclear what goes wrong and none of the required details are filled in. Please, feel free to open a new issue and fill in all the necessary details so we have enough information to have a look ourselves. The more information we have, the better we can help you :pray:


Segmentation fault - document.h line 618

if (name[member->name.data_.s.length] == '\0' &&
memcmp(member->name.data_.s.str, name, member->name.data_.s.length *
sizeof(Ch)) == 0)
				return member;
}

This code in document.h line 618 generate segmentation fault on the ARM
architecture.

Problem in the member->name.data_.s.str but I don't know why.
CString at member->name.data_.s.str exist.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 31 Jan 2012 at 2:09

in document.h :618

name[member->name.data_.s.length] == '\0' <-- might access out of boundary when
the "name[]" is shorter than any member

reverse the order of memcmp and the zero tail check might be better
From:
name[member->name.data_.s.length] == '\0' && memcmp(member->name.data_.s.str,
name, member->name.data_.s.length * sizeof(Ch)) == 0
To:
memcmp(member->name.data_.s.str, name, member->name.data_.s.length *
sizeof(Ch)) == 0
&& name[member->name.data_.s.length] == '\0'

Original comment by<EMAIL_ADDRESS>on 31 Jan 2012 at 6:18

From e-mail:

const char* json = "{\"message_id\": 3,\"uid\": 55367}"

Parsing code :
// (IwAssert(PARSER, ...) is assert macro)
Document document;

IwAssert(PARSER, !document.Parse<0>(json).HasParseError());
IwAssert(PARSER, document.IsObject());

/* Get Data */
IwAssert(PARSER, (document["message_id"].IsUint())); // error here

I check in the debugger:
name  - contain "message_id" with zero at end
member->name.data_.s.str - contain "message_id" with zero at end
member->name.data_.s.length - equal to 10
sizeof(Ch) - equal to 1

but any attempt access to the member->name.data generate segmentation fault.

Original comment by<EMAIL_ADDRESS>on 31 Jan 2012 at 3:16
Attachments:

locals_vars.jpg

any attempt access to the member->name.data generate segmentation fault

Original comment by<EMAIL_ADDRESS>on 31 Jan 2012 at 3:02

Let me start by saying that this library is absolutely beautifully written!  I
really want to use this library.

A similar problem occurs in document.h with ARM architecture here:

    //! Assignment without calling destructor
    void RawAssign(GenericValue& rhs) {
        memcpy(this, &rhs, sizeof(GenericValue)); // <<-- here
        rhs.flags_ = kNullFlag;
    }

I am running tutorial.cpp on ARM v7 little Endian.  Using a debugger, I find
the following:

this and rhs are instances of GenericDocument, which are derived from
GenericValue.  sizeof(GenericValue) is 24; the data portion of GenericValue all
fits into 20, but there is 4 bytes of padding to align on 8 byte boundaries,
apparently.  So, sizeof operator gives the size when an instance of
GenericValue is created.

When the actual instance is a derived class, the derived portion does not start
at 24, but at 20, which is the end of the data portion of GenericValue.  That
is, the 4 bytes padding is not used in this case.  The result is that memcpy
clobbers the pointer "allocator_" at the beginning of GenericDocument's data.

Also, there is comment like this in document.h for the struct Data:

    // 12 bytes in 32-bit mode, 16 bytes in 64-bit mode

This is apparently not true for ARM.  It is 32-bit, but the sizeof in memory is
padded to 16 bytes.  Also, even though the struct is a value type within an
instance of GenericValue, the struct is still padded.  This is, I guess,
controlled by specification, i.e., sizeof gives the size within the instance.
This is a different situation from the inheritance, where the padding is
dropped for GenericValue when appending the derived class's data portion.  C++
object layout is not controlled by specification.

In general, since the layout of objects is not controlled by specification,
would it not be better to avoid low level memory operations on C++ instances?

Original comment by<EMAIL_ADDRESS>on 8 Feb 2012 at 10:05

The cause of the problem in that platform is unaligned memory access.

It is found that, it is due to MemoryPoolAllocator::Malloc()/Realloc() did not
ensure 4-byte alignment in the return value.

Original comment by<EMAIL_ADDRESS>on 31 Jan 2012 at 6:20

Changed state: Started

@jasonchan35

It seems reversing the order will still possible to access out of boundary.

I think that a safer approach should first compute strlen(name) and compares it
with the member's, and then do memcmp().

But thank you for pointing out that.

Original comment by<EMAIL_ADDRESS>on 31 Jan 2012 at 6:26

This issue was closed by revision r53.

Original comment by<EMAIL_ADDRESS>on 19 Feb 2012 at 2:30

Changed state: Fixed

Hi Milo,

Again, a great library.  I really like it.

Regarding issue 11, I am still having problems.  It still segfaults, owing
apparently to assumptions about alignment of derived class GenericDocument
with respect to its base class, GenericValue.

Stack trace:

rapidjson (1) [BlackBerry Tablet OS Postmortem Debugging]
    QNX GDB Debugger (12-02-21 10:25 AM) (Suspended)
        Thread [1] (Suspended)
            5
rapidjson::MemoryPoolAllocator<rapidjson::CrtAllocator>::Realloc()
/home/johodgson/rapidjson/include/rapidjson/allocators.h:171 0x0010504c
            4 rapidjson::GenericValue<rapidjson::UTF8<char>,
rapidjson::MemoryPoolAllocator<rapidjson::CrtAllocator> >::Reserve()
/home/johodgson/rapidjson/include/rapidjson/document.h:377 0x00104e58
            3 rapidjson::GenericValue<rapidjson::UTF8<char>,
rapidjson::MemoryPoolAllocator<rapidjson::CrtAllocator> >::PushBack()
/home/johodgson/rapidjson/include/rapidjson/document.h:393 0x00103a20
            2 rapidjson::GenericValue<rapidjson::UTF8<char>,
rapidjson::MemoryPoolAllocator<rapidjson::CrtAllocator> >::PushBack<int>()
/home/johodgson/rapidjson/include/rapidjson/document.h:401 0x00102a64
            1 main()
/home/johodgson/rapidjson/example/tutorial/tutorial.cpp:88 0x001016cc
    /opt/bbndk-2.0.0/host/linux/x86/usr/bin/ntoarm-gdb (12-02-21 10:25
AM)
    <terminated, exit value:
0>/home/johodgson/rapidjson/Device-Debug/rapidjson (12-02-21 10:25 AM)

Variables:

this    0x00000000
originalPtr    0x0019042c
originalSize    96
newSize    192
__func__    0x0010b7a4
newBuffer    0x001905c4

Allocator (this) is NULL, because it gets clobbered with a memcpy using
sizeof(GenericValue) (=24 bytes).  I checked it out using View Memory in my
QNX Momentics IDE.  Layout of derived class GenericDocument object is
immediately following GenericValue (starting at byte 20), without the extra
four bytes of padding at the end of GenericValue.  The compiler puts in the
extra padding (I don't know why), which is what brings it out to the sizeof
24 bytes, but the compiler does not use this padding when laying out the
derived class.

My compiler is QCC:

qcc -o example/tutorial/tutorial.o ../example/tutorial/tutorial.cpp
-V4.4.2,gcc_ntoarmv7le_cpp -w1
-I/opt/bbndk-2.0.0/target/qnx6/../target-override/usr/include
-I/home/johodgson/rapidjson/include
-I/home/johodgson/rapidjson/thirdparty/gtest/include -D_FORTIFY_SOURCE=2 -c
-g -fstack-protector-all
qcc -o rapidjson example/tutorial/tutorial.o -V4.4.2,gcc_ntoarmv7le_cpp -w1
-lang-c++ -g -Wl,-z,relro -Wl,-z,now
-L/opt/bbndk-2.0.0/target/qnx6/../target-override/armle-v7/lib
-L/opt/bbndk-2.0.0/target/qnx6/../target-override/armle-v7/usr/lib

This change at the bottom of document.h fixes it for me, because it simply
moves GenericDocument data farther down, out of the range of clobbering.

     static const size_t kDefaultStackCapacity = 1024;
+    char padding_[4];
     internal::Stack<Allocator> stack_;
     const char* parseError_;
     size_t errorOffset_;

I think that this is a safe change for 32-bit architectures, but may not
work on a 64-bit architecture (may need to add more padding, e.g., 8 bytes).

I'm working on an ARM v7 on RIM PlayBook, which is 32-bit.  The new ARM v8
will be 64-bit, so stuff compiled as 64-bit may not work.

Your library is a really nice template approach.  The inheritance
GenericValue <- GenericDocument may be out-of-character.  Maybe there is a
way to move out the document-building code and avoid inheritance.  That may
be a better way of fixing the issue.

Please don't hesitate to contact me if I may be of any assistance, e.g., in
testing.

Kind regards, John.

Original comment by<EMAIL_ADDRESS>on 21 Feb 2012 at 4:16

I also try to use rapidjson on an arm platform.

The latest version 0.11 (rev98) still has the same alignment problem on ARM.
But John's workaround works. But I think Jonh's workaround is just a quick fix.
I'd be happy to test any official patches.

Original comment by<EMAIL_ADDRESS>on 11 Jan 2013 at 1:38

Original comment by<EMAIL_ADDRESS>on 22 Feb 2012 at 8:08

Changed state: Accepted
Added labels: Priority-High
Removed labels: Priority-Medium

Confirm the issue still exists with latest version and John's workaround works
as well. Tested on HTC Droid DNA.

Original comment by<EMAIL_ADDRESS>on 12 Mar 2013 at 3:38

Issue 43 has been merged into this issue.

Original comment by<EMAIL_ADDRESS>on 14 Nov 2012 at 3:49

https://github.com/miloyip/rapidjson/issues/8

Original comment by<EMAIL_ADDRESS>on 20 Jun 2014 at 8:25

Changed state: Fixed

I have pushed a fix for this issue to my GitHub fork at
https://github.com/pah/rapidjson/commit/7475a969

Original comment by<EMAIL_ADDRESS>on 1 Feb 2014 at 6:46


Check for updated inputs while executing DAG

Is your feature request related to a problem? Please describe.
My snakemake workflow can take several hours to complete. On rare occasions, an input file to the beginning of the workflow gets updated while snakemake is running. This means that any ongoing processing is basically wasted since snakemake will just have to do the entire workflow all over again at the next invocation.
Describe the solution you'd like
It would be nice to have the option to have snakemake recheck workflow inputs as jobs are completed to see if we need to at the very least to not start new jobs, or even better, automatically go back and rework the needed parts of the DAG to take into account new inputs.
Describe alternatives you've considered
The alternative is wasted processing time and delayed products.
Additional context
None

Note, such a feature would also help address a race condition that if an input file is updated while the first job is running (original file already read, but output file(s) not written yet), then the file is "newer", but it is still older than the eventual outputs. I actually don't know for sure whether or not snakemake can detect this situation right now, but depending on what metadata is recorded during the DAG build, it could be addressed with such a mode by noticing that the input file that triggered the DAG is no longer the same input file on disk.


@startup setpriv: libcap-ng is too old for "all" caps

Hello everyone,
The docker container Sinusbot/docker:lastes does not start. It is only shown in the log "setpriv: libcap-ng is too old for "all" caps".
Extract from the log:
Updating youtube-dl...
youtube-dl is up-to-date (2020.09.20)
User ID: 0
Change file owner...
Drop privileges...
Starting SinusBot...
PID: 11
setpriv: libcap-ng is too old for "all" caps
To my system:
Opensuse Thumbleweed (up to date)
What can I do to get it running again?

Seems like its an upstream bug, which kernel are you using? https://bugs.archlinux.org/task/67781

Hi,
My Release is Linux opensuse 5.8.10-1-default #1 SMP Mon Sep 21 11:00:19 UTC 2020 (af3e800) x86_64 x86_64 x86_64 GNU/Linux

Seems like not a bug or issue which we can fix on our side. It seems to be persistent on kernel version 5.8.X as linked in the issue above.

Same here. Not able to start the container.. @stertzi2012 were you able to find a workaround? Thx, Michal

@mxschmitt what do you propose? they closed the bug with new releasing new versions:
Closed by  Christian Hesse (eworm)
Wednesday, 02 September 2020, 09:10 GMT
Reason for closing:  Fixed
Additional comments about closing:  libcap-ng 0.7.11-2
util-linux 2.36-3

This should be fixed now, it would be nice if someone could test it and provide feedback.
Make sure to update as described in the README.

I can confirm that the error is now fixed.


unable to get login using correct credentials

i removed special characters from my password as i found noted in https://github.com/joshfraser/robinhood-to-csv/issues/5 -- even after doing that, i get the same error as before i changed my password:
Traceback (most recent call last):
File "d:\app\robinhood-to-csv\csv-export.py", line 46, in
logged_in = robinhood.login(username=username, password=password)
File "d:\app\robinhood-to-csv\Robinhood.py", line 81, in login
res = res.json()
File "d:\dev\Python37\lib\site-packages\requests\models.py", line 897, in json
return complexjson.loads(self.text, **kwargs)
File "d:\dev\Python37\lib\json_init_.py", line 348, in loads
return _default_decoder.decode(s)
File "d:\dev\Python37\lib\json\decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "d:\dev\Python37\lib\json\decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

I was able to get it to work for my use after making changes in Robinhood.py.

Change the login endpoint to "login": "https://api.robinhood.com/oauth2/token/" (Search for "login":  "https://api.robinhood.com/api-token-auth/")
Add client_id = "c82SH0WZOsabOXGP2sxqcj34FxkvfnWRZBKlBjFS" after positions = None
Search for "if mfa_code:"  and change as follows:
       if mfa_code:           fields = {'password': self.password, 'username': self.username, 'mfa_code': self.mfa_code',                           grant_type': 'password', 'client_id': self.client_id}       else:           fields = {'password': self.password, 'username': self.username, 'grant_type': 'password',                     'client_id': self.client_id}

Couldn't get code in the 3 bullet to format correctly but just follow the format in existing code.

Merged the fix for this. Thanks for your patience folks.


Did you forget to remove this? (15.5b4)

Did you forget to remove the 15.5b4 part from the app?

Was wondering the same thing as well cause I'm on the same iOS version :/

Was wondering the same thing as well cause I'm on the same iOS version :/

This is your screenshot btw :P

I did think that but I was like nah surely not


feature: provide state storage for n2k-signalk

Please include n2k-signalk version bump to 0.0.1 as it is directly related, see# 246.


Set jest environment for standard

After this change, the Atom plugin linter-js-standard also works.
See https://github.com/standard/standard/issues/122#issuecomment-217151207

Codecov Report

Merging #33 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #33   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          12     12
  Branches        1      1
=====================================
  Hits           12     12

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 5ea7ae5...67245b4. Read the comment docs.


reset() on return from PayPal throws Uncaught TypeError

HTML file given in cancel_return parameter of PayPal form.
HTML contains
<script src="//cdnjs.cloudflare.com/ajax/libs/minicart/3.0.3/minicart.min.js"></script>
<script>
            paypal.minicart.reset();
</script>

When this page is opened on return from PayPal, an exception is thrown and the cart is not cleared:
Uncaught TypeError: Cannot call method 'destroy' of undefined

I've been having exactly the same issue.  Placing the render statement on the return page fixed it.  Is there any chance of putting this fix on the homepage - I've spent hours getting this sorted.