the errors module is not installed

There is a missing __init__.py in spotty/errors so on a fresh install I get the following error:
spotty --version
Traceback (most recent call last):
  File "/home/glefalher/.local/share/virtualenvs/autosuggest-analysis-JWy0ztrb/bin/spotty", line 7, in <module>
    from spotty.commands.aws import AwsCommand
  File "/home/glefalher/.local/share/virtualenvs/autosuggest-analysis-JWy0ztrb/lib/python3.7/site-packages/spotty/commands/aws.py", line 3, in <module>
    from spotty.providers.aws.commands.create_ami import CreateAmiCommand
  File "/home/glefalher/.local/share/virtualenvs/autosuggest-analysis-JWy0ztrb/lib/python3.7/site-packages/spotty/providers/aws/commands/create_ami.py", line 6, in <module>
    from spotty.providers.aws.instance_manager import InstanceManager
  File "/home/glefalher/.local/share/virtualenvs/autosuggest-analysis-JWy0ztrb/lib/python3.7/site-packages/spotty/providers/aws/instance_manager.py", line 3, in <module>
    from spotty.errors.instance_not_running import InstanceNotRunningError
ModuleNotFoundError: No module named 'spotty.errors'

Thanks, Geraud. Fixed it.


Revert "Revert "api_core: Make PageIterator.item_to_value public. (#4702)""

Reverts GoogleCloudPlatform/google-cloud-python#4731

Do not merge until api_core 0.2.0 is being prepped for release.
Context:

This is done to prevent a release-the-world scenario in order to release bigquery, spanner, and datastore. It will be re-reverted before the next minor release of api_core.

I have added an alias property for the private method. This allows us to maintain compatibility even for some older clients. We can remove it at a later date.
I will merge this once CI is green.

nvm, I'm going to release the world anyway, I'm gonna remove the alias.


No shell to listener

Hi,
so I am using the exploit on a hackthebox machine. When executing the script, this is shown:
Strapi Framework Vulnerable to Remote Code Execution - CVE-2019-19609
please set up a listener on port 9001 before running the script. you will get a shell to that listener
Traceback (most recent call last):
File "/home/oceanhawk/CVE/CVE-2019-19609/exploit.py", line 30, in
response = requests.post(url, headers=headers, data=data, verify=False)
File "/usr/local/lib/python3.9/dist-packages/requests/api.py", line 119, in post
return request('post', url, data=data, json=json, **kwargs)
File "/usr/local/lib/python3.9/dist-packages/requests/api.py", line 61, in request
return session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python3.9/dist-packages/requests/sessions.py", line 530, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.9/dist-packages/requests/sessions.py", line 665, in send
history = [resp for resp in gen]
File "/usr/local/lib/python3.9/dist-packages/requests/sessions.py", line 665, in
history = [resp for resp in gen]
File "/usr/local/lib/python3.9/dist-packages/requests/sessions.py", line 166, in resolve_redirects
raise TooManyRedirects('Exceeded {} redirects.'.format(self.max_redirects), response=resp)
requests.exceptions.TooManyRedirects: Exceeded 30 redirects.
I get no shell to my listener on 9001.
The command I type is this:
python3 exploit.py api-prod.horizontal.htb lhost eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpZCI6MywiaXNBZG1pbiI6dHJ1ZSwiaWF0IjoxNjM0NjUxMTc5LCJleHAiOjE2MzcyNDMxNzl9.h_EP9dFh_0qAIZU3lvl1SpVkkYlBY_nN45BGQ7ptIqw http://api-prod.horizontall.htb/

well the error says requests.exceptions.TooManyRedirects: Exceeded 30 redirects. i think its giving you some redirect. i cant help you much because that box is still live. sorry about that if you have anything send me a email

@oceanhawk rhost spelling is incorrect "api-prod.horizontal.htb" <- missing a letter l on horizontall

hope the issue is fixed :D thanks @async-costelo


avoid generic exceptions

rather than raise Exception("Invalid write to address %04X" % addr) we should use a specific exception like raise InvalidWriteAddress(addr)

[ ] c++
[ ] python
[ ] rust
[ ] go
[ ] php

Hey just wanted to check, regarding the python section for this, you're referring to line 297 of the ram.py file? And if so, perhaps I could go ahead to create another custom exception in the errors.py file to handle this?

The if self.debug: print(...) parts in ram.py are things which are weird and noteworthy, but AFAIK wouldn't crash a real gameboy - in this case I'm mostly talking about the parts where we currently raise Exception(...), but there's no way to tell what went wrong (other than reading the exception message). I'd like those raise Exception(...) parts to be replaced with more specific Exception sub-classes, so we can do things like
try:
    [...]
except GameException:
    [...
    the game itself did something bad, like reading save data on
    a cart which doesn't support save data - so we should print
    out some useful data to help the game developer
    ...]
except UserException:
    [...
    the user did something bad like specifying a .gb file that
    doesn't exist, so we should print out a friendly error message
    ...]
except EmuException:
    [...
    the emulator itself did something bad, we should dump the
    state in a way that's useful for a rosettaboy developer to
    investigate
    ...]
except Exception:
    [...
    generic fallback for something totally unimaginable happening
    ...]

(Right now we don't even have a clear hierarchy for game / user / emulator exceptions - everything is just a subclass of EmuError - having a useful tree of exception classes just came to me while I was writing that comment ^^)

I see, so am I right to say that the end goal would kinda be to have each Raise Exception section modified to be like the try/except structure you mentioned above to catch as many specialized errors as possible?

The goal is for all the places we currently do a generic raise Exception(), to come up with a more specific kind of exception (ie replace the current raise Exception(...) with raise WriteToInvalidAddressException(address)) -- and then in main.py, we can catch the main categories of exceptions as in the try/except example above :)

Alright, so the first thing to do now would be to follow what you mentioned in your edited comment, which is to go to errors.py and create the three larger classes GameException, UserException, and EmuException so that their relevant smaller bugs can inherit from them? I wanted to also ask, right now EmuError simply exits with the exit code of 1, so would this similar, simple functionality be good enough for GameException, UserException, and EmuException too or would there need to be additional messages/functionality?

For exit codes, everything should exit 1 except for Quit / Timeout / UnitTestPassed (which should exit 0) and UnitTestFailed (which should exit 2)
For messages, only Timeout is really important (because the message there includes performance details) - the rest can have nice messages if it's easy to add, but not really needed

Actually, maybe different error codes for different categories of error...
After a bunch of experimentation, I think the C++ version is now in a state that I'm happy with -- so other languages should copy what's there (I've updated the original issue description to describe that)

Awesome, I'll just draw reference from what you've done with the C++ version and push up some code for the python version, and you can check if it looks good to you

hey, added some changes here #27. It's probably not complete/perfect. Do let me know if anything needs to be changed/added :)


Page jumps around on scroll in Chrome for Android

Should be fixed when the following feature is released  https://www.chromestatus.com/feature/6241601772847104

self-closing stale issue 👍


break when all data from tarball is found

This is the missing piece from #d299e183 that actually allows the command to break once it has found the info it needs. This prevents looping over all parts of the tar ball.

Hi @brentp this looks good. Just now noticing - should this be using tarfile.open as a context manager?

aye. I'll add that. Also, I noticed there's nothing to handle the case where both are not found.
Presumably that shouldn't happen, but should I raise an exception in that case?

I added the context manager.

Yes, I think if either index or recipe is None, a TypeError should be raised.

Actually, on closer inspection, the correct behavior should be: TypeError if index is missing. recipe should default to dict(). Not all valid conda packages contain info/recipe.json.

ok. I think I addressed all your suggestions. thanks.

This looks great. Thanks so much for your contribution @brentp!


Complete Revamp of Adapter

This version uses the AWS SDK directly and updates the adapter S3 parameter object to take advantage of all the S3 SDK parameters available to the API. Also, it adds headers to requests correctly and uses file.mimetype and file.size to correctly add Content-Type and Content-Length headers to the S3 upload Object. Finally, instead of using "s3.upload" this adapter uses "s3.putObject" which follows the SDK best practices and preserves any versioning that might be enabled on S3

Thanks @aaronfranco and sorry for the lack of feedback on this issue. I'm closing this PR in favour of #35 which stays close to the original API.


Deprecated getIterator function called in SSViewer_Scope

Module version(s) affected
5.2
Description
The getIterator function is being called in /src/View/SSViewer_Scope.php though the function is now deprecated in v5.2 as per https://api.silverstripe.org/5/SilverStripe/View/ViewableData.html#method_getIterator.

How to reproduce
N/A
Possible Solution
No response
Additional Context
No response
Validations

[X] Check that there isn't already an issue that reports the same bug
[X] Double check that your reproduction steps work in a fresh installation of silverstripe/installer (with any code examples you've provided)

What code is used in your template that results in this happening? Is this happening in to a core template, or a custom one?

This has had no response for 3 weeks, so I'll close it.
If you provide the missing information, tag me and I can reopen it.


Update Settings “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for ekhdmny processing.

Name
Link

🔨 Latest commit
da94fb820615a85b2b464645cf7cad3731b68c3a

🔍 Latest deploy log
https://app.netlify.com/sites/ekhdmny/deploys/63839914af39e80009bf855d


New misc tool for maintaining a spreadsheet of Uploads

Adds a simple utility to generate a .tsv file of Uploads, analogous to the table of Datasets produced by new_dataset_survey.py

Sample, with tabs expanded and aligned:
group_name   uuid                             hubmap_id       status has_data has_metadata n_md_recs validated last_touch note
Stanford TMC 04880713ae4602bd05219059e14bc93c HBM953.CGJP.957 New    True     False        0         False     2021-11-08
Stanford TMC 4baffd7722cf50a0e83596c35892db23 HBM776.GHGD.334 New    True     False        0         False     2021-11-08
Stanford TMC bb60b2e48b2c2232e1d54d0540b720d3 HBM726.HJQW.698 New    True     True         9         False     2021-10-15
Stanford TMC bc48e149c0d8da71af3cade8cb5de52a HBM247.HZJZ.644 New    True     False        0         False     2021-11-08


⚠ ClamAV has degraded performance

In 91f51a7, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 67ebd31.


Django 1.8 ContentType Model change

Hi all,
Love constance, and I've been using on my projects for a while now. I've been playing around with the dev version of Django (1.8a1/1.8b1), and noticed a change in the ContentTypes Model that breaks Constance on manage.py migrate.
https://docs.djangoproject.com/en/1.8/ref/contrib/contenttypes/
name
The human-readable name of the content type. This is taken from the verbose_name attribute of the model.
Changed in Django 1.8:
Before Django 1.8, the name property was a real field on the ContentType model.

In particular, the error is
File "/lib/python2.7/site-packages/django/db/models/sql/query.py", line 1369, in names_to_path
    "Choices are: %s" % (name, ", ".join(available)))
django.core.exceptions.FieldError: Cannot resolve keyword 'name' into field. Choices are: app_label, id, logentry, model, permission

from
File "/lib/python2.7/site-packages/constance/models.py", line 16, in create_perm
    model='config')

I was able to hack around it simply by removing the name='config' line from constance/models.py
Thanks for all the hard work!
Cheers,
Jonathan

I ran into the same thing and removing name='config' fixed it as well.

See #96


Better support for accessing args in runtime

Type: Feature Request
For a lot of the Python typing functions, we can determine the type of args from the arguments. For example, if you have a Union (or the union by | ing types), __args__ is a tuple[type[...]].
Example:
MY_TYPE = str | int | bool
print(MY_TYPE.__args__) # (str, int, bool)
type_args: tuple[str, int, bool] = MY_TYPE.__args__

The type checker could reasonably guess the types in that instance without me having to explicitly annotate it. Another example using literals:
from typing import Literal

MY_LITERAL = Literal[1, 2, 3, 4, 5]
print(MY_LITERAL.__args__) # (1, 2, 3, 4, 5)
literal_args: tuple[Literal[1], Literal[2], Literal[3], Literal[4], Literal[5]] = MY_LITERAL.__args__

Some similar inference could be done here. The current annotation of tuple[Any, ...] is correct but it would be nicer if interacting with __args__ in the runtime (such as pulling values out of a Literal).
Extension version: 2024.11.3
VS Code version: Code 1.95.3 (f1a4fb101478ce6ec82fe9627c43efbf9e98c813, 2024-11-13T14:50:04.152Z)
OS version: Darwin x64 24.0.0
Modes:

Could someone from the pylance team please transfer this to the pyright project?


Exclude workload deleted matching labels in webhook.

Revise the logic of the rollout controller to exclude workloads that have already deleted the matching labels specified in the webhook.
add test

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. Special notes for reviews

/lgtm

/approve


URL missing and typo

Describe the bug
URL missing and typo
To Reproduce
Steps to reproduce the behavior:

Go to Settings
See typo "You can learn more about Sempahore here"
It als doesn´t have a link

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

task completed and tested on staging.


Documented in_bulk behavior with unexisting id_list content.

As usual with me, triple check language :-)

