fixes dependency hence electron-prebuilt has been renamed to electron

fixes npm complaining about the following issue. same electron version has been preserved.
npm WARN deprecated<EMAIL_ADDRESS>electron-prebuilt has been renamed to electron. For more details, see http://electron.atom.io/blog/2016/08/16/npm-install-electron

LGTM. Thanks 👍


Can't compile from_json(std::valarray).

Following code couldn't be compiled because the from_json implementation uses the private member 'm_value'.
json Version: v3.7.0
#include <nlohmann/json.hpp>

using json = nlohmann::json;

int main()
{
   json j = json::array({1, 0, 2});
   j.get<std::valarray<int>>();
}

https://wandbox.org/permlink/ifYduziYXUtfgP4n

Indeed. Changing
std::copy(j.m_value.array->begin(), j.m_value.array->end(), std::begin(l));

to
std::copy(j.begin(), j.end(), std::begin(l));
``

fixes the issue.

🔖 Release item
This issue/PR will be part of the next release of the library. This template helps preparing the release notes.
Type

[ ] ✨ New Feature
[x] 🐛 Bug Fix
[ ] ⚡ Improvement
[ ] 🔨 Further Change
[ ] 🔥 Deprecated function

Description

fix conversion to std::valarray


Development 0.2

Merge in developments (fixes really) from 0.2 branch into master branch

Commit into master


Secure a Basic Node.js App with the Authorization and Trust Management Service (XSUAA)

Tutorials: https://developers.sap.com/tutorials/cp-cf-security-xsuaa-create.html
Write here how you think we can improve the tutorial ...
Under section 3: "Prepare the application security descriptor". Following code snippet is given for xs-security.json...
"oauth2-configuration": [
{
"redirect-uris": ["https://approuter-product-list-ap25.cfapps.eu10.hana.ondemand.com/login/callback"]
}
]
Above code is not correct..  Correct one is the following--- you need to remove the above high lighted [ and ].
"oauth2-configuration": {
"redirect-uris": [
"https://approuter-product-list-ap25.cfapps.eu10.hana.ondemand.com/login/callback"
]
}

@binsonvarikkasserilabraham : Thanks for pointing this out. We will update this example as soon as possible.


const infer no more available

I used const infer a lot, and now this options is no more available.
Do you plan on adding it again ?

I'll add it back in a release in about 8 hours? I didn't think anyone used it. Thank you for letting me know otherwise!

@ecyrbe This should not be deployed on www.jsonschema.net


 Are you thinking about this? There's a reason to not parse JS?
Reproduce it:
console.log(require('circular-json').stringify({ f: function(){ return 1; } } ));

Please read JSON specs and realize functions are not compatible.


Add kafka topic sync offset lag metrics by JMX

When we use mirror maker , we very concerned about sync offset lags.We wonder Progress of all data synchronization,  I  definition this sync lag.
In other words ,the lag  is the topc partition offset delay between soruce cluster and target clusterI use JMX to export metrics to show  the lag.In MirrorSourceTask realtime to calc lag.
Calc the sync lag, the lag means topic partition latest offset  minus the current sourceTask sync topic offset.
lag = latest offset in source cluster - current sync offset in source cluster

This is being done by KIP-971, so closing this PR.


Can I use this with react-router v4?

I'd like to be able to do something like this:
        <Route render={({ location }) => (

                    <ReactCSSTransitionReplace
                        transitionName="cross-fade"
                        transitionEnterTimeout={1000}
                        transitionLeaveTimeout={1000}
                    >
                        <Route key={location.key + "one"} exact path="/" component={HomePage}/>
                        <Route key={location.key + "two"}  path="/two" component={SomeOtherComponent}/>
                        <Route key={location.key + "three"}  path="/three" component={YetAnotherComponent}/>
                    </ReactCSSTransitionReplace>

        )}/>

However, I can't because RCTReplace expects an only child.  Is there any sort of workaround I can do?

BTW - the other Routes render null in this specific case (there will be one active, and two nulls, always), however in the general case other code could set up routes that have multiple tags rendering.

I think using the React Router <Switch> might be the answer. I just browsed through their docs and found this, I haven't actually tested it so I'd be interested to know whether it works for you. In your case it would mean:
<Route render={({ location }) => (

    <ReactCSSTransitionReplace {...}>
      <Switch>
        <Route key={location.key + "one"} exact path="/" component={HomePage}/>
        <Route key={location.key + "two"}  path="/two" component={SomeOtherComponent}/>
        <Route key={location.key + "three"}  path="/three" component={YetAnotherComponent}/>
      </Switch>
    </ReactCSSTransitionReplace>

)}/>

If that doesn't work RCTReplace might be modified to strip out nulls from its children before requiring a single remaining child. This might be a neat addition to the component either way, but if the above solution works that would be great.

I'm trying to do the same thing, but your proposed solution does not work, as RCTR detects the Switch as its child, not the component the route renders. I also tried wrapping the routes and also the components, but none of them work.
It seems like what we would have to do is:
<Switch>
  <ReactCSSTransitionReplace {...}>
    <Route {...}/>
    <Route {...}/>
  </ReactCSSTransitionReplace>
</Switch>

But this doesn't work either, since 1. ReactCSSTransitionReplace can only have one child and 2. Switch expects the immediate children to be routes (or at least have the props the route should have).
The only solution that I see at the moment, which doesn't involve changing both libraries, would be to add a special case for when the child is a Switch.

I have the same question with RR4 React Router Config:
import React from 'react';
import { renderRoutes } from 'react-router-config';
import ReactCSSTransitionReplace from 'react-css-transition-replace';
import { shape } from 'prop-types';
const Base = ({route, location}, {router}) => (

      {renderRoutes(route.routes)}

);
Base.propTypes = {
location: shape({}),
};
export default Base;

@marnusw your example is almost working. You need to pass the location to the Switch and set a key on the switch that changes when the location changes (e.g. location.pathname).
However, there are problems when no route matches, as Switch will render null, but the child of ReactCSSTransitionReplace is not null:
Uncaught TypeError: Cannot read property 'offsetHeight' of null
    at ReactCSSTransitionReplace.componentWillReceiveProps (ReactCSSTransitionReplace.js:167)
...

So I guess to handle this, you would need to check if the child renders something instead of just checking if there is a child.

Thanks @phyrog! If will be good if that works and then it should definitely be added to the docs. I'll see if I can address that error over the weekend, and if someone beats me to it with a PR that will also work.

Hello. Your examples not work with react-router-config and subchilds routes.
I think what trouble in Switch render.
const routes = [
	{
		component: LandingLayout,
		path: '/',
		exact: true,
		routes: [
			{
				path: '/',
				exact: true,
				name: 'Landing',
				component: Landing,
			},
		]
	},
	{
		component: AuthLayout,
		routes: [
			{
				path: '/login',
				exact: true,
				key: 'Login',
				component: Login,
			},
			{
				path: '/recover',
				key: 'Recover',
				component: Recover,
				routes: [
					{
						path: '/recover/:code',
						name: 'RecoverProcess',
						component: RecoverProcess
					}
				]
			},
		]
	},
]

In AuthLayout ReactCSSTransitionReplace not work((
<ReactCSSTransitionReplace
						component="div"
						transitionName="cross-fade"
						transitionEnterTimeout={1000}
						transitionLeaveTimeout={1000}>
						{renderRoutes(route.routes)}
					</ReactCSSTransitionReplace>

renderRoutes method render Switch without keys. How yout think its possible fix it?

Version 3 has been published and the docs include a React-Router 4 example.


Remove Meta class from Sector

Issue number: N/A
Description of change
It was wrong for multiple reasons:

the value of the ordering attribute was wrong, though this was not noticed as MPTT ordered query sets correctly anyway
it inherited from BaseConstantModel.Meta when the class didn't inherit from BaseConstantModel

This removes the Meta class completely as it's not required here for anything.
Checklist

[x] Have any relevant search models been updated?
[x] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[x] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[x] Has the admin site been updated (for new models, fields etc.)?
[x] Has the README been updated (if needed)?

Codecov Report

Merging #882 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #882   +/-   ##
========================================
  Coverage    96.68%   96.68%
========================================
  Files          162      162
  Lines         4773     4773
  Branches       460      460
========================================
  Hits          4615     4615
  Misses          98       98
  Partials        60       60

Continue to review full report at Codecov.

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


fatal exit without defer

/cmd\dicomutil\main.go line 67
Code:
if err != nil {
log.Fatalf("error parsing data: %v", err) // <-- will not use defer
}

I think you are pointing out that log.Fatalf exits the programs without running the defered functions? Thanks for pointing it out, I'll make some updates in #278 (probably dicomutil in general could use a refactor to be honest).

Addressed in dicomutil in #278


Suggestion: npm compatibility - add types to the generated package.json

Hi, i would like to report a suggestion to improve the generated package.json when using the compatibility layer.
When doing:
npx jsr add @std/uuid

I get the following package.json:

{
  "name": "@jsr/std__uuid",
  "version": "0.213.1",
  "homepage": "https://jsr.io/@std/uuid",
  "type": "module",
  "dependencies": {
    "@jsr/std__assert": "^0.213.1",
    "@jsr/std__bytes": "^0.213.1",
    "@jsr/std__crypto": "^0.213.1"
  },
  "exports": {
    ".": "./mod.js",
    "./v1": "./v1.js",
    "./v3": "./v3.js",
    "./v4": "./v4.js",
    "./v5": "./v5.js",
    "./constants": "./constants.js"
  },
}

We can see that types are not exported, thus, it result in a typescript error:

Howerver, by modifiyng a little bit the package.json it's easy to fix this error:
{
  "name": "@jsr/std__uuid",
  "version": "0.213.1",
  "homepage": "https://jsr.io/@std/uuid",
  "type": "module",
  "dependencies": {
    "@jsr/std__assert": "^0.213.1",
    "@jsr/std__bytes": "^0.213.1",
    "@jsr/std__crypto": "^0.213.1"
  },
  "exports": {
    ".": {
      "default": "./mod.js",
+      "types": "./mod.d.ts"
    },
    "./v1":{
      "default": "./v1.js",
+      "types": "./v1.d.ts"
    },
    "./v3": {
      "default": "./v3.js",
+      "types": "./v3.d.ts"
    },
    "./v4": {
      "default": "./v4.js",
+      "types": "./v4.d.ts"
    },
    "./v5": {
      "default": "./v5.js",
+      "types": "./v5.d.ts"
    },
    "./constants": "./constants.js"
  },
+  "typings": "./mod.d.ts",
+  "types": ["./mod.d.ts"]
}

So I would suggest to modify the generated package.json to include types, thus providing a better user experience for the devs using the compatibility layer.
While i got you, thanks for the amazing job 🎉, jsr is a true game changer !
Environnement
Webstorm 2024.1.5
Vite project without deno

You seem to have an old revision of the tarball. I suggest delete your lock file (package-lock.json / yarn.lock / pnpm.lock) and re-install. This will pull down a new package tarball revision where this is fixed.

You are right ! Thanks for the help. Indeed I got an old version instead of the new one :+1:


chore: add gitpod support

This PR adds Gitpod support with a .gitpod.yml file (which automates the dev environment) and updates the README.md.

Monaco editor (editor used in VSC) inside VSC 😆  Thanks for contribution!


Analytics.reset() should map to logout functions of integrations

Should delete the cookies so you can alias without fear!

@hankim813 What about the satus of this issue?
PS:
Mixpanel now has mixpanel.reset(); function.
Currently this is what I'm using on user logout.
analytics.reset();
mixpanel.reset();

cc @f2prateek

Tracking internally in https://segment.atlassian.net/browse/LIB-101


Language specific setup

I want to use cmp_tabnine as a source in only specific filetypes, along with my regular nvim-cmp setup. This is my current solution.
require('cmp').setup({
  ...
})

cmd [[autocmd FileType markdown,text,tex,gitcommit
\ lua require('cmp').setup.buffer {
\   sources = {{ name = 'cmp_tabnine' }},
\ }
]]

What I noticed however is that this overrides my original setup in those filetypes, and so I would have to duplicate it and add the whole config again inside the autocmd. Is there a nicer way to add language specific configurations?

local vimrc = {}

vimrc.utils = {}
vimrc.utils.concat = function(tbl1, tbl2)
  local tbl3 = {}
  for _, item in ipairs(tbl1) do
    table.insert(tbl3, item)
  end
  for _, item in ipairs(tbl2) do
    table.insert(tbl3, item)
  end
  return tbl3
end

vimrc.cmp = {}
vimrc.cmp.sources = {
  { name = 'nvim_lsp' },
  ...
}

local cmp = require('cmp')
cmp.setup {
  sources = vimrc.cmp.sources
}

vim.cmd([[
  autocmd FileType markdown,text,tex,gitcommit lua require('cmp').setup.buffer {
  \   sources = vimrc.utils.concat(vimrc.cmp.sources, { name = 'cmp_tabnine' }),
  \ }
]])

just write Lua.

Ok thank you for your answer. So there's no plan for adding a filetypes field to the tables provided to sources? For example:
cmp.setup({
  ...
  sources = cmp.config.sources({
    ...
    {
      name = 'cmp_tabnine',
      filetypes = { 'markdown', 'text', 'tex', 'gitcommit' }
    }
  })
})

Yes. I don't want to add it. sorry.
It seems that there are users with many different opinions :)
https://github.com/hrsh7th/nvim-compe/issues/174

Ok, I see :)


