weave forget [peers] didnot work when two peers have established connections.

i have use "weave connect peerB" on peerA, and connection established.
then i use "weave forget peerB" on peerA, the connection still there, is it right?
weave script 1.5.0
weave router 1.5.0
OS: centos-7 kerenel: 3.10.0-229.4.2.el7.x86_64

The command is called forget, not disconnect, for a reason. The docs say that it

prevents the peer from trying to reconnect to that host once connectivity to it is lost

Is there a specific reason for you wanting to force a disconnection?

Thanks for your reply.
yes, i want to disconnect a peer manually, and i think its a reasonable operation when i don't want to connect to a peer.
i cannot find a option to do that, and weave connect is along with weave forget, so i try it.

Please elaborate on what you are trying to achieve. If you want to remove a peer from a weave network, then simply stop it. And then run weave forget on any peers on which you previously ran weave connect.
In general, connections between peers can be established in either direction, and peers discover others automatically. Therefore when telling peer A to disconnect from B peer, it is quite likely that peer B would immediately establish a connection the other way, or peer A would attempt to reconnect to B because it learnt about A from other peers (rather than having been told in weave connect).

i know now all weave peers keep the same topology(connection information), so there is no use to cut one connection when there are other connections between two peers.
my achieve is very simple, just cut the connection to peerB on peerA, meanwhile peerB is still running and connect to other peers, such as peerE. it is impossible?
for example: peerA <-> peerB, peerA<->peerE, peerB<->peerC, peerB<->peerD. which <-> represent connection between two peers. now i want to cut the connection between A and B.
weave forget prevents reconnection, but not to cut connection.

weave forget prevents reconnection

It does not. When you run weave forget B on A, then A could still find out about B from other peers and hence attempt to connect to it.

The use case weave connect and weave forget are designed to handle is adding a peer to a weave network when the existing peers are behind a firewall.
It is not clear what use case you have in mind. Splitting a weave network?

thanks. got your point.
actually, i am just curious about these weave operations, and there is no concrete use case except the example previous which sounds like a splitting network.


Added eps attribute to FrozenBatchNorm2d

This PR aims at tackling #2169 by:

adding a eps attribute to  ̀FrozenBatchNorm2d`
renaming the n positional argument of  FrozenBatchNorm2d constructor to num_features
adding a unittest to ensure default eps is zero for backward compatibility
adding a unittest to ensure valid computation when epsilon is higher than zero by comparison with BatchNorm2d forward in eval mode.

@fmassa I'm not sure where the error comes from, it seems like the tests timed out in test_ops at NMSTester. Do you think it could be due to this PR?

Also, I took two choices here that any reviewer might want to take a look at:

renaming the constructor argument n to get closer to BatchNorm2d interface (I didn't see many reasons for someone to use kwargs, but it might be worth mentioning it as a breaking changes if merged into the next release)

rearranging the forward implementation by switching reshape to view (I might have missed a use case where it's needed), and delaying this reshape/view to the last moment to reduce the LoC. I made tests to ensure the computation is the exact same as previously, and I also checked execution time on both CPU & GPU which were the same.

Also, I made two choices here that any reviewer might want to take a look at:

renaming the constructor argument n to get closer to BatchNorm2d interface (I didn't see many reasons for someone to use kwargs, but it might be worth mentioning it as a breaking changes if merged into the next release)
rearranging the forward implementation by switching reshape to view (I might have missed a use case where it's needed), and delaying this reshape/view to the last moment to reduce the LoC. I made tests to ensure the computation is the exact same as previously, and I also checked execution time on both CPU & GPU which were the same.

For the first option, I'm not sure if it introduces a BC breaking because it just re-defines the name of the argument.

Test failures seem related

Thanks, I fixed those. There is a timeout remaining for windows builds and an unrelated (I think) failure on travis!


Overriding error message  for X-Prefixed Rules.

First of all, thanks  for this great  library.
I  am wondering if there is a way to define custom error messages for X-Prefixed Rules?  E.G.
const customLang = createLang({
  ...en,
  xRules: {
    validKey: 'Invalid key',
    strongPass: 'Password must be at least 8 characters long, contain at least one uppercase letter, one lowercase letter, one number and one special character'
  }
})
// Create an instance of Validator for the container element
const v = new Validator(registerForm, {
  lang: customLang,
  onFieldChangeValidation: true,
  xRules: {
    validKey: /^[a-zA-Z0-9-]+$/,
    strongPass: /^(?=.*[a-z])(?=.*[A-Z])(?=.*\d)(?=.*[@$!%*?&#])[A-Za-z\d@$!%*?&#]{8,}$/

  }
})

Thanks.

Hey @g5becks. Currently not unfortunately. But it's indeed a great idea. Feel free to make a PR, otherwise I'll implement it in due course.

Hey @g5becks. This was added in v1.12.0:
https://github.com/upjs/facile-validator#custom-error-messages


Starknet ID domains integration

Feature Request
Hello !
I'm Ben cofounder of Starknet ID. We would love to add Starknet ID resolving functions to starknet PY as we did with the other Starknet Libraries.
Here is some explanation https://github.com/starknet-id/naming_contract/issues/8
Tell me if it's ok and a member of our team will make this implementation !

Hey, let me discuss this with the rest of the team and will get back to you 👍

Hey Did you have some time to discuss about it ?

Hey, could you provide us with a more in depth plan on how integration like that would look like i.e. what methods would be added, where and how would they be integrated?
After an internal discussion we've had, we'd prefer not to included functionalities that will propagate through the library requiring us to support and maintain them in multiple places.


refactor: create communication journey in sms status webhook

closes #260
fixes PROD-SWC-WEB-4PP; PROD-SWC-WEB-4WW
What changed? Why?
Currently, instead of creating the communication journey and user records after sending a batch of messages, we're creating them in the status webhook. This prevents the webhook from receiving calls for messages that haven't yet been registered in our database.
How has it been tested?

[ ] Locally
[x] Vercel Preview Branch
[x] Unit test
[ ] Functional test

Since this will perform worse than what we have today, we’ll remove all calls to the user_communication table and pass campaign_name and journey_type via URL parameters instead. This will be done in #1553


Suggestion 2

Is your request related to a problem? Please describe.
Less practical in inputting the available balance quickly
Describe the solution you'd like
Need to add a percentage-based input balance option, especially on the swap feature
Describe alternatives you've considered
you also need to add an input balance based on prices at the dollar exchange rate, this can be a differentiator from other apps
My Wallet : 0x8B011EB29eC73aF3FF72F492ca2E778723969a2A

@CesarK2604 Thank you for your suggestions, we've taken your request and noted it within our potential enhancement document list!


Improve Boost compatibility check

string test was starting to annoy me as it was the same path, just had an extra backslash, etc.

LGTM, thanks!


Bug: celestial bodies wrong degrees

Let's take following origin of mine as an example:
{
    "year": "1998",
    "month": "07",
    "date": "12",
    "hour": "08",
    "minute": "20",
    "latitude": "44.95719",
    "longitude": "34.11079",
    "timezone": {
        "name": "Europe/Simferopol",
    }
}

Then, following property output value: 19° 18' 48''
_celestialBodies.all[0].ChartPosition.Ecliptic.ArcDegreesFormatted30

But expected value: 19° 40' 44.4"

When i use this origin and don't specify the timezone, i get the right result for Sun of 19° 40' 43".


Stop downloading fallback images

... since it's wasteful and doesn't allow us to use a fallback image that clearly identifies extensions with no data.

Ignoring build failures, which are caused by more extensions having dead docs links, and merging.


Best way to limit memory usage?

Is there a good way to set a hard limit on the memory usage of bolt? Going to be using it embedded on a user-spacey application and can't allow it to eat up all resources if a large record comes along...

I was looking at the same problem and it seems that you have to modify
bolt_platform.go (for example bolt_arm.go) and modify maxMapSize to the maximum size you want. There is no option to change, only options.InitialMmapSize which is nice, but not the same :)

Dang. Maybe something I can add config control for and pr soon. Thx anyways


[FEATURE] Update 1-click release process to publish shadow jar as well

Is your feature request related to a problem?
Currently 1-click release process uploads artifacts to maven only. It should upload a signed shadow jar to artifact bucket as well.
What solution would you like?
Add uploading a signed shadow jar to artifact bucket as well.
What alternatives have you considered?
N/A
Do you have any additional context?
Refer to https://github.com/opensearch-project/opensearch-build/issues/3614 and https://github.com/opensearch-project/sql-odbc/pull/52

waiting on https://github.com/opensearch-project/sql-jdbc/pull/106 to complete


testing if CI is broken

I believe CI is currently broken on julia master.

reproduces on my local machine also

So do you mean it is related to world age?

I suspect so. But the MethodError message doesn't say it exists in a newer world. So perhaps not


how generate IDEWorkspaceChecks.plist

when I remove xcodeproject file then generate one, there is a file named IDEWorkspaceChecks.plist lost, what should I do to fix this?

At the end of my script, I add
git checkout -f Example.xcworkspace/xcshareddata/IDEWorkspaceChecks.plist


[JornalNBridge] getName() fix

Simplified the way I got the feed name. I was being dumb and not reading the documentation properly, so I overlooked the existance of $this->getKey() which solved the issue.

no sure why prtester failed @Bockiii

no sure why prtester failed @Bockiii

added a fix. problem was in the combination of an optgroup and normal options. new pr should fix that.

Ok thanks!


Create a new main.menu

I'm trying to create a new item in the [[menu.main]], say About Me.
I tried to do something like:
[[menu.main]]
--
   url = "/about_me/"
   name = "About Me"
   weight = 4

Then I've created a directory in my content directory and it works.
However it gets the same structure as posts and project.
I would like to do something like by cliking in the About Me menu, just show a text with the description (written in .md) and not a post like page.
Is that possible to do something like that?
Here´s an example.
thank you

He explains here


 Note: there are basically three cases to test for each task: no existing output, fail due to existing output and overwrite=False, actually overwrite existing output. The first one is the one we are already testing, the second one should be somewhat easy (every time we run a task for the first time, we also re-run it right away with overwrite=False and catch the error).
[x] Maybe remove some of the new tests to make the CI a bit faster? Fixed with #504
[x] Add unit tests for prepare_label_group (mostly similar to the ones for write_table)
[x] Review manifest updates
[x] Fix error message in MIP task
[x] If/when we remove the contents of a zarr group through the zarr API, let's wrap this call in a small function that also logs what we are removing.
[x] Rename lib_zarr, or move its functions to more appropriate modules.

This PR is essentially ready on my side, and any feedback is welcome.
@jluethi could you go through it when you have time?
The PR is independent from the registration tasks introduced in #487, and the two branches have no conflict at the moment. I'd propose to merge this one first, and then address the overwrite-related features to the registration tasks somewhere else (either as part of #487 or as another PR about #498).
One last (minor) point to fix in this PR is maybe the code organization. Right now I added a broad-scope lib_zarr.py module, with these four functions:
open_zarr_group_with_overwrite
_write_elem_with_overwrite
write_table
prepare_label_group

I think this code organization is not ideal. Some possible improvements:

Keep this module as is, and rename it to something more general (like lib_writing).
Move ROI-related functions (i.e. the ones using anndata) into another module (lib_regions_of_interest.py, or even a new one).

@tcompa That's awesome! I'll try to review as soon as possible. But may be Friday afternoon or Monday next week given the current conference

@tcompa That's awesome! I'll try to review as soon as possible. But may be Friday afternoon or Monday next week given the current conference

No problem. The two branches at the moment have no conflicts (see https://github.com/fractal-analytics-platform/fractal-tasks-core/compare/registration_task...491-introduceupdate-overwrite-parameter-to-most-tasks), and they are both at a quite advanced stage - thus I think that extra care in the merging order is not needed.

Regarding the naming of lib_zarr: I agree, that's a bit too general. I'd be fine with lib_writing or any better ideas you have :) Could be lib_zarr_writing?

I went with lib_write, inspired by the (somewhat similar in scope) https://github.com/ome/ome-zarr-py/blob/master/ome_zarr/writer.py module.

In the future we may want to review the structure of the core library, and introduce some more consistent module naming and/or folder organization. We may also export a bunch of lib_* functions directly to the main namespace, so that they become available as from fractal_tasks_core import some_function (instead of fractal_tasks_core.lib_something import some_function).

I found one small issue in the illumination correction task

The parameter name for zarr.create was indeed wrong.
I think the parameter of build_pyramid must be always set to True, in the current version.

Side comment: this task includes a few unreachable code paths which cannot be tested, we'll need to review everything once we support overwrite_input=False.

and a few small docstring TBDs to be replaced.

Fixed, thanks!

This is ready to merge, as soon as we agree on the overwrite parameter in build_pyramid.

Agreed on the build_pyramid overwrite, we can merge! :)


React message in console in production mode.

Question
I am getting this message in production mode.

Download the React DevTools for a better development experience: https://fb.me/react-devtools

It should not appear in production mode, only in dev. Anyone know what may be going wrong?

I've tracked down to react-google-ad module. But oddly their Githup repo is kind of ghost town.  Whats the problem with this code that my trigger that message?
var React = require('react')

module.exports = React.createClass({
	propTypes: {
		height: React.PropTypes.string,
		width: React.PropTypes.string,
		client: React.PropTypes.string.isRequired,
		slot: React.PropTypes.string.isRequired,
		format: React.PropTypes.string.isRequired
	},

	getDefaultProps: function() {
		return {
			height: "100%",
			width: "100%"
		}
	},

	render: function() {
		return (

		)
	},

	componentDidMount: function() {
		if(typeof window !== 'undefined') {
			(window.adsbygoogle = window.adsbygoogle || []).push({});
		}
	}
})

Well, I've wrote the code myself and got no message this time:
import React, { Component, PropTypes } from 'react';

class GoogleAds extends Component {
    render() {
        return (

        )
    }

    componentDidMount() {
        if(typeof window !== 'undefined') {
            (window.adsbygoogle = window.adsbygoogle || []).push({});
        }
    }
}

GoogleAds.propTypes = {
    client: PropTypes.string.isRequired,
    slot: PropTypes.string.isRequired,
    format: PropTypes.string.isRequired
};

export default GoogleAds;

Closing Issue. Still curiose why the old code throw that message.

react-google-ad was using old version of react (0.14.7) - it was possibly loading that old version and causing the warning


feat(clerk-js): Ability to enforce password reset from OAuth callback

Description
This PR adds support for handling cases where a user is required to reset their password following an OAuth callback. In such scenarios, navigational options are hidden to enforce the password reset. Additionally, the option to "Sign out of all other devices" is hidden, as this will be the default functionality provided by our backend.
https://github.com/clerkinc/javascript/assets/684680/69454c2a-3052-49ca-857f-67a9bae69f3a
Checklist

[X] npm test runs as expected.
[X] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[X] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[X] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[X] @clerk/types
[ ] @clerk/themes
[X] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Copy/paste for key/value fields doesn't work

Steps:

Go to create deployment
Go to any Key/Value fields (Pod labels, Labels & Annotations, Networking tab has a couple)
Paste in text that has
foo=bar
foo1=bar1
foo2:bar2
foo3= bar3
foo4 = bar4
foo5: bar5
foo6 : bar6
foo7:
foo8=
a=b

Results:  All of the text is put in the Key value of first field.  It should create a new row for each of the fields
Current:

Should be:

Verified fixed


Read-Only vs Read/Write API

Is there a reason this API will only read and not update data on Mint?  I'm looking for an API that can update Mint transactions.

This is probably not the API for you, unless you send along a PR for it! :)

This is probably not the API for you, unless you send along a PR for it! :)

thanks mrooney!  i would be glad to make a PR for that functionality or just add it independently to my app.  I was just wondering if there was a technical or legal blocker that was preventing this functionality to be added as of yet.

thanks mrooney!  i would be glad to make a PR for that functionality or just add it independently to my app.  I was just wondering if there was a technical or legal blocker that was preventing this functionality to be added as of yet.


About Us Page

Goals

Have an about us page with card-style layout for each team member with short bios/pics
Links from each lesson page's author heading to their respective cards
Either we can include in mid-bottom section of main page or have an different page entirely

moved to #36


Id is not flattened

$ cat in.haml
%span{id: ['foo', ['bar']] }

$ haml in.haml

$ faml render in.haml

:dizzy_face:

Flattening is clearly documented, but I didn't know that.
http://haml.info/docs/yardoc/file.REFERENCE.html#class-and-id-attributes

The array will first be flattened and any elements that do not test as true will be removed.


Implement a progess bar for processing

For the ObsSurface.read_file function have a simple progress bar, see https://github.com/tqdm/tqdm

This feature was added by 6a9a7377b1f758bc36e2d2dd115fd4c4e9851333


Historical curiosity: Why did we delete half the spec, again?

I cannot for the life of me remember why this happened: https://github.com/opencontainers/distribution-spec/commit/c90b0f145ac6bc09d2636ee214486ac333edc284
There is a ton of stuff missing from distribution-spec, still.

It did improve readability. The spec carried over from Docker had a lot of generated cruft.
But it was a huge endeavor. I wouldn't doubt that things were missed.
What particularly are you finding missing?

I, for one, am missing the historical context and decision making behind many of the nontrivial spec changes, such as
- Actionable failure conditions, covered in detail in their relevant sections, are reported as part of 4xx responses, in a json response body.
- One or more errors will be returned in the following format:
+ A `4XX` response code from the registry MAY return a body in any format. If the response body is in JSON format, it MUST
+ have the following format:

and how Content-Type: application/octet-stream became a required request header when PATCHing blobs. Before #178 the spec included a table of headers and parameters that SHOULD be included, which did not include the Content-Type header. The Content-Type header was only mentioned in an illustrative example.
Also:

At one point we deprecated catalog, but I can't seem to find reference to it anywhere after #178

We also define a chunked upload, but not the streaming upload for blobs.

On 21/07/23 07:41 -0700, Jon Johnson wrote:

I can't find any reference to the pagination mechanism for tag listing. I think this got dropped when catalog was [re]moved?

open a PR to add it back! pagination was useful, and if folks have
already implemented it, then we ought not introduce new and different
ways of doing it.


simulator updates

update the net profit scale to start by 0.01 to reflect the reality

token price after 5 years = 1$ as default (if not already set)

changes to max token price should reflect on the graph x-axis

start at 0.01, end at max token price and choose arbitrary 10 points between

@maayarosama please check if this has been done on the new simulator

As it's not merged to the development branch in tf-farming-calculator, it's not deployed on the new simulator. I'll do it tomorrow.

start at 0.01, end at max token price and choose arbitrary 10 points between

start by 0.1


Javascript testing framework

This adds testing infrastructure using karma and mocha.  It also tests the jsonrpc function parsing, mostly as a proof of concept.
I removed babel transpiling for everything but es6 modules because of #54.  This means that PhantomJS won't work.  At the moment, it defaults to testing chrome and firefox, but you can configure the browsers it starts up with npm run test -- --browsers="Firefox,Safari".  You can also just run npm run test:debug and open your browser of choice to http://localhost:9876/debug.html.
It is intentionally set up like ipywidgets.  Though the javascript testing isn't comprehensive in ipywidgets, it does set up a mock communication layer that we might consider emulating to get descent coverage.

Current coverage is 38.35% (diff: 100%)

No coverage report found for master at 8291a5c.

Powered by Codecov. Last update 8291a5c...a4c6df8


Make Keycloak cluster size configurable

Closes #356
ps. there is no cluster online so I haven't tested it yet. Will do it tomorrow.

Ready for review :+1:


Can't try it on hg?

I can't run it on HG because my GPU quota is insufficient. This has not happened with other demo tests.

thanks for your interest, i am fixing the bug with huggingface


[SHOWCASE_REQUEST]

Please read the instructions here first:
https://github.com/Lissy93/dashy/blob/master/docs/showcase.md#submitting-your-dashboard
Complete the Following

Title of Dashboard: NAS Home Dashboard
Link to Screenshot: https://ibb.co/4tgBj8g
Would you like your name/ username included: Yes
Link to your Website/ Profile/ Twitter (optional)
Description (optional)

Thank you 😊
Added Here 👈

#74


Example code doesn't work

The example code in the README doesn't cover all the required steps to get a Chatto implementation working. I think it would be good to have a minimal implementation to help people get started, and ChattoApp (inside the Chatto repo) doesn't seem to be minimal.
I had to copy a whole lot of things from the ChattoApp in order to get the example code to work. Here's a list of some of the errors.

TextMessageHandler is not defined
TextMessageModel.chatItemType is not defined
FakeMessageSender is not defined
ViewController' has no member 'baseMessageHandler'
ViewController' has no member 'configureChatInputBar': removed
ViewController' has no member 'chatInputPresenter': removed

Solution to get it to build and run was as follows:

Copied FakeDataSource.swift SlidingDatasSource.swift Source/FakeMessageFactory.swift into my project.
Added the following to configure the messageSender and dataSource and baseMessageHandler:

    var messageSender: FakeMessageSender!
    var dataSource: FakeDataSource! {
        didSet {
            self.chatDataSource = self.dataSource
        }
    }
    private var baseMessageHandler: BaseMessageHandler

    required init?(coder aDecoder: NSCoder) {
        dataSource = FakeDataSource(count: 0, pageSize: 0)
        messageSender = dataSource.messageSender
        baseMessageHandler = BaseMessageHandler(messageSender: messageSender)
        super.init(coder: aDecoder)
    }

It was never the intention to provide a ready-to-compile code in the readme, but just an overview explaining the basics on how things work. Anyway, we take your feedback on board for future updates. Thank you!

A simplified example would be very appreciated. I looking at your example and have to figure out what to implement to make it work for me.

yeah agree with @ogezue please explain everything properly.

The example is kind of convoluted and confusing, a minimal usable example would really be great.

The chatto examples are so complex with multiple layers of inheritance. Thumbs down to this library.

Personally, I like the example, it helped me to be up and running in no time.


Error when upgrading to v19.0.0

when we upgrade to v19.0.0, the program prompts "failed to load latest version: version of store crisis mismatch root store's version; expected 4284158 got 0; new stores should be added using StoreUpgrades. "Does the data need to be resynchronized?

You cannot upgrade/run v19.0.0 until the upgrade height specified in the upgrade plan has been reached. Otherwise the required data migrations will not be run.


Issue with pj2 PRG2BIN in  section when path has parentheses - ")"

I am getting this error on converting some of my pj2 files with PRG2BIN:
2016-07-10T13:51:44  ----------------------------------------------------------------------------------------------------
Error 36, Command contains unrecognized phrase/keyword.
analyzecodeblock_textfiles, 11436
lcFile			= LOWER( ALLTRIM( STRTRAN( CHRTRAN( NORMALIZE( STREXTRACT( tcLine, ".ITEM(", ")", 1, 1 ) ), ["], [] ), 'lcCurDir+', '', 1, 1, 1) ) )
FoxBin2Prg: [C:\VFP_SOURCE\FOXBIN2PRG\FOXBIN2PRG.EXE] (EXE Version: v1.19.47)
tc_InputFile = [C:\VFP_SOURCE\VFPAPPS\TEST.PJ2]

This also happens in the most recent preview.
I've traced it down to this minimal example:
	*<TextFiles>
	.ITEM(lcCurdir + '..\..\..\program files (x86)\microsoft visual foxpro 9\ffc\_frxcursor.h').Type = 'T'
	*</TextFiles>

Looking in foxbin2prg.prg, I see that the STREXTRACT() in analyzecodeblock_textfiles is the only one that doesn't have something more after the cEndDelim ")" parameter.  For example, analyzeCodeBlock_ExcludedFiles has ").Exclude".
So perhaps line 11436 should change ")" to ").Type" like so:
lcFile			= LOWER( ALLTRIM( STRTRAN( CHRTRAN( NORMALIZE( STREXTRACT( tcLine, ".ITEM(", ").Type", 1, 1 ) ), ["], [] ), 'lcCurDir+', '', 1, 1, 1) ) )

Hi Nathan:
Thanks for the suggestion, I've fixed with it.
You can download v1.19.48 Preview-5 from VFPx:
https://vfpx.codeplex.com/releases/view/116407

@fdbozzo thanks for your effort on this.  Your tool is making working with FoxPro better.


Netsparker: Attempt to accommodate any date string format

Attempt to accommodate any date string format that could be present in the report. In the event that a super strange format cannot be parsed (an example I saw was 03/02/2019 15:50:29 (UTC-06:00) that is handled explicitly) then the current time will be used rather than throwing an exception
[sc-7855]

Acesslint can be ignored on this one 😂


Add ol.style.Circle#setRadius

See #5748.

This is certainly not right — all these commits.
@ahocevar ?

Looks like you made a mistake rebasing your branch upon master. I think it will be easiest if you close this pull request and create a new one from a branch where you just git cherry-pick 1b701f1.

Yeah, creating a new one.


feat: split monitor network url

…lear

What type of PR is this?
feat
What this PR does / why we need it:
split monitor network url with multiple labels to make it more clear
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

reopen one to avoid conflict


Fix existing backend unit tests

Describe the bug
Currently there is no consistent way to execute all python unit tests without an error.
To Reproduce
Steps to reproduce the behavior:

Go to '(einander-helfen)/etl'
Execute 'python3 install -r requirements'
Execute 'python3 -m unittest'
See error

Expected behavior
All python unit tests should be executed without an error that does not come from the created code.
Additional context
There is an pythonpath/directory naming issue.

closed with merge of #256

closed with merge of #256


Add scm secrets into credentials lookup mechnism & fix gitlab client creation

What does this PR do?
Screenshot/screencast of this PR

What issues does this PR fix or reference?

How to test this PR?

quay.io/redhat-appstudio/pull-request-builds:spi-controller-#
quay.io/redhat-appstudio/pull-request-builds:spi-oauth-#

/retest

Codecov Report
Attention: Patch coverage is 60.71429% with 11 lines in your changes are missing coverage. Please review.

Project coverage is 71.49%. Comparing base (70369e5) to head (f36c0a7).
Report is 20 commits behind head on main.

Files
Patch %
Lines

pkg/serviceprovider/lookup.go
55.00%
5 Missing and 4 partials :warning:

pkg/serviceprovider/gitlab/gitlabclientbuilder.go
66.66%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #939      +/-   ##
==========================================
+ Coverage   69.87%   71.49%   +1.62%
==========================================
  Files          94       93       -1
  Lines        6087     4774    -1313
==========================================
- Hits         4253     3413     -840
+ Misses       1465      996     -469
+ Partials      369      365       -4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

/retest

/retest


Netherlands with carnival

refs #

For information, read and make sure you're okay with the Contributing guidelines.

[x] Tests with a significant number of years to be tested for your calendar.
[ ] Docstrings for the Calendar class and specific methods.
[ ] Use the workalendar.registry_tools.iso_register decorator to register your new calendar using ISO codes (optional).
[x] Calendar country / label added to the README.rst file.
[x] Changelog amended with a mention like: "Added <country> by @pseudo (#)". Note Please do NOT change the version number here. It's the project maintainers' duty.

looks like you had trouble updating your branch from master... I know these operations are sometimes tedious tasks :o)
In order to have a clearer history / git log, I wish we could have those commits rebased from upstream. Or if it's too complicated, re-update your personal fork up to master and cherry-pick or apply the patch in a more "atomic" way.
Do you know how to do this? If you can't or don't have time, don't bother, I'll take over your branch and retrieve them and make them shine in their full glory! :o)

Always happy to learn. I'll make a new PR (with your suggested atomic way).


Update Identity README for azd CLI credential

Update the README's "Credential classes" section for the new credential type added via https://github.com/Azure/azure-sdk-for-js/pull/24180

API change check
API changes are not detected in this pull request.

Replaced by https://github.com/Azure/azure-sdk-for-js/pull/24826


How to fix lenguage

Hello guys,if I want to correct a few words in my (already existing) language, how can I do? I noticed that editing server-server-status.po status.mo files generates errors
how can this be done?

@darkban the best way is to edit them at POEditor and then ping me here or at<EMAIL_ADDRESS>and I will push them upstream. Otherwise if you edit server-status.po you need to regenerate server-status.mo file :slightly_smiling_face: But I would really prefer you editing the translation at POEditor so everyone can have proper translation :wink:

It is but the translations can still be edited AFAIK :slightly_smiling_face:

Fixed in 3d83291


Be able to set working directory or use the project path as default working directory

What would you like to be added:
As far as I can tell, horusec assumes that the current directory as the working dir. It would be useful if horusec supports a different working dir or if treats the project path as so.
Actual usage.
$ cd `project diretory` (where my horusec-config.json is in)
$ ./horusec start -e -p ./

If I don't do the first step, horusec does not find horusec-config.json file. Would be very nice if it does.
I could pass the working dir:
$ ./horusec start -e -p ./ -W ./

Or use the project path as working dir. (maybe it could be a brocking change)
Why is this needed:
Basically, I wanted to create a gradle task the wrap the horusec binary. I couldn't make it work because of this behave.

Hi @leandrorodrigueszup thanks for the suggestion.
Use the --config-file-path would not help you in this case?
./horusec start -e -p ./ --config-file-path path/to/your/config-file

Hi @matheusalcantarazup thanks for the reply!
Yes,  it works! :-)
But still a little verbose and unnatural in my opinion.
But anyways, I can move on now. Thanks!

This is good! I'll let this issue open for further discussion.
Thank very much for your suggestion.


Linewise visual block does not allow you to modify cursor position

When you activate linewise visual block (V), you lose visual indication of your cursor position.
This may be related to not being able to move the cursor to end of line, and then move to matching bracket (V$%).

As stated in the README, this package is no longer maintained and is deprecated. We recommend that people use the vim-mode-plus package instead. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!


Show Spinner for a minimum amount of time

If the request is reasonably fast, the spinner will become a flicker on the web page and users do not have enough time to understand what is happening. Besides wait some time before the spinner is shown.

This spinner is shown in the component  CSearchInput it was solved showing the spinner for 500 milliseconds.


[Bug Report] Checkbox with border has too much margin

Element UI version
2.5.3
OS/Browsers version
chrome 72/windows 10
Vue version
2.6.0-beta.3
Reproduction Link
https://element.eleme.io/#/en-US/component/checkbox#with-borders/
Steps to reproduce
Use checkboxes with borders
What is Expected?
to have the correct amount of margin
What is actually happening?
they have the INCORRECT amount of margin

Issues without a valid reproduction link will be closed directly.

Just look at the checkbox with borders in the reproduction link I sent. The margins are too large! It used to be much smaller.


Implementation of mesh:attachAttributes(), with instance divisor

Some features remain to be implemented, these are marked with TODOs.
Currently it is not allowed to attach a mesh which itself has attachments.

To test this, run the example in https://github.com/bjornbytes/lovr-docs/pull/3

Going to merge this to avoid merge pain with other mesh work


Address hypothes.is comment QgLqiD1wEem30aPn5VUxag

Hypothesis annotation https://hyp.is/QgLqiD1wEem30aPn5VUxag/jrnold.github.io/r4ds-exercise-solutions/transform.html<EMAIL_ADDRESS>
Somewhat more succinctly:
standardized_flights <- flights %>%
  filter(!is.na(air_time)) %>%
  group_by(dest,origin) %>%
  mutate(air_time_standard = (air_time - mean(air_time)) / sd(air_time)) %>%
select(origin,dest,month,day,carrier,flight,air_time,air_time_standard)

No need to change this code. In particular, the ungroup() is needed. See #237


duplicates when auto-generating patterns

I have an issue with the auto-generation of patterns.
Sometimes it generates the same pattern twice, which kinda defeats the purpose.
In my implementation, it happens more often than, what might be good.

You can create an array of patterns as a workaround for this issue, like below:
` let patterns = [ draw('square', '#1f77b4'), draw('line', '#ff7f0e'), draw('line-vertical', '#17becf'), draw('triangle', '#2ca02c') ]
And accessing the elements through a loop might solve this issue of duplicates.


