Larger tiles don't allow smaller tiles to stick to each other

Hi, great application!! I'm having an issue where the small tiles cant stick to each other when there's a larger tile on the layout.

Thanks for reporting this issue! This is actually a duplicate of #18, you can work around this by making your first tile (upper left corner) a 1x1 tile, and pressing Save Layout. That will let you place them in the grid properly.
I am working on getting this fixed so you can put something else besides a 1x1 in the first spot.
Closing as a duplicate, please track #18 thanks!

Thanks!


TS-3967: Set stream state to close after a RST_STEAM has been sent

https://issues.apache.org/jira/browse/TS-3967
The original patches are on the JIRA ticket. I updated the patches to resolve conflicts, and made little changes to satisfy h2spec tests.

rebased to follow the changes in master


Change: Drop separate start, startMax & end, endMax

This had been started in https://github.com/ptoussai/progenetix-next/commit/6e9f0eac17b942296b50d67fbe774701e2dd06e6 but reverted (probably due to parallel work). To emphasize:

start and startMax are just expressions for a start array w/ 1 or 2 values [start, startMax]. The backend just treats them as start[0] - 1 and start[-1]; i.e. a single value of 101 becomes [100,101]; 2 values of 101 & 500 become [100,500]
single input values are used for most purposes; the exception so far is the CNV request, where a start and end interval should be given, though single values could be used due to the interpolation above (a single base interval isn't very helpful there but not directly wrong); so rather a warning message

IMO the separate use of optionally 2 values for start and end is confusing & it would be better to provide them in a single field each as 20000001-21975098 etc. (in the current example), and to split. The query string anyway sends start= 20000001&start= 21975098.
(Other changes in the linked commit: materialType => filters ...)

I have implemented the value split & adjustment. However, query generation should become less "hard coded".

@mbaudis does b450e816987cfdfc39e4e6d5d66d849e7de1ddf5 solve the remaining issue?

I added some validation in this commit: 7c07c3ce843f59d5189e4ae2b7e089c589b4d7c3
The error message might now be accurate, could you check?
@mbaudis: please close the issue if this is OK.

Seems good from reading the changes, AFAIK.


Optional Java 8

Added Matchers for Optional of Java 8

Hi, Steve.
Would it be possible to create a new release with support for new versions?

@feh-wilinando Hey. I have Hamcrest Optional available on maven central.

Looks like we're long overdue for a new version. Life is much easier without existing users :)

@fwfurtado I have a plan to get support for Java 8 and 9+ ... , not sure if that plan will work but doing some proof of concepts, please can you rebase from master, as hamcrest-core and hamcrest-library have been refactored a lot and also deprecated, so that everything is just in hamcrest.

Going to try and kick start hamcrest, so if you want to get it merged, please rebase from the branch feature/v3.x-ea.
Still trying to understand how has permissions to perform a release.
Looking to keep v2.x as java 7, with v3 bumping to java 8.


Add option to save current config to a file

Have either/and a Menu, or a shortcut (CMD+S ?), to save config to a config file.
Then need a file selection dialog box (not trivial in Iced I recall...).
Save the config async.
Warn if the config is empty (no changes made)?
Cancel should cancel the process.

Assigned you @sundaram123krishnan since you are already working on it


Updated and Improved username behavior

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

This is a pull request


Add support for websockets when proxy is enabled

Please ensure that your pull request fulfills these requirements:

[X] The pull request is being made against the master branch
[ ] Tests for the changes have been added (for bug fixes / features)

What is the purpose of this pull request? (bug fix, enhancement, new feature,...)
Fixes #142
What changes did you make?
Based on #528:

removed changes unrelated to websockets
rebased to current master.

Provide some example code that this change will affect, if applicable:
Handles HTTP upgrades used for websockets by forwarding them in proxy mode.

maybe @thornjad can have a look at this? ;-)

@thornjad


Expense - Expense report shows a blank message instead of "detached a receipt from expense"

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: no new feature
If this was caught during regression testing, add the test name, ID and link from TestRail: n/a
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Go to staging.new.expensify.com
Go to DM.
Submit two expenses to the user - one of the expenses must be submitted with a receipt.
Go to Troubleshoot > Clear cache and restart > Reset and refresh.
Go to the same DM in Step 2.
Go to transaction thread of expense with receipt.
Click on the receipt.
Click 3-dot menu > Delete receipt > Delete the receipt.
Go to expense report.

Expected Result:
"detached a receipt from expense" system message will be visible in the expense report.
Actual Result:
"detached a receipt from expense" system message is not visible in the expense report.
The message is blank in the report. It only displays the message in LHN preview.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/38435837/e9fe797f-3bee-449b-b920-8e38db423117

View all open jobs on GitHub

@grgia FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

Proposal
Please re-state the problem that we are trying to solve in this issue.
Expense - Expense report shows a blank message instead of "detached a receipt from expense" when user delete receipt.
What is the root cause of that problem?
When user remove receipt, the server will return  report action with type: MANAGERDETACHRECEIPT. and it is categorized as old dot report action:
https://github.com/Expensify/App/blob/4dc1710ea72daaa44875e24ea0f71056b33ed8ef/src/libs/ReportActionsUtils.ts#L1162
In PR: https://github.com/Expensify/App/pull/40168 getMessageOfOldDotReportAction got modified:
https://github.com/Expensify/App/blob/4dc1710ea72daaa44875e24ea0f71056b33ed8ef/src/libs/ReportActionsUtils.ts#L1199-L1203
The content of reportActions is:

And will make the getMessageOfOldDotReportAction give incorrect result.
What changes do you think we should make in order to solve the problem?
If this is deploy blocker:
We should modify the getMessageOfOldDotReportAction into:
function getMessageOfOldDotReportAction(reportAction: OnyxEntry<ReportAction>): string {
    return reportAction?.message?.map((element) => getTextFromHtml(element?.text)).join('') ?? '';
}

I'm pretty sure this is caused this PR.

@deetergp yes, the fix is to revert the getMessageOfOldDotReportAction into:
function getMessageOfOldDotReportAction(reportAction: OnyxEntry): string {
return reportAction?.message?.map((element) => getTextFromHtml(element?.text)).join('') ?? '';
}

cc @nkdengineer

checking this now.

cc @c3024 The PR is ready.

This should be fixed now. I have asked for a retest here.

@deetergp issue fixed
https://github.com/Expensify/App/assets/43996225/50b57465-88df-4356-83d6-f6d0780f6605

Since the bug is fixed I'm going to close this one out. Thanks all 👋


[item] Blink items ideas

Once baumi said that is hard to come with ideas for blink dagger upgrades, i think he was wrong. Before you read, latter on i using word "trash" to analogy of items that preferably are not an upgrade of other items and i leave dev team to discuss about it. I chosen this word because i like it, i could also use word "ponny" but I'm not trash and like "trash" more than "ponny", so no need to offence. Above that i use two types of brackets(?) "[]" and "()" first one are my comments on stuff second one are addons to suggestions [important ones].
Ability: have chance to increase distance of blink (work similar to multicasting blink in old times of LoD)
2)Healing dagger.
Recipe: blink dagger plus tranquils (can be also recipe if needed)
I think everyone will know what it does.
3)Wind Dagger. [I think Merlin Monroe would not be very happy for that item]
Recipe, Blink dagger plus force staff.
Ability, have x charges that can be used to dash foword. Charges replenish over time. (suggest 3 chares where max length of using all 3 will be around 1.5 blink dagger range)
4)Warp dagger.
Recipe, blink plus something.
Ability, blink you and nerby units to target. Targets ground like normal blink.
5)Dual Dagger. [Who are the cutest backstabber? Yes you are :)]
Recipe, blink dagger plus diffusal 2 (or some other diffusal blade)
Ability, if target enemy unit blink to it and apply diffusal debuff, when no target work as normal blink.
6)Blink Hammer [13 out of 13 Thor would take it]
Recipe, blink dagger plus basher (or abbysal blade if you want)
Ability, after blinking stun enemy around for x second (0,75s should be ok with radius 350, with abbysal in recepe ofc should be more powerful)
7)Arcane dagger
Recipe, blink plus some int trash
Ability, after blinking next ability used have shorter cooldown and needs less mana if it was casted in x seconds of blink.
8)Flame dagger. [someone took knife to a flamethrower fight]
Recipe, blink dagger plus something with fire theme.
Ability, after blink put in flames path traveled, deal damage per second (damage dealing similar to meatball)
9)Acid dagger. [Maybe look like cheese but i bet it not taste like one]
Recipe, blink dagger, orb of venom and some trash.
Ability, after blink evry enemy unit that was on path is poisoned, poison deal dmg over time and resources armor.
10)Frozen dagger. [be careful, it's cold]
Recipe, dagger and some trash.
Ability, after blinking create ice path on traveled path(?) (work like invoker ice path but this version is long and narrow instead of short and wide)
11)Earth Dagger. [yep, still elemental family,youngest child but hit like a grandpa]
Recipe, as always, Dagger plus some trash.
Ability, after blink create wall on path, enemies hited by wall are stunned (work like es wall)

Heel Dagger [Worst nightmare of Achilles]

