use_reactive inside use_memo causes infinite re-renders

Problem
When using use_reactive inside use_memo I get the following error:
Write on signal at C:\Users\matth\.cargo\git\checkouts\dioxus-1e619ce595d3799d\74352f2\packages\hooks\src\use_reactive.rs:96:20 happened while a component was running. Writing to signals during a render can cause infinite rerenders when you read the same signal in the component. Consider writing to the signal in an effect, future, or event handler if possible.

I'm not using any signals inside the closure so I wouldn't expect this to happen 🤔 I'm not sure if use_memo uses signals internally though, that could be the cause.

Long error
```
2024-05-07T18:43:08.595108Z  WARN recompute: dioxus_signals::signal: Write on signal at C:\Users\matth\.cargo\git\checkouts\dioxus-1e619ce595d3799d\74352f2\packages\signals\src\memo.rs:98:18 finished in ReactiveContext(for scope: ScopeId(1, "ColorPicker")) which is also subscribed to the signal. This will likely cause an infinite loop. When the write finishes, ReactiveContext(for scope: ScopeId(1, "ColorPicker")) will rerun which may cause the write to be rerun again.
HINT:
This issue is caused by reading and writing to the same signal in a reactive scope. Components, effects, memos, and resources each have their own a reactive scopes. Reactive scopes rerun when any signal you read inside of them are changed. If you read and write to the same signal in the same scope, the write will cause the scope to rerun and trigger the write again. This can cause an infinite loop.

You can fix the issue by either:
1) Splitting up your state and Writing, reading to different signals:

For example, you could change this broken code:

#[derive(Clone, Copy)]
struct Counts {
    count1: i32,
    count2: i32,
}

fn app() -> Element {
    let mut counts = use_signal(|| Counts { count1: 0, count2: 0 });

    use_effect(move || {
        // This effect both reads and writes to counts
        counts.write().count1 = counts().count2;
    })
}

Into this working code:

fn app() -> Element {
    let mut count1 = use_signal(|| 0);
    let mut count2 = use_signal(|| 0);

    use_effect(move || {
        count1.write(count2());
    });
}
2) Reading and Writing to the same signal in different scopes:

For example, you could change this broken code:

fn app() -> Element {
    let mut count = use_signal(|| 0);

    use_effect(move || {
        // This effect both reads and writes to count
        println!("{}", count());
        count.write(count());
    });
}

To this working code:

fn app() -> Element {
    let mut count = use_signal(|| 0);

    use_effect(move || {
        count.write(count());
    });
    use_effect(move || {
        println!("{}", count());
    });
}
```

Steps To Reproduce
Steps to reproduce the behavior:
Create a use_memo(use_reactive(..)) using a closure without signals.
Expected behavior
This error message is great! I just think my case shouldn't be an error, so hopefully just a small bug with use_memo.
Environment:

Dioxus version: Master  74352f2
Rust version: Nightly
OS info: Windows
App platform: Custom

Questionnaire

[x] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

The error message is a warning, but I don't think it should cause an issue here. We should introduce a way to manually opt out of that warning for a hook. This could be a closure that sets a flag to disable the warning

Ah man I was causing a render loop through some code I didn't see, this error message is great!


Support loading Braintree from AMD modules

Description
The PR contains the needed modification to load the Braintree module either way, from the global window context or from the AMD module library. Meaning if in the global window scope the keys define and define.amd are defined the Braintree won't be accessible through the global window.braintree. Instead will be accessible via require module from an AMD library.
Why are we making these changes?
The Braintree script is supporting AMD modules, and that introduces a bug in the react-paypal-js repo. The main reason when this condition matches on the browser: typeof define === "function" && define.amd the Braintree script will be defined as an AMD module, not in the global context of the browser.
Please see this Jira ticket.
Issue
This PR contains the fixes for this issue.

We don't need to support AMD modules on this repo.


[backport] crosssdk is droped in oe-core

fixes https://github.com/OE4T/meta-tegra/issues/1246

Thanks!


Rewrite /cachegroupparameters/{{cachegroupID}}/{{parameterID}} to Go

I'm submitting an ...
improvement request (usability, performance, tech debt, etc.)
Traffic Control components affected ...

Documentation (possibly)
Traffic Ops

Current behavior:
/cachegroupparameters/{{cachegroupID}}/{{parameterID}} is handled by Perl
Expected / new behavior:
/cachegroupparameters/{{cachegroupID}}/{{parameterID}} should be handled by Go.
Minimum API Version
1.1
Request Methods

[ ] DELETE

Minimal reproduction of the problem with instructions:
N/A

i think this should be deprecated (not rewritten to Go and a deprecation notice added to the response) because not sure it's used anywhere nor do cachegroup params have any effect.


Fix(php-cs-fixer) Fix php-cs-fixer to v3.45 and apply code fixes

@thirsch @alquerci can rebase your ACPu PR, as this PR shoukd fix the CS errors :)


Document data retention, task and service APIs, and calculation methods

Please merge these changes into the 2.6 branch as well!
Due to breaking changes between versions and the fact that Accounting report docs for 2.7+ are in the https://github.com/nickjameswebb/docs-operating-pas repo, we are creating 4 separate PRs.
Please message me if you have any questions or need to pair on merging! Thanks!

Merged. Thank you, @nickjameswebb !


Can't python setup.py install zipline

Fails with "AttributeError: 'dict' object has no attribute 'NUMPY_SETUP'"
As reported by @easytrader in https://github.com/quantopian/zipline/issues/804#issuecomment-153539093
I was able to reproduce this on OSX.

@easytrader, are you able to install your checkout of master with pip install .?

@richafrank , I can build the branch. I give it up. I want to wait formal release zipline=0.80,haha.

I think this is because we are using some setuptools features. ping @ssanderson

I think this is an issue with bcolz, one of our dependencies.  I can reproduce this without doing any of the unusual things in our setup.py: https://github.com/Blosc/bcolz/issues/277.

there is also some issues with BColz and this np.view(int) code in the test suite as well. I will create a separate issue this evening for it.

This should be fixed now by that upstream bcolz issue.  Please re-open if you find that's not the case.


refactor(button): change div to span (#16350)

Div is categorized by HTML5 spec as flow content.
Button's permitted content is only phrasing content.

After presubmitting this through existing Google apps, we found it's a bit too breaking. This will be addressed in the future as part of switching to MDC-based components.


basemodel.label() Different types of masks overlap

Search before asking

[X] I have searched the Autodistill issues and found no similar feature requests.

Question
basemodel:GroundedSAM .Generate seg label file using the "basemodel.label()" command.
the label file as follows:

I want the two categories to not overlap, what should I do
Additional
No response

Hi @6yuhao6 👋🏻 ! Is that question only about this specific image or in general?

Hi @6yuhao6 👋🏻 ! Is that question only about this specific image or in general?

on my dataset is general.  Maybe it is because the target box generated overlaps, i guess.

Yes. That's exactly the reason. We could still apply some post processing - try to cat to area of one mask from the other.


[GHSA-7c44-7j7v-w554] Buildkite Elastic CI for AWS symbolic link following vulnerability

Updates

Affected products

Comments
Based on the info from the NVD site linked in the GHSA "versions prior to 6.7.1 and 5.22.5" are affected. The table at the bottom of the NVD site also implies that it should be versions >= 6.0.0, < 6.7.1
I made the same suggestion for https://github.com/github/advisory-database/pull/3264

👋 This pull request has been marked as stale because it has been open with no activity.  If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.

👋 This pull request has been marked as stale because it has been open with no activity.  If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.


We are facing a deprecation warning of "warning: [deprecation] Registrar in PluginRegistry has been deprecated import io.flutter.plugin.common.PluginRegistry.Registrar;"

In the latest Flutter version. We are facing errors from zsdk package. Can you help, to resolve it

But a warning is not an error... from the title of this issue I can not understand what is the problem you are facing?


[Coverity CID :187064] Control flow issues in /subsys/bluetooth/host/mesh/cfg_srv.c

Static code scan issues seen in File: /subsys/bluetooth/host/mesh/cfg_srv.c
Category: Control flow issues
Function: bt_mesh_label_uuid_get
Component: Bluetooth
CID: 187064
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

Marked as false positive in coverity (see the the issue there for a more detailed explanation)


Various improvements & upgrade ggml

Notable changes:

rwkv.cpp file version was bumped to 101
Q4_2 format was removed
all other quantized formats were changed in a backwards-incompatible way, older files can not be loaded anymore

@LoganDark Maybe want to take a look before I merge? :)

looks like you ran an auto-formatter that messed up some of the assert statements. was that intended?

@LoganDark It's intended, to keep the line not too long. Did asserts break because of this?.. Code compiled and tests passed, looks like they still work

@LoganDark It's intended, to keep the line not too long.

some of the asserts get long but should still be in only 1 line. An example is if they are in the middle of a list of other one-line asserts and splitting it into a block would be very not pretty. I think the max length should be set higher

if they are in the middle of a list of other one-line asserts and splitting it into a block would be very not pretty

I agree that there is value in symmetry of having lined-up single line statements. For me tho, it's okay to trade it off for the possibility of having all the code visible on the screen without the need to scroll long lines.

add a workflow to check code formatting

That's a good idea, I've added it to my backlog.

For me tho, it's okay to trade it off for the possibility of having all the code visible on the screen without the need to scroll long lines.

I don't think the lines are so long that becomes an issue, but  whatever I suppose

I forgot about the new Q8_1 ;D luckily #74 includes it


wrong links redirected to home page

I have added a .htaccess file which would redirect any wrong links to home page
link to the issue

@patelaryan7751 as the issue is of Level 2, kindly make a 404 not found page. If a user visits a wrong link, redirect it to the 404 not found page, and add a button to redirect it to the home page.

okay

@Harshal0902 I added a 404 page with the above said changes.


New Software Center not supported on Windows Server Core

from 519672
Need to add a note that the new Software Center app isn't supported on any version of Windows Server Core

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

ID: f86d400f-d2a6-d50b-8d1f-18893af56d67
Version Independent ID: 793a7196-8549-329c-bad5-c6b33f466ae7
Content: Supported clients and devices - Configuration Manager
Content Source: sccm/core/plan-design/configs/supported-operating-systems-for-clients-and-devices.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Fixed with linked internal pull request. Once merged, this change should be live with the next daily publishing process. Right now that should be tomorrow (Wednesday, 22 August) around 11:00 Pacific time. I'm closing this issue.


Can't scroll left or right inside the code example boxes

In /docs/user-guide/kubectl/kubectl_expose/ I can't scroll left or right inside the code example boxes.
Using Chrome 50.0.2661.86 (64-bit) on Ubuntu 14.04

I'm seeing the same thing in Chrome on Windows. It appears to be caused by the CSS for the headers.

I don't understand the purpose of this CSS in the first place. The offending two lines are here:
https://github.com/kubernetes/kubernetes.github.io/blob/master/_sass/_base.sass#L713
margin-top: -$header-clearance
height: $header-clearance

Please delete those and find a better solution for whatever problem they are supposed to solve. In any case they introduce new breakage.


Text not rendering correctly in the tour example.

Is there an existing issue for this?

[X] I have searched the existing issues.

Is this issue related to iced?

[X] My hardware is compatible and my graphics drivers are up-to-date.

What happened?

