brew update - Deleted Formulae homebrew/dupes/tidy

I did a brew update today and noticed homebrew/dupes/tidy was in the Deleted Formulae.
Is there a reason for this? I was previously using homebrew/dupes/tidy --HEAD for HTML5 linting.

See #456 and https://github.com/Homebrew/homebrew/pull/39010.

Thanks,
OK so now we need to use tidy5 instead of tidy in the command line. I've done a quick fix for myself but this could be a breaker for various plugins and apps that reference just 'tidy' and are expecting to get the HTML 5 version if installed.
FYI, My main usage is with the html tidy linter for SublimeText https://github.com/SublimeLinter/SublimeLinter-html-tidy

It's easy enough to add an optional symlink from tidy5 to tidy. It probably couldn't be mandatory given OS X natively ships a tidy executable we'd be overriding by placing another tidy executable earlier in the $PATH, but the maintainers may not hate the idea if it's optional.

Maybe note it like on the php installers? In the install output text, similar to:-
If you wish to swap the PHP you use on the command line, you should add the following to ~/.bashrc,
~/.zshrc, ~/.profile or your shell's equivalent configuration file:

      export PATH="$(brew --prefix homebrew/php/php56)/bin:$PATH"

Anything on this?
I have been getting really crazy, because of my "newbieness"...
The point is: should I just rename the tidy5 symlink brew automatically creates? Shall this bring any problem?
Thanks.

You could put
export PATH=$(brew --prefix)/bin:$PATH
alias tidy=tidy5

in your ~/.bashrc to solve this.

Would accept a PR to add an optional symlink to the tidy. But I'm not sure it's worth it since in the process of discovering the option, you'd realize what the problem is and just add a shell alias.

Just to add similar to @louisl I use the same Sublime Text linter plugin, the latest version of this now supports Tidy5 see https://github.com/SublimeLinter/SublimeLinter-html-tidy/releases/tag/1.0.15

Yeah, I was dealing with ST2...
Finally I surrendered to the evidence and installed ST3.
Thanks!


Cropped text inside flowgraph rectangles

It seems like every text I put inside quotes is cropped by one character (more or less).
Example:
Code:
---
title: Test chart
---
flowchart
    A["Some text"]
    B["Some text with an equation $Ax+b=y^{2n}$"]

	A -- "This $2x$ cropped too" ---> B

Output image:

It seems like it affects equations more than regular text.

It seems like the problem occurs vertically aswell

Okay, that's weird. The example you sent seems to work for me:

Did you install the plugin from the Obsidian plugin store?
Vertical cropping should not occur either, except for edge labels. But this is a problem of the Mermaid rendering/layout engine and i can't really do anything about it inside the scope of this plugin. If you encounter any other example where things get cropped just let me know.

Yes I did download it from the Obsidian plugin store directly from the Obsidian UI. Note that I am running on Linux in Ubuntu and Gnome, nothing fancy.
I also have the following problems:

If I have two diagrams in the same document, when I switch the current view to "reading" instead of "editing", then switch back, the whole document gets broken and becomes uneditable. I have to reload it. I have the same issue when there is a compilation error in the me(h)rmaid code while in the "live preview" mode.
If I export to PDF and I have two diagrams in the same document, the two diagrams overlap one another and the document becomes unintelligible.

I don't have any of these problems with "vanilla" mermaid.
Other than that I really enjoy the plugin. I just want to be able to write math formulas inside mermaid diagrams. I wish I had the Javascript skills to help you on this.

I have the same problem.

There should be no more issues when switching between reading and edit mode now. The reason was that both views render their own graphs, which lead to non-unique graph ids.
If you still encounter any sort of cropping, let me know and maybe give me some more detailed information on when this occurs.

Indeed, the Edit and Read mode bug seems to be fixed for me. But I still see the some cropping issue, but it did get better for the nodes at least (I still see it a lot for the text between the nodes, eg: A -- "The text here" --> B).

Also, I just realized that this bug doesn't happen when the following parameter is ON:

I'm using a CSS hack to get rid of the text cropping. This is what I've added to my snippets:
.mehrmaid-markdown-container {
  padding: 0 8px 0 0;
}

Before:

After:


Fix for client side anti cheat crashing player if they use deprecated collision group methods

This fixes the anti cheat crashing players if their console gets a deprecated collision group method warning. It was falsely triggered by matching getreg from GetRegisteredCollisionGroups in the warning message.

This is already fixed but the uploaded Adonis module is old

oh i didnt realize that was the case. Well this will just be a add myself to credits pr then


Moving from json tags to ion tags for Marshal/Unmarshal

Resolves #63
Given that users will be working with Ion, this change is more for clarity and moves away from the json tag to using the ion tag when marshalling and unmarshalling.

Codecov Report

Merging #64 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #64   +/-   ##
=======================================
  Coverage   76.17%   76.17%
=======================================
  Files          22       22
  Lines        4625     4625
=======================================
  Hits         3523     3523
  Misses        679      679
  Partials      423      423

Impacted Files
Coverage Δ

ion/fields.go
91.22% <100.00%> (ø)

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 82b4807...5885b08. Read the comment docs.

Oh wow, I vividly remember basing this on a yaml implementation somewhere that just reused the ‘json’ tag, but I just did a check aaaand nope all of the ones I’m familiar with use ‘yaml’. 😅😅😅 :+1: for switching to ‘ion’.


Sorting options other than by total size (IDFGH-12428)

Title.
At the very least, an option to sort the text (table) by IRAM is a necessity.
If possible, the ability to do multi-key sorting of the table, csv and tree formats would be really great for deeper analysis of the binary composition.

Hello @KaeLL ,
thank you very much for the suggestion. Do you think that something like sort -k behavior would be sufficient for table and csv format. Meaning there would be another command line option specifying by which column the table should be sorted. So e.g. idf.py size-components -c 2 would sort the table based on column 2. The names and output sections may differ on different targets, so this approach should be more generic.  The tree sorting could be probably done based on levels.  What do you think?
Thank you

Why isn't lexicographical sorting of .a(rchives) names supported (sorting of column 0)? I'd like to separate IDF's components from my projects' components, which can be easily done by sorting column 0 of a project which components' names have a common prefix, like mine do. By itself, that may not be reason enough, given that it's a me problem, but there's also the argument of consistency, as there's no reason why the archive names shouldn't be a valid criteria.

Hello @KaeLL,
Thank you for your feedback. The primary reason is likely that the archive names, or any row names in the archives/files/archive_details tables, do not include the diff value, which can also be used as a sorting value when the --sort-diff option is applied. I understand your point that this might seem inconsistent. Could you please file a new ticket for this issue? I will look into it.
Thank you very much.


MacOS Env Variables

how to start ollama with OLLAMA_ORIGINS=* OLLAMA_HOST=<IP_ADDRESS>:11435 ?

OLLAMA_ORIGINS='*' ollama serve
But before that make sure to stop the running service, for example:
> launchctl list | grep ollama
70718	0	application.com.electron.ollama.61282955.61282961
> launchctl stop application.com.electron.ollama.61282955.61282961

Is there a way to enclose it into the launchctl script ?

Yes! This can be done here: https://github.com/ollama/ollama/blob/main/docs/faq.md#setting-environment-variables-on-mac
In future versions of the macOS app this will get easier to do in Ollama itself


nimja has no tagged releases

This repository does not have tagged releases, so I cannot install previous versions.

I've created tags, please try again.


make "sbt run" working and small fixes so that users can follow instructions on readme

Thanks for the patch. Most of these issues have been fixed during these days commits.
For ChanaAkkaHttp, until it's been optimized for performance, it won't be the default one.


Please implement attendance check on the 7th and attendance check on the 30th.

hi
I'd like to suggest an additional attendance check function.
Thank you.

Try 1.21

Thank you. I'm currently applying it. I'll let you know if there's a
problem.
2022년 11월 1일 (화) 오전 7:13, Ittipon Teerapruettikulchai <
@.***>님이 작성:

Try 1.21
—
Reply to this email directly, view it on GitHub
https://github.com/insthync/rpg-php-service/issues/6#issuecomment-1297753391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5T3U34ZT6WQHCLU7HOTO3WGBACJANCNFSM5UVIS6DA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Error everytime pikabot runs for around 1hour

So,everytime my pikabot runs for around 1 hour,it shows error like this :

thank you

According to this thread, your IP address may be banned : https://github.com/PokemonGoF/PokemonGo-Bot/issues/4908

banned?if my ip has been banned,how can I still use it?

It is banned from Niantic servers, not from the Internet.

I had the same issue when there was a problem inside config file. For sure one pokemon name was written in a wrong way but I don't remember another mistake.

Please use issue template


Object types

What

Policy types

type PolicyType interface {
	GetGVK() schema.GroupVersionKind
	GetInstance() client.Object
	GetList(context.Context, client.Client, ...client.ListOption) ([]Policy, error)
	BackReferenceAnnotationName() string
	DirectReferenceAnnotationName() string
}

Type interface for all Gateway API types

type Type interface {
	GetGVK() schema.GroupVersionKind
	GetInstance() client.Object
}

Simplified the Policy interface: it should not have anything related to kuadrant.

Useful types to write common controller mappers and controllers that can be used by any policy type.
Work being done as part of https://github.com/Kuadrant/kuadrant-operator/issues/530
This is part of a series of PR's that will end up with a new controller that will only reconcile limitador limits configuration.

@Boomatang I would appreciate your :eyes: on this PR :bowing_man:

I see what is being done and kinda why it is being done. What I am concerned about is the adding of yet more interfaces and structs that we need to be aware of.
My mind is in two places over wide interfaces compared to deep interfaces. What is your view on the abstraction and abstracting to early?

I plan to  write common (reusable) controller mappers and generic (reusable) controllers that can be used by any policy type. This is kind of preparation work for it.


Sort default response

Feature Request
Desired behaviour
It appears that our default published tools and published workflows responses do not sort.
Sort them by something sane, like number of stars so that this doesn't always occur in the GUI.

Implemented by #856


Removing an image causes lockup

I'm using the following dependencies:
"dependencies": {
    "react": "17.0.2",
    "react-native": "0.68.1",
    "react-native-awesome-gallery": "0.3.5",
    "react-native-gesture-handler": "^2.4.1",
    "react-native-reanimated": "^2.8.0"
  },

Once an image is removed from the array, gestures stop being recognised. If you change the removed image position from 0 to say 2, it only locks up when you scroll to index 2 in the image array.
Seems to be a reoccurrence of https://github.com/Flair-Dev/react-native-awesome-gallery/issues/19.
import React, {useState} from 'react';
import {Button, View} from 'react-native';
import AwesomeGallery from 'react-native-awesome-gallery';
import {GestureHandlerRootView} from 'react-native-gesture-handler';

