Expose an API to react-server-cli's startServer.

Implements a simple export of the startServer API, as proposed in #33 and #50, so that servers can be started up programmatically.
I think it's probably also a good idea to expose the client compilation as an API to allow more flexibility, but I believe that's orthogonal to this PR.

LGTM

This is cool.


Fix GET /incoming-payments list

probably broken by recent changes to wallet address middleware that not longer sets the wallet address from query params (but should in this case). https://github.com/interledger/rafiki/pull/2115

partially fixed in https://github.com/interledger/rafiki/pull/2115#pullrequestreview-1697892764


data-sizes="auto" doesn't work in Chrome

Hello I'm using the autosizing and its works in Safari correctly, but in Chrome strangely doesn't not work.
My images are something like this:
<img src="data:image/gif;base64,R0lGODlhAQABAIAAAP///wAAACH5BAEAAAAALAAAAAABAAEAAAICRAEAOw=="
    data-src="image-1608x.jpg"
    data-sizes="auto"
    data-srcset="image-282x.jpg 282w, image-324x.jpg 324w, image-402x.jpg 402w, image-804x.jpg 804w, image-564x.jpg 564w, image-648x.jpg 648w, image-804x.jpg 804w, image-1608x.jpg 1608w"
    class="lazyload">

I know that may I'm doing something wrong, but I don't find any solution.

@rtorresn10
I can not use your bug report. Please always provide a test case. Especially data-sizes="auto" relies on the computed styles and therefore some HTML code alone doesn't help.
In most cases the readme helps!

@aFarkas Thanks for your response!
I made this example for you:
https://glitch.com/edit/#!/stormy-receipt?path=index.html%3A21%3A205

@aFarkas Thanks for you response.
I made this example for you:
https://glitch.com/edit/#!/stormy-receipt?path=index.html%3A38%3A10
In Chrome only take one image and when you resize the window the image don't change.
But in safari it works fine.


Fix cron expression get a wrong previous fire time

For the method CronExpression#getPrevFireTime, currently only the minimum increment within a cycle is considered, and the cross-cycle situation is not involved, resulting in differences in the minimum increment obtained. The current commit is to fix this bug.
I tried to create a Jira account to submit a bug, but so far no response has been received

You can now create an issue in GitHub for this. We have a new changelog format so you can also add a changelog entry that references that issue.


Make bot speak slower

Use SSML: https://learn.microsoft.com/en-us/azure/communication-services/how-tos/call-automation/play-action?pivots=programming-language-python#play-source---text-to-speech-with-ssml.

Done in c6634f8764643f8ef9a439cc7a86664bd3454bca.


[Style] Add a withWidth HOC

@nathanmarks I have noticed one regression linked to your previous fix:

@newoga I have changed the stateless functionnal component organisation because of https://github.com/yannickcr/eslint-plugin-react/pull/567.
Do you think I should be writing more tests for this PR?
I had to add one feature to the EventListener component.
Deprecate the styleResizable mixin and remove it from the documentation. That's was our last mixin #RIP. Now, we should be able to enforce react/prefer-es6-classes.
Remove usage of material-ui/lib from the documentation.

[Current coverage][cc-pull] is 38.87%