What is the expected behavior?
The text should not be distorted.
system info:
Device name	DESKTOP-EP4LQDE
Processor	Intel(R) Core(TM) i5-10400F CPU @ 2.90GHz   2.90 GHz
Installed RAM	32.0 GB (31.8 GB usable)
Device ID	A2B41964-60C8-49B9-B43B-78C3635331BA
Product ID	00330-80000-00000-AA231
System type	64-bit operating system, x64-based processor
Pen and touch	No pen or touch input is available for this display
Edition	Windows 10 Pro
Version	22H2
Installed on	11/6/2023
OS build	19045.3636
Experience	Windows Feature Experience Pack 1000.19053.1000.0

GPU = nvidia gtx 1650 driver version 536.23
cargo 1.73.0 (9c4383fb5 2023-08-26)
MSVC v143

code ran from iced master repository.
Version
crates.io release
Operating System
Windows
Do you have any log output?
No response

issue only present in debug build

I tested this as well and yes the text bugs out when you start to resize the text output.

Issue is caused by a recent patch to etagere: https://github.com/nical/etagere/pull/27#issuecomment-1806686983
You can fix it by manually downgrading the dependency to 0.2.8:
cargo update --package etagere --precise 0.2.8

The affected etagere version has been yanked; the issue should be fixed.


Merge in upstream fix for middleware

Merge in upstream fix for middleware

Coverage increased (+0.2%) to 95.878% when pulling 4f6fc3f2eb9fdfae4e81eb659af5d3c96c420fe1 on warnickr:master into 9ed3884eaddb74acaf790d315822ebaa09e1238b on davidakachaos:fix_middleware_breaking.


What must be changed, if you have the newest Version W10 20H2 x64 (AMD)?

[Hier Feedback eingeben]

Dokumentdetails
⚠ Bearbeiten Sie diesen Abschnitt nicht. Er ist für die Verknüpfung von docs.microsoft.com zum GitHub-Artikel erforderlich.

ID: c12e9c4a-af05-9ecd-a791-ac89aa43787c
Version Independent ID: 76517523-a960-c5fb-ce76-2c3efe642f37
Content: Windows Defender von Device Guard und Windows Defender Credential Guard - Microsoft 365 Security
Content Source: windows/security/identity-protection/credential-guard/dg-readiness-tool.md
Product: w10
GitHub Login: @SteveSyfuhs
Microsoft Alias: stsyfuhs

