Problem in the Build process of the package

Hi community,
I'm a newbie of ROS and i'm working on a project which involves a robot ABB Yumi  (IRB 14000) and i'm using this library: library link. In order to control the camera i have seen that is needed the RWS library and i found this repository associated to that. So i cloned the in the src folder this package and i tried catkin build and catkin_make_isolated and i had errors like the one reported. How can i solve?
I'm using ROS Kinetic Kame with Ubuntu 16.04 Xenial.
kappa95@kappa95-VivoBook-ASUS-Laptop-X505ZA-X505ZA:~/yumi_depends_ws$ catkin build
------------------------------------------------------------------
Profile:                     default
Extending:          [cached] /opt/ros/kinetic
Workspace:                   /home/kappa95/yumi_depends_ws
------------------------------------------------------------------
Build Space:        [exists] /home/kappa95/yumi_depends_ws/build
Devel Space:        [exists] /home/kappa95/yumi_depends_ws/devel
Install Space:      [unused] /home/kappa95/yumi_depends_ws/install
Log Space:          [exists] /home/kappa95/yumi_depends_ws/logs
Source Space:       [exists] /home/kappa95/yumi_depends_ws/src
DESTDIR:            [unused] None
------------------------------------------------------------------
Devel Space Layout:          linked
Install Space Layout:        None
------------------------------------------------------------------
Additional CMake Args:       None
Additional Make Args:        None
Additional catkin Make Args: None
Internal Make Job Server:    True
Cache Job Environments:      False
------------------------------------------------------------------
Whitelisted Packages:        None
Blacklisted Packages:        None
------------------------------------------------------------------
Workspace configuration appears valid.
------------------------------------------------------------------
[build] Found '27' packages in 0.0 seconds.
[build] Updating package table.
Starting  >>> abb_irb2400_moveit_plugins
Starting  >>> abb_irb2400_support
Starting  >>> abb_irb5400_support
Starting  >>> abb_irb6600_support
Starting  >>> abb_irb6640_support
Starting  >>> abb_librws
Starting  >>> gazebo_mimic
Starting  >>> industrial_deprecated
Finished  <<< abb_irb6600_support                          [ 0.2 seconds ]
Starting  >>> industrial_msgs
Finished  <<< abb_irb6640_support                          [ 0.2 seconds ]
Starting  >>> industrial_trajectory_filters
Finished  <<< abb_irb2400_moveit_plugins                   [ 0.3 seconds ]
Starting  >>> industrial_utils
Finished  <<< gazebo_mimic                                 [ 0.2 seconds ]
Starting  >>> yumi_cameras
Finished  <<< abb_irb2400_support                          [ 0.2 seconds ]
Starting  >>> yumi_description
Finished  <<< industrial_deprecated                        [ 0.1 seconds ]
Starting  >>> yumi_moveit_demos
Finished  <<< abb_irb5400_support                          [ 0.2 seconds ]
Starting  >>> yumi_support
Finished  <<< industrial_trajectory_filters                [ 0.2 seconds ]
Finished  <<< industrial_utils                             [ 0.2 seconds ]
Finished  <<< yumi_cameras                                 [ 0.2 seconds ]
Finished  <<< yumi_description                             [ 0.2 seconds ]
Starting  >>> yumi_test_controllers
Starting  >>> yumi_control
Starting  >>> yumi_moveit_config
Finished  <<< yumi_moveit_demos                            [ 0.2 seconds ]
Finished  <<< industrial_msgs                              [ 0.7 seconds ]
Starting  >>> simple_message
Finished  <<< yumi_support                                 [ 0.2 seconds ]
Finished  <<< yumi_control                                 [ 0.2 seconds ]
Finished  <<< yumi_test_controllers                        [ 0.2 seconds ]
Finished  <<< yumi_moveit_config                           [ 0.2 seconds ]
_______________________________________________________________________________
Errors     << abb_librws:make /home/kappa95/yumi_depends_ws/logs/abb_librws/build.make.000.log
In file included from /home/kappa95/yumi_depends_ws/src/abb_librws/include/abb_librws/rws_client.h:48:0,
                 from /home/kappa95/yumi_depends_ws/src/abb_librws/src/rws_client.cpp:41:
/home/kappa95/yumi_depends_ws/src/abb_librws/include/abb_librws/rws_poco_client.h:42:38: fatal error: Poco/Net/HTTPCredentials.h: File o directory non esistente
In file included from /home/kappa95/yumi_depends_ws/src/abb_librws/src/rws_poco_client.cpp:43:0:
/home/kappa95/yumi_depends_ws/src/abb_librws/include/abb_librws/rws_poco_client.h:42:38: fatal error: Poco/Net/HTTPCredentials.h: File o directory non esistente
In file included from /home/kappa95/yumi_depends_ws/src/abb_librws/include/abb_librws/rws_client.h:48:0,
                 from /home/kappa95/yumi_depends_ws/src/abb_librws/include/abb_librws/rws_interface.h:40,
                 from /home/kappa95/yumi_depends_ws/src/abb_librws/src/rws_interface.cpp:40:
/home/kappa95/yumi_depends_ws/src/abb_librws/include/abb_librws/rws_poco_client.h:42:38: fatal error: Poco/Net/HTTPCredentials.h: File o directory non esistente
compilation terminated.
compilation terminated.
compilation terminated.
make[2]: *** [CMakeFiles/abb_librws.dir/src/rws_interface.cpp.o] Errore 1
make[2]: *** Attesa per i processi non terminati....
make[2]: *** [CMakeFiles/abb_librws.dir/src/rws_client.cpp.o] Errore 1
make[2]: *** [CMakeFiles/abb_librws.dir/src/rws_poco_client.cpp.o] Errore 1
make[1]: *** [CMakeFiles/abb_librws.dir/all] Errore 2
make: *** [all] Errore 2
cd /home/kappa95/yumi_depends_ws/build/abb_librws; catkin build --get-env abb_librws | catkin env -si  /usr/bin/make --jobserver-fds=6,7 -j; cd -
...............................................................................
Failed     << abb_librws:make                              [ Exited with code 2 ]
Failed    <<< abb_librws                                   [ 1.7 seconds ]
Abandoned <<< abb_irb2400_moveit_config                    [ Unrelated job failed ]
Abandoned <<< abb_irb6640_moveit_config                    [ Unrelated job failed ]
Abandoned <<< industrial_robot_client                      [ Unrelated job failed ]
Abandoned <<< industrial_robot_simulator                   [ Unrelated job failed ]
Abandoned <<< yumi_hw                                      [ Unrelated job failed ]
Abandoned <<< yumi_launch                                  [ Unrelated job failed ]
Finished  <<< simple_message                               [ 0.3 seconds ]
[build] Summary: 18 of 25 packages succeeded.
[build]   Ignored:   2 packages were skipped or are blacklisted.
[build]   Warnings:  None.
[build]   Abandoned: 6 packages were abandoned.
[build]   Failed:    1 packages failed.
[build] Runtime: 2.1 seconds total.
[build] Note: Workspace packages have changed, please re-source setup files to use them.

This looks like a duplicate of #110.


Delete old .html files

Old rendered files are hanging around and are available on the live site, e.g. https://jhudatascience.org/AnVIL_Book_Getting_Started/student-account-setup.html
I submitted an issue to the template repo jhudsl/DaSL_Course_Template_Bookdown#318 to solve the underlying problem, but we may want to manually clear out some old files in the meantime.

I had noticed this too.. Could definitely be a problem if folks are going to old links. Thanks for making the issue!

Resolved by updates to the template github actions


Delete autogenerated kallax.go if generation failed

@erizocosmico Done the second part of your suggestion; comment added to the first one


CJS -> ESM migration

Closing in favor of this: https://github.com/hypermod-io/hypermod-community/pull/241


Feature request: Pydantic ^2 support

FastAPI is on Pydantic 2 since 0.100. I'd like to use ariadne-codegen with the newest FastAPI, hence the request :)

This is already done and we plan to ship it sometime next week :)


The tools is not valid

Hi I have Arcmap 10.8.1
And just download the tool for arcmap 10.8
But when i test run the tool its say tool is not vaild.
here is the error>

You have 2 options here.
Option 1: Set your geoprocessing environment to execute the gp tools as a foreground process. For more detailed information, please read this doc https://desktop.arcgis.com/en/arcmap/latest/analyze/executing-tools/foreground-and-background-processing.htm
Option 2: Install the "ArcGIS Editor for OpenStreetMap 10.8 (64-bit)" version from the downloaded zip file.

You have 2 options here.
Option 1: Set your geoprocessing environment to execute the gp tools as a foreground process. For more detailed information, please read this doc https://desktop.arcgis.com/en/arcmap/latest/analyze/executing-tools/foreground-and-background-processing.htm
Option 2: Install the "ArcGIS Editor for OpenStreetMap 10.8 (64-bit)" version from the downloaded zip file.


Plugins List  Generate URL based on getCurrentPanel()->getId() can get wrong when using not ID as slug when tenancy enabled

in resources/views/pages/table.blade.php
the route: filament()->getCurrentPanel()->getId() can get wrong when you use another key as the routekey when tenancy is enabled. the extra parameter "tenant" should be added to the route() helper
original:
<x-filament::icon-button :tooltip="trans('filament-plugins::messages.plugins.actions.generate')" tag="a" href="{{route('filament.'.filament()->getCurrentPanel()->getId().'.resources.tables.index', ['module'=>$item->module_name])}}">

