Investigate issue with tablet_integration_test.go

tablet_integration_test.go is being flakky, ocasionally failing with the following error:
    tablet_integration_test.go:135: trace should show only one shard but it showed 0 shards, shards: [] trace:
        Tracing session 003a78b03e8111ef8cdda3e1b5278e98 (coordinator: , duration: 0s):

We need to figure out where this issue comes from driver or scylla.

The initial guess was that the test is flaky because there is some kind of delay in getting trace and the fix would be to add some sleeps between executing query and reading from the trace buffer. But my tests showed that it is not the case. If the tracing is empty as in the example in the description it is not going to be not empty after any delay.
This is how I changed the test to check for how much of a delay is needed:
for k := 0; k < 100; k++ {
		i = 0
		for i < 50 {
			i = i + 1

			var pk int
			var ck int
			var v int

			buf := &bytes.Buffer{}
			trace := NewTraceWriter(session, buf)

			err = session.Query(`SELECT pk, ck, v FROM test1.table1 WHERE pk = ?;`, i).WithContext(ctx).Consistency(One).Trace(trace).Scan(&pk, &ck, &v)
			if err != nil {
				t.Fatal(err)
			}

			queriedHosts := 0
			for j := 0; j < 2000; j++ {
				for _, hostAddress := range hostAddresses {
					if strings.Contains(buf.String(), hostAddress) {
						queriedHosts = queriedHosts + 1
					}
				}
				if queriedHosts != 0 {
					if j != 0 {
						fmt.Println("IT IS HERE: ", j)
					}
					break
				}

				time.Sleep(100 * time.Millisecond)
				if j == 1999 {
					fmt.Println("not enough time: ", j)
				}
			}

			if queriedHosts != 1 {
				t.Fatalf("trace should show only one host but it showed %d hosts, hosts: %s trace:\n%s", queriedHosts, hostAddresses, buf.String())
			}
		}
	}

It fails on every execution:
not enough time:  1999
--- FAIL: TestTablets (207.73s)
    tablet_integration_test.go:88: trace should show only one host but it showed 0 hosts, hosts: [<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>] trace:
        Tracing session 38c33c904d7b11efb93e0a0ec56fcac0 (coordinator: , duration: 0s):
FAIL

@dkropachev  @roydahan

I compared the code in gocql to code in different drivers and that's my findings:

in python-driver there is a mechanism to retry the session detail fetch, because tracing details are stored asynchronously, if the trace is still not available after max_wait seconds, TraceUnavailable will be raised - https://github.com/scylladb/python-driver/blob/8a4387ae3f36522cc10841c607abb8182c6f8286/cassandra/query.py#L981-L994
in rust-driver there is a tracing_info_fetch_attempts field that defines how many times the driver will try to fetch tracing information with sleeps in between https://github.com/scylladb/scylla-rust-driver/blob/36d5edd3a06a6808ce19dd594ea4904c8b48c0c2/scylla/src/transport/session.rs#L1411-L1422

In gocql there is no such mechanism and there is a need for it. Personally I would prefer to go with the approach similar to rust-driver. @dkropachev let me know if you would suggest otherwise

I compared the code in gocql to code in different drivers and that's my findings:

in python-driver there is a mechanism to retry the session detail fetch, because tracing details are stored asynchronously, if the trace is still not available after max_wait seconds, TraceUnavailable will be raised - https://github.com/scylladb/python-driver/blob/8a4387ae3f36522cc10841c607abb8182c6f8286/cassandra/query.py#L981-L994
in rust-driver there is a tracing_info_fetch_attempts field that defines how many times the driver will try to fetch tracing information with sleeps in between https://github.com/scylladb/scylla-rust-driver/blob/36d5edd3a06a6808ce19dd594ea4904c8b48c0c2/scylla/src/transport/session.rs#L1411-L1422

In gocql there is no such mechanism and there is a need for it. Personally I would prefer to go with the approach similar to rust-driver. @dkropachev let me know if you would suggest otherwise

100% there should be retry.
Another problem that needs to be fixed in the same PR is reading of tracing info should be done asynchoniusly, otherwise enabling tracing will lead to query processing slow down.
Also I realized that traceWriter is actually not exactly usefull, it just write custom formatted messages into a buffer.
So, probably we should abondone traceWriter and stop using it even in tests.
And instead come up with an API that will solve things easily and without running tracing subroutines.
Have an API something around this:
func (t *tracer) Trace(traceId []byte)
func (t *tracer) AllTraceIDs() [][]byte
func (t *tracer) GetCoordinatorTime(traceId []byte) (coordinator, duration time.Duration, err error)
func (t *tracer) GetActivity(traceId []byte) (timestamp time.Time, activity string, source string, elapsed int, thread string, err error)

The idea of how to use it is following:
		trace := NewTraceWriter(session, buf)
		err = session.Query(`SELECT pk, ck, v FROM test1.table1 WHERE pk = ?;`, i).WithContext(ctx).Consistency(One).Trace(trace).Scan(&pk, &ck, &v)
		if err != nil {
			t.Fatal(err)
		}
		for _, traceID := range trace.AllTraceIDs() {
				coordinator, d, err := trace.GetCoordinatorTime(traceID)
				.....
				timestamp, activity, source, elapsed, thread, err := trace.GetActivity(traceID)
				.....
		}

And probably, since we are on the table of tracing we could have split between different type of traces: prepared statement, query, batch_query.
Regarding what to do next, suggestion is following:

Open a separate issue on changes we want to do in tracer and continue discussion there.
And close this one as bug, wontfix


drawing is not working on mobile devices

Any drawing not working on mobile or tablet devices.

Which devices?

Samsung S4 mini, Android 4.2.2 and
Ipad air 2 mini and
Chrome Dev Tools Touch Emulation

+1