Page https://docs.microsoft.com/de-de/windows/security/identity-protection/credential-guard/dg-readiness-tool
Script
If I tested my version (with an active DG/CG), there was printed a reference to "DG_Readiness.ps1 -Disable"
Where ist a real reference to this routine?
Goal: Using Bluestack with hardware-virtualiser is only possible without windows hyper-v (deaktivatet feature, but it seems to work in Background trough DG/DC
Because the machine has many users, working with regedit seems to be a small "worst case".
Thanks

Hello @DcPSDcPS ,
Have you seen this article?
Thank you

@DcPSDcPS We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific content change you can suggest to improve the docs. Thank you.


Create valuesOfType operator and use it in the test code

Unfortunately this breaks some things with evidence tracking / justification. There is already a lot of know issues with the justification process, and there is a plan to fix it that goes well beyond the scope of this PR. Just be cautious if you are relying on evidence tracking / justification, because the behavior between factsOfType(x).map(something(_.value)) can be different than valuesOfType(x).map(something(_))


Enable passing options to babel

Adds ability to pass options to babel, useful for plugins etc.

Hey @vlascik, this is great, thanks! I left a comment

@pswai where did you leave the comment? can't see anything except this one :/
Anyway, saw that you bumped node to v4, thanks, merged it here so that the CI would pass.

@vlascik Ah i forgot to submit the review. Ok this looks good now. Merging.


Add missing set command to JedisCluster.

Fixes #999

This PR applies only to 3.0. For 2.7 and 2.8 I'll provide a different PR.

@marcosnils We already have #878 for reviewing, and when we merge #878 this PR becomes invalid.
So I'd like you to review #878 first and make a decision which one we choose. :)

@HeartSaVioR I forgot about #878. You're right.


fix(bootstrapPort): Update the crd conversion webhook port on osm-bootstrap

Description:
Update the crd conversion webhook port on osm-bootstrap  to 9443 as port 443
cannot be used.
Reason : k8s fails to open the port 443 because it is a privileged port. All ports <1024 require special permissions, hence th eport was updated to 9443. An alternative is to keep 443 and update the target port, the former was chose for consistency.
Fixes #4178
Signed-off-by: Sneha Chhabria<EMAIL_ADDRESS>Testing done: Manually verified that the error message as described in issue 4178 no longer appears
Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Did you notify the maintainers and provide attribution?

Is this a breaking change? no

Codecov Report

Merging #4194 (531b7f2) into main (5c9b998) will increase coverage by 0.06%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #4194      +/-   ##
==========================================
+ Coverage   69.83%   69.90%   +0.06%
==========================================
  Files         212      212
  Lines       11623    11623
==========================================
+ Hits         8117     8125       +8
+ Misses       3454     3446       -8
  Partials       52       52

Flag
Coverage Δ

unittests
69.90% <0.00%> (+0.06%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cmd/osm-bootstrap/osm-bootstrap.go
47.82% <0.00%> (ø)

pkg/crdconversion/crdconversion.go
75.51% <0.00%> (+8.16%)
:arrow_up:

Continue to review full report at Codecov.

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

Is there an e2e we need to update or an e2e we can update?

Not at the moment, when we actually have a new crd version we can add a new e2e and update the upgrade test


GH-4527 use lowercase name for config class name and use full names for inner classes

GitHub issue resolved: #4527
Briefly describe the changes proposed in this PR:

PR Author Checklist (see the contributor guidelines for more details):

[ ] my pull request is self-contained
[ ] I've added tests for the changes I made
[ ] I've applied code formatting (you can use mvn process-resources to format from the command line)
[ ] I've squashed my commits where necessary
[ ] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change

@abrokenjester I was looking at the new CONFIG class that you introduced and have some suggestions for changes.
I propose we use Config instead of CONFIG. I haven't looked into why you went with CONFIG to begin with, but I was thinking that it breaks with our current naming conventions and I wasn't sure why.
I also noticed that there are some new abbreviations that I think will be confusing to users. I propose using the class names for the classes that the configs apply to. So Config.CustomGraphQueryInferencer instead of Config.Cgqi. I see that it messes a bit with the IntelliJ autocomplete, so maybe we could go with a suffix of sorts?

Since the old config classes are probably widely in use by users I think it might be a good idea to not remove them in 5.0.0 and leave them around until 6.0.0 so that users have extra long time to migrate their code.

Not sure why the PR verify is failing because the xml-format plugin can't be found.

@abrokenjester I was looking at the new CONFIG class that you introduced and have some suggestions for changes.
I propose we use Config instead of CONFIG. I haven't looked into why you went with CONFIG to begin with, but I was thinking that it breaks with our current naming conventions and I wasn't sure why.

We discussed this when I put up the original change - see https://www.eclipse.org/lists/rdf4j-dev/msg02136.html. The consensus was that we'd stick with all capitals because although that breaks with our general naming conventions, it aligns it with the convention that we use for all other vocabulary classes. They're all named for their prefix name, in all caps.

I also noticed that there are some new abbreviations that I think will be confusing to users. I propose using the class names for the classes that the configs apply to. So Config.CustomGraphQueryInferencer instead of Config.Cgqi. I see that it messes a bit with the IntelliJ autocomplete, so maybe we could go with a suffix of sorts?

Two things:

the abbreviations align with what is used in the actual URIs - so ideally if you want to change these abbreviations you'll have to adapt the actual URIs as well (and make sure that all templates are adapted, too);
I chose the abbreviations based on brevity, both from a Java point of view and for the actual RDF vocabulary itself. I agree that some, like Cgqi and Ca, are not the most intuitive, so happy to discuss alternatives, but I'd rather not go with the full class name as that makes it all very verbose.

Sorry for not following along when this was being discussed.
The main issue with the abbreviations is that they are really hard for users to discover. They have to read the Java docs to understand that the config for the ElasticsearchStore is now in ess.
I really believe we need to help users with discoverability. Maybe we could add a constant to all the main classes that point to the relevant vocabulary. For instance ElasticsearchStore.CONFIG would be equal to CONFIG.ess, and that way we would still maintain the vocabulary in a single location without any duplication.

The main issue with the abbreviations is that they are really hard for users to discover. They have to read the Java docs to understand that the config for the ElasticsearchStore is now in ess.

YMMV, but I don't find that "hard to discover". The IDE warns if a class or field is deprecated when you use it, as soon as you hover over it the javadoc tells you where to look for the replacement.
That said, anything we can do to make it even easier is welcome of course :)

I really believe we need to help users with discoverability. Maybe we could add a constant to all the main classes that point to the relevant vocabulary. For instance ElasticsearchStore.CONFIG would be equal to CONFIG.ess, and that way we would still maintain the vocabulary in a single location without any duplication.

I like this idea.

I think I'll just leave things as they are here and instead look into adding some way to link to the relevant vocabulary from within the different sails and repos. Hopefully it's as easy as introducing a constant.
I'll think about the abbreviations for a bit and see if I can come up with anything that is cleared without being too long.


Spacing adjustments on account-abstraction policies form

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR focuses on enhancing the layout and readability of the SponsorshipPolicies component in the AccountAbstractionSettingsPage by adjusting the spacing in flex containers and modifying text formatting for better clarity.
Detailed summary

Updated div elements to include gap-6 lg:gap-12 for improved spacing.
Changed text formatting to use <br /> for line breaks instead of plain text.
Made similar adjustments across multiple sections, including global spend limits and restrict to specific chains.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#4973  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MananTank and the rest of your teammates on  Graphite


spacebar: switch to apple-sdk_12; move to by-name

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353749

x86_64-linux

aarch64-linux

x86_64-darwin

:white_check_mark: 1 package built:

spacebar

aarch64-darwin

:white_check_mark: 1 package built:

spacebar


Missing cursor types on macos

I noticed there are some cursor types missing on macOS:
SizeAll
SizeTopLeftBottomRight
SizeBottomLeftTopRight
Based on some quick research, it seems they just aren't exposed or documented by apple, which is helpful.... but this SO answer suggests a solution which appears to work for me, implemented here
I should mention I'm not really experienced with objective-C at all, so apologies if that code is bad in any way... If not, I can PR whenever you'd like

Would probably be worth a try. What happens if you try to pass an invalid value (let's assume something is changed later)?

If I change [NSCursor _________] to use a non-existent value it compiles fine but then get SIGABRT at runtime.
Following the SO article again, checking if ([NSCursor respondsToSelector:@selector(______)]) would stop the SIGABRT, and could just return false from loadFromSystem(), which would match the current behaviour?
I'm using high Sierra 10.13.4 and Xcode 9.2 (9C40b), for the record.

Sounds reasonable. My company Mac isn't up to date, so can't really check at the moment.

What happens on minimum supported macOS versions?

but this SO answer suggests a solution which appears to work for me, implemented here

Interesting! This information wasn't available to me when #827 was being worked on.

What happens on minimum supported macOS versions?

Nothing. This would be an addition based on non-documented API from Apple so we do not enforce users to have a system supporting those cursors. Instead, the users can only be certain at runtime that the cursor exists. The documentation should therefore be very clear on that.
Implementation wise, a patch along those lines (and proper formatting) would be good I think:
// adding a private function to anonymous namespace
bool loadFromSelector(SEL selector, NSCursorRef& cursor) {
  if ([NSCursor respondsToSelector:selector]) { cursor = [NSCursor performSelector:selector]; return true; }
  else return false;
}

...
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wundeclared-selector"
switch (type) {
  case SizeBottomLeftTopRight: return loadFromSelector(@selector(_windowResizeNorthEastSouthWestCursor), m_cursor);
  // similarly for others...
}
#pragma clang diagnostic pop

@JonnyPtn, would you like to submit a PR? :)

Sure thing, I’m skiing this week so might not get round to it for a few days

Superseded by #1425.


Adding synonyms to exposure receptor

Adding the synonyms exposure target and exposure recipient to this term per a request from an ECTO user.
https://github.com/EnvironmentOntology/environmental-exposure-ontology/issues/165

@matentzn I've included the terms only as exact synonyms here per your statement in the previous PR. Also, there are just no other options based on the imports.
I don't know what the binary file changes are that the issue is stating I've made, I only made edits in protege so any thoughts on that would be helpful. Thanks!


Makes xz decompression an optional dependency in python<3.3

xz decompression in Python<3.3 requires liblzma which won't be available on every machine. Move xz decompression option to optional dependency (pip install smart_open[xz]) instead of having install fail when liblzma isn't available.

Currently travis doesn't test optional dependency. Happy to figure out how to add this if this looks promising.

@tdhopper Thank you for the pull request. It works as advertised, but I don't like the extra effort we're applying to integrate .xz support into smart_open. In particular, voodoo around import statements and Python versions, as well as explaining things like pip install smart_open[xz] seem like a step backwards to me.
Are you able to move the xz support from the library to the README? It's basically a copy-paste from here to here. You can replace the existing LZMA example in the README with the working cope from smart_open_lib.py.
You're a first-time contributor, so if it's too much, let me know and I'll deal with it on the weekend.

@mpenkov I'm willing. It seemed odd to me to rip out functionality that's already been released, but that's not my call.

For the record, here are my reasons for moving .xz support from code to examples:

Adding support for .xz (or .lzma or any other compression format) is trivial (it's like 5 lines of code).
Integration-like stuff like this is better handled outside the library. Supporting every compression format out there well out-of-the-box isn't our goal. We make exceptions for gzip and bz2, because they are so common.
The existing LZMA support is only partial. The lzma library has options that the current implementation plain ignores.
Finally, the LZMA stuff was introduced in a recent minor release, it's not too late to quietly find a better place for it

So, please go ahead.
When adding the example to the README, please check that it passes doctests. We already have test .xz data in the repo, so please use that.

@mpenkov Do you have examples doctests? I don't see one for this.

Yeah, that snippet is untested, because it's a hand-wavy example (as opposed to a real example, see above in that same file).
The actual test is here. You should be able to pretty much copy-paste it into the README.


Enemies can walk through towers

There's a bit of a question as to how this should be handled. Enemies can't be totally blocked from passing squares with towers, or else the player can simply wall off the map and prevent them from ever reaching the goal. My first intuition is to allow them to climb over, basically meaning that a tower on a square adds to its height.

That's an interesting option. For that one we might want to have a minimum range on the towers. It would be odd if a tower could throw a rock at itself for example.
Some other options to consider:

We prevent the user from being able to place a tower that would create a wall. This might be the easiest.
The enemies can destroy towers that block.

With regards to blocking the goal, there's a similar issue with terraforming. I'm not sure exactly what happens now when there's a hole in the map, but I recall we had discussed having a base layer beyond which you can't dig further.

Minimum range and climbing over towers have been implemented, open to further discussion after POC


Create Sprite for Cement Tile #301

I tried to made it seamless for tiles purpose.
If you think it is too gradient, let me know~

Hi @oktak! This will work! Thank you. :)


[Bug]: NgxEditor does not auto populate with content when used in a NgFor

What happened?
Hi, I have a simple use case where I want to Auto Populate content in my NgxEditor but the editor is used in a NgFor loop and it only auto-populates the last instance of the editor among all.
I tried auto-populating with ngModel and even tried using Reactive Forms but none of them were able to fulfil my use case.
I have multiple Mat Expansion Panels in which I have NgxEditor which should be auto populated in any matter. I'm trying to update my NgModel binded variable everytime I open my Mat Expansion Panel, but it seems to be not working. Below is the relevant code.
<mat-expansion-panel (opened)="panelOpenState = true" (closed)="panelOpenState = false" *ngFor="let item of elementDetails" (opened)="onPanelOpen($event, item)">

      <ng-container *ngIf="inp.type === 'rich_textarea'">

            {{ inp.name }}

              <ngx-editor-menu [editor]="editor" [toolbar]="toolbar" [(ngModel)]="html"></ngx-editor-menu>
              <ngx-editor [editor]="editor"></ngx-editor>

      </ng-container>

</mat-expansion-panel>

onPanelOpen(event: any, item: any): void{
    this.typeId = item.content_type;

    if(this.typeId === 2){
      this.html = item.content_details['content'];
    } else if(this.typeId === 3){
      this.html = item.content_details['help_info'];
    }
}

Here I'm updating my NgModeled bounded variable every time with it's new value in the onPanelOpen method. Mat Expansion panels have NgxEditor in it but it is not able to auto populate itself with the content. It is only able to display content of the last panel's Editor.
Can anyone help me here?

Version
v15.3.0
Angular Version
v14.3.0
What browsers are you seeing the problem on?
Chrome
Link to reproduce
No response
Relevant log output
No response
Willing to submit a PR?
None

Closing this issue as I've got a workaround for my use case.
SOLN:
I created an object of objects with initialising "new Editor()" in each object.


Create content card 13 blue

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

After building the component please raise a PR with a screenshot of the component and add the component in Content Cards/src/content-card-13-blue.
If you need to use any icon please use it from Hero Icons
Good luck.

I would like to work on this.

I would like to work on this.

I would like to work on this

@Tanmay09516 WOC will be over soon, so are you still working on it otherwise I can assign it to @SakshiShah29

I'm also willing to do this task
@RavindraP04

@RavindraP04 yes you can apologies i have been busy, you can do it for all tasks I have taken..

Assigned @Saishreekouda

I would like to work on this

@RavindraP04 I would like to work on it.

@Saishreekouda Are you still working on it ?

@RavindraP04 This issue must be closed as  content-card-13-blue.html is present in the code base.


make EmbeddedKafkaBroker.BROKER_LIST_PROPERTY take precedence to default one

As per the javadoc:

Instead of default spring.embedded.kafka.brokers system property, the address of the Kafka brokers can be exposed to any arbitrary and convenient property. For this purpose a spring.embedded.kafka.brokers.property (EmbeddedKafkaBroker.BROKER_LIST_PROPERTY) system property can be set before starting an embedded Kafka.

Literally it seems to imply when spring.embedded.kafka.brokers.property  is set, the default spring.embedded.kafka.brokers system property should not be set; but it seems regardless, the default is still set
Not 100% sure about my code changes and even my understanding is correct.

I agree about closing this as Works as Designed.
The motivation is indeed was to always have that default exposed independently of spring.embedded.kafka.brokers.property setting.

Yeah, later on I realized that. Though it is a little bit confusing for the
doc mentioned "instead of...", but it seems the reality is "in addition
to...", for the default one is always there. But anyway.
On Mon, Nov 13, 2023, 11:22 a.m. Artem Bilan @.***>
wrote:

I agree about closing this as Works as Designed.
The motivation is indeed was to always have that default exposed
independently of spring.embedded.kafka.brokers.property setting.
—
Reply to this email directly, view it on GitHub
https://github.com/spring-projects/spring-kafka/pull/2892#issuecomment-1808494985,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6UYAX7CW6OFLUMLPTR4U3YEJCLVAVCNFSM6AAAAAA7G6V5UKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBYGQ4TIOJYGU
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Missing videos in autograd docs page

The beta docs page for autograd - http://beta.mxnet.io/guide/packages/autograd/autograd.html has a couple of embedded videos. But the videos are blank.
cc: @aaronmarkham

@mxnet-label-bot add [Doc, Website]

Where is the source for this?

@larroy I think this is the source for the beta website - https://github.com/mli/new-docs

So the trick here is to move the images or videos (these videos are tiny, so they'll count as images) over to the static folder. I did something similar here: https://github.com/apache/incubator-mxnet/pull/15205/files#diff-dd9fe9390d558c60e8a5794696784ec6
I'm back from vacation so I can fix this later today unless one of you were already working on this.


x/pkgsite: package removal request for github.com/bernardbaker

What is the path of the package that you would like to have removed?
github.com/bernardbaker/qiba.core
Are you the owner of this package?
I am the owner.
What is the reason that you could not retract this package instead?
Accidental published all the files and a plain text password is listed.

This is done. Please allow a few hours to propagate.

Thank you @findleyr ⭐


Publish to gh-pages #278 - Service Workers relative path

Hi,
Here is a PR for #278.  Publish to gh-pages - service worker relative path.

Thanks, I'll just do a sanity check with a cordova build and merge it

@sinedied
Do you know what could have caused the below?
yarn run v1.5.1
$ ng lint --type-check && stylelint "src/**/*.scss" --syntax scss && htmlhint "src" --config .htmlhintrc
'ng' is not recognized as an internal or external command,
operable program or batch file.
error An unexpected error occurred: "Command failed.

@reflexdemon That was an issue fixed with another PR, it should be ok now that I updated the branch.
Before merging I want to make sure this works with cordova builds, but when I tested with Android I had an (unrelated) issue that I need to fix first, so it's taking a bit more time 😕

Thanks for response. I was curious to see if I could be of any help here.

Just tested on mobile, it does not change anything, so it's good to merge :)


Live search revamp: only use search results page for users without JS

MR here to address this: https://github.com/wagtail/guide/pull/161

@nicklee I couldn’t see the loading indicator on #161, and the results page still seems to be visible. Did you intend to take care of those too?

I've left the search page in for no-js users; the search button appears if no js, which then takes the user to the standalone search page. (Note: the search does show in a modal, and i'm not 100% about how BS modal handels no-js).
I didn't add a loading indicator, I simply moved the "* results" message into the input and improved it's responsiveness so as users type they get an immediate response which makes it obvious what is happening.
I did consider removing the input's browser autocomplete as this can conseal the initial area of the actual js autocomplete, but in the conceed that this could be useful to some users.

No longer as relevant now that the search has more robust "partial match" autocomplete behavior.


how to use

Sory,wrong title . How to use the top-right button  to dissmiss the keyboard?

Hi ZsIsMe, do you mean the backspace key? Custom layouts are not supported on MMNumberKeyboard  but you can customize some of the keys—like the return key and the special key (which does what you intend by default).
To learn more about customization, please refer to the documentation included in MMNumberKeyboard.h.
The built-in functionality already fulfils your dismissing needs, so I'm closing this issue. 😃
Cheers,
Matías.


add default timeout

for more info, see https://github.com/psf/requests/issues/3070

seem like the pull request is unable to access the api_key

timeout should be an argument in line https://github.com/zehengl/ezapi-tmdb/blob/master/tmdb/three/init.py#L75 for example

Sorry, my mistake.


Option to use AWS KMS to store VSecM safe master key (instead of a kubernetes secret)

AEGIS_CRYPTO_KEY_STORE = [k8s, kms, vault, none]
AEGIS_CRYPTO_KEY_AUTH_METHOD = 'iam'
if key store is kms, and if auth method is IAM then assume that you have access to what you need and use kms as a backing store for the root key.
Note that this is only for the root key.
When set up like this, if a key exists in KMS, Aegis will use it during bootstrap (if it is set up to automatically bootstrap); or it will generate and store it to the KMS (to be used next time)
If it is set up to bootstrap manually using AEGIS_MANUAL_KEY_INPUT, then it won’t read from KMS, and will expect the operator to manually set up tke key instead.

@v0lkan Could you assign that issue to me?

Assigned; I’ll ad some details to this.


Extras can be dropped if different dependencies require different sets

Say A requires B[x], but C requires B[y], and D requires B (no extra), there is no guarantee which one will be present in the lock file. Some consolidation logic is required to merge extras.
(I found this because Passa depends on Plette[validation], but RequirementsLib depends on Plette without specifying the extra, and now the Pipfile.lock no longer contains the extra flag.)

This will affect pin reuse as well. B[x,y] need to provide reusable pins for B, B[x], B[y], etc.

This will affect pin reuse as well. B[x,y] need to provide reusable pins for B, B[x], B[y], etc.

Can you elaborate a bit on that one?  or point at the code you are thinking of


[C++] DataBuffer Constructor (or a non default constructor) uses reserve instead of resize?

The default constructor of DataBuffer uses resize:
https://github.com/apache/orc/blob/764257a7fb7ade4de9fed3813c67cb36629d5b00/c%2B%2B/src/MemoryPool.cc#L54-L58
which has extra initialization and free operations than reserve:
https://github.com/apache/orc/blob/764257a7fb7ade4de9fed3813c67cb36629d5b00/c%2B%2B/src/MemoryPool.cc#L81-L94
However, in some use cases, for example, create a row batch (ColumnVectorBatch) and read data from ORC file, the resize operation is redundant and reserve should be enough. Not sure this is a problem but I found a significant amount of time will be spent on creating batch if the batch size is large.

cc @wgtmac , @stiga-huang , @coderex2522

reserve only allocates the memory but does not call constructor of the objects of the buffer.  This is what resize does and will be undefined behavior if not doing so. Could you share your use case or some profiling data (e.g. flame graph) so we can discuss optimization opportunities in detail? @XinyuZeng

We can refer to the FileScan utility: https://github.com/apache/orc/blob/main/tools/src/FileScan.cc#L32. When the batch_size is set to large (e.g., the number of rows in the whole file), the scan time increases from ~0.5 to ~0.7, and the additional time is on the ColumnVectorBatch creation, specifically new (buf + i) T() operation, but this is not necessary.
I am doing this because the method of scanning to ColumnVectorBatch first and then transforming to another in-memory format (e.g., arrow) is not zero-copy. There is an opportunity to transfer the memory ownership of ColumnVectorBatch to Arrow (link) (although it is hard given ORC's ColumnVectorBatch's API), but that requires the allocation step of ColumnVectorBatch to be efficient.

I am happy to work on this and probably also the zero-copy to arrow in April if this is not in a hurry. Sorry but I got busy this month.

No problem. Just take your time.

I faced the same problem, DataBuffer's constructor is too heavy.

I faced the same problem, DataBuffer's constructor is too heavy.
Lot's of time waste in memset()

Do you want to submit a fix? @Smith-Cruise

I faced the same problem, DataBuffer's constructor is too heavy.
Lot's of time waste in memset()

Do you want to submit a fix? @Smith-Cruise

Actually, i didn't know how to fix it. I've tried using reserve() instead of resize(), but it will sometimes cause decompression error. Just looks like that memset() is necessary.

Sorry for the delaying on this. I filed #1739 to address this issue.


[ENG-4001] Ember Navbar User Dropdown Menu

Ticket: https://openscience.atlassian.net/browse/ENG-4001
Feature flag: feature/a11y-auth-dropdown-role

Purpose
The purpose of these changes was to add an appropriate role to the menu list dropdown items for authorized users.
Summary of Changes
These changes included repositioning the role='menuitem' to within the OsfLink and adding a role='button' to the Sign up and Sign in buttons for authenticated users.
Screenshot(s)
TBU
Side Effects
This change will affect the role communicated by screen readers, and should let users know that they are focused on a menu item.
QA Notes
-Do accessibility checks now pass for role? Did repositioning cause any other tests to fail?
-Is the text spoken by a SR correct? Could any changes be made to its pace?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.693%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5432

Relevant Lines:
7358

💛  - Coveralls


Open selected files in splits with keybindings

This is for #15.
I used the term FuzzyOpenIn as I thought FuzzyOpen might be confusing, since FuzzyOpen is already used to open the fzy panel instead of opening a file.

This is cool, thanks. I have a few comments and edits I'd like before I can merge this. I will comment inline when I have time, hopefully in the next day or two.

This is merged as part of #39


Problem with some concepts for crafting recipes

There's no option on here to make recipes like the cake. When crafting the cake you use 3 milk buckets right? Once crafted the 3 buckets will be given back in return. If there's a possible way to make it so there's an option to do that with other recipes that would be great, thanks!

As far as I know, it is not possible in 1.12 or 1.13. However you could give the player an invisible advancement on crafting the item, giving him back the concerned items and removing the advancement so the player can re-trigger the craft mechanic.


Probably Spelling Mistake?

Change Local Action -> Change Locale Action
React Boilerplate
Thank you for contributing! Please take a moment to review our contributing guidelines
to make the process easy and effective for everyone involved.
Please open an issue before embarking on any significant pull request, especially those that
add a new library or change existing tests, otherwise you risk spending a lot of time working
on something that might not end up being merged into the project.
Before opening a pull request, please ensure:

[ ] You have followed our contributing guidelines
[ ] double-check your branch is based on dev and targets dev
[ ] Pull request has tests (we are going for 100% coverage!)
[ ] Code is well-commented, linted and follows project conventions
[ ] Documentation is updated (if necessary)
[ ] Internal code generators and templates are updated (if necessary)
[ ] Description explains the issue/use-case resolved and auto-closes related issues

Be kind to code reviewers, please try to keep pull requests as small and focused as possible :)
IMPORTANT: By submitting a patch, you agree to allow the project
owners to license your work under the terms of the MIT License.

Can you change the base branch to dev before I merge?

Closing because pointing to the wrong branch and no updates for 2+ months


Docker Installation Issue

Following the docker install instructions here: https://hub.docker.com/repository/docker/laiyer/llm-guard-api/general
I run docker run -p 8000:8000 -d -e DEBUG='true' -e XFORMERS_MORE_DETAILS=1 llm-guard-docker:latest
The container exits and docker logs shows:

INFO:presidio-analyzer:Loaded recognizer: UsBankRecognizer
INFO:presidio-analyzer:Loaded recognizer: UsLicenseRecognizer
INFO:presidio-analyzer:Loaded recognizer: UsItinRecognizer
INFO:presidio-analyzer:Loaded recognizer: UsPassportRecognizer
INFO:presidio-analyzer:Loaded recognizer: UsSsnRecognizer
INFO:presidio-analyzer:Loaded recognizer: NhsRecognizer
INFO:presidio-analyzer:Loaded recognizer: SgFinRecognizer
INFO:presidio-analyzer:Loaded recognizer: AuAbnRecognizer
INFO:presidio-analyzer:Loaded recognizer: AuAcnRecognizer
INFO:presidio-analyzer:Loaded recognizer: AuTfnRecognizer
INFO:presidio-analyzer:Loaded recognizer: AuMedicareRecognizer
INFO:presidio-analyzer:Loaded recognizer: CreditCardRecognizer
INFO:presidio-analyzer:Loaded recognizer: CryptoRecognizer
INFO:presidio-analyzer:Loaded recognizer: DateRecognizer
INFO:presidio-analyzer:Loaded recognizer: EmailRecognizer
INFO:presidio-analyzer:Loaded recognizer: IbanRecognizer
INFO:presidio-analyzer:Loaded recognizer: IpRecognizer
INFO:presidio-analyzer:Loaded recognizer: MedicalLicenseRecognizer
INFO:presidio-analyzer:Loaded recognizer: SpacyRecognizer
INFO:presidio-analyzer:Loaded recognizer: PhoneRecognizer
INFO:presidio-analyzer:Loaded recognizer: UrlRecognizer
INFO:presidio-analyzer:Loaded recognizer: PatternRecognizer
INFO:presidio-analyzer:Loaded recognizer: PatternRecognizer
(…)ngual-nli-2mil7/resolve/main/config.json: 100%|██████████| 1.09k/1.09k [00:00<00:00, 2.25MB/s]
model.safetensors: 100%|██████████| 558M/558M [00:20<00:00, 27.4MB/s]
(…)2mil7/resolve/main/tokenizer_config.json: 100%|██████████| 467/467 [00:00<00:00, 278kB/s]
spm.model: 100%|██████████| 4.31M/4.31M [00:00<00:00, 32.2MB/s]
tokenizer.json: 100%|██████████| 16.3M/16.3M [00:00<00:00, 26.8MB/s]
(…)nli-2mil7/resolve/main/added_tokens.json: 100%|██████████| 23.0/23.0 [00:00<00:00, 45.6kB/s]
(…)il7/resolve/main/special_tokens_map.json: 100%|██████████| 173/173 [00:00<00:00, 579kB/s]
(…)RTa-language-id/resolve/main/config.json: 100%|██████████| 756/756 [00:00<00:00, 1.20MB/s]
pytorch_model.bin: 100%|██████████| 336M/336M [00:12<00:00, 26.7MB/s]
(…)ERTa-language-id/resolve/main/vocab.json: 100%|██████████| 994k/994k [00:00<00:00, 1.89MB/s]
(…)ERTa-language-id/resolve/main/merges.txt: 100%|██████████| 483k/483k [00:00<00:00, 2.95MB/s]
(…)ge-id/resolve/main/tokenizer_config.json: 100%|██████████| 19.0/19.0 [00:00<00:00, 20.6kB/s]
WARNING:xformers:WARNING[XFORMERS]: xFormers can't load C++/CUDA extensions. xFormers was built for:
    PyTorch 2.1.0 with CUDA None (you have 2.0.1)
    Python  3.10.13 (you have 3.10.13)
  Please reinstall xformers (see https://github.com/facebookresearch/xformers#installing-xformers)
  Memory-efficient attention, SwiGLU, sparse and more won't be available.
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/xformers/_cpp_lib.py", line 121, in _register_extensions
    torch.ops.load_library(ext_specs.origin)
  File "/usr/local/lib/python3.10/site-packages/torch/_ops.py", line 643, in load_library
    ctypes.CDLL(path)
  File "/usr/local/lib/python3.10/ctypes/__init__.py", line 374, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: /usr/local/lib/python3.10/site-packages/xformers/_C.so: undefined symbol: _ZN3c106SymInt19promote_to_negativeEv

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

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/xformers/_cpp_lib.py", line 131, in <module>
    _build_metadata = _register_extensions()
  File "/usr/local/lib/python3.10/site-packages/xformers/_cpp_lib.py", line 123, in _register_extensions
    raise xFormersInvalidLibException(build_metadata) from exc
xformers._cpp_lib.xFormersInvalidLibException: xFormers can't load C++/CUDA extensions. xFormers was built for:
    PyTorch 2.1.0 with CUDA None (you have 2.0.1)
    Python  3.10.13 (you have 3.10.13)
  Please reinstall xformers (see https://github.com/facebookresearch/xformers#installing-xformers)
  Memory-efficient attention, SwiGLU, sparse and more won't be available.
(…)-base-injection/resolve/main/config.json: 100%|██████████| 996/996 [00:00<00:00, 2.04MB/s]
pytorch_model.bin: 100%|██████████| 738M/738M [00:26<00:00, 27.9MB/s]
(…)ction/resolve/main/tokenizer_config.json: 100%|██████████| 412/412 [00:00<00:00, 896kB/s]
spm.model: 100%|██████████| 2.46M/2.46M [00:00<00:00, 21.8MB/s]
(…)se-injection/resolve/main/tokenizer.json: 100%|██████████| 8.66M/8.66M [00:00<00:00, 15.3MB/s]
(…)injection/resolve/main/added_tokens.json: 100%|██████████| 23.0/23.0 [00:00<00:00, 37.9kB/s]
(…)ion/resolve/main/special_tokens_map.json: 100%|██████████| 173/173 [00:00<00:00, 289kB/s]
[nltk_data] Downloading package vader_lexicon to
[nltk_data]     /home/user/nltk_data...
(…)c-comment-model/resolve/main/config.json: 100%|██████████| 704/704 [00:00<00:00, 1.28MB/s]
pytorch_model.bin: 100%|██████████| 268M/268M [00:10<00:00, 26.1MB/s]
(…)model/resolve/main/tokenizer_config.json: 100%|██████████| 403/403 [00:00<00:00, 2.28MB/s]
(…)xic-comment-model/resolve/main/vocab.txt: 100%|██████████| 232k/232k [00:00<00:00, 5.93MB/s]
(…)omment-model/resolve/main/tokenizer.json: 100%|██████████| 466k/466k [00:00<00:00, 10.9MB/s]
(…)del/resolve/main/special_tokens_map.json: 100%|██████████| 112/112 [00:00<00:00, 457kB/s]
(…)model/resolve/main/tokenizer_config.json: 100%|██████████| 2.00/2.00 [00:00<00:00, 6.57kB/s]
(…)detection-model/resolve/main/config.json: 100%|██████████| 657/657 [00:00<00:00, 1.20MB/s]
(…)s-detection-model/resolve/main/vocab.txt: 100%|██████████| 232k/232k [00:00<00:00, 12.2MB/s]
(…)del/resolve/main/special_tokens_map.json: 100%|██████████| 112/112 [00:00<00:00, 215kB/s]
tf_model.h5: 100%|██████████| 268M/268M [00:14<00:00, 18.3MB/s]
2023-10-16 16:13:03.486775: W tensorflow/tsl/framework/cpu_allocator_impl.cc:83] Allocation of 93763584 exceeds 10% of free system memory.
2023-10-16 16:13:03.646938: W tensorflow/tsl/framework/cpu_allocator_impl.cc:83] Allocation of 93763584 exceeds 10% of free system memory.
2023-10-16 16:13:03.701131: W tensorflow/tsl/framework/cpu_allocator_impl.cc:83] Allocation of 93763584 exceeds 10% of free system memory.
2023-10-16 16:13:03.945515: W tensorflow/tsl/framework/cpu_allocator_impl.cc:83] Allocation of 9437184 exceeds 10% of free system memory.
2023-10-16 16:13:03.953374: W tensorflow/tsl/framework/cpu_allocator_impl.cc:83] Allocation of 9437184 exceeds 10% of free system memory.
(…)ware-url-detect/resolve/main/config.json: 100%|██████████| 1.01k/1.01k [00:00<00:00, 5.52MB/s]
pytorch_model.bin: 100%|██████████| 711M/711M [00:21<00:00, 32.4MB/s]
(…)etect/resolve/main/tokenizer_config.json: 100%|██████████| 367/367 [00:00<00:00, 1.20MB/s]
(…)alware-url-detect/resolve/main/vocab.txt: 100%|██████████| 996k/996k [00:00<00:00, 30.5MB/s]
(…)ect/resolve/main/special_tokens_map.json: 100%|██████████| 125/125 [00:00<00:00, 510kB/s]
(…)li_R1_R2_R3-nli/resolve/main/config.json: 100%|██████████| 703/703 [00:00<00:00, 552kB/s]
pytorch_model.bin: 100%|██████████| 1.43G/1.43G [00:49<00:00, 29.1MB/s]

I see there's an error with xformers. How do I fix this?
Running on mac os m1 ventura 13.6

Hey @h0n33badg3rc0d3 ,
Thanks for submitting the issue.
30 minutes ago, we released the new version (v0.3.0) to the docker. It uses Python 3.11. I checked on my Macbook Pro M1Pro today.
Can you please try again to pull the new image and run it?

Latest push doesn't seem to help. Interestingly, even when i delete the prior image and then re-pull, i get the same image I had 11 hours ago. The hash of the latest image I pull matches the hash on hub.docker too-- 63d4a3a440e7f3335da507c03a76a8da2202d1d1a9605c1ae8a5100d68e334c1
On a different note, are you able to create an image for linux/amd64 architecture too?

I used this command on my Mac
docker run -p 8000:8000 -d -e DEBUG='true' -e XFORMERS_MORE_DETAILS=1 laiyer/llm-guard-api:latest

And got this response
Digest: sha256:63d4a3a440e7f3335da507c03a76a8da2202d1d1a9605c1ae8a5100d68e334c1
Status: Downloaded newer image for laiyer/llm-guard-api:latest
d6fe92d1e3a7e336c1ea3b37456656343d5753db31ec04b0453b31513880b064

Then launched the container and after some time of loading all the models, it worked.
On the other question, I put it on the list for the next version. Thanks for the suggestion.

Hey @h0n33badg3rc0d3 , I added images for 2 types of architectures https://hub.docker.com/repository/docker/laiyer/llm-guard-api/tags?page=1&ordering=last_updated


Fix up #1373

Summary

Rarely segmentation fault when killing NPCs

Oops, it's very critical. Thank you for fixing it.

I did push the "Merge pull request" button from browser and the PR is certainly merged, but it is not marked as merged for some reason... I close it manually anyway.


Request: Post this on SideQuest

Hey, I just got my Quest 2 yesterday, and I realize this app is not on SideQuest, nor is any other VNC app I could find (just tried searching for VNC)
SideQuest is a storefront/website/desktop app, that allows installing and updating programs/addons/patches without going through Oculus Store. Exmaple Virtual Desktop (no linux support) is sold on the official store, but the feature that enabled you to play PC VR games over Wifi was not allowed in the store, and so you buy the app on the Oculus platform, and install the extra feature using SideQuest. Note that this official patch for Virtual Desktop anlone has over 215 thousand downloads via SideQuest.
Games like open source ports of Half-Life 1, Doom1/2 etc, Quake1, Quake 2 and more are also available on SideQuest, just requiring you to add the game asset files your self.
Would be cool to have OVRVNC available on SideQuest, so that maybe more people can try it or keep it updated without downloading files and running adb commands manually.

Agree.. been asking for years... I have to sideload Android VNC clients


[FEATURE REQUEST] Ability to constrain dragged item to boundaries

I have some tabs I need to reorder.
It looks really weird to drag them outside of their container!

I do really miss this feature too! Also, this would solve the case where the item moved out of the boundaries and released there, takes it's initial place rather than the last fittable one

This would be a wonderful feature to have. It looks rather odd when constraining to an axis and it overflows.

The feature doesn't seem to exist in smooth-dnd, though an issue was opened yesterday.
https://github.com/kutlugsahin/smooth-dnd/issues/42

Hey all,
Check out behaviour="contain" property in the latest version. That will only allow dragged item to be floated in the parent Container.


kubectl get nodes connection refused

I updated my boxes and started to get this error when running kubectl from host:
error: couldn't read version from server: Get https://<IP_ADDRESS>/api: dial tcp <IP_ADDRESS>:443: connection refused

I ssh into c1 and open port 443 in iptables and now I can connect:
vagrant ssh c1
sudo iptables -A INPUT -m state --state NEW -m tcp -p tcp --dport 443

Hi @urgent, can you provide a little more detail on what you mean by "updated my boxes"? Or possible steps to reproduce the issue?

Hi @aaronlevy
I got the same issue when building for the first time.
I followed the instructions https://coreos.com/kubernetes/docs/latest/kubernetes-on-vagrant.html#start-the-machines. To fix the issue I used the above method to add the iptables rule on c1

I followed the same instructions https://coreos.com/kubernetes/docs/latest/kubernetes-on-vagrant.html, and didn't change any configuration(all defaults).  And got the same issue, and add the iptables rule still doesn't work.  Using OSX 10.9.5 + VirtualBox 5.0.14

Hi @aaronlevy , I just ran vagrant box update from inside coreos-kubernetes/multi-node/vagrant/

Can someone give me the output of iptable -L and route -n and ip addr from hosts exhibiting this. I am super confused why this is happening and how #267 fixes it.

I'm not sure if it's related to this issue; but when I use the stable channel kubelet seems te be missing in the vagrant box. Which causes the same error as mentioned above.
$ vagrant ssh c1
CoreOS stable (835.13.0)
Update Strategy: No Reboots
Failed Units: 1
coreos-cloudinit-309581450.service
$ journalctl -f
Feb 21 12:45:12 c1 systemd[884]: kubelet.service: Failed at step EXEC spawning /usr/bin/kubelet: No such file or directory
In the alpha channel everything is running as expected.

@hdanes That issue is unrelated to this. There is a WIP PR to update the docs to use the new kubelet wrapper: https://github.com/coreos/coreos-kubernetes/pull/276

I seem to have a related problem. It ends up exactly the same as this but it have to do with me being behind a corporate proxy.
Don't know If I should file a new ticket on updating the documentation on how to get this working when behind a proxy or continue on this one?
Anyway, my problem is that the kubelet-wrapper can't contact the Internet since it's not configured correctly for my proxy.
I have this in my log:
$ journalctl -f
Mar 16 17:39:54 c1 kubelet-wrapper[2721]: run: Get https://quay.io?ac-discovery=1: dial tcp <IP_ADDRESS>:443: i/o timeout
Mar 16 17:39:57 c1 systemd-timesyncd[648]: Timed out waiting for reply from <IP_ADDRESS>:123 (3.coreos.pool.ntp.org).
Mar 16 17:40:04 c1 systemd[1]: kubelet.service: Service hold-off time over, scheduling restart.
Mar 16 17:40:04 c1 systemd[1]: Stopped kubelet.service.
Mar 16 17:40:04 c1 systemd[1]: Starting kubelet.service...
Mar 16 17:40:04 c1 systemd[1]: Started kubelet.service.
Mar 16 17:40:05 c1 kubelet-wrapper[2750]: image: using image from file /usr/share/rkt/stage1-fly.aci
Mar 16 17:40:05 c1 kubelet-wrapper[2750]: image: searching for app image quay.io/coreos/hyperkube
Mar 16 17:40:06 c1 locksmithd[674]: [etcd.service etcd2.service] are inactive
Mar 16 17:40:06 c1 locksmithd[674]: Unlocking old locks failed: [etcd.service etcd2.service] are inactive. Retrying in 5m0s.
Mar 16 17:40:07 c1 systemd-timesyncd[648]: Timed out waiting for reply from <IP_ADDRESS>:123 (3.coreos.pool.ntp.org).
Mar 16 17:40:17 c1 systemd-timesyncd[648]: Timed out waiting for reply from <IP_ADDRESS>:123 (3.coreos.pool.ntp.org).
Mar 16 17:40:20 c1 systemd[1]: kubelet.service: Main process exited, code=exited, status=1/FAILURE
Mar 16 17:40:20 c1 systemd[1]: kubelet.service: Unit entered failed state.
Mar 16 17:40:20 c1 systemd[1]: kubelet.service: Failed with result 'exit-code'.

Adding proxy settings with the vagrant-proxyconf plugin doesn't seem to help for the kubelet-wrapper but works for curl and docker.
$ docker run hello-world
Unable to find image 'hello-world:latest' locally
latest: Pulling from library/hello-world
03f4658f8b78: Pull complete
a3ed95caeb02: Pull complete
Digest: sha256:8be990ef2aeb16dbcb9271ddfe2610fa6658d13f6dfb8bc72074cc1ca36966a7
Status: Downloaded newer image for hello-world:latest

Hello from Docker.
[...]

I'm at a loss on how I may continue. I can't find any documentation that describes how I set this up behind a corporate proxy.
If I can get all services running under systemd to be configured with my proxy I think I should be fine but where do I set a systemd global proxy?

I managed to solve my proxy problem for the kubelet-wrapper in the controller and worker by updating the install scripts available in the ./generic folder.
Environment=KUBELET_VERSION=${K8S_VER}
+Environment=http_proxy=<proxy>
+Environment=https_proxy=<proxy>
+Environment=no_proxy=localhost,<IP_ADDRESS>
ExecStart=/usr/lib/coreos/kubelet-wrapper \

I hope this might be of use to anyone that ends up here having the same problem as me.

Seems that there a quite a few different issues being discussed here & I haven't been able to reproduce the original issue. I'm going to close this for now, but if there is still something unresolved, please reopen.

$ date
Mon Apr  4 17:14:40 EDT 2016
$ pwd
/Users/saintx/dev/coreos-kubernetes/single-node
$ git status
On branch master
Your branch is up-to-date with 'origin/master'.
Untracked files:
(use "git add ..." to include in what will be committed)
local-kube-config

nothing added to commit but untracked files present (use "git add" to track)
$ vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Checking if box 'coreos-alpha' is up to date...
==> default: Clearing any previously set forwarded ports...
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
default: Adapter 1: nat
default: Adapter 2: hostonly
==> default: Forwarding ports...
default: 22 => 2222 (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
default: SSH address: <IP_ADDRESS>:2222
default: SSH username: core
default: SSH auth method: private key
default: Warning: Connection timeout. Retrying...
==> default: Machine booted and ready!
==> default: Configuring and enabling network interfaces...
==> default: Machine already provisioned. Run vagrant provision or use the --provision
==> default: flag to force provisioning. Provisioners marked to run always will still run.
$ cat local-kube-config
kubectl config set-cluster vagrant-single-cluster --server=https://<IP_ADDRESS>:443 --certificate-authority=${PWD}/ssl/ca.pem
kubectl config set-credentials vagrant-single-admin --certificate-authority=${PWD}/ssl/ca.pem --client-key=${PWD}/ssl/admin-key.pem --client-certificate=${PWD}/ssl/admin.pem
kubectl config set-context vagrant-single --cluster=vagrant-single-cluster --user=vagrant-single-admin
kubectl config use-context vagrant-single
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"1", GitVersion:"v1.1.8", GitCommit:"a8af33dc07ee08defa2d503f81e7deea32dd1d3b", GitTreeState:"clean"}
$ . local-kube-config
cluster "vagrant-single-cluster" set.
user "vagrant-single-admin" set.
context "vagrant-single" set.
switched to context "vagrant-single".
$ kubectl get nodes
error: couldn't read version from server: Get https://<IP_ADDRESS>:443/api: dial tcp <IP_ADDRESS>:443: i/o timeout

In this case it's possible the api-server isn't actually running yet. The startup can take some time because it will need to download the kubelet container, and start all services / assets.
If you give it a couple minutes does the api-server start responding? Or if not, is there anything of interest in the kubelet service log?

@aaronlevy I am still having the same issue. Hopefully I can help to figure out what's going on. I followed https://coreos.com/kubernetes/docs/latest/kubernetes-on-vagrant.html instructions for multi-node install.
I have systemctl status for flanned, kubelet and iptables -L, route -n and docker ps output below. Let me know if you need any other informations. Thanks!
$ vagrant status
Current machine states:

e1                        running (virtualbox)
c1                        running (virtualbox)
w1                        running (virtualbox)

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"1", GitVersion:"v1.1.8", GitCommit:"a8af33dc07ee08defa2d503f81e7deea32dd1d3b", GitTreeState:"clean"}
error: couldn't read version from server: Get https://<IP_ADDRESS>:443/api: dial tcp <IP_ADDRESS>:443: connection refused

I can login to c1 and run the following:
core@c1 ~ $ systemctl status flanneld
● flanneld.service - Network fabric for containers
   Loaded: loaded (/usr/lib64/systemd/system/flanneld.service; disabled; vendor preset: disabled)
  Drop-In: /etc/systemd/system/flanneld.service.d
           └─40-ExecStartPre-symlink.conf.conf
   Active: active (running) since Tue 2016-04-05 05:34:34 UTC; 10h ago
     Docs: https://github.com/coreos/flannel
  Process: 1651 ExecStartPost=/usr/bin/docker run --net=host --rm --volume=/run:/run ${FLANNEL_IMG}:${FLANNEL_VER} /opt/bin/mk-docker-opts.sh -d /run/flannel_docker_opts.env -i (code=exited, status=0/SUCCESS)
  Process: 1527 ExecStartPre=/usr/bin/ln -sf /etc/flannel/options.env /run/flannel/options.env (code=exited, status=0/SUCCESS)
  Process: 1520 ExecStartPre=/usr/bin/touch ${FLANNEL_ENV_FILE} (code=exited, status=0/SUCCESS)
  Process: 1516 ExecStartPre=/usr/bin/mkdir -p ${ETCD_SSL_DIR} (code=exited, status=0/SUCCESS)
  Process: 1510 ExecStartPre=/usr/bin/mkdir -p /run/flannel (code=exited, status=0/SUCCESS)
  Process: 1505 ExecStartPre=/sbin/modprobe ip_tables (code=exited, status=0/SUCCESS)
 Main PID: 1531 (sdnotify-proxy)
   Memory: 9.2M
      CPU: 101ms
   CGroup: /system.slice/flanneld.service
           ├─1531 /usr/libexec/sdnotify-proxy /run/flannel/sd.sock /usr/bin/docker run --net=host --privileged=true --rm --volume=/run/flannel:/run/flannel --env=NOTIFY_SOCK...
           └─1539 /usr/bin/docker run --net=host --privileged=true --rm --volume=/run/flannel:/run/flannel --env=NOTIFY_SOCKET=/run/flannel/sd.sock --env=AWS_ACCESS_KEY_ID= ...

Apr 05 05:34:34 c1 sdnotify-proxy[1531]: I0405 05:34:34.797434 00001 ipmasq.go:50] Adding iptables rule: FLANNEL ! -d <IP_ADDRESS>/4 -j MASQUERADE
Apr 05 05:34:34 c1 sdnotify-proxy[1531]: I0405 05:34:34.809381 00001 ipmasq.go:50] Adding iptables rule: POSTROUTING -s <IP_ADDRESS>/16 -j FLANNEL
Apr 05 05:34:34 c1 sdnotify-proxy[1531]: I0405 05:34:34.811277 00001 ipmasq.go:50] Adding iptables rule: POSTROUTING ! -s <IP_ADDRESS>/16 -d <IP_ADDRESS>/16 -j MASQUERADE
Apr 05 05:34:34 c1 sdnotify-proxy[1531]: I0405 05:34:34.822878 00001 vxlan.go:153] Watching for L3 misses
Apr 05 05:34:34 c1 sdnotify-proxy[1531]: I0405 05:34:34.822878 00001 vxlan.go:159] Watching for new subnet leases
Apr 05 05:34:34 c1 sdnotify-proxy[1531]: I0405 05:34:34.825746 00001 vxlan.go:273] Handling initial subnet events
Apr 05 05:34:34 c1 sdnotify-proxy[1531]: I0405 05:34:34.825771 00001 device.go:159] calling GetL2List() dev.link.Index: 4
Apr 05 05:34:34 c1 systemd[1]: Started Network fabric for containers.
Apr 05 05:35:23 c1 sdnotify-proxy[1531]: I0405 05:35:23.448705 00001 vxlan.go:232] Subnet added: <IP_ADDRESS>/24

core@c1 /etc/systemd/system $ systemctl status kubelet.service -l
● kubelet.service
   Loaded: loaded (/etc/systemd/system/kubelet.service; enabled; vendor preset: disabled)
   Active: active (running) since Tue 2016-04-05 05:27:56 UTC; 10h ago
  Process: 1312 ExecStartPre=/usr/bin/mkdir -p /etc/kubernetes/manifests (code=exited, status=0/SUCCESS)
 Main PID: 1322 (kubelet)
   Memory: 8.7M
      CPU: 12.525s
   CGroup: /system.slice/kubelet.service
           ├─1322 /kubelet --api-servers=http://<IP_ADDRESS>:8080 --register-node=false --allow-privileged=true --config=/etc/kubernetes/manifests --hostname-override=<IP_ADDRESS> --cluster_dns=<IP_ADDRESS> --cluster_domain=cluster.local
           └─1737 journalctl -k -f

Apr 05 15:39:25 c1 kubelet-wrapper[1322]: E0405 15:39:25.402647    1322 kubelet.go:2291] Error updating node status, will retry: error getting node "<IP_ADDRESS>": Get http://<IP_ADDRESS>:8080/api/v1/nodes/<IP_ADDRESS>: dial tcp <IP_ADDRESS>:8080: connection refused
Apr 05 15:39:25 c1 kubelet-wrapper[1322]: E0405 15:39:25.402830    1322 kubelet.go:2291] Error updating node status, will retry: error getting node "<IP_ADDRESS>": Get http://<IP_ADDRESS>:8080/api/v1/nodes/<IP_ADDRESS>: dial tcp <IP_ADDRESS>:8080: connection refused
Apr 05 15:39:25 c1 kubelet-wrapper[1322]: E0405 15:39:25.403081    1322 kubelet.go:2291] Error updating node status, will retry: error getting node "<IP_ADDRESS>": Get http://<IP_ADDRESS>:8080/api/v1/nodes/<IP_ADDRESS>: dial tcp <IP_ADDRESS>:8080: connection refused

$core@c1 /etc/systemd/system $ sudo  iptables -L
Chain INPUT (policy ACCEPT)
target     prot opt source               destination
           tcp  --  anywhere             anywhere             state NEW tcp dpt:https

Chain FORWARD (policy ACCEPT)
target     prot opt source               destination
DOCKER-ISOLATION  all  --  anywhere             anywhere
DOCKER     all  --  anywhere             anywhere
ACCEPT     all  --  anywhere             anywhere             ctstate RELATED,ESTABLISHED
ACCEPT     all  --  anywhere             anywhere
ACCEPT     all  --  anywhere             anywhere

Chain OUTPUT (policy ACCEPT)
target     prot opt source               destination

Chain DOCKER (1 references)
target     prot opt source               destination

Chain DOCKER-ISOLATION (1 references)
target     prot opt source               destination
RETURN     all  --  anywhere             anywhere

