Error during /call

🐛 Bug
I create a simple contract with set/get funcs and public variable in testnet.
Contract address - 0x68badaa4b93cc043977a7c8647a05774df84d069
To reproduce

open testnet api page
call 0x68badaa4b93cc043977a7c8647a05774df84d069 with 0xf8a8fd6d data

Expected Behavior
you will get a error in console
14:08:54.893 WARN     ApiHandlerImpl   Internal error
java.lang.reflect.InvocationTargetException: null
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?]
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:?]
at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?]
at io.netty.channel.CombinedChannelDuplexHandler$
System information

Operating system: linux
Semux version: 2.0.2

The temporary workaround is to fill the from address. We will fix this issue in next release.

Fixed as in v2.1


Support Content Negotiation for returning Light Client related data as either JSON or SSZ

The REST API for light client related data should also be able to return results as SSZ.
Ref: https://github.com/ethereum/beacon-APIs/pull/247

Depending on the Accept header it can be returned either as JSON or SSZ-serialized bytes.
Is your feature request related to a problem? Please describe.

Superseded by #5128


[Feature Request]: Implement bulk apis in Salesforce transforms

What would you like to happen?
Improve current Salesforce transforms by adding the usage of bulk APIs to speedup the operations
https://developer.salesforce.com/docs/atlas.en-us.api_asynch.meta/api_asynch/asynch_api_intro.htm
Issue Priority
Priority: 3
Issue Component
Component: Transforms

I'd love to see this being implemented.


Update README.md

Add the keyword 'resource' so it comes up in a generic search such as https://github.com/search?o=desc&p=5&q=perl+resources&s=updated&type=Repositories

Thank you for your contribution 👍


Doesn't handle numbers in scientific notation

Parchment on mobile Chrome is producing this, which I think is not being parsed by Remglk, and causing the Emglken terps to crash: "buffermarginy":7.105427357601002e-15.
I'll see if I can find a solution on my end, but I'm also raising this as something you might want to look at, for full JSON compatiblility.
Originally posted by @curiousdannii in https://github.com/erkyrath/remglk/issues/46#issuecomment-1207351986

Gah, what a nuisance. And yes, remglk should be able to handle that even though it's a nuisance.

Take a look at branch https://github.com/erkyrath/remglk/tree/double-expo and see if that works?
Another change in this branch: where the library expects an integer, it will round to the nearest integer rather than towards zero. This does not effect metrics (where the library expects floats). But if you send a gen or window value of 9.9, it will now be rounded to 10.

Ah thanks! I'll give it a try soon.
Hmm, I think the only situation in which rounding makes sense is for mouse event coordinates, to cover the case where someone forgot to round them. For the generation number I'd expect that to be an error. But it's also probably very unlikely to happen, and if it simplifies the code a lot to just automatically round them, then it's probably okay.

Yep, that branch works perfectly, thanks!

Great.
We can't require a JSON-format spec to reject 10.0 when it expects 10. JS can't distinguish those cases. JSON libraries in other languages may or may not, so yeah, it's simpler if this one doesn't.


How do you add parameters to formData payload ?

I have a post call, I am trying to add parameters to formdata with no success

Can you give a bit more info? What do your swagger-blocks definitions look like and how are they not working?

here is an example
      swagger_path '/sessions' do
        operation :post do
          key :description, 'Login To Vestorly Platform'
          key :operationId, 'login'
          key :consumes, [
            "application/x-www-form-urlencoded"
          ]
          key :tags, [
            'session'
          ]
          parameter do
            key :name, :username
            key :in, :formData
            key :description, 'Username in the XXXX platform'
            key :required, true
            key :type, :string
          end

          parameter do
            key :name, :password
            key :in, :formData
            key :description, 'Password in XXXX Platform'
            key :required, true
            key :type, :string
          end

          response 200 do
            key :description, 'Response'
            schema do
              key :'$ref', :Session
            end
          end
        end
      end

Here is the call that works in CURL:
curl -X POST https://www.example.com/api/v2/sessions
  -d "username=USERNAME"
  -d "password=PASSWORD"

I cannot get swagger-blocks to send those username + password parameters in the post

also according to swaggers 2.0 spec, you can use the :formData parameter:
https://github.com/swagger-api/swagger-spec/blob/master/versions/2.0.md

That looks correct to me, except I don't understand what you mean by "get swagger-blocks to send those username + password parameters in the post"—Swagger::Blocks doesn't send anything, it's simply a way to define JSON in the Swagger style, for example as input to Swagger UI for generating API docs—it has nothing to do with actually how your server handles data.
If the output of your /apidocs endpoint (see main README) is incorrect, that's something I can help you with, but otherwise it sounds unrelated to swagger-blocks. Closing for now, but feel free to comment with more info if you think it is actually swagger-blocks related.

Sorry - I see why that is confusing. I meant when I generate swagger.json
and try to use this method in Swagger UI, the post does not send formdata.
On Wed, Apr 15, 2015 at 4:13 PM, Mike Fotinakis<EMAIL_ADDRESS>wrote:

That looks correct to me, except I don't understand what you mean by "get
swagger-blocks to send those username + password parameters in the
post"—Swagger::Blocks doesn't send anything, it's simply a way to define
JSON in the Swagger style, for example as input to Swagger UI
https://github.com/swagger-api/swagger-ui for generating API docs—it
has nothing to do with actually how your server handles data.
If the output of your /apidocs endpoint (see main README) is incorrect,
that's something I can help you with, but otherwise it sounds unrelated to
swagger-blocks. Closing for now, but feel free to comment with more info if
you think it is actually swagger-blocks related.
—
Reply to this email directly or view it on GitHub
https://github.com/fotinakis/swagger-blocks/issues/16#issuecomment-93552383
.

Can you check that the swagger.json looks correct according to the spec and maybe manually fiddle with it until it works with Swagger UI the way you expect? Unfortunately this sounds like you're just missing some part of the configuration between Swagger JSON and Swagger UI, but not actually a problem with how swagger-blocks generates JSON.

the issue was with swagger-ui: https://github.com/swagger-api/swagger-ui/issues/838
sorry for wasting your time :)

Not a waste, glad you found it. :+1:


update to react lifecycle methods per react 16.3

Using componentDidMount to perform flatpickr setting per react recommendations on replacement  of componentWillRecieveProps

Thank you!


Scenario titles of saved scenarios and presets should also be shown in the scenario-bar

When I open multiple preset scenarios I sometimes get confused which scenario is which since the title of the preset scenario (f.e. II3050 - European CO2 governance) is not shown in the scenario bar. I think this would be a nice improvement!
Additionally, scenario titles of saved scenarios created by other people do not show up in the scenario bar. I think this would also be nice :) When you then save the scenario yourself you can change that title if you want.
What do you think @noracato  and @antw?

Put this also in the report title. Now the report title is default value:

I'm using the ETM for every RES-region in the Netherlands. So I use the scenario-name, and want to use this name also in the Title of the report.

I agree with all the above.
We only show the title for your own saved scenarios; this is why you don't get a title when opening a featured scenario, or a saved scenario belonging to someone else. I agree this is not ideal, but I didn't get time to adjust it before the 2020.11 release.

@antw I would really like it when this would be done :) I use the featured scenarios often and get confused which one I've opened. Are there more people that would like this? @quintel/core

@antw I would really like it when this would be done :) I use the featured scenarios often and get confused which one I've opened. Are there more people that would like this? @quintel/core

Great that we can ow see titles of featured scenarios in the scenario navigation bar!
@antw, I think it would also be an improvement to show the titles of saved scenarios created by others in the navigation bar! I just had to open several saved scenarios from others and didn't know which one was which

If you're loading based on an ETEngine scenario rather than a saved scenario in ETModel (for example) then you will not get a scenario title. That's because the scenario title is not stored in ETModel; scenario titles in ETEngine are deprecated and likely to be removed in the future.

Okay! Yes, I was opening scenarios created through our API, so that's why.


Create PULL_REQUEST_TEMPLATE.md

See PULL_REQUEST_TEMPLATE.md for details.
Resolve #63

Went over with everyone.


Rust Back-End

Begin rewriting physics code in rust.

Using numpy tensor math is faster than rust bindings

The rust part of the project is unnecessary and should be deleted.


chore: add cloud functions destination support

PR summary
Includes the changes of cloud functions as a destination
Fixes: #4861
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

What is the current behavior?
no cloud functions destination suppport
What is the new behavior?
we can configure the cloud functions as a destination in EN
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

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


Explore AWS bucket versioning

Good question from @nickreich: would it be possible to use S3 bucket versioning as a way for users to access hub data at specific points in time?

Noting that the genome data that @elray1 and I are starting to work with seems to be using S3's versioning capabilities for this kind of work.


  Nginx server behind Rackspace load balancer.  Issue with compression?
https://www.hmhship.com/min/?f=/min/quick-test.css&debug=1

No version info provided

aside, here are curl request reproducing the problem:
response is gzip compressed:
➔ curl -s 'https://www.hmhship.com/min/?f=/min/quick-test.css&debug=1' -H 'If-None-Match: "pub1482439458;gz"' -H 'Accept-Encoding: gzip, deflate, sdch, br' -H 'Accept-Language: et,en;q=0.8,en-US;q=0.6,ru;q=0.4' -H 'Upgrade-Insecure-Requests: 1' -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.100 Safari/537.36' -H 'Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8' -H 'Cache-Control: max-age=0' -H 'Cookie: X-Mapping-fjhppofk=AB645B0CAE1BE6405643D25CBFCBE4B7' -H 'Connection: keep-alive' -H 'If-Modified-Since: Thu, 22 Dec 2016 20:44:18 GMT' --compressed|file -

/dev/stdin: gzip compressed data, max compression, from Unix

