Loading UTF8 KV file with Japanese (multi-byte?) text crashes Kivy under Windows

Versions

Python: 3.4.4
OS: Windows 10 Home
Kivy: 1.9.1
Kivy installation method: followed https://kivy.org/docs/installation/installation-windows.html#installation

Description
Loading a UTF8 KV file with Japanese (and perhaps other multi-byte?) text crashes Kivy under Windows. The example below uses the text field, but even a comment in Japanese will cause the crash. Changing the KV file encoding to Shift-JIS (approximately cp932) fixes the issue.
Full details can be found in this user support group post: https://groups.google.com/forum/#!topic/kivy-users/dlR0f6tmANw
The essence is that, this fact combined with the fact that Linux-type systems support UTF8 KV files and NOT Shift-JIS KV files makes cross-platform development very difficult.
Issue #4003 mentions a display glitch, but I believe a crash is more serious and may merit being dealt with.
Code and Logs
The following will reproduce the issue; the commented out alternatives are to show the options that work properly (without crashing).
utftest.py
# -*- coding: utf-8 -*-

# test using kivy KV File in UTF8 with Japanese text

from kivy.app import App
from kivy.uix.boxlayout import BoxLayout
from kivy.lang import Builder

# # This will load properly (ignoring the fact that a font set needs to be loaded separately for proper display)
# Builder.load_string('''
# <UtfTestScreen>
#     Label:
#         # font_name: '../SourceHanSerifJP/SourceHanSerifJP-Medium.otf'
#         id: inst_txt
#         text: u'テストテキスト'
#
# ''')

class UtfTestScreen(BoxLayout):
    def __init__(self, **kwargs):
        super(UtfTestScreen, self).__init__(**kwargs)

        # # Test using Japanese text from py file to kv specified parts
        # self.ids.inst_txt.text = 'テストテキスト'

class UtfTestApp(App):
    def build(self):
        return UtfTestScreen()

if __name__ == "__main__":
    UtfTestApp().run()

utftest.kv (saved in UTF8 encoding)
<UtfTestScreen>
    Label:
        # font_name: '../SourceHanSerifJP/SourceHanSerifJP-Medium.otf'
        id: inst_txt
        text: u'テストテキスト'

caused an error as below:
 Traceback (most recent call last):
   File "utftest.py", line 34, in <module>
     UtfTestApp().run()
   File "C:\Python34\lib\site-packages\kivy\app.py", line 801, in run
     self.load_kv(filename=self.kv_file)
   File "C:\Python34\lib\site-packages\kivy\app.py", line 598, in load_kv
     root = Builder.load_file(rfilename)
   File "C:\Python34\lib\site-packages\kivy\lang.py", line 1831, in load_file
     data = fd.read()
 UnicodeDecodeError: 'cp932' codec can't decode byte 0x86 in position 141: illegal multibyte sequence

Process finished with exit code 1

The fix is easy enough, read in as bytes and decode to utf8 after reading. The question is whether we can always assume that the file is in utf8? Or maybe we need to have a way to specify the encoding like a shabang. Then by default it'll use the platform encoding, but it can be overwritten.

There is some similar discussion in #4003 , and I agree with VinGarcia's comment on 11 Apr 2016, to quote:

The best would probably be to mimic python behavior:

Expect programmer to specify the encoding on the first line.
Or else use python default encoding (on python3 it would always be 'utf8' I think).

If the default were to be made UTF8 on both Windows and Linux, or else the encoding be made specifiable as above, it'd really help me on this issue...

From the user support group above, @Geekademy shared an idea to use
with open(filename, encoding='utf8') as f:
    Builder.load_string(f.read())

in place of the usual Builder.load_file() , which seems to work properly for me.
Although this is great as a temporary workaround, official Kivy Support of the above idea would be even greater...

I have uploaded the minimal case, including points from this issue as comments:

https://github.com/yk0242/incubation/blob/master/incubation-python/kv-utf8/utftest.py
https://github.com/yk0242/incubation/blob/master/incubation-python/kv-utf8/utftestkv.kv

Please feel free to use these for purposes of investigating this issue.

I found the same issue on answering an S.O. question yesterday.
SO, this is actually a bug on Kivy, since  any app that is build even on systems that use utf-8 by default will fail to properly load non-ascii characters from the .KV on the "automatic" loading way, when they are run under windows.
To put it in other words: for any i18ned KV file, the app will break either when run on Windows, unless the .KV file uses the native windows encoding on the developer's computer. (in which case it won't work elsewhere)
IMHO, the fix would be forcing kivy to load .kv files using "utf-8" encoding and document that.
(I have no windows install, but the OP just accepted my answer, indicating this was indeed the problem -
https://pt.stackoverflow.com/questions/281641/como-configurar-o-português-no-kivy/281664#281664 )

This just came up again in #5738. The solution is to use Builder.load_string(...) as mentioned above.
It's not actually a bug in Kivy.
If anything, it's an unfortunate design decision in Python and a missing feature in Kivy.
The real answer here is: it's complicated. If utf-8 were forced, it would simply break on files with bytes that aren't valid UTF-8. Compare UTF-8 and Windows-1252 codepages. There are combinations of bytes in either that break decoding of the other.

I understand that this is not a bug in Kivy per se, but rather a missing feature (that would be greatly welcomed by the likes of myself).
Having said that, I think the commit by jsbueno (that I just now noticed...) would circumvent the problem well, assuming that the 'combinations of bytes in either that break decoding of the other' mentioned by dolang will properly raise a UnicodeDecodeError when met. I'm not sure if this is indeed the case, and if this is not so (i.e., if the open function opens the file without raising said error and opens it in a broken manner), the said commit won't work in some environments.
Otherwise, I believe that a fix that tries both encodings by default (be it UTF-8 -> native as in commit, or explicitly UTF-8 -> CP932 -> native, etc.) would be extremely friendly for non-ASCII developing environments...

I've had a short look at it, there might be some issues with that pull request.
Just fyi, there's been some talk about how to fix the general problem. Ultimately the solution might be a bit different. And no promises yet, either.

Any valid fix would be greatly appreciated. No promises, I understand, but still keeping my hopes up...

This issue can be solved by setting PYTHONUTF8 as 1.
set PYTHONUTF=1
python
or -Xutf8 option can be simply add as below;
python -Xutf8
The option was added to python 3.7. Details are in https://www.python.org/dev/peps/pep-0540/


16 create route with spotify data statistics

closes #16

fix jwt guard from libs

create unit tests for statistics service and resolver

fix jest error with enums


Branch1

@savin02 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Unable to install after following all the instructions on MacOS Big Sur.

OS: Big Sur 11.2.2
Node: v14.16.0
npm: 6.14.11
Reference https://github.com/nodejs/help/issues/2826 implies the fix needs to be applied in this package.
Logs:
../src/NodePoppler.cc:535:22: error: call to 'Data' is ambiguous
  char* bufferData = node::Buffer::Data(bufferObj);
                     ^~~~~~~~~~~~~~~~~~
/Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/node_buffer.h:38:19: note: candidate function
NODE_EXTERN char* Data(v8::Local<v8::Value> val);
                  ^
/Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/node_buffer.h:39:19: note: candidate function
NODE_EXTERN char* Data(v8::Local<v8::Object> val);
                  ^
../src/NodePoppler.cc:536:25: error: call to 'Length' is ambiguous
  size_t bufferLength = node::Buffer::Length(bufferObj);
                        ^~~~~~~~~~~~~~~~~~~~
/Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/node_buffer.h:40:20: note: candidate function
NODE_EXTERN size_t Length(v8::Local<v8::Value> val);
                   ^
/Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/node_buffer.h:41:20: note: candidate function
NODE_EXTERN size_t Length(v8::Local<v8::Object> val);
                   ^
In file included from ../src/NodePoppler.cc:1:
In file included from ../../nan/nan.h:56:
In file included from /Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/node.h:67:
/Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/v8.h:201:5: error: static_assert failed due to requirement
      'std::is_base_of<Object, v8::Object>::value' "type check"
    static_assert(std::is_base_of<T, S>::value, "type check");
    ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../src/NodePoppler.cc:232:31: note: in instantiation of function template specialization 'v8::Local<Object>::Local<v8::Object>' requested
      here
    Local<Object> bufferObj = args[0]->ToObject(Nan::GetCurrentContext()).ToLocalChecked();
                              ^
In file included from ../src/NodePoppler.cc:1:
In file included from ../../nan/nan.h:56:
In file included from /Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/node.h:67:
In file included from /Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/v8.h:22:
In file included from /Library/Developer/CommandLineTools/usr/bin/../include/c++/v1/memory:652:
/Library/Developer/CommandLineTools/usr/bin/../include/c++/v1/type_traits:1425:59: error: incomplete type 'Object' used in type trait
      expression
    : public integral_constant<bool, __is_base_of(_Bp, _Dp)> {};
                                                          ^
/Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/v8.h:201:24: note: in instantiation of template class
      'std::__1::is_base_of<v8::Object, Object>' requested here
    static_assert(std::is_base_of<T, S>::value, "type check");
                       ^
../src/NodePoppler.cc:255:39: note: in instantiation of function template specialization 'v8::Local<v8::Object>::Local<Object>' requested
      here
    Local<Value> saveParam = Nan::Get(parameters, saveStr).ToLocalChecked();
                                      ^
/usr/local/include/poppler/qt5/poppler-form.h:46:7: note: forward declaration of 'Object'
class Object;
      ^
In file included from ../src/NodePoppler.cc:1:
In file included from ../../nan/nan.h:56:
In file included from /Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/node.h:67:
/Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/v8.h:201:5: error: static_assert failed due to requirement
      'std::is_base_of<v8::Object, Object>::value' "type check"
    static_assert(std::is_base_of<T, S>::value, "type check");
    ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../src/NodePoppler.cc:255:39: note: in instantiation of function template specialization 'v8::Local<v8::Object>::Local<Object>' requested
      here
    Local<Value> saveParam = Nan::Get(parameters, saveStr).ToLocalChecked();
                                      ^
In file included from ../src/NodePoppler.cc:1:
In file included from ../../nan/nan.h:56:
In file included from /Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/node.h:67:
In file included from /Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/v8.h:22:
In file included from /Library/Developer/CommandLineTools/usr/bin/../include/c++/v1/memory:652:
/Library/Developer/CommandLineTools/usr/bin/../include/c++/v1/type_traits:1425:59: error: incomplete type 'Object' used in type trait
      expression
    : public integral_constant<bool, __is_base_of(_Bp, _Dp)> {};
                                                          ^
/Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/v8.h:201:24: note: in instantiation of template class
      'std::__1::is_base_of<v8::Value, Object>' requested here
    static_assert(std::is_base_of<T, S>::value, "type check");
                       ^
../src/NodePoppler.cc:512:40: note: in instantiation of function template specialization 'v8::Local<v8::Value>::Local<Object>' requested
      here
        Nan::Set(fieldArray, fieldNum, obj);
                                       ^
/usr/local/include/poppler/qt5/poppler-form.h:46:7: note: forward declaration of 'Object'
class Object;
      ^
In file included from ../src/NodePoppler.cc:1:
In file included from ../../nan/nan.h:56:
In file included from /Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/node.h:67:
/Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/v8.h:201:5: error: static_assert failed due to requirement
      'std::is_base_of<v8::Value, Object>::value' "type check"
    static_assert(std::is_base_of<T, S>::value, "type check");
    ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../src/NodePoppler.cc:512:40: note: in instantiation of function template specialization 'v8::Local<v8::Value>::Local<Object>' requested
      here
        Nan::Set(fieldArray, fieldNum, obj);
                                       ^
In file included from ../src/NodePoppler.cc:1:
In file included from ../../nan/nan.h:176:
In file included from ../../nan/nan_callbacks.h:83:
../../nan/nan_callbacks_12_inl.h:28:5: error: assigning to 'v8::Value *' from incompatible type 'Object *'
    TYPE_CHECK(T, S);
    ^~~~~~~~~~~~~~~~
../../nan/nan.h:145:42: note: expanded from macro 'TYPE_CHECK'
      *(static_cast<T *volatile *>(0)) = static_cast<S*>(0);                   \
                                         ^~~~~~~~~~~~~~~~~~
../src/NodePoppler.cc:605:27: note: in instantiation of function template specialization 'Nan::ReturnValue<v8::Value>::Set<Object>'
      requested here
    info.GetReturnValue().Set(returnPdf);
                          ^
In file included from ../src/NodePoppler.cc:1:
In file included from ../../nan/nan.h:56:
In file included from /Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/node.h:67:
/Users/soumya/Library/Caches/node-gyp/14.16.0/include/node/v8.h:11110:3: error: static_assert failed due to requirement
      'std::is_void<v8::Value>::value || std::is_base_of<v8::Value, Object>::value' "type check"
  static_assert(std::is_void<T>::value || std::is_base_of<T, S>::value,
  ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../nan/nan_callbacks_12_inl.h:29:12: note: in instantiation of function template specialization 'v8::ReturnValue<v8::Value>::Set<Object>'
      requested here
    value_.Set(handle);
           ^
../src/NodePoppler.cc:605:27: note: in instantiation of function template specialization 'Nan::ReturnValue<v8::Value>::Set<Object>'
      requested here
    info.GetReturnValue().Set(returnPdf);
                          ^
4 warnings and 13 errors generated.
make: *** [Release/obj.target/pdf_fill_form/src/NodePoppler.o] Error 1

This PR should fix this issue: https://github.com/tpisto/pdf-fill-form/pull/101

New version 5.1.0 has now been released that has the fix by @gedaiu


What is the long term vision for Rational Emacs and how does it distinguish itself from other distributions of Emacs?

@daviwil I'm curious, what is your long term vision for Rational Emacs? After having played with it some, the general structure of it seems pretty similar to Doom. There are a number of modules Rational provides to the user, which the user can choose to include or exclude. The user then crafts his personal config around those modules. If someone were to ask me to characterize what Rational is, I would describe it something like a less opinionated Doom. Is that sort of the long term vision for this project? What ultimately distinguishes this project from something like Doom (or the other distributions out there).

This is probably something we should include on the README, but the core differences between this distro and Doom or Spacemacs are the following:

Don't introduce a new configuration system (new macros, "layers", etc) on top of existing Emacs configuration patterns.  The user's own configuration should be written as if it was done from scratch
Provide pre-configured modules that shorten the time it takes to build a nice, working configuration so that the user can focus on building the other parts they need on their own
Many customizations in modules should be opt-in so that there is no surprising behavior by default
Don't configure everything under the sun or provide configurations for every possible popular package.  A "lightly opinionated, comfortable base configuration" is the goal in my opinion.

In short, this distro is for Emacs users who want a nice baseline for building their own Emacs configuration while writing actual Emacs Lisp code, not some other distro's configuration code. They should also be able to learn configuration tricks and useful settings by reading the source!
I know a lot of this is already possible in other distros, but typically the convenience macros and patterns provided by those distros get prioritized over writing raw Emacs Lisp.  In the end it's more about the intent of using such a distro: do you want "batteries included" without doing much work, or do you really want to build your own configuration over time without having to write everything from scratch at first.
A potential path for a Crafted Emacs user is to eventually abandon Crafted Emacs once their own configuration is complete :)
@jeffbowman should feel free to chime in here with his thoughts too!
Hope that helps!

I am not sure I can add much to the thoughts from @daviwil , but I'll highlight things which I think are generally important to me:

This isn't a turn-key solution. This should help a user build a configuration quickly, but the expectation is their configuration fills in the holes we (maybe deliberately) leave.
Not everyone agrees, so everything should be easily reversible. A module that gets someone 80%-90% of what they want is still good enough as long as they can turn on/off the part that does or doesn't make sense in their config. Some people like the doom modeline, I personally do not, so I made it easy to turn off. Some people prefer shorter answers (y/n instead of yes/no), so I made that easy to reverse.
Speed is important, so using hooks and deferred configuration (ie with-eval-after-load for example) should be used appropriately.
Fewer packages, rather than more, and of the packages we choose, those that are closer to vanilla Emacs as possible - ideally doing more with less.

To support these things, I try to keep these things in mind when I'm working on the project:

Correctness is important. If we want people to be able to learn from what we provide, we need to make an effort to do things as correctly as possible. Using customize-set-variable for defcustom variables, using setq or in some cases setq-default as appropriate, putting the code in the correct place during initialization (early-config.el vs config.el and then possibly within the correct initialization hook as necessary).
Documentation should be thorough, complete, and easy to find.

As for a long-term vision, I'd love to see this project fulfill the goal of the opening line in the readme... a sensible place to start hacking your own Emacs config. I'd consider this project a success if someone could start from scratch and in an hour or less get at least 70% of their configuration completed, have something that works for most of their day-to-day workflow. Over time, I might consider that 70% number a bit too low, but for where we are now, we should be able to hit at least that in a lot of cases.
As a secondary goal, I'd echo what @daviwil  said - this should be a project that you could eventually outgrow as your configuration matures through your own implementation. Hopefully that means you, as the user, have learned enough about Emacs Lisp, Emacs configuration, making decisions on packages you prefer and configuring them to your taste. Eventually, Crafted Emacs becomes more and more transparent to the point you have replaced it with your own. That journey is worth going on!

I wonder if we should digest some of this thinking as an update to the README to flesh out the ideas further.  I think as time goes on, things are becoming clearer!  I've updated the issue to reflect the need to update the README with more details.

Yes and no... There is the concept that a user might craft their own configuration from certain modules provided by Crafted Emacs but not actually use Crafted Emacs itself.  Basically, they would provide their own custom early-init.el and init.el and then use some of the modules from this project.  While we endorse this idea, there hasn't really been any movement on "packaging" any of the modules.  IIRC, @daviwil  made passing reference to the idea in the very early days of (then) Rational (now Crafted) Emacs.  There is another issue (#251) which might take a step in this direction.
One of the difficulties at the moment is the tight coupling between the modules and the initialization (ie early-init.el and init.el).  This is not necessarily a "bad" thing per se, however, decoupling requires configuration of certain path variables which, if the modules were packages, would need further refinement, probably default values and customization options  - not the end of the world, nor impossible to accomplish, but will certainly require some thoughtful development.
For the moment, you can nearly achieve the same effect by following some of the instructions left in #251, specifically this comment.
Hope that helps!

Closing in anticipation of Crafted Emacs v2


React slider not showing last item after slickGoTo

I have implemented react-slick. am facing issue when i place slickGoTo and slidesToScroll. When page load initially am calling slickGoTo to 4th one. once its done, it lose the control of slidesToScroll and last item is not visible.
use this jsfiddle to reproduce your bug:
http://jsfiddle.net/svx2dzwL/

I didn't find any solution. using slickGoto i did small calculation and achieved that.


Worked yesterday, not today.

Please help. It must be missing something simple. I have redownloaded, restarted, and reconfigured and nothing helps. Grocy is working properly but now Shopping list is not.
Error : Failed setup, will retry: Cannot connect to host <IP_ADDRESS>:9192 ssl:default [[SSL] record layer failure (_ssl.c:1000)] but I have deactivated all of the SSL as directed.

As mentioned in the doc:
You SHOULD use https if your Grocy's module is configured to use SSL (even if you don't have any certificate). Use http:// if it's not checked
I can see on your Grocy's screenshot the SSL checkbox is not checked. So try to replace https with http or check the SSL checkbox and let me know if works

Well, it's working again thank you!
I did receive this warning message though. "Detected that custom integration 'shopping_list_with_grocy' calls async_forward_entry_setup for integration, shopping_list_with_grocy with title: ShoppingListWithGrocy and entry_id: 01JABXRBM83PDB1GT44NQYFF55, which is deprecated and will stop working in Home Assistant 2025.6, await async_forward_entry_setups instead at custom_components/shopping_list_with_grocy/init.py, line 70: hass.async_create_task(, please create a bug report at https://github.com/Anrolosia/Shopping-List-with-Grocy/issues"
Anyway, guess I misread the http to https part. I interpreted it as leave as https just turn off the verify function. Have a great day!

Awesome!
Yeah, I really need to push the update for that :) Thanks a lot and have a great one!


ThermostatRunningState support? (CON-939)

I want to indicate when a heater is "active", when the thermostat is actually telling it to heat when it's too cold in the room. Thermostat running state appears to be the appropriate attribute to set, but I see the following in the logs:
I (40592) chip[ZCL]: 0x3c12939cep 1 clus 0x0000_0201 attr 0x0000_0029 not supported
E (40602) esp_matter_attribute: Error updating Endpoint 0x0001's Cluster 0x00000201's Attribute 0x00000029 to matter: 0x86

I'm able to set some other attributes without issues

@apexskier Error 0x86 is UnsupportedAttribute. Did you add the attribute to the thermostat? Since ThermostatRunningState is optional attribute by default it will not be present on device, you have to add it yourself.

Thanks, that worked!
esp_matter::cluster::thermostat::attribute::create_thermostat_running_state(thermostat_cluster, 0);


Indexed component not found when adding component through EntityStore.CreateEntity

This test fails
    [Test]
    public static void Test_Index_ValueStructIndex()
    {
        var store   = new EntityStore();
        store.CreateEntity(new IndexedInt { value = 123 });

        var result = store.GetEntitiesWithComponentValue<IndexedInt, int>(123);
        AreEqual(1, result.Count);
    }

This does not fail
        var store   = new EntityStore();
        var entity1 = store.CreateEntity();
        entity1.AddComponent(new IndexedInt { value = 123 });
        var result = store.GetEntitiesWithComponentValue<IndexedInt, int>(123);
        AreEqual(1, result.Count);

batches fail too
        var store   = new EntityStore();
        var entity1 = store.CreateEntity();

        entity1.Batch().Add(new IndexedInt { value = 123 }).Apply();
        var result = store.GetEntitiesWithComponentValue<IndexedInt, int>(123);
        AreEqual(1, result.Count);

hey @dvdking
somebody tries this crazy stuff 👍🏼.
This is currently a known issue for all new Component Types.
That is the reason why these new types are only available in 3.0.0-preview releases.
Right now: AddComponent<>() and RemoveComponent<>() must be used.
I have to make adjustments in all extension methods in EntityExtensions and EntityStoreExtensions used in your first test.
Same for EntityBatch and CreateEntityBatch methods used in your last test.
Without your post I guess I would have missed those two for the final release.
Also the CommandBuffer needs adjustments.
thx for feedback!

Hi. Any news about supporting Indexed Components using CommandBuffers and EntityBatch ?

Fixed in 3.0.0-preview.14.
Implemented also all other cases adding / removing / changing indexed components.
Those cases are listed in my initial answer.


Initial merging of istiod into mainstream templates

This should run istiod in SDS mode and have all functionality from 'standalone' istiod.
However the injection will still use the old injector, and tests still point to the old port - different PR to take care of this. The goal is to switch most of the testing to the 'SDS by default' path which is the
riskier part of istiod.

@costinm: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_installer
0ae0f4dbdd72c24dc4245a6c6bc4dc2cc28a36c4
link
/test lint_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_installer
0ae0f4dbdd72c24dc4245a6c6bc4dc2cc28a36c4
link
/test lint_installer

build_installer
0ae0f4dbdd72c24dc4245a6c6bc4dc2cc28a36c4
link
/test build_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

build_installer
0ae0f4dbdd72c24dc4245a6c6bc4dc2cc28a36c4
link
/test build_installer

lint_installer
ead5f34dc2fc66e9b6ee5e18e2351fa77bec60a4
link
/test lint_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_installer
ead5f34dc2fc66e9b6ee5e18e2351fa77bec60a4
link
/test lint_installer

build_installer
ead5f34dc2fc66e9b6ee5e18e2351fa77bec60a4
link
/test build_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

build_installer
ead5f34dc2fc66e9b6ee5e18e2351fa77bec60a4
link
/test build_installer

lint_installer
8fb79bbc848fd3c148ca10fd68c5e44ff0857801
link
/test lint_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test lint_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test lint_installer

base_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test base_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test lint_installer

base_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test base_installer

noauth_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test noauth_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test lint_installer

base_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test base_installer

noauth_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test noauth_installer

demo_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test demo_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

base_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test base_installer

noauth_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test noauth_installer

demo_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test demo_installer

lint_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test lint_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

noauth_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test noauth_installer

demo_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test demo_installer

lint_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test lint_installer

base_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test base_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

noauth_installer
1c58a96dd132b11c772a8f9b99d6746d7f6ac135
link
/test noauth_installer

lint_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test lint_installer

base_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test base_installer

demo_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test demo_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test lint_installer

base_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test base_installer

demo_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test demo_installer

noauth_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test noauth_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

base_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test base_installer

demo_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test demo_installer

noauth_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test noauth_installer

lint_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test lint_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

demo_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test demo_installer

noauth_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test noauth_installer

lint_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test lint_installer

base_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test base_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

noauth_installer
453f65760e3c564b9010eb7a99297bf6e2cefad9
link
/test noauth_installer

lint_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test lint_installer

base_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test base_installer

demo_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test demo_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test lint_installer

base_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test base_installer

demo_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test demo_installer

noauth_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test noauth_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

base_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test base_installer

demo_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test demo_installer

noauth_installer
0e3dad26965daf212a21fce82abd2b12fba0f28b
link
/test noauth_installer

lint_installer
7b7b6fccdcfd80b0770378d08aadbcb52cf4c3dc
link
/test lint_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

PTAL - I'll have a follow-up to merge the inject template (and simplify it further), as well as enabling the injection and other things. Trying to keep it in small pieces.


Fix compatibity issue with BetterButtons

Better buttons changes the save action name to action_save, while newsletter expects it to be action_doSave.
This does a simple check to see if action_doSave exists, if it does not, it will use action_save

Fixes #58

I wouldn't fix this this way.
A better way is disable Gridfield Better Buttons via the config:
Newsletter:
  better_buttons_enabled: false

That way, the send-buttons etc. from Newsletter aren't overridden either (because better buttons does that)


No debugger attached when running npm run script with debug button

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:
Version: 1.76.1
Commit: 5e805b79fcb6ba4c2d23712967df89a089da575b
Date: 2023-03-08T16:48:08.231Z
Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 12_6_1) AppleWebKit/537.36 (KHTML, like Gecko) Code/1.76.1 Chrome/102.0.5005.196 Electron/19.1.11 Safari/537.36
OS Version:  macOs Monterey

Steps to Reproduce:
already all extension --disabled

Go to package.json
Click debug button

https://user-images.githubusercontent.com/35516239/224672994-6a281e7d-8d42-42d0-8c7a-075d39f59969.mov
expected

npm run with debugger attached

result

debugger don't attached

Please check that you don't override the NODE_OPTIONS environment variable in e.g. your bashrc/zshrc. You can check the contents by running the "Create JavaScript Debug Terminal" command and running echo $NODE_OPTIONS

Please check that you don't override the NODE_OPTIONS environment variable in e.g. your bashrc/zshrc. You can check the contents by running the "Create JavaScript Debug Terminal" command and running echo $NODE_OPTIONS

Wow you were right on the money, I can finally solve the problem, thanks for the reply.


rails-api 1.0: 1 processed / 4 skipped issue

Hi all
I used rails-api 5.1.2 ,let me show my code
swagger_doc.rb
`Swagger::Docs::Config.base_api_controller = ActionController::API
Swagger::Docs::Config.register_apis({
"1.0" => {
# the extension used for the API
:api_extension_type => :json,
# the output location where your .json files are written to
:api_file_path => "public",
# the URL base path to your API
:base_path => "http://localhost:3000",
# if you want to delete all .json files at each generation
:clean_directory => false,
# Ability to setup base controller for each api version. Api::V1::SomeController for example.
:parent_controller => ApplicationController,
# add custom attributes to api-docs
:attributes => {
:info => {
"title" => "Swagger Sample App",
"description" => "This is a sample description.",
"termsOfServiceUrl" => "http://xxx",
"contact" => "xxx",
"license" => "Apache 2.0",
"licenseUrl" => "http://www.apache.org/licenses/LICENSE-2.0.html"
}
}
}
})
`
UserController.rb
`class Api::V1::UsersController < ApplicationController
include Swagger::Docs::ImpotentMethods
swagger_controller :users, "User Management"
swagger_api :index do
summary "Fetches all User items"
response :unauthorized
response :not_acceptable, "The request you made is not acceptable"
response :requested_range_not_satisfiable
end
swagger_api :show do
summary "Fetches a single User item"
param :path, :id, :integer, :required, "User Id"
response :ok, "Success", :User
response :unauthorized
response :not_acceptable
response :not_found
end
get model list /api/v1/users/index
def index
@user = User.all
render :json => {users: @user} , :status => 200
end
get model by id  /api/v1/users/show?id=1
def show
@user = User.find(params[:id])
render :json => {user: @user}, :status => 200
end
create model
def create
@user = User.new(user_params)
if @user.save
  render '200'
else
  render '400'
end

end
update model by id
def update
end
delete model by id
def delete
end
private
def user_params
params.require(:user).permit(:name,:password,:address)
end
end
`
and the last , run command : SD_LOG_LEVEL=1 rake swagger:docs
then generator a the doc file what has not action structure on apis .
{   "apiVersion": "1.0",   "swaggerVersion": "1.2",   "basePath": "http://localhost:3000",   "apis": [     {       "path": "/api/v1/users.{format}",       "description": "User Management"     }   ],   "authorizations": null,   "info": {     "title": "Swagger Sample App",     "description": "This is a sample description.",     "termsOfServiceUrl": "http://xxx",     "contact": "xxx",     "license": "Apache 2.0",     "licenseUrl": "http://www.apache.org/licenses/LICENSE-2.0.html"   } }

i have this same error


make verify should validate that versioned APIs contain json tags for fields

There is such a validation in kubernetes and it is pretty clear that we need an equivalent one here.

#dibs

0.1.1 candidate


Update to version 4

After update to version 4 and tried to follow the instructions to extend the CustomConfig I'm get this error message in my building process.
client/src/app/config/auth-config.service.ts(3,14): error TS2415: Class 'MyAuthConfig' incorrectly extends base class 'CustomConfig'.
  Types of property 'providers' are incompatible.
    Type '{ google: { clientId: string; }; }' is not assignable to type 'IProviders'.
      Index signature is missing in type '{ google: { clientId: string; }; }'.

This is my auth settings class
import { CustomConfig } from 'ng2-ui-auth';

export class MyAuthConfig extends CustomConfig {
    baseUrl = 'http://localhost:3000';
    defaultHeaders = { 'Content-Type': 'application/json' };
    tokenPrefix = 'locaoffice';
    loginUrl =  '/api/sessions';
    signupUrl = '/api/users';
    providers = {google: {clientId: ''}};
}

I don't know where is the problem.

https://github.com/Microsoft/TypeScript/issues/1887
https://github.com/Microsoft/TypeScript/pull/6118
In short, it's a typescript issue... You'll have to add a : { [provider: string]: IProvider } or : any to the providers property

@ronzeidman Thanks for your reply. Do you know why this is happening? In your sample you don't do explicit convert. In case ng2-ui-auth types is registered in typings this could be solved without use an explicit convert?

Hi i have same issue please help I have tried using  : any to the providers property i am getting this error core.umd.js:3433Error: Uncaught (in promise): Error: Error in ./AuthComponent class AuthComponent_Host - inline template:0:0 caused by: No provider for AuthService!

"No provider for AuthService" means that you didn't provide the Ng2UiAuthModule to your base module. Can you share how you initialize your main module?

@ronzeidman  sorry my bad that issue was from my side .

Updated the readme... I'll try to fix this later... it will probably be a breaking change though (I'll use an interface instead of a class for it...)

@diegodfsd I can't seem to be able to reproduce the type issue, what typescript version are you using?

@ronzeidman I was using version 1.8.10 and now I update to 2.03 but the error continue if I remove the explicit conversion.
client/src/app/app.module.ts(35,39): error TS2345: Argument of type 'typeof MyAuthConfig' is not assignable to parameter of type 'Type<CustomConfig>'.
  Type 'MyAuthConfig' is not assignable to type 'CustomConfig'.
    Types of property 'providers' are incompatible.
      Type '{ google: { clientId: "*************"...' is not assignable to type 'IProviders'.
        Index signature is missing in type '{ google: { clientId: "**********"...'.
client/src/app/config/auth-config.ts(5,14): error TS2415: Class 'MyAuthConfig' incorrectly extends base class 'CustomConfig'.
  Types of property 'providers' are incompatible.
    Type '{ google: { clientId: "**********"...' is not assignable to type 'IProviders'.
      Index signature is missing in type '{ google: { clientId: "************"...'.

What's your tsconfig.json?

My tsconfig.json
{
  "compilerOptions": {
    "rootDir": "client/src/app",
    "target": "es5",
    "module": "commonjs",
    "moduleResolution": "node",
    "sourceMap": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "removeComments": false,
    "noImplicitAny": false
  },
  "exclude": [
    "client/config",
    "node_modules",
    "typings/main",
    "typings/main.d.ts",
    "typings/globals/node",
    "typings/globals/core-js"
  ],
  "compileOnSave": false
}

Can you try with version 4.1.0 typescript 2.0.3 and use @types/ instead of typings?

@types? How do I use it?