youtube.scss

converting yt.css to youtube.scss using tools converter

#425 i using chrome css to scss converter

@tzarebczan i think it almost done sir.

@ykris45 your branch is not up to date with the new youtube page change on the master branch.

@maximest-pierre I think you're going to have to do this one, see comments made in slack a few minutes ago.


fix issue #927

according to http://stackoverflow.com/questions/38176472/graph-write-pdfiris-pdf-attributeerror-list-object-has-no-attribute-writ
try import pydotplus as default pydot

Coverage decreased (-0.3%) to 35.331% when pulling 95bb6974cb625616cdd9be7c9ca46f043454712e on x007007007:master into e8770061290f0269ee28af37bfb7708ce7edca21 on django-extensions:master.

Thanks !


[WIP] Bug 1855660: helm-container

@timflannagan1 Is this our first casualty from accepting the change on release to rhel8 only since this is trying to get a rhel7 image?

@EmilyM1 When you get a chance, please make this into two commits, one with just your manual changes, another with everything caused by those changes due to build process. Then force push. Will make it easier to be safe and clear about review. Thanks.

@bentito  The first 2 commits were manual affecting glide.yaml file. I removed the manually added code in commit ebd733705b6c368655b54bd9b40a064d6cfdda68 so the only changes now are autogen'd

/retest

Now here https://github.com/kube-reporting/helm/pull/41


Change exclude test example

For our usage, we've found the biggest potential leaker of secrets is using un-redacted customer data in tests.
The example config should not have tests/ excluded.

@domanchi I have already said too much :laughing:

To be clear, it's a hypothetical usage we thought of with data scientists - who might be querying a lot of data that may or may not be effectively redacted. Better to be safe than sorry!


[WIP] Implement sync push mirror on push

Support synchronizing with the push mirrors whenever new commits are pushed.
Related Issues: #18220
Breaking changes:

field SyncOnPush is added to PushMirror struct, thus a database migration is necessary.

TODO:

[ ] update UI to add "sync on push" option
[ ] relevant doc updates
[x] database migration

Since services/mirror is not too big, Is it necessary to split it as two packages?

In services/mirror, functions dealing with pull mirrors would send new notifications. Then services/mirror imports modules/notification, which then imports modules/notification/mirror, which then must call functions in services/mirror again. It causes a circular imports.
To avoid it, push mirror related functions must be in a new package, so that:

services/mirror

import modules/notification

import modules/notification/mirror

import services/pushmirror (can't be services/mirror again)

In services/mirror, functions dealing with pull mirrors would send new notifications. Then services/mirror imports modules/notification, which then imports modules/notification/mirror, which then must call functions in services/mirror again. It causes a circular imports.
To avoid it, push mirror related functions must be in a new package, so that:

services/mirror

import modules/notification

import modules/notification/mirror

import services/pushmirror (can't be services/mirror again)

To avoid import cycle, we can move some into modules/mirror.

In services/mirror, functions dealing with pull mirrors would send new notifications. Then services/mirror imports modules/notification, which then imports modules/notification/mirror, which then must call functions in services/mirror again. It causes a circular imports.
To avoid it, push mirror related functions must be in a new package, so that:

services/mirror

import modules/notification

import modules/notification/mirror

import services/pushmirror (can't be services/mirror again)

To avoid import cycle, we can move some into modules/mirror.

What exactly differentiates packages in modules and services? I find the contribution guideline a bit vague on this. And the guideline says "Uses models and modules to handle the request," but there's no requests involved as well.
Sorry that I'm new to gitea contributions. Any more suggestions would be much appreciated.

I still think push mirror and pull mirror have to be separated. That will allow the event pipelines to flow from pull sync to push sync, through notifications.
graph LR;
    A["pull sync"] --> C["new commits"];
    B["git push"] --> C;
    C --> D["push sync"];

Separation of two mirror types would be the most natural fit for this event flow. Again, any other ideas welcomed.

Now it's using a modules/mirror package and the separate pushmirror package has been removed.

Codecov Report

Merging #19411 (6c0e721) into main (711cbcc) will decrease coverage by 0.05%.
The diff coverage is 55.58%.

@@            Coverage Diff             @@
##             main   #19411      +/-   ##
==========================================
- Coverage   46.91%   46.86%   -0.06%
==========================================
  Files         973      975       +2
  Lines      134721   134818      +97
==========================================
- Hits        63204    63182      -22
- Misses      63768    63877     +109
- Partials     7749     7759      +10

Impacted Files
Coverage Δ

modules/markup/markdown/toc.go
0.00% <0.00%> (ø)

modules/setting/i18n.go
100.00% <ø> (ø)

modules/setting/setting.go
47.59% <ø> (+0.38%)
:arrow_up:

modules/templates/helper.go
45.86% <ø> (ø)

routers/install/install.go
1.77% <0.00%> (ø)

routers/install/routes.go
44.00% <0.00%> (ø)

routers/private/hook_pre_receive.go
41.88% <0.00%> (ø)

routers/web/admin/admin.go
10.02% <0.00%> (ø)

routers/web/auth/auth.go
26.26% <0.00%> (-0.13%)
:arrow_down:

routers/web/auth/password.go
2.71% <0.00%> (ø)

... and 43 more

Continue to review full report at Codecov.

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

We need a push mirror queue to do the work.

make lgtm work

I'm currently testing this feature using 1.18.0+dev-478-g08609d439. There seem to be a few issues. First of all, looking at the database, it seems to have set all my existing push mirrors to sync on commit automatically upon updating. Second, checking the "Sync when commits are pushed" checkbox seems to do nothing. I tried with it checked and unchecked, and both times, the new mirror had sync_on_commit set to false in the database. The syncing only started working once I manually set it to true in the database.

@Arsen6331 please open a new issue for this, as otherwise things get lost in closed threads.


Add the missing BWIPP postscript files.

Fixes #10

tox should've found this because it installs your package into the virtualenv before testing it, potentially the way tests are getting invoked isn't finding things like this... maybe python setup.py test uses the non-virtualenv copy of your package in testing.

I would've expected it to too, but I'm really crap at packaging stuff - know only basics, that's how I forgot the files in the first place. If you have suggestions how to have a stab the tox issue - would like to hear them.


python311Packages.python-fx: 0.3.0 -> 0.3.1

Diff: https://github.com/cielong/pyfx/compare/refs/tags/v0.3.0...v0.3.1
Changelog: https://github.com/cielong/pyfx/releases/tag/v0.3.1
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 257915 run on x86_64-linux 1

8 packages built:

python310Packages.python-fx
python310Packages.python-fx.dist
python310Packages.qiling
python310Packages.qiling.dist
python311Packages.python-fx
python311Packages.python-fx.dist
python311Packages.qiling
python311Packages.qiling.dist


JCBC-1098 : N1qlQueryExecutor is allocating twice the memory it needs

Motivation
When the netty ByteBuf is not backed by a byte array.
N1qlQueryExecutor#executeQuery is allocating the byte array twice.
the first time in TranscoderUtils#byteBufToByteArray
the second with Arrays#copyOfRange
Modifications
this patch add a specialized version of
TranscoderUtils#byteBufToByteArray (copyByteBufToByteArray) that only
call Arrays#copyOfRange if the ByteBuf is backed by a byte array.
Result
greatly reduce the n1ql memory allocation

@benbenw this is a great change and I'd like to get this in of course! We are using gerrit for source code management, github is just a mirror. We have detailed contributions here https://github.com/couchbase/couchbase-java-client/blob/master/CONTRIBUTING.md
Would you mind following these steps to get it up on gerrit and then we get it in from there? If you don't want to go through all of it I'd at least need you to sign the CLA (also in the doc how) and then I can get it in for you.
Cheers,
Michael

@benbenw note that I'm out next week, so pinging @subalakr so we can push it forward in the coming week.

Ok
I let you do the gerrit integration ;-)

Le 28 juil. 2017 à 21:36, Michael Nitschinger<EMAIL_ADDRESS>a écrit :
@benbenw note that I'm out next week, so pinging @subalakr so we can push it forward in the coming week.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@benbenw just checking, did you get to sign the CLA?

http://review.couchbase.org/#/c/82004

merged, thanks!


Enhance Demo

I have done some modification to make our demos CLI more intuitive.

Last change I would like to do, rename :

leshan-standalone in leshan-server-demo
leshan-client-example in leshan-client-demo
leshan-bs-server in leshan-bsserver-demo
What do you think ?

Hi @sbernard31,
can you just shortly explain why so many methods now explicitly throw InterruptedException? The reason for this is not obvious to me from flying over the code and I want to understand what made this necessary because InterruptedException is an exception I do not see explicitly thrown that often ...

Ok :)
There is not so many changes, I mainly change the way interrupted exception is handled when we send request in synchronous way. Most throwing exception you see are in the tests, the real change is the API  on LwM2mServer.java)
Before this change, an InterruptedException was handled as a timeout (I mean the send method return null), but we see in practice this is not really a good idea. You don't really want to do the same thing if the thread is interrupted or if the request timeout.
So we choose to throw InterruptedException as it is the common way to handle thread interruption on blocking method in Java.
(If you use the asynchronous way to send request, you don't have InterruptedException but in our tests  we mainly use the synchronous way)

I see, thanks for the explanation, @sbernard31 :-)
+1 for renaming (this should help making clear that these implementations are not intended for production deployment)
+1 for merging

