Add snippets to Blob's javadoc and BlobSnippets class

Coverage decreased (-0.005%) to 85.865% when pulling 6a7ca65882efd2a7cb51637c86f99a4366d9bc04 on mziccard:blob-snippets into 36ba7c3ea5e33a6647da717f93f5153cd1eafbeb on GoogleCloudPlatform:master.

Coverage remained the same at 85.865% when pulling 9ac507980dcef1a67d437b8be716cb59d8394213 on mziccard:blob-snippets into 23eb620b8c752841a0148d9932964116c7e13463 on GoogleCloudPlatform:master.

/cc @lesv

LGTM


A trick to pass the authentication control

Hi, I have a problem with our web app using this AuthProvider where its quite easy to get round the authentication control against our oidc backend service. This by adding empty oidc items to the local/session storage.
How to do:

Browse to e.g. http://localhost
In dev console run following js code:
window.localStorage.setItem('oidc.foo', '{}')
window.sessionStorage.setItem('oidc.user:<theUrlToOIDCService>:<theClientId>', '{}')
Change the url to the app, e.g. http://localhost/myapp
The app is loaded and has skipped the authentication request to the backend

It seems like the app thinks that user is already signed in and redirects to the main page. But I dont really know if we have implemented this in a bad way or if its this component having this issue?
Thanks for any input.

Hello, @avanceaw!
Thanks for opening this up. It's intended to set a signed in state based on these localStorage. oidc-react does not handle any of this directly, we use the underlying library oidc-client-ts. Feel free to open a issue there though :)


JPG/PNG images containing alpha channel information break /model/predict

Receiving this input

/mode/predict returns HTTP code 500 and the console output contains a stack trace entry
similar to the following: ValueError: Cannot feed value of shape (1, 615, 461, 4) for Tensor 'pnet/input:0', which has shape '(?, ?, ?, 3)'

Fixed in #14


[DNM]schema(ticdc): non integer primary key is not handle key

What problem does this PR solve?

Issue Number: close #xxx
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/test all

/retest

/test cdc-integration-storage-test

/cherry-pick release-7.5


Data Science with Python : SARSA Algorithm

Welcome to 'DSWP' Team, good to see you here
This issue will helps readers in gaining all the guidance that one needs to know about SARSA Algorithm. Tutorial to SARSA Algorithm and how it's applied using sample code.
To get assigned to this issue, add your serial numbers mentioned in the spreadsheet of "Data Science with Python", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
If you had referred any resources, add them up in "DS Resources". Similarly if you had used datasets, include them in "DS Datasets".
Domain : Machine Learning
Mentors Assigned : Arya Patel / Ishika Kesarwani
Points to Note :
The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Changes should be made inside the Datascience_With_Python/ directory & Datascience_With_Python branch.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'GWOC' contributors of 'DSWP' domain.
All the best. Enjoy your open source journey ahead. 😎

/assign
Name:- Vadiraj Rao
Batch:- 9
Serial No:- 293
Type:- Documentation
Approach:- Introduction to topic, theoretical explanation of algorithm, explanation using diagrams, example code snippets

@prathimacode-hub , Mam
I am M. Ashish Reddy. I would like to do video on this issue.
Batch number: 5
serial number:174.
Approach: A detailed explanation with examples and real life uses

Issue assigned to @ashish-reddy-20-08 for video contribution

Hello, I am Deepthi M with serial number:172, Batch-5. I would like to do audio on this issue.
My approach: a detailed explanation

Issue assigned to @deepthi1107 for audio contribution


typeRoots is not resolved as part of compilation

TypeScript Version:   3.0.1  and  3.1.0-dev.20180907
(Windows 7, everything on local C:\ drive)
I can't make typeRoots to work at all. There is a lot of similar complaints over Stackoverflow and elsewhere, but I don't see a single complete workable example. So here's a simplest reproduction case. It doesn't work for me.
In summary, I install three.js from NPM into node_modules, but want to use typings from local directory.  For simplicity those are exact same types you get from @types/three — without any modifications.

Code
Project structure:
node_modules
   three  <------------ installed as usual with  "npm install three"
src
   package
      three   <------------ exact copy of typings for three.js
      webvr-api

useThree.ts
tsconfig.json

useThree.ts:
import * as THREE from "three";

function main(s: THREE.AmbientLight) {
}

tsconfig.json
{
   "compilerOptions": {
       "typeRoots" : ["./src/package"],
       "strict" : true
   }
}

Trying to compile with:
tsc.cmd  -p tsconfig.json

Expected behavior:
Compile successfully. Also show suggestions when you type THREE. in VSCode.
Actual behavior:
Compiler doesn't understand typeRoots at all.
error TS2688: Cannot find type definition file for 'three'.

Also VSCode treats THREE as any and doesn't offer suggestions like AmbientLight and others.
As I said, several of confused questions on Stackoverflow, with general answer reciting scarce TypeScript/tsconfig docs on the typeRoots:

If typeRoots is specified, only packages under typeRoots will be included. For example:
 {
    "compilerOptions": {
        "typeRoots" : ["./typings"]
    }
 }

You can see, I used the same syntax, to the point of leading ./ and lack of trailing /.
Quite possibly some other intricate (or obvious??) detail is missed on my part. In that case what would help A LOT is one small thing: a simple reproduction example where typeRoots works as expected.
I can see there are couple dozen tests exercising @typeRoots with harness virtualized file system, but would be good to have it clean and reproduceable with just tsc. Thanks!

I can't reproduce the problem.  Use the --traceResolution flag to get more information, and if you still can't fix the problem, please ask on Stack Overflow and we can troubleshoot there.

We'd need at minimum a sample repo for this, since we can't see what the filenames are, what the file contents are, whether there are package.json files, etc.

I've created a trivial reproduction repository:
https://github.com/mihailik/typeRoots

git clone   https://github.com/mihailik/typeRoots
npm install (getting typescript, three and @types/three packages in respective places)

typescript and three go in normal place
@types/three intentionally goes in a nested subdirectory — precisely the use case for typeRoots

npm run build to fire tsc with tsconfig

I've reproduced it on:

Windows 7 with node v8.11.1
Cloud9 (Ubuntu 14.04.5 LTS) with node v6.11.2

Should be pretty easy to check and hope you can point out the issue easily.
If it's a misconfiguration on my side, it would be awesome to have a similarly trivial sample case, but working. The typeRoots official docs are vague, no sample either. Let's help people use typeRoots correctly!
Thanks @RyanCavanaugh @mattmccutchen

@mihailik You should be using baseUrl/paths, not typeRoots, in this case.  See this Stack Overflow answer.
This seems to be a very common misconception and we should see if we can improve the documentation.

Thanks @mattmccutchen — follow-ups:

Do we need to include "baseUrl": "." or can it be omitted as default?
What would be the actual syntax for "path" in my case above? I'm confused whether to refer to "three" or "@types/three".

With that I can update the repo, and make it a definite working sample for other people to look at.
Thanks!

FBOFW, baseUrl is required when paths is specified.
The key should be "three" because that's what's being imported.  It's a good enough overview for present purposes that when you import ... from "foo", TypeScript checks for a declaration file in (a) any specified paths for foo, (b) node_modules/foo, and (c) node_modules/@types/foo in that order.  Given that you can never import ... from "@types/foo", a paths setting for @types/foo will never be used.

So the tsconfig.json in your example would be:
{
   "compilerOptions": {
       "lib": ["es6", "dom"],
       "baseUrl": ".",
       "paths": {
         "three": ["customtypings/node_modules/@types/three"]
       },
       "strict" : true
   }
}

@mihailik Did you get it working?

Yes, I did @darkmastermindz — following @mattmccutchen example.
Thanks a lot, by the way! 👍

Could be super-useful to put in docs too!


水印问题

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version:
Operating system and its version:ios15.6
PDF.js version: v3.3112
Is a browser extension:

Steps to reproduce the problem:

PDF的水印不见了。
2.ios顶部工具栏可点击，但不显示文字。

What is the expected behavior? (add screenshot)
What went wrong? (add screenshot)

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Please use only English in this bug tracker, since otherwise we can't help you!
Furthermore, please make sure that you clearly explain the problem (and note that Firefox is the primary development target for this library).


Remove .push() examples from documentation

With the merging of https://github.com/chef-cookbooks/audit/pull/328 using an Array of Hashes for node['audit']['profiles'] is deprecated. As such we should change all of the documentation to reflect this. This issue is a place to track this work.
Here are some of the sites I found:

[X] This repo's README.md (https://github.com/chef-cookbooks/audit/pull/358)
[ ] https://docs.chef.io/audit_cookbook.html (doesn't use .push but is still an Array of Hashes)
[ ] https://docs.chef.io/integrate_compliance_chef_server.html (doesn't use .push but is still an Array of Hashes)
[ ] https://automate.chef.io/docs/data-collection/
[ ] https://blog.chef.io/2016/11/09/the-audit-cookbook-a-how-to/

As a side note I feel we point all references on how to use the Audit cookbook to a single location.
@kagarmoe @mjingle thoughts?

@jerryaldrichiii It appears the new hash of hashes format doesn't actually work yet:
https://github.com/chef-cookbooks/audit/issues/339
Shouldn't we get this fixed before attempting to update documentation?

Done, closing. Thank you


Consider implementing bootstrap CSS styling

Vino reports that constructing the dropdown menus will be much easier. It's preferred to start a new project rather than use multi-modal code as there would be styling conflicts when bootstrap is implemented.

Also, if we are to implement the mobile layout as per wireframe in drive, bootstrap is necessary.

I am sorry to hear of this and its suggestion that we abandon the multi modal concept and all of its benefits. Such a step has MAJOR implications and should be based on a fundamental rethink of our whole implementation approach not as a byproduct of a single issue.  It would also make the use of BMD2 improvements impossible without rewrites of CEN and REG.

@Captainkirkdawson only Vino and I were at the FreeBMD2 scrum and I noted down her observations for consideration and discussion at the next meeting. I'm not sure I've noted them faithfully as I don't think Vino was suggesting we abandon multi-modal, but again, I could be wrong.

@Captainkirkdawson
Moving into the new design, needs lot of hard coding changes and more line of codes can be reduced by the bootstrap and everything else including the multi model system is going to be the same. So all I am proposing to use the bootstrap gem and a custom style to replicate the new design. Abandoning the multi model system is a big move and I am not thinking of that

I am sorry if I have overreacted but the story clearly said `It's preferred to start a new project rather than use multi-modal code' that to me was quite clear in its meaning

@vino @DeniseColbert I remain TOTALLY unclear on the issue that is pushing for a new project.
The current MyopicVicar displays are constructed from 3 partials and a page yield. The 3 partials are Header, Navigation bar, and Footer. The changes proposed to these partials are trivial and can be implemented with little effort. The page yield part is the "guts" of the display from a user perspective. All our displays use the same Header, Navigation bar and Footer and these are multimodal. The specific content displayed depends on what "view" is being displayed. That can be a search_query_form or any other view. What is being suggested for the BMD search page is similar to REG and CEN but with only ONE change and that is the use of drop down menus.  Surely that does not require us to throw out our whole design from the past 4 years? The use of a gem for drop down menus may be useful (I have not researched the requirement) but I see no reason why that cannot be done within the current structure.

@Captainkirkdawesson
The old layout needs a lot of modifications to get to new design. For example moving of site icon to the nav-bar alone needs lots of css style changes. We going to give new face for FreeBMD, I think it would be better to start as a new one instead of messing the existing styles. It would also be easy for making the new layout multimodal. It will also be helpful if want to make cosmetic changes to our site in future. Also we are using a lot of static HTML code in our erb files which needs to re-written in conventional Rails.
However, I can work with existing styles but I think it will be much easier and cleaner for me to go with the new assets by using the current multi modal work flow,
The main reason is, to install the boostrap gem to the current system might conflict with the current grid system we are using but this is my thought and am open to suggestion.

@Vino-S as discussed, we are putting bootstrap on hold until much later on in order to be able to reuse the current multi-modal code and speed along towards the MVP. At that point, we can review and look to roll it out on our other projects at the same time.


Memory leaks after running overnight

Hi There
I run the stylegan2-pytorch overnight, the memory increase from initial 3.5G to 22.6G, and the performance drops from 4s/it to 40s/it, it deteriorates 10 times,  would you check the memory leaking issue?
Thanks!

I haven't seen memory/performance problems like it. Could you specify conditions or parameters that you have used for training?

Hi, thanks for prompt reply, I am fighting with the memory leaks issue, here is my enviornment and command line, the memory leaks issue is computer RAM not GPU, GPU seems stable.

ubuntu 20.04 LTS
cuda 11

Training parameters
python train.py --iter 999999 --batch 16 --size 512 /data/db/portrait
What wired is, I use tracemalloc to trace the memory issue but the Python part seems normal, and I also call torch.cuda.empty_cache() at each epoch, there are the output of tracemalloc module.
[ Top 10 differences ]
/usr/local/lib/python3.8/dist-packages/torch/utils/data/sampler.py:110: size=7804 KiB (+7804 KiB), count=221927 (+221927), average=36 B
:580: size=300 KiB (+300 KiB), count=3266 (+3266), average=94 B
/usr/lib/python3.8/tracemalloc.py:111: size=74.1 KiB (+74.1 KiB), count=948 (+948), average=80 B
/usr/lib/python3.8/tracemalloc.py:125: size=68.7 KiB (+68.7 KiB), count=977 (+977), average=72 B
:219: size=44.2 KiB (+44.2 KiB), count=417 (+417), average=109 B
/usr/lib/python3.8/tracemalloc.py:472: size=38.6 KiB (+38.6 KiB), count=979 (+979), average=40 B
/home/swei/Pod/stylegan2-portrait/op/fused_act.py:97: size=38.2 KiB (+38.2 KiB), count=294 (+294), average=133 B
/home/swei/Pod/stylegan2-portrait/op/upfirdn2d.py:152: size=27.3 KiB (+27.3 KiB), count=199 (+199), average=140 B
/home/swei/Pod/stylegan2-portrait/op/upfirdn2d.py:97: size=22.5 KiB (+22.5 KiB), count=344 (+344), average=67 B
/home/swei/Pod/stylegan2-portrait/op/fused_act.py:58: size=15.1 KiB (+15.1 KiB), count=288 (+288), average=54 B

My tracemalloc instrumental  code
diff --git a/train.py b/train.py
index 533091b..5e052e4 100755
--- a/train.py
+++ b/train.py
@@ -10,6 +10,7 @@ from torch.utils import data
 import torch.distributed as dist
 from torchvision import transforms, utils
 from tqdm import tqdm
+import tracemalloc

 try:
     import wandb
@@ -127,7 +128,6 @@ def any_key_pressed():

 def train(args, loader, generator, discriminator, g_optim, d_optim, g_ema, device):
     loader = sample_data(loader)
-
     pbar = range(args.iter)

     if get_rank() == 0:
@@ -297,6 +297,14 @@ def train(args, loader, generator, discriminator, g_optim, d_optim, g_ema, devic
                 )
             )

+            # trace memory issue
+            torch.cuda.empty_cache()
+            snapshot2 = tracemalloc.take_snapshot()
+            top_stats = snapshot2.compare_to(snapshot1, 'lineno')
+            print("[ Top 10 differences ]")
+            for stat in top_stats[:10]:
+                print(stat)
+
             if wandb and args.wandb:
                 wandb.log(
                     {
@@ -464,4 +472,6 @@ if __name__ == "__main__":
     if get_rank() == 0 and wandb is not None and args.wandb:
         wandb.init(project="stylegan 2")

+    tracemalloc.start()
+    snapshot1 = tracemalloc.take_snapshot()
     train(args, loader, generator, discriminator, g_optim, d_optim, g_ema, device)

The memory increases about 1MB every seconds, I have to workaround it, exit and restart by a bash script.
diff --git a/train.py b/train.py
index 533091b..bbe7058 100755
--- a/train.py
+++ b/train.py
@@ -10,6 +10,7 @@ from torch.utils import data
 import torch.distributed as dist
 from torchvision import transforms, utils
 from tqdm import tqdm
+import psutil

 try:
     import wandb
@@ -28,6 +29,13 @@ from distributed import (
 )
 from non_leaking import augment

+def memory_leaks():
+    process = psutil.Process(os.getpid())
+    rss = process.memory_info().rss / (1024.0 * 1024 * 1024)
+    if rss > 5.0:
+        print("current memory usage: %0.2f GB" % rss)
+        return True
+    return False

 def data_sampler(dataset, shuffle, distributed):
     if distributed:
@@ -325,7 +333,7 @@ def train(args, loader, generator, discriminator, g_optim, d_optim, g_ema, devic
                         range=(-1, 1),
                     )

+            quit = any_key_pressed() or memory_leaks()
             if i % 10000 == 0 or quit:
                 torch.save(
                     {

Hmm maybe it is pytorch 1.6 & custom operator related problems. I will check about it.

Update: It might be related to dataset size / data loader / LMDB issue ...
I decreased the dataset size from 200,000 images to 10,000, now the memory usage is stable at 4.73G, should we set a memory limit to the LMDB? Even though my computer has 128G RAM, but the process takes more memory will perform worse, I guess it reaches 10G or more will trigger the performance degrade.

Yes, maybe it is related to that. I have used lmdb for larger size than that without problems. But lmdb can have subtle interactions with multiprocessing. So I will try to use better mechanisms.


update packages

@jeffeb3 This PR updates all possible packages. A few couldn't be updated due to dependencies. It builds and runs fine for me, but I did not do extensive testing.

Thanks for doing that. I did some testing and it looks good. The changes make sense.


Run linter

supsersedes #13
I followed this tutorial https://pre-commit.com/
What do you think?

Looks nice! Couple of comments:

Can the linter enforce brackets after assert (e.g. assert(x is not None) instead of assert x is not None)?
What happens when the linter is run in GitHub actions? Would the checks fail on lint or would it impose the formatting?

It looks like maybe not --  https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#parentheses
The linter should fail the test. But the pre-commit hook (if installed correctly) should prevent any commits that do not pass the linter (and will automatically lint your code, leaving the changes unstaged.)

RE: assert, that's probably correct behavior. Brackets in assert can lead to nasty unexpected stuff:
https://stackoverflow.com/questions/3112171/python-assert-with-and-without-parenthesis

I'm happy with this, feel free to merge unless there's anything missing.

Sounds good. Make sure you run pip install pre-commit and pre-commit install after you get these changes!


left sidebar: Re-narrow after initial streams list rebuild after reload.

We use handle_narrow_activated instead of update_sidebar_for_narrow because the latter doesn't call scroll_stream_into_view, resulting in wrong scroll position of the left sidebar after the streams list get rebuilt after reload. handle_narrow_activated also calls update_sidebar_for_narrow, so there should be no issues in switching to this function.
Fixes #9043.

Hi, @synicalsyntax, thanks for digging into this.  This is a tricky issue, because there are lots of different circumstances in which the stream list gets updated, but we want to be careful not to update the scroll in situations where the current user hasn't really done anything.  For example, suppose Cordelia is narrowed to stream A, but she starts scrolling down to page 2 looking for stream Y.  If at the same time Aaron renames stream B, we want the rename-stream event to redraw Cordelia's sidebar without jumping her scroll back up to page 1.
So, long story short, I think your fix is a bit too aggressive.  I'm not convinced we want to call handle_narrow_activated every time we call update_streams_sidebar.
We also want to be careful about names here.  The function handle_narrow_activated should only be called when you're activating a narrow.  We could rename it to something more general, of course, but I think it's better to just explicitly call scroll_stream_into_view in places where that makes sense.

I agree that calling scroll_stream_into_view is probably better.

When a narrow is reloaded, the following things happened:

subs.initialize() was called, which in turned called streams_list.build_stream_list()for the first time
hashchange.initialize()calls narrow.activate()
narrow.activate() in turn calls streams_list.handle_narrow_activated()
streams_list.handle_narrow_activated()calls streams_list.scroll_stream_into_view, which correctly scrolls the stream into view.
However,message_fetch.initialize() calls streams_list.build_stream_list()again for the second time, which triggers streams_list.update_streams_sidebar() (which doesn't scroll the stream into view).
Step 5 occurs 2 more times, and then nothing else gets called.

Given these circumstances, where do you think it would be best to call scroll_stream_into_view()?

@synicalsyntax I just added a comment about keeping as much logic as possible out of message_fetch.js.   think @timabbott was also seeing some blueslip errors.  Not sure if it's a bug or just a case where warn or info should have been used instead of error--he can give you more details.

@showell Alright, I updated my PR according to your feedback, and I changed the error to a warn. please let me know if there's anything else I should fix!

Merged as c3b1381c2a5bee6c0af62fa1e285f61196945627, thanks @synicalsyntax!
The overall behavior isn't ideal here, in that things bounce around in the left sidebar (ideally, they'd just land in the perfect state immediately), but we'd need to extend page_params with more data to prevent that.  An it's certainly better to bounce around and land in the right place, than to bounce around and land in the wrong place.


Migrate filtering to the Backend

Implement pagination through limit and offset
Refactor code in assessments to use ecto querying instead of raw sql
Implement filtering using query parameters
Change response for AdminGradingController :index to return an object with "count" and "data" instead of an array of submissions. Where "count" is the total number of submissions for the query (For pagination) and "data" is the array of submissions.

TODO:

Refactor code
Update tests to account for change in response

@RichDom2185
Do I need to implement tests for each query type e.g. by name/mission.

coverage: 94.753% (-0.5%) from 95.295%
when pulling 0fd3e6539cb8ec836a47c3ab317053928df838b4 on GabrielCWT:gabriel/pagination
into 18a71d366552ee36cdd58b9eff829a4bddc011c4 on source-academy:master.

(minor change) hi gabriel, i think the filter now to restrict views runs only when group == true, but this value represents showAllGroups. Hence, the intended functionality is actually flipped. Could you help to update this to false? Apologies for the late comment. Thank you! (Assessments.ex 1430)

Refere

@josh1248 To my knowledge it is meant to be whether "only the groups under the grader should be returned". See the old function all_submissions_by_grader_for_index in line 1244. There is a doc which explains it

okies, updated to match!

Relevant frontend PR: https://github.com/source-academy/frontend/pull/2787

@RichDom2185  I can remove the old stuff in assessments.ex and admin_grading_controller.ex if everything looks good. With regards to the routes, I'm not too sure which old routes you are talking about since we are using the same route as before.

@RichDom2185 I can remove the old stuff in assessments.ex and admin_grading_controller.ex if everything looks good. With regards to the routes, I'm not too sure which old routes you are talking about since we are using the same route as before.

Sorry, by "routes" I meant the old stuff you mentioned. But yes, I think it's fine to remove, since we're aiming for a breaking change, not a safe migration (in which case we would have both old and new functions first, then migrate frontend to new, then delete old entirely – that way there would be no downtime).

@RichDom2185 I can remove the old stuff in assessments.ex and admin_grading_controller.ex if everything looks good. With regards to the routes, I'm not too sure which old routes you are talking about since we are using the same route as before.

Sorry, by "routes" I meant the old stuff you mentioned. But yes, I think it's fine to remove, since we're aiming for a breaking change, not a safe migration (in which case we would have both old and new functions first, then migrate frontend to new, then delete old entirely – that way there would be no downtime).

@RichDom2185 There is still the ungraded_only param which is unresolved, not too sure what it does so I will leave the TODO comment on top of the new function.

Resolves #926

@RichDom2185 could you look through 2e3be04, I wasn't sure if notGradedFully should fall under the submissions builder

@RichDom2185 could you look through 2e3be04, I wasn't sure if notGradedFully should fall under the submissions builder

Looks good!


UAT | E-filing | Submission of Case File

Module: E-filing

On clicking fill now, user is unable to understand which fields to fill
Expected behaviour: Post the pop up asking users to fill optional fields, if the user selects fill now, the optional fields should be highlighted

@Taherabharmal @anirudh-0 I think this might be too much dev effort for now (v1) for very little gain

Don't know how to tag as enhancement

I added a label


New Feature Request: Referrals

Referrals
https://techqueria.org/referrals
https://techqueria.org/refer

Should be public-facing in case allies want to let their Latinx friends know about it
Sends email to the friend inviting them to sign up
example: https://communitybot.co/?ref=producthunt

Hi, I'd like to work on this if it's still open. Can I get more information about this feature :D

@vtostado hey - yeah basically we just want an easy way for people to refer others to become a member to the techqueria community
i was thinking for a v1, it could just be a landing page at https://techqueria.org/refer on the benefits of being a member and then showing them where to sign up
in the meantime, i have https://techqueria.org/refer redirecting to https://techqueria.org/join


~ in polling mode is not triggered by IntelliJ IDEA saving files in macOS

steps

Create a barebones SBT project, e.g. using new SBT project in IntelliJ IDEA. Add a test source file, e.g.

object Test extends App {
  println ("Hello SBT!")
}

Start SBT and run any command prefixed with ~, e.g.
~run

Edit the file in IDEA and save it.

problem
SBT does not detect file changes and does not re-run the command.
expectation
SBT should detect file changes and re-run the command.
notes
Changes made by other editors (e.g. TextMate, nano) do trigger SBT 1.0.4!
SBT 0.13.16 triggers correctly regardless of what program modified the file.
SBT 1.0.4 triggers correctly in nio mode (via -Dsbt.watch.mode=nio) albeit with a delay.
SBT version: 1.0.0 - 1.0.4
macOS version: High Sierra 10.13.1
Java version: 1.8.0_151

See also https://github.com/sbt/io/issues/86

@mechkg Good catch!

Upstream https://github.com/sbt/io/issues/86

I referenced this in #3695 but, as a temporary fix, I added an sbt plugin that replaces the WatchService for OSX with one that uses the Apple File System Events API. https://github.com/swoval/MacOSXWatchService

Re-opening to track sbt/io needs releasing for inclusion in 1.1.0-RC2.

@dwijnand this seems to be specific to IDEA
happens to me on windows, with sbt 1.0.0-1.0.4, with both polling and io, doesn't happen if i eg edit the file with notepad or use IO.touch

the only difference i can see is that when IDEA saves file - directory last modified time is updated, but not when i use touch

New io with the fix was merged in #3816.
If the issue persists with 1.1.0-RC2, then we will reopen this case.

This issue was about macOS though, where it's using polling. If there are issues with NIO and/or IntelliJ, on Windows, it should be reported.

I struck this problem with IDEA saves not triggering my ~compile. So I found this and upgraded to latest RC, but now I instead get this issue that it doesn't seem to resolve a softlink for a git-hook:
sbt:root> ~compile
Exception in thread "Thread-11" java.io.FileNotFoundException: No such file or directory: /Users/viktor/dev/projects/[project]/.git/hooks/pre-commit
	at sbt.internal.io.MilliPosixBase.checkedIO(Milli.scala:97)
	at sbt.internal.io.MacMilli$.getModifiedTimeNative(Milli.scala:248)
	at sbt.internal.io.MacMilli$.getModifiedTimeNative(Milli.scala:239)
	at sbt.internal.io.MilliNative.getModifiedTime(Milli.scala:66)
	at sbt.internal.io.Milli$.getModifiedTime(Milli.scala:357)
	at sbt.io.IO$.getModifiedTime(IO.scala:1163)
	at sbt.io.PollingWatchService$PollingThread.$anonfun$getFileTimes$3(PollingWatchService.scala:96)
	at scala.collection.mutable.ResizableArray.foreach(ResizableArray.scala:59)
	at scala.collection.mutable.ResizableArray.foreach$(ResizableArray.scala:52)
	at scala.collection.mutable.ArrayBuffer.foreach(ArrayBuffer.scala:48)
	at sbt.io.PollingWatchService$PollingThread.$anonfun$getFileTimes$2(PollingWatchService.scala:96)
	at sbt.io.PollingWatchService$PollingThread.$anonfun$getFileTimes$2$adapted(PollingWatchService.scala:93)
	at scala.collection.mutable.ResizableArray.foreach(ResizableArray.scala:59)
	at scala.collection.mutable.ResizableArray.foreach$(ResizableArray.scala:52)
	at scala.collection.mutable.ArrayBuffer.foreach(ArrayBuffer.scala:48)
	at sbt.io.PollingWatchService$PollingThread.getFileTimes(PollingWatchService.scala:93)
	at sbt.io.PollingWatchService$PollingThread.populateEvents(PollingWatchService.scala:110)
	at sbt.io.PollingWatchService$PollingThread.run(PollingWatchService.scala:86)

It is however there as a softlink:
~/d/p/a/.git ❯❯❯ ll  /Users/viktor/dev/projects/[project]/.git/hooks/pre-commit
lrwxr-xr-x  1 viktor  staff    23B Mar  7  2017 /Users/viktor/dev/projects/[project]/.git/hooks/pre-commit@ -> ../../bin/pre-commit.sh
Could this be related to the fix? Dunno why it's trying to grab that file, I have the sbt git plugin sure, but…

That's https://github.com/sbt/sbt/issues/3820.


Generic Nav tree behaviour - one node open only

Can we modify the nav tree behaviour:
Among peer nodes only one node can be open at any time. So if a user clicks on a new node, the open peer nodes are forcefully closed.
For Example,
Only 1 district drop down can be fully expanded to 1 school at a time
If I click on another school in the same cluster, the previously open school should close.
It may help close some of the issues.

@meghaklp working on figuring out why multiple nodes being open should cause these issues. If this is a UX request, then we'll implement it right away. But if this is a workaround to avoiding some of the bugs we're having the nav tree, I'd rather get to the bottom of it rather than dance around it. Please clarify.

@subhaklp From a UX perspective, if the nav tree has more expanded nodes it becomes clunkier to operate - more scrolling, more seeking, etc
But if you want to get to the bottom of the issue first and then make UX fix, I'm fine with that approach.


Sui Cli Bug: URL scheme not supported

Steps to Reproduce Issue
➜  ~ sui client switch --gateway localhost:5001
Active gateway switched to localhost:5001
➜  ~ sui client gas --address 0xb72418cc549c097429bbdbabd6ec50b7a88a5899
Networking or low-level protocol error: Invalid Url: URL scheme not supported, expects 'http' or 'https'
➜  ~ sui client switch --gateway https://gateway.devnet.sui.io:443
Networking or low-level protocol error: Invalid Url: URL scheme not supported, expects 'http' or 'https'

Expected Result
CLI shouldn't switch to incorrect URL and should switch back to correct one
Actual Result
CLI prints error Invalid Url on any command
System Information

OS: OS X 12.4
CLI: sui 0.6.2

cc @patrickkuo

Resolved by deleting local state at ~/.sui, but bug still there, it shouldn't work like that

Ya that’s not right, thanks for reporting @0xrinegade !
we are deprecating the gateway and will be replacing it with the full node, I will make sure this is fixed when we do the switch over.


Deployment to Non-Scratch Org Failed

Describe the bug
I'm configuring the B2C CRM Sync in a Non-Scratch Org and while Im making the deployment it gives an exception that tells me to report on github.
To Reproduce
Steps to reproduce the behavior:

Run: npm run crm-sync:sf:org:deploy

Expected behavior
Should make the deployment of meta-data
Screenshots
If applicable, add screenshots to help explain your problem.

Your local environment details:

Node version (node -v): 15.2.1
NPM version (npm -v): 7.24.2s
SFDX version (sfdx -v): sfdx-cli/7.118.1 win32-x86 node-v14.17.6

Your remote environment details:

Salesforce B2C Commerce version: 21.3
Salesforce Core version: [e.g. 52]

Additional context
Add any other context about the problem here.

Hello @luismiguelferreiraPWC ,
Thank you for posting this issue.
Can you please run the sfdx force:source:deploy -p "./src/sfdc/base" command and give us the error rendered by SFDX?
Thanks in advance!
Jordane

Hi @jbachelet ,
Thanks for the quick response.
Errors:

Hey @jbachelet
Now I was able to make the deployment with your command, but when I do the command of the documentation I have a new error. Could I still continue with the doc? or this command should be working before I pass to the next chapter?
Error:

@jbachelet

Im getting this error while trying to Extract the Public Key from the KeyStore

Hello @luismiguelferreiraPWC ,
Regarding the issue with the authentication against your dev hub, are you sure you correctly authenticated against the devhub through sfdx?
Regarding the issue with the Keystore, we never had this issue before.
Anyway, the CLI commands basically extract the certificate and the key from the .jks file.
You could also do this manually with command-line commands (thanks to OpenSSL) instead of using our CLI commands.
I'm unable to reproduce both issues by the way, so really hard for me to provide guidance/help on these

@jbachelet
I'm not using scratch orgs so do I need devhub?
Regarding the keystore I've tried to follow the doc 3 times in different PCs and I always get the same error.
Could you present me the  command-line commands to pass this step?
If you want we could have a call. Probably would be more easy to explain.

@jbachelet
I've tried these commands:
Export the .der file
keytool -export -alias zzhb -file sample.der -keystore id.jks

Convert the .der file to unencrypted PEM (crt file)
openssl x509 -inform der -in sample.der -out sample.crt

Export the .p12 file
keytool -importkeystore -srckeystore id.jks -destkeystore zzhb.p12 -deststoretype PKCS12

Convert the .p12 file to unencrypted PEM (key file)
openssl pkcs12 -in sample.p12 -nodes -nocerts -out sample.key

but didn't work.
Could we have a call or provide the commands to make it work?


Setup initial E2E tests closes #2848

Closes #2848
Proposed Changes

setup E2E tests

I submit this contribution under Apache-2.0 license.

This is the PR that will introduce E2E testing to KICS. We're currently working to provide utility functions to make the tests easier to write
cc: @rubensequeiracx @cxlucas  @joaoReigota1 @nunoocx


Update cookieconsent.js

Making the use of _target="blank" safe. See https://www.owasp.org/index.php/Reverse_Tabnabbing

nofollow is not relevant. At least just noopener.


TODO: pull/update from NYT repo

Gonna explore this for a bit, checked out the file reading interface in the RawDataService here https://github.com/mchaffee1/covid-analyzer/blob/dev/Sources/App/Services/RawDataset.swift#L21-L26. Going to look at changing the implementation over to something that can accept an initial dataset from a file, and load it on the wire later.
I assume we eventually run this as a service and it will need a system to regularly update underlying data.


Add Chunked or Segmented file reference

Feature Request
It would be useful to have a type of MediaReference that refers to a contiguous clip that has been "chunked" or "segmented" out over multiple files.
Description
Much like an ImageSequenceReference(#69) there is a similar kind of single conceptual asset that can be spread over multiple files but rather than containing a single sample (frame) from that file, they contain a subset of the samples that make up the source. These are typically referred to as "chunked" or "segmented" files.
Some common example cases where these may come up include:

Camera files recorded by RED cameras in both R3D and mov formats
Files used in streaming in either fragmented mp4 or .ts files

In most cases, this is done to work around file size limitations. Many software packages (like Nuke) have the user point to the first file in the sequence and then implicitly discover the other files in the sequence. It would be nice for OpenTimelineIO to represent this more explicitly to aid in detecting cases where perhaps the last file in a sequence is missing and the clip may be truncated as a result.
#630 is related to this, but it's a bit more targeted at bundling together alternate media references, this would be focussed on media on-disk that is meant to be concatenated to form the full contiguous source.

Support for these chunked files would be very useful for archiving/packaging timelines with the otioz adapter for instance.
I'll copy one of my initial ideas from slack discussion in here for future reference.

Each chunk or R3D file for instance would have it's own media ref and they all wrap together in a "container ref". That container ref can then tackle providing the correct file segment corresponding to time. Similar to ImageSequenceReference.target_url_for_image_number()


Test to ensure that RCOMBBASE instruction can be executed and proven

Add a test which ensures that `RCOMBBASE` instruction can be executed and proven.

Originally posted by @bobbinth in https://github.com/0xPolygonMiden/miden-vm/pull/1227#pullrequestreview-1857638288

Closed by #1232.


What is equivalent of "X-Forwarded-For" in GRPC on Cloud Run?

How to get client IP when using GRPC?
For example :
"X-Forwarded-For" header can be read to get client IP when using HTTP, Cloud Run HTTP proxy injects it.

It’s the same. Grpc is still http/2. But calls headers “metadata”.

Do you know if there are any equivalent headers to GAE headers such as X-AppEngine-Country, X-AppEngine-Region or X-AppEngine-City?

From what I have tested it only injects (currently) these X- prefixed Headers :
X-Cloud-Trace-Context
X-Forwarded-For
X-Forwarded-Proto


Update Vagrant to 2.0.1, unbork bundler.

Hi,
The current setup complains about two little things.

the protocol being insecure;
and the gem being set up twice.

The git source `git://github.com/mitchellh/vagrant.git` uses the `git` protocol, which transmits data without encryption. Disable this warning with `bundle config git.allow_insecure true`, or switch to the `https` protocol to keep your data secure.
Your Gemfile lists the gem vagrant-cloudstack (>= 0) more than once.
You should probably keep only one of them.
While it's not a problem now, it could cause errors if you change the version of one of them later.