there's even Content-Encoding: gzip header:
➔ curl -sI 'https://www.hmhship.com/min/?f=/min/quick-test.css&debug=1' -H 'If-None-Match: "pub1482439458;gz"' -H 'Accept-Encoding: gzip, deflate, sdch, br' -H 'Accept-Language: et,en;q=0.8,en-US;q=0.6,ru;q=0.4' -H 'Upgrade-Insecure-Requests: 1' -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.100 Safari/537.36' -H 'Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8' -H 'Cache-Control: max-age=0' -H 'Cookie: X-Mapping-fjhppofk=AB645B0CAE1BE6405643D25CBFCBE4B7' -H 'Connection: keep-alive' -H 'If-Modified-Since: Thu, 22 Dec 2016 20:44:18 GMT' --compressed
HTTP/1.1 200 OK
Server: nginx/1.4.6 (Ubuntu)
Date: Thu, 22 Dec 2016 21:05:24 GMT
Content-Type: text/css; charset=utf-8
Content-Length: 383
Connection: keep-alive
X-Powered-By: PHP/5.5.9-1ubuntu4.14
Vary: Accept-Encoding
Last-Modified: Thu, 22 Dec 2016 20:44:18 GMT
ETag: "pub1482439458;gz"
Cache-Control: max-age=0
Content-Encoding: gzip
X-Server: hmhship-live

I fixed this issue by modifying lib/Minify.php line:
if (self::$_options['encodeOutput']) {
to
if (false)
why is there no setting for this in config.php?

In config.php: $min_serveOptions['encodeOutput'] = false;


[QST] What is the best way of handling string UUIDs in Merlin?

❓ Questions & Help
Details
What would be the best way of going about this?
My original idea was to either Hash or Categorify the UUID values for training, then have the user_id_raw and item_id_raw string values be returned from the Feast store during inference so that my application can get the raw UUIDs. However, Merlin does not support this paradigm as I get type errors during inference.
Another option would be to map the UUIDs to Int separately but I'm not sure how and where this should be implemented with Merlin.
Thank you in advance.

@BlakeB415  as I get type errors during inference.
please share your error, and a toy example to repro the issue. thanks.


v3 Component API Revisions

We've updated our component API conventions with a few notable improvements:

State is now fed into parent components when using the parent/child structure
Use the convention value for data in, onChange for data out
Only unique child-specific state (ex: id) should ever be provided to the children directly
Form components that embed non-visible inputs for state should now use type="hidden"
For Svelte, we can utilize the new createContext utility to utilize type-safe Context API values
Additionally, we will continue to lean into framework-specific conventions when possible

Component Updates
The following components will need to be updated to adhere to the new guidelines:

[x] Nav Tile/Bar
[x] Segment Control
[x] Switch
Tabs - moved to #2759

General Changes

[x] Update React unit tests
[x] Update React schema assets

Doc Updates
We'll need a few documentation revisions:

[x] Update example snippets for all affected components.
[x] Update the Component Contribution guide to showcase these new guidelines

With the recent Zag integration this will need to re-prioritize this. I'll close this in favor of a new task.


calculate the delta

we need to show what changed
using JSON Patch, it will show the delta
encapsulate an opensource JSONP library to show what has changed to an object during the audited process/method

https://github.com/dbones-labs/auditable/commit/b4729da75f4c0db5445e255e8f8eacc5c264c093


auto keybinding issues

As many other players in ED I have a josytick to play. My primary controls usually are mapped to it but not all. I assume its only parsing the primary?
Parsing keybinds: Custom
JOY_3 is not in SCANCODE list!!
POS_JOY_RZAXIS is not in SCANCODE list!!
NEG_JOY_RZAXIS is not in SCANCODE list!!
JOY_POV1UP is not in SCANCODE list!!
JOY_POV1RIGHT is not in SCANCODE list!!
KEY_ENTER is not in SCANCODE list!!
JOY_2 is not in SCANCODE list!!
JOY_POV1LEFT is not in SCANCODE list!!
JOY_POV1UP is not in SCANCODE list!!
JOY_POV1DOWN is not in SCANCODE list!!
 is not in SCANCODE list!!
 is not in SCANCODE list!!
Error in setting keybind(s): ['UI_Back', 'Pause']
Successfully added 24 keybinds, 2 failed.

it looks like the code does a keyTranslate()  which will output this error message but then checks to see if the type is primaryDevice == Keyboard and if not jumps down to secondaryDevice and check to see if that is a Keyboard type.   So it looks like it works but need to clean up the invalid error messages

Fixed. Now the keyTranslate() will only kick in if the device == 'Keyboard'


Traces ingestion into Splunk platform

Hi all,
I see a limitation to ingest Otel traces only to Splunk Observability and not to Splunk platform. Is this posses by Splunk itself or is it viable to ingest traces to Splunk platform as well. If so, can this be addressed?
Thanks,
Hitesh

cc @rockb1017

Hi @rockb1017 @jodeev   @dmitryax
Can you pl confirm on this one if the limitation is posed by Splunk or is it for connector alone?
Thanks,
Hitesh

Hi Hitesh, Splunk can consume traces, but they are then rendered as JSON documents. This is not our preferred route: we want to make sure traces can be interpreted in a backend that can associate them together. As such, I am closing the ticket. Please reopen if you have a use case involving Splunk Core which warrants trace ingestion, and we will investigate further.


ValueError: Connection Error: failed 29 Segmentation fault after dl_docs

Description of the bug
After trying to download documents via dl_docs im getting following error:
Error log
$ pytr -v debug dl_docs .
2023-10-02 20:43:21+0200 main      DEBUG    logging is set to debug
2023-10-02 20:43:21+0200 account   INFO     Found credentials file
2023-10-02 20:43:21+0200 account   INFO     Phone: ********, PIN: ****
2023-10-02 20:43:21+0200 account   INFO     Web session resumed
2023-10-02 20:43:21+0200 account   INFO     Logged in
2023-10-02 20:43:21+0200 dl        INFO     Found 419 lines in history file
2023-10-02 20:43:21+0200 utils     INFO     Awaiting timeline
2023-10-02 20:43:21+0200 api       INFO     Connecting to websocket ...
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Users\xxxxxx\AppData\Local\Programs\Python\Python311\Scripts\pytr.exe_main.py", line 7, in
File "C:\Users\xxxxxx\AppData\Local\Programs\Python\Python311\Lib\site-packages\pytr\main.py", line 206, in main
asyncio.get_event_loop().run_until_complete(dl.dl_loop())
File "C:\Users\xxxxxx\AppData\Local\Programs\Python\Python311\Lib\asyncio\base_events.py", line 653, in run_until_complete
return future.result()
File "C:\Users\xxxxxx\AppData\Local\Programs\Python\Python311\Lib\site-packages\pytr\dl.py", line 59, in dl_loop
await self.tl.get_next_timeline(max_age_timestamp=self.since_timestamp)
File "C:\Users\xxxxxx\AppData\Local\Programs\Python\Python311\Lib\site-packages\pytr\utils.py", line 236, in get_next_timeline
await self.tr.timeline()
File "C:\Users\xxxxxx\AppData\Local\Programs\Python\Python311\Lib\site-packages\pytr\api.py", line 443, in timeline
return await self.subscribe({'type': 'timeline', 'after': after})
File "C:\Users\xxxxxx\AppData\Local\Programs\Python\Python311\Lib\site-packages\pytr\api.py", line 293, in subscribe
ws = await self._get_ws()
File "C:\Users\xxxxxx\AppData\Local\Programs\Python\Python311\Lib\site-packages\pytr\api.py", line 279, in _get_ws
raise ValueError(f'Connection Error: {response}')
ValueError: Connection Error: failed 29
Segmentation fault
To Reproduce
Steps to reproduce the behavior:
Ran the command pytr dl_docs .
Environment

Windows 10
pytr version: 0.0.15

Additional context
Tried python 3.7 python 3.9 and python 3.11
It seems that after the Trade Repbulic update they changed the API?!

Connecting to websocket ...
Traceback (most recent call last):
File "C:\Python39\Scripts\pytr-script.py", line 33, in
sys.exit(load_entry_point('pytr==0.1.5', 'console_scripts', 'pytr')())
File "C:\Python39\lib\site-packages\pytr\main.py", line 206, in main
asyncio.get_event_loop().run_until_complete(dl.dl_loop())
File "C:\Python39\lib\asyncio\base_events.py", line 641, in run_until_complete
return future.result()
File "C:\Python39\lib\site-packages\pytr\dl.py", line 59, in dl_loop
await self.tl.get_next_timeline(max_age_timestamp=self.since_timestamp)
File "C:\Python39\lib\site-packages\pytr\utils.py", line 236, in get_next_timeline
await self.tr.timeline()
File "C:\Python39\lib\site-packages\pytr\api.py", line 443, in timeline
return await self.subscribe({'type': 'timeline', 'after': after})
File "C:\Python39\lib\site-packages\pytr\api.py", line 293, in subscribe
ws = await self._get_ws()
File "C:\Python39\lib\site-packages\pytr\api.py", line 279, in _get_ws
raise ValueError(f'Connection Error: {response}')
ValueError: Connection Error: failed 29
on me
windows 10
python3.9

Same here.
Login seems to work. But everything else results in "failed 29".

Hi,
Same here on Mac. I can confirm, that it worked yesterday morning (last download 11:13 german time and the days before) but than next time 13:13 (did a reboot, this is why I know the exact time) I wanted to use it yesterday, it did not work any more. Same errors as above. Did not change anything or update on the Mac in between.
thanks
Juergen

I have the same issue. Perhaps it has to do with the new traderepublic app that was recently launched ....?

I had also the following error:
ValueError: Connection Error: failed 29
When you look in the traceback it points to api.py:249
Since the error is regarding the websocket connect function i looked in the devtools for the webapplication of TR. There it said it connects on 28 - Atleast for me...
So back to the api.py file. A littlebit over the error line (line 272) you find connect_id = 22. I tried to change it to 29 since this is what the error said. Result is it worked after saving the file.
I also tried 28 and 24, which also worked... It just didnt work for 21 or 22. So I dont exactly know why those numbers are not working but i think something changed on TR side.
I would have done a change request or smth... but i have never done this. Maybe someone smarter who is reading this can take over from here on.
Greetings

Good finding. Once i have checked it i will do a PR.
Is this repo still be maintained? As i can still see old PRs which doesnt be merged yet i guess no one gona merge mine. Lets see, at least we have a solution

So back to the api.py file. A littlebit over the error line (line 272) you find connect_id = 22. I tried to change it to 29 since this is what the error said. Result is it worked after saving the file.
I also tried 28 and 24, which also worked... It just didnt work for 21 or 22. So I dont exactly know why those numbers are not working but i think something changed on TR side.

Working for me as well. Thanks for the solution!

Can someone tell me, how to integrate the mentioned adjustments in my Win11 Python?
I am not that good in python. Just installed this long time ago for trade republic.
Thanks

So back to the api.py file. A littlebit over the error line (line 272) you find connect_id = 22. I tried to change it to 29 since this is what the error said. Result is it worked after saving the file.
I also tried 28 and 24, which also worked... It just didnt work for 21 or 22. So I dont exactly know why those numbers are not working but i think something changed on TR side.

Working for me as well. Thanks for the solution!

HI,
works here as well. Very easy, even no programmers can do the change, IMHO (I am non programmer :-) )
thanks a bunch
Juergen

