Question regarding Mitie backend with 'de'

I experimented the use of Mitie backend with 'de' language (with de feature extractor, while I just hoped the tokenizer would work). It seemed to work till I got the error below.
Is this the reason (or are there other reasons) why 'de' is not 'allowed' to be used with Mitie?
now do training
num training samples: 164
C: 200   f-score: 0.71875
C: 400   f-score: 0.71875
C: 300   f-score: 0.725
C: 310   f-score: 0.725
C: 250   f-score: 0.71875
C: 305   f-score: 0.71875
C: 302.5   f-score: 0.73125
C: 302.083   f-score: 0.725
C: 303.385   f-score: 0.725
C: 302.734   f-score: 0.725
best C: 302.5
test on train:
65  0  0  1  0  0  0
0 11  0  0  0  0  0
0  0 21  1  0  0  0
0  0  0 39  0  0  0
0  0  0  0  4  0  0
0  0  0  0  0 13  0
0  0  0  0  0  0  9
overall accuracy: 0.987805
Training time: 2746 seconds.
df.number_of_classes(): 7
Traceback (most recent call last):
File "/usr/local/miniconda2/installation/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/usr/local/miniconda2/installation/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "build/bdist.macosx-10.7-x86_64/egg/rasa_nlu/train.py", line 73, in
File "build/bdist.macosx-10.7-x86_64/egg/rasa_nlu/train.py", line 66, in do_train
File "build/bdist.macosx-10.7-x86_64/egg/rasa_nlu/trainers/trainer.py", line 28, in train
File "build/bdist.macosx-10.7-x86_64/egg/rasa_nlu/trainers/mitie_trainer.py", line 22, in train_entity_extractor
File "build/bdist.macosx-10.7-x86_64/egg/rasa_nlu/trainers/mitie_trainer_utils.py", line 28, in train_entity_extractor
File "build/bdist.macosx-10.7-x86_64/egg/rasa_nlu/trainers/mitie_trainer_utils.py", line 12, in find_entity
ValueError: <exception str() failed>

@steookk This doesn't seem to be a german related issue but rather an issue with your training data, if you don't mind sending me your training data<EMAIL_ADDRESS>I can help you with that error and maybe provide a better error message for the future.

The value error should be fixed in the latest master version.

@tmbo sorry for not having been very responsive. I made some tries and found that the problem is having a space at the beginning of an entity.
Example
      {
        "text": "Umsätze von heute und letzter Woche",
        "intent": "transactions",
        "entities": [
          {
            "start": 7,
            "end": 35,
            "value": " von heute und letzter Woche",
            "entity": "datetime"
          }
        ]
      }

Setting the start at 8 and removing the space at the beginning of the entity solved the problem

Glad you found the issue. I improved the error message which should allow faster debugging of such training data issues.


Bot not working for link (MP4)

Posted a video as an attachment and it worked fine
some else posted a video as a link (mp4) didn't work

my settings
react.txt

i assume it have something to do with the file name

i assume it have something to do with the file name

Can you check if you have the lastest version of the code?
There it should look for the embed type and not the file name.
If it is the latest version then try posting the link again, maybe it lagged and thats why it wasn't detected.
You can increase the sleep delay in the code (the seq 0 3) by setting it to seq 0 4 or 0 5, which makes the code slower but more consistent too.


GSP-669: Feature Lifecycle

This GSP has been approved, please update the tracking issue.
Most of them has been done, only the changes about https://github.com/beyondstorage/go-service-azblob/commit/14ce73dba0c1d00dd31c1ad86fbe42e0677528a8


fix(extension): check for empty mountpoints

Fix #116
Fix #163

:tada: This PR is included in version 3.23.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Examples not working, wrong paths. Worth to invest anyway?

Hi I want to try your software but I can't. Not online because unpkg.com cannot be reached and the current source doesn't work either because it wants to use a directory that doesn't exist: 'dist' I have used the timeline example.
What I want to know is this, does it provide timing information when processing a wav file. I need this to translate a PWM signal to binary with timing information.

unpkg.com cannot be reached

I think it was a temporary glitch, seems to work fine now.

current source doesn't work either because it wants to use a directory that doesn't exist: 'dist' I have used the timeline example.

After you download the source (using git or zip file), follow these instructions: https://github.com/katspaugh/wavesurfer.js#development
Or simply install using npm install wavesurfer.js


Add two additional CLI commands to storybook CLI

Issue: #1804
What I did
Added a passthrough for two pretty essential arguments to the react-native packager
How to test
Add a --root <ABSOLUTE OR RELATIVE PATH> and a --projectRoots <ABSOLUTE OR RELATIVE PATH> to the storybook start command in a react-native project and ensure the paths show up in the packager window.
Is this testable with jest or storyshots?
I didn't see any tests around this CLI tool
Does this need a new example in the kitchen sink apps?
I do not believe so. Probably just some documentation.
Does this need an update to the documentation?
I couldn't find a good spot in the docs for this to go on first pass. I will be sure to update documentation once I know the best place to put this information
NOTE: A very similar PR was opened #1861. I believe both of these could live happily together as #1861 allows you to specify an entire config and this one specifically lets you append to --root and --projectRoots

Can someone else review this as well @storybooks/team ?

Also, I would very much prefer one or the other in terms of this or https://github.com/storybooks/storybook/pull/1861. I think the command line is a simpler approach.

I have not tested haul. I can certainly do that but will likely be in the next few days that I can get that done.

I feel this would be OK to keep based on master and release as a patch, how do you feel about this @shilman ?

@danielduan Yeah I mostly agree as long as there isn't a need for rn-cli.config.js because some configuration cannot be done using argument passing.

I'm ok with merging this and releasing as next patch version. Do you agree @danielduan @shilman ?

@ndelangen yeah this feels like a patch version update

A quick test of haul shows that it works. I don't believe it respects the settings but that would make sense since haul is more configuration based then the react-native packager.

Any indication when this will be released? It's currently a pretty significant problem for our project and from my tests you cannot do npm git dependnecies with scoped packages

@shilman publish patch release soon? ^

@danielduan @k0nserv sorry guys fell behind on my github notifications and only saw this because of @danielduan 's mention on slack. i just published a patch release: https://github.com/storybooks/storybook/releases/tag/v3.2.11


Croatian used for Serbian

This was copy paste Croatian to Serbian. Now its proper Serbian.

This translation was originally added by @limonte in https://github.com/select2/select2/pull/2027.
I personally do not know the differences between the languages, and Google Translate is pulling up semi-valid translations for both of the languages when these strings are used. We do have translations for Croatian set up (select2/i18n/hr), should we maybe migrate some of these back?

Both cyrillic and latin alpabets are correct for Serbian. Cyrillic Alphabet is official alphabet in Serbia, but latin is used more nowadays.
As to me this PR have now much sense. Although typography is bad, @Mrkisha you replaced … (ellipsis) with ... (three dots) which isn't correct.

