No shortcuts installed in Start Menu

I installed via WinGet but no Start Menu items were created, using search for "Handy" returns nothing, and it's not like the installer told me where it was installing the executable. How do I run this thing?

Hi @Kaylakaze
After installation, the program shortcut must be created on the desktop
However, the installation path specified in Advanced Installer is: [AppDataFolder][Manufacturer][ProductName] => C:\Users\AppData\Roaming\HandyOrg\HandyWinget
You may also find the program in this path
C:\Users\Mahdi\AppData\Local\Temp.net\HandyWinget
I'm planning to rewrite the program by UWP, Therefore, this type of problem will not occur again.

Thanks. I don't use the desktop so of course, I didn't see it. Thanks for the install location.

For now there's still nothing installed in start menu, will this feature be added or there's something going wrong on my device?

IT hasn't changed. @ghost1372 has been pretty busy, and the last public release was barely 2 weeks after this post was first made.  https://github.com/HandyOrg/HandyWinGet/releases

IT hasn't changed. @ghost1372 has been pretty busy, and the last public release was barely 2 weeks after this post was first made. https://github.com/HandyOrg/HandyWinGet/releases

Got it. Thanks for response.


#528 - updated years

@fabriciofx Thanks, I will find someone to review this PR soon

@dmzaytsev this pull request is for you,please review

@fabriciofx you have created 4 pull requests for the one ticket ...
just push into the same branch instead of creation of PR

@dmzaytsev I'm so sorry! I'm a newbie, but I learn fast! :) Thank you for this information!

@fabriciofx according the quality rules http://at.teamed.io/qa.html

Pull request title explains the problem it is fixing.
Pull request description explains the solution proposed and contains a link to the original ticket it is related to.

please update the pull request description

@fabriciofx this is your years.sh
#!/bin/sh

if (grep -L -r 2015-`date +%Y` --exclude-dir ".git" --exclude ".*" --exclude-dir "est" --exclude "*.yml" --exclude "*.md" --exclude-dir "target" . | egrep -v "(curl-appveyor.cfg.asc|src/it/file-manager/invoker.properties|src/it/file-manager/src/main/resources/org/takes/it/fm/about.html|src/main/resources/org/takes/version.properties|src/site/resources/CNAME|src/site/resources/logo|src/test/resources/org/takes/http/keystore|src/test/resources/org/takes/rs|src/www.takes.org/index.html|years.sh)"); then
    echo "Files above have wrong years in copyrights"
    exit 1
else
    exit 0;
fi

take a look at the comment teamed/qulice#624 (comment)

@fabriciofx btw you not assigned to #528

@fabriciofx furthermore #523 not accepted by @yegor256 yet
I would suggest stop working on this issue for now

@yegor256 what will we do with this PR ?
#523 still not registered as a bug and @fabriciofx not assigned to the ticket

@dmzaytsev I would suggest to drop it

@fabriciofx please close this pull request

@yegor256 can we close this PR?


Q: get_terms mysteriously returns absolutely nothing

Expected behavior
Return taxonomy data so I can do something like {{department.name}}.
Actual behavior
Nothing gets returned. Nothing can be dumped.
It certainly appears, to me, like I've done everything I need to do. I can confirm that in the admin area the taxonomy is there and is being used by my team post type.
Steps to reproduce behavior
Here's a gist with my archive PHP and Twig files, as well as the relevant snippet of my functions file: https://gist.github.com/swthate/90330c173f80d547f0eca986bf5c89d8
What version of WordPress, PHP and Timber are you using?
Example: WordPress 4.5.2, PHP 5.6.10, Timber 1.0.3
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)
Wordpress dashboard

Would http://timber.github.io/timber/#terms work?

I did come across that, but to be completely honest, it doesn't make much sense to me 😕
That's for usage in a .twig tempalte file, right? Would its usage look like this?
{{ terms( 'department', true ) }}
    {% for term in terms %}
        {{ term.name }}
    {% endfor %}

I'm guessing not, since when I do that it complains a lot:
Fatal error: Uncaught exception 'Twig_Error_Syntax' with message 'Unknown "terms" function in "archive-team.twig" at line 18.' in /Users/steven/Sites/branch/schmidt-timber/www/wp-content/plugins/timber-library/vendor/twig/twig/lib/Twig/ExpressionParser.php:573 Stack trace: #0 /Users/steven/Sites/branch/schmidt-timber/www/wp-content/plugins/timber-library/vendor/twig/twig/lib/Twig/ExpressionParser.php(351): Twig_ExpressionParser->getFunctionNodeClass('terms', 18) #1 /Users/steven/Sites/branch/schmidt-timber/www/wp-content/plugins/timber-library/vendor/twig/twig/lib/Twig/ExpressionParser.php(144): Twig_ExpressionParser->getFunctionNode('terms', 18) #2 /Users/steven/Sites/branch/schmidt-timber/www/wp-content/plugins/timber-library/vendor/twig/twig/lib/Twig/ExpressionParser.php(84): Twig_ExpressionParser->parsePrimaryExpression() #3 /Users/steven/Sites/branch/schmidt-timber/www/wp-content/plugins/timber-library/vendor/twig/twig/lib/Twig/ExpressionParser.php(41): Twig_ExpressionParser->getPrimary() #4 /Users/steven/Sites/branc in /Users/steven/Sites/branch/schmidt-timber/www/wp-content/plugins/timber-library/vendor/twig/twig/lib/Twig/ExpressionParser.php on line 573

I am loving Timber a lot, but obviously am still having a hard time wrapping my head around some key concepts. Thanks for all the help you guys have been. Hope I'm not much of a bother! What's the best way to donate to the project? I'd love to pitch in some way outside the code! :)

I don't really use terms to be honest, see what this returns:
{{ dump(department.terms()) }}
Reporting bugs/issues and promoting Timber is good enough donation for us 👍

Still nothing :(
Well, I wonder, if this is even the right scenario to be using a taxonomy. It might be, but I wonder if there's a different way to go about this. Maybe I can get your opinion on this, which would be interesting since you said you don't use taxonomies (terms and taxonomies are interchangeable? or are you refering to something else) all that often.
So if you take a look here http://branchind.com/proofs/schmidtmankato/team/ you'll see it's basically an archive with all the custom team member posts. These posts are separated according to which department taxonomy they belong to. This also allows me to show the related department members on various product pages, like this: http://branchind.com/proofs/schmidtmankato/product/helmet-heat/. If you scroll to the bottom you'll see that each product post will have a small loop showing the foremen (department members) for that product.

Ah, taxonomy is correct here.
Try  {{ dump(post.terms('department')) }}

Had to switch over to my pc, everything web related is on my mac... I'll try that out later today and get back to you.

{% for post in posts %}

{{ dump( post.terms( 'department' ) ) }}

{% endfor %}

Returns the following: https://gist.github.com/swthate/bcf0db82741fa434a349143eb4954d59

Little bit of an update:
    {% for term in post.terms('department') %}

{{ term.name }}

    {% endfor %}

This successfully returns stuff. However, it very oddly returns the following:
Roofing Foremen
Gutter Foremen
Window Foremen
Window Foremen
Window Foremen
Window Foremen
Siding Foremen
Siding Foremen
Siding Foremen
Siding Foremen
I expected it to loop through each term and return this:
Door Foremen
Gutter Foremen
Leadership
Roofing Foremen
Siding Foremen
The Sales Team
Window Foremen
I don't see any logic to why it would output what it does (first list). It doesn't match the number of taxonomy terms or items, or the number of team posts using the taxonomy...

Hmm, yes not sure to be honest without jumping into the code myself. My guilty pleasure is using ACF for a lot of relationships like this...
If you think .terms isn't returning what it should be and you can put a detailed comment together about what you expect vs what it's returning I'll have a look at fixing it.
Cheers

Thanks @connorjburton
I think what I'll do is try to get an ACF alternative working (it does just feel weird to have most of the site powered by ACF except this one tiny area) and maybe submit a more detailed comment about what's going on with the taxonomy. If I can get ACF to work for me, though, that detailed comment would be on the back burner for a rainy day, if you ever got reeeal bored :)
Thanks again!

Okay, I apologize.
I never even thought to add some arguments to my get posts context. Change posts_per_page, change the other, and there they all are! 😵
I do think I will come up with an ACF solution still though.


if applied this will create comments in programs

if applied this will create comments in various files

if applied this will create comments in various files

if applied changes will be squashed and merged


Config reload does not reinitialize all changes.

Guilds v1.6.1-RELEASE
When you reload via /guilds reload, not all changes are reinitialized. For example the prefix does not change on reload. Requires a server restart entirely.

Fixed for Prefix


Transaction details: transaction fee isn't displayed

Ttransaction fee isn't displayed on the Transaction details screen (for sent transactions)

Checked.

Transaction details screen: transaction fee is displayed incorrect for transactions with shielded coins

checked


Macros in PreBuildEvent/PostBuildEvent don't work with SDK-style projects

As explained in other issues, macro properties like $(ProjectDir) don't work in PreBuildEvent and PostBuildEvent properties when using SDK-style projects. Instead, users should create new targets (using the 'exec' command) that run before or after the PreBuildEvent and PostBuildEvent targets.
Visual Studio has already been updated so that any pre/post build commands entered through the project properties UI will create new targets instead of using PreBuildEvent/PostBuildEvent directly. We should add a note to the docs, though, explaining this so that users migrating old csproj file to the new SDK-style format know that they will need to change PreBuildEvent and PostBuildEvent usage.
Other issues related to this problem: https://github.com/dotnet/sdk/issues/1055,

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

ID: 319cf42a-5391-8a60-c005-cfbb92faf1c8
Version Independent ID: ef79e00a-e2c9-12ec-6da9-2984c381e7d1
Content: Pre-build Event-Post-build Event Command Line Dialog Box - Visual Studio
Content Source: docs/ide/reference/pre-build-event-post-build-event-command-line-dialog-box.md
Product: visual-studio-windows
Technology: vs-ide-compile
GitHub Login: @ghogen
Microsoft Alias: ghogen

Thanks. I see you also submitted a related doc issue: https://github.com/dotnet/docs/issues/15248. I will coordinate with the .NET content dev on it.

This is fixed now.


Error running TensorRT in C++ on m and l

Hi,
I am running the ./bytetrack compiled program according to https://github.com/ifzhang/ByteTrack/tree/main/deploy/TensorRT/cpp
For yolox_s_mix_det, it is a success:

However, for yolox_m_mix_det, it is a failure:

I used tools/trt.py to generate the trt files for both yolox_s and yolox_m. Could you advise?

The error occurs on the doInference(*context, blob, prob, output_size, pr_img.size()); on line 480

It was my mistake to not change img_w and img_h


Add TIFF examples

          It would also be very helpful to include examples of the new slicing and string-indexing that xarray enables. Most/all of the official `iohub` examples currently focus on the zarr reader/writer.

Originally posted by @ieivanov in https://github.com/czbiohub-sf/iohub/pull/185#pullrequestreview-1894291116

All of the example scripts are about NGFF or CC. That's because these are the formats we can write with iohub so test data doesn't need to be downloaded during documentation building. We can ask developers to always run pytest before building docs so the test data can be reused. But this will likely be a fragile hack.

https://github.com/czbiohub-sf/iohub/pull/185#issuecomment-1957988051


AssignOffsetsToTensors with GREEDY_BY_SIZE causes a segfault on NVidia

Calling AssignOffsetsToTensors with MemoryStrategy::GREEDY_BY_SIZE appears to cause a segfault on NVidia OpenCL. See issue https://github.com/tensorflow/tensorflow/issues/53800 for more details.

Fixed some undefine/duplicate definition linker error (related to logging classes) when building for Android using CMake.

Please open against master. The nightly branch is only used to trigger nightly jobs and is always overriden by master


[Bug] Keybindings.json command is not surrouned by double quotes when auto-complete

VSCode Version: 1.4.0
OS Version: Ubuntu 16.04

Steps to Reproduce:

Open keybindings.json via Preferences: Open Keyboard Shortcuts.
Move the end of the file, and add new key block by type:

{
  "key": "ctrl+e",
   "command": <cursor>
}

Now type cur, it would show completion for cursorBottom, cursorBottomSelect, etc...
Press <tab>.  Expect to see "cursorBottom". But saw cursorBottom without double quotes.

Also notice that if at step 4 we start with "cur then no completion will be show up.

@letientai299 Thanks for filing the issue but I am not reproducing this from my side. I am on latest insiders.
We are releasing preview releases of the next stable VS Code version for everyone to try and give feedback. These preview releases are not 100% tested and might be unstable but contain our latest features and bugfixes. You can give our preview releases a try from:
http://code.visualstudio.com/Download#insiders


added namespacing matching

Added namespace matching.
Tested with and without proper namespace matching.
Gonna need to update the helm-chart too.

/jira open task


opt add summary metric for file scanner #45941

bp #45941

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

run buildall


translation: fixed german account update message

german message contained unused "%s" (maybe a relict?) - now deleted.

Thanks for the PR! We actually update our translations on our Crowdin project, as the CI will overwrite anything in the repo with those translations.
Would you be able to submit the change over there, or would you prefer someone does it on your behalf? 😃

ok - so it seems a fixed translation is already available but not selected. Then I will just close this - I also voted for one of the fixed translations.


fix typo

Dinstance -> Distance

Checklist

[x] I've looked at the contribution guidelines.
[x ] This pull request is complete and ready for review.

Description

Fixed a typo, changing "dinstance" to "distance"

Thanks for catching this!


User path config

This PR mainly concerns the following:

Allow a user defined user_config.toml file to input extra parameters. Currently used to define local paths and GCP parameters.
Add nl_open_data.utils and nl_open_data.tasks to use in flows that do not require external libs (such as downloading zipped folder of csvs)
Add test_zip_csv_flow.py to download and extract a zipped folder of csv's (NOT DONE).
Start conforming rivm.py and mlz.py to test_statline_bq_flow.py. Not fully operational, and will be concluded after test_statline_bq_flow.py is finalized.
Improve test_statline_bq_flow.py - currently fully operational as a local run, without lazy_loading yet.

While this is not a really "clean" PR, I haven't updated master in a while, and I think it's good to do.

NOTE - This PR works only with statline-bq/nl_open_data-deployment branch, not with statline-bq/main.

Ok, thanks for clarifying. Let's go ahead and merge then!

Ok, thanks for clarifying. Let's go ahead and merge then!


Access to fetch at 'https://api.mixpanel.com/track' has been blocked by CORS policy: Request header field content-type is not allowed by Access-Control-Allow-Headers in preflight response.

Looks like using fetch() from browser to send events directly to the API endpoint fails at preflight OPTIONS due to missing Content-Type from Access-Control-Allow-Headers from https://api.mixpanel.com/track
fetch('https://api.mixpanel.com/track', {
  mode: 'cors',
  credentials: 'omit',
  cache: 'no-store',
  redirect: 'follow',
  method: 'POST',
  body: JSON.stringify({
    event: 'foo',
    properties: {
      token: 'lorem ipsum'
    }
  }),
  headers: {
    'Content-Type': 'application/json'
  }
});

After speaking with support, using application/x-www-form-urlencoded with a data param (undocumented) works:
const payload = {
  event: 'foobar',
  properties: {
    ..._data,
    token: 'lorem ipsum'
  }
};

const data = JSON.stringify(data);

fetch('https://api.mixpanel.com/track', {
  mode: 'cors',
  credentials: 'omit',
  cache: 'no-store',
  redirect: 'follow',
  method: 'POST',
  body: (new URLSearchParams({data})).toString(),
  headers: {
    'Content-Type': 'application/x-www-form-urlencoded'
  }
});

I was having same problem and what you posted as support response didn't solve my problem. After a year, is there any update on this? or some documentation

I was having same problem and what you posted as support response didn't solve my problem. After a year, is there any update on this? or some documentation

OK forget about what I said :) It works


Inheriting from an external class (no base() call case)

A class-successor has to call a default constructor of an external base class even when base() call is not specified explicitly.
Related to #2735 and #2189.
Steps To Reproduce
https://deck.net/abf6fe87cb3641732d977c0f8543788b
public class Program
{
    [Init(InitPosition.Top)]
    public static void Init()
    {
        /*@
var A = (function () {
    function A() {
        System.Console.WriteLine("A initialized.");
    }
    return A;
}());
        */
    }

    public static void Main()
    {
        var b = new B();
    }
}

[Virtual]
[Namespace(false)]
public class A
{
}

public class B : A
{
    public B()
    	// : base()   // Uncomment to fix
    {
    }
}

Expected Result
Console output:
A initialized.

Actual Result
Nothing was written to output.

Fixed! The merged code went to Bridge upstream dev branch and will be part of next Bridge release!


Update ModSecurity to 2.9.4

Update ModSecurity to 2.9.4

Add to change download url to Github because checksum was changing on every download

Update alpine to 3.14
Always build images before testing
Add dependabot to ensure dependencies are updated
Add github action to run tests on pull request

Hi @jcmoraisjr!
This PR includes Github Actions, that appear to be disabled for this repository. I can remove that commit if you don't want to enable them.

Sorry about taking so long to have a look on your changes. It looks pretty good! I'm a bit surprised to see that the runner has docker compose (test uses) and a lot more utilities.
Merging now, lets see how it behaves. Thanks!

https://github.com/jcmoraisjr/modsecurity-spoa/actions/runs/1050797936
o/
great job


[RFC] Authorization middleware for Expressive and PSR-7 apps

Goal
The goal of this RFC is to propose a user authorization middleware for Expressive and PSR-7 applications.
The authorization module will be used to authorize authenticated user based on URL route and HTTP methods.
Requirements
The authorization module will require an authentication module to create a user instance to be passed as PSR-7 attribute. This user attribute will be the input of the authorization middleware.
We will require a MiddlewareAuthentication::class attribute, as follows:
$user = $request->getAttribute(AuthenticationMiddleware::class, false);
if (false === $user) {
    return new Zend\Diactoros\Response\EmptyResponse(401);
}

If the user will be not available, the authorization module will returns a 401 Unauthorized response.
Moreover, for the purpose of the authorization module, we will need the user's role. We can get the role using a $user->getRole() function, a $user->role property or a $user['role'] value. The choose of which method to use is left to the user implementation.
Implementation
The idea is to build an Expressive module that offers user authorization using a Role-based access control (RBAC). We will use the zend-permissions-rbac library for implementing the RBAC system.
Using the RBAC terminology, a role is the user's role and a resource is the route name. In this way we can authorize URLs and HTTP methods for a specific user's role.
This approach is the same used in this blog post. The authorization can be configured using a PHP file like to the follows:
return [
    'roles' => [
        'administrator' => [],
        'editor'        => ['administrator'],
        'contributor'   => ['editor'],
    ],
    'permissions' => [
        'contributor' => [
            'admin.dashboard',
            'admin.posts',
        ],
        'editor' => [
            'admin.publish',
        ],
        'administrator' => [
            'admin.settings',
        ],
    ],
];

where the roles can be configured using a parent role (e.g. administrator is parent of editor). The permissions are configured using route names (e.g. contributor is allowed to access the routes admin.dashboard and admin.posts).
This authorization middleware will be consumed during the pipe of a route. Here an example:
$app->get('/admin/dashboard', [
    AuthenticationMiddleware::class,
    AuthorizationMiddleware::class,
    Action\DashboardAction::class
], 'admin.dashboard');

Before the authorization, we assumed the usage of an authentication module (AuthenticationMiddleware). The AuthenticationMiddleware will store a user object as PSR-7 attribute.
Dynamic assertion
Using the route names as resources address the goal to authorize user's role based on URLs and HTTP methods. There are some use cases where this is not enough. For instance, in a blog application we may need to authorize a user if he/she is the onwer of an article.
This scenario can be implemented using the dynamic assertion feature of zend-permissions-rbac. Implementing  the Zend\Permissions\Rbac\AssertionInterface interface we can customize the authorization adding more constrains. For instance, we can limit the access of a route only for specific user's ID as the previous blog requirement.
Prototype
A prototype of the implementation will be available at ezimuel/zend-expressive-authorization.

Discussion has moved to the forums.


Setup Error

Hi!
Im using ambari version <IP_ADDRESS>  (Deploy HDP 2.6.4  hortonwork).
I download and install airflow mpack success.
But when i add service airflow on ambari it has error on action install.
It show message:

Caught an exception while executing custom service command: <class 'ambari_agent.AgentException.AgentException'>: 'Script /var/lib/ambari-agent/cache/common-services/AIRFLOW/1.9.0/package/scripts/airflow_worker_control.py does not exist'; 'Script /var/lib/ambari-agent/cache/common-services/AIRFLOW/1.9.0/package/scripts/airflow_worker_control.py does not exist'

I don't known why script package not exist on ambari-agent. Please check with version ambari you run ok?
Many Thanks!

Hi.
Should be any problem with this version.
Have you restarted ambari-server after installation?
Please check if these scripts exist on ambari-server host (/var/lib/ambari-server/resources/mpacks/airflow-ambari-mpack-1.3.6/common-services/AIRFLOW/1.9.0/package/scripts/)
Or try to reinstall mpak (--uninstall-mpack=airflow-ambari-mpack and then install mpack againg), then restart ambari-server and all agents.

The mpack adjusted to HDP 3.0. Long time no answer. Closing the issue.


Add optional param to the identity client and api

Hey, I just made a Pull Request!
continuation of: https://github.com/backstage/backstage/pull/14533
add optional parameter to the identity client and API
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

yaay the build passed finally! :) @Rugvip I think this is ready for another round of reviews, this addresses most of the comments from this review https://github.com/backstage/backstage/pull/14533

Not stale! Waiting for a review!


ihmc single-foot support

This PR includes functions and controls to send foot poses to the IHMC controller allowing for single-foot stances.

@MarcoCar : can you make sure that the version of director used by main-distro/master uses these changes?


Uninstall AdoptOpenJDK from Mac

Hello,
How to uninstall properly AdoptOpenJDK from MacOS Catalina? I installed it with the package OpenJDK11U-jdk_x64_mac_hotspot_11.0.6_10.pkg
Is it enough to delete the folder /Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk from the system? Is a script to uninstall already released?
Many thanks.

rm -rf /Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk

The command above is not applicable in my case. After running it I have:
➜  ~ java -version
openjdk version "15.0.1" 2020-10-20
OpenJDK Runtime Environment AdoptOpenJDK (build 15.0.1+9)
OpenJDK 64-Bit Server VM AdoptOpenJDK (build 15.0.1+9, mixed mode, sharing)
➜  ~ which java
/usr/bin/java
➜  ~ ll /usr/bin/java
-rwxr-xr-x  1 root  wheel   136K Jan  1  2020 /usr/bin/java
➜  ~ ll /usr/bin/javac
-rwxr-xr-x  1 root  wheel   136K Jan  1  2020 /usr/bin/javac
➜  ~ ll /usr/bin/javaws
-rwxr-xr-x  1 root  wheel   136K Jan  1  2020 /usr/bin/javaws
➜  ~
Macos Big Sur

The command above is not applicable in my case.

Because it deletes AdoptOpenJDK 11, not 15. For 15, you obviously have to do rm -rf /Library/Java/JavaVirtualMachines/adoptopenjdk-15.jdk
The programs in /usr/bin need to stay, however, because they are provided by macOS.


Converted to gem package

Makefile to control build[/test]/publish
Added workflow to publish
Moved version into package.json so more easily read by make
Gemspec fixes

There is scope here for placing the unit tests within the CICD workflow.
I haven't explicitly done this, as I'm unsure of the consequences for development.

Will revert version.rb

Do you object to me putting Name and Owner into version.rb as losing package.json doesn't provide a single value for name?

Closing this as it has been accidentally been polluted by my forked repo.
I'll reopen another pr.


The clusterrole is missing a confimag rule

Hi,
We install sbom-operator with the helm chart. We have noticed thatsbom-operator fails to use configmap as a target because of missing rule in the clusterrole. So, it would be good to add a rule like the one below to the clusterrole template of the helm chart.
- apiGroups:
  - ""
  resources:
  - configmaps
  verbs:
  - get
  - create
  - list
  - delete

Thanks, I added the permissions to the clusterrole in the latest helm-release.


How to convert defect4j coverage information collected at the statement granularity level to function granularity level

Please, is there a way of converting coverage information collected at the statement granularity level to function granularity level? I am just curious and i never know if it is possible and how to go about it.

Hi @eyinlojuoluwa,
You can aggregate the line-level information by mapping lines to functions. Maybe the cobertura report (xml) already provides that information?
Best,
René

Hi @eyinlojuoluwa,
You can aggregate the line-level information by mapping lines to functions. Maybe the cobertura report (xml) already provides that information?
Best,
René

I am closing this issue. Feel free to reopen it (or open a new one) if you have further questions about code coverage in Defects4J.

I am closing this issue. Feel free to reopen it (or open a new one) if you have further questions about code coverage in Defects4J.


+options to hide with game/OP UI

We already have (a more advanced) way to configure this in the plugin, so this is not needed.

I don't quite understand. In the advanced visibility settings, you can hide each component with hidden game UI (admittedly I did miss this), but the text underneath says requires the option "Hide overlay on hidden game UI" under "General" to be disabled which does not exist. There's also no option for hiding with Openplanet UI.

Oh, I might've moved that to the Overlay tab, I don't remember. It should be there somewhere!

There's no "Overlay" setting tab, and I don't see it anywhere in the code

If you mean the main Openplanet "Overlay" tab, that toggle is way too global and applies to all plugins. Also would not allow me to have hidden game UI and still show OP UI, which often I do wish to do

If you leave the setting disabled, you can still have hidden game UI and visible Openplanet UI (and Dashboard's advanced visibility settings)


  No connection could be made

i have tried everything.

Hi, could you please tell me which version of Opera you have and send me the client.log file, please.
Thank you.

It same like me, but work fine while using version 0.5.1
my opera GX version LVL 5 (core: 102.0.4880.117)
client.log


Feature/assaf cohen arazi/auth

Administrative Info
[Github Issue #20]
Changes
Added accounts to the backend, forgot password flow, phone number authentication flow.
Phone number one-time-password is untested, need to work on UI before being able to test it in the app.

TODO

Testing
Created successful Postman Requests for each function

TODO

Confirmation of Change
Call any of the routes in the backend and Firebase will authenticate the account.

I have fixed all merge conflicts @NirmalAgnihotri

Merge conflicts resolved, this branch only dealt with backend


PDF support in Azure Translator synchronous document translation API

Hello, I am seeking to use the synchronous document translation API to perform PDF translation. I am unable to complete my API requests because the API always responds with {"code":"InvalidFormat","message":"The format parameter is not valid."}. I have tried various source and destination languages and various PDF files, but have had no success. I can translate other formats successfully such as DOCX.
The API documentation doesn't explicitly confirm that the synchronous API supports PDF translation. Is the feature unimplemented, or just currently broken? I have seen another person asking about the same issue on the community support forum a month ago. Thank you.
Example API call:
curl -i -X POST "https://eelec-translation-uswest.cognitiveservices.azure.com/translator/document:translate?sourceLanguage=en&targetLanguage=hi&api-version=2023-11-01-preview" -H "Ocp-Apim-Subscription-Key:<omitted>" -H "X-ClientTraceId:pdf-translation-issue-b4e3ae21" -F "document=@blank.pdf;type=application/pdf" -o "out.pdf"
Response:
content-type: application/json; charset=utf-8
access-control-expose-headers: X-RequestId,x-ms-request-id,X-RequestId-Forwarded,X-Metered-Usage
x-requestid: 944d44da-dfca-4ed3-9a93-11a68fd787f5
x-ms-request-id: 944d44da-dfca-4ed3-9a93-11a68fd787f5
x-ms-error-code: InvalidRequest
strict-transport-security: max-age=31536000; includeSubDomains; preload
apim-request-id: 944d44da-dfca-4ed3-9a93-11a68fd787f5
x-content-type-options: nosniff
x-ms-region: West US
date: Mon, 29 Apr 2024 04:38:09 GMT
{"error":{"code":"InvalidRequest","message":"The format parameter is not valid.","target":"ContentType","innerError":{"code":"InvalidFormat","message":"The format parameter is not valid."}}}```

It appears that PDF it is not a supported format in the synchronous API after all: https://learn.microsoft.com/en-us/azure/ai-services/translator/document-translation/overview#synchronous-supported-document-formats
So this ticket becomes a report that the link to the list of supported formats does not go to the correct page. See here: https://learn.microsoft.com/en-us/azure/ai-services/translator/document-translation/reference/synchronous-rest-api-guide#request-body
The link "supported document formats" links to https://learn.microsoft.com/en-us/azure/ai-services/translator/language-support but it should link the the link above. I was not able to find this page on my own despite searching. This mis-link should be fixed.
Also, is it possible to know if PDF support is a planned feature for the synchronous API?
Thanks

@micander
Thanks for your feedback! We will investigate and update as appropriate.


Parsing L2VPN VLAN ranges

Issue is #81.

Coverage: 71.886% (+0.1%) from 71.769% when pulling 149b1e62f2bcb582401308ab41d1c8da68d671cd on 24.vlan_range into 333c8d04cdff1a76c7ba1f1b407753481ef2364f on main.


Accent color

Allow custom accent colors to be used instead of the preset dark blue.

The best would be to use Monet (Material You) to have a dynamically accentuated coloring, it is classy...

Wouldn't that be incompatible with older Android versions? @Ilithy

@yephny Indeed, MY is only supported from android 12 and newer if I'm not mistaken...I understand my mistake 😅 so the two are complementary.


[BUG] Test case test_backup_volume_list failed: failed to find backup in bv.backupList().data

Describe the bug
Test case test_backup_volume_list failed: failed to find backup in bv.backupList().data:
https://ci.longhorn.io/job/private/job/longhorn-tests-regression/7785/testReport/junit/tests/test_basic/test_backup_volume_list_nfs_/
set_random_backupstore = 'nfs'
client = <longhorn.Client object at 0x7f3eeca5b550>
core_api = <kubernetes.client.api.core_v1_api.CoreV1Api object at 0x7f3eed00a7d0>

    def test_backup_volume_list(set_random_backupstore, client, core_api):  # NOQA
        """
        Test backup volume list
        Context:
        We want to make sure that an error when listing a single backup volume
        does not stop us from listing all the other backup volumes. Otherwise a
        single faulty backup can block the retrieval of all known backup volumes.
        Setup:
        1. Setup minio as S3 backupstore
        Steps:
        1.  Create a volume(1,2) and attach to the current node
        2.  write some data to volume(1,2)
        3.
To Reproduce
Run test case test_backup_volume_list
Expected behavior

Support bundle for troubleshooting

Environment

Longhorn version: master-head
Impacted volume (PV):
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: v1.31.2+k3s1

Number of control plane nodes in the cluster:
Number of worker nodes in the cluster:

Node config

OS type and version: sles 15-sp6
Kernel version:
CPU per node:
Memory per node:
Disk type (e.g. SSD/NVMe/HDD):
Network bandwidth between the nodes (Gbps):

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):
Number of Longhorn volumes in the cluster:

Additional context

Workaround and Mitigation

@mantissahz Please help update the test case if needed.

Some backup test cases will fail because they still use the volume name as the backup volume name to get related backup information or delete backup volume. I am fixing the test cases.
After multiple backup stores feature is merged, backup volume is no longer the volume name.

Verification passed on master-head (longhorn-manager 1d87226) and v1.8.x-head (longhorn-manager 7f2cd62). Test case test_backup_volume_list passed.
master-head: https://ci.longhorn.io/job/public/job/master/job/sles/job/amd64/job/longhorn-tests-sles-amd64/1135/testReport/tests/test_basic/test_backup_volume_list_nfs_/
v1.8.x-head: https://ci.longhorn.io/job/public/job/v1.8.x/job/v1.8.x-longhorn-tests-sles-amd64/5/testReport/tests/test_basic/test_backup_volume_list_nfs_/


IE8: "Object doesn't support this property or method"   V1.0.6

Using V1.0.6
I'm still seeing some problems with IE8.
The same code works fine with FF/Chrome but IE8 gives me the following error (using unminified script):
Message: Object doesn't support this property or method
Line: 183
Char: 7
Code: 0
Line 183:
var els = dom.parentNode.querySelectorAll('[' + attribute + ']');
EDIT:
It looks like this is down to the DOCTYPE.
I've got some ancient pages! Anyway of having IE8 work with QuirksMode or:

IE8 is deprecated browser

Yes, but many cookie scripts can work fine with IE8 and QuirksMode - so depreciated or not, will Google be happy for AdSense publishers to work like this?

can you post an example site?

Here you go: http://www.hmmm.ip3.co.uk/bt-master-socket-nte5.shtml
Working in FF & Chrome but IE8 gives this error:
Message: Object doesn't support this property or method
Line: 1
Char: 1815
Code: 0
URI: www.hmmm.ip3.co.uk /cookieconsent.js
That seems to be pointing around here in the script:
{var i=e.parentNode.querySelectorAll("["+t+"]");
The newer pages on my site work fine with IE8 it's just the ancient ones running in QuirksMode that have the issue.
NB. I'm using a (GitHub supplied) hack to hide the banner after it's been seen on one page without visitors having to click - so using IE's InPrivate browsing works well for trouble shooting else the banner will automatically hide.

If you're running in QuirksMode then IE runs like it's IE7 which isn't something we support.


grunt (bad) karma

Npm-shrinkwrap.json freezes versions of our client dev tools. However, as mentioned in the documentation, it can't guarantee bit-exact stability, as authors may make changes to packages without bumping versions.
tl;dr:
Someone somewhere changed something and now grunt karma blows up.
Ouch.

Update:
tests working fine on master branch...
Todo: investigate delta between that and current (failing) branch.

Turns out that the issues were with any tests that called the eventBrowser.
Removed 4 test files that called eventBrowser.
todo:

update npm packages and triage. grunt watch functionality for triggering tests after a change to test files was breaking.
put tests back into client test suite.
if needed: increase inheritance of eventBrowser functions from base class and test there.

eventsBrowserTests.zip
Uploaded using ZenHub.io


Fix default iunknown_wrapping initialization

When not specified by the user, the iunknwon_wrapping setting was being set to the value of the disable_dxr setting instead of its default value.

CI gfxreconstruct build queued with queue ID 316007.

CI gfxreconstruct build # 5496 running.


typing 'default' in a typeswitch statement incorrectly highlights default

Typing:
typeswitch ($a)
    case document-node() return ()
    default return ()

results in the default token not being identified as a keyword, whereas case is. Note that pasting the above example in results in the correct behaviour.

Is this not a duplicate of https://github.com/ligasgr/intellij-xquery/issues/177 ?

Yes, sorry. I get confused as to which bugs I have raised :).

No problem :)
Thanks for raising them. I would not be able to find all of those edge cases myself!


Make access tier in storage module configuable and updatable

This is one of PRs for implementing #607 , this PR makes access tier in storage module configuable and updatable.

Hi @zhongyi-zhang , could you please help review the PR?


Adding servers in Bulk

Can is there a section for adding server in bulk? Could there be a way to download a sample file for adding servers with tags?

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

ID: 99fcece9-57ba-c2cb-a4a4-4d9188490bd5
Version Independent ID: 63fd5c3b-76c7-a45f-1700-6fdb81706aee
Content: Manage Servers with Windows Admin Center
Content Source: WindowsServerDocs/manage/windows-admin-center/use/manage-servers.md
Product: windows-server-threshold
GitHub Login: @nwashburn-ms
Microsoft Alias: niwashbu

https://docs.microsoft.com/en-us/windows-server/manage/windows-admin-center/use/get-started#csv-file-format-for-importing-connections

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


{appservice} Fix #8889: Update error message to clarify the cause

Fixes https://github.com/Azure/azure-cli/issues/8889
Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

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

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

appservice


WorkProcessorOperator framework + TopN, Table scan WorkProcessorOperators

WIP TODO:

tests

TODO:

WorkProcessorOperator metrics support
spill support

Next steps:

Support for reader (ORC) resource (blocks) reusing in TableScan + TaskOutoutOperator pipeline
Late materialization through ScanFilterAndProjectOperator.

Example Results:
Query:
presto> select * from hive.tpch_sf10_orc.lineitem order by partkey limit 10;

Results:
WITHOUT WorkProcessor pipeline and late materialization

1) Fragment 2 [SOURCE]
     CPU: 51.04s, Scheduled: 5.64m, Input: 59986052 rows (7.30GB); per task: avg.: 59986052.00 std.dev.: 0.00, Output: 480 rows (71.47kB)
     Output layout: [orderkey, partkey, suppkey, linenumber, quantity, extendedprice, discount, tax, returnflag, linestatus, shipdate, commitdate, receiptdate, shipinstruct, shipmode, comment]
     Output partitioning: SINGLE []
     Grouped Execution: false
     - TopNPartial[10 by (partkey ASC_NULLS_LAST)] => [orderkey:bigint, partkey:bigint, suppkey:bigint, linenumber:integer, quantity:double, extendedprice:double, discount:double, tax:double, returnflag:varchar(1), linestatus:varchar(1), shipdate:date, commitda
             CPU: 4.33s (0.73%), Scheduled: 7.20s (14.10%), Output: 480 rows (71.47kB)
             Input avg.: 1153577.92 rows, Input std.dev.: 30.50%
         - TableScan[hive:tpch_sf10_orc:lineitem, grouped = false] => [orderkey:bigint, partkey:bigint, suppkey:bigint, linenumber:integer, quantity:double, extendedprice:double, discount:double, tax:double, returnflag:varchar(1), linestatus:varchar(1), shipdat
                 Estimates: {rows: ? (?), cpu: ?, memory: 0.00, network: 0.00}
                 CPU: 46.71s (7.88%), Scheduled: 9.76m (1147.09%), Output: 59986052 rows (7.30GB)
                 Input avg.: 1153577.92 rows, Input std.dev.: 30.50%
                 returnflag := returnflag:varchar(1):8:REGULAR
                 linenumber := linenumber:int:3:REGULAR
                 quantity := quantity:double:4:REGULAR
                 orderkey := orderkey:bigint:0:REGULAR
                 shipmode := shipmode:varchar(10):14:REGULAR
                 discount := discount:double:6:REGULAR
                 tax := tax:double:7:REGULAR
                 suppkey := suppkey:bigint:2:REGULAR
                 partkey := partkey:bigint:1:REGULAR
                 shipinstruct := shipinstruct:varchar(25):13:REGULAR
                 linestatus := linestatus:varchar(1):9:REGULAR
                 extendedprice := extendedprice:double:5:REGULAR
                 comment := comment:varchar(44):15:REGULAR
                 receiptdate := receiptdate:date:12:REGULAR
                 commitdate := commitdate:date:11:REGULAR
                 shipdate := shipdate:date:10:REGULAR

2) WITH WorkProcessor pipeline and late materialization

 Fragment 2 [SOURCE]
     CPU: 22.81s, Scheduled: 4.44m, Input: 0 rows (0B); per task: avg.: 0.00 std.dev.: 0.00, Output: 480 rows (71.47kB)
     Output layout: [orderkey, partkey, suppkey, linenumber, quantity, extendedprice, discount, tax, returnflag, linestatus, shipdate, commitdate, receiptdate, shipinstruct, shipmode, comment]
     Output partitioning: SINGLE []
     Grouped Execution: false
     - TopNPartial[10 by (partkey ASC_NULLS_LAST)] => [orderkey:bigint, partkey:bigint, suppkey:bigint, linenumber:integer, quantity:double, extendedprice:double, discount:double, tax:double, returnflag:varchar(1), linestatus:varchar(1), shipdate:date, commitda
             CPU: 10.00ms (0.00%), Scheduled: 14.00ms (0.06%), Output: 480 rows (71.47kB)
             Input avg.: 9.23 rows, Input std.dev.: 28.87%
         - TableScan[hive:tpch_sf10_orc:lineitem, grouped = false] => [orderkey:bigint, partkey:bigint, suppkey:bigint, linenumber:integer, quantity:double, extendedprice:double, discount:double, tax:double, returnflag:varchar(1), linestatus:varchar(1), shipdat
                 Estimates: {rows: ? (?), cpu: ?, memory: 0.00, network: 0.00}
                 CPU: 22.80s (8.56%), Scheduled: 4.44m (1167.50%), Output: 480 rows (71.47kB)
                 Input avg.: 0.00 rows, Input std.dev.: ?%
                 returnflag := returnflag:varchar(1):8:REGULAR
                 linenumber := linenumber:int:3:REGULAR
                 quantity := quantity:double:4:REGULAR
                 orderkey := orderkey:bigint:0:REGULAR
                 shipmode := shipmode:varchar(10):14:REGULAR
                 discount := discount:double:6:REGULAR
                 tax := tax:double:7:REGULAR
                 suppkey := suppkey:bigint:2:REGULAR
                 partkey := partkey:bigint:1:REGULAR
                 shipinstruct := shipinstruct:varchar(25):13:REGULAR
                 linestatus := linestatus:varchar(1):9:REGULAR
                 extendedprice := extendedprice:double:5:REGULAR
                 comment := comment:varchar(44):15:REGULAR
                 receiptdate := receiptdate:date:12:REGULAR
                 commitdate := commitdate:date:11:REGULAR
                 shipdate := shipdate:date:10:REGULAR

TableScan CPU stats in 2) are stats for WorkProcessorPipelineOperators that combines TableScan and TopNPartial operators and uses late materialization. As one can see CPU reduction is over 50%
Stats for TopNPartial in 2) are actually stats for TaskOutputOperator (that's how LocalExecutionPlanner works).
FYI: @dain
Part of: https://github.com/prestosql/presto/issues/49

Added tests


[FLINK-31466][table] Apply FilterJoinRule misses opportunity to push filter to semijoin input to FlinkFilterJoinRule

What is the purpose of the change
The main idea is to backport optimization for FlinkFilterJoiRule from the Calcite's FilterJoinRule done in https://issues.apache.org/jira/browse/CALCITE-4499
Most of the things are done in core/src/main/java/org/apache/calcite/plan/RelOptUtil.java https://github.com/apache/calcite/pull/2349/files#diff-6da50f2ef212d40bcdda85600f1cc1b2022712667fd975fccc3d446122d12facR2804
Here only change usages to the improved method
Brief change log
FlinkFilterJoinRule
plans
Verifying this change
org.apache.flink.table.planner.plan.batch.sql.join.SemiAntiJoinTestBase
org.apache.flink.table.planner.plan.batch.sql.join.BroadcastHashSemiAntiJoinTest
org.apache.flink.table.planner.plan.batch.sql.join.NestedLoopSemiAntiJoinTest
org.apache.flink.table.planner.plan.batch.sql.join.SemiAntiJoinTest
org.apache.flink.table.planner.plan.batch.sql.join.ShuffledHashSemiAntiJoinTest
org.apache.flink.table.planner.plan.batch.sql.join.SortMergeSemiAntiJoinTest
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): ( no)
The serializers: ( no)
The runtime per-record code paths (performance sensitive): ( no )
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no )
The S3 file system connector: (no )

Documentation

Does this pull request introduce a new feature? ( no)
If yes, how is the feature documented? (not applicable)

@lincoln-lil , @godfreyhe sorry for the poke,
can someone of you have a look please?

Thanks for having a look @lincoln-lil
Yes, makes sense.
I added a test to SemiAntiJoinStreamITCase


Return of error "undefined method `mimes_for_respond_to'" on devise 3.5.1 rails 4.2.3

Hi guys,
I noticed an error on Devise that was supposed to be solved by #3153 and I want to report it.
As mentioned on the title, the problem is that an error undefined method 'mimes_for_respond_to' is being raised when I try access any route.
I setup Devise properly, following the Getting Starter, setting on my ApplicationController a before_filter :authenticate_user! properly (Off course User is my model for authorization).
 I'd appreciate if you guys could see at #3686
The problem showed off because my env don't have separated gemsets, trusting just on Bundler to organize which gems will be loaded. And by coincidence, I had an elderly version of responders installed (1.0.0).
I could workaround just by doing bundler update responders to bump responders to the last version.
But I think that as we know that responders is a dependency for devise, and as devise calls a method that exists only on responders 2.0+. So that version should be explicit on devise's gemspec.
What you guys think about?


ToTestWellFormattedTests.cs:28: 60m/DEV Implement rule:...

The puzzle 5-fee780ba from #5 has to be resolved:
https://github.com/DenisZhukovski/ObjectToTest/blob/4384f5ace376f98c7be30409f039f16cc4891621/tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs#L28-L28
The puzzle was created by Denis Zhukovski on 28-Nov-22.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 5-fee780ba has disappeared from the source code, that's why I closed this issue.


[RESOLVED] High ICMP ping response time

Problem started at 02:08:14 on 2023.07.17
Problem name: High ICMP ping response time
Host: ZO-FG
Severity: Warning
Operational data: Value: 163ms
Original problem ID: 32666
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23125&eventid=32666

Problem has been resolved in 1m 0s at 02:09:14 on 2023.07.17
Problem name: High ICMP ping response time
Host: ZO-FG
Severity: Warning
Original problem ID: 32666


Georgia no longer updating

https://epiforecasts.io/covid/posts/subnational/united-states/georgia/  seems to have stopped updating.

Minor bug in an update - should now be resolved. Thanks for flagging.


Artist and title

Somehow now every artist and every title happens to be "click here"

The FA devs are super pissy that Furry Network has an auto importer, so this is probably them trying to stop people from scraping data.

But anyway! that's what we've got!
Also, furry network is slow as.... something, dunno, but too slow xD

Yeah they have been having a massive influx of users and are having trouble keeping up :p

So my hunch was right, it did have to do with the admin notice at the top of the page. Now that it's gone, everything works correctly again.

Sorry for not responding earlier, was busy dealing with the craziness of a con weekend.
A lot of what the scraper goes off of is element counts within the page.  Unfortunately this breaks when notices and other stuff is added.  If it happens again I'll update the rules to be more robust.


[SPARK-34860][ML] Multinomial Logistic Regression with intercept support centering

What changes were proposed in this pull request?
1, use new MultinomialLogisticBlockAggregator which support virtual centering
2, remove no-used BlockLogisticAggregator
Why are the changes needed?
1, for better convergence;
2, its solution is much close to GLMNET;
Does this PR introduce any user-facing change?
No
How was this patch tested?
updated and new test suites

Test build #136625 has started for PR 31985 at commit cdaafc2.

@srowen @WeichenXu123
This is the last PR for LR supporting centering

Test build #136629 has started for PR 31985 at commit 461d111.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41205/

Test build #136625 has finished for PR 31985 at commit cdaafc2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136625/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41212/

Test build #136629 has finished for PR 31985 at commit 461d111.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41205/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136629/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41205/

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41212/

@srowen Thanks for reviewing!


Add alerts smoke test

Which issue this PR addresses:

Fixes https://issues.redhat.com/browse/ARO-9826
What this PR does / why we need it:

This PR adds a test that checks if any alerts are firing.
Test plan for issue:

Run E2E_LABEL=smoke make test-e2e locally.
Is there any documentation that needs to be updated for this PR?

N/A
How do you know this will function as expected in production?

Not production feature.

/azp run ci,e2e

/azp run e2e


Define classes

New version of the code producing the neutron data.

Next step is to use the SimEx base classes...


Run tests against Debian Python in CI

We test this in SciPy main now. That CI job may also be a useful reference if someone wants to work on adding a similar job on this repo: https://github.com/scipy/scipy/blob/2cea593ce9c59b223f9f347daa26fc7e93b57584/.github/workflows/linux_meson.yml#L102-L134

Thank you!


Wire up support for selftests

Depends on CMake support added via https://github.com/clar-test/clar/pull/100.

@ethomson This should be ready to review, now that #100 has landed.

Rebased and addressed feedback.

Thanks!


Inline various utility functions for dashboard

Pull Request Description

merge hooks.useInput into Input component
inlines and removes utils.handleEvent
splits components/common.tsx into individual files

Important Notes
The goal of this PR is to minimize the number of helper functions we are using.
Switching (back?) to well-known React functions makes it clearer to new contributors what exactly is happening.
⚠ This PR is completely optional and can be closed without merging. It is a relatively minor refactor that does not add any features.
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

on second thought, it's not that easy to use toast.loading with ts-result because ts-result errors aren't JS errors - but the type safety is a good thing, so i guess i'll hold off on that for now at least


Bind myProductId to In-app Purchase Product

I am using adapty_flutter_ui to show the paywall. I am making a one-time purchase for a product whose ID I want to bind so that if the user comes across the same product, I can identify that the purchase has already been made using myProductId. Since I am using adapty_flutter_ui, I don't have control over how the purchase was made.
Future<bool> buyMyOneTimeProduct(String locale, String myProductId) async {
    await _setMetadataInProfile();
    final paywall = await sdk.getPaywall(placementId: 'Property');
    final view = await ui.createPaywallView(paywall: paywall, locale: locale);
    await view.present();
    // How do I bind success purchase to my myProductId ??
  }

I can listen to the adapty.didProfileUpdate() stream, but then I don't know which purchase is related to myProductId.

Hi @burhanrashid52! Did you pass AdaptyUIObserver to the SDK? You can read more here about events handling.


Enable noUnusedLocals & noUnusedParameters to work in JavaScript

Seems like something we should support.

This is covered in https://github.com/Microsoft/TypeScript/issues/6802


Add Holdin' On by Skrillex

Song Name
Holdin' On
Artist Name
Skrillex
Source
Youtube
Youtube Link
eotSa-qXUpbfD143
Direct File Link
No response
Song ID
649039
Start Offset [ms]
44.0
Extra
Created from Auto Nong.
Created for level: 103350344

reject
invalid yt link


fix(integration tests): slowing down integration tests for BVT

:tada: This PR is included in version 3.8.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


implement TypeUuid for primitives and fix multiple-parameter generics having the same TypeUuid

Objective

Fixes #5432
Fixes #6680

Solution

move code responsible for generating the impl TypeUuid from type_uuid_derive into a new function, gen_impl_type_uuid.
this allows the new proc macro, impl_type_uuid, to call the code for generation.
added struct TypeUuidDef and implemented syn::Parse to allow parsing of the input for the new macro.
finally, used the new macro impl_type_uuid to implement TypeUuid for the standard library (in crates/bevy_reflect/src/type_uuid_impl.rs).
fixes #6680 by doing a wrapping add of the param's index to its TYPE_UUID

Generic types should not have a single static UUID.

I'm pretty sure they don't, this macro generates the same code as the derive, where a generic type's UUID will be a composite of their own uuid and their parameter's uuid.

does anyone know why compilation sometimes fails due to the smallvec import?

Can you implement TypeUuid for arrays? Ideally [T; 1] and [T; 2] have different TYPE_UUID.

Can you implement TypeUuid for arrays? Ideally [T; 1] and [T; 2] should have different TYPE_UUID.

I think it can be implemented for [T; N], by converting N to a u128 and then to a Uuid from that, and then compositing T::TYPE_UUID, a static uuid for all arrays, and N::TYPE_UUID. Does that make sense?

On the topic of handling arrays, do we handle tuples as well? If not, that might need to be added for completeness (up to twelve fields to match Reflect and other std impls).

I've had to move the all_tuples macro outside of bevy_ecs in order to avoid a circular dependency. I've moved it into a new proc macro crate in bevy_utils called bevy_utils_macros. This seemed the most suitable place to me but lmk if there is a better place.

the name bevy_utils_macros is very confusing when we already have a crate named bevy_macro_utils. i agree that bevy_ecs is not the best place for the macro but i'm not settled on the name and perhaps this should be part of a separate pr.

i'm happy for you to revert the change to using all_tuples! for the tuple implementation. i had originally thought we already had access to it (in bevy_reflect) for implementing Reflect, but that uses a custom macro - my mistake.

bevy_macro_utils is not the correct place for the macro imo. the macro utils crate is for crates like bevy_reflect_derive to use, not for a crate like bevy_reflect. i'm not really sure where the correct place for the macro is. bevy_utils seems like a sound fit but our current naming scheme of bevy_xxx_{macros, derive} doesn't fit very well here.
@alice-i-cecile - can you provide some guidance here?

Hmm, I agree with your analysis @soqb.

bevy_macro_utils_proc

This seems fine for now. @mockersf may have better ideas: he's very good at this sort of organizational thinking.

i don't think it's correct to put all_tuples in bevy_macro_utils so perhaps bevy_utils_proc_macros?

LGTM!
considering the all_tuples! move this should have C-Breaking-Change.

Is all_tuples! public API?

https://docs.rs/bevy/latest/bevy/ecs/macro.all_tuples.html
Apparently yes, although it has no docs.

@dis-da-moe can you please rebase? This is worth including in the release.

bors r+


[indy] Fix issue #1738

Signed-off-by: juraj-durech<EMAIL_ADDRESS>Changelog

Fix Issue #1738 (join a Indy consortium which has not been created by BAF)

Reviewed by
@sownak
Linked issue
#1738

@juraj-durech this looks good. Just check why the yaml validation is failing for a specific sample.


Updates name and id field in the devfile schema

What type of PR is this?
/kind api-change
What does does this PR do / why we need it:
TBA
Which issue(s) this PR fixes:
Fixes #?
PR acceptance criteria:

[X] Unit test

[X] Integration test

[ ] Documentation

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:
TBA

/approve

/hold cancel

Just so everyone else knows, this explains the errors I've been getting:
[odo]  ✗  invalid devfile schema. errors :
[odo] - commands.0: Additional property id is not allowed
[odo] - commands.0.exec: id is required
[odo] - commands.1: Additional property id is not allowed
[odo] - commands.1.exec: id is required
[odo] - commands.2: Additional property id is not allowed
[odo] - commands.2.exec: id is required
[odo] - commands.3: Additional property id is not allowed
[odo] - commands.3.exec: id is required
[odo] - components.0: Additional property name is not allowed
[odo] - components.0.container: name is required
[odo]

/approve
Went through the code and all of this looks good to me.
Tests pass.
Lets get this merged!


The dotnet command cannot be executed

Why can't I execute my dotnet command? The SDK is installed successfully. Version 3.1 and 5.0 are the same. The error message is “This program cannot be started because ‘api-ms-win-crt-runtime|1-1-0.dll’ is missing from your computer”
The server version is Windows Server 2012 R2

The processor model is  Intel Xeon Bronze 3204

After a long time, I tried to copy the prompt DLL file to the server from the development notebook C: Windows system 32. Finally, the error message changed to “Unable to locate program input point_ invalid_ parameter_ noinfo_ Noreturn can be found in the dynamic link library (DLL) C: \program files\dotnet\dotnet.exe”

You need to install some additional dependencies on Windows Server 2012 R2:
https://docs.microsoft.com/en-us/dotnet/core/install/windows?tabs=net50#additional-deps

@LymWm Please let us know if Maira's suggestion at https://github.com/dotnet/core/issues/6272#issuecomment-842649972 resolves the problem for you. Thanks!

An exception occurred when attempting to install "https://docs.microsoft.com/en-us/dotnet/core/install/windows?tabs=net50#additional-deps", the error message was ambiguous, and the error number was: 0x80240017 The error message is: No error is specified

dd_vcredist_amd64_20210518085622.log
This is a log file

Error 0x80240017 is "WU_E_NOT_APPLICABLE - Operation was not performed because there are no applicable updates."


IsSelected changed doesn't cause child item to come into view

If you have an item that is a child, and near the end of a long list of children that is out of scroll view when the parent is expanded...
If you change the item's IsSelected to true, then the item doesn't scroll into view.
To fix this I added this line:
"item.Dispatcher.BeginInvoke((Action)(() => FocusHelper.Focus(item, true)), Threading.DispatcherPriority.Input);"
Right after:
"item.ParentTreeView.SelectedItems.Add(item.DataContext);"
In the method MultiSelectTreeViewItem.OnIsSelectedChanged
If I don't use the BeginInvoke, then the scroll into view doesn't work in the case where the parent wasn't expanded.

I'm not sure this is a bug. When you set IsSelected = true on many items, the view would wildly scroll around. If you mean to bring an item into view (not only change its state to selected – which BTW does not mean it's focused!) you can call MultiSelectTreeView.FocusItem(item, true).

Oh, good call.  I can do it that way thanks.


Preserve previous log after a hot reload

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
In most cases, we modify code to fix previous issues. If we can preserve the previous logs, it would be easy for us to compare with the previous logs. The previously printed information may also contain critical information, and if cleared, we need to repeat the process to react. If we save the logs before the hot update, it will be cumbersome.
Proposal
As the title suggests, after each hot reload, preserve the previous logs and do not clear them.

Hi @iota9star
As far as I can see, the proposal is best suited to separated IDEs for better support there.
For Intellij: https://github.com/flutter/flutter-intellij
For VSCode: https://github.com/Dart-Code/Dart-Code
I'm not sure if this can be related to Flutter tool. Please correct me if I'm missing anything from your proposal.

It should be specific to the IDE platform, I will close this issue.


Adding support for linux/ppc64le in github action for kserve-controller

What this PR does / why we need it:
This pr enables github action to generate the multi-arch docker image for kserve-controller.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #2574
Type of changes
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevent result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A

[ ] Test B

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

@adilhusain-s Thanks for the PR, can you help add more description for what's the changes about and if his changes the image we publish for KServe ?

@yuzisun
no, it doesn't change linux-amd64 docker image in anyway. We're building multi-arch docker images here.
Please read official  docs.

@yuzisun
no, it doesn't change linux-amd64 docker image in anyway. We're building multi-arch docker images here.
Please read official docs.

Thanks for the docs! Can you help create a corresponding kserve issue and link to this PR?

@yuzisun
As requested, created the issue.
issue link

@yuzisun - Any Updates on this request, I'd be interested in these changes.

@yuzisun - Any Updates on this request, I'd be interested in these changes.

I think we will also want to update for router and qpext images
https://github.com/kserve/kserve/blob/master/.github/workflows/router-docker-publish.yml
https://github.com/kserve/kserve/blob/master/.github/workflows/qpextt-docker-publish.yml

@yuzisun
I'll raise the PRS for router and qpext  component. could you please merge this one?

@yuzisun
as you requested, the PRS have been raised for qpext and router.
qpext PR
router pr

@yuzisun @ddelange
I've just accepted the suggested changes and force pushed. could you please review and merge?

@adilhusain-s can you help sign off the commit ?

/lgtm
/approve


Create A Rapyd Wallet (Backend)

Rapyd doc
To create a Rapyd Wallet, you'll use Create Wallet with the following parameters:

contact: Describes details about the wallet contacts. Array of objects. See Wallet Contact Object.
email: Email address of the wallet owner.
ewallet_reference_id: Rapyd Wallet ID defined by the customer or end user. Must be unique.
first_name: First name of the Rapyd Wallet owner.
last_name: Family name of the Rapyd Wallet owner.
metadata: A JSON object defined by the client.
phone_number: Phone number of the wallet owner in E.164 format.
type: Type of wallet: company or person. Default is person.

Example Request:
{
"first_name": "Samson",
"last_name": "Smith",
"email": "",
"ewallet_reference_id": "Samson-Smith-09232021",
"metadata": {
"merchant_defined": true
},
"phone_number": "5555555555",
"type": "person",
"contact": {
"phone_number": "+14155551311",
"email"<EMAIL_ADDRESS>"first_name": "Samson",
"last_name": "Smith",
"mothers_name": "Jane Smith",
"contact_type": "personal",
"address": {
"name": "Samson Smith",
"line_1": "123 Main Street",
"line_2": "",
"line_3": "",
"city": "Anytown",
"state": "NY",
"country": "US",
"zip": "12345",
"phone_number": "+14155551111",
"metadata": {},
"canton": "",
"district": ""
},
"identification_type": "PA",
"identification_number": "1234567890",
"date_of_birth": "11/22/2000",
"country": "US",
"nationality": "US",
"metadata": {
"merchant_defined": true
}
}
}


Using 'double tick' on reconciled cell to "create a new item for this cell and all identical cells" results in empty identifierSpace

When reporting a bug please provide the following information to help reproduce the bug:
Version of OpenRefine used (Google Refine 2.6, OpenRefine2.8, an other distribution?):
OR 2.7, OR 2.8 and build from current master
Operating Systems and version:
MacOS High Sierra
Browser + version used - Please note that OpenRefine doesn't support Internet Explorer but works OK in most cases:
Safari 11.1
Steps followed to create the issue:
Try to reconcile a column, when a set of candidates have been found for a cell, click the double tick mark next to the option to "Create new topic"
Current Results:
The cell.recon.identifierSpace value (which is set by the reconciliation process) becomes unset or empty.
It is then not possible to (e.g.) use that cell to create a new item in Wikidata
See before/after screenshot with a custom text fact showing the cell.recon.identifierSpace

Expected Results:
The identifierSpace should stay as it is set
This problem only occurs when using the Double Tick to create new items. If the single tick is used, or if double or single ticks are used to match to a proposed item from the reconciliation service the identifierSpace stays set as expected

This problem also occurs when choosing Reconcile->Actions->Create one new item for similar cells

This problem also occurs when using the option to:
"Search for match"
In the pop-up Selecting "Match other cells with same content"
Clicking 'New item' button

I think this is essentially any time "recon-judge-similar-cells" is called with parameter shareNewTopics: true

Okay so there was probably a problem with my #1568, let me investigate.

@ostephens I cannot reproduce this. Based on your first screenshot above, it looks like you have not used the Wikidata reconciliation service, but another one (maybe VIAF?). So that would explain why you cannot create new Wikidata items based on these reconciled cells.

Hmm - I think I may have tested on a version without your #1568 fix in place - at least I've double checked I'm working from the most up-to-date version of Master and now I'm not seeing the issue.
Sorry


Formatting of null conditional operator

Environment data
dotnet --info output:
.NET Command Line Tools (1.0.4)

Product Information:
 Version:            1.0.4
 Commit SHA-1 hash:  af1e6684fd

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.12
 OS Platform: Darwin
 RID:         osx.10.12-x64
 Base Path:   /usr/local/share/dotnet/sdk/1.0.4

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.0
  Build    : e8b8861ac7faf042c87a5c2f9f2d04c98b69f28d

VS Code version: 1.18.1
C# Extension version: 1.13.1
Steps to reproduce

Set vscode setting "csharp.format.enable": false
Format a C# file

Expected  behavior
Formatted like so
var thing = oldThing.FirstOrDefault()?.property
Actual behavior
var thing = oldThing.FirstOrDefault() ? .property

@akshita31 Here's another formatting bug for you.

The setting requires you to restart vscode.Did you do that ?

@akshita31 I thought that I had, but that might have been the issue because I no longer see it. I will close this issue for now unless I am able to reproduce it.
Thanks!

@bsell93 By the way, you can just restart the "omnisharp" component of the VS Code for C# extension, which is what powers formatting. You can do this by running the "restart omnisharp" command from the ctrl-shift-p menu.

(That way you don't have to fully restart vs code)

It turns out these issue does exist, but only when the null conditional operator is after a method such as FirstOrDefault().

Also, the setting requires to restart vscode. Just restarting omnisharp, still shows the formatting options on right-click.But is it the expected behavior ? @rchande

@bsell93 Did you try updating your vscode version and checking whether the issue still persists ?

@akshita31 I found out the problem. The C# FixFormat extension was causing the problem.
https://marketplace.visualstudio.com/items?itemName=Leopotam.csharpfixformat


vi insert mode ctrl w/d not working

Platform NixOS
Terminal software kitty with nushell 0.63.0
Describe the problem you are observing.
crtl w doesn't delete a word like in emacs mode and ctrl d doesn't exit
i think some of the emacs binds should be moved to common keybinds https://github.com/nushell/reedline/blob/main/src/edit_mode/emacs.rs
Steps to reproduce

set edit mode to vi
ctrl w or d in insert mode

sidenote: i currently use zsh with these 2 sourced
https://github.com/softmoth/zsh-vim-mode
https://github.com/ohmyzsh/ohmyzsh/blob/master/lib/key-bindings.zsh

https://github.com/nushell/reedline/pull/432


Parser fails for ShortString/LongString values containing invalid UTF-8

This seems to be emitted by RabbitMQ's AMQP 1.0 plugin for one.
The RabbitMQ GUI handles this by showing Base64-encoded encodings instead for these properties.

I suppose the "proper" fix here would be to switch to returning Vec<u8> instead, and letting the user decode as needed. For the 6.0.x series, a workaround could be to switch to using lossy UTF-8 decoding instead.

Hi,
Thanks for the report. Which field contains invalid utf8 in this case?

<IP_ADDRESS> Strings

AMQP strings are variable length and represented by an integer length followed by zero or more octets ofdata.
AMQP defines two native string types:
Short strings, stored as an 8-bit unsigned integer length followed by zero or more octets of data. Shortstrings can carry up to 255 octets of UTF-8 data, but may not contain binary zero octets.
Long strings, stored as a 32-bit unsigned integer length followed by zero or more octets of data. Longstrings can contain any data.

We might want to make LongString Vec<u8> but non-utf8 ShortString is invalid according to the specifications

Thanks for the report. Which field contains invalid utf8 in this case?

The x-amqp-1.0-properties header created by RabbitMQ's AMQP 1.0 plugin


Using xDrip

Needs documentation: pulling in xdrip data to a standard openaps setup.

Handled by @audiefile. Thanks!


Lock the X or Y movement axis depending on the position of crates

Nevemind, this is not necessary anymore.


AP can be larger than 1

Describe the bug
I was trying to verify the validation accuracy of the model, and I got an abnormally high result. I dug a little bit deeper and realized that sometimes ap produced in calculate_map() in bounding_box_utils can be larger than one, which is artificially inflating the mAP of the model. Specifically, there's not a check to ensure that each prediction is matched to only one ground truth label, resulting in more true positives than ground truths, which results in the recall potentially being larger than 1, which results in the AP being larger than 1 in some cases.
To Reproduce
Steps to reproduce the behavior:

add a line e.g. if ap > 1:  print(ap) in the  calculate_map() function

Expected behavior
Each ground truth label should only have one corresponding true positive prediction.
Screenshots

System Info (please complete the following ## information):

OS: Ubuntu 20.04.5
Python Version: 3.8.10
PyTorch Version: 2.1.0+cu121
CUDA/cuDNN/MPS Version: CUDA 12.0
YOLO Model Version: YOLOv9-c

Hi,
Thank you for your report! I will work on addressing this issue in the coming days. In the meantime, I recommend using the JSON dataset as the source and pycocotools to calculate the mAP.
Hao-Tang Tsui

Hi!
Thanks for your reply! I used pycocotools to calculate the mAP, and realized that the results are quite a bit lower than the paper:

when using the conf and min_iou from the yolov9 official github

when using the default conf and min_iou from this repo.
This is using yolov9-c
Is there a setting that will help me reproduce the results from the YOLOv9 paper? (0.53 mAP)

Hi,
The default configuration for IoU and confidence is optimized for visualization purposes, but these values are not ideal for achieving the best mAP.
The discrepancy with the paper's results is due to the inference strategy used there, which involves sorting images by their width-to-height ratio and generating an anchor grid for each ratio. In this project, I haven't implemented that strategy, as it does not affect inference time and primarily helps in achieving a higher mAP. If you are looking for the same accuracy as in the paper, you can refer to the repository you mentioned. The strategy implementation has been added to our to-do list~
HenryTsui

Hi Henry,
Thanks for your reply! Could you point me to a more specific place in the official github where I can find their implementation of the aforementioned strategy? Thank you.

Hi,
You can find the operation at this link.
# utils/dataloaders.py Line 544
if self.rect:
    # Sort by aspect ratio
    s = self.shapes  # wh
    ar = s[:, 1] / s[:, 0]  # aspect ratio
    irect = ar.argsort()
    self.im_files = [self.im_files[i] for i in irect]
    self.label_files = [self.label_files[i] for i in irect]
    self.labels = [self.labels[i] for i in irect]
    self.segments = [self.segments[i] for i in irect]

Best regards,
Henry Tsui

Hi Henry,
Thanks for your help! If I'm not wrong this helps during training correct? Were the weights provided in this repo trained from scratch, hence the discrepancy with the official YOLOv9 version? If I want to match the official YOLOv9 version, will I have to implement this rectangular training strategy and retrain the model from scratch?


XXX_InternalExtensions makes code generation quite hard

Its not impossible for gogoprotobuf
https://github.com/gogo/protobuf/commit/6abcf94fd4c97dcb423fdafd42fe9f96ca7e421b
, but there are users that would like to have code generation, but still import golang/protobuf.
gogoprotobuf allows this using the gofast_out protoc option.
Having the extensions private makes it hard to do some things without more library functions:

marshal: EncodeInternalExtension
calculate marshaled size: SizeOfInternalExtension
string: StringFromInternalExtension
...
or even generate a GoString method that will be able to reproduce that instance of the structure: NewUnsafeXXX_InternalExtensions

cc @matloob
I don't think that we particularly want to export any more implementation details than we already do.  This new approach to extensions has eliminated an entire class of subtle race conditions, so I'm not inclined to roll it back.  Is there a deficiency in proto's public API that prohibits you from iterating over the extensions?

I don't think you need to roll back, but I would like the proto package to contain some extra functions.

Acknowledged.  I don't understand what these functions would do that can't be done with walking over the extensions using proto.ExtensionDescs and proto.GetExtensions.

Lets say for instance generated code that unmarshals the extensions as fast as possible.

This seems like premature optimization: you would only need to generate code to walk over extensions if the message declares that it has an extension range (which is not the majority of messages).  Besides, what these internal methods would do is very close to what the approach I just described would do, since the vast majority of the time would be occupied by the mutex acquisition.  I don't see a need for the proto package to export more functions that tie us to a particular implementation.
If you're worried about the code bloat, seems like gogoprotobuf should provide these common functions and then call into them from its generated code.

My majority of messages is different from your majority of messages.
While unmarshaling the mutex is unnecessary and should not be part of the execution path.
I am not too worried about code bloat, but gogoprotobuf has common functions, like AppendExtension, that is used during Unmarshaling.
https://github.com/gogo/protobuf/commit/6abcf94fd4c97dcb423fdafd42fe9f96ca7e421b#diff-a53e6beefff5a04e637d07e9223b569fR205
The problem is how would I go about unmarshaling using the golang/protobuf library and my generated code, since some users might decide to do this.
My only other option is to say gofast_out and extensions are not supported and to reference this issue.

Or maybe I have missed another way that I can still achieve this?

I don't know enough about gogoprotobuf to be able to comment on what is the optimal solution here.  I've gleaned enough from glancing at the code to realize you use extensions even for messages that don't have them, so I can see how this has more of an impact for gogoprotobuf.  In the short-term, you should be able to generate code that was compatible with the old extension mechanism and have it work: we do type-assert to use the older interface.
As for performance of the new-style functions: you are correct that unmarshaling does not acquire the mutex.  It would be best to minimize the number of calls to ExtensionDescs and GetExtensions during marshaling, because that will acquire a mutex.

I don't see how use extension unmarshaling for messages that don't have extensions.  I wasn't talking about the extensions used in gogoprotobuf, but the extensions used by the users of gogoprotobuf, like the company where I currently work.  We used extensions in our core messages, that need to serialized and deserialized a lot.  So much that I had to write and maintain gogoprotobuf, because it is a huge bottleneck.  I am worried about other users of gogoprotobuf that still want to import golang/protobuf.
I can't generate the old extension mechanism, because I wan't people that don't activate any gogoprotobuf extensions (gogoproto.proto) to have almost exactly the same code as golang/protobuf.  Also this causes a digression which is not ideal.  If this is your short term solution, what is the long term solution?
I am not clued in on how I would unmarshal without running into mutex code?  I think I might have missed how to do that?

For mutexes: if you examine the source, extensionsWritedoes not acquire the mutex, which is what is used when setting extensions.
I don't really have any advice for a solution because I do not understand what is broken here.  We made a change to our code generation that has had no significant change in our marshal/unmarshal benchmarks.  It seems like there's a hypothetical concern here that this may cause applications using gogoprotobuf to be slower: I'd like to see exactly what is becoming slower, because then this would be a good data point to see if our API is deficient in general.  Exporting new functions mean that we will need to maintain them indefinitely, so the barrier-to-entry for that kind of change is higher.
That being said, we are aware that golang/protobuf is not where it should be on performance.  We are actively investigating how to make it better, so stay tuned.

I think I should stop talking in hypotheticals and give some concrete code examples.  I'll work on that as soon as I get some time.

Yes seems I won't have time for this for a long time and I think you might have a point.
Thanks again for your efforts and time.


Add link to github in footer

There's a footer at the bottom left corner of the main screen. I think that's better to have a link to the github repository than to http://getkaiwa.com

It's better to use config variables


Auth token is invalid

I used your this npm package.
The response is valid and I do receive a bearer token but when I try to use it isn't working.
Node Version:
16.0.0
Anchor:
https://www.google.com/recaptcha/api2/anchor?ar=2&k=6Lc-QBkaAAAAAHk3cGD3GwhlLJsdIrwzwnFXXl26&co=aHR0cHM6Ly9iLmxvZy1vbi5jb206NDQz&hl=en&v=uEf7E1417z6GNSkRx7AyL8K8&size=invisible&cb=i6w981l8leye
Token response with npm package:
03AFcWeA6QnGBJNIMGZvKMk2tXcRDkW49KTf3kunSs00wm6A9pVzpvbL4ybmaFe52oq9k3AQ3S1i3CTsXnnJldChtX5VIw3L4B2WOjvGv4BcdUoZnBSgcrnaY-8KfSOrkHlcCkeuJfVFNJhhorgbhaF7vbxGaK7Acg-dHXGGrXNo9X6w4rUw3kFbdOfuF5yyvPITGp1szWtVZ7h1tTsWGrutU_RRK7Hs5YG71JCNzFADMXKk3zrdWURzhbl-tPtCMwgi2zM7wuGY_y--WPt6Hbz2pPJGAh__1_2HvZgUaMWWAh7HMN6AdPMkkPbanDYr450ejZCoM0R_FPYIVQRWE5S7suTuSGOdYPlD6NQjMHKb3YfMqVum9ik0s4qci58F56E3sjkAlccuG3-E2qpo2RjNjMcsSTadLCh5n0Jh5_XwrVmaoziH1v7ITNgl49sBttd9ZqNrS3pG4YPB88ehsHDjbCkE5_EzMNXmhVAlQ6edvYidBW5Lio76U
Token response with chrome devtools:
03AFcWeA4XCGI8q19mlOLt-dgr8bbxrwnejoyIJ45wzQ54U64M_-17X_S-oIGGhSnZcbNZ4vaP0w-B_oieHK-L83IpOYVXY85ubla2cjXNpMNasqGZ6IC7KTlB0IQ-SSH7bO8BV5wZyB1TBEk1q1yRywe1L-d_9J1v64jN5HtShaZWEuXnxA_qOicsoIs5gWBGvtkPZqmT4HREfQawwAlBkIyt2luoqwBaoKdQf0VtMepA-o4T7KETndmtNe8LWQghRShrkUUIh2U5BizvuzG_OfyQLg5vZJvYC43ldVTGgcgrHgKm-li7p8j9xeNW5eQFR6nx1NRJXlS_Wqq0f7SxuxmD6sgLTAeObyxiiG8ogor-zRenb3zBEnD8tsOBzZGO_dRUKkAgWEGwa_7UGm3RAG88rFeDBTkM24EugEDRBkkCesGvG6QbtrlOwUBSujmTLtflLrfXo-arG9WqcQtUTP5E9T7ZHMA6mEhYclfZZvtAChQ2xMW1AkaPZCgExVargFGIcs8KCAjXMnogbvi6rTK0Dr_UqbRSQqX1AtK9FYSdjXDS9k6_iCbkdx_dlVCmKLeUNzCo_70owZSNjpXBt3M-i72BOpbgg9fKxQJSYanie2Ct2cTO66k

there are might be some additional requirements of the resource you're trying to use the token with. It's not related to the package, I guess. The purpose of the package is to retrieve the token, that's it. Also, as I've mentioned in the README of the package, even if you did everything right, you might encounter some unexpected issues. Please either provide more context or close the issue.

I try to bypass a website auth.
The bearer token exported from the recaptcha.

When I use the package and try to use the recaptcha token it doesn't work.
When I use this command in chrome devtools
grecaptcha.execute("6Lc-QBkaAAAAAHk3cGD3GwhlLJsdIrwzwnFXXl26", {action: "submit_form"}).then(console.log)
The response is a token as well which does work but I am not sure why is that.


Updates

[Current coverage][cc-pull] is 25.73%

Merging [#15][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.08%

@@             master        #15   diff @@
==========================================
  Files            14         15     +1
  Lines           705        719    +14
  Methods           0          0
  Messages          0          0
  Branches         74         76     +2
==========================================
+ Hits            182        185     +3
- Misses          522        533    +11
  Partials          1          1

Powered by Codecov. Last updated by [04c12cc...b2367a4][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pypa/linehaul/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pypa/linehaul/compare/04c12ccae91177add13ee6a9c77e4a41268b2952...b2367a487dc79e971e3722f653e05ecf645940b7
[cc-pull]: https://codecov.io/gh/pypa/linehaul/pull/15?src=pr


Fix: Site title/logo overflow

What kind of changes does this PR include?

Changes to Starlight code

Description

When you have along site title and/or logo it would overflow under the searchbar. This makes it where it does a minmax based on the sidebar width or the content's intrinsic size (screenshots below)

Before:

After:

Thank you! Been meaning to fix this for ages.
I’m sorry you had to see this CSS — let alone edit it!

This was the only place --sl-title-min-width was used, so also going to go ahead and remove it from props.css where it was set.

Hey, we can't always be perfect all the time ;)


Unable to access via IP

if ng serve to start my project
I can pass http://localhost:4200/ to access
and my IP:<IP_ADDRESS>
but, I can't pass http://<IP_ADDRESS>:4200/ to access
I need to how to solve?

ng serve --host <IP_ADDRESS>


feat: added discord auth provider

Created a Discord provider based on the spotify provider.
The Discord provider is useful when you want to connect your Nuxt 3 project with your Discord Community.
Have information about your users displayed in your codebase or just simply create cool widgets or things visitors or users can see.

The auth flow requires a HTTPS redirect uri, I have it working using NGROk locally.
Discord provides a developer section where you can setup your applications and get your token information.
The user will have to setup their oauth settings at Discord like any other platform.
I found a challenge when introducing the redirect_uri as a configurable variable in the config because it needs to redirect back to 'https://your-domain.com + /auth/discord'
Edit: Improved it with the domain variable

Thank you for the PR @justserdar
You don't need to use ngrok actually, you can simply have HTTPS with Nuxt in development with npx nuxi dev --https
We should not have a variable for the domain IMO like the others Oauth providers we have.

Hey!
I removed the domain usage and went for the existing request path route as per the other oauth providers.
Thank you about the tip with --https!


Main branch failing on Ubuntu since few days

 Error:  testAttachToCamelWithPid  Time elapsed: 264.175 s  <<< FAILURE!
java.lang.AssertionError:
[The MBeanConnection has not been established.]
Expecting actual not to be null
	at com.github.cameltooling.dap.internal.CamelDebugAdapterServerTest.testAttachToCamelWithPid(CamelDebugAdapterServerTest.java:76)

first difference found:
https://github.com/camel-tooling/camel-debug-adapter/runs/4737617493?check_suite_focus=true#step:1:8
https://github.com/camel-tooling/camel-debug-adapter/runs/4805368206?check_suite_focus=true#step:1:8

sounds it is random...

no more failures in the last builds

they started to fail again this afternoon with same error

working again the day after

it is failing almost all afternoons and never at other part of the day...

it is no more occuring


[Feature Request]: Auto collapse expanded row upon clicking other expandable row in table

Requirements before reporting

[X] No duplicated issue reported
[X] I have checked the latest version if the feature exist there

Type of feature request
Functionality
What is your feature request?
Possible to automatically collapse the expanded row if user expands a different row
Right now, if user clicks on a different row both are in expanded state.

Background
As user expands many rows without collapsing already opened rows, user gets confused with too much on screen.
It makes it harder when there are so many rows in table that are in expanded state.
Idea
No response
Goals and objectives
No response
Timeline
No response
Resources
No response
Contact information
<EMAIL_ADDRESS>

PR: https://github.com/scania-digital-design-system/tegel/pull/758

Closing this issue since this feature is available in the latest versions of our packages 🙂


Zabbix 2.4 apiinfo.version requires unauthenticated call

https://www.zabbix.com/documentation/2.2/manual/api/reference/apiinfo/version
This method is available to unauthenticated users and should be called without the auth parameter in the JSON-RPC request. Starting from Zabbix 2.4 the method will return an error if the auth parameter is given.
This is pretty annoying :(.  Anyways, I'm open to working on this but wanted to get some thoughts together on the best way to approach it. I'd like to add an auth bool parameter to call.bytes and let it flex on using api.Auth[1], but this would require a slight change to all api methods that have been created. Thoughts?
[1]
func (api *API) callBytes(method string, params interface{}, auth bool) (b []byte, err error) {
        id := atomic.AddInt32(&api.id, 1)
        if auth {
                jsonobj := request{"2.0", method, params, api.Auth, id}
        } else {

                jsonobj := request{"2.0", method, params, "", id}
        }

Thanks for looking into it.
Let me think about it for few days. If I don't reply this week, please ping me.

Well, that was a long week :confounded:


[Feature] Configure views for subgraphs status data unioned from sharded databases

Description
Derek (@Data-Nexus) had the great idea to create views for the different tables from the subgraphs schema that union all of the different foreign tables available in each shard's schema. For example, we have a subgraph_deployment_vw,  subgraph_error_vw , subgraph_manifest_vw, subgraph_version_vw, and subgraph_vw. (Note: we didn't make one for subgraphs.subgraph_deployment_assigment because we found that table to be identical across all shards)
Would it be possible to automatically create views that union each foreign table from the shard_{shard name}_subgraphs schemas? That would make overview grafana dashboards for sharded deployments a lot easier to create and distribute.
Are you aware of any blockers that must be resolved before implementing this feature? If so, which? Link to any relevant GitHub issues.
If my other feature request is implemented, it would be useful to have a union view for public.ethereum_networks as well.
Some information to help us out

[ ] Tick this box if you plan on implementing this feature yourself.
[X] I have searched the issue tracker to make sure this issue is not a duplicate.

They are not needed for copy/grafting; graph-node handles that already by picking the right shard/foreign schema as needed.
The problem with these views is in how they interact with updates to the mappings, if they are ever needed: any update to them will drop the views. It'll require some more logic in this already very complicated piece of code. I'll have to think about that a bit.

Any thoughts on this one @lutter?


So...

How are things @jaytaph? :D

Yea, I think all of the developers are dead lmao

No... just barely alive :)
But moved on to other (exciting) things at the moment.


vdirsyncerStable: Init at 0.16.7

Motivation for this change
Adds a package for the latest stable release of vdirsyncer.
fixes #58563
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thank you for the review. I’ll make changes as soon as I’m able.

@gebner @matthiasbeyer Many of my comments also apply to the vdirsyncer expression. It would be nice if you could improve it.

@dotlambda Thanks for looking over it, I've made the changes to the default vdirsyncer expression.

Thank you again @dotlambda @gebner for your guidance.

Awesome! Now ... may I ask what "stable" means in the context of vdirsyncer? Because I really don't see how a "0.x.y" package can be "stable".
Or, phrased in another way: How do you want to update the new "stable" package? Only from 0.16.x -> 0.16.y?

As you can see here, the 0.17.0 releases are still considered alpha. Hence I suggest keeping the vdirsyncerStable expression until these are considered stable, then making it an alias for vdirsyncer.


[now-client] Ignore .env.* files by default

Ignore .env.* files by default.
PRODUCT-804

@AndyBitz @styfle This change silently broke some of our deployments 😞

Our deployments too. We are currently using dotenv-flow for the handling of env.

I put together a quick (and terrible) hack. I renamed the file to .env-production and create a symlink to .env.production in the build script 😨


Updated link in docs/ABOUT.md

Updated the links to the official golang website.
The given links in the ABOUT.md have changed;
golang.org has changed to go.dev and tour.golang.org has changed to tour.go.dev.
This commit fixes this issue.

Good catch!

Thanks!


Add a Domain type

As a counterpart to IPv4Address/IPv6Address. Domains also have internal structure, and there are a bunch of useful features we could add to such a type:

Providing a Collection of the domain's labels
Getting the Top Level Domain (TLD)
Checking for a particular suffix/eTLD
Decoding Internationalized Domain Names (once supported)
Detecting whether it is safe to display a decoded IDN hostname (see https://github.com/chromium/chromium/tree/main/components/url_formatter/spoof_checks)

It could also be interesting as a currency type.

https://karwa.github.io/swift-url/0.4.0/documentation/weburl/weburl/domain


dpkg rules: Not all package names are valid bazel rule names

In the dpkg rules, the bazel rule name is also the name of the package to be downloaded.  Not all package names are valid bazel rule names, e.g.
http_file(
    name = "libstdcpp6",
    sha256 = "f1509bbabd78e89c861de16931aec5988e1215649688fd4f8dfe1af875a7fbef",
    url = "http://deb.debian.org/debian/pool/main/g/gcc-4.9/libstdc++6_4.9.2-10_amd64.deb",
)

Ideally, I'd like to avoid
dpkg(
    name = "libstdc"
    package_name = "libstdc++"
    source = "@debian_jessie//file:Packages.json",
)

I'd rather support an optional name kwarg that defaults to package_name than drop the user on the floor.
We can also surface a better error message for this ourselves...