add Footer & Header clarification to docs

PR Details

improve documentation related to SetHeaderFooter field use
Description

The existing documentation does not identify the fields to be used when just wanting to set a simple header or footer to the page layout.  It is not obvious to folks who are not familiar with the inner data layout of Excel's data structures that 'OddFooter' and 'OddHeader' are overloaded as the primary fields to be referenced if only using one header and footer in the sheet.
Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] 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

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Codecov Report

Merging #1644 (f47f963) into master (3b2b8ca) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
@@           Coverage Diff           @@
##           master    #1644   +/-   ##
=======================================
  Coverage   98.91%   98.91%
=======================================
  Files          31       31
  Lines       26482    26482
=======================================
  Hits        26194    26194
  Misses        191      191
  Partials       97       97

Flag
Coverage Δ

unittests
98.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

sheet.go
100.00% <ø> (ø)


Add basic GUI tests

Two new tests running in virtual framebuffer, made surprisingly easy by pytest-xvfb :heart: :

Reproduce #71 with xfail mark
Test viewer GUI opens

OK, learned something about #71. This was occurring for me because I was using Python 3.11. Python 3.9 test which reproduces this error XPASS, while 3.10 XFAIL. We get these deprecation warnings in the 3.9 tests:
 test/test_gui.py::TestGui::test_gui_edit_pyfile_opens
  /home/runner/work/viscm/viscm/viscm/gui.py:1157: DeprecationWarning: an integer is required (got type float).  Implicit conversion to integers using __int__ is deprecated, and may be removed in a future version of Python.
    self.max_slider.setValue(viscm_editor.max_Jp)

test/test_gui.py::TestGui::test_gui_edit_pyfile_opens
  /home/runner/work/viscm/viscm/viscm/gui.py:1167: DeprecationWarning: an integer is required (got type float).  Implicit conversion to integers using __int__ is deprecated, and may be removed in a future version of Python.
    self.min_slider.setValue(viscm_editor.min_Jp)

See the commit message of the most recent commit for details about the code change to gui.py!


Add weather.gov alerts for given lat/lon

weather.gov has an endpoint at
https://api.weather.gov/alerts/active?point=, that lists out current alerts.  Need to wait for an alert to parse format since doc at weather.gov leaves a bit to be desired.


"Open containing folder" executes the download file

Just launched Multibit HD OSX and the auto-update installer appeared to download v0.1.0 .dmg. Once download was completed, it brings up the dialog:

Execute installer now
Manually install later

Open Containing Folder

The "Open containing folder" link should open the ~/downloads folder or whatever folder the installer was retrieved into. Instead, it launched the .dmg (mounted the drive).

I think we want to bump this to 0.2 - I don't think it's a quick fix.

Not sure I'm understanding the issue here. On OS X the .dmg contains the latest version of the installer. To run this manually you'd need to open the .dmg you'd just downloaded and double click the installer contained within - the manual option facilitates this by presenting the installer.

Yeah, I'm relatively new to OS X so I was actually not sure if that terminology is standard in installers. In Windows, I would consider "Open Containing Folder" to just send me to the appropriate directory of the received .zip or .exe in Explorer. The same sort of context as this Downloads list in Chrome:

I see where you're coming from. On OS X it's a bit different due to the wrapping performed by the .dmg which is essentially a mounted drive.
I think this can be closed, but thank you for the feedback and please continue to provide more as you explore the application.


pipenv run fails on Windows with non-silent "autorun" scripts

Issue description
On Windows, if you have an "Autorun" script for your Command Processor configured, which is not set up to start with an @echo off, running commands inside the pipenv with pipenv run will fail.
Expected result
Autorun scripts do not block running commands.
Actual result
Traceback (most recent call last):
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\lietu\AppData\Local\Programs\Python\Python36-32\Scripts\pipenv.exe\__main__.py", line 9, in <module>
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\vendor\click\core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\vendor\click\core.py", line 717, in main
    rv = self.invoke(ctx)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\vendor\click\core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\vendor\click\core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\vendor\click\core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\vendor\click\decorators.py", line 64, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\vendor\click\core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\cli\command.py", line 408, in run
    command=command, args=args, three=state.three, python=state.python, pypi_mirror=state.pypi_mirror
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\core.py", line 2328, in do_run
    do_run_nt(script)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\core.py", line 2267, in do_run_nt
    p = _launch_windows_subprocess(script)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\site-packages\pipenv\core.py", line 2257, in _launch_windows_subprocess
    return subprocess.Popen([command] + script.args, **options)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\subprocess.py", line 729, in __init__
    restore_signals, start_new_session)
  File "c:\users\lietu\appdata\local\programs\python\python36-32\lib\subprocess.py", line 1017, in _execute_child
    startupinfo)
FileNotFoundError: [WinError 2] The system cannot find the file specified

Steps to replicate
Open up regedit and set up e.g. the key HKEY_CURRENT_USER\Software\Microsoft\Command Processor\Autorun with a value pointing to a script, in my case %USERPROFILE%\init.cmd.
Set up this Autorun script with some contents that DOES NOT start with @echo off. In my case:
DOSKEY ls=ls -haF --color=always

Try to run any commands via pipenv run. In my case this was trying to run an invoke task via the command pipenv run invoke test.
Known workaround
Prefix the script with @echo off so making the above script be like this:
@echo off
DOSKEY ls=ls -haF --color=always

Debugging steps
I checked the backtrace and put a print above the subprocess.Popen on core.py:2257, saw it was putting a line showing the current path + the DOSKEY ... -command as the command. Checked how system_which resolves it and realized it uses the first line of output instead of e.g. the last one, so when you launch a new shell to run this command with, the first line will have the first line command from the Autorun script, and likely only the second to last line will have the path to invoke.exe in my case.

The relevant pieces of pipenv --support below:
$ pipenv --support
Pipenv version: '2018.11.26'
Pipenv location: 'c:\\users\\lietu\\appdata\\local\\programs\\python\\python36-32\\lib\\site-packages\\pipenv'
Python location: 'c:\\users\\lietu\\appdata\\local\\programs\\python\\python36-32\\python.exe'
Python installations found:

3.6.8: C:\Users\lietu\AppData\Local\Programs\Python\Python36-32\python.exe

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.6.8',
 'os_name': 'nt',
 'platform_machine': 'AMD64',
 'platform_python_implementation': 'CPython',
 'platform_release': '10',
 'platform_system': 'Windows',
 'platform_version': '10.0.17763',
 'python_full_version': '3.6.8',
 'python_version': '3.6',
 'sys_platform': 'win32'}

The issue and the debugging steps make sense to me, would you mind send a PR for it?

@lietu Note that where command on Windows will give multiple results. That said, using the last line will change the behavior.
So I suggest you avoid using non-silent autorun scripts since it is kind of hidden feature on Windows only. What's your thought?

Definitely the most important part is to for now recommending avoiding of the non-silent autorun scripts, as they're causing issues.
There's two ways that I see could possibly solve this:

See if there's an alternative way to execute the command, that doesn't trigger the autorun scripts
Loop through the results and return the first line which is a path to an existing file - checking if it's executable should be extraneous and is kinda more difficult on Windows iirc

Can this be rechecked on pipenv==2022.8.19?


Update tomcat-catalina, tomcat-coyote to 9.0.34

Updates

org.apache.tomcat:tomcat-catalina
org.apache.tomcat:tomcat-coyote

from 9.0.33 to 9.0.34.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.tomcat" } ]

labels: library-update, semver-patch

Superseded by #3336.


how to provide overrides needed for model compatibility

While transformers BERT is not yet fully working, we are getting closer.
However, there is the need to disable some data-dependent control flow (typically: checks) to get it to work.
Transformers itself hides some but not all behind a check for compiling, e.g.
https://github.com/huggingface/transformers/blob/0fdea8607d7e01eb0e38a1ebeb7feee30a22f0cf/src/transformers/modeling_attn_mask_utils.py#L256-L260
is_tracing = (
    torch.jit.is_tracing()
    or isinstance(inputs_embeds, torch.fx.Proxy)
    or (hasattr(torch, "_dynamo") and torch._dynamo.is_compiling())
)

So here is a candidate for how we might currently be able to run BERT after fixing #805 (and subsequent bugs):
import transformers, thunder, torch
@thunder.core.jit_ext.register_general_jit_lookaside(
    transformers.modeling_utils.PreTrainedModel.warn_if_padding_and_no_attention_mask
)
@thunder.core.jit_ext.interpreter_needs_wrap
def dummy(*args):
    pass

@thunder.core.jit_ext.register_general_jit_lookaside(
    torch._dynamo.is_compiling
)
@thunder.core.jit_ext.interpreter_needs_wrap
def is_compiling():
    return True

m = transformers.BertForSequenceClassification(transformers.BertConfig())
inp = torch.randint(1, 20, (1, 32))
jm = thunder.jit(m)
jm(inp)

We might submit an issue to transformers to check for is_tracing in transformers.modeling_utils.PreTrainedModel.warn_if_padding_and_no_attention_mask
but in general I wonder how to provide the such compatibility lookasides to users.
possible variants:

A compat executor,
detecting the module we are tracing and adding them just for transformers (too much magic),
always having the torch._dynamo.is_compiling-lookaside?
...

PyTorch is working on providing a public interface to queries like "Is Dynamo active now?", but it's returning just False for now:
https://github.com/pytorch/pytorch/blob/5c78581fc91aa50673790a7f591294a19b489e20/torch/compiler/__init__.py#L234

triage review:

we need some sort of magic to support transformers because they investigate input data unless inside dynamo
do we ask the user to do something before the transformer, or as a jit arg?
or do we try to detect that we're in transformers and do something?
maybe we just have a lookaside for 'is dynamo active'?
dynamo devs are creating a more public interface for querying whether or not we're in that compiler
would be great to participate in their interface design here
let's reach out to Meta, @tfogal to figure out a person here


change doc to reflect proxyv2

being default image
https://github.com/istio/istio/pull/5741

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: linsun
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [linsun]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: andraxylia, linsun
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [linsun]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[Feature] Handle starting from secondary tiles

Pretty simple, just gotta parse the tile arguments to suss out the Place information

Done.


pkgsMusl.colord: fix build

Description of changes

