Setting default for EmbeddingChatRequest.add_generation_prompt to False

Continuing the excellent work done on reward modeling in vLLM, I believe that the default for this parameter should be false.
It was likely copied over from the chat request, where adding the assistant turn before the beginning of the completion is desired by default. However, in embedding / reward modeling, the model is not expected to generate a textual response, but to output an embedding vector.
I'm not even sure that these parameters should be optional, but at the very least they should default to False.
@DarkLight1337 what do you think?

Feel free to remove the parameter altogether (and add a comment explaining why)

Created a different PR based on this discussion:
https://github.com/vllm-project/vllm/pull/10551

Closing as superseded by #10551


No error message shown when session token is expired

When a session token (https://docs.openshift.com/container-platform/3.3/rest_api/index.html#rest-api-session-tokens) is expired, there is no visual indicator that this has occurred. The fetch polls for resources fail silently in the background and nothing is communicated to the user.

@ibolton336  The issue description says, "When a session token is expired, there is visual indicator that this has occurred."  Is it supposed to say, "...there is no visual indicator..."? If so, I need some more background. How does the UI behave when the session token has expired? Sounds like it's frozen if fetch polls fail silently. Does that mean navigation is frozen too? Also, what is the corrective action a user needs to do - login to OCP again?
I'm thinking we should display a persistent toast notification explaining the error and redirecting (or linking) the user back to the OCP login page. But I don't know what's possible without better understanding what happens when the session token expires.

Hey @vconzola  oops yeah that was a typo. No visual indicator is shown. The session token expires after 24 hours.

@ibolton336 After reading through the PF documentation, this situation calls for a modal dialog error. Mockup below.

The Try again button should be primary; Cancel should be secondary. Let me know if you need details on any of the other PF components. Example here: https://www.patternfly.org/v4/design-guidelines/usage-and-behavior/modal#report-errors

It's possible that this is a regression. I had code that should redirect you to the oauth page to get a refreshed token at the client layer. Need to look into this.

@eriknelson i have seen the error occur only when the token expires and there is still a current user local storage token existing within the users browser. It’s a tough one to reproduce. A similar but separate issue is the lack error shown when the fetches fail for other connectivity reasons non auth related.

This is the edge case when a user has a stale token & tries to log into the UI

#685


HTML help text rendered incorrectly on Android

Here's a plunker example of a form that contains some HTML help fields. On a desktop browser, everything works as expected but on some mobile browsers, some HTML fields are not rendered.
The fields in the form are
[
    "name",
    {
        type: "help",
        helpvalue: "first"
    }, {
        type: "help",
        helpvalue: "second
heading
"
    }, {
        type: "help",
        helpvalue: "third bold"
    }
];

Heres' a screenshot of how the form is rendered on this Android browser.

Notice that the first and third help fields are rendered correctly, but the second one with content
helpvalue: "second
heading
"

is omitted. When the form is rendered on a desktop browser all 3 fields are rendered correctly.

I can workaround this problem by wrapping everything in a
 e.g. change
helpvalue: "second
heading
"

to
helpvalue: "
second
heading

"

Perhaps the problem with the original markup is that it's invalid, i.e. a block element is nested within an inline element, but it's strange that the problem should manifest itself only on mobile browsers.
At the very least, I think the docs should be updated to explain what kind of markup is considered valid.

Hi @donalmurtagh, I'm not sure that I'm following what you mean by "the original markup is that it's invalid". Sure invalid markup can do strange things and behave differently on different browsers, but I don't see any invalid markup in your plunkr... what am I missing?
Anyway the helpvalue template is very simple

So any html that ng-bind-html likes should be fine!

If the helpvalue template was something like

then a helpvalue of
helpvalue: "second
heading
"

would lead to invalid markup because you'd have a
 nested inside a . But now that I know what the helpvalue template actually looks like, you can ignore that remark.
So I've no idea why this help field is not rendered correctly on the Android browser. Were you able to reproduce the problem?

My samsung stock browser has the issue, but not chrome on my Android. I haven't the faintest idea why though :)


The lip shakes so much

@Weizhi-Zhong
Thanks for your extraordinary work. But when I run the code, I found the lip shakes so much. There may be lack of some postprocess? The result as following:
https://drive.google.com/file/d/1L6XfTZKV_nvqv1FW3fFp1-9quOUgALYe/view?usp=sharing

I did not modify the code. Run the following command and installation tutorial as you mentioned in the README: CUDA_VISIBLE_DEVICES=0 python inference_single.py --input ./test/template_video/129.mp4 --audio ./test/template_video/audio2.wav @Weizhi-Zhong
I'm so confused
do you install the correct version of torch and so on?

Yes, torch==1.10.0 torchvision==0.11.0

Hi~, I ran the command "CUDA_VISIBLE_DEVICES=0 python inference_single.py" as described in README.md, and I got the result on my machine with gpu RTX3090 like:
https://1drv.ms/v/s!Amqu9u09qiUGi7coEZFOuiL9P9xx_Q?e=wOwMDs
Besides torch==1.10.0 torchvision==0.11.0, the version of other packages may affect the result.
Please refer to requirement.txt. Thanks

--audio ./test/template_video/audio2.wav

Hi~, from your filename of GoogleDrive:"666result_N_25_Nl_15.mp4", it seems that your input_video_path is set to 666.mp4. Refer to the following code in inference_single.py
outfile_path = os.path.join(output_dir,'{}N{}Nl{}.mp4'.format(input_video_path.split('/')[-1][:-4] + 'result', ref_img_N, Nl))
But 666.mp4 is the file on my machine that is too large to be uploaded to GitHub, Sorry~
Just now, I update the inference_single.py, modifying the default path to the following:
parser.add_argument('--input', '--input_template_video', type=str, default='./test/template_video/129.mp4')

--audio ./test/template_video/audio2.wav

Hi~, from your filename of GoogleDrive:"666result_N_25_Nl_15.mp4", it seems that your input_video_path is set to 666.mp4. Refer to the following code in inference_single.py outfile_path = os.path.join(output_dir,'{}N{}Nl{}.mp4'.format(input_video_path.split('/')[-1][:-4] + 'result', ref_img_N, Nl))
But 666.mp4 is the file on my machine that is too large to be uploaded to GitHub, Sorry~
Just now, I update the inference_single.py, modifying the default path to the following: parser.add_argument('--input', '--input_template_video', type=str, default='./test/template_video/129.mp4')

@clumsynope

ok, I try it now.

ok, I try it now.

Hi~, does it work?

Sorry for replying to you late. The problem is fixed.


[PDPI] Renamed @unused to @unsupported.

PR Description :
[PDPI] Renamed @unused to @unsupported.
Build Results :
bibhuprasad@26c7ccef2fda:/sonic/src/sonic-p4rt/sonic-pins$ bazel build $BAZEL_BUILD_OPTS ...
INFO: Analyzed 257 targets (1 packages loaded, 49 targets configured).
INFO: Found 257 targets...
INFO: Elapsed time: 122.183s, Critical Path: 34.05s
INFO: 121 processes: 11 internal, 110 linux-sandbox.
INFO: Build completed successfully, 121 total actions
bibhuprasad@26c7ccef2fda:/sonic/src/sonic-p4rt/sonic-pins$
bibhuprasad@26c7ccef2fda:/sonic/src/sonic-p4rt/sonic-pins$
UT Results :
bibhuprasad@26c7ccef2fda:/sonic/src/sonic-p4rt/sonic-pins$ bazel test $BAZEL_BUILD_OPTS p4_pdpi/...
INFO: Analyzed 84 targets (0 packages loaded, 0 targets configured).
INFO: Found 48 targets and 36 test targets...
INFO: Elapsed time: 1.625s, Critical Path: 0.38s
INFO: 18 processes: 30 linux-sandbox.
INFO: Build completed successfully, 18 total actions
//p4_pdpi:p4_runtime_matchers_test                              (cached) PASSED in 0.6s
//p4_pdpi:sequencing_util_test                                  (cached) PASSED in 0.5s
//p4_pdpi/netaddr:ipv4_address_and_network_address_test         (cached) PASSED in 0.2s
//p4_pdpi/netaddr:ipv6_address_test                             (cached) PASSED in 0.2s
//p4_pdpi/netaddr:mac_address_test                              (cached) PASSED in 0.2s
//p4_pdpi/packetlib:packetlib_fuzzer_test                       (cached) PASSED in 15.4s
//p4_pdpi/packetlib:packetlib_matchers_test                     (cached) PASSED in 0.2s
//p4_pdpi/packetlib:packetlib_test                              (cached) PASSED in 0.1s
//p4_pdpi/packetlib:packetlib_test_runner                       (cached) PASSED in 0.0s
//p4_pdpi/packetlib:packetlib_unit_test                         (cached) PASSED in 0.9s
//p4_pdpi/string_encodings:bit_string_test                      (cached) PASSED in 0.2s
//p4_pdpi/string_encodings:byte_string_test                     (cached) PASSED in 0.2s
//p4_pdpi/string_encodings:decimal_string_test                  (cached) PASSED in 0.2s
//p4_pdpi/string_encodings:decimal_string_test_runner           (cached) PASSED in 0.0s
//p4_pdpi/string_encodings:hex_string_test                      (cached) PASSED in 0.0s
//p4_pdpi/string_encodings:hex_string_test_runner               (cached) PASSED in 0.0s
//p4_pdpi/string_encodings:readable_byte_string_test            (cached) PASSED in 0.2s
//p4_pdpi/testing:mock_p4_runtime_server_test                   (cached) PASSED in 0.8s
//p4_pdpi/utils:annotation_parser_test                          (cached) PASSED in 0.2s
//p4_pdpi:ir_tools_test                                                  PASSED in 0.3s
//p4_pdpi/testing:helper_function_test                                   PASSED in 0.3s
//p4_pdpi/testing:info_test                                              PASSED in 0.1s
//p4_pdpi/testing:info_test_runner                                       PASSED in 0.0s
//p4_pdpi/testing:main_pd_test                                           PASSED in 0.0s
//p4_pdpi/testing:packet_io_test                                         PASSED in 0.1s
//p4_pdpi/testing:packet_io_test_runner                                  PASSED in 0.0s
//p4_pdpi/testing:rpc_test                                               PASSED in 0.1s
//p4_pdpi/testing:rpc_test_runner                                        PASSED in 0.0s
//p4_pdpi/testing:sequencing_test                                        PASSED in 0.1s
//p4_pdpi/testing:sequencing_test_runner                                 PASSED in 0.1s
//p4_pdpi/testing:sequencing_util_test                                   PASSED in 0.0s
//p4_pdpi/testing:sequencing_util_test_runner                            PASSED in 0.0s
//p4_pdpi/testing:table_entry_gunit_test                                 PASSED in 0.4s
//p4_pdpi/testing:table_entry_test                                       PASSED in 0.1s
//p4_pdpi/testing:table_entry_test_runner                                PASSED in 0.1s
//p4_pdpi/utils:ir_test                                                  PASSED in 0.3s
Executed 17 out of 36 tests: 36 tests pass.
INFO: Build completed successfully, 18 total actions
bibhuprasad@26c7ccef2fda:/sonic/src/sonic-p4rt/sonic-pins$ bazel test $BAZEL_BUILD_OPTS p4rt_app/...
INFO: Analyzed 107 targets (0 packages loaded, 0 targets configured).
INFO: Found 68 targets and 39 test targets...
INFO: Elapsed time: 32.111s, Critical Path: 5.09s
INFO: 34 processes: 1 internal, 18 linux-sandbox, 15 local.
INFO: Build completed successfully, 34 total actions
//p4rt_app/sonic:app_db_acl_def_table_manager_test              (cached) PASSED in 0.5s
//p4rt_app/sonic:state_verification_test                        (cached) PASSED in 0.2s
//p4rt_app/sonic/adapters:fake_sonic_db_table_test              (cached) PASSED in 0.1s
//p4rt_app/tests/lib:app_db_entry_builder_test                  (cached) PASSED in 0.0s
//p4rt_app/utils:event_data_tracker_test                        (cached) PASSED in 0.0s
//p4rt_app/utils:table_utility_test                             (cached) PASSED in 0.7s
//p4rt_app/event_monitoring:app_state_db_port_table_event_test           PASSED in 0.5s
//p4rt_app/event_monitoring:app_state_db_send_to_ingress_port_table_event_test PASSED in 0.5s
//p4rt_app/event_monitoring:config_db_node_cfg_table_event_test          PASSED in 0.5s
//p4rt_app/event_monitoring:config_db_port_table_event_test              PASSED in 0.5s
//p4rt_app/event_monitoring:debug_data_dump_events_test                  PASSED in 0.5s
//p4rt_app/event_monitoring:state_verification_events_test               PASSED in 0.5s
//p4rt_app/p4runtime:ir_translation_test                                 PASSED in 0.4s
//p4rt_app/p4runtime:p4info_verification_schema_test                     PASSED in 0.4s
//p4rt_app/p4runtime:p4info_verification_test                            PASSED in 0.4s
//p4rt_app/p4runtime:packetio_helpers_test                               PASSED in 0.5s
//p4rt_app/sonic:app_db_manager_test                                     PASSED in 0.5s
//p4rt_app/sonic:app_db_to_pdpi_ir_translator_test                       PASSED in 0.4s
//p4rt_app/sonic:hashing_test                                            PASSED in 0.5s
//p4rt_app/sonic:packetio_impl_test                                      PASSED in 0.3s
//p4rt_app/sonic:packetio_port_test                                      PASSED in 0.2s
//p4rt_app/sonic:response_handler_test                                   PASSED in 0.4s
//p4rt_app/sonic:vrf_entry_translation_test                              PASSED in 0.4s
//p4rt_app/tests:acl_table_test                                          PASSED in 1.1s
//p4rt_app/tests:action_set_test                                         PASSED in 0.7s
//p4rt_app/tests:api_access_test                                         PASSED in 0.7s
//p4rt_app/tests:arbitration_test                                        PASSED in 0.8s
//p4rt_app/tests:debug_data_dump_test                                    PASSED in 0.7s
//p4rt_app/tests:fixed_l3_tables_test                                    PASSED in 2.6s
//p4rt_app/tests:forwarding_pipeline_config_test                         PASSED in 3.9s
//p4rt_app/tests:grpc_behavior_test                                      PASSED in 5.1s
//p4rt_app/tests:p4_constraints_test                                     PASSED in 0.1s
//p4rt_app/tests:p4_constraints_test_runner                              PASSED in 0.2s
//p4rt_app/tests:p4_programs_test                                        PASSED in 1.2s
//p4rt_app/tests:packetio_test                                           PASSED in 3.4s
//p4rt_app/tests:port_name_and_id_test                                   PASSED in 2.5s
//p4rt_app/tests:response_path_test                                      PASSED in 2.9s
//p4rt_app/tests:role_test                                               PASSED in 0.8s
//p4rt_app/tests:state_verification_test                                 PASSED in 1.1s
Executed 33 out of 39 tests: 39 tests pass.
INFO: Build completed successfully, 34 total actions
bibhuprasad@26c7ccef2fda:/sonic/src/sonic-p4rt/sonic-pins$

Co-authored-by: Steffen Smolka<EMAIL_ADDRESS>

@kishanps, Could you please review.
Thanks.


Update README.md

Application for Registering Team to CODE CAMP 1.0
This pull request template helps you complete an application to the CODE CAMP 1.0 HACKATHON. Use the checklist below to verify you have followed the instructions correctly.
Checklist ✅

[] I have read the instructions on the README file before submitting my application.
[] I have Submitted my team's details by making a folder of my team as instructed in PULL REQUEST VIDEO.
[] I have used the Markdown file template to add my information for the Hackathon.
[] I have given all the necessary details as mentioned in template index.md file and the details are correct and best to my knowledge.
[] I understand that a reviewer will merge my pull request after examining it or ask for changes in case needed.
[] I understand I should not tag or add a reviewer to this Pull Request.
[] I understand the Details added to the template will be used as a means of communication at the time of result declaration.
[] I have added the event to my Calendar.

@harshchoudhary272
Do not Spam the PR Section
I am Right Now closing this PR. As the changes you are making is unnecessary.
If you want to Contribute Do participate with your team. and register as told.
Regard,
Utkarsh
GitHub Moderator [CODE CAMP]
@ISTESRMNCR

I don't have any team and I want to participate on my own
On Sat, 11 Jul 2020, 19:01 Utkarsh Yadav<EMAIL_ADDRESS>wrote:

@harshchoudhary272 https://github.com/harshchoudhary272
Do not Spam the PR Section
I am Right Now closing this PR. As the changes you are making is
unnecessary.
If you want to Contribute Do participate with your team. and register as
told.
Regard,
Utkarsh
GitHub Moderator [CODE CAMP]
@ISTESRMNCR https://github.com/ISTESRMNCR
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ISTESRMNCR/CODE-CAMP-2020/pull/7#issuecomment-657062679,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLQFSQ5MIMQXUMAJASUPN3R3BSTFANCNFSM4OXH6CPA
.

@harshchoudhary272 ,Yes! You can participate Solo

Please tell me the steps
On Sat, 11 Jul 2020, 19:11 Utkarsh Yadav<EMAIL_ADDRESS>wrote:

@harshchoudhary272 https://github.com/harshchoudhary272 ,Yes! You can
participate Solo
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ISTESRMNCR/CODE-CAMP-2020/pull/7#issuecomment-657064183,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLQFSVZD7TXOGGTNF5R2PTR3BTXJANCNFSM4OXH6CPA
.

Check it out Here : https://codecamphackathon.netlify.app/
The pull request will be merged from 19th July, 2020....
Themes will be  displayed on 15th JULY, 2020... on social media platform . as well as on website.

How do I contribute ?
On Sat, 11 Jul 2020, 19:30 Utkarsh Yadav<EMAIL_ADDRESS>wrote:

Check it out Here : https://codecamphackathon.netlify.app/
The pull request will be merged from 19th July, 2020....
Themes will be displayed on 15th JULY, 2020... on social media platform .
as well as on website.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ISTESRMNCR/CODE-CAMP-2020/pull/7#issuecomment-657067256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLQFSWIXKMITLQABLRTTADR3BV6HANCNFSM4OXH6CPA
.

@harshchoudhary272 , Nothing to contribute for Now.
Only contribute as a Participant.. Do participate in the Hackathon. That would be a great contribution.
Everything is on website. Kindly Just go through it...
These PR session has been closed. So messages here will not be Replied.


AEP Ohio set up error

I removed my AEP Ohio integration and re-added it, wondering if this would detect my second meter that just got installed on my barn.  However, when trying to re-add my account, I get a failure and this error in the logs.
I may be able to work on a fix, but you have any ideas on where I'd start to work on this?
Logger: homeassistant.config_entries
Source: config_entries.py:604
First occurred: 3:51:57 PM (1 occurrences)
Last logged: 3:51:57 PM

Error setting up entry AEP Ohio (<redacted>) for opower
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 604, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/opower/__init__.py", line 19, in async_setup_entry
    await coordinator.async_config_entry_first_refresh()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 281, in async_config_entry_first_refresh
    await self._async_refresh(
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 354, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/opower/coordinator.py", line 82, in _async_update_data
    await self.api.async_login()
  File "/usr/local/lib/python3.12/site-packages/opower/opower.py", line 199, in async_login
    self.access_token = await self.utility.async_login(
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/opower/utilities/aepbase.py", line 158, in async_login
    await async_auth_saml(session, url)
  File "/usr/local/lib/python3.12/site-packages/opower/utilities/helpers.py", line 27, in async_auth_saml
    async with session.get(
  File "/usr/local/lib/python3.12/site-packages/aiohttp/client.py", line 1353, in __aenter__
    self._resp = await self._coro
                 ^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/aiohttp/client.py", line 684, in _request
    await resp.start(conn)
  File "/usr/local/lib/python3.12/site-packages/aiohttp/client_reqrep.py", line 999, in start
    message, payload = await protocol.read()  # type: ignore[union-attr]
                       ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/aiohttp/streams.py", line 640, in read
    await self._waiter
asyncio.exceptions.CancelledError

Errrr ... never mind, I just hit reload on the integration entry and it is now connecting 🤷


Allow Rest modules to dynamically create credentials

In situations where CAS is using a custom type of credential, allow the rest modules as well to dynamically locate a credential type and bind that to the request.

Method to obtain credentials are marked as protected in both rest and restlet module. They can be extended to provide custom types.


Problems with the treemap Algorithem.

Trying to follow the examples i met a strange glitch in the treemap code, can't seem to find the problem.
here is a fiddle with the problem:
https://fiddle.jshell.net/40xoLecy/
same code with different data works well:
https://fiddle.jshell.net/40xoLecy/3/
what seems to be the problem with the layout? why is it a a pile of rects?

after playing a bit with the data, i found out the 0 is what makes the pile problem , why aren't 0 allowed?

Unrelated to your issue (which I am investigating), but you don’t need d3.stratify to create a flat tree. You can use d3.hierarchy:
var rootNode = d3.hierarchy({children: clonedData}).sum(d => d);

Thank you for the quick fix, any date on the release?


Documentation for custom field. custom.md

[ ] step-by-step example of the creation one custom field.
[ ] add tests for custom field
[ ] explain all of the function the can be overwritten from SimpleString

we don't need it for now. It will be explained in the article with advanced usage.


Revert "Add CORS headers to dream server to ease integration with third-party web interfaces"

This reverts commit 91e826e5f425333674d1e3bec1fa1ac63cfb382d.
The cross-origin policy is an important security feature which #363 disabled entirely. Adding these headers means that any website you visit can talk to the web server, including submitting requests. That's certainly not something I'd expect to happen by default, and and our codebase hasn't really been built with the level of attention to security necessary for that to be at all safe (cf https://github.com/lstein/stable-diffusion/pull/133).
Perhaps this functionality can be added back at a later time as an opt-in under a flag like --dangerously-disable-cors, or where the flag allows you to specify a specific origin rather than allowing literally anything, but best to start by reverting this and then re-doing it in a safe way.
cc @SebastianAigner

Fair. I’ll get a new PR up that puts this behaviour behind a flag if that works for the project.

@SebastianAigner I don't think the precise behavior in your original PR is a good idea, even behind a flag, but a flag like --cors-allow-origin=example.com which allows you to name a specific origin would be fine, I think.


500 Internal Server Error printing invoices after v0.41 upgrade

After doing an upgrade from v0.41 a "500 Internal Server Error" happens when clicking when trying to print templates or generate pdf.
It should be noted that with a fresh siwapp-sf3 install before importing data from v.041 this error doesn't happen, so it must be related to the migration.
Error details from /var/logs/prod.log:
[2016-11-08 23:17:00] request.INFO: Matched route "{route}". {"route":"invoice_show_print","route_parameters":{"_controller":"Siwapp\\InvoiceBundle\\Controller\\InvoiceController::showPrintAction","id":"428","_route":"invoice_show_print"},"request_uri":"/web/invoice/428/show/print","method":"GET"} [] [2016-11-08 23:17:00] security.DEBUG: Read existing security token from the session. {"key":"_security_main"} [] [2016-11-08 23:17:00] security.DEBUG: User was reloaded from a user provider. {"username":"admin","provider":"FOS\\UserBundle\\Security\\UserProvider"} [] [2016-11-08 23:17:00] request.CRITICAL: Uncaught PHP Exception Twig_Error_Runtime: "An exception has been thrown during the rendering of a template ("DateTimeZone::__construct(): Unknown or bad timezone (es_ES)") in "SiwappInvoiceBundle:Invoice:print.html.twig" at line 94." at /var/cache/prod/classes.php line 5783 {"exception":"[object] (Twig_Error_Runtime(code: 0): An exception has been thrown during the rendering of a template (\"DateTimeZone::__construct(): Unknown or bad timezone (es_ES)\") in \"SiwappInvoiceBundle:Invoice:print.html.twig\" at line 94. at /var/cache/prod/classes.php:5783, Exception(code: 0): DateTimeZone::__construct(): Unknown or bad timezone (es_ES) at /var/cache/prod/classes.php:4942)"} [] [2016-11-08 23:17:00] security.DEBUG: Stored the security token in the session. {"key":"_security_main"} []

This seems to be related to issue date and due date variables. When removed from the template, it works.

how was those variables printed?
From the error message above i see
Unknown or bad timezone (es_ES)
so something funny goes on there

@ParisLiakos Here is the line (94):
`
Issue date:
{{invoice.issue_date|date('dd-MM-yyyy','es_ES')}}

`
This was on the default siwapp-sf1 template.

Yes, that definitely does not work now. Replace it with
 Issue date: {{ invoice.issue_date|localizeddate('medium', 'none', app.request.locale) }}

I guess I would also need to replace the template code for the due date.
What is the due date variable?
Thank you!

due_date ;)
Just use the same with above, then replace issue with due..should work

That code shows the dates as: "Nov 9, 2016". Is there any way to change this to format dd-MM-yyyy?

Answer: it's {{invoice.issue_date|localizeddate('medium', 'none', app.request.locale, null, 'dd-MM-yyyy')}}
@ParisLiakos What is the variable for due (pending) amount? {{invoice.due_amount|localizedcurrency(settings.currency, app.request.locale)}} shows 0 even if the invoice is still unpaid.

Hi, try with invoice.getDueAmount().
due_amount probably doesnt work because it is a computed value

@ParisLiakos It worked! Thank you very much.
BTW, It would be useful if these variables were documented somewhere, so people know them to modify their templates.

true! opened #17

I have done a migration from 0.4 to 1x and I cannot print invoices. However I cannot see any logs of the error.
O have also change the line commented in order to see if it was the problem. Do you have a default invoice template?

You can find the default here
https://raw.githubusercontent.com/siwapp/siwapp-sf3/master/src/Siwapp/InvoiceBundle/Resources/views/Invoice/print.html.twig

Using that file I am obtaining
Oops! An Error Occurred
The server returned a "500 Internal Server Error".
Something is broken. Please let us know what you were doing when this error occurred. We will fix it as soon as possible. Sorry for any inconvenience caused.
How can I track the error?

Finally I found the error log
[2018-03-13 06:53:34] snappy.DEBUG: Output from HTML ( <h). [] []
[2018-03-13 06:53:34] request.CRITICAL: Uncaught PHP Exception Symfony\Component\Process\Exception\RuntimeException: "The process has been signaled with signal "6"." at /var/www/siwapp-sf3/vendor/symfony/symfony/src/Symfony/Component/Process/Process.php line 396 {"exception":"[object] (Symfony\Component\Process\Exception\RuntimeException(code: 0): The process has been signaled with signal "6". at /var/www/siwapp-sf3/vendor/symfony/symfony/src/Symfony/Component/Process/Process.php:396)"} []

It was a problem with wkhtmltopdf version. I installed last version
on Ubuntu 16.04 (https://gist.github.com/srmds/2507aa3bcdb464085413c650fe42e31d) and now it is working fine. Thanks!


Usage with other loop based library

Hi,
I am considering CPPurses. I looked at the code samples and it looks pretty promising. Great Work!
In my workflow I use ros (Robot Operating System). ROS is an event loop based library.
A sample loop looks something like
while( ros::ok() )
{
    rate.sleep(10)
    ros::spinOnce();
}

My question is:
Can CPPurses be used with ROS? If yes could you give me some pointers to get it running?

I'm a bit late here, but I'll still answer in case it helps anyone.
The Event_loop class can be used to create a separate thread from the main UI thread, an event loop object can be placed within some Widget type you create. The void Event_loop::run_async(Fn loop_function) function will launch a new thread and will repeatedly call the loop_function parameter with an Event_queue& parameter.
So in your example you could do something like this:
struct My_widget : Widget {
    Event_loop loop_;
    My_widget() {
        loop_.run_async([this] (auto& queue) {
            if (!ros::ok())
                loop_.exit(1);
            rate.sleep(10);
            ros::spinOnce();
            queue.append(Paint_event{*this});
        });
    }
    auto paint_event(Painter& p) -> bool override {
        // use ROS data here in a thread safe way to update display.
        return Widget::paint_event(p);
    }
};

Every time the loop function returns, the event queue has a chance to process any events appended to it. In this case a Paint_event is appended, but in a more complex interaction, a Custom_event can be created, which can store a copy of the data from the event loop to be processed and displayed later, without any shared data threading issues.


Docker container "orderer.techracers.com" is Exited

When './buildERC20TokenNetwork.sh up' command executed, the error occured.
Error: failed to create deliver client: orderer client failed to connect to orderer.techracers.com:7050: failed to create new connection: context deadline exceeded
!!!!!!!!!!!!!!! Channel creation failed !!!!!!!!!!!!!!!!
========= ERROR !!! FAILED to execute End-2-End Scenario ===========

detail
./buildERC20TokenNetwork.sh up
Starting for channel 'mychannel' with CLI timeout of '10' seconds and CLI delay of '3' seconds
Continue? [Y/n] Y
proceeding ...
./buildERC20TokenNetwork.sh: line 46: /Users/ogasawara/hyperledger-fabric/ERC20-Token-On-Hyperledger/network/../bin/configtxlator: cannot execute binary file
LOCAL_VERSION=
DOCKER_IMAGE_VERSION=1.2.1
=================== WARNING ===================
  Local fabric binaries and docker images are
  out of  sync. This may cause problems.
===============================================
peer1.org2.techracers.com is up-to-date
Starting orderer.techracers.com ...
peer0.org2.techracers.com is up-to-date
peer1.org1.techracers.com is up-to-date
Starting orderer.techracers.com ... done
cli is up-to-date

 ____    _____      _      ____    _____
/ ___|  |_   _|    / \    |  _ \  |_   _|
\___ \    | |     / _ \   | |_) |   | |
 ___) |   | |    / ___ \  |  _ <    | |
|____/    |_|   /_/   \_\ |_| \_\   |_|

Channel name : mychannel
Creating channel...
+ peer channel create -o orderer.techracers.com:7050 -c mychannel -f ./channel-artifacts/channel.tx --tls true --cafile /opt/gopath/src/github.com/hyperledger/fabric/peer/crypto/ordererOrganizations/techracers.com/orderers/orderer.techracers.com/msp/tlscacerts/tlsca.techracers.com-cert.pem
+ res=1
+ set +x
Error: failed to create deliver client: orderer client failed to connect to orderer.techracers.com:7050: failed to create new connection: context deadline exceeded
!!!!!!!!!!!!!!! Channel creation failed !!!!!!!!!!!!!!!!
========= ERROR !!! FAILED to execute End-2-End Scenario ===========

ERROR !!!! Test failed

Docker container "orderer.techracers.com" is Exited, that causes the connection failer.
I checked docker cantainer log.
initializeLocalMsp -> CRIT 002 Failed to initialize local MSP: could not load a valid signer certificate from directory /var/hyperledger/orderer/msp/signcerts: stat /var/hyperledger/orderer/msp/signcerts: no such file or directory

How can I up the container "orderer.techracers.com" ?

The problem is that I used Python3.x.
After I change Python3.x to Python 2.x, the problem was solved.
Thank you


New package: IntelGEMM v0.1.0

Registering package: IntelGEMM
Repository: https://github.com/AStupidBear/IntelGEMM.jl
Created by: @AStupidBear
Version: v0.1.0
Commit: db547d88369761cb9179bb03259de60615f20ff2
Reviewed by: @AStupidBear
Reference: https://github.com/AStupidBear/IntelGEMM.jl/commit/db547d88369761cb9179bb03259de60615f20ff2#commitcomment-38747361

Name is fine. Ping me in 3 days.

[noblock] @DilumAluthge


Achtung Vs Z-levels

I'm getting an error every time I try and get my colonist to do anything directly, pick up weapons, prioritize anything, tame ect. The ones its claiming to have trouble with haven't given me an issue before except for the recently installed Z-levels for 1.3 (https://github.com/RamRod6198/Z-Levels) My only guess of why it's having this issue is because Achtung can't account for the ladders and being able to be a level above the main map, and thus is freaking out. Again, this is a guess. For all I know one of these other mods updated and it's creating an issue now. But it claimed to be Z-levels to be the most likely culprit so I titled this report appropriately.Here's the error if it's any help.Exception in RimWorld.FloatMenuMakerMap.ChoicesAtFor: System.NullReferenceException: Object reference not set to an instance of an object  at AchtungMod.FloatMenuMakerMap_AddJobGiverWorkOrders_Patch.Postfix (Verse.Pawn pawn, AchtungMod.ForcedWork __state) [0x0000e] in <5115af586046446eb35f0ed877695204>:0   at RimWorld.FloatMenuMakerMap.AddJobGiverWorkOrders (UnityEngine.Vector3 clickPos, Verse.Pawn pawn, System.Collections.Generic.List`1[T] opts, System.Boolean drafted) [0x00ee2] in <81af9f8a18324e17b286924a43555a7c>:0      - transpiler net.pardeike.rimworld.mods.achtung: IEnumerable`1 AchtungMod.FloatMenuMakerMap_AddJobGiverWorkOrders_Patch:Transpiler(IEnumerable`1 instructions, MethodBase original)     - prefix ZLevels.Mod: Boolean ZLevels.FloatMenuPatches+FloatMenuMakerMap_AddJobGiverWorkOrders_Patch:Prefix(Vector3 clickPos, Pawn pawn, List`1& opts, Boolean drafted, FloatMenuOption[]& ___equivalenceGroupTempStorage)     - prefix net.pardeike.rimworld.mods.achtung: Void AchtungMod.FloatMenuMakerMap_AddJobGiverWorkOrders_Patch:Prefix(Pawn pawn, ForcedWork& __state)     - prefix roolo.whatthehack: Boolean WhatTheHack.Harmony.FloatMenuMakerMap_AddJobGiverWorkOrders:Prefix(Pawn pawn)     - postfix net.pardeike.rimworld.mods.achtung: Void AchtungMod.FloatMenuMakerMap_AddJobGiverWorkOrders_Patch:Postfix(Pawn pawn, ForcedWork __state)  at RimWorld.FloatMenuMakerMap.AddUndraftedOrders (UnityEngine.Vector3 clickPos, Verse.Pawn pawn, System.Collections.Generic.List`1[T] opts) [0x000d9] in <81af9f8a18324e17b286924a43555a7c>:0      - prefix OskarPotocki.VFECore: Boolean AnimalBehaviours.VanillaExpandedFramework_FloatMenuMakerMap_AddUndraftedOrders_Patch:AvoidGeneralErrorIfPawnIsAnimal(Pawn pawn)     - postfix Children: Void Children.ChildrenHarmony+FloatMenuMakerMap_Patch:AddUndraftedOrders_Post(Vector3 clickPos, Pawn pawn, List`1 opts)  at RimWorld.FloatMenuMakerMap.ChoicesAtFor (UnityEngine.Vector3 clickPos, Verse.Pawn pawn, System.Boolean suppressAutoTakeableGoto) [0x000ee] in <81af9f8a18324e17b286924a43555a7c>:0      - postfix OskarPotocki.VFECore: Void VFE.Mechanoids.HarmonyPatches.FloatMenuMakerMap_ChoicesAtFor_Patch:Postfix(List`1& __result, Vector3 clickPos, Pawn pawn, Boolean suppressAutoTakeableGoto)     - postfix net.pardeike.rimworld.mods.achtung: Void AchtungMod.FloatMenuMakerMap_ChoicesAtFor_Postfix:Postfix(List`1 __result, Vector3 clickPos, Pawn pawn)     - postfix Shakesthespeare.RimsecSecurity: Void RimsecSecurity.FloatMenuMakerMap_ChoicesAtFor:Postfix(List`1& __result, Vector3 clickPos, Pawn pawn)     - postfix roolo.whatthehack: Void WhatTheHack.Harmony.FloatMenuMakerMap_ChoicesAtFor:Postfix(Vector3 clickPos, Pawn pawn, List`1& __result)     - finalizer net.pardeike.rimworld.mods.achtung: Exception AchtungMod.FloatMenuMakerMap_ChoicesAtFor_Finalizer:Finalizer(Exception __exception, List`1& __result)
-- submitted by<EMAIL_ADDRESS>

Will be fixed in v3.4.5


Endless Sync Loop when installing Helm Chart via ArgoCD

Describe the bug:
Helm chart's ValidatingWebhookConfiguration is missing a configuration section which Kubernetes adds in on Azure Kubernetes Service to every ValidatingWebhookConfiguration which causes ArgoCD to place the Helm chart into a constant sync loop and never become healthy.
Expected behaviour:
This ValidatingWebhookConfiguration
Steps to reproduce the bug:
Install this Application CRD on your cluster running ArgoCD 2.0.3 on an Azure Kubernetes Service cluster:
---
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  finalizers:
    - resources-finalizer.argocd.argoproj.io
  name: cert-manager
spec:
  project: default
  destination:
    namespace: cert-manager
    server: 'https://kubernetes.default.svc'
  syncPolicy:
    automated:
      prune: true
      selfHeal: true
  source:
    chart: cert-manager
    repoURL: 'https://charts.jetstack.io'
    targetRevision: v1.4.0
    helm:
      releaseName: cert-manager
      values: |+
        global:
          logLevel: 2
        ingressShim:
          defaultIssuerName: letsencrypt
          defaultIssuerKind: ClusterIssuer
        installCRDs: true
        prometheus:
          enabled: true
          servicemonitor:
            enabled: true

Once you do, this section will repeatedly want to delete webhooks[0].namespaceSelector.matchExpressions[2]:
- key: control-plane
  operator: DoesNotExist

Anything else we need to know?:
Nope.
Environment details::

Kubernetes version: 1.20.7
Cloud-provider/provisioner: Azure Kubernetes Service
cert-manager version: v1.4.0
Install method: e.g. helm/static manifests: ArgoCD 2.0.3 using above Application CRD instance.

/kind bug

Had the same problem after adding it to argocd using kustomize. In my case, I added the following patch:
patchesJson6902:
- target:
    group: admissionregistration.k8s.io
    version: v1
    kind: ValidatingWebhookConfiguration
    name: cert-manager-webhook
  patch: |-
    - op: add
      path: /webhooks/0/namespaceSelector/matchExpressions/-
      value:
        key: control-plane
        operator: DoesNotExist

/reopen
/remove-lifecycle rotten

@jstewart612 Thanks for the PR and apologies for not replying sooner.
The cert-manager Helm chart has grown a number of very specific parameters, such as this, over the years
and it is difficult for us to test them all.
If at all possible we're trying to avoid adding more parameters.
Would you be able to work around this some other way?
For example by using the kustomize based work around from @jemag in https://github.com/jetstack/cert-manager/issues/4114#issuecomment-955076635 (above).
Or could you use any of the work arounds described in:

https://github.com/argoproj/argo-cd/issues/3698

I worked around this using ArgoCD IgnoreDifferences. This way you won't have to change the cert-manager deployment. See the ArgoCD Docs for more information
ignoreDifferences:
  - group: admissionregistration.k8s.io
    kind: ValidatingWebhookConfiguration
    name: cert-manager-webhook
    jqPathExpressions:
      - .webhooks[].namespaceSelector.matchExpressions[] | select(.key == "control-plane")

It'd be nice if https://github.com/cert-manager/cert-manager/issues/4114#issuecomment-1008162907 was documented in the installation instructions somewhere. -- I'd consider that a "fix' for this.
Alternatively, perhaps we can just get this fixed in ArgoCD:
https://github.com/argoproj/argo-cd/issues/4276#issuecomment-907797060

I found another alternative.
In their documentation AKS & microsoft explain you can disable the Admissions Enforcer that is adding that matchExpression with either a label or an annotation.
I simply dropped this into my helm configuration
webhook:
  validatingWebhookConfigurationAnnotations:
    admissions.enforcer/disabled: "true"

And the diff is gone. I know AKS doesn't recommend you doing this since it can affect protected namespaces but given this webhook is configured to only affect cert-manager CRD's and not vanilla k8s resources this is a better solution for people like me who uses ApplicationAets and changing one single app is not feasible.

/reopen
We should document the work around for this in:

https://cert-manager.io/docs/installation/continuous-deployment-and-gitops/

xref: https://github.com/cert-manager/website/issues/320

/reopen
We should document the work around for this in:

https://cert-manager.io/docs/installation/continuous-deployment-and-gitops/

xref: cert-manager/website#320

Hi @wallrj
I would like to take this issue and update the existing document with the workaround for cd with argocd.
Could you pls assign the issue to me?
Thanks

@wallrj
requesting for a review for the PR: https://github.com/cert-manager/website/pull/1350


Need help getting deployed

Hi,
I'm trying to get this deployed on my own github pages to make changes. I'm having some issues with that. My intention is to get this packaged up so it can be a PWA to install on phones (via obtanium) and then integrate amber so users can use their existing keys to sign in. once that happens i intend to start picking up some of the existing issues to add some features.
I've forked the repo here: https://github.com/auggie-lahey/jesterui
It publishes to pages here: https://auggie-lahey.github.io/jesterui/
but i just get the readme
so
i also forked https://github.com/jesterui/jesterui.github.io
to https://github.com/auggie-lahey/chess.github.io
which goes https://auggie-lahey.github.io/chess.github.io/
but i get nothing
still new at github pages, so please let me know what i'm doing wrong. For the PWA, i intend to implement the requirements here: https://www.pwabuilder.com/reportcard?site=https://jesterui.github.io
but i need to have the dev env

Hey @auggie-lahey!
It seems https://auggie-lahey.github.io/chess.github.io/ is working now, right?
Nice that you are interested in the project.
I plan to release new features and updates (especially on the protocol specific parts) in the next months.
Currently, this repo contains just the code and the deployment is done manually to repo jesterui.github.io.
If you have any other question, please feel free to ping me anytime :pray:

yeah, i got it out. how did you get the url to be short?
https://auggie-lahey.github.io/chess.github.io/
vs
https://jesterui.github.io/
i talked more about hzrd with this and realized how complex the moderator would need to be for this to be functional for small sat bets and decided i wanted to get some other devops tools built out first. but i definitely will be contributing next winter if i get this other stuff built out

yeah, i got it out. how did you get the url to be short? https://auggie-lahey.github.io/chess.github.io/ vs https://jesterui.github.io/

"jesterui" is the name of the organisation. If you name your repo <organisation_name>.github.io it will be reachable with the corresponding url.

i talked more about hzrd with this and realized how complex the moderator would need to be for this to be functional for small sat bets and decided i wanted to get some other devops tools built out first. but i definitely will be contributing next winter if i get this other stuff built out

Yeah, I'll let you know. Any help with that is highly appreciated. First will be a small reference application based on NIP-64. Interested in helping me with this? :slightly_smiling_face:


Is there a way to add screenshot to JUnit report as attachment?

Hi
I'd like to generate a JUnit report for my tests, and include a screenshot, which is automatically saved by codeceptjs on fail.
Searching didn't give any results. But maybe there is a simple way to achieve this though?
This is a part of my config describing reporter options:
  helpers: {
    Puppeteer: {
      uniqueScreenshotNames: true,
      [...]
    }
  },
  mocha: {
    reporterOptions: {
      codeceptjs-cli-reporter: {
        stdout: "-",
        options: {
          debug: true
        }
      },
      mocha-junit-reporter: {
        stdout: "./output/console.log",
        options: {
          mochaFile: "./output/result.xml"
        }
      }
    }
  },

here's how I run tests:
codeceptjs run --reporter=mocha-multi

@iljao well, I took some times to look at this, there are several things to make this works:

The current npm of this repohttps://github.com/michaelleeallen/mocha-junit-reporter is not updated, which means attachments will not work -> https://github.com/michaelleeallen/mocha-junit-reporter/commit/36ea8b8ee95f8a3c3950d2dd7a19f2ea58f453c1#diff-168726dbe96b3ce427e7fedce31bb0bcR23 -> to solve this, you need either wait for the author to publish the latest version or clone it ;)

We need also to turn it on from codeceptjs side, the config should be like this

mocha: {
        reporterOptions: {
          'codeceptjs-cli-reporter': {
            stdout: "-",
            options: {
              debug: true
            }
          },
          'mocha-junit-reporter': {
            stdout: "./test/codeceptjs/output/console.log",
            options: {
              mochaFile: "./test/codeceptjs/output/result.xml"
            },
            attachments: true // <<< this line
          }
        }
      },

and also there is a change in the core of the this file screenshotOnFail.js
if (Container.mocha().options.reporterOptions['mocha-junit-reporter'] && Container.mocha().options.reporterOptions['mocha-junit-reporter'].attachments) {
          test.attachments = [path.join(global.output_dir, fileName)];
        };

in the end, it will work when everything is changed.
<?xml version="1.0" encoding="UTF-8"?>
<testsuites name="Mocha Tests" time="0.989" tests="1" failures="1">
  <testsuite name="Root Suite" timestamp="2019-03-26T11:05:48" tests="0" failures="0" time="0">
  </testsuite>
  <testsuite name="@test" timestamp="2019-03-26T11:05:48" tests="1" file="/Users/thanhnguyen/Desktop/ucb-client/test/codeceptjs/tests/smokeCheck/test.js" failures="1" time="0.989">
    <testcase name="@test: test" time="0.989" classname="test">
      <system-out>[[ATTACHMENT|/Users/thanhnguyen/Desktop/ucb-client/test/codeceptjs/output/test_1553598349.failed.png]]</system-out>
      <failure message="expected web application to include "sbc"" type="Error"><![CDATA[expected web application to include "sbc"

Scenario Steps:

- I.see("sbc") at Test.Scenario (test/codeceptjs/tests/smokeCheck/test.js:5:7)
- I.amOnPage("https://google.com") at Test.Scenario (test/codeceptjs/tests/smokeCheck/test.js:4:7)

]]></failure>
    </testcase>
  </testsuite>
</testsuites>

The PR is merged for mocha-junit-reporter. Is there a possibility that this feature request gets in?

Hi again!
Thanks for adding this feature!
I have finally tried it just recently and it didn't work for me at first.
Then I figured out that adding attachments: true line is insufficient and it also needs to be duplicated in the options object, like this:
mocha-junit-reporter: {
   stdout: "./output/console.log",
   options: {
      mochaFile: "./output/result.xml",
      attachments: true //<-- also here
   },
   attachments: true //<-- here as described
}

With both lines it works as expected.
Probably you need to do something about it :)


forestry conf files

pls consider adding forestry config files...thats helps writing blog posts easily

Someone could write a tutorial on that or fork it and create a Forestry starter. Bleda will remain a Markdown blog starter.

i get your point but forestry is still markdown but instead of writing from your pc, you can write posts from a website.

and to make it forestry compatible, we just need to add a .forestry folder and conf file inside.

I understand, but there could be lots and lots of services like Forestry - what do we do then? Add config files for each of them, pick a favorite? I prefer keeping such third parties in userland, especially since it sounds so easy to just add a config file.


Are breaks billable?

In Angus's timesheet there are 5 entries with "activity":"Break"
Should we treat these separately?
As a rule of thumb, I personally bill all breaks of <= 30 minutes to the project I happen to be working on that day, and I count all breaks of > 30 minutes as PTO.
Some formats may have an explicit way to indicate breaks. That matters when calculating aggregates, we don't want the total number of hours worked to become different for different data formats with/without explicit handling of breaks.
I think we should have translations for such business rules between our data formats, probably?

Sorry this is more a question for https://github.com/federatedbookkeeping/timesheets but I guess it's worth asking what the Timeld.org view on this is.


Add NodeJS, remove Ruby, update target folder listing to 15.2

NodeJS: As discussed with @jyotisingh and validated on Linux.
Ruby: There is jruby in the tools folder, that also is invoked in the build process. External Ruby should thus not be necessary anymore - or am I wrong?
Ant: Unsure but I was not able to see a shipped Ant somewhere, and it definitely is used. Could you validate this please independent of me? (Otherwise I'll see into it next week)

Could you sign the CLA. It is a one time legal requirement.

Closing for lack of CLA


OverflowException when converting .DDS to .FTEX

File : nvidia_clp_nmp.dds
Location : Metal Gear Solid Ground Zeroes\data_01\Assets\tpp\ui\ModelAsset\sys_logo\Pictures
How to reproduce :

Create a texture of the same size (2048x1024 in e.g.) in Photoshop using Nvidia's DDS Format converter plugin.
Convert the DDS file to FTEX by drag&dropping that file on ftextool.exe

Exact files used : https://www.mediafire.com/?ba9zu2b19tkpnaf
Screenshot : http://prntscr.com/5plp32
That should cover it. Keep in mind that I've been able to do texture modifications on other files without a problem in the past, so this is quite abnormal.

Also if you're curious as to what I'm trying to do: it's for a fake WiiU Trailer I'm making.

Thanks for your finding. Fox Engine textures use 1-10 mip maps.
You didn't generate any mipmaps in your example dds file.
I'll release a fix for importing mipmapless dds files some time later.


[Serialization] Remove several unnecessary bits of helper logic

Several commits that simplify Serialization in mostly trivial ways, mainly by removing Serialization's ad-hoc copy of an existing helper function. No intended functionality change.

@swift-ci Please test


Update backup_restore.md

Update to solve the issue where the service could not be found, in some configurations the database is defined as 'db'.
If not configured in an explicit way the user might get a "ERROR: No such service: database" when running the command

Thank you for your contribution. The docker-compose files have recently been updated so that all installations use the same service names. The backup instructions refer to the latest version (database).


The user is displayed with a "Login Error" after logging out from the app

Background: The user is displayed with a "Login Error" after logging out from the app.
Environment: Test
Steps to Reproduce:

Log in to the PSA-Digital-Journey portal on test with valid credentials.
Click on "Logout".

Actual Behavior: The user is displayed with a "Login Error" after logging out from the app. The user can not log back in as a different user unless using an incognito window which must be closed and re-opened after logging out.

Expected Behavior: The user should not be displayed with any errors and should be redirected to the login screen after logging out.

Hey team! Please add your planning poker estimate with ZenHub @simensma-fresh @iman-jamali-fw @qudsia-khan-fw


Add JDK 9 for Kotlin

JDK 9 is now required to build Kotlin:
https://github.com/JetBrains/kotlin/blob/4b524b8589afce0ea8ebef6500325b90cc1b4d15/buildSrc/src/main/kotlin/LibrariesCommon.kt#L23

Oh, I see now why this was necessary: https://github.com/TWiStErRob/gradle-enterprise-oss-projects/actions/runs/4185836952/jobs/7253624444


[QOL Improvement] Show summary of type combos in addition to the Pokemon in offensive coverage weak/resist list.

In the offense section, pick "Fighting" and "Flying" types.
In Pokedex Coverage, select "Resist" list.
The following Pokemon are shown.These Pokémon are resistant to all of the selected types. [Fighting][Flying]
#026 Raichu (Alolan Form)				Electric/Psychic
#145 Zapdos						Electric/Flying
#479 Rotom (Rotom)					Electric/Ghost
#479 Rotom (Fan Rotom)					Electric/Flying
#587 Emolga						Electric/Flying
#642 Thundurus (Incarnate Forme)			Electric/Flying
#642 Thundurus (Therian Forme)				Electric/Flying
#679 Honedge						Steel/Ghost
#680 Doublade						Steel/Ghost
#681 Aegislash (Shield Forme)				Steel/Ghost
#681 Aegislash (Blade Forme)				Steel/Ghost
#702 Dedenne						Electric/Fairy
#741 Oricorio (Pom-Pom Style)				Electric/Flying
#785 Tapu Koko						Electric/Fairy
#848 Toxel						Electric/Poison
#849 Toxtricity (Amped Form)				Electric/Poison
#957 Wattrel						Electric/Flying
#958 Kilowattrel					Electric/Flying
#977 Gholdengo						Steel/Ghost

Show Pokemon type combo (de-duplicated) at the start of offensive coverage resist list.These 6 type combos are resistant to all of the selected types. [Fighting][Flying]
* Electric/Fairy
* Electric/Flying
* Electric/Ghost
* Electric/Poison
* Electric/Psychic
* Steel/Ghost

These Pokémon are resistant to all of the selected types. [Fighting][Flying]
#026 Raichu (Alolan Form)				Electric/Psychic
#145 Zapdos						Electric/Flying
.
.
.
.
.
#958 Kilowattrel					Electric/Flying
#977 Gholdengo						Steel/Ghost

This would save a bit of time in cases where the list of Pokemon resistant/weak to certain move types spans multiple pages.
The same suggestion would apply to offensive coverage weakness list.

Cool idea!

Cool idea! Are you interested in working on a PR for this?
I'm thinking about spending some time today working on issue https://github.com/wavebeem/pkmn.help/issues/132 so I'm not sure if this is something I would get to soon or not.


bugfix + cosmetics

blame says it's mine https://github.com/cocaman/yara-scan-service/blame/8700c6619d17eef8a96659b68ddd32b40c994fe6/yara_scan_upload.py#L56 :)


Timeline does not work properly

Current Behavior
The first row of the timeline shows all possible objects with different dates
Steps To Reproduce

go to https://forschung.stadtgeschichtebasel.ch/timeline.html
see first row

Anything else?
It may have to do with the inability of timeline to handle all edtf data. possibly especially BC dates

Issue: Timeline Date Handling Error for Negative and Incorrect Date Formats
Description
In the current timeline view at Stadtgeschichte Basel Timeline, dates are not displayed correctly. Specifically:

The timeline does not handle negative dates (BCE dates) properly.
The platform requires specific formatting for years:

Years must have four digits, or be represented by "X" if unknown.
Formatting rules include:

100 BC should be formatted as -0100
12500 BC as Y-12500
500 AD as 0500

However, the timeline currently does not support negative year values, causing issues with BCE dates.
Steps to Reproduce

Upload date entries including BCE years and view the timeline.
Observe that BCE dates and potentially other incorrectly formatted dates are not displayed as expected.

Expected Behavior

Timeline should support BCE dates (negative year values) as well as AD dates, displaying them in proper chronological order.

Potential Solutions

Fix Date Format Handling: Ensure all years include at least four digits, padding with zeros where necessary.
Enable Negative Year Support: Modify timeline parsing logic to accommodate negative values for BCE dates.
Additional Investigation: Further testing is recommended to identify any other potential date format inconsistencies.

Next Steps

Action: Investigate and implement support for negative values in the timeline component.

Reported by: Cristina Muench & @koilebeit
Date: October 27, 2024


Qualification tool: Error handling while processing large event logs

This fixes https://github.com/NVIDIA/spark-rapids/issues/3430 .
In this PR, we catch OOM Error and log it with a hint to increase heap size to the user. Earlier it would complete without any log.
Not sure how to add tests for this. I ran it locally with small heap size and got the logError on the console.
 java -Xmx512M -cp tools/target/rapids-4-spark-tools_2.12-21.12.0-SNAPSHOT.jar:/home/nartal/spark-3.1.1/spark-3.1.1-bin-hadoop3.2/jars/* com.nvidia.spark.rapids.tool.profiling.ProfileMain --csv /home/nartal/CPU_GPU_eventLogs/CPU_runs/

21/09/29 17:44:53 WARN Utils: Set SPARK_LOCAL_IP if you need to bind to another address
21/09/29 17:44:56 ERROR ApplicationInfo: OOM error while processing large files. Increase heap size to process file:/home/nartal/CPU_GPU_eventLogs/CPU_runs/application_1630450374626_0001_1

so this should not just be for OOM's, i've seen us swallow other exceptions as well, we should at least print out when an exception happens to let the user know.

build

build

build

build

build

build

build


Readability fix for Android environment setup

Pretty straightforward

By analyzing the blame information on this pull request, we identified @mkonicek and @csholmq to be potential reviewers.

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.

@skyebook There is a conflict now, could you rebase on master then we can try merging again.

@janicduplessis Easy one, that file has been deleted in the docs. Closing 👍

@skyebook True, the setup docs were completely remade recently.


fix: plurality of ipfs-swarm-addrs in help text

Flag name is now --ipfs-swarm-addrs instead of --ipfs-swarm-addr. We were displaying the old flag name in the help text after the improvements from #2792 landed

Would it be better to reference the flag name directly, as we do here?

Yes!! I didn't know we could do that. Will also add some help methods to make this a little cleaner.


python310Packages.xknx: 0.22.1 -> 1.0.0

Description of changes
https://github.com/XKNX/xknx/releases/tag/1.0.0

Things done

Built on platform(s)

[x] 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.11 Release Notes (or backporting 22.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

3 packages built:

home-assistant-component-tests.knx
python310Packages.xknx
python39Packages.xknx


Clippy compliance 1.47

The new version of the compiler is bundled with an updated clippy, which was able to find some new issues in the codebase. There are also some new lints, 2 of which can yield performance improvements:

match_like_matches_macro, fixed in https://github.com/AleoHQ/snarkOS/commit/54efd538171028741b5bdd0532d0d55967e1207e
stable_sort_primitive, handled in https://github.com/AleoHQ/snarkOS/commit/3b1585c948b98227ece9a4e5cf3eaee3c5ee62ef
same_item_push (commit https://github.com/AleoHQ/snarkOS/commit/30a1d76325a7ac358e6ebeabfed1ba1248acc0a6)

Rebased.

Rebased.


Add deploy script

Failing with:
Run npm ci
  npm ci
  shell: /usr/bin/bash -e {0}
npm ERR! fsevents not accessible from chokidar

Possible workarounds from https://github.com/bahmutov/npm-install/issues/103:

Add npm install -g npm step to CI pipeline, or
Make fsevents optionalDependencies in all places in package-lock.json

Add npm install -g npm step to CI pipeline

This worked 🔥


refactor: auto-formatting sdk files with black

Description
The following three files are revised for a future PR: mqtt.py, channel_manager.py and config.py. As a prep, these files are auto-formatted with black.
Type of Change

[ ] Bug Fix
[ ] New Feature
[ ] Breaking Change
[x] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[ ] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[ ] I have verified this change is not present in other open pull requests
[ ] Functionality is documented
[ ] All code style checks pass
[ ] New code contribution is covered by automated tests
[ ] All new and existing tests pass

Codecov Report

Merging #386 (3f71f82) into main (f6c291e) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #386   +/-   ##
=======================================
  Coverage   15.13%   15.13%
=======================================
  Files          48       48
  Lines        2775     2775
=======================================
  Hits          420      420
  Misses       2327     2327
  Partials       28       28

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


Health check from other service configured with retry policy to config server can hang.

We encountered some strange behavior when we have a service, which is configured to contact the config-service with a retry policy (for instance try for 30 minutes) with regards to that service health check page.
Looks like the same retry policy is used for updating the health information of the config service in that service. However, when the config service is down, this check is retrying and retrying. At that time, the /health page of the service hangs and doesn't show.
We use external automated monitoring on /health to identify the health status. In the above case, it marks the service as down, while it isn't.
I'd guess the health check towards the config server should not take into account the configured retry policy and just show DOWN instantly when it is not there.
Kind regards,
Wouter

It should eventually give up though.  Do you have spring.cloud.config.failFast=true set in your config client?

These are our settings:
spring:
cloud:
config:
failFast: true
retry:
multiplier: 1.1
maxAttempts: 100
maxInterval: 30000
I guess we could indeed fiddle around with those to let the health check page hang less time, but I have the feeling that would only be a work-around.

I guess the question is how fast do u want to give up trying to connect to the config server if it is down.  I don't think you want it to keep retrying too long.

You're right about that. But the issue is that, even if it would only try for 2 minutes, that the /health page of the service is unavailable.

I see your point as well.  @dsyer or @spencergibb does it make sense to try and make the health endpoint available while we continue to try to connect to the config server.  I havent looked at the code, but I can imagine we must be able to return a down/unknown status from the interface while we continue to try and make a connection.

In this case, I would disable the config server health check. The retry is annotation based. I'm unsure how to selectively turn off retry. #260 is related.

Since this is a problem, in principle, for all health indicators, maybe we should look for a solution in spring boot? OTOH it seems like a really bad thing that we made it worse with the retry interceptor. There are ways to solve the problem in Spring Cloud as well. For instance, our health indicator could be aware of the problem and try to extract the target from the proxy and call it directly. Or it could create a new instance of the property source locator. That doesn't seem like a bad idea?

I think if the problem can be fixed in spring boot than I think that sounds like the best solution,


add "Apply" button for Interface Detail Page

Modify Interface's return data is very useful when debug the web app in local develop environment, but every time after I modify the return data, I have to click "Save" button and the page will go back to Interface list page. In fact, I want to modify the return data  more than one  time but I have to start from list page again and again.
If there's an "Apply" button in the detail page will be a great help for everyone. When the return data modified, just click "Apply" and the modification will come into force immediately and the page stays in the detail page for next modification.
THANKS and Mock2easy is an excellent tool for web developers !

I will fix on v0.3.18, thank you for your issues.


TypeError line 102

Hi ,
There's a TypeError at line 102 in smart-video-counter.py.
One of the parameters is a float while the function wants a int.
Seems like the function as been updated.
Running system:

RPI 3 B
Rasbian Jessie
opencv 3.2.0 + contrib_modules

I acutally correct the TypeError by replacing
flow = cv2.calcOpticalFlowFarneback(prev, nextim, 0.5, 1, 15, 3, 5, 1.2, 0)
by
flow = cv2.calcOpticalFlowFarneback(prev, nextim, None, 0.5, 1, 3, 15, 3, 5, 1)

Thanks Alexis for the issue and for the solution. This has been integrated in the code and pushed back to GitHub.


This program can have Dynamic input

This program can have Dynamic input
File to be updated : https://github.com/anandfresh/program/blob/35fe14503d0f23832fb92ccfc61c126f7ae966f7/program/find-surface-area-of-a-cylinder/FindSurfaceAreaOfACylinder.fs
Originally posted by @brundabharadwaj in https://github.com/codinasion/program/pull/7242#pullrequestreview-1381292390

!assign

Hey @RiteshK-611, this issue is already assigned to @anandfresh !!!
Please choose another issue.
Thanks for your interest in contributing to this project.


Runtime error with arrays

Copy/paste into http://elm-lang.org/try
import Array exposing (Array)

type alias Box =
  { value : String }

convertTo2D : Array Box -> Int -> Array (Array Box)
convertTo2D boxArray numRows =
  let
    nextRow = Array.slice 0 numRows boxArray
  in
    if Array.length nextRow > 0
    then Array.push nextRow (convertTo2D (Array.slice numRows (Array.length boxArray) boxArray) numRows)
    else Array.empty

oneDimArray = Array.initialize 100 (\index -> { value = toString index })

temp = convertTo2D oneDimArray 10

This is boiled down from a larger application. The relevant error is:
"Uncaught TypeError: Cannot read property 'height' of undefined  everything.js:831"

Did you check it's not one of these existing issues about arrays?

I checked. It could possibly be this issue (https://github.com/elm-lang/core/issues/349) but it's unclear whether it's the same. None of them mention the same runtime error.

Thanks!

I've played a bit with your example. By unrolling the recursion a bit (repeatedly inlining convert2D) and trying with reduced overall size, I reduced it to this:
import Array

temp =
  let
    n = 10
  in
    Array.initialize (4*n) identity
    |> Array.slice n (4*n)
    |> Array.slice n (3*n)
    |> Array.slice n (2*n)
    |> Array.slice n n

This shows the same error.
And interestingly, changing n = 10 to n = 8 lets the error go away (while it remains for n = 9). Noticing that for n = 8 we have 4*n = 32, the "magic number" from #349, it might have something to do with that issue after all, or maybe share a common root problem with that one...

Closing in favor of #649 which will track all array issues and progress on a new implementation.

I suspect this line is the culprit:
var slot = new createNode(a.height - 1, 0);

in:
https://github.com/elm-lang/core/blob/master/src/Native/Array.js#L730
createNode() generates an new node, it should not be new'ed up itself


Update guidance for XFS - nobarrier option removed

See issue #55705, opening a new issue per the notes in the last one. Documentation still shows a last update note from 2016, new guidance is still needed for XFS on Azure.

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

ID: f7792d5a-de76-c48a-0c8d-46b3e918a503
Version Independent ID: 23b37b12-4a16-d108-2705-2d96ceb484c8
Content: Optimize your Linux VM on Azure - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/optimization.md
Service: virtual-machines-linux
Sub-service: disks
GitHub Login: @RicksterCDN
Microsoft Alias: rclaus

@StoicTom Thanks for the question. We're investigating this and will get back to you shortly.

@mimckitt Can you please check and add your comments on this doc update request as applicable

This document is no longer being updated

I added a backlog item for the linux team to investigate if we need a new doc in this area or not.
#please-close

This document is no longer being updated

I added a backlog item for the linux team to investigate if we need a new doc in this area or not.
#please-close


pass entity id to recipientInput

RCINT 4356 When click to SMS an entity's number, the entity ID should be carried over to CTI

Coverage decreased (-0.02%) to 40.283% when pulling 0104651c1215b76f6c98e13d5cd20c10aee212fc on alvita:rcint-4356 into 67b17fae3c12be8324277daae804259c4a823ecf on ringcentral:master.


[Bug]: Compiler Plugin not identifies the transitive dependencies

Description
When using the Compiler Plugin built in the dev branch of module-ballerinax-wso2.mi, the user-code throwing the following error:
error: compilation failed: The compiler extension in package 'ballerinax:ballerina:0.1.0' failed to complete. org/antlr/v4/runtime/Recognizer

Steps to Reproduce
clone the following repo with specific branch:
https://github.com/ballerina-platform/module-ballerinax-wso2.mi/tree/dev
build the project:
./gradlew clean build -x test -x check
Create new ballerina project:
bal new project1
Add the following compiler plugin dependency in Ballerina.toml file:
[[dependency]]
org = "ballerinax"
name = "ballerina"
version = "0.1.0"
repository="local"

Use the following sample ballerina code for the ballerina new project:
import ballerinax/ballerina as _;

public function foo(string i, string j) returns string {
    return i + j;
}

build the new ballerina-project
bal build
Affected Version(s)
ballerina 2201.9.0
OS, DB, other environment details and versions
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
@gayaldassanayake

Once we manually add the missing jar (antlr) into bre/lib in the ballerina pack, this issue is resolved.

The issue here is due to the missing antlr dependency.
In 8.x, the graphql-tool has used antlr as a dependency. Furthermore handlebars has been used in multiple places in lang, openapi, distribution etc. (https://github.com/search?q=org%3Aballerina-platform handlebars&type=code). As a result, despite specifying handlebars, antlr in the CompilerPlugin.toml or not, they will be picked from the distribution and used.
When moving into U9, the graphql-tool has removed antlr as a dependency. Therefore, antlr is not packed with the distribution anymore. However, handlebars is still packed in the distribution.
In compiler plugins, if the same class is available from both the bal distribution and as a URL in CompilerPlugin.toml, priority is given to the bal distribution, due to the nature of the URLClassLoader used. As a result, handlebars is still picked from the bal dist. However, handlebars from the distribution do not have access to the antlr` dependency added in the CompilerPlugin.toml. Hence the below error.
java.lang.NoClassDefFoundError: org/antlr/v4/runtime/CharStream
        at com.github.jknack.handlebars.Handlebars.<init>(Handlebars.java:294)
        at com.github.jknack.handlebars.Handlebars.<init>(Handlebars.java:332)
        at io.ballerina.stdlib.mi.plugin.Utils.generateXml(Utils.java:52)
        at io.ballerina.stdlib.mi.plugin.model.Component.generateInstanceXml(Component.java:56)
        . . .

The first solution from the above comment can be done. handlebars has been added to bre/lib from the ballerina-lang. However, it is not used in runtime. So we can safely remove handlebars from bre/libs. With that, handlebars from the CompilerPlugin.toml is picked.


Multiple omnisharp.json settings for one project

Currently, you can override FormattingOptions by placing an omnisharp.json file at the project root folder.
In my projects, I'm using a lot of sub-modules (git) developed by third parties with different C# settings, but all changes made (for merge request to upstream) in these sub-modules must be formatted by hand to avoid losing original formatting.
I would be happy to be able to customize FormattingOptions for each of these sub-modules.
For each folder containing an omnisharp.json, all sub-folder would recursively apply these settings until a new omnisharp.json is found.
Environment data
dotnet --info output:
Outils en ligne de commande .NET (2.1.104)
Product Information:
Version:            2.1.104
Commit SHA-1 hash:  48ec687460
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.17134
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.1.104\
Microsoft .NET Core Shared Framework Host
Version  : 2.0.6
Build    : 74b1c703813c8910df5b96f304b0f2b78cdf194d
VS Code version: 1.23.1
C# Extension version: 1.15.2

This issue was moved to OmniSharp/omnisharp-roslyn#1208


[FLINK-15840][table-planner-blink] ClassCastException is thrown when use tEnv.from for temp/catalog table

What is the purpose of the change
TableEnvironment.from/scan(string path) cannot be used for all temporaryTable and CatalogTable (not DataStreamTable and ConnectorCatalogTable). Of course, it can be bypassed by TableEnvironment.sqlQuery("select * from t"), but from/scan are very important api of TableEnvironment and pure TableApi can't be used seriously.
Brief change log
The problem is that CatalogSourceTable.toRel wants to get the translator of the compute column. At present, CatalogSourceTable.toRe has two places to call:

The parser stage, which passes the correct compute column translator.
In the rule optimization stage, the correct compute column translator is not passed in the TableScanRule, so an error is reported.

There are two solutions:

Don't use ToRelContext to transfer the translators of compute column. Use FlinkContext to transfer so that we can get the correct translators of compute columns at any stage.
In CatalogSourceTable.toRel, when there is a compute column and the compute column translator cannot be obtained, an error is reported, and other cases pass normally. The disadvantage of this is that it is currently unable to support compute columns on TableApi.

Considering:

No plan to support compute column on TableApi.
Solution 1 changed too much
Solution 1 is also an intermediate version. In the next version, it is considered to separate the compute column translation and complete translation in the parser stage.
We consider using solution 2.

Another bug is CatalogSourceTable need override explainSourceAsString. Otherwise hep rule optimizer can not distinguish CatalogSourceTable and TableSourceTable.
Verifying this change
TableScanTest
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)

I find that Calcite TableScanRule uses ViewExpanders.simpleContext which isn't compliant with CatalogSourceTable. Is that possible to provide a new TableScanRule with the proper FlinkToRelContext?  cc @danny0405
I think in this way, watermark and computed column on Table API can also be supported in the future.

Hi @wuchong , you can take a look to FlinkPlannerImpl, it needs many things:
config: FrameworkConfig,
catalogReaderSupplier: JFunction[JBoolean, CalciteCatalogReader],
typeFactory: FlinkTypeFactory,
cluster: RelOptCluster

TableScanRule also can not find these arguments.
And I don't think this is a good way. FlinkToRelContext is bad idea to me. Just fix TableScanRule looks like a very temporary fix, and we don't know if there is another ToRelContext.
In 1.11, there is another plan to support computed column in table api, as I said above, it is considered to separate the compute column translation and complete translation in the parser stage.

Hi @JingsongLi , I'm not sure whether it works to separate the computed column translation to parser stage, because adding a watermark node requires the FlinkToRelContext too and it has to happen in CatalogSourceTable#toRel.
I didn't mean to create a FlinkPlannerImpl in the rule, I mean we can pass the FlinkToRelContext to the rule via FlinkContext.

Hi @wuchong , you can take a look to my solution 1, I think that is what are you mean. We don't need FlinkToRelContext any more, we can get FlinkPlannerImpl directly from FlinkContext. As I said, I think FlinkToRelContext is a bad way.
I wanted to do that way.
But discussed with @danny0405 offline:

Support computed column and watermark(I don't think there are some difference between computed column and watermark) in table api is not the goal of 1.10.
"Separate the computed column translation to parser stage" is his plan to do in 1.11.

Updated description.

Thanks @JingsongLi , according to the updated description. I'm in favor of option#1 which passes FlinkPlannerImpl via context, because it is a true fixing not a temporary workaround and gives us the ability to extend features in the future.  Btw, please try to expose a small interface, rather than a whole FlinkPlannerImpl.


Promo images

Top notch images! Thanks a lot. Will put these on the play store today

Related ticket: #24


Add Tzdata.HTTPClient behaviour and make hackney optional

If users didn't already directly depended on Hackney they'd need to do so now. This is documented in README and if they haven't followed the steps, they'd see this error from poller:

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:
Thank you @wojtekmach! Hackney will not be optional just yet. But eventually when there are be warnings by default.


Delete multiple instances of data at one time in Rasa X interface

Description of Problem: Primarly, as a bit of a time-saver, it would be great if the Rasa X interface could support deleting multiple NLU data items in one go (in addition to currently allowing to delete items individually). Currently, there have been a few ocassions where I have wanted to, for example, remove more than 1 'Story' or 'Response' at a time and it appears that currently this is only achievable on a one-by-one basis in the Rasa X GUI.
Overview of the Solution: On each or any of the pages, "NLU data", "Responses", "Stories", "Rules", it could add to the user experience and potentially the development efficiency, if there was a feature to delete multiple data items/instances in addition to the existing 'delete' icon to remove items individually. If possible, it would be good if the user was prompted to clarify if they wish to delete the selected items so that they have a chance to change their mind. Otherwise, it could significantly change their project if they delete a lot of their work unintentionally.
Definition of Done:

[ ] Add feature to "NLU data", "Responses", "Stories" and "Rules" pages to enable the user to select and delete multiple instances of data at one time.
[ ] Set a prompt to clarify if the user if the user is sure they want to delete the multiple items
[ ] Tests carried out to see that the function performs as described/intended
[ ] Add feature description to any relevant support docs
[ ] Have feature mentioned in the changelog

Thanks for submitting this feature request 🚀 @rgstephens will get back to you about it soon! ✨

@jameshenderson12 Thanks for the suggestion. Adding this as an enhancement request.

Thank you for raising this issue about Rasa X. We decided to stop supporting the Community Edition (free version) of ‘Rasa X’ (see more info here: https://rasa.com/blog/rasa-x-community-edition-changes/). That’s why we’ve closed your issue. We suggest that Rasa Enterprise customers raise the issue with our customer support team if they haven’t done so already.


Input image transforms for some of the models may be improved

Is your feature request related to a problem? Please describe.
The input image transforms in some of the models may not be configured to offer optimal input to the model during training, validation, and inference.
For instance the endoscopic_inbody_classification example:

uses a Resized transform that shrinks images to 256×256 pixels, but it does not enable anti_aliasing. Especially when downscaling large video frames with sharp details, patterns, or lines in them, this can lead to aliased artifacts that may cause the model to learn the wrong things or recognize structures that aren't really in the original image.
uses NormalizeIntensityd with nonzero set to true. If there are zero-valued pixels in the image, they will not be scaled and offset together with the rest, which may cause discontinuities and again make it seem as if there are structures that aren't really there.
uses NormalizeIntensitydwithout specifying a fixed subtrahend and divisor. This means the intensities in each image will be normalized according to the mean and standard deviation in that specific image. If the image only contains a narrow range of intensities, for instance a dark image with sensor noise, this will be blown up to a big noisy mess in which the model might recognize random things. The usual way of working for ImageNet and such, is to calculate the mean and stddev across the entire training set and use those values everywhere.

Describe the solution you'd like

Set "anti_aliasing": true in the Resized transforms.
Consider leaving nonzero at false in the NormalizeIntensityd transform unless there really is a good reason for it.
Consider setting a fixed subtrahend and divisor in the NormalizeIntensityd transform.
Re-train models with whatever parameters were changed.

Additional context
A paper that discusses the impact of aliasing in convolutional networks
I have tested the impact on processing time of enabling anti_aliasing. On a GPU (RTX A2000), the impact is tiny: transforming a 720p video frame and adding it to a batch takes 11ms instead of 9. On CPU the impact is much larger (55ms instead of 9).

Hi @DrLex0 ,
Thanks for your feedback, I think the bundle config files show the exact config content we used to train the model. If you want to change any config content in runtime, you can override the args in your CLI, refer to:
https://github.com/Project-MONAI/MONAI/blob/dev/monai/bundle/scripts.py#L655
Thanks.


Enable shell in PNaCl kernel

Recent modifications to naclports have enabled subprocess in Python.  We should be able to enable Jupyter notebook shell magic in PNaCl if we properly support this.  However, we're going to be limited by the size of our distribution, which we've started trimming; including more executables makes this more interesting, but pushes the required memfs size up.

This version of colaboratory is no longer supported, please switch to https://colab.research.google.com.


[ENH] Add suggestion message when the webgpu is unsupported

While trying the demo I run into this problem:
Find an error initializing the WebGPU device OperationError: Required limit<PHONE_NUMBER>) is greater than the supported limit (268435456). - While validating maxBufferSize - While validating required limits
Init error, Error: Find an error initializing WebGPU: OperationError: Required limit<PHONE_NUMBER>) is greater than the supported limit (268435456). - While validating maxBufferSize - While validating required limits

The rason for this is not using the latest chrome, it is pretty clearly stated in the instructions section, but the error could at least suggest this, because I imaging a lot of people like me to just rush to try the demo without much reading. This can cause a lot of pointless error repots.

just pushed a commit to add this message on the website

@jinhongyii I don't think it'll cut it
I envisioned something like a popup in the "chat" so users will definitely notice it
My point is that most don't read those instructions and jump straight to usign the demo

Or at least move the message closer to the chat and make it stand out

Thank you for your suggestion, we have made an update to the API so the error is more clear https://github.com/apache/tvm/pull/14640


Update example-web-client.js

Callback argument was channels but data being returned is team information.
Altered to avoid confusion where someone might follow the example and expect channel information.

Sorry, I didn't notice this PR! Totally my bad - given the somewhat painful merge / rebase given I've now brought 2.0.0 to master I'm going to create a separate commit with this change in - thanks for catching it!


Dungba/developing

Pull Request Test Coverage Report for Build 425

35 of 44 (79.55%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 52.909%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

commons/gridgo-pojo/src/main/java/io/gridgo/utils/pojo/PojoUtils.java
11
12
91.67%

commons/gridgo-pojo/src/main/java/io/gridgo/utils/pojo/getter/PojoGetterRegistryImpl.java
12
13
92.31%

commons/gridgo-pojo/src/main/java/io/gridgo/utils/pojo/setter/PojoSetterRegistryImpl.java
11
14
78.57%

connector/src/main/java/io/gridgo/connector/impl/resolvers/ClasspathConnectorResolver.java
1
5
20.0%

Totals

Change from base Build 416:
-0.05%

Covered Lines:
3147

Relevant Lines:
5948

💛  - Coveralls


Update kube-router network policy controller

Proposed Changes
Update network policy controller code to resync with kube-router: v1.0.0-rc2 -> v1.1.1 (master)
https://github.com/cloudnativelabs/kube-router/compare/d6f9f31a7b...ee9f6d890d10609284098229fa1e283ab5d83b93
Types of Changes
bugfix/upstream
Verification
Run normal test suite
Linked Issues
#2876
Further Comments
This update pulls in several bug fixes and a restructuring of the code from upstream. I have attempted to minimize the number of changes to the upstream code in order to make future resyncs easier. This includes modifying our Run wrapper to set up and sync the shared informers similar to how upstream's controller setup does it.
Most of the changes are limited to:

Modifying NewNetworkPolicyController function to use the k3s config.Node struct instead of KubeRouter's
CLI configuration, eliminate use of a WaitGroup for shutdown sequencing, and drop Prometheus metrics support.
Relocate or remove module references
Switch out glog for klog
Fix spelling in comments that our codespell CI plugin barfs on
Removing tests that exercise removed code

The updated network policy controller also wants to know about a couple new cluster-level configuration items, such as the cluster service CIDR range and the node-local port range, so those need to be plumbed through.

Thanks for updating this!
If possible could we also add the network_policy_controller test?

Thanks for updating this!
If possible could we also add the network_policy_controller test?

I'm going to merge this without squashing so that the modifications remain obvious. Still going to work on getting some modularity pushed upstream but until then we should try to make this easier to update ex-situ.


bug: Navbar title mispalced on Android

Type: bug
Platform: android 4.4 browser
Version: v1.0.0-beta.14

Hi,
There seems to be an issue with the title in the navbar on Android devices. This appears when the navbar is hidden and then shown again using the hide-nav-bar attribute of ion-view.

I was not able to create a Codepen demo, because it only happens when I run the app on the device or using ionic server --lab. Therefore, I have created a test repo where this can be simulated.
Steps to reproduce:

Run the app using ionic server --lab.
Go to Playlists using the side menu
Click the Toggle icon to hide the nav bar
Click on a item in the list
Go back to the Playlists view and press the Toggle icon again to show the nav bar

Result:
The title overlaps the menu icon.

Greetings @petarov!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Any movement here?
I'm currently using a nasty hack to overcome this.
  $scope.$on('$ionicView.afterEnter', function(event, state) {
    if (state.stateName === 'app.prevPage1' || state.stateName === 'app.prevPage2') {
      $timeout(function() {
        $ionicNavBarDelegate.align('left');
      }, 650);
    }
  });

This is also happening to me, please fix it

Also can confirm the same view title issue for Android on both chrome and on ionic-lab.

Works fine in v1.0.0-rc.2-nightly-1184
Thanks!

I had a similar issue and fixed it by applying title class to h1 tags
<ion-nav-title>

Title

</ion-nav-title>

Hi, I face this issue for a second while switching between tabs. this happens only for the first time.

hi, i have exaclty the same bug, on the latest 1.3.1 release of ionic. I don't have the pb in the browser and on iOS.
I have found the pb with the remote debugger 👍
If i modify this generate ionic code, it's work fine
I have delete "transform: translate3d(-74.625px, 0px, 0px);"
Compte :

Same bug in the latest ionic release on a v1 app! Any workarounds, please?

Bug still exists in Ionic v1 :-(