Comments for all callback API methods

Well-commented APIs, at the level of detail (or more) as the sync or async APIs.

Thanks so much for the detailed review @srini100 ! I'll address your comments soon.

Thanks, @srini100 . I addressed all comments from your round of review.

This pr is conflicted because of #18502. I will fix that in a few hours

@karthikravis - this is the PR that I mentioned that adds comments to server_builder . It looks like we're all blocked for merging at the current time but do let me know if this is going to hold up your revival of #18242

A lot of infrastructural failures


[bug] GuardRunnable may not be reporting tokens appropriately if reasks are allowed

Describe the bug
The tokens reported back in an AIMessage when using GuardRunnable appear to only be for one call.  What is the behaviour when reasks are enabled on a GuardRunnable?
To Reproduce
Use a Guard with LCEL with reasks enabled.  Inspect token count reported back in the AIMessage
Expected behavior
Token counts should be accurate.
Library version:
0.5.0
Additional context
AIsre reasking even possible when using GuardRunnable?
If not, we should warn to raise when a user tries to set them.

Reasking is not currently supported for GuardRunnable's i.e. Langchain integration.


Add network interface fields to configs.

The listen address should be string. But current configuration file accepts (only) network ports as integer. And then converts it to listen address. (":%s", port)
In this commit I have added a way to specifically provide network interface to listen.
I wanted to make it full string. Instead of accepting port as integer, it should accept the whole network interface like localhost:8080 or <IP_ADDRESS>:8080.
For backward compatibility, I just added a way to specify network interface addresses.
From now users can specify what interface they want Bramble to listen. But the interface and port have to be in different fields. (will probably get fixed in v2).
Solves #65

Thanks for your PR @hsblhsn, I adapted it slightly in #70
We run our deployment of bramble only in k8s so have never needed to specify anything other than the local port to listen on, this should give a bit more flexibility while providing backwards compatibility.
Does it fit your use case?


Zoom and shading of the background image

Hi Jason, I really like the unblurred artist image as background and think it still looks good. Would it be possible to choose our own zoom percentage and also the shading? I guess you zoom it to more than 200%, and most of time less might be still looking good as well showing more of the original artist image.
I use your tool on a 22" widescreen LCD monitor by Oracle/Sun with screen resolution set to 1680x1050 and 125% scaling which are the recommended Windows 10 settings. I hardly see any ugly pixeling in the background pictures, so making them brighter might work for me, too.

I need to take a look at the code again, but I believe it zooms just enough to fill the screen both vertically and horizontally (whichever is longer). Zooming out would result in letterboxing, which I don't like the idea of. As for the darkness of the background, I don't really want to affect the text readability, especially since the track/artist text colors are carefully calculated/adjusted to always be readable.
I won't shoot down the ideas right away, but I also want to be wary of how many user options I add. At some point it's detrimental to the user experience, so there are trade offs to consider...

Following my thoughts in the comment above, I've decided to keep things the way they are with the background. I don't want to stray too much from the current style. I hope it's still satisfactory enough though!


copyIn / copyOut for Typedefs not exported in *SplDcps.h

Hi,
Is there a reason the copyIn and CopyOut methods are not exported for typedefs in the IDL when the compiler is run over them.
They exist in the body just not exported in the header.
This prevents me from having a base IDL in it's own library and then using the base IDL in another project.
IE:
// Library 'Base' built using base.idl below

module base
{
   typedef sequence <T_Char, 500> T_LongString;
   typedef T_LongString T_DescriptorType;
};

// Second library using the base.idl and linking to the Base library.

#include "base.idl"
module lib2
{
    struct Lib2Type
   {
      T_DescriptorType A_acousticGunshotDetectionSystemTypeName;
   };
};

Hi James,
Good catch, thanks!!. I have reviewed your fix, and it seems OK to me. We will try to process pull request shortly.
Regards,
Erik Hendriks.


add SA1 names and borders

Each SA2 covers a number of SA1s,
we would like to show the following after any SA2 is chosen on the dashboard

names of SA1s covered by that particular SA2 in the sidebar
SA1 borders inside that particular SA2 with a light color

Names won't be too hard, but borders will take some work. There's a way to get halfway there with Mapbox's hosted service, but that costs money. It's possible to do borders entirely on our own server, but it will require a fair bit of preprocessing since raw SA1 borders are too much data to send to browsers. We'd need to simplify them and also split them up by SA2.

Justin, yes, I agree let's leave the borders for later, also there is a text box
"Select a community to learn more about opportunity in that area."
the text box covers the sidebar
While keeping the textbox in the page, we need to change sidebar structure in a way that this problem does not happen

The sidebar shows the SA1 codes not names.
Justin could you please check for that?

That's right. SA1s don't have names, just codes. See<EMAIL_ADDRESS>Subject/12<IP_ADDRESS>~July 2016~Main Features~Statistical Area Level 1 (SA1)~10013.

SA1s are not named. They are identified either by an 11-digit fully hierarchical code, or by a truncated 7-digit code comprising the State and Territory, SA2 and SA1 identifiers.

SA1s don't have names and adding their borders is out of scope. We can open a new issue if that becomes a thing again.


GraphQL, Phoenix and React fullstack app

Could you please assign this example on me?

@EricSimons

hi @sunndeath, I'm also working on a phoenix/graphql version. If you'd like to share/discuss something let me know.

hi @sunndeath, I'm also working on a phoenix/graphql version. If you'd like to share/discuss something let me know.

hi, @barnabasJ,
ok.
I've configured the project for running Phoenix with "Create react app": https://github.com/sunndeath/elixir-phoenix-graphql-react-realworld-example-app. I'm going to continue my version of implementation, because this is very interesting for me and I did not receive answers about the possibility of making an implementation based on another stack.


Language filtering not working

when filtering using the language filter feature no results come back at all. this is in contrast to the extension

In newest release


script doesn't create uploads folders on macOS (apple silicon)

As described in the Readme.
npm i -g zx
npm run init-directories

should create an uploads folder. But when the script was run using zsh on macOS Ventura 13.4.1 (c), nothing happened. Upload folders needed to be created manually.  This should be mentioned in the Readme

I tested.

Or if you want I can fix it. The script has some logic problem. That's all.

thanks @aninda052


chore(main): release 0.2.0

:robot: I have created a release beep boop
0.2.0 (2024-03-16)
Features