We can't build the daemon in pkgsMusl, because rustc does not support dynamic Musl targets, which means we can't build Spidermonkey, which means we can't build Polkit.  But it still makes sense to support the rest of colord, because an application should still be able to link against it to talk to a colord daemon built in some other way (e.g. provided by a different distro).
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Doesn't build of OfBorg due to having recently acquired a transitive dependency of gperftools, which is currently broken on Musl.  Will be fixed by #179102.

Sigh. OfBorg lib tests have failed due to a curl test failure. Let's try again.
@OfBorg eval


jest.mock should override original flow types

Do you want to request a feature or report a bug?: bug
What is the current behavior?
When using jest.mock, Flow doesn't understand that the imported and mocked file is different from the original import.
If the current behavior is a bug, please provide the steps to reproduce and either a repl.it demo through https://repl.it/languages/jest or a minimal repository on GitHub that we can yarn install and yarn test.
import {fn} from './file.js'

jest.mock('./file.js', () => {
  return {
    fn: jest.fn(() => {}),
  }
})

// $FlowError: property `mockClear`: Property not found in statics of function
fn.mockClear()

What is the expected behavior?
Somehow either flow or Jest flow typing should be able to override the original types.
Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.
jest: 20.0.4
flow-bin: 0.50

we'll be redesigning our mock system to be more compatible with flow.
Most likely it'll look something like
impart {fn} from './file.js';
jest.mock.clear(fn);

so we don't magically mutate/patch any objects in runtime, but rather provide a separate API to extract this information from mocked functions

@joeybaker one of the workarounds is to do something like:
(fn: any).mockClear()

I think I found the most suitable solution (or rather, my colleague suggested it to me):
let mockFn: JestMockFn<[], object>
jest.mock('./file.js', () => {
  mockFn = jest.fn(() => {})
  return {
    fn: mockFn,
  }
})

mockFn.mockClear()


Link badges to PyPI and the license

So that clicking them takes you to PyPI and the license instead of just badge images.
Preview:
https://github.com/hugovk/pytest-watcher/blob/011962f6712170c4128d79b62a997adc4b5a1777/README.md

Hey,
That's awesome!
Thanks a lot!


feat(@schematics/angular): Allow generation of suffix-free files

Allow generation of suffix-free file and symbol names.
Fix #16811

@alan-agius4 / @clydin
Any advice on ci/circleci: build-bazel  FAIL?
ci/circleci: flake-jail is getting passed on local but not on github.

@mgechev, can you give this a quick look? We're not sure exactly what you were expecting from the design perspective here. Is this what you had in mind?

We haven't discussed design yet. My suggestion would be to deprecate the type option (in v10) and replace it with a suffix one, which accepts an empty string. With @clydin we discussed few alternatives and this seems like the cleanest one.
If you see corner cases, we can draft a design document.

@mgechev / @dgp1130  Should I close this PR?

Something that can be fixed for the current version is the type option on several of the schematics.  When setting the type to an empty string the file names generated will contain an extra period (.).  Fixing this would act as an enabler for the proposed name change to suffix in a later version.

Spoke with @mgechev and @jelbourn, they both agree Minko's design of converting type to suffix and accepting an empty string is the way to go. So I think we have consensus on the design side.
I think we can follow @clydin's suggestion to allow an empty string type to and go from there.

@dgp1130 If nobody started working on it yet then I can do the mentioned changes for "type"

@sacgrover, I don't believe anyone else has started on this, so you're good to go!

@dgp1130  I have opened another PR for this. https://github.com/angular/angular-cli/pull/17040

Closing as the work for this is progressing in the above mentioned PR.


Issues when setting wrapWithLayout

When using this plugin and wrapping the pages in a layout using Gatsby wrapWithLayout, the component exported is not wrapped inside <IntlProvider>.
resulting in

I believe the problem is that the plugin should rather use wrapRootElement instead of wrapPageElement as according to Gatsby docs
This is useful to set up any Provider components that will wrap your application. For setting persistent UI elements around pages use wrapPageElement.
I have seen this issue also on other plugins tbh, not sure if there is any limitation to use the suggested function

my bad I meant to write wrapPageElement.
so currently the plugin is using wrapPageElement to add the providers, instead gatsby suggests to use wrapRootElement and keep wrapPageElement for UI

I think the same issue is also on https://www.gatsbyjs.com/plugins/gatsby-plugin-react-i18next/ and I found an article about a workaround for that particular plugin https://andremonteiro.pt/gatsby-i18next-wrap-page-element/

Do you have your own react-intl provider somewhere?

I  cannot reproduce the problem, but I'm more than happy to help as soon as I can :)

I solved this same issue using the snippet below on gatsby-browser.js
export const wrapPageElement = ({ element, props }) => {
    return (
        <IntlProvider locale={navigator.language}>
            <Layout {...props}>{element}</Layout>
        </IntlProvider>
    )
}

Actually the plugin should inject this:
https://github.com/openscript-ch/gatsby-plugin-i18n-l10n/blob/main/src/wrapPageElement.tsx
with
https://github.com/openscript-ch/gatsby-plugin-i18n-l10n/blob/main/gatsby-ssr.ts
On which version of Gatsby are you? @armandoborge

It is a private repository.
The issue occurs if the gatsby-browser.js file contains an export wrapPageElement, in my case, I am using this function export to wrap all my pages inside a layout component. As soon as I remove my function the error disappears.
It seems the custom declaration overwrites the one created by the plugin.


Game broken

When attempting to access this at https://githubschool.github.io/github-games-youzihao1016, I am getting a 404. This could be caused by a couple things:

GitHub pages needs to be enabled on master. You can fix this in the repository settings.
the index.html file is incorrectly named inde.html. We will fix this together in class.

Can you please fix the first bullet @youzihao1016?

I got a 404, too. And I haven't found how to go to the repository setting yet.


Sample module doesn't help promote good security in tblgatewaylog

Currently the return value for a capture (such as in the function merchantgateway_capture) shows as including the raw response from the payment gateway like this:  'rawdata' => $responseData, at merchantgateway.php:287
That raw data is then dumped by WHMCS to tblgatewaylog.
Many gateways return a bunch of data so that the requesting system (in this case the gateway module) can validate the request. This return data often includes the credit card number.
To ensure the sample module encourages best practices for storing of credit card data (as in, not storing credit card data inadvertently in this instance), and because it's not explicitly shown in the sample module that the 'rawdata' var in the array will be entirely stored in the gateway log, I believe the sample module should include something similar to what you see below. This will help indicate that the developer should use (and how best to use) proper security practices prior to passing the raw response data back to WHMCS for storage in the gateway log.
/*If the response data contains a credit card number (check the gateway API documentation to confirm), it's important to mask the credit card number before WHMCS inserts it unencrypted into the gateway log. Here's an example where the API returns an array key of 'CreditCardNumber': */
$responseData['CreditCardNumber'] = '****' . substr($responseData['CreditCardNumber'], -4);

I've also posted a variation of this here, because it wasn't clear which sample module is the currentlly active one -- it looks like this one is more complete, so this seems to be the better spot to discuss things / report issues.

