Avoid passing ++timeout option in zerocopy pingpong makefile

Original issue: https://charm.cs.illinois.edu/redmine/issues/1821

Verbs autobuild uses "++timeout 180 +isomalloc_sync" in TESTOPTS. Since the zerocopy direct api pingpong added in patch (~~https://charm.cs.illinois.edu/gerrit/#/c/2967/~~ https://github.com/UIUC-PPL/charm/commit/504bc22b81afa75788cb0e38974f0268e52ecec0) already uses a "++timeout 200", having two timeout options passed to charmrun causes the execution to crash. This caused the verbs autobuild failure after merging the zerocopy direct api patch.

Original date: 2018-03-07 15:58:04

Fix: ~~https://charm.cs.illinois.edu/gerrit/#/c/3819/~~ https://github.com/UIUC-PPL/charm/commit/04643ab253b314b516be0267a4d175cb3421b3be


Home page news section

To-do:

Home decoupled from Index, Profile default page
Logged in/out code moved to separate components


chore: fix unused variables from linter

This makes mathlib pass the new unused variables linter added to lean 4 core.

It would be great if we could check the core linters in CI as well.
bors r+


Implement FM & DAC VolEnv

Extend support for using VolEnv's for FM and DAC through a switch. Currently hardcoded to only support PSG, make the ability to support FM and DAC as well at the cost of 2 bytes of RAM and an extra tracker command.

Implemented in the latest commit


WIP: issue#417 Manage sources.list and make bootstrap script purge lists before updating

@aneeshusa
Hi,
As discussed I have created this pull request. Let know me what you think could be the issue.

One way to test this is manual verification inside a Vagrant VM,
I tried this as well after rebuilding the VMs, I checked in servo-master1 vm but could find the XXX string in the vm's sources.list file.

Perhaps I am missing to modify some other config files.
Fixes #417.

This change is

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @aneeshusa (or someone else) soon.

:umbrella: The latest upstream changes (presumably #531) made this pull request unmergeable. Please resolve the merge conflicts.

@aneeshusa Hey, I squashed all my commits in one. Let me know if everything is fine, it still shows the label 'needs-rebase'.

Interesting - everything seems to work fine in Vagrant, but the tests are failing on Travis, only on the builds from scratch. I'm not really sure what the problem is, so we'll have to do some research/try some different things out. Off the top of my head I'm guessing the issue may be related to ordering of the Salt states, which we can fix by adding requisites, or something apt related, e.g. maybe we need to run apt-get update after changing the sources.list file.

Interesting - everything seems to work fine in Vagrant, but the tests are failing on Travis, only on the builds from scratch. I'm not really sure what the problem is, so we'll have to do some research/try some different things out. Off the top of my head I'm guessing the issue may be related to ordering of the Salt states, which we can fix by adding requisites, or something apt related, e.g. maybe we need to run apt-get update after changing the sources.list file.

:umbrella: The latest upstream changes (presumably #573) made this pull request unmergeable. Please resolve the merge conflicts.

@aneeshusa Hi, sorry I am bit late with the update here. I added the multiverse related code back in servo-dependencies/init.sls but some of the tests in travis build-2283 failed. I will try removing multiverse entries in sources.list and see if any tests fail due to these changes.

The travis build still failed after removing multiverse entries from sources.list but this time for a different reason. Could you please help in fixing them? @aneeshusa

Ping @aneeshusa or @edunham.

@choudhary001 I've taken a look at this and I have an idea why this is happening, namely that we don't update the apt package cache after updating the sources.list file, and hence apt-get has no idea how to install packages. A good fix will be a little involved because it has to handle multiple OSes and bootstrapping, so I will need to think about it a little before I can get back to you with more concrete suggestions. The code so far looks great, though!

@choudhary001 I've identified at least two places this can make some progress while I figure out how to test bootstrapping (#631):

We'll need to refresh the package databases after updating the sources.list file. Salt includes a pkg.refresh_db module function for this. Since this is an execution module function and not a state function, we'll need to call it via the module.run state. Add an onchanges requisite so that we only refresh the package database if we just updated the sources.list file.
We're adding support for Ubuntu 16.04. As part of this, the sources.list file will likely need to be templated to handle both Trusty and Xenial machines. Spin up a Xenial VM to look at its sources.list and see how it differs from this one - I'm guessing it will be pretty similar. I believe the oscodename grain will be helpful for templating. See #649 for some examples of this.

Hi @aneeshusa ,
To refresh the package database, I am adding a new state to ubuntu/init.sls file as below. I am not sure about how test this though. Let me know if this approach is correct or not.
refresh_pkg_db:
module.run:
- name: aptpkg.refresh_db
onchanges:
- file: /etc/apt/sources.list
Differences in sources.list for trust and xenial:
Trusty: deb http://security.ubuntu.com/ubuntu osname-security main
Xenial: deb http://security.ubuntu.com/ubuntu osname-security main restricted
Trusty: deb-src http://security.ubuntu.com/ubuntu osname-security main
Xenial: deb-src http://security.ubuntu.com/ubuntu osname-security main restricted

@choudhary001, that looks good for the new refresh_pkg_db state! I think pkg.refresh_db may work instead of aptpkg.refresh_db, if you push that change we can see what Travis thinks of it.
We should be able to use Jinja templating to add the restricted field on Xenial, using the {% if %}...{% endif %} construct: http://jinja.pocoo.org/docs/dev/templates/#if

@aneeshusa I have added refreshdb changes. I read Jinja templating documentation that you referenced. However, I am not sure about how to integrate it with existing code. I will need some help there.
Thanks!

@choudhary001 You closed this PR, did you mean to push your refreshdb changes instead?
And yes, that code will work find in the sources.list file, make sure to add - template: jinja to the Salt state for that file so that it knows to run it through Jinja.

Also, I'm not sure how long it might take to get an automated test for mach bootstrap, so I will just test that manually once this is ready to go.

The PR got closed by mistake when I was trying to push refreshdb changes. Sorry about that.
I have pushed jinja templating changes for sources.list. Let me know if any changes are required.

@aneeshusa
The travis build failed with below error, I am not sure why:
[ERROR   ] Data passed to highstate outputter is not a valid highstate return: {'local': ["No function declared in state 'onchanges' in SLS 'ubuntu'"]}

Sorry about the many delays, this LGTM and is r=me. I'll merge this after #719, which fixes our build.

@bors-servo r+

:clipboard: Looks like this PR is still in progress, ignoring approval

@bors-servo r+

:pushpin: Commit 15c1068 has been approved by aneeshusa

:hourglass: Testing commit 15c106846ad68f50ec45efb42f49d167db6caffc with merge 229ea9e742d19bd6409ffeb434e5b305bd9e649e...

:sunny: Test successful - status-travis
Approved by: aneeshusa
Pushing 229ea9e742d19bd6409ffeb434e5b305bd9e649e to master...


Empty wiki!

Background
Describe your issue here.
How is it possible that the most important project today in crypto world have an empty wiki?

We use the docs folder primarily, you're welcome to attempt to add to the Wiki yourself, or help improve our documentation.


Null Deference

File Name: MessagingExtensionQnaCard.cs
File Path: C:\user\MS-Teams\microsoft-teams-apps-faqplusv2-master\Source\Microsoft.Teams.Apps.FAQPlusPlus\Cards\MessagingExtensionQnaCard.cs
Vulnerable Parameter: userActions
Vulnerability:  @Line no 1092:  userActions is not checked for null value before being dereferenced
The method UpdateHistory() in MessagingExtensionQnaCard.cs can dereference a null-pointer on line 1092, thereby raising a NullException.

Thank you for the feedback. I will share the suggestion with the concerned team.

Thank you for reporting this, we have added it to our backlog of issues to fix.

Please let me know that reported security issue is fixed or Not?


property is null in interstitialDidClose

Hi there,
I am having a strange problem and I hope someone will be able to help me.
I am using a ScrollView that should scrollTo an X coordinate on interstitialDidClose.
It works fine when the component mounts for the first time.
As soon as I leave the scene (react-native-router-flux) and the component unmounts, this behavior doesn't work anymore and I get an error message saying that the property (this.scrollView) is null. If I do a console.log, the property is not null and actually contains the ref to my scrollview.
here is a gist
https://gist.github.com/tuxtux00/63d654a8c7553eb93abd99a4d46b8632
library that I am using
react-redux
react-native-router-flux
Thanks in advance for your help folks.

My bad, forgot to call removeEventListener on componentWillUnmount.
This solved my bug


(Feature) e2e tests for Wizard

Closes https://github.com/poanetwork/token-wizard/issues/509
Problem: we have no integrated e2e tests for Token Wizard
Solution: integrate token-wizard-test-automation as a submodule

@fvictorio @fernandomg let's do not merge any PR until we will not add e2e tests.

Pull Request Test Coverage Report for Build 1588

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 9.544%

Totals

Change from base Build 1572:
0.0%

Covered Lines:
258

Relevant Lines:
1948

💛  - Coveralls

@fernandomg @fvictorio is there anything stops us from merging?

@vbaranov I'm running it locally, but I'm getting the following
[2018-03-07T14:05:08.235Z]  Test PASSED.'Owner can  finalize (after all tokens were sold)
    ✓ Owner can  finalize (after all tokens were sold) (75902ms)
    ✓ Investors receive right amount of tokens after finalization)

  14 passing (13m)
  2 failing

  1) POA token-wizard. Test suite #1
       Investor can buy  amount = minCap:

      AssertionError [ERR_ASSERTION]: Test FAILED.Investor can NOT buy minCap
      + expected - actual

      -false
      +true

      at Context.<anonymous> (test/testSuite1.js:158:16)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)
  From: Task: POA token-wizard. Test suite #1 Investor can buy  amount = minCap
      at Context.ret (node_modules/selenium-webdriver/testing/index.js:182:10)

  2) POA token-wizard. Test suite #1
       Investor can buy less than minCap after first transaction:

      AssertionError [ERR_ASSERTION]: Test FAILED. Investor can NOT buy less than minCap after first transaction
      + expected - actual

      -false
      +true

      at Context.<anonymous> (test/testSuite1.js:173:16)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)
  From: Task: POA token-wizard. Test suite #1 Investor can buy less than minCap after first transaction
      at Context.ret (node_modules/selenium-webdriver/testing/index.js:182:10)

@fernandomg please try to restart tests. You can see it worked with this PR. Anyway, @dennis00010011b will try to increase stability in the next releases of test-automation script.

@fernandomg There is should be result.log file(see path ./results/result_date_time/) Could you send it me?

There is should be result.log file(see path ./results/result_date_time/) Could you send it me?

Sorry, I moved to another issue and cleaned up the workspace. Now I worked. Thanks

@vbaranov I think this can be merged now.


Allow Android to resize the Activity window when the software keyboar…

…d appears to accommodate all buttons.
Purpose
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Open app to Setup Activity
Open Edit Text to enter your name

What to Check
Verify that the following are valid