Thanks Jacob for the review!


openal-soft: support bsd

OpenAL builds fine on FreeBSD but linking doesn't work. I'm not sure how to fix that.
https://github.com/llvm/llvm-project/issues/30308


[tool_crash] FileSystemException: Cannot open file, OS Error: The system cannot find the specified path., errno = 3

Command
flutter create --template app --overwrite .

Steps to Reproduce

...
...
...

Logs
FileSystemException: Cannot open file, OS Error: The system cannot find the specified path., errno = 3
#0      _File.throwIfError (dart:io/file_impl.dart:675:7)
#1      _File.openSync (dart:io/file_impl.dart:490:5)
#2      ForwardingFile.openSync (package:file/src/forwarding/forwarding_file.dart:74:16)
#3      ErrorHandlingFile.openSync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:295:22)
#4      _runSync (package:flutter_tools/src/base/error_handling_io.dart:590:14)
#5      ErrorHandlingFile.openSync (package:flutter_tools/src/base/error_handling_io.dart:294:12)
#6      ErrorHandlingFile.copySync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:338:27)
#7      _runSync (package:flutter_tools/src/base/error_handling_io.dart:590:14)
#8      ErrorHandlingFile.copySync (package:flutter_tools/src/base/error_handling_io.dart:333:5)
#9      Template.render.<anonymous closure> (package:flutter_tools/src/template.dart:313:27)
#10     _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:625:13)
#11     Template.render (package:flutter_tools/src/template.dart:248:24)
#12     CreateBase.renderMerged (package:flutter_tools/src/commands/create_base.dart:480:21)
<asynchronous suspension>
#13     CreateBase.generateApp (package:flutter_tools/src/commands/create_base.dart:503:23)
<asynchronous suspension>
#14     CreateCommand.runCommand (package:flutter_tools/src/commands/create.dart:350:31)
<asynchronous suspension>
#15     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1336:27)
<asynchronous suspension>
#16     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#17     CommandRunner.runCommand (package:args/command_runner.dart:212:13)
<asynchronous suspension>
#18     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:296:9)

[✓] Flutter (Channel stable, 3.10.1, on Microsoft Windows [Version 10.0.22621.1702], locale ru-RU)
    • Flutter version 3.10.1 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d3d8effc68 (32 hours ago), 2023-05-16 17:59:05 -0700
    • Engine revision b4fb11214d
    • Dart version 3.0.1
    • DevTools version 2.23.1

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at C:\Users\mike_\AppData\Local\Android\Sdk
    • Platform android-33, build-tools 33.0.2
    • Java binary at: D:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Professional 2022 17.5.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Professional
    • Visual Studio Professional 2022 version 17.5.33530.505
    • Windows 10 SDK version 10.0.22000.0

[✓] Android Studio (version 2022.2)
    • Android Studio at D:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22621.1702]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 112.0.5615.138
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 113.0.1774.35

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Flutter Application Metadata
Type: malformed
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: null
Creation framework version: null

Hi @vld7wn, do you still have this issue if you just try to create the project without the --overwrite flag? Also, what directory are you running the command in?

Hi @vld7wn
Are still facing the same error ?
if you are getting same error then make sure you are giving proper path of image.
this error only happens when you are giving wrong path of image and trying to load it so make sure to print the path and check that the image is placed at that path or not and then after try again.


SES: Implement SendBulkTemplatedEmail

Feature: Implemented SendBulkTemplatedEmail API feature from SES.
This is #5360

Apologies for the lint error. Some error in my lint set up. Corrected it.


Save list functionality on Browse page

When a user is on the Browse page we want them to be able to curate a list in the UI and then save the list as it is.

Closed PR #133


add support for bit literals; add another sql error code

@sougou this adds support for bit literals, similarly handled as hex values.  Also found a new error code to support.

LGTM


Add error code

@zghosts

@BackEndTea Do you need a hand adding a test for this change?

Closing due to inactivity, feel free to pick this back up at your leisure.


Docker doesn't work...

I tried your docker...
I think it's missing Postgresql inside the container.
Can you check?

@FatBrownDog
The Openkoda docker image does not contain PostgreSQL Server. To run Openkoda in Docker with Postgres, please use Docker compose, that starts separate PostgreSQL contrainer: https://github.com/openkoda/openkoda#docker-compose


add shared object locking costs

Add costs for read and write shared object locks.
Currently they're set to 0.
@andll will give feedback on reasonable values.
Todo:

Figure out reasonable nonzero costs
Add tests

Putting this back in draft mode while we discuss internally on how we want to structure the costing.

Decided on different approach


Can you add instructions on manually installing the extension