modified:
@if ((bool) filament()->hasTenancy())
    @php
       $urlParams = [
          'tenant' => filament()->getTenant()->{filament()->getTenant()->getRouteKeyName()},
          'module' => $item->module_name,
      ];
   @endphp
@else
    @php
        $urlParams = [
            'module' => $item->module_name,
        ];
    @endphp
@endif
<x-filament::icon-button :tooltip="trans('filament-plugins::messages.plugins.actions.generate')" tag="a" href="{{ route('filament.' . filament()->getCurrentPanel()->getId() . '.resources.tables.index', $urlParams) }}">

hi, @eelco2k thanks for your report, I will try to fix it ASAP, thanks for your support.

No problem! ✌🏽
Regarding the issue: I still find it strange that it does not fill the correct tenant param itself correctly via the route() helper function. It should, as I can remember i had this issue before where I needed to add the “tenant” param explicitly. But after configuring somewhere (have to check where that is) that the default routeParam for the tenant would be different. I could remove the “tenant” param for all the routes() as extra param.  So it is weird that in your package it does not work.
Thank you for following up on this issue

I think it was related to Route Model Binding
but as Model/Table does not use this via getRouteKeyName() function it uses the default (id) from the inherited Model getRouteKeyName() function.
Perhaps you could move the logic to Table model into that getRouteKeyName() function where it checks if tenancy is enabled and. If so return
filament()->getTenant()->{filament()->getTenant()->getRouteKeyName()};

i've changed the code above a little bit, as the param 'tenant' also accepts filament->getTentant()  (so it uses the Model associated with the Tenancy. and will use and also fallback to the correct routekeyname )

hi @eelco2k i think the last one maybe the best solution and I will try to apply it, thanks for your support.

fixed on the last version.


Duplicates using upsert+PK

I'm using Debezium and Cassandra in CDC mode with snapshots enabled. I see that when Cassandra makes a snapshot it also sends regular inserts. In Kafka and in the final Iceberg table they end up as dups and I assume this is expected since it actually receives duplicate inserts? Or is there a recommended way of handling this? The only solution I can think of now is to process snapshots and non-snapshots in different pipelines..

You may find this useful: https://tabular.io/webinar-everything-cdc/
typically most people take a two-table approach: the raw events (which can contain multiple rows per PK because of CDC updates, etc) and then run a secondary process that does some sort of MERGE so there is only the latest value for PK.

Thank you, the link was very useful! I guess that the upsert functionality is not needed then.

In my experience upsert only works for small tables and requires aggressive behind the scenes compaction, which can interface with other operations on the table.  For a sizable table, the query performance was always poor.  Which is why most folks go the two-table route.


Port Bondage from ZapJS to Python

Summary
Port Bondage from ZapJS to Python
Reference
https://github.com/zapproject/zap-monorepo/blob/master/packages/Bondage/src/index.ts
Definition of Done
All the functions should be able to be called and return the proper values.

George is tasked with this issue

Going to be picking up on this.

Please refer to pull request #45 for review of this port.


Clarify securing traffic capabilities with Container to Container Networking

Hi,
This PR just adds some documentation on what Container to Container networking supports with regards to securing traffic.

Thank you @christianang


Incorrect link for submodules that specify the remote port

Gitea version (or commit ref):
d4c9399 and any older one that I've tested
Operating system: Ubuntu 16.04 (using the docker image)
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL)
https://try.gitea.io/BugReproduction871239/test213
[ ] No
[ ] Not relevant

Log gist:

Description
In the example URL, there is a submodule called test214 that uses the port (22) as user name in its link.
The submodule configuration is valid and works on the command line.

Hi,
Is this fix on the Docker image already? I've tested commit "da1edbf" here and it doesn't seems to solve the problem. Is there anything I should do to "update" the links?

This has been merged to latest Docker image I think.

The problem persists. The submodule link shows "http(s):////" instead of "http(s)://:/".
The workaround I'm using is to set a relative path on the submodule URL. Fortunately, the workaround is the correct way of doing things :+1:, but older revisions still have the wrong link.

Hi Governa, could you upload another testcase to try, so I can check this out?

@Governa is the link to the come_on submodule in https://try.gitea.io/arandomer/arepoopo an example of this?
https://try.gitea.io/22/arandomer/come_on which should be https://try.gitea.io/arandomer/come_on
generated from git submodule add ssh://git@try.gitea.io:22/arandomer/come_on.git
Now looking at:
https://github.com/go-gitea/gitea/blob/f6eedd4dc8fb10df869750c69e2bead0521ec0eb/modules/git/submodule.go#L32-L77
I'm not particularly fond of this code - Why aren't we using the default url parsing code built into go?
Strangely it seems like a similiar case is being tested for in:
https://github.com/go-gitea/gitea/blob/f6eedd4dc8fb10df869750c69e2bead0521ec0eb/modules/git/submodule_test.go#L23

I think git@github.com:a/b.git is not a standard URL.

yep it isn't just tested it.

The one case I had bookmarked is now working, so works for me :+1:

This has since been fixed and included in testcases


Use a JS wrapper for shell commands in package.json scripts

To run the debug scripts on Windows, which does not have commands named cp and rm, we need to use a JS wrapper.
SHX looks like a very good drop in wrapper to use and worked for me under Windows.
Can others test that the debug and other scripts continue to function as expected on other platforms such as Linux and MacOS?
Also update the Readme.md to be slightly more precise when referring to names of commands.

Thanks !


Fixed django>=1.11 support.

Since Django 1.11 the query argument of the SQLCompiler constructor has always a subquery property, in fact if the SQL query has no real sub query it is set to False.
As a result, the test done by Django-cachalot in the _get_tables function is no more accurate and cause a dramatic slow down on 1.11 django installs.
This fix should be backwards compatible with older versions of Django.

Coverage remained the same at 99.598% when pulling 179d8f2e21f8d8b6c23c2dba420cf92af5f58567 on izimobil:master into a302f6b77a19d84ddf382c5dd798adaaac60fb88 on BertrandBordage:master.

That’s a good step forward, thanks!
It’s not enough for 1.11 support however, there are other issues I’m currently fixing.

This one was hard to tackle... I'm currently using cachalot / django 1.11 for a project, just curious, what are the issues you are working on ? are they blocking ? Maybe I can help.

Mostly I had to investigates tests no longer passing. In most of the cases it was finally OK. For example, random subqueries like Test.objects.filter(pk__in=Test.objects.order_by('?')) were unexpectedly cached. That’s because Django now does an optimisation: in a subquery, if there is not limit or offset, the order is removed. So I had to add slicing to the subquery.
The other issues are minor:

PostgreSQL JSON field not working the same so queries are either not cached or crash
PostgreSQL & MySQL not caching queries passing a binary parameter

I already fix all these, so the latest build on Travis CI, build 438 will fully pass when it’s finished.
There’s also #70 I have to fix before releasing a new version, as well as answering other issues.

OK, that's good to know. By the way, I'm using cachalot with django-tenant-schemas (postgres schemas) and It works great (I just had to customize get_query_cache_key and get_table_cache_key to add the postgres schema name). Thanks for your great work !


Docs and Test Script don't exist

Just wanted to let you know that the links to the Documentation and Test Script on the Readme aren't currently working.
Thanks for the constant updates!

Fixed, thanks for reporting 👍🏻


Page Loading Bug

Describe the bug

To Reproduce

Expected behavior

Screenshots

Applicable Versions:

Netlify CMS version: [e.g. 2.0.4]
Git provider: [e.g. GitHub, BitBucket]
OS: [e.g. Windows 7]
Browser version [e.g. chrome 22, safari 11]

Node.JS version:

CMS configuration

Additional context

Could you provide more detail about what's happening, steps to reproduce etc so we can help you out?

No problem, I was navigating in between pages and kept getting an error
message. When I refreshed the page and logged in again the problem went
away. I was navigating around the "collections" pages when it happened.
On Tue, Jun 11, 2019 at 2:41 PM Tom Rutgers<EMAIL_ADDRESS>wrote:

Could you provide more detail about what's happening, steps to reproduce
etc so we can help you out?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/netlify/netlify-cms/issues/2373?email_source=notifications&email_token=AMKTZWEJDACL7MFHWJXCELLP2ALXPA5CNFSM4HXDRS32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXOSYAI#issuecomment-501033985,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMKTZWFNIAKSMGXFS775ZZLP2ALXPANCNFSM4HXDRS3Q
.

--
Kyle Fanthorpe
Content Strategist
<EMAIL_ADDRESS>| LinkedIn https://www.linkedin.com/in/kylefanthorpe/

Glad it straightened itself out - you can open a new issue with the error message text if it happens again.

Thanks so much!
On Thu, Jun 13, 2019 at 12:33 PM Shawn Erquhart<EMAIL_ADDRESS>wrote:

Glad it straightened itself out - you can open a new issue with the error
message text if it happens again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/netlify/netlify-cms/issues/2373?email_source=notifications&email_token=AMKTZWEYZZPWWD5ZRKYPCELP2KOJVA5CNFSM4HXDRS32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXUZRYY#issuecomment-501848291,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMKTZWCZK4M6BG6T3MBGSTDP2KOJVANCNFSM4HXDRS3Q
.

--
Kyle Fanthorpe
Content Strategist
<EMAIL_ADDRESS>| LinkedIn https://www.linkedin.com/in/kylefanthorpe/


Fix typo

Fix typo (tiny change, probably not need changelog?)

Before submitting the PR make sure the following are checked:

[ ] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[ ] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[ ] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[ ] Added tests.
[ ] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[ ] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Good catch! No changelog is needed here.

Thanks!


feat: open governance referendum via link