[!] There was an error parsing `Gemfile`: You cannot specify the same gem twice coming from different sources.
You specified that vagrant-cloudstack (>= 0) should come from source at `.` and source at `.`
. Bundler cannot continue.

 #  from /home/yoan/soft/vagrant-cloudstack/Gemfile:19
 #  -------------------------------------------
 #  group :plugins do
 >    gem 'vagrant-cloudstack', path: '.'
 #  end
 #  -------------------------------------------

I've also upgraded Vagrant to 2.0.x and managed to start/ssh/destroy a machine with it.
Best,

Coverage remained the same at 88.293% when pulling 6d5363bac130af7900f9e427f9c57aa58089f3a2 on exoscale:vagrant2 into 9704968fb3eb3bc90984b642e0788240f9e00511 on MissionCriticalCloud:master.


Change descriptions of some registry keys

Change the description for ConservationIdleTime and PerformanceIdleTime to better reflect when they are used.

Thanks again Daniel!  We appreciate the PR.


webView is not dismiss upon cancelling (decline) reddit access

Hello!
I was testing the login changes recently pushed. I found this little issue. When reddit asks to accept or deny the authorization, upon choosing 'decline' the webView does not go away.

It redirects you to the page shown on your screenshot when you press "Decilne"? (before the fix)
For me it still stays on the permission prompt and allows me to click "Allow" instead.

