Wrong collection type in Set assertion

Summary
The assertion error displays the wrong collection type, as noticed by @Patouche.
Example
import org.assertj.core.api.Assertions;
import org.junit.jupiter.api.Test;

import java.util.Set;
import java.util.stream.Collectors;
import java.util.stream.Stream;

class AssertJTests {

    @Test
    void iterable_bad_name() {
        final Set<String> actual = Stream.of("toto", "tata").collect(Collectors.toSet());
        Assertions.assertThat(actual).contains("tutu");
    }
}

The error is as follows:
java.lang.AssertionError:
Expecting ArrayList:
 <["toto", "tata"]>
to contain:
 <["tutu"]>
but could not find the following element(s):
 <["tutu"]>

ArrayList should not be displayed in the error message.

The issue happens due to:
https://github.com/assertj/assertj-core/blob/8912e7324fab207e2863891d499cd111eb10f129/src/main/java/org/assertj/core/internal/Iterables.java#L342
introduced in a06f16c500c317fb7f07c45c73c6c6db4e848058 to address non-restartable iterables.
The wrong error message can probably be fixed with some refactoring, but I am wondering if creating a copy makes sense in case of Collection instances.
@assertj/core thoughts?

I thinks it make sense as an iterator can be consume only once.
The simplest way to solve this issue will be probably to provide the class to the org.assertj.core.error.ShouldContain#shouldContain method.
If you want, I already solve the test I've written before .

Thanks @Patouche, what I meant is with Collection instances we probably don't need to use an iterator and create a new array list to perform the contains assertion, we could simply delegate to Collection#contains

Mmmh, not sure we can do that with the ComparisonStrategy.
The Collection#contains method will use the equals methods of the iterable elements. The Iterables#comparisonStrategy will not be used anymore if we change that with contains method.

@Patouche I agree with you.
The change was made to support iterable that can be traversed only once by an iterator (a use case a bit far fetched but why not).
Passing the actual type to the error message is a good solution IMO.

Passing the actual type to the error message is a good solution IMO. WDYT @scordio ?

Looks good to me, we might still improve the part of the copy for performance reasons. I'll have a look at it.

The change was made to support iterable that can be traversed only once by an iterator (a use case a bit far fetched but why not).

Yes, I understand the use case but I wonder how this can work when chaining some assertions. For example :
class SampleTests {
    @Test
    void sample() {
        Iterable<String> iterable = createSinglyIterable("val-1", "val-2")
        assertThat(iterable)
            .contains("val-1")
            .contains("val-2");
    }
}

Maybe I'm wrong but I think this should fail on the contains("val-2") line as the iterator was already consumed on the previous line.

Looks good to me, we might still improve the part of the copy for performance reasons. I'll raise a separate issue to look into it.

Yes, this is a little bit out of the scope of this PR but maybe instead of newArrayList we can use IterableUtil.toCollection in the Iterables#contains which will prevent the copy you're talking when the iterable is already a Collection. In all others cases, an ArrayList may be recreated.

class SampleTests {
    @Test
    void sample() {
        Iterable<String> iterable = createSinglyIterable("val-1", "val-2")
        assertThat(iterable)
            .contains("val-1")
            .contains("val-2");
    }
}

There could be ways to have this assertion pass (since you don't have to iterate until the end to find the first value.
But I think the following should fail for sure:
class SampleTests {
    @Test
    void sample() {
        Iterable<String> iterable = createSinglyIterable("val-1", "val-2")
        assertThat(iterable)
            .contains("val-2")
            .contains("val-1");
    }
}

#justmy2cents

@fbiville would you mind elaborating on why the second example should fail?
Although I understand the single iteration constraint, I'm concerned that the order of the assertions would lead to different results which might be nonobvious from user perspective.
Specifically, following your example:
assertThat(createSinglyIterable("val-1", "val-2")).contains("val-1"); // succeeds

assertThat(createSinglyIterable("val-1", "val-2")).contains("val-2"); // succeeds

assertThat(createSinglyIterable("val-1", "val-2"))
  .contains("val-1"); // succeeds
  .contains("val-2"); // succeeds or fails?

assertThat(createSinglyIterable("val-1", "val-2"))
  .contains("val-2"); // succeeds
  .contains("val-1"); // fails

assertThat(createSinglyIterable("val-1", "val-2")).contains("val-1", "val-2"); // succeeds

assertThat(createSinglyIterable("val-1", "val-2")).contains("val-2", "val-1"); // succeeds or fails?

It might be we should just copy all the elements in a collection during assertThat execution in case the input is an Iterable, so no assumption is taken about the underlying implementation. I tend to think that the side effect (if there is a need for it) should be consistent and not depending on how the assertions are written/executed in the test.
As mentioned before, we are talking about a niche use case so I'm actually wondering if we should take any action to address it, considering that there is no issue raised about the current behavior.


PR: Migrate RunConfig to Plugin with new API

Description of Changes

[x] Wrote at least one-line docstrings (for any new functions)
[x] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #13099
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @goanpeca

Hello @goanpeca! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/run/confpage.py:

Line 50:33: E128 continuation line under-indented for visual indent
Line 51:33: E128 continuation line under-indented for visual indent
Line 53:33: E128 continuation line under-indented for visual indent
Line 54:33: E128 continuation line under-indented for visual indent
Line 56:33: E128 continuation line under-indented for visual indent
Line 57:33: E128 continuation line under-indented for visual indent
Line 112:9: E303 too many blank lines (2)

Hi @dalthviz, this one is ready for QA testing. Thanks!

This is ready for review @ccordoba12

@dalthviz and @ccordoba12 this one is ready for review.

one quick question so I understand things better: why do you think it's important to create a plugin for this config page? Since the plugin API for Run is not used anywhere in this case, it seems unnecessary to me.

Again the idea of generalizing all these things is to be able to have a single way to handle all the things. All plugins are loaded in the same way no matter what. This also applies to config pages.
This "run" plugin is a plugin that depends on the Editor AND the Console so that it can do things, so I think having it as a separate plugin (besides simplifying code in preferences/ and in main window) opens the door to handle in a cleaner way how the console interacts with the editor to run stuff. (like the current implicit dependency the ipyconsole shellwidget has on the editor). This is the plugin that should be in charge of all the Run actions (not the editor). Also the core plugin would override the run action (that would be handled by this plugin). All these things will become apparent with the editor migration.
That is why all the "global preferences" should become a plugin.

Ok, thanks for the clarification.


isLoading indicator on all the hooks on the auth client

i am making a request to add a loading indicator on the signUp and signIn hooks cause it helps the user experience when loading indicator is shown when any functions is run on the client

Since those are just promises, tanstack-query's useMutation can achieve the same result.
No need to add it here IMHO.
Here's how I would do it:
import { useMutation } from '@tanstack/react-query';
import { signIn } from '@/lib/auth-client';

const useSignIn = () => {
  return useMutation({
    mutationFn: async (credentials: { email: string; password: string }) => {
      return await signIn(credentials);
    },
  });
};

// Usage in a component
const SignInComponent: React.FC = () => {
  const { mutate: signIn, isLoading, isError, error } = useSignIn();

  const handleSignIn = () => {
    signIn({ email<EMAIL_ADDRESS>password: 'password' });
  };

  return (
    <button onClick={handleSignIn} disabled={isLoading}>
      {isLoading ? 'Signing In...' : 'Sign In'}
    </button>
  );
};

TLDR: use tanstack

you can do what @Loa212  recommended here or you can also have your own loading state and set it on a callback.
const [loading, setLoading] = useState(false)

await signIn.email(
     {
         email,
         password
     },
     {
          onResponse: () => {
                setLoading(false);
           },
          onRequest: () => {
                 setLoading(true);
          },
          onError: (ctx) => {
                toast.error(ctx.error.message);
          },
      }
 )

I tried the tanstack approach but there is a issue associated with that ,
which is you can't do the same with socials cause it need callbackurl which
will not be redirected with tansack query and also with email, there is a
issue with when signing up with email it is complicated . the way I will
use Bereket way but having to tackle with usestate, I just wish we could
have the same time done like tansack query which we have the mutate,
isloading, data, error it will be a good developer experience and also for
the user
On Tue, Oct 15, 2024, 5:39 PM Bereket Engida @.***>
wrote:

you can do what @Loa212 https://github.com/Loa212 recommended here or
you can also have your own loading state and set it on a callback.
const [loading, setLoading] = useState(false)
await signIn.email(
{
email,
password
},
{
onResponse: () => {
setLoading(false);
},
onRequest: () => {
setLoading(true);
},
onError: (ctx) => {
toast.error(ctx.error.message);
},
}
)
—
Reply to this email directly, view it on GitHub
https://github.com/better-auth/better-auth/issues/180#issuecomment-2414509240,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5RGP2XYFP5JJ57SKQPM52TZ3VANJAVCNFSM6AAAAABP7ODJLSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJUGUYDSMRUGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

You can do it like this:
const {data, isPending, error} = useMutation({
    mutationFn: async () => {
         return await client.signIn.social({
           provider: "github",
           callbackURL: "/dashboard",
          })
       },
  });

I just wish we could have the same time done like tansack query which we have the mutate, isloading, data, error it will be a good developer experience and also for the user

This would require us to turn all the client methods into hooks, which we're not planning to do because we want the client to remain agnostic and consistent across different frameworks.


FEATURE REQUEST: Automatically center window

When working with multiple displays, the window isn't centred across displays. Let's say it is first launched on the MacBook screen and when launched on the other screen thereafter, it's launched not centred.
Although the option exists in the options menu, it'd be nice to have it do that automatically.

This is a known bug. I’m working on it as we speak!

Fixed in v1.5.1. You’ll probably have to center the window through the menu one more time.
And of course, do tell me if the issue persists please.
Thank you for your continued interest in my app 🙏

Awesome work! Seems ok for now, will keep monitoring.
Thx for an awesome app!


ipc/ipc.go: sometimes a werid dealock for every proc

Hi.
Sorry for an old and weird problem I have.
I meet a problem with an older syzkaller version, like https://github.com/google/syzkaller/commit/cb976f63e0177b96eb9ce1c631cc5e2c4b4b0759.
I developed from this version and added some new functions for research.

I'm curious, why the go can't execute at handshake to let all the proc runs correctly.
	wp.Close()
	// Note: we explicitly close inwp before calling handshake even though we defer it above.
	// If we don't do it and executor exits before writing handshake reply,
	// reading from inrp will hang since we hold another end of the pipe open.
	inwp.Close()

	if c.config.UseForkServer {
		if err := c.handshake(); err != nil {
			return nil, err
		}
	}
	tmp := c
	c = nil // disable defer above
	return tmp, nil
}

I just added another Queue to store interesting seeds(progs) and execute them for more power. I don't know why this would cause such a weird situation.
What could be the cause of this problem and why stop like above?
I also can't figure out the root cause;
I'll provide more details for debugging if needed.
Thanks!

Hi @xrivendell7,
This happened 700 commits ago. pkg/ipc and the gate logic does not exist whatsoever. I would suggest to update to HEAD.

Hi @dvyukov,
I know this this happend long time ago and syz-manager/syz-fuzzer have refactored totally.
However, It's hard to transplant my older code to HEAD now.
So, I mean, have we encountered similar problems before, or is there a better way to troubleshoot this problem?
Because this problem has really bothered me for a long time, and it happens so often that I am very frustrated to handle it.
Thanks!

I don't remember any similar symptoms.  Maybe something was fixed at some point, you may check commits to pkg/ipc after you base commit.


Possibility to include validators

Possibility to include validators as follow:
# [schema.yaml]
key: include('my_validator')

---
my_validator: map(int(), num())

Validators can be included inside list/map/any objects, but can't be defined recursively.

Hi @blopker, I have finished to implement the new functionality and did a PR. Please let me know if something isn't alright and feel free to edit what I did!

Very cool. I'll check it out this week.
You should rebase off the current Yamale master. It looks like the changes from the last PR are still in your branch.

Sorry for this... I just did the rebase, normally only changes from this PR should appear.

Thanks for your submission. We decided to use the implementation from #60 for this item.


Can't compile native modules due to iojs errors

VSCode Version: 1.26.1
OS Version: Arch Linux

Steps to Reproduce:
All the steps were run in a clean instalation

yarn install

Output:
error /build/code/src/code/node_modules/native-keymap: Command failed.
Exit code: 1
Command: node-gyp rebuild
Arguments:
Directory: /build/code/src/code/node_modules/native-keymap
Output:
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| linux | x64
gyp http GET https://atom.io/download/electron/v2.0.5/iojs-v2.0.5.tar.gz
gyp http 200 https://atom.io/download/electron/v2.0.5/iojs-v2.0.5.tar.gz
gyp http GET https://atom.io/download/electron/v2.0.5/SHASUMS256.txt
gyp http 200 https://atom.io/download/electron/v2.0.5/SHASUMS256.txt
gyp info spawn /usr/bin/python2
gyp info spawn args [ '/usr/lib/node_modules/node-gyp/gyp/gyp_main.py',
gyp info spawn args   'binding.gyp',
gyp info spawn args   '-f',
gyp info spawn args   'make',
gyp info spawn args   '-I',
gyp info spawn args   '/build/code/src/code/node_modules/native-keymap/build/config.gypi',
gyp info spawn args   '-I',
gyp info spawn args   '/usr/lib/node_modules/node-gyp/addon.gypi',
gyp info spawn args   '-I',
gyp info spawn args   '/build/.node-gyp/iojs-2.0.5/common.gypi',
gyp info spawn args   '-Dlibrary=shared_library',
gyp info spawn args   '-Dvisibility=default',
gyp info spawn args   '-Dnode_root_dir=/build/.node-gyp/iojs-2.0.5',
gyp info spawn args   '-Dnode_gyp_dir=/usr/lib/node_modules/node-gyp',
gyp info spawn args   '-Dnode_lib_file=/build/.node-gyp/iojs-2.0.5/<(target_arch)/iojs.lib',
gyp info spawn args   '-Dmodule_root_dir=/build/code/src/code/node_modules/native-keymap',
gyp info spawn args   '-Dnode_engine=v8',
gyp info spawn args   '--depth=.',
gyp info spawn args   '--no-parallel',
gyp info spawn args   '--generator-output',
gyp info spawn args   'build',
gyp info spawn args   '-Goutput_dir=.' ]
gyp info spawn make
gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
make: Entering directory '/build/code/src/code/node_modules/native-keymap/build'
  CXX(target) Release/obj.target/keymapping/src/string_conversion.o
  CXX(target) Release/obj.target/keymapping/src/keymapping.o
  CXX(target) Release/obj.target/keymapping/deps/chromium/x/keysym_to_unicode.o
  CXX(target) Release/obj.target/keymapping/src/keyboard_x.o
In file included from /build/.node-gyp/iojs-2.0.5/src/node.h:63,
                 from ../src/keymapping.cc:6:
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:15: error: unterminated #ifndef
 #ifndef INCLUDE_V8_H_

In file included from /build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:26,
                 from /build/.node-gyp/iojs-2.0.5/src/node.h:63,
                 from ../src/keymapping.cc:6:
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8config.h:294:20: error: ‘inline’ can only be specified for functions
 # define V8_INLINE inline __attribute__((always_inline))
                    ^~~~~~
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:8873:3: note: in expansion of macro ‘V8_INLINE’
   V8_INLINE static internal::Object* IntToSmi(int value) {
   ^~~~~~~~~
In file included from ../src/keymapping.cc:6:
/build/.node-gyp/iojs-2.0.5/src/node.h:97:8: warning: attribute ignored in declaration of ‘struct v8::internal::SmiTagging<8>::uv_loop_s’ [-Wattributes]
 struct uv_loop_s;
        ^~~~~~~~~
/build/.node-gyp/iojs-2.0.5/src/node.h:97:8: note: attribute for ‘struct v8::internal::SmiTagging<8>::uv_loop_s’ must follow the ‘struct’ keyword
/build/.node-gyp/iojs-2.0.5/src/node.h:100:1: error: expected unqualified-id before ‘namespace’
 namespace v8 {
 ^~~~~~~~~
/build/.node-gyp/iojs-2.0.5/src/node.h:106:1: error: expected unqualified-id before ‘namespace’
 namespace node {
 ^~~~~~~~~
In file included from /usr/include/features.h:428,
                 from /usr/include/bits/libc-header-start.h:33,
                 from /usr/include/stdint.h:26,
                 from /usr/lib/gcc/x86_64-pc-linux-gnu/8.2.0/include/stdint.h:9,
                 from /build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:19,
                 from /build/.node-gyp/iojs-2.0.5/src/node.h:63,
                 from ../src/keymapping.cc:6:
/usr/include/assert.h:64:1: error: expected unqualified-id before string constant
 __BEGIN_DECLS
 ^~~~~~~~~~~~~
In file included from ../src/keymapping.cc:6:
/build/.node-gyp/iojs-2.0.5/src/node.h:197:1: error: expected unqualified-id before ‘namespace’
 namespace node {
 ^~~~~~~~~
In file included from /usr/include/c++/8.2.0/map:60,
                 from ../src/keymapping.cc:7:
/usr/include/c++/8.2.0/bits/stl_tree.h:75:1: error: expected unqualified-id before ‘namespace’
 namespace std _GLIBCXX_VISIBILITY(default)
 ^~~~~~~~~
In file included from /usr/include/c++/8.2.0/map:61,
                 from ../src/keymapping.cc:7:
/usr/include/c++/8.2.0/bits/stl_map.h:66:1: error: expected unqualified-id before ‘namespace’
 namespace std _GLIBCXX_VISIBILITY(default)
 ^~~~~~~~~
In file included from /usr/include/c++/8.2.0/map:62,
                 from ../src/keymapping.cc:7:
/usr/include/c++/8.2.0/bits/stl_multimap.h:64:1: error: expected unqualified-id before ‘namespace’
 namespace std _GLIBCXX_VISIBILITY(default)
 ^~~~~~~~~
In file included from ../src/keymapping.h:10,
                 from ../src/keymapping.cc:9:
/build/.node-gyp/iojs-2.0.5/deps/uv/include/uv.h:27:8: error: expected unqualified-id before string constant
 extern "C" {
        ^~~
In file included from ../src/keymapping.cc:9:
../src/keymapping.h:15:1: error: expected unqualified-id before ‘namespace’
 namespace vscode_keyboard {
 ^~~~~~~~~
../src/keymapping.cc:11:1: error: expected unqualified-id before ‘namespace’
 namespace vscode_keyboard {
 ^~~~~~~~~
../src/keymapping.cc:30:1: error: expected ‘}’ at end of input
 } // namespace vscode_keyboard
 ^
In file included from /build/.node-gyp/iojs-2.0.5/src/node.h:63,
                 from ../src/keymapping.cc:6:
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:8864:34: note: to match this ‘{’
 template <> struct SmiTagging<8> {
                                  ^
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:8872:3: error: expected ‘}’ at end of input
   }
   ^
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:8808:20: note: to match this ‘{’
 namespace internal {
                    ^
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:8872:3: error: expected ‘}’ at end of input
   }
   ^
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:64:14: note: to match this ‘{’
 namespace v8 {
              ^
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:3305:19: warning: inline function ‘void* v8::Object::GetAlignedPointerFromInternalField(int)’ used but never defined
   V8_INLINE void* GetAlignedPointerFromInternalField(int index);
                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /build/.node-gyp/iojs-2.0.5/src/node.h:63,
                 from ../src/keymapping.h:9,
                 from ../src/keyboard_x.cc:6:
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h: In static member function ‘static void v8::V8::SetAllowCodeGenerationFromStringsCallback(v8::DeprecatedAllowCodeGenerationFromStringsCallback)’:
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:10176:19: warning: cast between incompatible function types from ‘v8::DeprecatedAllowCodeGenerationFromStringsCallback’ {aka ‘bool (*)(v8::Local<v8::Context>)’} to ‘v8::FreshNewAllowCodeGenerationFromStringsCallback’ {aka ‘bool (*)(v8::Local<v8::Context>, v8::Local<v8::String>)’} [-Wcast-function-type]
           callback));
                   ^
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h: In static member function ‘static void v8::V8::RemoveGCPrologueCallback(v8::GCCallback)’:
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:10221:53: warning: cast between incompatible function types from ‘v8::GCCallback’ {aka ‘void (*)(v8::GCType, v8::GCCallbackFlags)’} to ‘v8::Isolate::GCCallback’ {aka ‘void (*)(v8::Isolate*, v8::GCType, v8::GCCallbackFlags)’} [-Wcast-function-type]
       reinterpret_cast<Isolate::GCCallback>(callback));
                                                     ^
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h: In static member function ‘static void v8::V8::RemoveGCEpilogueCallback(v8::GCCallback)’:
/build/.node-gyp/iojs-2.0.5/deps/v8/include/v8.h:10228:53: warning: cast between incompatible function types from ‘v8::GCCallback’ {aka ‘void (*)(v8::GCType, v8::GCCallbackFlags)’} to ‘v8::Isolate::GCCallback’ {aka ‘void (*)(v8::Isolate*, v8::GCType, v8::GCCallbackFlags)’} [-Wcast-function-type]
       reinterpret_cast<Isolate::GCCallback>(callback));
                                                     ^
make: *** [keymapping.target.mk:102: Release/obj.target/keymapping/src/keymapping.o] Error 1
make: *** Waiting for unfinished jobs....
make: Leaving directory '/build/code/src/code/node_modules/native-keymap/build'
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/lib/node_modules/node-gyp/lib/build.js:262:23)
gyp ERR! stack     at emitTwo (events.js:126:13)
gyp ERR! stack     at ChildProcess.emit (events.js:214:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)
gyp ERR! System Linux 4.18.5-zen1-1-zen
gyp ERR! command "/usr/bin/node" "/usr/bin/node-gyp" "rebuild"
gyp ERR! cwd /build/code/src/code/node_modules/native-keymap
gyp ERR! node -v v8.11.4
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok

Note: It doesn't always fail on this module, but it always fails due to errors in iojs' source.
Why is iojs being used?? How can I make the build system use my local node installation?

Apparently, you have to specify the arch using --arch for this to work. Great error handling :ok_hand:.

Did you install the prerequisites?
see https://github.com/Microsoft/vscode/wiki/How-to-Contribute#prerequisites

Yes, I do.

@joaomoreno do you have any idea. I know that you use Arch Linux from time to time.

What yarn version are you using?

The latest in the repos, 1.9.4.

Does it work if you call yarn with a new HOME variable? Did you try git clean -xfd . and retry?

That shouldn't affect anything as I build in a clean chroot (clean vanilla acrh instalation) but I think I found the issue here. If I invoke yarn with any argument, it fails. Is this intended (deliberately unsupported) or a bug?
Right now it's faiiling because the visual studio marketplace is down. I am not patching product.json so this is weird.
[17:37:29] Starting 'vscode-linux-x64-min'...
internal/streams/legacy.js:59
      throw er; // Unhandled stream error in pipe.
      ^

Error: Request https://marketplace.visualstudio.com/_apis/public/gallery/extensionquery failed with status code:503
    at Request._callback (/build/code/src/code/node_modules/gulp-remote-src/index.js:70:33)
    at Request.self.callback (/build/code/src/code/node_modules/gulp-remote-src/node_modules/request/request.js:186:22)
    at emitTwo (events.js:126:13)
    at Request.emit (events.js:214:7)
    at Request.<anonymous> (/build/code/src/code/node_modules/gulp-remote-src/node_modules/request/request.js:1081:10)
    at emitOne (events.js:116:13)
    at Request.emit (events.js:211:7)
    at IncomingMessage.<anonymous> (/build/code/src/code/node_modules/gulp-remote-src/node_modules/request/request.js:1001:12)
    at Object.onceWrapper (events.js:313:30)
    at emitNone (events.js:111:20)

Not sure what is going on here. This happened this week with @dbaeumer and it worked to simply clean everything: git clean -xfd. Have you tried it?

Yes, I did. I also did a new clone of the repository and still the same.

The iojs one. Last one beats me. Can you repro in other machines?

It works on my machine. It just doesn't work when compiling in a clean chroot (clean Arch Linux installation), which is required to build the packages for the repos.

I'm going to have to close this, sorry for that. I'd be interested in knowing what the problem is, if you ever find it, though.

How am I going to be able to solve without help from the actual team? I would think you would be interested in vs code be moved to an official repo, this way your users wouldn't be put at risk by having to go to alternative sources to install it. Do you have any guidelines on how to act on these situations or you just think this is worth solving? IMO this is shows an enormous disrespect for your users. Don't know if that's what you trying to do but it's certainly where it's going.

Your issue is about yarn/node_gyp/etc, not about VS Code. It doesn't hit users, it hits you because you are trying to compile VS Code in a chroot environment. Usually, we do try and help out people with these kind of issues, but we simply can't address every single  environment which isn't one of the officially supported ones.

You clearly don't get what a chroot is. We build in a chroot to isolate problems. What I mean by clean chroot is simply a fresh Arch Linux installation.
I suspect the issue here is that some native dependencies were missing and I had everything listed in your build guide.
And this does hit the users as I am required to build official packages in a clean installation, so you're basically denying to help in my efforts to make vscode available in the official arch repos.
Either way, I was able to build 1.27.1 (without --production) with the following instructions.
https://git.archlinux.org/svntogit/community.git/tree/trunk/PKGBUILD?h=packages/code
The only issue now is that VS Code always starts in the app folder. Do you have any ideas on why this is happening? I assume it's the way electron starts apps.

So you think it was one of the 'electron' 'libsecret' 'libxkbfile' dependencies?

This is how we do it in the product: https://github.com/Microsoft/vscode/blob/master/resources/linux/bin/code.sh

Are you interested in solving this? This is the issue preventing the app from being moved to the official Arch Linux repos. This affects a ton of users, not just Arch Linux's ones but also the users from distributions that rely on our repos, ex.: Manjaro, Antergos, etc..

That's why we distribute a ZIP package. As an Arch user myself, I know that is not ideal. But we (VS Code team) have to draw the line on what targets we support. We simply can't do everything.

This issue doesn't seem to hard to fix with your help. You don't even need to make changes in the upstream, I can apply the patches locally before generating the package.

How can I help?

Thank you! Probably the first step would be to find the lines where vscode spawns new processes.
You can have a look at Atom's patch if you like. VS Code's one will probably be drastically different.
https://git.archlinux.org/svntogit/community.git/tree/trunk/use-system-electron.patch?h=packages/atom

I'm not going to do exploratory work on a platform we don't officially support. Let me know if there's something specific I can help with.

I am not asking for you to do the exploratory work. Just ping person responsible for the loaders and them if they know where this happens.

@joaomoreno can you tell me if you set the desktopName in the js code? Right now the main problem is that the app is being launched but it has the electron icon and when pinned, electron is the one effectively pinned.

That's odd. How does the .desktop file look like?
I didn't set the desktopName variable until last Friday (https://github.com/Microsoft/vscode/commit/3eac4d8e3e4c766d375e26f4411d37d0b1672db3) when I implemented URL handling for Linux. I think electron uses that key only for that purpose.

I used your model: https://github.com/Microsoft/vscode/blob/master/resources/linux/code.desktop
These are the replacements I do: https://git.archlinux.org/svntogit/community.git/tree/trunk/PKGBUILD?h=packages/code#n63

Hm :thinking: and you do you initially pin it? Or how do you initially run it?

I ran it.

From the command line? From Gnome's launcher? From rofi?

I am using plank.


Faker::PhoneNumber.area_code and .exchange_code returning nil

Both of these methods are returning nil instead of numbers. Faker::PhoneNumber.subscriber_number seems to be working fine.
2.4.0 :005 > Faker::PhoneNumber.area_code
 => nil
2.4.0 :006 > Faker::PhoneNumber.exchange_code
 => nil
2.4.0 :007 > Faker::PhoneNumber.subscriber_number
 => "1320"

@aarona Faker::PhoneNumber.area_code and Faker::PhoneNumber.exchange_code methods are available only for US locale. please check here. Other than US local, It will return nil.

 I had to learn the hard way about setting Faker::Config.locale to get that to work. Maybe the documentation where it says "US Only" should add reference to this setting? Also, Canada's locale should probably support this as well since they have the exact same format as the US?

You can submit a PR to add the missing info to the yml files

I believe I may do that, thanks!


Add new mod: Felps.SorrisoRonaldo

Closes #773
https://github.com/7Felps/SorrisoRonaldo

Hey @7Felps . Would you mind improving the description a bit?
Also, this isn't that big of a deal, but keep in mind that the text in the "about" section of your github repo will be used as the mod's description on the website and mod manager. So you might want to make it clearer than just "outer wilds mod".

Also, you're missing a release zip here: https://github.com/7Felps/SorrisoRonaldo/releases/tag/v.0.1.0
You need to upload your zipped mod as a release file

Done! Thanks for the help.


is there a stats function for enemies?

what version are you using? the latest version already has a stats function for enemies
npm install genshin-db@latest


Load sequences issue on Linux with 1.1.1

I am not able to load image jpg images or even sequences on Fedora39 with 1.1.1 it is saying that it cannot open, same thing with verison 1.1.0
Working fine loading exrs.
Update:
I just tested the same with version 1.09 and it is working fine with the same files.

I am not able to load image jpg images or even sequences on Fedora39 with 1.1.1 it is saying that it cannot open, same thing with verison 1.1.0 Working fine loading exrs.
Update: I just tested the same with version 1.09 and it is working fine with the same files.

I was able to reproduce the issue, when dragging files from Nautilus to mrv2.  Is that how you are loading them?  Or is there any other way that don't load either (command-line, File->Open->Movie or Sequence, etc)?

Here's a beta of v1.1.2 that should fix the issue.  There was a conflict with the FLTK JPEG library and the one used in tlRender:
.deb:
https://mega.nz/file/nCJTCIQL#PsVVT8-SZTmwfeMu1ZrR05RP9ts2JFOFrBzinSLROsE
.rpm:
https://mega.nz/file/3PwzDIjL#9PC_XYxG3R6dmBboiw6nfySE2ukY-YpGs_AdcqEkmhI
.tar.gz:
https://mega.nz/file/uCwxDYbb#qKlys6xaAsQxMQaOhl5JJw9ql5RQxz2WjGYKNV6Uieo

I did a more correct fix to the problem.  Please us these instead of my previous (now deleted) email:
.deb:
https://mega.nz/file/qXBTlKBA#bJcQuVFSPPVrbb_sLuIs_i-7jpoRJydGlIn0nCcA2Io
.rpm:
https://mega.nz/file/iDQxXTRA#1TblabGz7r5ERy3JCGmgA2jZPVqC6elFo1VyenzUQeo
.tar.gz:
https://mega.nz/file/6KAjgJZR#ZOrgzDPtIP1LhSRC8Drk2zd4Z7a7FylexCbpPyrq2TY

It is working fine now when I double click the jpg files to load now.
Thanks!

Fixed in v1.1.2.


Rename fileService to fileStorage, queueService to queueStorage.

FileService->FileStorage
QueueService -> QueueStorage
Replace file playback json files

Split the switch account stuff to another PR https://github.com/Azure/azure-sdk-for-java/pull/4710.
This is blocked by the board decision about naming.

Closed as it is not top priority for preview 3


tutorials may become broken after next scikit-learn release

In the tutorials we rely on the fact classifier and vectorizer are fit after a pipeline is fit - see e.g. http://eli5.readthedocs.io/en/latest/tutorials/sklearn-text.html. But this is changed in scikit-learn development version, so the tutorials should be updated to make no use of this pattern.
Maybe it already became easier after https://github.com/TeamHG-Memex/eli5/pull/177, but we need to check that, and it also needs to be fixed for explain_prediction. Alternative is to avoid using Pipeline in these tutorials, or use pipe.named_steps instead of original clf/vec.

I don't think this is changed in current scikit-learn master, is it? That's
a backward incompatibility we can't afford.
On 2 May 2017 at 23:32, Mikhail Korobov<EMAIL_ADDRESS>wrote:

In the tutorials we rely on the fact classifier and vectorizer are fit
after a pipeline is fit - see e.g. http://eli5.readthedocs.io/en/
latest/tutorials/sklearn-text.html. But this is changed in scikit-learn
development version, so the tutorials should be updated to make no use of
this pattern.
Maybe it already became easier after #177
https://github.com/TeamHG-Memex/eli5/pull/177, but we need to check
that, and it also needs to be fixed for explain_prediction. Alternative is
to avoid using Pipeline in these tutorials, or use pipe.named_steps instead
of original clf/vec.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/TeamHG-Memex/eli5/issues/179, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEz67eD_mN0J8F5Gdp351eopsc-5F8Rks5r1zCCgaJpZM4NOJTf
.

Oh, right, for some reason I thought https://github.com/scikit-learn/scikit-learn/pull/8350 is already merged and that it is not backwards compatible, while both of it is false. Thanks for the catch!


Genome scans

Run some simple genome scans using minimap2 and then compare bam files to find reads that match to one bam file but are not in another

Hey, you should write a README.md


Add webp versions of hero images

This makes the hero images 2x to 3x smaller for browsers that support WebP (all except Safari).

https://reactspectrum.blob.core.windows.net/reactspectrum/5f2afb68b55ed8c994204f0f3c047e9af3acd34f/docs/react-spectrum/index.html doesn't seem to be working for me?

Oops, good catch!


issue with the Numpy() method for TH2

I have seen that someone commented that in TH1  values and allvalues methods were mixed somehow and this is fixed.. However, there is still an issue with TH2 when calling the Numpy() method on them.. I get :
File "data_MC_plots.py", line 18, in
arr, bins= histo_pythia.numpy()
ValueError: too many values to unpack (expected 2)
I have an older uproot version for my python 2 installation and this implementation  used to work.. now for some reason it does not !

I can get Numpy()  to work  by defining only one variable but then the histogram entries and bin edges would be in a single array that I need to take slices of, which I find  harder to use ...

Now it's
arr, xbins, ybins = histo_pythia.numpy()

The discussion was about making the output of this method agree with the format of Numpy's numpy.histogram2d. (It used to be different from Numpy, now it's the same.) You can also find this out by printing the output of this method and seeing its structure.
Also, you shouldn't have to use an old uproot for Python 2. uproot still supports Python 2 (at least, Python 2.7— we had to drop Python 2.6 support, but that was before uproot-methods existed, so that's not what you're talking about).

Our comments crossed in the aether. I don't understand what you're saying about the single array: two-dimensional edges are in two arrays, not one (I just checked the code).

Oh,I see what you mean— by "one variable," you mean to take it all as a tuple and decompose it with [0], [1], [2], right?

I see, I do not mind it this way.. In fact , it is better as you said..
I tried it and it worked like a charm ... I just did not know about the change
Regarding my python 2.7 , I just did not upgrade the uproot installation..
Thanks for the quick reply

Oh,I see what you mean— by "one variable," you mean to take it all as a tuple and decompose it with [0], [1], [2], right?

Yes, exactly

Okay, glad it's working for you now!

But after you're clarification.. I do not an issue here, at all


Re-Implementation of GreedySearch

I suggest we try to re-implement the GreedySearch algorithm in order to be compatible with the Graph data structure.

[x] Re-implement Greedy Search
[ ] Test the algorithm

I made GreedySearch more compatible and write some comments.

I made a version of Greedy Search too


The test_data is not used on speaker_verification_plda.py

It's in the line 178. Why does it do like this?

Now, I find it is right.


JSON Support?

Hi,
The tool is very useful for merging YML structures (even as library/util to be used by programs).
Is there any chance to make work with JSON too?
Thank you.

Hi!
Definitely no.
The main intention was to preserve comments while updating YAML configuration files.
JSON does not support comments so there is no such problem.
You can use jackson to merge json files (and then (pretty) write it back, if required).

JSON does not support comments so there is no such problem.

Some configurations are in JSON, (as not everything is YAML, especially Node and JS parts), so even without comments merging would still be helpful.

You can use jackson to merge json files (and then (pretty) write it back, if required).

Sure, but the "merge" behaviour would not be 100% compatible with what yaml-updater provides.
Since JSON is a subset of YAML, https://yaml.org/spec/1.2.2/ , I was thinking more of a on the fly conversion to YAML, merging as it works now, and depending on the option (e.g. --output=json) converting back, or letting it as YAML output (this would also allow to merge e.g. a JSON with a YAML).

Sure, but the "merge" behaviour would not be 100% compatible with what yaml-updater provides.

Current behaviour is "don't override existing values". In jackson the same behaviour would be if you load sequentially two json files: first file with modifications and then the original file (jackson would preserve all values from the first file that not exists in second one and override existing values).

Some configurations are in JSON, (as not everything is YAML, especially Node and JS parts, or some REST response), so even without comments merging would still be helpful.

I really doubt anyone have problems with that. JSON is a very simple format and it is well supported (even merge cases).
For YAML, the problem is the absence of no parser correctly preserving comments and formatting, so it is impossible
to read and write yaml file without chages (becuase it is almost impossible to correctly associate comments with nodes in yaml tree).
Only becuase of this I have to write custom YAML parser which would be able to preserve file structure (and comments). This is not a strict parser with many assumptions, supposed to solve only one task - preserve original file format and be able to modify file (know its structure). The parser didn't even have errors handling - it use snakeyaml parser for validations.
For JSON you don't need such complications: any merge scenario could be written in a few lines of code (literally).
Technically it's possible to adopt project to work with JSON, but I don't see any reason for this: jackson could aready do it, why using somehting else (possibly buggy).


Every command that has to do with currency doesn't work

I tried using .gc and .timelyset 100 1. For .timelyset, it didn't set and for .gc nothing happened. Can't do any other command since I don't have currency.

Are you self-hosting?

SERVER OWNERS ARE NOT BOT OWNERS. All owner-only commands require you host the bot. Hosting means you are running a nadekobot on your own computer. If you want to learn how to host the bot yourself, type .guide


Is wttr.in down

Hopefully its not something I'm doing on my side.  Accessing the wttr.in site gives a blank page except for the social links.  Trying to specify weather for a city lands me with Unknown location; please try ~42.926326,10.761625 no matter what city I specify using curl.

It's not just you - I'm getting similar issues.

It was fixed in a short time after you reported the outage. No outages/downtimes since that time. I close the issue and thank you for reporting!


[Preview news page] Incorrect displaying of long title and content

Environment:
Google Chrome Version 80.0.3987.163 (32-bit)
Windows 10 Pro (64 bit)
Steps to reproduce:

Sign in to the 'Green City' application as a registered user
Go to 'Create news' page
Enter more than 50 characters in a 'Title' field (e. g.: "qwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiop")
Enter more than 150 characters in a 'Content' field (e. g.: "qwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiopqwertyuiop")
Click on 'Preview' button

Actual result:
Title and content are displayed each as a single-line text that extend beyond the viewing area

Expected result:
Title and content should be displayed each as a multi-line text within the viewing area according to mock-up

Reproducible:
Always

Verified.


Fix: Small fix for location hash/fragment

The current location hash implementation will replace all # chars in the fragment, which shouldn't be the case.
While, according to spec, # is not allowed in the fragment portion of the URL, browsers tend to be lenient towards enforcing this. Opening a URL in your browser (be it Chrome or Firefox) with # in the fragment will get accepted and returned in your location hash.
Consider https://github.com/leptos-rs/leptos/#test#bar. Your location hash will return (in Javascript) #test#bar.
The current implementation in Leptos will return testbar while the new implementation will return test#bar, which fits the docs better of the function:
/// Current [`window.location.hash`](https://developer.mozilla.org/en-US/docs/Web/API/Window/location)
/// without the beginning #.

A small side-note: I've contemplated simply always removing the first char of the fragment (since this should always be # anyways). Like so: if s.is_empty() { s } else { s[1..].to_string() }. I've decided against it, but if you (the reviewer) prefers this implementation, please let me know and I'll update it.

Hi! Your solution seems fine, if a bit verbose.

Thanks for fixing this!


25% faster decompression with cloudflare's zlib fork

I came across an AWS recommendation to use a fork of zlib that cloudfare maintains because it's faster.
Basically,
git clone https://github.com/cloudflare/zlib.git
cd zlib
./configure --prefix=`pwd`
make
make install

and then to hackily swap between the two
# copy original zlib back
cp ~/julia/julia-1.7.0-beta4/lib/julia/bck_libz.so.1.2.11 ~/julia/julia-1.7.0-beta4/lib/julia/libz.so.1.2.11

# copy cloudflare over original
cp ~/julia/cloudflare/zlib/libz.so.1.2.8 ~/julia/julia-1.7.0-beta4/lib/julia/libz.so.1.2.11

Trying out @btime transcode(ZlibDecompressor, a) between the two on raw compressed basket bytes, I saw between a 25 and 40% speedup. We can still see a similar speedup with
julia> using UnROOT; const t = LazyTree(ROOTFile("Run2012BC_DoubleMuParked_Muons.root"), "Events");

# regular zlib
julia> @btime sum(t.nMuon) # 770.774 ms (11366 allocations: 544.33 MiB)
julia> @btime sum(length,t.Muon_pt) # 5.322 s (18391 allocations: 2.93 GiB)

# cloudflare fork
julia> @btime sum(t.nMuon) # 542.433 ms (11366 allocations: 544.33 MiB)
julia> @btime sum(length,t.Muon_pt) # 3.975 s (18391 allocations: 2.93 GiB)

This was on a x86_64 AMD EPYC 7702P, but hopefully one can reproduce it on other systems. From another thread/PR, I think we wrote down that zlib-ng was giving a 10-15% improvement over the usual zlib library, but this cloudflare fork is even faster!
I bet since it's a fork, not some next-gen rewrite, that it'll be easier to make into binaries (@Moelf ? ;) ). And I think this could benefit many workflows due to ROOT using zlib by default.

Based on http://www.htslib.org/benchmarks/zlib.html, I tried the Intel fork but got the same speed as the default zlib. I guess it makes sense that cloudflare would want a super optimized zlib...

As a further note, it could be worth it to look into
https://github.com/jakobnissen/LibDeflate.jl
Technically we allow a streaming-API for decompressing each basket, but from what I saw, we only enter the while decompressedbytes < totalexpectedbytes loop once.

we only enter the while decompressedbytes < totalexpectedbytes loop once.

at some point I hit this bug so I'm pretty sure it "happens"

we only enter the while decompressedbytes < totalexpectedbytes loop once.

at some point I hit this bug so I'm pretty sure it "happens"

True. If LibDeflate.jl does bring a drastic improvement, then we could just check if uncompbytes, tkey.fObjlen match when we first enter the loop. If so, we know it's a single decompression and we break out after using LibDeflate.

Oh that's interesting, thanks @aminnj for the information. To me it sounds like it would make sense to provide a jll for the cloudfare zlib, but this is of course much more work and I am sure there are more people (outside of the ROOT world) who could potentially help out. Currently I even struggle to get the hdf5lib compiled on an M1, let alone to automate it: https://github.com/JuliaIO/HDF5.jl/issues/870
LibDefalte.jll is also quite interesting and I think it totally makes sense since as you said, we are not streaming.

https://github.com/jakobnissen/LibDeflate.jl/issues/1#issuecomment-979050361
sorry to break the news for you but LibDeflate is not our out ;)

https://github.com/JuliaPackaging/Yggdrasil/pull/3946
I can't get the same branch to build cross-platform unfortunately.

Looks like LibDeflate.jl is giving the same performance gain as cloudflare. Just need to drop the first two header bytes which are associated with zlib (rest is DEFLATE data):
nbytes = decompress!(Decompressor(), uncomp_data, rawbytes[3:end])

PR incoming


Add small canvas sample

Trying out the sample from https://www.youtube.com/watch?v=2osWIorQDMU
It was immediately apparent that CanvasAPI is currently incomplete.
This will unfortunately be the case for a lot of cases.

@zth would you mind reviewing this?
Please focus on the new test, approach to fillStyle and getContext modelling.
Does this seems reasonable?

@zth I'm gonna merge this, sorry for my lack of patience.
I've add a great deal of lore in the contributing section of the docs.
Feel free to point out any remarks you might have, we can always revisit things later.


I can't put background - image

We have written the needed data into your clipboard because it was too large to send. Please paste.

no puedo visualizar el background de una imagen


Streaming messages not compliant with openAI spec

LocalAI version:
274ace2
Describe the bug
When enabling the stream API each choice completion object is missing the finish_reason property except for the last message
To Reproduce:
Example response from LocalAI:
data: {"object":"chat.completion.chunk","model":"ggml-gpt4all-j","choices":[{"delta":{"role":"assistant"}}],"usage":{"prompt_tokens":0,"completion_tokens":0,"total_tokens":0}}

data: {"object":"chat.completion.chunk","model":"ggml-gpt4all-j","choices":[{"delta":{"content":"1"}}],"usage":{"prompt_tokens":0,"completion_tokens":0,"total_tokens":0}}

data: {"object":"chat.completion.chunk","model":"ggml-gpt4all-j","choices":[{"delta":{"content":" 2"}}],"usage":{"prompt_tokens":0,"completion_tokens":0,"total_tokens":0}}

data: {"object":"chat.completion.chunk","model":"ggml-gpt4all-j","choices":[{"delta":{"content":" 3"}}],"usage":{"prompt_tokens":0,"completion_tokens":0,"total_tokens":0}}

Expected behavior
Each message should contain the finish_reason property set to null instead of omitting it entirely
Example:
data: {"id":"chatcmpl-blablabla","object":"chat.completion.chunk","created":1684665888,"model":"gpt-4-0314","choices":[{"delta":{"role":"assistant"},"index":0,"finish_reason":null}]}

data: {"id":"chatcmpl-blablabla","object":"chat.completion.chunk","created":1684665888,"model":"gpt-4-0314","choices":[{"delta":{"content":"1"},"index":0,"finish_reason":null}]}

data: {"id":"chatcmpl-blablabla","object":"chat.completion.chunk","created":1684665888,"model":"gpt-4-0314","choices":[{"delta":{"content":" "},"index":0,"finish_reason":null}]}

data: {"id":"chatcmpl-blablabla","object":"chat.completion.chunk","created":1684665888,"model":"gpt-4-0314","choices":[{"delta":{"content":"2"},"index":0,"finish_reason":null}]}

Additional context
Previously partially addressed by #341

I think this is a good first issue for somebody, sounds like relatively straight forward to be solved.

Hello, I wasn't able to replicate this issue on my end. I made an http stream call to the v1/chat/completions endpoint, below is the request I sent.
{
     "model": "gpt-4",
     "messages": [{"role": "user", "content": "How are you?"}],
     "temperature": 0.9,
     "stream": true
}

I also attached the responses I received which do contain finish_reason:"" for all,
the last one
{
    "created":<PHONE_NUMBER>,
    "object": "chat.completion.chunk",
    "id": "6b2eb69d-eb26-41a6-a2be-eecaa1f663a2",
    "model": "gpt-4",
    "choices": [
        {
            "index": 0,
            "finish_reason": "stop",
            "delta": {
                "content": ""
            }
        }
    ],
    "usage": {
        "prompt_tokens": 0,
        "completion_tokens": 0,
        "total_tokens": 0
    }
}

the one before the last one
{
    "created":<PHONE_NUMBER>,
    "object": "chat.completion.chunk",
    "id": "6b2eb69d-eb26-41a6-a2be-eecaa1f663a2",
    "model": "gpt-4",
    "choices": [
        {
            "index": 0,
            "finish_reason": "",
            "delta": {
                "content": "?"
            }
        }
    ],
    "usage": {
        "prompt_tokens": 0,
        "completion_tokens": 0,
        "total_tokens": 0
    }
}

Could this be due to variations among different receivers? I'm wondering maybe some terminals remove empty fields unknowingly.


Adjust compiler options used to build Python

In order to improve parity with the upstream Docker Hub Python image builds, the build scripts used for our Python binary builds have been adjusted as follows:

The Ubuntu security hardening compiler/linker flags are now retrieved using dpkg-buildflags and passed to the make invocation. See:

https://wiki.ubuntu.com/ToolChain/CompilerFlags
https://wiki.debian.org/Hardening
https://github.com/docker-library/python/issues/810

Configure is now called with an explicit --build architecture.
The directory into which Python is installed during packaging has been changed to make it clearer that this it is only a temporary packaging path (and so why this path doesn't match that used in the CNB for example), since Python is relocated by both this buildpack and the CNB into different locations.

After these changes, our compiler/linker options are now closer to:
https://github.com/docker-library/python/blob/330331fbe3c8d19befaba10ee329c5bf3a9dc225/3.12/slim-bookworm/Dockerfile#L70-L89
These changes are being made now since we'll soon be generating new Python binaries/archives under a new URL structure, which will provide a safer/more convenient transition point to switching to these new compiler options (vs overwriting the existing archives on S3, or only making this change for new Python releases onwards).
GUS-W-14217295.

An example build using these new options (it's based on a branch for a later PR, but uses these new options):
https://github.com/heroku/heroku-buildpack-python/actions/runs/8703318932


Update scalacheck-1-14 to <IP_ADDRESS>

Updates org.scalatestplus:scalacheck-1-14 from <IP_ADDRESS> to <IP_ADDRESS>.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalatestplus", artifactId = "scalacheck-1-14" } ]

labels: test-library-update

Codecov Report

Merging #1515 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1515      +/-   ##
==========================================
- Coverage   85.84%   85.75%   -0.10%
==========================================
  Files          58       58
  Lines        2197     2197
  Branches      107      107
==========================================
- Hits         1886     1884       -2
- Misses        311      313       +2

Impacted Files
Coverage Δ

...scala/io/circe/shapes/LabelledHListInstances.scala
92.59% <0.00%> (-3.71%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
84.79% <0.00%> (-1.17%)
:arrow_down:

...les/core/shared/src/main/scala/io/circe/Json.scala
79.88% <0.00%> (-0.56%)
:arrow_down:

.../core/shared/src/main/scala/io/circe/Decoder.scala
85.05% <0.00%> (ø)

...re/shared/src/main/scala/io/circe/JsonNumber.scala
93.97% <0.00%> (+2.40%)
:arrow_up:

Continue to review full report at Codecov.

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


Support images in the content property

@thorn0 - sorry about the delay on this one, there was another very large PR open that was just merged. If this PR is still needed after all the changes in the other one, you'll need to pull from master first and resolve conflicts before it can be merged.

Done. Please take a look.

Look good to me, thanks.


Issue: Renaming REST resources

Fix #104

 (0) PR Issue: Renaming REST resources

 (0) PR Issue: Renaming REST resources


Add upgrade test

Toward https://github.com/giantswarm/giantswarm/issues/14622
Checklist

[ ] Update changelog in CHANGELOG.md.

Tested on https://github.com/giantswarm/app-exporter/pull/44 (successful case)

Failure cases detect: https://app.circleci.com/pipelines/github/giantswarm/app-exporter/213/workflows/4b6bd458-47ba-47bc-9758-5e7938379017/jobs/948

Changed all logs and vars. PTAL :)


(SSR)Warning: WithConditionalRedirectWrapper returned an empty object from getInitialProps

warning is displayed when a component that uses the wrapper is mounted. in ssr and ssr-ts
WithConditionalRedirectWrapper returned an empty object from getInitialProps. This de-optimizes and prevents automatic static optimization. https://err.sh/zeit/next.js/empty-object-getInitialProps_

If you meant to do this and understand the consequences you can ignore this message as it is only shown in development.

It's intentional. getInitialProps is used to determine whether the user is authenticated. It's required for looking at the cookies.


Update I-mode Scalings

This reopens https://github.com/cfs-energy/cfspopcon/pull/46#event-12848216546, now merging into main.
The PR contains simple updates to the I-mode LI power threshold and taue scalings from Amanda Hubbard. Addresses standing issue https://github.com/cfs-energy-internal/internal-cfspopcon/issues/149.

@tbody-cfs I saw you closed https://github.com/cfs-energy/cfspopcon/pull/46#event-12848216546. I naively went and opened this, merging into main, but it seems many of the changes for cfs-energy:infra_cleanup_part2 aren't in main yet.
What should we do for the simple I-mode updates? They are few enough that I'd be happy to just rebase them onto main first, but if you are moving things around concurrently and I should wait for a bit that would be good to know.


Update TinyGo to 0.28.1

BTW, looks like there's some flakiness in multiphase test @M4tteoP
https://github.com/corazawaf/coraza-proxy-wasm/actions/runs/5320207330/attempts/1?pr=207


HPO to panel bed

This PR adds:

A script for generating gene coordinates for Ensembl and RefSeq genes (clean_gtf.py, credit to @delvinso who made the original script for STAGER, https://github.com/ccmbioinfo/sampletracker2stager/blob/master/variants/clean_gtf.py)
A script to generating a bed file corresponding to the genes listed in the HPO files we receive from CHEO (from Phenome Central or G4RD), hpo_to_panel.py. This script queries the genes listed in the HPO file against both the Ensembl and RefSeq gene files to extract coordinates (most genes have Ensembl gene IDs, but not all). It also looks for gene aliases in the HGNC file and then re-queries the RefSeq gene file for genes that are not mapped in the initial step.
conditional rules for generating the bed file and panel reports if HPO terms are specified in the config.yaml (credit to @aarthi-mohan who started this in the hpo-to-panel branch)
documentation in the README.md

Looks good! Thanks for adding the genes/missing_genes.txt and readme on generation of necessary flat files.


AsyncSteps: investigate possible "production mode"

Potentially, it is possible to improve performance by removing some internal sanity checks when program is fully tested in development/QA.

Implemented as part of v2.0.0 release


Added support of ellipses

Motivation and context
Resolved #579

TODOs:

[x] Export import to some formats (<ellipse .. /> in CVAT formats, and masks if the format supports them)
[x] Add several API tests to cvat-core
[x] Support new shape type in filters

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[x] I have added a description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[x] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@bsekachev , should we keep the shape size/position unchanged if some part of the shape is outside of a frame? Otherwise it is impossible to annotate some objects which are particularly outside of a frame.

@nmanovic
For now for all shapes where it is possible we cut shape if it is out of image. Should we cut ellipses or not is it a question. Probably we should (to keep consistensy), but we also need to add an ability to configure this behaviour for all the shapes.

@nmanovic
For now for all shapes where it is possible we cut shape if it is out of image. Should we cut ellipses or not is it a question. Probably we should (to keep consistensy), but we also need to add an ability to configure this behavior for all the shapes.

What is the major difference with rotated bounding boxes? From my point of view an ellipse shape and a rotated bounding box  are very similar. I agree that it is better to configure the behavior but for some shapes like a rotated bounding box or an ellipse it is impossible to cut them without conversion to a polygon. Also could you please give me an example when the implemented behavior makes sense?

@ActiveChooN @klakhov @zhiltsov-max
Guys, could you please take a look at this patch?

Hi,
Ellipse is not clipped for the size of the frame as you show in the gif (for all shapes), but in the develop it works fine

Will ellipses support rotation as bounding boxes? Looks like it can be helpful in some cases.

@ActiveChooN

Ellipse is not clipped for the size of the frame as you show in the gif (not only ellipses btw), but in the develop it works fine

It was changed according to @nmanovic ideas. GIF just is not updated.

@bsekachev , another idea which is similar for rotated bounding boxes. Rotated shapes better to annotate using 4 points.Thus you can draw a rotated bounding box or fit an ellipse into it.
BTW, how to rotate ellipse?

@bsekachev great job! What do you think about quick way to draw a circle? Usually we can hold shift using ellipse tool to draw circles

@nmanovic

I will recommend to draw an ellipse as it is done by Photoshop

There was default way to draw ellipses provided by the svg library. Now I redesigned it according to photoshop default way of drawing.

BTW, how to rotate an ellipse?

Actually it was disabled because nobody asked us about rotation of ellipses. Now, If you wish, I enabled it.

another idea which is similar for rotated bounding boxes.

Agree, but it is a different feature where additional design stage is necessary for both bounding boxes and ellipses.

@klakhov
That's interesting, but as far as I remember there were not such requests for bounding boxes for years. Maybe nobody needs it? And if we decide to implement the feature, it is probably better to do it for both bounding boxes and ellipses.

@klakhov
That's interesting, but as far as I remember there were not such requests for bounding boxes for years. Maybe nobody needs it? And if we decide to implement the feature, it is probably better to do it for both bounding boxes and ellipses.

@bsekachev , @klakhov ,
I agree that it is extremely rare case when you need circle or square. We are annotating real objects, not draw something on the canvas.
P.S. 'bounding box', 'ellipse' should have mostly the same implementation. The only difference is representation (how we are drawing it and export to a file).

Hi, @dvkruchinin
Could you please add some tests to ellipses?
Drawing shapes, tracks / rotation / interpolation

Hi, @dvkruchinin
Could you please add some tests to ellipses? Drawing shapes, tracks / rotation / interpolation (I have already added a command to draw ellipses actually and updated test with statistics).

Hi @bsekachev,
Sure. A test will be prepared.

@TOsmanov @aschernov
Guys, could you add a piece of documentation about the new supported type of shape?

Hi @bsekachev,
Yes, the documentation will be updated

please help me, why I can't find the ellipse shape in the web version? how to add the ellipse shape?

@mshz88
If you speak about cvat.org, the server was not updated yet. We are working on this, but it takes long time because of significant changes in CVAT. I hope the process will be finished the next week.

Thank you, @bsekachev CVAT is the best tool among others with many features. We are looking forward to seeing the ellipse shape in the updated version.


Add a note that you cannot use specific fields when you delete with predicate

https://docs.influxdata.com/influxdb/cloud/reference/syntax/delete-predicate/#limitations
As reported in this issue https://github.com/influxdata/influxdb/issues/20704 and the related issues, it is not possible to delete a specific _field value in InfluxDB. We should update the docs to reflect this.

@russorat will deleting/dropping field will be implemented in InfluxDB ever?
https://github.com/influxdata/influxdb/issues/6150 is from 2016
https://github.com/influxdata/influxdb/issues/20399 is 6 months old


split payment_success and order_success views

I think just an ordinary redirect will suffice. Post and get data is ignored. Let me know what you think.
I want to merge this change because...
#1987
(Please mention all relevant issue numbers.)
(If your changes affect the UI, providing "before" and "after" screenshots will
greatly reduce the amount of work needed to review your work.)
Pull Request Checklist
(Please keep this section. It will make maintainer's life easier.)

[x] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Database queries are optimized and the number of queries is constant.
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[x] JavaScript code quality checks pass: eslint.

Well, I think this should indeed be enough.

Simple test for the new handler would be nice

@Pacu2 Both views seem to be covered by checkout integration tests. Do you mean a specific test only to check the payment-success view?


Hook calls from jinja should have access to context

When calling hooks as a jinja extension, the hooks don't have access to context items such as output_dict. For instance when calling
stuff:
  and: things
get->: "{{get('stuff/and')}}"

The get call doesn't have a dict to look up to.  The issue is when the hook is called, it was instantiated without the context and thus the context needs to be reinjected when it is called.
Naively trying to inject the context doesn't work.
        rendered_template = template.render(render_context, context.dict())

Going down this path leads to a whole bunch of jinja craziness but things get dropped around line 1286 in environment, Template.render where the kwargs are dropped. Could patch this or other places where the extension is called by passing down the kwargs. This is hairy though.
        ctx = self.new_context(dict(*args, **kwargs))

What could be better is instead of instantiating all the hooks when the environment is loaded but instead using this:
    variables = meta.find_undeclared_variables(context.env.parse(raw))

To find all the hooks if they are not in undeclared_variables and then injecting them into the environment at render time. Could then remove them and would never compete with ambiguous calls.

This is fixed now.  Jinja extensions have access to all contexts which are built lazily on-render. Jinja filters on the otherhand don't.  They might be dropped altogether or perhaps only brought into scope with a flag.


Dbeaver generated sql reports errors when trying to execute it .

I have one table in one database which i want to copy paste in another database ( not the context .... just the table and it's schema , empty table ) so i am trying ... CTRL+C  and then trying to save it .... error .... trying with Dbeaver generated sql .... error ... what kind of black magic bug is this ...
The tool that generates it is confessing that it generates it wrongly ?  My emotions now are To DBeaver or not to DBeaver anymore.

CREATE TABLE ats_local.lottery_wagers (
	wager_id bigint(20) NOT NULL,
	`type` tinyint(4) NOT NULL,
	game int(11) NOT NULL,
	version int(11) NOT NULL,
	wager_prtn tinyint(4) NULL,
	fdraw bigint(20) NULL,
	wager_id_parent bigint(20) NULL,
	wager_type tinyint(4) NULL,
	operator int(11) NULL,
	channel int(11) NULL,
	`columns` int(11) NULL,
	cost decimal(20,8) NULL,
	discount decimal(20,8) NULL,
	edraw bigint(20) NULL,
	advance_draws int(11) NULL,
	multiple_draws int(11) NULL,
	employee varchar(32) CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL,
	player int(11) NULL,
	retailer int(11) NULL,
	terminal int(11) NULL,
	status smallint(6) NULL,
	blocking_status smallint(6) NULL,
	sys_tm timestamp(3) DEFAULT CURRENT_TIMESTAMP(3) NOT NULL DEFAULT_GENERATED on update CURRENT_TIMESTAMP(3),
	client_tm text CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL,
	gm_client_ver int(11) NULL,
	boards text CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL,
	multiplier text CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL,
	part_draws json NULL,
	special text CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL,
	input_method int(11) NULL,
	playslip_id bigint(20) NULL,
	productId int(11) NULL,
	team_shares int(11) NULL,
	play_draw bigint(20) NULL,
	addon_md text CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL,
	subscription_id text CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL,
	selections_type int(11) DEFAULT 0 NULL,
	syndicate_id text CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL,
	CONSTRAINT `PRIMARY` PRIMARY KEY (wager_id,game,`type`,version)
)
ENGINE=InnoDB
DEFAULT CHARSET=latin1
COLLATE=latin1_swedish_ci
COMMENT='';
CREATE INDEX full_index USING BTREE ON ats_local.lottery_wagers (game,wager_prtn,fdraw,edraw,wager_id,`type`,version);
CREATE INDEX game_draw_excl_channel_idx USING BTREE ON ats_local.lottery_wagers (game,wager_id,fdraw,channel,edraw);
CREATE INDEX sys_tm USING BTREE ON ats_local.lottery_wagers (sys_tm);
CREATE INDEX wager_id_parent_idx USING BTREE ON ats_local.lottery_wagers (wager_id_parent);

Finally found what was the problem ........ needed to install MYSQL8 because json type is not defines in MYSQL5 .... if i am not wrong ....


topics: anchor outputs fixup

Noticed this after tweeting to consult this topic!
https://twitter.com/jonatack/status/1379839115676561408

This is a nexcellent suggestion


1.22.2 - Flutter plugin not installed; this adds Flutter specific functionality.

How do I fix this android plugin error? This error is being come after upgrade to 1.22.2
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 1.22.2, on Mac OS X 10.15.7 19H2, locale en-LK)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
[!] Android Studio (version 4.1)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] VS Code (version 1.50.1)

[✓] Connected device (1 available)

! Doctor found issues in 1 category.

Android Studio

@bhanuka96
The plugin warnings do not matter. See this comment for reference. Also this is similar to 67986.
Originally posted by @darshankawar in https://github.com/flutter/flutter/issues/68274#issuecomment-709924282

Closing this as duplicate of 67986
Even if you are getting those errors, you should be able to build without any issues. See the above referred link for more details.

Hey @bhanuka96  try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

@kyughost now working

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Thanks this worked for me just now.

@kyughost thank you from France ;) it works.

@kyughost thank you from France ;) it works.

My pleasure, from Congo

@kyughost Thanks it worked :)

@kyughost Thank you from Bangladesh :) it works

@kyughost does this also work on windows?

@kyughost does this also work on windows?

No sir. Sorry

please people how can I fix this problem on windows?

Thanks so much! just got a new Macbook Air 2020, and was giving me the same problem.

please people how can I fix this problem on windows?

I'm sorry. Mac user here

Thanks @kyughost  It worked for me too too

嘿@ bhanuka96 试试这一行代码，对我有用
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1
nice！

Funcionó :) perfecto, muchas gracias

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Thanks, this worked for me

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Thanks, this worked for me

how to do for win10

how to do for win10

how to do for win10?

how to do for win10?

How to fix it? On win 10 i tried all solutions, buyt nothing work (
Only work on dev brunch, but i need stable

How to fix it? On win 10 i tried all solutions, buyt nothing work (
Only work on dev brunch, but i need stable

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Thanx bro this worked for me :)

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Thanx bro this worked for me :)

Nice !! Thanx !!!

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Nice !! Thanx !!!

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

worked for me thx!

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

worked for me thx!

Can I use this

Can I use this

I've android studio 4.1.2 so Can I used this instead of 4.1 ?
Like as?
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1.2/plugins ~/Library/Application\ Support/AndroidStudio4.1.2

I've android studio 4.1.2 so Can I used this instead of 4.1 ?
Like as?
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1.2/plugins ~/Library/Application\ Support/AndroidStudio4.1.2

Can someone please tell how to resolve this issue in win10 ?

Can someone please tell how to resolve this issue in win10 ?

Can someone please tell how to resolve this issue in win10 ?

for win10 you can consult with this for windows

Can someone please tell how to resolve this issue in win10 ?

for win10 you can consult with this for windows

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Thanks. This is working

ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Thank you. Worked for me!

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

Thanks. It work!

please, how can solve this problem in win10 ?

ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1
this worked for me as well. thanks

I have just upgraded android studio to the latest version 4.1.2, and the problem were fixed

嘿@ bhanuka96 试试这一行代码，对我有用
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1
nice！

I can't thank you enough!! It was perplexing me!!

Hey @bhanuka96 try this line of code, worked for me
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.1/plugins ~/Library/Application\ Support/AndroidStudio4.1

(Y) Worked

For Android Studio 4.2.1
Flutter 1.22.5 • channel unknown • unknown source
Framework • revision<PHONE_NUMBER> (6 个月前) • 2020-12-10 11:54:40 -0800
Engine • revision ae90085a84
Tools • Dart 2.10.4

Dart SDK version: 2.10.4 (stable) (Wed Nov 11 13:35:58 2020 +0100) on "macos_x64"

just change version:
ln -s ~/Library/Application\ Support/Google/AndroidStudio4.2/plugins ~/Library/Application\ Support/AndroidStudio4.2

Worked for me.

3 Working Solution Added Flutter plugin not installed error. When running flutter doctor


Better way to key on renderers

Solve the problem that keys are ultimately strings, so we need some way to differentiate between renderers of the same type (e.g. when using multi).
Can't use an index as we would end up reusing containers, which won't work (a component needs to be called on an empty container).
Using identity doesn't work if multiple renderers of the same type are used, because they're the same type.
Look at add an id property to renderers and key on this.

Resolved in #441


remove src from template in favor of <proj_name>/<proj_name>

Description
As mentioned in #208 it is desired to move from a src directory to a <proj_name>/<proj_name> directory.  I did a quick test to see if I could get it to work with just a few changes and it appears that most of the project is working.  I would definitely take in any further changes/suggestions/tests.  Note I did not test all cli commands, and some such as kedro build-docs and kedro docker may need some updates.
Development notes

I moved the contents of the src directory to the {{ cookiecutter.repo_name }} directory.
I added includes to the setup.py.

    - packages=find_packages(exclude=["tests"]),
    + packages=find_packages(include=["{{ cookiecutter.python_package }}", "{{ cookiecutter.python_package }}.*"], exclude=["tests"]),

tested my changes manually
cookiecutter /path/to/template

I called the project default_kedro, then ran some various commands to ensure the project was working as expected.
default_kedro # runs the pipeline
kedro run # runs the pipeline
kedro ipython # loads in catalog and context
%run_viz # starts kedro viz

TODO
[ ] kedro build-docs
[ ] kedro docker
Checklist

[x] Read the contributing guidelines
[x] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added a description of this change and added my name to the list of supporting contributions in the RELEASE.md file
[ ] Added tests to cover my changes

Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

I am submitting this as myself Waylon Walker in my own free time.

Hey @WaylonWalker! Thanks so much for spending time on this PR, we really appreciate it.
I think if we circle back to this one then we'd like to give users the flexibility to choose if they want src or proj_name. We have discovered that this is quite a well-discussed issue in the Python world. So for now I'll park this PR but thank you so much for raising this.

That sounds like a fantastic idea, thanks for investigating.


Observe data in InstrumentTrace

InstrumentTrace just adds a new label to metric ignoring values that should be Observed

seems go 1.9 is no longer supported
https://github.com/cespare/xxhash/issues/34#issuecomment-561905133


[PrematureStopping ] Add signal handler to init methods.

Needs rebase once #3848 gets merged. Only bf2160c is important.

@vigsterkr


Back button Icon with RTL support

💬 Question
I have an issue when trying to implement RTL where the back icon doesn't flip with the UI.
In react native blog, they mentioned flipping the Image using transform property but that didn't work.
This is how it looks right now. The back arrow should be facing the other direction

The code for the Top Navigation
<TopNavigation
        alignment="center"
        title="الحدث"
        accessoryLeft={
          <TopNavigationAction
            icon={<Icon {...props} name="arrow-back" />}
            onPress={e => navigation.goBack()}
          />
        }
        accessoryRight={
          <TopNavigationAction
            icon={<Icon {...props} name="edit" />}
            onPress={e => setEditMode(!isEditMode)}
          />
        }
      />

UI Kitten and Eva version

Package
Version

@eva-design/eva
5.1.1

@ui-kitten/components
5.1.1

@ajakka it seems you are not correctly applying the approach described in the blog.
Please try something like this:
  const rtlStyle = {transform: [{scaleX: -1}]};
  return (
      <TopNavigation
        alignment="center"
        title="الحدث"
        accessoryLeft={() =>
          <TopNavigationAction
            icon={({style, ...props}) => <Icon {...props} style={[style, rtlStyle]} name="arrow-back"/>}
          />
        }
      />

  );


Testing automation with testplan generation.

It'd be nice to generate test plans to run them using lomax. We can write these test cases in a standard markdown format (json?) and then let lomax parse them and run the tests accordingly.

Done! https://github.com/opendns/lemming/commit/5132c0f13c9073f9155f435ef2765cbfaccba498


upgrade for specific file extentions

For inspiration... I added preview of

HTML
PDF
Office documents
Images
archives

I like it.  But did you have to delete the original bash file?


Use the latest nvd3 source.

The npm module was published 6 months ago. It makes sense to use the latest api in order to utilize new chart components parameters. So I think it makes sense to update 'nvd3' deps to take source from github master branch rather than npm.

I've updated the nvd3 version to the last commit. It's the only way I've found to prevent a broken version after an nvd3 update.
Thank you for your help.

It works, thank you.


Nil error in signups#create

undefined method name' for nil:NilClass   app/views/meals/_signup_form.html.erb:8:in _app_views_meals__signup_form_html_erb___219508808312718898_70118591529580'
Add a feature spec for this case (validation fail) since this always seems to get broken
Originally created by Tom Smyth at 2017-06-27 21:08, migrated from redmine-#6732

Version set to 4.3

By Tom Smyth at 2017-07-06 16:30

Status changed from New to Closed

By Tom Smyth at 2017-06-29 19:23


Inline terminal chat is too small when response comes in

On a new window, ask a question of terminal chat
See the result:

all sorts of weird things happens if it's too small really

The issue is still relevant on stable 1.96.0 (Copilot Chat 0.23.0) on Mac OSx with zsh shell.


Support for conditionally enabling of alpha-level CRDs

If CRD doesn't contain any alpha version of resources, produced CRD templates are almost identical to CRD manifests in crds/ directory.
A given alpha-version CRD can be enabled by adding its GVK to experimental.api helm chart variable.
If there's a single alpha version resource in a CRD, the whole thing is rendered conditionally:
{{- if has "waypointlifecyclemanagers.admin.gloo.solo.io/v2alpha1" $.Values.enabledExperimentalApi }}
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
  annotations:
    crd.solo.io/version: 2.4.0
  labels:
    app: gloo-mesh-apis
    app.kubernetes.io/name: gloo-mesh-apis
  name: waypointlifecyclemanagers.admin.gloo.solo.io
