dependencies for apex and nano bus. Maybe Hermit is your cup of tea

great project. Like all projects i immediately hit dependency issues.
you might like hermit. It is not your average dependency system.
https://github.com/cashapp/hermit
It can install rust and golang or anything that's a binary, but also products of them also like some binary.
It fully supports different OS and ARCH.
it can then fully manage the env and keep each repo your working on having only the versions of the binary dependences you want.
It also very fast...
It seems like a god thing for all layers. so for nanonbus too, etc.
it will create a folder called "bin", and then mange all the binaries in there for you.
here is an example: https://github.com/cashapp/hermit/tree/master/bin

@gedw99 This is definitely interesting. We NanoBus (and building Wasm modules) there are definitely a lot of toolchains to install which we've heard creates a lot of friction. It looks like Hermit could help bootstrap our prerequisites. Thanks for suggesting this!

I'm going to close this issue and we will investigate.


[🐛 Bug]: next/cache: Handle changed type signature of Next.js 13.5+

          > Hello! I have been debugging `revalidateTag` inconsistency and found out some breaking changes of upstream.

Internal cache handling behavior of Next.js seems to be changed starting from version 13.5 and above. IncrementalCacheValue no longer provides tags attribute, results in inability of flushing cache with tags. (despite revalidate code works fine)

looks like the type signature changed in next.js :(
https://github.com/vercel/next.js/blob/canary/packages/next/src/server/response-cache/types.ts
would you be able to open a new issue for this @susemeee? i recall on your other issue you ticked the box that you'd be interested in helping fix the bug. would you like to give this issue a go? if not, just lmk and i'll be happy to look into a fix
Originally posted by @james-elicx in https://github.com/cloudflare/next-on-pages/issues/451#issuecomment-1822392271

In request headers there is x-vercel-cache-tags property exists, and I think it is okay to use it if original property is missing.
@james-elicx Could you check if it's safe to use? If so, I will make a PR with the fix.

In request headers there is x-vercel-cache-tags property exists, and I think it is okay to use it if original property is missing.

Ah yeah, I remember that header from when I first did the cache, and ended up going with the property instead because, well, it was on the object.
Sounds good to me, falling back to the header makes sense.


termtosvg: 0.6.0 -> 0.7.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/termtosvg/versions.
meta.description for termtosvg is: '"Record terminal sessions as SVG animations"'.
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
/nix/store/f5z028fl9rhranjpjn3w4p71gn33vjj4-termtosvg-0.7.0/bin/.termtosvg-wrapped passed the binary check.
/nix/store/f5z028fl9rhranjpjn3w4p71gn33vjj4-termtosvg-0.7.0/bin/termtosvg passed the binary check.
2 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 0.7.0 with grep in /nix/store/f5z028fl9rhranjpjn3w4p71gn33vjj4-termtosvg-0.7.0
directory tree listing: https://gist.github.com/6d92a509c066f3f33991dcfb09fe77a7
du listing: https://gist.github.com/d07466b4e73713d5eebb49a8d0395e12

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

4 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
termtosvg

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/f5z028fl9rhranjpjn3w4p71gn33vjj4-termtosvg-0.7.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A termtosvg https://github.com/r-ryantm/nixpkgs/archive/29481937426b6d2e233adb4948471ce9aa9cc475.tar.gz

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

cc @ma27 for testing.

@GrahamcOfBorg build termtosvg


eStorage tensor fix, eval clears operations in sequence, added one test and modified tests that were broken

This might might be too large PR. Sorry about that. Because of my mistake PR #302 is also included in this.
Two main things are that we can now create eStorage tensors with nullptr as data without undefined behaviour and evalAwait (and functions calling it) now clear operations stored in sequence.

eStorage changes

Added a concept "device only" tensors that can be checked with isDeviceOnlyTensor(). At the moment eStorage is only device only tensor.
This is used to determine if we need to do raw data mappings and unmappings. Also in postEval of OpTensorCopy made it such that it does not copy raw data if the tensor that we copy from is device only nor it does not copy raw data to a tensor that is device only.
This fixes undefined behaviour that we had with eStorage tensor. Now we can create device only tensors with manager by setting data to nullptr.

// Create device only tensor with 10 floats
auto tensor = mgr.tensor(nullptr, 10, sizeof(float), kp::Tensor::TensorDataTypes::eFloat, kp::Tensor::TensorTypes::eStorage);

I added one test about copying in and out from eStorage tensors.

eval changes

Motivation behind changes: There was lot of tests that were chaining evals together. For example

    mgr.sequence()
      ->eval<kp::OpTensorSyncDevice>({ tensorA })
      ->eval<kp::OpTensorCopy>({ tensorA, tensorB });

This is straight up broken because we never clear vk::CommandBuffer nor operations stored in the sequence. When second eval is called it runs OpTensorSyncDevice again because it never got cleared form the sequence. This can be checked from the logs.
Because I think chaining evals is quite intuitive api and I like it I wanted to fix this so what I propose is that evalAwait would always clear the operations stored in the sequence. So now you have to manually record operations again but I don't think this is a problem.
I added vk::CommandBufferUsageFlagBits::eOneTimeSubmit to command buffer create info such that we are sure vk::CommandBuffer gets cleared every time we start record to it.
With this change some tests got broken or they had bugs in them. I also fixed them.

BTW I did not figure out hot to generate doctrings.hpp automatically so I modified it manually.

Closing as superceeded by https://github.com/KomputeProject/kompute/pull/316


Issue with /docs/tutorials/kubernetes-basics/explore-interactive/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
Terminal inside the page won't load - renders as a blank gray image:

Page to Update:
https://kubernetes.io/docs/tutorials/kubernetes-basics/explore-interactive/

Chrome Version 57.0.2987.98 (64-bit)
MacBook Air (13-inch, Early 2015)

FWIW, this also wasn't working for me several minutes ago but is working again now.

Can’t repro either. @g-wattz can you please check if it works now, if so can you please close?

Looks like it works now, I'll close this out.


BUG: t411_api invalid get requests with terms

When searching using terms the GET request sent to the server is sending invalid characters when chaining those terms. The result is those terms are not being filtered by t411 search server
For example, in this pcap file you can see the request and the response
t411_api_flexget.zip
The real request should be:
/torrents/search/NCIS?cid=433&term%5B46%5D%5B%5D%27=949&limit=100&offset=0&term%5B45%5D%5B%5D%27=980&term%5B51%5D%5B%5D%27=1210&term%5B51%5D%5B%5D%27=1212&term%5B51%5D%5B%5D%27=1213&term%5B51%5D%5B%5D%27=1214
and it should give out 3 results only
{"query":"NCIS","offset":"0","limit":"100","total":"3","torrents":[{"id":"5505701","name":"NCIS.S13E12.FRENCH.HDTV.x264-FRiES","category":"433","rewritename":"ncis-s13e12-french-hdtv-x264-fries","seeders":"136","leechers":"1","comments":"1","isVerified":"1","added":"2016-05-05 23:02:43","size":"253477370","times_completed":"422","owner":"8039783","categoryname":"S\u00e9rie TV","categoryimage":"video-tv-series","username":"superbus49","privacy":"normal"},{"id":"5505743","name":"NCIS.S13E12.FRENCH.720p.HDTV.x264-FRiES","category":"433","rewritename":"ncis-s13e12-french-720p-hdtv-x264-fries","seeders":"169","leechers":"1","comments":"2","isVerified":"1","added":"2016-05-06 01:00:35","size":"923681846","times_completed":"459","owner":"8039783","categoryname":"S\u00e9rie TV","categoryimage":"video-tv-series","username":"superbus49","privacy":"normal"},{"id":"5505863","name":"NCIS.S13E12.FRENCH.HDTV.XviD-ZT","category":"433","rewritename":"ncis-s13e12-french-hdtv-xvid-zt","seeders":"219","leechers":"0","comments":"1","isVerified":"1","added":"2016-05-06 09:36:11","size":"326451286","times_completed":"668","owner":"99613071","categoryname":"S\u00e9rie TV","categoryimage":"video-tv-series","username":"Seuph","privacy":"normal"}]}
This is my config file
http://pastie.org/10809799

Thanks for your excellent analysis but I cannot reproduce your bug.
I think your bug appear during request serialization done by requests at https://github.com/Flexget/Flexget/blob/develop/flexget/plugins/api_t411.py#L254.
Key param strings seems serialized with adding a b prefix. That means your are using Python 3 and probably explain why I cannot reproduce (indeed, I use python 2).
Key param string is built  at https://github.com/Flexget/Flexget/blob/develop/flexget/plugins/api_t411.py#L314.
My python skills are insufficient to further.

I use python 2.7. I'm on debian.
I don't have much python skills
Is there something else I can give to help pinpoint the issues?

I've added a log that displays the content of the param object before it's handed out to requests at
https://github.com/Flexget/Flexget/blob/develop/flexget/plugins/api_t411.py#L254.
with log.debug('URL_PARAMS : %s' % (params))
2016-05-18 13:07 VERBOSE  discover      tv-t411         Searching for `Vikings S02E02` with plugin `t411` (12 of 166)
2016-05-18 13:07 DEBUG    t411_api      tv-t411         Category named "Série TV" resolved by id 433
2016-05-18 13:07 DEBUG    t411_api      tv-t411         Resolving terms : [u'Fran\xe7ais (VFF/TrueFrench)', u'Multi (Fran\xe7ais inclus)', u'Multi (Qu\xe9b\xe9cois inclus)', u'Qu\xe9b\xe9cois (VFQ/French)', u'Saison 02', u'Episode 02']
2016-05-18 13:07 DEBUG    t411_api      tv-t411         None312312 API token. Authenticating with "kevmegforest" account...
2016-05-18 13:07 DEBUG    utils.requests tv-t411         Fetching http://api.t411.ch/auth
2016-05-18 13:07 DEBUG    t411_api      tv-t411         URL_PARAMS_BEFORE: {u'cid': 433, 'term[46][]': [938], u'limit': 100, u'offset': 0, 'term[45][]': [969], 'term[51][]': [1210, 1212, 1213, 1214]}
2016-05-18 13:07 DEBUG    t411_api      tv-t411         URL_PARAMS : {u'cid': 433, 'term[46][]': [938], u'limit': 100, u'offset': 0, 'term[45][]': [969], 'term[51][]': [1210, 1212, 1213, 1214]}
2016-05-18 13:07 DEBUG    utils.requests tv-t411         Fetching http://api.t411.ch/torrents/search/Vikings
2016-05-18 13:07 DEBUG    t411_api      tv-t411         Search produces 100 results including 0 'on pending' (the latter will not produces entries)
2016-05-18 13:07 DEBUG    discover      tv-t411         Discovered 100 entries from t411

The only thing that looks different is the terms  are not unicode. Could it have an impact with the requests plugin?

The solution is to make everything unicode. I think the requests function, when seeing the first parameter as unicode (u'cid') expects to see everything else as unicode or vice-versa if it's ascii string.
this is why the terms which are standard string are decoded with the b prefix by the request function
If this line is changed:
https://github.com/Flexget/Flexget/blob/develop/flexget/plugins/api_t411.py#L314
to term_type_key_param = unicode('term[' + str(term_type_id) + '][]'),
it fixes the problem.
I don't think that's the best solution since this code also need to work with python3.
Since I'm a noob in python2 or 3, I really don't know what's the best way to make this line compatible with those 2 incompatible python versions.

I can reproduce your bug since I updated flexget.
@kevmegforest Good job :+1:
I beg you to commit your patch :smile:

@lildadou Do you know a way to make this command compatible with python3 and 2, since flexget seems to be making a big transition to it?

@kevmegforest That was the right pytonic way by @gazpachoking Thanks to him


Why count query will transformed into count(distinct id)

Hi there,
I got a question about COUNT query. Why CountNodes will transformed into SQL count(distinct id). Why's the design?

It should only do that if your entity has been defined as Unique in its schema.
Have a look at dialect/sql/sqlgraph/graph.go:598
You should see this piece of code, that is driver agnostic :
if q.Unique {
		selector.SetDistinct(false)
		selector.Count(sql.Distinct(selector.C(q.Node.ID.Column)))
	}

thx.


Add support for both XML and JSON response

Done. System will now check for XML and JSON files in the api path and return which ever is found. JSON file will get the priority, if both are present.


Unknown props 'mobile' warning

This warning when using react-numeric-input 2.0.4 in react 15.2.1 does not seem to break anything but it is annoying:
Warning: Unknown props `mobile`, `onSelectStart` on <input> tag. Remove these props from the element. For details, see https://fb.me/react-unknown-prop
    in input (created by NumericInput)
    in span (created by NumericInput)
    in NumericInput (created by NumberControl)
    ...

Already reported here: https://github.com/vlad-ignatov/react-numeric-input/issues/16

This is fixed v2.0.5 which adds compatibility with React 15.2+


praw 3 requests dependency version

Issue Description
In PRAW 3, the setup.py file pins the requests version at 'requests >=2.3.0'. However, actually trying to use requests v2.3.0 will raise an exception. The problem is that the merge_environment_settings method wasn't added until v2.4.0.
  File "/usr/local/lib/python3.4/dist-packages/praw/__init__.py", line 1522, in refresh_access_information
    refresh_token=refresh_token or self.refresh_token)
  File "<decorator-gen-11>", line 2, in refresh_access_information
  File "/usr/local/lib/python3.4/dist-packages/praw/decorators.py", line 294, in require_oauth
    return function(*args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/praw/__init__.py", line 745, in refresh_access_information
    retval = self._handle_oauth_request(data)
  File "/usr/local/lib/python3.4/dist-packages/praw/__init__.py", line 657, in _handle_oauth_request
    response = self._request(url, auth=auth, data=data, raw_response=True)
  File "/usr/local/lib/python3.4/dist-packages/praw/__init__.py", line 451, in _request
    response = handle_redirect()
  File "/usr/local/lib/python3.4/dist-packages/praw/__init__.py", line 432, in handle_redirect
    verify=self.http.validate_certs, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/praw/handlers.py", line 138, in wrapped
    return function(cls, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/praw/handlers.py", line 56, in wrapped
    return function(cls, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/praw/handlers.py", line 101, in request
    settings = self.http.merge_environment_settings(
AttributeError: 'Session' object has no attribute 'merge_environment_settings'

This bug was initially discovered here https://github.com/michael-lazar/rtv/issues/325. I've tested on requests v2.4.0 and verified that it solved the problem. I can fix it on my end by changing the package requirements, but if you're still supporting bug fixes for PRAW 3 I think it's worth updating here too.
System Information
PRAW Version: 3.6.1
Python Version: 3.4.0
Operating System: Ubuntu 14.04

Thanks for reporting and great work finding the bug. If you want to make a PR to the PRAW3 branch I will merge it in, however, I am not likely to make a new PRAW3 release unless a large number of issues accumulate. I'm going to close the issue since it's not a problem with the current version.


Update openapi-circe-yaml, openapi-model to 0.9.0

About this PR
📦 Updates

com.softwaremill.sttp.apispec:openapi-circe-yaml
com.softwaremill.sttp.apispec:openapi-model

from 0.8.0 to 0.9.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.apispec" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.softwaremill.sttp.apispec" }
}]

labels: library-update, early-semver-major, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #418.


Without internet access the server runs with very high CPU

I happen to kill my WIFI during lunch, so was able to unintentionally reproduce this twice.
1.) Start up the server
2.) Kill your Wifi.
Observed:
In a few minutes you'll likely hear the fan kick into high gear on your machine and you'll notice the server running at very high CPU. I killed mine at around 400%.
Expected
The server doesn't "overheat" because it doesn't have internet. On some platforms, maybe this is an edge case, but I imagine mobile might regularly go through packages with limited or no connectivity.

