Doesn't work on Linux Mint

I couldn't get it to work on Linux Mint. Probably because it is not properly installed; the installation instructions need work.
Some of it is not your fault. It is specified that it depends on libffi - and installing that "out of the box" didn't work, either, producing some bizarre errors about missing macros. I figured out that compiling libffi depends on two things that were not installed by default on my machine:

libtool
makeinfo (in order to install that one, you actually have to install "texinfo")

After installing them, libffi compiled and installed. After running autogen.sh, configure and make, the things compiles, albeit with a lot of warnings. Most of them are about ignoring the return value of asprintf(), but there are others:
types.c:51:13: warning: format '%lu' expects argument of type 'long unsigned int', but argument 2 has type 'size_t' [-Wformat=]
builtin_error("cannot handle size %lu", type->size);
types.c:79:9: warning: passing argument 2 of 'check_parse_long' from incompatible pointer type [enabled by default]
if (check_parse_long(parameter, &n)) {
util.h:4:6: note: expected 'long int *' but argument is of type 'intmax_t *'
bool check_parse_long(const char *number, long *result);
unpack.c:83:25: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'arrayind_t' [-Wformat=]
element->ind);
unpack.c:237:25: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'arrayind_t' [-Wformat=]
element->ind);
The installed program simply doesn't work - trying the various examples in the test directory results in segmentation faults and other errors.

Those warnings are harmless, it's safe to ignore them.
I'm not familiar with Linux Mint, but I think the issue here is that ./configure is detecting that not all dependencies are available and disabling some features that the testcases use.
You probably just need to do something like this:
apt-get install libffi-dev libdw-dev libelf-dev, and then run ./configure again, make sure it doesn't say "AUTOMATIC STRUCT SUPPORT IS DISABLED" at the end.

Some of those warnings are harmless on amd64, but not on e.g. i386 or other 32-bit platforms.  The first is probably not a problem as long as size_t matches long on your platform (seems very likely), but is also easy to fix — replace %lu with %zu.
The second is a problem on platforms with 32-bit long.  Usually intmax_t is 64-bit on those platforms.  So either pass in a long to check_parse_long() or change check_parse_long() to emit an intmax_t (probably requires changing other callers, though?).
I don't know what arrayind_t is, but assuming it's size_t, again, just change %lu to %zu.
As far as the seg faults, can you paste backtrace(s) or try installing libffi-dev, libtool, texinfo, etc and trying again?  Cheers.

Those warnings are harmless, it's safe to ignore them.

When I was studying programming (a lifetime ago), I was told that a good programmer should strive to reduce the number of compiler warnings to zero otherwise some of them will bite him in the ass sooner or later. The compiler outputs them for a reason and not polishing the code to avoid them is just a sign of sloppyness.

apt-get install libffi-dev libdw-dev libelf-dev

Well, mentioning this in the installation instructions might be a good idea. Same for libffi-dev. (I tried installing "libffi" but there was no such thing in the repository, so I had to rebuild it from GitHub.)

run ./configure again, make sure it doesn't say "AUTOMATIC STRUCT SUPPORT IS DISABLED" at the end

It does not. I can't see any complaints about missing stuff any more, either. I rebuilt the program (make clean; make; sudo make install) and now some of the tests work - but not all.
In particular, these fail:

math.sh - where is libm.so supposed to be located? Maybe it can't be found for some reason?

stat.sh - it prints

/etc/passwd
uid:  0
gid:  0
mode: 0
size: 0
FAIL

structs.sh - it prints

Testing nested anonymous and named structs...
PASS
Testing unions work, and selecting union members...
PASS 1/2
PASS 2/2
Testing structs with many different types...
FAIL

wget.sh - I figured our where the bug in it is and opened a separate issue about it.

structs.sh exits after the first fail. I commented out the exits, so that I can see which tests pass and which fail. Here is the result:
Testing nested anonymous and named structs...
PASS
Testing unions work, and selecting union members...
PASS 1/2
PASS 2/2
Testing structs with many different types...
FAIL
Testing structs with arrays...
PASS
Testing structs with embedded enums...
FAIL
Check that anonymous structures referenced via typedef work...
PASS
Check that structs with funky packing work...
PASS
Testing structs that dont work yet, but shouldnt crash (errors are normal)...
./structs.sh: line 136: struct: multi-dimensional arrays are not currently supported
./structs.sh: line 136: struct: warning: complexarray could not be found; check help struct for more
./structs.sh: line 137: struct: unions of complex types (e.g. structures) are not currently supported
./structs.sh: line 137: struct: warning: complexunion could not be found; check help struct for more
PASS

Hopefully distributions will eventually package the library and installation will be a breeze - but until then we have to assume some familiarity with debugging autotool builds - there are too many possible distributions that we can list detailed instructions for them all.
Silencing compiler warnings is a noble goal, but it is entirely unrelated to any problems here.

math.sh - where is libm.so supposed to be located? Maybe it can't be found for some reason?

If libm is not in your standard library search path, then you would have big problems - so that's probably not the issue here.
On some systems libm.so is a linker script, and the real library is libm.so.6 but that is supported, and I have never seen any other configuration. What is the output?

structs.sh exits after the first fail. I commented out the exits, so that I can see which tests pass and which fail. Here is the result:

Could you get the output of bash -x structs.sh that will better explain what happened...

Here are the traces from mash.sh and structs.sh. Apparently, math.sh fails due to a localization issue? (Again, the whole world is not the USA, you know.) I can't figure out why structs.sh fails - something to do with the default int/long size, perhaps? This is a 32-bit Linux.
mash.sh:

source ctypes.sh
++ initialize_ctypes_module
++ local prefix=/usr/local
++ local exec_prefix=/usr/local
++ builtins=(callback dlcall dlclose dlopen dlsym pack unpack struct sizeof)
++ local -a builtins
++ enable -f ctypes.so callback dlcall dlclose dlopen dlsym pack unpack struct sizeof
++ enable -f /usr/local/lib/ctypes.so callback dlcall dlclose dlopen dlsym pack unpack struct sizeof
++ unset initialize_ctypes_module
++ declare -r NULL=pointer:0
++ declare -r STDIN_FILENO=int:0
++ declare -r STDOUT_FILENO=int:1
++ declare -r STDERR_FILENO=int:2
set -e
dlopen libm.so
verify_result sin double:0 double 0.000000
dlcall -n result -r double sin double:0
test double:0,000000 '!=' double:0.000000
echo FAIL
FAIL
exit 1

structs.sh:

source ctypes.sh
++ initialize_ctypes_module
++ local prefix=/usr/local
++ local exec_prefix=/usr/local
++ builtins=(callback dlcall dlclose dlopen dlsym pack unpack struct sizeof)
++ local -a builtins
++ enable -f ctypes.so callback dlcall dlclose dlopen dlsym pack unpack struct sizeof
++ enable -f /usr/local/lib/ctypes.so callback dlcall dlclose dlopen dlsym pack unpack struct sizeof
++ unset initialize_ctypes_module
++ declare -r NULL=pointer:0
++ declare -r STDIN_FILENO=int:0
++ declare -r STDOUT_FILENO=int:1
++ declare -r STDERR_FILENO=int:2
set -e
dlopen ./structs.so
echo 'Testing nested anonymous and named structs...'
Testing nested anonymous and named structs...
struct nested nested
++ sizeof nested
compare_gdb_size nested 16
gdb -q -ex 'file structs.so' -ex 'q sizeof(nested) != 16' -ex 'q 0'
test int '!=' int
test int '!=' int
test int '!=' int
test int '!=' int
echo PASS
PASS
echo 'Testing unions work, and selecting union members...'
Testing unions work, and selecting union members...
struct hasunion hasunion
++ sizeof hasunion
compare_gdb_size hasunion 16
gdb -q -ex 'file structs.so' -ex 'q sizeof(hasunion) != 16' -ex 'q 0'
test int '!=' int
test uchar '!=' uchar
test -z ''
test -z ''
test uchar '!=' uchar
echo PASS 1/2
PASS 1/2
struct -u g:f,:i hasunion hasunion
test double '!=' double
test ushort '!=' ushort
test -z ''
test -z ''
echo PASS 2/2
PASS 2/2
echo 'Testing structs with many different types...'
Testing structs with many different types...
struct manytypes manytypes
++ sizeof manytypes
compare_gdb_size manytypes 36
gdb -q -ex 'file structs.so' -ex 'q sizeof(manytypes) != 36' -ex 'q 0'
test uchar '!=' uchar
test ushort '!=' ushort
test unsigned '!=' unsigned
test uint64 '!=' ulong
echo FAIL
FAIL
echo 'Testing structs with arrays...'
Testing structs with arrays...
struct hasarray hasarray
++ sizeof hasarray
compare_gdb_size hasarray 128
gdb -q -ex 'file structs.so' -ex 'q sizeof(hasarray) != 128' -ex 'q 0'
test int '!=' int
test int '!=' int
test -z ''
test -z ''
echo PASS
PASS
echo 'Testing structs with embedded enums...'
Testing structs with embedded enums...
struct hasenum hasenum
++ sizeof hasenum
compare_gdb_size hasenum 8
gdb -q -ex 'file structs.so' -ex 'q sizeof(hasenum) != 8' -ex 'q 0'
test int '!=' int
test int '!=' long
echo FAIL
FAIL
struct -a unnamed_t unnamed
echo 'Check that anonymous structures referenced via typedef work...'
Check that anonymous structures referenced via typedef work...
++ sizeof -a unnamed_t
compare_gdb_size unnamed_t 8
gdb -q -ex 'file structs.so' -ex 'q sizeof(unnamed_t) != 8' -ex 'q 0'
test int '!=' int
test long '!=' long
echo PASS
PASS
echo 'Check that structs with funky packing work...'
Check that structs with funky packing work...
struct mixedpack mixedpack
++ sizeof mixedpack
compare_gdb_size mixedpack 16
gdb -q -ex 'file structs.so' -ex 'q sizeof(mixedpack) != 16' -ex 'q 0'
test uchar '!=' uchar
test unsigned '!=' unsigned
test uchar '!=' uchar
test unsigned '!=' unsigned
echo PASS
PASS
echo 'Testing structs that dont work yet, but shouldnt crash (errors are normal)...'
Testing structs that dont work yet, but shouldnt crash (errors are normal)...
struct complexarray complexarray
structs.sh: line 136: struct: multi-dimensional arrays are not currently supported
structs.sh: line 136: struct: warning: complexarray could not be found; check help struct for more
true
struct complexunion complexunion
structs.sh: line 137: struct: unions of complex types (e.g. structures) are not currently supported
structs.sh: line 137: struct: warning: complexunion could not be found; check help struct for more
true
struct bitfields bitfields
echo PASS
PASS
exit 0

Thanks for the debugging output - but I'm not sure what the correct fix is here, my ignorance of localization means I have more questions.  When you're writing C, Java or whatever - would you normally write 3,141 instead of 3.141 when writing floats?
I /think/ we need to disable localization, but maybe I don't know what I'm talking about.
The structs.sh error does indeed look like some 32bit problem, that is simple to solve. Luckily it seems like everything is basically working, the test cases just need some improvement.

Yes, disabling the localization within the math.sh script fixes the problem. Just put a line
LC_ALL=C
at the beginning. I don't know how to solve the problems in structs.sh and stat.sh, though. Oh, wait, I forgot the stat.sh trace. Here it is:

+ source ctypes.sh
++ initialize_ctypes_module
++ local prefix=/usr/local
++ local exec_prefix=/usr/local
++ builtins=(callback dlcall dlclose dlopen dlsym pack unpack struct sizeof)
++ local -a builtins
++ enable -f ctypes.so callback dlcall dlclose dlopen dlsym pack unpack struct sizeof
++ enable -f /usr/local/lib/ctypes.so callback dlcall dlclose dlopen dlsym pack unpack struct sizeof
++ unset initialize_ctypes_module
++ declare -r NULL=pointer:0
++ declare -r STDIN_FILENO=int:0
++ declare -r STDOUT_FILENO=int:1
++ declare -r STDERR_FILENO=int:2
+ set -e
+ struct -m statbuf stat passwd
++ uname
+ test Linux == Linux
+ dlcall -r int __xstat 0 /etc/passwd pointer:0x8224d08
+ unpack pointer:0x8224d08 passwd
+ printf '/etc/passwd\n'
/etc/passwd
+ printf '\tuid:  %u\n' 0
	uid:  0
+ printf '\tgid:  %u\n' 0
	gid:  0
+ printf '\tmode: %o\n' 0
	mode: 0
+ printf '\tsize: %u\n' 0
	size: 0
+ dlcall free pointer:0x8224d08
++ wc -c
+ test 0 -eq 1939
+ echo FAIL
FAIL
+ exit 1

Yes, that xstat call is 64bit specific - I think just changing it to __xstat 3 instead of __xstat 0 would work.
OK, so I think we should close this issue because installation is working as intended, and open these:

Most importantly, figure out how to handle localization. Are roundtrips between scanf and printf always guaranteed to work? If so, then we just have to make sure scripts declare localization if they hardcode something. If not, then we need to disable it somewhere.
Minor issue, some of the testcases assume 64bit.
Should probably not have a testcase that uses sockets, but for 1.2 I'll probably try to integrate with autoconf tests anyway so make check works.

For test values, set the locale to C as Vesselin suggests.  I don't know if scanf and printf are guaranteed to round trip in all locales (and even if so, if they do so correctly on all platforms…).

OK, sounds ok to me.

Done - fix committed to head. I'll close this for now and make sure tests support 32bit when the tests are ported to autoconf.
Luckily this seems to just be incomplete tests and examples, the code looks like it's working correctly here!
Thanks so much for testing @bontchev!

Most importantly, figure out how to handle localization. Are roundtrips between scanf and printf always guaranteed to work? If so, then we just have to make sure scripts declare localization if they hardcode something. If not, then we need to disable it somewhere.

It's a rare and complicated issue. For instance,
printf("%1.2f\n", 3.14);
displays 3.14 regardless of the locale. The locale mostly impacts the shell (and maybe some higher-level outputs - GTK, perhaps?) - but bash can't do floating point, so it's normally not a problem there. ksh can do floating point and the suggested solution there is precisely to disable the localization via LC_ALL=C, if you are going to use hard-coded floating point constants in expressions:
https://unix.stackexchange.com/questions/87745/what-does-lc-all-c-do
Thanks for fixing it and please don't forget to update the prerequisites/install instructions.

Bash doesn't support floating point, but ctypes.sh does via our type-marshaling system. But I just tested all locales via for i in $(locale -a), and it looks like round-trips do seem to work - so the only issue is if we use hardcoded values in scripts, where LC_ALL=C will do.
If that turns out to not be reliable somewhere, we will need something like push(locale, "C"); xxx; poplocale() in ctypes routines.  I think it's looking okay though.

Regarding the installation instructions - I /think/ I'm happy with how they are, but maybe you can give me an example of how you think it can be improved?
The prerequisites are already clearly listed on the front page of the project, and in the README - I'm not sure what else we can do here. I could add more verbose output to autoconf, but I think we're approximately aligned with what other projects are doing...
We could give detailed instructions on installing packages, but I'd rather leave that up to the distributions, duplicating them here seems like a never ending task.

If you want to be very surgical about it, instead of disabling the whole locale, you could probably just set
LC_NUMERIC=en_US.UTF-8
i.e., set the value of the decimal point to whatever the US locale uses (a dot).
Regarding the installation, I had to install many things that weren't obvious from the instructions. libffi-dev instead of libffi, for sure. libdw-dev, too; it wasn't present on my system and it wouldn't work without it. Maybe libtool and texutil? I needed to install them, in order to run autogen.sh and configure and compile the libffi from its GitHub repository - but they will probably be needed for the corresponding configuration scripts of this project, too? libc6-dbg, maybe? Although I think it was already up-to-date on my system... The instructions say that libelf and elfutils are optional, but configure was complaining that they were missing.

You need libtool to build from git HEAD (mostly intended for developers), but not for the stable release build?
The way that distribution name and organize packages varies, for example libffi-dev is needed on debian, but libffi-devel is needed on RedHat. On Debian you need libdw-dev, but elfutils-devel on RedHat and it's completely different again on FreeBSD.
So i think we don't want to have distribution specific instructions, and instead allow distributions to document how to translate "libffi" into a package name. To give you an example of how complicated this gets, on RedHat you would do yum search libffi, on debian apt-cache search libffi, Fedora dnf search libffi...and so on. I think it's best to let distributions have generic instructions here...because I have no idea what you do on SuSE or arch or NetBSD.
libelf and elfutils really are optional, that's not a bug, and when libc6-dbg (that's called glibc-debuginfo on redhat!) is missing, it says "type help struct for more information" which does explain what to do.


Can't install into an existing project, double installs React

Summary
Installed into an existing "ejected" project that had react-native at the node_modules level via yarn install
When I ran pod install, it added RNDeviceInfo, but also React.
When I then
| Version     | 0.24   |
| Affected OS | iOS   |
| OS Version  | 12.0  |
Current behavior
yarn add react-native-device-info
yarn install
react-native link react-native-device-info
cd ios && pod install & cd ..

It installs RNDeviceInfo, but also React as a dependency?
yarn ios
System fails with:
This warning is caused by a @providesModule declaration with the same name across two different files.
Loading dependency graph, done.
error: bundling failed: ambiguous resolution: module `/Users/thomasclarke/dev/mobile-notifications-native/index.js` tries to require `react-native`, but there are several files providing this module. You can delete or fix them:

  * `/Users/thomasclarke/dev/mobile-notifications-native/ios/Pods/React/package.json`
  * `/Users/thomasclarke/dev/mobile-notifications-native/node_modules/react-native/package.json`

Expected behavior
No ambiguous resolution warning

Solution:
I didn't need to run react-native link react-native-device-info

Fixed my problem thus:
https://stackoverflow.com/questions/52737251/how-do-i-prevent-react-native-device-info-causing-react-ambiguity-on-my-setup/52776146#52776146

@tclarke-scottlogic were you really able to fix your problem? I keep getting a failed build on Check dependencies


Added compiler flags that address spdlog compilation issues on some platforms

With the new Raspberry Pi OS spdlog causes the same issues as the ones observed on x86 platforms. Additional compiler flags are needed to address this.

This change breaks the old Raspberry PI OS, so at least for now the flags have to be provided with EXTRA_FLAGS where needed :-(.

@rdmark You might consider to address this in easyinstall, by checking the OS version and setting the EXTRA_FLAGS accordingly. Or even better checking the gcc version, because spdlog causes issues based on the compiler being used.
gcc 10 and newer require EXTRA_FLAGS="-DSPDLOG_FMT_EXTERNAL -DFMT_HEADER_ONLY" in order to work with spdlog.

@uweseimet That's a good idea. I raised https://github.com/akuker/RASCSI/issues/437 for this.


[AutoDiff] Improve @differentiable attribute type-checking.

Rewrite custom primal/adjoint func decl lookup function using
TC.lookupUnqualified.
The new lookup function is significantly simpler and more general. It handles overloaded decl refs, which the previous function didn't.
Fix various type-checking bugs, update tests.

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow


Add a save file

This pull request adds an "auto-save" feature for each run. The intent is for this to autostart and run in the background while a user uses their computer, providing an easier way to view this information.
The file is saved in $XDG_CONFIG_HOME/scusage.txt
Also this is my first pull request ever :D Please consider.

Thank you very much for your contribution!
I am very happy that you feel this is the right project for your first PR :)
I have 2 really minor suggestions:

I think it would be nice to have that as an optional parameter (--output?)
If scusage will get other fies in the future,having XDG_HOME/scusage/result as default output might be better (or just specify the destination in the CLI parameter)

What are your thoughts on this?

Those are great ideas!
I will also check out maybe making that file a consolidated list so people don't have to scroll through each session

Nice! If you go in that direction, I would recommend to split data and representation. For example, I would store the raw data (command distribution) in a JSON file that can be "resumed" and do the output formatting on demand with scusage.

FYI: I did a "proof of concept" implementation on a separate branch (1-save-resume), if you are interested @REALERvolker1 .
Unfortunately, I have not been able to test it at all, since I just re-installed my system to play around with wayland and sway :P

Do you have discord by any chance? Mine is volker1#0001. I have some other ideas I would like to talk to you about.

Hi,
I am really sorry, but I don't have any "real time" channel, mostly because my github projects are purely in my spare time, and that is quite the scarce resource :(
Also, I think for a community project, it makes sense to publicly track ideas in an issue tracker. Hope that is still fine for you, I am very interested in your ideas!

@REALERvolker1 do the changes work for you? if so, i will go ahead and merge them to main.
thanks!

Hello, sorry for the extremely late reply, I had some irl stuff to do. As for testing this, I have yet to do that. It is a lot more polished than my implementation.

no worries, there's no hurry to this.
thank you and all the best!

fixed in #2


APP-585: extra bracket at esp8266 and cc32xx source

Can one of the admins verify this patch?

@kaa-leeroy test this please

@kaa-leeroy add to whitelist


Alignment on CFD version when rolling over

In order to detect and more gracefully handle problems like https://github.com/itchysats/itchysats/issues/1617 we could consider sending the CFD version of the taker to the maker when triggering a rollover. That would allow the maker to know that the taker ran into some trouble and Reject the rollover with a specific reason.
That does not solve the problem reported in https://github.com/itchysats/itchysats/issues/1617 but it would make the problem explicit.

closed in favor of https://github.com/itchysats/itchysats/issues/1621


[stable/dex] Add initContainers support

What this PR does / why we need it:
It adds an option to run initContainers (https://kubernetes.io/docs/concepts/workloads/pods/init-containers/) when deploying dex chart.
Special notes for your reviewer:
@desaintmartin could you please review this small change.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[ ] Variables are documented in the README.md (there is no readme file)
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @desaintmartin

Thanks!
Could you rebase?
/ok-to-test

@desaintmartin I've rebased to master branch.

@mhrabovcin README has been finally added to this chart so could you please document the parameter you've introduced in this PR (initContainers). Also as far as your changes are user facing could you please update a minor version of the chart (rather than patch version)

@mhrabovcin
Could you resolve the conflict file again please? Thank you.
stable/dex/Chart.yaml

/lgtm


Limit github authz scopes requested by drone

I've heard complains drone is asking for too many permissions, including direct code access.
Would the following still suffice for all the features drone currently provides for github?

@crhym3 without the repo scope we would be unable to use drone with private repositories. The repo scope is required to clone a private repository, check individual private repository permissions, and helps generate the repository list in the user interface.
If you are only using drone with public repositories, however, and don't need access to private repositories perhaps we can instead make the scope configurable?
We could add the following configuration parameters to our GitHub configuration:
https://github.com/drone/drone/blob/master/remote/github/github.go#L55
github.Scopes = params.Get("scope")
if github.Scopes == "" {
    github.Scopes = DefaultScopes
}

In fact, this is what I did with lgtm.co
https://lgtm.co/docs/install/#configuration:ee40007639ba1d9861b5cc3e7ade0aef

I see. Too bad. Yeah, I'll change this to use scope param and fall back to defaults.
Thanks!

Done. @bradrydzewski PTAL.


[Snyk] Upgrade org.eclipse.jetty.websocket:websocket-client from 9.4.44.v20210927 to 9.4.46.v20220331

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade org.eclipse.jetty.websocket:websocket-client from 9.4.44.v20210927 to 9.4.46.v20220331.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 2 months ago, on 2022-03-31.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Ignored as took a later one instead.


Add support for null format property of Hive tables

Allow Presto to correctly read from exernal Hive tables that use serialization.null.format to sotre NULL in an alternate format.
This works with TEXTFILE, RCTEXT, and SEQUENCEFILE formats. There are no tests for the latter two (because they can't be easily embedded in the tests because they are binary file formats and Presto can't insert data into external Hive tables).

2020-06-18T02:27:32.4934712Z [INFO] --- modernizer-maven-plugin:1.7.1:modernizer (modernizer) @ presto-hive ---
2020-06-18T02:27:32.5529715Z [ERROR] /home/runner/work/presto/presto/presto-hive/src/test/java/io/prestosql/plugin/hive/TestHiveIntegrationSmokeTest.java:3422: Prefer String.toLowerCase(java.util.Locale)

I have second thoughts about the name of parameter. If null_value would not be better than null_format. @electrum, @findepi opinion on that?

This works with TEXTFILE, RCTEXT, and SEQUENCEFILE formats.

@jirassimok what did we end up doing with RCTEXT and SEQUENCEFILE?

I have second thoughts about the name of parameter. If null_value would not be better than null_format. opinion on that?

@losipiuk
Since this is applied to TEXTFILE only, it should probably be textfile_...., unless the plan is to extend it to RCTEXT and SEQUENCEFILE.
If i were to come with my own name, i would use ..._null_representation, but there is strong advantage to using name resembling Hive's equivalent.
Hence i propose textfile_null_format or null_format (depending on the RCTEXT/SEQUENCEFILE).

This works with TEXTFILE, RCTEXT, and SEQUENCEFILE formats.

@jirassimok what did we end up doing with RCTEXT and SEQUENCEFILE?

We decided to have followup PR with that. That one will also include product tests. Jacob is working on that.

I have second thoughts about the name of parameter. If null_value would not be better than null_format. opinion on that?

@losipiuk
Since this is applied to TEXTFILE only, it should probably be textfile_...., unless the plan is to extend it to RCTEXT and SEQUENCEFILE.

We plan.

If i were to come with my own name, i would use ..._null_representation, but there is strong advantage to using name resembling Hive's equivalent.
Hence i propose textfile_null_format or null_format (depending on the RCTEXT/SEQUENCEFILE).

We are generally not very consistent with hive if you look at the names we have already (see textfile_field_separator vs field.delim for example).
I do not have a very strong feeling if we should use one name or another. Just wanted to run in through you, as you may have :) Let's keep it null_format then.


doc update: The project MUST have a general policy (formal or not) that as major new functionality is added to the software produced by the project, tests of that functionality should be added to an automated test suite.

Done.


Wrong minification on invalid input

Hi,
I have custom strings inside my css,
body { border: "unit(--borderWidth, px)" solid "color(color-1)"; }

CSS-Nano removes the strings after minification, and it becomes
body{border:solid}

