Adding Github/Gitlab stats to summary

What do you think about adding not just git related information, but also some stats about a Github/Gitlab Repo to the summary? Like stars, followers and forks.

Cf. #63
IMO, the perks of having onefetch being an offline tool (better response time , usable offline) overpowers the benefits of the additional information.

I see that onefetch should not become bloat software.
But close to nobody is using a git repo without a vcs host like GitHub or Gitlab.
Onefetch can still be a offline tool. Onefetch is always passively including informations, e.g package manager and deps. It could just do a simple api call and look if it has internet connection and can successfully get a response from GitHub. If none of this is working just ignore it. This can also happen async too. I don't think response time should be an issue.
Code complexity wise, IMO, is not going to be a concern. A thin client wrapper around the GH api with more or less one call to get the repo stats. This can be completely modularised. We could also discuss if this is an extra feature only usable with conditional compilation.

@HallerPatrick FYI, in response to the conclusion to #63, I started this project 🙂
Querying the host data brings up these questions:

What happens if the remote and local repositories' histories differ (behind, ahead, diverged)? What would be the correct commit count and repo size? The local data or the remote data?
Onefetch's detected language and the repo host's detected language can differ. For example, onefetch supports Markdown as a language, while GitHub doesn't include it in its language statistics (awesome-c's language is null). Should onefetch's language detection using tokei be overridden by the language detected by the API?
These APIs often have a query limit for unauthenticated users. Should onefetch then support authentication tokens?
Simple API calls alone may not be enough for accurate data. For example, let's say we report statistics on labels. An API call would report that this repo has 0 enhancement issues, because we use Type: Enhancement. So should onefetch assume the host's defaults, or allow the user to configure how the data is queried/interpreted?

All those problems are arising the more information you want to include in your summary. All I would add is Stars, Forks and Watcher. So basically all you see on the top right of a repo on Github.com projects. Everything else needs an own project like yours.
What could be a problem is the authentication. Does the GH API allow for anonymous calls to it? If not, one could simply parse
a simple project GET requests of the project, which is easily done.

But close to nobody is using a git repo without a vcs host like GitHub or Gitlab.

I guess I am one of the few that uses git with no/private remotes for personal stuff or during early prototyping.

Querying the host data brings up these questions:

I got a few myself.

How to handle when multiple remotes have been configured?  Just loop through all of them?
What about selfhosted GitLab or Github Enterprises URLs that do not indicate if they are using one of these services?  i.e. git.company.com or host.your.code.com  I guess every non GitLab or GutHub URL would need to be probed.  This could lead to slowdowns due to extra checks for non GitLab or GutHub remotes.
Should GitLab or GutHub data be displayed if the remote is flagged as a mirror or fork?  Could lead to misleading information being reported.
Should offline be the default?

An API call would report that this repo has 0 enhancement issues, because we use Type: Enhancement

Wouldn't a case insensitive comparison do the job?

All I would add is Stars, Forks and Watcher.

For this request overall, I could take it or leave it.  If this gets done, it should be limited to a small set of data.

@HallerPatrick

Does the GH API allow for anonymous calls to it?

Yes, but it limits them, with certain limits.

Search API calls (e.g. is:pr is:merged) is about 10 per minute I believe
GraphQL requires authentication, I believe

If limited to stars, watchers, and forks, I think a call to /repos/:user/:repository would be sufficient. There is a limit on unauthenticated calls for this URL, but it's very high (1000 per minute, I think), so can only reasonably be reached if somebody was running onefetch in a loop.
So, to summarize, if only stars, watchers, and fork count are needed, authentication probably isn't necessary 🙂

@geeseven

Wouldn't a case insensitive comparison do the job?

For this repository, yes. Sorry, I was trying to make the point that a repo owner could use any label as their "enhancement" label, which is why an optional alias would be necessary. Somebody could even rename the enhancement label to :rocket:, for example. However, as long as the set of data doesn't include anything about issues, my concern about this is irrelevant 🙂

I will probably go for a minimal implementation and will then make some performance test and will present them to you.

All I would add is Stars, Forks and Watcher.

And these stats will be retrievable only for Git repositories hosted by GitHub (and maybe GitLab ). I'm afraid these use cases won't cover the majority of our user scenarios; Git repos are universal: people can choose to self host, use TFS, BitBucket...or stay local. I agree, this would definitely be a "Nice to have" feature. But unfortunately, it's also far from being  "cost free" (performance, complexity); Even if we

do a simple api call and look if it has internet connection and can successfully get a response from GitHub. If none of this is working just ignore it. This can also happen async too.

These will be positive for maybe 5% of the total use cases but impact 100% of them - I'm just guessing numbers here.
Besides, except for relatively big open source projects, most people don't have stars, forks or watchers on their repos.
But this is just my opinion...

I will probably go for a minimal implementation and will then make some performance test and will present them to you.

Go for it, but no guaranty this will go to master.

I see that this feature request is not well received. And I don't want to push something which does not make sense in the context of onefetchs philosophy.
I mean @spenserblack repofetch is already doing what we talked about. Maybe I can patch up a fork of onefetch, that uses repofetch as a subcommand or something.:D
I would therefore close this issue.


Multi cat behaviour

Fixes #214

@smoh for your review

(will merge anyway if I don't hear from you, as I will be on holiday for a while)

Looks good to me and please do.


weekly highlights 2023-02-16

What does this PR do?
collect contribution for weekly highlights

I have tried to give some suggestions to enhance the target user 🙏


Update to<EMAIL_ADDRESS>implement marks

There are some mechanical changes to the API:

Allow negative delete arguments for Doc.splice and Doc.spliceText
Add Doc.mark, Doc.marks and Doc.marksAt
Add the Mark and ExpandMark types
Add PatchAction.Conflict and PatchAction.Marks
Add a marks field to PatchAction.SpliceText and PatchAction.Insert

One slightly more significant change is the addition of unsafe impl Send{} and unsafe impl Sync{} on the Doc we export to uniffi. This is required because currently automerge::Automerge is not Send or Sync due to the internal use of an Rc. The unsafe impls are okay because we guarantee that all accesses of the document on the Swift side are from a single thread. In future we should be able to remove the Rc from automerge to make it Send again and remove these unsafe impls.
resolves #21

I added two (very small) commits to hopefully resolve the CI issues - one that clean up a bit of extraneous whitespace that rustfmt was whinging about, and another that I'm hoping will clear the CI test issues - we'll see after it runs. While I was doing through, I did notice that clippy tossed out a warning, but didn't know enough to know if it was important or not, so I'll tack it on here for your looksee:
warning: you should consider adding a `Default` implementation for `SyncState`
  --> src/sync_state.rs:16:5
   |
16 | /     pub fn new() -> Self {
17 | |         Self(RwLock::new(am::sync::State::new()))
18 | |     }
   | |_____^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#new_without_default
   = note: `#[warn(clippy::new_without_default)]` on by default
help: try adding this
   |
15 + impl Default for SyncState {
16 +     fn default() -> Self {
17 +         Self::new()
18 +     }
19 + }
   |

Thanks for the fixes!
Yeah that particular warning from clippy I find over-zealous. This may just be because I am a bit grumpy about the extensive use of Default in the Rust ecosystem, which I consider to be a mistake. In this case it's probably fine though, I'll add an impl just to keep clippy happy :slightly_smiling_face:


Introduce optimization variable

This PR introduces the next item of the message data model: Variable. It comes complete with everything the other items already have and only minor changes to Column that are required for this to work, so without further DRY improvements, this PR size is as small as it gets (for the remaining Equation).
Some questions and notes that come up while working on this:

We already have a Variable of the iamc category. This requires awkward naming conventions at times. I usually defaulted to OptimizationVariable for this new one, but this leads to a table name of optimization_optimizationvariable (because the docs table is built on the model.__tablename__ -- which of course could be changed by overwriting the __tablename__ locally, and maybe that's preferable) and to OptimizationVariable being importable from ixmp4.core, which I'm not sure we like. Please let me know what you think.
How many levels and marginals do Variables (and Equations, for that matter) have? At the moment, I'm assuming that every Key (one row of entries in Variable.data) can have its own level and marginal, making those two just arbitrary columns in Variable.data (though maybe we could limit them to being floats or so).
Similarly, I'm wondering: do Variables need to get data added to them manually via an add() function? Or is there data sort of filled in automatically by the gdx code? I don't think I remember lots of variable.create(), variable.add_data() functions from the westeros tutorials.

Working on the transport tutorial, I noticed that I had completely forgotten about adding variables = db.relationship() in the Run DB-table. I have even run migrations for these other PRs, but no error came up. I'm wondering if we even need these relationship() declarations at all.

Looking at the transport tutorial, I noticed that it includes a variable like this
Variables
     z       total transportation costs in thousands of dollars ;

I.e, without constraints to indexsets. So I set out to make constrained_to_indexsets optional and in the process, stumbled upon some more questions:

For now, variable.columns for an unconstrained variable returns []. (Technically, one might consider levels and marginals to be columns, but we only call things columns that we can bring into optimization.Column form, i.e. constrain to an IndexSet.) We could rework this to return None instead, but at the moment, variable.data returns {} if it's empty since the JsonDict type requires a dictionary, and not None, so [] is consistent with that. So do we want to keep it this way?
Should there be a documented workflow for adding Columns to a Variable later on? I guess one could already call the private backend.optimization.variables._add_column(), but it is private, so not ideal to recommend as a default workflow. Once we have a delete() function, we could recommend delete() and create() again. But is there even a need, do people create Variables forgetting that they actually have dimensions and should be constrained to them?
I was thinking of introducing a test like      # Test that giving column_names, but not constrained_to_indexsets raises
      with pytest.raises(ValueError, match="Received `column_names` to name columns, "
                         "but no `constrained_to_indexsets`"):
          _ = test_mp.backend.optimization.variables.create(
              run_id=run.id,
              name="Variable 0",
              column_names=["Dimension 1"],
          )

But then I realized that this is a special case of column_names and constrained_to_indexsets not having the same length. So we would catch this error any way, but with a dedicated if check and error message, we could serve people running into this a specific error message, which they might need if they have misunderstood the exact relation and nature of column_names and constrained_to_indexsets. Would we want to include this or is it enough to remind them that the two parameters need to share the same length (if both are present)?

Context for the latest commit: typing.Never can be used to type hint empty lists. Unfortunately, it was only introduced with python 3.11. However, typing.NoReturn was introduced with 3.6.2 and these are equivalent according to the docs.

Hi I investigated the ValueError comment further:
I created a test platform with ixmp4 platforms add test.
Then I started the ixmp4 service in non-manged mode: IXMP4_MANAGED=false ixmp4 server start
Then I executed this test script:
import ixmp4
from ixmp4.conf import settings
from ixmp4.conf.auth import SelfSignedAuth
from ixmp4.conf.manager import PlatformInfo
from ixmp4.data.backend import RestBackend

rb = RestBackend(
    PlatformInfo(name="test", dsn="http://localhost:9000/v1/test/"),
    auth=SelfSignedAuth(settings.secret_hs256),
)

mp = ixmp4.Platform(_backend=rb)

run = mp.runs.create("Model", "Scenario")
ids = mp.backend.optimization.indexsets.create(run_id=run.id, name="Indexset 1")

_ = run.optimization.variables.create(
    "Variable 0",
    constrained_to_indexsets=[ids.name],
    column_names=["Dimension 1", "Dimension 2"],
)

Which results in the following two interactive terminal outputs:

Showing us that the ValueError, as expected, is not propagated to the client.
For now you can fix this by making a custom ixmp4 exception, but we should investigate why the test setup cannot reproduce this (i assume because the TestClient class does not prevent server side exceptions being propagated to the client) and how to fix it.

Thanks for the instructions and for catching this error! It was fixed in #115 and this branch rebased/reworked to accomodate these changes :)


Tombs of Amascut Weapons Case#'s need to be added

Tumeken's Shadow
Osmumten's Fang

added on PR https://github.com/winterdaze/tick-counter/pull/18 thank you


pip check fails on amd64 linux and macOS

👋🏼 Hi, it looks like pip 24.2 causes ninja to register as incompatible with my local OS due to something in the package's metadata tags.
Environments tested (all amd64):

Python 3.9 through 3.12 on WSL2 (Ubuntu 22.04 LTS)
Python 3.9 on GitHub Actions runners (both Ubuntu 22.04 and 24.04)
Python 3.9 on macOS

Steps to replicate:

Create a new virtualenv (python3.12 -m venv .venv)
pip install --upgrade pip
pip install ninja
pip check

Expected behavior:

Output saying there are no package conflicts found

Actual behavior:

ninja <IP_ADDRESS> is not supported on this platform

I'm running into this error as well on GitHub Actions with Ubuntu 22.04.4 LTS, Python 3.8.18 or 3.10.14, and pip 24.2.
I'm able to work around this error by downgrading pip to 24.1.2.

A coworker confirmed that macOS on ARM is also affected, and downgrading pip to 24.1.2 does indeed work around it.

See pypa/pip#12884 for the root of the issue.
The current wheel for ninja <IP_ADDRESS> includes a blank line in the WHEEL file which makes the pip check break.
Can we get a new release of ninja that fixes the contents of WHEEL metadata to get this fixed? Thanks!

Should be fixed in <IP_ADDRESS>


Bugs with referencing parent selectors

There are some cases that work ok in other pre-processors and it would be cool to support them in Sass too:
Concatenating with elements
When I write 'p&' I expect the & to expand to the parent selector, so I could narrow the class to the element and from
.class {
  color: lime;
  p& {
    color: red;
  }
}

I would get the
.class {
  color: lime;
}
p.class {
  color: red;
}

Right now I get the error “"&" may only be used at the beginning of a selector.” — I think that such limitation is unneeded.
Concatenating the parent selector with the postfix
When I write &_disabled I expect it to expand to concatenate with the selector, so I could write
.class {
  color: lime;
  &_disabled {
    color: red;
  }
}

and get this:
.class {
  color: lime;
}
.class_disabled {
  color: red;
}

But right now I get this (the space occurs between the postfix and selector):
.class {
  color: lime;
}
.class _disabled {
  color: red;
}

Duplicate of #282

@nex3 ok, I've understood the arguments against prefix& case, but don't the &postfix is different? There are a lot of techs that are heavy class-based due to rendering speed optimizations, so having such feature would be great.
Also, why other preprocessors do this and you can't :)

The answer to both of your questions is that Sass has a very thorough understanding of selectors. It doesn't treat them just as strings that can be sliced and diced; it fully parses them and understands their structure. A side effect of this is that &, Sass's addition to the selector syntax, needs to fit into the standard parsing model of selectors. The examples you're citing involve having & mean "replace this text with the text of the parent selector", but Sass instead thinks of it as "replace this simple selector with the parent selector". It's a subtle distinction, but it's important for features like @extend that use the full selector parse tree, as well as for SCSS being a true superset of CSS3. These are things that other preprocessors don't really worry about.
Chris and I are talking about ways to address these use cases (see #286). If we end up allowing the use of & in SassScript, you'll be able to do your examples as so:
.class {
  color: lime;
  p#{&} {
    color: red;
  }
}

.class {
  color: lime;
  #{&}_disabled {
    color: red;
  }
}

#{&} would be better than nothing, so I'll look forward to it, thank you!


How to make 'Card works like Suggestion' in dialogflow?

The text in the Suggestion helps to move the next intent that has the text of Suggestion.
new Suggestion(Option1)
But I want to make this same thing with Card button.
How can I do it?
new Card {
    "buttonText": "Option1",
    "buttonUrl": "????????????",
    "imageUrl": "https://xxxxxxx.jpg",
    "text": "Text1",
    "title": "Title1"
},

Thanks!

SOLVED
Just put a text as buttonUrl, and in Messanger it is treated as a suggestion.


wip: unit test setup

What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR adds support to use catch2 as a testing framework, initially it allows only to test falco_engine but it will be expanded to other areas as soon as we extract functionalities into libraries.
There's no actual tests in this PR since this one is to setup the environment that will then be used in other PRs checking in the tests.
Here's how a unit test will look like:
#include <catch.hpp>

using namespace std;
using Catch::Matchers::Equals;

TEST_CASE("someFunction check if false is false") {
    REQUIRE_FALSE(false);
}

Tests will be organized under the tests folder.
For example, falco_engine tests will go under tests/engine using the same structure as for userspace.
For every other type of assertion and details, refer to the the catch2 documentation.
Which issue(s) this PR fixes:
Fixes #686
Special notes for your reviewer:

Does this PR introduce a user-facing change?:
new: unit test setup

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fntlnz
If they are not already assigned, you can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold
I propose to hold it until we integrate them in Travis.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 01cc25da86d7eecea8dd41ecfef828caa2171488


Add support for using Pipeline CSS/JS packages in forms and widgets.

Django allows forms and widgets to define the CSS or JavaScript files
needed on the page, automatically including those files in the
administration UI. This works fine for standalone media files, but
weren't an option for Pipeline packages.
This change introduces a branch new PipelineFormMedia class that a
form or widget's Media class can inherit from. The Media class can then
define css_packages or js_packages attributes that work just like
the standard css/js attributes but reference package names instead
of individual files.
Upon accessing the css  or js attributes, the files defined in the
packages will be collected/processed just like with the template tags
and the resulting filenames returned for use in the page.
Using this, any page compatible with Django's form media support will
work automatically with any form or widget that wants to use Pipeline
packages, without any modifications to templates or any other
workarounds that were needed before.
This implements issue #154.

Looks good! Would you mind adding a section to the docs too?
Thanks in advance!

You know, I meant to do that but had some other things take my attention away from it, and just forgot. I'll try to work on docs tonight.

No worries, and no hurry 😉

Updated to include docs. Let me know if you want any changes made to that.

Thanks! :)
I'm also working on a change for the upcoming release to make the Collector a little more self-managing and less dependent on an HttpRequest. Basically, the form media generation process doesn't have access to the request, and this means we can trigger collection a few times.
The plan for this work is to have the collector track what it's collected, and to be smart about whether it needs to collect again. It will listen for the signals from Django indicating when a request has begun and ended, and use those to clear state. On top of this, it will be more thread-safe than the old design (multiple threads all trying to collect at the same time won't stomp on each other like they currently can).

Sounds awesome! Thank you!


BE-802 Release v1.1.2

Signed-off-by: Atsushi Neki<EMAIL_ADDRESS>

Before this release, we need to apply the following package upgrade to Explorer (client).
https://github.com/facebook/create-react-app/issues/9469#event-3648594619


adding a hypercube transform function

With @vhaasteren's ppf additions to parameter.py, I've written a short function to transform from a unit hypercube to the prior space for nested sampling

Hi @astrolamb, the added hypercube transform looks fine to me. It just needs some added unit tests. I'm going to be a bit explicit about what else needs to be done. My explanation is probably more lengthy than what you need to do :)
However, the PR should probably be redone a bit. Ideally, when there is code you want to add upstream with a PR, you would start off with a fresh branch based on the dev branch. Then you add your code there, upload it to github under its own branch, and file a PR. Then your PR is clean and the diff is with respect to the dev branch. No conflicts.
Here you have started from a branch that diverges from Enterprise dev. You can either take your changes and transfer them to a fresh branch manually (easiest in your case), or use the git cherry-pick command to transfer your commits. You would do that by identifying your commit hashes using git log (you have 4 commits that are relevant), and then adding them to your fresh branch based off the dev brach like so:
git cherry-pick commit-hash
You do that for all relevant commits, in order. In your case, I would just copy over the function and file a new PR.
Anyway, extra complication is that your PR is actually based off of a PR of mine that is not yet merged, so you can't even file the PR yet. I could add your hypercube transform to my PR, or we can bother @AaronDJohnson to hurry up and review my PPF additions in #353, which is ready for review/merge. Or, yet another option is for you to file the PR against my own github ppf branch on github.com/vhaasteren/enterprise, which is the branch that is on queue for merge in Enterprise.
Lots of options for you. Anyway, for now your first order of business is to see if you can come up with a unit test. In tests/test_parameter.py you can see how unit tests are written. The parameter tests are executed with:
pytest tests/test_parameter.py
Those are also automatically checked by the github CI system, and your new code needs to be covered before it is allowed to be merged. If there are (too many) untested lines of code, the system will not allow you to merge.

I've improved this PR in PR #389. I will close this original one


Import the experimental biaffine parser

Imports the biaffine parser with its full git history.

One remaining issue is that the overfitting tests sometimes fail. I think the issue is that the bilinear are sometimes hard to fit with so few examples.

Just install it by hand for the CI and add an extra to the setup as for thinc.


Requires complete lodash package

Now it compiles with the complete lodash package (540 kB), that is, from my point of view, is unnecessary.

Hmmm... May be I can build a custom version of lodash which only contains the methods used by this library... Do you include the plugin.js directly, or use this library via npm and webpack?

I use webpack and by the way I already forked your project and altered it in this part by cherry-picking the  lodash methods used in the project. Now I need to test it a bit and I'll make a PR to your repo shortly.

Great to hear that! Thanks!

PR merged.


Blazor Support

Is Blazor Support possible?
What could a TimeProvider look like? There is no direct dispatcher and the StateHasChanged method is not very granular. I could still see there being a benefit in having something to automatically call StateHasChanged based on property updates or similar though.

I see, Blazor also seems to have a SynchronizationContext, so it may be possible to do something.

I've implemented SynchronizationContextTimeProvider for Blazor.
StateHasChanged should be executed manually.

I've introduced AddBlazorR3 for default TimeProvider as request scoped.
https://github.com/Cysharp/R3?tab=readme-ov-file#blazor

I've implemented SynchronizationContextTimeProvider for Blazor. StateHasChanged should be executed manually.

I think this is great! Having StateHasChanged called manually seems like a reasonable expectation since we don't have a better way to manage it and it would be better to be careful with how many updates we push there.


problem with cu3  gate

and i have a problem when we will use yy gate and ms gate.

Hi @XIECHO thanks for reporting, it will be fixed asap. (You can help/speedup by reporting more details.)

i think  the parameter is not right . aha

the cu3 matrix

@XIECHO What issues you have with YY gate? It looks good to me.

yy is good. i just want to know yy means , and when we will use yy gate.
I am sorry that makes you confused.
发自我的iPhone
------------------ Original ------------------
From: Petar Korponaić<EMAIL_ADDRESS>Date: Thu,Jul 18,2019 4:39 PM
To: perak/quantum-circuit<EMAIL_ADDRESS>Cc: XIECHO<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [perak/quantum-circuit] problem with cu3  gate (#26)
@XIECHO What issues you have with YY gate? It looks good to me.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@XIECHO OK, published. Please update quantum-circuit package to version 0.9.94.
XX, YY and ZZ gates are implemented natively in trapped-ion quantum computers. It is added by user request - I am not using it as well :)


OWNERS: remove filters

What this PR does / why we need it:
filters are not handled by the approve plugin.
https://www.kubernetes.dev/docs/guide/owners/#filters
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note:

NONE

/approve
/lgtm

Many thanks!


NER example: fix divisions by zero

Description
Prevents divisions by zero in the NER example
Checklist
Essentials

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[X] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[X] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ X ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ X ] fix divisions by zero in NER metrics

Comments

this change is not backward incompatible

@mxnet-label-bot add [Example, Python, pr-awaiting-review]


chore(package): update model-attributes to version 2.2.2

https://greenkeeper.io/

Current coverage is 76.15% (diff: 100%)

Merging #195 into master will increase coverage by 14.56%

@@             master       #195   diff @@
==========================================
  Files             1          1
  Lines           151        151
  Methods          22         22
  Messages          0          0
  Branches         20         20
==========================================
+ Hits             93        115    +22
+ Misses           58         36    -22
  Partials          0          0

Powered by Codecov. Last update ab77a1b...8d653b6


Deliverable Planning Meeting, Scheduled, Wednesday, September 27, 6:00 pm, Discord

Topics

[ ] Go over ideas in the Google Doc. Everyone grab topics in the SRS of interest. Breakdown tasks for writing SRS.
[ ] Report ML paper findings.
[ ] Sync up WebRTC research.
[ ] Plan for next meeting.

Attendance

[ ] @qstommyshu
[ ] @yex33
[ ] @marischan888
[ ] @jiaoa3
[ ] @KehaoHuang

see which team do we want to have for our primary reviewers.

see which team do we want to have for our primary reviewers.

Sb

Decisions Made

Write the four books in the Meyer’s template in the following order: Environment -> Goals -> System -> Project
[ ] @qstommyshu, @jiaoa3 Environment (due by next meeting)
[ ] @marischan888, @KehaoHuang Goals (due by next meeting)
[x] ~Ask Jason to review our capstone project.~ Pending reply


RefreshIndicator, show without launch onRefresh

Is there any way to achieve this behavior?

@JavierSegoviaCordoba I would advise you to post it on StackOverflow as well as help there may come faster.

@BondarenkoStas I think this issue should be marked as feature request

Hello @BondarenkoStas, do you know when this is going to be implemented or if there is a way to "hack" it until it is added?

If you use a globalKey you can show / hide it as you want/wish
Here is an example using flutter_hooks (but you can do the same with statefulWidget i you want)
final refreshKey = useMemoized(() => GlobalKey<RefreshIndicatorState>());
 useEffect(() {
      refreshKey.currentState.show();
      return null;
    }, const []);

return  RefreshIndicator(
        key: refreshKey,
        onRefresh: () {
          return store.loadScenes();
        },
        child: ......
);

@jaumard the problem is it launches the onRefresh function, so store.loadScenes() will be executed.
I need a way to only show it without launch onRefresh.

ho ok I understand now, but why would you like to show it without calling the onRefresh ? that's the all point of the widget. Why would you need to show the loading without actually loading the data. Maybe you need to change what's under the onRefresh instead ?

Because the loading state can be attached to other requests, so I pref observe a variable "isRefreshing" to show/hide the refresh indicator.

So basically you want a RefreshIndicator that doesn't trigger any refresh when you pull on it ? Did I get it right ?
Something like:
RefreshIndicator(
        key: refreshKey,
        isRefreshing: true/false,
        child: ......
);

Because the loading state can be attached to other requests, so I pref observe a variable "isRefreshing" to show/hide the refresh indicator.

hmm seems like we should hack our code so that we can handle more than one request without calling code on method called by onRefresh
In my case, I use onRefresh in RefreshIndicator to call news list,, and then I also have search request and want to show the loading showed by RefreshIndicator if user want to search something news.  The problem is after search request ran, then the RefreshIndicator is also call the news list API. This make the app didn't show the news by search keyword,, the app always show the default news list.
Because of seems like this issue is still don't have any solution to prevent RefreshIndicator calling code on onRefresh,,, so I "hack" it by changing my code become like this. Another request (search request) will also call _refreshData(true, "keyword").
LiquidPullToRefresh(
        key: _refreshIndicatorKey,
        child: _buildListNotif(),
        onRefresh: () async => _refreshData(),
        color: AppColors.liquidRefreshCircle,
        backgroundColor: AppColors.liquidRefreshBgColor,
        showChildOpacityTransition: false,
        springAnimationDurationInMilliseconds:
            AppConstant.liquidRefreshDuration,
        height: AppDimen.liquidRefreshHeight,
      ),
.
.
.
.

_refreshData({bool wannaSearchList, String keyword}) {
   if (wannaSearchList != null) {
       if (wannaSearchList) {
            // call search news API
       } else {
           // call default news API
      }
  } else {
     // call default news API
  }
}

*note : LiquidPullToRefresh behavior is same with RefreshIndicator

Yes, sorry I forgot to reply :/


Some footprint/symbol name will break the file

Symbol name containing whitespace characters and footprint name containing parentheses should be quoted by "".

Thank you for reporting this issue.
I corrected it, tell me if it works for you.
If not, please give me the part number you are using so I can reproduce the issue.

Oh, thank you. It is ok now.
But there is another issue: these imported symbols are dismantled to several parts.
Following screenshot is one of them LOL.
Part #: C2290, C72038

