[youtube] fix: extract mix playlist ids from ytInitialData (#33)

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Fixes #33

Thanks you so much for your effort fixing this, but still getting Collected 0 video ids.
I've implemented your changes to the extractor, re-built from source and tried these mix ids:
RD-gJaYKwreEc
RDKobbM94mXbo

@someziggyman could you try running it with  --write-pages "RD-gJaYKwreEc" and attach the output .dump file?

@someziggyman could you try running it with --write-pages RD-gJaYKwreEc and attach the output .dump file?

Sure, here it is.
RD-gJaYKwreEc.zip

@someziggyman thank you!
Using your html as input, this code works for me (it extracts the video ids and starts downloading them). Could you possibly double check you compiled these changes in? Maybe also throw a print("yes!") in youtube.py

@someziggyman thank you!
Using your html as input, this code works for me (it extracts the video ids and starts downloading them). Could you possibly double check you compiled these changes in? Maybe also throw a print("yes!") in youtube.py

I'm 100% sure the changes are made and compiled correctly. I'm on MacOS and my Python version is 2.7.16.. Maybe, this could be the reason why it fails? Strange and confusing.  Wonder if anyone else can test case this and report their behaviour.

It is indeed working as it should.
@insaneracist you need to resolve conflicts.

Ok, guys. I'm confused. downloaded the latest release now. Still can not get mix playlist to work :-) I swear, I'm not stupid))
What am I doing wrong? Python version, MacOS? I removed cache dir, using no cache dir, restarted my Mac. Any ideas you can share to help are greatly appreciated.
Here's my log:
yt-dlc-2020.10.31 user$ ./youtube-dlc -v --no-cache-dir -F RD-gJaYKwreEc
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'--no-cache-dir', u'-F', u'RD-gJaYKwreEc']
[debug] Loading archive file None
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dlc version 2020.10.25
[debug] Python version 2.7.16 (CPython) - Darwin-19.6.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
[youtube:playlist] RD-gJaYKwreEc: Downloading page 1 of Youtube mix
[download] Downloading playlist: RD-gJaYKwreEc
[youtube:playlist] playlist RD-gJaYKwreEc: Collected 0 video ids (downloading 0 of them)
[download] Finished downloading playlist: RD-gJaYKwreEc

@someziggyman, this isn't in the latest release. It will probably get merged into the next version.

@someziggyman, this isn't in the latest release. It will probably get merged into the next version.

can you please share your working extractor py file, so that I could test it on my end. I know python is rather tricky with spaces, new lines and things like that. maybe, I've pasted the changes wrong.

Bingo! it now works with your recent python2 fix commit. Thank you so much for your hard work!

you were right, it was python2 unicode/str types. can you try this branch again?
"T-t-this time it will work..."

you were right, it was python2 unicode/str types.
"T-t-this time it will work..."

Bingo! it now works with your recent python2 fix commit. Thank you so much for your hard work!


Extra validations as part of e2e tests

example validate VS, DR, pod, deploy

Tests now include checks on deployments that are present and final traffic distributions.


GEP 2162 - Supported features in GatewayClass Status

/kind gep
What this PR does / why we need it:
This GEP proposes changes to GatewayClassStatus API to include the installed GatewayClass supported and unsupported features .
Which issue(s) this PR fixes:

Fixes #2162
Does this PR introduce a user-facing change?:

Allow end-users to see what features their installed GatewayClass support

/cc @shaneutt

/retest

LGTM for provisional, but I won't do the command until @robscott's changes are done.

Thanks @LiorLieberman!
/lgtm


Improve MapUtils with the null checks, add JUnit for it and add Javadoc for the parameter indent.

Coverage increased (+0.002%) to 90.007% when pulling b547f879efdb7ff1d6f7f3090ca5ebb61dfbc519 on dota17:NullCheckForMapUtils into fbc3c06d787e1c4e594f1b661ec24adcc43255be on apache:master.

@dota17
Please resolve conflicts if this PR is still viable.
TY!

@dota17
Please resolve conflicts if this PR is still viable.
TY!

Thank you for your review. I had fixed it,

Hi @dota17 Sorry for the churn, may you please rebase on git master so we can get a green set of builds?

@dota17 You might have done a merge instead of a rebase but the PR is now too messy.


add a dropdown that gets automaticaly opened on hover

example of dropdowns that open on hover here.

We'll stick to on click vs hover, thanks though!

+1


Make it clear auth endpoint don't store tokens

From twitter:

Quick question - in the auth flow, you authorize against (link: https://github-editor-auth.herokuapp.com) github-editor-auth.herokuapp.com. Does that service store any auth tokens or is it stateless and simply passes them back to the editor? A bit cautious about granting access to private repos :)

We should make it clear in the auth-endpoint front-end that the token doesn't get stored there, but within the editor, and is encrypted in keychain.

Good point, yes, I'll add that information to the frontend 👍

This is fixed, forgot to update the issue. I've also added information about which server we're connecting to, so it's clearer.


DockStream component config defaulting to some unknown config?

Hi,
I noticed that if my pathing is incorrect for the DockStream configuration_path, i.e. it leads to nowhere, REINVENT will still run and still perform docking, but with some unknown configuration. What is it running on in this case?
To test, run the component with any string in configuration_path which isn't an actual file.

Turns out it isn't a default config; REINVENT just runs with DockStream not working and returns 0s for that component. The same is true if the vina version doesn't match what DockSream requires.
REINVENT should crash if DockStream isn't working properly, instead of continuing with no error message.


Cannot extract files with a tab character

Traceback (most recent call last):
  File "E:\Development\Sublime\package_reviewer\package_reviewer\repo_tools.py", line 70, in download
    zipf.extractall(path=str(dirpath))
  File "c:\python35\Lib\zipfile.py", line 1344, in extractall
    self.extract(zipinfo, path, pwd)
  File "c:\python35\Lib\zipfile.py", line 1332, in extract
    return self._extract_member(member, path, pwd)
  File "c:\python35\Lib\zipfile.py", line 1395, in _extract_member
    open(targetpath, "wb") as target:
OSError: [Errno 22] Invalid argument: 'C:\\Users\\Fichte\\AppData\\Local\\Temp\\pkg-rev_w6sei_6a\lyzz0612-Cocos-Creator-Snippet-16f53bc\\creator_snippet\\cc\\Boolean\t.sublime-snippet'

Note to self: Tab characters are not valid in filenames on Windows. Ideally I would post an error message when I encounter invalid characters in filenames inside the zip archive, which is mostly relevant for Windows systems. For simplification, I will most likely require these regardless of whether the package should be installable on Windows or not. Just use sane file names.


feat: make --gas-report w/ --json output one JSON blob and add contract_path to output

Motivation

Closes: https://github.com/foundry-rs/foundry/issues/9111
Solution
Looks as follows after passing through | jq:
It does not render pretty formatted in the terminal by itself
[
  {
    "contract": "src/Contracts.sol:ContractOne",
    "deployment": {
      "gas": 103375,
      "size": 255
    },
    "functions": {
      "foo": {
        "calls": 1,
        "min": 45387,
        "mean": 45387,
        "median": 45387,
        "max": 45387
      }
    }
  },
  {
    "contract": "src/Contracts.sol:ContractThree",
    "deployment": {
      "gas": 103591,
      "size": 256
    },
    "functions": {
      "baz": {
        "calls": 1,
        "min": 260712,
        "mean": 260712,
        "median": 260712,
        "max": 260712
      }
    }
  },
  {
    "contract": "src/Contracts.sol:ContractTwo",
    "deployment": {
      "gas": 103375,
      "size": 255
    },
    "functions": {
      "bar": {
        "calls": 1,
        "min": 64984,
        "mean": 64984,
        "median": 64984,
        "max": 64984
      }
    }
  }
]

A contract with overloaded functions (setNumber(int256) / setNumber(uint256)) looks as follows:
[
  {
    "contract": "src/Counter.sol:Counter",
    "deployment": {
      "gas": 101137,
      "size": 250
    },
    "functions": {
      "a": {
        "calls": 1,
        "min": 2261,
        "mean": 2261,
        "median": 2261,
        "max": 2261
      },
      "b": {
        "calls": 1,
        "min": 2305,
        "mean": 2305,
        "median": 2305,
        "max": 2305
      },
      "setNumber": {
        "setNumber(int256)": {
          "calls": 2,
          "min": 23648,
          "mean": 33604,
          "median": 33604,
          "max": 43560
        },
        "setNumber(uint256)": {
          "calls": 2,
          "min": 23604,
          "mean": 33560,
          "median": 33560,
          "max": 43516
        }
      }
    }
  }
]

@zerosnacks argh, always a bit to late to comment 😓
In the previous version https://github.com/foundry-rs/foundry/pull/9063 the exact signature was always part of the output, now it seems like it's only visible when the function is overloaded.
Imo this is a regression, i understand the motivation behind avoid unnecessary nesting of non-overloaded function names, but not at the price of losing the signature.
For parsing and diffing it's also cumbersome that the level of nesting changes based on the addition or removal of other functions.
Would anything speak against doing a non nested structure, this would be so much easier to process:
[
  {
    "contract": "src/Counter.sol:Counter",
    "deployment": {
      "gas": 101137,
      "size": 250
    },
    "functions": {
-     "a": {
+     "a(bool)": {
        "calls": 1,
        "min": 2261,
        "mean": 2261,
        "median": 2261,
        "max": 2261
      },
-     "b": {
+     "b(address)": {
        "calls": 1,
        "min": 2305,
        "mean": 2305,
        "median": 2305,
        "max": 2305
      },
-      "setNumber": {
        "setNumber(int256)": {
          "calls": 2,
          "min": 23648,
          "mean": 33604,
          "median": 33604,
          "max": 43560
        },
        "setNumber(uint256)": {
          "calls": 2,
          "min": 23604,
          "mean": 33560,
          "median": 33560,
          "max": 43516
        }
-    }
    }
  }
]

Added a follow-up fix addressing @sakulstra's comment in https://github.com/foundry-rs/foundry/pull/9229


file input support for payloads in config file

Hi @projectdiscovery team,
An Awesome & Nice tool to work with : )
And It will be great and more helpful if implement support for loading wordlists file on permutations.yaml for payloads just like loading files on other yaml configs
Example permutations.yaml :
patterns:
  - "{{word}}-{{sub}}.{{suffix}}"
  - "{{sub}}-{{word}}.{{suffix}}"

payloads:
   word: /path/to/wordlists.txt

Thanks & Regards,
@zy9ard3

@zy9ard3 this is now supported in latest release - https://github.com/projectdiscovery/alterx/releases/tag/v0.0.2


Added clearer error reporting when skipping pre-releases

Added clearer error reporting when skipping pre-releases by splitting the output by regular/pre-release versions.

This pull request is to fix: https://github.com/pypa/pipenv/issues/2046


Expose TamperSheetGesture and related classes as public APIs

First thanks for open source this great package!
And I have seems some API marking as "TODO: Expose this as a public API.", eg what we need is the swipe down to dismiss to be working for more draggable sheet. Make TamperSheetGesture related API public will create help and make package more extendable.
It seems like it is a quick win to just remove the @internal, is there any timeline to release this?

I've omitted them from the public APIs mainly due to the lack of documentation and examples.

It seems like it is a quick win to just remove the https://github.com/internal, is there any timeline to release this?

I'm currently focused on bug fixes related to the on-screen keyboard, so it won't be released soon. However, I'll work on this task before the first stable release.

What we need is the swipe down to dismiss to be working for more draggable sheet.

Could you explain in more detail what you're trying to achieve?


add minimum liquidity check to removeLiquidity

referenced issue: https://github.com/code-423n4/2022-06-yieldy-findings/issues/48

LGTM


Update Isovalent's logo to stacked version

Change the logo for Isovalent, from the horizontal to the stacked version, for better readability.
Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[ ] Have you picked the single best (existing) category for your project?
[ ] Does it follow the other guidelines from the new entries section?

(Above does not apply, this PR does not add a new entry.)

[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Deploy Preview for landscape ready!
Built with commit 10443463af6cc5979c8b4b962953cb714fc51b9f
https://deploy-preview-2102--landscape.netlify.app

LGTM


[chore]: add challenge 17

Add the question for challenge 17 as well as the code

resolved issue: https://github.com/akebu6/Kotlin-Code/commit/8759328d34b9d2004e7f8905c9c9f2cd3adc629e


Implement flag to avoid blue bubble/s

Description
Implement flag to avoid "user education" blue bubbles nudges (eg., "… give your browser a new look", Memory Saver)
Who's implementing?

[ ] I'm willing to implement this feature myself

The problem
I'm always frustrated when "user education" blue bubbles nudges (eg., "… give your browser a new look", Memory Saver) show up.
Possible solutions
Implement --ash-no-nudges like chrome upstream documented here.
Alternatives
No response
Additional context
No response

Is it not? https://github.com/ungoogled-software/ungoogled-chromium/blob/992a33ed3c0ebfc6afb62dca06436533424e1121/patches/extra/ungoogled-chromium/remove-uneeded-ui.patch#L32

@opzch @PF4Public
This is my user experience.

@PF4Public @opzch
This is my user experience at the moment.


Prepare the 2.1.145 release.

Closes #2218

Breaking this small release out to help the Nix folks packaging Pex.


⚠ Jenkins Plugins Artifactory has degraded performance

In 0681d1a, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1158 ms

Resolved: Jenkins Plugins Artifactory performance has improved in dc056e0 after 12 minutes.


Race condition causes multiple healthcheck containers

In the node monitoring, a health-check container is created if it doesn't already exist.
The timeout of the container creation is 60 seconds and the monitor action recurs every 30 seconds. If the container creation takes too long and is not created by the time the next monitoring action runs, another container is created and you end up with multiple health-check containers.

I suggest that the healthcheck container is created and managed by its own service. That way we don't depend on the actual creation of the container to assume that the container has never been created.
Instead, should be if service already exists.


pkg/httpclient: Consolidate HTTP hijacking.

Since we're using an uniform client for doing http requests, we can provide a
convenience method for hijacking. This also standardises hijacking -- the attach
endpoint used status 200, but http.SwitchingProtocols seems more appropriate.
Whereas Stream is defined as a SSE stream of JSON objects, Hijack is more
general and unspecified, allowing for custom protocols where SSE would not work
well.
Signed-off-by: Blaž Hrastnik<EMAIL_ADDRESS>

Note that I wasn't able to get rid of utils.HijackRequest yet, as a controller test references it. I think the controller tests should have the utility methods replaced with httpclient, then we can replace that piece of code with a call to httpclient.Hijack. That would also unify all of our API calls to use httpclient, one drawback however is that the tests expect http.Response objects; in most cases we use them to simply check the status code, which could be replaced with checking for a specific error (t.Assert(err, c.IsNil) for 200, t.Assert(err, Equals, client.NotFoundError) for 404).

@flynn/devs needs a review.

LGTM


Line position when comparing

Issue Type: Bug
When I compare two files, only the file on the right side hava the "status bar", i don't know if is this the name, but only the right side have this.
For me, i need to compare the selected text between, but it not show the exactly place, and i need to count.
VS Code version: Code 1.33.1 (51b0b28134d51361cf996d2f0a1c698247aeabd8, 2019-04-11T08:27:14.102Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E3-1226 v3 @ 3.30GHz (4 x 3292)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.89GB (1.23GB free)

Process Argv
C:\Users\Baccan\Desktop\SEFIP_GTECH_04.2019.RE

Screen Reader
no

VM
0%

Extensions (23)

Extension
Author (truncated)
Version

rainbow-brackets
2gu
0.0.6

Bookmarks
ale
10.4.3

pascal
ale
8.0.1

pascal-formatter
ale
2.2.0

vscode-eclipse-keybindings
alp
0.10.0

google-translate-ext
ben
1.3.1

vscode-postgres
cko
1.1.10

githistory
don
0.4.6

gitlens
eam
9.7.3

prettier-vscode
esb
1.9.0

code-runner
for
0.9.9

vscode-google-translate
fun
1.1.5

innosetup
idl
1.2.0

vscode-add-angular-native-files
jos
0.0.11

vsliveshare
ms-
1.0.181

vscode-docker
Pet
0.6.2

LiveServer
rit
5.6.1

code-settings-sync
Sha
3.2.9

javascript-typescript
sou
0.0.7

open-in-browser
tec
2.0.0

nativescript
Tel
0.10.0

vscode-icons
vsc
8.6.0

local-history
xyz
1.7.0

(1 theme extensions excluded)

Can you provide a screenshot and point out what the "status bar" is?

I posted two pictures, one i have the mouse focus on the left, and the status bar only show the position of my cursor on the right text.
Focus os left

Focus os right


update maintainer and remove dead code

@mfojtik ptal pending maintainer

argh. merged the wrong tab.  will revert if necessary.

@bparees the cleanup looks good, but not sure if the MAINTAINER addresses are correct. The address for centos image should be SoftwareCollections.org<EMAIL_ADDRESS>and for rhel there should be none (at least thats what I see in the ruby-2.2). And whats with the<EMAIL_ADDRESS>@smarterclayton suggested ?

Please, don't use<EMAIL_ADDRESS>the images are supposed to be not only for OpenShift.

fixing here:
https://github.com/openshift/mongodb/pull/65


need support for changing the tax field.

Want to delete the tax field from the incoice and members please can you guys help.

https://demo.gymie.in
fixed


Calling TryValidateModel(object) created unnecessary errors when using [Required] on string properties

I'm using RazorPages and I have some troubles validating the user inputs.
As I learned from #6891 validating the PageModel always returns true at the moment.
I observed this bug in my program, so as mentioned as a workaround, wrapping the whole model properties as a in a dedicated viewmodel class seems to be a legal option to me.
This approach works nearly perfect, except for the fact that when I'm manually validating my viewmodel with TryValidateModel(object), I get some strange behaviour with my strings properties which are notated as [Required].
After the manual validation, where are double errors for these string fields...
One validation for key "StringValueProp" one for "model.StringValueProp".
With behaviour printing out the ValidationSummary isn't usable because all non-filled required string fields are displayed two times.

Could you share a repro for this?

Thanks for contacting us, @Groman123. @kichalla, assigning this to you for further investigation. Thanks!

@Groman123 I unable to repro the issue you are seeing at my end. A repro could help.

Closing this issue now as we haven't received any responses yet. Feel free to re-open it with additional details.


How to configure Settings Sync via command line?

https://code.visualstudio.com/docs/editor/command-line#_working-with-extensions shows how to install an extension via command line. For this extension, this means code --install-extension Shan.code-settings-sync, which works perfectly.
But now I still have to manually edit the gist id. Is there a way to do this step via command line as well?
PS: This question is now also at StackOverflow: https://stackoverflow.com/questions/47685100/how-to-configure-visual-studio-code-extensions-from-command-line

Duplicate of https://github.com/shanalikhan/code-settings-sync/issues/403
Currently its Limitation from Code, editor doesnt allow extension to perform actions from CLI

Currently its Limitation from Code, editor doesnt allow extension to perform actions from CLI


Implement quick sort in C++

We only have quicksort in Python, please implement in C++.
The code should be inside the folder "classical_algorithms"

Hello 👋 @larissalages I am new to the open-source ecosystem and also in the Hacktoberfest right now, it will be really a good first issue for me. Can you assign this issue to me, so that I can work on it?

Hello 👋 @larissalages I am new to the open-source ecosystem and also in the Hacktoberfest right now, it will be really a good first issue for me. Can you assign this issue to me, so that I can work on it?

sure @soumya997 !

is there any issue with my code @larissalages. if yes then suggest me the way to fix it. so that I can make changes

@larissalages if my commit is suitable then please merge it.

@dikshantmali , as it is already assigned to me so I would like to that.
And I am new to open source and I am taking it as one of my Hacktoberfest contributions as the issue has been tagged as Hacktoberfest .
I know how to do that, as I am taking it as my Hacktoberfest contribution that's why i did not make any PR.
it will be very helpful for me if you kindly revert the PR.

Sure brother
On Tue, 22 Sep 2020, 10:20 pm Soumyadip Sarkar<EMAIL_ADDRESS>wrote:

@dikshantmali https://github.com/dikshantmali , as it is already
assigned to me so I would like to that.
And I am new to open source and I am taking it as one of my Hacktoberfest
contributions as the issue has been tagged as Hacktoberfest .
I know how to do that, as I am taking it as my Hacktoberfest
contribution that's why i did not make any PR.
it will be very helpful for me if you kindly revert the PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/larissalages/code_problems/issues/2#issuecomment-696845680,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALNDZC34MWDU36OOONOEVZLSHDIVRANCNFSM4RTGLUAA
.

@dikshantmali , as it is already assigned to me so I would like to that.
And I am new to open source and I am taking it as one of my Hacktoberfest contributions as the issue has been tagged as Hacktoberfest .
I know how to do that, as I am taking it as my Hacktoberfest contribution that's why i did not make any PR.
it will be very helpful for me if you kindly revert the PR.

@soumya997 all the best for Hacktoberfest. I am also participating the first time but if you want to contribute to it you are welcome.

@soumya997 all the best for Hacktoberfest. I am also participating the first time but if you want to contribute to it you are welcome.

Thank you.
I will surely work on it and will try to make PR at the beginning of October.

@larissalages can I work on it?

We only have quicksort in Python, please implement in C++.
The code should be inside the folder "classical_algorithms"

I can do that for you.

We only have quicksort in Python, please implement in C++.
The code should be inside the folder "classical_algorithms"

DONE!
https://github.com/larissalages/code_problems/pull/47

@kartikeysingh6 you should ask and get issue assigned before a PR.

@kartikeysingh6 you should ask and get issue assigned before a PR.

I'm relatively new here, how can I issue an assign? Also I've already made PR, will it get rejected?

Ask Author to assign you before u pull a request. @kartikeysingh6

We only have quicksort in Python, please implement in C++.
The code should be inside the folder "classical_algorithms"

Hey @larissalages , I know you've already assigned this task to @soumya997 but it has been 13 days and no progress has been made, if you could assign this task to me I'd be grateful and I've already done the task and assigned a PR.
Thank you,
Kartikey Singh

I will be making the PR by tomorrow, I got involved to other Repos actually. My bad.
But, i promise it will be done tomorrow.


Fix for nil class error if chef client handlers are not defined.

Signed-off-by: Vasu1105<EMAIL_ADDRESS>Description
If user sets chef-client configuration attributes in the following format in his cookbook
default['chef_client']['config'] = {
  rubygems_url: 'https://example.com'
}

it results into the undefined method empty? for NilClass  error as mentioned in the issue #611
Reason is above value overrides default attributes due to which handlers are set to nil, which results in failing of empty? check for handlers and throws above mentioned error.
Chef handlers configuration should be defined in array as mentioned in the Readme. I have added the class check i.e. Array and then if its not empty array and removed the unwanted code.
Taken care of #612
Issues Resolved
#611
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

@tas50 & @lamont-granquist Please review.


operators [R] couchdb-operator (1.0.14 1.1.0 1.2.1 1.3.1 1.4.4 2.2.0)

Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@J0zi or @mvalarh are you able to advise on the errors I'm seeing in the logs?
It looks like there is a problem with the dependency graph, but I can't figure out what changes to make to the latest version to resolve that.
The end goal is to add 2.2.0 into a new 2.2 channel and the stable channel. And make it upgradeable from all other versions. I was expecting the old.skipRange to implement that, but perhaps something else is required?

@mvalarh we could very much use assistance here
tests are complaining about several issues that seem internal

level=error msg="permissive mode disabled"
incorrect "opm index add" usage
a warning "this may be due to incorrect channel head"

any idea how this may be fixed?

@welshDoug i think you have many same channels and they don't replace each other. I would reduce channels to have replaces graph filled correctly.

@mvalarh how can I remove channels?
Just removed 2 CSVs, so each channel (except stable) only has 1 entry and all use skipRange instead of replaces

/merge possible

/merge possible

@mvalarh Think I fixed it by removing the earliest release from the base of all channels.
So this PR will affect all releases, are you okay to publish that?

/merge possible


fix double close on error messages and leaks in body response not closed

This change is

Coverage increased (+0.06%) to 53.309% when pulling 4c8783cbab7b515daad1e411a775d5f136ce8e5c on feature/err_message_liort into 2e726168fa4e402cad1df3ac1d69b1999df7232d on dev.


DNN-7151: Convert to Web Application Project

initial conversion to a wap project. very basic smoke testing (installed clean site, no upgrade tests yet)

I'll merge this once we finalize the CTP 3 build

Merged with a couple of small modifications.

So far - and that was not really related.  I am sure other issues will arise - there might be an upgrade issue, but they will be treated as bugs moving forward.

Reviewed code, looks good.


Different Readthedocs/latest and pip version

Hello,
I was able to access the open_access column from my Arxiv dataframe after downloading arcas from pip. It seems like the current Master version is not the same as the one available through pip and that the documentation is for the Github version, not the pip one.
This was confusing for me as I could not reproduce parts of the documentation.
Thank you for your work,
Cheers,
ilyass

Hello @ilyasst,
Thank you very much for your issue.
You are right they are not the same version, that's my fault. Thank you for pointing it out. I will make a new pip release later today.
Did you end up cloning the repository? Is everything working for you now?

I ended up cloning the repository after installing the pip version, everything worked fine after that.
I got an issue with IEEE that I could not get to work with my api_key, but I did not spend that much time trying to fix that yet.
Another thing that is not super clear to me is how to gather large amounts (hundreds and hundreds) of publications without hitting the 504 error ...

Thanks again for your issue. The new release is now out.
The way that I collected data was to ask for 10 articles per request and iterate over the start argument. There were times that I did hit a 504 error and just waited for a day or so before I starting again.
That being said there this library:https://pypi.org/project/ratelimit/ which has a function decorator preventing a function from being called more often than that allowed by the API provider. But you need to know what the rules are for the specific APIs and I am afraid I do not know remember.
I read you blog post and it was a really good read! It's unfortunate that not all providers have contain information on whether the article is open access or not.


scala sbt quickstarter broken - file:/?/.sbt/preloaded/org/fusesource/jansi/jansi/1.4/jansi-1.4.pom. Please retry. (0ms)

Rundeck provisionioning of sbt / scala quickstarter:
problem while downloading module descriptor: file:/?/.sbt/preloaded/org/fusesource/jansi/jansi/1.4/jansi-1.4.pom: Downloaded file size doesn't match expected Content Length for file:/?/.sbt/preloaded/org/fusesource/jansi/jansi/1.4/jansi-1.4.pom. Please retry. (0ms)
18:55:46 |
18:55:46 |
18:55:46 | :: USE VERBOSE OR DEBUG MESSAGE LEVEL FOR MORE DETAILS
18:55:46 | :: retrieving :: org.scala-sbt#boot-scala
18:55:46 | confs: [default]
18:55:46 | 5 artifacts copied, 0 already retrieved (24560kB/29ms)
18:55:48 | java.lang.IllegalArgumentException: URI has a query component
18:55:48 | at java.io.File.(File.java:427)
18:55:48 | at sbt.IO$.uriToFile(IO.scala:160)
18:55:48 | at sbt.IO$.toFile(IO.scala:135)
18:55:48 | at sbt.Classpaths$.sbt$Classpaths$$bootRepository(Defaults.scala:2028)
18:55:48 | at sbt.Classpaths$$anonfun$appRepositories$1.apply(Defaults.scala:1998)

The whole quickstarter needs Engineering- i suggest to store the Template locally to increase Performance and also provide version safety.


Diff Image is empty

We implemented successfully snapshot tests for critical scenarios using XCUITests and snapshots. When I run locally using Fastlane or Xcode. it is working fine. Currently, all tests are failing on builds machining We are not able to see any diff change in diff image. I have set a 0.985 match. and I also tested 1 match locally and works fine as well.

Builds machines and local simulators are same (iPhone 11 Pro) iOS 13.3
We also override status bars to keep time, network, and batteries. That's keep everything same for all snapshot

Can you share the images please ?

Have you tried using https://kaleidoscope.app in order to compare your diffs ?
This SnapshotTesting.diffTool = "ksdiff" will simplify using kaleidoscope.app for the diff process.

Yes, I am using SnapshotTesting.diffTool = "ksdiff" that I am calling from functions. Still failing

I also check both images on https://online-image-comparison.com and seem valid. I haven't seen any diff

Even I do as: .wait(for: timeout, on: .image(precision: 0.80)), still faiing

@mackoj Any help ?

@gonzalezreal

Have you tried to change the output and not use image but something like recursive description for exemple ?

@jambudi It's not empty actually. You can check this by downloading diff image and using adjust color feature in mac os x preview.

@jambudi Were you able to figure out the problem or are you able to provide more info?

@stephencelis  No, We were not able to fix it. I reduce the percentage match to 0.85

I can provide more information. Let me know what exactly you need.  @stephencelis

@ShurupuS big 👍, I am also uploading color settings that worked for me:

@ShurupuS big 👍, I am uploading the color settings that worked for me after hitting Auto Levels button:


Add support for tail follow?

Currently i want to use tail -f -n X but as deno native sadly this lib can't do follow. Would be nice if it could

Yes that would be a nice addition.
I guess it would have to use Deno.watchFs to be notified of file changes.
I don't know if that could sit behind an async iterator.
Maybe it would have to be event based like:
const file = await Deno.open('log.txt');
new Tail(file).addEventListener('line', () => { /* callback here */ });

I'll see if I can make an example soon.


Plugin is not working!

Hi,
I tried creating a simple example with the help of this plugin. But it didn't work.
Can you please tell me what is it that I am doing wrong?
I am attaching the file with my ticket.
index.txt
Any help would be greatly appreciated.
Best regards,
Ashar

Hi,
I don't know, if this question has been solved yet. If not, I can give you a hint what was wrong:

The import of the jquery file had a syntax error (to many


Add a Dockerfile and usage

Hello @mingrammer
I suggest you add a Dockerfile to this project. If a user already uses docker can enable it without any installations and easily collaborate with other docker containers.
It would also be great if the generated log out from stdout instead of a file as default.

Hi cometkim. Thank you for first PR :)
I'll check it when getting back home today.
And, sure, I have a plan for also supporting stdout and gz log type soon. (stdout will be default type)

go-dep-onbuild enforce the ENTRYPOINT to app. So, we can provide the docker installation when stdout is supported as default type.

Yeah. it also can be used with file or gz option with shared volume, to provide logs to other containers. 👍

From https://github.com/mingrammer/flog/releases/tag/v0.1.0 flog supports stdout, gz option :) I'll register the docker image with your Dockerfile and merge it.

Thank you @cometkim !


Preview DocC feature on staging

We'll want to help preview/test upcoming DocC features. The plan is to use our staging environment for this and build documentation for a selection of packages.
See this post on the forums for details of the first feature that would benefit from this: https://forums.swift.org/t/support-for-docc-experimental-features/61276

Brief update on the progress. The builder side changes are done. The remaining todos are:

[ ] source list of packages to allow-list for building on dev
[ ] implement allow-list trigger downscaling override in the server
[ ] deploy builder changes
[ ] deploy server changes
[ ] end-to-end test process with SemanticVersion package

Re: toolchain install, that's Swift Development Snapshot 2022-11-03 (a).
After installing, use the following command to validate the installation:
xcrun --toolchain org.swift.57202211031a swift --version

This should print
Apple Swift version 5.8-dev (LLVM 1299cdab04e4506, Swift 56665d5ce9f6420)
Target: arm64-apple-macosx13.0

Here's a script to list installed toolchains:
#!/usr/bin/env swift

import Foundation

let toolchainsDir = URL(fileURLWithPath: "/Library/Developer/Toolchains")

struct ToolchainInfoPlist: Codable {
    var bundleIdentifier: String
    var createdDate: Date
    var displayName: String

    enum CodingKeys: String, CodingKey {
        case bundleIdentifier = "CFBundleIdentifier"
        case createdDate = "CreatedDate"
        case displayName = "DisplayName"
    }
}

func readPlist(url: URL) throws -> ToolchainInfoPlist {
    let data = try Data(contentsOf: url)
    return try PropertyListDecoder().decode(ToolchainInfoPlist.self, from: data)
}

func getToolchainInfo(toolchainsDir: URL) throws -> [ToolchainInfoPlist] {
    try FileManager.default.contentsOfDirectory(atPath: toolchainsDir.path)
        .map { toolchainsDir.appendingPathComponent($0) }
        .filter(\.hasDirectoryPath)
        .map { $0.appendingPathComponent("Info.plist")}
        .map(readPlist(url:))
}

print("Installed toolchains:")
try getToolchainInfo(toolchainsDir: toolchainsDir)
    .map(\.bundleIdentifier)
    .forEach {
        print("xcrun --toolchain \($0) swift --version")
}

Initial allow list:

SemanticVersion
SPIManifest
ArgumentParser

[
  "38D534BF-4BE1-498E-B973-E7E885FC3A4E",
  "B16EF7C1-A039-4D41-AFBE-FCAB873713CE",
  "112BDD86-7BBB-490F-9484-BD4020ED5A50",
]

Crosspost from https://forums.swift.org/t/support-for-docc-experimental-features/61276/6

We're almost done implementing this but have hit a weird last minute error. When generating the docs on our build machine, we're getting a "Bad CPU type in executable" error with the Nov 3 nightly toolchain. This works fine with the regular 5.7 toolchain.
Our build machine is an M1 Mac mini which doesn't have Rosetta installed. That might explain why in my testing this same command works fine on my M1 MBP which has Rosetta.
Is there anything new in the 5.8 doc generation workflow that might be calling our to an x86 only binary? We can install Rosetta on the build machines if need be but if we can avoid it that'd be even better.
Here's the whole command sequence that replicates our process and works on my machine but fails on our builder with the above mentioned error:
git clone --depth=1 --branch 0.3.5 "https://github.com/SwiftPackageIndex/SemanticVersion.git"

cd SemanticVersion

mkdir -p .docs/swiftpackageindex/semanticversion

cat <<EOF >> Package.swift

package.dependencies.append(
    .package(url: "https://github.com/apple/swift-docc-plugin", from: "1.0.0")
)
EOF

xcrun --toolchain "org.swift.57202211031a" env DOCC_JSON_PRETTYPRINT="YES" swift package --allow-writing-to-directory ".docs/swiftpackageindex/semanticversion/0.3.5" generate-documentation --disable-indexing --transform-for-static-hosting --output-path ".docs/swiftpackageindex/semanticversion/0.3.5" --hosting-base-path "swiftpackageindex/semanticversion/0.3.5" --target "SemanticVersion"

Crosspost from https://forums.swift.org/t/support-for-docc-experimental-features/61276/7

Oh, it's actually the docc binary in that toolchain itself that's x86 only:
❯ file /Library/Developer/Toolchains/swift-DEVELOPMENT-SNAPSHOT-2022-11-03-a.xctoolchain/usr/bin/docc
/Library/Developer/Toolchains/swift-DEVELOPMENT-SNAPSHOT-2022-11-03-a.xctoolchain/usr/bin/docc: Mach-O 64-bit executable x86_64

vs
❯ file /Applications/Xcode-14.1.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/docc
/Applications/Xcode-14.1.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/docc: Mach-O universal binary with 2 architectures: [x86_64:Mach-O 64-bit executable x86_64] [arm64]
/Applications/Xcode-14.1.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/docc (for architecture x86_64):	Mach-O 64-bit executable x86_64
/Applications/Xcode-14.1.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/docc (for architecture arm64):	Mach-O 64-bit executable arm64

The problem with the nightly toolchain was fixed but the fix broke the toolchain build and had to be rolled back: https://github.com/apple/swift-docc/issues/433
However, in the process of addressing this, Apple have created a toolchain build that I've validated works and which we can install on mb5 for now:
https://ci.swift.org/job/swift-PR-toolchain-macos/418/artifact/branch-main/swift-PR-62306-418-osx.tar.gz
This would need to go into /Library/Developer/Toolchains and it should be listed when running the lookup command from above:
❯ list-toolchains.swift
Installed toolchains:
xcrun --toolchain org.swift.57202211031a swift --version
xcrun --toolchain org.swift.pr.62306.418 swift --version

[ ] update builder with new toolchain version

I removed the 20221103 version and installed this the 62306 version. Here is the output of list-toolchains.swift from mb5:
mb5:~ $ ./list-toolchains.swift
Installed toolchains:
xcrun --toolchain org.swift.pr.62306.418 swift --version

Running xcrun --toolchain org.swift.pr.62306.418 swift --version opened up the "Untrusted developer" window on macOS, and the process was killed. I manually ran the swift command from within the installed toolchain and confirmed it was safe, and now xcrun --toolchain org.swift.pr.62306.418 swift --version reports:
mb5:~ $ xcrun --toolchain org.swift.pr.62306.418 swift --version
Apple Swift version 5.8-dev (LLVM f21cfd5346ab53b, Swift f6429bee5531fbd)
Target: arm64-apple-macosx13.0

I only did that for the swift binary, so if we need more than that then let me know which ones we need.

Interesting, I didn't get any such alert when I tested the toolchain locally. I wonder what that's about 🤔
Anyway, I've run my test script from above and it worked fine. So we should have all binaries available.
mb5:SemanticVersion (0.3.5*) $ xcrun --toolchain "org.swift.pr.62306.418" env DOCC_JSON_PRETTYPRINT="YES" swift package --allow-writing-to-directory ".docs/swiftpackageindex/semanticversion/0.3.5" generate-documentation --disable-indexing --transform-for-static-hosting --output-path ".docs/swiftpackageindex/semanticversion/0.3.5" --hosting-base-path "swiftpackageindex/semanticversion/0.3.5" --target "SemanticVersion"

Fetching https://github.com/apple/swift-docc-plugin from cache
Fetched https://github.com/apple/swift-docc-plugin (0.58s)
Computing version for https://github.com/apple/swift-docc-plugin
Computed https://github.com/apple/swift-docc-plugin at 1.0.0 (0.50s)
Creating working copy for https://github.com/apple/swift-docc-plugin
Working copy of https://github.com/apple/swift-docc-plugin resolved at 1.0.0
Generating documentation for 'SemanticVersion'...
Building for debugging...
[5/5] Compiling SemanticVersion resource_bundle_accessor.swift
Build complete! (13.83s)
Converting documentation...
Conversion complete! (0.06s)
Generated DocC archive at '/Users/builder/Downloads/SemanticVersion/.docs/swiftpackageindex/semanticversion/0.3.5'

Also our builder CI now passing so we should be good to go:
https://gitlab.com/finestructure/swiftpackageindex-builder/-/pipelines/712400478

Failed in end-to-end testing. For some reason the deciding env variable isn't picked up in process:
WARNING: environment variable API_BASE_URL is not set

Perma-link to job logs: https://gitlab.com/finestructure/swiftpackageindex-builder/-/jobs/3418633688
This means we're falling back to release-5.7 and building with the old docs.
I think the problem is that while API_BASE_URL is set where we run the builder, this isn't re-exported into its env.

Well, the variable is actually named API_BASEURL 🤪

Ok, this is working now:

https://staging.swiftpackageindex.com/SwiftPackageIndex/SemanticVersion/0.3.5/documentation/semanticversion

I've rebuilt the docs for all three packages in our current allow-list:

SemanticVersion
SPIManifest
swift-argument-parser

If anyone's watching, please let us know (here or via email/Discord) if there are other packages you'd like us to allow-list for processing on dev!


2-3 Lines Introduction in README.md

Hello,
it is not directly evident what the use of this library is from the README.md,
Adding an introduction of 2-3 lines will tremendously help to understand what is the main purpose of this library!

Short summary added


Credits

Hey guys,
I see this repo keeps being used and since it started off https://github.com/teocomi/dug-dynamo-unchained maybe some credits could be added?
Or even better it could be hosted by an org account instead?
Cheers

Hey @teocomi , of course, i honestly thought we had either forked (which would have a notice) or had a section about it, for when we did it for Berlin.  Maybe it was in the during the class only ?
In any case, added a notice front & center until we figure out where the best place would be for it (what org?), sound ok ?

Cheers man!


When will changes in master be merged?

I am waiting for https://github.com/jazzband/django-model-utils/issues/330 to be included in master as this presently breaks caching for my application.
Should use the unreleased version / master?
Should I fork and merge the change as a hotfix to current release?

I see travis builds are failing in master branch and this PR solves this problem: https://github.com/jazzband/django-model-utils/pull/358

you can install from github master branch using pip.

Well of course I can do that. But what other changes are in master? When are the changes in master going to be promoted to a new release? Is there an issue to track this?


Update leverage-containers-orchestrators.md

No downtime deploy is achievable when using some mecanisms like Azure App Service Slots.
Summary
Describe your changes here.
Being more realist about the consequences of monolith app deployment.

Hi @karoldeland, thank you for submitting this PR.


FYI: Security Header: Clear Site Data

https://scotthelme.co.uk/a-new-security-header-clear-site-data/

"storage" - The storage type indicates that the server wishes to remove locally stored data associated with the origin of a particular response’s url. This includes storage mechansims such as (localStorage, sessionStorage, [INDEXEDDB], [WEBDATABASE], etc), as well as tangentially related mechanism such as service worker registrations.

I'd rather trust my browser/client to do this, just a shame sanitizing IDB by host (or time frame) still isn't possible
Class, discuss!

At a bare minimum, user agents ought to offer the same functionality to users that they offer to web developers.

Clear-Site-Data RFC

Three days is long enough :trollface: Besides, it doesn't really help us (relying on servers).


method setEnableSpeakerphone can not triggered event AudioRouteChanged when the headset is pluged in

Iam looking for a way to switch to Speakerphone or Earpiece when Headset still pluged in. I try method setEnableSpeakerphone() but seem it not work (event AudioRouteChanged not triggered ). Can someone give me some solution? Many thanks

have you followed the doc? https://docs.agora.io/en/Interactive Broadcast/API Reference/react_native/classes/rtcengine.html#setenablespeakerphone

have you followed the doc? https://docs.agora.io/en/Interactive Broadcast/API Reference/react_native/classes/rtcengine.html#setenablespeakerphone
Thank LickKing, yes i see it, so there's no way to switch audio output to other one right? ( in case: Headset Plugged in).
Like some other apps, it can switch to speakerphone while still plugging in headset.

I have the same issue, it should let the user choose the route.


Change wording on index page

Fixes #1.

While I'm a fan of tabs over spaces for indentation, this PR alas mixes them somewhat!

@doismellburning thanks -- fixed!


Add support for generating and downloading reports for DAGs

Is your feature request related to a problem? Please describe.
Building an adhoc web UI (a la cornflow-app) for each optimization problem is very hard and takes a lot of time. Also, once built it's not possible to share the charts/UI with people without access to the web UI.
Describe the solution you'd like
Quarto (https://quarto.org/) reports (*.qmd file) that get compiled with quarto on the server side (in airflow it would seem) into an html, a Word or a Latex pdf.
The file would be self-contained (or not) and be stored in a bucket and served from cornflow as pure html with a download button for a self-contained portable file.
Describe alternatives you've considered
Other report-generation apps such as RMarkdown.
Additional context
Some of the changes required would be [references are for PR 536]:

optional report directory in the dag directory with a collection of reports (cornflow-dags/DAG/tsp/report/report.qmd in PR https://github.com/baobabsoluciones/cornflow/pull/536/files).
a way to tell airflow to generate a report automatically after solve and store it somewhere (bucket). And store the report address in the execution table. Could be a solve config parameter that asks for a report name to be produced.
a way to ask cornflow via the REST API to return the report associated with an execution.
a way in cornflow-app to open and show an html/pdf report.

See the TSP example for more detail: https://github.com/baobabsoluciones/cornflow/pull/536

I'm trying to look for the right place to tell Airflow to generate the report.
Options:

cornflow_client.airflow.dag_utilitied.cf_solve
cornflow_dags.DAG.activate_dags.create_dag

In the first one, we could add an additional piece of code that only runs if the dataset is solved, generates the report and then writes it somewhere.
In the second one, we could create a new airflow task/ operation (PythonOperator?) that reads the execution after the solving part is finished and generates the report as a separate airflow step.
I prefer the second option but I'm not sure how this was done in airflow. I will add a proposal to the TSP PR


Auth and ACL Auth Api timeouts

We are noticing intermittent spikes in timeout errors
logs:
ClientIp@ip:ports [Auth http] Request auth path: /service/v1/emqx/auth, error: timeout
Also noticing this logs whenever timeouts error occure
ehttpc: Received 'gun_error' message from unknown stream ref
I am not sure whether this is due to connection timeout or request timeout, how to figure that out ?
Anything we can do here, currently we are using default settings, will increasing pool size help ? currently for auth + acl auth APIs we have about ~4000 requests per second

https://github.com/emqx/emqx/issues/8566


ER_CANT_CREATE_TABLE: Can't create table 'test.#sql-1_2' (errno: 150)

I have a migration that I use to set up the initial schema. For my tests, I start up a Docker mysql instance and another for my tests to run and link them together through fig. Then I use the migration API to initialize my MySQL tables prior to running my tests.
before(function (done) {
  bookshelf.knex.migrate.latest().then(function() {
    done();
  });
});

after(function(done) {
  bookshelf.knex.migrate.rollback().then(function() {
    done();
  });
});

Migration for tables:
exports.up = function(knex, Promise) {
  return knex.schema.createTable('users', function(t) {

    t.increments('id').primary();
    t.string('username').unique().notNull();
    t.string('email').unique().notNull();
    t.string('password').notNull();
    t.timestamps();
  }).createTable('companies', function(t) {

    t.increments('id').primary();
    t.string('name').unique().notNull();
    t.timestamps();

  });
};

exports.down = function(knex, Promise) {
  return knex.schema.dropTable('companies').dropTable('users');
};

The error I am getting only happens on my Docker instances run from Ubuntu. When I am using boot2docker on my mac, everything works as it should (e.g. tables created, tests run, rollback occurs):
ER_CANT_CREATE_TABLE: Can't create table 'test.#sql-1_2' (errno: 150)
web_1 |     at Query.Sequence._packetToError (/src/node_modules/mysql/lib/protocol/sequences/Sequence.js:48:14)
web_1 |     at Query.ErrorPacket (/src/node_modules/mysql/lib/protocol/sequences/Query.js:82:18)
web_1 |     at Protocol._parsePacket (/src/node_modules/mysql/lib/protocol/Protocol.js:271:23)
web_1 |     at Parser.write (/src/node_modules/mysql/lib/protocol/Parser.js:77:12)
web_1 |     at Protocol.write (/src/node_modules/mysql/lib/protocol/Protocol.js:39:16)
web_1 |     at Socket.<anonymous> (/src/node_modules/mysql/lib/Connection.js:82:28)
web_1 |     at Socket.emit (events.js:95:17)
web_1 |     at Socket.<anonymous> (_stream_readable.js:765:14)
web_1 |     at Socket.emit (events.js:92:17)
web_1 |     at emitReadable_ (_stream_readable.js:427:10)
web_1 |     at emitReadable (_stream_readable.js:423:5)
web_1 |     at readableAddChunk (_stream_readable.js:166:9)
web_1 |     at Socket.Readable.push (_stream_readable.js:128:10)
web_1 |     at TCP.onread (net.js:529:21)
....

Does this error make sense? What would be causing it to try and create that sql table? Any insight on what is happening under the hood would be appreciated? It may be a docker issue but right now I'm thinking it may be something with knex?

First, return the promises in your before and after hooks if you're using Mocha. What you're doing right now won't catch errors properly and will just time out if something goes wrong.
As for that particular error, I can't see how knex would be involved. If you turn on debug and share the queries, happy to help you debug further.

From looking at the MySQL error codes, it looks as though that code/message is referring to a foreign key constraint issue?


Remove --no-deps from pure Python packages

This is no longer in the example staged recipe, as it is no longer required.
See https://github.com/conda-forge/staged-recipes/commit/32435821900329c6d52b9f66c5953bb01a3db50b and
https://github.com/conda/conda-build/issues/3254

Note: I haven't run sphinx yet on these changes so there's a slim chance it might not compile. But I'm fairly certain it will.


[New] jsx-max-depth: validate a specific depth for JSX

Fixes #1219

Is there anything stopping this rule from moving forward? This seems pretty useful to me! 😄
It probably needs a rebase though, at least one file has a conflict.

Happy to rereview after a rebase.

@chriswong any chance you could rebase this?

@ljharb I rebased this on master. How should I submit the changes?

Certainly not with a duplicate PR, which I can never delete from my git history :-( Posting a link to the branch would be more than sufficient.

I've rebased this properly and pushed the changes up to here and to #1676.


Feature Request - Support go install command

Is your feature request related to a problem? Please describe.
In some projects it is possible to use the go install command
go install github.com/<user>/<project>@<revision>

However jx fails because the go.mod file is setting the module as example.com/jx causing this issue
go install github.com/TwoBitCoders/jx@latest
go: downloading github.com/TwoBitCoders/jx v0.0.7
go: github.com/TwoBitCoders/jx@latest: version constraints conflict:
<EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: example.com/jx
                but was required as: github.com/TwoBitCoders/jx

Describe the solution you'd like
I think the go.mod file needs to be updated to this:
- module example.com/jx
+ module github.com/TwoBitCoders/jx

I'm not a go dev, so sorry if this is not as simple as this change.
Describe alternatives you've considered
Clone and build from source. Understandable but I think the go install command is a nicer approach if you just want to install the package.

Fixed in d675ce9. Thank you for this excellent report!

pls add this instruction to installation


404 at /engine/reference/commandline/build/run.md

I found a broken link : https://docs.docker.com/engine/reference/commandline/build/run.md#set-ulimits-in-container---ulimit
I arrived on this page through: https://docs.docker.com/engine/reference/commandline/build/
The link in "flag values" in this part is broken:
https://docs.docker.com/engine/reference/commandline/build/
"Set ulimits in container (--ulimit)
Using the --ulimit option with docker build will cause each build step’s container to be started using those --ulimit flag values."

Thanks for reporting; looks like this was fixed in https://github.com/docker/cli/pull/2780, and the link should now work 👍

Thanks for reporting; looks like this was fixed in https://github.com/docker/cli/pull/2780, and the link should now work 👍


Evaluating perplexity

Hello, I am trying to evaluate my model using the perplexity metric provided in keras-wrapper. However, it seems that nmt-keras was not integrated with this, as I get the following error:
[12/04/2020 13:16:12] Evaluating on metric ppl
Traceback (most recent call last):
  File "train.py", line 271, in <module>
    nmt_model.trainNet(dataset, training_params)
  File "/data/home/pcori/Chatbot/chat-env/lib/python3.6/site-packages/keras_wrapper/cnn_model.py", line 923, in trainNet
    self.__train(ds, params)
  File "/data/home/pcori/Chatbot/chat-env/lib/python3.6/site-packages/keras_wrapper/cnn_model.py", line 1152, in __train
    initial_epoch=params['epoch_offset'])
  File "/data/home/pcori/Chatbot/chat-env/lib/python3.6/site-packages/keras/legacy/interfaces.py", line 91, in wrapper
    return func(*args, **kwargs)
  File "/data/home/pcori/Chatbot/chat-env/lib/python3.6/site-packages/keras/engine/training.py", line 1709, in fit_generator
    initial_epoch=initial_epoch)
  File "/data/home/pcori/Chatbot/chat-env/lib/python3.6/site-packages/keras/engine/training_generator.py", line 255, in fit_generator
    callbacks.on_epoch_end(epoch, epoch_logs)
  File "/data/home/pcori/Chatbot/chat-env/lib/python3.6/site-packages/keras/callbacks.py", line 152, in on_epoch_end
    callback.on_epoch_end(epoch, logs)
  File "/data/home/pcori/Chatbot/chat-env/lib/python3.6/site-packages/keras_wrapper/extra/callbacks.py", line 290, in on_epoch_end
    self.evaluate(epoch, counter_name='epoch')
  File "/data/home/pcori/Chatbot/chat-env/lib/python3.6/site-packages/keras_wrapper/extra/callbacks.py", line 544, in evaluate
    split=s)
TypeError: compute_perplexity() got an unexpected keyword argument 'pred_list'

compute_perplexity() expects arguments y_pred, and y_true, and does not expect extra_vars or pred_list. Is there a way I can still evaluate perplexity?

Oh, that function was a utility that I quickly wrote to evaluate a model. It shouldn't be there. I've removed it.
To evaluate the perplexity of a model, you can pass 'perplexity' as a KERAS_METRIC in config.py. This is now done by default:
https://github.com/lvapeab/nmt-keras/blob/5a29099098e47ff84579b5e8c28c7769c952918c/config.py#L32
PS: it requires to update Keras (pip install  . -U --force-reinstall  --no-cache-dir --no-deps keras)

Got it, thanks. However, it only computes perplexity during training. Is there a way to compute perplexity on the validation set? I would like to have the metric_name here be 'perplexity' rather than sacrebleu:
callbacks.append(PrintPerformanceMetricOnEpochEndOrEachNUpdates(nmt_model,
                                                                dataset,
                                                                gt_id='target_text',
                                                                metric_name=['sacrebleu'],
                                                                set_name=['val'],
                                                                batch_size=256,
                                                                each_n_epochs=20,
                                                                extra_vars=search_params,
                                                                reload_epoch=0,
                                                                is_text=True,
                                                                input_text_id=input_text_id,
                                                                index2word_y=vocab,
                                                                sampling_type='max_likelihood',
                                                                beam_search=True,
                                                                save_path=nmt_model.model_path,
                                                                start_eval_on_epoch=0,
                                                                write_samples=True,
                                                                write_type='list',
                                                                verbose=True))

Thanks again for the help.

Added ().
You can now evaluate with Perpleixty as any other metric, e.g.:
    METRICS = ['sacrebleu', 'perplexity']

You'll need to update multimodal-keras-wrapper and nmt-keras.

Great, thank you.


Link .ty and .o files

actonc produces a .ty and .o file. The .ty file describes the type interface of a module whereas the .o contains the runnable code.
I'm not sure how these can fall out of sync, but there are probably intricate scenarios where they could.
I wonder if we could solve this, either by baking these two files together, like can the .ty stuff be contained in a section in a .o file or something?
Or we could maybe somehow link them. I'm thinking of that sort of cool trick the dsymutil on Mac OS X does. It stamps a UUID into every compiled binary and the debug symbols are also marked with the same UUID, so then finding the correct debug symbols to a particular binary is a piece of cake!

The concept of linking stuff is called BuildID in gcc for linking together an executable with debug symbols...

The way we are now building things using the zig build system, I think we are very much in control of our build and something like this is just not very likely to happen. We won't ship .ty and .o files separately to anyone, it's not really something user visible, so the need for this is pretty much nil. Closing.


Support Xcode16-RC

Fixed build errors occurring in Xcode16-RC.

@b1ackturtle
I have corrected the points raised in your comment.
Thank you again for your review.

@simorgh3196
Thank you for making the changes! Could you please squash all the commits into one.

@b1ackturtle
Completed! Best regards.


Support for AdditionalFiles

Hello.
I propose this pull request as an answer to #1510.
github is unfamiliar, but I hope it helps a little.

thanks, could you add a test for this?

Hello.
I added the AdditionalFiles test.

I think code side looks good, did you test in practise that these changes fixes #1547 ? Based on StyleCopAnalyzers repository documentation it indeed should but didn't test this by myself yet.

Of course, I confirmed that the setting of stylecop.json was reflected.
But that may not be perfect.

Thanks for the contribution @kurobon-jp!

👏


[PE-D][Tester F] Duplicate Suppliers Check

I think currently, to check the uniqueness of suppliers, it is done either by checking if they are the same name or same phone number.
But I think in the real world, it is acceptable for companies to have the same name, especially private companies.
Two people could also have the same phone number, especially if they are from different countries (may not be very common but still possible).
It may be good to make the check stricter? (ie. address and name have to be different to be unique). Just a food for thought.

Labels: severity.Low type.FeatureFlaw
original: sltsheryl/ped#12

Should we keep it as is for now?

Ill just add a check for address too

eh wait feature flaw cannot do

Will move it under feature flaw/not fixing column


remove support for Python 3.8

Related Issue(s):

n/a

Proposed Changes:

Python 3.8 is EOL this year, so there are no new projects that should be using it, and existing projects don't need to update to the next major-minor.
In the last two changes I've tried to make, I've had incompatibilities with 3.8 that have required a workaround, so supporting it is becoming a burden.

PR Checklist:

[X] I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

Could add 3.12?
It's already in there.

It's already in there.

🤦🏼

Also @gadomski, the only reason I was reluctant to remove it was we have this idea that it may be possible to convert an existing container that runs processing (possibly on an older version of Python b/c scientists) into a stactask Task, but I think that's not feasible in most cases, so need to find another way to do that anyway.


Template following expo or CRA?

Currently, it's in between both.
everything in root. this would install more packages than necessary. ex. mobile not selected
https://github.com/expo/expo/tree/master/templates/expo-template-bare-typescript
additional template directory with all the files with template.json
https://github.com/facebook/create-react-app/tree/master/packages/cra-template-typescript

Why move to a template directory?


Unable to remove artifacts after upload scan

https://discord.com/channels/785153694478893126/787340960173391903/965611894931083265

data.zip


Fix tank utility token

Description:
Fix 400 Bad Request on Invalid Token
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

Hi @proferabg,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Please run black code formatter from the project root and commit the changes.
black --fast homeassistant


MSID: 89912 Version: 1 DOI: 2<IP_ADDRESS>0263

"msas": "Cell Biology"
"msid": "89912"
"preprintDoi": "10.1101/2<IP_ADDRESS>0263"
"Reviewed Preprint posted": "2023-10-19"
"Sent for peer review": "2023-07-02"
"Posted to bioRxiv": "2023-07-24" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>0263v1/")
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=89912)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/89912/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>0263)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/89912/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/89912)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/89912

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/89912

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/89912/v1/89912-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/89912)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Under 'Methods' the heading 'If' should be all caps like in bioRxiv, it is not on eLife


