move MCP settings to meshConfig

pilot's --mcp-server-addr flag was deprecated several months ago in favor of specifying
a list of MCP server addresses in the config map (allows adding/removing mcp servers dynamically).
This PR fixes the helm templates to add MCP info into config map instead of using the deprecated
CLI option in pilot.
Signed-off-by: Shriram Rajagopalan<EMAIL_ADDRESS>

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

Needs approval from an approver in each of these files:

install/OWNERS [rshriram]

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

/lgtm

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

Needs approval from an approver in each of these files:

install/OWNERS [rshriram]

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

@rshriram: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test istio-integ-k8s-tests

prow/e2e-bookInfoTests-v1alpha3.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test istio-integ-k8s-tests

prow/e2e-bookInfoTests-v1alpha3.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-simpleTests-cni.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test istio-integ-k8s-tests

prow/e2e-bookInfoTests-v1alpha3.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-simpleTests-cni.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test e2e-simpleTests-cni

prow/e2e_pilotv2_auth_sds.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

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

Needs approval from an approver in each of these files:

install/OWNERS [rshriram]
pilot/OWNERS [hzxuzhonghu,rshriram]

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

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test istio-integ-k8s-tests

prow/e2e-bookInfoTests-v1alpha3.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e_pilotv2_auth_sds.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test istio_auth_sds_e2e

prow/e2e-simpleTests-cni.sh
6b8e493b66f3b02124a2f175a6e553a2ce56617a
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e_pilotv2_auth_sds.sh
27993314f12a6640b47fb345235f698d3988c544
link
/test istio_auth_sds_e2e

prow/e2e-simpleTests-cni.sh
6b8e493b66f3b02124a2f175a6e553a2ce56617a
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
6b8e493b66f3b02124a2f175a6e553a2ce56617a
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
6b8e493b66f3b02124a2f175a6e553a2ce56617a
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
6b8e493b66f3b02124a2f175a6e553a2ce56617a
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
6b8e493b66f3b02124a2f175a6e553a2ce56617a
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
6b8e493b66f3b02124a2f175a6e553a2ce56617a
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
6b8e493b66f3b02124a2f175a6e553a2ce56617a
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
6b8e493b66f3b02124a2f175a6e553a2ce56617a
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
6b8e493b66f3b02124a2f175a6e553a2ce56617a
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm
cc @wenchenglu

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

Needs approval from an approver in each of these files:

install/OWNERS [rshriram,ymesika]
pilot/OWNERS [hzxuzhonghu,rshriram,ymesika]

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

@rshriram you can pull changes in https://github.com/istio/istio/pull/11805 to get the circleci passed.

@irisdingbj All required tests are passing on this one.
The CLA isn't because @hzxuzhonghu pushed to @rshriram's branch and the bot doesn't like it.


Understanding: babel-preset-react-app