It redirects you to the page shown on your screenshot when you press "Decilne"? (before the fix)

Yes. It loads that page. After the fix the login window closes and the toast "you must press allow..." comes up.


404 error while trying to login via Live (Microsoft) account.

HI,
When i try to login using Skype account, its working fine
But when i try to login using Live (Microsoft) account, its throwing 404 error
('404 response from PUT https://client-s.gateway.messenger.live.com/v1/users/ME/endpoints/5434e4d6-34f7-9d2c-783d-8dbc04cfe714/presenceDocs/messagingService', <Response [404]>)
Account type

Conversation details

Steps to reproduce
Just trying to login
from skpy import Skype
from getpass import getpass
<EMAIL_ADDRESS>getpass())

Result or traceback
Password:
Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/skpy/main.py", line 67, in init
self.conn.readToken()
File "/usr/local/lib/python3.6/site-packages/skpy/conn.py", line 284, in readToken
raise SkypeAuthException("No token file specified")
skpy.core.SkypeAuthException: No token file specified
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "test.py", line 5, in
<EMAIL_ADDRESS>getpass())
File "/usr/local/lib/python3.6/site-packages/skpy/main.py", line 69, in init
self.conn.getSkypeToken()
File "/usr/local/lib/python3.6/site-packages/skpy/conn.py", line 262, in getSkypeToken
self.liveLogin(user, pwd)
File "/usr/local/lib/python3.6/site-packages/skpy/conn.py", line 361, in liveLogin
self.getRegToken()
File "/usr/local/lib/python3.6/site-packages/skpy/conn.py", line 421, in getRegToken
endpoint.config()
File "/usr/local/lib/python3.6/site-packages/skpy/conn.py", line 864, in config
"version": "908/<IP_ADDRESS>"}})
File "/usr/local/lib/python3.6/site-packages/skpy/conn.py", line 215, in call
raise SkypeApiException("{0} response from {1} {2}".format(resp.status_code, method, url), resp)
skpy.core.SkypeApiException: ('404 response from PUT https://client-s.gateway.messenger.live.com/v1/users/ME/endpoints/5434e4d6-34f7-9d2c-783d-8dbc04cfe714/presenceDocs/messagingService', <Response [404]>)

Duplicate of #47 -- I have yet to see this on my end; it appears to be a temporary issue with Skype's servers.
If you can reliably reproduce this, please compare the failing request with the corresponding one made by Skype for Web in dev tools, it might be doing something differently to handle this case.

HI,
I did a comparison,After the below post
<= [10/04 11:28:24] POST https://client-s.gateway.messenger.live.com/v1/users/ME/endpoints
{'json': {'endpointFeatures': 'Agent'}}
we are getting 200 for live account
=> [10/04 11:28:27] 200
[{'id': '12de317a-4c98-4825-97a0-fbc75c2dd2db',
'isActive': False,
'networks': [],
'policies': {},
'subscriptions': [{'channelType': 'HttpLongPoll',
'conversationType': 0,
'eventChannel': '0',
'eventServiceName': '',
'id': 0,
'interestedResources': ['/v1/threads/ALL',
'/v1/users/ME/conversations/ALL/messages',
'/v1/users/ME/conversations/ALL/properties',
'/v1/users/ME/contacts/ALL'],
'packageSid': '',
'template': '',
'type': 'Subscription',
'xuid': ''}],
'type': 'Endpoint'},
{'id': '5434e4d6-34f7-9d2c-783d-8dbc04cfe714',
'isActive': True,
'networks': [],
'policies': {},
'subscriptions': [{'channelType': 'TcpSocket',
'conversationType': 0,
'eventChannel': '',
'eventServiceName': '',
'id': 0,
'interestedResources': ['/abch',
'/v1/threads/ALL',
'/v1/users/ME/conversations/ALL/messages',
'/v1/users/ME/conversations/ALL/properties',
'/v1/users/ME/contacts/ALL'],
'packageSid': '',
'template': '',
'type': 'Subscription',
'xuid': ''}],
'type': 'Endpoint'}]
But for skype account we are getting 201
[10/04 11:47:44] 201
End point url for skpe account is different
= [10/04 11:47:44] PUT **https://co4-client-**s.gateway.messenger.live.com/v1/users/ME/endpoints/{575b348f-f468-40b8-87de-f100b649501b}/presenceDocs/messagingService
{'json': {'id': 'messagingService',
'privateInfo': {'epname': 'skype'},
'publicInfo': {'capabilities': '',
'nodeInfo': 'xx',
'skypeNameVersion': 'skype.com',
'type': 1,
'version': '908/<IP_ADDRESS>'},
'selfLink': 'uri',
'type': 'EndpointPresenceDoc'}}
for live account endpoint it is
[10/04 11:28:27] PUT https://client-s.gateway.messenger.live.com/v1/users/ME/endpoints/12de317a-4c98-4825-97a0-fbc75c2dd2db/presenceDocs/messagingService
{'json': {'id': 'messagingService',
'privateInfo': {'epname': 'skype'},
'publicInfo': {'capabilities': '',
'nodeInfo': 'xx',
'skypeNameVersion': 'skype.com',
'type': 1,
'version': '908/<IP_ADDRESS>'},
'selfLink': 'uri',
'type': 'EndpointPresenceDoc'}}
and its returning 404

