Unhandled js exception

invariant violation:module name prefixes showld've been stripped by the native side but wasn't for rctjscexecutor

I updated react-native to v0.39, is issue still there?


chore: adds pypi badge

Too see how it looks:
https://github.com/Aiwizo/pytorch-datastream/blob/pypi-badge/README.rst

Change tag to doc :)


JSDoc type inconsistency when using typeof with noImplicitAny = false

Bug Report
🔎 Search Terms
JSDoc
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about JSDoc

⏯ Playground Link
Playground link with relevant code
💻 Code

const test = { a: 1 }

/** @type {typeof test} */
const anExampleVariableA = { a: 1 }
console.log(anExampleVariableA.b)

/** @type {{ a: number }}} */
const anExampleVariableB = { a: 1 }
console.log(anExampleVariableB.b) // Property 'b' does not exist on type 'b'.(2339)

🙁 Actual behavior
The TS2339 error don't appear when using typeof
🙂 Expected behavior
The TS2339 error should appear when using typeof

This is consistent, just with something else: anExampleVariableA has the same read behavior as test, which is the behavior requested by typeof
/** @type {typeof test} */
const anExampleVariableA = { a: 1 }
console.log(anExampleVariableA.b)
const test = { a: 1 }
console.log(test.b);

Oh ok, with this example it makes sense, thanks @RyanCavanaugh
but I think I simplified too much my example, what I was not understanding was when I dumped the types they had the same type but it's not the case with my example
declare namespace test {
    const a: number;
}
/** @type {typeof test} */
declare const anExampleVariableA: typeof test;
/** @type {{ a: number }}} */
declare const anExampleVariableB: {
    a: number;
};

So yeah here I have two types
But with
const test = { a: 1 }

/** @typedef {typeof test} A */
/** @type A */
const anExampleVariableA = { a: 1 }
console.log(anExampleVariableA.b)

/** @typedef {{ a: number }} B */

/** @type B */
const anExampleVariableB = { a: 1 }
console.log(anExampleVariableB.b) // Property 'b' does not exist on type 'b'.(2339)

Playground Link
I have
declare namespace test {
    const a: number;
}
/** @typedef {typeof test} A */
/** @type A */
declare const anExampleVariableA: A;
/** @typedef {{ a: number }} B */
/** @type B */
declare const anExampleVariableB: B;
type A = {
    a: number;
};
type B = {
    a: number;
};

So for me they had the same type, but it might only be an issue with the .d.ts output


Build system failed!!

Check why  build system of circleci failed See  the circleci label build
NB:
For each commit / push build system should still 💯 correct!! GREEN

now it is ok!!


Intent to implement deploy_dir option

Q
A

Issue Type
Feature Request

Deployer Version
6.x

Description
I'm working on an application based on microservices architecture. At the moment each services has its own Git repository and they are deploy with Deployer.
I'm going to merge all repository in a monolithic repository. For the new deployment process, I will checkout all the monorepository on the server but I want to keep a deploy for each services with their own current directories.
I thought about adding an option deploy_dir to specify which directory of the repository need to be deploy for a host.
Is a feature that could be interesting for Deployer ?

This is possible. I think to add it too in new version.

Is it planed ?
I start to look at, how I can implement this feature.

Yes, also you can find PR here with same feature #1688

I don't really want to implement the same feature.
I have a Git repository that host multiple app. Something like this:
myRepository.git
├── app1
├── app2
├── app3

When I want to deploy, all the repository is clone, but I only want to link 1 app.
So I want to implement an option to do that on the deployed server:
deployDirOnServer
├── current -> releases/1/app1
├── releases
│   ├── 1
│   │   ├── app1
│   │   ├── app2
│   │   ├── app3
├── shared

Through this configuration :
host('prod')
    ->hostname('prod')
    ->set('deploy_path', '/var/www/my-project')
    ->set('deploy_dir', 'app1')
    ->set('repository', 'git@gitscm.com:provider/project.git')

But maybe the name deploy_dir is not really explicit.

This is possible, override tasks.

Yes I could override the deploy:symlink task.
But through this issue, I want to ask if it's interesting to implement the feature natively in Deployer ?

I dont think so, as this very non common way.

OK, no problem with that.
I'm going to mutualize it in a custom recipes 👍 .
Thanks for your work on the tool and for the time you taken to discuss on this issue.


Support Grok (xAI) models

xAI now has an OpenAI-compatible API for Grok2 and upcoming Grok models. I expect these models to be serious players in this space, so it would be great to support them.

that is a great idea, I believe you can do that today with the OpenAI plugin: https://github.com/TheFireCo/genkit-plugins/tree/main/plugins/openai


SPI transmit assertion fail? (IDFGH-8291)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
ESP-IDF v4.2.2, ESP32-WROOM-32E
I have two SPI devices connected to VSPI host. Everything is working as expected but from time to time I am getting this:
assert failed: spi_device_transmit spi_master.c:865 (ret_trans == trans_desc)
Backtrace:0x40081fa2:0x3ffb33b00x40092b55:0x3ffb33d0 0x4009941d:0x3ffb33f0 0x4010d281:0x3ffb3510 0x400e03a9:0x3ffb3540 0x400e0474:0x3ffb3590
0x40081fa2: panic_abort at /home/me/esp/esp-idf-v4.4.2/components/esp_system/panic.c:402
0x40092b55: esp_system_abort at /home/me/esp/esp-idf-v4.4.2/components/esp_system/esp_system.c:128
0x4009941d: __assert_func at /home/me/esp/esp-idf-v4.4.2/components/newlib/assert.c:85
0x4010d281: spi_device_transmit at /home/me/esp/esp-idf-v4.4.2/components/driver/spi_master.c:865 (discriminator 1)
0x400e03a9: AMIS30543SPI::enc_transfer16(unsigned short) at /home/me/esp/esp32/my_project/build/../main/myg_driver_AMIS30543.h:159
    uint16_t enc_transfer16(uint16_t data) {
            spi_transaction_t t;
            esp_err_t ret;

            memset(&t, 0, sizeof(t));                       // Zero out the transaction
            t.length = 2 * 8;                               // Len is in bytes, transaction length is in bits.
            t.tx_data[0] = data >> 8;                       // Data
            t.tx_data[1] = data & 0xff;                     // Data
            t.flags = SPI_TRANS_USE_RXDATA | SPI_TRANS_USE_TXDATA;
            ret = spi_device_transmit(spi_enc, &t);         // Transmit!
            ESP_ERROR_CHECK(ret);

            ets_delay_us(3);

            return (((t.rx_data[0] << 8) | t.rx_data[1]) ^ 0xffff);
    }

What could be the root for the issue?
Best regards

This.

Hi @KaeLL, Thanks for the reply!
I do not access same device from different tasks but I am initializing the SPI bus from one task which adds two slave devices and accesses the first device only, the second device is accessed from a task created from the first task.
The access to the first device is stopped as the second task is created.
So, I do not think that this is the issue.
However, I have switched to polling mode to see if there will be any issues. For now all is fine.
Best regards


Training for 2x

How train this model for 2x scale

Right now the model is hard-coded for 4x. If you want to train 2x model, simply comment line 72 in model.py.


Adding npm run qwik add qwik-react  is breaking the app right now

Qwik Version
0.9.0
Operating System (or Browser)
Windows and Chrome
Node Version (if applicable)
16+
Which component is affected?
Qwik React
Expected Behaviour
 npm run qwik add qwik-react was working very well for me previously