Hi,
Just for understanding purpose to check either i misunderstood or a bug.
In babel-preset-react-app package between this lines (https://github.com/facebookincubator/create-react-app/blob/master/packages/babel-preset-react-app/index.js#L56-L63) you are adding two react-plugins for development/test mode.
Now lines (https://github.com/facebookincubator/create-react-app/blob/master/packages/babel-preset-react-app/index.js#L80-L88) will add full babel-react-preset itself which contains those two packages as dependencies

Now can help me understand why we are adding separately once again on top ?

Thanks for the question @bboysathish! babel-preset-react does have those plugins as dependencies, but if you look at the source code, it does not enable them at the moment.
Here's a discussion in the Babel repository about adding them to the preset: https://github.com/babel/babel/issues/4702

@fson Thanks for the info. Made a pull request to explain.


Roll Flutter Engine from 8ab189b77b8d to 2e9f0df868b3 (1 revision)

https://github.com/flutter/engine/compare/8ab189b77b8d...2e9f0df868b3
2023-11-16<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from AuDKY8ob6GrGwDlmk... to M0zM3CJLIrd5lb0u0... (flutter/engine#48115)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from AuDKY8ob6GrG to M0zM3CJLIrd5
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jonahwilliams@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Create SECURITY.md

What does this PR do?
<-- Added security. -->
Fixes #74
Type of change

This change requires a documentation update

How should this be tested?

[ ] Test A
[ ] Test B

Mandatory Tasks

[yes] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Please merge my pull request and label me
@iamkanhaiyakumar

I made the modification
Please merge the pull request and label me level2
@GauravKesh  @iamkanhaiyakumar


Met AttributeError when doing batchqc

Hello!I met an AttributeError when doing batchqc with stereopy v1.2.0 or v1.3.0.
AttributeError: No attribute named 'StPipeline.batch_qc'

Did you run it in a python script named as batch_qc.py? If so, change the file name to other.

I ran it with jupyter notebook and the bug still persists  no matter how I change the name.

the version of setuptools is 69.5.1
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2024年7月1日(星期一) 下午4:00
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [STOmics/Stereopy] Met AttributeError when doing batchqc (Issue #300)
Check if the version of setuptools is higher than 69.5.1, if so, downgrade to this version.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Sorry that I forgot something to tell you the torch must be installed, you can to install some others necessary dependencies mentioned in requirements_for_image.txt.


BUG: stats.shapiro inplace modification of user array

Describe your issue.
stats.shapiro
AFAICS: #15777 introduced inplace modification that changes the user data
test failure in statsmodels with dev version of scipy
I don't have dev version of scipy, so cannot confirm
Reproducing Code Example
-

Error message
-

SciPy/NumPy/Python version information
SciPy                1.10.0.dev0+0.df3fe4e

Thanks. gh-16427 fixes it.


fix price value

Fixes #542
Example to test on rinkeby : https://market.oceanprotocol.com/asset/did:op:5445724541a15b257013588c1CDB878B5C2A33Bc

Yes, we need to do this in compute branch as well @bogdanfazakas . Not sure yet what the exact difference is, will check out the graph.

Yes, we need to do this in compute branch as well @bogdanfazakas . Not sure yet what the exact difference is, will check out the graph.

i'll take a look, so we should use everywhere the consume price for pool price assets

Yes, only when you create the price object


Create a Swift Package for iOS SDK

As a maintainer, I need to create a simple way for users to import the SDK
App store does not allow to publish apps that has a framework embeds another framework (umbrella frameworks). We can use it to build locally but it will fail when publishing to appstore. Each framework will have to be imported as its own Swift Package.

Swift Package Manager have the binary dependency option. We can add the Flutter framework, the plugin framework and the other dependencies as binary dependencies to a package. Each one needs to be published as an individual package too.

Acceptance criteria

User can import the Swift Package from github and build the project with TikiSdk


feat(cargo-generate): add a topic to find cargo-generate templates

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md.
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

I think it would be a good idea to help people find rust application templates that cargo-generate can use to build. The cargo-generate docs links already to the GitHub topic.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Thanks, I adjusted it according to your feedback. (find notes inline)

Oh, sorry, looks like the linter caught a few more style problems:
  1) Failure:
topics::cargo-generate topic#test_0004_uses the right format for 'released' [/home/runner/work/explore/explore/test/topics_test.rb:52]:
please format 'released' like MONTH DD, YYYY with the month in English.
Expected ["January", "February", "March", "April", "May", "June", "July", "August", "September", "October", "November", "December"] to include "Oct".

  2) Failure:
topics::cargo-generate topic#test_0025_follows the Topic Page Style Guide [/home/runner/work/explore/explore/test/topics_test.rb:328]:
Always use correct capitalization when referring to "Git".
Expected "cargo-generate is a developer tool to help you get up and running quickly with a new Rust project by leveraging a pre-existing git repository as a template." to not include "git ".

wow, I'm impressed how maliciously this linter is :)
should be all fixed now.

Oh yeah, it's super picky. (Which is a good thing, I know I'd forget to look for stuff that it checks for...) I'd expect a little bit of back-and-forth to get everything green 👀

There, it'll merge when green. Thanks for the patience 😄


Asp.Net MVC/Razor Sample

Can someone please post a sample of this library that can demonstrate how to track changes on an Asp.Net web client e.g. MVC or Razor Pages? Or can someone guide how to achieve this?
The sample is not sufficient enough for realtime problems in the web.

Some preliminary work was done on a JavaScript client for Trackable Entities. But it would entail a great deal more work to complete and interest is not sufficient to warrant.


mrc-4554: Add python location list/add/remove/rename

This will add 4 new functions

orderly_location_list
orderly_location_add
orderly_location_remove
orderly_location_rename

Actually thinking about this, these should probably become outpack_location_*?
Added a few TODOs from this

Marking packets as orphaned when removing a location (will do this after have packet pulling)
Rebuilding the index after a location has been removed, again will come later
Validation? In R we're validating things like name is a string - I can't see anywhere we are doing that here and reading around sounds fairly non-pythony but wondering if you've thought about that yet?
Reserializing back to JSON this isn't handling potential of keys in the JSON which we do not serialize out like we talked about. I did see we could look at Undefined.INCLUDE from https://pypi.org/project/dataclasses-json/ which I think might achieve what we want. But can look at this later

There's a partial miss on one line but it's nbd and will become properly covered soon enough


Correct isizex8 documentation

isizex8 is now correctly described as 8 lanes.

Looks good! Thank you for your contribution!


Some issues remaining in Java 1.6

The constructor Base64(int) is undefined
The method defaultIfBlank(String, String) is undefined for the type StringUtils
ACCESS_EXTERNAL_DTD cannot be resolved or is not a field
ACCESS_EXTERNAL_SCHEMA cannot be resolved or is not a field
The constructor Base64(int) is undefined
The constructor XMLSignature(Element, String, boolean) is undefined
The import java.nio.charset.StandardCharsets cannot be resolved
The method decodeBase64(byte[]) in the type Base64 is not applicable for the arguments (String)
The method decodeBase64(byte[]) in the type Base64 is not applicable for the arguments (String)
The method sha1Hex(byte[]) is undefined for the type DigestUtils
The method sha256Hex(byte[]) is undefined for the type DigestUtils
The method sha384Hex(byte[]) is undefined for the type DigestUtils
The method sha512Hex(byte[]) is undefined for the type DigestUtils

@mguessan I forked your version. Can you help.

Looks like you don't have the right commons-codec jar.
I just had to disable tests to build with Maven:
mvn -DskipTests=true package


Kitty image previews are not scaled

What system are you running Yazi on?
macOS
What terminal are you running Yazi in?
kitty 0.31.0
Yazi version
yazi 0.1.5
Did you try the latest main branch to see if the problem has already been fixed?
Tried, but the problem is still present
Describe the bug
Terminal images previews in kitty does not scale and are not usable

Expected Behavior
This is how fzf with the jcorrect preview working
  EXTERNAL_COLUMNS=$COLUMNS \
  fzf $argv \
  --preview='kitten icat --clear --transfer-mode=memory --scale-up --place="$COLUMNS"x"$LINES"@(math $EXTERNAL_COLUMNS-$COLUMNS)x0 --align center --stdin=no {} > /dev/tty' \
  --preview-window "right,50%,border-left"

To Reproduce
Open any folder containing images using yazi in kitty
Configuration
This is reproducible on totally default configuration
Anything else?
I assume that there might be a missing --scale-up argument when getting a preview from the kitty. I can try to check the code and see if I can fix this issue once will get a green light from maintainers.

Hi, you need to tune the max_width and max_height up as what you want - which will fit your terminal automatically, that's necessary for pre-caching images

Thank you, @sxyazi for a quick answer, I am sorry for somehow missing this. I did what you said and set both of them to 4000. I also set image_alloc to 0 and image_bound to [0,0] (all unlimited) and still seeing this

Did you do a yazi --clear-cache after setting max_width and max_height inside [preview] in your ~/.config/yazi/yazi.toml?

Yes exactly

Oh, sorry, I seem to have misunderstood your intention.
Are you looking to enlarge an image that is initially smaller than your terminal window and to fill in it, rather than having an image larger than the terminal window to downscale to the terminal size?
If so currently, this is only can be implemented by the custom previewer - I'm not sure if it makes sense since enlarging a small image usually results in blurriness. However, I'm willing to discuss it and explore any possible way to impl it. Let me know your thoughts.

I'm sorry to come back to a closed issue, but I feel this is a feature that could be interesting in some cases.
I'm not well versed in Rust, but I looked at the Image implementation, especially at the downscale method: to me it seems like it could already work as a upscale method too if a (to add) flag can be passed to skip the fast path check and the check before the resize.
Pseudo-code (idk if its correct Rust):
// Fast path.
if img.width() <= w && img.height() <= h && orientation <= 1 && !upscale {
	return Ok(img);
}

tokio::task::spawn_blocking(move || {
	if (img.width() > w || img.height() > h) || upscale  {
		img = img.resize(w, h, Self::filter())
	}
	Ok(Self::rotate(img, orientation))
})


lint_serial_number_longer_than_20_octets does not check the encoded value length of the serial number

The serial number length check in
https://github.com/zmap/zlint/blob/71e2966bbbbd8a3ae6e7ff5f07502f6f59926757/v2/lints/rfc/lint_serial_number_longer_than_20_octets.go#L51 permits overly long serial numbers. Namely, serial numbers consisting of 160 bits with the MSB set to 1 will require a leading 0x00 octet in the encoded value to denote the sign, which yields an encoded length of 21 octets; the current logic does not account for this leading octet. While RFC 5280 is not clear on whether this is a violation, discussion on MDSP (specifically this comment: https://groups.google.com/g/mozilla.dev.security.policy/c/b33_4CyJbWI/m/flGvK3DeBgAJ) indicates that several client implementations will not accept certificates with a serial number whose encoded value exceeds 20 octets.
In order for certificates that pose interoperability concerns to be appropriately flagged, the check should cause the lint to error out if the BitLen of the serial number value exceeds 159 bits.

Example where certlint correctly flags but zlint does not: https://crt.sh/?id=177041525&opt=cablint,zlint


Zooming on SVG element is not working

Subject of the issue
SVGElement.offsetWidth/offsetHeight has been deprecated on Chrome and returns "undefined". Panzoom still makes use of it, see https://github.com/timmywil/jquery.panzoom/blob/master/src/panzoom.js#L555.
Your environment

jquery.panzoom.min.js 3.2.2
jquery.min.js 3.1.1
Chrome 55

Expected behaviour
SVG should zoom
Actual behaviour
SVG is not zooming

+1

+1
I think my issue is related.

@BramVanroy
https://github.com/timmywil/jquery.panzoom/issues/310#issuecomment-282638161
helped me


Add external links page section

Implements issue #95.
TODO:

Need to find better logos for external cloud providers
Refactor common code
Add tests

Hey @epapbak thanks for getting this done! I think we're already pretty close. A couple of things I've noticed:
* The flex layout you're using for the provider cards is aligning the content on the left side. The proposal had the content centered with the left card taking 1/3 and the right box 2/3 of the horizontal space with each cards content centered. My flexbox / grid layout knowledge is somewhat limited @KKoukiou @lucasgarfield could you assist on this? I'll gladly share our wireframe with you.

* I don't know what the purpose of the empty LinkHeader.ts file is. Can we remove this?

Sure - please share the expected view (mockup) and some instructions on how to build this ideally, so that I can try this out.

Hey @epapbak thanks for getting this done! I think we're already pretty close. A couple of things I've noticed:
* The flex layout you're using for the provider cards is aligning the content on the left side. The proposal had the content centered with the left card taking 1/3 and the right box 2/3 of the horizontal space with each cards content centered. My flexbox / grid layout knowledge is somewhat limited @KKoukiou @lucasgarfield could you assist on this? I'll gladly share our wireframe with you.

* I don't know what the purpose of the empty LinkHeader.ts file is. Can we remove this?

Sure - please share the expected view (mockup) and some instructions on how to build this ideally, so that I can try this out.

I've send you the sketch file. You can run the project after "npm install" either through "npm run start:dev" or "npm run build && npm start" for the production build.

Hey @epapbak thanks for getting this done! I think we're already pretty close. A couple of things I've noticed:

The flex layout you're using for the provider cards is aligning the content on the left side. The proposal had the content centered with the left card taking 1/3 and the right box 2/3 of the horizontal space with each cards content centered. My flexbox / grid layout knowledge is somewhat limited @KKoukiou @lucasgarfield could you assist on this? I'll gladly share our wireframe with you.
I don't know what the purpose of the empty LinkHeader.ts file is. Can we remove this?

the empty LinkHeader.ts --> my bad, I started refactoring and pushed it without noticing. Removing =)