Can someone tell me, how to integrate the mentioned adjustments in my Win11 Python? I am not that good in python. Just installed this long time ago for trade republic. Thanks

Hi,
in the error it says somewhere at the end:
File "/usr/local/lib/python3.10/site-packages/pytr/api.py", line 279, in _get_ws
  raise ValueError(f'Connection Error: {response}')
ValueError: Connection Error: failed 29

which gives you the path where the api.py file is sitting.
Take an editor (notepad, vi, ... NOT WORD) and

Can someone tell me, how to integrate the mentioned adjustments in my Win11 Python? I am not that good in python. Just installed this long time ago for trade republic. Thanks

Hi, in the error it says somewhere at the end:
[...schnipp...]
save the file and start again. Worked for me, and a couple of others immediately.

Well explained, even for beginners. Also worked for me immediately. Thank you from the sunny beach of the Baltic Sea.

Can someone tell me, how to integrate the mentioned adjustments in my Win11 Python? I am not that good in python. Just installed this long time ago for trade republic. Thanks

Hi, in the error it says somewhere at the end:
File "/usr/local/lib/python3.10/site-packages/pytr/api.py", line 279, in _get_ws
  raise ValueError(f'Connection Error: {response}')
ValueError: Connection Error: failed 29

which gives you the path where the api.py file is sitting. Make a copy of this file api.py file, if the following does not work Take an editor (notepad, vi, ... NOT WORD) and edit this api.py (original, not copy). Go to line 272 in this file. You should see something like: connect_id = 22 or serach for this line, if it is not there. Than change the 22 to a 29 so that it looks like connect_id = 29
save the file and start again. Worked for me, and a couple of others immediately.
thanks Juergen

(base) C:\Users\xxx>pytr portfolio
17:44:40 Found credentials file
17:44:40 Phone: +49***********, PIN: ****
17:44:40 Web session resumed
17:44:40 Logged in
17:44:40 Connecting to websocket ...
17:44:40 Connected to websocket ...
17:44:40 Received error message: '1 E {"errors":[{"errorCode":"BAD_SUBSCRIPTION_TYPE","errorField":null,"errorMessage":"Unknown topic type","meta":{"source":"MAPPER"}}]}'
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Users\xxx\anaconda3\Scripts\pytr.exe_main.py", line 7, in
File "C:\Users\xxx\anaconda3\Lib\site-packages\pytr\main.py", line 215, in main
Portfolio(login(phone_no=args.phone_no, pin=args.pin, web=not args.applogin)).get()
File "C:\Users\xxx\anaconda3\Lib\site-packages\pytr\portfolio.py", line 73, in get
asyncio.get_event_loop().run_until_complete(self.portfolio_loop())
File "C:\Users\xxx\anaconda3\Lib\asyncio\base_events.py", line 653, in run_until_complete
return future.result()
^^^^^^^^^^^^^^^
File "C:\Users\xxx\anaconda3\Lib\site-packages\pytr\portfolio.py", line 16, in portfolio_loop
_subscription_id, subscription, response = await self.tr.recv()
^^^^^^^^^^^^^^^^^^^^
File "C:\Users\xxx\anaconda3\Lib\site-packages\pytr\api.py", line 352, in recv
raise TradeRepublicError(subscription_id, subscription, payload)
pytr.api.TradeRepublicError: ('1', {'type': 'portfolio'}, {'errors': [{'errorCode': 'BAD_SUBSCRIPTION_TYPE', 'errorField': None, 'errorMessage': 'Unknown topic type', 'meta': {'source': 'MAPPER'}}]})
Changing 22 to 29 doesn't work for me... :-(

File "C:\Users\xxx\anaconda3\Lib\site-packages\pytr\api.py", line 352, in recv
raise TradeRepublicError(subscription_id, subscription, payload)
pytr.api.TradeRepublicError: ('1', {'type': 'portfolio'}, {'errors': [{'errorCode': 'BAD_SUBSCRIPTION_TYPE', 'errorField': None, 'errorMessage': 'Unknown topic type', 'meta': {'source': 'MAPPER'}}]})

Be careful. Don't mess up things.
We discuss here the problem with "ValueError: Connection Error: failed 29". This will get solved with changing the 21/22 to 29.
If you get an error with BAD_SUBSCRIPTION_TYPE, then this problem here is solved already. The portfolio is not available since they changed the API and will return this BAD_SUBSCRIPTION_TYPE error.

Hi,
The error “BAD_SUBSCRIPTION_TYPE” was solved with the miry fork version.
But I think the issue is back now.
Changing the ID to 29 solves the error for command “dl_docs”.
After this “Connection Error: failed 29” - , even with the fork version, doesn't work.
Can someone confirm this?

Hi,
The error “BAD_SUBSCRIPTION_TYPE” was solved with the miry fork version. But I think the issue is back now.
Changing the ID to 29 solves the error for command “dl_docs”.
After this “Connection Error: failed 29” - , even with the fork version, doesn't work.
Can someone confirm this?

Same problem here

Unfortunately, no. I changed the connect id and installed with this method again.
$ pip3 install .
$ pytr -v debug portfolio
But I am getting the same error as before.

Hi,
Can someone confirm this?

Hi,
I just tried:
pytr portfolio
and got
16:35:35 Received error message: '1 E {"errors":[{"errorCode":"BAD_SUBSCRIPTION_TYPE","errorField":null,"errorMessage":"Unknown topic type","meta":{"source":"MAPPER"}}]}'
Traceback (most recent call last):
  File "/usr/local/bin/pytr", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.10/site-packages/pytr/main.py", line 215, in main
    Portfolio(login(phone_no=args.phone_no, pin=args.pin, web=not args.applogin)).get()
  File "/usr/local/lib/python3.10/site-packages/pytr/portfolio.py", line 73, in get
    asyncio.get_event_loop().run_until_complete(self.portfolio_loop())
  File<EMAIL_ADDRESS>line 649, in run_until_complete
    return future.result()
  File "/usr/local/lib/python3.10/site-packages/pytr/portfolio.py", line 16, in portfolio_loop
    _subscription_id, subscription, response = await self.tr.recv()
  File "/usr/local/lib/python3.10/site-packages/pytr/api.py", line 352, in recv
    raise TradeRepublicError(subscription_id, subscription, payload)
pytr.api.TradeRepublicError: ('1', {'type': 'portfolio'}, {'errors': [{'errorCode': 'BAD_SUBSCRIPTION_TYPE', 'errorField': None, 'errorMessage': 'Unknown topic type', 'meta': {'source': 'MAPPER'}}]})

As I am not a programmer, I was happy, that the automated doc DL worked again, I never used anything else.
thanks
Juergen

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python312-arm64\Scripts\pytr.exe\__main__.py", line 7, in <module>
  File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python312-arm64\Lib\site-packages\pytr\main.py", line 215, in main
    Portfolio(login(phone_no=args.phone_no, pin=args.pin, web=not args.applogin)).get()
  File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python312-arm64\Lib\site-packages\pytr\portfolio.py", line 120, in get
    asyncio.get_event_loop().run_until_complete(self.portfolio_loop())
  File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python312-arm64\Lib\asyncio\base_events.py", line 664, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "C:\Users\xxxxx\AppData\Local\Programs\Python\Python312-arm64\Lib\site-packages\pytr\portfolio.py", line 57, in portfolio_loop
    pos['netValue'] = response['last']['price'] * pos['netSize']
                      ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
TypeError: can't multiply sequence by non-int of type 'str'

Still getting the frozen runpy error after debugging portfolio.

Fixed in latest version


Era names and codes for ethiopic

As said, ethiopaa calendar has only one era, so no era code is necessary.
Generally, this unique era is called "Anno Mundi", but you can also call it "Amete Alem", abbreviated "A. A.", narrow "A".
Here is one possible source : https://www.google.fr/books/edition/Internal_Rivalries_and_Foreign_Threats/hCqedAA_Z5IC?hl=fr&gbpv=1&dq=amete+alem&pg=PR11&printsec=frontcover
ethiopic use two eras. Expert name the current one "Age of Mercy" ("An de grâce"), or "Incarnation", but IMHO, if we display the months as a transliteration of the original language, we should do the same for the era. If you follow the beforementioned source, you can use:

Amete Alem -> code aa
Amete Mihret -> code am

These codes are simple, unique, and well understood by people who know about the Ethiopic culture.
BTW, the first era displayed by Intl. should be "Amete Alem" or "Anno Mundi" or "from the World's Creation" but NOT "before Incarnation", as years are not counted backwards, but from the year -5493 epoch.

If we adopt the proposed Gregorian era codes in https://github.com/FrankYFTang/proposal-intl-era-monthcode/issues/5#issuecomment-1297496031, then we should most likely follow the same convention of using the calendar ID as the era code name here for Ethiopic, too.
This would mean era codes should be

ethiopic = Amete Mihret / Incarnation (current era)
ethioaa = Amete Alem / Anno Mundi