Yes I know the schematic generation is far from ideal, but for those components I recommend you to use the schematic symbols from the standard library, and then link the imported footprint to them.
I will work on this in the future when I have time.
For now, I think the initial issue has been solved, so I am closing this issue. Do not hesitate to reopen it if you still encounter this issue.


Sorry, don't know how to check guest version of Virtualbox Guest Additions on this platform

 vagrant reload
==> default: Attempting graceful shutdown of VM...
==> default: Checking if box 'mcandre/windows-amd64' version '0.0.1' is up to date...
==> default: Clearing any previously set forwarded ports...
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
    default: Adapter 1: nat
==> default: Forwarding ports...
    default: 5985 (guest) => 55985 (host) (adapter 1)
    default: 5986 (guest) => 55986 (host) (adapter 1)
    default: 22 (guest) => 2222 (host) (adapter 1)
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
    default: WinRM address: <IP_ADDRESS>:55985
    default: WinRM username: vagrant
    default: WinRM execution_time_limit: PT2H
    default: WinRM transport: negotiate
==> default: Machine booted and ready!
Sorry, don't know how to check guest version of Virtualbox Guest Additions on this platform. Stopping installation.
==> default: Checking for guest additions in VM...
    default: No guest additions were detected on the base box for this VM! Guest
    default: additions are required for forwarded ports, shared folders, host only
    default: networking, and more. If SSH fails on this machine, please install
    default: the guest additions and repackage the box to continue.
    default:
    default: This is not an error message; everything may continue to work properly,
    default: in which case you may ignore this message.
==> default: Rsyncing folder: /Users/knarf/projects/Portal/ => /cygdrive/c/vagrant
==> default: Machine already provisioned. Run `vagrant provision` or use the `--provision`
==> default: flag to force provisioning. Provisioners marked to run always will still run.

Vagrantfile:
# -*- mode: ruby -*-
# vi: set ft=ruby :
Vagrant.configure("2") do |config|
  config.vm.box = "mcandre/windows-amd64"
  config.vm.provider "virtualbox" do |vb|
    vb.gui = true
  end
end

Same with config.vm.box = "gusztavvargadr/iis".
Is this plugin even supposed to work on Windows guests?

I am with the same problem on Windows.

v0.20.0 was just released with some support for windows guests.
From: Jacqueline Costa notifications<EMAIL_ADDRESS>Reply: dotless-de/vagrant-vbguest reply<EMAIL_ADDRESS>Date: 27. September 2019 at 19:42:37
To: dotless-de/vagrant-vbguest vagrant-vbguest<EMAIL_ADDRESS>CC: Subscribed subscribed<EMAIL_ADDRESS>Subject:  Re: [dotless-de/vagrant-vbguest] Sorry, don't know how to check guest version of Virtualbox Guest Additions on this platform (#348)
I am with the same problem on Windows.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/dotless-de/vagrant-vbguest/issues/348?email_source=notifications&email_token=AAADEJ2NFIFD5VXEK2AN6Y3QLZAY3A5CNFSM4IUYBO2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7ZTWSQ#issuecomment-536034122, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAADEJ2FPMV5252R5Z45IETQLZAY3ANCNFSM4IUYBO2A.


Error: Bad Request on try authentificate in Google Apps account

Error: Bad Request
Your client has issued a malformed or illegal request.

Same error here

This might be caused if you're authenticated in google with multiple accounts. Logout from gmail, login again with only one account and will work. See http://stackoverflow.com/questions/30511843/google-oauth2-returns-bad-request-when-logged-with-multiple-accounts


Request to update the Notification Hub tutorial document

Please update the tutorial with the latest information. This tutorial isn't user-friendly.
Especially , this completed code is not completed one since some library remains old and we cannot build it as it is on the latest android studio .We also hope to fix Connect your app to the notification hub Part more detail.
Thanks

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

ID: 6fd4915d-64a9-b703-f139-91338412825a
Version Independent ID: e731d0e6-e063-d209-7faf-82db773e5d30
Content: Send push notifications to Android using Azure Notification Hubs and Firebase SDK version 1.0.0-preview1
Content Source: articles/notification-hubs/android-sdk.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@kekatahi Thanks for the feedback! I have assigned the issue to the content author to review further and update the document as appropriate.

As we are moving to a new feedback system, please reopen this issue or contact Microsoft Support if you still want to provide feedback on the documentation. Thank you! #please-close


fatal error: 'google/sparse_hash_map' file not found

I have installed sparsehash without any troubles. While trying to build fast_align getting the following error:
error:/Users/shreesh/Project/fast_align-master/src/hashtables.h:5:10: fatal error: 'google/sparse_hash_map' file
      not found
#include <google/sparse_hash_map>
         ^~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
make[2]: *** [CMakeFiles/fast_align.dir/src/fast_align.cc.o] Error 1
make[1]: *** [CMakeFiles/fast_align.dir/all] Error 2
make: *** [all] Error 2

I tried manually changing the paths in hashtables.h but still couldn't get it to work.

Missed sparsehash library in the path.
Set CMAKE_CXX_FLAGS option(in CMakeLists) to:
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -std=c++11 -O3 -g -I/usr/local/include")
instead of:
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -std=c++11 -O3 -g")

Meet the same problem, tried to set the CMAKE_CXX_FLAGS as above. But it doesn't work for me.

not working for me too? any update?

fixed the issue as @mcavdar suggested.
Set CMAKE_CXX_FLAGS option to
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -std=c++11 -O3 -g -I/usr/local/include", where /usr/local/include should be the path where the SparseHash was found from the output of cmake ../  command.
Clear the cache by deleting the build directory, then start again.

Meet the same problem, tried to set the CMAKE_CXX_FLAGS as above. But it doesn't work for me.

On macOS sparsehash might be on another path, e.g. set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -std=c++11 -O3 -g -I/opt/homebrew/include")


Make aggregationTemporalitySelector an optional property

We should make aggregationTemporalitySelector an optional property on the PushMetricExporter interface, and add logic to the MetricReader to default to AggregationTemproality.CUMULATIVE for all instruments if `aggregationTemporalitySelector is not there.
This would reduce duplicated code in third party exporters, while still allowing the exporter to override the default.

See also: https://github.com/open-telemetry/opentelemetry-js/pull/3153#discussion_r957431579

Requires #3153 to be merged first.

I can work on this. I'll self assign. :slightly_smiling_face:


fix: convar rename

Thank you for your contribution. There was a large update of the docs with the new sdk rewrite, this PR is no longer required. I will be closing this pull request. If you have any questions, please let us know.


1.2.11 table not exist

I have problem with version 1.2.11, in this commit: https://github.com/noam148/yii2-image-manager/commit/458d69c60a7034772daeba5255edf61123810050#diff-4d288fa17e65c8738aafbec4aabdb456
you change {{%imagemanager}} to {{%ImageManager}},
in windows server the plugin run ok , but in linux plugin crash because not find the table in db, i think it's because unix systems are case-sensitive. I was forced to go back to the version 1.2.10. Excelent plugin, regards

Please update your table name to ImageManager instead of imagemanager. It's update by accident to lowercase. That's the reason is update it back to ImageManager. Sorry for the confusion.


Request new feature: Auto merge PR of dist-git once CI passed

By providing a job(e.g. auto_dist_git_pr_merge) which automatically merge dist-git PR based on CI results, we can have a human-free auto build system:

pull_from_upstream create dist-git PR.
auto_dist_git_pr_merge merge PR after CI passed.
koji_build create rpm build on dist-git commit.
bodhi_update create bodhi on koji build finish.

Hi, we are already tracking this here: https://github.com/packit/packit-service/issues/1723
Feel free to comment there and let me close this issue as duplicate.


RuntimeError: gather(): Expected dtype int64 for index

Hi, I got an error during validation step, here is a traceback:

As I understand, the variable perm_mask should be a tensor with integer values, but it is obtained from division in self.num_nodes in advance method as follows:
prev_k = bestScoresId / self.num_nodes
...
perm_mask = prev_k.unsqueeze(2).expand_as(self.mask)
self.mask = self.mask.gather(1, perm_mask)

So it looks like it cannot work in any case. Any help will be highly appreciated.

I have found a solution in this issue https://github.com/chaitjo/graph-convnet-tsp/issues/16:
prev_k = bestScoresId // self.num_nodes
Please validate if it is correct for torch 1.10. It will also be great to fix this issue if it is correct for your setting Ubuntu 16.04, using Python 3.6.7, PyTorch 1.2.0 and CUDA 10.0. It can help people who trying to run the code on modern versions.

Hi @vpozdnyakov, thank you for your interest!
Yes, the issue you have referenced can be a fix if you are using the latest PyTorch.
This codebase + empirical results were obtained using an older version of PyTorch 1.2, where I do not recall encountering the same issue. At the moment, I can consider updating the codebase(s) for my TSP projects, but I do not have the bandwidth and resources to actually test these changes or ensure that the results do not change dramatically. What do you suggest?


Incorrect metric data [not matching the tag key value] is pulled for AWS/CertificateManager

Is there an existing issue for this?

[X] I have searched the existing issues

YACE version
Tried both v0.51.0 and v0.52.0
Config file
  apiVersion: v1alpha1
  discovery:
    jobs:
    - type: AWS/CertificateManager
      regions:
        - us-west-2
      period: 300
      length: 300
      nilToZero: true
      metrics:
        - name: DaysToExpiry
          statistics: [Minimum]
      searchTags:
        - key: platform
          value: kube-srvyace-usw2-plt1
      dimensionNameRequirements:
        - CertificateArn

Current Behavior
I have the configuration to pull metrics of different services from cloudwatch with same tag key and value and its working fine, except for AWS/CertificateManager.
Yace is installed successfully , there are no errors/warning.
aws_certificate_manager_info has right and precise data which is matching searchTags details but the metric DaysToExpiry with value i.e., aws_certificate_manager_days_to_expiry_minimum is displaying data that does not match with tag key-value. In specific it is displaying data of all CertificateArn present in us-west-2 region which is not matching with tag value.
Even tried with exportedTagsOnMetrics but result remains same.
  apiVersion: v1alpha1
  discovery:
    exportedTagsOnMetrics:
      AWS/CertificateManager:
        - platform
    jobs:
    - type: AWS/CertificateManager
      regions:
        - us-west-2
      period: 300
      length: 300
      nilToZero: true
      metrics:
        - name: DaysToExpiry
          statistics: [Minimum]
      searchTags:
        - key: platform
          value: kube-srvyace-usw2-plt1
      dimensionNameRequirements:
        - CertificateArn

Please find the output:
aws_certificate_manager_info has right data.
-> aws_certificate_manager_info{name="arn:aws:acm:us-west-2:*********:certificate/***************adb1",tag_Product="kube",tag_platform="kube-srvyace-usw2-plt1"} 0
aws_certificate_manager_days_to_expiry_minimum has wrong data i.e., value that is not matched with tag is displayed .
-> aws_certificate_manager_days_to_expiry_minimum{account_id="",dimension_CertificateArn="arn:aws:acm:us-west-2:***:certificate/****************099e",name="global",region="us-west-2",tag_platform=""} 0
Expected Behavior
aws_certificate_manager_days_to_expiry_minimum{account_id="",dimension_CertificateArn="arn:aws:acm:us-west-2:***:certificate/***************adb1",name="global",region="us-west-2",tag_platform="kube-srvyace-usw2-plt1"} 0
No response
Steps To Reproduce
No response
Anything else?
No response

Any help/update please

i meet the same problem，the search tag config is useless at acm monitor.

AWS/Backup appears to have the same problem.


Quick question about the source code

I'm a novice and trying to get into AutoHotkey. I feel like this tool would be the perfect way to get my foot in the door, just because I really like it, and even more so would like to "make it my own" by customizing it, adding features, etc.
I am exploring the source code just for the sake of familiarizing myself with it and getting a grasp on the structure.  I noticed this file is formatted differently than the rest:

It looks like the majority of the code in this file resides on one line. Is there a particular benefit/purpose to structuring it this way? Maybe to be parsed later?

If you read the documentation, it says that I use an Excel Macro to manage my code for this portion. There is no advantage to writing everything on one line.

I did read that - I just didn't make the connection. Thanks!


The route does not change it's state when changing the address directly in browser's address bar.

Hi.
I have, for example, two next files:
//parent-router.ts
import {Router, RouterConfiguration} from 'aurelia-router';
export class ParentRouter{
    router:Router;
    configureRouter(config:RouterConfiguration, router:Router) {
        config.map([
            {
                route: '',
                name: 'items-list',
                moduleId: './items-list',
                nav: true,
            },
            {
                route: 'update/:itemId',
                name: 'item-edit',
                moduleId: '../item-edit/item-edit',
                nav: false,
                title: 'ItemEdit'
            }
        ]);
        this.router = router;
    }
}

//child-router.ts
import {RouterConfiguration, Router} from "aurelia-router";
export class ChildRouter {
    router:Router;
    configureRouter(config:RouterConfiguration, router:Router) {
        config.map([
            {
                route: '',
                name: 'item-details',
                moduleId: './item-details/item-details',
                nav: true
            },
            {
                route: 'item-deep-details',
                name: 'item-deep-details',
                moduleId: './item-deep-details/item-deep-details',
                nav: true
            }
        ]);
        this.router = router;
    }
}

The routes from the last file are used for a navigation.
Now the situation:

I go to the address /update/12.
Then I change the address to /update/13 directly(manually) in the browser's address bar and press enter.
I get the updated information about my item on my page.
But. The links of my navigation, that generated this way:

<li repeat.for="row of router.navigation">
   <a href.bind="row.href">row.title</a>
</li>

have not been updated.
And even more, even if we look at the router.navigation, it will have old value.
Could you, please, look at this issue?
Thank you.

I believe this was already reported and fixed.


I changed the video to another video and the other video has another extension witch is mp4 and a problem happened.

Here is the line of code where it says that there is a problem.
let videoURL: NSURL = NSBundle.mainBundle().URLForResource("drone", withExtension: "mov")!
When I ran the app this appeared in the debugger section "fatal error: unexpectedly found nil while unwrapping an Optional value".
How can I fix this?

The URLForResource:_:withExtension: function returns an optional, which may be nil if the file you're trying to load doesn't exist in the bundle. You can prevent the crash by either

Ensuring that drone.mov exists before running your code, in which case the force unwrap will work just fine or
Changing line 21 to an if let to allow it to fail gracefully.


fix filename case errors in app-complete-sample

The app-complete-sample won't start on Linux due to casing discrepancies, this PR fixes it.

@clemos  - could you please to sign our Contributor License Agreement?


fixed line overlap

Problem
What is the problem this work solves, including
Link to Asana Ticket
Solution
Created border instead of divider class
Change summary:

Tidy, well formulated commit message
Another great commit message
Something else I/we did

Dev-Ops
If you added an ENV variable, please check off that you've updated the following

[ ] .env & .env.sample
[ ] webpack.config.js
[ ] GCP Secret Manager
[ ] Github Secrets
[ ] GithubActions Workflow

Steps to Verify:

A setup step / beginning state
What to do next
Any other instructions
Expected behavior
Suggestions for testing

Screenshots (optional):
Show-n-tell images/animations here

@PriyaRajarathinam - Cool! It's looking good so far - I made a few comments below on some of my suggestions:

[ ] Instead of using the dotted borders, do you think we can omit that to be a little more accurate to the Figma design?

[ ] If we can touch up the styling to be a little lighter too, I think that could look a bit more appropriate.

[ ] See if we can use something like text-width too to align the borders correctly also.

@PriyaRajarathinam - Cool! It's looking good so far - I made a few comments below on some of my suggestions:

[ ] Instead of using the dotted borders, do you think we can omit that to be a little more accurate to the Figma design?

[ ] If we can touch up the styling to be a little lighter too, I think that could look a bit more appropriate.

[ ] See if we can use something like text-width too to align the borders correctly also.


Revert "Added feedback form and sitemap to the footer"

Reverts shravan20/github-readme-quotes#323

@RajatX24 : Reverting since the FE is breaking; can you please debug it once?


Lua errors after updating

Describe the bug
Pulled the latest version from master (3.32) was on 3.24 previously. Add-on is broken with the following error
Message: ...terface/AddOns/LFGBulletinBoard/CustomCategories.lua:95: attempt to index field 'CustomFilters' (a nil value)
Time: Mon Jul  1 11:11:12 2024
Count: 1
Stack: ...terface/AddOns/LFGBulletinBoard/CustomCategories.lua:95: attempt to index field 'CustomFilters' (a nil value)
[string "@Interface/AddOns/LFGBulletinBoard/CustomCategories.lua"]:95: in function `InitializeCustomFilters'
[string "@Interface/AddOns/LFGBulletinBoard/GroupBulletinBoard.lua"]:541: in function `Init'
[string "@Interface/AddOns/LFGBulletinBoard/GroupBulletinBoard.lua"]:857: in function `?'
[string "@Interface/AddOns/LFGBulletinBoard/LibGPIToolBox.lua"]:117: in function <Interface/AddOns/LFGBulletinBoard/LibGPIToolBox.lua:114>

Locals: (for generator) = <function> defined =[C]:-1
(for state) = <table> {
 RDF = <table> {
 }
 BLOOD = <table> {
 }
 BOOSTS = <table> {
 }
 INCUR = <table> {
 }
}
(for control) = "RDF"
key = "RDF"
preset = <table> {
 sortIdx = 1
 levels = <table> {
 }
 name = "Random Dungeon"
 tags = <table> {
 }
 isDisabled = true
 key = "RDF"
}
(*temporary) = nil
(*temporary) = <table> {
 sortIdx = 1
 levels = <table> {
 }
 name = "Random Dungeon"
 tags = <table> {
 }
 isDisabled = true
 key = "RDF"
}
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index field 'CustomFilters' (a nil value)"
presets = <table> {
 RDF = <table> {
 }
 BLOOD = <table> {
 }
 BOOSTS = <table> {
 }
 INCUR = <table> {
 }
}
isModuleInitialized = false

To Reproduce
Steps to reproduce the behavior:

Go from 3.24 to 3.32 and login

@juemrami for visibility


Create adoc for boostlook testing

Boostlook needs a CI setup where on PR or push to master/develop it renders the sample.adoc into HTML and publishes it, so at any time we can look at the master or develop rendering and know what the latest style looks like.
../../../b2 cxxstd=20 toolset=gcc-13 define=BOOST_CHARCONV_RUN_BENCHMARKS STL_benchmark linkflags="-lfmt" -a release .
needs to  go into a code block and we want to see that in the sample.adoc
This sample should include all options used in the adoc.

@rbbeeston sample.adoc is input or output?   Could you provide a preview/build.sh script that builds what you would like to see?  What is the input, and what is the output?  Everything could go in a preview/ folder.