Actual Behaviour
But now its giving the following error the moment I add this command  npm run qwik add qwik-react and everything crashes.
4:13:11 am [vite] Internal server error: module is not defined
      at /node_modules/react/index.js:2:0
      at instantiateModule (file:///C:/Users/rahul/OneDrive/Desktop/qwik-app/node_modules/vite/dist/node/chunks/dep-a713b95d.js:53390:15)

Additional Information
No response

Has this been fixed yet?

Hi @MeRahulAhire
Just gave it a quick run on 0.12.1 and on my end everything is working. Do you and/or @Starnamics wants to give it a spin as well? Tested npm run build and npm run dev as well 👍

Hi @MeRahulAhire
Just gave it a quick run on 0.12.1 and on my end everything is working. Do you and/or @Starnamics wants to give it a spin as well? Tested npm run build and npm run dev as well 👍

Sweet, thank you!
I'll try it out in a bit

I'll try and let you know

Indeed this should be fixed in latest release!


[Bug fix] Wrong reference to common.debug() in kodiops

The plugin was exiting with an error ("NameError: global name 'common' is not defined") when writing debug output for AdjustLocale.
Types of changes

[ x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] I have read the CONTRIBUTING document.

All Submissions:

[ ] Have you followed the guidelines in our Contributing document?
[x ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes to Core Features:

[x ] Have you added an explanation of what your changes do and why you'd like us to include them?
[x ] Have you successfully ran tests with your changes locally?

Well, the way it is now, calling common.debug() throws an error for me - in the GUI as well as in the logs. Calling debug() does not.
That's why I changed it to use debug() as in line 39.

Arf, I'm tired lol. I was confu between old and new line in the commit ^^
So please forgot my previous comments :)

While we're there, may I ask what was the language missing? Maybe we can also fix it...

Just had a look in the debug logs:
"DEBUG: [plugin.video.netflix (0)] AdjustLocale - missing mapping conversion for locale: en-GB
DEBUG: [plugin.video.netflix (0)] AdjustLocale - missing mapping conversion for locale: en-GB"

commit it's ok
i will add the missing mapping
en-GB = en-UnitedKingdom


please bump npm package

fix for #196 is not in the npm registry yet. All oauth-based passport packages that I've tried these past few days are spewing the same error message:
createCredentials() is deprecated, use tls.createSecureContext instead

Let's save some souls from headache.
Temporary fix is by forking parent package and update oauth dependency to:
"oauth": "git://github.com/ciaranj/node-oauth"

よろしくお願いします!

Urgh. Clunk. Head hits table.
I'm getting this too. Is there any update?

If you actually look into the code on this repo, auth2.js still uses the createCredentials method. On the 18th Decemeber, 2014 @ciaranj put the line back in that refers to the method.
In the fix #196 that @neofreko mentioned simply removed the line because it was unused. I don't know why it was put back in... I'm guessing it might have been a merge mishap.

  My life is currently a little upside down dealing with some premature twins :/ sorry, when I get a chance I'll sort it!

:+1:

bump

+1

+1

I have this error too...

+1

+1

+1

+1

+1

+1

:+1:

@ciaranj think you could bump it soon? Looks like lots of people need it bumped...

+1

+1

+1

+1

Yo - +1

+1

me to

+1

+1

+1

+1

+1

+1

+1

+1

+1 any news on this? can anyone tell me how to fix this? plus, I dont know whether it is related to this warning, but using passport-facebook the profile object comes back undefined!

+1?

+1

any timetable for this?

+1

+1

:+1:

+1 please

https://github.com/ciaranj/node-oauth/pull/244
This pull request solved for me

Would be really nice to see a fix for this merged, especially since there's a solution

This project seems abandoned.

I'm here because it's used by various Oauth-based back-ends for Passport.

'Funny' story I finally found a spare moment between feeding ill babies to push a fix (twins are hard) and npm seems knackered:

Not really sure how to proceed :/

huh. weird. It worked anyway!

+1


API integration with App Store to Show list of Apps from hosting.zaviago.com list

seem like it work now


[WIP] Implement support for P2SH payouts for proposals

Implements #4505

@UdjinM6 time to revive this for 0.18?


[#5152] fix: OB catalog UTs failed on Mac

What changes were proposed in this pull request?
TBD
Why are the changes needed?
Fix: #5152
Does this PR introduce any user-facing change?
(Please list the user-facing changes introduced by your change, including

Change in user-facing APIs.
Addition or removal of property keys.)

How was this patch tested?
(Please test your changes, and provide instructions on how to test it:

If you add a feature or fix a bug, add a test to cover your changes.
If you fix a flaky test, repeat it for many times to prove it works.)

Can you please fix the style issue @featherchen ?

@jerryshao Sure, but actually the OB catalog UTs will still fail after these changes.
I want to ask how can I get the obd error logs (where should I run obd display-trace)

@mchades would you please help on this PR?

you can find the container logs in catalogs/catalog-jdbc-oceanbase/build/testcontainers.log after running ./gradlew :catalogs:catalog-jdbc-oceanbase:test -PskipDockerTests=false on MacOS

@mchades  Unfortunately, I am still suffer from the memory insufficient issue : [ERROR] OBD-2000: (<IP_ADDRESS>) not enough memory. (Free: 4G, Buff/Cache: 833M, Need: 6G), Please reduce the memory_limitormemory_limit_percentage`
I have created #5189 to keep track of it.


Feature Request: Griveous Rune (Pick)

It isn't working

It looks like the grievous rune adjustments aren't applying.

Closed in favor of new issue https://github.com/foundryvtt/pf2e/issues/10705


Outdated whois information shown

IRRexplorer seems to have shown outdated whois information on multiple occasions. This is confusing for users.
Most recent example: https://irrexplorer.nlnog.net/prefix/<IP_ADDRESS>/22 shows in a popup:
whois -h whois.altdb.net <IP_ADDRESS>/22

route:          <IP_ADDRESS>/22
descr:          Client
origin:         AS264677
mnt-by:         MAINT-RAH20
changed<EMAIL_ADDRESS>20190916
source:         ALTDB

However, when I run the same command:
❯ whois -h whois.altdb.net <IP_ADDRESS>/22
%  No entries found for the selected source(s).

It seems there is some kind of cache that doesn't get updated properly.

Inspection shows the data is in the irrd database:
irrd=> select * from rpsl_objects where prefix = '<IP_ADDRESS>/22' and source = 'ALTDB';
                  pk                  |        rpsl_pk         | source | object_class |                                                                                parsed_data                                                                                |                        object_text                        | ip_version |  ip_first   |    ip_last    | ip_size | asn_first | asn_last |            created            |           updated            | rpki_status | prefix_length | scopefilter_status |     prefix
--------------------------------------+------------------------+--------+--------------+--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-----------------------------------------------------------+------------+-------------+---------------+---------+-----------+----------+-------------------------------+------------------------------+-------------+---------------+--------------------+----------------
 e5516053-a900-484a-a3e2-920a67ff706b | <IP_ADDRESS>/22AS264677 | ALTDB  | route        | {"descr": ["Client"], "route": "<IP_ADDRESS>/22", "mnt-by": ["MAINT-RAH20"], "origin": "AS264677", "source": "ALTDB", "changed"<EMAIL_ADDRESS>20190916"]} | route:          <IP_ADDRESS>/22                           +|          4 | <IP_ADDRESS> | <IP_ADDRESS> |    1024 |    264677 |   264677 | 2021-07-28 01:15:33.301292+00 | 2021-07-28 01:18:02.38775+00 | invalid     |            22 | in_scope           | <IP_ADDRESS>/22
                                      |                        |        |              |                                                                                | descr:          Client                                   +|            |             |               |         |           |          |                               |                              |             |               |                    |
                                      |                        |        |              |                                                                                | origin:         AS264677                                 +|            |             |               |         |           |          |                               |                              |             |               |                    |
                                      |                        |        |              |                                                                                | mnt-by:         MAINT-RAH20                              +|            |             |               |         |           |          |                               |                              |             |               |                    |
                                      |                        |        |              |                                                                                | changed<EMAIL_ADDRESS>20190916+|            |             |               |         |           |          |                               |                              |             |               |                    |
                                      |                        |        |              |                                                                                | source:         ALTDB                                    +|            |             |               |         |           |          |                               |                              |             |               |                    |
                                      |                        |        |              |                                                                                |                                                           |            |             |               |         |           |          |                               |                              |             |               |                    |
(1 row)

But a query on the local whois services shows it's aware the prefix isn't present in altdb anymore:
$ whois -h localhost -p4343 <IP_ADDRESS>/22
route:          <IP_ADDRESS>/22
descr:          LACNIC generated route for Comunicaciones Metropolitanas METROCOM, S.A.
origin:         AS266853
remarks:        LACNIC generated route for Comunicaciones Metropolitanas METROCOM, S.A.
remarks:        maxLength 24
mnt-by:         MNT-CR-CMMS-LACNIC
changed:        20220218
source:         LACNIC
remarks:        ***************************************************
remarks:        This object may have been modified
remarks:        For more information, please query whois.lacnic.net
remarks:        ***************************************************
last-modified:  2022-02-18T22:35:05Z
rpki-ov-state:  valid

route:          <IP_ADDRESS>/22
origin:         AS266853
descr:          MetroCom CR Customers
admin-c:        MRA-LACNIC
tech-c:         MRA-LACNIC
remarks:        maxLength 24
mnt-by:         MAINT-AS27884
changed<EMAIL_ADDRESS>20220211  #17:31:48Z
source:         RADB
rpki-ov-state:  valid

route:          <IP_ADDRESS>/22
descr:          RPKI ROA for <IP_ADDRESS>/22 / AS266853
remarks:        This AS266853 route object represents routing data retrieved
                from the RPKI. This route object is the result of an automated
                RPKI-to-IRR conversion process performed by IRRd.
max-length:     24
origin:         AS266853
source:         RPKI  # Trust Anchor: lacnic

I think this is related to what I just reported in #156

It was a similar issue. The problem was specifically on https://irrexplorer.net. I don't see any issues on my local setup. I cleaned up the database, the old entries are gone now.

After cleaning the old entries we needed to force reload all IRRd data (using irrd_mirror_force_reload) to ensure new data was imported properly.


rename from starfish to starfix

should update readme/code etc. to use starfix.
things like:
starfish becomes starfix
dk.xam -> dev.starfix

I've fixed the missing/bad renames and pushed to master so we can move to next.


adapt bdi-viz to new api

Looks like the error above may be related to the panel version. The latest panel version that supports Python 3.8 (which I had installed) is 1.2.3. So we either find a way to make this work with panel 1.2.3 or we might need to drop support for Python 3.8.

Yeah, I faced some issues related to the panel version (1.2.3). After changing the column in the combo box, it doesn't update correctly the section "Value Matches"  pannel==1.4.4 fixed that problem.

Another issue I found: it looks like the right-side histogram is not shown after we accept the match. For example, select BMI and bmi from the Dou and Cao datasets, and click the button to accept the match. The right-side histogram will disappear.

Another issue I found: it looks like the right-side histogram is not shown after we accept the match. For example, select BMI and bmi from the Dou and Cao datasets, and click the button to accept the match. The right-side histogram will disappear.

It is because the selection is not obtained after the heatmap gets refreshed. I need to figure out a way to solve that in the later version. But I think we can merge the current version, this is a minor bug.

Yeah, I faced some issues related to the panel version (1.2.3). After changing the column in the combo box, it doesn't update correctly the section "Value Matches", pannel==1.4.4 fixed that problem.

Regarding the panel version issue, panel 1.2.3 might have some bugs. However, my virtual environment with Python 3.8.14 and panel 1.2.3 works fine. We need to investigate this version issue further in the long run.
If the interaction part isn't working, one possible problem could be related to the browser cache. Try clearing the cache and refreshing Jupyter Lab/Notebook.


Wrong error message for missing properties in types composed of the intersection of certain multiple unions

Bug Report

🔎 Search Terms
intersection of union required property error message
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about it
I use the playground and observed the same behavior for versions 4.2 through 4.9
⏯ Playground Link

Playground link with relevant code
💻 Code

type Arg =  (
  | {
      prefLabel?: string;
      showPref: false;
    }
  | {
      prefLabel: string;
      showPref?: true;
    }
)
& (
    | { other: false; }
    | { other: true; }
    );

const fn = (thing: Arg) => {};

fn({ showPref: false }); // <<< error on this line

fn({ showPref: false, other: true }); // <<< this is ok

🙁 Actual behavior
I see an error for fn({ showPref: false });:
Argument of type '{ showPref: false; }' is not assignable to parameter of type 'Arg'.
  Type '{ showPref: false; }' is not assignable to type '{ prefLabel: string; showPref?: true | undefined; } & { other: true; }'.
    Property 'prefLabel' is missing in type '{ showPref: false; }' but required in type '{ prefLabel: string; showPref?: true | undefined; }'.(2345)
input.tsx(7, 7): 'prefLabel' is declared here.

This is wrong because actually other is missing. The following line does not produce an error
🙂 Expected behavior
I would expect the error message to refer to the missing required property, not to an irrelevant union member

In the absence of finding a discriminant (which, long story short, this union doesn't have one) we end up reporting on a somewhat arbitrarily-chosen member of the target union. The error message isn't wrong, it's just not talking about the target type you wished it was.

In the line showing the error, showPref is not required but the error message says it is. That seems pretty wrong to me.
And if my understanding of unions is correct, showPref is not in the same union as other.
Here's a variation that shows a different and still unhelpful message -- the error is about incompatible types for prefLabel but the type is fine if I add other to the object.
Regardless, if you believe the current behavior is not wrong, feel free to rephrase my issue to read "unhelpful and misleading error message" instead of "wrong."

In case it's helpful, this is a variation with (what I hope is) a discriminant in the first union:
type Arg =  (
  | {
      type: "a";
      prefLabel?: string;
      showPref: false;
    }
  | {
      type: "b";
      prefLabel: string;
      showPref?: true;
    }
)
& (
    | { other: false; }
    | { other: true; }
    );

const fn = (thing: Arg) => {};

fn({ type: "a", showPref: false });

fn({ type: "a", showPref: false, other: true });

The error is still mentioning prefLabel in this case.


@Ontheway361 did you give up changing it Tf to Torch?

        Absolutely Yes!

<PHONE_NUMBER>

<EMAIL_ADDRESS>

—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.
[
{
@.": "http://schema.org",
@.": "EmailMessage",
"potentialAction": {
@.": "ViewAction",
"target": "https://github.com/seasonSH/Probabilistic-Face-Embeddings/issues/15#issuecomment-897166059",
"url": "https://github.com/seasonSH/Probabilistic-Face-Embeddings/issues/15#issuecomment-897166059",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
@.": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]


Possibility to delete the organization for users

Is your feature request related to a problem? Please describe.
As a user, I cannot delete an organization by myself, only through SE.
Describe the solution you'd like
I would like to have an option to delete an organization when it is no longer needed without asking SE to do so.
Describe alternatives you've considered

Additional context
SEs receive this request frequently, making it possible for customers to delete their organizations can help remove the middle-man.

This was implemented in MAPI


signup and login with google, facebook, twitter, github et al

With all the online profiles I allready have, it would be nice to being able to reuse one of them to create a profile on gravity.
maybe G+ and Facebook are not the best choice as they are competitors but twitter and Github definitely are and OpenID should be an option.

Good idea! Thanks for the input!

With current useraccounts package it's fairly easy to implement:
https://github.com/meteor-useraccounts/core/blob/master/Guide.md#3rd-party-login-services-configuration
Question is - will it merge multiple social logins/signups from same user together? Well, there's package for it: https://atmospherejs.com/splendido/accounts-meld

Thanks. I would highly recommend the merging process. Otherwise users may not remember that the already have an account or have already signed in with a different provider.

Is someone working on it?

I completed the closed issue #55 that allowed users to link their social media accounts to their profile, but this is different.  So I don't think anyone is working on this one.

That was to link the accounts after signing in and adding links to the text box.
This is regarding sign up and login.
Correct me if I am seeing

Yes

If you want to work on this one, I came across these packages that you could probably use:
- https://atmospherejs.com/meteor/accounts-facebook
- https://atmospherejs.com/meteor/accounts-google

hey, Kindly create application in github, and respective facebook and google account so that meteor can use its auth key .

Great! Could you provide the keys via Slack? BTW. Do you have heroku? Then I can add you and you can deployments anytime.

Yes, I am there on heroku my ID is<EMAIL_ADDRESS>.
There's one issue here, how should I add {{loginbuttons}} in template or in router something like this

AccountsTemplates.configure({
defaultTemplate: 'Auth_page',
defaultLayout: 'App_body',
defaultContentRegion: 'main',
defaultLayoutRegions: {}
});

Have just added you as a collaborator on Heroku.

We're using the useraccounts package so simply adding the meteor related packages should do the trick.

It's not happening that way.  We need to add it to our view part by adding it in template or in routes.

Then I would add it to the template rather than the route

Oh where will I find the sign in template? We are using accounts package and just configuring the messages.


Combine two excel sheets

Hello,
I am using jquery calx to add some formulas that I have inside a table, my formula in Excel is the following:
=SI(N3="BLK";BUSCARV(M3;'RC'!J:M;4;FALSO);BUSCARV(M3;'RC'!J:M;3;FALSO))
Hello,
I am using jquery calx to add some formulas that I have inside a table, my formula is the following:
=IF(N3="BLK";VLOOKUP(M3;'RC'!J:M;4;FALSE);VLOOKUP(M3;'RC'!J:M;3;FALSE))
Which the word RC is my second excel sheet and therefore when putting it as in the examples of jquery calx it marks an #ERROR, I have the following syntax using razor syntax in c # mvc:
`
@Html.DisplayFor(modelItem => item.ACTUALaxemty)
`
My script is:
`@section scripts
{

<script type="text/javascript">
    $('#TablaEnrut').calx();
    $('#RC').calx();
</script>

}`
I would appreciate if someone had a similar error, could support me or guide me.
Thanks.

Hi @MADAI96, to make 2 sheets work together, you need to initialize it like $('#sheet_1, #sheet_2').calx(), instead of initializing it independently


[Spring] Enhance managed component log stream

Enhance managed component log stream when -i/--instance or --all-instances are not specified

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

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[ ] My extension version conforms to the Extension version schema

For new extensions:

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

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Spring


Data rates not in scope

Our custom LSM303 library is interfering with the data rate values from being seen:
client-compLaunch.ino:65:29: error: 'LSM303_ACCEL_DATA_RATE_400HZ' was not declared in this scope
client-compLaunch.ino:66:29: error: 'LSM303_MAG_DATA_RATE_220HZ' was not declared in this scope

For reference, here are lines 64-66:
    // Initialize the sensors
    accel.setDataRate((byte)LSM303_ACCEL_DATA_RATE_400HZ);
    accel.setDataRate((byte)LSM303_MAG_DATA_RATE_220HZ);

I can't reproduce this error. Our custom libraries are the ones that defined these values, and the setDataRate function. Did you install it?

I did, so I don't know why I get this error. I'll try again later.
--
Ky-Cuong Huynh
On Wed, May 13, 2015 at 7:02 PM, Vladimir Vysotsky
<EMAIL_ADDRESS>wrote:

I can't reproduce this error. Our custom libraries are the ones that defined these values, and the setDataRate function. Did you install it?
Reply to this email directly or view it on GitHub:
https://github.com/UCLARocketProject/2014-2015/issues/16#issuecomment-101881139

When you include the library using the arduino ide make sure you are
including the custom one. The ide might be storing the library under the
same name as another.
Alexander Omar
B.S. Computer Engineer (Spring 2017)
University of California Los Angeles
(530)448-3778
On May 13, 2015 7:08 PM, "Ky-Cuong Huynh"<EMAIL_ADDRESS>wrote:

I did, so I don't know why I get this error. I'll try again later.
--
Ky-Cuong Huynh
On Wed, May 13, 2015 at 7:02 PM, Vladimir Vysotsky
<EMAIL_ADDRESS>wrote:

I can't reproduce this error. Our custom libraries are the ones that
defined these values, and the setDataRate function. Did you install it?
Reply to this email directly or view it on GitHub:

https://github.com/UCLARocketProject/2014-2015/issues/16#issuecomment-101881139
—
Reply to this email directly or view it on GitHub
https://github.com/UCLARocketProject/2014-2015/issues/16#issuecomment-101881559
.

Fixed by copying Alex's libraries atop Ky-Cuong.


remove bootId from USN parser

Addressed #17
I updated the domain part of the USN parser to make it more consistent.
Removed the assignment for the conditionals to allow Kotlin to make inference about non-null.
Removed the first root device handling section because it's handled at the end as well.

I forgot you did this - whoops - thanks! It's fixed now 👍


Cache issue after upgrade - ValueError at /en/ not enough values to unpack (expected 4, got 3)

Issue Summary
Upgrading from:
wagtail 2.7.4 to 2.15.1
wagtail-modeltranslation 0.10.1 to 0.10.17
When browsing to a page the following error is encountered:
 ValueError at /en/ not enough values to unpack (expected 4, got 3)
Installed Middleware:
['django.middleware.cache.UpdateCacheMiddleware',
 'django.contrib.sessions.middleware.SessionMiddleware',
 'django.middleware.locale.LocaleMiddleware',
 'django.middleware.common.CommonMiddleware',
 'django.middleware.cache.FetchFromCacheMiddleware',
 'django.middleware.csrf.CsrfViewMiddleware',
 'django.contrib.auth.middleware.AuthenticationMiddleware',
 'django.contrib.messages.middleware.MessageMiddleware',
 'django.middleware.clickjacking.XFrameOptionsMiddleware',
 'django.middleware.security.SecurityMiddleware',
 'wagtail.contrib.redirects.middleware.RedirectMiddleware']

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/django/core/handlers/exception.py", line 47, in inner
    response = get_response(request)
  File "/usr/local/lib/python3.9/site-packages/django/core/handlers/base.py", line 204, in _get_response
    response = response.render()
  File "/usr/local/lib/python3.9/site-packages/django/template/response.py", line 105, in render
    self.content = self.rendered_content
  File "/usr/local/lib/python3.9/site-packages/django/template/response.py", line 83, in rendered_content
    return template.render(context, self._request)
  File "/usr/local/lib/python3.9/site-packages/django/template/backends/django.py", line 61, in render
    return self.template.render(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 170, in render
    return self._render(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 162, in _render
    return self.nodelist.render(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 938, in render
    bit = node.render_annotated(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 905, in render_annotated
    return self.render(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/loader_tags.py", line 150, in render
    return compiled_parent._render(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 162, in _render
    return self.nodelist.render(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 938, in render
    bit = node.render_annotated(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 905, in render_annotated
    return self.render(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/loader_tags.py", line 62, in render
    result = block.nodelist.render(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 938, in render
    bit = node.render_annotated(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 905, in render_annotated
    return self.render(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/defaulttags.py", line 514, in render
    values = {key: val.resolve(context) for key, val in self.extra_context.items()}
  File "/usr/local/lib/python3.9/site-packages/django/template/defaulttags.py", line 514, in <dictcomp>
    values = {key: val.resolve(context) for key, val in self.extra_context.items()}
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 671, in resolve
    obj = self.var.resolve(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 796, in resolve
    value = self._resolve_lookup(context)
  File "/usr/local/lib/python3.9/site-packages/django/template/base.py", line 858, in _resolve_lookup
    current = current()
  File "/usr/local/lib/python3.9/site-packages/wagtail/core/models/__init__.py", line 1284, in get_site
    url_parts = self.get_url_parts()
  File "/usr/local/lib/python3.9/site-packages/wagtail/core/models/__init__.py", line 1159, in get_url_parts
    possible_sites = [
  File "/usr/local/lib/python3.9/site-packages/wagtail/core/models/__init__.py", line 1161, in <listcomp>
    for pk, path, url, language_code in self._get_site_root_paths(request)

Exception Type: ValueError at /en/
Exception Value: not enough values to unpack (expected 4, got 3)

This error was apparently fixed in this PR
https://github.com/wagtail/wagtail/pull/6515
after this issue: https://github.com/wagtail/wagtail/issues/6513
But I am dealing with it here, or something very similar.
Steps to Reproduce
With an existing project in an old version, make sure you have a valid cache. Rerun the app container with the updated image. Navigate to a page (maybe /admin/).
Technical details

Python version: 3.9.10
Django version: 3.2.10
Wagtail version: 2.15.1

Site.get_site_root_paths() has additional checks for this specific use case 🤔 (specifically L150-175)
Could it be a particularity of your setup? Could you clear the cache after deployment for good measure?

Can I join to be part of creating solution for this Task? @edward-springload


Support pagination when fetching IAM users. #12

Added pagination support for fetching users via ListUsersPages (partially addressing #12), and a mocked version of ListUsersPages as well. The mocked call only returns a single page so should likely be fleshed out a bit better (whether part of this PR or a separate one).

Merged ! Thanks for your contribution.


Add scikit-learn dependency to pyproject.toml

anemoi-graphs requires scikit-learn, e.g. in anemoi.graphs.utils:
https://github.com/ecmwf/anemoi-graphs/blob/6cfcade797ba91d21ea186b81012db46aa3557ed/src/anemoi/graphs/utils.py#L14
but this package is not included in the pyproject.toml, so I got an ImportError right after installing anemoi-graphs. This very simple PR adds this dependency.

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

No problem! Thank you for opening all of this code up to the community!


Apple rejects our binaries because of references to Location

When i submit my app to Apple to validate, i received App Store connect warning about that my app is missing NSLocationAlwaysUsageDescription.
The problem is doesn't use any of these permissions, but because the my app is references to Plugin.Permission and Plugin.Permission is referenced to some of LocationManager of iOS SDK, so they're forcing us to have a string in Info.plist file.
But if we put the string into Info.plist file, our App in AppStore will always show something like : "This app may use your location even when it isn't open, which can decrease device battery life". And this will definitely not making the client happy.
More worse, some of other libraries is referenced to this package unfortunately, so i have to copy and paste these libraries in our app and clone this package.
So my suggestion is we should split the permissions in this package into smaller package for avoiding unnecessary permission references. Something like Plugin.Permission.Core and Plugin.Permission.Camera - Plugin.Permissions.Location for example

@jamesmontemagno any updates on this?

@jamesmontemagno
Hi we have the same problem here:
Here the Mail from apple after the binary upload:
Dear Developer,
We identified one or more issues with a recent delivery for your app, "SOME APP NAME". Please correct the following issues, then upload again.Missing Purpose String in Info.plist File - Your app's code references one or more APIs that access sensitive user data. The app's Info.plist file should contain a NSCalendarsUsageDescription key with a user-facing purpose string explaining clearly and completely why your app needs the data. Starting Spring 2019, all apps submitted to the App Store that access user data will be required to include a purpose string. If you're using external libraries or SDKs, they may reference APIs that require a purpose string. While your app might not use these APIs, a purpose string is still required. You can contact the developer of the library or SDK and request they release a version of their code that doesn't contain the APIs. Learn more (https://developer.apple.com/documentation/uikit/core_app/protecting_the_user_s_privacy).
Though you are not required to fix the following issues, we wanted to make you aware of them:Missing Purpose String in Info.plist File - Your app's code references one or more APIs that access sensitive user data. The app's Info.plist file should contain a NSLocationAlwaysUsageDescription key with a user-facing purpose string explaining clearly and completely why your app needs the data. Starting Spring 2019, all apps submitted to the App Store that access user data will be required to include a purpose string. If you're using external libraries or SDKs, they may reference APIs that require a purpose string. While your app might not use these APIs, a purpose string is still required. You can contact the developer of the library or SDK and request they release a version of their code that doesn't contain the APIs. Learn more (https://developer.apple.com/documentation/uikit/core_app/protecting_the_user_s_privacy).
Missing Purpose String in Info.plist File - Your app's code references one or more APIs that access sensitive user data. The app's Info.plist file should contain a NSLocationWhenInUseUsageDescription key with a user-facing purpose string explaining clearly and completely why your app needs the data. Starting Spring 2019, all apps submitted to the App Store that access user data will be required to include a purpose string. If you're using external libraries or SDKs, they may reference APIs that require a purpose string. While your app might not use these APIs, a purpose string is still required. You can contact the developer of the library or SDK and request they release a version of their code that doesn't contain the APIs. Learn more (https://developer.apple.com/documentation/uikit/core_app/protecting_the_user_s_privacy).
Best regards,The App Store Team
Which means we need to add:
NSCalendarsUsageDescription
NSLocationAlwaysUsageDescription
NSLocationWhenInUseUsageDescription
??
Thx Stefan

Did somebody tried the Beta 4.0.4 build already?
I can't find here any release / update notes.

Interesting.... I am back from holiday and will look at it this week.
If you add all of the Descriptions into the info.plist does it go away?

Hey @jamesmontemagno , welcome back from holiday. Hope you had a good time ;)
Yesterday i played a bit with the store upload. I added just the Calendar description now.
I just saw it today in the mail: For the NSCalendarsUsageDescription they wrote "Please correct the following issues"
For the location they wrote "Though you are not required to fix the following issues, we wanted to make you aware of them".
I have just added the calendar now and the app is in review with the other 2 warnings.
I just know it from the past, Apple can be annoying when you have the usage descriptions for features which you are not using.
I will tell you in some hours or days when the review is done.
Thx a lot for your work,
and greetings from austria.

Interesting.... I am back from holiday and will look at it this week.
If you add all of the Descriptions into the info.plist does it go away?

Hi @jamesmontemagno
The problem is we can't just add everything into Info.plist
If we add all descriptions into Info.plist, then App Store will tell user something like : "ABC app may use your Location when you're not using the app" or something similar than that. Users will feel like they're being spy although our app doesn't use any user's location.
Not to mention that we're references a lot of permissions in the Plugins. And the customers will not be happy about that.

And if we don't add the descriptions correctly, Apple would be annoying and make the reviewing steps longer than expected.

Thats the real problem! Yes :)
You don't need to add the location description but apple will ask you for sure whats going on here, depends on which reviewer you will get ;)

I'm having this issue as well. Maybe wrapping everything up in preprocessor directives and having a bunch of symbols could help?
Changes to code:
// ...
#if (INCLUDE_CONTACTS)
PermissionStatus ContactsPermissionStatus
{
    get
    {
        var status = ABAddressBook.GetAuthorizationStatus();
        switch (status)
        {
            case ABAuthorizationStatus.Authorized:
                return PermissionStatus.Granted;
            case ABAuthorizationStatus.Denied:
                return PermissionStatus.Denied;
            case ABAuthorizationStatus.Restricted:
                return PermissionStatus.Restricted;
            default:
                return PermissionStatus.Unknown;
        }
    }
}
#else
PermissionStatus ContactsPermissionStatus = PermissionStatus.Unknown;
#endif
// ...

The default nuget package would have all of these added to Project Settings > Build > Compiler > Define Symbols. Developers who don't want to include these can just clone the repo and compile it themselves excluding the symbols they don't need.

Works is in progress on PR #137, it is a huge change. There will be a new API that will need to be adopted for things to be fully linker safe for everyone, however it is not a breaking change for existing apps, but you will need to update to ensure that the linker kicks in on iOS/Android to remove things. Hopefully will have it done an in beta this weekend.

Hi @jamesmontemagno,
Just to add one more data point. Our app uses the location plugin and only requires the location while it is running. It has worked like that for several months. Now, suddenly, the location stopped working with a location not granted error even on the version that is in the app store.
I tried following the advice by Apple (we also got their warning email) but it does not solve the problem. I am looking forward to this change but there may be more underneath it (or I am doing something really dumb somewhere).

Hi @jamesmontemagno ,
Any ETA on when you can merge it and release a new version?

Now that build is over, i should be able to get it done this week :) I will atleast have it in preview, and update my other libraries to have a dependency on the new API.

Shipping 5.0 today to beta and updating my plugins that use it

Thanks for your hard work..but I just tried uploading to the App Store with the version 5.0 beta, and I’m still getting the same outcome James. “Info.plist file should contain a NSLocationAlwaysUsageDescription key“ I’m off to double check nothing else might be causing it in my code, has the fix worked for others??

I have also updated my Geolocator and Media Plugins to new beta pacakges.
If you use Geolocator you will need that in there as the code uses both of them. If you aren't then you would want to use Xamarin.Essentials as it only uses "WhenInUse"

Also, i completely changed the API... so read the readme

Thanks @jamesmontemagno you hit the nail on the head, I'm an idiot... I am using the geolocator plugin!  We also have the Xamarin.Essentials for something else so I might make the effort now and migrate to it for geolocation.

Will be the easiest migration ever :)

Will be the easiest migration ever :)

Yeah, you're right, I also swapped over DeviceInfo, thanks for the great plugins!


Timed ban command

Make the ban command accept time parameter like
!ban member_id 2 weeks
Save this data to database #14

It's called tempban.

Thank you. Have a nice day!


Try PowerMock in other IDE's than Eclipse

From<EMAIL_ADDRESS>on September 22, 2008 20:04:16
Idea, netbeans
Original issue: http://code.google.com/p/powermock/issues/detail?id=15

From<EMAIL_ADDRESS>on September 22, 2008 11:04:39
Labels: -Type-Other Type-Task

From<EMAIL_ADDRESS>on October 31, 2008 00:53:17
Works in Intellij and Netbeans
Status: Fixed


Is this project abandoned?

As far as I can tell, the last time changes were merged into this project was June 2020 (two years ago), and the project owner hasn't responded to any of the issues since March 2020.
For anyone looking for an actively used and maintained version of this package, we have one over at https://github.com/packbackbooks/lti-1-3-php-library which contains the following:

Fixes security vulnerabilities found in the existing package
Adherence to PSR-4 standards
Fixes to many of the bugs and open issues on the existing package

composer require packbackbooks/lti-1p3-tool

Holy smokes... this is amazing, thank you so much!
I ended up essentially doing what you did -- taking this official library and extending it to accommodate changes in the 1.3 specification and implementing things it didn't already support.
I started to encounter a bunch of issues with the Deep-Linking 2.0 flow from using the current implementation of this official library, and that's when I went searching for issues and saw this😂

As far as I can tell, the last time changes were merged into this project was June 2020 (two years ago), and the project owner hasn't responded to any of the issues since March 2020.
For anyone looking for an actively used and maintained version of this package, we have one over at https://github.com/packbackbooks/lti-1-3-php-library which contains the following:

Fixes security vulnerabilities found in the existing package
Adherence to PSR-4 standards
Fixes to many of the bugs and open issues on the existing package

composer require packbackbooks/lti-1p3-tool

Holy smokes... this is amazing, thank you so much!
I ended up essentially doing what you did -- taking this official library and extending it to accommodate changes in the 1.3 specification and implementing things it didn't already support.
I started to encounter a bunch of issues with the Deep-Linking 2.0 flow from using the current implementation of this official library, and that's when I went searching for issues and saw this😂

Hi,
Is it possible to have a code example to create a provider that uses this package? I need to understand LTI 1.3 and LTI Advanced and create a provider to integrate it into Canvas LMS.
Thanks in advance!!!
Regards.

If you follow the link in my original post, the wiki has some code examples to help get you started.


[CHANGE ME] Re-generated to pick up changes from self.

This PR was generated using Autosynth. :rainbow:
Synth log will be available here:
https://source.cloud.google.com/results/invocations/c31c946c-ef0a-4159-9a30-b49732d0b5d5/targets

[ ] To automatically regenerate this PR, check this box.

Closing this PR as these files need to be excluded from auto-generation.

Closing this PR as these files need to be excluded from auto-generation.


AutoComplete can not be cleared after a selection is made (with redux-form 7)

See https://codesandbox.io/s/w6ll91mom8 for a demo with redux-form 7.3.0

same here, when using the dataSourceConfig


Add a benchmark.

Added a benchmark, which parses automatically generated json data.
I think the performance is much worse than pest. Maybe test on pest later.
And I think we should add tests and benchmarks to CI.

Summary by CodeRabbit

New Features

Introduced benchmarking for JSON parsing using pest3 and serde_json.
Added support for benchmarking deep and shallow JSON objects.

And we can add some features to improve typed api's performance, such as using an allocator/arena and adding a rule optimizer (but it's harder on typed api, and in some cases it can only work as a linter).
I may have some time after around a month.
@tomtau Do you have any plan on our future works?


Can't assert that an async function taking parameters and awaiting, throws?

I'm trying to use assertThrows or assertRejects to assert that an async function taking a parameter and awaiting a fetch call, throws. This is the function:
async function asyncFuncThatAwaitsAndThrows(format = 'json') {
  await fetch(`https://api.ipify.org?format=${format}`);
  throw new Error('This async function awaited and had to throw');
}

assertRejects(asyncFuncThatAwaitsAndThrows) correctly does not throw and the test passes. Because asyncFuncThatAwaitsAndThrows must take parameters, it needs to be called like this from assertRejects:
assertRejects(() => asyncFuncThatAwaitsAndThrows('text'));

But that leads to an error that leaks were detected:

"fetchCancelHandle" was created during the test, but not cleaned up during the test. Close the resource before the end of the test.
An async operation to send a HTTP request was started in this test, but never completed. This is often caused by not awaiting the result of a fetch call.

However, asyncFuncThatAwaitsAndThrows did await the fetch.
Using assertThrows apparently makes the function NOT throw:
assertThrows(async () => await asyncFuncThatAwaitsAndThrows('text'));

error: AssertionError: Expected function to throw.

Steps to Reproduce
import { assertRejects, assertThrows } from "@std/assert";

async function asyncFuncThatThrows(param?: string) {
  throw new Error(`This async function throws ${param}`);
}

async function asyncFuncThatAwaitsAndThrows(format = 'json') {
  await fetch(`https://api.ipify.org?format=${format}`);  // i.e. some async operation in a real use case
  throw new Error('This async function awaited and had to throw');
}

function funcThatThrows(param?: string) {
  throw new Error(`This function throws ${param}`);
}

Deno.test('this test should pass and not throw',  async () => {
  // await asyncFuncThatThrows();  // correctly throws, test fails
  assertRejects(asyncFuncThatThrows);  // correctly does NOT throw, test passes
  // assertRejects(asyncFuncThatThrows('bar'));  // incorrect syntax, must use arrow function to pass param
  assertRejects(() => asyncFuncThatThrows('bar'));  // correctly does NOT throw, test passes
  assertThrows(() => funcThatThrows('foo'));  // correctly does NOT throw, test passes

  // The goal: assert that an async function taking a parameter and awaiting, throws

  // Can't use assertRejects below due to error Leaks detected:
  // -  "fetchCancelHandle" was created during the test, but not cleaned up during the test. Close the resource before the end of the test.
  // - An async operation to send a HTTP request was started in this test, but never completed. This is often caused by not awaiting the result of a `fetch` call.
  /** But {@link asyncFuncThatAwaitsAndThrows} did await the fetch! */
  assertRejects(() => asyncFuncThatAwaitsAndThrows('text'));  // <-- BUG?

  // await asyncFuncThatAwaitsAndThrows('text');  // correctly throws (sanity check)

  // Comment the last assertRejects() call above, and uncomment the assertThrows line below
  // "error: AssertionError: Expected function to throw."
  assertThrows(async () => await asyncFuncThatAwaitsAndThrows('text'));  // <-- BUG?
});

Environment

OS: Fedora Linux
deno version: 2.1.3
@std/assert version: 1.0.9

You need to await assertRejects(

Thank you for the clarification @BlackAsLight. After adding that await, I get the same error as from the next statement, assertThrows(async () => await asyncFuncThatAwaitsAndThrows('text'));:

error: AssertionError: Expected function to throw.

But I believe the assertThrows statement should succeed, because asyncFuncThatAwaitsAndThrows('text') does throw. Maybe I'm missing something else here?

assertRejects expects a function that returns a promise to reject. assertThrows expects a non-async non-promise returning function to throw.

So assertRejects was the right track, but I had to await it, and consume the response body to prevent that leak. This works; is it the recommended structure?
import { assertRejects } from '@std/assert'

async function asyncFuncThatAwaitsAndThrows(format = 'json') {
  const res = await fetch(`https://api.ipify.org?format=${format}`);
  await res.text();  // consume the response body to prevent resource leaks
  throw new Error('This async function awaited and had to throw');
}

Deno.test('asyncFuncThatAwaitsAndThrows should throw', async () => {
  await assertRejects(async () => await asyncFuncThatAwaitsAndThrows('json'));
});

I don't know what you mean by consume, but the code you have here is correct. To not have a leak you need to have all promises created in the test either resolved or rejected before the test returns. If there is any promises still hanging around in the event loop from the test then that's considered a leak.


Updated classifier-tc-definitions.md :Added new classifier definitions

Added 12 new Trainable Classifier Definitions

Please do not complete/merge this PR. Ping me on teams before completing.

Learn Build status updates of commit cc67da5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/compliance/classifier-tc-definitions.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@microsoft-github-policy-service agree [company="{Microsoft}"]

back to ursenha
line 221, I think the incorrect word is being used.  "abiding" is used and I think the term should be "binding"

@microsoft-github-policy-service agree [company="Microsoft"]

@microsoft-github-policy-service agree


Task1

Step: 1 Add Input Dialog Box and give the input
Step 2: Add Message box to display a message

@gowribolem
After completing the code,  analyzed the file once, changed the default activity names of each activity, and then added annotations
Task -1
Code verified. Default activity names

Task -2
Code verified. Default activity names

Task -11
Code verified. Default activity names


[FEAT] Add flag for changing stream persistence to Memory

It would be great to be able to configure all of the streams that wadm creates to use https://docs.rs/async-nats/0.37.0/async_nats/jetstream/stream/enum.StorageType.html#variant.Memory instead of File, which would be great to create a more ephemeral environment in NATS.
It would be great to be able to have a flag --stream-persistence that defaults to File but can be overridden to be Memory.

Hey @brooksmtownsend I would pick it up this one. Or rather will create a PR, so you can assign it to me. I'll have some questions about some testing, but will ask there rather.


使用vscode写js文件变卡了

您好，最近使用vscode for Mac版本才写了4000行的js文件，写代码的时候会有卡的现象，比如说，点击折叠代码的+没有任何效果，或者写了一个fun也没有任何提示，插件才装了16个

Translation here: Recently he wrote a js(javascript) file with vscode for Mac, in the process he experienced pauses/lags. For example, when click on the folding "+" symbol and nothing happened. Some times there is no code completion prompt. He has 16 plugins installed.

Closing as we cannot investigate this issue without more information, such as if this happens when extensions are disabled


Kong OAuth2.0 Admin API breaking change.

Summary
Seems a patch to a consumers oauth2.0 client_id / client_secret endpoint has new breaking behavior.
My programmatic code doing the update(which used to do our oauth2.0 client credentials grant pattern):

This used to work, not sure which version specifically it stopped working but somewhere around:
2.1.4 or earlier.
I would think that a patch should not require these extra fields such as hash_secret (which for me is set to false on all our existing oauth2 cred pairs, because its a newer field generally that people use I suppose for encrypting their id/secret) seems like it should be an optional field set to false for admin API behavior w a patch.
Going to manually test setting it to false now because I am already sending a client_secret field in so this must be the only other new field causing a problem.
Response from my patch call:
HTTP Status: 400

{"message":"schema violation (all or none of these fields must be set: 'hash_secret', 'client_secret')","name":"schema violation","fields":{"@entity":["all or none of these fields must be set: 'hash_secret', 'client_secret'"]},"code":2}

Steps To Reproduce

Try a patch with just client_id and client_secret sent to admin API like my code screenshot.

Additional Details & Logs

Kong version 2.1.4

Just confirming adding the hash_secret field set to false did fix the problem for my patch. Still don't think the admin API is behaving properly for that call, should not be a required field on patch if it naturally defaulted to false for all my existing resources at the time it was introduced. Feel free to close this if not something that needs addressing in Kong's opinion, just bringing it up.

@jeremyjpj0916, yes, @hbagdi originally reported this:
https://github.com/Kong/kong/issues/6173
See more info on that #6173, it is quite tricky. If you have a solution or proposal, I would love to hear about that.

@bungle
Hmm can't say I have the all the answers for impl, as you are the wizard there haha, but I can say for how I think RESTful design works and considering the HTTP PATCH method would say what I think on your questions posed in that thread:

A base case, no confusion here I think.

patch client_secret=x -> should we hash the secret (as the previous was hashed) OR should we add x as clear text and change hash_secret=false (the default?)

They did not pass what I think should be an optional field hash_secret, and yes therefor should be clear text and hash_secret set to false. In the case  hash_secret on the existing resource is already true then you would honor the existing config and hash x as its patched into the db.

patch hash_secret=true -> should we hash the secret stored in table, or perhaps if it was already true should we just skip transformation so that we don't double transform?

A patch of the same config value ='s no change to the resource imo, hence no re-hash if already true in the first place.

patch hash_secret=false -> if it was already false then skip transformation, if it was previously true what can we do, show an error (we cannot de-hash easily :-)?

Hmm 4. is slightly interesting but should not come up if the operator understood the ramifications of his design choices(to use the hashing feature). Probably a 400 status and an error message stating credentials were hashed prior and by design this request is not fulfillable. Now if Kong had used an encrypt/decrypt design on the db data vs hashing checks the reverting would be possible :P .
Anyone come up with counter arguments to the above logic when thinking through how patch is supposed to function against restful resources?


Clarify ODE ImplicitSolve documentation

Closes #3395

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#3443
@jandrej
@pazner
@cjvogl + @bensworth + @v-dobrev
2/8/23
⌛due 2/22/23
⌛due 3/1/23

PR Checklist

[ ] Code builds.
[ ] Code passes make style.
[ ] Update CHANGELOG:

[ ] Is this a new feature users need to be aware of? New or updated example or miniapp?
[ ] Does it make sense to create a new section in the CHANGELOG to group with other related features?

[ ] Update INSTALL:

[ ] Had a new optional library been added? If so, what range of versions of this library are required? (Make sure the external library is compatible with our BSD license, e.g. it is not licensed under GPL!)
[ ] Have the version ranges for any required or optional libraries changed?
[ ] Does make or cmake have a new target?
[ ] Did the requirements or the installation process change? (rare)

[ ] Update continuous integration server configurations if necessary (e.g. with new version requirements for each of MFEM's dependencies)

[ ] .github
[ ] .appveyor.yml

[ ] Update .gitignore:

[ ] Check if make distclean; git status shows any files that were generated from the source by the project (not an IDE) but we don't want to track in the repository.
[ ] Add new patterns (just for the new files above) and re-run the above test.

[ ] New examples:

[ ] All sample runs at the top of the example source file work.
[ ] Update examples/makefile:

[ ] Add the example code to the appropriate SEQ_EXAMPLES and PAR_EXAMPLES variables.
[ ] Add any files generated by it to the clean target.
[ ] Add the example binary and any files generated by it to the top-level .gitignore file.

[ ] Update examples/CMakeLists.txt:

[ ] Add the example code to the ALL_EXE_SRCS variable.
[ ] Make sure THIS_TEST_OPTIONS is set correctly for the new example.

[ ] List the new example in doc/CodeDocumentation.dox.
[ ] If new examples directory (e.g.examples/pumi), list it in doc/CodeDocumentation.conf.in
[ ] Companion pull request for documentation in mfem/web repo:

[ ] Update or add example-specific documentation, see e.g. the src/examples.md.
[ ] Add the description, labels and screenshots in src/examples.md and src/img.
[ ] In examples.md, list the example under the appropriate categories, add new categories if necessary.
[ ] Add a short description of the example in the "Extensive Examples" section of features.md.

[ ] New miniapps:

[ ] All sample runs at the top of the miniapp source file work.
[ ] Update top-level makefile and makefile in corresponding miniapp directory.
[ ] Add the miniapp binary and any files generated by it to the top-level .gitignore file.
[ ] Update CMake build system:

[ ] Update the CMakeLists.txt file in the miniapps directory, if the new miniapp is in a new directory.
[ ] Add/update the CMakeLists.txt file in the new miniapp directory.
[ ] Consider adding a new test for the new miniapp.

[ ] List the new miniapp in doc/CodeDocumentation.dox
[ ] If new miniapps directory (e.g.miniapps/nurbs), add it to MINIAPP_SUBDIRS in the makefile.
[ ] If new miniapps directory (e.g.miniapps/nurbs), list it in doc/CodeDocumentation.conf.in
[ ] Companion pull request for documentation in mfem/web repo:

[ ] Update or add miniapp-specific documentation, see e.g. the src/meshing.md and src/electromagnetics.md files.
[ ] Add the description, labels and screenshots in src/examples.md and src/img.
[ ] The miniapps go at the end of the page, and are usually listed only under a specific "Application (PDE)" category.
[ ] Add a short description of the miniapp in the "Extensive Examples" section of features.md.

[ ] New capability:

[ ] All new public, protected, and private classes, methods, data members, and functions have full Doxygen-style documentation in source comments. Documentation should include descriptions of member data, function arguments and return values, template parameters, and prerequisites for calling new functions.
[ ] Pointer arguments and return values must specify whether ownership is being transferred or lent with the call.
[ ] Any new functions should include descriptions of their intended use e.g. for internal use only, user-facing, etc., along with references to example code whenever possible/appropriate.
[ ] Consider adding new sample runs in existing examples to highlight the new capability.
[ ] Consider saving cool simulation pictures with the new capability in the Confluence gallery (LLNL only) or submitting them, via pull request, to the gallery section of the mfem/web repo.
[ ] If this is a major new feature, consider mentioning it in the short summary inside README (rare).
[ ] List major new classes in doc/CodeDocumentation.dox (rare).

[ ] Update this checklist, if the new pull request affects it.
[ ] Run make unittest to make sure all unit tests pass.
[ ] Run the tests in tests/scripts.
[ ] (LLNL only) After merging:

[ ] Update internal tests to include the new features.

Please add the ready-for-review label if/when this is ready.

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

@jandrej, can you elaborate on what the Newton mention is clarifying?

@jandrej, can you elaborate on what the Newton mention is clarifying? Perhaps I am missing something, but the addition reads to me that ImplicitSolve will always implement a Newton approach... is that the clarification?

The addition should give a hint to the person that implements this method that usually a nonlinear solve is required but it also gives the clarification what the residual and linearization look like (which was not immediately clear to me).

I think the extra documentation is a good idea, but you have to be careful with mass matrices - should k have a mass matrix and the identity inside be a mass matrix? I am not positive without writing it out more carefully because I usually write out RK methods in a stage value rather than stage derivative formulation, but please check.
Also minor, but I suggest bringing the minus sign inside of the Jacobian. As written, I think you are inverting a negative definite sort of operator (say your problem is linear, then df/du has negative eigenvalues/FOV for stability, and you then subtract I). Because solvers are typically formulated based on assumptions of positivity, swapping the sign to be I - df/du IMO is a more natural way to look at it, and then you also have the form of a standard fixed point iteration, x_{i+1} = x_i + M^(-1)r_i.

Also relevant:
https://github.com/mfem/mfem/pull/1602

To @bensworth's point about the mass matrix, perhaps it is worth noting first that ImplicitSolve is for solving
$$\frac{dy}{dt} = g(y,t)$$
whether the $\vec{f}(\vec{z},t)$ in the comments represents $\vec{g}(\vec{z},t)$ in
$$M \frac{d\vec{y}}{dt} =\vec{g}(\vec{y},t)$$
or $M^{-1} \vec{g}(\vec{z},t)$ in
$$\frac{d\vec{y}}{dt} = M^{-1} \vec{g}(\vec{y},t)$$

Good point @cjvogl. If the point of this PR is to make the implicit solve requirements more clear, I would personally vote to pose it in terms of mass matrices, or at least include that formulation as well, because virtually every implicit solve done through MFEM will take that form

Have all of the concerns been addressed here?  @pazner @bensworth @jandrej @cjvogl

I believe this is waiting on https://github.com/mfem/mfem/pull/3443#issuecomment-1470808256

@jandrej Can you address this https://github.com/mfem/mfem/pull/3443#issuecomment-1470808256

As this is apparently more confusing than the previous docs, I'll close the PR.

@jandrej the old documentation is not good in my opinion, addressing https://github.com/mfem/mfem/pull/3443#issuecomment-1470808256 should not be a big task and makes a lot more sense than just dropping the PR altogether

I agree that the current documentation should be improved.  For a tutorial that I am writing, it took a nontrivial amount of time to re-parse the current documentation and explain things.  This is what I came up with, in case it is useful:

@bensworth I asked @cjvogl to use the material he put together for his tutorial to address this.  Stay tuned.

@jandrej, are you ok if I push changes to this branch/PR or would you prefer I start a new branch/PR?

go ahead

@jandrej, sorry that I hadn't gotten around to updating my comment here.  I found it useful to expand the clarification to the documentation of TimeDependentOperator as a whole... so I started a new PR #4115

@jandrej, I believe this is all being addressed in PR https://github.com/mfem/mfem/pull/4115
Do you want to close this one?


Fix falsy override

This fixes a problem when overrides are set to falsy values.
In that case overrides would not be applied.
Also fixes the hasKey function which before was not working correctly.

:+1:

nice catch, thanks for this

@rawls238 could you release this to npm?

@TimBeyer done


Feature

Deleting useless file, updating readme file, creating new directory.

Add .patch or .diff to the end of URLs for Git’s plaintext views.


Lose XBox wireless adapter connection when exiting from application back to Flex Launcher

I'm using Flex Launcher to switch between different retro arcade/mame builds. If I launch a build and am away from the computer for a bit, the Xbox controller will go to sleep. When I come back, the controller reconnects fine to the wireless adapter, but as soon as I exit the app and go back into Flex Launcher, the connection is lost and the only way to get it back is to unplug and plug back in the adapter.

Which OS are you using, and which model of Xbox controller?

Windows 10 w/ latest updates, and both Xbox One S & Xbox Series X with the latest firmware.

I'll need a debug log to try to diagnose this. Run the launcher using flex-launcher.exe -d. Reproduce the issue, then quit the program. That will create a log file flex-launcher.log in the same directory as the executable. Upload it directly or via a pastebin site.
One other question: is this exclusively a program with Flex Launcher? For example, in your other application, does it detect the controller after you've reconnected it?

flex-launcher.log
Hi, thanks for checking this out - log is attached. And yes, the controller is detected by the other app after awakening, It only loses connection when I exit that app and go back into Flex Launcher menu.
In the log I am starting Flex Launcher with the controller disconnected and then turning it on so it then connects wirelessly. I launch the app, use the app a bit, then walk away from the machine and eventually the controller sleeps and you can see in the log that gamepad is disconnected. Then I come back, wake up the controller, you see gamepad connected again, I use the app a bit and then exit out of the app at which time you see gamepad disconnected before I hit escape to exit out of Flex Launcher.

Actually, I don't even need to start an app. I've found like I can launch FL, connect the XBOX controller, then turn the controller off, then power it on and try to reconnect again and it doesn't connect. And I still can't connect even if I unplug and plug back in the wireless usb adapter.

Thanks for the log. That is helpful for debugging.
When you say the controller doesn't connect, are you referring to the actual wireless connection with the adapter? If so, it sounds like the cause might be libsdl-org/SDL#3468. Users of other SDL-based programs have reported bugs with similar symptoms as yours, like here for example. The good news is that there is a fix is available for that SDL bug.

Yes, I was referring to the wireless connection. The gamepad does shutoff after trying to reconnect again, just like in the links you shared.

OK. Download this development build, which was built against the latest stable version of SDL which has the fix for that bug incorporated. Extract flex-launcher.exe and put it into your setup and let me know if it fixes your issue.

Works great now! Thanks very much for responding to this so quickly. I use this app as a front end on a dedicated retro arcade PC and I think it's great to easily switch between different builds.

Glad it hear its working now.
I'm going to try to do another release soon. It's been a while since the last one. You can continue using that development build until then.

@complexlogic Did you manage to do a release with this change? Couldn't see it in compare. Having the same issue with 2.1


Update to ring 0.16.

This is blocked on dependencies of Krill requiring ring 0.14.
Closing for now.


Publish CPU load

I think it would be nice to have sensors for the cpu load of the last 1, 5 and 15 minutes. It says much more about the total load of the system than the average cpu usage of the last 60 seconds.

Hm, would have to redesign the collection process, but might be interesting indeed


[Concurrency] Fix Android C libdispatch build

We need to pass CMAKE_ANDROID_NDK and CMAKE_ANDROID_ARCH_ABI to the
build.

@swift-ci smoke test

@drodriguez I think I came up with a solution. Would you mind taking another look?

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

Thanks for the fix. I will send a follow up for Android ARMv7, which doesn't seem to like the CMAKE_SYSTEM_PROCESSOR for some reason. It should be easy to map to the right one.

Thanks! Please tag me in your PR.


fix: Resolve issue with copyright year

Related Issue

Description

Screenshots

@Komal-Rauniyar Thanks for the contribution but currently this feature is already up and working.


[WIN10] [WSL] Gleam Language Server client: couldn't create connection to server.

Windows 10 Home - 22H2
WSL 2 - Debian
I installed Gleam in both the Linux and Windows systems.
For Linux, I installed it via Homebrew (4.2.17) and added /home/linuxbrew/.linuxbrew/bin/gleam to the $PATH
For Windows, I installed it via Scoop(0.3.1) and added C:\Users\askme\scoop\apps\gleam\current to the Path
I can confirm via the Debian and Powershell terminal that Gleam is functioning and that I can run my project from both terminals.

Is there something I missed, I only installed Gleam into Windows 10 as I assumed VSCode would be using that and it didn't make a difference. Any help would be appreciated.

Sorry forgot the error output
[Error - 10:02:36 PM] Gleam Language Server client: couldn't create connection to server.
Launching server using command gleam failed. Error: spawn gleam ENOENT

I don't know what changed by in the spar of the moment I restarted my computer once more and it fixed the issue. I don't know, I didn't even change anything in between the last restart I did. Oh well, all good now...


Bad URL requests hang forever

For example, localhost:4444/xapi/v2/u/Dad-12262/stats/general or
localhost:4444/api/v0/u/Dad-12262/heroes.
Console Log:
2016-08-05 13:01:07,923 - [INFO] OWAPI -> Started OWAPI server.
2016-08-05 13:01:17,291 - [INFO] Kyokai -> HTTP/1.1 GET /xapi/v2/u/Dad-12262/stats/general - 404
2016-08-05 13:01:17,298 - [ERROR] asyncio -> Task exception was never retrieved
future: <Task finished coro=<Kyoukai.delegate_request() done, defined at /usr/local/lib/python3.5/site-packages/kyoukai/app.py:316> exception=AttributeError("'NoneType' object has no attribute 'groups'",)>
Traceback (most recent call last):
  File "uvloop/task.pyx", line 126, in uvloop.loop.BaseTask._fast_step (uvloop/loop.c:97501)
  File "/usr/local/lib/python3.5/site-packages/kyoukai/app.py", line 352, in delegate_request
    await self.handle_http_error(fof, protocol, ctx)
  File "/usr/local/lib/python3.5/site-packages/kyoukai/app.py", line 303, in handle_http_error
    resp = wrap_response(await error_handler.invoke(ctx))
  File "/usr/local/lib/python3.5/site-packages/kyoukai/route.py", line 108, in invoke
    matches = self.matcher.fullmatch(ctx.request.path).groups()
AttributeError: 'NoneType' object has no attribute 'groups'

Not sure if this is more OWAPI or Kyoukai related.
Possible solutions: Redirect malformed requests somewhere, or return a general error.

This is definitely a Kyoukai issue.

I'll post it there and close it here then.


[Veeam] Fix restore of reimported VMs

Description
Using the VMware hypervisor with the Veeam plugin active, when restoring a VM that has been unmanaged from ACS and reimported, an error occurs because ACS duplicates the register in the database of this VM. This PR aims to fix this behavior.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?
It was tested in a local lab:

I created one new VM via ACS, unmanaged this VM, and then reimported this VM again to ACS;
I added this VM to one backup offering and made one manual backup;
I tried to restore this VM backup;
Before ACS failed to restore the VM and duplicate the VM in the UI and database;
ACS now completes the restore successfully, and there is no VM duplication.

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 5592

@blueorangutan test rocky8 vmware-67u3

@DaanHoogland a Trillian-Jenkins test job (rocky8 mgmt + vmware-67u3) has been kicked to run smoke tests

Trillian test result (tid-6189)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8
Total time taken: 44569 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7239-t6189-vmware-67u3.zip
Smoke tests completed. 108 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

@SadiJr
if you want this in <IP_ADDRESS>, can you rebase with 4.18 branch ? thanks

moved to <IP_ADDRESS> as the target branch is main

@SadiJr can you please check review comments

@SadiJr can you answer @harikrishna-patnala 's concerns?

ping @SadiJr

@harikrishna-patnala  @DaanHoogland Sorry for the delay, I will check the suggestions and respond/work on this PR

@SadiJr @harikrishna-patnala @DaanHoogland
I tried to reproduce the issue on veeam12 environment with some changes (see https://github.com/apache/cloudstack/pull/8241)
However, the restore of backup succeeds in my testing

unmanage a running vm
import it to cloudstack
got two VMs with same instance name

+-----+---------------+--------------------------------------+---------------+---------------+---------------------+---------------------+
| id  | name          | uuid                                 | instance_name | display_name  | created             | removed             |
+-----+---------------+--------------------------------------+---------------+---------------+---------------------+---------------------+
| 162 | admin-001-001 | c13f4662-dd08-4f24-abd3-c1b581a3ecd2 | i-2-162-VM    | admin-001-001 | 2023-12-01 08:22:55 | 2023-12-07 20:00:16 |
| 225 | i-2-162-VM    | a974a38d-5b9a-40d2-a0fb-feb80ef88762 | i-2-162-VM    | NULL          | 2023-12-07 20:01:01 | NULL                |
+-----+---------------+--------------------------------------+---------------+---------------+---------------------+---------------------+
2 rows in set (0.00 sec)

assign backup offering
create a backup
stop the vm, and restore the backup.

I checked the description and code again, I think we should not fix an issue with veeam backup/restore by changing the processes of unmanage/manage VM.
We should fix the issue in veeam plugin, if the issue can be reproduced.

@weizhouapache can you please share the state of VMs admin-001-001 and i-2-162-VM in your database after doing your test? And about fixing an issue with Veeam by changing the unmanage/manage process, I think having two VMs with same instance_name in database is a inconsistency that is being generated by the manage/import VMware VMs. The internal name is expected to be unique.

@weizhouapache that is strange. Your results differ a lot from mine. I review the changes in #8241, but, from what I understand, they should not impact this behavior. I will repeat my tests, using the most recent community version, this PR of mine and #8241, and I will return an opinion. Unfortunately, this may take some time as I am without my VMware environment at the moment.

Hi @SadiJr Please resolve conflicts, and let me know if this PR can be targeted for 4.19.1 (if so, rebase with 4.19 branch).

ping @harikrishna-patnala

Hi @harikrishna-patnala Please check/confirm if your comments are addressed. Thanks.

@blueorangutan package

@sureshanaparti a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 10076

@blueorangutan test alma9 vmware-70u3

@sureshanaparti a [SL] Trillian-Jenkins test job (alma9 mgmt + vmware-70u3) has been kicked to run smoke tests

[SF] Trillian test result (tid-10576)
Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server a9
Total time taken: 42102 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7239-t10576-vmware-70u3.zip
Smoke tests completed. 105 look OK, 10 have errors, 16 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_restore_vm
Error
3614.41
test_restore_vm.py

test_02_restore_vm_allocated_root
Error
7.75
test_restore_vm.py

ContextSuite context=TestRestoreVM>:teardown
Error
15.11
test_restore_vm.py

test_01_prepare_and_cancel_shutdown
Error
2.54
test_safe_shutdown.py

test_02_trigger_shutdown
Failure
301.27
test_safe_shutdown.py

test_01_scale_vm
Error
0.02
test_scale_vm.py

test_02_scale_vm_negative_offering_disable_scaling
Error
0.02
test_scale_vm.py

test_03_scale_vm_negative_vm_disable_scaling
Error
0.02
test_scale_vm.py

test_04_scale_vm_with_user_account
Error
2.99
test_scale_vm.py

test_04_scale_vm_with_user_account
Error
2.99
test_scale_vm.py

test_05_scale_vm_dont_allow_disk_offering_change
Error
0.08
test_scale_vm.py

ContextSuite context=TestScaleVm>:teardown
Error
0.22
test_scale_vm.py

ContextSuite context=TestServiceOfferings>:setup
Error
2.87
test_service_offerings.py

ContextSuite context=TestSetSourceNatIp>:setup
Error
0.00
test_set_sourcenat.py

ContextSuite context=TestSnapshotRootDisk>:setup
Error
0.00
test_snapshots.py

ContextSuite context=TestSnapshotStandaloneBackup>:setup
Error
0.00
test_snapshots.py

test_05_stop_ssvm
Error
0.12
test_ssvm.py

test_06_stop_cpvm
Error
0.11
test_ssvm.py

test_07_reboot_ssvm
Error
0.11
test_ssvm.py

test_08_reboot_cpvm
Error
0.11
test_ssvm.py

test_09_reboot_ssvm_forced
Error
0.11
test_ssvm.py

test_10_reboot_cpvm_forced
Error
0.10
test_ssvm.py

test_11_destroy_ssvm
Error
0.09
test_ssvm.py

test_12_destroy_cpvm
Error
0.09
test_ssvm.py

test_01_import_storage_policies
Error
5.08
test_storage_policy.py

ContextSuite context=TestVMWareStoragePolicies>:teardown
Error
5.11
test_storage_policy.py

ContextSuite context=TestCopyAndDeleteTemplatesAcrossZones>:setup
Error
0.00
test_templates.py

ContextSuite context=TestCreateTemplate>:setup
Error
0.00
test_templates.py

ContextSuite context=TestTemplates>:setup
Error
5.75
test_templates.py

test_01_positive_tests_usage
Failure
2.78
test_usage_events.py

test_01_positive_tests_usage
Error
2.78
test_usage_events.py

all_test_usage
Skipped
---
test_usage.py

all_test_vm_autoscaling
Skipped
---
test_vm_autoscaling.py

all_test_vm_deployment_planner
Skipped
---
test_vm_deployment_planner.py

all_test_vm_life_cycle
Skipped
---
test_vm_life_cycle.py

all_test_vm_lifecycle_unmanage_import
Skipped
---
test_vm_lifecycle_unmanage_import.py

all_test_vm_schedule
Skipped
---
test_vm_schedule.py

all_test_vm_snapshot_kvm
Skipped
---
test_vm_snapshot_kvm.py

all_test_vm_snapshots
Skipped
---
test_vm_snapshots.py

all_test_vnf_templates
Skipped
---
test_vnf_templates.py

all_test_volumes
Skipped
---
test_volumes.py

all_test_vpc_ipv6
Skipped
---
test_vpc_ipv6.py

all_test_vpc_redundant
Skipped
---
test_vpc_redundant.py

all_test_vpc_router_nics
Skipped
---
test_vpc_router_nics.py

all_test_vpc_vpn
Skipped
---
test_vpc_vpn.py

all_test_host_maintenance
Skipped
---
test_host_maintenance.py

all_test_hostha_kvm
Skipped
---
test_hostha_kvm.py

@blueorangutan test rocky8 vmware-70u3

@weizhouapache a [SL] Trillian-Jenkins test job (rocky8 mgmt + vmware-70u3) has been kicked to run smoke tests

@weizhouapache that is strange. Your results differ a lot from mine. I review the changes in #8241, but, from what I understand, they should not impact this behavior. I will repeat my tests, using the most recent community version, this PR of mine and #8241, and I will return an opinion. Unfortunately, this may take some time as I am without my VMware environment at the moment.

Hi @SadiJr Any update of the tests, Is it good for 4.19.1? cc @weizhouapache

@weizhouapache that is strange. Your results differ a lot from mine. I review the changes in #8241, but, from what I understand, they should not impact this behavior. I will repeat my tests, using the most recent community version, this PR of mine and #8241, and I will return an opinion. Unfortunately, this may take some time as I am without my VMware environment at the moment.

Hi @SadiJr Any update of the tests, Is it good for 4.19.1? cc @weizhouapache

@sureshanaparti I'll try to repeat the described tests using the community version and, if I see that the described problem remains, I will test this PR to see if it still fixes the issue after #8241 was merged.

@weizhouapache that is strange. Your results differ a lot from mine. I review the changes in #8241, but, from what I understand, they should not impact this behavior. I will repeat my tests, using the most recent community version, this PR of mine and #8241, and I will return an opinion. Unfortunately, this may take some time as I am without my VMware environment at the moment.

Hi @SadiJr Any update of the tests, Is it good for 4.19.1? cc @weizhouapache

@sureshanaparti I'll try to repeat the described tests using the community version and, if I see that the described problem remains, I will test this PR to see if it still fixes the issue after #8241 was merged.

cool, looking forward to your testing results @JoaoJandre

[SF] Trillian test result (tid-10580)
Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server r8
Total time taken: 74679 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7239-t10580-vmware-70u3.zip
Smoke tests completed. 128 look OK, 3 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_restore_vm
Error
3621.09
test_restore_vm.py

test_02_restore_vm_allocated_root
Error
3619.90
test_restore_vm.py

test_01_deploy_vm_on_specific_host
Error
21.89
test_vm_deployment_planner.py

test_02_deploy_vm_on_specific_cluster
Error
3602.87
test_vm_deployment_planner.py

test_03_deploy_vm_on_specific_pod
Error
4.45
test_vm_deployment_planner.py

test_04_deploy_vm_on_host_override_pod_and_cluster
Error
15.78
test_vm_deployment_planner.py

test_05_deploy_vm_on_cluster_override_pod
Error
3605.89
test_vm_deployment_planner.py

test_09_expunge_vm
Failure
424.64
test_vm_life_cycle.py

@blueorangutan test rocky8 vmware-80

@weizhouapache a [SL] Trillian-Jenkins test job (rocky8 mgmt + vmware-80) has been kicked to run smoke tests

[SF] Trillian test result (tid-10610)
Environment: vmware-80 (x2), Advanced Networking with Mgmt server r8
Total time taken: 52461 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7239-t10610-vmware-80.zip
Smoke tests completed. 130 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL
Error
732.48
test_vpc_redundant.py

@weizhouapache that is strange. Your results differ a lot from mine. I review the changes in #8241, but, from what I understand, they should not impact this behavior. I will repeat my tests, using the most recent community version, this PR of mine and #8241, and I will return an opinion. Unfortunately, this may take some time as I am without my VMware environment at the moment.

Hi @SadiJr Any update of the tests, Is it good for 4.19.1? cc @weizhouapache

@sureshanaparti I'll try to repeat the described tests using the community version and, if I see that the described problem remains, I will test this PR to see if it still fixes the issue after #8241 was merged.

cool, looking forward to your testing results @JoaoJandre

Hi @JoaoJandre Can you confirm/share your test results please.

@weizhouapache that is strange. Your results differ a lot from mine. I review the changes in #8241, but, from what I understand, they should not impact this behavior. I will repeat my tests, using the most recent community version, this PR of mine and #8241, and I will return an opinion. Unfortunately, this may take some time as I am without my VMware environment at the moment.

Hi @SadiJr Any update of the tests, Is it good for 4.19.1? cc @weizhouapache

@sureshanaparti I'll try to repeat the described tests using the community version and, if I see that the described problem remains, I will test this PR to see if it still fixes the issue after #8241 was merged.

cool, looking forward to your testing results @JoaoJandre

Hi @JoaoJandre Can you confirm/share your test results please.

Hey @sureshanaparti, @weizhouapache and @DaanHoogland
After testing, I was unable to reproduce the issue on <IP_ADDRESS>. It was probably fixed in #8241 or somewhere else; therefore, I'll be closing this PR.

Hey @sureshanaparti, @weizhouapache and @DaanHoogland
After testing, I was unable to reproduce the issue on <IP_ADDRESS>. It was probably fixed in #8241 or somewhere else; therefore, I'll be closing this PR.

@JoaoJandre
Thanks for your update


Merge pull request #3 from danieleschmidt/main

Demo pull request and merge

We successfully did our first code review!


Add MintCeloScheduleWrapper

Description
This PR adds MintCeloScheduleWrapper
Other changes
There are some changes mostly related to the test setup:

time-sensitive tests should be now deterministic (due to a fixed timestamp)
in contractkit there's a new utility method to activate MintCeloSchedule
due to the devchain update some data changed and snapshots had to be updated as well

Apart from this GoldTokenWrapper ABI is now taken from @celo/abis-12 because it's needed for the MintCeloSchedule setup.
Tested
Locally and on CI.
Backwards compatibility
It is backwards compatible.

PR-Codex overview
This PR introduces support for the MintGoldSchedule contract in the Celo network.
Detailed summary

Updated blockNumber in info-l2.test.ts
Added MintCeloScheduleWrapper in contractkit
Added MintGoldSchedule to kit.ts
Updated dependencies in package.json
Updated GoldTokenWrapper import in GoldTokenWrapper.ts
Added MintCeloScheduleWrapper to contract-cache.ts
Added MintCeloSchedule functions in test-utils/utils.ts

The following files were skipped due to too many changes: packages/sdk/contractkit/src/wrappers/MintCeloScheduleWrapper.test.ts, yarn.lock, packages/sdk/contractkit/src/wrappers/FeeCurrencyDirectoryWrapper.test.ts, packages/sdk/contractkit/src/wrappers/MintCeloScheduleWrapper.ts, docs/sdk/contractkit/classes/wrappers_GoldTokenWrapper.GoldTokenWrapper.md, docs/sdk/contractkit/classes/web3_contract_cache.Web3ContractCache.md, packages/cli/src/commands/network/parameters-l2.test.ts, docs/sdk/contractkit/modules/base.md, packages/sdk/contractkit/src/web3-contract-cache.ts, docs/sdk/contractkit/classes/wrappers_EpochRewards.EpochRewardsWrapper.md, packages/sdk/contractkit/src/test-utils/utils.ts, docs/sdk/contractkit/enums/base.CeloContract.md, docs/sdk/contractkit/classes/kit.ContractKit.md, packages/cli/src/commands/network/__snapshots__/contracts-l2.test.ts.snap, docs/sdk/contractkit/classes/wrappers_MintCeloScheduleWrapper.MintCeloScheduleWrapper.md, docs/sdk/contractkit/classes/contract_cache.WrapperCache.md

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

thinking its best to wait for https://github.com/celo-org/celo-monorepo/pull/11040