:+1: for merging


Bring into core?

Hi,
thank you for your awesome work, I was astonished when I found out this is not working out of the box with doctrine-migrations. For me it seems the namespaced migration feature does not make too much sense without either allowing to configure the entity manager per namespace or allowing to pass the namespace + em to the migrate command :thinking: I ended up having multiple EMs and connections defined + migration namespaces... and then d:m:m happily ignores that and runs all migrations of all namespaces against the default connection :eyes: Feels like half of the feature got missing :)
Have you thought about bringing the adjustments you have made into doctrine migrations core?

Hi @krossekrabbe

and then d:m:m happily ignores that and runs all migrations of all namespaces against the default connection eyes

I'm not sure about what you have set up. We have been using this package for 2+ years and haven't observed an issue like that (nor have other users from this package). Please try replicating the problem in a repository with instructions to see what's happening.

Have you thought about bringing the adjustments you have made into doctrine migrations core?

I would ping at https://github.com/doctrine/DoctrineMigrationsBundle/issues/38. I don't have the time/knowledge to do a PR that probably would not get merged. I think it's more important to have a conversation with the maintainers of the project to know if they want to support a feature like this.

Hi @devnix

and then d:m:m happily ignores that and runs all migrations of all namespaces against the default connection eyes

I'm not sure about what you have set up. We have been using this package for 2+ years and haven't observed an issue like that (nor have other users from this package). Please try replicating the problem in a repository with instructions to see what's happening.

Misunderstanding, your bundle works perfectly fine! I was talking about the original doctrine-migrations commands which are missing the functionality this bundle provides.


Questions regarding design pattern in scala

I'm going to try to find some time to work on this topic


SoLoader license incompatible, yet is statically instantiated

Since SoLoader is still on BSD+Patents, using Yoga in an environment where the BSD+Patents license is not approved is still impossible because Yoga statically uses SoLoader in YogaNode and YogaConfig:
https://github.com/facebook/yoga/blob/3dfb68887db34a8a77e894c66a50e747d831b0b9/java/com/facebook/yoga/YogaConfig.java#L18-L24
https://github.com/facebook/yoga/blob/3dfb68887db34a8a77e894c66a50e747d831b0b9/java/com/facebook/yoga/YogaNode.java#L19-L25
Can we make this loading optional or configurable? Or even better would be to just to convert SoLoader to MIT...

SoLoader has been moved to the more permisive Apache license. https://github.com/facebook/SoLoader/commit/fa4d02f679ba123302e70cdd6c00cb8ef6143c50


Opening religon tab cause CTD when using Reformed Religion/Custom heresy created in CK3

[fixedwindow.cpp:2951]: interface/added_countryreligionview.gui: Missing Icon 'defender_of_the_faith_tier' in window 'countryreligionview'.
Any custom religion seems to do it.
Version: 0.7 Gayomarthian
EU4 Version: 1.31 and 1.31.1

We're aware, thanx. You're operating on an unreleased experimental build.

Please grab the latest build and see if all is well.


Fix content visibility issues on DeFi Asset Tracking page

Fixed #530
This update addresses critical visibility issues on the DeFi Asset Tracking page, where content was challenging to read. Adjustments have been made to ensure text contrast is appropriate in both dark and light modes. Additionally, layout elements have been restructured to prevent overlapping and enhance readability, delivering a more accessible and user-friendly interface.
https://github.com/user-attachments/assets/12b7f77a-2807-418e-9730-f2b9db59306f

@MastanSayyad

@vanshchauhan21 remove the levels!

@vanshchauhan21 remove the levels!

Okay sir


Regex doesn't work for version which contains '+' symbol

Hi,
Puppet 4.10.0
Module version: v1.18.2
I try install airflow from url but puppet tries to uninstall it.
ensure_resource(python::pip, 'airflow', {url => https://github.airflow/arhive/1.8.0.zip,                                          pkgname => 'airflow',                                          ensure => '1.8.0+apache.incubating')
Issue is that your regex doesn't like '+' in the version string '1.8.0+apache.incubating'.
file: puppet-python/manifests/pip.pp
strings:
136   if $ensure =~ /^((19|20)[0-9][0-9]-(0[1-9]|1[1-2])-([0-2][1-9]|3[0-1])|[0-9]+\.\w+\+?\w*(\.\w+)*)$/ {
209   /^((19|20)[0-9][0-9]-(0[1-9]|1[1-2])-([0-2][1-9]|3[0-1])|[0-9]+\.\w+\+?\w*(\.\w+)*)$/: {
Thank Pav

Hi @pleochroic. Could you try the latest release and our master branch? Does the error ocour there as well?


Alt+L locks Windows after swapping Alt and Win

Microsoft PowerToys version
0.74.1
Installation method
WinGet
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce

Use Apple Magic Keyboard (alt and win are the wrong way around on Windows)
Remap alt to win and win to alt
press Alt + L

✔ Expected Behavior
I am using GlazeWM for window management, and have Alt+L bound to switch focus right.  Expecting Windows to not go to lock screen.
❌ Actual Behavior
Windows goes to lock screen.
Other Software
No response

I tried searching for a solution but was unable to find one.
Microsoft Keyboard Layout Creator does not allow for remapping of the modifier keys.
I don't want to disable Win+L.
Any solution where the keys are mapped to their correct positions, Alt+L does not trigger lock screen, and Win+L works would be much appreciated. 🙏

Physically reconfiguring a magic keyboard is beyond my skill and therefore not an option.

I am going to close this issue because Win+L is reserved by Windows and cannot be remapped by an application. This is probably for privacy and security reasons.


MultiWidgetSelectionDelegate. replace @Synchronized to SynchronizedObject

Compose 1.5.0-rc04

@Synchronized doesn't work for native/web

Duplicate https://github.com/JetBrains/compose-multiplatform/issues/3363

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


error running plugin

Hi
I am trying to use confluence and i am getting the following error.
steampipe query "select * from confluence_content;"
Error: rpc error: code = Internal desc = hydrate function listContent failed with panic runtime error: invalid memory address or nil pointer dereference (SQLSTATE HV000)
+----+-------+-----------+--------+------+----------------+------+
| id | title | space_key | status | type | version_number | _ctx |
+----+-------+-----------+--------+------+----------------+------+
+----+-------+-----------+--------+------+----------------+------+
any ideas why ?
Logs are attached
Thanks
plugin-2024-06-21.log
steampipe-2024-06-21.log
database-2024-06-21.log

@shemreader thanks for the logs.
Does this also happen if you add a LIMIT to your query (e.g. SELECT * FROM confluence_content LIMIT 10;) or when you query for spaces (SELECT * FROM confluence_space;)?

I see this warning on line 52 of your Steampipe log:
2024-06-21 12:02:40.341 UTC [WARN]  steampipe [c66a]: Error waiting for notification: read tcp <IP_ADDRESS>:56204-><IP_ADDRESS>:9193: use of closed network connection

This page suggests it may be due to two websockets connected to a singular address using the same port from the same machine. Do you have multiple Steampipe processes running, possible in the background?

Hiw do i check it ?
On Fri, Jun 21, 2024, 23:11 Ellis Valentiner @.***>
wrote:

I see this warning on line 52 of your Steampipe log:
2024-06-21 12:02:40.341 UTC [WARN]  steampipe [c66a]: Error waiting for notification: read tcp <IP_ADDRESS>:56204-><IP_ADDRESS>:9193: use of closed network connection
This
https://stackoverflow.com/questions/59314363/read-tcp-addr-addr-use-of-closed-network-connection
page suggests it may be due to two websockets connected to a singular
address using the same port from the same machine. Do you have multiple
Steampipe processes running, possible in the background?
—
Reply to this email directly, view it on GitHub
https://github.com/ellisvalentiner/steampipe-plugin-confluence/issues/59#issuecomment-2183388689,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHW564RU3G4HPPKD3EX3BDLZISCIXAVCNFSM6AAAAABJVYF3E6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBTGM4DQNRYHE
.
You are receiving this because you were mentioned.Message ID:
@.***
com>


Add support for profiling on Windows

This PR adds (initial) support for profiling on Windows. It uses the Event Tracing for Windows framework to get very fast, low-overhead captures.
In order to expedite getting something working, this PR calls out to the xperf.exe tool to start/stop tracking and save output to a file. It then parses events from that file, adding them to a profile. This is probably good enough for 95% use cases, but in the future it would be nice to use Etw directly to create an in-memory trace, handling events as they come in. Etw is just such an annoying API to work with that I didn't want to tackle that; I use the ferrisetw crate for event parsing, and it can probably be used for kicking off a kernel trace, but the process for enabling kernel providers is convoluted.
There's a few things that I need to fix:

Sort out Windows administrator privilege elevation. I need administrator privileges for two things: to run xperf, and to query to list & global address of kernel drivers. Right now, I require that samply be run as administrator. But a better solution would be to spawn an elevated child process that can do all the elevated things, while the main process takes care of launching the process to be profiled (right now it's launched as Administrator, not great!)
Implement profiling by pid. Trivial, I just haven't written the code yet. I'll also add support for just profiling the entire system, basically samply start and wait for input to stop profiling. (Or samply start --time 10s).
Sort out some arch bits, right now there's some hardcoded aarch64 which isn't even correct (should be ARM64).
Grab thread names. Double check main-thread rules (I'm not sure if the main thread and the process have the same PID, or if there's a different TID for a process's main thread. I think the latter, but double check.)
Actually test on a system where I can use the profile provider :) ARM64 Windows seems to have a bug where the kernel PROFILE provider just doesn't work; errors out when you try to start. And I'm on a Apple Silicon Mac using Parallels, with no x86_64 (or non-VM ARM64) for another week.

But it works, and works quite well!
Future work:

Add support for JIT events, especially CoreCLR. Luckily, ETW is pretty pervasive on Windows, and CoreCLR delivers Jit events via ETW, so no need to mess around with jitdump files.
A bunch of things that will probably involve helping the profiler front-end to not be so Firefox specific so that we can capture additional data, but I'm scared of the front end code :|

I suppose this would also fix #77.

Nice work! We have another implementation at https://github.com/jrmuizel/etw-profiler/tree/main/etw-gecko that we've been using to profile Firefox. I should have added a comment to #77 pointing to our implementation, so I'm sorry if I've caused you to do extra work by not doing so. However, given the state of ETW documentation, I'd say having multiple implementations is probably a good thing.
For Administrator privilege elevation I was planning to use the runas crate.

A bunch of things that will probably involve helping the profiler front-end to not be so Firefox specific so that we can capture additional data, but I'm scared of the front end code :|

If you file issues in the Firefox profiler repo, I can add them to the "General-purpose profiler" list.

I should have added a comment to https://github.com/mstange/samply/issues/77 pointing to our implementation, so I'm sorry if I've caused you to do extra work by not doing so.

No problem, I really should've gone looking first. This was also a learning exercise, and I need to do CoreCLR ETW events anyway. From a quick glance at that crate, it's focused on Firefox profiling (understandably), and has lots of logic for SpiderMonkey etc. How do you want me to proceed here -- would you accept this PR as a "lightweight" generic Windows implementation into samply, and keep the firefox etw-profiler/etw-gecko stuff as-is separate?

The original plan was to just dump the etw-gecko + etw-reader code into this repo as-is (plus run cargo fmt and fix the warnings + clippy), and then change it to use more of the Linux importer's infrastructure, e.g. crate::linux_shared::processes::Processes and everything that comes attached with that, to get support for the --reuse-threads option.
There's not a ton of Firefox-specific stuff in there. There's a little bit for Firefox-specific ETW trace event support. The JIT stuff supports both Spidermonkey and V8, and also everything else that happens to use Microsoft-JScript/MethodRuntime (I don't know if there are any other users). And all the files other than main.rs are already part of samply/src/linux_shared.
The next step in the original plan was to stop using xperf and do the recording ourselves, by starting with https://github.com/jrmuizel/etw-profiler/blob/ce597a3e1d54181a3fd6a9b4a249e15e620643bb/etw-reader/examples/record.rs and adding everything needed to make it work.
Jeff also did a lot of work on the etw-reader side to correctly parse ETW trace events coming out of Windows and out of Chrome. The etw-reader code has diverged quite a bit from the original ferrisetw code, and most of the changes were about fixing various parse errors and to adding support for more (undocumented) event types, as far as I understand.
I need to look at your implementation more closely before I can decide how to proceed. I don't feel strongly about which code to start with (you could say the etw-gecko code isn't the cleanest), as long as we eventually end up with something that isn't worth than either implementation.
Based on a quick glance at your implementation, here are a few things where the etw-gecko implementation is more complete:

The LibraryInfo data is read from KernelTraceControl/ImageID, KernelTraceControl/ImageID/DbgID_RSDS, and MSNT_SystemTrace/Image events, without reading the binary file. This probably only works because xperf does its own post-processing of the trace during the merging process, and we'll likely have to read the binary file ourselves once we stop using xperf.
I've spent a lot of effort pairing up kernel and user stacks in a way that I think is mostly correct now.
We correctly compute CPU deltas and off-cpu samples based on CSwitch information (if you run with -stackwalk profile+cswitch).

Yeah, etw-reader/etw-gecko is definitely more complete. I also don't feel strongly about my implementation vs etw-gecko -- ultimately I just want something that works (and roll out internally to everyone, I'm tired of the terrible cross-platform profiling story). My overall implementation is really dead-simple. I haven't even profiled it, and I know there are probably lots of easy perf improvements (e.g. repeated parsing of the same event structure annoys me).
Good to know about KernelTraceControl events -- I didn't know about those, and the library ID stuff was annoying (due to the asyncness), I'll add that. I'll take a look at the CSwitch info as well. For kernel + user stack lining up -- tell me more? I'll take a look at the code, but I assumed that that merging already happened into a single stack.
For xperf vs native... is that really worth it? The only advantage that I really see in not using xperf is not needing to have it available, and not needing additional disk space for the etl file. The ability to process events as they come in instead of via intermediate file seems like a mixed bag. It would mean that the event processing must be as lightweight as possible, because it'll be happening during the trace itself, and overall much more code to maintain. (A good reason might be if there were plans to support a "live" profiling display, but I don't think the JSON profiling format can really support that?) Seems like a nice to have, but not something that I'd wait on landing windows support for (either this or via etw-reader).

For kernel + user stack lining up -- tell me more? I'll take a look at the code, but I assumed that that merging already happened into a single stack.

The kernel and user parts of the stack come in separate stack events. And sometimes you can stay in the kernel for multiple samples, and then once you get back out into user-land, a single user stack is captured which applies to all the previously-taken kernel stacks. At least that's what I think is happening; this assumption has given us the most reasonable looking profiles so far.

For xperf vs native... is that really worth it? The only advantage that I really see in not using xperf is not needing to have it available

This is the part that I care about the most. I don't like asking users to install other software first.
The other part I care about is reducing the post-processing time.

The ability to process events as they come in instead of via intermediate file seems like a mixed bag. It would mean that the event processing must be as lightweight as possible, because it'll be happening during the trace itself

True. Though for lower-overhead profiling, samply import trace.etl would still remain an option.
The Linux backend also has this overhead already (and more, because it copies raw stack bytes out of the kernel).

and overall much more code to maintain.

Some, sure.

Seems like a nice to have, but not something that I'd wait on landing windows support for (either this or via etw-reader).

Oh I agree, the first shipping version should use xperf.

(A good reason might be if there were plans to support a "live" profiling display, but I don't think the JSON profiling format can really support that?)

No plans at the moment.

I dug into etw-gecko a bit more. It's a ton more complete. Some quick thoughts, there's two things I like about my implementation better, though they're all surface/structural stuff:

I kinda like how I did the event parsing in https://github.com/mstange/samply/pull/141/files#diff-a399eae2dd67b64464441b07522fc74f2ee27f68830cc92cb2019646609b9bd3 more than writing it all out in the big switch statement. Not every event can be handled that way though, there's some with a lot of logic that I looked into less.
I did the path song-and-dance to convert NT paths to DOS ones. (It's not 100% complete, won't handle network stuff, but I should just do the GLOBALROOT stuff for those)

But it would be pretty quick to turn etw-gecko into a library and just plug it into the samply integration code that's here, replacing my parsing. Thoughts? I'm happy to do that work if you or @jrmuizel weren't going to get to it soon. Then we can do some refactoring of the etw-gecko code. (Would also be nice to move it into this repo once that happens for easier future work, but depends if you two would be up for that)

I'm happy to do that work if you or @jrmuizel weren't going to get to it soon. Then we can do some refactoring of the etw-gecko code. (Would also be nice to move it into this repo once that happens for easier future work, but depends if you two would be up for that)

That would be great! I would like to have it in this repo. And I think it should just be part of the samply package for now, so that we can easily change it without having to release anything separately to crates.io.
I was going to get to it in two weeks or so, but if you get to it before then, please go ahead!

I did the path song-and-dance to convert NT paths to DOS ones. (It's not 100% complete, won't handle network stuff, but I should just do the GLOBALROOT stuff for those)

I was really hoping there'd be a crate that does this, and it seems like dunce would be the one that you'd expect to do it, but it doesn't simplify those paths. Here's an issue asking about it.


How to building Docker Image in Maven Project of multi-module

project structure:
--parent
----subA
------pom.xml
----subB
------pom.xml
----pom.xml
How config above project with dockerfile-maven-plugin to building Docker Image?

At least from my point of view, this involves adding the plugin to the pom that builds the jar you're actually interested in running. So I guess it would depend on what the current output of your project is?

我也想知道,如何在多模块项目构建docker镜像,并推送到制定仓库


Doc change: Series UID and Instance UID in imaging studies are not proper UUID

What happened?
Just a Doc Change on hte CSV page in the wiki. In imaging studies, the CSV export page lists Series UID and Instance UID as UUID format, but they are not in official UUID format. Format for Series and Instance UID is something like:
1.2.840.99999999.1.26930466.662735060367
which is not a properly formatted UUID, which is more something like:
3e67a206-82c1-04f3-ab5c-08aaf72757e9
Setting Series and Instance UID as UUID in Postgres results in an loading error
Environment
- OS: Linux beast 5.19.0-50-generic #50-Ubuntu SMP PREEMPT_DYNAMIC Mon Jul 10 18:24:29 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
- Java:openjdk version "11.0.20" 2023-07-18

Relevant log output
No response

I was actualy able to change the wiki, so I went ahead and made the change myself. Wiki is world editable, neat!


New version: WiseCleaner.WiseProgramUninstaller version 3.0.4

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.1 schema?

Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-No-Executables


If the whole realm object is deleted from JSON file - it is not deleted from the keycloak on the json file import

Current Behavior
If the whole realm object is deleted from JSON file - it is not deleted from the keycloak on the json file import
Expected Behavior
As the imported JSON files are being considered as a single source of truth and the desired state - we could not rely on this concept anymore, as the config in JSON files does not reflect the actual config in the keycloak.
Please note that the "import.remote-state.enabled" is set to "false". So theoretically it should purge all the condfiguration that is not in the JSON file.
Steps To Reproduce
1. Deploy Keycloak
2. Run importer and import multiple realms
3. Delete one realm JSON object from file
4. It is still seen in the GUI after re-importing the JSON

Environment

Keycloak Version: 21.1.1
keycloak-config-cli Version: 5.6.1

Anything else?
No response

Hi @roman-aleksejuk-telia, this tool is meant to only change realms which are provided in the import. I also can't imagine any use case why a realm should be deleted through automation. Just in case someone would mess up the file this would mean that all of your users would be deleted, it's not a good idea to add this feature 😉


"information on whether [assay] performed" terms do not specify specimen type

Terms in VEuPath labeled "information on whether [assay] performed" have definitions that do not mention the type of specimen on which the assay was performed (e.g. blood, stool). However, these terms in ClinEpi .owl files are categorized with reference to blood or stool testing. My suggestion is to change the VEuPath definitions to specify specimen types.

Discussed 2020-02-19. Decided that it is not necessary to change the definitions of such terms in VEuPath to make them more specific.


Upgrade packages and fix types

Description
This PR updates a number of outdated packages, and fixes the issues related to breaking changes from these updates.
Related issue
N/A
Motivation and Context
This PR cleans up the codebase and makes future changes easier.
How Has This Been Tested?
N/A
Screenshots (if appropriate):

Trade History amounts suddenly shows amounts with $ symbol, need to remove it


[Feature]

The problem
Doesn't have a way back to reset your coordinates to your original set of coordinates before spoof
The solution/feature
A reset button to disable the spoof and set your coordinates to your actual location
Alternatives
No response
Additional context
Could be a keybind or a reset button in the F6 menu

You can change it back to the original by setting the mode to absolute and the coords to 0 0 0. But it's a good idea to add an extra button for this functionality.

Added mode Vanilla, which doesn't spoof your coords. Also added KeyBindings for every mode.
Download: 1.19.3-1.19.4 1.20-1.20.1 1.20.2 1.20.3-1.20.4


Prepare jupyterlab-lsp 3.9.3 release

Minor patch for https://github.com/jupyterlab/language-packs/issues/198 and dev/testing dependencies refresh.

@krassowski sorry, was noticing this on my phone while running around, but didn't have a time to chime in: the bump of the minimum python did land on both jupyterlab-lsp (which got a minor bump) and on jupyter-lsp which didn't even get that .
I guess on conda-forge, I guess I'll just not bump the jupyter-lsp build number...

Good catch; I updated release notes to mention this. I mean EOL is this week. jupyter-lsp artifacts were not uploaded (cannot upload new one with the same version) so no need to worry about it I think. jupyterlab-lsp indeed will complain on Python 3.6 if someone tries to install 3.9.3.

Yeah, no harm no foul, i think.


feat(grpc-js): upgrade our grpc protoc tools to 1.37_2

What this PR does / why we need it:
grpc-node packages have been deprecated since Apr 2021, this PR updates all the js packages to rely on @grpc/grpc-js instead.
1.37_2 is newer than the 'latest' tag image.

JIRA ID: DT-355

Notes for your reviewer:

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


White screen on navigate with clear history parameter

From @mayureshjadhav on May 26, 2017 3:47
I am using NativeSctipt 2.5 and nativescript-angular 1.2.0 in my app.
Scenario is I am navigating from page 1 to page 2 while navigating I am clearing the history but I can see a white screen while moving from page 1 to page 2 .
What is the issue?
Copied from original issue: NativeScript/NativeScript#4269

From @mayureshjadhav on May 31, 2017 4:21
Actually, I created project from hello word template and added two more pages and when I try to navigate from one page to another by using following code
this.routerExtensions.navigate(["about"], {
                            clearHistory: true
                        });

I can see a white screen and then it moves to about (another) page.
Actually, you will not see the difference in a small application but when your app becomes complicated you will observe this issue.

@mayureshjadhav I was able to fully reproduce this issue using Angular application and ClearHistory set to true with some PageTransition (the default one as well). The issue is not reproducible with core NativeScript project. Marking this one as a bug - keep track of this issue for further updates.
Reproducible with both iOS and Android but only in Angular based applications.
Sample application to reproduce the issue can be found here.
Steps to reproduce:

load the app
navigate with clearHistory set to true+ page transition  (use some of the provided options in the app)

Actual result: the current page is replaced with a white screen and then the navigation proceeds.
Expected result : the current page to stay during the navigation transition as in this NativeScript core application

@mayureshjadhav there might be multiple reasons for your issue so please post more detailed information on how to reproduce your case. As the latest version is 3.x.x we will need both the content of your package.json file and the code used to create your application - the best case would be to create a sample application that demonstrates the issue and what you have achieved so far.

A guess: This is caused because when clearHistory is set the Angular component tree gets destroyed before the navigation begins causing views to be removed for the surface and resulting in a white page. Working on a PR to get that resolved.

@sis0k0 @Buuhuu sorry to drag this out again but using nativescript-angular (latest) and tns-core-modules (latest), This white screen still pops up after clearHistory.

Can you check if ngDestroy() is called before or after the navigation happened? Might e a regression with the latest changes and if so, it would probably make sense to open a new issue.

@jogboms I think this is a regression after the 4.2. There is a PR(#977) that might solve the issue again - can you check if it will work in your case.

@vakrilov I just install the branch.
JS: ERROR Error: Uncaught (in promise): Error: Outlet is not activated
JS: Error: Outlet is not activated
JS:     at PageRouterOutlet.detach (file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/nativescript-angular/router/page-router-outlet.js:152:19) [angular]
JS:     at ActivateRoutes.detachAndStoreRouteSubtree (file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:4525:64) [angular]
JS:     at ActivateRoutes.deactivateRouteAndItsChildren (file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:4511:18) [angular]
JS:     at file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:4541:55 [angular]
JS:     at forEach (file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:498:13) [angular]
JS:     at ActivateRoutes.deactivateRouteAndOutlet (file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:4541:13) [angular]
JS:     at ActivateRoutes.deactivateRouteAndItsChildren (file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:4514:18) [angular]
JS:     at ActivateRoutes.deactivateRoutes (file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:4500:22) [angular]
JS:     at file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:4466:19 [angular]
JS:     at Array.forEach (native) [angular]
JS:     at ActivateRoutes.deactivateChildRoutes (file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:4464:29) [angular]
JS:     at ActivateRoutes.activate (file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:4450:14) [angular]
JS:     at file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/@angular/router/bundles/router.umd.js:4064:22 [angular]
JS:     at SafeSubscriber._next (file:///data/data/com.rfnl.fixershubclient/files/app/tns_modules/rxjs/Observable.js:224:21) [angular]

@Buuhuu The incoming components ngOnit is triggered first then the navigatedFrom's component gets destroyed. All these happens before the navigatedTo component is displayed.

@jogboms Do you have project I can test with?

@vakrilov I'd make one up before the end of today and share it here.

@vakrilov created a sample app here

#977 FIxed. nativescript-angular@next

+1 from t.1129772

@jogboms I'm using nativescript-angular@next along Angular 4.2.5 and I still experience the issue.
Precisely, I'm using<EMAIL_ADDRESS>(gotten from node_modules/nativescript-angular/package.json)

This one is already resolved using nativescript-angular@next with<EMAIL_ADDRESS>Will be included in the official<EMAIL_ADDRESS>release.

@vakrilov sorry to bump that very old issue but i am seeing this in my app using 5.4.0 inside a Vue project. Could you explain (recall) what you changed in Angular to fix it?
Might have to apply the same fix to Vue $navigateTo
Thanks

This issue still there in current version :(

@alexrainman indeed still see this. But it is NOT related to angular!
The original issue need to be unlocked / reopen
And i think it might come from the iOS page implementation. I think it might come from some mix-up calls between viewWillAppear and viewDidAppear

@farfromrefug it is not just iOS, i happens in Android too.

@alexrainman ok then it might be an angular issue for you.
Will open a new issue for my white screen :D


Fixed rounding vol.2

Your function uses tabs for indentation. Please use 4 spaces instead.

You don't have to close the pr, you can just push new commits and it will update the pr.

Unfortunately, I can't test this right now, discord seems to have temporarily ip-banned me and I can't log in or run any bots. I'll remember to test this later.


multi process with websocket have more 8,000 time wait

my django project use django-channels to  Information exchange through webscoket, I use redis as message middleware.
when I start 15 same processes for sending msg to front end, then i found that there are more 8000 tcp time_wait connection,
so my project trobule with Cannot assign requested address, please help me out

Today I use Redis monitor command, found that so much to execute ZREMRANGEBYSCORE and ZRANGE

I don't use Django-channels as I've implemented a native way to handle them. I believe channels uses some LPUSH and LPOP command rather than Pub/Sub, and furthermore might not actually be closing these connections: https://github.com/django/channels_redis/issues/83 it seems to be a channels issue of just not closing the connection properly.
A quick fix if you would like to continue using Django channels is find a setting to close the Redis connection automatically after a certain amount of time.
You can check out my native implementation at cookiecutter-django to help with your chat application. I highly advise you to use a Pub/Sub connection by manually coding the channels layer rather than use django-channels Redis instance as channels is doing some LPOP and LPUSH command, which might be more inefficient. And then you want to manually close the connection since channels might not be doing so.
Have fun!


Fix NDC continue as new to non-current branch deadlock

Fix issue that continue as new to non current branch may encounter deadlock if target workflow is the current workflow

Coverage increased (+0.4%) to 67.083% when pulling b432c80e630296fcac763b041b447c2b65dc6317 on ndc-continue-as-new-stale-branch into 0d160a9d66dfe85401054a03e10e9ae5358a5c8c on master.


Add wait to existing resource

Ensure that adding wait to an existing resource works (previously this failed because it didn't notice that the resource had changed)

I wrote the test such that it fails without this change and succeeds with it.
Currently kubeResourceUpdate is called because terraform knows there's a change to the resource. However, before this change, kubeResourceUpdate would error with "No change" because it was only checking if manifest had changed, because it hadn't been updated to check if wait had changed.
Now kubeResourceUpdate doesn't error, and just carries on and changes the resource to add wait

So the only code change is adding the check to have the provider error if the update is called, but there is no diff for wait

It's actually the opposite - the code change is so that the provider doesn't error when wait is different but manifest is unchanged.

Got it, thanks. I'll look into releasing this today if possible.

https://github.com/kbst/terraform-provider-kustomization/releases/tag/v0.9.2


Fix .integration CI

Motivation
This PR updates the no_output_timeout to 60 minutes for the .integration CI job (sync CDN to tip).

Closed in favor of #2265


Fix/a11y registration

Summary
Addressed a11y changes recommended by Ari, in the /Registration tab, in the following document:
https://docs.google.com/spreadsheets/d/1ZSilJkhbqsjjJi0BC-wJvI1AVXedFIE5S8vNbUJMtxI/edit#gid=157159830
Changes were initially applied to /signup_email, then additionally added to /signup_lti
Ticket Link
https://trello.com/b/OFWhBnfw/riff?menu=filter&filter=member:brechanson
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Has UI changes

 a11y - Registration

force pushed to rebase to head of develop for review (and to see how new CI works)

force pushed to current head of develop for review (eventually, I'm not ready to start reviewing just yet)

@mlippert - it looks like this story is in play, and waiting a code review by you - am I understanding that correctly?

@adonahue Yes that is correct.

thanks @mlippert . I moved to in progress, and realized I'm not sure how we should go about estimating / accounting for small pieces of work like code reviews or small bugs. I stuck a 1 on this story since it's our smallest unit, but I'm guessing this is not a full day's worth of work, right?

I'm anticipating this review needing more time than usual (which is why I haven't tackled it yet) because it's modifying the work done by BrightScout which adhered more closely to the practices of the main mattermost codebase, and I want to ensure that the changes here, continue to do that.

Holding off for right now. At this time, still on track for this sprint, but needs a closer look.

force pushed to the head of develop

@mlippert - what's the status of this story? I'd like to close it out, but I'm not sure what work / steps are remaining. Thx!

This is in rifflearning/mattermost-webapp 1.0.0-dev.30 which was in the edu-mm version 1.0.0-dev.14 which was just deployed to staging.riffedu.com

@brecriffs - I went through and marked the spreadsheet tab above. All the green rows look good. The orange and yellow rows are not passing as best I can tell. I think row 8 is also related to the aria stuff. I'm a little out of my element in the a11y testing....Can you take a look and see if these need to be addressed? Thanks!


Matterbridge crashes because of Mattermost

Describe the bug
Bridging IRC to Mattermost and Discord, the bridge regularly crashes because of what seems to be a Mattermost related issue.
To Reproduce
Connect to Mattermost server (Team Edition 0.15.0, but also happend also with previous versions) and wait a few hours/days for it to crash.
Expected behavior
Should not crash :)
Screenshots/debug logs
time="2019-10-03T00:38:05Z" level=error msg="Connection is not alive: &model.AppError{Id:"model.client.connecting.app_error", Message:"model.client.connecting.app_error", DetailedError:"Get https://mattermost.example.com/api/v4/users/me: net/http: request canceled (Client.Timeout exceeded while awaiting headers)", RequestId:"", StatusCode:0, Where:"https://mattermost.example.com/api/v4/users/me", IsOAuth:false, params:map[string]interface {}(nil)}" prefix=matterclient

time="2019-10-03T06:59:39Z" level=error msg="error:read tcp <IP_ADDRESS>:32868-><IP_ADDRESS>:443: use of closed network connection" prefix=matterclient

time="2019-10-03T07:12:13Z" level=error msg="SendMessage failed: : Invalid or expired session, please login again., " prefix=gateway

Environment (please complete the following information):

OS: Official Docker images
Matterbridge version: 42wim/matterbridge:latest

That's an issue with sessions on the mattermost side.
Look at https://forum.mattermost.org/t/solved-removing-the-session-timeout/3033 for a solution or use personal tokens, those don't expire. See https://docs.mattermost.com/developer/personal-access-tokens.html

@42wim strangely even after switching to a personal access token this error persists.

@poVoq if you get the Invalid or expired session, please login again message when using personal tokens, please open an issue at https://github.com/mattermost/mattermost-server


Datetime CAST error

When testing a Stream Analytics Query locally, it fails when trying to cast an ISO datetime string to datetime type:
Cannot cast value '2020-01-06T10:10:343.4589604Z' of type 'nvarchar(max)' to type 'datetime' in expression 'CAST ( [timestamp] AS datetime ).
Complete Query:
SELECT
    COUNT(*) AS eventCount,
    TopOne() OVER (ORDER BY CAST([timestamp] AS datetime) DESC) AS LastEvent,
    unitID
  FROM iothub
  GROUP BY unitID, HOPPINGWINDOW(second, 15, 1)

Isn't the string in the correct format? When leaving the CAST out, it says that ordering over nvarchar(max) is not possible.
I'm developing with Visual Studio Code on Ubuntu 18.04 LTS.

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

ID: 12eb08ab-be52-29e2-e0ff-76727c867c9e
Version Independent ID: 6849d9fa-653c-701d-d8ac-964e98c5370d
Content: CAST (Azure Stream Analytics) - Stream Analytics Query
Content Source: stream-analytics-docs/cast-azure-stream-analytics.md
Service: stream-analytics
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

@AlexR please take a look when you have a moment. Thanks! Jason

Hi @awoSYS! You are receiving this error because the value is not a valid datetime.
2020-01-06T10:10:343.4589604Z has 343 seconds, but the seconds place should only have two digits. Removing the extra digit should resolve the error.

I hope this information was helpful! I will now proceed to close this thread. If you have any additional questions, please reopen the issue.
#please-close


Display of cities, Bundesland and Germany at the same time

Is it possible to display a line for the Bundesland and for Germany as a whole in addition to individual cities?
I briefly looked in the code, but did not find a quick way. Otherwise I would have started a pull request right away ;-) I'm sure you're better at the code, so maybe it's not so complicated.
It would be enough for me if one state (Bavaria) and Germany as a whole could be displayed additionally to three cities.
Thanks for this great module!

Unfortunately it is not possible as the data source is not providing Bundesland and total Germany data.
For this data you might need to add an additional module that gives more macro orientated data such as MMM-COVID.

If you want to have the Bundesland only, please simply add the module twice and disable the headers. This should do the trick. But unfortunately the data source has no data point for Germany as a whole.

Hi
unfortunately this is not the solution. The module then shows my three cities in the first instance and Bavaria three times in the second, even if I specify only one city there.
Nevertheless, thanks for the effort and for thinking.


Could I decode a QR in a esp32 with only 4MB of code space?

Interesting question, although the size of the binary should not be an issue at all, I believe. The more limiting factor is most likely the RAM size and the size of your images. Do you have concrete numbers for those?

I've already spec'd out the sram.  I only need to know about code size.
Since I asked the question I found code that runs in my esp32: https://github.com/alvarowolfx/ESP32QRCodeReader
So I've answered my own question.  But mucho thanks for your reply.


Grafana Dashboard: Change Fahrenheit to Celsius?

Is there an easy and simple way to switch from Fahrenheit to Celsius or do I have to change the items all one by one?
(By the way: there is a missing "h" in "Fahrenheit".)

This is a grafana specific question, the templates are only starters, so yes you would need to change each one

It is clear that it is Grafana. But since the users of these templates are here, there is also a high probability that someone here knows the solution.
If anyone has the same question:
I have meanwhile found out that both Fahrenheit and Celsius are included in the templates, you can show or hide them in edit mode.
Finally, thank you very much for nothing, closing my question in this arrogant way with false information.

I'm glad you found the edit and the hide buttons in Grafana.
This isn't a community support portal and no one here owes you anything.

I fully agree with the content of the replies. But it's a question of friendly interaction. Nobody is forced to answer. You should either answer in a friendly manner or not at all.
All the best for you

I don't think you get to decide how we handle our property. No one comes to your house and tells you how you need to treat solicitors.
No one was rude to you. No one called you names. No one lied to you or provided you false information. If you don't like the answer you shouldn't respond.


removed gotos from fpfrno

@perazz I think this is right but double check!
Note: I didn't know what to name that extra loop (just used inner but maybe there is a better name?)

Looks great @jacobwilliams, thanks a lot! Much appreciated.


+coinmarketcap and coincap feeds for ALTCAP(.X) V2

Patched to give publish the correct number for pricefeed. Doesn't look to be correct because of issue #33.

could you please direct pull requests to the develop branch instead?