Marker rotation

Hi there, are there any ways for custom markers to follow the rotation of the map?
I have tried implementing it with an Animated custom marker view and I can get it to rotate to any desired angle, but it's useless as I cannot figure out the heading of the map. The region spit out by on region change doesn't contain the map rotation details and maybe I'm being stupid, but can't think of any way to derive it.
Even bearing angle doesn't make sense because it doesn't account for the rotation of the map, just the angle between two points.
Thanks in advance for any help

I think this would be doable with some transform code if we could just read the Map rotation, but unfortunately: https://github.com/airbnb/react-native-maps/issues/487 I think that's unsupported.

#1544 makes this doable, right?

@aaronbuchanan I don't know, there's a possibility this feature might finally be implemented. I opened a thread about this directly a little over a year ago (referenced above) in case anyone found a way to make it work

#1544 should close that

You can do this to have a Marker who is following the map rotation (for example direction in a navigation):
const MyMap = ({}) => {

  const mapView = useRef<MapView>();
  const [myLocation, setMyLocation] = useState();

  const setMyPositionRotation = async () => {
    // Here get your Location {latitude, longitude}, call it newLocation
    let camera = await mapView.current?.getCamera();
    let realHeading = newLocation.heading - camera?.heading;
    newLocation.heading = realHeading;
    setMyLocation(newLocation);
  }

  return (
    <MapView ref={mapView}>
      <Marker
          icon={require("myMarker.png")}
          coordinate={myLocation}
          rotation={myLocation.heading}
      />
    </MapView>
  )
}


Added ban method to guild member object

Summary
I have added a ban method to the guild member object that allows you to ban a member without using the bot's REST client
and by simply using await member.ban(). It takes the same optional parameters as hikari.api.rest.RESTClient.ban_user.
Checklist

[x] I have run nox and all the pipelines have passed.
[x] I have made unittests according to the code I have added/modified/deleted.
[x] I used a test bot and the bot could ban a member using the code I have added.

Related issues

#37

Force merging due to codeclimate not working


Highlight using Mac Ports instead of Homebrew for RVM

https://www.macports.org/
https://rvm.io/integration/macports
rvm autolibs macports

Closed by 58ea5d1ba89982d6d8900d576cc261daeccc2f0f and 4250ef993d0f4505cbe07b3f6460dfa20d53e9bb


Added pausing button combo

Adapted from my original code here
https://github.com/gnargle/arduboy-pause-check
I felt that the arduboy needed a specific button combo that could be used as standard across all games, so I've created a function, checkPause that checks all 4 dpad directions and changes a bool if they're all pressed at the same time.
True = paused, false = not paused.
There's also a second function, setupPause, that relates to how often the bool can be changed.

CONTRIBUTORS.md, keywords.txt and Pausing.ino have been saved with CR/LF line terminators, not LF terminators.

Refactored checkPause():
bool Arduboy::checkPause(bool isPaused)
{
  if (pressed(RIGHT_BUTTON + LEFT_BUTTON + UP_BUTTON + DOWN_BUTTON)
       && (pause_countdown = 0)) {
      isPaused = !isPaused;
      pause_countdown = pause_countdown_set_val;
  }
  if (pause_countdown > 0) {
    pause_countdown--;
  }
  return isPaused;
}

Note that pause_countdown is defined as byte, which is unsigned. It can never go below 0 so testing for <= 0 is unnecessary.

The Arduino environment prefers cammelCase. pause_countdown should be pauseCountdown and pause_countdown_set_val should be pauseCountdownSetVal