It's unclear from what I can find online whether dates before Amete Mihret are rendered counting backwards (B.C. style), or whether they count forwards from the Amete Alem epoch. I've found https://ethiopiancalendar.wordpress.com/history/ to be a fairly informative source, though it's hard to verify. It does say the following:

The Geez Calendar (ቀለንጦስ) is divided into old and new. The old era which is equivalent to the B.C. is Zemene Bluy (Z.B.) or (ዘመነ ብሉይ). Zemene Haddis (Z.H.) or (ዘመነ ሓዲስ) is Anno Domini (A.D.), though it is commonly referred to as Amete Mihret (A.M.) which means “years of mercy”.

To support the case of Zemene Bluy:

pre-ethiopic = counting backwards from Amete Mihret (year 1 in pre-ethiopic is the year before year 1 in ethiopic)

The good thing is that we don't really need to know which one is the canonical representation. We can support pre-ethiopic for input, and if we later find that it is invalid for output, we can just stop outputting it, but we can still accept it as input.
Another case here is how we write dates that are before the beginning of Amete Alem. According to the Ethiopian church, such dates simply don't exist. However, since they exist in ISO-8601, we should have a way of representing them. I think in this case, as in other cases, we just use negative numbers in the ethioaa era.

@sffc has found a treasure about calendars in general, not only about the calendars of Ethiopia.
Let me quote a part:

According to Ethiopian scholars such as Aleqa Kidane Wold Kiflie (ኣለቃ ኪዳነ ወልድ ክፍሌ), the Ethiopic Calendar A.D. differs from other Christian calendars because of the continuity to these years after completion of the 5500 years and the former is religious while the latter is based on history. The Ethiopic years are seven years behind the Western and Eastern Church calendars. The seven years difference by Meskerem 1 or መስከረም ፩ becomes eight on January 1. Ethiopic uses the 5500 E.B.C. years in proleptic as well as modern calendrical calculations.

This says that the Ethiopic calendar is used "in proleptic" for years before incarnation, i.e. that calendar is used for a period of time where it did not exist. But the author also mentions "the continuity to these years after completion of the 5500 years" as a difference with other Christian calendars.
One does not see why defining an epoch, here 5500 years before 8 A.D., other than for a starting point. Scaliger had the same idea when he defined the "Julian Day 0" as Jan. 1, 4713 B.C.: taking a day very far in the past, he had the intention to count only forwards.


LIS Irrigation output log messages needed

Currently, LIS can only generate daily irrigation output (i.e., not monthly) and only runs if irrigation is written out when irrigation is turned on (selected on in the MODEL_OUTPUT table).
No alerts are provided to the user who may set such options. The following two situations require alerts in LIS to the users, who need to run with irrigation turned on:

When irrigation is turned on in the LIS config file but not activated in the MODEL OUTPUT table, LIS simply crashes with no message given. Conditional statements need to be added to let user know.

Currently, LIS will run with the "Irrigation output interval:" set to "1mo" (monthly interval), but two issues are encountered:  all 0's are written out and the month-date in the file naming convention is generated incorrectly. Also here, conditional statements are needed to ensure the user does not select this option, until a fix is implemented.

Finally, LIS irrigation documentation needs a couple of minor updates and additional details.

Check to see if this was ever fixed ...


latest version is broken: nagios_plugin3 dependency not found

