Autoconfigure CF Admin Deployer

As a developer, I'd like to Autoconfigure CF Admin deployer, so I don't have to do component scanning to discover and register beans at each SPI level.

Has anybody tried to use this on CF? I'm getting the following error when I deploy the admin app:
2016-02-08T11:02:52.52-0500 [APP/0]      ERR Caused by: org.springframework.beans.factory.NoUniqueBeanDefinitionException: No qualifying bean of type [javax.sql.DataSource] is defined: expected single matching bean but found 2: mydb,dataSource

Also CloudFoundryModuleDeployerConfiguration now has spaces instead of tabs for indentation. It should also have copyright year of 2015-2016. Same for ContextLoadsTests.

This seems to be an issue with DataSourceAutoConfiguration not detecting that there is already a DataSource available. If you rename your explicit db from mysql to dataSource, then everything is fine.

before my bad merge. It had been working with binding a 'mysql' service to the admin, which also used DataSourceAutoConfig I would assume.  The new enable annotation should be impacting just how the SPI impl is discovered, but there is some odd side effect it would seem.

These seem to be side-effects when a config is moved into auto-configuration and things happen on a different order. https://github.com/spring-cloud/spring-cloud-dataflow/issues/366 is one of those ticking bombs which will blow up if i.e. EnableDataFlowServer is moved into auto-config or something else is kicking different order of config use.
Also DataSourceAutoConfiguration is a little brittle and I had to exclude its auto-config from gpfdist sink. One thing in DataSourceAutoConfiguration may be that it's checking beans of  DataSource and XADataSource and if explicitly created bean only implements one of those, I think boot does(maybe, possibly) it's own magic.

One of the problems with our current approach is that
@EnableAutoConfiguration (and hence its excludes) is set on a class that is
in the starter, and so can't be overridden by consumers
On Tue, Feb 9, 2016 at 3:17 PM, Janne Valkealahti<EMAIL_ADDRESS>wrote:

These seem to be side-effects when a config is moved into
auto-configuration and things happen on a different order.
spring-cloud/spring-cloud-dataflow#366
https://github.com/spring-cloud/spring-cloud-dataflow/issues/366 is one
of those ticking bombs which will blow up if i.e. EnableDataFlowServer is
moved into auto-config or something else is kicking different order of
config use.
Also DataSourceAutoConfiguration is a little brittle and I had to exclude
its auto-config from gpfdist sink. One thing in
DataSourceAutoConfiguration may be that it's checking beans of DataSource
and XADataSource and if explicitly created bean only implements one of
those, I think boot does(maybe, possibly) it's own magic.
—
Reply to this email directly or view it on GitHub
https://github.com/spring-cloud/spring-cloud-dataflow-admin-cloudfoundry/issues/32#issuecomment-181882488
.

@ericbottard which starter? Because that's really bad and may cause something I tracked down in https://github.com/spring-cloud/spring-cloud-dataflow/issues/367.

TBH, we have taken the most simple thing in Spring 1.0 - different apps inject a different impl of a strategy interface and made it complex.  That commit in the issue you point to is not an obvious solution.  Maybe we can have several @EnableDataFlow annotation, one for each SPI impl so that they can control what they need to themselves.

@EnableDataFlow is only enabling shared stuff from admin server(controllers, repos, etc) and afaik there's currently nothing spi specific.

What @jvalkeal said... also, let's avoid doing anything drastic at the @EnableDataFlowServer level until we have a better understanding of the boundaries of the generic app-deployer SPI, and specifically what, if anything, has to be configured in addition to that to support the Data Flow Server on each target platform.

I'm talking about AdminConfiguration, which is the one that bears @EnbleAutoConfiguration

lol, ok, 367 was about removing that @EnableAutoConfiguration. We're on a same page here ;)

So w/ the rant over, I think I understand the issue ATM, thanks Janne.   We prob don't even need this configuration class at all (cf auto-reconfiguration would inject the right stuff).  An @ConditionalOnMissingBean for the datasource here seems to work and also if there are tested configuration classes (not needed here, prob a cut-n-paste thing wrt to moving config out of a common admin) they should have @Conditional on them (as ilaya also found in the mesos admin).  I'll try out a few things and hope to get this resolved today.

Long term, want to get away from auto-reconfiguration anyway.

Fixed this by simply removing the explicit configuration and let auto-reconfig handle it for now.
2d7e051dca52a85fa89be36c54452343957cb6a8


Taruti/vendor update go logging

This has updates and is not racy + logger catches up with go-logging API changes.

lgtm :thumbsup:


Implement TOML ConfigStore in Node.js SDK

Implement TOML ConfigStore in Node.js SDK
Description

Plan a Config accommodation in the TOML format.
Add TOMLConfigStore class to the Node.js SDK stores.
Consider using the @iarna/toml library or similar for parsing and stringifying TOML during implementation.
We're open to other library suggestions or, if preferred, we can implement the APIs ourselves.

Motivation
To extend Configu's available stores collection and support the TOML format, which is a popular configuration format. This ConfigStore will be integrated into the Configu CLI as the Node.js SDK is utilized internally.
Additional context

TOML - Official GitHub Repository
Configu Config Store Documentation
ConfigStore TypeScript Definition
Configu Node.js SDK Stores

@cc @Sadaf-A

Hi @Dev-Arhaan, thanks for your interest in contributing! Currently, @Sadaf-A is working on this, and he has opened a PR for it. If you're still interested in helping, you might want to reach out to Sadaf and see if there are any opportunities for collaboration. Your enthusiasm is much appreciated!

Is anyone assigned to this task? If not I would like to work on it

@Dev-Arhaan Hey, it's great to see that you want to contribute. You can join our discord server and ping me. If you are up for it we can co-author the PR to get the job done faster.

Hi @Dev-Arhaan, Sadaf is currently unavailable to continue work on his PR. Let me know if you want to take it up and I'll assign you the issue.

Hi, can i tackle this issue?

Hey Richard, I Saw this very late, I'd love to work on it, let me hop on discord and try to understand the problem


Installation failed on Windows 10

Hi, I have an issue during installing  dependancies:
(Node version 9.5.0, npm 5.6.0)
npm ERR<EMAIL_ADDRESS>preinstall: node contrib/install-plugin-deps.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>preinstall script.
Could you please help me?

Try using node 6, we aren't supporting 9.5 yet.


[WIP][Utility] Implement servicer protocol E2E Relay w/ signature verification but w/o rewards

Objective
Implement an end-to-end (E2E) relay system for Servicers with signature verification but without rewards. This system should handle relay requests and responses, signature verification, and relay volume estimation.
Origin Document
V1 Utility Specification - Volume Estimation

Goals

Implement an E2E relay system for Servicers in the Pocket Network
Handle relay requests and responses between Applications and Servicers
Perform signature verification for relay requests and responses

Deliverable

[ ] Implement a relay system for Servicers that handles relay requests and responses
[ ] Implement signature verification for relay requests and responses

Non-goals / Non-deliverables

Implementing rewards for Servicers (handled in a separate ticket)

General issue deliverables

[ ] Update the appropriate CHANGELOG
[ ] Update any relevant READMEs (local and/or global)
[ ] Update any relevant global documentation & references
[ ] If applicable, update the source code tree explanation
[ ] If applicable, add or update a state, sequence or flowchart diagram using mermaid

Testing Methodology

[ ] All tests: make test_all
[ ] LocalNet: verify a LocalNet is still functioning correctly by following the instructions at docs/development/README.md

Creator: @jessicadaugherty

Closing this out in exchange for the process being put together in #733.


[Styles package]: refactor component styles for V2

The problem
Various component styles in the styles package have become very bloated and unwieldy. This is further exacerbated by components that have styles in both the styles and web-component packages.
The solution
A refactor of some component styles is overdue. It would also be a great opportunity to merge components which have styles in styles and web-components.
Application/website
Carbon for IBM.com
Business priority
Medium Priority = upcoming release but is not pressing
What time frame would this ideally be needed by (if applicable)
~Q1 2022
Examples
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

discussed in the Engineering Huddle.  Kenny to create additional issues.

Components to consider:

Card
Link with Icon
Link List

This will be in tandem with migrating to sass modules.

Tracked in https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/10373


Lambda index out of bounds when accumulating free energies

Hi,
I'm getting the following error when I use alchemical-analysis for the second part of my run (bonded, LJ and masses).
File "/Users/dvr/Work/alchemical-analysis-master/alchemical_analysis/alchemical_analysis.py", line 1226, in
main()
File "/Users/dvr/Work/alchemical-analysis-master/alchemical_analysis/alchemical_analysis.py", line 1212, in main
totalEnergies()
File "/Users/dvr/Work/alchemical-analysis-master/alchemical_analysis/alchemical_analysis.py", line 583, in totalEnergies
dF[name] += df_allk[k][name]
IndexError: index out of bounds
Everything works fine for the first and third part of the transformation with the same alchemical-analysis version and settings. Detailed output and input files can be found here:
https://www.dropbox.com/sh/vmjh7rmswbg0bd8/AADUpt8622PBUnxill6eOoIma?dl=0
Kind regards
Dries

Thanks. I'm guessing this is somewhat related to
https://github.com/MobleyLab/alchemical-analysis/issues/65 in that we've
probably not tested this with the same number of or distribution of lambda
components that you're using. Hopefully the fix will be simple, though as
the other issue notes there are more long-range planning issues to deal
with.
Sorry for the delay here. I'm swamped in the first part of the week so it
may take me til late this week to get a look at it.
On Sun, Jan 24, 2016 at 4:10 AM, Dries VR<EMAIL_ADDRESS>wrote:

Hi,
I'm getting the following error when I use alchemical-analysis for the
second part of my run (bonded, LJ and masses).
File
"/Users/dvr/Work/alchemical-analysis-master/alchemical_analysis/alchemical_analysis.py",
line 1226, in
main()
File
"/Users/dvr/Work/alchemical-analysis-master/alchemical_analysis/alchemical_analysis.py",
line 1212, in main
totalEnergies()
File
"/Users/dvr/Work/alchemical-analysis-master/alchemical_analysis/alchemical_analysis.py",
line 583, in totalEnergies
dF[name] += df_allk[k][name]
IndexError: index out of bounds
Everything works fine for the first and third part of the transformation
with the same alchemical-analysis version and settings. Detailed output and
input files can be found here:
https://www.dropbox.com/sh/vmjh7rmswbg0bd8/AADUpt8622PBUnxill6eOoIma?dl=0
Kind regards
Dries
—
Reply to this email directly or view it on GitHub
https://github.com/MobleyLab/alchemical-analysis/issues/70.

--
David Mobley
Associate Professor
Department of Pharmaceutical Sciences
Department of Chemistry
3134B Natural Sciences I
University of California, Irvine
Irvine, CA 92697
<EMAIL_ADDRESS>work<PHONE_NUMBER>
cell<PHONE_NUMBER>

Investigating this now. It could be it is also related to #64 .

This should be resolved by #72 . Can you test that it works for you?
You may also want to read #65 , as there is a larger enhancement we need to make to make the code report the correct labels for different lambda components in cases where you're not doing just a simple consecutive Coulomb-then-van-der-Waals transformation, as you aren't here. But at least now things should work.