Here's the later part of my server log:
11:28:26.066 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:28:26.078 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:28:26.420 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:28:28.060 [ERROR] [pool/runLoop] error starting https://zec.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:28:28.068 [ERROR] [pool/runLoop] error starting https://bch.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:28:28.079 [ERROR] [pool/runLoop] error starting https://ltc.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:28:28.415 [ERROR] [pool/runLoop] error starting https://btc.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:29:28.062 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup zec.blockbook.api.openbazaar.org: no such host
11:29:28.068 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:29:28.080 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:29:28.416 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:29:30.064 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup zec.blockbook.api.openbazaar.org: no such host
11:29:30.076 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:29:30.088 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:29:30.426 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:29:32.071 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup zec.blockbook.api.openbazaar.org: no such host
11:29:32.075 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:29:32.084 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:29:32.426 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:29:34.065 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup zec.blockbook.api.openbazaar.org: no such host
11:29:34.075 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:29:34.081 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:29:34.425 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:29:36.066 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup zec.blockbook.api.openbazaar.org: no such host
11:29:36.075 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:29:36.086 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:29:36.426 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:29:38.069 [ERROR] [pool/runLoop] error starting https://zec.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:29:38.074 [ERROR] [pool/runLoop] error starting https://bch.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:29:38.085 [ERROR] [pool/runLoop] error starting https://ltc.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:29:38.424 [ERROR] [pool/runLoop] error starting https://btc.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:30:38.070 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup zec.blockbook.api.openbazaar.org: no such host
11:30:38.074 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:30:38.085 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:30:38.425 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:30:40.074 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup zec.blockbook.api.openbazaar.org: no such host
11:30:40.082 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:30:40.093 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:30:40.430 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:30:42.080 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup zec.blockbook.api.openbazaar.org: no such host
11:30:42.080 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:30:42.090 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:30:42.427 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:30:44.076 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup zec.blockbook.api.openbazaar.org: no such host
11:30:44.076 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:30:44.090 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:30:44.431 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:30:46.074 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup zec.blockbook.api.openbazaar.org: no such host
11:30:46.075 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup bch.blockbook.api.openbazaar.org: no such host
11:30:46.086 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup ltc.blockbook.api.openbazaar.org: no such host
11:30:46.425 [ERROR] [client/setupListeners] reconnect websocket: dial tcp: lookup btc.blockbook.api.openbazaar.org: no such host
11:30:48.072 [ERROR] [pool/runLoop] error starting https://zec.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:30:48.074 [ERROR] [pool/runLoop] error starting https://bch.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:30:48.085 [ERROR] [pool/runLoop] error starting https://ltc.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners
11:30:48.424 [ERROR] [pool/runLoop] error starting https://btc.blockbook.api.openbazaar.org/api: unable to connect websocket to setup listeners

@placer19 this is you


clean up ambiguous semantics of data_processing_id

When we introduced data_processing_id, it was meant to allow for multiple data processings to exist side-by-side, and we included statements like this:

It is expected that typical Repertoires might only have a single DataProcessing, in which case repertoire_id and data_processing_id will be semantically equivalent and only the former should be used.

However, this creates ambiguity for a data repository. Does this mean that they should only store data_processing_id if there are two or more? Can it be stored as a null value?
Furthermore, in the airr-schema spec we have this:

If this field is empty than the primary data processing object is assumed.

This causes further difficulty for data repositories as they have to determine if an arbitrary query is filtering data_processing_id or not, and if not then has to determine which data processing is the primary one, then presumably alter the query or filter to enforce it, that is exclude results from the non-primary data processings. This is impractical or cumbersome for arbitrary queries on the /rearrangement endpoint.
My suggestion is that we clean up this ambiguity, in particular.

State that data repositories must always have a data_processing_id on rearrangement records, i.e. null is not allowed. This is enforced on the output from the API. Data repositories can store the data in their database however they want.

Eliminate the requirements around the assumption of the primary data processing. Thus data_processing_id will be treated just like any other field in an ADC query, through a literal interpretation of the query filter.