When deploying the latest version (latest/edge rev 13 commit b43c26e) from charmhub the error below is encountered. Apparently there is a missing python file named nagios_plugin3, which I suspect to be [1].
[1] https://git.launchpad.net/nrpe-charm/plain/files/nagios_plugin3.py
2022-07-05 16:20:58 ERROR unit.openstack-service-checks/1.juju-log server.go:327 Hook error:
Traceback (most recent call last):
File "/var/lib/juju/agents/unit-openstack-service-checks-1/.venv/lib/python3.8/site-packages/charms/reactive/init.py", line 71, in main
bus.discover()
File "/var/lib/juju/agents/unit-openstack-service-checks-1/.venv/lib/python3.8/site-packages/charms/reactive/bus.py", line 426, in discover
_register_handlers_from_file(search_path, filepath)
File "/var/lib/juju/agents/unit-openstack-service-checks-1/.venv/lib/python3.8/site-packages/charms/reactive/bus.py", line 466, in _register_handlers_from_file
_load_module(root, filepath)
File "/var/lib/juju/agents/unit-openstack-service-checks-1/.venv/lib/python3.8/site-packages/charms/reactive/bus.py", line 444, in _load_module
return importlib.import_module(package + module)
File "/usr/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 671, in _load_unlocked
File "", line 848, in exec_module
File "", line 219, in _call_with_frames_removed
File "/var/lib/juju/agents/unit-openstack-service-checks-1/charm/reactive/openstack_service_checks.py", line 33, in
from lib_openstack_service_checks import (
File "/var/lib/juju/agents/unit-openstack-service-checks-1/charm/lib/lib_openstack_service_checks.py", line 21, in
from files.plugins.check_resources import RESOURCES, RESOURCES_BY_EXISTENCE
File "/var/lib/juju/agents/unit-openstack-service-checks-1/charm/files/plugins/check_resources.py", line 13, in
from nagios_plugin3 import CriticalError, UnknownError, WarnError, try_check
ModuleNotFoundError: No module named 'nagios_plugin3'
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60 Traceback (most recent call last):
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "/var/lib/juju/agents/unit-openstack-service-checks-1/charm/hooks/upgrade-charm", line 22, in
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60     main()
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "/var/lib/juju/agents/unit-openstack-service-checks-1/.venv/lib/python3.8/site-packages/charms/reactive/init.py", line 71, in main
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60     bus.discover()
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "/var/lib/juju/agents/unit-openstack-service-checks-1/.venv/lib/python3.8/site-packages/charms/reactive/bus.py", line 426, in discover
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60     _register_handlers_from_file(search_path, filepath)
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "/var/lib/juju/agents/unit-openstack-service-checks-1/.venv/lib/python3.8/site-packages/charms/reactive/bus.py", line 466, in _register_handlers_from_file
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60     _load_module(root, filepath)
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "/var/lib/juju/agents/unit-openstack-service-checks-1/.venv/lib/python3.8/site-packages/charms/reactive/bus.py", line 444, in _load_module
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60     return importlib.import_module(package + module)
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "/usr/lib/python3.8/importlib/init.py", line 127, in import_module
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60     return _bootstrap._gcd_import(name[level:], package, level)
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "", line 1014, in _gcd_import
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "", line 991, in _find_and_load
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "", line 975, in _find_and_load_unlocked
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "", line 671, in _load_unlocked
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "", line 848, in exec_module
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "", line 219, in _call_with_frames_removed
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "/var/lib/juju/agents/unit-openstack-service-checks-1/charm/reactive/openstack_service_checks.py", line 33, in
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60     from lib_openstack_service_checks import (
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "/var/lib/juju/agents/unit-openstack-service-checks-1/charm/lib/lib_openstack_service_checks.py", line 21, in
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60     from files.plugins.check_resources import RESOURCES, RESOURCES_BY_EXISTENCE
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60   File "/var/lib/juju/agents/unit-openstack-service-checks-1/charm/files/plugins/check_resources.py", line 13, in
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60     from nagios_plugin3 import CriticalError, UnknownError, WarnError, try_check
2022-07-05 16:20:58 WARNING unit.openstack-service-checks/1.upgrade-charm logger.go:60 ModuleNotFoundError: No module named 'nagios_plugin3'

(by rgildein)
I proposed the fix in MP 1.

(by rgildein)
I proposed the fix in MP 1.

(by vultaire)
Immediate workaround would be to install charm-nrpe as a subordinate.  There appears to be an undocumented cross-charm dependency; the charm-o-s-c code imports a module which is installed by charm-nrpe.

(by rodrigo-barbieri2010)
Paul, the immediate workaround has not worked for me unfortunately. That was the first thing I tried, and I even retried it today attempting to reinstall openstack-service-checks after having nrpe subordinate installed on the node. Despite the nagios_plugin3.py file being present in the VM, the error is still the same and wouldn't work.
root@juju-688852-fx-nagios-13:/var/log/juju# find / -name "nagios_plugin3.py"
/usr/lib/nagios/plugins/nagios_plugin3.py
/usr/local/lib/nagios/plugins/nagios_plugin3.py
/var/lib/juju/agents/unit-nrpe-6/charm/files/nagios_plugin3.py
/var/lib/juju/agents/unit-nrpe-6/charm/files/plugins/nagios_plugin3.py

(by vultaire)
Confirmed that the workaround does not work; the nrpe subordinate won't install until after certain hooks on the principal charm have fired.
While code which imports nagios_plugin3 has existed in the charm for many years, it was always in the context of the plugin modules in the files directory, which I don't believe would get fired until run via NRPE.  In other words, the plugin files depended on code in the NRPE charm, but they were only intended to be used in conjunction with that charm and not by the code in o-s-c.
Commit 9299fe70 introduced an import from the files/plugins directory, introducing the cross-dependency into the charm code proper and causing this breakage.
Long story short is, the breakage needs to be addressed in some way.


Preserve order in v1 RecordForm.contents.

I ran into this again the other day, and wondered if we'd placed it on the v2 roadmap. Nice!

For context, this is the conversation where it came up: https://github.com/scikit-hep/awkward-1.0/pull/1164#issuecomment-976964033.

For context, this is the conversation where it came up: #1164 (comment).

Ah, looks like I should have filed an issue at the time! Nice catch.


Improve local k8s developer experience

Compile all commands listed for kind setup into one make command via make kind-setup and updated docs accordingly and removes minikube setup as there is neglible chances that someone will use it over kind and we have to make our docs clean
Closes #1849

This is also a nit but checkpatch is detecting you have duplicate signature in your commit message. You can use git commit --amend to remove the unnecessary line. The commit should look like this (with empty lines):
<subsystem>: <title>

<body>

<signature>

updated . thank you @mtardy


Fix splitVertices to address issue #1268

Issue #1268 gives a corner case where splitVertices breaks, resulting in failure to build a very simple model.
The cases covered by splitVertices are quite varied, and the code is not simple, and it has not had problems in a long time, and we have a longer-term development effort to renovate how we handle graphical model relationships.  For these reasons, the intention here is as simple a fix as possible.
The details are as follows.  There is a maxVertexID determined early on in model building that attempts to be an upper bound on the possible number of graph vertices (i.e. nodes) needed for a model.  A vector of that length is then used to track how many scalar elements correspond to each vertex.  It turns out sometimes splitVertices results in skipping vertex IDs, with the result that a vertexID could end up larger than the maxVertexID, but it would be rare for it to happen (as in Issue #1268).  The maxVertexID logic wasn't wrong, but to strictly ensure that no vertexID is larger than it, we would have to go in and handle any cases where skipping occurs so that it would never occur.  This would be a pain and involve careful rethinking that would not be productive for code that is on the way out anyway.  Instead I have made a proposed fix that simply checks if an indexID gets larger than what is already being tracked and extends the tracking vector if necessary.  I think this is a minimal fix and we will see what testing turns up.
Note that in any case, the vertexIDs at issue are only temporary, as they are before topological sorting, so any skipped indices do get cleaned up at a later stage of model building.
I also have a slightly more involved additional fix that could reduce some of the index-skipping behavior, and I might like to see if it passes tests, but even if it does there might be a case for omitting it to stick to the absolute simplest fix right now, even inelegant as it is.

@perrydv test failure is unrelated. Shall we merge?


When applicant opens incomplete application, jump to first unanswered question

This will help the applicant feel like they're "picking up right where they left off."

I think this already happens

Great! Thanks. Will review.

You're right, it does that already


Timeout Issue - start

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Note: for build related issues you can open up an issue on the ionic-app-scripts repo
https://github.com/driftyco/ionic-app-scripts
Please ensure that you are on the latest version of the CLI.
npm view ionic@latest version
Short description of the problem:
Unable to create new project with sidemenu template.
What behavior are you expecting?
To create a sidemenu template app with name myApp using ionic start myApp sidemenu
Steps to reproduce:

install latest version if CLI. (Version 3.0.0)
try to create a new project using ionic start myApp sidemenu

ionic start myApp sidemenu
✔ Creating directory /Users/kumaresan/Desktop/untitled folder/myApp - done!
Fetching app base (https://github.com/driftyco/ionic2-app-base/archive/master.tar.gz)
✖ Downloading - failed!
Error: Timeout of 25000ms reached for https://github.com/driftyco/ionic2-app-base/archive/master.tar.gz

Post the output of ionic info below please
global packages:
@ionic/cli-utils : 1.0.0
Ionic CLI        : 3.0.0

System:
Node       : v7.10.0
OS         : macOS Sierra
Xcode      : Xcode 8.3.1 Build version 8E1000a
ios-deploy : 1.9.1
ios-sim    : 5.0.13

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)

Can you download the file https://github.com/driftyco/ionic2-app-base/archive/master.tar.gz manually?
How long does it take?
How fast is your internet?
Anything special about your internet connection or network (monitoring, proxy, cencorship...)?

@piotrowski
yes i could download the file manually. Download starts within milli seconds.
Speed is around 1.93 Mbps.
yes we have a dedicated firewall.

Would it be possible that you terminal can't access the internet because of this Firewall? Maybe try executing wget https://github.com/driftyco/ionic2-app-base/archive/master.tar.gz to check (although I don't know enoug about Mac to know if this would be enough as a test)

have you resolve this problem?  if you do ,teach me,thank you !

@RodgerLai not yet...

I also have the same problem

@kumaresan-cgvak  i uninstall ionic,then install again and create hello word successfully !

@RodgerLai  It doesn't solve the problem. Please team help us with the fix. I can help from my side to collect more information on this issue.

@kumaresan-cgvak @ramsundhar20 @vjunjoced I'm guessing the issue is due to your firewall. (Assuming you all have firewalls.) While curl and wget in your terminal may work, node programs such as npm and ionic need separate configuration for firewalls.
This is a known issue with the latest CLI and is being tracked here: https://github.com/driftyco/ionic-cli/issues/2234
Please update this issue if any of you are not using a firewall.

it was very strange for me... after upgrading CLI to 3.0.0 I also start facing this problem
but when I tried same command ionic start myProject blank from powershell then it worked!

@dwieeb No firewalls. I am working in my home in my personal laptop. I don't think ISP would be blocking github.

@ramonornela   uninstall and install ionic is not good resolution. run ':npm i -g ionic@legacy' ,that's working,thank you ! but how do you know the '@legacy' param.  teach me,thanks~

@RodgerLai https://github.com/driftyco/ionic-cli#legacy-version

@ramsundhar20 The new CLI has a timeout of 25 seconds. I'd like to debug this with you. Obviously using the legacy CLI is not a long-term solution. Can you answer these questions?

Can you download the file https://github.com/driftyco/ionic2-app-base/archive/master.tar.gz manually?
How long does it take?
How fast is your internet?
Anything special about your internet connection or network (monitoring, proxy, cencorship...)?

@ramsundhar20
Yes, the library we're using in the CLI for HTTP (superagent) follows redirects. How long does a typical ionic start take with the legacy (2.x) CLI for you?
Perhaps simply increasing the timeout will work for you for now. Your Github profile says you're in India. I think a better solution might be to offer a globally distributed CDN download to make this a better experience. I'm not sure if Github does this themselves or not.
Could you please paste the output of running $ ping codeland.github.com for a few seconds?

+1

@Kangmo Please elaborate.

This morning, I tried to create ionic app and chose 'conference' but failed to download the conference in 25s, which is the time limit with my cell phone which has 3Mbps bandwidth.
But now it successfully downloaded. Looks like it was server issue,  when the server is congested, looks like one can not download the file in 25 seconds.
https://github.com/driftyco/ionic-conference-app/archive/master.tar.gz
Thanks for the great tool~!~!~! I love ionic!

@Kangmo you're right. We have slow network connections here where I live in Brazil, then I can reproduce the issue perfectly.
I can start a tabs, blank or sidemenu app because it only has a few KB. When I try start a conference app, which download file has almost 6MB, I can see the download running through network but the timeout is reached because the internet speed is insufficient.
@dwieeb I think that the solution in this case is just increase the timeout.

Modify this code!!!!
path:C:\Users...\AppData\Roaming\npm\node_modules\ionic\dist\lib\start.js
Object.defineProperty(exports, "__esModule", { value: true });
const fs = require("fs");
const path = require("path");
const zlib = require("zlib");
const tar = require("tar");
const chalk = require("chalk");
const cli_utils_1 = require("@ionic/cli-utils");
function tarXvfFromUrl(url, destination) {
    return new Promise((resolve, reject) => {
        const archiveRequest = cli_utils_1.createRequest('get', url)
        #     **.timeout(25000)**
            .on('response', (res) => {
            if (res.statusCode !== 200) {
                reject(new Error(`Encountered bad status code (${res.statusCode}) for ${url}\n` +
                    `This could mean the server is experiencing difficulties right now--please try again later.\n\n` +
                    `If you're behind a firewall, you can proxy requests by using the HTTP_PROXY or IONIC_HTTP_PROXY environment variables.`));
            }
        })

@Kangmo @ricardo-mello @fssgh Thanks! I'll increase the timeout to 3 minutes for now, but I really want to look into speeding this up in the future.
I did not know the conference app was such a beast.
If you like, you can test with npm i -g ionic@canary.

I have the same problem but in ubuntu, I can not find the file :c
Windows be like:

path:C:\Users...\AppData\Roaming\npm\node_modules\ionic\dist\lib\start.js
Ubuntu be like:
? :c

@CarlosTheory please try to update ionic to latest version running npm i -g ionic@canary as @dwieeb said. Don't need to edit start.js manually.

@ricardo-mello I just read it :/ I edited the file in

/usr/lib/node_modules/ionic/dist/lib/start.js

It works, but now I have other problem it says npm stopped working

Disable you Firewall it will work ...

@kanoor Many people use Ionic for their work and have corporate firewalls that they cannot disable.

I solve just close the localhost chrome tab.

I am able to solve this problem by creating a new folder for the project and running the start command with template name and not by selecting the template.
eg: ionic start mynewproject sidemenu

npm i -g ionic@canary
great， fix it 。txh


DL4J: Replace Old*Op instances with new equivalents

DL4J has a number of layers etc using OldSubOp, OldMulOp, OldDivOp, etc.
The new equivalents for these should be drop-in replacements, no other changes required (same behaviour).

https://github.com/SkymindIO/deeplearning4j/pull/110


Create a struct for EmpiricalHistogram

Issue: https://github.com/JuliaEarth/GeoStats.jl/issues/157

Codecov Report

Merging #77 (843e0a7) into master (e06933f) will decrease coverage by 22.35%.
The diff coverage is 10.00%.

@@             Coverage Diff             @@
##           master      #77       +/-   ##
===========================================
- Coverage   91.47%   69.11%   -22.36%
===========================================
  Files         101      103        +2
  Lines        1841     1865       +24
===========================================
- Hits         1684     1289      -395
- Misses        157      576      +419

Impacted Files
Coverage Δ

src/GeoStatsBase.jl
100.00% <ø> (ø)

src/plotrecipes/histograms.jl
0.00% <0.00%> (ø)

src/statistics/data.jl
100.00% <ø> (+9.37%)
:arrow_up:

src/histograms.jl
60.00% <60.00%> (ø)

src/plotrecipes/distplot2D.jl
0.00% <0.00%> (-97.73%)
:arrow_down:

src/plotrecipes/domains/structured_grid.jl
0.00% <0.00%> (-97.15%)
:arrow_down:

src/plotrecipes/distplot1D.jl
0.00% <0.00%> (-96.00%)
:arrow_down:

src/plotrecipes/hscatter.jl
0.00% <0.00%> (-95.24%)
:arrow_down:

src/plotrecipes/ensembles.jl
0.00% <0.00%> (-95.24%)
:arrow_down:

src/plotrecipes/cornerplot.jl
0.00% <0.00%> (-94.45%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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

Thank you @juliohm for reviewing this PR!
I've made changes based on your review and checked with the following code
using  GeoStatsBase
using Plots

sdata = georef((z=rand(310),))
eh = EmpiricalHistogram(sdata, :z; nbins=10)
plot(eh)

It works

Adding the missing constructor gives the following warning
[ Info: Precompiling GeoStatsBase [323cb8eb-fbf6-51c0-afd0-f8fba70507b2]
WARNING: Method definition (::Type{GeoStatsBase.EmpiricalHistogram{H} where H})(Any) in module GeoStatsBase at C:\Users\Threxys\.julia\dev\GeoStatsBase\src\histograms.jl:15 overwritten at C:\Users\Threxys\.julia\dev\GeoStatsBase\src\histograms.jl:23.
  ** incremental compilation may be fatally broken for this module **

@cyborg1995 that is awesome work, thanks for the quick improvements. Whenever you feel that the PR is ready for review, you can request in the top right a PR from myself. I will be notified by email accordingly.

Now we need some tests for the new histogram plot. Can you follow the pattern in the test suite and add visual regression tests with the @test_ref_plot macro defined in runtests.jl? There are a couple examples of usage already.

Sure, I'll do it!

@juliohm, just to confirm, I have to replace the .png files with the new plots right?

Yes, the ones you're adding and the ones that changed from the previous
version according to your modifications. :)
On Thu, Feb 11, 2021, 06:13 Gaurav Wasnik<EMAIL_ADDRESS>wrote:

@juliohm https://github.com/juliohm, just to confirm, I have to replace
the .png files with the new plots right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JuliaEarth/GeoStatsBase.jl/pull/77#issuecomment-777299249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZQW3MLOLVABZJDSA6FVOTS6ONSRANCNFSM4XLGFVLQ
.

What are the new ones that I'm supposed to add?

The new EmipiricalHistogram plot with plot(hist) :)
On Thu, Feb 11, 2021, 07:08 Gaurav Wasnik<EMAIL_ADDRESS>wrote:

What are the new ones that I'm supposed to add?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JuliaEarth/GeoStatsBase.jl/pull/77#issuecomment-777333211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZQW3O5ML7IJBXSUWR3H73S6OUDVANCNFSM4XLGFVLQ
.

The cornerplot and distplot1d are showing the histograms with bars as opposed to steps. Can you double check why this is happening? These two plots shouldn't change.

@juliohm, is everything alright here?

The empirical histogram plots you added are still showing bars instead of steps. Can you double check?

@juliohm, I'm not able to figure out what is causing that. Could you please look into it?
The distplot1D was missing
    seriestype --> :step
    seriescolor --> :black

@cyborg1995 where are the changes to the distplot1d you had before? They disappeared in the PR?

@juliohm, apologies for the late reply, I'll look into it again :)

How did that happen? I have no idea why the changes disappeared, probably that is causing problems. I'll fix it.

@juliohm after checking the previous commits, it seems that there were no changes made to distplot1d. Some modifications were made only to be restored to the original later.

Will try to take a look soon in this PR. Thanks for pushing it further.
On Sat, Feb 13, 2021, 07:15 Gaurav Wasnik<EMAIL_ADDRESS>wrote:

@juliohm https://github.com/juliohm after checking the previous
commits, it seems that there were no changes made to distplot1d. Some
modifications were made only to be restored to the original later.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JuliaEarth/GeoStatsBase.jl/pull/77#issuecomment-778594893,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZQW3MF6LBAIEHVC5F3EFLS6ZGKNANCNFSM4XLGFVLQ
.

@cyborg1995 I have fixed the issues locally, did you check the box of this PR that allows editions by maintainers? I can't push to your fork to update the PR.

Nevermind, I was using the wrong git push command. I've updated the code and now we just need to update the figures by running the tests in your computer. I am currently on Julia v1.6-beta1 and the figures have changed a bit which will cause the tests on GitHub to fail.
Could you please update the figures and push them to the PR? :)

@juliohm Thank you so much for fixing the issue! I'm trying to understand from your solution what went wrong.
I have updated the figures, this PR will be ready to be merged once the test completes! :)

The issue was partially caused by the fact that the old cold with EmpiricalHistogram was returning a simple fit(Histogram, ...) I wrapped this result into a EmpiricalHistogram(fit(Histogram, ...)) and the plot recipe was triggered correctly.
Additionally I added a new Base.values implementation that returns the abscissa and ordinates of the fitted histogram.
Thanks for fixing the issue. Looking forward to merge :)

Thank you @cyborg1995 for these great contributions ❤

Thank you @juliohm!
I'm all set to tackle another issue!


Lost my braket twirling test

Some how I lost my twirling test for braket.  Need to add it back

Closed by #32


Add BERT example notebook.

Update README to point to new notebook. Minor README formatting changes.

View the notebook here: https://github.com/dellis23/iree-torch/blob/5e4c29fcfb404b1de31e553572290d36223983ce/examples/bert.ipynb
@silvasean Do you have a quick explanation I can include for why we need to create the wrapped model?  I'll update and remove the TODO if so.

PyTorch models typically produce multiple outputs, some of them are only used during training, some are only important for statistics tracking. For this model, only the next-token "logits" (which are basically just the next-token probabilities) are relevant.

FYI, we usually squash all commits (and tidy up the resulting commit message) before merging to avoid local development steps from showing up in the mainline history.

FYI, we usually squash all commits (and tidy up the resulting commit message) before merging to avoid local development steps from showing up in the mainline history.

Ha, yeah I just messaged you about that.  It looks like squash and merge is disabled on this repository.  I know I can do it manually, but I think using the github button will retain the history in the PR and correctly squash in main.

The github button naively squashes the commit messages. We don't want intermediate messages like use_traced -> use_tracing in the mainline history though, since they don't actually describe the final patch itself.

Ah, I see.  I thought it let you modify the messages.  Thanks, I'll do it via the command line.


Doesn't include declaration file (.d.ts) while compilation

When I have project-scoped declaration file(*.d.ts) in my project, packaging with serverless-plugin-typescript emits compile error like below:
Cannot find name 'TypeInterfaceName'.

(But regardless of the error, packaging is completed successfully and proceed to deployment process.)
So currently I have to write all the declaration files to normal typescript file, and import them to every functions now.
It'd be better to support them :)

+1

has any maintainer looked into this issue? because this is very important one.

Definitely a problem for me!

Also hit this one. My workaround is to include the declaration in the tsconfig.json types property. Path seems to need to be relative to the build folder.
e.g. with a declaration defined in ./src/types/serverless-http.d.ts specify:
...
"types": [
  "../src/types/serverless-http"
],
...

My original workaround caused a few issues with my IDE. But I found if I package my local declarations differently I could use typeRoots in tsconfig.json instead.
e.g. Originally I had defined my types as a single file src/types/serverless-http.d.ts. Instead I repackaged it to be src/types/serverless-http/index.d.ts.
And changed my tsconfig.json
"typeRoots": [
  "node_modules/@types",
  "src/types"
],

What also works is using old-school <reference> comments:
/// <reference path='../types/module.d.ts' />

I hit this one as well, but struggled with VS Code not recognizing the definitions if I defined the path as "../external-type-definition" (relative to the build folder). My solution was to define the following in tsconfig.json:
"typeRoots": [
      "./",
      ".build/"
    ],
    "types": [
      "../external-types/serverless-http"
    ],
}

My original workaround caused a few issues with my IDE. But I found if I package my local declarations differently I could use typeRoots in tsconfig.json instead.
e.g. Originally I had defined my types as a single file src/types/serverless-http.d.ts. Instead I repackaged it to be src/types/serverless-http/index.d.ts.
And changed my tsconfig.json
"typeRoots": [
  "node_modules/@types",
  "src/types"
],

Only this worked for me.
I had a d.ts file with a name like: myservice.gql.d.ts in a types folder, but nothing worked.
Then I just renamed the path to: myservice/index.d.ts, and voila!
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>I can't really wrap my head around what makes the plugin ignore the d.ts files, as my IDE worked fine, and when I ran ts compilation directly (npx tsc --noEmit) that picked up the d.ts file too without the whole index.d.ts rework. :man_shrugging:

I encountered this issue today.  None of the fixed I found here or elsewhere worked.  I've tried:

using typeRoots and types
specifically referencing my index.d.ts in files, include, and both
Making a @types folder in /src with a package.json with types set to index.d.ts
Putting index.d.ts at various levels in the project

Like others have observed, the project does compile and IDE intellisense does work.  I can successfully run tsc, use ts-node to run the .ts files, run mocha with ts-node...but I would like to understand and fix why I get Cannot find name 'X' when I use the Serverless package, deploy, and invoke-local functions.

I'm also annoyed by this. I have an index.d.ts in my project and a functional workaround seems to be to add this as the first line in all my .ts files (might need a different relative path if your files are not all in the same dir with the type file):
/// <reference path='index.d.ts' /> workaround for https://github.com/serverless/serverless-plugin-typescript/issues/92


Allow PHP 8

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Related tickets

Documentation

License
MIT

What's in this PR?
Allow this package to be installed on PHP 8.
Checklist

[ ] Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix
[ ] Documentation pull request created (if not simply a bugfix)

Let's remove php-cs-fixer from composer.json and use a github action or phar instead.

What about StyleCI: does that already validate against PHPCS?

https://github.com/php-http/vcr-plugin/releases/tag/1.2.0


Allow URLs in targets defined via a JSON file

This enables defining blackbox_exporter targets (which can be URLs,
because of relabeling) in a JSON file.
Not sure if this is the best approach, but current behaviour is
inconsistent (UnmarshalYAML does not have this check) and breaks
officially documented way to use blackbox_exporter.

This is a holdover from when addresses used to be full URLs, it's a safety check.
It's probably safe to remove by now, however if you're using JSON files then you can set things up not to need any relabelling.

Hu, I remember doing a similar change to enable the blackbox prober in the past, no?

That change was just for target groups in the config file, this should be safe to put in by now as it's a long time since instance labels were URLs.

Okay, if it makes things work 👍


[vm] List.insertAll can leave unsound null elements when interrupted

If List.insertAll is interrupted by an exception, it can leave the modified list in an inconsistent state where space has been made for the new elements, but they have not (all) been written yet. This can cause a list with a non-nullable type to contain null elements.
The situation can be triggered by inserting a list into itself:
main() {
  List<int> list = [1, 2];
  try {
    list.insertAll(1, list);
  } catch (_) {}
  print(list.length);
  print(list[2]);
}

or if the inserted list throws on access:
import 'dart:collection';

class BadList extends ListBase<int> {
  int get length => 2;
  int operator [](int index) => throw "BOOM";

  set length(int l) => throw "unimplemented";
  void operator []=(int index, int value) => throw "unimplemented";
  void add(int e) => throw "unimplemented";
}

main() {
  List<int> list = [1, 2];
  try {
    list.insertAll(1, BadList());
  } catch (_) {}
  print(list.length);
  print(list[2]);
}

Both of these print:
4
null

Producing a segfault is left as an exercise to the reader.
The offending code is:
https://github.com/dart-lang/sdk/blob/e4e7d31ec7e89c940dca88327638928fd5d20e1f/sdk/lib/_internal/vm/lib/growable_array.dart#L42-L65
Judging by the reference implementation in https://github.com/dart-lang/sdk/blob/e4e7d31ec7e89c940dca88327638928fd5d20e1f/sdk/lib/collection/list.dart, the first example is supposed to work as expected (it also works in dart2js). This can be fixed by also creating the temporary list (the iterable = iterable.toList();) when the inserted list is identical to this.
For fixing the second case of a throwing iterable, there are two approaches we could take:

Create the temporary list unless the iterable is of a known, non-throwing type. Use _List.of(iterable) instead of iterable.toList() to make sure that the new list is also non-throwing.
Keep the optimistic avoidance of the extra copy, but wrap the setAll in a try / finally which restores the original list on error.

I prefer option (1) since it is obviously atomic. In most cases the temporary array is ephemeral, and code following the copy is less polymorphic. I think for JavaScript we would copy everything that is not a JavaScript Array or typed array.
It is not clear that self-insertion is supposed to work. In the reference implementation, a.insertAll(x, a.length) uses addAll and will fail because list.addAll(list) fails.
The documentation for insertAll is silent on the self-insertion issue.

[ ] Test for the unsound case
[ ] Decision on intended behaviour with self-insertion @lrhn
[ ] Test for self-insertion
[ ] VM fix
[ ] DDC fix
[ ] dart2js fix - https://github.com/dart-lang/sdk/issues/48134

Another option is to pre-fill the increased length with, say, the last value of the original list. (If there are not existing value, then just do an addAll instead.)
It seems we check for efficient length, then convert to List if there is none.
Yet another approach is to do:
void insertAll(int index, Iterable<E> iterable) {
  // check index valid
  var originalLength = this.length;
  this.addAll(iterable);
  var delta = this.length - originalLength;
  this
    ..reverseRange(index, oldLength)
    ..reverseRange(oldLength, this.length)
    ..reverseRange(index, this.length);
}

That's one store and two moves for each element in the iterable,
and two moves for every element from index to original length of the target,
and if iterable throws, the intermediate step is going to be completely weird.
Alternatively, catch the error and adjust the result to only contain the elements iterated so far:
   _setLength(newLength);
   setRange(index + insertionLength, this.length, this, index);
   var i = 0;
   try {
     for (var element in iterable) {
       i++;
       if (i >= insertionLength) throw ConcurrentModificationError(iterable);
       this[index + i] = element;
     }
  } catch e {
    if (i < insertionLength) {
      setRange(index + i, this.length - (index + insertionLength), this, index + insertionLength);
      _setLength(this.length - insertionLength + i);
    }
    rethrow;
  }

(This also handles a too-long iterator, one which has more elements than its length says.)

We have to change insertAll to fix soundness.
I think we should take this opportunity to make the implementations consistent, so programs developed on one platform work the same on another.
I would expect more consistency between dart2js and VM, and between the system List and WrapList.
import 'dart:collection';

main() {
  test([], 0);
  test([11], 0);
  test([11], 1);
  test([11, 22], 0);
  test([11, 22], 1);
  test([11, 22], 2);

  test(WrapList([]), 0);
  test(WrapList([11]), 0);
  test(WrapList([11]), 1);
  test(WrapList([11, 22]), 0);
  test(WrapList([11, 22]), 1);
  test(WrapList([11, 22]), 2);
}

test(List a, int index) {
  print('a: $a, index: $index');
  try {
    a.insertAll(index, a);
    print('a: $a');
  } catch (e, s) {
    print('$e');
    print('a: $a');
    print("$s".split('\n').take(5).join('\n'));
  }
  print('');
}

class WrapList<E> extends ListBase<E> {
  final _list;
  WrapList(this._list);

  int get length => _list.length;
  void set length(int len) => _list.length = len;
  E operator[](int i) => _list[i];
  void operator[]=(int i, E value) => _list[i] = value;
  void add(E value) => _list.add(value);

  String toString() => 'WrapList($_list)';
}

dart2js:
a: [], index: 0
a: []

a: [11], index: 0
a: [11, 11]

a: [11], index: 1
a: [11, 11]

a: [11, 22], index: 0
a: [11, 22, 11, 22]

a: [11, 22], index: 1
a: [11, 11, 22, 22]

a: [11, 22], index: 2
a: [11, 22, 11, 22]

a: WrapList([]), index: 0
a: WrapList([])

a: WrapList([11]), index: 0
a: WrapList([11, 11])

a: WrapList([11]), index: 1
Concurrent modification during iteration: Instance of 'WrapList<dynamic>'.
a: WrapList([11, 11])
Concurrent modification during iteration: Instance of 'WrapList<dynamic>'.
    at Object.wrapException (o2.js:361:17)
    at ListIterator.moveNext$0 (o2.js:3155:17)
    at WrapList.addAll$1 (o2.js:3330:71)
    at WrapList.insertAll$2 (o2.js:3375:15)

a: WrapList([11, 22]), index: 0
a: WrapList([11, 22, 11, 22])

a: WrapList([11, 22]), index: 1
a: WrapList([11, 11, 22, 22])

a: WrapList([11, 22]), index: 2
Concurrent modification during iteration: Instance of 'WrapList<dynamic>'.
a: WrapList([11, 22, 11])
Concurrent modification during iteration: Instance of 'WrapList<dynamic>'.
    at Object.wrapException (o2.js:361:17)
    at ListIterator.moveNext$0 (o2.js:3155:17)
    at WrapList.addAll$1 (o2.js:3330:71)
    at WrapList.insertAll$2 (o2.js:3375:15)

VM:
a: [], index: 0
a: []

a: [11], index: 0
a: [11, 11]

a: [11], index: 1
RangeError (end): Invalid value: Not in inclusive range 1..2: 3
a: [11, null]
#0      RangeError.checkValidRange (dart:core/errors.dart:356:9)
#1      ListMixin.setRange (dart:collection/list.dart:415:16)
#2      List.setAll (dart:core-patch/growable_array.dart:69:7)
#3      List.insertAll (dart:core-patch/growable_array.dart:64:5)
#4      test (file:///usr/local/google/home/sra/play1/bug1c3h.dart:22:7)

a: [11, 22], index: 0
a: [11, 22, 11, 22]

a: [11, 22], index: 1
RangeError (end): Invalid value: Not in inclusive range 1..4: 5
a: [11, 22, null, 22]
#0      RangeError.checkValidRange (dart:core/errors.dart:356:9)
#1      ListMixin.setRange (dart:collection/list.dart:415:16)
#2      List.setAll (dart:core-patch/growable_array.dart:69:7)
#3      List.insertAll (dart:core-patch/growable_array.dart:64:5)
#4      test (file:///usr/local/google/home/sra/play1/bug1c3h.dart:22:7)

a: [11, 22], index: 2
RangeError (end): Invalid value: Not in inclusive range 2..4: 6
a: [11, 22, null, null]
#0      RangeError.checkValidRange (dart:core/errors.dart:356:9)
#1      ListMixin.setRange (dart:collection/list.dart:415:16)
#2      List.setAll (dart:core-patch/growable_array.dart:69:7)
#3      List.insertAll (dart:core-patch/growable_array.dart:64:5)
#4      test (file:///usr/local/google/home/sra/play1/bug1c3h.dart:22:7)

a: WrapList([]), index: 0
a: WrapList([])

a: WrapList([11]), index: 0
a: WrapList([11, 11])

a: WrapList([11]), index: 1
Concurrent modification during iteration: Instance of 'WrapList<dynamic>'.
a: WrapList([11, 11])
#0      ListIterator.moveNext (dart:_internal/iterable.dart:336:7)
#1      ListMixin.addAll (dart:collection/list.dart:284:23)
#2      ListMixin.insertAll (dart:collection/list.dart:546:7)
#3      test (file:///usr/local/google/home/sra/play1/bug1c3h.dart:22:7)
#4      main (file:///usr/local/google/home/sra/play1/bug1c3h.dart:13:3)

a: WrapList([11, 22]), index: 0
a: WrapList([11, 22, 11, 22])

a: WrapList([11, 22]), index: 1
a: WrapList([11, 11, 22, 22])

a: WrapList([11, 22]), index: 2
Concurrent modification during iteration: Instance of 'WrapList<dynamic>'.
a: WrapList([11, 22, 11])
#0      ListIterator.moveNext (dart:_internal/iterable.dart:336:7)
#1      ListMixin.addAll (dart:collection/list.dart:284:23)
#2      ListMixin.insertAll (dart:collection/list.dart:546:7)
#3      test (file:///usr/local/google/home/sra/play1/bug1c3h.dart:22:7)
#4      main (file:///usr/local/google/home/sra/play1/bug1c3h.dart:16:3)

I'm not sure I care whether error behavior is compatible between platforms. If your iterator throws, the result is unspecified.
It just has to be sound, there is no "correct". So, no leaving null values visible.
(Obviosuly, we can go the extra mile and make it consistent with inserting each element, one by one, and stopping when the iterator throws. As long as it doesn't cost us much, it's "nice to have".)
There are other errors which can occur if the iterable changes length (or if its length doesn't match the number of elements of its iterator, which requires some very bad, and likely deliberate, coding to happen).
I'm writing a test :grin:.

I prefer option (1) since it is obviously atomic. In most cases the temporary array is ephemeral, and code following the copy is less polymorphic. I think for JavaScript we would copy everything that is not a JavaScript Array or typed array.

These are very good arguments. I like the cleanness of giving insertAll the semantics that it (conceptually) first extracts all elements from the iterator, and then inserts them into the list. It's easy to understand and unambiguously defines all corner cases, such as self-insertion, length mismatches and exceptions.
Thus, the pre-copy code would look something like this:
if (!isInternalList(iterable) && iterable is! TypedData || identical(iterable, this)) {
  iterable = List.of(iterable, growable: false);
}

where isInternalList is a target-dependent check. We could opt to add such a function to dart:_internal to make this check also available to the target-independent reference implementation.

You also wouldn't need to copy an internal Set, internal Map's keys or values iterator or even an internal Queue, because we know accessing them cannot have side-effects, they have efficient length and consistent iteration.
Whether it's worth the effort is a different question. I don't know how often anyone inserts a set into a list (addAll is more likely).

I'm not sure I care whether error behavior is compatible between platforms. If your iterator throws, the result is unspecified.
It just has to be sound, there is no "correct". So, no leaving null values visible.

@lrhn The point I am most concerned about is behaviour that is an error on one platform and succeeds on another. Hopefully that is included in your :smiling_imp: test.

I'm writing a test :smiling_imp:.

Assigned to you to reflect this status.


main() method that returns a Future should be guaranteed to be awaited

Use case:
import 'dart:async';

Future<Null> main() async {
  try {
    print('before');
    await new Completer().future;
    print('after');
  } catch (_) {
    print('error');
  } finally {
    print('finally');
  }
}

This produces the following output:
before

... and then the process completes successfully (exit code 0).
And yet... if you add a timeout, like so:
await new Completer().future.timeout(const Duration(seconds: 2));

... then it prints before, then waits 2 seconds, and prints error and finally.
This is highly counterintuitive! Can the VM guarantee that the future returned by main() is always awaited?

@cbracken @goderbauer

The first future, await new Completer().future;, will never be completed. Duplicate of https://github.com/dart-lang/sdk/issues/23797

What do you think should happen with this code?:
import 'dart:async';

main() {
  print('before');
  var f = new Completer().future;
  print('after');
  return f;
}

I think the process shouldn't die until it's killed via a signal.

I actually think Hixie's example should terminate, and the one using await shouldn't.
The two examples are similar, and will have the same runtime behavior up to the point where no further computation will happen. The only question is what happens then.
The difference between the two is that the async/await example looks like straight-line code, and users are likely to expect it to work that way. That makes "hanging" more understandable than exiting and skipping the finally block.
For the whenComplete example, it's clear that control has run to the end of the main function. You can't wait in a non-async method, so when main completes, you are waiting for callbacks, which may or may not happen.
So, it's all about user perception. You expect straight-line code to either throw, complete or hang. Just exiting in the middle isn't even considered an option, which is why we get bug reports like this when it happens anyway. For non-await code, the user won't think they are in the middle of it, so there is no perception conflict.
So, I still suggest that the VM should hang if an isolate runs out of things to execute while waiting at an await. (You should still be able to kill the isolate explicitly in that case, if that's what you want). It should be possible to implement this using a single counter that is incremented on await and decremented when the await completes, and checked in the "about to exit" code.
It is not practically possible to hang the program if there is any uncompleted future. That happens far too easily, and such futures are just ignored and nobody notices. That's a good thing.
Looking at a future returned by main is a red herring. There are many ways to have a program run asynchronously without returning a future from main.

That makes perfect sense - I'm sold.

FWIW, I frequently change code from using async to using explicit features and back. I think it would be very confusing if the semantics were different. I also think it'll be confusing that this code will hang or not hang depending on the implementation details of the function foo():
void main() {
  foo().whenComplete(() { print('done'); });
}

// if this is the implementation, the program terminates
Future foo() {
  return new Completer().future;
}

// if this is the implementation, the program hangs
Future foo() async {
  return await new Completer().future;
}

It might be slightly confusing that the program hangs in one case and terminates in the other, but nothing near as confusing as a program terminating in the middle of a try/finally.
I would hope that the observatory could point you to the await that is hanging. That should aid debugging much more than just terminating.
That said, it's a change, and I don't know if there is someone out there who is using an () async { handle(await somethingThatMightHappen); }() as a kind of single-event handler, and those programs would start hanging when "something" didn't happen anyway. It's valid code now, so I can't rule it out.

I don't know if there is someone out there who is using ...

Philosophical question: if 99 developers cheer this change, and 1 curses it, did we do the right thing?


Support floating point values for charge_current_limit

The value charge_current_limit (channel Parameter.Inverter.AcALim) is represented as an integer. The EV Charger accepts floating point numbers for this parameter.
Some solutions for optimized charging (using solar power), that use this value in combination with the "fast_charge" mode to control the charging power as an alternative to using SMA's own Sunny Home Manager and the physical dial on the charger. For those use cases, having a finer-grained control over the charging current might be beneficial. Could you change the number format to floating point?
I'm quite new to Home Assistant, so unfortunately I cannot easily create a pull request myself. Thanks a lot!

Should work with version 1.2.0.


Add MacOS setup steps to readme

Had a few users asking about this so might be nice to add to readme. Here are the steps from dilden
Make sure you have XCode and Homebrew (https://mac.install.guide/homebrew/2.html)
Make sure you have the latest version of node, I have: v16.15.0
If you update, close and restart terminal.
node -v
brew install node
brew install git
sudo npm install -g yarn
git clone https://github.com/zer0-os/zChain.git
sh install.sh
cd zChain
cd packages/meow-cli
sudo npm link --force
cd ../..
meow sandbox

Thanks @tomdickharryeth i have added steps for mac installation in README. Few points:

you shouldn't use brew for nodejs installation on macs. Use nvm
avoing using sudo npm and reinstall node with non-sudo priveleges.


HTML5?

Does pdfkit support HTML5?
I'm trying to use some HTML5 CSS features and they seem to be ignored when I generate a PDF from the HTML + CSS.
If needed, I can provide examples.

We support HTML5 if and only if wkHTMLtoPDF supports it.
If you find that wkHTMLtoPDF supports it and you want to contribute some notes that it does support it (and if necessary, how to enable that), feel free to do so!

Otherwise, I'll close this in a couple weeks if we don't hear from you. :)

Go ahead and close it. If I find anything out, I'll let you know.
Chuck
On Mon, Jul 11, 2016 at 7:02 AM, Ian Cordasco<EMAIL_ADDRESS>wrote:

Otherwise, I'll close this in a couple weeks if we don't hear from you. :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pdfkit/pdfkit/issues/366#issuecomment-231704636, or mute
the thread
https://github.com/notifications/unsubscribe/AOJjWTFCVc6P_bHZDMleMRDAk_pN20vwks5qUiK0gaJpZM4JJQE1
.

--
Chuck Williams, Software Engineer
Metova Federal, LLC
(703) 338-5162 | www.federal.metova.com


Metric Bucket Transformer

Transform metrics to attributes to explain metrics using correlated other metrics. Buckets floating point value by percentiles, converting them to string attributes that are tagged with either the bucket range or the bucket index.
See MetricAsExplanationTest for complete usage. Using smaller buckets allows more precise explanations with higher risk ratio.

Coverage remained the same at 94.18% when pulling c7a2e8e8ac8012a099f8708c0e16c68c480e2ad6 on metricexplain into 574bb6fbe9813d393bd8f128e00c5d78845b0430 on master.

Coverage remained the same at 94.18% when pulling 738c78f5ebfcbe1fd56745ba477132027dcb7e4e on metricexplain into 574bb6fbe9813d393bd8f128e00c5d78845b0430 on master.

Good to go @edgan8

Coverage remained the same at 94.18% when pulling adc93a48e5e0143e58a7259cfa2bffb1f4245c46 on metricexplain into 543cfadc6213dff2d78a5385ce0d9b83fe5a42f7 on master.


ci: add secrets, using env vars defined in the github environment 'SecondLife_1'

ci: add secrets, using env vars in the github environment 'SecondLife_1'

how is it that the moment I merge this to main, the 'license activation strategy' issue is fixed... But the file changed is nothing related to license activation...?

ah i see:
in
https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions
there is this statement
Workflows triggered using the pull_request event have read-only permissions and have no access to secrets
looks like other people can open PR, but they will not be able to run the unity test runner or builder...


Support web standards:  routes like "users/sign_in" should be "users/sign-in"

This is not a bug that's breaking anything, but it's generally recognized that URL routes should be constructed using hyphens, rather than underscores.
Current behavior
GET users/sign_in
Expected behavior
GET users/sign-in

Devise's default URL behaviour confused other developers when consuming our API, who are used to having all URL routes delineated by hyphens rather than underscore.
Since changing devise's default URL routes to hyphens now from underscores could lead to many breaking API changes, I'm wondering if it would make sense to have devise alias both "sign_in" and "sign-in" as valid URL routes for devise to standardize with commonly accepted practice.
Thoughts?

@KelseyDH You're right, this is a breaking change, so I suggest you override these routes paths in your application for now, as suggested here: https://github.com/plataformatec/devise/wiki/How-To:-Change-Default-Sign_up---Registration-Path-with-Custom-Path
Thank you.


docs(demos): add table column management demo

fixes https://github.com/patternfly/patternfly-next/issues/2626
they're the last demos under /demos/table

Preview: https://patternfly-next-pr-2756.surge.sh

this looks great @mcoker!!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: