Is index.version.created supposed to be fixed?

Hi,
In my index settings I changed the index.version.created number from the example 1070499 to the propper version to match my ES deployment (2030399) and creating mappings after doing so fails with
Mapping definition for [image] has unsupported parameters: [feature: ......... ]

Is this hard-coded somewhere in the plugin itself?
Cheers,
D

No, elasticsearch check the version itself.

Interesting. The results are the same if I leave out index.version.created altogether. The mappings just aren't available anymore.

@kiwionly Do you know what value Elasticsearch is querying specifically? The plugin is built for v2.3.3 and I'm using the Docker elasticsearch:2.3.3 image. I'm struggling to see why any value beginning with 1xxxxxx works, but the moment it changes to 2xxxxxx the mappings stop working.

Could you explain more, I think u should just leave the index version settings when creating index as ES check the versioning.

If I set the index.version.created value in the index properties to be an elasticsearch version between 1000000 and 1999999 it works fine. The second I change it to match the actual version of Elasticsearch I'm using (2.3.3 == 2030399) it fails to apply the mapping and says the image in my index has unsupported parameters.
Example error when creating an index using elasticsearch_dsl + python:
import elasticsearch_dsl

class ImageField(elasticsearch_dsl.CustomField):
    name = 'image'
    builtin_type = 'image'

class MyDoc(elasticsearch_dsl.DocType):
    image = ImageField(**{'feature': {'JCD': {'hash': ['BIT_SAMPLING']}}})

index = elasticsearch_dsl.Index('test')
index.settings(**{'index.version.created': 2030399})
index.doc_type(MyDoc)
index.create()

Results in this error from elasticsearch_dsl:
RequestError: TransportError(
    400,
    u'mapper_parsing_exception',
    u'Mapping definition for [image] has unsupported parameters: '
    [feature : {JCD={hash=[BIT_SAMPLING]}]'
)

The errors from Elasticsearch directly are pretty much the same:
MapperParsingException[Failed to parse mapping [product]:
    Mapping definition for [raw_image] has unsupported parameters:
        [feature : {JCD={hash=[BIT_SAMPLING]}]];

Thanks,
D

I check ES code it check the versioning, that why put that here to ensure it work.
You might want to review the code in ES.

Response from the Elasticsearch team:
it sounds like the author of the plugin is relying on 1.x features that have since been removed. Sounds like you need to take it up with him

https://github.com/elastic/elasticsearch/issues/19869

This plugin is port from ES 2.1 and I suspected the fork I using possible not port to index version more then 2 yet.
My first goal is to make it work, for the upgrade I still no plan, probably will take a look only when got time.


[BUG - Release]

Branch or Release
3.0.1
Game and Engine Version
Halo The Master Chief Collection 4.21
Describe the bug
Crash after intro screen. With or without mods installed. I used custom signatures from a different issue that allowed things to actually work, but resulted in a crash right after the intro plays.
Mods directory
Default mods
To Reproduce
Steps to reproduce the behavior:

Launch game
Wait for intro screen to finish
Fatal Error

Expected behavior
Game launches and lets me into the main menu
Desktop (please complete the following information):

OS:Windows 11 Pro

UE4SS.log

We don't provide support for games with online PvP.

Uh... OK? The game literally has mod tools released by the game developers and an instance dedicated to turn off the anti cheat for mods, including Steam Workshop mods. If that doesn't give the green light for any kind of modification of the game then I don't know what does. The only thing the mod tools don't cover is the menus which use UE4.

Apologies, that is a misjudgement on my part, I see the microsoft modding tools for this game now.


[WIP] add diffie-hellman

work in progress, open-source workshop

Thanks for deciding to help us add exercises to Exercism.
It's been a while since you marked this as a Work-In-Progress, are you still working on this?
If you are, is there anything you need help with?
If you're not, use the 'close request' button on Github so someone else can work on adding the problem.
If we don't hear back from you, we will close it ourselves one week from now.
(On or after the 30th July 2016)


Make the sync bar appear even if only 1 archive is active

This implements what was requested in #473:

Early versions of archive sync on the Staging site allowed for the 'syncing' of a single archive. This had the helpful benefit skipping through an archived stream based on local time rather than time in the archive.

Minimal changes to the code was applied (1 line only lmao). It should be noted though that the i18n string for views.multiview.sync.nothingToSync probably needs to be changed from "Nothing to sync, please open two overlapping archives (eg. collab streams)" to a more fitting text since we don't really require at least two archives now.
I have to say though: this change makes it look like that the requested functionality is something like a hidden feature. As a regular user, I wouldn't expect the archive sync to do that.
Closes #473

NAISU


upgrade to latest dependencies (#3232)

bumping knative.dev/reconciler-test 7286e0a...26cee79:

26cee79 [release-1.11] Create a client and transport per request in the TLS case to force TLS handshakes (# 553)
bumping knative.dev/eventing 6911db0...248a471:
248a471 Eventing TLS: Configure CRDs to have addresses and CA certs (# 7105)
d2ce623 [main] Upgrade to latest dependencies (# 7109)
6bfc2d4 [main] Format Go code (# 7108)
855fbed Auto create events in channels (# 7089)
0054bf3 Eventing TLS: Configure certificates to force rotate private keys (# 7104)

Signed-off-by: Knative Automation<EMAIL_ADDRESS>

OCF Webhook is merging this PR


abigen outputs docstring with example code that on runs when using cargo test -- --ignored

docstring in question: https://github.com/gakonst/ethers-rs/blob/bedb16f3363e1d8256f1d202e5c9fa37bea70b1b/ethers-contract/ethers-contract-abigen/src/contract/methods.rs#L85-L92

no_run is worse because it will never compile, without passing --ignored

why does this even run with --ignored, this is just a function def, no?

Because that's what --ignored does on test binaries:
        --ignored       Run only ignored tests

but there's nothing to run if this is inside a function?

@gtklocker What would you like it to be? It's ignored for a reason, so I don't see why we should remove or change this


deploy-policybot_bots_postsubmit is failing

Bug Description
I noticed that the https://prow.istio.io/job-history/gs/istio-prow/logs/deploy-policybot_bots_postsubmit test has been failing consistently for a long while.
Looking at the logs, I see:
The push refers to repository [gcr.io/istio-testing/policybot]
d6ecf2f8517b: Preparing
f6c6196dc61b: Preparing
ca43459aca41: Preparing
d9dde51d2cdd: Preparing
c5ff2d88f679: Preparing
c5ff2d88f679: Layer already exists
d6ecf2f8517b: Pushed
d9dde51d2cdd: Pushed
ca43459aca41: Pushed
f6c6196dc61b: Pushed
20230222150745: digest: sha256:5fbe62e7723af21dc6f0fe0b312b8d58a5193af5908343380874b75d15a1845e size: 1370
Built container and published to gcr.io/istio-testing/policybot:20230222150745
gcloud container clusters get-credentials "policy-bot" --project="istio-testing" --zone="us-central1-a"
Fetching cluster endpoint and auth data.
kubeconfig entry generated for policy-bot.
error validating "STDIN": error validating data: ValidationError(Ingress.spec): unknown field "backend" in io.k8s.api.networking.v1.IngressSpec; if you choose to ignore these errors, turn validation off with --validate=false
Error from server (NotFound): error when creating "STDIN": namespaces "test-pods" not found

so it looks like the push: target completes and then the deploy_only: failed.
Looking for backend in the deployment yamls, I only see ingress.yaml and it's using networking.k8s.io/v1.
Version
Current master branch

Additional Information
No response

The Ingress issue is now solved. We are seeing multiple:
Error from server (NotFound): error when creating "STDIN": namespaces "test-pods" not found

messages

At this point, I think someone who knows more about the Google infrastructure behind the clusters where policy runs is needed.

Not stale. Test still failing

This was fixed.


[Feature] Bump Karpenter version to latest and using Karpenter API v1beta1

Describe the solution you'd like
Newer version of Karpenter enables more feature and bug fix. Karpenter API in Helm Charts should also be upgraded to v1beta1 to work with latest version of Karpenter.

Delayed due to latest version of EKS Blueprints (1.13.1) still using Karpenter v0.31, which is using v1alpha5 API version.

Close as #116


Whether there is a way to custom mode/determine which mode will connect to the server ex:  VPN over ICMP / VPN over DNS functions

Hi who's there!
Whether there is a way to custom mode/determine which mode will connect to the server, for example a VPN via ICMP/VPN over dns functionality is a great feature but we can only use this if the softether automatically decides so,
for example even if https is allowed, but connecting via icmp or dns is a great advantage or option, so having the ability to choose it before softether decides is awesome.
Secondly we found it difficult to test the feature because we had to block https and other traffic to "force" softether vpn to connect via vpn via icmp or dns, these two reasons (users can't specify) and users are forced to block connections to test it, is the reason why the powerful feature isn't used most of the time, even if it's a very good advantage.

Duplicate of #779


Feature/uar 150 add mo radio buttons to bo type page

JIRA link
https://companieshouse.atlassian.net/browse/UAR-150
Change description
Added total of 5 type of BO/MO radio button options
Work checklist

[x] Tests added where applicable
[x] UI changes meet accessibility criteria

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

Could you please squash your commits and also use the correct format for the commit message. Something like "UAR-150: add MO radio buttons to bo type page"


Deprecate flags with typo (--nodex-min, --nodex-max) in 'template nodepool'

This PR renames the flag --nodex-min to --nodes-min and --nodex-max to --nodes-max in the template nodepool comand.
When using the old spelling, an error will be shown as yo can see below.
Usage
Template Giant Swarm Node Pool CRs.

Usage:
  kubectl gs template nodepool [flags]

Flags:
      --availability-zones strings                     List of availability zones to use, instead of setting a number. Use comma to separate values.
      --aws-instance-type string                       EC2 instance type to use for workers, e. g. 'm5.2xlarge'. (default "m5.xlarge")
      --azure-vm-size string                           Azure VM size to use for workers, e.g. 'Standard_D4s_v3'. (default "Standard_D4s_v3")
      --cluster-id string                              Tenant cluster ID.
  -h, --help                                           help for nodepool
      --nodepool-name string                           NodepoolName or purpose description of the node pool. (default "Unnamed node pool")
      --nodes-max int                                  Maximum number of worker nodes for the node pool. (default 10 on AWS, or 3 on Azure)
      --nodes-min int                                  Minimum number of worker nodes for the node pool. (default 3 on AWS, or 3 on Azure)
      --num-availability-zones int                     Number of availability zones to use. Default is 1 on AWS and 0 on Azure.
      --on-demand-base-capacity int                    Number of base capacity for On demand instance distribution. Default is 0.
      --on-demand-percentage-above-base-capacity int   Percentage above base capacity for On demand instance distribution. Default is 100. (default 100)
      --output string                                  File path for storing CRs. (default: stdout)
      --owner string                                   Tenant cluster owner organization.
      --provider string                                Installation infrastructure provider. (default "aws")
      --use-alike-instance-types                       Whether to use similar instances types as a fallback.

Global Flags:
      --debug   Toggle debug mode, for seeing full error output.

Deprecation message
Flag --nodex-min has been deprecated, please use --nodes-min.

I just noticed that MarkDepecated means that a message will be shown, but the command will be executed anyway. Instead my intention was the command to fail, so that users update their usage. Will add to the PR.


Project Status

Is that Project still active?

Which one?

+1 No updates since May, any particular reason?

@jeveloper
The original author has announced (in the gitter chat) that he will be phasing out his involvement in the project:
"By the way everyone I've stared on a new project
reapp was incredible to build, and has tons of promise
if anyone would be interested in taking it over, someone who has worked with it and helped with it, get in touch with me at my email natewienert / gmail"
I have seen some questions pop-up regarding commit permissions and PRs, but he hasn't really answered those.

@jeveloper
reapp is still active! Once I get write access from @natew I'll be processing any outstanding PRs.

@jhopper28 I've added a couple people, I thought I had added you! Will do that now.

it's alive. This should be closed.


spl-token-swap crate not updated

The spl-token-swap (https://github.com/solana-labs/solana-program-library/commits/master/token-swap/program) program receives constant updates but I can't see that the crate is being updated (https://crates.io/crates/spl-token-swap/versions). That is an issue cause it causes dependency problems for people using the crate as library.

Kindly open this live chat link to talk to the customer service directly on your issue  https://direct.lc.chat/14164218/


Major changes to Chapter 7

Here's the scope of the changes:

Reordered both browser chrome sections quite dramatically. Do note that I used a lot of newtab_rect[3] and back_rect[2] but the intention is to turn those into Rect fields in a later PR.
Reorder LineLayout and TextLayout's layout methods so they flow a bit better.
Explicitly bring up the "reverse pipeline" analogy for click handling
Make Browser.active_tab a Tab, not an index. Both are a little dangerous but this is way shorter.
Moved the layout tree hit testing exercise to a later chapter, where it will do more good.
Move the HTML chrome exercise to the chapter on inputs and buttons, where it's at least hypothetically possible.
Renamed Browser.load to Browser.new_tab, I think the old name was more confusing
Don't store previous_word in BlockLayout, just compute it when needed

The tests don't pass yet and I need to do the forward pass through later chapters to make sympathetic changes, but it should be ready for a review of the chapter itself.

In retrospect, this should have been, like, five PRs. I'm still working on getting tests to pass and forward-propagating all changes. Yeesh!

Ok, I think it is done. The main things there were hard were:

Renaming Browser.load to Browser.new_tab required changes in every tests file
Changing active_tab to be the Tab, not the index, required complex and far-reaching changes in many chapters
Some of the later chapters haven't yet been updated to use wbetools.patch and there are nontrivial differences between chapters where we fixed a bug in one chapter but not the next.
Anyway, I think it's ready for re-review and merging.


Allow the planner to be used across an FFI boundary, more simply

You just have to call Planner::note_requests and then use the results of those queries to call Planner::plan_with_spendable_notes.

Looks like we'll want to disable the governance-related parts of the smoke test pending #1845


First go at at a dub.json for sdfmt.

Try locally with dub run sdc:sdfmt
Once this is added to the registry people can use it using the above command too.

Closed by 2b77c94c9e71ee839577f8249b59e6b34562cb82 .


Revisited with new JupyterLab configuration system

The hook that I used to implement this originally in #1 have been removed, but I proposed new hooks in https://github.com/jupyterlab/jupyterlab/pull/1804 that allow this to work. From the command palette, it's usable in its current state.
The drop-down menu component is not working yet due to some confusion on my part, but that should be an easy fix.

OK, this is ready for review/merge. It requires jupyterlab 0.17.0, which is planned to be released tomorrow.
Now that it's in a working state, should we put it on PyPI? The recommend naming pattern has changed from *-jupyterlab to jupyterlab_* (see https://github.com/dask/dask-labextension/issues/15) so we should rename the repo and the package first. Can someone with the necessary permissions do that?

@rgbkrk Do you have the bandwidth to look over this?

I can click the merge button. 😉  You'll need @blink1073 for an earnest review of the plugin setup.

Looks good to me, though I don't have JupyterHub setup to confirm that it actually works.  :wink:

I would rename it to jupyterlab_...


Why is the Stack Rank blank for some items?

I have a (modified) CMMI project, and some of the backlog items (mostly Requirements and Tasks) don't have any values in the Stack Rank field. This means that I can't recreate the order of the items from a CSV export.
Is there something I can do to ensure that a Stack Rank value is assigned to every item (or at least every item that has a sibling)?
This question also asked on stackoverflow:
https://stackoverflow.com/questions/59644746/how-do-i-ensure-all-azure-devops-backlog-items-have-a-stack-rank-value

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

ID: 02ab6fbc-4df4-bc2d-da58-19dee9395256
Version Independent ID: e613f4a0-37a5-c2fa-fdf5-345c945de6db
Content: Backlogs for portfolios, priority, & multi-team ownership - Azure Boards
Content Source: docs/boards/backlogs/backlogs-overview.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@johnyesberg Thank you for the feedback.
Stack Rank must be manually assigned by the user, please refer to issue https://github.com/MicrosoftDocs/vsts-docs/issues/1696.
For alternatives, such as re-ordering items through the Reorder backlog API, please see Prioritize work.

Thanks @msebolt
Looks like I can rely on the order in the CSV export for rows that have no Stack Rank.


LIS 7.5 code deliverable updates (GHI)

Code updates for LIS 7.5 deliverable

Pushed additional python code updates by developer, smahanam

This work was completed in #1234


Parallelize Engine code builds (Server/Client/Game/Editor)

It should be possible to complete Engine builds quicker than today by distributing it over multiple machines. Server, Client, Game, Editor code builds - they don't need to happen on the same box. Longtail should offer a high-performance storage backend that would allow sharing the artifacts between processing steps.
The big question is really: can we accomplish this split neatly within the BuildGraph?

Moved to https://github.com/falldamagestudio/UE4-GHA-Engine/issues/5


Update GETTING_STARTED.md

Update Readme to fit new version

Thanks @AdamZaczek! I've merged this as 85c214d5526d1f0375f662a5a5b21679eca0bc79


cleanup(virtctl): Remove dryRun options

What this PR does / why we need it:
This PR removes dryRun options and adds functionality to CleanNamespaces to remove VirtualMachinePreference and VirtualMachineClusterPreference instead. It also updates help message for this command.
Release note:
None

/retest

Looks good to me, is this needed for Instancetypes too?

Looks good to me, is this needed for Instancetypes too?

It is already done for Instancetypes.

/retest

/approve

/retest

/retest


Comments on this package so far, and suggestions for possible changes

Comments on file docs/README.md:

This repository contains a skeleton BioCro module library (skelBML), which is an R package representing a BioCro module library with a single example module.

It's confusing to describe the contents of biocro/skelBML as a BioCro module library or an R package since it only becomes such upon running the setup script.  Even after this is done, the package name likely won't be skelBML, unless perhaps the user chooses this name simply in order to create a minimal library so as to try things out.  So I think it would be better to describe this as framework for creating an R package.

Run the startup script

I think it's better to call this the "setup script."  (Similarly for the two other places where you refer to the "startup script.")

Commit the changed files to your new repository.

At this point, when the setup is run for the first time, there will be no changed files, only new ones.

Rerun the startup script, as described above.

Slightly clearer possibly: "Rerun the startup [or setup] script, running it as described above" (since you didn't describe rerunning it).

I'm confused about the three "skeleton" files.  Perhaps this is because I first assumed, from the names, that these were skeleton package DESCRIPTION, LICENCE, and NEWS files, in the same sense that skelBML is (or at least can be used to produce) a skeleton (i.e. bare bones) module library/R package.  But now I think you probably just mean that these are to be considered the description, license, and news files for the skelBML repository.  Even so, I remain a little confused.
  So it is confusing to me to have this file, that looks like an R package description file (except for the freeform license information text at the end) and that is named as if it is the basis for one.  It would be less confusing if this file were called something like "skelBML_respository_description".  Perhaps it's okay to keep the style of an R package description file, with a succession of lines of the form "FieldName: Value", but I think it might be less confusing if it were otherwise.  In any case, if you do keep to this style, I'd replace the first line with "Repository: biocro/skelBML"  Then, in the "Description" line, I'd say something like "Files and code for creating a basic BioCro module library ... ."
LICENSE_skeleton.md
Again, this would be better called LICENSE_for_skelBML.md, if that is in fact what it is.  Given that, except for a few whitespace differences, LICENSE_skeleton and script/templates/LICENSE.md are identical, it seems to me that perhaps the setup script should remove this file.  That biocro/cgbml (on the update-from-skeleton branch) has both LICENSE.md and LICENSE_skeleton.md (again, identical except for whitespace) is very confusing.
NEWS_skeleton.md
This also might be better called something like skelBML_NEWS.md.  And it also seems out of place in any derived package, so I think the setup script should remove it as well.
Other comments:
Would it make sense to add "Depends: R (>= 3.5.0)" to the DESCRIPTION template file?

One other potentially confusing thing:
The setup script writes the file LICENSE.md at the top level.  The docs/README.md file includes the instruction "If you wish to use a different license for distributing your package, you will need to modify LICENSE.md, DESCRIPTION, and README.md."
But the license file the setup script writes is simply a copy of version 2 of the GNU General Public License, and it includes the line "Everyone is permitted to copy and distribute verbatim copies of this license document, but changing it is not allowed."
So perhaps it would be less confusing if the README said something like this: "If you wish to use a different license for distributing your package, you will need to modify the DESCRIPTION and README.md files, and remove or replace LICENSE.md, as appropriate."
(But maybe I shouldn't worry about this any more than I should have worried about the rather awe-inspiring tag on my pillow that said "UNDER PENALTY OF LAW THIS TAG NOT TO BE REMOVED!" (in the olden days, before they added "EXCEPT BY THE CONSUMER").)

I'm confused about the three "skeleton" files. Perhaps this is because I first assumed, from the names, that these were skeleton package DESCRIPTION, LICENCE, and NEWS files, in the same sense that skelBML is (or at least can be used to produce) a skeleton (i.e. bare bones) module library/R package. But now I think you probably just mean that these are to be considered the description, license, and news files for the skelBML repository.

Yes, I was intending for them to be the the description, license, and news files for the skelBML repository. I can see why this would be confusing. I can rename them to add skelBML prefixes as suggested: skelBML_dsecription, skelBML_license.md, and skelBML_news.md. I think the prefix and different capitalization should help differentiate them from any description, license, or news files from a BioCro module library R package.

 So it is confusing to me to have this file, that looks like an R package description file (except for the freeform license information text at the end) and that is named as if it is the basis for one. It would be less confusing if this file were called something like "skelBML_respository_description". Perhaps it's okay to keep the style of an R package description file, with a succession of lines of the form "FieldName: Value", but I think it might be less confusing if it were otherwise. In any case, if you do keep to this style, I'd replace the first line with "Repository: biocro/skelBML" Then, in the "Description" line, I'd say something like "Files and code for creating a basic BioCro module library ... ."

I will make some changes so the content of this file looks less like that of an R package description.

LICENSE_skeleton.md
Again, this would be better called LICENSE_for_skelBML.md, if that is in fact what it is. Given that, except for a few whitespace differences, LICENSE_skeleton and script/templates/LICENSE.md are identical, it seems to me that perhaps the setup script should remove this file. That biocro/cgbml (on the update-from-skeleton branch) has both LICENSE.md and LICENSE_skeleton.md (again, identical except for whitespace) is very confusing.

I think that any BioCro module library R package derived from skelBML should retain a copy of the skelBML license, just as we retain a copy of the boost library license. People aren't always careful about this kind of thing, but I think we can provide a nudge in the right direction!
It's not necessarily the case that a derived package will use the same license, although the GNU GPL makes that very difficult in practice. I am wondering if maybe it would be better to license the skeleton library with the MIT or BSD licenses instead. (Same for the framework.) We talked about this before, but it's had some time to stew in my head and I've started to see the benefits of not using the GNU GPL, at least for the central parts of BioCro.

NEWS_skeleton.md
This also might be better called something like skelBML_NEWS.md. And it also seems out of place in any derived package, so I think the setup script should remove it as well.

Yeah, this could probably be removed from derived packages. As long as the skelBML description file is retained, it should be clear which version of skelBML was used, and then the news file in the skelBML repository could be referenced.

Would it make sense to add "Depends: R (>= 3.5.0)" to the DESCRIPTION template file?

I'm not sure. We added that requirement to BioCro because some of the data files were saved in a newer version of R, and they can't be opened with older versions below 3.5.0. There's no data in a freshly-created module library, so that consideration doesn't apply. On the other hand, I guess it's unlikely that the code will work for any version of R, and since it would be tedious to figure out the actual minimal version, we could just set it to 3.5.0. I guess that's your favored solution. There probably aren't too many potential BioCro users using older versions of R that they cannot upgrade, so I guess it wouldn't hurt to add that. If it causes a problem for someone, we could always change it as necessary.

@gsrohde I've tried to address all the comments here in PR https://github.com/biocro/skelBML/pull/8.


tests: drivers: optee: Add CAPS and UID to the unit tests

Optee driver checks CAPS and UID during initialization, so the unit tests should provide valid data during callback execution or otherwise error message should be printed.
Also fixed the behavour of RPC_FUNC_ALLOC. Optee-os expects new shm address on a3 and a4 and then moves it to a1 a2 when executing other calls, such as RPC_FUNC_FREE. Made normal_call behavior the save as in OP-TEE

@firscity  updated


CI: initial configuration

Configures GitHub Actions to run clippy, check rustdoc, run rustfmt, and run the test suite

@stevefan1999-personal we generally use a stock rustfmt configuration, particularly so it doesn't involve nightly

Hmm, seeing some test failures I wasn't seeing locally

@stevefan1999-personal we generally use a stock rustfmt configuration, particularly so it doesn't involve nightly

Sure, just delete the configs that needed nightly for now


Feat/track txs locally

Try out this version of the Hiro Wallet - download extension builds.

With the issues being reported via discord and all the extensive feedback in https://github.com/blockstack/stacks-wallet-web/issues/1723, I wanted to create a proof of concept of how we could better track local transactions from this https://github.com/blockstack/stacks-wallet-web/discussions/1712 topic.
I've tested this and I feel as though it's a good combination between relying on what currently exists, with adding the minimal to be able to leverage locally persisted transactions.
Here is a video to show how it works. Very open to better suggestions around how to display them, what to label them (if we need any).
https://user-images.githubusercontent.com/11803153/133943982-ca0fc52e-d4a9-4f3d-af1e-a1ed5db6e3ef.mp4
cc/ @aulneau @kyranjamie @fbwoolf

I like the feedback this provides when it moves into pending quickly, but I'm curious how users will respond if they have backed up Submitted transactions for 24+ hrs ...this would hopefully prevent that by keeping track of the nonce locally? Also, just questioning what we do with available balance? We will have to implement this in the explorer too in some way ...the Pending list would have a Submitted state at top?

We will have to implement this in the explorer too in some way ...the Pending list would have a Submitted state at top?

Just realized that makes absolutely no sense bc the explorer won't have any awareness of this ...so we just do nothing in the explorer?

I like the feedback this provides when it moves into pending quickly, but I'm curious how users will respond if they have backed up Submitted transactions for 24+ hrs ...this would hopefully prevent that by keeping track of the nonce locally?

Yes -- part of the reason that some transactions are pending for so long is due to not having a 100% accurate nonce all the time. This hopefully will help that somewhat. Additionally, this is more about fixing an issue where sometimes transactions don't even show up, let alone show up for 24 hours. So i'd ask which is better: no transaction, or a long pending transaction (which currently happens anyways).

We will have to implement this in the explorer too in some way ...the Pending list would have a Submitted state at top?

Just realized that makes absolutely no sense bc the explorer won't have any awareness of this ...so we just do nothing in the explorer?

Yeah this is completely local and there would be no way to do this. This is meant for the local state of a users wallet, nothing more.

Also, just questioning what we do with available balance?

This is an open question and has its own set of complexities, unrelated to this feature.

Had a more thorough look. Works alright. Appreciate it being kept simple for now (no Suspense, leaving TxList compnent as is etc).

Yep it was my goal to have it be as minimal as possible, while adding the new feature.

Think we should have a separate visual style (grey) with a tool tip maybe explaining what submitted means.

Yeah, thinking about it more, "queued" seems to make more sense to me vs "submitted"

I guess we'd need to think through how we swap between nonce calculation logic, purely the presence of local txs?

Yeah, see this line https://github.com/blockstack/stacks-wallet-web/pull/1726/files#diff-2ac8b8a230a77c0902125ef70aa464b65f2868d7d0e9a75c63aaf62e45ea018eR40, this will be the last tx, and only if the tx does not exist in the mempool response. the logic is simple, if the last local tx nonce is greater than the api nonce, use the local tx nonce. Might need some work or adjustment.

Looking good to me from a UX perspective, though perhaps it'd be more consistent to show it under "Today" but instead of "Pending" label it "Submitted" with a different color, perhaps a (mid-tone) grey as @kyranjamie suggests?

Okay, to summarize my changes per comments given:

I've modified the nonce calculation to include the key isMissing to allow us to default to that, rather than a missing nonce. see this commit
I've changed the language to "Queued" vs "submitted"
I've updated the colors of the status so it does not look like a pending transaction
I've moved all the new components around so they are in their own files

To test this yourself, it's best to use the throttle functionality in the chrome network dev tools panel
See the video:
https://user-images.githubusercontent.com/11803153/134354793-735f0f12-7cb2-43e1-8122-f9405b20de9d.mp4

@aulneau I think you missed my previous comments:
#1726 (comment)
#1726 (comment)

see this https://github.com/blockstack/stacks-wallet-web/commit/a25ce7fad542ff1ae54f19dc5a0411ec2183f9ab
and this commit: https://github.com/blockstack/stacks-wallet-web/pull/1726/commits/1e3910f21f4b1604523169a06521914c31c0909c

@aulneau When I test the issue for https://github.com/blockstack/stacks-wallet-web/issues/1723,am seeing the tx being under 'Queued' instead as 'Submitted' as the video above

Has the terminology changed?

@Eshwari007 yes see https://github.com/blockstack/stacks-wallet-web/pull/1726#issuecomment-924943067

Test looking good.cc @aulneau

@aulneau @kyranjamie @fbwoolf When user selects the Queued tx -Activity,they are directed to Transaction not found page.Is that the intended way or should we have user friendly message?Attaching the link for the Queued Explorer link:https://explorer.stacks.co/txid/0x469057577930e30b81b07948cac0ab5a9b7e7cfc7d748895732b5a726cdb4039?chain=testnet&queued=true

I tested the url for https://explorer.stacks.co/txid/0x469057577930e30b81b07948cac0ab5a9b7e7cfc7d748895732b5a726cdb4039?chain=testnet&queued=false yet displaying the same tx not found as well

I tested the url for https://explorer.stacks.co/txid/0x469057577930e30b81b07948cac0ab5a9b7e7cfc7d748895732b5a726cdb4039?chain=testnet&queued=false yet displaying the same tx not found as well

We have an issue for this in the explorer: https://github.com/blockstack/explorer/issues/550
The page needs a design or I can just throw up something temporary today?

Let's bring this up during Monday meeting to get designs for the explorer and make a release next week.
cc @jasperjansz

@aulneau mind posting an updated screen capture video so @jasperjansz can review the entire design integration on Monday before we release this?

@aulneau mind posting an updated screen capture video so @jasperjansz can review the entire design integration on Monday before we release this?

this is up to date https://github.com/blockstack/stacks-wallet-web/pull/1726#issuecomment-924943067

I'd recommend decoupling this PR's release with the related Explorer enhancement per https://github.com/blockstack/explorer/issues/550

@Eshwari007 @timstackblock can one of you QA this once more, but just testing for this specific use case:

set network speed to slow in chrome network panel
send transaction
should show queued
when it switches to "pending" replace by fee the transaction
the transaction should go back to "queued" and then to pending again

There might be 1-2 seconds where there are two transactions, but I don't think that can be avoided currently.

@aulneau Looking good in first round of testing.Will test around few other tokens and between the networks

Test passed.Moving the ticket to QA Done
cc @aulneau


[URL change] KomikIndo

Source information
KomikIndo 1.4.30
Source language
Indonesia
Source new URL
https://komikindo.icu/
Other details
No response
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

URL sebelumnya apa ya?

komikindo.sbs
On Wed, Oct 23, 2024, 09:04 Eddiesti @.***> wrote:

What is the previous URL?
—
Reply to this email directly, view it on GitHub
https://github.com/keiyoushi/extensions-source/issues/5387#issuecomment-2430667415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANM2AOZJRR7JKVVRKMRMFWLZ437ZFAVCNFSM6AAAAABPOFPPZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZQGY3DONBRGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

https://komikindo.sbs/


Fix GPTQ CI

What does this PR do?
First I'm trying to get the tests to pass and then I'll try simplifying / making it more maintainable because currently there's too much wait time from the unittest setup. Things like memory footprint, layers check, serialization, can be tested with tiny models in seconds. And only generation correctness requires a real model. It would also be better to check perplexity instead of exact output sequences.
good news ; even though tests were failing, nothing is wrong with gptq code.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

Who can review?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


⚠ Bofrost.at has degraded performance

In 0535b65, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1638 ms

Resolved: Bofrost.at performance has improved in 3b4a400 after 25 minutes.


[ble mesh] provisioning data store to sd card (IDFGH-12242)

Is your feature request related to a problem?
If we would like to use esp32 as ble mesh provisioner then we eventually hit the nvs flash limits.
Im not saying i plan to build mesh with 65000 nodes, but 1000+ (maybe more) nodes is on the horizon. Storing all nodes data into flash is not possible, due to nvs limitations and probably flash size too.
Describe the solution you'd like.
To have option to store mesh nodes to sd card os some similar solution.
Thank you
PS is there a way to exchange provisioning data between provisioners, like export nodes and keys info and import data?
nrf mesh has it, but its broken, silab mesh app too

Ok, i think i have all API needed to implement this.
To export data:
const esp_ble_mesh_node_t **esp_ble_mesh_provisioner_get_node_table_entry(void)

and to import data:
int bt_mesh_provisioner_provision(const bt_mesh_addr_t *addr, const uint8_t uuid[16],
                                  uint16_t oob_info, uint16_t unicast_addr,
                                  uint8_t element_num, uint16_t net_idx,
                                  uint8_t flags, uint32_t iv_index,
                                  const uint8_t dev_key[16], uint16_t *index, bool nppi)

Thanks


feat: add useLink to NuxtLink

🔗 Linked issue

resolves #22169
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Check title: I need to add a test and include useLink type
📝 Checklist

[x] I have linked an issue or discussion.
[x] I have added tests (if possible).
[ ] I have updated the documentation accordingly.

We need to move the second test to some fixture

Looks good, nice job. Didn't sanity check all of the logic but if tests are passing it should be good :shrug:


Unable to install Realm via Cocoapods

I'm using Realm in my app and I'm able to install it via Cocoapods before. Recently whenever I tried to install the Realm version using cocoapods,    pod 'Realm', '0.95’
I'm getting the following error

[!] /bin/bash -c set -e sh build.sh cocoapods-setup

core is not a symlink. Deleting...
Downloading dependency: core <IP_ADDRESS>
mv: rename ./RealmSwift-swift2.0/../Realm/Realm-Info.plist to ./RealmSwift-swift2.0/RealmSwift-Info.plist: No such file or directory

Any idea why this is happening?

You're using a version of Realm that's over two years old. We cannot provide support for such an old version.


Add animation in homepage

Add animation on the header.

@somyadipghosh Please assign this issue to me.


Overlay data from different periods in a line graph

What problem does this feature solve?
My context :
Every day, some statistics are calculated about the use of our applications.
My needs :

I would like to be able to show the statistics from a period (for exemple the current year) --> done with a line graph
I would like to be able to add an other time period to my graph, to be able to compare the data between two (or more) different time periods. It could allow my team to predict how our applications will be used, and on wich projects we should be more focus depending on the past years' statistics.

Why is it usefull ?
Being able to superimpose data can be usefull in many other cases, if the user wants to compare data over several time period.
How should the graph look like ?
I tried to do this on my own, while using a lot "formatter" methods but it is really hard and the rendering can be quite slow. On the top of this, catching every interactions between the user and the graph isn't always possible.
But my idea is to put a different lineStyle for every period we want to add (the first one is solid, the second dashed and the third dotted). Then, every time periods should be added on the xAxis, one above the other, so that we can easily see where which day is displayed. Maybe it could be great to adapt the text style of each period to the corresponding lineStyle (by underlining it for example ?). Then, the tooltips should be corrected to, in order to know which value corresponds to which date.
This is a screenshot of what I am trying to explain :

What does the proposed API look like?
The proposed API could look like that :
let options = {
    xAxis: {
        data: { list_1, list_2 }
    },
    series: [
        {
            type: 'line',
            name: 'whatever',
            data: list_whatever1,
            overlay: 0
        },
        {
            type: 'line',
            name: 'whatever',
            data: list_whatever2,
            overlay: 1
        }
    ]
}

With the "overlay" attribute saying to which xAxis data list the line is bounded to.

hi @Zigotote, thanks a lot for such a detailed suggestion, in version 5.0 we consider adding some data analysis related features. If you are interested, you can join us to discuss or develop together. At that time, I will organize the current plan into an issue.

Hi @deqingli, its seems indeed quite interesting to me ! How could I join you to discuss more about that ?


Button can still be clicked if overlapped with another control with MOUSE_FILTER_STOP if the user does not release mouse button(-s)

Tested versions

Reproducible in v4.2.2.stable.official [15073afe3]

System information
Godot v4.2.2.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1660 (NVIDIA; <IP_ADDRESS>23) - AMD Ryzen 7 2700X Eight-Core Processor (16 Threads)
Issue description
I have a number of GUI-s. When I click on a button, it opens a fullscreen blocking GUI (mouse_filter = STOP). However, it seems to be still possible to press this initial button, even if it is completely blocked by the GUI (any control with MOUSE_FILTER_STOP).
A workaround would be to always check is_hovered first, however, the button UI will display presses anyway. If you want to NOT display clicks at all, you will need your own button...
I'm not sure about the severity of the issue, but in some projects this may mean that Godot's buttons should never be used at all (or disable styles, textures, etc. to never let it display clicks), and instead you should always use your own buttons.
Steps to reproduce

Launch project
Press the Click me button with RIGHT or MIDDLE mouse button. Hold it (don't release!)
Now click the button with LEFT mouse button. Keep the RIGHT / MIDDLE button pressed
The panel pops up and blocks the button. Keep the RIGHT / MIDDLE button pressed
While still holding the RIGHT / MIDDLE button down, start clicking LEFT mouse button multiple times.
Panel lets the clicks come through to the button. This should NOT happen.

p.6 won't happen if you don't hold RIGHT / MIDDLE button at p.2
If you try the steps above with check_is_hovered = true (property on click_me_button.gd), you will see that the behavior is mainly valid (except that button UI still displays clicks)
Minimal reproduction project (MRP)
button_mrp.zip

One small update is that while you hold RIGHT / MIDDLE button (p.2) no other control can receive clicks as they all get consumed by this button

Also reproduces in v4.3.beta2.official [b75f0485b]. I'll take a look, maybe I can fix it.

This also works:

Press right down on the button, hold it
Move mouse off the button
While still holding right, click with the left
Button emits pressed

That probably shouldn't happen.
I'm guessing it's ok that you can't click other buttons while holding down a button from elsewhere, but it shouldn't click the original button either.

@anniryynanen thank you for the quick reply! Do you think this one is easy enough to get fixed - and maybe even shipped in 4.3?

Shouldn't be a problem as long as someone has time to review it, I found a simple enough solution for the presses. I'm still trying to fix the button staying hovered when the panel opens and mouse is still holding something down... but it's less important so I won't spend too much time on it. The hover disappears on the next mouse button or move event.


13, 117, 118 - documentation, reconcile naming, make transport a struct

fixes #13 #117 #118
to do: add documentation badge and integration.

Current coverage is 98.24% (diff: 100%)

Merging #119 into master will increase coverage by 0.15%

@@             master       #119   diff @@
==========================================
  Files            31         31
  Lines          1259       1253     -6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           1235       1231     -4
+ Misses           24         22     -2
  Partials          0          0

Powered by Codecov. Last update 3bea4b0...0c0c379


Add feature for unused type

Created for #232
Now, added cleaning by unused type.
Question:

We don't use vendoring. For that feature added 2 library "honnef.co/go/tools/...", but we don't have a vendor instrument like golang/dep
Not clear - How to change snapshots for command line flags
Have fail test on my laptop:

# github.com/elliotchance/c2go/build/tests/stdlib
build/tests/stdlib/main_test.go:36: undefined: pthread_attr_t
build/tests/stdlib/main_test.go:39: undefined: pthread_attr_t
build/tests/stdlib/main_test.go:44: undefined: pthread_attr_t
...

For future - we can add removing unused function, constants, variables.

This change is

Add PR for solving wrong result of unused library:
https://github.com/dominikh/go-tools/issues/199

PR not ready


Request to add an IP address to the source code

This issue is a template for a request to add an IP address to the source code of the node. This is necessary to maintain and expand the network so that new nodes can connect to the network using the maximum number of peers.
The current list of nodes can be seen in the file https://github.com/pocketnetteam/pocketnet.core/blob/0.20/contrib/seeds/nodes_main.txt
Please fill in the fields below to review the request:
IPv4 Address (e.g. <IP_ADDRESS>): <IP_ADDRESS>
IPv6 Address (If available): 2a03:4000:55:f34:185e:14ff:feea:8207
Public HTTP API Port: 38081 (default) - configured in pocketcoin.conf, but don't see it via the netstat command
Public WS API Port: 8087 (default) - yes
I confirm that my node is available at the specified addresses and ports 24/7
My pocketcoin.conf config:

Are there any updates? The node is up and running and I'm using it myself and see the usage by the others as well

Are there any updates? The node is up and running and I'm using it myself and see the usage by the others as well


Importing Fiona sets default CURL_CA_BUNDLE environment variable interfering with requests library

Expected behavior and actual behavior.
I expected my os.environ environment to remain unchanged when importing fiona.  However, when I import fiona (via geopandas) it adds a default value for CURL_CA_BUNDLE for the entire process: https://github.com/Toblerity/Fiona/blob/d36e0c897c545e0e51fe759e540c85c117bf3fc1/fiona/_env.pyx#L60
This interferes with the requests library for other parts of code using private CAs
Steps to reproduce the problem.
>>> import os
>>> assert "CURL_CA_BUNDLE" not in os.environ
>>> import fiona
>>> assert "CURL_CA_BUNDLE" in os.environ

This is specifically a problem for an mTLS application such as the following:
import requests

def query_https():
    s = requests.Session()
    s.verify = "/tmp/certs/rest_cacert.crt"
    s.cert = ("/tmp/certs/rest_certificate.pem", "/tmp/certs/rest.key")
    resp = s.get("https://my_domain.com/my_route")
    print(resp.text)

query_https()
import fiona

query_https()

Resulting in one successful call, but then the second:
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:1131)

This is because the verify parameter in requests is overridden by the default environ value above.  See:
https://github.com/psf/requests/blob/79f2ec3acc4e24fef6e6ce31ad7b1d4e2f77be31/requests/sessions.py#L765-L770
Operating system
Distributor ID: Debian
Description:    Debian GNU/Linux 10 (buster)
Release:        10
Codename:       buster

Fiona and GDAL version and provenance
Fiona
version 1.8.21 installed from PyPI using pip version 22.1
GDAL
wget --quiet https://zipline-build-resources.s3.us-west-2.amazonaws.com/src/gdal-3.2.1.tar.gz && \
    tar xzf gdal-3.2.1.tar.gz && cd gdal-3.2.1 && ./configure --with-python && \
    make -j$(nproc) && make install \

See https://github.com/Toblerity/Fiona/issues/1091 (I suppose this is a duplicate issue).

I can verify the os.environ behaviour with 1.8.21 from PyPI, and this appears to be resolved with the latest prerelease. If possible, try using: pip install --pre --upgrade fiona, which may hopefully get binary wheels for your platform (i.e. does not require GDAL to be pre-installed).
The included fix for the 1.9a1 prerelease uses GDAL_CURL_CA_BUNDLE and PROJ_CURL_CA_BUNDLE variable names in os.environ instead of CURL_CA_BUNDLE. Any further suggestions are much appreciated before the next release gets out.

Thanks for that pointer @mwtoews!  We will look into using the pre-release. In general I think that it's fine for a library to consume an environment variable, but I was definitely surprised to see that an import added new environment variables.  (Especially as this was a side effect of importing geopandas).
From a hygiene perspective it might be preferable to use a context manager to temporarily set the environment variable where you need it in the fiona code.  For example, I do this a lot in unitests.  Thoughts?

@hugocarr yes, we use a context manager for GDAL config options and could figure out a way to add the environment variable to that one.


Fix parameterization issues

Fix parameterization issues
Now that we have non-text input model parsers, we can see some issues in our parameterization.

ASR and Image-to-Text modelparsers should NOT be ParameterizedModelParser instances, since their inputs cannot be parameterized.

Parameter resolution logic shouldn't throw errors in the case where it's parsing prompts belonging to regular model parsers.

Test Plan:
Ran the translation prompt for this aiconfig:
prompts: [
{
      "name": "image_caption",
      "input": {
        "attachments": [
          {
            "data": "https://s3.amazonaws.com/lastmileai.aiconfig.public/uploads/2024_1_10_18_41_31/1742/bear_with_honey.png",
            "mime_type": "image/png"
          },
          {
            "data": "https://s3.amazonaws.com/lastmileai.aiconfig.public/uploads/2024_1_10_18_41_31/7275/fox_in_forest.png",
            "mime_type": "image/png"
          }
        ]
      },
      "metadata": {
        "model": {
          "name": "HuggingFaceImage2TextTransformer",
          "settings": {
            "max_new_tokens": 10,
            "model": "Salesforce/blip-image-captioning-base"
          }
        },
        "parameters": {}
      },
      "outputs": [
        {
          "output_type": "execute_result",
          "execution_count": 0,
          "data": "a bear sitting on a rock eating honey",
          "metadata": {}
        },
        {
          "output_type": "execute_result",
          "execution_count": 1,
          "data": "a red fox in the woods",
          "metadata": {}
        }
      ]
    },
    {
      "name": "translation",
      "input": "Once upon a time, in a lush and vibrant forest, there lived a magnificent creature known as the Quick Brown Fox. This fox was unlike any other, possessing incredible speed and agility that awed all the animals in the forest. With its fur as golden as the sun and its eyes as sharp as emeralds, the Quick Brown Fox was admired by everyone, from the tiniest hummingbird to the mightiest bear. The fox had a kind heart and would often lend a helping paw to those in need. The Quick Brown Fox had a particular fondness for games and challenges. It loved to test its skills against others, always seeking new adventures to satisfy its boundless curiosity. Its favorite game was called \"The Great Word Hunt,\" where it would embark on a quest to find hidden words scattered across the forest. \n\nOne day it got very old and died",
      "metadata": {
        "model": {
          "name": "HuggingFaceTextTranslationTransformer",
          "settings": {
            "model": "Helsinki-NLP/opus-mt-en-fr",
            "max_length": 100,
            "min_length": 50,
            "num_beams": 1
          }
        },
        "parameters": {}
      }
    }
]

Before:
File "/opt/homebrew/lib/python3.11/site-packages/aiconfig/util/params.py", line 235, in get_prompt_template
    raise Exception(f"Cannot get prompt template string from prompt: {prompt.input}")
Exception: Cannot get prompt template string from prompt: attachments=[Attachment(data='https://s3.amazonaws.com/lastmileai.aiconfig.public/uploads/2024_1_10_18_41_31/1742/bear_with_honey.png', mime_type='image/png', metadata=None), Attachment(data='https://s3.amazonaws.com/lastmileai.aiconfig.public/uploads/2024_1_10_18_41_31/7275/fox_in_forest.png', mime_type='image/png', metadata=None)] data=None

After:

Beautiful translation

Stack created with Sapling. Best reviewed with ReviewStack.

#826
#867
-> #866
#863

Filed this issue for TS SDK: https://github.com/lastmile-ai/aiconfig/issues/868
The comments are non-blocking so I will resolve on top of stack. I need to land this to publish a package update


the version in apirl may have some bug for the model inference

Kitti model can get result and effect is normal. however, I change the dateset to dair, the model can inference in the Dec, version, but not success in Apirl version. the apirl version  output the ramdom result.
another question, can you submit a cpu pillar version?? the cuda random pillar cause the pillar features are not steady.

Are you on Jetson? Or x86? There's a bug for fp16. Just fixed.
BTW, you could find cpu pillar version in previous commits.

I used X86 platform. Which  previous commits can use CPU pillar? since I compile the Dec. last commit version, the cpu pillar can not    compile success. So, myabe I need you to tell which commit can use this function.


HBASE-26208 Supports revoke @ns single permission

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
5m  4s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 47s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 27s
the patch passed

-0 :warning:
rubocop
0m  7s
The patch generated 13 new + 48 unchanged - 5 fixed = 61 total (was 53)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+1 :green_heart:
asflicense
0m 15s
The patch does not generate ASF License warnings.

14m 30s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3602/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3602

Optional Tests
dupname asflicense javac rubocop

uname
Linux 5c807d337da6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 6ad7eb88c3

Default Java
AdoptOpenJDK-1.8.0_282-b08

rubocop
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3602/1/artifact/yetus-general-check/output/diff-patch-rubocop.txt

Max. process+thread count
79 (vs. ulimit of 12500)

modules
C: hbase-shell U: hbase-shell

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3602/1/console

versions
git=2.17.1 maven=3.6.3 rubocop=0.80.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
5m  8s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 51s
branch-2 passed

+1 :green_heart:
javadoc
0m 17s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 26s
the patch passed

+1 :green_heart:
javadoc
0m 14s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
7m 16s
hbase-shell in the patch passed.

21m 35s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3602/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3602

Optional Tests
javac javadoc unit

uname
Linux 8d297fa6b1ea 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 6ad7eb88c3

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3602/1/testReport/

Max. process+thread count
2376 (vs. ulimit of 12500)

modules
C: hbase-shell U: hbase-shell

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3602/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 51s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
6m  1s
branch-2 passed

+1 :green_heart:
javadoc
0m 18s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 37s
the patch passed

+1 :green_heart:
javadoc
0m 14s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
7m  0s
hbase-shell in the patch passed.

22m 30s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3602/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3602

Optional Tests
javac javadoc unit

uname
Linux 850842fdd5a8 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 01:11:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 6ad7eb88c3

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3602/1/testReport/

Max. process+thread count
1571 (vs. ulimit of 12500)

modules
C: hbase-shell U: hbase-shell

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3602/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Is this issue branch-2 specific? Usually we should open a PR against master branch first, unless the issue is specific to some branches.
Thanks.

@fanxinxin

Thanks for the review code：
This function is supported by both branch-2 and master branch. This is the PR to master branch.
https://github.com/apache/hbase/pull/3626/files
Thanks.
------------------ 原始邮件 ------------------
发件人:                                                                                "apache/hbase"                                                                                @.>;
发送时间: 2021年8月25日(星期三) 上午10:14
@.>;
<EMAIL_ADDRESS>主题: Re: [apache/hbase] HBASE-26208 Supports revoke @ns single permission (#3602)
Is this issue branch-2 specific? Usually we should open a PR against master branch first, unless the issue is specific to some branches.
Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Thanks for the review code：
This function is supported by both branch-2 and master branch. This is the PR to master branch.
https://github.com/apache/hbase/pull/3626/files
Thanks.
------------------ 原始邮件 ------------------
发件人:                                                                                "apache/hbase"                                                                                @.>;
发送时间: 2021年8月25日(星期三) 上午10:14
@.>;
<EMAIL_ADDRESS>主题: Re: [apache/hbase] HBASE-26208 Supports revoke @ns single permission (#3602)
Is this issue branch-2 specific? Usually we should open a PR against master branch first, unless the issue is specific to some branches.
Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


Adjust Rails stack

Problem
The Rails stack in the README uses ActiveRecord and other internal pieces of Rails to describe the stack, which to me is missleading, especially compared to the other stacks in the REAME
Solution
I've removed some of the internal Rails pieces and brought the stack more inline with what I believe to be similar to the other stacks.  Let me know if this isn't correct.

Thanks for the PR @thebadmonkeydev - we're taking the stacks in a bit more of a use-case / starter kit focused direction, to avoid getting into traps about what should belong where. Have a look at the README and see if that makes sense. Do you have a Rails starter that you like or is there a particular feature or type of Rails app that you could suggest tools around?

I mean if we're talking about more like a "starter-pack" approach, I'm a big fan of the stack that comes out of this post https://blog.heroku.com/a-rock-solid-modern-web-stack
It's basically the stack I turn to when setting up a new side-project...I have a rails template in progress if that's more the thing you're looking for.  I'll review the updated README and see if things are a bit clearer.
Thanks for the feedback @dzello!

@thebadmonkeydev Yes! That would be perfect to have!

@dzello just pushed an update against the newest README adding my interpretation of the stack mentioned above.  Let me know what you think and if there are any changes necessary.

Looks good 👍 Nice to have some Rails on the page. Thanks again!


proposal: runtime: add GOEXE

Is your feature request related to a problem? Please describe.
cross-platform go code has the same concerns as the "GOEXE" variable, effectively you want to append ".exe" if windows.
Describe the solution you'd like
runtime.GOEXE which acts the same as "GOEXE" except it is a constant, so that windows binaries will know to look for .exe when launching commands.
Describe alternatives you've considered
Currently, we define moreos.EXE for this, using a constant based on the OS.

This seems a bit too trivial to be worth new API.

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

Based on the discussion above, this proposal seems like a likely decline.
— rsc for the proposal review group


The circular-buffer stub file was not rewritten from javascript to wren

This looks great, it's just the last 2 lines I'm wondering about. :-)

I'm not sure where we should go with the errors in the stub (if we provide any at all - ah I guess we have to so that the import doesn't crash the VM so the test suite can run...)... In my proof I went with classes:
class BufferEmptyError {
  static error { "buffer empty" }
}

And return the class itself (not an instance) as the error... I'm going to ask on the Wren discord and see if there are some better conventions here that perhaps we should use (and encourage)... though that still might leave open what we should or shouldn't include in the stub.
If we decide there are "best practices" should we still go with the "simplest thing that barely works" in the stub?  I realize this is the kind of thing that would normally get covered in concepts, but we don't have concept exercises, only practice for Wren.

One other, Minesweeper was still JS but I just corrected it.

The only error throwing mechanism I see is Fiber.abort(msg), so I wonder if the "hierarchy of Error classes" is something that wren wants us to do at all.
I haven't yet wrapped my head around catching aborts. The Fibers concept is akin to coroutines, and transferring data in and out while using try is something I still need to practice.

The only error throwing mechanism I see is Fiber.abort(msg),

Right but msg can be any class or instance reference, just just a string.  So for consistency you'd want to use constants for unique errors, not "magic strings"... the tests don't care though since they just care whether the constant's value matches or not.


Initial support for schema templates.

Closes #5043.
This adds the ability to start a schema from a previously saved schema template.
A schema template is a schema with some missing values.
It is also possible to pre-define some templates in the Wikibase manifest, so that those are added to OpenRefine when the manifest is first loaded.
I think it would be useful to add an interface to list existing schema templates, which would enable:

deleting schema templates
exporting a schema template to JSON (useful to add it to the manifest afterwards)

I have just tested this, and I indeed am able to create and save new schema templates and then switch between them.
I can save them with or without prefilled values. I think that's good. See the difference between information/artwork and the Art photo example in the short screen recording I made:
https://user-images.githubusercontent.com/9881645/185386296-0e24659b-9951-45f7-9643-be706531eebb.mov
I noticed that I can't save schemas (not schema templates, but schemas) with empty values, is that indeed the behavior we want to maintain? I have no opinion (or memory of any decision) either way.
Also, if I recall correctly, @lozanaross suggested some changes in placement of certain buttons; that may be something to look at in a separate issue I can imagine.

I noticed that I can't save schemas (not schema templates, but schemas) with empty values, is that indeed the behavior we want to maintain? I have no opinion (or memory of any decision) either way.

Yes, a schema is meant to be something that is able to translate rows to edits: if there is a missing value in a schema, then we cannot use it to generate edits. This is consistent with the previous behaviour (not being able to use a schema if it is incomplete).
Of course this is different from the "novalue" / "somevalue" support (#1713), where the user would explicitly specify that they want to use those "special values" (rather than the schema being incomplete.

Also, if I recall correctly, @lozanaross suggested some changes in placement of certain buttons; that may be something to look at in a separate issue I can imagine.

Yes, I am waiting for those issues and I think @lozanaross is aware.

What issues are missing? I thought I made everything that was agreed to be feasible within this round of funding as issues a while back in July / August.

Correct, there is https://github.com/OpenRefine/OpenRefine/issues/4819 already actually!


Second Order Cone and quadratic problems

Hello,
I'm opening this issue to raise attention on SOCP and quadratic modelling that may be highly interesting for some energy system applications, and I'm wondering if they have been considered for linopy as potential upgrades.
SOCP and QP are clearly non linear, so they may not suite the purpose of the package but may be highly valuable for power systems.
Moreover, many solvers now support those techniques, so I was wondering if there is any plan to support those techniques

Note that QP are now supported


newbie questions

Hello
I've been reviewing static site generators with the aim of finding something for generating documentation.
Nuxtjs and nuxtend-module look really interesting!
I have a few questions I hope you can help with.
Navigation
I am completely new to VueJs - so please forgive me if this is a stupidly simple question. I understand that, to add new pages (or content) I simply put a new .md file in the content directory. This is very simple. How can I then create a navigation system for these documents?
Exporting Static content
I understand I can export a static site. But I could not find how to do this. Can you please clarify this?
Export as custom format
Is it possible to export in another format besides html? I have been trying Hugo and one nice feature is all of the different formats it supports. On use case I am looking for is to be able to export json or yaml. I am wanting to document various micro services and would like to be able to export certain settings as json or yaml - using a template - so that the resulting file can be used as a deployment config in OpenShift. Is such a thing possible?
Thanks and regards

Navigation: Every page has a path property. So you can automate the navigation by fetching all pages and creating links with their path. Also, you have access to page anchors (headings inside page), so you can use those for within-page navigation if the content page is long. (For a reference, here's the content navigation example.)
Exporting site: Since the latest release, @medfreeman made it so that you just run nuxt generate. Your site will be available in dist folder and all content files will be generated for you. 👍
Custom formats: Are you refering to the generation of content files or of the entire site? We generate the content files as json data so that they can be seamlessly used within nuxt pages. We do this because our primary concern is to integrate with Nuxt so that the API is simple, with minimal overhead. As for custom generation of the actual site (i.e. pages), Nuxt handles that.
If I answered all your questions, feel free to close this! And thanks again for using nuxtent!

Thanks @alidcastano
The navigation example you gave is like a Table of Contents? This will be very useful.
But what I am stuck on is how to present a list of all documents. I realize I can just type the path in the browser. But users who want to view the site will not know what documents are available.
I appreciate this is probably very basic, however I - and the users who will be adding docs - are more content creators than vue developers.

Custom formats: Are you refering to the generation of content files or of the entire site?
Im interested in both. But exporting the whole site to html is completely sufficient.

However, it would be really useful if it was possible to export certain page data (in Hugo lingo its called front matter) using some kind of template. In the same way that vue templates use html tags, is it possible to have another view that could export data as say yaml or json - depending on a vue template?

A list of all documents works in a similar way - just iterate through the fetched content pages:
asyncData: async ({ app }) => ({
    articles: await app.content('/articles').getAll()
})

articles will be an array of content pages that you can iterate through and render as you please.

Can you define what you mean by export? As far as I'm aware, front-matter works to specify/override page data and Nuxtent supports this. Are you asking to allow different types of front-matter in markdown files? Right now we only support yaml front matter. But if you like you can open new issue with feature request.
Closing this since questions have been answered.


Fail at running the project

I have try running the project, by trying both methods, and it always fails. Most probably for dependency or version issues.
I had dependency issues with poetry itself (outside of the project) which I succeeded to solve.
When I run : poetry install && poetry shell
I get :
  RuntimeError

  The lock file is not compatible with the current version of Poetry.
  Upgrade Poetry to be able to read the lock file or, alternatively, regenerate the lock file with the `poetry lock` command.

  at /usr/lib/python3/dist-packages/poetry/packages/locker.py:481 in _get_lock_data
      477│                 "Upgrade Poetry to ensure the lock file is read properly or, alternatively, "
      478│                 "regenerate the lock file with the `poetry lock` command."
      479│             )
      480│         elif not lock_version_allowed:
    → 481│             raise RuntimeError(
      482│                 "The lock file is not compatible with the current version of Poetry.\n"
      483│                 "Upgrade Poetry to be able to read the lock file or, alternatively, "
      484│                 "regenerate the lock file with the `poetry lock` command."
      485│             )

When I try :
python3 -m venv venv
source venv/bin/activate
pip install .

I get for the last command :
Processing /home/dim/Documents/Serieux/Stages/M2/Stage/Outils/Database/web-content-extraction-benchmark
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... error
  error: subprocess-exited-with-error

  × Preparing metadata (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [14 lines of output]
      Traceback (most recent call last):
        File "/home/dim/Documents/Serieux/Stages/M2/Stage/Outils/Database/web-content-extraction-benchmark/venv/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 363, in <module>
          main()
        File "/home/dim/Documents/Serieux/Stages/M2/Stage/Outils/Database/web-content-extraction-benchmark/venv/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 345, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
        File "/home/dim/Documents/Serieux/Stages/M2/Stage/Outils/Database/web-content-extraction-benchmark/venv/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 164, in prepare_metadata_for_build_wheel
          return hook(metadata_directory, config_settings)
        File "/tmp/pip-build-env-ctqbwd3c/overlay/lib/python3.10/site-packages/poetry/core/masonry/api.py", line 42, in prepare_metadata_for_build_wheel
          poetry = Factory().create_poetry(Path(".").resolve(), with_groups=False)
        File "/tmp/pip-build-env-ctqbwd3c/overlay/lib/python3.10/site-packages/poetry/core/factory.py", line 60, in create_poetry
          raise RuntimeError("The Poetry configuration is invalid:\n" + message)
      RuntimeError: The Poetry configuration is invalid:
        - data.repository must be uri

      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

I have try to remove the lock file and it did not fix the issue.
I have also try different path for the repository variable in the pyproject.toml file.
I don't know how to solve the issue.
Thank you for your help.

The issue is fixed. I don't know what was causing the problem.
I have uninstall poetry (with sudo apt remove python3-poetry), remove all local changes of the project (one important issue was with one of the local changes), commented Dragnet and ExtractNet between lines 54 and 66 in pyproject.toml, use the pip install in venv and it worked.


chore: bump serde_json version

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[ ] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
10 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.008%) to 73.33%

Files with Coverage Reduction
New Missed Lines
%

kclvm/tools/src/LSP/src/state.rs
1
86.15%

kclvm/sema/src/core/scope.rs
3
65.58%

kclvm/query/src/query.rs
3
74.39%

kclvm/sema/src/core/symbol.rs
3
46.54%

Totals

Change from base Build<PHONE_NUMBER>:
0.008%

Covered Lines:
49002

Relevant Lines:
66824

💛  - Coveralls


Enums erroneously removed from unions by type guards

enum E {}
enum V {}

var x: number|E|V;

if (typeof x === "number") {
	x; // Should be number|E|V, but is just number
}

Fixed by #5442.


Add just_finished function in addition to is_finished

I have found myself needing a just_finished function quite often to for example signal an event that might get fired multiple times with just an is_finished (because of possible frame delays). It would just fire a single time per animation similar to the Timer.just_finished. Just curious what your thoughts on this are? Maybe I am just overcomplicating things again.

That makes sense, yes.
Might be worth to finally think about Events (and/or Triggers?) for this use case as well. But just_finished seems useful on it's own. If you have an idea on how to implement it go ahead, otherwise I will try to throw something together next week.

Alright cool, I will try to hack something together tomorrow.
Ohhh, right Events are also a possibility, I didn't think about that at all. I will give that some thought as well.

I just remembered that I used timers internally in the past. I think I changed it to the way it is now so it is closer to the (3d) animation stuff from bevy, but I do not think we ultimately gained a lot from that.
Maybe we should go back to timers to get some of this stuff for "free".

Yeah I was just going through the Timer.just_finished to get an idea of how they implemented it. It would be fairly hard to add just_finished without also adding quite a bit of state. I wouldn't mind going with Timers.

I also thought about how to add Events, but I actually don't know if that is such a good idea. For example, would you fire an Event with the ID of the entity the animation is on (say whenever an animation just finished)? Depending on how many animations you have this might be quite a lot and the user would still need to check if that event is the one he actually cares about.
What I could imagine works better here is to add a Component like a AnimationJustFinished to the entity where the AnimationPlayer2D is on. Then you could query for Added<AnimationJustFinished and know for certain that the animation finished. I am not a huge fan of this control flow though, but I can't really think of a way in that Events are really useful, maybe you have some.

I am not sure about the performance implications of adding/removing components like that. Feels kinda weird anyways, but I need to think about it a bit more.
In general the ergonomics on how to interface with he animationplayer are not ideal.

I tried with timers in #25, but I did not really like it. Maybe I did not do it the best way, but negative speeds/reverse animation do not seem to work well at all with timers.
Instead I have #26, which just adds just_finished on top. @PraxTube mind taking a look?


Mvh/sofria fixes

This PR fixes 2 problems encountered with Diegesis.Bible:

SOFRIA rendering failed when a w wraps other markup. I don't think this is legal USFM but eBible seems to do it a lot. I've 'fixed' this by generating an additional w on the fly.
SofriaRenderFromProskomma wasn't generating chapter and verse callBacks for each new paragraph.

I added some tests which pass.

Also need to bump the version to (maybe) 0.6.4.

I've changed a lil' bit the code in 'renderItem' from PerfRenderFromProskomma : https://github.com/Proskomma/proskomma-json-tools/pull/15/files#diff-da65f3e5ff8bea7a94affdcc46f32c07dba1b93841424418e8264c408dd1c3c9


Please add lld

I can confirm this issue: https://github.com/brechtsanders/winlibs_mingw/issues/118. GNU ld is extremely inefficient.
p/s: I only ask for lld, not the full LLVM or Clang.

It's at least worth investigation, though I don't know when I'll get to it
myself. I haven't noticed performance issues with ld in w64dk so far, but
I haven't been linking gigantic C++ images (using Boost, etc.). Do you
know a public project I could try that shows a significant difference
between ld and lld? A contrived example would be fine; even preferable.
For those following along: If you have Visual Studio installed with the
Clang component, in a vcvars environment you can -fuse-ld=lld in w64dk to
link with lld. It works fine for C programs, though it struggles linking
libstdc++, perhaps related to Bintuils #26103. (Probably an issue that
would need to be solved here.) It's one way to get a sneak peek of lld in
w64dk.

Do you know a public project I could try that shows a significant difference between ld and lld?

CopperSpice comes immediately to mind.

Yes, CopperSpice. You don't need to build the library from source. You can use their prebuilt library (MinGW version) and try to build their sample program, KitchenSink. Please note that you need to use a different MinGW, e.g., their own MinGW distro on their website. Yours will not work because it's lacking dynamic libraries (libstdc++, libwinpthread,...). I used to try compiling CopperSpice (the library) in MSYS2. Even though I have plenty of RAM, ld ate many GBs of memory and stuck for hours. Switching to lld solved the problem immediately. Linking time from hours now only takes minutes.

Thanks, this seems like the sort of example I want!

A bit off-topic but this is my experience: Clang is more memory efficient than GCC, even on Unix. For example, the system has 4 cores and 8GB of RAM, by default ninja will launch 6 make jobs (1.5*number of cpu cores). This will only work with Clang. For GCC, you have to limit it to just 4 make jobs. Otherwise you will encounter OOM.

One thing worth mentioning is that lld doesn't support GCC lto, though that is disabled in w64devkit anyway. If mold ever begins supporting PE I think it will be the better choice here.

Does lld still only supports linker scripts on ELF targets? If that's the case, that'd be a deal breaker for me

You don't need it but I need. This FR is about adding lld, not removing GNU ld, so feel free to use whatever you want to use.


[GUI] "Device timeout" attribute is not editable when an "Actuator" attribute has been added to the template

Steps to reproduce the problem:

Click "New Template"
Add an actuator attribute
Add "Device Timeout" configuration attribute

Test OK.


Error start

This compiler is provided as part of the Microsoft (R) .NET Framework, but only supports language versions up to C# 5, which is no longer the latest version. For compilers that support newer versions of the C# programming language, see http://go.microsoft.com/fwlink/?LinkID=533240
Sistem:
Windows 10
NET Framework 4.5

I agree, it would be very well if @GlebSBrykin update this project to NET 5.

This compiler is provided as part of the Microsoft (R) .NET Framework, but only supports language versions up to C# 5, which is no longer the latest version. For compilers that support newer versions of the C# programming language, see http://go.microsoft.com/fwlink/?LinkID=533240

This is not an error, but a warning from the compiler. This should not prevent the exe from being built.

I agree, it would be very well if @GlebSBrykin update this project to NET 5. :)

In any case, it will be a separate branch that does not cancel or overlap the main one, which is intended for the built-in Windows dotnet.

In any case, it will be a separate branch that does not cancel or overlap the main one, which is intended for the built-in Windows dotnet.

Maybe it's better to create another repo to this purpose. It allow you to support several project versions at the same time for both repos without messing up in a single repo.

@alexey267, I've cloned this repo and I try to update project to .NET 5 in the future and add missing features at my glance. :)


authfe shows much higher latency for cortex than distributor

From @jml on November 2, 2016 17:18
Filing under service, as it's the most general.
The Cortex dashboard starts with a graph of perceived QPS & latency from authfe—i.e. QPS & latency for cortex routes as measured by authfe.
Sometimes, the 99%ile authfe latency is very high (e.g. O(seconds)) compared to the distributor latency (e.g. O(10ms)).
We need to understand why. Current best guess is that it's due to a discrepancy in the buckets we're using.
Copied from original issue: weaveworks/service#988

Belongs in monitoring or cortex, but not here.

Cortex is not exposing latency metrics for queries at all.

https://cloud.weave.works/admin/prometheus/graph?g0.range_input=1h&g0.expr=topk(10%2C+histogram_quantile(0.99%2C+sum(rate(scope_request_duration_seconds_bucket{job%3D"default%2Fauthfe"%2C+route+!~+"admin.*"%2C+ws%3D"false"}[1h]))+by+(route%2C+le)))&g0.tab=0

Not fixed yet.

http://frontend.dev.weave.works/admin/prometheus/graph?g0.range_input=1h&g0.expr=histogram_quantile(0.5%2C+sum(rate(cortex_request_duration_seconds_bucket{job%3D"cortex%2Fdistributor"}[5m]))+by+(le%2C+route))&g0.tab=0

Okay they are now for some reason.

but not in the dashboard

Okay I think this is actually working, just queries make up a small number of the QPS.

Nah this isn't working

This is the result broken down by route:

But when aggregating that "bump" doesn't show:

The only difference between the way we capture cortex latency and authfe is buckets.  So I'm going to use same buckets in cortex as authfe.


[ic-chips]: Truncate label if label is too long

Summary
At the moment, a max-width is not set on chips, so a very long label will increase the width which goes against the guidance: https://design.sis.gov.uk/components/chips#labels
💬 Description
A max-width is required on the chips with the label truncating once it overflows past the max width.
💰 Use value
Consumers are expecting certain behaviour when using chips based on the guidance provided so the built component should adhere to it.
Additional info
Development considerations:

This might work: https://developer.mozilla.org/en-US/docs/Web/CSS/text-overflow

Design considerations:

Once the text overflows, should there be a tooltip to display the full label?
What is the max-width? Should we set one based on our guidance or give the control to the developers so it's consistent across applications? We can provide the developer with a CSS custom property to set the max-width.

Suggestion is that we won't do  the truncation would cause an accessibility error, design to confirm

Relates to https://github.com/mi6/ic-ui-kit/issues/332


Cannot create an API using swagger json file if is bigger than 15Kb.

Description:
Cannot create an API using swagger json file if is bigger than 15Kb.
For example if I upload swagger petstore it work ( 14Kb) but with pretty printed version of the same swagger I receive this error

In attachment logs and swagger used.
Notes: I'm using HTTPS
Steps to reproduce:

Login as admin and open publisher
Click CREATE API  "I Have an existing rest api"
Upload attached "petstore_swagger_prettyprinted.json"

If I create api using an url with petstore_swagger_prettyprinted.json it work, but I cannot modify the json/yaml from apim webapp beacuse I receive the same error
Affected Product Version:
wso2 apim 3.1.0
Environment details (with versions):

OS: Ubuntu 18
Client: Firefox/Chrome
Env: Virtual server

Attachments
apim.log
petstore_swagger_prettyprinted.json.gz
petstore_swagger.json.gz

Hi @giosdas
Is this a vanilla wso2 apim 3.1.0 pack ?

Hi @tmkasun
It's a wso2 apim configured with a wso2 IS 5.10.0 as key manager


Reset button not vanishing income/deductions input

Issue #9
fixes 2

Describe the changes you've made
In delete function I have entered two statements to delete value entered in Income and Deductions so that when reset button is pressed those two inputs are also delete.
But delete function is also called in in calculate function.
The purpose of calling delete function in calculate function was to clear Tax calculated in previous result and show white space for new result.
so I removed the delete function and just added the purpose.
I did this because new delete function is clearing input of income and deductions, which is not desirable in calculate function
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[ ] I have added my name in the contributors list at the end of README.md file.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] The title of my pull request is a short description of the requested changes.
[ ] Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)

@ShreyaKanodia Why you have mentioned issue number 2 in the description.
Edit your comment.
I'll review your PR soon.

@QAZIMAAZARSHAD  Sorry it was by mistake.
Could you please explain me about checklist, I am not getting that

Application not working after your changes.
Retry.


Relations & sort order

How kan I specify the sort order of the items in the dropdownlist of a relation field?

You must take a look at how to do it with selectize. Maybe we should add some extra options to the relationships attributes.

Would a 'sort' option not do it? So that with the minimongo find() the sort options could be added?

You can use the score function of selectize.
That function is not supported right know, but it wouldn't be difficult to do it. Do you wan't to make a pull request?


Terminal: run active file in terminal does not work

Steps to Reproduce:

open a file
pick the command "Run Active File in Terminal"

=> 🐛 does not work because we seem to check for file scheme

/duplicate https://github.com/microsoft/vscode/issues/73451


Feat(model)

disparate yet related adventures in wiring and documentation

Codecov Report

Merging #94 (2f31c99) into main (4b75560) will decrease coverage by 0.95%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
- Coverage   85.78%   84.82%   -0.96%
==========================================
  Files          30       35       +5
  Lines        3686     3789     +103
==========================================
+ Hits         3162     3214      +52
- Misses        524      575      +51

Impacted Files
Coverage Δ

pynhm/base/meta.py
83.90% <ø> (-0.19%)
:arrow_down:

pynhm/analysis/model_graph.py
23.80% <23.80%> (ø)

pynhm/atmosphere/PRMSBoundaryLayer.py
91.48% <60.00%> (-0.72%)
:arrow_down:

pynhm/base/model.py
93.67% <81.81%> (-2.17%)
:arrow_down:

pynhm/__init__.py
100.00% <100.00%> (ø)

pynhm/analysis/__init__.py
100.00% <100.00%> (ø)

pynhm/atmosphere/__init__.py
100.00% <100.00%> (ø)

pynhm/base/__init__.py
100.00% <100.00%> (ø)

pynhm/hydrology/PRMSChannel.py
81.87% <100.00%> (+0.11%)
:arrow_up:

pynhm/hydrology/PRMSGroundwater.py
100.00% <100.00%> (ø)

... and 3 more

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 4b75560...2f31c99. Read the comment docs.


HWKALERTS-198 custom hawkular-alerts.properties file not recognized

it looks like maybe an unintended '.' prefix was leftover from the past

The "." is on purpose.
Linux/Unix convention is to prefix with "." application configuration files that are set up on $HOME.
Also, metrics used this one:
https://github.com/hawkular/hawkular-metrics/blob/master/api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/config/ConfigurableProducer.java#L90
Note the ".metrics.conf" file.
Also it was even documented :)
https://github.com/hawkular/hawkular.github.io/blob/pages/src/main/jbake/content/docs/components/alerts/index.adoc#external-configuration-file
But this "Installation page" was moved in one of the hawkular.org re-organizations.

Sorry for the confusion. Closing...


Make use of vue-template-compiler to avoid the need for babel

https://github.com/visualfanatic/vue-svg-loader/issues/126

Thank you for the PR! In the latest vue-svg-loader@beta I did it slightly differently, I moved the logic back to the main repo, vue-template-compiler is no longer required and I also included Vue 3 support.


Readme contradiction

The Readme specifies installing pytorch separately, but then includes it in requirements.txt. Should pick one.

Seems fine to just include it in reqs.

Waiting on #16 as that may just resolve this.


mispell

Coverage remained the same at 100.0% when pulling 992346a7d7baaaea60f8a5a6e567cf785e0dd82a on typo into 4fa55244a95efe92c68cfec96b8118bd7e6338a5 on master.

Coverage remained the same at 100.0% when pulling a3acb4da70db70196470d725b01331b6241b45f2 on typo into e8c9474a43902cacb757080cb8460e6a43aa80ce on master.

Coverage remained the same at 100.0% when pulling 1fb7a7b83c98a4c8225d7c0de10b46a1195a6a39 on typo into ae6fa387f0d904fc46195ed7558cca6df6792207 on master.