if we want to do this right, there should be a Jamfile which imports the boostlook module

develop.boostlook.cpp.al
master.boostlook.cpp.al


build: Update Vulkan-Headers to 1.2.186

CI Vulkan-ExtensionLayer build queued with queue ID 9120.

CI Vulkan-ExtensionLayer build # 31 running.

CI Vulkan-ExtensionLayer build # 31 passed.


TypeError: process.nextTick is not a function

I have used it in React native project.
react native :  0.61.5
mailgun.js: 2.0.6
var mg = mailgun.client({
username: 'api',
key: process.env.MAILGUN_API_KEY || '  ',
public_key: process.env.MAILGUN_PUBLIC_KEY || 'my public key'
});
mg.messages.create(' app.walkin-tour.com', {
from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>subject: "Hello",
text: "Testing some Mailgun awesomness!",
// message: ""
})
.then(msg => console.log('msg------',msg)) // logs response data
.catch(err => console.log('error------',err)); // logs any error
the log error says TypeError: process.nextTick is not a function like below ：

Could you please try with the new version of the lib?

Hello @Jackyaung.
Do you still have trouble with this or it is not an issue anymore and we can close the ticket?


feat: add override colors func

LGTM

thank you

@sergei-durkin i just test this feature, but look like no use

ok u put the override out of opt param.


bug(internal): avoid reusing exec.Cmd objects on retry

Currently, retries will always fail, with an "already started" error, because,
per the exec.Cmd docs "A Cmd cannot be reused after calling its Run, Output or CombinedOutput methods"
This is visible in the logs of https://source.cloud.google.com/results/invocations/ca1df97e-b278-4fdc-b03e-95455d8c8a31/targets/cloud-devrel%2Fgo%2Fgolang-samples%2Fsystem-tests%2Fgo117/log
(search for "already started")
i discovered this while digging into the logs of issue #2444