I have VSIX file of the extension that I downloaded from the marketplace. There are no instructions on the page to install this file manually. It would be useful for other developers to include it in the documentation.
Steps to install:
Navigate to the Azure DevOps Server Extensions page on your server.
(For example, http://someserver/_gallery/manage)
Click upload new extension and select the file you have just downloaded.
After the extension has successfully uploaded, click Install and select the Team Project Collection to install into.
[Enter feedback here]

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

ID: 46849bf5-07ed-62cd-66a5-2ebabe449855
Version Independent ID: d12ad05e-4348-f13a-ea35-7bb4761c3b0b
Content: Install extensions for on-premises servers
Content Source: docs/marketplace/get-tfs-extensions.md
Product: devops
Technology: devops-marketplace
GitHub Login: @elbatk
Microsoft Alias: elbatk

Hi @akuchotrani, Thanks for your feedback. I'm following up on your request and will get back to you soon!

Hi @akuchotrani, At this time, we are unable to process your request for providing the manual steps within the current documentation. We do appreciate your feedback, so thank you. In the future, if you have a suggestion or feature request for the tool, or need technical support, please go here: https://visualstudio.microsoft.com/vs/support/#talktous

@akuchotrani were you able to proceed? I'm experiencing the same issues


Update the argparser and fix some bugs

Update the argument parser help message for clarity
Use ERA5 superclass for ERA5T
Fix bug where DEM downloads even when not needed
Ensure argparse inputs are typed correctly
Remove antiquated code
Ensure that argparse inputs are case-insensitive
Initial stab at ensuring CF-1.8 HDF5 compliance
Should fix #26, also addresses issues raised in #27 and #28

For some reason this did not trigger the circle CI to run a test. Will need to look into it.

Yes, I saw that, looked for it to start but didn't see anything. We can look into it tomorrow.

circle ci is running on it


Implement RFC #5: RPC Queue

This implements RFC #5: RPC Queue.
This is a cleaner version of the feature/rpcqueue branch with only the RPC Queue specific changes and the changes from:
PR #407 and PR #408
RFC for this is pending review and will be attached to this PR once it is approved

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

a minor thing but you should've "Squash and Merge" @NoelStephensUnity  :)


access denied when using autoform

I'm trying to use autoform with meteor validatedmethod simple schema and collection2. However when I enter values in the textfields I get:
undefined error : 403 errorType : "Meteor.Error" message : "Access denied [403]" reason : "Access denied" stack : "Error↵    at Connection._livedata_result (http://localhost:3000/packages/ddp-client.js
My code uses the following template:
`
{{#autoForm collection=club id="insertClubs" type="insert"}}

Add a Club
    {{> afQuickField name='name'}}
    {{> afQuickField name='number'}}
    {{> afQuickField name='updated'}}
    {{> afQuickField name='created'}}

Insert
  {{/autoForm}}
`
then I'm using an event helper to listen to the submit and invoke a validatedmethod:
`'submit .btn btn-primary'(event, instance) {
console.log('test');
 insert.call( {
  name: 'test',
  number: 3,
  updated: new Date(),
  created: new Date()
}, (err, res) => {
   console.log(err);
  if (err) {
    if (err.error === 'validation-error') {
    }
  }
 });

}`
and this is the insert call itself:
`export const insert = new ValidatedMethod({
name: 'Clubs.methods.insert',
validate: Clubs.simpleSchema().validator(),
run(newClub) {
// In here, we can be sure that the newClub argument is
// validated.
console.log('insert new club');
if (!this.userId) {
throw new Meteor.Error('Clubs.methods.insert.not-logged-in',
'Must be logged in to create a club.');
}
Clubs.insert(newClub)

}
});`
I think that this setup does not actually invoke my insert method as I don't see the console.log but meteor comes back with the error. Any idea what could be the problem?

How did you set your allow/deny?


OWNERS: add a7i as a reviewer

Per @ingvagabund adding myself as a code reviewer

/assign ingvagabund

/lgtm
/ok-to-test

@a7i  Congratulations~

/approve 🎉 Thanks for everything you've done Amir! We should also add you to the kubernetes-sigs org as the bot said. Feel free to open a PR there, I will +1 your nomination :)

@a7i any luck getting added to the kubernetes-sigs org? Let me know if you need any assistance with the process. Thanks!

/verify-owners

/verify-owners
Thank you for your patience!

@a7i thank you for being so patient! Sorry about the confusion over the +1s... congrats and thank you for all of your help with the project so far :)


[UI] Improved UI in Performance Profile test results

Signed-off-by: Meghana-12<EMAIL_ADDRESS>Description
Improved the UI

This PR fixes #3798
Notes for Reviewers
This is a quick fix.
I need a valid URL and details that I can use to test this properly.
Signed commits

[x] Yes, I signed my commits.

Too much white space on the right, don't you think?

Too much white space on the right, don't you think?

Actually, I'm using Material UI GridList to create 3 columns. Since I'm just mapping title array contents here (dunno how long they can get). Couldn't think of anything else.
And I'm unable to wrap the content somehow, trying to fix that and to make the word before colon bold.

Demo in tomorrow's Meshery Dev call?

Demo in tomorrow's Meshery Dev call?

I need a valid URL and details that I can use to test this properly.

And UI integration checks at failing, what causes that ?

@Meghana-12 what are you talking about the endpoint URL? you could just use https://google.com

Does res.Labels always have only 2 entries? Title of the test and URL ?
https://github.com/meshery/meshery/blob/master/ui/lib/chartjs-formatter.js#L60

And is it okay if I remove the commented code in these two files and refactor a bit?

Tried to add the percentiles to the right of the chart. But when view on slightly smaller screens it's leading to a lot of white space.

So I'm just displaying the percentiles in a row

I wish I could work on this further but my schedule's packed due to exams and assignments ;(
Will be back in a week!

Thank you, @Meghana-12. Sounds good.


Release Qualification - fortestonly

This is a generated PR that triggers release qualification tests, and will be automatically merged if all tests pass. In case some test fails, you can manually rerun the failing tests using /test. Force merging this PR will suppress the test failures and let the release pipeline continue.
none

@hklai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-perf-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-perf-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-perf-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-perf-tests

prow/istio-unit-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-perf-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-perf-tests

prow/istio-unit-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-unit-tests

prow/daily-e2e-rbac-no_auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-perf-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-perf-tests

prow/istio-unit-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-unit-tests

prow/daily-e2e-rbac-no_auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth

prow/daily-e2e-rbac-no_auth-default.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth-default

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-perf-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-perf-tests

prow/istio-unit-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-unit-tests

prow/daily-e2e-rbac-no_auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth

prow/daily-e2e-rbac-no_auth-default.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth-default

prow/daily-e2e-rbac-auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-perf-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-perf-tests

prow/istio-unit-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-unit-tests

prow/daily-e2e-rbac-no_auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth

prow/daily-e2e-rbac-no_auth-default.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth-default

prow/daily-e2e-rbac-auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-auth

prow/daily-e2e-rbac-auth-default.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-auth-default

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-perf-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-perf-tests

prow/istio-unit-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-unit-tests

prow/daily-e2e-rbac-no_auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth

prow/daily-e2e-rbac-no_auth-default.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth-default

prow/daily-e2e-rbac-auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-auth

prow/daily-e2e-rbac-auth-default.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-auth-default

prow/daily-e2e-cluster_wide-auth-default.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-cluster_wide-auth-default

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-perf-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-perf-tests

prow/istio-unit-tests.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test istio-unit-tests

prow/daily-e2e-rbac-no_auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth

prow/daily-e2e-rbac-no_auth-default.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-no_auth-default

prow/daily-e2e-rbac-auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-auth

prow/daily-e2e-rbac-auth-default.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-rbac-auth-default

prow/daily-e2e-cluster_wide-auth-default.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-cluster_wide-auth-default

prow/daily-e2e-cluster_wide-auth.sh
9693a52fe36201fd9090b3b9f0311d23ba536a1c
link
/test daily-e2e-cluster_wide-auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[BUG] URLError when running adaptive freezing example with CIFAR10 ResNet18

Describe the bug
An URLError (SSL: certificate verify failed) was raised when I tried to run the adaptive freezing example with CIFAR10 ResNet18
To Reproduce

Change line 17 of file examples/adaptive_feeezing/adaptive_freezing.py to:
os.environ['config_file'] = 'adaptive_freezing_CIFAR10_resnet18.yml'
Ensure that this is the first time running this example; i.e., no content was downloaded and directory data/ and model/ have not been created yet under directory examples/adaptive_feeezing/
Execute adaptive_freeezing.py
Encounter URLError, SSL certificate verify failed, when the program attempts to download the dataset and model

Expected behavior
No error should be raised and the content should be downloaded successfully
OS environment (please complete the following information):

OS: Windows 10 64-bit
Using Python 3.9
Using IDE PyCharm Community Edition 2021.3

Additional context
Possible Solution:
I resolve the error by adding the code below to file plato/datasources/cifar10.py:
import ssl
ssl._create_default_https_context = ssl._create_unverified_context

I think the code bypasses the error by using the unverified SSL. I'm not sure whether that is a good solution.

This cannot be reproduced here as it can be related to issues on the client side when downloading from a HTTPS server. Closing.


Simplify bn_from_montgomery to do the minimum work necessary

Apologies in advance if I am misunderstanding things.
I read this comment in https://github.com/openssl/openssl/blob/83cf7abf8e9abbd4d0b68c63dc1cb43374aafe63/crypto/bn/asm/x86_64-mont5.pl#L2136-L2141
If I understand the comment bn_from_montgomery writes outside where its argument points. In particular, it assumes that the argument points into a larger buffer, and it zeroes that entire larger buffer.
If so, this makes bn_from_montgomery less useful than it could be. In particular, it would be great if we could use bn_from_montgomery to accelerate BN_from_montgomery() in general.
I also find this to be difficult to understand from a security review standpoint. I would prefer that all zeroing in RSA exponentiation be done via OPENSSL_cleanse/OPENSSL_free. It seems like there is already an OPENSSL_cleanse() at https://github.com/openssl/openssl/blob/166f0082e7ce53ed608d8519526b99893ca7925e/crypto/bn/bn_exp.c#L1109 so the zeroing being done in bn_from_montgomery may be redundant now? That is, maybe the optimized zeroing code is actually slowing things down now?
Also, that comment indicates that bn_from_montgomery does some extra copying to ensure its arguments don't create false sharing. Since bn_from_montgomery is only called once during an exponentiation, maybe we should just do the simplest thing that works, and avoid the re-alignment adjustment?

/cc @dot-asm

If I understand the comment bn_from_montgomery writes outside where its argument points.

No. It says "alias ... modulo 4096", i.e. goal is to avoid overlap of 12 least significant bits of stack frame's address and 12 least significant bits of external arguments' address. It doesn't mean that it modify external buffers in non-prescribed way, only that manipulating stack frame doesn't interfere with prescribed access to external buffers.  There is something called memory disambiguator, which is responsible for canceling loads that were speculatively moved ahead of stores. I mean if programming order is "store to X" followed by "load from Y", and "load from Y" is speculatively executed prior store, then if X and Y are same, processor would have to cancel speculative load and re-execute it after store. However it looks only at 12 least significant bits, so that X and X+4096 count as same address. Aligning stack frame in specific manner relative to external buffers allows to avoid this false positives.

In particular, it assumes that the argument points into a larger buffer, and it zeroes that entire larger buffer.