Render multiple models

I have an array containing multiple model instances.
[
   Task.new,
   Contact.new
]

Is it possible to do something like this:
{
  super_search: [
   {
     name: 'Some name',
     description: 'Blah',
     _type: 'Task'
   },
   {
     full_name: 'Some contact name',
     avatar_url: 'alkjsdlfjadsf.png',
     _type: 'Contact'
   },
  ]
}

from two different model types?

There's not a way to specify multiple blueprints for a given list as far as I know, but if you want to achieve that you may be able to using a single blueprint with conditional fields:
require 'blueprinter'
require 'json'

class Task
  def name
    "Name"
  end

  def description
    "Description"
  end
end

class Contact
  def full_name
    "Full Name"
  end

  def avatar_url
    "Avatar Url"
  end
end

class MyBlueprint < Blueprinter::Base
  field :name, if: ->(_field_name, record, options) { record.class == Task }
  field :description, if: ->(_field_name, record, options) { record.class == Task }
  field :full_name, if: ->(_field_name, record, options) { record.class == Contact }
  field :avatar_url, if: ->(_field_name, record, options) { record.class == Contact }
  field :_type do |record|
    record.class.to_s
  end
end

p MyBlueprint.render([Task.new, Contact.new])

Thanks for the detailed response!
I may just end up handling this thus, with an index_by and the type of model, so I can reuse my other blueprints:
SearchBlueprint < ApplicationController
  association: :task, blueprinter: TaskBlueprint, view: :list
  association: :contact, blueprinter: ContactBlueprint, view: :full
  ...
end

We can close this - it will be serve as good documentation for anyone looking to do something similar.
Cheers!

One follow up - the transformer is working brilliantly, thanks a lot!


Fix details text and summary left alignment

Description
Adjusted the details component padding to align the text and summary (See issue 615).
Checklist

[x] Tested against our testing policy (Resolution, Browser & Accessibility)
[x] Follows our coding standards and style guide
[x] CHANGELOG entry

I need to update the backstop tests.


[SPARK-16320] [DOC] Document G1 heap region's effect on spark 2.0 vs 1.6

What changes were proposed in this pull request?
Collect GC discussion in one section, and documenting findings about G1 GC heap region size.
How was this patch tested?
Jekyll doc build

Test build #64141 has started for PR 14732 at commit e325f7b.

Test build #64141 has finished for PR 14732 at commit e325f7b.

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

Merged build finished. Test PASSed.

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

Looks good!

LGTM. Merging to master and branch 2.0. Thanks @srowen

Oh heh, too late. No problem we may further improve the GC docs soon anyway. The existing link wasn't wrong.

ah sorry... Feel free to send out a small pr to add the link back :)

Nah it still has a link to the same document actually (which is why I removed one of them), the only problem is I noticed the one I kept was a link to an older version. It's not worth another change just for this. Will touch it up in a future change to the GC docs.


broker not responding to 'stop' command - using pkill beam to close the broker

Environment
Mac OSX El-Capitan / Ubuntu 14.04LTS
after running the broker via: ./bin/emqttd start
the command: ./bin/emqttd stop
Always returns:
Node 'emqlocal@mycomputername' not responding to pings.
The behavior happens like stated on two machines
my local development and the production Ubuntu server running on Amazon EC.
I have to kill the beam manually every time before I can start
the broker again, via: pkill beam

@syberkitten  Please check your ~/.erlang.cookie file exists,If it exists, delete it

I did, but it did not change the behavior.

@syberkitten

The node name should be Name@Host, where Host is IP address or the fully qualified host name.
Configure the /etc/hosts file if FQN is used, for example: <IP_ADDRESS>   emq.mydomain

I run emqtt first time on my mac os x 10.13.It failed to start and throwed some erlang error.After I saw the advice to delete ~/.erlang.cookie,It works.Confusing


MaskDecoder resolution

Has the current mask decoder only a learnable output resolution of 256x256 and then only interpolated?
Are you not going to see a lot of interpolation artifacts going from 256x256 to 1024x1024 without any other intermediate learnable step/layer?

yes, we only do interpolation and I think it's the common approach to handle different resolutions.

Have you tried to learn an additional up sampling layer to reduce the artifacts?

unfortunately, we didn't try upsampling layer. if you are interested, you can conduct some experiments and we welcome merging to our code base if that demonstrate better performance.

I have tried a few more layers to progressively upscale up to 1024. Of course these extra are initialized from scratch but it seems hard for them to learn. This is why I have asked if you have experimented with this.
An upscale from 256x256 to 1024 you are going to lost a lot of details.


Select the size for the trade

Task
As a user I would like to select the size for the trade
ACs

[x] It should allow me to select the amount using a slider that has the leverage percentage applied to my total balance for the settlement asset

[x] It should have a minimum amount set for this type of trade
[ ] It should have the maximum amount set to the leveraged balance

[ ] It should show the amount selected by the slider below with label “Size ([Currency, e.g. USDC])” and a tooltip with more info
[ ] It should show the current estimated price with label “Est. Price” and a tooltip with more info
[ ] It should show the estimated close out price with label “Est. Close Out” and a tooltip with more info
[ ] As a user I would like to see information on the fees to make a trade (no fees)
[x] It should match the design
[x] It should be A11Y friendly
[x] It should work in dark/light mode

Question: what goes into the tooltip


Failing to run my first ever Flutter App

I'm getting below error on running my first ever flutter app:

Error running Gradle:
Exit code 1 from: D:\PROJECTS\softwareProjects\AndroidProjects\flutter_app_2\android\gradlew.bat app:properties:
Download https://jcenter.bintray.com/org/bouncycastle/bcprov-jdk15on/1.56/bcprov-jdk15on-1.56.jar

========================================

Error running Gradle:
Exit code 1 from: D:\PROJECTS\softwareProjects\AndroidProjects\flutter_app_2\android\gradlew.bat app:properties:
Download https://jcenter.bintray.com/org/bouncycastle/bcprov-jdk15on/1.56/bcprov-jdk15on-1.56.jar

FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring root project 'android'.

Could not resolve all files for configuration ':classpath'.
Could not download bcprov-jdk15on.jar (org.bouncycastle:bcprov-jdk15on:1.56)
Finished with error: Please review your Gradle project setup in the android/ folder.
Could not get resource 'https://jcenter.bintray.com/org/bouncycastle/bcprov-jdk15on/1.56/bcprov-jdk15on-1.56.jar'.
Response 304: Not Modified has no content!

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Get more help at https://help.gradle.org

=====================================================
What would possibly be the solution ? Help!!

Could you please post the "flutter doctor -v" output.