Recipe, dagger plus medalion or his upgrade.
Ability, after blinking aply debuff on caster and surrounding enemies.(debuff of medaliin or his update, or creative variation)
13)Time dagger [i rly wanted to finish in this one, i promise i wanted end on 13 ]
Recipe, dagger and some .........  [yep, you guessed right]
Ability, if targeted hero work like glimpse, if not work normally [i dunno how to describe it better on 2am]
14)Crossbow for daggers [you shoot daggers? Damn son, that must hurt]
Recipe, blink dagger plus daedalus (preferably crystalis [but i get hilarious name for daedalus one so i was must use it instead of crystal dagger or something like that ])
Ability, after blink have 100% chance of critical strike for 1 attack within x seconds (having in mind kunkka it shouldn't aply any type of cleave[trust me, you don't want it ever])
15)Hero dagger [he is doing super hero landing... Woo... hoo...!!!]
Recipe, dagger plus some [yep].
Ability, after blink in destination every unit in small aoe is pushed away, deals damage to enemy [add to this hero landing animation and crater from sun strike then it will be top level hero landing :D]
16)Shadow Dagger
Recipe, blink dagger plus shadow blade [surprisingly every recipe have blink dagger in it, accident? I don't think so.]
Ability, when blinking make caster invisible and create illusion with walks towords for some time [rip old Phantom lancer chlip...  [Clapp... Clapp... Clapp... Most creative name i ever heard of, anyway, there is something in your back ]
Recipe, [like most of it, skip...]
Ability, work like normal blink but enables to deal more damage when attacking from back. (could be fused with dual dagger or dual dagger could be upgraded version of Dagger The Dagger[it's catchy and fun to say, for your and your family, annoy your surrounding, today!])
18)Gold Dagger. [someone was touching it before]
Recipe, blink dagger plus hand of midas
Ability, work as normal blink but passively increase every gold income by 25% (not safe one, forgot the name [where i puted my brain? Where it was? Heh... I think it's on a Walk]
19)Blink shield [rly? You have so many options to choose from, and you choose dagger with shield? Wow, you are truly a wise guy]
Recipe, blink dagger plus vanguard (could be also other shields)
Ability, after blinking create shield around hero that block all damage,lasts 7 seconds or until destroy, hp of shield is 5% of wielder hp, have no reductions.[maybe because it is an reduction, but who am i to debate about it]
20)Nightmare Dagger [sweet dreams dear enemy]
Recipe,  Blink dagger plus some trash
Ability, after blinking causes fear in surrounding enemy (work like lone druid one, not work on illusions) [i bet he is saying "bu" when apire from nowhere]
21)Miracle Dagger [you still reading it? Wow, ok, i know that i writing it like 2 hours but man... This is so much text, how you even have strength to read these]
Recipe, [skip...]
Ability, after blink casts random spell at random unit in range (for ballace spells should be categoried for enemy one's, can be casted only on enemies, neutral, can be castet on enemies and team, friend [zone], can be casted only on friend's) [ballance in item that his main fuel is 0 ballance... Gj]
If you get to the end i want to thank you and leave comment what you liked and why or maybe you dislaked something and you want share why then feel free to also comment. I hoped even if you dislaked ideas at least you had fun. Thanks, now comment time. [Free ban for first person that write "First", get it quickly or you will be to late]

In general, I don't think there should be more blink dagger recipes, some were already suggested  link and some are against the general rules this.
Please read the Guidelines and the already suggested things list, and filter out some of your ideas, I like the elemental flavored ones especially - there should be a pencil icon in the top-right corner of your issue 😉
Also, if you want to create an ordered list, just type the number follower with a period and a space: 1.
Everything else is here.

I don't have available computer to be able edit this, there is no such button on mobile version as i can see. Did that tranquil plus dagger was just added? C'man, it's neat upgrade to tranquils, you not lose healing while fighting with creeps. It's like mini portable heart ;D and rly, one recipe to up blink is not enough :)!BTW, i think my upgrade for same recipe is more interesting :)

In regards of tranquil + blink:
It wasn't added, it is basically on the auto-deny list.
Baumi explicitly used this as an example for "only giving and extra inventory space".
If you think about it, it is true; it gives nothing more, just the combined effects and consequently one less space; and it is uninteresting - boring.
Sorry

Same thing you can tell about daedalus, you combine dmg item and crit, and you get nothing new from it.

Since you have referenced this is a more recent issue, I will assume you have taken whatever you thought  were the best ideas and reposted them there. If you haven't, please do so.

@Haganeko it was not my post xD


Send only profileRequest, never routingRequest

Our code will be simpler if we make AnalystClusterRequest a wrapper around ProfileRequest. Currently it can contain either a ProfileRequest or a RoutingRequest. Non-transit requests do not really need to use profile routing, because travel time does not vary as a function of departure time, but we can handle this by making the window zero-width or just ignoring the end time when no transit modes are specified.

worker and broker updated in OTP branch nontransit-analyst

Done.


Filter chart error when no time column

How to reproduce the bug
Create a chart type filter with a dataset that has no time column.
Uncheck "date filter" in data.
Time column is empty.
When clicking on "run", the following error appear:
Unexpected error
Time column "timestamp" does not exist in dataset
Expected results
In previous version, the filter runs without timestamp column
Actual results
It returns an error.
Environment

browser type and version:
Chrome Version 99.0.4844.51 (Official Build) (64-bit)
Firefox 98.0 (64-bit)
superset version: superset version 1.4.1
python version: python --version : 3.8.12

Checklist

[ x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x ] I have reproduced the issue with at least the latest released version of superset.
[x ] I have checked the issue tracker for the same issue and I haven't found one similar.

Closing this as a "won't fix" since the FilterBox is on a long path toward deprecation, and Dashboard Native Filters should be replacing this in day to day use. If anyone wants to go after this, we'll be happy to review PRs.


Fix memory leaks

FIX MEMORY LEAKS:

Parent view reference must be weak
Use weak proxy to broke memory leak with CADisplayLink
Use private properties in place of global

Great job @asam139. Love your contributions.

100k+ leaks when running ARVideoKit-Example using SceneKit and recording for a few seconds using these updates.

100k+ leaks when running ARVideoKit-Example using SceneKit and recording for a few seconds using these updates.

Hi @mourlamstudios, I have checked if theses leak memories existed before this pull request and it exists. So this pull request is not the problem, these leak memory leaks are of the beginning.

@mourlamstudios, thanks for sharing this. Will investigate the source of the leaks and push some changes when I get the chance to.

@asam139 Ah ok. I got excited when I saw the title of the PR "Fix memory leaks". I figured it fixed all the memory leaks I've been seeing. I noticed the retain cycles in previous releases as well. Glad to see you resolved those. I'll try to take a look at the memory leaks this week as well.


add :Z suffix option to docker --volume

This allows to run this Gradle plugin on Linux distributions with SELinux, such as https://getfedora.org, RHEL, etc.
#21 has full background.

I'm not an expert on Docker options.
What exactly does :Z allow that isn't allowed by default? All I can find about :Z is that on SELinux it should tell Docker to label the volume content in such a way that it can only be used by the current container - but it also looks like there's an :ro suffix. Would that work or be more appropriate since we don't intend to modify the mounted content with buf?

What exactly does :Z allow that isn't allowed by default? All I can find about :Z is that on SELinux it should tell Docker to label the volume content in such a way that it can only be used by the current container

Correct, to quote from https://docs.podman.io/en/latest/markdown/podman-run.1.html:

To change a label in the container context, you can add either of two suffixes :z or :Z to the volume mount. These suffixes tell Podman to relabel file objects on the shared volumes. The z option tells Podman that two containers share the volume content. As a result, Podman labels the content with a shared content label. Shared volume labels allow all containers to read/write content. The Z option tells Podman to label the content with a private unshared label.

This is for podman,  but it should be the same for docker... https://docs.docker.com/engine/reference/run/ et al. does mention it, but e.g. http://manpages.org/docker-run has more complete clear documentation.

but it also looks like there's an :ro suffix. Would that work or be more appropriate since we don't intend to modify the mounted content with buf?

The :ro suffix is "orthogonal" to using :Z. :ro is not sufficient on an SELinux enabled distro like Fedora:
$ docker run --volume "$(pwd):/workspace:ro" --workdir /workspace bufbuild/buf lint
Emulate Docker CLI using podman. Create /etc/containers/nodocker to quiet msg.
Failure: could not clean relative path "".

but combining both, for different reasons (:Z for RW or RO access under SELinux, and :ro to limit to RO but not RW) seems like a great idea:
docker run --volume "$(pwd):/workspace:ro,Z" --workdir /workspace bufbuild/buf lint

This works. I'll amend this PR to add ro.

Ok, sounds good. Would you mind rebasing this PR on main? I have updated the version of the unit test result publication action since it seemed to be failing on this PR.

Would you mind rebasing this PR on main?

done

Huh - I was too quick to assume ro - buf build writes the image JSON. Let's leave it with just Z.

Huh - I was too quick to assume ro - buf build writes the image JSON. Let's leave it with just Z.

done


Use non-deprecated Jedi functions >= 0.15.0

Definition.get_signatures instead of Definition.params (introduced in 0.15.2): #9458
Script(....).get_names() instead of jedi.api.names(...) (introduced in 0.16.0)
Script(....).complete() instead of Script(...).completions (introduced in 0.16.0)
Script(....).get_signatures() instead of Script(...).call_signatures (introduced in 0.16.0)

And probably others.

Check changelog for what got introduced in 0.17.0: https://github.com/microsoft/vscode-python/pull/11252

Closing in favour of https://github.com/microsoft/vscode-python/issues/11995


Animations goToFrame: Fix breaking change introduced in #15126

See https://forum.babylonjs.com/t/animation-blending-and-key-interpolation-bug-in-new-cdn/54729/4.
My change in #15126 was a breaking change, because when the weight isn't -1, the animation is considered additive, and this can break if there isn't at least one animation that isn't additive.
I needed this change for Teams Avatar, so I made a change so that the old behavior is the default, but I added a flag so that I could use the animation weight on demand.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15818/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15818/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15818/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15818/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15818/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15818/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15818/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15818/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15818/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15818/merge/testResults/webgl2playwright/index.html


Change folder structure

Changes to follow the project structure suggested at nodebestpractices.
In short, instead of group files by technical role (controllers, models, repositories...), its better group by components (notes, users...). Additionally, inside each component files are named with its technical role (notes.controller.js, notes.router.js...).

Thank you for your help. I applied to the master. Verify that it is OK and submit another pullrequest.
Thank ;)


