Display Player Name Above Character Icon

Addressing issue #112
As usual, version published on my github pages

Added player name to show above player icon
Feature will work in conjunction with the Show player names settings. Namely, the name(s) will show if the settings is on, vice versa

slight issue with the draggable characters, the map expand/collapse does mess with the characters' positioning, meaning if you placed someone in "electrical" in skeld, they'll end up somewhere else once the map is collapsed/expanded

Please discuss said thing on #138

please merge and close this if it looks good. thanks!


ImportError: cannot import name 'container_abcs' from 'torch._six' (D:\PycharmProjects\nanodet-main\p3.8venv\lib\site-packages\torch_six.py)

use pytorch version 1.8.0

Have you solved the problem？

You cab take as base this commit to fix this

https://discuss.pytorch.org/t/cannot-import-name-int-classes-from-torch-six/126611


Bug 2130279: ceph: fixing ClientID of log-collector for RGW instance

The Client_ID generated by operator was
different from the log rotate file created
The Clinet_ID= rgwceph.client.rook.ceph.rgw.my.store.a and log file name= ceph-client.rgw.my.store.a.log
So changed the CLient_ID to ceph-client.rgw.my.store.a for correct working and this follow the patterns how other modules Client_ID is generated
Closes: https://github.com/rook/rook/issues/8692
Signed-off-by: parth-gr<EMAIL_ADDRESS>(cherry picked from commit fc7905a7bd37529c7e3e1be5e96bc3b2a507b543)

Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide).
[ ] Skip Tests for Docs: If this is only a documentation change, add the label skip-ci on the PR.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

/assign @travisn @agarwal-mudit

/approve
/lgtm
The BZ just needs acks


Release 0.12 checklist