split "Table Design Pattern" section on pieces

Please, split "Table Design Pattern" section on pieces. It's too hard to navigate in a giant section.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ba460612-aea2-45da-dacb-ddc8324be954
Version Independent ID: 666c7881-ef9d-1595-0c72-2a03a70d8f9c
Content: Azure Storage Table Design Guide
Content Source: articles/cosmos-db/table-storage-design-guide.md
Service: cosmos-db
GitHub Login: @SnehaGunda
Microsoft Alias: sngun

@SychevIgor Thanks again for your very detailed feedback regarding the documentation. I have assigned this to the content author to evaluate and update as appropriate.

@SychevIgor thanks for the feedback, I have created a DevOps work item in my next sprint (starting last week of this month) to update this doc. Will close this issue here and track it internally.

#please-close


Horizontal Scrolling as default

Hi,
Is there a way to set Horizontal Scrolling as default using viewer.html?
I'm using iframe solution in this way:

I not able to find a config file where to set this as default.
Thank you

Based on https://github.com/mozilla/pdf.js/wiki/Third-party-viewer-usage, you can e.g. set the "scrollModeOnLoad" option the following way:
document.addEventListener("webviewerloaded", function() {
  PDFViewerApplicationOptions.set("scrollModeOnLoad", 1);
});