Ensure the Join Call button is not obscured

Other Information

How much of the text it obscures depends on the size of the screen.  Here's what it looks like for a Pixel 2:

think it is ok. The keyboard can be closed to see back text.

tested


Added Auto LF and .editorconfig settings for aligning formatting rules

@jeremydmiller @mysticmind @jokokko if you're fine with this .editorsettings then I'll do code clean up and change warnings settings into errors.
See rules explanation: https://docs.microsoft.com/en-US/visualstudio/ide/editorconfig-code-style-settings-reference?view=vs-2017

I don't see anything in there that would bug me. Might also be worthwhile to add R# settings too though

I don't see anything in there that would bug me. Might also be worthwhile to add R# settings too though

@jeremydmiller I'm not using currently Resharper, but reading that it appears that Resharper should respect .editorconfig

@jokokko imho it's better to keep it in new line, for me it's not readable and confusing when we have it in single line.


Create a Type pool to avoid creating same types over and over.

Instead of a Term owning its Type, let it reference an instance in the Type pool.
Check for a new Type in the pool before creating it.

Resolved in https://github.com/KareemErgawy/types-and-programming-languages/commit/ded1cb93aafd4fa22fdd03b2832a46ab7f05e461.

[ ] Task 1.
[ ] Task2.


[hotfix] Fix some typo in HiveOptions.

What is the purpose of the change
Fix some typo in HiveOptions.
Brief change log

Using withDescription instead of withDeprecatedKeys for some config options.

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no

cc @luoyuxia

@MartijnVisser Could you please help merge this fix, thanks~


Fix typo in CDN status slug

A typo in the CDN status slug means that it will be unable to get the CDN status.
roblox-s-cdn-api-endpoint should just be cdn-api-endpoint

Thank you for your contribution @TheCakeChicken!


adding iso files and then stall

Not sure if the script completes?
Using prox.iso...
extracting kernel...
extracting initrd...
/tmp/initrd.img     : 146541568 bytes
adding iso file ...

And then it never finishes..
running on wsl2 ubuntu 20.04
➜  pxeboot ll
total 1013M
-rwxrwxrwx 1 root root 1002M Jul  9 21:07 initrd
-rwxrwxrwx 1 root root   11M Jul  9 21:00 linux26

looks like iso is in initrd?

nevermind.. just me being impatient.. it eventually finished..


Feature/refactor news provider

Minor code cleanup.

Current coverage is 0.00% (diff: 0.00%)

Merging #105 into develop will not change coverage

@@           develop   #105   diff @@
=====================================
  Files            1      1
  Lines           37     34     -3
  Methods          0      0
  Messages         0      0
  Branches         0      0
=====================================
  Hits             0      0
+ Misses          37     34     -3
  Partials         0      0

Powered by Codecov. Last update 18cc929...79e99ad


Friktion volt logo not showing up

The Friktion volt logo is not what it should be (SOL logo for SOL, BTC logo for BTC, etc)

same as https://github.com/Avaulto/Compact-DeFi/issues/4
closed :)


Better fallback for missing composer file

With the current implementation, if the root composer json isn't either a TYPO3 extension or a composer project, vite searches parent directories for a valid file. This can lead to memory issues.
Maybe there's a better fallback for this? composer.json files without type: project seem to be more common out there than anticipated.
https://github.com/s2b/vite-plugin-typo3/blob/main/src/utils.ts#L92

The fallback is ok, an error is shown. The problem seems to be related to the incomplete configuration after the early exit of the plugin. Probably a basic configuration should be provided nontheless to make sure that vite doesn't scan the whole project.
It might be enough to move the ignore list above the early return.


Set gRPC channel argument about keepalive ping's timeout and ping without rpc  and  Adding method that both reads all remaining messages & finishes session.  and  Fix off-by-one bug in ReadStreamChannelResponsesAndFinish.

PR Description :
Set gRPC channel argument about keepalive ping's timeout and ping without rpc.
[p4_runtime_session] Add method that both reads all remaining messages & finishes session.
[p4_runtime_session] Fix off-by-one bug in ReadStreamChannelResponsesAndFinish.
Build Results :
bibhuprasad@b621e980bfc1:/sonic/src/sonic-p4rt/sonic-pins$ bazel build $BAZEL_BUILD_OPTS p4_pdpi/...
INFO: Analyzed 70 targets (88 packages loaded, 13482 targets configured).
INFO: Found 70 targets...
INFO: Elapsed time: 3390.870s, Critical Path: 120.96s
INFO: 5622 processes: 821 internal, 4801 linux-sandbox.
INFO: Build completed successfully, 5622 total actions
bibhuprasad@b621e980bfc1:/sonic/src/sonic-p4rt/sonic-pins$ bazel build $BAZEL_BUILD_OPTS p4rt_app/...
INFO: Analyzed 92 targets (18 packages loaded, 456 targets configured).
INFO: Found 92 targets...
INFO: 760 processes: 421 internal, 339 linux-sandbox.
INFO: Build completed successfully, 760 total actions
bibhuprasad@b621e980bfc1:/sonic/src/sonic-p4rt/sonic-pins$
UT Results :
bibhuprasad@b621e980bfc1:/sonic/src/sonic-p4rt/sonic-pins$
bibhuprasad@b621e980bfc1:/sonic/src/sonic-p4rt/sonic-pins$ bazel test $BAZEL_BUILD_OPTS p4_pdpi/...
INFO: Analyzed 70 targets (0 packages loaded, 0 targets configured).
INFO: Found 44 targets and 26 test targets...
INFO: Elapsed time: 2.604s, Critical Path: 0.38s
INFO: 27 processes: 41 linux-sandbox.
INFO: Build completed successfully, 27 total actions
//p4_pdpi:ir_tools_test                                                  PASSED in 0.4s
//p4_pdpi/netaddr:ipv4_address_and_network_address_test                  PASSED in 0.2s
//p4_pdpi/netaddr:ipv6_address_test                                      PASSED in 0.2s
//p4_pdpi/netaddr:mac_address_test                                       PASSED in 0.2s
//p4_pdpi/string_encodings:bit_string_test                               PASSED in 0.2s
//p4_pdpi/string_encodings:byte_string_test                              PASSED in 0.3s
//p4_pdpi/string_encodings:decimal_string_test                           PASSED in 0.0s
//p4_pdpi/string_encodings:decimal_string_test_runner                    PASSED in 0.0s
//p4_pdpi/string_encodings:hex_string_test                               PASSED in 0.0s
//p4_pdpi/string_encodings:hex_string_test_runner                        PASSED in 0.0s
//p4_pdpi/string_encodings:readable_byte_string_test                     PASSED in 0.2s
//p4_pdpi/testing:helper_function_test                                   PASSED in 0.3s
//p4_pdpi/testing:info_test                                              PASSED in 0.0s
//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.0s
//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.1s
//p4_pdpi/testing:sequencing_test                                        PASSED in 0.0s
//p4_pdpi/testing:sequencing_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.0s
//p4_pdpi/testing:table_entry_test_runner                                PASSED in 0.1s
//p4_pdpi/utils:annotation_parser_test                                   PASSED in 0.2s
//p4_pdpi/utils:ir_test                                                  PASSED in 0.3s
Executed 26 out of 26 tests: 26 tests pass.
INFO: Build completed successfully, 27 total actions
bibhuprasad@b621e980bfc1:/sonic/src/sonic-p4rt/sonic-pins$ bazel test $BAZEL_BUILD_OPTS p4rt_app/...
INFO: Analyzed 92 targets (0 packages loaded, 0 targets configured).
INFO: Found 61 targets and 31 test targets...
INFO: Elapsed time: 24.801s, Critical Path: 5.08s
INFO: 32 processes: 1 internal, 19 linux-sandbox, 12 local.
INFO: Build completed successfully, 32 total actions
//p4rt_app/event_monitoring:app_state_db_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: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_acl_def_table_manager_test                       PASSED in 0.4s
//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:packetio_impl_test                                      PASSED in 0.3s
//p4rt_app/sonic:packetio_port_test                                      PASSED in 0.3s
//p4rt_app/sonic:response_handler_test                                   PASSED in 0.4s
//p4rt_app/sonic:state_verification_test                                 PASSED in 0.2s
//p4rt_app/sonic/adapters:fake_sonic_db_table_test                       PASSED in 0.1s
//p4rt_app/tests:acl_table_test                                          PASSED in 1.0s
//p4rt_app/tests:action_set_test                                         PASSED in 0.6s
//p4rt_app/tests:api_access_test                                         PASSED in 0.6s
//p4rt_app/tests:arbitration_test                                        PASSED in 0.7s
//p4rt_app/tests:fixed_l3_tables_test                                    PASSED in 2.2s
//p4rt_app/tests:forwarding_pipeline_config_test                         PASSED in 3.1s
//p4rt_app/tests:grpc_behavior_test                                      PASSED in 5.1s
//p4rt_app/tests:p4_programs_test                                        PASSED in 1.1s
//p4rt_app/tests:packetio_test                                           PASSED in 3.2s
//p4rt_app/tests:port_name_and_id_test                                   PASSED in 0.8s
//p4rt_app/tests:response_path_test                                      PASSED in 1.4s
//p4rt_app/tests:role_test                                               PASSED in 0.7s
//p4rt_app/tests/lib:app_db_entry_builder_test                           PASSED in 0.0s
//p4rt_app/utils:event_execution_time_monitor_test                       PASSED in 0.2s
//p4rt_app/utils:table_utility_test                                      PASSED in 0.4s
Executed 31 out of 31 tests: 31 tests pass.
INFO: Build completed successfully, 32 total actions
bibhuprasad@b621e980bfc1:/sonic/src/sonic-p4rt/sonic-pins$

Is this ready for review ? Can you fix the conflicts ?

Is this ready for review ? Can you fix the conflicts ?

All the merge conflicts are resolved. Now it is ready for review.

Co-authored-by: Srikishen Pondicherry Shanmugam<EMAIL_ADDRESS>Co-authored-by: Jonathan Dilorenzo<EMAIL_ADDRESS>

@rkavitha-hcl , @bibhuprasad-hcl
Changes in #39 are repeated here, can you pls check.

@bibhuprasad-hcl Pls resolve the conflicts

@bibhuprasad-hcl @rkavitha-hcl Can you check the instructions for the easycla failing check and fix that ?

/easycla

/easycla

@rkavitha-hcl Pls use<EMAIL_ADDRESS>for the missing easycla commit id.
@zhenk14  Once your name shows up, can you pls sign the CLA here.

/easycla


Fixes https://github.com/davisking/dlib/issues/2350

This fixes https://github.com/davisking/dlib/issues/2350

Oh, and I've amended the previous commit message (type_safe_union stuff) to make it more succinct (I need to get better at using git).

Nice, thanks.  Can you clean up this PR though?  It's got changes from the last PR you submitted in it.  Like check out the Files changed tab.
You can't amend commits that are already landed in the master branch.  I mean, you can amend and history edit locally, but pushing them to a repo that already has those changes isn't going to work.

Balls. Yep sorry about that. I won't have time to touch this until Monday. Given how small this is, maybe it's easier if you commit straight to master and I'll close this.

Merged it in.


Does this work with AVA?

I run my tests with AVA, but I never see the status bar update.
No output in the console under TDD.
Thanks.

Yes, unfortunately it doesn't yet, as noted in the known issues. I'll come back and update this thread once I find a way to do that reliably.


[NextJS13] GraphQLClient url undefined when called inside client component

I'm working on a NextJS13 project.
I'm using the version graphql-request ^6.1.0.
I've set up my GraphQLClient like this:
// file: graphqlClient.ts
import { GraphQLClient } from "graphql-request"
export const graphqlClient = new GraphQLClient(process.env.NEXT_INTERNAL_GRAPHQL_URL as string)

Inside a RSC, when I console.log(graphqlClient), I can see that the url is being defined:
import { graphqlClient } from "@/utils/lib/graphql/graphqlClient"

export default async function SignupPage() {
  console.log(graphqlClient)
  /* GraphQLClient {
    url: 'http://host.docker.internal:14000/graphql',
    ...
  } */
}

But when I'm inside a client component the url inside graphqlClient is undefined:
"use client"

import { graphqlClient } from "@/utils/lib/graphql/graphqlClient"

export default async function SignupForm() {
  console.log(graphqlClient)
  /* GraphQLClient {
    url: undefined,
    ...
  } */
}

Am I missing something or is it a bug ?

Ok I just needed to add NEXT_PUBLIC_ before my env variable to make it available to my client components ...
https://nextjs.org/docs/app/building-your-application/configuring/environment-variables#bundling-environment-variables-for-the-browser

Ok I just needed to add NEXT_PUBLIC_ before my env variable to make it available to my client components ...
https://nextjs.org/docs/app/building-your-application/configuring/environment-variables#bundling-environment-variables-for-the-browser


Math magician react setup

Math Magician react setup features

[ ] Initialized project with react
[ ] Eslint and Stylelint linter setup
[ ] Modified readme file

STATUS: INVALID CODE REVIEW REQUEST 🛑 🛑
Hi @Nasirkhan294,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You do not have a descriptive README. Kindly format your README using the Microverse template.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

STATUS: APPROVED ✔ ✔
Hi @Nasirkhan294,
✔ App is well set up with CRA
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Inaccessible nodes in generated trees

Sometimes the generated trees in the model contain nodes that are not accessible, increasing model files unnecessarily
All of the nodes in a tree should be accessible from the root node, but sometimes a node will have no left or right children that refer to the node.
e.g.
node 0: left_child = 1, right_child = 2
node 1: left_child = -1, right_child = -1
node 2: left_child = -1, right_child = -1
node 3: left_child = -1, right_child = -1
node 3 is not accessible from the root
This can be seen clearly using the json encoded model (as demonstrated below), but I have also seen this issue within the binary model files.
import json

import numpy as np
import xgboost as xgb

sample_count = 1000
feature_count = 100

# make dataset

np.random.seed(0)

X = np.random.randn(sample_count, feature_count)
y = np.random.choice(2, sample_count)

dtrain = xgb.DMatrix(X, y)

param = {
    'eta': 0.1,
    'objective': 'binary:logistic',
    'nthread': 1,
    'eval_metric': 'logloss',
    'gamma': 0.1,
}

bst = xgb.train(param, dtrain, num_boost_round=50)

model_filename = 'model0.json'
bst.save_model(model_filename)
with open(model_filename, 'rt') as f:
    m = json.load(f)

trees = m['learner']['gradient_booster']['model']['trees']

for tid, t in enumerate(trees):
    node_count = len(t['left_children'])
    left_children = set(nid for nid in t['left_children'])
    right_children = set(nid for nid in t['right_children'])
    all_children = left_children | right_children
    expected = set(range(1, node_count))
    if not expected <= all_children:
        print('tree:', tid, 'node_count:', node_count,
              'inaccessible node ids:', expected - all_children)

output:
tree: 22 node_count: 73 inaccessible node ids: {72, 71}
tree: 23 node_count: 35 inaccessible node ids: {17, 18}
tree: 33 node_count: 81 inaccessible node ids: {61, 62}
tree: 36 node_count: 55 inaccessible node ids: {53, 54}
tree: 37 node_count: 59 inaccessible node ids: {53, 54}
tree: 44 node_count: 45 inaccessible node ids: {19, 20}
tree: 45 node_count: 77 inaccessible node ids: {67, 68, 53, 54}
tree: 47 node_count: 51 inaccessible node ids: {49, 50}

These are pruned nodes.

I suspected this was the case. The issue is then minor - there is a very slight increase in the size of the model file when this happens (which is infrequent), but there is no impact on predictions.

I want to convert all tree methods to use pre-pruning.  But that will be another topic.  Closing.


Shell script syntax highlights messed up

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.75.0
OS Version: Ubuntu 22.04.1 LTS

Steps to Reproduce:

Open any shell script (more complex ones)
Somehow the syntax highlight is messed up and comments originally in green (comment) became orange (string)

I can confirm the syntax highlight incorrect is due to the 1.75.0 update. Curious.

Its a duplicate of #173336 . closing.


Selecting cycleroutes doesn't work when routing.

Selecting cycleroutes doesn't work when a route is on the map. Expected would be to either remove the current route, show the cycleroute anyway or hide the route selection options.

I do not agree with "or hide the route selection options".
It's necessary to be able to display a route on the map even when the routing result is displayed !

Fixed in 59bfd39, will need to add another filter option to hide all the routes.


ADR: Parallel drafts track & plan for deprecated components

Read the rendered markdown
Context: As we work on maturity of our components, we will sometimes need to deprecate components that are replaced with new components. Here's something to start that conversation and converge on a strategy.
tl;dr: We should detangle "drafts" from component lifecycle.
1. Rename Experimental to Candidate in Component maturity lifecycle: Candidate, Alpha, Beta and Stable.
2. Rename drafts to experimental. A component can have beta maturity in the experimental bundle.
(scrapped the above 2 suggestions for the churn they'll cause in the ecosystem, see this comment for more context. We do have the optional of changing drafts to next or labs if that's more descriptive)
3. Draft components should not be collocated with "main bundle" components in the documentation or status page. They should have their own section, as draft components are not recommended as an alternative yet.

Flowchart:

📺 Here's a video of a dry run of the flowchart: https://github.rewatch.com/video/arc9fb2sivdkm44j-draft-parallel-track-dry-run
Code example for the stages:
// Stage 0:
import { PrimaryButton } from '@primer/react'

// Stage 1: new component starts in drafts

// you can continue using current component
import { PrimaryButton } from '@primer/react'
// or try the new one
import { Button } from '@primer/react/lib-esm/drafts'

// Stage 2: new component goes from drafts to main

// you can migrate to the new component (breaking change, recommended)
import { Button } from '@primer/react'
// or continue to use the old, now deprecated one by changing the import
import { PrimaryButton } from '@primer/react/lib-esm/deprecated'

// Stage 3: deleting the deprecated component
// you need to migrate to the new component (breaking change)
import { Button } from '@primer/react'

This ADR feels silly because we already ran this whole pipeline once 😅
Should we merge it as accepted 🤷♀ and keep improving it?

As discussed among the PRC maintainers group, let's try assigning a decision-maker here so we can move this ADR toward a conclusion.
ADR decision maker: @pksjce
Decision deadline: Thursday, 5 May
I'm going to work on more enduring documentation about this process, but in the meantime:
Your role as decision maker is to gather feedback, probe for discussion and disagreement, and then make an informed decision based on the feedback, identified risks, and trade-offs about whether to adopt the ADR or not. Your role is not to decide based on your own personally preferred approach.
Keep in mind that ADR decisions can be reversed or changed in the future if new information is exposed that make the ADR worth revisiting.
You can fulfill your decision maker role by doing the following:

review the ADR, comments, and feedback closely
seek additional feedback from any necessary parties, as needed; for example, request a review from any individuals or teams that you know would be unduly affected by this ADR, or who have extensive experience in using either the preferred or non-preferred approach and may have perspective to share about it
follow up on any remaining concerns, questions, or unexamined risks around accepting the proposed approach
on or before the deadline, make a decision as to whether the ADR is accepted ✅ , which you should signal by approving this PR, or rejected 🚫 , which you should signal by closing this PR with a comment explaining the reason(s)


[US] Create, Update and Deactivate Cron Schedules

As an application user, I want to be able to modify the scheduling of cron jobs from the UI so that I can customize task execution according to my unique workflow and preferences.

MoSCoW
Risk
Story Points

Must have
Medium
6

Acceptance Criteria

user interface allows for the easy and intuitive scheduling and modification of cron jobs.
Input validation to ensure that the cron syntax entered is correct.
Inclusion of tooltips explaining cron syntax.

Risks

Unauthorized access to scheduling could lead to security vulnerabilities.
Over-scheduling or poor scheduling can strain system resources.
Users might set incorrect schedules leading to system inefficiencies or failures.

Pre-requisites

[x] Meet with the stakeholder to clarify the requirements
[x] Complete the requirements of this US

Tasks

[x] Add schedule and un-schedule functions in cron.sql
[x] Create typescript functions that access and execute the SQL functions
[x] Implement the use of these functions from within the CronTabsJobs Page
[x] Add auto-refresh of tables on button click

Signing off!

Sequence Diagram


Add iterators to classes that don't have one(if applicable)

For example, Min Heap does not have an iterator.Note that it is better to implement the iterators similar to the one's that already exist in all the tree classes as well as the list classes.

Issue closed as i created the iterators.


clamp function in Raylib.Util.Math can be more haskellish

Haskell-styled function should be good at combination.
e.g. clamp
clamp has to be used in this way
clamp Value_to_clamp Lower_bound Upper_bound

more haskellish args ord should be
clamp Lower_bound Upper_bound Value_to_clamp

if change the order of args the endo-function will change
from (\x -> clamp x (-40) 40)
to clamp (-40) 40
which makes this function combine easily with other Float -> Float function
Same thing goes:

normalize
remap
wrap

Resolved by #32


Add v8 patch to preserve async context for custom thenables

Patches v8 to allow async context to propgate to async thenables
@qard ... this probably doesn't tick all of the boxes v8 would need to land this (I skipped tests and there might be some additional debug asserts they'd want in a couple places) but this patch should cover the basics. Could I bother you to give it a once over? ... specifically this patch: patches/v8/0015-Attach-continuation-context-to-Promise-thenable-task.patch

@Qard ... updated to include the factory.cc changes and extended the test to cover that case.

Thank you @Qard !


SetScanFilter is needed

