Possibility of using with Koin

@KevinnZou
How to achieve being able to recreate the WebView instance ahead of usage through having it defined in a Koin single module?
Coming here from attempting to do this in low-level code on WKWebView and getting faced with multiple issues.
The target is to save the effort/time the platform takes to initialize the WebView and always maintains a ready-to-use instance.

@akiirax Thank you for your feedback! In the Android version of this library, I did provide a factory method that allows developers to use a custom WebView object. However, in situations where the library is being used across multiple platforms, it may not be feasible to provide this method because WebView instances differ across various platforms. Do you have any suggestions on how to handle this?

@akiirax Thank you for your feedback! In the Android version of this library, I did provide a factory method that allows developers to use a custom WebView object. However, in situations where the library is being used across multiple platforms, it may not be feasible to provide this method because WebView instances differ across various platforms. Do you have any suggestions on how to handle this?

@KevinnZou in iOS for example, I guess the possibility of defining the WKWebView in the .swift part of the project would make sense and then using it afterwards down the road.
Not sure if it's possible to do from the commonMain tho.

@cayolblake It's easy to implement for a single platform, but seems to be impossible in CommonMain because platforms have different WebView objects and no common interface. You are welcome to submit a pull request if you have any ideas. Thank you in advance!

@KevinnZou I goes implementing for platform modules rather than common will be great, if possible. It only makes sense also because that’ll be considered early enough to be instantiated even synchronously and use/inject it later.


Feature Request: Event Recurrence Support 'BYSETPOS'

Is your feature request related to a problem? Please describe.
I need a way to set a recurring event on the last/first/second/third/etc month of the week.
Describe the solution you'd like
The currently supported list of rules is described here: https://schedule-x.dev/docs/calendar/plugins/recurrence
I'd like to have the rrule 'BYSETPOS' supported.
Additional context
https://datatracker.ietf.org/doc/html/rfc5545#section-3.3.10 'BYSETPOS' is described here

Thanks for opening this issue.
I think SETPOS will be one of those rules that can be safely implemented as is and be compatible with drag & drop.


Adding tests

Issue #, if available:
Description of changes:
Adding additional unit tests to the repo
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #37 (1cf0f5d) into develop (2e91793) will increase coverage by 3.91%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop      #37      +/-   ##
===========================================
+ Coverage    35.37%   39.28%   +3.91%
===========================================
  Files          239      239
  Lines         8278     8278
  Branches       849      849
===========================================
+ Hits          2928     3252     +324
+ Misses        5162     4796     -366
- Partials       188      230      +42

see 19 files with indirect coverage changes


Release 0.79.4, via point-release

Tooling release
In order to accommodate reindexing workflows, we should release a patch for the 0.79.x series, adding all applicable ABCI events. Doing so will enable node operators to reindex chain state across upgrade boundaries. See tooling in https://github.com/penumbra-zone/reindexer for context.
Changes to include
Specifically, we want any added ABCI event emissions that were not present in the 0.79.x release series. No other changes should be included.

[x] https://github.com/penumbra-zone/penumbra/pull/4755 (shipped in v0.79.3)
[ ] https://github.com/penumbra-zone/penumbra/pull/4779
[ ] Links to specific PRs TK

Compatibility
The point release should be compatible with the pre-existing 0.79.3 release. It should not contain any consensus-breaking changes from the 0.80.x series, only new events not already included in 0.79.3. Additionally, we should take care to mark the Github release as not latest, to ensure that node operators do not mistakenly retrieve it when trying to get set up, or upgrade.

Discussed with @cronokirby out of band, and only https://github.com/penumbra-zone/penumbra/pull/4779 is required for backwards compat. That's what we'll release as 0.79.4.

Shipped: https://github.com/penumbra-zone/penumbra/releases/tag/v0.79.4


Incorrect month is opening when tapping Oct2023 statement in OldDot app

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: needs reproduction (don't have statement)
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @cead22
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1698869124487529
Action Performed:

Go to OldDot and login
Tap on Oct 2023 statement

Expected Result:
Statement should open in NewDot with correct header and translation. Link shou
Actual Result:

Tapping on the link to the Oct 2023 statement in old app opens new dot, but with a header Noviembre 2023 Statement
The translation is wrong, it should be  Estado de cuenta de Octubre 2023
Opening the statement in NewDot opens a new safari window and lands in sign in page

Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f250a418a4564fbf
Upwork Job ID:<PHONE_NUMBER>365579776
Last Price Increase: 2023-11-02

I'm not sure if this can be external so going to apply internal for now.

hey @mananjadhav can this be external? Let me know your thoughts!

@Christinadobrzyn Catching up, I'll test this by tomorrow.

I just checked this. Due to the lack of context from OldDot, I am not sure how do I reproduce this. @Christinadobrzyn are you able to reproduce this?
@m-natarajan Can you please elaborate the steps to reproduce and also post a video instead of screenshot?

Tap on Oct 2023 statement

I don't know what we mean by this.
and do I need to open Expensify in the iOS Native? to test this?  I also followed through the slack conversation but no info there.

No, I was not able to reproduce this, sorry I pushed it forward because it was reported by internal engineers. I'll reach out to QA in Slack to see if they can test again and update the OP steps so I can get a better idea of what is happening. Asking QA for some help - https://expensify.slack.com/archives/C9YU7BX5M/p1699384867315839

@cead22 can you pls help us out on this issue? we do not have access to statements to verify reproduction again.

How can I help?
This only applies to accounts that have expensify cards. If you have one and have received a statement, you should be able to navigate to https://new.expensify.com/statements/202310 to see the October statement (hence the 202310 in the URL), and you'll see November shows on the header

So it sounds like this should be internal since it relates to OldDot and Expensify Cards. I'll add an engineering label to see if we can get someone internal to work on this.

Oh no, @cead22 it assigned to you 😞 Let me know if you want me to try and find another engineer to work on this.

If you can, please do

Reached out to see if other engineers would be able to grab this - https://expensify.slack.com/archives/C01SKUP7QR0/p1699635102177169

asking the team again - https://expensify.slack.com/archives/C01SKUP7QR0/p1699909684516149

Hi @hayata-suenaga would you be able to help us with this fix?

hi @Christinadobrzyn
I can try to move the issue forward. To begin with, could you tell me how I can find my Expensify card statements on OldDot?

Hi! Thank you! I'm not getting statements so I'm having a hard time nailing down how to reproduce this. I think they are related to Expensify Card purchases.
I wonder if @Gonals might know more about this based on https://github.com/Expensify/Expensify/issues/234459

@Gonals ended up not working on https://github.com/Expensify/Expensify/issues/234459 so they are not familiar with statements.
I'm not sure what to do here as I am not getting statements. Posting to the team to see if anyone can take this forever or at least while I'm ooo until Nov 27th - https://expensify.slack.com/archives/C01SKUP7QR0/p1700265001449449

I'm going to assign someone else from the BZ team to see if they might be able to help with guidance on what to do next to reproduce this.

I gonna so ask the engineering team if anyone is familiar with the statement generation logic.

I clicked https://new.expensify.com/statements/202310 and it shows November in the heading. That's the bug right?

I clicked https://new.expensify.com/statements/202310 and I see November 2023 in the heading.
That's the bug, right?

These are generated internally and we grab the month from the URL -

I clicked new.expensify.com/statements/202310 and I see November 2023 in the heading.

This does seem like the bug. I'd first check this locally and seeing what happens here

will work on this next week.
Is it okay we change the priority to weekly?

Doesn't seem as high priority as some of the other stuff going on. Let's put daily back on next week

gonna go back to the Slack convo and see what I need to do

PR is up and under review.

🎉

prepping for payment!
Payouts due:
Issue Reporter: NA
Contributor: NA
Contributor+: $500 @mananjadhav (paid through NewExpensify)
Eligible for 50% #urgency bonus? N
Upwork job is NA
@mananjadhav do we need a regression test for this?

I don't think we do, but I would still want to confirm with @hayata-suenaga. This was fixed internally and I reviewed the code.

I also don't think we need a regression test

Awesome! I think this can be closed out but @mananjadhav please make sure to request payment through NewExpensify.

$500 payment approved for @mananjadhav based on this summary.


Minor changes to code style

Some minor changes to code style in file 'predict_first_ntree.py'.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2806   +/-   ##
=========================================
  Coverage     43.11%   43.11%
  Complexity      200      200
=========================================
  Files           151      151
  Lines         11509    11509
  Branches       1166     1166
=========================================
  Hits           4962     4962
  Misses         6216     6216
  Partials        331      331

Continue to review full report at Codecov.

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


Fix #10524: Add suport for override entry points in tsconfig.json

Fixes https://github.com/DefinitelyTyped/DefinitelyTyped/issues/10524
flux has two entry points, flux\index.d.ts and flux\utils\index.d.ts; this adds support for loading these and publishing them.
the change only impacts 3 packages, flux, angular-mocks, and history. all three are loading additional files that should be loaded.

//cc: @andy-ms

Closing in favor of #136


Expose handleAuthenticationResponse()

I'd like to expose the handleAuthenticationResponse function. We are unable to rely on MSAL to consistently use the redirect id_token to complete the login flow in our Angular 5 app.
Exposing this functions lets us bootstrap the MSAL library in our custom auth service when the app starts. We've been doing this in production for over a month now and have seen it consistently work.
I'd appreciate if this change made it in. Thanks!
Edit: Note that I did not run the dist build for this, only TS changes.

We are making API changes to the core library which will address some of this issue. We will track this for the Angular improvements planned next.


Exception - didnt manage to figure it out

TypeError: Cannot read property '__id' of undefined
at sourceFingerprint (calendar.js:60)
at

It is showing error in calendar.js file. I think this is not related to the plugin.


Possible values for command in mb_param_request_t type were not defined or included. (IDFGH-9305)

I found out that <MB_FUNC_> were defined in modbus/include/mbproto.h but is not accesible. I think that list should be in esp_modbus_master.h or esp_modbus_common.h where <mb_param_request_t> is.
https://github.com/espressif/esp-modbus/blob/3f6537fc940c932884db43c9d62608e982461a70/freemodbus/common/include/esp_modbus_master.h#L102

@mrcamilletti,
I can confirm this issue as minor. The Modbus controller code was designed to be isolated from freemodbus. This is not good idea to move these defines to esp_common.h but agree this can be useful there. Propose the change as below:
mbproto.h :
typedef enum
{
    MB_FUNC_NONE =                        (  0 ),
    MB_FUNC_READ_COILS =                  (  1 ),
    MB_FUNC_READ_DISCRETE_INPUTS =        (  2 ),
    MB_FUNC_WRITE_SINGLE_COIL =           (  5 ),
    MB_FUNC_WRITE_MULTIPLE_COILS =        ( 15 ),
    MB_FUNC_READ_HOLDING_REGISTER =       (  3 ),
    MB_FUNC_READ_INPUT_REGISTER =         (  4 ),
    MB_FUNC_WRITE_REGISTER =              (  6 ),
    MB_FUNC_WRITE_MULTIPLE_REGISTERS =    ( 16 ),
    MB_FUNC_READWRITE_MULTIPLE_REGISTERS =( 23 ),
    MB_FUNC_DIAG_READ_EXCEPTION =         (  7 ),
    MB_FUNC_DIAG_DIAGNOSTIC =             (  8 ),
    MB_FUNC_DIAG_GET_COM_EVENT_CNT =      ( 11 ),
    MB_FUNC_DIAG_GET_COM_EVENT_LOG =      ( 12 ),
    MB_FUNC_OTHER_REPORT_SLAVEID =        ( 17 ),
    MB_FUNC_ERROR =                       ( 128u )
} eMBCommands;

esp_modbus_master.h

/**
 * @brief Modbus supported commands to share with the mbc_master_send_request()
 */
typedef eMBCommands mb_function_codes_t;

/**
 * @brief Modbus register request type structure
 */
typedef struct {
    uint8_t slave_addr;                          /*!< Modbus slave address */
    mb_function_codes_t command;    /*!< Modbus command to send */
    uint16_t reg_start;                           /*!< Modbus start register */
    uint16_t reg_size;                            /*!< Modbus number of registers */
} mb_param_request_t;

Hi @alisitsyn ok understood, now I get why it was defined elsewhere. I agree with the changes to avoid redefining the commands table, or possible errors due to bad assignments. Thanks!


RipMe: Not working anymore (with DeviantArt)?

Tried with the latest version both of java and firefox. With a DeviantArt page URL, I got this message:
'Can´t rip this URL: com.rarchives.ripme.ripper.rippers (null) does not appear to be a valid package' Can you help me? (RipMe v1.1.8)

my guess is that's a problem on your end. I even managed to download the whole gallery without any problems

Win8.1 64bit maybe? Seems like you have Win7

Downloaded the .jar again. working now. Didn´t change anything, weird.

Looks like there's no action to take here. Closing.


BroxDir missing

I tried to download the BroxDir file by clicking the link you provided, but It doesn't seem to work.
The link I am talking about is into the ./generate_date folder.
Could you please fix it?
Thank you in advance.

Following the comment from @Gio-97, would it be possible to upload the "BroxDir" file from the OF folder? Unfortunately the link provided in the README file BroxDir seems to be broken. Maybe they can also be uploaded to the same URL as the AUNets weights.
Thanks for the sharing the code and the weights for the pretrained model.

Hi
If it can help you, Brox dense optical flow is included in opencv (with cuda contribution module). It was necessary to compile OpenCV with this contrib.
I adapted the code found in AUNets to this library: It seems i got the same results
main.cpp.txt

Thanks @thiravet for the adapted version of the code. Indeed, after reinstalling OpenCV with CUDA support I managed to compile and run the code.
I just had to do a couple of modifications:

Use "experimental/filesystem" instead of "filesystem"
Add a vector sort statement at the end of the getFileNamesFromDirAsVector function. This ensures the frames are correctly returned to the OF logic, as in some runs I had a mix of frames (i.e. calculating the OF between the last and first frames).

Now I am able to run AUNets correctly. :)