Hi, just tested the patch and it looks like everything's working.
Thanks.
#65 looks like a very interesting enhancement, it would be nice to see the contribution of various terms.
As everything's working on my end, I'll go ahead and close this issue. I'd be happy to help if you require any further testing though.


More examples needed

need example of using fireEmit and fireBroadcast for modal. Not clear how to have function run when modal is opened/closed

Need example of how to use $show, $toggle, I kind of get $hide. This gets injected into the scope of the modal and will hide the modal. What good would $show or $toggle be if they are injected into the modal scope? The modal would already be shown if you have access to these. I would like to show/hide/toggle the modal from the parent scope, not from inside the modal. Maybe an example would help.
Also, it would be nice to have a way to know when the modal is closed (in the parent scope) and have a way to pass scope into the modal. Similar to how ui-bootstrap works with promises.
The promise that gets returned from the  $modal command resolves too soon. What good is that??
$scope.openModal = function (customTitle) {
var modalOptions = {
title: customTitle || 'Add Project',
contentTemplate: "/app/partials/modal/add-project.html",
theme: 'danger',
size: 'lg',
autoDestroy: true //we should destroy after close, because a new modal would be generated for each call
}
$modal(modalOptions).$promise
.then(function(){
//this returns before modal closes
});
}

Yes, each modal have own scope and you can acces it like below;
var myModal;
$scope.openModal = function (customTitle) {
   var modalOptions = {
   title: customTitle || 'Add Project',
   contentTemplate: "/app/partials/modal/add-project.html",
   theme: 'danger',
   size: 'lg',
   autoDestroy: true //we should destroy after close, because a new modal would be generated for each call
   }
   myModal = new $modal(modalOptions);
   var oldHide = myModal.hide;
    myModal.hide = function(){
      var promise = oldHide();
      promise.then(function(){
        ....you staff
      myModal = null;
      })
    }
}

//you can bind modal scope to you parent like below

$scope.myModalScope = .$hide();
}

//also you have modal factory named in your code now

$scope.closeMyModal = function(){
   myModal &&   myModal.hide();
    //or
   myModal &&  myModal.$scope.$hide();
}

OK Great!. All that works. The key was to use myModal=new $modal(modalOptions). The example on the web site doesn't show creating a new instance of $modal, so this was confusing. Thanks for clearing it up.
Also, is there anyway to specify a different controller for the modal? It seems like if we have a different scope we should be able to have a different controller as well. Otherwise, i'd like to inject the parent scope into the modal if that is possible.


Add <!DOCTYPE html> to put the page in Standards Mode

If it isn't included, browsers will use quirks mode, which emulates the behaviours of some older browsers instead of following the standards.

Thanks! Should I release a patch release for this?


feat: Display cumulative comments count

Currently we are showing the number of comments under each post. However, this is the number of immediate comments, and if there nested level 2+ comments they won't be included in this counter. However, it makes more sense to show cumulative count of all comments in the comments tree for each post. We can either calculate this number directly in UI or include counter into the smart contract logic.
Example
https://near.org/#/devgovgigs.near/widget/gigs-board.pages.Post?id=711
Current behaviour
It shows that there are only 3 comments
Expected behaviour
It should display 5 (3 immediate comments + 2 nested)
Suggested implementation

Introduce a new function get_posts_recursively next to get_post function on DevHub smart contract, so we don't need to fetch all the nested posts from UI one by one crashing the API server
Use that new function from DevHub components, and thus count the cumulative comments on the UI properly.

Tips
Here is the starting point on the UI side:
https://github.com/near/neardevhub-widgets/blob/b9c26222f672d605f2f171a8d50ff21a124bcab3/src/gigs-board/pages/Post.jsx#L55-L59

Hi, Would love to try it.

Hi @carina-akaia are you still working on this issue?

@Rishabhraghwendra18 hi, it's still in the backlog and I didn't start working on it yet. I can reassign it to you if you want to tackle this

@carina-akaia please assign it to me

@carina-akaia please assign it to me

@Rishabhraghwendra18 done!

@carina-akaia I need some help to set up neardev-contract repo. I have opened a decision regarding this: https://github.com/near/neardevhub-widgets/discussions/447
Thanks

Hi @Rishabhraghwendra18 – how is this ticket going? Are you still blocked?

Hello @ori-near, I am busy with my full-time job for next 2 weeks. Anyone can take it up, otherwise I will get back on this after 2 weeks.
Really sorry for inconvenience.

Hi @Rishabhraghwendra18 – hope all is well. I am putting this ticket back in the backlog. Let us know if you'd still like to pick this up. Otherwise, anyone else can grab it. Thanks.

The example nearmax mentioned is outdated so here is the new one: https://near.org/devhub.near/widget/app?page=post&id=711

Will close this issue, since proposals and announcements will fix many of the issues by design or will require rethinking of the approach


libwebp: fix MFSA-TMP-2023-0001

Description of changes
see https://www.mozilla.org/en-US/security/advisories/mfsa2023-13/#MFSA-TMP-2023-0001
no CVE id or other identifier has been assigned
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

no CVE id or other identifier has been published yet


Enchantment Shortcut

So i have tried repeatedly to change the shortcut to the GUI for my server. Everytime i try to change the GUI to False it just creates a new line of code below saying its true again. I have also tried to change this to On-Shift-Right-Click but the same issue persists, Is there any possible way around this or?

Fixed in 1.10r-4

Problem still persists

Try downloading from here https://github.com/dsdd/EnchantmentsPlusMinus/releases/tag/1.10r-4


Cannot populate @OneToMany when using Embeddables

Describe the bug
Hey there! First of all, thanks a lot for this very stable and intuitive TS ORM! ❤
Since a couple of versions, an @Embeddable can have a @ManyToOne relationship through its parent. This works fine.
However, the inverse side that would use a @OneToMany collection cannot populate the user.
Stack trace
No stack trace since no error.

To Reproduce
Here is an example setup:
@Embeddable()
export class Address {
  constructor(street: string, city: string, country: Country) {
    this.street = street
    this.city = city
    this.country = country
  }

  @Property()
  street!: string

  @Property()
  city!: string

  @ManyToOne()
  country!: Country
}

@Entity()
export class Country {
  @PrimaryKey({ autoincrement: true })
  id!: number

  @Property()
  name!: string

  @OneToMany(() => User, (user) => (user as any).address_country)
  users = new Collection<User>(this)
}

@Entity()
export class User {
  @PrimaryKey({ autoincrement: true })
  id!: number

  @Property()
  name!: string

  @Embedded(() => Address)
  address!: Address
}

You can populate the country property from the side of the User, but you cannot populate the users collection from the Country's side.
Expected behavior
One should be able to populate the users collection also from the Country's side.
Additional context
Here is a test suite that explains everything in detail:
import { MikroORM } from '@mikro-orm/core'
import { SqliteDriver } from '@mikro-orm/sqlite'
import { Address } from './address'
import { Country } from './country'
import { User } from './user'

describe('Issue', () => {
  let orm: MikroORM

  beforeAll(async () => {
    orm = await MikroORM.init({
      entities: [User, Country, Address],
      dbName: ':memory:',
      driver: SqliteDriver,
      allowGlobalContext: true,
    })
    await orm.schema.createSchema()
  })

  afterEach(async () => {
    await orm.schema.clearDatabase()
  })

  afterAll(() => orm.close(true))

  beforeEach(async () => {
    const germany = orm.em.create(Country, { name: 'Germany' })
    orm.em.create(User, { name: 'John Doe', address: new Address('Main Street', 'Berlin', germany) })
    await orm.em.flush()
  })

  test('the country can be populated through the address of a user', async () => {
    const user = await orm.em.findOneOrFail(User, { name: 'John Doe' }, { populate: ['address.country'] })
    expect(user.address.country.name).toBe('Germany')
  })

  test('the users can be populated through the address of a country', async () => {
    const country = await orm.em.findOneOrFail(Country, { name: 'Germany' }, { populate: ['users'] })
    expect(country.users[0].name).toBe('John Doe') // ❌ this one fails
  })

  test('can init the user collection of a country', async () => {
    const country = await orm.em.findOneOrFail(Country, { name: 'Germany' })
    await country.users.init()
    expect(country.users.isInitialized()).toBe(true)
    expect(country.users[0].name).toBe('John Doe')
  })
})

Versions

Dependency
Version

node
16.15.1

typescript
4.9.5

mikro-orm
5.6.13

sqlite3
5.1.4

This is not supported - you could tell by the need to cast things to any to be able to define this. You can't define inverse side of such relation because embedded entity does not have a primary key, hence it has no identity.
I'll keep this open as a feature request, as it could work for inlined mode. It is partially connected to #3887, the root cause is the same.

Thanks for you quick response! Yeah, that's what I figured. Wasn't sure whether to declare this issue as a feature request or a bug.


Add template-level parameters and file_name per template config

This PR adds two new optional fields in the templates section of a Weaver configuration file:

params: Defines a map of key-value pairs which will be merged with the ones defined at the top level of the weaver.yaml file. CLI parameters can also be used to override these parameters.
file_name: Defines the relative file path of the output generated from the template. This field is a Jinja expression which will be evaluated with the same context used in the template itself. The parameters mentioned above are also accessible.

An example of usage:
params:
  exclude_deprecated: true

templates:
  - pattern: attribute.py
    params:
      exclude_stability: ["stable"]
    filter: "semconv_grouped_attributes($params)"
    application_mode: each
    file_name: "incubating/{{ctx.root_namespace | snake_case}}.py"
  - pattern: attribute.py
    params:
      exclude_stability: ["experimental"]
    filter: "semconv_grouped_attributes($params)"
    application_mode: each
    file_name: "{{ctx.root_namespace | snake_case}}.py"

Note: params are now merged in an additive way. For example, if a parameter is defined in the params section of a template, then the final params section for this template will include both the params inherited from the file-level configuration and the params defined in the template configuration. If the override is a parameter set to null, then this parameter is removed from the final params section. This modification should not have a significant impact on the existing codegen, as I don't believe any codegen was relying on this specific behavior.
Closes open-telemetry/weaver#288.

I think we can do this in a non-breaking way by deprecating the "pattern" attribute and adding "template" where we pull from pattern if it exists, otherwise use template.

Can be done with a serde alias. I will also update the docs.

@jsuereth I renamed templates.pattern to templates.template and added pattern as a serde alias of template for backward-compatibility. I also updated documentations and examples.


enforce holy import trinity using goimports

This PR changes our gofmt check to
goimports -local github.com/sapcc/kubernikus.
This causes the tests to fail if imports are not ordered in the following way:
stdlib
[empty line]
external deps
[empty line]
kubernikus internal deps

I purposly excluded the generated files from this check to avoid confusion when we regenerate stuff.
I added y'all as reviewers because I want everyone to be aware of this change.

resolved merge conflicts


How to reset the timer on demand?

I tried returning the variable TimeoutDialog and calling keepAlive, setupDialog, etc. but it results to multiple timeout dialogs.
Basically, I have several pages in my application that makes ajax calls to the server and refresh the contents of the page. I would want to reset the timer on-demand whenever the ajax call is successfully complete. This is to maintain an active session with the web container.
Any ideas?

Hey, just stumbled across this, but before I ask a question, are you still around @MichaelApproved ?


SAK-31700 Log if wrong wrapper is used for quartz

When you wrap a job that you expect to be stageful in a non-stateful wrapper.

I haven't checked all the Sakai jobs, but locally we saw a job that we thought was stateful being run concurrently because of this.


chore: remove monaco-editor - already a peer

This removes monaco-editor because it's a peer dependency of react-monaco-editor and since we are on npm 8, the correct expected version of monaco-editor by react-monaco-editor will be installed automatically.
The will remove pitfalls like https://github.com/freeCodeCamp/freeCodeCamp/pull/44708#issuecomment-1007324841

Unfortunately there are a few problems with this.  The react-monaco-editor > 0.40 needs React 17 and we're stuck on 16 for now. Also, it's version 0.46 where they start using peer-deps, so there isn't a version we can use.
Finally, we need to make sure monaco-editor-webpack-plugin is compatible with the version of monaco-editor we install, that gets harder if we let react-monaco-editor choose.

This gets less messy once we can upgrade React, so I'll try to get to that sooner rather than later.

Finally, we need to make sure monaco-editor-webpack-plugin is compatible with the version of monaco-editor we install, that gets harder if we let react-monaco-editor choose.

For what it's worth, I'm pretty sure npm 8 "handles" this. If react-monaco-editor calls for 0.1.0 and monaco-editor-webpack-plugin calls for 0.2.0, npm should throw a "module resolution error" and fail.

@nhcarrigan yeah, that's true.  We'll still get errors, but that just means we'll have to handle this a bit more manually than other dependencies.


urlManager[showScriptName] can't hide index.php

i have set
urlManager' => [
'showScriptName' => false,
]
but
yii\helper\Url::to() => index.php//
and
ActiveForm default action /index.php//
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.0.28

Operating system
windows server 2016 +iis

http://www.yiiframework.com/forum/index.php/topic/69745-my-way-to-hide-indexphp-on-iis-7/page__p__296024__hl__ii++how+criptname#entry296024


Fixed Typo Getting Started Tutorial

Fixes issue #128

I was just going to do this similar fix. Looks good to me

Thanks so much for catching this @jojo1311 🎉


Updated link to vm drivers link to list names of drivers

This PR fixes https://github.com/istio/istio.github.io/issues/1806

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

Needs approval from an approver in each of these files:

content/OWNERS

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

Hi @brian-avery. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: brian-avery, geeknoid
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:

content/OWNERS [geeknoid]

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

Thanks @geeknoid


adds proj=True argument to plot_raw

closes #4190.  @Eric89GXL are you reviewing this one or should I ping someone else?

Changes look reasonable but Travis is not happy:
https://travis-ci.org/mne-tools/mne-python/jobs/223691431
I'll try it tomorrow. @jaeilepp feel free to have a look to, it changes some aesthetics

Codecov Report

Merging #4210 into master will decrease coverage by 0.01%.
The diff coverage is 90%.

@@            Coverage Diff             @@
##           master    #4210      +/-   ##
==========================================
- Coverage   86.18%   86.16%   -0.02%
==========================================
  Files         356      356
  Lines       64078    64083       +5
  Branches     9752     9752
==========================================
- Hits        55223    55220       -3
- Misses       6159     6164       +5
- Partials     2696     2699       +3

Works great over here, will merge once the CIs come back happy

Flake:
https://travis-ci.org/mne-tools/mne-python/jobs/224028374#L3162
You can test locally with make flake or make pep

arggh, was editing in a different editor than usual, the ruler was at 80 instead of 79.  Should be fixed now.

Thanks @drammock


purge: updated flow to allow the user to opt out

Playbook performs some initial local checks, then checks
each ceph host has cephadm and the right fsid. Once
these plays are passed, the user is prompted to purge or
not, so they may abort
Closes #21, closes #22, closes #23
Signed-off-by: Paul Cuzner<EMAIL_ADDRESS>

jenkins test all_daemons

jenkins test deploy_and_purge

jenkins test deploy_and_purge

jenkins test deploy_and_purge

@pcuzner could you squash the two first commits please?
thanks!

@pcuzner could you squash the two first commits please?
thanks!

squashed - so you have one commit handling the playbook changes and another tox - hope that's what you meant

@pcuzner could you squash the two first commits please?
thanks!

squashed - so you have one commit handling the playbook changes and another tox - hope that's what you meant

yup :slightly_smiling_face:


Can you provide the code for training the diffusion model

Dear authors,
Thanks for sharing the code! I did not find the code to train the diffusion model. Could you upload those code? Thanks!
Sincerely

Hello @SliencerX!
The code is already available in this repository (in the src folder).
Please refer to the Launch a training run section of the README for instructions on how to launch a training run.
Also, you can find the diffusion-specific code in src/models/diffusion.
Hope this helps!

Hi,
Thanks for the quick replay!. I noticed that in the paper you also implement a DDPM based diffusion. Could you also provide the code for the DDPM version of the diffusion model?
Sincerely


Move kong build tools version to .requirements file

Summary
Move kong build tools version to .requirements file.
Since more tools depend on kong build tools being pinned somewhere, it's easier to grep on the .requirements file than hardcoded on  the Makefile
Full changelog

Move kong build tools version as KONG_BUILD_TOOLS_VERSION in .requirements file

@eskerda Since the RC1 build for 2.1.0 is out and we want to avoid making changes to the release branch at this point if possible. Should we consider merge this onto next instead?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 6 committers have signed the CLA.:white_check_mark: javierguerragiraldez:white_check_mark: locao:white_check_mark: eskerda:x: hutchic:x: dndx:x: bungleYou have signed the CLA already but the status is still pending? Let us recheck it.

@eskerda, could you please rebase this to next, and I'll merge it then.

@bungle rebased!

Rebased again...


cc.RenderTexture.saveToFile runs async and no callback

RenderTexture.setPosition(cc.p(winSize.width / 2, winSize.height / 2));
RenderTexture.beginWithClear(255, 255, 255, 0.0);
cc.director.getRunningScene().visit();
RenderTexture.end();
var imgPath = jsb.fileUtils.getWritablePath();
var result = RenderTexture.saveToFile(fileName, cc.IMAGE_FORMAT_PNG);
if (result) {
	imgPath += fileName;
		open_img(imgPath); // open_img a function to open file, but it say file not found!!!
	}

please add finish callback for it.

Hi.
I have added a timeout for this feature but it is not async so far.

which commit and how to use?

hello, any update for this bug?


api: Add a Negative test clCloneKernel (#747)

Add a negative test for clCloneKernel that expects the CL_INVALID_VALUE
return code.
Other improvements:

Remove unused code or at least cast to void.
Change NULL to nullptr which includes type information.

Correction: these test_ functions do not correspond to test entry points and my comment above doesn't apply.

Thanks for the feedback. We will reconsider how best to address this case.


An error occurred (AccessDenied) when calling the AssumeRole operation: MultiFactorAuthentication failed with invalid MFA one time pass code.

We have Setup Our Personal Repository CD Pipeline with Github Actions. From last year it were Running Well. In the Last Week, For Build the Pipeline Prompt mfa step security echo mfa same . When Enter mfa shows Successfully set secrets!
Successfully cleared secrets.  Then After 5 minute it shows the title error.

@HADEED-143 can you please share more details, e.g. can you share the workflow? This seems to me like a config issue with AWS, because the wait-for-secrets action just makes a secret available in the workflow.
Feel free to email<EMAIL_ADDRESS>if you prefer that.

Hey Varun, We are Running this workflow from last year there was no issue
and no change. From the Last Week on All Repository This Issue Raise. We
cant do any deployment because of it take credential and echo same
credentials stuck for a some minute and then fail 252 exit code. My
Question is this ! is this generic issue ? other users also have raise this
issue or just it is from us ? .
Thanks & Regards
Hadeed.
On Thu, Jul 18, 2024 at 11:12 AM Varun Sharma @.***>
wrote:

@HADEED-143 https://github.com/HADEED-143 can you please share more
details, e.g. can you share the workflow? This seems to me like a config
issue with AWS, because the wait-for-secrets action just makes a secret
available in the workflow.
Feel free to email @.*** if you prefer that.
—
Reply to this email directly, view it on GitHub
https://github.com/step-security/wait-for-secrets/issues/216#issuecomment-2235702649,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLY6QBGPZXW7H773VZWF2DZM5MDJAVCNFSM6AAAAABK36ZVE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZVG4YDENRUHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Now I am using the simple input mfa parameter for the workflow. For the
time being, i have removed all the securitystop check and commands in the
workflow.
The Sample File is attached below you can check for reference.
On Thu, Jul 18, 2024 at 8:02 PM Varun Sharma @.***>
wrote:

No one else has raised it.
I can see a recent successful run on a public repo as well:
https://github.com/ghiscoding/slickgrid-react/actions/runs/9922211414/job/27411023267#step:11:13
I recommend running a simple workflow to make sure you are getting the
secret. If so, then looking at the error you are getting in the release
workflow and investigating the root cause - this looks like a config issue
with AWS.
—
Reply to this email directly, view it on GitHub
https://github.com/step-security/wait-for-secrets/issues/216#issuecomment-2236805407,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLY6QBODTVY3RX6EMJ62OTZM7KJDAVCNFSM6AAAAABK36ZVE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZWHAYDKNBQG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

I dont see the attachment, can you please attach it?

Please check now. securitycheck.yaml name file is attached also with the
last email.
On Fri, Jul 19, 2024 at 10:15 AM Varun Sharma @.***>
wrote:

I dont see the attachment, can you please attach it?
—
Reply to this email directly, view it on GitHub
https://github.com/step-security/wait-for-secrets/issues/216#issuecomment-2238174200,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLY6QHQEYBCYZJWIYKTPCDZNCOG7AVCNFSM6AAAAABK36ZVE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZYGE3TIMRQGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix/metrics

Summary | Résumé

Fixed Exposed metric not being published at the right place
Fixed OtkEntered metric being published every time we upload new TEKs

Looks good - I'll let @smcmurtry give the thumbs up or down


Crashes on loading main window in folder

I'm not sure if this is the right place to ask this, but when I launch the program, it always seems to crash. Anyone have any idea what I'm doing wrong?

Boostrapper.cs
using Stylet;
using Tester.ViewModel;
namespace Tester
{
    public class Boostrapper: Bootstrapper<MainViewModel>
    {
    }
}

MainView.xaml
<Window x:Class="Tester.Views.MainView"
       <!--cut content, standard xmlns -->
        Title="MainView" Height="450" Width="800">
    <Grid>
    </Grid>
</Window>

MainViewModel.xaml
using Stylet;
namespace Tester.ViewModel
{
    public class MainViewModel: Screen
    {
        private readonly WindowManager _windowManager;

        public MainViewModel(IWindowManager windowManager)
        {
            _windowManager = (WindowManager)windowManager;
        }
    }
}

What exception does it throw?
Also, why cast the IWindowManager to WindowManager? Not that it should be causing a crash...
On 15 April 2020 06:09:04 BST, micah686<EMAIL_ADDRESS>wrote:

I'm not sure if this is the right place to ask this, but when I launch
the program, it always seems to crash. Anyone have any idea what I'm
doing wrong?

Boostrapper.cs
using Stylet;
using Tester.ViewModel;
namespace Tester
{
   public class Boostrapper: Bootstrapper<MainViewModel>
   {
   }
}

MainView.xaml
<Window x:Class="Tester.Views.MainView"
      <!--cut content, standard xmlns -->
       Title="MainView" Height="450" Width="800">
   <Grid>
   </Grid>
</Window>

MainViewModel.xaml
using Stylet;
namespace Tester.ViewModel
{
   public class MainViewModel: Screen
   {
       private readonly WindowManager _windowManager;

       public MainViewModel(IWindowManager windowManager)
       {
           _windowManager = (WindowManager)windowManager;
       }
   }
}

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/canton7/Stylet/issues/118

I get a Stylet.StyletViewLocationException: 'Unable to find a View with type Tester.View.MainView'
And I was casting the window manager because I thought that was how I needed to do it, or should it just be IWindowManager?
I've attached the project as well.
StyletTester.zip

Aha. As the message says, it's looking for Tester.View.MainView, but your MainView is called Tester.Views.MainView.
You can change the namespace that MainView is in, or you can tell the ViewManager to look in Tester.Views, see NamespaceTransformations here.
Yes, the idea is that you use IViewManager. Then you can inject a mock ViewManager in your unit tests.

So, a few questions.
1)I would have to keep both the View and ViewModel folders singular or plural, not one of each, right?
2) Do I have to keep all of the Windows/UserControls in the View(s) root folder, or can I have them in a Views/Dialogs/ folder, for instance? I was having crashing trouble when I was testing additional windows out before, though I think they could just be this main issue.

Unless you set up your own name mappings, yes
You can have sub folders. The algorithm is described here. So the View for ViewModels/Dialogs/FooViewModel.cs goes at View/Dialogs/FooView.xaml. Likewise Dialogs/ViewModels/Foo.cs and Dialogs/Views/Foo.xaml, or Dialogs/FooViewModel.cs and Dialogs/FooView.xaml. Simply take the name and namespace of the ViewModel and replace "ViewModel" with "View" to get the name and namesoace of the View.

So, all of the Views (Window, UserControl, Page,...) xaml files must end in ...View.xaml, and all of the associated Viewmodels must end in ...ViewModel.cs? Is that correct?

Yes, but this is configurable, see my first link. I strongly recommend reading the whole of that documentation page, as it answers many of the questions you've asked.
There's typically no need to use Page - navigation is done using other mechanisms in Stylet, see the docs.
On 16 April 2020 06:24:37 BST, micah686<EMAIL_ADDRESS>wrote:

So, all of the Views (Window, UserControl, Page,...) xaml files must
end in ...View.xaml, and all of the associated Viewmodels must end in
...ViewModel.cs? Is that correct?
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/canton7/Stylet/issues/118#issuecomment-614422671


Getting error while deploying application from VS code to Azure while selecting code folder

Repro steps:

Action: appService.Deploy
Error type: object
Error Message: {}
Version: 0.15.0
OS: win32
Product: Visual Studio Code
Product Version: 1.28.1
Language: en

Please provide repro steps.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Change suffix of upgrade steps functions from plan to step

removed unnecessary parallel parameter.

It was asked to land those changes on main. The #235 was opened to this.


KeyError: 'DisplayName' during GCP service account import

I see following errors in /var/log/securitymonkey.log for GCP iamserviceaccounts
2017-04-25 20:43:38,813 INFO: Started thread pool with 25 core threads and 30 maximum threads [in build/bdist.linux-x86_64/egg/apscheduler/threadpool.py:52]
Traceback (most recent call last):
File "/usr/local/src/security_monkey/venv/bin/monkey", line 11, in <module>
load_entry_point('security-monkey', 'console_scripts', 'monkey')()
File "/usr/local/src/security_monkey/security_monkey/manage.py", line 625, in main
manager.run()
File "/usr/local/src/security_monkey/venv/lib/python2.7/site-packages/Flask_Script-0.6.3-py2.7.egg/flask_script/init.py", line 397, in run
result = self.handle(sys.argv[0], sys.argv[1:])
File "/usr/local/src/security_monkey/venv/lib/python2.7/site-packages/Flask_Script-0.6.3-py2.7.egg/flask_script/init.py", line 376, in handle
return handle(app, positional_args, **kwargs)
File "/usr/local/src/security_monkey/venv/lib/python2.7/site-packages/Flask_Script-0.6.3-py2.7.egg/flask_script/commands.py", line 145, in handle
return self.run(args, kwargs)
File "/usr/local/src/security_monkey/security_monkey/manage.py", line 77, in find_changes
sm_find_changes(account_names, monitor_names)
File "/usr/local/src/security_monkey/security_monkey/scheduler.py", line 52, in find_changes
(items, exception_map) = cw.slurp()
File "/usr/local/src/security_monkey/security_monkey/watchers/gcp/iam/serviceaccount.py", line 84, in slurp
return slurp_items()
File "/usr/local/src/security_monkey/venv/lib/python2.7/site-packages/cloudaux-1.1.8-py2.7.egg/cloudaux/gcp/decorators.py", line 132, in decorated_function
itm, exc = func(*args, kwargs)
File "/usr/local/src/security_monkey/security_monkey/watchers/gcp/iam/serviceaccount.py", line 75, in slurp_items
name=sa['DisplayName'],
KeyError: 'DisplayName'