No, it's not about argument pointing into a larger [external] buffer, but stack frame. Yes, bn_from_montgomery does allocate larger stack frame than it actually uses. And it does cleanse more stack than it actually used. But there is nothing that prohibits it to do that. It allocated a stack frame and it cleansed it. But how this larger-than-needed stack frame is allocated? And how much is cleansed? It's allocated using exactly same algorithm as other subroutines, so that it overlaps ... previously allocated stack frames. And it cleanses as much as other subroutines used. This way previously called subroutines can be let off the hook cleansing their stack frames. I mean customarily subroutine developer have to assume that caller doesn't even know how to cleanse subroutine's stack frame and hence subroutines will be bound to cleanse it on per-call basis. But if you know that caller calls specific subroutine at the end, you can use the opportunity to cleanse stack only once. That's what bn_from_montgomery does [besides performing prescribed calculations]. In other words one shouldn't look at stack handling from bn_from_mongomery's perspective alone, but even from others'. Yes, this means that bn_from_montgomery is not meant to be called "standalone" but only in specific sequence, equivalent to one observed in bn_exp.c.

Just in case it wasn't clear. As far as cleansing goes, bn_from_montgomery cleans up after its "fellow" assembly subroutines, and only does things that can't be done in high-level language [such as C]. It doesn't have any side effects that would be observable from high-level language. In other words it doesn't actually steps on caller's toes as original message implied. Of course standing suggestion is to take alloca from assembly domain, so that cleansing becomes caller's business...

this means that bn_from_montgomery is not meant to be called "standalone" but only in specific sequence

"Not meant" doesn't mean "can't". If called "standalone" it will produce expected result. But it will do that with non-observable side effect of allocating larger stack frame [and cleansing it] than minimally required.


Optimize RCTImageView resizes/reloads to avoid re-freshing/re-decoding when the source is a remote image

So it appears that RCTImageView, when resized, has a comment that says:
        // If the existing image or an image being loaded are not the right
        // size, reload the asset in case there is a better size available.

I can see this making sense in the case of assets from disk, where we may need a higher or lower resolution. But in the case of remote assets pulled over the network, it (hopefully!) pulls the data from the cache and then calls decodeImageDataWithoutClipping() on it, which still feels like a waste of resources.
I assume it's worth an resize optimization to avoid reloading images when they are remote? I'm not clear on the best way to check this (or verify how helpful it'll be in the big pauses I'm seeing). I'm also not able to edit/debug any ReactNative code for some reason (investigating here), so not able to go in and debug the performance impacts of changing this at the moment.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/optimize-rctimageview-resizesreloads-to-avoid-re-freshingre-decoding-when-the-source-is-a-remote-image
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.


amass: 3.13.3 -> 3.13.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/OWASP/Amass/releases.
meta.description for amass is: "In-Depth DNS Enumeration and Network Mapping"
meta.homepage for amass is: "https://owasp.org/www-project-amass/"
meta.changelog for amass is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/xc52y7mghsg190b0px2dhn0g7mzbw5hr-amass-3.13.4/bin/amass passed the binary check.

1 of 1 passed binary check by having a zero exit code.

0 of 1 passed binary check by having the new version present in output.

found 3.13.4 with grep in /nix/store/xc52y7mghsg190b0px2dhn0g7mzbw5hr-amass-3.13.4

found 3.13.4 in filename of file in /nix/store/xc52y7mghsg190b0px2dhn0g7mzbw5hr-amass-3.13.4

directory tree listing: https://gist.github.com/f8dfc9b88a963aa138b71ff3f700055f

du listing: https://gist.github.com/2dbed9a87e5e41203642606f8a2f68d7

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
amass

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xc52y7mghsg190b0px2dhn0g7mzbw5hr-amass-3.13.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A amass https://github.com/r-ryantm/nixpkgs/archive/8b34f0cbbe0b96504993825b76ecb88ab76534c3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xc52y7mghsg190b0px2dhn0g7mzbw5hr-amass-3.13.4
ls -la /nix/store/xc52y7mghsg190b0px2dhn0g7mzbw5hr-amass-3.13.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

amass

Maintainer pings
cc @kalbasit @fabaff @kalbasit for testing.

conflict, closing


Create README on tests for FAN_TRAY type.

Pull Request Functional Test Report for #2982 / 5dded62717ce298a34f92ccd03ceb0f75864ad29
No tests identified for validation.
Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1983

Relevant Lines:
3573

💛  - Coveralls

I would suggest updating gnmi-1.4 instead.
https://github.com/openconfig/featureprofiles/tree/main/feature/platform/tests/telemetry_inventory_test

Closed in favor of https://github.com/openconfig/featureprofiles/pull/3046


The version of detekt applied by the build plugin was not updated

Goal

Testing

Release Notes

WHAT:
WHY:
WHO:

Current dependencies on/for this PR:

master

PR #216   👈

This stack of pull requests is managed by Graphite.


Saner approaches to getting metadata for Relations

Up to now, the dbt-spark plugin has leveraged a handful of metadata commands:
-- find all current databases
show databases

-- find all current tables in a database
show tables in my_db

-- determine if a relational object is a view or table
show tblproperties my_db.my_relation ('view.default.database')

The main issue with running one statement per relation is that it's very slow. This is justifiable for the get_catalog method of dbt docs generate, but not so at the start of a dbt run. Most databases have more powerful, accessible troves of metadata, often stored in an information_schema. Spark offers nothing so convenient; describe database returns only information about the database; describe table [extended] must be run for every relation.
@drewbanin ended up finding a semi-documented statement in the Spark source code that does most of the thing we want:
show table extended in my_db like '*'

It returns the same three columns as show tables in my_db, for all relations in my_db, with a bonus column information that packs a lot of good stuff:
Database: my_db
Table: view_model
Owner: root
Created Time: Wed Jan 29 01:58:46 UTC 2020
Last Access: Thu Jan 01 00:00:00 UTC 1970
Created By: Spark 2.4.4
Type: VIEW
View Text: select * from my_db.seed
View Default Database: default
View Query Output Columns: [id, first_name, last_name, email, gender, ip_address]
Table Properties: [transient_lastDdlTime=1580263126, view.query.out.col.3=email, view.query.out.col.0=id, view.query.out.numCols=6, view.query.out.col.4=gender, view.default.database=default, view.query.out.col.1=first_name, view.query.out.col.5=ip_address, view.query.out.col.2=last_name]
Serde Library: org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe
InputFormat: org.apache.hadoop.mapred.SequenceFileInputFormat
OutputFormat: org.apache.hadoop.hive.ql.io.HiveSequenceFileOutputFormat
Storage Properties: [serialization.format=1]
Schema: root
 |-- id: long (nullable = true)
 |-- first_name: string (nullable = true)
 |-- last_name: string (nullable = true)
 |-- email: string (nullable = true)
 |-- gender: string (nullable = true)
 |-- ip_address: string (nullable = true)

The same command also exists in Hive. This is a "big if true" find that could immediately clean up our workarounds for relation types. We'll want to check that it's supported by all vendors/implementations of Spark before committing to this approach, so we'll benefit from folks testing in other environments.

The difficulty there is in parsing the information column, which is a big string delimited by \n, rather than additional rows per property as in describe table extended.

I think a regex approach could probably make quick work of the column names and data types. I will take a quick stab at that and post back here.

This regex search string seems to work on the sample output from above:
\|-- (.*): (.*) \(nullable = (.*)\b

This regex string outputs the three captured pairs:

column name
column type
nullable (true/false)

Link to test results and demo of this regex: https://regex101.com/r/E5YHCs/1

Very nice!
As far as the

more performant approach to the get_catalog updates
that I mentioned above, I think a natural fit here is the _get_one_catalog method that is introduced in https://github.com/fishtown-analytics/dbt/pull/2037 and will hopefully ship in the 0.16.0 release of dbt Core.

Between now and then, we can still try to ship the proposed enhancements to get_catalog as written (looping across all relations and running describe ... extended) in the 0.15.0 release of this plugin.

I'm going to close this and open a more specific issue that suggests reimplementing _get_one_catalog to operate more efficiently on one schema at a time, instead of one relation at a time.


Performance improvement for queries having join between TABLE_CONSTRAINTS and KEY_COLUMN_USAGE views

Description
This commit addresses poor performance issues in queries that join the TABLE_CONSTRAINTS and KEY_COLUMN_USAGE views. The problem was caused by inaccurate row estimates resulting from a CASE expression in the TABLE_CONSTRAINTS view, leading to suboptimal query plans.
Signed-off-by: Sumit Jaiswal<EMAIL_ADDRESS>Issues Resolved
Task: BABEL-5427
Test Scenarios Covered
Test are already present for modified views.
Performance Result

Improvement for query having join between TABLE_CONSTRAINTS and KEY_COLUMN_USAGE:

Objects
Before (ms)
After (ms)
Improvement %

1181 tables and primary key constraints
4781
87
98.18%

3000 tables and primary key constraints
33011
98
99.70%

3000 tables, primary key, unique and check constraints
14173
116
99.18%

2 Improvement for different queries:

Objects
Query
Before (ms)
After (ms)
Improvement %

1181 tables and primary key constraints
SELECT * FROM INFORMATION_SCHEMA.TABLE_CONSTRAINTS
29
32.2
-11.03%

^
SELECT * FROM INFORMATION_SCHEMA.TABLE_CONSTRAINTS WHERE TABLE_NAME = 't456'
38
33.6
11.58%

^
SELECT * from INFORMATION_SCHEMA.TABLE_CONSTRAINTS WHERE TABLE_SCHEMA = 'dbo'
13.8
16.8
-21.74%

^
SELECT * from INFORMATION_SCHEMA.TABLE_CONSTRAINTS WHERE CONSTRAINT_TYPE = 'PRIMARY KEY'
24.4
16.6
31.97%

^
SELECT * from INFORMATION_SCHEMA.KEY_COLUMN_USAGE
22
20.2
8.18%

^
SELECT * from INFORMATION_SCHEMA.KEY_COLUMN_USAGE WHERE TABLE_NAME = 't456'
8
6.4
20%

^
SELECT * from INFORMATION_SCHEMA.KEY_COLUMN_USAGE WHERE TABLE_SCHEMA = 'dbo'
21.4
19.2
10.28%

3000 tables, primary key, unique and check constraints
SELECT * FROM INFORMATION_SCHEMA.TABLE_CONSTRAINTS
105.2
128
-21.67%

^
SELECT * FROM INFORMATION_SCHEMA.TABLE_CONSTRAINTS WHERE TABLE_NAME = 't456'
79.8
58.2
27.07%

^
SELECT * from INFORMATION_SCHEMA.TABLE_CONSTRAINTS WHERE TABLE_SCHEMA = 'dbo'
77.8
100.6
-29.31%

^
SELECT * from INFORMATION_SCHEMA.TABLE_CONSTRAINTS WHERE CONSTRAINT_TYPE = 'PRIMARY KEY'
61.4
53.2
13.36%

^
SELECT * from INFORMATION_SCHEMA.KEY_COLUMN_USAGE
71.4
65.6
8.12%

^
SELECT * from INFORMATION_SCHEMA.KEY_COLUMN_USAGE WHERE TABLE_NAME = 't456'
12
10
16.67%

^
SELECT * from INFORMATION_SCHEMA.KEY_COLUMN_USAGE WHERE TABLE_SCHEMA = 'dbo'
75.6
67.2
11.11%

NOTE: We don't observe ANY degradation if we have 100 user schemas
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 74.826%

Totals

Change from base Build<PHONE_NUMBER>6:
0.004%

Covered Lines:
46439

Relevant Lines:
62063

💛  - Coveralls


Player Controls (previous, play, next) are disabled on lock screen in iPad.

Here is my code to setup the player.
Player Controls on lock screen is disabled.
TrackPlayer.updateOptions({
// One of RATING_HEART, RATING_THUMBS_UP_DOWN, RATING_3_STARS, RATING_4_STARS, RATING_5_STARS, RATING_PERCENTAGE
ratingType: TrackPlayer.RATING_5_STARS,
// The maximum height or width that the artwork can have. It will be resized when necessary. The lower the number is, the lower is the memory used
maxArtworkSize: 800,
// An array of media controls capabilities
// Can contain CAPABILITY_PLAY, CAPABILITY_PAUSE, CAPABILITY_STOP, CAPABILITY_SEEK_TO,
// CAPABILITY_SKIP_TO_NEXT, CAPABILITY_SKIP_TO_PREVIOUS, CAPABILITY_SET_RATING
capabilities: [
TrackPlayer.CAPABILITY_PLAY,
TrackPlayer.CAPABILITY_PAUSE,
TrackPlayer.CAPABILITY_SKIP_TO_PREVIOUS,
TrackPlayer.CAPABILITY_SKIP_TO_NEXT,
TrackPlayer.CAPABILITY_SEEK_TO
],
// Notification Color (Must be an ARGB Hexadecimal number)
color: 0x2ecc71
});
TrackPlayer.setupPlayer().then(() => {
var index = 0;
this.state.songs.map(song => {
index ++;
TrackPlayer.add({
id: 'track'+index,
url: song.url,
title: song.title,
artist: song.artist,
artwork: song.songImage
});
});
		    // Starts playing it
		    TrackPlayer.play();

		});

Hello, this issue was previously reported and addressed here: https://github.com/react-native-kit/react-native-track-player/issues/77


Mouse events don't work after the program runs for a while

version:  0.9.5
Looks like mouse events travel through the application's window,The application's window does not handle mouse events，
Mouse events are responded to on the desktop
After running for a while, this problem will appear occasionally
Is there any solution for this problem

That's a very old version.
xrdp isn't aware of any applications on the desktop - it just delivers mouse events via the X server. So if you're getting desktop events it's hard to see how a problem with your application can be an xrdp problem. I think you need to look at the X11 infrastructure in general.
I can give you a few pointers in what you can look for:-

Try running xev and see if the events are getting to that.
xdotool can let you do things like send mouse up events to your application. See this stackexchange question for more info.

Ok, thank you for your reply, the xserver I use is xorgxrdp，Should I go from xorgxrdp to solve this problem?

ok thanks i will try


added fullscreen support

Closes #6
@jwerle
much full. so screen.

Looks good. However, we need the frame to occupy 100% of the screen. This should be overridable by an option passed into the constructor (maintainAspectRatio: true|false)

I've updated this branch with master so it is now possible to merge

:+1:


Is it possible to add new methods to APIM using c# code?

Is it possible to add new methods to APIM using c# code?
Basically, we want to generate APIM instance from c# code and publish to Azure.

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

ID: d08ff505-e3c3-8951-4804-1c697b48d9b7
Version Independent ID: b71f9065-a350-bed4-dac7-8195d82c2b0f
Content: Tutorial - Import and manage APIs - Azure API Management and Visual Studio Code
Content Source: articles/api-management/visual-studio-code-tutorial.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

Hello @codemaker - Thank you for your feedback! I think the creation, publishing of the service can be done via Powershell or ARM (not sure if the SDK can do it) but for all of the other operations, SDK should cover them: https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.management.apimanagement?view=azure-dotnet

@codemaker I hope the links in the previous comment help you get started. Since we didn't determine any changes to this doc upon reviewing your feedback, we will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

Some sample code in the document would help the community.


Crash on startup

Latest payday release (not the beta) crashes on startup with the editor installed.
Crash log:
PAYDAY 2 Crash log
Application has crashed: C++ exception
[string "lib/utils/game_state_machine/gamestatemachi..."]:601: attempt to index a nil value
SCRIPT STACK
-------------------------------
Callstack:
         payday2_win32_release  (???)     ???
         payday2_win32_release  (???)     ???
         payday2_win32_release  (???)     ???
         payday2_win32_release  (???)     ???
         payday2_win32_release  (???)     zip_get_name
-------------------------------
Current thread: Main

https://github.com/simon-wh/PAYDAY-2-BeardLib-Editor/issues/7


FCirc Multi 3 - Validation Database Changes

Summary Change Description
Almost everything changed in here relates to changes necessary to make the file validation code and validation database work with the new multi tracer/label related models (infusate, tracer, tracer label, etc.  The maitained model had to change as well because of the previously un-encountered issue of performing maintained field operations on a different database.  When validation failed, the buffered updates had to be flushed and the model managers had to be aware of which database to update.  The managers were changed from plain managers to querysets "as managers" so that .using(db) could be applied to them.
Affected Issue Numbers

Resolves #395
Resolves #398

Code Review Notes
The only substantial portion of these changes that are not related to the theme of this PR are the fcirc, animal_label, and possibly peak_group_label loads in the sample table loader and .using(db) changes in the sample table loader code (which is also in a separate PR).
I turned off the CI tests for this PR.  These changes are just copies of files from the fcirc_multi branch dropped into a new branch off the infusate_label branch (the starting point of the master branch for this effort).  I am breaking that code review up into these smaller PRs that are separated by theme.  I am doing it by file simply because it's easier.  If any code here references something you cannot find, just consult to fcirc_multi branch/PR.
The PRs I will be submitting are:

fcirc_multi_models
fcirc_multi_format
fcirc_multi_validation (this PR)
fcirc_multi_pagination
fcirc_multi_tests
fcirc_multi_minor (comments/renames/etc)

Checklist
These are the checks for the overall change set...

[x] All issue requirements satisfied (or no linked issues)
[x] Linting passes.
[x] Migrations created & committed (or no model changes)
Tests

[ ] Tests implemented (or no code changes)
[x] All multi_working tag tests pass locally
[x] All example load tests pass (remotely) (or their failures predated and are unrelated to this branch)
[x] @tag("multi_working") added to newly passing test functions/classes (or no newly passing tests)
[x] @tag("multi_broken"), @tag("multi_unknown"), or @tag("multi_mixed") removed from newly passing test functions/classes (or no newly passing tests)

Fine by me! [though I would prefer eventually swapping the multiple databases paradigm]

Yeah, I would prefer that as well.  Unfortunately, validation of the accucor files requires that the sample table already be loaded.  Perhaps we should have an issue to implement validation without the sample load being necessary? Is that something you'd be interested in tackling?
The other reason is that running the accucor load in debug mode has database side-effects (#357), so even if we worked out how to do the accucor validation without loading the sample table, validating still isn't cleanly possible yet either.

Defering merge to #493


Prevent using github.com/tj/assert and remove existing use

What does this PR do?

Add github.com/tj/assert to the import deny list, and replace the single existing use with github.com/stretchr/testify/require.
Motivation

github.com/tj/assert is a wrapper around github.com/stretchr/testify/require, but having the same package name as github.com/stretchr/testify/assert it is actually auto-completed accidentally, and its behavior is counter-intuitive when familiar with testify.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Adding automate-backend-kibana

Signed-off-by: FaizanSRE<EMAIL_ADDRESS>:nut_and_bolt: Description: What code changed, and why?
For Code Movement to the Automate Repo we have 12 components, so as part of this PR we have targeting below components.
automate-backend-kibana

:chains: Related Resources
:+1: Definition of Done
As Part of this we want the components to move in automate-repo and can be build their habitat package in that environment.
:athletic_shoe: How to Build and Test the Change
To build this package
Cloning this PR changes
Enter hab studio using hab studio enter
build respective components like for example , build components/automate-backend-kibana
:white_check_mark: Checklist
All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[ ] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[ ] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[ ] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[ ] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[ ] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

@kalroy I have resolved the conflicts , now we can merge it


question: Authenticating across domains

It's been asked before about authenticating across subdomains in #57. But can it be done across different domains?

So I have a great setup working authenticating against a local backend with multiple different subdomains being secured.
Now, I want to secure a different domain. For example, if my portal / auth domain is auth.example.com, and it is already securing say wiki.example.com, I now want to secure something.another.co.uk. Is this possible at all?

Existing caddy config

auth.mooncarrot.space {
	route {
		# Ref https://github.com/greenpau/caddy-auth-portal/issues/175#issuecomment-927305564
		authp {
			backend local /srv/caddy/data/users_db.json local
			cookie domain mooncarrot.space
			ui {
				links {
					"User Settings" "/settings" icon "las la-cog"
					"Identity (debug)" "/whoami" icon "las la-bug"
				}
			}
			transform user {
				match origin local
				action add role authp/user
			}
		}
	}
}
*.mooncarrot.space {
# Ref https://github.com/caddyserver/format-encoder
log {
format formatted
}
tls /etc/letsencrypt-cluster/live/mooncarrot.space/fullchain.pem /etc/letsencrypt-cluster/live/mooncarrot.space/privkey.pem

route {
	jwt {
		primary yes
		allow roles authp/admin authp/user
		set auth url https://auth.mooncarrot.space
    }

	reverse_proxy https://elsewhere.mooncarrot.space:443 {
		#header_down strict-transport-security	"max-age=31536000"
	}
}

header strict-transport-security	"max-age=31536000"
header x-frame-options				"SAMEORIGIN"
header x-xss-protection				"1; mode=block"
header x-content-type-options		"nosniff"

}

...if I just do something like this:
sub.anotherdomain.co.uk {
	# Ref https://github.com/caddyserver/format-encoder
	log {
		format formatted
	}

	tls /path/to/fullchain.pem /path/to/privkey.pem

	route {
		jwt {
			primary yes
			allow roles authp/admin authp/user
			set auth url https://auth.mooncarrot.space
		}

		root * /srv/stuff
		file_server
	}

	header strict-transport-security	"max-age=31536000"
	header x-frame-options				"SAMEORIGIN"
	header x-xss-protection				"1; mode=block"
	header x-content-type-options		"nosniff"
}

....then it redirects in an infinite loop between sub.anotherdomain.co.uk and auth.mooncarrot.space once I have signed in.

@sbrl , try researching the following and see how it can be useful to you:
Access-Control-Allow-Origin: http://foo.bar.com
Access-Control-Allow-Credentials: true

@sbrl , after re-reading your question, i think it might not be possible.

Hrm, ok. I've tried adding those headers with
# ...

	route {
		header "access-control-allow-origin" "https://ci.starbeamrainbowlabs.com"
		header "access-control-allow-credentials" "true"

		jwt {
			primary yes
			allow roles authp/admin authp/user
			set auth url https://auth.mooncarrot.space
			#crypto key verify a138a4f3-830e-42f4-bb55-27fd3943c368

	    }

		reverse_proxy https://fabio.mooncarrot.space:443 {
			#header_down strict-transport-security	"max-age=31536000"

		}

	}

# ....

....but it doesn't seem to send the headers in question.
I guess to solve this problem I need to either:

Move ci.starbeamrainbowlabs.com to be ci.mooncarrot.space.
Setup a new caddy-auth-portal instance on auth.starbeamrainbowlabs.com
Use HTTP basic authentication on this domain instead

@sbrl , i would say option 2 is in order.


AnimationPlayer: cannot insert key for Rect2 properties (like rect2.x)

Godot version:
3.1 stable
OS/device including version:
Ubuntu 18.04
Issue description:
Trying to insert a key for a rect property like Sprite:region_rect:x throws an error in output and no key is added, this break the possibility of making animated regions by axis.
The error is:
core/undo_redo.cpp:291 - Error calling method from signal 'track_insert_key': 'Animation::track_insert_key': Method expected 3 arguments, but called with 2.

Is possible that more built in types are affected by this bug.
This works fine on 3.0.x and there are no issues with Vector2 for example.
Steps to reproduce:
Try to animate region_rect:x on a sprite (not the full rect).

This error happens because value variable from this line:
https://github.com/godotengine/godot/blob/7b67c5857fd7715774c7a2e07a05ff1fb7304ed1/editor/animation_track_editor.cpp#L3753
Is never initialized, in _find_hint_for_track function
https://github.com/godotengine/godot/blob/7b67c5857fd7715774c7a2e07a05ff1fb7304ed1/editor/animation_track_editor.cpp#L2967-L2981

What should it be initialized to?

Like the rest, current value I guess? If it cannot be retrieved, a Rect(0,0,0,0) would be fine.

I think you should be animating region_rect:position:x, since Rect2 has no member named x.

God(ot)s, I'm dumb :sweat_smile:
Thanks @bojidar-bg !
ps: I guess the player should be more clear when adding a broken property, but this is solved then.


Handling TextField Input

Implemented functionality for reading/storing/retrieving text field inputs in a way that makes them persistent accross different unity sessions
UI Changes:
The "Deploy Bundle" Section of quick deploy looks like this:

Resolved all the comments.
Current UI view:

Current View of the UI on the "Deploy Bundle" Section(the only one affected by this PR):


Fix deprecated setting true for action_dispatch.show_exceptions

@yuki24 Could this be merged in?  Set to :all instead. (called from block (3 levels) in <class:RambulanceTest> at test/initializers/rambulance_test.rb:51)

Thanks @jclusso for your PR and @olivier-thatch for the heads up. I wanted to cut a new release sooner than later so I made a change by myself rather than leaving feedback here. Regardless, I really appreciate your time and effort. Thank you!

This should be fixed with the newly released version 3.1.0.

Thank you! 🚀


Final switch of DNS for production environment

[ ] Ensure that the new prod environment is online and functioning.
[ ] Ensure that the site has a homepage (we redirect to DMPTool at the moment)
[x] Update the data stack so that S3, Cognito and DynamoDB are protected from accidental deletion.
[ ] Have IAS point dmphub.cdlib.org to point to the new DMPHub prod system

this is done. we have been running the system in production since January


How to update adaptive cards added to conversations through messaging extensions

When an adaptive card with actionable buttons has been added to a message/conversation through a messaging extension (query). How can that card/message be updated?
I found no support through the Microsoft Graph to update a message (PATCH?) and the bot builder (v4) updateActivity() does not seem to work. It gives an error of Error: Unknown activity type when executing the updateActivity in an incoming composeExtension/onCardButtonClicked activity

@wictorwilen Bot cannot update an adaptive card sent through Message extension because it is sent by a user and not Bot.

@wictorwilen The work-around for this would be to use the action-based flow with the message preview, that way the message in the channel comes from a bot, and not the user. See: https://docs.microsoft.com/en-us/microsoftteams/platform/concepts/messaging-extensions/create-extensions#respond-with-an-adaptive-card-message-sent-from-a-bot

@clearab the link you point to is no longer valid. How can we achieve this?

That link redirects to the correct page. I'm not sure what you mean by invalid?
https://docs.microsoft.com/en-us/microsoftteams/platform/messaging-extensions/how-to/action-commands/define-action-command#respond-with-an-adaptive-card-message-sent-from-a-bot


Add ja-JP translation

I have added the Japanese (ja-JP) localization in this pull request.
I look forward to any feedback, and I am ready to make any required modifications.
best!

Nice, thanks for the contribution!
Tonight I will fix the build error that happened on the Flutter dependencies update.


JwtSecurityTokenHandler.CreateJwt no longer sets JwtSecurityToken.SigningCredentials

Moved from https://github.com/aspnet-contrib/AspNet.Security.OpenIdConnect.Server/issues/232.

The recent changes introduced by @brentschmaltz in this commit broke CreateJwt, which is no longer able to flow the SigningCredentials taken as a parameter to the JwtSecurityToken it returns due to the (weird) lossy serialization + deserialization process it's now using:
https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/blob/KDev/src/System.IdentityModel.Tokens.Jwt/JwtSecurityTokenHandler.cs#L371
This is obviously a serious regression since calling WriteToken to serialize the JWT now results in a compact JWT that lacks the signature.
/cc @brentschmaltz @tushargupta51

@PinpointTownes thanks, will look into this.

Will put back the original behavior.

Thanks! :clap:

@brentschmaltz people want your fix :trollface:

@PinpointTownes It was committed today. Should be in tonight's myget.release.

Which package would that be? Because force-refreshing my packages from aspnetcidev doesn't seem to do the trick.

It's not in the aspnetcidev feed yet, it's only in the azureadwebstacknightly:
https://www.myget.org/gallery/azureadwebstacknightly
See the version for: System.IdentityModel.Tokens.Jwt
You'll need to add that feed to your nuget config if you want to pull that package from there before it gets into aspnetcidev.

Ah, well that both explains it and makes sense at the same time!

Unfortunately, trying to use the azureadwebstacknightly feed - as I had initially suggested - will cause more pain because ASOS needs to be fixed to react to a breaking change in IdentityModel: https://github.com/aspnet-contrib/AspNet.Security.OpenIdConnect.Server/issues/232#issuecomment-207046066
Sadly, since the updated package is not on the aspnetcidev feed we use, I can't push the fix ATM.


runjb commands for gitbash on windows?

I'm working on a project that utilizes a makefile to build a jupyterbook. On windows, this results in the codec error, which led me to install runjb to my conda environment.
However, attempting to run runmyst.sh and runjb.sh on gitbash results in a command not found error.
I am unsure if I am missing something obvious or if runjb is not configured to work on gitbash on windows.

Luckily, the newest version of jupyter-book has fixed all the windows problems that we know of.    The new release is being tested here:  https://github.com/executablebooks/jupyter-book/pull/911
so you can either wait a day or two for version 0.8 to come out, or install it like this:
pip install<EMAIL_ADDRESS>--upgrade
if that doesn't work and you can point me to a repo with your book I can try  to diagnose


Add CodeLite IDE

Add CodeLite IDE
Microsoft Reviewers: Open in CodeFlow

Blocked by #91 Also the Url is not pointing to specific version but to latest stable version.

/AzurePipelines run


Game not recognizing Saves

Issue checklist
Note: these are not necessarily requirements

[X ] I have checked existing OPL issues for duplicates and found none @ http://psx-scene.com/forums/official-open-ps2-loader-forum/ & https://github.com/ifcaro/Open-PS2-Loader/issues
Please don't kill me if I missed something on PSX-Scene (Please create a list with all current and past issues that were reported in the OP, 50 pages are not funny to read through)

Open PS2 loader version
0.9.3 (booted via FreeMCBoot v1.953, installed on a 8MB Memory Card)
Open PS2 loader mode (USB, SMB, HDD)
USB
Open PS2 type (Normal, Childproof, VMC, GSM, PS2RD)
VMC + GSM + PS2RD
Issue description - Describe the problem you are having
Dragonball Z Budokai Tenkaichi 3 greys out 'Continue' option in the Main Screen (which usually happens only if there is no Save Game present). If I press 'New Game' though, it tries to save the game and asks me if I want to overwrite my current Save Game (Normal expected behaviour, as if the Save Game would be (and it is) there)
Expected result
Be able to press 'Continue' in the Main Menu of Dragonball Z Budokai Tenkaichi 3
Actual result
Main Menu does not offer 'Continue' option (greyed out), though a Save Game is present (that the game also seems to recognize when trying to save, as it asks to overwrite it)
How to reproduce the issue

PlayStation 2 Slim (SCPH-77004)
Use VMC (64MB) on a 32 GB FAT32 formatted SanDisk 3.0 USB (don't have the exact model)
The VMC file was generated by OPL, with the default File Name
The Game ISO was not split and sits in /DVD

Please tell if some information is missing, I'm happy to provide it (given that I have that information)

Hello,
The VMC is not 100% compatible with all games, according with the compatibility list:
http://sx.sytes.net/oplcl/games.aspx
this game is "untested" with the VMC loaded from USB, and probably is just not compatible.
Also is not recommended an VMC more big than the standard 8MB because they can be
more incompatible, try with a new one of 8MB instead 64MB.
And last please for this kind of issues open a thread in the official forums:
http://psx-scene.com/forums/official-open-ps2-loader-forum/
Best regards.

@ElPatas1
Thanks for the Advice! This Issue was indeed caused by the 64MB VMC file. Generating an 8MB file fixed the problem. The game seems to be compatible with VMC (Games are loading and saving without problems)

I'm glad that the solution was easy and all runs fine.
Best regards.


Utility for replacing BatchNorm with MultiNodeBatchNorm

Many models written in Chainer are using normal BatchNorm and we have to replace them with MultiNodeBatchNorm when we use ChainerMN. It would be nice if Chainer/ChainerMN has a utility for this replacement.

FYI: My code used during OIC'18.
import chainer
import chainermn
import copy

from . create_bn_frozen_model import FrozenBatchNormalization
from oic.links.batch_normalization import MultiNodeBatchNormalizationWithPureNccl

def create_mnbn_model(link, comm, using_pure_nccl=False):
    """Returns a copy of `link`, where BatchNormalization is replaced by MultiNodeBatchNormalization.

    """
    if isinstance(link, chainer.links.BatchNormalization) and \
            not isinstance(link, FrozenBatchNormalization):
        if using_pure_nccl:
            mnbn = MultiNodeBatchNormalizationWithPureNccl(
                size=link.avg_mean.shape,
                comm=comm,
                decay=link.decay,
                eps=link.eps,
                dtype=link.avg_mean.dtype,
                use_gamma=hasattr(link, 'gamma'),
                use_beta=hasattr(link, 'beta'),
            )
        else:
            mnbn = chainermn.links.MultiNodeBatchNormalization(
                size=link.avg_mean.shape,
                comm=comm,
                decay=link.decay,
                eps=link.eps,
                dtype=link.avg_mean.dtype,
                use_gamma=hasattr(link, 'gamma'),
                use_beta=hasattr(link, 'beta'),
            )
        mnbn.copyparams(link)
        for name in link._persistent:
            mnbn.__dict__[name] = copy.deepcopy(link.__dict__[name])
        return mnbn
    elif isinstance(link, chainer.Chain):
        new_children = [
            (child_name, create_mnbn_model(link.__dict__[child_name], comm, using_pure_nccl))
            for child_name in link._children
        ]
        new_link = copy.deepcopy(link)
        for name, new_child in new_children:
            new_link.__dict__[name] = new_child
        return new_link
    elif isinstance(link, chainer.ChainList):
        new_children = [
            create_mnbn_model(l, comm, using_pure_nccl) for l in link]
        new_link = copy.deepcopy(link)
        for i, new_child in enumerate(new_children):
            new_link._children[i] = new_child
        return new_link
    else:
        assert isinstance(link, chainer.Link)
        return copy.deepcopy(link)


[DOCFIX] Fix HDFS Doc minor stuff

Fix some typos in HDFS doc.

alluxio-bot, merge this please

@Xenorith Please help cherry pick this as well. Thanks!

alluxio-bot, cherry-pick this to branch-2.7 please


Upgraded Qobo cakephp-menu plugin to v13.* (task #809)

For more details see release v13.0.0

Codecov Report

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

@@            Coverage Diff            @@
##             master     #591   +/-   ##
=========================================
  Coverage     29.47%   29.47%
  Complexity     1115     1115
=========================================
  Files            94       94
  Lines          3719     3719
=========================================
  Hits           1096     1096
  Misses         2623     2623

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 fdd794b...f16a16e. Read the comment docs.


Add attention layers

We should work out a common API and implementation for working with attention layers. Right now they are easily implemented as custom layers, but we should have conveniences to save user's time

Can I take this on?

Yes, definitely! Let me know if you run into any issues or have questions


Autocad has problem with some entity witch draw with dxf-writer

When i draw  Point or Arc , autocad cant open generated dxf file even if use generateAutocadExtras()

This issue is resolved in PR #63

This issue is resolved in PR #63

Thanks


Update bcpkix-jdk18on to 1.76

About this PR
📦 Updates org.bouncycastle:bcpkix-jdk18on from 1.75 to 1.76
Usage
✅ Please merge!
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.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (1.75).
You might want to review and update them manually.
website/static/img/undraw_code_review.svg
website/static/img/undraw_operating_system.svg

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.bouncycastle", artifactId = "bcpkix-jdk18on" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.bouncycastle", artifactId = "bcpkix-jdk18on" }
}]