add support for mistral instrut (947834d)
infra to support mutiple chat templates (12d972b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/chenhunghan/chat-templates/releases/tag/v0.2.0 :sunflower:


Got panic, when use github.com/google/go-cmp on generated messages

I see, that there is a package google.golang.org/protobuf/testing/protocmp special for using github.com/google/go-cmp with protocolbuffers.
Seems, that generated protobuf messages should be comparable with cmp.Equal and cmp.Diff may be used, but I got panic because of unexported fields in generated messages.
func TestCmp(t *testing.T) {
	t.Log(cmp.Diff(&descriptorpb.DescriptorProto{}, &descriptorpb.DescriptorProto{}))
}

panic: cannot handle unexported field at {*descriptorpb.DescriptorProto}.state:
	"google.golang.org/protobuf/types/descriptorpb".DescriptorProto
consider using a custom Comparer; if you control the implementation of type, you can also consider using an Exporter, AllowUnexported, or cmpopts.IgnoreUnexported [recovered]
	panic: cannot handle unexported field at {*descriptorpb.DescriptorProto}.state:
	"google.golang.org/protobuf/types/descriptorpb".DescriptorProto
consider using a custom Comparer; if you control the implementation of type, you can also consider using an Exporter, AllowUnexported, or cmpopts.IgnoreUnexported

Package versions:
github.com/google/go-cmp v0.4.0
google.golang.org/protobuf v1.21.0

Can you clarify how cmp can and can't be used with generated protobuf messages?

Seems, that generated protobuf messages should be comparable with cmp.Equal and cmp.Diff

It was never intended for cmp to be directly used with generated protobuf messages, so this panic is working as intended. Prior to protocmp, it was expected that usages of cmp be provided with the proper way to compare protobuf messages using a custom comparer:
cmp.Equal(x, y, cmp.Comparer(proto.Equal))

Unfortunately, there was a chance that cmp.Equal (or cmp.Diff) got the comparison "right" even if that option was not specified, giving the illusion that you don't actually need to specify the option.
For some background, I originally wrote cmp in mid-2017 precisely because reflect.DeepEqual did not work properly with protobuf messages. It had always been my vision to release a helper package to cmp that provided options for how to better compare protobuf messages. However, protobuf reflection did not exist in 2018, so we had to build that first. protocmp (at least for me) was the original motivating factor for why protobuf reflection was a necessary feature, but it was also a complex and huge undertaking and took time to convince others that it was necessary and also to actually implement. It's 2020, but we finally have protocmp.

Can you clarify how cmp can and can't be used with generated protobuf messages?

Even prior to the v1.4.0 release, there has always been the presence of pseudo-internal "XXX_" fields that the compatibility document says should not be accessed (even indirectly by libraries like reflect.DeepEqual or cmp.Equal. For example, the XXX_sizecache field exists to speed up proto.Marshal. The presence of a cache means that there was always (even before v1.4.0) a decent chance that cmp.Equal would get the comparison wrong if it wasn't taught how to compare protobuf messages with either cmp.Comparer(proto.Equal) or protocmp.Transform().

Thank you for your detailed response!  It would be great to put this examples to protocmp package doc.
Thank you, for proto API v2 and graceful migration schema!
My colleagues and I had written a lot in-company tools requiring proto reflection, before API v2 release. Now such tools may be written so much simpler.


How to customize TOKEN such as UUID

Expected Behavior
By simply configuring a customized Token, it seems that changing the Token generation strategy is very cumbersome at present?
Current Behavior
Context

I referred to the document docs and found that the client failed the verification based on the custom TOKEN, and many verification methods need to be rewritten. Is there a method similar to spring-cloud-oauth2.0?

@levin950825 Questions are better suited to Stack Overflow. We prefer to use GitHub issues only for bugs and enhancements.


Server not found! message on login page

Right after installation of tendrl I have Server not found! message written on login screen. After I reload the page the screen is loaded correctly.
Tested with:
tendrl-dashboard-1.2.1-04_13_2017_04_11_05.noarch
tendrl-performance-monitoring-1.2.2-04_13_2017_03_02_03.noarch
tendrl-node-agent-1.2.2-04_13_2017_03_06_16.noarch
tendrl-api-1.2.2-04_13_2017_01_51_04.noarch
tendrl-commons-1.2.2-04_13_2017_04_42_04.noarch
Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0

Oh dear! This seems to have made a re-appearance. Anup/Nishanth - can we take a look at this, or, if provided with adequate information - I can run it under debug and provide additional detail.

@Tendrl/qe Fixed. Please verify in tendrl-dashboard-1.2.1-04_17_2017_17_24_12.noarch.rpm

@gnehapk It seems that the problem persists. This screenshot was taken right after tendrl installation. After reloading the page it seems ok.
Tested with:
Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0
tendrl-dashboard-1.2.1-04_18_2017_04_11_04.noarch

@fbalak seems to me more like browser issue, cached things

It was really a problem with cached things. Now it seems good --> closing.


Set dashboards admin for security enabled tests

Description
Fix dashboards admin missing for workspace security enabled tests
Issues Resolved
[List any issues this PR will resolve]
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Seems some cases failed. I guess it may be related to use case change?

Seems some cases failed. I guess it may be related to use case change?

Yeah. I think the root cause is the workspace use case UI updates. The dashboards admin failed will show Create workspace xxxxx failed. in https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/9424921737/job/26059464301?pr=1353. This PR just for fixing this error. The UI update PR will be raised after MDS backport merged.


Fix typo in meetup tests

Fixed an instance of the word 'map' which should have been 'may'

Thanks!


andWhereRaw method for clarity

I noticed that there is a orWhereRaw method as well as an andWhere method (which has identical usage as a simple where) so I thought that for clarity's sake, it would make sense to add an andWhereRaw method.
Otherwise, it seems the naming conventions seem a little inconsistent.

For other searching for this:
whereRaw, orWhereRaw, havingRaw, orHavingRaw are now just .where(knex.raw(rawQuery, bindings)), etc.


rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error

I gave a try to follow our example, but I got some issues as follows.
[root@test /root/Workspace/go/src/github.com/ghz/testdata]
#netstat -lanp|grep httpd
tcp        0      0 <IP_ADDRESS>:80              <IP_ADDRESS>:*               LISTEN      751/httpd
unix  3      [ ]         STREAM     CONNECTED     18157    751/httpd

[root@test /root/Workspace/go/src/github.com/ghz/testdata]
#ghz -proto ./greeter.proto -call helloworld.Greeter.SayHello -d '{"name":"Joe"}' <IP_ADDRESS>:80

Summary:
  Count:	200
  Total:	13.97 ms
  Slowest:	0.00 ms
  Fastest:	0.00 ms
  Average:	-9223372036854.78 ms
  Requests/sec:	14319.87

Response time histogram:

Latency distribution:
Status code distribution:
Error distribution:
  [200]	rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: authentication handshake failed: tls: oversized record received with length 20527"

[root@test /root/Workspace/go/src/github.com/ghz/testdata]
#ghz -insecure -proto ./greeter.proto -call helloworld.Greeter.SayHello -d '{"name":"Joe"}' <IP_ADDRESS>:80

Summary:
  Count:	200
  Total:	18.78 ms
  Slowest:	0.00 ms
  Fastest:	0.00 ms
  Average:	-9223372036854.78 ms
  Requests/sec:	10649.79

Response time histogram:

Latency distribution:
Status code distribution:
Error distribution:
  [150]	rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: <nil>
  [50]	rpc error: code = Unavailable desc = transport is closing

how to solve this issue? thanks in advance!

Hello,
Are you running an actual server at <IP_ADDRESS>:80?

nope.

please provide some more information on the context of this test?

I'm fresh guy for gPRC, I wanna know how to start a gPPC server, thanks in advance!

Hello, you need a grpc server / application running in order to execute load tests against it. You should probably familiarize yourself with gRPC framework in general first. There is great documentation at the official website as well as additional tutorials.


user documentation: next-gen cluster provisioning (those using base clusters)

Hi @cre8minus1 , can you help us with this for Arlon 0.9.0?


unable to use mod in general

I recently tried enabling better nether and better end together and got directed to download BClib, and so I did. The problem now is that; even after disabling my only other mod in the mod pack (because at first, I thought it may be a backwards compatibility error), it seems like BCLib is making my game unplayable by an error when I try to start up my game. Any time I attempt to start up the software with BClib installed into my modpacks folder the game simply will not open. I know this because it won't open while it's in my active mods folder but it will open once BClib and the better mods are in my disabled mods folder.
TL;DR BClib makes it so I can't even open up my game as it gives me an error every time I try to open up my Fabric API version of Minecraft, and once it is disabled I can play with no problems
P.S. Yes, I did try reinstalling and the version is the newest one available as of MC 1.18.1 on February 2, 2022

Please post your most recent log and crash log in pastebin and link here so that details can be seen.  There is an issue currently involving TerraBlender, but exact mods installed and their versions plus any error messages would identify if this is the same or something else.

https://pastebin.com/CKZfjura is the best I can manage, unsure if this contains all the necessary information

You are loading mod for 1.17 on 1.18, please use correct mod version


Error: issubclass() arg 1 must be a class

with replicate version 0.23.0
Running this code:
import replicate
output = replicate.run(
"mistralai/mistral-7b-instruct-v0.1:5fe0a3d7ac2852264a25279d1dfb798acbc4d49711d126646594e212cb821749",
input={
"debug": False,
"top_k": 50,
"top_p": 0.9,
"prompt": "Can you write me a poem about steamed hams?",
"temperature": 0.7,
"max_new_tokens": 500,
"min_new_tokens": -1,
"prompt_template": "{prompt}",
"repetition_penalty": 1.15
}
)
print(output)

I got this error:
TypeError Traceback (most recent call last)
Cell In[2], line 1
----> 1 import replicate
2 output = replicate.run(
3 "mistralai/mistral-7b-instruct-v0.1:5fe0a3d7ac2852264a25279d1dfb798acbc4d49711d126646594e212cb821749",
4 input={
(...)
14 }
15 )
16 print(output)

File ~\anaconda3\lib\site-packages\replicate_init_.py:1
----> 1 from replicate.client import Client
2 from replicate.pagination import async_paginate as _async_paginate
3 from replicate.pagination import paginate as _paginate

File ~\anaconda3\lib\site-packages\replicate\client.py:22
19 from typing_extensions import Unpack
21 from replicate.about import version
---> 22 from replicate.collection import Collections
23 from replicate.deployment import Deployments
24 from replicate.exceptions import ReplicateError

File ~\anaconda3\lib\site-packages\replicate\collection.py:5
1 from typing import Any, Dict, Iterator, List, Optional, Union, overload
3 from typing_extensions import deprecated
----> 5 from replicate.model import Model
6 from replicate.pagination import Page
7 from replicate.resource import Namespace, Resource

File ~\anaconda3\lib\site-packages\replicate\model.py:8
6 from replicate.identifier import ModelVersionIdentifier
7 from replicate.pagination import Page
----> 8 from replicate.prediction import (
9 Prediction,
10 _create_prediction_body,
11 _json_to_prediction,
12 )
13 from replicate.resource import Namespace, Resource
14 from replicate.version import Version, Versions

File ~\anaconda3\lib\site-packages\replicate\prediction.py:27
23 from replicate.client import Client
24 from replicate.stream import ServerSentEvent
---> 27 class Prediction(Resource):
28 """
29 A prediction made by a model hosted on Replicate.
30 """
32 _client: "Client" = pydantic.PrivateAttr()

File ~\anaconda3\lib\site-packages\pydantic\main.py:197, in pydantic.main.ModelMetaclass.new()

File ~\anaconda3\lib\site-packages\pydantic\fields.py:506, in pydantic.fields.ModelField.infer()

File ~\anaconda3\lib\site-packages\pydantic\fields.py:436, in pydantic.fields.ModelField.init()

File ~\anaconda3\lib\site-packages\pydantic\fields.py:552, in pydantic.fields.ModelField.prepare()

File ~\anaconda3\lib\site-packages\pydantic\fields.py:668, in pydantic.fields.ModelField._type_analysis()

File ~\anaconda3\lib\typing.py:852, in _SpecialGenericAlias.subclasscheck(self, cls)
850 return issubclass(cls.origin, self.origin)
851 if not isinstance(cls, _GenericAlias):
--> 852 return issubclass(cls, self.origin)
853 return super().subclasscheck(cls)

TypeError: issubclass() arg 1 must be a class

Hi @bertello72. Based on this issue, this looks like an incompatibility between typing_extensions == 4.6.0 and pydantic < 1.10.8. Try upgrading to pydantic 1.10.8 or higher and let me know if that fixes your problem.

It works updating  pydantic to 1.10.8
Thank you
Il giorno ven 26 gen 2024 alle ore 20:23 Mattt @.***>
ha scritto:

Hi @bertello72 https://github.com/bertello72. Based on this issue
https://github.com/pydantic/pydantic/issues/5821, this looks like an
incompatibility between typing_extensions == 4.6.0 and pydantic < 1.10.8.
Try upgrading to pydantic 1.10.8 or higher and let me know if that fixes
your problem.
—
Reply to this email directly, view it on GitHub
https://github.com/replicate/replicate-python/issues/233#issuecomment-1912574493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFTUJOPU3ADN4OP6Q3FNRATYQP7CZAVCNFSM6AAAAABCMRR5UWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJSGU3TINBZGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fail to apply typeinfo to a python module in a subdirectory

The following will fail :
> python toto\toto.py
( this generates a type_info.json in current directory )

>dir
[...]
25/07/2018  06:22    <DIR>          toto
25/07/2018  06:21               784 type_info.json

>pyannotate -3 toto\toto.py --type-info type_info.json
No files need to be modified.
NOTE: this was a dry run; use -w to write files

strange, there are type annotations in type_info.json with the correct path
>type type_info.json
[
    {
        "path": "toto\\toto.py",
        "line": 2,
        "func_name": "add",
        "type_comments": [
            "(*int) -> int",
            "(*List[int]) -> List[int]",
            "(*Tuple[int, int]) -> Tuple[int, int]"
        ],
        "samples": 3
    },
    {
        "path": "toto\\toto.py",
        "line": 8,
        "func_name": "add2",
        "type_comments": [
            "(Tuple[int, int], Tuple[int, int]) -> Tuple[int, int, int, int]",
            "(List[int], List[int]) -> List[int]",
            "(int, int) -> int"
        ],
        "samples": 3
    },
    {
        "path": "toto\\toto.py",
        "line": 11,
        "func_name": "main",
        "type_comments": [
            "() -> None"
        ],
        "samples": 1
    }
]

edit the type_info.json to remove the "toto\"
>type type_info.json
[
    {
        "path": "toto.py",
        "line": 2,
        "func_name": "add",
        "type_comments": [
            "(*int) -> int",
            "(*List[int]) -> List[int]",
            "(*Tuple[int, int]) -> Tuple[int, int]"
        ],
        "samples": 3
    },
    {
        "path": "toto.py",
        "line": 8,
        "func_name": "add2",
        "type_comments": [
            "(Tuple[int, int], Tuple[int, int]) -> Tuple[int, int, int, int]",
            "(List[int], List[int]) -> List[int]",
            "(int, int) -> int"
        ],
        "samples": 3
    },
    {
        "path": "toto.py",
        "line": 11,
        "func_name": "main",
        "type_comments": [
            "() -> None"
        ],
        "samples": 1
    }
]

try again
>pyannotate -3 toto\toto.py --type-info type_info.json
Refactored toto\toto.py
--- toto\toto.py        (original)
+++ toto\toto.py        (refactored)
@@ -1,14 +1,18 @@
+from typing import Any
+from typing import List
+from typing import Tuple
+from typing import Union

-def add(*args):
+def add(*args: Any) -> Union[List[int], Tuple[int, int], int]:
     ret = args[0]
     for v in args:
         ret += v
     return v

-def add2(v1, v2):
+def add2(v1: Union[List[int], Tuple[int, int], int], v2: Union[List[int], Tuple[int, int], int]) -> Union[List[int], Tuple[int, int, int, int], int]:
     return v1+v2

-def main():
+def main() -> None:
     print( add(1,2,3) )
     print( add([1,2], [3,4]) )
     print( add((1,2), (3,4)) )
Files that need to be modified:
toto\toto.py
NOTE: this was a dry run; use -w to write files

it worked...
It looks like pyannotate is trimming directories from type_info.json too agressively.

Does it work better if you use forward / on the command line (starting from the top)?

Not really. Even if I do that and I convert the backslash to forward slash in type_info.json .

Hm... What if you add an empty __init__.py file to the toto folder? (I'm
quite sure that this works at Dropbox.)

I tracked down the problem up to here : https://github.com/dropbox/pyannotate/blob/a01510d2960cb8fb0096f6c017391ea9b9e98689/pyannotate_tools/fixes/fix_annotate_json.py#L205
self.class.top_dir is 'D:\work\pyann\pyannotate\toto' and it['path'] is 'toto\toto.py' . So joining them together won't work, the directory toto is duplicated in the path.
Looks like crawl_up() ( https://github.com/dropbox/pyannotate/blob/a01510d2960cb8fb0096f6c017391ea9b9e98689/pyannotate_tools/fixes/fix_annotate_json.py#L43 ) was not designed to handle such cases. It miscalculates the top directory in this case.

The code could use more tests in general, and in particular to capture precisely this kind of behavior.
I would expect that this is needed if you are working with multiple in development packages and you are adjusting sys.path to pick all these packages (maybe even with pip). In this case, relating getcwd()  current execution dir is no longer the right thing to do.

I have encountered the same issue and have a small patch based on observations from @gvanrossum that works for me, maybe someone else will benefit from this. For a proper testing there will be more work involved, and I think that a command line option to guide the matching of sources to the json would be good. It could also be embedded in the json collection, so that we find the supposed topdir and embed what we find at collection time into the json that would help the annotator match up.

FWIW I also ran into this, and I just commented out crawl_up() in my local copy for now.  For some reason it caused the paths in the items comparison to be one subdirectory off.
Before the change, I had a good type_info.json file, but nothing was being applied.  After the change, everything was applied as far as I can tell.

I am also experiencing the exact same thing.
It only works by mofiying the paths in the JSON from:
[
    {
        "path": "C:/Users/Me/Documents/GitHub/sabnzbd3/SABnzbd.py",
        "line": 102,
        "func_name": "GUIHandler",
        "type_comments": [
            "() -> None"
        ],
        "samples": 1
    },
]

to
[
    {
        "path": "SABnzbd.py",
        "line": 102,
        "func_name": "GUIHandler",
        "type_comments": [
            "() -> None"
        ],
        "samples": 1
    },
]

After this it did exactly what I hoped it would do. Thanks very much for this tool ❤

Unfortunately I have no time available to work on this. If someone could prepare a patch, including a bunch of tests that show how it works in various common and edge cases, I would very much appreciate it.

Yes I will do it
On Fri, Mar 22, 2019, 11:14 Guido van Rossum<EMAIL_ADDRESS>wrote:

Unfortunately I have no time available to work on this. If someone could
prepare a patch, including a bunch of tests that show how it works in
various common and edge cases, I would very much appreciate it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dropbox/pyannotate/issues/75#issuecomment-475659594,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APqoaWuMEu0TbiozBy9zwhrlJ2K9MufOks5vZPNGgaJpZM4Vfa9i
.

I was thinking it could be as simple as adding a flag like --crawl-up or --no-crawl-up ?  As mentioned I could simply delete the crawl_up() call and things work (for me), but that might break some unknown things witin Dropbox?


Changing module loading logic so modules aren't merged into runner until loading is complete

This PR is a blocker for fixing the relative_submodule_import_test, and resolving several other issues.
It's a fairly substantial internal refactor to the module-loading process, but the only outward-facing change should be that modules are loaded as child modules of the module loading them, as opposed to loading at the top-level of the module namespace.

Thank Luke, I am happy to merge this.


updating Gaia to 8.0.1

updating Gaia to 8.0.1

@clydedevv https://github.com/akash-network/cosmos-omnibus/blob/v0.3.24/.github/workflows/publish.yml#L40 needs to be updated as well

bumped https://github.com/akash-network/cosmos-omnibus/pull/423


ENH/API: add cropping  / tmin-tmax paramters to time_frequency functions to avoid edge artifacts

... like fieldtrip does.  Also related to GSOC-topic-4

@dengemann Is it finished? If not, can I take it?

Is it finished? If not, can I take it?

I thinkt this is not done yet. It should be consistently implmented for tfr_* functions.

@dlsrnsi go for it !

@agramfort Thanks sir!


[ENHANCE] Add "Account ID parameter name" to Register/Update Provider forms, & to provider detail modal

Summary
Requiring providers to use account_id key for account ID parameter in method args is not flexible, e.g. in the case of a javascript contract it would more likely use camel case, or in the case of other contracts, e.g. I-Am-Human, they use a different name, account, for consistency within their contract.
Therefore, we want to provide the option for a provider registrant to specify the name of their account ID parameter, and they should also be able to update this.
Probably helpful if it defaults to account_id
Description
See above
Acceptance Criteria
[List of criteria that must be met for the enhancement to be considered accepted]
Additional Information

See updated Sybil readme for latest documentation (see ProviderExternal struct and register_provider and update_provider methods)

This non-breaking smart contract change has been deployed to all three contracts (testnet, staging & mainnet)

Done


Pico 4-Button Scene Controller

Is your feature request related to a problem? Please describe:

Describe the solution you'd like:

Are you able to add the 4-button scene controller PICO - this would be huge!  PJ2-4B-GWH-XXX remotes
Describe alternatives you've considered:

Additional context:

This looks like a dupe of #26. If you could grab the output of the script there, I can add support pretty quickly.


Add Vector2Dir function to get the direction vector of two given points

I always add this function anyway on my raylib projects, so I think that could be usefull to get this function directly in raymath.
But it's really not mandatory.
Note: the function name is quite "quick and dirty", but hey, it is probably enough?

Isn't this just Vector2Subtract with the arguments flipped? What's the need for this?

Isn't this just Vector2Subtract with the arguments flipped? What's the need for this?

Yes, totally! Indeed, but actually it's just the name which matters. Anyway, I didn't even took attention to Vector2Substract! Gosh I'm tired haha

I understand the intention of @Elkantor. Calculating the direction vector of a line (or a plane) is a common operation when working with vectors. Here is my opinion on this. In case we are adding the function, I would:

Rename it to Vector2Direction, which is a more self-explanatory.
Vector2 result = Vector2Subtract(v2, v1); instead of Vector2 result = { p2.x - p1.x, p2.y - p1.y };
Add the same function for a plane (defined by 3 points)

But this is just my opinion :) I hope it helps.

@Peter0x44 thanks for looking into this, I was aware of some similar function already available.
@Elkantor @idircarlos A Vector2Direction() could be useful but probably as a normalized vector, afaik a direction is more useful normalized, right? Not sure if there is any similar alternative.


chore: release screenpipe to official Homebrew/core

name: chore: release screenpipe to official Homebrew/core
about: updates the docs to use official homebrew instead of repository tap
title: "chore: release screenpipe to official Homebrew/core"
labels: ''
assignees: ''

description
release screenpipe to official Homebrew/core
Closes #614
/claim #614
related PR on homebrew https://github.com/Homebrew/homebrew-core/pull/196345
type of change

[x] documentation update

checklist

[x] MOST IMPORTANT: this PR will require less than 30 min to review, merge, and release to production and not crash in the hand of thousands of users
[x] i have read the CONTRIBUTING.md file
[x] i have updated the documentation if necessary
[x] my changes generate no new warnings
[ ] i have added tests that prove my fix is effective or that my feature works

@Anshgrover23 thanks a lot
can you share what i need to do to maintain this?
what happen when i release a new version on this repo?
what happen if i need an additional dependency or
seem to work with linux? not arm i suppose?

/approve

@Anshgrover23 thanks a lot for this


Check for config dir existence first

Description

As a user, I would expect driftctl to check for the config-dir availability and writability before going further and download a provider.
❯  driftctl scan --config-dir /yadda
[...]
DEBU[0000] New provider library created
DEBU[0000] provider not found, downloading ... path=/yadda/.driftctl/plugins/linux_amd64/terraform-provider-aws_v3.44.0
Downloading terraform provider: aws
DEBU[0000] Downloading provider path=/yadda/.driftctl/plugins/linux_amd64 url=https://releases.hashicorp.com/terraform-provider-aws/3.44.0/terraform-provider-aws_3.44.0_linux_amd64.zip
DEBU[0005] Decompressing archive dst=/yadda/.driftctl/plugins/linux_amd64 src=/tmp/terraform-provider274317203
mkdir /yadda: permission denied
❯  echo $?
1

Environment

OS: fedora 34
driftctl version: 0.10.0-pre
terraform version: 1.0.0
terraform providers versions<EMAIL_ADDRESS>
How to reproduce

See above (--config-dir /yadda is enough)
Possible Solution

dir exists?
dir is writable?

Additional context

nevermind


Update to ReSharper 9.0

Looks like it's not a trivial update... was anybody able to update it to R# 9?


Secrets configuration

Hi
I'm trying to run container with next configuration:
{
  "LogLevel": "Debug",
  "Secrets": {
    "Path": "/app/test",
    "Mapping": {
      "ambariConnectionUsername": "AmbariConnection--Username",
      "ambariConnectionPassword": "AmbariConnection--Password"
    }
  },
  "BaseExporterConfiguration": {
    "ClusterName": "cli-cluster",
    "AmbariServerUri": "http://<IP_ADDRESS>:8080/api/v1/clusters",
    "DefaultLabels": {
      "TestLabelName": "TestLabelValue"
    }
  },
  "PrometheusExporterConfiguration": {
    "Port": 8086
  },
  "LivenessConfiguration": {
    "SamplingPeriodInMinutes" : 1
  }
}

I'm using next command:
docker run -it --rm --network=host -v $PWD/SecretFolder:/app/test -v $PWD/appsettings.json:/app/appsettings.json ambari-exporter:latest

SecretFolder contains 2 files:
ambariConnectionPassword
ambariConnectionUsername

I can successfully access ambari with command:
curl -u admin:password http://<IP_ADDRESS>:8080/api/v1/clusters/

But app is crashing with an error:
Fetching secrets from folder /app/test.
SecretConfigurationExtension::AddSecretProvider - resolved 2 secrets.
SecretConfigurationExtension::AddSecretProvider - mapped the following: 'ambariConnectionPassword' -> 'AmbariConnection:Password'.
SecretConfigurationExtension::AddSecretProvider - mapped the following: 'ambariConnectionUsername' -> 'AmbariConnection:Username'.
{"LogTimestamp":"2020-01-15T16:03:08.8934489Z","Level":"Debug","LevelId":1,"Message":"Hosting starting","CustomFields":{"EventId":{"Id":1},"SourceContext":"Microsoft.Extensions.Hosting.Internal.Host"}}
{"LogTimestamp":"2020-01-15T16:03:09.1784481Z","Level":"Information","LevelId":2,"Message":"Starting Prometheus MetricServer at 01/15/2020 16:03:09.","CustomFields":{"SourceContext":"App.Services.Hosted.PrometheusExporterHostedService"}}
{"LogTimestamp":"2020-01-15T16:03:09.2078564Z","Level":"Information","LevelId":2,"Message":"Starting the service - initializing and starting the timer.","CustomFields":{"SourceContext":"App.Services.Hosted.LivenessHostedService"}}
Application started. Press Ctrl+C to shut down.
Hosting environment: Production
Content root path: /app/
{"LogTimestamp":"2020-01-15T16:03:09.2100355Z","Level":"Debug","LevelId":1,"Message":"Hosting started","CustomFields":{"EventId":{"Id":2},"SourceContext":"Microsoft.Extensions.Hosting.Internal.Host"}}
{"LogTimestamp":"2020-01-15T16:03:09.2133152Z","Level":"Information","LevelId":2,"Message":"Running HealthCheck.","CustomFields":{"SourceContext":"App.Services.Hosted.LivenessHostedService"}}
{"LogTimestamp":"2020-01-15T16:03:09.3101379Z","Level":"Error","LevelId":4,"Message":"Failed to get response from Ambari: http://<IP_ADDRESS>:8080/api/v1/clusters.","Exception":{"Type":"System.Net.Http.HttpRequestException","Message":"Status code: Forbidden isnt Ok.","StackTrace":"   at Infrastructure.Providers.Concrete.AmbariApiContentProvider.GetResponseContentAsync(String uriEndpoint) in /app/src/Infrastructure/Providers/Concrete/AmbariApiContentProvider.cs:line 58"},"CustomFields":{"SourceContext":"Infrastructure.Providers.Concrete.AmbariApiContentProvider"}}
{"LogTimestamp":"2020-01-15T16:03:09.3340873Z","Level":"Error","LevelId":4,"Message":"HealthCheck failed.","Exception":{"Type":"System.Net.Http.HttpRequestException","Message":"Status code: Forbidden isnt Ok.","StackTrace":"   at Infrastructure.Providers.Concrete.AmbariApiContentProvider.GetResponseContentAsync(String uriEndpoint) in /app/src/Infrastructure/Providers/Concrete/AmbariApiContentProvider.cs:line 58\n   at App.Services.Hosted.LivenessHostedService.HealthCheck(Object state) in /app/src/App/Services/Hosted/LivenessHostedService.cs:line 73"},"CustomFields":{"SourceContext":"App.Services.Hosted.LivenessHostedService"}}

Unhandled Exception: System.Net.Http.HttpRequestException: Status code: Forbidden isnt Ok.
   at App.Services.Hosted.LivenessHostedService.HealthCheck(Object state) in /app/src/App/Services/Hosted/LivenessHostedService.cs:line 82
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ThreadPoolWorkQueue.Dispatch()

It looks like I misconfigured login and password

Hey @xadcoh ,
Thank you for your feedback and sorry for the very late response.
I will look into that and communicate back.


Pin image to v2.3.1

Quarkus doesn't support the latest version yet so we need to pin this to v2.3.1.

@skateman fyi

/retest


change Title and Navabar name "CareerVibe"  replace to "Sanglinaukri.com"

we need to change Title and Navabar name "CareerVibe" replace to "HelloSangli" disscuss with Owner

cc4a1db


Pos Report Information

Hi,
We also try to use the pos report file, have you informations about the files contents, the definitions ?
Regards
Sébastien

The file contains the position of boatss at every report.


Update covariates when model structure changes

Covariates need to be updated when the model structure is changed. If the user put a covariate on CL and then changes the model to Michaelis Menten elimination there is no CL parameter anymore and the program will crash. Covariates should be updated similarly to rvs in _update_rvs_from_model.

Fixed in https://github.com/pharmpy/modelbuilder/commit/001497f11d8e778ce964a7adcccc9e23a910838a

This is a good point. I'll keep this open so we can double check how this works.


X-Axis labels cutting from bottom

Expected Behavior
X-Axis labels visible properly
Current Behavior
X-Axis labels cuts from bottom
https://jsfiddle.net/nitinjs/3nkwbeh0/7/
to replicate in firefox press CTRL + shift + m in firefox

Possible Solution
maybe margin from bottom resolves the issue
Steps to Reproduce (for bugs)
checkout the fiddle https://jsfiddle.net/nitinjs/3nkwbeh0/7/
Context
I am creating vertical x axis labels however the labels cut off at bottom on mobile device:

Environment

Chart.js version: 2.9.3
Browser name and version: Firefox 72.0.2
Link to your project: https://jsfiddle.net/nitinjs/3nkwbeh0/7/

Found solution: https://jsfiddle.net/nitinjs/3nkwbeh0/29/
it was adding
maintainAspectRatio: false,
in options
SO thread https://stackoverflow.com/questions/60147365/chartjs-x-axis-labels-cutting-at-bottom/60225797#60225797


Paperspace Last cell error

Hello, I keep getting this error in the paperspace version:
ControlNet v1.1.150
ControlNet v1.1.150
Loading weights [a09b824f36] from /notebooks/sd/stable-diffusion-webui/models/Stable-diffusion/Soyou_Anime_25D.safetensors
loading stable diffusion model: RuntimeError
Traceback (most recent call last):
File "/notebooks/sd/stable-diffusion-webui/webui.py", line 195, in initialize
modules.sd_models.load_model()
File "/notebooks/sd/stable-diffusion-webui/modules/sd_models.py", line 424, in load_model
state_dict = get_checkpoint_state_dict(checkpoint_info, timer)
File "/notebooks/sd/stable-diffusion-webui/modules/sd_models.py", line 271, in get_checkpoint_state_dict
res = read_state_dict(checkpoint_info.filename)
File "/notebooks/sd/stable-diffusion-webui/modules/sd_models.py", line 250, in read_state_dict
pl_sd = safetensors.torch.load_file(checkpoint_file, device=device)
File "/usr/local/lib/python3.9/dist-packages/safetensors/torch.py", line 100, in load_file
result[k] = f.get_tensor(k)
RuntimeError: shape '[77, 768]' is invalid for input of size 0
Stable diffusion model failed to load, exiting

Wait, nvm. It seems I have issue with my model. I think something to do with Paperspace unable to upload files from outside US.

check your disk space limits

Is meant disk space C

ControlNet v1.1.150
ControlNet v1.1.150
Calculating sha256 for /models/model.safetensors: f6cbc1d7302d08737dd4686c0a6455e2c2918d4c2ea09d7af1cdf877b9a864cc
Loading weights [f6cbc1d730] from /models/model.safetensors
Creating model from config: /notebooks/sd/stable-diffusion-webui/configs/v1-inference.yaml
LatentDiffusion: Running in eps-prediction mode
DiffusionWrapper has 859.52 M params.
Applying cross attention optimization (Doggettx).
Textual inversion embeddings loaded(0):
Model loaded in 14.0s (calculate hash: 7.2s, create model: 1.1s, apply weights to model: 1.7s, apply half(): 1.4s, load VAE: 2.2s, move model to device: 0.5s).
This cell shows running and after waiting for a long time I'm not getting the webui link. Yesterday it worked fine for me but today not getting the link and the cell shows its running....Any idea what's the issue here ? Please help

This cell shows running and after waiting for a long time I'm not getting the webui link. Yesterday it worked fine for me but today not getting the link and the cell shows its running....Any idea what's the issue here ? Please help

I started having the same issue just now.


[W10] App crashes when clicking on a menu item after opening a context menu

Reproduction: https://github.com/ferreira-tb/context-menu-crash
Note: This doesn't happen all the time, but most of the time it does. If it doesn't, restart the app and try again.
How to reproduce:

Open a context menu by right-clicking anywhere in the window.
Click on "item1"
Open the "main-menu" on the main menu bar.
Click on "sub-item1"
Click on "sub-item2"
The app crashes.

Environment

OS: Windows 10.0.19045 X64
WebView2: 124.0.2478.80
MSVC: Visual Studio Community 2022
rustc: 1.78.0 (9b00956e5 2024-04-29)
cargo: 1.78.0 (54d8815d0 2024-03-26)
rustup: 1.27.1 (54dd3d00f 2024-04-24)
Rust toolchain: stable-x86_64-pc-windows-msvc (environment override by RUSTUP_TOOLCHAIN)

Packages

tauri [RUST]: 2.0.0-beta.19
tauri-build [RUST]: 2.0.0-beta.15
wry [RUST]: 0.39.5
tao [RUST]: 0.28.0
muda [RUST]: 0.13.2
tauri-cli [RUST]: 2.0.0-beta.13
@tauri-apps/api [NPM]: 2.0.0-beta.11
@tauri-apps/cli [NPM]: 2.0.0-beta.13

This crash looks the same as #9470, please try the suggested solution in https://github.com/tauri-apps/tauri/issues/9470#issuecomment-2111339835


Converted SuperTextField into a platform adaptive widget (Resolves #413)

Converted SuperTextField into a platform adaptive widget (Resolves #413)
@miguelcmedeiros can you give this PR a try in the Superlist app and let me know if the new adaptive SuperTextField widget satisfies your x-platform text field needs?

@venkatd - you might be interested in this PR because it impacts the text field widget APIs

@miguelcmedeiros

I also noticed that the desktop, iOS and Android text field widgets have new fields, but they are not used.

Can you elaborate on this? I'm not sure what you mean.

@miguelcmedeiros

we can't use it in it's current shape since we are passing keyboardHandlers or hintBehavior

I unified the hint APIs across all the text fields, and exposed hintBehavior and keyboardHandlers in SuperTextField.

This is a tricky one since we have a text style builder and the dev's won't really know which line height to pass here.

I changed the requirements so that you don't need to pass lineHeight for single-line fields, but you're going to need to provide it multi-line fields. Please see the demo app examples.
Can you take another pass and see if this now works for your app?

I believe autofocus should be exposed on SuperTextField in a similar manner to SuperEditor.

Tried it out in our app and noticed the following behaviour which is reproduced using the demo app:

In case the text of the SuperTextField fits in the given viewport and when moving the selection handles close to the edge of the viewport, then there is a slight motion of the content.

@matthew-carroll
It seems that the keyboard appearance (light and dark modes) is not according to the OS settings:

I'll address the couple of API issues that @miguelcmedeiros mentioned.
For the rest of what you guys find, can you file individual tickets for those things? Whatever the scrolling problem or keyboard theming problem, those must be about 6 months old. I'd like to focus this review on the goal of unifying the existing text fields so that developers don't have to worry about which platform they're on, if they don't want to.

@miguelcmedeiros textAlign is now used, and lineHeight is only required when maxLines is > 1

@matthew-carroll looks great to me!
This may allow us to remove some workarounds we've had in place to use a SuperTextField on desktop and the built-in TextField on mobile devices.
How is defaultTextFieldKeyboardHandlers used? I guess this is only something that is taken into consideration on platforms that have actual keyboard handling?

@venkatd

How is defaultTextFieldKeyboardHandlers used? I guess this is only something that is taken into consideration on platforms that have actual keyboard handling?

Yeah, the only way I can think to unify the API is to take mobile and desktop pieces together. Otherwise, you must know what platform you're on, in which case you wouldn't need a unified API.
I'm gonna merge this in. Please let me know ASAP if you think we should make further adjustments to the unified API.

@matthew-carroll all LGTM! If defaultTextFieldKeyboardHandlers is handled just as before, the API looks good to me 👍


Implement a custom httpclient and a golang client to interact with the API

Codecov Report

Merging #6 (ee73f9b) into master (520350d) will decrease coverage by 34.97%.
The diff coverage is 5.97%.

@@             Coverage Diff             @@
##           master       #6       +/-   ##
===========================================
- Coverage   71.75%   36.77%   -34.98%
===========================================
  Files          12       17        +5
  Lines         177      378      +201
===========================================
+ Hits          127      139       +12
- Misses         40      227      +187
- Partials       10       12        +2

Impacted Files
Coverage Δ

pkg/client/api/client.go
0.00% <0.00%> (ø)

pkg/client/api/v1/client.go
0.00% <0.00%> (ø)

pkg/client/api/v1/project.go
0.00% <0.00%> (ø)

pkg/client/perseshttp/client.go
0.00% <0.00%> (ø)

pkg/client/perseshttp/request.go
9.44% <9.44%> (ø)

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 520350d...ee73f9b. Read the comment docs.


What is the relationship between cursorcore and cursor

CursorCore is a project focuses on implementing the core features of Cursor.


feat: scroll selected resource into view if not visible

introduced AppContext to avoid prop drilling of the windowSize
This PR...
Changes

scrolls selected resource into view if it's not visible, which allows us to click links in the navigator/editor and the newly selected resource will be scrolled into view

Fixes

Checklist

[x] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

thanks @catalinhoha - does this also handle the case of expanding the containing section/subsection if needed to show the selected resource?

we can do it in a separate issue/PR if you prefer - i.e. "Auto-expand sections when scrolling into view"...

ok, that works, let's merge this in


kong1.3.0 compile on arm64 failed

we compile kong on arm64 failed with source code. when executing 'kong migratioins up', the prompt 'bad light userdata pointer'

Kong version (1.3.0)
Operating system: centos
Architecture: arm64

Refer to the online Dockerfile, I made a 1.3.0 version of the arm64 image. I tested the service and routes' query and add are normal, you can just try this.
bevisy/kong:1.3.0_arm64

Refer to the online Dockerfile, I made a 1.3.0 version of the arm64 image. I tested the service and routes' query and add are normal, you can just try this.
bevisy/kong:1.3.0_arm64
I did not find the image(bevisy/kong:1.3.0_arm64). Can you send a link to me, or send me an email(hxp628@163.com), thank you.

I did not find the image(bevisy/kong:1.3.0_arm64). Can you send a link to me, or send me an email(hxp628@163.com), thank you.

you can find here

Hi, ARM64 support was added to docker-kong very recently, in case you want to give them a look as well (they use Ubuntu for the OS). Please let us know if that fixes your issue.


Update sys-dm-exec-connections-transact-sql.md to add more information on the encrypt_option column.

This stems from the following IcM.
https://portal.microsofticm.com/imp/v5/incidents/details/518135342/summary. The purpose of the change is to show that encrypt_option is not valid for HADR endpoints, For HADR endpoints, users need to use the sys.database_mirroring_endpoints dmv to find out whether encryption is enabled on it.

Please create this PR internally.

Replaced by https://github.com/MicrosoftDocs/sql-docs-pr/pull/31012.


Hot-reload support to celery

Description
This PR adds support for celery hot-reloading while developing, on changes to tasks.py and celery_app.py files when setting up a project with docker and celery.
Rationale
I have been using this Cookiecutter template for many projects and while developing with celery tasks I had to restart the worker manually to test the changes. This minor change will allow developers to test their celery tasks faster
Use case(s) / visualization(s)
-> Develop celery tasks -> notice a bug -> change the code -> save -> test the new code without restarting the worker manually

Nice addition!
Is the pattern not to tight? Now the restarts do not happen when there are changes made to settings.py or models.py. So why not set it to *.py?

@foarsitter I configured it that way on purpose, since I write all my tasks in those files.
I tried setting it as *.py before and then it restarts the worker on every save and the hot reload takes more time and mess up the logs, so I think this is the most efficient way to go :)

Nice addition indeed. Good question for the pattern, I'm not sure I undertsnad the problem, but I probably just need to try it out 😄

Nice work! Consider using watchgod instead of watchdog.

This is a great feature idea. Is there anything still needed before this can be merged (besides simply resolving the merge conflicts)?

Yes, I kind of forgot about it, I think it's good to go.

Thanks for your contribution @yotamtal 🎉


Allow non-empty server params

As per PSR-7:
    /**
     * Retrieve server parameters.
     *
     * Retrieves data related to the incoming request environment,
     * typically derived from PHP's $_SERVER superglobal. The data IS NOT
     * REQUIRED to originate from $_SERVER.
     *
     * @return array
     */
    public function getServerParams();

Can you also tag a patch release please?

Tagged 0.2.1


Feature requested: auto-scroll to first merge conflicts

When a file in merge conflict is opened from Source Control list, it would be more productive to auto-scroll to the first merge conflict area.

@Jnp95, are you using the 3-way merge editor, or the code lenses to resolve merge conflicts?

Essentially the code lens merge editor.
(Not fan of 3 way merge editor)
Le mer. 15 mars 2023, 18:38, Ladislau Szomoru @.***> a
écrit :

@Jnp95 https://github.com/Jnp95, are you using the 3-way merge editor,
or the code lenses to resolve merge conflicts?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/176742#issuecomment-1470473137,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARRBX3KSM6X4QUWGY2DUOQLW4H5CHANCNFSM6AAAAAAVWFXCSI
.
You are receiving this because you were mentioned.Message ID:
@.***>

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

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Structs can't have finalizers because they are not tracked by the GC

> crystal spec
Error in line 1: while requiring "./spec/bulk_operation_spec.cr"

in spec/bulk_operation_spec.cr:1: while requiring "../src/mongo"

require "../src/mongo"
^

in src/mongo.cr:2: while requiring "./bson"

require "./bson"
^

in src/bson.cr:3: while requiring "./bson/*"

require "./bson/*"
^

in src/bson/value.cr:14: structs can't have finalizers because they are not tracked by the GC

    def finalize
        ^~~~~~~~

From https://github.com/crystal-lang/crystal/blob/master/CHANGELOG.md#0204-06-01-2017

(breaking change) Defining a finalize method on a struct now gives a compile error

Because of memory leaks, see https://github.com/crystal-lang/crystal/issues/3840

Fixed in #15


cleanup / fix setup.py, and add license

re and requests weren't used in this file, and weren't declared in
setup_requires causing some build issues. Removed these imports.
Removed mixed tabs & spaces.
Explicitly added the Apache License to list of Classifiers so this
project can be identified as Apache Licensed at pypi.org. This matches
the license file.

Coverage remained the same at 39.475% when pulling 268440965e453371955ffcb0672c8283b10af1ad on drfloob:master into a3cd665e8344e10a12ffad5575944a66188df6b0 on ChristianTremblay:master.

Eek, how did the mixed tabs/spaces get in there?  I'll have to put a vi mode-line in to fix that.  Many thanks for the contribution.

@sjlongland  Can you / do you plan to cut a patch release so we can pull this via pypi? It would be very helpful.

On 9/2/19 8:45 am, Adam J Heller wrote:

@sjlongland https://github.com/sjlongland Can you / do you plan to cut
a patch release so we can pull this via pypi? It would be very helpful.

I'm not sure when the next release will be.  For what it's worth, I have
to sync VRT's branch up so all WideSky-related fixes go into mainline.
I'll chat with Christian about it. :-)
--
Stuart Longland (aka Redhatter, VK4MSL)
I haven't lost my mind...
...it's backed up on a tape somewhere.

@drfloob Thanks for the update. I will try to make that available ASAP.
@sjlongland I’ll try to reach you so we can make that happen

Just realised a cock-up I just made, that should have been merged to develop not master… ahh well.  Not a difficult thing to fix.


0.2.0

Merging into master

Changes Unknown when pulling f7ec637767528ec2ee0594f1f921972f8e60fea9 on 0.2.0 into ** on master**.

Changes Unknown when pulling caf1d6152dc1dfbff49206916b2e85f96415c7e6 on 0.2.0 into ** on master**.

Merging back into master. Developing in master makes more sense. Releasing versions through github releases instead.


Broken links on the Wiki

The following Wiki page links

All seems to point to the equivalent pages on @FlorianRappl fork, which all are empty. The links in the sidebar works fine.
I could not edit them myself because community edits are disabled

Thanks, yes updated. I wanted to move away Wiki anyway with a full documentation, but currently this is all being delayed due to my workload.


steempy fails with error RPCErrorRecoverable

Version of Python you are running
Tried with 3.6.4 and 3.6.5 on macOS 10.13.5
Version of steem-python you are running
steempy  --version
steempy 1.0.1

Expected Behavior
setting of nodes should work
Actual Behavior
steempy set nodes https://gtg.steem.house:8090
WARNING:root:Retry in 1s -- RPCErrorRecoverable: non-200 response: 400 from None
WARNING:root:Retry in 2s -- RPCErrorRecoverable: non-200 response: 400 from None
WARNING:root:Retry in 3s -- RPCErrorRecoverable: non-200 response: 400 from None
WARNING:root:Retry in 4s -- RPCErrorRecoverable: non-200 response: 400 from None
WARNING:root:Retry in 5s -- RPCErrorRecoverable: non-200 response: 400 from None
WARNING:root:Retry in 6s -- RPCErrorRecoverable: non-200 response: 400 from None
WARNING:root:Retry in 7s -- RPCErrorRecoverable: non-200 response: 400 from None
WARNING:root:Retry in 8s -- RPCErrorRecoverable: non-200 response: 400 from None
WARNING:root:Retry in 9s -- RPCErrorRecoverable: non-200 response: 400 from None
WARNING:root:Retry in 10s -- RPCErrorRecoverable: non-200 response: 400 from None
ERROR:root:Failed after 10 attempts -- RPCErrorRecoverable: non-200 response: 400 from None
Traceback (most recent call last):
  File "/Users/b5413b/.pyenv/versions/3.6.4/bin/steempy", line 11, in <module>
    sys.exit(legacyentry())
  File "/Users/b5413b/.pyenv/versions/3.6.4/lib/python3.6/site-packages/steem/cli.py", line 776, in legacyentry
    steem = stm.Steem(no_broadcast=args.no_broadcast, **options)
  File "/Users/b5413b/.pyenv/versions/3.6.4/lib/python3.6/site-packages/steem/steem.py", line 59, in __init__
    steemd_instance=self.steemd, no_broadcast=no_broadcast, **kwargs)
  File "/Users/b5413b/.pyenv/versions/3.6.4/lib/python3.6/site-packages/steem/commit.py", line 96, in __init__
    self.wallet = Wallet(self.steemd, **kwargs)
  File "/Users/b5413b/.pyenv/versions/3.6.4/lib/python3.6/site-packages/steem/wallet.py", line 61, in __init__
    self.prefix = self.steemd.chain_params["prefix"]
  File "/Users/b5413b/.pyenv/versions/3.6.4/lib/python3.6/site-packages/steem/steemd.py", line 65, in chain_params
    props = self.get_dynamic_global_properties()
  File "/Users/b5413b/.pyenv/versions/3.6.4/lib/python3.6/site-packages/steem/steemd.py", line 419, in get_dynamic_global_properties
    return self.call('get_dynamic_global_properties', api='database_api')
  File "/Users/b5413b/.pyenv/versions/3.6.4/lib/python3.6/site-packages/steembase/http_client.py", line 288, in call
    raise e
  File "/Users/b5413b/.pyenv/versions/3.6.4/lib/python3.6/site-packages/steembase/http_client.py", line 246, in call
    % (response.status, self.hostname))
steembase.exceptions.RPCErrorRecoverable: non-200 response: 400 from None

```### Steps to reproduce ###
just run any command using steempy

### Stack Trace ###

already provided

I have the same problem. I can't reset to another node

For me the same behavior. 1.0.2


masonry: Rename a scale factor variable

This helps make it more clear when skimming through this code that this is the scale factor being used to transform the scene.

This may seem silly, but this cost me a lot of time in tracking down why my own integration wasn't working correctly.
I should probably help add some comments / documentation somewhere to help document / describe how window scale factors are used.


Add "Macro expand" tool to Playground

This PR add new tool "Macro expand" to Playground  (#158). Under the hood it call cargo run -- --Zunstable-options --pretty=expanded with nightly compiler.
How it looks:

👍

Thank you! I'll try to set aside some time to review this in the near future!

FWIW, I force-pushed your branch to make some small changes, but will merge and deploy soon.
Thanks!


Completely crashes monitor after starting the app

When starting the app my monitor turns black after a couple of seconds. I cant even turn it off via its hardware button in this state. When closing MonitorControl the screen turns on again and everything is fine.
For debugging I have turned off DDC/CI in my monitors osd settings and even then the bug above appeared.
Monitor: BenQ GW2780
btw by using just https://github.com/kfix/ddcctl I can control the monitor without any issues via terminal.

fixed in 1.4.0


Organize popover menu form

This is the first of a three-part PR involving the popover menu. In part one, the goal was the split up the logic and the structure of the popover component. Also, the goal of this was the remove potential mismatches of related data. For example, the component name depends on the component label, so this PR allows one to derive the name from the label.
Let me know what you think! I tried to split up the code in a way I thought was logical, but if you disagree, let me know and we can find a better solution!
Part 2 will be validating the form.
Part 3 will be testing the form.
Also, I didn't realize that Jon is editing the Popover so I apologize for the merge conflicts we will encounter at some point.
closes #56

I'd like to test this some more before it gets merged in.

Okay, this is good. There is currently a bug where starting a schedule from scratch causes the app to crash, but that'll be fixed in another PR.

Wow, this looks a lot nicer! The organization looks good for now though it will likely need updating again when the additional fields needed for export are added (and when the location field is separated into building and room number). The organization of the code seems to make sense. There will likely be merge conflicts with my code as well (because I update the interfaces in #58).

Wow, this looks a lot nicer! The organization looks good for now though it will likely need updating again when the additional fields needed for export are added (and when the location field is separated into building and room number). The organization of the code seems to make sense. There will likely be merge conflicts with my code as well (because I update the interfaces in #58).

Thanks for the heads up! We'll make a PR in the future to resolve those new features.


Minor issues with pop-up card

When I lock the screen or screen off while a pop-up card is in the foreground, next time when I unlock the phone, I must click to open another pop-up card before I can open the same one that was left on the screen before the screen is off/lock.
To reproduce the issue:

Trigger a pop-up card to open (e.g. #kitchen)
Screen off / lock the phone.
Trigger again the same pop-up (e.g. #kitchen) and it won't work.
Trigger any other pop-up and then pop-up #kitchen and it will work again.

Hi! This is now fixed in the new v1.6.0 beta!
https://github.com/Clooos/Bubble-Card/releases/tag/v1.6.0-beta.1


docker-compose up ERROR

Hello everyone, I started docker-compose up, and the error log: ”(root)Additional property name is not allowed“
Where is the problem?
Docker Compose version v2.2.2

any more details? logs?

docker-compose  up
error log:
(root) Additional property name is not allowed

[root@Cluster spline]# docker-compose up
(root) Additional property name is not allowed
[root@Cluster spline]# docker-compose ps
(root) Additional property name is not allowed
[root@Cluster spline]# docker-compose down
(root) Additional property name is not allowed
[root@Cluster spline]# docker-compose -f compose.yaml up
(root) Additional property name is not allowed

The current docker-compose command is available, I don’t know where the problem is, please help, thank you very much!

Thanks, I will try your Docker-compose version.

I think you can remove name property from compose.yml. Maybe it can be OK


Toilet

Song Name
Toilet
Artist Name
Me
Source
Direct file download
Youtube Link
No response
Direct File Link
No response
Song ID
818027
Start Offset [ms]
No response
Extra
Used in "final destination*

Accept


MobileServiceClient Signature changed

I want to use the Azure Mobile Client,
I updated to this Version:
package id="Microsoft.Azure.Mobile.Client" version="2.0.1" targetFramework="win81"
but the Signature of following method has changed, so that I cannot set the "application key":
public static MobileServiceClient MobileService = new MobileServiceClient(
"https://nameofapp.azure-mobile.net/", "xxxxxxxxxxxxxxxxxxxxxxxxxxxxx" );

What to do here?
When is a UWP - Version available?

There is no longer a App Key built into the Azure Mobile Apps version, so it is not necessary to provide one.  @lindydonna , @adrianhall can one of you provide UWP information?

Also the new .Net client now lives off of https://github.com/Azure/azure-mobile-apps-net-client for future issues/questions.

You might want to take a look at this document for more info on upgrading to the latest client and server:
https://azure.microsoft.com/en-us/documentation/articles/app-service-mobile-net-upgrading-from-mobile-services/
Have you run into problems using the 2.0.1 package in a UWP app? We are still in the process of doing full UWP testing and this is why we don't have a quickstart and full documentation for it yet, but we're not aware of any major blocking issues that should impact UWP scenarios.
If you do run into any UWP specific issues/questions for the Microsoft.Azure.Mobile.Client package, please file them in the new repository that Phillip linked to above.

Thank you for your help. I'm using at the moment:
"SQLite.Net-PCL": "3.1.1",
"SQLite.Net.Async-PCL": "3.1.1",
"WindowsAzure.Messaging.managed": "<IP_ADDRESS>",
"WindowsAzure.MobileServices": "2.0.0-beta-3",
"WindowsAzure.MobileServices.SQLiteStore": "2.0.0-beta-2"
It works, but should I change any Package to stay up to date?

You want to flip your client packages to the renamed "Microsoft.Azure.Mobile.Client : 2.0.1 and Microsoft.Azure.Mobile.Client.SQLiteStore : 2.0.1

ok, thank you very much.


'Operands shape mismatch' when num_hidden>1

Hi,
Running the following example regression code with num_hidden>1 generates an 'Operands shape mismatch' error (it works with the deafult num_hidden=1). The problem is the same as the following issue: https://github.com/dmlc/mxnet/issues/949:
`data(BostonHousing, package="mlbench")
train.ind = seq(1, 506, 3)
train.x = data.matrix(BostonHousing[train.ind, -14])
train.y = BostonHousing[train.ind, 14]
test.x = data.matrix(BostonHousing[-train.ind, -14])
test.y = BostonHousing[-train.ind, 14]
data <- mx.symbol.Variable("data")
fc1 <- mx.symbol.FullyConnected(data, num_hidden=2)
lro <- mx.symbol.LinearRegressionOutput(fc1)
mx.set.seed(0)
model <- mx.model.FeedForward.create(lro, X=train.x, y=as.array(train.y),
ctx=mx.cpu(), num.round=50, array.batch.size=20,
learning.rate=2e-6, momentum=0.9, eval.metric=mx.metric.rmse)`
The error is:

Auto detect layout of input matrix, use rowmajor..
Start training with 1 devices
[02:07:10] C:\Users\g\Downloads\mxnet-20160229\mxnet-20160229\dmlc-core\include\dmlc/logging.h:245: [02:07:10] C:\Users\g\Downloads\mxnet-20160229\mxnet-20160229\src\ndarray\ndarray.cc:227: Check failed: from.shape() == to->shape() operands shape mismatch
Error: [02:07:10] C:\Users\g\Downloads\mxnet-20160229\mxnet-20160229\src\ndarray\ndarray.cc:227: Check failed: from.shape() == to->shape() operands shape mismatch

The traceback() is:

6: stop(list(message = "[02:07:10] C:\Users\g\Downloads\mxnet-20160229\mxnet-20160229\src\ndarray\ndarray.cc:227: Check failed: from.shape() == to->shape() operands shape mismatch",
call = NULL, cppstack = NULL))
5: .External(list(name = "CppMethod__invoke_void", address = <pointer: 0x0000000009d78150>,
dll = list(name = "Rcpp", path = "C:/Users/g/Documents/R/win-library/3.2/Rcpp/libs/x64/Rcpp.dll",
dynamicLookup = TRUE, handle = <pointer: 0x000000006abc0000>,
info = <pointer: 0x0000000003f0ee00>), numParameters = -1L),
<pointer: 0x0000000009d759b0>, <pointer: 0x0000000009d5b800>,
.pointer, ...)
4: exec$update.arg.arrays(arg.arrays, match.name, skip.null)
3: mx.exec.update.arg.arrays(train.execs[[i]], s, match.name = TRUE)
2: mx.model.train(symbol, ctx, input.shape, params$arg.params, params$aux.params,
1, num.round, optimizer = optimizer, train.data = X, eval.data = eval.data,
metric = eval.metric, epoch.end.callback = epoch.end.callback,
batch.end.callback = batch.end.callback, kvstore = kvstore)
1: mx.model.FeedForward.create(lro, X = train.x, y = as.array(train.y),
ctx = mx.cpu(), num.round = 50, array.batch.size = 20, learning.rate = 2e-06,
momentum = 0.9, eval.metric = mx.metric.rmse)

I compiled my own libmxnet.dll without cudnn (because I have an older GPU), but i'm not sure this is relevant, as the default example works both on cpu and gpu. I'm using R 3.2.3 and latest mxnet files. Any ideas what I'm doing wrong please?
Thanks, Gavin.

Problem solved. Basically, it's my lack of understanding of NN structure. For regression, the output layer always needs  num_hidden=1.


Document security reporting process

It would be great to have the readme indicate the proper process for reporting security issues - be it sending an e-mail to<EMAIL_ADDRESS>some other mailing list, or filing an issue with a particular format.

I have created a Security label for filing security issues with Cadence server.  I also update the readme with instructions.  In the meantime please file a github issue and label it accordingly.

Thanks. I don't have an issue to report - sorry if I gave that impression.


Horizontal Scroll-ability to allow long labels

I dont know how to fix long labels being too crowded

Is there already a way to add horizontal scrollability?
I couldn't find an example like this in the demo

@djsegal typically, you could do something like this:

...

However, this doesn't appear to work. It may be because of the use of ember-scrollable.
Would love to hear from the collaborators on this as well.
P.S. may be related to #131

@jamesdixon I'll take a look at this soon as I can. It'll likely be about 2 weeks. How big of an issue is this for you right now?

@taras not a big issue at this point. I know how busy you are, so whenever you're able to look at it, it's very much appreciated!

@jamesdixon I have a client who just pinged me about this. I told them that I'll look at it in 8 days. So I'll be back :)

Until an official solution is provided, something that worked for me was to set each column with a fixed width and then wrapping the table with two divs like this:

...

If you want it to work with the resize option too, you can bind the width with a computed property that recalculates the total width after a resizing.

@HenryVonfire very nice! I'll give this a shot

@HenryVonfire thanks for the solution!
Refinement: use min-width: total_width_columns instead of width and you get:

scrollbar when needed (i.e. when available space is below that defined in the total of the column widths)
maximal use of horizontal space when available (e-l-t expands columns beyond their specified column width, space permitting)

In this way, we don't end up with an awkwardly narrow table in larger screens because we are defensively designing for smaller ones.
CC @jamesdixon

@jamesdixon, so are we making @bargar's solution the official one?
it seems a little too roll-your-owny

@djsegal I'm really sorry for the delay on this. @taras and I have been a bit preoccupied by some other high priority tasks. We hope to have a solution to this sometime in the near future.

@djsegal #222 solves this use-case. That PR is a rudimentary solution that is largely a play on what @bargar suggested. I dont mind merging it in as a temporary solution but I want to do so with confidence. Feel free to test it out and let me know.

Resolved in #222.


Peformance improvements

The failing test is already failing on master, starting with commit ed4da692a283f5b84eaf2a3803d31da68d5daced.  @aschlick Do you mind taking a look and fixing that test?


Fix broken links and add segue

Description

Fix broken links in prepare_site_init.md for skipping the LDAP section.
Add a segue in the CSM service install page to route users to the validation page.
Checklist Before Merging

[ ] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[ ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[ ] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport release/1.3

/backport release/1.2


Improve state transition between step 1 and 2

I actually meant fixing the transition performance issues.
I fixed them now via https://github.com/nimiq/gift-cards/commit/c6d731a14a02d6fb1a5982243d327f6fa38f082b


selectMenu a11y issues

Environment
Operating System: Windows_NT
Node Version: v18.18.0
Nuxt Version: 3.10.1
CLI Version: 3.10.0
Nitro Version: 2.8.1
Package Manager<EMAIL_ADDRESS>Builder: -
User Config: -
Runtime Modules: -
Build Modules: -
Version
2.13.0
Reproduction
.
Description

When the options panel opens you cannot navigate using the keyboard keys (when searchable=false)
if you select an option , the panel closes but the focus does not return to the input. (when multiple=false)
When I press the space I must select/deselect an option and not write "" in the search (When multiple=true)

guide:
https://primevue.org/multiselect/#accessibility
Additional context
No response
Logs
No response

Did you encounter this while using the SelectMenu or on ui.nuxt.com?

In ui.nuxt.com

This should be fixed already by https://github.com/nuxt/ui/commit/d0f4530e8572a08d544041dec1f24a51bbc3b1e8, this was a regression of #1199.


Pass missing isVisibleListing variable to mutation

The simpleProductUpdate mutation doesn't pass the new value of isVisibleListing to backend

Task linked: SALEOR-1090 Newly created products are not visible in search in the storefront

It's ok, but please also update typescript types by running npm run build-types

@orzechdev  Done. Thanks!


autocomplete for plotly.graph_objs not working

Pylance seems to not understand the "lazy import" method used when:
import plotly.graph_objs as go
when then refering to go.Scatter(...) , resulting in an "Scatter" is not a known member of modulePylance (reportGeneralTypeIssues)
now I understand that I can set Pylance to not show these errors, but it would be great if I could still use the docs-strings and other autocomplete comforts.
Is there a workaround for this?

@timovp, thanks for the report.
Clarification question: what does "lazy import" mean in this case? Is this an internal detail of how plotly does imports?

Sorry, wasn't that clear indeed.
import plotly.graph_objs as go refers to: plotly graph_objs on github
which invokes just from plotly.graph_objs import *. The functions needed such as Scatter are then found inside subfolder from there. Maybe nested is a better term than lazy.
Oh I see I made a typo, edited that above.

It appears that Plotly makes use of their own import procedure using the relative_import.  I'm not sure whether that is something Pylance could/should ever be able to resolve?
Over on the Plotly side, there is an open issue about possibly adding typing annotations (plotly/plotly.py#1103), so perhaps this might be fixed by a Plotly contributor?

I don't think we'd ever want to attempt to support these sorts of import hacks; they're not specified in any PEP and there are a few projects which do different variants of this (like tensorflow, who has "fixed" this already in latest versions).
If plotly needs to use these sorts of import tricks, then that would be a motivator for having type stubs which write things the "normal" way.

What is the recommended way of dealing with this issue in pylance? (i.e. a library which has incorrect type annotations).
Currently I need to scatter # type: ignore all over the code which is not ideal...

@anton164, I presume that you have enabled type checking within pylance? This issue was initially about autocomplete, not about type checking. The two are related (since autocomplete functionality leverages type information), but type checking requires more accurate and complete type information.
If you enable type checking, then I recommend disabling the useLibraryCodeForTypes option. When this feature is enabled, pylance attempts to infer types from the library code. These inferred types are good enough for autocomplete but are often not good enough for type checking. That explains why you're seeing all of these false positive errors. If you disable this feature, pylance will assume that all symbols imported from a library are "unknown" if it does not include type annotations.
It looks like the maintainers of plotly are open to adding type information. If you're so inclined, you could contribute PRs to that project to add type information (in the form of inlined types or stub files). That would benefit everyone in the community who use plotly.


chore: update docs Attach,Backdrop,BankCard,BaseModal,CardImage

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
8260

Relevant Lines:
9212

💛  - Coveralls


Update postmen test collection

What
Please include a summary of the changes and the related issue
Why
Please include details of the reasoning for these changes
Type of change
Please delete options that are not relevant.

[x] Postman collection update

Checklist:

[x] I have performed a self-review of my code
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have updated the Changelog with details of my change in the UNRELEASED section if this change will affect end users
[x] A corresponding change has been made to the Mapping Documentation repository

Looks good. However, should we call the collection Medicus Test Collection PSS Adaptor? I think we should remove Medicus from the title, as another NME might want to use it.


minor: Improve resilience of match checking

In bug condition the match checking strives to recover giving false no-error diagnostic.
Suggested in https://github.com/rust-analyzer/rust-analyzer/pull/9105#discussion_r644656085

bors r+


In Demo app, MapBox Streets has disappeared

When you start up demo app, there is no map and Mapbox Streets is the default map layer. Mapbox Satellite still works so I don't think its a subscription issue.

To confirm fix, visit:
https://ols-demo.apps.gov.bc.ca/index.html
and you should see the mapbox street layer:

I believe this is done and should be closed now.

Verified in test


Synonyms file

How do I specify the location of the synonyms file for my index definition using esrally? My index is failing to be created because elastic can't find the synonyms file.

Hi @imranazad, we reserve Github for bug reports and feature requests only. Please ask questions like these in the Rally forum instead. Thank you!


[ChartJs] Chart y scale sometimes hidden

https://maestrano.atlassian.net/browse/IMPAC-121

fixed: https://github.com/nnnick/Chart.js/issues/1705


Add copyFormatting keybinding arg and array support

Summary of the Pull Request
Adds array support for the existing copyFormatting global setting. This allows users to define which formats they would specifically like to be copied. A boolean value is still accepted and is translated to the following:

false --> ["plain"]
true --> ["plain", "html", "rtf"]

This also adds copyFormatting as a keybinding arg for copy. As with the global setting, a boolean value and array value is accepted.
References
#5212 - Spec for formatted copying
#2690 - disable html copy
PR Checklist

[X] Closes #4191
[X] Closes #5262
[ ] Tests added/passed
[X] Requires documentation to be updated

Detailed Description of the Pull Request / Additional comments
CopyFormat is a winrt enum where each accepted format is a flag. Currently accepted formats include plain, html, and rtf. A boolean value is accepted and converted. true is a conjunction of all the formats. false only includes plain.
For the global setting, null is not accepted. We already have a default value from before so no worries there.
For the keybinding arg, null (the default value) means that we just do what the global arg says to do. Overall, the copyFormatting keybinding arg is an override of the global setting when using that keybinding. We represent inheritance of the global setting as -1.
Validation Steps Performed
Validated behavior with every combination of values below:

copyFormatting global: { true, false, ["plain"], ["plain", "html"] }
copyFormatting copy arg: { null, true, false, ["plain"], ["plain", "html"]}

shh bot, chill

Urgh. Thanks. I'll get to this one today.

@DHowett Latest commit tries to use the new Json Utils but I had some trouble. No longer builds. Could I get some help?

@DHowett (for when you get back)
This works now! We just need to do that weird merge/new PR with JsonUtilsNew. Thanks!

/azp run


snake_case functions

Hi. Nice plugin! Just in case, you might want to reconsider global helper functions naming style to fit Laravel's naming conventions: my_function()

@LufoX11 Thank you very much for your opinion. 🙏
and i will take this into consideration in the next development 😍


feat: cherry-pick changes to the user pkg

Codecov Report

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

@@           Coverage Diff           @@
##             v1.x    #2362   +/-   ##
=======================================
  Coverage        ?   20.55%
=======================================
  Files           ?      138
  Lines           ?    15736
  Branches        ?        0
=======================================
  Hits            ?     3235
  Misses          ?    12194
  Partials        ?      307


How are these docs built?

Sorry if this is the wrong place to ask this, but I really like how these docs look and I was curious what SSG and theme you were using. I'm guessing the repo here is just the "content" directory of a Hugo site but I'm not totally sure, and I'd love to know what theme you're all using if it's one that's public/open source.
Just a curiosity, thanks in advance!

Hey @wmehilos , you're right about hugo! We've got a modified fork of docsy you can take a look at https://github.com/veertuinc/docsy
Let me know if you have any questions


[<IP_ADDRESS>.1] Validate DTs, DSs & ORs. - Nadhirah

Issue Details
Description
Validate this Subset of Non-Functional Requirements (NFRs), which consists of Data Transfer Requirements (DTs), Data Storage Requirements (DSs) and Other Requirements (ORs), to make sure our NFRs, that we created on a previous issue, are good based on the 7 aspects.
Success or Completion Criteria
A list of valid DTs, DSs and ORs in our Report.
Allocated Time
4 days
Start Date: 08/09/23, 0900
End Date: 13/09/23, 2359

Checked through existing NFRs to ensure it follows the 7 aspects.

Issue Comments
Description
Validated existing NFRs to ensure they follow the 7 criteria of a good requirement.
Changes to success or completion criteria? y/n (if y, describe)
n
Start Date: 08/09/23, 0900
End Date: 13/09/23, 2359


Experimental way to show you may rank higher

Since Google says "[....]in cases where there are many pages that may be similar in relevance, page experience can be much more important for visibility in Search.", I wonder if it would be a good idea to show that you may rank higher if the top 1 or 2 positions have bad CWV metrics, and as top 3 with also bad CWV metrics and a possibility to improve, with something like this:

Closing as Google updated the page and remove the mentioned section.


RPCS3 Achievements Causing Stutters

Enabling RPCS3 Achievements and launching a rpcs3 game causes the game to stutter. Disabling the setting stops the stuttering. In-game fps is about 25% of expected with jumpy, jolty audio.

Misattribution of error, not Success Story's fault. Apologies.


cannot find symbol class ViewHolder

cannot find symbol class ViewHolder

will you post your adapter class, please?


Fix Cuda configuration on Windows

When creating a genrule for a set of files, we should use copy instead of
symlink on Windows.
This addresses an error reported at stackoverflow:
https://stackoverflow.com/questions/44038367/genrules-without-outputs-dont-make-sense-bazel-windows-10-build
Applied the same approach to python_config.bzl to simplify the code.

Jenkins test this please.


Create Mechanism to Display and Set Execution Platform and Version through UI

[ ] set execution platform and version are displayed in status bar

[ ] both execution platform and version must be set (execution platform without version not allowed)
[ ] execution platform and version are retrieved from file

[ ] execution platform and version can be set
[ ] older files without execution platform and version are handled (either set automatically or offer setting manually)

Example

Child of https://github.com/camunda/camunda-modeler/issues/2323

User Test with Curran (13.07.2021)

positive feedback in general
Question: Are the errors the way they are displayed (cf. Zeplin) the right abstraction layer? Will users understand them? The idea of displaying linting errors in the diagram/form later would make them easier to understand and to correlate
Why not show early indication of whether choosing an execution platform would lead to compatibility issues?

Sketch

Design issue: https://github.com/camunda/camunda-modeler-design/issues/26

Closed through https://github.com/camunda/camunda-modeler/pull/2381.


Specify valid engine ranges

https://app.clubhouse.io/augur/story/16030/specify-valid-engine

Coverage remained the same at 63.81% when pulling b212c91c16ef621d64ff3b958d665440fe51f64d on justinbarry/ch16030/specify-valid-engine into d0d4f18bc4d368018b8eeeefbc7ec23130983e14 on master.


HUE-9650-2 [Django Upgrade] updating the Django Babel version for py2

The new version of Django Babel is added in the ext-py file (the older one is not removed yet).

The new version of Django Babel is added in the ext-py file (the older one is not removed yet).

Rebuilding image https://hub.docker.com/repository/docker/gethue/hue/builds used by CI

Rebuilding image https://hub.docker.com/repository/docker/gethue/hue/builds used by CI


Update to 1.20.1?

Hiii, i love this mod so much (the model is sooo pretty) and was hoping that it may be updated to 1.20.1!

Done. Glad you like it! I'm going to add more things to it for Halloween this year, so stay tuned for that!

Yay! Thank you very much!! Im excited to see what you will add o.0


feat(web-ui): rework ui

WARING it's not finish I open pre to show how it's look now.
But it's need a lot of cleaning. Also I'm not expert in svelte so if I'm missing any practical bones, let me know.

Before this change:
vite v4.4.4 building for production...
✓ 41 modules transformed.
dist/index.html                  2.06 kB │ gzip: 0.78 kB
dist/assets/index-4a129365.css   0.64 kB │ gzip: 0.27 kB
dist/assets/index-b890d69c.js   21.32 kB │ gzip: 8.41 kB
✓ built in 497ms

After this change:
vite v4.4.4 building for production...
✓ 48 modules transformed.
dist/index.html                  0.75 kB │ gzip:  0.44 kB
dist/assets/index-9d6f33c7.css  15.12 kB │ gzip:  3.02 kB
dist/assets/index-3d5ac82e.js   29.36 kB │ gzip: 10.66 kB
✓ built in 1.02s

It's a significant increase, not sure it's worth it.

It's a significant increase, not sure it's worth it.

I couldn't agree more. You can do it using classic CSS. But increase js is about making reusable components.

and what do you thinks about design ?

and what do you thinks about design ?

I don't really have any opinion, I'm happy to not to design decisions because I feel out of my depth.

But increase js is about making reusable components.

It seems to me that we are increasing complexity for very little benefit, Caritat UI does not aim to become more than it currently is, I kinda doubt we would ever benefit from the reusability.
My preference would be to keep it as minimal a runtime as possible, so it's easier to maintain and more accessible to folks with bad internet connection.

I will try to simplify such à possible


Animate transposing a matrix

Came as a suggestion of @xiaodaigh during the stream today.

Shouldn't it be generalized to "Animate Latex elements"

We are making tiny steps 😂 but yeah in general the most general form of it is preferred.

One idea: https://upload.wikimedia.org/wikipedia/commons/e/e4/Matrix_transpose.gif

Current state using morphing:

So can we give it a bit of motion blur? xD
or maybe maybe increase the opacity of digits as we move towards the end of the animation

Not yet but if you have time today we can discuss this during a stream :joy:

The first version is merged so I'll close this. There are still quite a lot of things to do though.

is there example code on how to achieve this?

A tutorial is on its way @xiaodaigh !


Basic Usability

[x] Create resource doesn't send you to concept page
-[x] Flashcard doesn't show difficulty (or collapse this into: easy, hard (repeat), didn't get it (repeat quick)).
[ ] Show number of problems attempted during walk, time spent on the walk as a whole, time spent on current problem, and estimated time left in the walk. Number of difficult and number of fail...
[ ] Edit Field modal, create field
[x] Edit problem modal (for problem title)
[x] New problem not a modal, submiting problem brings you back to start of new problem creation
[x] Create quick problems
[x] Walk concept path on concept page
[ ] format concept: add day, week, month badges to problems pill.
[x] Path does not load problem names within concept
[x] fix path bug (can't create new path)
[ ] Path displays concepts as trees with leaves as problems
[x] flash card question answer side-by-side
[ ] flag problem for editing in path flashCard
[ ] Problem-leaves are colored according to last answer, and grayed out until answered correctly in path, concept tree expanded when problem presented
[ ] Nursery formatted to see problems that are week, day, month old - number incorrect.
[ ] Nursery resource, chapter out of uib-tab, concepts created in resource have field attached?
[ ] edit concept mastery
[x] clicking on resources tab brings you out of chapter mode
[x] on page refresh, it sets User.loggedIn from $cookies
[ ] refresh resources table on editing of chapter modal
[x] close chapter modal when it redirects you to concept
[x] When you create a concept in the chapter modal, it should attach the field to the concept
[x] Image upload to s3 should name image by user_id
[x] Add/edit connection modal
[ ] display connections page
[ ] display connections on target?
[ ] multicreate problem modal should send concept_set id, not list
[ ] e2e and unit
[ ] rename problems (need a save button on edit problem model)
[x] bug with in-order problem path
[ ] typeaheads for grove, seed catalogue, concept and chapter on the problem model with tagging ui that lets you delete tags, but persists them from one to the next. Present the chapter typeaheads like "chaptername (resource name)".

when taking a walk, the default should remove a problem when it is correctly guessed. You have the option to do one repeat after correct. You also have the option to only show questions that haven't been answered correctly in some amount of time...

when taking a walk, the default should remove a problem when it is correctly guessed. You have the option to do one repeat after correct. You also have the option to only show questions that haven't been answered correctly in some amount of time...

[ ] - A seed catalogue has various concepts associated with it, and when you create a new problem, it get's tagged with all those concepts, unless you select out. As you create problems, the problem modal persists what you last filtered out, in terms of concepts, but you can refresh full, add new, or refresh individual tags.


  A pr contains all changes in a folder.

This change is

docs/api-reference/v1.9/index.html doesn't exist any more. I deleted it earlier today.

The current ref doc is at /docs/reference/generated/kubernetes-api/v1.9/index.html. And it is an automatically generated doc, so we don't edit it directly. Instead, we edit the source, and then regenerate the doc. The source for the text is in kubernetes/kubernetes. The source for the examples is at kubernetes-incubator/reference-docs/tree/master/gen-apidocs/generators/examples. The process is documented here.

@steveperry-53  Thanks for your reply. I will close this pr.


ErrorException with Laravel 5.6.24

Laravel version : 5.6.24
Postgis version : ^3.3
Can't use package because of this error :
ErrorException: Declaration of Bosnadev\Database\Schema\Blueprint::point($column) should be compatible with Illuminate\Database\Schema\Blueprint::point($column, $srid = NULL)

Hi, thanks for reporting the issue.
I can see it is related to the dependecy package for which you have already opened a pull request to fix: https://github.com/bosnadev/database/pull/34
We will see if the maintainer merges it, otherwise I may need to remove that package as a dependency and roll it myself.

Thank you :)

@njbarrett The PR was merged. Would you tag the library update? :smile:


mdc-chip: events not triggered/handled (0.18.1)

What vue-mdc-adapter Version are you using?

Please be specific, e.g. 0.18.1

What browser(s) is this bug affecting?

All

What OS are you using?

OSX

What are the steps to reproduce the bug?

Use mdc-chip / mdc-chipset and add an event listener:
<mdc-chip-set> <mdc-chip trailingIcon="close"  @MDCChip:trailingIconInteraction='removeChip()'>Chip</mdc-chip> </mdc-chip-set>

What is the expected behavior?

Event MDCChip:trailingIconInteraction is fired and removeChip() is called

What is the actual behavior?

I think MDCChip:trailingIconInteraction is fired but it's not handled (or I'm doing something wrong)

Any other information you believe would be useful?

It worked in 0.17.x

Have you tried to add the native modifier? Worked for me, example from my code:
@MDCChip:removal.native="onTagRemoved"
Without the native modifier my callback was not called.


make the banner link point to the homepage

in the technical pages, the main banner link points to the page itself, which is probably not the intended behavior. Here’s a quick fix.
@autra r?

@autra PR updated, thanks for the quick review :)

LGTM, thanks!


A* Pathfinding Algorithm with GUI

Is your feature request related to a problem? Please describe.
It is a feature for additional enhancement
Describe the solution you'd like
This is a feature that helps to find the shortest distance between two nodes. It uses pygame, and has keys functionalities that even a layman can use.
Describe alternatives you've considered
Alternatives include creating the same without a GUI

Well, no need of PyGame and GUI, just add the A* algorithm.
@proghead00 please get started.

Well, no need of PyGame and GUI, just add the A* algorithm.
@proghead00 please get started.


Add codeql workflow

Description

Add the codeql workflow, this is mostly auto-generated from the github interface.
Did you:

[x] Join the Python Discord Community?
[x] Read all the comments in this template?
[x] Ensure there is an issue open, or link relevant discord discussions?
[x] Read and agree to the contributing guidelines?

Could you give context for others why we want to implement this?

I'm not even proposing me merge this at all, currently just test running the workflow to see if it's useful. Probably should have said that in the description.

We have decided the extra 4-5 minutes of CI wait time isn't worth the minimal benefits this brings


Feature Request: Pipe text into ttyper

Would be nice if you could pipe text directly into ttyper, as opposed to having to make it a file first, could allow for some cool mixups, like requesting a random quote, and then typing that out through ttyper.
Example:
curl -sSL https://api.quotable.io/random | jq -r .content | ttyper

Side note, currently text doesn't seem to wrap to term size properly, that should be fixed before this would be added ofc

Is this available?

This is already supported. You just need to set the input file to /dev/stdin:
curl -sSL https://api.quotable.io/random | jq -r .content | ttyper /dev/stdin

Or, alternatively, you can use process substitution:
ttyper <(curl -sSL https://api.quotable.io/random | jq -r .content)

The reason the text isn't wrapping correctly is because ttyper splits files into "words" at newlines, not spaces and other whitespace. This is for a few reasons:

It allows words to contain spaces.
Splitting input into lines is standard for CLI tools, so it improves compatability.
Lexical analysis is language specific. English words break at lots of characters like spaces, em-dashes, and en-dashes, whereas some languages (like Japanese) don't have any seperators between words besides punctuation.

As an example of the second point, you can make the text wrap correctly (most of the time) by also piping through tr to replace spaces with newlines:
ttyper <(curl -sSL https://api.quotable.io/random | jq -r .content | tr " " "\n")

/dev/stdin is not a thing on Windows, this is not a natively supported feature, that is an OS feature, not a crossplatform feature