Merging [#4126][cc-pull] into [master][cc-base-branch] will decrease coverage by -0.00%

@@             master      #4126   diff @@
==========================================
  Files           137        139     +2
  Lines          4373       4404    +31
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1701       1712    +11
- Misses         2672       2692    +20
  Partials          0          0

File ...ls/styleResizable.js (not in diff) was modified. more

Misses +3
Partials 0
Hits -3

Powered by Codecov. Last updated by [273a769...da78969][cc-compare]
[cc-base-branch]: https://codecov.io/gh/callemall/material-ui/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/callemall/material-ui/compare/273a769c84fc3cf1377c6168efdb6bbd1245c208...da78969516ac7d6f21eb3d41ea39eef1b3ecb1eb
[cc-pull]: https://codecov.io/gh/callemall/material-ui/pull/4126?src=pr

[Current coverage][cc-pull] is 38.87%

Merging [#4126][cc-pull] into [master][cc-base-branch] will decrease coverage by -0.00%

@@             master      #4126   diff @@
==========================================
  Files           137        139     +2
  Lines          4373       4404    +31
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1701       1712    +11
- Misses         2672       2692    +20
  Partials          0          0

File ...ls/styleResizable.js (not in diff) was modified. more

Misses +3
Partials 0
Hits -3

Powered by Codecov. Last updated by [273a769...da78969][cc-compare]
[cc-base-branch]: https://codecov.io/gh/callemall/material-ui/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/callemall/material-ui/compare/273a769c84fc3cf1377c6168efdb6bbd1245c208...da78969516ac7d6f21eb3d41ea39eef1b3ecb1eb
[cc-pull]: https://codecov.io/gh/callemall/material-ui/pull/4126?src=pr

@callemall/material-ui There are unrelated topics inside this PR. If you think this chunk is too big. I can break it down.

@newoga I have changed the stateless functionnal component organisation because of yannickcr/eslint-plugin-react#567.

That's fine by me! As long as we're consistent 😄

Deprecate the styleResizable mixin and remove it from the documentation. That's was our last mixin #RIP. Now, we should be able to enforce react/prefer-es6-classes.

🎉 Finally! 🎉

@oliviertassinari doing a PR for that right now

Should we put the withWidth HoC in util instead of style. Conceptually I always thought of util as our internal files and components, and style and the public util. If I'm correct thinking that way, then maybe util is a better home for it.
Thoughts?

Now, we should be able to enforce react/prefer-es6-classes.

@oliviertassinari Should we just do that as part of this PR?

Should we put the withWidth HoC in util instead of style. Conceptually I always thought of util as our internal files and components, and style and the public util. If I'm correct thinking that way, then maybe util is a better home for it.

Yes, internal is internal components, utils internal utilities, and styles  (mostly) public style / theme stuff.

@oliviertassinari Should we just do that as part of this PR?

Done I have

enforced react/prefer-es6-class
move withWidth HOC to the /utils folder
Add two small test
Split the PR into 4 distinct commits.

@oliviertassinari This looks good to me. Nice commits!

Nice work @oliviertassinari! 💯

@oliviertassinari: I was going to use this new component but the widths used don't match the Material Design window sizes. Is there are reason they are different?

@damonmaria Good question, Material-UI completely overlook the responsive aspect of the specification. In this PR, I was focusing on porting the mixin into a high order component.
We definitely need to improve this breaking points 👍.


Propagate default tolerarion between subnamespaces

We need to be able to propagate the defaultToleration between child subnamespaces.
This is a functionality that used to exist before #65 that we need to add back now.
https://github.com/dana-team/hns/commit/ee99094daca9ddb3ce628720735c80a1b320aae0#diff-324d5fd7272d8b393ef4d0e8ce7b8f8e10dc41d6eb43912276b966c6f81e4521L251-L254
/kind bug

/assign @omer5yerushalmi


Enter number Amount with left handed style

Hello Guys,
I know number is is generally come with right side alignment. But actually i've some little bit problems during enter numbers with 2 decimals like
Example -
The default text style with mask is look like 0,00
Currently when i need to enter 12.00 i must need to enter amount 1200 but it will be possible that i'm just enter amount 12 and then rest of decimal .00 is automatically added at last and if user will need to change .00 values then he will change it itself by moving cursor.
But my requirement is to add 12 as a before .00 by default when i start entering amount.
Thanks in advance.
Dilip.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28849574-enter-number-amount-with-left-handed-style?utm_campaign=plugin&utm_content=tracker%2F1022469&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1022469&utm_medium=issues&utm_source=github).

The another way to tell this is there is any possibility to trigger this input mask when onblur event is triggered.

Related to #118
Pull requests are welcome.

Closing since it is a duplicate of #118.


Link grid-animation to caniuse

Interesting! It's in the caniuse-lite package, but the package brings everything over from https://github.com/Fyrd/caniuse including features marked as "shown": false. I'll open an issue on caniuse-lite to see if they'll omit those entries or expose the shown property.
Alternatively, though less conveniently, we could pull in caniuse's data directly.

@ddbeck with https://github.com/browserslist/caniuse-lite/issues/119 resolved, can this be revisited?


[v2.9] Bump Wrangler version to v2.0.2

The previous wrangler commit included all of the v2 changes. Except for the import path changes.
(cherry picked from commit 0825036af7ab07c6becee1e51262c3db4463912a)

What this PR does / why we need it:

Which issue(s) this PR fixes
Issue #
Special notes for your reviewer:
Checklist:

[ ] squashed commits into logical changes
[ ] includes documentation
[ ] adds unit tests
[ ] adds or updates e2e tests
[ ] backport needed

@mjura are we good with the api diff failing?


Link between nodes separates

This task is automatically imported from the old Task Issue Board and it was originally created by Greg Ociepka.
Original issue is here.

When dragging a node around noticed that the link separates from the parent. Vertical separation is small between the two
https://discord.com/channels/401396655599124480/1063108713340215368
and
Connection shape fragments sometimes are not properly removed
https://discord.com/channels/401396655599124480/1065600305301049375

Just saw it today on one of the demos:

Video from the discord discussion:
https://user-images.githubusercontent.com/919491/218528724-0daf5514-b4d2-4f8a-8b7f-3fd159ab4200.mp4

Just saw it today on one of the demos:

If I remember correctly, this small thing on the left should be there, but should be behind node. So probably there is something wrong with the edge placement, but this small thing is not an issue. I might be wrong though, its a long time since I was touching it

@hubertp reported that the issue became worse recently. I can't confirm it. I checked a few commits and reproduced the issue even on commit f53696eda4c6514d9fdc77e8899694c09cfa3ca1, which is more than one month old. It is quite easy to reproduce, indeed.
https://user-images.githubusercontent.com/6566674/226448613-c2758658-c776-4473-b3f2-ecdc3dd3a2d4.mp4

Fixed in #6155


Test error

Test how this issue fails!

Build failed because of:
No cached entry, and can not fetch: homeaway Can't resolve stock ticker AWAY; please manually add a "stock_ticker" key to landscape.yml or set to null


powell method optimizer function implementation by gpu code

@kylechampley  Do you have any plan to implement current python's scipy.optimize.minimize into your own gpu c++ code ?
That could bring faster auto geo-calibration result.

No, I had no intension to do such a thing.  The main computations in performing this geometric calibration are with the calculation of the cost function itself which I already implemented for the GPU.

I see you can close this issue.


Requests are too protocol chatty, should be optimized

re: https://github.com/nats-io/go-nats/issues/294
Must retain request and requestOne semantics, while minimizing network calls to subscribe to a a new inbox channel.

Final fix is in #196


Help getting started

I'm just trying to get the backtest example working so I didn't bother updating the config-example.yml with my Poloniex keys.
InfluxDB is running, and I used this command to populate it with historical data:
DB_RELEASE=7-5-2017 DB_NAME=historical-poloniex INFLUX_DIR=/usr/local/var/influxdb ./local-services/influxdb/restore.sh
After trying to run the backtest like this:
python3 examples/backtest.py -c config-example.yml -s buffed-coin
I receive the following error log:
INFO:staticconf.config:Unexpected value in moneybot configuration: [('influxdb.host', 'localhost'), ('influxdb.port', 8086), ('influxdb.username', 'root'), ('influxdb.password', 'root'), ('influxdb.database', 'historical-poloniex'), ('poloniex.key', 'YOUR_API_KEY'), ('poloniex.secret', 'YOUR_API_SECRET'), ('trading.fiat', 'BTC'), ('trading.interval', 86400)]
INFO:moneybot.evaluate:Testing from 2017-01-01 00:00:00 to 2017-01-15 00:00:00
INFO:moneybot.evaluate:Testing from 2017-01-15 00:00:00 to 2017-01-29 00:00:00
INFO:moneybot.evaluate:Testing from 2017-01-29 00:00:00 to 2017-02-12 00:00:00
INFO:moneybot.evaluate:Testing from 2017-02-12 00:00:00 to 2017-02-26 00:00:00
INFO:moneybot.evaluate:Testing from 2017-02-26 00:00:00 to 2017-03-12 00:00:00
INFO:moneybot.evaluate:Testing from 2017-03-12 00:00:00 to 2017-03-26 00:00:00
INFO:moneybot.evaluate:Testing from 2017-03-26 00:00:00 to 2017-04-09 00:00:00
INFO:moneybot.evaluate:Testing from 2017-04-09 00:00:00 to 2017-04-23 00:00:00
INFO:moneybot.evaluate:Testing from 2017-04-23 00:00:00 to 2017-05-07 00:00:00
INFO:moneybot.evaluate:Testing from 2017-05-07 00:00:00 to 2017-05-21 00:00:00
INFO:moneybot.evaluate:Testing from 2017-05-21 00:00:00 to 2017-06-04 00:00:00
INFO:moneybot.evaluate:Testing from 2017-06-04 00:00:00 to 2017-06-18 00:00:00
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/usr/local/Cellar/python3/3.6.1/Frameworks/Python.framework/Versions/3.6/lib/python3.6/multiprocessing/pool.py", line 119, in worker
    result = (True, func(*args, **kwds))
  File "/usr/local/Cellar/python3/3.6.1/Frameworks/Python.framework/Versions/3.6/lib/python3.6/multiprocessing/pool.py", line 44, in mapstar
    return list(map(*args))
  File "/Users/plingampally/dev/moneybot/moneybot/evaluate.py", line 65, in backtest
    return list(fund.begin_backtest(start, end))
  File "/Users/plingampally/dev/moneybot/moneybot/fund.py", line 112, in begin_backtest
    val = self.step(date)
  File "/Users/plingampally/dev/moneybot/moneybot/fund.py", line 44, in step
    market_state = self.market_adapter.get_market_state(time)
  File "/Users/plingampally/dev/moneybot/moneybot/market/adapters/__init__.py", line 45, in get_market_state
    charts = self.market_history.latest(time)
  File "/Users/plingampally/dev/moneybot/moneybot/market/history.py", line 173, in latest
    result_set = self.client.query(q)
  File "/Users/plingampally/dev/moneybot/venv/lib/python3.6/site-packages/influxdb/client.py", line 392, in query
    in data.get('results', [])
  File "/Users/plingampally/dev/moneybot/venv/lib/python3.6/site-packages/influxdb/client.py", line 391, in <listcomp>
    for result
  File "/Users/plingampally/dev/moneybot/venv/lib/python3.6/site-packages/influxdb/resultset.py", line 23, in __init__
    raise InfluxDBClientError(self.error)
influxdb.exceptions.InfluxDBClientError: database not found: historical-poloniex
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "examples/backtest.py", line 73, in <module>
    main(args)
  File "examples/backtest.py", line 43, in main
    window_distance_days=14,
  File "/Users/plingampally/dev/moneybot/moneybot/evaluate.py", line 89, in evaluate
    results = p.map(backtest, time_tuples)
  File "/usr/local/Cellar/python3/3.6.1/Frameworks/Python.framework/Versions/3.6/lib/python3.6/multiprocessing/pool.py", line 260, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/usr/local/Cellar/python3/3.6.1/Frameworks/Python.framework/Versions/3.6/lib/python3.6/multiprocessing/pool.py", line 608, in get
    raise self._value
influxdb.exceptions.InfluxDBClientError: database not found: historical-poloniex

@plingampally Thanks for opening this issue.
For starters, make sure your influxd isn't running when you do the restore step. Bug us again if the issue persists.

That was it, thanks!


Update for Terraform 0.12.20 and AWS provider 2.49.0

Update for Terraform 0.12.20 and AWS provider 2.49.0
terraform validate complains about the dashboard body indentation if it's made "pretty", so I've kept the unsightly but valid changes from terraform 0.12upgrade.

Closing this since it's solved in #21 , lmk if you still have issues


mj-column doesn't work on Comcast (web), GMX (web) & Telstra/Bigpond (web)

Describe the bug
Comcast (web), GMX (web) & Telstra/Bigpond (web) don't render mj-column correctly. instead they stack it.
To Reproduce
Steps to reproduce the behavior:

Create a file with this MJML code: <mjml>...</mjml>

<mj-section background-color="{{ color }}" padding="0" direction="rtl">     <mj-column>         <mj-image src="{{ imagePath }}/{{ img.file }}" href="{{ block.uri }}" alt="{{ block.img.alt }}" padding="0" fluid-on-mobile="true" />     </mj-column>     <mj-column padding="0 0 15px 0" vertical-align="top">         <mj-text font-size="17px" font-weight="600" line-height="22px" color="#000" padding="15px">             {{ block.title | safe }}         </mj-text>         <mj-text font-size="14px" line-height="22px" color="#000" padding="15px">          {{ block.texts | safe }}         </mj-text>     </mj-column> </mj-section>

Render it to HTML
Send the HTML to an email address with '...'
We tested it in https://www.emailonacid.com/
See error

Render Error:

Expected behavior
Render expected behaviour:

MJML environment (please complete the following information):

OS: [e.g. MacOS] : macOS Catalina v 10.15.1
MJML Version [e.g. 4.2.0]<EMAIL_ADDRESS>
Email sending environment(for rendering issues):
Tested in: https://www.emailonacid.com/
Sent from: Acoustic https://www.ibm.com/customer-engagement/marketing/launch/
&
Rendered code pasted from build.
Both gave same results.
Affected email clients (for rendering issues):

Email Client [e.g Gmail] : Comcast (web), GMX (web) & Telstra/Bigpond (web)
OS: [e.g. Windows] : macOS Catalina v 10.15.1 & windows 7
Browser [e.g. Google Chrome] : ie11, chrome, firefox

Screenshots
If applicable, add screenshots to help explain your problem.
(see renders above.)
Additional context
Add any other context about the problem here.
We tried to find out what is wrong. We found out that the system creates a style tag:
<style type="text/css">       @media only screen and (min-width:480px) {         .mj-column-per-100 { width:100% !important; max-width: 100%; } .mj-column-per-50 { width:50% !important; max-width: 50%; } .mj-column-per-32 { width:32% !important; max-width: 32%; } .mj-column-per-2 { width:2% !important; max-width: 2%; } .mj-column-per-49 { width:49% !important; max-width: 49%; } .mj-column-per-37 { width:37% !important; max-width: 37%; } .mj-column-per-63 { width:63% !important; max-width: 63%; } .mj-column-per-33 { width:33.333333333333336% !important; max-width: 33.333333333333336%; }       }     </style>
This is ignored in Comcast (web), GMX (web) & Telstra/Bigpond (web)

Mjml is mobile first so if a mail client doesn’t support media queries, it will fallback to mobile version.

On 18 Dec 2019, at 17:26, Jonas Van der Sande<EMAIL_ADDRESS>wrote:

Describe the bug
Comcast (web), GMX (web) & Telstra/Bigpond (web) don't render mj-column correctly. instead they stack it.
To Reproduce
Steps to reproduce the behavior:
Create a file with this MJML code: ...
      {{ block.title | safe }}   {{ block.texts | safe }}
Render it to HTML
Send the HTML to an email address with '...'
We tested it in https://www.emailonacid.com/
See error
Render Error:
Expected behavior
Render expected behaviour:
MJML environment (please complete the following information):
OS: [e.g. MacOS] : macOS Catalina v 10.15.1
MJML Version [e.g. 4.2.0]<EMAIL_ADDRESS>Email sending environment(for rendering issues):
Tested in: https://www.emailonacid.com/
Sent from: Acoustic https://www.ibm.com/customer-engagement/marketing/launch/
&
Rendered code pasted from build.
Both gave same results.
Affected email clients (for rendering issues):
Email Client [e.g Gmail] : Comcast (web), GMX (web) & Telstra/Bigpond (web)
OS: [e.g. Windows] : macOS Catalina v 10.15.1 & windows 7
Browser [e.g. Google Chrome] : ie11, chrome, firefox
Screenshots
If applicable, add screenshots to help explain your problem.
(see renders above.)
Additional context
Add any other context about the problem here.
We tried to find out what is wrong. We found out that the system creates a style tag:

This is ignored in Comcast (web), GMX (web) & Telstra/Bigpond (web)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks @iRyusa for the feedback.
Although i do understand the fallback, is there a workaround you can suggest for this specific mail clients?

The issue is not with MJML or those clients, you can see in this test it's working fine on all 3 clients you mentioned: https://app.emailonacid.com/app/acidtest/8zceYrJEUjDWHCceB5UWzn2HQAOqCofEvsNN6D2uQbuAe/list.
The issue is either with your code (it's hard to tell as you didn't provide the whole code to reproduce), or your method of sending (IBM Acoustic).

@iRyusa agreed. this should be the case when media queries aren't supported.
But should the content not also grow with the column?
I made it so the image is always 100% the width of the column.
But the generated width on the image, td and table Is half the width of the mail itself. This width is generated by the parent column.
When the column doesn't work and the fallback is shown, doesn't that mean that the column with a width of 50% should become one of 100%?
In my generated code I have this in my head:
<style type="text/css">
    @media only screen and (max-width:480px) {
      table.full-width-mobile { width: 100% !important; }
      td.full-width-mobile { width: auto !important; }
    }
</style>

It is also present as a class on the column.
So a media query for mobile, that isn't supported on a mail client, isn't used in a framework that works with mobile first? Shouldn't this "full width" be present by default and overwritten by a media query on desktop?
Just to be sure.

Column expand to 100% width and content too, but image "full width" is handled by a media query as you pointed out so it won't work if there's no MQ supported too.
Images are a special case because width are defined in px, not in % (mostly to ensure outlook doesn't explode on large image).

Shouldn't this "full width" be present by default and overwritten by a media query on desktop?

We've tried, and everytime Outlook desktop just break the layout. This is far from the best solution, but in common cases it works fine.

@iRyusa Ok, thank you for the feedback!


Add flags

Add flags

Added here we don't need to add each proposal id if there is a link we can flag


Submission request, next.js template

Hi, I opened this issue to submit a Next.js template to the store.
The template uses the new serverless mode introduced in Next.js 8 to create a single file for each page.
It was built based on the node10-express-template in a way it is compatible with the features of the base template.
It comes with 2 sample pages and the pages include emotion for styling:

'/': Home page
'/about': About page

You can check it deployed here.
Please feel free to reach me if you have any suggestions about how to make the template better or if you encounter any issue.
The template is openfaas-serverless-nextjs-template.
Thanks.

Thanks for your PR.
What does the "8" refer to?
OpenFaaS Next.js 8 Serverless Mode template

Template based on Node.js 10 Express Template for OpenFaaS

Hi, it refers to the version which the serverless mode was introduced.
https://nextjs.org/blog/next-8
Thinking about it now, maybe it is not necessary.

@johnnyeric I like this, but I'm a bit confused about the README - I'm seeing examples of the node10-express template mixed in.
Can you re-write the docs or help me understand how the two play together? I'd love to see something like this merged and available in the store.
Alex

Hi @alexellis, thanks for reviewing this.
I have improved some parts of the template and I've now updated the README with better explanations about how it works and how to customize.
About the examples, my idea was that this template was fully compatible with the node10-express template, but I improved that by defining more focused examples while giving the ability to use the context features.
Can you please take a look again?
Please, let me know if you find it is ok, then I can send a pull request.

Thanks @johnnyeric 👍
@kenfdev @burtonr do either of you have time to have a more detailed look at the template?

Ping cc @phumberdroz

The only thing that looks conflicting to me is that Root License is MIT and the NPM package is marked as ISC besides that the code looks fine.

Hi @phumberdroz, thanks for reviewing the template. I adjusted the license on the package.json files on my repo.


fix: update vitejs/plugin-react to prevent peer dependency warning

I recently created a new Tauri project using React and TS. After installing with pnpm, I encountered this warning:
 WARN  Issues with peer dependencies found
.
└─┬ @vitejs/plugin-react 4.0.3
  └── ✕ unmet peer<EMAIL_ADDRESS>found 5.0.0

@vitejs/plugin-react 4.0.3 has a dependency on vite 4.x. Updating to 4.2.1 of the plugin fixes this warning.

Thank you


Explain what the bars mean in the CLI interface

I'm trying to understand what the bars mean in the CLI interface, but there's nothing in the README. A screenshot and explanation in the README would help a lot here.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/8514196-explain-what-the-bars-mean-in-the-cli-interface?utm_campaign=plugin&utm_content=tracker%2F298985&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F298985&utm_medium=issues&utm_source=github).

The bars show which pieces and blocks the client is requesting.
Each bar is labeled with the piece number. Each piece is made up of 16KB blocks, labeled red for missing, blue for requested, and green for received.
The bars should probably be disabled by default (I just made an issue for that: https://github.com/feross/webtorrent/issues/296). They're a bit overkill for most casual users, but useful when trying to debug issues.


CII Badge: New functionality test and documentation requirements

The project MUST have a formal written policy that as major new functionality is added, tests for the new functionality MUST be added to an automated test suite.

The project MUST include, in its documented instructions for change proposals, the policy that tests are to be added for major new functionality.

https://github.com/zephyrproject-rtos/zephyr/blob/master/CONTRIBUTING.rst
Policy exists to govern adding new functionality and testing.


Fix stack overflow during btree iteration

Migrating from rocksdb to paritydb (here) I discovered that stack randomly overflows during BTreeIterator::{next, prev} calls.
Seems like rust compiler failed to optimize some recursive functions with no state with TCO (even in release mode). Looks like as all optimizations, TCO is also eventual and is not guaranteed in any way. More on the history of it in the compiler here.
So this pr basically rewrites recursive functions using loops so that we would guarantee that we will reuse memory and won't allocate stack frame.
Better to review pr commit by commit, and for the first one it would be helpful to ignore indentations.

@i1i1 can you run it under gdb to catch backtrace when stack overflow happens?

@i1i1 thanks for the stacktrace and the PR. Small question what is the size (in number for keys) of your column ? (One of my first implementation for the btree index did use a dedicated file for nodes which we replace by just using table values entry to simplify a bit as perf was not the first concern, but maybe I should try to resurrect the branch IIRC it is an easy perf gain and not that much code in fact).

Small question what is the size (in number for keys) of your column?

In that case, it was around 3000 keys. Scenario was as follows:

At first, we insert 3000 key values in batches of 256 I believe
Then we do those things in a loop:

try to find N keys starting from the beginning of the db and N keys from the end

remove those keys and add some new ones

So everything blows at iterating over db as we hit some case when we need to recur after removing key in commit overlay

@cheme can you merge it please? Or are we waiting for something?

I think it's good. @arkpar I am merging let me know if anything off.


Initial Version of Save Artifacts in mlflow.pytorch

What changes are proposed in this pull request?
Adds the ability to log artifact files along with model during mlflow.pytorch.save_model or mlflow.pytorch.log_model
How is this patch tested?
Some tests similar to the Keras, fastai ones have been written to ensure model exporting and autologging works in most cases.
Release Notes
Is this a user-facing change?

[ ] No. You can skip the rest of this section.
[] Yes. Give a description of this change to be included in the release notes for MLflow users.

(Details in 1-2 sentences. You can just refer to another PR with a description if this PR is part of a larger change.)
What component(s), interfaces, languages, and integrations does this PR affect?
Components

[x] area/artifacts: Artifact stores and artifact logging
[ ] area/build: Build and test infrastructure for MLflow
[ ] area/docs: MLflow documentation pages
[ ] area/examples: Example code
[ ] area/model-registry: Model Registry service, APIs, and the fluent client calls for
Model Registry
[ ] area/models: MLmodel format, model serialization/deserialization, flavors
[ ] area/projects: MLproject format, project running backends
[ ] area/scoring: Local serving, model deployment tools, spark UDFs
[ ] area/tracking: Tracking Service, tracking client APIs, autologging

Interface

[ ] area/uiux: Front-end, user experience, JavaScript, plotting
[ ] area/docker: Docker use across MLflow's components, such as MLflow Projects and MLflow Models
[ ] area/sqlalchemy: Use of SQLAlchemy in the Tracking Service or Model Registry
[ ] area/windows: Windows support

Language

[ ] language/r: R APIs and clients
[ ] language/java: Java APIs and clients

Integrations

[ ] integrations/azure: Azure and Azure ML integrations
[ ] integrations/sagemaker: SageMaker integrations

How should the PR be classified in the release notes? Choose one:

[ ] rn/breaking-change - The PR will be mentioned in the "Breaking Changes" section
[ ] rn/none - No description will be included. The PR will be mentioned only by the PR number in the "Small Bugfixes and Documentation Updates" section
[x] rn/feature - A new user-facing feature worth mentioning in the release notes
[ ] rn/bug-fix - A user-facing bug fix worth mentioning in the release notes
[ ] rn/documentation - A user-facing documentation change worth mentioning in the release notes

http://ec2-3-21-204-186.us-east-2.compute.amazonaws.com:5000/#/experiments/8/runs/f5fba4e099fe4449aa2a48cd448ae814

For tests, we should be able to:

Compute a temporary file path, e.g. my_example_artifact = tmpdir.join("some-file").strpath, where tmpdir is the pytest tmpdir fixture
Write some dummy contents to my_example_artifact
Save a model via mlflow.pytorch.save_model(model path, artifacts={"myCustomPath": my_example_artifact})
Load back the artifact via code like:client = mlflow.tracking.MlflowClient()
# See API docs for download_artifacts: https://www.mlflow.org/docs/latest/python_api/mlflow.tracking.html#mlflow.tracking.MlflowClient.download_artifacts
downloaded_path = client.download_artifacts(os.path.join(model_path, "myCustomPath"))

Verify the contents of the artifact

For tests, we should be able to write a test in which we:

Compute a temporary file path, e.g. my_example_artifact = tmpdir.join("some-file").strpath, where tmpdir is the pytest tmpdir fixture
Write some dummy contents to my_example_artifact
Save a model via mlflow.pytorch.save_model(model, model_path, artifacts={"myCustomPath": my_example_artifact})
Load back the artifact via code like:client = mlflow.tracking.MlflowClient()
# See API docs for download_artifacts: https://www.mlflow.org/docs/latest/python_api/mlflow.tracking.html#mlflow.tracking.MlflowClient.download_artifacts
downloaded_path = client.download_artifacts(os.path.join(model_path, "myCustomPath"))

Verify the contents of the artifact

Thanks you very much for the detailed inputs. I will add the tests as well and raise a PR in upstream branch

@shrinath-suresh thanks for the PR! Would you be able to document the artifacts parameter and add tests? Feel free to also open this PR against the upstream MLflow repo: https://github.com/mlflow/mlflow, as it touches code that's part of upstream MLflow. That way, we can include the new functionality in our next release :)

I have raised a PR against upstream branch - https://github.com/mlflow/mlflow/pull/3382

For tests, we should be able to write a test in which we:

Compute a temporary file path, e.g. my_example_artifact = tmpdir.join("some-file").strpath, where tmpdir is the pytest tmpdir fixture
Write some dummy contents to my_example_artifact
Save a model via mlflow.pytorch.save_model(model, model_path, artifacts={"myCustomPath": my_example_artifact})
Load back the artifact via code like:client = mlflow.tracking.MlflowClient()
# See API docs for download_artifacts: https://www.mlflow.org/docs/latest/python_api/mlflow.tracking.html#mlflow.tracking.MlflowClient.download_artifacts
downloaded_path = client.download_artifacts(os.path.join(model_path, "myCustomPath"))

Verify the contents of the artifact

I have added 4 Unit tests using tmpdir fixture

Verifying artifacts parameter in mlflow.pytorch.log_model
Verifying artifacts parameter in mlflow.pytorch.save_model
Invalid path as input for artifacts parameter (Negative test)
Invalid type of input for artifacts parameter (Negative test)


Update network.go

as mentioned in issue #350
vlanId should not be greater than 4094

Describe what this PR does / why we need it
vanId range bugfix
Does this pull request fix one issue?
Fixes #350
Describe how you did it
changed 4096 to 4094
Describe how to verify it
Special notes for reviews

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

Codecov Report
Base: 40.95% // Head: 40.88% // Decreases project coverage by -0.07% :warning:

Coverage data is based on head (56563f8) compared to base (c5bc5b9).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #352      +/-   ##
==========================================
- Coverage   40.95%   40.88%   -0.08%
==========================================
  Files          43       43
  Lines        2769     2769
==========================================
- Hits         1134     1132       -2
- Misses       1519     1520       +1
- Partials      116      117       +1

Flag
Coverage Δ

unittests
40.88% <0.00%> (-0.08%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/daemon/utils/network.go
0.00% <0.00%> (ø)

.../controllers/networking/subnetstatus_controller.go
91.83% <0.00%> (-4.09%)
:arrow_down:

pkg/controllers/networking/quota_controller.go
84.00% <0.00%> (-4.00%)
:arrow_down:

pkg/controllers/networking/ipam.go
77.41% <0.00%> (+6.45%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Can't seem to get youtube playlists to work

Hey guys,
Firstly -- musicbox is fantastic! Very happy overall. Want to make that clear :-).

Go to search, choose 'youtube' enter youtube playlist uri, it finds the first song, but not the whole list.

I looked at the mopidy youtube connector (https://github.com/dz0ny/mopidy-youtube), and tried a generic stream prefixed with yt:, but that doesn't seem to work either.
Any thoughts?
Matthew

If you have not done so, please first update mopidy-youtube to the latest version by running pip install --upgrade mopidy-youtube. If you are still experiencing problems please post at discuss.mopidy.com as this issue tracker is really meant for bugs and feature requests rather than user support. Thanks

i did the upgrade. but now it seems that my webpage isn't working anymore. any idea?
i dit an apt-get upgrade after to updateeverything.
still nothing.
i got this;
ERR_CONNECTION_REFUSED?

Recent versions of mopidy-youtube probably depend on the later versions of Mopidy which Musicbox does not currently support. Musicbox v0.6 was not designed to be upgraded using apt-get.


Added youtube embed for ep3

Signed-off-by: Brent Salisbury<EMAIL_ADDRESS>

Whoops, merged this in #14 missed this one! :)


search: signal if more than 3 authors

closes #1873

Screenshot

Awesome! Could we get a screenshot? I'm nervous that "and other(s)" might get a bit long

Oups sorry I took the screenshot but forgot to add it and add some discussion here :sweat_smile: .
So I tried with ... too and it didn't stand out clearly enough (a bit too small and squeezed) - felt like a rendering artifact. I tried more and et al., but more wasn't quite the right word in the end and et al. is typically strictly for "and others" plural, there is the translation thing your mentioned, and perhaps it's a bit too high brow.
This morning I just thought about a more nuanced version, which might be better, which is the "counted others". Something like:
for 1 more author

for more than 1 more author:

I saw translation mechanisms that should allow this. I will change it and wait for more feedback.

That looks really nice! One thought-could you link it to the record landing page? Since the other authors are linked it would be nice if the "n others" led you to where you see who they are.

That looks really nice! One thought-could you link it to the record landing page? Since the other authors are linked it would be nice if the "n others" led you to where you see who they are.

Yes, that makes sense!

Done :santa:


Fields component and Form examples

Adds form examples and the <Fields /> component.
EDIT
These turned out to be non issues.  The doc site shows usage:
Fixes #36 #38 #92.

Makes sense :fallen_leaf:


AWS signature canonical string and query not urlencoded.

Hey,
I am forming the Authorisation header and I am not using the Boto libraries. While forming the canonical uri and canonical querystring the aws documentation(https://docs.aws.amazon.com/general/latest/gr/sigv4-create-canonical-request.html ) says that we need to urlencode them. However, i do not see any urlencoding in the init.py file. Can you pls. check if that's missing or there is some gap in my understanding ? This is with reference to the s3

Hello,
Because this implementation is specific for the requests package, the query string is assumed to be already urlencoded.  The qs variable comes from the urlparse of r.url, which itself is a requests.models.PreparedRequest object (see code for requests_auth_aws_sigv4/init.py line 109).
It's possible this assumption is incorrect, so if you are seeing a problem or errors, please let me know what you did for me to recreate it.
Thanks!

Thanks @andrewjroth for the answer. Got it.
However, I think there can be some issue since requests uses quote_plus and aws needs the encoding in format as space = %20 (quote) and not + . I am trying to pass the params in the url itself then, rather than passing it as params dictionary as below, to get the %20 encoding.
for eg --> requests.get("http://abc.com/xyz?uploadid=13xxsfw&partnumber=2) # rather than passing params={params_dict}
Also, can you pls. help me with the understanding of Canonical URI in https://docs.aws.amazon.com/general/latest/gr/sigv4-create-canonical-request.html as it says for s3 we should not double encode it. Isn't the requests module already encoding it or it is expected that we encode it once more, since I now pass the entire thing as uri. What is expected here which is different for s3 than other services ?

Do you have an example failure case?  Do you have sample code which fails?  I see what you are saying, but I am not sure how to test it.  It's possible the handling of parameters and encoding could be improved.

I believe this may be addressed by #9


How to manage node version installed by homebrew and anaconda

I have node installed with anaconda and homebrew? Now I won't to remove all of them and manage node version via nvm. I can do uninstall node via brew do I also need remove anaconda version?

You have to remove those versions the same way you installed them. I’m not familiar with anaconda, so i can’t help you there.


[FLINK-2047] [ml]  Rename CoCoA to SVM

The CoCoA algorithm as implemented functions as an SVM classifier.
As CoCoA mostly concerns the optimization process and not the actual learning algorithm, it makes sense to rename the learner to SVM which users are more familiar with.

LGTM. Will merge it.

Just noticed a small mistake in the example, I'll fix it and push again.

Great. Then I'll merge it today.
On Wed, May 27, 2015 at 4:38 PM, Theodore Vasiloudis <
<EMAIL_ADDRESS>wrote:

Just noticed a small mistake in the example, I'll fix it and push again.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/flink/pull/733#issuecomment-105935329.


How to actually use? (not duplicate)

I have the controllers connected, and Mario theme and gyro works fine. However, I cannot figure out how to use this with a game. (The game is ssf2: https://www.mcleodgaming.com/games/ssf2/downloads(the controller compatibility is not enabled in browser))
On Windows 10  Intel Pretty sure I'm 64 bit (x86)
P.S. Sorry if this post looks horrible.
Edit: Also, I do not see anything in Vjoy monitor

Also, fairly sure it is not combining l and r joy cons right

Do I need UCR?

(the controller compatibility is not enabled in browser)

Pretty much wrecks everything here.

I know but the downloaded version allows controllers.

My question wasn't about that specific game, just in general

My question wasn't about that specific game, just in general. Also, the downloaded version supports controllers, but not combined ones.

Did you possibly forget to check the "Combine JoyCons" box?

No

Did you set VJoy to 32 buttons?

Yes

I don't know then, sorry.

Thanks.  I'm pretty sure I just don't know how to integrate it with a game.


Desktop - Chat - The attached picture blinks a few times after coming online

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.31-3
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4940615&group_by=cases:section_id&group_id=292107&group_order=asc
Issue reported by: Applause Internal Team
Action Performed:

Open the app
Log in with any account
Navigate to any 1:1 chat
Go offline
Send an image attachment
Go online

Expected Result:
It shouldn't blink.
Actual Result:
The attached picture blinks a few times after coming online.
Workaround:
Unknown
Platforms:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/a3c89b1d-220f-44fa-a485-4205a56c99ce

View all open jobs on GitHub

Adding to my test list and will get to it later this week.

No update yet

Other GHs have been a priority; I'll review them soon.

No update

I've been sick this week and haven't been able to review this GitHub. There is no major update here; I'll try to carry on next week.

It's not overdue; I'm still catching up from being sick at the end of last week.

No update

I'll get to this one soon.

Closing - I was unable to replicate the experience and didn't get a triple blink or any blink when I came back online


Hide basic auth password for HTTP status checks

Fixes #302.

The ideal solution is probably to add a “show password” toggle.


can i have some help please

what should I do here?

oh wait.. its build not rebuild .....

update: the same error with --build

Hi. Try googling: "docker-compose is not recognized as the name of a cmdlet". It's a core problem


Add clockSupported function

This is only a very simple function, but I'd like to know whether my code style is correct for the kernel :smiley:
By the way, I put clockSupported at the end of the jump table to avoid corelib from doing calls to the wrong functions. Is that the correct way to do it?

Sorry, fixed in master (commit https://github.com/KnightOS/kernel/commit/8dd42f66ceaf467df7b60b5578a4de511f5157e4).

Thanks!


Fixed issue with i18n messages when using options hash (like through: :association) to load and authorize resources

When using i18n messages: https://github.com/CanCanCommunity/cancancan/wiki/Translations-(i18n)
And having an en.yml file that looks like this:
en:
  unauthorized:
    read:
      all: "error for %{subject}"
      post: "error for post"

Having the ability:
can :read, Blog
cannot :read, Post

And loading resources like:
load_and_authorize_resource :blog
load_and_authorize_resource through: :blog

When requesting the index action, it will display the error error for hash instead of error for post.
This happens because in the method unauthorized_message, the subject param is a hash, and the code tries to get the Hash object class name. The hash on subject looks like this:
{ <Blog id:1> => Post(id:integer) }

This PR fixes those error messages having the unauthorize message methods look for the correct key instead of "hash" in those cases.

@coorasse since you are the active maintainer, is this something that we should fix? I came across this while playing with i18n for custom permission messages.


mgmt, apimanagement, add live tests

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

/azp run prepare-pipelines

/azp run java - apimanagement - mgmt - tests

/azp run java - apimanagement - mgmt - tests

/azp run prepare-pipelines

/azp run java - apimanagement - mgmt - tests


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_5441f6e2-7002-430f-928a-8aa19c1df785/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_5441f6e2-7002-430f-928a-8aa19c1df785.git

Docs Build status updates of commit a0ff3c9:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-11-07-16-53-04 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-11-07-16-53-04

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Example data not showing up in API docs

Describe the bug
After publishing a vetiver API, the EXAMPLE section is not picking up data from ptype. The same data shows up in SCHEMA section though.
To Reproduce
Use this API: https://colorado.rstudio.com/rsc/bike-predict-python-api/
Expected behavior
Example should also be populated with default values.
Screenshots

I believe this feature was lost when changing from Swagger to Rapidoc for visual API documentation (to match R + better UI. Unfortunately, it can't be changed easily with updating the openapi.json specification as anticipated.
For now, I'm adding this to the vetiver planning backlog, as I don't anticipate I will work on this more in the next 6-8 weeks. Currently you can still access the Swagger docs at {url}/docs if you prefer to use documentation with filled data.


Move various tools into eng/common directory

Move the following tools to eng/common directory

create-pull-request.yml
create-tags-and-git-release.yml
publish-blobs.yml
copy-docs-to-blobstorage.ps1

I would like to follow the same template structure we use everywhere else under eng/common. So can these go into eng\common\pipelines\templates\steps\?

sorry didn't mean to close.

We should also include the support scripts for these templates under  eng/common. For example the git-branch-push.ps1 and Submit-PullRequest.ps1 scripts and any other supporting scripts/files these templates need. We shouldn't end to clone or reference another repo in order to use these templates.

I would like to follow the same template structure we use everywhere else under eng/common. So can these go into eng\common\pipelines\templates\steps\?

Moved yaml files as you suggested


compiler: avoid some unsafe.Pointers

These seem unneeded, so best to avoid.

Do we know the recent source of test failures?

Looks like it probably was https://github.com/tetratelabs/wazero/pull/1453, weird that the tests passed in that PR 🤔
It seems like a linter issue, one of the fields on the compiler engine isn't used anymore, do you want to address as part of this change?

Fixed one of the CI failures, the other one is pretty out of scope.
Trying to address it elsewhere.

I nuked the cache with the following
$ gh extension install actions/gh-actions-cache
$ gh actions-cache list
# look for stuff
$ gh actions-cache delete --confirm gojs-test-binary-1.20

then had to re-kick all the jobs in because the failure had a dependency, and that was easier.


typo in "Install Windows ADK for Windows 10" - should be 'User State Migration Tool (USMT)'

see title

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

ID: 7ab08370-69ab-d025-cea2-c41c3aa4b908
Version Independent ID: 9f5603f8-10e6-2651-3120-6d5627217cc9
Content: Prepare for deployment with MDT (Windows 10)
Content Source: windows/deployment/deploy-windows-mdt/prepare-for-windows-deployment-with-mdt.md
Product: w10
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@anwarmahmood1 Good catch and thanks for bringing that to our attention! I'll get it fixed.


Unify docstrings and comments

Also

Fix typos
Add missing information

@dshil yes, but unifying the signatures of the methods will be done in a different PR.


beneficiaries: missing posting authority ?

The beneficiaries argument in post doesn't work for me.
steembase.exceptions.RPCError: 3030000 tx_missing_posting_auth: missing required posting authority
I'm using a dictionary for beneficiaries like this:
[{'account': 'account1', 'weight': 5000}, {'account': 'account2', 'weight': 5000}]
Even if I set only the posting account as beneficiary, it returns the same error.

The json data structure steem-python generates seems to be correct, however the bytes output that goes into transaction signing is not, hence the missing authority error.
This suggests there might be a bug somewhere in the serializer, particularly in regards to the StaticVariant type.

Some solution in sight?

Some solution in sight?

I just found out, how it works.
This doesn't work:
[{'account': 'account1', 'weight': 5000}, {'account': 'account2', 'weight': 5000}]
This works:
from collections import OrderedDict
[OrderedDict('account': 'account1', 'weight': 5000), OrderedDict('account': 'account2', 'weight': 5000)]

Actually it still doesn't work :/
steembase.exceptions.RPCError: 3030000 tx_missing_posting_auth: missing required posting authority
Missing Posting Authority

@Netherdrake
Hi, Is there some update about this issue?
I encountered the same problem when I tried to post comment with beneficiaries.
Thank you.

The issue is still present I'm going to check to see if piston has some way of solving this.

I had the same issue for SteemJ and I can confirm @Netherdrake theory - For me it was related to the byte representation.
You can check my implementation of the byte transformation starting with the toByteArray method of the CommentOptionsOperation and the CommentPayoutBeneficiaries
I hope that helps a little bit :/
Best regards,
@dez1337

Until this issue is fixed in steem-python you can use piston like so:
from piston import Steem
s = Steem(keys=[private_posting_key])
s.post(
    title="This is a Test Post",
    body="Hello World this is a test post.",
    author="username",
    meta={
        'extensions': [[0, {
            'beneficiaries': [
                {'account': 'beneficary_account_username', 'weight': 5000},
            ]}
        ]]
    },
    tags=['spam','test']
)

My issue is that I was using Anaconda with beem v 0.24.21 but this version apparently has some issue. So instead of using the "conda" command, I had to use pip to install beem 0.24.26 then I stopped getting the error! I hope this helps someone, because I spent multiple days tracking this down!
pip install beem==0.24.26


doc/manual,CONTRIBUTING.md: add treewide change syntax

Motivation for this change
https://github.com/NixOS/nixpkgs/pull/147441#issuecomment-986340870
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

On one hand, I like it. On the other I am not sure if it is worth it to increase the complexity of the contributing document for such a niche use case.

Looking back at this I see your point. It is annoying to read now that enough time has passed for me to forget what it said before :P


inline_refs doesn't work with list or full

Based on the docs I should be able to use inline_refs: list, but I got this error:

Config value 'plugins': Plugin 'ezglossary' option 'inline_refs': Expected one of: ('none', 'short', 'full') but received: 'list'

I then tried the proposed full option, but that resulted with a different error:

jinja2.exceptions.TemplateNotFound: refs-full.html

Will be fixed with v1.7.0 release. The correct configuration entry is: inline_refs: list.


Option to include visualized pipelines in the generated document

Description
An option to include the image of visualized pipelines in the Sphinx document generated by kedro build-docs command
Context
kedro-viz offers kedro viz command that can generate interactive visualized pipelines.
This visualization is very useful to explain to the stakeholders and it is even nicer to automate the manual operation to run kedro viz command, access the URL, take a screenshot, and paste it in the document.
![image](https://user-images.githubusercontent.com/33908456/61184262-542f8000-a67e-11e9-872e-3fa40e31b81c.png)

Possible Implementation
Programmatically communicate with the kedro_viz.server.
Possible Alternatives
Use a graph visualization tool such as graphviz.

Hi @Minyus, what sort of document would you like the image to be pasted in? Currently Kedro doesn't really produce any soft deliverables apart from perhaps the automated documentation. But providing the visualization there feels like a special case.

Hi @Pet3ris, it would be awesome if a visualized pipeline image file can be included in the HTML document automatically generated by kedro build-docs command.
Alternatively, would it be possible to output the pipeline graph data used by Kedro-Viz so that users can visualize the pipeline using common graph visualization tools such as GraphViz and Gephi?

Hi @Minyus, this sounds like a very interesting feature and I agree that it can be useful to add the pipeline graph to the docs. Since this feature request is more of a request to the kernel-viz plugin, rather than Kedro, maybe we can have the feature request posted https://github.com/quantumblacklabs/kedro-viz instead.
The implementation of this feature may prove to be quite involved, since it will require running the kedro-viz frontend in a headless browser and then take a screenshot there, which could be used for the docs. Given that this feature is nice to have and not very important for the user's workflow, but quite high on the effort, I will put it as low priority.

@Minyus you can actually access the graph representation. kedro uses a hidden folder called .log or similar when sharing graph outputs with the graphviz so to answer your second question that's definitely possible.
Thoughts @idanov?

@Pet3ris Where can we find the hidden folder? Can we use it without running kedro viz command?

@Pet3ris that doesn't exist anymore
@Minyus If you'd like to see how Viz extracts the graph info from Kedro you can look here
https://github.com/quantumblacklabs/kedro-viz/blob/develop/package/kedro_viz/server.py#L53

@tolomea Noted. Thank you.

Hi @Minyus, I am going to close this issue as I have moved it under the kedro-viz umbrella - please see https://github.com/quantumblacklabs/kedro-viz/issues/31.


Overlay parcel lines on both maps

From Jesse at Public Works

This is a great tool for looking at personal property. It is very general without the Parcel line overlay, that could provide much needed context. If the parcel layer we included, they would not need to be a function that is turned on and off.

won't fix


Can't run externals since update

Since I update to the latest commit today, I can't use cornerstonetools as normal.
The error message is ,
TypeError: Cannot read property 'getEnabledElement' of undefined
at getElementToolStateManager (app.js:10284)
at Object.addStackStateManager (app.js:18531)
it seems that cornerstone can't be found, the problem may exist in externalModules.js,
let cornerstone = window.cornerstone;
let cornerstoneMath = window.cornerstoneMath;
can't find cornerstone, so how to fix this?

Are you using a packaged source or imports? What was your version before
you updated? Can you include a snippet of your setup code and maybe the
version number of each cornerstone library you are pulling in?
On Dec 6, 2017 9:35 PM, "Cody"<EMAIL_ADDRESS>wrote:

Since I update to the latest commit today, I can't use cornerstonetools as
normal.
The error message is ,
TypeError: Cannot read property 'getEnabledElement' of undefined
at getElementToolStateManager (app.js:10284)
at Object.addStackStateManager (app.js:18531)
it seems that cornerstone can't be found, the problem may exist in
externalModules.js,
let cornerstone = window.cornerstone;
let cornerstoneMath = window.cornerstoneMath;
can't find cornerstone, so how to fix this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cornerstonejs/cornerstoneTools/issues/302, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFh21IDApJrtyw65sqbwDKEdoHXxQh8Tks5s907TgaJpZM4Q45uN
.

The previous version I use is 0.9.1, but I made some change to the source code and build a package source myself. My code runs well before commit e3f788b98d23c2ac3cc26c5ecf7c4cb62cb04dda on Oct 18, since the commit "inject dependency", my code throw errors.
import cornerstone from 'cornerstone-core';
import dicomParser from 'dicom-parser';
import cornerstoneTools from '../../library/cornerstoneTools';
this is my import part

If you're using the latest versions for these packages, you will notice
there was a major version bump. Major version bumps generally correlate
with breaking changes.
If one of the cornerstone libraries in your project have been upgraded to
1.0+, I recommend upgrading them all to latest: Cornerstone,
CornerstoneTools, and the image load libraries
And then use the npm/import specific instructions here:
https://tools.cornerstonejs.org/installation.html
If you have difficulty, or feel like the docs aren't clear enough, please
let me know and I can help you and update the docs to be a little clearer.
On Dec 6, 2017 10:30 PM, "Cody"<EMAIL_ADDRESS>wrote:

The previous version I use is 0.9.1, but I made some change to the source
code and build a package source myself. My code runs well before commit
e3f788b
https://github.com/cornerstonejs/cornerstoneTools/commit/e3f788b98d23c2ac3cc26c5ecf7c4cb62cb04dda
on Oct 18, since the commit "inject dependency", my code throw errors.
import cornerstone from 'cornerstone-core';
import dicomParser from 'dicom-parser';
import cornerstoneTools from '../../library/cornerstoneTools';
this is my import part
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cornerstonejs/cornerstoneTools/issues/302#issuecomment-349852374,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFh21KHR4x5q3bNgQFF9iN9X5aRDXDCsks5s91vCgaJpZM4Q45uN
.

I see your issue in cornerstone, and I tried to solve this error.
After I add this code,
cornerstoneTools.external.cornerstone = cornerstone;
it runs well, but this may cause confusion to other developers if they don't notice this.

Unfortunately, I'm not the best person to answer this question. @swederik or @kofifus may be able to explain the change.
I believe we have slated more verbose errors for common setup problems like this -- Ideally, we want problems caused by major version changes to have easy/clear upgrade paths.

We needed to add a method to inject the cornerstone core version into tools (etc..) because some framework's build processes do not resolve duplicate dependencies properly. There were complaints of duplicate copies of cornerstone being loaded into applications because the WADO Image Loader depended on version 0.x.y and Tools depended on 0.x.z. With Meteor specifically, which we use in the OHIF Viewer, we had problems with this. I know some bundlers (e.g. webpack) would resolve the two versions safely, so this may not have been an issue for everyone.
For that reason, we decided to force the specification of the dependency packages and remove the NPM package dependencies themselves, and put them as dev dependencies.
It's not pretty, but it works. We put this information in the changelog and README, and made it during a major version update so it wouldn't break anyone's builds.
Sorry for the headache, but we are open to other ideas if you have any suggestions.

ok, I see it in README. Seems that every import should contain a few more lines of code to solve the external problems, but I haven't thought of any better solutions yet.
Thanks for your answer and patience.

Hi swederik,
I have posted instructions for loading ES6 modules directly using latest Chrome in https://github.com/cornerstonejs/cornerstone/pull/191#event-1353847488
These can be incoreperated into the Wiki/Readme
They rely on
https://github.com/cornerstonejs/cornerstone/pull/191
and
https://github.com/cornerstonejs/cornerstoneWADOImageLoader/pull/144
But somehow the discussion of those PRs stopped
Is there any plans of merging them ? or anything I can do to advance this ?
Thanks!

I can't merge the WADO Image Loader issue until the tests run. I don't have any urge to set external to a default export on cornerstone core since I plan to drop the only dependency very shortly, so it would be wasted effort.
You can help get that WADO Image Loader ticket running properly. That would be great!


Error reported when running train_picie.py file，What is the reason？

File "/home/workshop/PiCIE/data/coco_eval_dataset.py", line 108, in
fine_to_coarse_map       = np.vectorize(lambda x: fine_to_coarse_dict[x]) # not in-place.
KeyError: 183

Hi,
It seems like there is an issue with label transformation part during evaluation.


fix(headless,headless-react): make ssr terminology more consistent

https://coveord.atlassian.net/browse/KIT-2680
Here's a chart of the server-side-rendering flow:
sequenceDiagram
  participant Headless as @coveo/headless
  participant Static as Static definition
  participant SSR as Server-side
  participant CSR as Client-side
  alt Create definition
    Static->>+Headless: const definition = defineSearchEngine({ ... });
    Headless->>-Static: engine definition
  end
  CSR->>+SSR: request page
  alt Execute the first search
    SSR->>+Static: const stateA = definition.getStateA();
    Static->>SSR: First search response
    Static->>-SSR: State of every controller
  end
  alt render page
    SSR->>SSR: const html = generateHTMLFromState(stateA.controllers);
  end
  SSR->>CSR: send html
  SSR->>-CSR: send stateA
  CSR->>CSR: display HTML
  alt Obtain engine and controllers
    CSR->>+Static: const stateB = definition.getStateB(stateA.searchResponse);
    Static->>CSR: Search engine
    Static->>-CSR: Controllers
  end
  CSR->>CSR: subscribe to controllers

For StateA and StateB, we've been using inconsistent terminology.
In this PR, my goal is to align all the terms to be consistent. Here are the two main options we've been discussing:

Option 1 (option I picked): for stateA, use "initial state". For stateB, use "hydrated state"

A file shared between both server and client calls defineSearchEngine, which returns an EngineDefinition<...>
The server calls fetchInitialState

Returns an InitialState<...>

The client calls hydrateInitialState

Takes an InitialState<...> as a parameter.
Returns an HydratedState<...>

Option 2: for stateA, use "ssrState". For stateB, use "csrResult"

A file shared between both server and client calls defineSearchEngine, which returns an EngineDefinition<...>
The server calls fetchSSRState

Returns a SSRState<...>

The client calls hydrateSSRState(?)

Takes a SSRState<...> as a parameter.
Returns a CSRState<...>

I picked option 1, since it feels more natural to me after all.

Woops, missed those because I was doing a search and replace for SSR/CSR State to Static/Hydrated state.
Tried again, but this time used regular expressions like CSR([A-Z][a-z]*) -> Hydrated$1 and Static(?!State|Filter|CDN|Facet|Response|Resource)([A-Z][a-z]*) -> StaticState$1 to catch more. Of course, I didn't blindly approve every replacement.


Detect users calendar settings

Description
Ability to detect the device calendar settings on iOS.
Use Case
When iOS devices have a different type of calendar set, non-gregorian for example. We should be able to detect this setting so we change date notation.

Android after API24 has different calendar types too. This is user setting similar to locale that I could see this package supporting, but it would need more interest and a PR. I'm going to close this for now but if someone submitted a promise-based PR that implemented it that would be great

Running a poll with a proposal to defer to @react-native-community/react-native-localize package where this is likely handled better https://github.com/react-native-community/react-native-device-info/issues/627


How to get the original coco idx

Please refer to this section of the readme

Thanks so much! I Just want to double check that you understand that I want to retrieve the coco index in the original coco dataset rather than the indices in the NSD dataset.

Ah sorry, I see -- you can consult this file and match the NSD indices to the actual COCO index
https://natural-scenes-dataset.s3-us-east-2.amazonaws.com/nsddata/experiments/nsd/nsd_stim_info_merged.csv

Thanks so much! Sorry my initial inquiry is ambitious


Fixed #30024 -- Made urlencode() and Client raise a TypeError when None is passed as data.

https://code.djangoproject.com/ticket/30024

Thanks for the review Tim. I've made the suggested changes. Let me know what you think about the edits to the release notes.


Math mode snippets also work in label command inside math environment

When I have the following code, and want to make the label eq:malus1 then this autoexpands to eq:malus_{1}.
Is there a way to tell the auto-subscript snippet that it is no longer inside math mode?
\begin{equation}
    P(\theta ) = P_{0}\cos ^2 \left( \theta  \right)
    \label{eq:malus_{1}} % supposed to be \label{eq:malus1}
\end{equation}

When the text environment is used (\text{malus1} ) the auto-subscript snipped does not autoexpand. I would like the snippets the label command to not get recognised as a math environment and would like similar behavior to the text command. Do you know how this can be achieved?

Thanks for the detailed explanation, I'll take a look at it over the weekend.

:1st_place_medal:  Thank you!


USDC decimals incorrect

Background
Transfers for USDC are showing wrong decimals.
Correct decimals are captured in the object returned by getPool. Note that all LP tokens will be 18 decimals.

duplicate of #2815


Fix array of array iterator issue

Resolves #8912

Create issue https://github.com/ballerina-platform/ballerina-lang/issues/8996 to track futher improvments to closuers. Hence merging this PR.


Question: How to debug .NET MAUI from Visual Studio 2019?

I was able to debug from VS Code, however I am not positive how to launch it from VS 2019.

You need at least 16.9, but we have a manual step to create a EnableWorkloadResolver.sentinel file:
https://github.com/dotnet/net6-mobile-samples#using-ides
I don't think the HelloMaui app will work yet, multi-targeting is still in progress.

Ahhh gotcha! normal .NET for Android worked, but couldnt' get the .NET MAUI project as the startup. Makes sense.

You could try deleting <TargetFrameworks> plural and putting a single <TargetFramework>net6.0-android</TargetFramework> and restart VS to Debug the HelloMaui app.
I was able to use the Remote iOS Simulator this way using net6.0-ios, too. You would need to install the iOS workload on Windows, then install all the things on the Mac build host.
I think this would help anyone that comes across this, thanks.

I'm using VS 2019 16.11 preview and I am unable to build.  I have dotnet  6.0.100-preview.4.21255.9 installed.  Yet I get the error:
The framework 'Microsoft.NETCore.App', version '6.0.0-preview.4.21253.7' was not found.
I also noticed the ReadMe still reflects to 16.10.

Does your global.json match what is installed? Preview 4 final is a different number than what your error message says:
https://github.com/dotnet/maui-samples/blob/d2a8b835c8bd72938ec76c475fe7835802bfb1a4/global.json#L3

Good point.  I just cloned the repo night before last and it shows the version I posted.  Let me re-clone and see what happens.


Creating a restore cognito users runbook

This section formatting with # and ### doesn't seem to work:

By the way, will this appear on the wiki after merging?

This section formatting with # and ### doesn't seem to work:
By the way, will this appear on the wiki after merging?

I'm not sure we can check after merging and if it doesn't appear we can fix it?

Tested the runbook with Stefan. Everything works and looks good.
The email that is sent for restoration is really cryptic - it sends the username that should be used to login and the temporary password. I've asked Stefan to have a look to see if there is a way to customize the restoration email. This post might be a starting point: https://bobbyhadz.com/blog/aws-cognito-customize-emails.
The userpool that I tested (iva-test-pool) uses username to sign-in (See Sign-in Experience > Cognito user pool sign-in inside the userpool). Meanwhile the Cognito pools uses email for the sign in experience. Maybe that is why the email shows the username instead of email.
https://files.slack.com/files-pri/T0155DZFVM0-F04159PL0U8/image.png

Sounds like a big task, I've created a new ticket for it - https://biomage.atlassian.net/browse/BIOMAGE-2098


work-api KEP 2: Sync applied workload resource status

Enhancement proposal to update the Work API to solve https://github.com/kubernetes-sigs/work-api/issues/16
Signed-off-by: Mike Ng<EMAIL_ADDRESS>

/assign pmorie
/assign RainbowMango

CC @qiujian16 @yue9944882 @jnpacker @ryanzhang-oss

Just summarize the conclusion made on today's community meeting. (Please @mikeshng correct me If anything I get wrong)

We are going to work out a new configuration API to define the rules on how to collect status.
We can go back to the discussion about how to reference the new API and how to show the status in the Work API after the new API.
We got some wonderful comments from @pmorie that the new API should consider the QPS thing, such as the frequency the status should be synced, and how long the status should be kept.

Thank all of you folks who are contributing to this proposal, hope we can work out the final solution together!

Based on the community feedback, we have enhanced the proposal to introduce a new manifest configuration API that defines the rules and the frequency/threshold of the workload manifest resource status sync. Please take a look and provide feedback. Thanks. @RainbowMango @qiujian16 @yue9944882 @jnpacker @ryanzhang-oss

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale


Duplicate: values flagged as different when they are the same

CSV:
MarylandHospitals_993.zip
Fields in the CSV that are pushing into the target fields:
OID
Num_beds
Repro Steps:
Used the widget to load the attached CSV data into a target service
Then did it again so that all records would be duplicates -- opened one and saw that even though the data are the same from the user's perspective, they are getting flagged as different

Need to test.

Couldn't replicate.


Added more logging and fixed docs for search endpoint

Logging the requests for retrieving the tokens and refreshing the tokens will make it easier to debug issues that users have with the library. For example, we could've solved #605 faster if these log messages were emitted. I also fixed the docs for the search endpoint to mention that you can search for multiple types at the same time.

👍


Publish new version with 'Fix compatibilty with Tool Bar package'

Hey @thomaslindstrom, it would be fantastic if you could publish a new version that includes https://github.com/thomaslindstrom/color-picker/commit/335fce538b09064eea93483eefc14e74906fdb4a.
We ran into the same problems as https://github.com/suda/tool-bar/issues/130 over at  https://github.com/atom/atom/pull/11790 and would benefit from that fix as well.
See https://github.com/atom/atom/pull/11790#issuecomment-230154158 and following comments for some context.

Hi, sorry, I thought I'd done this. It's out in 2.2.0.

Awesome, thank you!


Documentation explaining how to deal with multi level nested fields

Hello,
I used awesome nested fields on a project that has multi level nested fields and was difficult to discover some things, so I documented how to do it.
Expect it helps.
Thanks.

Nice Juliana! You figured out the puzzle! lol
I moved your documentation to the wiki (along with the notes about multiple nested fields on the same page) since the README should contain just essencial information. I also edited some parts of your text and converted the code to JS (people can get really irrational emotional about CoffeeScript).
Thanks for your contribution!


Document the raw parameter

It should be documented.

Fixed by https://github.com/opscode-cookbooks/firewall/pull/54.


feat: add SMTP server and contact us route to flask API

Add SMTP server with Flask-Mail extension
Add error mailing in production config
Add new blueprint 'utils' to handle miscellaneous routes
Add asynchronous mailing function with threading
Add Flask API route for Contact Us form

@ivantha please check now


Added unpinned version test

What's changed?
Added a new test to validate the behavior of ChangeDependency on unpinned dependencies.
Although we cannot check managed versions of Gradle right now, if the previous dependency is managed, we are going to assume that the new one will be managed too, and do not add a version tag. That was the previous behavior of the recipe before recent changes.
However, with the the new parameter overrideManagedVersion it will always add the tag version to the new dependency.
This way we can force to add the newVersion, but we keep the default previous behavior.
What's your motivation?
This recipe is also used in rewrite-java-dependencies and rewrite-spring, and it was not clear the behavior of the individual recipe.
Checklist

[x] I've added unit tests to cover both positive and negative cases

@joanvr, there are actually 2 more checks on the version that need to be addressed.
With Gradle versioning we have to contend with 3 different rewrite types: J.Literal, G.GString, and G.MapEntry. Each of these should have this same check.

does it make sense to add it for GString? For what I see in the code it's expecting to be somewhat like "groupId:artifactId:${version}", so, we will always have a version there, right?

Fixed and allowing Map dependencies without a version (it was checking that version != null to apply the changes).
Fixed and added tests for GString dependencies, previously the GString was maintained and something like "newGroupId:newArtifactId:newVersion${variable}" was produced, keeping the variable at the end of the already full GAV coordinates.


[cloud] ssh shim part 5: run own mutagen daemon

Summary
This PR enables running devbox's own mutagen daemon by setting MUTAGEN_DATA_DIRECTORY.
Refactored code so that more of the devbox-specific logic is centralized in mutagenbox
package, which then calls the generic mutagen package.
How was it tested?
in progress...

Current dependencies on/for this PR:

main

PR #348

PR #349

PR #351

PR #352

PR #355   👈

This comment was auto-generated by Graphite.


Fixed the new flink container precommit

Addresses https://github.com/apache/beam/issues/33180

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Looks like I cannot trigger the new workflow file after I changed the workers to 2.


JWT Authentication

User Story
As a user, I want to log in securely using JWT so that I can access the application and be authenticated across multiple requests.
Description
Set up JWT authentication for login, ensuring tokens are generated and validated correctly.
Tasks:

[x] Implement JWT token generation and validation.
[x] Create a login API that returns JWT on successful authentication.
[x] Add middleware to check JWT on secure endpoints.

already done in authenticaton feature


explicitly specify 2.4 GHz

Fair enough - we do talk about the ability to use audio compression codecs to reduce airtime on 2.4 but good call out nonetheless!


Color filters not working on iOS for some animations on v6

Description
Some animations are not obeying color filters in v6.1.2 on iOS. Many animations still work as expected.
The problematic animation works as expected on Android.
Tested on:

iPhone SE + iOS 14.8.1
iPhone X + iOS 16.1.1
Pixel 4a + Android 13

Steps to Reproduce

Use Lottie_Navigation_Messages_Fix03.json.zip; note that the animation is red in the json

const animation = require('path/to/Lottie_Navigation_Messages_Fix03.json') as AnimationObject
const colorFilters = animation.layers.map((layer) => {
    return {
      keypath: layer.nm,
      color: '#ffffff',
    };
  })

<LottieView
  source={animation}
  colorFilters={colorFilters}
/>

Expected behavior:
https://github.com/lottie-react-native/lottie-react-native/assets/1574103/dd59cf62-5f4f-4f1c-bfbe-695ef71eb06f
Actual behavior:
https://github.com/lottie-react-native/lottie-react-native/assets/1574103/64ac6652-780e-4c9c-93ed-82cc0278d475
Minimal reproduction
See above
React Native Environment

RN 0.72.3
No Expo

Lottie Version
Version: 6.1.2
Workaround
I found an old discussion about color filters, and the workaround appears to mitigate the issue
const white = [1, 1, 1]

animation.layers?.forEach((layer, layerIndex) => {
  layer.shapes?.forEach((shape, shapeIndex) => {
    shape.it?.forEach((it, itIndex) => {
      if (it.c?.k != null) {
        animation.layers[layerIndex].shapes[shapeIndex].it[itIndex].c.k = white
      }
    })
  })
})

animation.assets?.forEach((asset, assetIndex) => {
  asset.layers?.forEach((layer, layerIndex) => {
    layer.shapes?.forEach((shape, shapeIndex) => {
      shape.it?.forEach((it, itIndex) => {
        if (it.c?.k != null) {
          animation.assets[assetIndex].layers[layerIndex].shapes[shapeIndex].it[itIndex].c.k = white
        }
      })
    })
  })
})

Hey!
Can you send me the result of your mapping? The object that is being passed to color filters.

@matinzd JSON.stringify(colorFilters) comes out to
[{"keypath":"IconAnimation_Messages_APPROVED (Justin Revision)","color":"#ffffff"}]

Good news: It looks like the color filters work correctly when NOT using flipped vectors.
Working version of the animation:
Lottie_Navigation_Messages_LottieFix02_v4.json.zip

Ok! Great. Thanks for updating us :)
Closing it as completed.

Just to be clear, I do believe there's a bug here; AFAIK there's no documentation indicating that flipped vectors are unsupported


"not verified" response - cognito

Hi, maybe I am missing something obvious, but when authenticating using an auth token from cognito, the fastapi-cloudauth responds with 401 {"detail": "Not verified"}
This does not refer to the user I guess, as the user is email & phone verified, but does it refer to the domain (localhost, local dev)?
How would I test my app locally then?
I hope you can point me in the right direction, as this package would be awesome to use!

There are two types of token available in Cognito: idToken and accessToken. Make sure that you are using correct one.

Can I check which of these is considered "correct"?
We've an API Gateway in front of an app. which uses fastapi-cloudauth and the Gateway seemingly accepts either. fastapi-cloudauth, however, seems only to accept the IdToken.
Not sure if I missed something in the docs
as to why that would be or if it's configurable…?

Ignore me: I RTFM'd a little harder and that first example finally clicked (AccessUser for the AccessToken; CognitoClaims for the IdToken.)

I've implemented the code exactly as listed in the documentation, but regardless of if I pass the idToken or accessToken I still receive {detail: 'Not verified'}.

@JLongley, can I check how you're getting the tokens? And how you're forming the requests?
I've just copy/pasted the example from the README and it's seemingly working for both ID and Access tokens.


Sometimes data is not displayed, when expanding a node

It happens in the cityofchicago resource, that the server sends a reply, but the expanded subtree is not displayed:

Very strange, I've never seen this before. I will need more context to find the problem.

I've reproduced this with another file, but I will need to investigate further.

It appears to me that this issue occurs only when maxLevel is set to 1 (as in the URL in the screenshot you posted). A simple workaround would be to not allow setting lazy-loading to 1. I think this is a satisfactory solution, do you agree?

Wouldn't this mean that only the children from the context-node are fetched from the server? So it should be a valid option?

I'm not sure what you mean, can you elaborate?

To elaborate further on the bug, the frontend appliaction is assuming that when the metadata is returned from the API, the value field of a metadata object with type object will be an array, not an object. This is in fact how the API behaves. However, when maxLevel=1, the value field is an empty object, which is not what the application expects.
I should be able to work around this issue, without needing to limit the possible values for maxLevel.

It seems to me that maxLevel=1 returns only the node specified. This means that if I hover the mouse over node=12 when maxLevel=1, then only node 12 is returned, and no subnodes. I think that we must disable a value of 0 for lazy loading, else it will be impossible to (lazy) load deeper. I also fixed the issue above (regarding expecting an array instead of an object), and saw that it was now impossible to lazy load. I do think that the solution is to not allow a maxLevel=1.


Warn when (re)using a river in an international boundary relation

Perhaps warn when (re)using a river in an international boundary relation.
(See e.g., Paradox of the Moving Boundary: Legal Heredity of River Accretion and Avulsion,)
or even on state/territory etc. borders. Also when using trails too perhaps. Telling the user "unless you are sure that they define the border as in treaties, etc.
However one might say that this is an area that goes beyond the job of the editing program. If so just close this bug.

@jidanni Yes, there can be issues with this pattern but I'd rather leave the decision up to mappers than make iD have throw a warning anywhere.


Request

Hi @nfa1 ,
Welcome to our community.
I see that you are forking repos in to @The-Lively-Developers-Community multiple times. I request you to kindly not fork it onto The-Lively-Developers-Community but instead to your profile.

My apologies for the error. I will take care to not fork repos to the community repo but to my profile. Thank you for the feedback.


MD032: Example in documentation uses lazy continuation and is not an example of a violation

Following pic shows the problem:

MD032 only shows up for the line above a list and ignores the one below.
It also only adds a blank line after Some text
According to https://github.com/DavidAnson/markdownlint/blob/main/doc/md032.md a blank line should be added before and after list items.

Ahh ok. Can MD032 be extended such that one can choose if lazy line continuation is allowed or not.
I think lazy line continuation makes the markdown file fairly hard to read.


We don't need CMake in the base build image

@dsazonoff see any issue with removing this?

@dsazonoff see any issue with removing this?

Everything looks good.


pass src_dir as a relative path to up.srpm()

This should fix a case in the service when the specfile is not in the root of the git repo.

Tests are passing, merging, Rado also verified this on stg.


Fix failing CodeQL

Autobuild does not work due to relying on Hermit environment setup.
https://github.com/block/example-java-kotlin-maven-multimodule/actions/runs/11847422492/job/33017060434
  Error: 1-14 23:45:12] [autobuild] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.0:compile (default-compile) on project java-kotlin-maven-example-api: Fatal error compiling: error: invalid target release: 21 -> [Help 1]

Looking good cc: @leordev @finn-block


Touch Input ignores LayerSurfaces

Touchscreen input ignores LayerSurfaces, making it impossible to interact with applications like eww, rofi, or wf-osk  via the touchscreen.

fixed among other things in 34cd8b125ad9f03ababd7789ebe12eac69e23227


Add new Salesforce Functions features in 1.37.0

@saracope , please review two new features in this week's RC for Salesforce Functions commands.  I found these when I regenerated the CLI Reference.  But there might be more new/changed features because a lot of functions PRs were included in the RC.  So feel free to add more!

FYI I'm going to merge this PR because the current RC contains these changes already.   Y'all can figure out the -j issue outside of this :)


Model request Qwen 2.5 7B 4bit

Highest capability models that can run on latest iPhone would be useful. The best I've found to fit in 8gb RAM is Qwen 2.5 7B 4bit?

check out my PR (#18), I'm adding the 3B and 0.5B models there

check out my PR (#18), I'm adding the 3B and 0.5B models there

Thanks, 7B would be great too


mobile_menu breaks after repeated reveal/hide

Tested in:

Safari
Firefox
Edge

@pascallaliberte self-assigned this 3 minutes ago

Thank you. 🙏

I initially thought this was caused by the introduction of turn.js transitions, but after testing with an older commit (and seeing what turn.js does), the problem pre-dates that new feature.
There's a race condition happening that wasn't there before. It could be caused by an update in the reveal stimulus controller. I plan to add an extra few steps in the system tests to check for this regression long-term.

@andrewculver I'll need to add ids to elements in the light theme to add a system test that covers this problem. This means I'll have two PRs: one in the bullet_train starter repo where system tests are found, and the other for changes to the light theme.
But maybe not all themes will have a mobile menu coded using stimulus-reveal like the light theme does. Can I add a system test to the light theme?

Yes, 100%!

This issue might also need a PR in this repo, for changes to the mobile_menu_controller.js Stimulus controller, in addition to changes in the light theme and in the starter repo.
It's messed up.
Here's what seems to be the problem: there's a bit of magic to what stimulus-reveal does and it's not obeying commands to re-hide or re-show elements if there are multiple elements to coordinate.
We animate these two elements separately:

the backdrop
the sliding menu

So I think I'll have to break those two elements into two instances of stimulus-reveal, and babysit the revealing/hiding of each instance by keeping check when both are done their little business.
More progress next week.

Matt Swanson's post and example on his boringrails site makes me want to ditch stimulus-reveal.

Sorry I missed this one when opening https://github.com/bullet-train-co/bullet_train-themes-light/issues/47, I'll go ahead and close that over there.


Semantic Tagging Task: Make partial for show recipe page

Add an article tag and prep for styling

Done! Added semantic tagging and 2 header tags to make the page look and feel a little more organized


Looks like great work

Haven't tried this out yet, but it looks like a pretty solid way to run an ephemeral containerized samba server and definitely an evolution from how it was done in dperson/samba. I really like the idea of defining the shares configuration in a separate, static yaml file that gets parsed by the entrypoint script. Only suggestion would be that some method be exposed for easy user configuration of the default [global] section. Right now it looks like you can only add global options, not override the defaults.

@jmpolom

Only suggestion would be that some method be exposed for easy user configuration of the default [global] section. Right now it looks like you can only add global options, not override the defaults.

Yes some of them are hardcoded but you can override each of them if you add a new value in the YAML config like server multi channel support = yes:
global:
  - "server multi channel support = no"

$ docker-compose logs samba
...
samba    | # Global parameters
samba    | [global]
samba    |      ...
samba    |      server multi channel support = No
samba    |      ...
...


Faster pandas build

Now the tox step is taking about 110 s instead of 718 s. Pandas has wheels available for all python versions we need, it was just not picked up without the wheel package it seems.
So the builds are back to 3-4 minutes with this PR.

Codecov Report

Merging #189 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #189   +/-   ##
=======================================
  Coverage   97.33%   97.33%
=======================================
  Files          41       41
  Lines        2513     2513
  Branches      477      477
=======================================
  Hits         2446     2446
  Misses         35       35
  Partials       32       32

Weird, I was assuming wheel package is only needed to build wheels, not to use them. Thanks for the fix!

Weird, I was assuming wheel package is only needed to build wheels, not to use them. Thanks for the fix!
hmm, actually wheel is not required - I just checked it with pandas locally, maybe there is something fishy here, let me double-check

Yeah, the change in this branch is not related to wheel speedup: https://travis-ci.org/TeamHG-Memex/eli5/builds/229393979?utm_source=github_status&utm_medium=notification
Probably the reason for speedup is that pandas wheels were uploaded yesterday (maybe late in the evening?): https://pypi.python.org/pypi/pandas/0.20.1


[consensus] Add universal committer

Description
The UniversalCommitter uses a collection of BaseCommitter's to commit a sequence of leaders. It will be configured to enable multi-leader & pipelining in Mysticeti.
Note: Feel free to ignore test util changes as most of that will be reviewed and checked in with PR#15806
Test Plan
Added a few basic tests to ensure things are working, but will add more tests in a follow up PR so this PR can remain small.

ak/universal_committer


Achieving UX parity with Android default "phone" app

React Native: 42
react-native-twilio-programmable-voice: 0.6.2
Device: Nexus 6 (Android 7)
What I am experiencing currently is any inbound call opens the app (if not already in foreground) and creates a notification in all app states (background, foreground, screen off).  The notification does not show on screen, just in the status bar.
The default Android phone app does things differently for inbound calls.

When device screen is off the phone app functions just like this library.  It opens the app in foreground to an "answer/reject" style page of the phone app itself.  It also puts a notification in the status bar (not onscreen).

But, when app is in foreground and background a notification is shown on screen, but the app does not come to foreground automatically.  If rejected, the app does not open and notification goes away.  If answered, the app comes to foreground and notification is modified.

I am not yet familiar enough with the Android API to modify the NotificationHelper.java code itself to achieve UX on parity with the default phone app.  Would this be controlled in the "getLaunchIntent"...or maybe the "NotificationCompat.Builder"s themselves?  I am not yet seeing where to modify this behavior.  Any pointers would be appreciated.

@jrperson in my app the incoming call notification opens the react native app. What is your AndroidManifest.xml?
The relevant code is this block:
https://github.com/hoxfon/react-native-twilio-programmable-voice/blob/master/android/src/main/java/com/hoxfon/react/TwilioVoice/NotificationHelper.java#L78-L107
Basically create an App launch intent to use when the notification is dealt with.
I think this is relevant:
<activity
            android:name=".MainActivity"
            android:label="@string/APP_NAME"
            android:launchMode="singleTop" //<--------
            android:configChanges="keyboard|keyboardHidden|orientation|screenSize"
            android:windowSoftInputMode="adjustResize"
            android:screenOrientation="portrait">

@fabriziomoscon Yes, that is the exact experience I am having "incoming call notification opens the react native app."  That is not what I want to have happen.  I only want the incoming call notification to trigger an opening of the react native app when the screen is off.  All other times (screen on/ app in background, screen on/app in foreground) I just want the notification to show onscreen (but the react native app does not open).  This is more in line with the default "phone" app behavior for Android.
Because what I am looking for is conditional behavior
if(screen off) {
  open app;
}
else {
  don't open app;
  just show notification on screen;
}

I assumed it wouldn't be a simple AndroidManifest.xml value.

Yes, sorry for the confusion. What you described it is what happens.
It might be good to have a session on hangout or seeing a GIF on what happens.
It might also be worth reading this thread:
https://github.com/hoxfon/react-native-twilio-programmable-voice/issues/12
Let's organise a time so I can help you.

@jrperson is this still an issue? Check out the latest branch which uses the latest Twilio SDK for iOS and Android.