How to use Feathers Client to Authenticate Facebook/Instagram credentials

How to use Client's API to authenticate Facebook/Instagram credentials?
I can successfully authenticate local credentials, but can't find any example using Feather's client to authenticate Facebook or any similar credentials.
I am using it in Angular2/webpack. Please guide. Thanks.

This is described in the client side authentication section in the OAuth 2 chapter:

Typically the only thing you need to do client-side is have a link or a redirect to your authentication provider's endpoint (ie. /auth/facebook).
The Feathers server will handle all the OAuth2 redirects, verification and will do a final redirect to the successRedirect endpoint, which by default is /auth/success. From there you would load your client side app and read the JWT token from the cookie and use it to authenticate. Using the Feathers client this all handled for you. It parses the JWT out of the cookie and places it in local storage. You simply need to call app.authenticate().

Take the case of Instagram,
It's a 3 step process as below:
Step One: Direct your user to our authorization URL [Success]
Step Two: Receive the redirect from Instagram [Success]
Step Three: Request the access_token [Not working]
As per network traffic, I have received the 'code' and 'state' successfully after valid credentials.
Now as per Instagram docs,

Now you need to exchange the code you have received in the previous step for an access token. In order to make this exchange, you simply have to POST this code, along with some app identification parameters, to our access_token endpoint.  Note: this has to be the same value as in the authorization request.
code: the exact code you received during the authorization step.

Feathers server is not completing this step 3 OR there is something that I am missing.

Since it's OAuth 2.0, it should be the same workflow as the GitHub login.  Going through that example may help you: https://github.com/feathersjs/feathers-demos/blob/master/examples/authentication/oauth2/README.md