This will affect clients of the ADC API, as they will be need to provide data_processing_id in their queries if they really want only the data within a specific data processing. That doesn't seem a big burden on them as they are probably doing that anyways, but the upside is that data repositories are relieved of the burden of treating data_processing_id in a special way.
This shouldn't affect tools directly. As before, we don't require tools to acknowledge repertoire_id, data_processing_id and so forth, it's optional. They can if they want, but if not then users just need to make sure that data is appropriately separated.

Philosophical question:  Can two data sets that have been processed differently be the same Repertoire?
If Repertoire had a concrete biological meaning then I would say yes, but it doesn't...

@javh I mean, that came up on the call Monday --philosophical or not, I think we're going to have to address it.

@javh @scharch Yeah, I'm fine with going into that but let's do that in a separate ticket, I'd like to keep this issue focused on those specific statements that cause problems for data repositories. Open up a ticket on reimagining Repertoire, with hindsight and the new Clone and Cell objects, I certainly have my own ideas about that.
But the short answer is, depends what you mean by a Repertoire. It's the case right now that you need both repertoire_id and data_processing_id to collect the set of rearrangements to be analyzed together. If you only used repertoire_id then you would (potentially) be mixing rearrangements together that should not be mixed. Does that cause confusion? Probably, yes.

This issue would be made moot by PR #453 as the whole point of primary_annotation is to handle multiple data processing, and wouldn't be needed anymore if there was only a single one.

See my comment on the pull request... This breaks compatibility on a number of things and I am not sure it makes sense to change DataProcessing until we sort out #441 as that is almost certainly going to entail another change. Does an intermediate change make sense we know more changes are going to emerge soon. It seems that we can probably live with the current situation in the short term until we sort out #441?

@schristley same with this? v2.1?

Cleared milestone as its unknown when/if data processing will be re-designed.


feat: Generate tests from selected code

What is this?
Currently we have a feature to generate refactoring code suggestions. This would be a similar feature to generate a test based on a text selection.
Suggested Approach
OpenAI already provides this via Copilot, but I am not sure the model.

We'll add similar stuff to this page in the future too, so this one should be built with extensibility in mind.
(Just replacing the prompt, changes the action that we perform)

I've been looking for ways to get the dependencies of a repository. That way the test suggestions can be tailored according to the test suites being used by the project. Using the REST API to get the dependency graph requires PAT. I'll try out scraping to see if we can avoid the PAT auth.

I've been looking for ways to get the dependencies of a repository.

We just need to give a  hint to the developer, that's what copilot nightly does right now without knowing the test suite:

Even copilot chat:

There is a lot of background needed to write a test (also need to know how other tests are written), so if you want, we can leave the specific details out, since it won't provide any additional value anyway.

There is a lot of background needed to write a test (also need to know how other tests are written), so if you want, we can leave the specific details out, since it won't provide any additional value anyway.

This is a good goal to achieve. We won't provide a perfect test but a test. Background on how test are written is not need to ship this feature. An engineer should be to take the output and be inspired to write the test that works. The expectations should not be copying the test wholesale.
In the future we can have the extension look into the test folder or look for .test files but that level of complexities is not needed.
My thought is that if a function is highlighted a unit test can be made based all on assumptions.


valid_ip removed $data which was causing exception

That data parameter is odd and appears that it should be removed, as you noted. Please also remove its reference from the inline docs (@param).
What is the inet_ntop? You should explain additions and ideally add them in a separate PR.

@MGatner inet_ntop and pton are converting the IPv4 or v6 string to its binary/human readable form.
I am just thinking about extending Entity as well to store it bydefault like this... and this extending will be preety easy:
in castAs:
``
        case 'ip':
            $value = inet_ntop($value);
            break;

in __set
        if($castTo === 'ip')
        {
            $value = inet_pton($value);
        }

``
in fact I am using this in my extended entity class to do not use setters every time when I need to stroe IP in that efficient way.

Since the tests are all passing this doesn't seem to have affected the outcomes, while potentially improving the service. It looks good to me, will merge - thanks!
Re: Entity auto-cast, I like the idea. Since casts are only used when a developer requests them I think it is a fine addition. If you want to add it please start a new PR and be sure to update the User Guide and have tests with the new cast.


Slow block time on PoS

[ Subject of the issue ]
Description
Every 5 or 6 blocks (normally 2 seconds each) , we get 2 blocks that take 15 seconds. It only seems to do this on PoS. PoA was 2 seconds for every block. 10 bootnodes. 50 staked validators.
Epoch at 100. 107,000 blocks made. BLS is on.
Your environment

OS and version. Ubuntu 20.04 server LTS
The version of the Polygon Edge.
(Confirm the version of your Polygon edge client by running the following command: polygon-edge version --grpc-address GRPC_ADDRESS)
The branch that causes this issue.
Locally or Cloud hosted (which provider).
Please confirm if the validators are running under containerized environment (K8s, Docker, etc.).

Steps to reproduce

Tell us how to reproduce this issue.
Where the issue is, if you know.
Which commands triggered the issue, if any.
Provide us with the content of your genesis file.
Provide us with commands that you used to start your validators.
Provide us with the peer list of each of your validators by running the following command: polygon-edge peers list --grpc-address GRPC_ADDRESS.
Is the chain producing blocks and serving customers atm?yes

Expected behavior

Tell us what should happen. Block time of 2 seconds.
Tell us what happened instead. Average time of 7 seconds.

Logs
Provide us with debug logs from all of your validators by setting logging to debug output with: server --log-level debug
Proposed solution
If you have an idea on how to fix this issue, please write it down here, so we can begin discussing it

Hi @Nexsuperne101 ,
Please provide the full logs so we can look into this. Thank you.

How do I do that please? I've only been using this a couple of weeks now. Before that, I was an Electrical Engineer for 30 years. This is definitely a " learning curve"!

By the full logs I mean the logs of a node. The logs usually look something like:
2022-11-10T17:13:47.493+0200 [INFO]  polygon.server.ibft: block committed: number=248 hash=0x3048a27042b4a225a667a799eb0f1f19101e134c9d0b1897cf4dfa243e5a7ad3 validation_type=bls validators=5 committed=4
2022-11-10T17:13:47.493+0200 [INFO]  polygon.server.ibft.consensus: sequence done: height=248
2022-11-10T17:13:47.493+0200 [INFO]  polygon.server.ibft.consensus: sequence started: height=249
2022-11-10T17:13:47.493+0200 [INFO]  polygon.server.ibft.consensus: round started: round=0
2022-11-10T17:13:47.517+0200 [INFO]  polygon.blockchain: block already inserted: block=248 source=syncer
2022-11-10T17:13:49.591+0200 [INFO]  polygon.blockchain: new block: number=249 txs=0 hash=0x7fed5b2ce0a0ea2317db091146ee3383fdeb2e53db5abb2904744e2f7419a669 parent=0x3048a27042b4a225a667a799eb0f1f19101e134c9d0b1897cf4dfa243e5a7ad3 generation_time_in_seconds=2

@Nexsuperne101 Yes, that would be helpful to figure this out.

@Nexsuperne101 For the other issue you have currenlty open too.

I reinstalled the 10 nodes. The problem has gone now.


Ghost Rendering For Markdown cells

Follow on to #167
It looks like currently the cell decorations that we apply to render cells as ghost cells doesn't work for markdown cells. I think this maybe because when markdown cells are inserted; they are inserted in rendered mode not edit mode. If we were to put them in edit mode maybe the cell decorations would apply.
I think we are hitting some limitations with VSCode Notebook API

MD cells can't be inserted in edit mode
You can put cells into mode but I think you may have to put the focus on them; Claude says

vscode.commands.executeCommand('notebook.cell.edit');

I think this puts the focus on the new cell which we may not want to do

Do we want the AI to generate markdown cells?

If the current focus is on a markdown cell then it makes sense that the AI should be strongly biased towards generating a code cell (at least as its first completion)

However if the current cell is a code cell; possibly one that's just been executed then we might want to generate a markdown cell

For example, we'd like the AI to reason about the output

For Markdown we could potentially use HTML & CSS to do the styling

The vscode repository pointed me at StackOverflow so I posted a question there
https://stackoverflow.com/questions/78823333/how-to-programmatically-insert-vscode-notebook-cells-in-edit-mode
I found a function updataEditState but I don't think that is part of the public API.

I wonder if https://code.visualstudio.com/api/extension-guides/markdown-extension could be leveraged to replicate the grayed-out ghost presentation. I haven't played around with it much yet, and it's unclear how much contextual awareness the individual cells have. However, it wouldn't be the first time working around the APIs inside of the Extension Platform.

Another workaround would be to turn the markdown into a comment included in the code cell.
This isn't an ideal solution but would provide some visual affordance while not forcing us to completely drop the AIs "thinking"

The frontend change is in https://github.com/stateful/vscode-runme/releases/tag/3.9.0


GUS-864 update readme to include example usage of double references

Description

add missing explicit documentation / example usages for two references in a comparison operator

Test Coverage

[x] unit

References

Part of GUS-864

Is it worth to cut a version without functional changes? I'd say just make the commit, but don't cut a version


programs.konsole.defaultProfile not working as expected

  programs.konsole = {
    enable = true;

    defaultProfile = "default";
    profiles.default = {
      name = "default";
      font = {
        name = "FiraCode Nerd Font Mono";
      };
    };
  };

Expectations: Set default profile to "default".
Reality: I have to set programs.konsole.defaultProfile = "default.profile" instead to make it work.

You're right I didn't even think about that. I agree we shouldn't have to add .profile to defaultProfile manually. I'll fix this soon :)


Page formating

Section titles for "Result",  "Remarks" and "Examples" have bad formatting.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 16bb2377-bca2-2c5f-35c2-96e35c3050cc
Version Independent ID: 68a7b48c-7c0e-a061-59c5-324e6aabaae9
Content: WdfTimerStart function
Content Source: wdk-ddi-src/content/wdftimer/nf-wdftimer-wdftimerstart.md
Service: unspecified
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

Thanks, @reuvenab. I believe the page is now rendering in all its beauty. Let us know if you see anything else like this. You can always edit the content and submit a pull request, too.
Cheers!
Eliot