Hi @petreeftime, @andreivasiliu2211,
thank you for your effort, it is a great lib.
I'm wondering if you could add "Adapter.SetDiscoveryFilter" method to tinyb library? This is really needed by a plugin for OpenHAB project.
https://git.kernel.org/cgit/bluetooth/bluez.git/tree/doc/adapter-api.txt
Particularly we are interested in the "RSSI" filter.
Thanks,
Vlad

Hey @vkolotov I implemented this, however have zero time to test it tonight.  If you have the time to test please test the latest master of my fork here.  Otherwise I can get to it in a day or two.

thanks mate, I'm busy now as well. Will be able to test it on weekends.
However, I'm not quite sure how would I bind std::vector to a java object (array or list)...

You need to declare in java class like so: https://github.com/intel-iot-devkit/tinyb/blob/master/java/BluetoothGattService.java#L91
Then, on the C++ side you need a function like so: https://github.com/intel-iot-devkit/tinyb/blob/master/java/jni/BluetoothGattService.cxx#L145, more specifically, this function: https://github.com/intel-iot-devkit/tinyb/blob/master/java/jni/helper.hpp#L84 converts a C++ vector of type T to a java List of objects of type T.

@petreeftime I finally got down to this. That helper method you mentioned above (https://github.com/intel-iot-devkit/tinyb/blob/master/java/jni/helper.hpp#L84) converts a c++ object to a java object. In our case we need to convert the other way around. Do we have that method already anywhere? Any examples? Thanks.


InitializeInEditor() method name is generic, obscures its purpose

"InitializeInEditor" doesn't say what it is initializing or why. There's nothing hinting at this being part of the DI system.
I would prefer the method be called InjectReferencesInEditor and likewise InjectReferencesAtRuntime.

Next I will work on this package a little in the immediate future, will take a look! Thanks for the feedback!

See Changelog


Infusion of Light (Holy Paladin Critical Holy Shock Proc) missing

Currently you only implemented "Art of War" for Ret Paladins.
There's a proc if you do a critical hit with Holy Shock, called "Infusion of Light" (https://www.wowhead.com/wotlk/spell=53576/infusion-of-light).
Looking like this on retail: https://i.imgur.com/EdJE0S8.png
If you can please implement it. I will also try to create a pull request asap if I find time. :)

Thanks for the suggestion. Looks like an excellent candidate.
The latest branch implements this spell.
It procs when one of these buffs is found:

Infusion of Light (Rank 1)
Infusion of Light (Rank 2)

I’m currently using the "Daybreak" texture:

Here is a demonstration of what I've come up with: Infusion of Light
What do you think of it?
If you have a beta key, you can test it by installing the addon manually:
https://github.com/ennvina/spellactivationoverlay/archive/refs/heads/infusion-of-light.zip
(you will need to rename the folder from spellactivationoverlay-infusion-of-light to SpellActivationOverlay)

Implemented in #32


Delete After Eviction might cause leak

If we get a blocked eviction on pdb watcher and then just a pure delete we will not notice the delete and not scale down the target.
Likely we need to watch pod deletion or the pdb itself to catch this.

This is solved with #36


allow proxy to convert to schema1 without connectivity to remote

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "proxy-patch" git@github.com:marcospedreiro/distribution.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


navbar-toggle problem

Hi, how are you.
I am writing this message because i have a problem, i'm creating a web site and i'm usin Bootstrap 5, i'm doing a main menu that when the website is running in a mobile it gets small, and when it gets small the menu is hidden and at that moment apear the toggle button, but when y clic on the button nothing happens, i believe i have the code right so i don't know wath si happening.
I really hope you help me, thank you very much.
(However i attach my code in case i have something wrong)
Code below:
This is the button:
<button class="navbar-toggler position-absolute d-md-none collapsed" type="button" data-bs-toggle="collapse" data-bs-target="#sidebarMenu" aria-controls="sidebarMenu" aria-expanded="false" aria-label="Toggle navigation">

</button>
And this is the nav  i want to show:

*another code goes here

Bug reports must include a live demo of the issue. Per our contributing guidelines, please create a reduced test case via CodePen or JS Bin and report back with your link, Bootstrap version, and specific browser and operating system details.

This is a saved reply.
Also, make sure you've included the JS.


Added validation of file type

Added validation of file type
Sample Config:
$config = new Config([
'tempDir' => DIR . '/tmp',
'mimeAccept' => array(
'image/gif',
'image/jpeg',
'image/png',
'image/bmp'
)
]);

Coverage decreased (-7.89%) to 92.11% when pulling 252e55b941d76afb409b98641c60739887b5d8c5 on antkowiakit:master into de7151c7b4cf8617cbee891efe42639d64a27d01 on flowjs:master.

Bump

Hi, thanks, I have left a comment. I think we need to refactor this in a more flexible manner


Remove the obsolete nginx/nginx_site.conf.tmpl

This was stopped being used very early on, as it didn't promote reuse with each project copying the full template to make changes

Build for PR had passed but not updated status. Merging!


Add: frontend input validation for add restaurant component and login screen

Codecov Report

Merging #74 (4707d10) into main (9c85153) will increase coverage by 0.09%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main      #74      +/-   ##
==========================================
+ Coverage   88.79%   88.88%   +0.09%
==========================================
  Files          40       41       +1
  Lines         366      369       +3
  Branches       40       38       -2
==========================================
+ Hits          325      328       +3
  Misses         37       37
  Partials        4        4

Impacted Files
Coverage Δ

...omponents/restaurant-details/RestaurantDetails.tsx
91.66% <0.00%> (ø)

.../components/restaurant-list/list-card/ListCard.tsx
100.00% <ø> (ø)

frontend/src/components/review-card/ReviewCard.tsx
100.00% <ø> (ø)

frontend/src/components/login/Login.tsx
28.57% <16.66%> (+1.74%)
:arrow_up:

frontend/src/components/star-rate/StarRate.tsx
90.00% <66.66%> (ø)

...nd/src/components/add-restaurant/AddRestaurant.tsx
100.00% <100.00%> (ø)

frontend/src/components/add-review/AddReview.tsx
100.00% <100.00%> (ø)

frontend/src/util/joiValidators.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Bypassing login screen using URL params, no remote feed displayed

Please see my issue posted here: https://stackoverflow.com/questions/65943776/not-recieving-remote-video-stream-when-accessing-by-directly-modifying-url-param
I'm clueless as to the cause and would be grateful for any support.
Many thanks.

thanks @miroslavpejic85 for the solution


Material and Section library are missing when running Grasshopper script locally

Material and Section library are missing when running Grasshopper script locally
Originally posted by @andosca in https://github.com/strusoft/femdesign-api/issues/296#issuecomment-1092540529

@ShahoRuhani


What is sufficient balance?

What amount of xpx is sufficient balance?
Diskspace isn't involved? What means 1 month / 172800?

https://github.com/proximax-storage/sirius-storage-mainnet-onboarding/blame/main/director/README.md#L75
will update 172800 to 172800 blocks (30days x 5760 blocks per day)
https://storagedocs.xpxsirius.io/docs/cli/dfms/contract/#examples
Diskspace is mentioned when creating drive "space": 100 where 100 refers to 100MB

Clear


Feature request: delay notifications until business hours

I would love to see a setting where I can set business hours and outside of those, all notifications would be either muted or delayed until the next business day.
A switch that lets mentions, channels or mentions in specific channels ignore this would probably be useful too.

Tracking this in the zulip/zulip repo, since we'd want the implementation of this feature to live on the server. There are two issues for it:

https://github.com/zulip/zulip/issues/6629
https://github.com/zulip/zulip/issues/9265

Android, at least, has a "do not disturb" feature at the OS level that you can do this with.

This is true (and iOS has a similar feature too) but for a lot of users it doesn't solve the problem. The "do not disturb" feature is designed much more for "it's night and I don't want any notifications, unless it's something important" than for "it's out of business hours and I don't want any notifications from work, only from non-work", which is the use case this request is about. It'd be a pain to configure it to serve the latter use case, and it's basically impossible to do so while also using it for the intended use case.


Calling Auth::User()->Model->first()->first() gives unauthorized result

Hi there :smile:
I came across this, and though its my/devs fault for doing something stupid like calling first() twice, it would be great if it would give the first authorized result back or maybe just an error, and not just the first found record.
Here's an example of something I was doing (wrong):
$project = \Auth::User()->projects->first()
[...]
$project->first()->id; // Gives the ID of the first found project record, even when its not relational to the user.

I almost overlooked this but luckily noticed some IDs in the DB were off, as there isn't any error it can be overlooked quickly.

Did another test, also unlikely scenario that it happens, but also gives the same like result.
dd( \Auth::User()->projects()->find(14)->id ); // 14 is related to this user, all good.
dd( \Auth::User()->projects()->find(15)->id ); // 15 is NOT of this user, gives back an error, all good
dd( \Auth::User()->projects()->find(14)->find(15)->id ); // Gives back '15', not from this user, undesired result

Auth::User()->projects()->first() is what you want

// Gives back '15', not from this user, undesired result

That is intended.

$project->first()->id; // Gives the ID of the first found project record, even when its not relational to the user.

That's because the project object is just the project, and has nothing to do with relations.
At this point your code is the same as writing Plan::first()->id.


Updated Newtonsoft.Json nuget to 8.0.1.

We needed the latest version (8.0.1) of Newtonsoft.Json nuget to be referenced by Force.com-Toolkit-for-Net. I went through updated the package reference and compiled. It appears to work fine. I'm going through production testing now.
I thought I'd share the changes which were only the update to the reference. If it's useful for you, feel free to merge.

There appears to be errors involved in the implementation.


Dragging stacks of blocks over other stacks disconnects them

Expected Behavior
Dragging a stack of blocks around the screen has no lasting impact on other blocks / scripts that are not part of the drag source or target.
Actual Behavior
As a stack of two or more blocks is dragged over other block stacks, the other stacks become disconnected (although they stay positioned as if they were still connected).
Steps to Reproduce
Here's a screen capture of me showing the behavior of dragging two blocks over a stack in a chrome browser. Notice how 'all' blocks are detached once the drag is complete even though I didn't release the drag over the stack? This is happening on https://llk.github.io/scratch-gui/ has this page got the latest changes from github?

Operating System and Browser
Windows 10 with Chrome

This line is probably the culprit: https://github.com/LLK/scratch-blocks/blob/develop/core/connection.js#L376


Demo application

Looks good.
Should this be in this repo, or in the ds-auth-omniauth repo? Cause its actually a demo of how to use that gem, not the auth app itself directly...
Also there are a few scaffold files that can be removed as they are not needed for the demo eg app/assets, initialiser files that are to do with other aspects of the rails app etc


Can't get history list correctly in case of annotated tag

It was possible to get the history even with annotated tag


Code snippet page usage and style tabs not working

Steps to recreate:

go to https://www.carbondesignsystem.com/components/code-snippet/code
try to click on the usage or style tab
notice it doesn't change the page

closing as duplicate


Re-enable incremental compilation

Rust 1.55 has been released, re-enables incremental compilation as detailed in #1010.