use layouts for governance refernedum lists and referendum model

Looks great 💪💪💪


[v1.5.x] fix license issues (#15806)

Description
Cherry pick #15806 to fix the code license issue on the v1.5.x branch.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the 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.
Interesting edge cases to note here

@roywei @szha

I haven't run RAT, but fixes look good to me Tao.


Verify builds with 0.14.x compiler

Things that were outputted via stderr have been switched to stdout. Thus, some tooling (e.g. purescript-psa) needed to be updated to handle this fact. Fortunately, this tool has been updated in v0.8.0 to handle both v0.13.x and v0.14.x PureScript compiler releases.

Should check what our build command parsing looks like and it's suitable. I recall it's pretty lax about "finding some json output somewhere" but don't recall if it looks in both stderr and stdout or not.

This is indeed a problem. It currently only looks in stderr. I just ran into it. I guess the best would be to find out the version of the compiler and then look in the appropriate channel based on that?

This is indeed a problem. It currently only looks in stderr. I just ran into it. I guess the best would be to find out the version of the compiler and then look in the appropriate channel based on that?

My plan here is just to look at both stderr and stdout opportunistically, prefer to avoid casing on version wherever possible - the json errors parsing already skips past junk looking for lines that start JSONy and tries to parse that. I think being permissive can avoid issues with other tools that might be in the way passing that output along (whether psa, spago, some makefile or whatever).

My plan here is just to look at both stderr and stdout opportunistically, prefer to avoid casing on version wherever possible - the json errors parsing already skips past junk looking for lines that start JSONy and tries to parse that. I think being permissive can avoid issues with other tools that might be in the way passing that output along (whether psa, spago, some makefile or whatever).

There were related changes made in psa in https://github.com/natefaubion/purescript-psa/pull/44 and https://github.com/natefaubion/purescript-psa/pull/49; I'm not sure if either of those will be helpful in language server support for PureScript 0.14.

Release is out with basic changes for the build output and a completion issue (the removal of Kind). Some tidying and improvement to do but hopefully nothing urgent


black mishandles functions that end in a comment

For the repro file too_many_returns.py, I have a pylint: disable / pylint: enable at the beginning and end of a function (but it is within the scope of the function):
"""``black`` issue repro."""

def squared(value):
    """Just squares a number."""
    # pylint: disable=too-many-return-statements
    if value == 1:
        return 1

    elif value == 2:
        return 4

    elif value == 3:
        return 9

    elif value == 4:
        return 16

    elif value == 5:
        return 25

    elif value == 6:
        return 36

    elif value == 7:
        return 49

    elif value == 8:
        return 64

    elif value == 9:
        return 81

    elif value == 10:
        return 100

    else:
        return value * value
    # pylint: enable=too-many-return-statements

def cubed(value):
    """Just cubes a number."""
    return value * squared(value)

Running black (installed from HEAD as in #30) does two "strange" things:

Moves the # pylint: enable comment to just before the next function
Adds 2 empty lines in between the two functions (so there is now 4, instead of 2)

$ venv-HEAD/bin/black too_many_returns.py
reformatted too_many_returns.py

See the diff
diff --git a/too_many_returns.py b/too_many_returns.py
index b7251d5..3b65b6c 100644
--- a/too_many_returns.py
+++ b/too_many_returns.py
@@ -36,9 +36,11 @@ def squared(value):

     else:
         return value * value
-    # pylint: enable=too-many-return-statements

+
+
+# pylint: enable=too-many-return-statements
 def cubed(value):
     """Just cubes a number."""
     return value * squared(value)

I realize the pylint: enable... at the end isn't needed, since the disable is scoped to the function, but I figured you'd still want to see this surprising reformatted result.

This is the same issue as #16. Funnily they're consecutive powers of two.

@ambv Happy to close this, though I think the added newlines makes this slightly different than #16?

Thanks a lot @ambv!


Fix conj and scalar multiplication of EmptyStorage

Codecov Report

Merging #756 (7d1b8a1) into main (ae3b49d) will decrease coverage by 44.16%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main     #756       +/-   ##
===========================================
- Coverage   76.35%   32.18%   -44.17%
===========================================
  Files          84       83        -1
  Lines        9951     9827      -124
===========================================
- Hits         7598     3163     -4435
- Misses       2353     6664     +4311

Impacted Files
Coverage Δ

src/NDTensors/empty.jl
21.01% <0.00%> (-47.34%)
:arrow_down:

src/NDTensors/octavian.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/site_types/boson.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/site_types/fermion.jl
0.00% <0.00%> (-98.39%)
:arrow_down:

src/physics/site_types/qudit.jl
0.00% <0.00%> (-97.30%)
:arrow_down:

...c/ContractionSequenceOptimization/three_tensors.jl
0.00% <0.00%> (-96.16%)
:arrow_down:

src/mps/observer.jl
7.14% <0.00%> (-92.86%)
:arrow_down:

src/iterativesolvers.jl
0.00% <0.00%> (-89.62%)
:arrow_down:

src/mps/projmps.jl
0.00% <0.00%> (-88.38%)
:arrow_down:

src/mps/diskprojmpo.jl
0.00% <0.00%> (-83.34%)
:arrow_down:

... and 63 more

Continue to review full report at Codecov.

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


A PHP Error was encountered

admin/orders  and in order table when i click more info following error is occuered
Products
A PHP Error was encountered
Severity: Warning
Message: A non-numeric value encounteredFilename: ecommerce/orders.php
Line Number: 146
Backtrace
:File: C:\xampp\htdocs\shop\application\modules\admin\views\ecommerce\orders.php
Line: 146
Function: _error_handler
File: C:\xampp\htdocs\shop\application\third_party\MX\Loader.php
Line: 355
Function: include
File: C:\xampp\htdocs\shop\application\third_party\MX\Loader.php
Line: 294
Function: _ci_load
File: C:\xampp\htdocs\shop\application\modules\admin\controllers\ecommerce\Orders.php
Line: 73
Function: view
File: C:\xampp\htdocs\shop\index.php
Line: 292
Function: require_once
http://localhost/shop/
Quantity:
A PHP Error was encountered
Severity: Notice
Message: Array to string conversion
Filename: ecommerce/orders.php
Line Number: 156Backtrace
:File: C:\xampp\htdocs\shop\application\modules\admin\views\ecommerce\orders.php
Line: 156
Function: _error_handler
File: C:\xampp\htdocs\shop\application\third_party\MX\Loader.php
Line: 355
Function: include
File: C:\xampp\htdocs\shop\application\third_party\MX\Loader.php
Line: 294Function: _ci_loadFile: C:\xampp\htdocs\shop\application\modules\admin\controllers\ecommerce\Orders.php
Line: 73
Function: view
File: C:\xampp\htdocs\shop\index.php
Line: 292
Function: require_once
Array / Price: ₹
Vendor:Total amount of products: 0 ₹
in product details how to avoid this error and get product details

@aswinsasi do you still have this problem?


Localhost redirect_uri validation bug

Describe the bug
Hi spring-authorization-server gurus,
The localhost validation erroneously triggers in the face of the suggested host value <IP_ADDRESS>, ex:

localhost is not allowed for the redirect_uri (http%3A%2F%2F<IP_ADDRESS>%3A4200%2Flogin). Use the IP literal (<IP_ADDRESS>) instead.

Unfortunately it took unreasonable time to pin down the root cause, as no errors was present in the logs even on TRACE-level and the symptom only manifested itself by caching the error-wrapped request instead of the expected request, ex:
2022-08-03 15:09:40.987 DEBUG 34689 --- [o-auto-1-exec-3] o.s.s.w.s.HttpSessionRequestCache        : Saved request http://localhost:44303/error?scope=openid&response_type=code&redirect_uri=https%3A%2F%2Flocalhost%3A4200%2Fdebug&state=.SNIP..

This behavior subsequently breaks the auth-flow, as the user is authenticated at this time but the cached request invalid.
Ie. the auth-flow worked as expected for  redirect_uri https://oidcdebugger.com/debug , resulting in quite the head scratching and gnashing of teeth :/
To Reproduce

Register a authorization_code client with a redirect_uri like http://<IP_ADDRESS>:4200/login
Perform the authorization_code flow

Notice how requestedRedirectHost is null (orange arrows), triggering the first condition on line 598:

Expected behavior
The suggested  redirect_uri host value <IP_ADDRESS> is accepted by the validation.
Plus another plea for logging of server-side (validation) errors (see also #159)
Sample
N/A ~ should be possible to prevent regression with unit-test ~ consider also covering a regular IP address.

Bah, was a case of RestTemplate implicitly double URL-encoding the location query-parameters :/
But my plea for debug logging still stands 🙏


Fixed: Irregular undefined calls

Just small changes from the file merge

I thought that this is pretty useful and it may be a must-have

I thought that this is pretty useful and it may be a must-have

Interesting; would it take a relative path as well ("a/b/c")?

Never tried it, but currently i have a project that contains multiple versions of itself in the same folder and I needed to include the complex library to be usable in all the versions separately ;)

Renamed this to common.addLibrary(sB, ...) and updated it with vararg in the last patch

API now adds directories including the Lua files directly
common.addLibrary("E:/Documents/Lua-Projs/ZeroBraineIDE/myprograms","a/b/c")
local t = require("test")
print(t.add(3,7)) -- Prints 10


Adding the consumption of DTK by KMM.

If the user has specified the DTK_AUTO build arg in his Dockerfile, KMMO will now find the correct DTK image for each kernel version and add that build ARG to the Dockerfile.
This commit is following the design in
docs/enhancements/using-dtk-for-building-driver-containers.md
Signed-off-by: Yoni Bettan<EMAIL_ADDRESS>

/cc @yevgeny-shnaidman
/hold
I want to add 2 more UT for the updated Reconcile funcions.
The code should be ready though.

FYI @enriquebelarte

/cc @enriquebelarte

/unhold

Still need to update the UT but the code is 95% done.
@yevgeny-shnaidman, @qbarrand PTAL.

/lgtm

This PR was tested manually using this module:
apiVersion: kmm.sigs.k8s.io/v1beta1
kind: Module
metadata:
  name: kmm-ci-build
spec:
  moduleLoader:
    container:
      modprobe:
        moduleName: kmm_ci_a
      kernelMappings:
        - literal: 4.18.0-372.26.1.el8_6.x86_64
          containerImage: quay.io/ybettan/kernel-module-management:non-existing
          pull:
            insecure: true
          build:
            pull:
              insecure: true
            push:
              insecure: true
            dockerfile: |
              ARG DTK_AUTO
              FROM ${DTK_AUTO}
  selector:
    kubernetes.io/hostname: test-infra-cluster-7363ce8a-master-0

@ybettan maybe you can push this test Module into some directory, like examples?


#108 added missing fields to items object

Hopefully I understood correctly what was needed.

Please add a comment in the issue saying you're willing to work on it, so that I can assign it to you and we can avoid duplicate efforts for someone else.

Please squash all your commits once you've made the changes. :)

It should be all good now :)

Thanks!


[WiP] Endpoints for account details

Closes #2010
Example response from `tokenCreate':
{
  "data": {
    "tokenCreate": {
      "token": "eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJlbWFpbCI6InF3ZUBleGFtcGxlLmNvbSIsImV4cCI6MTUyNDY2NjE4MSwib3JpZ19pYXQiOjE1MjQ2NjU4ODF9.sKTiFapywujgeao6hxp4a7gY9dBxV9DH2RBOFdlj00s",
      "user": {
        "id": "VXNlcjoyMg==",
        "email"<EMAIL_ADDRESS>        "permissions": [
          "product | product | Can view products"
        ]
      }
    }
  }
}

and tokenVerify:
{
  "data": {
    "tokenVerify": {
      "payload": {
        "email"<EMAIL_ADDRESS>        "exp":<PHONE_NUMBER>,
        "orig_iat":<PHONE_NUMBER>
      },
      "user": {
        "id": "VXNlcjoyMg==",
        "permissions": [
          "product | product | Can view products"
        ],
        "email"<EMAIL_ADDRESS>      }
    }
  }
}

TODOs:

Mutations for user
Tests

Pull Request Checklist
(Please keep this section. It will make maintainer's life easier.)

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[ ] JavaScript code quality checks pass: eslint.

"product | product | Can view products" is rather useless both in code and as a display label. I think we're after something like:
[
    {
        "code": "products.can_create_product",
        "name": "Can create products"
    }
]


Use slog for logging

What should be done?
We should switch our logging to log/slog package:

https://go.dev/blog/slog
https://pkg.go.dev/log/slog

Among other things, that would make the usage of the embedded FerretDB package easier.
We probably should not remove all zap logging in this issue, but it should be hidden from the public interface of the ferretdb package.
That's a breaking change for v2.
Where?
TBD
Definition of Done
TBD

handler updated;
all backends updated;
unit tests added/updated;
integration/compatibility tests added/updated;
spot refactorings done;
user documentation updated or an issue to create documentation created;
something else?

Would love to start contributing to this whenever this is marked as ready. I believe it'll be many small PRs


Fix test_phase_unphasePAPER in test_uvdata

Memory dump of my recollections looking into this:
The error is the zenith_ra's don't agree in the input miriad and output miriad after phasing and unphasing.  read_miriad gets zenith_ra from the lst time in the miriad file.  lst is written by the PAPER correlator, which uses aipy (which uses pyephem) to calculate lst from the JD timestamp.  Once the data is phased, zenith_ra is deleted and phase_center_ra is written.  Once the data is unphased, zenith_ra is re-written by pyuvdata which uses pyephem to calculate lst from the JD timestamp.  While the fact that both values are calculated from scratch both times makes it seem plausible they don't agree, but it's also somewhat surprising, since the lat/lon/alt agree in the input file and the pyuvdata known telescopes lists (I think) and both use pyephem to calculate lst.

Adding another data point that I think is related. We created a blank uvfits file using @aelanman's script (see issue #89). We then created two UVData objects, phased one to the first time in time_array, and phased the other to the first's phase_center_ra, phase_center_dec, phase_center_epoch.

The result is two objects that do not match - particularly the ra/dec do not agree. Note the test of equality also seems to be broken (see issue #93).

I'm spending a little time this morning trying to narrow down this problem. Here are some notes which may or may not be useful.
I recreated the UV_raw and UV_phase objects from the test in ipython to play with. First thing is to plot the difference in the zenith_ra arrays.

I also tried created a UV_phase2, which is the same as UV_phase, but phased to ra,dec=(0.1,0.1), then unphased. The result is identical to UV_phase, suggesting the intermediate phase location is irrelevant.
Next I thought I'd try to recreate the calculation done by the PAPER correlator. @dannyjacobs suggests this is futile because lsts from the correlator are not to be trusted. Nevertheless, I'll document what I've done here. I used Zaki's PAPERCORR repo to look for where lst is calcuated. I found an example of the script PAPERCORR/casper_correlator-1.2/tests/paper_test.py calling dacq.DataReceiver, which does the calculation. I attempted to copy it, using the lat, lon, and jd from our UV_raw object, and interestingly I get the same answer as UV_phase.zenith_ra. I think this confirms that the lst is simply wrong in data file we're using, and our code could actually be right.
I think the way to proceed is to replace the lst in our test file with an independently-calculated correct value. What do people think?

To summarize the last couple commits:
We moved on to a newer data file (from HERA), which did not exhibit the ramping behavior seen above. Instead there was a constant RA/DEC offset ~ 1e-6 rad. This turned out to be due to the RA/DEC in the file being calculated from the Lat/Lon in the file, which we were overwriting with known_telescopes. For the test, we added a new keyword to read_miriad which allows the user to keep the original antenna location, even if it does not match the values in known_telescopes. With this fix the test passes.


Load program.evc from the network, the screen cannot be displayed normally.

Load program.evc from the network, the screen cannot be displayed normally.

If you need to refresh the screen again after successfully reading the data, you can consider adding setState().

Fixed in v0.5.4


Combine ceph system and image datastores

Author Name: John Noss (John Noss)
Original Redmine Issue: 4877, https://dev.opennebula.org/issues/4877
Original Date: 2016-10-21

Currently using ceph as the backend requires two separate datastores, image and system; it would be nice if these could be combined so that root disks and volatile disks/checkpoints lived in the same datastore/pool (this would be another route to facilitate selection of the proper system datastore in the case of multiple system datastores, see feature req http://dev.opennebula.org/issues/4876)
Ceph system and image datastores could either be linked in the scheduler so that if an image is on a ceph datastore that the scheduler knows to select the system datastore with the same ceph pool, or, these could be merged so that there is only one datastore required if using ceph.
Note also this could be a configurable - either a oned/sched.conf config or a config variable on the ceph image datastore - USE_SYSTEM_DS = , or otherwise.

Original Redmine Comment
Author Name: Ruben S. Montero (@rsmontero)
Original Date: 2016-11-01T09:43:55Z

Hi
I believe this is already implemented with Ceph as a system datastore in 5.0.  Volatile disks are created in the pool designated in the system datastore.  We made several designs to store also the checkpoints in the pool.  Unfortunately libvirt cannot write/read checkpoints from a Ceph pool,  so we decided to avoid the checkpoint trashing.
Note that you can use the same pool for the system datastore (or not,  e.g. use a pool with less replication for volatile disks).  This requires to define both datastores,  but we rather keep this approach.
Cheers


Newliquidcrystal_1.3.5 and LcdMenu - compatibility issue

Hi Thomas
Your library seem exactly was I need, but I have a problem.
I changed to use the I2C library into the LiquidMenu_config.h but I have an error at the line " lcd.init();)" ->int LiquidCrystal_I2C::init()' is private within this context". I expect that the library "Newliquidcrystal_1.3.5" is not compatible with your code.
I tryed to cancel it, but nothing works.
May you tell me which library is to use with your code? Thank you in advance and best regards,
Stefano

I found a replacement library called "liquidcrystal_I2C" that works!

You can look at the library dependency for the exact library I used
platformio.ini


Compatibility changes for RN 0.25 and greater

React API must be now required from react package (previously a warning on 0.25)

@brettpappas
Thank you for your contribution!
It looks good.
I will merge this PR and release next version.
Thanks!


"Izvoarele Sucevei", "Bobeica" and "Brodina" villages are listed in the drop-down menu.

Description:
The villages are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions:
The website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Izvoarele Sucevei".

Expected results
The location is listed in the drop-down menu as "Sat Izvoarele Sucevei SUCEAVA (Izvoarele Sucevei)".

Step 1
Write in the search bar "Bobeica".

Expected results
The location is listed in the drop-down menu as "Sat Bobeica SUCEAVA (Izvoarele Sucevei)".

Step 3
Write in the search bar "Brodina".

Expected results
The location is listed in the drop-down menu as "Sat Brodina SUCEAVA (Izvoarele Sucevei)".


Secor not safe to null payloads

com.pinterest.secor.message.Message.fieldsToString not null safe.
Somehow we got a null payload in kafka, and secor throws a NullPointerException.
15/05/11 19:12:24 ERROR main.ConsumerMain: Thread Thread[Thread-4,5,main] failed
java.lang.NullPointerException
at java.lang.String.(String.java:554)
at com.pinterest.secor.message.Message.fieldsToString(Message.java:35)
at com.pinterest.secor.message.Message.toString(Message.java:44)
at java.lang.String.valueOf(String.java:2981)
at java.lang.StringBuilder.append(StringBuilder.java:131)
at com.pinterest.secor.reader.MessageReader.read(MessageReader.java:141)
at com.pinterest.secor.consumer.Consumer.run(Consumer.java:92)
I would fix this myself but I can no longer build secor due to twitter dependencies that are no longer available.
[ERROR] Failed to execute goal com.twitter:maven-finagle-thrift-plugin:0.0.9:compile (thrift-sources) on project secor: Execution thrift-sources of goal com.twitter:maven-finagle-thrift-plugin:0.0.9:compile failed: A required class was missing while executing com.twitter:maven-finagle-thrift-plugin:0.0.9:compile: com/google/common/collect/ImmutableSet
I tried lots of ways to find the dependencies but nothing would fix it completely.
I am a committer on this project and really would like to contribute some more, if someone could help with the dependency problems

i did a clone, blew away my .m2 cache and was able to rebuild successfully. Can you try the same to see if you are still having issues?

I moved my .m2 to .m2.save, cloned a new version of secor and did a build.
I am building with java version "1.7.0_60"
I get this error.
[ERROR] /home/lwoessner/workspace/secor/target/generated-sources/thrift/gen-java/com/pinterest/secor/thrift/TestMessage.java:[35,33] error: package com.twitter.finagle.thrift does not exist
do you have something in your settings.xml that would change things?
Leo

Leo,
Not sure what is going on with your environment?
My setup:
java -version
java version "1.7.0_67"
Java(TM) SE Runtime Environment (build 1.7.0_67-b01)
Java HotSpot(TM) 64-Bit Server VM (build 24.65-b04, mixed mode)
nothing in settings.xml
What is the line in the generated file TestMessage.java? Is maven still having trouble finding finagle-thrift jar?

I commented out that line of the generated TestMessage.java and did a mvn package and it is find.  Not sure how to get this line to be left out by default.

Just throwing my hat into the ring to say I'm having this trouble on the dev branch as well. I just updated my fork, and the behavior continues.
Either of you guys make progress the original issue with the Failed toString() invocation on an object of type [com.pinterest.secor.message.Message] error?

Am able to build on OSX 10.10.5, but cannot build on CentOS 7.1. Get the same errors as above about missing com.twitter.finagle.thrift

Ditto on this null payload problem.  I have changes in my tree to fix this, as I did not see that someone had already submitted a pull (112), but that pull seems 'bad'?  What's the status on fixing this?  Should I submit another pull request?

Richard, did you make the same change I did?
public Message(String topic, int kafkaPartition, long offset, byte[]
payload) {
mTopic = topic;
mKafkaPartition = kafkaPartition;
mOffset = offset;
mPayload = payload;
if (mPayload == null) {
mPayload = new byte[0];
}
}
I don;t know why that changed broke the integration tests.  But if you can
get it to work, go for it.  A friend at another company and I are both
running my fix.
Let me know how it goes.
Leo
On Wed, Aug 19, 2015 at 10:32 AM, Richard Lee<EMAIL_ADDRESS>wrote:

Ditto on this null payload problem. I have changes in my tree to fix this,
as I did not see that someone had already submitted a pull (112), but that
pull seems 'bad'? What's the status on fixing this? Should I submit another
pull request?
—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/92#issuecomment-132685977.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a https://www.uber.com/invite/ern6a

I made a similar change.  https://github.com/TiVo/secor/tree/handle-null-payloads https://github.com/TiVo/secor/tree/handle-null-payloads
However, I preserved the null value.  It’s important to know when the value is null to detect tombstones, but I suspect a zero length byte field is pretty close to null, and code could be written to deal with that, instead.  Also, your approach doesn’t require fixing the FileWriter.
Richard
Key ID: 3BDED925D4F316B3

On Aug 19, 2015, at 11:50 AM, estezz<EMAIL_ADDRESS>wrote:
Richard, did you make the same change I did?
public Message(String topic, int kafkaPartition, long offset, byte[]
payload) {
mTopic = topic;
mKafkaPartition = kafkaPartition;
mOffset = offset;
mPayload = payload;
if (mPayload == null) {
mPayload = new byte[0];
}
}
I don;t know why that changed broke the integration tests. But if you can
get it to work, go for it. A friend at another company and I are both
running my fix.
Let me know how it goes.
Leo
On Wed, Aug 19, 2015 at 10:32 AM, Richard Lee<EMAIL_ADDRESS>wrote:

Ditto on this null payload problem. I have changes in my tree to fix this,
as I did not see that someone had already submitted a pull (112), but that
pull seems 'bad'? What's the status on fixing this? Should I submit another
pull request?
—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/92#issuecomment-132685977.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a https://www.uber.com/invite/ern6a
—
Reply to this email directly or view it on GitHub https://github.com/pinterest/secor/issues/92#issuecomment-132740226.

Richard,
I think I like your solution better.  Can you make a pull request to
secor?  Then I will get your changes.
Thanks for working with me
Leo
On Wed, Aug 19, 2015 at 2:08 PM, Richard Lee<EMAIL_ADDRESS>wrote:

I made a similar change.
https://github.com/TiVo/secor/tree/handle-null-payloads <
https://github.com/TiVo/secor/tree/handle-null-payloads>
However, I preserved the null value. It’s important to know when the value
is null to detect tombstones, but I suspect a zero length byte field is
pretty close to null, and code could be written to deal with that, instead.
Also, your approach doesn’t require fixing the FileWriter.
Richard
Key ID: 3BDED925D4F316B3

On Aug 19, 2015, at 11:50 AM, estezz<EMAIL_ADDRESS>wrote:
Richard, did you make the same change I did?
public Message(String topic, int kafkaPartition, long offset, byte[]
payload) {
mTopic = topic;
mKafkaPartition = kafkaPartition;
mOffset = offset;
mPayload = payload;
if (mPayload == null) {
mPayload = new byte[0];
}
}
I don;t know why that changed broke the integration tests. But if you can
get it to work, go for it. A friend at another company and I are both
running my fix.
Let me know how it goes.
Leo
On Wed, Aug 19, 2015 at 10:32 AM, Richard Lee<EMAIL_ADDRESS>wrote:

Ditto on this null payload problem. I have changes in my tree to fix
this,
as I did not see that someone had already submitted a pull (112), but
that
pull seems 'bad'? What's the status on fixing this? Should I submit
another
pull request?
—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/92#issuecomment-132685977.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a https://www.uber.com/invite/ern6a
—
Reply to this email directly or view it on GitHub <
https://github.com/pinterest/secor/issues/92#issuecomment-132740226>.

—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/92#issuecomment-132764588.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a https://www.uber.com/invite/ern6a

@llamahunter indeed, you should consider proposing a change to the mainstream secor branch. I think it is a useful extension.

Richard are you planing to make a pull request for your fix to the null
payload?  If not I might follow your pattern and request a pull.
Leo
On Wed, Aug 19, 2015 at 2:08 PM, Richard Lee<EMAIL_ADDRESS>wrote:

I made a similar change.
https://github.com/TiVo/secor/tree/handle-null-payloads <
https://github.com/TiVo/secor/tree/handle-null-payloads>
However, I preserved the null value. It’s important to know when the value
is null to detect tombstones, but I suspect a zero length byte field is
pretty close to null, and code could be written to deal with that, instead.
Also, your approach doesn’t require fixing the FileWriter.
Richard
Key ID: 3BDED925D4F316B3

On Aug 19, 2015, at 11:50 AM, estezz<EMAIL_ADDRESS>wrote:
Richard, did you make the same change I did?
public Message(String topic, int kafkaPartition, long offset, byte[]
payload) {
mTopic = topic;
mKafkaPartition = kafkaPartition;
mOffset = offset;
mPayload = payload;
if (mPayload == null) {
mPayload = new byte[0];
}
}
I don;t know why that changed broke the integration tests. But if you can
get it to work, go for it. A friend at another company and I are both
running my fix.
Let me know how it goes.
Leo
On Wed, Aug 19, 2015 at 10:32 AM, Richard Lee<EMAIL_ADDRESS>wrote:

Ditto on this null payload problem. I have changes in my tree to fix
this,
as I did not see that someone had already submitted a pull (112), but
that
pull seems 'bad'? What's the status on fixing this? Should I submit
another
pull request?
—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/92#issuecomment-132685977.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a https://www.uber.com/invite/ern6a
—
Reply to this email directly or view it on GitHub <
https://github.com/pinterest/secor/issues/92#issuecomment-132740226>.

—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/92#issuecomment-132764588.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a https://www.uber.com/invite/ern6a

It’s unclear to me that my approach is better than yours, as, eventually, the value that both of us will write out to the archive via SequenceFile is a zero length array.  I am actually partial to your approach, as it is a single change that protects any other code in secor from having to check for a null value, as the payload variable is always non-null, if sometimes sort of pointless, being zero bytes.  For instance, I think my approach needs an additional code change to make the DelimitedTextFileWriter not crash on null.
The only optimization I might make to your approach is to have a static final zero byte array in the Message class that is used for null payloads so that there’s not object churn making new ones of these.  See, for instance, the implementation of the org.apache.hadoop.io.BytesWritable class.

On Aug 26, 2015, at 9:39 AM, estezz<EMAIL_ADDRESS>wrote:
Richard are you planing to make a pull request for your fix to the null
payload? If not I might follow your pattern and request a pull.
Leo
On Wed, Aug 19, 2015 at 2:08 PM, Richard Lee<EMAIL_ADDRESS>wrote:

I made a similar change.
https://github.com/TiVo/secor/tree/handle-null-payloads <
https://github.com/TiVo/secor/tree/handle-null-payloads>
However, I preserved the null value. It’s important to know when the value
is null to detect tombstones, but I suspect a zero length byte field is
pretty close to null, and code could be written to deal with that, instead.
Also, your approach doesn’t require fixing the FileWriter.
Richard
Key ID: 3BDED925D4F316B3

On Aug 19, 2015, at 11:50 AM, estezz<EMAIL_ADDRESS>wrote:
Richard, did you make the same change I did?
public Message(String topic, int kafkaPartition, long offset, byte[]
payload) {
mTopic = topic;
mKafkaPartition = kafkaPartition;
mOffset = offset;
mPayload = payload;
if (mPayload == null) {
mPayload = new byte[0];
}
}
I don;t know why that changed broke the integration tests. But if you can
get it to work, go for it. A friend at another company and I are both
running my fix.
Let me know how it goes.
Leo
On Wed, Aug 19, 2015 at 10:32 AM, Richard Lee<EMAIL_ADDRESS>wrote:

Ditto on this null payload problem. I have changes in my tree to fix
this,
as I did not see that someone had already submitted a pull (112), but
that
pull seems 'bad'? What's the status on fixing this? Should I submit
another
pull request?
—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/92#issuecomment-132685977.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a https://www.uber.com/invite/ern6a
—
Reply to this email directly or view it on GitHub <
https://github.com/pinterest/secor/issues/92#issuecomment-132740226>.

—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/92#issuecomment-132764588.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a https://www.uber.com/invite/ern6a
—
Reply to this email directly or view it on GitHub https://github.com/pinterest/secor/issues/92#issuecomment-135100755.

I can see it both ways, but my change is breaking the secor integration
tests and I do not know how to fix that.  I was hoping your approach would
pass the integration tests.
I will try some more.
here is the integration failures if you are interested.
https://travis-ci.org/pinterest/secor/builds/75126991
On Wed, Aug 26, 2015 at 10:53 AM, Richard Lee<EMAIL_ADDRESS>wrote:

It’s unclear to me that my approach is better than yours, as, eventually,
the value that both of us will write out to the archive via SequenceFile is
a zero length array. I am actually partial to your approach, as it is a
single change that protects any other code in secor from having to check
for a null value, as the payload variable is always non-null, if sometimes
sort of pointless, being zero bytes. For instance, I think my approach
needs an additional code change to make the DelimitedTextFileWriter not
crash on null.
The only optimization I might make to your approach is to have a static
final zero byte array in the Message class that is used for null payloads
so that there’s not object churn making new ones of these. See, for
instance, the implementation of the org.apache.hadoop.io.BytesWritable
class.

On Aug 26, 2015, at 9:39 AM, estezz<EMAIL_ADDRESS>wrote:
Richard are you planing to make a pull request for your fix to the null
payload? If not I might follow your pattern and request a pull.
Leo
On Wed, Aug 19, 2015 at 2:08 PM, Richard Lee<EMAIL_ADDRESS>wrote:

I made a similar change.
https://github.com/TiVo/secor/tree/handle-null-payloads <
https://github.com/TiVo/secor/tree/handle-null-payloads>
However, I preserved the null value. It’s important to know when the
value
is null to detect tombstones, but I suspect a zero length byte field is
pretty close to null, and code could be written to deal with that,
instead.
Also, your approach doesn’t require fixing the FileWriter.
Richard
Key ID: 3BDED925D4F316B3

On Aug 19, 2015, at 11:50 AM, estezz<EMAIL_ADDRESS>wrote:
Richard, did you make the same change I did?
public Message(String topic, int kafkaPartition, long offset, byte[]
payload) {
mTopic = topic;
mKafkaPartition = kafkaPartition;
mOffset = offset;
mPayload = payload;
if (mPayload == null) {
mPayload = new byte[0];
}
}
I don;t know why that changed broke the integration tests. But if
you can
get it to work, go for it. A friend at another company and I are both
running my fix.
Let me know how it goes.
Leo
On Wed, Aug 19, 2015 at 10:32 AM, Richard Lee <
<EMAIL_ADDRESS>wrote:

Ditto on this null payload problem. I have changes in my tree to
fix
this,
as I did not see that someone had already submitted a pull (112),
but
that
pull seems 'bad'? What's the status on fixing this? Should I submit
another
pull request?
—
Reply to this email directly or view it on GitHub
<
https://github.com/pinterest/secor/issues/92#issuecomment-132685977>.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a <
https://www.uber.com/invite/ern6a>
—
Reply to this email directly or view it on GitHub <
https://github.com/pinterest/secor/issues/92#issuecomment-132740226>.

—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/92#issuecomment-132764588.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a https://www.uber.com/invite/ern6a
—
Reply to this email directly or view it on GitHub <
https://github.com/pinterest/secor/issues/92#issuecomment-135100755>.

—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/92#issuecomment-135106538.

--
Leo Woessner
Sign up for Uber at this link and we both get free rides :)
https://www.uber.com/invite/ern6a https://www.uber.com/invite/ern6a


represent the first image error

when i click second or third image,turn back to click the first image ,it does not represent the first,but the sencond

Are you sure?

sorry, i said wrong


Add MaxNumSpans param to limit spans amount per trace

Which problem is this PR solving?
Resolves #100
Short description of the changes
Add new parameter MaxNumSpans that limits amount of spans fetched per each trace.

@EinKrebs could you please take a look?

From my side it looks fine


2nd iteration of Graph Explorer product tour

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

We are treating Graph Explorer product tour and Graph Explorer product changelog as 2 separate features.
Details and work progress on the product changelog can be found here: https://github.com/microsoftgraph/microsoft-graph-explorer-v4/issues/1215

Closing this issue because it's a duplicate


resin local should have ways to deploy container like resin deploy

resin local push is very handy to locally work on a device, but all the containers are still built on the device itself. It would be useful to have a tool to do similar to resin deploy, where an already built container can be pushed onto the device (resin local deploy?)
Forums

True! But already filed - this is a duplicate of #613


Add support for division by zero errors

Throw an exception when this occurs to indicate our simpliciation passes resulted in a singularity somewhere. Related to #1236

converting this to draft as its lower priority than the others

Closing this out as I'm adding this to PR#1281


orientation data in exif is always ORIENTATION_UNDEFINED

What are you trying to achieve or the steps to reproduce?
Read orientation data from exif
How did you initialize FA?
// init camera
        fotoApparat = Fotoapparat
                .with(getContext())
                .cameraErrorCallback(this)
                .cameraProvider(CameraProviders.defaultProvider(getContext()))
                .into(cameraView)
                .previewScaleType(ScaleType.CENTER_CROP)
                .build();
    }

// start camera
 fotoApparat.start();

// take picture

 fotoApparat.takePicture().toPendingResult().whenAvailable(new PendingResult.Callback<Photo>() {
                @Override
                public void onResult(Photo photo) {
                    handlePhoto(photo.encodedImage);
                }
            });
        }

// read exif from the byte array (using exif support library)
        InputStream in = null;
        ExifInterface exifInterface = null;
        try {
            in = new ByteArrayInputStream(encodedImage);
            exifInterface= new ExifInterface(in);
        }
        catch (IOException e) {}

        }

        exifInterface.getAttribute(ExifInterface.TAG_ORIENTATION); // <- always zero
       // ORIENTATION_UNDEFINED = 0;

What was the result you received?
Orientation data in exif is always ORIENTATION_UNDEFINED = 0;
checked on several devices
What did you expect?
correct orientation data
Context:
The activity is locked into portrait mode (using manifest)
I do rotate the device into landscape mode to take photos Photo.rotationDegrees will have correct values around 90 or 270 when device is in landscape.
FA Version: 1.4.1
Nexus 6p, Nexus 5x

I've created a demo project showcasing the issue
https://github.com/AviranAbady/FotoApparatIssueDemo
the project below has the activity set to portrait only.
The "take photo" button will  appear only when the phone is rotated to landscape position (There's an orientation listener)
Once an image is captured, exif is extracted - orientation will be undefined.
@Diolor @dmitry-zaitsev please assist, thanks.

Honestly I don't know how EXIF data behave. Did you find your solution?

@Diolor unfortunately, no.

will photo#rotation work for you?

@Diolor no, i cant tell from tham if the photo is upside down.
on most phones, rotating to the left (90 deg counter clock wise) will result in a normal photo
but on some phone (like nexus 5x)  if you rotate to the left (90 deg couter clock wise) you will get an upside down photo.
photo#rotation would be the same on both phone.
I've created a demo project, linked in the original post, if you wanna have a look.

https://github.com/Fotoapparat/Fotoapparat/blob/master/sample/src/main/java/io/fotoapparat/sample/MainActivity.java#L225

@Diolor it's not about how to rotate the bitmap, this is about knowing when it needs rotation.
In an app, when the activity is set to portrait only. If a photo is taken when the phone is rotated to landscape... the rotation of the image is unpredictible.
on most phones, if you rotate clock wise it will be upside down.
on some phones, it's the other way around...
currently it's impossible to know when you need to rotate it back to normal.

for everyone who reads this discussion, here's the answer:
val orientation = ExifInterface(file.absolutePath).getAttribute(ExifInterface.TAG_ORIENTATION)

I was having this issue as well. After checking checking the code I found that saveToFile write this attribute async, so I just wait it to be done.
photoResult.saveToFile(targetFile)
                    .whenDone(unit -> {
						// do you your stuff here
                    });


Editor crashing after mismatching ROS2 distributions

Describe the bug
When compiling Ros2ForUnity asset and launching Unity Editor with different distro sourced, the Editor crashes without any log info.
To Reproduce
Steps to reproduce the behavior:

Build R2FU asset for ROS2 foxy version
Run Unity Editor with ROS2 galaxy sourced
Run simulation with example talker included in scene

Expected behavior
The Editor doesn't die and logs information that ROS2 distros has been mixed.
Desktop (please complete the following information):

OS: Ubuntu20.04
ros2 distro foxy/galactic
ros2-for-unity version 8bd00f3ce5f2da1e67812cd2d2f4cdbdba584e91
ros2cs version cbb0e00c773fd45fca9c5d64d27e20f04be53fff  (preload_typesupport branch)
ros2 dds middleware w/e
ros2 environment setup single pc

To investigate: how to find out which distribution plugins belong to. If it is not possible in a nice manner (e.g. without assumptions or dirty hacks), consider providing distro info file as part of the Asset build.


Support to async/await

Hi.
Will slumber support async/await?

Merge/cooperate with https://github.com/chassing/aionap ?


Optimize Select#selected_options and Select#value using JS

Continuation from #540

If we don't call #options we still need to add a line for element_call(:wait_for_exists) to make sure select list is there before working with it.

Done.

There is always a chance element goes stale during script execution, so this should probably be wrapped with element_call(:wait_for_exists).

merged: 1f22570e0cb46e66ca9296f5837fb78792d99e20


⚠ Bofrost.at has degraded performance

In 7ac67a6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1085 ms

Resolved: Bofrost.at performance has improved in b99220d after 6 minutes.


Restructure GCE instructions for consistency with AWS instructions

Coverage remained the same at 80.108% when pulling eb3fc7bb7cc10a0496984d6e3c0fa3aba1e75e4d on Georgehe4:gce_notes into 1d8731580319fd2216a9ebd02210e225c9d9b761 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/874/
Test PASSed.

Thanks @Georgehe4 !


ADDED MARRIAGE PREDICTOR GAME

THIS WILL PREDICT WHETHER YOU WILL MARRY IN NEAR FUTURE OR NOT.
A SMALL GAME MADE FOR FUN JUST BY HTML,CSS AND JAVA SCRIPT
PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes  Issue #3233

Mark the task you have completed ✅

[✅ ] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[✅ ] I have performed a self-review of my own code or work.
[ ✅] I have commented my code, particularly in hard-to-understand areas.
[✅ ] My changes generates no new warnings.
[ ] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ ✅] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[✅ ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Nih1tGupta ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @Nih1tGupta,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗

Thank you  @Nih1tGupta , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

But sir, you have assigned me earlier,what to do now.
please review my PR


Cinder: fix thread unsafe global client Microversion modification

What this PR does / why we need it:
The #620 and #786 PRs haven't considered that modified variable is global, and this action is not thread-safe. This PR resolves unsafe global variable modification.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #804
Special notes for your reviewer:
@mape90 @ramineni @lingxiankong @adisky please review this PR and let me know if the bug has to be solved in another way
Release note:

Cinder: fix thread unsafe global client Microversion modification

/ok-to-test

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.15

@lingxiankong @ramineni @zetaab @adisky any news on this PR?

Thanks.
/lgtm

/approve

The error message occurs only in golang 1.13.x, see https://github.com/golang/go/issues/31859
Pushed the fix.

Another issue https://github.com/kubernetes/kubernetes/pull/84662

/recheck cloud-provider-openstack-acceptance-test-csi-cinder

/retest cloud-provider-openstack-acceptance-test-csi-cinder

/test cloud-provider-openstack-acceptance-test-csi-cinder

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-k8s-cinder

/test cloud-provider-openstack-acceptance-test-flexvolume-cinder

/test cloud-provider-openstack-acceptance-test-keystone-authentication-authorization

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

go version was fixed, now there is another issue with go mod checksum: https://github.com/kubernetes-sigs/kubebuilder/issues/1169
@adisky is there an ability to merge this without waiting the dependency fix? this PR already contains the go 1.13 fix, which could be used in other PRs

/test cloud-provider-openstack-acceptance-test-flexvolume-cinder

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.14
/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.15

/test cloud-provider-openstack-acceptance-test-e2e-conformance
/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.14
/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.15

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.14

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.15

@adisky @ramineni should be fine to merge

/lgtm


full traffic time delay queues implementation

Ayal: This domain encoding goes against the RDDL principle that structured
complexity should be expressed through nonfluents and not hardcoded in the
domain itself.  Use a time-delay object and a previous/next nonfluent to
determine how M_t gets updated from M_t+1; you can convert these objects to
actual integers (if needed) through another nonfluent.  This also allows
you to adjust the time "memory" in an instance.  Thanks -Scott
On Sun, Dec 25, 2022 at 11:17 AM ataitler @.***> wrote:

Merged #139 https://github.com/ataitler/pyRDDLGym/pull/139 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/ataitler/pyRDDLGym/pull/139#event-8108566257, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABRA3JG7Y2IQOWXMUYPIXILWPBXS3ANCNFSM6AAAAAATJBZTHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Admin Area - Capitalization on several fields is still wrong

We got some of these in #1046, but some still remain:
Change Chant page:

[x] "Melody id" -> "Melody ID"

Change Source page:

[ ] "Rism siglum" -> "RISM siglum"
[x] "Fragmentarium id" -> "Fragmentarium ID"
[x] "Dact id" -> "DACT ID"

I can't figure out how to change RISM Siglum. Specifying a verbose_name in the Source model doesn't have any effect, and a verbose name with proper capitalization is already defined in the RismSiglum model anyways. Trying to set other ForeignKey fields' verbose names has similar non-effects.
Going to open a PR that addresses everything but RISM siglum, since this was supposed to be a quick fix and we might rename the model eventually anyways (#621)


no-op implementation support

In my current case, the retrofit url manager only needed to switch base url at development phase, so the releaseImplementation not needed

cc @JessYanCoding

In app/build.gradle
android {
    ...

    buildTypes {
        debug {
            buildConfigField "boolean", "LOG_DEBUG", "true"
        }

        release {
            buildConfigField "boolean", "LOG_DEBUG", "false"
         }
   }
}

In okhttp initialization
if (BuildConfig.LOG_DEBUG) {
     RetrofitUrlManager.getInstance().with(OkHttpClient.Builder);
}


chore(dep) Upgrade to Cicero 0.20.6

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>

These builds are failing currently in Netlify, looks like something with webpack: https://app.netlify.com/sites/accordproject-studio/deploys/5df01b976798c600083d4806

These builds are failing currently in Netlify, looks like something with webpack: https://app.netlify.com/sites/accordproject-studio/deploys/5df01b976798c600083d4806

Thanks! I had missed that.
I've updated to Cicero 0.20.6 and Cicero UI 0.3.13.
I've also fixed the web pack configuration to ignore jsdom. This seems to deploy properly again:
https://app.netlify.com/sites/accordproject-studio/deploys/5dfb97522addbc0009311010


Promote to Beta

Package name: google-cloud-compute
Current release: alpha
Proposed release: beta
Instructions
Check the lists below, adding tests / documentation as required. Once all the "required" boxes are ticked, please create a release and close this issue.
Required

[x] Server API is beta or GA
[x] Service API is public
[x] Client surface is mostly stable (no known issues that could significantly change the surface)
[ ] All manual types and methods have comment documentation
[ ] Package name is idiomatic for the platform
[x] At least one integration/smoke test is defined and passing
[ ] Central GitHub README lists and points to the per-API README
[ ] Per-API README links to product page on cloud.google.com
[ ] Manual code has been reviewed for API stability by repo owner

Optional

[ ] Most common / important scenarios have descriptive samples
[ ] Public manual methods have at least one usage sample each (excluding overloads)
[ ] Per-API README includes a full description of the API
[ ] Per-API README contains at least one “getting started” sample using the most common API scenario
[ ] Manual code has been reviewed by API producer
[ ] Manual code has been reviewed by a DPE responsible for samples
[ ] 'Client LIbraries' page is added to the product documentation in 'APIs & Reference' section of the product's documentation on Cloud Site

google-cloud-compute is already 1.27.0.
https://central.sonatype.com/artifact/com.google.cloud/google-cloud-compute/1.27.0


Ensure Java >= 8 on launched nodes

As raised in #142, Spark 2.0 deprecates support for Java 7. Furthermore, lots of Catalyst/Tungsten optimizations are enabled by running on Java 8.
Flintrock should, by default, ensure that all launched nodes have Java 8 or higher installed and configured to be the default.
If the user uses an AMI that already has this setup, Flintrock should detect that and do nothing.

We should probably do this by having an ensure_java8() function inside of core.py that takes an SSH client as input and makes sure the host on the other end has Java 8 or higher. We should use this function anytime new instances are created--i.e. during initial launch or when slaves are added to an existing cluster.

Do you plan to do this? Or is this issue available for anyone to tackle on? :)

PRs are welcome. 👍 Thanks for asking!
I suggest waiting until after #115 gets in to work on this, since that will affect how this feature is implemented.

Okay. I'll work on this as soon as the PR is merged.


URL support

Hi there! Does this company support connecting URLs instead of plain IPs? I had some problems with this when I wrote the LDPL IRC Bot library, and solved it using this.
If your library doesn't support URLs, please consider using this add-on. I will try to upload it as a library asap, but you can get it from that link atm.

I have compiled and uploaded said library here: https://github.com/Lartu/ldpl-url-resolver

It does support hostnames, in theory. I use that feature in Gild and just added a simple Gopher example to this repo that will prompt you for a hostname and then fetch a raw Gopher response:
$ ldpl examples/gopher-client.ldpl
* Loading examples/gopher-client.ldpl
* Compiling examples/gopher-client.ldpl
* Compiling examples/../ldpl_socket/ldpl_socket.ldpl
* Including C++ extension examples/../ldpl_socket/ldpl_socket.cpp
* Building examples/gopher-client-bin
* Saved as examples/gopher-client-bin
* File(s) compiled successfully.
$ ./examples/gopher-client-bin
~ Enter hostname of Gopher server: sdf.org
~ Response from sdf.org:70 on socket 3:
iWelcome to the SDF Public Access UNIX System .. est. 1987.....

What error or issue were you running into? I'm sure there are still bugs here. 🐛  🐞 🐝

Wonderful! I wasn't running into any errors or issues, I just wanted to know, haha. Great to know that it works, thank you very much!


Popup Window Management

Issue
Currently, the Signer will open a separate popup instance when required for Connecting, Signing or Importing a key.
This can lead to multiple popups being open simultaneously which in turn causes confusion for the user, sync issues and just clutters the desktop.
Solution
The solution here is to centralise the control of opening and closing windows via a container interface in the same vein as the architectures for Connection and Signing.
By channelling all window management through one point the currently open window can be tracked and ensure only one popup is open at a time.
Asset
Signer (PR 188)

I tried to build extension. And have got this error on Import account page

STR:

Open extension and go to the Import Account page
Import account and close popup window
Open extension and go to the Import Account page again

@vladimir-gachkovsky bug should be fixed with https://github.com/casper-ecosystem/signer/pull/188/commits/7b49283f2bf51b00eaf1b624c482133e33e912ac - just a small change.
cc @piotrwitek

@piotrwitek @vladimir-gachkovsky
I've just pushed the update to resolve your comments, https://github.com/casper-ecosystem/signer/pull/188/commits/00a3c1f76847d6d6aa117960df9aea5ef6d7778e
I think it would be worthwhile creating other refactoring PRs once this current set of PRs is published e.g. I'd like to store all of the RPC endpoints in a shared file to avoid typos between background/frontend but I don't think it's a relevant change for this work.

Found something...
https://user-images.githubusercontent.com/97024085/151217558-0b9126ae-4085-4938-9f12-8afd9fe94bda.mp4
I suggest to close opened popup after operation will complete
STR:

Open extension and import account (don't close opened popup after action will complete)
Click on Sign In button in header menu on CSPR.live site
Click Sign In button inside Casper Signer section on CSPR.live site

Actual result: You have got 2 popup windows
Expected result: Should be single popup window

There is an issue with the PopupManager which still opens a new window instead of reusing the existing one.
STR: it happens when invoking the connection request.
ER: There is only one window.
AR: Each request will open a new window.
Tomorrow I'll analyze the code and propose a solution.

@George-cl @vladimir-gachkovsky I have fixed both issues reported by me and Vladimir, need you both to test all possible scenarios for any additional bugs.

@piotrwitek Code changes look good - I've tried it out and it seems like it's working well

@vladimir-gachkovsky what is the status?


Allow users to set a startupProbe

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
We've previously had issues with rolling updates where traefik pods have become ready and start accepting traffic before all objects are loaded. For a brief period this means requests end up with a 404 response.
I've found a previous issue that seem to be related and the user resolved it by setting a startupProbe (as we have done) to get around this (our probe executes a command to query the api to make sure resources have been loaded as expected).
https://github.com/traefik/traefik/issues/9078
Would it be possible to allow users to set their own startupProbe in a values file (as you do liveness and readiness probes)? At the moment we're having to manage the _podtemplate.tpl template ourselves to create the probe.

I've raised a PR to support this; any comments more than welcome: https://github.com/traefik/traefik-helm-chart/pull/956

@87gs : which startupProbe do you set to avoid these HTTP 404? Did you manage to setup one that would wait for all routes to be ready?
I have the same issue. Another side effect is that Traefik can present the wrong TLS certificate, depending on the order with which routes are setup.


PropsParserContext.h 'react/utils/ContextContainer.h' file not found

Environment
RN0.72.3
rnpagerview 6.2.0
Description
On ios building in xcode I get the following error:
PropsParserContext.h 'react/utils/ContextContainer.h' file not found

Hello @jslok,
Did you run pod install inside the ios directory?

Does an empty react native app build without the react-mative-pager-view lib? if yes, that;s not a react-native issue.
If that's not the case, please open an issue in the react native repo with a reproducer created starting from this template

Workaround until resolved by react native:
Add 'react-native/ReactCommon' to the header search path in your Podfile (in my case, the "RNScreens" pod was giving the problem):
    installer.pods_project.targets.each do |target|
          if target.name == "RNScreens"
              append_header_search_path(target, "${PODS_ROOT}/../../node_modules/react-native/ReactCommon")
          end
        end

Also define the  append_header_search_path in your Podfile:
  def append_header_search_path(target, path)
      target.build_configurations.each do |config|
          # Note that there's a space character after `$(inherited)`.
          config.build_settings["HEADER_SEARCH_PATHS"] ||= "$(inherited) "
          config.build_settings["HEADER_SEARCH_PATHS"] << path
      end
  end

Workaround until resolved by react native:
Add 'react-native/ReactCommon' to the header search path in your Podfile (in my case, the "RNScreens" pod was giving the problem):
    installer.pods_project.targets.each do |target|
          if target.name == "RNScreens"
              append_header_search_path(target, "${PODS_ROOT}/../../node_modules/react-native/ReactCommon")
          end
        end

Also define the append_header_search_path in your Podfile:
  def append_header_search_path(target, path)
      target.build_configurations.each do |config|
          # Note that there's a space character after `$(inherited)`.
          config.build_settings["HEADER_SEARCH_PATHS"] ||= "$(inherited) "
          config.build_settings["HEADER_SEARCH_PATHS"] << path
      end
  end

Did not work for me. Any other work around?

RNScreens

How did you know that RNScreens was giving you the problem ?

Closing as its not related to react-native-pager-view


Implement superjson to trpc as the default transformer

Currently Next.js is not handling date from js objects correctly

Using custom transformer for now


[ZEPPELIN-3722] Zeppelin UI shows "configured HomePage is not existed" with default config

What is this PR for?
With default zeppelin-site.xml config i.e. "zeppelin.notebook.homescreen" set to empty string Zeppelin home page shows "configured HomePage is not existed".
What type of PR is it?
[Bug Fix]
What is the Jira issue?

ZEPPELIN-3722

Questions:

Does the licenses files need update? N/A
Is there breaking changes for older versions? N/A
Does this needs documentation? N/A

@zjffdu  can you help review this.

Thanks @prabhjyotsingh I have fixed in #3133

Sure, did not notice this, will close this.


Make properties for preview configurable

Added options pattern to read property aliases from appsettings.
Added notification handler for server variables to make configured property aliases useable in preview controller.
Changed controller to use configured aliases to search for properties.

Hi @Mantus667
Thanks for your PR 👍
Unfortunately I haven't had notifications enabled for this repository, so I haven't seen your PR before now.
Since the preview is a property editor, I think the best approach is to be able to configure this via the data type. I had the same thought about being able to configure the properties, and since I hadn't seen your PR, I have now added this to the latest pre-release of the package.
When editing the data type, you'll now get the following options:

You may specify number of properties for either field, and the property editor will then go through them one by one until it has found a value. If a title value isn't found, the page's name will be used as fallback.
As I ended up implementing this on my own, it also means that your PR will not be merged. I hope that makes sense 😉


Apologies for delayed response

Hi sveltestrap users,
I ended up at a role last year that took most of my free time for Sveltestrap; my apologies for lack of response these past few months.
I've left that position and accepted a better role, so in next 2 weeks approve and release any PRs ready to merge, and will also look at getting a Bootstrap 5.3 (beta) release ready.  I'll try and triage any important issues as well.
Thanks again for your patience and for using Sveltestrap 💜

Thank you!
The #356 i think is the most urgent.

@bestguy - If there is anything we can do to assist, please let us know.

Thanks @ruckc @8lall0 ,  please see my comment here:
https://github.com/bestguy/sveltestrap/pull/356#issuecomment-1264538484
I'd love to fix this for Sveltekit, but not willing to break usage in svelte.dev REPL
Any and all help that could address that would be appreciated.

Update, we have an additional maintainer added to project, and will do best to catch up with issues and PRs.

Thank you @bestguy for the amazing effort.
I know how difficult it is to juggle between the day job and beloved open source masterpiece.
All the best :heart:


WIP Check dual stack hash

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

/test pull-kubevirt-e2e-k8s-1.18

all passed on
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/3922/pull-kubevirt-e2e-k8s-1.18/1289872709878026240


providers/redfish: implement user deletion method

What does this PR implement/change/remove?

Implements user deletion over redfish

The HW vendor this change applies to (if applicable)

Dell

What version of tooling - vendor specific or opensource does this change depend on (if applicable)
github.com/stmcginnis/gofish v0.12.0

Description for changelog/release notes

User deletion over redfish

Features list updated