Thanks for the input. If I want to make these changes, do I need to make them in my own repo first then pull request again?
(This is my first ever pull request so I'm not sure on what to do!)

This is functionality for individual games to define, this isn't something for the core library.  most games already use a single button for pause/unpause.  This button combo is very strange and hard to press (I know because I used it to trigger soft reboots in by button library).

If you change the same branch of your repo, the changes will automatically be added to the pull request.

This is functionality for individual games to define

Although I've been helping out with this, I tend to agree that it should be up to each individual sketch.
We could maybe turn this into an example, though. It demonstrates one method of debouncing buttons that perform toggle actions.

It demonstrates one method of debouncing buttons that perform toggle actions.

However, "wait for release" might be a better suited technique for this, in most circumstances.

I appreciate that it's perhaps not the usual thing for the library, but I only really went to the trouble because Kevin suggested I put in a pull request on twitter.
All the same, I appreciate the help on my code.

@gnargle

We could maybe turn this into an example, though. It demonstrates one method of debouncing buttons that perform toggle actions.

This is a good suggestion and one I like for this feature request.
A pull request to a public repository usually initiates a discussion about the addition and then a process that involves some editing or refactoring that results in the feature being flatly rejected, accepted, or worked into the library as best our process allows.
I would enjoy your PR as an example personally. That way it is always available to users of the library, but does not dictate the default action of a complex combo of keystrokes.
It's a bummer there is no start button, and it can be added through software if a designer wishes, but the hardware simply doesn't have it, so I think this is why it's outside the core lib's scope.


Add big-bang-umbrella folder back to Zarf examples...

...with a README that redirects the reader to the new repo that replaced it
...also fix broken E2E test

/test all

/test all

/test all

/test all

/test all

/test all

Ready for review & merge

@RothAndrew I'm confused on this PR a little bit, I see the readme updates and makes total sense, but "...also fix broken E2E test" seems completely unrelated to the topic PR.  Now of course I don't mind combining these as I do it all the time, but think the PR topic needs to be updated because a README change and E2E tests are very different, very unrelated things.  Thoughts?

I can pull the test change out into its own PR. I definitely want to practice what I preach 😄

/test all

@jeff-mccoy ready for another review


Ref. by empty

@MinMah23 In section 4.2.6 Ref. by fields in IM are empty.

Thank you for your comment, I think they're right in this way.


Update deps and node to 15

Some libp2p deps have increased min. node support to 15.

Blocked by https://github.com/libp2p/js-libp2p-crypto/issues/221
let's try node 16 instead.

Stopping this effort for now. Will get dependabot to do individual package updates with #347


Automerge-Repo in a SharedWorker under Vite loses "connect".

See also: https://github.com/Menci/vite-plugin-wasm/issues/37
In the meantime, Orion has a workaround which just recreates the worker after a couple hundred MS but... it's ugly, and it makes using a SharedWorker very inconvenient. There are a few ways we could reduce the grossness of this without waiting for an upstream fix in Chrome but I think it might boil down to including an async import.
We don't 100% have to fix this before 1.0 but it's just so awful I'd like to fit it in if we have time.

I've fixed this with dynamic imports as a workaround; it's inelegant and vaguely embarrassing to include in example code but it's less bad than the old "just keep remaking the worker" approach.


Refactor

Codecov Report

Merging #656 (cce0750) into main (22db314) will decrease coverage by 0.00%.
The diff coverage is 98.34%.

@@            Coverage Diff             @@
##             main     #656      +/-   ##
==========================================
- Coverage   85.72%   85.71%   -0.01%
==========================================
  Files         312      312
  Lines       29055    29046       -9
==========================================
- Hits        24907    24898       -9
  Misses       4148     4148

Impacted Files
Coverage Δ

lib/hir-mir/src/main_function_compiler.rs
0.00% <0.00%> (ø)

lib/mir-fmm/src/expressions.rs
86.73% <80.00%> (ø)

lib/ast-hir/src/import_compiler.rs
96.94% <100.00%> (ø)

lib/hir-mir/src/environment_creator.rs
100.00% <100.00%> (ø)

lib/hir-mir/src/expression_compiler.rs
84.86% <100.00%> (ø)

lib/hir-mir/src/type_checker.rs
98.98% <100.00%> (ø)

lib/hir-mir/src/type_coercer.rs
93.10% <100.00%> (ø)

lib/hir-mir/src/type_extractor.rs
96.24% <100.00%> (ø)

lib/hir-mir/src/type_inferrer.rs
94.64% <100.00%> (ø)

lib/hir/src/analysis/ir/variable_renamer.rs
99.18% <100.00%> (ø)

... and 9 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 22db314...cce0750. Read the comment docs.


cosine: sometimes distances are all 1

In some cases all k (cosine) distances are one (1.), which is not really plausible. Is there a hyper parameter to tune or could it be related to a bug? At the moment I do not really know how to debug it properly.
Update:I already checked the data set and there are indeed some "similar" vectors, so it seems like a bug or a poor hyper parameter selection.

Hi @inkrement!
Yes, that is not normal.
Can you calculate the distance to the returned vectors (by using ids) and compare it to the distances outputted by the library?

Sorry for the late reply. Due to upcoming deadlines I temporarily switched to another library. Nevertheless, I just checked it and seems like the cosine similarities are calculated wrongly. Could it be the case, that the library is just able to find the best "in-sample" matches? I want to find nearest neighbours for new (previously) unseen vectors. Or does it assume unit-vectors as input?

@inkrement thanks for the reply!
It does not assume unit-vector input and it should works for new (previously) unseen vectors, so not sure what is the problem. Can you share the data?

@yurymalkov The data cannot be made available to the public. However, I could share it with you via mail.

Ok! Please sent me at my github login at mail dot ru.
I'll look at it during the weekend. Thanks!

Hi, I just wanted to reproduce it, but now it seems to work. Really strange.. However, thanks for the great support! I'll close the issue for now.


Next-routes + Express / CreateServer not working on local

Hi everyone,
I'm trying to run next + next routes on a local development.
So everything is working pretty well but when I try to make it work with Express or even not express it can't find the pages.

I made a repo with a specific branch : https://github.com/JulienHe/firebase-next-routes-example/tree/localNext
Working with Next.js v5
Hope someone can tell me where I screw up :+1:
Thanks!

Hey, it looks like the dev script is running the next command but it should instead run the custom server file like so: node src/server.js


Be able to shut up activity stream

This approach is ~loosely~ inspired by what AWX does.

well...

Good suggestion, I'll add that back in.


Update apply-fine-grained-permissions-with-attribute-based-access-con…

Fixed typo

Hey @postmeback , thanks for the contribution!
To merge it, please accept the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository. The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a PR, this will mark the commit as verified.

Done as per instruction.

Thanks @postmeback !


Scope SHU token bridge and multisig on Gnosis

Defining a clear scope, requirements, process, and timeline for:

Deploying SHU on Gnosis Chain
Making SHU available in the Gnosis Bridge UI
Setting up a Safe on Gnosis Chain which is controlled by Shutter DAO 0x36 on Ethereum Mainnet (via oSnap or similar)

Resources:

Post in Shutter's forum
Example of a token on gnosis

The following are my point-of-view on how we should proceed with the scope for achieving the goal requested:
$SHU TOKEN BRIDGE
Setup
The scope for testing this token path from Ethereum to Gnosis can be understand by the following steps:

Create the repository with Blockful's hardhat boilerplate.

We will be testing superficial functions mostly, there is no need for complex tests or solidity development as the OmniBridge will provide all the features to make the bridge possible.

Create our ERC20 token resembling $SHU.

Scripts
We cannot rely on tests or forks for the communication test between bridges. We must use the scripts where each will do one step of the process: send from Ethereum to Gnosis and back from Gnosis to Ethereum.

Create deploy script for the $SHU token on Sepolia
Create script to relay tokens from source chain to destination chain - relayTokens(…)
Create script to burn synthetic tokens from the destination chain to redeem the underlying on the source chain - transferAndCall(…)

The first time the token is minted on the destination chain we cannot know in advance with small effort which contract address the $SHU token will receive on the other end
Validation

Create script to fetch the token total supply and the token amount of sender/receiver

Conclusion

Documentation and optimizations of processes.

Gnosis Multisig
Setup

First step is fund the accounts on both Sepolia and Gnosis Chain.

Since UMA's oSnap is not available on Chiado (Gnosis Testnet) we must pretend that Sepolia is Ethereum, while Gnosis will be the mainnet itself since we have no other choice but to pay for gas fees during these tests.

Create the mock Spaces on Snapshot in the Sepolia network where we have the ENS.

We must create an ENS name as a requirement for Space creation on Snapshot, even on testnet.
Deployments

Deploy Gnosis Safe on Gnosis chain.
Deploy a mock ERC20 on Gnosis chain.

We will be using the Safe to control the token assets stored in it. Executing proposals with raw bytes for functions like approve(…) and transferFrom(…).
oSnap Setup

Go to our Space's snapshot settings and add oSnap plugin. Place our treasury address and select Gnosis Chain.
To active the oSnap, we choose to be redirected to our Treasury contract, signing the module upgrade and finishing the activation after confirming the transaction.

Snapshot Transactions Tests

Now when we create proposals on Snapshot, we may opt. for the check box saying: "Yes, use oSnap for transactions”.
Last step is to configure the proposal and select a saved Safe where it will have the transaction executed.


NIFI-10352: remove the meaningless code of GenerateTableFetch.java, I…

…t is confusing for reading code.

Summary
NIFI-10352
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[ ] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[ ] JDK 8
[ ] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

+1 LGTM, verified this is a copy-paste error from a similar lambda above (where it IS necessary) and is not necessary in the specified code block. Thanks for the improvement! Merging to main


adding itchetumal

@gerardolgvr I have to close this request without merging, because you requested to add a non-existent domain to this repository: ;; connection timed out; no servers could be reached Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!

Hi @philipto , could you elaborate how you are getting that message?
The domain exists http://itchetumal.edu.mx/
`nslookup itchetumal.edu.mx
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53
Non-authoritative answer:
Name:	itchetumal.edu.mx
Address: <IP_ADDRESS>`


Filter by book type

Currently it is showing light novels and manga, but Aidoku doesnt support light novels. Is there a way to filter to only include manga?
A workaround is to create an Aidoku user and only give it access to the manga libraries, and use the API key from that user in Aidoku

yeah I can test it for you, but this isnt a major issue since the workaround works pretty well


Proper Way to get value of change event for a HTMLElement

VSCode Version: 0.10.11
OS Version: OSX

I am in the process of writing a html select ui element (I will be opening a PR for this eventually), however I am having an issue getting the value of the dropdown on a change event.
The onchange function is below.  This issue is e.target.value is null because it is not defined on EventTarget.  I have tried fetching the changed value via this.$select.attr('value') but that returns null as well.  Is there a better way to structure this or what am I doing wrong here or can we update the core EventType to have the property value on it?
 this.$select.on(['change'], (e: Event, b: Builder) => {
    console.log(e.target.attr('value'));
});

Changing it to the following works and the value of the select is returned but the linter throws errors on it because value is not defined on EventTarget.  I guess a dirty hack here would excluding it from the linter, what would be the correct syntax for this?
 this.$select.on(['change'], (e: Event, b: Builder) => {
    console.log(e.target.value);
});

I also tried the following but the property value is not defined on HTMLElement.
this.$select.on(['change'], (e: Event, b: Builder) => {
    this._opts.onChange(b.currentElement.value);
});

The code I have for the dropdown is below.
/*---------------------------------------------------------------------------------------------
 *  Copyright (c) Microsoft Corporation. All rights reserved.
 *  Licensed under the MIT License. See License.txt in the project root for license information.
 *--------------------------------------------------------------------------------------------*/

'use strict';

import 'vs/css!./select';

import {Builder, $} from 'vs/base/browser/builder';
import {Widget} from 'vs/base/browser/ui/widget';

export interface SelectOption {
	value: string;
	display: string;
}

export interface ISelectOpts {
	className: string;
	options?: SelectOption[];
	defaultOption: SelectOption;
}

export class Select extends Widget {

	/*protected*/ $el: Builder;
	private _opts: ICheckboxOpts;
	public dropdown: HTMLElement;
	private $select: Builder;

	private _checked: boolean;

	constructor(container: HTMLElement, opts: ISelectOpts) {
		super();
		this._opts = opts;

		this.$el = $('.select').appendTo(container);

		this.$select = $('select');
		this.$select.addClass(this._opts.className);

		let option1 = $('option').appendTo(this.$select);
		option1.value(opts.defaultOption.value);
		option1.text(opts.defaultOption.display);

		if (opts.options !== undefined && opts.options.length !== 0) {
			for (var i = 0; i < opts.options.length; i++) {
				var option = opts.options[i];
				let optionElement = $('option').appendTo(this.$select);
				optionElement.value(option.value);
				optionElement.text(option.display);
			}
		}

		this.$select.on(['change'], (e: Event, b: Builder) => {
			console.log(e.target.attr('value'));
		});

		this.$el.append(this.$select);
	}
}

This is resolved, I ended up using the HTMLSelectElement element instead of the above referenced HTMLSelectElement element.  The value property exists on the HTMLSelectElement element.


⚠ SceneTime has degraded performance

In 6df1232, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1294 ms

Resolved: SceneTime performance has improved in 3fbdb2c after 8 minutes.


Validation misbehaves for array of objects and minItems + uniqueItems rules

Prerequisites

[x] I have read the documentation;
[x] In the case of a bug report, I understand that providing a SSCCE example is tremendously useful to the maintainers.

Description
Combination of

Array of objects
minItems > 1
uniqueItems: true

Produces incorrect(or at least unexpected) validity state at the very first render. As far as I could investigate, minItems: 2 makes the form compute a list of 2 default items, and since default === default it breaks on uniqueItems validation. The only reliable workaround I could think of is providing an empty array in formData.
Steps to Reproduce
Example schema and formData on Playground.
Expected behavior
The field's validation returns should NOT have less than 2 items error.
Actual behavior
The field's validation returns should NOT have duplicate items (items ## 0 and 1 are identical) error.
Version
1.0.0

Workaround: default: [],

I have made a new playground to test default: []: here
Observations:

When you define your array directly in a field, default: [] will not produce expected value(it seems it makes array of [undefined, undefined] which get invalidated on uniqueItems check - to workaround it you must provide formData: { field: [] }
When you define your array in definitions and reference it in you field, default: [] seem to work as expected

Even i'm facing similar issue with minItems
If I'm passing "minItems" more 1. Its start showing me " Timeout - Async callback was not invoked within the 5000ms timeout specified by jest.setTimeout."
Even in test JSON structure i'm sharing correct array of object


Add ability to mark project as "skip this month"

"cycle" in project and/or subproject field -- for now, has to be one or the other, can't be both (not immediately clear how it should be handled if both are present)
0: every month
1: 01, 04, 07, 10
2: 02, 05, 08, 11
3: 03, 06, 09, 12

"cycle" in project and/or subproject field -- for now, has to be one or the other, can't be both (not immediately clear how it should be handled if both are present)
0: every month
1: 01, 04, 07, 10
2: 02, 05, 08, 11
3: 03, 06, 09, 12


Clean up garbage characters in MS MARCO dataset

I've noticed that MS MARCO data has garbage characters. A bit of probing, appears to be MS-related encoding issues: https://stackoverflow.com/questions/2477452/â€-showing-on-page-instead-of
Let's try some cleanup and see if it improves effectiveness?

Does this apply to the MS MARCO Document Ranking dataset as well? I looked at some of the indexed documents and the characters seem to be encoded correctly

Okay, here's a concrete example:
{"id": "7287406", "contents": "Why do Bears hibernate? March 31, 2010, Joan, Leave a comment. Why do bears hibernate? When we hear the word \u00e2\u0080\u0098hibernate\u00e2\u0080\u0099 we always associate it with bears. That is because, while a lot of animals go through hibernation during the winter season such as squirrels, rodents and bats, the bear is the most famous when it comes to hibernating. What comes first to our mind is why do bears hibernate? First of all, let\u00e2\u0080\u0099s get to know the meaning of hibernation."}

In browser, it gets rendered as:

Note the â garbage.

Sorry for all the back and forth here, will try to summarize what I found while digging around (let's work with the single example above but it should extend to others as well):
TLDR
Using Latin-1 should work since the underlying byte values are the same as Windows-1252. Also Python is unable to encode characters between \u0080 and \u009f using Windows-1252 since they are undefined - therefore we need to use Latin-1.
Latin-1 vs Windows-1252
I found a Stack Overflow answer indicating that \u00e2\u0080\u0099 is from ISO-8859-1 which is the same as Latin-1
This is confusing because it does not match the original Stack Overflow you sent in the issue description that mentions Windows-1252. However, apparently the Windows-1252 encoding is the exact same as Latin-1 encoding except for the fact that the 0x80-0x9F range maps to different characters.
Okay, now let's look at the raw bytes for the text:
b'7287406\tWhy do Bears hibernate? March 31, 2010, Joan, Leave a comment. Why do bears hibernate? When we hear the word \xc3\xa2\xc2\x80\xc2\x98hibernate\xc3\xa2\xc2\x80\xc2\x99 we always associate it with bears. That is because, while a lot of animals go through hibernation during the winter season such as squirrels, rodents and bats, the bear is the most famous when it comes to hibernating. What comes first to our mind is why do bears hibernate? First of all, let\xc3\xa2\xc2\x80\xc2\x99s get to know the meaning of hibernation.\n'

Notice that \xc3\xa2\xc2\x80\xc2\x99  converted to UTF-8 is  \u00e2\u0080\u0099. What likely happened is the original data was encoded properly as UTF-8, mistakenly decoded as Latin-1 or Windows-1252 when reading the text (creating garbage), and then encoded back as UTF-8 when saving (preserving the same garbage).
Reverting this is just the same two steps but reversed:

Decode the UTF-8, which should give Windows-1252 or Latin-1 encoded bytes
Encode the Windows-1252 or Latin-1 bytes back into UTF-8

Theoretically, since Windows-1252 and Latin-1 produce the exact same byte values from 0-255, both character sets should also decode the UTF-8 to the same values. This explains why latin-1 decoded things properly in my PR. It's also is where the difference in the 0x80-0x9F range comes into play. Apparently in Python, this range for cp1252 is undefined and throws the error shown below, so it cannot encode the original bytes, whereas latin-1 has the values defined properly - hence we need to use latin-1.
>>> '\u0080'.encode('cp1252')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Python37\lib\encodings\cp1252.py", line 12, in encode
    return codecs.charmap_encode(input,errors,encoding_table)

vs
>>> '\u0080'.encode('latin1')
b'\x80'

Why Decoding Sometimes Fails
To explain why some decodings still fail even after the proper encoding/decoding (~46 lines in the passage dataset), we can look at this page (https://www.i18nqa.com/debug/bug-double-conversion.html) concerning a double mis-conversion from Windows-1252 to UTF-8. This states that for 5 specific characters (Á Í Ï Ð Ý), once the text has already been incorrectly converted from UTF-8 to another format, it is impossible to convert it back to it's valid form. Instead, the UTF-8 character will always be invalid.
Take the second row as an example here:

Thank you for the pointer to the double mis-conversion from Windows-1252 to UTF-8. I have seen that before but never understood!


The values of permissions show undefined on Connection Summary dialog

Storage Explorer Version: 1.12.0
Build: 20200218.2
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu 18.04
Architecture: ia32/x64
Regression From: Previous release(1.12.0)
Steps to reproduce:

Select one Non-ADLS Gen2 storage account -> Right click it.
Click 'Get Shared Access Signature...' -> Copy the connection string.
Open Connect dialog -> Select use a connection string -> Paste the copied connection string -> Click 'Next'.
Check the values of permissions on the Connection Summary dialog.

Expect Experience:
Show the exact permissions value: Like Read, List.

Actual Experience:
The values of permissions show undefined.

More Info:
This issue also reproduces for services (blob container/ file share/ queue/ table).

This issue has been fixed on build 20200226.1. So we close it.


PUPPETSERVER_HOSTNAME is not unset

According to the documentation, PUPPETSERVER_HOSTNAME is supposed to be unset, but in fact the container explicitly sets it to puppet. This appears to be due to https://github.com/voxpupuli/container-puppetserver/blob/main/puppetserver/Dockerfile#L26. This also breaks compatibility with the old Puppet Labs-maintained images. I had to explicitly unset it myself in my docker compose file.

thx for the hint. which puppetlabs maintained containers do you mean? as far as i know we have this containers here, because they dont't maintain theirs anymore. (to be more precise they handed this repo over to us, so we could care about)

I'm referring to the original images, as in https://hub.docker.com/r/puppet/puppetserver. My point is that if you're migrating from the old images to these ones, it doesn't work properly, because of the aforementioned issue with PUPPETSERVER_HOSTNAME.

oh, okay, now i see. will check the code for it and make a fix

Thanks for the quick fix!


instruction for dreambooth is missing

Dear auhtors,
I am trying to run image-sculpting on my own images. It seems we need LoRA finetuned models based on the editing script, however in readme, there is no instruction on how to proceed such step. Could you kindly update this part on which code base, and which hyperparameter to set for LoRA?
Thank you!

Hi! Yes, we need dreambooth to utilize the input image. We use the implementation from diffusers which can be found here.


[Feature Request] Import and export settings.

I just left a similar feature request on the Play Store and just realised you're on Github :)
I use a couple of machines and Wavebox on both, as well as Chrome. It would be great to export settings (especially the card colours) and import them to the others.
Not sure if this is possible, but it would be a great addition.

Hi! You can export settings from any board (and import them into another). Check your board settings and you will see it at the top.

Ha! How did I miss that!? Many thanks :)

Just an update...
If I copy the settings to the clipboard from Chrome and then immediately attempt to paste it back in, it gives me the error - "Oops... Bad JSON in Clipboard...'.
I'm wondering if it's anything to do with the "undefined" setting in the file (see above).
Hope all this helps!

Great update on 3.3.
I just tried the export and import clipboard and this time it's accepting the JSON file.
However, although the board settings change, the colours for the board columns are not being applied still.
Exported JSON:
{"beautify-markdown":false,"compact-cards":false,"custom-background":false,"custom-background-input":"","custom-css":false,"custom-css-input":"","filters":{},"full-screen-cards":false,"hide-activity-entries":false,"hide-add-list":false,"labels-size":"medium","listEnchancements":{"backlog":{"background":"#ffffff50"},"current-sprint":{"background":"#00000020"},"done":{"background":"#00ff0050"},"done0706-2020":{"background":"#00ff0050"},"done1406-2020":{"background":"#00ff0050"},"fixed":{"background":"#00ff0050"},"in-progress":{"background":"#00000030"},"internalissues-tweaks":{"background":"#ff000080"},"roadmap":{"background":"#ffffff50"}},"parse-hashtags":false,"parse-labels":false,"parse-markup":true,"parse-points":false,"parse-price-entries":false,"parse-priority-marks":false,"parse-projects":false,"parse-time-entries":false,"show-list-stats":false,"show-list-stats-progressbar":false,"visible-card-numbers":false,"visible-labels":false}

parse-markup gets applied, but none of the colours.

Yeah, when it comes to colors and layout in general, it's kind of a personal experience. While sharing settings in general makes sense for teams, these significant visual changes are left for personalisation - to individuals. I think this was my way of thinking when I did not include them while processing imported settings. However, I will look into the code and figure out if it's ez and maybe just import all settings.

Cheers chap. Great work though. I've just donated to the cause :)


Update bluebird_v3.x.x Bluebird$BluebirdConfig

According to http://bluebirdjs.com/docs/api/promise.config.html, Promise.config() doesn't just take a boolean warning key but instead can take an object with a wForgottenReturn key. This should fix that.

@EvHaus Since this change is correct (and still relevant!), if you're keen to rebase it and fix any broken specs, we can probably get it merge in.

@GAntoine I've rebased and all looks good. Tests are failing but it's not due to this change. The tests are failing due to this bug which is still open: https://github.com/flowtype/flow-typed/issues/1272

Closing in favour of #764

In favor of https://github.com/flowtype/flow-typed/pull/1457 actually. :)


DataFusion weekly project plan (Andrew Lamb) - Nov 6, 2023

The idea of this ticket is to log plans I have each week for DataFusion related items.
This is largely for my own personal organizational needs and I think it will help me remain better focused, but I also figured it might serve as an interesting summary of what is happening in DataFusion this week
Major items I know for this week:

[ ] Several more editing passes for the SIGMOD paper: https://github.com/apache/arrow-datafusion/issues/6782 with @ozankabak @Dandandan @yjshen and @JayjeetAtGithub
[ ] Write up some idea of "growing and sustaining the DataFusion community" that I have discussed with @andygrove @thinkharderdev @sunchao @viirya  and others
[ ] Assist with cleaning up array functions, and getting good patterns going: https://github.com/apache/arrow-datafusion/pull/8054 with @jayzhan211  (e.g. https://github.com/apache/arrow-datafusion/pull/8054)
[ ] Unify BuiltInScalarFunctions with ScalarUDF: https://github.com/apache/arrow-datafusion/issues/8045, get agreement and get first few PRs ready with @2010YOUY01
[ ] Assist with sort based optimization work started in https://github.com/apache/arrow-datafusion/pull/8006 (encapsulating more structs, e.g https://github.com/apache/arrow-datafusion/pull/8034 and reviews) with @ozankabak  and @mustafasrepo
[ ] Review any new ticket / bug reports

This is a good idea, maybe we can also list some tasks that have been identified as valuable but do not yet have someone working on them.

I updated the "unify function implementation plan" with @2010YOUY01  -- https://github.com/apache/arrow-datafusion/issues/8045 and made an initial PR to make ScalarUDF fields non pub: https://github.com/apache/arrow-datafusion/pull/8079

It turns out that https://github.com/apache/arrow-datafusion/issues/8078 is important for an internal project in IOx, so I plan to work on that as well

Next week's plan: https://github.com/apache/arrow-datafusion/issues/8151


DSET-4050: Improve logging related to shutting down the client

🥅 Goal
Make the logging during shutdown less confusing.
🛠 Solution
During the shutdown process some of the events and buffers may not be processed yet. We have been logging messages like - "not all buffers has been processed". This was misleading since it was expected. Also log more clearly when the shutdown has started and when finished.
🏫 Testing

Codecov Report

Merging #42 (53c4061) into main (87228f9) will decrease coverage by 0.09%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
- Coverage   75.65%   75.56%   -0.09%
==========================================
  Files          11       11
  Lines        1491     1514      +23
==========================================
+ Hits         1128     1144      +16
- Misses        300      307       +7
  Partials       63       63

Impacted Files
Coverage Δ

pkg/client/add_events.go
85.63% <75.00%> (-1.22%)
:arrow_down:


[Bug]The GPU was not used for acceleration.

🐛 Bug Report

Steps to Reproduce

start the node as prover
chech the cpu/mem/gpu usage

Expected Behavior
the GPU should be used to imporve the performance.

The GPU should be used to improve the performance, but the GPU usage is 0.
cpu/mem usage

GPU usage

Your Environment

 2.2.7

 1.76.0

 ubuntu 22.04

I have the same question, did you find the answer?

Relevant Issue - https://github.com/AleoNet/snarkOS/issues/2708


tf.custom_gradient expects an additional output when declaring temp Variable()

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Fedora 29
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v2.1.0-rc2-17-ge5bf8de
Python version: Python 3.7.6
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A

Describe the current behavior
Thanks in advance for your time.
I'm attempting to implement a custom loss function. It requires storage of a temporary variable, and custom gradient. For many reasons, numerical stability and flexibility, I'd like to just implement the gradient by hand. The model here is just a foo/bar model.
It seems it’s related to this issue: https://github.com/tensorflow/tensorflow/issues/31945
I'm also happy to open a PR.
However, I haven't been able to get the patch suggested to work, because I can't build from source via some Bazel issue.
Any references are appreciated.
I've also tried to implement the loss function as a subclass of type Loss, but was unsuccessful.
Thanks for all of the hard work that goes into this project.
Describe the expected behavior
Run model via EagerExecution. When using GradientTape() to evaluate gradients, I'm getting error: ValueError: not enough values to unpack (expected 2, got 1). The loss function is only a function of one input, and thus only has one partial derivative, wrt to that input.
Standalone code to reproduce the issue
import tensorflow as tf
from tensorflow.keras.layers import Layer
import numpy as np

class Linear(Layer):
  """y = w.x + b"""

  def __init__(self, units=32):
      super(Linear, self).__init__()
      self.units = units

  def build(self, input_shape):
      self.w = self.add_weight(shape=(input_shape[-1], self.units),
                               initializer='random_normal',
                               trainable=True)
      self.b = self.add_weight(shape=(self.units,),
                               initializer='random_normal',
                               trainable=True)

  def call(self, inputs):
      return tf.matmul(inputs, self.w) + self.b

@tf.custom_gradient
def loss_fn(x):
    r = tf.Variable(tf.zeros([100]), dtype = tf.float32)
    ## create r
    def grad(df, variables = None):
        return [df * 2 * tf.reduce_sum(r)]

    return tf.pow(tf.norm(r), 2), grad

M = 100
m = np.arange(0, M)
x = [[m / M]]

linear_layer = Linear(10)

optimizer = tf.keras.optimizers.SGD(learning_rate=1e-3)

dataset = tf.data.Dataset.from_tensor_slices(x)

for step, x in enumerate(dataset):

    with tf.GradientTape() as tape:
        logits = linear_layer(x)
        loss = loss_fn(x)

    gradients = tape.gradient(loss, linear_layer.trainable_weights)
    optimizer.apply_gradients(zip(gradients, linear_layer.trainable_weights))

@drezap I tried to reproduce the issue but  i am facing different error. Please find the gist here.
In order to expedite the trouble-shooting process, could you please provide the complete code to reproduce the issue reported here. Thanks!

@saikumarchalla Whoops! Here:
Thanks for your time!
import tensorflow as tf
from tensorflow.keras.layers import Layer
import numpy as np

class Linear(Layer):
  """y = w.x + b"""

  def __init__(self, units=32):
      super(Linear, self).__init__()
      self.units = units

  def build(self, input_shape):
      self.w = self.add_weight(shape=(input_shape[-1], self.units),
                               initializer='random_normal',
                               trainable=True)
      self.b = self.add_weight(shape=(self.units,),
                               initializer='random_normal',
                               trainable=True)

  def call(self, inputs):
      return tf.matmul(inputs, self.w) + self.b

@tf.custom_gradient
def loss_fn(x):
    r = tf.Variable(tf.zeros([100]), dtype = tf.float32)
    ## create r
    def grad(df, variables = None):
        return [df * 2 * tf.reduce_sum(r)]

    return tf.pow(tf.norm(r), 2), grad

M = 100
m = np.arange(0, M)
x = [[m / M]]

linear_layer = Linear(10)

optimizer = tf.keras.optimizers.SGD(learning_rate=1e-3)

dataset = tf.data.Dataset.from_tensor_slices(x)

for step, x in enumerate(dataset):

    with tf.GradientTape() as tape:
        logits = linear_layer(x)
        loss = loss_fn(x)

    gradients = tape.gradient(loss, linear_layer.trainable_weights)
    optimizer.apply_gradients(zip(gradients, linear_layer.trainable_weights))

I could be replicate the issue on colab.Please find the gist here.Thanks!.

That’s exactly the error I’m getting.
“not enough values to unpack...”
On Friday, April 24, 2020, saikumarchalla<EMAIL_ADDRESS>wrote:

I could be replicate the issue on colab.Please find the gist here
https://colab.research.google.com/gist/saikumarchalla/207a8bd425cf07e0f746cb32f012759c/untitled26.ipynb
.Thanks!.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/38622#issuecomment-618794430,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACY543BC62OCG73OUT5VOWDROEIIBANCNFSM4MKQHG4A
.

--
Best,
Andre Zapico

@saikumarchalla
more generally, the question is "how to implement custom loss function that requires storage of a temp tf.Variable, iteration, and a custom gradient".
I could get custom loss functions that were trivial to "compile" with some models (this dummy one, and helpful friend on the TF forum's model), for example using tf.mean() and other simple functions, which i guess is using TF's autodiff.
What I'd like to do, is add some exception that "ignores" TF's autodiff for the function body, and only takes into account the custom gradient I've implemented. The body will be complex and could blow out memory, and be unstable. And is probably no good with autodiff, via intuition.
However, when I've tried to add custom gradients, won't "compile". Temp variable, won't "compile".
I haven't investigated debugging the problem.

any progress on this?

@drezap Can you please explain little more about this loss functions (loss_fn)? There is a grad function inside loss function with two arguments and the return of loss_fn has grad without any arguments. Input x was also not used at all.
@tf.custom_gradient
def loss_fn(x):
    r = tf.Variable(tf.zeros([100]), dtype = tf.float32)
    ## create r
    def grad(df, variables = None):
        return [df * 2 * tf.reduce_sum(r)]

    return tf.pow(tf.norm(r), 2), grad

@jvishnuvardhan
Yeah, happy to, thank you.
The loss function implementation has 3 caveats:

storage of a temporary variable,
requires that I have nested for loops
accumulation.

No work arounds.
I haven't been able to get a model to run, using TF's datatypes, with an implementation of this loss function. If I have a long input sequence, I'm anticipating any expression graph generated would blow out memory.
If this still isn't clear, can I please email you a python implementation of the loss function? I'd just ask that you not publicize it.
TLDR:

There is a grad function inside loss function

How do I include my own gradients for a loss function, and include this in a model?

If this isn't possible with the current modeling language, I'm willing to go as far as implementing C code within a compiled model, or something like that.

Mind if I just email you the loss function?
It should make it clear
On Wednesday, April 29, 2020, Vishnuvardhan Janapati <
<EMAIL_ADDRESS>wrote:

@drezap https://github.com/drezap Can you please explain little more
about this loss functions (loss_fn)? There is a grad function inside loss
function with two arguments and the return of loss_fn has grad without
any arguments. Input x was also not used at all.
@tf.custom_gradient
def loss_fn(x):
r = tf.Variable(tf.zeros([100]), dtype = tf.float32)
## create r
def grad(df, variables = None):
return [df * 2 * tf.reduce_sum(r)]
return tf.pow(tf.norm(r), 2), grad

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/38622#issuecomment-620990964,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACY543AIS2BRXZSJ4HPMNGLRO6Y5PANCNFSM4MKQHG4A
.

--
Best,
Andre Zapico

May be this issue is similar to this @custom_gradient issue.


Platform Engineering Maturity Model tab improvement

On the website we have a tab design.
This has been useful because we have a more web friendly view which differentiates from a single PDF.
However, it has created issues with anchor links to the tabs and within them.
Issue 1: URLs with anchors to the tabs do not actually open to the tabs
If you click on the table aspect Investment it will open the correct tab and change the URL to https://tag-app-delivery.cncf.io/whitepapers/platform-eng-maturity-model/#Interfaces, however the page does not scroll to focus on the newly opened tab and should someone go directly to that link, the page opens from the top.
Issue 2: Subheaders within the tabs do not anchor
Within the tabs there are anchor links (even have the 🔗 icon on them) for the subheaders. They are unique as they include both common info (e.g. Level 1 - Provisional) but also the unique descriptors (e.g. Level 1 - Provisional -- by request for Operations).
These also set the URL when the 🔗 is clicked on and it is copied for the user, but they remove the aspect details from the URL even if it was there. When these links are used directly, they do not open the right tab nor scroll to the right section.

This is a nice to have fix, and there is an option to just move back to a single long scroll page.

I have made a new pull request for this issue, can u assign this issue to me please,
Thanks!

Hey @Aditya-138-12 , do you have a branch I can pull down and see what you see when it comes to the diff?

Hi @abangser,
I've made the changes directly on the main branch of my forked repository. You can check out my fork and see the changes here:
Forked Repository: https://github.com/Aditya-138-12/Aditya-138-12-tag-app-delivery
Let me know if you need any further information or if there's anything else I can assist with.
Thank you!
Aditya Saroha
Changes I have Made!!

Added a node_modules folder in the websites folder
Also edited other files
Changes paths of some of the node modules
edited the code for giving id for some of the tags, etc etc

Thanks so much @Aditya-138-12, I have just cloned locally and followed the readme and saw some things working great but other odd behaviour locally.
So I am curious on a couple of things:

When you say that you are deploying to Vercel, how are you doing this? Is this through a PR to this repo that I missed?
What are the behaviours on Vercel? Hard to know if what I am documenting is the oddities you have seen.

Thanks again for working on this, it seems your changes have cracked the main issue we raised here, and I know we can iron out the other differences!
For debugging:

this is how I ran it locally
[10:15:21] [~/dev/tmp]
$ cd Aditya-138-12-tag-app-delivery
[10:15:27] [~/dev/tmp/Aditya-138-12-tag-app-delivery/website] git:(main ✔)
$ cat website/README.md
###### NOTE: TRUNCATED THIS OUTPUT TO THE BIT OF THE README I USED ######
...

## Testing locally

### Run in local OS

To run the site from your local OS:

1. Install [Hugo Extended](https://gohugo.io/installation/linux/#editions) and [npm](https://www.npmjs.com/)
4. Run `git clone git@github.com:cncf/tag-app-delivery.git && cd tag-app-delivery`
5. Initialize submodules with `git submodule update --init --recursive`
6. Change into the website directory: `cd website`
7. Install dependencies with `npm install`
8. Run the site using `npm run serve`. To have the site run locally with a functioning local search, run `npm run serve:with-pagefind`.
9. Output from the previous command includes the address to browse to preview the site, by default <http://localhost:1313/>.
...
[10:15:54] [~/dev/tmp/Aditya-138-12-tag-app-delivery] git:(main ✔)
$ git submodule update --init --recursive
Submodule 'website/themes/docsy/assets/vendor/bootstrap' (https://github.com/twbs/bootstrap.git) registered for path 'website/themes/docsy/assets/vendor/bootstrap'
Submodule 'website/themes/hugo-dynamic-tabs' (https://github.com/rvanhorn/hugo-dynamic-tabs) registered for path 'website/themes/hugo-dynamic-tabs'
Cloning into '/Users/abbybangser/dev/tmp/Aditya-138-12-tag-app-delivery/website/themes/docsy/assets/vendor/bootstrap'...
Cloning into '/Users/abbybangser/dev/tmp/Aditya-138-12-tag-app-delivery/website/themes/hugo-dynamic-tabs'...
Submodule path 'website/themes/docsy/assets/vendor/bootstrap': checked out 'a716fb03f965dc0846df479e14388b1b4b93d7ce'
Submodule path 'website/themes/hugo-dynamic-tabs': checked out '38f93d6f4e5d04b3ab605af14ea25e8be8542c30'
[10:26:12] [~/dev/tmp/Aditya-138-12-tag-app-delivery] git:(main ✔)
$ cd website
[10:26:14] [~/dev/tmp/Aditya-138-12-tag-app-delivery/website] git:(main ✔)
$ npm install

added 204 packages, and audited 205 packages in 8s

55 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities
npm notice
npm notice New minor version of npm available! 10.2.3 -> 10.8.2
npm notice Changelog: https://github.com/npm/cli/releases/tag/v10.8.2
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice
[10:26:25] [~/dev/tmp/Aditya-138-12-tag-app-delivery/website] git:(main ✔)
$ npm run serve:with-pagefind

> serve:with-pagefind
> hugo --baseURL=/ && npm_config_yes=true npx pagefind --site 'public' --output-subdir '../static/pagefind' && npm run serve

Start building sites …
hugo v0.115.4-dc9524521270f81d1c038ebbb200f0cfa3427cc5+extended darwin/arm64 BuildDate=2023-07-20T06:49:57Z VendorInfo=gohugoio

                   | EN  | ZH  | KO  | JA  | ES
-------------------+-----+-----+-----+-----+------
  Pages            |  70 |  27 |  27 |  32 |  11
  Paginator pages  |   1 |   0 |   0 |   0 |   0
  Non-page files   |  24 |  17 |  17 |  19 |   0
  Static files     | 144 | 144 | 144 | 144 | 144
  Processed images |   0 |   0 |   0 |   0 |   0
  Aliases          |   2 |   0 |   0 |   0 |   0
  Sitemaps         |   2 |   1 |   1 |   1 |   1
  Cleaned          |   0 |   0 |   0 |   0 |   0

Total in 1943 ms

Running Pagefind v1.1.0 (Extended)
Running from: "/Users/abbybangser/dev/tmp/Aditya-138-12-tag-app-delivery/website"
Source:       "public"
Output:       "public/../static/pagefind"

[Walking source directory]
Found 110 files matching **/*.{html}

[Parsing files]
Found a data-pagefind-body element on the site.
↳ Ignoring pages without this tag.

[Reading languages]
Discovered 4 languages: ja, ko, zh, en

[Building search indexes]
Total:
  Indexed 4 languages
  Indexed 42 pages
  Indexed 14894 words
  Indexed 0 filters
  Indexed 0 sorts

Finished in 1.073 seconds

> serve
> hugo serve --minify -DFE -w

Watching for changes in /Users/abbybangser/dev/tmp/Aditya-138-12-tag-app-delivery/{operator-whitepaper,platforms-maturity-model,platforms-wg,platforms-whitepaper,website}
Watching for config changes in /Users/abbybangser/dev/tmp/Aditya-138-12-tag-app-delivery/website/config.toml
Start building sites …
hugo v0.115.4-dc9524521270f81d1c038ebbb200f0cfa3427cc5+extended darwin/arm64 BuildDate=2023-07-20T06:49:57Z VendorInfo=gohugoio

                   | EN  | ZH  | KO  | JA  | ES
-------------------+-----+-----+-----+-----+------
  Pages            |  70 |  27 |  27 |  32 |  11
  Paginator pages  |   1 |   0 |   0 |   0 |   0
  Non-page files   |  24 |  17 |  17 |  19 |   0
  Static files     | 208 | 208 | 208 | 208 | 208
  Processed images |   0 |   0 |   0 |   0 |   0
  Aliases          |   2 |   0 |   0 |   0 |   0
  Sitemaps         |   2 |   1 |   1 |   1 |   1
  Cleaned          |   0 |   0 |   0 |   0 |   0

Built in 684 ms
Environment: "development"
Serving pages from memory
Running in Fast Render Mode. For full rebuilds on change: hugo server --disableFastRender
Web Server is available at http://localhost:1313/ (bind address <IP_ADDRESS>)
Press Ctrl+C to stop

this is some screenshots from local env
Note the lack of arrows:

Note the scroll works. but goes a bit far:

Note that when I click a top bar link (e.g. blog) from the MM page it does not change pages (top bar links do work from other pages):

Hi @abangser
Thanks for cloning the repo and checking things out! I'm glad to hear that some parts are working well for you.
To answer your questions:

Deployment to Vercel: When I make a pull request on the repo, the deployment to Vercel is triggered automatically. I did raise a PR, but after noticing that the page was not working as expected with the changes made, I deleted the PR.

Behaviors on Vercel: On Vercel, subheaders are not linked, they are also invisible, the content becomes messed up, This might help identify if the oddities you are documenting match what I have seen.

If there's any specific part of the deployment process or behavior you'd like me to elaborate on, please let me know. I'm happy to help iron out these differences and get everything running smoothly!
Thanks again for your efforts and feedback. It's great to see progress on the main issue, and I'm confident we can resolve the remaining quirks together.


wildcard import with specific style gets deleted

isort deleted the following line from one of my files altogether
from findorserver.coupon.models import*

.isort.cfg contents:
[settings]
line_length = 120
multi_line_output = 5
balanced_wrapping = true
include_trailing_comma = true
skip = migrations

properly formatted wildcard imports were unaffected
thanks for the great tool!

Thanks for reporting this issue! I've prepared and merged in a resolution into develop, and am therefore closing this

awesome, thanks for the great work


Anime galaxy Hub

Addon Manifest URL
https://www.animegalaxyhub.xyz/manifest.json
Addon Description
Anime Galaxy Hub - Official Anime Galaxy Hub #1 Watch High Quality Anime Online Without Ads. You can watch anime online free in HD without Ads. Best place for free find and one-click anime.
Language of Content
Multilingual (Default)
Choose Labels

[X] anime
[ ] asian drama
[ ] bollywood
[X] cartoons
[ ] debrid support
[x] http streams
[X] live tv
[ ] metadata
[ ] misc
[x] movies
[ ] music
[ ] nsfw
[ ] podcasts
[ ] radios
[ ] subtitles
[ ] torrents
[X] tv shows
[ ] video games

could you guide me how do it properly .


Setting workbench.startupEditor=readme|newUntitledFile doesn't work in Workspace Settings

Issue Type: Bug
Save an empty workspace with a README.md file.
Set "workbench.startupEditor": "readme" in the workspace.code-workspace file.
Close all tabs and restart VS Code.
VS Code should open the README.md preview on startup, but it doesn't. The workspace setting is ignored.
As a User Setting workbench.startupEditor=readme works as expected. The preview ist shown on startup.
The setting value newUntitledFile behaves the same: doesn't work as workspace settings, works as user setting.
IIRC the settings none|welcomePage|gettingStarted work correctly
VS Code version: Code 1.56.2 (054a9295330880ed74ceaedda236253b4f39a335, 2021-05-12T17:13:13.157Z)
OS version: Windows_NT x64 10.0.19042
Extensions (16)

Extension
Author (truncated)
Version

gitlens
eam
11.4.1

auto-close-tag
for
0.5.10

path-autocomplete
ion
1.17.1

svn-scm
joh
2.13.5

vscode-cfml
Kam
0.5.3

VS-code-vagrantfile
mar
0.0.7

vscode-apache
mrm
1.2.0

vscode-language-pack-de
MS-
1.56.2

mssql
ms-
1.10.1

remote-ssh
ms-
0.65.4

remote-ssh-edit
ms-
0.65.6

pdf
tom
1.1.0

highlight-matching-tag
vin
0.10.0

vscode-icons
vsc
11.4.0

markdown-pdf
yza
1.4.4

markdown-all-in-one
yzh
3.4.0

Assigning to @JacksonKearl who owns this setting for initial investigation. Please assign it back if the config service is not working as expected.

This is as-designed, unfortunately. The goal is to not allow tracking via images/etc requested from the readme, so the readme configuration value of workbench.startupEditor can not be set at a resource level (workspace settings, folder settings). I've updated the documentation to reflect this.
newUntitledFile is documented to only apply in an empty workbench, so it would not make sense to set in a folder's settings.
I tested none, welcome, and gettingStarted to work correctly, if you see otherwise that's a bug.


Latest version from github is not available on ajax.googleapis.com

Latest on the cdn is
https://ajax.googleapis.com/ajax/libs/webfont/1.6.26/webfont.js
Github has a 1.6.28 release?

I also just noticed this, after wanting to upgrade to 1.6.28
Strange, seeing as the latest version was released over a year ago.

The latest version is available on these two CDNs:
https://www.jsdelivr.com/package/npm/webfontloader
https://cdnjs.com/libraries/webfont

👍
who is in charge of the google cdn entry?

According to other connections into ajax.googleapis.com, I am interested to have 1.6.28 on it. Who is in charge really?!


[AsciiDoc conversion] Tables are rendered as paras

Some tables aren't being rendered. Instead, the table directives are being converted into paras, and the paras are rendered on the page.
Example 1:

Franklin page: https://main--prisma-cloud-docs-website--hlxsites.hlx.live/en/enterprise-edition/content-collections/administration/configure-iam-security/remediate-alerts-for-iam-security
Underlying AsciiDoc code: https://github.com/hlxsites/prisma-cloud-docs/blob/af5e6b7b9e98c1ada9b7a1e20a67d670f71cc97e/docs/en/enterprise-edition/content-collections/administration/configure-iam-security/remediate-alerts-for-iam-security.adoc?plain=1#L229-L255

Example 2:

Franklin page: https://main--prisma-cloud-docs-website--hlxsites.hlx.live/en/enterprise-edition/content-collections/connect/connect-cloud-accounts/onboard-gcp/prerequisites-to-onboard-gcp
Underyling AsciiDoc code: https://github.com/hlxsites/prisma-cloud-docs/blob/af5e6b7b9e98c1ada9b7a1e20a67d670f71cc97e/docs/en/enterprise-edition/content-collections/connect/connect-cloud-accounts/onboard-gcp/prerequisites-to-onboard-gcp.adoc?plain=1#L85-L670
Same table on current Palo Alto Networks doc site: https://docs.paloaltonetworks.com/prisma/prisma-cloud/prisma-cloud-admin/connect-your-cloud-platform-to-prisma-cloud/onboard-gcp/prerequisites-to-onboard-gcp

@iansk this is caused by the issue we discussed Friday, a table nested inside a procedure block


[BUG] Add a creenshot to the README

Describe the bug
The README shows sample code, but not the resulting webpage/app.  I would even put the screenshot first and then introduce the code with a sentence like "this app can be created with only 30 lines of pycob code as follows"

Great idea!

Done! Thanks again!


Restart the Elasticsearch client if it shuts down due to an OOM error

The Elasticsearch client uses the Apache HTTP client to perform HTTP requests. This runs an event loop that allocates memory and, if there is significant memory pressure, the client can stop functioning if memory cannot be allocated. This PR adds code to detect this state and restarts the Elasticsearch client automatically if it has stopped.
See https://github.com/elastic/elasticsearch/issues/42133 and https://github.com/elastic/elasticsearch/pull/57973 for more information.

The OOMs happen because the XMLs are large. It's not dangerous but this PR makes sure that one large file doesn't break all subsequent files.


CWE Integration and Relationships

Integrate CWE's definition of weakness into D3FEND ontology, relate weaknesses to relevant D3FEND Digital Artifacts and components.

@netfl0 shouldn't CWE be managed in a separate file and have a specific namespace? https://github.com/d3fend/d3fend-ontology/compare/feature/netfl0-66-cwe-integration-and-relationships#diff-c9e7a3b5c8509ab3baab259fd3b54f3b3c82fb12d6858868d1f253d0aad105e7R2811
Conflating everything in a single file hinders manageability. Moreover, I suggest having a script that generates the CWE taxonomy out of an authoritative file. The CWE ttl can then be generated by the CI in this repo.
WDYT?

Q1:
Totally agree, however we chose to do this for the near-term intentionally. While there are technical solutions for solving this problem, they tend to be hurdles for newbies and we want to make things very easy. catalog files in particular :)
We intend to address this in the future. Is there an example of the hindrance?
Q2:
The subroutine taxonomy is related to CWE but I would not say derived from.

Is there an example of the hindrance?

CWE updates require merges on the d3fend.ttl, which is already huge;
I can provide a CWE -> Ontology python script if you want

We need to look at the options for dealing with namespaces. AFAIK, we can take the catalog file approach, or we can develop something custom using robot and merges.
Currently I'm leaning towards the robot merge approach. We've seen very odd behavior of toolkits with the catalog file, perhaps we need to learn more about it.
Cc @hack-sentinel

We've seen very odd behavior of toolkits with the catalog file

Feel free to share :)

@hack-sentinel lets integrate the HW CWE's into this branch.

Now has Memory DAs merged into this feature branch.


draggable markers on mobile also drag map with first drag

I thought this was an issue with my own implementation, but I noticed that this happens on the 'switch style' demo as well.
When you first press/drag, it drags the point as well as the map, but every subsequent drag only drags the marker as desired. This happens on my Android Pixel XL as well as my iPhone 7.

Hey @Valid Could you confirm this is still happening with react-mapbox-gl version 3.8.0, we did massive changes on the event logic around drag events for Feature component. In the meantime I opened an issue to improve the demo website to be mobile friendly so that it is easier to test things on mobile: https://github.com/alex3165/react-mapbox-gl/issues/535

I can confirm that this is still an issue in version 3.8.0, as I am trying to go around it somehow.
My idea is to create an overlay div with the pin pointing to the center of the map, and just move the map instead of the pin. I wont even be using a Layer or a Marker, since these move with the map. Just plain position:absolute on an image. But in this case, I also need a way to deal with a shifted map center (https://github.com/mapbox/mapbox-gl-js/issues/1740)
Any suggestions to overcome these issues or to a better solution for my case?

I'm also experiencing this issue with 3.8.0.

I found that the source of the problem for me was the safari window bouncing behaviour on iOS.  Using iNoBounce fixed this for me.

We also experience this issue with version 4.8.6. It seems that map.dragPan.disable() / map.dragPan.enable() is not called in case of touchstart/touchend events on layers with draggable features.

Still issue

Still issue

Still an issue

Still an issue


Replace NODC seanames CSV file with XML

Hmm, tests are passing on my end.  Let me have a better look at this.

Superseded by 582ec5b2f4a6713abc9eee8f8fbc54fcec9cee2a, closing.


Add keccak hashing family into zuse

This implements the keccak family of hashing functions (keccak, sha3 (FIPS 202), shake and rawshake). These are required for my ongoing Ethereum integration, but are worth their own PR.
Invoke as such:
=,  keccak:crypto
=/  input-a=@  (swp 3 'some string')
=/  input-b=@  0xabcd.1234
(keccak-256 input-a)
(sha3-512 input-b)
(shake-128 256 input-a)
(rawshake-224 64 input-b)
::  outputs, in order, cast from @ to @ux:
::  0x83c7.37ad.570e.9f3e.71e0.d280.0958.e447.70d8.12e9.2db2.c175.8626.613d.1e6b.a514
::  0x1cae.3332.3396.c8d7.ddb8.cfbb.3d86.b66f.2273.fe62.0f8a.2d9f.869a.b6ca.cdae.a4cb.4669.b080.f3d4.bb80.71e0.4630.7d8a.ce9b.87ee.bd9a.1391.4d08.7d04.10ea.ed12.eae6
::  0x971d.f044.e825.bed0.dcbe.0650.342d.ebab.1732.5fe0.bd41.16a4.3b8b.388d.44cf.5e34
::  0x9dd2.45ee.bef0.e9a3

Shake has variable output length, so you specify that manually (in bits).
Number atoms can be hashed as-is, but we have to reverse the byte order in strings because endianness.
For the record, these are the resources that helped me understand and implement the mechanics behind keccak. Byte ordering had me confused for longer than I care to admit.

Keccak official reference document
Pseudocode summary of the spec
This textbook chapter
A Python implementation
Padding and byte ordering explanations #1
Padding and byte ordering explanations #2
Easily finding expected output of arbitrary input strings

The code should contain enough comments to match lines of code against parts of the spec.
++sponge function isn't generic currently, it assumes a permutation state of 5x5 words. This is fine for keccak, but may not be appropriate for other algorithms that want a sponge construction. I consider that out of scope for now though, we can always touch it up later.
Before this gets merged, I still want to write a test file for these so you won't have to take my word for it that these actually produce correct output. I've verified is actually does for empty input, some random numbers, and a paragraph of lorem-ipsum, but, you know.
In the mean time, I'm sure there's people who have strong opinions about the interface of this thing and how I put it into zuse. Let's discuss!

Okay, so I did a bit more digging and experimenting, and every Urbit atom that gets passed in needs to be made big-endian for processing.
This... gets a bit hairy wrt input length though, and how that's currently treated. (ie 0b1000 (4 bits) becomes 0b0001 aka 0b1 (1 bit).) We can't really take input with lots of leading zero bits, because then it'll just register as shorter input. So, we need to be able to explicitly specify the amount of bits our input has.
I think the best solution there is to make the interface not input=@ but input=?(@ [@ud @]), so that the input length (bits) can be specified if needed. If not explicitly specified, we should assume it's (mul 8 (met 3 input)) aka bytes without leading null bytes, which is a good fit for the most common use case: hashing a string.
This ties into a small issue I discovered that causes this to not fully adhere to the spec: it can't take input that isn't a multiple of 8 bits. More precisely, the padding function operates on bytes, and so our input gets treated as bytes rather than bits. This makes processing input like 0b1101 incorrect.
Should be a fairly easy fix, will have that pushed soon.

You run into this frequently when implementing crypto algos from languages that don’t understand big numbers. It should be just an atom but it’s something more complicated...
Sent from my iPhone

On Mar 2, 2018, at 2:49 PM, Fang<EMAIL_ADDRESS>wrote:
Okay, so I did a bit more digging and experimenting, and every Urbit atom that gets passed in needs to be made big-endian for processing.
This... gets a bit hairy wrt input length though, and how that's currently treated. (ie 0b1000 (4 bits) becomes 0b0001 aka 0b1 (1 bit).) We can't really take input with lots of leading zero bits, because then it'll just register as shorter input. So, we need to be able to explicitly specify the amount of bits our input has.
I think the best solution there is to make the interface not input=@ but input=?(@ [@ud @]), so that the input length (bits) can be specified if needed. If not explicitly specified, we should assume it's (mul 8 (met 3 input)) aka bytes without leading null bytes, which is a good fit for the most common use case: hashing a string.
This ties into a small issue I discovered that causes this to not fully adhere to the spec: it can't take input that isn't a multiple of 8 bits. More precisely, the padding function operates on bytes, and so our input gets treated as bytes rather than bits. This makes processing input like 0b1101 incorrect.
Should be a fairly easy fix, will have that pushed soon.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

you definitely don't want ?(@ [@ @]), just ++octs. Which I think is how our
existing external-crypto functions work?
On Fri, Mar 2, 2018 at 2:52 PM, cgyarvin<EMAIL_ADDRESS>wrote:

You run into this frequently when implementing crypto algos from languages
that don’t understand big numbers. It should be just an atom but it’s
something more complicated...
Sent from my iPhone

On Mar 2, 2018, at 2:49 PM, Fang<EMAIL_ADDRESS>wrote:
Okay, so I did a bit more digging and experimenting, and every Urbit
atom that gets passed in needs to be made big-endian for processing.
This... gets a bit hairy wrt input length though, and how that's
currently treated. (ie 0b1000 (4 bits) becomes 0b0001 aka 0b1 (1 bit).) We
can't really take input with lots of leading zero bits, because then it'll
just register as shorter input. So, we need to be able to explicitly
specify the amount of bits our input has.
I think the best solution there is to make the interface not input=@ but
input=?(@ [@ud @]), so that the input length (bits) can be specified if
needed. If not explicitly specified, we should assume it's (mul 8 (met 3
input)) aka bytes without leading null bytes, which is a good fit for the
most common use case: hashing a string.
This ties into a small issue I discovered that causes this to not fully
adhere to the spec: it can't take input that isn't a multiple of 8 bits.
More precisely, the padding function operates on bytes, and so our input
gets treated as bytes rather than bits. This makes processing input like
0b1101 incorrect.
Should be a fairly easy fix, will have that pushed soon.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/pull/651#issuecomment-370076655, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABxXhqvfDNM79LBi76hYjpx0bjpMqyqVks5tac0VgaJpZM4SaMNM
.

Oh yeah, almost forgot about that. But judging by its data type (@t), name, and ++as-octs ((met 3 ...) that's a pair of byte-length and content, not bit-length and content. Of course the types still work out, but it's a semantic mismatch.
(also imo it's nice to have the option of a simpler interface (just @), but this may lead to unexpected behavior in uncommon cases so, uh, I concede)

Good catch, Anton.
You’ll only be using keccak in standard ways on standard widths, all of which should fit ++octs...
Sent from my iPhone

On Mar 2, 2018, at 3:13 PM, Fang<EMAIL_ADDRESS>wrote:
Oh yeah, almost forgot about that. But judging by its data type (@t), name, and ++as-octs ((met 3 ...) that's a pair of byte-length and content, not bit-length and content. Of course the types still work out, but it's a semantic mismatch.
(also imo it's nice to have the option of a simpler interface (just @), but this may lead to unexpected behavior in uncommon cases so, uh, I concede)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Pushed fixes and tests. For keccak, we're taking a handful of inputs from the known answer tests. For all others, since they only differ from keccak in their parameters, we test just a single null input
Test file is a tad ugly, I couldn't get the logic in the keccak test arms to work without adding a face to core state.
As for performance, using a 255-byte input it takes about 40 seconds to generate 100 keccak-512 hashes.
(turn (gulf 1 100) |=(a=@ (keccak-512:keccak:crypto 255 (add input a))))
That's under half a second per hash, which isn't too shabby for Hoon code without new jets. And in most of my upcoming cases (Ethereum integration), inputs will be significantly shorter. (For 100 bytes, the above takes about 21 seconds. For 60 bytes, a mere 11.)
As far as I'm concerned, this is ready to be merged after final review.

Actually, are there even any use cases where you want the hash value instead of just the string? Thinking we should maybe just output a tape or @t instead of an @...

That’s going too far!  But I can’t say why :-)
Sent from my iPhone

On Mar 7, 2018, at 7:13 AM, Fang<EMAIL_ADDRESS>wrote:
Actually, are there even any use cases where you want the hash value instead of just the string? Thinking we should maybe just output a tape or @t instead of an @...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

At the very least we want helper functions somewhere for pretty-printing @ux in a real-world-usable format. That is, without the .s and with leading zeroes where appropriate. We'll want that for @ub, @ud, @sd too, but this would all be a separate PR since it's not strictly related.
Current status: still ready for merge afaic.

We already have @i but need more of that.  Easy to add.
Sent from my iPhone

On Mar 7, 2018, at 8:07 AM, Fang<EMAIL_ADDRESS>wrote:
At the very least we want helper functions somewhere for pretty-printing @ux in a real-world-usable format. That is, without the .s and with leading zeroes where appropriate. We'll want that for @ub, @ud, @sd too, but this would all be a separate PR since it's not strictly related.
Current status: still ready for merge afaic.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I’ll review this PR today.
I had the same feeling as @cgyarvin about returning a string. I think the
feeling might come from some idea that the lack of use of the formal
numeric output is only incidental. If we didn’t provide a public interface
for it, a use case that appears later would require us to change this code.
Better to implement the convenient semantics on top of the formal ones.
On Wed, Mar 7, 2018 at 8:50 AM cgyarvin<EMAIL_ADDRESS>wrote:

We already have @i but need more of that. Easy to add.
Sent from my iPhone

On Mar 7, 2018, at 8:07 AM, Fang<EMAIL_ADDRESS>wrote:
At the very least we want helper functions somewhere for pretty-printing
@ux in a real-world-usable format. That is, without the .s and with leading
zeroes where appropriate. We'll want that for @ub, @ud, @sd too, but this
would all be a separate PR since it's not strictly related.
Current status: still ready for merge afaic.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/pull/651#issuecomment-371203068, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA1Y9d3lkXlCW33sk3egyQlk9rBodB1wks5tcA9vgaJpZM4SaMNM
.

@eglaysher @Fang- I wonder if you need a face on the testing library because of @ohAitch's change to ++mold in https://github.com/urbit/arvo/pull/644 ?  I feel like that's not the case, but I'm not sure.

At the very least we want helper functions somewhere for pretty-printing @ux in a real-world-usable format. That is, without the .s and with leading zeroes where appropriate. We'll want that for @ub, @ud, @sd too, but this would all be a separate PR since it's not strictly related.

Pull x-co d-co etc out of ++co, maybe? Like, we mostly have the parsers and printers for all of these, they're just in cores in hoon.hoon
-1 on doubling the @u namespace in the manner of @ui, these definitely do not belong as valid hoon literals.

Think Anton’s right here...
Sent from my iPhone

On Mar 7, 2018, at 11:01 AM, Anton Dyudin<EMAIL_ADDRESS>wrote:
At the very least we want helper functions somewhere for pretty-printing @ux in a real-world-usable format. That is, without the .s and with leading zeroes where appropriate. We'll want that for @ub, @ud, @sd too, but this would all be a separate PR since it's not strictly related.
Pull x-co d-co etc out of ++co, maybe? Like, we mostly have the parsers and printers for all of these, they're just in cores in hoon.hoon
-1 on doubling the @u namespace in the manner of @ui, these definitely do not belong as valid hoon literals.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Forgot to mention the tests depend on #659, but that will be in soon.


Tracking issue for our cosmwasm-std PR

https://github.com/CosmWasm/cosmwasm/pull/1483

@aeryz probably great to upgrade our patch to the latest cosmwasm-std version

yeap, it seems like 1.2.0 is released.

Updated to 1.2.0

September.


Add new keywords for Readline 7.0 / Bash 4.4

I noticed that the latest keywords were missing for readline syntax highlighting in vim, so I added them. They appear in the same order as in the Bash man page, as do the existing ones as far as I can tell, and keywords that are Bash specific are properly in the section for readline_has_bash.
There is also a whitespace change (removal of a trailing blank).

Thanks for the update. I had fixed it back it December to work for my needs, but foolishly never checked to make sure I had added all of the new keywords for v7.


Further load test resource requests

Further follow up from https://github.com/kubernetes/perf-tests/pull/2031 and https://github.com/kubernetes/perf-tests/pull/2032
This should help with #1311 - currently the pods relatively small, so scheduling another one often doesn't change the scores for a given pod for subsequent pods. Increasing the number of resources means that the score will be changing more frequently.
/assign @mborsz

/lgtm


Compatibility with jupyterlab-1.0 alphas

https://github.com/pyviz/pyviz_comms/pull/34

Still some Hope ?

Apologies never came back to this. Want to fix the CI before I tag.

Will try to do it this afternoon.

Okay, didn't get there over the weekend but I just pushed it now, you can get it with:
jupyter labextension install @pyviz/jupyterlab_pyviz@next

which is also mentioned in the README.


.Net: Fix redis package reference

Motivation and Context
Fix #2953
Description
Looks like, since #1799, there's been a problem. However, due to references from other projects, tests and samples have not shown the problem.
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

Is there a better way to solve this problem?
Tested with a simple project, works fine with .net7.0 and .net8.0.
using Microsoft.SemanticKernel;
using Microsoft.SemanticKernel.Connectors.Memory.Redis;
using sk_redis_test;

var apiEndpoint = Env.Var("AzureOpenAI:Endpoint");
var apiKey = Env.Var("AzureOpenAI:ApiKey");
var kernelBuilder = new KernelBuilder();
kernelBuilder.WithAzureChatCompletionService("gpt-35-turbo", apiEndpoint, apiKey);
kernelBuilder.WithAzureTextEmbeddingGenerationService("text-embedding-ada-002", apiEndpoint, apiKey);
using var memoryStore = new RedisMemoryStore("localhost:6379");
kernelBuilder.WithMemoryStorage(memoryStore);
var kernel = kernelBuilder.Build();

var collectionName = "test";

for (int i = 1; i <= 10; i++)
{
    var key = await kernel.Memory.SaveInformationAsync(collectionName, i.ToString(), i.ToString(), i.ToString(), i.ToString());
    Console.WriteLine(key);
}

await memoryStore.DeleteCollectionAsync(collectionName);

<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net8.0</TargetFramework>
    <RootNamespace>sk_redis_test</RootNamespace>
    <ImplicitUsings>enable</ImplicitUsings>
    <Nullable>enable</Nullable>
    <UserSecretsId>5ee045b0-aea3-4f08-8d31-32d1a6f8fed0</UserSecretsId>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.Extensions.Configuration.UserSecrets" />
  </ItemGroup>

  <ItemGroup>
    <ProjectReference Include="..\src\Connectors\Connectors.AI.OpenAI\Connectors.AI.OpenAI.csproj" />
    <ProjectReference Include="..\src\Connectors\Connectors.Memory.Redis\Connectors.Memory.Redis.csproj" />
  </ItemGroup>

</Project>

@JadynWong Thanks for quick fix! Yes, we have similar problem in other packages as well. Looks like NRedisStack depends on System.Text.Json >= 7.0.2 which depends on Microsoft.Bcl.AsyncInterfaces >= 7.0.0, so there is version mismatch.

Is there a better way to solve this problem?

Usually, you can just specify concrete package version where it's needed, like this:
<PackageReference Include="Microsoft.Bcl.AsyncInterfaces" Version="7.0.0" />

But with central package management it's not allowed to specify version in PackageReference attribute, so we need to use PackageVersion instead:
<PackageVersion Update="Microsoft.Bcl.AsyncInterfaces" Version="7.0.0" />

We use similar approach in other existing packages as well.
@JadynWong Did you try to remove the line with package version update and verify that issue can be reproduced in your scenario with simple project .net7.0/.net8.0, and then when you add it back, the issue is resolved?

I reproduced the issue in the simple project using the same approach as you described. When adding Microsoft.Bcl.AsyncInterfaces and set the version to 7.0.0, the issue was solved.

@dmytrostruk Yes. I reproduced the issue in the simple project using the same approach as you described. When adding Microsoft.Bcl.AsyncInterfaces and set the version to 7.0.0, the issue was solved.

@JadynWong Sounds awesome, thank you!


[BUG] Using HtmlEditor with a Flutter TextField

Describe the bug
When using the HtmlEditor on Android and iOS with a TextField in the same page, focusing on the TextField and then HtmlEditor, the focus will not go away from TextField
To Reproduce
I have created a minimal bug reproduction project
Steps to reproduce the behavior:

Clone the bug minimal project
Run on Android or iOS
there is a TextField and an HtmlEditor on top of each other
Tap on the TextField to focus on it
Now tap on the HtmlEditor to focus on it

Expected behavior
Tapping on HtmlEditor should remove the focus from TextField, but it does not
Device:
Android and iOS
Additional context
Although I've created a complete reproduction project, here is the simplified Widget that triggers this bug:
class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key}) : super(key: key);

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final controller = HtmlEditorController();

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: ListView(
        children: [
          const TextField(),
          HtmlEditor(controller: controller),
        ],
      ),
    );
  }
}

For now I'm doing it this way, in the editor's onfocus, I remove the focus from the text field, on the other hand I try to focus the editor but the keyboard doesn't open again.

It isn't possible to fix with the current state of webivews in Flutter. They do not handle focus correctly and it is by far one of the worst issues with this package that I wish I could fix.


FancyZones: Override default Windows Maximize/Restore behaviour and apply it on underlying zone instead of the monitor

Description of the new feature / enhancement
I believe that when hitting the Max/Restore icon of a window or when double-clicking on the title bar, the window should get maximized or restored in the underlying zone and not in the underlying window, which is of course the Windows default behaviour unless the user presses and holds the shift key.
Scenario when this would be used?
I should not need to click and drag a window to maximize it in the underlying zone. Hitting the Maximize button or double-clicking the title bar should be enough.
Supporting information
This is the default behaviour of DisplayFusion which I'm trying to replace.

/dup #24263


DASHBOARD RETURNS INVALID REDIRECT URI

Dashboard error.

I have set bot secret to the correct secret. no clue whats happening.

NEVER MIND!!!
If you get this error, heres how to fix
Go to discord dev dashboard and add this to redirect URIs:
http://<IP_ADDRESS>:8080/api/callback
http://<IP_ADDRESS>:8080/selector


Remove non-working welcome screen, fixes #27

The welcome screen (http://peterdotjs.com/quick-find-text-search/#content) is not working, or more precisely, is pointing to the wrong page.

Updating url to point to GitHub README

Sounds good! 😊


Update how-to-connect-password-hash-synchronization.md

Missing word.

@CBrasga : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Running slow on grid with qsub

I was wondering a preferable way to run the python script faster on grid.  This should parallelize but still seems to run very slowly.
Can I split my SAM into numerous sub SAMs and submit as separate jobs? There are non-primary mappings that may end up in different files, would this effect TranscriptClean?

Right now, the best way to cut the runtime is to split up your sam file and then concatenate the results from the different runs at the end. I’m working on a parallel version of TranscriptClean, but unfortunately it is not finished yet.


Validate b3dm

I'm not sure if I did this correctly. Please let me know if there is anything I need to fix.
#9

Sorry for waiting so long to get back about this issue. I tried to start the validateB3dmSpec file and I've been reading about Promises and readTileSpec, but I am still a bit confused on what readTileSpec and validateB3dmSpec is supposed to do..?
I also just made a js file with the methods that would return invalid b3dm tiles, but I am not sure how I would use this to actually generate the tiles and test the method against the these tiles.

Closed in favor of #16


SW-2225 Remove QuantityRepairer

It was a one-time repair job, and now it has finished. Remove it.
DEPLOY NOTE: Tag the commit before this one as a release, and then tag this commit
(or a later one) as a second release. We want the repair job to actually run
before we delete it.

Current dependencies on/for this PR:

main

PR #806

PR #807   👈

This comment was auto-generated by Graphite.

Ugh, GitHub not knowing how to deal with stacks.


Bad link

We try to send our root inclusion request to Microsoft. In the root policy of Microsoft we find an application form in this website https://aka.ms/rootcertapply, but it's a wrong link, we cannot fin the application form we need to apply for the root inclusion, please help me to get this form thank you!

#please-close


Data type issue with BioSemi ActiTwo

I've tested mne-lsl with https://github.com/labstreaminglayer/App-BioSemi and most things seem to work, but when I obtain a new chunk of samples using get_data(), the resulting numpy array consists of floats that are extremely negative (e.g., scaled from -5431.59 to -772.0). I suspect there's a mismatch between data types somewhere and bytes are just wrongly interpreted. Other LSL viewer software works okay.

It's entirely possible that the App-Biosemi is streaming those values. On the mne-lsl side, the array you are getting from StreamLSL.get_data() is an aggregation of consecutive StreamOutlet.pull_chunk operations, which comes down to: https://github.com/mne-tools/mne-lsl/blob/ee8d56ee68bf7c12dad9901a0559df86f146e688/src/mne_lsl/lsl/stream_inlet.py#L396-L398
The datatype used to interpret the bytes is provided by the LSL stream as well, i.e. by the App-Biosemi.
Maybe the viewers you used are applying filters or other processing on the stream, which are absent from:
import time

from mne_lsl.stream import StreamLSL

stream = StreamLSL(bufsize=2).connect()
time.sleep(2)
data = stream.get_data()

You can of course apply such filters on the stream as well, e.g. with the filter method.

Overall, -5431.59 to -772.0 doesn't strike me as strange values for a raw un-processed EEG dataset which usually suffer from DC offset and drifts. For instance, I opened a raw recording from an ANT Neuro eego amplifier with 64 channels gel-caps. In 10s, the recording shows values between -0.0655369 and 0.00927377 Volts. Moreover, the eego2lsl app (equivalent to App-Biosemi) streams data in uV, thus the values in the stream would be oscillating between -65 5536.98 and 9 273.77.

Overall, -5431.59 to -772.0 doesn't strike me as strange values for a raw un-processed EEG dataset which usually suffer from DC offset and drifts. For instance, I opened a raw recording from an ANT Neuro eego amplifier with 64 channels gel-caps. In 10s, the recording shows values between -0.0655369 and 0.00927377 Volts. Moreover, the eego2lsl app (equivalent to App-Biosemi) streams data in uV, thus the values in the stream for this recording would be oscillating between -65 536.98 and 9 273.77.

That's a good point - however, I should have mentioned that the values also seem sign-swapped, i.e. I have a photosensor connected to the amplifier and the signal goes down when I increase the light input to the sensor. Again, other viewers don't show this problem.
But you're comments have prompted me to have closer look once I'm back in the lab!

For a photosensor, or for EEG caps/bipolars channels as well, the "direction of change" depends on the wiring, i.e. what is connected to the + and what is connected to the -. For instance, in our facility we have both:

bipolar photodiodes (homemade, with touchproof connectors) which we can sign-swap by swapping the touchproof connectors
photodiodes to connect to a BrainProduct StimTrak on a 5-pin connector. This 5-pin connector also has a + and - pin (indicated in the pin layout of the manual); and if I would exchange the 2 wires connected on those pins, I would get a sign-swapped signal.

Overall, a sign-swapped signal just means that you have in your mind a convention/preference for the "direction of change"; and that the wiring of your measuring component has a + and - swapped compared to this convention/preference.
The fact that the photodiode actually shows a variation correlated to the light change (even if sign-swapped) is actually very reassuring and sign of a working system.

You’re absolutely right - it’s not really an issue as long as the relative values remain plausible, which they do. However, I’d still like to understand why both the BrainVision LSL Viewer and BioSemi’s own (non-LSL) viewer display the correct orientation for the signal. Could there be something in the LSL stream that other LSL viewers recognise but is somehow missed by mne-lsl?

I think you’re right, although it feels somewhat counterintuitive that BrainProducts would implement specific handling for BioSemi channels. I’ll conduct a full test with actual EEG data on Monday and share the results afterwards.


implementing dynamic loader for TouchUI JavaScript. fixes #989

This is essentially the same technique used for the existing Classic UI libraries. As with that, the existing two libraries (limit and placeholders) are enabled by default for backwards compatibility reasons.

Coverage increased (+0.04%) to 32.709% when pulling 35be622972041f3dd56f95543adef9cb01d70000 on justinedelson:issue/dynamic-touchui-clientlibs into f7424effb1fd68e8d494988fe74c009b4ccab4bb on Adobe-Consulting-Services:master.

Coverage increased (+0.04%) to 32.709% when pulling 35be622972041f3dd56f95543adef9cb01d70000 on justinedelson:issue/dynamic-touchui-clientlibs into f7424effb1fd68e8d494988fe74c009b4ccab4bb on Adobe-Consulting-Services:master.

Thanks for working in a fix for #989 I really appreciate it!


Python load_xdf fails with numpy < 1.14

Have a look at https://github.com/numpy/numpy/issues/11015
rcond=None added in https://github.com/sccn/xdf/pull/17 will break on numpy <1.14
I tried with rcond=-1 and things seem to work alright.

Fixed in https://github.com/xdf-modules/xdf-Python/pull/4.


HW 17.10.16 (Grandle, Flavours)

https://github.com/Aliaksei-Karaliou/OneChat/commit/0018a7544846071dc898fc863199f096a659308d

if you change buildType to release, you project will not compile because in release mode you don't have SOME_STRING variable

e1ef12b

https://github.com/Aliaksei-Karaliou/OneChat/commit/e1ef12b9a15eec9d35cf2695ca99b36aa687db12


Reland fix TextField helper top padding on M3

This is a reland of https://github.com/flutter/flutter/pull/145087 which was reverted in https://github.com/flutter/flutter/pull/145168.
https://github.com/flutter/flutter/pull/146637 was merged and might help here.
Description
InputDecorator adds a 8.0 gap beetween the container and the helper text.
From the Material 3 specification, this gap should be 4.0.
See https://m3.material.io/components/text-fields/specs#0c5c8d6d-2169-4d42-960c-51f6ee42eb57.
This PR sets the correct values for M3 without changing the value for M2.

Before
After
M3 Spec

If this change is accepted, a future step will be to make this value configurable, probably by InputDecorationTheme.
Related Issue
Fixes https://github.com/flutter/flutter/issues/144984
Tests
Updates a value used by several existing tests.

@justinmc Let's see if this is ok now that https://github.com/flutter/flutter/pull/146637 has landed :crossed_fingers:

Google testing started :crossed_fingers:

@justinmc Google testing failed but did it failed on the contrast test that led to the previous revert?

@Renzo-Olivares If you have some bandwith can you have a look to the Google testing failure on this PR.
For context, this is a reland which was reverted due to an accessibility test failure, see https://github.com/flutter/flutter/pull/145087#issuecomment-2000811471.
In the meantime, https://github.com/flutter/flutter/pull/146637 landed so Justin and I wonder if this accessibility test is still failing?

Google test failures are expected due to the visual change, but we need to figure out whether the accessibility failure is still there or not. I will plan to go through those next week and see.

I don't see the accessibility test failure, and I've checked all of the failing image diff tests and they're all expected. Let's try merging this.

@justinmc Thanks a lot for checking this and above all for pointing the right direction in https://github.com/flutter/flutter/pull/145087#issuecomment-2000811471. 🙏
Let see if this change sticks without a revert in the coming days 🤞

@justinmc Looks like you added the auto submit label but forgot to approve the PR.


Longest Increasing subsequence using binary search most optimal approach (Modified)

Description of Change

Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

I did not get what you are trying to say can you please elaborate

is this for hacktoberfest?

Yes this is for hacktoberfest

Why is the pull request called longest increasing sequence but it is an implementation of floyd warshall?

Why is the pull request called longest increasing sequence but it is an implementation of floyd warshall?

now its correct

Why is the pull request called longest increasing sequence but it is an implementation of floyd warshall?

By mistake i have pushed the floyd warshall code with longest increasing sequence now i have removed that code.


How can i hide the automatically detected operating systems

How can i hide the automatically detected operating systems
I only installed OS X and Windows 10.
And would also like to have it as clean as yours.
Thanks

Hi, I'm also interested..


ant/grt: updates for repair_antennas

Does this need to be upstreamed to OpenLane?

Does this need to be upstreamed to OpenLane?

Not yet. This WIP requires a few more updates before going to production. I'm just creating this PR to avoid a PR with lots of updates in the future.