spec:
  group: admin.gloo.solo.io
  names:
    categories:
    - solo-io
    kind: WaypointLifecycleManager
    listKind: WaypointLifecycleManagerList
    plural: waypointlifecyclemanagers
    singular: waypointlifecyclemanager
  scope: Namespaced
  versions:
  - name: v2alpha1
    schema:
      openAPIV3Schema:
        properties:
          spec:
...
    served: true
    storage: true
    subresources:
      status: {}
{{- end  }}

If a CRD contains multiple versions and one of them is alpha, then only that version can be rendered conditionally:
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
  annotations:
    crd.solo.io/version: 2.4.0
  labels:
    app: gloo-mesh-apis
    app.kubernetes.io/name: gloo-mesh-apis
  name: workspaces.admin.gloo.solo.io
spec:
  group: admin.gloo.solo.io
  names:
    categories:
    - solo-io
    kind: Workspace
    listKind: WorkspaceList
    plural: workspaces
    shortNames:
    - ws
    singular: workspace
  scope: Namespaced
  versions:
  - name: v2
    schema:
      openAPIV3Schema:
...
    served: true
    storage: true
    subresources:
      status: {}
  {{- if has "workspaces.admin.gloo.solo.io/v2alpha1" $.Values.enabledExperimentalApi }}
  - name: v2alpha1
    schema:
      openAPIV3Schema:
        properties:
          spec:
...
    served: true
    storage: false
    subresources:
      status: {}
  {{- end }}
---

This partially resolves https://github.com/solo-io/gloo-mesh-enterprise/issues/9019
BOT NOTES:
resolves https://github.com/solo-io/gloo-mesh-enterprise/issues/9019

Most changes here to support rendering of codegen/templates/manifests/crd.yamltmpl.
A couple of small questions (maybe) to resolve: https://github.com/solo-io/skv2/pull/462/files#diff-0edd4bd3f13b4db965a1337273d05e142d5c93e63559722098c88c9873458997R148


SendMessage

Hi,
Iam trying to use SendMessage function to send message to unsaved contact ,but it  give me an error "Chat not found "
and i have tried to call getuser before sendmessage but not help
how can i fix theis issue?

Show us some of your code


fix incorrect version number on diff link text

This is a typo fix on text of diff links.

@rkuchan done.

Awesome, @saboorian. Merging :)


fix(nfts): clippy warnings

Resolve all clippy warnings in the forked pallet-nfts.

Codecov Report
Attention: Patch coverage is 77.61194% with 30 lines in your changes missing coverage. Please review.

Project coverage is 68.37%. Comparing base (aff6408) to head (4a5f17c).

Files with missing lines
Patch %
Lines

pallets/nfts/src/features/attributes.rs
46.15%
0 Missing and 7 partials :warning:

pallets/nfts/src/features/create_delete_item.rs
88.09%
0 Missing and 5 partials :warning:

pallets/nfts/src/features/metadata.rs
66.66%
0 Missing and 4 partials :warning:

pallets/nfts/src/features/settings.rs
62.50%
0 Missing and 3 partials :warning:

pallets/nfts/src/features/approvals.rs
66.66%
0 Missing and 2 partials :warning:

pallets/nfts/src/features/buy_sell.rs
60.00%
0 Missing and 2 partials :warning:

pallets/nfts/src/features/transfer.rs
77.77%
0 Missing and 2 partials :warning:

pallets/nfts/src/impl_nonfungibles.rs
0.00%
2 Missing :warning:

...lets/nfts/src/features/create_delete_collection.rs
85.71%
0 Missing and 1 partial :warning:

pallets/nfts/src/lib.rs
66.66%
0 Missing and 1 partial :warning:

... and 1 more

@@            Coverage Diff             @@
##             main     #391      +/-   ##
==========================================
- Coverage   68.38%   68.37%   -0.02%
==========================================
  Files          70       70
  Lines       11846    11841       -5
  Branches    11846    11841       -5
==========================================
- Hits         8101     8096       -5
  Misses       3486     3486
  Partials      259      259

Files with missing lines
Coverage Δ

pallets/nfts/src/benchmarking.rs
85.79% <100.00%> (ø)

pallets/nfts/src/common_functions.rs
80.00% <100.00%> (ø)

pallets/nfts/src/features/atomic_swap.rs
90.51% <100.00%> (ø)

pallets/nfts/src/features/lock.rs
90.12% <100.00%> (ø)

pallets/nfts/src/features/roles.rs
96.25% <100.00%> (+0.09%)
:arrow_up:

...lets/nfts/src/features/create_delete_collection.rs
84.33% <85.71%> (ø)

pallets/nfts/src/lib.rs
71.82% <66.66%> (ø)

pallets/nfts/src/migration.rs
0.00% <0.00%> (ø)

pallets/nfts/src/features/approvals.rs
93.10% <66.66%> (-0.16%)
:arrow_down:

pallets/nfts/src/features/buy_sell.rs
90.69% <60.00%> (ø)

... and 6 more

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


build: upgrade to new contract

Support the latest contract https://github.com/Rate-Limiting-Nullifier/rln-contract/commit/572bc396b6eef6627ecb2f3ef871370b4c0710f3

CI failed but tracked in https://github.com/Rate-Limiting-Nullifier/rlnjs/issues/96#issue-1875299992


update packages

:x: Build TheIdServer 6.3.1-fix-dependencies1-0088 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/2ad09fee55 by @)

:x: Build TheIdServer 6.3.1-fix-dependencies.1+88.build.2555 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/2ad09fee55 by @)


fullscreen does not seem to work on safari

request full screen doesnt seem to work on safari. image is shown only in small size in top left corner. any fix?

Ok, thanks. I have never tested it on Safari... I will provide to figure out. If you want, any help is welcome. I suppose that it's a CSS error. Can you try to find some hint using the developer console? Thanks

I will try this on a Apple PC asap. ;)

Hey :wave:
If I remember correctly, Safari is not handling percentage width correctly.


Provider refresh causes Infinite redirect at sign-in page

Environment

Operating System: Linux
Node Version:     v22.3.0
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      modules, build, auth, routeRules
Runtime Modules:  ../src/module.ts
Build Modules:    -

Reproduction

Clone the repo
cd nuxt-auth/playground-refresh
Edit pages/index.vue to comment definePageMeta({ auth: false }) out
Run dev server and open local pages

Describe the bug
Error "500 Infinite redirect in navigation guard" is shown.
The same procedure in playground-local has no problem.
The documentation states that the local provider and the refresh provider is the same except refreshing tokens.
The refresh provider should treat the login path as special as the local provider does.
Additional context
No response
Logs
No response

https://stackblitz.com/edit/github-4rxvta?file=pages%2Findex.vue,app.vue,nuxt.config.ts
Key points are:

enable globalAppMiddleware
not disable auth locally (by defining page meta)

The local provider seems to prevent this error.
https://github.com/sidebase/nuxt-auth/blob/8e6519f90f00c7b4478ed34088d69e5bcb0ba5dc/src/runtime/middleware/auth.ts#L70-L75

Hi @hiroki-otaka 👋
This makes sense. The expectation for the local provider was added wayyyy back in 2023 #543, before the refresh provider was added. I assume it slipped through when we created it, however the fix should be very easy. Would you like to push a PR and include the refresh provider in the same check?

Thank you for explaining backgrounds.
And sure, I'll make a PR.


Block names should probably have a prefix

To avoid name clashes in the future with other blocks of similar nature but different platforms/libraries. So accelerometer_get_x should probably be microbit_accelerometer_get_x or ubit_acc_get_x.

Fixed with fcfa9172b322ca56962aaec19515bcc9bd989b3f


Create API Keys for Plugins

If people want to use Screeenly in Plugins, and doesn't want to expose their API key to the public...

Merged into #19.
I think the best solution is to have the opportunity to create multiple API keys per account.
Still have to develop it first.


Getting cross origin issues on IOS

Its not working on IOS as its through cross origin issue

This issue is resolved by v1.2.1
Tested on Safari 11 (iOS)


Re-export proc_macro2::TokenStream

This type is an important part of quotes public API - for convenience it should be re-exported by the crate.

I don't have a strong argument either way, but I am leaning toward not making this change. We already have a proc_macro::TokenStream and a proc_macro2::TokenStream. Adding quote::TokenStream into the mix would make things more confusing.


Add encrypt option to encryption cookie value

for example :
...
this.$cookie.setKey("my hash key")
...

+1

+1


Maybe add an age of active accounts stats

Something like: 20% of accounts that made edit in 2022 signed up in 2022, 10% in 2021, etc
Maybe add also filtering by editor? 20% of accounts that made edit in 2022 using JOSM signed up in 2022, 10% in 2021, etc

I was thinking about plotting usage of (new) users before but I wasn't sure what kind of question would be interesting.
Maybe:
General => "How many people are contributing each month" (here):

add new plot "new contributors per month" (amount of contributors with their first edit monthly)

Editing Software => "How many people are contributing per editing software each month?" (here):

add a new plot "monthly new contributor count per editing software" (amount of contributors per editing software with their first edit in total (or with this software?) monthly)

Do you think a yearly table for editing software (with all top 100) in addition to the plot is also interesting? if so, it should probably be an other question I think.


Add option to favorite tabs directly from the Popular Tabs section

Maybe a heart on the right side of the list_item_browse_tabs.xml.  It would have to be implemented in the favorites section too.

This should be an option in any list of songs


[Question]: Not publishing to RTSP

Ask your question
Earlier I managed to use the RTSP stream of my Eufy 2k wired doorbell directly in Homeassistant. Now the rtsp server says nothing is publishing to the path.
I see no errors in the docker. Should I enable the publishing to this path some how?

Wrong repo


Fine-tune public landing page -- acartio.io/home

[x] Change "Sighting age" to "Location age" or something else more generic so that not only visual, but also acoustic (and other sensors used to detect presence/absence) are included in the term.
[x] Add some legend or other tool to explain the significance of the size (radius) of the markers on the map
[x] Explain somewhere that this map shows occurrence of all marine species
[ ] Should we explain that you can click on a marker to get more info (and then clarify what each field means, and why we are presenting UTC timestamps => e.g. this is a data cooperative from which visualizations or applications like orcamap can be created; it's not meant to be a real-time application or visualization tool itself? Maybe we need a "how to" demo to orient new users?

Created this issue in forked repo to address final checkbox, so closing.