where the value 1 corresponds to horizontal scrolling, see https://github.com/mozilla/pdf.js/blob/7b0c52fdfed4642366796021093833df63236550/web/ui_utils.js#L44-L49

Thank you really for your reply.
I added this few lines both in the page containing the iframe and in the viewer.html page, but nothing changed. Do I have to place it somewhere else?
<script> document.addEventListener("webviewerloaded", function() {   PDFViewerApplicationOptions.set("scrollModeOnLoad", 1); }); </script>

When opening an issue, please provide all of the information requested in https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md
Also, please see https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md (emphasis mine):

If you are developing a custom solution, first check the examples at https://github.com/mozilla/pdf.js#learning and search existing issues. If this does not help, please prepare a short well-documented example that demonstrates the problem and make it accessible online on your website, JS Bin, GitHub, etc. before opening a new issue or contacting us in the Matrix room -- keep in mind that just code snippets won't help us troubleshoot the problem.

Thanks for your suggestion.
I sorted it out changing the function getDefaultPreferences inside viewer.js.
There is the variable scrollModeOnLoad that I set to 1.
Now it works.
Best,
Luca


FtCLI.exit(Exit) creates an unnecessary custom Object

In the current implementation of the method org.takes.http.FtCLI.exit(Exit) ,
if  the line exit.ready() || custom.ready(); evaluates to true, we will have already incurred the additional overhead costs (computing time & resources) of creating a custom Object that was not necessary to do so in the beginning since it will never used!
The current method implementation needs to be changed to be as follows:
private Exit exit(final Exit exit) {
	final Exit toExit;
	if (exit.ready()) {
		toExit = Exit.ALWAYS;
	} else {
		final long start = System.currentTimeMillis();
		final long max = this.options.lifetime();
		toExit = new Exit() {
			@Override
			public boolean ready() {
				return (System.currentTimeMillis() - start) > max;
			}
		};
	}
	return toExit;
}