core@c1 /etc/systemd/system $ route -n
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
<IP_ADDRESS>         <IP_ADDRESS>        <IP_ADDRESS>         UG    1024   0        0 eth0
<IP_ADDRESS>        <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 eth0
<IP_ADDRESS>        <IP_ADDRESS>         <IP_ADDRESS> UH    1024   0        0 eth0
<IP_ADDRESS>        <IP_ADDRESS>         <IP_ADDRESS>     U     0      0        0 flannel.1
<IP_ADDRESS>       <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 docker0
<IP_ADDRESS>      <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 eth1

core@c1 /etc/systemd/system $ ip addr
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
2: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc fq_codel state UP group default qlen 1000
    link/ether 08:00:27:fa:a9:e4 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global dynamic eth0
       valid_lft 80283sec preferred_lft 80283sec
    inet6 fe80::a00:27ff:fefa:a9e4/64 scope link
       valid_lft forever preferred_lft forever
3: eth1: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc fq_codel state UP group default qlen 1000
    link/ether 08:00:27:6c:5f:1f brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global eth1
       valid_lft forever preferred_lft forever
    inet6 fe80::a00:27ff:fe6c:5f1f/64 scope link
       valid_lft forever preferred_lft forever
4: flannel.1: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1450 qdisc noqueue state UNKNOWN group default
    link/ether 46:69:24:eb:58:ac brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/16 scope global flannel.1
       valid_lft forever preferred_lft forever
    inet6 fe80::4469:24ff:feeb:58ac/64 scope link
       valid_lft forever preferred_lft forever
5: docker0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc noqueue state DOWN group default
    link/ether 02:42:12:27:d6:52 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 scope global docker0
       valid_lft forever preferred_lft forever

core@c1 /etc/systemd/system $ docker ps
CONTAINER ID        IMAGE                                  COMMAND             CREATED             STATUS              PORTS               NAMES
2a7662fa658b        gcr.io/google_containers/pause:0.8.0   "/pause"            10 hours ago        Up 10 hours                             k8s_POD.6d00e006_kube-proxy-<IP_ADDRESS>_kube-system_55cd7edd6b9f452ae2261be14a240e5b_68f2863d
9e07aa407878        gcr.io/google_containers/pause:0.8.0   "/pause"            10 hours ago        Up 10 hours                             k8s_POD.6d00e006_kube-podmaster-<IP_ADDRESS>_kube-system_7bd9ef00a6fcc78ab5409baa900f3e94_f3f9beab
00716cce8e9f        gcr.io/google_containers/pause:0.8.0   "/pause"            10 hours ago        Up 10 hours                             k8s_POD.6d00e006_kube-apiserver-<IP_ADDRESS>_kube-system_734dccbdca01b2f29191fceb5377dfb9_1f4fb81a

So it doesn't look like any of the kubernetes control-plane is running (based on the docker ps output). Can you see if there are pods that had died (docker ps -a) and then their logs might give you more info (docker logs <container id>)

@aaronlevy I did docker ps -a, no died dockers. Out of desperation, I rebooted c1, and then it comes back with one error message which did not exist the first time:
MBPSF:vagrant sfeng$ vagrant ssh c1
Last login: Tue Apr  5 17:10:44 2016 from <IP_ADDRESS>
CoreOS alpha (1000.0.0)
Update Strategy: No Reboots
Failed Units: 1
  coreos-cloudinit-336841772.service

core@c1 ~ $ systemctl status  coreos-cloudinit-336841772.service
● coreos-cloudinit-336841772.service - Unit generated and executed by coreos-cloudinit on behalf of user
   Loaded: loaded (/run/systemd/system/coreos-cloudinit-336841772.service; static; vendor preset: disabled)
  Drop-In: /run/systemd/system/coreos-cloudinit-336841772.service.d
           └─50-Description.conf, 50-ExecStart.conf
   Active: failed (Result: exit-code) since Tue 2016-04-05 15:50:02 UTC; 1h 22min ago
  Process: 737 ExecStart=/bin/bash /var/lib/coreos-cloudinit/scripts/336841772 (code=exited, status=1/FAILURE)
 Main PID: 737 (code=exited, status=1/FAILURE)

Apr 05 15:50:02 c1 systemd[1]: Started Unit generated and executed by coreos-cloudinit on behalf of user.
Apr 05 15:50:02 c1 bash[737]: Missing required config value: ETCD_ENDPOINTS
Apr 05 15:50:02 c1 systemd[1]: coreos-cloudinit-336841772.service: Main process exited, code=exited, status=1/FAILURE
Apr 05 15:50:02 c1 systemd[1]: coreos-cloudinit-336841772.service: Unit entered failed state.
Apr 05 15:50:02 c1 systemd[1]: coreos-cloudinit-336841772.service: Failed with result 'exit-code'.

The system does work now after the reboot. I am going to destroy everything to see if I can re-produce.
core@c1 ~ $ docker ps
CONTAINER ID        IMAGE                                      COMMAND                  CREATED             STATUS              PORTS               NAMES
37ea5e760d13        quay.io/coreos/hyperkube:v1.1.8_coreos.0   "/hyperkube scheduler"   About an hour ago   Up About an hour                        k8s_kube-scheduler.d7af6bd8_kube-scheduler-<IP_ADDRESS>_kube-system_ba0ffcd76178cb64f5a40cb39237b663_84da12ba
8aed6eda3959        gcr.io/google_containers/pause:0.8.0       "/pause"                 About an hour ago   Up About an hour                        k8s_POD.6d00e006_kube-scheduler-<IP_ADDRESS>_kube-system_ba0ffcd76178cb64f5a40cb39237b663_52172f5e
d6bfe3c3578e        quay.io/coreos/hyperkube:v1.1.8_coreos.0   "/hyperkube controlle"   About an hour ago   Up About an hour                        k8s_kube-controller-manager.f55cdde6_kube-controller-manager-<IP_ADDRESS>_kube-system_e71b39ee832e3cbede008579a3b6e46f_73b3a8e4
e253a9610b47        gcr.io/google_containers/pause:0.8.0       "/pause"                 About an hour ago   Up About an hour                        k8s_POD.6d00e006_kube-controller-manager-<IP_ADDRESS>_kube-system_e71b39ee832e3cbede008579a3b6e46f_77cc512f
ed1277e4f024        quay.io/coreos/hyperkube:v1.1.8_coreos.0   "/hyperkube apiserver"   About an hour ago   Up About an hour                        k8s_kube-apiserver.fb534df3_kube-apiserver-<IP_ADDRESS>_kube-system_734dccbdca01b2f29191fceb5377dfb9_b59c2622
dd44ba6b3f04        quay.io/coreos/hyperkube:v1.1.8_coreos.0   "/hyperkube proxy --m"   About an hour ago   Up About an hour                        k8s_kube-proxy.924154b4_kube-proxy-<IP_ADDRESS>_kube-system_55cd7edd6b9f452ae2261be14a240e5b_e752dcdf
6ad4d665b798        gcr.io/google_containers/podmaster:1.1     "/podmaster --etcd-se"   About an hour ago   Up About an hour                        k8s_controller-manager-elector.2d387256_kube-podmaster-<IP_ADDRESS>_kube-system_7bd9ef00a6fcc78ab5409baa900f3e94_8dc77b91
5368fd0eb68d        gcr.io/google_containers/podmaster:1.1     "/podmaster --etcd-se"   About an hour ago   Up About an hour                        k8s_scheduler-elector.2611672a_kube-podmaster-<IP_ADDRESS>_kube-system_7bd9ef00a6fcc78ab5409baa900f3e94_80dc29ac
2cfb0500f7b1        gcr.io/google_containers/pause:0.8.0       "/pause"                 About an hour ago   Up About an hour                        k8s_POD.6d00e006_kube-apiserver-<IP_ADDRESS>_kube-system_734dccbdca01b2f29191fceb5377dfb9_81f13219
f90c0eddb02e        gcr.io/google_containers/pause:0.8.0       "/pause"                 About an hour ago   Up About an hour                        k8s_POD.6d00e006_kube-podmaster-<IP_ADDRESS>_kube-system_7bd9ef00a6fcc78ab5409baa900f3e94_c861042b
83585871ca35        gcr.io/google_containers/pause:0.8.0       "/pause"                 About an hour ago   Up About an hour                        k8s_POD.6d00e006_kube-proxy-<IP_ADDRESS>_kube-system_55cd7edd6b9f452ae2261be14a240e5b_70db16fc

@aaronlevy destroy and rebuild the cluster works! You are right that it does take some time. For me the kubelet-wrapper spent 8 minutes to download ACI image and only after that the controller-plane pods start showing up. It's about 10 minutes waiting. If you are on a slower network (like my earlier report, I was on slow home network), it would take much much longer.

I am still having this issue. Can someone please help?
$ kubectl --kubeconfig=kubeconfig get nodes
Unable to connect to the server: dial tcp <IP_ADDRESS>:443: i/o timeout
CoreOS version
$ vagrant ssh c1
Last login: Wed Mar 15 04:38:29 UTC 2017 from <IP_ADDRESS> on pts/0
Container Linux by CoreOS alpha (1339.0.0)
Failed Units: 1
update-engine.service
KUBECONFIG FILE
apiVersion: v1
clusters:

cluster:
certificate-authority: ssl/ca.pem
server: https://<IP_ADDRESS>:443
name: vagrant-multi-cluster
contexts:
context:
cluster: vagrant-multi-cluster
user: vagrant-multi-admin
name: vagrant-multi
current-context: vagrant-multi
kind: Config
preferences: {}
users:
name: vagrant-multi-admin
user:
client-certificate: ssl/admin.pem
client-key: ssl/admin-key.pem


Add checkbox column to enable multi-selection

Current status
Multi-selections are only possible on Windows, because the control key + click triggers the right click menu on a Mac.
Suggestion
Add a checkbox column to the most left, if multi-selection is enabled in LineUp (so not by default). When activating a checkbox a selection event should be triggered.
This would make it easy for every OS and device (also touch) to select multiple rows.

I created a new column type: 'selection', see e.g. https://github.com/Caleydo/lineup.js/blob/master/demo/test.js#L45 how to apply it

Thanks for the fast implementation. I will give it a try.


add version number in title

You can see easy than version with problems like games

Not entirely sure what you mean. Are you talking about the FBI version, game versions, or something else entirely?

to easily identify the version of your program and you see if the latest version

The version is already on the main menu, right under the logo.

if I can but if you add in the title of every window you can control if people have the latest version when you report errors

Eh, I guess I don't see why not. Implemented.


Moved logos from Metadata to default screen parts

Fixes a breaking change made in AssemblyLine 2.7 that would override logos of other packages.This fixes it and goes back to the original logo.

Looks like you fixed this already in https://github.com/IllinoisLegalAidOnline/docassemble-ILAO/commit/6ed5ddb772bd1cf9cd15ea526b25881da558a25f! I'll close this.


Typescript support

Hey! I could not find any types for this project at @types/node-color-log
Would you be interested in typescript support? I'm willing to tackle it but I won't if you are not interested.

@jollyjerr  Please help. Thanks a lot! :)

I think it would be fine if we have a .d.ts file? Could you help create a PR?

@tigercosmos sounds perfect. I'll submit a PR sometime soon!


Update MongoDbEndpointBuilderFactory.java

Add missing greater sign (recovered from old documentation at http://people.apache.org/~dkulp/camel/mongodb.html).

@zregvart it looks like the > is not rendered?

Oh interesting :) I see, the comment exists twice (once in the factory for the endpoint, once in the endpoint). Yes, could be a rendering problem?