Skype uses multiple "clouds", with different users directed to different host domains.  The Live account appears not to get redirected and uses the base domain (which afaict is a valid case).

...please compare the failing request with the corresponding one made by Skype for Web in dev tools...

Does Skype for Web use the base domain too?

Hi.
I have the same problem.

Does Skype for Web use the base domain too?
No. In my case Skype use db5-client-s.gateway.messenger.live.com If i change API_MSGSHOST to db5-client-s.gateway.messenger.live.com then everything works fine. Any ideas how to get the correct URL?

Just tried it again with my test Live account:
>>> sk =<EMAIL_ADDRESS>getpass())
Password:
>>> sk
Skype(userId='live:fred_2')
>>> sk.conn.msgsHost
'https://db5-client-s.gateway.messenger.live.com/v1'

So it looks like alternate hosts are working in some cases.
@AshotS: Can you tell from Skype from Web, how it might be deciding to switch hostnames?  Take a look at the last request made to the base domain, it'll likely be reading something out of the Location header or JSON response.  The relevant SkPy code for detecting the hostname can be found here.
I'm wondering if there's an endpoint from a different domain, then you're supposed to follow that...

@OllieTerrance Your code works correctly. Now I tried to find a problem but everything works. It is very strange.

That's what everyone's said, it always seems to be temporary...  😞

I think I found the problem. https://ibb.co/jwVyWQ When I added to the headers "BehaviorOverride": "redirectAs404"  "Location" appeared in response.
{'Cache-Control': 'no-store, must-revalidate, no-cache',  'Content-Length': '106',  'Content-Type': 'application/json; charset=utf-8',  'ContextId': 'tcid=3024622139195838359,server=CO2SCH020011330',  'Date': 'Thu, 20 Apr 2017 13:46:14 GMT',  'Location': 'https://db5-client-s.gateway.messenger.live.com/v1/users/ME/endpoints',  'Pragma': 'no-cache',  'Set-RegistrationToken': 'registrationToken=XBZMkYwYVc5dVVtVmhaRTl1YkhrPTs=; '                           'expires=1492782373; '                           'endpointId={6317ef51-1d84-413e-8185-9be34f0d3cae}'} {'errorCode': 752,  'message': "User is in a different cloud. See 'Location' header for users "             'current cloud.'}

I've added the BehaviorOverride header -- I've yet to come across this issue, and coupled with its temporary nature, it's a bit tricky to test!  Explicitly testing for the error code shouldn't be necessary as we're always checking the Location header?

Explicitly testing for the error code shouldn't be necessary as we're always checking the Location header

Verification is needed in order to send a repeated request to the URL received from Location. At least in Web Skype is implemented in this way. If response return 404 client get URL from Location and post request again in new URL. Otherwise there is the exact same error.

I'm inclined not to check for server error codes when, due to being a private, undocumented API, they're liable to change or mean different things.  But it should now be the case that if we get a different hostname back, we always re-request until it sticks.

Yes, this idea is much better. Thank you.

Hopefully that'll cover it.  Still unsure why it only happens temporarily...
I'll get it pushed out on PyPI shortly as this has affected quite a few users now.  If we still get this cropping up, I'll reopen for further investigation.


feat: support slotted poster img (e.g. next/image)

https://github.com/muxinc/next-video/discussions/146

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 59.01%. Comparing base (72dcc92) to head (96886d8).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #189      +/-   ##
==========================================
+ Coverage   58.84%   59.01%   +0.17%
==========================================
  Files          30       30
  Lines        2347     2357      +10
  Branches      123      123
==========================================
+ Hits         1381     1391      +10
  Misses        961      961
  Partials        5        5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Bulk data format changed

In the IO based consumption module (eia_io_trading.py), there is a data download from the EIA to capture all of the trading data in one file. At some point between v1.0.1 and now, EIA has changed the date/time format in that file, resulting in a runtime error from line 149.
ValueError: time data '2016-01-01 00:00:00+00:00' does not match format '%Y-%m-%d %H:%M:%S%z'

So it appears that this used to be an issue with strptime not supporting the ":" in the timezone adjustment "+00:00". Apparently there was a fix in python 3.7 for this, so it's gone away. I'm leaning towards closing this with the expectation that updates to the requirements and setup files will ensure that this won't happen in the future.

I believe 3.6 is about to lose all support so might be good to update the python requirements across the board anyway


@connect_ib problem

What does this @connect_ib does?  I know it is called before the function, but it looks like the function is not called at all even after a successful connect.
In my case, during retireve_data_online, it calls retrieve_data_ib which has the @connect_id decorator, after it call ib_global["connected"] is true, which should mean IB is connected and the IBKR_gateway confirmed it is connected, but the retireve_data_ib never get called, and it  give this following message
INFO 63 IB retrieval of symbol failed, fallback on YF
AMD: Period '3y' is invalid, must be one of ['1d', '5d', '1mo', '3mo', '6mo', '1y', '2y', '5y', '10y', 'ytd', 'max']

Okay, found out what the problem is by adding
except Exception as error:
1/ Multiple arguments for it_is_index: that's due to the used_api being added to the function retrieve_data_ib.  The first time it's working because both it_is_index and use_api is false.  Second time use_api is true while it_is_index is false.  Remove used_api argument solve that.
2/  Unexpected 'client' keyword,  remove kwargs['client'] resolve problem
  def wrapper(*args,**kwargs):
       #kwargs['client'] = IBData.resolve_client(None)
       IBData.resolve_client(None)
       return func(*args,**kwargs)
   return wrapper

3/ now I got ERROR 159 IBData.init() got an unexpected keyword argument 'single_key'
Looks like there are multiple problem, so further troubleshoot is required.

I will try to understand.


issue : Include of non-modular header inside framework module

No news about this issue : #https://github.com/googleads/googleads-ima-ios/issues/3  ? i have the same issue :s
IDE : Xcode 7.3
Cocoapods 0.39

This should be fixed with the most recent iOS release, 3.2.0. If you're still seeing the issue there let us know!

It's working again! Thanks

With : pod 'GoogleAds-IMA-iOS-SDK', '~> 3.2.0'
`duplicate symbol OBJC_CLASS$_GAD_GTMStringEncoding in:
/Users/.../Pods/GoogleAds-IMA-iOS-SDK/GoogleInteractiveMediaAds/GoogleInteractiveMediaAds-GoogleIMA3/GoogleInteractiveMediaAds.framework/GoogleInteractiveMediaAds(GAD_GTMStringEncoding.o)
/Users/.../Pods/Google-Mobile-Ads-SDK/GoogleMobileAdsSdkiOS-7.7.1/GoogleMobileAds.framework/GoogleMobileAds(flat-x86_64)
duplicate symbol OBJC_METACLASS$_GAD_GTMStringEncoding in:
ld: 2 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
`
With :   pod 'GoogleAds-IMA-iOS-SDK-For-AdMob', '~> 3.2.0'
All it's ok !! Thx you for your work guys :) 💯 🚀


[WIP] ekf run output predictor immediately with new IMU data

Nice!

@LorenzMeier This arose from a a discussion between myself and @dagar  yesterday and will help clear the path to use of multiple EKF instances without a significant time delay penalty.

@priseborough I thought so. It was too directed to not be based on your guidance 8)

Here is the replayed output tracking error using the  ecl version from master:

Here it is using the version from this PR:

A general observation is that there is a lot of change in this PR when the only functional addition required for immediate update of the output attitude quaternion is addition of the calculate_quaternion() function

Commit https://github.com/PX4/ecl/pull/471/commits/aaff047e5852761f8d15bc695c7a1ca70beeeeb1 has fixed the difference:
This PR at commit aaff047e5852761f8d15bc695c7a1ca70beeeeb1 :

Master:

The CI test is a bit bogus and calls update() multiple times without new IMU data.
It's now failing because update has been changed to returns true or false based on actually updating or not. Previously the return was based on tilt_align, yaw_align, and finite quaternion.

Does this generally reduce latency on the attitude output?

@mhkabir yes it does if used in combination with https://github.com/PX4/Firmware/pull/9832 to take advantage of the calculate_quaternion() function which can run immediately after setIMUData(...)


[JustWatch Search] ...

Extension
https://www.raycast.com/rishabswift/search-justwatch
Description
Hi Raycast Team: is it impossible to change the JustWatch extension so after the URL of "justwatch.com/ca" it can open as "justwatch.com/us" because the "ca" in the Raycast extension I believe is Canada though I'm hoping to open the USA location? Thank you. - Dave
Who will benefit from this feature?
Everyone who uses the popular JustWatch website in the USA.
Anything else?
Thank you for that.

Hey,
You can already switch the country, in which case it will open on whatever country you set.

I totally missed that. Gulp. Thank you for that.
On December 22, 2022, raycastbot @.***> wrote:

Hey,
You can already switch the country, in which case it will open on
whatever country you set.

—
Reply to this email directly, view it on GitHub
<https://github.com/raycast/extensions/issues/4009#issuecomment-
1363337474>, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/A45FDKYTXCBILOXZIUJNM63WOS6ZRANCNFSM6AAAAAATHDUA5I>.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Sent with HEY https://hey.com/sent — Email as it should be

@raycastbot close this issue


feat: Move use case CreateCandidate to new profile

Description

moves use case CreateCandidate to new profile recruitment/create-candidate
subsequently removes profile recruitment/candidates

Motivation and Context
Discussion in this comment, main reason was different support from providers.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I haven't repeated the code. (DRY)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Should I keep this PR open until we deprecate recruitment/candidates profile?

Should I keep this PR open until we deprecate recruitment/candidates profile?

Catalog and registry do not support deprecating of profiles yet. We can update recruitment/candidates profile description. It can be part of this PR.


Fix incorrect links in doc. Doc change only!

@michael-tims - Please review and merge. Thank you.

Good point. Fixed.