Looks like there's a ci glitch 👀

Looks like there's a ci glitch :eyes:

Shouldn't this branch be against staging, though :detective:?

good catch 👍


Pull more optimize and simplification changes from tuner branch

Description
Various minor tweaks for optimization, simplification and leading up to tuner
Remove redundant ops and function KernelCompute
Launch calls OpenMP class directly instead of engine (engine calls into OpenMP class to reserve cores)
Unit testing/perf testing tweaks
Checklist
Essentials

[ ] Passed code style checking (make lint)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] For user-facing API changes, API doc string has been updated.
[ ] To my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Intersting edge cases to note here

I am doing rebase, no idea why it is showing baaafc5


Missing expected argument for --router_lookahead

Looks like there is a failure on the latest commit of https://github.com/SymbiFlow/symbiflow-arch-defs/commit/6575fce443854c489f621e1f7e86da3f5e562494
See the failure at https://source.cloud.google.com/results/invocations/e8b064d0-ea51-447f-a1e5-631a71dc706c/log
Missing expected argument for --router_lookahead

usage: vpr architecture circuit [--pack] [--place] [--route] [--analysis]
       [--disp {on, off}] [--save_graphics {on, off}] [-h] [--version]
       [--device DEVICE_NAME] [-j NUM_WORKERS] [--timing_analysis {on, off}]
       [--disable_errors DISABLE_ERRORS] [--suppress_warnings SUPPRESS_WARNINGS]
       [--route_chan_width CHANNEL_WIDTH] [OTHER_OPTIONS ...]

Something weird is going on there is a --router_lookahead connection_box_map in the output...
[939/1912] Generating rr_graph_xc7a50t-basys3_test.rr_graph.real.xml.cache, rr_graph_xc7a50t-basys3_test.lookahead.bin, rr_graph_xc7a50t-basys3_test.place_delay.bin
FAILED: cd /tmpfs/src/github/symbiflow-arch-defs/build/xc7/archs/artix7/devices && /tmpfs/src/github/symbiflow-arch-defs/build/env/conda/bin/python3 /tmpfs/src/github/symbiflow-arch-defs/utils/check_cache.py /tmpfs/src/github/symbiflow-arch-defs/build/xc7/archs/artix7/devices/rr_graph_xc7a50t-basys3_test.rr_graph.real.xml /tmpfs/src/github/symbiflow-arch-defs/build/xc7/archs/artix7/devices/rr_graph_xc7a50t-basys3_test.rr_graph.real.xml.cache rr_graph_xc7a50t-basys3_test.lookahead.bin rr_graph_xc7a50t-basys3_test.place_delay.bin || ( /tmpfs/src/github/symbiflow-arch-defs/utils/quiet_cmd.sh /tmpfs/src/github/vtr-verilog-to-routing/build/vpr/vpr /tmpfs/src/github/symbiflow-arch-defs/build/xc7/archs/artix7/devices/xc7a50t-basys3-roi-virt/arch.timing.xml --device xc7a50t-basys3-test /tmpfs/src/github/symbiflow-arch-defs/common/wire.eblif --read_rr_graph /tmpfs/src/github/symbiflow-arch-defs/build/xc7/archs/artix7/devices/rr_graph_xc7a50t-basys3_test.rr_graph.real.xml --outfile_prefix xc7a50t-basys3_test_cache --pack --place --write_router_lookahead rr_graph_xc7a50t-basys3_test.lookahead.bin --write_placement_delay_lookup rr_graph_xc7a50t-basys3_test.place_delay.bin --constant_net_method route --clock_modeling route --place_delay_model delta_override --router_lookahead connection_box_map --disable_errors check_unbuffered_edges:check_route:check_place --suppress_warnings sum_pin_class:check_unbuffered_edges:load_rr_indexed_data_T_values:check_rr_node:trans_per_R --route_chan_width 500 --allowed_tiles_for_delay_model BLK-TL-SLICEL,BLK-TL-SLICEM && /tmpfs/src/github/symbiflow-arch-defs/build/env/conda/bin/python3 /tmpfs/src/github/symbiflow-arch-defs/utils/update_cache.py /tmpfs/src/github/symbiflow-arch-defs/build/xc7/archs/artix7/devices/rr_graph_xc7a50t-basys3_test.rr_graph.real.xml /tmpfs/src/github/symbiflow-arch-defs/build/xc7/archs/artix7/devices/rr_graph_xc7a50t-basys3_test.rr_graph.real.xml.cache ) && /usr/bin/cmake -E copy vpr_stdout.log rr_graph_xc7a50t-basys3_test.cache.out

Is it related to https://github.com/SymbiFlow/symbiflow-arch-defs/issues/1187 ?

Is this the right VPR version?
Version: 8.0.0-rc1+2780988d5
Revision: v8.0.0-rc1-1212-g2780988d5
Compiled: 2019-12-20T06:39:59
Compiler: GNU 5.4.0 on Linux-4.4.0-116-generic x86_64
Build Info: release IPO VTR_ASSERT_LEVEL=3

This bug is because the kokoro CI is using the master branch of Symbiflow/vtr rather than the master+wip branch, and as a result it errors out because the modifications required to run the symbiflow-arch-def flow are missing.  It is unclear if this was the intention when this target was set up.
There is another kokoro configuration that uses Symbiflow/vtr master+wip, and that one works as expected.

This will be fixed once upstream VPR can run with symbiflow-arch-defs, which is soon!

Is this fixed? I'm seeing this error when using --router_lookahead connection_box_map on current master.
VPR FPGA Placement and Routing.
Version: 8.1.0-dev+d6d69ff92
Revision: v8.0.0.rc2-5415-gd6d69ff92
Compiled: 2020-11-20T18:17:03
Compiler: GNU 7.3.0 on Linux-4.15.0-1077-gcp x86_64
Build Info: Release IPO VTR_ASSERT_LEVEL=2

University of Toronto
verilogtorouting.org
<EMAIL_ADDRESS>This is free open source code under MIT license.

Missing expected argument for --router_lookahead

usage: vpr architecture circuit [--pack] [--place] [--route] [--analysis]
       [--disp {on, off}] [--save_graphics {on, off}]
       [--graphics_commands GRAPHICS_COMMANDS] [-h] [--version]
       [--device DEVICE_NAME] [-j NUM_WORKERS] [--timing_analysis {on, off}]
       [--disable_errors DISABLE_ERRORS] [--suppress_warnings SUPPRESS_WARNINGS]
       [--route_chan_width CHANNEL_WIDTH] [OTHER_OPTIONS ...]
make: [Makefile:77: output_fun/out.net] Error 1 (ignored)

I'm using vpr from: symbiflow-arch-defs/env/conda/envs/symbiflow_arch_def_base/bin/vpr. Any help is appreciated!

It's fixed in the sense that we no longer used connection_box_map at all?  We've actually just removed it from our build of VPR, and are currently using upstream VPR directly.

PR with relevant change: https://github.com/SymbiFlow/symbiflow-arch-defs/pull/1667

Thanks! That worked.


YAML support

Hello!
This tool is pretty helpful, thanks for it!
But how about the swagger YAML format support for a converter?

Added YAML support at 0.9.0, thanks!


Updated Targets plugin

Can you update the date in the "updated" field as well?

@dkoper Sorry, missed that. Done now.

Thanks!


Error after error.

Hi friend. I came across this repository and it looked neat so I cloned it down and followed the installation instructions to a T. Unfortunately, I'm running into multiple issues. All of the issues I've tracked down thus far have just been small programming errors, though.
What I'm wondering is, if I take the time to fix them, will a merge-request be accepted?

I'm not actively maintaining this repository right now. I have an unfinished cleanup/partial rewrite branch in my local clone (lots of the code here is written in haste and is pretty bad), but I somewhat gave up on it. But sure, I may still merge your fixes.
But first, please tell me about your system (OS, Python version, Pygame version, etc.), and give me a few examples of the errors you've encountered. I've tested this on a few Linux and Windows systems, and haven't ran into any issues so far.


[WIP] render subpkg for managing req/resp payloads

See #84

Closing this branch, replacing it with newer work.


Tweak number-count function to treat 20000 as a number

I am also thinking about making changes to the count number function. Currently, (1) it takes 20000 as a year (becasue '20000' contains 2000), and (2) does not count a number that appears at the end of a text unless the number is followed by a black. I make the following changes:
def number_count(doc):

  doc = re.sub('(?!=[0-9])(\.|,)(?=[0-9])', '', doc)
  doc = doc.translate(str.maketrans(string.punctuation, " " * len(string.punctuation)))

  doc = re.findall(r'\b[-+\(]?[$€£]?[-+(]?\d+\)?\b', doc)
  doc = [x for x in doc if not re.match(r'(199|20[01])\d{1}?\b', x)]
  return len(doc)

Originally posted by @yiyangw2 in https://github.com/iangow/ling_features/issues/4#issuecomment-817965428

@yiyangw2 I would suggest making a test notebook for this one. Provide some examples where the current function fails, and some examples where it works, then these can form tests for the new function.

It seems that the overwhelming majority of detected "years" are years. It might be worth looking at some of the cases that look like bad matches (e.g., 2017357).
For details on ~ and regexp_matches, see here.
library(dplyr, warn.conflicts = FALSE)
library(DBI)

pg <- dbConnect(RPostgres::Postgres(),
                bigint = "integer",
                check_interrupts = TRUE)

rs <- dbExecute(pg, "SET work_mem TO '3GB'")

speaker_data <- tbl(pg, sql("SELECT * FROM streetevents.speaker_data"))

# Note that PostgreSQL doesn't have \b and using R means I need to "escape"
# backslashes. But it can be faster to process regexes in the database when
# possible.
year_regex <- "((?:199|20[01])\\d\\w*)\\W"

year_data <-
  speaker_data %>%
  filter(speaker_text %~% year_regex) %>%
  mutate(years = regexp_matches(speaker_text, year_regex)) %>%
  select(file_name, last_update, context, section, speaker_number, years) %>%
  compute()

year_data %>%
  mutate(year = unnest(years)) %>%
  count(year) %>%
  filter(n > 1) %>%
  arrange(desc(n)) %>%
  print(n = 50)