Hi there,
Thanks for taking the time to open an issue.
As I understand it, you are working with a payment gateway that is returning a full unmasked credit card number in the API response you receive to a payment request for capture.
I can't recall a time I've ever seen a payment gateway return sensitive information such as a full credit card number in plain text format. It would not only be very unusual to see something like this but also a significant security concern with the gateway itself. If you were to find a payment gateway API that was doing something like this, I would suggest contacting that gateway and reporting it to them as a security concern at the very least.
That said, if there is sensitive information contained within the response data, then masking it or removing it entirely would certainly be appropriate before passing it to logging. However from our experience, API response data containing sensitive data is rare and also very gateway specific, and that's why the sample code we provide does not cater to this at this time. We can consider adding some kind of inline note within the code, however, it would need to be something generic enough to make sense in all scenarios.
I will also add that it is always expected that our sample module code will be used in conjunction with our developer documentation where it is stated that any data returned will be logged (ref https://developers.whmcs.com/payment-gateways/merchant-gateway/).
Thanks,
Matt


feat: Support for auth customization & automatic refreshing

Resolves #966

Codecov Report

Merging #1462 (72c57fc) into master (3e4e162) will decrease coverage by 0.17%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1462      +/-   ##
==========================================
- Coverage   99.13%   98.95%   -0.18%
==========================================
  Files          86       87       +1
  Lines        6679     6706      +27
  Branches     1171     1174       +3
==========================================
+ Hits         6621     6636      +15
- Misses         30       38       +8
- Partials       28       32       +4

Impacted Files
Coverage Δ

src/schemathesis/__init__.py
100.00% <100.00%> (ø)

src/schemathesis/auth.py
100.00% <100.00%> (ø)

src/schemathesis/schemas.py
99.39% <100.00%> (+<0.01%)
:arrow_up:

src/schemathesis/specs/openapi/_hypothesis.py
100.00% <100.00%> (ø)

src/schemathesis/_compat.py
70.37% <0.00%> (-7.41%)
:arrow_down:

src/schemathesis/extra/pytest_plugin.py
95.00% <0.00%> (-5.00%)
:arrow_down:

src/schemathesis/cli/callbacks.py
98.57% <0.00%> (-1.43%)
:arrow_down:

src/schemathesis/utils.py
97.78% <0.00%> (-0.89%)
:arrow_down:

Continue to review full report at Codecov.

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


Add config to enable/disable read/writes to non-default buckets

For enhanced security it would be best if the ability to read or write to an arbitrary bucket can be controlled using config settings.
E.g. add to config.yml:
allow_any_bucket_read: true
allow_any_bucket_write: false
On AWS or Azure it's certainly possible to do the equivalent based on policy settings, but the above config will be a good default for those who inadvertently use credentials that allow read/write access to any bucket in their account.

This is checked into master.
The default config is:
allow_any_bucket_read: true
allow_any_bucket_write: true
for backward compatibility since I didn't want anyone to have mysterious failures with the update.


New rule for integration with gql-tag-operations preset of GraphQL-Codegen

const Doc = gql(/* GraphQL */ `
  query A {
    user {
       id
       login
     }
  }
`)

const MyComponent = () => {
  const data = useQuery(Doc)

   data.id
}

// would raise User.login field is unused in code...

@B2o5T We can follow it to some extend and recommend people to use fragments for defining component dependencies.

See https://github.com/relayjs/eslint-plugin-relay/blob/main/test/unused-fields.js


When installing k3s server on multiple centos9 machines, <IP_ADDRESS> cannot be accessed

I created two centos9 virtual machines on the same network segment under pve, namely <IP_ADDRESS> and <IP_ADDRESS>, and installed two independent k3s servers on the two virtual machines. It was found that only one of the two independent clusters could be started successfully. Through investigation, it was found that <IP_ADDRESS> was affected. If the two clusters set the same apiserver address <IP_ADDRESS>, host 2 would be able to access host 1 https://<IP_ADDRESS>:443 causes k3s of host 2 to fail to start. I tested the installation under Ubuntu and there was no such problem. The problem occurred 100% under Centos9. Centos9 does not have iptables installed by default. Could this be related?
我在pve下创建两个两个同网段的centos9虚拟机，分别为<IP_ADDRESS>和<IP_ADDRESS>，并在两个虚拟机上安装了两个独立的k3s server。发现两个独立集群只有一个可以成功启动，通过排查发现<IP_ADDRESS>影响到了，如果两个集群设置了相同的apiserver地址<IP_ADDRESS>，会导致主机2可以访问主机1的https://<IP_ADDRESS>:443的地址，从而导致主机2的k3s无法启动。我在ubuntu下测试安装没有这个问题，centos9下就100%出现问题，centos9默认没有安装iptables，会不会跟这个有关系？
K3s Version: v1.27.4+k3s1
os version: CentOS-Stream-9
k3s check-config

cat: /sys/kernel/security/apparmor/profiles: No such file or directory
Verifying binaries in /var/lib/rancher/k3s/data/b20b68fab02222c85f08adc9d865ac2052f2f2f831a883e3b42654cf04acf31c/bin:

sha256sum: good
aux/ip6tables: symlink to xtables-nft-multi
aux/ip6tables-restore: symlink to xtables-nft-multi
aux/ip6tables-save: symlink to xtables-nft-multi
aux/iptables: symlink to xtables-nft-multi
aux/iptables-restore: symlink to xtables-nft-multi
aux/iptables-save: symlink to xtables-nft-multi
links: good

System:

/var/lib/rancher/k3s/data/b20b68fab02222c85f08adc9d865ac2052f2f2f831a883e3b42654cf04acf31c/bin/aux iptables v1.8.8 (nf_tables): ok
swap: disabled
routes: ok

Limits:

/proc/sys/kernel/keys/root_maxkeys: 1000000

modprobe: FATAL: Module configs not found in directory /lib/modules/5.14.0-366.el9.x86_64
info: reading kernel config from /boot/config-5.14.0-366.el9.x86_64 ...
Generally Necessary:

cgroup hierarchy: cgroups V2 mounted, cpu|cpuset|memory controllers status: good
CONFIG_NAMESPACES: enabled
CONFIG_NET_NS: enabled
CONFIG_PID_NS: enabled
CONFIG_IPC_NS: enabled
CONFIG_UTS_NS: enabled
CONFIG_CGROUPS: enabled
CONFIG_CGROUP_PIDS: enabled
CONFIG_CGROUP_CPUACCT: enabled
CONFIG_CGROUP_DEVICE: enabled
CONFIG_CGROUP_FREEZER: enabled
CONFIG_CGROUP_SCHED: enabled
CONFIG_CPUSETS: enabled
CONFIG_MEMCG: enabled
CONFIG_KEYS: enabled
CONFIG_VETH: enabled (as module)
CONFIG_BRIDGE: enabled (as module)
CONFIG_BRIDGE_NETFILTER: enabled (as module)
CONFIG_IP_NF_FILTER: enabled (as module)
CONFIG_IP_NF_TARGET_MASQUERADE: enabled (as module)
CONFIG_NETFILTER_XT_MATCH_ADDRTYPE: enabled (as module)
CONFIG_NETFILTER_XT_MATCH_CONNTRACK: enabled (as module)
CONFIG_NETFILTER_XT_MATCH_IPVS: enabled (as module)
CONFIG_NETFILTER_XT_MATCH_COMMENT: enabled (as module)
CONFIG_NETFILTER_XT_MATCH_MULTIPORT: enabled (as module)
CONFIG_IP_NF_NAT: enabled (as module)
CONFIG_NF_NAT: enabled (as module)
CONFIG_POSIX_MQUEUE: enabled

Optional Features:

CONFIG_USER_NS: enabled
CONFIG_SECCOMP: enabled
CONFIG_BLK_CGROUP: enabled
CONFIG_BLK_DEV_THROTTLING: enabled
CONFIG_CGROUP_PERF: enabled
CONFIG_CGROUP_HUGETLB: enabled
CONFIG_NET_CLS_CGROUP: enabled
CONFIG_CGROUP_NET_PRIO: enabled
CONFIG_CFS_BANDWIDTH: enabled
CONFIG_FAIR_GROUP_SCHED: enabled
CONFIG_RT_GROUP_SCHED: missing
CONFIG_IP_NF_TARGET_REDIRECT: enabled (as module)
CONFIG_IP_SET: enabled (as module)
CONFIG_IP_VS: enabled (as module)
CONFIG_IP_VS_NFCT: enabled
CONFIG_IP_VS_PROTO_TCP: enabled
CONFIG_IP_VS_PROTO_UDP: enabled
CONFIG_IP_VS_RR: enabled (as module)
CONFIG_EXT4_FS: enabled (as module)
CONFIG_EXT4_FS_POSIX_ACL: enabled
CONFIG_EXT4_FS_SECURITY: enabled
Network Drivers:

"overlay":

CONFIG_VXLAN: enabled (as module)
Optional (for encrypted networks):

CONFIG_CRYPTO: enabled
CONFIG_CRYPTO_AEAD: enabled
CONFIG_CRYPTO_GCM: enabled
CONFIG_CRYPTO_SEQIV: enabled
CONFIG_CRYPTO_GHASH: enabled
CONFIG_XFRM: enabled
CONFIG_XFRM_USER: enabled
CONFIG_XFRM_ALGO: enabled
CONFIG_INET_ESP: enabled (as module)
CONFIG_INET_XFRM_MODE_TRANSPORT: missing

Storage Drivers:

"overlay":

CONFIG_OVERLAY_FS: enabled (as module)

STATUS: pass

node2 curl https://<IP_ADDRESS>:443 -k

{
"kind": "Status",
"apiVersion": "v1",
"metadata": {},
"status": "Failure",
"message": "Unauthorized",
"reason": "Unauthorized",
"code": 401
}

cc @brandond

from the curl seems that node2 can reach <IP_ADDRESS> and you get unauthorized because you need to use the right token to get the access to the api server. Are you trying to create two different cluster or a cluster with two servers?

two different cluster
------------------ Original ------------------
From: Roberto Bonafiglia @.>
Date: Tue,Nov 28,2023 11:30 PM
To: k3s-io/k3s @.>
Cc: 微擎团队 @.>, Author @.>
Subject: Re: [k3s-io/k3s] When installing k3s server on multiple centos9machines, <IP_ADDRESS> cannot be accessed (Issue #8822)
from the curl seems that node2 can reach <IP_ADDRESS> and you get unauthorized because you need to use the right token to get the access to the api server. Are you trying to create two different cluster or a cluster with two servers?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Are you starting K3s with a custom configuration or default one?

[root@localhost ~]# cat /etc/systemd/system/k3s.service
[Unit]
Description=Lightweight Kubernetes
Documentation=https://k3s.io
Wants=network-online.target
After=network-online.target
[Install]
WantedBy=multi-user.target
[Service]
Type=notify
EnvironmentFile=-/etc/default/%N
EnvironmentFile=-/etc/sysconfig/%N
EnvironmentFile=-/etc/systemd/system/k3s.service.env
KillMode=process
Delegate=yes
Having non-zero Limit*s causes performance problems due to accounting overhead
in the kernel. We recommend using cgroups to do container-local accounting.
LimitNOFILE=1048576
LimitNPROC=infinity
LimitCORE=infinity
TasksMax=infinity
TimeoutStartSec=0
Restart=always
RestartSec=5s
ExecStartPre=/bin/sh -xc '! /usr/bin/systemctl is-enabled --quiet nm-cloud-setup.service'
ExecStartPre=-/sbin/modprobe br_netfilter
ExecStartPre=-/sbin/modprobe overlay
ExecStart=/usr/local/bin/k3s
server
'--write-kubeconfig-mode'
'644'
'--tls-san'
''
'--node-external-ip'
''
'--system-default-registry'
'registry.cn-hangzhou.aliyuncs.com'
'--kubelet-arg=image-gc-high-threshold=70'
'--kubelet-arg=image-gc-low-threshold=60'


Merge Brandwatch changes

This PR merges the changes from the fork made by Brandwatch (https://github.com/brandwatch/migrate-ld-cc).

Sorry, I opened the PR in the wrong fork. Closing it.


Fix CodeQL config error

https://github.com/scabench/l4j-tp1/actions/runs/7453693386/job/20279601718 shows a failing GitHub CodeQL run using Autobuild. It doesn't like JDK17:
Error: 1-08 22:02:54] [autobuild] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project l4j-tp1: Fatal error compiling: error: invalid target release: 17 -> [Help 1]

It's confusing that Autobuild is running at all, since the latest commit https://github.com/scabench/l4j-tp1/commit/a4e07f918696616bb77a18ab13acb8ea5728843a happened before this run, and that commit overrides Autobuild with a manual mvn compile command.

Indeed at the time a4e07f9 was committed, there was a different run that failed with a different error: https://github.com/scabench/l4j-tp1/actions/runs/7452906671/workflow
Using the nice actionlint, it turns out the problem was just bad YAML indentation.


last_played doesn't fill in correctly for first time players

When a player joined the server for the first time, for as long as he hasn't logged off yet, the tag<EMAIL_ADDRESS>will not return the proper information.
Wherever the data is stored, it should also be updated on login and not just logout, to avoid that issue

Check if player is currently online first before using it?

That's not how it fails. The player can be online and this tag could still fail or atleast return incorrect data. When the player logs on for the first time, if you then try to use the tag on him for as long as he is online and has not logged of yet, it'll return incorrectly. i believe it returns d@0t (making my server thing he's been inactive since january 1970). This data is already updated every single time the player logs out, i doubt it'll be that much more intense if it also updates when the player logs in.

Oh, I misinterpreted your request, I thought you meant constantly update the value, not on login only!
:+1:

:+1:

would seem we rely on a spigot function for this.
http://puu.sh/kZqpT.png
I believe, then the best solution would be, if the player is currently online, return the current time rather than lastPlayed.
Consider if that would be acceptable, @BlackCoyote ?

that would probably be a way better solution in various ways than what I initially asked


Improve perf of the router

app-router diff: https://github.com/blasten/app-router/commit/fddbf96f4e7b84eed774f2f5bc1560afec4699b9
Dependency:

[ ] https://github.com/PolymerElements/iron-component-page/pull/56

Changes LGTM. Deferring to @frankiefu for a second review.

@frankiefu I cleaned up the console.log https://github.com/blasten/app-router/commit/d9c50a833d6d22923b50ba06b83f855350b67a17

LGTM :+1:
Let's merge after https://github.com/PolymerElements/iron-component-page/pull/56 is merged and released.


[storage-blob-preview] az storage blob upload: support jumbo block blob

The maximum size supported for a block blob increases from 4.75 TiB to 190.7 TiB (50,000 blocks, each block from 100MB to 4000MB).

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[x] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

storage-blob-preview

storage-blob-preview


Add Unsupported Map type

Just because aya doesn't support working with some map types doesn't mean we should't allow them to be loaded.
Add a new Unsupported map type to facilitate this, also add a debug log to let the user know we just loaded and unsupported map type.

Yep I agree, let me work on adding this!

Ok I think this should do it @dave-tucker.
The way I had to re-write the parse_map function as an impl of bpfLoader in order to get access to the new allow_unsupported_maps field within it was a bit tricky....so If you want me to redo that just let me know
Also are there existing tests for these bpfLoader impl functions? I couldn't find any.

@alessandrod for the API changes:

BpfLoader::allow_unsupported_maps()
And the functional change, which is:
If you attempt to load bytecode which uses an unimplemented map type, you get BpfError::UnsupportedMap
If you set BpfLoader::allow_unsupported_maps() then we allow the bytecode to be loaded anyway, even if the map can't be interacted with from the Aya code. This is to support usecases like bpfd, where we're loading code on behalf of others, but aren't interacting with their maps.

/assign @alessandrod

Shoot that won't work and I still can't assign folks here :(

Shoot that won't work and I still can't assign folks here :(

Just sent you an invite


Improve Display of Page Width and Height

The dimensions of a letter-sized page from Get Info on PDF are reported as:
Width: 612
Height: 792
This seems to be in pixels and based upon 72 ppi which is probably not what people are expecting to see.  Perhaps it can be displayed in cm and inches in addition to pixels along with the ppi value.
Additionally, the "Standard Size" value is "Custom" whereas it should be Letter.
Thank you @Frooodle for developing this wonderful project.

ahh let me round these numbers actually


Feature/xhub 5328/exoplayer extension for supplemental properties

Thanks for opening a PR!
Since this is a volunteer project and is very active, anything you can do to reduce the amount of time needed to review and merge your PR is appreciated.
The following steps will help get your PR merged quickly:
Update the documentation
If you've added new functionality, update the README.md with an entry for your prop or event.
The entry should be inserted in alphabetical order.
Update the changelog
After you open the PR, update the CHANGELOG.md file with an entry pointing to your PR.
Provide an example of how to test the change
If the PR requires special testing setup provide all the relevant instructions and files. This may include a sample video file or URL, configuration, or setup steps.
Focus the PR on only one area
Testing multiple features takes longer than isolated changes and if there is a bug in one feature, prevents the other parts of your PR from getting merged until it gets fixed.
If you're touching multiple different areas that aren't related, break the changes up into multiple PRs.
Describe the changes
Add a note describing what your PR does. If there is a change to the behavior of the code, explain why it needs to be updated.

misclick on target branch
original: https://github.com/miquido/react-native-video/pull/16


emp:: block free error

I am using emp:: block in oblivious transfer and I am setting up of a list of emp::block instance using new, like block* ot_msg = new block[100]
However, the pre-allocated space is facing a free error using delete[] after I transfer the data inside block *ot_msg via NetIO. The error shows invalid pointer is referred for p.
Why is this happening?

unrelated to SEAL


Update tutorial-managed-virtual-network-on-premise-sql-server.md

Added instruction on +x permissions on the script. suggestion to run the script as service

@zmustafa : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

This change has been merged now.  Thank you for your contribution to make the docs better!
#please-close


Refactor WrappedPromise construction

This doesn't include exposing the wrap function to Reason, nor do I think it needs to at this time.

Thanks @ckknight!
@ncthbrt we are not optimizing for performance (yet), but we may indeed have to inline function calls, etc., in the future. And thanks again for your earlier change :)


fix: The last "Authorise" button on the onboarding now working

Hi everyone!
The "Authorise" button in the Consent screen (on the onboarding) was visually broken.
Actually, the build method of the "real" Homepage was called… but never displayed.
The main issue is that the onboarding is mixing Navigator v1 and v2 calls.
Thus, the onboarding is considered as the "/" route, which is the same as the "real" homepage.
The trick is to have a difference between the two screens + manually clear the stack.

Codecov Report
Attention: Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.

Project coverage is 6.94%. Comparing base (4d9c7fc) to head (f64852a).
Report is 379 commits behind head on develop.

Files with missing lines
Patch %
Lines

...ib/pages/onboarding/onboarding_flow_navigator.dart
0.00%
8 Missing :warning:

.../smooth_app/lib/pages/navigator/app_navigator.dart
0.00%
6 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5723      +/-   ##
==========================================
- Coverage     9.54%   6.94%   -2.60%
==========================================
  Files          325     404      +79
  Lines        16411   21615    +5204
==========================================
- Hits          1567    1502      -65
- Misses       14844   20113    +5269

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


ZEPELIN-3586. Use hadoop 2.7.3 for embedded spark

What is this PR for?
Simple PR to use hadoop 2.7.3 for the embedded spark, otherwise user will hit weird issue if they read csv file or parquet file due to low hadoop version api.
What type of PR is it?
[Bug Fix ]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-3586?filter=-2

How should this be tested?

Ci pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

LGTM


schedule is ignored

Package ecosystem
npm
Package manager version
yarn 1.22.1
Language version
node 12
Manifest location and content prior to update
./package.json
dependabot.yml content
https://github.com/mui-org/material-ui/blob/02d6348a22fe8825f7f3ef0b34c33a74f4fd2ba3/.github/dependabot.yml
Updated dependency
Bump next from 10.2.0 to 11.0.1
What you expected to see, versus what you actually saw
No update at this time. The PR was opened on Wednesday when updates are scheduled for Sunday/Monday
Native package manager behavior
N/A
Images of the diff or a link to the PR, issue or logs
https://github.com/mui-org/material-ui/pull/26904/files
🕹 Bonus points: Smallest manifest that reproduces the issue

It looks like the config file was changed around that time, which will trigger an update, per the docs. I'm also seeing a few errors and timeouts, which could have pushed things back a few hours (or a day).
Closing as WAI, but please re-open if you're seeing other behavior that is different than what is described above or in the docs.

It looks like the config file was changed around that time

Could you clarify what you mean by "changed"? The contents wasn't changed since we upgraded: https://github.com/mui-org/material-ui/commits/148b0737e2d14c8940a9e08bf27226c984e16323/.github/dependabot.yml
We've been getting upgrades a couple of times over the past few days without ever touching the config..
It's unfortunate that this is yet another instance where the dependabot team closes an issue prematurely.
yarn v2 support dropped, no grouping, no concurrency limit. Now the schedule is no longer respected. We'll be migrating to Renovate. Thanks to all the people that were working on dependabot prior to the GitHub acquisition.

Sorry you feel that way. I'll admit that we're pretty underwater on the issue backlog, so we've been trying to resolve what we think are straightforward issues faster, and sometimes we're on the wrong side of that.
W.r.t. "config file changes", we'll re-trigger an update if the manifest file or the dependabot.yaml is updated (since in the former case, potentially a new dependency was added we should try to update, or a problematic one was removed thus we can succeed), or in the latter, there may be a new ecosystem, new ignore conditions, etc.
It looks like /docs/package.json has been updated several times in the past few days, and one of those I'm seeing is attempting to update next from 10.2 to 11.0 (the above PR). It looks like https://github.com/mui-org/material-ui/commit/7759e3f687caf3852a8f0ed673bfc18d915ea563#diff-adfa337ce44dc2902621da20152a048dac41878cf3716dfc4cc56d03aa212a56 is the PR that caused us to kick off the job.
I recognize that this behavior might not be super intuitive: do you have any suggestions for how we might make this more obvious?


Add docs on terraform 'override' option

Add a documentation page on terraform and how and when to use the 'override' option

updated docs which can be found here:
https://panhandler.readthedocs.io/en/master/examples/terraform_skillet.html#example-terraform


Connection.close() does not await connection close

Using spring-data-r2dbc v1.0.0.BUILD-SNAPSHOT with rdbc 0.8.0.RC1 dependencies on spring-boot-starter-parent 2.2.0.M6.
I have configured an r2dbc connection to postgresql by extending AbstractR2dbcConfiguration.  I have also enabled repos with @EnableR2dbcRepositories.
After moderate usage, I begin getting error messages stating "sorry, too many clients already" (full gist: https://gist.github.com/joshfix/5db5bacf2b65b018d8ecc75486bce9d8).
It is recommended to use the connection pool, as the connection close does not await a response of the disconnect message towards Postgres.  Per Mark Paluch, this should be improved upon, although it's unclear to me what actual improvements need to be made, but it was recommended that I start an issue, so here it is :)

Not sure if this is relevant, but here is how I was constructing my ConnectionFactory:
@Configuration
@AllArgsConstructor
public class PostgresConfig extends AbstractR2dbcConfiguration {

    private final DataConfigProps configProps;

    @Bean("postgresConnectionFactory")
    @Override
    public ConnectionFactory connectionFactory() {
        PostgresqlConnectionConfiguration.Builder builder = PostgresqlConnectionConfiguration.builder()
                .host(configProps.getHost())
                ...);
        return new PostgresqlConnectionFactory(builder.build());
    }
}

@joshfix can you attach a bit of code that shows the functionality where you experience the exception? After revisiting the Postgres protocol documentation, I'm no longer sure that the initial assumption renders true.

Sure, this is a bit ugly and unwieldy, but it's what i have :)
    public void init() {
        queuedItemFlux = Flux.interval(Duration.ZERO,
                Duration.of(configProps.getInterval(), configProps.getIntervalUnit()))
                // get all customers
                .flatMap(l -> customerService.get())
                // get all jobs for the customer
                .flatMap(customer -> {
                    log.debug("got customer: " + customer.toString());
                    return Mono.just(customer);
                })
                .filter(customer -> customer.getJobIds() != null)
                .flatMap(customer -> {
                    log.debug("customer jobs: " + Strings.join(customer.getJobIds().iterator(), ','));
                    return Flux.fromIterable(customer.getJobIds());

                })
                .flatMap(jobService::getById)
                // execute the saved search for the current job
                .flatMap(job -> client.runSavedSearch(job.getSearchId())
                        // turn the list of returned features into a flux
                        .filter(itemPage -> itemPage.getFeatures() != null)
                        .flatMapMany(itemPage -> {
                            log.debug("Search generated " + itemPage.getFeatures().size());
                            return Flux.fromIterable(itemPage.getFeatures());
                        })
                        // group the items by item type
                        .collectMultimap(item -> item.getProperties().getItemType(), item -> item.getId())
                        // get an entry set from the
                        .filter(productMap -> productMap.entrySet() != null)
                        .flatMapMany(productMap -> {
                            log.debug("product map has " + productMap.size() + " items");
                            return Flux.fromIterable(productMap.entrySet());
                        })
                        // remove any items that have already been ordered
                        .flatMap(entry -> Flux.fromIterable(entry.getValue())
                        //.flatMap(entry -> itemService.getNotContains(job.getId(), entry.getValue())
                                // put the items in the queued_item table
                                .flatMap(id -> {
                                    Item item = new Item()
                                            .id(id)
                                            .jobId(job.getId())
                                            .type(entry.getKey())
                                            .queuedDate(new Date())
                                            .status(Status.PENDING);

                                    log.debug("built item: " + item.toString());
                                    return itemService.create(item);
                                })
                                .count()
                                .flatMap(count -> {
                                    SearchTask searchTask = new SearchTask()
                                            .createdDate(new Date())
                                            .id(UUID.randomUUID().toString())
                                            .status(Status.PENDING)
                                            .jobId(job.getId())
                                            .uniqueResults(count.intValue());
                                    return searchTaskService.create(searchTask);
                                })
                                .flatMap(searchTask -> jobService.update(job.addSearchTask(searchTask.getId())))
                        )
                );

    }

May I assume that jobService::getById and client.runSavedSearch(…) interact with the database? Can you switch from flatMap(…) to concatMap(…) and test whether you see a change in behavior?

All of the xxxService objects end up calling the repos and interacting with the db.  The client class does not, it ends up making a call to WebClient.  The error would kick in during the flatMap to create a new Item and call return itemService.create (which in turn just calls repo.save(item)).  There were about 250 new items being created.  You can see that debug output in my gist (link in original post).
Also, everything was fine for quite a while and the issue started appearing after a couple hours of development, then it wouldn't go away until I restarted my postgres container.
I have since changed the way I create the ConnectionFactory to:
    public ConnectionFactory connectionFactory() {
        // ref: https://github.com/r2dbc/r2dbc-pool#usage
        return ConnectionFactories.get(ConnectionFactoryOptions.builder()
                .option(DRIVER, "pool")
                .option(PROTOCOL, "postgres") // driver identifier, PROTOCOL is delegated as DRIVER by the pool.
                .option(HOST, configProps.getHost())
                ...
         }

and haven't experienced the issue again so far.

I assume that flatMap(…) fan out in a higher concurrency rate than your server is able to handle. concatMap(…) reduces parallelity to 1, therefore, it reduces the number of required connections.

Ok good to know... I will make sure whenever we are in a spot where we expect a large number of db calls that we use concatMap.

Both, driver and pool have a work queue, but reducing parallelity is something that can prevent lockup. As the issue seems resolved, I'm going to close it.


Added AWS beginners course

Fixes #643
Added and updated AWS beginners course.
Hope this solves the issue.
Looking forward for merging this PR.

AWS category is already created in the readme file , please add at that place and also add in json file


Update setup.cfg to address security alerts

Update various Python dependency versions.

Note that this upgrades pillow to >=10.2.0 cc @teetone @michiyasunaga @JosselinSomervilleRoberts


Allow comments in security group specifications

As security group specifications become fine grained, the specification file becomes harder to understand  without any comments to show the intend of each rule.
For example, the security group applied on one of my spaces look like this:
[
  {"protocol":"tcp","destination":"<ip-foo>","ports":"3306"},
  {"protocol":"tcp","destination":"<some-ip>-<another-ip>","ports":"55882"},
  {"protocol":"tcp","destination":"<ip-bar>","ports":"443"}
]

There is no easy way to understand the intend behind each rule.
Allowing a comment field will greatly improve this. Like so:
[
  {"protocol":"tcp","destination":"<ip-foo>","ports":"3306", "comment": "Allow database connection to PostgreSQL at hosted-postgres-service.com"},
  {"protocol":"tcp","destination":"<some-ip>-<another-ip>","ports":"55882", "comment": "Allow logging to hosted-logging-service.com"},
  {"protocol":"tcp","destination":"<ip-bar>","ports":"443", "comment": "Allow monitoring service at hosted-monitoring-service.com"}
]

As things stand, if I were to attempt this, I will get the following error:
Server error, status code: 400, error code: 300001, message: The security group is invalid: rules rule number 1 contains the invalid field 'comment', rules rule number 2 contains the invalid field 'comment', rules rule number 3 contains the invalid field 'comment'

This is as intended.
The comment field should be marked as a valid field.

Hi @sdqali That seems like a good idea. I've asked our PM to prioritize a story in our backlog to get this feature done. We'd also be happy to merge in any pull request with this functionality!
Thanks!
Zak, CF Runtime Team

Thank you, @zaksoup. I have a patch I will push shortly.

Just noticed that I need to sign the CLA. Running it past my employer. :-)

Hi @sdqali,
Another option is to keep comments in your manifest.
This is still valid yaml, so it shouldn't cause any problems.
[
  # Allow database connection to PostgreSQL at hosted-postgres-service.com
  {"protocol":"tcp","destination":"<ip-foo>","ports":"3306"},
  # Allow logging to hosted-logging-service.com
  {"protocol":"tcp","destination":"<some-ip>-<another-ip>","ports":"55882"},
  # Allow monitoring service at hosted-monitoring-service.com
  {"protocol":"tcp","destination":"<ip-bar>","ports":"443"}
]

Hi @sdqali
Just wanted to follow up about the patch you said you might have?

Hi @sdqali,
We have added this as a story.
Thanks for bringing this to our attention,
@Quintaminant and @utako, CF CAPI Team


test(ticdc): skip kafka test for integration test clustered_index temporarily

What problem does this PR solve?

Issue Number: close #4727
What is changed and how it works?
Remove integration test clustered_index temporarily since kafka test fail
Check List
Release note
`None`.

/run-check-issue-triage-complete

Codecov Report

Merging #4728 (37e7b02) into master (9607554) will decrease coverage by 0.3911%.
The diff coverage is 53.4028%.

Flag
Coverage Δ

cdc
59.9065% <53.4028%> (-0.0158%)
:arrow_down:

dm
51.3600% <ø> (-0.6689%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4728        +/-   ##
================================================
- Coverage   55.6402%   55.2490%   -0.3912%
================================================
  Files           494        518        +24
  Lines         61283      63954      +2671
================================================
+ Hits          34098      35334      +1236
- Misses        23750      25128      +1378
- Partials       3435       3492        +57

/run-dm-compatibility-test
/run-dm-integration-test
/run-kafka-integration-test
/run-integration-test

/run-dm-compatibility-test
/run-dm-integration-test
/run-kafka-integration-test
/run-integration-test

/run-leak-test

/run-kafka-integration-test

/merge


Not all animations are exported

Describe the bug
There are 6 animations in this file but only 5 get exported
Pig.blend.zip
Same file from https://github.com/KhronosGroup/glTF-Blender-IO/issues/433
To Reproduce
Steps to reproduce the behavior:

Load into blender
File->Export->gltf2.0
Check to sample animations
Export

Expected behavior
6 Animations are exported

In particular of those 6, "Run" is missing
Version

OS: macOS 10.14.5
Blender Version 2.79

We are currently exporting, for each object:

active action
action from NLA tracks

You're "run" action is not the active one, and is not in any NLA track.
We are not exporting all actions, because some actions are not linked to the object. Maybe in the future will can add a check on each action to check if this action is linked to current object.

I see.
Just as a reference I imported the file into Unity and Unreal and both import all 6 animations so maybe importing all 6 is what users are more used to?

Just as a reference I imported the file into Unity and Unreal and both import all 6 animations so maybe importing all 6 is what users are more used to?

This isn't really possible. Unused Actions in Blender are not associated with any particular object, and can't be exported to glTF in isolation... The requirement to export only active Actions, NLA Tracks, and stashed Actions allows the exporter to identify which Actions are associated with which objects.

I stashed all of my actions, and the ones that were missing from the export before I stashed them are still missing from the export after I stashed them.

@SoapSpangledGames please attach a blend file.  We can't investigate without your test case

Thank you very much for your quick response. I appreciate it. I've attached my Blender file with all of the animations.
alpha dragon stashed animations.zip

In your file:

Some armature animation are linked to mesh object, so these animations can't be run, they are not exported
Seems that only FlapWings.001 to FlapWings.005 are missing, but this is empty action, so it's not exported
14 other actions are exported

So ... What did you expect to be exported and is currently not?

HappyDance
KnockedBackFlying
Perched
VictoryPump

These actions are not stashed to armature object.
All your action are bone animations, so they need to be stashed to your rig object (MetaRig), not to your mesh object (Cube)
You can see that all bone animations linked to not armature object are wrong (see the red line under each channel):

Thank you so much! I stashed all of the animations to my metarig, and they all exported.


Show monsters aggro area

Show monsters' aggro area, (Show the area of which you can not step out of, or the monsters become aggro again.)

Yes this is something I wanted to ask also. It's interesting how some monsters have very different aggro area/attack range/aggro range inside that area.

Can be closed, as this is now in the "game". (NPC Aggression Timer -plugin)

You should be able to close this issue yourself. But yes, this has been implemented.


Timeout in restart_homepage script

The restart_homepage.py needs some tinkering:
http://docs.python-requests.org/en/master/user/quickstart/#timeouts
But I won't do a push before it's alive again
note to self: change line 12 to html = requests.get("http://chaosthebot.com/", timeout=30).text
Will create PR later

Using requests in this manner just throws an exception if the server is down and restarts the whole bot, so it needs to be wrapped in try ... except anyway.

Leaving this for somebody else to pick up... Not really my specialty 😉
And who knows... maybe this is already fixed?
/cc @amoffat


Timestamp convertor support millisecond and fix time zone

What's the Problem?

millisecond support;
fix time zone(even switched between window, sometime the app detected the clipboard and changed the time zone).

Solution/Idea

add the support
add setting to chose default time zone and fix it.

Alternatives
nope
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

DevToys Version
Version <IP_ADDRESS> | X64 | RELEASE | 8bcfa60 | 8bcfa60
Comments
No response

By default, it is set to your computer's time zone.
Would you like to change it to keep the last selected?

Duplicate with #683


Add RSpec/Its

This cop would be disabled by default.
If you use rspec-its then you can write
subject { foo }

its(:bar) { should be_valid }

With this cop enabled it would enforce the following:
Bad
subject(:thing) { foo }

it 'has a valid bar' do
  expect(thing.bar).to be_valid
end

Good
subject(:thing) { foo }

its(:bar) { should be_valid }

I've a couple of concerns here:

I'd consider enabling this by default. The RSpec team explained why its is problematic.

When I do see rspec-its used, it tends to be a mix of its and regular it family examples. I'm not sure from your example if the cop would allow that?

Interesting point. I'll need to think more before considering implementing this.


Wrong namespace for related models with different namespaces

Versions:

ide-helper Version: 2.10.0
Laravel Version: 8.49.2
PHP Version: 8.0.8 (Laravel Sail: 1.8.4)

Description:
In my project the models are divided into multiple namespaces:

\App\Models
\App\Models\Pivots
\App\Models\Cashier
...

IDE Helper is generating docs as expected apart from relations.
Lets say we have the \App\Models\User and the \App\Models\School model. Also since a user can be training at different schools they are related M:N via the \App\Models\Pivots\SchoolUser pivot model with relations school -> \App\Models\School and user -> \App\Models\User.
When parsing relations IDE helper seems to assume, that all share the same namespace. Therefore the namespace for the related model is omitted in the _ide_helper_models.php. The \App\Models\Pivots\IdeHelperSchoolUser mixin will have a property user -> User and school -> School. This is absolutely ok if they all are in the same namespace. Since in my case the namespaces are different my IDE cant find the referenced Models (\App\Models\Pivots\User and \App\Models\Pivots\School don't exist) and fails to properly hint types/properties/methods.
Steps To Reproduce:

Setup fresh Laravel project with IDE Helper
Create Models in the \App\Models namespace
Create Pivots for a N:M relationship within the \App\Models\Pivots namespace
Create relations on the pivot model to its related models
Run php artisan ide-helper:models or sail artisan ide-helper:models
Look at the helper mixin for the Pivot Model, the namespace for the related models is omitted though its needed in this case

Note: im talking about martial arts schools, obviously pupils would not be attending multiple schools. ;)
Suggestion: always use fully qualified namespaces, even if a short one might be sufficient. This might solve the problem already.

I've started to run into this more in one of my projects where I have been more aggressive with namespacing my models. I may try to find a fix for it.

I think this can be fixed by using force_fqn in config/ide-helper.php. I've enabled that and it fixed the current case of not referencing the namespaced class. It isn't clear what other impact this might have and curious why this isn't the default?

I think because people don't like the full namespace as it clutters the code more. But if it's an issue, you can create a PR to change the default.

@barryvdh It is unclear to me if it's an issue if there is an option to change it. It just wasn't immediately obvious to me that it was an option that existed at all until I dived into the ide helper code directly.

I think because people don't like the full namespace as it clutters the code more.

Do you think the "clutter" worry is an older/outdated concern? I wonder if the force_fqn default maybe made more sense before there were more options to hide the generated code away from the models themselves? If you think this is likely, I'd be more inclined to send a PR to change the default. :)
Either way, if force_fqn is not true, could we find a way to let the user know? Perhaps a message like, "Detected models related to models in a different namespace. Please set force_fqn to true if your IDE is not able to resolve the related model classes on its own."
Happy to work on this from either or both directions. :)


Spotify tracks won't play on Firefox

Known issue.
https://community.spotify.com/t5/Spotify-for-Developers/Web-Playback-SDK-not-working-in-Firefox/td-p/5391485

Fixed on 6/17/22


Storyshots

Get Storyshots working to get snapshot tests from our stories automagically.

Test work locally. No idea why Heroku CI is unhappy 😦. Investigating.

Tests pass and this is ready for review.


[ci] Update lewagon/wait-on-check-action to latest version.

Similar to flutter/packages/pull/461, this PR updates the version of lewagon/wait-on-check-action on the release workflow so it waits for more than 30 checks.
Fixes: flutter/flutter#88120
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

LGTM, can land when the new hash is white listed by @godofredoc

LGTM, can land when the new hash is white listed by @godofredoc

It is added already to the allowlist.

Correct, this version is already added to the allow-list; this is the internal bug that @godofredoc and I are using to coordinate this: b/199300032


Release the binary for darwin_arm64

│ Provider registry.terraform.io/a10networks/thunder v1.1.0 does not have a
│ package available for your current platform, darwin_arm64.

Please provide provider binaries for darwin_arm64.
Update:
I managed to build it locally, but would be good if I would not have to.
gh repo clone a10networks/terraform-provider-thunder
cd terraform-provider-thunder
dos2unix scripts/*.sh
find . -name \*.go|xargs dos2unix
make build

$ uname -prv
21.6.0 Darwin Kernel Version 21.6.0: Sun Nov  6 23:31:13 PST 2022; root:xnu-8020.240.14~1/RELEASE_ARM64_T6000 arm

https://github.com/a10networks/terraform-provider-thunder/commit/e744ce3d4823903afb3842a76981d3297d15f359


Better Documentation and Examples (Especially Query Related Documentation and Examples)

For an external person have a glance at this project it is difficult to say how data is to be queried and also how to get started.

You're absolutely right. The project status is also not very well reflected. Check out the ambling document to get a feel for it.
The main input file format for icicle is Zebra, but psv is supported as well.


add docs for uploading files using multer for v0.13.2

Would like to use Multer for file upload in new wasp version but am having issues running gists in documentation—they are now reported as outdated in v0.13.2.
Help with this issue would be much appreciated.

existing docs:
https://docs.opensaas.sh/guides/file-uploading/#using-multer-to-upload-files-to-your-server


debian packaging improvements

Hope that helps.

I'm not sure if the libola-dev dependency is a good idea.
We've got automated linux builds on OBS (OpenSuse Build System) and OLA is problematic.
I already dedicated some efforts to include the OLA plugin in the OBS builds, but in the end I wasn't able to succeed.
When that happens, we can add libola-dev to the dependencies

Hi @mcallegari and thanks for your answer
Just so you know, the package I've been building for my LibraZiK project (ie: 4.10.5 as of yesterday) is built with this build dependency and everything looks fine from a packaging point of view (ie: it did deb-compiled and I didn't see anything strange in the build description).
LibraZiK is basically a debian "jessie" stable.
I can, remove the build libola-dev dependency if you want and then, you can merge the rest of this commit if it's all good for you. Just let me know.

LibraZiK is basically a debian "jessie" stable.

There you have your answer. In a jessie distro, OLA is officially present and supported. On OBS it is not.

I can, remove the build libola-dev dependency if you want and then, you can merge the rest of this commit if it's all good for you. Just let me know.

Yes, please remove the ola dependency for now, and I'll merge the rest. Thanks.
During the xmas holidays I'll put some more effort to build the OLA plugin on OBS as well.
In case you're interested, this is where it happens: https://build.opensuse.org/project/show/home:mcallegari79

Yes, please remove the ola dependency for now, and I'll merge the rest. Thanks.

Done.

During the xmas holidays I'll put some more effort to build the OLA plugin on OBS as well.
In case you're interested, this is where it happens: https://build.opensuse.org/project/show/home:mcallegari79

I don't know about OBS. I'll have a look when I free up some time.

:+1:


[Feature][task plugin] Add new task plugin for apache linkis

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Linkis builds a computation middleware layer to decouple the upper applications and the underlying data engines, provides standardized interfaces (REST, JDBC, WebSocket etc.) to easily connect to various underlying engines (Spark, Presto, Flink, etc.), while enables cross engine context sharing, unified job& engine governance and orchestration.
the product's official website is https://linkis.apache.org, we should add new task plugin to support apache links, which is a some of easy task, for any new contributor or first-time contributor is welcome
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Let me try it.

Let me try it.

thanks for your help, I had assigned to you, looking forward your contributing

Let me try it.

Hello, I'm PPMC PeaceWong from linkis, we can communicate if you have any questions


Request for specific λ settings for solider, loot, redandblack, and longdress during training at different bitrate points

Hello,
I have been reviewing your paper and was particularly interested in the experiments involving the solider, loot, redandblack, and longdress. However, I couldn't find the specific settings of the λ (lambda) values for the different bitrate points used during training.
Could you kindly provide the details or point me towards the section where I can find the λ values for these datasets at the different bitrate points? This information would greatly assist in replicating the results for my own experiments.
Thank you for your time and for making your work available.
Best regards,
Vettel

Hello,
Sorry that I didn't specify the settings of λ in the paper. We keep λf and λg identical for each bit rate and set their values as 1, 5, 10, 50 to attain different bit rates.
Hongning

Hello,
Sorry that I didn't specify the settings of λ in the paper. We keep λf and λg identical for each bit rate and set their values as 1, 5, 10, 50 to attain different bit rates.
Hongning

Thanks a lot for your prompt reply! I will try to use the lambda you give to reproduce!


Power Download buttons

We should explore having buttons for downloads, could be Globus focused first (then add wget)
Figure out how to deal with data from multiple nodes, do we limit it to one node or try to remove replica datasets? Not an issue at first
Also entire cart (if more than 100 datasets)
Added bonus to build out directory structure but need to confirm if this is practical to do with Globus tools

(1) Button enabled for whitelisted user
(2) Button performs file count check on frontend
(3) displays warning for files > N
(4) passes criteria to backend for file search for Globus transfer or wget script

Latest thought on this.  Lets not support wget power downloads.  Instead we should integrate esgpull which is much more robust than the wget script templates..


[Release-1.22] etcd-snapshot subcommand does not out debug logs even when --debug flag is provided

Backport https://github.com/k3s-io/k3s/issues/4920 to release-1.22

Validated using commitid 354477adbfbd596828a56ce9882b081629321ed8 from branch release-1.22

Multiple debug logs are now passed when passing the --debug flag to the snapshot command. See below snippet of the command and the start of the logs which now contain DEBU for debug outputs.

~$ sudo k3s etcd-snapshot save --debug
WARN[0000] Unknown flag --write-kubeconfig-mode found in config.yaml, skipping
WARN[0000] Unknown flag --tls-san found in config.yaml, skipping
WARN[0000] Unknown flag --token found in config.yaml, skipping
INFO[0000] Managed etcd cluster bootstrap already complete and initialized
INFO[0000] Applying CRD addons.k3s.cattle.io
DEBU[0000] DesiredSet - Patch apiextensio


OpenAPI definitions incorrectly encode array and object parameters

According to the OpenAPI specification, a parameter like
      - name: messages
        in: query
        description: An array containing the IDs of the target messages.
        schema:
          type: array
          items:
            type: integer
        example: [4, 8, 15]
        required: true

should actually serialize as messages=4,8,15 rather than messages=[4, 8, 15]. Since we’re expecting the latter, this parameter should have been defined as
      - name: messages
        in: query
        description: An array containing the IDs of the target messages.
        content:
          application/json:
            schema:
              type: array
              items:
                type: integer
        required: true

I’m not sure whether whether the example should be written [4, 8, 15] or "[4, 8, 15]" (or perhaps it should be [4, 8, 15] but indented to be a child of schema).
The current test_openapi doesn’t accept this, so it needs to be fixed to accept the correct definition and ideally reject the incorrect one.

This might be something @Hypro999 could fix fairly quickly.
We would probably need to change render_table in zerver/lib/bugdown/api_arguments_table_generator.py as well as fixing any tests here.
Or it's possible that we can do any transformations entirely within get_openapi_parameters.
We may want to make a shared schema for all the places we pass arrays of ids, under components.

I'll look into it today :slightly_smiling_face:

I’m not sure whether whether the example should be written [4, 8, 15] or "[4, 8, 15]" (or perhaps it should be [4, 8, 15] but indented to be a child of schema).

Well, based on the specification, Content should be a mapping from string to a single Media Type Object. The Media Type Object will contain a schema along with an example both at the same level. So going with the very first suggestion of keeping the example un-quoted and at the same level as schema (instead of under it) seems like it's not a bad idea.
One bit of ambiguity however comes from this line:

Furthermore, if referencing a schema which contains an example, the example value SHALL override the example provided by the schema.

So in theory the example could fit under the schema level as well. But it's not needed.
So I think that we should ultimately go with:
      - name: messages
        in: query
        description: An array containing the IDs of the target messages.
        content:
          application/json:
            schema:
              type: array
              items:
                type: integer
        example: [4, 8, 15]
        required: true

We would probably need to change render_table in zerver/lib/bugdown/api_arguments_table_generator.py as well as fixing any tests here.

Just a note, apart from these, we'd also have to take care of get_openapi_param_example_value_as_string in zerver/lib/bugdown/api_code_examples.py and maybe even some other places.

https://github.com/zulip/zulip/issues/14629

@orientor can you investigate this issue next?

I think this issue is fixed. As discussed in the example validator PR,
we would use
- name: messages
        in: query
        description: An array containing the IDs of the target messages.
        content:
          application/json:
            schema:
              type: array
              items:
                type: integer
            example: [4, 8, 15]
        required: true

When we want message="[4,8,15]" but zulip wants message = [4,8,15] which is what is currently being returned. And the example validator is also not showing any error. So I think the issue is solved.
@andersk ?

Fixed how? The messages parameter is still specified incorrectly on master, and the examples validator is incorrectly accepting it.

Ok so the example should be example: 4, 8, 15 according to openapi?

I think I see where your confusion is. Look at it this way. If the parameter’s underlying content were a string, we would have

messages=foo if it’s not JSON-encoded at all,
messages="foo" if it’s JSON-encoded once,
messages="\"foo\"" if it’s JSON-encoded twice.
messages="\"\\\"foo\\\"\"" if it’s JSON-encoded thrice.

But it’s not a string, it’s an array, so we have

messages=4,8,5 if it’s not JSON-encoded at all (because this is a query parameter, and the default style for query parameters is form, and array values with style form encode like this),
messages=[4,8,5] if it’s JSON-encoded once,
messages="[4,8,5]" if it’s JSON-encoded twice,
messages="\"[4, 8, 5]\"" if it’s JSON-encoded thrice.

I should clarify that example: [4, 8, 15] is a correct example—this is just a YAML array that says nothing about how the array should be serialized. The problem is that we need to specify that we expect it serialized as JSON, and the test suite fails to verify that our specification matches this expectation.

Ok, now I get the issue. I should probably read up on JSON.

This was mostly resolved in #15607, but I think there's valuable follow-up work to do here.
On the validators side, ideally, we'd be able to change how message_to is encoded to avoid having something that might be JSON or a string and handle its encoding and validation via REQ so that test_openapi can validate it properly.

With this logging patch:
diff --git a/zerver/tests/test_openapi.py b/zerver/tests/test_openapi.py
index cd8f71bdf4..bbfd9adc94 100644
--- a/zerver/tests/test_openapi.py
+++ b/zerver/tests/test_openapi.py
@@ -553,6 +553,7 @@ do not match the types declared in the implementation of {function.__name__}.\n"
                     # with later.  In either case remove the variable
                     # from `json_params`.
                     if vtype == str:
+                        print(function.__name__, vname)
                         json_params.pop(vname, None)
                         continue
                     else:

we can see that there are only a handful of parameters where this logic is active:
update_user_backend full_name
send_message_backend to
json_change_notify_settings notification_sound
update_stream_backend description
update_stream_backend new_name

And I think on send_message_backend would not pass if we deleted json_params, so I think we can clean up the validation hack added as part of resolving this if we just fix to in send_message_backend to have its type correctly expressed.


Missing documentation for populating media field with multiple values

I've got a content type with a field containing multiple images. When I examine an entry of that type, it looks like so:
array(2) {
  [0] =>
  array(1) {
    'sys' =>
    array(3) {
      'type' =>
      string(4) "Link"
      'linkType' =>
      string(5) "Asset"
      'id' =>
      string(22) "7Iu3xRVvOOK9WTkpPD5JTC"
    }
  }
  [1] =>
  array(1) {
    'sys' =>
    array(3) {
      'type' =>
      string(4) "Link"
      'linkType' =>
      string(5) "Asset"
      'id' =>
      string(22) "2sX3nNprxD22HHzWcx5iB1"
    }
  }
}

My problem is that when I upload a file, create and publish an asset for it, and add the asset's Link to the entry's field array, using setField(), saving the entry gives me "422 Unprocessable Entity".
What am I doing wrong?

Thanks for noticing this! I have updated the documentation. The proper way to push existing assets is to use the Link object, see below:
`$images = [
new \Contentful\Core\Api\Link('2PngieRxZeL854z0IDPZho2', 'Asset'),
new \Contentful\Core\Api\Link('4O0kU2nI3PM2ot6tTWnVJJd', 'Asset'),
];
$entry = new \Contentful\Management\Resource\Entry('product');
$entry->setField('name','en-US','product name');
$entry->setField('productImages', 'en-US', $images);
$environmentProxy->create($entry);`
I am going to close this issue, feel free to re-open if you need more help.


Typo in generation of loop.

I have a loop in Java as follows:
for(int iter = 0 ; iter < 5 ; iter++) {
                    Card currentCard = shuffledDeck.get(0);
                    shuffledDeck.remove(0);
                    dealCard(cardBack, playerContainer, cards.getImage(currentCard.getFileName()), currentCard);
                    currentCard = shuffledDeck.get(0);
                    shuffledDeck.remove(0);
                    dealCard(cardBack, rivalContainer, cards.getImage("card_back.png"), currentCard);
                }

At runtime, this loop runs for 6 iterations instead of 5.  I found that the issue is that it generated code like:
...
case 6:
        ccu_Container187_addComponent667(b, a);
        if ($rt_suspending()) {
            break $main;
        }

        if (0 < 5) {
            e = ju_ArrayList801_get69($this.val$shuffledDeck1757, 0);
            ju_ArrayList801_remove70($this.val$shuffledDeck1757, 0);
            f = $this.this$01755;
            b = $this.val$playerContainer1754;
            c = ccuu_Resources1851_getImage827(ccdp_Poker1283_access$1001057($this.this$01755), ccdp_Poker$Card742_getFileName1058(e));
            $ptr = 17;
            continue $main;
        }
        g = new ccu_TextArea1377;
        d = $rt_s(988);
        $ptr = 7;
        continue $main;
...

case 17:
        ccdp_Poker1283_access$2001059(f, a, b, c, e);
        if ($rt_suspending()) {
            break $main;
        }
        f = ju_ArrayList801_get69($this.val$shuffledDeck1757, 0);
        ju_ArrayList801_remove70($this.val$shuffledDeck1757, 0);
        e = $this.this$01755;
        c = $this.val$rivalContainer1752;
        d = ccuu_Resources1851_getImage827(ccdp_Poker1283_access$1001057($this.this$01755), $rt_s(987));
        $ptr = 18;
        continue $main;
    case 18:
        ccdp_Poker1283_access$2001059(e, a, c, d, f);
        if ($rt_suspending()) {
            break $main;
        }
        i = i + 1 | 0;
        if (i < 5) {
            e = ju_ArrayList801_get69($this.val$shuffledDeck1757, 0);
            ju_ArrayList801_remove70($this.val$shuffledDeck1757, 0);
            f = $this.this$01755;
            b = $this.val$playerContainer1754;
            c = ccuu_Resources1851_getImage827(ccdp_Poker1283_access$1001057($this.this$01755), ccdp_Poker$Card742_getFileName1058(e));
            $ptr = 17;
            continue $main;
        }
        g = new ccu_TextArea1377;
        d = $rt_s(988);
        $ptr = 7;
        continue $main;

The problem being on the line if (0 < 5) {
I can manually fix this problem by changing it to:
i = i | 0
if (i<5){

I don't currently have this project on GitHub.  Is this enough information for you to track this down, or should I post the project?
Thanks
Steve

This problem is the same as https://github.com/konsoletyper/teavm/issues/87 Working on it

Thanks for fixing #87.   After that fix, I'm still seeing this issue.  I have uploaded the project to github.
https://github.com/shannah/cn1-teavmport-demos
(The Poker project).


Movie and time: Testing - Create test plan

As a user I want to select a movie, time and select a seat

[ ] Create test plan

the user should be able to see all movies, select a movie, choose a date and time for that movie, choose a seat for the movie

when selecting a movie and the given movie does not exist, reask.


do not set default squeeze params in InvSqueeze

The normal decode flow is:

input image has N channels
it does Squeeze with default (empty) parameters
at header decode, MetaSqueeze is called which will fill in the Squeeze params with default values, to get O(N log N) channels
after channel decode, InvSqueeze is called which undoes the Squeezes (it now has actual parameters, since MetaSqueeze replaced the empty params with default params)

So there is no reason to set default squeeze params in InvSqueeze — if the params are still empty at that point, it's a bug.

Correction: if the parameters are still empty at the point of InvSqueeze, it's not necessarily a bug: it could also be that the image is very small (less than 8x8) and the default parameters end up being just empty (no-op).
So the check can just be removed, and in case of empty params it just doesn't do anything.
Also changed the encoder to just not do Squeeze if it is a no-op Squeeze (saves 1 byte).
Updated the header size test with a tighter bound for the 1-pixel image with alpha.


lv_drivers @ 39fe864 404 not found

lv_drivers @ 39fe864 404 not found
or download it as zip here.
but https://codeload.github.com/littlevgl/pc_simulator_sdl_eclipse/zip/dev-6.0 404 not found

I've fixed it.
I don't know codeload, but you should use the master branch because dev-6.0 was deleted after the release.

I don't know codeload

I assume that is the server handling "Download ZIP" requests.


Handle Resources With the Same Name

Right now we have some places in the dashboard where we assume that there will be only one resource with a particular name in the resource list. This is not true across types of resources. There are also variants of this (e.g. https://github.com/dotnet/aspire/issues/1072) where we have assumptions about the uniqueness of properties other than the name.
We should discuss the right way to handle this both from a user display perspective (for example, what should be shown in the select list on the Console Logs page) and from a logic standpoint (for example, when we get a change notification, how do we ensure we're replacing the correct resource) and also somewhere in between (for example, how do we construct a reasonable URL to link directly to the console logs for a particular resource since the current /ConsoleLogs/ResourceName format won't work).
Opening up this issue to track the discussion. We can split it off into multiple issues once we decide what work needs to be done.
cc @smitpatel  @karolz-ms

Resource names are unique 🛩


qr-code alignment on poster

First of all: Thank you! That's great work.
I have just a little design issue:
example_full.py generates a svg_bytes with 70198 bytes length.
With that amount of data the qr-code is aligned into the center of the poster/portrait.svg
In my test i had smaller amounts of data in my event_description.
And with e.g. only 61444 bytes you get a smaller qr-code and its alignment is a little more left then it should be.
Perhaps it would help to check svg_bytes.getbuffer().nbytes to determine the necessary scale dynamically
(i guess with the landscape template it is the same, but i didn't test it.)

It also similarly fails if the variable length content is maxed out (ie. with 100 Characters Location and Description):


Help wanted for prop-types Type check

If I don't wanna use the type and import it from above, would this be a good way to do it in the classic ES6 style?
But I keep getting the error Unexpected Token for the part where the prop-types object ends
Home.propTypes = {
  fetchData: () => void
};

const connector = connect(
  ({ home }) => ({ home }),
  dispatch => ({
    fetchData: () => dispatch(action.fetchData()),
    actionVar: () => dispatch(action.actionVar()),
  }),
);

Just trying to work my way through the old and the new versions of this starter
Also why even change from prop-types to flow?
Thanks!

Well I figured it out with prop-types

Flow is a powerful type checking utility, it help me prevent from many troubles. It's easy to learn and understand. Take your time to check it.


Request for Documentation: Improve scope documentation for autolink plugin

Mattermost user jesse.hallam from https://community-release.mattermost.com has requested the following be documented:
I think there's an opportunity to improve the autolink documentation around use of the scope feature, as there's just a single example currently in the docs.

See the original post here.
This issue was generated from Mattermost using the Doc Up plugin.

May I take this one? :smile:


Add browser id api call

Feature branch
This adds a call for fetching a new browser id.  It will be much smaller after #241 is merged

You plan to switch the https://github.com/cbdr/cb-id-generator over to using this endpoint after it's in master?

@ausmith not sure....the new version requires an oauth token

@oochelz I'm good here, you good?


Mime Types Instead of File Extensions

Overview
This pull request changes a check from using everything after the dot in the URL as the extension to determine image validity, to checking the actual mime type as stored in the database. Under the current way of seeking a file extension in URL, the editor fails to insert an image if the image does not have a file extension at the end of the URL. It instead adds a link to the image. This is a bug for us with our particular use case and I hope that more well rounded method will prove to be useful for others as well.
Testing
Tested a variety of images and they seem to work fine.

You're testing this with Curator, correct?
I'm thinking that it will break the default MediaAction, as it doesn't store a mime type anywhere to be referenced.

You're testing this with Curator, correct?
I'm thinking that it will break the default MediaAction, as it doesn't store a mime type anywhere to be referenced.

I suppose so, I will work on an alternative solution and update this PR.

Closing this for now. Feel free to reopen if you find a solution.
Thanks.


[kind] Remove 1.22-based test iteration

Signed-off-by: Matthias Wessendorf<EMAIL_ADDRESS>Fixes #

Proposed Changes

Remove kind tests for 1.22, since we are already on min: 1.23 and also the HPA/v2 (dependency from eventing upstream) is NOT present on 1.22...

Release Note

Docs

/assign @pierDipi

/test channel-integration-tests-sasl-plain

/test channel-integration-tests-sasl-plain

/test channel-integration-tests-sasl-plain


Object is possibly 'null' (TS2531) even after null check

Typescript throw an Object is possibly 'null' error even thought after the if check the object can't be null.
Is there something I'm missing ?
  const wrapperRef = useRef(null);
  const [isOpen, setIsOpen] = useState(false);

  const handleClickOutside = (event: Event) => {
    if (wrapperRef.current && !wrapperRef.current.contains(event.target)) {
      setIsOpen(false);
    }
  };

src/components/Menu/Menu.tsx:17:32 - error TS2531: Object is possibly 'null'.

17     if (wrapperRef.current && !wrapperRef.current.contains(event.target)) {
                                  ~~~~~~~~~~~~~~~~~~

Found 1 error.

Same here :
  const wrapperRef = useRef(null);
  const [isOpen, setIsOpen] = useState(false);

  const handleClickOutside = (event: Event) => {
    if (wrapperRef.current == null) return;
    if (!wrapperRef.current.contains(event.target)) {
      setIsOpen(false);
    }
  };

src/components/Menu/Menu.tsx:18:10 - error TS2531: Object is possibly 'null'.

18     if (!wrapperRef.current.contains(event.target)) {
            ~~~~~~~~~~~~~~~~~~

Found 1 error.

Is there a workaround ?
Thanks

This is the cause:
const wrapperRef = useRef(null);

What is the type of wrapperRef? It's MutableRefObject<null>, and the type of wrapperRef.current is null. So to TypeScripts awareness the value will always be null - it doesn't know about any other type. (Tho I expect here an error about dead code.)
You can fix this by being more explicit or lax with your types.
const wrapperRef = useRef<any>(null);

Thanks a lot.


How to Run Multiple .exe and how to customize configuration of .exe???

The earlier versions of this scanner wasn't compiled into one exe file which allowed for customization of the step distance, scan delay, and number of threads? How can we customize these options now for the exe and why did he compile it to an exe for windows in the first place????
And how does one now run multiple instances of the exe on one windows screen??? When i try to open a second window, the first one exits and reopens...

I'm interested in this too. More specifically I'd like to run multiple instances (with their own accounts or not) and have the results joined into a single map. This because the scan interval of 10 seconds (why is that btw..?) makes a full scan of distance 5 take about 10 minutes, Which means new pokemon could be off in about 3 minutes.
Even more interesting would be to be able to make a list of specific areas to scan, as spawning locations seem to be fixed. I know the spawning locations around me and i want to scan those, without all the known deadzones in between them. It takes a bit of preparation work, but saves about 50% of scan-time.

You are able to run multiple instances by running the .exe more than once and logging in with a different account. All the scans will show up on the same map. However, this process is tedious and I would like to see it implemented as a feature where the program remembers multiple accounts and their locations. @dudeman999 It sounds like you're running the installer when you're restarting the program. It should have generated a shortcut to a .exe on your desktop. You can run multiple instances from that.

Having trouble running more than 2 instances, the 3rd one runs but just says waiting. Using a different PTC account for all 3

When you create a new Pokemon Go account you have to accept the in-game terms of service . Until you do that it won't work in the scanner. Just open the account in the Pokemon Go app and accept the term of service. Then it should work fine.