Add support for docker >= 1.10

Fixes https://github.com/twitter-fabric/galley/issues/40

Finished pass. #opensource

👋 @finneganh

@finneganh what do you think about this?

👍 let's squash and release!


XSS vulnerability for hyperlinks

This project has the same vulnerability as described at pptx2html.
Further information:

https://github.com/g21589/PPTX2HTML/issues/7
https://dan.lousqui.fr/xss-vulnerability-report-for-pptx2html-en.html

fixed in ver. 1.10.4

fixed in ver. 1.10.4


Using Geolocator from service with no activity

I currently use the Geolocator from within a service. Everything works fine as long as the activity that started the service is running or stopped. However, if I swipe the activity away using the app switcher, the activity is destroyed and the Geolocator no longer works from within the service. I'm guessing this is because the activity is being used as the context object within the locator. When the activity is swiped away the context is no longer valid and the Geolocator doesn't work from the service. If I start the activity back up the service is again able to use the Geolocator.
Previously, I was using Xamarin.Mobile and the Geolocator worked fine when initialized with Application.Context from within the service. Is there a reason why this context can't be used within the CrossGeolocator? Is there any other way to get the Geolocator to work from a service with no activity?

Yes, this should still work fine as the Current Activity is only used to check permissions, and if it is in a service it uses the ApplicationContext.
https://github.com/jamesmontemagno/Xamarin.Plugins/blob/master/Geolocator/Geolocator/Geolocator.Plugin.Android/GeolocatorImplementation.cs#L41
I create the manager with the application context as you can see.
You might be on an android system that is killing your android service when you kill the activity and you would have to re-launch the service: see how i did it here: https://github.com/jamesmontemagno/My-StepCounter/blob/master/StepCounter.Android/Services/StepService.cs#L58-L100

Ah, yes you are right.
The update to Plugin.Permissions 1.1.6 seems to have fixed it.
Thanks for your quick response (and all your awesome code).

:) Yeah I had an issues in the permissions plugin that I wasn't checking for a null current activity.. :( so that was the problem for you. I am going to publish new versions next week with some updates and with new dependencies.


AttributeError: unreadable attribute

Hello,
I'm getting this error (full traceback)
Traceback (most recent call last):
  File "main_wasmtime.py", line 4, in <module>
    config.wasm_multi_memory(True)
AttributeError: unreadable attribute

when I run the following code:
from wasmtime import Config

config = Config()
config.wasm_multi_memory(True)
config.wasm_threads(True)

I need the threads enabled for a wasm file I'm working with, however it doesn't look like it can be set using the wasmtime-py API correctly.
My environment is:

Python 3.9.7
Wasmtime 0.39.1
Ubuntu 20.04 (Linux e127538 5.13.0-44-generic 49~20.04.1-Ubuntu SMP Wed May 18 18:44:28 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux)

Am unsure how to proceed as I can't think of a workaround.

I believe these are properties, so does config.wasm_multi_memory = True work instead?

That worked! Thanks very much!


multiple meters - wrong data for some parameters

Per your request in issue #4 , I am posting the debug data json file (sensitive data removed). I am diffing against the old web site data (cp.city-mind.com)
I can see the following:

All 3 meters that I have are identified correctly: meter id, names, readouts.
The "consumption-daily" has 2 values for each meter which represents yesterday's and today's consumption. Unfortunately these 2 values are identical to all 3 meters which is wrong. Looking at the website data, these numbers don't belong to any of the 3 meters :-)
"Consumption forcast" fields seems OK
"consumption-monthly" shows same val for all 3 meters. Also what is the diff between "consumption forcast" and "consumption-monthley" ?
customer-service fields looks OK
I think all the numbers are a bit off (few %). Again, I am comparing to the data posted in the old API. Maybe it it a bit different. Let's deal with it later.

{
  "version": 1,
  "minor_version": 1,
  "key": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
  "data": {
    "consumption-daily": {
      "192": [
        {
          "commonCons": 0.0,
          "cons": 3.2269999999999186,
          "consDate": "2022-10-16T00:00:00",
          "estimationType": 0,
          "meterCount": 0,
          "meterStatusDesc": ""
        },
        {
          "commonCons": 0.0,
          "cons": 0.6879999999999313,
          "consDate": "2022-10-17T00:00:00",
          "estimationType": 0,
          "meterCount": 0,
          "meterStatusDesc": ""
        }
      ],
      "193": [
        {
          "commonCons": 0.0,
          "cons": 3.2269999999999186,
          "consDate": "2022-10-16T00:00:00",
          "estimationType": 0,
          "meterCount": 0,
          "meterStatusDesc": ""
        },
        {
          "commonCons": 0.0,
          "cons": 0.6879999999999313,
          "consDate": "2022-10-17T00:00:00",
          "estimationType": 0,
          "meterCount": 0,
          "meterStatusDesc": ""
        }
      ],
      "194": [
        {
          "commonCons": 0.0,
          "cons": 3.2269999999999186,
          "consDate": "2022-10-16T00:00:00",
          "estimationType": 0,
          "meterCount": 0,
          "meterStatusDesc": ""
        },
        {
          "commonCons": 0.0,
          "cons": 0.6879999999999313,
          "consDate": "2022-10-17T00:00:00",
          "estimationType": 0,
          "meterCount": 0,
          "meterStatusDesc": ""
        }
      ]
    },
    "consumption-forcast": {
      "192": {
        "estimatedConsumption": 38.307764705882136
      },
      "193": {
        "estimatedConsumption": 3.184941176470602
      },
      "194": {
        "estimatedConsumption": 41.38352941176452
      }
    },
    "consumption-monthly": {
      "192": [
        {
          "cons": 44.02799999999979,
          "consDate": "2022-10-01T00:00:00"
        }
      ],
      "193": [
        {
          "cons": 44.02799999999979,
          "consDate": "2022-10-01T00:00:00"
        }
      ],
      "194": [
        {
          "cons": 44.02799999999979,
          "consDate": "2022-10-01T00:00:00"
        }
      ]
    },
    "customer-service": {
      "Email": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
      "description": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
      "municipalID": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
      "phoneNumber": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx"
    },
    "last-read": [
      {
        "meterCount": 192,
        "meterId": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
        "read": 1365.963
      },
      {
        "meterCount": 193,
        "meterId": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
        "read": 386.666
      },
      {
        "meterCount": 194,
        "meterId": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
        "read": 1824.072
      }
    ],
    "last-update": "2022-10-17 18:09:11.060463",
    "me": {
      "accountNumber": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
      "firstName": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
      "lastName": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
      "municipalId": xxxxxxxxxxxxxxxxxxxxxxxxxxxx,
      "phoneNumber": {
        "AdditionalPhoneNumber": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
        "countryCode": null,
        "phoneNumber": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx"
      }
    },
    "meters": [
      {
        "fullAddress": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
        "meterCount": 192,
        "meterSn": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx"
      },
      {
        "fullAddress": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
        "meterCount": 193,
        "meterSn": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx"
      },
      {
        "fullAddress": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
        "meterCount": 194,
        "meterSn": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx"
      }
    ],
    "my-alerts": [],
    "my-messages": [],
    "settings": [
      {
        "alertTypeId": 23,
        "mediaTypeId": 1
      }
    ],
    "token": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx",
    "vacations": []
  }
}

since I have just 1 meter on my account, I can't reproduce it and find how it should work,
there are 2 options to make it work, over online call or you assistint with developer tools at your browser extracting the requests and responses of the portal,
please let me know how do you prefer to promote it
thanks

Sorry for the late reply. I was quite busy at work.
I'd like to do the debug on my own if that's ok with you, but I'll probably need help so let's do an online call where you can show me what to look for in my browser.
I don't think there's a way to send PMs in Github, so I'll send you my contact info through @maorcc

Hi,
Can you please open the portal while developer yools is opened on network tab?
Once you have done it, make sure the preserve checkbox is checked,
Then perform a login,
A list of web api / pages will appear in that tab of network,
I need to understand what is the data being send and return for the first page (request url / response json),
Once you have it I can use that information to understand if we need add additional requests to the integration.
Thanks,
Elad

Created a discord channel for better communication
https://discord.gg/Nwv9QmAs

Ah..I think I now understand what's going on:
When you log in, you get the list of all meters (unique id (integer) and name (string)). You also get the last readout.
This is done through this URL:
https://api-ctm.city-mind.com/consumption/monthly/2022-10/2022-10-31
To get more details on a specific meter you need to choose the meter from a drop down menu

In the menu all meters appears. I have 3 so there are 4 items to choose from:

Once I choose a meter, I see the above URL with the addition of the chosen meter
https://api-ctm.city-mind.com/consumption/monthly/<METER_NUMBER>/2022-10/2022-10-31
where meter number is any of the integers returned earlier in the "meters" field when I logged in.
mine are numbered 192, 193 and 194.
Hope that helps

So I guess dedicated GETs should be performed for every meter

Ok, that's make perfect sense and exactly what i was looking for, to understand how the parameter is being passed for each of the calls,
Can I send you a postman project to verify all calls are working correctly?
Thanks

@arnonsha thanks for the details, it assisted understanding the new API much better,
please check if v2.0.3 works for you.
thanks

Thank you @elad-bar
I updated and will check it in the coming days.

Hi
I can confirm that all numbers for the individual meters are correct. That is great and makes this add on usable for anyone with multiple meters. Thank you very much !
2 improvements that may help:

As we have discussed above, the rym-pro.com landing page returns the readouts/consumption forecast for all meters combined.  Currently I don't use to this info, however someone else might. I suggest to have this info as dedicated device. So or n  meters there would be n+2 devices: n "water meter" devices,  1 "account" device, and 1 "all meters" device. What do you think ?

Not sure if this is a homeassistant issue or the addon issue:
I am trying to rename the meter device and give them a meaningful name, rather than stick with the meter serial number:

The new device name is kept some time for some time (not sure if it's hours or days),after which it is overrun by the original name with the serial number. Is that the add on or homeassistant responsible for this behavior ?

Great news, thanks for the update!
Regarding 1 - the component should provide the capability not aggregation,
Aggregation can be performed in more suitable for that purpose like Grafana.
For now there is a device for the account and for each meter,
If you would like to have aggregation, you can use aggregation template sensors or Grafana ad I mentioned above (which I'm using).
2 sounds like an issue, will check it, please report on it as different issue while this one should be closed.
Thanks

OK, thanks for explaining that.
I will close this one and open a new one for #2 above


Add E2E tests for PSP

This adds E2E tests for PSP beacon updates.

Do we need tests for multiple subscriptions? I might need help with setting those up.

I guess that would be nice. What's the problem with that?

@acenolaza @Siegrift Can you take a look at this as well?

Oh, one thing I forgot, can you update README with instructions on how to run the e2e tests?

I've added tests for retries, invalid subscriptions, a second valid subscription, updated the README, checking updated values, and fixed a problem with hardhat. I think all PR improvements are now covered so please feel free to review again :smile:

Is this still waiting for something?

Is this still waiting for something?

Only @acenolaza's final approval :smile:


Feature to change lang on landing page

Done at https://github.com/MarckFp/ppoc-gen/commit/c1ec8fa0be66a6c7df0951355d185f81fd3ff5d9


WebSocket request to an EVM node fails with "Frame size of X bytes exceeds maximum accepted frame size"

Tried to increase the quota:


Wrong bit mask specification for HRESULTs

On chapter MS-ERREF 2.1[1] the bits are ordered from 0 to 31, but it should from 31 to 0, similar to [2]
[1]  https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-erref/0642cb2f-2075-4469-918c-4441e69c548a
[2] https://docs.microsoft.com/en-us/windows/desktop/com/structure-of-com-error-codes

@champagnie @Krishaug Could either of you please take a look at this issue?

The numbering of bits is documented in [MS-DTYP] section 2.1 Common Base Types.  Perhaps there should just be a normative reference from [MS-ERREF] to that.


python3: can't open file 'train.py': [Errno 2] No such file or directory

python train.py --logtostderr --train_dir=training/ --pipeline_config_path=training/faster_rcnn_inception_v2_pets.config
train.py not exist

where are you running train.py from? Also did you copy the file over from /object_detection/legacy folder?

can You read readme?
6. Run the Training
UPDATE 9/26/18: As of version 1.9[...]
python train.py --logtostderr --train_dir=training/ --pipeline_config_path=training/faster_rcnn_inception_v2_pets.config
and look at repository train.py not exist

"UPDATE 9/26/18: As of version 1.9, TensorFlow has deprecated the "train.py" file and replaced it with "model_main.py" file. I haven't been able to get model_main.py to work correctly yet (I run in to errors related to pycocotools). Fortunately, the train.py file is still available in the /object_detection/legacy folder. Simply move train.py from /object_detection/legacy into the /object_detection folder and then continue following the steps below."
This has been a well known issue for many and is clearly highlighted in the Readme.
Can you read readme?

this are not clear

raining': [Errno 2] No such file or directory

as he said @aidoshacks , noone reads the readme , just replace "python train.py" to "python model_main.py"

If this problem is on colab
try changing from
=>drive.mount('/content/gdrive')
%cd '/content/gdrive/'
to particular path where the train.py is present
=> drive.mount('content/gdrive')
%cd '/content/gdrive/My Drive/models/research/object_detection/'

please help me
I tried to make yolov5 in google colab
https://colab.research.google.com/github/roboflow-ai/yolov5-custom-training-tutorial/blob/main/yolov5-custom-training.ipynb#scrollTo=eaFNnxLJbq4J
when I write
!python legacy/train.py --img 416 --batch 16 --epochs 150 --data /content/gdrive/My Drive/dataset/data.yaml --weights yolov5s.pt --cache
it makes error
python3: can't open file 'legacy/train.py': [Errno 2] No such file or directory

hi,  I'm trying tosolve this problem on google colab but i don't find solution please help me

 hi, I'm trying tosolve this problem on google colab but i don't find solution please help me

you forget install requirments,at the top


task.schema.json: add satisfies property;  add one more task category

Add some missed features of tasks to schema.

/azp run

reopened here to make CI checks passed: #1066


Feature: Added support for custom timestampable fieldnames

@TomasVotruba  Do you need any further changes, so this can be merged? For SoftDetelte I would create a separate MR, once this is merged.

Closing as not adjusted to avoid BC breaks

@TomasVotruba Sorry, I totally forgot I still had this one open. However, I currently don't have time to fix it. If I find the time, I will create a new PR and add in the fixes regarding BC.

No problem, I forgot about it too :D thanks for reaching out! Any furute PR is of course welcomed :+1:


Allow rf_crs to accept StringType.

Convert it with LazyCRS.

We can make a gt 2.x release that depends on a new Proj4j version, would it help you?

@pomadchin Not yet, as what I really want to do is just upgrade to 3.0 as soon as it's ready. Rather contend with all those things then.


fix: Use SVGs for some stacked delims

This PR uses a single SVG to render certain stretchy delimiters that have until now been rendered using a stack of font glyphs. The browser's imprecision in span placement is thus evaded.
The delimiters thus treated are: \lparen, \rparen, \brack, \rbrack, \vert, \Vert, \lfloor, \rfloor, \lceil, and \rceil, along with all the other functions that call the same stretchy delimiters.
Resolves issues #3650 and #3168.

I'll fix the screenshotter pics tomorrow.

@edemaine This PR is now ready for review. Since it is a purely visual effort, the best way to review it may be by looking at the  screenshotter diffs for commit https://github.com/KaTeX/KaTeX/pull/3686/commits/ae407cf39ddf3e5b043e6365f707f0c06ee774b1. The final commit just used those images to replace the earlier ones, so it is checking against itself and won't tell you anything.

@ronkok This sounds awesome! Thanks for working on this long-standing issue.
Am I reading the screenshot diffs correctly that | now has butt line caps, whereas before they were round line caps? It seems like in LaTeX they are supposed to be round line caps:

[QuickLaTeX rendering]
(Admittedly a very small difference!)

Re: butt line caps. You are correct. That's just sloth on my part. I can take a few minutes and create a radius.

Line caps are rounded. Screenshots are refreshed. This PR should be ready to go.

This is a friendly reminder that this PR is ready for review and it resolves a long-standing issue.

Sorry, I haven't forgotten; it's just been a busy time. I will try to review next week, but if not, should definitely have time the following week.

Thank you for the PR! I wonder whether we can get rid of these glyphs.

Thanks @ylemkimon!!
If you have additional time, it'd be great to get your PR #3385 revised, as we recently had a request for \ifmmode, which would be easy to add along with \if@display. (If you don't have time, though, I'll revise it eventually.)


Issue with k8s.io/docs/concepts/services-networking/ingress/

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
When explaining the contents of YAML using line numbers (e.g.  Under Ingress Resource POST YAML explanation in https://kubernetes.io/docs/concepts/services-networking/ingress/), without the YAML being annotated with line numbers, it is difficult to correlate between the text and YAML file
Proposed Solution:
Annotate YAML file display with line numbers.
Page to Update:
https://kubernetes.io/docs/concepts/services-networking/ingress/
Though this comment is applicable to all pages.

1.7

@steveperry-53 👋


Alpine Linux v3.13

https://alpinelinux.org/posts/Alpine-3.13.0-released.html

Thx, looks good :+1:

Thx, looks good :+1:


slim3 in subdirectory confuses Specfinder because of missing leading slash

When running slim3 in a subdirectory, the behaviour of getPath is to remove the leading slashas explained here:
https://github.com/slimphp/Slim/issues/1724
This causes issues with specfinder which tries to match a path with leading / with one without.
Spec: v3/teams spec: #/v3/teams/?$#
Also the openapi spec says leading slashes in paths is mandatory.
Not too sure how this should be solved.

Not sure that I've got the problem. Validator accepts requests with invalid paths? Could you please provide schema and request example?

My project structure:
web/
      openapi/ <--- definition is here
      api/  <---slim is here
      webapp/ <--- stuff

openapi.json
  "paths": {
    "/v3/teams": {
        "get": {
            "tags": ["Teams"],
            "summary": "Get the list of all the teams",
            "description": "Get the list of all the teams",
            "operationId": "teamActions::getAll",
            "responses": {
                "200": {
                    "description": "OK",
                    "content": {
                        "application/json": {
                            "schema": {
                                "type": "array",
                                "items": {"$ref": "#/components/schemas/Team"}
                            }
                        }
                    }
                }
            }
        }
    }
 },
  "servers": [
    {
      "url": "https://localhost:4443/api",
      "description": "Local server"
    },

This causes:
Type: League\\OpenAPIValidation\\PSR7\\Exception\\NoOperation
Message: OpenAPI spec contains no such operation [v3/teams,get]
File: /data/src/restapi/vendor/league/openapi-psr7-validator/src/PSR7/Exception/NoOperation.php
Line: 16

When going to https://localhost:4443/api/v3/teams

only way I can make this work is to remove the leading slash in the path definition which is not standard and breaks many openapi tools.

I think you can try with replacing this middleware with your own, which also replaces the path in request object before validation (but keeping the original for later use)
https://github.com/thephpleague/openapi-psr7-validator/blob/master/src/PSR15/ValidationMiddleware.php


Added "Awesome Amazon Sellers" list

https://github.com/ScaleLeap/awesome-amazon-seller
A curated list of tools and resources for Amazon sellers.
Reviewed PRs:

#1513
#1523

By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
Requirements for your pull request

You have to review at least 2 other open pull requests. Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration.
I have read and understood the instructions for creating a list.
This pull request has a descriptive title in the format Add Name of List (Example: Add Swift), not Update readme.md or Add awesome list.
The entry in the Awesome list should:

Include a short description about the project/theme of the list. It should not describe the list itself.Example: - [Fish](…) - User-friendly shell., not - [Fish](…) - Resources for Fish..
Be added at the bottom of the appropriate category.

The list I'm submitting complies with the below requirements.

Requirements for your Awesome list

Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
It's the result of hard work and the best I could possibly produce.
Non-generated Markdown file in a GitHub repo.
Includes a succinct description of the project/theme at the top of the readme. (Example)
The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
Not a duplicate.
Only has awesome items. Awesome lists are curations of the best, not everything.
Includes a project logo/illustration whenever possible.

Either centered, fullwidth, or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to maximum half the width of the original image.

Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
Includes the Awesome badge.

Should be placed on the right side of the readme heading.

Can be placed centered if the list has a centered graphics header.

Should link back to this list.

Has a Table of Contents section.

Should be named Contents, not Table of Contents.
Should be the first section in the list.
Should only have one level of nested lists, preferably none.

Has an appropriate license.

That means something like CC0, not a code licence like MIT, BSD, Apache, etc.
WTFPL and Unlicense are not acceptable licenses.
If you use a license badge, it should be SVG, not PNG.

Has contribution guidelines.

The file should be named contributing.md. Casing is up to you.

Has consistent formatting and proper spelling/grammar.

The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
The description starts with an uppercase character and ends with a period.
Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

Doesn't include a Travis badge.You can still use Travis for list linting, but the badge has no value in the readme.

Go to the top and read it again.

Thanks for making an Awesome list! 🙌
It looks like you didn't read the guidelines closely enough. I noticed multiple things that are not followed. Try going through the list point for point to ensure you follow it. I spent a lot of time creating the guidelines so I wouldn't have to comment on common mistakes, and rather spend my time improving Awesome.

@sindresorhus it's a draft, not ready for review yet...

@sindresorhus now it is ready for review

It's not. Please review the guidelines more closely.

I give up ... I have looked it over several times and I cannot find the clues. How many hints do I get?

I don't understand how you can miss it if you read it point for point...

This pull request has a descriptive title in the format Add Name of List (Example: Add Swift), not Update readme.md or Add awesome list.

The entry in the Awesome list should:

Include a short description about the project/theme of the list. It should not describe the list itself.

Includes a succinct description of the project/theme at the top of the readme. (Example)

This should describe what Amazing is.

The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.

The entries in the Blogs section should have a description.

❯ awesome-lint https://github.com/ScaleLeap/awesome-amazon-seller
✖ Linting

  readme.md:62:3
  ✖  62:3  Link to https://thesellingfamily.com/blog/ is dead  remark-lint:no-dead-urls

  1 error

Thank you for your feedback. I guess different people have different attention to detail.

* This pull request has a descriptive title in the format `Add Name of List` (Example: `Add Swift`), not `Update readme.md` or `Add awesome list`.

My PR title was:

Added "Awesome Amazon Sellers" list

Are you mad about the quotes or something? If so, I think the instructions are a bit vague. It should say "exactly the same format as".

Include a short description about the project/theme of the list. It should not describe the list itself.

I have "Amazon sellers" now as the description. Since I cannot say word "resources" per instruction. That is the only thing that can be said. I don't really know how else to write it. That is the theme.

Includes a succinct description of the project/theme at the top of the readme. (Example)

Does that mean it needs a longer description like the example. I guess I am not sure what part is the description. The part that is quoted ("Carefully curated list of awesome Quantified Self resources.") or the longer bit that is underneath it.

* The repo should have `awesome-list` & `awesome` as [GitHub topics](https://help.github.com/articles/about-topics). I encourage you to add more relevant topics.

Done

The entries in the Blogs section should have a description.

OK

The entries in the Blogs section should have a description.

Added.

Tweet: https://twitter.com/awesome__re/status/1102210142446088193


Add a test that interpolated content should not be re-interpolated

This is basically the same as the test case of issue #32, but for basic (non-section) tags.

Merging because it does not seem controversial.


Generate module names in the singular

First of all, I would like to thank you for this wonderful library. The examples used a module named cake, I was expecting when generating the module name entity in singular (one entity = one row). However, unfortunately for me, I got cakes, which confused me a bit. For that reason, this PR is here.
P.S. Perhaps you should think about using the model name instead of a table, to simplify, for example, the singularization 🤔
Adds

Adds a new argument to sea-orm-cli --singularize, which generates singular module names.

Breaking Changes
This would probably break the sea-orm-codegen calls from the code, but I'm not sure if it was used that way or not.

Adds singularize property for Entity, Relation, ConjunctRelation in  sea-orm-codegen.
Requires one more argument in EntityTransformer::transform

Closing in favor of https://github.com/SeaQL/sea-orm/pull/680. But I'm still not sure it's the best option.


Update cray-application image to 0.2.0

Summary and Scope
Updates to take the latest stable cray-application image 0.2.0
Reformat vars.sh to adapt to new image publishing format
Use new Jenkins credential for publishing
Issues and Related PRs

Resolves CASMUSER-3060

Testing

rocket

Test description:
Verified the new image would boot. Performed the following checks

No root password by default
spire-agent started
/proc/cmdline could not be read by a regular user so the presigned url could not be determined
DHCP hostname could be set

Risks and Mitigations
There may be unforeseen changes from lts/csm-1.2 to lts/csm-1.3, however, this feature is a tech preview
and will continue to be worked on. This is an important change to pick up security fixes via vendoring of
node-images and csm-rpms. Future updates will be much easier now that this change has been made.
Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

Waiting on this PR until merge of:
https://github.com/Cray-HPE/application-image-build/pull/1


 We should update this to be a multiselect so that users can filter for multiple SR statuses at once.
BE ticket to support this - https://github.com/ethyca/fidesops/issues/660
Mockup
Aha! Link: https://ethyca.aha.io/features/OPS-55

@adriaaaa @seanpreston After discussing the multi-select design for the SR status dropdown with Adria, the SR dropdown will display multiple selected items in a comma delimited list. In the event the comma delimited list does not fit the width of the dropdown container, an ellipsis will be appended to the right of the text. If the user is to hover over the dropdown, a tooltip will be displayed rendering the comma delimited list in its entirety.

@adriaaaa @seanpreston I have updated the PR. Its ready for Code Review. However, can someone give me any guidance as to updating any appropriate Admin UI documentation if needed as part of the PR? Thank you.


Fixed vulnerabilities in Docker image.

Fixes #10. Update resolves all fixable vulnerabilities. Tested Manually.

Updated to debian testing (stretch).
Updated to go v1.7.
Added .dockerignore file.

@balajismaniam do you have a list of the vulnerabilities before and after the change, for posterity?

Yes, see below.
Before: https://quay.io/repository/kubernetes_incubator/node-feature-discovery/image/51a3065f6dcaed9a067b407a4a71a8fa34e27b90580df3312f2a15e5632e3686?tab=vulnerabilities
After: https://quay.io/repository/kubernetes_incubator/node-feature-discovery/image/00c8170ccd35ec51b65d6eda82764b8713372ca5ec3ac2f66afc372be25f9e86?tab=vulnerabilities


Broken when installing from setup.py

Describe the bug
When I install via setup.py, version.py does not seem to be generated.
To Reproduce
Steps to reproduce the behavior:

Download the latest release tarball
python setup.py install
Try to use the extension

Original exception:
Traceback (most recent call last):
  File "/usr/lib/python3.9/site-packages/sphinx/registry.py", line 417, in load_extension
    mod = import_module(extname)
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 790, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/usr/lib/python3.9/site-packages/sphinx_argparse_cli/__init__.py", line 5, in <module>
    from .version import __version__
ModuleNotFoundError: No module named 'sphinx_argparse_cli.version'

Traceback (most recent call last):
  File "/usr/lib/python3.9/site-packages/sphinx/registry.py", line 417, in load_extension
    mod = import_module(extname)
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 790, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/usr/lib/python3.9/site-packages/sphinx_argparse_cli/__init__.py", line 5, in <module>
    from .version import __version__
ModuleNotFoundError: No module named 'sphinx_argparse_cli.version'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib/python3.9/site-packages/sphinx/cmd/build.py", line 276, in build_main
    app = Sphinx(args.sourcedir, args.confdir, args.outputdir,
  File "/usr/lib/python3.9/site-packages/sphinx/application.py", line 245, in __init__
    self.setup_extension(extension)
  File "/usr/lib/python3.9/site-packages/sphinx/application.py", line 402, in setup_extension
    self.registry.load_extension(self, extname)
  File "/usr/lib/python3.9/site-packages/sphinx/registry.py", line 420, in load_extension
    raise ExtensionError(__('Could not import extension %s') % extname,
sphinx.errors.ExtensionError: Could not import extension sphinx_argparse_cli (exception: No module named 'sphinx_argparse_cli.version')

Extension error:
Could not import extension sphinx_argparse_cli (exception: No module named 'sphinx_argparse_cli.version')

Expected behavior
Building via PEP 517 and via setup.py should both work.
Environment (please complete the following information):

Operating System and Version: Arch Linux

Additional context
I am packaging this project for Arch Linux.

Have you provisioned the build dependencies within the python you're invoking?

Yes.

I wonder why is sphinx being invoked during the build (see registry).

That output is for when you try to use the extension, it is not running during the build.

The build succeeds, it just doesn't include the version module.

Seems like toml was missing and setuptools-scm wasn't reading pyproject.toml because of that. Sorry.


Deprecate either asUInt or asBits

Right now they do they exact same thing. I don't see the purpose for this, and it might make the user think they actually do different things. (I checked the source code to check that they were equivalent)
For what its worth, rocket-chip never uses asBits...

asBits is nearly useless in practice, as it does the same thing as asUInt but returns a less concrete type.  I'd be fine with nuking it.  It wasn't in chisel2, so there's little compatibility concern.

Definitely +1 on deprecating asBits.
It had a nice symmetry with fromBits (asBits and fromBits were nominally
inverses) but we can probably introduce an alias for fromBits that restores
this.
On Mon, Aug 29, 2016 at 1:47 PM, Andrew Waterman<EMAIL_ADDRESS>wrote:

asBits is nearly useless in practice, as it does the same thing as asUInt
but returns a less concrete type. I'd be fine with nuking it. It wasn't in
chisel2, so there's little compatibility concern.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ucb-bar/chisel3/issues/269#issuecomment-243250783,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9_-F8RsE2zRIebofFl2yBrRnKYmI89ks5qk0VYgaJpZM4Jv39j
.

Looks like this has been implemented, asBits is deprecated.


Breaking Change Request: redesign updateSub's API

Problem
Consider
const subID = "xxx"
const stream1 = await pool.newSub(subID, {...});
const stream2 = await pool.updateSub(subID, {...});

Currently, stream1 and stream2 are the same reference.
This is an abstraction invented by nostr.ts since nostr protocol does not have the concept of updating an existing subscription. Our implementation close the previous subscription and creates a new subscription with the same name.
This design looks nice and convenient at first, but after several months' practice, I find this API behavior might not be desireable.

Hard to debug
In Blowater's production, we encountered dangling promises or starvation of channels. https://github.com/BlowaterNostr/blowater/issues/115
The root cause is still unknown but this API design should be related because Blowater did not have this problem before introducing updateSub

Dangling Async Functions
Consider https://github.com/BlowaterNostr/blowater/blob/main/features/profile.ts#L23-L38
If add is called multiple times, many resp.chan will be created. Logically it's still correct because they all add to the same database instance. But we will create unncessary amount of promises that never resolves. This is result in memory leak over time.

Unclear responsibility for channel consumers
In a real world situation, stream1 and stream2 are likely to be constructed in different places of the code and in different time. Since stream1 and stream2 are the same reference. Their consumers effectly are racing to read from the same channel which might not be desired and might cause surprising data loss from individual consumer's perspective.

Suprising Data for consumers
Following No.2, the reader of stream1 might be surprised that it gets data which doesn't match its original filter.

Proposal

Must
We should close the channel of stream1 when updateSub is called, making stream1 and stream2 2 separate references.

Maybe
Forbid the reuse of the same subscription id for different subscriptions and remove updateSub.

In this case, the caller of this API should do
const stream1 = await pool.newSub("profileStream 1", {...});
const stream2 = await pool.updateSub("profileStream 2", {...});

Use a suffix index to indicate that it's from the same feature but for different iterations.

I think both proposals should be done.But we can do it step by step.We can do the first proposal first, and then do the second.

@GuandeLi2 Why do you think No.2 is necessary?

After many months of experiment, I decide to remove updateSub for all the complexity it introduces. #35