Hi guys, is ver 0.3.x going to have touch support for mobile devices (as of 0.2.3 I can't draw shapes on a mobile device)? if so, when can we expect a stable release? Cheers.


fix hbase scan startkey is null NPE

Fix bug: HBase 表预分区 分区扫描数据发生NPE
fix #1690

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

感谢贡献和反馈问题, 欢迎后续 hbase 相关更多的 PR 和设计提交~

感谢贡献和反馈问题, 欢迎后续 hbase 相关更多的 PR 和设计提交~

感谢 review ，hbase 和 tidb 相关的会持续贡献


Compilation error for dynamic keys

When using keys as owned strings instead of &'static str:
slog = { version = "2.5.1", features = ["dynamic-keys"] }

this crate is not compiled:
Checking slog-kvfilter v0.7.0
error[E0277]: the trait bound `&str: std::borrow::Borrow<slog::Key>` is not satisfied
  --> /home/xx/.cargo/registry/src/github.com-1ecc6299db9ec823/slog-kvfilter-0.7.0/src/lib.rs:38:69
   |
38 |         let matched = if let Some(keyvalues) = self.pending_matches.get(&key) {
   |                                                                     ^^^ the trait `std::borrow::Borrow<slog::Key>` is not implemented for `&str`

error[E0277]: the trait bound `slog::Key: std::cmp::Eq` is not satisfied
  --> /home/xx/.cargo/registry/src/github.com-1ecc6299db9ec823/slog-kvfilter-0.7.0/src/lib.rs:38:69
   |
38 |         let matched = if let Some(keyvalues) = self.pending_matches.get(&key) {
   |                                                                     ^^^ the trait `std::cmp::Eq` is not implemented for `slog::Key`

error[E0277]: the trait bound `&'a str: std::borrow::Borrow<slog::Key>` is not satisfied
  --> /home/xx/.cargo/registry/src/github.com-1ecc6299db9ec823/slog-kvfilter-0.7.0/src/lib.rs:48:34
   |
48 |             self.pending_matches.remove(&key);
   |                                  ^^^^^^ the trait `std::borrow::Borrow<slog::Key>` is not implemented for `&'a str`

error[E0277]: the trait bound `slog::Key: std::cmp::Eq` is not satisfied
  --> /home/xx/.cargo/registry/src/github.com-1ecc6299db9ec823/slog-kvfilter-0.7.0/src/lib.rs:48:34
   |
48 |             self.pending_matches.remove(&key);
   |                                  ^^^^^^ the trait `std::cmp::Eq` is not implemented for `slog::Key`

error: aborting due to 4 previous errors

For more information about this error, try `rustc --explain E0277`.
error: Could not compile `slog-kvfilter`.

ok, help would be appreciated ;-)


Multitenancy & MultiTenantRowLookupScript CPU performance

Hi,
I've read the tutorial for extending RowLookupScript<> to filter by 'TenantId' (link). The Expiration property is set to a negative value to disable caching at the DynamicScriptManager level and to rely on the cache at the LookupScript level.  I've noticed that the DynamicScriptManager.Item.EnsureContent() method will create one or two streams for every request when the caching is disabled at this level. The second stream is a compressed (gzipped) one and will be created if the length of the first stream exceeds 4096. I'm concerned about the frequent gzipping of long lookup scripts (big # of rows and many included columns in the lookup script).
Do you think this will impact CPU performance enough to worth searching for a different approach?

I don't think it affects much but could possibly make it optional.

If you don't think it affects much then it's no need to make it optional. In the future I will try to avoid multitenant apps and go with one database per customer approach as you suggested in the guide.
Thank you for your prompt response!


A test randomely fails

Hitting restart job on travis usually fixes it.

That's because the tolerance on the "stop and resume" test is too tight.
There are two 500ms delays created in the parse, and it expects the total parse time to be 1s +/- 10% but on slower machines it often takes more than 1.1s because of delays over and above the artificial ones.
So on the ARM machines in the Fedora build farm it typically takes around 1.3 or so (two runs I just did were 1318ms and 1323ms) while the i686 and x86_64 machines do finish in the allotted time.
I think a 10% margin on the minimum end is find (arguably no margin should be fine there are it should always take at least 1s in total) but on the maximum end it needs to be large, or not be checked at all, as you have no idea how long it might take over and above the artificial pauses.


cfitsio: 4.4.0 -> 4.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/cfitsio/versions.
meta.description for cfitsio is: Library for reading and writing FITS data files
meta.homepage for cfitsio is: https://heasarc.gsfc.nasa.gov/fitsio/
meta.changelog for cfitsio is: https://heasarc.gsfc.nasa.gov/FTP/software/fitsio/c/docs/changes.txt
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.4.1 with grep in /nix/store/var9r12gaj5r44y0gvk822fz15cs891v-cfitsio-4.4.1
found 4.4.1 in filename of file in /nix/store/var9r12gaj5r44y0gvk822fz15cs891v-cfitsio-4.4.1

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

138 total rebuild path(s)

138 package rebuild(s)

First fifty rebuilds by attrpath

apacheHttpdPackages.mod_tile
apacheHttpdPackages_2_4.mod_tile
btc-rpc-explorer
cfitsio
cloudcompare
entwine
faircamp
fitsverify
gdal
gdalMinimal
gildas
gmt
gnuastro
gplates
grass
gwyddion
haskellPackages.gi-vips
haskellPackages.hgdal
imaginary
imgproxy
indi-full
indilib
joplin
kdePackages.gwenview
kstars
labplot
lemmy-ui
libsForQt5.gwenview
mapcache
mapnik
mapproxy
mapserver
merkaartor
mysql-workbench
nip2
nufraw
nufraw-thumbnailer
octavePackages.fits
octavePackages.mapping
openorienteering-mapper
paraview
pdal
perl536Packages.Tirex
perl538Packages.Tirex
phd2
plasma5Packages.gwenview
postgresql12JitPackages.postgis
postgresql12Packages.postgis
postgresql13JitPackages.postgis

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/var9r12gaj5r44y0gvk822fz15cs891v-cfitsio-4.4.1 \
  --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 cfitsio https://github.com/r-ryantm/nixpkgs/archive/88926cbc7ea5a7af0324b343be329fa596572f82.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/88926cbc7ea5a7af0324b343be329fa596572f82#cfitsio

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/var9r12gaj5r44y0gvk822fz15cs891v-cfitsio-4.4.1
ls -la /nix/store/var9r12gaj5r44y0gvk822fz15cs891v-cfitsio-4.4.1/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 took longer than 120m and timed out

Maintainer pings
cc @xbreak @hjones2199 for testing.

This reverts license change to NASA Open Source Agreement (NOSA) that was introduced without changelog in 4.4.0.

Version 4.4.1 - Jun 2024

Removed NOSA license and restored previous license file


Rails 7 support

Hi everyone!
I'm currently using clearance in an application I would love to upgrade to rails 7.
Due to dependencies of clearance I'm unable to upgrade.
I attempted to upgrade clearance myself using appraisal, but I'm bumping into some issues with sprocket-rails being extracted from rails.
Are there any plans to support rails 7?  Or can we consider clearance in maintenance mode and we should attempt to switch to devise?

Rails 7 is now supported and tested, I'm working on having Turbo support
https://github.com/thoughtbot/clearance/commit/dff0141f2ad4db8570e2ca1afd4f5403aa741d8f

Hi Dorian!
Thank you 🙏.
How do I buy you a beer or a coffee?


Problem with Darius-cnn-ip

I download this whole project with  to my pynq board and change the overlay path to the right place. When I run the first time, the data printed is the same as  darius-cnn-ip.ipynb, so it's loaded right . However, when I explore the detailed  parameter, I find many problems.
First, when I print the OFM array, I find many negative number in it??? The input IFM and weight is the same as the demo which is random ranging from 0 to 255. I can't explain why the convolution and max pooling can change positive integer to negtive integer.......
Then, I change the parameter for pooling and run the demo again, I find the OFM does't change. I can't explain why this happens.
You can also find the first problem, just by changing your demo in  darius-cnn-ip.ipynb:
for i in range(0, 15, 4):     print(hex(ofm[i]))
change to a larger range:
for i in range(0, 159, 4):     print(hex(ofm[i]))
and many problems.
You can try different pooling parameter and see if the output OFM is changed.
If my operation is wrong, can you show me the right using process?

Hi,
Thanks for trying out Darius notebook and getting back to us on your concerns.
Firstly, let me clarify some assumptions. Any 16 bit signed integers are acceptable for IFM and weight values to support all kinds of image formats or intermediate layers.
And to answer your questions:

The convoluted output can also be negative as it is also declared as signed int16. So, it can range from -32,768 to 32, 767. Even if 2 positive integers have been convoluted, the result can be out of bounds of 15 bits and can be represented as negative integer with 16th bit as sign bit. The maxpooled output can be a negative integer for cases where the maxpool window of convoluted output has all negative integers and maximum could again be negative.

Can you provide the testcase with different pool parameters for which you are getting the same ofm? I have tried different pool dimensions and each time I got different OFM values.

Also we would like to understand if the overlay you are using is different from the overlay provided. Because, if it is same one, you don’t have to change the path.
Thanks
Radhika

Hi,
Since there was no response for a week, closing the issue. I am assuming things have been clarified. If there is still an issue, please raise a new one.
Thanks
Radhika


CI: insert sast-coverity-check

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test


Trace logging for consensus rules

Found a better solution. See https://github.com/stratisproject/StratisBitcoinFullNode/issues/2249


[Feature]: Add flag to make theme push idempotent

What area(s) will this request affect?
Theme
What type of change do you want to see?
New feature
Overview
Would it be possible to add a flag that makes theme push idempotent?
Given a theme name, it would only create a single theme with that name.
Right now, if the --unpublished flag is passed, it will create multiple themes with the same name. Without --unpublished, it will error if the theme isn't created.

Motivation
To support PR reopening in a GitHub, we need the ability to push to a theme named after a branch multiple times and have only a single theme created.
In the pre-1.0 release of Shopkeeper I had to add a create command in order to make this possible.
I walk through how someone can use GitHub Actions to write workflows to provide a CI/CD experience for Shopify like you might have with other tools like Heroku or Vercel's branch previews.

Not sure how to keep this open... The problem is a bit of a tricky one in terms of the existing command and flags. @isaacroldan do you have a sense of whether we should pursue a fix for this in the CLI or if I should continue to handle it with my plugin?
I'm happy to contribute a change, but would need guidance to align it with Shopify's vision.

Hi @jeffreyguenther, thanks for the proposal. @karreiro or @mgmanzella would be better to answer you this question :)

+1 This would be a great thing!

Still relevant

This issue ist still relevant and always should have been the default behavior.


waydroid: 1.4.2 -> 1.4.3; add update script, move to by-name, format, and other cleanup

Things done
Picking up https://github.com/NixOS/nixpkgs/pull/339882 which was closed by the original contributor.

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

It's been a week with no review, so I'm going to merge this.
I guess you can see me as a reviewer since I didn't create the change :P


500 error if backup code is incorrect

While setting up backups using Frontend, if the confirmation text you enter is incorrect, Backend returns a 500 with the message "Completely unhandled exception: Cannot read property 'data' of undefined".
From a first look, it seems that Google returns an object with a property named response, but Backend expects a property named res:
https://github.com/getodk/central-backend/blob/e9ffd2c0c3aa1a9475852e1397b8259e2b03165a/lib/resources/config.js#L71-L73
I think this explains this Sentry issue: https://sentry.io/organizations/getodk/issues/1875192039/?project=1298632

sigh. they change this all the time and they don't document it.


ci: set custom job name for jobs with matrix strategy

Codecov Report

Merging #30 (0a8efc9) into main (6858ecd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #30   +/-   ##
=======================================
  Coverage   92.24%   92.24%
=======================================
  Files          18       18
  Lines         696      696
  Branches       49       49
=======================================
  Hits          642      642
  Misses         50       50
  Partials        4        4

Flag
Coverage Δ

apps
94.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 6858ecd...0a8efc9. Read the comment docs.


limit schedds by domain for DUNE global pool

Quick stab at making dune global pool work again.  Limiting schedd's to ones that match our domain name.
(This in theory could then work at bnl, as well... maybe?)

Okay, so now revisited, it does the whole business of picking a schedd list with one condor collector.query()
Then we aren't contacting the schedd directly, and we don't have problems with the offsite schedd (unless they
set the IsJobsubLite flag on it, but that's another whole kettle of fish).


findDOMNode is different from react-dom

function findDOMNode() {      return e && e.base || null }
but in react-dom
function findDOMNode(componentOrElement) {   // ...   if (componentOrElement == null) {     return null;   }   if (componentOrElement.nodeType === 1) {     return componentOrElement;   }   // ...
so when I use findDOMNode(element)，react-dom return element，but preact return null. This cause react-slick make a mistake.

We are encountering the same issues with react-slick.  I created a fix for this issue in PR #495

Can confirm this is triggering errors in react-autocomplete too.

@mlaursen suspects that findDOMNode may also cause the incompatibility with react-md. I've made sure preact-compat is at 3.18.4 in my reproduction repo (linked above), but I still see the issue.


Phi-3.5-vision-Instruct muliples images loading

How is it possible to load multiples images for Phi-3.5-vision-Instruct ?
And referencing them as Image ?
Maybe it is supported but now example to show how.

By modifying code I was able to load 2 pictures and it seems to work

@Aveline67 can you please share the code? Phi 3.5 vision instruct can support multiple images, just add messages with the correlated image!

I just did call multiple times .add_phiv_image_message() but I had to comment out the   candle_core::bail!("Can only process one image per batch"); condition in mistralrs-core\src\vision_models\phi3_inputs_processor.rs
I am looking to create a proper PR, also some changes are needed in phi3.rs to ensure to pass all pictures dimensions

@Aveline67 I see! It looks like this should be a fix as well as what you mentioned. Please feel free to open a PR!

I did try this one https://github.com/EricLBuehler/mistral.rs/pull/829
not sure about images sizes

@Aveline67, were you able to use multiple Images? I f so pls let me know

sure i did, i will publish the PR

And I want to run multiple prompts at once, but not successful, if you have any idea regarding it, pls let me know

@Aveline67 have you been able to publish the PR?


[RELEASE] Security vulnerability update for rubocop

Coverage remained the same at 100.0% when pulling 37cc791479a3de6cadf362337136363af53e7958 on UsmanAfzal:master into 664a34780562c31a7ab736863c2ea011597e891f on ukparliament:master.


[ Suggestion ] Make the Spotify Modal restart the song at a specific threshold

Make it so that pressing the "previous" button while the song has a duration longer than a specific amount (say 5 seconds), it replays the song instead of playing the previous song.
so this would restart the song:

and this would play the previous song:

excuse the last gif being weird, i can't record things properly.

Wouldn't it make more sense to just mimic what the spotify client does? Which is that you have to press the button twice if the song has been playing for longer than 2 seconds for it to go to the previous song.

Which essentially is just if it has been playing for longer than 2 seconds you repeat else go back


Autocomplete and NgModel

The autocompletion from materialize doesn't trigger the ngModel when you click on it. The ngModel is using the string we wrote and not the one we clicked on in the autocompletion.
I'm wondering if it's a error comming from materialize or angular 2.

Would you be able to provide example code that reproduces that? Thanks.

I'm also experiencing this issue with the following code:
<input type="text" [(ngModel)]="createThreadLocation" name="createThreadLocation"
          id="createThreadLocation" materialize="autocomplete" [materializeParams]="[{'data': {'apple': null, 'google': null}}]">

With the following versions:
    "angular2-materialize": "^5.1.0",
    "materialize-css": "^0.97.7"

Ie if the user types in "a", clicks "apple" in the drop down list, then the createThreadLocation property is still set to "a".

I've been struggling with this as well.
jQuery events don't trigger Angular2's events. See: http://stackoverflow.com/questions/38557175/angular2-and-jquery-change-events
Just hooking the jQuery event as in acdcjunior's answer seems best.
Also, I noticed that there's actually a bug related to this in materialize-css 0.97.7, trigger('change') is missing. This has been fixed in master:
https://github.com/Dogfalo/materialize/blob/master/js/forms.js#L353
$input.trigger('change');
Until the next materialize-css release, you'll have to get materialize-css from GitHub and build it manually.
package.json
"materialize-css": "Dogfalo/materialize#47b2f3f8c4b149d4330d554415bc798b889331c8",

cd node_modules/materialize-css
npm install
grunt release

I didn't understand how to workaround. Suggest, please.


[Bug Report] El-column bugs after toggle using if in render function

Element UI version
2.4.5
OS/Browsers version
Google Chrome 68.0.3440.106
Vue version
2.5.17
Reproduction Link
https://jsfiddle.net/mmx38qxw/5214/
Steps to reproduce
The initial render of columns works as expect, but if i tell to disable the level '1-2' works fine, but when i re-enable the level 1-2 (after disable it, obviously) the structure turns malformed! (After try to re-enable looks 1-3 and 1-4 columns).
What is Expected?
Columns rowspan and colspan works as expect after re-inserting the node in dom using render function.
What is actually happening?
Columns rowspan and colspan bugs when i toggle some node on render function.

After some search in repository issues, i found the solution by #11153 , just put the key value for every el-table-column node returned from render function. =)


Bumped logging lib version to solve lostash logger problem

👍

@rafaeljesus yup, forgot to rebase, did now

👍

👍


undefined variable in background.js

Hey barbushin, I'm using your plugin in tandem with nekojira's wp one if that's important.  When I reload a page on a develop site to get errors, I never see the password button to login, and I get an error when I inspect the extension in chrome.  It says:
extensions::uncaught_exception_handler:8 Error in response to tabs.get: TypeError: Cannot read property 'isSslOnlyMode' of undefined
at D.a.q (chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:25:173)
at Object.callback (chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:17:320)
at x (chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:17:232)
at chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:17:407
at XMLHttpRequest.l.onreadystatechange (chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:19:105)
Anyway, I'm not sure if this is related to nekojira or yourself.  But any help would be appreciated.

Hey! Can you provide link to that page? I'll try to fix it as soon as possible.

Sure, it's johnjhoward.net. The password for the wp php console is 123456
if that's needed.
On Jul 1, 2015 9:33 PM, "Sergey"<EMAIL_ADDRESS>wrote:

Hey! Can you provide link to that page? I'll try to fix it as soon as
possible.
—
Reply to this email directly or view it on GitHub
https://github.com/barbushin/php-console/issues/84#issuecomment-117902527
.

Thanks. I removed your message, so password will not be shared with another GitHub users.

Oh, thanks. I'm emailing you so I wasn't aware of that.
On Jul 1, 2015 9:42 PM, "Sergey"<EMAIL_ADDRESS>wrote:

Thanks. I removed your message, so password will not be shared with
another GitHub users.
—
Reply to this email directly or view it on GitHub
https://github.com/barbushin/php-console/issues/84#issuecomment-117904551
.

I've fixed Cannot read property 'isSslOnlyMode' of undefined in version 3.0.27. Try to reinstall extension. And as before, it works on your site, see http://i.imgur.com/XsufheU.png & http://i.imgur.com/72Cna5G.png
Also try if your version of PHP Console extension works on http://php-console.com/instance/examples/#eval_terminal

OK, I'll get to it tomorrow. Thanks!
On Jul 2, 2015 12:32 AM, "Sergey"<EMAIL_ADDRESS>wrote:

I've fixed Cannot read property 'isSslOnlyMode' of undefined in version
3.0.27. Try to reinstall extension. And as before, it works on your site,
see http://i.imgur.com/XsufheU.png & http://i.imgur.com/72Cna5G.png
Also try if your version of PHP Console extension works on
http://php-console.com/instance/examples/#eval_terminal
—
Reply to this email directly or view it on GitHub
https://github.com/barbushin/php-console/issues/84#issuecomment-117941930
.

Hmm. Now I reinstalled and am getting two additional errors:
Error in response to cookies.get: TypeError: x is not a function
at Object.callback (chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:7:479)
at chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:7:268
at IDBRequest.index.get.onsuccess (chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:36:239)handler @ extensions::uncaught_exception_handler:8
extensions::uncaught_exception_handler:8
Error in response to cookies.get: Error: Attempting to use a disconnected port object
at chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:9:457
at chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:8:350
at Object.callback (chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:7:479)
at chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:7:268
at IDBRequest.index.get.onsuccess (chrome-extension://nfhmhhlpfleoednkpnnnkolmclajemef/background.js:36:239)

Are you using 3.0.31 version?

Chrome only updated to 3.0.29, I'll try to update again.
*Gabe Blodgett, *   Co-Owner & Director of Marketing and Engineering ,
MPzero Studios
[image: photo]
http://facebook.com/MPzeroStudios
http://linkedin.com/comany/mpzero-studios
http://plus.google.com/+Mp0Studios http://twitter.com/mpzerostudios
http://www.youtube.com/channel/UC7ksEPpuepoSyeVC3-jk2kQ
Phone:<PHONE_NUMBER>
Email<EMAIL_ADDRESS>Website: MPzeroStudios.com
Address: 1207 10th st, La Grande OR 97850
Get a signature like this: Click here!
http://ws-promos.appspot.com/r?rdata=eyJydXJsIjogImh0dHA6Ly93d3cud2lzZXN0YW1wLmNvbS9lbWFpbC1pbnN0YWxsP3dzX25jaWQ9NjcyMjkzOTI3IiwgImUiOiAiNTczMjU2ODU0ODc2OTc5MiJ9
On Thu, Jul 2, 2015 at 2:17 PM, Sergey<EMAIL_ADDRESS>wrote:

Are you using 3.0.31 version?
—
Reply to this email directly or view it on GitHub
https://github.com/barbushin/php-console/issues/84#issuecomment-118171126
.

Okay, updated to 3.0.31, a new error shows up, but I don't see what it's
referring at all.  Here's the image:
*Gabe Blodgett, *   Co-Owner & Director of Marketing and Engineering ,
MPzero Studios
[image: photo]
http://facebook.com/MPzeroStudios
http://linkedin.com/comany/mpzero-studios
http://plus.google.com/+Mp0Studios http://twitter.com/mpzerostudios
http://www.youtube.com/channel/UC7ksEPpuepoSyeVC3-jk2kQ
Phone:<PHONE_NUMBER>
Email<EMAIL_ADDRESS>Website: MPzeroStudios.com
Address: 1207 10th st, La Grande OR 97850
Get a signature like this: Click here!
http://ws-promos.appspot.com/r?rdata=eyJydXJsIjogImh0dHA6Ly93d3cud2lzZXN0YW1wLmNvbS9lbWFpbC1pbnN0YWxsP3dzX25jaWQ9NjcyMjkzOTI3IiwgImUiOiAiNTczMjU2ODU0ODc2OTc5MiJ9
On Thu, Jul 2, 2015 at 2:25 PM, Gabe Blodgett<EMAIL_ADDRESS>wrote:

Chrome only updated to 3.0.29, I'll try to update again.
*Gabe Blodgett, *   Co-Owner & Director of Marketing and Engineering ,
MPzero Studios
[image: photo]
http://facebook.com/MPzeroStudios
http://linkedin.com/comany/mpzero-studios
http://plus.google.com/+Mp0Studios http://twitter.com/mpzerostudios
http://www.youtube.com/channel/UC7ksEPpuepoSyeVC3-jk2kQ
Phone:<PHONE_NUMBER>
Email<EMAIL_ADDRESS>Website: MPzeroStudios.com
Address: 1207 10th st, La Grande OR 97850
Get a signature like this: Click here!
http://ws-promos.appspot.com/r?rdata=eyJydXJsIjogImh0dHA6Ly93d3cud2lzZXN0YW1wLmNvbS9lbWFpbC1pbnN0YWxsP3dzX25jaWQ9NjcyMjkzOTI3IiwgImUiOiAiNTczMjU2ODU0ODc2OTc5MiJ9
On Thu, Jul 2, 2015 at 2:17 PM, Sergey<EMAIL_ADDRESS>wrote:

Are you using 3.0.31 version?
—
Reply to this email directly or view it on GitHub
https://github.com/barbushin/php-console/issues/84#issuecomment-118171126
.

I dont see any image attached

I opened http://johnjhoward.net, authorized in php console, and I see that it works. All JS errors in JS console are not relative to php console, it's because of internal site scripts. If I disable php console I see the same JS errors. And there are no errors in JS console of php console background page.

See http://i.imgur.com/dfvbB69.png mlPageTracker and sendTrackEvent are in widget-narrow.js, and it does not belong to php console.

When I go to the extensions tab and click "collect errors" under the
plugin, it shows an error that says "Cannot load extension with file or
directory name metadata. Filenames starting with "" are reserved for use
by the system." in reference to the
"background": { "page": "background.html" },
section.
*Gabe Blodgett, *   Co-Owner & Director of Marketing and Engineering ,
MPzero Studios
[image: photo]
http://facebook.com/MPzeroStudios
http://linkedin.com/comany/mpzero-studios
http://plus.google.com/+Mp0Studios http://twitter.com/mpzerostudios
http://www.youtube.com/channel/UC7ksEPpuepoSyeVC3-jk2kQ
Phone:<PHONE_NUMBER>
Email<EMAIL_ADDRESS>Website: MPzeroStudios.com
Address: 1207 10th st, La Grande OR 97850
Get a signature like this: Click here!
http://ws-promos.appspot.com/r?rdata=eyJydXJsIjogImh0dHA6Ly93d3cud2lzZXN0YW1wLmNvbS9lbWFpbC1pbnN0YWxsP3dzX25jaWQ9NjcyMjkzOTI3IiwgImUiOiAiNTczMjU2ODU0ODc2OTc5MiJ9
On Thu, Jul 2, 2015 at 3:29 PM, Sergey<EMAIL_ADDRESS>wrote:

See http://i.imgur.com/dfvbB69.png mlPageTracker and sendTrackEvent are
in widget-narrow.js, and it does not belong to php console.
—
Reply to this email directly or view it on GitHub
https://github.com/barbushin/php-console/issues/84#issuecomment-118184861
.

I can't find any Collect errors tab or button on extensions page. Usually I check extension errors in background.html console, see http://i.imgur.com/GTtsv6z.png Could you please send me screen screenshot where can I find that collect errors link?

Here's how you enable chrome to collect error: (click on the 1.)  To view errors, click on the 2. when you reload the webpage you want to use it on.

Here's the error I'm getting:

Wow.. I'm using Chrome version 43.0.2357.130 m and I don't have Collect errors option. Also there is no _metadata file or directory in extension files list. Whatever, php-console still not works on your PC?

I'm using Chrome version 45.0.2438.3 dev-m (64-bit).  For me Chrome automatically updates, so I'm not sure what the issue is.
No, it still doesn't work.

So you're using dev version? Sorry, but php-console is based and tested only on stable Chrome version. And as I said before: there is no _metadata file or directory in extension files list, so you should check if there is something wrong with your Chrome or website.
Let me know if you'll have a problem with any version of stable Chrome edition. Thank you.

Okay, I uninstalled Chrome, downloaded the same version that you have and suddenly, it works!  Thanks for the help, and I'm sorry to bug you!  I was unaware that I was running the dev version (a bad idea really).  Anyway, I appreciate everything!


What items are models trained on...

What is the model training item

input: bert vector + phone,
output: wave


Create VensStylePPTextures.netkan

You'll need the spec_version stanza to be like this to use a subfolder in your install_to:
"spec_version": "v1.2",


vstest rerun fails with nunit testcase

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: VSTest@2
Environment

Server - tfs on-premises

If using TFS on-premises, provide the version:

About Azure DevOps Server
Version Azure DevOps Server 2020 Update 1

Agent - Hosted or Private:

private, win 10, Current agent version: '2.181.2'

Issue Description

NUnit adapter <IP_ADDRESS>
Microsoft Test Execution Command Line Tool Version 17.2.0-preview-20220401-08

VSTest@2 with nunit3 adapter fails to execute the test re-run due to character escaping:
Error logs
First run:
vstest.console.exe "tests1.dll" "tests2.dll"
/TestCaseFilter:"Category=X&Category=Y&Category!=Z&Category!=W"
/Settings:"E:\ANSYSDev\TFSAgent\_work\_temp\ctrrwcwxein.tmp.runsettings"
/Logger:"trx"
/TestAdapterPath:"E:\ANSYSDev\TFSAgent\_work\_temp\Tests"

retry:
vstest.console.exe /TestCaseFilter:"FullyQualifiedName=Ansys.Artemis.Aim.Integration.Daily.Tests.Internal.FluidConvergenceControlsShould.SolveMultiBodyVolumeMonitor(4\,2.7068d\,1.4408d)|FullyQualifiedName=..."
"E:\ANSYSDev\TFSAgent\_work\_temp\Tests\AIM.Services.Artemis.Testing.22.2.0-unstable1462\lib\net47\Ansys.Artemis.Aim.Integration.Daily.Tests.dll"
/Settings:"E:\ANSYSDev\TFSAgent\_work\_temp\ctrrwcwxein.tmp.runsettings"
/Logger:"trx"
/TestAdapterPath:"E:\ANSYSDev\TFSAgent\_work\_temp\Tests"

##[error]An exception occurred while invoking executor 'executor://nunit3testexecutor/': Filter string '4\,2.7068d\,1.4408d' includes unrecognized escape sequence.

The test:
        [Test]
        [Category(TestCategories.BoundaryMonitor)]
        [Timeout(StandardTimeoutValues.ms.TenMinutes)]
        [TestCase(1, 2.944379, 1.4609, Description = "scenario 1 - ...")]
        [TestCase(2, 2.944379, 1.4609, Description = "scenario 2 - ...")]
        [TestCase(3, 2.7068, 1.4408, Description = "scenario 3 - ...")]
        [TestCase(4, 2.7068, 1.4408, Description = "scenario 4 - ...")]
        public void SolveMultiBodyVolumeMonitor(int scenario, double max, double ave)
        {...}

I have the same problem. Since test case arguments are enclosed in parenthesis ( ) in the full name of the test, they need to be escaped via \ to work. See vs test docs "Escape Sequences"
In your case the \ character needs to be escaped as well like this: \(4\\,2.7068d\\,1.4408d\)
In my opinion there should be some automatic sanitization of the test case names which performs these escapes before passing them to the TestCaseFilter parameter since I couldn't find a way to configure this manually.

We have the same issue.
[TestCase]
[TestCase("text")]
In generated TestCaseFilter string to rerun braces and quotes are not escaped.  But there is no way to edit it manually.
wrong generated string
/TestCaseFilter:"FullyQualifiedName=ConsoleApp3.ProgramTests.TestNameNoParams()"
/TestCaseFilter:"FullyQualifiedName=ConsoleApp3.ProgramTests.TestName(\"text\")"
should be
/TestCaseFilter:"FullyQualifiedName=ConsoleApp3.ProgramTests.TestNameNoParams\(\)"
/TestCaseFilter:"FullyQualifiedName=ConsoleApp3.ProgramTests.TestName\(\""text\""\)"
" -> \""
( -> \(
) -> \)
I have prepared test solution and file with example of correct and broken string.
https://github.com/g00dass/azure-pipeline-test-issue

Same issue with test cases, and rerunning due to the ( and ) characters.
Also, some of our tests are being named by other nunit attributes that apply the ( and ) characters within the fully qualified name. I'm unable to identify a way to correct this through nunit.

We have the same issue. Do you folks have any estimations on when it's going to be fixed? If you have, could you please kindly share them?

We've identified a work around for this issue. If you select "Specify a batch size" under the "Advanced execution options" the tests that include ( and/or ) in their names will rerun

Similar issues reported in https://github.com/microsoft/azure-pipelines-tasks/issues/10709 and https://github.com/microsoft/azure-pipelines-tasks/issues/10932

HI,
any update here?

Hello,
Also waiting for the fix. As of now, I get either error in Log (if allow to batch tests automatically) or incorrect amount of tests in summary. Parametrized tests describe different behavior, have different Test Cases, it is more convenient to treat them as different tests.
I use MSTest DaraRowAttribute.
Error in Log:
##[error]Error occurred while publishing test results : System.Exception: No tests were discovered while attempting to rerun failed tests. This is likely to happen if these tests are xunit or nunit datadriven/parameterized tests or tests with custom display names with spaces or special characters which are not supported with rerun. Please reach out to us at https://github.com/microsoft/azure-pipelines-tasks/issues if you are hitting this for tests other than the ones mentioned before.

Same as @gbatig - we use MSTest and DataRows. Test re-running has been failing for a few months now, but I've only just found this bug.
For us, a typical failure when re-running failed tests would look like:
> vstest.console.exe /TestCaseFilter:"FullyQualifiedName=[...].Ticks_skipped_if_expired (-2\,True)|FullyQualifiedName=[...].Ticks_skipped_if_expired (-1\,False)|FullyQualifiedName=[...].Ticks_skipped_if_expired (0\,False)"

Starting test execution, please wait...
##[error]Filter string '-2\,True' includes unrecognized escape sequence.
No test matches the given testcase filter '...' in [...]
##[error]Test Run Failed.
Vstest.console.exe exited with code 1.

##[error]Error occurred while publishing test results : System.Exception: No tests were discovered while attempting to rerun failed tests. This is likely to happen if these tests are xunit or nunit datadriven/parameterized tests or tests with custom display names with spaces or special characters which are not supported with rerun. Please reach out to us at https://github.com/microsoft/azure-pipelines-tasks/issues if you are hitting this for tests other than the ones mentioned before.
Execution Result Code 1 is non zero, checking for failed results

This particular test looks like this in code:

It looks like the logic to produce the TestCaseFilter value is not escaping the string correctly.

Same as @gbatig we are using MSTest and Datarow's, and this is for our UI Tests, so can occassionally fail which is why we have the rerun... then this makes it crash out which is pretty frustrating.
Seems simple to duplicate so a bit annoying there has been no fix.
Will try the @mike9900 workaround.

No reason to close afaik

You won't get the escaping error with the workaround but MSTest seperate DataRows are not identified anymore.
With batchsize:

Without:

We are also seeing this issue and it seems to have started after we upgraded to Azure DevOps Server 2022.1 so I guess we have been blissfully unaffected until we got the updated build/test tasks.
The failure means that the test is no longer categorized as a flaky test (as it should) but 'just' as a regular failing test in that build.
I guess we would have no issues if we just didn't have any flaky test, but I don't see that happening anytime soon.

Any update on this issue?

Any chance that we could get a status on the fix for this regression that spoils test stability?

I use to be able to use the Specify Batch advanced configuration to get this to work. Something Changed and now when I run the Specify Batch size it will not use my Filter Criteria when its supposed to be rerunning failed tests saying ThisWillNotBeUsed and attempts to run all of my tests.

Similar issue was observered but there was no fix https://github.com/nunit/nunit3-vs-adapter/issues/817

We're seeing this as well. The Specify Batch size workaround works for us.

The problem for me is that batch stopped working like it was. Now batch
runs all tests not using the filter I have setup for the tests.
On Wed, Jul 3, 2024, 4:29 PM genesisjschubert @.***>
wrote:

We're seeing this as well. The Specify Batch size workaround works for us.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azure-pipelines-tasks/issues/16187#issuecomment-2207199415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGVOYPHFEUG7CIYDSQ3P5TZKRNJXAVCNFSM5ULQAA22U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMRQG4YTSOJUGE2Q
.
You are receiving this because you commented.Message ID:
@.***>

I had the the issue with NUnit tests.
The batch size workaround here worked for me
Thanks @mike9900  and @IFYates

Yeah that use to work for me but now when I run that it can't find
something then attempts to run all my tests.
On Wed, Oct 9, 2024, 4:36 PM anthony-costar @.***>
wrote:

I had the the issue with NUnit tests.
The batch size workaround here
https://github.com/microsoft/azure-pipelines-tasks/issues/16187#issuecomment-1508935000
worked for me
Thanks @mike9900 https://github.com/mike9900 and @IFYates
https://github.com/IFYates
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azure-pipelines-tasks/issues/16187#issuecomment-2403383824,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGVOYKA6ZNIPKCVYFG4PTDZ2WHWDAVCNFSM5ULQAA22U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TENBQGMZTQMZYGI2A
.
You are receiving this because you commented.Message ID:
@.***>


Self-closing JSX tags mess with indentation

Hopefully this gif explains it:

Is anybody able to reproduce this? Is this expected behavior?

The only way I can recreate this is by having the react package enabled/active? This package monkey patches the indentation atom code. If this is the case I suggest you disable it and restart Atom and try again.
I also have a package setting Auto Indent JSX that will always indent JSX correctly regardless. See the readme for more info.

Closing this issue. Re-open if required.

Thank you, that resolved it. I really appreciate your assistance!

HI all, I'm brand new to Atom and React. I'm trying to take a Udemy React course, and I literally have no idea what packages I need in Atom, but I'm getting the same error as stated above. I have the React and Language Babel packages installed. Which is preferred for learning Atom and React? Is there a reason I should choose one over the other? Thanks!


Cannot approve request

Describe the bug
While trying to approve a session proposal an error occurs.
Error Message

JsonRpcInteractor: Received unknown object type

This is the approval message data;

Approve
(
proposerPublicKey=PublicKey,
namespaces={
eip155=Session(

accounts=[eip155:1:0xAddress],
methods=[eth_sendTransaction, eth_signTransaction, eth_sign, personal_sign, eth_signTypedData],
events=[chainChanged, accountsChanged],
extensions=[])
},
relayProtocol=irn
)

SDK Version

Client: Kotlin
Version 1.6.0
Android Project is Java

To Reproduce
Steps to reproduce the behavior:

Go to https://react-app.walletconnect.com
Select Ethereum chain and click on Connect
Scan QR
Approve proposal

Expected behavior
Successful connection establishment.
UnExpected behavior
Connection fails but when the page is refreshed an unknown wallet can be seen on previous connections. Clicking the "Unknown Wallet" sends a new connection proposal but connection fails again with the same error. App metadata is already set, so the metadata is not working.

`        fun sessionSettle(
requestId: Long,
proposal: SignParams.SessionProposeParams,
sessionTopic: Topic,
pairingTopic: Topic,
) {
val selfPublicKey = crypto.getSelfPublicFromKeyAgreement(sessionTopic)
val selfParticipant = SessionParticipantVO(selfPublicKey.keyAsHex, selfAppMetaData)
val sessionExpiry = ACTIVE_SESSION
val unacknowledgedSession = SessionVO.createUnacknowledgedSession(sessionTopic, proposal, selfParticipant, sessionExpiry, namespaces)
        try {
            sessionStorageRepository.insertSession(unacknowledgedSession, pairingTopic, requestId)
            metadataStorageRepository.insertOrAbortMetadata(sessionTopic, selfAppMetaData, AppMetaDataType.SELF)
            metadataStorageRepository.insertOrAbortMetadata(sessionTopic, proposal.proposer.metadata, AppMetaDataType.PEER)
            val params = proposal.toSessionSettleParams(selfParticipant, sessionExpiry, namespaces)
            val sessionSettle = SignRpc.SessionSettle(id = generateId(), params = params)
            val irnParams = IrnParams(Tags.SESSION_SETTLE, Ttl(FIVE_MINUTES_IN_SECONDS))

            **jsonRpcInteractor.publishJsonRpcRequest(sessionTopic, irnParams, sessionSettle, onFailure = { error -> onFailure(error) })**
        } catch (e: SQLiteException) {
            sessionStorageRepository.deleteSession(sessionTopic)
            // todo: missing metadata deletion. Also check other try catches
            onFailure(e)
        }
    }`

The error fires at selected line

Could you provide us with a sample payload in JSON format?

{
"id":<PHONE_NUMBER>919,
"jsonrpc": "2.0",
"method": "wc_sessionSettle",
"params": {
"relay": {
"protocol": "irn"
},
"controller": {
"publicKey": "db59666709780c54d8dcc115de18d82006c924f96c3ecd0e66e40c132cfa0b64",
"metadata": {
"name": "Enno Wallet",
"description": "Enno Wallet is a secure, multi-chain crypto wallet and decentralized finance gateway",
"url": "https://ennowallet.com",
"icons": ["https://ennowallet.com/assets/ennowallet_icon_ennocash.svg"],
"redirect": {}
}
},
"namespaces": {
"eip155": {
"accounts": ["eip155:1:0x6C2ea434Be6Ba7B56b62AeE27A574e4b39f5dabf"],
"methods": ["eth_sendTransaction", "eth_signTransaction", "eth_sign", "personal_sign", "eth_signTypedData"],
"events": ["chainChanged", "accountsChanged"],
"extension": []
}
},
"expiry":<PHONE_NUMBER>
}
}

Tried the new release;
implementation("com.walletconnect:android-core:1.8.0")
implementation("com.walletconnect:sign:2.6.0")

unfortunately still the same

Hey @hiletmis!
The error: JsonRpcInteractor: Received unknown object type indicates that there's sth wrong when receiving the subscription message not with sending. Could you please double-check and provide all error messages that you face?
Is your project open-source so that I can have a look at your Java implementation?
If your project is not available, can I have a code snipped with the initialization?

The new release seems to be working.
Thanks.

As you mentioned before the problem is json response from the dapp. I tried to intercept the json message that I received from the dapp and this is what I get;
{
"id":<PHONE_NUMBER>733,
"jsonrpc": "2.0",
"method": "irn_subscription",
"params": {
"id": "97e3c549aae5c486084cb10ac5fb3a16900d2aec083e80f9a6f52a1868b54542",
"data": {
"topic": "b292ca3c172be2c863225967ad9bdfa8637fe945b05f4ab79ca720e992d3c0b3",
"message": "AO3WlbJyWYgf2dlmNiWVZvdKZrUrn0xWfb1E+KF/XlhZCKR1H541orGLABq0mlxQpnL1On4IvDltDExYuiAITcDatxCJ2fkreKUurwSgYx/ZaLb+UB+EwehLxQ9x/ScnBqLlQ9kxozOKlEF+h7af/8meZDqSKHpPfjvMKQCMS+6DYqfeAbjJy1Skb/xtMKjixzkkddt1/8LZ0T2mhhP0jbv45b2W2tbt3XHth+IxfeUEcCjh+NtRIsNOM3kpYe4KooV3/BOhShC/csv4IStXp6sLtx7UycEmu3gOikjsH/xcWoBvudbnRT0vzi6ZqAb3Q+F9kGo6Cey7fHPWePPS0kTXeYYlStifuOI9lF8rsxaiESJz06m+oiXA+z+HChn+1DLdz26NBdWUFvoBNs1ENkoVMmkDVFML7kts6zjfYYDcAmYoqbA71d2Hxskpzl7xHw+fuUI+m2bzM2L176w3vuRzDOJ6DBArcHemIf+Gck3IYJtBVN8dnvB/ehshciXpkOaucAInskoLT1etxrzphDlKPPd5ZbWGp5VULQ/0LPjHXjEpM6JsnD1H+df1qdv9Pm/DvoF5kNEMCy+Q9YqZdytUAvFBs8YCNffn4tk76eSDgVz38YOPjfzNM9ND6qvv9KirqeeFQMomplcQ8SrZdh2i5I212IfEpcZGup88U5BxgiFuIU65pcc6qBejMzhRGD5h4fEDRNYuOluNIKi5g3UC6TJCfDJXIhD3YplkeL4OadWahezuJTiSADxyhhTJpOCkcY6YvkK2lLq41x/Zjd1aWrOHnlnuMjtpefa2S8rmYGspTPv4uFsy8aNUih7LeoUZUQ+GO7bZ6jJeomQVlBRBObR7lB9J8yem8ylTNMa/5eIJAIaXspjTcBMJS81ta2Yq7hG4VpKD00U53y1832Nyui1UzBnaAEW3XkWhV3sFpvy/r4cBdit3PLe7RrYklck6p99MlKowLJDv3iwGtU3Wx2Bea4zP7YqYV1jbQ4M6",
"publishedAt":<PHONE_NUMBER>420
}
}
}
the response of the official walletconnnect test dapp works properly.
but when trying to connect to another website I get the same error I have mentioned before (JsonRpcInteractor). although the iOS version accepts the same response from the same dapp request.
I think this is a dapp error but for some other reasons the response is only not acceptable by android sdk.
DApp:
https://swop.fi/exchange
How to connect:
Connect -> Keeper Mobile


Is it possible to use ApplicationDefaultCredentials for SignUrlOption?

Currently Storage.SignUrlOption receives com.google.gcloud.AuthCredentials.ServiceAccountAuthCredentials, but how to use ApplicationDefaultAuthCredentials with that method? I think we should allow using ApplicationDefaultAuthCredentials if possible.

I think there should be an interface for Credentials and use it as the argument, just my 2 cents

Indeed as of now you can only explicitly set com.google.gcloud.AuthCredentials.ServiceAccountAuthCredentials.
However if you don't specify it signUrl may still work based on your service auth config (if it is configured with AuthCredentials.ServiceAccountAuthCredentials or using ApplicationDefaultAuthCredentials AND the default auth config is associated with a service account.
ApplicationDefaultAuthCredentials is not required to be associated with a service account (and provide private-key).

Ok, thanks! I didn't know that you can not use the GCE service account for signing URL. However, isn't it appropriate for that method to receive an interface? Especially if there is a possibility that GCE service account can be used for signing in the future (I'm not sure though).
Related:
http://googlecloudplatform.github.io/gcloud-python/stable/storage-blobs.html#gcloud.storage.blob.Blob.generate_signed_url
https://github.com/GoogleCloudPlatform/gcloud-python/issues/922

Yes, we could do something similar to gcloud-python and accept AuthCredentials instead of the specific ServiceAccountAuthCredentials but similar to the Python API this class does not expose
in its API a service-account or private key.
We could still get it and then reject any instances that are not known to us to support the signing.
I don't like that such enforcement is a run-time failure but maybe that is not too bad as a runtime
failure is already possible when not providing anything.
Also, I don't like that this way a user could not implement its own AuthCrendentials to work
with the signing as the signature does not specify it.
We could define a Signing interface and make some of our AuthCrendentials credentials implement it.
I am +1 for it. Is that what you are suggesting?
@ajkannan is it something that you will be interested in and have time for it?

@aozarov Yeah, that seems a right approach!

Sure, I can take this issue.

Fixed with #854


Add Terraform configs format check.

#10

I am not too sure about this.. the configs themselves should be tested with go_test and fmt should be done as part of the engine binary.
The reason is there are some cases where we can't know what the final indentation etc will look like e.g. when we have optional vars being added.
Would having a go_test for all examples and running fmt on the generated configs automatically fix this?

I am not too sure about this.. the configs themselves should be tested with go_test and fmt should be done as part of the engine binary.
The reason is there are some cases where we can't know what the final indentation etc will look like e.g. when we have optional vars being added.
Would having a go_test for all examples and running fmt on the generated configs automatically fix this?
Yes. That will fix it. The test in the PR is just temporary so that before we have those implemented, and as we are actively make changes to the templates. I would like to have something very simple just to make sure they are also sanity checked. Because right now, everytime I make changes to the templates, I do that manually locally.

Let me add this as a test after #21. It shouldn't take too long, just need to do a bit of minor refactoring.

sgtm


Visual Captioner

Does this code includes visual captioner ?

We did not release the code for extracting captions. However, we provided the extracted captions in the data link. If you want to try other captioner, you just need to format the captions as ours, then you should be able to directly run this codebase.

actually  you have provided the model link for pretrained visual captioner  (LaViLa model ) but can you help me  to load and run this model
because

I am getting like these

Dict object is not callable

You can refer to the LaViLa codebase (https://github.com/facebookresearch/LaViLa) for how to use it. Our checkpoint follows the same structure as the LaViLa base (TSF-B + GPT-2) model.
For inference, you can refer to the script that LaViLa's authors provided: https://colab.research.google.com/drive/1gHWiEWywIotRivYQTR-8NQ6GJC7sJUe4.

what are the values  of the parameters that you have taken to get that accuracy
for mine everything its giving the response as -1 and the accuracy is 0

Can you give me more details about your experiments? I setup this repo again and everything works fine on my device.


small citation plugin fixes

Overview

started by looking into that weird htmlsafe import, but also found a bunch of small problems

fix(i18n): fix typo
fix(citation): add missing argument to citation card in dummy app
fix(citation): remove the unnecessary type and add the html-safe tag at the rendering site
fix(citation): correct erroneous arguments to AuAlert

connected issues and PRs:

relates to #159
Setup

How to test/reproduce

citations without articles now show a proper message instead of an empty square
the citation sidebar card actually works now on the dummy app
Checks PR readiness

[x] UI: works on smaller screen sizes
[x] UI: feedback for any loading/error states
[x] Check if dummy app is correctly updated
[x] Check cancel/go-back flows
[x] changelog
[x] npm lint
[x] no new deprecations

When searching for citations and nothing is found, there is still a grey box:

Not really part of what you fixed, but as you made some error UI changes, might be an idea to fix this too.
I couldn't test the case of no articles, because I don't know which citation has no articles :)

@x-m-el not sure what's going wrong on your end but here it renders correctly?

@x-m-el not sure what's going wrong on your end but here it renders correctly?

Yes, my bad. Probably wrong branch or something :)
approved :+1:


Add a settings object, along with a couple methods for it

Proposed Changes
Adds a Settings object along with a couple methods for it.
This was originally going to be more comprehensive but my need for these bindings has vanished. However I figured someone else might still find the work I did accomplish to be useful.

Note to @he4d, Clippy report a lot of error remaining of precedent commits. We should clean up the Clippy warning in the current code to prevent false alarms of the CI.

Done!

I'm gonna be up front about something: I'm not willing to do further work on this PR. It's being offered as-is. If I get a request for change I will probably close the PR. I've left the systemd sphere in my professional life and don't want to setup the environment necessary to work on this. No hard feelings if you close this PR.


SCSS-Lint logo

I think it would be really great if SCSS-Lint had a better identity and I think that a good logo would go a long way. I figured I'd open this issue to collect ideas. I have some friends who have had reasonable luck using 99designs so something like that might be a reasonable option, but I think it would be best if it came from the community.
@ericam @hcatlin @malrase @jina you are active in the Sass community and either are or know good designers. Want to pitch in? ;) I'm thinking something pink would probably be the way to go, but after that I'm out of ideas.

:+1:

Quickly throwing something together from the Sass brand page:

I'm no designer, but if we're gunning for a 1.0 release as discussed in #725 then we should probably have something as a logo. Would love other suggestions.

I think this is a pretty good start! Here's my feedback:

I think the icon should be on the left
The text seems unbalanced with the icon. I think maybe either the icon is too small in comparison and/or the text is too bold. Maybe adjust the proportions a little?
While I like the intention, I'm not 100% on the red dotted underline. It is pretty subtle, the color doesn't feel right, and it makes me feel like something is wrong. JSCS uses a similar thing, but I think more effectively. I think we should just remove it.

Here's something I hacked up real quick based off of your idea. Typeface is Avenir Next (Medium and Italic) with a -1% letter spacing for better kerning. I like the italics because I think it implies that this tool can help your team go faster.

Then I got the idea to combine your dotted line and the cog and came up with this:

Which I quite like, although we'll need to fix the dashed line so each segment is the same size.
If you like one of these, I can try to work up a vector version. Although I don't have a vector program so I'll need to figure that out first, lol. Or, if you happen to have Illustrator or something handy and want to run with this, that sounds good to me too.
Also, I think we really need to chat with the Sass folks and make sure that it is okay to use their seal in our logo before we go forward with this concept. Once we agree on a concept, I'd be happy to reach out to them.

Nice! I find your last one very aesthetically pleasing. The dotted lines maintain the playfulness of the original logo, which I think people associate with the Sass brand (at least I do).
We're good to manipulate the logo. The Sass brand guidelines mention the work is licensed under Creative Commons Attribution-NonCommercial-ShareAlike 3.0 Unported License, which means we can modify as long as we mention we modified it and link back. Their only usage rule is to be respectful in its use (no sexist imagery), and I think we're good on that front.

Great sleuthing on the license. I reached out to Hampton anyway just to make sure we are cool, but I think he is in Japan so it might take him a while to get back to me.

I like it! It's fine usage-wise. If you want design blessing, ask Jina Bolton!
On Sun, Feb 21, 2016 at 7:57 AM -0800, "Joe Lencioni"<EMAIL_ADDRESS>wrote:
I think this is a pretty good start! Here's my feedback:
I think the icon should be on the left
The text seems unbalanced with the icon. I think maybe either the icon is too small in comparison and/or the text is too bold. Maybe adjust the proportions a little?
While I like the intention, I'm not 100% on the red dotted underline. It is pretty subtle, the color doesn't feel right, and it makes me feel like something is wrong. JSCS uses a similar thing, but I think more effectively. I think we should just remove it.
Here's something I hacked up real quick based off of your idea. Typeface is Avenir Next (Medium and Italic) with a -1% letter spacing for better kerning. I like the italics because I think it implies that this tool can help your team go faster.
Then I got the idea to combine your dotted line and the cog and came up with this:
Which I quite like, although we'll need to fix the dashed line so each segment is the same size.
If you like one of these, I can try to work up a vector version. Although I don't have a vector program so I'll need to figure that out first, lol. Or, if you happen to have Illustrator or something handy and want to run with this, that sounds good to me too.
Also, I think we really need to chat with the Sass folks and make sure that it is okay to use their seal in our logo before we go forward with this concept. Once we agree on a concept, I'd be happy to reach out to them.
—
Reply to this email directly or view it on GitHub.

I like the dashed line, although I think maybe "sass" should use a bolder weight and/or a rounded font? I also kind of preferred the pink text, but I don't feel too strongly on that front.

I like the last logo on this page (dashed line, black text).
If you want some design feedback:
The kerning looks like it could use some love, but otherwise I'm fine with the font and weight. May I suggest trying to swap the weight&styles so that "lint" is more emphasized than the "scss" part?
But otherwise this is your project. Do as you like! :)

Thanks @jina! I didn't have a proper image editing program so I couldn't adjust kerning letter-by-letter, but I was planning on tweaking all of that when I do it in vectors.

I've gone ahead and added the logo in question to the README. Even if it still needs work, it looks "good enough" to me and gives the project a more welcoming home page.
Thanks for your work on this!


Modal does not close on confirm?

Hi,
I've downloaded confirm_with_reveal.js just now and implemented like this:

Foundation from here: https://packagist.org/packages/bmatzner/foundation-bundle

script src="/bundles/bmatznerfoundation/js/foundation/foundation.js">
script src="/bundles/bmatznerfoundation/js/foundation/foundation.reveal.js">
script src="/bundles/agrihealthahp/js/offlineHandler.js">   <-- thats mine
script src="/bundles/agrihealthahp/js/confirm_with_reveal.js">

              button data-confirm="{"ok":"Yup","cancel":"Nope"}">Delete /button>

I catch the confirm event with:
jQuery('#' + model + '_edit .form_actions.delete').show().on('confirm.reveal', null, { name: model, rowId: rowId }, call.deleteRecord );
which works fine. However the modal does not close, I'm using
jQuery('.reveal-modal').hide();
jQuery('.reveal-modal-bg').hide();
to close it manually.
It is closing on Cancel.
Any idea why  this would be happening?

Currently, the plugin does not close the dialog when the confirmation button is pressed — this was by design, for what I felt was the more common case of these dialogs triggering full page refreshes as opposed to Ajax or front-end MVC updates (the idea being that leaving the dialog up while the new page loads prevents any other actions from being taken in the mean time if the response is not immediate).
I will see about adding a configuration option for this behavior, probably something along the lines of close_after_ok or the like, for cases where people want to use this to confirm actions that don't entail a page refresh.

Cheers, I can live with that. However it it was to close by default, the people using it to link to other pages  would not have any downside either?

The only downside to closing the dialog by default when the OK button is hit is that if the dialog is confirming (for example) deletion of a resource, or some other important action, and the server response takes more than a few tenths of a second to refresh the page in the user's browser, the user may mistakenly think that the action was carried out via Ajax, and navigate off the page before the response arrives.
I recognize the value of having a close_after_ok option and will look into adding that, but I think the safer or more conservative setting is to have that behavior as a configuration option that must be explicitly chosen by the developer using the plugin.

Fixed with this in code on 49 line, before:
if ($el.is('form, :input')) {
return $el.closest('form').removeAttr('data-confirm').submit();
}
After:
if ($el.is('form, :input')) {
modal.foundation('reveal', 'close');
return $el.closest('form').removeAttr('data-confirm').submit();
}

I agree closing the modal when clicking confirm should become the deafult. (Nowadays most apps are SPA i.e. there's no full page reload after clicking confirm.)
However IMHO modal.foundation('reveal', 'close'); should be one line above (outside of the if clause).


Create 012_SUM_OF_TWO_ARRAYS

You are given a number n1, representing the size of array a1.
You are given n1 numbers, representing elements of array a1.
You are given a number n2, representing the size of array a2.
You are given n2 numbers, representing elements of array a2.
The two arrays represent digits of two numbers.
You are required to add the numbers represented by two arrays and print the arrays.

Input Format
A number n1
n1 number of elements line separated
A number n2
n2 number of elements line separated
Output Format
A number representing sum of two numbers, represented by two arrays.

can I do this request?

Can I bear this request?


[Suggestion] Use youtube-dl to accept youtube links

Would be really helpful to include https://rg3.github.io/youtube-dl/ and use it to automatically download and import mp3s from youtube.

I felt this would be good if I could use this to download a playlist that contains a OST.

This was addressed in 33faf30520d2cd4d2cbdf87b3ea9c9009a1e1cc4
Thanks!


PR MS2: UI- "Book" functionality

I tried clicking "book". But I was unable to book the ride.

We have not added a Pop-Up button yet which shows the booking confirmation.
If you have provided the right email address, then you should have received a mail.

That seems to be nice. Then please change this task as a backlog item or an
enhancement appropriately. Or we can close this issue if the idea was not
in the scope of the requirements for these milestones.
Thanks ,
Harish.
On Wed, Mar 6, 2019 at 12:07 AM iarora93<EMAIL_ADDRESS>wrote:

We have not added a Pop-Up button yet which shows the booking confirmation.
If you have provided the right email address, then you should have
received a mail.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airavata-courses/CookieArmy/issues/58#issuecomment-469970351,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ApWZfnKXEfwLVXEuvx2qLvzdV1NKy8Mpks5vT00SgaJpZM4bbf0W
.


xnContextRunXmlScriptFromFileEx() crashes on repeated loading of config with USER node

I believe I've isolated an issue with the new generation codebases of:
OpenNI <IP_ADDRESS> for Win32
PrimeSense NITE <IP_ADDRESS> for Win32
PrimeSense SensorKinect Win32 Device Driver v0.7 (based on Primesense <IP_ADDRESS>)
Given an XML config file which has a DEPTH, IMAGE, and USER nodes, then repeated reloading of it using xnContextRunXmlScriptFromFileEx() causes a crash in less than 10 reloads.
If there is no USER node, then I can reload it over 100 times (tested) with no issues.
Before each reload, nodes are being released. The crash is not occurring in the releasing. It is instead occurring within xnContextRunXmlScriptFromFileEx()

This is fixed in NITE 1.5 and above.
Thanks,
Ziv.


Eval returns length of an empty string as undefined

var jexl = require("jexl");
jexl.eval("name.length", {name: ""}).then(console.log);
// undefined
jexl.eval("name.length == 0", {name: ""}).then(console.log);
// false
jexl.eval("name.length == undefined", {name: ""}).then(console.log);
// true

Confirmed! Will dive in.

Fix written, release pending.

Published fix in 1.1.4; will be rolled in to the 2.0 branch soon.

That's great! Thank you


About x264: why use a github mirror repo instead of a videolan repo?

I've noticed that the build script used to use the videolan repo
https://github.com/BtbN/FFmpeg-Builds/blob/b4bbce3020bbb1de61a7cc1361a8cd57ad417d91/scripts.d/50-x264.sh#L3-L4
then switched to the github mirror repo
https://github.com/BtbN/FFmpeg-Builds/blob/3119034c9234be55afaccecfe3a457bb4e0eb402/scripts.d/50-x264.sh#L3-L4
but the videolan repo seems to be more actively maintained, so I'd like to know why

Cause videolan infrastructure is too unreliable, specially in regards to their peering with github.
Also, the army of builds this repo and its forks produce is quite a substantial server and bandwidth load, so whenever a project has a Github mirror, I will prefer it.

On github, Anton Mitrofanov
, one of the primarily developer of x264, has a x264 repo that maintains a more active than the mirror/x264 repo, could this be a better alternative?

Supposedly videolan fixed their peering issues with Github, so I'll give it a try again.
Cachine has improved a lot since then as well.


Deploy/Start Instance Modal Doesn't Open until Endpoint Was Retrieved

Describe the Bug

When clicking the button to open either the deploy or start instance modal without an engine running nothing happens for a few seconds.

Steps to Reproduce

Open Camunda Modeler without engine running
Click button to open deploy/start instance modal
Nothing happens for a few seconds

I've identified this place as the source of the behavior. getDefaultEndpoint checks for a running engine and times out if there is none. Until the timeout, the modal doesn't open.
Expected Behavior

I see two possibilities:

open the modal immediately and retrieve the endpoint in a non-blocking way
indicate loading until the endpoint was retrieved

Environment

Camunda Modeler Version: v4.8.1

I believe we should go for option :one:; open modal immediately. For the user it is not transparent what is going on and why opening a modal would take so long.
/CC @andreasgeier

The initial call to the engine before we open the modal is required to determine the default endpoint. If we detect Tomcat distro, http://localhost:8080/engine-rest is displayed. Otherwise, it's http://localhost:8080/rest. We could also just open the modal immediately and then change the endpoint if necessary.

Option 1⃣,  opening the modal immediately and indicating that the endpoint is still being retrieved would be my preferred solution.


Plan Comet 0.1.0 Release

What is the problem the feature request solves?
During the Comet public meeting this morning, there were questions about when the first official release would be. We do not really have an answer to that yet, but we can use this issue to discuss it.
Here are some ideas for milestones that we may want to achieve before creating an official source release (note that we do not necessarily need to create binary releases right away).

[ ] https://github.com/apache/datafusion-comet/issues/524
[x] Demonstrate performance improvements with TPC-H benchmarks (see updated README)
[x] https://github.com/apache/datafusion-comet/issues/394
[x] https://github.com/apache/datafusion-comet/issues/459
[x] https://github.com/apache/datafusion-comet/issues/248
[x] https://github.com/apache/datafusion-comet/issues/142
[ ] https://github.com/apache/datafusion-comet/issues/387
[ ] Upgrade to the official DataFusion 40.0.0 release

Nice to have:

[ ] https://github.com/apache/datafusion-comet/issues/510

Related tasks:

[ ] Prepare blog post announcing the release. Google doc for collaborating on this is here.

Describe the potential solution
No response
Additional context
No response

Ensure that currently supported operators and expressions are fully compatible with all supported Spark versions

I think this is good requirement for the first release

Achieve 100% coverage for TPC-H and/or TPC-DS benchmark with a clear performance advantage

TPC-H for sure. TPC-DS may be a stretch goal.

One benefit of creating a release is that it is a good opportunity to write a blog post to announce the release and provide an update on the status of the project, and try and encourage more people to contribute. It can also demonstrate the momentum that the project has. I suppose I am now making an argument against waiting until we have great benchmark results before making the first release. I would be interested to hear opinions on this.

I propose that we create the 0.1.0 source release as soon as we have upated the project to use the upcoming DataFusion 39.0.0 release which should be avalable around June 10.

+1. But we may need new arrow-rs release 52.0.0 too. Will it be released before DataFusion 39.0.0?

Here is the tracking issue for arrow-rs 52: https://github.com/apache/arrow-rs/issues/5688
It should be available next week. I updated the DataFusion 39 release issue to add this to the prerequisites for the release: https://github.com/apache/datafusion/issues/10517

I created a Google doc for the community to collaborate on a blog post announcing the release:
https://docs.google.com/document/d/1rnxnbi66oFr5B-OTUxtpi9pnifOxNkmvOVIRTN0BfhY/edit?usp=sharing

We plan to do binary release, although it might not be able to catch up the 0.1.0 source release. Publishing to Maven repo needs more works to do. Comet involves native code, so it becomes more complicated than pure Java/Scala projects. We need to include pre-built binaries for different platforms in the published jar.

I assume the source release will tag the repo with a release-0.1.0 tag. Even though a maven artifact would not be published, it does allow projects to build their own, or even add comet as a git submodule, based on a relatively 'stable' version.

I assume the source release will tag the repo with a release-0.1.0 tag. Even though a maven artifact would not be published, it does allow projects to build their own, or even add comet as a git submodule, based on a relatively 'stable' version.

Yes, absolutely. We (or anyone) can choose to release binary artifacts from the source release or the tag in the repo. ASF does not have any special involvement in that.

I created a milestone where we can track the priority issues for the 0.1.0 release
https://github.com/apache/datafusion-comet/milestone/1

I think we are getting close to being able to release 0.1.0 now that we are using an official DataFusion release again (or will be in a few days when DF 40 is released to crates.io).
There a few remaining issues in the 0.1.0 milestone, the most important ones (IMO) being:

https://github.com/apache/datafusion-comet/issues/524
https://github.com/apache/datafusion-comet/issues/387

@parthchandra @viirya is there anything else that you think we need to address for the first source release?

is there anything else that you think we need to address for the first source release?

No. I think the above two issues are most notable at the moment.

is there anything else that you think we need to address for the first source release?

No. I think the above two issues are most notable at the moment.

I think this is good.

@viirya @parthchandra I no longer think that it is critical to fix https://github.com/apache/datafusion-comet/issues/387 before we release, because users can already enable shuffle, so this is just a config change from user point of view.
If there are no objections, I will plan on creating 0.1.0-rc1 next week, after review documentation to make sure all known issues are documented.

+1

I no longer think that it is critical to fix #387 before we release, because users can already enable shuffle, so this is just a config change from user point of view.

Agreed


send cipher metrics separately for every agent

I can see exponential grow of counters in cipher metrics:
cipher:{agent_name:"downloader" failure_count:2692991220776944783 last_failure:{seconds:1629394310 nanos:428869571} tc:{} tc:{error_code:CIPHER_ERROR_NOT_READY} tc:{error_code:CIPHER_ERROR_DECRYPT_FAILED} tc:{error_code:CIPHER_ERROR_UNMARSHAL_FAILED} tc:{error_code:CIPHER_ERROR_CLEARTEXT_FALLBACK} tc:{error_code:CIPHER_ERROR_MISSING_FALLBACK} tc:{error_code:CIPHER_ERROR_NO_CIPHER count:10138609964923216130} tc:{error_code:CIPHER_ERROR_NO_DATA count:5392372803064314065}}

Seems, it comes from the usage of the same global CipherMetricsMap in cipher/metrics.go for every agent. So, we need to split our publications to use different key and CipherMetrics for every publish.
Signed-off-by: Petr Fedchenkov<EMAIL_ADDRESS>

#2247 solves the problem and have additional care for details


Replace Luabase API with Forta labels API

@kovart  Luabase is planning to sunset the query API in early February, so the Malicious Account Funding bot will not be able to access addresses with malicious labels. To prevent any disruption, we created a new API to replace Luabase.
You can now query the addresses with the malicious labels with the following endpoint:
Get addresses with labels exploit, heist, and phish --> https://api.forta.network/labels/state?labels=exploit,heist,phish-hack&sourceIds=etherscan

Get address name tags by wildcard query
https://api.forta.network/labels/state?labels=*xploit*,*hish*&sourceIds=etherscan-tags

For polygon tags --> set sourceIds=polygon-tags
For fantom tags --> set sourceIds=fantom-tags
Here's an example getting an ethereum tag for the address 0xe74b28c2eAe8679e3cCc3a94d5d0dE83CCB84705.
You can get the tag from events[idx]["label"]["label"]
$ curl "https://api.forta.network/labels/state?entities=0xe74b28c2eAe8679e3cCc3a94d5d0dE83CCB84705&sourceIds=etherscan-tags" | jq
{
  "events": [
    {
      "id": "0xb1f5af8d8f36fe96844974f0bfe2deed264010b8975aa37e84f37f5e990f1333",
      "created_at": "2023-01-11T19:43:03.015925716Z",
      "label": {
        "label": "Wintermute Exploiter",
        "confidence": 1,
        "entity": "0xe74b28c2eae8679e3ccc3a94d5d0de83ccb84705",
        "entityType": "ADDRESS",
        "remove": false
      },
      "source": {
        "id": "etherscan-tags"
      }
    }
  ]
}

You can also pass in multiple addresses. They just need to be comma-separated:
 curl "https://api.forta.network/labels/state?entities=0xe74b28c2eAe8679e3cCc3a94d5d0dE83CCB84705,0x000002Cba8DfB0a86A47a415592835E17fac080a&sourceIds=etherscan-tags" | jq

For a smooth transition, could you use this API instead of Luabase before February 1st?

This was resolved.


When typing Restart in the PowerToys Run box, let the PC use sign-in info after restarting

Description of the new feature / enhancement
When we type "Restart" (without the quotation marks) in the PowerToys Run box, the PC doesn't use sign-in info after restarting. The command line is used as "shutdown /r", which completely restarts the computer without using sign-in info. Now, if we type "Restart" in the PowerToys Run box, the command line "shutdown /r /g" must be used. This allows the PC to use sign-in info and lock the user automatically.
Scenario when this would be used?
Type "Restart" (without the quotation marks) in the PowerToys Run box, and the computer will restart. After restarting, the sign-in info will be used, and the user will be automatically locked.
Supporting information
As long as the "Automatic Restart Sign-On" option is enabled in Settings->User Accounts->Sign-in options, PowerToys Run will check if the computer can restart and use sign-in info after typing "Restart" and clicking it. If "Automatic Restart Sign-On" is disabled, typing "Restart" and clicking it in PowerToys Run will fall back to "shutdown /r".

Same for shutdown with /s and /sg.

Same for shutdown with /s and /sg.

This is not supported in combination with hybrid boot.

This was fixed/implemented in the latest release (v0.83.0).


Roles update

The roles for issuefab have been changed to use the new permissions that userwidgets/model 0.5.x uses.
Its not completely backwards compatible so it is going to require changes in issuefab/app.
3 roles:

user: normal user
financialController: Got access to do everything on a given org.
admin: Got access to do everything on all orgs on a given application.

This PR requires the userwidgets/model 0.5.x update. Do not merge until that is included in this branch.
userwidgets/model 0.5.x is not included yet because it is not available.


Array support

simply matching  values, I think it would be very cool to be as powerful as elixir binary patterns later

of ["a", "b"]:
  # ye

similar to support for sequences


adds tests for every method of the assert object

100% code coverage 🎉 This PR adds tests for each method of the assert object.

thanks!


HADOOP-16950.Extend Hadoop S3a access from single endpoint to multipl…

Addresses the following issue:
https://issues.apache.org/jira/browse/HADOOP-16950

First, this is a pretty wild idea. Make sense there for things like ceph which tries to offer HA without any load balancer games: the client needs to be clever.
Second: how I review https://github.com/steveloughran/formality/blob/master/styleguide/styleguide.md
We also have a strict "no declared endpoint, no review" policy, as covered in the s3a testing doc. If you are doing something for ceph, I'm going to have to request that you also run against AWS S3 for regression testing.
We also like our s3a tests to have integration tests, with ITest prefix, sorry can verify things actually work with real S3A implementations. This is going to be pretty critical here. At the very least, a list of the same endpoint, repeated, should work for operations against the store.
And, and this might disappoint you, all PRs have to be against hadoop-trunk, currently v 3.4.0.
Regarding the code itself:
cglib & dynamic code generation vs new AmazonS3 implementation,
I really do not want cglib to be used here.

It + ASM have been brittle in the past across JVM byte code versions; it is a recurrent issue (https://github.com/GoogleContainerTools/jib/issues/2015)
it's adding complexity and maintenance grief

Can't we just wrap the Amazon S3 client instead? We do that for the inconsistent client already, after all.
the risk there is: new methods in later SDKs need to be tracked, but as its a wrap/relay to AmazonS3, were any new methods to be added the wrapper class would fail to compile "Unimplemented Method", we will find out fast. The runbook for an SDK update will need to cover this extra work, but that's all.
Tests
yes, integration tests are mandatory. Especially those exploring failure conditions. Unit tests are nice but not a substitute.
Documentation
That too. Especially extensions to the troubleshooting docs. The easiest way to add new docs would be a whole new markdown file in the site/ dir, with a reference off index.md.


A possible dashboard, for nteract, desktop and exportable to web

Hi, opening this issue to talk with @cabhishek and anyone else who is interested on how we want to design a dashboard for nteract. Ideally this would be rendered upon opening nteract without opening an actual file.

Here is one small idea to start and iterate on maybe

This reminds me why we cared about having some kind of dashboard even for the desktop app instead of giving them a fresh notebook. Not sure what my opinion is yet -- super cool mock though!

Want to post the sketch file and PNG to this repo as PR for inclusion?

Absolutely, I'll send it up soon
On Tue, Jan 17, 2017 at 1:12 PM Kyle Kelley<EMAIL_ADDRESS>wrote:

Want to post the sketch file and PNG to this repo as PR for inclusion?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nteract/mocks/issues/11#issuecomment-273300883, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKcARukb2Fq56_jrfTx0l66R3p7rEK14ks5rTS6jgaJpZM4Lf7pg
.

--
Have a wonderful day,
John J. Detlefs


sherlock: 0-unstable-2024-06-04 -> 0-unstable-2024-06-09

Description of changes
Update to 2024-06-09

Move to pytest/tox based testing.

https://github.com/sherlock-project/sherlock/compare/ef124acf34e90626f4e59ab88bba1ed6141a4126...d678908c00f16c7f6c44efc0357cef713fa96739
I am notified by the upstream as sherlock-project/sherlock/issues/2111 and sherlock-project/sherlock/issues/2127 are almost ready which will use tox and poetry instead.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

@applePrincess Upstream merged.
Not sure if your build needs builds to succeed offline or not, but in case they do (as with rpms)...
tox -e offline will exclude those unit tests that require an internet connection.
If running pytest directly, rather than through tox, pytest -m "not online" will achieve the same.

Indeed it needs to be offline to successfully build a package. (at least on Nixpkgs' GitHub Action and on Nixpkgs' hydra CI system)

Result of nixpkgs-review pr 313395 run on x86_64-linux 1

2 packages built:

sherlock
sherlock.dist

When you move a package to pkgs/by-name, you should also remove its entry from all-packages.nix.
Else eval will fail.
In addition moving to pkgs/by-name should be done in a seperate commit like https://github.com/NixOS/nixpkgs/pull/317382/commits/96f6a37b84a5f028aff95f96213853197beee4a2.

Result of nixpkgs-review pr 313395 run on x86_64-linux 1

2 packages built:

sherlock
sherlock.dist

Didn't expect this in my notifications today
We've just adopted properly tagged releases [ref], in case that makes it easier for y'all to maintain. No more nonsense version numbers or git refs are needed, with some better feature/bug consistency across package managers.
A bit easier to see when updates are needed as well, probably.


Frozen-lake tutorial

Hi there!
My question is about the plot you shown in frozen-lake example.
What does it show to us?
How can we understand that the problem is solved?

Hello, @mehrdad78.
As the number of steps increases, we can observe an increase in the average return value.
Furthermore, starting from around 10,000 steps, the return value consistently stabilizes around 1.
This indicates that the model has been successfully trained in the right direction.


style: fix sidebar on responsive design

in all page where there are charts, when toggle the sidebar the charts rerender


get instance and zone from metadata

PTAL

PTAL

PTAL

@aalexand - any comments?

Could someone merge this in for me?
I would like to do the fixes needed before we can merge in #26 on top of this change, if possible, because those changes will require a lot of changes to files which this also touched.

@ofrobots Can you merge? I don't have the Merge button enabled.

Merge is disabled because the CI/checks are not green. Coverage dropped more than an acceptable amount.

I landed this. I couldn't see anything pressing in the coverage report that should block this PR at this point.


Update layer.py

Improves CPU latency by nearly 33% on my computer when the redundant multiplication by one is not perfomed. The same trick might be applied elsewhere.

Also, you could consider saving the merged result of lora_A and lora_B as attributes for each adapter. For example:
adapter_combined_weight = torch.matmul(lora_b.weight, lora_a.weight)
During the forward pass, use this combined weight instead of performing the two matrix operations (lora_A(x) -> lora_B(x)). While this approach was less effective on the CPU, I suspect it might perform better on the GPU.
If needed, i can repeat the experiments using the GPU.

Thanks for this proposal. Could you please share how you benchmarked the time saving from dropping the scalar multiplication?
I tried something very simple and for me the difference was much smaller:
>>> a = nn.Linear(16, 4096)
>>> b = nn.Linear(4096, 16)
>>> x = torch.randn(4, 4096)
>>> %timeit a(b(x))
38.2 µs ± 465 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)
>>> %timeit a(b(x)) * 1.0
41.4 µs ± 429 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

So that's only 8% difference. And this operation is only a small part of the whole computation, so I'd think the difference would be hardly noticeable. Finally, most of the time alpha != rank, so the optimization would not apply.

Also, you could consider saving the merged result of lora_A and lora_B as attributes for each adapter. For example:
adapter_combined_weight = torch.matmul(lora_b.weight, lora_a.weight)

This has some disadvantages. First, if we save this, we require extra memory and the matrix is relatively big. Second, this operation bypasses the forward method, which could lead to hooks not being called. That can be important, e.g. for distributed training.

Initially, I did something very similar to your test. But even when performing the same test, I achieved results similar to my previous ones:
`
a = nn.Linear(16, 4096)
b = nn.Linear(4096, 16)
x = torch.randn(4, 4096)
%timeit a(b(x))
51.8 µs ± 551 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)
%timeit a(b(x)) * 1.0
68.4 µs ± 711 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)
However, when I ran the same code on a different machine, I got the following results:
%timeit a(b(x))
221 µs ± 6.33 µs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
%timeit a(b(x)) * 1.0
264 µs ± 20.9 µs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)
`

This has some disadvantages. First, if we save this, we require extra memory and the matrix is relatively big. Second, this operation bypasses the forward method, which could lead to hooks not being called. That can be important, e.g. for distributed training.

Understandable, my intention was to speed up inference, not training. During training, merging the matrices A and B defeats the purpose of LoRA. I should have phrased it better.

Thanks a lot for your further explanations. Even if I'm being skeptical, I want to emphasize that I appreciate the discussion. I would just like to avoid making the already complex code more complex if the benefit is not noticeable on a realistic task. Note that a lot more code would need to be changed to make this consistent everywhere.

The perfomance difference seems to be machine dependent.

Yes, that was to be expected, but I did not foresee the difference being that big. Could you please also test on your machine how the change would affect a normal training run? My guess would be that in the grand scheme of things it's not noticeable.

Understandable, my intention was to speed up inference, not training.

Note that for inference, it's already possible to merge the adapter into the base weight, entirely removing the overhead caused by LoRA.

Yes, that was to be expected, but I did not foresee the difference being that big. Could you please also test on your machine how the change would affect a normal training run? My guess would be that in the grand scheme of things it's not noticeable.

Yes, I will compare the code during a training run and inference, then link the code to it.


Update default WMS client settings

Updates default WMS client settings to match updates made to docker containers here: https://github.com/hmakelin/gisnav-docker/pull/8
Improves package.xml description
Adds correct LICENSE.md copyright year

Note: Accidentally clicked close before merge but got it merged in any case.


Fixed size height can cause stretching

Hi, I noticed a small problem with the style of this extension: currently the emoji height is fixed to 1.5em
https://github.com/the-turk/flarum-flamoji/blob/efce7475e439e0d3b4e6f474508cc917db63588e/less/forum.less#L47
This can cause stretching if the image has a smaller height than that value, for example old school emoticons are usually 16x16px in size, with a base font-size of 14px 1.5em will be resolved to 21px causing a stretch.
In my flarum installation I overwrote the styles to use a max-height instead of height to avoid this issue, I would suggest to do the same thing here (use max-height in combination with max-width to create a maximum size box) or to make that height customizable directly from the admin dashboard to allow the admins to change it if necessary :smiley:  .

Thanks for the report! I think max-height: 1.5em; will fix it without needing max-width definition.

Hey, sorry I missed the notification between other ones.
max-width probably will be useful with a sane value just to avoid edge cases like an image that is really short and wide, it will probably work without it I agree.


Allow to specify huggingface cache in other directory instead of ~/.cache/huggingface

This is being done with the environment variable HF_DATASETS_CACHE (see https://huggingface.co/docs/datasets/cache ).
I agree that we can add it to the config file and modify it from within the program with os.environ.

Cool, I think it is enough to mention this in the docs.