I`ve tried, and if the input is only:
body { border-width: "unit(--borderWidth, px)"; }

The output will remain that string.
body{border-width:"unit(--borderWidth, px)"}

Expected result, those css strings should remain at the minified result.

Sorry, we expect you to feed valid CSS to cssnano. Minification should be done as the final step in your CSS pipeline.

Yeah, but why it remove the string? if it not familiar with that it suppose to keep it...

Because the code makes executive decisions based on known keywords and CSS structure in order to optimise the values. You can disable this behaviour by disabling orderedValues.
It's hard to determine what the code is supposed to do given that the input is invalid, and attempts to handle it would be somewhat ambiguous (e.g. should the first value be a width, style, or a colour?). If you believe that it should be handled differently from how it works currently, you're welcome to state your case, and if possible, submit a PR.

Can you point the place of the minification logic?

https://github.com/ben-eb/cssnano/tree/master/packages/postcss-ordered-values

Seeing as we've landed on not changing the current behaviour (#403), I'm closing this issue.
@felixmosh Thank you for contributing to the project. Even though your contribution ended up being rejected, your involvement is very much appreciated. Unfortunately, sometimes we have to reject PRs in order to keep a certain level of maintainability. I hope this doesn't discourage you from contributing in the future.


Discovery misses some hosts

Please see extended discussion: https://kentik.slack.com/archives/C03MQCJ1Z2Q/p1677232845830249
In this case, there was a host that did respond to SNMP but was not picked up by ktranslate. I did some investigation but it wasn't really conclusive. ktranslate uses DeviceScanner.Scan() from https://github.com/liamg/furious. This method seems to ignore the provided port list and instead attempts to connect to TCP port 1. As this port is used by no one and is likely firewalled everywhere, I don't understand how this works at all. Needs more investigation.

There's a known bug where a basic disco scan will see that hosts are down when they are actually up. Two ways that the system can work around this are:

Specify a /32 range for known devices which are not getting picked up. This forces ktrans to double check these IPs.

https://docs.newrelic.com/docs/network-performance-monitoring/advanced/advanced-config/#discovery

Set this variable to true in the config file: check_all_ips
From the docs:
When set to true, forces the discovery job to attempt SNMP connectivity against every target IP address from the cidrs array, without checking for liveliness first via TCP port scan. This setting will slow down discovery jobs, but can help bypass issues where discovery is failing against devices that are not listed in your cidrs array with /32 overrides. By default, it's set to false

What is the known bug? I'm just wondering what causes this.

Maybe you could use syn scan also? This requires root which is why I didn't pick it but could be an option. Or separately, connect option does take the port param so could be used too.


No Mac version for 3.1.2

Only Windows downloads are available for TerraFirma version 3.1.2 at the moment.

Yeah, I'm having problems compiling on Big Sur.  I'll try again on monday.

Sounds pretty legit, I'm trying to find an enchanted sword shrine lol

Mac and Linux binaries have been uploaded.

Thank you very much, kind sir!
On Nov 24 2020, at 12:34 pm, Sean<EMAIL_ADDRESS>wrote:

Mac and Linux binaries have been uploaded.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (https://github.com/mrkite/TerraFirma/issues/113#issuecomment-733128949), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AJJZWYXD5MAVVQIX4Z7WKNDSRPVAVANCNFSM4T56DJLA).


TypeError: undefined is not a function

I've added Restivus to a package I've made. When I call new Restivus() it crashes with TypeError: undefined is not a function
Any ideas what the issue could be? I'm running Meteor version 1.2

Are you calling that on the server? Restivus is only available on the server. You should get that error on the client.

@kahmali no, this is on the server. It's wrapped in a if (Meteor.isServer) { and in the package the code is only included on the server as well.

What version of Restivus are you using?

And no, there's no hidden dependency that would cause Restivus to be completely unavailable on the server.

Thanks, from what I can tell if we had Restivus to our main app, then things work. When we create a package with Restivus for some reason it's not working. We tried to create a sample project using Restivus in a package and it worked. So, there's likely some conflict in Restivus (or its dependencies) with our own app. We'll have to go through all the code and dependencies to see what might be keeping it from working.

What version of Restivus are you using in the package? Check the .versions file in the app when you load it as a package. Are you specifying the version as 0.8.4? That happens to me often with meteor packages in my apps. It defaults to a very low version if you don't specify, as there may be some perceived version conflict in your app.

Thanks, that fixed it! I wasn't setting the version number and it defaults to a really old version. Thanks again! Looking forward to using this! :)

Awesome! That gets me all the time. Thanks for reporting the issue and following up. I'm sure you won't be the last to bump into that one.


allow @var phpdocs before return statements

This PR stops phpdocs before return statements from being converted to comments, if they are a @var definition.
This is a common requirement with SA tooling to assist the tool with the return type.
e.g:
https://psalm.dev/r/0de99e28de
<?php

interface EncoderInterface
{
    /**
     * @return bool|float|int|string
     */
    public function encode(array $data);
}

function returnsString(EncoderInterface $encoder): string
{
    /** @var string */
    return $encoder->encode(
        ['invoice_number' => 1],
    );
}

We don't want /** @var string */ converted to // @var string in this instance.
Thanks.

Alternative proposal related to this: #4649.

This was rejected before https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/3611

seems like a slightly different use case?
no 3rd party annotations here.

It's not a problem of 3rd party annotations, but interpretation of phpDocumentor specs. In linked ticket, it was interpreted that not specifying structural element in inline phpDoc, it's not conforming with phpDocumentor specs. In case of @var, this is even more strictly prohibitted. From https://docs.phpdoc.org/latest/references/phpdoc/tags/var.html

Syntax
@var [“Type”] [$element_name] [<description>]

The @var tag MUST contain the name of the element it documents.

I see. Thanks for the clarification
On Tue, 11 Aug 2020, 17:52 Gabriel Ostrolucký<EMAIL_ADDRESS>wrote:

It's not a problem of 3rd party annotations, but interpretation of
phpDocumentor specs. In linked ticket, it was interpreted that not
specifying structural element in inline phpDoc, it's not conforming with
phpDocumentor specs. In case of @var, this is even more strictly
prohibitted. From
https://docs.phpdoc.org/latest/references/phpdoc/tags/var.html
Syntax
@var [“Type”] [$element_name] []
The @var tag MUST contain the name of the element it documents.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/5109#issuecomment-672086534,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYV4CR7PCBCTCZXAWK5DTSAFZLPANCNFSM4P3DDDQA
.

I would just point that that phpstorm, phpstan and psalm all support this.

I think this fixer should just be turned off tbh, if you want that. This fixer was designed to enforce PSR-5 doc positioning. If you want to document types for static analyzers, then you probably don't want this fixer. I no longer use it, myself.

In case of @var, this is even more strictly prohibitted. https://docs.phpdoc.org/latest/references/phpdoc/tags/var.html
The @var tag MUST contain the name of the element it documents.

But on the next line there is following:

An exception to this is when property declarations only refer to a single property. In this case the name of the property MAY be omitted.
A return statement operating only single property at once, so there is no need in name.

A return statement is not a single property. A single property is this:
class Foo
{
    /** @var string */
    public $foo; // <- i am a single property
}

The phpdocumentor spec does not allow documentation above return statements.

You could argue that static analyzers started to miss-use phpdoc for this, but there's really no alternative and that's how "it evolved".
The practical choice would be to go separate paths, but what about interop then? ~Competitors~ alternatives to php-cs-fixer might come up with different ideas/approaches and then the fragmentation gets worse.
PS: I'm leaning towards being practical, but…

People are used to write @var above any line of code expecting it to work or at least do something they want. In PHPStan I had to add support for @var above if statements, whiles, throws, switches, returns, echoes...
I dislike inline @vars  because they shouldn't be necessary at all - the source should already have the correct type when obtaining the value. They're also used for different reasons which makes it impossible to validate them - sometimes they're used to narrow down types, but sometimes also to widen or even change the types totally in case the source's PHPDoc is wrong...

They're also used to will nonexistent variables into existence when used in the global scope, typically view files. A total mess :)

Yeah, that's why I think /** @infer string */ should be used instead as it doesn't muddy the waters (I'm happy to add support for this in an upcoming version of Psalm).

I dislike inline @vars because they shouldn't be necessary at all -  the source should already have the correct type when obtaining the value

If there is typehinted a parent class as return value in a parent method's signature, how would I tell to PHPStan that I 100% sure that in the subclass I will get a subclass of return value?
It is widely used in Repository pattern.
// find() is overwritten above to ignore soft-deleted entities
public function findDeleted($id, $lockMode = null, $lockVersion = null): ?Project
{
    // it returns object|null by default,
    // I will get "should return Project|null but returns object|null" from PHPStan
    return parent::find($id, $lockMode, $lockVersion);
}

@fluffycondor You have many tools at your disposal:

Generics
Stub files
Dynamic return type extensions
Conditional types

All of them are better than copying @var over your codebase and them becoming inconsistent over time...

I agree this is a de facto standard so we should support it, I see little value in having it behind some option as I consider this a bug fix.

I agree now, too. Disregard my earlier comments, or consider them in the context of 2020. ;)

closing due to merge of #7402


Encoding issue on chart.js

Hello, I have a little problem with encoding.
How can I solve this problem?
Seems like issue has been opened a few years earlier but now it is closed.
https://github.com/chartjs/Chart.js/issues/4494
I have exact issue but solution with  in head does not work.
Have there been similar calls after the closure?

Don't see any issues: https://jsfiddle.net/Leelenaleee/95Lc146a/5/

@XaspMe we're going to need a test case that reproduces this to help you out

@XaspMe we're going to need a test case that reproduces this to help you out

Closing due to lack of test case

Closing due to lack of test case


Inconsistent paste behavior with multiple cursors

VSCode Version: Code 1.3.1 (e6b4afa53e9c0f54edef1673de9001e9f0f547ae, 2016-07-12T13:35:06.227Z)
OS Version: Windows_NT ia32 10.0.10586

Steps to Reproduce:
1: With the following text
abcd
1234
abcd
1234

2: Select all occurrences of 1234 with two cursors
3: Cut and paste the content and the original text will be restored.
4: Extend the selection to include the trailing new line.
5: The output will now be
abcd
1234

1234
abcd
1234

1234

Expected result would be to have the original text restored.

I believe this is the same issue as #4996.

Is there any movement on this? I am trying to move from Atom to VSCode but this is a huge deal-breaker, I use multiple copy and paste all the time.

any update?

Fixed with the fix for #4996 (e07bba8de99e116d63eb18e142a711ed92b04445)


api docs: Live reload the OpenAPI spec on update.

Automatically detect if the OpenAPI spec file has been modified since the last time it was loaded into memory, and if it has, automatically reload it to have the latest version.
This feature is designed with development environments in mind. The main benefit is being able to see the changes made to the OpenAPI document without needing to restart the development server, which is tedious and slows the documentation workflow down.

Testing Plan: mypy and lint checks passed locally. Made sure that zerver.tests.test_openapi.OpenAPIToolsTest passed and no coverage was lost.

So, actually, while we're changing this, our research in https://github.com/zulip/zulip/issues/9953 found that OpenAPI loading was actually like 5% of the total startup time for the Zulip server.  We should only load the OpenAPI stuff when one actually tries to visit /api/something; there's no reason for random manage.py commands to be slower because of OpenAPI :).
I think basically that means just removing the self.reload() inside __init__ and changing spec() to handle self.last_update being initialized to None, so only a couple lines of changes.

Whoopsie, that probably shouldn't take that long. I'll try to look into it and see if we can optimize it.
In the meantime, I just added a commit that does what you described. Thanks for the review!

Thanks for reviewing, @andersk! Addressed your comments and updated the PR.

Nice, merged after adding a test that calling the spec function twice doesn't call reload the second time.  Thanks @YagoGG!


MWPW-160826-remove unused code

... that kodilak reported as issue (Harcdoded non crypto secret in JavaScript) but this code block initially were reused from milo (https://github.com/adobecom/milo/blob/stage/libs/blocks/global-navigation/features/search/gnav-search.js#L130 ), where used in heather and url generation, so it is not really vurnelability issue, it is false positive check in kodilak
test url:

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

Project coverage is 82.63%. Comparing base (f07f48a) to head (a29643b).

Additional details and impacted files
@@           Coverage Diff           @@
##            stage     #143   +/-   ##
=======================================
  Coverage   82.63%   82.63%
=======================================
  Files          10       10
  Lines        3305     3305
=======================================
  Hits         2731     2731
  Misses        574      574

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


[ODS-5257]Upgrade AppCommon to verify .net 6

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Eric Stockdale seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[auto-merge] branch-24.04 to branch-24.06 [skip ci] [bot]

auto-merge triggered by github actions on branch-24.04 to create a PR keeping branch-24.06 up-to-date. If this PR is unable to be merged due to conflicts, it will remain open until manually fix.

SUCCESS - auto-merge


Tapping the dropdown on a currently signed in account results in unwanted navigation

Describe the bug

As described in title, clicking the dropdown button in the currently signed in account results in both the opening of the dropdown and an unwanted navigation to my profile page.
To Reproduce
Steps to reproduce the behavior:

Open main.bsky.dev
Go to settings
Tap the ellipsis button on the right of currently signed in account

Expected behavior

Screenshots

https://github.com/user-attachments/assets/67e94577-714f-4fca-89ed-c6cba9d36b6f
Details

Platform:  Chrome Android
Platform version: Android 14, Brave 1.71.12 Chromium: 128.0.6613.40 (Official Build) canary (64-bit)
App version: Build version: 1.90.0; Bundle info: dev (prod); Bundle date: 24082301; Platform: web

Additional context

N/A

Culprit might be https://github.com/bluesky-social/social-app/pull/4980

I can reproduce this in Safari on iPadOS.

Commit reverted, closing for now


[Request] Floor Plan Integration into TileBoard

@resoai
There is any possibility to integrate the floor plan that is developed in Lovelace into TileBoard?

I just layered images and buttons.  They recently added photo tiles that work well.

Thanks for your answer.
My use case is this. I already have the ploor plan written in lovelace. You say that I can port the already build code into TileBoard?

Oh, I see what you are saying.  I think the only way to do this would be to create a tab within lovelace that is in panel mode and then use that link as an Iframe within tileboard.  Not clean, but I can't think of another method at the moment.  My suggestion was to rebuild it in tileboard with individual buttons and images, which is what I did.

I tried with iframe but looks really bad. I really don't want to write again the code into TileBoard.

You can try Kiosk mode.
https://www.home-assistant.io/blog/2017/07/15/release-49/#kiosk-mode

@tyler-danielson -  Hi Come across this tread apologies -i see it been closed - Hoping for some assistance - been busy with it for about two weeks and learnt a lot just reading the forum still a newbie. My tileboard’s looking great need some help with my floor plan got images layered - now how would i toggle a switch to call up a image

@tyler-danielson - i see you've done this -sample code would be great.
I have layered the lights-on over my image in the Tv Room need to toggle it now.
I would really appreciate any assistants sample codes would be great
Thanks in advance


Preserve data

In this branch we persisted data provided by the user.  We made sure that On exit from the app, all the data IS SAVED (books, people, and rentals) into JSON files.
We also implemented classes and methods to ensure that on startup of the app, data is loaded from files.

Thanks @jonas-45 , great team member.
Thanks @prolajumokeoni  for taking a dive at this Milestone. 🙏🏿


Multiple error windows opened for the same project.

When I split a different file from the same project another error window will open above the already existing one.
I'm not sure if this is done intentionally or if there is a way to only allow 1 error list to be opened at a time.

This is #400 anniversary edition. :smile: Meanwhile syntastic died of Vim 8 related causes, and you should perhaps consider using ALE instead.

my bad, did not see that issue.


Replace esp module?

Not sure if this can be answered but i have 2 boards that i fried the esp module and want to replace. I tried using an air station like i would a normal IC chip but it wouldn't budge. Anyone have any suggestions or familiar with how these are manufactured that can give me some pointers to remove the esp module from the PCB board?

You need a heat gun, the temperature needs to be adjusted to about 330°, and then it must be heated for several minutes before it can be completely disassembled.

Thank you sir! I'm using a hot air rework station and have been using 400C. Heating from top and bottom for 20 mins! Thing won't budge....

You can add some solder oil to assist heating, or you can add some low-temperature solder paste to neutralize the lead-free solder on the PCB.
Also, the air outlet of the hot air gun should cover the entire ESP32 as much as possible.


Profiling doesn't support gRPC and HTTP requests

gRPC and HTTP method names are not propagated in the context. tag.Do calls in the gRPC and HTTP handlers cannot automatically collect with profiles associated with gRPC and HTTP requests.
Automatically add gRPC and HTTP request name as a pprof label.

We probably should add the method key/value in the in-process context and eliminate it before we  serialize instead.


Fix typo.

This PR corrects a typo, that breaks part of the AAA website / build.

Did this break the AAA build?

This broke the book / build partly: #362
There is also an issue with Travis CI, which committed the broken build. But @berquist is working on it :)

The issue got resolved in #392


Autocomplete filenames.

Fixes #1.

Thanks for the change @anntzer, I was in a limbo and only managed to take care of this today.


update master

update master branch

Thank you for contributing to openstack/cinder!
openstack/cinder uses Gerrit for code review.
Please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Revert "chore: update SECURITY.md"

Reverts eclipse-tractusx/sig-release#22

updated to the current state of the TRG https://eclipse-tractusx.github.io/docs/release/trg-7/trg-7-01


In Container Widget, Order by is not working as expected

Version: 1.9.x (as of 10/01/2016)
Scenario: Enable Lists module, add Containable part to BlogPost, create a new List, add some BlogPosts, reorder, create Container Widget with "order by" weight option.
The List renders BlogPosts by recent descending...
Am I missing anything?

This might be a bug., but I believe the Order By is not supposed to work anymore, and the option should probably be removed, because the order is governed by the list itself. If you want to control ordering, you should use a Projection Widget and a Query that uses the List filter.


SetOffloadPolicies for Clusters

Signed-off-by: Renkai<EMAIL_ADDRESS>
(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes https://github.com/streamnative/pulsar/issues/1640
Motivation
We need API to set offload policies for the whole cluster.
Modifications
Add Cmd for client side and implementation for cluster side
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (yes)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Another PR will be submitted for this issue


Updated Author-MS-Author Names

Updated Author-MS-Author Names

#sign-off

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @yeturis

Closing this PR as forked the main branch.


Use vcpkg_add_to_path

Prefer the built-in function instead of directly modifying ENV{PATH}.

I happened upon this when looking at a boringssl port.
https://github.com/microsoft/vcpkg/blob/master/ports/boringssl/portfile.cmake#L5-L15
Relevant docs at https://vcpkg.readthedocs.io/en/latest/maintainers/vcpkg_add_to_path


[bug] Tauri unwantedly converts PascalCase package product name to kebab-case

Describe the bug
The string configured as package.productName in tauri.conf.json in my project is "OpenDeck". This is the string I would like to be shown in start menus, docks, app launchers, software stores (e.g. GNOME Software), etc.
However, Tauri unwantedly converts this string to "open-deck" for the output artifact names (resulting in open-deck_2.0.0_amd64.deb and so forth) and for the executable name (meaning that it's open-deck to run it from a shell). I would like these both to be opendeck. However, the docs don't appear to mention any property that can be changed to override this behaviour.
Expected behavior
I should be able to specify an alternative string to use for these properties in my tauri.conf.json instead of having to deal with an inaccurate case conversion.
Platform and versions
[✔] Environment
    - OS: Ubuntu 22.04 X64
    ✔ webkit2gtk-4.0: 2.42.2
    ✔ rsvg2: 2.52.5
    ✔ rustc: 1.73.0 (cc66ad468 2023-10-03)
    ✔ cargo: 1.73.0 (9c4383fb5 2023-08-26)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-x86_64-unknown-linux-gnu (environment override by RUSTUP_TOOLCHAIN)
    - node: 18.18.0
    - yarn: 1.22.21
    - npm: 10.2.4

[-] Packages
    - tauri [RUST]: 1.5.2
    - tauri-build [RUST]: 1.5.0
    - wry [RUST]: 0.24.4
    - tao [RUST]: 0.16.5
    - tauri-cli [RUST]: 1.5.6
    - @tauri-apps/api [NPM]: 1.5.1
    - @tauri-apps/cli [NPM]: 1.5.6 (outdated, latest: 1.5.7)

[-] App
    - build-type: bundle
    - CSP: default-src 'self';
    - distDir: ../build
    - devPath: http://localhost:5173/
    - framework: Svelte
    - bundler: Vite

@FabianLars Any update?

@FabianLars I'm planning to start distributing stable builds of my application soon. I would like the package name to be in order beforehand, so that users don't end up with two versions installed later down the line when the package name gets changed. Do you know of any way to resolve this?

@ninjadev64 I have created a PR to support this kind of use case. Would you mind try and let me if it works for you?

While I'd love to try your changes, I'm currently using Tauri v1. I'll be moving to v2 as soon as it's stable, though, so thanks for the PR!


[Cognitive Services Text Analytics API] API Review

New API Review meeting has been requested.
Service Name: Cognitive Services Text Analytics API
Review Created By: Ranran Wang @jessicawrr
Review Date: 3/2/2023 4:00 PM PT
PR:
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description:
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

@jessicawrr Your meeting request has no linked PR. Please add the PR or other materials to be covered in the meeting.

@jessicawrr
Please provide PR link!

PR link is here https://github.com/Azure/azure-rest-api-specs/pull/22466

API Stewardship Board Review: 2-Mar-23
Per the discussion & review, this API is good for preview.
Meeting Recording


Correlate output error with the input in ExecutionContext

When we run a few calculations in an ExecutionContext, we may lose information about what request failed. So instead of a simple GoResult we should keep some information about the source object.
This could be tricky, as we send IOs.

This could be achieved by FoldToGoResult before sending to ExecutionContext.


SLING-4517 Update debian packaging to use crankstart.

Patch switches debian packaging over to using crankstart and sling-s3 contrib projects.
Allows user to install a generic sling server, then edit one config file to select one of:
tar files for nodes and data
tar files for nodes, s3 for data
mongo for nodes and data
mongo for nodes, s3 for data

With the upcoming migration to git this repository will be deleted. If the pull request contains data that should be re-submitted I suggest that you save the changes locally and resubmit them after the migration, which will likely happen on Monday the 9th of October.


HHH-10885 - JPA Native Queries with ordinal parameter are zero based

https://hibernate.atlassian.net/browse/HHH-10885

Seems reasonable until we can drop "Hibernate positional parameter" support.

:+1:

Thanks @sebersole ,
applied upstream


Deployment version label in New Wizard.

Hi Amazon team.
New publishing wizard doesn't allow to set version on first publish.
I have several Environment for application: test, beta, production. I'm publishing with different build configurations and transformed configs.
I'm using version to distinguish which version is designed for the environment (t123, b123, p123). It's important because sometimes I need to restore previous environment version and I can simple look at first char of version.
With new wizard I can't use version for that.
Could you please return version input to wizard?

Hi Jimfl.
Thank you. I know about legacy wizard. I hope this missing functionality will be added to new wizard before removing legacy wizard.

Promoting to a feature request. Thanks for letting us know this is valuable to you.

Since this is not directly SDK related, I have moved the item to our internal backlog.


REGRESSION: DNS broken for vpn-tcp in some network configurations

The regression is due to fast-failing of known search domains not leading to the expected follow-up query under some network conditions. Unfortunately, our test suite does not hit these conditions in CI, hence the regression. See this gist for an example trace.
Let's revert our fix for #192. We will also need to reevaluate #557 thoroughly. This will require being able to simulate the problematic network conditions.

This will be fixed in the next release.


Please add Costco Icon

Type
Icon request
Info
Please add Costco Icon
Icon(s)
No response

Here is a few examples


Contribute language on Transifex

Hello, I think will be awesome if put the translations on Transifex and get contributions for another languages (inclusive my language portuguese).

Yes, I would like to help translating it to Portuguese!


upgrade to webpack5,  libraryTarget system use externals cause error

Bug report
when i upgrade my builder to webpack5，it compile right, but it run failed with the error:
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'render')

here is my demo code:
import { useState } from 'react';
import ReactDOM from 'react-dom';
import { Button } from 'antd';
import { StepBackwardOutlined } from '@ant-design/icons'

const App = () => {
  const [cn, setCn] = useState(5);
  return (

<Button type="primary">start test</Button>

      cn value is {cn}

<StepBackwardOutlined />

);
};

What is the current behavior?
I debuged, and found the defference between from v5 and v4:
v4:

v5:

webpack5 when set output.libraryTarget to system, the default import is directly from the module（module['default']）, but webpack4 import with webpack_require.n(webpack_require.n(module))，it will tell wether the module is esmodule。
and I also found, maybe the problem is all the extarnal treat as an esmodule, so when import it ignore whether is an esmodule or not. but react or react-dom is not an esmodule, which don't have a default export

If the current behavior is a bug, please provide the steps to reproduce.
here is my demo:
git clone https://github.com/closertb/webpack5Demo.git
npm i
npm start
open in chrome, you can find the bug. when checkout the branch to v4. and npm i, npm start, you can see it render ok in the browser

What is the expected behavior?
the code runs correct without any change;

don't treat all the external module to esmodule when output.libraryTarget is system。
Other relevant information:
webpack version:  5.74.0
Node.js version:  14
Operating System:  macos
Additional tools:

@vankop @alexander-akait can someone have a look?

Change to: import { render } from 'react-dom';.
After babel you have ECMA modules code, so webpack assume code in ECMA Modules and you have default import, but you use system js (and load UMD from CDN, when remap methods).
Not sure what is the best solution here...
Also you can change code to:

Change to: import { render } from 'react-dom';.
After babel you have ECMA modules code, so webpack assume code in ECMA Modules and you have default import, but you use system js (and load UMD from CDN, when remap methods).
Not sure what is the best solution here...
Also you can change code to:

Or always remap all methods in defaults...

at first, I try to solve the problem by change the export with the default. but i found I should change every project export which use the builder;
now i write the a plugin, change the output code like:
setters: [
			function(module) {
                                module = _selfDefaultHelper(module);
                                Object.keys(module).forEach(function(key) {
					__WEBPACK_EXTERNAL_MODULE_react__[key] = module[key];
				});
			},
			function(module) {
                                module = _selfDefaultHelper(module);
                                Object.keys(module).forEach(function(key) {
					__WEBPACK_EXTERNAL_MODULE_react_dom__[key] = module[key];
				});
			}]

for now, it is work. but i still suggest, we should solve this problem, whatever it will take

@closertb How we can solve it? You use default import, but want to load it in system js, it should be solved on application side, we align with esm spec and output of other bundlers


Wasabi: Only the first 1001 items per folder seem to be listed

Hi there,
I'm trying to sync a borg backup archive to Wasabi. I'm using Ubuntu 16.04 64-bit and tried rclone 1.36, 1.37 and also the latest beta 1.37-117, with similar results.
The archive consists of about 4000 5 MB files in a single directory, together with a few other files one level up. rclone uploads them all without a problem, and I can see them in the Wasabi web interface.
When I sync again, rclone wants to re-upload all files beyond the first 1000 in the archive folder. It seems that the list returned from Wasabi only shows the first 1000 items.
A rclone ls also only lists about 1000 items:
      26 borg-backup/README
     635 borg-backup/config
   19431 borg-backup/hints.4787
51516698 borg-backup/index.4787
     190 borg-backup/integrity.4787
      16 borg-backup/nonce
 5253087 borg-backup/data/0/100
 5383536 borg-backup/data/0/1000
 5272999 borg-backup/data/0/1001
 5420980 borg-backup/data/0/1002
 [...]
 5412105 borg-backup/data/0/1929
 5253476 borg-backup/data/0/193
 5289456 borg-backup/data/0/1930
 5274946 borg-backup/data/0/1931

--fast-list doesn't make a difference, either.
Wasabi was configured according to their documentation (https://wasabi.com/wp-content/uploads/2017/08/Wasabi_AppNote_Rclone.pdf?x55444).
I'm new to S3-compatible storage. Maybe I'm missing some configuration on the backend? Or is Wasabi simply not behaving correctly?
Thanks
Jochen

Here's  a simple test case to reproduce the behaviour. It creates two folders with 1101 files each, syncs it and then does a ls. rclone lists 1001 items per folder, including the folder itself.
$ mkdir /tmp/test
$ cd /tmp/test
$ for d in 1 2; do mkdir $d; for i in `seq 1000 2100`; do dd if=/dev/urandom of=$d/$d-$i bs=4k count=1 status=none; done; done

$ find . -type f | wc -l
2202

$ rclone sync --transfers 32 /tmp/test wasabi:borgbackup/test

$ rclone ls wasabi:borgbackup/test | wc -l
2002

I'm AFK at the moment. I'd be surprised if this was a bug in the rclone S3 backend, but it isn't impossible! Have you tried the same test with S3?
You can investigate further with --dump-headers and --dump-bodies with -vv if you want.

As for my actual problem, I'm working around it by limiting borg-backup to 1000 files per folder. Still it'd be nice to figure out what's wrong here.
Unfortunately I don't have access to a regular S3 account, and I also don't know how a "good" output should look like. From what I gather, it seems rclone is requesting up to 1024 keys. As it only gets ~1000 in return, does this lead to the assumption there's no more data?
 This seems to work, it lists the 1101 files in each directory of the test case I mentioned earlier.
$ s3cmd ls s3://borgbackup/test/2/|wc -l
1101

Ok, now I've looked a bit further at the debug outputs of the two clients.
It seems s3cmd doesn't pass a key limit, and gets a response with the default (?) limit of 1000 keys back. The response ends with a <IsTruncated>yes</IsTruncated>, then s3cmd issues a second GET request.
DEBUG: Sending request method_string='GET', uri='/?prefix=test/2/&delimiter=/'

Response:
mazonaws.com/doc/2006-03-01/"><Name>borgbackup</Name><Prefix>test/2/</Prefix><MaxKeys>1000</MaxKeys><Marker></Marker><Delimiter>/</Delimiter><Contents> [...] </Contents><IsTruncated>true</IsTruncated><NextMarker>test/2/2-1999</NextMarker></ListBucketResult>

DEBUG: Sending request method_string='GET', uri='/?marker=test/2/2-1999&prefix=test/2/&delimiter=/'

<?xml version="1.0" encoding="UTF-8"?>\n<ListBucketResult xmlns="http://s3.a
mazonaws.com/doc/2006-03-01/"><Name>borgbackup</Name><Prefix>test/2/</Prefix><MaxKeys>1000</MaxKeys><Marker>test/2/2-1999</Marker><Delimiter>/</Delimiter><Contents> [...] </Contents><IsTruncated>false</IsTruncated></ListBucketResult>

rclone, otoh, adds a key limit of 1024, only gets 1000 keys back and thus the server thinks the result is not truncated. Not sure who's at fault here, I'd assume Wasabi?
DEBUG : GET /borgbackup?delimiter=%2F&max-keys=1024&prefix=test%2F2%2F HTTP/1.1

<?xml version="1.0" encoding="UTF-8"?>
<ListBucketResult xmlns="http://s3.amazonaws.com/doc/2006-03-01/"><Name>borgbackup</Name><Prefix>test/2/</Prefix><MaxKeys>1024</MaxKeys><Marker></Marker><Delimit
er>/</Delimiter><Contents>[...]</Contents><IsTruncated>false</IsTruncated></ListBucketResult>

Looking at the S3 API documentation, it says about max-keys:

Sets the maximum number of keys returned in the response body. If you want to retrieve fewer than the default 1,000 keys, you can add this to your request.

1024 is not fewer... seems to me that requesting more than the default is undefinied.

Thanks for your excellent debugging!
Setting the list chunks to 1000 in rclone seems to fix the problem.
Here is a beta with the fix
https://beta.rclone.org/v1.37-118-g2944f760/ (uploaded in 15-30 mins)
I'll report this to my contact at wasabi as it is probably something they should fix there too.

@joschug @ncw - many thanks for your help on this....we'll look into getting this fixed on the Wasabi side and I'll update this git issue once it is has been scheduled & released.

@ncw Thank you for the quick bug fix. I tried beta 1.37-118 and can confirm that it works as expected with the test data set:
 ./rclone -v --transfers 32 sync /tmp/test wasabi:borgbackup/test
2017/09/06 19:00:07 INFO  : S3 bucket borgbackup path test/: Modify window is 1ns
2017/09/06 19:00:37 INFO  : S3 bucket borgbackup path test/: Waiting for checks to finish
2017/09/06 19:00:37 INFO  : S3 bucket borgbackup path test/: Waiting for transfers to finish
2017/09/06 19:00:37 INFO  : Waiting for deletions to finish
2017/09/06 19:00:37 INFO  :
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:              2202
Transferred:            0
Elapsed time:       30.5s

@wavejd Glad I could help. Thanks for looking into it!

@wavejd I tried the original code with Wasabi and this is now fixed at Wasabi too - thanks!
I'm going to close this issue now that it is fixed in both places!


Page.click(selector) or Handle.click throws error with target closed

Tell us about your environment:

Puppeteer version: 1.2
Platform / OS version: Macbook Pro 2017 10.13.3 / Darwin
URLs (if applicable): uit.no/canvas
Node.js version: 9.6.1

What steps will reproduce the problem?
1.
Run this file:
const puppeteer = require('puppeteer');

(async () => {
  const browser = await puppeteer.launch();
  const page = await browser.newPage();
  const response = await page.goto('https://uit.no/canvas')
  const listHandle =
      await page.$('div.selectize-input.items.not-full.has-options')
  const hndle = listHandle.asElement()
  page.click('div.selectize-input.items.not-full.has-options')
  await browser.close();
})();

What is the expected result?
Puppeteer doesn't throw on clicked element.
What happens instead?
(node:77224) UnhandledPromiseRejectionWarning: Error: Protocol error (Runtime.callFunctionOn): Target closed.
    at Promise (/Users/ev1stensberg/Documents/GitHub/canvas-to-gh/node_modules/puppeteer/lib/Connection.js:198:56)
    at new Promise (<anonymous>)
    at CDPSession.send (/Users/ev1stensberg/Documents/GitHub/canvas-to-gh/node_modules/puppeteer/lib/Connection.js:197:12)
    at ExecutionContext.evaluateHandle (/Users/ev1stensberg/Documents/GitHub/canvas-to-gh/node_modules/puppeteer/lib/ExecutionContext.js:71:75)
    at ElementHandle.$ (/Users/ev1stensberg/Documents/GitHub/canvas-to-gh/node_modules/puppeteer/lib/ElementHandle.js:212:50)
    at Frame.$ (/Users/ev1stensberg/Documents/GitHub/canvas-to-gh/node_modules/puppeteer/lib/FrameManager.js:308:34)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:160:7)
(node:77224) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:77224) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

This method scrolls element into view if needed, and then uses page.mouse to click in the center of the element. If the element is detached from DOM, the method throws an error.

But doing the same thing ( queryselect on element ) inside chrome devtools doesn't throw. Why's that? The DOM node is not in shadow DOM

@evenstensberg did this work for you?


Semantic one-shot channel

Different from https://github.com/crossbeam-rs/crossbeam/issues/199, a semantic one-shot channel may help in reasoning.
That said, bounded(1) doesn't mean the channel will handle only one event all the time, but it can buffer at most one event.
A semantic one-shot channel is like a CountDownLaunch with count=1, which means only one event can be delivered and the following will be rejected with an error.
I don't find an implementation in crossbeam-channel. Thus, I file this issue to see if it's a valid suggestion or we can implement it with current structs.

What is the actual purpose of this?
If the purpose is to restrict the API, you might be able to do this just by creating a wrapper type.
If the purpose is performance, it is not clear how large the actual performance benefit is, as I felt the single-capacity optimized variant I tried in https://github.com/crossbeam-rs/crossbeam/issues/199 was similar to the one used in tokio::sync::oneshot.

If the purpose is to restrict the API, you might be able to do this just by creating a wrapper type.

That sounds reasonable...Do you have a demo wrapper for such usage?

I know that I can wrap a OneShotSender that has a fn send(self, ...) now.
Thanks for your well-written library for use!


Close GUI journal with 'E' or 'Esc'

Could you make it so the gui journal closes with E and Esc just like every other inventory menu?
Feels way more convenient than having to click the close button

Unfortunately, this isn't possible. QuestsGUI doesn't track which prompt you're in - it's merely an overlay for the standard text-based editor - and therefore wouldn't know which numerical input to submit for "Exit". The best I could do is add an option that, if you close the inventory, it can send you back to text-based until the next time you open the editor.
Let me know if that's something you'd like to see in the QuestsGUI discussion thread, where all QuestsGUI bugs/suggestions should be directed to:
https://www.spigotmc.org/threads/questsgui-paid.399346/


fix: group by aggregate

I restarted the build for https://github.com/prisma/prisma/pull/6520 to kick in, but tests are still failing on Node 14 in setup.ts.

Hmm that didn't work out. I just merged master, let's see if that helps

@timsuchanek master is red for GitHub Actions after your merge, could you check that now?
https://github.com/prisma/prisma/runs/2324378489

Also fails for https://github.com/prisma/prisma/pull/6528

I'll have a look!

Also fails in https://github.com/prisma/prisma/pull/6217 🤔

Note: Thanks for fixing it with https://github.com/prisma/prisma/commit/e81740b9326569036f0afe20405af13b8fe69773 (it was not related to this PR in the end)


fix gpt_6.7B 'hidden_size'

fix gpt_6.7B 'hidden_size' from 1024 to 4096 in yaml

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.lizhenyun seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


(3.1.0) disabled step over and step into icons in debugger

If you notice, the cursor is over the step over button, but the button is not highlighted and that means that the icon button doesn't work ...

I've had this happen to me a couple of times today..... and using the Debug menu items doesn't appear to reenable the icons when they stop being enabled

bumping this one to critical ... happens pretty much randomly, but today it seems like every third debugger comes up with the icons disabled and using the menu to step is VERY annoying...
Assigning this to @LisaAlmarode so that perhaps work on this particular bug can be prioritzed

Hopefully this has not been sitting around for 2 years still critical!  Reassign to @dalehenrich for updated status, and more information on reproducing it.

Close as old.


Use exec to call standalone.sh / domain.sh

In some cases, it is more usefull to rely on the configured shebang to
parse the configuration and execute the script. This is true when you
want to use $( evaluated expressions ) in java arguments.
One example could to to load the logging manager early in the wildfly
startup process since a javaagent might require it (jmx_explorer
really):
Example use case:
$java_opts = [
  '-Xbootclasspath/p:$( echo ${JBOSS_HOME}/modules/system/layers/base/org/jboss/logmanager/main/jboss-logmanager-*.jar | head -n1 )',
  '-Djboss.modules.system.pkgs=org.jboss.byteman,org.jboss.logmanager',
  '-Djava.util.logging.manager=org.jboss.logmanager.LogManager',
].join(' ')

Coverage remained the same at 74.385% when pulling cfbbd160d7ba472816440e25b05cfb6018ecb0b5 on vrtdev:feature/dynamic_options into 7ea34d96103c203e743dbffcf3c9c50c63247698 on biemond:master.

Coverage remained the same at 74.385% when pulling cfbbd160d7ba472816440e25b05cfb6018ecb0b5 on vrtdev:feature/dynamic_options into 7ea34d96103c203e743dbffcf3c9c50c63247698 on biemond:master.

Coverage remained the same at 74.385% when pulling cfbbd160d7ba472816440e25b05cfb6018ecb0b5 on vrtdev:feature/dynamic_options into 7ea34d96103c203e743dbffcf3c9c50c63247698 on biemond:master.

I'll merge it, but I think your issue would be better addressed here: https://github.com/wildfly/wildfly-core/blob/master/core-feature-pack/src/main/resources/content/docs/contrib/scripts/systemd/launch.sh

I opted to merge PR #228 instead of this one. It pretty much this with a few other features and it's older.


docs: remove docs that suggest Fuchsia support

Fuchsia has not been properly supported for a while now, and the
documentation suggesting so has been a source of confusion. For now,
we will just remove those references.
For more context, see https://fxbug.dev/101134.

Humm... but @behouba just updated all of this and claimed it works in basic form. Deleting all of the updated docs feels somewhat unfair to that contribution. Maybe we just add a bold warning at the top saying it's not fully working and list things that don't work yet?

That's a really good point! Sorry, @behouba , your contribution is greatly appreciated and I didn't mean to discount it.  I'll look into adding a more specific caveat instead.


The Ping class causes a bluescreen if you stop debugging while a ping is still in progress

Everytime a connection to the plc  is established with the Plc.Open method, the IsAvailable property is checked. If the plc is not reachable the ping needs some time to recognize this. If you now stop debugging in Visual Studio you will get a bluescreen. You can also reproduce this by writing a small console application which pings an non reachable address (without using the S7 library).
Although this isn't a bug of the S7 library, I'd be if the library avoids using the Ping class. Instead just try to connect to the provided IP and catch the SocketException. To provide something like "Destination IP-Address is not available" you could check for the ErrorCode 10060 (see https://msdn.microsoft.com/en-us/library/windows/desktop/ms740668(v=vs.85).aspx).
You can find some information about the Ping class bug here:

https://connect.microsoft.com/VisualStudio/feedback/details/721557/bluescreen-process-has-locked-pages-netframework-ping-send
https://stackoverflow.com/questions/17756824/blue-screen-when-using-ping

Let me know if you want to get rid of the Ping class, then I will send you the changes as a pull request.

Ok I will get these merged.


Can I create the probes and detectors plugins by myself？

I greatly appreciate the contributions your team has made to the safety of large models and have some questions I'd like to ask. Our team has developed an application based on a large language model and would like to develop a set of tools for detecting vulnerabilities in large models for internal use. Recently, we came across your team's Garak project and related articles. We hope to build on Garak to do the following: develop probes, develop detectors, and create custom reports. Does the Garak framework support these needs?

Yes, the project supports custom plugin development, and we highly encourages upstream contribution if you feel the plugins will have value to others. You can also work in a private clone if you don't want publish or contribute the custom plugins upstream at this time.
Also watch #384 for progress on when the project completes support for adding custom local user plugins that do not have to be in the installed package folder enabling first class support for bring your own plugin patterns.


[TypedClient] Can't parse SubAggregations

Hello,
I was using the TypedClient to perform a search with multiple aggregations and I was not getting the result I expected.
Example:
Request:
GET /_search?typed_keys=true
{
  "aggs": {
    "genres": {
      "terms": {
        "field": "genre"
      },
      "aggs": {
        "max_play_count": { "max": { "field": "play_count" } }
      }
    }
  }
}

Response :
{
  ...
  "aggregations": {
    "terms#genres": {
      "doc_count_error_upper_bound": 0,
      "sum_other_doc_count": 0,
      "buckets": [
        {
          "key": "electronic",
          "doc_count": 6,
          "max#max_play_count": {
            "value": 40
          }
        },
        {
          "key": "rock",
          "doc_count": 3,
          "max#max_play_count": {
            "value": 20
          }
        },
        {
          "key": "jazz",
          "doc_count": 2,
          "max#max_play_count": {
            "value": 10
          }
        }
      ]
    }
  }
}

But as in the (s *StringTermsBucket) UnmarshalJSON function, the code is expecting an "aggregations" field, it won't be able to parse the reponse correctly.
As @Anaethelion pointed out, it should have something to do with the TypedClient's builder and the implementation of AdditionalProperties
I think that it concerns the class SingleBucketAggregateBase and MultiBucketBase as they both implements
  implements AdditionalProperties<AggregateName, Aggregate>

But unfortunately, I didn't find the builder for the TypedClient so I can't help you to resolve this issue 😕
Thanks,
Ludo

Stumbled upon this today on types.StringTermsBucket and types.MultiTermsBucket. As a result the Aggregation property on buckets is always an empty map.
That's a pretty big bug 😕

This could be closed with the release of 8.7.1

Thanks for noticing, this should have been closed automatically.
Closing now!


Error while adding Material UI component Buttons to Table SuperHeader

Problem description
Have been trying to add a button to the Table Superheader that will dispatch an action on selected elements.
Link to minimal working code that reproduces the issue
        <TableRow key={'abc'}>
           <TableHeaderColumn key={'ggg'}>
             <RaisedButton label="Assign Selected" style={styles.nonSortableLabel}/>
           </TableHeaderColumn>
          </TableRow>
          <TableRow style={muiTheme.tableRow}>
                {An Array Of MUI Flatbuttons which is rendering when superheader is removed}
          </TableRow>
        </TableHeader>

However the table does not render and I get the following error

React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in. Check your code at CustomDatagrid.js:69.
    in CustomDatagrid (created by MuiComponent)
    in MuiComponent (at EditorAssignView.js:16)
    in div (created by List)
    in div (created by Card)
    in div (created by Paper)
    in Paper (created by Card)
    in Card (created by List)
    in div (created by List)
    in List (created by getContext(List))
    in getContext(List) (created by

However this is rendering the button (snippet of above)-->
        <TableRow key={'abc'}>
          <TableHeaderColumn key='aaa'>
            <button> Assign To Self </button>
          </TableHeaderColumn>
        </TableRow>

Cannot really figure out what is causing this to happen. Just wanted to report.
Versions

Material-UI - 0.17.4
React - 15.5.4
Browser - Chrome.

I believe that the issue is right here:

React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in. Check your code at CustomDatagrid.js:69.

Makes sure you are correctly importing the RaisedButton component. Try console.log it for instance, it's undefined.
Also, please submit support requests and questions to StackOverflow using the tag material-ui. StackOverflow is better suited for this kind of support. The issue tracker is for bug reports and feature discussions. See also our CONTRIBUTING guidelines.


[NoTicket] Fix Hera crashing when invalid channel mention supplied to bind

Updated role and user parsing function to also correctly handle this error


Removed people who shouldn't be suggested as friends. Updated import ballots code to support a "state_code" variable.

👍


Not possible to use the bot on private servers?

Hello,
It seems like the bot can't run on private servers even though they are using the FFA mode.
An address for a private server could look like this:
http://agar.io/?ip=<IP_ADDRESS>:443
Any ideas what to change in the source code to allow "custom" or "private" servers?
Regards

I'll look into it later.

//Server hotfix! (Add this somewhere in the launcher.user.js)
if(window.location.search.split("?ip=")[1]){
setTimeout(()=>{connect("ws://"+window.location.search.split("?ip=")[1]);},4000);
}
Also, my browser is complaining about line 360: q = new WebSocket(a); //Apparently your script and agar.io interfere with each other. How about making your script run after the entire page has loaded and everything is done? (Would prevent some future problems...)

When saying "my script" i assume you meant the server i linked to, right?
I guess it could cause errors atm, due to not being online.
Although if you make a pull request or something like that, I can apply it to my fork and test it on private servers.

"//Apparently your script and agar.io interfere with each other. How about making your script run after the entire page has loaded and everything is done? (Would prevent some future problems...)" That was directed at Apostolique..
And, should I make a pull request with the cheap server hotfix?

Doesn't seem to work. I applied your hotfix to the Launcher.user.js

What was the server IP? (I currently don't have any IP that I could use to try it out.) You might want to try increasing the delay.. (The number 4000)

http://agar.io/?ip=<IP_ADDRESS>:443

It seems to work perfectly for me.
if(window.location.search.split("?ip=")[1]){
setTimeout(()=>{connect("ws://"+window.location.search.split("?ip=")[1]);},4000);
}

^^I added that code at the very top of the launcher...
Here is one that waits 14 seconds before connecting.
if(window.location.search.split("?ip=")[1]){
setTimeout(()=>{connect("ws://"+window.location.search.split("?ip=")[1]);},14000);
}

That works for me! Amazing! You should make this a pull request for the official bot :+1:

If you didn't update to the latest launcher, please do, I pushed a fix.


Enable Cargo to work without any network access

Build farms need to build Cargo projects without hitting the network. Assuming that distros rewrite their Cargo.tomls to convert dependencies to local paths, then as far as I know the other source of network access is just updating the index.
re https://internals.rust-lang.org/t/perfecting-rust-packaging-the-plan/2767

It'd also be nice if cargo didn't blow up if it fails to hit the network for downloading the index and instead tries to continue as far as possible.

cc https://github.com/rust-lang/cargo/issues/1882

For solving gecko's case, where they primarily don't want the build servers talking to the network, @alexcrichton favors a cargo package --deployment that creates a single tarball that contains the source code of the whole dag.
Even if we can sell that to gecko, I'm skeptical that Linux distros will like it - they want to use their package system to track Rust packages, and they don't want to duplicate the source of dependencies.

So for Yocto, I would be comfortable with something similar to the Gecko use case. With Yocto the compilation of a package is broken up into several tasks (similar to Gentoo) and once the do_fetch task has ended there shouldn't be anymore network access. This is because Yocto supports users to build in no network access environments and to provide their own source cache which had previously been fetched. To best support this for Cargo built packages I call cargo fetch with a per-package CARGO_HOME set. This usually works but every now and then the sun/moon alignment upsets Cargo and it wants network access again during cargo build.
Hopefully that helps explain my use case and adds some support behind this item.

+1
Maven (build tool for Java) does have offline mode, for example.

Having a node background, this behavior on cargo was unexpected:

npm install installs dependencies
npm install --cache-min Infinity installs dependencies without hitting the network (slight hack)
npm start runs the start script

These are separate steps because there are no guarantees as to where someone might be working from. I think Cargo should be reliable, regardless if people are hacking from the artic circle or enjoying the privilege of a fiber connection.
I think cargo build being build + install is a sane default, but there should be a way to opt-out of this so everyone's use cases can be met.


Fix #24 provide feedback when a request fails

This patch posts an alert when an HDAP HTTP request fails. For example:

It factors out the messages into their own store as several components need to provide this sort of feedback to the user.

Rebasing to resolve merge conflicts, and then will merge to get on with next steps

The change looks okay, but there are test failures. Working on those.


Updated disk-encryption-key-vault.md

Updated warning message to create a key vault is the same region and subscription as that of VM.

@maamleka : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Nice catch. Thank you. #sign-off


Patch proto jar with KtToolchainIdeInfo

I am working on toolchain support for Kotlin. For now this is just to configure the compiler settings but I'll rework how the stdlibs and kotlin_home are managed afterwards.
There will be a KtToolchainIdeInfo added to the TargetIdeInfo.  This only contains a ArtifactLocation to  the real ide_info which is a Json file compiled from a kt_toolchain_ide_info rule which gathers info from the kt_toolchain toolchain. kt_toolchain_ide_info  is a singleton rule defined in io_bazel_rules_kotlin.
The proto jar in the repo seems to be packaged elsewhere. So I am currently rendering KtToolchainIdeInfo reflectively in IdeInfoFromProtobuf. Could the following update be applied to the proto jar.
message KtToolchainIdeInfo {
  ArtifactLocation location=1;
}

message TargetIdeInfo {
  string label = 1 [deprecated = true];
  repeated string dependencies = 4 [deprecated = true];

  // kind is one of {JAVA,ANDROID}_{LIBRARY,BINARY,TEST} and JAVA_IMPORT
  JavaIdeInfo java_ide_info = 7;
  AndroidIdeInfo android_ide_info = 8;

  repeated string tags = 9;
  repeated string runtime_deps = 10 [deprecated = true];

  ArtifactLocation build_file_artifact_location = 11;

  CIdeInfo c_ide_info = 12;
  CToolchainIdeInfo c_toolchain_ide_info = 13;

  string kind_string = 14;

  TestInfo test_info = 15;

  ProtoLibraryLegacyJavaIdeInfo proto_library_legacy_java_ide_info = 16;
  JavaToolchainIdeInfo java_toolchain_ide_info = 17;

  PyIdeInfo py_ide_info = 18;

  TargetKey key = 19;

  repeated Dependency deps = 20;

  reserved 21;

  AndroidSdkIdeInfo android_sdk_ide_info = 22;

  reserved 23;

  repeated string features = 24;

  GoIdeInfo go_ide_info = 25;
  JsIdeInfo js_ide_info = 26;
  TsIdeInfo ts_ide_info = 27;
  DartIdeInfo dart_ide_info = 28;
  KtToolchainIdeInfo kt_toolchain_ide_info = 29;

  // Next available: 30
}

I could update all of the paths to use com.google.devtools.intellij.ideinfo.IntellijIdeInfo instead, but I suspect there is a reason the repackaged version is used ?

@brendandouglas @chaoren I've got toolchains working end-to-end. As mentioned I'm picking up the actual toolchain file from a singleton rule defined in the rule repo. At the moment I am piggy backing the location of the json file via the java toolchain ide info (see) .
Will it take long to patch the proto jar ?

To handle incompatible versions in the classpath, we need to repackage the base protobuf classes. There's no need to repackage our own generated proto classes though (such as IntellijIdeInfo).
I'm making a change now to no longer repackage those classes, which should unblock you in the short term.
We'll also look at exposing our repackaging rule (we didn't do it originally due to missing dependencies, but after a recent refactor the only remaining dep is jarjar, which we can add here). That will take longer though.

@brendandouglas any updates on this one ?

Done


docs: grammar edits, sentence-case secondary headings

A thorough review of the docs' wording and readability. Also changes some heading levels so only one primary heading appears per page. I hope no egos are bruised.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[N/A] Have you written new tests for your changes? Here's an example.
[N/A] Have you successfully run brew tests with your changes locally?

Praise be to @EricFromCanada: King Of The Docs!

I'd like to thank Regular Expressions, without whom none of this would be possible...

Some people, when confronted with a problem, think “I know,
I'll use regular expressions.”  Now they have two problems.

http://regex.info/blog/2006-09-15/247

This programmer, when writing a reStructuredText highlighter, thought "I know, I'll use regular expressions." He had many problems.


feat: add oceanbase-ce module

OceanBase is a distributed relational database developed by Ant Group, and its community edition is open sourced at https://github.com/oceanbase/oceanbase.
There is a docker image maintained by the official team, and in this pull request I add OceanBaseContainer and OceanBaseContainerProvider for a new oceanbase-ce module based on the docker image. https://github.com/oceanbase/oceanbase/tree/master/tools/docker/standalone.

Thanks for your contribution! I have left some suggestions. Please, also take a look at at the build which just failed.

Thank you for your review, I have addressed the comments in the latest commit.
For the CI failure, it seems the container start failed, and I guess the reason may be the memory is not enought. To start the docker container of OceanBase, at least 2 cpu and 8g memory are required.

To start the docker container of OceanBase, at least 2 cpu and 8g memory are required.

I think this is a concern due to IIRC Docker Desktop provides a default of 4g or 8g so it will be at limit all the time not even allowing running other test in parallel. Is there any way to consume less resources. So far, I've seen DB2 consuming ~1.75g

To start the docker container of OceanBase, at least 2 cpu and 8g memory are required.

I think this is a concern due to IIRC Docker Desktop provides a default of 4g or 8g so it will be at limit all the time not even allowing running other test in parallel. Is there any way to consume less resources. So far, I've seen DB2 consuming ~1.75g

We have a plan to achieve lightweight deployment, but it may be difficult to achieve it in the near future.
Now we can't deploy the container on a free GitHub agent. For reference, we use the docker image through testcontainers on a self-hosted CI service in flink cdc.

Now we can't deploy the container on a free GitHub agent.

TBH, I think for now it will keep on hold if that the case 😕

Now we can't deploy the container on a free GitHub agent.

TBH, I think for now it will keep on hold if that the case 😕
Another option is to self-host the module as others do.

OK, I understand. I would convert this pull request to a draft, and we can continue the process after https://github.com/oceanbase/oceanbase/issues/1557 is basically completed.

great! Thanks for your understanding! I'll be following the issue in order to get updates

I updated the container class with the new image oceanbase/oceanbase-ce:<IP_ADDRESS> which can be deployed on GitHub Action now https://hub.docker.com/r/oceanbase/oceanbase-ce.
Currently the startup time is still more than 1 minute, I think we can use 3 min in this version and update it when https://github.com/oceanbase/oceanbase/issues/1573 is released. WDYT? @eddumelendez

Hi @whhe, great to see the improvements. Currently, 1 min is the maximum value we are setting. I would say reaching that limit would produce a not good experience when running tests, of course, it depends how those are set too. IMO, we can wait a little bit more and provide the best experience possible when the other issue is fixed.

Hi @whhe, great to see the improvements. Currently, 1 min is the maximum value we are setting. I would say reaching that limit would produce a not good experience when running tests, of course, it depends how those are set too. IMO, we can wait a little bit more and provide the best experience possible when the other issue is fixed.

Thanks for the suggestion, I think it makes perfect sense, will update this PR with the new image later.

Hi @eddumelendez, I updated this PR with our latest docker image, which can start in less than 30s with FASTBOOT enabled. I think we can go back to the review process now.

@eddumelendez PTAL if you are available.

Hi @eddumelendez, is there any plan for this pr?

Thanks for your contribution, @whhe ! and sorry for the late reply. I have left some comments and some more general:

Rename the module to oceanbase. Is it ok for you?
Package should be org.testcontainers.oceanbase
Run ./gradlew spotlessApply

Thanks for your review and sorry for the late reply, I just came back from the Chinese new year holiday.
I will address the comments soon, but for the module name, I used 'oceanbase-ce' because the community will most likely release a 'oceanbase-xe' docker image in the future for the OceanBase Enterprise Edition. So I think we'd better keep it here.

I think in that case the module can still be oceanbase and contain two implementations OceanbaseCEContainer and OceanbaseXEContainer, it depends in how much difference there is between both of them.

@eddumelendez PR updated, PTAL.

@eddumelendez Look forward to your feedback.

One last change, can we add an entry for oceanbase here, please?

Thanks for your contribution, @whhe !


IPNS entry not republished

If a OpenBazaar store stays online for 7+ days, their IPNS entry will not be republished. The repointer.go has logic to republish message and moderator pointers every 24 hours, but not the IPNS entry.

IPFS handles republishing of IPNS records.

Is this an issue that needs to be pushed up stream?