labels: library-update, old-version-remains, commit-count:1

Superseded by #549.


Better test data required

Currently our test data is limited to a handful of small and synthetic VCFs (see https://github.com/ga4gh/server/tree/master/tests/data). We need a wide spread of test data to exercise our APIs, across variants, reads and references. We should have a collection typical small data files spanning many anticipated use cases.

This issue is too general.  Can you try and narrow the scope a bit?

OK, fair enough. We need the following to close this issue:

Test reference data;
Test read data;
More test VCF data, covering different format versions, numbers of samples, ploidy levels, etc. How much more depends on how much we think it's worth covering exotic corner cases.

I'm not really informed enough to know what would be considered a representative spread of examples, so it would be good to get some wider opinions here. Anyone like to weigh in here?

@jeromekelleher, I think you gave a great suggestion a while back in using the Illumina's Platinum Genomes dataset:
https://github.com/ga4gh/compliance/issues/13#issuecomment-59219513
~p

missing a test example such that has a variant site, where for the INFO and FORMAT field, NUMBER=R.
For details, see http://samtools.github.io/hts-specs/VCFv4.2.pdf, section 1.2.2 and 1.2.4

NUMBER=R is new for vcf4.2, a related entry is found on http://sourceforge.net/p/samtools/mailman/message/32900784/

Test data has been greatly improved since this issue.


sudo apt-get update twice in Install the Azure Functions Core Tools?

For steps 4 and 5 under the "Install the Azure Functions Core Tools" steps for linux, is sudo apt-get update really supposed to be run twice in a row? If so, please add a comment indicating this isn't a mistake

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

ID: b2646ad4-34b7-fc5e-dcc9-d53a061df394
Version Independent ID: bbff4fda-24e4-2b62-24b9-94cda5f53bed
Content: Work with Azure Functions Core Tools
Content Source: articles/azure-functions/functions-run-local.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Thanks for the question! We are taking a look into this and will get back to you soon.

@bichuga , Its not required to be run twice. I will change the documentation. Thanks for your feedback


[Bug]: Too Long strings cause the Container to crash.

What happened?
We've enabled Librechat and configured it to use OpenAI and Assistants.
When using Assistants and entering a large string like code, it crashes the container.
Also the container is running on Kubernetes.
librechat-chart-689f58748d-94hbr librechat-chart 2024-03-25 11:40:17 error: [/assistants/chat/] 400 Invalid 'messages[0].content': string too long. Expected a string with maximum length 32768, but got a string with length 49468 instead.
librechat-chart-689f58748d-94hbr librechat-chart 2024-03-25 11:40:17 error: There was an uncaught error: Cannot set headers after they are sent to the client
librechat-chart-689f58748d-94hbr librechat-chart npm notice

Furthermore the 'error: [/assistants/chat/] 400 Invalid ' - isnt shown as Error, but is actually a Debug.
When filtering in ELK like logging solutions - this error is picked up under DEBUG category and not Error.
Its probably because of this:
https://github.com/danny-avila/LibreChat/blob/a4f4ec85f8ebb9a08acd2728848feff452722822/api/server/routes/assistants/chat.js#L57
thank you very much!!
Steps to Reproduce

Enable assistants and create a Code optimized assistant
Paste a large string - larger than 32768 strings.

Error is thrown just before crash:
librechat-chart-689f58748d-94hbr librechat-chart 2024-03-25 11:40:17 error: [/assistants/chat/] 400 Invalid 'messages[0].content': string too long. Expected a string with maximum length 32768, but got a string with length 49468 instead.
librechat-chart-689f58748d-94hbr librechat-chart 2024-03-25 11:40:17 error: There was an uncaught error: Cannot set headers after they are sent to the client

What browsers are you seeing the problem on?
Chrome
Relevant log output
librechat-chart-689f58748d-94hbr librechat-chart 2024-03-25 11:40:17 error: [/assistants/chat/] 400 Invalid 'messages[0].content': string too long. Expected a string with maximum length 32768, but got a string with length 49468 instead.
librechat-chart-689f58748d-94hbr librechat-chart 2024-03-25 11:40:17 error: There was an uncaught error: Cannot set headers after they are sent to the client

Screenshots
No response
Code of Conduct

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

@tip-dteller Thanks for reporting this, I will fix it promptly. can you share the stack trace from error logs?

For what it's worth it doesn't crash for me because the error is caught, are you on the latest commit? I can still do something about this
2024-03-25 08:38:38 error: [/assistants/chat/] 400 Invalid 'messages[0].content': string too long. Expected a string with maximum length 32768, but got a string with length 5500... [truncated]
2024-03-25 08:38:38 debug: [/assistants/chat/] Request aborted on close

Can confirm on 2 different deployments that the error is already caught on the latest commit. However, I'm adding an informative message:

@danny-avila  thanks for the ultra quick response!!!
is it possible that you can publish a new version v6.11.x?

@danny-avila thanks for the ultra quick response!!! is it possible that you can publish a new version v6.11.x?

My pleasure. v0.7.0 will release this week, creating a milestone tracker so it is more apparent which issues need addressed before release.


Link pyspark docs in generated docs

Need to configure intersphinx

I'd merge a pull request that does this; otherwise, being pretty stale, I'm going to close this one as "won't fix in the foreseeable future"


captive portal: idle timeout triggers prematurely

via https://forum.opnsense.org/index.php?topic=328.0

works now :)