(From: https://github.com/poliastro/poliastro/wiki/HOWTO-Release - Next time let's have a template, see #516)
Preparing the release

[x] Rerun all notebooks (pay special attention to new features)
[x] Review documentation, optionally improving the user guide or adding new notebooks showcasing new functionality
[x] Write the changelog and list of contributors

Commits since last release branch https://github.com/poliastro/poliastro/compare/0.11.x...master
Issues closed in this milestone https://github.com/poliastro/poliastro/milestone/12?closed=1 (note that there might be issues without milestone!)
Pull requests merged since last branching https://github.com/poliastro/poliastro/pulls?utf8=✓&q=is%3Apr+merged%3A>2018-11-05+base%3Amaster

[x] Add 0.12.x branch to CI scripts

Before the beta release

[x] New branch
[x] Bump version to 0.12b1 in:

README.rst
__init__.py
Sphinx conf.py
AppVeyor CI script

[x] Check that the release branch will be tested on CI
[x] Check all the badges in README point to the appropriate git branch (replace master by new branch sed -i 's/master/.../g' README.rst)
[x] Check that docs badges and URLs point to appropriate tag (replace latest by new tag, without sed!)
[x] Commit
[x] Generate sdist and bdist_wheel
[x] twine upload dist/* --repository-url https://test.pypi.org/legacy/
[x] Tag
[x] Bump master to next development version

Beta release

[x] Push branch to GitHub and tags
[x] Check on Test PyPI that the badges will work
[x] Upload sdist and bdist_wheel to PyPI - this step cannot be undone if the release is removed! twine upload dist/* --repository-url https://upload.pypi.org/legacy/
[ ] Check Read the Docs (check https://docs.readthedocs.io/en/latest/webhooks.html#github first)

Before final release

[ ] Backport any bugs
[ ] Bump version to 0.12.0 in:

README.rst
__init__.py
Sphinx conf.py
AppVeyor CI script

[ ] Check that docs badges and URLs point to appropriate tag (replace v0.12b1 by v0.12.0)
[ ] Commit
[ ] Generate sdist and bdist_wheel
[ ] twine upload dist/* --repository-url https://test.pypi.org/legacy/
[ ] Tag
[ ] Cherry pick the changelog with the release date

Final release

[ ] Push tag to GitHub
[ ] Upload sdist and bdist_wheel to PyPI - this step cannot be undone if the release is removed! twine upload dist/* --repository-url https://upload.pypi.org/legacy/
[ ] Check Read the Docs (check https://docs.readthedocs.io/en/latest/webhooks.html#github first)
[ ] Create GitHub release
[ ] Add Zenodo badge to GitHub release
[ ] Send announcement email
[ ] Add link to announcement email in GitHub release
[ ] Close milestone

poliastro 0.12b1 is out! https://pypi.org/project/poliastro/0.12b1/

Candidates to backport, at least partially: #578, #574, #573.

poliastro 0.12.0 is available https://github.com/poliastro/poliastro/releases/tag/v0.12.0

Announcement sent https://groups.io/g/poliastro-dev/message/85 closing!


[SPARK-39527][SQL] RenameTable should support same catalog name in  newTable's identifier

What changes were proposed in this pull request?
For a simple case of RenameTable
  test("rename a table") {
    sql("ALTER TABLE h2.test.empty_table RENAME TO h2.test.empty_table2")
    checkAnswer(
      sql("SHOW TABLES IN h2.test"),
      Seq(Row("test", "empty_table2")))
  }

It will throw exception as below
[info] - rename a table *** FAILED *** (2 seconds, 358 milliseconds)
[info]   org.apache.spark.sql.catalyst.analysis.NoSuchNamespaceException: Failed table renaming from test.empty_table to h2.test.empty_table2
[info]   at org.apache.spark.sql.jdbc.H2Dialect$.classifyException(H2Dialect.scala:117)
[info]   at org.apache.spark.sql.execution.datasources.jdbc.JdbcUtils$.classifyException(JdbcUtils.scala:1176)
[info]   at org.apache.spark.sql.execution.datasources.v2.jdbc.JDBCTableCatalog.$anonfun$renameTable$1(JDBCTableCatalog.scala:102)
[info]   at org.apache.spark.sql.execution.datasources.v2.jdbc.JDBCTableCatalog.$anonfun$renameTable$1$adapted(JDBCTableCatalog.scala:100)
[info]   at org.apache.spark.sql.execution.datasources.jdbc.JdbcUtils$.withConnection(JdbcUtils.scala:1184)
[info]   at org.apache.spark.sql.execution.datasources.v2.jdbc.JDBCTableCatalog.renameTable(JDBCTableCatalog.scala:100)
[info]   at org.apache.spark.sql.execution.datasources.v2.RenameTableExec.run(RenameTableExec.scala:51)
[info]   at org.apache.spark.sql.execution.datasources.v2.V2CommandExec.result$lzycompute(V2CommandExec.scala:43)
[info]   at org.apache.spark.sql.execution.datasources.v2.V2CommandExec.result(V2CommandExec.scala:43)
[info]   at org.apache.spark.sql.execution.datasources.v2.V2CommandExec.executeCollect(V2CommandExec.scala:49)
[info]   at org.apache.spark.sql.execution.QueryExecution$$anonfun$eagerlyExecuteCommands$1.$anonfun$applyOrElse$1(QueryExecution.scala:98)
[info]   at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withNewExecutionId$6(SQLExecution.scala:111)
[info]   at org.apache.spark.sql.execution.SQLExecution$.withSQLConfPropagated(SQLExecution.scala:171)
[info]   at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withNewExecutionId$1(SQLExecution.scala:95)
[info]   at org.apache.spark.sql.SparkSession.withActive(SparkSession.scala:779)
[info]   at org.apache.spark.sql.execution.SQLExecution$.withNewExecutionId(SQLExecution.scala:64)

From here we pass all identifiers to the backend catalog with catalog, it will failed the query.
In this pr we do below things:

support both :
ALTER TABLE h2.test.src_table RENAME TO test.dst_table
ALTER TABLE h2.test.src_table RENAME TO h2.test.dst_table
If use catalog.db.table in newIdent, should check catalog is same as old table ident

Why are the changes needed?
Fix bugs. RENAME TABLE 's newTable and oldTable in V2 should both bring catalog.
Does this PR introduce any user-facing change?
When people want to use RENAME TABLE in v2, should bring catalog info for new ident too.
How was this patch tested?
Added UT

ping @cloud-fan

ping @cloud-fan How about current?

gentle ping @dongjoon-hyun @HyukjinKwon @cloud-fan

Any suggestion?


How do I use/cast OpaquePointer to Sprite?

I'm pretty excited about this library - and the instructions worked perfectly!
However I'm running into an issue (that may be my misunderstanding of how to use PlaydateKit!)
When I use setUpdateFunction on a sprite - the closure defaults to passing me a variable called sprite (which makes perfect sense - I probably want access to that sprite to do something with it) - but the type is OpaquePointer...
Is there a way to cast this into a PlaydateKit Sprite? What is the best way to use this?

Looks like a good solution. Thanks for the quick response - and great work!

Looks like a good solution. Thanks for the quick response - and great work!

done! example usage here:
https://github.com/finnvoor/PlaydateKitTemplate/blob/main/Sources/PlaydateKitTemplate/Game.swift


This  plugin  disable  my  'yy'  normal  command.

This  plugin  disable  my  'yy'  normal  command.
I don't  know why.
I  also  use the  'YankRing'  plugin.

Can you tell me the output of following commands?
:verbose nmap yy
:verbose omap y

this is my  verbose map   output. (and there is no nmap for 'yy' )
:verbose  nmap  y
ns y             (operator-flashy)
Last set from /mySaved/dot-vim/.vim/vim-scripts/plugin-vim/plugin-common.vim
:verbose omap  y
o  y             YRMapsExpression("68_", "y")
Last set from /mySaved/dot-vim/.vim/bundle/YankRing.vim/plugin/yankring.vim

now,  I use this  mapping  to  fix  the  bug:
nnoremap  yy  yy

That fixes the bug, but then you don't get the flashy effect when you use yy, right? I have the same problem. I would like to have a yank-flashy-line if that would be possible


placeholder not working for boolean fields

Describe The Bug
The placeholder attribute on a property is being ignored.  For example,
"logErrors": {
    "title": "Log errors",
    "type": "boolean",
    "placeholder": true
}

Results in the GUI displaying the checkbox without it being checked, when the expected behavior is for it to be checked.
Placeholder is working fine for other field types, e.g. number and string.
Logs
No response
Config
No response
Homebridge UI Version
homebridge-config-ui-x v4.50.6
Homebridge Version
v1.6.1
Node.js Version
v18.17.1
Operating System
Ubuntu / Debian (or a variant)
Environment Info

[ ] Using Docker?
[ ] Using Hyper-V?
[ ] Using hb-service?

Raspberry Pi Model
None

You would want to do default instead of placeholder.

You would want to do default instead of placeholder.

Yes default works.  But it has different behavior... that will write the setting to config.json whether it is changed from the default or not.  I was looking for a way to only write settings that are changed from the default value.
I think I understand why it is this way... because the support is really tied up in how browsers / HTML allow a placeholder value to be displayed in an entry field and that is not available for checkboxes.
It can be worked around if you go whole-hog and write a custom UI script.  But that is a lot of work.
Thanks.


Udpate labelImage

For fix my environement

HI @SnowMasaya ,
May I know why you would like to add this?

Hi, Sorry. I mistake this commit. You don't need to add this.


Don't force -lcudart on non-CUDA programs when using CUDA build of Charm++

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

If CUDA supposed is detected by configure it not only builds the CUDA features, but also modifies the link options for programs that do not use Charm++ CUDA support to include -lcudart, such that even CPU-only binaries cannot run without CUDA installed.

Gerrit patch: https://charm.cs.illinois.edu/gerrit/#/c/charm/+/5176/

Jim is only talking about the CUDA build of Charm++ here. At the time, non-CUDA programs were forced to link in the CUDA libraries because of the contents of src/arch/common/conv-mach-cuda.sh.

The patches to fix this cause more problems, reverting them for now (#2466). Reopening this issue.

Jim is only talking about the CUDA build of Charm++ here. At the time, non-CUDA programs were forced to link in the CUDA libraries because of the contents of src/arch/common/conv-mach-cuda.sh.

Actually, I was talking about non-CUDA builds of Charm++, and the fix I was expecting was to not do anything CUDA-dependent unless the user explicitly requested CUDA when building Charm++. The new issue title is incorrect.

@jcphill The file that forces programs to link in the CUDA library, src/arch/common/conv-mach-cuda.sh, isn't it only used in CUDA builds? Currently I don't think there's anything CUDA-related that is required unless Charm++ is built with cuda.

The problem is/was that if nvcc etc. was detected by configure it would automatically enable the CUDA features in Charm++ even if they were not requested.

In any case, I can't seem to reproduce this anymore.

Looking at the current code in configure.ac, Makefile, and conv-mach-cuda.sh, cuda support shouldn't be activated without the user explicitly specifying the cuda option . Closing this for now.


When is a component a view?

Fantastic example app btw; it's been really helpful. Thank you!
Quick conceptual question: why are some components in the views folder and some in components? I can see that views are much simpler, but just wondering where the line is?
BTW: My specific example is a sidebar menu component. It has state, it's always included in other views, but it's simple.. my guess is with this criteria it's a component (being included in a view).

Sloppy work on my part. To be honest, I'm not convinced that they shouldn't all be lumped together as "components". I think I was going for "things that are routes" and "things that are not".

Ok cool. That makes sense.
I'll move them all into the components directory and play around with some organizational folders.
:)


docstr updated for use_docker in execute_code

Why are these changes needed?
Mentioned in Issue #214 , Made the required changes

Related issue number
Closes issue #214

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report

Merging #233 (157355e) into main (6156faa) will decrease coverage by 2.40%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #233      +/-   ##
==========================================
- Coverage   43.31%   40.92%   -2.40%
==========================================
  Files          17       17
  Lines        2133     2133
  Branches      481      481
==========================================
- Hits          924      873      -51
- Misses       1126     1173      +47
- Partials       83       87       +4

Flag
Coverage Δ

unittests
40.92% <ø> (-2.30%)
:arrow_down:

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

Files
Coverage Δ

autogen/code_utils.py
39.65% <ø> (-9.49%)
:arrow_down:

... and 1 file with indirect coverage changes

The code formatting issue is not caused by this PR. Please recover the space after '-'.
I made a PR to fix the code formatting #242

The code formatting issue is not caused by this PR. Please recover the space after '-'. I made a PR to fix the code formatting #242

Sorry for the misunderstanding.... i will revert the changes

@sonichi  I have reverted the changes by adding space in -

@qingyun-wu your requested change blocks merging.

@sonichi are there any changes to be done from my side..


Pulsar can get stuck on a single unreadable entry

Describe the bug
Pulsar can get stuck on a single unreadable entry in bookkeeper
To Reproduce
Increase max message size from the default 5M to i.e 10M.
Write a ledger/stream with entries under 5M, entry < 5M, and then some that are less than 5M.
reduce max message size back to 5M.
try to process the ledger
Pulsar gets stuck on the entry > 5M and autoSkipNonRecoverableData does not help
Pulsar logs
org.apache.bookkeeper.mledger.impl.OpReadEntry - ... read failed from ledger at position:X:Y : Bookie handle is not available

Expected behavior
autoSkipNonRecoverableData to allow skipping such entry
Additional context
This is not a problem right now (worked around this) and I will not spend more time on this, mostly an FYI in case anyone hits this.
Below is rather untested diff in case anyone needs it; to deal with this normally it would require unit tests with repro of such situations and/or similar tests in the bookkeeper (plus, possibly, better handlings of such entries on the bookie side)
diff --git a/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/OpReadEntry.java b/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/OpReadEntry.java
index 91a6e26f567..fd0b0519280 100644
--- a/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/OpReadEntry.java
+++ b/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/OpReadEntry.java
@@ -23,6 +23,8 @@
 import io.netty.util.Recycler;
 import io.netty.util.Recycler.Handle;
 import java.util.List;
+
+import org.apache.bookkeeper.client.BKException;
 import org.apache.bookkeeper.mledger.AsyncCallbacks.ReadEntriesCallback;
 import org.apache.bookkeeper.mledger.Entry;
 import org.apache.bookkeeper.mledger.ManagedLedgerException;
@@ -97,6 +99,22 @@ public void readEntriesFailed(ManagedLedgerException exception, Object ctx) {
                 callback.readEntriesComplete(entries, ctx);
                 recycle();
             }));
+        } else if (cursor.config.isAutoSkipNonRecoverableData()
+                && exception.getCause() instanceof BKException.BKBookieHandleNotAvailableException) {
+            // It is possible to create situation when bookie client won't be able to read valid existing entry.
+            // Specifically: write large entry and then reduce max message size
+            // Bookie client will disconnect on attempt to deal with this
+            // and throw the exception BKBookieHandleNotAvailableException.
+            log.warn("[{}][{}] read failed from ledger at position:{} : {}; will skip the entry",
+                    cursor.ledger.getName(),
+                    cursor.getName(),
+                    readPosition,
+                    exception.getMessage(),
+                    exception);
+            // Move to next valid position, skipping this one entry
+            final Position nexReadPosition = readPosition.getNext();
+            updateReadPosition(nexReadPosition);
+            checkReadCompletion();
         } else if (cursor.config.isAutoSkipNonRecoverableData() && exception instanceof NonRecoverableLedgerException) {
             log.warn("[{}][{}] read failed from ledger at position:{} : {}", cursor.ledger.getName(), cursor.getName(),
                     readPosition, exception.getMessage());

@dlg99 Thanks for open this issue, Would you like to provide a fix for this?

+        } else if (cursor.config.isAutoSkipNonRecoverableData()
+                && exception.getCause() instanceof BKException.BKBookieHandleNotAvailableException) {

The BKBookieHandleNotAvailableException is thrown when the client is not able to connect to a bookie, so this fix would result in a lot of data dropped :)

@merlimat hence this and not a pull request.
The decent fix would require change not the bookie side to handle this situation differently/return a different error,

The issue had no activity for 30 days, mark with Stale label.

Closed as stale. Please create a new issue if it's still relevant to the maintained versions.


Multiple progress bars

Both for heatmap and videos, multiple progress bars are displayed. There should be just one progress bar.

Sure. Assigning it to you


Added User Story Section to Product_Management.md

Overview of user stories and the role that they play in product management. This includes descriptions of each of the major sections of a user story and insight into how they are created and maintained. Additionally, examples and templates are included throughout the document to demonstrate how to write effective user stories.

Thank you for the feedback. Yes, as you mentioned, user stories would typically be created by the product manager (or product owner), and so I thought it fit best in this document. Also, my original idea to work on this topic came from the product management lecture slides, which included a subtopic on user stories.
However, seeing what has now been merged into the Product Management document, I agree that this topic would be out of place. I hadn't considered the 'Development Processes' document but think that would work well. I also think that my section would work well if it were linked to the 'Software Engineering' document. It would probably fit best under the Scrum category since user stories are generally associated with that framework.

Changed file location and resolved merge conflicts

Great job, all feedback has been incorporated well.


Implemented chages to the message.java file

This is a new PR for the implemented changes to the message.java file. Please ignore the index.html part; I never knew it will still appear in this new PR.

Nice work, Stanley!
What happens when you run mvn appengine:devserver with this version? Do you see any errors? We want to make sure that the version of the code in master always compiles and runs correctly, so it's good to check this.
It looks like the change to index.html is from your "Just testing" commit, right? If you want to undo that change, you can use git revert:

First find the commit hash, using git log and looking for the long string after "commit".  That will make a new commit that undoes the original change.

Finally, run git push so we can see the undo here in GitHub.

That's a lot of work to undo just a couple of lines, but it's a handy tool to know (and it will get rid of the "This branch has conflicts that must be resolved" warning that won't let you merge). Here's a GitHub blog post that has a lot more examples of ways to undo all kinds of different things: https://github.blog/2015-06-08-how-to-undo-almost-anything-with-git/
Great work sticking with this and learning more about Git! :smile:

@kyrabusser Thank you for your continued guidance! Yes I tested the code locally with ### mvn appengine:devserver and it runs correctly according to the steps 1 - 4 of the familiarization project. The existing functionality which includes sending messages and looking at the feed works correctly.
Some new functionality we should notice when we run this code include the addition of a query parameter to the POST requests that are sent when a user submits a message. But the behavior remains basically the same as it was originally coded.

@kyrabusser @CharlesArday @hengelman Please can someone review this pull request and merge with the master?
I'm working to finish the remaining steps of my familiriazation project. @CharlesArday Please notify me when working on showMessageFormIfViewingSelf() so we don't break each other's code.
Thanks everyone!


Install metalb in host cluster

Signed-off-by: Zhonghu Xu<EMAIL_ADDRESS>What type of PR is this?

What this PR does / why we need it:
Install metalb to host cluster as well, this is pre step to install argocd
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

I have tested it.


Changes Xamarin Android package name

With this change, it will now be possible to use the native and the
Forms versions of the sample viewer side-by-side on Android without errors. This brings the experience up to par with that of iOS.

Hi @mbranscomb, GitHub suggested I ask you to review this. Feel free to unassign yourself if you don't have the time.
This change was made because without it, it is challenging to work with both the Xamarin.Android and the Xamarin.Forms Android solutions simultaneously. Attempting to start one of the viewers while the other is installed would result in a TypeLoadException because the package names conflicted.


Export TextBuffer for external package usage

@kevinsawicki @bolinfest

@kevinsawicki Is there any reason why this wasn't exported before?

Is there any reason why this wasn't exported before?

Probably because no packages ever needed it. They always accessed the buffer from the editor and so they never needed to create one directly.

OK, cool, just making sure there wasn't any trickiness like there is for Directory.

So are you planning on creating the text buffer manually and passing into a manually created text editor?

Yep, that's our existing case but we require the original text-buffer it in a tricky way.
We will extend TextBuffer's setPath function (https://github.com/atom/text-buffer/blob/master/src/text-buffer.coffee#L326) to create the remote file instance we have instead of it being created using new File() which is pathwatcher's version of the File
Also, after this, I'll do another PR to the text-buffer that makes the existing setPath call a method: createFile(path) - where we can simply extend and provide our version of the file instead of having to copy the contents of the setPath function.
And this way, we don't require the added setFile function to the API introduced in https://github.com/atom/text-buffer/pull/42 - then we don't even need this PR.

Merging this sounds good to me then, @nathansobo/@benogle any objections?

@kevinsawicki could we also have this with a new atom version release?

PING

No objections, though we may be replacing TextBuffer with a new document model some time in the next few months. We can make it API-compatible however and we're already exposing instances anyway.


Invalid access inside LinInterpolation

I have merged the integration branch within master, hoping that all the code will reside within one branch. With the latest code, the Python code crashes.
I tried debugging the issue locally on my Mac and this is the stack trace that I get:
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=EXC_I386_GPFLT)
    frame #0: 0x0000000106c496b9 pyhellweg.so`LinInterpolation(x=0.0027446617833126615, X=0x000000010233c4b0, Nx=10, A=0xf0000000102301c5) at Functions.h:561
  * frame #1: 0x0000000106c48e5d pyhellweg.so`TBeam::Integrate(this=0x000000010233c3c0, Par=0x00000001068acd60, I=0x00000001007ad620, Si=0) at Beam.cpp:1673
    frame #2: 0x0000000106c6b950 pyhellweg.so`TBeamSolver::Integrate(this=0x00000001068aca30, Si=0, Sj=0) at BeamSolver.cpp:4039
    frame #3: 0x0000000106c70605 pyhellweg.so`TBeamSolver::Step(this=0x00000001068aca30, Si=0) at BeamSolver.cpp:4552
    frame #4: 0x0000000106c707c6 pyhellweg.so`TBeamSolver::Solve(this=0x00000001068aca30) at BeamSolver.cpp:4593
    frame #5: 0x0000000106c999e4 pyhellweg.so`HellwegBeamSolver::solve(this=0x000000010078f430) at libHellweg2D.cpp:115
    frame #6: 0x0000000106ca05a1 pyhellweg.so`__pyx_pf_9pyhellweg_19PyHellwegBeamSolver_2solve(__pyx_v_self=0x00000001066c9408) at pyhellweg.cpp:1791
    frame #7: 0x0000000106ca0464 pyhellweg.so`__pyx_pw_9pyhellweg_19PyHellwegBeamSolver_3solve(__pyx_v_self=0x00000001066c9408, unused=0x0000000000000000) at pyhellweg.cpp:1772

The previous error happens for 63e52d006a10eb820cb85f7271f8637fa55f697c.

At Beam.cpp:1673:
Hx=LinInterpolation(r*lmb,Par.Hext.Piv.X,Par.Hext.Dim.Nx,Par.Hext.Field[0]);

The problem seems to be the access of Par.Hext.Field[0], which is a TField**. Somehow it seems that the second pointer is not properly set and we are getting a segmentation fault due to reading a faulty memory address.

I think I solved it. Please, check. The Par.Hext.Field array was deleted before the function call.

@skutsaev The build is still failing: https://travis-ci.org/radiasoft/rslinac/builds/231993099
Every checkin to master results in a build on travis-ci.org so you can check the results here:
https://travis-ci.org/radiasoft/rslinac
If you have time, it would be good to get rid of the compiler warnings. Thanks.


Clearly Scope and Specify What SDK, NDK and API Versions Are Supported by Makefile-Android

At first glance, a developer might be led to believe that Makefile-Android supports any arbitrary version of NDK, SDK, and API.
However, such is not the case. Makefile-Android and/or supporting documentation should clearly specify what NDK, SDK, and API versions are supported.

/cc/ @robszewczyk

With checkout c1e02de826b17fe95cca336dc5a57f3e1847ed87, I ran a somewhat old version of the NDK with:
% make -f Makefile-Android ANDROID_HOME=/usr/local/android/android-sdk-macosx ANDROID_NDK_HOME=/usr/local/android/android-ndk-r10 JAVA_HOME=/usr ANDROID_API=13

which fails with some C++11 issues.

With checkout 50e179b1ed778287a4680b3ca12ad9dcbd09754f, and NDK r12b, I was able to get a little farther with:
% make -f Makefile-Android ANDROID_HOME=/usr/local/android/android-sdk-macosx ANDROID_NDK_HOME=/usr/local/android/android-ndk-r12b JAVA_HOME=/usr

where I then run into an issue about :
  CXX      ../../src/inet/libInetLayer_a-InetFaultInjection.o
In file included from /Users/gerickson/Source/github.com/openweave/openweave-core/src/inet/../../src/inet/InetLayer.cpp:69:0:
/Users/gerickson/Source/github.com/openweave/openweave-core/third_party/android/platform-libcore/android-platform-libcore/luni/src/main/native/ifaddrs-android.h:21:19: fatal error: cstring: No such file or directory
 #include <cstring>
                   ^
compilation terminated.
make[4]: *** [Makefile:895: ../../src/inet/libInetLayer_a-InetLayer.o] Error 1
make[4]: *** Waiting for unfinished jobs….

With checkout 50e179b, and NDK r16b, I was able to get a little farther with:
% make -f Makefile-Android ANDROID_HOME=/usr/local/android/android-sdk-macosx ANDROID_NDK_HOME=/usr/local/android/android-ndk-r12b JAVA_HOME=/usr

where I then run into an issue Java, which is more about macOS than Android:
Making all in device-manager/java
  CXX      libWeaveDeviceManager_la-WeaveDeviceManager-JNI.lo
  CXX      libWeaveDeviceManager_la-PairingCodeUtils-JNI.lo
  CXX      libWeaveDeviceManager_la-AndroidBleApplicationDelegate.lo
  CXX      libWeaveDeviceManager_la-AndroidBlePlatformDelegate.lo
  MKDIR         bin
  JAVAC
Unable to find a $JAVA_HOME at "/usr", continuing with system-provided Java...
warning: [options] bootstrap class path not set in conjunction with -source 6
warning: [options] source value 6 is obsolete and will be removed in a future release
warning: [options] target value 1.6 is obsolete and will be removed in a future release
warning: [options] To suppress warnings about obsolete options, use -Xlint:-options.

So, it would seem NDKr16b is good.


Add unit tests into build validation

Adding the unit tests into the build validation.
I could not find the unit tests, so added them. @tamirkamara - it looks like the upload/publish of tests results were in the DockerBuild workflow (thanks, didn't spot that), but the tests were never actually run. So I've cleaned this up

IIRC, the docker files include running the tests, so how is this different than the workflow that build images?

Adding the unit tests into the build validation.
I could not find the unit tests, so added them. @tamirkamara - it looks like the upload/publish of tests results were in the DockerBuild workflow (thanks, didn't spot that), but the tests were never actually run. So I've cleaned this up

@ross-p-smith I'm used to always running rests as part of the docker build process so if something is wrong there it means that an image could be built inspite of failing tests.
I'd prefer to see that working there rather than externalizing it. But that my opinion...

Adding the unit tests into the build validation.
I could not find the unit tests, so added them. @tamirkamara - it looks like the upload/publish of tests results were in the DockerBuild workflow (thanks, didn't spot that), but the tests were never actually run. So I've cleaned this up

@ross-p-smith I'm used to always running rests as part of the docker build process so if something is wrong there it means that an image could be built inspite of failing tests. I'd prefer to see that working there rather than externalizing it. But that my opinion...

Just saw you comment @tamirkamara - I'll move the tests back into the Docker build as understand now - the api app is containerised there


Fixes issue #175 retrying on Scan operation on error

Fixes scan never getting started when retrying on failed scan operation

Sure @uKL . I will add it.

Thanks a lot. Great job addressing this bug! :) 🥇

@uKL My pleasure! Thanks for this wonderful library :)
Added a couple of tests. Let me know if I should change anything.

Got it. I will make the necessary changes.

@dariuszseweryn Will this approach be fine?
    given:
    TestSubscriber testSubscriber = new TestSubscriber<>()
    bleAdapterWrapperSpy.hasBluetoothAdapter() >> true
    bleAdapterWrapperSpy.isBluetoothEnabled() >> true

    when:
    def scanObservable = objectUnderTest.scanBleDevices(null)

    then:
    0 * bleAdapterWrapperSpy.isBluetoothEnabled()

    when:
    scanObservable.subscribe(testSubscriber)

    then:
    1 * bleAdapterWrapperSpy.startLeScan(_) >> true

Kindly check and let me know if anything is missing.

@BharathMG Thank you for your support :) 👍

Dear Contributor,
similar to many open source projects we kindly request to sign our CLA if you'd like to contribute to our project. This license is for your protection as a Contributor as well as the protection of Polidea; it does not change your rights to use your own Contributions for any other purpose.
You can find a link here: https://cla-assistant.io/Polidea/RxAndroidBle


chore: Add missing ScreenshotArea

Description
Add missing ScreenshotArea wrapper for a token group screenshot testing page.
Related links, issue #, if available: n/a
How has this been tested?

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

Changes include appropriate documentation updates.
Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
Changes do not include unsupported browser features, see CONTRIBUTING.md.
Changes were manually tested for accessibility, see accessibility guidelines.

Security

If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

Changes are covered with new/existing unit tests?
Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Actually I think the best solution would have been to have the token group without top padding in a different page. Right now the part corresponding to an existing integration test as well as an h2 title are being captured for the visual test, and any change to those will interfere with the visual test of the token group without top padding.

Or include the token group without top padding to the permutations.


Add a maximum download speed option

At the moment, the process saturates all of the bandwidth available, which isn’t very nice if you aren’t alone in the house. It would be great of there was an option to set a max speed — nothing fancy like rclone’s schedule, but at least a target value like --max-download-speed 1M

@ZinRicky Hey, probably a little late for you now, but you can effectively do that with the options already present.
Giving the script -p 1 will force it to download one album at a time, and then you can do --wait-after-download X to make it wait X seconds between each album download.
It isn't quite a 'max speed' option, which I'm not sure I really have any way to enforce, but it will effectively lower your throughput to something more reasonable for sharing your connection.

Thank you for the reply. I had already tried the -p 1 option and played with the wait setting, but I was really looking into a throttle feature like the one in wget.
I know this request isn’t easy at all, but I was hoping there would be a chance

@ZinRicky I know it's been a while, but we just switched to using a Curl backend for downloads, and with that I believe that we should be able to semi-support a max speed setting. Is that something you (or anyone who sees this) is still interested in?

I’m interested, yes


replace tap with native test runner

I have made my first contribution to an open source project by replacing the tap test runner with the native Node.js 18.x test runner, and removing the unnecessary cross-env dependency.

@all-contributors please add @jochri3 for test


Fix grammar error in lesson 00

Hi @mrdbourke , minor grammar error in https://www.learnpytorch.io/00_pytorch_fundamentals/#reshaping-stacking-squeezing-and-unsqueezing.
It says:
These methods help you make the right elements of your tensors are mixing with the right elements of other tensors

I think it should be:
These methods help you make sure the right elements of your tensors are mixing with the right elements of other tensors

Thanks

I agree! Thank you for the fix!


Feat: Dynamic enums for OSC

Implementation

Added dynamic enum generation based on OSC field
Bug: Fixed array-based transformation while comparing the order of the key

Need to be implemented in future -

Title in place of folder path in enum
Addition of autocomplete dropdown in future

I think for our use case first option is a good idea to implement and making it more generic for any file path and properties.
AbstractEditor I think won't be possible to be implemented on runtime 🤔


Add beadm to the node installation script and document it's use

beadm is a great little tool to manage boot environments. Sometimes it just happens that you update your FreeBSD installation, and it breaks something. That's where beadm comes in: it can mark a certain boot environment active (even over SSH), then you simply reboot the machine and you are back online to a working environment.

The documentation has been added here, on the main docs site:
https://docs.hoster-core.gateway-it.com/2._Backups/2._beadm/


Drop language keys that are not able to be translated

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
Other
🔖 Feature description
Such as Telegram, Discord.
✔ Solution
Drop them.
❓ Alternatives
No response
📝 Additional Context
No response

Fixed by #2812


LucasDotVin\Soulbscription\Enums\PeriodicityType::getDateDifference(): Argument #3 ($unit) must be of type string, null given, called in /home/schneider/Code/tonote/vendor/lucasdotvin/laravel-soulbscription/src/Models/Concerns/HandlesRecurrence.php on line 20

LucasDotVin\Soulbscription\Enums\PeriodicityType::getDateDifference(): Argument #3 ($unit) must be of type string, null given, called in /home/schneider/Code/tonote/vendor/lucasdotvin/laravel-soulbscription/src/Models/Concerns/HandlesRecurrence.php on line 20

Hey! Thanks for reporting this bug! Could you provide some details about? I'm gonna work to publish a release today to fix it if possible. 🤘

thank you when i try to
$subscriber->consume('Number of Users', 1);
it returns the error

Oh, so it is thrown when you try to consume a feature... Could you send me some screenshots of how your plans are configured? 😅

@lucasdotvin

$basic = Plan::create([
'name'                      => 'Basic',
'role'                      => 'User',
'description'               => 'Basic',
'type'                      => 'Subscription',
'amount'                    => 0,
'periodicity'               => 1,
'periodicity_type'          => PeriodicityType::Month,
]);
    $pro = Plan::create([
        'name'                      => 'Pro',
        'role'                      => 'User',
        'description'               => 'Pro',
        'type'                      => 'Subscription',
        'amount'                    => 9900,
        'periodicity'               => 1,
        'periodicity_type'          => PeriodicityType::Month,
    ]);

    $business = Plan::create([
        'name'                      => 'Business',
        'role'                      => 'User',
        'description'               => 'Business',
        'type'                      => 'Subscription',
        'amount'                    => 19900,
        'periodicity'               => 1,
        'periodicity_type'          => PeriodicityType::Month,
    ]);

$usersFeature = Feature::where('name', 'Number of Users')->first();
$envelopsFeature = Feature::where('name', 'Number of Envelops')->first();
$trailFeature = Feature::where('name', 'Access to audit trail')->first();
$sealFeature = Feature::where('name', 'Digitise, Signature, Stamp and Seal')->first();
$brandingFeature = Feature::where('name', 'Personal Branding')->first();
$smsFeature = Feature::where('name', 'SMS Notifications')->first();
    $basic->features()->attach($usersFeature, ['charges' => 1]);
    $pro->features()->attach($usersFeature, ['charges' => 3]);
    $business->features()->attach($usersFeature, ['charges' => 10]);

    $basic->features()->attach($envelopsFeature, ['charges' => 10]);
    $pro->features()->attach($envelopsFeature, ['charges' => 50]);
    $business->features()->attach($envelopsFeature, ['charges' => 100]);

    $basic->features()->attach($trailFeature);
    $pro->features()->attach($trailFeature);
    $business->features()->attach($trailFeature);

    $basic->features()->attach($sealFeature);
    $pro->features()->attach($sealFeature);
    $business->features()->attach($sealFeature);

    $business->features()->attach($brandingFeature);
    $business->features()->attach($smsFeature);

Hi! I tried to replicate the bug locally, but everything ran as expected. Could you send me a screenshot of the seeder in which you registered the features? 😅

Hi @lucasdotvin ,
Kindly check this scenario
In my seeder
$smsBalance = Feature::create(['consumable' => true, 'quota' => true, 'name' => 'sms-balance']);
$smsPlan = Plan::create([ 'name' => 'sms-plan', 'periodicity_type' => PeriodicityType::Month, 'periodicity' => 1]);
$smsPlan->features()->attach($smsBalance, ['charges' => 1]);

then after checkout I'm subscribing the user and give to him some tickets
$subscriber->subscribeTo($plan);
$subscriber->giveTicketFor('sms-balance', today()->addMonth(), $event->data['plan']['qty']);

then
>>> $u->hasFeature('sms-balance');
=> true

However the other remaining functions returns
>>> $u->canConsume('sms-balance', 1);
=> TypeError: App\Models\User::getSubscriptionChargesForAFeature(): Return value must be of type float, null returned

Hi! I tried to replicate the bug locally, but everything ran as expected. Could you send me a screenshot of the seeder in which you registered the features? 😅

Hi @lucasdotvin it occurs if you try to consume a feature whose periodicity_type is null

 $basic = Plan::create([
            "name"             => "basic",
            'periodicity_type' => null,
            'periodicity'      => null,
        ]);

        $monthly = Plan::create([
            'name'             => 'monthly',
            'periodicity_type' => PeriodicityType::Month,
            'periodicity'      => 1,
        ]);

In my case, both plan has same problem as the title even with the date type


Remove unnecessary engines key.

Node 4 is still LTS for a little while longer and this isn't needed.

Basically anything that wants to support serving a wasm file needs to call define() or upgrade node.

Switched engine requirement to >= 4.0.0


Possible issue with #requires in secretmanagement.keepass.extension.psm1

I'm unable to register a vault using the module secretmanagement.keepass.extension, it claims that the PoshKeepass module is not loaded.
If I explicitly import the module with verbose output, I can see that PoshKeepass is indeed imported but is apparently not accepted by the #requires statement in the psm1-file. If I comment out the #requires line and add 'Import-Module -Name "$pscriptroot..\poshkeepass\poshkeepass.psd1" -Force', everything works as expected. Tested in WindowsPowershell 5.1 and Powershell 7 on Windows

@salbeck thanks I'll take a look. I use an internal version of PoshKeePass because I had to modify it to make it work with Secret Management, I may just throw it out and go straight to the native DLLs directly :)

Fixed in d211331054bb3e66f9a3360edcb517dc347e9f38 and pushed as <IP_ADDRESS>


cannot create Argocd Cluster: rpc error: code = PermissionDenied

Maybe someone can help me to get provider-argocd running, please.
I followed the tutorial in the readme and always end up in a PermissionDenied error.
What happened?
Each provider-argo resource I try to install is not syncing or creating argocd clusters or projects.
For example applying the cluster:
kubectl apply -f https://raw.githubusercontent.com/crossplane-contrib/provider-argocd/main/examples/cluster/cluster.yaml
results in:
kubectl describe clusters.cluster.argocd.crossplane.io example-cluster

 Message:               create failed: cannot create Argocd Cluster: rpc error: code = PermissionDenied desc = permission denied: clusters, create, https://kubernetes.default.svc, sub: provider-argocd, iat: 2022-01-31T00:31:22Z
    Reason:                ReconcileError
    Status:                False
    Type:                  Synced
Events:
  Type     Reason                        Age                From             Message
  ----     ------                        ----               ----             -------
  Warning  CannotCreateExternalResource  1s (x12 over 12s)  managed/cluster  cannot create Argocd Cluster: rpc error: code = PermissionDenied desc = permission denied: clusters, create, https://kubernetes.default.svc, sub: provider-argocd, iat: 2022-01-31T00:31:22Z

How can we reproduce it?
Simply follow all steps in the readme. Only difference I did was using minikube instad of kind
Sidenotes:

I also found out, that the ProviderConfig in the /examples/provider differs slightly (in the Readme spec.plainText: false not mentioned.

How to install crossplane is not declared in the docs, yes it's obvious, but would be nice when following the steps

I installed it by helm upgrade --install crossplane crossplane-stable/crossplane --namespace crossplane-system --create-namespace --wait

Installing the provider-argocd is not mentioned in the docs, I know this is also obvious, but is also breaking the installation flow slightly.

I installed the provider-argo via manifest, like mentioned in one of the issues: (https://github.com/crossplane-contrib/provider-argocd/issues/17#issuecomment-1021110129)

I created my ArgoCD user by patching the ArgoCD configMap:
kubectl patch configmap/argocd-cm -n argocd --type merge -p '{"data":{"accounts.provider-argocd":"apiKey, login"}}'

What environment did it happen in?
Crossplane version: helm.sh/chart=crossplane-1.6.1
Crossplane Provider argocd version:
apiVersion: pkg.crossplane.io/v1
kind: Provider
metadata:
  name: argocd-provider
  annotations:
    argocd.argoproj.io/sync-wave: "-46"
spec:
  package: "crossplane/provider-argocd:v0.1.0"

I hope I didn't forgot anything.
It would be awesome to finish the installation process successfully.
Thank you in advance

Ok I found out how solve that puzzle:
ArgoCD does not allow creating resources, because the user I created is not allowed. In the Argo docs say, I can create Argo specific RBAC rules, so I had to add them in the argocd-rbac-cm
apiVersion: v1
kind: ConfigMap
metadata:
  name: argocd-rbac-cm
  namespace: argocd
  labels:
    app.kubernetes.io/name: argocd-rbac-cm
    app.kubernetes.io/part-of: argocd
data:
  policy.default: "role:admin" # or something like below
  # policy.csv: |
  #   p, role:provider, applications, *, */*, allow
  #   p, role:provider, clusters, *, *, allow
  #   p, role:provider, projects, *, *, allow
  #   p, role:provider, repositories, *, *, allow
  #   g, provider-argocd, role:provider

I can also patch it:
kubectl patch configmap/argocd-rbac-cm -n argocd --type merge -p '{"data":{"policy.default":"role:admin"}}'
All fixed.
But it would still be nice adding these steps to the docs:
installing Crossplane:
helm upgrade --install crossplane crossplane-stable/crossplane --namespace crossplane-system --create-namespace --wait
creating user:
`kubectl patch configmap/argocd-cm -n argocd --type merge -p '{"data":{"accounts.provider-argocd":"apiKey, login"}}'
allow doing everything (for testing):
kubectl patch configmap/argocd-rbac-cm -n argocd --type merge -p '{"data":{"policy.default":"role:admin"}}'
I propose a pull request


Update README.md

Add qgeminiserver to the "Programming" section.

Will send another PR later on.


Is paging possible or partial load data ?

Hello Paul,
Is paging possible ? Please suggest the version no.

It will be in the future. You can subscribe us:
https://www.patreon.com/bePatron?u=17038338&redirect_uri=https%3A%2F%2Fbossanova.uk%2Fjexcel%2Fexamples%2Fincluding-formulas-on-your-spreadsheet&utm_medium=widget
To help improve our library. New nice features in the future if we can get enough support...


compile both VS2015 and VS2017

i have install both VS2015 and VS2017, but after i do :
vcpkg.exe install jsoncpp
it only compile vs2017, how to compile vs2015 also?

Vs2017 (and 2019) are abi compatible with 2015, so it should be not necessary. Am I missing something?

Hi @wjx0912, you can use Visual Studio 2015 using this set:
set(VCPKG_PLATFORM_TOOLSET v140)

Just add it into your triplet file in VCPKG_PATH/triplets/.
Thanks.

hi, @cenit
thanks for your reply, i'll test the compatible  of 15/17/19.


locale.conf bugfix in base.Dockerfile

The file /etc/locale.conf contained incorrect data.
With this minor change the errors in login shells such as tmux should be fixed. As this issue is a couple months old, I thought I'd pitch in.
Feedback is appreciated :)

Thanks for the PR @jeg-b


Add view modifier to feeState interface in IPerformanceFeeCalculator.

Pull Request Test Coverage Report for Build 7645

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

Totals

Change from base Build 7636:
0.0%

Covered Lines:
1710

Relevant Lines:
1710

💛  - Coveralls


How to adapt Reinvent 3.2 Reinforcement learning example to Reinvent 4?

Hi, you have a sample notebook here
https://github.com/MolecularAI/ReinventCommunity/blob/master/notebooks/Reinforcement_Learning_Demo_DockStream.ipynb
How can I adapt it to reinvent 4? I tried to call resulting json config using
reinvent -f json RL_DockStream_config.json

but it throws me an error module 'reinvent.runmodes' has no attribute 'run_reinforcement_learning'

The notebooks are outdate regards config file generation (but not in their general applicability).  You would need to use the new format which is documentend in config/ in the repository.  I would recommend to use the TOML path.


Fixed typos in docs

What changes were proposed in this pull request?
Typos at a couple of place in the docs.
How was this patch tested?
build including docs
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

We typically ask anyone who proposes a tiny typo fix PR to please have a look at the rest of the doc and related docs while they're at it, to try to cut down on the number we process. The change is fine though.

Yes, this is the consolidated list across all the docs. I don't think there are any more typos in the .md files under the docs folder.

Thanks - merging in master/branch-2.2.


Flapping connections to cluster agent

What kind of request is this (question/bug/enhancement/feature request):
bug?
Steps to reproduce (least amount of steps as possible):
No reproduction available, only happens to one of the clusters managed by our central Rancher Management Server:
Every 30 seconds, the connection to the cluster is interrupted and the following lines are logged in Rancher Management Server:
2019/07/05 07:53:24 [INFO] Stopping cluster agent for c-sz8ln
E0705 07:53:24.368513       6 request.go:853] Unexpected error when reading response body: &http.httpError{err:"net/http: request canceled (Client.Timeout exceeded while reading body)", timeout:true}
E0705 07:53:24.368662       6 reflector.go:134] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v1.ConfigMap: Unexpected error &http.httpError{err:"net/http: request canceled (Client.Timeout exceeded while reading body)", timeout:true} when reading response body. Please retry.
2019/07/05 07:53:24 [ERROR] ClusterController c-wtb9x [user-controllers-controller] failed with : failed to start user controllers for cluster c-sz8ln: timeout syncing controllers
2019/07/05 07:53:24 [INFO] Registering project network policy
2019/07/05 07:53:24 [INFO] registering podsecuritypolicy cluster handler for cluster c-sz8ln
2019/07/05 07:53:24 [INFO] registering podsecuritypolicy project handler for cluster c-sz8ln
2019/07/05 07:53:24 [INFO] registering podsecuritypolicy namespace handler for cluster c-sz8ln
2019/07/05 07:53:24 [INFO] registering podsecuritypolicy serviceaccount handler for cluster c-sz8ln
2019/07/05 07:53:24 [INFO] registering podsecuritypolicy template handler for cluster c-sz8ln
2019/07/05 07:53:24 [INFO] Registering monitoring for cluster "c-sz8ln"
2019/07/05 07:53:25 [INFO] Starting cluster controllers for c-sz8ln
W0705 07:53:25.075028       6 reflector.go:270] github.com/rancher/norman/controller/generic_controller.go:175: watch of *v3.ClusterLogging ended with: too old resource version: 50009123 (50376441)
[...]

Result:
The connection to the cluster is interrupted and then restarted. On the Rancher webinterface the message This cluster is currently Unavailable; areas that interact directly with it will not be available until the API is ready. is displayed.
Other details that may be helpful:

The rancher cluster agent and the node agents do not log anything when this reconnect happens.
In the browser console I see:

Socket connecting (id=1, url=wss://rancher.example.com/v3/subscribe...)
Socket opened (state=connecting, id=1)
Cluster Subscribe connected (sockId=1)
Socket connecting (id=2, url=wss://rancher.example.com/v3/clusters/c-sz8ln/subscribe...)
Socket opened (state=connecting, id=2)
 Subscribe connected (sockId=2)
 Subscribe disconnect (sockId=2)
Socket closing (state=connected, id=2)
Socket 2 closed
 Subscribe disconnected (sockId=null)

Probably related, probably not: Memory usage of the Rancher Management Server goes up and up until it gets OOM-killed and restarted.
This connection issue only happens with one cluster, the Rancher Management Server manages a lot more clusters without any issues.
The managed cluster with this issue is behind a firewall with NAT.

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.2.4
Installation option (single install/HA): HA, exactly as described here.

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom (installed with rke on Ubuntu 18.04)
Machine type (cloud/VM/metal) and specifications (CPU/memory): VM, 32 GB RAM, 8 vCPU
Kubernetes version (use kubectl version):

Server Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.5", GitCommit:"2166946f41b36dea2c4626f90a77706f426cdea2", GitTreeState:"clean", BuildDate:"2019-03-25T15:19:22Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client:
 Version:           18.09.7
 API version:       1.39
 Go version:        go1.10.8
 Git commit:        2d0083d
 Built:             Thu Jun 27 17:56:23 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          18.09.7
  API version:      1.39 (minimum version 1.12)
  Go version:       go1.10.8
  Git commit:       2d0083d
  Built:            Thu Jun 27 17:23:02 2019
  OS/Arch:          linux/amd64
  Experimental:     false

@tobru Hi, have you resolved this problem? We have this problem in latest 2.3.3 version, with an interval of about 30~40 seconds.

Any update on this? I'm seeing similar behavior. Notably it happens because of giant collections of configmaps and secrets in the particular cluster. If it takes more than 30 seconds to read the body of the configmap return it fails. I would love to know how to increase that read timeout value.


Algebraic errors in AST generation

Right now, errors are thrown as strings within AstGen, which is not desirable because we ideally want errors to be a part of the API, and for other parts of the compiler to be able to switch on errors. Instead, errors should be represented as an enum with data members, which should be convertible to reporting errors.

Ill take this


Fix coverage.yml

Motivation
Coverage.yml is broken. Fix it.
Description of the changes

Delete unnecessary spaces.
Fix indent level.

@HideakiImamura Could you review this PR?

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@48904d4). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage        ?   73.00%
=======================================
  Files           ?        2
  Lines           ?      226
  Branches        ?        0
=======================================
  Hits            ?      165
  Misses          ?       61
  Partials        ?        0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Handler for tag 1 does not exist

Hi,
I got this error using PanGestureHandler.
My code looks like:
<PanGestureHandler enabled maxPointers={1} onGestureEvent={this.handleEvent}>
     <Animated.View { ...someStuff }>
     </Animated.View>
</PanGestureHandler>

Am I missing something ?

Hi, @foyarash
Your code seems to be fine (however enabled is not required, it's enabled by default, but it's not a matter of this issue). Would you mind giving me some snippet for reproduce this issue?

@osdnk In my case, I have a FlatList on which each item has a PanGestureHandler.
The problem seems to occur when I have my item component in a different file from the FlatList.
The following works correctly: https://pastebin.com/XVX1bnX4
But having the Item component in a separate file breaks, and the error appears

@foyarash
https://github.com/osdnk/flat-list-react-native-gesture-handler-sample
I made a project to see your issue and still cannot reproduce it

This project works only for iOS. Do you face this issue on both platforms?

Tested your project on both iOS and android.
android does not work at all, but doesnt display any red box
iOS works when the Text tag has a style

Android does not work, because I didn't configure it 😃 .
Well, it's quite interesting issue on iOS. I'll take a look

@osdnk another thing i noticed is that if i set a margin <= 5 on the Text, it crashes

I have the same issue (on Android). I have investigated this problem and found, that createGestureHandler method have invoked after attachGestureHandler method. Other words, library try to attach gesture handler that doesn't exist yet.
It is happening because of "setImmediate" method in componentDidMount.
So when app started, componentDidMount was invoked, and then componentDidUpdate was invoked immediately (because of redux action in componentDidMount). When i insert console.log into GestureHandler.js file, i see the following sequence of methods invocation:

componentDidMount
componentDidUpdate and attaching of gesture handler (that doesn't exist yet) to view
setImmediate inside componentDidMount, creation of gesture handler

When i remove redux actions from componentDidMount, i haven't error

@Saranchenkov thank you for pointing this out, i was able to fix my issue thanks to this !
My issue was that i launched an animation on componentDidMount, i just removed this animation and it correctly works now

@osdnk  Thank you for being so reactive

@foyarash thanks for reporting and responding so quickly 👏 . Just published 1.0.1 that includes this fix: https://github.com/kmagiera/react-native-gesture-handler/releases/tag/1.0.1
Let us know if it fixes your problem so that we can close the issue.

@kmagiera can confirm it works, thank you!


Add "ABC Alchemy" plugin.

This plugin includes 2 major features:

An overlay that highlights the intersection between the alchemy spell and the optimal inventory slot.
A sidebar panel that shows the most profitable items to alch. All prices use RuneLite APIs.

Your panel icon is too large. Although it is resized visually for the sidebar button, it takes up memory per pixel when read by Java. Please scale it down.
Code LGTM otherwise

Done! thanks for taking the time to review :)

You resized the wrong icon. You resized assets/icon.png which isn't packaged with your plugin. src/main/resources/icon.png, which is actually shipped with your code, remains too large.

you are correct, sorry.


Issue with multiple tables after upgrade to 2.0.2

Just upgraded from 1.6.1 to 2.0.2.      I then connect them to various templates throughout the app.
If I load the app at a template containing a {{>tabular}}, that table works fine.  If I then go to a different template with a {{> tabular}}, I get errors.  (Going back to the original one with the back button then continues to generate the same errors).  I can go to any template and do a refresh, and that one will load successfully, with all future ones failing.
I also can trigger the errors if I reactively change the Tabular.Table passed to the {{> tabular }}
The only changes I made to the app since upgrading are:

changed import { Tabular } statements to import Tabular
added the code to load the bootstrap theme (I have confirmed that removing this code does not impact the issue)
npm installed jquery and bootstrap theme per instructions on readme

Ideas?
The errors:
at HTMLTableElement.<anonymous> (http://localhost:3000/packages/aldeed_tabular.js?hash=6066b0a17709cadd128432da01678e42222fe418:1864:38)
    at Function.each (http://localhost:3000/packages/modules.js?hash=3eb8274840bbd327db988c16fa3da945e0d0f776:33851:23)
    at jQuery.fn.init.each (http://localhost:3000/packages/modules.js?hash=3eb8274840bbd327db988c16fa3da945e0d0f776:33603:17)
    at jQuery.fn.init.DataTable [as dataTable] (http://localhost:3000/packages/aldeed_tabular.js?hash=6066b0a17709cadd128432da01678e42222fe418:1821:8)
    at jQuery.fn.init.$.fn.DataTable (http://localhost:3000/packages/aldeed_tabular.js?hash=6066b0a17709cadd128432da01678e42222fe418:16056:18)
    at Blaze.View.<anonymous> (http://localhost:3000/packages/aldeed_tabular.js?hash=6066b0a17709cadd128432da01678e42222fe418:406:27)
    at http://localhost:3000/packages/blaze.js?hash=ef41aed769a8945fc99ac4954e8c9ec157a88cea:1875:20
    at Function.Template._withTemplateInstanceFunc (http://localhost:3000/packages/blaze.js?hash=ef41aed769a8945fc99ac4954e8c9ec157a88cea:3687:12)
    at http://localhost:3000/packages/blaze.js?hash=ef41aed769a8945fc99ac4954e8c9ec157a88cea:1873:29
    at Object.Blaze._withCurrentView (http://localhost:3000/packages/blaze.js?hash=ef41aed769a8945fc99ac4954e8c9ec157a88cea:2214:12)```

and
```Error: TypeError: Cannot read property 'aDataSort' of undefined
    at _fnSortFlatten (http://localhost:3000/packages/aldeed_tabular.js?hash=6066b0a17709cadd128432da01678e42222fe418:6806:35)
    at _fnSortingClasses (http://localhost:3000/packages/aldeed_tabular.js?hash=6066b0a17709cadd128432da01678e42222fe418:7144:14)
    at HTMLTableElement.<anonymous> (http://localhost:3000/packages/aldeed_tabular.js?hash=6066b0a17709cadd128432da01678e42222fe418:2197:4)
    at Function.each (http://localhost:3000/packages/modules.js?hash=3eb8274840bbd327db988c16fa3da945e0d0f776:33851:23)
    at jQuery.fn.init.each (http://localhost:3000/packages/modules.js?hash=3eb8274840bbd327db988c16fa3da945e0d0f776:33603:17)
    at jQuery.fn.init.DataTable [as dataTable] (http://localhost:3000/packages/aldeed_tabular.js?hash=6066b0a17709cadd128432da01678e42222fe418:1821:8)
    at jQuery.fn.init.$.fn.DataTable (http://localhost:3000/packages/aldeed_tabular.js?hash=6066b0a17709cadd128432da01678e42222fe418:16056:18)
    at Blaze.TemplateInstance.<anonymous> (http://localhost:3000/packages/aldeed_tabular.js?hash=6066b0a17709cadd128432da01678e42222fe418:521:20)
    at http://localhost:3000/packages/blaze.js?hash=ef41aed769a8945fc99ac4954e8c9ec157a88cea:3341:22
    at Function.Template._withTemplateInstanceFunc (http://localhost:3000/packages/blaze.js?hash=ef41aed769a8945fc99ac4954e8c9ec157a88cea:3687:12)```

Downgraded Tabula Tables to 1.6.1

@dovrosenberg You'll need to create a simple app that reproduces the issue and post a link. I don't know what the issue would be.

Nevermind, I ran across this eventually and I think this fixes it.

Also be sure to use a selector like this if you're grabbing the dataTable instance directly:
var dt = $('.dataTables_scrollBody table.dataTable').DataTable();

If you select table.dataTable without the scrollBody part, you could be selecting the wrong table element because there are also head and foot table elements. This can lead to the Cannot read property 'aDataSort' of undefined error.

That didn't fix it for me.  I will try to create a simple reproduction.

FYI; I have site with multiple (around 10) datatables and just updated today to latest version of this extension. All seems to work fine for me and no issues whatsoever (although I have been using different clientside selectors for each table).


Create an IRC room

Please create an IRC room for #dokku-alt. We occasionally get questions on dokku-alt specific stuff in #dokku on freenode, and we're unable to handle them as the projects have diverged quite a bit.
As an example, our zero-downtime implementations are quite different at this point, and I just had a question asked about yours. Ideally they aren't, and we can support your users, but that isn't the case :(

Definitely a good idea :+1:

Let's start with gitter.im. I'll later prepare synchronization between IRC channel and Gitter: https://gitter.im/dokku-alt/dokku-alt?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge