NewsAPI Element

Creating a pull request to add the NewsApi Element JSON and ReadMe.

@nick-bair
This was discussed via email 1) on the /everything endpoint it was weird that pagesize was required. Typically that is not a required parameter so I changed it to not be required and found it worked so I would make that optional
2) It is unclear how to use the q in the where filter on /everything endpoint as well. Should it look like where='q="something" ' ? I was having trouble trying to query a publishedAt date.
3) In the future consider making a CE usable sandbox and putting it in one pass for testing purposes but I went ahead and made a free trial to get an api key so it was not a blocker just an extra hurdle
These changes were made and it should be good to merge I would do one last check to make sure that the files are named to work with the import functionality

Hi @nick-bair , I've added a commit with the requested changes, please advise if there is any other action required from me. Thank you!

Hi @George-Novac, awesome thanks!  One last ask, can you delete .DS_Store from your PR?  You may need to delete it, then add a .gitignore with the following:
.DS_Store
.gitignore

Let me know if that doesn't make sense or if you need some help.
Thanks

Hey @George-Novac closing out this PR so that you can re-initiate it from the CE UI (this will be the standard process going forward).  @kiawnna can you share instructions with George directly as he is not in the UTS channel?  Thanks


refactor preprocessing from regex to XSLT

Easier to maintain, more powerful and better suited for the task.

The version pushed is currently untested.
Also, I don't know how to do preprocessing step 3, copying the first coordinate of the arrow head and make it the last coordinate of the arrow body.

I will look into it today!

@Yagnap: Sorry for the delay, I will look at it now.

Why use XSLT version 1 from 1999? I suggest using the newest XSLT version 3 from 2017.

Sounds good, if I read that correctly this would also solve our regex issue, as I believe XSLTv2+ supports regexes

How do you validate and execute the XSLT file?
I tried gui$ xsltproc scripts/preprocessing.xsl img/tmp.svg -o img/test.svg but I get:
gui$ xsltproc scripts/preprocessing.xsl img/tmp.svg -o img/test.svg                                          redesign
scripts/preprocessing.xsl:8: parser error : XML declaration allowed only at the start of the document
		<?xml version="1.0" encoding="UTF-8"?><svg xmlns="http://www.w3.org/2000/svg"
		     ^
scripts/preprocessing.xsl:35: parser error : Opening and ending tag mismatch: svg line 8 and xsl:template
	</xsl:template>
	               ^
scripts/preprocessing.xsl:36: parser error : Opening and ending tag mismatch: template line 6 and transform
</xsl:transform>
                ^
cannot parse scripts/preprocessing.xsl

Ugh, I got a merge conflict but my changes weren't much so I am reverting it to get on the same state as you. I've just added auto formating to the XSLT file with the "npm run fmt" script so we get less conflicts in the future, however I advise pulling this in as soon as possible because the reformating can cause merge conflicts right now.

I will continue this today.

You can now start the preprocessing with node node/preprocess.js after npm update to get the new developer dependencies.
The XSLT itself still needs to be expanded.

Steps

[ ] 1. round x,y,height and width, e.g. 5.831902424 -> 5.0
[ ]  2. arrow head/body declaration
[ ] 3. hyperlinks to ids, remove links
[ ] 4. viewport
[ ] 5. add arrow body id = edgeId + "ArrowBody"
[ ] 6. generic defs add y="5"

@Yagnap: Do we actually need the y="5" for the generic defs?


[prometheus] rename metrics options

Description
For this plugin to work, each metric the user want to be exported should be manually enabled using the corresponding option. But the name of the exported metric and its corresponding option are different, which is confusing for users.
This PR renames all the options to use their actual metric name.
Type of change
Please delete options that are not relevant.

[x] Breaking change (fix or feature that would cause existing functionality to not work as
expected)
[x] This change requires a documentation update

The last commit is a suggestion, perhaps it could be better to separate metrics enabling/disabling options from the rest, just as we do for labels.
It looks better for types and will be easier to explain in documentation I think.


Refactor flask rest x

Made the following changes

Update codecov since the current system was not properly producing code cov reports for pull requests
Changed BDD step files to use new /api/orders over /orders
Changed service/static/js/rest_api.js to use new /api/orders over /orders
Updated poetry file to import flask-restx
Added API to service/__init__.py
Pruned Error Handlers (Since they're not needed after Flask-RestX refactoring)
Refactored service/routes.py for flask-restx

Added model definitions for orders and items
organized orders and items into API rest-flaskx system

OrderResource (Get(Read), Put(Update), Delete(Delete)
OrderCollection (Get(List), Post(Create))
ItemResource (Get(Read), Put(Update), Delete(Delete)
ItemCollection (Get(List), Post(Create))
CancelOrderResource
DeliverOrderResource
PackOrderResource
ShipOrderResource

Adjusted tests/test_routes.py as needed

Need a look at
4 BDD tests are failing, specifically:
Query Items
Read an Order
Read an Item
Cancel an Order

looking into this

Set the Code Coverage back to version 3.1.4

some BDD test cases were being affected by random dummy values added during manual testing. hence updated the test cases to be independent of such scenarios.

corrected typo in queryString for querying items by product_id

re-added code to sort orders by order_date

All BDD tests are passing.

All TDD tests are passing.


Restore previous state of light when adaptive lighting is turned off

I would love to have the optional feature to restore a light to its previous state when adaptive lighting mode is turned off. This would be really useful for those of us who like to switch between using their lights for fun (like when watching a movie or relaxing) and using them for utility (like cleaning or eating). It's annoying to have to reset the scene every time.
Expected Behavior

Adaptive lighting is not currently turned on for my room. I change the scene for the room. I have a green, blue, and pink light, all at 50% brightness.
I turn on adaptive lighting. The lights update their settings to the current adaptive lighting color/brightness levels.
I turn off adaptive lighting. The lights revert back to green, blue, and pink, and 50% brightness.

I suppose I could do this by creating an automation that saves the state of the room lights, then turns on adaptive lighting, and restores said state when adaptive lighting is turned off, but having it as a native option would be really nice.

Another vote here. I would love to have a way to toggle all lights that are currently on, to 100% brightness and disable adaptive control. For when I am looking for something, cleaning, or need the light.

https://www.home-assistant.io/integrations/scene/#creating-scenes-on-the-fly

https://www.home-assistant.io/integrations/scene/#creating-scenes-on-the-fly

That's what I'm currently doing, but I think in this situation it would make sense to have it as an option when setting up a group.

This feature is very unnecessary.
The correct sequence of events you're looking for:
> ensure you have take_over_control: true set in the adaptive-lighting config
> turn on adaptive-lighting's main switch
> change the scene for the room to your green, blue, and pink light, at your desired 50% brightness.
at this point adaptive-lighting will fire a manual control event for that light group, and will stop adapting it until you call the service adaptive_lighting.set_manual_control with the service data manual_control: false (or just turn off and on the main adapt switch)

Stale/duplicate - closing.


The Control is not made available

I have installed the ColorfulOptionsetGridSol_managed.zip solution on my DEV org (online).
Everything went ok but the control is not in the available ones for an option set field.
I get only only 4 but yours was not between them.
Anything else I should to make it work?
Regards and thank you!

Hi  @Octavianbr ,
What you are looking for is the ColorfulOptionset PCF (https://github.com/ORBISAG/ORBIS.PCF.ColorfulOptionset). There you will find the PCF who can be used as an Optionset Control.
This repository is the ColorfulOptionsetGrid, which is a dataset PCF, who can be attached on views or entitiy grids.
Kind regards,
Diana


ERROR: [PornHub]<PHONE_NUMBER>: Unable to download webpage: Remote end closed connection without response

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
ERROR: [PornHub]<PHONE_NUMBER>: Unable to download webpage: Remote end closed connection without response
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
ERROR: [PornHub]<PHONE_NUMBER>: Unable to download webpage: Remote end closed connection without response (caused by RemoteDisconnected('Remote end closed
 connection without response')); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template.

Post a full verbose log

/usr/local/bin/yt-dlp -vU https://www.pornhub.com/view_video.php?viewkey=6411d10dc49b4
[debug] Command-line config: ['-vU', 'https://www.pornhub.com/view_video.php?viewkey=6411d10dc49b4']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (zip)
[debug] Python 3.10.0 (CPython x86_64 64bit) - Linux-5.10.0-21-amd64-x86_64-with-glibc2.31 (OpenSSL 1.1.1n  15 Mar 2022, glibc 2.31)
[debug] exe versions: ffmpeg N-62475-g130d19bf20-static (setts), ffprobe N-62475-g130d19bf20-static
[debug] Optional libraries: no_Cryptodome-None, sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 91cad9f121c1f6f0a81b747415c46ecba0ff331ed38cc6433040b4ac7b6e15ca
yt-dlp is up to date<EMAIL_ADDRESS>[PornHub] Extracting URL: https://www.pornhub.com/view_video.php?viewkey=6411d10dc49b4
[PornHub] 6411d10dc49b4: Downloading pc webpage
[PornHub] 6411d10dc49b4: Downloading tv webpage
ERROR: [PornHub] 6411d10dc49b4: Unable to extract encoded url; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 694, in extract
ie_result = self._real_extract(url)
File "/usr/local/bin/yt-dlp/yt_dlp/extractor/pornhub.py", line 403, in _real_extract
js_vars = extract_js_vars(
File "/usr/local/bin/yt-dlp/yt_dlp/extractor/pornhub.py", line 340, in extract_js_vars
assignments = self._search_regex(
File "/usr/local/bin/yt-dlp/yt_dlp/extractor/common.py", line 1242, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)


[stable/prometheus-operator] needs higher container cpu limits or need customization parameters

Is this a request for help?:

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
BUG REPORT

Version of Helm and Kubernetes:
current
Which chart:
stable/prometheus
What happened:
Frequent CPUThrottlingHigh alerts on the following pod.containers:
alertmanager-prometheus-prometheus-oper-alertmanager-0.config-reloader
prometheus-prometheus-prometheus-oper-prometheus-0.prometheus-config-reloader
prometheus-prometheus-prometheus-oper-prometheus-0.rules-configmap-reloader
There is no way to raise the current hardwired cpu limits (100m, 50m, and 100m) on them.
What you expected to happen:
Need a way to raise the cpu limits on them when need.  for instance, we estimate that we would need to raise the cpu limites to 200m, 150m, and 200m respectively to eliminate the frequent CPUThrottlingHigh alerts on them.  There are other hardwired resource limites generated by the operator that may need similar customization support.
How to reproduce it (as minimally and precisely as possible):
This seems a self-evident design issue.
Anything else we need to know:
This is similar to #9540 but different.
We observed this CPUThrottingHigh alert while using stable/prometheus 4..0.0 and 4.1.1.
The premetheus-operator itself must also make a change either to raise the hardwired cpu limits, or add support for the cpu limit customization.

@wu105 This change was made possible by operator 0.29.0 and there's an open pull request to get this change merged into the chart right now.


How can I remember the last bluetooth device connected?

I'm trying to develop something that needs to connect to the device the first time, then remember the choice and connect-it automaticaly. How can I do that?
I've been seen that flutter shared preferences is an option but we can't save objects so not possible for my case.
Any suggestion?
Thanks so much!

Our friend here has a good example https://github.com/moti0375/FlutterBDT


feat: add Stripe checkout integration

What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Description

This PR:

Replaces the waitlist button on the reports page with a button to subscribe with Stripe Checkout
The button initiates the subscription checkout flow to Stripe
Needs https://github.com/open-sauced/api.opensauced.pizza/pulls to  to be merged/deployed/configured
Needs better copy

Related Tickets & Documents

#394
Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

LGTM

I get this error

https://deploy-preview-702--oss-insights.netlify.app

The whole flow is working in the preview environment now. The only issue is that when you finish the checkout process it redirects you to beta instead of back to the deploy preview, which is fine for testing as the deploy preview uses the same data.


new values for Songbird Sources

I'm trying to setup 'multiple' rooms, by changing the sounds and room sizes when certain triggers are hit.
Changing the room dynamically works well, setting new Materials and Dimensions.
But swapping out the sounds (sources) and their properties doesn't seem to work. My idea was to setup the DOM just one time with an array of generic AudioElements etc. Then, each time the room changes, update the mp3 URL, the Gain, the Maximum Distance, etc. The sounds do get added into the Room but they're all equally loud, like they're just 'inheriting' the properties from the previous generic setup.
If you try to create new Sources and splice them into the AudioSources array, any sourceOptions you feed them are ignored (e.g. I tried setting Gain and Max distance to 0).
So for now I've resorted to creating a new Songbird instance each time, a new array of AudioElements, AudioSources, etc, but I can't imagine this is good for memory or the ideal approach.
Ideally Sources would have the same method as a Room, e.g. SetSourceProperties(maxDist, gain, etc etc).
Any help appreciated. :-)

Also, you can change the source's properties directly but yes, we should expose the individual properties like you mention. I'll do that when I return to work. :) (Currently on a week vacation).

Thanks for response (on your holiday no less). ;-)
The code I'm working with now is a mess so I've done a new minimal test showing what happens, based on your room model example.
Just realized that the Inspector shows the values (e.g. position, maxdist) do in fact change, but there is no change in the audio.
But I'm not really a programmer, so I could very well be missing something obvious. Any help appreciated.
`var songbird;
// Define source audio files.
var audioSources = [
'resources/CubeSound.wav', 'resources/SpeechSample.wav'
];
// This is used to scale the appropriate dimension.
var materials = {}
var dimensions = {
  width : 4,
  height : 4,
  depth : 4
}

var audioElements = [];
var songbirdSource = [];
var audioElementSources = [];
var context;

var onLoad = function () {

  context = new AudioContext();
  audioElementSources = Array(audioSources.length);

  // Create <audio> streaming audio sources.
  for (var i = 0; i < audioSources.length; i++) {
    audioElements[i] = document.createElement('audio');
    audioElements[i].src = audioSources[i];
    audioElements[i].load();
    audioElements[i].loop = true;
    audioElements[i].play();
    audioElementSources[i] =
      context.createMediaElementSource(audioElements[i]);
  }

  // Initialize Songbird and create Source(s).
  songbird = new Songbird(context, {
    ambisonicOrder: 3,
    dimensions: dimensions,
    materials: materials
  });
  for (var i = 0; i < audioSources.length; i++) {
    songbirdSource[i] = songbird.createSource();
    audioElementSources[i].connect(songbirdSource[i].input);
  }
  songbird.output.connect(context.destination);

};

// ini basic songbird with two sources
window.addEventListener('load', onLoad);

function updateSources() {

	for (var i = 0; i < audioSources.length; i++) {

		// way #1, alter properties directly
		songbirdSource[i].maxDistance = 0.1;
		console.log(songbirdSource[i]);
		// way #2, create new sources and put in old array
		 var sourceOptions = {
		  position: [-3, 0, 3],
		  forward: [0, 0, -1],
		  up: [0, 1, 0],
		  minDistance: 0.1,
		  maxDistance: 0.1,
		  rolloff: 'logarithmic',
		  gain: 0,
		  alpha: 0,
		  sharpness: 1,
		  sourceWidth: 0
		}

		songbirdSource[i] = songbird.createSource(sourceOptions);
		console.log(songbirdSource[i]);
	}

}

// now update sources
setTimeout(updateSources, 2000);`

Hi Luke,
Currently, songbird sources don't allow the direct updating of attenuation parameters like minDistance via the way you're trying to invoke (in fact, the property .minDistance is being created in your code, but this is because the assumed use case for minDistance/maxDistance/gain/etc. will be mostly constant over the lifetime of the source. To change the sound quality, you should adjust the position (via source.setPosition() which does most of the heavy lifting to change where the sound appears to come from. Also, you'll want to use songbird.setListenerPosition() to change the position of the listener as well, and the relationship between the source position, listener position and their relative positions in the room (which is centered around the origin 0,0,0) produce the overall effect.
For more detailed information about how the current API works, please consult the documentation: https://cdn.rawgit.com/google/songbird/master/doc/index.html
If there are still functions you feel would benefit you outside of the those provided, we can certainly consider introducing them.


Fix all dep and vendor issues

dep ensure will now world properly. Also boilerplate was moved into a local file and no longer sites "Kubernetes Authors"
To make this PR readable, i have not comitted the make generate or update vendor directory.

Thanks for this, and sorry for the delay in taking a look at your PR!
Could you add the results of make generate to this PR as it's own standalone commit, as well as the vendor directory in it's own commit too? Best to keep everything in sync :)

I've now merged #7 which encompasses the changes here too. Thanks for the work though, and once again sorry it took me a while to get back to you!


Down key bug

Down key bug when the cursor is at the middle, and scroll down for the emty command.
https://user-images.githubusercontent.com/26076131/190366032-aa1c2bbd-16e6-437e-b65b-dcf24024df6b.mp4

Fixed in badc44d995859c7b050912d2cbe9299a4f19e4bb


How accurate is the conversion of text content into knowledge graphs based on large models?

Describe the issue
I dont know it .if we use general llm
Steps to reproduce
No response
GraphRAG Config Used
No response
Logs and screenshots
No response
Additional Information

GraphRAG Version:
Operating System:
Python Version:
Related Issues:

agree, the processing progress should be more visiable and analyzable to make adjustment decisions.


Implement CSV input plugin for autoinvest on Binance.com

Autoinvest transactions can not be retrieved via the REST API and will need to be processed via CSV. They currently have no plans to implement this via API either.
For now, this is low priority. I'm hoping they implement something via the API by the end of the year, so that we don't have to do this twice.

This needs to be implemented along with BETH to ETH conversion in order to completely import all Binance.com data.

Resolved with #69


Bump libovsdb to 0.3.1

This PR modifies source of libovsdb from github.com/socketplane/libovsdb to github.com/ovn-org/libovsdb because the former was removed.
This PR also bump the libovsdb version to 0.3.1 because v0.1 and v0.2.0 causes error on make dep as follows. And 0.4.0 and 0.5.0 requires go1.16.
libovsdb v0.1
$ make dep
/home/honma/git/ovs-cni/build/_output/bin//go//bin//go mod tidy
go: errors parsing go.mod:
/home/honma/git/ovs-cni/go.mod:13: no matching versions for query "v0.1"
make: *** [Makefile:89: dep] Error 1

libovsdb v0.2.0
$ make dep
/home/honma/git/ovs-cni/build/_output/bin//go//bin//go mod tidy
go<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: github.com/ebay/libovsdb
	        but was required as: github.com/ovn-org/libovsdb
make: *** [Makefile:89: dep] Error 1

release-note
Bump libovsdb to 0.3.1

Signed-off-by: Masashi Honma<EMAIL_ADDRESS>

Hi !
I am a beginner of ovs-cni.
I already passed all the test of make test.
Is there any other test I should run ?

/ok-to-test
Hello, welcome to the project! Let's see what CI says. Apart from integration tests you've already run, we also have end-to-end test suite running on a Kubernetes cluster. You can run it locally via make cluster-up cluster-sync functest

@phoracek

You can run it locally via make cluster-up cluster-sync functest

Thanks ! I will try it.

pull-e2e-ovs-cni failed with lint. Sorry, I forgot to run make lint. I will fix it tomorrow.

Fixed.

/test pull-e2e-ovs-cni

Just now I recognize one mistake, I will fix it.

Ok. Fixed.


WSL is not recognized as the name of a cmdlet, etc

Windows 10 20170 ARM64
Device: Microsoft Surface Pro X
WSL Version: 2
I'm trying to run wsl from powershell, but can't get it working. It always says that the file wasn't found. It works perfectly with cmd. System32 folder is in path and wsl.exe exists.
Even if I point directly to the file, it is totally ignored.

Get-Childitem also ignores wsl.exe:

This is insane....

It's visible and executable in explorer and cmd, but completely ignored by powershell

".\wsl.exe" might not be valid PS syntax, it should start just by typing "wsl" as this is linked command.

".\wsl.exe" might not be valid PS syntax, it should start just by typing "wsl" as this is linked command.

Just typing "wsl" also says that the command wasn't found

Usually means a 32-bit powershell.
/dupe #4660, #4259, others


Fix additional issues with cyberware

Mainly:

[x] biomonitor status is checked from equipment every warning attempt
[x] metabolic editor and detoxifier flags must be refreshed on attach
[x] symptoms appellations must be updated
[ ] chemical names must be updated for healers: search from lore

A quick search on forum for game's lore + google seems to indicate:

nano surgeons
adrenaline: Epinephrine (C9H13NO3)
painkillers: Paracetamol (C8H9NO2)

Since it seems nanosurgeons being lost lore, it would be funny to get nanites traces in higher tier of MaxDOC, BounceBack and the likes.

As of NeuroBlocker:

result is not perfect as there's trailing commas and ellipsis, but it'll do for now.


onSwipeValueChange on SwipeRow is only called once

When I implemented onSwipeValueChange on my individual SwiperRow component, it is only called once when I swipe right, or left. This is pretty much useless as I need the X movement in real time.
When onSwipeValueChange is implemented on the SwipeListView, this function is called multiple times per second which is the correct behaviour.
Am I missing something on the onSwipeValueChange implementation? Here's a snip of my code below:
<SwipeRow
    leftOpenValue={Dimensions.get('window').width}
    rightOpenValue={-Dimensions.get('window').width}
    onSwipeValueChange={this.onSwipeValueChange}>

    <HiddenView/>
    <VisibleView/>

</SwipeRow>

Hey @conradjonp. Nope not missing anything. Just double checked and everything seems to be working properly... anything else you could tell me to help diagnose this? Platform, version numbers, stuff like that? Thanks!

@jemise111 thanks alot for checking, I figured out what was wrong, it was a mistake on my side, apologies :)


[P2] Chart. NT implement option to change date period

There should be option to look statistics through the last 7 days or change it to the whole period again

Added period picker to the chart and combined its output with journal as well.
Moved chart and journal to the different views


Docs for Client::get and similar methods say it 'executes' a request

... but of course it just builds a request to be executed later.
http://hyperium.github.io/hyper/hyper/client/struct.Client.html#method.get

Good point!


Add stateful dl import to smoke test

title
Fixes #{issue number}
Changes

@atalman Is that change what you had in mind? Can you also confirm if it is fine to land this in main branch (with the check being about 0.8.0) and the cherry pick this to release/0.8 branch?


fix: Fix issues with UDTs for Kusto Cluster module

Description
Resolves issues with invalid user defined types and parameter values for the specified API version.
Pipeline Reference

Pipeline

Type of Change

[ ] Update to CI Environment or utilities (Non-module affecting changes)
[ ] Azure Verified Module updates:

[ ] Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[x] Breaking changes and I have bumped the MAJOR version in version.json.
[ ] Update to documentation

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] I have run Set-AVMModule locally to generate the supporting module files.
[ ] My corresponding pipelines / checks run clean and green without any errors or warnings

Hey @AlexanderSehr, is anything in my PR sticking out to you by chance that could be causing my deployments to hang? I am getting this error in the deployment logs for the kusto cluster logs.
``
{
"status": "Failed",
"error": {
"code": "InternalServerError",
"message": "Encountered internal server error. Please try again."
}
}


Export as a GIF

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Feature
The drawio currently supports Flow animation which is great. However, to fully utilize these animated diagrams outside of the Draw.io platform, there is a need for an "Export as GIF" option.
Benefits:

Improved Communication: Animated diagrams can significantly enhance the clarity and impact of ideas, making them ideal for presentations, educational content, and project documentation.

Simplified Sharing: Exporting diagrams as GIFs would allow users to seamlessly share animated content with colleagues, clients, or stakeholders who may not have access to the Draw.io web app.

Requested Functionality:

Export Options: Upon selecting "Export as GIF," users should have the ability to customize the size, frame rate, and quality of the exported GIF to suit their specific needs.

Animation Preservation: The exported GIF should retain the same fluidity and smoothness of the original "Flow animation" from the Draw.io canvas.

Thank you for considering this feature request. Your efforts in making Draw.io even more functional and user-friendly are greatly appreciated.
Best,
Saman Saybani

Thanks for your suggestion. Having reviewed the feature request the effort to implement it is excessive for the benefits and so we won't be implementing it.

Work around (not ideal) - capture animation sequence as video (e.g. obs) then use ffmpeg to convert to animated gif: https://homehack.nl/create-animated-gifs-from-mp4-with-ffmpeg/

Even easier solution. Install this: https://www.screentogif.com/ - Super easy to use

I exported to SVG:

Then used https://html5animationtogif.com/svg-to-gif (which is quite good, it allows to select specific frames for the start and end, so the result is only 4-5 frames):

I would love to see this feature implemented!
The diagrams with flow are much more impressive!


Resource routes break when exporting ErrorBoundary in v2

What version of Remix are you using?
2.0.1
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce

create a resource route containing loader export
export ErrorBoundary from the same route

Expected Behavior

remix should ignore ErrorBoundary (as in v1) as it doesn't make sense if no react component structure gets returned
optionally display warning

Actual Behavior
A document page gets returned containing a parent route or empty page (if the parent is root) and the server prints:
Matched leaf route at location "/path/to/resource/" does not have an element or Component. This means it will render an <Outlet /> with a null value by default resulting in an "empty" page.
Looks like when ErrorBoundary is exported, remix expects default export to be present and renders classic route page.

See https://github.com/remix-run/remix/issues/6106

Interesting, thanks! In that case this is a breaking change, would be great to mention it in the docs at least, thank you for the context!

although maybe not breaking if the ErrorBoundary was there by mistake in the first place 🤔

PR with the docs update: https://github.com/remix-run/remix/pull/7539

Thank you for the docs update!
Just for clarity, this was a bug fix in v1, not a breaking change (unfortunately, every bug fix is potentially a breaking change for someone if they were relying on broken behavior 😉 ).  Resource routes are documented as "not part of your application UI" which used to incorrectly only look for an exported default UI component, but should have always looked for an exported ErrorBoundary as well since that means the route has UI associated with it.


Add ability to evict specific candidate cookies conditionally

Context
As part of the incident 240306 we need to be able to guarantee that the users who hold cookies that allow them to access accounts they do not belong to have their access removed.
We can do this by targeting them by updating their cookie expiration to a time that logs them out, in conjunction with a Feature Flag.
On every request, the application checks the last request time of the user to see if it’s been too long since their last request and if it is, they are logged out.
Changes proposed in this pull request
Add Warden::Manager callback on after_set_user to evict any Candidates whose last_request time is before the activation of the Feature Flag. Any candidates with requests after the activation time are not evicted
Guidance to review
Thorough testing required
Link to Trello card
Trello Ticket

 Incident 240306: Evict ghost candidate sessions

LGTM, although this feature has a limited life span it's good to keep coding standards high so glad we've identified and fixed the issues in recent commits.

Code looks good! One question (not blocking) is if we should add some logging like "Candidate 1 got expired" or something similar?

Nice idea. Added with https://github.com/DFE-Digital/apply-for-teacher-training/pull/9185/commits/34c7a58c10cee3bb1da9278f2318ad74b3cf19dc

Code looks good! One question (not blocking) is if we should add some logging like "Candidate 1 got expired" or something similar?

Nice idea. Added with 34c7a58

Awesome 🚀


DSGM Fade In and Out

In DSGM_FadeOutToBlack() and DSGM_FadeOutToWhite(), my understanding is that the function checks if brightness==0 and then performs the while loop, but this may not be the case as the user may want to manually set the brightness at some point using DSGM_SetBrightness() to something other than 0 during the game. The if statement should not be here in this case, just perform the while loop normally (why restrict the user to only fade at brightness 0?).
Also, why is the brightness read using DSGM_BOTTOM? Shouldn't it be 'screen'? I thought DSGM_BOTTOM is a preprcessor macro for 0 (bottom screen), no?
(Also, nice to see you again. Hope everything is well.)

Thanks for the report, Baron!
I've fixed this as of commit: https://github.com/CTurt/dsgmLib/commit/4e57cf65f93c8c585ddc558945555105fc3c009d


Multi-Module / Visibility

When using multiple modules and only having protocols (and not concrete implementations) available, the compile-time safety of factory cannot be sustained.
We have a package "Domain" which has access to a UserRepositoryProtocol. Another package "Data" contains the UserRepository implementation. Data depends on Domain.
When defining the DI-Container in Domain, we have to use
public static var userRepositoryFactory: Factory<UserRepositoryProtocol>!

And in Data
DomainContainer.userRepositoryFactory = DataContainer.userRepository

Is this the suggested solution for multi module projects?

I believe this was mentioned somewhere, but my recommendation would be...
public static var userRepositoryFactory: Factory<UserRepositoryProtocol?> { nil }

And then registering a new service elsewhere...
DomainContainer.userRepositoryFactory.register { UserRepository() }

Yes, it's optional, but it's also safe should someone, somewhere, fail to provide the needed registration.

Thanks for your suggestion, that seems a little cleaner than my approach.
In hindsight, it's common that DI Frameworks produce runtime exceptions when services are not registered, so i'll go with
public static var userRepositoryFactory: Factory<UserRepositoryProtocol!> { nil }

"it's common that DI Frameworks produce runtime exceptions when services are not registered"
Yeah, but avoiding that is why I created Factory in the first place....


PCA block size optimizations

PCA block size optimizations: fit and transform. Covariance optimizations were created for fit stage optimization.

/intelci: run

/intelci: run


Programmatically switch off API

What I would like to do is check certain conditions when an call is made and based on those either pass the request to the in memory web api, or pass this request to a real backend api.
The request would be the same, let's say 'api/user/1' gives me info of user 1.
When the user is not logged in, I'd like to have the in-memory-web-api handle the request, when the user is logged in then the real backend should process it.
A solution could be to add the option 'disable' (default: false) to InMemoryBackendConfigArgs and add the relevant logic to the code.

I don't think that would work because configuration happens once. I'm also reluctant to add more and more switches for rare corner cases.
I also don't understand why you want this feature. You never want the in-mem web api in production.
You can handle this yourself I suspect by creating your own subclass or wrapper of the in-mem web api.
While I appreciate your affection for this library, I'm afraid I'll have to close this issue because I have no intention of acting on it. Sorry.


Optimize lambda functions

Currently, lambda functions are slow. (Almost same as normal user functions.)
If a lambda is used in map(), it is about 7x slower than a string expression,
and about 3x slower than a built-in function.
Benchmark script: https://gist.github.com/k-takata/fa07f1382b2ce31ed06f6eeac34236cd
Lambda:   2.387348 sec
String:   0.351491
Built-in: 0.810164

This makes lambda functions about 2.8x faster.
(Still slower than string expressions, though.)
Lambda:   0.849935 sec

Hmm, it seems that there are some problems...

Fixed. Not sure why lgtm-com complains.

Currently, lambda functions are slow. (Almost same as normal user functions.)
If a lambda is used in map(), it is about 7x slower than a string expression,
and about 3x slower than a built-in function.
Benchmark script: https://gist.github.com/k-takata/fa07f1382b2ce31ed06f6eeac34236cd
Lambda:   2.387348 sec
String:   0.351491
Built-in: 0.810164

This makes lambda functions about 1.5x faster.
(Still slower than string expressions, though.)
Lambda:   1.580712 sec

When looking into make Vim script faster, it's the creating of the
arguments and the local dict that add a lot to the overhead.
It might be feasible to check if the a:0 and a:000 variables are used.
If not, then the a:000 list can be omitted.
Since the arguments are added to the l: dict, perhaps the a: dict is
unused?
"firstline" and "lastline" should be unused.
--
If Microsoft would build a car...
... the oil, water temperature, and alternator warning lights would
all be replaced by a single "General Protection Fault" warning light.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I tried removing a:fistline and a:lastline and it makes the bench mark about 4% faster.
Checking if a:0 and a:000 are used might be not so easy. I removed a:0 and a:000 without checking as a trial and it makes about 4% faster.
I also found that call_func() can be optimized. If partial->pt_func is not NULL, there's no need to lookup a function from the name. This makes about 6% faster.

Now the benchmark is about 1.33 sec.

I tried removing a:fistline and a:lastline and it makes the bench mark about 4% faster.
Checking if a:0 and a:000 are used might be not so easy. I removed a:0 and a:000 without checking as a trial and it makes about 4% faster.

How about a strstr() call?  Would be difficult to use a:000 without a
match.

I also found that call_func() can be optimized. If partial->pt_func is not NULL, there's no need to lookup a function from the name. This makes about 6% faster.

I wonder if it's possible to drop the whole a: dictionary.  The
arguments are added to the l: dict.
For Vim9 script we can probably compile the expression and use
call_def_function().
--
I'm so disorganized my keyboard isn't even in alphabetical order!
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

How about a strstr() call? Would be difficult to use a:000 without a match.

Ah, searching "a:" is easy and enough, even it is not perfect.

I wonder if it's possible to drop the whole a: dictionary. The arguments are added to the l: dict.

I think initializing the a: dictionary (init_var_dict()) is still needed, but we can stop adding a: variables.
I implemented this and the benchmark is now 0.83 sec.
Now the speed of lambdas is almost same as built-in functions, and only about 2.4x slower than string expressions.

For Vim9 script we can probably compile the expression and use call_def_function().

What do you think of closures? I think lambdas without closures might be not so useful.

Codecov Report

Merging #5727 into master will increase coverage by 0.04%.
The diff coverage is 97.29%.

@@            Coverage Diff             @@
##           master    #5727      +/-   ##
==========================================
+ Coverage   84.06%    84.1%   +0.04%
==========================================
  Files         138      138
  Lines      152376   152459      +83
==========================================
+ Hits       128088   128232     +144
+ Misses      24288    24227      -61

Impacted Files
Coverage Δ

src/userfunc.c
91.07% <97.29%> (+0.23%)
:arrow_up:

src/ui.c
71.29% <0%> (-1.12%)
:arrow_down:

src/gui_gtk_x11.c
58.06% <0%> (-0.25%)
:arrow_down:

src/os_unix.c
63.51% <0%> (-0.23%)
:arrow_down:

src/if_xcmdsrv.c
86.17% <0%> (-0.18%)
:arrow_down:

src/term.c
81.34% <0%> (-0.06%)
:arrow_down:

src/ops.c
89.65% <0%> (-0.05%)
:arrow_down:

src/window.c
88.7% <0%> (-0.04%)
:arrow_down:

src/getchar.c
84.42% <0%> (-0.03%)
:arrow_down:

src/gui.c
63.58% <0%> (ø)
:arrow_up:

... and 6 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 5381c7a...a3fc291. Read the comment docs.

This is ready to be included.
I have used this for a week and no regressions found.

Let me include this now, we can further improve on it later.


Addition : Issue template 🛠

What i am suggesting to add ?
I am suggesting to add Issue Templates to the repository, it can be as many categories as needed - primary categories of issues are mostly : bugs, docs, general feature and so on.
Why to add it ?
Adding an issue Template allows you to organize the issues made by folks in a much better way.

We can know if someone wants to add a feature.
We can know if someone wants to write a test or maybe fix something in the website
Here's a general idea :

Can i work on it ?
Yes, i will be glad to help an Opensource project grow more !

Looks Good.
@IAmTamal You can work on this

@Debajyoti14 ,PR has been made at #215 , feel free to let me know of any changes


Upgrade mypy to version 0.470 (and used package renamed from mypy-lang)

This is a reasonably advanced task requiring reasonably good experience with python and mypy.
Apparently, mypy finally got the name mypy on pypi and thus we need to replace mypy-lang==0.4.6 with mypy==0.470 in our requirements/mypy.txt.
We'll also need to fix whatever new errors show up with the updated version.
Might be a good prerequisite to #3447 in case upstream has fixed relevant bugs in the meantime.

I have started working on this.

Last summer I used to ocassionally run mypy's master branch on zulip to see if there are changes to be introduced which could make mypy fail on zulip. Such changes were either bugs or wrong annotations in Zulip or they were bugs in mypy or typeshed. In either case, knowing them was helpful for zulip because we could gradually prepare for upgrading our version of mypy.
So maybe we can automate this process. We could check if bleeding-edge mypy's output is changing on Zulip and get notified every time this happens.

It's not a bad idea; I've usually just been upgrading every month or two when we upgrade the rest of our Python dependencies; often it's pretty fast.

I just tested with 0.471 (released a few hours ago), and looks like there's still a few issues:
Warning: --silent-imports has been replaced by --ignore-missing-imports --follow-imports=skip
tools/show-profile-results.py:22: error: Missing named argument "stream" for function "Stats"
tools/show-profile-results.py:23: error: overloaded function has no attribute "print_stats"
tools/show-profile-results.py:24: error: overloaded function has no attribute "print_stats"
zproject/jinja2/__init__.py:32: error: "Environment" has no attribute "install_gettext_translations"
zerver/decorator.py:643: error: Need type annotation for variable
analytics/views.py:1024: error: Argument 2 to "sorted" has incompatible type Callable[[Dict[str, Sequence[str]]], str]; expected Callable[[Dict[str, Sequence[Sized]]], Any]

The Stats stuff at least looks like https://github.com/python/typeshed/pull/905; I wonder why that didn't fix it?


Tool running Wordpress Modules on a domain which does not support Wordpress

If a website is made over WordPress then only it should check for wp-user-enum, wp-plugins, wp-themes.
Websites that are not built on WordPress, the tool should skip those checks.
It will affect and time will be wasted if we use -s for scanning all subdomains because in that case WordPress modules will be checked on all subdomains and not every subdomain will be made on WordPress in normal cases.
The tool should first check the cms it is using by calling wappalyzer or whatcms modules and then detect if any WordPress cms is present or not. If it has cms then it should go for WordPress modules, else should skip that subdomain for WordPress checks.

OS: Linux
OS Version: Ubuntu 18.04
Python Version: 3.6.9

@aman566 - this is a good feature request suggestion, however with less priority. I would expect that the users will run CMS-specific scans (WordPress/Joomla/Drupal) only after confirming the actual CMS using cms_detection_scan module, wappalyser or whatcms (for Internet-facing sites) for now. We can (and we definitely should) implement this, but a bit later

Yes sure.


Support mounting yaml and json files as is?

My use case: I'd like to pass a sops encrypted .yaml configuration file to my app after decryption as is, instead of picking a single key from the file. So, sops-nix is a little too clever for my use case and it should focus only on decrypting the file, not parsing it.
In other words, I use this file ./secrets/my-config.yaml in my flake:
my-secret1: ENC[AES256_GCM,data:tkyQPQODC3g=,iv:yHliT2FJ74EtnLIeeQtGbOoqVZnF0q5HiXYMJxYx6HE=,tag:EW5LV4kG4lcENaN2HIFiow==,type:str]
my-secret2: ENC[AES256_GCM,data:tkyQPQODC3g=,iv:yHliT2FJ74EtnLIeeQtGbOoqVZnF0q5HiXYMJxYx6HE=,tag:EW5LV4kG4lcENaN2HIFiow==,type:str]
sops:
    kms: []
    gcp_kms: []
    azure_kv: []
    hc_vault: []
...

It should be mounted as /run/secrets/my-config.yaml:
my-secret1: hello
my-secret2: hello

Then I could pass the /run/secrets/my-config.yaml to my app to read any secrets from.
I could do this using a format=binary file, but then I have to remember to use format=binary everywhere, including when invoking sops. Alternatively, I could use a different file extension, but then I'd lose syntax highlight. Yet another alternative would be to use a literal multi line yaml string, but then again I'd lose highlight.
I wonder if this is already supported, because the README.md mentions home-assistant-secrets.yaml, which is similar to what I'm looking for? But I think that was defined as something like this:
home-assistant-secrets.yaml: |
    my-secret1: hello
    my-secret2: hello

(Notice how the renderer fails to highlight the yaml keys my-secret1 and my-secret2).
Does this make any sense to you?

Currently I don't see any other way but using either the multi-line variant or encrypting the yaml file as a binary, in which case you would still not be able to use sops as an editor of the file.
Personally I use multline yaml values to encode yaml secrets.

Notice that this a limitation in sops itself, where sops-nix has little control over.

Does it? To me it seems it's sops-nix doing the unmarshalling.
A hack would be to write decryptSecret to something like this (search for SkipUnmarshal), but I have not tested this:
func decryptSecret(s *secret, sourceFiles map[string]plainData) error {
	sourceFile := sourceFiles[s.SopsFile]
	if sourceFile.data == nil || sourceFile.binary == nil {
		plain, err := decrypt.File(s.SopsFile, string(s.Format))
		if err != nil {
			return fmt.Errorf("Failed to decrypt '%s': %w", s.SopsFile, err)
		}
		if s.Format == Binary || s.SkipUnmarshal {
			sourceFile.binary = plain
		} else {
			if s.Format == Yaml {
				if err := yaml.Unmarshal(plain, &sourceFile.data); err != nil {
					return fmt.Errorf("Cannot parse yaml of '%s': %w", s.SopsFile, err)
				}
			} else {
				if err := json.Unmarshal(plain, &sourceFile.data); err != nil {
					return fmt.Errorf("Cannot parse json of '%s': %w", s.SopsFile, err)
				}
			}
		}
	}
	if s.Format == Binary || s.SkipUnmarshal {
		s.value = sourceFile.binary
	} else {
		val, ok := sourceFile.data[s.Key]

		if !ok {
			return fmt.Errorf("The key '%s' cannot be found in '%s'", s.Key, s.SopsFile)
		}
		strVal, ok := val.(string)
		if !ok {
			return fmt.Errorf("The value of key '%s' in '%s' is not a string", s.Key, s.SopsFile)
		}
		s.value = []byte(strVal)
	}
	sourceFiles[s.SopsFile] = sourceFile
	return nil
}

Yet another way would be to reserialize nested keys back to yaml/json and set that string into s.value, but I'm not sure if I like that

I think I would prefer format=binary over adding another option which does the same thing.

Yeah, I understand and thus I'm trying to come up with a good idea. What about supporting setting s.Key to nil? In other words:
func decryptSecret(s *secret, sourceFiles map[string]plainData) error {
	sourceFile := sourceFiles[s.SopsFile]
	if sourceFile.data == nil || sourceFile.binary == nil {
		plain, err := decrypt.File(s.SopsFile, string(s.Format))
		if err != nil {
			return fmt.Errorf("Failed to decrypt '%s': %w", s.SopsFile, err)
		}
		if s.Format == Binary || s.Key == nil {
			sourceFile.binary = plain
		} else {
			if s.Format == Yaml {
				if err := yaml.Unmarshal(plain, &sourceFile.data); err != nil {
					return fmt.Errorf("Cannot parse yaml of '%s': %w", s.SopsFile, err)
				}
			} else {
				if err := json.Unmarshal(plain, &sourceFile.data); err != nil {
					return fmt.Errorf("Cannot parse json of '%s': %w", s.SopsFile, err)
				}
			}
		}
	}
	if s.Format == Binary || s.Key == nil {
		s.value = sourceFile.binary
	} else {
		val, ok := sourceFile.data[s.Key]

		if !ok {
			return fmt.Errorf("The key '%s' cannot be found in '%s'", s.Key, s.SopsFile)
		}
		strVal, ok := val.(string)
		if !ok {
			return fmt.Errorf("The value of key '%s' in '%s' is not a string", s.Key, s.SopsFile)
		}
		s.value = []byte(strVal)
	}
	sourceFiles[s.SopsFile] = sourceFile
	return nil
}

I think I would prefer format=binary over adding another option which does the same thing. At the end of the day you would need an option to treat it differently.

I ran into wanting the same request personally.  I do agree that functionally, you get some of the same things with a binary encoding.  But the biggest thing you don't get with a binary encoding is one of SOPS's selling points, which is a nice textual diffs in source control.  So I don't think it's completely fair to say it's the same thing, though it's close.
So that said, is this feature something we think we can get upstreamed?  That would inform the effort and approach I take personally in trying to get it to work.

Hi, I came across with similar use cases. Any updates on this feature request?

Would love to have this too. It's really useful for passing a complex configuration file as-is to a program, while still retaining the benefits of SOPS (diff-able plain text).


Port AMRInterpolator

Having looked into this a bit, I think we should do #3 first so that we know how to handle derived quantities.


test: add a few tests and small refactor

add a few tests for the NewMockClock function to check if it will create the Mock object as expected. I'm not completely sure if the way I did the test on TestNewMockClockWithNoArguments  is the best approach, but I didn't find a better way to do this one.
I did a small refactor in two functions to remove the else.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ustrajunior sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Sorry about the mess. I was trying to fix the message to pass the mixergithub/commit-message-check check but is not working as I expected.

So, I could fix the commit message. Sorry about all the renamings and edits.

Looks good, thanks for your contribution!


Syntax Highlighter doesn't like Optional Inputs

I just used the attached wdl file (saved as .txt so github would attach it) in IntelliJ and the highlighter seems to find an error with the first '?' in the file. The error pops up as: , WdlTokenType.IDENTIFIER or WdlTokenType.LSQUARE expected, for '?'
Any subsequent '?' don't have an error.
OptionalInput.wdl.txt

This no longer reproduces in my hand but reopen if I'm wrong


Cannot find device after reverting snapshot

2016-11-04 15:44:08.541778: iteration = 1 thread = 4 : Device ready after 2.220 seconds
2016-11-04 15:44:08.541864: iteration = 1 thread = 4 controller = 619d5a3c500dba645eb65a4c5a36a567500c881c970b23148199143837a2355a
<.... Many lines deleted ...>
2016-11-04 15:46:09.868969: iteration = 1 thread = 4 : FAIL TO WAIT FOR DEVICE READY, docker logs:
time="2016-11-04T22:44:06Z" level=info msg="Starting with replicas ["tcp://<IP_ADDRESS>:9502" "tcp://<IP_ADDRESS>:9502"]"
time="2016-11-04T22:44:06Z" level=info msg="Connecting to remote: <IP_ADDRESS>:9502"
time="2016-11-04T22:44:06Z" level=info msg="Opening: <IP_ADDRESS>:9502"
time="2016-11-04T22:44:06Z" level=info msg="Adding backend: tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:44:06Z" level=info msg="Connecting to remote: <IP_ADDRESS>:9502"
time="2016-11-04T22:44:06Z" level=info msg="Opening: <IP_ADDRESS>:9502"
time="2016-11-04T22:44:06Z" level=info msg="Adding backend: tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:44:08Z" level=info msg="SCSI device /dev/sde created"
time="2016-11-04T22:44:08Z" level=info msg="Creating device /dev/longhorn/vol4 8:64"
time="2016-11-04T22:44:08Z" level=info msg="Device /dev/longhorn/vol4 is ready"
time="2016-11-04T22:44:08Z" level=info msg="Listening on localhost:9501"
time="2016-11-04T22:44:46Z" level=info msg="Snapshot: <IP_ADDRESS>:9502 10e91896-f9a3-45f5-87c7-91a29fa82f8a"
time="2016-11-04T22:44:46Z" level=info msg="Snapshot: <IP_ADDRESS>:9502 10e91896-f9a3-45f5-87c7-91a29fa82f8a"
<IP_ADDRESS> - - [04/Nov/2016:22:44:46 +0000] "POST /v1/volumes/dm9sNA==?action=snapshot HTTP/1.1" 200 180
time="2016-11-04T22:45:48Z" level=info msg="Shutdown SCSI device for <IP_ADDRESS>:iqn.2014-09.com.rancher:vol4"
time="2016-11-04T22:45:49Z" level=info msg="Shutdown SCSI target iqn.2014-09.com.rancher:vol4"
time="2016-11-04T22:45:49Z" level=info msg="Reverting to snapshot volume-snap-10e91896-f9a3-45f5-87c7-91a29fa82f8a.img on tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:45:49Z" level=info msg="Reverting to snapshot volume-snap-10e91896-f9a3-45f5-87c7-91a29fa82f8a.img on tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:45:49Z" level=warning msg="Socket server connection closed"
time="2016-11-04T22:45:49Z" level=fatal msg="Failed to start up frontend: Failed to execute: tgtadm [--lld iscsi --op bind --mode target --tid 1 -I ALL], output tgtadm: failed to send request hdr to tgt daemon, Transport endpoint is not connected\n, error exit status 107"
Process Process-4:
Traceback (most recent call last):
File "/usr/lib/python2.7/multiprocessing/process.py", line 258, in _bootstrap
self.run()
File "/usr/lib/python2.7/multiprocessing/process.py", line 114, in run
self._target(*self._args, **self._kwargs)
File "teardown-test.py", line 162, in run_test
check_data(thread, pattern1)
File "teardown-test.py", line 76, in check_data
data = readat_direct("/dev/longhorn/vol" + str(i), page * PAGE_SIZE, PAGE_SIZE)
File "teardown-test.py", line 41, in readat_direct
f = os.open(dev, os.O_DIRECT | os.O_RDONLY)
OSError: [Errno 2] No such file or directory: '/dev/longhorn/vol4'

Another failure in the same setup, slightly different controller logs:
root@ubuntu:/home/sheng# docker logs 3b08fbc80b2e
time="2016-11-04T22:50:14Z" level=info msg="Starting with replicas ["tcp://<IP_ADDRESS>:9502" "tcp://<IP_ADDRESS>:9502"]"
time="2016-11-04T22:50:14Z" level=info msg="Connecting to remote: <IP_ADDRESS>:9502"
time="2016-11-04T22:50:14Z" level=info msg="Opening: <IP_ADDRESS>:9502"
time="2016-11-04T22:50:14Z" level=info msg="Adding backend: tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:50:14Z" level=info msg="Connecting to remote: <IP_ADDRESS>:9502"
time="2016-11-04T22:50:14Z" level=info msg="Opening: <IP_ADDRESS>:9502"
time="2016-11-04T22:50:14Z" level=info msg="Adding backend: tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:50:17Z" level=info msg="SCSI device /dev/sdh created"
time="2016-11-04T22:50:17Z" level=info msg="Creating device /dev/longhorn/vol14 8:112"
time="2016-11-04T22:50:17Z" level=info msg="Device /dev/longhorn/vol14 is ready"
time="2016-11-04T22:50:17Z" level=info msg="Listening on localhost:9501"
time="2016-11-04T22:50:44Z" level=info msg="Snapshot: <IP_ADDRESS>:9502 8f7ac338-ca1e-40d5-91a7-2cc80031c8af"
time="2016-11-04T22:50:44Z" level=info msg="Snapshot: <IP_ADDRESS>:9502 8f7ac338-ca1e-40d5-91a7-2cc80031c8af"
<IP_ADDRESS> - - [04/Nov/2016:22:50:44 +0000] "POST /v1/volumes/dm9sMTQ=?action=snapshot HTTP/1.1" 200 180
time="2016-11-04T22:51:50Z" level=info msg="Shutdown SCSI device for <IP_ADDRESS>:iqn.2014-09.com.rancher:vol14"
time="2016-11-04T22:51:51Z" level=info msg="Shutdown SCSI target iqn.2014-09.com.rancher:vol14"
time="2016-11-04T22:51:51Z" level=info msg="Reverting to snapshot volume-snap-8f7ac338-ca1e-40d5-91a7-2cc80031c8af.img on tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:51:51Z" level=info msg="Reverting to snapshot volume-snap-8f7ac338-ca1e-40d5-91a7-2cc80031c8af.img on tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:51:51Z" level=warning msg="Socket server connection closed"
panic: exit status 242
goroutine 77 [running]:
panic(0xa0bde0, 0xc4202f41e0)
/usr/local/go/src/runtime/panic.go:500 +0x1a1
github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi.startDaemon(0xc420024390, 0x0)
/go/src/github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi/target.go:193 +0xfd
created by github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi.StartDaemon
/go/src/github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi/target.go:163 +0x87

Similar failure, this time with tgtd.log:
First, test output:
2016-11-04 15:54:28.182771: iteration = 4 thread = 11 : FAIL TO WAIT FOR DEVICE READY, docker logs:
time="2016-11-04T22:52:27Z" level=info msg="Starting with replicas ["tcp://<IP_ADDRESS>:9502" "tcp://<IP_ADDRESS>:9502"]"
time="2016-11-04T22:52:27Z" level=info msg="Connecting to remote: <IP_ADDRESS>:9502"
time="2016-11-04T22:52:27Z" level=info msg="Opening: <IP_ADDRESS>:9502"
time="2016-11-04T22:52:27Z" level=info msg="Adding backend: tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:52:27Z" level=info msg="Connecting to remote: <IP_ADDRESS>:9502"
time="2016-11-04T22:52:27Z" level=info msg="Opening: <IP_ADDRESS>:9502"
time="2016-11-04T22:52:27Z" level=info msg="Adding backend: tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:52:29Z" level=info msg="SCSI device /dev/sdb created"
time="2016-11-04T22:52:29Z" level=info msg="Creating device /dev/longhorn/vol11 8:16"
time="2016-11-04T22:52:29Z" level=info msg="Device /dev/longhorn/vol11 is ready"
time="2016-11-04T22:52:29Z" level=info msg="Listening on localhost:9501"
time="2016-11-04T22:52:50Z" level=info msg="Snapshot: <IP_ADDRESS>:9502 a2ceb554-4e50-45f1-8e62-ef5fafeedffa"
time="2016-11-04T22:52:50Z" level=info msg="Snapshot: <IP_ADDRESS>:9502 a2ceb554-4e50-45f1-8e62-ef5fafeedffa"
<IP_ADDRESS> - - [04/Nov/2016:22:52:50 +0000] "POST /v1/volumes/dm9sMTE=?action=snapshot HTTP/1.1" 200 180
time="2016-11-04T22:54:07Z" level=info msg="Shutdown SCSI device for <IP_ADDRESS>:iqn.2014-09.com.rancher:vol11"
time="2016-11-04T22:54:07Z" level=info msg="Shutdown SCSI target iqn.2014-09.com.rancher:vol11"
time="2016-11-04T22:54:07Z" level=info msg="Reverting to snapshot volume-snap-a2ceb554-4e50-45f1-8e62-ef5fafeedffa.img on tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:54:07Z" level=info msg="Reverting to snapshot volume-snap-a2ceb554-4e50-45f1-8e62-ef5fafeedffa.img on tcp://<IP_ADDRESS>:9502"
panic: exit status 242
goroutine 84 [running]:
panic(0xa0bde0, 0xc420205840)
/usr/local/go/src/runtime/panic.go:500 +0x1a1
github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi.startDaemon(0xc420308028, 0x0)
/go/src/github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi/target.go:193 +0xfd
created by github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi.StartDaemon
/go/src/github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi/target.go:163 +0x87
Process Process-11:
Traceback (most recent call last):
File "/usr/lib/python2.7/multiprocessing/process.py", line 258, in _bootstrap
self.run()
File "/usr/lib/python2.7/multiprocessing/process.py", line 114, in run
self._target(*self._args, **self._kwargs)
File "teardown-test.py", line 162, in run_test
check_data(thread, pattern1)
File "teardown-test.py", line 76, in check_data
data = readat_direct("/dev/longhorn/vol" + str(i), page * PAGE_SIZE, PAGE_SIZE)
File "teardown-test.py", line 41, in readat_direct
f = os.open(dev, os.O_DIRECT | os.O_RDONLY)
OSError: [Errno 2] No such file or directory: '/dev/longhorn/vol11'
Next, docker log:
sheng@ubuntu:~$ docker logs f0
time="2016-11-04T22:52:27Z" level=info msg="Starting with replicas ["tcp://<IP_ADDRESS>:9502" "tcp://<IP_ADDRESS>:9502"]"
time="2016-11-04T22:52:27Z" level=info msg="Connecting to remote: <IP_ADDRESS>:9502"
time="2016-11-04T22:52:27Z" level=info msg="Opening: <IP_ADDRESS>:9502"
time="2016-11-04T22:52:27Z" level=info msg="Adding backend: tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:52:27Z" level=info msg="Connecting to remote: <IP_ADDRESS>:9502"
time="2016-11-04T22:52:27Z" level=info msg="Opening: <IP_ADDRESS>:9502"
time="2016-11-04T22:52:27Z" level=info msg="Adding backend: tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:52:29Z" level=info msg="SCSI device /dev/sdb created"
time="2016-11-04T22:52:29Z" level=info msg="Creating device /dev/longhorn/vol11 8:16"
time="2016-11-04T22:52:29Z" level=info msg="Device /dev/longhorn/vol11 is ready"
time="2016-11-04T22:52:29Z" level=info msg="Listening on localhost:9501"
time="2016-11-04T22:52:50Z" level=info msg="Snapshot: <IP_ADDRESS>:9502 a2ceb554-4e50-45f1-8e62-ef5fafeedffa"
time="2016-11-04T22:52:50Z" level=info msg="Snapshot: <IP_ADDRESS>:9502 a2ceb554-4e50-45f1-8e62-ef5fafeedffa"
<IP_ADDRESS> - - [04/Nov/2016:22:52:50 +0000] "POST /v1/volumes/dm9sMTE=?action=snapshot HTTP/1.1" 200 180
time="2016-11-04T22:54:07Z" level=info msg="Shutdown SCSI device for <IP_ADDRESS>:iqn.2014-09.com.rancher:vol11"
time="2016-11-04T22:54:07Z" level=info msg="Shutdown SCSI target iqn.2014-09.com.rancher:vol11"
time="2016-11-04T22:54:07Z" level=info msg="Reverting to snapshot volume-snap-a2ceb554-4e50-45f1-8e62-ef5fafeedffa.img on tcp://<IP_ADDRESS>:9502"
time="2016-11-04T22:54:07Z" level=info msg="Reverting to snapshot volume-snap-a2ceb554-4e50-45f1-8e62-ef5fafeedffa.img on tcp://<IP_ADDRESS>:9502"
panic: exit status 242
goroutine 84 [running]:
panic(0xa0bde0, 0xc420205840)
/usr/local/go/src/runtime/panic.go:500 +0x1a1
github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi.startDaemon(0xc420308028, 0x0)
/go/src/github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi/target.go:193 +0xfd
created by github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi.StartDaemon
/go/src/github.com/rancher/longhorn/vendor/github.com/yasker/go-iscsi-helper/iscsi/target.go:163 +0x87
Finally, tgtd.log:
sheng@ubuntu:~$ cat var/log/tgtd.log
tgtd: iser_ib_init(3431) Failed to initialize RDMA; load kernel modules?
tgtd: work_timer_start(146) use timer_fd based scheduler
tgtd: bs_init(387) use signalfd notification
tgtd: device_mgmt(246) sz:70 params:path=/var/run/longhorn-vol11.sock,bstype=longhorn,bsopts=size=1048576
tgtd: bs_thread_open(409) 16
tgtd: bs_longhorn_request(101) cmd->scb[0]: 35
tgtd: device_mgmt(246) sz:70 params:path=/var/run/longhorn-vol11.sock,bstype=longhorn,bsopts=size=1048576
tgtd: bs_thread_open(409) 16
connect error: No such file or directory


Pipenv to poetry

This PR migrates the dependency manager for Sir Lancebot to Poetry.
This is not intended to be merged anytime soon but is more of an experiment to see whether the tool integrates well and has good support across platforms, hence why it is currently a draft.
There are minor things that have had to be altered in order to make support work, as it's building on Python 3.9.
If we decide to go through with this then we'll probably merge this and then make alterations to new Python 3.9 syntax (i.e. typing with generics) in a later PR.

@Kronifer The default task system is meant to run python modules, such as the main bot for example, it doesn't seem able to run arbitrary commands like taskipy. Taskipy is closer in functionality to pipenv scripts, and scripts from other languages' package managers. This is needed for tasks such as linting with flake8. A possible alternative that uses the builtin system would be defining all of our tasks as python modules, but that seems a little pointless.

Tested it out. It's pretty nice and locking deps didn't take 5 years. So +1 for a basic Windows user.

For something that may not be noticed in testing: poetry does not support automatic selection of python interpreter (even though it claims it does). This is something that has bitten us in the staff pyweek project for some people, and something I've dealt with on my own projects.
Poetry claims to be able to select the correct python version from your path, like pipenv does, but that doesn't seem to work (from my testing on windows, and from reports from linux users). Since I have python 3.9 as the base interpreter on my device, I had no problem with projects such as pyweek, or forms (they both use 3.9), or this PR since it bumps python version up to 3.9. Bumping back down to 3.8 causes it to fail (I tried: 3.8.* & 3.8 & >=3.8 <=3.9, the star format being something I can confirm works for 3.9. Locking to my specific python version does not work correctly, even if fully specified 3.8.6.)
Installation with 3.8 for the python version results in:
The currently activated Python version 3.9.4 is not supported by the project (3.8.*).
Trying to find and use a compatible version.

  NoCompatiblePythonVersionFound

  Poetry was unable to find a compatible version. If you have one, you can explicitly use it via the "env use" command.

  at c:\python\python39\lib\site-packages\poetry\utils\env.py:735 in create_venv
       731│                     python_minor = ".".join(python_patch.split(".")[:2])
       732│                     break
       733│
       734│             if not executable:
    →  735│                 raise NoCompatiblePythonVersionFound(
       736│                     self._poetry.package.python_versions
       737│                 )
       738│
       739│         if root_venv:

(The first line in that error matches whatever python version I specify in the pyproject.toml.)
For more info about my setup: I have 3.8.6 from the default python.org install. I have it on my path. I can access it with py -3.8. I can not get poetry env to pick it up, unless I explicitly pass in the executable path (poetry env use C:\Python\python38\python.exe).
Ultimately, that does work, but it is nowhere near as simple as pipenv. The main selling point for poetry for me was the ludicrous speed at which it does locks and relocks, but in the grand scheme of things, we may relock once every blue moon, and a regular contributor may never relock at any point. All contributors will have to use the install command at one point or another. If pipenv does that better, and poetry's only selling point is speed, it does not make sense to switch to poetry.

d.py 1.5.1 depends on multidict, which needs gcc installed. Bumping d.py to 1.7.2 removes the need for build tools in the image.
d.py 1.7 introduced the linesep param in the paginator, so this needed to be added to our linepaginator subclass.


Dollar Protocol Add Space

Fixes # .
Changes proposed in this pull request:

https://dollarprotocol.com

Please lowercase the space key and css class

@bonustrack @e-asia, we deleted skin.scss and added space.png image.
let me know if it is good to merge.

@satoshinate Sorry but we've changed the repository for the spaces can you make a PR here instead: https://github.com/bonustrack/snapshot-spaces ? Have a look here on the updated docs: https://docs.snapshot.page/guides/create-a-space I'm closing this PR for now feel free to reach our Discord for help.


allow to remove projects with scenarios [MARXAN-215]

See https://vizzuality.atlassian.net/browse/MARXAN-215
There's a test in projects.e2e-spec.ts that covers deletion of projects that contain scenarios - basically all projects tests must pass.

I'm getting an error while running the project's tests: FAIL test/projects.e2e-spec.ts
● ProjectsModule (e2e) › Projects › A user should be able to get a list of projects and related scenarios
expect(received).toBeDefined()
Received: undefined
Is not related to the feature discussed here, (delete a project should delete it scenarios) But something to look at.

I'm getting an error while running the project's tests: FAIL test/projects.e2e-spec.ts
● ProjectsModule (e2e) › Projects › A user should be able to get a list of projects and related scenarios
expect(received).toBeDefined()
Received: undefined
Is not related to the feature discussed here, (delete a project should delete it scenarios) But something to look at.

yep, this test flipped and started failing at some point - I wanted to check that after finishing the PR for protected area selection but last time I run the test locally in the PA selection PR all was working. I'd check if it restarts acting up again.

merged via CLI. GitHub didn't think we did merge, but we actually did 🤷🏼


On static method Error TS2339: Property 'then' does not exist on type '() => Promise'

TypeScript Version:  1.8.10 and nightly (2.1.0-dev.20160804)
Code
/// <reference path="typings/globals/core-js/index.d.ts" />

export class CryptoService
{
  static getToken()
  {
    return Promise.resolve('123');
  }

  static setXsrf()
  {
    return this.getToken.then( token  => {} );
  }
}

Not expected error:

error TS2339: Property 'then' does not exist on type '() => Promise'.

You wanted to write this.getToken() (notice the ()) right?

@DickvdBrink, you say that I can not get a token?

Change:
return this.getToken.then( token  => {} );
to
return this.getToken().then( token  => {} );

Thank you guys! =)

Unless you have reason to believe there's actually a compiler bug, please use Stack Overflow for questions or log an issue with the definition file. Thanks!


pretrained weights for Faster-RCNN

Thank you for your excellent work!
It seems that the link provided in the README for the pretrained Faster-RCNN weights is not correct as it links to mask-rcnn weights. Could you kindly update the links or provide an alternative way to access the weights?

Thank you for your excellent work! It seems that the link provided in the README for the pretrained Faster-RCNN weights is not correct as it links to mask-rcnn weights. Could you kindly update the links or provide an alternative way to access the weights?

Hi, the Faster R-CNN and the Mask R-CNN has the same pretrained weights since we only pretrain the detection head. During fine-tuning stage, Mask R-CNN's mask head is initialized randomly and the detection head is pretrained


Parallel Objective C versions of iOS functionalities

Hi,
Our team needed to have an Objective C version of device calendar (mainly due to build and binary size limitations) of your great plugin. However, maintaining a fork is rarely a good idea, and we would like to get the implementation upstream. As a small added feature from our side, we have added feature for subscribing for calendar changes, so that Flutter side could be notified when events are CRUDed in one of the subscribed calendars.
Our developer @andreychernikovich created such an implementation, and found out a way on hosting two ios implementations within a project. However, some work is needed to get builds work properly, and we would not like to do that if there is no chance of getting the PR accepted.
The question is - would you accept hosting two ios implementations in this project?

Hi,
Currently, our priority is to maintain Swift code but we're happy to review if you'd like to contribute and open a PR for Objective C code.

@nickrandolph yes, today i'm will be fix it

Ping @nickrandolph would you be able to check Andrey's MR - it should be rebased now.

@nickrandolph Don't want to be pushing you, but can you give a quick heads-up on your current status - we all are busy from time to time, but would really appreciate feedback on this MR that is very much needed to take your fine library in use. We'd just like to know if the problem is that you're in hurry right now, or it is also ok to say that this MR is not acceptable and we'd need to fork the library in order to move forward.

The PR #213 seems to be removing swift and implement Obj-C.
Can we allow both to co-exist? Otherwise a fork might be necessary.


lisp-modules: Use OUTPUT-FILE arg to COMPILE-FILE instead of faslExt

Description of changes
The point of faslExt was to be able to load the pre-compiled ASDF. With :OUTPUT-FILE there's no need to know the extension because the full path can be specified.
It resolves the problem with Clozure where it uses different extensions depending on the CPU architecture, and so ccl.pkgs couldn't work on arm.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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.

Add a :+1: reaction to pull requests you find important.

@ofborg build abcl.pkgs.clop ccl.pkgs.clop clisp.pkgs.clop ecl.pkgs.clop sbcl.pkgs.clop

@ofborg build sbcl.pkgs.nyxt sbcl.pkgs.stumpwm

Now I learned that "The manner in which a source file is distinguished from a compiled file is implementation-dependent."
In particular, ECL seems to distinguish between fasl/source by pathname-type.
Getting rid of faslExt is still possible by modifying how builder.lisp is subsitituted, but using :output-file for that is not portable.


Editing template in constraints doesn't work.

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create a cluster, navigate to dashboard, enable gatekeeper.
Create a constraint under 'k8sallowedrepos'.
Edit the constraint and try to change template 'k8srequiredlabels'

Result:
Template doesn't get changed.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-2559-head- 02956b782
Installation option (single install/HA): Single

@khushboo-rancher what type of edit did you attempt? I've successfully edited a template before.

@codyrancher Changing constraint template in constraint. This is editing of constraints and changing the constraint template of it.
Steps:

Go to constraint.
Click edit in form view.
Change the template from dropdown.

Oh, I see. Thanks.

@vincent99 @tfiduccia  to support switching a constraint's constraint template I would have to delete the current constraint and create a new one. I'm a little weary of doing this as this would need to be transactional to be correct.
Would you be alright if I disabled switching the template when editing a constraint?

Yes, disable editing the template on edit

Verified on master branch of 2.4 rancher set up.
Template field is uneditable in edit constraint in form view.
Steps :

Create a constraint.
Click edit as form.
Template is not editable.


more reasonable start parameter

In make_temporal_cutoffs, this PR allows start to be a list of times, corresponding to an associated instance id. Each one specifies the first cutoff t ime to be made for the corresponding instance.

Codecov Report

Merging #149 into master will decrease coverage by 1.33%.
The diff coverage is 92.59%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage    92.7%   91.37%   -1.34%
==========================================
  Files          72       72
  Lines        7693     7743      +50
==========================================
- Hits         7132     7075      -57
- Misses        561      668     +107

Impacted Files
Coverage Δ

featuretools/utils/time_utils.py
94.44% <90%> (-5.56%)
:arrow_down:

featuretools/tests/utils_tests/test_time_utils.py
94.28% <94.11%> (-1.37%)
:arrow_down:

featuretools/primitives/aggregation_primitives.py
73.79% <0%> (-18.78%)
:arrow_down:

featuretools/utils/wrangle.py
57.24% <0%> (-15.4%)
:arrow_down:

featuretools/primitives/transform_primitive.py
86.29% <0%> (-11.49%)
:arrow_down:

...turetools/computational_backends/pandas_backend.py
91.74% <0%> (-0.34%)
:arrow_down:

...ols/tests/feature_function_tests/test_agg_feats.py
98.5% <0%> (-0.02%)
:arrow_down:

.../feature_function_tests/test_transform_features.py
98.84% <0%> (-0.01%)
:arrow_down:

featuretools/utils/api.py
100% <0%> (ø)
:arrow_up:

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 580c8e5...fc79076. Read the comment docs.


Add Mensurib's file

So far i have accomplished the following: created branch, created a file, made a commit, and opened a pull request

This pull request updates Mensur's file to the _post folder .


Seeking advice on making test more repeatable

In the zipp project (part of CPython's stdlib), I've employed big O to test the complexity of zipp.CompleteDirs._implied_dirs. The tests fail intermittently, particularly on slower platforms (macOS, Windows, PyPy) but also even on Ubuntu in CI. The tests succeed fairly reliably locally. I'm fairly confident the implementation is linear, but sometimes the test reports linearithmic or cubic.
I'm wondering if you have any advice on how I might be able to tune the test to be more reliable. I'm wondering if garbage collection is at play, or maybe the problem is simply the variability from using shared compute resources.

hi @jaraco  normally you should be able to make the tests more robust by increasing the n_measurements, n_repeats, and n_timings .
Looking at the zipp code, it seems that those arguments are left at their default values, which means 10 measurements, 1 repeat and 1 timing per repeat.
Increasing the number of measurements between n_min and n_max would improve the accuracy of the complexity estimate, but usually robustness is achieved with higher n_repeats: big_o takes the minimum of the repeats, making sure that each repetition is not influenced by temporary system load, or initial caching , or similar.
Let me know if increasing these values helps!


How to set a timeout for GoogleCredentials?

GoogleCredentials can be initialized with an optional HttpTransportFactory. But this doesn't provide any means of setting the connection and read timeout for the HTTP requests made by the credentials implementation. Is there a way to set these timeout values for GoogleCredentials?
More context: In the Google HTTP client, the recommended way to set timeouts is via an HttpRequestInitializer that gets passed into the HttpRequestFactory.
myTransport.createRequestFactory(new HttpRequestInitializer() {
      @Override
      public void initialize(HttpRequest request) throws IOException {
        request.setConnectTimeout(10000);
        request.setReadTimeout(10000);
      }
    });

But currently there appears to be no way to  specify a request initializer for GoogleCredentials. As a result, there's no simple way to configure the timeouts for the HTTP requests made by this implementation.
This is a somewhat critical issue, since GoogleCredentials get invoked before any other HTTP requests are made by the client. The default timeout settings of GoogleCredentials essentially mask and override any timeout values set in the client.
myTransport.createRequestFactory(new HttpRequestInitializer() {
      @Override
      public void initialize(HttpRequest request) throws IOException {
        credentials.initialize(request);
        request.setConnectTimeout(10000);
        request.setReadTimeout(10000);
      }
    });

In the above example you expect requests to timeout after 10 seconds. But because credentials are in the initializer chain, it won't timeout until 60s (the default).

There's also some retry logic baked into the flow that is not configurable:
https://github.com/googleapis/google-auth-library-java/blob/f4f05be44ab073ee16d92a1647e74b8bb187a442/oauth2_http/java/com/google/auth/oauth2/ServiceAccountCredentials.java#L404-L425
By default failing requests are retried up to 10 times with exponential backoff.

I agree, there does not seem to be a easy way to do this today. Is your use case you would like to fail faster in your code? Although it is not ideal, I bet you could add timeouts in by wrapping GoogleCredentials as not a lot seems to be final.

Yes, fail fast would be one use case.

Although it is not ideal, I bet you could add timeouts in by wrapping GoogleCredentials as not a lot seems to be final.

Does that imply implementing some sort of a timer around the GoogleCredentials.refreshAccessToken() API?
It appears there's no easy way to implement actual connection/socket timeouts since HTTP request logic is essentially buried in the GoogleCredentials.refreshAccessToken() method. Couple of solutions I can think of:

Make it possible to pass an HttpRequestInitializer into GoogleCredentials.
Expose an initializeTokenRequest(HttpRequest req) method from GoogleCredentials that users can override. Default implementation will do nothing.

Does that imply implementing some sort of a timer around the GoogleCredentials.refreshAccessToken() API?
That method needs to be implemented, its default impl from the base class is to throw an exception. Are you using GoogleCredentials directly or one of the other classes that implement it?

I will bring those possible solutions up with my team. Thank you for your input!

Ours is also a developer product: https://github.com/firebase/firebase-admin-java
Developers pass GoogleCredentials instances to our SDK. I believe most developers would use one of the factory methods to create credentials -- GoogleCredentials.fromStream(...). The actual instance they get is something like ServiceAccountCredentials or ApplicationDefaultCredentials.
It would be great if there was an API to wrap an existing GoogleCredentials instance with a timeout. This way our SDK can "inject" a timeout to the object passed in by the developer:
GoogleCredentials copy = credentials.withTimeout(connectTimeout, readTimeout);

// Or more generally
GoogleCredentials copy = credentials.withHttpRequestInitializer(initializer);

But we can also settle for a solution that would require developers to initialize GoogleCredentials with a timeout or a request initializer.

@hiranya911 @yoshi-automation Is there an update on this?

@hiranya911 @yoshi-automation Is there an update on this?
I am sending pushes using firebase-admin sdk.
The following domain could not be reached due to an internal network issue.
oauth2.googleapis.com
fcm.googleapis.com

When connecting to fcm.googleapis.com, the settings are as follows.
connectionTimeout = 3000
readTimeout = 30000

However, the setting value of the request to oauth2.googleapis.com in ServiceAccountCredentials.refreshToken() of google-auth-library-auth2-http-1.7.0 was the default.
connectionTimeout = 20000
readTimeout = 20000

ServiceAccountCredentials.refreshToken()
HttpRequestFactory requestFactory = transportFactory.create().createRequestFactory();
HttpRequest request = requestFactory.buildPostRequest(new GenericUrl(tokenServerUri), content);

request has ApacheHttpTransport, and the settings in it are as follows.
connectionTimeout = 3000
readTimeout = 30000
However, request does not use the transport configuration, and uses the default value of 20 seconds.

I think this should be patched.
Do you have any patch plans?

https://github.com/googleapis/google-auth-library-java/pull/1053
please check merge request


Preserve contained resource meta when encoding to dao

The contained resource meta data is not persisted when saving parent resource to database. When persisting resource to database the following setting is used on the database:
parser.setDontEncodeElements(ResourceMetaParams.EXCLUDE_ELEMENTS_IN_ENCODED);
This is okay for the parent resource beause the meta data is saved somewhere else, but for the contained resource the meta is just lost when not persisting.
Test to reproduce problem: https://groups.google.com/forum/#!topic/hapi-fhir/uxVxxldqsBI

Hi @basecade ,
Thanks for the PR! This is neat.
Can I ask you to include unit tests that demonstrate and test the functionality being added?

Hi @basecade,
did you pay respect to these constrains of the FHIR spec?

Contained resources SHALL NOT contain meta.versionId, meta.lastUpdated, or meta.security.

Hi James,
I have actually written a test but was not sure where to place it:
public void encodeResourceToString_withEXCLUDE_ELEMENTS_IN_ENCODED_metaKeptInContainedResource() {
// Arrange
Organization containedOrganization = new Organization();
containedOrganization.getMeta().addProfile("http://organization.custom");
    Patient patient = new Patient();
    patient.setId(UUID.randomUUID().toString());
    patient.getMeta().addProfile("http://hl7.org/fhir/StructureDefinition/Patient");
    patient.setGeneralPractitioner(Arrays.asList(new Reference(containedOrganization)));

    IParser parser = ResourceEncodingEnum.JSONC.newParser(new DummyFhirContext());
    parser.setDontEncodeElements(ResourceMetaParams.EXCLUDE_ELEMENTS_IN_ENCODED);

    // Act
    String encodedPatient = parser.encodeResourceToString(patient);

    // Assert
    assertThat(encodedPatient).contains("\"meta\":{\"profile\":[\"http://organization.custom\"]}");
    assertThat(StringUtils.countMatches(encodedPatient, "\"meta\"")).isEqualTo(1);
}

Did you have another test in mind?
Best Regards,
Anders Havn

Contained resources SHALL NOT contain meta.versionId, meta.lastUpdated, or meta.security

Hi Patrick,
I did not know that. Is it the Hapi framework that should enforce this or the user of the framework?
Best Regards,
Anders Havn

@basecade One of the JsonParserXXXTest or XmlParserXXXTest classes would be a good home for tests for this new functionality.
And to @patrick-werner 's point, it would be good to have tests that confirm that the FHIR rules he mentioned are still being respected.

I have now tried to update the code and added a test.

Nice work, this looks like it fits the bill. 👍


[bug] The update in the cluster directory did not trigger an action.

Is your feature request related to a problem? Please describe.

The update in the cluster directory did not trigger an action.
mysqluser-related folders also need to be able to trigger actions

Describe the solution you'd like
Describe alternatives you've considered
Additional context

https://github.com/radondb/radondb-mysql-kubernetes/pull/277


TypeError: Cannot read property 'hasOwnProperty' of undefined (verb 0.8.3, verb-cli 0.7.0)

Hi,
in recent versions (0.8.x) of verb there seems to be a problem that causes an exception for all of my repositories. I've reduced the verbfile.js to
var verb = require('verb')

verb.task('default', function () {
  verb.src(['.verb.md'])
    .pipe(verb.dest('./'))
})

and I have an empty .verb.md file and still get the following exception:
TypeError: Cannot read property 'on' of undefined
    at proxyStream (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/node_modules/event-stream/node_modules/duplexer/index.js:65:15)
    at Array.forEach (native)
    at forEach (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/node_modules/event-stream/node_modules/duplexer/index.js:11:20)
    at duplex (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/node_modules/event-stream/node_modules/duplexer/index.js:27:5)
    at Object.module.exports (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/node_modules/event-stream/node_modules/stream-combiner/index.js:8:17)
    at createStack (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/lib/stack.js:96:24)
    at Object.exports.src (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/lib/stack.js:35:10)
    at Verb.src (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/index.js:65:16)
    at Verb.<anonymous> (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/verbfile.js:22:8)
    at module.exports (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/node_modules/orchestrator/lib/runTask.js:34:7)

No problems with<EMAIL_ADDRESS>

try to update verb-cli to lastest and report back?

[21:35:38] CLI version 0.7.0
[21:35:38] Local version 0.8.3
/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/lib/transforms/init/load.js:29
  if (paths.hasOwnProperty(name)) {
           ^
TypeError: Cannot read property 'hasOwnProperty' of undefined
    at opts (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/lib/transforms/init/load.js:29:12)
    at Verb.module.exports (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/lib/transforms/init/load.js:16:25)
    at Verb.Template.transform (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/node_modules/template/index.js:272:8)
    at Verb.<anonymous> (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/lib/init.js:54:9)
    at Verb.on (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/node_modules/template/node_modules/config-cache/node_modules/component-emitter/index.js:63:8)
    at Verb.Emitter.emit (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb/node_modules/template/node_modules/config-cache/node_modules/component-emitter/index.js:131:20)
    at Liftoff.run (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb-cli/bin/verb.js:124:12)
    at Liftoff.<anonymous> (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb-cli/node_modules/liftoff/index.js:192:16)
    at module.exports (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb-cli/node_modules/liftoff/node_modules/flagged-respawn/index.js:17:3)
    at Liftoff.<anonymous> (/home/nknappmeier/privat/node-libraries/multilang-extract-comments/node_modules/verb-cli/node_modules/liftoff/index.js:185:9)

Those are the current master branches of both modules. Same exception if I use the current versions from npm.

mm yea same here, confirmed.
@jonschlinkert mark it as bug, i'll review it this night and try to help

but verb-cli is on 0.7. have you tried that version?

i'll review it this night and try to help

great thanks!

@tunnckoCore: Thanks for looking into it. I don't know why, but when I tried this first (two days ago) I really thought I had done npm -g install verb-cli before. Somehow, I was still using 0.6.2 although 0.7.0 was already published for some days.
I've updated the issue name to match the exception that comes with 0.7.0, just in case someone else is looking for it.

did you also try npm -g cache clean?

not until @tunnckoCore told me to try the latest verb-cli version. I have some questions or suggestions here, but I'll open a new issue for that. It's a bit off-topic here.

I have some questions or suggestions here,

great, would love to hear them! I'll be on the lookout for the issues

The issue seems to be resolved with verb 0.8.7. Thanks.
I think 0.8.7 is not on npm,yet. Is that possible? Could you publish it?

I think 0.8.7 is not on npm,yet. Is that possible? Could you publish it?

really? strange I have a workflow for publishing.. not sure how that happened. I'll get it pushed up
thx for letting me know

Mhm, this could be an issue with npm: http://registry.npmjs.org/verb shows a package "0.8.7": "2015-05-28T10:21:39.541Z". But http://registry.npmjs.org/verb/0.8.7 returns {"error":"version not found: 0.8.7"} while http://registry.npmjs.org/verb/0.8.6 clearly returns the registry-entry for version 0.8.6
But https://www.npmjs.com/package/verb shows
jonschlinkert jonschlinkert published 3 days ago
0.8.6 is the latest of 77 releases
github.com/assemble/verb
MIT license

And if I do npm install verb', even after a npm -g cache cleanand after wiping thenode_modules` folder, I still get version 0.8.6.
May be some inconsistencies between npm-mirrors. Lags in couchdb-replication? But I doubt that it really takes that long for the mirrors to synchronize.

I'm closing this issue, since it is resolved.  Not having the newest version on npm is a different thing that may not be an issue of verb

ah, wait - I remember.... I uninstalled that version because right after I pushed up I noticed a potential bug  and wanted to get it fixed first

If you use the latest version npm install verbose/verb --save-dev it work. But the probably the bug @jonschlinkert is talking about is not fixed ;-)

:+1:
@jonschlinkert, whenever you have time to push 0.8.7, it definitely fixes this issue occurring with 0.8.6 using with<EMAIL_ADDRESS>for me.

occurring with 0.8.6 using with<EMAIL_ADDRESS>for me.

same here.

great to know, thanks! I'll get this updated asap. fwiw I have a bunch of other changes to push up, and I refactored template to speed things up in verb... so I'll try to get all of that pushed up by this weekend

and I refactored template

good to hear, we'll stay tuned.

@jonschlinkert Just a friendly reminder that pushing a 0.8.7 version would be great to fix this.

FYI @jonschlinkert has released<EMAIL_ADDRESS>and<EMAIL_ADDRESS>These latest versions appear to be compatible with each other. I did notice an incompatibility between<EMAIL_ADDRESS>and<EMAIL_ADDRESS>so be sure to update both.
To minimize compatibility issues, I'd also recommend you install verb-cli locally to your project, rather than globally.
npm i --save-dev<EMAIL_ADDRESS><EMAIL_ADDRESS>
Then you might want add an npm script to run verb, which will use your project's version of verb-cli. In your package.json:
{
  "scripts": {
    "verb": "verb"
  }
}

Then run verb with:
npm ru verb

I did notice an incompatibility between<EMAIL_ADDRESS>and<EMAIL_ADDRESS>so be sure to update both.

hmm, you're right. perhaps we should un-release both patches and re-release as minor bumps. normally that would be odd but given that both are probably used standalone and not being consumed as a lib in another application, it will probably be ok. thoughts?

@adjohnson916 btw thank you for being on top of this!

@jonschlinkert Minor bumps to replace the patches would probably be better. This migbt happen again though. Ideally we'd have some sort of test suite that could build a compatibility matrix, and the two projects could check for this or at least refer to it on any unexpected errors, then more helpfully inform the user.

Ideally we'd have some sort of test suite that could build a compatibility matrix

agreed, tests are a big part of the next release.

pull more logic out of the CLI into the core, or perhaps formalize an more extensible interface between them.

I was just going to respond something to this effect lol. I'll try to push up a baseline for the next release so we can discuss possibilities for this.


G post001

Completed basic add and delete post.
Need to connected post with a user.

Code runs as expected. Merging shows up as blocked for me, presumably until David's requested changes are addressed.

@ookmm I'd submit a new commit or amendment to your existing commit to address these changes. The markers will be checking out and grading the source code. It will reflect a lot better if we don't include dead code. Thanks!


[issue bug report] -  we could change the order of items, in the way that the context comes before the configuration items.

Summary
my suggestion is those 3 items should be higher up on the form, I think it makes viewing easier.

Steps to Reproduce
Expected Results
Actual Results

who are going to fix it, it's better to have the context before the configuration, at least that's how I imagine it.
Issue Type
Feature Idea
Additional Information
check this issue, https://github.com/ansible/ansible-navigator/issues/450, the important informations about the scenario are at the end.
Code of Conduct

[X] I agree to follow the Ansible Code of Conduct

+1