While upgrading the angular version from 8.2.14 to 9...getting error

Command
help
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Command used
npx @angular/cli@9 update @angular/cli@9 @angular/core@9
also tried with command ng update
npm ERR! 404  '@angular/cli@9' is not in this registry.
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.
node version
v16.17.0
Minimal Reproduction
npx @angular/cli@9 update @angular/cli@9 @angular/core@9
ng update
npm update
Exception or Error
No response
Your Environment
Angular CLI: 8.3.29
Node: 16.17.0
OS: win32 x64
Angular: 8.2.14
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.803.29
@angular-devkit/build-angular     0.803.29
@angular-devkit/build-optimizer   0.803.29
@angular-devkit/build-webpack     0.803.29
@angular-devkit/core              8.3.29
@angular-devkit/schematics        8.3.29
@angular/cli                      8.3.29
@ngtools/webpack                  8.3.29
@schematics/angular               9.1.15
@schematics/update                0.803.29
rxjs                              6.4.0
typescript                        3.5.3
webpack                           4.39.2

Anything else relevant?
No response

Hi @PragGawande,
This looks like any issue with your environment, if you are using a private NPM repository make sure that is synced with the public repository.
It is also important to point out that Angular versions older than version 12 are no longer supported. For more information please see https://angular.io/guide/releases#actively-supported-versions