I’ll be on holiday for the next two weeks so this is the last feedback I have for a while! I was actually able to make a lot of progress on the layout. I recommend using <Card> components in a <Grid>. You can do something like this:
      <PageSection variant={PageSectionVariants.light}>
        <Grid hasGutter>
          <GridItem md={4} sm={12}>
            <InternalLinksCard />
          </GridItem>
          <GridItem md={8} sm={12}>
            <ExternalLinksCard />
          </GridItem>
        </Grid>
      </PageSection>

For the external links card, getting the icons to work well is a bit trickier. I settled on taking this approach, using <Flex>:
const loadImage = (name: Provider) => {
  switch (name) {
    case Provider.aws:
      return aws_logo;
    case Provider.azure:
      return azure_logo;
    case Provider.google:
      return google_logo;
  }
};

const displayText = (name: Provider) => {
  switch (name) {
    case Provider.aws:
      return 'Launch in AWS';
    case Provider.azure:
      return 'Launch in Azure';
    case Provider.google:
      return 'Launch in Google Cloud';
  }
};

enum Provider {
  'aws',
  'azure',
  'google'
}

type ProviderCardProps = {
    provider: Provider;
  }

const ProviderCard = ({ provider }: ProviderCardProps) => {
  return (
              <Stack>
              <StackItem>
              <Bullseye>

              </Bullseye>
              </StackItem>
              <StackItem>
              <Bullseye>
              <Button variant="link" isLarge>
                {displayText(provider)} <ArrowRightIcon />
              </Button>
              </Bullseye>
              </StackItem>
              </Stack>
            )
}

const ExternalLinksCard: React.FunctionComponent = () => {
  return (
    <Card isLarge isFlat>
      <CardTitle>
      <Stack>
      <StackItem>

      </StackItem>
      <StackItem>

Get the latest Red Hat Enterprise Linux certified image for cloud deployment

      </StackItem>
      </Stack>
      </CardTitle>
      <CardBody>

Launch the latest RHEL certified cloud image through available cloud service provider marketplaces.

    <Flex justifyContent={{ default: 'justifyContentCenter'}}>
    <FlexItem>
      <ProviderCard provider={Provider.aws} />
    </FlexItem>
    <FlexItem>
      <ProviderCard provider={Provider.google} />
    </FlexItem>
    <FlexItem>
      <ProviderCard provider={Provider.azure} />
    </FlexItem>
    </Flex>
      </CardBody>
    </Card>
  );
};

For the internal links card, I took this approach:
const InternalLinksCard: React.FunctionComponent = () => {
  return (
    <Card isFlat isFullHeight isLarge>
      <CardTitle>
      <Stack>
      <StackItem>

      </StackItem>
      <StackItem>

Build your own RHEL cloud image

        </StackItem>
      </Stack>
      </CardTitle>
      <CardBody>With Red Hat's Image Builder, you can create customizable and repeatable operating system images and server images.</CardBody>
      <CardFooter>
    <Button variant="link" isLarge>
      Build an image <ArrowRightIcon />
    </Button>
      </CardFooter>
    </Card>
  );
};

There are still a few small things to do here and there. You’ll need to remove the padding on the left from the “Build an image →” link. And I have not yet added <Divider> components in-between the cloud provider icons.
I actually think the dividers are going to be quite difficult. How does it behave at small viewports? Should the icons stack vertically? How does the transition from horizontal to vertical occur? I feel like what I have now works pretty well. Maybe you can ask the designer if the <Divider> is really required? Here’s how what I’m proposing works:

We actually have a very similar pattern in the Image Builder Wizard, and we don’t use dividers for the cloud icons. So that is an established pattern you can point to, for consistency’s sake.

@lucasgarfield I really like that look! This is way more mobile friendly then the rest of the page.

I’ll be on holiday for the next two weeks so this is the last feedback I have for a while! I was actually able to make a lot of progress on the layout. I recommend using <Card> components in a <Grid>. You can do something like this:
      <PageSection variant={PageSectionVariants.light}>
        <Grid hasGutter>
          <GridItem md={4} sm={12}>
            <InternalLinksCard />
          </GridItem>
          <GridItem md={8} sm={12}>
            <ExternalLinksCard />
          </GridItem>
        </Grid>
      </PageSection>

For the external links card, getting the icons to work well is a bit trickier. I settled on taking this approach, using <Flex>:
const loadImage = (name: Provider) => {
  switch (name) {
    case Provider.aws:
      return aws_logo;
    case Provider.azure:
      return azure_logo;
    case Provider.google:
      return google_logo;
  }
};

const displayText = (name: Provider) => {
  switch (name) {
    case Provider.aws:
      return 'Launch in AWS';
    case Provider.azure:
      return 'Launch in Azure';
    case Provider.google:
      return 'Launch in Google Cloud';
  }
};

enum Provider {
  'aws',
  'azure',
  'google'
}

type ProviderCardProps = {
    provider: Provider;
  }

const ProviderCard = ({ provider }: ProviderCardProps) => {
  return (
              <Stack>
              <StackItem>
              <Bullseye>

              </Bullseye>
              </StackItem>
              <StackItem>
              <Bullseye>
              <Button variant="link" isLarge>
                {displayText(provider)} <ArrowRightIcon />
              </Button>
              </Bullseye>
              </StackItem>
              </Stack>
            )
}

const ExternalLinksCard: React.FunctionComponent = () => {
  return (
    <Card isLarge isFlat>
      <CardTitle>
      <Stack>
      <StackItem>

      </StackItem>
      <StackItem>

Get the latest Red Hat Enterprise Linux certified image for cloud deployment

      </StackItem>
      </Stack>
      </CardTitle>
      <CardBody>

Launch the latest RHEL certified cloud image through available cloud service provider marketplaces.

    <Flex justifyContent={{ default: 'justifyContentCenter'}}>
    <FlexItem>
      <ProviderCard provider={Provider.aws} />
    </FlexItem>
    <FlexItem>
      <ProviderCard provider={Provider.google} />
    </FlexItem>
    <FlexItem>
      <ProviderCard provider={Provider.azure} />
    </FlexItem>
    </Flex>
      </CardBody>
    </Card>
  );
};

For the internal links card, I took this approach:
const InternalLinksCard: React.FunctionComponent = () => {
  return (
    <Card isFlat isFullHeight isLarge>
      <CardTitle>
      <Stack>
      <StackItem>

      </StackItem>
      <StackItem>

Build your own RHEL cloud image

        </StackItem>
      </Stack>
      </CardTitle>
      <CardBody>With Red Hat's Image Builder, you can create customizable and repeatable operating system images and server images.</CardBody>
      <CardFooter>
    <Button variant="link" isLarge>
      Build an image <ArrowRightIcon />
    </Button>
      </CardFooter>
    </Card>
  );
};

There are still a few small things to do here and there. You’ll need to remove the padding on the left from the “Build an image →” link. And I have not yet added <Divider> components in-between the cloud provider icons.
I actually think the dividers are going to be quite difficult. How does it behave at small viewports? Should the icons stack vertically? How does the transition from horizontal to vertical occur? I feel like what I have now works pretty well. Maybe you can ask the designer if the <Divider> is really required? Here’s how what I’m proposing works:

We actually have a very similar pattern in the Image Builder Wizard, and we don’t use dividers for the cloud icons. So that is an established pattern you can point to, for consistency’s sake.
I think the code snippets above should be enough to get the idea but you can check out my branch here as well, apologies for the mess on the code formatting and commits 😉:
https://github.com/lucasgarfield/cloud-image-directory-frontend/tree/external-links

next level code reviewing 😮 !!
I played around with grids and levels and couldn't make anything close to this, so I'll be borrowing this code 😉

Thanks @epapbak, glad you like it! :grinning:
Let me know when it's ready for a second look, and don't hesitate to ping me if you get stuck on something.

@epapbak Just a quick status check. Is this ready for a final review or do you need more time?

It is ready for review, yes :)
El miércoles, 26 de julio de 2023, FKolwa @.***>
escribió:

@epapbak https://github.com/epapbak Just a quick status check. Is this
ready for a final review or do you need more time?
—
Reply to this email directly, view it on GitHub
https://github.com/redhatcloudx/cloud-image-directory-frontend/pull/141#issuecomment-1651576238,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADZHUU4W4GAJ2TFN3BOTZKLXSD3Z5ANCNFSM6AAAAAAZWUSUHM
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Looking good, thanks for getting this done!
@lucasgarfield @KKoukiou anything left that needs changing?
Otherwise I'd vote for creating follow up issues and getting this in.

Yep, ever growing pull requests (plus my messed up availability) might block some other work, so I'd prefer follow-up issues too.

LGTM! I agree that any issues that arise can be dealt with in follow-up PRs. Great work @epapbak. :grin:


How to develop a chrome app for gmail attachments->open with->connect more apps?

I am new to chrome apps, i don't know how to develop chrome apps.
i have a pdf in gmail attachments, if i click the pdf will open with popup screen and in there open with is there in top? if i click on open with, connect more app will appear, so like that if i search any apps it will take me to web store, like how to develop and publish my own app?
I tried but im not gtiing till now,... Please help me?
emailid<EMAIL_ADDRESS>

+1


Lombok specific IntelliJ inspections

Hi guys,
I created and collected a couple of IntelliJ inspections (Structural Search and Replace templates) for Lombok and I would like to get your opinion about them whether at least the concept of them is correct or they may need refinements. Also if you know any other such code inspection that could help Lombok users, please let me know.
You can find the document here: https://paper.dropbox.com/doc/Lombok-inspections--AX3NQGoGN5gbgPU2AT9FlpZrAQ-A3YaGGSh10Fn3Y7WK5tHX

Not sure I get the point of the 'turn lombok-generated nullchecks via the lombok.NonNull annotation into explicit nullcheck' one, and I can think of a lot more of these, but this is awesome.
I'm not too familiar with intellij, and lombok support for intellij is provided via the lombok intellij plugin which is a separate project. I assume you would add this XML to the config of the plugin at which point these templates show up as available inspections?
The next step would be to file an enhancement over at the github issue tracker for lombok-intellij-plugin, possibly with a PR, but if Michael Plushnikov is anywhere near as positively inclined as I am about further IDE support, this should be accepted forthwith :)
If I misunderstood and there's some more action we need to take, by all means let us know :)

@rzwitserloot Thanks. :)
The reason I created them is that I didn't see any notification from the Lombok plugin marking these issues. But I will check again to make sure I didn't miss anything.
Right now what I know is that these XML configs can be included in the global config folder of IDEA, or it can be made project specific by putting them into <project>/.idea/inspectionProfiles/Project_Default.xml file by the project developers.
I'm not familiar with IntelliJ plugins, so I'm not sure whether they can be used in the plugin in this form or if they would need to be reimplemented in Java.

@picimako Please open an issue in https://github.com/mplushnikov/lombok-intellij-plugin. I'm owner of this plugin an can add some usefull inspections to the plugin :)

@mplushnikov Alright, I will open an issue there. Thanks. :)


Could you provide a more detailed Readme?

Thank you for your excellent work. Could you provide more details on the training and data preparation to help me reproduce it?

Hi, I will try to update the readme next week.

Thanks! lol

Hi, I provided some details in https://github.com/sporsho/3D_OutDet/tree/2-could-you-provide-a-more-detailed-readme
I will merge the branch after further fine tuning the text. In the meantime, you can get started with the new branch.

Hi, I provided some details in https://github.com/sporsho/3D_OutDet/tree/2-could-you-provide-a-more-detailed-readme I will merge the branch after further fine tuning the text. In the meantime, you can get started with the new branch.

Thank you for your reply. This is a detailed instruction.


docs(tags): improve writing style

Description

Made doc sections more consistent
Replace duplicated styling in mermaid diagram with classes
Update yaml examples to use consistent formatting and descriptions

Related issue(s)
Fixes #3410
See also #3375

Summary by CodeRabbit

Documentation

Restructured and clarified the "Tags" document in AsyncAPI for improved readability.
Simplified definitions and enhanced explanations of the tags object and its properties.
Expanded usage section for tags at the info level to clarify their categorization role.
Updated examples and visual representations for consistency and readability.

/update


Custom registry using X-AUTH HTTP header for auth

I have been using the spotify's maven plugin(customized to a CI servie) to build a docker image ,then  push it to a registry of my company.
I do not know the specific implement of our registry along with the auth system.What I do know is after building the image, I push/post it to a http url instead of https.like http://registry.com/app/ciservice:0.0.1 .
And used a specific header like X-AUTH: ${TOKEN} for auth.
I consider jib's ability to build image without docker daemon is quite fancinating and can be quite useful.But when I run

mvn jib:build

I have no idea how I can push the image to our customized dockerregistry and customized auth system.As I have mentioned , a post request to http url along with a specific header.
So, how can I achieve my purpose ? If customizing the jib-maven-plugin is unavoidable, which part of the code should I start with.

Jib supports building to a tar with the jib:buildTar goal, with the resulting tar placed in target/jib-image.tar.  What format does your registry accept?

Hi @zy475459736,
I see there is currently no way to set the X-Auth header for auth.
We set the Authorization header (using the basic access authentication scheme), which we set by calling Reqeust.setAuthorization(). The places where we set the header are

RegistryAuthenticator.authenticate()
RegistryEndpointCaller.call()

And it looks like your registry doesn't support HTTPS, so you will need to set allowInsecureRegistries and -DsendCredentialsOverHttp, but you should know the danger and security risks of using them.

And I got curious: how do you set the X-Auth with the Spotify plugin? Did you customize it yourself or does it come with the feature to configure it?

@zy475459736 How are you pushing/posting your image?  With docker push?  Or some other means?

Oh, forgot to say this: if you have other means to push an image from your local Docker daemon to your company registry, note that mvn jib:dockerBuild will push an image to your local daemon instead of a remote registry. You may be able to use this workaround too.

@briandealwis Thanks for your remind of command mvn jib:buildTar.I had a try and I think it does satisfy the purpose for I can get the same image from the tars.
But as you have mentioned, although I got the tars, I still need the docker daemon to obtain the images and push them to the registry.It seems there is no other way left for me to reach my goal( image stored in registry) directly .
I will check with my colleague about the details of the registry next week, to see whether the transition from tars to images can be done by the registry or some other way.

@chanseokoh the Spotify maven plugin supports pushing  the images generated to registry.
I have done some digging on the plugin and find out that the both the build and push goal of Spotify's plugin are ultimately done by docker daemon.It is more like a Client/Server ,the plugin acting as a client.

@chanseokoh Thanks for your warning about the risk HTTP has. But the registry can only be accessed inside the company, and the colleague who maintains the registry is busy with some other projects so...But thanks anyway.
My boss just wants to have the image without docker daemon so we can have more convenience when we design the CI/CD and some other infrastructure.
I will do some digging on the jib code and try to get my X-Auth-header added next week.

@zy475459736 out of curiosity, were you able to make this work?

@briandealwis Somehow, I do achieve my goal.
We have devloped an OAuth and ContainerRegistry of our own running inside the company.
The reason why I raised this issue is that the CI-service I devloped before relyed on the Spotify's maven-plugin to transfer the code to Docker Image and to push it to the ContainerRegistry if necessary.

The goods to use this plugin ,as have been mentioned in the video ,there will be no need for Java devlopers to write the Dockerfile which is kind of like a script,not to mention the size of the image also depends on how you write the Dockerfile.
The Spotify's maven-plugin use the info you offer(the orgin version of the plugin is on the pom.xml, I did some work here so the devloper can edit the info on the webUI... And also some other functions) to arrange a Dockerfile and then connects to Docker Daemon to do the docker build & docker push .

I did some customization on the Spotify's plugin so it can be compatible with the OAuth and the ContainerRegistry I mentioned before: in short, add the token I get from OAuth-WebUI (which is long-term effective for now because the whole system is not public anyway) to the plugin, it will use the token as a header to push it to a url (which is a HTTP for now).

@briandealwis  com.google.cloud.tools.jib.builder.BuildSteps.forBuildToDockerRegistry()
public static BuildSteps forBuildToDockerRegistry(BuildConfiguration buildConfiguration) {
    return new BuildSteps(
        DESCRIPTION_FOR_DOCKER_REGISTRY,
        buildConfiguration,
        stepsRunner ->//steps:
            stepsRunner
                .runRetrieveTargetRegistryCredentialsStep()//retrieve the credential you arrange,like username&password
                .runAuthenticatePushStep()//the OAuth system has a api ,so I can use username&password to exchange for an access token for pushing.
                .runPullBaseImageStep()
                .runPullAndCacheBaseImageLayersStep()
                .runPushBaseImageLayersStep()
                .runBuildAndCacheApplicationLayerSteps()
                .runBuildImageStep()
                .runPushContainerConfigurationStep()
                .runPushApplicationLayersStep()
                .runFinalizingPushStep()
                .runPushImageStep()
                .waitOnPushImageStep());
  }

And thanks for the suggestion @chanseokoh offered, I set allowInsecureRegistries and -DsendCredentialsOverHttp so I can push the image to HTTP url.
I can do some more customization to make jib more suitable for use inside my company. Anyway, it does satisfy the requirement (build and push a image without DockerDaemon and I can pull it from the ContainerRegistry).

Reference about container registry auth.  In case someone might need it.

@zy475459736 Thanks for providing all these details! Glad this all works for you now.

@zy475459736 I see. So your requirement was not that you need to set a different header (i.e., X-Auth-Token: instead of Authroization:), but rather you mistakenly thought Jib could not push to your registry without knowing Jib by default doesn't pass auth to HTTP (although it sounds like you still needed some customization to Jib to retrieve the auth info from your Web UI and make Jib to pick it up).


Clearing filters sends request first

When filter panel is open and I click CLEAR, it sends the request first, during which we don't see any progress. Sometimes it's misleading to end users who do not know something is working in the background. This is also reinforced by not emptying the filters before the request but after.
I understand this could be in case the clearing request fails but is it the best way?

Sorry, but I'm not able to follow completely. What do you mean with "it sends the request first" (first of what)? Do you mean that the request is being sent before the form elements are emptied? If so, that´s intended behaviour (although I'm not sure if it´s the best possible option).

I assume it's intended, just from the usability perspective it may not be the best, therefore is there a way to change the order of execution of those things, or it's rather not configurable?


Error while installing npm modules

I am getting following errors. I tried deleting nmp_modules folder and re-installing as per instructions. But it's not working. I am not able to start the application, it says
2017-11-24T00_21_33_157Z-debug.log
Please get back at the earliest.

You will need to use Node.js 7. Node.js 8 may work, but the problem you appear to be running into is compilation issue with node-sass.

Thank you for the response. Upper version of nodejs should ideally work.
What shall I have to do to resolve compilation problem with node-sass.
Regards,
Jayashree
On Nov 24, 2017 6:36 PM, "Eric W. Greene"<EMAIL_ADDRESS>wrote:
You will need to use Node.js 7. Node.js 8 may work, but the problem you
appear to be running into is compilation issue with node-sass.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/training4developers/mastering-angular-starter/issues/5#issuecomment-346824883,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgXx83GnrXj7MfWdjjUgweiaQr-bWOwvks5s5r9ygaJpZM4QpQ1B
.

Please log into GitHub to read my reply. I updated my original post on the issue with instructions. Thanks!

Hallo Eric,
I started fresh with installing latest version of node js, python 2.7 and turbo c++(for c++ compiler). This time I am getting following errors. Unless I run the demo code, I won't be able to do any changes and hands on.
2017-12-13T09_19_33_888Z-debug.log

I am very sorry you are having problems. It looks like you are having several problems including downloading files, build tool problems, file permissions and such. These issues are something you will need to resolve with your overall system setup. If you are on Windows, the recommended C++ compiler for Node.js is Visual Studio. I recommend downloading the community edition and trying that. You may need to uninstall Turbo C++ to avoid conflicts in terms of paths to the C++ compiler and such. I have not used Turbo C++ since the 1990's so I am not sure how it would interact with Node.js, the latest versions of Windows or the project.
Another option is to use the Angular CLI. You can install that and follow along with the videos too.

In my humble opinion, this whole problem should go away. I paid cash money to learn Angular 2 and what I got is a package which will not work as a demonstration example. I think the package should work with the latest releases of ALL components. Why should I have to use a backwards environment to learn something that should be new? If you can't spend the time to make your products "up-to-date" then you should not be charging for their use. I am not a happy camper as I have spent way too much time compensating for your lack of attention to this matter....

Hi WeAreUSS,
I replied privately to your direct email to me (within minutes of receiving it) with the best solution to your problem. This git repo solution was the starter project solution I used before the Angular CLI became very popular and stable. I recommend using the Angular CLI instead of this project for new projects. The videos are only valid for the current versions of the software as of the time of recording.
So, I just re-installed Node.js 6, cloned this repo, installed the packages and everything ran perfectly for me. So I am not sure what is wrong with your system, but providing technical support on your system is outside of the scope of what I can do for you. Sorry.
Also, when I immediately replied to your private, direct email to me it is not appreciated for you to chase this down in a public forum especially when the software does work. I see you are new to GitHub. GitHub is a collaborative community where developers work together to solve problems (just like I kindly attempted to do with you privately). Unfortunately, coding software is often filled with frustrations especially with JavaScript projects dependent upon NPM packages.
As I said, I just freshly downloaded the project using Node.js 6 and all worked perfectly for me the first time. I hope you are able to get this working.
Thanks,
Eric


Update vue.d.ts

hi! I want use Vue with typescript but found the d.ts files was outdated. I updated it and should be ok for use with vuejs 0.12.14.

vue/vue.d.ts
to author(@odangosan). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

@TsumiNa is this still needed? can you update the PR?

Hello and thank you for your contribution!
Due to an excessively long queue of pull requests that have become stale, we are "declaring bankruptcy" and closing all PRs opened before May 1, 2016. If you'd still like to merge this code in, please open a new PR that has been merged and rebased with the master branch.
Going forward, we are committing to review or merge all PRs on a regular basis so this bankruptcy will not occur again. We apologize for the incovenience and hope you will continue to contribute to DefinitelyTyped in the future.
Thanks
The TypeScript and DefinitelyTyped teams


Learn LaTeX

Business analyst must learn to use latex for all documentation

We decided for now not use Latex and if needed to switch to Latex


[Feature]:  Auto Commenting when an Issue is Closed

Description
The solution is to automatically post a comment on the issue whenever it gets closed. This comment should thank the issue creator for their contribution and inform them that their issue has been closed. The message can include the issue number and a friendly greeting to make it more personal. This automation ensures that every closed issue receives a consistent and timely response, improving communication and contributor experience.
Why implement this feature?
Contributors may feel neglected or uncertain about the status of their issues if they are closed without any feedback or acknowledgment. This can lead to frustration and a decrease in motivation to contribute further.
Any additional information?

Checklist

[X] I have checked the existing issues and made sure that this is not a duplicate feature request form.

@devXprite please assign me this issue


Frontend / Backend Feature Tracking

Character Management
Attribute management

[ ] Backend - add an adjust_attributes api that can adjust multiple character attributes at once.

Ability management

[ ] Backend - add which path abilities come from when doing web scraping
[ ] Backend - when scraping web text, if the first lines are italic, note it as flavor and don't include that text in the main description.
[ ] Frontend - keep track of the total amount of XP spent

Create item / weapon shop