SystemJS build with rollup

From @AnthonyBr on February 15, 2017 17:44
Hello,
When I generate my angular2 project:
node_modules\.bin\rollup -c rollup-config.js
I have the error in the title:
?   'ContextMenuComponent' is not exported by node_modules\angular2-contextmenu\
angular2-contextmenu.js

To bundle my project, I use rollup, as described here:
https://angular.io/docs/ts/latest/cookbook/aot-compiler.html
As said in the documentation:
Rollup can only Tree Shake ES2015 modules which have import and export statements.
And ContextMenuComponent has no export statement.
The project works well when I start the project with systemjs.
Thank you for help,
A. Breneliere
Copied from original issue: isaacplmann/angular2-contextmenu#48

You're right.  I'm not really sure the best way to export both commonjs (for webpack et al.) and es2015 (for rollup).  If you could point me towards another library that does this well or a how-to article, that will help me resolve this.

On the subject, is there no longer a commonjs export for this module? I'm getting an unexpected token export error now that I'm trying to upgrade from angular2-contextmenu because ngx-contextmenu seems to point to an es6 module by default.

Yes, I think that broke in the migration.  I need to figure out how to fix that.


add metrics

What this PR does:
Add metric for prometheus.
Which issue(s) this PR fixes:

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

not for 0.1.0


fix(configtree):  fix handling org-level tree revisions in statefile

The Revision context manager stores the latest revision in the Statefile to
avoid typing revision explicitly every time. However the recent change to make
the Project the default context caused some mishandling for Org-level trees.
This caused all the keys to be stored with ProjectGUID as the top-level.
The fix is to not provide the =project_guid= parameter to Statefile helper
functions in case of Org-level tree.

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


Very slow dev server startup with typescript.typeCheck: true when debugger attached in VSCode

Environment

Operating System: Windows_NT
Node Version:     v20.18.0
Nuxt Version:     3.12.4
CLI Version:      3.12.0
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      routeRules, hooks, runtimeConfig, modules, delayHydration, sitemap, googleFonts, app, experimental, css, typescript, multiCache, nitro, vite, compatibilityDate
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
When using
Describe the bug
When using
export default defineNuxtConfig({
  typescript: { strict: true, typeCheck: true }
})

on a Nuxt project with a large amount of files (~180 components and ~50 pages) the dev server launched by Visual Studio Code takes about 15 minutes to start.
Disabling typeCheck or launching "npm run dev" outside VSCode takes 30 seconds.
My launch.json file is as following:
{
  "version": "0.2.0",
  "configurations": [
    {
      "type": "node-terminal",
      "request": "launch",
      "name": "server: nuxt",
      "command": "npm run dev"
    },
    {
      "name": "client: chrome",
      "type": "chrome",
      "request": "launch",
      "url": "http://<IP_ADDRESS>:3000",
      "webRoot": "${workspaceFolder}"
    }
  ],
  "compounds": [
    {
      "name": "fullstack: nuxt",
      "configurations": ["server: nuxt", "client: chrome" ]
    }
  ]
}

Additional context
No response
Logs

Hi
On Wed, 13 Nov 2024, 6:27 pm niddu85, @.***> wrote:

Environment

Operating System: Windows_NT
Node Version: v20.18.0
Nuxt Version: 3.12.4
CLI Version: 3.12.0
Nitro Version: 2.9.7
Package Manager: @.***
Builder: -
User Config: routeRules, hooks, runtimeConfig, modules,
delayHydration, sitemap, googleFonts, app, experimental, css, typescript,
multiCache, nitro, vite, compatibilityDate
Runtime Modules: @@., @@.,
@., @@.,
@., @@.
Build Modules: -

Reproduction
When using
Describe the bug
When using
export default defineNuxtConfig({
typescript: { strict: true, typeCheck: true }
})
on a Nuxt project with a large amount of files (~180 components and ~50
pages) the dev server launched by Visual Studio Code takes about 15 minutes
to start.
Disabling typeCheck or launching "npm run dev" outside VSCode takes 30
seconds.
My launch.json file is as following:
{
"version": "0.2.0",
"configurations": [
{
"type": "node-terminal",
"request": "launch",
"name": "server: nuxt",
"command": "npm run dev"
},
{
"name": "client: chrome",
"type": "chrome",
"request": "launch",
"url": "http://<IP_ADDRESS>:3000",
"webRoot": "${workspaceFolder}"
}
],
"compounds": [
{
"name": "fullstack: nuxt",
"configurations": ["server: nuxt", "client: chrome" ]
}
]
}
Additional context
No response
Logs
—
Reply to this email directly, view it on GitHub
https://github.com/nuxt/nuxt/issues/29903, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BF53BVBGQTAJVNIN24W7EH32AMSQ3AVCNFSM6AAAAABRWEHC7WVHI2DSMVQWIX3LMV43ASLTON2WKOZSGY2TIOJXGE3TEMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

For anyone looking for a solution, I solved increasing the maximum memory for NodeJs in my launch command in package.json:

"scripts": {
    "build": "nuxt build",
    "dev": "cross-env NODE_OPTIONS=--max-old-space-size=4096 nuxt dev",
    "generate": "nuxt generate",
    "preview": "nuxt preview",
    "analyze": "nuxt build --analyze",
  },

I added cross-env NODE_OPTIONS=--max-old-space-size=4096 and now is lightning fast.
Since i'm on window i had to use cross-env so I had to install it with
npm i -D cross-env


Add test for prepare_url

This only test if URLs are combined correctly.

Fantastic work dude, thanks @fengkaijia 💯


Integrate into firefox search bar?

Most search engines offer a way to be added to the search bar in Firefox. For example, https://search.nixos.org/packages does. It would be great if noogle had that as well.

This seems to do it https://github.com/hsjobeki/noogle/pull/17 ?

Can confirm that it works! Thanks a lot!


When navigating by UPDATE_LOCATION guards are ignored

This is a...

[ ] feature request
[x] bug report
[x] usage question

What toolchain are you using for transpilation/bundling?

[x] @angular/cli
[ ] Custom @ngTools/webpack
[ ] Raw ngc
[ ] SystemJS
[ ] Rollup
[ ] Other

Environment
NodeJS Version: 10.16.0
Typescript Version: 3.2.4
Angular Version: 7.2.1
@angular-redux/store version: 10.0.0
@angular/cli version: (if applicable): 7.2.2
OS: Win 10 x64
Link to repo showing the issus
example-app
Expected Behaviour:
When I navigate by dispatching the UPDATE_LOCATION - Action the router-State should only change if the navigation succeeds. If the navigation is canceled, e.g. by a guard, the current router-State should not update to the intented target location.
Actual Behaviour:
When I navigate by dispatching the UPDATE_LOCATION - Action the router-State is updated to the target location, whether the target is reached or not. This leads to an inconsistent state.
Stack Trace/Error Message:
Additional Notes:
The behaviour can be easily reproduced in the example-app:

Create an Action that navigates the user to /lions:

  @dispatch()
  goToLions() {
    return {
      type: UPDATE_LOCATION,
      payload: '/lions'
    }
  }

Protect the lions-Route with a guard:

{ path: 'lions', component: LionPageComponent, canActivate: [LionsGuard] },

...

export class LionsGuard implements CanActivate {

  canActivate(): any {

    console.warn('hit guard');
    return false;
  }

}

Running goToLions() will update the router-State to /lions, even if the guard keeps me on /elephants

I navigate a lot by returning the UPDATE_LOCATION Action in Epics. Any ideas how to fix or workaround this behaviour?

any updates?


[BUG] "Exit" Button does not work

When trying to exit the game via the corresponding button, nothing will happen.

Hello,
you have to hold the exit button for less than a second to exit the game, the reason for that is to avoid closing the game on accident when playing on mobile.


Show more detailed date of transaction on mouse over

Display the whole date on the mouse over with the HTML title attribute

Issue https://github.com/OriginProtocol/origin-dollar/issues/833

LGTM


Add natural pauses during speech.

It would be nice to have some small pauses during speech. Currently, the audio is continuously played, the best way to simulate natural pauses would be to have either a constant duration or a randomized duration for a pause.
The pauses can be triggered on whitespaces.
The current implementation has the following mechanism set up, but the speech sounds monotonic. Investigate this.

Resolved via #9.


[Hardware] Issue with sleep, logout + clamshell sleep = shutdown

I test three ways, as follows

directly cover, will not sleep display and macos, consuming the power
lock and cover, will not sleep display and macos, consuming the power
log out and cover, after a short time, it's okay, but even a bit longer time, it will be shut down incidentally
In conclusion, at current this script will disable the normal function of sleeping macbook.

This is interesting. I do have a B4 installation to test with - just not enough free time.

I suspect these issues will be resolved if you use the latest release.

Issue not affecting macOS Sierra, and is only applicable to High Sierra.


build shared library using TensorflowCC::Static

Hello, I'm using the static version of TensorflowCC, it works great when a build a binary, now I'm trying to build a shared library to dlopen() from another binary
the problem: I can not link it
ADD_LIBRARY(iadlmodel SHARED
iadlmodel.cc)
target_link_libraries(iadlmodel TensorflowCC::Static)
ERROR:
/usr/bin/ld: CMakeFiles/iadlmodel.dir/iadlmodel.cc.o: relocation R_X86_64_32 against `_ZTVN4iadl12TrainedModel5ModelE' can not be used when making a shared object; recompile with -fPIC
CMakeFiles/iadlmodel.dir/iadlmodel.cc.o: error adding symbols: Bad value
collect2: error: ld returned 1 exit status
CMakeFiles/iadlmodel.dir/build.make:95: recipe for target 'libiadlmodel.so' failed
is this posible ?

Hi @payas, although it is strange, it seems that your library is not build with position independent code. Try to add:
set_property(TARGET iadlmodel PROPERTY POSITION_INDEPENDENT_CODE ON)

to your CMakeLists.txt.

oh god.. I did put the POSITION_INDEPENDENT_CODE, I just forgot the "ON"
it's perfect now
-rwxrwxr-x 1 payas payas 78967360 Sep 26 12:32 libiadlmodel.so*
thank you very much!


Dropdown navigation menu, when in mobile client

Hi
I followed the tutorial for django-SHOP for installation and its working just fine.
As I wanted to try out the navigation menu, when its collapsed, such as in a mobile client, the tapping/clicking event does not expand the navigation menu. The CSS class collapse always states display:none;
I checked all the dependencies, such as angular.min.js, angular-animate.min.js and ui-bootstrap-tpls.js and it is all there and loaded...
The event triggers, but then the CSS does not change.
A screenshot of the libraries, rendered for the site:

I am quite confused, why it does not work, as in all the examples I found, it looks exactly the same.

Confirmed.
I'm currently checking why this happens. Presumably an AngularJS directive hides the scope.

@markusmo Please retry with the latest HEAD from https://github.com/jrief/djangocms-bootstrap3
If that works for you, I'll bundle a new version on PyPI.

Works like a charm :-)
Thanks a lot for the help.

Just released djangocms-bootstrap3 version 0.4


support multiline values in backticks

#12
Pay attention: I don't know expectation. But backticks values are parseable now

Thank you @vanodevium ! I appreciate all your help!


V1.2 adds Team Packages

Include class Team, TeamName and TeamList
Lineup related codes not written yet

Codecov Report

Merging #39 (a4aa9ce) into master (bf87981) will decrease coverage by 1.71%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #39      +/-   ##
============================================
- Coverage     72.15%   70.44%   -1.72%
  Complexity      399      399
============================================
  Files            70       73       +3
  Lines          1232     1262      +30
  Branches        125      127       +2
============================================
  Hits            889      889
- Misses          311      341      +30
  Partials         32       32

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/team/Team.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/team/TeamList.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/team/TeamName.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Use new Parse Server logo.

Static asset for this logo was uploaded via GitHub Issues.

Sure. PTAL.


Svelte Flow: getIntersectingNodes always returns

Describe the Bug
In Svelte Flow, SvelteFlowProvider's getIntersectingNodes function does not work with Rect passed as the first parameter.
What I think is going on here is  that https://github.com/xyflow/xyflow/blob/main/packages/svelte/src/lib/hooks/useSvelteFlow.ts#L139 should use && instead of ||. The line above calls getNodeRect when a node is passed, so both the nodeRect and node variables are populated, but when a Rect is passed, only the nodeRect variable is populated, so line 139 fails and returns [].
Your Example Website or App
https://stackblitz.com/edit/cozxhv-sv6ngc?file=src%2Fexample%2FInner.svelte
Steps to Reproduce the Bug or Issue
Call getIntersectingNodes with a Rect that does actually overlap one or more Nodes.
Expected behavior
The function should return the intersected Node(s) in an array.
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

If my logic makes sense to you, I'm happy to fix this, add some tests etc.

 I can fix this promptly and it will be in the new release - if you are eager to contribute please tell me.
We are currently rethinking the way we do testing, so a test will not be necessary.

released in 0.0.29 https://svelteflow.dev/whats-new/2023-12-13


Update to<EMAIL_ADDRESS>

The following libraries got updated:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Updating cargo.toml seems to not break the build but I leave the PR to someone with test samples.

It doesn't break the build but I should go in and remove all the unsafe code that does copying of fixed-point values and structs containing them (unless you want to have some fun with that)

I'll leave it for you since you are more familiar with the codebase. :slightly_smiling_face:

Removing the unsafe code is sadly not fully possible yet because of https://github.com/open-flash/rust-swf-fixed/issues/5.
But also I can't fully remove the workaround for missing Copy impls, because swf-tree is missing Copy on Matrix and Copy + Default on ColorTransformWithAlpha (probably all types that can implement those traits in swf-tree, should).

I did a round of updates, here are the latest versions:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Unsafe code is gone in 35fe6ca, I just need to update buttons to use swf_tree::tag::DefineButton.


Feature request: pre-commit hook for formatter

It would be great if there were a pre-commit hook for the TOML formatter. My current go-to formatter is pretty-format-toml, but this suffers from a fairly serious issue that comments are sometimes deleted. If this were implemented, I think it could easily become the best formatter with a pre-commit hook.

Thanks for the suggestion! I guess this needs both the creation of some files in this repo and a PR for the pre-commit repository. This project is currently passively maintained, so only bugs are fixed. New functionalities need a PR contribution. Feel free to open one if you get the chance.

@maresb you can use mine if want: https://github.com/nikaro/taplo-pre-commit
Beware that the Docker hooks don't work on arm64 machines, until the next release of Taplo, cf. this issue. The native Rust ones are fine, but the build will take a lot of time the first time you run it (after that it should be in your cache so it will be ok).
Feel free to reuse it to open a PR here if you want.

Thanks @nikaro, I just now got around to trying this out. My experience is that it was going very slowly during the initial install. My suspicion is that pre-commit was downloading a rust toolchain in the background, and that's why it's so slow.
I was wondering what's the purpose of your Docker hooks, but if I understood the above correctly then it saves on all the compile time. Running Docker from pre-commit seems pretty crazy to me.  I wish it could just download binaries and use those. Unfortunately I'm not familiar with rust development, so I have no sense of the challenges of doing this.
Speaking of which, there are supposed to be binary releases but it seems like the links are broken and no binary artifacts are being generated by the GitHub workflows. Should I open a separate issue for that?

Thanks @nikaro, I just now got around to trying this out. My experience is that it was going very slowly during the initial install. My suspicion is that pre-commit was downloading a rust toolchain in the background, and that's why it's so slow.

You are right that's what's happening: downloading rust toolchain and compiling. And that's why the -docker hook is here, to serve as a workaround to avoid having to compile (but it indeed requires Docker...).

I wish it could just download binaries and use those. Unfortunately I'm not familiar with rust development, so I have no sense of the challenges of doing this.

There is PR for that in pre-commit bit it seems a bit stale lately... :-/

Speaking of which, there are supposed to be binary releases but it seems like the links are broken and no binary artifacts are being generated by the GitHub workflows. Should I open a separate issue for that?

Yep, looks like the release workflow is broken: https://github.com/tamasfe/taplo/actions/runs/7742605055/workflow#L268
And it seems to be my fault 😬 the env.RELEASE_VERSION variable is not available in the matrix.

As a stopgap I quickly threw together a hook that uses the binaries published to NPM: https://github.com/mdekstrand/taplo-pre-commit

Since https://github.com/tamasfe/taplo/pull/549 is merged, Taplo will be published as a PyPI package on https://pypi.org/project/taplo. One could add a .pre-commit-hook.yaml to this repository, but a lightweight "mirror" (think of ruff-pre-commit or black-pre-commit-mirror) would be preferable given that this repo is about 23MB. An example of the required files are at https://github.com/ComPWA/mirrors-taplo, but would be better to set up an official repo.

Incredible work @redeboer, I appreciate all the persistence it took to get this done!!! I have started to use https://github.com/ComPWA/mirrors-taplo as a pre-commit hook and it's working beautifully!

but would be better to set up an official repo

:+1:

@redeboer could you please just archive the project next time? Deleting it just broke our - and I guess many more - CI pipelines 😉

I'm pretty confused by the last two comments.
@flying-sheep, what's wrong with autoupdate? Seems to work for me (maybe @redeboer already fixed the issue)?
Also, @MaxWinterstein, what do you mean, are you referring to https://github.com/ComPWA/taplo-pre-commit/pull/18?
Maybe these should be issues in https://github.com/ComPWA/taplo-pre-commit?

@redeboer could you please just archive the project next time? Deleting it just broke our - and I guess many more - CI pipelines 😉

Sorry about that. It was a fork that was meant for testing https://github.com/tamasfe/taplo/pull/549#issue-2123053258. It's back online now and I'll archive it.

Maybe these should be issues in https://github.com/ComPWA/taplo-pre-commit?

Yeah probably better to post there. Once/if there is an official taplo/toml-formatter organisation (https://github.com/tamasfe/taplo/issues/403#issuecomment-1913342635) that can host this hook, I'm happy to transfer that repo there.

@flying-sheep, what's wrong with autoupdate?

I was responding to redeboer’s statement “[it] would be better to set up an official repo [for the pre-commit hook mirror]”
Once that exists, ComPWA/taplo-pre-commit is duplicate.


Refactor unused ctor args in presto-main

#4188
This PR is a preparation before enable UnusedVariable error prone check.

CI failed -- https://github.com/prestosql/presto/issues/5758

Merged, thanks!


CASSANDRA-13483: fixed test failure in snapshot_test.TestSnapshot.test_snapshot_and_restore_dropping_a_column

Jira: https://issues.apache.org/jira/browse/CASSANDRA-13483

+1


Update to match FluentKit's declared version minimums

This should help fix building in Xcode.

Codecov Report

Merging #239 (74785bd) into main (2579aa9) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #239   +/-   ##
=======================================
  Coverage   30.34%   30.34%
=======================================
  Files          10       10
  Lines         748      748
=======================================
  Hits          227      227
  Misses        521      521

These changes are now available in 2.9.1


Fixed UndoHelper

When items get deleted to fast, they were not removed from the recyclerView, because notifyCommit() was also called on the new inserted history (once to often).
Unwanted sequence:
User calls remove() -> creates History 1 -> call show() Snackbar 1 -> onShown() executes doChange() -> user calls remove() before Snackbar 1 dismissed -> notifyCommit() called for History 1 -> creates new History 2 -> call show() Snackbar 2 -> Snackbar 1 dismissed by consecutive event -> notifyCommit() on History 2 -> Snackbar 2 onShown() -> doChange() is called on null History -> History 2 not removed from recycler view

Thanks for the PR. And sorry for the super late response (I was traveling for 3 months, and not really available)
Please note that I had to minimal modify the PR as the deselcting is not expected behaviour.


FastAPI - Pydantic - ValueError Raises Internal Server Error

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from fastapi import FastAPI
from pydantic import BaseModel, validator
from fastapi import Depends, HTTPException

app = FastAPI()

class RankInput(BaseModel):

    rank: int

    @validator('rank')
    def check_if_value_in_range(cls, v):
        """
        check if input rank is within range
        """
        if not 0 < v < 1000001:

            raise ValueError("Rank Value Must be within range (0,1000000)")
            #raise HTTPException(status_code=400, detail="Value Must be within range (0,1000000)")
        return v

def get_info_by_rank(rank):
    return rank

@app.get('/rank/{rank}')
async def get_rank(value: RankInput = Depends()):
    result = get_info_by_rank(value.rank)
    return result

Description

This Throws an Internal Server Error when Input is out of Range for a GET Request

Can we customize the Model to Raise a ValueError with a Custom Status Code
Example - raise ValueError(status_code=400, detail="Value Must be within range (0,1000000)")

I know we can do this by raising an HTTPException like
raise HTTPException(status_code=400, detail="Value Must be within range (0,1000000)")

Log
INFO:     <IP_ADDRESS>:59427 - "GET /info/?rank=-1 HTTP/1.1" 500 Internal Server Error
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/uvicorn/protocols/http/h11_impl.py", line 396, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/uvicorn/middleware/proxy_headers.py", line 45, in __call__
    return await self.app(scope, receive, send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/fastapi/applications.py", line 199, in __call__
    await super().__call__(scope, receive, send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/applications.py", line 111, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/middleware/errors.py", line 181, in __call__
    raise exc from None
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/middleware/errors.py", line 159, in __call__
    await self.app(scope, receive, _send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/exceptions.py", line 82, in __call__
    raise exc from None
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/exceptions.py", line 71, in __call__
    await self.app(scope, receive, sender)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/routing.py", line 566, in __call__
    await route.handle(scope, receive, send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/routing.py", line 227, in handle
    await self.app(scope, receive, send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/routing.py", line 41, in app
    response = await func(request)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/fastapi/routing.py", line 195, in app
    dependency_overrides_provider=dependency_overrides_provider,
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/fastapi/dependencies/utils.py", line 550, in solve_dependencies
    solved = await run_in_threadpool(call, **sub_values)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/concurrency.py", line 34, in run_in_threadpool
    return await loop.run_in_executor(None, func, *args)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "pydantic/main.py", line 400, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for GetInput
rank
  ValueError() takes no keyword arguments (type=type_error)
ERROR:uvicorn.error:Exception in ASGI application
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/uvicorn/protocols/http/h11_impl.py", line 396, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/uvicorn/middleware/proxy_headers.py", line 45, in __call__
    return await self.app(scope, receive, send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/fastapi/applications.py", line 199, in __call__
    await super().__call__(scope, receive, send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/applications.py", line 111, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/middleware/errors.py", line 181, in __call__
    raise exc from None
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/middleware/errors.py", line 159, in __call__
    await self.app(scope, receive, _send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/exceptions.py", line 82, in __call__
    raise exc from None
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/exceptions.py", line 71, in __call__
    await self.app(scope, receive, sender)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/routing.py", line 566, in __call__
    await route.handle(scope, receive, send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/routing.py", line 227, in handle
    await self.app(scope, receive, send)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/routing.py", line 41, in app
    response = await func(request)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/fastapi/routing.py", line 195, in app
    dependency_overrides_provider=dependency_overrides_provider,
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/fastapi/dependencies/utils.py", line 550, in solve_dependencies
    solved = await run_in_threadpool(call, **sub_values)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/starlette/concurrency.py", line 34, in run_in_threadpool
    return await loop.run_in_executor(None, func, *args)
  File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "pydantic/main.py", line 400, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for GetInput
rank
  ValueError() takes no keyword arguments (type=type_error)

Operating System
macOS
Operating System Details
macOS 11.4
FastAPI Version
0.68.1
Python Version
Python 3.7.7
Additional Context
I posted the question on StackOverflow - https://stackoverflow.com/questions/68914523/pydantic-value-error-raises-internal-server-error
I did find a few questions on GitHub for the same but No clear answer was available.
Thank you for taking a look in advance.

Hey there,
To me, this looks more like you have a bug in your code somewhere --> explained here: https://fastapi.tiangolo.com/tutorial/handling-errors/?h=validation#requestvalidationerror-vs-validationerror

FastAPI uses it so that, if you use a Pydantic model in response_model, and your data has an error, you will see the error in your log.
But the client/user will not see it. Instead, the client will receive an "Internal Server Error" with a HTTP status code 500.
It should be this way because if you have a Pydantic ValidationError in your response or anywhere in your code (not in the client's request), it's actually a bug in your code.

Maybe you could start with changing your validator name. Instead of
@validator("*")

try with
@validator("my_validator")

More on validators you can read on Pydantic docs: https://pydantic-docs.helpmanual.io/usage/validators/
Hope this helps.

So what I tried to do is remove the Interval attribute from the Code.
class RankInput(BaseModel):
    """
    Get Rank Input class
    """

    rank: int

    @validator('rank')
    def check_if_value_in_range(cls, v):
        """
        check if input rank is within range
        """
        if not 0 < v < 1000001:

            raise ValueError("Rank Value Must be within range (0,1000000)")
        return v

the Code still ran into internal server error when a negative input was passed.

Also this is the Controller Calling the Model
@router.get('/rank/{rank}',status_code=200)
def get_info_by_rank(value: RankInput = Depends()):

    result = get_by_rank(value.rank)

    return result

Hey there,
To me, this looks more like you have a bug in your code somewhere --> explained here: https://fastapi.tiangolo.com/tutorial/handling-errors/?h=validation#requestvalidationerror-vs-validationerror

FastAPI uses it so that, if you use a Pydantic model in response_model, and your data has an error, you will see the error in your log.
But the client/user will not see it. Instead, the client will receive an "Internal Server Error" with a HTTP status code 500.
It should be this way because if you have a Pydantic ValidationError in your response or anywhere in your code (not in the client's request), it's actually a bug in your code.

Edit: This is because Pydantic validates your data both coming "in" and "out" of your application. So if data coming from your app doesn't validate through Pydantic, you will get 500 Internal Server Error.
I can see you're trying to validate everything at once. Maybe you could try firstly validating only one attribute.
@validator("rank")

or
@validator("interval")

More on validators you can read on Pydantic docs: https://pydantic-docs.helpmanual.io/usage/validators/
Hope this helps.

I removed the interval attribute. Check Above. Still Face the same Issue when a negative value is sent

I can confirm that I'm facing the exact same issue and that https://github.com/tiangolo/fastapi/issues/1474#issuecomment-803021344 fixes it but is definitely not nice.

I can confirm that I'm facing the exact same issue and that https://github.com/tiangolo/fastapi/issues/1474#issuecomment-803021344 fixes it but is definitely not nice.

Gave another suggestion there https://github.com/tiangolo/fastapi/issues/1474#issuecomment-1049987786 which seems to also work.


Update binaries for 2.7.0

Updated the binaries for 2.7.0
Updated script to reflect new install location of registry binary in official Dockerfile

Could you create 2.7.0 tag on Docker Hub?
https://hub.docker.com/_/registry/?tab=tags


add rbac_crn ouput in confluent_kafka_topic  resource

add rbac_crn ouput in confluent_kafka_topic  resource

Thanks for opening an issue!
For now you could use something like
resource "confluent_role_binding" "topic-example-rb" {
  principal   = "User:${confluent_service_account.test.id}"
  role_name   = "DeveloperWrite"
  crn_pattern = "${confluent_kafka_cluster.basic.rbac_crn}/kafka=${confluent_kafka_cluster.standard.id}/topic=${confluent_kafka_topic.orders.topic_name}"
}

Do you think have a new computed resource_name attribute might be useful?

I know you can build your own crn_pattern I think it could be more easy:
resource "confluent_kafka_topic" "orders" {
  kafka_cluster {
    id = confluent_kafka_cluster.basic-cluster.id
  }
  topic_name         = "orders"
  partitions_count   = 4
  rest_endpoint      = confluent_kafka_cluster.basic-cluster.rest_endpoint
  credentials {
    key    = confluent_api_key.app-manager-kafka-api-key.id
    secret = confluent_api_key.app-manager-kafka-api-key.secret
  }
}

resource "confluent_role_binding" "topic-example-rb" {
  principal   = "User:${confluent_service_account.test.id}"
  role_name   = "DeveloperWrite"
  crn_pattern = confluent_kafka_topic.orders.rbac_crn
}

PD: cluster resource export the attribute rbac_crn, why doesn't topic expose the attribute? :)

That's a great question @javiervelamindcurv!
The reason is it's an API limitation in some sense. If you closely look at cluster / topic resource definitions you'll see that even though confluent_kafka_cluster includes environment { id }, confluent_kafka_topic doesn't use environment { id }. The reason is GET cluster/id/ endpoint needs environmentID whereas for topics we use a completely different API and GET topic/id/ doesn't need environment { id }.
As a result, we can't really construct rbac_crn a topic since it should contain environment { id } which is effectively unknown at a topic resource level.
Let me know whether it makes sense!


SQL Diagnosis Report (WIP)

Generate the SQL diagnosis report, render by the backend, use bluma as CSS framework.

Remain work:

Fold the SubValues in the table default
Handle the router

Current progress:

Remain work:

Connect with the real data
Handle Router

Use real data, update the style:

Add time selector
Save the report to SQLite (resolve the auth issue as well)

Add tooltip to show the row comment:

Should all done, please help take a look. @breeswish @crazycs520 thanks!


sequential and general model in keras implementation

Hi,
When reading through some blogpost regarding keras implementations, I see some comments about sequential model and general model. For instance, VGG is sequential and Alexnet is not.  I am quite confusing about this comments. And moreover, when should we use sequential, and when should we use general model to implement some deep learning models in Keras?

Use sequential when your network is a basic list of layers, where each layer's output only goes to the next layer.
Use the functional API when your network is a DAG. E.g. http://rucs.ca/assets/xu-2layer.png, http://i.imgur.com/G3vwhsu.jpg.
BTW, I believe AlexNet is sequential. It only had the "two tower" structure so it could run on 2 GPUs.

Hi valtron, thanks for the reply. Can I understand this way, the network like FCN, U-Net that includes merge layer, then we have to use functional API. For the network like encoder-decoder, we can use sequential structure.

That's right. Also, you can take a look at the examples to get a better sense of the intended usage of each model.

Many thanks.


Fix geov-entity for embedding in website

Problem 1: Download RDF Button icons
Icons do not appear:

Solution 1
use "icon" attribute in ion-icon.
Problem 2
Class Label is rendered twice.

The problem seems to be caused by inproper hydration:

The string "Person" (rendered on server side) is not removed / attatched to the slot. It is unclear why.
Solution 2
Add two tests for component geov-entity-class-label that reproduce the problem.
Test 1:
(as in geov-entity-download-rdf.spec.tsx lines 62ff.)
Render the geov-entity-class-label server side
Test 2:
(as in geov-entity-download-rdf.spec.tsx lines 123ff.)

Use the output of test 1 as input of test 2.
Test the hydration and hopefully reproduce the bug

Then figure out a solution. Maybe it is enough to wrap the content of the component in a div.

Update: Tests of geov-entity-class-label are passed (no bug), so I'll create two news tests in geov-entity.

issue is resolved by this commit.
Solution: Change the  to  resolved the issue.


Error: cannot load such file -- EGP_Rates (LoadError)

This is the stack trace of the error
/usr/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in require': cannot load such file -- EGP_Rates (LoadError) 	from /usr/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in require'
from /var/lib/gems/2.3.0/gems/EGP_Rates_CLI-0.0.1/lib/EGP_Rates_CLI.rb:2:in <top (required)>' 	from /usr/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in require'
from /usr/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in require' 	from /var/lib/gems/2.3.0/gems/EGP_Rates_CLI-0.0.1/bin/egp:4:in <top (required)>'
from /usr/local/bin/egp:23:in `load'
I don't know Ruby so I am not sure what I needed  to do but I already installed EGP_Rates using
gem install EGP_Rates
and then ran
gem install EGP_Rates_CLI
No errors encountered until I run egp where I get that error.
I am running this on Ubuntu 16.04