Fix resync for SFC

Coverage decreased (-0.4%) to 71.272% when pulling 5109e66e6db55ce53568352c9d3055ca1be94d7d on rastislavszabo:sfc into 9d856e11e124a6af1da224109950103fe17e7c6b on contiv:master.


I got problem to set ontablistener

How to change ontablistener, i want to open new activity on tab selected. Thanks

You can set OnTabSelectedListener where in onEndSelected() method start new activity.


Empty/incomplete commit message

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
yarn
Package manager version
1
Language version
No response
Manifest location and content before the Dependabot update
https://github.com/mistic100/Photo-Sphere-Viewer/blob/main/package.json
dependabot.yml content
version: 2
updates:
  - package-ecosystem: npm
    directory: '/'
    schedule:
      interval: daily
    assignees:
      - mistic100
    ignore:
      - dependency-name: '*'
        update-types: ['version-update:semver-patch']
      - dependency-name: '@typescript-eslint/*'
        update-types: ['version-update:semver-minor']
      - dependency-name: 'marked'
        update-types: ['version-update:semver-major']
    commit-message:
      prefix: 'chore'
      include: 'scope'
  - package-ecosystem: github-actions
    directory: '/'
    schedule:
      interval: weekly
    assignees:
      - mistic100
    commit-message:
      prefix: 'chore'
      include: 'scope'

Updated dependency
No response
What you expected to see, versus what you actually saw
Recent update commits message are empty, only containing ---, see examples bellow. On the other hand the PR title is complete.
I didn't made any recent changes in the configuration.
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs

https://github.com/mistic100/Photo-Sphere-Viewer/pull/1310
https://github.com/mistic100/Photo-Sphere-Viewer/pull/1309
https://github.com/mistic100/Photo-Sphere-Viewer/pull/1312
https://github.com/mistic100/Photo-Sphere-Viewer/pull/1311

Smallest manifest that reproduces the issue
No response

One more example: https://github.com/codehaus-plexus/plexus-xml/pull/34/commits

Another example (fixed manually to make it pass gitlint), with an original commit message of:
---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>
.github/dependabot.yml:
version: 2
updates:
  - package-ecosystem: "github-actions"
    directory: "/"
    schedule:
      interval: daily
    commit-message:
      prefix: "build(deps)"

This is happening for me too with Python dependencies.

Here's another example: https://github.com/remarkablemark/setup-codeclimate/pull/329/commits

💸 🔥

This appears to be affecting all package managers - I'm seeing it for GitHub Actions, npm packages and NuGet packages.

Happens in bundler too 🥲

We have same issue on multiple private repos

In case this is helpful in tracking down when this went live:
Last good dependabot commit message I've seen was 2024-05-20T17:36:14Z and first bad commit was 2024-05-20T18:36:23Z - so looks like a dependabot release some time between 17:36 and 18:36 for the repos I've got visibility over.

Same issue with the ecosystems nuget, gitmodules, github-actions & devcontainers`

Same issue with the ecosystems npm, pip

Yeah, I noticed this. I also observed that when doing the squash merge via the gh CLI, it is not Dependabot, and it produced the title ---. Initially, I thought I might have forgotten to update the default PR title settings for that repository. However, now that Dependabot is writing commits with this title, could it be related to the API as well?

It's been affecting our workflows as well for the last 2 days, GitHub actions and Rust crates:

https://github.com/wrpc/wrpc/pull/79
https://github.com/wasmCloud/wasmCloud/pull/2163

We are seeing this in our repo when dependabot PRs were merged

just for completeness: it also happens on gitsubmodule ecosystem. Example: https://github.com/OpenImageDebugger/OpenImageDebugger/pull/339/commits

Same here even with package-ecosystem: "github-actions". The custom commit-message is completely ignored

Any estimate when this is going to be fixed?

I just squash-merged a commit about 5 minutes ago with @dependabot squash and merge and it looks like the commit message was fixed!

This one was for the github actions ecosystem.
I also confirmed one for the python ecosystem, specifically Poetry:

Looks like a fix got deployed very recently.

the fix for this issue has been deployed

According to GitHub Support 🎉

This appears to be fixed for new PRs, but current broken PRs will remain broken, even if you do @dependbot recreate or @dependabot squash and merge.

This appears to be fixed for new PRs, but current broken PRs are still broken, even if you do @dependbot recreate or @dependabot squash and merge.

Ya, you have to life with that or manually fix the git commits/history.

With hundreds, if not thousands, of Dependbot PRs open with invalid ---'s in the messages, wouldn't it be at least a little better to fix @dependabot recreate to fix these?


Please always check the error cores of mlock/munlock

I see that mlock fails, but you don't check for errors.

Why should I? e.g. what action could I take if I knew it failed?
App needs to run nevertheless, even if mlock fails. This just adds unnecessary code.
Instead I've removed some unnecessary mlocks (https://github.com/OpenMusicKontrollers/synthpod/commit/f8edb245bd03234102c61ed8dd67a0d2a7cf8e72), because I mlockall at app start already.

The can fail for any reason, ex. bug in your app, bug in the platform.

what action could I take if I knew it failed?

At least print a warning. Otherwise you may as well remove them, because it's ok when they fail too.


Incompatibility with GoCD version 20.2.0

Everything worked fine on GoCD version 20.1.0 but since GoCD 20.2.0 release gomatic can't create pipelines as it seems the configuration file is changed and it's not compatible anymore. Are there any plans to support it?
Example response:
RuntimeError: Could not post config to Go server (https://DNS/go/api/admin/config.xml) [status code=400] (and result was not json): <Response [400]>

Did you ever get to the bottom of this @arminc? I am facing a similar error, however I believe it must be something to do with the config in place in GoCD, I have two GoCD deployments (a dev version and a prd version) and Gomatic is working for one but not the other.

@leandrogualter any thoughts on this at all ^

FYI: There is a similar conversation happening on the GoCD mailing list: https://groups.google.com/d/topic/go-cd/7nmIWkO0QmY/discussion

Not yet, we paused the problem for now by using the old version. We still need to investigate why so we can upgrade.

I can confirm that it's specifically version 20.2 with the issue, 20.1 is fine. When I get time I will put together something to reproduce this bug.

Hi,
For those of you not subscribed to the related issue in the Go project - I have solved this problem, at least on the server I was dealing with today.  It's Jetty filesize upload limits being reset by the wrapper properties file being overwritten.
I've still not got to root-cause, but I have managed to resolve this on our instance
https://github.com/gocd/gocd/issues/7991#issuecomment-998880093