Make runs only one instance of Cumulative targets

Summary
Homebrew's make command appears to execute only a single instance of a Makefile target when it is specified multiple times with double colons. This appears to happen only when run from a formula, never when run manually on the command-line. I've created a test tap to try to simplify the recreation of the issue.
Survey

[x] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[x] ran a brew command and reproduced the problem with multiple formulae? [I've tested both with a hand-edited formula and with the test tap]
[x] ran brew update and can still reproduce the problem?
[x] ran brew doctor, fixed all issues and can still reproduce the problem?
[x] ran brew config and brew doctor and included their output with your issue?

Configuration
$ brew config
HOMEBREW_VERSION: 1.7.3
ORIGIN: https://github.com/Homebrew/brew
HEAD: 61ace27006a1cb444552c0be10087fcbc3051643
Last commit: No signature
3 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: f6e820d7dac23c14aed1be2067986c081f41b83a
Core tap last commit: 75 minutes ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_GIT: git
CPU: quad-core 64-bit broadwell
Homebrew Ruby: 2.3.7 => /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby
Clang: 10.0 build 1000
Git: 2.17.0 => /usr/bin/git
Curl: 7.54.0 => /usr/bin/curl
Java: N/A
macOS: 10.13.6-x86_64
CLT: <IP_ADDRESS>.1.1527767617
CLT headers: N/A
Xcode: N/A
XQuartz: N/A

$ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Unbrewed dylibs were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected dylibs:
  /usr/local/lib/libcrypto.1.1.dylib
  /usr/local/lib/libgmp.10.dylib
  /usr/local/lib/libhogweed.4.4.dylib
  /usr/local/lib/libnettle.6.4.dylib
  /usr/local/lib/libssl.1.1.dylib

Warning: Unbrewed header files were found in /usr/local/include.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected header files:
  /usr/local/include/gmp.h
  /usr/local/include/nettle/aes.h
  /usr/local/include/nettle/arcfour.h
  /usr/local/include/nettle/arctwo.h
  /usr/local/include/nettle/asn1.h
  /usr/local/include/nettle/base16.h
  /usr/local/include/nettle/base64.h
  /usr/local/include/nettle/bignum.h
  /usr/local/include/nettle/blowfish.h
  /usr/local/include/nettle/buffer.h
  /usr/local/include/nettle/camellia.h
  /usr/local/include/nettle/cast128.h
  /usr/local/include/nettle/cbc.h
  /usr/local/include/nettle/ccm.h
  /usr/local/include/nettle/cfb.h
  /usr/local/include/nettle/chacha-poly1305.h
  /usr/local/include/nettle/chacha.h
  /usr/local/include/nettle/ctr.h
  /usr/local/include/nettle/curve25519.h
  /usr/local/include/nettle/des-compat.h
  /usr/local/include/nettle/des.h
  /usr/local/include/nettle/dsa-compat.h
  /usr/local/include/nettle/dsa.h
  /usr/local/include/nettle/eax.h
  /usr/local/include/nettle/ecc-curve.h
  /usr/local/include/nettle/ecc.h
  /usr/local/include/nettle/ecdsa.h
  /usr/local/include/nettle/eddsa.h
  /usr/local/include/nettle/gcm.h
  /usr/local/include/nettle/gosthash94.h
  /usr/local/include/nettle/hkdf.h
  /usr/local/include/nettle/hmac.h
  /usr/local/include/nettle/knuth-lfib.h
  /usr/local/include/nettle/macros.h
  /usr/local/include/nettle/md2.h
  /usr/local/include/nettle/md4.h
  /usr/local/include/nettle/md5-compat.h
  /usr/local/include/nettle/md5.h
  /usr/local/include/nettle/memops.h
  /usr/local/include/nettle/memxor.h
  /usr/local/include/nettle/nettle-meta.h
  /usr/local/include/nettle/nettle-stdint.h
  /usr/local/include/nettle/nettle-types.h
  /usr/local/include/nettle/pbkdf2.h
  /usr/local/include/nettle/pgp.h
  /usr/local/include/nettle/pkcs1.h
  /usr/local/include/nettle/poly1305.h
  /usr/local/include/nettle/pss-mgf1.h
  /usr/local/include/nettle/pss.h
  /usr/local/include/nettle/realloc.h
  /usr/local/include/nettle/ripemd160.h
  /usr/local/include/nettle/rsa.h
  /usr/local/include/nettle/salsa20.h
  /usr/local/include/nettle/serpent.h
  /usr/local/include/nettle/sexp.h
  /usr/local/include/nettle/sha.h
  /usr/local/include/nettle/sha1.h
  /usr/local/include/nettle/sha2.h
  /usr/local/include/nettle/sha3.h
  /usr/local/include/nettle/twofish.h
  /usr/local/include/nettle/umac.h
  /usr/local/include/nettle/version.h
  /usr/local/include/nettle/yarrow.h
  /usr/local/include/openssl/aes.h
  /usr/local/include/openssl/asn1.h
  /usr/local/include/openssl/asn1_mac.h
  /usr/local/include/openssl/asn1t.h
  /usr/local/include/openssl/async.h
  /usr/local/include/openssl/bio.h
  /usr/local/include/openssl/blowfish.h
  /usr/local/include/openssl/bn.h
  /usr/local/include/openssl/buffer.h
  /usr/local/include/openssl/camellia.h
  /usr/local/include/openssl/cast.h
  /usr/local/include/openssl/cmac.h
  /usr/local/include/openssl/cms.h
  /usr/local/include/openssl/comp.h
  /usr/local/include/openssl/conf.h
  /usr/local/include/openssl/conf_api.h
  /usr/local/include/openssl/crypto.h
  /usr/local/include/openssl/ct.h
  /usr/local/include/openssl/des.h
  /usr/local/include/openssl/dh.h
  /usr/local/include/openssl/dsa.h
  /usr/local/include/openssl/dtls1.h
  /usr/local/include/openssl/e_os2.h
  /usr/local/include/openssl/ebcdic.h
  /usr/local/include/openssl/ec.h
  /usr/local/include/openssl/ecdh.h
  /usr/local/include/openssl/ecdsa.h
  /usr/local/include/openssl/engine.h
  /usr/local/include/openssl/err.h
  /usr/local/include/openssl/evp.h
  /usr/local/include/openssl/hmac.h
  /usr/local/include/openssl/idea.h
  /usr/local/include/openssl/kdf.h
  /usr/local/include/openssl/lhash.h
  /usr/local/include/openssl/md2.h
  /usr/local/include/openssl/md4.h
  /usr/local/include/openssl/md5.h
  /usr/local/include/openssl/mdc2.h
  /usr/local/include/openssl/modes.h
  /usr/local/include/openssl/obj_mac.h
  /usr/local/include/openssl/objects.h
  /usr/local/include/openssl/ocsp.h
  /usr/local/include/openssl/opensslconf.h
  /usr/local/include/openssl/opensslv.h
  /usr/local/include/openssl/ossl_typ.h
  /usr/local/include/openssl/pem.h
  /usr/local/include/openssl/pem2.h
  /usr/local/include/openssl/pkcs12.h
  /usr/local/include/openssl/pkcs7.h
  /usr/local/include/openssl/rand.h
  /usr/local/include/openssl/rc2.h
  /usr/local/include/openssl/rc4.h
  /usr/local/include/openssl/rc5.h
  /usr/local/include/openssl/ripemd.h
  /usr/local/include/openssl/rsa.h
  /usr/local/include/openssl/safestack.h
  /usr/local/include/openssl/seed.h
  /usr/local/include/openssl/sha.h
  /usr/local/include/openssl/srp.h
  /usr/local/include/openssl/srtp.h
  /usr/local/include/openssl/ssl.h
  /usr/local/include/openssl/ssl2.h
  /usr/local/include/openssl/ssl3.h
  /usr/local/include/openssl/stack.h
  /usr/local/include/openssl/symhacks.h
  /usr/local/include/openssl/tls1.h
  /usr/local/include/openssl/ts.h
  /usr/local/include/openssl/txt_db.h
  /usr/local/include/openssl/ui.h
  /usr/local/include/openssl/whrlpool.h
  /usr/local/include/openssl/x509.h
  /usr/local/include/openssl/x509_vfy.h
  /usr/local/include/openssl/x509v3.h

Warning: Unbrewed .la files were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected .la files:
  /usr/local/lib/libgmp.la

Warning: Unbrewed .pc files were found in /usr/local/lib/pkgconfig.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected .pc files:
  /usr/local/lib/pkgconfig/hogweed.pc
  /usr/local/lib/pkgconfig/libcrypto.pc
  /usr/local/lib/pkgconfig/libssl.pc
  /usr/local/lib/pkgconfig/nettle.pc
  /usr/local/lib/pkgconfig/openssl.pc

Warning: Unbrewed static libraries were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected static libraries:
  /usr/local/lib/libcrypto.a
  /usr/local/lib/libgmp.a
  /usr/local/lib/libhogweed.a
  /usr/local/lib/libnettle.a

Pretty sure those extra files have no impact on this issue.
Goals
To help us debug your issue please explain:

What you were trying to do (and why)

I'm working on a formula that installs Perl CPAN modules via cpanminus. I'm doing this to create a tidy, self-contained install of Sqitch.

What happened (include command output)

Homebrew's make command appears to execute only a single instance of a Makefile target when it is specified multiple times with double colons. This appears to happen only when run from a formula, never when run manually on the command-line.

What you expected to happen

I expected all instances of a cumulative target specified with two colons (::) to execute.

Step-by-step reproduction instructions (by running brew commands)

Install the tap and install makefail:
brew tap theory/makefail
brew install --keep-tmp makefail

Sample output:
==> Installing makefail from theory/makefail
==> Downloading https://raw.githubusercontent.com/theory/homebrew-makefail/master/tryme.tgz
Already downloaded: /Users/david/Library/Caches/Homebrew/downloads/36509c8de1f57856d162c29aa3219c6c50900bc352d5485bbb150fab404d727d--tryme.tgz
==> cpanm --quiet --notest --local-lib-contained tryit List::MoreUtils::XS Exporter::Tiny
==> cpanm --verbose --notest --local-lib-contained tryit --installdeps .
Last 15 lines from /Users/david/Library/Logs/Homebrew/makefail/02.cpanm:
Checking if you have List::MoreUtils::XS 0.426 ... Yes (0.428)
Checking if you have ExtUtils::MakeMaker 0 ... Yes (6.66)
Checking if you have Exporter::Tiny 0.038 ... Yes (1.002001)
OK
Building List-MoreUtils-0.428 ... cp lib/List/MoreUtils/Contributing.pod blib/lib/List/MoreUtils/Contributing.pod
cp lib/List/MoreUtils.pm blib/lib/List/MoreUtils.pm
cp lib/List/MoreUtils/PP.pm blib/lib/List/MoreUtils/PP.pm
Appending installation info to /private/tmp/makefail-20180906-25247-4rfpkj/tryme/tryit/lib/perl5/darwin-thread-multi-2level/perllocal.pod
OK
Successfully installed List-MoreUtils-0.428
Installing /private/tmp/makefail-20180906-25247-4rfpkj/tryme/tryit/lib/perl5/darwin-thread-multi-2level/.meta/List-MoreUtils-0.428/MYMETA.json
Installing /private/tmp/makefail-20180906-25247-4rfpkj/tryme/tryit/lib/perl5/darwin-thread-multi-2level/.meta/List-MoreUtils-0.428/install.json
! Installing the dependencies failed: Module 'List::MoreUtils' is not installed
! Bailing out the installation for ..
1 distribution installed
==> Kept temporary files
Temporary files retained at /private/tmp/makefail-20180906-25247-4rfpkj

If reporting this issue please do so at (not Homebrew/brew or Homebrew/core):
https://github.com/theory/homebrew-makefail/issues

Note that cpanminus reports that List::MoreUtils was "successfully
installed", but then later says that it was not. Examine the full log output
in ~/Library/Logs/Homebrew/makefail/02.cpanm (sample).

Switch to the temp directory into which files have been installed, indicated
by the log line that begins with "Entering". In the sample,
its /private/tmp/makefail-20180906-25247-4rfpkj/tryme. Note  that
MoreUtils.pm has not been copied into the tryit directory.
$ find tryit -type f
tryit/lib/perl5/darwin-thread-multi-2level/.meta/List-MoreUtils-XS-0.428/MYMETA.json
tryit/lib/perl5/darwin-thread-multi-2level/.meta/List-MoreUtils-XS-0.428/install.json
tryit/lib/perl5/darwin-thread-multi-2level/.meta/XSLoader-0.24/MYMETA.json
tryit/lib/perl5/darwin-thread-multi-2level/.meta/XSLoader-0.24/install.json
tryit/lib/perl5/darwin-thread-multi-2level/.meta/Exporter-Tiny-1.002001/MYMETA.json
tryit/lib/perl5/darwin-thread-multi-2level/.meta/Exporter-Tiny-1.002001/install.json
tryit/lib/perl5/darwin-thread-multi-2level/.meta/List-MoreUtils-0.428/MYMETA.json
tryit/lib/perl5/darwin-thread-multi-2level/.meta/List-MoreUtils-0.428/install.json
tryit/lib/perl5/darwin-thread-multi-2level/auto/List/MoreUtils/XS/XS.bundle
tryit/lib/perl5/darwin-thread-multi-2level/auto/List/MoreUtils/XS/XS.bs
tryit/lib/perl5/darwin-thread-multi-2level/auto/List/MoreUtils/XS/.packlist
tryit/lib/perl5/darwin-thread-multi-2level/auto/Exporter/Tiny/.packlist
tryit/lib/perl5/darwin-thread-multi-2level/auto/XSLoader/.packlist
tryit/lib/perl5/darwin-thread-multi-2level/List/MoreUtils/XS.pm
tryit/lib/perl5/darwin-thread-multi-2level/perllocal.pod
tryit/lib/perl5/darwin-thread-multi-2level/XSLoader.pm
tryit/lib/perl5/Exporter/Tiny.pm
tryit/lib/perl5/Exporter/Tiny/Manual/Exporting.pod
tryit/lib/perl5/Exporter/Tiny/Manual/QuickStart.pod
tryit/lib/perl5/Exporter/Tiny/Manual/Etc.pod
tryit/lib/perl5/Exporter/Tiny/Manual/Importing.pod
tryit/lib/perl5/Exporter/Shiny.pm

Switch to the work directory reported in 02.cpanm and then into the
List-MoreUtils-0.428 directory and examine List-MoreUtils-0.428/Makefile
(sample). Note that there are two pure_site_install
targets, both specified with a double colon, so make should know to run
them both.  It does
not appear to have been executed.

Run make install manually:
/usr/local/Homebrew/Library/Homebrew/shims/mac/super/make install
Installing /private/tmp/makefail-20180906-25247-4rfpkj/tryme/tryit/lib/perl5/List/MoreUtils.pm
Installing /private/tmp/makefail-20180906-25247-4rfpkj/tryme/tryit/lib/perl5/List/MoreUtils/PP.pm
Installing /private/tmp/makefail-20180906-25247-4rfpkj/tryme/tryit/lib/perl5/List/MoreUtils/Contributing.pod
Appending installation info to /private/tmp/makefail-20180906-25247-4rfpkj/tryme/tryit/lib/perl5/darwin-thread-multi-2level/perllocal.pod

Note that MoreUtils.pm has not been installed. Questions:

Why didn't it get installed by the execution of the formula?
Is there something in the environment that changes the behavior of make?
Is it the case that make only runs one pure_site_install target when run by the formula?

Try again, but turn on debugging output for make install:
brew install --keep-tmp --with-install-debug makefail

The build appears to succeed, but only because of the way cpanminus has
been executed. MoreUtils.pm still will not have been installed. See
~/Library/Logs/Homebrew/makefail/02.cpanm for the complete debug output
sample.

I don't have the make-foo knowledge to understand the debugging output. :-(

I have since added --with-install-verbose, which turns on the Perl installation verbosity. It looks like both pure_site_install targets are called after all, it's just that the modules aren't installed at all. So now I'm less inclined to think it's an issue with Homebrew's make, though it's still weird that I can't replicate the issue when executing the various steps manually. That might be down to something in cpanminus, too. Still futzing with it, and apologies for the noise as I try to work it out.

Finally replicated outside of Homebrew, after dumping Homebrew's environment and adding it to a script. Then I eliminated variables until I found the one that was the problem. This one-liner exhibits the same bug:
env MAKEFLAGS=-j4 cpanm --verbose --notest -L tryit --install-args 'NOECHO= VERBOSE=1' List::MoreUtils

So I don't think this is a Homebrew bug now. So far I can only replicate it with the macOS system Perl, and not any custom Perl builds, so that's weird, but nothing to do with Homebrew. Appreciate your forbearance as I worked through this issue.

@theory Glad you figured it out, thanks for closing and filling out the issue template 👍


Add syntax highlighting / mode for Splunk (SPL)

Hopefully this is the right place to go. I'm an avid user of PasteBin, which appears to use codemirror. I noticed that Splunk / SPL is not currently a supported language mode, so thought I would give it a shot and put a request in here in hopes it could eventually show up as an option within PasteBin.
In case it helps, it appears someone created an SPL definition for highlight JS. I figured I would include it here, maybe it will help with creating a definition in codemirror's system format:
https://github.com/swsoyee/highlightjs-spl
Edit: Huh, I guess Slack uses codemirror as well, so it'd be awesome to see Splunk/SPL formatting available there one day as well 😁

No new modes are added or accepted in this project. The recommended way to build/distribute them is as a separate package.


Use native hexx clockwise, counterclockwise and angle methods

Added in https://github.com/ManevilleF/hexx/pull/16 :D
Blocked on the 0.3 release of hexx.

#292 uses a fresh hexx branch, so this is unblocked.


bumped mantle to ~1.5.0

Mantle 1.5 added a few useful features, namely (for myself):
+ (NSValueTransformer *)mtl_valueMappingTransformerWithDictionary:(NSDictionary *)dictionary defaultValue:(id)defaultValue reverseDefaultValue:(id)reverseDefaultValue;

@spenrose @sgtsquiggs @prachigauriar Thoughts on what to do with this?

@macdrevx Perhaps we under-specify the version number to just ~> 1.4. That way if someone specifies 1.4 or higher (including 1.5+) in their podfile, we’ll use that.

If you don't really care the version than I would do what prachi is saying. Otherwise maybe just bump it to 2.0 or something.

Closing due to inactivity.


HiDPI macOS, mouse position is in physical pixels whereas windows creation happens in logical pixels

I'm using macOS with a HiDPI display, with the scale factor of 2.0.
There is an unfortunate mismatch in the API where the input mouse events are reported in physical pixels (they are scaled according to the scale factor reported by the window), whereas the window creation API naively requests a window of the size of the input values, without specifying the scale factor. A window of those logical pixel dimensions is created, but the physical pixels are double of that.
This leads the user of the library having to scale the mouse values down, or window dimensions up by themselves – and possibly fail to do that, if they are developing with scale factor of 1.0 and don't notice the problem.
Besides, the library documentation speaks mostly of "pixels", but "points" are mentioned too in a few APIs. I guess that the distinction refers to physical and logical pixels, but a explanation of those would be warranted in the documentation itself. (Note: please correct me if I'm wrong! Also, I'd be interested to know how if the distinction was actually considered when writing the docs – do the docs actually reflect some intention of separating those, or was pixel meant to be used as a general term referring both?)
I want to help with the current situation (at least for the macOS implementations), but I'm unaware what plans @tomaka has for the API.
Should a WindowBuilder.with_dimensions(800, 600) create a window of those logical dimensions? In that case, the docs should be changed to refer points, and methods should be added to get the scale factor. Or if with_dimensions was meant to create a window of those physical pixel dimensions, the implementation itself should be fixed, since at the moment it doesn't do that.
I'm going to send PRs once I get input from @tomaka what was the intention.

I want to help with the current situation (at least for the macOS implementations), but I'm unaware what plans @tomaka has for the API.
Should a WindowBuilder.with_dimensions(800, 600) create a window of those logical dimensions?

You might be interested in discussion in #105. I proposed to use physical pixels everywhere.

methods should be added to get the scale factor.

There are Window::hidpi_factor() and MonitorId::get_hidpi_factor() methods to get scale factor of the window and monitor but the one for monitor is currently not implemented for macOS.

Thanks for the pointer. It seems that there is a push for being HiDPI aware by default, with https://github.com/tomaka/winit/pull/319 merged and all. That means that one should expect the window creation api to create windows based on physical pixels, right?

Yes, that's the current intention.

However, because HiDPI-aware windows are meant to adapt to different resolutions when moved to different monitors, the "meant to be" physical pixel dimensions are device-dependent. That means that the window creation API should be reflect that. At the moment it doesn't take in the device the window is going to be drawn on.

@sodiumjoe window creation on macOS uses physical pixels now, right?

@francesca64 I believe that's the case as of https://github.com/tomaka/winit/pull/435

@sodiumjoe that looks like it only affects the size methods though? macOS HiDPI is something I can't test, since it doesn't seem to want to cooperate with my 1440p monitor.

@francesca64 sorry about the late reply. It looks to me like it's still using logical pixels.
The following:
    let _window = winit::WindowBuilder::new()
        .with_title("A fantastic window!")
        .with_dimensions(1440, 900)
        .build(&events_loop)
        .unwrap();

produces a window that fills the screen of my retina 13" macbook pro, which has a native resolution of 2560x1600, but is scaled to 1440x900.
I'm a little confused by logical vs physical, esp. with the default scaling on this machine.

Okay I seem to have been wrong about that: https://github.com/tomaka/winit/issues/105#issuecomment-388220663

I convinced macOS to set one of my monitors to (very low resolution) HiDPI, so I'll be able to take care of this.

Yea this just bit; so is this fixed in a release? I'm on winit 0.18 and I'm pretty sure I still see this issue on macOS retina laptops which use these coordinates for raycasting; do I still have to manually scale for x,y coordinates like this?
                        winit::WindowEvent::CursorMoved {
                            device_id: _,
                            position: winit::dpi::LogicalPosition { x, y },
                            ..
                        } => {
                            let (x, y) = (x as f32, y as f32);


Backport "Three fixes to SAM type handling" to LTS

Backports #21596 to the 3.3.5.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to backport-lts-3.3-21857.
Reference


Setup github actions and add tests

Can be extended with caching and multiple Node.js versions: https://github.com/actions/setup-node

I guess the action won't run until merged to develop, since I don't have admin/write access to this repo :P but it passed on my fork so should be good

Yup! Rn this'll run on all PRs to develop as well as merge (push) to develop


[ja] Update content/ja/docs/concepts/containers/runtime-class.md to the latest.

Fixed content/ja/docs/concepts/containers/runtime-class.md according to the English text, because it is outdated.
Fixes #36821

@riita10069
Thank you for your review.
I updated about your review comment.
Could you please confirm.

@t-inu

CRIの設定 {#cri-configuration}
Need to set this heading ID.

I have given the anchor to it.
Could you please confirm?

@s-kawamura-w664 It's OK. Work fine on the preview.
/lgtm

/assign @nasa9084
Please confirm this PR.


swift-argument-parser repo README link broken

The link for swift-argument-parser repo in the README file goes to a 404 page (https://github.com/apple/swift-argument-parser/blob/master/Documentation/07 Completion Scripts.md).

Fixed by #166

Closing since #166 was merged in


Beacon Settings Caused crash.

Mod source
Factorio Mod Portal
Which mod are you having an issue with?

[ ] pyalienlife
[ ] pyalternativeenergy
[X] pycoalprocessing
[ ] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[ ] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[ ] Graphical
[X] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
Moving Beacon AM FM sliders caused the crash.
34258.123 Error MainLoop.cpp:1391: Exception at tick 201546306: The mod Pyanodons Coal Processing (2.1.8) caused a non-recoverable error.
Please report this error to the mod author.
Error while running event pycoalprocessing::on_gui_click (ID 1)
pycoalprocessing/control.lua:305: attempt to index local 'beacon' (a nil value)
stack traceback:
pycoalprocessing/control.lua:305: in function 'handler'
stdlib/stdlib/event/event.lua:342: in function 'pcall'
stdlib/stdlib/event/event.lua:362: in function 'dispatch_event'
stdlib/stdlib/event/event.lua:438: in function <stdlib/stdlib/event/event.lua:396>
34258.123 Error ServerMultiplayerManager.cpp:92: MultiplayerManager failed: "The mod Pyanodons Coal Processing (2.1.8) caused a non-recoverable error.
Please report this error to the mod author.
Error while running event pycoalprocessing::on_gui_click (ID 1)
pycoalprocessing/control.lua:305: attempt to index local 'beacon' (a nil value)
stack traceback:
pycoalprocessing/control.lua:305: in function 'handler'
stdlib/stdlib/event/event.lua:342: in function 'pcall'
stdlib/stdlib/event/event.lua:362: in function 'dispatch_event'
stdlib/stdlib/event/event.lua:438: in function <stdlib/stdlib/event/event.lua:396>"
Steps to reproduce
No response
Additional context
No response
Log file
No response

Fixed. Thank you.


Passing mandatory args

Is there a way to pass a , such as a file path?

See #44 and #25. This sort of logic is best left up to the user.


can not build

HI,
I got an error;
CMake Error: The source directory "/home/peng/cpplot/build" does not exist.
when using;
cmake . -B build
Any idea about this?
Thank you so much!

What directory are you running this from (it should be home/peng/cpplot)?
Have you tried running mkdir build before this?
What version of cpplot are you using?

I have followed the info
git clone https://github.com/thclark/cpplot
cd cpplot && cmake . -B build
and got the error

Can you answer the questions above, please? It helps OSS maintainers support you if you give clear answers to questions, rather than repeating the original issue.
I suspect that cmake isn't making the build directory for you. Try:
cd /home/peng/cpplot
mkdir build
cmake . -B build

OH, Sorry for the late response.
This problem has been solved. But got another error. I suspect it caused by dependencies.
Thank you.


Support new python oracledb plugin

The cx_Oracle plugin has been renamed to oracledb. This new plugin supports a thin client mode by default which does not require any Oracle libraries unlike the cx_Oracle plugin.

Would indeed be nice to have.

+1
Would be nice to update with this new python oracledb module

Support for oracledb library as exchange for cx_Oracle lib:
https://github.com/kobil-systems/ansible-oracle-modules/commit/69ecaa4c82ea6606ec6ba2183be4d4c6640b03e4


Add reference doc on identities

See live preview at https://awrichar.github.io/firefly/head/reference/identities.html

Codecov Report

Merging #997 (6f8b600) into main (d812ffe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #997   +/-   ##
=======================================
  Coverage   99.98%   99.98%
=======================================
  Files         302      302
  Lines       19834    19834
=======================================
  Hits        19832    19832
  Misses          2        2

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Rebased and added one tweak to include the new verification from https://github.com/hyperledger/firefly/pull/1006.


Merge snorkel/model into snorkel/labeling/model

Removes snorkel/model
Merges currently-used classifier functionality into LabelModel, strips away everything else
Removes log writers, checkpointers, and a lot of logging for LabelModel

Test plan
All unit tests run, manually verified logging output for correctness
+9% repo coverage!

@bhancock8 changes made! @paroma would be great to get comments from you


Activity bar’s border vanishes when it is moved to the right

Environment specs:

VSCode Version: 1.40.2
OS Version: Windows 10 Pro (1909) x64

Steps to Reproduce:

Open VS Code
Open one item of the activity bar, e. g. the SCM tab
Load editor’s global settings.json
Add the following block to it:

    "workbench.colorCustomizations": {
        "activityBar.activeBorder": "#ffffff",
        "activityBar.activeBackground": "#425080"
    }

Save the settings file → activity bar tab appears with background color and border as expected
Now also add the following setting:

    "workbench.sideBar.location": "right"

Save the settings file → activity bar moves to the right and tab still appears with background color (expected), but the white border is suddenly gone (unexpected), cf. this screenshot:

Does this issue occur when all extensions are disabled?: Yes
Thanks for taking a look at this and keep up the great work!

@m2u-84 does this happen with all views or just the SCM view? Does it change if you open a project with Git? I'm not able to reproduce, though I do see that the border is small.


Update dependencies to resolve vulnerabilities in d3-color

Right now a vulnerable package (d3-color@1) is pulled in via a transitive dependency.
Is there any chance of getting an updated package with recent dependencies?
I'd like to help out, but can not run the tests, as there seem to be some files missing.

Sadly, the dependencies cannot be easily upgraded. There are deprecated dependencies and API changes in newer versions of D3. I just spent quite some time trying to get this to run with newer versions and cannot even built the package any more. Could you build it?

I have a project with this dependency (and not much else) where I had success with using the following in package.json:
  "overrides": {
    "d3-array": "3",
    "d3-color": "3",
    "d3-fetch": "3",
    "d3-format": "3",
    "d3-geo": "3",
    "d3-selection": "3",
    "d3-transition": "3"
  }

I upgraded the dependencies, but the code that is built after the upgrade doesn't work any more. I don't see a new release coming any time soon and you're probably better of replacing d3.geomap with a different mapping library.

Ok, understood. Thanks for having a look and trying it out.


I can't exit from full screen

I can't exit from full screen
getHTMLMediaElement.js:194 Uncaught TypeError: document.cancelFullScreen is not a function
at exitFullScreen (getHTMLMediaElement.js:194)
at HTMLDivElement.zoom.onclick (getHTMLMediaElement.js:175)

Hi,
This can be solved by editing getHTMLMediaElement.js
change "exitFullScreen()" to "exitFullscreen()"
just capital "S" to small "s".
This solved the problem.
code :
if (document.fullscreen) {
document.exitFullscreen();
}
if (document.mozFullScreen) {
document.mozCancelFullscreen();
}
if (document.webkitIsFullScreen) {
document.webkitCancelFullscreen();
}
👍


Login and registration pages selection in site settings does not take localization in mind

Description of bug
In site settings, when we select the pages for login and registration, we can only select one page for all languages.
Current result
When using the skin objects, it will link to that language, no matter what your current selected language is.
Affected version

[X ] 9.3.2

I cannot reproduce in 9.4.1, I would suggest starting with updating and opening a new issue with more steps to reproduce if it is still the case in current version.
Things to note:
That list of pages goes by the selected language in the persona bar corner and will only show pages that have a login module on them (or registration depending which dropdown).

Sorry. I'll try separating this into smaller and clear issues.
I can't test 9.4.1 right now  until 2sxc is fixed for dynamic apis.

I haven't noticed this issue in DNN 9.3.2
Please make sure, you switch to each anguage and set the pages appropriately.


?_sort=rowid with _next= returns error

For example:

Go to https://cryptics.eigenfoo.xyz/clues/clues?_next=100 (this is the second page of results in a Datasette site)
Search anything using the FTS search bar. For example, searching for hello will take you to https://cryptics.eigenfoo.xyz/clues/clues?_search=hello&_sort=rowid&_next=100
A 500 Error: list index out of range is raised.

This is because the search URL includes the &_next=100 UTM parameter, carried over from where the FTS search was run. However, there isn't a second page in the search results, so a list index out of range error is raised. You can confirm that removing this UTM parameter from the URL returns the appropriate search results.
The FTS search request should strip any _next UTM parameter.

datasette, version 0.58.1
sqlite-utils, version 3.17

I'll open a separate issue for removing _next= when running a search.


Error we get when running rake db:migrate in step 1 of the migration - DMPonline to DMProadmap migration

Please complete the following fields as applicable:
Expected behaviour:
Run through the migration
Actual behaviour:
Full Error
D, [2019-03-25T10:41:07.666885 #23753] DEBUG -- :    (14.8ms)  SELECT COUNT() FROM languages WHERE languages.default_language = 1
D, [2019-03-25T10:41:07.695116 #23753] DEBUG -- :   Language Load (0.3ms)  SELECT  languages. FROM languages WHERE languages.default_language = 1  ORDER BY languages.id ASC LIMIT 1
rake aborted!
I18n::InvalidLocale: :en_GB is not a valid locale
/var/lib/gems/2.3.0/gems/i18n-0.7.0/lib/i18n.rb:284:in enforce_available_locales!' /var/lib/gems/2.3.0/gems/i18n-0.7.0/lib/i18n.rb:151:in translate'
Steps to reproduce:
Upgrade Step 1
Tasks: transition to Rails 4.2, new user permissions and seed new lookup tables
git fetch --all
git checkout dmponline4_upgrade_step1
git pull origin dmponline4_upgrade_step1 This is not necessary if you have just cloned the repo
Comment out the pg or mysql2 gem in the Gemfile if necessary
bundle install
rake db:migrate
rake migrate:seed
rake migrate:permissions
rake migrate:data_integrity WARNING: this step will replace invalid user email addresses with a temporary email address. If it does, it will record the invalid email and the user info in the log/migration.log file. Please review this log after running the script. You may need to contact the user or their organisational admin to get their correct email address. If this file is empty or missing that means that no invalid email addresses were found and all data migrated successfully.
git stash

Hi,
I'd like to help out with this as I wrote those migrations a few years ago.
After which command did this present the above error?

Thanks, I've resolved this issue

How did you resolve it?
just in case anybody runs into the same problem down the line and are searching the repo for a solution.
Thanks


Bug Report: Contact Us page not working

Reproduction steps
When contact us page is accessed, the validation for it is not working and the data is also not submited.
Expected behavior
When the user clicks on submit button, validation is not working and the data is also not submitted.
Actual Behavior
When the user submits his/her data, the form should be validated and the data should be inserted in the database.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

@vaishnavi2810-code Go for it!

Can you please let me know how can I access the database as I need to create a relation for data to be submitted. The rest of the code is done but it shows me an error that the realtion does not exist.

assign these issues to me


motoc: 0.3.1 -> 0.3.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for motoc is: MOnado Tracking Origin Calibration program
meta.homepage for motoc is: https://github.com/galister/motoc
meta.changelog for motoc is: https://github.com/galister/motoc/releases
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.3 with grep in /nix/store/s35fckrjlhvs3dil6q1sbr31mrq9k9i7-motoc-0.3.3
found 0.3.3 in filename of file in /nix/store/s35fckrjlhvs3dil6q1sbr31mrq9k9i7-motoc-0.3.3

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

motoc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/s35fckrjlhvs3dil6q1sbr31mrq9k9i7-motoc-0.3.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A motoc https://github.com/r-ryantm/nixpkgs/archive/9193757c8e491d87a67c27b6c7ca1617737c2555.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/9193757c8e491d87a67c27b6c7ca1617737c2555#motoc

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/s35fckrjlhvs3dil6q1sbr31mrq9k9i7-motoc-0.3.3
ls -la /nix/store/s35fckrjlhvs3dil6q1sbr31mrq9k9i7-motoc-0.3.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

motoc

Maintainer pings
cc @Pandapip1 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


enable 2fa for rubygems

Howdy
after rest-client/rest-client#713, I think it's wise for all rubygem owners here to enable 2fa on their rubygem accounts.

[x] @michaelglass
[x] @paulca
[ ] @ento

Done. Thanks @michaelglass for the heads up.


Visibility.combine_vis can handle different cunks of a SkyModel

It should be able to handle:

If the SkyModel is split up randomly
If the Observation is split up, e.g. by channel.

Currently, when generating random sources, the Imager fails with the following error:
Traceback (most recent call last):
  File "/users/vtimmel/karabo/Karabo-Pipeline/karabo/examples/time_karabo.py", line 156, in <module>
    main(n_random_sources=200000)
  File "/users/vtimmel/karabo/Karabo-Pipeline/karabo/examples/time_karabo.py", line 104, in main
    deconvolved, restored, residual = imager_askap.imaging_rascil(
  File "/users/vtimmel/karabo/Karabo-Pipeline/karabo/imaging/imager.py", line 224, in imaging_rascil
    blockviss = create_visibility_from_ms_rsexecute(
  File "/users/vtimmel/miniconda3/envs/karabo_dev_env/lib/python3.9/site-packages/rascil/workflows/rsexecute/visibility/visibility_rsexecute.py", line 46, in create_visibility_from_ms_rsexecute
    [
  File "/users/vtimmel/miniconda3/envs/karabo_dev_env/lib/python3.9/site-packages/rascil/workflows/rsexecute/visibility/visibility_rsexecute.py", line 47, in <listcomp>
    rsexecute.execute(create_visibility_from_ms, nout=nout)(
  File "/users/vtimmel/miniconda3/envs/karabo_dev_env/lib/python3.9/site-packages/rascil/processing_components/visibility/base.py", line 465, in create_visibility_from_ms
    ms_vis = ms.getcolslice(datacolumn, blc=blc, trc=trc)
  File "/users/vtimmel/miniconda3/envs/karabo_dev_env/lib/python3.9/site-packages/casacore/tables/table.py", line 1077, in getcolslice
    return self._getcolslice(columnname, blc, trc, inc,
RuntimeError: Slicer error:infer: startResult>=shape

When using a real sky, this does not happen. We should make sure that this error is OK, because it's random sources and not that it is an error by combining the visibilities.

Consitent result, should be checked by a test.
@kenfus will send @rohitcbscient visiblities generated by karabo parallelisation.

The idea of using chunks in the skymodel should be motivated by user science case, not a general feature.


Update conformer energy notebook and CLI to work for single conf molecule

Previously the conformer energy tool wouldn't work for single-conformer molecules. This updates it to handle single-conformer molecules.
I don't know how to test this. In the future we might include a separate set of CI tests for CLI utilities. But for now I'd just like to push a quick fix, since we're getting near industry partner renewal season.

Codecov Report

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

✅  broken on master
✅  works with patch

Click for log!
(openff-dev) [conformer_energies] python                                                                                master  ✱
Python 3.7.6 | packaged by conda-forge | (default, Jun  1 2020, 18:33:30)
[Clang 9.0.1 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from openforcefield.topology import Molecule
>>> mol = Molecule.from_smiles('CCOCC#N')
>>> mol.generate_conformers()
>>> mol.to_file('mol.sdf', file_format='sdf')
>>> exit()
(openff-dev) [conformer_energies] time python conformer_energies.py --f mol.sdf                                                                                master  ✱
Traceback (most recent call last):
  File "conformer_energies.py", line 93, in <module>
    compute_conformer_energies_from_file(args.filename)
  File "conformer_energies.py", line 18, in compute_conformer_energies_from_file
    molecules = [loaded_molecules[0]]
TypeError: 'Molecule' object is not subscriptable
python conformer_energies.py --f mol.sdf  1.13s user 0.33s system 92% cpu 1.579 total
(openff-dev) [conformer_energies] git checkout conformer-energies-single-mol                                                                                master  ✱
Branch 'conformer-energies-single-mol' set up to track remote branch 'conformer-energies-single-mol' from 'upstream'.
Switched to a new branch 'conformer-energies-single-mol'
(openff-dev) [conformer_energies] time python conformer_energies.py --f mol.sdf                                                             conformer-energies-single-mol  ✱
1 unique molecule(s) loaded, with 1 total conformers
C4H7N1O1 : 1 conformers
Parametrizing C4H7N1O1 (may take a moment to calculate charges)
Conformer         Initial PE         Minimized PE       RMS between initial and minimized conformer
    1 /     1 :    6.736 kcal/mol    5.721 kcal/mol     0.037 Angstroms
python conformer_energies.py --f mol.sdf  1.87s user 0.42s system 94% cpu 2.429 total
(openff-dev) [conformer_energies] cat mol.sdf                                                                                conformer-energies-single-mol  ✱

  -OEChem-06242010573D

 13 12  0     0  0  0  0  0  0999 V2000
    1.0616   -0.2681   -0.0006 C   0  0  0  0  0  0  0  0  0  0  0  0
    1.9101    0.9126   -0.4220 C   0  0  0  0  0  0  0  0  0  0  0  0
    1.5170    1.3236   -1.7228 O   0  0  0  0  0  0  0  0  0  0  0  0
    2.2819    2.4338   -2.1743 C   0  0  0  0  0  0  0  0  0  0  0  0
    1.8570    2.8363   -3.5149 C   0  0  0  0  0  0  0  0  0  0  0  0
    1.5209    3.1562   -4.5783 N   0  0  0  0  0  0  0  0  0  0  0  0
    1.3393   -0.6108    1.0001 H   0  0  0  0  0  0  0  0  0  0  0  0
   -0.0000   -0.0002   -0.0006 H   0  0  0  0  0  0  0  0  0  0  0  0
    1.1801   -1.0990   -0.7040 H   0  0  0  0  0  0  0  0  0  0  0  0
    2.9684    0.6361   -0.4446 H   0  0  0  0  0  0  0  0  0  0  0  0
    1.7730    1.7499    0.2687 H   0  0  0  0  0  0  0  0  0  0  0  0
    3.3347    2.1371   -2.1769 H   0  0  0  0  0  0  0  0  0  0  0  0
    2.1633    3.2516   -1.4571 H   0  0  0  0  0  0  0  0  0  0  0  0
  1  2  1  0  0  0  0
  2  3  1  0  0  0  0
  3  4  1  0  0  0  0
  4  5  1  0  0  0  0
  5  6  3  0  0  0  0
  1  7  1  0  0  0  0
  1  8  1  0  0  0  0
  1  9  1  0  0  0  0
  2 10  1  0  0  0  0
  2 11  1  0  0  0  0
  4 12  1  0  0  0  0
  4 13  1  0  0  0  0
M  END
$$$$```

</details>

Thanks, Matt!


chore(ansible): don't update cache when installing curl

DO NOT MERGE: Testing if build works

@am I see this PR has been opened about a month ago, I'd like to close it as I'm not sure this is required at this time.


BUG: Not all available transformation heuristics function correctly.

euro, range, and string transform heuristics appear to be broken

euro replaces commas with full stops/periods. It should be the opposite
range produces NaN for single values. Is this correct?
string produces the string 'missing value' - Is this (still) correct?

After discussion:

string should be removed
euro is fine
range should be removed for (now perhaps revisit later?)


Loading JKS file written by OpenJDK Java 21 results in ASN.1 error

I'm not sure whether this issue stems from pyjks or pyasn1. I might post it in both projects.
If I use jks.KeyStore.load on a JKS file containing a certificate/key bunde that was imported using keytool from OpenJDK Java 21 (java-21-openjdk-<IP_ADDRESS>.9-1.fc40.x86_64), then an ASN.1 error occurs:
>>> import jks
>>> jks.KeyStore.load('javakeystore21.jks', '[PASSWORD]')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/username/.local/lib/python3.12/site-packages/jks/util.py", line 90, in load
    ret = cls.loads(input_bytes,
          ^^^^^^^^^^^^^^^^^^^^^^
  File "/home/username/.local/lib/python3.12/site-packages/jks/jks.py", line 525, in loads
    entry.decrypt(store_password)
  File "/home/username/.local/lib/python3.12/site-packages/jks/jks.py", line 197, in decrypt
    algo_params = encrypted_info['encryptionAlgorithm']['parameters'].asOctets()
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/username/.local/lib/python3.12/site-packages/pyasn1/type/univ.py", line 915, in asOctets
    return bytes(self._value)
           ^^^^^^^^^^^^^^^^^^
  File "/home/username/.local/lib/python3.12/site-packages/pyasn1/type/base.py", line 213, in plug
    raise error.PyAsn1Error('Attempted "%s" operation on ASN.1 schema object' % name)
pyasn1.error.PyAsn1Error: Attempted "__index__" operation on ASN.1 schema object

If I instead import the same SSL bundle using keytool from OpenJDK Java 17 (java-17-openjdk-<IP_ADDRESS>.9-1.fc40.x86_64), then loading it works fine:
>>> import jks
>>> jks.KeyStore.load('javakeystore17.jks', '[PASSWORD]')
<jks.jks.KeyStore object at 0x7f04826bd6a0>

This happens with pyjks 20.0.0 and pyasn1 0.6.0 on Python 3.12.
The keytool import command that I run looks like this:
keytool -importkeystore -srckeystore bundle.p12 -srcstoretype pkcs12 -srcalias mybundle -destkeystore javakeystore.jks -destalias mybundle
I have a feeling that OpenJDK 21's keytool command does not give encrypted private keys any encryption algorithm parameters when they're placed into JKS files. When I compare javakeystore21.jks and javakeystore17.jks using keytool -list -v, they're exactly the same. But the two files are slightly different sizes. And if I use the strings command on the two of them, it looks like the encrypted private key under mybundle is quite different.

Thanks for reaching out with the excellent report, Paul. Unfortunately, the maintainers of this package (myself included) haven't touched a JKS in a long while. If you make any progress, we're happy to review a PR with a test/fix. Thanks again!


Variable explorer search button keeps pressed after hiding the search field with Esc

@dalthviz, this is a small bug I noticed after merging PR #9384: if you hide the search field with Esc, then the search button keeps pressed in the interface, as shown in this screenshot (I took it in the light interface theme because it's easier to see):

However, that shouldn't be the case because it's confusing. So after pressing Esc the search button also needs to appear as not pressed.

Should we instead of the Search: label show and x button as in the Find widget? What do you guys think @ccordoba12 @goanpeca ?
Variable Explorer:

Find Widget:

I'm +1 so that users quickly realize that the new field in the Variable Explorer is just like any other search field.

I like the idea of keeping the search/find uniform so go for it :-)


fuse: Handle/format all node types correctly

Closes #1463

Codecov Report

Merging #1465 into master will decrease coverage by 0.29%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #1465     +/-   ##
=========================================
- Coverage   52.81%   52.51%   -0.3%
=========================================
  Files         136      137      +1
  Lines       10859    10896     +37
=========================================
- Hits         5735     5722     -13
- Misses       4247     4303     +56
+ Partials      877      871      -6

Impacted Files
Coverage Δ

internal/restic/node.go
40.93% <0%> (-1.03%)
:arrow_down:

cmd/restic/format.go
16.66% <0%> (-3.34%)
:arrow_down:

internal/fuse/other.go
0% <0%> (ø)

internal/fuse/dir.go
0% <0%> (ø)
:arrow_up:

internal/checker/checker.go
67.55% <0%> (-3.78%)
:arrow_down:

internal/archiver/archiver.go
65.03% <0%> (+0.21%)
:arrow_up:

internal/backend/s3/s3.go
64.74% <0%> (+1.07%)
:arrow_up:

Continue to review full report at Codecov.

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


ClearLeds Error on Petal and Versio

User ArguZ on the Daisy forum has run into an issue with the Petal and Versio targets.

Both targets produce compiler errors


tf.StridedSlice op doc fix

I think that the source of truth at the moment is third_party/tensorflow/core/api_def/base_api/api_def_StridedSlice.pbtxt (we actually have a tool to generate the ODS from the proto registration)

api_def_StridedSlice.pbtxt updated


Add dummy AppStream repository

What does this PR change?
Adds a Salt state, for the standard server, setting up a local dummy AppStream repository to be used by the testsuite.

@NamelessOne91 do we need a port to 5.0? Probably, no?


New Private Project - Add "Columns" as one part to be copied

Expected behaviour
New Private Project - Add "Columns" as one part to be copied
Actual behaviour
Currently when we try to create new private project, it shows "Which parts of the project do you want to duplicate?" containing existing parts. These parts does not list out "Columns". Can we add a feature to allow copying that as well?
Steps to reproduce

Configuration
Copy and paste the configuration section from the Kanboard settings page or fill these fields:

Kanboard version:1.0.26
Database type and version:
PHP version:
OS:
Browser:

When you duplicate a project, you always duplicate the columns.

Sorry. My bad. This is working as you said.


Fix incomplete kwargs of resource logs (#3793)

Pull request opened by the merge tool on behalf of #3793

Merged into branches iso4 in 0e9e0d41df3826de0b09616abaa55df9d249abf3


Installing on Apple TV 4K 2021: "This app was not built to support this device family"

I wouldn't be surprised if this wasn't made to work with the Apple TV 4K 2021 A12 model, but installing the 8.0.1 IPA anyway shows this (with AltServer Linux):
Writing File: /tmp/4020021b-0b81-4f89-ae2b-dbdfe91411bd/misaka.app/mkdir to: PublicStaging/misaka.app/mkdir
Installation Progress: 0.745989
Writing File: /tmp/4020021b-0b81-4f89-ae2b-dbdfe91411bd/misaka.app/opainject to: PublicStaging/misaka.app/opainject
Installation Progress: 0.747995
Writing File: /tmp/4020021b-0b81-4f89-ae2b-dbdfe91411bd/misaka.app/sudo to: PublicStaging/misaka.app/sudo
Installation Progress: 0.75
Finished writing to device.
Installation Progress: 0.7625
Installation Progress: 0.7875
Installation Progress: 0.8
Installation Progress: 0.825
Alert: Could not install misaka-v8.0.1.ipa to unknown.
    This app was not built to support this device family; app is compatible with (
    1,
    2
) but this device supports (
    3
)

If this is not the way to install on Apple TV then what does "AppleTV Support" mean..??

Misaka for Apple TV released, still need to find KFD offsets for AppleTV11,1.


Support for input-group with input-group-addon element

Context
When using TypeAhead on a search element, it would be nice to display a search icon next to the <input/> element, where this fiddle shows a few approaches on how to realize that.
Problem
When using the simplest approach: an input-group with an icon wrapped in a input-group-addon, this seems to interfere with TypeAhead.
This HTML

        <input class="typeahead form-control" type="text" placeholder="Search...">

will be transformed by TypeAhead to something like:

            <input class="typeahead form-control" type="text" placeholder="Search...">

...

Notice that the menu that contains the suggestion becomes part of the input-group element making it:

break the rounding of the left border of the <input/> element
overlap the <input/> element. This makes it difficult for the user to continue its search adventure.

Would it be possible to use plain CSS to prevent the menu from overlapping?
Seems to be related to:

https://github.com/twitter/typeahead.js/issues/847
https://github.com/twitter/typeahead.js/issues/1049

UPDATE
As suggested in this comment adding style="float: none;" to the input element seems to resolve the overlapping issue for Safari, FireFox, Chrome, Internet Explorer 11, Microsoft Edge. Have not tested Internet Explorer < 11 yet.

due to the position: relative of the input, try:
.tt-menu.tt-open {
  top: 34px !important;
}

mm, ok, seems to work indeed.
When looking at this part of the source code, is it possible the class tt-dropdown-menu changed to tt-menu?
It seems https://github.com/bassjobsen/typeahead.js-bootstrap-css/pull/12 did not completely cover https://github.com/bassjobsen/typeahead.js-bootstrap-css/issues/11.
Instead of supporting:

tt-dropdown-menu for TypeAhead < 0.11
tt-menu for TypeAhead >= 0.11

it might be beter to release a new version which bumps the required version of TypeAhead to >=0.11?

yes, thanks.
You're right about that. I fixed this with https://github.com/bassjobsen/typeahead.js-bootstrap-css/commit/e691715a3af04d374c9f8efe719addb4e233cf60
I also agree that the next release should support only >= -0.11

Great, thanks for the swift support.
Since this change is backwards compatible, would it justify a new release with increased patch version? 1.2.1?


Client Allocation Request for: LeoCheung

Client Allocation Questions
Core Information

Name: LeoCheung
Website / Social Media: https://github.com/LeoZCH
Region: Greater China
DataCap Requested: 100TiB
Addresses to be Notarized: f3v7x4a2aapgx6o2r477tenoin3u5oadaeqyd7kjdsitykvf4ok7vq2utcyh34lmu5u7oybs25ff6s4dbudpma
Notary Requested: s0nik42

@LeoZCH Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

this DataCap is for Slingshot repair project, please review it


login with 2fa otp auth

My account have 2fa enabled via google authenticator.
How can i login with this lib in this case (i can generate 2fa code via go package based on secret)

Haven't looked at that yet, will put it on the todo!

I have updated the wiki with the following example:
...

err := insta.Login()
if err == goinsta.Err2FA {
  err2FA := insta.TwoFactorInfo.Login2FA(code)
  if err2FA != nil {
    ...
  }
} else if err != nil {
  ...
}

Can you let me know if this works for you?
Also, which go package do you use to generate the 2FA code?


API change: Remove template usage in NearCacheConfig and EvictionConfig

Removed template usage in NearCacheConfig and EvictionConfig.

Removed EvictionPolicyEvaluator support for this release. The near cache will be reimplemented and we can add this in a more proper way later.

Changed ClientConfig to use stack copied NearCacheConfig.

fixes https://github.com/hazelcast/hazelcast-cpp-client/issues/648
fixes https://github.com/hazelcast/hazelcast-cpp-client/issues/654

Linux test FAILed.

Linux test PASSed.

Linux test PASSed.

Windows test FAILed.

Windows test FAILed.

Windows test FAILed.

Windows test PASSed.

Linux test PASSed.


computeThreshold is inconsistent

computeThreshold function does not calculate a threshold index (which then can be multiplied by maxValue) but gives a threshold value instead

The issue was solved in the following commit:  8ca381f
Related issue: #374


feat(services): add NewRelic service

Hi folks,
My company is in the process of rolling out New Relic internally and I saw an opportunity for adding a new notification service for it to integrate with its Deployment Marker functionality.
Not sure if the contributing process involving opening up an issue first, etc, so just let me know if changes are need to be made.
Also, pretty new to the Argoproj codebase, so let me know if I missed any aspect of adding a new notification service, or if there are any other considerations/suggestions. Haven't tested e2e, not sure the best way of doing so within this codebase or integrated with the ArgoCD one.
Issue: #93

@alexmt Just making sure I'm following the right approach here

@ryota-sakamoto Maybe you can shed some light on the contributing process here?

Hey @saviogl , i am going to review it

@pasha-codefresh Thanks for getting back to me, I wasn't sure if this repository was still active, or how I could follow up on the contribution process. I had left a few messages back in the day on Slack, but to no avail.
Glad that we are coming around to it, it's been a while so I might need a refresher myself. I ended up using Webhooks Notifications Service to setup the foundation for the company that I worked for but it would be great to have this abstraction as a supported service.
Let me know if you have any questions...
On another note, If I wanted to get more involved with the argo project, where would be the best place to start?

ArgoCD is good candidate, but notification is also part of it, i would say that notification also great place for beginning.
About PR, the code itself looks good to me, I just want run a manual test on it today, and after we can merge

@saviogl i got it working, great job, lets just fix small issues in documentation

Hey @saviogl , do you need some help with solve comments that i provided?

Hi @pasha-codefresh, I'll be looking at this today, and I'll let you know if I come across any questions? Are you in CNCF Slack workspace?

Yep @saviogl , messaged you in slack

Great job, thank you!


Remove empty localStorage entries

Make sure that EasyMDE do not create empty localStorage entries.

Hi @Ionaru, do you know when this could be merged? :)


494 data tool fullerton

@LizNobriga confirmed this looks good.
Pinged Jon Jensen @xjensen about connecting up our usual PR review tool here


New resource: device image upload

https://www.juniper.net/documentation/us/en/software/mist/api/http/api/sites/devices/add-site-device-image

added in v0.1.44


DSP-1052 : Migration task to replace empty strings with dummy "FIXME"

resolves DSP-1052

This looks good, but you forgot to change KnoraBaseVersion in webapi/package.scala.

Thanks for pointing this out and reviewing the PR.

@SepidehAlassi could you maybe have look at the upgrade integration test run on Github-CI? The test "passes", despite an error being thrown.

@SepidehAlassi could you maybe have look at the upgrade integration test run on Github-CI? The test "passes", despite an error being thrown.

I guess it is a bug with Github CI. If you have a look at the commits 6fafd94 and e54e4d9, there the upgrade integration test has failed. In the commit 05d532f, I have just added an empty line and all of a sudden upgrade intergration test passes.

@benjamingeer & @subotic Can I merge this now?

Looks good to me, thanks very much for doing this!

Looks good to me, thanks very much for doing this!

Can you please approve the changes then? Merging is blocked.


added skipping auto enrollment info

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@greg-lindsay - Can you please assist to check, approve and merge the content for this PR based on issue #3263?
Thank you.

@greg-lindsay - I would like to follow up. Can you please assist to check, approve and merge the content for this PR based on issue #3263?
Thank you.

@greg-lindsay - We would like to follow up. Can you please assist to check, approve and merge the content of this PR?
Thank you.


Restrict Splitter when layout cannot shrink any more

Fixed in feature branch "ValueTree" - will be merged, when stretch issue is solved

Merged into master


Expose tests in series from API endpoint

Problem
The question I ultimately want answered is "Given a date, what was the SHA of HEAD on ?". I want to avoid, e.g., merge commits. Basically I want HEAD@{<date>} but without the reflog. (This is in the service of running regression tests and comparing the performance of, e.g., trunk 2 weeks ago to trunk today.)
With some massaging, you can get a lossy version of this information from a long-running cstar_perf instance. Given data returned from <tests/artifacts/<test_id>/stats, you can tell that trunk was at sha data['revisions'][n]['git_id'][<blade_name>] if data['revisions'][n]['revision'] is 'apache/trunk'. So, each SHA meeting that requirement was checked out as HEAD on trunk at some point.
Solutions
The missing part of this is an API endpoint that provides UUIDs to use as <test_id>.

The most basic version of this is something like an /api/tests endpoint that gives back the results of SELECT test_id FROM tests. From there the client making the request can find the time from the UUIDs returned and choose tests to query.
It would be about as easy to also provide a UUID or timestamp as an endpoint, or multiple endpoints, as in the proposed series changes.

Both of those changes would be pretty easy as they just query over the tests table and return test_id values. They'd require no changes to the data model. Deciding between the two, as far as I can tell, is just a question of whether either causes problems for the test database. I don't think either will.
Other possibilities that might make this simpler for a client, but I think aren't useful enough in other cases to change cstar_perf for:

Push the logic for determining the SHA of HEAD on a branch on a given date or in a given test into the cstar_perf app.
Push information about SHAs or branches into the database.

The benefits of those approaches are that even if the data returned from the /stats endpoint changes, client code won't break as long as the SHA/branch API stays the same. I don't think that's worth the effort at this stage, since we're the only people building clients for this information.

@mambocab So, what we need is basicaly only 1 API endpoint:

/api/tests (with timestamps filter if the DB allows it)

The endpoint would only return the UUID (test_id) of the test. It would be easy to add /api/tests/:UUID to return all information in the future. (or now?)

Yeah, ultimately, I can do everything I need with an /api/tests endpoint (optionally with timestamps for filtering) that returns a list of all the test UUIDs.

On it.

@mambocab I cannot add a filter by timestamp for this endpoint due to how the schema is. The scheduled_date is in fact the test_id column (timeuuid) and since it's the primary key I can only filter with EQ and IN operator.
However, since we are already forced to fetch all test_id, I will apply the filter on the backend endpoint rather than directly in the DB. It will avoid returning extra data on each request.

Just to make sure I understand: you can't filter in the database query, but you can filter at the application layer? Sounds good to me.

Yes, exact.

Yeah, ultimately, I can do everything I need with an /api/tests endpoint (optionally with timestamps for filtering) that returns a list of all the test UUIDs.

How do you plan to pass the from/to filters? timestamp or human date?

I don't have strong feelings -- timestamps are what I had in mind, but probably just because that's what the API for series requests uses. As far as I'm concerned, do what you think is best.

Fixed


Processing Error

I'm running the latest release, but the program stopped at this error:
Process ID=03148: Unhandled Exception: System.IO.FileNotFoundException: Could not load file or assembly 'Base, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies. The system cannot find the file specified.
Process ID=03148:    at CombineMS2.Program.Main(String[] args)

Another error happend when converting O-glycan gdb from gwp
Traceback (most recent call last):
File "glycoworkbench_to_gdb.py", line 211, in
File "glycoworkbench_to_gdb.py", line 196, in ReadOneGWP
File "glycoworkbench_to_gdb.py", line 116, in generate_subtree_by_gwb
File "glycoworkbench_to_gdb.py", line 93, in generate_subtree_by_gwb
UnboundLocalError: local variable 'i' referenced before assignment
[15384] Failed to execute script 'glycoworkbench_to_gdb' due to unhandled exception!

I'm running the latest release, but the program stopped at this error: Process ID=03148: Unhandled Exception: System.IO.FileNotFoundException: Could not load file or assembly 'Base, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies. The system cannot find the file specified. Process ID=03148: at CombineMS2.Program.Main(String[] args)

This seems like .NET version's problem, check dotnet --version

Another error happend when converting O-glycan gdb from gwp Traceback (most recent call last): File "glycoworkbench_to_gdb.py", line 211, in File "glycoworkbench_to_gdb.py", line 196, in ReadOneGWP File "glycoworkbench_to_gdb.py", line 116, in generate_subtree_by_gwb File "glycoworkbench_to_gdb.py", line 93, in generate_subtree_by_gwb UnboundLocalError: local variable 'i' referenced before assignment [15384] Failed to execute script 'glycoworkbench_to_gdb' due to unhandled exception!

This is indeed a but due to I changed the python version from 2.7 to 3.9. You can download it from the release page now: https://github.com/pFindStudio/pGlyco3/releases

I'm running the latest release, but the program stopped at this error: Process ID=03148: Unhandled Exception: System.IO.FileNotFoundException: Could not load file or assembly 'Base, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies. The system cannot find the file specified. Process ID=03148: at CombineMS2.Program.Main(String[] args)

This seems like .NET version's problem, check dotnet --version

I checked I installed .NET 6.0 and 8.0, with 3.5 feature enabled in Windows 10. Could you please tell me what is the targeting version of pGlyco3?

Another error happend when converting O-glycan gdb from gwp Traceback (most recent call last): File "glycoworkbench_to_gdb.py", line 211, in File "glycoworkbench_to_gdb.py", line 196, in ReadOneGWP File "glycoworkbench_to_gdb.py", line 116, in generate_subtree_by_gwb File "glycoworkbench_to_gdb.py", line 93, in generate_subtree_by_gwb UnboundLocalError: local variable 'i' referenced before assignment [15384] Failed to execute script 'glycoworkbench_to_gdb' due to unhandled exception!

This is indeed a bug due to I changed the python version from 2.7 to 3.9. I have fixed it int he new version. You can download it from the release page now with pyinstaller-6.0.0 from https://github.com/pFindStudio/pGlyco3/releases

I started by testing conversion of N-glycan database this time, but it encountered a new error below.
Already registered!
N
Traceback (most recent call last):
File "glycoworkbench_to_gdb.py", line 211, in
File "glycoworkbench_to_gdb.py", line 196, in ReadOneGWP
File "glycoworkbench_to_gdb.py", line 93, in generate_subtree_by_gwb
UnboundLocalError: local variable 'j' referenced before assignment
[2916] Failed to execute script 'glycoworkbench_to_gdb' due to unhandled exception!

I'm running the latest release, but the program stopped at this error: Process ID=03148: Unhandled Exception: System.IO.FileNotFoundException: Could not load file or assembly 'Base, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies. The system cannot find the file specified. Process ID=03148: at CombineMS2.Program.Main(String[] args)

This seems like .NET version's problem, check dotnet --version

I checked I installed .NET 6.0 and 8.0, with 3.5 feature enabled in Windows 10. Could you please tell me what is the targeting version of pGlyco3?

I think all .NET framework versions fine, there may be some issues on diff version of .NET core


Failed to create docker container with error: API error (500): No command specified

With the help of kubernetes guestbook, I changed the image value and modified the imagepullpolicy as follows
"image":"suse13.1"
"imagepullpolicy":"never"

Since I am in China, I can't pull "gcr.io/google_container/pause:0.8.0". My friend in the USA pulled an image "Kubernetes/pause" for me and I specified --pod_infra_container_image="Kubernetes/pause"
But it failed and kubelet log says:

E0713 14:20:34.182718   48272 manager.go:1556] Failed to create pod infra container: API error >?(500): No command specified; Skipping pod "suseapache_default"

I can't understand this.

I don't know if your capiltalization is just a language thing or literal, but it does matter:
"image":"suse13.1"
"imagepullpolicy":"never"

is wrong
"image":"suse13.1"
"imagePullPolicy":"Never"

is correct

I'm sorry that I made a mistake when asking this question but I do used the correct one in my json file.
I found that when I pull the "kubernetes/pause" image from dockerhub then the image was useful. But when I saved the image for another computer which have no access to the Internet, kubernetes failed to create container again.

E0713 14:20:34.182718 48272 manager.go:1556] Failed to create pod infra container: API error >?(500): No command specified; Skipping pod "suseapache_default"

My commands for saving the image are as follows.
# docker save -o pause.tar kubernetes/pause:latest
# cat pause.tar | docker import - pause

@dchen1107
On Mon, Jul 13, 2015 at 6:44 PM, dieuroi<EMAIL_ADDRESS>wrote:

I'm sorry that I made a mistake when asking this question but I do used
the correct one in my json file.
I found that when I pull the "kubernetes/pause" image from dockerhub then
the image was useful. But when I saved the image for another computer which
have no access to the Internet, kubernetes failed to create container again.
E0713 14:20:34.182718 48272 manager.go:1556] Failed to create pod infra
container: API error >?(500): No command specified; Skipping pod
"suseapache_default"
My commands for saving the image are as follows.
docker save -o pause.tar kubernetes/pause:latest
cat pause.tar | docker import - pause
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11134#issuecomment-121108523
.

"No command specified" is an error code returned by docker. Looks like you docker import removes all configurations from an image including both Entrypoint and Cmd.
I think running docker load -i pause.tar should fix your issue here.

I found that the configurations are missing and modified them manually, now the issue is fixed. I also tried running docker load -i and this command did help me. Thank you for your instructions!

@dieuroi
Hi:
I have the same problem with you, can you tell me how to solve?
E1015 10:36:21.521270    5744 manager.go:1579] Error running pod "test_default" container "test": API error (500): No command specified

@dieuroi
你的问题解决了没，我也遇到了api500的问题，


fix(contract): change metadata judgement of epoch number

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:

Codecov Report

Merging #154 (6e2635c) into main (3d1149d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #154   +/-   ##
=======================================
  Coverage   37.12%   37.12%
=======================================
  Files          78       78
  Lines        5349     5349
=======================================
  Hits         1986     1986
  Misses       3363     3363


tree List  UI change select function

hello!
this is good lib!!
ex)
[Tree Structure]
▼ 1
....▼1-1
.......▶1-1-1  <- selected
....▶1-2
....▶1-3
[i want]
this.$refs.tree.find({text: 'aaa' }).select();
or
this.$refs.tree.find({text: 'aaa' }).select(true);
// let children= this.$refs.tree.selected();
// let parent = this.$refs.tree.find({text:children.parent.text});
// parent.select();
[i want ui]
▼ 1
....▼1-1 <- selected
.......▶1-1-1
....▶1-2
....▶1-3
but I didn't solve it.
:D

oh... pardon me
Oh, I got it.
[Solution]
this.$nextTick( () => {
this.$refs.tree.find({text: 'aaa' }).select();
})


Thread length & stddev: Enhancements

Overview
Some enhancements to: #6
Tasks

[ ] Average thread length and std deviation for: category
[ ] Average thread lengthand std deviation for: all threads combined

Thought this article might be helpful for binning and stddev work.


Add support for Brazilian Portuguese language

This is just a reference for a PR I'm submitting

I've added Portuguese (Brazilian). You can select the language as follows:
Swagger2MarkupConfig config = new Swagger2MarkupConfigBuilder()
                .withOutputLanguage(Language.BR)
                .build();
Swagger2MarkupConverter.from(file)
                .withConfig(config)
                .build()
                .toFolder(outputDirectory);

It should be available in version 1.1.1-SNAPSHOT


Add Day Of Year Function To OpenSearch

Description
Adds the day_of_year function to the SQL plugin. It behaves like the dayofyear function which already exists in the new engine, and both are aligned with MySQL
Issues Resolved
#722
Check List

[X] New functionality includes testing.

[X] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[X] New functionality has user manual doc added

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

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

I think you might have used the two dev/integration branches for this PR.

integ-add-legacy-syntax-for-matchphrase-function from dev-add-legacy-syntax-for-matchphrase-function-alternate


Added logs of yurt-manager component

if a ValidatingWebhook is disabled, the following logs will always be output continuously. and this affects the user's ability to parse yurt-manager's logs.
so modify the log level from warning to info5
#1820

/lgtm

/approve

Fixes #1820


[Addition]: ONE OF THE BIGGEST CLUSTERS I EVER SEEN

Hotspot b58 Addresses
11hR1dfPPuK5mnF9KtS1xs6DRhBha6eFrmriNURMriarukmohmX
Discord Handle
No response
Reason(s)
Something strange is happening in Cambodia. In the last 30 days appear these strange group of hotspots. These are Heltec. Close to them there is another cluster of Browan.
spoofed location in the middle of the forests.
Later I will provide all the hotspots list.

Thanks berserkrrr

This issue can be closed as completed. ✅
Now that https://github.com/helium/denylist/issues/6792 is resolved, all hotspots on the list are already on the denylist.

Right .


Made some small changes to file structure.

I'm not sure if I'm doing this right.
Base repository: "edgriebel/rit-iste422-project" Base: "master"
Head Repo: "head: zacheasRIT/rit-iste422-project"  compare: "move-to-dirs"

Please post pull request against your repo not mine, for example:

Base repository: "zacheasRIT/rit-iste422-project" Base: "master"
Head Repo: "head: zacheasRIT/rit-iste422-project" compare: "move-to-dirs"

Also, take a look at your build script. It won't compile code as written because you need to add the new directories to the classpaths; adding a prefix of the directory to the class names won't work


Update README.rst

Check PR testing

Thanks for testing this PR @adityakavalur i am going to close this

Thanks for testing this PR @adityakavalur i am going to close this


Implement numerically stable log_softmax()

A numerically stable implementation of log_softmax(). It's similar to the current implementation of softmax() and is what's used in PyTorch and SciPy too.
log_softmax([1000,1])

// Current implementation
// > Array [ 0, -Infinity ]

// Proposed implementation
// > Array [ 0, -999 ]

Thanks so much! 🤗 Can you add a unit test for this to tests/maths.test.js? One normal case and one case where the unstable version will break? 🔥

Done :)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

(ignore failing tests; related to something else)


Ignore tests for test coverage

What does this PR do?
Ignore */tests/* for test coverage
Before
---------- Coverage report ----------

Name                                       Stmts   Miss Branch BrPart  Cover
----------------------------------------------------------------------------
datadog_checks/vsphere/__about__.py            1      0      0      0   100%
datadog_checks/vsphere/__init__.py             3      0      0      0   100%
datadog_checks/vsphere/cache_config.py        30      0      2      0   100%
datadog_checks/vsphere/common.py               1      0      0      0   100%
datadog_checks/vsphere/errors.py               5      0      0      0   100%
datadog_checks/vsphere/event.py              130     46     34     11    63%
datadog_checks/vsphere/metadata_cache.py      32      0      2      0   100%
datadog_checks/vsphere/mor_cache.py           61      1     18      1    97%
datadog_checks/vsphere/objects_queue.py       18      0      0      0   100%
datadog_checks/vsphere/vsphere.py            531     71    196     26    84%
tests/__init__.py                              0      0      0      0   100%
tests/conftest.py                             17      0      0      0   100%
tests/test_cache_config.py                    29      0      4      0   100%
tests/test_metadata_cache.py                  31      0      0      0   100%
tests/test_mor_cache.py                       76      0      8      0   100%
tests/test_objects_queue.py                   26      0      0      0   100%
tests/test_vsphere.py                        377      1     14      1    99%
tests/utils.py                                92      2     48      5    95%
----------------------------------------------------------------------------
TOTAL                                       1460    121    326     44    90%

After
---------- Coverage report ----------

Name                                       Stmts   Miss Branch BrPart  Cover
----------------------------------------------------------------------------
datadog_checks/vsphere/__about__.py            1      0      0      0   100%
datadog_checks/vsphere/__init__.py             3      0      0      0   100%
datadog_checks/vsphere/cache_config.py        30      0      2      0   100%
datadog_checks/vsphere/common.py               1      0      0      0   100%
datadog_checks/vsphere/errors.py               5      0      0      0   100%
datadog_checks/vsphere/event.py              130     46     34     11    63%
datadog_checks/vsphere/metadata_cache.py      32      0      2      0   100%
datadog_checks/vsphere/mor_cache.py           61      1     18      1    97%
datadog_checks/vsphere/objects_queue.py       18      0      0      0   100%
datadog_checks/vsphere/vsphere.py            531     71    196     26    84%
----------------------------------------------------------------------------
TOTAL

Motivation
There is no value having coverage report for tests.
Review checklist (to be filled by reviewers)

[x] PR title must be written as a CHANGELOG entry (see why)
[x] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[x] PR must have changelog/ and integration/ labels attached
[x] Feature or bugfix must have tests
[x] Git history must be clean
[x] If PR adds a configuration option, it must be added to the configuration file.

test coverage of tests does not make a lot of sense

One argument for keeping the coverage for tests is to have an indicator for tests that are not run.
If you are ok, I'm closing this PR. If not, please share you point of view :)

Closing the PR, feel free to reopen it if necessary :)


fix: add for to labels in default/example/dialog-demo.svelte

Added the html for attribute in labels for best practice. For some reason, this was featured in New York style but not default.

thank you!


Android 2.0.9 transaction history not shown

Describe the bug
Activity is blank in the All activity section under the token type.
Android 2.0.9
To Reproduce
Open app
Click the white $ icon on the lowest left
-you should see your total token in $ (my settings are on $USD) on this screen
Click on any option on lower half of screen: HNT, SOL, IOT, MOBILE
I clicked on HNT
On this screen is the amount of HNT on the top half and an Activity section.
The activity section shows All activity on the left and Filter on the right
Expected behavior
Expected to see previous activity on the activity / filter area
Screenshots
If applicable, add screenshots to help explain your problem.

Smartphone (please complete the following information):

Device: Pixel 6
OS: Android 13
App Version 2.0.9

fixed this


fix(Table): fix react key error on sibling cells with same string content

Fixes #4034, also include a test and documentation for tableData attribute usage in Table component.
Please have a look especially at line 68 of src/collections/Table/TableRow.js.
The newly created react key of "string containing" cells will have the following format (screenshot taken from docs):

In my opinion, this should be a good way to create "unique" keys for the cells because the cell index is also used for key creation.
But I don't know if creating the React Key attribute this way may have mentionable performance impact in huge lists. At least it should have a little but in sake of errorless rendering.

Codecov Report

Merging #4198 (a7dddc2) into master (bb2db59) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #4198   +/-   ##
=======================================
  Coverage   99.75%   99.75%
=======================================
  Files         180      180
  Lines        3241     3244    +3
=======================================
+ Hits         3233     3236    +3
  Misses          8        8

Impacted Files
Coverage Δ

src/collections/Table/TableRow.js
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


add script that installs s4tf and deps on ubuntu1804

The file comment describes what the script does and what the options are.
The purposes of this script are:

Make it easy to automatedly install S4TF on Ubuntu 18.04 so that automated S4TF jobs can be written.
Factor this logic out of the existing Dockerfiles that we have in various repos. (After this lands, I'll gradually refactor the Dockerfiles to refer to this).
We could refer to this in installation instructions.

Pinging @dan-zheng for a review, and @saeta for visibility. Thanks for creating this, @marcrasi! 😄

We now have Ubuntu 18.04 machines. Can we get this submitted so I can play around with it? :)

CI failed:
...
Step 8/12 : ENV PATH="/swift-tensorflow-toolchain/usr/bin:${PATH}"
 ---> Running in a21be64c8e10
Removing intermediate container a21be64c8e10
 ---> c34073a35c69
Step 9/12 : WORKDIR /swift
 ---> Running in 99c393143efb
Removing intermediate container 99c393143efb
 ---> 1a420755994f
Step 10/12 : COPY . .
 ---> a5dbd768d6b5
Step 11/12 : RUN swift build -Xswiftc -enable-experimental-quasiquotes
 ---> Running in 4954f273b6c3
error: root manifest not found
The command '/bin/sh -c swift build -Xswiftc -enable-experimental-quasiquotes' returned a non-zero code: 1

@marcrasi: wonder if you know what's going on? error: root manifest not found suggests that there's no Package.swift in the working directory.

Oh it's probably just that this branch is so old that CI is confused. I'll update the branch and see if CI works.
Also, since it's been a really long time since I made this, the script may be out of date in some ways. I'll retest it and upload fixed before merging.


Add workflow to update default pack version

This is to create a PR to update the default pack version in github-actions/setup-pack/action.yml when a new pack version is released.
References update-version-on-release-ga.yml.
Issue: pack #1244
Related PR(s): pack #1263

@jkutner the test logs don't seem very informative, what should I do for the failing tests?

@importhuman i think that was just the minimum label enforcer. i've added the labels


Feature/krenn1/uqpipeline

Added a wrapper and tests for uqpipeline sampling methods.

@jsemler, @daub1 or @FrankD412,
Could you spare some time this week or next to look at this pull request?
The major change is support for uqpipeline samplers, and I'd like to release it either before (or soon after) my workflow talk next week.
Thanks very much!
-Chris

@FrankD412 and @daub1,
Thank you very much for your comments and suggestions!
I doubt I can get to them before the talk on Monday, but I will make it a priority afterwards.
-Chris


It is apparently impossible to use multiple contexts in a suite if using Symfony2Extension

I have tried every config combination humanly possible, but cannot get two contexts to run.  As soon as I add a 'contexts:' key to my config, even the FeatureContext class is no longer recognised.
This is my behat.yml file:
default:
    suites:
      my_suite:
        type: symfony_bundle
        bundle: MyCoolBundle
    extensions:
        Behat\Symfony2Extension: ~
        Behat\MinkExtension:
            base_url: http://ams.ldev
            selenium2: ~
            symfony2: ~
            sessions:
                default:
                    symfony2: ~

And I have the default directory structure created by the behat init command.  How can I change the above config to allow me to have two context classes for the suite?
Ideally I'd like to define a 'global' context class that contains database-related code that should run on each execution of my tests. But if I can even just get this working inside one suite that'd do me for now.
Thanks for any help.

Please reopen this issue in the Symfony2Extension. This is a Behat repository


How can I detect the tiny object in image?

I have a segmentation task to detect the tiny object. The image like below.

Too tiny one I can't detect , but object bigger than first image I can detect successfully. Like below.

Do anyone have experience to deal with the tiny object segmentation?
How can I change my setting on training to get better result?
Help me , PLZ!!

Maybe you can use cropping?

Do you have the results? Please help me


Type not assignable for string literal in signature

TypeScript Version:  3.8.3

Search Terms:
string literal
string alias
is not assignable to parameter of type
Expected behavior:
No type check errors
Actual behavior:
No overload matches this call.
  The last overload gave the following error.
    Argument of type 'tAlgorithm' is not assignable to parameter of type '"oct"'.
      Type '"EC"' is not assignable to type '"oct"'.

Related Issues:
Code
interface KeyStore {
    generate(kty: 'EC', crv?: string, parameters?: object, isprivate?: boolean): void;
    generate(kty: 'OKP', crv?: string, parameters?: object, isprivate?: boolean): void;
    generate(kty: 'RSA', bitlength?: number, parameters?: object, isprivate?: boolean): void;
    generate(kty: 'oct', bitlength?: number, parameters?: object): void;
}

type tAlgorithm = "RSA" | "EC" | "OKP" | "oct";

function generateKey(alg: tAlgorithm, crvOrSize: string | number) {
    return ({} as KeyStore).generate(alg, crvOrSize);
}

Output
"use strict";
function generateKey(alg) {
    return {}.generate(alg, "P-256");
}

Compiler Options
{
  "compilerOptions": {
    "noImplicitAny": true,
    "strictNullChecks": true,
    "strictFunctionTypes": true,
    "strictPropertyInitialization": true,
    "strictBindCallApply": true,
    "noImplicitThis": true,
    "noImplicitReturns": true,
    "useDefineForClassFields": false,
    "alwaysStrict": true,
    "allowUnreachableCode": false,
    "allowUnusedLabels": false,
    "downlevelIteration": false,
    "noEmitHelpers": false,
    "noLib": false,
    "noStrictGenericChecks": false,
    "noUnusedLocals": false,
    "noUnusedParameters": false,
    "esModuleInterop": true,
    "preserveConstEnums": false,
    "removeComments": false,
    "skipLibCheck": false,
    "checkJs": false,
    "allowJs": false,
    "declaration": true,
    "experimentalDecorators": false,
    "emitDecoratorMetadata": false,
    "target": "ES2017",
    "module": "ESNext"
  }
}

Playground Link: Provided

Another reproduction link: https://codesandbox.io/s/laughing-shockley-hlpr1?file=/src/index.ts

alg can be one of four different string literals. Only two overloads accept a string as a second argument, but you pass the string "P256".
This is not a bug, it's TypeScript protecting you.

This clearly shouldn't work given that generateKey("EC", 128) can be substituted with
({} as KeyStore).generate("EC", 128)

and shouldn't match any overload.

How about this?
interface KeyStore {
    generate(kty: 'EC', crv?: string, parameters?: object, isprivate?: boolean): void;
    generate(kty: 'OKP', crv?: string, parameters?: object, isprivate?: boolean): void;
    generate(kty: 'RSA', bitlength?: number, parameters?: object, isprivate?: boolean): void;
    generate(kty: 'oct', bitlength?: number, parameters?: object): void;
}

type tAlgorithm = "RSA" | "EC" | "OKP" | "oct";

function generateKey(alg: tAlgorithm) {
  return ({} as KeyStore).generate(alg);
}

This still gives the error.
Playground

Sorry to not also link to the more general problem earlier on (it would've spared you some time): #1805 has some explanation on why the pattern is currently not supported.
Though, since then, I can sort of imagine us doing better given some of the work that went into https://github.com/Microsoft/TypeScript/pull/30779.

However, I think that even with the ideas of #30799 incorporated, it wouldn't help you write the original code easily without a type assertion or duplicate overloads.


Use absolute path in build script to avoid issues with Buildship

I'm not sure why the build has failed. I can't reproduce the issue locally.


Remove spin box arrow from input of type number

I don't know if it is the suitable place to mention this feature but I actually using normalize in my project and I'm using a TextField of type number. The component contains the spin arrows box which are annoying and it looks like TextField Example.
According to this reference ::-webkit-inner-spin-button and CSS: Hide “spinners” (up/down arrows) on inputs with type=”number” using a class I added a style.css to hide the spin arrow box. This is the reference TextField without spin arrow box.
Is not possible to add this feature in normalize.css or there are other constraints?

It's a useful accessibility feature. IMO normalize.css shouldn't disable useful accessibility features.

It's a useful accessibility feature. IMO normalize.css shouldn't disable useful accessibility features.

I agree accessibility shouldn't be disabled.  With that said, incorporating ARIA tags is of more value than those little-bitty buttons which are both an accessibility and usability issue.  The ability to use the arrow keys as well as typing the value is more practical for all users.  That's what should not be removed.
"The increase and decrease buttons are not keyboard focusable, but can be operated with a mouse." so there should be no accessibility issues (REF:  https://www.accede-web.com/en/guidelines/rich-interface-components/spinbuttons-simulated-using-aria/).
Perhaps normalize could consider including CSS to increase the size of each spinner's clickable target area of the up/down arrows to at least half the full height of the field, if not remove them.

"The increase and decrease buttons are not keyboard focusable...

But, Except IE11 and less, all other desktop browsers allow up & down arrow buttons to step the numbers.

Hey
I want to work on this issue. Could u please assign it to me?

@gurnoor-garry Just submit a PR

I agree with mvasilkov, the spinners or the arrows in the input type number is a good accessibility feature and should not be removed through normalize CSS. If it is not required then we can hide the arrows or style the arrows in our local CSS. Moreover, users are getting used to the arrows and it is a good feature too.


Judges general feedback is broken

regarding_type isn't set and blows up when submitting the feedback. Later the feedback-form also throws errors on regarding->exitst.

Fixed in pull #32


Issuing credits

Issue 100000 credits to @gardenenigma.

 Follow up
On Sunday, 17-Mar-24 02:25:04 UTC by Gov4Git dev
Issued 100000 credits to member @gardenenigma.


make types hashable

This will be useful when we need to do type mapping, for example in the cli when we map QIIME types to click types.

This came up again today in the Qiita plugin - @ebolyen, could you do this as part of your work this week on the types?

:+1:
On Wed, May 18, 2016 at 4:19 PM, Greg Caporaso<EMAIL_ADDRESS>wrote:

This came up again today in the Qiita plugin - @ebolyen
https://github.com/ebolyen, could you do this as part of your work this
week on the types?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/qiime2/qiime2/issues/46#issuecomment-220186335

@ebolyen, this one is done, right?

Yep!


Update repository after transfer

See https://github.com/senzing-garage/garage-roadmap/issues/3

Pull requests:

https://github.com/senzing-garage/subway-map/pull/13


Saving a layout as a copy doesn't work

Description of the bug
When you want to make a copy of your existing record page, you are not able to and you are receiving the error message: Failed to clone layout: handle not unique
Reproduction steps

Open a namespace that has an existing page
Open the page builder of your page from your admin panel
Press on Save as copy
Notice that the copy doesn't work and you are receiving an error

Expected behavior
Since the future is there, I should be able to copy a page and have an exact copy of my layout/page.
Screenshot/Video

Version
Branch: 2023.3.x

for today: make plan and discuss it with Jože

Layout is copied successfully but:

the new layout isn't selected like the success message says
after creating a copy of a layout, the title of the new and previous copy are changed

Added CL:
{
      "what": [
        "copy of page layout in {APP_NAME_COMPOSE}."
      ],
      "refs": [
        "https://github.com/cortezaproject/corteza/pull/1230"
      ]
    }


bower package name throws error

I got a bower install error that said The name contains upper case letters; this seems to block it from installing through bower.  The bower docs mention:

The package name must adhere to the bower.json spec.

bower.json spec goes on to say that:

Should be slug style for simplicity, consistency and compatibility. Example: unicorn-cake
Lowercase, a-z, can contain digits, 0-9, can contain dash or dot but not start/end with them.

I assume this means that the name in bower.json should be all lower case something like "ng-infinite-scroll".

please try "npm install --save ng-infinite-scroll"

bower is no longer supported

fyi there's another package with that name
https://github.com/ash-developer/ng-infinite-scroll

@thienedits tryyarn add ng-infinite-scroll


unable to access St2 webui

I am unable to access St2 on webui.
I have used Quick Install for installation of StackStorm on Ubuntu (AWS)

Have you checked for any firewall issues if you are working in production environment. Could you tell what is the error that you are getting?
Had the same case and the server's IP was blocked by the company's administrator.


CSRF handler and tags in forms and for Rails UJS

https://medium.com/rubyinside/a-deep-dive-into-csrf-protection-in-rails-19fa0a42c0ef


[1.x] Improve enabled flag

Currently, if you "disable" Pulse it acts as if you never installed it.

No events are registered
The dashboard is not accessible
The commands are not accessible
The pulse class is not registered as a singleton
etc.

This might seem like a good idea, but i think it is more problematic.
Imagine you are running Pulse and you are having troubles.
If you had php artisan pulse:restart in your deploy script, you deployment just broke because we don't register that command any more.
I think that the enabled switch should really only impact the recorders. We already document that this is what the flag does.

Question
One question I do have, though, is if the pulse:work command and pulse:check command should first look to see if pulse is enabled before starting?

One question I do have, though, is if the pulse:work command and pulse:check command should first look to see if pulse is enabled before starting?

I think pulse:work should continue to work regardless of the setting. You might disable recording but still want to process your ingest backlog. If you don't want that to happen, you can stop the command. I.e. it would be more flexible.
Similarly, I think pulse:check probably shouldn't fail to start/continue - any supervisor stuff will just keep trying to bring it back. My question is whether it should continue to emit beat events?
Another question is if/how this should affect direct calls to Pulse::record that aren't in a recorder? E.g. in the user's controllers/jobs. I think it should probably be a no-op.

As we chatted about, we are fine with beats being emitted, as the recorders are disabled.
We've also made sure that we disable recording on boot when PULSE_ENABLED is false.