document: false doesn't work

Need to use fetch here.

@justinko Which gem version are you using? I am using 4.6.0 and it works.

But in 4.8.0 it doesn't work.

4.9.0 works again :)

This should be fixed now with #351


If args is a list, the value of parameter shell should be true.

If args is a list, shouldn't the value of parameter a be true? Otherwise, only the first item in the args list will be executed as a shell character.

If args is a list, shouldn't the value of parameter shell be true? Otherwise, only the first item in the args list will be executed as a shell character.

@Doswind Can you please sign CLA. Thanks!

@Doswind Can you please sign CLA. Thanks!

I've signed the CLA, but it's still showing me as unsigned.  And i've checked  i'm using the same email/username in my GitHub and Google CLA.  Can you rescan this PR?  Thanks!.

Is there something that fails without this patch and passes with it?

Is there something that fails without this patch and passes with it?

No errors were found. However,i think the usage is wrong. In the actual test, only the first command in the list will be executed, and the other command parameters will not be executed unless the program does so intentionally

Please add a unit test

@Doswind Can you please check @mihaimaruseac's comments and keep us posted ? Thanks!

@Doswind Any update on this PR? Please. Thanks!


Custom Redirect Screen

It would be amazing if, when returning a redirect, you could configure a Jinja template to be sent along with it. This would allow for custom styling if your internet is slow.
Technically this feature could be a part of Werkzeug, but I believe that it is more fitting for Flask. See below for some examples and code.
What's currently being sent:
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>Redirecting...</title>

Redirecting...

You should be redirected automatically to target URL: {display_location}. If not click the link.

(It also forgets a
 tag.) Here would be a nicer way to do this:
<!DOCTYPE html>

  <title>Site Name - Redirecting</title>
  <link rel="stylesheet" href="{{ url_for('static', filename='style.css') }}">

Don't worry, the page is loading!

You accessed a redirect URL. Please wait a few seconds to be redirected.

If it doesn't work, click {{ display_location }}

Then a custom function could be written:
def redirect(location: str, code: int = 302, Response: t.Optional[t.Type["Response"]] = None, template: str = "redirect.html"):
  ...

  response = Response(render_template(template), code, mimetype="text/html")
  response.headers["Location"] = location
  return response

(Code left out for shortness.) I'm sure there is a better way to define a template, but it's just a sketch.
Thanks for this wonderful package (and hopefully feedback!)
~BD103

redirect returns a Response. You can assign any valid body you want to response.data.


[WIP] Adding EdgeSource

Fixes https://github.com/knative/eventing-sources/issues/33
Proposed Changes

Add EdgeSource

Release Note
NONE

/hold
@evankanderson is going to take over this. via fork or dealers choice.


Updated Berkeley CS course links.

Found that the Berkeley CS links were all dead (the youtube video playlist had been taken down).
I checked each of the courses for posted video alternatives. Some have links, other who didn't have alternative links were just removed.

Seems a shame. I wonder why they were taken down? Thanks for your efforts on this.


api-rpc: Added get_producers and get_producer_schedule api call

I think they are needed... If there is a better way to get the producers schedule please let me know

Hi @regisBafutwabo.
Are you going to include @eosio-toppings/api-rpc in your own project?
Adding these two api would help on your project?

Hi @regisBafutwabo.
Are you going to include @eosio-toppings/api-rpc in your own project?
Adding these two api would help on your project?

Yes, it would help me. I would be grateful if you could add them. If you prefer, I can also add the remaining RPC calls from time to time.
Thank You.

@regisBafutwabo
I am going to test it with these two Apis.
And yes, we are glad to have your help ( as well as from EOSIO community ) to making our software better. Please help to add the remaining PRC calls if you could.
We will make a merge on this package after all changed has been made and tested.
Many Thanks!!

@regisBafutwabo
I am going to test it with these two Apis.
And yes, we are glad to have your help ( as well as from EOSIO community ) to making our software better. Please help to add the remaining PRC calls if you could.
We will make a merge on this package after all changed has been made and tested.
Many Thanks!!

Thanks, I will do my best!

This PR has been merged to develop branch, this functionality will be published in the next release


Charset for Russian language

Hi, I have a question like this.
How can I display the names of the players if they are in Russian language?
Inexplicable characters are now displayed instead of names.
Can I use UTF-8 somehow or what should I do?
Thank you in advance!

Hey there!  Hmm, according to the Steam documentation it should return "the current users persona name in UTF-8 format."   So I would think that it would be readable.  I've not seen it give weird responses like that before though.
Try printing the name with: name.to_utf8( ) Let me know if that makes it readable.

Not working :c

The to_utf8() function returns a PoolByteArray. I try to translate it back into text, but nothing changes ...
P.S. I also tried changing the font in game (OpenSans international) :)

Hmm, I am sure that I've used getFriendPersonaName() as well and I was able to display Russian names

The source code is here:
https://github.com/Antokolos/know-everything-os/blob/master/godotsteam.gd
Maybe the problem is with font? Some fonts does not display Russian characters properly. Try the one from our game to be sure:
https://github.com/Antokolos/know-everything-os/blob/master/assets/fonts/roboto.regular.ttf

@Antokolos I don’t know, but nothing helps me ... You can send me your builded Godot engine, maybe the problem is in it and my build system. I can’t even imagine what could be the problem ...

Yes, correct, I use GDNative libraries, they are in the following folder
https://github.com/Antokolos/know-everything-os/tree/master/lib/godotsteam
but it looks like that you've already found it :)

@chrom007   Yeah, as I was falling asleep last night I realized my suggested code would not work.
@Antokolos   Hmm, after testing it, he is right.  I get just ????????????? ????? sent back through the console.  If I print it to a label with your font I get the same thing.  Not sure why that is.  Gonna test some more stuff and see if I can't get it to respond.

@Gramps GDNativeSteam lib is perfect working, all nicknames is displayed. I think its bug in C# or C++ code, but I don’t know where he is. I will also try to fix this, although I am not very strong in C++
All hope for you :D

That's odd.  I presume the code in the GDNative version is the same as the module version; in fact, after reviewing the two, it is identical.  So that's extra strange.

@Antokolos   What version of C++ are you compiling the GDNative version with?

On Windows I've built it using Visual Studio 2015, on Linux it was clang++ (because I've compiled it with use_llvm=yes)

clang++ -v clang version 6.0.0-1ubuntu2 (tags/RELEASE_600/final) Target: x86_64-pc-linux-gnu Thread model: posix InstalledDir: /usr/bin Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/4.9 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/4.9.3 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/5 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/5.5.0 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/6 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/6.5.0 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/7 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/7.4.0 Found candidate GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/8 Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/4.9 Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/4.9.3 Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/5 Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/5.5.0 Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/6 Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/6.5.0 Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/7 Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/7.4.0 Found candidate GCC installation: /usr/lib/gcc/x86_64-linux-gnu/8 Selected GCC installation: /usr/bin/../lib/gcc/x86_64-linux-gnu/7.4.0 Candidate multilib: .;@m64 Candidate multilib: 32;@m32 Candidate multilib: x32;@mx32 Selected multilib: .;@m64

Hmm, it's an older version than I am using.  I think I have Visual Studio 2017 but using the command line tools.  I haven't tested this out on Linux yet; maybe it is working there.   I don't really see why this isn't working.

@Gramps is there any progress?
Or does anyone have any thoughts on this?

I have not yet.  I tried modifying the C++ code on the Windows side and recompiling but it never comes out any differently.  Not sure why the GDNative one would respond differently even though it has the same code base as the module.
I was going to try Linux later this afternoon and see if something there is different.  Should have a response later in the day.

OK, so this is really strange.  On my Linux build printing a name from getFriendPersonaName shows no special characters at all.  It seems it automatically strips them out.  The Steam ID you gave me just returns an empty string.  I used some with friends I have and it just took out any special characters except things like [ { . - > and such.
I'm not sure what the difference between the Linux and Window builds are since they use the same code base.  On Linux I use GCC 8 and Windows uses who knows what.

@Gramps at first it was the same on my Windows, but after that (I don’t remember why) special characters were written. So I think this is not due to the OS
Is it possible to run your library without Godot, and try to display information about the nickname, what will it give out? I think we need to check it first, if everything is fine there, then is it a matter of Godot?

There is sort of a way to do it... but it will still need to display through Godot's console.  Basically you have it print the response before passing it back with either sprintf or printf.  I did try this in Windows and it gave me a different output than what Godot got but they were still both wrong.  When I copied the text exactly from Steam itself, with special characters, Godot was able to print if fine with no issues.
I am going to do some game dev for a bit then I'll get back to messing with it on Linux and see what else I can find out!

;(

Haven't had much luck on the Linux front since last time.  Not really sure what is going on.  Did you compile it yourself or did you use the pre-compiles?  I wonder if it is some locale thing.  I don't imagine it's the actual codebase but maybe something with the way it is compiled.

I tried ready-made compiled solutions and compiled several times myself, but nothing worked.

Hmm, well it must be something.  The code is correct and the same as @Antokolos GDNative scripting so I'm not sure where the problem actually lies.  My Godot seems to print Cyrillic characters fine but names from the getFriendPersonaName() are missing certain special characters and Cyrillic characters.
At this point I will have to enlist outside help.  I'll ask around to see if I can get some answers!

Still searching for some solution.  Gonna try some things I've read and see if that helps!

With Godot 3.2 there still seems to be no change and I have found nothing about character sets that helps.  My only assumption is there is something during compilation that is wrong, as there is no real issue with the Steamworks functions.
I hate this issue being in limbo for so long but still haven't found a solution.

I'm crying. in the same way that I have not tried, everything is unsuccessful ...

Yeah, it is a strange issue.  I presume the problem lies somewhere in the compiling as the function the GDNative and module version us is exactly the same, so the code itself shouldn't be the issue.  Especially since this seems like some kind of Unicode problem.

Holy crap!  After almost a year later I think I have solved the problem.  For whatever reason the string passed back is not wide enough to support Unicode 16, or so it seems.  If pass it to wchar_t first then return it to our Godot string, it will print the Cyrillic characters fine.
Current / non-working code for Cyrillic is just passing the response along to string:
return SteamFriends()->GetFriendPersonaName(userID);
Creating a wchar_t variable first, then passing the response to it, then passing that to string:
wchar_t *username = new wchar_t[256];
mbstowcs(username, SteamFriends()->GetFriendPersonaName(userID), 256);
return username;

It seems like a really hack-y way to solve this problem but it did in fact work.  I presume there is a more elegant solution to this but this is the only thing I have found so far that actually does work.

@Gramps Oh my God! 😂
I kept waiting and finally it happened! I hope this will be included in your next library update.
I think the entire Russian-speaking community is very grateful to you and this fix!
Thank you for continuing to support all questions and errors even after a year ❤

Hey there!  Welcome!  I'm glad it finally got fixed after all this time.  It will definitely be in the next update.  I'm guessing I may need to implement this everywhere a username is acquired.

OK, newest version of GodotSteam is up with the fix for Cyrillic characters!  Uploading pre-compiles too.  Let me know if you find any functions I may have missed!


Snackbar and FloatingActionButton hidden

Snackbar and FloatingActionButton are no longer above the bottom bar. They are hidden behind it.

Hello! I've investigated this issue and I found a workaround. The Snackbar will no longer be overlapped by the BottomBar if you use a CoordinatorLayout as your root layout and bind the BottomBar to the CoordinatorLayout.
If you use a FAB, this will also prevent the Snackbar from overlapping that as well.
Here is my layout:
<?xml version="1.0" encoding="utf-8"?>
<android.support.design.widget.CoordinatorLayout android:id="@+id/ah_rl_root"
    xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical"
    xmlns:app="http://schemas.android.com/apk/res-auto">

    <MyContentCustomLayout
        android:id="@+id/ah_vp_content"
        android:layout_width="match_parent"
        android:layout_height="match_parent"/>

    <android.support.design.widget.FloatingActionButton
        android:id="@+id/fab_btn"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_gravity="bottom|right"
        android:layout_marginBottom="16dp"
        android:layout_marginRight="16dp"
        app:rippleColor="@color/colorGray"
        android:src="@android:drawable/ic_menu_search"
        app:fabSize="normal"
        app:borderWidth="0dp"/>

</android.support.design.widget.CoordinatorLayout>

The Snackbar will be shown above the BottomBar.

@NegreaVlad Where in your layout is the BottomBar?

Hi @Servus7 !
Doesn't the layout in the sample app work for you?
For me there's no problems with the following Snackbar code:
Snackbar.make(findViewById(R.id.myCoordinator), "HELLO", Snackbar.LENGTH_LONG).show();

I cannot see a slim horizontal bar in your picture. But I saw that on other posts, including myself. I've asked how to eliminate it, but removed by roughike.
Could you tell me how to hide/eliminate it please？
Below is the slim horizontal bar example: you can see a bar between the content and the icons.
https://github.com/roughike/BottomBar/issues/436

@roughike You're right. There's no problem with the Snackbar. But the behavior is shy and i want a static bottom bar.

My code for binding the BottomBar is:
        mBottomBar = BottomBar.attach(mRootCL, savedInstanceState); // mRootCL = R.id.ah_rl_root
        mBottomBar.noTabletGoodness();
        mBottomBar.setItems(R.menu.home_activity_bottombar_menu);

        mBottomBar.setActiveTabColor(UIUtils.getColor(R.color.light_blue));

@NegreaVlad I'm using version 2.0.2. BottomBar.attach() ist no longer available.
But your right. The old Version is pushing up the Snackbar and the FAB.

@Servus7 sorry about that, I tested on my code that was outdated.

when used with app:bb_behavior="shifting", snackbar is not visible when shown

In version 2.0.2, the snackbar is visible only if the behavior is "shy".

I can confirm what @gdemiollis  is saying and found a somewhat hacky workaround but would love to see a proper fix in a future release. The snackbar is indeed being drawn behind the bottom bar in 2.0.2 unless the shy behavior is activated.
The code for the shy behavior (collapsing and expanding the bottom bar) is in BottomNavigationBehavior and basically applies a bottom margin to the snackbar so that it appears above the bottom bar. In my case I needed that margin adding behavior but not the collapsing/expanding of the bottom bar.
So I extended BottomNavigationBehavior overriding the methods that would cause the collapsing/expanding but leaving the rest intact.
The hacky bit is injecting this custom behavior into the bottom bar because the bottom bar attaches the behavior to itself somewhere in its onLayout method. So I had to extend BottomBar as well and override onLayout, where I inject my own custom behavior should the super class have changed it.
All the code can be found here: https://gist.github.com/jschwertfeger/86996858d47a76f5247730f4dc7b55b7

@gdemiollis Can use another version for see the SnackBar when i use behavior "shifting" ? I love shifting but i lose all my SnackBar :(

I fixed this by adding bottom margin to SnackBar.
public static void displaySnackBarWithBottomMargin(Snackbar snackbar, int sideMargin, int marginBottom) {
    final View snackBarView = snackbar.getView();
    final CoordinatorLayout.LayoutParams params = (CoordinatorLayout.LayoutParams) snackBarView.getLayoutParams();

    params.setMargins(params.leftMargin + sideMargin,
                params.topMargin,
                params.rightMargin + sideMargin,
                params.bottomMargin + marginBottom);

    snackBarView.setLayoutParams(params);
    snackbar.show();
}

(Credits to http://stackoverflow.com/a/34844669/2897341)
(dp2px example http://stackoverflow.com/a/17410076/2897341)
Set the bottom margin ~same as the height of the bottom bar:
displaySnackBarWithBottomMargin(snackbar, 0, dp(51)); // dp method is converter for dp to px

My bottomBar height is 56, but making it 51 removes the white line between a snackBar and bottomBar (shadow maybe).
I use wrapper for showing a snackbar, so it was easy enough to fix them everywhere (gist).

@alashow the shadow is fixed in the latest version, and in the next version a small fix for when you don't want the shadow will be released

@roughike is this thread still relevant?

I just tried the approaches in this issue with 2.3.1, snackbar still only works if app:bb_behavior is set to shy. Otherwise, the snackbar is never shown.

My app:bb_behaviour is set to shy and my FAB draws behind the BottomBar.

My layout looks like this:
<?xml version="1.0" encoding="utf-8"?>
<RelativeLayout
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:fitsSystemWindows="true"
    android:layout_height="match_parent">

    <android.support.design.widget.CoordinatorLayout
        android:fitsSystemWindows="true"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        tools:context=".activity.MainActivity">

        <android.support.design.widget.AppBarLayout
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
<EMAIL_ADDRESS>
            <android.support.v7.widget.Toolbar
                android:id="@+id/toolbar_main_view"
                android:layout_width="match_parent"
                android:layout_height="?attr/actionBarSize"
                android:background="?attr/colorPrimary"
                app:layout_scrollFlags="scroll|enterAlways"
<EMAIL_ADDRESS>/>

        </android.support.design.widget.AppBarLayout>

        <FrameLayout
            android:id="@+id/main_container"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            app:layout_behavior="@string/appbar_scrolling_view_behavior" />

        <android.support.design.widget.FloatingActionButton
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:layout_gravity="bottom|end"
            android:layout_margin="16dp" />

        <com.roughike.bottombar.BottomBar
            android:id="@+id/navigation"
            android:layout_width="match_parent"
            android:layout_height="50dp"
            android:layout_gravity="bottom"
            app:bb_tabXmlResource="@xml/navigation_tabs"
            app:bb_showShadow="true"
            app:bb_inActiveTabAlpha="0.6"
            app:bb_behavior="shy"
            app:bb_inActiveTabColor="@color/black"
            app:bb_activeTabColor="@color/colorAccent"
            app:bb_activeTabAlpha="1" />

    </android.support.design.widget.CoordinatorLayout>
</RelativeLayout>


Updated Hungarian (Magyar) language file

Updated Hungarian (Magyar) language file.

Thanks it's merged.


The path of the RouteConfig is the same as the alias content, which will cause an infinite loop.

Version
3.0.1
Reproduction link
https://codepen.io/chenfeng163/pen/qJzxGW
Steps to reproduce
The path of the RouteConfig is the same as the alias content
What is expected?
no error
What is actually happening?
infinite loop

The route of alias overrides the route of the path, causing the path and matchAs to be the same. When the url is parsed, an infinite loop occurs.

#2462

I think a warning makes more sense here as I don't see a reason to have a path with the same value as an alias

Yes, but the problem now is that if path and alias are the same, an error will be reported. In #2462 , I have added warning.


#877 Release notes for 10.2.9

Finished release notes - something happened with the push/sync... no idea why it didn't push 2 weeks ago.

Sorry about hurry work from me.
Added toc info and updated index pages.
(Screen dumps has been created over several weeks, so diffs might appear.)


RFC: Even higher-level cache maintenance operations

#40 landed CMSIS-style DCache cleaning and invalidating by address and size, both usize. This maps well to the actual hardware operations but is inconvenient to use.
We could add some convenience methods that operate on normal Rust objects:

clean_dcache_by_slice<T>(slice: &[T]) which passes slice.as_ptr() as usize and core::mem::size_of<T>() * slice.len() to the underlying methods,
clean_dcache_by_ref<T>(obj: &T), which passes T as *const _ as usize and core::mem::size_of<T>() to the underlying methods.
As above for invalidate_dcache and clean_invalidate_dcache

Big safety issue: all the cache methods operate on cache lines of 32 bytes. If you pass something whose size is not a multiple of 32, then some extra bytes after that object will be cleaned or invalidated. In general this is definitely unsafe. Perhaps these methods could panic on non-32-byte-multiple sizes?

If you pass something whose size is not a multiple of 32, then some extra bytes after that object will be cleaned or invalidated. In general this is definitely unsafe.

Why is this unsafe?

The basic problem is that invalidating an object you didn't expect to can lead to writes "going missing" if they hadn't yet been cleaned to main memory, which we can abuse to read uninitialised memory and/or produce invalid values, both of which are explicitly UB.
Really I think this means the existing D-cache invalidate operations, as well as any new invalidate by ref/slice operations, should probably be unsafe. Possibly also I-cache and maybe also clean operations. I'm not sure if you can technically trigger UB with only clean, though.
It's been a few years so I made a quick example. First we clean the cache, writing whatever is leftover in our stack through to main memory. Next we stack allocate a small buffer followed by a bool, but the writes to initialise them haven't been cleaned to main memory, they're only in D-cache. Then we invalidate the buffer, causing the bool to also be invalidated, so on next read it contains an invalid value for a bool (32 in this case), which was leftover in the stack.
#![no_std]
#![no_main]

extern crate panic_halt;
extern crate stm32f7;
use cortex_m_rt::entry;
use cortex_m_semihosting::dbg;

// Proposed new invalidate function
fn invalidate_dcache_by_ref<T>(scb: &mut cortex_m::peripheral::SCB, obj: &T) {
    scb.invalidate_dcache_by_address(obj as *const _ as usize, core::mem::size_of::<T>());
}

// Demonstrate undefined behaviour
fn trigger_ub(scb: &mut cortex_m::peripheral::SCB) {
    // Create a buffer and a bool flag in the stack.
    // Zeroing these variables has gone through to the D-cache but not
    // been cleaned to the main memory.
    let buf = [0u8; 16];
    let flag: bool = false;

    // Printing out their addresses prevents optimiser removing them
    dbg!(&buf as *const _ as usize);
    dbg!(&flag as *const _ as usize);

    // Invalidate the cache line containing `buf`.
    // This _also_ invalidates `flag`, since buf only uses half the cache line.
    // Both buf and flag will be fetched from main memory on next access.
    invalidate_dcache_by_ref(scb, &buf);

    // Cast underlying flag value to a u8 and print its value.
    // It "must" be 0 or 1...
    let flag_value = unsafe { *(&flag as *const bool as *const u8) };
    dbg!(flag_value);
}

#[entry]
fn main() -> ! {

    // Enable D-cache
    let peripherals = cortex_m::peripheral::Peripherals::take().unwrap();
    let mut scb = peripherals.SCB;
    let mut cpuid = peripherals.CPUID;
    scb.enable_dcache(&mut cpuid);

    // Clean current stack through to main memory
    scb.clean_dcache(&mut cpuid);
    cortex_m::asm::dmb();

    // Trigger UB
    trigger_ub(&mut scb);

    loop {
        cortex_m::asm::nop();
    }
}

[src/main.rs:23] &buf as *const _ as usize = 537067432
[src/main.rs:24] &flag as *const _ as usize = 537067451
[src/main.rs:34] flag_value = 32

:zipper_mouth_face:

Wow, thanks for that demonstration! I though the invalidation would ensure to also flush modified cache lines back to RAM, but if it doesn't it makes sense that things might break.


JwtBearerMiddleware instantiation issues in integration tests.

Souldn't  app.UseJwtBearerAuthentication work without explicitly adding UrlEncoder to DI when Integration-testing?
System.InvalidOperationException : Unable to resolve service for type 'System.Text.Encodings.Web.UrlEncoder' while attempting to activate 'Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerMiddleware'.
      Stack Trace:
           at Microsoft.Extensions.Internal.ActivatorUtilities.ConstructorMatcher.CreateInstance(IServiceProvider provider)
           at Microsoft.Extensions.Internal.ActivatorUtilities.CreateInstance(IServiceProvider provider, Type instanceType, Object[] parameters)
           at Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.<>c__DisplayClass3_0.<UseMiddleware>b__0(RequestDelegate next)
           at Microsoft.AspNetCore.Builder.Internal.ApplicationBuilder.Build()
           at Microsoft.AspNetCore.Hosting.Internal.WebHost.BuildApplication()
           at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()
           at Microsoft.AspNetCore.TestHost.TestServer..ctor(IWebHostBuilder builder)
           at ConsoleApplication.TestTest.<Blupp>d__0.MoveNext()
        --- End of stack trace from previous location where exception was thrown ---
           at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
           at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
        --- End of stack trace from previous location where exception was thrown ---
           at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
           at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
        --- End of stack trace from previous location where exception was thrown ---
           at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
           at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

Example test.
namespace X
{
    public class Startup
    {
         public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
         {
            app.UseJwtBearerAuthentication(new JwtBearerOptions());
         }
    }

    public class TestTest
    {
        [Fact]
        public async Task Test()
        {
            var builder  = new WebHostBuilder().UseStartup<Startup>();
            var server = new TestServer(builder);
        }
    }
}

adding this to Startup resolves the issue
public void ConfigureServices(IServiceCollection services)
{
    services.AddScoped<System.Text.Encodings.Web.UrlEncoder>(_ => System.Text.Encodings.Web.UrlEncoder.Default);
}

UrlEncoder is required by the base class AuthenticationMiddleware. services.AddAuthentication() will register it for you.

Thanks

Thank you. This saved the day for me!


Clarify in the docs how to handle special characters in file marks

The documentation here: https://carvel.dev/ytt/docs/latest/file-marks/#available-marks
Shows that you can specify file marks on CLI as follows:
-file-mark '<path>:path=<new-path>'

This means that characters such as ':' and '=' have a special meaning in the notation (they are used to separate different 'parts' of the file mark argument. But if we wanted to use these characters in some of those values as well? And yes it is valid for example to have file names with a ':' or '=' as part of their path/name.
Does the notation allow for escaping these special characters when we don't want them to be interpreted as a 'seperator'? If no, perhaps that is a situation that should be rectified. If yes, then the docs should probably explain how.

Does the notation allow for escaping these special characters when we don't want them to be interpreted as a 'seperator'?  If no, perhaps that is a situation that should be rectified.

it does not today, given that it's very rare use case that was not previously requested. do you have some concrete examples where you are relying on :s and =s in paths? i think if we ever decide to add support for such characters in paths, then we would probably add --file-mark-json flag that allows to specify such values unambiguously.
current state might still be worth documenting though.

I do not have a specific use case. However we have a tool that wraps YTT and passes options/values from the wrapping tool into YTT to filemarks. The question on how to encode special characters thus came up in that context. As it is now, we just pass the options as is and don't do anything about any special characters in them. That may have unpredictable effects but it seems to be the best that we can do. Also I suppose as long as users of our tool avoid aren't using any special characters (which is probably the case) there's no problem.

i think if we ever decide to add support for such characters in paths, then we would probably add --file-mark-json flag that allows to specify such values unambiguously.

What do you think of simple quoting?
--file-mark '"<path>":path=<new-path>'
--file-mark '<path>:"path"=<new-path>'
--file-mark '<path>:path="<new-path>"'

... and combinations thereof.

@kdvolder: two questions:

Did the folks who brought up this question...

The question on how to encode special characters thus came up in that context.

do they have any specific strings in mind?

Would the approach of quoting, noted above, be feasible from your perspective?

Did the folks who brought up this question...

The 'folks' here was just me. Sorry if that wasn't clear.

do they have any specific strings in mind?

Not really. We are just passing through a string that comes from 'some user'. Did not have a particular user in mind, nor any particular use-case that requires specific characters. However when our tool is asked to pass along information a user provided to us as an argument / input, I think it is our tool's responsibility in turn that this string is passed faithfully to ytt. This means essentially that it should work regardless of any particulars about the string in question or whether or not it contained any 'special ' characters. So what I meant when I said that 'the question came up in that context' is that I considered the question 'how do I pass a string faithfully down to ytt, regardless of what might be in that string exactly?'. I did not have an answer to that question so I came here and asked about it.
Nevertheless, I can tell you that the characters I was thinking of specifically where ':' and '=' because they clearly have special meaning in the 'filemark' syntax. And so therefore that probably means that if they are used as 'just characters in a string' they might need to be escaped.

What do you think of simple quoting?

Simpler is better in my view. I.e. from my point of view I just have to know two things to answer my question:

what characters are to be treated specially and need some kind of escaping (e.g. just provide a list of characters that 'need escaping')
what is the mechanism for 'escaping' such characters (e.g. if a character is special you can escape it by preceding it with a \, this implies backslashes themselves are 'special' and need to be escaped by using \\).

I do not particularly care about the exact mechanics, just that they are clearly defined so I can implement an 'escape special characters when passing a string to ytt' function.
From your point of view things may be more complex since you may also need to consider how it interferes with shell escape mechanics (i.e. I can imagine scenarios where you would need double escaping or other 'funny business' because the who escape mechanism interact and overlap). But from my point of view this doesn't matter because the scenario is just one tool (ours) calling another tool (ytt) and we are passing the tool arguments as a string array.
In your proposal, I think toy may have some issues with using quotes to escape things, but that is what the shell does. So if we wanted to use quotes to escape a '=' character we would have to include quotes in the string to escape the '=' but we would have to also tell the shell not to treat those quotes as special so escape them for the shell. I don't even want to think of what happens when we want to escape both kinds of quotes in the same string as well as some special '=' and ':' characters.
Note: In some ways maybe this is more of a 'academic' than a practical issue. As you have rightfully observed, it is questionable whether this causes any real problems in practical / real use-cases. Then again, these kinds of 'improper escape handling' issue are also somewhat notorious for turning into security bugs (SQL injection is probably the most prominent example this kind of thing I guess).
These are just my thoughts on the matter. Feel free to use your own best judgement on whether this is important enough to fix and or what kind of escaping mechanism would fit best with how YTT works, if you do decide to address it.

More random thoughts...  To avoid interaction between 'shell' escapes and 'ytt escapes' (which is rather tricky)... Maybe the simplest solution would be not to have any special characters part of ytt syntax and only use shell escape syntax. That this means essentially is that ytt cannot use ':' and '=' characters within a shell argument as being somehow special. Instead we would have to just rely on the shell to pass the 3 bits of information for a 'file mark' as separate shell arguments.
So instead of something like:
--file-mark 'generated.go.txt:path=gen.go.txt'

You would do instead:
--file-mark 'generated.go.txt' 'path' 'gen.go.txt'

Drawbacks:

less readable / intuirive (i.e. the special characters help somewhat to understand / parse command for human readers, we'd loose that).
this would be a breaking change (unless you keep the old filemark option and provide a new one with a different name using this new syntax

Advantages:

ytt doesn't need to worry about any escaping, neither does another tool passing filemark arguments to ytt
users use shell escape mechanics in the normal way that they are probably familiar with

So what I meant when I said that 'the question came up in that context' is that I considered the question 'how do I pass a string faithfully down to ytt, regardless of what might be in that string exactly?'. I did not have an answer to that question so I came here and asked about it.

Appreciate you bringing it up and having the conversation, @kdvolder. 🙏🏻
I'm better getting a sense of the scope of what you're aiming at: tooling integration.
Given all this, I'm returning to @cppforlife's observation and suggestion:

... i think if we ever decide to add support for such characters in paths, then we would probably add --file-mark-json flag that allows to specify such values unambiguously.

Let's:

not introduce a breaking change or complicate the existing flag;
document that : is not a supported character in paths for this flag (in fact, = works just fine for file mark path values);
recast this issue as a feature request to provide a new flag: one that puts no restriction on path values: #553

In this way, we can prioritize creating a new flag that supports the use case that's at the heart of this issue.

For clarity's sake, we'll close this issue. It's referenced by the recast feature request, so remains discoverable.

FWIW: I think that you made a very reasonable decision and I'm pretty happy about the thoughtfullness with which you have handled this request/ticket. And yes, I can confirm that if such a json or yaml option, would have existed, I would have used that and it would have serve our needs just fine.


Fix workflows operations queued on embedded_ansible

The CrudCommon module was always using the embedded_ansible role which is not required for workflows functionality.  If this role wasn't enabled then these operations would hang.


Tailwind plugin wanders into scss files then crashes on build

When adding included Tailwind plugin to an existing project with sass/scss files in static directory, the plugin on buildStart will walk the static directory in search for css files. However when it bumps into scss files which postcss isn't able to parse, it throws the error below:
error: Uncaught (in promise) CssSyntaxError: <css input>:22:17: Unknown word

Seemingly caused by the line below in the plugin, which matches any files ending with css, including *.scss. Changing exts from "css" to ".css" seemed to solve the issue, but I'm not 100% sure if this was due to a design decision or simply a bug.
https://github.com/denoland/fresh/blob/main/plugins/tailwind.ts#L103

Just checked and this is resolved in Fresh 2 which will be released soon.


Add Best Practise Examples around Typescript.

Simple examples like Why React. Fc is discouraged - See issue (https://github.com/facebook/create-react-app/pull/8177).
Why inline the type declaration should not be used.

Better way to infer types:
// Declaring type of props - see "Typing Component Props" for more examples
type AppProps = {   message: string; };  // use interface if exporting so that consumers can extend
// Easiest way to declare a Function Component; return type is inferred.
const App = ({ message }: AppProps) =>
{message}
;
// you can choose annotate the return type so an error is raised if you accidentally return some other type
const App = ({ message }: AppProps): JSX.Element =>
{message}
;

@Danishkhurshid Great idea! Got started here: https://entermedia-llc.github.io/best-practices/javascript/typescript/
Feel free to add & create a PR


ci: add ci/cd

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@e46ab80). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main        #1   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         1
  Lines           ?         1
  Branches        ?         0
========================================
  Hits            ?         1
  Misses          ?         0
  Partials        ?         0

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 e46ab80...784603f. Read the comment docs.


KR:Notify, Earmuff, GamePad, Tracker

Thank you for the update!

@mirpasec because of your fix for other locale files (ru in particular), my PR got a conflict and wasn't merged in the update. Please don't try to fix the locales that others are already actively working on.


DS-3154 fix javadoc

https://jira.duraspace.org/browse/DS-3154
This actually fixes the bulk of the JavaDoc errors. Only a few remain.

+1 via code review. This is almost all comment fixes, with a few small spelling errors fixed in log statements, etc.

+1 by inspection

+1 by inspection.  Much improvement here.

Merging as this is at +2. I'll add a note to make the few (minor) comment fixes in a followup PR.  @helix84 noted that there seems to be few javadocs issues left, so I'll see if I can quickly resolve those in a followup.


[2.0] Move variables and constants to the head of the file

Move variables and constants to the head of the file

Great @aidantwoods I believe we know have a better code organization to get this project grow and get some framework adapters

Awesome! Thanks for all the great work! 👍


Remove Qiskit IBM Provider API reference

URL to the relevant documentation
https://docs.quantum.ibm.com/api/qiskit-ibm-provider/ibm_provider
Select all that apply

[ ] typo
[ ] code bug
[X] out-of-date content
[ ] broken link
[ ] other

Describe the fix.
qiskit-ibm-provider was deprecated earlier this year and has since been archived. Its API reference page should be removed since it's no longer supported.

We should set up a custom 404 error page that has links to the migration guide https://docs.quantum.ibm.com/migration-guides/qiskit-runtime-from-ibm-provider

@jyu00 will users still be able to retrieve old jobs using the Qiskit IBM Provider package? If not, is there another way they could do that?

@jyu00 will users still be able to retrieve old jobs using the Qiskit IBM Provider package? If not, is there another way they could do that? Just working out what to say in the docs on getting old jobs.

Kind of. qiskit-ibm-provider uses Qiskit Runtime, so users can continue to use it or qiskit-ibm-runtime to retrieve old jobs submitted by qiskit-ibm-provider.
However, jobs submitted by qiskit-ibmq-provider (notice the extra q), which was deprecated 2 years ago, can only be retrieved by qiskit-ibm-provider (for now anyway, I believe IQP plans to disable that path entirely).


Restore the message crediting the Mind the Gap report

I have added acknowledgments to Mind The Gap, and to Arcadia, into the "About" page


Add videosink example

As discussed in periph/devices#26. The example command uses only periph.io and the standard library.

Codecov Report

Merging #3 (c31e01b) into main (19e40d1) will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##           main     #3   +/-   ##
===================================
  Coverage   0.0%   0.0%
===================================
  Files         8      8
  Lines       482    482
===================================
  Misses      482    482

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 19e40d1...c31e01b. Read the comment docs.


Error updating newer version when it's user installed

I tried to upgrade poetry to `0.10.1’
I originally install it with pip3 --user poetry (Python 3.6.5)
$ poetry --version
Poetry 0.9.1

$ poetry self:update
Updating to 0.10.1
  - Getting dependencies
  - Vendorizing dependencies
  - Updating poetry

 [CalledProcessError]
 An error has occured: Command '['/usr/bin/pip', 'install', '--upgrade', '--no-deps', '/tmp/poetry-update-

 FShfiE/poetry-0.10.1-py2.py3-none-any.whl']' returned non-zero exit status 2

 Processing /tmp/poetry-update-FShfiE/poetry-0.10.1-py2.py3-none-any.whl Installing collected packages: poetry   Found
 existing installation: poetry 0.9.1
 Uninstalling poetry-0.9.1:
   Successfully uninstalled poetry-0.9.1

 Rolling back uninstall of poetry Exception: Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/pip/basecommand.py", line 215, in main
    status = self.run(options, args)
  File "/usr/lib/python2.7/site-packages/pip/commands/install.py", line 365, in run
    strip_file_prefix=options.strip_file_prefix,
  File "/usr/lib/python2.7/site-packages/pip/req/req_set.py", line 784, in install
    **kwargs
  File "/usr/lib/python2.7/site-packages/pip/req/req_install.py", line 854, in install
    strip_file_prefix=strip_file_prefix
  File "/usr/lib/python2.7/site-packages/pip/req/req_install.py", line 1069, in move_wheel_files
    strip_file_prefix=strip_file_prefix,
  File "/usr/lib/python2.7/site-packages/pip/wheel.py", line 345, in move_wheel_files
    clobber(source, lib_dir, True)
  File "/usr/lib/python2.7/site-packages/pip/wheel.py", line 316, in clobber
    ensure_dir(destdir)
  File "/usr/lib/python2.7/site-packages/pip/utils/__init__.py", line 83, in ensure_dir
    os.makedirs(path)
  File "/usr/lib64/python2.7/os.py", line 157, in makedirs
    mkdir(name, mode) OSError: [Errno 13] Permission denied: '/usr/lib/python2.7/site-packages/poetry-0.10.1.dist-info'

Yes, there is a lot of improvement to be done in the self:update command, especially detecting if poetry has been installed with --user.

The new installer and the improved self:update command should fix this.


Fix bug when slicing a Dense Tensor with mixed ranges and integers

This fixes a bug when slicing a Dense Tensor, where T[3, 1:2] returns a slice with dimensions (3, 2) instead of (2,).
Fixes #774

Codecov Report

Merging #775 (baa1772) into main (b8f87a7) will decrease coverage by 45.82%.
The diff coverage is 100.00%.

:exclamation: Current head baa1772 differs from pull request most recent head d1a7a0c. Consider uploading reports for the commit d1a7a0c to get more accurate results

@@             Coverage Diff             @@
##             main     #775       +/-   ##
===========================================
- Coverage   77.36%   31.53%   -45.83%
===========================================
  Files          92       91        -1
  Lines       10701    10670       -31
===========================================
- Hits         8279     3365     -4914
- Misses       2422     7305     +4883

Impacted Files
Coverage Δ

src/NDTensors/dense.jl
32.35% <100.00%> (-47.15%)
:arrow_down:

src/Ops/ops_mpo.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/Ops/trotter.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/NDTensors/octavian.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/site_types/boson.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/site_types/qudit.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/site_types/fermion.jl
0.00% <0.00%> (-98.39%)
:arrow_down:

...c/ContractionSequenceOptimization/three_tensors.jl
0.00% <0.00%> (-98.08%)
:arrow_down:

src/mps/projmps.jl
0.00% <0.00%> (-95.35%)
:arrow_down:

src/ITensorChainRules/ITensorChainRules.jl
0.78% <0.00%> (-94.67%)
:arrow_down:

... and 66 more

Continue to review full report at Codecov.

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


[6391]Caching with multithreading

Ticket: BAU-6391
Added decorator for caching data from functions
Added tests

By the way. Changed seats in views for my convenience


ValueError: numpy.ndarray size changed

When I go to run
python ./src/run_benchmarks_train.py -l ./logs/stock_training.log,
I get this numpy ValueError:
Traceback (most recent call last):
  File "/home/u151382/Consolvo/20221027_creditcardfraud/credit-card-fraud-detection/./src/run_benchmarks_train.py", line 15, in <module>
    from utils.training import split_data, DBSCAN_Clustering, lgbm_model_train
  File "/home/u151382/Consolvo/20221027_creditcardfraud/credit-card-fraud-detection/src/utils/training.py", line 11, in <module>
    from lightgbm import LGBMClassifier
  File "/home/u151382/.conda/envs/FraudDetection_stock/lib/python3.9/site-packages/lightgbm/__init__.py", line 8, in <module>
    from .basic import Booster, Dataset, Sequence, register_logger
  File "/home/u151382/.conda/envs/FraudDetection_stock/lib/python3.9/site-packages/lightgbm/basic.py", line 20, in <module>
    from .compat import PANDAS_INSTALLED, concat, dt_DataTable, is_dtype_sparse, pd_DataFrame, pd_Series
  File "/home/u151382/.conda/envs/FraudDetection_stock/lib/python3.9/site-packages/lightgbm/compat.py", line 6, in <module>
    from pandas import DataFrame as pd_DataFrame
  File "/home/u151382/.conda/envs/FraudDetection_stock/lib/python3.9/site-packages/pandas/__init__.py", line 22, in <module>
    from pandas.compat import is_numpy_dev as _is_numpy_dev  # pyright: ignore # noqa:F401
  File "/home/u151382/.conda/envs/FraudDetection_stock/lib/python3.9/site-packages/pandas/compat/__init__.py", line 18, in <module>
    from pandas.compat.numpy import (
  File "/home/u151382/.conda/envs/FraudDetection_stock/lib/python3.9/site-packages/pandas/compat/numpy/__init__.py", line 4, in <module>
    from pandas.util.version import Version
  File "/home/u151382/.conda/envs/FraudDetection_stock/lib/python3.9/site-packages/pandas/util/__init__.py", line 2, in <module>
    from pandas.util._decorators import (  # noqa:F401
  File "/home/u151382/.conda/envs/FraudDetection_stock/lib/python3.9/site-packages/pandas/util/_decorators.py", line 14, in <module>
    from pandas._libs.properties import cache_readonly
  File "/home/u151382/.conda/envs/FraudDetection_stock/lib/python3.9/site-packages/pandas/_libs/__init__.py", line 13, in <module>
    from pandas._libs.interval import Interval
  File "pandas/_libs/interval.pyx", line 1, in init pandas._libs.interval
ValueError: numpy.ndarray size changed, may indicate binary incompatibility. Expected 88 from C header, got 80 from PyObject

I decided to upgrade numpy, and got further incompatibilities by installing the newest numpy.
pip install --upgrade numpy
Requirement already satisfied: numpy in /home/u151382/.local/lib/python3.9/site-packages (1.19.5)
Collecting numpy
  Downloading numpy-1.23.4-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (17.1 MB)
     |████████████████████████████████| 17.1 MB 4.3 MB/s
Installing collected packages: numpy
  Attempting uninstall: numpy
    Found existing installation: numpy 1.19.5
    Uninstalling numpy-1.19.5:
      Successfully uninstalled numpy-1.19.5
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
transformers 4.16.2 requires pyyaml>=5.1, which is not installed.
transformers 4.16.2 requires requests, which is not installed.
transformers 4.16.2 requires tqdm>=4.27, which is not installed.
openvino-dev 2022.1.0 requires defusedxml>=0.7.1, which is not installed.
openvino-dev 2022.1.0 requires pyyaml>=5.4.1, which is not installed.
openvino-dev 2022.1.0 requires requests>=2.25.1, which is not installed.
openvino-dev 2022.1.0 requires tqdm>=4.54.1, which is not installed.
openvino 2022.1.0 requires numpy<1.20,>=1.16.6, but you have numpy 1.23.4 which is incompatible.
openvino-dev 2022.1.0 requires numpy<1.20,>=1.16.6, but you have numpy 1.23.4 which is incompatible.
openvino-dev 2022.1.0 requires numpy<=1.21,>=1.16.6; python_version > "3.6", but you have numpy 1.23.4 which is incompatible.
openvino-dev 2022.1.0 requires pandas~=1.1.5, but you have pandas 1.5.1 which is incompatible.
openvino-dev 2022.1.0 requires scikit-learn~=0.24.1, but you have scikit-learn 1.1.3 which is incompatible.
openvino-dev 2022.1.0 requires scipy~=1.5.4, but you have scipy 1.9.3 which is incompatible.
Successfully installed numpy-1.23.4


Tester takes place in the flow

My modal have some custom styling to make it fixed and centered in the viewport:
  .bbm-wrapper {
    display: table;
    width: 100%;
    height: 100%;

    position: fixed;
    top: 0;
    right: 0;
    bottom: 0;
    left: 0;
    z-index: 10;

    text-align: center;
  }

    .bbm-modal {
      display: table-cell;
      vertical-align: middle;
    }

It works great, however, the  element you create gets a relative position and this make it takes spaces in the flow with my styling (in Firefox & IE at least, doesn’t seems to happen in Chrome).
I’ve made some tests and using an absolute position seems to fix the problem without breaking anything apparently.
Would it be possible for you to use an absolute positioning on the  element? You probably have more examples to test with to be sure it still works in any case.
Thanks!

Closing due to age, but feel free to reopen.


Unable to use 2 same TabItem values

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
I have a situation where I need to demonstrate the json code of two types, NoShow and Unscheduled.
<Tabs groupId="codeGroup">
<TabItem value="json" label="NoShow">

```json
{  ...
    "MerchantInitiatedTransactionType": "NoShow",
}
```
</TabItem>

<TabItem value="json" label="Unscheduled">

```json
{  ...
    "MerchantInitiatedTransactionType": "Unscheduled",
}
```
</TabItem>
</Tabs>

I am unable to use json for both. I need to use the same language across both scenarios representing them in tabs.
Reproducible demo
No response
Steps to reproduce

In you markdown file, use the following code

<Tabs groupId="codeGroup">
<TabItem value="json" label="NoShow">

```json
{  ...
    "MerchantInitiatedTransactionType": "NoShow",
}
```
</TabItem>

<TabItem value="json" label="Unscheduled">

```json
{  ...
    "MerchantInitiatedTransactionType": "Unscheduled",
}
```
</TabItem>
</Tabs>

Expected behavior
Expected to have two tabs "NoShow" and "Unscheduled" with the json code.
Actual behavior
Getting an error Error:
Docusaurus error: Duplicate values "json" found in <Tabs>. Every value needs to be unique.
Your environment

Public source code:
Public site URL: locally built
Docusaurus version used: 2.0.x
Environment name and version (e.g. Chrome 89, Node.js 16.4): Not relevant
Operating system and version (e.g. Ubuntu 20.04.2 LTS): MacOs, yet not relevant

Self-service

[ ] I'd be willing to fix this bug myself.

What is your intended effect? Show two tab items simultaneously?

I understand Docusaurus doesn't support it. But can you add it as an enhancement please? Because my case is valid here.
Yes I want to add two tabitems but of the same language

value is not for displaying the language. value is purely a unique identifier to identify the tab. Underlyingly, Docusaurus is going to record that "whenever I see a tab with the group codeGroup, I'm going to display the item with value json, regardless of its label". What do two identical values mean?

Are you reading our docs section "Multi-language support code blocks"? Would it help if the examples look like this?
<Tabs>
<TabItem value="1" label="JavaScript">

```js
function helloWorld() {
  console.log('Hello, world!');
}
```

</TabItem>
<TabItem value="2" label="Python">

```py
def hello_world():
  print("Hello, world!")
```

</TabItem>
<TabItem value="3" label="Java">

```java
class HelloWorld {
  public static void main(String args[]) {
    System.out.println("Hello, World");
  }
}
```

</TabItem>
</Tabs>

Ok, got it. I can display both of them regardless of their values. This makes sense. Thanks! Closing this case


invoice incorrect when optional element  is present

taxcurrencycode is 0..1 but if it's 1, the validation will fail
in https://www.zugferd-community.net/forum/viewtopic.php?f=4&t=66
this zip contains a "invalid" sample file and a "fixed" one
zi.zip. I checked against the ZF2.1 TA that TaxCurrencyCode is 0..1

I can confirm this possible misbehavior.
When adding those lines in xml
	 <ram:TaxCurrencyCode>EUR</ram:TaxCurrencyCode>       <ram:InvoiceCurrencyCode>EUR</ram:InvoiceCurrencyCode>
the validator throws the error

If the VAT accounting currency code (BT-6) is present, then the Invoice total VAT amount in accounting currency (BT-111) shall be provided.

even though the value is provided in the xml further below
<ram:TaxTotalAmount currencyID="EUR">133.65</ram:TaxTotalAmount>
Deleting the TaxCurrencyCode line in the xml file makes the xml valid again. But it should be valid in the first place in my understanding.

After a litle research: this is also in the CEN schematrons as CII-SR-314 and apparent the  TaxCurrencyCode is (only?) to be specified if it differs from the invoice currency code.


C# method snippets

Current coverage is 81.52% (diff: 95.74%)

Merging #415 into master will increase coverage by 0.08%

@@             master       #415   diff @@
==========================================
  Files           185        185
  Lines          6918       6961    +43
  Methods           0          0
  Messages          0          0
  Branches        899        903     +4
==========================================
+ Hits           5634       5675    +41
- Misses         1015       1016     +1
- Partials        269        270     +1

Powered by Codecov. Last update e606f95...9d996a9

Haven't looked at the implementation at all yet. Just first feedback on the generated code. When it's looking more likely to compile, I'll pull it and see what else is missing.


This module breaks ":to" links in nuxt v3

Environment

Operating System: Linux
Node Version:     v18.17.1
Nuxt Version:     3.7.4
CLI Version:      3.9.0
Nitro Version:    2.6.3
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, modules
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://github.com/florealcab/NuxtI18nBug
Describe the bug
If you are in a page, and the URL ends with a slash (example: /sub/toto/), then any link on the page with the "to" parameter has no more effect (example : Index) : click action is ignored, nothing in logs client side or server side.
If you just remove the trailing slash, all works well.
To reproduce it with the repo I give:
Go to http://localhost:3000/sub/test/ and try to click on the link "Go to index". Nothing happen. No logs.
Now, just remove the trailing slash (url become http://localhost:3000/sub/test). Click to the link "Go to index" again: it works!
Additional context
This bug appears with the 8.0.0-rc3 version and does not exists before (tested with 8.0.0-beta10, rc1 and rc2), and always after (tested until rc5 which is the latest version for now.
This bug seems to be related to this module. I install all and then just downgrade or upgrade this module. I try to use the rc5 version and manually downgrade vue-i18n and vue-i18n-routing to the same version they had with the 8.0.0-beta10 of this module: the bug is still here.
But I don't know what causes this on this module, by the way...
Logs
No response

I had the same issue, it was causing issues when navigating to pages via an alias (which is defined via definePageMeta)

I'm getting the same problem. With each version, there are only more and more bugs.

Downgrade to ^8.0.0-beta.11

I'm getting the same problem. With each version, there are only more and more bugs.

@Frallen
We're working hard to make v8 stable. If you run into bugs, report them and provide a minimal reproduction. Or better yet, open a PR to fix the issue, all contributions are appreciated!

Sorry, when this fix release?


The Correct way to add additional ClusterRoleBindings for Operand Service Accounts

Type of question
Are you asking about community best practices, how to implement a specific feature, or about general context and help around the operator-sdk?

Best Practice
General Context
Help with operator-sdk in relation to OLM

Question
What did you do?
The syndesis operator is migrating to operator-sdk 1.0.0+ and as such is changing to the correct file layout and bundle generation. The operator binary has up until now generated a CSV manifest via custom golang code which we would like to replace with $(KUSTOMIZE) build $(KOPTIONS) manifests | operator-sdk generate bundle .... In short, we wanted to use the operator-sdk to get to this CSV.
What did you expect to see?
Since the syndesis operator uses the service-account syndesis-operator and its operands use a couple of extra service-accounts syndesis-server and syndesis-public-oauthproxy, we have ClusterRoles and ClusterRoleBindings providing the relevant permissions for each one. Since the OLM installs the operator with ClusterAdmin privileges, it seemed sensible to add all of these service-accounts to the clusterpermissions section of our CSV.
What did you see instead? Under which circumstances?
Having migrated to operator-sdk bundle generation, the ClusterRole permissions for syndesis-operator are included in the CSV but the ClusterRoles and ClusterRoleBindings for the other service-accounts are presented in their own resource files alongside the CSV. The problem is that the ClusterRoleBinding resource files specify a namespace in the subject portion, for the service-account, that is not appropriate for the user installing the operator.
To avoid this issue, I've refactored the ClusterRoleBindings for the additional service-accounts into the operator infrastructure, ie. the operator is now responsible for their installation and not the CSV. This has meant that the operator now requires the ClusterRoleBinding GET/UPDATE/DELETE permissions at the cluster-level.
This seems, architecturally, to be correct since:

The operator is in charge of creating the operands and therefore their service-accounts;
The namespace of the operands will only be known at the point of their installation and not when the operator is installed;

My reluctance (hence this question) concerns the necessary addition of privileges to the operator:

Is it best practice for an operator to be creating ClusterRoleBindings in this context?
Should an operator and its operands only use one service-account throughout?
How else is it possible to apply cluster-level permissions to an operand's service-account that is namespace-specific & is different to the operator's own service-account?

We discussed this at the OLM Dev triage meeting this morning.
The CSV describes, the permissions (ClusterRole/Bindings and Role/Bindings) and ServiceAccount that the Operator requires.  This is the Operator's Security Domain.
For Operands, you have two choices:

Your Operator can dynamically create the ClusterRole/Bindings and Role/Bindings and ServiceAccounts for the Operand services when reconciling the Operand CR.  With this option, the Operand's permissions must be a subset of the Operator's permissions, because Kubernetes will prevent privilege escalation.
Your Operator can statically create the ClusterRole/Bindings and Role/Bindings and ServiceAccounts for the Operand services by including them in the bundle manifests directory.  These will be created by OLM and can have a different Security Domain of the Operator (doesn't need to be a subset).   However, these resources are only created in the Operator namespace.  If you are supporting anything other than OwnNamespace mode, then these resources are not copied into the other namespaces in the OperatorGroup and you should consider option 1 instead.

Wearing my Security hat:
Customers are increasingly wanting to have a narrow permissions granularity as possible.  In my experience, they do not like AllNamespaces install mode because the Operator reaches into Namespaces (tenants).  The only time this is acceptable is if the Operator is really designed to extend the entire cluster.
Customers also do not like ClusterRoleBindings since they also have cross-namespace or cluster-wide permissions.
That said:

Create RoleBindings instead of ClusterRoleBindings wherever possible.
The Operator's ServiceAccount and Operand ServiceAccount should be separate (different security domains)
It's OK for each Operand in a namespace to share the same ServiceAccount.
Create the Operand resources from the Operator dynamically for maximum flexibility and narrow security domain.
Use the CSV ONLY for the Operator's permissions and serviceaccount.  Not the Operand.

Really useful information. Thanks very much.

Good question.  I personally never tried this, as I never recommend creating permissions this way.
From looking at the code:
https://github.com/operator-framework/operator-lifecycle-manager/blob/master/pkg/controller/operators/catalog/operator.go#L1778-L1791
It appears that the only way this would work is if the namespace and serviceaccount are well-known (hard-coded).
@dinhxuanvu do you have any background on the use case for creating a ClusterRoleBinding via the bundle manifests?
Looking at the public doc:
https://olm.operatorframework.io/docs/tasks/creating-operator-manifests/
There is no official support for any RBAC manifests although the code applies them.

From our view, we like the cleanliness of having the RBAC laid down up front by whatever does the deployment (be that OLM, or a traditional approach to deployment), rather than having to "roll up" all the permissions needed by the operands into the operator ontop of the permissions the operator actually needs.
By moving RBAC setup out of the operator we end up with an operator that has a much smaller set of permissions that are only what the operator itself needs, rather than a superset of everything needed by the operand(s).  With the principle of assigning least permission everywhere, we think it makes sense to be able to restrict the operator in this way.

It appears that the only way this would work is if the namespace and serviceaccount are well-known (hard-coded).

Yeah, that was our conclusion too.  We've ended up keeping namespace-scoped RBAC static/in OLM, and resorting to dynamic for the cluster-scoped permissions we couldn't lock down to the namespace.


What's missing to transform ChatWithRob.ai POC into a successful product?

Hi,
We've built an early prototype of a LLM based tool that can help you practice chatting in any foreign language.
The early version of the tool (alpha) can be seen here:
https://app.bluejay.boullier.org/
Capabilities:
Current Prototype:

Can come up with new and (hopefully) intersting conversation subject in any language.
Can sustain a conversation in any language.
Will still try to understand what you were trying to say if you make mistake and help correct those.

Coming Soon:

Ability to send notification (like whatsapp) so it can re-engage with the user if there is no activity.
Translate the message from the AI Agent from the foreign language into your native language.
Listen to the messages from the AI Agent (on top of text).
Talk to the AI Agent (instead of typing).

Target User:

B2C.
Anyone who needs to practice chatting/talking in a foreign language.

User Story:

I have some free time (in the bus, on the train) let's chat with Rob so I can practice my [enter your language here].

Key Questions:

Monetization strategy, paid vs free features, pricing.
Marketing: how do we "get the word out".

Things To Keep In Mind:

This is still not a product.
We still have a lot of things to finalize to get to a MVP...

Current Screenshots:
These are screenshots of v0.14.1 (far from a polished product...).

We've released the latest version of the "ChatWithRob.ai" proof of concept app:
New functionalities:

translate any message in your native language
Listen to the message in the foreign language.

Still early stage (obvioulsy) as we are mainly focusing on the "capabilites" i.e. orchestrating several LLM Agents and traditional AI ML model to make it work.
Sreenshots for the latest version (in light mode)


Is --volumes-from supported in marathon

Hi
Is --volumes-from supported in marathon , I want to mount a volume in the app container from another container ? This means that second container should be present in all the hosts which has the first container marathon should consider these two container as a group. Is that possible to do this ? If what version of marathon supports this and where is the documentation on that ?

This is not possible right now.
Mesos plans to add support for pods, which will behave similar to kubernetes pods and share volumes across containers.
Once Mesos supports pods, we'll also add support to Marathon.


Many blocks are missing from page response

After a recent update, many blocks are no longer returned for our pages. One page dropped 50-100 blocks from the end of the page.
In page bc7f910ef23b421487ca21b959c67c49, block 771f85de-f188-46e6-90d6-9b2dd814ba10 is listed in the content of the page, but the block content is not actually included.
We noticed that by deleting early blocks on the actual Notion page, missing blocks can be fetched by the API — in other words, blocks at the end of the page are more likely to be missing.

Hmmm, comparing https://www.notion.so/Copy-of-2020-Q4-bc7f910ef23b421487ca21b959c67c49 with https://react-notion-x-demo.transitivebullsh.it/bc7f910ef23b421487ca21b959c67c49, I'm not able to see any difference for the part of the document in question.
Have you tried using https://github.com/NotionX/react-notion-x?
See attached screenshot.

This probably has to do with notion-api-worker not fetching as many async blocks as notion-client does https://github.com/NotionX/react-notion-x/blob/master/packages/notion-client/src/notion-api.ts#L71

Wow! NotionX looks awesome! Trying now 👍

I confirmed that notion-client is fetching all of the blocks.

Hello, any update on this?

Hey @JhumanJ + @dvdsgl
We just merged this now, thanks @JhumanJ !


Bug/shaking off multi input output breaks

This is to fix #224 (Shaking off operators breaks connection if Operator has more than one output)
Don't pull yet, are still some bug. I commented more on the issue #224

The code looks good and it's already working much better than before but I found this glitch when pressing undo after shaking off an op with two outputs:
https://user-images.githubusercontent.com/1732545/208317004-148e140b-6aea-4339-88f5-275b254bb901.mp4
I currently looking into this and will merge after I fixed it.

The only thing missing was that the connect fetch also needed to change for the SlotIds

I merged this PR to dev.


How to declare Object.observe?

I can't seem to find a proper declaration of Object.observe.
StackOverflow suggests this:
interface Object {
    observe(beingObserved: any, callback: (update: any) => any) : void;
}

Object.observe(o, observer);

but it generates the following error with tsc 1.4.1:
error TS2339: Property 'observe' does not exist on type 'ObjectConstructor'.

This makes sense since observe is a static property of the Object class, but since we cannot merge class declarations, I don't see how to declare it as such.
Of course I can use a type-unsafe workaround, but it's frustrating:
(<any>Object).observe(…)

Is there a better way?

You have to use the interface, not the variable:
interface ObjectConstructor  {
  observe(beingObserved: any, callback: (update: any) => any) : void;
}

Thanks, this works.
I though I had tried this and it didn't know about ObjectConstructor, but now it works.
It would be good to have it in the library though.

Based on https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/observe it's not standardized or widely supported enough to put in lib.d.ts at this time.


Support for Managed Identity for Azure resources

Added new error handling and keyword ActiveDirectoryMsi

Coverage increased (+0.05%) to 75.449% when pulling eb2fc63a2aeb1f10dceade50c9d7f5832b68b22c on yitam:adMSI into 3679b48df2e307b0a2f1f833450963f5e8ef93ed on Microsoft:dev.

Codecov Report

Merging #875 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #875   +/-   ##
=======================================
  Coverage   80.37%   80.37%
=======================================
  Files          25       25
  Lines        7649     7649
=======================================
  Hits         6148     6148
  Misses       1501     1501

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 3679b48...eb2fc63. Read the comment docs.


Javascript is required to navigate

File: index.md, CC @johndmulhausen
The navbar nav-sidebar does not work without javascript, and no indication is displayed to alert the user that javascript is required to make use of the site.
I would suggest implementing proper fallback methods (or at the very least a notice) so that users who, by choice or otherwise, are not running client-side scripts can still make use of the site.
Reproduction steps:

Disable javascript.
Navigate to any Docker documentation page.
Attempt to use the sidebar.
Notice that despite the sidebar being responsive (with CSS), navigation requires javascript (it is indeed possible to achieve this with HTML+CSS, so requiring javascript is overly restrictive).

It seems like this affects the Docker website more extensively than I expected.
Notice that javascript is also required to view the second resultset on https://hub.docker.com/explore/#.

Sorry, but this is working as intended and for many reasons we can never support a JS-free version of the docs. :(


add defaults for funding element

for funder ID in our EDIutils package we can use the wikidata urls
the majority of projects the funder is USBR:
USBR - https://www.wikidata.org/wiki/Q1010548
Sometimes there are contributions from state funds:
CDWR - https://www.wikidata.org/wiki/Q5020440
CDFW - https://www.wikidata.org/wiki/Q5020421
please investigate schema for multiple funders in structured funding information
make any necessary updates to template workbook and instructions based on what you find out

I will take over this task.

I have a couple ideas on how to do default funding. It depends on how many details we want to add on to each funding source. I will check with Sadie to decide on this in the next couple of days.

I have a couple ideas on how to do default funding. It depends on how many details we want to add on to each funding source. I will check with Sadie to decide on this in the next couple of days.


Added Tarjan's Algorithm.

📥 Pull Request
Description
Added Tarjan's Algoritm
Fixes #523
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Hey @ajay-dhangar
I done all changes.


Failed to connect to the Docker daemon

Description of problem:
After install docker by running curl -sSL https://test.docker.com/ | sh , and DOCKER_HOST environment variable is not set in my shell, run docker cmd, but return :
$ docker ps
Cannot connect to the Docker daemon. Is the docker daemon running on this host?
$ service docker status
docker start/running, process 28966
docker version :
Client:
 Version:      1.9.0-rc5
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   9318004
 Built:        Tue Nov  3 05:59:27 UTC 2015
 OS/Arch:      linux/amd64
Cannot connect to the Docker daemon. Is the docker daemon running on this host?

docker info :
Cannot connect to the Docker daemon. Is the docker daemon running on this host?

uname -a :
Linux docker 3.13.0-43-generic #72-Ubuntu SMP Mon Dec 8 19:35:06 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux

Environment details (AWS, VirtualBox, physical, etc.):OpenStack Cloud instance
Actual Results: docker daemon cannot work
Expected Results: docker daemon can work Ok

Could you please try running
sudo su -
service docker start
docker images

See if that works. And if it does, please add your user to the docker user group like this =>
sudo usermod -aG docker <username>

@Skarlso ,  I did "sudo usermod -aG docker ubuntu" , but not log out and back in to take effect. :smile:

Hah, I couldn't count on one hand how many times that happened to me as well. :D :+1:

Try with
sudo gpasswd -a ${USER} docker

To anyone who stumbles across this issue in the future.
I just spent a bit of time debugging this issue in a few of my servers. I now realize ssh ControlPersist multiplexing makes the whole 'log out log back in' thing not work the way you think it does. It makes sense why this happens, I just never thought about it happening like this before.
To fix this
$ ssh -O check -S<EMAIL_ADDRESS>server.com
Master running (pid=14920)
$ ssh -O stop -S<EMAIL_ADDRESS>server.com
Stop listening request sent.

Then log back into your server.

I'm having the same problem but neither of the solutions provided worked.
service docker status

Redirecting to /bin/systemctl status  docker.service

● docker.service - Docker Application Container Engine
   Loaded: loaded (/usr/lib/systemd/system/docker.service; disabled; vendor preset: disabled)
   Active: active (running) since Wed 2015-12-16 18:19:56 EST; 17h ago
     Docs: https://docs.docker.com
 Main PID: 5181 (docker)
   Memory: 2.3M
   CGroup: /system.slice/docker.service
           └─5181 /usr/bin/docker daemon -H fd://

When I try to execute other commands, it shows "cannot connect to the docker daemon"
# docker version
Client:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 13:25:01 UTC 2015
 OS/Arch:      linux/amd64
Cannot connect to the Docker daemon. Is the docker daemon running on this host?

Has any one tried other troubleshooting? I'm running redhat/softlayer.

Solved by manually starting the daemon. For some reason, the service is not actually starting it.
ran  docker daemon & and now it's working fine. will have to investigate further to see why the service is not fully starting it.

We got exactly the same issue. Also for us "/usr/boin/docker -d" worked but we'd much prefer to be fixed when restarting via upstart.

thanks @Skarlso

On Fedora 22
Installing Docker:
$> curl -fsSL https://get.docker.com/ | sh
After installing Docker:
A user needs to be added to the docker group.
$> sudo usermod -aG docker
The docker daemon needs to be started
$> sudo service docker start
You can set the daemon to start at boot
$> sudo chkconfig docker on
You can verify the docker service is running
$> service docker status
And one last final check
$> docker run hello-world

The issue is neatly documented in Docker Documentation.
My environment:
VirtualBox
OS: CentOS-7 -x86_64-Minimal

After adding user to the group, execute the following command.
sudo chkconfig docker on
At first, I couldn't view the server config details on the execution of docker version , and I received the same error "Cannot connect to the Docker daemon. Is the docker daemon running on this host?"
After executing sudo chkconfig docker on , everything works super fine :)

On the latest Arch Linux, the only way I can get docker to work is to
sudo systemctl stop docker
sudo docker daemon &
Then it works appropriately.

@hopeseekr was that an upgrade from an existing installation, or a fresh install? Anything in the logs? Any modifications you made to the systemd unit file?

Fresh install on a new Arch Linux install that I setup yesterday.  I don't see any docker-specific log files in /var/log.

I actually absolutely hate systemd, and I have NO IDEA how to debug or even look at what commands a systemd service actually runs. God! I miss OpenRC so much!

@hopeseekr haha 🙊
you can probably get some information from sudo systemctl status docker and journalctl -xe -u docker. Did you use the package from our apt repositories or the arch-linux repos?

Yes, I used the official packman -S docker route.
After a fresh reboot:
● docker.service - Docker Application Container Engine
   Loaded: loaded (/usr/lib/systemd/system/docker.service; enabled; vendor preset: disabl
   Active: active (running) since Tue 2016-05-24 09:28:34 CDT; 12s ago
     Docs: https://docs.docker.com
 Main PID: 7302 (docker)
    Tasks: 19 (limit: 512)
   Memory: 15.4M
      CPU: 134ms
   CGroup: /system.slice/docker.service
           ├─7302 /usr/bin/docker daemon -H fd://
           └─7308 docker-containerd -l /var/run/docker/libcontainerd/docker-containerd.so

May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.277017655-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.289658653-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.295186622-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.341449086-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.388887470-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.389074804-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.389156085-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.389239746-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.393658963-05:00" level=inf
May 24 09:28:34 libran systemd[1]: Started Docker Application Container Engine.

journalctl:
-- Subject: Unit docker.service has begun start-up
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit docker.service has begun starting up.
May 24 09:28:33 libran docker[7302]: time="2016-05-24T09:28:33.271562128-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.277017655-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.289658653-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.295186622-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.341449086-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.388887470-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.389074804-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.389156085-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.389239746-05:00" level=inf
May 24 09:28:34 libran docker[7302]: time="2016-05-24T09:28:34.393658963-05:00" level=inf
May 24 09:28:34 libran systemd[1]: Started Docker Application Container Engine.
-- Subject: Unit docker.service has finished start-up
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit docker.service has finished starting up.
--
-- The start-up result is done.

Yes, I used the official packman -S docker route.

Yes, those are packages maintained and supported by arch linux; it's possible that they ship with different options and configuration.
From the output you gave, it looks like there's something incorrectly set in the systemd unit-file, or there's a daemon.json present somewhere; the daemon that's started through systemd is listening on <IP_ADDRESS>:2375;
API listen on <IP_ADDRESS>:2375

(note that that is a dangerous configuration, if your host is accessible from the internet; it gives unprotected access to the API through port 2375 of your computer)
When you start it manually, it's using the defaults, and correctly listening on a local socket only;
INFO[0001] API listen on /var/run/docker.sock

So;

check if the systemd unit for docker has the right command-line options set for docker. you can see the unit-files we ship in the official packages here; https://github.com/docker/docker/tree/master/contrib/init/systemd
it's possible there's a drop-in (override) file installed somewhere that overrides the default settings. see https://docs.docker.com/engine/admin/systemd/#custom-docker-daemon-options

For ubuntu 14.04 on virtual box vm with docker installation
sudo groupadd docker
sudo gpasswd -a ${USER} docker
sudo service docker restart
Now, restart the OS. This solution is well explained here
https://jee-appy.blogspot.com/2016/02/installing-docker-in-virtual-box.html

I experiment the same issue on Ubuntu Xenial (and on the previous version on ubuntu).  Although the daemon is running, I cannot connect to it with my regular user (which is part of the docker group). Sometimes I restart, and it magically works! No idea why....

I got same issue on Cent OS 7 which I installed docker by yum. I tried all @xecronix mentioned here but still can't fix it.
At last I found I should run docker after sudo. The hint doesn't mean docker daemon isn't running...

I got the same issue in CentOS Linux release 7.2.1511 (Core)  which I installed by "https://get.docker.com/ | sh".  I tried "sudo docker daemon &", and the issue is fixed.

In my case, if I check:
sudo service docker status
I can see the daemon is running. Running :
sudo service docker restart
Apparently solves the problem (I don't know why).

On ubuntu 14.04 for me workaround is not simply run docker (not service, but client) as sudo. Before run "docker run --rm busybox true", for example, move to sudo bash by typing "sudo su". In sudo bash docker is working fine.

fully restart the os, not just logging out and back in. was the solution for me.

The use group changes doesn't apply immediately when a user is added in a group (the user need to log off and log on again for the changes to apply), but it is better to just restart the OS.

@Skarlso: can you please add to your answer that the user must log out and back in for the solution to work? Thanks!

@dandv Done.

@Skarlso: awesome, thanks! I'll delete my now obsolete comments from this too-long thread.

@wenchma: can you please correct the typo in the title of this issue, so searches can find it more easily? Also, "but not log out and back in to take effect." -> "but had to log out and back in."

No problem man. :-) glad I could help out folks. :-)

On my latest Mac OSX Sierra,
I also got the error, after I install Docker Toolbox, when I run docker on my terminal:
git:(master) ✗ docker version
Client:
 Version:      1.12.2
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   bb80604
 Built:        Tue Oct 11 17:00:50 2016
 OS/Arch:      darwin/amd64
Cannot connect to the Docker daemon. Is the docker daemon running on this host?

Based on the number of people giving a thumbs up to @Skarlso's comment about
sudo usermod -aG docker <username>

maybe there should be check for that so you could say:

You need to be in the 'docker' group

instead of the misleading advice

Cannot connect to the Docker daemon. Is the docker daemon running on this host?

@johshoff docker 1.13 will give a more informative error message if the client fails to connect due to permissions error; see this pull request; https://github.com/docker/docker/pull/26797

I get on my AWS instance following stuff:
time="2017-02-01T12:34:46.999769865Z" level=info msg="libcontainerd: new containerd process, pid: 2737"
time="2017-02-01T12:34:48.004600950Z" level=fatal msg="Error starting daemon: error initializing graphdriver: devicemapper: Non existing device docker-docker--pool"

What should I do then?
P.S. Full log

@Suncatcher This Issues is closed. If you have a problem, please open a new one.

Dear all, just in case this would happen to solve you issue ...
I did all the abode, but only running docker daemon in interactive mode solved my problem.
When running the daemon like this, I got the missing information:

Error starting daemon: error initializing graphdriver: /var/lib/docker contains several valid graphdrivers: aufs, devicemapper; Please cleanup or explicitly choose storage driver (-s )

Previously I was using the old docker from Ubuntu 14.04 and upgraded it using the official upstream docker repository...
After specifying the correct devince mapper (aufs) a lot of checks were performed for existing images and everything was fine afterwards.
I hope this will help others :-)

What means  running docker daemon in interactive mode?
Could you please describe the exact steps you did?

Should we fix the typo in the issue name? *Failed

Should we fix the typo in the issue name? *Failed
Definitely!

There underlying problem is the docker group isn't created ; so a usermod -G user docker will also fail

Dunno where the problem is, but on standard Amazon linux Docker is installable like a charm, so I recommend to use standard.
But to ship ECS-optimized Linux AMI (yes-yes, pay attention to these letters - it's container-optimized) - to ship it without ability to being configured for Docker is a bullshit!
It's simply a shit! *ucking Amazon is trying to make money from nowhere and it's frustrating.
 They make me to buy their paid support plan, I see no other reason.

Had the same issue in AWS-hosted Archlinux.
Fixed via:
pacaur -S docker-machine
docker-machine env default
sudo systemctl start docker

centos 6.6 ,docker 1.7.1
> yum install device-mapper-event-libs
> sudo service docker restart

it's worked for me

I had the same problem on Arch Linux (Docker package version:  1:17.03.1-1). I modified this on /usr/lib/systemd/system/docker.service and I got it working fine after that.
#ExecStart=/usr/bin/dockerd -H fd://
ExecStart=/usr/bin/dockerd

Just follow below steps to resolve the issue.
$:sudo groupadd docker
$:sudo gpasswd -a ${USER} docker
$:sudo service docker restart
$:newgrp docker

Thank you RakeshMsr.
You save my hard working and day.

Type:
$: dockerd
If you see a message: "Please upgrade your kernel to 3.10.0 or newer.", upgrade your linux kernel.


Simplify the code by method SharedPreferences#contains

Exists method SharedPreferences#contains:
https://developer.android.com/reference/android/content/SharedPreferences#contains(java.lang.String)
Then you can simplify the code of the following functions:
https://github.com/freeotp/freeotp-android/blob/67f1c1c2261adb67db1aee9f980f89c0194207ed/mobile/src/main/java/org/fedorahosted/freeotp/TokenPersistence.java#L79
https://github.com/freeotp/freeotp-android/blob/67f1c1c2261adb67db1aee9f980f89c0194207ed/mobile/src/main/java/org/fedorahosted/freeotp/TokenPersistence.java#L103-L110
In general it is incorrect compare String like as this:
if (key == restore_uuid) {

Thank you for the recommendation, PRs are welcome.


Remove Dagger from the Kythe repository [phab:D2099]

Created by ronshapiro at 2017-12-12 17:06:41:
The dagger plugin is now available at https://github.com/google/dagger/commit/de6e30cb81c2135364c87990f3c5251d269d13cd in com_google_dagger//java/dagger/internal/codegen:kythe.
If you'd prefer that I keep the WORKSPACE intact and register the plugin similar to cl/178767858, I can do that.

schroederc accepted this revision at 2017-12-12 17:08:54

schroederc closed via commit 2df6b4ff5bc1fa805fa692b3e17c94a8bc0f87c7 at 2017-12-12 17:10:29


Storybook: Set up demo stories for all packages

Demo stories should be small, focused and meant to be the primary point of exploration.
TODO: create subtasks

outdated


Remove reliance on getTileAtPoint for wrapping

Fix #1065.
I have some tests issues on my computer. I am not sure what is going on. Let's see if travis complains.

Hugh, travis has no log for that build.

Ok travis is now showing the same error as I have on my computer.
I will investigate later.

This wrapping thing is turning out to be quite the issue! Thank you for diving into it.

Ok, the tests failures were unrelated. It looks like we are firing the tile-drawn event faster than before.
I still have tests failures on chrome and firefox:

Multi-Image: Transparent image on top of others (different failure on chrome and firefox)
navigator on firefox only

Interesting... I'm not getting any test failures on Mac Chrome or Firefox with the latest master or with this patch. Are you getting the errors consistently? And it's the same with this patch and with master?

I wonder, is there any kind of unit test we can add to cover the #555 case?

nteresting... I'm not getting any test failures on Mac Chrome or Firefox with the latest master or with this patch. Are you getting the errors consistently? And it's the same with this patch and with master?

Ok, so the Firefox ones where because I had the browser zoom set to 90%.
The chromium one happens on master as well. When seeing #1073 I thought that might fix it but apparently not...

I wonder, is there any kind of unit test we can add to cover the #555 case?

I think adding tests to TiledImage._getCornerTiles should do it. I will do that later.

Done! Ready for me.

Awesome… Thank you for adding the tests (and for fixing the issue in the first place!)
Can you file an issue for the Chrome test failure you're seeing?


Loop not working

Hi,
Great plugin, only problem I have is that my video is not looping. I added data-video-options attribute and set loop to true, but still no loop. Did I forget something in the code?
Regards,
Ali

Hi. Reencode it: http://diveintohtml5.info/video.html#miro.
Or Or http://stackoverflow.com/questions/8088364/html5-video-will-not-loop.


Dropdown for github URL

It would be streamline the process of submitting a project to the app if a list of the users repos were presented so they didn't have to have memorized the url or open a new tab to find it.

Moving work to here from #4.

Did we type this at the exact same time?

I believe we did. Maybe this is what destiny feels like.

@zachlatta should we close this issue because we don't really use shipped right now?
/cc @MaxWofford

Fine with this for the time being.

My bad, didn't mean to close the issue

Closing because we're deprecating the repo.


Option to disable proxying requests

Fixes #222.
Add a flag named -dp or --disableproxy to disable the use of proxy servers when scrapping tweets.

@haseebelahi
This looks like an useful addition, but I think it is better to rename to argument to use_proxy and set the default value to True, instead of the way it is implemented not. It does not look nice, to pass the negative of an argument.

@taspinar
Hi, I got a problem with proxy as well so I'm interested in this issue.
If the argument is renamed to use_proxy, wouldn't it be confusing that when you set the flag for use_proxy the proxy will be disabled?
It would be fine if the default value is False since activating the flag = using the proxy, but I can understand that it would be better to keep using proxy as the default behavior.


Update orchestration readme with QC and compliance check sections

Leaving as draft PR until relevant code is merged in.

Now ready for review after #38


Using crate or version

I assume this is the most up to date repository for Mango, but there is no version nor crate that I found. Did I overlook something or is that something up for grabs

yeah this is not published in any crate. The only way to include this source code is to use the git. How important is this for you?

3.3 solves it