#> # Source:     lazy query [?? x 2]
#> # Database:   postgres [iangow@/tmp:5432/crsp]
#> # Ordered by: desc(n)
#>    year         n
#>    <chr>    <int>
#>  1 2019    304159
#>  2 2018    228285
#>  3 2010    211250
#>  4 2017    206699
#>  5 2015    206663
#>  6 2016    196445
#>  7 2014    194728
#>  8 2012    194355
#>  9 2011    192004
#> 10 2008    187504
#> 11 2009    184163
#> 12 2013    179339
#> 13 2007    155638
#> 14 2006    140269
#> 15 2005    128132
#> 16 2004    113174
#> 17 2003     88523
#> 18 2002     50922
#> 19 2000     36374
#> 20 1995     33795
#> 21 2001     24760
#> 22 1999      7419
#> 23 1998      5592
#> 24 1990s     5556
#> 25 1997      4183
#> 26 2000s     3695
#> 27 1996      3413
#> 28 1990      3265
#> 29 1994      2661
#> 30 1992      2282
#> 31 1993      2197
#> 32 1991      1884
#> 33 2010s       89
#> 34 2013s       69
#> 35 2017s       53
#> 36 2014s       50
#> 37 2012s       46
#> 38 2016s       41
#> 39 2015s       41
#> 40 2019s       40
#> 41 2018s       38
#> 42 2011s       28
#> 43 20000       27
#> 44 20009       22
#> 45 2011and     17
#> 46 2009s       15
#> 47 20022       15
#> 48 2007s       14
#> 49 2017357     14
#> 50 20111       13
#> # … with more rows

Created on 2021-04-12 by the reprex package (v2.0.0)

So 2017357 is part of CK-2017357, which is a product name (not a year!).
You might find it easier to use compute(name = "year_data", temporary = FALSE) and then switch to Python (should be small enough to use pd.read_sql if you use WHERE to filter out rows).
library(dplyr, warn.conflicts = FALSE)
library(DBI)
library(stringr)
library(tidytext)

pg <- dbConnect(RPostgres::Postgres(),
                bigint = "integer",
                check_interrupts = TRUE)

rs <- dbExecute(pg, "SET work_mem TO '3GB'")

speaker_data <- tbl(pg, sql("SELECT * FROM streetevents.speaker_data"))

# Note that PostgreSQL doesn't have \b and using R means I need to "escape"
# backslashes. But it can be faster to process regexes in the database when
# possible.
year_regex <- "((?:199|20[01])\\d\\w*)\\W"

year_data <-
  speaker_data %>%
  filter(speaker_text %~% year_regex) %>%
  mutate(years = regexp_matches(speaker_text, year_regex)) %>%
  select(file_name, last_update, context, section, speaker_number, years) %>%
  compute()

word_to_check <- "2017357"

temp <-
  year_data %>%
  mutate(year = unnest(years)) %>%
  filter(year == word_to_check) %>%
  inner_join(speaker_data) %>%
  collect()
#> Joining, by = c("file_name", "last_update", "context", "section", "speaker_number")

temp %>%
  unnest_sentences("sentence", speaker_text) %>%
  filter(str_detect(sentence, word_to_check)) %>%
  mutate(context = str_extract(sentence,
                               str_c(".{0,50}", word_to_check, ".{0,50}"))) %>%
  select(context)
#> # A tibble: 16 x 1
#>    context
#>    <chr>
#>  1 "age programs, including omecamtiv mecarbil and ck-2017357, which we'll refe…
#>  2 "ate from our skeletal muscle activator program ck-2017357 or ck-357; one in…
#>  3 "age programs, including omecamtiv mecarbil and ck-2017357, which we'll refe…
#>  4 "age programs, including omecamtiv mecarbil and ck-2017357, which we'll refe…
#>  5 "ur most recent developments in connection with ck-2017357, which we'll refe…
#>  6 "omere activator program and our drug candidate ck-2017357, or ck-357, which…
#>  7 "hase iia evidence of effect clinical trials of ck-2017357 or ck-357; one in…
#>  8 "and as fady has described, our compound ck-2017357, or we will just call it…
#>  9 "ctives on the phase iia clinical trial data of ck-2017357 in patients with …
#> 10 "covery and advancement of omecamtiv as well as ck-2017357."
#> 11 " the agenda; first speaking to a rationale for ck-2017357 in als, dr."
#> 12 "ities, next steps for our clinical program for ck-2017357, which we'll refe…
#> 13 "speaking, as ritu pointed out, to our compound ck-2017357."
#> 14 "carbil and tirasemtiv, formerly referred to as ck-2017357."
#> 15 "adoption of tirasemtiv as the generic name for ck-2017357."
#> 16 "l development of tirasemtiv, formerly known as ck-2017357, for the potentia…

Created on 2021-04-12 by the reprex package (v2.0.0)

For example, run this code:
library(dplyr, warn.conflicts = FALSE)
library(DBI)

pg <- dbConnect(RPostgres::Postgres(),
                bigint = "integer",
                check_interrupts = TRUE)

rs <- dbExecute(pg, "SET work_mem TO '3GB'")

speaker_data <- tbl(pg, sql("SELECT * FROM streetevents.speaker_data"))

# Note that PostgreSQL doesn't have \b and using R means I need to "escape"
# backslashes. But it can be faster to process regexes in the database when
# possible.
year_regex <- "((?:199|20[01])\\d\\w*)\\W"

rs <- dbExecute(pg, "DROP TABLE IF EXISTS year_data_temp")

year_data <-
  speaker_data %>%
  filter(speaker_text %~% year_regex) %>%
  mutate(years = regexp_matches(speaker_text, year_regex)) %>%
  select(file_name, last_update, context, section, speaker_number, years) %>%
  compute(name = "year_data_temp", temporary = FALSE)

Created on 2021-04-12 by the reprex package (v2.0.0)
Then, this part could be done in Python:
library(dplyr, warn.conflicts = FALSE)
library(DBI)
library(stringr)
library(tidytext)

pg <- dbConnect(RPostgres::Postgres(),
                bigint = "integer",
                check_interrupts = TRUE)

rs <- dbExecute(pg, "SET work_mem TO '3GB'")

speaker_data <- tbl(pg, sql("SELECT * FROM streetevents.speaker_data"))

year_data<- tbl(pg, "year_data_temp")
word_to_check <- "2011and"

temp <-
  year_data %>%
  mutate(year = unnest(years)) %>%
  filter(year == word_to_check) %>%
  inner_join(speaker_data) %>%
  collect()
#> Joining, by = c("file_name", "last_update", "context", "section", "speaker_number")

temp %>%
  unnest_sentences("sentence", speaker_text) %>%
  filter(str_detect(sentence, word_to_check)) %>%
  mutate(context = str_extract(sentence,
                               str_c(".{0,50}", word_to_check, ".{0,50}"))) %>%
  select(context)
#> # A tibble: 17 x 1
#>    context
#>    <chr>
#>  1 we have posted and the outlook going forward into 2011and beyond.
#>  2 the book-to-bill ratios for 2011and 2012, what is our degree of confidence i…
#>  3 do you see that impact demand this year then into 2011and beyond?
#>  4 ke to do if i can is start with a quick review of 2011and then finish up bef…
#>  5 is that we could potentially see well activity in 2011and i wouldn't expect …
#>  6 p brands and products for strong growth in fiscal 2011and beyond.
#>  7 timated levels of revenue in q4 and the full year 2011and ebitda margin for …
#>  8 these statements speak only as of april 21, 2011and pnc undertakes no obliga…
#>  9 competition will be there throughout 2011and beyond.
#> 10 y, i would love to get an idea that as we go into 2011and hopefully start se…
#> 11 leases, your lease business, is up for renewal in 2011and 2012, and is it mo…
#> 12 ectivity's second-quarter results for fiscal year 2011and our updated outloo…
#> 13 we'll grow reserves in 2011and increase production over 2010 levels.
#> 14 ou for attending eastplat's conference call on q4 2011and year 2011 financia…
#> 15 l report on form 10-k for the year ended june 30, 2011and other filings, par…
#> 16 ur employees focused on delivering our results in 2011and into 2012, but obv…
#> 17 very positive results for 2011and a lot of positive momentum at national pen…

Created on 2021-04-12 by the reprex package (v2.0.0)
It seems that 2011and is picking up years, as each case should read "2011 and". Not much one can do with these … there will be errors in a database this large.

Thanks a lot for your explanation! That makes sense, we should take 2011and as a year (not a number).
By the way, I was wondering which R package has the function of unnest_sentences. I tried stringr, tidyverse, and lexRankr, but it does not work.

tidytext

we should take 2011and as a year (not a number).

We shouldn't adapt the function too much for specific cases. I guess we might want to exclude any case where letters are involved from number_count. So for now don't add 2011and to the "is_year" regex.

Thanks a lot for your explanation! I begin to see the tradeoff. Being exact has its costs. BTW, tidytext works!

I find that 2020 is a not a rare case. So should we also include it?
def number_count(raw):
""" Function to count the number of numbers appearing in a
    passage of text.
"""
results = re.findall(r'\b(?<=-)?[,0-9\.]+(?=\s)', raw)
results = [result for result in results
            if not re.match(r'(199|20[012])\d', result)
              and re.search(r'[0-9]', result)]
return len(results)