I think the issue is that some of the service accounts do not have displace name, hence name field needs to be parsed and used.

@supertom ^^ issue is the one we discussed on gitter

Fixed in #701


Parameters are not properly deserialized when calling update API of StorageAccountsOperations class

StorageAccountCreateParameters Class is being used while deserializing though StorageAccountUpdateParameters class should have been used.
Code -
azure_storage_client = StorageManagementClient(credentials, subscription_id)
parameters = {'enable_https_traffic_only': True}
azure_storage_client.storage_accounts.update(resource_group_name, account_name, parameters)
Error -
File "/Users/amit.a/PycharmProjects/qa-auto/venv/lib/python2.7/site-packages/azure/mgmt/storage/v2017_10_01/operations/storage_accounts_operations.py", line 159, in create
body_content = self._serialize.body(parameters, 'StorageAccountCreateParameters')
File "/Users/amit.a/PycharmProjects/qa-auto/venv/lib/python2.7/site-packages/msrest/serialization.py", line 475, in body
raise errors[0]
ValidationError: Parameter 'StorageAccountCreateParameters.sku' can not be None.

was using wrong API by mistake, was actually using create instead of update


jenkins: Failed to prepare loop device: device or resource busy

+ /lxc-ci/build/tmp.I8h5uKv6TO/go/bin/lxc storage create lxdtest-rm8-pool-btrfs lvm volume.block.filesystem=btrfs volume.size=200MB --verbose
INFO[11-05|22:48:24] Creating LVM storage pool "lxdtest-rm8-pool-btrfs"
Error: Failed to prepare loop device: device or resource busy

https://jenkins.linuxcontainers.org/job/lxd-github-commit/2955/arch=amd64,backend=lvm,compiler=golang-1.10,restrict=lxd-priv/console
https://jenkins.linuxcontainers.org/job/lxd-github-pull-test/5215/arch=i386,backend=lvm,compiler=golang-1.11,restrict=lxd-priv/console

It's kinda unclear what call is getting us an EBUSY but I think it'd be worth going through our cgo code there and see what calls may get us EBUSY and add a retry loop if that makes sense (either in cgo or in go).

This can only happen when
ioctl(fd_loop, LOOP_SET_FD, fd_img);

fails. Which means that the loop device we detected in get_unused_loop_dev() has been stolen in between detecting it and setting it up. That seems especially plausible since
+ /lxc-ci/build/tmp.WZlKeADwLc/go/bin/lxc storage create lxdtest-jKSaH2H3J-pool13 lvm source=/dev/loop12 lvm.vg_name=lxdtest-jKSaH2H3J-pool13-dummy_vg_4 volume.size=25MB --verbose

references a specific loop device. I'm not sure there's a need for a loop around this which would just stall. But I can send a POC pr.


4.0.rc1 Object.new() error

Godot version
v4.0.rc1.official.8843d9ad3
System information
Windows 10, Forward+, NVIDIA GeForce GTX 860M
Issue description
cannot use Object.new()

Steps to reproduce
NA
Minimal reproduction project
NA

This was supposed to be fixed by #41516 but indeed I still see this error.

Hi, as this is still a bug, what would be a good workaround for now? would Node.new() be suitable? just because I am in the middle of converting my project to 4.0 and have got stuck on this bug! many thanks.

workaround: use a subclass which extends Object, like RefCounted, or an empty class which extends Object:
MyObject.gd:
extends Object
class_name MyObject

it depends whether you should use Node:
https://docs.godotengine.org/en/latest/tutorials/best_practices/node_alternatives.html

Sweet that works for me, thanks!
On Thu, 2 Mar 2023, 08:34 jinyangcruise, @.***> wrote:

workaround: use a subclass which extends Object, like RefCounted, or an
empty class which extends Object:
MyObject.gd:
extends Object
class_name MyObject
it depends whether you should use Node:
https://docs.godotengine.org/en/latest/tutorials/best_practices/node_alternatives.html
—
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/73213#issuecomment-1451487624,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJJI52O3N55PUMLA2JXVF7DW2BLR5ANCNFSM6AAAAAAU2DKFPM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I' m not sure what is the correct fix here. We can fix it here by not adding the "Object" as a builtin type:
https://github.com/godotengine/godot/blob/db5113de36aa59b3b3ffcd3399d06f4b76476df8/modules/gdscript/gdscript_parser.cpp#L52-L57
But then I saw that this code uses the above function:
https://github.com/godotengine/godot/blob/db5113de36aa59b3b3ffcd3399d06f4b76476df8/modules/gdscript/gdscript_analyzer.cpp#L2779-L2782
Do we want to use Object() as a constructor which I don't think works correct anyway, it returns an object that prints <Object#null>. I can remove that code too.
We can also fix it in other ways but it seems more clean this way.

@dalexeev's PR didn't make it to 4.1's release window, but it will certainly be cherrypicked for 4.1.x versions.


No security-lib-ci yaml for reference

[Enter feedback here]
Where is the security-lib-ci yaml?

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

ID: 86285f72-9e28-da97-59bb-c29eb60f627d
Version Independent ID: 18d5a591-a7d3-c261-6bff-8808ae433f54
Content: Configure pipeline triggers - Azure Pipelines
Content Source: docs/pipelines/process/pipeline-triggers.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @steved0x
Microsoft Alias: sdanie

I found it. And I am closing this Issue.


Provide build tool support

Packaging will solve that for the users of the package. Some naive support for daily chores during development is present (Makefile for cleaning), git ignore for filtering, and setup tools interface for packaging.