thanks for reporting this... it should be now fixed
(BTW, you can ignore gem install EGP_Rates it should be automatically bundled (installed within) the gem install EGP_Rates_CLI command...

should be fixed now... please run gem install EGP_Rates_CLI to update
and please don't hesitate to report any other issue you find 😉


Fix some OpenSMILE setup issues

This fixes two issues that I ran into while setting this up for OpenSMILE:

It looks for SMILExtract in $OPENSMILE_HOME/bin but couldn't find it because it was actually in $OPENSMILE_HOME/bin/linux_x64_standalone_static in my installation.
It couldn't parse the OPENSMILE_HOME variable from the config because configparser automatically converted it to lowercase. I added a line to prevent this.

The pipeline opensmile_is10 works for me after these changes.

Thanks. I didn't even realize these bugs existed.


Add travis-debug command

This imports rwhogg/travis-debug into Linuxbrew.
I've rewritten it in Ruby rather than the original, which was in idiosyncratic Python. Also, it inlines the Dockerfile to avoid polluting the repo.
EDIT: It now uses https://github.com/Linuxbrew/docker/blob/master/travis/Dockerfile for its Dockerfile

I've updated this command to make use of the Travis CI Dockerfile.

Looks good! Good to merge with my two stylistic suggestions.
A feature request if you're interested: take an URL to a pull request on the command line, pull the PR and run brew test-bot inside the image.

Looks good! Good to merge with my two stylistic suggestions.

I'm good with both of the suggestions. I've been holding off on merging this until the images are available on Docker Hub.

I'm good with both of the suggestions. I've been holding off on merging this until the images are available on Docker Hub.

Anything other than time holding you back, anything you need from me?

Anything other than time holding you back, anything you need from me?

I need some automation around building the images. Pushing manually from my laptop is way too slow, the images are huge.

Use Docker Hub Automated Builds to automate building the images. https://hub.docker.com


Option to publish test name as published by framework

MethodFormat option to publish test name as published by framework
This used to be a default option, however it was removed while resolving another issue. Described in detail here

Any update on this? We are facing the same issue, and are therefore stuck on version 3.0.98 until this is resolved.

@GMillerVarian Can you give an example of what testing framework you are using and how you are setting the name of the method?

By the way, I spot checked @tamaratomilova example from #30, and it does look like the RMS name should be getting logged.
using Microsoft.VisualStudio.TestTools.UnitTesting;

[TestMethod("@RMS_TEST_NUMBER RMS Test name")]
public async Task MethodUnderTest_Scenario_Result()
{
    Assert.Whatever();
}

I was able to repro with the following:
using Microsoft.VisualStudio.TestTools.UnitTesting;

[TestMethod("@RMS-TEST-1 RMS Test without data rows")]
public void ValidateTest1()
{
    Validate(_sut, WritePolicy, "DeleteAsync");
}

[TestMethod("@RMS-TEST-2 RMS Test with data rows")]
[DataRow(ReadPolicy, "GetAsync")]
[DataRow(WritePolicy, "DeleteAsync")]
public void ValidateTest2(string policyName, string methodName)
{
    Validate(_sut, policyName, methodName);
}

Before (JunitXml.TestLogger 3.0.98):
<testcase classname="ValidateTests" name="@RMS-TEST-1 RMS Test without data rows" time="0.0005247" />
<testcase classname="ValidateTests" name="@RMS-TEST-2 RMS Test with data rows" time="0.0004483" />
<testcase classname="ValidateTests" name="@RMS-TEST-2 RMS Test with data rows" time="0.0003163" />

After (JunitXml.TestLogger 3.0.124):
<testcase classname="ValidateTests" name="@RMS-TEST-1 RMS Test without data rows" time="0.0004905" />
<testcase classname="ValidateTests" name="ValidateTest2 (ReadPolicy,GetAsync)" time="0.0004159" />
<testcase classname="ValidateTests" name="ValidateTest2 (WritePolicy,DeleteAsync)" time="0.0002794" />

So test 1 is fine, but we lose traceability in our RMS for test 2 with the DataRows.

@GMillerVarian  I'm having trouble re-producing that.

test.csproj

<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFramework>net7.0</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.5.0" />
    <PackageReference Include="MSTest.TestAdapter" Version="3.0.2" />
    <PackageReference Include="MSTest.TestFramework" Version="3.0.2" />
    <PackageReference Include="JunitXml.TestLogger" Version="3.0.98" />
  </ItemGroup>

</Project>

UnitTest1.cs

using System;
using System.Collections.Generic;
using System.Globalization;
using System.Reflection;
using System.Threading.Tasks;
using Microsoft.VisualStudio.TestTools.UnitTesting;

namespace NUnit.Xml.TestLogger.NetFull.Tests
{
    [TestClass]
    public class UnitTest1
    {
        [TestMethod("@RMS_TEST_NUMBER RMS Test name")]
        public void MethodUnderTest_Scenario_Result()
        {
            Assert.Fail();
        }

        [TestMethod("@RMS-TEST-2 RMS Test with data rows")]
        [DataRow("GetAsync")]
        [DataRow("DeleteAsync")]
        public void ValidateTest2(string methodName)
        {
            Assert.Fail();
        }
    }
}

dotnet test --logger:junit Gives the same result as your 'after' example. Can you give me a minimal example like this?

It looks like there is some inconsistent behavior depending on whether tests include rows with param of "Type" or "Type[]". Try adding the following tests (apologies, this is like what I had originally tested with, but I had edited the results to simplify things - not realizing this was actually the cause):
    [TestMethod("@RMS-TEST-3 RMS Test 3")]
    [DataRow("PostAsync", null)]
    [DataRow("GetAsync", typeof(string))]
    [DataRow("DeleteAsync", typeof(int))]
    public void ValidateTest3(string methodName, Type type)
    {
        Assert.Fail($"Failed on {methodName}");
    }

    [TestMethod("@RMS-TEST-4 RMS Test 4")]
    [DataRow("PostAsync")]
    [DataRow("GetAsync", typeof(string), typeof(string))]
    [DataRow("DeleteAsync", typeof(int))]
    public void ValidateTest4(string methodName, params Type[] extraParams)
    {
        Assert.Fail($"Failed on {methodName}");
    }

We can also see this with xUnit:
    [Fact(DisplayName = "@RMS-Test-1 Test 1")]
    public void Test1()
    {
        Assert.False(false);
    }

    [Theory(DisplayName = "@RMS-Test-2 Test 2")]
    [InlineData("param1")]
    [InlineData("param2")]
    public void Test2(string param)
    {
        Assert.False(false);
    }

Before (JunitXml.TestLogger 3.0.98):
    <testcase classname="TestProject1.UnitTest1" name="@RMS-Test-2 Test 2(param: "param2")" time="0.0027804" />
    <testcase classname="TestProject1.UnitTest1" name="@RMS-Test-2 Test 2(param: "param1")" time="0.0000143" />
    <testcase classname="TestProject1.UnitTest1" name="@RMS-Test-1 Test 1" time="0.0000507" />

After (JunitXml.TestLogger 3.0.124):
    <testcase classname="TestProject1.UnitTest1" name="Test2(param: "param2")" time="0.0020273" />
    <testcase classname="TestProject1.UnitTest1" name="Test2(param: "param1")" time="0.0000086" />
    <testcase classname="TestProject1.UnitTest1" name="Test1" time="0.0000483" />

That helped a bit.
So, test explorer and the loggers are seeing 4 tests (numbered in the image below) with 8 total results when we test your updated MSTest example. Each of those 4 tests can be run individually in test explorer. The tests using the display names each have 3 results, and the ones using the actual method names just have one.
In the 'best case' output using junit 3.0.98, we get 6 results using the display name and 2 using the method name. @GMillerVarian is that what you would expect or would want an option to output?

2023-04-27_18_16_47.trx.txt
TestResults junit 3.0.98.xml.txt
TestResults junit 3.0.124.xml.txt
@codito I'm surprised by the existence of the tests I numbered 2 and 4 above. But they seem to be real, and not linked to the others when they get logged. I'm including trx logger here as a cross check, and the GUIDs agree with that assessment. That suggest to me that any implementation which logs the display name is going to seem like logging the wrong thing in many places (see the 3.0.98 log). Also seems like we would need a feature flag for this on the core logger. What do you think?

We expect to see the 6 tests cases shown using display name (i.e. including the @RMS tag). This is what I see if I port that example to xUnit using junit 3.0.98. I'm not sure about the other 2 results with the method name, they seem redundant - but since they don't include the @RMS tag they should be irrelevant when we publish test results to our RMS.
Whether this is the default behavior or requires us to specify an option doesn't matter, as long as there is some way for us to get this behavior.

@codito @Siphonophora Any update on this issue?

@GMillerVarian @Siphonophora I think we have multiple issues here. (1) below is in scope for this logger. (2) and (3) are test framework specific concerns, we can take those to https://github.com/Microsoft/testfx repo.
(1) Behavior change between junit 3.0.98 and junit 3.0.124

That suggest to me that any implementation which logs the display name is going to seem like logging the wrong thing in many places (see the 3.0.98 log).

I agree with this. We should make MethodFormat tokenizable similar in spirit to LogFileName. E.g., dotnet test --logger:junit;MethodFormat={displayname} {method}, the possible tokens can be sourced from TestResultInfo data structure.
So far, the decision of what to report in MethodFormat is only configurable for junit. I'd suggest to keep it that way for now. If there's enough ask, we can push this functionality to core TestLogger like the LogFileName feature. @Siphonophora what do you think?
(2) MSTest reporting 8 test results vs xUnit reporting 6 results
2 extra test results in MSTest appears like a bug to me. Re https://github.com/spekt/junit.testlogger/issues/57#issuecomment-1526665206, trx also has 8 tests. Although trx is unique in supporting the parent/child relationship: we have two tests (starting with RMS tag as specified in TestMethod ctor) with 3 test results.
(3) MSTest using different displayName than xUnit
Hard to say whether this is a bug or by design. Test Explorer supports the same hierarchical model like trx.

@Siphonophora we're now exposing DisplayName in TestResultInfo. It uses TestCaseDisplayName as default, but is available for us to add tokenization support as proposed in above comment.

@codito @Siphonophora Are you able to provide an update as to when we can expect this to be supported?

@GMillerVarian sorry, we don't have an ETA for this.
If someone from the community would like to create a PR, I will be happy to support with review, merge and cutting a release. /cc:@Siphonophora


fix: add workaround for auth policy backend replication delays + lock DA into exact provider versions

Description

Add workaround for auth policy backend replication delays: https://github.com/terraform-ibm-modules/terraform-ibm-watsonx-saas-da/issues/148
Lock DA into exact provider versions: https://github.com/terraform-ibm-modules/terraform-ibm-watsonx-saas-da/issues/155

Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

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

GitHub release
v1.4.11

Your semantic-release bot :package::rocket:


Nothing happens

After Installing the package i added populate like this..
BelongsTo::make('Customer')->searchable()->prepopulate(),
but nothing happens at all, same as with only searchable, do i miss something?
regards
Denis

@Dontorpedo  would you guys mind testing version 1.2.0 and let me know if that fixes things for you? Thanks!

@Dontorpedo I'll close this issue for now. Please feel free to respond if you are still having this problem. Thank you! :)


scrolling on phone ?

i tried use the example on iphone. Is it easy to implement scrolling so that user can scroll through the document rather than clickign next button on top nav bar ?

Duplicate of #146


PEO-370 Implement i18n-tasks runner and add tests

PEO-370
Description
Need to implement i18n-tasks runner.
Configuration:
If there is a configuration file in the application - it should be read it by default, otherwise, it's read from our configuration.
By default each level of the issue equal low.
Before submitting the merge request make sure the following are checked

[x] Followed the style guidelines of this project
[x] Performed a self-review of own code
[x] Wrote the tests that prove that fix is effective/that feature works

This branch is out-of-date with the base branch.
Merge the latest changes from develop into this branch.

@rostikkkk2 could you please rebase with the develop branch and add integration tests, thanks


support pydantic 2.x conflict with fastapi

hey
I'm using this library with fastapi that uses pydantic >2
to update my swagger version
but because you have pinned to a version older than 2 .
I'm unable to use the news feature on fastapi
Thank you in advance.

Thanks for reminding me, I have plans to support pedantic 2 by the end of the week.

Published new version 0.3.2 with support for both pydantic 1 & 2.


How to define the output directory for .obj

Hi,
How can I defined in premake the compiler output :

I try to add to -o=path/to/my/output to the cudaCompilerOptions variable, but it didn't work.
Is it possible to defined this parameters with your module ?

Right now, the CUDA output directory (CudaIntDirFullPath) is the same as the project output directory. I will look into adding this feature.

Can you please check if the latest commit fulfils your use-case by specifying cudaIntDir as shown in the premake5.lua example file?

Hi,
Thanks for looking at my problem. I try to use cudaIndDir with the last commit, but it didn't make any change in my project or in the test project of your repository.
For example in your test premake I use these lines :
local intDir = "intDir/"
cudaIntDir = intDir

but in visual studio it's still the default value :

Hello! In the test project, it was added for ExampleProjectEXE not ExampleProjectDLL (as in your screenshot):

I will check if it works for ExampleProjectEXE, or if there is something weird going on.

@davidAlgis I added it to the EXE project and it works for me. Could you confirm you didn't accidentally expect ExampleProjectEXE to have the variable set when the test project did it only for ExampleProjectDLL?

Strange. I will think about why it works for me and not for you. If you have any theories, let me know. I will try borrowing someone else's machine to test on as well. One of us must have got the configuration wrong, and I will investigate to see if something is wrong on my end.

Ok thanks, I'll check on my side too ! Tomorrow I'll use another computer, maybe it'll suceed on it...

I think I find the problem. It comes from the fact that I assigned the value of cudaIntDir with a variable. If I assigned it with a const value (not sure if it's the name in lua) it wortks.
with variable assignation:

with const assignation;

Maybe it's a basic mistake in lua, but how should I set cudaIntDir with another variable ?

Perhaps you can try this:
local myDir = "obj"
cudaIntDir(myDir)

Indeed it works ! Thank you a lot 👍

Great, I shall close this them.


use fa-info-circle instead of fa-bars

See #653

See if you can make this a full button with text "Brew Info" in the same style as the snippet buttons, and we can merge this.

OK.
I can't see any built in styles directly suitable (the other items in the editor menu bar are all dropdown groups).
I can hand-craft the necessary css rules, will first take a closer look around for existing styles (is hb built upon a style library, or are they all hand-crafted?).

Everything is hand-crafted. Even though the snippets are drop downs you can still borrow the css from them to get the look right.

If you want to just apply the same class as the other snippet buttons, the CSS should be handled fine. We can merge this in with v2.10.7

If you want to just apply the same class as the other snippet buttons, the CSS should be handled fine. We can merge this in with v2.10.7


How to prevent image unit conversion?

Is there a simple way of preventing the '9pc' in:
https://github.com/ramiromagno/wiley-njd/blob/885a1827617952d3d01156ab68e6af8c8e118337/template.qmd#L131
getting converted to inches in latex:
\begin{figure}[t]
{\centering \includegraphics[width=\textwidth,height=1.5in]{empty.pdf}}
\caption{\label{fig-fig1}This is the sample figure caption.}
\end{figure}

The motivation is getting a latex template that is identical to the journal's provided template. These small changes, although not semantically different, do make an automatic file comparison between templates a lot more difficult.

Being addressed here: https://github.com/jgm/pandoc/issues/8957.


[WIP] Use govuk_app_config for logging config

Closing until https://github.com/alphagov/govuk_app_config/pull/10 is merged & released.


runtimeConfig is empty in plugins

Environment

Operating System: Linux
Node Version:     v14.18.1
Nuxt Version:     2.16.0-27358576.777a4b7f
Package Manager<EMAIL_ADDRESS>Builder:          webpack
User Config:      privateRuntimeConfig, publicRuntimeConfig, plugins, bridge, buildModules
Runtime Modules:  -
Build Modules<EMAIL_ADDRESS>
Reproduction
Old configuration: https://codesandbox.io/s/hardcore-dream-07bz0n?file=/nuxt.config.js
New configuration: https://codesandbox.io/s/peaceful-williamson-vddeiq?file=/nuxt.config.js
Describe the bug
I've just copied the example from https://github.com/nuxt/framework/pull/4254, created new plugin and page, which prints config values in console. It logs nothing in plugin (plugins/plugin.js) but outputs public values at runtime (see pages/index.js)
Additional context
No response
Logs
No response

@pi0 Hi, any estimate for when this could be fixed (or when you plan to make the move to that new repo)? In my case I don't get any runtimeConfig values on the serverside when accessing them through this.$config in Vue components, and I'm guessing it's the same issue.

Checking against the latest version (v3.0.0-27497942.c519cee) it is fully compatible with both new and old syntax. Please check and ping to reopen if still having problems with runtimeConfig.

@pi0 I don't see any such version on npm, but the latest one available (3.0.0-27498038.f18e68b) has that same issue.
Here's how the config is defined in the nuxt config:

Here's how I'm reading it in my component/mixin:

And here's how it executes on the server-side:

Checking against the latest version (v3.0.0-27497942.c519cee) it is fully compatible with both new and old syntax. Please check and ping to reopen if still having problems with runtimeConfig.

work for me, i use setup syntax
and  i think runtimeConfig it's just nitro only , when bridge : { nitro : false } ,will show error,  It would be better if there was more detailed documentation or just tell users not to do this

I can't reproduce the original issue here: https://stackblitz.com/edit/github-frh3le.
Splitting @aldarund's issue into a separate one.