const IMAGE_DATA = [
  'https://cdn.pixabay.com/photo/2017/02/20/18/03/cat-2083492_960_720.jpg',
  'https://cdn.pixabay.com/photo/2014/04/13/20/49/cat-323262__340.jpg',
  'https://cdn.pixabay.com/photo/2015/11/16/14/43/cat-1045782__340.jpg',
  'https://cdn.pixabay.com/photo/2021/10/19/10/56/cat-6723256__340.jpg',
];

const App = () => {
  const [images, setImages] = useState(IMAGE_DATA);

  const removeImage = () => {
    setImages(i => i.slice(1));
  };

  return (
    <GestureHandlerRootView style={{flex: 1, backgroundColor: 'red'}}>
      <AwesomeGallery
        data={images}
        keyExtractor={(item, index) => item}
        doubleTapInterval={150}
        onTap={() => console.log('tap')}
      />
      <View style={{position: 'absolute', top: 0, left: 0}}>
        <Button onPress={removeImage} title="REMOVE"></Button>
      </View>
    </GestureHandlerRootView>
  );
};

export default App;

Seems I'm getting the app frozen ("locked") some of the times I swipe to other positions and turn back fast to the previous photo.

I would say it is related somehow, but I don't have any clue on the issue.
Anyone experienced something like that?

This might not be the most elegant solution, but I added a key with the length of the data array and it's working.
<AwesomeGallery     key={images.length}     data={images}     keyExtractor={(item, index) => item}     doubleTapInterval={150}     onTap={() => console.log('tap')} />

Released in v0.3.6


[Question] Using redis both for cache and for signalR

Hi, I was wondering if in the followin scenario I would break any of the functionalities of CacheManager when using Redis.
Lets assume I have multiple instances of an application that use CacheManager with following setup:
CacheFactory.Build(settings =>
                {
                    settings.WithSerializer(typeof(GzJsonCacheSerializer))
                        .WithUpdateMode(CacheUpdateMode.Up)
                        .WithMaxRetries(3)
                        .WithMicrosoftMemoryCacheHandle("InMemoryCache")
                        .WithExpiration(ExpirationMode.Absolute, TimeSpan.FromMinutes(1))
                        .And
                        .WithRedisConfiguration("RedisCache", "ConnectionStringHere")
                        .WithRedisBackplane("RedisCache")
                        .WithRedisCacheHandle("RedisCache")
                        .WithExpiration(ExpirationMode.Absolute, TimeSpan.FromMinutes(15));
                });

I would like to use the same Redis Server for signalR purposes - it also works based on Pub/Sub mechanism, but does not store any data.
I assume everything will work correctly as cacheManager and signalR would use different channels of Redis server. Am I correct?

I assume everything will work correctly as cacheManager and signalR would use different channels of Redis server. Am I correct?

I would assume so, too. yes. I'm pretty sure SignalR's Redis implementation uses a named channel and doesn't listen to everything, Same as my stuff does, so, it shouldn't cause any interference


Domoticz server settings not saved

Bridge control settings page--domoticz server entries are not saved.  Note, I do have a Vera configured as well, and those settings are saved.
Version 4.5.6 on RPi 3

Just to make sure, you hit the green add button to the right of the settings row?

Yes, I do hit the green save button, and the Bridge restarts, but the entry
disappears.  Other factors that may be at play--my HA-bridge controller is
on a different subnet than the Domoticz server (but so is my Vera), and I
don't currently have Hue-controllable devices on the Domoticz.
--Rob
On Mon, Oct 30, 2017 at 7:45 AM, BWS Systems<EMAIL_ADDRESS>wrote:

Just to make sure, you hit the green add button to the right of the
settings row?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bwssytems/ha-bridge/issues/772#issuecomment-340447929,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Afpbfk-KG8HzlqWz7Go_MgjHK8hjjjIgks5sxdL4gaJpZM4QKDl6
.

I cannot get this to occur.

OK.  It must be something unique to my environment.  When I get some time,
I will spin up another server and see if I can replicate it or hunt down
the root cause.  Thank you for checking!
--Rob
On Wed, Nov 15, 2017 at 1:42 PM, BWS Systems<EMAIL_ADDRESS>wrote:

I cannot get this to occur.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bwssytems/ha-bridge/issues/772#issuecomment-344722148,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Afpbfj1WCyNQxxJ8pW7X2UOtEw3lSKcOks5s20zMgaJpZM4QKDl6
.


Change how space tip is presented to user - add quotes.  Fixes #580

Changes how the space tip is presented, and Fixes issue #580

Hey @mcowger! Thanks for the PR. As the bot says, we need to ensure you've signed the CLA before merging even a trivial text change like this. Perhaps you could join a GitHub org that you know has already signed?

I've sent over the CLA to the CF.org people already - I'm sure they can work it out.  I work for EMC, which has already signed, but EMC doesn't let most employees join their GitHub org (life at big companies!), including me :(.  I'm a member of the emccode org, but thats separate.
I totally understand the need to wait to merge until we have the stuff signed.  I just prefer to get stuff done and send over the PRs while we wait for the business to catch up :).

@mcowger these are all of the currently permitted orgs:

18F
84codes
ActiveState
att-cloudfoundry
blueboxgroup
altoros
anynines
appfog
cfibmers
cfcisco
clarkparsia
cloudcredo
cloudfoundry
codethinklabs
ECSTeam
emcpcf
fareoffice
Forio
fujitsu-cf
hpcloud
HuaweiTech
ibm
innovationfactory
intalio
iron-io
libertymutual
lds-cf
mendix
nirds
nttlabs
nurego
OGIS-RI-EOS
Orange-OpenSource-Cf
piston
pivotal
pivotal-cf
Pivotal-Field-Engineering
pivotal-golang
rakutentech
redislabs
sap-cloudfoundry
sassoftware
searchbox-io
sendgrid
swisscom
symcfmembers
Tier3
uhurusoftware
vchs
velankanisys
zephirworks
ZJU-SEL

I'm now a member of the emcpcf group which is on the list - should be good from a CLA perspective now.


Add support for custom Meta-Data (aka properties) in Cloud Template

What feature do you want to see added?
We need to set 'Meta-Data' properties when creating cloud machines, (--property option when doing a server create command)
It seems the current template options do not expose this option.
Could it be possible to add a list of properties passed to the server when it's created ? (simple list of key-value)
Upstream changes
No response
Are you interested in contributing this feature?
No response

Hi,
we're looking for too the capability to setup meta-data on agent created for resource monitoring purpose.
I was supposed the sub menu "node properties" was used for but not, perhaps this menu should be the right location to add meta-data keys/values we can add on related template.
Thanks.

I created a small MR to add this functionality (first time for me)
@olivergondza could you have a look at it please ? Your guidance will be appreciated :)
Tests seems to have been broken for quite some time and need attention. Is someone already looking into this ?
I'm also getting some errors related to building to java11, which is normal, but even if I specify JDK 17 + 21 in the jenkinsfile, it's still trying to build on JDK11


SSH session content or WSL session content is not being recognized in the accessible terminal buffer

Does this issue occur when all extensions are disabled?: Yes/No
Yes

VS Code Version: 1.76.0-insider
Commit: 92da9481c0904c6adfe372c12da3b7748d74bdcb
Date: 2023-02-28T11:50:30.886Z
OS Version: Windows 11 22.621.1265

Steps to Reproduce:

Open VS Code.
Open a terminal via Control+ or Control+Shift+`.
create a SSH session or type "wsl" in the terminal editor (Make sure WSL is installed)
Type any command such as ls or mkdir test as soon as the SSH session or WSL instance opens up.
Notice, the output will be announced
Now move to the accessible terminal buffer by using Shift+Tab keystroke.
Try to navigate/read the content
Notice, the content of the SSH session or WSL will not be read out.

I hope a fix for this lands soon.

@meganrogge I'm guessing this is already fixed.

yes, very old version. pls update VS Code


Move background initialization to the main CorrectBackgroundAccurate pipeline step.

The functionality provided by the CreateBackground pipeline step can be folded into the main CorrectBackgroundAccurate step without loss of functionality. This would simplify the code base and avoid some duplications.

related to #161


Poll: Create starter repos

Summary
I've noticed that while Alembic can be used as a proper jekyll theme and as remote theme on the GitHub Pages platform, I'm still seeing people fork the repo and mod it accordingly. This isn't a problem, infact it's great to see people using the project to build their own sites and projects. Forking the repo is a good way to lock down the version updating and have all the files needed to modify to your own needs.
However I'd like to make it easier for people to setup their projects using Alembic as a theme and on on Jekyll-friendly platforms like Siteleaf, Netlify and more. If you'd like to give input on this concept then use one of these responses on this description:

👍 : I like this idea and would use it
👎 : I'm happy with the current state
🎉 : I like this, make a starter for my fave platform too
😄 : I like this, but just a single starter repo will do

😄 I've completely hit this issue with GitHub pages! I thought it was amazingly easy to use the GitHub remote theme gem to use the theme, but most of why I like the theme is because of all f the fancy bits and bobs that you have included. Examples of pages, buttons, navigation, images, etc. After figuring out the GitHub pages remote gem theme install and not seeing all the extra goodies, I ended up downloading the repo and getting it to work locally, then git pushing to my GitHub.io repository. Benefit: can run local because I now have all the relevant gems installed for that. Con: amount of frustration bumbling through it as a newb, and the likely unnecessary components that I now have on my GitHub.io. I would have found an extra set of instructions tacked on to the "As a GitHub Pages remote theme" tutorial super helpful, specifically including what contents need to be copied over for one to easily use/tweak all of the pre-made goodies that go are shown on the example page. Thanks for designing this, btw, I do like it or I never would have struggled to get it working with my woeful lack of knowledge on a Sunday ;p

@3CatMax hey, thanks for using my theme! It's great that you got it all setup, but shame it wasn't as smooth as I'd hoped. So you were struggling to get the specific includes working? Or was it the gems that are setup as dependancies in the theme? We might be able to get this solved and not only will you have the theme hooked up right, but I can also update my documentation to help in these sorts of scenarios

Look to the even more basic than includes or dependancies ;p Following the GitHub instructions to install Jekyll was easy, but it doesn't give the user anything to build off of, not even a _posts folder. Installing the alembic theme gem works perfectly(as far as I can tell). The problem for the user that knows nothing about Jekyll (me) is that the theme "just" changes the colors and text of the demo page and does nothing else (add content folders/files), which I don't think it is created to do anyway.
For example, adding the basics of the website (boilerplate?):

home page
navigation bar
categories page
search bar
hero image
buttons
social icons
etc.

Since the GitHub pages custom theme didn't add all the actual content of folders and files (_posts, _includes, _layouts?, _sass?, assets, blog, and all the files in the home folder ;p) so that I could mess around with those instead of starting from scratch, that is when I realized I would need to add all of those files to get it to look the same as your example (https://alembic.darn.es/).
My problem is that I don't reeeaaaaallly understand all of the steps and regular practices using Jekyll and I am only familiar with pushing my website through Visual Studio's git tools ;p Which means I don't know which files I don't need when using the alembic remote theme gem (can list in .gitignore on my local copy and can delete on my GitHub.io directory). I'm pretty sure that I have everything working correctly on my GitHub.io site (maybe ;p Mostly still looks like your example as I am only slowly making changes at this point).
What would have helped me:
Append to the "As a GitHub Pages remote theme" instructions:

list of folders to add to the users own GitHub.io directory
list of files in the home folder to add the users own GitHub.io directory

Past that, if I find any _includes or dependencies that don't work, I'll try to take notes and check  back in ;p

This makes sense now! Thanks for writing out that detailed reply. I'll be honest, I had a downloadable boilerplate at one point that included those very files you needed. However it became hard to manage in a single repo, which is probably what sparked this issue in the first place.
From what you've said on how you're using the theme that seems just fine, you'll miss out on theme updates but they happen a lot less now as I've got to what I think is a "feature complete" state. I'll definitely look into making a starter repo though. In the meantime it might be worth checking our my starter repo that deploys it to Netlify https://github.com/daviddarnes/alembic-netlify-cms It does come with some new learnings but you'll be in sync with the theme and you'll be able to use gems that GitHub Pages haven't whitelisted. Buuuuut from the way you're talking about your setup you seem to have it just the way you like it :)

I think my brain/sponge is already filled with all it can take with new learnings for the nonce ;p It might be worth it to add that link to the read.me file for this repo for any one else that doesn't know what they are doing and might decide to go with the cms approach instead. Thanks though!

I wouldn't discredit yourself! You've done really well for someone who is new to all this!

I made some starter kits if you want to try them out! https://alembic.darn.es/#quick-setup

I tried out your starter kit for the GitHub pages and think that is a step in the right direction. Still, the theme doesn't even show up yet, and I would argue that if a user wants to get up and running quickly with GitHub pages, they should go straight for the remote theme, which you can actually add right into your starter gem file and _config.yml file.
My opinion to make even gooder ;p

[ ] Add gem "jekyll-remote-theme" to the Gemfile
[ ] Add jekyll-remote-theme to the plugins list in _config.yml
[ ] Add remote_theme: daviddarnes/alembic to the _config file
[ ] Remove theme: alembic-jekyll-theme from the _config file
[ ] change the readme file to add the configuration information only since it is for GitHub pages via the ZIP selection.

This gives immediate theme colors and nav bar instead of a white page with black text but as a boilerplate, adding more to flesh it out would be helpful.
Things that would make it fantastic for my use case:

icon on top left
right align nav menu
the elements page (could be titled the about page with the side box and all the nifty social icon links)
the hero image and text
the  button examples (I don't know what short codes are😅)

Thanks for picking this up again, btw!

@3CatMax So are you saying the main Alembic theme repo should be changed to mainly use the remote theme mode? Alembic was designed to be a Jekyll theme gem first, GitHub Pages remote theme feature is a ring-fenced version of that. I'd prefer to keep it as a proper theme gem first, and provide the remote theme as an option.
I think you're right with the fleshing out though. Adding a generic logo would be useful for all of the starter kits. The nav should right align when a logo is present in the config. The feature image I will add to the some of the pages, but the elements page is a bit much as it's only meant to demo the shortcode (Jekyll includes). Would it help if the elements page on the theme site had them below each item?

Closing this as I've made a couple of starter repos for people to get rolling with Alembic quicker https://alembic.darn.es/#quick-setup

This is so awesome, thanks for putting in the extra work!


[Jira Time Tracking] Filter by Assigned to me

Extension
https://www.raycast.com/niallpaterson/jira-time-tracking
Description
At the moment when I try using this I get the full list of all Jira tickets from within my organisation. This is massive and I cannot really find my tickets easily.
Could it be possible to add like a setting to only show tickets assigned to me?
Who will benefit from this feature?
No response
Anything else?
No response

@louisdb03 if we allowed you to set a custom SQL via preferences you could set it yourself to only show tickets assigned to you. Would that work?

@raycastbot assign me

@haydencbarnes I could try. If I can find the DB schema of Jira somewhere, I'll probably figure it out.

@louisdb03 okay, the basics are here:
https://support.atlassian.com/jira-service-management-cloud/docs/use-advanced-search-with-jira-query-language-jql/
I will work on this at some point this week and get an update out soon

@louisdb03 Since not everyone knows JQL, I just added it as a separate preference, for now, to make it as easy as possible to set. If it is decided/requested, we need to be able to edit the actual JQL, and I can make another PR for this. Sorry, it took me a week longer than expected. Here is the PR: https://github.com/raycast/extensions/pull/15568

@raycastbot close this issue


Changelog domain for self-hosted Gitlab

Question

How can I change the domain url included in the changelog?
When I run poetry run semantic-release --strict version, I get https://gitlab.com instead of https://gitlab.my-domain.com as the link for each commit inside my changelog.
Configuration

Semantic Release Configuration
[tool.semantic_release]
assets = []
build_command = "poetry build -n"
commit_message = "{version} [skip ci]\n\nAutomatically generated by python-semantic-release"
commit_parser = "angular"
logging_use_named_masks = false
major_on_zero = true
tag_format = "v{version}"
version_toml = ["pyproject.toml:tool.poetry.version"]
version_variables = ["bmt_scripts/__init__.py:__version__"]

[tool.semantic_release.branches.main]
match = "(main|master)"
prerelease = false
prerelease_token = "beta"

[tool.semantic_release.changelog]
changelog_file = "CHANGELOG.md"
exclude_commit_patterns = [
  "^Merge branch '.*'$",
  "^Merge pull request #.* from .*$",
]
template_dir = "templates"

[tool.semantic_release.changelog.environment]
autoescape = true
block_end_string = "%}"
block_start_string = "{%"
comment_end_string = "#}"
comment_start_string = "{#"
extensions = []
keep_trailing_newline = false
lstrip_blocks = false
newline_sequence = "\n"
trim_blocks = false
variable_end_string = "}}"
variable_start_string = "{{"

[tool.semantic_release.commit_author]
default = "semantic-release <semantic-release>"
env = "GIT_COMMIT_AUTHOR"

[tool.semantic_release.commit_parser_options]
allowed_tags = [
  "build",
  "chore",
  "ci",
  "docs",
  "feat",
  "fix",
  "perf",
  "style",
  "refactor",
  "test",
  "merge",
]
minor_tags = ["feat"]
patch_tags = ["fix", "perf"]

[tool.semantic_release.remote]
ignore_token_for_push = true
name = "origin"
type = "gitlab"

[tool.semantic_release.remote.token]
env = "CI_JOB_TOKEN"

[tool.semantic_release.publish]
dist_glob_patterns = ["dist/*"]
upload_to_vcs_release = true

Recommend reading up on the configuration documentation for defining the remote.


Fix link for Compensating Transaction Pattern

@nzthiago,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/patterns/event-sourcing.md
:white_check_mark:Succeeded
View

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

Good catch, thanks!


Error when creating widget.

When trying to import StringWidget (or any other widget) it gives me an error: Type 'any' is not a constructor function type. on the last line.
import { Component } from '@angular/core';
import { StringWidget } from 'angular2-schema-form';

@Component({
  selector: 'mdl-sf-string-widget',
  templateUrl: './string.widget.html'
})

export class MyStringWidget extends StringWidget {}

How should I resolve this error?

Here is a working example https://github.com/ebrehault/ng-sf-mdl-demo
I would be curious to know what was wrong in your case (so I can update the doc).

Yes. After importing the default widgets like that, the error is gone. But now I get another error: Cannot find name 'require' on this line: let ZSchema = require('z-schema');.
I did try to install requirejs through node, but that doesn't seem to work either.

Which command are you using to build your angular project ?

I used ng build to build the project... It gives me two exact same errors: Cannot find name 'require'.

Try installing @types/node :
npm install @types/node --save-dev

Still the same two errors...

Add declare var require: any; in your src/typings.d.ts and the compiler should be happy.

It's working! Thanks a lot 😄


paths are unexpectedly resolved from context

Problem
This package says that it resolves extends, and is intended to work how the TypeScript compiler / language services do in figuring out your current configuration. But it doesn't do that.
Specifically, when TypeScript language services are resolving paths, any value for paths within tsconfig.json is resolved relative to the local tsconfig.json file (similar to Node resolution for modules). However, when this library is merging tsconfig configurations, it neither resolves paths nor keeps track of where those paths were declared.
This results in the "final" tsconfig having a paths definition that is altogether different from how TypeScript would resolve it, which makes the output difficult to use with a package like alias-reuse.
The use case is that with a Nuxt setup, a sub-directory is generated with a generated tsconfig.json file. I can manually resolve those paths knowing that the path information from get-tsconfig is wrong, but I thought it would still be good to file a bug.
Expected behavior
Either paths should fully resolve into full paths, or there should be some way to resolve those paths by knowing which file they came from OR there should be an option to do either. (Ideally, it would just resolve into the actual path.)
Minimal reproduction URL
https://stackblitz.com/edit/stackblitz-starters-em1thz?description=Starter project for Node.js, a JavaScript runtime built on Chrome's V8 JavaScript engine&file=tsconfig.json,package.json,index.mjs,deep%2Fdeeper%2Ftsconfig.json,deep%2Ftsconfig.json&title=node.new Starter
Version
4.7.3
Node.js version
v18.18.0
Package manager
npm
Operating system
macOS
Contributions

[ ] I plan to open a pull request for this issue
[ ] I plan to make a financial contribution to this project

I see what you mean:
❯ tsc --showConfig
{
    "compilerOptions": {
        "paths": {
            "@": [
                "../../src"
            ],
            "@/*": [
                "../../src/*"
            ]
        }
    },
    "files": [
        "./src/t.ts" // test file I added to make `tsc` work
    ]
}

But this deviation was a deliberate choice because users cannot use this data without the context that ../../src is relative to ./deep/deeper/tsconfig.json.

Also, please try to be concise. I usually just read the code.

Closing as this is by design, and there's no follow-up proposal.


Failed to load dependencies

In agreement with similar issues, it seems that kscript is unable to resolve dependencies.
I am trying to use a kscript posted by this repo's author here: https://www.biostars.org/p/64346/
Using the author's docker image, I tried to run this scriptlet:
docker run -i holgerbrandl/kscript https://git.io/vbJ4B features.bed
Output:

[kscript] [ERROR] Failed while connecting to the server. Check the connection (http/https, port, proxy, credentials, etc.) of your maven dependency locators. If you suspect this is a bug, you can create an issue on https://github.com/holgerbrandl/kscript
[kscript] [ERROR] Exception: org.sonatype.aether.resolution.DependencyResolutionException: failed to load 'de.mpicbg.scicomp:kutils:jar:0.9.0 (runtime)' from ["jcenter (https://jcenter.bintray.com/, releases+snapshots) without authentication"] into /root/.m2/repository

I also tried running this locally:
docker run -i holgerbrandl/kscript - < gtf2bed.kts Idua_features.bed
Output:

[kscript] Resolving dependencies...
[kscript]     Resolving de.mpicbg.scicomp:kutils:0.9.0...[kscript] [ERROR] Failed while connecting to the server. Check the connection (http/https, port, proxy, credentials, etc.) of your maven dependency locators. If you suspect this is a bug, you can create an issue on https://github.com/holgerbrandl/kscript
[kscript] [ERROR] Exception: org.sonatype.aether.resolution.DependencyResolutionException: failed to load 'de.mpicbg.scicomp:kutils:jar:0.9.0 (runtime)' from ["jcenter (https://jcenter.bintray.com/, releases+snapshots) without authentication"] into /root/.m2/repository

Closed because of inactivity. Feel welcome to reopen if needed.


Fix setting cgroup permission to user/privileged devices

Fixes #21466
@vdemeester @jfrazelle
Signed-off-by: Tonis Tiigi<EMAIL_ADDRESS>

is there a way to add an integration test 0:)

Testing this manually, it looks OK with docker v1.10.3 and v1.9.1 inside the container, and it seems it is fixed (although I have failure with 1.9.1 and the test intgeration suite of libcompose but I think it's something else).
Thanks @tonistiigi

@vdemeester @jfrazelle I noticed that we already had tests that read cgroups files directly to check the applied settings. I added these new cases like this also.

LGTM

LGTM

all green! merging


PoC: Automatic upstream upgrades via Dependabot

Try to have Dependabot create automatic updates in Pulumi providers when the upstream TF provider updates.
@jaxxstorm has done some prior art with automatic upgrades.  See:

https://github.com/jaxxstorm/pulumi-rke/blob/master/.github/dependabot.yml#L8
https://github.com/jaxxstorm/pulumi-rke/blob/master/.github/workflows/update.yml

Notes:

Be sure not to include pre-releases in the update process.
Set PULUMI_PROVIDER_MAP_ERROR=true in the the GHA environment.
Don't merge anything automatically.

Trying this on pulumi-snowflake was a fail because there's an upstream dependency that gives a 404 (the repo was likely moved), which causes Dependabot to fail.  We're going to try again on pulumi-okta.

@stack72 If these look good, let me know next steps:

https://github.com/pulumi/pulumi-okta/pull/143
https://github.com/pulumi/pulumi-okta/pull/142


implement teuthology_path similar to suite_path

In case we want to reference a local directory instead of reseting from
a clone of teuthology.
Signed-off-by: Loic Dachary<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1535/
Tests failed for this pull request.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1539/
Tests passed for this pull request.

Seems like we should have this documented somewhere...

@zmc ooops forgot to update docs/siteconfig.rst accordingly. Done and repushed !

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1546/
Tests failed for this pull request.

test this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1582/
Tests passed for this pull request.

One difference between this PR and suite_path is that the latter is implemented as a flag for teuthology-suite whereas this teuthology_path is an item in teuthology.yaml. I imagine you have reasons for this, and that's fine. What I'd like to see, though, is the default set in teuthology.config - would you mind doing that?

updated & repushed !


Angular-seed on Heroku

Hey everyone,
Just wondering if anyone has successfully deployed this on Heroku? I could really use a few pointers.

Closing as this is not actionable.


chunithm jp support?

Not possible due to security concerns. The JP WebUI only allows login with username/password, and session tokens are short lived.


Cookie initialisation

I currently use Proxy authentication. I have re-initialise it regularly to keep it working. I decided to try cookies. However, when I follow the instructions the landing page for alexa.amazon.com (and alexa.amazon.co.uk) it simply takes you to a page to download the app.
Is to get another way to get a cookie?

Hi ! I have the same problem ..... Thanks for the help !

Seems to have stopped saving the cookie for me too.  Perhaps HA has prevented writing to the location?

Ditto here.  I need to reauthorize every day now.

Perhaps HA has prevented writing to the location?

This sounds likely.  I'm using the built-in add-on of node red, inside home assistant, and my file path for the proxy method is:
/homeassistant/node-red/test3.txt
Home Assistant is also inside a VM if that makes a difference.

Does this file location work?

Does this file location work?

Well, for about 12 hours, then I have to restart Node Red.
22 Apr 16:57:55 - [error] [alexa-remote-account:Name] Error while checking Authentication: Error: getaddrinfo EAI_AGAIN pitangui.amazon.com
That seems to pop up too.
Turning "Auto Query Activities" off and on fixes it, but I need it on permanently.

I placed the authentication file in the share folder which allows writting. Thanks guys

Hello, Where / What is the share folder? Thank you

Hello, Where / What is the share folder? Thank you

I placed the authentication file in the share folder which allows writting. Thanks guys

I didn't change my path, but it seems to work pretty reliably now.  I'll give the 'share' folder a try if it starts playing up, thanks.

Hey can someone help im not able to get the cookie, also running home assistant.
i'll go to the loginpage on amazon, enter email, enter password, enter 2FA key and then nothing happens 404 and no cookie received.

using proxy method


Implement writing the hardware RTC

We need to implement actually storing the time in the RTC of the Nintendo DS. Note that the melonDS emulator doesn't play well with libnds, see melonDS issue 2038.
This might get a little messy because to write the RTC we need to involve the ARM7 processor, which means we need to create a second main() for that processor, and a FIFO, at the bare minimum.

Fixed in commit 0d16670a7d0e38fe035d2065559a30cc7c35f792.


dts: avoid resizing sram0 for nrf9160 and nrf5340

This node is needed to have the size of the RAM available.
Signed-off-by: Håkon Øye Amundsen<EMAIL_ADDRESS>

Closing as this is superseded by https://github.com/zephyrproject-rtos/zephyr/pull/24637


Corrects two minor typos.

Great paper -- I learned several things. It was fast to follow for someone knowledgeable in spectral-method data analysis.  I'll send a few other comments/questions by email.

Thanks!


eureka: supports credentials in EUREKA_SERVICE_URL

This adds support for credentials embedded in EUREKA_SERVICE_URL user info, as is typically done in spring cloud.
Fixes #3697

thanks for the look @anuraaga!


Links to archived devkits broken

I'm trying to download the 4.5.2 devkit as I need Ruby 1.8.7 on a project, but the links on the archive aren't working: https://rubyinstaller.org/downloads/archives/.

Thanks for highlighting this! I've fixed the links: https://github.com/oneclick/rubyinstaller.org-website/commit/6a1d94e13326ef9fbdc3c9f347585f75aa2f1fc3

Great, thanks for fixing so quickly!


balx execution broken in alpha4: invalid program file format

Description:
ballerina can no longer execute compiled (balx) files
Steps to reproduce:
$ ballerina build hello.bal
$ ballerina run hello.balx
ballerina: ballerina: invalid program file format
$ ls
hello.bal       hello.balx
$ ballerina run hello.bal
Hello, World!
Affected Versions:
0.970.0-alpha4
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Fixed


Exif orientation support broken with iOS 13.4.1 and Safari 13.1 Desktop

In iOS 13.4.1 (released a few days ago) as well as Safari 13.1 desktop, the EXIF orientation value is no longer being handled properly, and the compressed image that is output will not have the correct orientation.
The easiest way to reproduce this is to go to the demo site https://donaldcwl.github.io/browser-image-compression/example/basic.html using a Safari or Chrome browser on a device running iOS 13.4.1 and attempt to upload the attached image. This image has orientation EXIF value of 3 (rotated 180 degrees).
The demo page will return the image rendered upside down. However, the same image on a device not running iOS 13.4.1 will return the image rendered with the proper orientation.

I believe that this change in Webkit (used by Safari) is the source of the problem: https://bugs.webkit.org/show_bug.cgi?id=201123
The required action for this library would be to ignore the EXIF orientation value when on Safari 13.1 (or later).

Echo this issue on chrome ios and android.


Add opam 2.2

This already exists. See line 216

@kit-ty-kate Should I now separate opam 2.2 and opam master for testing against the two as per https://github.com/ocurrent/opam-repo-ci/issues/331?

sure


Fatal error when trying to open a stream

For whatever reason, eventstore sends a 401 for any internal projection (e. g. $ce-things). EventStore::ensureStatusCodeIsGood only checks for 404 and 410.
GET: http://<IP_ADDRESS>:2113/streams/$et-asdf -> 401
GET: http://<IP_ADDRESS>:2113/streams/et-asdf -> 404
In my opinion, anything that's not 2xx should be considered a failure.
The concrete error:
Catchable Fatal Error: Argument 1 passed to
  EventStore\StreamFeed\StreamFeed::__construct() must be of the type
  array, null given, called in
  [...] EventStore.php
  on line 207

I would prefer to be explicit in status code mapping, what other codes may happen?
e.g. 403

Any code may happen. At the moment, those three are the only ones I encountered. I'm all for making things explicit but in this case the RFC has defined those codes, thus making them available to the eventstore to use at any time.

@nicolopignatelli any suggestion about exception name?

we can easily map the status codes we know about, then fallback on a generic error if the status code is not 2xx

+1

I already answered this on twitter but this is not a bug.
$streams (internal streams) require admin access by default (this can be changed by changing the default ACL). See default ACLs http://docs.geteventstore.com/server/3.1.0-pre/access-control-lists/
If you do not send credentials a 401 is quite a reasonable status code to send.
Note that it will send a 401 for any $ prefixed stream without a user.

Thanks for clarifying again. After thinking more about it, it absolutely makes sense.

This should be fixed in master.


Add GOES example

@rsignell-usgs , what do you think? Note that I added some metadata. In a case like this, with a single reader in a catalog, it's not obvious whether the descriptive stuff belongs to the whole catalog, the data spec (i.e., URL) or the xarray-specific reader.
(side note: the storage options end up in the xarray kwargs, which is fine, but there is an argument that they are an inherent attribute of the data source, since using fsspec is the only way to do anything with the reference set)

Should I add the rechunked version as a second dataset in the same catalogue?

Should I add the rechunked version as a second dataset in the same catalogue?

We probably shouldn't include the GOES example as it only does 24 time steps.
(I'll add the HYCOM one though!)

Can we make a bigger GOES, then?


Run generator to update tests

Quite a few empty. I hope that's okay.

Its all there if the file is viewed, the generator is just setting the files back to being executable again.

Cool! That's great :)


Labels for tags and name override each other depending on ordering

It appears that com.dnsdock.tags and com.dnsdock.name cannot be used together as they populate the same space in the domain name.  It is further complicated by the fact that which ever one is defined last wins.  Is there a reason we don't set it up such that tags and name populate different parts of the domain name? Also, the docs are unclear about what to use when, and appear contradictory.
If you are open to changing a bit of how the names are built I can work on a PR and doc updates.
When looking at all the options this seems to make the most sense to me.  I'm wondering what you think....
<tag>.<name>.<region>.<image>.<environment>.<domain>

Also, it looks like you can specify many tags, comma separated, but it will only ever use the first one.  Was the eventual intention if there were multiple tags that it would use the tags and create an entry per tag?

Hi !
You're completely right about this issue !
I am OK with your proposal of having multiple alias with the scheme you propose.
Feel free to do a PR. If you can't tell me I'll do the modifications

Sorry for the delay here. I'm working on something and hope to put a PR together by the end of the week.  Working through learning the right way to develop dnsdock w/ rocker, etc.

PR #99 was submitted to address this issue.

Any feedback on this PR/Issue?


Fix #1856: Deletion of stack

Description
Bug: Deletion of parent note deleted the entire stack automatically.
Fix: Different note-deletion-actions get handled via confirmation dialogs.
Style: Changed confirmation dialogs according to styling (https://xd.adobe.com/view/4a92e55c-64fc-401f-a671-0c565d51d563-1fc1/)
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] The light- and dark-theme are both supported and tested
[x] The design was implemented and is responsive for all devices and screen sizes
[x] The application was tested in the most commonly used browsers (e.g. Chrome, Firefox, Safari)

(Optional) Visual Changes
https://user-images.githubusercontent.com/49522775/195931286-f559d3fb-8877-4a97-99ca-6daee1d745a7.mp4

@brandstetterm @BenedictHomuth could one of you review the backend code?

@Resaki1 Thank you for the review :)! Fixed the points as u suggested.


MacOS Support

Can DeepDaze be run on MacOS? It seems to require CUDA which was discontinued for Mac a while back. Is there a way around this?

Hi! You can try the PR version in #103. I just added CPU support. It is probably very slow and will take up quite some RAM, but it runs on my ubuntu laptop.

It might be possible to run on an older mac / mbpro that has a discrete Nvidia GPU on it.  However, you'd be stuck using a very old compute capability if it worked at all.  If it did work, then I'm sure it would be better than CPU.


Translation of "Localizing docs in Sphinx" documentation

Hello Natalia,
That excellent document you have written and which is referenced in many discussions here on GH would surely deserve more exposure and, of course, to be translated !
Would you mind sharing the sources to allow the community to give back by making your content accessible to the rest of the world ?
Thank you for the work and for sharing it openly
TY
J

Le mar. 22 nov. 2022 à 18:34, Natalia @.***> a écrit :

Oh, discussions, really?

Yes indeed ! I found it searching for sphinx-intl on GH to see how others
are implementing it and found your guide referenced several times :)
Actually, that search was triggered by discussion on the matter for the
Write The Docs project https://github.com/writethedocs/www/issues/1155
If you want to translate the guide, I'll open the source, no problem.

Awesome ! I would gladly translate it to French for sure :)
Plus it would be nice to have the guide itself implementing its own
recipe... Eat your own dog food style ;)

Just give me some time to review it and maybe make some recommendations.

Of course, take it easy, I have more than enough in my plate already 😅

Here you are: https://github.com/authoress/sphinx-intl-guide

Le jeu. 24 nov. 2022 à 11:57, Natalia @.***> a écrit :

Here you are: https://github.com/authoress/sphinx-intl-guide

Awesomely awesome 😍
Didn't expect it that early 💥🙏
So, let's push it up in the French translation to-do, just after
sphinx-intl !
Thank you very much
TY
J
PS: If you care to do the same with the miniconda guide... :)

You are welocome ))
Miniconda is a little tricky to translate since the source is written in Russian and then translated into English.

Le jeu. 24 nov. 2022 à 14:09, Natalia @.***> a écrit :

Miniconda is a little tricky to translate since the source is written in
Russian and then translated into English.

I have to confess sub zero skills in Russian! 😅
But, even if unorthodox (pun intended 😉) I could do it from the English
version. Transifex editor even gives you the choice of source language for
translation... And this will stay between you and me 🤫

Well, let's get back to Miniconda when we're done with sphinx-intl, okay?
And I'm closing this issue.

Le ven. 25 nov. 2022 à 09:39, Natalia @.***> a écrit :

Well, let's get back to Miniconda when we're done with sphinx-intl, okay?

Absolutely ! 👍


http2 throws ERR_HTTP2_INVALID_STREAM with minor amount of concurrency

Moving from https://github.com/nodejs/node/issues/22135; as per the comments, it was confirmed to be koa-specific.

Version: v10.8.0
Platform: Linux 86f664ae731c 4.9.93-linuxkit-aufs #1 SMP Wed Jun 6 16:55:56 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux (ubuntu:18.04 Docker image)
Subsystem: http2

After a small handful of concurrent requests, http2 fails with the following error:
internal/http2/core.js:1791
      throw new ERR_HTTP2_INVALID_STREAM();
      ^

Error [ERR_HTTP2_INVALID_STREAM]: The stream has been destroyed
    at ServerHttp2Stream.sendTrailers (internal/http2/core.js:1791:13)
    at ServerHttp2Stream.onStreamTrailersReady (internal/http2/compat.js:377:8)
    at ServerHttp2Stream.emit (events.js:182:13)
    at Http2Stream.onStreamTrailers [as ontrailers] (internal/http2/core.js:318:15)
    at ServerHttp2Stream.submitRstStream (internal/http2/core.js:328:19)
    at ServerHttp2Stream.finishCloseStream (internal/http2/core.js:1533:3)
    at closeStream (internal/http2/core.js:1517:7)
    at ServerHttp2Stream.close (internal/http2/core.js:1846:5)
    at state.streams.forEach (internal/http2/core.js:2691:46)
    at Map.forEach (<anonymous>)

Test case:
cat > server.js << EOM
	const app = new (require('koa'))();
	const crypto = require('crypto');
	const fs = require('fs');
	const http2 = require('http2');

	app.use(async ctx => {
		await new Promise(resolve => setTimeout(resolve, 1000));

		ctx.body = 'balls';
		ctx.status = 200;
	});

	http2.createSecureServer(
		{
			allowHTTP1: true,
			cert: fs.readFileSync('cert.pem'),
			key: fs.readFileSync('key.pem'),
			dhparam: fs.readFileSync('dhparams.pem'),
			secureOptions:
				crypto.constants.SSL_OP_NO_SSLv3 |
				crypto.constants.SSL_OP_NO_TLSv1
		},
		app.callback()
	).listen(
		31337
	);
EOM

killall node
node server.js &
echo -n > count
while ps ux | grep server.js | grep -v grep 2> /dev/null ; do
	echo -e '1\n2\n3\n4\n5\n6\n7\n8\n9\n10'
done | xargs -P10 bash -c '
	echo >> count
	curl -sk https://localhost:31337 > /dev/null
'

When the error occurs, you can manually kill the command with ctrl+C and wc -l count to view how many requests were initiated before the failure. For me it's been anywhere between 10 and 70.
This doesn't seem to be reproducible when using either https or spdy instead of http2.

One thing that may be different from the Node.js issue is that your non Koa test did not use the http/2 compat mode while Koa will use the compat mode. It just makes me wonder if the issue is in Node.hs core in the compat mode code perhaps.

Hm, do you know how I would test that? All I see about a compatibility API here is that there's a non-TLS http2.createServer method, which doesn't seem relevant since my koa example already uses http2.createSecureServer and which curl doesn't seem to like (curl: (8) Weird server reply).

Got it, thanks! I pasted more or less the same thing in the other thread for reference, but I wasn't able to reproduce the issue using the compatibility API without koa. Switching back to the original test with koa still reliably produces the error for me.

Darn, it was worth a shot. Perhaps a member with a similar platform to yours will be able to take a look 👍

It might be reproducible in Docker on Windows (I'm using Docker on OS X right now with the ubuntu:18.04 image).

Yea, I noticed you had Docker listed. I don't have it installed and about to head to sleep for the night. If no one else stops by I'll take another stab with Docker.

@buu700 did you manage to solve this issue?
I just see this in a simple route (which does an external axios request and return the result).
Koa return the request fine with body set, but then http2 secure server crashes with:
Error [ERR_HTTP2_INVALID_STREAM]: The stream has been destroyed
    at ServerHttp2Stream.sendTrailers (internal/http2/core.js:1786:13)
    at ServerHttp2Stream.onStreamTrailersReady (internal/http2/compat.js:382:8)
    at ServerHttp2Stream.emit (events.js:182:13)
    at ServerHttp2Stream.EventEmitter.emit (domain.js:442:20)
    at Http2Stream.onStreamTrailers [as ontrailers] (internal/http2/core.js:325:15)
    at ServerHttp2Stream.submitRstStream (internal/http2/core.js:335:19)
    at ServerHttp2Stream.finishCloseStream (internal/http2/core.js:1523:3)
    at closeStream (internal/http2/core.js:1507:7)
    at ServerHttp2Stream.close (internal/http2/core.js:1840:5)
    at state.streams.forEach (internal/http2/core.js:2687:46)

Nope, as far as I'm aware there's no way around this. I'm just temporarily using the older spdy package instead of the native http2 module for now.

Is this a Koa problem or a problem of the http2 module? Is express affected?

It seems to be Koa-specific based on the issue linked at the top.

I was trying to do the same but express does not seem to support http2 if not with hacks.
const appe = express();

appe.get('/', async (req, res) => {
    try {
        let { data: result } = await axios.get('xxx');
        res.send(1);
    }
    catch (error) {
        res.send(1);
    }
});
http2.createSecureServer({ cert, key }, appe).listen(8444, error => error ? console.error(error) : console.info(`express+http2s serving on port ${8444}`));

Throws:
TypeError: Cannot read property 'readable' of undefined
    at IncomingMessage._read (_http_incoming.js:95:19)
    at IncomingMessage.Readable.read (_stream_readable.js:452:10)
    at resume_ (_stream_readable.js:899:12)
    at process._tickCallback (internal/process/next_tick.js:63:19)

Not sure though this is the proper way to use express + http2

FYI: adding a 50ms delay await sleep(50) after the axios call prevent the ERR_HTTP2_INVALID_STREAM error and process SIGINT.
Is anyone actively investigating on this...?

@damianobarbati what node version are you using and how do you launch your repro code? Naively running it like this does not work:
/usr/local/Cellar/node/10.11.0/bin/node server.js
/Users/andre/work/koa-http2/server.js:1
(function (exports, require, module, __filename, __dirname) { import pkg from './package.json';
                                                                     ^^^

SyntaxError: Unexpected identifier
    at new Script (vm.js:79:7)
    at createScript (vm.js:251:10)
    at Object.runInThisContext (vm.js:303:10)
    at Module._compile (internal/modules/cjs/loader.js:657:28)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
    at Module.load (internal/modules/cjs/loader.js:599:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
    at Function.Module._load (internal/modules/cjs/loader.js:530:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:742:12)
    at startup (internal/bootstrap/node.js:279:19)

@edevil NODE_ENV=development HTTPS=8443 node -r esm index.js

FWIW I also managed to reproduce this issue after sending a bunch of requests:
$ /usr/local/Cellar/node/10.11.0/bin/node -r esm server.js
koa+http2s serving on port 8443
http2s serving on port 8444
Error [ERR_HTTP2_INVALID_STREAM]: The stream has been destroyed
    at ServerHttp2Stream.sendTrailers (internal/http2/core.js:1786:13)
    at ServerHttp2Stream.onStreamTrailersReady (internal/http2/compat.js:382:8)
    at ServerHttp2Stream.emit (events.js:182:13)
    at ServerHttp2Stream.EventEmitter.emit (domain.js:442:20)
    at Http2Stream.onStreamTrailers [as ontrailers] (internal/http2/core.js:325:15)
    at ServerHttp2Stream.submitRstStream (internal/http2/core.js:335:19)
    at ServerHttp2Stream.finishCloseStream (internal/http2/core.js:1523:3)
    at closeStream (internal/http2/core.js:1507:7)
    at ServerHttp2Stream.close (internal/http2/core.js:1840:5)
    at state.streams.forEach (internal/http2/core.js:2687:46)

Using the 8444 port (no Koa) I was not able to crash it.

This issue seems like a race-condition. Like @damianobarbati mentions, the await sleep(50) call prevents the issue from happening (at least for the number of requests I tried). sleep(0) also seems to delay the problem for a few more requests. I also managed to reproduce the issue using a simple app.use(async ctx => {}) handler, albeit after a lot more requests.

I've fixed the issue in Node, and the PR has been merged. https://github.com/nodejs/node/pull/23146
If anyone wants to try they can build the latest master or wait for the next release.

Did anyone have a chance to verify the fix?

I am going to close this issue as the PR is merged and released and nobody has raised concerns/question about this since late 2018.


Frontend Continues to Stream Content Character by Character After Model Output Completion in v0.5.0, Especially Noticeable with Fast-Response Models

Bug Report
Installation Method
I installed Open WebUI using docker-compose.
Environment

Open WebUI Version: v0.5.0
Operating System: Windows 10
Browser: Edge 131.0.2903.112 (Official Build) (64-bit)

Expected Behavior:
After the large model completes its output, the frontend should display the full content immediately, rather than slowly streaming it character by character.
Actual Behavior:
Even after the large model has completed its output, the frontend continues to stream the content character by character at a slow pace. This issue is particularly noticeable with fast-response models like o1-mini, and the difference before and after upgrading to v0.5.0 is significant.
Description
Bug Summary:
After upgrading to v0.5.0, the frontend continues to stream content character by character even after the large model has completed its output. This issue is especially pronounced with fast-response models like o1-mini.
Reproduction Details
Steps to Reproduce:

Start Open WebUI using docker-compose.
Open the browser and access Open WebUI.
Use a fast-response model like o1-mini for a conversation.
Observe whether the frontend continues to stream content character by character after the model has completed its output.

Additional Information
This issue only appeared after upgrading to Open WebUI v0.5.0. In previous versions, the frontend displayed content at a speed consistent with the model's output. However, after the upgrade, there is a noticeable delay in the frontend's display, especially with fast-response models.

This is likely due to the fact that responses get saved every new chunk to enable async chat support. Optimization PRs are welcome, however I really don’t see a way to optimize.


Ui for glyph editor

Hi Karsten,
I've tried to add an editor box with an input text for each glyph like this :

The state code is based on this hdom example but it has some error.
If there is only 1 glyphs editor box... it works: the re-render when input text changes works. But with more glyphs... re-render does not works anymore.
You can see this behaviour by toggle comment here: https://github.com/thi-ng/font/pull/1/files#diff-fbea1a62407700a59a4adae2175239a5R24
I think it is related with innerHtmlWrapperSvg that I took from this hdom example

Hey @nkint - just checked out your fork and tried to build it, but the data import is missing (forgot to check in the file?). Am on the road this weekend, but will try again once the file is there - can't tell what's wrong without building it... sorry & thank you!!!

Hi @postspectacular yes sorry the file was not commited 🤦♂
Just added.

Will check it out and report back asap!


Bug in Inviting Multiple Users to a Doc

Small bug when inviting multiple users to a document. The instructions say to enter one email address per line, but it's possible to enter many comma-separated emails in one line, which is then added to one invite.

Would be nice to improve this, by either

Enforcing one email per row. We already have UI for preventing comma separated emails in the manage users dialog (see below). Could change the text to reinforce instruction that emails should be entered one email per line.
Make it possible to submit multiple email addresses that are comma separated. This is better but a bigger change, not sure if it's worth it.

@vviers This was something the ANCT team helped implement. 💖 Flagging in case there's any interest in addressing this bug.


Add support for rendering Asciidoc files served up by gemini servers

I've been thinking for a long while now wether to suggest this or not. I understand the argument for gemtext and the protocol being mostly frozen. But markdown is such a barbaric language. You can barely format in it. Asciidoc is incredibly simple to write and allows for some control of formatting, tables, graphing, etc.
I do not want to see Gemini devolve into the god forsaken web, but it's goal to be simple is way beyond simple, its bondage for content and writing. IMO it's greatly hampering Gemini's adoption. If lagrange would support the rendering of asciidoc I think it would make Gemini more widely adopted and the content, the ACTUAL content, not ads, or malware, but the content people write, far more compelling than being stuck with markdown. I'm definitely interested in discussing this. What does the community think?

I think Asciidoc is a complex enough system so that it shouldn't be re-implemented as part of Lagrange, but should rather be integrated with as an external tool, when available. Such integration actually shouldn't be too difficult to pull off, although Lagrange's document layouts don't support all features of Asciidoc (e.g., tables) and these would have to be emulated somehow with preformatted blocks or images.
But of course, that would make the ability to view Asciidoc an optional feature, and content authors wouldn't be able to rely on readers being able to view it as intended.
This generally comes down to the same issue as with any non-Gemtext format. If a document benefits from additional/fancy formatting, it should be served as both Gemtext and in whatever full-fledged Asciidoc/Markdown/PDF is most fitting, so it can be viewed in a simple form by any Gemini client and in its true format using appropriately specialized viewer apps.


Command binding does not work with a custom dependency property of ICommand type.

When I developed a custom user control that exposes a custom dependency property of ICommand type the exception is shown saying "there is no ID for given dependency property".
[DependencyProperty( Properties.OpenNotificationsCommand)]
public ICommand OpenNotificationsCommand { get; set; }

Maybe it is caused by the aspect  [WpfControl( typeof( Properties ) )]  which does not allow to expose commands ID's (Property of a command type is not accepted?).
Possible workaround solution is to use a control template and then template bindings. In that case exception is not thrown.

I don't fully understand the error. Could you paste the exact error message you get (the stack, e.g. as such)? Is it one I throw from the DependencyPropertyFactory, if so, which one exactly where? Or is it an error from WPF? When does it get thrown? Upon initialization? When trying to trigger the command?

Control code behind (non-framwork way is working):
[WpfControl( typeof( Properties ) )]
public partial class NotificationButton
{
	public enum Properties
	{
		SomeCommand
	}

	[DependencyProperty( Properties.SomeCommand )]
	public ICommand SomeCommand { get; set; }

	public static readonly DependencyProperty OpenNotificationsCommandProperty =	DependencyProperty.Register(
	"OpenNotificationsCommand", typeof( ICommand ),
	typeof( NotificationButton ) );
}

Usage in parent control (ActivityBar):
<common:NotificationButton
		OpenNotificationsCommand="{wtc:CommandBinding {x:Static barBinding:Commands.ShowNotifications}}"
		SomeCommand="{wtc:CommandBinding {x:Static barBinding:Commands.ShowNotifications}}"/>

XAML usage
<Button
	Cursor="Hand"
	Style="{Binding ButtonStyle}"
	Command="{Binding SomeCommand}">
	<Image Source="{Binding ButtonImage}" />
</Button>

Or 2nd option:

<Button
	Cursor="Hand"
	Style="{Binding ButtonStyle}"
	Command="{commandFactory:CommandBinding {x:Static common:NotificationButton.Properties.SomeCommand}}">
	<Image Source="{Binding ButtonImage}" />
</Button>

And finally inner exception thrown during compile for 1st option:
"No CommandFactory for ID type "Laevo.ViewModel.ActivityBar.Binding.Commands" in type "Laevo.View.Common.NotificationButton" found."
stack trace:
at System.Windows.Markup.WpfXamlLoader.Load(XamlReader xamlReader, IXamlObjectWriterFactory writerFactory, Boolean skipJournaledProperties, Object rootObject, XamlObjectWriterSettings settings, Uri baseUri)
and for 2nd option:
"'Laevo.View.Common.NotificationButton.Properties.SomeCommand' StaticExtension value cannot be resolved to an enumeration, static field, or static property."
stack trace:
System.Windows.Markup.XamlParseException occurred
HResult=-2146233087
Message=Provide value on 'MS.Internal.Markup.StaticExtension' threw an exception.
Source=PresentationFramework
LineNumber=0
LinePosition=0
StackTrace:
at System.Windows.Markup.WpfXamlLoader.Load(XamlReader xamlReader, IXamlObjectWriterFactory writerFactory, Boolean skipJournaledProperties, Object rootObject, XamlObjectWriterSettings settings, Uri baseUri)
InnerException: System.ArgumentException
HResult=-2147024809
Message='Laevo.View.Common.NotificationButton.Properties.SomeCommand' StaticExtension value cannot be resolved to an enumeration, static field, or static property.
Source=System.Xaml
StackTrace:
at System.Windows.Markup.StaticExtension.ProvideValue(IServiceProvider serviceProvider)
at MS.Internal.Xaml.Runtime.ClrObjectRuntime.CallProvideValue(MarkupExtension me, IServiceProvider serviceProvider)
InnerException:
Took 1 hour to describe all that and my visual studio almost died from the amount of exception, hope it helps. :D


fix go mod. iavl version.

cosmos-sdk 0.45.6
tendermint v0.34.19
ibc-go v3.1.1
wasmd - v0.28.0
iavl 0.17.3

(delete branch after merge)

Downgrades at iavl  :(

Downgrades at iavl :(

No downgrades, just postponing..


Example usage

Hi,
I created a Phoenix project that I am using to test Guardian in its current state. It is available at https://github.com/Gazler/guardian_test, the specific commit for the Guardian features is https://github.com/Gazler/guardian_test/commit/07e5d11b3e61cf600f709dda0252f83593baebe0
I have also added a couple of things to Guardian, I would just like to confirm that this is the way it is intended to be used, if not then if you can advise and I will pull-request open a pull request with that. https://github.com/Gazler/guardian/commit/2ceb4b8f7ed1164563698847faaf6498caa08361
Thanks.

This is not ready for the World yet. Hopefully later on I'll have more time to pour into it.


Use write instead of write_all and remove flush

Flush: does not seem to change anything
Write returns the length that has been written. write_all does not.

write_all does not return the size since it "writes all". To me, write + the assert_eq! is basically write_all. Is that not so?

The doc doesn't say it. The doc says it calls write, one by one, until the end of the buffer. It might also be less efficient to call write_all as, it seems, to call N times the write system call. write with a buffer of size N will call only once the syscall write. In any case, we should, for consistency, check that we actually wrote 32 bytes.

I agree that we could use write here. But flush is more useful with write than write_all since write may not write everything right now (it can be buffered).

We don't need to flush if we know we exactly wrote the 32 bytes (or N bytes for other calls). The next stream we send to the file will be different, and we will check also the right amount of bytes have been written correctly. Also, rust-analyzer gives me a flush implementation returning simply Ok(()).

I'm concerned that this is fragile to the buffer size and go's behaviour. Does this fix anything?

The buffer has a fixed-length of 32 bytes. Go will wait for 32 bytes, see here. Therefore, it is the exact behavior expected by Go. Also, it will writes the 32 bytes in one-shot, not one by one.

Also, I think we still want to flush, to ensure that we don't move on before the go process has read all of the data

When we wrote exactly 32 bytes, the function will continue here. Calling flush is unnecessary if we do check we sent 32 bytes. Also, as said here, flush doesn't seem to do anything.

libc::write may choose to write fewer than 32 bytes.

Right, I did not know! Good point. However, we do check if we wrote the 32 bytes.


Laravel Migration Error: Syntax error or access violation: 1071 Specified key was too long; max key length is 767 bytes

Using Laravel 5.4 i have face this issue , and  this cmd :  php artisan quickadmin:install  gives the above error
now i have changed the  /app/database.php file under mysql section
'charset' => 'utf8',
'collation' => 'utf8_unicode_ci',
//'charset' => 'utf8mb4',
// 'collation' => 'utf8mb4_unicode_ci',
now cmd is working

@moon759 Laravel News wrote about it in Laravel 5.4: https://laravel-news.com/laravel-5-4-key-too-long-error


If a machine doesn't have usb support, an error will be thrown upon process completion

➜  ~  t2 list
ERR! WARNING: No usb controller found on this system.
INFO Searching for nearby Tessels...
^C/Users/Jon/Work/technical/t2-cli/lib/usb_connection.js:339
  usb.removeAllListeners('attach');
     ^
TypeError: Cannot read property 'removeAllListeners' of undefined
    at USB.Scanner.stop (/Users/Jon/Work/technical/t2-cli/lib/usb_connection.js:339:6)
    at TesselSeeker.stop (/Users/Jon/Work/technical/t2-cli/lib/discover.js:126:18)
    at process.<anonymous> (/Users/Jon/Work/technical/t2-cli/lib/controller.js:108:16)
    at process.g (events.js:199:16)
    at process.emit (events.js:104:17)
    at Signal.wrap.onsignal (node.js:655:46)

We should just check if usb is defined before trying to remove listeners.

I'm getting ERR! WARNING: No usb controller found on this system. and don't understand this issue - is this a warning from within t2-cli?

You need to upgrade to node 4.2.1 or newer and run npm rebuild or just reinstall t2-cli

thanks, fixed!


fastai compatibility

would it be possible to make the distill vit compatible with fastai? both the vanilla vit and efficient vit work fine.

@lwomalley Hi Logan! I know of FastAI but not too familiar with their API
What would it take to be compatible?

In the vanilla vit, the forward pass takes in an image and returns its transformed output. In the distilled vit, the distillwrapper forward pass takes an image + labels and returns the loss. Fastai gets tripped up on this because it automatically supplies the model with labels and is expecting the forward pass to return a transformed output. If there is a way to decouple the forward pass and the loss calculation I suspect it may work.

@lwomalley yup, but the problem is the distillation comes with an auxiliary loss that gets returned.  Will FastAI know to add this to the main loss it calculates?

Dont know if can help, but are "vit" and "distilled vit" layers? what is the auxiliary loss? or how it will be in a "normal loop" used?
This is where the loss is calculated, but AFAIK we have the options of callbacks and also we can "replace" this method for the learner itself with a new implementation/different code.
https://github.com/fastai/fastai/blob/master/fastai/learner.py#L172-L173

@tyoc213 yeah, it won't work, because that line needs to also add the distillation loss https://github.com/lucidrains/vit-pytorch/blob/main/vit_pytorch/distill.py#L120 i could return the logits on the first element of the tuple, and the auxiliary loss on the second, but FastAI will still need to sum the auxiliary loss into the calculated one

mmmm, so the result of https://github.com/fastai/fastai/blob/master/fastai/learner.py#L169 in fastai returns a tuple IIRC and we can apply watever we want doing a transform at https://github.com/fastai/fastai/blob/master/fastai/learner.py#L174 which is "call the transforms that answer to this event" and all the transforms have access to learner and other things see https://docs.fast.ai/callback.core.html#Callback

after_loss called after the loss has been computed, but before the backward pass. It can be used to add any penalty to the loss (AR or TAR in RNN training for instance).

@tyoc213 I see, so I'd have to store the auxiliary loss on the instance somewhere? and then in the callback it would be fetched and added to the main loss?

@tyoc213 do you have any code examples of how you are using ViT with FastAI?

Not, but maybe this tiny bit can help https://youtu.be/4w3sEgqDvSo?t=1148 ?


Use major version instead of exact version

Only set the major version to "force" "copy-paste" users to always use the latest version of the action by default.
This should avoid the need to delete older releases like what happened in #20

This works but only in the context of the current situation.  But thanks, I'll merge this.

@thadguidry so this doesnt seem to work, neither with @v0.2, nor @v0 (or @v2).
Seems like you have to enable something as described here

WHAT specifically is not working for you.  Can you give me some reproducible steps you are doing?  or show me your workflow?

My method of forcing users (breaking old workflows that have a semantic version guarantee - whereby I deleted prior version tags) is unpopular by many users in this circumstance, but it's the only way to truly force users to move forward.


(WIP) update dependency with pytorch 1.7.0

This PR aims at changing the project requirements.
We bump the version of PyTorch from 1.5.0 to 1.7.0

Bug are too big for now and updating this PR Is going to get complicated (Gradient explosion in test). To try later.

Bug are too big for now and updating this PR Is going to get complicated (Gradient explosion in test). To try later.


Update nan 2.1.0

Refer: #4

@devongovett I am using font-manager in Mancy.  Thanks

Thanks. Merged in 0a5a124c2f187f24a9505b13e7d1fd04967a2986. Had to do a little more work to fix some build issues in older versions of node. Travis build now passing back to 0.8! https://travis-ci.org/devongovett/font-manager/builds/99230457


[feat] Backend pixel validation

Add checks to all the pixel routes to validation position and color ( make sure they are in the correct range ) before using them.
References

pixel routes
other relevant route

I'd like to work on this

Awesome, I'll assign you @aweolumidedavid . Welcome !

Hey @aweolumidedavid , are you actively working on this? I'd like to free it for someone to take if not. I'll be adding more issues in the future you can take if you haven't had time for this.


Delete Account is not implemented

There is no possibility to delete the account.

There are several complications deleting an user account

What is happening to the jobs created by the user?
What is happening to the applications the user has committed?
What is happening with the users' organization and the assigned employees?

If a user wants to delete his account, you can send him to a confirmation page, where everything is listed, what will be deleted:

Jobs he created
Applications he submitted
Organizations and Employees
Other things he uploaded

As an enhancement the user could for his organizations/jobs choose another person/account who will manage them

That's a nice idea...

We can inactivate accounts now.
Deleting Users well be don in 0.26

@fedys If you don't mind, please implement this feature. In the first step, just the listing of related objects that will be removed should be shown and a confirm and cancel button.

@TiSiE No problem, I will implemented. I am busy this week. I will start on it next week.

@fedys: Nevermind, @cbleek is always keen to have new features available :smirk:
Just do the tests and reopen the pr or create a new one (I don't know what's easier with github...)


MultiSPW - Should all channels in a band have the same SPW

The freq_mapping divide the channels in multiple bands according to the channels-per-integration option but no division according to the SPW is done. When cpi is fixed to -1, there is only one band that results from the frequency mapping in which all the SPW's are. It results in an error in the grid worker in the image_data_products() call in which all frequency channels should be the same accross the band (gridder.py l.380).
assert (ds.FREQ.values == freq).all()

A simple fix a tried is to use the ddids to create the band along the SPW :
            for ti, (tlow, tcounts) in titr:

                It = slice(tlow, tlow + tcounts)
                ridx = row_mapping[ms][idt]['start_indices'][It]
                rcnts = row_mapping[ms][idt]['counts'][It]
                # select all rows for output dataset
                Irow = slice(ridx[0], ridx[-1] + rcnts[-1])

                fitr = enumerate(zip(freq_mapping[ms][idt]['start_indices'],
                                    freq_mapping[ms][idt]['counts']))
                nfi = len(freq_mapping[ms][idt]['start_indices'])
                for fi, (flow, fcounts) in fitr:
                    Inu = slice(flow, flow + fcounts)

                    subds = ds[{'row': Irow, 'chan': Inu}]
                    subds = subds.chunk({'row':-1, 'chan': -1})
                    if opts.gain_table is not None:
                        # Only DI gains currently supported
                        subgds = gds[ids][{'gain_time': It, 'gain_freq': Inu}]
                        subgds = subgds.chunk({'gain_time': -1, 'gain_freq': -1})
                        jones = subgds.gains.data
                    else:
                        jones = None

                    datasets.append([subds,
                                    jones,
                                    freqs[ms][idt][Inu],
                                    chan_widths[ms][idt][Inu],
                                    utimes[ms][idt][It],
                                    ridx, rcnts,
                                    radecs[ms][idt], ddid*nfi+fi,
                                     ti, ims, ms])

    futures = []
    associated_workers = {}
    idle_workers = set(client.scheduler_info()['workers'].keys())
    n_launched = 0
    nds = len(datasets)
    while idle_workers and len(datasets):   # Seed each worker with a task.
        # pop so len(datasets) -> 0
        (subds, jones, freqsi, chan_widthi, utimesi, ridx, rcnts,
         radeci, bi, ti, ims, ms) = datasets.pop(0)

        worker = idle_workers.pop()
        future = client.submit(single_stokes,
                        dc1=dc1,
                        dc2=dc2,
                        operator=operator,
                        ds=subds,
                        jones=jones,
                        opts=opts,
                        freq=freqsi,
                        chan_width=chan_widthi,
                        utime=utimesi,
                        tbin_idx=ridx,
                        tbin_counts=rcnts,
                        radec=radeci,
                        antpos=antpos[ms],
                        poltype=poltype[ms],
                        xds_store=xds_store.url,
                        bandid=bi,
                        timeid=ti,
                        msid=ims,
                        wid=worker,
                        pure=False,
                        workers=worker,
                        key='image-'+uuid4().hex)

I seem to have broken the frequency<->band mapping in the multiple SPW case. I will attempt a fix shortly

All right, I'll keep an eye on when the #117 and #121  passes and test again the VLA data.


Get access to the state outside components

Hi guys !
Is there a way to get access to the state outside components (without connect) ? I mean something like :
import store from 'something';

const state = store.getState();

In fact, I have a util function named request which is basically a decorator for fetch. In this function I  need the state to check if the user is connected and retrieve his access_token.
Thanks for the help !

I am only guessing here, but you should be able to replace something with a reference to your createStore.js file.
import store from 'store/createStore';
const state = store.getState();

@marshallford that should work!
@eeynard what I do in this situation is have middleware that will intercept specific types of actions that are decorated to say "this is an API" request and pulls the access token from the store automatically.
I modelled it after Dan's great example in the redux code base "redux real-world". Might be a more idiomatic way of doing all your API requests in general.

@marshallford I already tried this option and unfortunately it did not work.
@JMacLulich You are right it seems to be a much better solution. I will write my own middleware inspired by yours :smiley:
Thanks for your help guys, appreciate it  !


fixed data_dir parameter to not only use default value.

I tried altering the path to load the data from my desktop but it didn't work, turned out that the parameter was being ignored and was just falling back to the default DATA_DIR with a value of ~/.eegnb/data

Nice, thanks!


Add config option for complex matching against s3 buckets

Addresses https://github.com/gruntwork-io/cloud-nuke/issues/108
Purpose
Allow terminating AWS resources via more complex matching rules via a config file.
Scope
Because s3 buckets are quick to create, this PR is scoped to support nuking s3 buckets based on matching by name. (Matching by region is also possible but outside the scope.)
CLI changes
I propose we allow something like this:
cloud-nuke aws --config path/to/file.yaml

Config file format
Yaml.
We need the structure of the config to reflect the way AWS describes resources. S3 buckets have names and regions, so for the scope of this project:
s3:
  include_names_regex:
    - ^alb-alb-.*-access-logs$
    - ...

Looking ahead:
Eventually, we might support something like:
regions:
  include:
    - us-east-1
    - us-east-2
    - us-west-1

ec2_instance:
  include_tags_regex:
    - ^test_.*

iam_role:
  include_names_regex:
    - ^test_.*
    - ^cloud-nuke-TestListEksClusters-.*

s3:
  include_names_regex:
    - ^alb-alb-.*-access-logs$

The above would terminate everything within the specified regions. If you want to specify deleting only s3 buckets only in certain regions, you could say:
s3:
  include_names_regex:
    - ^alb-alb-.*-access-logs$
  exclude_regions:
    - us-west-1

I'm not sure if the aws specifier is important to include, so could it be this?

IIRC, urfave/cli (the CLI framework we are using) is not ergonomic enough in the argument parsing such that if you do the latter option, you have to always provide it there before the subcommand. This may or may not be intuitive. E.g.:
cloud-nuke --config /path/to/config aws --region us-east-1

Given that, I think it would probably be better to repeat the args for each subcommand for CLI consistency.

@yorinasub17 Updated the description to reflect that. Thanks for explaining why that's important.

I propose we use a config format that is universal, consistent with whatever we might already be using elsewhere, and something users use a lot. Does yaml fit the bill?

Yaml makes sense to me!

I propose we use the _filter to be clear, but I'm open to thoughts here.

I prefer being more explicit, e.g. include_names_regex. This makes it clear that:

Each entry is a regex.
The regex is for inclusion, and not exclusion. This also allows us to expand to include exclude_names_regex in the future.
The regex matches on names.

NIT: tag_filter should be a map instead of a list of maps -
ec2_instance:
  tag_filter:
    Name: ^test_.*

I prefer being more explicit, e.g. include_names_regex. This makes it clear that:
* Each entry is a regex.

* The regex is for inclusion, and not exclusion. This also allows us to expand to include `exclude_names_regex` in the future.

* The regex matches on names.

So, the yaml format could be something like this:
[resource_type]:
  [include|exclude]_[field_plural]_[format]:
    - [ list of matchers ]

Maybe we assume plaintext matching if they only specify include_names? Or we require include_names_plaintext?
About the ec2_instance one, I don't think the Name: ^test makes sense, how about this?
ec2_instance:
  include_tags_regex:
    - ^test_.*

About the ec2_instance one, I don't think the Name: ^test makes sense, how about this?

Well, tags are key-value pairs so what you want to do is filter by tags that match keys and values. So you either need to be specific about matching the tag keys (which I think is what you were going for in the alternative proposal), or matching values of specific tags (which is what I was going for in my suggestion). Does that make sense?

Maybe we assume plaintext matching if they only specify include_names? Or we require include_names_plaintext?

I think assuming include_names as plain text, exact matches is a reasonable assumption to make. 👍

Okay, I get it now about the key-value pair matching, for tags. Updating the description now.

Supporting regions as a high level "resource type" is a little bit inconsistent with the rest of the config format. I think we could hash that out later? Maybe regions are just special.
I think everything else feels consistent enough. Let me know if we want to move forward with implementing something for nuking s3 buckets by name.

+1 to focusing on just the s3 names filter and punting on supporting regions!

@yorinasub17 Okay! Great. Updated the description with what we've discussed.

The updated description LGTM!

@yorinasub17 Thanks so much for your comments! I don't have time to take care of those concerns at the moment. But they will be helpful for the future.
I've updated the PR description with wrap-up comments (collapsed in a Summary). :tada:
Thanks, all!

[ ] Add an integration test with the getAllS3Buckets function to make sure the filtering logic works

@brikis98 @yorinasub17 Could use another review, possibly someone to pull and run this, too. I noticed we can't delete s3 buckets that have no region, such as the two buckets that are currently stuck in our nuclear wasteland account. Even the AWS UI doesn't let me delete them.

@brikis98 @yorinasub17 Could use another review, possibly someone to pull and run this, too. I noticed we can't delete s3 buckets that have no region, such as the two buckets that are currently stuck in our nuclear wasteland account. Even the AWS UI doesn't let me delete them.

Happy to guinea pig this at some point. I haven't used cloud-nuke yet so it would be good learning for me. LMK if there are specific configs / accounts you'd like flexed!

@zackproser I'm using the nuclear wasteland account, and if you look at s3_test.go, search for Filter you'll find a test that runs through 3 different types of mock config files. The config_test.go file also references a bunch of mock config files in the config/mocks dir. Look through there to see what kinds of things I'm testing.
I'd love if we found a config format that actually breaks functionality.

Looks like the CI build failed on TestFilterS3Bucket_Config, with error:
--- FAIL: TestFilterS3Bucket_Config (17.52s)
    --- FAIL: TestFilterS3Bucket_Config/Exclude (10.54s)
        s3_test.go:560:
            	Error Trace:	s3_test.go:560
            	Error:      	Not equal:
            	            	expected: 6
            	            	actual  : 7
            	Test:       	TestFilterS3Bucket_Config/Exclude

This runs in phxdevops, so perhaps there's a bucket in there that is matching?

When I run this locally against phxdevops there are 191 buckets matching config/mocks/s3_exclude_names.yaml so that can't be what's happening, right?

Just fyi, we are running against the sandbox account, I believe. Unless these labels are wrong..
Using environment variables from project settings and/or contexts:
  ...
  SANDBOX_AWS_ACCESS_KEY_ID=**REDACTED**
  SANDBOX_AWS_SECRET_ACCESS_KEY=**REDACTED**

This last test run passed. Rerunning.

Running these tests locally against both phxdevops and sandbox, they all keep passing. There's probably some "random region" choice that fails, or something else I missed.
e.g.:
cd cloud-nuke/aws
houston exec sandbox -- go test -v -run TestFilterS3
houston exec phx -- go test -v -run TestFilterS3

@zackproser I'm using the nuclear wasteland account, and if you look at s3_test.go, search for Filter you'll find a test that runs through 3 different types of mock config files. The config_test.go file also references a bunch of mock config files in the config/mocks dir. Look through there to see what kinds of things I'm testing.
I'd love if we found a config file that actually breaks functionality.
I have updated the description to describe how to pull and run and test this out!

Thanks for updating with testing instructions! Just a quick update to say the happy paths you specified are working for me and I had zero issues running this as described. I'll try to flex it a bit more as you suggest.


adding in the missing data to clavel2023 disscused in issue #1110

Pull Request
This PR is for a

[ ] New Publication(s)
[x] Correction

For the following list(s):

[ ] ancientmetagenome-environmental (README)
[ ] ancientmetagenome-hostassociated (README)
[x] ancientsinglegenome-hostassociated (README)

New Publication
Publication Information
This pull request is to add samples from the following publication(s):
This is to close
Checklist

[ ] Publication is published (preprints currently not accepted)?
[ ] Checked the publication is not already in the database?
[ ] Checked samples in this publication are not previously published data (newly re-sequenced metagenomes are OK!)?
[ ] Samples are shotgun metagenomes (hostassociated-singlegenome may also contain whole-genome enriched data)?
[ ] Checked the list follows conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)?
[ ] Changelog is updated to include the publication under 'Added'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from @SPAAM-community/ancientmetagenomedir?
[ ] (If applicable) Made a separate PR to add new categories to controlled lists (stored under /assets/enums, e.g. material or archive)? If so the corresponding PR is here:

@theHatIsBack I think you need to check the TSV, there appear to be multiple empty columns at the end!

Whoops, I must have accidentally added them when copying the rows into the tsv in excel. That should be fixed now.

Awesome thanks @theHatIsBack ! Next you need to add to the libraries TSV and update the changelog :)

Thanks @theHatIsBack !


Bump enr to v0.10.1

Bump enr dep to v0.10.1, which introduces breaking changes.

Yeah. Thought it didn't have breaking changes. Yanked 10.1 and moved it to v11


Makes the parser stricter

Support ISO_DATE_TIME codes  If we get an offset, convert to local UTC
No longer expect additionalProperties in the event

The applications that were sending additional fields have fixed their messages, we can now remove this data.
As agreed we are accepting timezones - internally we convert them to UTC when parsing.

⚠ We must make sure the apps that were sending invalid actions are deployed on prod before this is deployed on prod.

warning We must make sure the apps that were sending invalid actions are redeployed on prod with their fix before this is deployed on prod.

Already deployed on prod


Create a Visio Diagram option (-Diagram)

Does the -Diagram option create a Visio native file?
The documentation (and PS scripts) eludes to it being in the Visio format.
When I've used this option, it is always a XML output which opens in Draw.io
Can someone clarify please?

@Claudio-Merola - Thank you for clarifying this option.
Much appreciated :)


Send notification when transformation is waiting for Q&A

Solves https://issues.redhat.com/browse/FLPATH-835
Send backstage notification with Move2Kube URL (for now internal URL) when waiting for Q&A

/lgtm


Added new utxo endpoints, including the original transaction

This is still work in progress, needs bux to be updated first

Squashed the branch with the fixes from the comments you made