`

The frequency 2000 is catched by your code is more than that of 2019

I find that 2020 is a not a rare case. So should we also include it?
def number_count(raw):
""" Function to count the number of numbers appearing in a
    passage of text.
"""
    results = re.findall(r'\b(?<=-)?[,0-9\.]+(?=\s)', raw)
    results = [result for result in results
        if not re.match(r'(199|20[012])\d', result)
              and re.search(r'[0-9]', result)]
    return len(results)

Yes. Perhaps look at r'(199|20[0123])\d' just in case firms are talking about 2030s already.

Yes, I checked and as you can see, they are often decent amount. I checked their content and I think we should use r'(199|20[0123])\d'


Tweak

Little trick that make the Window non movable when we click on the close button, that way we can cancel the 'close' action if we move out the mouse

Good idea. Thanks for the PR!


A Fixer to migrate "${foo}" to "{$foo}"

Feature request
PHP 8.2 will deprecate certain string interpolations, see https://wiki.php.net/rfc/deprecate_dollar_brace_string_interpolation
I've searched a legacy codebase I'm currently working with for occurences of this deprecated syntax and found ~4k affected LOC. In order to make my code forward-compatible with PHP 8.2, I'd like to run a fixer on a codebase that converts all deprecated interpolations ("${foo}") to the non-deprecated equivalent ("{$foo}" or "$foo").
I think, this would be a great addition to PHP CS Fixer.

We already have a rule to convert from "${foo}" to "{$foo}": simple_to_complex_string_variable 🙂

Awesome, I'll try that! Thank you and sorry that I didn't look into the documentation before opening the issue. 😓

No worries :)


Design and implement service APIs

This tracks the work to design the APIs for the document Service APIs endpoint.
This includes:

Service APIs -- GetStatistics

Here are a couple of quick links to help with that first SearchServiceClient.GetStatistics method to get the client hierarchy in place:
Source: https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/search/Azure.Search/src/SearchServiceClient.cs#L201
Single Test ported from Track 1: https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/search/Azure.Search/tests/SearchServiceClientTests.cs#L133


Question: did you try to package your project and check how it plays on consumer side?

First, thanks for the useful example.
I was searching for the one working in .Core.
I have my own T4 (MS VS based) solution in https://github.com/dadhi/DryIoc/blob/master/src/DryIoc/Container.Generated.tt,
but it does not work in .Net Core, and randomly works at all.
The main problem is that I want to publish tt and ttinclude files to content-based nuget package, and find a multiple issues on the package consumer side, like wrong pathes, unable to load assemblies, etc.
Did you try to package you project and check what's happening on the other side?

Well, I didnt try to package that. I decided to translate my whole project from C# to C++ as the result of research. Im developing automated translator for that task.

Ok, thanks.


next_cursor should be None if there are no more results.

Description
Describe your issue here.
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[x] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
slackclient version: 2.5
python version: any
OS version(s): any
Steps to reproduce:

Call any method that has pagination
If there are more results, next_cursor is set.
If there are no more results, next_cursor is set. (to "")

Expected result:
If there are no results, next_cursor should be removed from response_metadata instead of being set to ""
Setting next_cursor to "" means everyone needs to do ugly things like
            while resp.get('response_metadata').get('next_cursor') != '':

Actual result:
What actually happened
Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc.

Thanks for the response.  I think it makes sense for a library to convert
non-native values to native values in some cases. (i.e. converting null to
None, etc.)
I can also understand how this may not be a preferred pattern for some, or
cause issues with code that's already been written.  But maybe it could be
considered for a new major version.
--Adam
On Wed, Jan 29, 2020 at 6:01 PM Ankur Oberoi<EMAIL_ADDRESS>wrote:

Thanks for sharing your idea. I agree that it would be easier to think of
next_cursor as an key in the response_metadata dictionary that is set to
the value None when there's no more pages left.
However, we try not to apply changes in the data returned from Slack
within the package itself. We feel as though modifications like this create
problems with consistency, and make it harder to have an intuition about
what to expect when you read general documentation on api.slack.com.
We could reconsider this if there's a strong case for how it might
simplify the code for most developers. For now, I'd suggest using the
response values from pagination enabled methods as iterators, as the
following example shows:
https://github.com/slackapi/python-slackclient/blob/40595e11ac64da8dd33098e3243b6b928ce756df/slack/web/slack_response.py#L24-L41
This should make it unnecessary to check for the presence of next_cursor
on your own.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/slackapi/python-slackclient/issues/603?email_source=notifications&email_token=AJH7D3PKZF433HNNMG3IWDTRAIYGBA5CNFSM4KKLM4EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKJNWRA#issuecomment-580049732,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJH7D3KF4I2DMXBHD3WZHZTRAIYGBANCNFSM4KKLM4EA
.


I cannot open BoilR. (Nobara 36)

Describe the bug
I cannot open BoilR with Flatpak or with normal one.
To Reproduce
Idk just use Nobara 36 GNOME.
Expected behavior
It should open and popup, but it doesn't do anything.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Nobara Linux 36
BoilR Version [e.g. v.1.4.5]
Launcher Heroic

nvm

What was the problem?
On Fri, Nov 11, 2022, 21:50 noemsus @.***> wrote:

nvm
—
Reply to this email directly, view it on GitHub
https://github.com/PhilipK/BoilR/issues/275#issuecomment-1312181179, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEE7DAUK563UVB4ATR6UVTWH2WQFANCNFSM6AAAAAAR55Q4BA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Publish to the official plugin repository

Hello,
Thanks for working on the plugin!
Is there a chance it will be published to the official plugin repository?

in progress

https://plugins.jetbrains.com/plugin/23659-advanced-java-folding-2-fork-


Don't know how to uncompress image

Run bash /home/runner/work/_actions/pguyot/arm-runner-action/v2.5.2/download_image.sh https://dietpi.com/downloads/images/DietPi_RPi-ARMv8-Bullseye.7z
Don't know how to uncompress image  *
Error: Process completed with exit code 1.
Run case true in
umount: /tauri-linux: no mount point specified.
Error: Process completed with exit code 32.

The url to this image doesn't exist anymore. The new url is:
https://dietpi.com/downloads/images/DietPi_RPi-ARMv8-Bullseye.img.xz
As a workaround you can manually add the new url inside your workflow like this:

uses<EMAIL_ADDRESS>with:
base_image: https://dietpi.com/downloads/images/DietPi_RPi-ARMv8-Bullseye.img.xz

This was fixed by #95


⚠ Bofrost.fr has degraded performance

In 44032de, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1949 ms

Resolved: Bofrost.fr performance has improved in 7249631 after 10 minutes.


Deploy logging to all AWS giantswarm testing installations

isn't that a duplicate from this one : https://github.com/giantswarm/roadmap/issues/2639 ?


Movie Page under Casting Button Crashed App

Movie Page under Casting Button Crashed App

i get error on crash reports
adapter.ObjectAdapter.getView (ObjectAdapter.java:61)


docs(CONTRIBUTING.md): use the name of this project

Summary
Previously, the Contributing guide specified github-project as a reference to this project's name.
This PR updates the name to it's repository name with owner, copilot-extensions/preview-sdk.js.

github-project

not a copy & paste fail from https://github.com/gr2m/github-project


EKF: Improve accuracy during sustained yaw spinning

Sustained yaw spinning during hover can result in a gradual divergence of the Z accel bias estimate and increase in vertical position and velocity innovations. I some cases this can result in an estimation error that prevents the height controller completing a landing.
This is caused by the covariance prediction slightly lagging the vehicle motion due to the 12msec prediction time step (the output observer runs at the IMU rate of 4msec).
Here are the vertical innovations for replay of a log that demonstrates the behaviour

Reducing the EKF internal prediction time step from 12 to 8 msec eliminated the problem.

NOTE
Requires an increase in RAM allocation of 837 Bytes to allow for the longer IMU and output predictor buffers that can be created.
Processor utilisation on a pixracer board increased by 1.5 %

Thanks! We've saved RAM and CPU recently, so we can afford taking the hit.

Flight test log here: https://logs.px4.io/plot_app?log=00700bd3-a40e-4a47-aa5a-d4cb733d0409
Note: RAM allocation in ekf2_main.cpp task_spawn() was increased for this test.


Feature: option to shorten directory name to top level

as the title says, would be nice to have an option to shorten the directory section for the prompt in flexprompt configure such that it only shows the top most directory, IE c:\tools\cmder\ becomes cmder on its own

Have you found the documentation in flexprompt_modules.lua about the available options?
How does what you want differ from the type=folder or type=smart options?
For options available for each module, refer to the comments for each module in the flexprompt_modules.lua file.
-- CWD MODULE:  {cwd:color=color_name,alt_color_name:rootcolor=rootcolor_name:type=type_name:shorten}
--  - color_name is a name like "green", or an sgr code like "38;5;60".
--  - alt_color_name is optional; it is the text color in rainbow style.
--  - rootcolor_name overrides the repo parent color when using "rootsmart".
--  - type_name is the format to use:
--      - "full" is the full path.
--      - "folder" is just the folder name.
--      - "smart" is the git repo\subdir, or the full path.
--      - "rootsmart" is the full path, with parent of git repo not colored.
--
-- The 'shorten' option can abbreviate parent directories to the shortest string
-- that uniquely identifies the directory.  The first and last directories in
-- the string are never abbreviated, and git repo or workspace directories are
-- never abbreviated.  Abbreviation occurs when the cwd string is longer than 80
-- columns (2 line style) or longer than half the terminal width (1 line style),
-- or when the terminal width is not wide enough for the left and right prompts
-- to fit on a single line.
-- The 'shorten' option may optionally be followed by "=rootsmart" to abbreviate
-- only the repo's parent directories when in a git repo (otherwise abbreviate
-- all the parent directories), or by "=32" or other number to only use
-- abbreviation when the path is longer than the specified number of columns.
--
-- The default type is "rootsmart" if not specified.

-- Returns the folder name of the specified directory.
--  - For c:\foo\bar it yields bar
--  - For c:\ it yields c:\
--  - For \\server\share\subdir it yields subdir
--  - For \\server\share it yields \\server\share

I'm changing the title to clarify the question.
The "top level" directory in "c:\tools\cmder" would be "c:".
The "leafmost" directory would be "cmder".

Multiple already options exist for this.


terminus sites aliases should also include organizations' sites

I could be imagining this, but I think a previous version of terminus did include aliases of organization sites.  Regardless it would make things hugely easier for people that need to use remote drush commands against a wide variety of environments.

I don't know whether I've seen it return organizational sites before, but this is definitely something that is inconsistent and we will look into how we can solve it.

I'd like to see this addressed as well so I can interface with the org owned sites with drush aliases.

Closing in favor of #779

@peterfeddo @dalin- We've encountered some issues with the efficiency of providing this functionality on a system level which we are working to have resolved, but in the mean time I've constructed a script using Terminus as a library which creates the aliases for you.
https://github.com/pantheon-systems/cli/pull/808
Assuming you have pantheon-systems/cli installed, you can use it like so:
<?php

require (__DIR__ . '/PantheonAliases.php');

$aliaser = new PantheonAliases(
  [
    'username' => 'username',
    'password' => 'password',
    //'location' => '/this/is/optional',
  ]
);

I hope this helps!


submit multiple data Array or Object with Inertiajs

How can I submit arrays or object data using Vuejs
I tried that:
Inertia.post('/',{
    data:{'first':this.data,'second':this.order}
})

is there any other way?

Inertia.post('/settings', {
    favorite_colors: ['green', 'red', 'blue']
})

What @ordago is correct. With Inertia you can send arrays and objects just like you would send strings or integers. 👍


Decouple scrollable's cacheExtent and implicit accessibility scrolling

We should let people toggle cacheExtent for the purposes of pre-fetching and caching etc. But it's currently tied do implicit accessibility scrolling.
Sounds like that's already a flag on SemanticsFlag so it should theoretically be composed with list views to prevent implicit scrolling (such as in case of PageViews where it should never implicitly scroll regardless of cacheExtent) https://docs.flutter.io/flutter/dart-ui/SemanticsFlag/hasImplicitScrolling-constant.html.
But the flag is not currently used in the iOS accessibility bridge. Implement something similar to https://github.com/flutter/engine/pull/5941. i.e. reads to kIsHidden should also be gated behind hasImplicitScrolling.
cc @goderbauer

But it's currently tied do implicit accessibility scrolling.

This is only true for iOS. Android uses a different mechanism to deal with implicit scrolling.


Regular and compact outline buttons with dropdown do not resize according to the button text (Figma).

Describe the bug
Both regular and compact outline buttons with dropdown do not have auto layout, so they do not resize according to the button text.
How to reproduce
It's a Figma component error.
Steps to reproduce the behavior:
Just create an instance of the outline button component with a dropdown (compact or regular) and try to write a text whose width is longer than the component's width.
Expected behavior
The button should resize according to the text in it.
Versions
Clarity project:

[x] Clarity Core
[ ] Clarity Angular/UI

Clarity version:

[ ] v3.x
[ ] v4.x
[x] v5.x

Framework:

[ ] Angular
[ ] React
[ ] Vue
[ ] Other:

Framework version:
ie: Angular 11
Device:

Type: [e.g. MacBook]
OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional notes
Add any other notes about the problem here.

@colinreedmiller


How do I access a calendar without needing authorization?

I am trying to access a calendar that does not require authorization (login). I am trying to use ajax as in the second answer in this StackOverflow question:
<script> 	$.ajax({ 		url:"https://www.googleapis.com/", 		success: console.log("Yes") 	}); 	</script>
However, when I use any google api paths as it was shown in the post (hidden here for security), I am getting a 404. What is the right format to use to access my calendar?

Thanks for reaching out. This bug tracker is intended to report bugs in the Google API JavaScript Client. It looks like this is a question about using the Calendar API.
For support with Calendar API issues, please see https://developers.google.com/calendar; if you still need help, ask a question on Stack Overflow under the google-calendar tag. It will help to include a complete example to reproduce the issue as well. Best of luck!


SDF traversal trait

Need to port the sphere tracing implementations from bevy-test and unify them under an SDF traversal trait that defers to SignedDistanceField::distance.

Done.


mrcnn_class_loss function/dataset problem

I tried to train a simple model with your code, but unfortunately I get the following error when I try to train:
InvalidArgumentError (see above for traceback): assertion failed: [] [Condition x == y did not hold element-wise:] [x (mrcnn_class_loss/SparseSoftmaxCrossEntropyWithLogits/Shape_1:0) = ] [4 16] [y (mrcnn_class_loss/SparseSoftmaxCrossEntropyWithLogits/strided_slice:0) = ] [1 64]
	 [[Node: mrcnn_class_loss/SparseSoftmaxCrossEntropyWithLogits/assert_equal/Assert/Assert = Assert[T=[DT_STRING, DT_STRING, DT_STRING, DT_INT32, DT_STRING, DT_INT32], summarize=3, _device="/job:localhost/replica:0/task:0/device:CPU:0"](mrcnn_class_loss/SparseSoftmaxCrossEntropyWithLogits/assert_equal/All/_4229, mrcnn_class_loss/SparseSoftmaxCrossEntropyWithLogits/assert_equal/Assert/Assert/data_0, mrcnn_class_loss/SparseSoftmaxCrossEntropyWithLogits/assert_equal/Assert/Assert/data_1, mrcnn_class_loss/SparseSoftmaxCrossEntropyWithLogits/assert_equal/Assert/Assert/data_2, mrcnn_class_loss/SparseSoftmaxCrossEntropyWithLogits/Shape/_4231, mrcnn_class_loss/SparseSoftmaxCrossEntropyWithLogits/assert_equal/Assert/Assert/data_4, mrcnn_class_loss/SparseSoftmaxCrossEntropyWithLogits/strided_slice/_4233)]]

As I see the problem is with the tensor shapes, so it can happen that there is a problem with my dataset.

When I call dataset.load_image(...) it returns an image with shape (128, 128, 3)
When I call dataset.load_bbox(...) it returns a list of bounding boxes with shape: (nb_of_bboxes, 4) and a list of class ids to the corresponding bboxes

I hope you encountered with the same problem and can help me with this.

I think the problem is with the VotePooling.
Instead of pooled = tf.expand_dims(pooled, 0) the right output is pooled = tf.reshape(pooled, (self.batch_size, self.num_rois, self.channel_num)) as your comment says that the returned tensor should have a shape of (batch, num_rois, class_num)

I only tested the model under batch_size=1
It seems that the model goes wrong when bs>1
Bug labeled.


support jxcore command line

I came across to a problem with node 0.10.38 and doppio.
node(45688,0x7fff78557300) malloc: *** error for object 0x3dc503571b70: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug

I saw the discussion from https://github.com/joyent/node/issues/9273 but Node 0.12 is not an option for me due to other deps. and I don't want to deploy two separate versions of node for a basic demo. I don't even know if it's doable this way. However I gave a shot with jxcore master on github and it went fine with v8 build. At the beginning I received another issue  then I've cleaned up everything and install the tools and deps using jx install. It doesn't crash anymore.
I suspect if somehow npm compiles native addons using node 0.12 references because I had it installed and replaced back to 0.10.
To use with jxcore I've also needed to update doppio from
node $(dirname $0)/build/release-cli/console/runner.js "$@"

to
jx $(dirname $0)/build/release-cli/console/runner.js "$@"

Now I'm able to run the my jar file using doppio on top of node 0.10.38 framework (but using jx binary). I'm no c++ or java expert and don't know much about the solution on the core but I would really like to have doppio working with node 0.10.x / jxcore without patching or forcing to upgrade node 0.12.
Anyway doppio is great and keep up the good work!

Thanks for the compliments. I didn't think the upgrade would affect many people, since Doppio is intended to be used primarily in the browser -- after all, most people can simply run their favorite JVM natively on the command line if needed. :) The Node interface is used primarily for development purposes.
As indicated in the node bug, the segfaults are from a bug in Node v0.10's implementation of JavaScript strings. Conveniently, both JavaScript and the JVM use UTF-16 for strings, so Doppio goes ahead and converts string constants from JVM class files into JavaScript strings. I fixed a showstopping bug in this conversion process that prevented a core JVM class in JDK8 from functioning properly -- with rippling impacts that prevented things line parseInt from working!! (tl;dr: JDK8 uses a string to encode binary character set data as a string for some kooky reason; this data is used to convert characters to integers. Since the string is binary data in string form, it exercises a number of "fun" UTF-16 corner cases. Once fixed, the the segfaults began.)
Unfortunately, the segfault happens because Node has a bug in its string implementation. I lack the domain knowledge to fix or isolate the bug for them.
Now, Doppio does not need to eagerly convert all JVM string constants into JavaScript strings. JVM strings, including the weird character data string, are still represented as an array of bytes (actually, as an ArrayBuffer). However, we do need things like method and class names as JavaScript strings, as we use them in our bytecodes. There's no "flag" on these entries to differentiate them from string constants that are consumed primarily by JVM code, and changing string constant "resolution" to support two different use cases would complicate JVM logic in a number of places.
I may revisit this at another time, but I hope you understand that I have larger priorities at the moment.

If you've already gotten things working with jxcore (which it sounds like you have?), we'd be happy to keep a branch around with your changes. I agree that the node v0.12 requirement is a pain at moment.

I suspect jxcore uses a different version of V8, and thus works because it's not buggy. The Node crash results from a buggy backported string fix from V8 trunk to Node v0.10's old version of V8.

And yes, if jxcore support isn't too cumbersome, I'd be fine with supporting it -- it'd also make testing on SpiderMonkey from the command line feasible.

@perimosocordiae I didn't do anything special, just updated the command-line to jx.
@jvilk Surely I can use 'java' command line instead but after troubling a lot with node 0.10 I'd started to use doppio command line to reproduce the problem. It saved a lots of time.
Seems like there is not much change needed. I don't know how you could put 'jx' instead 'node' to the command line or if it's ever needed. Maybe you could simply add a note to somewhere that if anyone wishes to use it with node 0.10 and having a trouble with it. They can simply use it with jxcore or node 0.12.
Thanks


[All Hosts] (ui) more fixes for 'UI-less' terminology

A few further changes that I missed in the earlier PR today.

Docs Build status updates of commit db5a4e1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/dialog-api-in-office-add-ins.md
:white_check_mark:Succeeded
View

docs/develop/dialog-best-practices.md
:white_check_mark:Succeeded
View

docs/develop/json-manifest-overview.md
:white_check_mark:Succeeded
View

docs/includes/enable-debugging-on-edge-devtools.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


A0-1883: Create a workflow that builds and pushes docker images for ink compiler

Can we add
concurrency:
  group: ${{ github.ref }}-${{ github.workflow }}
  cancel-in-progress: true

b/c I can already see it's running multiple jobs right now.


[discussion] how to redesign Permission

We want to redesign the permission about DS, if you have some ideas ,please leave message.

#1321

After tested version 1.1, the following is my advice.

Tenant
In my opinion, tenants need to be empowered. Some administrator tenants(such as HDFS and Hive) are at risk if they are exposed to ordinary users.
Suggest  change "Tenant Name" could not non-repeatable

Resources
(1) I personally think that there is no need to involve the tenant.
I don't agree with the design that the user tenant will copy the files to the directory of the new tenant, which will cause file redundancy(From the results of testing on HDFS, other file centers have not tested).

(2) Enter "Resources-> FIle Manage"
Suggest change "Name and FileName" to "FileName, MD5Name (FileName+owner+createtime to unique identifier), Owner, LastModifyUser".
The FileName of the same user cannot be duplicated, and the HDFS storage file name is MD5Name. The files are placed in one directory.
File permissions can be designed to make the files visible to team members in the project or to themselves

Project
The project created by the administrator, the administrator has the highest authority of the project.
Ordinary users can only in the project development for workflow

Process definitions
The user of the workflow is recommended to bind to the project ID, not the user.
If the user is deleted, the workflow invalidation is an unreasonable design.

Suggestions for improvements to permission control from others were redirected to this issue, but no progress has been seen in four years. Currently, permissions for all functions need to be explicitly granted to everyone, which is very complicated for a multi-person collaboration platform. I recommend introducing RBAC to solve the existing problem.


unify build identifier of project inspectors

Currently we have several identifiers in a project inspector (and its subclasses), due to their different natures (Jenkins, Travis, Github, etc).
It would be introduce a uniform access method to an identifier string such as projectInspector.getIdentifier() that would be overwritten in the subclasses.

This would allow getting rid of instanceof in the patch classification function introduced in #1208


Stuck at boot on "Network time syncronization"

Describe the bug
Yesterday after rebuilding on unstable and attempting to reboot I got stuck at the boot screen on the message "Network Time Synchronization"
Steps To Reproduce
Steps to reproduce the behavior:

Rebuild on unstable
Reboot

Additional context
My gpu is gtx 1080 if that plays any importance.
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
Unable to run this on boot so I booted into 24.05 and run it there
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"

system: "x86_64-linux"
host os: Linux 6.6.41, NixOS, 24.05 (Uakari), 24.05.20240724.d0907b7
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.18.5
nixpkgs: /nix/store/svas0rbjm1vfiqhls8rmzcygwiwvcj32-source

Just noticed I posted this question twice on accident #338069


Git: Use Preview for Working Tree Diffs

VSCode Version: 1.12.2
OS Version: Windows 10 and elementary OS 0.4

Steps to Reproduce:

Add git repo as working folder
Change file in repo
Open Git tab, single (not double) click on file under "CHANGES"
The diff should open with the tab tile: "filename (Working Tree)", do not modify any contents or double click on tab
Open another file
Verify tab in step 4 is still open

I will do this for each of the files I have changed and thinking of committing to review my changes before I stage them. I believe the behavior is the same for staged files. This adds many tabs that I really don't need open when I go from one diff to another. I would like the tab to be closed/replaced by a preview. I would think the diff tab would be like any other file tab; therefore if I opened a diff, then opened a file or another diff by single clicking, the diff would be replaced by the new file's preview.

#26184