PFB:
C:\Users\silve>flutter doctor -v
[√] Flutter (Channel beta, v0.1.5, on Microsoft Windows [Version 10.0.16299.309], locale en-IN)
• Flutter version 0.1.5 at C:\Users\silve\flutter
• Framework revision 3ea4d06340 (4 weeks ago), 2018-02-22 11:12:39 -0800
• Engine revision ead227f118
• Dart version 2.0.0-dev.28.0.flutter-0b4f01f759
[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
• Android SDK at C:\Users\silve\AppData\Local\Android\sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-27, build-tools 27.0.3
• Java binary at: C:\Program Files\Android\AndroidStudio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
[√] Android Studio (version 3.0)
• Android Studio at C:\Program Files\Android\AndroidStudio
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
[√] Connected devices (1 available)
• Android SDK built for x86 • emulator-5554 • android-x86 • Android 8.0.0 (API 26) (emulator)
• No issues found!

Sounds like an issue with Gradle

how to fix Gradle issue?? The jar file, downloading from the link seems to be broken.

You can probably fix that by going opening a command prompt to your project/android folder and running gradlew build --refresh-dependencies.
Once you've done the initial flutter run or flutter build command (or pressed build in your IDE) you can use gradle as you would for any other android project.
Quick question though - have you added anything to your gradle dependencies in android/build.gradle or android/app/build.gradle? I don't remember bouncy castle being needed for flutter.
Another thing to try would be deleting your gradle cache. See this stackoverflow question clearing gradle cache.
If that doesn't work and you haven't added anything manually, it would help to list which packages you depend on in your pubspec.yaml.

I'm getting same error while running gradlew build --refresh-dependencies or trying to remove the gradle cache.
d:\PROJECTS\softwareProjects\AndroidProjects\flutter_app_2\android>gradlew build --refresh-dependencies
FAILURE: Build failed with an exception.

What went wrong:
Could not determine java version from '9.0.4'.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Get more help at https://help.gradle.org

I deleted the cache and tried to run the App from my ide but still getting the same issue.

Hmm. Well the gradlew build error seems like a completely different issue - are you running java 9? If so, you need to be using a recent version of Gradle or it won't work (I believe 4.4+ but I could be wrong about that). To upgrade gradle you can modify android/gradle/wrapper/gradle.properties and change the gradle distribution url to https://services.gradle.org/distributions/gradle-4.6-all.zip. Or just make sure java 8 is in your path when working on this project.
Flutter Doctor says it's using java 1.8 though so that shouldn't affect the IDE's build or anything using flutter commands.
Try adding maven central to your repositories in android/build.gradle. That might be the problem, although the errors it's giving is a bit weird.
allprojects {
    repositories {
        google()
        jcenter()
        mavenCentral()
    }
}

I changed the gradle distribution to url to https://services.gradle.org/distributions/gradle-4.6-all.zip.
tried with the Java version 9 in my machine but it didn't work. then downgraded my Java version to 8 and
added mavenCentral() to allprojects.
Nothing is working. Even checked my firewall in case it's blocking the download of the file, but firewall also is not the issue here.

Hmm well I'm at a little bit of a loss. Is your project something that can be made public? I'd be willing to give it a try if you're able to share the source.
If not, if you could post your android/build.gradle, android/app/build.gradle, and pubspec.yaml that might help me figure out what's going on.

Ah. This might not be anything to do with your project, although I'm still slightly curious as to why bcprov-jdk15on is needed unless you've explicitly required it. But check this and this out - it seems as though there's an issue with jcenter's Bintray configuration.
If you remove jcenter() or put it after mavenCentral() that might help as it should then try to download the dependency from mavenCentral first.

Sure I can upload my project. It's just a demo app to test run flutter on my new pc.

flutter_app_3.zip

@rmtmckenzie Please find above. I zipped the entire app folder to make sure I was missing nothing.

I just added both mavenCentral(),  jcenter() in allprojects and it worked!!  :)
I did it once previously also but it didn't work. May be I did something wrong that time but now it's working!!
@rmtmckenzie Thanks for the suggestion. PFB
========= below code is from fixed project=======================
buildscript {
repositories {
google()
mavenCentral()
jcenter()
}
dependencies {
    classpath 'com.android.tools.build:gradle:3.0.1'
}

}
allprojects {
repositories {
google()
mavenCentral()
jcenter()
}
}
rootProject.buildDir = '../build'
subprojects {
project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
project.evaluationDependsOn(':app')
}
task clean(type: Delete) {
delete rootProject.buildDir
}

Glad it worked out! I think the problem is that jCenter is having issues right now, and by putting MavenCentral before it in your build.gradle gradle checks that first and so doesn't fail when jCenter craps out =D.

yeah even i was facing this issue and it works after adding mavenCentral(), thanks @1SouravGhosh

The same problem,adding mavenCentral(),But it did not work properly, and it gave an error.
on windows

Error running Gradle:
ProcessException: Process "E:\flutter\first_app\android\gradlew.bat" exited abnormally:
FAILURE: Build failed with an exception.
What went wrong:
org/gradle/api/internal/artifacts/ivyservice/ErrorHandlingConfigurationResolver$BrokenResolvedConfiguration
Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
Get more help at https://help.gradle.org
BUILD FAILED in 1s
Command: E:\flutter\first_app\android\gradlew.bat app:properties

The following is my build file
`buildscript {
repositories {
google()
mavenCentral()
jcenter()
}

**dependencies {
    classpath 'com.android.tools.build:gradle:3.2.1'
}**

}
allprojects {
repositories {
google()
mavenCentral()
jcenter()
}
}
rootProject.buildDir = '../build'
subprojects {
project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
project.evaluationDependsOn(':app')
}
task clean(type: Delete) {
delete rootProject.buildDir
}
`


Make nephio compliant with the OpenSSF checks

Currently, Nephio does not covers all the OpenSSF checks and does not fulfill the criterion of OpenSSF checks:
Below are the scores of the individual OpenSSF checks.
Aggregate score: 4.7 / 10

SCORE
NAME
REASON
REFERENCE

10 / 10
Binary-Artifacts
no binaries found in the repo
link

3 / 10
Branch-Protection
branch protection is not maximal on development and all release branches
link

9 / 10
CI-Tests
26 out of 28 merged PRs checked by a CI test -- score normalized to 9
link

2 / 10
CII-Best-Practices
badge detected: in_progress
link

9 / 10
Code-Review
found 1 unreviewed changesets out of 28 -- score normalized to 9
link

10 / 10
Contributors
project has 9 contributing companies or organizations
link

?
Dangerous-Workflow
no workflows found
link

0 / 10
Dependency-Update-Tool
no update tool detected
link

0 / 10
Fuzzing
project is not fuzzed
link

10 / 10
License
license file detected
link

10 / 10
Maintained
10 commit(s) out of 30 and 14 issue activity out of 30 found in the last 90 days -- score normalized to 10
link

?
Packaging
packaging workflow not detected
link

0 / 10
Pinned-Dependencies
dependency not pinned by hash detected -- score normalized to 0
link

0 / 10
SAST
SAST tool is not run on all commits -- score normalized to 0
link

0 / 10
Security-Policy
security policy file not detected
link

?
Signed-Releases
no releases found
link

?
Token-Permissions
no github tokens found
link

?
Vulnerabilities
internal error: vulnerabilitiesClient.ListUnfixedVulnerabilities: osvscanner.DoScan: package {../../krm-functions/configinject-fn Go lockfile:/tmp/repo3160905993/controllers/pkg/go.mod} does not have a commit, PURL or ecosystem/name/version identifier
link

We need to make the score card good in order to make this project compliant with the OpenSSF best practices.

Security policy is handled as part of #454


And.left and And.right have malformed constant info

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Check that your issue is not already filed.
Reduce the issue to a minimal, self-contained, reproducible test case. Avoid dependencies to mathlib4 or std4.

Description
In v4.8.0-rc1, the ConstantInfo for And.left and And.right are malformed and cause lean to segfault (at least on aarch64-apple-darwin). Only the field TheoremVal.all seems affected so this is perhaps related to #4035. Also note that And.left and And.right are structure fields rather than standalone theorems.
Context
Observed after updating code to v4.8.0-rc1 when an alias foo := And.left declaration caused a server crash. Zulip mention
Steps to Reproduce
This file causes lean to segfault.
import Lean

def test : Lean.CoreM (List Lean.Name) := do
  let .thmInfo tval ← Lean.getConstInfo `And.left | unreachable!
  return tval.all

#eval test

Version: Lean (version 4.8.0-rc1, aarch64-apple-darwin, commit dcccfb73cb24, Release)
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

This is perhaps limited to declarations in Init. The following does not cause a segfault:
import Lean

structure Both (p q : Prop) : Prop where
  fst : p
  snd : q

def test : Lean.CoreM (List Lean.Name) := do
  let .thmInfo tval ← Lean.getConstInfo `Both.fst | unreachable!
  return tval.all

#eval test


Create entrezdirect/esummary nf-core module

Write entrez-direct nf-core module: entrezdirect/esummary.

Latest entrez-direct version: 16.2-0 Bioconda Package Index.

The nf-core pull-request has been merged


Filename islands

This is to reduce magic as mentioned in #555. Islands are now grabbed from the components directory and require .island.{ts,tsx,js,tsx} extension.
Things to do:

[x] Add flag to toggle this behaviour
[ ] Allow islands to be imported from subdirectories
[ ] Add tests
[ ] Add example
[ ] Add documentation

Update: added options to dev method to toggle this behaviour. The default behaviour is to use the islands directory

I'm closing this as it hasn't had any progress in the last 9 months. Also, IMO, the current islands folder setup works fine. Thanks anyway 🙂


Thorium freezes when I try to edit a bookmark

System Details

OS: Ubuntu 22.04.04
Thorium Version M121.0.6167.204

Problem
When I try to edit a bookmark, the thorium window shakes and thorium freezes. only a pkill -f thorium can end thorium.
Screenshots
see little Video:
https://github.com/Alex313031/thorium/assets/41851582/ef6c8116-12c0-42ec-83b0-1b339695734b
Additional Notes
The same effect is also in Google Chrome!

Tested:
With M120.0.6099.235 there's no Problem with editing a bookmark.

If chrome also has this problem, it might be a bug from upstream
@LinuxOpa

see here:
https://issues.chromium.org/issues/325307460

@LinuxOpa So it's an upstream thing.

Fixed in Google Chrome and Chromium, when in Thorium?

From the problem report link you provided, I don't see any patch links related to this issue, so I'm not sure on which branch they fixed this issue, maybe the M122 version, maybe the TOT version.


'more' option crashes program

When using the 'more' option, the program will crash with:
Exception: 'pentestGPT' object has no attribute 'step_reasoning_response'
Full session before crash
C:\home\kali\PentestGPT> python3 main.py --reasoning_model=gpt-4
- ChatGPT Sessions Initialized.
Please describe the penetration testing task in one line, including the target IP, task type, etc.
> HackTheBox single host challenge. Windows machine. <IP_ADDRESS>.
- Task information generated.

PentestGPT suggests you to do the following:
Based on the information provided, I'll update the tasks and provide a recommendation for the next task.

Initial tasks:
1. Information gathering
   1.1 Perform network scanning
   1.2 OS fingerprinting
   1.3 Service enumeration

Next Task:
1.1 Perform network scanning - Use a tool like Nmap to scan for open ports and running services on the target machine (<IP_ADDRESS>).
You may start with:
Sure, here's an example of what the first step could look like:

Task: Perform a port scan on the target IP address to identify open ports.

Command:

nmap <target_ip>

Description:
The 'nmap' command is used to perform a port scan on the target IP address. This will help identify which ports are open and what services are running on them. The output will provide important information about the target system's
architecture, operating system, and potentially vulnerable services. You can use the '-A' option to enable OS detection, version detection, script scanning, and traceroute all in one command.

Example:

nmap -A <IP_ADDRESS>

This command will perform a comprehensive scan on the IP address '<IP_ADDRESS>'. It will use aggressive scanning techniques to identify open ports, running services, and the operating system. It may also run scripts to detect
vulnerabilities. The output will be displayed in the terminal window.
>  more
Invalid task, try again.
> more
Exception: 'pentestGPT' object has no attribute 'step_reasoning_response'

Thanks for this. This is a bug to fix on the list. It should be fixed in a commit pushed just now.


pull request for e2e test at 11/27/2016 14:56:07

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


implemented --no-browser option closes#12

Coverage increased (+0.5%) to 78.864% when pulling bd331208cde9c162ffbc52baacb3570bb7d76330 on topic-no-browser-setup-#12 into 2d48f91e93e2772858734c8c8dfda67bf9b214da on master.


refactor 3-appfactory

remove unused TF code (only cymbal-bank/frontend was utilized by code, which has been renamed cymbal-bank)

i think the intent is for each app having its own appfactory, each app's cicd pipeline is created in their respective appfactory admin project. similar to https://github.com/GoogleCloudPlatform/terraform-google-enterprise-application/pull/78, the 5 other apps also have their integration tests.

remove unused TF code (only cymbal-bank/frontend was utilized by code, which has been renamed cymbal-bank)

i think the intent is for each app having its own appfactory, each app's cicd pipeline is created in their respective appfactory admin project. similar to #78, the 5 other apps also have their integration tests.

Thanks @yliaog!
Looking through the code and CI logs, it appears the frontend appfactory is currently used for all apps in stage 5 - hence my rename to cymbal-bank.  However if the intention was to use a dedicated appfactory for each of those (frontend, accounts-contacts, etc), I can certainly make that change in https://github.com/GoogleCloudPlatform/terraform-google-enterprise-application/pull/112 by updating appfactory_test.go to point to apps/cymbal-bank instead of apps/cymbal-bank/frontend - probably need some downstream tweaks as well.
I'll mock it up prior to our next call. :)


Fix duplicate symbol compile error in amalgamation build.

Let me know if you'd prefer to fix this another way, e.g. renaming those functions but keeping them file-local.

Don't know what the maintainer will say, but I would say to keep the
function in the compiler. The rationale is that it an language detail.

There's a fairly tight coupling between the compiler and the VM though, and it doesn't make much sense to duplicate the functionality, especially if it's important both methods are kept in sync.

This is not really a tight coupling, it is an implementation choice of the
language with that VM.
Anyway, it concerns a grammatical rule of the language, it should stick
next to the language definitions.

Thanks @gw3583 I think the fixes make sense here. I'm going to merge this into master once I bring the 0.3.0 branch over. Part of the goal 0.3.0 is fixing all the build processes and this is one of those fixes, so thanks!

The fix went into the 0.3.0 branch and was merged in. I appreciate the contribution even though this one wasn't merged!


Filter 2fa giveaway project list

Fixes #12138 (cc @hugovk)

Confirmed fixed, thanks!

https://pypi.org/security-key-giveaway/


Define the wrap function for the async example in Best Practices page

This example came from https://strongloop.com/strongblog/async-error-handling-expressjs-es7-promises-generators/ but the definition of wrap() wasn't replicated here.  This adds the definition to avoid confusion.

Thanks for the PR @davidjb
Good suggestion, but I'd prefer to add the information outside of the code block.  Something like this right after the code example:
The `wrap()` function is a wrapper that catches rejected promises and calls `next()`.
For details, see [https://strongloop.com/strongblog/async-error-handling-expressjs-es7-promises-generators/#cleaner-code-with-generators](Asynchronous
Error Handling in Express with Promises, Generators and ES7).

@crandmck Thanks for the suggestion; updated.  I removed the repeated link to https://strongloop.com/strongblog/async-error-handling-expressjs-es7-promises-generators/ as it would have been there twice in the same text block.


add querystring to options; add JetBrains project dir to .gitignore;

I noticed that there was no way to increase the requested image resolution without weird hacks in the template. Here's a way to apply a query string to all of the Gravatars by string or by object.

Nice @azurelogic! Thanks for the awesome addition :+1: :cake:
Adding the sizing options was always on my to-do list, just never got around to it. I've just been doing it in the templates.
With these changes, can sizes be controlled on a per-entry basis? Or are they set globally?

It's global. I considered a per entry basis, but I thought the config could get weird.
This:
metalsmith.use(gravatar({
  stevenschobert<EMAIL_ADDRESS>}));

would become this:
metalsmith.use(gravatar({
  stevenschobert: {
    email<EMAIL_ADDRESS>    querystring: {...}
  }
}));

It's not so bad with one or two, but it could look a little crazy if you did it for a bunch of them in various groupings. I suppose the global setting could be overridden by individual settings. I'll see what I can do with that this evening.

Yeah I can see it would get a little busy with multiple entries. Though we could make it so that you can supply either a string or an object.
metalsmith.use(gravatar({
  stevenschobert: {
    email<EMAIL_ADDRESS>    size: {...}
  },
  testuser<EMAIL_ADDRESS>}));

This would make the changes backwards compatible, as well as let people opt-in to controlling options on specific entires.

Something else that you got me thinking about; should we use the option names like size rating default, rather than exposing querystring as an option?
metalsmith.use(gravatar({
  stevenschobert: {
    email<EMAIL_ADDRESS>    size: 200,
    rating: "g",
    default: "http://myapp.com/default_avatar.png"
  },
  testuser<EMAIL_ADDRESS>}));

I'm wondering if that might be a little more user friendly than exposing querystring? This means the user won't have to look up the options Gravatar supplies, and figure out how to build the querystring themselves. It would also give us a place to put additional logic around individual options later, without breaking the API.
What do you think? Does gravatar supply too many options that would make that difficult to implement?

Also, you are awesome for helping, and I really appreciate it! If you don't have to time to fuss with any of that I totally understand!

I'm inclined against the granular options only because it allows for Gravatar to add (or even change) their API without requiring someone any updates to the mappings in this plugin. I figured that this configuration option was a good compromise:
{
  "plugins": {
    "metalsmith-gravatar": {
      "options": {
        "querystring": {
          "s": 200,
          "r": "pg"
        }
      },
      "avatars": {
        "stevenschobert"<EMAIL_ADDRESS>        "authors": { ... }
      }
    }
  }
}

As for the individual settings, I'm thinking of adding them in such a way that they would just override the global settings. Sound good?

I've got something working. It's not quite as elegant as before, because we need to detect whether the user id is being set to an email address string or an object in the middle of mutateStringsInObject(). I need to finish the documentation, then I'll can either squash everything into a single pull request, or you can merge this one, and I'll make a separate PR. Let me know which you prefer, and I'll get it wrapped up in the next few days.

I see your point about flexibility and the gravatar api. Though I don't see gravatar changing their api in the near future, so I'm not convinced that flexibility is really necessary. Really I was thinking more about "who uses metalsmith? do they know what a querystring is?".
As for the pull-request, you can keep all the commits in this PR and squash, so we see all the changes and conversation in the same place. :)

Sure. I guess my thought was that metalsmith users have to have some knowledge of node, npm, and front-end dev. I would hope that most of them to know what a query string is. In addition, I added a link to Gravatar's docs from the documentation. If you think it adds value, we can add custom mapping, but I felt it would make the code less elegant and more brute force.
I'm currently prepping a talk to give tomorrow that was sprung on me last minute. I should be back to this in the next few days though.

Alright, I have everything together for individual overrides, including tests and documentation. I think it works pretty well this way, and it's more hands off in the case of future additions/changes. If you still think that it still needs more explicit query string params though, let me know. We would have to figure out whether we want to allow both "s" and "size" or just "size".

@stevenschobert What are your thoughts on merging this in and deploying it on npm? Do you think we need the additional mapping, or do you want to proceed from here?

Just saw this in my list of open PRs and thought I would follow up.


In v1.0.0 tag's tx_id seems wrong

In v1.0.0 when I get a tx_id value from tags table I find that in transactions there is no record with id equal to it. This looks like a bug (and it is confirmed by always empty tags arrays returned).
I am checking whether v0.12.1 is also vulnerable.

Always empty tags array was probably because of (erroneous) special handing of name: "Content-Type" in tags: [{ name: "Content-Type", values: [... }].


Update base MR service to include labels and annotations for the BFF

Description
Adding a new label called component and setting it to model-registry. This will allow us to filter services in the BFF rather than looping through all services.
Also adding annotations for displayName and description. The BFF looks for these annotations to display the name of the MR when changing registries in the UI. Everything will work without these but this is a nicer UX and can help with determining which model registry is which.

How Has This Been Tested?
Tested locally on kind and got:
curl localhost:4000/api/v1/model_registry
{
	"data": [
		{
			"name": "model-registry-service",
			"displayName": "Default Model Registry",
			"description": "The default model registry"
		}
	]
}

Merge criteria:

All the commits have been signed-off  (To pass the DCO check)

[x] The commits have meaningful messages; the author will squash them after approval or in case of manual merges will ask to merge with squash.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work.
[x] Code changes follow the kubeflow contribution guidelines.

If you have UI changes

[ ] The developer has added tests or explained why testing cannot be added.
[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Verify that UI/UX changes conform the UX guidelines for Kubeflow.

/lgtm @tarilabs @dhirajsb what do you think know?

it looks to me https://github.com/kubeflow/model-registry/pull/525#discussion_r1831765502 is not yet addressed.


Azure event hub missing

the very first image shows Event Ingestion can take place with an Azure Event Hub.  There is nothing anywhere in this document that discusses how this can be accomplished.

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

ID: 23d54769-c90f-e289-563e-5f877d629a28
Version Independent ID: 86d52c37-4e7c-b4bf-89b4-1c3de4409473
Content: Data scenarios involving Data Lake Store
Content Source: articles/data-lake-store/data-lake-store-data-scenarios.md
Service: data-lake-store
GitHub Login: @nitinme
Microsoft Alias: nitinme

@doghousesoftware Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

@doghousesoftware Thanks for your interest in Data Lake Store. Under the "Streamed data" section of the article, there is a bullet point that tells how to use "EventProcessorHost" to receive events from EventsHub.
#please-close


Document generic data sources feature

Document the AnyVolumeDataSource feature gate. Don't include any user-facing documentation at this time, because the the feature is experimental and only relevant to developers.

/assign @msau42

Link to the relevant section of the preview:
https://deploy-preview-19315--kubernetes-io-master-staging.netlify.com/docs/concepts/storage/persistent-volumes/#data-population

/lgtm

Hi @bswartz
We are very close to the docs final deadline i.e All 1.18 enhancements docs should be in ready to merge state by Monday 16th March. Please make sure to complete all the docs reviews before the deadline.
Thanks!

Hi @bswartz
We are very close to the docs final deadline i.e All 1.18 enhancements docs should be in ready to merge state by Monday 16th March. Please make sure to complete all the docs reviews before the deadline.
Thanks!

This is ready to merge. Just waiting for approval.

@bswartz please target the dev-1.18 branch for this PR.
/cc @VineethReddy02 @zacharysarah
/milestone 1.18

@bswartz as @jimangel mentioned please point this to dev-1.18 as these docs are specific to a feature enhancement targeted for 1.18 release.

@msau42 PR rebased on dev-1.18

@bswartz I think you'll need to change the base branch for this PR to dev-1.18; it's currently master.
For safety, I'm going to add a
/hold
Happy to run through details in case this is supposed to merge into master.  https://kubernetes.io/docs/contribute/start/#choose-which-git-branch-to-use explains how SIG Docs uses branches.

@sftim Thank you!

/remove- area blog
/remove-language de
/remove-language es
/remove-language id
/remove-language it
/remove-language ja
/remove-language ko
/remove-language pl
/remove-language zh

/remove-area blog

Hey @bswartz , The changes look okay. I assume that this feature is a new addition for 1.18 or are you documenting an existing Alpha feature? Typically, there should be docs to introduce the feature.

Hey @bswartz , The changes look okay. I assume that this feature is a new addition for 1.18 or are you documenting an existing Alpha feature? Typically, there should be docs to introduce the feature.

@kbhawkey We decided to skip that for this feature since it's mostly developer facing (until beta) and any user-facing explanations would probably cause more problems then help. Earlier on I had a few paragraphs in the storage section, but I removed them after discussion with @msau42 .

/lgtm
/approve


NaN in training

Hi,
Sorry to bother you again. I got your code working with the data you provided, but unfortunately after only a few steps the regularization loss AND the elbo become NaN (and don't recover from this). Have you experienced something similar?
I'm using Pytorch from latest master btw.
Thanks again,
Justus
EDIT: This does not seem to be a problem with the data. I tried to overfit it on 5 samples and it worked well for 1700 batches (batchsize=4) but then the NaNs occured

I made two modifications in this code that seems to comply with the official re-implementation with a higher chance of avoiding NaNs in the default \beta=1 setting. They are: 1) When calculating the reconstruction loss I average over the batch size, which should be more appropriate compared to how KL is computed, which is also the approach in the re-implementation. 2) All weight initialization are orthogonal (mentioned in the appendix H of the paper).


Is element.vue save to use?

Is there a clean way to get the current vue instance from a given DOM node or is it even future save to access element.__vue__?
The use case is the integration with a large famous enterprise content management system where we would like to build up all content elements out of vue components.
Unfortunately the CMS ships with javascript code which can't be changed without too high efforts.
These third party scripts allow authors of the cms to use drag'n'drop to place new content element (e.g. a carousel or a teaser) to the page.
After the drag'n'drop happened an event is fired which we would like to bind on and reinitialize all new placed inline template elements.
We managed to come up with a hack but maybe there is a better solution for this use case:
https://jsfiddle.net/5vxwr34o/
/**
 * Content change handler
 */
function handleContentChange() {
  const inlineTemplates = document.querySelector('[inline-template]');
  for (var inlineTemplate of inlineTemplates) {
    processNewElement(inlineTemplate);
  }
}

/**
 * Tell vue to initialize a new element
 */
function processNewElement(element) {
  const vue = getClosestVueInstance(element);
  new Vue({
    el: element,
    data: vue.$data
  });
}

/**
 * Returns the __vue__ instance of the next element up the dom tree
 */
function getClosestVueInstance(element) {
  if (element) {
    return element.__vue__ || getClosestVueInstance(element.parentElement);
  }
}

To some extent yes - the official devtool relies on it too, so it's unlikely to change or break.

Thanks @yyx990803 - you see any reason not to follow this approach?

@jantimon can't really say much because this is such a niche case, but I'd suggest tearing down existing Vue instances that are no longer needed.

@yyx990803 I'm not sure if this is really a niche case. This pretty much affects every WYSIWYGish UI representation for editors of any content management system with such a feature.
From the point on you need or rely on some client-side rendering technique in a content component this case becomes present.
That are just my cents for this discussion 😃

we have had some issues were an outer component just delegates to another component using <component :is="...">  in those cases the outer component and the inner component both linked to the same $el and the vue property was sometimes linked to the out component and sometimes to the inner component. So we don't rely on vue to find the correct component all the time.

Nitpick: save is a verb, safe is an adjective. These are different words.

What's the equivalent in Vue 3? I see there's __vueParentComponent but I have to now say __vueParentComponent.props.x instead of vue.x.

I think the vue3 equivalent would be element.__vueParentComponent.ctx.$props.


feat: some improvements in bundle batch

Add bundle batch routes
add bundle transaction methods (WIP)

Any chance this can get up to date with master and merged please? Would really like to use the batch/transaction endpoints

@jjosef Sure I can look at this today. Appreciate the mention as this got lost for me.

@jjosef Sure I can look at this today. Appreciate the mention as this got lost for me.

@jjosef Sure I can look at this today. Appreciate the mention as this got lost for me.

Hi @zeevosec let me know if you need help with this! Cheers


Tree not visible for PR_000001954 in CL

https://www.ebi.ac.uk/ols4/ontologies/cl/classes/http%253A%252F%252Fpurl.obolibrary.org%252Fobo%252FPR_000001954?isDefiningOntology=false

this is also broken in OLS3: it just displays the root "cell" class
https://www.ebi.ac.uk/ols/ontologies/cl/terms?iri=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FCHEBI_36080

Fix has been deployed. Thanks.


Missing IrcPacket.java?

You note in your last commit that you're redirecting traffic through IrcPacket, but I don't see it in the source tree. Perhaps it needs to be added?

Well that was fast! :)
Thanks. I've been thinking about messing around with an IRC client
that does things the way I want. I like the simplicity of what you've
set up.
RJ
On Sun, Nov 13, 2011 at 11:58 AM, sorcix
<EMAIL_ADDRESS>wrote:

Has been fixed in commit 0846c85484542b4a440d1285d5ed1aad65e40bff!

Reply to this email directly or view it on GitHub:
https://github.com/sorcix/sIRC/issues/2#issuecomment-2723160

--
RJ

If you have any issues with it, don't hesitate to let me know!


tsc trying to compile typings

TypeScript Version:
1.7.5 / 1.8.0-beta / nightly (1.9.0-dev.20160217)
Code
Here is my test/tsconfig.json content:
{
    "compilerOptions": {
        "target": "es5",
        "module": "commonjs",
        "moduleResolution": "node",
        "isolatedModules": false,
        "jsx": "react",
        "experimentalDecorators": true,
        "emitDecoratorMetadata": true,
        "declaration": false,
        "noImplicitAny": false,
        "removeComments": true,
        "noLib": false,
        "preserveConstEnums": true,
        "suppressImplicitAnyIndexErrors": true
    },
    "filesGlob": [
        "**/*.ts",
        "**/*.tsx",
        "!node_modules/**"
    ],
    "files": [
        "simple-organizer.test.ts"
    ],
    "atom": {
        "rewriteTsconfig": true
    }
}

Expected behavior:
When running tsc -p test, the file simple-organizer.test.ts should be compiled.
Actual behavior:
The script fails with this error:
james@JM-LAPTOP:~/Development/tidyfolders$ tsc -p test
typings/main/definitions/tape/index.d.ts(9,26): error TS2307: Cannot find module 'stream'.

@Jameskmonger do you have a reference from the test.ts file to the tape definitions?

I think you should fix the error in typings, typings will be compiled, but no js file generated.

"**/*.ts"

Doesn't this map to .d.ts files in typings?

@blakeembrey Yes I have a reference to the tape definitions from the test.ts file.
@plantain-00 The typings file is downloaded from the typings project.
@DanielRosenwasser My project layout is like this:
-test
-- test.ts
-- tsconfig.json
-typings

That glob shouldn't affect the typings files @DanielRosenwasser

You need a reference to node.d.ts

Right you are @blakeembrey! Issue solved :smile:


OO/A1H Changes to Error a File with an Invalid Equipment Model Name

Problem statement
A1H data sometimes has an invalid equipment model name, but we don't fail the file, so A1H knows there is a problem to be fixed.
Acceptance criteria

[x] A file with an invalid equipment model name fails on the front-end.

To do

[x] Add cardinality: ONE to equipment_model_name
[x] Add mapperOverridesValue: true to equipment_model_name

Dependancies

[ ] #3699 must be deployed first.

complete.


Route Parser returns false, don't know why

I have a route like so:
const path = '/account/json/wall/post/:id/comments/?start=:start&max=:max';

I call this with an object
const vars = {
  id:50,
  start:0,
  max:12
};

return new Route(path).reverse(vars);

It returns false?

So, yeah, I found the bug. route-parser will accept a number if there is only one key in the object, but not if there are more. :-(
Time to get some new kevlar!

Thank you for this! I think I said "bullet proof" before I added reversing... Should be fixed in master. I think I'm going to take a pass at query params and updating dev dependencies before cutting a new version for NPM.
Let me know if that doesn't work for you.


Widgets should be able to customize assigned to.

Suggested new feature or improvement
All the github widget options are tied to me.
But I may want to track issues assigned to multiple people on my team.

Scenario
The same as for 'me', but be able to look at my team.
Arguably I should be able to customize a string that i know GitHub supports, like labels and assigned to etc.
Maybe the simple way to do it would be to support the link
https://github.com/microsoft/devhome/issues?q=is%3Aissue+is%3Aopen+devdrive
Additional details
No response

OK, playing around, I think my problem was the enter button was not working.


Example

The example is less than enlightening since creating a database is a feature of the task itself, without inline sql. How about a simpler example of database creation and second example of inline sql?

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

ID: edb0dd0e-99f2-2fc1-6e28-fc89ab457b5c
Version Independent ID: f6cd5426-6258-9a75-dda7-81b377c29bed
Content: MySQL Database Deployment On Machine Group task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/mysqldb-deployment.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

Please re-open this issue.
Please add a simple example of database creation without inline SQL.


Convert the full package written in Python / Latex and generate the plots with matplotlib.

This would possibly be easier to maintain in the long term.

This might be really time consuming. It might be worth converting bash / matlab scripts to python though..
This would make the pipelines in Python / R only and generated reports in Latex.

Need to convert sb_param_estim__copasi, and sb_param_scan__single_perturb from BASH to Python.
The run_test.py doesn't currently work for sb_simulate.py.

Need to convert sb_param_estim__copasi, and sb_param_scan__single_perturb from BASH to Python.

sb_param_estim__pw is not ported to Python. This is not urgent though.

Another idea could be to import R from Python (I think it is using the rp2 package). Not sure whether this is much more convenient though. It adds a bit of control over R, but it the dependency on R remains.

OR
forget about the statistics (for now) and just port the plots from R to matplotlib. In particular, the most relevant plots are for sb_simulate and sb_param_scan__single_perturb.
Note: The code for comparing experiments and data is also coded in R right now.
It might be worth starting with one (e.g. sb_simulate) and see how it goes.

Actually... the statistics generated in sb_simulate are quite important.. One can reproduce all the data from a stochastic simulation easily from the summary file.
Aside from the decrease in dependencies, I am not sure what other additional benefit one can have by converting these plots from R to Python.
This conversion has low priority for now.

It is better to leave statistics and plots in R, so that they can be executed for data analysis separately if needed


Error on VSCode reload

Which OS are you using?
Windows
Expected Behaviour
Saw an error from this extension when I reloaded VSCode
Actual Behaviour
...
I also see the following error:
[2023-03-06 14:29:36] |   ERROR   |         Git        | fatal: not a git repository (or any of the parent directories): .git

Reproduction steps

Go to '...'
Click '...'
See error '...'

Additional Notes
No response
Log
STARTUP
[2023-03-06 14:29:35] |   DEBUG   |     Filesystem     | Created directory at "/Users/kerry/Library/Application Support/Code/User/globalStorage/sumneko.lua/addonManager"
[2023-03-06 14:29:36] |   ERROR   |         Git        | Failed to checkout repo!
[2023-03-06 14:29:36] |   ERROR   |         Git        | fatal: not a git repository (or any of the parent directories): .git

Can you please share what is at /Users/kerry/Library/Application Support/Code/User/globalStorage/sumneko.lua/addonManager? I have seen this issue on UNIX machines due to pathing issues, but not on Windows.

Closing stale issue


[WIP] Implementing chemical MC switching

Beginning of initial implementation of direct switching between compounds/mutations. Currently getting together the FFGeometryEngine (so named because it uses forcefield data to propose geometries), followed by the proposal system for small molecules and then a script to run it.

Currently brainstorming ways of efficiently traversing the molecular graph to decide the order of position assignment.

The simplest approach here is to repeatedly go through the list of torsions and see if you can place atoms 1 or 4 in the 1-2-3-4 torsion list.  You can place atom 4 if positions for 4 are undefined, but you have positions for atoms 1, 2, and 3.  Similarly, you can place tom 1 if positions for 1 are undefined, and atoms 2, 3, and 4 have defined positions.  There are only O(N) torsions, so this scales well.
You can later make this more efficient by pruning the list of torsions used for this purpose if neither atoms 1 or 4 have undefined positions.
You can get the list of torsions from the OpenMM System object, PeriodicTorsionForce object.

sounds good! Thanks!

I forgot to mention that this will not work the same way for improper torsion since the bond and angle definitions will be between different atoms. you can probably just ignore impropers, but if you need to, there should be a way to create an equivalent atom placement scheme for impropers.

Is this PR ready to be merged yet?

I need to fix the computation of the determinant of the jacobian here.

Ok, current status:

I couldn't really get sympy to do what I want.
I went back and looked at the code that uses autograd -- turns out I had made a mistake. I fixed that, and now autograd provides jacobians whose determinants are not 0. So, I think this is now working. I suppose the hairy question is now how to test it...

I suppose the hairy question is now how to test it...

There are a few tests that come to mind:

Checking invariants.  For example, J(r,theta,phi) should be independent of phi for any (r,theta)
Running a Monte Carlo simulation in (r,theta,phi) with a simple form of p(r,theta,phi) that should produce a uniform distribution in (x,y,z) within a sphere.  You might have to use something like p(r,theta,phi) \propto [r^2 sin(theta)] to get a uniform distribution.

Come to think of it, isn't the Jacobian determinant just J(r,theta,phi) = r^2 sin(theta)?

Ok, I can do those tests. The first in particular is pretty easy.

Come to think of it, isn't the Jacobian determinant just J(r,theta,phi) = r^2 sin(theta)?

So, the same as spherical coordinates? Hm, maybe.

Empirically it seems to be the same (albeit with the opposite sign)

Empirically it seems to be the same (albeit with the opposite sign)

The Jacobian determinant should definitely be positive.  r^2 sin(theta) should also be positive in the allowed domain theta \in [0,\pi].

Well, the acceptance criterion uses the absolute value of dett(J), no?
Point taken about the allowable domain of theta, though. I'll take a look.

Well, the acceptance criterion uses the absolute value of det(J), no?

Ah, good point!  So it doesn't matter.

I think maybe it'd be good to get this merged and implement tests in the next PR? I've also got a couple other components about ready for PRs (proposal, biasing engine), so it might help to clean up.

Sure, that works.
Travis is currently failing on some imports (numexpr and oechem).  That means we'll have to add these as dependencies, right?

oh, let me add those before merging then!

If you want to incorporate the openeye tools into testing, here is an example from openmoltools:

license file encryption: encryption travis changes (note that this should probably be done within the actual choderalab/perses checkout with a direct push because of the possibility for accidentally encrypting for your local branch instead)
Installation and testing: travis block
License file environment variable: travis block

Ok. I'll add that in after lunch and then merge when the tests pass.

I think the license file is actually already installed, but cannot be decrypted in a pull request. Otherwise, the openeye toolkits now install on travis.

Correct---the license can't be decrypted in a PR from your fork---only branches in this repo.  The only alternative is to skip those tests in PRs.
If it is ready to go, feel free to merge!


Sankaku not downloading

What steps will reproduce the problem?
Setting the source to Sankaku and trying to download an image in any way
What is the expected behavior? What do you get instead?
To download images
Problems every single time. No matter what I set the download folder to. No matter what I set the links to. There's always a problem
How often does this problem occur? On which sources?
All the time. Sankaku
What version of the program are you using? On what operating system?
6.0.3 windows 7
Please provide any additional information below
I'm not sure if its supposed to work yet, but if it is, then please tell me anything to try to try to make it work. Because there are images only Sankaku. And Sankaku usually has twice as many

Try changing user agent in Sankaku's source to Mozilla/4.78 [en] (Windows NT 5.0; U)

See https://github.com/Bionus/imgbrd-grabber/issues/1321

Problems every single time. No matter what I set the download folder to. No matter what I set the links to. There's always a problem

You should explain your "problems" in more details,; so that they can be fixed.


Option for local storage not working

Describe the bug
I've enabled the option for local storage and set the folderpath to "assets/{timestamp}_{filename}".
Then I clicked update and rebooted the container.
I then created a new note which worked without a problem.
I would have expected to find that new note saved as a markdown file inside the container under "/var/opt/memos" or under "/usr/local/memos/assets".
But I only found database files and in the assets folder only files with the ending .js .woff or .ttf``
Steps to reproduce

Run memos with command from docs
Login as Admin and go to settings
Go to storage and set it to "local file system"
Wonder where  your files are saved

The version of Memos you're using
v0.21.0
Screenshots or additional context
No response

You can use local storage to store your resources in your own server.

Reference docs: https://www.usememos.com/docs/advanced-settings/local-storage

You can use local storage to store your resources in your own server.

Reference docs: https://www.usememos.com/docs/advanced-settings/local-storage

Is this only applicable to resource files, not memos?

@Venkman42 All data including users/memos/.... should and will be stored in database(SQLite/MySQL/PostgreSQL). Only the binary of resources can be chose to upload into database/local file system/s3.

@Venkman42 All data including users/memos/.... should and will be stored in database(SQLite/MySQL/PostgreSQL). Only the binary of resources can be chose to upload into database/local file system/s3.

Oh okay, got it,thanks :) Maybe you could make this a little clearer in the docs. I get where you're coming from, but as a new user I just thought "resources" applies to everything, not just what you internally call resources


New PR for potjans connectome.

Break fixing change.
The file src/contrib/genPlasticWeights-erdos-renyi-potjans.jl
Doesn't look substantially different from default genPlasticWeights-erdos-renyi.jl but it has the a necessary change in its indexs:
    exc_selected = sort(exc_ordered[1:end])
    inh_selected = sort(inh_ordered[1:end])

The file genPotjansConnectivity.jl does away with SparseArrays now possibly making it more compatible with GPU code.
src/contrib/genPotjansConnectivity.jl
I typed most or all unresolved types of: Vectors, and dictionaries and I got a speed up.
I put the invocation to this new use of the plugin connectomes in the test directory.
I put the plugin files in a contrib directory.
Besides changing plugin paths in test/Potjansdata/init.jl
Init.jl needs to calculate cell size, to pre-allocate sizes that need to be known in other init.jl contexts, for the basic default code to work.
scale =1.0/30.0
function get_Ncell(scale=1.0::Float64)
	ccu = Dict{String, Int32}("23E"=>20683,
		    "4E"=>21915,
		    "5E"=>4850,
		    "6E"=>14395,
		    "6I"=>2948,
		    "23I"=>5834,
		    "5I"=>1065,
		    "4I"=>5479)
	ccu = Dict{String, Int32}((k,ceil(Int64,v*scale)) for (k,v) in pairs(ccu))
	Ncells = Int32(sum([i for i in values(ccu)])+1)
	Ne = Int32(sum([ccu["23E"],ccu["4E"],ccu["5E"],ccu["6E"]]))
    Ni = Int32(Ncells - Ne)
    Ncells, Ne, Ni, ccu

end
Ncells,Ne,Ni, ccu = get_Ncell(scale)

src/cpu/loop.jl
Line 240 of loop has to be this,
if post_ci!=0

for loop to tolerate connection matrices that are ragged (ie ragged arrays packed into a dense rectangle will look sparse, they will have empty, zero values).
Introducing that line is the easiest way I know to resolve the conflict.
I use a package called distributions too, to draw PRNGs that are drawn from a distribution.
I don't reverted an unintentional change to the file: src/init.jl. It still has a superficial formatting white space difference, but not a substantial difference.
I thought they where changes that you had made that got mixed in with my PR.
If that is true I could try to delete them.
Also I think this GitHub GUI has a squash merge option too.

If I run julia from the tests directory,
include("potconntest.jl")
After some time the algorithm converges on a correlation value of 0.96 and it does testing too.
Loop no. 100, task no. 1
correlation: 0.9610312565565572
elapsed time: 2.321516990661621 sec
firing rate: 33.96662786185487 Hz

trial #97, task #1: 1.46 sec
trial #98, task #1: 1.44 sec
trial #99, task #1: 1.38 sec
trial #100, task #1: 1.43 sec

the ragged adjacency matrices are now vectors of vectors.  to save memory for networks for which the number of presynaptic synapses is not uniform.
also, the params.jl file now needs a new variable init_code.
lastly, you shouldn't have to change the loop code.  why do you make sure that post_ci!=0?  that should never be the case.

"lastly, you shouldn't have to change the loop code. why do you make sure that post_ci!=0? that should never be the case." --at the time it was because presynaptic synapses where not uniform, and it was the only way to get the TrainSpikingNet.jl to evaluate post loading a non uniform presynaptic network.

I made some changes to make the weight matrix a ragged array format.
Still working on this it is a very slow burning PR.


feat(tekton): locked property support

PR summary
Add support for locked properties in Tekton pipelines and triggers.
Added miscellaneous updates copied from the node-sdk-template, and generated with a newer version of the SDK generator, hence there's a lot of other changes included
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

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


Module based aws_alb resource not recognized

Description :
I have a TF template which calls an ALB module. My tests should check that tags required for aws_alb resources (in addition to other required tags). These steps are being skipped stating it did not find any aws_alb resources.
To Reproduce

I am using terraform-compliance version 1.1.15

Feature: Resources should be properly tagged
  In order to keep track of resource ownership
  As engineers
  We'll enforce tagging on all resources

# Check required tag for ALB and CF
  Scenario: Ensure that waf_policy for ALB/CF tag is defined
    Given I have aws_alb resource defined
    When it contains tags
    Then it must contain waf_policy
    And its value must match the "^(internal|external|custom)$" regex

  Scenario: Ensure that waf_custom for ALB/CF tag is defined
    Given I have aws_alb resource defined
    When it contains tags
    Then it must contain waf_custom

Output:
 Scenario: Ensure that waf_policy for ALB/CF tag is defined
	* SKIPPING: Can not find aws_alb resource defined in target terraform plan.
        Given I have aws_alb resource defined
        When it contains tags
        Then it must contain waf_policy
        And its value must match the "^(internal|external|custom)$" regex

    Scenario: Ensure that waf_custom for ALB/CF tag is defined
	* SKIPPING: Can not find aws_alb resource defined in target terraform plan.
        Given I have aws_alb resource defined
        When it contains tags
        Then it must contain waf_custom

I am attaching my plan.json
plan.out.json.txt

Hi @eerkunt,
Hope you are doing well.  Any ideas on this one?

Having a look on this right now.

This is a weird problem, still couldn't fix this. I am taking a day off from work tomorrow and will focus on terraform-compliance.
Hoping to fix it tomorrow release(s)

Hi @anthonycolon25,
Looks like I was quite confused on this one :)
The sentence on GIVEN step is wrong.
Instead of ;
Given I have aws_alb resource defined

if you change this to ;
Given I have aws_alb defined

or
Given I have aws_alb resource configured

It should work :)

So sorry for that. I should have noticed that as well.
I changed the two scenarios above to
Given I have aws_alb defined
It now seems to find the aws_alb resource.  It is correctly passing the first test but failing on the second.  I do have a "waf_custom" property set to a value of "".   If I set a value for "waf_custom" then the test passes.
I would think the following test would pass, right?
Then it must contain waf_custom
Does that just check for the tag existence or both existence and non-empty value?
In my use-case, waf_custom would only have a value if waf_policy is set to "custom". Otherwise it would be just an empty string

It won't check a non-empty value. It will check if that key exists.
Sounds like waf_custom is not even initialised, thus key doesn't exist on an empty value.
Couldn't exactly understood the details of your use case, but sounds like you need filtering with ;
When its waf_custom is <something>

and then execute additional operations.

I'll give that a try.  Just to clarify the use case a bit more.

We have several required tags for all taggable resources (10 tags)
For alb resources there are two additional required tags (waf_policy and waf_custom)
Possible values for waf_policy are "internal", "external" and "custom".
If the waf_policy is set to "internal" or "external" then the value for waf_custom is not really relevant.  Most people just leave it as the default of ""
If the waf_policy is set to "custom", then waf_custom must hold the name/ID of the waf policy assigned.

I hope this helps clarify.

I looked at the plan.out.json and I see that "waf_custom" key does exist in the tags with an empty value.
I also looked into your recommendation above but that seems to only check the value when its non-empty.  It doesn't really help just check for key existence.

Hi @anthonycolon25 ,
I think you can utilise preconditions for achieving what you want.
For e.g. ;
Scenario: Precondition check for waf_policy value
        Given I have aws_alb defined
        When it has tags
        Then it must contain tags
        And it must contain waf_policy
        When its waf_policy is custom

    @precondition(test.feature: Precondition check for waf_policy value)
    Scenario: Ensure waf_custom is set to name/ID of the waf policy
        Given I have aws_alb defined
        When it has tags
        Then it must contain tags
        And it must contain waf_custom
        And its value must match the "ABC" regex

which the execution will look like ;
Scenario: Precondition check for waf_policy value
        Given I have aws_alb defined
        When it has tags
        Then it must contain tags
        And it must contain waf_policy
        When its waf_policy is internal

    @precondition(test.feature: Precondition check for waf_policy value)
    Scenario: Ensure waf_custom is set to name/ID of the waf policy
        Given I have aws_alb defined
        When it has tags
        Then it must contain tags
        And it must contain waf_policy
        When its waf_policy is internal
        Given I have aws_alb defined
        When it has tags
        Then it must contain tags
        And it must contain waf_custom
		Failure: waf_custom property in module.alb.aws_alb.alb resource does not match with ABC case insensitive regex. It is set to .
		Failure: waf_custom property in module.alb.aws_alb.alb resource does not match with ABC case insensitive regex. It is set to .
		Failure: waf_custom property in module.alb.aws_alb.alb resource does not match with ABC case insensitive regex. It is set to .
        And its value must match the "ABC" regex

Let me know if this will fix your issue.

In addition to @eerkunt's response, could you have a try with 1.2.5?
Your original features should pass now.

I have tested with 1.2.5 and the original feature now passes.  Thank you, @Kudbettin and @eerkunt .

Thanks @anthonycolon25 for re-testing it 🎉
Closing the issue as its solved. Please do not hesitate to create a new one if the problem (or a similar one) occurs.


first successful colcon build

The delta on this is suspiciously small for the number of commits.
the majority of these commits are merge commits, which are likely erroneous.
Recommendation: REBASE dev onto master and do not trust the "update" button.

Yea I haven’t merged this quite yet, cause there’s still some issues integrated the crates in using right now. I’ve resolved the merge commits, but there’s still some bugs that need to be sorted still. As such, there will be some additional commits in this PR before I merge it.


Auto-Login?

Is there a way to auto-login or skip having to create a password? Would you please just enter a nickname and go?

Hi @glecas , currently there is not support for auto-login in Khernet, however I will consider this feature for a future release.

I've been testing Khernet on my network, and it meets all my requirements, with the exception of auto-login. The necessity to log in each time one needs to communicate with another person is leading to ineffective communication because the other party often forgets to log in. IMHO, this poses a significant issue for a workplace network.

Hi, @Starbuck1991, indeed the auto-login feature is not supported right now but I will prioritize this request.


Fix declaration of get_miniscope_root_data_dir

These modifications don't require a version update.

These modifications don't require a version update.

Is not a patch release?

These modifications don't require a version update.

Is not a patch release?

Based on internal agreement, there is no requirement to release a new version for these changes.

These modifications don't require a version update.

Is not a patch release?

Based on internal agreement, there is no requirement to release a new version for these changes.

I haven't agreed to that. In the future please do one of the following:

update the version on any PR to be merged to the main branch - THIS IS SOP.
Explain why we're deviating from standard procedures in the PR description.
Only request reviews from reviewers with whom you've previously agreed to deviate from standard procedures to wrap up the PR as quickly as possible


protocol/request-response: Fix memory leak

Cancellation handles did not get removed after the request was handled successfully and would instead stay in the cancellation handle hashmap until the node was shut down.

Was this one responsible for 8GB memory usage by litep2p?

Sadly not, this was leaking about 8 bytes of memory per outbound request but I found this leak while I was hunting for the bigger leak. That memory is still hiding somewhere

But decided to make PR instead of pushing directly master because there's now two of us working on this repository

Yes, thanks. That's how the expertise on the codebase is built.


Adjust categories labels

Description
Wording adjustments on the new categories that will be visible after the classifier is enabled on Idunn.
(See also https://github.com/Qwant/idunn/pull/249 )

nice! I'm just not sure about Doctors office with an "s" ?

I will keep it this way for now, for consistency with "qwant-maps-common":
https://github.com/Qwant/erdapfel/blob/1ec6bda/local_modules/qwant-maps-common/src/place_category_name.js#L84


Wunderkind Saas Consent Connector

Closes https://github.com/ethyca/fidesplus/issues/517
Code Changes

[ ] Adds a new Wunderkind Consent Connector

Steps to Confirm

[x] In your test env privacy center, mark Data Sales or Sharing as the consent preference that is executable (the config.json file). Right now this is the only consent preference that can be persisted to third party services.

[x] Run nox -s "fides_env(dev)"

[x] Go to the admin ui  Add a New Connector and search for Wunderkind - it is a saas connector

[x] Main required parameter is a website_id - we don't have one of these, so I am just making one up, like test_wunderkind_website_id

[x] In the privacy center, modify a test user's consent preferences<EMAIL_ADDRESS>and opt in or out of data sales or sharing
[x] Return to the admin app and approve the associated privacy request
[x] Verify that the privacy request completes.  Verify execution logs, and that there's a single log for wunderkind
[x] Verify the API request made in the logs to wunderkind. Verify you get the correct query parameter depending on whether you opted in or opted out - see "DESCRIPTION OF CHANGES" below

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[x] Update CHANGELOG.md

Description Of Changes
Note that consent connectors aren't yet linked to specific data uses.  This connector sets a consent value of "1YN" if the user is opting in, and "1YY" if the user is opting out.  To link this to Data Sales/Data sharing, the customer should set up their privacy center to have that particular data use be executable.

If user has opted out of Data Sales/Data Sharing then "1YY"
If user has opted in out of Data Sales/Data Sharing then "1YN"

@allisonking do you know why the privacy ceneter cypress test might be failing?

@allisonking do you know why the privacy ceneter cypress test might be failing?

nope they've been failing for a few days now. sometimes it goes away though... I can never repro locally https://ethyca.slack.com/archives/CBKK8TS74/p1675902294810199

ok no problem @allisonking thanks for weighing in

Thank you for the help in review @pattisdr ! Everything looks great, the only other question I had was around some of the documentation that will go around setting some of these items up properly in the config.json - is there a separate issue on the docs repo for that already?

Good question, I have some a draft of docs on the docs site that Sai will polishhttps://github.com/ethyca/fidesdocs/pull/35#issuecomment-1430902122.
This is supposed to be temporary too until we have a proper mapping from data uses -> which requests fire.  That's why we currently restrict to one because no mapping exists.

Remaining saas failure is longstanding: test_friendbuy_nextgen_erasure_request_task, as is the ctl-external failure: tests/ctl/core/test_system.py::test_scan_system_aws_passes.  Note that these tests aren't usually run and only run here because I added the "run unsafe ci checks".


pkg/trace/api: Fix tracer flare proxy host

What does this PR do?
Use AgentConfig.Site as the base hostname
Update subdomain logic:

Only apply app. when we detect one of a set of datadog domains
Prefixes the domain with the agent version

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Skipqa applied as this is a fix found during QA of 7.50.0.

/merge


calling "local" api endpoints from within a +page.ts causes hooks.handle to be executed twice

Describe the bug
I actually detected this behaviour because i use trpc-sveltekit in my project.
trpc creates a route under the app (/trpc), which is responsible to handle every procedure call.
when running a procedure from inside any +page.ts, the hooks.handle gets executed twice (one for the actual +page.svelte, and other to the /trpc/{procedureName}.
this becomes a problem because in my hooks.handle, i make a db call to get the current user based on cookies. hence, every "user navigation" causes 2 db calls to get the user.
this started happening only after this major update (the one that added the + prefix to everything.
Reproduction
start a new sveltekit project and create the following files:
// src/routes/api/+server.ts
import type { RequestHandler } from './$types';

export const GET: RequestHandler = async () => {
	return new Response(
		JSON.stringify({
			hello: 'world'
		})
	);
};

// src/routes/+page.ts
import type { PageLoad } from './$types';

export const load: PageLoad = async ({ fetch }) => {
	const r = await fetch('/api');

	return await r.json();
};

// src/hooks.ts
import type { Handle } from '@sveltejs/kit';

export const handle: Handle = async ({ event, resolve }) => {
	console.log(event.url.pathname);

	const response = await resolve(event);
	return response;
};

// routes/+layout.server.ts
import type { LayoutServerLoad } from './$types';

export const load: LayoutServerLoad = async () => {
	return {
		layoutLoad: 'layoutLoad'
	};
};

// src/routes/+layout.svelte
<slot />

// src/routes/+page.svelte
<script lang="ts">
	import type { PageData } from './$types';

	export let data: PageData;

	console.log(data);
</script>

then, access `http://<IP_ADDRESS>:5173/some-page
your editor console should log the following:
/some-page
/api

here is a repro for reproduction
Logs
No response
System Info
System:
    OS: Linux 5.18 Pop!_OS 22.04 LTS
    CPU: (12) x64 Intel(R) Core(TM) i5-10400 CPU @ 2.90GHz
    Memory: 10.38 GB / 15.54 GB
    Container: Yes
    Shell: 5.8.1 - /usr/bin/zsh
  Binaries:
    Node: 16.15.0 - ~/.nvm/versions/node/v16.15.0/bin/node
    npm: 8.13.2 - ~/.nvm/versions/node/v16.15.0/bin/npm
  Browsers:
    Chrome: 104.0.5112.79
    Firefox: 103.0
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.65
    @sveltejs/kit: next => 1.0.0-next.431
    svelte: ^3.44.0 => 3.49.0
    vite: ^3.0.4 => 3.0.9

Severity
annoyance
Additional Information
No response

I don't think there's any bug here. handle runs on every request to the server; you're making two requests to the server.

This is the intended behavior. And it definitely is not new with the routing rework.
Calls to internal endpoints made from the browser are real, requests coming into your server, and so handle is called for them. Calls to internal endpoints made from the server during SSR also call handle again (although this time with a constructed event object without there being an actual new HTTP request) for consistency with the browser behavior.


 Do you have a go-to method that you can recommend? I couldn't do it with pickle and joblib. So I tried pickling the clause banks and weight banks separately but that seems to work only if I pickle them before the classifier is used for any prediction which in my case is not an option.
I'm attaching a minimal example just in case but I imagine there must be another way to do it that I haven't figured out.
saving.py
import pickle
from tmu.models.classification.vanilla_classifier import TMClassifier
from tmu.data import TMUDatasetSource

data = TMUDatasetSource().get_dataset(
        "XOR_biased",
        cache=True,
        cache_max_age=1,
        features=["X", "Y"],
        labels=["xor"],
        shuffle=True,
        train_ratio=1000,
        test_ratio=1000,
        return_type=dict
    )

tm = TMClassifier(20,10,2)
tm.fit(data["x_train"], data["y_train"])

# This method works only if these two lines are commented out:
# y_pred = tm.predict(data["x_test"])
# print((y_pred == data["y_test"]).mean())

for i in range(len(tm.weight_banks)):
    with open('wb%s.pkl' % i, 'wb') as f:
        pickle.dump(tm.weight_banks[i], f)
for i in range(len(tm.clause_banks)):
    with open('cb%s.pkl' % i, 'wb') as f:
        pickle.dump(tm.clause_banks[i], f)

loading.py
from tmu.models.classification.vanilla_classifier import TMClassifier
from tmu.data import TMUDatasetSource
import os
import pickle

data = TMUDatasetSource().get_dataset(
        "XOR_biased",
        cache=True,
        cache_max_age=1,
        features=["X", "Y"],
        labels=["xor"],
        shuffle=True,
        train_ratio=1000,
        test_ratio=1000,
        return_type=dict
    )

tm_new = TMClassifier(20,10,2)
tm_new.clause_banks = []
tm_new.number_of_classes = 2
tm_new.weight_banks = []

files = os.listdir('/home/folder/')
cb_count = sum(1 for file in files if file.startswith('cb'))
wb_count = sum(1 for file in files if file.startswith('wb'))

for i in range(wb_count):
    with open('wb%s.pkl' % i, 'rb') as f:
        wb = pickle.load(f)
        tm_new.weight_banks.append(wb)
for i in range(cb_count):
    with open('cb%s.pkl' % i, 'rb') as f:
        cb = pickle.load(f)
        tm_new.clause_banks.append(cb)

y_pred_new = tm_new.predict(data["x_test"])
print((y_pred_new == data["y_test"]).mean())

The error that I get when I first predict something and then save the clause and weight banks is the following:
Traceback (most recent call last):
  File "loading.py", line 36, in <module>
    y_pred_new = tm_new.predict(data["x_test"])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/envs/tmtest/lib/python3.11/site-packages/tmu/models/classification/vanilla_classifier.py", line 311, in predict
    self.clause_banks[i].calculate_clause_outputs_predict(self.encoded_X_test,
  File "/home/envs/tmtest/lib/python3.11/site-packages/tmu/clause_bank/clause_bank.py", line 168, in calculate_clause_outputs_predict
    self.lcm_p,
    ^^^^^^^^^^
AttributeError: 'ClauseBank' object has no attribute 'lcm_p'. Did you mean: 'lcc_p'?

Thanks for the help!

Dear @anayurg,
May I suggest just loading the whole TM directly?
saving:
import pickle
from tmu.models.classification.vanilla_classifier import TMClassifier
from tmu.data.tmu_datasource import TMUDatasetSource

data = TMUDatasetSource().get_dataset(
    "XOR_biased",
    cache=True,
    cache_max_age=1,
    features=["X", "Y"],
    labels=["xor"],
    shuffle=True,
    train_ratio=1000,
    test_ratio=1000,
    return_type=dict
)

tm = TMClassifier(20,10,2)
tm.fit(data["x_train"], data["y_train"])

with open('tm.pkl', 'wb') as f:
    pickle.dump(tm, f)

loading:
from tmu.models.classification.vanilla_classifier import TMClassifier
from tmu.data.tmu_datasource import TMUDatasetSource
import os
import pickle

data = TMUDatasetSource().get_dataset(
    "XOR_biased",
    cache=True,
    cache_max_age=1,
    features=["X", "Y"],
    labels=["xor"],
    shuffle=True,
    train_ratio=1000,
    test_ratio=1000,
    return_type=dict
)

with open('tm.pkl', 'rb') as f:
    tm_new = pickle.load(f)

y_pred_new = tm_new.predict(data["x_test"])
print((y_pred_new == data["y_test"]).mean())

I've also corrected
AttributeError: 'ClauseBank' object has no attribute 'lcm_p'. Did you mean: 'lcc_p'?
Thanks for finding this :)


Update for bevy 0.11

Hey, as bevy 0.11 just released I figured I might contribute a little update to this plugin!
Take it with a grain of salt through because I'm unsure about this part:
 .add_systems(
                    Update,
                    framerate_limiter
                        .in_set(RenderSet::Cleanup)
                        .after(World::clear_entities)

Hi! I have also done this migration, but I have found a big issue: the plugins is no longer functional.
If you launch the demo example you will see that the reported oversleep and frametime is constantly 0.0. Using bevy's built in diagnostics it also reports no change in framerate.
Somethings has been changed in how bevy functions.
Also in demo.rs: "PipelinedRendering" is now only part of "DefaultPlugins" if the "multi-threaded" feature is enabled, so it can be removed from the build function. (or it will not launch)

I think I've fixed these issues after testing locally.


mongodb::globals value for group param being ignored

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: Enterprise 2017.2
Ruby: ruby 2.1.9p490 (2016-03-30 revision 54437) [x86_64-linux]
Distribution: Red Hat Enterprise Linux Server release 7.6 (Maipo)
Module version: 2.4.1

How to reproduce (e.g Puppet code you use)
class { 'mongodb::globals' :
  manage_package_repo => true,
  version             => '3.6.12-1.el7',
  group               => 'mongodg',
} -> class { 'mongodb::server':
  ensure              => present,
  dbpath              => '/data/mongo/db',
  dbpath_fix          => true,
  pidfilepath         => '/var/run/mongodb/mongod.pid',
  logpath             => '/data/mongo/log/mongod.log',
  logappend           => true,
  fork                => true,
  port                => 27017,
  journal             => true,

}
What are you seeing
Systemd unit file being created with default group instead of my overridden value specified in the globals settings causing service to fail at startup:
[Service]
User=mongod
Group=mongod
Environment="OPTIONS=-f /etc/mongod.conf"
ExecStart=/usr/bin/mongod $OPTIONS
ExecStartPre=/usr/bin/mkdir -p /var/run/mongodb
ExecStartPre=/usr/bin/chown mongod:mongod /var/run/mongodb
(Note both the "Group=" line and the "ExecStartPre=" line don't specify the group I set in the globals.
What behaviour did you expect instead
That is would use my specified group name instead of the default.
Output log
Any additional information you'd like to impart
If I manually update the systemd unit file to specify the correct group, the next agent run succeeds and mongod is started.

any update on this one?


stable/velero: update readme to use stable repository in install command

Signed-off-by: Adnan Abdulhussein<EMAIL_ADDRESS>

/lgtm


Feature request: Read age parameters form environment variables

Hello,
I have to encrypt to various, often changing public keys. It is a bit tedious to have to edit the neovim configuration every time or use the commands, especially if you want to script the editing.
Would it be possible to read the age key location and the recipients from environment variables if set? (falling back to the current behaviour)

Hi @pinpox! I'm glad I'm not the only one who found it useful.
Please try it with
use {
    'histrio/age-secret.nvim', branch = '1-feature-request-read-age-parameters-form-environment-variables' }
}

I'll merge it if the implementation is ok for you.