[ ] Backend - Support characters owning an item container with item name, bulk capacity, and description. Ideally we would have an API for returning all available (https://vennt.fandom.com/wiki/Item_Containers)
[ ] Backend - would be great if we made an API to return all item options instead of storing info on both frontend and backend (https://vennt.fandom.com/wiki/Equipment & https://vennt.fandom.com/wiki/Consumables)
[ ] Frontend - need to make new page for this - probably accessible on character management side

Enemy Management

[ ] Backend - support enemy abilities / custom enemy abilities.
[ ] Backend - Allow for enemies to be copied and renamed if multiple are needed for a fight.
[ ] Frontend - Make enemy creation page

[ ] Allow you to import specific numbers (re-purpose character import page)
[ ] Allow you to make an enemy as a character (see https://vennt.fandom.com/wiki/Course_of_Combat, re-use character creation flow)
[ ] Allow you to import enemies from the bestiary (https://vennt.fandom.com/wiki/Category:Bestiary)
[ ] Allow you to create new enemies (see https://vennt.fandom.com/wiki/Course_of_Foes)

Campaigns / Combat

[ ] Backend - add a log for all combat / campaign actions so everyone can see a history of what's happening
[ ] Backend - add an endpoint for rolling attribute checks and adds result to log. Should be able to optionally provide your roll or the server will roll for you (see initiative example: https://github.com/joshmiller17/vennt-server/blob/main/api_initiative.py#L48-L56)
[ ] Backend - create some utility function for rolling dice

[ ] Should support Flow / Ebb
[ ] Should support using a hero point - e.g. roll 3d6 + 9 and drop the lowest dice

[ ] Backend - add API for modifying entity details - e.g. number of actions / reactions / gm_only etc.
[ ] Backend - add basic API for creating an attack

[ ] supports taking target & damage / accuracy numbers
[ ] supports taking target & weapon & optional roll

[ ] Backend - API for responding to an attack.

[ ] supports inputing damage taken
[ ] supports allowing server to calculate and apply damage (e.g. using your vim, checking if its a glancing blow, etc.)
[ ] supports commonly used special cases:

[ ] blocking attacks (if entity has ability)
[ ] dodging attacks (if entity has ability)

[ ] Backend - when starting a new turn, allow the number of actions / reactions received to be modified in some way (maybe a constant in the entities table? Ideally we should be able to track where bonuses are coming from / when they are supposed to end. Maybe a solution here can be generalized for other campaign / combat features)
[ ] Frontend - implement sending campaign invites
[ ] Frontend - implement updating user roles if campaign owner
[ ] Frontend - implement spectator campaign view
[ ] Frontend - implement character campaign view
[ ] Frontend - implement GM campaign view
[ ] Frontend - add easy enemy insertion from enemy creator for GMs

Why do we need adjust_attributes?
Path info should be available in abilities.json
I think feature for enemy copying already exists, unless it was only the Discord bot that could do that, I can check
Note that basically everything on the Bestiary probably needs to be updated
Rolling dice is already supported; again, we want the server to handle low-level rolling, frontend can calculate flow etc. I'm fine with a function for "change the lowest to 9" for hero point usage.

adjust_attributes would help for when I want to adjust a bunch of character attributes at once so I can do it with 1 API call instead of 2 or 3.

I don't think enemy copying exists right now in the code base
good to know about the bestiary - it looks like there are a couple of enemies in there that look fine? E.g. https://vennt.fandom.com/wiki/Thunder_Lizard
I think I've had a change of heart on handling dice rolling - I would really like to support manually rolling dice (like in real life) or allowing the frontend to roll dice. Since users can input information for everything else, this doesn't seem too strange. Rolling dice is handled in the sense that we have d20, not in the sense that we have some centralized utility that we can use to handle all of the weird edge cases etc.

Or instead of making a new API like adjust_attributes, we could update set_attribute to be able to update multiple attributes at once

Or instead of making a new API like adjust_attributes, we could update set_attribute to be able to update multiple attributes at once

I prefer this approach. Better yet might be to replace set_attribute with a  set_attributes call that takes a list (though that list can be only one element). Up to you.

adjust_attributes would help for when I want to adjust a bunch of character attributes at once so I can do it with 1 API call instead of 2 or 3.

I don't think enemy copying exists right now in the code base
good to know about the bestiary - it looks like there are a couple of enemies in there that look fine? E.g. https://vennt.fandom.com/wiki/Thunder_Lizard
I think I've had a change of heart on handling dice rolling - I would really like to support manually rolling dice (like in real life) or allowing the frontend to roll dice. Since users can input information for everything else, this doesn't seem too strange. Rolling dice is handled in the sense that we have d20, not in the sense that we have some centralized utility that we can use to handle all of the weird edge cases etc.

I'm not sure if Thunder Lizard's stats are correct as of the latest L-based function on the Foes page, though I'm not sure it matters a great deal.
Regarding rolling, I would be perfectly fine with having rolls be client-side rather than server-side. Although I think we should provide a single roll utility where the client can specify a rollstring and we simply return how d20 parses it.


bug(windows): copying an IP resource copies the "/32" at the end

Describe the bug
If I click on a resource it copies the /32 as if it's a CIDR, which makes it impractical to paste in a web browser.
To Reproduce
Sign in, open the tray menu, click on a resource, try pasting it in a browser
Expected behavior
If it's actually a CIDR resource like a /24, this is fine, but for an IPv4 /32 or an IPv6 /128, it should drop that off. Otherwise it'll be interpreted as the path in a URL I think.
Screenshots / Logs

Platform (please complete the following information):

Firezone Version - Any recent version
OS: Windows 11

Additional context
N/A

@ReactorScram The issue here is with connlib -- it normalizes IPs coming from the portal IIRC. See https://github.com/firezone/firezone/issues/2829


Do you have an example capture file?

Hi,
I'd like to add your Lua plugin script to Wireshark's repository (I'm one of the developers).
Do you have an example packet capture to use with the script, that you would be willing to have publicly viewable?
-hadriel

Sorry, I don't have an example capture on my hand. Maybe I could try to handcraft such packets.

No don't worry about it - it was more of a nice-to-have than anything.  Your Lua plugin is in the repository here:
https://wiki.wireshark.org/Contrib


classExists does not have @psalm-assert class-string since 1.8.0

For some reason it was removed in https://github.com/webmozart/assert/commit/efafa7461ecf5f5f2eace1c31e91d8175a981f63#diff-793d5a51af95587404301a8a49a3a364L1506
@Ocramius , was it on purpose?

probably not: can you send a patch with a test please?

Sure!


Testing Requirements

Currently there is a feature to add code testing to the server application but tests havent been written. This issue will act as a checklist for pending tests
Interface testing
The first tests we have to implement is to check whether the various core components are setup properly based on the environment variables set. We can use the pytest MonkeyPatch and fixture to easily test these.
Database connection testing
These connection testing should only test whether connections work or not without create alterations in production level data. If possible, test command on dummy connection if supported by packages used.

[ ] SQL database connection testing
[ ] MongoDB database connection testing
[ ] Redis cache connection testing

Route testing
Using the fastapi TestClient we can send request and check whether the right response is received from those requests. These tests will happen on data stored in TestSource which is a internal container used to dummy data source. Route checking should also check how well the request handles errors and exceptions.

The difference between the dummy source used here and dummy connection mentioned above is that the dummy source is to emulate a data storage whereas a dummy connection is to emulate a connection to a database/datastore of that type

Account Routes testing
Test conducted to check the authentication and authorization systems in place.

[x] Auth check
[x] Register Programmer
[x] Register User
[x] Login User
[x] Login Programmer
[x] Token verification for User
[x] Token verification for Programmer
[ ] Token verification for Admin use

ARgorithm Route testing
Test conducted to check the argorithm management interface

[ ] ARgorithm list
[ ] ARgorithm submit
[ ] ARgorithm update
[ ] ARgorithm run
[ ] ARgorithm delete
[ ] ARgorithm view

Admin Routes testing
Test conducted to check admin priveleges

[ ] Admin delete
[ ] Admin blacklist and check if programmer has been blacklisted properly or not
[ ] Admin whitelist and check if programmer has been whitelisted properly
[ ] Admin grant and check if granted
[ ] Admin revoke and check if revoked
[ ] Admin management of ARgorithm not authored by admin

Report Route Testing

Need Bug report mechanism to be fully implemented fist
refer #15

Tests for report mechanism

[ ] User report submit
[ ] System report submit

Monitoring testing
Check Monitoring package

[ ] Performance monitor
[ ] Request middleware for successful requests
[ ] Request middleware for failed requests
[ ] Log data verification

Closing issue as repo is being archived.


add mfence for rdtsc instruction in tsc timer driver

The RDTSC instruction is not serializing or ordered with other
instructions. add mfence to prevent reordering.
There aready are mfence instructions before wrmsr function in seperate
functions in apic_tsc.c, place it in wrmsr function.
Signed-off-by: Meng xianglin<EMAIL_ADDRESS>

I removed the Stale label until @andyross has a chance to comment on this


Add a wrapper for delay

Add a wrapper for delay in IO.h instead of using arduino's delay directly

Done


jenkins: test jenkins in a loop

From the current CI weekly run job, we create a new branch that will trigger all the builds and handle the result of each stages. This branch will then upload the results of all the runs to ci-artifacts, which the CI weekly job will fetch and add to the job summary table.

It's probably easier to use the Jenkins API rather than make fake commits.

ref for commit 6206c01cdd0a87a28f9e40fbe7beafb872a055b8: https://raw.githubusercontent.com/commaai/ci-artifacts/refs/heads/model_replay_jenkins_stress/2f4452b03ccb98f0|2022-12-03--13-45-30_model_tici_6206c01cdd0a87a28f9e40fbe7beafb872a055b8.bz2All Model Replay Plots


Avoid jQuery dependency.

Fairly straight forward except for the deferred object.
http://stackoverflow.com/questions/14841338/jquery-promises-is-there-a-reusable-alternative
http://www.i-programmer.info/programming/jquery/4788-jquery-promises-a-deferred.html

Must remove Deferred in order to allow Sizzle, and reduced Zepto includes.

Resolved by #6


Cross compile to CORTEXA9 does not complete

After make clean, I ran
sudo make BINARY=32 CC=/usr/local/bin/gcc FC=/usr/local/bin/gfortran HOSTCC=gcc TARGET=CORTEXA9
This goes a LONG way forward, but dies on
ar  -ru ../libopenblas_cortexa9p-r0.2.19.dev.a samax_k.o samin_k.o smax_k.o smin_k.o isamax_k.o isamin_k.o ismax_k.o ismin_k.o sasum_k.o saxpy_k.o scopy_k.o sdot_k.o sdsdot_k.o dsdot_k.o snrm2_k.o srot_k.o
...
...
zomatcopy_k_rt.o zomatcopy_k_cnc.o zomatcopy_k_rnc.o zomatcopy_k_ctc.o zomatcopy_k_rtc.o zimatcopy_k_cn.o zimatcopy_k_rn.o zimatcopy_k_ct.o zimatcopy_k_rt.o zimatcopy_k_cnc.o zimatcopy_k_rnc.o zimatcopy_k_ctc.o zimatcopy_k_rtc.o zgeadd_k.o zneg_tcopy.o zlaswp_ncopy.o lsame.o scabs1.o dcabs1.o cpuid.o
ar: sgemm_kernel.o: No such file or directory
make[1]: *** [libs] Error 1
make[1]: Leaving directory `/home/dmazzoni/Downloads/OpenBLAS/OpenBLAS-develop/kernel'
make: *** [libs] Error 1

There shpuld be some error before.
Can you attach full build output?
$ make clean
$ make [your params] 2>&1 >>make.log

In non-cross compiling, this fault with sgemm_kernel.o appears to come up when the build system is confused about the type and capabilities of the cpu (wrong combination of arguments supplied to make, or running in an emulator that provides an unusual subset of host cpu features to the virtual machine).
What is your build environment here ?

Hello all -
Thanks for your suggestions and help. I've got OpenBLAS to cross compile using:
make CC=arm-linux-gnueabihf-gcc FC=arm-linux-gnueabihf-gfortran HOSTCC=gcc TARGET=CORTEXA9
Dave

From: Martin Kroeker<EMAIL_ADDRESS>Sent: Friday, July 29, 2016 04:03
To: xianyi/OpenBLAS
Cc<EMAIL_ADDRESS>Author
Subject: Re: [xianyi/OpenBLAS] Cross compile to CORTEXA9 does not complete (#936)
In non-cross compiling, this fault with sgemm_kernel.o appears to come up when the build system is confused about the type and capabilities of the cpu (wrong combination of arguments supplied to make, or running in an emulator that provides an unusual subset of host cpu features to the virtual machine).
What is your build environment here ?
�
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/xianyi/OpenBLAS/issues/936#issuecomment-236119409, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAgSBoVy191qI3oH9R1wiPaZnVFsoSJzks5qabPLgaJpZM4JWr_h.

An d who closes this ticket?

Would it make sense to create a wiki entry out of this (I see there is "how to build for android, but no cross-compiling tip that fits davmaz' case) ?

Yes, that would be nice. I can outline what I did for everything to work (thanks to you).

From: Martin Kroeker<EMAIL_ADDRESS>Sent: Thursday, August 04, 2016 08:54
To: xianyi/OpenBLAS
Cc<EMAIL_ADDRESS>Author
Subject: Re: [xianyi/OpenBLAS] Cross compile to CORTEXA9 does not complete (#936)
Would it make sense to create a wiki entry out of this (I see there is "how to build for android, but no cross-compiling tip that fits davmaz' case) ?
�
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/xianyi/OpenBLAS/issues/936#issuecomment-237543867, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAgSBuAzVp4nGJ9hJhWGOOkp6BKJl7Thks5qceEKgaJpZM4JWr_h.

I believe the wiki is currently open for anyone here to write.
(btw brada4's not so subtle hint refers to the fact that a ticket can only be closed by its creator or by the project owner - xianyi appears to be pretty busy with other things at the moment and there is a slow but steady buildup of resolved but not closed tickets that threatens to hide the really important open problems.)

These steps worked:
Find all installed packages for the ARM compiler: sudo aptitude search arm-linux-gnueabi
Purge ALL gnueabi versions: sudo apt-get purge --auto-remove gfortran-arm-linux-gnueabi, etc for any "installed"
Install only gnueabihf versions: sudo aptitude install gfortran-arm-linux-gnueabihf (this also installs gnueabihf gcc)
From the OpenBLAS directory: make CC=arm-linux-gnueabihf-gcc FC=arm-linux-gnueabihf-gfortran HOSTCC=gcc TARGET=CORTEXA9
Install the ARM libs: sudo make PREFIX=/usr/local install

I put the steps that worked for me in a final comment and closed the issue.
Thanks!

From: Martin Kroeker<EMAIL_ADDRESS>Sent: Thursday, August 04, 2016 11:53
To: xianyi/OpenBLAS
Cc<EMAIL_ADDRESS>Author
Subject: Re: [xianyi/OpenBLAS] Cross compile to CORTEXA9 does not complete (#936)
I believe the wiki is currently open for anyone here to write.
(btw brada4's not so subtle hint refers to the fact that a ticket can only be closed by its creator or by the project owner - xianyi appears to be pretty busy with other things at the moment and there is a slow but steady buildup of resolved but not closed tickets that threatens to hide the really important open problems.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/xianyi/OpenBLAS/issues/936#issuecomment-237596847, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAgSBvjFJoL5ju9TmKRnJkscWMdLN3Utks5qcgrjgaJpZM4JWr_h.

Add an example for cross compiling ARM.
https://github.com/xianyi/OpenBLAS/wiki/User-Manual#compile


Feature Request: Last changed for entities in entities card

Once it was possible to show last changed text in Home Assistant but they removed the feature I believe.
Maybe Lovelace can bring it back?
What I want: (now achieved with custom-UI by Andrey)

Example config:
cards:
   - type: entities
     title: Kitchen
     entities:
       - entity: light.kitchen_ceiling_light
         show_last_changed: true

Yes, now that we have easy options for rows, I agree that we can bring this back as an option.
However, I think that I would want to implement the option slightly differently, to allow more flexibility:
cards:
   - type: entities
     title: Kitchen
     entities:
       - entity: light.kitchen_ceiling_light
         second_line: last_changed

For now it would just support last changed, but allows more flexibility for the future. btw probably need a better name than second_line but lacked creativity.

I think second_line is a great name. And it would be great if it was indeed more flexible. Really think people would benefit from a secondary line with information from attributes like power consumption or last changed.


add NM_CONTROLLED=no to Azure network script

Fixes the issue of failing to create CentOS VM with multiple NICs.

Please do not merge this PR. It needs to be updated. Thanks.

Having some issues with this PR, close it before the issues are fixed.


NetTruyen v3

After updating to v3, there is an error that does not display comics

Can you explain further? I'm guessing you mean manga that you added to your library previously won't load, which is the case because the URL changed so the manga ids changed as well. You need to open the source and re-add your manga to your library.


Setting internal properties?

While upgrading one of my projects to Ditto v0.9.0-beta, I hit an error in the new PropertyInfoInvocations class - it said that the setter method passed to BuildGetAccessor is null.
I checked my model and found something like this...
public string MyProperty { get; internal set; }

I'm not entirely sure why I had originally set the property to have an internal setter, (I probably didn't need to do it), however this did work with Ditto v0.8.x, so my question is...  Should we correct this?
I'm mixed on the semantics of this, whether internal should be that only the user's code should set the value, or that it means once it's set (by Ditto), it's effectively read-only.

Hmm, should this be a config option? (default off) If so, then it's an
option of setting globally or per mapping?

I've been playing with a unit-test for this, it the setter is omitted, then Ditto will ignore it.  The internal setter throws an exception, (which is only a null check needed to fix it).
I'm currently swaying towards that we set any marked as internal.

I've put my unit-test in PR #173

Did it work with the lazy loading on 0.8x?
I'm not sure Ditto should try to set it. If you're marking something as internal there's usually good reason and we'd be breaking contract rules. We certainly shouldn't throw an exception anyway.

I didn't test lazy-loading. I'll give it a try.
Reviewing the rest of my app code, my intention for having an internal setter was to let Ditto populate the value, then let it be read-only in the my app. But I can see that it wasn't the best design decision (given no one is ever going to try to update the value in my app). I've now made it public.

Cool. I tend to avoid internal setters unless they are absolutely necessary. Even then I always feel like I have done something wrong.

Just pushed the test I did for the lazy-loading, brace yourselves, weird results.
https://github.com/leekelleher/umbraco-ditto/pull/173/files#diff-a79b95e85e860f5f18ac6cd5331f1462
The non-virtual property with internal setter doesn't set the value, but the virtual property with internal setter does. :confused:
@JimBobSquarePants For the virtual properties, does this use the Proxy code to call the Setter?

@leekelleher  Yeah it does because the proxy is intercepting the Get/Set MethodBase. It doesn't actually Get/Set the property on the item, rather it updates an internal dictionary.
You can see that here:
https://github.com/leekelleher/umbraco-ditto/blob/c68d5e633d6e3712861188637424c3221f0b5694/src/Our.Umbraco.Ditto/Proxy/LazyInterceptor.cs#L56
I think the best fix rather than augmenting PropertyInfoInvocations is to add extra sanitising code  earlier when we are collecting the properties to parse.
The line here:
https://github.com/leekelleher/umbraco-ditto/blob/c68d5e633d6e3712861188637424c3221f0b5694/src/Our.Umbraco.Ditto/Extensions/PublishedContentExtensions.cs#L318
Should probably read something like.
var properties = type.GetProperties(BindingFlags.Public | BindingFlags.Instance)
                                 .Where(x => x.CanWrite && x.GetSetMethod() != null).ToArray();

I think that should do it.

Thanks @JimBobSquarePants, good idea! I'll give it a try/test later today.

Closing this issue off, updated PR #173 with fixes.
Agreed approach is that if the property's get method is not public, then we don't attempt to set the value.


Doctrine Issue after upgrading Yii2 - Ubuntu 14.04

After updating composer doctrine is giving following error:
Return value of Doctrine\Common\Annotations\AnnotationRegistry::registerFile() must be an instance of Doctrine\Common\Annotations\void, none returned
Additional info

Q
A

Yii version
2.0.6

PHP version
7.1.1

Operating system
Ubuntu 14.04

How is this a Yii2 issue? Yii does not use Doctrine.

It is using doctrine/lexer (See Attachment) with the latest update and It is used for Doctrine Annotations. I am getting this Type Error. Earlier it was working with the older version.
So, I thought to put forward this issue here to be get rectified. I did 8-10 projects in Yii2 and I faced this issue the first time. Please suggest to solve it thanks.
New Installation:

My Current Project:

Closing since it's not Yii but Doctrine errors. Please report to Doctrine.

I have found the issue which was related to Authorize.Net. I am closing it because it is not relevant here. As the issue is defined over here. #issue


Torculus not split at all

When processing folio 010r of the Salzinnes manuscript, the three torculi weren't split into three puncta. They remained whole, and the interactive classifier identified all three as obliques. The screenshot is of the glyph as it was in the IC.

This was fixed with further training of the IC. And torculi are always a challenge anyway! Closing the issue.


Permit to use already opened socket

Hello,
I would like to use the vncdotool API to connect to noVNC server managed by my OpenNebula.
The problem is that noVNC use websocket, so I can not use vncdotool.api.connect().
Here is the use case:

I open a websocket pointing to my VM VNC
I pass that socket to vncdotool

I need something like vncdotool.api.adopt to use reactor.adoptStreamPort and use the websocket transparently.
Do you think it's possible?
Regards.

Its reasonable to expect that should work and I'd be open to having such an API in vncdotool.

Thanks for the quick reply, I add to my todo list to make some tests and, I hope, a pull request.
Regards.


3add testing framework

Tests can be run with npm run test:watch
@turnerniles I added a .travis.yml - we should consider setting up a continuous integration system

Definitely, this is great!!


Checksum Generator provide Output Type option

What improvement do you think would an existing feature or tool in DevToys?
When I upload a text file to the Checksum Generator, I might need to get the Base64 encoded text from the hash value.
It because the limitation of the "TextBox" treat newline as \r by default which is hard to Hash the right way using Hash Generator. So I have to save the text to a file and use Checksum Generator to do the hash.
Related issue here: https://github.com/veler/DevToys/issues/960
Solution/Idea
The Hash Generator have a Output Type to switch between Hex and the Base64. I think Checksum Generator should do the same thing.
Comments
No response

Hi,
This should be fixed in DevToys 2.0. We merged the Hash generator and Checksum generator into 1 unique tool, and DevToys 2.0 will support both \r and \n\r.


Option to ignore errors linked to genome

Is your feature request related to a problem? Please describe.
When we add dnaapler in a pipeline, it's hard to distinguish if dnaapler fails because some sequences have issues (e.g. 0 blast hit) or if there is a fatal error (unable to run the program). We don't want the whole process to fail because dnaapler is unable to make its magic on one or some contigs.
Describe the solution you'd like
While it might not be the recommended way for punctual use, it would be nice to have an option that ignores this kind of error, and the contigs affected would just return as they were in the input, so we can continue processing the data.

One of the simple changes could be to replace the current logger.error to logger.warning such as this one.

Hi @jsgounot ,
Agreed and this will be an easy request. I’m a bit short on software maintenance/improvement time right now but will get to it in a few weeks probably.
George

Coll thanks!

Seconding this request, would be fantastic! Thank you for this tool.

Hi @jsgounot ,
I've implemented a fix for this (looks like #74 had the same bug/request). So in the next version once released, if you have no BLAST hits and specified -a none, then dnaapler will behave as you wish.
George

Ok, I'll wait for the release then. Thanks.

available in v0.8.0


Remove dead reference to pressure penalty factor when defining strong form for verifying enthalpy-porosity model

Resolved as of 42ad052e9b804c16c4ea18a706534ae3c1d1baf7