Remove typings library and install typings from npm: npm I -D @types/<your lib name> no more configuration should be needed (it's one of the major updates of ts2)

@ronzeidman I tried install as you recommended but something went wrong.
$ npm i -D @types/ng2-ui-auth
npm ERR! Darwin 16.0.0
npm ERR! argv "/Users/diego/.nvm/versions/node/v6.3.1/bin/node" "/Users/diego/.nvm/versions/node/v6.3.1/bin/npm" "i" "-D" "@types/ng2-ui-auth"
npm ERR! node v6.3.1
npm ERR! npm  v3.10.3
npm ERR! code E404

npm ERR! 404 Not found : @types/ng2-ui-auth
npm ERR! 404
npm ERR! 404  '@types/ng2-ui-auth' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! Please include the following file with any support request:
npm ERR!     /npm-debug.log

Do you have any idea about what's happening?

@types like typings is only required for JS libraries. TS libraries have their type definitions built-in

make sense. so do you have an alternative to avoid explicit conversion?

It shouldn't be an issue... I gave you my settings and they don't reproduce the issue

are you sure? because I'm just install your package like I did with othera and only with ng2-ui-auth I got this error. Are you import your package directly from source or from npm install?

npm install. Try to compile the ng2-ui-auth-example

try adding the following to your tsconfig: "suppressImplicitAnyIndexErrors": true

@ronzeidman I tried change my tsconfig.json but when I remove the explicit conversion I get this error messages:
client/src/app/app.module.ts(36,39): error TS2345: Argument of type 'typeof MyAuthConfig' is not assignable to parameter of type 'Type<CustomConfig>'.
  Type 'MyAuthConfig' is not assignable to type 'CustomConfig'.
    Types of property 'providers' are incompatible.
      Type '{ google: { clientId: string; scope: string[]; }; facebook: { clientId: string; scope: string[]; ...' is not assignable to type 'IProviders'.
        Index signature is missing in type '{ google: { clientId: string; scope: string[]; }; facebook: { clientId: string; scope: string[]; ...'.
client/src/app/config/auth-config.ts(5,14): error TS2415: Class 'MyAuthConfig' incorrectly extends base class 'CustomConfig'.
  Types of property 'providers' are incompatible.
    Type '{ google: { clientId: string; scope: string[]; }; facebook: { clientId: string; scope: string[]; ...' is not assignable to type 'IProviders'.
      Index signature is missing in type '{ google: { clientId: string; scope: string[]; }; facebook: { clientId: string; scope: string[]; ...'.

I don't have ideia what's the problem. I have changed my tsconfig to include some keys that I took from ng2-ui-auth-example but the errors persist. I don't know if this could be the problem, but I'm using gulp + systemjs while you are using webpack.
This is my tsconfig.json
{
  "compilerOptions": {
    "rootDir": "client/src/app",
    "target": "es5",
    "module": "commonjs",
    "moduleResolution": "node",
    "sourceMap": true,
    "noEmitHelpers": true,
    "declaration": false,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "removeComments": false,
    "noImplicitAny": false,
    "suppressImplicitAnyIndexErrors": true
  },
  "exclude": [
    "client/config",
    "node_modules",
    "typings/main",
    "typings/main.d.ts",
    "typings/globals/node",
    "typings/globals/core-js"
  ],
  "compileOnSave": false
}

My configuration
import { CustomConfig } from 'ng2-ui-auth';

import { ConfigService } from './config.service';

export class MyAuthConfig extends CustomConfig {
    defaultHeaders: { [name: string]: string; } = { 'Content-Type': 'application/json' };
    baseUrl = ConfigService.BASE_ADDRESS;
    tokenPrefix = 'locaoffice';
    loginUrl = '/api/sessions';
    signupUrl = '/api/users';
    providers: { [provider: string]: any } = {
        google: {
            clientId: '***********',
            scope: ['profile',
                'email'
            ]
        },
        facebook: {
            clientId: '*********',
            scope: ['public_profile',
                'email'
            ]
        }
    };
}

When I run the ng2-ui-auth-example everything works fine.

I have no idea.
I'll leave this open if so I'll look into it further in the future. in the meantime try:
export class MyAuthConfig extends CustomConfig {
    defaultHeaders: { [name: string]: string; } = { 'Content-Type': 'application/json' };
    baseUrl = ConfigService.BASE_ADDRESS;
    tokenPrefix = 'locaoffice';
    loginUrl = '/api/sessions';
    signupUrl = '/api/users';
    providers = <any>{
        google: {
            clientId: '***********',
            scope: ['profile',
                'email'
            ]
        },
        facebook: {
            clientId: '*********',
            scope: ['public_profile',
                'email'
            ]
        }
    };
}

That should resolve the error

If anyone can provide a solution I'll be happy to accept, in the meantime I'm closing since there is a workable workaround.


Feature: Update .NET Framework to .NET 9.0

As a software developer,
I want to update the .NET framework of the project to .NET 9.0,
so that I can leverage the latest features, performance improvements, and security enhancements provided by the new version.
Acceptance Criteria:

The project files (.csproj, .sln) are updated to target .NET 9.0.
The GitHub Actions workflow (

.github/workflows/dotnet-desktop.yml
) is updated to use .NET 9.0.
3. All dependencies and NuGet packages are compatible with .NET 9.0.
4. The project builds successfully without errors or warnings.
5. All existing unit tests pass without any issues.
Upgrade Notes:

New Features: .NET 9.0 introduces new language features and APIs that can improve code quality and developer productivity.
Performance Improvements: The latest version includes performance optimizations that can make the application run faster and more efficiently.
Security Enhancements: Upgrading to the latest framework ensures that the application benefits from the latest security patches and improvements.
Long-term Support: Using the latest version ensures that the project remains supported and receives updates from Microsoft.
Community and Ecosystem: Staying up-to-date with the latest version allows the project to take advantage of the latest tools, libraries, and community support.

#17


Kubernetes probe fails when creating a dump

Description
I tried to add the monitor-container as sidecar to my ASP.NET Core application. The main-container in the deployment also has Kubernetes livenessProbe and readinessProbe configured. When creating a dump with the monitor (calling the /dump) endpoint, the main-process of the application is suspended as stated in the docs. This causes the probes to fail, so Kubernetes will restart the application before the dump-creation succeeds.
Is there any configuration I can try to make this work? I cannot remove the probes in a production application. 😉
If this is not possible with dotnet-monitor at the moment, is there another way to create a dump or do other analysis to a running Kubernetes pod without exec to the running container and run some preinstalled tools/scripts?
Configuration
ASP.NET Core version: 3.1.17 (based on mcr.microsoft.com/dotnet/core/aspnet:3.1-alpine)
Dotnet Monitor: 5.0.0-preview.6.21370.3+9c1714943e349b437ca3adeb8f4f8cb0df1355b3
OS: Alpine (both application and monitor)
Arch: x64
Env: Bare-Metal Kubernetes-Cluster (no cloud-provider)
Kubernetes Deployment YAML:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: feature-diagnostics-api
spec:
  selector:
    matchLabels: {}
  template:
    spec:
      containers:
      - image: our.aspnet-core-application:latest
        imagePullPolicy: IfNotPresent
        livenessProbe:
          failureThreshold: 3
          httpGet:
            path: /configuration/getVersion
            port: http
            scheme: HTTP
          initialDelaySeconds: 90
          periodSeconds: 10
          successThreshold: 1
          timeoutSeconds: 5
        name: api
        ports:
        - containerPort: 80
          name: http
          protocol: TCP
        resources:
          limits:
            cpu: 1500m
            memory: 1Gi
          requests:
            cpu: 250m
            memory: 400Mi
        readinessProbe:
          failureThreshold: 3
          httpGet:
            path: /configuration/getVersion
            port: http
            scheme: HTTP
          initialDelaySeconds: 90
          periodSeconds: 10
          successThreshold: 1
          timeoutSeconds: 5
        securityContext:
          capabilities:
            drop:
            - ALL
          privileged: false
          runAsNonRoot: true
          runAsUser: 1000
        volumeMounts:
        - mountPath: /tmp
          name: diagnostics
      - args:
        - --urls
        - https://*:52323
        env:
        - name: DotnetMonitor_Metrics__Enabled
          value: "false"
        - name: DotnetMonitor_Kestrel__Certificates__Default__Password
          valueFrom:
            secretKeyRef:
              key: pass
              name: monitorcert
        - name: DotnetMonitor_Kestrel__Certificates__Default__Path
          value: /etc/aspnet/customcert/cert.pfx
        - name: DotnetMonitor_ApiAuthentication__ApiKeyHashType
          value: SHA256
        - name: DotnetMonitor_ApiAuthentication__ApiKeyHash
          valueFrom:
            secretKeyRef:
              key: ApiAuthentication__ApiKeyHash
              name: monitorkey
        image: mcr.microsoft.com/dotnet/monitor:5.0.0-preview.6-alpine
        imagePullPolicy: IfNotPresent
        name: monitor-sidecar
        ports:
        - containerPort: 52323
          name: http
          protocol: TCP
        securityContext:
          capabilities:
            drop:
            - ALL
          privileged: false
          runAsNonRoot: true
          runAsUser: 1000
        volumeMounts:
        - mountPath: /tmp
          name: diagnostics
        - mountPath: /etc/aspnet/customcert
          name: customcert
      volumes:
      - emptyDir: {}
        name: diagnostics
      - name: customcert
        secret:
          defaultMode: 420
          secretName: monitorcert

Regression?
Not tested.
Other information
Nothing, please ask if some info is missing.

A short update here. We finally were able to create full memory dumps within a few seconds. Christian had the problems mentioned above when he uses heap dumps (default value of dotnet-monitor /dump endpoint). After switching to full dumps it only takes a few seconds.
But nevertheless it would be nice if there are some official best practices. I think the time the dump takes will depend on the overall workload. As we have not migrated the production systems into our cluster we currently can not make any assumptions how long it will take in the future

Description
I tried to add the monitor-container as sidecar to my ASP.NET Core application. The main-container in the deployment also has Kubernetes livenessProbe and readinessProbe configured. When creating a dump with the monitor (calling the /dump) endpoint, the main-process of the application is suspended as stated in the docs. This causes the probes to fail, so Kubernetes will restart the application before the dump-creation succeeds.
Is there any configuration I can try to make this work? I cannot remove the probes in a production application. 😉
If this is not possible with dotnet-monitor at the moment, is there another way to create a dump or do other analysis to a running Kubernetes pod without exec to the running container and run some preinstalled tools/scripts?
Configuration
ASP.NET Core version: 3.1.17 (based on mcr.microsoft.com/dotnet/core/aspnet:3.1-alpine)
Dotnet Monitor: 5.0.0-preview.6.21370.3+9c1714943e349b437ca3adeb8f4f8cb0df1355b3
OS: Alpine (both application and monitor)
Arch: x64
Env: Bare-Metal Kubernetes-Cluster (no cloud-provider)
Kubernetes Deployment YAML:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: feature-diagnostics-api
spec:
  selector:
    matchLabels: {}
  template:
    spec:
      containers:
      - image: our.aspnet-core-application:latest
        imagePullPolicy: IfNotPresent
        livenessProbe:
          failureThreshold: 3
          httpGet:
            path: /configuration/getVersion
            port: http
            scheme: HTTP
          initialDelaySeconds: 90
          periodSeconds: 10
          successThreshold: 1
          timeoutSeconds: 5
        name: api
        ports:
        - containerPort: 80
          name: http
          protocol: TCP
        resources:
          limits:
            cpu: 1500m
            memory: 1Gi
          requests:
            cpu: 250m
            memory: 400Mi
        readinessProbe:
          failureThreshold: 3
          httpGet:
            path: /configuration/getVersion
            port: http
            scheme: HTTP
          initialDelaySeconds: 90
          periodSeconds: 10
          successThreshold: 1
          timeoutSeconds: 5
        securityContext:
          capabilities:
            drop:
            - ALL
          privileged: false
          runAsNonRoot: true
          runAsUser: 1000
        volumeMounts:
        - mountPath: /tmp
          name: diagnostics
      - args:
        - --urls
        - https://*:52323
        env:
        - name: DotnetMonitor_Metrics__Enabled
          value: "false"
        - name: DotnetMonitor_Kestrel__Certificates__Default__Password
          valueFrom:
            secretKeyRef:
              key: pass
              name: monitorcert
        - name: DotnetMonitor_Kestrel__Certificates__Default__Path
          value: /etc/aspnet/customcert/cert.pfx
        - name: DotnetMonitor_ApiAuthentication__ApiKeyHashType
          value: SHA256
        - name: DotnetMonitor_ApiAuthentication__ApiKeyHash
          valueFrom:
            secretKeyRef:
              key: ApiAuthentication__ApiKeyHash
              name: monitorkey
        image: mcr.microsoft.com/dotnet/monitor:5.0.0-preview.6-alpine
        imagePullPolicy: IfNotPresent
        name: monitor-sidecar
        ports:
        - containerPort: 52323
          name: http
          protocol: TCP
        securityContext:
          capabilities:
            drop:
            - ALL
          privileged: false
          runAsNonRoot: true
          runAsUser: 1000
        volumeMounts:
        - mountPath: /tmp
          name: diagnostics
        - mountPath: /etc/aspnet/customcert
          name: customcert
      volumes:
      - emptyDir: {}
        name: diagnostics
      - name: customcert
        secret:
          defaultMode: 420
          secretName: monitorcert

Regression?
Not tested.
Other information
Nothing, please ask if some info is missing.

Hey any update on this??


Forked snapshot

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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 decreased (-0.02%) to 82.53%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
4951

Relevant Lines:
5999

💛  - Coveralls


How do I create package parameters?

I have a package where I create a single parameter using a JSON object as the value. I do this manually now with wsk action create ... -P PackageParams.json where the file contains something like:
{
   "credentials": {
      "username": "something",
      "password": "something else"
   }
}

How would that manifest in the YML exactly?

@ScottChapman  please try using this manifest:
packages:
    validate_json: #this is a package name
        actions:
            validate_json_params: # this is a action name
                function: actions/dump_params.js
                runtime: nodejs:6
                inputs:
                    credentials:
                        type: json
                        value: '{ "username": "something", "password": "something else" }'

The above was for action params, but relocated it to package level worked! Thanks!


Image upload not working

I have a collection where I try to upload an Asset, a jpg file of 516kb.
The loader gets to a 100% but then the "Something went wrong." message appears.
The file is not stored in the uploads directory, it is however present in the storage/tmp directory.
I checked and the storage/uploads directory is writable.
info
PHP version: 7.2.11
Upload file size limit: 4M
Memory limit: 128M
Cockpit version: 0.8.3

@drieshooghe check in the devtools network tab what response message you have for the upload request

@paoloamgomes I forgot to mention, the application returns a 500 error

sure, but what you have in the network tab response, check below:

just to understand if there is anything more about the error

@drieshooghe I know you said that you have correct permissions, but just for be sure, give full permissions (including group and others - 777) on the storage/uploads folder and test again.

I still get the same result.

Yeah, was expecting that 😞 Do you can upload assets on the asset page?

Nope, no luck...

@pauloamgomes I resolved the issue.
The PHP GD extension wasn't installed correctly, which caused the problem.
All is well! Thank you for your help.
Perhaps the php GD extension is an idea for an additional check in the installer script?

@drieshooghe I'm having the same problem running Cockpit with the Dockerfile. What was wrong with GD in your setup? Maybe I have the same problem.

It gets worse. I run cockpit locally with the exact same docker compose file and uploading works.

Some more information:
This is the status code returned: 413 Request Entity Too Large
Strange, since the file being uploaded is barely 3MB and the .htaccess file sets the upload max size to 256M

Found the reason... Locally I'm accessing the cockpit instance directly, but on my remote server it's running behind nginx as reverse proxy. Something must have changed in the nginx config that limits the size of the upload files. I updated the nginx config and it all works now.

So for future ref this is how I dealt with it
Step 1 : go to bootstrap.php line 100 and set 'debug' = true, since my error was Call to undefined function finfo_open()
Step 2 : install fileinfo	Universal Extensions if is not installed on your php
Step 3 : go to php.ini of your website and find extension=fileinfo and remove the ;
Step 4 : reload and restart php service on your server.
Now I can upload files. Cockpit still rules


allow matplotlib > 3.8

PyPlot.jl previously didn't support mpl 3.9, it should now

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.22%. Comparing base (6ffefec) to head (7320e3b).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
- Coverage   88.23%   88.22%   -0.02%
==========================================
  Files          18       18
  Lines        1488     1486       -2
==========================================
- Hits         1313     1311       -2
  Misses        175      175

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Omit DSA sigalgs from TLS 1.3-only ClientHellos

The TLS 1.3 spec strongly discourages sending DSA sigalgs in the ClientHello, though we are forced to continue to do so due to our stability guarantees within a release branch (given that master will become 1.1.1, not 1.2.0).  So, take the step that we can and do not send DSA sigalgs when we are only offering TLS 1.3.
Inspired by discussion on #3294 .
(N.B. the perl bits probably need help.)
Checklist

[x] tests are added or updated

Looks good now, thanks for the pointers.

I realized that the new tests should be skipped for no-dsa builds ... which were currently broken anyway.  So there's a couple more commits to fix that.

@levitte I think this should be ready for your second review

Rebased past 70-test_tls13messages.t conflict; needs @mattcaswell reconfirm and second review.  @levitte , you had looked at previous versions; want to look again?

So should this also cover SHA1/SHA224. Or is that for a separate PR?

I think we can roll that into this PR; the spec's guidance in this space changed between draft-19 (which was current when I wrote this code) and draft-20 (currently current).

(I am pretty deep in a grungy rebase and may not get to this for a couple days.)

Whoops, I have to update this for SHA224 now, don't I.

Okay, the Message.pm defined constants have the SHA224 algs now (but namespaced separately, since they are excluded from the TLS 1.3 draft entirely), and the tests also check for SHA224.
(The previous upload had the code check to disallow SHA224 for TLS 1.3-only ClientHellos.)

What's the state of this?  Ready or needs a re-confirm?

Needs a reconfirm.
@mattcaswell had looked at it previously, but that was several weeks ago, at this point.

@mattcaswell not quite yet, it seems, so please go ahead and push.

@mattcaswell I guess "yet" became "now" quite quickly.

Merged, the four commits listed above.


Add edit in place option

Closes #6
Using like this:
yarn-tools fix-duplicates --in-place -- yarn.lock

The -- arg is required or it won't work, I think it's a limitation of commander.js

@scinos could you please review this PR?

Hey @felipemsantana
Thanks for the PR.
I have been working on major refactor of this library does not merge cleanly anymore :( Sorry about that.
The new version is in the branch 1.0-dev. I already included a flag to write yarn.lock instead of printing it to the console. It is similar to your solution: by default it saves yarn.lock in place, unless the flag --print is passed. Could you take a look and validate it is still working please?
Also I like your tests for the flag. Would you be able to open a PR to 1.0-dev with the tests for that functionality please?

Ok, I'm gonna do that


Pytz needs a version bump

https://pypi.org/project/pytz/#history
Just released this month: 2018.4

Shipped to develop.


Feature: EmmyLua-like metamethod documenting

It would be awesome if we included some type of EmmyLua documentation extension to support metamethods (particularly the Mathematic Operators).
I'd like to be able to code something like this:
---@class MyClass
---@metamethod add fun:(self: MyClass, other: MyClass) : MyClass
---@metamethod subtract fun:(self: MyClass, other: MyClass) : MyClass
local x = {}

---@type MyClass
local a

---@type MyClass
local b

--- Ideally, `c` would be inferred to be MyClass off of the annotations
local c = a + b

This would be really useful for __call aswell!
local tbl = setmetatable({}, {
	__call = function (s) return s end
})

print(tbl("Hi"))

see #599


Preview becomes corrupted if source ANSI is updated

I've noticed I have to close the preview and reopen it every time the ANSI being previewed is updated, otherwise it displays with semi scrambled text and formatting.

In 1.0.5, I've made some fixes that could resolve this. @jaydenseric, could you please check if the bug is still there? If it still is, I would appreciate some more info on how to reproduce this.

Really nice work in the v1.0.5 release! Fantastic. I just played around and the preview seems to update ok; I'll see if it happens again this week while working on more snapshot tests. If it does, we can reopen :)

This issue is still reproducible on my machine with version 1.1.0
My vscode:
Version: 1.52.1
Commit: ea3859d4ba2f3e577a159bc91e3074c5d85c0523
Date: 2020-12-16T16:32:10.090Z
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 4.15.0-134-generic

This issue is still reproducible on my machine with version 1.1.0
My vscode:
Version: 1.52.1
Commit: ea3859d4ba2f3e577a159bc91e3074c5d85c0523
Date: 2020-12-16T16:32:10.090Z
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 4.15.0-134-generic


Quickstart 6_AspNetIdentity build error with context.IdP

File IdentityServer4.Samples/Quickstarts/6_AspNetIdentity/src/IdentityServerWithAspNetIdentity/Controllers/AccountController.cs contains the following code in the Login method:
           if (context?.IdP != null && (await _signInManager.GetExternalAuthenticationSchemesAsync()).Any(p => string.Equals(p.Name, context.IdP, StringComparison.InvariantCultureIgnoreCase)))             {                 return ExternalLogin(context.IdP, returnUrl);             }
This causes the build error CS0103	The name 'context' does not exist in the current context.

Just hit same compile issue in the combined aspnet & ef example on both release and dev branches
https://github.com/IdentityServer/IdentityServer4.Samples/blob/release/Quickstarts/Combined_AspNetIdentity_and_EntityFrameworkStorage/src/IdentityServerWithAspIdAndEF/Controllers/AccountController.cs

Added a PR here https://github.com/IdentityServer/IdentityServer4.Samples/pull/248


Update README.md

To promote new way of using it rather than old way as discussed in https://github.com/TomWright/dasel/discussions/154

to promote the new way of using it rather than the old way as discussed in https://github.com/TomWright/dasel/discussions/154


subdomain support

is it possible to configure subdomain routing as like laravel ?

I don’t know how laravel does it be you can define a route to only match on a certain host, details are in the documentation.
Sent from ProtonMail Mobile
On Sun, May 17, 2020 at 20:55, Mohammad Emran<EMAIL_ADDRESS>wrote:

is it possible to configure subdomain routing as like laravel ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

My intention is something like this.
$router
->map('GET', '/acme/route', 'Acme\Controller::getMethod')
->setHost('{subdomain}.example.com');
Where subdomain variable will be passed to Acme\Controller::getMethod.

It just picks up the host based on the request.
Sent from ProtonMail Mobile
On Sun, May 17, 2020 at 21:09, Mohammad Emran<EMAIL_ADDRESS>wrote:

My intention is something like this.
$router
->map('GET', '/acme/route', 'Acme\Controller::getMethod')
->setHost('{subdomain}.example.com');
Where subdomain variable will be passed to Acme\Controller::getMethod.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Will close this as matching based on host is already possible


feat: add global --json flag

Motivation

Related: https://github.com/foundry-rs/foundry/issues/8794
Adds a global --json flag on the Shell level
Opening for visibility as I ran into a strange stalling bug related to sh_println!
Solution

Ideally we would have a FormatChoice similar to ColorChoice but to avoid breaking changes the current approach is preferred.
Whilst JSON output is currently not supported everywhere this makes it easier to support it.

I think we should really just drop -j as short for json, it is not common anywhere and usually means jobs (threads), like in makefile

I think we should really just drop -j as short for json, it is not common anywhere and usually means jobs (threads), like in makefile

makes sense, we introduced --j for tests already


Support object constructor expression

Description:

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Fixed in #25637


Evaluation Metrics

Hello,
Thanks for your great work, your contributions have truly impressed me! I read your paper and have some questions about evaluation. I'm curious about whether the FVD, PSNR, LPIPS metrics are calculated across the entire 256x256 region or only within the outpainted region.
Thanks! Any clarification you could provide would be sincerely appreciated.

Hello! Do you have the answer?


fix dropout always being used in train mode in scaled_dot_product_attention

@michalozeryflato please merge this. I don't have write permissions


Authentication verification fails within function validate_login

Initially entering API creds and run the import, validation fails.
If function
validate_login(auth_info, out) will be commented out -> auth works
in:
[(https://github.com/CrowdStrike/MISP-tools/blob/main/cs_misp_import/check_config.py )]

Hi @ps4signedout -
Does this still occur with the latest version (0.7.2)?


504 Server Error: GATEWAY_TIMEOUT

I have been observing some strange behavior from the API today and thought it was a problem on my side, but just now I got a 504 Server Error: GATEWAY_TIMEOUT. Is the API server going through some maintenance now is there any way I can debug this problem?
Thanks much!

There weren't any outages planned for today, but that doesn't mean there weren't any :smile:  A 504 is probably server-side and not something you're doing. I'm not seeing this behavior right now so it's hard for me to diagnose. If you see this happen again with the API, try the website. If the website also doesn't work, that's more evidence for capacity problems or other server issues.

Thanks for the feedback. The website works fine, but I have a list of profiles I'd like to check.
Just now I tried the API again and this time got a 500. It seems like mashape must be going through some maintenance or something.

Our lab has a few APIs behind Mashape and all of them received error reports yesterday from users, so signs indeed point to Mashape having issues. Are your issues resolved today?

I think the Mashape issues have been cleared up; we haven't heard of these errors happening recently. Please reopen if you're still having issues.

Added entry to FAQ.


The effectiveness of GT_AUG?

@sshaoshuai , will GT_AUG increase the result greatly? I am a little bit confused about GT_AUG, since the gt-bboxes and inside points from other scenes do not have the same point distribution of current scene. For example, a gtbox with inside points is put in the shadow of a car in the current scene, where no points should appears.

GT_AUG is important for the performance on KITTI dataset since the training samples of KITTI dataset are relatively small.
You are right, GT_AUG will destroy the far-away context information and introduce more noise to the dataset, but it could simulate more situations and could also balance the number of positive samples and negative samples in the scene, and finally the performance got improved.


[Segment Replication] Update RefreshPolicy.WAIT_UNTIL to wait for replica refresh.

Current behavior of RefreshPolicy.WAIT_UNTIL waits to ack an index request until the operation is visible/searchable to the client. It does this by registering listeners that complete when the last written translog location after a refresh is at least the furthest translog location from the writes executed in the request.  This logic breaks with SR because replicas do not index before writes to the translog, so the last written location is not guaranteed to be visible.
Rather than comparing xlog locations, we should compare the latest max seqNo that the replica has refreshed on.  This is made possible with this change, where replicas now receive an accurate seqNo.

An idea of how to accomplish this:

Update TransportShardbulkAction#performOnReplica to also return the max seqNo from a BulkShardRequest in a Tuple or new class so that both segrep & non segrep behavior can be supported by this method.  The max seqno from a set of operations is not guaranteed to be the latest xlog write location, so we must compute it from the primary's BulkItemResponse.
Update WriteReplicaResult to include the maxSeqNo.
Update RefreshListeners to register refresh listeners with the seqNo instead of xlog location, and fire off the listeners after refresh iff the latest replicationCheckpoint's seqNo is at least that #.
Overload IndexShard.addRefreshListener to register listeners with updated method in RefreshListeners.
Update AsyncAfterWriteAction to optionally include the seqNo.  In its run method pass the seqno (new tuple/class) to indexShard.addRefreshListener.


Error with latest v0.2.3 in HA 2021.6.0

Getting this error with latest v0.2.3 in HA 2021.6.0
This error originated from a custom integration.
Logger: homeassistant.config_entries
Source: custom_components/gardena_smart_system/init.py:88
Integration: Gardena Smart System integration (documentation, issues)
First occurred: 19:04:42 (1 occurrences)
Last logged: 19:04:42
Error setting up entry for gardena_smart_system
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 293, in async_setup
result = await component.async_setup_entry(hass, self)  # type: ignore
File "/config/custom_components/gardena_smart_system/init.py", line 51, in async_setup_entry
await hass.async_add_executor_job(gardena_system.start)
File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/gardena_smart_system/init.py", line 88, in start
self.smart_system.update_locations()
File "/usr/local/lib/python3.8/site-packages/gardena/smart_system.py", line 209, in update_locations
response_data = self.__call_smart_system_get(f"{self.SMART_HOST}/v1/locations")
File "/usr/local/lib/python3.8/site-packages/gardena/smart_system.py", line 180, in __call_smart_system_get
if self.__response_has_errors(response):
File "/usr/local/lib/python3.8/site-packages/gardena/smart_system.py", line 159, in __response_has_errors
r = response.json()
File "/usr/local/lib/python3.8/site-packages/requests/models.py", line 900, in json
return complexjson.loads(self.text, **kwargs)
File "/usr/local/lib/python3.8/site-packages/simplejson/init.py", line 525, in loads
return _default_decoder.decode(s)
File "/usr/local/lib/python3.8/site-packages/simplejson/decoder.py", line 370, in decode
obj, end = self.raw_decode(s)
File "/usr/local/lib/python3.8/site-packages/simplejson/decoder.py", line 400, in raw_decode
return self.scan_once(s, idx=_w(s, idx).end())
simplejson.errors.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

II have the same problem

same here. I think this addon is simply dead. Sad but true :/

Hello ! The addon is not dead. I probably made an issue while releasing the
new version..
I am currenty trying to figure what and i will release a new versiln soon !
Le ven. 4 juin 2021 à 19:45, Chris Lange @.***> a
écrit :

same here. I think this addon is simply dead. Sad but true :/
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/py-smart-gardena/hass-gardena-smart-system/issues/68#issuecomment-854899733,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAARGIYFMZDCMMS4HBCUPYDTREGMXANCNFSM46BFWQKQ
.

I just released a new 0.2.3 version in beta. Could you try uninstall and reinstall the new one ?
I am currently running Home Assistant 2021.6.1 and it seems to work from my first tests.

Sorry, I was referring to the badge seen in HA which states "unmainted as of 2020". But happy to hear, that the development is still ongoing! Thanks for your efforts!
I did an upgrade to the new beta and I can confirm that it's working again!
HA 2021.6.1
Thanks a lot for your quick reaction!

Good to know :)
I updated the badge after the last failing release !
I will close this ticket. Do not hesitate to repoen it if it is still failing !
Jérémie

I'm still getting the same error. I have removed everything and reinstalled the latest 0.2.3.
When I enter email, pw, and app id I get this in the logs.
This error originated from a custom integration.
Logger: custom_components.gardena_smart_system.config_flow
Source: custom_components/gardena_smart_system/config_flow.py:133
Integration: Gardena Smart System integration (documentation, issues)
First occurred: 10:40:19 (1 occurrences)
Last logged: 10:40:19
Unexpected exception
Traceback (most recent call last):
File "/config/custom_components/gardena_smart_system/config_flow.py", line 66, in async_step_user
await self.hass.async_add_executor_job(
File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/gardena_smart_system/config_flow.py", line 133, in try_connection
smart_system.update_locations()
File "/usr/local/lib/python3.8/site-packages/gardena/smart_system.py", line 209, in update_locations
response_data = self.__call_smart_system_get(f"{self.SMART_HOST}/v1/locations")
File "/usr/local/lib/python3.8/site-packages/gardena/smart_system.py", line 180, in __call_smart_system_get
if self.__response_has_errors(response):
File "/usr/local/lib/python3.8/site-packages/gardena/smart_system.py", line 159, in __response_has_errors
r = response.json()
File "/usr/local/lib/python3.8/site-packages/requests/models.py", line 900, in json
return complexjson.loads(self.text, **kwargs)
File "/usr/local/lib/python3.8/site-packages/simplejson/init.py", line 525, in loads
return _default_decoder.decode(s)
File "/usr/local/lib/python3.8/site-packages/simplejson/decoder.py", line 370, in decode
obj, end = self.raw_decode(s)
File "/usr/local/lib/python3.8/site-packages/simplejson/decoder.py", line 400, in raw_decode
return self.scan_once(s, idx=_w(s, idx).end())
simplejson.errors.JSONDecodeError: Expecting value: line 1 column 1 (char 0)


Add 'mermaidOptions' as plugin options

mermaidOptions can now be used to specify additional mermaid
initialization options.
See README.md for an example.

Thanks! This is a good addition, would you mind linking to https://mermaidjs.github.io/#/mermaidAPI in the README so people know where to look for the available options?

Nevermind, looks like I an do that myself :)

Thanks @ChappIO for doing the merge.  Do you mind publishing it to npm when you're ready so that I can add it to my next project?

@rheinardkorf it has been published now

cheers


Details

Use this issue to discuss the details component in the ONS Design System

It isn't possible to apply custom classes to the details element.

@will-madebymoon and I think we should remove the arrow animation as it doesn't add any value.

When a user closes the details component using the inner button, the summary element should be refocused for screen readers to be able to easily move on to the next item on the page.

Buttons used in details components are type submit. These should be type button.

@will-madebymoon Paragraph content in an expanded details component is getting a bit lost when it's displayed inline with a lot of paragraphs in body content...

Could we consider indenting it or something like e.g....

Details needs a default bottom margin

Maybe we should address all all of these issues when we look at spacing: https://trello.com/c/h0RbO4UV/99-spacing

Details element cannot be accessed with dragon naturally speaking as a button or as a link

Details chevron hidden when screen colours inverted (using v12 ZoomFusion)

Pointer events need to be turned off on all elements within the summary as GA fails to pick up click events

details__title should use a semantically appropriate heading tag (not a <span>)

Safari rendering issue: font-size rem units ignored on elements following <details> and <summary> (https://colloq.io/blog/safaris-detailssummary-rem-font-size-issue)

'Open' attribute does not validate
<div id="collapsible" class="collapsible js-collapsible" open data-btn-close="Hide this">
In here: https://ons-design-system.netlify.com/components/collapsible/examples/collapsible/
In the open state.
I'm looking at the open attribute ...js-collapsible" open data-btn-close... it does not validate

Collapsible chevron doesn't left align with other page elements when collapsed

Collapsible component no javascript fallback not working
(Contents should display with a semantic heading above (no chevron or link style) See: #711


TODO: patch of server check

The new script resources/server/bin/helpers/check-requirements-linux.sh will likely need a patch, to ne adjusted for whatever the minimal requirements for this fork are.
Note: what the exact minimal requirements are and how this makes the difference to VSCodium is still missing in https://github.com/Alex313031/codium/blob/master/README.md (which is tracked in #1 and #2 - one of those may need to be reopened):

@Alex313031 wrote:

However, I could update the docs to use the new name and make it clear that this is the (currently) only vscode/vscodium fork to still support win7/8/8.1 or ubuntu 16.04.

This part is very important, as the question comes up again also for Ubuntu 18/RHEL7, as electron dropped support for that (but there are now the experimental glibc-217 node binaries, which may be used as drop-in replacement (also checked for VSCodium see https://github.com/VSCodium/vscodium/issues/1629)

@GitMensch This should still work with 18.04. I build and test it in a 18.04 VM.

The point is that vscodeserver would likely not work if you don't patch that file - as this raises an error when glibc is considered to be too old.

@GitMensch They added a patch for this upstream now, and besides, I don't ship the web version, as it doesn't make sense for this fork (which is for running on older OSes)


Spelling fix

What Changed

Just a small spelling fix. I was reading through the docs and found a repeat word that shouldn't be there. Everything else looked good!
Change Type

[ ] maintenance
[x] documentation
[ ] patch
[ ] minor
[ ] major

📦 Published PR as canary version: 0.0.21--canary.18.1506888.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in v0.0.24 :rocket:


TypeError: expected str, bytes or os.PathLike object, not dict

Describe the bug
my dataset is in form : train- image /n -labels
and tried the code:
from datasets import load_dataset

data_files = {
    "train": "/content/datasets/PotholeDetectionYOLOv8-1/train/",
      "validation": "/content/datasets/PotholeDetectionYOLOv8-1/valid/",
      "test": "/content/datasets/PotholeDetectionYOLOv8-1/test/"
}
dataset = load_dataset("imagefolder", data_dir=data_files)
dataset

got error:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
[<ipython-input-29-2ef1926f73d9>](https://localhost:8080/#) in <cell line: 8>()
      6       "test": "/content/datasets/PotholeDetectionYOLOv8-1/test/"
      7 }
----> 8 dataset = load_dataset("imagefolder", data_dir=data_files)
      9 dataset

6 frames
[/usr/lib/python3.10/pathlib.py](https://localhost:8080/#) in _parse_args(cls, args)
    576                 parts += a._parts
    577             else:
--> 578                 a = os.fspath(a)
    579                 if isinstance(a, str):
    580                     # Force-cast str subclasses to str (issue #21127)

TypeError: expected str, bytes or os.PathLike object, not dict

Steps to reproduce the bug
as share above
Expected behavior
load images and labels , but my dataset only uploads images

https://huggingface.co/datasets/Andyrasika/potholes-dataset

Environment info
colab pro

You should be able to load the images by modifying the load_dataset call like this:
dataset = load_dataset("imagefolder", data_dir="/content/datasets/PotholeDetectionYOLOv8-1")

The imagefolder builder expects the image files to be in path/label/image_file (e.g. ..../train/dog/image_1.jpg), so the solution in your case is to create metadata files (one for each split; as explained here) that map the images to their labels.

You should be able to load the images by modifying the load_dataset call like this:
dataset = load_dataset("imagefolder", data_dir="/content/datasets/PotholeDetectionYOLOv8-1")

The imagefolder builder expects the image files to be in path/label/image_file (e.g. ..../train/dog/image_1.jpg), so the solution for the labels in your case is to create metadata files (one for each split; as explained here) that map the images to their labels.

I tried like this but only uploads images and not labels, Andyrasika/potholes-dataset

As explained in my previous comment, you need to define metadata files to load the labels or update the paths to be in the format train/label/image (train- image /n -labels is not supported by the loader).

I downloaded my file after annotating using roboflow . It gives train-
images, labels ,  test- images, labels , valid- images, labels . I hope it
gives you an idea of the dataset . Please advise on this dataset
On Tue, Oct 10, 2023 at 18:12 Mario Šaško @.***> wrote:

As explained in my previous comment, you need to define metadata files to
load the labels or update the paths to be in the format train/label/image
(train- image /n -labels is not supported by the loader).
—
Reply to this email directly, view it on GitHub
https://github.com/huggingface/datasets/issues/6285#issuecomment-1755335215,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4LJNN56FWWTSBYTSTUWHLX6U7CVAVCNFSM6AAAAAA5YHCSTGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJVGMZTKMRRGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Create structure for testing

Added pom.xml dependency and file tree structure for the jUnit testing.
Includes two DELETEME files, since it didn't want to stage empty folders for push

Consider adding Mockito dependency. It allows to use mocks, which could be very useful in our project because we are going to use database and external API.
https://mvnrepository.com/artifact/org.mockito/mockito-core/3.5.2

It is very convenient to use other packeges for testing in Spring a short article about testing in Spring
I would suggest to edit the current version of pom.xml and add the required dependencies to it instead of modifying the outdated version.

This PR got deprecated since the migration to Spring.


Support namespace-scoped classes in svcat get

🛑 This depends upon WIP work in https://github.com/kubernetes-incubator/service-catalog/pull/2116. Please don't start until that's merged.
Currently svcat get class NAME is written to work only with cluster-scoped service classes. Now that we support namespace-scoped broker resources, this command should be extended to understand both types ClusterServiceClass and ServiceClass.
$ svcat get class NAME [--cluster] [--namespace] [--all-namespaces]

--cluster is a boolean flag that needs to be added to this command. This change has already been made in that open PR and is why you want to wait before doing this issue.
svcat get class NAME --cluster should retrieve a ClusterServiceClass with NAME
svcat get class NAME should retrieve a ServiceClass with NAME in the current namespace. You should be able to modify the call to c.App.RetrieveClass to accept a scope filter with the cluster and namespace flags set. The CLI should not be doing the logic of figuring out how to split out those calls itself, and should delegate that to the svcat package.

See my PR #2116 for an example of what that looks like for listing service classes with svcat get classes. I've already added an interface to both classes so that you can use the interface instead of the concrete types (again see the PR for how the interface is used).
https://github.com/kubernetes-incubator/service-catalog/blob/9130c341675ac3d1bdb0168da14c9b78a510b9de/pkg/svcat/service-catalog/class.go#L33-L43

This was fixed by #2116


Need to have DNS servers exposed at the time of NIC create

Description
az network nic create should expose the option of being able to set a custom DNS server. The PowerShell equivalent of this is:
New-AzureRmNetworkInterface -Name "NetworkInterface1" -ResourceGroupName "ResourceGroup1" -Location "centralus" -SubnetId "/subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/resourceGroups/ResourceGroup1/providers/Microsoft.Network/virtualNetworks/VirtualNetwork1/subnets/Subnet1" -IpConfigurationName "IPConfiguration1" -DnsServer "<IP_ADDRESS>", "<IP_ADDRESS>"
Also can be found at this link: https://docs.microsoft.com/en-us/powershell/module/azurerm.network/New-AzureRmNetworkInterface?view=azurermps-4.0.0
CLI does show this when I do az network nic show as below:
az network nic show --name nic1 -g pseudovipskip
{
"dnsSettings": {
"appliedDnsServers": [],
"dnsServers": [
"<IP_ADDRESS>"
],
"internalDnsNameLabel": null,
"internalDomainNameSuffix": "io4hp2nl3juexm2h50u44p1uub.xx.internal.cloudapp.net",
"internalFqdn": null
},
Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here: pip
CLI Version: What version of the CLI and modules are installed? (Use az --version)
Answer here: 2,0,0
OS Version: What OS and version are you using?
Answer here: Windows
Shell Type: What shell are you using? (e.g. bash, cmd.exe, Bash on Windows)
Answer here: cmd.exe

This is a duplicate of #3558.

You can set these currently with:
az network nic update -g {rg} -n {nic} --set dnsSettings.dnsServers=['<IP_ADDRESS>', '<IP_ADDRESS>']
You may need to adjust the syntax somewhat and do any necessary escaping based on your shell of choice.


update to dotnet 5.0.0-preview.7.20364.11

PR Summary
update the build to use dotnet 5.0.0 preview.7.20364.11
PR Context
We need to keep up to date with the various dotnet previews
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

No manual PR required anymore


THRIFT-4984: Ignore EOF errors in TSimpleServer, take 2

Client: go
This is a different approach to take THRIFT-4984: Instead of checking
EOF errors in place, return them and check them before logging.

@dcelasun

@dcelasun I added comments regarding why do we ignore EOF errors, as per discussion in the ticket.

  Did you forget to use a TMultiplexProtocol in your client?
Hi!
2019/10/29 09:02:20 error processing request: TException
--- FAIL: TestTJSONProtocol (0.00s)
    thrifttest_driver.go:224: TestException Xception failed
2019/10/29 09:02:20 error processing request: TException
--- FAIL: TestTBinaryProtocol (0.01s)
    thrifttest_driver.go:224: TestException Xception failed
2019/10/29 09:02:20 error processing request: TException
--- FAIL: TestTCompactProtocol (0.01s)
    thrifttest_driver.go:224: TestException Xception failed
FAIL
FAIL	tests	0.412s
ok  	dontexportrwtest	0.003s

@dcelasun I fixed the broken test. PTAL.


Merging forked toolkit repo with head

Docs Build status updates of commit eae6674:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/toolkit/toolkit-overview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


readme: add explaination about the MC command prompt

fixes #22
Signed-off-by: Gaetan de Villele<EMAIL_ADDRESS>

LGTM :palm_tree:


[FEATURE] add cursor pointer to see next post

Description
add cursor pointer to see next post instead of normal arrow
Screenshots

Additional information
No response

Description
add cursor pointer to see next post instead of normal arrow
Screenshots

Additional information
No response

Go ahead!