Kokoro CI check failures are due to other flaky tests and do not appear related.
(e.g. a spanner create failure with its own separate issue: #2441 )

This LGTM, will force push through for now.


[fix] Fixed bug in direct XML parsing of UPF v2 PPs

Fixed bug when parsing UPF files without NLCC. When there is no core correction, the default json behavior is to allocate an array for it filled with zeros. So far, the pugixml parser gave a size zero vector, leading to a crash. This PR fixes it.

cscs-ci run default

cscs-ci run


streamed users

feat(connected-users): add sort option to API
fix(connected-users): yield fetched users asap

:tada: This PR is included in version 1.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix Meetup exercise tests

Test with int as specified in students file stub.
This requires adjusting the example solution to show it can still be done.
Also remove superficial strict types file headers.

Follow-up of #592 (closed by creator)

This change would invalidate a bunch of solutions, how can we prevent re-running tests for too old solutions? Or shouldn't we?

This change would invalidate a bunch of solutions, how can we prevent re-running tests for too old solutions? Or shouldn't we?

I'm not sure how we should proceed about this, @ErikSchierboom do you have any suggestion?
As this aligns the type to the student file stub, to me we should proceed with this change. It's been 3y this stub is like this, so I wouldn't expect this to break too much student code.
This also aligns with the canonical-data.json from problem specifications.

I just realized that this is the featured exercise for #48in24 week 32 and PHP will be one of the 3 languages of that week. So I decided to re-run all tests on community solutions to get as much "working code" there as possible.


Fine tune Grounding DINO as open set with custom coco

Hi, Thank you for sharing this amazing open source. I've tried to fine tune grounding DINO as open set because fine-tuned grounding DINO as closed set is not compatible with grounded SAM repo. I guess when fine-tuning it only with coco format, it turns to closed set algorithm and the architecture is different from the original model.
is there a way to fine-tune grounding dino as open set with my custom coco? then which configuration file do I need to use once I successfully converted my custom coco to ODVG?

Mark


Added additional template methods to typeahead.js

https://github.com/twitter/typeahead.js/blob/master/doc/jquery_typeahead.md#datasets

thanks mate!


Update astro examples to Astro 4 and astrojs/node 8

Hi! 👋
Following up #20, this PR updates the Astro examples to v4 and astrojs/node v8
I believe the only change that needs to be made was renaming the HTTP method names to uppercase, as Astro v4.0 removes support for lowercase names:
👉  https://docs.astro.build/en/guides/upgrade-to/v4/#removed-lowercase-http-method-names
All the examples are working as expected:
Github OAuth
https://github.com/lucia-auth/examples/assets/22222078/6c8d281d-74f1-4c2c-a389-bb6ff825d27c
Email and password
https://github.com/lucia-auth/examples/assets/22222078/59744eaf-e008-49a5-9438-17a99c831f31
Username and password
https://github.com/lucia-auth/examples/assets/22222078/b5aa672d-4bbf-43b8-881d-2bcd0b1b8be2

Thanks!


As a result of SPARK-34637 there is a change to AdaptiveSparkPlanExec when employing DPP and AQE

For more info look at https://github.com/apache/spark/commit/b6d57b6b99

Note we should make sure to add tests for DPP and AQE.
Note this jira was required for a change we made to spark at https://github.com/apache/spark/pull/32195 (SPARK-35093), which I think is also linked to https://github.com/NVIDIA/spark-rapids/issues/2450


AsyncLazy and Task.Run(...) inside

Was looking at the AsyncLazy<T> implementation, and thinking why the class wasn't just implemented like this:
public class AsyncLazy<T> : Lazy<Task<T>>
{
    public AsyncLazy(Func<Task<T>> valueFactory)
        :base(valueFactory)
    {
    }

    public Task<T> ValueAsync()
    {
        return this.Value;
    }
}

A use-case:
public class Foo { }

public class FooService
{
    private AsyncLazy<Foo> _lazyFoo;

    public FooService()
    {
        _lazyFoo = new AsyncLazy<Foo>(LoadFooAsync);
    }

    // This method returns a cached result on subsequent calls.
    public Task<Foo> GetFooAsync()
    {
        return _lazyFoo.ValueAsync();
    }

    private async Task<Foo> LoadFooAsync()
    {
        // In some cases (HttpWebRequest on WP8 for example) this should be started on UI thread.
        Debug.WriteLine("LoadFooAsync Start: {0}", Thread.CurrentThread.ManagedThreadId);

        await Task.Delay(500);

        Debug.WriteLine("LoadFooAsync Finish: {0}", Thread.CurrentThread.ManagedThreadId);

        return new Foo();
    }
}

The implementation of AsyncLazy<T> in the library explicitly sends the factory method on the thread-pool via Task.Run(...), which, in my opinion, has a few implications:

Usually xxxAsync methods use the initial thread (before take off to another one) only for some simple state checks and initialization. Probably, most of the time, sending the method to another thread just to check for ArgumentException is overkill. In either case, it should be responsibility of the asynchronous method to not hold the thread so long.

In some cases, async methods need access to the current synchronization context. For example, HttpWebRequest in WP8 should be called from UI thread (otherwise it will likely deadlock the UI thread, because internally it implemented around the web browser control and tries to sync back to UI thread to send the request).

Are there any serious drawbacks in the implementation of the AsyncLoad<T> as above?
Thanks!

There are two main responsibilities for Lazy which have been carried over to AsyncLazy:

Delay creation of the instance until the first time it is needed.
Provide a threadsafe way to only create a single instance that is then shared among all threads.

The problem with just invoking the async delegate directly is one of context. If your use case is just a single UI thread that just wants to delay creation, then AsyncLazy can look like overkill. In fact, you don't even need Lazy at all; you can just have a pattern like this:
private Task<Foo> _loadFoo;
public Task<Foo> GetFooAsync()
{
  if (_loadFoo == null)
    _loadFoo = LoadFooAsync();
  return _loadFoo;
}

Back to the AsyncLazy design: consider more complex use cases. For example, if the FooService is used by both the UI and a background thread, then the context in which the async delegate is executed could be the result of a race condition: whoever asked for the instance first would have their context used. Another example is on ASP.NET, where multiple requests can access the same service, each coming from a different context.
So, AsyncLazy sidesteps this issue by always guaranteeing that the factory delegate is executed on a thread pool thread. I did briefly consider capturing the context when the AsyncLazy is constructed and using that, but I felt it was too complex, non-intuitive, and would not play well with static AsyncLazy values.
The current solution (always sending the delegate to the thread pool) is not perfect, but it is reliable and predictable.


How to deal with qubits when they are function/operation parameters

When initializing a qubit using SUse or SUseS, we can add the qubit to the context. However, what should be done when elaborating function parameters? For reference, this decision needs to be made in the curry function on line 115 of Elab.ml.

I think while declaring the callable we may need to add the arguments to the context to be able to type check the body, but when we are done with the body, the local arguments/parameters will need to be removed from the context. This is similar to entering a smaller scope and exiting it.
@jakezweifler does that answer your question?


Bump Dependencies #4204938006

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/4204938006

 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.jenkins-is-staging 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.


Constantly create new nodes

render() {
    return (
        <React.Fragment>
                    <div ref = { this.setSVGRef } />
        </React.Fragment>
    );
}
onSVGLoaded = data => {
    let lElement = Snap(this.refSVG);
    lElement.append(data);
}
 Snap.load('images/Printer.svg', this.onSVGLoaded);

lElement.append(data);
how to clear it?
lElement.emove();?

I found the reason: because there is no call to stop the animation(When the component is componentWillUnmount(), there are running animation frames), and there are no svg instances on other pages, all the results of the animation are all attached to the page body.
componentWillUnmount() {
    this.AS(0X01);
    this.AS(0X00, '#H');
}

AS is a function to Stop the corresponding animation。


Pandoc citation output is (???)

Hi,
Most of my pandoc citations come out correctly. However, many are showing up as ??? when i view my markdown pandoc work in marked2
Any idea?

What is your citekey pattern set to, what version of BBT do you have, and what does the key look like in Zotero?

closing for inactivity

What is your citekey pattern set to, what version of BBT do you have, and what does the key look like in Zotero?
cite key format?? [auth:lower][year]
BBT version- seems to update every 12 hours or so. ha.
the key in zotero?- hmmm-

Even if BBT updates frequently (and it has the past few weeks), I still want to know the version that you are currently on.
How are you generating the ??? keys? CAYW, export, drag-drop?

Ah okay. It's 5.0.170.
I use the CAYW zotpicker applet. Most citations come out just fine.

Can you right-click the reference that gives you this output and select BBT->send error report?

(it works best if you first pick this references using CAYW and then right-click it and send the report)

Also, try clicking http://localhost:23119/better-bibtex/cayw?format=pandoc . What do you see in your browser after you've picked that reference?

Hi @retorquere thanks for trying to help me.
I didn't get anything back from the browser one. And nothing happens when I right click a reference in the zotpicker.
Is there just something wrong with the cite keys potentially? Is it possible to automatically recode them all?

I can tell you more when I have the debug report. Make sure zotero is running, click the link, pick a reference, and then send a BBT error report. Without I have to resort to guessing what's wrong.

Hey man, sorry, I think i've figured it out. I think that it's just innaccurate cite keys. Not sure why some where being cited in a different pattern to what I had set in BBT. Some were showing journal article etc aswell. When I've changed them to the standard pattern they now show up properly.
Sorry about that. I'm very new to all of this stuff. And I don't know anyone who uses anything other than Microsoft Word for everything. I'm the rogue one.

I really still don't know what's going on, but if you are good to go, that's OK.


bug in line 3

Challenge Waypoint: Use CSS Selectors to Style Elements has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.80 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

This is not a bug, this is an error in your code.  You have placed your h2 element inside your style element.
In the future please ask for help in the Help Chat rather than posting an issue.  Issues are for problems with the site, not for asking for help.

okkk...


Upgrading existing SST project (2.16.4) to 2.18.2 yields errors on deploy (bucket policy conflicts)

It seems like bucket policies are new, sometime after 2.16.4. If one already has a stack with buckets, and they upgrade from 2.16.4 to 2.18.2, then deploy, the deploy fails if the buckets already have policies on them, created manually via AWS Console.
Steps to reproduce:

Create new SST project, edit stack to have a single bucket
Downgrade to SST 2.16.4
Deploy
Use AWS Console to add a bucket policy
Upgrade to SST 2.18.2
Deploy

Results:
✔  Building...

|  API MyBucket/Bucket/Policy AWS::S3::BucketPolicy CREATE_FAILED The bucket policy already exists on bucket aa-splooshies.
|  API MyBucket/Bucket/Policy AWS::S3::BucketPolicy DELETE_COMPLETE
|  API AWS::CloudFormation::Stack UPDATE_ROLLBACK_COMPLETE
⠋  Deploying...

✖  Errors
   API UPDATE_ROLLBACK_COMPLETE
   MyBucket/Bucket/Policy: The bucket policy already exists on bucket aa-splooshies.

Manually changing AWS resources causes stack drift. That's not a scenario CloudFormation (and therefore CDK/SST) is designed to handle. You can explicitly check for stack drift in CF for this reason.


Fix sample data "regions" field typo

The sample data shows the field is named "regions", not "region"

@bodazhao thanks!


Issues with Multisig

I tested sparrow wallet with a single Sig, and everything worked fine. Now I tried to go to multi sig, and I can't move any of my funds. I would like to go back to Single Sig, but I don't know how or if it is possible. I'm scared of losing my funds! Please help! I don't want to be a bother to anyone. Thank you for your attention.

Closing, see #1071


[SPARK-31472][CORE] Make sure Barrier Task always return messages or exception with abortableRpcFuture check

What changes were proposed in this pull request?

Rewrite the periodically check logic of  abortableRpcFuture to make sure that barrier task would always return either desired messages or expected exception.
This PR also simplify a bit around AbortableRpcFuture.
Why are the changes needed?

Currently, the periodically check logic of  abortableRpcFuture is done by following:
...
while (!abortableRpcFuture.toFuture.isCompleted) {
   messages = ThreadUtils.awaitResult(abortableRpcFuture.toFuture, 1.second)
   ...
}
return messages

It's possible that abortableRpcFuture complete before next invocation on messages = .... In this case, the task may return null messages or execute successfully while it should throw exception(e.g. SparkException from BarrierCoordinator).
And here's a flaky test which caused by this bug:
[info] BarrierTaskContextSuite:
[info] - share messages with allGather() call *** FAILED *** (18 seconds, 705 milliseconds)
[info]   org.apache.spark.SparkException: Job aborted due to stage failure: Could not recover from a failed barrier ResultStage. Most recent failure reason: Stage failed because barrier task ResultTask(0, 2) finished unsuccessfully.
[info] java.lang.NullPointerException
[info] 	at scala.collection.mutable.ArrayOps$ofRef$.length$extension(ArrayOps.scala:204)
[info] 	at scala.collection.mutable.ArrayOps$ofRef.length(ArrayOps.scala:204)
[info] 	at scala.collection.IndexedSeqOptimized.toList(IndexedSeqOptimized.scala:285)
[info] 	at scala.collection.IndexedSeqOptimized.toList$(IndexedSeqOptimized.scala:284)
[info] 	at scala.collection.mutable.ArrayOps$ofRef.toList(ArrayOps.scala:198)
[info] 	at org.apache.spark.scheduler.BarrierTaskContextSuite.$anonfun$new$4(BarrierTaskContextSuite.scala:68)
...

The test exception can be reproduced by changing the line messages = ... to the following:
messages = ThreadUtils.awaitResult(abortableRpcFuture.toFuture, 10.micros)
Thread.sleep(5000)

Does this PR introduce any user-facing change?

No.
How was this patch tested?

Manually test and update some unit tests.

Test build #121413 has started for PR 28245 at commit ec84940.

Merged build finished. Test PASSed.

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

cc @WeichenXu123 @jiangxb1987 @cloud-fan Please take a look, thanks!

Test build #121413 has finished for PR 28245 at commit ec84940.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please.

Test build #121476 has started for PR 28245 at commit ec84940.

Merged build finished. Test PASSed.

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

Test build #121476 has finished for PR 28245 at commit ec84940.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

Test build #121533 has started for PR 28245 at commit 8a74170.

Test build #121533 has finished for PR 28245 at commit 8a74170.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class AbortableRpcFuture[T: ClassTag](val future: Future[T], onAbort: Throwable => Unit)

Merged build finished. Test FAILed.

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

retest this please.

Test build #121552 has started for PR 28245 at commit 8a74170.

Merged build finished. Test PASSed.

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

Test build #121552 has finished for PR 28245 at commit 8a74170.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class AbortableRpcFuture[T: ClassTag](val future: Future[T], onAbort: Throwable => Unit)

Merged build finished. Test PASSed.

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

thanks, merging to master/3.0!

@Ngone51 can you send backport PR to 3.0?

sure, thanks all!

@Ngone51 has this been backported to 3.0 ?

@Ngone51 has this been backported to 3.0 ?

yes.


KeyError: 'edge_sidecar_to_children' & KeyError: 'graphql'

Describe the bug
When Instaloader is used to download saved photos/videos of a user, an attempt to download some posts throws KeyError at edge_sidecar_to_children and graphql.
To Reproduce
Steps to reproduce the behaviour:
When executing instaloader :saved --login <username>, certain posts (observed with all posts having multiple photos and not with posts having a single photo) cannot be downloaded, and the above error is thrown.
Expected behavior
A clear and concise description of what you expected to happen (if not obvious).
Error messages and tracebacks

Traceback (most recent call last):
File "/home/ubuntu/.local/lib/python3.7/site-packages/instaloader/structures.py", line 156, in _field
d = d[key]
KeyError: 'edge_sidecar_to_children'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/ubuntu/.local/bin/instaloader", line 10, in
sys.exit(main())
File "/home/ubuntu/.local/lib/python3.7/site-packages/instaloader/main.py", line 410, in main
storyitem_filter_str=args.storyitem_filter)
File "/home/ubuntu/.local/lib/python3.7/site-packages/instaloader/main.py", line 153, in _main
post_filter=post_filter)
File "/home/ubuntu/.local/lib/python3.7/site-packages/instaloader/instaloader.py", line 49, in call
return func(instaloader, *args, **kwargs)
File "/home/ubuntu/.local/lib/python3.7/site-packages/instaloader/instaloader.py", line 731, in download_saved_posts
downloaded = self.download_post(post, target=':saved')
File "/home/ubuntu/.local/lib/python3.7/site-packages/instaloader/instaloader.py", line 462, in download_post
for sidecar_node in post.get_sidecar_nodes():
File "/home/ubuntu/.local/lib/python3.7/site-packages/instaloader/structures.py", line 226, in get_sidecar_nodes
for edge in self._field('edge_sidecar_to_children', 'edges'):
File "/home/ubuntu/.local/lib/python3.7/site-packages/instaloader/structures.py", line 159, in _field
d = self._full_metadata
File "/home/ubuntu/.local/lib/python3.7/site-packages/instaloader/structures.py", line 142, in _full_metadata
self._obtain_metadata()
File "/home/ubuntu/.local/lib/python3.7/site-packages/instaloader/structures.py", line 134, in _obtain_metadata
self._full_metadata_dict = pic_json['entry_data']['PostPage'][0]['graphql']['shortcode_media']
KeyError: 'graphql'

Instaloader version
4.2.8

File "instaloader\structures.py", line 157, in _field
KeyError: 'edge_sidecar_to_children'
I got this error using latest code pulled from master yesterday. I wrapped the if post.stypename == 'GraphSidecar' section with a try / except (instaloader.py line 496 to allow it to continue on after the error.
            if post.typename == 'GraphSidecar':                 edge_number = 1                 try:                     for sidecar_node in post.get_sidecar_nodes():                         # Download picture or video thumbnail                         if not sidecar_node.is_video or self.download_video_thumbnails is True:                             downloaded &= self.download_pic(filename=filename, url=sidecar_node.display_url,                                                             mtime=post.date_local, filename_suffix=str(edge_number))                         # Additionally download video if available and desired                         if sidecar_node.is_video and self.download_videos is True:                             downloaded &= self.download_pic(filename=filename, url=sidecar_node.video_url,                                                             mtime=post.date_local, filename_suffix=str(edge_number))                         edge_number += 1                 except:                     downloaded = True             elif post.typename == 'GraphImage':

I still see this issue with 4.2.10 and the alpha 4.3a1/4.3a2.

I have the same issue with v4.3 on Windows, installed today:
File "instaloader\structures.py", line 157, in _field
KeyError: 'edge_sidecar_to_children'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "main.py", line 459, in
File "main.py", line 455, in __main
File "main.py", line 412, in main
File "main.py", line 159, in _main
File "instaloader\instaloader.py", line 49, in call
File "instaloader\instaloader.py", line 773, in download_saved_posts
File "instaloader\instaloader.py", line 492, in download_post
File "instaloader\structures.py", line 227, in get_sidecar_nodes
File "instaloader\structures.py", line 160, in _field
File "instaloader\structures.py", line 143, in _full_metadata
File "instaloader\structures.py", line 137, in _obtain_metadata
TypeError: 'NoneType' object is not subscriptable
[15356] Failed to execute script main

@billjoe you get a different error message than the one reported here, hence it's not the same issue, although they may look very similar on the first glance. The error you experience has been reported in #451, but unfortunately there is no fix yet. Feel free to comment your observations there, especially how to reproduce the error.


Propose parameter typo fix

should this typo be fixed?  Does this help?

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

Codecov Report

Merging #1903 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #1903   +/-   ##
=======================================
  Coverage   52.54%   52.54%
=======================================
  Files        4649     4649
  Lines      114723   114723
=======================================
  Hits        60281    60281
  Misses      54442    54442

Impacted Files
Coverage Δ

...servermanager/operations/power_shell_operations.py
10.52% <0%> (ø)
:arrow_up:

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 a479294...784df08. Read the comment docs.

Hi @jeis2497052. This code is generated from an OpenAPI 2 description and needs to be fixed there (otherwise next generation will erase it)

I fixed the first one: https://github.com/Azure/azure-rest-api-specs/pull/2420
The second one is trickier, since this might be a breaking change for Go (that generates SDK from the same OpenAPI description). Since it's a local variable only in Python, I won't fix it for now and I will follow up with the Go team if they want to take the breaking change or not.

Thanks for your contribution :)


JAMES-2586 Develocity metrics for postgres branch

We want to see the metrics for postgres-jmap-integration-tests module (which is taking too long)

Alright let's see the info we can get :)

Alright let's see the info we can get :)

In the meantime, at least we still can manually get the tests runtime from build artifact:
postgres-jmap-its-test-runtime.txt
The total time so far is 32 minutes > 30m timeout.

Alright let's see the info we can get :)

https://ge.apache.org/s/dqudut5akzxr6/tests/goal/org.apache.james:postgres-jmap-rfc-8621-integration-tests:surefire:test@default-test/overview?collapse-all

https://ge.apache.org/s/dqudut5akzxr6/tests/goal/org.apache.james:postgres-jmap-rfc-8621-integration-tests:surefire:test@default-test/overview?expand-all

Some actions taken: https://github.com/apache/james-project/pull/2119

Yes agree. Tests in fact individually look alright, it's just that repeated tests are a time killer.

Will come when we rebase the branch on master. It filled its purpose, closing it.


GoogleEarthPro.yml

I wrote a yml file for google earth pro  :
#GoogleEarthPro
Warning: This application currently cannot run on ubuntu 16.04.7 LTS and lower.ONLY ubuntu 18.04LTS and up are supported. Working hard to fix it.
More detail:This application require 3.3.0 of the Protocal Buffer runtime libary to run. however ubuntu 16.04.7LTS came with 2.6.1 version. I'm trying to ask the software developer for help. When there's a fix I will patch it.
#Author: linlinger
app: GoogleEarthPro
ingredients:
packages:
- libasound2
- libc6
- libcups2
- libdbus-1-3
- libfontconfig1
- libfreetype6
- libgcc1
- libglib2.0-0
- libglu1-mesa
- libgstreamer1.0-0
- libgstreamer-plugins-base1.0-0
- libproxy1
- libsm6
- libstdc++6
- libx11-6
dist: bionic
sources:
- deb https://mirrors.tuna.tsinghua.edu.cn/ubuntu/ bionic main restricted universe multiverse
#- deb http://cn.archive.ubuntu.com/ubuntu/ bionic main restricted universe multiverse
script:
- wget https://dl.google.com/dl/linux/direct/google-earth-pro-stable_7.3.4_amd64.deb
script:

sed  -e 's/Application;//' -e 's/MultipleArgs/X-MultipleArgs/' ./opt/google/earth/pro/google-earth-pro.desktop >> ./google-earth-pro.desktop
cp opt/google/earth/pro/product_logo_256.png  ./google-earth-pro.png
echo  '#!/bin/bash
HERE="$(dirname "$(readlink -f "${0}")")"

export LD_LIBRARY_PATH="$HERE/usr/lib":$LD_LIBRARY_PATH
export LD_LIBRARY_PATH="$HERE/usr/lib/i386-linux-gnu":$LD_LIBRARY_PATH
export LD_LIBRARY_PATH="$HERE/lib":$LD_LIBRARY_PATH
export LD_LIBRARY_PATH="$HERE/lib/i386-linux-gnu":$LD_LIBRARY_PATH
"$HERE/opt/google/earth/pro/googleearth" "$@" | cat' > AppRun

Thanks @dezbracaty. Cool!
Perhaps the newer version of the Protocal Buffer runtime libary can be compiled on the oldest still-supported Ubuntu release, which is Bionic at this point?


Add SQL formatter for SET, RESET, CREATE REPOSITORY, RESTORE SNAPSHOT

Summary of the changes / Why this improves CrateDB
This fixed the statement representation inside the sys.jobs_log table when the statements are issued via the PG simple query mode.
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Superseded by #14343.


add support for opencl compliant division

This is an implementation based on OpFDiv to avoid loosing too much
performance.
When possible it can be reduced to a single OpFDiv if we are divided
by a constant value.
This implementation passes the CTS for divide/half_divide on Intel,
AMD, ARM and Nvidia.
Also remove kHalfRecip and kHalfDivide from ReplaceOpenCLBuiltinPass
as they are in fact managed through clpsv--.bc.
Apply NoContraction Decoration to a bit more fp operands as GPU like
Intel are doing optimization with some of them (CTS contractions test
are failing without it on OpFNegate for example).

When you say "passes CTS" is that wimpy more or full mode?

I mean full mode with all the vector size.


TranslationModel.translate: Always write a translation in the jsonb f…

TranslationModel.translate: Always write a translation in the jsonb field
Currently, in the translate function of TranslationModel, the translations dict is not created if we try to translate the default language.
The suggestion is to always create the translations jsonb field, and always write inside, even for the default language.
Thus, all the languages are stored equally and the jsonb field always contains all the available translations. If the default language is changed later in the project, the default translation is kept and still available in the jsonb translations field.
The original attribute is still written in, as a shortcut for accessing the default language.
This could also simplify data migrations from one translation framework to nece, by copying every existing old translations into a jsonb translations field, to prepare the terrain for nece.

Coverage decreased (-0.03%) to 93.367% when pulling b24bb597e37adfba9f66bdac7bc316dba28f8993 on eschalk-pc:eschalk/always-writing-in-jsonb-field into d16d49eef23cac093d373f33f8fa2abc9718dbe9 on tatterdemalion:master.


HPE Procurve with a space in message id

PRI=188
MESSAGE=%%10IFNET/4/INTERFACE UPDOWN(t):   Trap <IP_ADDRESS>.<IP_ADDRESS>.5.4: Interface 9437225 is Up, ifAdminStatus is 1, ifOperStatus is 1, ifDescr is GigabitEthernet1/0/42, ifType is 117

:tada: This issue has been resolved in version 2.14.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Get-DbaLogin - needed for 1.0

Is this a feature OR bug:
feature
System Details
We need Get-DbaLogin like the other basic commands. Please add the last login time to SMO object and the default output
SELECT MAX(login_time) AS [Last Login Time], login_name [Login]
FROM sys.dm_exec_sessions where login_name = '$($login.name)'

per https://mohammaddarab.com/find-last-login-date-sql-server-login/

closing, merged