The following Object is then added to the interface org.takes.http.Exit
/**
 * Always exit.
 */
final Exit ALWAYS = new Exit() {
	@Override
	public boolean ready() {
		return true;
	}
};

@yegor256 please do something about it, see par.21

@simonjenga I believe it's fixed in e4848a7, thanks


Implement remove on the viewlet

View supports removals but the API is not yet exposed to the particles through the Viewlet API.

need to implement mapping entity IDs (in addition to view IDs) in inner & outer PECs

https://github.com/PolymerLabs/arcs/pull/416


The window session popup only shows the first two windows

When saving more than 2 sessions the list, that pops up when clicking the Window Session icon in the extensions bar, only contains the first two saved windows.
The badge that shows the amount of tabs in the window session does update, but the window itself does not show up in the list.
Clicking it in a window that has not been saved yet does show the input field to name the window. But naming it doesn't cause it show up in the list.
(Windows 10, Chrome 63.0.3239.132 (Official Build) (64-bit) (cohort: Stable))

I know very little about chrome extensions, but it seems that upon inspecting the extensions' background after a fresh start of the browser, it starts with two errors:

Unchecked runtime.lastError while running browserAction.setBadgeText: No tab with id: 241.
at Object.update (chrome-extension://koeeegbghbokaeaikifjhdbbkjbolcep/background.js:17352:30)
background.html:1 Unchecked runtime.lastError while running browserAction.setBadgeBackgroundColor: > No tab with id: 241.
at Object.update (chrome-extension://koeeegbghbokaeaikifjhdbbkjbolcep/background.js:17353:30)

The two lines it refers to (I assume they were transpiled, so line numbers might be useless) are :
        chrome.browserAction.setBadgeText({ text, tabId });
        chrome.browserAction.setBadgeBackgroundColor({ color: config_1.default.badge.color, tabId });

I just noticed that closing a window also throws an error:

Unchecked runtime.lastError while running windows.get: No window with id: 687.
at resolve (chrome-extension://koeeegbghbokaeaikifjhdbbkjbolcep/background.js:17224:28)
at Object.get (chrome-extension://koeeegbghbokaeaikifjhdbbkjbolcep/background.js:17223:16)
at Object.update (chrome-extension://koeeegbghbokaeaikifjhdbbkjbolcep/background.js:17311:37)
at Object.onChanged (chrome-extension://koeeegbghbokaeaikifjhdbbkjbolcep/background.js:17784:32)

It seems to happen in this code:
    /* HELPERS */
    async get(id) {
        return new Promise(resolve => {
            chrome.windows.get(id, { populate: true }, resolve);
        });
    },

Ok, more info:
When trying to add a session (after the initial two) the following error gets thrown after pressing return in the popup's input field:

Unchecked runtime.lastError while running storage.set: QUOTA_BYTES_PER_ITEM quota exceeded
at resolve (chrome-extension://koeeegbghbokaeaikifjhdbbkjbolcep/background.js:17720:33)
at Object.set (chrome-extension://koeeegbghbokaeaikifjhdbbkjbolcep/background.js:17719:16)
at Object.map (chrome-extension://koeeegbghbokaeaikifjhdbbkjbolcep/background.js:17734:26)

Interestingly, the extension icon does get a badge with the (correct) tabs count, it won't appear in the popup though.

Thank you for your messages.
From the last stack trace it seems the maximum storage quota is reached for some reason, I'll have to investigate further on this.
I'm not sure when this will be fixed given that I'm currently pretty busy with other stuff, in case you'd like to submit a PR I'll be happy to review it and hopefully merge it.

I wish I could, but I have 0 working knowledge about Babel and practically no chrome plugin development experience either :/
If I look at the data in chrome.storage.sync though it's name2window key is pretty big and the window2name key contains the windows I wanted to save, but aren't appearing in the json in name2window.

Adding a new window with just one tab in it ("www.google.com") works fine, so I assume the stored name2window just exceed some maximum size limit once I add a new window with actual tabs in it? Without knowing anything about the actual implementation, could it be an idea to store a key for each window and use the window2name lookup to find that key directly instead of in the name2window data?

Did you find time to look into this issue?

@hesselmonk not yet, unfortunately. I've been keeping this tab open on Chrome as a reminder though. If you have the time it would be great if you could try to find the problem and submit a pull request for it, I can't say when I'll have to time for fixing this.

I addressed every problem I could find in v1.1.0, now the extension should actually work. Check out the commit history to learn what changed.


[Ubuntu] Update version of libssl

Description
In scope of this pull request, we update libssl version
Related issue: https://github.com/actions/runner-images/issues/9309
Check list

[x] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run ubuntu2004,ubuntu2294

/azp run ubuntu2204

Seems to be a duplicate of https://github.com/actions/runner-images/pull/9306


"Google account ID not expected" when opening Markdown file for the first time with StackEdit

Never used StackEdit before.

I was logged in to 2 Google accounts
Tried to use StackEdit for the first time to open a .md file in Drive under account 1 (account 0 is the one you are redirected to when you go to a URL such as https://drive.google.com (which will redirect you to https://drive.google.com/drive/u/0/my-drive)
When authorizing StackEdit, I get the error "Google account ID not expected'

Similar to #1324 .

Same issue here.

Can you try to add your 2nd Google Drive account in the sync or publish menu ?

@benweet I can't even reproduce the original issue - either something has changed, or the steps I described above were not sufficient to reproduce it.
Can you still reproduce it, @cjbottaro?

Same issue here, but logged in on just one Google account (with a managed brand account associated)
What additional information may I provide to further investigate?


add html /css

ITP-CODEYOUR FUTURE|SARAAMIRI|Module-User-Focused-Data|WEEK1
NW4 | Carol Owen | HTML-CSS-Module | Week1
Complete the task list below this message.
If your PR is rejected, check the task list.
-->
Learners, PR Template
Self checklist

[x] I have committed my files one by one, on purpose, and for a reason
[x] I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
[x] I have tested my changes
[x] My changes follow the style guide
[x] My changes meet the requirements of this task

Changelist
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.

Hi there @saraamiiri . Thanks for this. Can you update your PR to follow the format asked for? Look on https://github.com/CodeYourFuture/Module-User-Focused-Data/pulls to see what everyone else is doing.

Hi @saraamiiri please also update the PR title itself to match the desired format:
ITP-CODEYOUR FUTURE|SARAAMIRI|Module-User-Focused-Data|WEEK1


Implementing OTP based forgot password frontend + backend

Is your feature request related to a problem? Please describe.
As we are providing users an option to login or signup, what happends if user forgot their password. Theres no way for users to get their account back if they forget their password
Describe the solution you'd like
I would like to implement forgot password functionality, so users can reclaim their account if they forgot their password.
my idea:

user enters their mail id
an otp is sent to user's mail id
user enters the otp
backend verifies the otp
user resets his password

This will add more security and it will be an additional feature, to help users reclaim their account
Additional context
Add any other context or screenshots about the feature request here.

go ahead @Sourabh782


run: automatically generate a container name from image ref

e.g., nerdctl run nginx creates a container named "nginx-62edd".
See pkg/referenceutil/referenceutil_test.go for more examples.
The generated names are more meaningful than Docker/Podman-generated names such as "jolly_kepler". However, the generated names MUST NOT be parsed.
The name can be unset by specifying --name="" explicitly.

Let me rush this in v0.16 🙏


multithreading

Hi, how do I specify using a perl version with multithreading?

Currently, there is no way to specify using multithreading.
If I get a lot of requests, I might implement it.

I'd like to be able to use a perl with thread support too.

https://github.com/marketplace/actions/setup-perl-environment#action-inputs
the multi-thread option is now available 🎉

Thanks!

Thank you!!


Review permissions needed

Regarding the permissions preset in .java.policy_move_to_home_dir:
        permission java.io.FilePermission "${user.dir}/-", "write, read, delete";

Do we really need these permissions on the whole installation directory recursively?
        permission java.io.FilePermission "${user.home}/-", "read";

Looks like only the subdirectory ${user.home}/.m2/-needs to be readable.
There might be other files and subdirectories in ${user.home} which do not need and should not be readable.

Re  permission java.io.FilePermission "${user.dir}/-", "write, read, delete"; : good question . I don't like that one, generally speaking.

I tested a bit around and I think we can restrict the File Permissions further like in 330eff06fe6082f74876c266d97acb8d825a39d0
Do you like this better @dr0i and @Phu2 ? :)

Looks good to me.
This is related to https://github.com/metafacture/metafacture-playground/issues/122 right?

+1 for https://github.com/metafacture/metafacture-playground/commit/330eff06fe6082f74876c266d97acb8d825a39d0


read: allow use of ReadCache without std

Closes #652

Changed to use BTreeMap instead of hashbrown so that it doesn't need a new feature.