2021 Functional Goals

The purpose here is to create a document that we can use as a way to agree on the scope of work for 2021, and specifically what functionality to we want to be able to demonstrate by the end of the year.
We may use this folder as a place to store evolving use cases as well as we develop them.
At the moment, we are starting with some very high level goals from NCATS, but through discussion on this PR and the call we should get more specific.

So I think I heard on yesterday's Architecture call that the scope is moving away from drug repurposing and towards target landscaping.   @southalln @sstemann @cartmanbeck is that right?   Can we flesh out that part of this a bit?

That is correct, and I will let Noel talk more about what that really means, but in my mind, it's simply a broadening of the same question... not only what drugs can treat a disease, but what targets COULD a drug to treat the disease be designed to target? It broadens the scope, but fundamentally is getting at a similar idea.

It's certainly getting at a similar idea / biology / set of queries, but I think we need to be as clear as we can on what the specific goals are - Is target landscaping the goal, or target landscaping + drug repurposing is the goal, or target landscaping is the goal and drug repurposing is a stretch goal, or some other formulation?

@cartmanbeck @cbizon - in the clinical working group we've been thinking in a drug repurposing direction.
@qianzhu2018 @karafecho @CaseyTa how do you read the conversation above relative to our work? Do we need to change direction or do the kinds of questions we've been considering fit comfortably within @cartmanbeck's "broadened" question?

Maybe we should be thinking about this in terms of what kind of graph/answer set we need to assemble to allow a user to ask questions about diseases and treatments. That gives us a concrete goal to shoot for and allows NCATS to have some degree of flexibility in exactly what question they ask during a demo.

tagging the committee @NCATSTranslator/architecture-core for follow up