I think so.

I think in JavaDoc we need to use HTML entities, > in this case.

This should be fixed now. I used "greater than". Thanks.


Update the game over message

Update the game over message so people know how to play again!

successful merge


Cannot borrow multi-line strings?

I encountered something I find to be somewhat perplexing. I have this relatively simple type that I want to deserialize from TOML into; it looks like this:
#[derive(Deserialize, Debug)]
pub struct Template<'t> {
    #[serde(borrow)]
    pub variables: Option<HashMap<&'t str, &'t str>>,
}

It works... until there's a newline in the string. In other words, this deserializes just fine:
[variables]
foo = "bar"

as does this:
[variables]
foo = """bar"""

but this fails with an error:
[variables]
foo = """bar
baz"""

The error given is:
invalid type: string "bar\nbaz", expected a borrowed string for key `variables.foo` at line 2 column 7'

What is it about a newline character that suddenly makes this balk? Newline characters can be borrowed too! 😅

Are you maybe using Windows newlines?  The tokenizer normalizes newlines to \n, and to do that it needs to create an owned string.

I have to say I find it kind of uncomfortable that the behavior of the deserializer will change depending on the line separator. If the TOML format allows \r\n for newline, which it does, why can't the deserializer hand the data to me as-is?

Can you confirm that \r\n is what's actually in the string being parsed? If that's the case then there's not really much we can do about this. Newlines are normalized to \n, and if the file contents have \r\n then there's nothing to do but allocate a new string.

Yes, can confirm that the file contains \r\n, like all text files on Windows unless explicitly made to be otherwise.

Newlines are normalized to \n

Yes, I understand that this is the case, but I don't understand what the rationale is. TOML allows \r\n as newlines. Forcing the use of use owned strings for any app that wants to be able to deserialize TOML on Windows seems very much like an anti-feature to me.

Is this normalizing behavior one place in the code or all over it? I don't mind forking and removing it in my version if that's easier for everyone.

@alexschrod
TOML specs don't enforce particular rules for preserving/normalizing: link

TOML parsers should feel free to normalize newline to whatever makes sense for their platform.

Having normalized everything to \n has more advantages to toml-rs:

Cross-platform behavior.
Reading the same toml file in windows and linux and then comparing the deserialized structs will yield full equality.
Simpler code.
Preserving \r\n adds additional complexity to the code. Where the lexer which is a totally pure function will have some platform-specific behavior otherwise (and this totally sucks).

I agree that this design decision may not fully fit your use case, but the fruits of normalizing \r\n seem to be more useful in the general case.
I'd recommend pre-normalizing crlf in your string input to \n as a workaround.


BUG: Playlist ui delete icon

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
@Satyam1923
check this out

the delete icons shown in center insted we can display it at the end of hover effect (right side)
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I want to work on this issue please assign this to me under gssoc


[RFD] Architectural Choices

Discussion Ideas and Topics
This document contains a list of topics for discussion regarding OChami and its design going forward and other considerations in the future. It is only meant to spark discussions and it not meant to be permenant and is subject to change. Some topics may eventually be removed or converted to its own RFD if needed.

Naming Nodes Within And Across Microservices #10

See related RFD 10 in Roadmap.

Could the mandatory xname check be made optional? That way, if the xname string is valid, it will follow the CSM path for handling them. Otherwise, it would probably make more sense to have a generalized naming convention.

Modularity between microservices

For example, instead of sending Magellan's output to SMD, could we instead send it to BSS or any other microservice completely and have it do something with it instead? The data structures representing a Redfish endpoint and inventory could be standardized in a JSON format between microservices, and each microservice can handle data structure input in a way that is relevant to the functionality of the microservice (or do nothing at all).

Compatibility with existing CSM architecture

There is interest within the CSM/HPC community to maintain some compatibility going forward with the current CSM software. Should there be an extended discussion about the specifics on what should be kept or how to proceed with implementing backwards compatible?

Maintaining compatible creates an interesting dynamic between intentionally moving away from CSM distributed monolithic design of microserves keeping old

Changing how discovery/inventory tracking works

SMD has a rediscovery feature that polls for inventory updates. Whenever a new Redfish endpoint is added, SMD will try to (re)discover new hardware and adds the hardware to the inventory tracking system (?)

With a separate Redfish event listener microservice, discovery can be event driven and initiated whenever the event listener detects a change in inventory hardware instead of polling.

API contracts and boundaries #39

The input and output API of each microservice should be clearly defined. Ideally, each microservice should be able to operate in isolation without a dependency on another microservice.

Rewrite VS Refactor

Each microservice will likely require more or less time and effort to do either a rewrite or to refactor. Therefore, it may be beneficial to identify which microservices API would favor each.

Diagram showing microservices relationship (WIP)

The diagram is organized around functionality of microservices which serves as the main focus. It also shows the CSM microservice that is capable of providing functionality as well as an open-source alternative.

The diagrams shows the inputs for the microservice to work and outputs of each function

%%{
    init: {
        'theme':'forest',
        'title':'Node',
        'themeVariables': {
            'lineColor': 'green',
            'secondaryColor': '#006100',
            'tertiaryColor': '#fff',
            'fontSize': '26px',
            'background': '#000000',
            'nodeTextColor': '#ff0000'
        }
    }
}%%
flowchart LR;
    subgraph smd[SMD]
        discovery
        sm
        inventory
        rfe
    end

    subgraph bss[BSS]
        boot
    end
    subgraph discovery[Discovery of BMC nodes]
        direction LR
        discovery_alt[Magellan]
    end

    subgraph sm[State Management]
        direction LR
        sm_alt[N/A]
    end

    subgraph inventory[Inventory Tracking]
        direction LR
        inventory_alt[N/A]
    end

    subgraph rfe[Redfish Event Listener]
        direction LR
        rfe_alt[N/A]
    end

    subgraph boot[Boot Parameters and Config]
        direction LR
        bss_alt[N/A]
    end

    discovery -- output --> sm
    smd -- Components --> bss
    inventory --- sm
    rfe --- sm

I have updated this RFD to refer to the other focused discussions that are related.  We should update others as needed.


Redirect URLs after login/logout have problems with reverse proxying

Originally reported on Google Code with ID 604
What steps will reproduce the problem?

1. Run gitblit GO with http
2. behind a reverse proxy (Apache)
3. configure apache to use https
4. access gitblit via https and observe the URLs during login & logout.

What is the expected output? What do you see instead?

I'd expect to stay on https but the url changes to http. Hard to spot for the user
that the connection is no longer secure.

What version of the product are you using? On what operating system?
gitblit 1.6.7, centos 7

Please provide any additional information below.

All URLs gitblit creates are correct and work just fine. I suspect that the generated
redirect URLs don't heed the X-Forwarded-Proto header or something alike.

Workaround:

Redirect any access to gitblit via http to https. I think the user then gets redirected
twice, but it does the trick:

RewriteEngine On
RewriteCond %{HTTPS} !=on
RewriteRule ^/gitblit/(.*) https://%{SERVER_NAME}/gitblit/$1 [R,L]

This makes it impossible to access gitblit via http, as the user will always end up
using https, which is what you'll probably want anyway.

Here's the apache config I'm using (without the workaorund):

ProxyPass         /gitblit  http://localhost:8082/gitblit nocanon
ProxyPassReverse  /gitblit  http://localhost:8082/gitblit
ProxyRequests     Off
AllowEncodedSlashes NoDecode
ProxyPreserveHost On
ProxyVia Off
<Proxy>
        AddDefaultCharset off
        Order deny,allow
        Allow from all
</Proxy>
Header edit Location ^http://([^⁄]+)/gitblit/ https://$1/gitblit/
RequestHeader set X-Forwarded-Proto https
RequestHeader set X-Forwarded-Port 443

Reported by smurn.org on 2015-07-06 18:28:16

I usually lost proper URL when clicking any button on a page.  All other links are fine. Did update to canonical url - no help.

I had similar problem with tickets, login/logout worked for me
Adding
Header edit Ajax-Location ^http://([^⁄]+)/ https://$1/
to apache configuration solves the problem for me.

Related to #965

I have similar problem, but only when try to login. After login browser redirect me to https://localhost. What is interesting - when i go back to proper url i'm logged in and all works fine. How to fix this? (rev proxy made on apache2).

When using nginx, here's how the config for the forwarding could be done:
server {
        server_name mygitblit.example.com;
        listen 80;
        return 301 https://$host$request_uri;
}

This is in addition to the separate server entry for the actual 443/proxy forwarfing. If you don't run any other sites on that machine, you can make it the default:
server {
        server_name _;
        listen 80 default_server;
        return 301 https://$host$request_uri;
}

You may also want to set HSTS so once Gitblit opened via HTTPS your browser will never use HTTP again. In nginx you add:
server {
    ...
    add_header Strict-Transport-Security max-age=15768000;
    ...
}

Personally I use HTTP/2 only (no HTTP/1.1); modern browsers doesn't allow to open http/2 website without TLS.


Relase for scala 3

I could not find a release for scala-3. I can see that the projects cross-builds to it though. Would you mind making one with the current master? If there are blockers left, I can try to help.

Sorry, I have no bandwidth to maintain this repo.
Please consider ScalablyTyped instead 🙇
I believe it supports aws-sdk-js-v3 too.


Haskell: configuration-common: updates to overrides

Motivation for this change
Reports and PRs are merging and closing, often overrides await a release.

xmobar: update patch description
statistics: free from overriding
rib: update override description
binary-instances: update override description
nixfmt: update override description
pantry: free from override and upstreamed patch
feed: free from override
hnix: rm generateOptparseApplicativeCompletion, since according binary not found: https://hydra.nixos.org/build/122819514/nixlog/1

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@Anton-Latukha Thx for this work!
Note that I have a slightly more sophisticated fix for hnix at https://github.com/NixOS/nixpkgs/pull/91251 so maybe you want to remove this from this PR or rebase it once the other one got merged.
Other changes look good to me, but haven‘t tested the builds.

Oh, thanks. You are right.

I like the improvement of the comments.
On a more general note we should definitely strive to have some standard of consistently telling our future selves when to touch an override again.

I think someone in the past, several years ago and more (somehow I think about thoughtpolice or some other Uroboros type avatar) - had a scrip that tested the broken/overridden packages.
It should be pretty simple to setup such crawler/walker.

https://github.com/NixOS/nixpkgs/pull/91312#issuecomment-647780093
Yes, that is obvious, IDK why others do not do such things. I for example even for myself made a function that inserts those # ISO-short-date: NOTE: , so I can indicate myself how long ago I made that remark, it is pretty useful everywhere, good example is NixOS derivations list.

@GrahamcOfBorg build haskellPackages.xmobar haskellPackages.statistics haskellPackages.rib haskellPackages.nixfmt haskellPackages.pantry haskellPackages.feed

I am not sure, that we need the test on packages were you only changed the comments.^^

Yep, automatically populated the list. I used to updating regular packages on master. I probably made the bot too worried.

@GrahamcOfBorg build haskellPackages.xmobar
@GrahamcOfBorg build haskellPackages.statistics
@GrahamcOfBorg build haskellPackages.rib
@GrahamcOfBorg build haskellPackages.binary-instances
@GrahamcOfBorg build haskellPackages.nixfmt
@GrahamcOfBorg build haskellPackages.pantry
@GrahamcOfBorg build haskellPackages.feed

Looks like everything is building on at least Linux!  Thanks for looking into this and fixing all these up!