As for elipsis, I get same thing when I copy elipsis … and when I type 3 dots ....
As for language, Serbian can be written with latin (croatiаn alphabet) but its not Serbian alphabet. Serbian alphabet is called Azbuka (Азбука). The Constitution of R. Serbia, article 10. says Serbian language is written with Cyrillic alphabet (Aзбука).
`

@Mrkisha I respect the Constitution as much as you.
Here's two most popular serbian news portals:

http://www.blic.rs/
http://www.b92.net/

As you can see latin alphabet is using by both. I think about 90% of serbian websites are using latin alphabet.

Blic and B92 are foreign owned! And for real, Blic? Geez.
http://www.rts.rs/ and http://www.politika.rs/ are well respected medias in Serbia. Also official website of the parliament http://www.parlament.gov.rs/ and official website of government http://www.srbija.gov.rs/. Just to mention few.
I'm not going into history of why and how croatian latin was introduced to Serbia.

This is your personal opinion, I respect it. But objectivity isn't on your side:
http://www.alexa.com/topsites/countries/RS
My examples are on 5th and 6th places. Your are 52nd and 56th.

Offtopic: I'll explain you @Mrkisha  why I personally hate, as Russian citizen, cyrillic alphabet, which is pretty same in Serbian and Russian languages.
Arabic numerals and so math came to Europe widely in 15th century. So Newton and other heroes of science could make revolutions one after another.
On the other side, due to cyrillic alphabet and related to it Цифирь Russia got Arabic numerals and math only in 17th century.
So these guys Ćirilo i Metodije beside introducing cyrillic alphabet to us, took at least 200 years of math and science from Russia.

So, now that we've established that there are two variants of the Serbian language (Cyrillic and Latin), we can figure out what the best course of action to take is.
Select2 uses the ISO 639-1 language code to name the language file. In cases where there are variants for a language (pt-PT and pt-BR, for example), we keep the main language file as the dominant language (pt-PT becomes pt) when needed, or separate the two (zh-CN and zh-TW) when there is no dominant language.
In this case, the ISO language code for Serbian is sr. The IANA Language Subtag Registry defines subtags for script variants (Latin is Latn and Cyrillic is Cyrl) which we can use to differentiate between the two. So the existing translation would be sr-Latn and the new translation would be sr-Cyrl. For backwards compatibility, I'd rather keep sr-Latn as sr, just so people don't have to change that on a minor update (even if it might not be the dominant or official script, backwards compatibility is important).

So, if these changes were added as a new language, sr-Cyrl, I think it would be the best compromise of script variants and backwards compatibility.

@kevin-brown :+1:

I could not disagree more! Who are we to decide which alphabet Serbia is using for Serbian language? Serbian Constitution article 10 states that Serbian Cyrillic is Serbian Alphabet. Those who what transliterated version of Serbian can use Croatian translation (same language, different alphabet and they are compatible 1 to 1).

We have two sides looking to paint the words on the shed with the Cyrillic alphabet and the Latin alphabet. The One True Alphabet™ is not something that we're going to argue on this tracker.

Who are we to decide which alphabet Serbia is using for Serbian language?

We aren't. Well, we are because we have to keep select2/i18n/sr as the Latin alphabet for backwards compatibility reasons. It's not something I'm going to budge on, because it's a noticeable breaking change and not a bug.
I'm perfectly fine with a select2/i18n/sr-Cyrl existing for those who need Serbian translations with the Cyrillic alphabet.

same language, different alphabet and they are compatible 1 to 1

I'm going to have to disagree with this part, simply going based off the fact that there is enough of a difference to have them called different languages. And Wikipedia demonstrates a (small) difference.

@kevin-brown I believe that @Mrkisha won't change his mind and so PR, so I made #3943 as discussed here.
btw. @Mrkisha you forgot to replace latin a with cyrilic а in lines 22 and 31 :)

So drupal released today their new major version 8. And lets visit their translations page and lets choose Serbian https://localize.drupal.org/translate/languages/sr/translate?project=drupal&status=1&release=454938. What alphabet is used for translation?

@kevin-brown using wikipedia to prove me wrong is pathetic. Anyone can write anything on wikipedia. Second since you mentioned the language differences, what language do you speak? American, English, Schotish, Welsh, Southafrican, Australian...? Seriously, you do not even know what dialect is, not to mention anything else... Please do not embarrass yourself any more.

@Mrkisha for which language are you native speaker - Serbian or Croatian?

I've closed this off now that https://github.com/select2/select2/issues/3943 has been created.

We have two sides looking to paint the words on the shed with the Cyrillic alphabet and the Latin alphabet. The One True Alphabet™ is not something that we're going to argue on this tracker.


ownCloud autologin with same user credentials

In many cases the mail user can be the same as that of ownCloud.
The plugin for roundcube has some triggers to automatically save user data in the database and not have to put them manually.
They could do something similar to that of rainloop?
example: https://github.com/hypery2k/owncloud/blob/master/roundcube/src/main/php/lib/RoundCubeAuthHelper.class.php

i implemented rainloop in a symfony backend and a Dashboard for users! the user need to login to the personal dashboard. if they want to acces the mail area, they need to login again. Is there a way to autologin with a server token ?


Left of Register with reg:

Please describe your system environment before submitting your Issue.
Is there any way to put a character to the left of the register, similar to "R", "I", or "J" for types of instruction.  For example:
{reg:
[
{bits: 26,  name: 'address'},
{bits: 6,  name: 'opcode'},
],
config:{bits: 32}
}
You could put a J to the left of the register to signify a Jump instruction from the MIPS ISA.

No, it is not implemented.

This request belongs here: https://github.com/drom/bitfield/issues


[TINY] Fix to #7452 - Query : invalid cast for queries with nullable bool and coalesce

Problem was that COALESCE (nullable_bool, 1) results in an int value, but we expect bool instead.
Fix is to add explicit cast around Coalesce that returns boolean in projections. In case of nesting, we only need to do this for the outermost COALESCE.

@anpete @smitpatel

new version up


Cannot exec a2enmod on phpbrew install

I am trying to install php using phpbrew on Debian Wheezy using the following command.
phpbrew install php-5.4.36 +default +mysql +gettext +mcrypt +intl +iconv +ftp +exif +dba +openssl +soap +apxs2=/usr/bin/apxs2 -- --with-libdir=lib/x86_64-linux-gnu --with-gd=shared --enable-gd-native-ttf --with-jpeg-dir=/usr --with-png-dir=/usr --enable-wddx --with-mysql-sock=/var/run/mysqld/mysqld.sock
However the install fails with the following from the log:
Build complete.
Don't forget to run 'make test'.
Installing PHP SAPI module:       apache2handler
Use of uninitialized value $includedir in concatenation (.) or string at (eval 9) line 1.
/usr/share/apache2/build/instdso.sh SH_LIBTOOL='/usr/share/apr-1.0/build/libtool' libphp5.4.36.la /usr/lib/apache2/modules
/usr/share/apr-1.0/build/libtool --mode=install cp libphp5.4.36.la /usr/lib/apache2/modules/
libtool: install: cp .libs/libphp5.4.36.so /usr/lib/apache2/modules/libphp5.4.36.so
libtool: install: cp .libs/libphp5.4.36.lai /usr/lib/apache2/modules/libphp5.4.36.la
libtool: install: warning: remember to run libtool --finish /home/ben/.phpbrew/build/php-5.4.36/libs' chmod 644 /usr/lib/apache2/modules/libphp5.4.36.so [preparing modulephp5' in /etc/apache2/mods-available/php5.load]
Can't exec "a2enmod": No such file or directory at /usr/bin/apxs2 line 558.
'a2enmod php5' failed
make: *** [install-sapi] Error 2

Use of uninitialized value $includedir in concatenation (.) or string at (eval 9) line 1.

looks like some perl script is using uninitialized value.

Not sure if this issue is outside of the scope of phpbrew or not, but do you by chance have any idea on how I might be able to resolve this issue?

I think this is related to Debian, can you run which a2enmod and see if you can execute it?
It should be existing on every Debian linux system.

I ran which a2enmod, but it couldn't find anything (i.e. nothing was returned).

If your system does not provide a2enmod you're probably missing some packages or you need to configure (load the module) by yourself.

I get a similar error
/usr/share/apache2/build/instdso.sh SH_LIBTOOL='/usr/share/apr-1.0/build/libtool' libphp5.4.45.la /usr/lib/apache2/modules
/usr/share/apr-1.0/build/libtool --mode=install cp libphp5.4.45.la /usr/lib/apache2/modules/
libtool: install: cp .libs/libphp5.4.45.so /usr/lib/apache2/modules/libphp5.4.45.so
libtool: install: cp .libs/libphp5.4.45.lai /usr/lib/apache2/modules/libphp5.4.45.la
libtool: install: warning: remember to run `libtool --finish /home/carlton/.phpbrew/build/php-5.4.45/libs'
chmod 644 /usr/lib/apache2/modules/libphp5.4.45.so
[preparing module `php5' in /etc/apache2/mods-available/php5.load]
Can't exec "a2enmod": No such file or directory at /usr/bin/apxs2 line 555.
'a2enmod php5' failed

Version
me@debian:/home/me$  lsb_release -da
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 6.0.10 (squeeze)
Release:	6.0.10
Codename:	squeeze

I can see the location of a2enmod when running sudo which a2enmod but not as my normal user.
Is there an additional step that needs to be done so that a2enmod can be found & executed?

The issue is outdated and closed now.


Expose image data pixel array as a Uint8ClampedArray

Seems to work with the latest commit on https://github.com/jacereda/purescript-arraybuffer.

This should no longer be an issue, since we have:
type ImageData = { width :: Int, height :: Int, data :: Uint8ClampedArray }


Fix for Qt asks access for Contacts and Calendar in MAC

Qtum Qt asks access for Contacts and Calendar when trying to choose a custom datadir at first launch.
The proposed fix is to update the code to use the non native folder selection dialog.

Replaced by https://github.com/qtumproject/qtum/pull/754


How does goenv handle $GOPATH?

Does goenv set it based on currently selected version or just set it to the same location for all versions?

@athrunsun goenv combined with go-build (by default it's so) unsets GOPATH and GOHOME (ref: https://github.com/syndbg/goenv/blob/master/plugins/go-build/bin/go-build#L964) during install or uninstall (e.g goenv install 1.7.4, goenv uninstall 1.6).
tl;dr You must set GOPATH by yourself.

@athrunsun goenv combined with go-build (by default it's so) unsets GOPATH and GOHOME (ref: https://github.com/syndbg/goenv/blob/master/plugins/go-build/bin/go-build#L964) during install or uninstall (e.g goenv install 1.7.4, goenv uninstall 1.6).
tl;dr You must set GOPATH by yourself.

Thanks for your reply, @syndbg !
Here is another question. From my understanding (please correct me if I understand anything wrong):

python: we cannot install multiple versions of a package using pip, different projects share the same set of packages unless using virtualenv or pyenv.
ruby: we can have multiple versions of a gem, different projects share the same set of gems unless using rvm or rbenv.
nodejs: we can install multiple versions of a package using npm, but usually we will have project specific dependencies in a sub folder node_modules.

Whose behavior is the closest to golang?

By default Go falls into
python: we cannot install multiple versions of a package using pip, different projects share the same set of packages unless using virtualenv or pyenv.

But I highly recommend using https://github.com/kardianos/govendor or https://github.com/tools/godep to get the best (imo) dependency management behavior that is:
nodejs: we can install multiple versions of a package using npm, but usually we will have project specific dependencies in a sub folder node_modules.

@syndbg Thank you for your instructions! I'll try your recommendations!

As a newcomer to Go, I fell into the exact situation you outlined above (fresh goenv install, fresh install of 1.7.4, try to install something with go get and find out my $GOPATH isn't set). So where should I point GOPATH?

@kevbo $HOME/gocode is my preference. Note that in Golang 1.8 there's a default $GOPATH. https://beta.golang.org/doc/go1.8#gopath

@syndbg Thanks for answering that remedial question. Now I have another, more focused on goenv:
If most Golang projects are checked out/installed to GOPATH, how do you leverage goenv to pin a version of Golang to a specific project?
Apologies if that doesn't make sense or if I'm asking the wrong question. I might be looking for too much parity between Golang and Python, which I'm more familiar with. In Python, I use pyenv to keep multiple versions around. Then I'm able to pin a specific Python version to a project by dropping a .python-version file into the repo root. Finally, I'd create a virtualenv inside that directory for near-complete isolation for everything else.

@kevbo you can basically do the same thing:

Install your golang versions with goenv
Set $GOPATH to the default of ~/go.
Always create repos within your ~/go path in the normal golang directory structure.
Create a .go-version in your repo to specify the golang version to use.
Use govendor to install deps to a vendor directory within your repository.

Add the following to your repo's gitignore:vendor/*
!vendor/vendor.json

New versions of golang will pick this up as of 1.6.0. 1.5.0 will need export GO15VENDOREXPERIMENT=1. Older versions will need some love...


Show USD amount for requests

why requests but not on the main activity view?

talked IRL, could make it so the amount display switcher up top also changes how amounts are displayed


convert tool error when convert caffe model

liuhao@umaster:~/OpenSource/Anakin/tools/external_converter_v2$ python converter.py
Traceback (most recent call last):
File "converter.py", line 16, in
from parser import *
File "/home/liuhao/OpenSource/Anakin/tools/external_converter_v2/parser/init.py", line 8, in
from graph import *
File "/home/liuhao/OpenSource/Anakin/tools/external_converter_v2/parser/graph.py", line 6, in
from proto import *
File "/home/liuhao/OpenSource/Anakin/tools/external_converter_v2/parser/proto/init.py", line 5, in
from graph_pb2 import *
File "/home/liuhao/OpenSource/Anakin/tools/external_converter_v2/parser/proto/graph_pb2.py", line 15, in
import node_pb2 as node__pb2
File "/home/liuhao/OpenSource/Anakin/tools/external_converter_v2/parser/proto/node_pb2.py", line 15, in
import operator_pb2 as operator__pb2
File "/home/liuhao/OpenSource/Anakin/tools/external_converter_v2/parser/proto/operator_pb2.py", line 22, in
serialized_pb=_b('\n\x0eoperator.proto"e\n\x07OpProto\x12\x0c\n\x04name\x18\x01 \x01(\t\x12\x16\n\x0eis_commutative\x18\x02 \x01(\x08\x12\x0e\n\x06in_num\x18\x03 \x01(\x05\x12\x0f\n\x07out_num\x18\x04 \x01(\x05\x12\x13\n\x0b\x64\x65scription\x18\x05 \x01(\tb\x06proto3')
TypeError: new() got an unexpected keyword argument 'serialized_options'

and the config file is:
OPTIONS:
Framework: CAFFE
SavePath: ./output
ResultName: resnet50
Config:
LaunchBoard: ON
Server:
ip: <IP_ADDRESS>
port: 8888
OptimizedGraph:
enable: OFF
path: ./output/resnet50.anakin.bin
LOGGER:
LogToPath: ./log/
WithColor: ON
TARGET:
CAFFE:
# path to proto files
ProtoPaths:
- /home/liuhao/framework/caffe/src/caffe/proto/caffe.proto
PrototxtPath: ./ResNet-50-deploy.prototxt
ModelPath: ./ResNet-50-model.caffemodel

seems protobuf need to be 3.4.0. After us protobuf 3.4, the error disappeared, but there will be another problem about caffe_yolo_pb, make some change to script will ignore this.

@HaoLiuHust hi, you convert tool successs?


Added Russian language

Please, read contribution guide: https://github.com/denysdovhan/purifier-card/blob/master/CONTRIBUTING.md#how-to-add-translation
You must include your translation in src/localize.js

Sorry, did not notice, fixed

Thanks for contributing!


[Rule] No implicit empty string arguments

Inspired by #1144, I think it would be nice to have a rule to prevent passing implicit empty strings as arguments like this:
*** Test Cases ***
My Test
    VAR    &{dict}    key1=    key2=actual value
    Log    ${dict}

That makes a lot of sense... In fact, it makes so much sense that it didn't even occur to me that people might do this!
What do you think of the rule names:

undefined-argument-value
undefined-argument
undefined-value
undefined-argument-assignment

The rationale behind this and the suggested fix would be the same as that in #1144

Yeah those sound good to me.


Created Grafana Dashboard and related charts/manifests

Closes #150

@ptx96 may I ask you to share also a screenshot of the dashboard?
Would be great having it in the documentation.


Issues using the lib

I'm having a tough time using this lib. I've copied pybind_json.hpp into my include dir as I don't have access to conda.
Then my main file is:
#include <pybind_json.hpp>

int main()
{
    nlohmann::json j;
    j["a"] = 1;
    j["b"] = 2;

    py::dict d;
    d["a"] = 1;
    d["b"] = 2;

    py::dict d2 = j;
    nlohmann::json j2 = d;
}

The last two lines give compilation errors of:
"no suitable user-defined conversion from "nlohmann::json" to "pybind11::dict" exists"
"no suitable user-defined conversion from "pybind11::dict" to "nlohmann::json" exists"
respectively. So I am unable to do any conversion. I must be missing something simple. I am able to compile the code with the last 2 lines commented out so I know the nlohmann and pybind11 libs are properly being used.

Casting py::dict as an object works (which makes sense), but I am trying to repeat the example. Should I just be casting or change the serializer return type?

Thanks for the report, it is possible that the README example does not compile... I don't recall testing it. I guess you can change the serializer return type if you want.

Maybe we should add overloads to the serializer that would do the casting automatically.
Also you can write your code this way (this is how it's written in tests):
py::object obj = py::dict("number"_a=1234, "hello"_a="world");
nl::json j = obj;

I fixed the README example, thanks again for reporting :)


Modal dialog prevents the screen from being moved

When a modal dialog is shown the screen a user cannot move the UI or redock it etc. some modals take a long time and the UX is not the best. redesign/rethink

fixed


Box props being passed to underlying dom

When passing width or order prop to Box component they're being passed to underlying html.
Dependencies:

styled-components 3.4.9
styled-system 3.1.6
@rebass/grid 6.0.0-3

Expected:
width and order props are cleaned from the underlying html.
Screenshot:

For v6, this is the expected behavior, see https://github.com/rebassjs/rebass/pull/474 for some of the rationale behind this change

@jxnblk is there any proper temporary workaround for removing these props from the DOM?

Also wondering about @karolisgrinkevicius's question.


When I am running that qt5 default command on ubuntu terminal it is showing qt 5 is not available

try sudo apt-get install qtbase5-dev qtchooser qt5-qmake qtbase5-dev-tools
if you are on ubuntu 21.04, else let me know your ubuntu version.
reference: https://askubuntu.com/questions/1335184/qt5-default-not-in-ubuntu-21-04


Setup dependencies

Steps to reproduce the behaviour
Using 'yarn':

to start a new project / initialize package.json: yarn init
to add a:

dependency: yarn add <packagename>
dev. dependency: yarn add --dev <packagename>

The c772b97 commit has incorrect message,
corrected at bdbf2af


Correctly handle urls starting with "-"

Fixes case https://github.com/rg3/youtube-dl/blob/master/README.md#how-do-i-download-a-video-starting-with-a---

Thanks!


Enable BlazorAppScheme to be a configurable value

Looking to be able to override the PhotinoWebViewManager.BlazorAppScheme value to be file (and in turn affecting PhotinoWebViewManager.AppBaseUri) which will enable file system access from the webview process, not limited to the wwwroot published folder for an offline desktop application.

@trevjacobson Can you provide some more information on your use case? Usually, direct access from a browser engine is not permitted due to the sandbox they run in. Loading the index.html using the file:// scheme is not supported.
You can use C# to open and manipulate files and select files for this purpose using the typical JS APIs.
Using the file scheme as you (seem) to propose could also create issues with differences in file path structure across operating systems. E.g. Windows using something like C:\.... while UNIX type OSs are using /home/username/...
https://stackoverflow.com/a/12711835
https://stackoverflow.com/a/372333


Improve class initialization

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area tests

/area proposals

/area CI

What this PR does / why we need it:
This PR refactors member variables initialization so that they are done in the constructor initialization list.
They were detected by cppcheck.
Which issue(s) this PR fixes:

Fixes #3073
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: sgaist
Once this PR has been reviewed and has the lgtm label, please assign lucaguerra for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: bba4873503e26fad06b18b25ae3b2f332aa9f89e

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: incertum, sgaist
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/unhold

New changes are detected. LGTM label has been removed.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

d034dd5 refactor: make falco_exception a std::runtime_error
99b7518 fix(c++): move trivial initializations to declaration site

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.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

d034dd5 refactor: make falco_exception a std::runtime_error
99b7518 fix(c++): move trivial initializations to declaration site
99b6571 fix(c++): add missing member initialisation to grpc server

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.

/milestone 0.38.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, incertum, sgaist
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 11f862a17bdc9e18fd9eb9c62a37538613199ccf

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash: 9df54a0b87e02fa96709e32dd3608a7d3f3525d0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, incertum, sgaist
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, incertum, sgaist
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Unknown authentication strategy "azure-active-directory" Error

I've cloned the repo, created azure app, configured config/activeDirectory.json file as instructed and started the application via docker-compose. Unknown authentication strategy "azure-active-directory" appears once the browse to the http://localhost:3000

can you instruct me to solve it?


A left shift after an as-cast can mess up paren matching

If I create a file containing only the following code (which does compile without warnings or errors), the closing brace to the function is not recognized by the parenthesis-matcher (although the closing parenthesis around the bitshift is):
fn test(x: f64) -> u64 {
    x as u64 * (1 << 3)
}

Interestingly, if I put the x as u64 in parentheses, it works just fine:
fn test(x: f64) -> u64 {
    (x as u64) * (1 << 3)
}

This occurs if such an expression is present anywhere in the function.

M-x describe-char on the first < shows:
               syntax: (>	which means: open, matches >

So I think rust-is-lt-char-operator is getting the wrong answer.  In particular probably from
       ;; If we're looking back at an identifier, this depends on whether
       ;; the identifier is part of an expression or a type
       ((rust-looking-back-ident)
        (backward-sexp)
        (or
         ;; The special types can't take type param lists, so a < after one is
         ;; always an operator
         (looking-at rust-re-special-types)

         (rust-is-in-expression-context 'ident)))

Haven't dug deeper yet.

This code breaks paren matching too:
   fn add(dr: u16, sr1: u16, sr2: u16) -> u16 {
        (Op::add << 12) | ((dr & 0b111) << 9) | ((sr1 & 0b111) << 6) | (1 << 5) | sr2 & 0b111
    }

I found that this problem can be temporarily solved by (setq rust-match-angle-brackets nil)
But this will cause <> to fail to match


feat: configured dark theme for wallet page

Pull Request Title
This PR adds darkmode support to wallet page. Also improving the UI with simple and minimal animations.
Issue Number: Fixes #220  (issue number)
🛠 Type of Change

[ ] 🐛 Bug fix
[X] ✨ New feature
[ ] 🚨 Breaking change
[ ] 📚 Documentation update

✅ Checklist
To ensure a smooth review process, please check off each item as you complete it:

[X] Code Style: My code adheres to the project’s style guidelines.
[X] Self-Review: I have reviewed my own code and made improvements.
[X] Comments: I’ve added comments to explain complex or non-obvious parts of the code.
[ ] Documentation: I’ve updated the documentation to reflect my changes.
[ ] Warnings: My changes introduce no new warnings.
[ ] Tests: I’ve added tests to verify that my changes work as expected.
[ ] Passes Locally: All new and existing unit tests pass on my local machine.

Here is the video for ui reference
https://github.com/user-attachments/assets/dc1de012-1f08-4e02-b8bf-fdbf6afd2b61

@cb7chaitanya @Rudra-Sankha-Sinhamahapatra  will you please review this pr.
The question that may arise for layout file is that i have removed duplicate lines and added themeprovider to it

Already someone was working on the issue


feat: implement commit-msg hook (#45)

This pull request implements a commit-msg hook to verify that commit messages are following the conventional commit best practices. It is calling a new target git-verify that is also capable of validate a full commit log as well as the list of new commits provided by a branch.
The pull request also restructures the organization and availability of some default variables in Makefile.base to improve maintainability, customizabillity, and testability.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 9 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
294

Relevant Lines:
294

💛  - Coveralls


Use findByType instead if getByType

👍

Coverage remained the same at 78.929% when pulling a46ebcb1c64f2b053b837ad3a5131d6ba21d4d1e on 14/use-findByType into bcaed7059c218ca31666545a05e106750e2eb102 on master.


[WEB-902] [turbopack] API routes in nextjs 13.3 stopped working

What version of Turborepo are you using?
not sure, whatever comes with next 13.3.0
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Mac
Describe the Bug
GET http://localhost:3000/api/listings
error rendering: TypeError: Cannot read properties of undefined (reading 'middleware_edge')

Debug info:
- error rendering: TypeError: Cannot read properties of undefined (reading 'middleware_edge')
      [at file:///Users/kain/src/videoexplorer/frontend/node_modules/.pnpm/next@13.3.0_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/server/web/sandbox/sandbox.js:67:50]
      [at process.processTicksAndRejections (node:internal/process/task_queues:95:5)]
      [at async Module.runEdgeFunction (file:///Users/kain/src/videoexplorer/frontend/.next/server/app/chunks/ssr/[next]_internal_edge.ts_4acfa5._.js:28:20)]
      [at async (file:///Users/kain/src/videoexplorer/frontend/.next/server/app/chunks/ssr/_c634e4._.js:92:5)]
      [at async endOperation (file:///Users/kain/src/videoexplorer/frontend/.next/server/app/chunks/ssr/[next]_internal_08c02e._.js:152:13)]
      [at async (file:///Users/kain/src/videoexplorer/frontend/.next/server/app/chunks/ssr/[next]_internal_08c02e._.js:100:13)]

status: exit status: 1

Expected Behavior
without --turbo the endpoint works.
To Reproduce

Reproduction Repo
No response
WEB-902

Experiencing similar but different issue TypeError: Cannot read properties of undefined (reading 'middleware_middleware') on the latest 13.1 canary 4.
Full error log:
`Debug info:

Execution of get_from_source failed
Execution of resolve_source_request failed
Execution of NextRouterContentSource::get failed
error during Next.js routing for /
TypeError: Cannot read properties of undefined (reading 'middleware_middleware')
[at file://C:\Users\fream\Desktop\BOBBLL\bobbll.crm.next\node_modules.pnpm\next@13.3.1-canary.4_react-dom@18.2.0_react@18.2.0\node_modules\next\dist\server\web\sandbox\sandbox.js:67:50]
[at async TurbopackDevServerProxy.runMiddleware (file://C:\Users\fream\Desktop\BOBBLL\bobbll.crm.next\node_modules.pnpm\next@13.3.1-canary.4_react-dom@18.2.0_react@18.2.0\node_modules\next\dist\server\next-server.js:1409:24)]
[at async TurbopackDevServerProxy.runMiddleware (file://C:\Users\fream\Desktop\BOBBLL\bobbll.crm.next\node_modules.pnpm\next@13.3.1-canary.4_react-dom@18.2.0_react@18.2.0\node_modules\next\dist\server\dev\next-dev-server.js:774:28)]
[at async Object.fn (file://C:\Users\fream\Desktop\BOBBLL\bobbll.crm.next\node_modules.pnpm\next@13.3.1-canary.4_react-dom@18.2.0_react@18.2.0\node_modules\next\dist\server\next-server.js:1552:42)]
[at async Router.execute (file://C:\Users\fream\Desktop\BOBBLL\bobbll.crm.next\node_modules.pnpm\next@13.3.1-canary.4_react-dom@18.2.0_react@18.2.0\node_modules\next\dist\server\router.js:297:32)]
[at async resolveRoute (file://C:\Users\fream\Desktop\BOBBLL\bobbll.crm.next\node_modules.pnpm\next@13.3.1-canary.4_react-dom@18.2.0_react@18.2.0\node_modules\next\dist\server\lib\route-resolver.js:171:9)]
[at async Module.route (file://C:\Users\fream\Desktop\BOBBLL\bobbll.crm.next.next\build\chunks\router[next]entry_router.ts_3d8e62..js:44:28)]
[at async Module.run (file://C:\Users\fream\Desktop\BOBBLL\bobbll.crm.next.next\build\chunks\router[turbopack-node]ipc_evaluate.ts_d53664..js:177:39)]`

meanwhile I did some progress on the app and retried turbo, I'm using have aws sdk v3 in my route handlers and I have this:
error rendering: Error: Cannot find module 'os'

Debug info:
- error rendering: Error: Cannot find module 'os'
      [at file:///app/.next/server/app/edge/chunks/app_api_listings_[[...key]]_route.ts_next-edge-bootstrap.ts_fc1fbe._.js]
      [at file:///app/.next/server/app/edge/chunks/node_modules_c5accc._.js:36145:15]
      [at file:///app/.next/server/app/edge/chunks/node_modules_c5accc._.js:36148:3]
      [at<EMAIL_ADDRESS>(ecmascript) (file:///app/.next/server/app/edge/chunks/node_modules_c5accc._.js:36166:3)]
      [at file:///app/.next/server/app/edge/chunks/app_api_listings_[[...key]]_route.ts_next-edge-bootstrap.ts_fc1fbe._.js:412:21]
      [at runModuleExecutionHooks (file:///app/.next/server/app/edge/chunks/app_api_listings_[[...key]]_route.ts_next-edge-bootstrap.ts_fc1fbe._.js:456:3)]
      [at instantiateModule (file:///app/.next/server/app/edge/chunks/app_api_listings_[[...key]]_route.ts_next-edge-bootstrap.ts_fc1fbe._.js:410:3)]
      [at getOrInstantiateModuleFromParent (file:///app/.next/server/app/edge/chunks/app_api_listings_[[...key]]_route.ts_next-edge-bootstrap.ts_fc1fbe._.js:498:10)]
      [at esmImport (file:///app/.next/server/app/edge/chunks/app_api_listings_[[...key]]_route.ts_next-edge-bootstrap.ts_fc1fbe._.js:181:18)]
      [at file:///app/.next/server/app/edge/chunks/node_modules_c5accc._.js:36363:175]
      [at<EMAIL_ADDRESS>(ecmascript) (file:///app/.next/server/app/edge/chunks/node_modules_c5accc._.js:36381:3)]

status: signal: 9 (SIGKILL)

https://github.com/aws/aws-sdk-js-v3/blob/1e806ba3f4a83c9e3eb0b41a3a7092da93826b8f/packages/shared-ini-file-loader/src/getHomeDir.ts#L1
not sure why

This may be fixed with https://github.com/vercel/next.js/pull/48791
it was just merged, so it's going to be in the next canary version (tonight)

@masterkain can you confirm if you're still seeing this with the latest canary?

@padmaia I got some other problems but I tried with a new project and routes appear to work, thanks

Okay, closing this ticket out then. If you're still experiencing issues with a more complex route handler, let us know.


Add INFO logging to generic_thermostat component for keep-alive turn …

Description:
Improve INFO level logging for the generic_thermostat when Keep-Alive is being used.
When the keep alive config option is present the turn on/off service is called on the heater entity every time interval specified in keep alive. This created some odd behaviour in my particualr case.
Currently when the generic_thermostat turns on/off the heater/HVAC a log message is published but nothing is sent when Keep-Alive is called. This pull request adds some INFO level logging to make these service calls more obvious and help an end user debug/understand the functionality of keep alive.
Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the development checklist

Hi @nemccarthy,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

@nemccarthy please format your code with black e.g. black {source_file_or_directory} :)

Thanks @springstan - have done that


Error when opening the sample project, "Type mismatch. Required: Observer<in RoomViewEvent!> Found: (RoomViewEvent) → Unit"

hey, i'm a bit new to android development and just trying twilio, there is an error in RoomActivity line 233, in method setupThumbnailRecyclerView() when i try to open the project. can someone give me a hand in this error? thank you

adding "Observer" before "{ viewEvent.." solved my issued, sorry and thank you


fix nuget issue, set nuget to latest release

nuget was preview and not available. Used latest release of nuget

@Calnoras did you check if your changes are fully working? The preview version of Automapper was needed because the latest release was not supporting .NET 7.

@Light4Code
I did not encounter any errors when using the latest nuget release instead of the preview.
The automapper nuget was upgraded to netstandard2.1 since nuget Version 11.
So normaly i would expect it to work with net7.0 without a hitch.


Documentation for OS version in TFMs should have the "earliest supported OS version"

Type of issue
Missing information
Description
See details in https://github.com/dotnet/sdk/issues/30103#issuecomment-2051870339
Starting with .NET SDK 9 Preview 3, the earliest bindings are supported.
For example, net8.0-ios17.0 will remain supported, regardless of whether new versions are released
So, it would be good to list the versions for every platform and for every TFM (and document that this works only with .NET 9 SDK Preview 3 and later)
Also, as the "earliest supported version" can change between previews and stable if I understand correctly, but once it's in stable, it won't change. So that would be good to note.
So, an extra column is needed in the table, which will be "earliest supported version". Then, in times of previews, there should be a note between parentheses that this version will change in stable SDK.
cc @rolfbjarne in case I got any info wrong.
Page URL
https://learn.microsoft.com/en-us/dotnet/standard/frameworks
Content source URL
https://github.com/dotnet/docs/blob/main/docs/standard/frameworks.md
Document Version Independent Id
4352856b-5ccf-8e62-48b4-3151340cac56
Article author
@gewarren
Metadata

ID: 49205137-dcec-a120-e5e7-1b9c79cdafbb
Service: dotnet
Sub-service: standard-library

So, it would be good to list the versions for every platform and for every TFM (and document that this works only with .NET 9 SDK Preview 3 and later)

If everything goes as planned, it will also work in a future service update to .NET 8.
In fact I'd hold off on changing anything until this change has been released to stable.


Customize default behaviour when command not found

Is there a way to customize or define the behaviour of nomnom when a command is not found? For Example I want to print my own message or want to look for scripts in a particular directory.

https://github.com/scopely/sugar/blob/002f7598bfc72b15c6615d64022a56ed41567310/src/sugar.js#L426-L438
This is the horror I've had to face to deal with this.


Remove files that should be excluded

In a Jekyll project, some files should only exist locally and not in version control
Delete vendor/
Delete .DS_Store
Delete _site

And I recommend creating a .gitignore file like this
https://github.com/MichaelCurrin/jekyll-blog-demo/blob/master/.gitignore

Thank you for the nice suggestions!


Standalone pathfinder.sql merged in to /setup

At the moment new set-ups have to remember to bring in the standalone pathfinder.sql from the repo via a msyql import -- can we get this static data chucked in to /setup?

The first  SQL query was moved to /setup route
Bootstrapping for system_neightbour table was moved to /setup route.
I added a new UI-panel for that.

This trunkates system_neightbour table and (re)build the index for all systems in EVE (>5k).
There are two reasons why you should use this new function:

When you set up your database for the first time.
When CCP added or removed solarsystems from EVE

Important: The Index build function takes some seconds. Be sure to increase max_execution_time to ~30s!

... and the remaining SQL data can now be im/exported as *.csv files
Tables with static data:

wormhole - wormhole data (lifetime, ...)
constellation_wormhole - static wormhole data for systems [by constellationId]
system_wormhole - static wormhole data for a specific system (shattered wormholes) [by systemId]

Dump files for those tables are located under export/csv. I added new im/export buttons to /setup page. They can be used es an alternative to the pathfinder.sql import for those tables.

Important: The table/column structure has changed for those tables! If you have a private installation running, you have to rebuild (drop/create) those 3 tables with the upcoming version. ->  clear cache afterwards!


Bug: Empty message for  list-rcon-channels

[OpenttdDiscord.Infrastructure.Discord.DiscordCommandService] gownokontodupa executed unsuccessfully list-rcon-channels

I fixed it in the past while fighting with other empty message


Social: Add 'say' command

We want the ability to say things to people that are in the same room as us. By defaults, we should be able to say things in common:
$ say hey there
> Hrothgar says "hey there"

If you know a language, you should be able to speak in it:
language goblin
> You are now speaking in goblin
say hey there
> Hrothgar says in goblin "hey there"

But if you don't know goblin:
> Hrothgar says "difa yasl"

We can do a skill check per word and replace it with some phonetic differences based on the language. So we'll need a 'language class' that interprets phrases based on folks skill checks.
(Should be fun!)
Note that your language should impact all things that you say or shout.
Everyone know 'common' by default. We can get better at learning languages by hearing them spoken (slow) and an INT check or by study with a book, but that part can come later.

This is basically in place at this point. There's rudimentary language support, although for the most part everyone is running around speaking the equivalent of 'common'.

Closing.


Short solution needed: "Install NJS (Nginx Javascript) module" (nginx)

Please help us write most modern and shortest code solution for this issue:
Install NJS (Nginx Javascript) module (technology: nginx)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

Install NJS (Nginx Javascript) module


[Bug]: First graph in new project is always invisible

What happened?
First graph is not shown in "Graphs" unless I save the project
What was the expected functionality?
First graph is immediately shown in "Graphs"
Describe your environment
MacOS 14.2.1
Relevant log output
No response
Relevant screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

It's not really invisible it's just... not saved anywhere yet. Kind of intentional, but should probably still be changed.

I got confused by this as well when I got started.


Can you add feature

Thank you for creating such a clean and stable comfyui custom node—it’s been a lot of fun to use! Would it be possible to add an extra feature, like a preset prompt list? Something like:

Combine Image1 and Image2
Replace the hair color of the man in Image1

There are times when we’re not sure what to do with the node, and having a few preset ideas would be really helpful. We appreciate your work and look forward to any updates you can make!

Add preset prompt list on new version, update the code and give me some feedback


WIP: Use environment.yml in CIs

This should be nearly equivalent to what we already do, just with fewer lines and using an environment.yml file. This moves us toward more easily testing things like #4523 when we think Py3k might actually be ready for primetime.

Codecov Report

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

@@             Coverage Diff             @@
##           master    #4570       +/-   ##
===========================================
- Coverage   87.04%   51.17%   -35.88%
===========================================
  Files         349      349
  Lines       65692    65692
  Branches    10191    10191
===========================================
- Hits        57184    33619    -23565
- Misses       5629    30029    +24400
+ Partials     2879     2044      -835

looks like travis did not run

is this now working?
but only on 2.7 right?

Yes 2.7, OSX and Linux, probably not on CircleCI (some weird error), and AppVeyor also causes crashes. But it's some progress at least. I'll revert CircleCI so we can merge and iterate further.


Unhandled promise rejection

I've used the validation plugin successfully previously but I'm encountering an issue on a different view in the same application.  Below is the relevant portion of my view-model:
import {inject} from "aurelia-framework";
import {Validation} from "aurelia-validation";

@inject(Validation)
export class Create {

  constructor(validation){
    this.validation = validation.on(this)
      .ensure('content.curriculumTitle', (config) => {config.computedFrom('type')} )
      .if(() => { return this.type === '01'; })
        .isNotEmpty()
      .endIf();
  }

  save() {
    this.validation.validate()
      .then(()=> {
        console.log("validated");
      });
  }

My view:
<form role="form" validate.bind="validation">

                Enter the curriculum title
                <input type="text" value.bind="content.curriculumTitle" class="form-control"
                       id="curriculumTitle" placeholder="Title">

It works when content.curriculumTitle has a value but when it doesn't, I get this error:
Unhandled promise rejection
	ValidationResult {isValid: false, properties: Object}
		isValid: false
		properties: Object
		content.curriculumTitle: ValidationResultProperty
			failingRule: "isRequired"
			group: ValidationResult
			isValid: false
			properties: Object
			content.curriculumTitle: ValidationResultProperty
				failingRule: "isRequired"
				group: ValidationResult
				isDirty: true
				isValid: false
				latestValue: ""
				message: "is required"
				onValidateCallbacks: Array[0]
				__proto__: ValidationResultProperty
			__proto__: Object
		__proto__: ValidationResult
		isDirty: true
		isValid: false
		latestValue: ""
		message: "is required"
		onValidateCallbacks: Array[0]
		__proto__: ValidationResultProperty
	__proto__: Object
__proto__: ValidationResult
(anonymous function) @ shim.min.js:1786

shim.min.js:1786 Unhandled promise rejection ValidationResult {isValid: false, properties: Object}(anonymous function) @ shim.min.js:1786

I've also tried it without the if and I get the same error.  I can't see how this different from the view that is working.  Any advice?
Thanks,
Ross

Shouldn't you just add a catch handler?
The promise will be rejected, if the validation result is invalid
save() {
  this.validation.validate()
      .then(()=> {
         console.log("validated");
      })
      .catch(() => {});
}

Yes, promises are new to me and I've got a better handle on them since I posted this.  I still have an issue that the validation error message isn't displayed.  I think that   may have to do with the fact that the form is hidden when the view loads and is shown based on the user's choices.  I'm still playing with it.
Thanks for your response.

As mentioned you need to add a catch for when the promise fails.  As far as why the view isn't showing the error I'm not sure but if you figure it out please feel free to open a separate issue.


pykdtree installed with pip on macOS can't find OpenMP

❯ ipython                                                                                (pykdtree)
Python 3.11.10 | packaged by conda-forge | (main, Sep 10 2024, 10:57:35) [Clang 17.0.6 ]
Type 'copyright', 'credits' or 'license' for more information
IPython 8.27.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import pykdtree
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
File /opt/homebrew/Caskroom/miniforge/base/envs/pykdtree/lib/python3.11/site-packages/pykdtree/__init__.py:4
      3 try:
----> 4     from . import kdtree
      5 except ImportError as err:

ImportError: dlopen(/opt/homebrew/Caskroom/miniforge/base/envs/pykdtree/lib/python3.11/site-packages/pykdtree/kdtree.cpython-311-darwin.so, 0x0002): symbol not found in flat namespace '___kmpc_for_static_fini'

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

ImportError                               Traceback (most recent call last)
Cell In[1], line 1
----> 1 import pykdtree

File /opt/homebrew/Caskroom/miniforge/base/envs/pykdtree/lib/python3.11/site-packages/pykdtree/__init__.py:6
      4     from . import kdtree
      5 except ImportError as err:
----> 6     raise ImportError(
      7         "Pykdtree failed to import its C extension. This usually means it "
      8         "was built with OpenMP (C-level parallelization library) support but "
      9         "could not find it on your system. To enable better performance "
     10         "OpenMP must be installed (ex. ``brew install omp`` on Mac with "
     11         "HomeBrew). Otherwise, try installing Pykdtree from source (ex. "
     12         "``pip install --no-binary pykdtree --force-install pykdtree``)."
     13     ) from err

ImportError: Pykdtree failed to import its C extension. This usually means it was built with OpenMP (C-level parallelization library) support but could not find it on your system. To enable better performance OpenMP must be installed (ex. ``brew install omp`` on Mac with HomeBrew). Otherwise, try installing Pykdtree from source (ex. ``pip install --no-binary pykdtree --force-install pykdtree``).

In [2]:

pykdtree is working when installed with mamba:
mamba install pykdtree

just recommending mamba on macOS for now


Add epoch and validation loss to checkpoint

Update the checkpoint filename to include the epoch value and validation loss value so we sample against a checkpoint with a lower (or not!) validation loss. From Karpathy's char-rnn implementation.

Looks good. I think versioning checkpoints on more parameters is a sensible idea. Quick question - why do we need to store both the epoch value, and iteration number? Can epochs not be derived from iteration count?


Spaces in YAML translation files with nuxt srcDir config

Environment

Operating System: Darwin
Node Version:     v18.13.0
Nuxt Version:     3.3.1
Nitro Version:    2.3.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      runtimeConfig, app, ssr, srcDir, modules, vite, nitro, googleFonts, i18n, content, plausible
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-rg81ao
https://stackblitz.com/edit/github-rg81ao-u5wrhd (without srcDir)
Describe the bug
Updating from 8.0.0-beta.10-27985006.4ff543e to 8.0.0-beta.10-27995650.c1d4ec9 it seems that strings with spaces are no longer accepted in YAML translation files when not wrapped into quotes:
demoStringQuotes: 'String with spaces with quotes'
demoString: String with spaces

Run npm run generate in the reproduction repo on StackBlitz to see how the build fails.
This seems to be related to the nuxt srcDir config. Without this everything works as expected. Compare with npm run generate in the reproduction repo without srcDir.
Additional context
No response
Logs

I did test the recent npm edge releases. 8.0.0-beta.10-27993103.e65d8f2 works. 8.0.0-beta.10-27995650.c1d4ec9 fails.
See: https://github.com/nuxt-modules/i18n/compare/e65d8f2...c1d4ec9

It looks like getLayerLangPaths is missing srcDir in the path:
layers.ts
export const getLayerLangPaths = (nuxt: Nuxt) => {
  return (
    nuxt.options._layers
      .filter(layer => layer.config.i18n?.langDir != null)
      // @ts-ignore
      .map(layer => pathe.resolve(layer.config.rootDir, layer.config.i18n.langDir)) as string[]
  )
}

vs. module.ts
const langPath = isString(options.langDir) ? resolve(nuxt.options.srcDir, options.langDir) : null

But this does not yet solve the issue. When I add srcDir to the resolve method I get another error message from rollup instead:
[vite]: Rollup failed to resolve import "js-cookie" from "virtual:nuxt:~/.nuxt/i18n.internal.mjs".
This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
`build.rollupOptions.external`

Sorry! Looks like I introduced this bug, I have tried it locally and it seems to work when adding srcDir to getLayerLangPaths path resolving.
I'm not sure why the quotes in yaml changes the behaviour though...

Maybe #1964 fixes the problem after all. Running this fix with npm link I got the js-cookie error message (not only in my project, but also in a fresh nuxt project). But when I edit this directly in<EMAIL_ADDRESS>is seems to work. No clue though how this should be related to js-cookie or YAML strings without quotes…


Implement UNLESS CONFLICT ... ELSE

Closes #1639, though there will be some follow-ups.

I found two problems which I think I need to fix to merge this:

[ ] "Bare" uses of ELSE, without being wrapped in a SELECT with a shape, fail to generate a RETURNING clause in the SQL. So like INSERT Person { name := "Phil" } UNLESS CONFLICT ON .name ELSE (UPDATE Person SET { tag := "tag" }); won't work.
[ ] ELSE UPDATE returns the old data, not the newly updated data.

I found two problems which I think I need to fix to merge this:

I think both are related, and are likely because the top-level SELECT wrapper only has the original INSERT in its path_rvar_map, whereas, in reality it should be a UNION over the INSERT and the UPDATE.

I found two problems which I think I need to fix to merge this:

I think both are related, and are likely because the top-level SELECT wrapper only has the original INSERT in its path_rvar_map, whereas, in reality it should be a UNION over the INSERT and the UPDATE.

Looking at the pgsql ast dump, the top-level SELECT has the UNION cte in its path_rvar_map for default::Person in my example. That must not be getting linked up properly with the underlying relations?


Expose a trait to generalize over CsMat variants

This should enable easier and more general APIs.
For instance the vstack, bmat constructors could benefit from this API with an API like
pub fn vstack<N>(mats: AsRef<[CsMatTrait<N>]>)

Turns out this trait won't help the ergonomics of vstack, as callers would need to box their inputs.
Therefore a macro would be more suited for this particular case. A trait remains useful for genericty though.


(Feature-ish request) Hash texture names at compile time?

This would speed things up a bit I think (along with the implementation of this into the TPK tool so you don't have to do it manually in the INI)
dunno how to implement the algorithm into a macro though, but thankfully C++ 11 has our backs and allows us to just write constant expressions which have the same end result as a macro while having more flexibility as you no longer need hacks to do recursion (which has to be fixed length too)
here's a quick matching implementation of it (you can also make a non-constexpr copy of it for runtime computation of hashes, could be useful in giving the different controllers string names in the INI without needing to rewrite much maybe?)
constexpr unsigned int BIN_HASH(const char* string)
{
    char* _string = ((char*)string);
    int result = -1;

    while (*_string != 0) {
        result = result * 0x21 + (unsigned int)(*_string);
        _string = _string + 1;
    }

    return (unsigned int)result;
}

i.e. rather than
#define PS4_DPAD_UP_HASH 0x52FB3FD2

you'd have
#define PS4_DPAD_UP_HASH BIN_HASH("PS4_DPAD_UP")

and it'd expand to 0x52FB3FD2 at compile time

Hmmm that would be useful to do during compile time. Good idea!
I wasn't very familiar with constexpr, to be perfectly honest with you, but I could implement this.
I currently use the ripped bStringHash function but this appears to be the exact same thing anyways, so if you want to, you may submit a PR for this.
Or I'll do it myself, your choice.

it's a version of bStringHash I decompiled with Ghidra then manually cleaned up to remove redundancy

FYI I'm working on making the hash macros use this instead
I'll open a PR once I'm done

Opened pull request at https://github.com/xan1242/NFS-XtendedInput/pull/24

With 0995a41 I'm considering this implemented, closing.

Nuh-uh, after further snooping, constexpr doesn't do its job. Reverting all changes.

Proof here: IDA shows this in GetAppropriateButtonTex function. This is not right and it should NOT happen.

It should look a little something like this

hmmm
that's odd

I'll look into what's causing this and if it can be fixed
otherwise, time to make that macro then :D

I've tested with Carbon but that doesn't have a barring on this, this is the compiler's doing.

There isn't a way to make this a "macro" per se, from what I've read. Maybe I could be wrong, but constexpr is the way to go.

The solution is to convert the hash macros into enums instead. That way it'll actually get processed by the compiler.

I think I fixed it with a dirty hack?

// compile time hasher thanks to C++11
// I tried making a macro version but failed xdddd
// P.S. it causes IntelliSense to freak out in some cases, but it still compiles
static constexpr int __HASH(char* string)
{
	if (string == NULL) // sanity check
		return 0;

	char* _string = ((char*)string);
	int result = -1;

	while (*_string != 0) { // loop through each char until string terminator is reached
		result = result * 0x21 + (unsigned int)(*_string);
		_string = _string + 1;
	}

	return result;
}

template <int V> static constexpr int _HASH = V;

#define HASH(str) _HASH<__HASH(str)>

it seems to work, as I don't see any instances of the source strings in the output binary now

tested ingame and yeah it works

Result is OK in this case as well. Either way would work (enums or templates).

implemented in 666286e, closing


integrations: Update Zabbix Documentation.

I have updated the documentation for the Zabbix integration to give the
correct instructions for the latest version of Zabbix (5.2). The old
instructions are now obsolete.
I have also updated the message that is PMd to a user if the webhook
doesn't receive a complete payload to also align with the new
instructions.

Testing plan:
Ran all the linters.
GIFs or screenshots:

Merged as the series ending with e1aa7dd882 (including fixing the 2-digit formatting issue).
Thanks @adambirds!


[BUG] UglifyJs only works with ES5

Do you want to request a feature or report a bug?
bug

What is the current behavior?
When adding the UglifyJsPlugin, I get an error like
ERROR in global.js from UglifyJs
Unexpected token: name (resolve) [global.js:1218,8]

where the problem like is like this:
function animationFrame() {
    let resolve = null; // <--------------------------------- RIGHT HERE
    const promise = new _promise2.default(function (r) {
        return resolve = r;
    });
    window.requestAnimationFrame(resolve);
    return promise;
}

If the current behavior is a bug, please provide the steps to reproduce.
This is a bug, because not all loaders convert JavaScript to strictly ES5 (which UglifyJs requires). For example, buble-loader can convert JavaScript code for relatively new browser targets, which results in let and const statements that UglifyJs seems to crash on.

What is the expected behavior?
I should be able to stick buble-loader into my Webpack project, and I should be able to use something like UglifyJsPlugin to minify it.
However, this only works if one uses something like Babel to completely convert to ES5.
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
I'm on Webpack 3.4.1.
Look how many people have had this problem: https://github.com/joeeames/WebpackFundamentalsCourse/issues/3

Maybe UglifyJs can use the ES6+ version.

Alright, I found a solution that works with more modern code: babili-webpack-plugin.
So I'm using
const BabiliPlugin = require("babili-webpack-plugin");
new BabiliPlugin

instead of
const UglifyJsPlugin = require('webpack/lib/optimize/UglifyJsPlugin')
new UglifyJsPlugin

https://github.com/babel/babili

see also https://github.com/webpack-contrib/uglifyjs-webpack-plugin/issues/105

@trusktr  can you comment you entire webpack.config here?  I have the same issue but is not working for me.

@damsorian just replace webpack/lib/optimize/UglifyJsPlugin with babili-webpack-plugin. You have to npm i babili-webpack-plugin --save-dev first.

in my case, this answer helped me.


Add example: Streaming results to a database

Add example or tutorial to the documentation on streaming results to a database with a custom data sink.
Please note: https://duckdb.org/2023/03/03/json.html for getting started with DuckDB.

[x] Prepare BQ example
[ ] Check/verify SQL model
[ ] Decide on what API we go forward with

DuckDB is done, BQ is next:

Schema: https://cloud.google.com/bigquery/docs/reference/standard-sql/data-types
Table creation: https://cloud.google.com/bigquery/docs/tables#python

@nicholasjng I assume that the PR #85 addresses the first task in this ticket ("Prepare BQ example") and the two following are still to be done (" Check/verify SQL model" and "Decide on what API we go forward with") - I created a separate ticket for them https://github.com/aai-institute/nnbench/issues/86.
Please shout if my assumption is wrong.


fix broken link

The link was not working to me. I update it with the current link.

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


WIP[do not merge!]: update latest figures + add data

Hi @KamalakerDadi here we go. This brings the latest figures + the intermediate computation and data needed to play with the results. Let's have a careful look before merging.

OK, it seems the data files I'm trying to add are too large for GitHub. I'll try to find a workaround.

Thanks for the PR. Please let me know when to review.

@dengemann What is the idea? Would you like to make the scripts runnable?

Hi Kamalakar, the diff is too large unfortunately, but perhaps you can pull and look on your computer or check out my fork https://github.com/dengemann/proxy_measures_2020/tree/curated_viz

Great work. Thank you!


Points of Interest used in Released Web Map but not released

id:3A56228feb2f684aaa9db108f588666fc7
Point of Interest Map Service used in the released web maps. Need to be released for demo purposes at least.
Content is in release review so assigning to @lfunkhouser.

id:56228feb2f684aaa9db108f588666fc7
@nfeuerstein This item is in Release Review - is this supposed to be released as released content or demo content?

If @nfeuerstein had it in release review I would assume it is for release.

@lfunkhouser It was in release review prior to there being a demo review group.  I went through and reshared a bunch of items to the correct group but apparently missed this one.  I'll go through again.  To be clear, it is for demo review, the symbols are awful but the data is good.

Thanks for clarifying @nfeuerstein
Fixed typo "maybe" to "may be" in description
Released by Demo Intelligence
Updated extent of portal item to AOI so basemaps are available
Back at you @conklinbd

Agree on the symbols.  There was no standard set that I could use to support this range of objects, so I mixed and matched to try to get something somewhat consistent- sort of like international map/road signs.  Something for improvement in a future release.

Verified released. I am guessing my original id was a typo. I will have to retest a couple of apps to be certain.


Rethink the Mechanism concept?

Maybe merge the IP / Parameter / Form mechanism into a single HTTP mechanism.
Change the getName() for the HTTP client to add the Authenticator and ProfileCreator names.

PR: https://github.com/pac4j/pac4j/pull/230


Ultra: Generate Map Error

High Level Description
Under ultra-develop branch, when generating the map using command scl scenario build-all ultra/scenarios/pool, an exception raised for Thread-72 because there is no directory ultra/scenarios/pool/experiment_pool/behavior_race/70kmh/map.net.xml
Desired SMARTS version
0.4.16
Operating System
ubuntu 16.04
Problems
None

Also I check the ultra-develop branch, the exception raised is No such directory "ultra/scenarios/pool/2lane_curvy_t/70kmh/map.net.xml".

Documentation is updated in #944.

It works thank you for your reply.


Added some methods for convenience

Thank you for the PR.
I'm totally onboard with the TryGetMimeType, but the xxxFromFilename method doesn't really fit with what I would expect for an API. The implementation can be greatly simplified by using Path.GetExtension(filename), so a user can easily just call
GetMimeType(Path.GetExtension(filename))
That said, since a call to GetMimeType with a filename will always return the default value, which is not what one would expect, I would be perfectly fine making the default behavior for TryGetMimeType (and this GetMimeType) to automatically try with the extracted extension if no mime type was found.


Convert executor/restore_test.go to unit test

Summary
executor/restore_test.go was basically a huge unit test all along. It has now been taken apart and re-implemented as an actual unit test in preparations for #274.

Checklist

[x] Keep pull requests small so they can be easily reviewed.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking,
as they show up in the changelog
[x] Update the documentation.
[x] Update tests.
[x] Link this PR to related issues.

I've implemented the feedback. There have not been any other changes.

I've implemented the feedback. There have not been any other changes.


A link in the terms of use is not working correctly.

One fo the link in the terms of use is not working. It is missing an L in the HTML but even with an L it is not working.
the link in the production website in the 'Privacy Statement" works so we should use that link and fix this issue.

Mouad has completed this task and Sruti and Abhishek need to Test this.


Add an implementation of my Kruskal's algorithm in Java

HacktoberFest 2021 Pull Request

This is a HacktoberFest 2021 Contribution
Can you please add the HacktoberFest tags to this pull request ?
Thank You in advanced!


EOS-25248: [Containerization] Log rotation policy to be implemented

Fixed log path for elect-rc leader, proto-rc and consul-watch handler scripts.
Log rotation will be configures in post_install for LR and in 'start' for LC
Log path will be template value in code, during log rotation configuration those will be replaced by actual paths
Changed log path of 'hare-hax' . New path will be as follows
/share/var/log/cortx/hare/log/aaa120a9e051d103c164f605615c32a4/hare-hax.log
Disclaimer: Testing is in progress
Test done:
[root@storage-node1 /]# cat /etc/logrotate.d/hare
/share/var/log/cortx//hare/log/aaa120a9e051d103c164f605615c32a4/*.log
/share/var/log/cortx//hare/log/aaa120a9e051d103c164f605615c32a4/hare_deployment/*.log
{
    rotate 10
    maxsize 50M
    weekly
    compress
    missingok
    copytruncate
}

#cat /etc/cortx/hare/config/aaa120a9e051d103c164f605615c32a4/consul-client-conf/consul-client-conf.json
.
.
.
{
  "enable_local_script_checks": true,
  "watches": [
    {
      "type": "key",
      "key": "leader",
      "args": [
        "/opt/seagate/cortx/hare/libexec/elect-rc-leader",
        "--conf-dir",
        "/etc/cortx/hare/config//aaa120a9e051d103c164f605615c32a4",
        "--log-dir",
        "/share/var/log/cortx//hare/log/aaa120a9e051d103c164f605615c32a4"
      ]
    },
    {
      "type": "service",
      "service": "ios",
      "handler_type": "http",
      "http_handler_config": {
        "path": "http://storage-node1:8008",
        "method": "POST",
        "timeout": "10s"
      }
    },
    {
      "type": "keyprefix",
      "prefix": "bq/",
      "handler_type": "http",
      "http_handler_config": {
        "path": "http://storage-node1:8008/watcher/bq",
        "method": "POST",
        "timeout": "10s"
      }
    },
    {
      "type": "service",
      "service": "ios",
      "args": [
        "/opt/seagate/cortx/hare/libexec/consul-watch-handler",
        "--log-dir",
        "/share/var/log/cortx//hare/log/aaa120a9e051d103c164f605615c32a4"
      ]
    },
    {
.
.
.
.

@SwapnilGaonkar7, is the PR not yet ready for the review? its still a draft.

Tests done:

for start in LC env.

changed log path in confstore

storage:
      config: /etc/cortx
      local: /etc/cortx
      log: /share/var/log/cortx/test1/

created hare-hax container using hare_setup start
configuration was updated

cat /etc/logrotate.d/hare
/share/var/log/cortx/test1//hare/log/aaa120a9e051d103c164f605615c32a4/*.log
/share/var/log/cortx/test1//hare/log/aaa120a9e051d103c164f605615c32a4/hare_deployment/*.log
{
    rotate 10
    maxsize 50M
    weekly
    compress
    missingok
    copytruncate
}

logs were generated

[root@storage-node1 /]# ls /share/var/log/cortx/test1//hare/log/aaa120a9e051d103c164f605615c32a4
hare_deployment  hare-hax.log

log rotation configuration at post_install stage, for LR environment.

log path in confstore: /share/var/log/cortx/pi/

 /opt/seagate/cortx/hare/bin/hare_setup post_install --config yaml:///etc/cortx/cluster.conf
2021-10-18 12:48:06,630 [INFO] Server type (physical)
[root@storage-node1 /]# cat /etc/logrotate.d/hare
/share/var/log/cortx/pi//hare/log/aaa120a9e051d103c164f605615c32a4/*.log
/share/var/log/cortx/pi//hare/log/aaa120a9e051d103c164f605615c32a4/hare_deployment/*.log
{
    rotate 10
    maxsize 50M
    weekly
    compress
    missingok
    copytruncate
}

hare deployment logs were generated

[root@storage-node1 /]# ls /share/var/log/cortx/pi//hare/log/aaa120a9e051d103c164f605615c32a4/
/share/var/log/cortx/pi//hare/log/aaa120a9e051d103c164f605615c32a4/hare_deployment/setup.log

@mssawant I have rebased code and tested deployment and services start in K8.
Please have a look


[enhancement - aws/aws-sdk-pandas] - Update README.md - improving and tweaking grammar and syntax

README.md Enhancement Proposal
Repository and File

Repository: aws/aws-sdk-pandas
File: README.md

Enhancement Type

This enhancement focuses on improving and tweaking the grammar and syntax used in the README.md file.

Scope of Enhancement

The goal is to enhance the clarity, polish, and overall readability of the content in the README.md file.

Implementation

Review the current content of the README.md file.
Make adjustments to improve grammar and syntax, including rephrasing sentences and correcting punctuation.
Ensure the text is well-structured for better comprehension.

Tracking

This enhancement is expected to be tracked as an issue or a pull request in the GitHub repository.
Contributors are encouraged to discuss, review, and collaborate on the proposed changes.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 47417ce09eec9c16a37e6c4e6040521b425bf38b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 47417ce09eec9c16a37e6c4e6040521b425bf38b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Thank you for PR however the suggested changes are inconsistent with the project's voice. We are open to feedback on other ways to improve the documentation.


New modding menu dislays inconsistently

Want to preface this by saying I can't offer much concrete evidence as I can't test it on my own;
The new modding menu that MINT introduces as it does currently on the master branch, loads fine when alone (as host in your own lobby) Whenever a client joins that is not on a new version of mint or maybe just a mismatched version (this is mainly the part I cannot test) I'm no longer able to view my own modlist as host.
Also, as client the modding menu will simply never display any mods EXCEPT when the host of the lobby I am in is also on that same version. If host is not, the modding menu will even say I am on "Built-in Support" which is not the case
In the attached screenshot an example of the behaviour as client; I (doddle) am told I'm on built-in support because host (Alkyayzzz) is on an older version

Another screenshot indicating showcasing that behaviour in this case I am "bea" but the cause remain the same; host is on an old version


Add a verify connection function

Moving the info fetching out of the connect function introduced an issue: now it's possible to "connect" even if the configuration is wrong, because we don't test it (e.g. entering a wrong lnbits admin key).
Each connector can have its own verification function

If we don't call one of the WebLN functions (like getBalance) we don't really have a good way to identify if the connection works.
We should:

Balance component should request the balance from webln directly
Show a "?" and/or error where it is shown instead of 0 balance ()
Any call that is made to webln should raise an error
We should not expose functions from the store to get connector info
Do not do any sort of caching for now.


Using Webpack with monorepo

Hi.
Researching a way to use Webpack with front end monorepo. I was wondering if I can use one common Webpack config and place it to the root directory and then extend this config in other apps. But, when I tried to setup common configuration and extend it I was stuck into running context, so the building is running in the root directory, not in each of apps.
My question is: do you have Monorepo support or can I use common configs and then override what I need in each of the applications, but the main configuration is stored in the root?
Thank you in advance!

I encourage you to ask your question on StackOverflow instead. We try to use the issue tracker for bug reports and feature requests, as well as some discussions, but this appears to be a very localized question.
I’m closing for these reasons.

I haven't found any info according to my question in your documentation. So, if this info presents, it will be great if you add this to docs. So it can be a feature request.

Please open issue in webpack-docs, it is related to documentation

@evilebottnawi not sure if it's related to documentation. Maybe Webpack can't do that, so documentation is empty according to this as well


New package: GorrilaDevs.Ferium version 4.5.0

Pull request has been created with Komac v1.11.0 :rocket:

Resolve: #137328

Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run

Automatic Validation ended with:

2024-01-31T15:56:45.1524192Z ##[error] One or more errors occurred. (SQL error or missing database (0x87AF0001)) SQL error or missing database (0x87AF0001)

(Automated response - build 684.)

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping @Exorcism0666, I'll get a notification.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Hi @Exorcism0666,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 691.)

Automatic Validation ended with:

Installation failed with exit code -1978335187 2024-01-31 12:41:18.010 [FAIL] Installer failed security check. Url: https://github.com/gorilla-devs/ferium/releases/download/v4.5.0/ferium-windows-msvc.zip Result: 0x80004005 MSI (s) (94:A4) [12:41:04:611]: Windows Installer installed the product. Product Name: Microsoft Visual C++ 2022 X64 Minimum Runtime - 14.38.33130. Product Version: 14.38.33130. Product Language: 1033. Manufacturer: Microsoft Corporation. Installation success or error status: 0. MSI (s) (94:08) [12:41:05:767]: Windows Installer installed the product. Product Name: Microsoft Visual C++ 2022 X64 Additional Runtime - 14.38.33130. Product Version: 14.38.33130. Product Language: 1033. Manufacturer: Microsoft Corporation. Installation success or error status: 0. [0114:15D4][2024-01-31T12:41:02]w363: Could not create system restore point, error: 0x80070422. Continuing...

(Automated response - build 691.)

/AzurePipelines run

Hi @Exorcism0666,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 694.)

Automatic Validation ended with:

Installation failed with exit code -1978335187 2024-01-31 19:22:30.994 [FAIL] Installer failed security check. Url: https://github.com/gorilla-devs/ferium/releases/download/v4.5.0/ferium-windows-msvc.zip Result: 0x80004005 MSI (s) (A0:B8) [19:22:18:031]: Windows Installer installed the product. Product Name: Microsoft Visual C++ 2022 X64 Minimum Runtime - 14.38.33130. Product Version: 14.38.33130. Product Language: 1033. Manufacturer: Microsoft Corporation. Installation success or error status: 0. MSI (s) (A0:08) [19:22:19:114]: Windows Installer installed the product. Product Name: Microsoft Visual C++ 2022 X64 Additional Runtime - 14.38.33130. Product Version: 14.38.33130. Product Language: 1033. Manufacturer: Microsoft Corporation. Installation success or error status: 0. [0508:1AD8][2024-01-31T19:22:15]w363: Could not create system restore point, error: 0x80070422. Continuing...

(Automated response - build 694.)

Other than that the manifest works as expected for me:
where ferium
# %LOCALAPPDATA%/Microsoft/WinGet/Packages/GorrilaDevs.Ferium__DefaultSource/ferium

https://github.com/microsoft/winget-pkgs/issues/137328#issuecomment-1925291267

@stephengillie @ST-DDT @RokeJulianLockhart I will reopen a pull request because of the typo :c


[Action] Deploy to App Engine

I've pulled all the gcloud setup into a separate package which is used by the action. After this PR I will refactor the setup-gcloud actions to use the new package.
I need help know how to proceed forward with have the separate package. Are we going to publish to npm or how can we refer to it in typescript?
Also, I would like to make sure there are enough tests.

Great contribution! Regarding the library question, since this is the first consumer of the lib I think we can just use the internal code reference for now, and circle back to the discussion of publishing an npm package as needed.

Looks great. Would love to see the re-factor work for the setupGcloud action also included :)

Closing this PR to push to a non-forked branch.


Auto-generated Vault certs

I'm investigating hooking up Vault with Fabio and I'm a little confused about the cert store config. The documentation suggests Fabio will fetch certs dynamically from Vault but the config seems to imply that the certs are expected to be manually dropped into a generic 'secret' store in Vault. Is there also support for the PKI secret store and its ability to auto-generate certificates?

Not yet. I'll check what it takes to add that.

I can manually issue a cert using the following:

vault write pki/issue/server-dev common_name=foo.server.dev

In this example I've set up a pki backend with an internally generated root cert for *.server.dev and then I've created a role 'server-dev' that is allowed to serve certs for .server.dev based on the root cert. Using the above command will return both the private and public parts of the new cert as well as a copy of the CA.

I'd really like to see Vault's PKI backend supported, as well.   I've been chewing on this for an hour or so and I think the model of multiple certs per cert store doesn't exactly mesh with Vault's PKI backend, but if each vault-pki certificate source has a single pki secret path instead of multiples, that would probably work fine. One of the config options to the vault-pki cert source would be a list of SANs that would be on the cert, perhaps?

I'm curious how this would work independent of vault since you could mimic this with a script or LetsEncrypt or something else. The current CertSource model is that certificates exist and that fabio just picks them up. If a cert does not exist fabio will refuse the connection.
In the model of a generated cert that would no longer happen unless the cert provider refuses to generate the cert.
Who controls which certs are permissible? Can fabio just generate any cert? What if I announce urlprefix-www.google.com/ ?

The Vault operator defines what certificates can be issued on a given path.
From fabios perspective there is almost no difference to fetching certs from the generic secret backend. In Go, it looks something like this:
import "github.com/hashicorp/vault/api"

var client *api.Client

response, err := client.Logical().Write("pki-fabio/issue/example.com", map[string]interface{}{
	"common_name": "foo.example.com",
	"alt_names": "localhost,<IP_ADDRESS>",
	"ttl":         "24h", // uses Vault's default if omitted
})

pemKey := response.Data["private_key"].(string)
certChain := response.Data["certificate"].(string) + "\n"
for _, cert := range response.Data["ca_chain"].([]interface{}) {
	certChain += cert.(string) + "\n"
}

So very similar to what the current vault store does. client.Logical().Read is replaced with client.Logical().Write, and the data structure is a bit different from what the current store expects. The "pki-fabio/issue/com.example" path has to be configured in fabio (like for the Consul store).
Each Write() call generates a new certificate. It will not return previously generated certs. Previously generates certs remain valid. This is different from how Letsencrypt works.
For completeness, this is what a Vault operator might do in advance:
# Mount a pki backend for fabio to use. Certificates will expire after 7 days
# by default (7*24=168), and attempting to issue certificates with a validity
# period of more than 30 days will be denied (30*24=720).
vault mount \
    -path pki-fabio \
    -default-lease-ttl=168h \
    -max-lease-ttl=720h \
    pki

# Define which certificates can be issued via the pki-fabio/issue/example.com path.
vault write pki-fabio/roles/example.com \
    allowed_domains="example.com" \
    allow_subdomains="true" \
    allow_localhost="true"

Many more role options are available. I omitted the commands for configuring the CA for the pki-fabio mount.
Would you be interested in a PR with a rough draft for an implementation?

Yes, a PR would be welcome. Generated certs need to be cached for as long as they're valid (or TTL/2) since this method is called per connection. I assume the base path defines the root domain for which you can generate certs. I think that's fine but you should take into account use-cases for multiple domains.

In the "pki-fabio/issue/example.com" path, "pki-fabio" represents a single CA, and "example.com" represents the constraints for issuing certs from that CA, such as allowed domains (what Vault calls a role).

The question is whether you want to allow one or all domains in the pki or whether you want to be able to restrict it to some (e.g. foo.com and bar.com but not google.com for example)

I don't know how to achieve that and whether that's something you can do within the vault config that fabio just has to send or whether there needs to be something to be done in fabio and/or whether we need to extend the config DSL. Just keep that in mind when creating the PR, please.

🍿 watching with excitement. Happy to help test.

Regarding the authorization of the domain, I don't think fabio needs to be involved. If someone asks for foo.company.com and vault allows that cert to be fetched, fabio can serve the cert it fetches (and cache it). If vault doesn't allow it, fabio can serve a default cert or throw an error?

That's exactly what the PR currently implements. If a cert cannot be issued fabio drops the request. There is no default cert because strictmatch is implied with the vault-pki source.


Training STAC summary_loss = nan

Hi!
I am now training your STAC model using a baseline trained using tf_efficientdet_d0.
But now I am getting summary_loss = nan for every training step and this:
arning:  multi_tensor_applier fused unscale kernel is unavailable, possibly because apex was installed without --cuda_ext --cpp_ext. Using Python fallback.  Original ImportError was: ModuleNotFoundError("No module named 'amp_C'")
2020-10-27T00:13:26.180480
LR: 1e-05
Gradient overflow.  Skipping step, loss scaler 0 reducing loss scale to 256.0
Gradient overflow.  Skipping step, loss scaler 0 reducing loss scale to 1.0
Gradient overflow.  Skipping step, loss scaler 0 reducing loss scale to 0.00390625
Gradient overflow.  Skipping step, loss scaler 0 reducing loss scale to 1.52587890625e-05
Would appreciate any help!

I haven't trained on d0. Could you provide more details on your pipeline (e.g. batch size, data pipeline, resolution)?

Thanks for your reply!
I moved to d2 which my computer can handle but still getting the same issue.
I am using the original data to train your baseline which works fine for d2 (though now you mention it, using the 1024x1024 images could very well be the cause of my previous CUDA errors, did you resize the images?). I had reduced batch size to 2 so that I wouldn't get these CUDA errors. Then I am using your full sized data that you uploaded to Kaggle to train your STAC model which is giving the above error.
Also, I was looking through your code to try and replicate STAC in YOLOv5; was your approach just to use the augmentations stated in the paper and then weight the pseudo loss rather than using consistency loss?

I'm not sure if d2 works, and reducing batch size could bring performance regression.
Regarding STAC, I did try consistency loss but it did not improve my model.


Update with current beta builds

https://www.theiphonewiki.com/wiki/Beta_Firmware/iPhone

Thank you for this wonderful contribution!


Whitelist import ?

Hi there !

Is there a way to import a whitelist (steam:...) into txAdmin ? We'd love to drop our legacy WL script to enable txAdmin one.
Is there a way to disable WL requests and to allow only admins to add / remove whitelisted users ?

Thanks and congrats on the great job!

Whitelist requests are submitted as soon as someone trys to connect to the server, with those request id's Admins use the discord channel (if properly setup) to whitelist that player (i.e. /addwl GD8D-83H3) or straight from the web interface

Import from which resource?
I'm currently finishing the ban import thing, and for now I didn't do whitelist importing thing since I don't know any well spread WL resource out there, and the fact that only 12% of txAdmin servers use whitelist (opposed to the 94.8% that use the ban list).

Hi @PLOKMJNB @tabarra, We're using a "custom" script, reading a file containing all steamhex, and checking is the player is allowed to connect.
I can try to work on the import myself and to send a PR if needed, but is there some documentation about how are whitelisted identifiers stored ?

That's why I asked the source of the whitelists.
If its not widespread I rather not have it in txAdmin... I like to keep it lean (check the feature graveyard docs).

For reference:
src\components\playerController\index.js -> registerAction

I'm closing this issue, but feel free to continue discussing here if you feel like it is needed.


[WIP] Add  jUNitXML plugin to create the output xml file

Going forward polarion supports only xml reporting. Hence testing jUNitXML plugin in ansible to o/p logs into xml format
OSPRH-8374

Task Run Podified EDPM deployment failed running on host controller

recheck

Thank you for making this investigation, unfortunately, we're going with the other option since we can't configure the name and classname like we want for polarian.
Other option: https://github.com/infrawatch/feature-verification-tests/pull/132
Please close this PR


merge package from arlac77/npm-package-template

.travis.yml

chore(travis): merge from template .travis.yml

Codecov Report

Merging #428 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #428   +/-   ##
=======================================
  Coverage   73.94%   73.94%
=======================================
  Files           2        2
  Lines         476      476
=======================================
  Hits          352      352
  Misses        124      124

Continue to review full report at Codecov.

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

Coverage remained the same at 71.626% when pulling 21d1dcd8e7ea56f0db324f1555d57e7f22a7fead on template-sync-1 into 6e1f8feec6ea2bfc7aca3ea5d62ffce29b9b413d on master.

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat: metrics reporter for Jaeger collector exporter

This PR adds a metrics reporter to Jaegers' collector exporter, which is similar in implementation to the OTLP equivalent.
It measures the following:

request duration
error count

I haven't had a chance yet to test with a real application, but I will look into this soon.

@chrisholmes sorry this sat around for a long time - do you mind addressing the merge conflict with main?

@plantfansam that's OK. I only got around to properly testing last week. The merge conflict should be addressed now.


Make ol.linenums use display:inline-block

Without this, the ol.linenums section won't expand to
fill its pre parent if we set the parent to overflow:scroll.
This change fixes that bug.

/cc @mikesamuel

ping @mikesamuel