Just for clarity – Target Landscape was introduced at the same time as drug repurposing during the Translator Trajectory presentationhttps://drive.google.com/file/d/1gVSsgwho2F2xt7t2w_2r2NEwlUERKUik/view?usp=sharing (slide 2) which I’ve also included in the PM Repohttps://github.com/NCATSTranslator/TranslatorPM/blob/main/How to:  Announcements.md at the Relay on Monday and Friday.  While these two are related, I wouldn’t say we’ve broadened the scope but are just wanting to make sure that both are approaches scoped for the December demo (and product launch).
From: cbizon<EMAIL_ADDRESS>Sent: Wednesday, February 24, 2021 2:33 PM
To: NCATSTranslator/TranslatorArchitecture<EMAIL_ADDRESS>Cc: Stemann, Sarah (NIH/NCATS) [C]<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [NCATSTranslator/TranslatorArchitecture] 2021 Functional Goals (#29)
So I think I heard on yesterday's Architecture call that the scope is moving away from drug repurposing and towards target landscaping. @southallnhttps://github.com/southalln @sstemannhttps://github.com/sstemann @cartmanbeckhttps://github.com/cartmanbeck is that right? Can we flesh out that part of this a bit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/NCATSTranslator/TranslatorArchitecture/pull/29#issuecomment-785320499, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJEL5IWFX4ST2TNA3X42JMLTAVH6ZANCNFSM4YBLA6ZA.

@sstemann : Thanks for the clarification. I do have a question, however. Specifically, I'm wondering how to interpret the third bullet "[redefining disease]". This question has been raised in various discussions, so I want to make sure I understand the intent. Thanks!

@sstemann : Thanks for the clarification. I do have a question, however. Specifically, I'm wondering how to interpret the third bullet "[redefining disease]". This question has been raised in various discussions, so I want to make sure I understand the intent. Thanks!

This is something that has been part of the hopes for Translator since its inception, the idea being that we would use data to define diseases not based on similar phenotypes but based on real genetic, pathway, and other mechanistic data. I hope we can get to that point with Translator, but it's a big ask, admittedly.

Indeed! 'Redefining disease' or 'data-driven clinical regrouping' is clearly part of the vision for Translator. It's just that the closed brackets confused me. :-)

Can we try to draft some actionable requirements to guide our development for Dec?
Let's try to nail down some specifics to support this.
I see two types of queries here: drug repurposing and target landscaping. Can we get for each 1. what kind of search term or input might we get from a user and 2. what kind of result would delight a user?
For example, for drug repurposing, will a use submit the name of a disease as input and expect as output a list of potential repurposed drug treatments?
We have use cases and requirements from previous phases of translator. Can we consult those?

I would like to raise some issues regarding the notion of  "drug repurposing" which we need to consider if we make this a major functionality. Clearly, was diatomsRcool describes is not very realistic (yet): "For example, for drug repurposing, will a use submit the name of a disease as input and expect as output a list of potential repurposed drug treatments?"
Drug repurposing starts with formulating a biological repurposing strategy  - only then can we formulate a meaningful query.   There are different classes of repurposing strategy - some examples below:
_
(1) EXPLOITING a TOXIC EFFECT  FOR TREATMENT:  "Drug A has side effect X (constipation) - and hence it could be used to treat disease Y (diarrhea). "   This strategy has led to many successful drugs in the past 30 years that I have been following. The Translator query would then simply be:  Which drug has side effect X that is the opposite of the symptom Y that I  want to treat?
(2) IDENTIFTCATION OF INTERSECTS IN THE MOLECULAR PATHOGENESIS PATHWAY:  "In disease disease X, protein kinase K is over active - and there are so many kinase inhibitors looking for indications  - is there one that inhibits K"? This strategy also has been pretty successful, but is more recent.
(3) IDENTIFICATION OF OVERLAPPING PATHOPHYSIOLOGY:   ....  (this is an older strategy...)
The point is: the user will first have to come up with the repurposing strategy, and formulate the query. And only then perform the query. The Translator cannot do the former for you

I'm confused about process here. Typically, a PR discussion is more narrowly about whether to accept or reject the content of the pull request and the discussion thread is buried after a decision is made. The present question is whether we should add this stub document to the arch repository. However this discussion includes a lot of proposals for what could be future PRs that would add very important specificity to the stub document. That discussion is very important, and should be happening somewhere, but probably not in a PR discussion thread? Moreover, I love the idea that we "fully flesh out" the user stories listed here, and that work should probably be assigned to someone, but it's tough to do that in a PR discussion thread and hold anyone accountable. What should the process for all this be?

I think you make a good point @southalln .   I think we need to do a bit of work to define processes of the kind you discuss, and bring together some of the streams of requirements.  But this PR is probably not a good place for all of that discussion.
So if we can refocus on this particular stub, what I think we need is:

Removing "redefining disease" as a target for December
An explanation of what is meant by the two query types called out
If possible, adding detail to the Results section: can we say more than we'll know it when we see it?
Add a link to whatever process/repos we decide on for tracking the use-case fleshing out discussions.


Parsing String 'null'

How to parse key=null?
Want to get null not 'null'.

Currently not possible, but pull request are welcome and you also become a collaborator member of the project if you contribute 👍

It is possible with a custom function:
var qs = new MongoQS({
custom: {
myParam: function(query, input) {
query.myParam = input === 'null' ? null : input;
}
}
});

ok, thx!


Inquiries about translated file locations

Hello, Anyscale.
Ray is a popular library in Korea, too. So I will contribute by translating Korean users so that they can easily study through this tutorial. Where should I put the translated notebook file?
Thank you!

Thanks for offering to do a translation. We are making many changes now. A translation might be out of date quickly. I recommend waiting another month or two, when the first complete version is finished.
Does it make sense to the translated files under a kr or korean directory? Other translations could use their own directory. I can move the original files under english.

Thanks!


BS4 Compatibility - Multi Checkbox Control

Instead of waiting till I can get everything working, I thought I'd push things out incrementally per control, this has been on the back burner for way too long already :/
Nested inputs for multi option controls required the changes in #242 to be altered a bit, in order to avoid duplicate inputs, and to retain the ability the disable labels per option configuration - hopefully this won't come back to bite me with the remaining controls.
Errors aren't displayed properly yet, but this is also true for other controls, so I'm going to work on that later on when all controls are rendering the correct markup.
For reference

https://getbootstrap.com/docs/4.1/components/forms/#horizontal-form
https://getbootstrap.com/docs/4.1/components/forms/#inline
#223

in order to avoid duplicate inputs

I thought I had taken care of duplicates, guess not.
Reading test cases for generated HTML is a PITA. We need a demo app to see the generated output :slightly_smiling_face:.

Yeah, it's not pretty :) I have a simple app that I'm using for visualization and markup inspection purposes, I'll upload it later today.

Here you go, it's very basic, it simply lists a bunch of controls per alignment, and shows the code alongside of it. The code formatting validates (and unfortunately fixes) the HTML too, and will output warnings accordingly, the original code can be viewed in the "Raw" tabs.
https://github.com/ndm2/bootstrap-ui-test-app

Nice, I checked with your demo app and the display looks good to me.
Just for /pages/default-align#select-checkbox-inline-anchor the input tags end up with inline="1" so the inline key needs to be removed/ignored at appropriate location.

Good spot, didn't noticed that, I've completely forgot to add tests for the inline option initially.
If you're OK with the markup, then I'm going to finish up the tests.

@ndm2 Anything more to be done here?

I think we're good for now. Groups in inline forms could need some additional work, but I'd save that for later, as it's really some freaky edge case.

Great, thanks.


FOOTER IS COVERING CREATE USER PAGE

Footer covers the whole page not sure why.

never mind its not hitting the route


2 small changes to data  screening function

added code to remove estimated as well as provisional data and append site number to data screening output

closes #15


bug: telescope lsp keys are not overwritable

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.10.0
Operating system/version
MacOs 14.5
Describe the bug
According to https://www.lazyvim.org/plugins/lsp#-customizing-lsp-keymaps
I can remap lsp keys using lazyvim api to get lsp keymaps
I am trying to setup glance.nvim for my lsp keymaps
  {
    "dnlhc/glance.nvim",
    opts = function()
      local keys = require("lazyvim.plugins.lsp.keymaps").get()

      keys[#keys + 1] = { "gd", false }
      keys[#keys + 1] = { "gr", false }
      keys[#keys + 1] = { "gy", false }
      keys[#keys + 1] = { "gI", false }

      return {
        border = {
          enable = true,
        },
        use_trouble_qf = true,
      }
    end,
    cmd = { "Glance" },
    keys = {
      { "gd", "<CMD>Glance definitions<CR>", desc = "Goto Definition" },
      { "gr", "<CMD>Glance references<CR>", desc = "References" },
      { "gy", "<CMD>Glance type_definitions<CR>", desc = "Goto t[y]pe definitions" },
      { "gI", "<CMD>Glance implementations<CR>", desc = "Goto implementations" },
    },
  },

But this gets overwritten by telescope extras that is by default enabled
https://github.com/LazyVim/LazyVim/blob/00f53315b59fd35ef05ea74e66423944a2012974/lua/lazyvim/plugins/extras/editor/telescope.lua#L267
So I am stuck
Steps To Reproduce

Expected Behavior
Should allow overwriting lsp keys from builtin extras
Repro
No response

The docs say to put those commands in nvim-lspconfig spec. Not glance.nvim spec. There's a difference.

You indeed need to disable those keymaps in the lspconfig spec.
Your custom plugins are parsed after all LazyVim specs, so the telescope one should not override your specs.

my bad, thanks


Force renewale of token

Hello,
This is relative to another issue where I asked that here => #17
Hello @rdelrosario
Thank you for your answer. So I've tried .Reset(), it works, but it does not seems to trigger a renew for the token. It is empty when I log in again after calling this method. Thought that by resetting token it will force firebase to generates a new one ... but what I've got is just "".
Thanks.

After reset you should get the token here:

  CrossFirebasePushNotification.Current.OnTokenRefresh += (s,p) =>
  {
        System.Diagnostics.Debug.WriteLine($"TOKEN : {p.Token}");
  };

You are not getting the token there?

this is not working for me either, it resets fine but does not fire the event to referesh. Also how do you reset on iOS?


convert integers to seconds before calling ago.

This commit converts integers to seconds before calling ago.  I also
removed the default values from the method signature because the method
is never called with default values, so we don't have to think about
what it means to call the method with less than 3 parameters

:shipit:


Issue compiling grpc from source - _gRPC_CPP_PLUGIN-NOTFOUND

What version of gRPC and what language are you using?
1.59.1
What operating system (Linux, Windows,...) and version?
Linux (Red Hat)
What runtime / compiler are you using (e.g. python version or version of gcc)
gcc (ver 7.3.1-17)
What did you do?
I am trying to use grpc in my C++ project and using Linux (Red Hat). I cloned the latest grpc (1.59.1) into my C++ project under third-party/grpc folder. In my projects root CMakeLists.txt file, I added the line add_subdirectory(third-party/grpc).
When I compile my project, grpc source code compilation fails at the very end with the below error -
gmake[2]: *** No rule to make target third-party/grpc/_gRPC_CPP_PLUGIN-NOTFOUND', needed by gens/src/proto/grpc/reflection/v1alpha/reflection.grpc.pb.cc'.  Stop. gmake[2]: *** No rule to make target third-party/grpc/_gRPC_CPP_PLUGIN-NOTFOUND', needed by gens/src/proto/grpc/channelz/channelz.grpc.pb.cc'gmake[1]: .  Stop. *** [third-party/grpc/CMakeFiles/grpc++_reflection.dir/all] Error 2 gmake[1]: *** Waiting for unfinished jobs.... gmake[1]: *** [third-party/grpc/CMakeFiles/grpcpp_channelz.dir/all] Error 2 Consolidate compiler generated dependencies of target grpc++_error_details Consolidate compiler generated dependencies of target grpc++_alts [100%] Linking CXX shared library ../../app-bin/lib/x86_64/libgrpc++_error_details.so [100%] Linking CXX shared library ../../app-bin/lib/x86_64/libgrpc++_alts.so [100%] Built target grpc++_error_details [100%] Built target grpc++_alts gmake: *** [all] Error 2
What did you expect to see?
Successful compilation of grpc
What did you see instead?
Compilation failed
Anything else we should know about your project / environment?
No

Am I missing grpc C++ plugin? How can I get it if the source is not automatically compiling it?

i have the same issue too
any solution?

Would you provide a small repo? Note that we have https://github.com/grpc/grpc/blob/master/test/distrib/cpp/run_distrib_test_cmake_as_submodule.sh that tests this use-case so you can take a look at this to figure out what's difference.


Create Test Automation for Azure.Identity running in Azure Hosted Environments

Epic: Create Test Automation for Azure.Identity running in Azure Hosted Environments

[x] Azure Cloud Shell

[x] /sdk/e2e test assets checked-in so that they can be referenced from Vendor Instructions
[x] Language-specific instructions added to https://github.com/Azure/azure-sdk/issues/795

[x] Azure VM

[x] /sdk/e2e test assets checked-in so that they can be referenced from Vendor Instructions
[x] Language-specific instructions added to https://github.com/Azure/azure-sdk/issues/815

[ ] Azure Functions

[ ] /sdk/e2e test assets checked-in so that they can be referenced from Vendor Instructions
[ ] Language-specific instructions added to https://github.com/Azure/azure-sdk/issues/899

[ ] Azure Kubernetes

[ ] /sdk/e2e test assets checked-in so that they can be referenced from Vendor Instructions
[x] Language-specific instructions added to https://github.com/Azure/azure-sdk/issues/814

[ ] Azure App Services

[ ] /sdk/e2e test assets checked-in so that they can be referenced from Vendor Instructions
[ ] Language-specific instructions added to https://github.com/Azure/azure-sdk/issues/900

[ ] Azure WebJobs

[ ] /sdk/e2e test assets checked-in so that they can be referenced from Vendor Instructions
[ ] Language-specific instructions added to https://github.com/Azure/azure-sdk/issues/902

Updating issue as Functions and App Services has been moved to the backlog

Done, test assets checked in under sdk/samples/manual-tests/


Provide file sr15_specs.yaml in repository

This file is needed by many notebooks. It can be built from the "categories_indictors" notebook, but it would be better to save users the effort.

thanks @nworbmot for that suggestion!


Search Image doesn't work

Steps to reproduce
Search Images not work and returned []

Failure Logs
[]

Expected behavior
.
Current behavior
.
Version
Default
Anything else?
No response
Checklist

[X] I am running the latest version.
[X] I checked the documentation and found no answer.
[X] I have searched the existing issues and made sure this is not a duplicate.
[X] I have provided sufficient information.

Seems to be a similar problem to #41. Will look into it today.
Thanks for reporting!


unable to set current index

When reload data, we usually want to set an first display index.
seems like your code does not include that feature

Yes, I can add this feature in the next version, your feedback is great, thanks


If the ideology column is displayed the mod breaks when the colony has a baby

You can see it gives up right when it reaches Parsons' ideology. I assume this is because the baby has no ideology, and the mod was written with the (safe before version 1.4) assumption a pawn will always have an ideology.
None of the mod buttons respond at all, meaning the column cannot be removed.

I love this bug report, thank you for the laugh.
I'll get right on that.

Fixed.


feat(robot-server): add instrument firmware update and monitoring capability

Closes RCORE-482, RCORE-483
Overview
Adds endpoints to the robot-server to facilitate updating instruments' firmware. These endpoints will be primarily used to update instruments that get attached after a robot has finished auto-updating firmware of the instruments attached during a robot update.
The work for this feature started in a different branch whose code was reviewed here if you want to take a look. Had to create a separate PR for untangling the multiple merge conflicts and for git tree readability.
Test Plan
Test updating OT3 instruments and fetching the update status
NOTE- you will need to temporarily modify the robot environment manually to make it so that it thinks there's an update available.-

Send a POST request to /instruments/updates endpoint with the mount specified in the body as

{"mount": <mount name>}

Fetch update status by sending GET request to /instruments/updates/{:update_id} where update_id is a unique ID generated by the POST update request.
Test that the following conditions make the update to fail with an appropriate error-

No instrument attached on the mount
Initiating an update while that mount is already updating
Initiating an update when there's no new firmware version available
Trying to fetch update status using a non-existent update ID

Check how long does it take for an update request to respond (useful for testing the timeout we can use for waiting for the update status)

Anything else you can think about.
Changelog

added the above two endpoints to update & fetch instruments.
added an UpdateProgressMonitor class to keep track of the update processes that were started

Review requests

Please go through the code and check if it makes sense
Check that I have covered all test cases
Check that inline comments/ todos make sense
Test it on a robot! Bonus points if you could take screenshots of the responses so we can use them while wiring up the UI components.

Risk assessment
Low for the code in edge. But could be detrimental to an instrument itself if an update process fails mid-way.

Superceded by #12504


Small typo on the README.md

Thanks for your keen observation!

Thanks for your keen observation!


Alex/struct checker apr15

Latest update of struchk port

@greglandrum @AlexanderSavelyev
I've added some poor-man's parity checking here.  Instead of using parity, I just use RDKit's mechanisms for finding unspecified centers.  I'm not really sure if this is the right thing to do, but it does make it more like avalon.
Adding the check inside SplitFragments with my data set has the following failures
+stereo_error - we detect and avalon doesn't
-stereo_error - avalon detects and we don't
(root) PHUSCA-L25338:testmols kellebr5$ ls -1 failures/+stereo_error/ | wc -l
    5582
(root) PHUSCA-L25338:testmols kellebr5$ ls -1 failures/-stereo_error/ | wc -l
      62

Adding the check inside CheckStereo for CCW/CW atoms
(root) PHUSCA-L25338:testmols kellebr5$ ls -1 failures/+stereo_error/ | wc -l
    1117
(root) PHUSCA-L25338:testmols kellebr5$ ls -1 failures/-stereo_error/ | wc -l
     225

which is on the right track, but we are obviously doing something different :)

I just looked at some random +stereo_error examples.

There are some 'three attachments colinearity violation' errors that avalon doesn't generate. I think we're doing the right thing for some of them
Substance_310925001_310950000-023908 has "stereobond to nonstereogenic" errors that are probably connected to a chiral Phosphorous
Substance_310925001_310950000-022574 generates "three attachments colinearity violation" and "colinearity or triangle rule". and that looks right.

So for at least some of these it seems like we are doing the right thing and Avalon is not, but it would still be good to understand why.
I'm not around this weekend and am busy most of the day Monday, but I can start to drill into these on Monday afternoon if it would help.

@AlexanderSavelyev : The augmented atom rules are order-dependent:
In [26]: m = Chem.MolFromSmiles('FC(F)O')

In [27]: checker.CheckMolStructure(m)
[07:32:08] * CheckAtom idx=0, F status=0 neighbours=1, C
[07:32:08] * CheckAtom idx=0 F status=0 neighbours=1
[07:32:08] RecMatch ret TRUE N,O,S,F,Cl,Br,I(-C,Si,Ge,Sn,Pb,T,D,H,N,P,As,Sb,Bi,O,S,Se,Te,Po)
[07:32:08] * CheckAtom idx=1, C status=0 neighbours=3, F, F, O
[07:32:08] * CheckAtom idx=1 C status=0 neighbours=3
[07:32:08] UNMATCHED atom idx=1 C status=0 nmatch=1
[07:32:08]     atom 1   AA : C(-F)(-F)(-O)
[07:32:08] * CheckAtom idx=2, F status=0 neighbours=1, C
[07:32:08] * CheckAtom idx=2 F status=0 neighbours=1
[07:32:08] RecMatch ret TRUE N,O,S,F,Cl,Br,I(-C,Si,Ge,Sn,Pb,T,D,H,N,P,As,Sb,Bi,O,S,Se,Te,Po)
[07:32:08] * CheckAtom idx=3, O status=0 neighbours=1, C
[07:32:08] * CheckAtom idx=3 O status=0 neighbours=1
[07:32:08] RecMatch ret TRUE N,O,S,F,Cl,Br,I(-C,Si,Ge,Sn,Pb,T,D,H,N,P,As,Sb,Bi,O,S,Se,Te,Po)
Out[27]: 32

In [28]: m = Chem.MolFromSmiles('OC(F)F')

In [29]: checker.CheckMolStructure(m)
[07:32:15] * CheckAtom idx=0, O status=0 neighbours=1, C
[07:32:15] * CheckAtom idx=0 O status=0 neighbours=1
[07:32:15] RecMatch ret TRUE N,O,S,F,Cl,Br,I(-C,Si,Ge,Sn,Pb,T,D,H,N,P,As,Sb,Bi,O,S,Se,Te,Po)
[07:32:15] * CheckAtom idx=1, C status=0 neighbours=3, O, F, F
[07:32:15] * CheckAtom idx=1 C status=0 neighbours=3
[07:32:15] RecMatch ret TRUE C(-C,N,P,O,S,Se,F,Cl,Br,I)(-C,N,P,O,S,F,Cl,Br,I)(-C,F,Cl,Br,I,D,T)
[07:32:15] * CheckAtom idx=2, F status=0 neighbours=1, C
[07:32:15] * CheckAtom idx=2 F status=0 neighbours=1
[07:32:15] RecMatch ret TRUE N,O,S,F,Cl,Br,I(-C,Si,Ge,Sn,Pb,T,D,H,N,P,As,Sb,Bi,O,S,Se,Te,Po)
[07:32:15] * CheckAtom idx=3, F status=0 neighbours=1, C
[07:32:15] * CheckAtom idx=3 F status=0 neighbours=1
[07:32:15] RecMatch ret TRUE N,O,S,F,Cl,Br,I(-C,Si,Ge,Sn,Pb,T,D,H,N,P,As,Sb,Bi,O,S,Se,Te,Po)
Out[29]: 0

Making it private is enough in this case I think.
thread apis should use a const StructureChecker object anyway which does the same thing at a higher level.

Brian Kelley

On Oct 13, 2016, at 9:59 AM, AlexanderSavelyev<EMAIL_ADDRESS>wrote:
@AlexanderSavelyev commented on this pull request.
In Code/GraphMol/StructChecker/StructChecker.h:

RECHARGED = 0x1000,

STEREO_TRANSFORMED = 0x2000,
TEMPLATE_TRANSFORMED = 0x4000,
TAUTOMER_TRANSFORMED = 0x8000,
// mask:
BAD_SET = (BAD_MOLECULE | ALIAS_CONVERSION_FAILED | STEREO_ERROR |

          STEREO_FORCED_BAD | ATOM_CLASH | ATOM_CHECK_FAILED |

          SIZE_CHECK_FAILED),

TRANSFORMED_SET = (TRANSFORMED | FRAGMENTS_FOUND | EITHER_WARNING |

                  DUBIOUS_STEREO_REMOVED | STEREO_TRANSFORMED |

                  TEMPLATE_TRANSFORMED | TAUTOMER_TRANSFORMED | RECHARGED),

} StructureFlags;
// attributes:
private:
const StructCheckerOptions Options;
Options are const in order to explicitly forbid changing it outside while execution (e.g. in threads). If you want to change options, please create another StructChecker instance

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@greglandrum Except for stereo checking, this passes all of the pubchem substance checks.  What are the next steps?

@bp-kelley I think we merge it and add a comment in the header that it's still beta and that the API isn't finalized
Justification: It's nice work, useful as is, and can be used to get a broader discussion started about what should be done next.

@greglandrum I'm going to fix the snprintf's on windows, it will take a bit.


Bump dependencies

Fixes #30 & #31

@phillipsj - any chance of this getting through soon?

I have stepped back from this project for a while and I thought there were
some commitments to maintain. I will incorporate any changes and do the
upgrades and get a version released this weekend.
On Tue, Jun 18, 2019 at 9:49 AM Dag H. Baardsen<EMAIL_ADDRESS>wrote:

@phillipsj https://github.com/phillipsj - any chance of this getting
through soon?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cake-contrib/Cake.XdtTransform/pull/32?email_source=notifications&email_token=AAL63BILYHLRTXLGYDWSJLDP3DRVXA5CNFSM4HHLBEK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX6VZXI#issuecomment-503143645,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAL63BIMBCX5Q6PQOVJTBCLP3DRVXANCNFSM4HHLBEKQ
.

@dag I have published version 0.17.0 with the upgrade to Cake 0.33.0.
https://www.nuget.org/packages/Cake.XdtTransform/
On Tue, Jun 18, 2019 at 9:53 PM Jamie Phillips<EMAIL_ADDRESS>wrote:

I have stepped back from this project for a while and I thought there were
some commitments to maintain. I will incorporate any changes and do the
upgrades and get a version released this weekend.
On Tue, Jun 18, 2019 at 9:49 AM Dag H. Baardsen<EMAIL_ADDRESS>wrote:

@phillipsj https://github.com/phillipsj - any chance of this getting
through soon?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cake-contrib/Cake.XdtTransform/pull/32?email_source=notifications&email_token=AAL63BILYHLRTXLGYDWSJLDP3DRVXA5CNFSM4HHLBEK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX6VZXI#issuecomment-503143645,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAL63BIMBCX5Q6PQOVJTBCLP3DRVXANCNFSM4HHLBEKQ
.

That's really cool - thank you @phillipsj !


Trending groups via SQL view

[Active Record] was built on the notion that SQL is neither dirty nor bad, just verbose in the trivial cases. The focus is on removing the need to deal with the verbosity in those trivial cases (writing a 10-attribute insert by hand will leave any programmer tired) but keeping the expressiveness around for the hard queries—the type SQL was created to deal with elegantly.

— DHH, Agile Web Development with Rails
This basically calculates the z-score via two SQL views, one of which is a "materialized view".  One view is used to generate a "histogram" of GroupMembers by created_at.  The other (materialized) view then uses this data to calculate the z-score.
The materialized view is updated every 5 minutes by a Sidetiq worker.
On display, we remove groups the current user is in already.

Looks good, maybe @cybrox can check SQL as well -- not my strong point. :+1: from me.

Looks like sql madness to me.
Logic-wise, I don't see anything wrong, so I'll drop an LGTM :dromedary_camel:


update Papa example

Describe at a high level the solution you're providing
I have converted the PapaDuck Example to work with AWS IoT core instead of the discontinued IBM Watson IoT Platform
Is this a patch, a minor version change, or a major version change
Minor
Testing Methodology. How can someone test your pull request?
Follow the instructions created here
Wiki has also been updated here

I tested the AWS IoT Papa and it works. The device is connected and the data is publishing.
Will wait to test for Issue#328.


Initial commit to co-lab

Initial commit to the co-lab for the CSP connector.

 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.Alex Dunker seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add dask module annotation

Description
Dask library is based on pandas classes and methods for data frames with the extension for large parallel computing and shuffling.
Some module-specific methods required annotating, including file system methods from pandas such as read_csv.
Type of change

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Before adding annotation:
import dask.dataframe as dd
df = dd.read_csv("tests/simple_data.csv")

After:
import dask.dataframe as dd
df = dd.read_csv("tests/simple_data.csv")
df.compute()
df.head()

I see, you are correct @lionsardesai. I mistook those two for a method of DataFrame class.
Would explode, melt, pop, drop, dropna, drop_duplicates be a part of it since they mutate the object?
What side_effects should visualize and head have If those should even have annotation?

Thanks to @lionsardesai for giving me better insight into what annotations are required for dask.
After checking all methods in detail, the only method that could be used for annotations is pop because it changes the variable inplace.
I updated the annotations and test files.

Very nice! 🎉
Thanks to you too @lionsardesai


[CCFPCM-498] [DRAFT]

CCFPCM-0498
Objective:

Tickets 498/374
round 4 heuristics

copied to another branch


ci: replace slash with dash for CF preview URLs

Otherwise, '/' can't be used in hostname, so results in an incorrect preview URL
Also detected that if branch name is too long it gets cut. In one case it was cut to 28 chars but unsure if 28 is the hard limit or if they remove parts (separated by -)

#288  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


Path with space in causes problem

Removed original claim of 'fails to build' as I'm getting that fail without a space in the path now, but there is still a problem with a path with space:
[Edited - still a problem]
If the path where the modelfusion repo is cloned has a space in it then the pnpm install generates an extra folder:
Reproduce with:
mkdir 'My Path'
cd 'My Path'
git clone https://github.com/lgrammel/modelfusion.git
cd modelfusion
pnpm install

Then a new folder 'My' is made in the parent folder.

Fixed - this was caused by Node v22 - rolling back to Node v20.12 fixed the initial fail to install, and the extra folder created.


identity: Dex uses CRDs since 1.8.x (before: TPRs).

Reflects the current state since release of Tectonic 1.8.4-tectonic.1 on 2017-12-19:

Dex uses CustomResourceDefinitions (CRDs) (link to release notes included)
Former usage of ThirdPartyResource (TPRs) mentioned

This PR was created in the fork, not in the upstream repo.


Getting RuntimeError in resnet50 example

While running examples/resnet50.py (needed to change loss.init_parameters(rng_key, *next(batches)) from loss.init_parameters(*next(batches)) on line 100 apparently), I get
RuntimeError: Internal: Unable to launch convolution with type forward and algorithm (0, none).
Completely mysterious to me, but maybe obvious to @JuliusKunze ?

Hey @harpone, thank you for pointing that out! I hadn't tested the resnet example after some API changes I made earlier today. Just fixed it and also updated the colab version. Please let me know if you are running into more issues.


Cannot drag tab to existing window

System information:

Operating system: MacOS Big Sur: 11.2.3 (20D91)
DBeaver version: <IP_ADDRESS>106090921

Connection specification:

Database name and version: Microsoft SQL Server 2019 (RTM) - 15.0.2000.5 (X64)
Sep 24 2019 13:48:23
Enterprise Edition: Core-based Licensing (64-bit) on Windows Server 2019 Standard 10.0 (Build 17763: ) (Hypervisor)
Driver name: don't know
Do you use tunnels or proxies (SSH, SOCKS, etc)?: Yes, F5 Big-Ip

Describe the problem you're observing:
I cannot drag a log or output tab from one window to another, as shown in this video.
Steps to reproduce, if exist:
Try to drag a log or output tab from one window to another.
Include any warning/errors/backtraces from the logs

thank you for the report. will be implemented in the scope of #11533

Thanks @uslss
I do not think that the problem I am reporting is related to #11533 . I just want to move a tab from one DBeaver window to another, which I think I have done in the past. I am sure that in the past I have been able to have all the tabs for results, statistics, log, output in the window pane below the sql query.
This seems like an Eclipse tab and window management problem, and is certainly not a "using results from existing queries as a source for future queries" issue.
Arthur

@artgoldberg Hello. #11533 is a separate feature, but while implementing it we also added the possibility to move all tabs from the right side (Output log, variables etc) to the bottom with resultset

Thank you @Matvey16
When do you expect #11533 to be released?
In the meantime, how can I "have all the tabs for results, statistics, log, output in the window pane below the sql query" as I did with the previous version of DBeaver?

@artgoldberg You can't move them from the right side for now. But the new version with an option to do so is scheduled to be released between 18th and 19th of July depending on your time zone.

Thanks @Matvey16 , that sounds good. Let's leave this issue open until I test it in the new version.


Listen for file find/replace in CustomEditor extensions

Hi!
I'm porting a rich text editor from this online web app https://magebook.github.io to a downloadable VSCode extension.
My editor already knows how to find/replace text, and I already have a widget for that (I just need to show it when it's needed). However, I found no way to bind this behaviour to the "Edit → Find Ctrl+F" inside the VSCode navbar.
Am I missing something? If not, I would suggest adding this feature.
The closest issues I found is https://github.com/microsoft/vscode/issues/91564, but it was closed several times ago.
In my opinion, the ideal solution would be to add something like onFindCommand inside the  CustomReadonlyEditorProvider interface, or to provide an event.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


cli: bugfix connectivity perf command

Currently, regular connectivity tests and perf tests are still a bit coupled, but they can't be run concurrently.
In the case of LRP enabled, we were deploying only perf pods, but the validation of deployments was waiting for lrp pod too:
✨ Deploying perf-client deployment...
✨ Deploying perf-client-other-node deployment...
✨ Deploying perf-server deployment...
⌛ Waiting for deployment cilium-test-1/perf-client to become ready...
⌛ Waiting for deployment cilium-test-1/perf-client-other-node to become ready...
⌛ Waiting for deployment cilium-test-1/perf-server to become ready...
⌛ Waiting for deployment cilium-test-1/lrp-client to become ready...
timeout reached waiting for deployment cilium-test-1/lrp-client to become ready

As we have to separate functions for deploying deployments and validating deployments, we need to keep them in sync. Now in validation of deployments, return early in case of perf command, just like in a deploy function.
cli: fix a case when connectivity perf command was hanging if LRP was enabled in the cluster

/test


error setting up sensor

after upgrading to 10.3.  i keep on getting "unavailable" timings with detail logs below.
Logger: homeassistant.components.sensor
Source: custom_components/lta/sensor.py:195
Integration: Sensor (documentation, issues)
First occurred: 4:47:21 PM (1 occurrences)
Last logged: 4:47:21 PM
Error while setting up lta platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/lta/sensor.py", line 195, in async_setup_platform
BusArrivalSensor(coordinator, idx) for idx, ent in enumerate(coordinator.data)
TypeError: 'NoneType' object is not iterable

Hi, see below.
sensor:

platform: lta
api_key: removed
bus_stops:

code: '67141'
buses:

'86'
'119'
'136'

code: '67139'
buses:

'119'
'136'

I am unable to replicate this error with your config. Have you tried removing and adding ha-lta again? Make sure it's using v0.3.0.

yes, tried removing and reinstall but still same error.

yep. tried removing and putting back. i think incompatible with latest version of HA.  it was working fine when i was on 8.2
from the log looks like error is from ha python script.
Logger: homeassistant.components.sensor
Source: custom_components/lta/sensor.py:195
Integration: Sensor (documentation, issues)
First occurred: 8:21:51 PM (1 occurrences)
Last logged: 8:21:51 PM
Error while setting up lta platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/lta/sensor.py", line 195, in async_setup_platform
BusArrivalSensor(coordinator, idx) for idx, ent in enumerate(coordinator.data)
TypeError: 'NoneType' object is not iterable

Same issue here. Tried to reinstall and reset everything. Removed sensors from configuration and unavailable state, restart and added 1 stop, OK. Upon adding new stop and restart then the error appears.
The image attached shows a sensor at another continent. Lol. Thou not a big issue

Ok, will have to definitely look into this. Just to confirm, do the sensors work with only one bus stop configured?

After clear off and start from scratch.. after that, restart, same thing

Hi,  this is definitely something with 10.3 HA.   I rolled back to 8.2 and it works again.   Then I upgraded again to 10.3 with my original configration.yaml from 8.2 and still works.  But when I load the 10.3 config with new MQTT: config fromat, it doesn't work again.  And when I totally delete all my other sensors, mqtt,etc leaving only lta sensor still doesn't work.  But  when I load back my original 8.2 config, works again.  I am not able to pinpoint which one is causing that HA python error.

yes. it's working fine.

What do you mean by new MQTT config format?

is the new config format required for >= 10.3
https://www.home-assistant.io/integrations/binary_sensor.mqtt/#new_format


Query on 'IP network rules can't be used in the following cases'

Hello,
Regarding Grant access from an internet IP range, this section states: "Services deployed in the same region as the storage account use private Azure IP addresses for communication. Thus, you can't restrict access to specific Azure services based on their public outbound IP address range."

I’m of the impression that Azure services deployed in the same region do not always have to use a private IP address as the source of the communication. E.g. An Azure Function Consumption plan deployment would connect to an Azure Storage Account from a public/internet IP address. I get the impression that where it states “Azure Services deployed in the same region”, it means Azure Services connecting via Service Endpoint (VNET Rule) or Private endpoints. Is that correct?

A similar statement is made in the PowerApps documentation Azure Synapse Link with Data Lake
"Currently, you can't provide public IPs for the Azure Synapse Link for Dataverse service that can be used in Azure Data Lake firewall settings. Public IP network rules have no effect on requests originating from the same Azure region as the storage account. Services deployed in the same region as the storage account use private Azure IP addresses for communication. Thus, you can't restrict access to specific Azure services based on their public outbound IP address range. More information: Configure Azure Storage firewalls and virtual networks"
I gather that DataVerse cannot take advantage of features like Service Endpoints (VNET Rule) or Private endpoints, so what mechanism is this connection from a DataVerse Private IP address using?

Putting aside IAM controls, is this ability to by pass the Storage Account firewall only limited to services in the same Azure AD tenant?

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

ID: 2632af75-7664-b79b-ac5b-c7c17e6a609a
Version Independent ID: 6ecf71b6-77c4-3ab6-8ec9-353ea407d70f
Content: Configure Azure Storage firewalls and virtual networks
Content Source: articles/storage/common/storage-network-security.md
Service: storage
Sub-service: common
GitHub Login: @jimmart-dev
Microsoft Alias: jammart

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

@STDSA
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.
@jimmart-dev
Can you please check and add your comments on this doc update request as applicable.

Hello, Just wanted to check in on this item. Is there any update?

Thank you for your dedication to our documentation.
We sincerely apologize for the delayed response. After a careful review, we are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close

Thanks for your dedication to our documentation.
We sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary due to various constraints. Therefore, we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close


The lint command line reference is incorrect

Description
The ng lint command line reference incorrectly states that, "When a project name is not supplied, executes the lint builder for the default project." In fact, like ng test it will execute for all projects.
What is the affected URL?
https://angular.io/cli/test
Please provide the steps to reproduce the issue

Clone TrevorKarjanis/angular-issue-43217.
Run npm install.
Run ng lint.

Please provide the expected behavior vs the actual behavior you encountered
As per the documentation, the command should only execute on the default project. Instead, it executes on all projects. This is reasonable, but the documentation does not reflect that.
Please provide a screenshot if possible

Please provide the exception or error you saw
"When a project name is not supplied, executes the lint builder for the default project."

Is this a browser-specific issue? If so, please specify the device, browser, and version.
No response

Related to #21473 but for ng lint


[otbn] The otbn_multi_err test currently fails with an assertion error

Description
This seems to be independent of the seed. For an example, run:
util/dvsim/dvsim.py hw/ip/otbn/dv/uvm/otbn_sim_cfg.hjson -i otbn_multi_err --fixed-seed 123

You'll get the following assertion failure:
xmsim: *E,ASRTST (/home/rjs/work/opentitan/scratch/work/otbn-sim-xcelium/default/src/lowrisc_ip_otbn_0.1/rtl/otbn_core.sv,927): (time 7498330 PS) Assertion tb.dut.u_otbn_core.ImemAddrOKnown_A has failed
UVM_ERROR @   7498330 ps: (otbn_core.sv:927) [ASSERT FAILED] ImemAddrOKnown_A

I spent a bit of time digging down into this while looking through the trace. It seems that we're trying to pop from an empty call stack. I notice that the call stack machinery got rewritten last September, so maybe we've done something silly.
Given that this is the multi-error test, it's possible we're supposed to be causing an error in this situation anyway, so we just need to qualify the assertion with "only care if things aren't dying for other reasons"!
@GregAC: Since you rewrote the code in question, would you mind taking a look? I'm optimistic that this will be pretty easy to "waive" at the assertion level.

There are some potential RTL fixes to apply here https://github.com/lowRISC/opentitan/pull/22115 and https://github.com/lowRISC/opentitan/pull/22116

This is still broken. It looks like the requested imem address ends up with 'x. Looking quickly at the trace, it appears that we're branching based on x when running the (intentionally buggy) software in beq-0.s.
To reproduce:
util/dvsim/dvsim.py hw/ip/otbn/dv/uvm/otbn_sim_cfg.hjson -i otbn_multi_err --fixed-seed=123 --waves=shm

The trace will look like this:

Will either of your PRs above fix this?


Update Version to 0.2.10-SNAPSHOT after 0.2.9 release.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #351   +/-   ##
=======================================
  Coverage   60.91%   60.91%
=======================================
  Files          64       64
  Lines        1658     1658
  Branches      250      250
=======================================
  Hits         1010     1010
  Misses        542      542
  Partials      106      106

Continue to review full report at Codecov.

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


Error in cf dev start

Hi,
I am getting the following error when I gave the "cf dev start" command on linux machine
[root@localhost deb]# cf dev start
Using existing image.
Allocating 4096 MB out of 15951 MB total system memory (7662 MB free).
Importing VM...
Starting VM...
FAILED
Error: failed to start VM: failed to execute 'VBoxManage startvm pcfdev-v0.461.0 --type headless': exit status 1: VBoxManage: error: VT-x is not available (VERR_VMX_NO_VMX)
VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component ConsoleWrap, interface IConsole
Waiting for VM "pcfdev-v0.461.0" to power on...
.

Hi @dsubudhi26,
You are attempting to start PCF Dev inside of a VM, or on a machine that does not support the VT-x processor extension. The VT-x processor extension is required to run 64-bit VMs in Virtualbox (such as the PCF Dev VM).

Hi Stephen,
Thanks for the info .Can PCF Dev run inside a VM, which has VT-x processor
extension or it needs a physical machine ?
Regards,
Debabrata
On Wed, Jul 12, 2017 at 8:01 PM, Stephen Levine<EMAIL_ADDRESS>wrote:

Hi @dsubudhi26 https://github.com/dsubudhi26,
You are attempting to start PCF Dev inside of a VM, or on a machine that
does not support the VT-x processor extension. The VT-x processor extension
is required to run 64-bit VMs in Virtualbox (such as the PCF Dev VM).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pivotal-cf/pcfdev/issues/244#issuecomment-314787506,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AclO4TvMW5xnO4ig8BfC13X4_35u0aByks5sNNikgaJpZM4OVWFX
.

PCF Dev can technically run in a VM that exposes the VT-x processor extension, but it may not be very performant depending on the resources available to the outer VM. I would encourage you to run PCF Dev on a physical machine if possible.
(We test PCF Dev nested in vSphere VMs with VT-x that have 8-16 Xeon CPUs and 64 GB of RAM available.)


Refactoring the Organization Page

As discussed a while back in #280, I stripped the custom stats lang logic out of markdown.js and created a component that maps out the projects to cards instead of a table:

This is still a work in progress but I think this view makes the information a bit more approachable, especially on smaller screens. Here's what's done and left to do:

[x] Replace stats lang with a component
[x] Move project list to json sheet
[x] Replace table with card view
[ ] Link to this page somewhere throughout the site
[ ] Clean up the styling and fix a hack with Contributors I had to add
[ ] Discuss the configuration for this page with @bebraw

The last one is the biggest hold up, currently this only works on the dev-server. If we like the direction of this then maybe @bebraw you might be able to give me an idea how best to configure this route?

Do you get some particular error when building?

@bebraw no build errors, it's just that nothing is created in the build folder for the organization page. I was reading through the antwar docs and it seems that mapping a route just to a component (not to a component that accepts content like Page) wasn't really the intended use case?

@skipjack nice work! Looks good visually (based on your gif)! I'll test it when it's a bit further along.

@bebraw no build errors, it's just that nothing is created in the build folder for the organization page. I was reading through the antwar docs and it seems that mapping a route just to a component (not to a component that accepts content like Page) wasn't really the intended use case?

Yeah, it has been designed so that it pushes found content through given layouts. We probably have to extend the system so that you can do something like path: function() { ... return loaded jsx here ... }. Then you can have custom views like this.
An alternative would be to extend the current / definition (it matches only markdown files) and build it there although I can see the value of functionality specific overrides like the one you did.

@SpaceK33z sounds good I'll let you know when it's in better shape.
@bebraw -

We probably have to extend the system so that you can do something like path: function() { ... return loaded jsx here ... }. Then you can have custom views like this.

Ah, didn't think of that, that would then just pass it to a simple wrapper component I guess? I was also thinking maybe the json could be loaded through the path configuration and be passed to the component that way instead of directly imported. Might be a good solution for auto-generated json that we want to feed to multiple pages with the same general layout.

An alternative would be to extend the current / definition (it matches only markdown files) and build it there although I can see the value of functionality specific overrides like the one you did.

Yeah, that might work. If necessary maybe we have to do a quick hack to get it in for now and then figure out something more long term (not sure what effort any of these solutions would take on the antwar side). There's probably a few other places, e.g. the voting app, that could use a similar kind of configuration. Anyway, let's take this offline in gitter or something. I'll ping you once I have a bit more time to play around with it.

@skipjack I implemented something you might find useful. You can write
standalone: {
  path() {
    return require('./layouts/Standalone').default;
  }
}

now. It's important that path is wrapped like that as otherwise it won't work. But this should give you overrides.
If you want to give it a go, you can find the feature in 22838a12 release.

@bebraw awesome, I'll try this out as soon as I can.

Alright. efa30336 is the freshest version (I had to do a couple of fixes/tweaks). You can reach me through Gitter if there's something funky going on.

Ok with @bebraw's help, I got this into a better state (now it's passing the build). Working on wrapping it up but I have a few questions/comments:
@webpack/documentation-team

@sokra @NejcZdovc can one of you test the voting app to make sure everything is working as you would expect? I updated the config for it and made a fix in the List component. Seems to be ok, but I haven't messed with it much before so I don't know if I'm missing something.

Any ideas where we should link to this page from? The footer's getting kinda full...

For the projects that didn't have maintainer's associated with them, I was going to link here although that issue is now closed. Is there somewhere better to point people to?

I'm going to finish the JSON sheet manually, but at some point we could probably auto-generate this like we are for plugins and loaders. @bebraw @sokra let me know once the plan for auto-generation is more fleshed out and I'll try to follow suit for this page.

I'm going to finish the JSON sheet manually, but at some point we could probably auto-generate this like we are for plugins and loaders. @bebraw @sokra let me know once the plan for auto-generation is more fleshed out and I'll try to follow suit for this page.

If we stick with the current scheme for loaders and plugins, that could be easily generalized to work across the whole organization (certain filter needs to be skipped for this page).

@skipjack Tried it out and vote app is working correctly as far as I can see.

@NejcZdovc great, thanks!
@SpaceK33z should be ready for a some testing now, when you have a chance. We still need to decide where to link to it from but for now you can just hit localhost:8080/organization on build-test.

@skipjack Poke me if you want to get rid of that components/organization/projects.json. It can be generated with a little bit of effort. That maintainer bit might be harder, but we can probably assume it's in package.json of each project or so.

@SpaceK33z great, I'll find a place to add a link to this page tonight and then merge it.
@bebraw I'll merge it the way it is for now bc I'd like to discuss automation a bit more before implementing. I'll ping you soon re that and the other stuff we've been discussing.


It opened the Whatsapp with the text but didn't send automatically.

It opened the Whatsapp with the text but didn't send automatically.
pywhatkit.sendwhatmsg_instantly('+60183546457', 'Hello Testing Pywhatkit')

it did sent but only after a while.

Hey @eddieonthego, there is a wait time before the message is sent. For sending messages instantly, you can use the sendwhatmsg_instantly function. Thanks


Small fixes and performance improvements

Hi!
Despite the huge changes, there is no changes in logic itself.
Some improvements:

Usage of static final in fields where possible
All instance variables final
Unnecessary use of String.toCharArray replaced by direct String.charAt
.isEmpty() instead of .equals("")
Private methods to the end of file
Added javadoc regarding Thread-safety (Issue #18)

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


Unable to find module

After doing pod install app is still unable to find module. I have used the dependency from readme project. is there something needed to be changed in info.plist?

It's difficult to help with such little info:

What exact error are you getting?
What version of Chatto are you trying to use? (1.0.x/dev)
What version of cocoapods are you using?
How can we reproduce the same issue? (provide an example project if possible)

Nothing is required regarding info.plist

Sample project - https://bitbucket.org/HarshitBangar/chat-client-ios Xcode - 7.3.1(7D104)  and cocoa pod - 1.0.1 . I am able to import all the other projects like AlomaFire but some how chatto and chatto additions are not visible.

@bangarharshit I can't reproduce your problem - it just works fine for me. Can you push a branch after running pod install and some use of Chatto/ChattoAdditions?
If you're starting now with Chatto I highly recommend you to use "dev" branch - Chatto 2.0 is "imminent" and has some breaking API changes and a lot of bugfixes and improvements

Chatto 2.0 can I download through pod (or anything without git). I see in your dev branch pod spec version is still 1.0

You can use this in your Podfile to use dev branch:
pod 'Chatto', :git => 'https://github.com/badoo/Chatto', :branch => 'dev'
pod 'ChattoAdditions', :git => 'https://github.com/badoo/Chatto', :branch => 'dev'

Cool this is working for me. Closing the issue.


CEA 608 captions not working with shaka 3.1.3 and 3.2.1

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
We see this problem with shaka 3.1.3 and 3.2.1
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from master?
Yes
Are you using the demo app or your own custom app?
Custom app
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
Chrome on MAC
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
What are the manifest and license server URIs?
We will share stream and license URL information if necessary.
But I expect that this can be reproduced using any stream that has 608 captions.
What configuration are you using?  What is the output of player.getConfiguration()?
Will provide this if necessary
What did you do?
Play stream with CEA-608 text track selected
What did you expect to happen?
Captions should be presented in the UI
What actually happened?
Caption are not displayed.
I have isolated the problem to this commit in the v3.1.x branch
fix(ttml): Fix ttml erroneously dismissing cues.	7ae5d107 Oct 12, 2021 at 12:31 PM
prior to that commit our 608 captions work, after they do not

I have shared info on how to access stream via<EMAIL_ADDRESS>


Dark Mode Support

I realize it hasn't quite come to the Windows world, but would love to have support for Dark mode built into Aurora. Would love to see this tie into WET as well.
There are going to be a bunch of related issues around assets:
https://github.com/wet-boew/wet-boew/issues/8691

Windows 10 provides a dark theme, and a website can determine the user's preference and present itself with it's own dark theme, as in the example provided from this Stack Overflow answer:  https://stackoverflow.com/a/52986538/4930982
See also "Detecting the desire for light or dark color schemes: the prefers-color-scheme feature", at https://drafts.csswg.org/mediaqueries-5/#prefers-color-scheme


Trainings suggestions and recommendations

Description
Action items from the product champions session
Definition of done

[ ] PM trainings
[ ] Devops trainings
[ ]

Attachments
title

Devops CoP (Run by two 18F'ers) - https://digital.gov/communities/devops/


Make sure external links all use rel="noopener"

Currently the only external link is Github but for custom script descriptions. Also not just noopener but nofollow and all the other ones too


fix getting dbtable from magento env

Description
https://github.com/SnowdogApps/create-alpaca-theme/blob/dfc0e6af81edac01e161ab0243f5c7caf68ead6a/src/database-actions.js#L21
When creating a new theme using npx @snowdog/create-alpaca-theme command on a new warden magento2 install the above line doesn't parse the etc/env.php db table correctly. Probable cause is that it filters array item based on the first host occurence in the array tree. In case of warden install, first occurence would be queue array(finding IP of rabbitmq matches the IP this script tried to connect to.

app/etc/env.php snippet

...
'queue' => [
        'amqp' => [
            'host' => 'rabbitmq',
            'port' => '5672',
            'user' => 'guest',
            'password' => 'guest',
            'virtualhost' => '/'
        ],
        'consumers_wait_for_messages' => 1
    ],
...
'db' => [
        'table_prefix' => '',
        'connection' => [
            'default' => [
                'host' => 'db',
                'dbname' => 'magento',
                'username' => 'magento',
                'password' => 'magento',
                'model' => 'mysql4',
                'engine' => 'innodb',
                'initStatements' => 'SET NAMES utf8;',
                'active' => '1',
                'driver_options' => [
                    1014 => false
                ]
            ]
        ]
    ],
...

Solution
Below change seemed to work locally and command was able to finish but maybe there's some better way for getting the needed db info
const dbTable = file.toString().split('[').filter((x) => x.includes('host'))
to
const dbTable = file.toString().split('[').filter((x) => x.includes('dbname'))

@adamwaclawczyk Thanks! Checked it locally, works perfectly!


undefined convertTXTToStruct

When I run MOT_cross_validation.m, there is an error: A function that does not define the input parameters of the'char'type : convertTXTToStruct. And I did not find convertTXTToStruct.m in the folder of MDP_Tracking-master.

Were you able to solve it ? I am having similar issue

same problem  Anyone know how to save?

should download the MOT motchallenge-devkit--> https://bitbucket.org/amilan/motchallenge-devkit/


aiohttp multidict query parameters

aiohttp supports multidict for headers and query parameters, allowing the user
to specify either a dict or a list of 2-item tuples as input. For example:
async with aiohttp.ClientSession() as s:
    params = [('a', 'alpha'), ('b', 'bravo'), ('a', 'Alpha')]
    async with s.get('http://<IP_ADDRESS>:8888', params=params) as r:
        buf = await r.text()

This patches vcrpy to handle this specific case instead of only dict-like
objects.

@ginoledesma Any chance you can resolve conflicts here?


Add Lowest common ancestor algorithm

In graph theory and computer science, the lowest common ancestor (LCA) of two nodes v and w in a tree or directed a cyclic graph (DAG) T is the lowest (i.e. deepest) node that has both v and w as descendants, where we define each node to be a descendant of itself (so if v has a direct connection from w, w is the lowest common ancestor).
There's many methods to find the LCA, one can be using Euler tour and a segment tree or by using Binary lifting which is the implemented method.

please modify your code with OOP concept, and conform to C++11 standards with detailed comments.

Thanks a lot for the advice, I'll divide it into a .h and .cpp and apply OOP concepts 👍

I have confirmed to C++11 standards and added detailed comments describing the flow of the algorithm.

please make changes to README.md too, add entry to your alg.


Gold cron job does not appear to be running

Logs:
https://pantheon.corp.google.com/logs/viewer?project=flutter-dashboard&minLogLevel=0&expandAll=false&timestamp=2021-10-07T23:44:44.525000000Z&customFacets=&limitCustomFacetWidth=true&interval=PT1H&resource=gae_app%2Fmodule_id%2Fdefault%2Fversion_id%2Fversion-2615f0f&logName=projects%2Fflutter-dashboard%2Flogs%2Fappengine.googleapis.com%252Fstdout&logName=projects%2Fflutter-dashboard%2Flogs%2Fappengine.googleapis.com%252Fnginx.request&logName=projects%2Fflutter-dashboard%2Flogs%2Fappengine.googleapis.com%252Frequest_log&scrollTimestamp=2021-10-07T23:40:00.498000000Z&filters=text:push-gold-status&dateRangeEnd=2021-10-07T23:44:40.352Z&dateRangeUnbound=backwardInTime
The last time it ran was 2021-10-07 15:40:25.670 PDT then it was not called for an hour. The cron job is scheduled to run every 5 minutes.
The next time it appeared in the logs was an hour later and it appears to have timed out with a 504 status code.
2021-10-07 16:40:00.498 PDT GET 504 160B 3,600 s  /api/push-gold-status-to-github
It looks like the last run did not finish, and it may have led to the timeout

It appears to have resolved itself, not sure if there was a service interruption like our github api limit being reached? At the same time it went down the check for waiting pull requests api was throwing errors.

FYI @godofredoc and @keyonghan


Error when running on production env

Hello,
running the example https://github.com/zeit/next.js/tree/master/examples/with-next-page-transitions on production env triggers an error:

TypeError: Cannot read property 'isRequired' of null
at Object. (/home/freddy/Workspace/workspace-projects/workspace-together/with-next-page-transitions/node_modules/next-page-transitions/lib/PageTransition.js:300:37)

Which refers to this:  line 300 - PageTransition.js
timeout: _PropTypes.timeoutsShape.isRequired,
When looking from the origin of the problem, I found this line on the react-transition-group dependency
var timeoutsShape = process.env.NODE_ENV !== 'production' ? _propTypes.default.oneOfType([_propTypes.default.number, _propTypes.default.shape({   enter: _propTypes.default.number,   exit: _propTypes.default.number }).isRequired]) : null;
Note: this problems does not appear on the 1.0.0-alpha.1 release because back then we had on PageTransition.js:
timeout: _PropTypes.timeoutsShape
Regards,
Freddy

So can it now be used in production?

I am using it now in my Next app in production and it's working! https://www.pymusician.com


docs: describe steps for Private Voting use case.

Describe Private Voting steps and the roles in each step.

Closing and reopening with updated branch.


feat(api-client): remove axios

closes #2895
Went for the easiest way to remove axios, serialized headers and what not. Seems to be working, needs heavy testing! Also added bit of type safety and removed what I hope was unneeded code 🦑, and I think cookies should work now? need to test

OMG, I love this, so great!
I rebased the PR and fixed a few tests! A rabbit hole I got lost into:
When fetch() gets FormData with a File the void server only receives the beginning of the body. It seems that fetch() is switching to streaming for files, which might be what we want, but I’m not able test this.
Long story short: The File test is disabled, and we should double check files are still working properly.

Yea I think we can do things differently with fetch but I didn't want to change too much of marcs buffer logic. We can definitely switch to using the appropriate parsing methods (json, blob etc) instead of just arrayBuffer.
But that would mean much more changes, I'll see if file works

Alright I went deep on the last test and it turns out that it is unrelated to our code. Looks like the fetch its using has an issue or we need to patch it or something. In the future we can try using node-fetch or something else which might work. Going to merge it for now!


[CELEBORN-869][DOC] Document on Integrating Celeborn

What changes were proposed in this pull request?
As title.
Why are the changes needed?
As title.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual test.

cc @pan3793 @cfmcgrady @AngersZhuuuu

Thanks, merging to main.


Web service logging does not work for calls returning a soap fault

For the Fault Tolerance feature the behaviour of the web services has changed. In stead of a response with not ok, a soap fault is returned.  As a result I expect there will be no web service logging, which is using the web service response, present.
First test if the ws service logging indeed fails as expected in case of soap fault.
If so, add the appropriate logging...

This error is testwed on the new logging module 'opsp_logging'
The SOAP faults are logged correct in the table "web_service_monitor_log"
so this issue can be closed


Deprecate dockbit service - discontinued

In reference to #2344

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @assimovt!

  Generated by :no_entry_sign: dangerJS

Hi! Thanks for picking this up! We handle discontinued services by replacing them with a deprecatedService as in #2352. Would you like to update this along those lines? If not I or one of the other maintainers can do.

@paulmelnikow Cool, I've updated PR.


Fix: Improves Breadcrumb accessibility

What's the purpose of this pull request?
This PR aims to improve Breadcrumb's accessibility, as reported on: https://github.com/vtex/faststore/issues/1936

DropdownButton

How it works?
Adding a aria-label to DropdownButton fixes the error.
How to test it?
The store should look the same.
References
Issue: Improve Breadcrumb Accessibility #1936

I think we can remove the focus from the div inside

image
code


[Bug] FLOPs counter is currently not supported for centerpoint

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (dev-1.x) or latest version (dev-1.0).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
main branch https://github.com/open-mmlab/mmdetection3d
Environment
LOPs counter is currently not supported for centerpoint
Reproduces the problem - code sample
LOPs counter is currently not supported for centerpoint
Reproduces the problem - command or script
get_flops.py
Reproduces the problem - error message
LOPs counter is currently not supported for centerpoint
Additional information
LOPs counter is currently not supported for centerpoint

We do not yet support point cloud related FLOPs calculations


Running the App Ends up opening Problem Report for Runner in iOS

Is there an existing issue for this?

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

Steps to reproduce
Any working project with the following dependencies.
name: sky_migration
description: Sky application migration
version: 1.0.0+1
publish_to: none

environment:
  sdk: ^3.0.0

dependencies:
  another_flushbar: ^1.12.30
  bloc: ^8.1.2
  flutter:
    sdk: flutter
  flutter_bloc: ^8.1.3
  flutter_local_notifications: ^14.1.0
  flutter_localizations:
    sdk: flutter
  flutter_svg: ^2.0.6
  lottie: ^2.3.2
  permission_handler: ^10.2.0
  responsive_sizer: ^3.1.1
  rounded_loading_button: ^2.1.0
  sky_client: # custom package
    path: /Users/mr.minnu/Documents/sky_client
  tabler_icons: ^2.5.0+1
  url_launcher: ^6.1.11

dependency_overrides:
  # intl: ^0.18.1
  collection: ^1.17.2

dev_dependencies:
  bloc_test: ^9.1.2
  flutter_lints: ^2.0.1
  flutter_test:
    sdk: flutter
  mocktail: ^0.3.0

flutter:
  uses-material-design: true
  generate: true
  # To add assets to your application, add an assets section, like this:
  assets:
    - assets/
    - assets/fonts/
    - assets/lottie/
    - assets/images/
    - assets/images/ic/
    - assets/images/ic/banners/
    - assets/images/ic/flags/
  fonts:
    - family: Satoshi
      fonts:
        - asset: assets/fonts/SatoshiRegular.otf
        - weight: 500
        - asset: assets/fonts/SatoshiBold.otf
        - weight: 700

NOTE:
This was observed in Sonoma - MacOS 14.0 (23A5286i) - Beta
Model Name:	MacBook Pro
Chip:	Apple M2 Pro
Total Number of Cores:	10 (6 performance and 4 efficiency)
Memory:	16 GB
System Firmware Version:	10<IP_ADDRESS>.3
OS Loader Version:	10<IP_ADDRESS>.3
System Version:	macOS 14.0 (23A5286i)
Kernel Version:	Darwin 23.0.0
Boot Volume:	Macintosh HD
Boot Mode:	Normal
Secure Virtual Memory:	Enabled
System Integrity Protection:	Enabled

Expected results
Should open the app normally.
Actual results
Opens Problem Report for Runner Logs - runner.log
Code sample
sky-master.zip
sky_client-master.zip
Screenshots or Video
Not available
Logs
Flutter run verbose logs
Flutter Doctor output
Doctor output
[!] Flutter (Channel stable, 3.10.5, on macOS 14.0 23A5286i darwin-arm64, locale en-IN)
    • Flutter version 3.10.5 on channel stable at /Users/mr.minnu/Development/flutter3
    ! Warning: `flutter` on your path resolves to /Users/mr.minnu/Development/flutter/bin/flutter, which is not inside your current
      Flutter SDK checkout at /Users/mr.minnu/Development/flutter3. Consider adding /Users/mr.minnu/Development/flutter3/bin to the
      front of your path.
    ! Warning: `dart` on your path resolves to /Users/mr.minnu/Development/flutter/bin/dart, which is not inside your current Flutter
      SDK checkout at /Users/mr.minnu/Development/flutter3. Consider adding /Users/mr.minnu/Development/flutter3/bin to the front of
      your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (4 weeks ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update
      checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/mr.minnu/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A5195m
    • CocoaPods version 1.12.1

[✗] Chrome - develop for the web (Cannot find Chrome executable at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.80.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.68.0

[✓] Connected device (3 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64 • Android 14 (API 34) (emulator)
    • iPhone 14 Pro Max (mobile)  • D30B3598-14D4-4CB1-8810-1DC22222C0C1 • ios           • com.apple.CoreSimulator.SimRuntime.iOS-17-0
      (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64  • macOS 14.0 23A5286i darwin-arm64

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

! Doctor found issues in 2 categories.

Hi @yahu1031
Since this issue happens on your personal project and it's using many 3rd packages that it's not sure if it's a Flutter framework issue.
You should narrow down the issue by removing the usage of 3rd party plugins/packages one by one, unrelated/unaffected pages, and isolating the affected page into a single main.dart file that can be run independently. If the issue's still reproducible, please provide that main.dart file so that we can verify the issue properly. Thanks!

@huycozy Looks like flutter_keychain is the culprit. But not sure though. Need to cross-check it.


[Subtask]: Expire snapshots and orphan files clean handle the Iceberg statistic files

Search before asking

[X] I have searched in the issues and found no similar issues.

Description
Expire snapshots and orphan files clean should handle the Iceberg statistic files correctly.
Parent issue
#1354
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

related to #1959


Issue working with chain webpack

Have an issue getting this to work well with chain webpack and vue-cli. I have tried multiple approaches to get it working however the constructor arguments make me wonder if its compatible.
const SiteMap = require('vue-cli-plugin-sitemap')
const requireESM = require('esm')(module)
const routes = requireESM('./src/routes')
const pluginAPI = require('@vue/cli-service').PluginAPI
...

config
.plugin('vue-cli-plugin-sitemap')
.use(SiteMap, [{
  url: 'https://example.com,
  routes
}])

> vue-cli-service serve

 INFO  Starting development server...
 ERROR  TypeError: Plugin is not a constructor
TypeError: Plugin is not a constructor

Then using init, I dont know where I would accessthe PluginAPI constructor parameter for the vue cli server or why I would have to:
config
.plugin('vue-cli-plugin-sitemap')
.init((init, args) => {
   api = new pluginAPI('any-id', ???)
   return init(api, args[0])
 })
.use(SiteMap)
.tap((options) => {
  options[0] = {
     sitemap: {
      url,
      routes: routes.rawRoutes
     }
   }
  return options
])

so after stepping through the code more it became clear that all i had to do was set up the plugin options object and forgo the chain webpack method in order to have the plugin load. it was not clear to me initially that the vue.config.js module system supported both.
module.exports = {
  pluginOptions: {
    sitemap: {}
  },
  chainWebpack: (config) => {
   ...
 }
}


Google Pay Request failed An unexpected error has occurred Please try again later [DeveloperErrorException]

We are getting this error after calling  showPaymentSelector.
Screenshot

Plugin Version
flutter_stripe: ^3.1.0
pay: ^1.0.10

AndroidManifest
<meta-data
    android:name="com.google.android.gms.wallet.api.enabled"
    android:value="true" />

config.json
{
  "provider": "google_pay",
  "data": {
    "environment": "TEST",
    "apiVersion": 2,
    "apiVersionMinor": 0,
    "allowedPaymentMethods": [
      {
        "type": "CARD",
        "tokenizationSpecification": {
          "type": "PAYMENT_GATEWAY",
          "parameters": {
            "gateway": "stripe",
            "stripe:version": "2020-08-27",
            "stripe:publishableKey": "pk_test_SETTLE_OUR_PK"
          }
        },
        "parameters": {
          "allowedCardNetworks": [
            "VISA",
            "MASTERCARD",
            "DISCOVER",
            "JCB",
            "AMEX"
          ],
          "allowedAuthMethods": [
            "PAN_ONLY",
            "CRYPTOGRAM_3DS"
          ]
        }
      }
    ],
    "merchantInfo": {
      "merchantId": "01234567890123456789",
      "merchantName": "OUR_COMPANY_NAME"
    },
    "transactionInfo": {
      "countryCode": "JP",
      "currencyCode": "JPY"
    }
  }
}

Dart code
final Pay _payClient = Pay.withAssets([AppConstant.applePayConfPath, AppConstant.googlePayConfPath]);

final paymentResult = await _payClient.showPaymentSelector(
      provider:  payProvider,
      paymentItems: _createPaymentItems(context),
    );

Logcat
$ adb -d logcat -s WalletMerchantError
-> Nothing displayed

$adb -d logcat -s pay
-> 06-10 23:01:21.297 11152 19749 E Volley  : [2075] cuv.b: Unexpected response code 500 for https://payments-pa.googleapis.com/payments/apis-secure/ui2/buyflowservice/initialize

Other

Apple Pay works fine.

I resolved this issue by adding PaymentItemStatus.final_price and PaymentItemType.total to PaymentItem, while Apple Pay doesn't require them.
It should be required params, isn't it?

https://developers.google.com/pay/api/android/guides/tutorial#paymentdatarequest

Important: Merchants that process transactions in the European Economic Area (EEA) or any other countries that are subject to Strong Customer Authentication (SCA) must include the countryCode, totalPrice, and merchantName parameters to meet SCA requirements.