Unable to resolve module `semantic-ui-css/semantic.css

Bug Report
Following exact steps to install and import results in failure to resolve module.
Steps
Attempt to install semantic-ui-css by following setup instructions and adding the relevant import statement:
yarn add semantic-ui-css
import 'semantic-ui-css/semantic.min.css'
Expected Result
Installed and available
Actual Result
Unable to resolve module semantic-ui-css/semantic.cssfrom/Users/dude/Documents/myapp/app/index.js: Module semantic-ui-css/semantic.css does not exist in the Haste module map
Version
<EMAIL_ADDRESS>

What bundler are you using? Webpack? Is css-loader configured properly?
Please provide a repo with a minimal repro and will try to help. However, this issue is not related to SUIR at all.

@layershifter thanks for the reply. I have simply created a new React Native app as per React docs and then followed Semantic UI steps to add the default theme. I am not customizing any themes at this stage.
repo:

same error in Create React App with latest update package tag

same error in Create React App with latest update package tag. before update haven't any error.
./src/index.js  Module not found: Can't resolve 'semantic-ui-css/semantic.css' in 'C:\...' Compiling... Failed to compile.

@manemmao it seems that you forgot to install semantic-ui-css package, try to run yarn add semantic-ui-css and then run your app again.

@samuelgoldenbaum Thanks for your reply. As I know you can't use SUIR with React Native, https://github.com/Semantic-Org/Semantic-UI-React/issues/790#issuecomment-258333177. That's root issue, styles are also incompatible 😢

@layershifter no i didnt forgot. it was working right before update. not stable version, latest one.

But 2.4.1 is the latest for semantic-ui-css:

Status?

@levitomer this is 99% related to your tooling. Please provide more details.

facing same Ishh :( semantic-ui-css thing messing up my react app and i still want to use it.
I am using webpack react/electronjs

Well I found a way around it :)
I linked the semantic.  in index.html
then called import { Button } from 'semantic-ui-react'; in my component :) well it might not be the best way since its better to use CDN if the app is hosted on a server if locally like me using it along side electron then you can use my method or dowload the semantic-ui separately then use the link:href


Fix Glaxx blocks and change type counting

Fixed Glaxx blocks not being see-through.
Changed type counting to start from 0.

Have you tested to make sure BlockRenderLayer.TRANSLUCENT behaves for all other block types? It should work fine but I want to be sure.

@InsomniaKitten
My knowledge is very limited, all I ever did MC-Modding related is editing some things here and there.
So.. what do you mean by "behaves for all other block types" ?
As far as I can see only Glaxx is ever BlockRenderLayer.TRANSLUCENT.
register(new BlockXtone("glaxx", new ItemStack(Blocks.GLASS)).setSeeThrough());
And that works just fine, with and without OptiFine.

The off by one was intentional as most users don't have a development background and find starting at 0 odd

If you remove the type count change, I'll pull this, though I still need to fix them causing shadows

@TehNut reverted it, btw, I tried it myself but failed to add the flat lamps. That's a beyond my knowledge of forge and co. Are you going to add them?

Unsure. Possibly

Formatting nitpick, could you re-add the space after the +

I'll cherry-pick to 1.11 when I get home


Will you add support for ollama and lmstudio?

Will you add support for ollama or lmstudio?

This process adds support for all ollama downloaded models to GPT4All. Since GPT4All is supported by PentestGPT, this should provide a workaround to get ollama model support for PentestGPT.


Add the $.body prefix to DslPart matchers

Fixes #300
I'm not entirely sure that this is the right solution. I had to alter some unit tests to make this pass, but perhaps I got it wrong, so please review this.
This change actually limits DslPart to only be used for creating matchers for body. Perhaps we need to generalize it so it can also be used for header and other HTTP parts (see V2 specs).

Looks all good to me, thanks for the PR. I think this is an example of something ending up after a series of refactors over time.


Show Drives instead of expand or minimize the label

What's the Problem?
When clicking on "Drives" in the left panel it will expand or minimize the group. I think it will be more useful if it shows the drivers when clicked on it, like in the image below:
Solution/Idea

And it can continue expanding and minimizing the group.
Alternatives

Priorities
No response
Files Version
<IP_ADDRESS>
Windows Version
10.0.22478.1000
Additional comment
No response

This is what the home page is designed for you can turn off all elerments except drives. It should also not be added that way as the sidebar entry will become usless with it opening and closing constantly. See Windows 11 media player playlist for an example.


Error when parsing currency with 0.7.12

Just updated to version 0.7.12 and found that price checking currency gives a parsing error.
Price checking uniques, rares, maps etc works fine, just the currency that gives the error. Any fix to this?

+1 error too

me and my friend with the same issue (including essences, fossils, etc.)

experiencing this now as well. Essences, fossils, catalysts and some currency eg annulment are showing "an error occurred while parsing item"

This issue has been fixed in v0.7.13


[docs] Add a shortcut to access the search bar

Shortcut CTRL + K for accessing search bar was added and issue with active element gets fixed.
Fixes #23804
https://deploy-preview-23959--material-ui.netlify.app/getting-started/installation/

@mnajdova I have experimented a bit with using the system over makeStyles in 0e09c6e477544d342c385e7c2b2c6bdc4a62b93b. I have faced an issue with TypeScript, that I have tried to solve with the diff you can see. However, I gave up on the idea of pushing it further. The results of https://github.com/mui-org/material-ui/blob/next/benchmark/browser/README.md#output made me too worried about the performance.

I fail to even imagine how somebody could write code with speech only.

I came across this:
https://serenade.ai/
(Demo: https://www.youtube.com/watch?v=HtkelODtCkM)
While looking for this:
https://www.joshwcomeau.com/blog/hands-free-coding/
(Demo inline)

@mbrookes The best would be to have it read our mind 👌

Not quite mind reading, but combined with speech to text, this is pretty damned close: https://www.youtube.com/watch?v=fZSFNUT6iY8 (although presumably a carefully scripted example).


Specify port in 'Test_isValidResolvConf'

Proposed Changes
This PR expands testing of the 'Valid ResolvConf' scenario (pkg/agent/config/config_internal_test.go).
Types of Changes
New Feature
*I was not able to run tests locally, so I'm opening a pull request for the explicit purpose of running tests in CI.  I'd like to know whether or not port numbers are supported, when installing with the --resolv-conf flag.
In other words, where an end-user might run a DNS server on an alternate port, and provide that number in a configuration file (e.g./etc/rancher/k3s/resolv.conf ) when installing like the following:
curl -sfL https://get.k3s.io | sh -s - --write-kubeconfig-mode 644 --resolv-conf /etc/rancher/k3s/resolv.conf
Verification
If CI passes, I expect that to be enough verification.
Testing

Yes.
Linked Issues

N/A
User-Facing Change

NONE

Further Comments

I just want to mention to the maintainers, as I understand it from my own local testing, running a DNS service on port 5300 and specifying it in /etc/rancher/k3s/resolv.conf was not respected.
Can anyone confirm?
That said, I do not recommend this PR to be merged since it seems the tests passing are a false positive.


chore(main): release 1.0.4

:robot: I have created a release beep boop
1.0.4 (2024-11-26)
Bug Fixes

deps: update dependency eslint-plugin-perfectionist to v4.0.3 (#67) (43a1772)
deps: update dependency eslint-plugin-perfectionist to v4.1.2 (#83) (df887ec)
deps: update dependency typescript-eslint to v8.16.0 (#76) (3ac721e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.4
:sunflower:


Notebook: line breaks in a text cell is not rendered properly

see screenshots below

I am expecting 3 lines but all the text are in one line.

Let's see if this is inconsistent with VSCode markdown preview

This is the same behavior as the VSCode markdown preview.

@ronychatterjee @yualan Can you help find a contact on the VSCode side?

This is by design: https://markdown-guide.readthedocs.io/en/latest/basics.html
To get a newline in markdown, you need either two spaces at the end of the line, or an explicit line break.


support for more dims in pymc_marketing\model_config.py

Howdy,
Trying to follow the example here with a larger list of dimensions and I encountered this message:

File "C:\Users\Lorenzt01\PycharmProjects\pymcmarketinghierarchy.venv\Lib\site-packages\pymc_marketing\model_config.py", line 210, in create_dim_handler
raise UnsupportedShapeError(
pymc_marketing.model_config.UnsupportedShapeError: At most two dims can be specified. Raise an issue if support for more dims is needed.

My use case does in fact involve more than two dims; I'm quite certain a workaround will be findable, but I do like to follow directions.  FWIW, this pymc version 5.15.1 and pymc-marketing 0.7.0.
Thanks for all the work on a lovely software package!
TL

Hi @lorenze3
This is solved with the Prior class. Have your tried the example on the latest / main branch instead of 0.7.0?

If I flip the documentation URL over to /latest/ instead of /stable/, I can see that there have been (or will be?) changes, but I haven't managed to get a different version of the package installed. Iam not finding a 'how to install dev version' or 'nightly release' version anywhere.  0.7.0 is the current latest version, yes?
I can work through a non-dim and coord- approach to building my model and wait for the changes to work there way through the system.

Ok, I've managed to get that example (the version here: https://www.pymc-marketing.io/en/latest/notebooks/mmm/mmm_components.html ) running by just running the modules out of a download of the main branch of the repo. . . I'll try my 'many many dimensions' using the Prior class -- thanks for the pointer!.

Correct, 0.7.0 is the latest version at the moment. I will make a release shortly which will include the Prior class as well.
Happy to hear that you got it running. If you have any issues with the new class, feel free to raise another issue!


drogon_ctl create model fails if DB password has a space in it

Describe the bug
If your model.json file has a space character in the "passwd" field e.g.
{
    "rdbms":"postgresql",
    "host":"<IP_ADDRESS>",
    "port":5432,
    "dbname":"somedbname",
    "user":"foo",
    "passwd":"123456789<PHONE_NUMBER>",
    "tables":[]
}

then calling:
drogon_ctl create model ./models/
gives:
postgresql
Connect to server...
Source files in the ./models/ folder will be overwritten, continue(y/n)?
20210810 05:36:39.612090 UTC 583264 FATAL Socket fd < 0, Usually this is because the number of files opened by the program exceeds the system limit. Please use the ulimit command to check. - PgConnection.cc:70

removal of the space in the "passwd" field fixes the issue.
Tested on an Intel Linux machine and a MacBook M1.

Which version of Drogon are you using? The bug should have been solved back in #587

I cloned and built drogon yesterday. Changelog says:
[1.7.1] - 2021-06-24

Could you share the output of drogon_ctl version?

     _
  __| |_ __ ___   __ _  ___  _ __
 / _` | '__/ _ \ / _` |/ _ \| '_ \
| (_| | | | (_) | (_| | (_) | | | |
 \__,_|_|  \___/ \__, |\___/|_| |_|
                 |___/

A utility for drogon
Version: 1.7.1
Git commit: 477c3dca7dabb42ad5f0748499f0b0a23ba57825
Compilation:
  Compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
  Compiler ID: AppleClang
  Compilation flags: -std=c++17 -I/usr/local/include
Libraries:
  postgresql: yes  (batch mode: no)
  mariadb: no
  sqlite3: yes
  openssl: yes
  brotli: yes
  boost: no
  hiredis: no
  c-ares: yes

@bradleysmith Please update and use the latest patch. It should have solved the issue. Feel free to re-open it if still fails on your machine.

That did the job. Many thanks for the rapid fix.


Newest airline update causes fzf.vim to behave oddly

environment

vim: neovim v0.3.1
vim-airline:: master
Have you reproduced with a minimal vimrc: yes
What is your airline configuration:
`
set laststatus=2

let g:airline#extensions#tabline#enabled    = 1
let g:airline#extensions#virtualenv#enabled = 1
let g:airline#extensions#vimagit#enabled    = 1
let g:airline#extensions#branch#enabled     = 1
set encoding=utf-8
if !exists('g:airline_symbols')
let g:airline_symbols = {}
endif
"let g:airline#extensions#tabline#left_sep = '⮀'
"let g:airline_left_sep           = '▶'
"let g:airline_right_sep          = '◀'
let g:airline_left_sep = ''
let g:airline_left_alt_sep = ''
let g:airline_right_sep = ''
let g:airline_right_alt_sep = ''
let g:airline_symbols.crypt      = '🔒'
let g:airline_symbols.linenr     = '␤'
let g:airline_symbols.maxlinenr  = '☰'
let g:airline_symbols.branch     = '⎇ '
let g:airline_symbols.paste      = 'Þ'
let g:airline_symbols.spell      = 'Ꞩ'
let g:airline_symbols.notexists  = '∄'
let g:airline_symbols.whitespace = 'Ξ'
`
if you are using terminal:

terminal: bash
$TERM variable: xterm-256color
color configuration (:set t_Co?): not set
if you are using Neovim:
does it happen in Vim: yes

actual behavior

expected behavior

I've noticed with the latest commits to master (the six commits on Nov 20th) that my fzf is broken now as is shown by the gifs above.
I verified this by removing all but fzf.vim and airline and when I install/uninstall I notice the issue appear/disappear.
I realize that this is a combination of two plugins but given that airlines latest commits may be the issue I though I would just bring the issue here. Sorry if this is not the correct thread for this.

I also encountered this problem using neovim.
It not only affects fzf, but all terminals. The simplest way to reproduce the problem is to run nvim 'term://ls'.
The phenomenon appears since  the commit 31f5a9307d5.
Disabling the autocmd TermOpen fixes the problem.

Does it happen in vim?

Am 21.11.2018 um 05:12 schrieb Sha Liu<EMAIL_ADDRESS>I also encountered this problem using neovim.
It not only affects fzf, but all terminals. The simplest way to reproduce the problem is to run nvim 'term://ls'.
The phenomenon appears since the commit 31f5a9307d5.
Disabling the autocmd TermOpen fixes the problem.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I ran into this issue today as well, but have managed to resolve it with for now by specifying a commit (ada0ba8ae3eea778d165ec4794ee557df98fab87) to install with Plug:
Plug 'vim-airline/vim-airline', { 'commit': 'ada0ba8ae3eea778d165ec4794ee557df98fab87' }

This will checkout the version of vim-airline prior to the change that causes this issue 👍
You will have to remove the vim-airline folder from ~/.config/nvim/plugins, and then run PlugInstall.
✌

I see two different commits here mentioned. Which one causes the issue? And how do I  reproduce? Please give step-by-step instroductions, as I don't use fzf.

Let me summarize what I found:

the first commit that has the problem is f9ccb57e7.
even for the latest commit, the problem can be solved by changing line 131 from if exists("##TerminalOpen") || exists('##TermOpen') to if exists("##TerminalOpen"), which disables the TermOpen autocmd.
To reproduce the problem, you do not need to use fzf. Simply use terminal to run a command, such as :term ls. I cannot see any output if the above line is not corrected.
vim8 does not have the problem since TermOpen is neovim specific
I feel the problem is caused by line 212 of lighlighter.vim: call airline#highlighter#highlight(['inactive']), but I don't know how; since I am not familiar with the code base.

:term ls works okay for me in nvim:

this is with nvim 0.2.2 (debian stable or something)

i also uses nvim0. 3.1, the same as the op.
It has precompiled binary available for download on the official site.
Christian Brabandt<EMAIL_ADDRESS>于 2018年11月21日周三 下午11:05写道：

:term ls works okay for me in nvim:
[image: grafik]
https://user-images.githubusercontent.com/244927/48849682-2ae87980-eda7-11e8-94df-a27f40351420.png
this is with nvim 0.2.2 (debian stable or something)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/vim-airline/vim-airline/issues/1828#issuecomment-440696034,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL4iZo9lsVftu8InyWs8i7tTW5fUhqkdks5uxWuqgaJpZM4YsNac
.

I'll have a closer look later.

tried the nightly appimage , same thing:

same here.

@jinmel, could you please help to confirm whether other terminal commands behaves correctly? run :term ls in you vim. @chrisbra does not use FZF.

@liushapku it just opens a new buffer which has empty contents

@jinmel, thanks. Please also confirm that the cwd is not empty.
My bad to have chosen ls. I have changed the command to :term pwd which should always has output.

@liushapku the cwd was empty. I also tried with pwd and it's just a blank line with [process exited]

This is the indication that all terminals (at least non-interactive terminals) have the same problem. The same as what I had.

okay, let me revert the TermOpen autocommand, as this seems to be broken for Neovim.


how do I distinguish an incoming text msg from binary ?

This is a snippet from my script :
require_once("vendor/autoload.php");
use Ratchet\MessageComponentInterface;
use Ratchet\ConnectionInterface;

class WebSocketHandler implements MessageComponentInterface {
    public function onOpen(ConnectionInterface $conn) {
		// Handle new WebSocket connection
		echo "New connection! ({$conn->resourceId})\n";
    }

    public function onMessage(ConnectionInterface $from, $msg) {
		// Handle incoming WebSocket message
		$now = DateTime::createFromFormat('U.u', microtime(true));
		echo "New message! " . $now->format("m-d-Y H:i:s.u") . "\n";

		$myFile = $now->format("Y-m-d H:i:s.u") . "_" . uniqid() . ".txt";
		$fh = fopen($myFile, 'a');
		fwrite($fh, $msg);
		fclose($fh);
    }

When $msg contains binary... things don't go as expected.
I suppose because I'm treating the binary content as text.
How can I distinguish the two ?

Hello @Zixim,
IMHO and if I'm not mistaken, some familiarization with the WebSocket protocol is required to conclude that at the moment there's no such functionality in the Ratchet library.

Broadly speaking, there are types for textual data (which is interpreted as UTF-8 [RFC3629] text), binary data (whose interpretation is left up to the application), and control frames (which are not intended to carry data for the application but instead for protocol-level signaling, such as to signal that the connection should be closed).

The WebSocket Protocol. (2023, September 15). https://www.rfc-editor.org/rfc/rfc6455
5.7.  Examples

   o  A single-frame unmasked text message

      *  0x81 0x05 0x48 0x65 0x6c 0x6c 0x6f (contains "Hello")

   o  A single-frame masked text message

      *  0x81 0x85 0x37 0xfa 0x21 0x3d 0x7f 0x9f 0x4d 0x51 0x58
         (contains "Hello")

   o  A fragmented unmasked text message

      *  0x01 0x03 0x48 0x65 0x6c (contains "Hel")

      *  0x80 0x02 0x6c 0x6f (contains "lo")

   o  Unmasked Ping request and masked Ping response

      *  0x89 0x05 0x48 0x65 0x6c 0x6c 0x6f (contains a body of "Hello",
         but the contents of the body are arbitrary)

      *  0x8a 0x85 0x37 0xfa 0x21 0x3d 0x7f 0x9f 0x4d 0x51 0x58
         (contains a body of "Hello", matching the body of the ping)

   o  256 bytes binary message in a single unmasked frame

      *  0x82 0x7E 0x0100 [256 bytes of binary data]

   o  64KiB binary message in a single unmasked frame

      *  0x82 0x7F 0x0000000000010000 [65536 bytes of binary data]

11.8.  WebSocket Opcode Registry

This specification creates a new IANA registry for WebSocket Opcodes
  in accordance with the principles set out in RFC 5226 [RFC5226].

  As part of this registry, IANA maintains the following information:

  Opcode
     The opcode denotes the frame type of the WebSocket frame, as
     defined in Section 5.2.  The opcode is an integer number between 0
     and 15, inclusive.

  Meaning
     The meaning of the opcode value.

  Reference
     The specification requesting the opcode.

  WebSocket Opcode numbers are subject to the "Standards Action" IANA
  registration policy [RFC5226].

  IANA has added initial values to the registry as follows.

    |Opcode  | Meaning                             | Reference |
   -+--------+-------------------------------------+-----------|
    | 0      | Continuation Frame                  | RFC 6455  |
   -+--------+-------------------------------------+-----------|
    | 1      | Text Frame                          | RFC 6455  |
   -+--------+-------------------------------------+-----------|
    | 2      | Binary Frame                        | RFC 6455  |
   -+--------+-------------------------------------+-----------|
    | 8      | Connection Close Frame              | RFC 6455  |
   -+--------+-------------------------------------+-----------|
    | 9      | Ping Frame                          | RFC 6455  |
   -+--------+-------------------------------------+-----------|
    | 10     | Pong Frame                          | RFC 6455  |
   -+--------+-------------------------------------+-----------|

The OP_TEXT and OP_BINARY constants defined in src/Messaging/Frame.php could be used in the onControlFrame() method to distinguish an incoming text message from binary, however, it is assumed a text-based connection.
I hope this will help,


Visio diagram for the enterprise scale landing zone design.

I would like to see an editable and scalable Visio diagram for the enterprise-scale landing zone design for trey research
https://github.com/Azure/Enterprise-Scale/blob/main/docs/reference/treyresearch/README.md#what-will-be-deployed

There is a link to that here:
https://github.com/Azure/Enterprise-Scale/discussions/570

Thanks, we could post this link on the page though


Repair Vic3 Terrain Map

Fix the Vic3 Terrain Map using https://github.com/linnest2020/vic3-mapdata-editor and the EU4 Terrain Map from in-game full world screenshot.

When converting, use a file provided by Vic3 mod if at all possible, then use one in configurables.

Or just use decentralized world folder for the override


Learning paths: Redesign competency graph

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I strictly followed the server coding and design guidelines.
[x] I documented the Java code using JavaDoc style.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[x] I strictly followed the client coding and design guidelines.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context

Similar to the student learning path UI the competency graph for students should be redesigned. This PR is part of the redesign of the complete student learning path UI. Therefore it will be reviewed and tested before getting merged into the base PR #8733. (which then will be merged into develop)
Description

Students should have the ability to quickly see how the competencies they achieve in the learning path are roughly connected to each other.
Steps for Testing

Prerequisites:

1 Students
at least 2 competencies connected to each other (assumes, extends or similar)
at least 2 lecture units (or exercises) each connected to a competency

Log in to Artemis
Navigate to Learning paths student view
Open the navigation overview
Open the competency dependency graph

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[x] Test 1
[ ] Test 2

Test Coverage

Screenshots

Before

After

Tested on ts3. In the instructor view for learning paths translations are now missing:
I also get a lot of internal server errors, I'll re-test on another server and see if they are related (as i can't access grafana ATM)

This is related to TS3. Had the same issue yesterday. The screenshot also still shows the old graph. Don't know why there is this problem on TS3.

Tested on ts3. In the instructor view for learning paths translations are now missing:
I also get a lot of internal server errors, I'll re-test on another server and see if they are related (as i can't access grafana ATM)

This behavior also happens in ts2, but the changes work fine on TS1

Tested on TS4. When no tasks are completed in the learning path, the starting competency node of the graph displays an arrow that points to nothing

I can't reproduce it on TS4. Can you give me more insights?


Allow users to hide footer

Is your feature request related to a problem? Please describe.
As a user who embeds the Clocks dashboard into MS Teams or Confluence page, I want to remove the footer: it is not needed in that context and only consumes valuable vertical space.
Describe the solution you'd like
Probably good idea will be if Footer appears by default, but has a control in Settings that allows user to deselect it. By default checkbox "Display footer" is ON and footer is visible. User may switch it OFF, then footer disappears.
Additional context
This is how page looks now once embedded into MS Teams:

@SheratriumEPAM, please check and approve


Add support for objc_import

Part of #73 and #86.

This doesn't have an example, as someone else can create multi-arch, proper iOS version .a and attach it somewhere :smile:.


This means, all dependent workers, like network_manager, must
be destroyed as well before we can destroy the cloud manager
I introduce a child_manager association and queues up a destroy of all child_managers
https://github.com/ManageIQ/manageiq/pull/14848
@cben @Ladas @jrafanie

@zeari

@zeari @jrafanie please have a look

@blomquisg does it make sense from a UI / Business perspective to delete all dependent managers? Could we have a case where the cloudmanager is about to be deleted, but e.g. a storage manger should be kept?

@blomquisg does it make sense from a UI / Business perspective to delete all dependent managers? Could we have a case where the cloudmanager is about to be deleted, but e.g. a storage manger should be kept?

I dont think so, but i could be wrong. I would solve this by changing the relation to :dependent => destroy and having disable! run recursively through the child managers, disabling them as well. The next time orchestrate_destroy comes out of the queue, all workers related to all managers (children and parent) would be off and destroying the parent manager would destroy the children managers.
cc @moolitayer

changing the relation to :dependent => destroy

this is already the case, via e.g. the has_network_manager_mixin.
@zeari why not calling destroy_queue as I did? This method is the entry point for all orchestrated destroy stuff, like disabling and re-scheduling etc.

@zeari why not calling destroy_queue as I did? This method is the entry point for all orchestrated destroy stuff, like disabling and re-scheduling etc.

This way should work fine.
I think its a little more efficient to destroy everything in one queued action by destroying the parent instead of queueing and destroying each manager separately. (But it might be a trivial optimization)

@blomquisg could you have a look if that makes sense from a user perspective?

@agrare could you have a look at this one?

@agrare please have a look again.
I moved the scheduling of destroy for child_managers out of schedule_destroy_queue because this gets called repeatedly if destroy fails.
The class method destroy_queue now just delegates to the instance method, which in turn calls schedule_destroy.

FWIW, looks good to me. 👍


Can not get module

Thanks for your helpful library!
Today, when I run go get github.com/go-redis/redis
It stopped with this error:
package go.opentelemetry.io/otel/api/trace: cannot find package "go.opentelemetry.io/otel/api/trace" in any of:
/usr/local/go/src/go.opentelemetry.io/otel/api/trace (from $GOROOT)
/go/src/go.opentelemetry.io/otel/api/trace (from $GOPATH)
Seems go.opentelemetry.io/otel/api/trace updated new version, please help us solved it.

https://github.com/go-redis/redis/issues/1534

What about this?
go get github.com/go-redis/redis/v8
cannot find package "github.com/go-redis/redis/v8" in any of:
/usr/local/go/src/github.com/go-redis/redis/v8 (from $GOROOT)
/home/vini/go/src/github.com/go-redis/redis/v8 (from $GOPATH)


felix/bpf: ICMP conntrack fix

state->dport is overloaded and used as ICMP type/code for policy and
must be zeroed before it is used in conntrack lookup. The value gets
propagated into post_nat_dport and thus also this has to be zeroed.
Description
fixes https://github.com/projectcalico/calico/issues/5358

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

/merge-when-ready


Set up @interactors/material-ui in new repo

Motivation
Combine interactors repos.
Approach
WIP
After merge

[ ] Move issues from https://github.com/thefrontside/material-ui-interactors
[ ] Archive https://github.com/thefrontside/material-ui-interactors
[ ] npm deprecate material-ui-interactors

The preview packages of this pull request have been published.
Click on the following packages for instructions on how to install them:
@interactors/html

Install using the following command:
$ npm install @interactors/html@material-ui-move

Or update your package.json file:
{
  "@interactors/html": "material-ui-move"
}

@interactors/material-ui

Install using the following command:
$ npm install @interactors/material-ui@material-ui-move

Or update your package.json file:
{
  "@interactors/material-ui": "material-ui-move"
}

@interactors/with-cypress

Install using the following command:
$ npm install @interactors/with-cypress@material-ui-move

Or update your package.json file:
{
  "@interactors/with-cypress": "material-ui-move"
}

  Generated by :no_entry_sign: dangerJS against eee44a76519cdef77e28b2553cff625842ab932e

The material-ui-interactors repo was using covector, not changesets. Does changesets work with the .alpha.minor scheme in use here?

@jbolda any insight here?


With IAM Authentication enabled, Connector does duplicate refresh

Using the Cloud SQL Proxy with debug logging enabled, you can see that the Go Connector makes two refresh operations for one connection attempt:
$ ./cloud-sql-proxy enocom-experiments-304623:us-central1:pg2 --debug-logs --auto-iam-authn
2024/04/15 10:26:49 Authorizing with Application Default Credentials
2024/04/15 10:26:49 [enocom-experiments-304623:us-central1:pg2] Connection info added to cache
2024/04/15 10:26:49 [enocom-experiments-304623:us-central1:pg2] Connection info refresh operation started
2024/04/15 10:26:49 [enocom-experiments-304623:us-central1:pg2] Connection info refresh operation complete
2024/04/15 10:26:49 [enocom-experiments-304623:us-central1:pg2] Current certificate expiration = 2024-04-15T17:26:49Z
2024/04/15 10:26:49 [enocom-experiments-304623:us-central1:pg2] Connection info refresh operation scheduled at 2024-04-15T17:22:49Z (now + 56m0s)
2024/04/15 10:26:49 [enocom-experiments-304623:us-central1:pg2] Listening on <IP_ADDRESS>:5432
2024/04/15 10:26:49 The proxy has started successfully and is ready for new connections!

2024/04/15 10:26:58 [enocom-experiments-304623:us-central1:pg2] Accepted connection from <IP_ADDRESS>:47630
2024/04/15 10:26:58 [enocom-experiments-304623:us-central1:pg2] Connection info refresh operation started
2024/04/15 10:26:58 [enocom-experiments-304623:us-central1:pg2] Connection info refresh operation complete
2024/04/15 10:26:58 [enocom-experiments-304623:us-central1:pg2] Current certificate expiration = 2024-04-15T17:26:57Z
2024/04/15 10:26:58 [enocom-experiments-304623:us-central1:pg2] Connection info refresh operation scheduled at 2024-04-15T17:22:57Z (now + 56m0s)
2024/04/15 10:26:58 [enocom-experiments-304623:us-central1:pg2] Now = 2024-04-15T16:26:58Z, Current cert expiration = 2024-04-15T17:26:57Z
2024/04/15 10:26:58 [enocom-experiments-304623:us-central1:pg2] Cert is valid = true
2024/04/15 10:26:58 [enocom-experiments-304623:us-central1:pg2] Dialing <IP_ADDRESS>:3307

This is happening because the Proxy calls EngineVersion to warm the cache, but EngineVersion doesn't support a way to specify if IAM Authn is used or not, and so assumes it's not in use (the nil argument).
When the call to connection info occurs, it enables IAM auth, and causes the existing refresh to be canceled.

This will be fixed with https://github.com/GoogleCloudPlatform/cloud-sql-go-connector/issues/731 most likely.

This bug has a few different manifestations.

https://github.com/GoogleCloudPlatform/cloud-sql-proxy/issues/2224
https://github.com/GoogleCloudPlatform/cloud-sql-proxy/issues/2212


Quick Accent feature : reserved, copyright (and trademark) ® / © / ™

Description of the new feature / enhancement
Reserved (®) on "r"
Copyright (©) on "c"
Trademark (™) on "t"
Scenario when this would be used?
While working on documents brands related, it'll be very useful to have these symbols incorporated in the quick accent tool.
Supporting information
No response

® is an option under r
© is an option under c
™ is an option under t
fyi they are at the ends of these lists, so it can quickly be accessed by holding shift while tapping space to reverse the navigating direction.


fix: indentation for non-tls bootstrapping kubeconfig

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

error from kubelet which this fixes:
Feb 14 18:01:20 aks-agentpool0-95253376-vmss000000 kubelet[16243]: Error: failed to run Kubelet: invalid kubeconfig: error loading config file "/var/lib/kubelet/kubeconfig": yaml: line 16: mapping values are not allowed in this context

only affects non-tls bootstrap

opting to merge due to ADO issues with e2e today


[BUG] Opensearch data node get's constantly excluded from shard allocations

What is the bug?
Opensearch data node get's constantly excluded from shard allocations
How can one reproduce the bug?
Deploy the opensearch operator and cluster with the configuration shown below.
What is the expected behavior?
Working opensearch cluster without yellow health state / no excluded data nodes.
What is your host/environment?
Debian 12 x64 / k3s v1.28.8+k3s1
Do you have any screenshots?
Outputs listed below.
Do you have any additional context?
I have more than enough disk space available and writing only 4-6 GiB per Day into opensearch.
Hello! I'm using the Helm OpenSearch-Operator and Helm OpenSearch-Cluster.
My cluster is showing a yellow status due to unassigned replica shards.
I ran GET _cluster/settings and received the following output:
{
  "persistent" : {
    "plugins" : {
      "index_state_management" : {
        "template_migration" : {
          "control" : "-1"
        }
      }
    }
  },
  "transient" : {
    "cluster" : {
      "routing" : {
        "allocation" : {
          "enable" : "all",
          "exclude" : {
            "_name" : "opensearch-cluster-data-nodes-0"
          }
        }
      }
    }
  }
}

And the health of my cluster: GET _cluster/health?pretty
{
  "cluster_name" : "opensearch-cluster",
  "status" : "yellow",
  "timed_out" : false,
  "number_of_nodes" : 6,
  "number_of_data_nodes" : 3,
  "discovered_master" : true,
  "discovered_cluster_manager" : true,
  "active_primary_shards" : 171,
  "active_shards" : 335,
  "relocating_shards" : 0,
  "initializing_shards" : 0,
  "unassigned_shards" : 108,
  "delayed_unassigned_shards" : 0,
  "number_of_pending_tasks" : 0,
  "number_of_in_flight_fetch" : 0,
  "task_max_waiting_in_queue_millis" : 0,
  "active_shards_percent_as_number" : 75.62076749435666
}

I attempted to remove data-nodes-0 from the exclusion list using a PUT request, but it automatically gets added back to the list after a few seconds.
Here are some details about my setup:
helm list
NAME                    NAMESPACE       REVISION        UPDATED                                         STATUS          CHART                           APP VERSION
opensearch-cluster      opensearch      5               2024-04-22 13:53:25.886739305 +0200 CEST        deployed        opensearch-cluster-2.5.1        2.5.1
opensearch-operator     opensearch      4               2024-04-22 14:13:59.756698617 +0200 CEST        deployed        opensearch-operator-2.5.1       2.5.1

kubectl get OpenSearchCluster
NAME                 HEALTH   NODES   VERSION   PHASE     AGE
opensearch-cluster   yellow   6       2.8.0     RUNNING   66d

kubectl describe OpenSearchCluster opensearch-cluster
Name:         opensearch-cluster
Namespace:    opensearch
Labels:       app.kubernetes.io/managed-by=Helm
Annotations:  meta.helm.sh/release-name: opensearch-cluster
              meta.helm.sh/release-namespace: opensearch
API Version:  opensearch.opster.io/v1
Kind:         OpenSearchCluster
Metadata:
  Creation Timestamp:  2024-02-15T15:44:05Z
  Finalizers:
    Opster
  Generation:        5
  Resource Version:  86865705
  UID:               73cea515-fdf2-4d78-a841-ec91a889a1a3
Spec:
  Bootstrap:
    Resources:
  Conf Mgmt:
  Dashboards:
    Enable:  true
    Opensearch Credentials Secret:
      Name:    admin-credentials-secret
    Replicas:  1
    Resources:
      Limits:
        Cpu:     600m
        Memory:  4096Mi
      Requests:
        Cpu:     300m
        Memory:  4096Mi
    Service:
      Type:   ClusterIP
    Version:  2.3.0
  General:
    Drain Data Nodes:  true
    Http Port:         9200
    Monitoring:
    Service Name:          opensearch-cluster
    Set VM Max Map Count:  true
    Vendor:                opensearch
    Version:               2.8.0
  Init Helper:
    Resources:
  Node Pools:
    Component:  cluster-managers
    Disk Size:  100Gi
    Replicas:   3
    Resources:
      Limits:
        Cpu:     1600m
        Memory:  4096Mi
      Requests:
        Cpu:     500m
        Memory:  2048Mi
    Roles:
      cluster_manager
    Component:  data-nodes
    Disk Size:  4096Gi
    Replicas:   3
    Resources:
      Limits:
        Cpu:     2000m
        Memory:  24576Mi
      Requests:
        Cpu:     1000m
        Memory:  24576Mi
    Roles:
      data
  Security:
    Config:
      Admin Credentials Secret:
        Name:  admin-credentials-secret
      Admin Secret:
      Security Config Secret:
        Name:  securityconfig-secret
    Tls:
      Http:
        Ca Secret:
        Generate:  true
        Secret:
      Transport:
        Ca Secret:
        Generate:  true
        Secret:
Status:
  Available Nodes:  6
  Components Status:
    Component:  Restarter
    Status:     Finished
  Health:       yellow
  Initialized:  true
  Phase:        RUNNING
  Version:      2.8.0
Events:
  Type    Reason          Age                 From                     Message
  ----    ------          ----                ----                     -------
  Normal  RollingRestart  53m (x68 over 63m)  containerset-controller  Starting to rolling restart

Here are my Helm values for the OpenSearch cluster:
opensearchCluster:
  enabled: true
  general:
    httpPort: "9200"
    version: 2.8.0
    serviceName: "opensearch-cluster"
    drainDataNodes: true
    setVMMaxMapCount: true
  dashboards:
    enable: true
    replicas: 1
    tls:
      enable: false
      generate: false
    opensearchCredentialsSecret:
      name: admin-credentials-secret
    resources:
      limits:
        memory: "4096Mi"
        cpu: "600m"
      requests:
        memory: "4096Mi"
        cpu: "300m"
  nodePools:
    - component: cluster-managers
      diskSize: "100Gi"
      replicas: 3
      roles:
        - "cluster_manager"
      resources:
        limits:
          memory: "4096Mi"
          cpu: "1600m"
        requests:
          memory: "2048Mi"
          cpu: "500m"
    - component: data-nodes
      diskSize: "4096Gi"
      replicas: 3
      roles:
        - "data"
      resources:
        limits:
          memory: "24576Mi"
          cpu: "2000m"
        requests:
          cpu: "1000m"
          memory: "24576Mi"
  security:
    config:
      adminCredentialsSecret:
        name: admin-credentials-secret
      securityConfigSecret:
        name: securityconfig-secret
    tls:
      transport:
        generate: true
      http:
        generate: true

And here are the Helm values for the operator:
COMPUTED VALUES:
fullnameOverride: ""
installCRDs: true
kubeRbacProxy:
  enable: true
  image:
    repository: gcr.io/kubebuilder/kube-rbac-proxy
    tag: v0.15.0
  livenessProbe:
    failureThreshold: 3
    httpGet:
      path: /healthz
      port: 10443
      scheme: HTTPS
    initialDelaySeconds: 10
    periodSeconds: 15
    successThreshold: 1
    timeoutSeconds: 3
  readinessProbe:
    failureThreshold: 3
    httpGet:
      path: /healthz
      port: 10443
      scheme: HTTPS
    initialDelaySeconds: 10
    periodSeconds: 15
    successThreshold: 1
    timeoutSeconds: 3
  resources:
    limits:
      cpu: 50m
      memory: 50Mi
    requests:
      cpu: 25m
      memory: 25Mi
  securityContext: null
manager:
  dnsBase: cluster.local
  extraEnv: []
  image:
    pullPolicy: Always
    repository: opensearchproject/opensearch-operator
    tag: ""
  imagePullSecrets: []
  livenessProbe:
    failureThreshold: 3
    httpGet:
      path: /healthz
      port: 8081
    initialDelaySeconds: 10
    periodSeconds: 15
    successThreshold: 1
    timeoutSeconds: 3
  loglevel: info
  parallelRecoveryEnabled: true
  readinessProbe:
    failureThreshold: 3
    httpGet:
      path: /readyz
      port: 8081
    initialDelaySeconds: 10
    periodSeconds: 15
    successThreshold: 1
    timeoutSeconds: 3
  resources:
    limits:
      cpu: 350m
      memory: 8192Mi
    requests:
      cpu: 100m
      memory: 1024Mi
  securityContext:
    allowPrivilegeEscalation: false
  watchNamespace: null
nameOverride: ""
nodeSelector: {}
securityContext:
  runAsNonRoot: true
serviceAccount:
  create: true
  name: ""
tolerations: []

Here are some log entries from my operator that repeat every few seconds:
{"level":"info","ts":"2024-04-22T13:00:10.876Z","msg":"Reconciling OpenSearchCluster","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"opensearch-cluster","namespace":"opensearch"},"namespace":"opensearch","name":"opensearch-cluster","reconcileID":"153edb1f-786a-426c-9e88-23569d99696b","cluster":{"name":"opensearch-cluster","namespace":"opensearch"}}
{"level":"info","ts":"2024-04-22T13:00:10.892Z","msg":"Generating certificates","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"opensearch-cluster","namespace":"opensearch"},"namespace":"opensearch","name":"opensearch-cluster","reconcileID":"153edb1f-786a-426c-9e88-23569d99696b","interface":"transport"}
{"level":"info","ts":"2024-04-22T13:00:10.892Z","msg":"Generating certificates","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"opensearch-cluster","namespace":"opensearch"},"namespace":"opensearch","name":"opensearch-cluster","reconcileID":"153edb1f-786a-426c-9e88-23569d99696b","interface":"http"}
{"level":"info","ts":"2024-04-22T13:00:10.894Z","msg":"ServiceMonitor crd not found, skipping deletion","controller":"opensearchcluster","controllerGroup":"opensearch.opster.io","controllerKind":"OpenSearchCluster","OpenSearchCluster":{"name":"opensearch-cluster","namespace":"opensearch"},"namespace":"opensearch","name":"opensearch-cluster","reconcileID":"153edb1f-786a-426c-9e88-23569d99696b"}

I haven't found any errors in the logs for the manager-node or data-node.
After I remove data-node-0 from the exclusion list
curl --insecure -H "Content-type: application/json" -X PUT https://admin:password@opensearch-cluster.opensearch.svc.cluster.local:9200/_cluster/settings -d '{ "transient" : { "cluster.routing.allocation.exclude._name" : null } }'

The unassigned shards slowly being processed.
However, data-node-0 is then re-added to the exclusion list after some seconds.

Hi @hollowdew. Not sure why this is happening. The only operator components that set the exclusion are the restarter and the upgrader. And accoding to your status none of them are doing anything.
Could you please set drainDataNodes to false (the drain should only be needed for emptyDir and since you have no extra persistence config it uses PVCs) and see if that stops the node being added to the exclusion list?

Hello @swoehrl-mw,
I greatly appreciate your suggestion. I have updated my cluster and manually removed data-node0 from the exclusion list once more. I will monitor to see if it gets excluded again and will provide an update tomorrow.
Best regards

Hello @swoehrl-mw,
sorry for that late reply.
It seems to be fixed after setting drainDataNodes to false.
Thank you very much for your help.


python2/3 incompatibility

Writing mypy to oracle_pb2.pyi
Writing mypy to types_pb2.pyi
Traceback (most recent call last):
  File "/home/lpp/go/src/github.com/nmiculinic/x/.venv/bin/protoc-gen-mypy", line 313, in <module>
    main()
  File "/home/lpp/go/src/github.com/nmiculinic/x/.venv/bin/protoc-gen-mypy", line 310, in main
    sys.stdout.write(output)
TypeError: write() argument must be str, not bytes
--mypy_out: protoc-gen-mypy: Plugin failed with status code 1.

python is mapped to: Python 3.6.4

We have encountered a similar thing internally. We're trying to see if anyone has fixed it yet. You're also welcome to try and submit a PR if you can figure this out. In the meantime you can use Python 2 as a workaround.

@nmiculinic Would you be able to test the supposed patch? (Otherwise it'll be a while before I'll get around to it.)

Sure, I'll be able to test it

Yes, it appears to be working!


Add ConvertTrial to SubscriptionService mock

It seems like the changes in the latest PR (https://github.com/blacklightcms/recurly/pull/142) was missing an update to the SubscriptionsService mock to include the new ConvertTrial method.
This was causing tests to fail.

Travis build: https://travis-ci.org/github/blacklightcms/recurly/builds/714985722
@cristiangraz


Add cdc transmission levels

This adds the latest cdc transmission level to the json adn csv output.
Note that this does not add to the timeseries data.  it's not a ton more work, but figured this was the minimum steps to get it working.

Test build with the current cdc transmission levels is this run


Restrict to delete last full access key promted

https://github.com/near/near-cli/issues/834
add prompt Y\n if try to delete last full access key

readline
Author: craig brookes
Description: Simple streaming readline module.
Homepage: https://github.com/maleck13/readline

Createdover 8 years agoLast Updatedabout 4 years agoLicenseBSDMaintainers2Releases14Direct DependenciesKeywordsreadline, line by line and file

README
readline

Read a file line by line.

Install
Important. In node 10 there is a core module named readline. Please use linebyline instead, it is the same module just renamed:
Npm linebyline
npm install linebyline

Test
npm install .
npm test

What's this?
Simple streaming readline module for NodeJS. Reads a file and buffers new lines emitting a line event for each line.
Usage
Simple
  var readline = require('linebyline'),
      rl = readline('./somefile.txt');
  rl.on('line', function(line, lineCount, byteCount) {
    // do something with the line of text
  })
  .on('error', function(e) {
    // something went wrong
  });

ASCII file decoding
As the underlying fs.createReadStream doesn't care about the specific ASCII encoding of the file, an alternative way to decode the file is by telling the readline library to retain buffer and then decoding it using a converter (e.g. iconv-lite).
  var readline = require('linebyline'),
      rl = readline('./file-in-win1251.txt', {
    retainBuffer: true //tell readline to retain buffer
  });
  rl.on("line", function (data,linecount){
    var line = iconv.decode(data, 'win1251');
    // do something with the line of converted text
  });

##API
readLine(readingObject[, options])
Params:

readingObject - file path or stream object
options can include:

maxLineLength - override the default 4K buffer size (lines longer than this will not be read)
retainBuffer - avoid converting to String prior to emitting 'line' event; will pass raw buffer with encoded data to the callback

Return:

EventEmitter

License
BSD © Craig Brookes

New dependencies added: readline.

  Generated by :no_entry_sign: dangerJS against eed38e722b696fbb758e398b372db7d04bbece12

@TarasenkoFirelabs thank you for this contribution! I want to release it ASAP, so I moved it to this PR. Let's continue the discussion there.


Link to .NET Upgrade Assistant from MAUI migration doc

@danroth27 I'm against merging this PR. Linking to a marketing doc from a technical doc is never a good idea. I get that the current migration doc isn't helpful, but it was never intended to be the final doc. It's literally what was published last summer, and will be replaced for GA.

@danroth27 I'm against merging this PR. Linking to a marketing doc from a technical doc is never a good idea. I get that the current migration doc isn't helpful, but it was never intended to be the final doc. It's literally what was published last summer, and will be replaced for GA.

The purpose of adding the link is to point users to instructions on how to actual run the .NET Upgrade Assistant as directed. We could point to https://docs.microsoft.com/dotnet/core/porting/upgrade-assistant-overview instead.


Better default for Tab Close Button on macOS (and maybe Linux??)

Type: Feature Request
On macOS, the Tab Close Button setting should be set to "left" by default. This is more in line with the expected behavior for this OS. Not sure if it applies to Linux as well (maybe it depends on the distro) but if it does the change should be made there as well.
VS Code version: Code 1.83.1 (f1b07bd25dfad64b0167beb15359ae573aecd2cc, 2023-10-10T23:57:32.750Z)
OS version: Darwin arm64 23.0.0
Modes:

https://github.com/microsoft/vscode/pull/196011

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


RotateLB hangs with v6.11.0-beta1

The Enzo-E load-balancing regression test hangs when RotateLB is used.  Some observations:

works fine for +p2 and +p3: no hangs within the 10 balancing steps performed
+p4 runs hang when calling the balancer for the 4th time
+p4 hanging on the 4th attempt is repeatable
hanging behavior does not occur in charm-6.10.2
no other load balancers tested hung

Thanks for the report. Can you provide build information and the command line you used to execute so I can reproduce this?

This is using the solver-dd branch of https://github.com/jobordner/enzo-e.git run using "charmrun +p4 bin/enzo-p input/load-balance-4.in +balancer RotateLB"  I just tried the master branch of the main repository https://github.com/enzo-project/enzo-e.git and that worked fine.  Charm was built using ./build charm++ netlrts-linux-x86_64   -j4  --with-production --enable-tracing.

I've been able to reproduce this, I'm currently trying to get to the bottom of what's going on. It seems like load balancing finishes when the hang happens, objects get their ResumeFromSync functions called, so it might be an issue in Enzo-E.
One question, it seems like the number of Block chares increases with every load balancing period, I assume this is expected because of refinement?

Using the +LBSyncResume flag seems to solve this in my tests, so I think there's some kind of object registration / calling AtSync race that's happening here.

Should be fixed in #3286.


Cannot read property 'onDidChangeContent' of null

Issue Id: 417afc5f-2549-60c9-ee3a-193e8d7adcc9Versions - 1.14.1Stack TypeError: Cannot read property 'onDidChangeContent' of null/vs/workbench/parts/preferences/browser/preferencesRenderers.ts#L84:40 (onDidChangeContent)/vs/base/common/types.ts#L168:6 (apply)/vs/platform/instantiation/common/instantiationService.ts#L162:26 (apply)/vs/platform/instantiation/common/instantiationService.ts#L78:15 (_createInstance)/vs/workbench/parts/preferences/browser/preferencesEditor.ts#L700:39 (createInstance)vs/workbench/electron-browser/vs/base/common/winjs.base.raw.js#L1170:0 (onComplete)vs/workbench/electron-browser/vs/base/common/winjs.base.raw.js#L852:0 (_notify)vs/workbench/electron-browser/vs/base/common/winjs.base.raw.js#L1068:0 (enter)vs/workbench/electron-browser/vs/base/common/winjs.base.raw.js#L1036:0 (_run)vs/workbench/electron-browser/vs/base/common/winjs.base.raw.js#L1813:0 (complete)

Fixed by #32895


fix: add binary for x86 linux in .releaserc

:tada: This PR is included in version 2.2.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Suggested note for Regional Segment doc's

It's not made clear to customers that they cannot transfer workspaces between regions on our doc's.
Suggested Note: Please Note: Transferring workspaces between regions is not supported.
Proposed changes
When investigating ticket: https://segment.zendesk.com/agent/tickets/492849
We found that:
When testing a customer write key, and it appears to have an issue with apiHost setting, which we read to send to the correct API endpoint.
{"integrations":{"Mixpanel (Actions)":{"versionSettings":{"componentTypes":[]}},"AWS S3":{"versionSettings":{"componentTypes":[]}},"Intercom Cloud Mode (Actions)":{"versionSettings":{"componentTypes":[]}},"Segment.io":{"apiKey":"V7XLWTDEYT5CKj2jdgjWdtOiifAkqJHk","unbundledIntegrations":[],"addBundledMetadata":true,"maybeBundledConfigIds":{},"versionSettings":{"version":"4.4.7","componentTypes":["browser"]}}},"plan":{"track":{"__default":{"enabled":true,"integrations":{}}},"identify":{"__default":{"enabled":true}},"group":{"__default":{"enabled":true}}},"edgeFunction":{},"analyticsNextEnabled":true,"middlewareSettings":{},"enabledMiddleware":{},"metrics":{"sampleRate":0.1},"legacyVideoPluginsEnabled":false,"remotePlugins":[]}
Above is the customers workspace settings and it does not appear.
For comparison, here's the workspace we tested on settings:
{"integrations":{"Segment.io":{"apiKey":"8cjl7Z85hryP7JFh8ruAoDsIr850Lm2S","unbundledIntegrations":[],"addBundledMetadata":true,"maybeBundledConfigIds":{},"versionSettings":{"version":"4.4.7","componentTypes":["browser"]},"apiHost":"events.eu1.segmentapis.com/v1"}},"plan":{"track":{"__default":{"enabled":true,"integrations":{}}},"identify":{"__default":{"enabled":true}},"group":{"__default":{"enabled":true}}},"edgeFunction":{},"analyticsNextEnabled":false,"middlewareSettings":{},"enabledMiddleware":{},"metrics":{"sampleRate":0.1,"host":"events.eu1.segmentapis.com/v1"},"legacyVideoPluginsEnabled":false,"remotePlugins":[]}
The issue is related to a transferred workspace. When you transfer a workspace to an EU workspace, then the source becomes unsupported.
If you transferred the source from a regular workspace to EU workspace, then the API host remains the default one (api.segment.io), which means no API host is added to the CDN file to override the US endpoint.
Merge timing

ASAP once approved?

@PcooperSegment approved!


fix: Attach to the console on Windows

This makes std::cout working again after switching to a GUI application in https://github.com/prefix-dev/pixi/pull/2067. It still does not open a console when launched from the Windows Explorer, but when launched from a terminal, it will now print to the terminal again, as before.

I tested on Windows natively (in a shell) and on Linux (in WSL) and it works for me in both.

This seems rather complex for an example. Is this worth having the ability to start the built executable from outside pixi?
Instead of std::cout we could also use https://wiki.libsdl.org/SDL2/SDL_ShowSimpleMessageBox perhaps?

The std::cout is used for error messages if SDL cannot be initialized, so probably we can't use SDL itself to show the error.
Yes, it's quite complex, so maybe not worth doing. This seems to be the right approach to get it fully working on Windows, so I posted it here.
Regarding the ability to start outside of pixi, I would say that depends on what the goal of pixi is: is it supposed to create standalone executables that run on their own, or is the goal to always require pixi to setup the proper environment?

I think the goal of this example is more to show of the task caching and the ablity to build cpp in pixi environments. We don't have to go indepth on these "applications" themself.
Thanks for your input @certik, I'll close the PR for now.


Migrate to strong coplyeft license AGPL

This license preserves core software freedoms.
https://www.gnu.org/philosophy/free-sw.en.html#four-freedoms
Current license does not respect core software freedoms.
License info and comparisons:
https://choosealicense.com/
After a quick glance, the AGPL seems to be compatible with current dependencies.

I have read up on this and I don't think its a good idea to move from MIT to a copy left. If we were to change the license now, we would need to get approval from all contributors, no matter how big or small their contribution has been. Some contributors have only helped through discussions or feedback on Discord and other social media (such as facebook) and to be honest, I haven't kept track of everyone.
The MIT license is very open, which has worked well. Moving to a copyleft license would actually put restrictions on the project, which (to me) is a bit counterproductive from what we want to do, even though I fully understand the pros of using copyleft.
MIT is very popular in open-source projects, I think staying with it makes it easier for new contributors to understand and agree to the terms.

I wanted to thank you for considering and to close the issue, but after writing my reply I figured there is more to discuss.
Foremost: I consider copyleft as less restrictive, since it retains users' software freedoms, which MIT does not.
As for the new users, the most common licenses on F-Droid are of GPL type which are copyleft. Apache follows, and only then MIT. If developers joining is of concern, copyleft is of benefit!
MIT has more benefits for corporate setting when someone wants to close the source to boost their monetization model and prevent competition from gaining useful code.
Switching to copyleft could be achieved via forking and maintaining that as a derivative work, without consent from others. Alternatively dual license, but that sounds complicated.
By the way, we're just having a discussion about licenses over at F-Droid. Even their logo has a 🄯 in it! You're very welcome to join us if you're curious. I'd actually be very interested to have you there so it's not totally biased towards one side - I'd like to know what drawbacks copyleft has for other people's decision-making.


feat: add CreateOrganizationMutation

Refers: opencollective/opencollective/issues/3468
For https://github.com/opencollective/opencollective-frontend/pull/5083
Description: Add website field in the createCollectiveMutation V2

Just leaving a note that I will get to this this morning, apologies for the delay.

@sbinlondon Can you help me look at this. When you get time today

Hi Joy, this is looking good, does the mutation work well with your frontend work?

Yes it works fine both for creating a collective and an organisation
On Wed, 7 Oct 2020, 12:18 pm Kate Beard<EMAIL_ADDRESS>wrote:

Hi Joy, this is looking good, does the mutation work well with your
frontend work?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/opencollective/opencollective-api/pull/4597#issuecomment-704806960,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHDRICEG2PMVDV6SDTLTHW3SJQW6NANCNFSM4RVQ7MBQ
.

Tested it and it works good for me too 👍  Make sure to rebase onto the latest master branch.

👍
On Wed, 7 Oct 2020, 1:52 pm Kate Beard<EMAIL_ADDRESS>wrote:

Tested it and it works good for me too 👍 Make sure to rebase onto the
latest master branch.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/opencollective/opencollective-api/pull/4597#issuecomment-704856331,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHDRICAGB45TMVO4VPSNLV3SJRCAFANCNFSM4RVQ7MBQ
.

Hi @Betree @sbinlondon I need some help with writing mutation. I have created new ones both for edit and create organization but I have thins error Error: Unknown type "OrganizationCreateInput". Did you mean "CollectiveCreateInput", "CommentCreateInput", "CreditCardCreateInput", "ExpenseItemCreateInput", or "OrderCreateInput"? I have created a new OrganizationInput but as thought its not recognised

Hi @Betree @sbinlondon I need some help with writing mutation. I have created new ones both for edit and create organization but I have thins error Error: Unknown type "OrganizationCreateInput". Did you mean "CollectiveCreateInput", "CommentCreateInput", "CreditCardCreateInput", "ExpenseItemCreateInput", or "OrderCreateInput"? I have created a new OrganizationInput but as thought its not recognised

Disregard this. I have fixed it

/cc @sbinlondon Ready for review 🚀


wxWidgets/Qt occasional crash when file open dialog closes

Occasionally get this crash on linux - not consistently reproducible, looks like some issue with correctly initializing wxWidgets
Also strange that it seems to call QtWidgets.QFileDialog instead of the wxWidgets equivalent
Should check if it also happens sometimes on windows
pygame 2.1.2 (SDL 2.0.16, Python 3.9.7)
Hello from the pygame community. https://www.pygame.org/contribute.html
wx._core.wxAssertionError: C++ assertion ""Assert failure"" failed at /tmp/pip-install-9ngm6sal/wxpython_0f8c5d902fc246a9bb48e9905e41957d/ext/wxWidgets/src/gtk/dialog.cpp(196) in EndModal(): either wxDialog:EndModal called twice or ShowModal wasn't called

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/liam/.pyenv/versions/3.9.7/bin/motor_task_prototype", line 8, in <module>
    sys.exit(main())
  File "/home/liam/SSC/motor-task-prototype/src/motor_task_prototype/__main__.py", line 11, in main
    user_choices = mtp.setup()
  File "/home/liam/SSC/motor-task-prototype/src/motor_task_prototype/setup.py", line 122, in setup
    filename = fileOpenDlg(allowed="Psydat files (*.psydat)")
  File "/home/liam/.pyenv/versions/3.9.7/lib/python3.9/site-packages/psychopy/gui/qtgui.py", line 580, in fileOpenDlg
    filesToOpen = QtWidgets.QFileDialog.getOpenFileNames(parent=None,
SystemError: <built-in function getOpenFileNames> returned a result with an error set
1.5493  WARNING         We strongly recommend you activate the PTB sound engine in PsychoPy prefs as the preferred audio engine. Its timing is vastly superior. Your prefs are currently set to use ['sounddevice', 'PTB', 'pyo', 'pygame'] (in that order).

Turns out psychopy uses pyqt5, pyqt4, or wxWidgets in that order of preference.
I thought it used wx by default, so was directly wx using this while psychopy was using pyqt5.
So we should also use pyqt5 and see if that fixes this.
This also suggests that we could remove the wxWidgets dependency from psychopy on CI


Is there an in-place mechanism for converting a single notebook for nbsphinx dev purposes?

Hi, right now it seems like the only way to see the output of nbsphinx on a notebook is indirectly through sphinx. Is there an API or some straightforward workflow for use in the style of nbconvert on a single notebook to see what the comparable html/rst/LaTeX output would be?
One example of a case that seems like too much of a hassle to fight with indirectly is debugging https://github.com/spatialaudio/nbsphinx/issues/82. If there is a more iterative way of interacting with the nbsphinx codebase and to try out these things, then I think I'd be able to contribute here more easily.
If not, no worries, I just wanted to check.

AFAIK, there is no way to run Sphinx for a single source file.
It also doesn't really make sense, since none of the links to or definitions from other files would work.
I don't think it would help in your debugging of #82, since the links wouldn't work anyway.
You can also try other things:

create a brand-new Sphinx project with only a simple index.rst and only one other source file, i.e. the notebook you want to debug.

use nbsphinx.Exporter.from_filename() to get a reST string from your notebook. Note however, that this will disappear with #36.

The links still wouldn't work either way.
For nbsphinx development, I normally use the nbsphinx docs. Most of the supported features are also used in the docs, e.g. the stuff about links from .rst files to .ipynb file is there.
If something is missing in the docs, please let me know.

@mpacer Does that answer your question?
Do you have any more comments?


Uncaught SyntaxError: Failed to execute 'addColorStop' on 'CanvasGradient': The value provided ('rgba(NaN,NaN,NaN,1)') could not be parsed as a color.

[Enter steps to reproduce below:]

...
...

Atom Version: 1.4.0
System: Mac OS X 10.10.5
Thrown From: color-picker package, v2.1.0
Stack Trace
Uncaught SyntaxError: Failed to execute 'addColorStop' on 'CanvasGradient': The value provided ('rgba(NaN,NaN,NaN,1)') could not be parsed as a color.
At /Users/vipul/.atom/packages/color-picker/lib/extensions/Alpha.coffee:103

Error: Failed to execute 'addColorStop' on 'CanvasGradient': The value provided ('rgba(NaN,NaN,NaN,1)') could not be parsed as a color.
    at Error (native)
    at Object._this.canvas.render (/Users/vipul/.atom/packages/color-picker/lib/extensions/Alpha.coffee:103:35)
    at /Users/vipul/.atom/packages/color-picker/lib/extensions/Alpha.coffee:112:29
    at Object.emit (/Users/vipul/.atom/packages/color-picker/lib/modules/Emitter.coffee:11:23)
    at Object.emitColorChanged (/Users/vipul/.atom/packages/color-picker/lib/extensions/Saturation.coffee:24:22)
    at /Users/vipul/.atom/packages/color-picker/lib/extensions/Saturation.coffee:211:61
    at Object.emit (/Users/vipul/.atom/packages/color-picker/lib/modules/Emitter.coffee:11:23)
    at Object.emitSelectionChanged (/Users/vipul/.atom/packages/color-picker/lib/extensions/Saturation.coffee:18:22)
    at Object._this.control.setSelection (/Users/vipul/.atom/packages/color-picker/lib/extensions/Saturation.coffee:200:43)
    at /Users/vipul/.atom/packages/color-picker/lib/extensions/Saturation.coffee:208:30
    at Object.emit (/Users/vipul/.atom/packages/color-picker/lib/modules/Emitter.coffee:11:23)
    at Object.emitInputColor (/Users/vipul/.atom/packages/color-picker/lib/ColorPicker-view.coffee:234:22)
    at Object.open (/Users/vipul/.atom/packages/color-picker/lib/ColorPicker-view.coffee:311:32)
    at atom-text-editor.<anonymous> (/Users/vipul/.atom/packages/color-picker/lib/ColorPicker.coffee:42:68)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:260:29)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:536:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:359:22)
    at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeydown (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:97:36)
    at HTMLDocument.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:3:61)

Commands
     -5:17.4.0 autocomplete-plus:confirm (atom-text-editor.editor.is-focused.autocomplete-active)
     -5:17.4.0 snippets:expand (atom-text-editor.editor.is-focused.autocomplete-active)
     -5:15.2.0 autocomplete-plus:confirm (atom-text-editor.editor.is-focused.autocomplete-active)
     -5:15.2.0 snippets:expand (atom-text-editor.editor.is-focused.autocomplete-active)
  3x -5:13.7.0 core:move-right (atom-text-editor.editor.is-focused)
     -4:52.2.0 editor:newline (atom-text-editor.editor.is-focused)
     -4:50 core:backspace (atom-text-editor.editor.is-focused)
     -4:47.9.0 autocomplete-plus:confirm (atom-text-editor.editor.is-focused.autocomplete-active)
     -4:47.9.0 snippets:expand (atom-text-editor.editor.is-focused.autocomplete-active)
     -4:40.5.0 core:cut (atom-text-editor.editor.is-focused)
     -4:37.7.0 core:paste (atom-text-editor.editor.is-focused)
     -4:15.4.0 core:save (atom-text-editor.editor.is-focused)
  5x -4:08.3.0 core:backspace (atom-text-editor.editor.is-focused.autocomplete-active)
  2x -3:43 color-picker:open (atom-text-editor.editor.is-focused)
     -0:14.8.0 core:backspace (atom-text-editor.editor.is-focused)
     -0:04.0 color-picker:open (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "git-control"
    ],
    "themes": [
      "atom-material-ui",
      "atom-dark-syntax"
    ]
  },
  "color-picker": {
    "triggerKey": "K"
  }
}

Installed Packages
# User
antialiased, v0.1.0
atom-beautify, v0.28.21
atom-html-preview, v0.1.13
atom-jshint, v2.0.0
atom-material-ui, v0.8.0
atom-wrap-in-tag, v0.5.0
autoclose-html, v0.19.0
autocomplete-angular-material, v0.4.0
browser-plus, v0.0.54
color-picker, v2.1.0
css-snippets, v0.9.0
file-icons, v1.6.14
git-plus, v5.12.0
js-refactor, v0.6.0
linter, v1.11.3
linter-eslint, v5.2.7
linter-htmlhint, v1.0.0
minimap, v4.19.0
minimap-bookmarks, v0.2.0
minimap-cursorline, v0.1.0
minimap-find-and-replace, v4.4.0
minimap-git-diff, v4.1.8
minimap-hide, v0.3.0
minimap-highlight-selected, v4.3.1
minimap-linter, v1.1.1
open-in-browser, v0.4.6
open-in-browsers, v0.0.7
pigments, v0.20.0
preview-plus, v1.2.6
regex-tester, v0.2.0

# Dev
No dev packages

Please update color-picker to v2.1.1 and restart Atom. This should be fixed now!


Remind users to assert for unique content on each new page

Closes https://github.com/bullet-train-co/magic_test/issues/44
Tasks

Modal to show reminder to assert for unique content on each new page

Issue with default alert
When subscribing to on page load event and trying to show normal alert box from the browser it throws the error:
Error:
BasicsTest#test_getting_started:
Selenium::WebDriver::Error::UnexpectedAlertOpenError: unexpected alert open: {Alert text : this breaks the page}
  (Session info: chrome=98.0.4758.109)
    test/application_system_test_case.rb:243:in `execute'
    test/system/basics_test.rb:7:in `block in <class:BasicsTest>'

Due to this, I have implemented a simple JS and CSS modal for this feature instead of using alert.
UI Screenshot

@coolprobn This is close, but there's a big issue: The JS fires the reminder even during normal running of system tests, which breaks the entire Bullet Train test suite. Instead, we need to make it so that this new reminder feature is only activated after the test encounters the magic_test command, and it is deactivated right after the user concludes their magic_test session, e.g. in lib/magic_test/support.rb:
    def magic_test
      empty_cache
      @test_lines_written = 0
      begin
        # 👋 Do something here to activate the notification JS.
        magic_test_pry_hook
        binding.pry
        # 👋 Do something here to disable the notification JS.
      rescue
        retry
      end
    end

@coolprobn Also, please make sure the Bullet Train test suite is passing when linked to this branch for it's magic_test dependency in the Gemfile.

@andrewculver I should have checked and tested properly, sorry.
I will fix as required and push new changes soon. Thanks.