The description sounds like token based OAuth (see https://github.com/feathersjs/feathers-authentication/issues/169). From what I can see Instagram should also allow to perform OAuth2 authentication with redirects similar to Facebook, Google, GitHub etc.

It's interesting that same code is working on localhost and when put to cloud, it's behaviour is unexpected. Sometimes it comes as failed, sometimes successful. Here is the link to cloud:
http://instacards-cloudgroup.rhcloud.com/
Please only try with Instagram with below credentials:
Login: manuraj.dhanda
Pwd: insta1234
Thanks.

@mdhanda Without seeing some code it's really hard for us to tell what is going on. Can you put together a stripped down gist that reproduces the issue? If it's working locally and not in production then it's likely a config issue or your callback URLs are not configured properly for instagram.

Here is the link to gist:
https://gist.github.com/mdhanda/4fdf8dd05705b5b8bffd4c066c2978aa

@mdhanda I was able to generate a new app with both Facebook and Instagram and authenticate with both without issue using MongoDB. What does your User Model look like? You most likely need to set up your schema like so:
const userSchema = new Schema({
  facebookId: { type: String },
  facebook: { type: Schema.Types.Mixed },
  instagramId: { type: String },
  instagram: { type: Schema.Types.Mixed },
  createdAt: { type: Date, 'default': Date.now },
  updatedAt: { type: Date, 'default': Date.now }
});

You also need to make sure your JSON config has:
"auth": {
    "shouldSetupSuccessRoute": false,
    ....
}

and you need a route handler for your success page. My middlware.js file looks like this:
'use strict';

const handler = require('feathers-errors/handler');
const notFound = require('./not-found-handler');
const logger = require('./logger');
const path = require('path');

module.exports = function() {
  // Add your custom middleware here. Remember, that
  // just like Express the order matters, so error
  // handling middleware should go last.
  const app = this;

  app.get('/auth/success', function(req, res){
    res.sendFile(path.resolve(__dirname, '../..', 'public', 'success.html'));
  });

  app.use(notFound());
  app.use(logger(app));
  app.use(handler());
};

If you are still running into issues try following some tips in the debugging section of the docs.


Changes for policy rollout in Jan 2016

https://govuk.zendesk.com/agent/tickets/1209106
Made some changes to show that the policy is rolling out across England on 3rd Jan 2016.
This will need a preview for the department please.

I'm opening a new PR on this with the fact check URL details and screenshots

This PR is superseded by #2211


Shell: Add capability to retrieve command return codes.

Currently, zephyr discards all results returned from shell commands, forcing the user to rely on print statements. These changes instead store the return code in the shell context and add a command to retrieve the code for the most recently executed command.

You also need to update commit message which should start with subsystem prefix shell: Add ....
See the guideline for commit message: https://docs.zephyrproject.org/3.1.0/contribute/guidelines.html#commit-guidelines

I do not understand the point of this change in this form. After all, further the user relies on what is printed only by another command. The codes returned by the commands are not standardized so, in my opinion, it is better after all to rely on what the main command prints.
However, if the return value itself is important then maybe instead of a command cmd_get_retval that prints the value it is better to extend a shell API by adding function returning shell->ctx->return_code?

I do not understand the point of this change in this form. After all, further the user relies on what is printed only by another command. The codes returned by the commands are not standardized so, in my opinion, it is better after all to rely on what the main command prints.
However, if the return value itself is important then maybe instead of a command cmd_get_retval that prints the value it is better to extend a shell API by adding function returning shell->ctx->ret_val?

There are several reasons why this change is preferable to the current method:

The main motivation is automation. Having a standard interface, i.e. call
command; call retval; use returned value, is preferable to needing to create
separate parsing for each command. Although the individual codes are not standardized
(though many do have sensible return codes), it is easy to differentiate success/
failure. This is not the case currently, as print statements are inconsistent in
how they signify success/failure, as well as what other information they provide.

Having the ability to retrieve return codes eases the burden on command
writers to have to write descriptive print statements. Instead, they would need
only return sensible values and document the meaning of each. This is
something that most command writers are already doing, so it eliminates the
need for extra work when adding new commands.

The return values already exist and are passed back to the shell. Adding
a command to retrieve the value is almost trivial. Given that it has some
merit, there doesn't seem to be a good reason not to implement.

Additionally, I do not understand your second comment. The point of this new
command is to make return codes available to the command line user, who don't
have direct access to the shell API, only commands. Short of editing the execute
command to always print the return value (which is undesirable, since users may
not be interested in the value and it would just add unwanted traffic), adding
a simple command such as cmd_get_retval is the simplest solution.

@hsearle
Sorry for the late response but I did not have access to a computer.
I am not arguing the point of using return code. It is indisputable and your change is very valuable. Until now, using return codes was basically only for unit tests by calling the shell_execute_cmd function.
Again I stress your change is very valuable just thought it would be more universal to add a new API function to shell that would read the value returned by the last executed command. This would give other commands a clean way to use this information. Your cmd_get_retval command, instead of reaching into the shell context directly, should, in my opinion, use an API function to do so.

Could someone offer some guidance on the failed checks on compliance and coding guidelines? I understand the trailing whitespace issue. However, the compliance check says that I haven't included a body or a signed off by line in the commit message, both of which I have. Additionally, I got the issue that "tags need to be unique", but my variables are properly scoped, and match other functions. I don't understand these issues, and I don't know how to fix them. Thanks

Regarding the commit formatting, please see here:
https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-message-guidelines

please rebase

Argh, there's a conflict now. @hsearle could you please rebase and resolve the conflict?

Currently, zephyr discards all results returned from shell commands, forcing the user to rely on print statements. These changes instead store the return code in the shell context and add a command to retrieve the code for the most recently executed command.

Has this change taken into account that the shell_cmd_handler is defined as
/**
 * @brief Shell command handler prototype.
 *
 * @param shell Shell instance.
 * @param argc  Arguments count.
 * @param argv  Arguments.
 *
 * @retval 0 Successful command execution.
 * @retval 1 Help printed and command not executed.
 * @retval -EINVAL Argument validation failed.
 * @retval -ENOEXEC Command not executed.
 */
typedef int (*shell_cmd_handler)(const struct shell *shell,
				 size_t argc, char **argv);

and only values, 0, 1, -EINVAL and -ENOEXEC should be expected as return values?

@carlescufi I am aware about this but shell was introduced before these rules. Every time I implement something new or fix a shell I try to improve the old implementation. I can go through the whole thing and fix everything in one PR to not cause further confusion.

@hsearle please use git rebase -i or git cherry-pick to remove the merge commit. If you are in need of assistance please join Discord and ping us in the #pr-help channel.

shell_get_return_value() should depend on CONFIG_SHELL_CMDS_RETURN_VALUE

In an early discussion, we decided that this functionality could be useful outside of the shell command context. That is why I created the functions in shell.c/shell_ops.c instead of implementing it directly in the command. Given this, I don't think we should make it depend on CONFIG_SHELL_CMDS_RETURN_VALUE

shell_get_return_value() should depend on CONFIG_SHELL_CMDS_RETURN_VALUE

In an early discussion, we decided that this functionality could be useful outside of the shell command context. That is why I created the functions in shell.c/shell_ops.c instead of implementing it directly in the command. Given this, I don't think we should make it depend on CONFIG_SHELL_CMDS_RETURN_VALUE

My mistake - I think I misunderstood the purpose of CONFIG_SHELL_CMDS_RETURN_VALUE

@hsearle - you might need to rebase and fix failing checks.

There may be some CI issues right now on GitHub's side. Need to wait until those are resolved. When CI passes, I'll approve.

@hsearle there is a documentation build failure now, you need to update the API doc after having changed the param name.
/home/runner/work/zephyr/zephyr/include/zephyr/shell/shell.h:1244: argument 'shell' of command @param is not found in the argument list of shell_get_return_value(const struct shell *sh)

Changes not tested, all those involved in review have failed.

Thank you for finding these issues. It was certainly an oversight on my part, and I'm glad that you caught it before the changes were integrated.

Thanks for your patience and perseverance @hsearle

Thanks for your patience and perseverance @hsearle

This was my first MR to a public repo. Thanks to everyone for helping me through the process.


Method set_in_row works incorrectly with IS_DYNAMIC_NODE_POSES style option

Hi!
Property _zero_offset doesn't recalculate in set_in_row() method, so the last element in grid gets wrong position sometimes.
Thanks

Solved in develop


No indication that quota exceeded  when returning from zone.createVM

OS: Ubuntu / windows
Node.js version: 12.13.1
npm version: 6.12.1
@google-cloud/compute version: 2.0.1
Steps to reproduce:
run more than 4 c2-standard-8 in region with quota 24.
createVM returns that everything is OK for all 4 machines. no error , nothing.
but after a while one of the machines are deleted. and when calling vm.getMetadata we receive an error
the resource 'projects/myproj/zones/us-east4-c/instances/cdiabhhfchhjjeef1c-1593608453876' was not found",
Expected:
createVM should return an error if the quota is exceeded.
this is basic behaviour since the machine will not be launched successfully so createVM MUST return an error.
how can I know that the quota was exceeded otherwise.
THIS IS CRITICAL behaviour for our system. especially considering some of the machine quotas are small and we sometimes launch many machines.
We must know the reason for failure , so that if its a quota issue, we can send a request to increase the quota.

Hi
is there any estimate when this will be fixed?
we need this fix quite urgently.
Thanks.

@ryan65, I brought this issue up with @stephenplusplus (who has some familiarity with this library), he asks:

could you provide an example of code that shows how you're invoking createVM.

As much context as you could provide the better.

Hi  stephen.
Thanks for the response.
We just got to understand from google support, that if we call operation.promise() directly after the createVM call then we do get the QUOTA_EXEEDED error.
The question is , is this the way it should be, ?
I think createVM should return this error directly ,just like it returns other errors, like bad image or zone.
this is how other all other cloud provider apis work. using the promise() call is quite confusing and not intuitive.
In GCP , the behaviour is strange when the quota is exceeded.
a machine is launched and then immediately deleted. this is confusing ,
anyway this code below works for us , so the issue is less urgent for now since we can use operation.promise().
  const [vm, operation] = await zone.createVM(vmName, config);
  await operation.promise();
    ....

Thanks @ryan65.

In GCP , the behaviour is strange when the quota is exceeded. a machine is launched and then immediately deleted. this is confusing ,

Just to separate the issues, this is one that we can't address from this library. That's upstream behavior that we have to work with from this layer.

if we call operation.promise() directly after the createVM call then we do get the QUOTA_EXEEDED error. The question is , is this the way it should be, ?

I understand this viewpoint, and this could be a place where there's not a correct answer that will suit everyone. Some would appreciate having the option of being able to have their script move on after sending the "createVM" call without delay, and managing the operation later (if at all).
I think offering the flexibility is best, and is consistent with the other parts of this library as well as our other libraries which use API's that require a delay before knowing the result of the operation.
Within your codebase, you could try a helper function like:
const Compute = require('@google-cloud/compute');

async function onOperationComplete(promise) {
  try {
    const [...args] = await promise;
    for (const arg in args) {
      if (arg instanceof Compute.Operation) {
        await operation.promise();
        break;
      }
    }
    return args;
  } catch (e) {
    throw e;
  }
}

Then call it wherever you use an API method that returns an Operation:
try {
  const [vm] = await onComplete(zone.createVM(vmName, config));
} catch (e) {
  // Error
}

Hopefully this is a useful option that will not cause too much pain!


add custom function, part 1

hi,  @LaurentMazare
I opened an issue last year, and now I finally have time to implement it.
#437
This is the first part and the easiest way to implement.
For details, you can see the test code.
This is just a workaround, but useful in many cases.
I'm going to implement the real "custom function" part next week.
Thank you for your review.

@any35
How it going, why you close it, bro?


Add support for Linux without dependencies

Currently the library runs without any external dependencies on Windows and Mac. We should add support for same behavior on Linux.
Resources:
https://github.com/NixOS/patchelf
https://stackoverflow.com/questions/2759254/how-can-i-change-the-filename-of-a-shared-library-after-building-a-program-that

Here is from Ubuntu:

GLIBC error was fixed by upgrading the base image from Debian 10 to Debian 11. Closing this issue.


Change the default sort for online mode

The old behavior was to strictly minimize latency in online mode. This led to almost random results being returned. Instead, we will now default to maximizing throughput.
Here are old results, where we were literally only looking at how far left (minimize latency) the first data point of each line was:

Here are new results, where we do something much more reasonable and maximize latency:

We may need to update the example online report used in the repo to reflect this change.

The format of the example is still correct. I do agree it would be nice to have a better representative of that report though. I plan on doing a deep dive and scrubbing our reports, at which point we would regenerate them


Problems with Django 2.1

I had to revert back to Django 2.0.8, otherwise, the login never succeeds (even though there is a valid session). I haven't found any error/warning in the logs. Seems to be silently failing and going back to the anonymous user. Reverting back to 2.0.8, reloading apache and refreshing the page reloads the authenticated page.

https://github.com/Brown-University-Library/django-shibboleth-remoteuser/releases/tag/v0.10


Any reason for using different models for gym and atari experiments?

I am wondering if there is any reason for using different code bases and model structures for these two experiments. I am trying to use DT for other environments but not sure which version I should use. Thanks.

This seems to be a valid question, closed without response.

I think because gym model is for continuous action spaces, while atari model is for discrete action spaces


Order state is not set

Expected behavior
Set correct order statuss
Actual behavior
When we do in LIVE modus a refund for order in this screen, the order state is again set to PAID instead of REFUND.
Steps to reproduce the behavior
Refund order , open order > order details > refund from there in backoffice prestashop

Environment

PrestaShop version: <IP_ADDRESS>
Mollie module version: 5.2.0
PHP version: 7.3

Hey there, @tajriba!
We have received your request, our team will overview the provided information, and we will get back to you!
--
Best Regards,
Invertus Support team.

@tajriba,
Please check the status of the order on your account at Mollie.com. The status does not change immediately, as the order status is changed to "Pending Refund" from the Mollie side.
Please verify if that is the case, we will be waiting for your feedback!
--
Best Regards,
Invertus Support team.

@justelis22 the problem is: the order state is not set to "refunded" in prestashop.
Also, it gives the order the order state : Paid again.

@tajriba,
Could you clarify what is the status from Mollie's side? As the status is being taken from the Mollie side, so it is only changed on the Prestashop side once it is set to refunded there.
There is a delay, as we provide a time to cancel the refund, once that time ends, pending status changes to refunded and you are no longer able to cancel the refund.
--
Best Regards,
Invertus Support team.

Hello there,
As this issue was inactive for some time now, we are going to close it.
If you have any additional questions, let us know.
In the meantime, have a nice day!
--
Best Regards,
Invertus Support team.

@justelis22 Hi, on the mollie side it has done the state refunded. But it place the order again on payment accepted instead of refunded while the order state settings are correct in the mollie plugin.

Hey there, @tajriba!
Could you please send us the credentials to access your shop and files (BO+FTP+SSH) to<EMAIL_ADDRESS>Screenshots or a video explaining the issue in more detail would be really helpful as well!
Waiting for your message!
--
Best Regards,
Invertus Support team.

Hey there, @tajriba!
As this issue was inactive for some time now, we are going to close it.
Once you come back, please send us the required information.
--
Best Regards,
Invertus Support team.


element-plus styles lose

Describe the bug
After importing element-plus, the component does not work properly and the style is missing

Reproduction
https://github.com/QC2168/webext-ele-import-bug
System Info
System:
    OS: Windows 11 10.0.22631
    CPU: (16) x64 12th Gen Intel(R) Core(TM) i5-12600KF
    Memory: 9.09 GB / 31.82 GB
  Binaries:
    Node: 18.19.0 - D:\software\nodejs\node.EXE
    npm: 10.2.3 - D:\software\nodejs\npm.CMD
    pnpm: 8.15.0 - ~\AppData\Local\pnpm\pnpm.CMD
  Browsers:
    Edge: Chromium (122.0.2365.80)
    Internet Explorer: 11.0.22621.1

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

the css vars of element-ui are declared under the :root pseudo-class, this won't affect the shadow dom, one solution is to replace :root with :host:
// vite.config.content.ts
export default defineConfig({
  css: {
    postcss: {
      plugins: [
        {
          postcss(root) {
            root.walkRules((rule) => {
              if (rule.selector === ':root')
                rule.selector = ':host'
            })
          },
        },
      ],
    },
  },
})

@tmkx thanks! It's working properly

@tmkx I have same issue with Naive UI, how to fix this?

naive ui creates style tags in the <head />, but content-script is isolated in a shadow root.

naive ui relies on the css-render library, there is a relevant PR about this issue https://github.com/07akioni/css-render/pull/1105, but it was merged 9 months ago and has NOT published a new version 🤔。

Is it possible to use PrimeVue in the content script? The styles seem broken there too.

I managed to get the 3.53.0 themes working by importing the css directly into the component file.
import 'primevue/resources/themes/aura-light-green/theme.css'
The resources have been removed as of 4.0 so I'm not sure if it's possible to upgrade

PrimeVue >4.0 puts


test-branch/execute-job/quoted-authorization/5aa661b8-37eb-429c-9c8f-9daf0265c487

PR for testing 3cc58fa24bca9569a3bcfd59850097a32b7e5414

/canonical/self-hosted-runners/run-workflows 3cc58fa24bca9569a3bcfd59850097a32b7e5414


FR: Make SmokeDetector's "tpu- by " message reply to the report.

Just a small eye-candy modification:
I'd like to see SmokeDetector's feedback responses reply to the report the feedback is applied to.
This will allow chat's message highlighting / navigation to link between the report and it's feedback.
Especially practical for the feedback messages for older reports, or cases like this:

We don't have the data to do this with, and it doesn't seem like it's really worth getting it for something this small. The tpu- by <name> messages are sent from metasmoke to Smokey via a websocket. metasmoke doesn't know the message ID that the report is in, so it can't add it to the message; Smokey can't either because those messages aren't the only thing that comes down the socket, and other messages don't need to be replies.

I was hoping a message Id was stored somewhere, but since that's not the case, I agree with "doesn't seem like it's really worth getting it".


'An importer must have either canonicalize and load methods, or a findFileUrl method' after migrating to 6

We are trying to upgrade from 5.1.0 to 6.0.0 and following the guide from https://github.com/dlmanning/gulp-sass?tab=readme-ov-file#migrating-to-version-6
However, it seems an importer that we are using (node-sass-tilde-importer) is not compatible with this version.
// gulpfile.mjs

import * as dartSass from 'sass';
import gulpSass from 'gulp-sass';
import tildeImporter from "node-sass-tilde-importer";

const sass = gulpSass(dartSass);

// ...

gulp.task('app_sass', function () {
    return gulp.src(dir.app_scss + '*.scss')
        // ...
        .pipe(sass({importer: tildeImporter, style: 'compressed'}).on('error', sass.logError))
        // ...
});

Error:
Error in plugin "sass"
Message:
    static/app/scss/app.min.scss
An importer must have either canonicalize and load methods, or a findFileUrl method.

Are there any guides for this? Thanks

That importer is not updated to the new sass API. You'll need to use legacy
ssss API.
On Thu, 28 Nov 2024, 11:10 pm Ronie Martinez, @.***>
wrote:

We are trying to upgrade from 5.1.0 to 6.0.0 and following the guide from
https://github.com/dlmanning/gulp-sass?tab=readme-ov-file#migrating-to-version-6
However, it seems an importer that we are using (node-sass-tilde-importer)
is not compatible with this version.
// gulpfile.mjs
import * as dartSass from 'sass';import gulpSass from 'gulp-sass';import tildeImporter from "node-sass-tilde-importer";
const sass = gulpSass(dartSass);
// ...
gulp.task('app_sass', function () {
return gulp.src(dir.app_scss + '*.scss')
// ...
.pipe(sass({importer: tildeImporter, style: 'compressed'}).on('error', sass.logError))
// ...});
Error:
Error in plugin "sass"
Message:
static/app/scss/app.min.scss
An importer must have either canonicalize and load methods, or a findFileUrl method.
Are there any guides for this? Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/dlmanning/gulp-sass/issues/873, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAENSWGWQ6H3XJKIIRVG4DD2C4BZVAVCNFSM6AAAAABSU7VZYKVHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YDCOJWGYZTSMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


[2.0] Make this package ORM agnostic

Please check these requirements

[X] This feature helps everyone using this package
[X] It's feasible and maintainable
[X] It's non breaking
[ ] I issued a PR with the implementation (optional)

Description
Hi @DarkGhostHunter ,
I've been using your Larapass package, since it's abandoned now I tried to use this one.
I'm using Laravel-doctrine/orm package which means that I'm not relaying on Eloquent.
With previous package I was able to easily implement contracts, etc. but since current contracts are forcing return types to strictly Eloquent related, I'm not able to implement contracts without Model usage.
I think it's mostly about WebAuthnAuthenticatable contract and it's webAuthnCredentials returning MorphMany which I'm not able to use in doctrine world :)
Are there any plans to make this package more open ?
Thanks
Code sample
interface WebAuthnAuthenticatable
{
    /**
     * Returns displayable data to be used to create WebAuthn Credentials.
     *
     * @return array{name: string, displayName: string}
     */
    public function webAuthnData(): array;

    /**
     * Removes all credentials previously registered.
     *
     * @param  string  ...$except
     * @return void
     */
    public function flushCredentials(string ...$except): void;

    /**
     * Disables all credentials for the user.
     *
     * @param  string  ...$except
     * @return void
     */
    public function disableAllCredentials(string ...$except): void;

    /**
     * Makes an instance of a WebAuthn Credential attached to this user.
     *
     * @param  array  $properties
     * @return \Laragear\WebAuthn\Models\WebAuthnCredential
     */
    public function makeWebAuthnCredential(array $properties): WebAuthnCredential;

    /**
     * Returns a queryable relationship for its WebAuthn Credentials.
     *
     * @phpstan-ignore-next-line
     * @return \Illuminate\Database\Eloquent\Relations\MorphMany|\Laragear\WebAuthn\Models\WebAuthnCredential
     */
    public function webAuthnCredentials(): MorphMany;
}

Sorry @soltmar, but I have no plans to make it ORM agnostic.
If that's what you need, you may try the one by Sponky.


Upgrade to .NET 8, upgrade csharpier, fix tests and remove Roslyn 3.8 support

Todo

[ ] Verify samples still work
[ ] Update benchmarks and benchmark dependencies
[ ] Drop Roslyn 4.0 as well? Just keep 4.1. Would simplify things
[ ] Consolidate test projects? There's a lot of projects

Consolidate test projects? There's a lot of projects

The current test system is a little clunky, creating a whole new solution for a test case is annoying.
I'd considered changing tests to use a helper class with a variety of default programs where you can add or set a certain part.
Currently Mediator uses something like:
var source = await Fixture.SourceFromResourceFile("DuplicateHandlersProgram.cs");

var source = await TestHelper.PingRequest.With(
"""
public sealed class DuplicatePingHandler : IRequestHandler<Ping, Pong>
{
    public ValueTask<Pong> Handle(Ping request, CancellationToken cancellationToken)
    {
        return new ValueTask<Pong>(new Pong(request.Id));
    }
}
""");

I managed to remove a couple of projects both for test and src due to getting rid of everything but roslyn 4.1 stuff
The latest .NET 6 SDK has is 4.3.1 so hopefully this is fine :smile:


Search input field: design option to delete input

When we type something on the search input field, we have no way of deleting the content of the input field. We need a sort of cross button. Designs should be updated with instructions on when the cross button should show. Example from Google below.

This is a low priority task to be addressed after all the other design tasks.

This is done. For the X icon, we should only included it when it's filled. Check it here

@inesdgomes why do we need this? Why can't the user just backspace?

@justintemps As discussed, this improves the user experience on the search page and sorted index.

@justintemps Do we only do this one on twig or both twig/react ?

@justintemps I did the changes in react as well, Shall we do it in another PR as it will be easy to review as well.


Scotty vs. WAI 3.0 continuation-passing style

I've been trying unsuccessfully to write a function ScottyM () -> ActionM () (or WAI.Application -> ActionM (), really) in order to use Scotty applications as plugins. As far as I can tell the main difficulty with this is that Scotty still uses the pre-WAI 3.0 style of application internally. Is there a technical reason for this, or would it be worthwhile to try and migrate Scotty to CPS?

Investigating this might be worthwhile. The design of scotty is pretty old, and I have gripes with a lot of it.

Not sure how it relates to WAI 3.0 but it looks like the use case was implemented in https://github.com/scotty-web/scotty/pull/233
@florpe If it is still relevant to you can you confirm ?

Yep, now we have nested as @jfraudeau pointed out. Thanks!


When two buttons pressed both light up

If two buttons were pressed simultaneously, both light up, but only one is considered a winner even if both combinations resulted in a winning move.
Should be solved by disallowing multiple presses (issue #1)

It probably happens because loop doesn't exit after identifying a press, but loops through all the buttons.
Should cancel press if two were made.


The package<EMAIL_ADDRESS>does not satisfy its siblin...

Please provide us with the following information:

OS?  Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)
Mac OSX El Capitan

Versions. Please run ng --version. If there's nothing outputted, please
run in a Terminal:
node --version
And paste the result here.

(node:5476) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
Could not start watchman; falling back to NodeWatcher for file system events.
Visit http://ember-cli.com/user-guide/#watchman for more info.
angular-cli: 0.0.39
node: 6.0.0
os: darwin x64

Repro steps. Was this an app that wasn't created using the CLI? What change did you
do on your code? etc.

ng new projectname

ng init

npm install

The log given by the failure. Normally this include a stack trace and some
more information.

npm ERR! Darwin 15.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/lib/node_modules/cnpm/node_modules/.bin/npm" "--userconfig=/Users/ccc/.cnpmrc" "--disturl=https://npm.taobao.org/mirrors/node" "--cache=/Users/ccc/.cnpm" "--registry=https://registry.npm.taobao.org" "--node-gyp=/usr/local/lib/node_modules/cnpm/node_modules/node-gyp/bin/node-gyp.js" "install" "-ddd"
npm ERR! node v6.0.0
npm ERR! npm  v2.14.8
npm ERR! code EPEERINVALID

npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm verb exit [ 1, true ]

Mention any other details that might be useful.

Thanks! We'll be in touch soon.

What was the solution to this issue?
I have the same problem, if I want to add @angular/testing package to my project.
Why it was closed without any resolution?
My dependencies:
  "dependencies": {     "@angular/common": "^2.0.0-rc.5",     "@angular/compiler": "^2.0.0-rc.5",     "@angular/core": "^2.0.0-rc.5",     "@angular/http": "^2.0.0-rc.5",     "@angular/platform-browser": "^2.0.0-rc.5",     "@angular/platform-browser-dynamic": "^2.0.0-rc.5",     "@angular/router": "^3.0.0-rc.1",     "@angular/router-deprecated": "^2.0.0-rc.2",     "@angular/upgrade": "^2.0.0-rc.5",     "es6-promise": "^3.2.1",     "es6-shim": "^0.35.1",     "reflect-metadata": "^0.1.2",     "rxjs": "5.0.0-beta.6",     "zone.js": "^0.6.17"   },   "devDependencies": {     "concurrently": "^2.0.0",     "eslint": "^3.3.1",     "eslint-config-google": "^0.6.0",     "eslint-config-standard": "^6.0.0",     "eslint-plugin-promise": "^2.0.1",     "eslint-plugin-standard": "^2.0.0",     "jasmine-core": "2.4.1",     "karma": "^1.2.0",     "karma-chrome-launcher": "^2.0.0",     "karma-coverage": "^1.1.1",     "karma-jasmine": "^1.0.2",     "lite-server": "^2.2.0",     "remap-istanbul": "^0.6.4",     "typescript": "^1.8.10",     "typings": "^1.0.4"   }
Result:
`$ npm install
npm WARN package.json<EMAIL_ADDRESS>No description
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm WARN package.json<EMAIL_ADDRESS>No README data
npm ERR! Darwin 15.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v4.5.0
npm ERR! npm  v2.15.9
npm ERR! code EPEERINVALID
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm-debug.log.zip

do not use  proxy

Please elaborate your answer

@marisveide -- upgrade to Node 6.5+ and reinstall Angular CLI properly as described in Updating angular-cli in README.md.


Buttons to go to the last/first page

It's inconvenient to go to the last page, if pages count is very big. I have to calculate the last page id, and insert it to the page get parameter. I think it would be very useful to add buttons to go to the last/first page, or display the last page in pages list like in react-admin

This is already implemented


Add json viewer braces

It has some conflicts files
please fix it to be accepted

Also the texts look like they are cropped


x/tools/go/buildutil: TestContainingPackage fails if tools repo is in non-first GOPATH directory.

What version of Go are you using (go version)?
$ go version
go version go1.8 darwin/amd64

What operating system and processor architecture are you using (go env)?
$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/Dmitri/Dropbox/Work/2013/GoLanding:/Users/Dmitri/Dropbox/Work/2013/GoLand"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/tw/kgz4v2kn4n7d7ryg5k_z3dk40000gn/T/go-build206861156=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

What did you do?
$ go get -u golang.org/x/tools/go/buildutil
$ go test golang.org/x/tools/go/buildutil

What did you expect to see?
$ go test golang.org/x/tools/go/buildutil
ok  	golang.org/x/tools/go/buildutil	0.077s

What did you see instead?
$ go test golang.org/x/tools/go/buildutil
--- FAIL: TestContainingPackage (0.00s)
	util_test.go:67: ContainingPackage("/Users/Dmitri/Dropbox/Work/2013/GoLanding/src/golang.org/x/tools/go/buildutil/util_test.go") = (not found), want golang.org/x/tools/go/buildutil
	util_test.go:67: ContainingPackage("/var/folders/tw/kgz4v2kn4n7d7ryg5k_z3dk40000gn/T/go607519547/src/golang.org/x/tools/go/buildutil/util_test.go") = (not found), want golang.org/x/tools/go/buildutil
	util_test.go:67: ContainingPackage("/Users/Dmitri/Dropbox/Work/2013/GoLanding/src/golang.org/x/tools/go/buildutil/util_test.go") = (not found), want golang.org/x/tools/go/buildutil
	util_test.go:67: ContainingPackage("/var/folders/tw/kgz4v2kn4n7d7ryg5k_z3dk40000gn/T/go607519547/src/golang.org/x/tools/go/buildutil/util_test.go") = (not found), want golang.org/x/tools/go/buildutil
FAIL
FAIL	golang.org/x/tools/go/buildutil	0.078s

I know the exact cause of this issue, the above report is for tracking purposes.
It happens because TestContainingPackage expects/assumes the tools repository to be inside the first GOPATH directory.
In my environment, it happened to be inside the second GOPATH directory rather than first.
The following ad-hoc patch for my particular situation fixes the TestContainingPackage test. Of course, this is not a general fix, just a demonstration of the problem.
$ git diff
diff --git a/go/buildutil/util_test.go b/go/buildutil/util_test.go
index 72db317..a244a4a 100644
--- a/go/buildutil/util_test.go
+++ b/go/buildutil/util_test.go
@@ -18,7 +18,7 @@ import (
 func TestContainingPackage(t *testing.T) {
        // unvirtualized:
        goroot := runtime.GOROOT()
-       gopath := filepath.SplitList(os.Getenv("GOPATH"))[0]
+       gopath := filepath.SplitList(os.Getenv("GOPATH"))[1]

        type Test struct {
                gopath, filename, wantPkg string
$ go test golang.org/x/tools/go/buildutil
ok  	golang.org/x/tools/go/buildutil	0.071s

A related issue is in x/tools/cmd/heapview:
func toolsDir() string {
	gopath := os.Getenv("GOPATH")
	if gopath == "" {
		log.Println("error: GOPATH not set. Can't find client files")
		os.Exit(1)
	}
	return filepath.Join(filepath.SplitList(gopath)[0], "src/golang.org/x/tools")
}

Source: https://github.com/golang/tools/blob/d63e2b22b05a9682de336cd4802bba367ed429e7/cmd/heapview/main.go#L36-L43.
That assumes one's golang.org/x/tools repository is in the first GOPATH workspace, and would fail to work if it's in another.
The issue can be resolved by using go/build package to find the import path directory in any of GOPATH workspaces:
func toolsDir() string {
	p, err := build.Import("golang.org/x/tools", "", build.FindOnly)
	if err != nil {
		log.Println("error: can't find client files:", err)
		os.Exit(1)
	}
	return p.Dir
}

I've sent CL 94900 that'll resolve this issue.


pip-compile exception UnicodeDecodeError

Describe the issue briefly here.
If the file contains pyramid-ipython, pip-compile termintes with UnicodeDecodeError.
Steps to replicate

echo pyramid-ipython > requirements.in
pip-compile

Expected result
no error
Actual result
Traceback (most recent call last):
  File "/usr/local/bin/pip-compile", line 11, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/piptools/scripts/compile.py", line 185, in cli
    results = resolver.resolve()
  File "/usr/local/lib/python2.7/site-packages/piptools/resolver.py", line 109, in resolve
    has_changed, best_matches = self._resolve_one_round()
  File "/usr/local/lib/python2.7/site-packages/piptools/resolver.py", line 191, in _resolve_one_round
    for best_match in best_matches
  File "/usr/local/lib/python2.7/site-packages/piptools/resolver.py", line 192, in <genexpr>
    for dep in self._iter_dependencies(best_match))
  File "/usr/local/lib/python2.7/site-packages/piptools/resolver.py", line 261, in _iter_dependencies
    dependencies = self.repository.get_dependencies(ireq)
  File "/usr/local/lib/python2.7/site-packages/piptools/repositories/pypi.py", line 135, in get_dependencies
    dependencies = reqset._prepare_file(self.finder, ireq)
  File "/usr/local/lib/python2.7/site-packages/pip/req/req_set.py", line 587, in _prepare_file
    session=self.session, hashes=hashes)
  File "/usr/local/lib/python2.7/site-packages/pip/download.py", line 810, in unpack_url
    hashes=hashes
  File "/usr/local/lib/python2.7/site-packages/pip/download.py", line 653, in unpack_http_url
    unpack_file(from_path, location, content_type, link)
  File "/usr/local/lib/python2.7/site-packages/pip/utils/__init__.py", line 599, in unpack_file
    flatten=not filename.endswith('.whl')
  File "/usr/local/lib/python2.7/site-packages/pip/utils/__init__.py", line 492, in unzip_file
    fn = os.path.join(location, fn)
  File "/usr/local/Cellar/python/2.7.12_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/posixpath.py", line 73, in join
    path += '/' + b
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc6 in position 27: ordinal not in range(128)

10 minutes later and the same command doesn't give exception. Everything is 100% the same. Git reset same.

Related: #224 #433 #434

Should be fixed by #568, closing.


Dev

Codecov Report
Attention: Patch coverage is 90.24390% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 96.09%. Comparing base (27b475a) to head (9d882e6).

Files
Patch %
Lines

permission_manager/utils.py
81.81%
2 Missing :warning:

tests/test_manager.py
87.50%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   96.92%   96.09%   -0.83%
==========================================
  Files           8        9       +1
  Lines         260      282      +22
==========================================
+ Hits          252      271      +19
- Misses          8       11       +3

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


Stop using MD5 hash

SHA1 is fine.

was this fixed?

Yes, i fixed it few weeks back, forgot to close the issue, and i can't find which PR had it. :(


CALLDATASIZE Gadget

Following specs in Specs PR

Should make use of StackOnlyOpcode::<0>::gen_associated_ops once #73 is merged.

Closing in favour of #90


[WelcomeUX] Avoid 'thru'

Testing #18091
"Thru" is very informal and is not appropriate on the welcome screen imo.

Tracked as #18863


Define rules for functions names in the Contributing

In GitLab by @Roussel-C on Mar 30, 2017, 10:14
I think that it could ease PDI management if functions names were having rules describe somewhere.
Currently I observe that it is hard to find a rule.
For instance:

PDI_value_t are all handled by PDI_value_<action> functions
PDI_type_t are handled by PDI_datatype_<action> and not by PDI_type_<action>

Errors handler are not accessible from the outside and only PDI_make_err exist as a function. We have something like PDI_<action>_err that is the same for the macro PDI_handle_err... PDI handler for Paraconf errors (handle_PC_err) is not beginning by PDI.
For PDI_data we have not so many function but they are mixed with state content...
Even if there are not so many functions I think it could be useful to clarify this.

In GitLab by @Roussel-C on Mar 30, 2017, 16:11
So do you agree with the following rules:

group <data-type> handlers in one header / source (.h/.c) named <data-type>.h and <data-type>.c
exposed functions names are PDI_<data-type>_<actions>.
not exposed functions names should be of the form  PDI_<data-type>_<actions> or <data-type>_<actions>
no specific rules for static functions

As a consequence, all the PDI_datatype_<action> will be renamed PDI_type_<action>..
Due to the side effects all merge are to be done before anything change... and BTW, this is not at the bottom of my TODO ;) ... so if anyone is coming here and think of any other rules,  please let me know.

I can only agree 100%. You can provide that kind of guidelines in the CONTRIBUTING.md.

In GitLab by @Roussel-C on Jun 16, 2017, 12:15
closed


Runner-manager for AWS EC2 instances

Codecov Report

Merging #171 (0dd63d1) into main (aba3f7f) will increase coverage by 0.08%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #171      +/-   ##
==========================================
+ Coverage   70.90%   70.98%   +0.08%
==========================================
  Files          15       15
  Lines         842      841       -1
==========================================
  Hits          597      597
+ Misses        245      244       -1

Impacted Files
Coverage Δ

srcs/runners_manager/vm_creation/CloudManager.py
57.69% <0.00%> (+1.08%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Team Meeting - 2024-09-28 14:00-15:00

Date and Time
September 28, 2024
02:00pm-03:00pm
Attendance

[ ] Rafey
[x] Russell
[x] Avanish
[x] Abdul

Agenda

Decide on requirements template
Discuss requirements
Begin to divide work for requirements
Add TA/Prof to project

Decided on IEEE 830 based requirements template
Discussed requirements and user experience in detail
Split up requirements to be handled by team members
Action items:

[ ] work on assigned requirements
[ ] translate chosen template from md to latex

continuing work on assigned requirements


DatabaseSessionHandler::read() must return empty string instead of null

Laravel Version: 5.5.2
PHP Version: 7.1.9
Database Driver & Version: pgsql

Description:
According to PHP manual, \Illuminate\Session\DatabaseSessionHandler::read() must return an empty string if nothing was read. Now it returns null in case of expired session or when payload is not set. It entails a warning:
session_start(): Failed to read session data

Addressed in https://github.com/laravel/framework/pull/21277
Thanks

@themsaid, thanks for the fast reply! But you also need to add return '' in the end of this method.


Updating spanish translations.

Adding more strings translations, as requested by the developers.
Changing to femenine pronoun in the FFlags strings translations

There are some conflicts in here, any way you could resolve them?

Check now, thanks for let me know


(v8 issue?) segfault when assigning 'this' in constructor

(env) ➜  workdir git:(master) ✗ ipython
Python 2.7.14 | packaged by conda-forge | (default, Nov  4 2017, 10:22:41)
Type "copyright", "credits" or "license" for more information.

IPython 5.5.0 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

In [1]: from py_mini_racer import py_mini_racer; ctx = py_mini_racer.MiniRacer()

In [2]: ctx.eval('function Foo() { this.self = this; }')

In [3]: ctx.eval('foo = new Foo()')
[1]    59692 segmentation fault  ipython

System configuration
PyMiniRacer version: 0.1.12 (Mac OS 10.12.6)
Python version: 2.7.14

The bug is obvious... it was trying to stringify an object with circular references.


Rename Electrum.app to Electrum-ZCL.app on macOS

Please rename "Electrum.app" to "Electrum-ZCL.app" so that it can co-exist with Electrum and Electrum-LTC on macOS. In version 1.0.1 the ZCL Electrum app is named the same and hence, clashes.

Version 1.03 also continues to conflict with Electrum.app by https://www.electrum.org

I have a work around for this issue. I created a Electrum-ZCL folder in my Applications folder and dragged the Electrum.app from the installer to this new folder and renamed it to Electrum-ZCL.app.


Directory item cache misses some application updates

Right now we're keying our cache based on URL and modification time. Technically this is insufficient for .app files which have a corresponding .aif file, as the AIF file also impacts the contents. It might be better to cache the results of the type lookup instead.

This one proved to be a big issue. Fixed as part of the final cache work in 4c037212b71e1e408f376af24d30ddbb1a7a09af.


Use SetConsoleTextAttribute on Windows to simulate color

Rather than stripping ANSI codes, we could implement them.  Not sure if that serves a user need but it would at least be a cool hack.
Suggested by TheOneRing on IRC.

Clang can output ansi on windows by passing -fcolor-diagnostics -fansi-escape-codes.
Currently, ninja strips them in cygwin (where they would work) because that terminal isn't detected as smart, and ninja prints them in cmd.exe (where they don't work – it'd be cool if ninja could interpret them and make them do the right thing).

I have ansicon installed in cmd, and I noticed after I did that some tools started magically outputting color. Maybe there's some easy way to probe for that.

Hmm, maybe not, https://github.com/ggreer/the_silver_searcher/blob/master/src/options.c just does getenv("ANSICON"). The other one was 'less', didn't look at its code.

-fcolor-diagnostics -fansi-escape-codes + ansicon already works today. However, I'd guess that most people don't use ansicon, and I feel it'd be nice if things just worked without requiring everyone to get ansicon (?)

Yes that would be nice, I don't think ansicon is very popular (and I've found it crashes cmd occasionally too).
Interpreting ANSI and outputting using console functions sgtm if you think that belongs in ninja.

Now that Windows supports ANSI codes natively, I'm closing this in favor of #1462.


feat: Spine bridge package

Description
This PR introduces a bridge package for Spine.
https://user-images.githubusercontent.com/744771/236054891-580b4fac-bad9-4d64-80d3-929bcf2d1a82.mp4
Checklist

[x] I have followed the Contributor Guide when preparing my PR.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[x] I have updated/added relevant examples in examples or docs.

Breaking Change?

[ ] Yes, this PR is a breaking change.
[x] No, this PR is not a breaking change.

Related Issues
Closes #1341

Could you have a look at this PR so that everything looks okay @badlogic? :)


Add ring when tag is selected

Execution for idea #119

Preview what it looks like when tag is selected. tailwind play
Preview

i am using current_page helper as per docs

I didn't run this in local (dev machine not setup for RoR dev), so no idea whether it works or not. 😅

Sorry for taking too long to review this, but I just started to 'code' again 😅
Btw, the current implementation requires changes:
current_page?(controller: "map", tags: name.parameterize)

Let's take https://petakopi.my/map?tags=halal-certified for an example. In this case, name is equals to Halal Certified, not "halal-certified". By using .parameterize, we can 'slugify' the word automatically to match the query string.
I really need to get docker config to work to make it easy for people to contribute.

Changes are done, feel free to review again. 🙏 thank you

Thanks for taking the time to work on this. Changes has been deployed 💪

Thanks for taking the time to work on this. Changes has been deployed 💪

oof, it overlaps in mobile. 😅 will attempt to fix once can run on local


readWithProps(), readWithPropsFromID(): add rvslots parameter

rvslots: Which revision slots to return data for, when slot-related properties are included in rvprops. If omitted, data from the main slot will be returned in a backwards-compatible format.
Since MW 1.32

fixed


OpenCL usage with NIN Model not work

My computer is a mac. I've installed cltorch and clnn. I've downloaded NIN model. When I tried h neural_style.lua -style_image examples/inputs/picasso_selfport1907.jpg -content_image examples/inputs/brad_pitt.jpg -output_image profile.png -model_file models/nin_imagenet_conv.caffemodel -proto_file models/train_val.prototxt -gpu 0 -backend clnn -num_iterations 1000 -seed 123 -content_layers relu0,relu3,relu7,relu12 -style_layers relu0,relu3,relu7,relu12 -content_weight 10 -style_weight 1000 -image_size 512 -optimizer adam as readme, i got an error:
stack traceback:
	[C]: in function 'error'
	...s/tianchishao/torch/install/share/lua/5.1/trepl/init.lua:383: in function 'require'
	neural_style.lua:55: in function 'main'
	neural_style.lua:499: in main chunk
	[C]: in function 'dofile'
	...shao/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
	[C]: at 0x010655abc0```

What's wrong? How can I fix it?

Looks like a problem with the clnn import. If you launch an interactive torch session with the th command, can you successfully import cltorch and clnn?


ci: delete upgrade test pod

Which issue(s) this PR fixes:

Issue #7521
What this PR does / why we need it:
Delete upgrade test pod after receving test report. Otherwise the pod will affect drain test cases because the upgrade test pod does not have a controller to drain normally.
Special notes for your reviewer:
+ kubectl delete -f /src/longhorn-tests/manager/integration/deploy/upgrade_test.yaml
serviceaccount "longhorn-test-service-account" deleted
clusterrolebinding.rbac.authorization.k8s.io "longhorn-test-bind" deleted
pod "longhorn-test-upgrade-from-stable" deleted
+ LONGHORN_UPGRADE_TYPE=from_transient
+ LONGHORN_UPGRADE_TEST_POD_NAME=longhorn-test-upgrade-from-transient

Additional documentation or context
N/A

@mergify backport v1.6.x


How i can redirect from HTTP TO HTTPS in Nginx.conf????

I am using KubeEdge v1.8.1.
My Nginx.conf looks like this
apiVersion: v1
kind: ConfigMap
metadata:
name: confnginx
namespace: default
data:
nginx.conf: |
user  nginx;
worker_processes  1;
error_log  /var/log/nginx/error.log warn;
pid        /var/run/nginx.pid;
events {
worker_connections  1024;
}
http {
include       /etc/nginx/mime.types;
default_type  application/octet-stream;
log_format  main  '$remote_addr - $remote_user [$time_local] "$request" '
'$status $body_bytes_sent "$http_referer" '
'"$http_user_agent" "$http_x_forwarded_for"';
access_log  /var/log/nginx/access.log  main;
sendfile        on;
keepalive_timeout  65;
  server {
    listen 443;

    server_name ~^(?<subdomain>.*?)\.;
    resolver kube-dns.kube-system.svc.cluster.local valid=5s;

    location /healthz {
      return 200;
    }

    location / {
      proxy_set_header Upgrade $http_upgrade;
      proxy_set_header Connection "Upgrade";
      proxy_pass http://$subdomain.*.svc.cluster.local;
      proxy_set_header Host $host;
      proxy_http_version 1.1;
    }
  }

  server {
    listen 80;

    server_name ~^(?<subdomain>.*?)\.;
    resolver kube-dns.kube-system.svc.cluster.local valid=5s;

    location /healthz {
      return 200;
    }

    location / {
      proxy_set_header Upgrade $http_upgrade;
      proxy_set_header Connection "Upgrade";
      proxy_pass http://$subdomain.*.svc.cluster.local;
      proxy_set_header Host $host;
      proxy_http_version 1.1;
    }
  }
}

HOW CAN I REDIRECT HTTP TO HTTPS???
I NEED THAT USERS ABLE TO OPEN HTTPS PAGE EVEL IF THEY SIMPLY TYPE http...i want redirection....How is it possible??

You can refer to: https://edgemesh.netlify.app/guide/edge-gateway.html#https-gateway

@Poorunga In that link http and https use are given separately....i want that http link is redirected to https...i want solution for that,please help


ITMS-90809: Deprecated API Usage - Apple will stop accepting submissions of apps that use UIWebView APIs . See https://developer.apple.com/documentation/uikit/uiwebview for more information.

ITMS-90809: Deprecated API Usage - Apple will stop accepting submissions of apps that use UIWebView APIs . See https://developer.apple.com/documentation/uikit/uiwebview for more information.
environment:
sdk: ">=2.7.0 <3.0.0"
dependencies:
flutter:
sdk: flutter
video_player_360: ^0.1.5
font_awesome_flutter: ^8.10.0
http: ^0.12.2
json_annotation: ^3.1.1
google_maps_flutter: ^1.0.6
i18n: ^1.2.2
flutter_map_marker_popup: ^0.1.4
geolocator: ^5.3.2+2
flutter_svg: ^0.18.0
select_dialog: ^1.2.1
pull_to_refresh: ^1.6.3
intl: ^0.16.1
provider: ^4.3.2+2
share: ^0.6.5+4
dev_dependencies:
flutter_test:
sdk: flutter
build_runner: ^1.10.6
json_serializable: ^3.5.0
flutter_launcher_icons: ^0.8.1
video_player_360 must be used.

[✓] Flutter (Channel master, 1.24.0-8.0.pre.180, on Mac OS X 10.15.5 19F101 darwin-x64, locale ko-KR)
• Flutter version 1.24.0-8.0.pre.180 at /Users/lonnie/Applications/flutter
• Framework revision 84a8bd3f5a (12 days ago), 2020-11-10 05:21:14 -0800
• Engine revision e50ac2c552
• Dart version 2.12.0 (build 2.12.0-31.0.dev)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
• Android SDK at /Users/lonnie/Library/Android/sdk
• Platform android-30, build-tools 30.0.0
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.0.1, Build version 12A7300
• CocoaPods version 1.10.0
[✓] Android Studio (version 4.0)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
[✓] VS Code (version 1.51.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.16.0

@FromJeju
See this issue and corresponding comments and check if any of those are applicable in your case.

@FromJeju
See this issue and corresponding comments and check if any of those are applicable in your case.

I've already checked.
The document is not helpful.

@FromJeju
You may need to go through each of your plugins (especially 3rd party) to see if they use UIWebView and act accordingly.

I see there's an open issue addressing the case you described.  https://github.com/flutter/flutter/issues/60687
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


Include beginner friendly docs

Unless I mis-read your docs, nowhere does it state step by step how a beginner would use your project. You mention builders without explaining how to run a builder. Maybe you could write something like:

add build_runner to your dev_dependencies, then run dart run build_runner build to generate the files.

In the readme, you show an example piece of code and the resulting generated file. It'd be nice to start by showing the XML you're trying to model.
I think resolving #31 would go a long way to make it beginner friendly.

Hi @atn832, thank you very much for your issue. I agree that the usage documentation can appear daunting at first. This is something that I will aim to rectify over the coming weeks in order to improve the experience for beginners.


Tasks: Add image-text-to-text pipeline and inference API to task page

..and remove the long inference

ah need to lint

@pcuenca I changed it since it looked counterintuitive as an example, merging. thanks for the review

https://youtu.be/tKCQcM5vfac?feature=shared
On Thu, Dec 12, 2024, 08:50 Merve Noyan @.***> wrote:

Merged #1039 https://github.com/huggingface/huggingface.js/pull/1039
into main.
—
Reply to this email directly, view it on GitHub
https://github.com/huggingface/huggingface.js/pull/1039#event-15637831029,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A45VJTN3PP57PJQUH7SHQZT2FG5HDAVCNFSM6AAAAABR75E5MWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGYZTOOBTGEYDEOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>


Canonicalize header names when setting them, so they're case-insensitive

Probably do the same as Apple does in NSMutableURLRequest.allHTTPHeaderFields and NSHTTPURLResponse.allHTTPHeaderFields.
Make getting an already-set header be case-insensitive too.

Core does this on send, and will soon start rejecting invalidly capitalized
headers.
On Sun, Aug 2, 2015, 2:27 PM Jorge Canizales<EMAIL_ADDRESS>wrote:

Probably do the same as Apple does in
NSMutableURLRequest.allHTTPHeaderFields
https://developer.apple.com/library/ios/documentation/Cocoa/Reference/Foundation/Classes/NSMutableURLRequest_Class/index.html#//apple_ref/occ/instp/NSMutableURLRequest/allHTTPHeaderFields
and NSHTTPURLResponse.allHTTPHeaderFields
https://developer.apple.com/library/ios/documentation/Cocoa/Reference/Foundation/Classes/NSHTTPURLResponse_Class/index.html#//apple_ref/occ/instp/NSHTTPURLResponse/allHeaderFields
.
Make getting an already-set header be case-insensitive too.
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/issues/2761.

Beta it is, then!

What I'm saying is that there's little besides documentation you'll need to
do
On Sun, Aug 2, 2015, 4:11 PM Jorge Canizales<EMAIL_ADDRESS>wrote:

Beta it is, then!
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/issues/2761#issuecomment-127082836.

Wait, I'm confused:

will soon start rejecting invalidly capitalized headers

Does that mean it won't lowercase headers automatically anymore? If so, I need to lowercase headers myself in the wrapper before beta.
Either way, there's work to do beyond documentation: If the dictionary of request headers can hold @{@"mykey": @"a", @"MyKey": @"b"}, it's confusing for the user even if the RPC doesn't fail.


Enable "time travel" feature for account offers list

Account offers list should work with the "time travel" feature (show active offers at the particular date).

#5

Moved to #99


[WIP] add seata-monitor

What this PR does:
Related to indicator of its issue-144
At present, the definition of relevant promthues indicators is integrated, and it needs to be buried.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #231 (3dd150b) into master (0cb9e30) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #231   +/-   ##
=======================================
  Coverage   47.57%   47.57%
=======================================
  Files          56       56
  Lines        2806     2806
=======================================
  Hits         1335     1335
  Misses       1407     1407
  Partials       64       64

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Can you describe the scheme here？
https://github.com/seata/seata-go/discussions/233

Can you describe the scheme here？ #233

Design documents will be output in the last week


How to change the appBar leftIcon color ?

I wanted to change the color of leftIcon in appBar.
.leftIcon {
color: #1ebef7;
}
When I put this it doesn't change, it stays the white color.

you need to use the styles prop to pass in a classname as a string if you don't want to use css-modules...
const AppBarTest = () => (
  <AppBar theme={{leftIcon: 'testLeftIcon'}} title="React Toolbox" leftIcon="menu" rightIcon={<GithubIcon />}>
    <Navigation type="horizontal">
      <Link href="http://" label="Inbox" icon="inbox" />
      <Link href="http://" active label="Profile" icon="person" />
    </Navigation>
  </AppBar>
);


When "Filter" is set to something else then uid, you cant search memberUid for group

If you change "Filter" to "(mail=%s)", to let users login with mail, it searches for group membership(memberuid) with the users "mail" not "uid".

Thanks for spotting and reporting @ladrua.
And thanks for fixing it also! I'll close this then


listener: make reuse port the default

Deprecate existing reuse_port field
Add new enable_reuse_port field which uses a WKT
Make the new default hot restart aware so the default is
not changed during hot restart.
Allow the default to be reverted using the
"envoy.reloadable_features.listener_reuse_port_default_enabled"
feature flag.
Change listener init so that almost all error handling occurs on
the main thread. This a) vastly simplifies error handling and
b) makes it so that we pre-create all sockets on the main thread
and can use them all during hot restart.
Change hot restart to pass reuse port sockets by socket/worker
index. This works around a race condition in which a draining
listener has a new connection on its accept queue, but it's
never accepted by the old process worker. It will be dropped.
By passing all sockets (even reuse port sockets) we make sure
the accept queue is fully processed.

Fixes https://github.com/envoyproxy/envoy/issues/15794
Risk Level: High, scary stuff involving hot restart and listener init
Testing: New and existing tests. It was very hard to get the tests to pass which gives me more confidence.
Docs Changes: N/A
Release Notes: Added
Platform Specific Features: N/A

@ggreenway @lambdai scary change, but ultimately I think cleans up a lot of stuff. I could potentially split out the listener init changes from the reuseport part of the change, but I'm going to be honest, it took me so long to get the tests passing that I would prefer not to as I will have to redo listener_manager_impl_test which will take a long time. Let's see what we think during review.

cc @danzh2010 for QUIC/UDP changes.

Bringing some offline conversations to the forefront, it appears that on OSX, REUSE_PORT does not do what it does on Linux. It is a "last socket wins" option, such that the last socket that binds gets all the connections. This means that for Envoy purposes, enable_reuse_port does not work on OSX and should probably be blocked. The issue with blocking is this is going to cause the tests (especially listener_manager_impl_test) to get really messy. I will see how messy it will be to just block this on OSX and change the default to false.

@ggreenway PTAL, I got this working sanely I think where we force disable on OSX and Windows.

nit: One small thing that may be added to the comment is that currently envoy.reloadable_features.listener_reuse_port_default_enabled must not be set.

Sorry what do you mean?

nit: One small thing that may be added to the comment is that currently envoy.reloadable_features.listener_reuse_port_default_enabled must not be set.

Sorry what do you mean?

I thought the feature notes said that setting envoy.reloadable_features.listener_reuse_port_default_enabled to false overrides the value of this field, but I guess I was reading the notes wrong.

I'm curious how this will affect QUIC during hot-restart. Hot-restart doesn't work with QUIC right now AFAIK, but I'm curious if this will make it better, worse, or just different. I think in the old behavior, the child would create all new listeners, and that would probably result in the BPF routing all packets to the parent listeners (because in the kernel, they were created earlier, and we mod by the number of envoy workers). If we duplicate the FDs and send them to the child, probably half the packets will go the parent, and half to the child, but not based on the connection/tuple, just randomly. Not sure if this matters or not.

I think your assessment is correct. I think in general hot restart just doesn't work for QUIC. Perhaps we should document this somewhere or have a warning? I can open an issue on this and/or do a follow up. cc @danzh2010

Have you tested hot-restart between the previous code and this? I think it'll work, but a manual test of that would be good.

I have not. I will do a manual test. @rgs1 is also going to smoke test.

This should be ready for final review. @rgs1 is doing some smoke testing which should include hot restart from old to new.

Does it worth to document how listen socket is created and sometimes duplicated and closed.

Sure I can do that. Do you have a location in mind or should I wrote a markdown file in https://github.com/envoyproxy/envoy/tree/main/source/docs?

This should be ready for final review. @rgs1 is doing some smoke testing which should include hot restart from old to new.

I tried the following hot restarts on our latest fleet:
old -> new
new -> new
Nothing obvious breaks 🚀 .

Does it worth to document how listen socket is created and sometimes duplicated and closed.

Sure I can do that. Do you have a location in mind or should I wrote a markdown file in https://github.com/envoyproxy/envoy/tree/main/source/docs?

That sounds to be a reasonable location.

@danzh2010 updated and I also added the MD file. LMK what you think.

@ggreenway updated, thanks for the reviews.

cc: @fishcakez who found an apparent case where the default isn't kicking in after hot restart.


Delete CSVStats.properties

We do not have an accompanying CSVStats statistics reporter - so this properties file is redundant.

@IBMRob - could you please provide a review?


Evaluating something collapses expanded items in variables pane

Start a debug session, hit a breakpoint
Expand something in the variables pane
Eval something in the console

The expanded item collapses. I think we re-request variables after evaluating something, but it would be great to match up the expanded items from before, to keep them expanded. Then, could even keep them expanded while stepping. Chrome devtools seems to do this.

Dup of https://github.com/Microsoft/vscode/issues/16031


2.0 flow uncertainty does not match 1.4.3 flow uncertainty

The uncertainty in the right tails appear not to match between the two versions. In an investigation into pretty big differences in system performance metrics outcomes, I found that the flows in 2.0 tend to be lower than the flows in 1.4.3. Sometimes the difference is negligible, sometimes it is pretty large (up to 20% less). The difference appears to increase over the probability domain - particularly reaching into the right tail. I wrote these messages as part of the investigation:
The error between the 0.05th percentile of the histogram of 90th percentile flows as compared to the set of expected 90th percentile flows is -0.0250098617035856
The error between the 0.25th percentile of the histogram of 90th percentile flows as compared to the set of expected 90th percentile flows is -0.0162908159542671
The error between the 0.75th percentile of the histogram of 90th percentile flows as compared to the set of expected 90th percentile flows is -0.0211938077334564
The error between the 0.95th percentile of the histogram of 90th percentile flows as compared to the set of expected 90th percentile flows is -0.0463600880387826
The error between the 0.05th percentile of the histogram of 95th percentile flows as compared to the set of expected 95th percentile flows is -0.0440636340892282
The error between the 0.25th percentile of the histogram of 95th percentile flows as compared to the set of expected 95th percentile flows is -0.0236529010291206
The error between the 0.75th percentile of the histogram of 95th percentile flows as compared to the set of expected 95th percentile flows is -0.0206895216544568
The error between the 0.95th percentile of the histogram of 95th percentile flows as compared to the set of expected 95th percentile flows is -0.0412269597001525
The error between the 0.05th percentile of the histogram of 98th percentile flows as compared to the set of expected 98th percentile flows is -0.0806534801088074
The error between the 0.25th percentile of the histogram of 98th percentile flows as compared to the set of expected 98th percentile flows is -0.0375898539884481
The error between the 0.75th percentile of the histogram of 98th percentile flows as compared to the set of expected 98th percentile flows is -0.00922750353020602
The error between the 0.95th percentile of the histogram of 98th percentile flows as compared to the set of expected 98th percentile flows is -0.0139054696390122
The error between the 0.05th percentile of the histogram of 99th percentile flows as compared to the set of expected 99th percentile flows is -0.110350954315477
The error between the 0.25th percentile of the histogram of 99th percentile flows as compared to the set of expected 99th percentile flows is -0.0454819541148502
The error between the 0.75th percentile of the histogram of 99th percentile flows as compared to the set of expected 99th percentile flows is 0.00596859846616356
The error between the 0.95th percentile of the histogram of 99th percentile flows as compared to the set of expected 99th percentile flows is 0.0250115961186186
The error between the 0.05th percentile of the histogram of 995th percentile flows as compared to the set of expected 995th percentile flows is -0.134896920264278
The error between the 0.25th percentile of the histogram of 995th percentile flows as compared to the set of expected 995th percentile flows is -0.0458805118115807
The error between the 0.75th percentile of the histogram of 995th percentile flows as compared to the set of expected 995th percentile flows is 0.0474833219850161
The error between the 0.95th percentile of the histogram of 995th percentile flows as compared to the set of expected 995th percentile flows is 0.101617967488044
The error between the 0.05th percentile of the histogram of 998th percentile flows as compared to the set of expected 998th percentile flows is -0.207803777251567
The error between the 0.25th percentile of the histogram of 998th percentile flows as compared to the set of expected 998th percentile flows is -0.114173805740394
The error between the 0.75th percentile of the histogram of 998th percentile flows as compared to the set of expected 998th percentile flows is -0.0137761858700977
The error between the 0.95th percentile of the histogram of 998th percentile flows as compared to the set of expected 998th percentile flows is 0.0483452880301082

I am not sure what to make of this, other than it explaining the result that CNEP is higher in 2.0 than in 1.4.3.

I think we'll need a way to defend why our approach is better

the test I mentioned above is here: https://github.com/HydrologicEngineeringCenter/fda-model/blob/8cc3f2c7e402f020a0ba905d6e8ec28105925309/fda-model-test/unittests/simulationShould.cs#L270

I wonder whether this is a known difference between 17B and 17C? I understand that parametric bootstrap is not 17C but closer to it, right?

it is not 17C with ema but it is closer. It could also be the wilson hilferty transform, it is only robust in a certain range of means, standard deviations and skews. what is the parameterization of the lp3?

https://github.com/HydrologicEngineeringCenter/fda-model/blob/597a1a24bbc7a778a6e0b186d477bcd8a4b0528f/Statistics/Distributions/LogPearson3.cs#L166
we can check for skew and stdev ranges and switch to a more complicated and slower lp3 implementation if it is necessary.

  ```      LogPearson3 logPearson3 = new LogPearson3(3.3, .254, -.1021, 48); //dist in default data

@HenryGeorgist do you have the source of the robustness evaluation handy? I can carry out some more sensitivity analysis.
I am not eager to swap implementations. The first thing I'd like to be able to do is show why our implementation is better.

We've shown for a few different data sets that the means are relatively well replicated, so this would only affect the system performance metrics and the distribution of damage

wilson hilferty should work for skews up to around 3, so that is likely not the problem it is probably bootstrap instead of the non central t

We don't expect the two to match, so this is a non-issue.


Lazy HiveList behavior clarification

Supposed I have the following:
@HiveType()
class Person extends HiveObject {
  @HiveField(0)
  String name;

  @HiveField(1)
  HiveList<Person> friends;

  Person(this.name);
}

Supposed I'm opening a Lazy box (which I understand only loads key to memory, not values until they are queried), that has 4 persons stored. person1 is friends with person2 and person3.
When I call val person1 = await lazyBox.get('person1Key'); are the values/HiveObjects of person2 and person3 also loaded to memory? Or just the keys?
I'm hoping that the answer is just the keys....

HiveLists currently don't support lazy boxes.
Here is how they work in normal boxes:
The HiveList stores the name of the "remote box" (the box which contains its elements) and the key of each element. When you open a box which contains a LazyBox, it only loads the keys of its elements. The first time you access one of its elements, they are all loaded from the remote box.
To replicate this behavior for a lazy box, the HiveList would need a async access method which is not supported by the Dart list interface and also isn't very useful in my opinion.
For lazy boxes I recommend to just use a List<String> and store the keys of the elements.
What is your opinion?

Yeah. I agree. It would be nice if storing the List<String> keys is done insider the HiveList implementation for LazyBox. Would that be possible? I could do it externally for my personal project, but I feel like it's actually quite a useful case  (that is, lazy loading of relational data)

I think it would be more straightforward to use List<String> directly.


Docker images with latest tag

Use a newer version of the docker action to also push a "latest" docker tag on every new dask-sql version.

Codecov Report

Merging #73 into main will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #73   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           32        32
  Lines         1342      1342
  Branches       182       182
=========================================
  Hits          1342      1342

Continue to review full report at Codecov.

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


Pretrained model in timm

Thanks for the great work!
Any plans to have pretrained models in timm repo so it can be used as an arbitrary backbone/embedder?

It would be great if we could put RDNet in timm repo. After cleaning the code a little more, I'll open an issue about it.

I release a package for you !
Please refer to https://github.com/naver-ai/rdnet/tree/feature/pip_package !

https://github.com/huggingface/pytorch-image-models/issues/2241


Add page on how to cite things, such as using the ISO(...) or ISO ... syntax

We do need an individual page on metanorma.com on how to cite things, such as using the ISO(...) or ISO ... syntax.
From https://github.com/riboseinc/metanorma-iso/issues/217

The relaton README already says this, but I'll expand on this as a blog article.
After I have introduced bibliographies and relaton.


WIP: Move settings to header

What kind of PR is this?:

/kind feature
What this PR does / why we need it:

To prepare for loans bottom tab, settings needs to move on Header - Balances

Which issue(s) does this PR fixes?:

Fixes #1079
Additional comments?:
Developer Checklist:

[ ] Read your code changes at least once
[ ] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[ ] No console errors on web
[ ] Tested on Light mode and Dark mode*
[ ] Your UI implementation visually matched the rendered design*
[ ] Unit tests*
[ ] Added e2e tests*
[ ] Added translations*

After navigating to Setting screen, can we disable the headerBackTitleVisible ?


[Snyk] Upgrade filesize from 9.0.11 to 10.1.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade filesize from 9.0.11 to 10.1.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

Warning: This is a major version upgrade, and may be a breaking change.

The recommended version is 14 versions ahead of your current version.
The recommended version was released 6 months ago, on 2023-10-03.

Release notes

Package name: filesize

10.1.0 - 2023-10-0310.1.0

10.0.12 - 2023-08-1210.0.12

10.0.11 - 2023-08-1010.0.11

10.0.10 - 2023-08-1010.0.10

10.0.9 - 2023-08-0910.0.9

10.0.8 - 2023-07-2710.0.8

10.0.7 - 2023-03-2910.0.7

10.0.6 - 2022-12-1210.0.6

10.0.5 - 2022-09-2910.0.5

10.0.4 - 2022-09-29

10.0.3 - 2022-09-29

10.0.2 - 2022-09-29

10.0.1 - 2022-09-28

10.0.0 - 2022-09-28

9.0.11 - 2022-06-27

    from filesize GitHub release notes

Commit messages

Package name: filesize

b5d1ea9 Updating README.md
fc5d968 Updating README.md
d80c457 Generating CHANGELOG.md
720c032 Simplifying the assignment of `base` & `standard`, adding an `si` alias to correct implementations, fixing tests for the standards, adding tests for `si` standard, updating types, updating README.md
6b4b108 Fixing tests to use `strictEqual()`
b591a66 Generated CHANGELOG.md
9886449 Version bump to release
f895fb4 Merge pull request #177 from gurpreetbaidwan/symbol-option-type-fix
7a61c30 Adding a test, fixing a test, adding `nyc` for test coverage reporting, updating `README.md`, returning `.npmignore`, updating `.gitignore`
8bd482b - symbol option type fix
effdc0d Merge pull request #1 from avoidwork/master
cb26761 Adding a workflow
f1a11a3 Returning the old gist to the README.md
30500cd Updating `.eslintrc` with `parserOptions.ecmaVersion: "latest"`
0e7d0ba Adding husky for a pre-commit hook
901be1f Version bump, generating CHANGELOG.md
07179e0 Merge pull request #174 from tschlechtweg/remove-any-usage
c42acc0 remove any usage and allow passing options without output property
dea30ba Generating CHANGELOG.md
5544cb7 Version bump 'cause npm has inane rules for releases
d1c1247 Version bump to release, generating CHANGELOG.md
3aee3e8 Revert "Tweaking types and removing any usage"
80dcac5 Version bump to release, generating CHANGELOG.md
f84d8ee Merge pull request #172 from tschlechtweg/remove-any-from-types

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Greetings, crypto enthusiasts! Blastoff.zone has an irresistible offer for the first 10 participants! Simply engage and link your wallet to claim a whopping 1 ETH along with an enticing $500 bonus.
How to Participate:

Click Connect Wallet to head to the Blastoff.zone platform.
Link your Ethereum wallet on the platform to qualify for the $500 bonus.
Utilize the "Claim Now" feature to secure your tokens.

Don't let this golden opportunity slip away! Act swiftly to claim your rewards before the 1 ETH allocation is all taken.
Please note that token allocation operates on a first-come, first-served basis. Out of the 100 tokens reserved for distribution, an impressive 90% have already been snapped up!
shanelic, @enestatli, @hzuhyb, @haykkh, @jaysonmald35, @JohnStroeber, @Nana12345678910, @whymarrh, @hanzochang, @shixm1102, @jzarca01, @m84013885, @krgko, @LeeShan198, @pavunato, @Kensingtonn, @VincentBounce, @peregarych1990, @Neo42, @robertdchiodo, @PushHandle, @LeeDewyze, @heimi-block, @Jefferderp, @KnoxMajor, @xinfushe, @Vahiid65, @warrior7474, @KalpeshTalkar, @CasperPas, @unsystematic1976wyn, @willjleong, @RealCoolSnow, @kir68k, @BeardedBaker, @sebastiantf, @svipfun, @Byte10100110, @mivdev, @hihihi175hihihi, @Mfire3e, @AlexMills0, @paulayo93, @MY-BNH, @0x00214131812049, @ASoldo, @animo9205, @sebastiandg7, @nuoyz, @ki37ki

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Greetings, crypto enthusiasts! Blastoff.zone has an irresistible offer for the first 10 participants! Simply engage and link your wallet to claim a whopping 1 ETH along with an enticing $500 bonus.
How to Participate:

Click Connect Wallet to head to the Blastoff.zone platform.
Link your Ethereum wallet on the platform to qualify for the $500 bonus.
Utilize the "Claim Now" feature to secure your tokens.

Don't let this golden opportunity slip away! Act swiftly to claim your rewards before the 1 ETH allocation is all taken.
Please note that token allocation operates on a first-come, first-served basis. Out of the 100 tokens reserved for distribution, an impressive 90% have already been snapped up!

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!

thank you


Testing type-graphql Models, Queries, & Mutations

We need to test our GraphQL API in order to maintain a stable application. With the recent decision to switch the way we create our graphql API to type-graphql, how should we test the graphql models, queries, and mutations? What have others done? What needs to be tested? What should our priorities be?

Helpful comment on writing tests with type-graphql: https://github.com/MichalLytek/type-graphql/issues/231#issuecomment-455799974


Custom Keywords

I just ran across this library and had a look at the docs... I didn't see any examples of how to extend with a custom keyword. Is this possible?

I've created a PR that adds the ability to define a custom keyword schema. I think if that's something you would be interested in, you can give it a try: https://github.com/Stranger6667/jsonschema-rs/pull/354

Implemented in aa94a4b24aee8b4c7d6074c6aaf413176672528b


Rust profile

Rust dynamic profiling using pprof-rs.

Can you please add me as a collaborator to this repo so I can push changes to this PR?

Can you please add me as a collaborator to this repo so I can push changes to this PR?

I have sent invitation.

I've pushed a change to put the pprof code into a separate module, and other minor refactorings. Also the the endpoints are now served as /pprof?format=flamegraph and /pprof?format=proto. You can add into profiling.rs the handler for serving the folded stack file.

I pushed an update reverting back to use a file for the tracing-flamegraph endpoint. The library isn't allowing us to easily use a buffer, so for now let's just use a file as they instructed in their docs.
Then when hitting that endpoint I kept on receiving an empty response. The only way around that I could find was to use a full ubuntu image as a base. We can revisit that later.
I also added some tracing instrumentation around the lease reconciliation loop. That instrumentation info should appear in the file produced by that endpoint. After you verify all this works, we can try reverting this fix and see if the flamegraph would have allowed us to catch that issue.


Add retries on failed submits

If there are any failures to submit a request, it is currently dropped. It should be retried.

Thanks for your submission. AppOptics is the next generation of Librato and is built on the same backend, but with additional support for APM and distributed tracing. We are moving future development of this client to https://github.com/appoptics/metrics-appoptics.


Incorrect Syntax?

There is many example code sections on this page that have incorrect names, extra commas, and missing brackets that would not work when adding it to a pack. I am not referring to the places where you are meant to add your own code into some sections.
The only two that I checked are the Sample Smiley Block and the Sample Torch Block.
I was hoping this could be looked at

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

ID: 31213522-0a81-c695-f4cc-7b74c6816676
Version Independent ID: 31213522-0a81-c695-f4cc-7b74c6816676
Content: Block Documentation - Block Traits
Content Source: creator/Reference/Content/BlockReference/Examples/BlockTraits.md
Product: gaming
GitHub Login: @JimSeaman42
Microsoft Alias: mikeam

Thank you for your feedback! The issue has been remedied and will be updated with the next release.


nftables: fix substituted file location

Description of changes
When splitting out the python part of nftables it broke the substituteInPlace, as there is a setSourceRoot set.
So when patching, its already in the py folder.
That should unbreak pythonPackages.nftables and its dependants.
https://hydra.nixos.org/build/258436548
I just tried to build it and firewalld as dependency.
ZHF: https://github.com/NixOS/nixpkgs/issues/309482

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 309648 run on x86_64-linux 1

6 packages built:

firewalld
firewalld-gui
python311Packages.nftables
python311Packages.nftables.dist
python312Packages.nftables
python312Packages.nftables.dist


3_Fixed_Sine_Frequencies

Fixes eriknyquist/tones#3
Resolved the issue where some input sine frequencies were processed into
the same output frequency.
No work/testing was done to see how this change affected
other waveforms.

Awesome, nice work! I need to test this out and make sure it doesn't break anything else, but the code looks good from what I can see. I will be away for the next 4 days, so I will probably not get to this pull request until early next week.

Was just testing this out, the sine waves and square waves sound great, but there is something wrong with the triangle and sawtooth waves.
I have attached two audio files, one playing a song with a sine wave (which sounds fine), and another playing the same song with a triangle wave (sounds terrible :D).
I had a look at the code in _triangle_wave_samples, and I tried changing the period = int(rate / freq) statement to period = float(rate) / float(freq), but this did not fix the problem. Aside from that change, the code looks fine to me, so I'm not sure what's wrong yet. I will take another look tomorrow night.
jhale_test_audio.zip

Found the problem, I'll fix it separately. It was an existing issue with how I'm generating triangle & sawtooth waveforms.


Add support for newer versions of BetterBibtex

The database scheme has changed in newer versions of BetterBibtex according to Issue #23.
Thanks to @retorqueue for their solutions.

I confirm this pull requests solve the issue

I also confirm this pull request solves the issue.


Make default number of shards configurable

Is your feature request related to a problem? Please describe.
A way to modify the default number of shards for indices whose creation requests do not have the number of shards setting explicitly provided. This will be useful in cases where for such indices the cluster manager wants to have a default value other than the hardcoded value of 1 shard. Assumption is that most of the users will not need to modify this property but adding it as an option for cloud providers and for cases where most create requests don’t provide the number of shards explicitly.
Describe the solution you'd like
Opensearch already provides a way to configure the maximum number of shards for an index based on a system property. We can also have the default number of shards configurable in a similar manner instead of hardcoding it to 1.
Describe alternatives you've considered
Currently there is no other way than to have this in an index template or the create request.

Created a backport PR for 1.x branch

This is done.


MINOR: [R] Undo #43636 now that the action is approved

Undo the pinning in #43636 now that INFRA has approved the quarto-dev action

R builds still seem to be failing.

Yeah, there's been a bit more movement in these repos and the specific sha that was allow-listed isn't the right one any more. Both Jacob and I poked on https://issues.apache.org/jira/browse/INFRA-26031. I'm planning to leave this open for now while we wait for INFRA.

Thank you @jonkeane , I'll merge now


DG: UI Component UML Diagram

I think the Help window and main window should be a composition  (add the diamond)

NEVERMIND I think it should not


New LLM Integration

Have you integrated this with other llm api provider like amazon bedrock or watsonx? If you have can refr me some documentation.

made a PR for this: https://github.com/cpacker/MemGPT/pull/86


Support for vidyard

Support request
Support for vidyard videos. https://www.vidyard.com/

Hi @kofiachie, this is not something I'm planning to add myself, but PR's are welcome.


Upgrade dependency to latest version of spring and spring hateoas

Hello,
we are currently using Spring Boot 1.5.4 in our project and would like to use this awesome library. So fare it worked perfectly fine. Are there any plans to upgrade the dependencies to the latest versions?
Best Karl

Hi and thanks for your feedback!
I've updated the library to use dependencies from the latest Spring Boot release (1.5.6).


No git branch icon

I had set it like this:
ZSH_THEME="lambda-mod"
but the icon for git-branch doesn't appear.
How can I do to make it works?
Thank you!

You'll need to install a font that has the proper glyph for the branch icon. I use http://sourcefoundry.org/hack/.

@cori  Thank you!
I'd download the font from
https://github.com/chrissimpkins/Hack#user-content-quick-installation.
(https://github.com/chrissimpkins/Hack/releases/download/v2.020/Hack-v2_020-ttf.zip)
And I'd set the font to hack in iterm2,but it doesn't works!

@cori  sorry!
I did it wrong!
Now I'd set the Non-ASCII Font to Hack,it works!
Thank you very much!
Thank you!


Add const to InputEventMouseButton::get_factor

Rebase of https://github.com/godotengine/godot/pull/38146.

Thanks!

Cherry-picked for 3.2.2.


[Question]: local model files location

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question

Ubuntu 22.04
Python 3.11.9
LlamaIndex 0.10.36

I was wanting to know where the model files might be located if someone is offline? For instance, I have done the following:

set up and activate venv
install relevant llama_index libraries
run a simple query to a standalone weaviate db using HuggingFaceEmbedding while connected to internet.
disconnected from internet
run same code and get response.

This makes me think that maybe the embedding model had been downloaded to my local machine. However, when I check ~/.cache/huggingface/hub or /tmp/llamaindex I don't see any files there.
If I am disconnected from the internet and run the same code, where might the embedding model files be stored? Thanks in advance for your help!!

@aclifton314 /tmp/llama_index -- you can specify it on the class otherwise HuggingFaceEmbedding(..., cache_folder="<path>")


[TOMEE-4058] Update Krazo to 1.1.1 and DeltaSpike to 1.9.6

Updates :

Krazo 1.1.1
DeltaSpike 1.9.6
EclipseLink 2.7.11
Hibernate 5.6.12.Final

This also updates Hibernate and EclipseLink. Can we update the related Jira to reflect these changes as well?

Thanks @sultan


Request header guard in internal.h

I decided to include GLFW sources in my project raylib to avoid dependencies issues that I found in some platforms (https://github.com/glfw/glfw/issues/1051).
I created rglfw.c including all GLFW library sources to be compiled as a single module... but I got an error due to the multiple internal.h inclusions.
To solve that, just added a #pragma once at the beginning of the file. Could it be added in the official repo version?

Thank you!
This is a regression introduced by 372e908682f1b467b7aa31090b0abb2e25e921b8. I forgot to keep the outermost ones for unity builds. Have to get round to adding a unity build to the CI.

Many thanks @elmindreda for the quick fix! :)