Sticky container qa pass

#236

@achord @CheetoMao
This PR addresses the issues you guys pointed out - both the issue with the sticking behavior being janky because of a lack of a placeholder and the responsive width issue. Thank you guys again for the suggestions.
I'll have a QA branch open in my fork of the repo. Just let me know if you guys catch anything else, and I'll patch in fixes.

Thanks!


isolate/spawn_uri_nested_vm_test timing out on dartium

FAILED: none-dartium-checked release_ia32 isolate/spawn_uri_nested_vm_test
Expected: Pass
Actual: Timeout
CommandOutput[dartium]:
stdout:
Dom could not be fetched, since the test timed out.
stderr:
This test timed out. The delay until the test actually started was: 0:00:00.039132.
BrowserOutput while running the test (* EXPERIMENTAL *):
BrowserOutput.stdout:
BrowserOutput.stderr:
[1025/155346:ERROR:nacl_helper_linux.cc(233)] NaCl helper process running without a sandbox!
Most likely you need to configure your SUID sandbox correctly

Dartium is dead! Closing.


docker run error

Thank you for the nice work!
I faced a problem in running via docker.
I followed the setup procedure
# Clone the project
git clone https://github.com/dvcrn/chatgpt-ui
cd chatgpt-ui

# Install Elixir
mix deps.get

# Install node dependencies
yarn install      # or npm install
mix assets.setup
mix assets.build

# Start the dev server
export OPENAI_API_KEY=<api-key>
export OPENAI_ORGANIZATION_KEY=<org-key>
mix phx.server

and successfully running.
but when I follow the docker procedure,
docker build . -t chatgpt
docker run -e SECRET_KEY_BASE=<keybase here> -e HOST="localhost" -p 4000:4000 chatgpt

it's stuck at
05:30:11.447 [info] Running ChatgptWeb.Endpoint with cowboy 2.9.0 at :::4000 (http)
05:30:11.447 [info] Access ChatgptWeb.Endpoint at https://localhost
05:30:11.447 [info] downloading tokenizer model: bert-base-multilingual-uncased

Is there any missing step?

if I build docker in "dev" mode, I face the error below.
=CRASH REPORT==== 27-Apr-2023::05:39:59.435250 ===
  crasher:
    initial call: application_master:init/4
    pid: <0.2061.0>
    registered_name: []
    exception exit: {{shutdown,
                      {failed_to_start_child,kernel_safe_sup,
                       {on_load_function_failed,'Elixir.Tokenizers.Native',
                        {error,
                         {load_failed,
                          "Failed to load NIF library: 'Error loading shared library ld-linux-x86-64.so.2: No such file or directory (needed by /home/elixir/app/lib/tokenizers-0.3.0/priv/native/libex_tokenizers-v0.3.0-nif-2.16-x86_64-unknown-linux-gnu.so)'"}}}}},
                     {kernel,start,[normal,[]]}}
      in function  application_master:init/4 (application_master.erl, line 142)
    ancestors: [<0.2060.0>]
    message_queue_len: 1
    messages: [{'EXIT',<0.2062.0>,normal}]
    links: [<0.2060.0>,<0.2059.0>]
    dictionary: []

Looks to be a problem with the tokenizer

05:30:11.447 [info] downloading tokenizer model: bert-base-multilingual-uncased

This is when the Tokenizer downloads the model for tokenization from huggingface, but it's strange that this doesn't work from within the container, but does from the outside
Are you running the containers on x86-64? What's the host architecture?

build and run the container on x86-64 @dvcrn


feat(Commerce Atomic): add atomic-commerce-facet

Add new atomic-commerce-facet component
Following slack discussion: Went with the decision to not implement the FacetConditionsManager.

Why do we need a feature branch? 🤔


Difference in sub-queries and group_by in 1.5 and 1.6

My query in 1.5 and 1.6 returns different results:
SELECT * FROM (select last(message), level from alerts GROUP BY host,alertName) WHERE level='CRITICAL';

on 1.5.2
name: alerts
time                alertName  host                   last                                 level
----                ---------  ----                   ----                                 -----
1528214360000000000 Disk Usage host1 /var/db used:  90.10234375% CRITICAL
1531754832000000000 Disk Usage host2 /var used:  90.00331070232626%       CRITICAL

on 1.6
name: alerts
tags: alertName=Proc Deadman, host=master5-sdm.scc.kit.edu
time                alertName    host                    last         level
----                ---------    ----                    ----         -----
1528214360000000000 Disk Usage host1 /var/db used:  90.10234375% CRITICAL

name: alerts
tags: alertName=Disk Usage, host=mchat.scc.kit.edu
time                alertName  host              last                        level
----                ---------  ----              ----                        -----
1531754832000000000 Disk Usage host2 /var used:  90.00331070232626%    CRITICAL

why is this difference?

can we get more info about this?


Not good enough search results

Below is a list of search words (with actual results) that I expected to return functions that make a string uppercase or lowercase (so the functions upper and lower), but that do not return these functions:

It is not at all obvious that one should go looking for the abbreviated forms lower and upper. Having performed the searches, and having looked through the doc page on strings, I concluded that no such functions exist, and made my own version with regex matching.
The issue then is that the existing functions are not discoverable enough, as I feel that my search for them is about the maximum amount that can be expected of new users, and I could not find them. I do now know if one would have similar problems with other specific searches, but having searched the docs a few times now, I am left with a general feeling that the hits are not great. Often I have to try multiple searches with different formulations, as opposed to what I find most obvious.
A possible solution would be to add a keywords section on the different pages to ensure that related keywords bring up the page.

The search indeed isn't great, it could benefit from keywords and a bit better matching algorithm. We'll improve it in time.
I'm a bit confused by the first two screenshots though as they do show lower and upper as the top hit. They just show the display name (which is Lowercase / Uppercase).

Oh snap, my bad! That kind of takes the wind out of the sails of this issue... I am not sure at what point I have brainfarted in my inability to find the function then.
So if the correct functions are returned, and the general issue is on the agenda, there is no point in this one staying open.
As a sidenote, I want to mention that I find the displayname and actual function name distinction to be somewhat confusing. Perhaps aliases for the same function could be another approach? So that lower(args) just calls lowercase(args), with the underlying function having long descriptive names, and some of them getting convenient aliases on top. But perhaps this does not solve the problem that the displayname/functionname distinction originally tries to solve - thoughts?

Come to think of it, I believe this is what happened:
I tried to use a non-existing function uppercase, which did not work. When it did not, I searched the docs. So the hit on some uppercase function can not possibly be what I am looking for, because no such function exists!
So this displayname/functionname distrinction is I think what caused my confusion. Should I make a new issue on that, or can someone put that point to rest immediately with some good reason why it should be that way?

The function name is often short, e.g. mat for matrices, but when you're searching the docs, Matrix is a much more helpful hit. Perhaps we could show Lowercase and lower instead of Lowercase and "Function".


Feature: Wallet Change hook

See documentation for Keplr: https://docs.keplr.app/api/#change-key-store-event
See documentation for Cosmostation: https://docs.cosmostation.io/docs/Developer Guide/Cosmostation Extension/Cosmos chains/event#vanilla-code
Would be great to have a way for me to register action hook or similar, that gets sourced by the wallet.

Hi, @TrevorJTClarke We've add event feature.
docs here. https://docs.cosmoskit.com/use-manager#events
usage example here. https://github.com/cosmology-tech/cosmos-kit/blob/main/packages/example/pages/event.tsx
The only event so far is refresh_connection. Every time registered events connectEventNamesOnWindow and connectEventNamesOnClient (usually include accounts change event in wallet extension) are triggered, refresh_connection would also be triggered.


bag it creation fails for resource with hidden chararacter in title

@Castronova discover that for this resource https://www.hydroshare.org/resource/1fedd80cd20245409a0ea02cc2a1ecf0/ downloading as a  bagit file fails . Downloading a simple zip works. I did some resource and the title does contain hidden characters:
Achieving Reproducible Computational Hydrologic Models by Integrating Scientific Cyberinfrastructures
Testing on beta by removing the characters does indeed solve the problem and the BagIt file creates correctly.
we should consider to

add additional checks to sanitize the title and other textfields
develop a management script to remove illegal characters

@martinseul This is in a group of related situations in which the code responds to ASCII but receives unicode. Similarly, Linux handles unicode in filenames, but our UI does not.

@sblack-usu can we close this as it should have been addressed with the Python 3 upgrade


SpinnerView example:   semantics of this.$variableName?

Working my way through the very helpful  HelloWorld.ipynb, I find that I do not quite understand  the meaning of "$" in these statements within SpinnerView.render:
this.$input = $('<input />');
this.$el.append(this.$input);
this.$spinner = this.$input.spinner({...

I recognize the jQuery function call shorthand:
$('<input />')

Does the other uses of $, of which three are shown just above,  all in this style
 this.$variableName

indicate that the object slot is a jQuery object?  The comments in the notebook are very complete and helpful, but this point eludes me.
Thanks!

It's fairly common practice to name variables (or object fields) that hold a jQuery element with names starting with a $. There is no programmatic meaning to the $ itself in the variable name. It's just convention.
For instance, in a DOMWidgetView, you have access to both this.el, which represents the actual DOM element the widget view is attached to, and this.$el, a jQuery element equivalent to $(this.el). Thus, both this:
this.$el.text('Hi Paul');

and this
this.el.textContent = 'Hi Paul';

would have the same behaviour.

Thank you!


Mulitple Fixes to _handlePolling

Multiple fixes :

fix Uri.parse is not working with https and not working if an encryption layer is added to token and session id , so instead so we replace it bt Uri.https that accept both http and https
Add more information when we receive a non 200 status code from the janus gateway
add a helper method to extract the domain needed by Uri.https  to make the flow more smooth
fix parse method that was throwing error if the return is an empty string

@all-contributors please add @mazen930  for code


Test Issue created at Wed, 16 Feb 2022 02:22:15 GMT

Test issue body  Wed, 16 Feb 2022 02:22:15 GMT

Comment on issue at Wed, 16 Feb 2022 02:22:16 GMT


Updated Spring xsd to reference gh-pages

As per the bean-mapping.xsd, also applied changes to dozer-spring.xsd

@orange-buffalo ; can you review please


Fixes 'teams conversationmember add' command. Closes #2671

Fixes teams conversationmember add command. Closes #2671
Make use of v1.0 endpoint.

Thanks @SmitaNachan 👏 we will review shortly

Merged manually. Thank you! 👏


Python wrapper fails upon import

I compiled and installed the C++ and then Python according to instructions and when I finally do import minisam I get this error:

ImportError                               Traceback (most recent call last)
 in
----> 1 import minisam
~/anaconda3/lib/python3.7/site-packages/minisam/init.py in
1
2 # wrapped c++ module
----> 3 from . _minisam_py_wrapper import *
4
5
ImportError: /home/maintenance/anaconda3/lib/python3.7/site-packages/minisam/_minisam_py_wrapper.cpython-37m-x86_64-linux-gnu.so: undefined symbol: _Z13wrap_geometryRN8pybind116moduleE

What am I missing?

Did you manage to solve this? I am also getting this error

Same error here, does anyone has a solution?

After you have done all the cmake and make commands.
Can you go into the build folder and  try to execute with your virtual environment (activated) : python3 setup.py
Then, start python3 and and try to import minisam
do you still get the error?

After you have done all the cmake and make commands.
Can you go into the build folder and  try to execute with your virtual environment (activated) : python3 setup.py
Then, start python3 and and try to import minisam
do you still get the error?

I'm having same problem, too.
With Python 3.9 in Windows, it works.But with Python 3.8 in Windows or Python 3.6 in Ubuntu, it shows error same with above.

I saw the same thing and was finally able to import minisam in Python without seeing the undefined symbol error by installing an older version of Sophus (commit a0fe89a323e20c42d3cecb590937eb7a06b8343a) and then re-installing minisam. This is Ubuntu 18.04 on an ARM64 (Jetson Xavier NX) by the way.

you have to compile with Sophus dependency. without it doesn't work.


ref(sdk): use the macro the generate bindings

NOTE: this fixes a test that was using an outdated rev of wit-bindgen

Closes #1347


Content not receiving scroll event

Ionic version:  (check one with "x")
[ ] 1.x
[x ] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[] feature request
[] support request
Current behavior:
Content is not receiving the scroll event. Since the setters for many of its properties rely on this event to assign their values, the result is that while, for example getContentDimensions will successfully give us the correct contentHeight and scrollTop in the object it returns, we can't use scrollToTop because the instance members scrollTop and contentHeight are never set. When I put breakpoints in scroll-view.ts's scrollCallback, they are never hit, and when I add callbacks to ionScroll they are never called.
Expected behavior:
Content should always receive the scroll event when its contents are scrolled.
Steps to reproduce:
The Content that we have the issue with is a fullscreen Content inside of a Slider inside of a Content. However, when I try to reproduce the issue I cannot.
Related code:
This is the plunker I tried to reproduce the issue in--unsuccessfully, unfortunately.
http://plnkr.co/edit/9yHRflqnzFDWaXH3E5Fc?p=preview
Other information:
I can't be sure that this isn't a bug on our end since I'm unable to reproduce it, but since I can see that the Content has been scrolled when I use getContentDimensions it should definitely be receiving scroll events from the HTML. Unfortunately, I can't manually change the values of the properties I need to set to make scrollToTop work.
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.6
Ionic CLI Version: 2.1.13
Ionic App Lib Version: 2.1.7
ios-deploy version: 1.9.0
ios-sim version: 5.0.11
OS: macOS Sierra
Node Version: v4.5.0
Xcode version: Xcode 8.2.1 Build version 8C1002

I think I have similar problem after upgrade to 2.0.0. final. There is uncaught error which is called on scroll event.
Uncaught TypeError: Cannot read property 'ev' of null     at Content._readDimensions (content.js:525)     at dispatch (dom-controller.js:128)     at DomController._flush (dom-controller.js:102)     at rafCallback (dom-controller.js:95)
It's because this._scroll is not initialized in content.js file on line 525:
var scrollEvent = this._scroll.ev;
Maybe it's connected.

I'm seeing something similar in 2.0.0 final. I've set up a new app using the "blank" template, and followed the testing setup suggested by Josh Moroney in his blog. After running and successfully completing a single Component test using the --single-run flag with karma, I see:
PhantomJS 2.1.1 (Mac OS X 0.0.0) ERROR
  TypeError: null is not an object (evaluating 'this._scroll.ev')
  at webpack:///~/ionic-angular/components/content/content.js:525:0 <- src/test.ts:10518

I can see the same error in the console when I run the tests in Chrome.

Hello all! I am going to close this issue as a duplicate of https://github.com/driftyco/ionic/issues/10186. Thanks for using Ionic!


FileBufferingReadStream _maxRentedBufferSize is used as an exclusive bound

FileBufferingReadStream _maxRentedBufferSize is checked as an exclusive bound staying just below 1MB.
We currently carry around our own constant that computes 1024 * 1024 -1 just to stay under that slightly unnatural max buffer rent size.
In the interest of usability it should probably just respect any size up to and including 1MB.

So you're basically saying that this code should be a <= instead of a <? Seems reasonable. Interested in sending a quick PR?

We can do this change as part of https://github.com/aspnet/AspNetCore/pull/11569 or create a quick one here with the change to have it isolated.


build: make tooling improvements

coverage: 100.0%. remained the same
when pulling 1c67beeb75591cecaf2a103d45a2ef239fbc4568 on feature/build-tools
into 4b8e4f6fcae2dda2276ce312de755cdc72814941 on main.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (4b8e4f6) to head (1c67bee).

Additional details and impacted files
@@            Coverage Diff             @@
##              main       #32    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           27        27
  Lines          669       773   +104
==========================================
+ Hits           669       773   +104

Flag
Coverage Δ

gi
100.00% <ø> (ø)

optional
100.00% <ø> (ø)

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


Better runtime failure messages

Meaning:

Make runtime failure messages visible in release mode, which were previously only available in debug builds
Add messages for runtime failures, which didn't have a message at all, like arithmetic overflow.

All this is done via debug info, so there is no overhead in code generation. Thanks @adrian-prantl for the idea of implementing this with inline functions in the debug info.
When having debug info available, e.g. when running in the debugger, the failure message is displayed as inline function.
Examples:
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_INSTRUCTION (code=EXC_I386_INVOP, subcode=0x0)
    frame #0: 0x0000000100000c9c a.out`testit1(_:) [inlined] arithmetic overflow at test.swift:4:12 [opt]
   1
   2   	@inline(never)
   3   	func testit1(_ a: Int8) -> Int8 {
-> 4   	  return a + 1
   5   	}
   6

* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_INSTRUCTION (code=EXC_I386_INVOP, subcode=0x0)
    frame #0: 0x0000000100000cbf a.out`testit3(_:) [inlined] Unexpectedly found nil while unwrapping an Optional value at test.swift:14:11 [opt]
   11
   12  	@inline(never)
   13  	func testit(_ a: Int?) -> Int {
-> 14  	  return a!
   15  	}
   16

This PR is not complete yet, I still need to add tests.
rdar://problem/51278690

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - 79f89a2137345b45a1e02bdf313b90cc89ea1cb6

Build failed
Swift Test OS X Platform
Git Sha - 79f89a2137345b45a1e02bdf313b90cc89ea1cb6

@swift-ci test

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - edf78542909ddc4921c4d39725b591a217eb2ee9

Build failed
Swift Test OS X Platform
Git Sha - edf78542909ddc4921c4d39725b591a217eb2ee9

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - edf78542909ddc4921c4d39725b591a217eb2ee9

Build failed
Swift Test OS X Platform
Git Sha - edf78542909ddc4921c4d39725b591a217eb2ee9

@eeckstein We've reverted this because we think it may have broken four tests:

SILOptimizer/pound_assert.swift
SILOptimizer/existential_transform.swift
IRGen/condfail_message.swift
DebugInfo/linetable-codeview.swift

See e.g. https://ci.swift.org/job/oss-swift_tools-RA_stdlib-DA_test-simulator/715 for examples of the failures.


Add composer-compile-plugin

composer-compile-plugin allows library packages to define their own compilation steps (similar to post-install hooks).  Adds a permission/whitelist mechanism to prompt downstream developers for permission before executing new compilation tasks.

Thank you for contributing! 👍


Add basic usage instructions

The README is blank and there's no setup.py.

I'm working to apply this to my own locality and would be happy to take up this issue.

Did either of you figure out how to get this to run properly?

Nope, I ended up writing my own code from scratch.

Nope, I ended up writing my own code from scratch.

Man, I am a complete amateur at this but could you tell me how you did it? I was hoping to create something that pulls the outages from my local utility that uses Kubra and keep a time of how long they last.

My employer should clear me to publish the repository in a few days.

@yk0055544 Here it is: https://github.com/justlab/Outages . I can't really provide support, but it's something.

thanks @Kodiologist  how could the code be modified to pull from the zip codes layer instead of the "area cluster" it's on?

That's the kind of support I'm not providing.


(🧪) Add tests for preview support

Perfect for gorgeous looks 🚀
This crashes and I have no idea why. 😕

Thank you, I will take a look into this failure.

@KotlinIsland It can be fixed by upgrading the dependency on mockserver-netty to 5.14.0
You can do it via rebasing on the master branch, I've just bumped it up in the build script.


Bad credentials response

I'm using the simple login example from the README.md - this always worked. For both my accounts. Both are North America Somfy, and the main difference is the email address. One has a plus in it, e.g<EMAIL_ADDRESS>and the other one doesn't.
Since a few hours ago the email with a plus stopped working with this error:
{'errorCode': 'AUTHENTICATION_ERROR', 'error': 'Bad credentials'}
The other one still works. I independently confirmed that logging in and out still works through the Tahoma app for both accounts.
I'm led to believe that the reason that pyoverkiz doesn't work anymore is simply because of the plus-sign in the email.
Can you confirm this?

Now the other account is also responding with "bad credentials". It still works with their app, but no longer with pyoverkiz.
Anyone else having this issue?

Ok, now they both work again.. nothing changed on my end. I guess this was Overkiz backend related. Closing.


Publish events as iCal/ICS

Create an endpoint that publishes upcoming events as iCal so they can be imported into one's own calendar.

This will require end times to be listed on events. I think it's still a worthy tradeoff.

After #11, the DB schema allows events without dates. Therefore, it should now be easy enough to add this feature without requiring breaking changes to the database.
When generating the iCal, we should ignore events that don't yet have an end date (as that needs to be added manually for each event). Ideally, the editor should disallow saving events without an end date, but we probably want to make an exception for migrated events that already don't have an end date.
As this is a major schema change, adding this to the v2.0.0 milestone.


Pylint

In this PR:

added pylint config from sibling project howfairis
fixed the errors

Thanks for reviewing and the docstrings!


Changing the entry field

How can I change the entry field in the config? Can't find enough usage examples to use rescripts right :(

If you want to do it from your config, just do the following:
rescriptsrc.js
module.exports = [require.resolve(path/to/entry/rescript.js)]

You could also do it from your package.json:
...
"rescripts": [
  path/to/entry/rescript.js
],
...

Whats do you mean by "path/to/entry/rescript.js"? I'm looking to edit the following property:
      // Include an alternative client for WebpackDevServer. A client's job is to
      // connect to WebpackDevServer by a socket and get notified about changes.
      // When you save a file, the client will either apply hot updates (in case
      // of CSS changes), or refresh the page (in case of JS changes). When you
      // make a syntax error, this client will display a syntax error overlay.
      // Note: instead of the default WebpackDevServer client, we use a custom one
      // to bring better experience for Create React App users. You can replace
      // the line below with these two lines if you prefer the stock client:
      // require.resolve('webpack-dev-server/client') + '?/',
      // require.resolve('webpack/hot/dev-server'),
      isEnvDevelopment &&
        require.resolve('react-dev-utils/webpackHotDevClient'),
      // Finally, this is your app's code:
      paths.appIndexJs,
      // We include the app code last so that if there is a runtime error during
      // initialization, it doesn't blow up the WebpackDevServer client, and
      // changing JS code would still trigger a refresh.
    ].filter(Boolean),

in webpack.config.js without ejecting

Yep––although I'd have to refer to the Webpack docs to be certain. That is correct usage of Rescripts though!


NOISSUE - Apply policies to Channels

What does this do?
This PR adds policies for Channels entity encapsulating following operations; view, update and delete. The admin can do all of the operations on the all channels.
Which issue(s) does this PR fix/relate to?
List any changes that modify/break current functionality
Updated Channels repository and tests.
Have you included tests for your changes?
Did you document any new/modified functionality?
Notes

Codecov Report

Merging #1505 (abae40f) into master (be3e98f) will decrease coverage by 0.11%.
The diff coverage is 57.69%.

@@            Coverage Diff             @@
##           master    #1505      +/-   ##
==========================================
- Coverage   68.24%   68.12%   -0.12%
==========================================
  Files         132      132
  Lines       10747    10781      +34
==========================================
+ Hits         7334     7345      +11
- Misses       2826     2841      +15
- Partials      587      595       +8

Impacted Files
Coverage Δ

things/service.go
64.98% <47.61%> (-4.82%)
:arrow_down:

things/postgres/channels.go
80.61% <100.00%> (+0.12%)
:arrow_up:

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 be3e98f...abae40f. Read the comment docs.


celicli transfer with ledger display bug

fixing maxFeePerGas setting and and using native transfer for transfers instead of erc20

@nicolasbrugneaux can you please provide a number for "Elapsed + remaining (hours)"?

fixed by #316


The README.md in the marketplace is not updating for Swimm.swimm

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

VS Code Version: 1.69.1
OS Version: macOS 12.4

Our new README.md for our extension (Swimm.swimm) is not showing/updating in the marketplace. One suspect here is that the extension used to be universal and download binaries from the net and now it is platform specific. But there might be any other issue causing this. If I should direct this issue towards somewhere else, please let me know.
Extension: https://marketplace.visualstudio.com/items?itemName=Swimm.swimm

@segevfiner aw no! We are looking into it on priority. I will keep you posted on the progress

@segevfiner We are aware of this issue, https://github.com/microsoft/vsmarketplace/issues/415 has more details.
We will soon be rolling out a fix for this.
Regarding the license & changelog.md, I am able to view the content and don't see 404 error.

We have now released a new version with a new README.md and it hasn't updated yet again. 😢

The fix for the issue is committed and should become available with a future release. I should update this issue in 2-3 weeks once we have the fix deployed.

We have confirmation from #468 that the issue is fixed. I will close this one.


Use default arch in CI

There are warnings (e.g., https://github.com/ranocha/SummationByPartsOperators.jl/actions/runs/12125109358?pr=307) in the CI runs because we request x64 on a macOS runner with arm64. I propose to simply use the default architecture, i.e. to remove the arch argument.

Regarding the failing tests, see https://github.com/trixi-framework/P4est.jl/pull/135#issuecomment-2512286826.

Could you please update this with option 2 from the other PR you linked above, i.e.,

Exclude macos-latest for Julia v1.6 (and include macos-13 for Julia v1.6)

I would like to keep this package working also for older versions of Julia if possible

Could you please update this with option 2 from the other PR you linked above, i.e.,

Exclude macos-latest for Julia v1.6 (and include macos-13 for Julia v1.6)

Done in 658752d.

Ready for review @ranocha (remember to change the required CI checks).

coverage: 92.56%. remained the same
when pulling 4225b32a0f4a983597d29daa501ba95f812b2b4d on JoshuaLampert:arch-ci
into b9a260909b676558a369019a15ead28ffcdb2462 on ranocha:main.


Tab does not automatically complete tag matching

VSCode Version: Code 1.15.0 (8b95971d8cccd3afd86b35d4a0e098c189294ff2, 2017-08-09T20:16:06.988Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author (truncated)
Version

vscode-icontheme-nomo-dark
be5
1.3.2

vscode-eslint
dba
1.2.11

javadebugger
don
0.1.3

jquerysnippets
don
0.0.1

vscode-html-css
ecm
0.1.7

vscode-great-icons
emm
2.1.14

vsc-material-theme
Equ
1.0.2

vscode-javac
geo
0.0.46

vue-snippets
hol
0.1.2

beautify
Hoo
1.1.1

vscode-icon-theme
jtl
1.5.0

vscode-JS-CSS-HTML-formatter
lon
0.2.3

debugger-for-chrome
msj
3.1.7

material-icon-theme
PKi
2.1.0

vscode-icons
rob
7.12.0

D3Snippets
yoy
0.1.11

(4 theme extensions excluded)

Steps to Reproduce:
1.Enter a div
2.press the Tab key does not automatically complete the label match

Reproduces without extensions: Yes/No

https://code.visualstudio.com/blogs/2017/08/07/emmet-2.0

You have 3 choices

In VS Code 1.15, tab is no longer associated with Emmet. You can bind any other keyboard shortcut to the command editor.emmet.action.expandAbbreviation
Install latest Insiders from https://code.visualstudio.com/insiders and set emmet.triggerExpansionOnTab to true. You will get emmet back on pressing tab
Wait for the VS Code 1.15.1 release which will be tomorrow. Set emmet.triggerExpansionOnTab to true. You will get emmet back on pressing tab


HSEARCH-3947 Avoid creating huge arrays for collectors when a query does not have an upper bound limit

https://hibernate.atlassian.net/browse/HSEARCH-3947

the commit https://github.com/hibernate/hibernate-search/pull/2361/commits/3c8bd9a619605a5fb621d60902e7141dafc223e9 is also to check that limited-unbounded queries on large indexes but with small matches still work.

The implementation looks good, but I think some tests are missing? See my comments above.

@yrodiere thanks for the review, changes I think was done...

Looks good to me, thanks. Feel free to merge once the build passes.

Thank you!


Use "xl" breakpoint earlier

This might be a bit of a controversial change, but the xl breakpoint is now triggered at 1300 instead of 1900 pixels.
Due to this VelCom will now use nearly the full width of the window a lot earlier and not have as much space around it, even if you open it on a smaller monitor or split/resize the window.
If most feel this is a "Verschlimmbesserung", the PR can be closed.
Before

After

Looks useful, although scrolling past the detail graph to get to the controls has become a bit harder.


Prime sign (’) is less compatible than single quote (')

Although the prime sign (’), for example, "Man’s" as opposed to "Man's" , is semantically more accurate than single quote ('), in many cases, single quotes are easier to handle when it comes to ASCII code system (even in Excel on MAC). So it would be easier to use your great efforts if all the primes are replaced by single quotes.

This makes sense to me. Want to file a pull request?


ipset kernel module/support missing?

Looks like something is missing from the kernel to support ipset:
root@intel-corei7-64-06fd714:/# ipset create domain-filter-ipv4 hash:net
ipset v6.23: Kernel error received: Invalid argument

root@intel-corei7-64-06fd714:/# ipset list
ipset v6.23: Kernel error received: Invalid argument

The same works fine on the RPi1, RPi2 and RPi3 builds.
This one is a show-stopper.
 Front conversations

Thank you for the report. We will take this on and let you know.

This issue was moved to resin-os/resinos#68


simple-add and vector-add samples - build process broken without (an optional) Quartus dependency

Summary
The build of simple-add and vector-add samples fails. It appears to require Quartus to proceed.
It is expected that most users will not build for FPGA target. So this dependency seems to be bogus
Version
oneAPI Base Kit 2023.1.0
oneAPI-samples: 5be94fc7db259dafd54e8aedfcc99b39dc025cfc
Environment
Ubuntu 20.04
Steps to reproduce

Clone this repo
cd oneAPI-samples/DirectProgramming/C++SYCL/DenseLinearAlgebra/vector-add
mkdir build
cd build
cmake ..
make

Observed behavior
The make process fails with the following error output:
Scanning dependencies of target vector-add-buffers.fpga_sim
[ 12%] Building CXX object src/CMakeFiles/vector-add-buffers.fpga_sim.dir/vector-add-buffers.cpp.o
[ 25%] Linking CXX executable ../vector-add-buffers.fpga_sim
aoc: This compilation stage requires an installation of Intel(R) Quartus(R) Prime software!
aoc: However Quartus was not found on the PATH or passed in using QUARTUS_ROOTDIR_OVERRIDE.
aoc: If you have a valid installation, please pass it in using QUARTUS_ROOTDIR_OVERRIDE or add it to the PATH.
aoc: Otherwise, consider downloading Quartus from here: https://www.intel.com/content/www/us/en/software-kit/661712/intel-quartus-prime-pro-edition-design-software-version-19-2-for-linux.html
llvm-foreach:
icpx: error: fpga compiler command failed with exit code 1 (use -v to see invocation)
make[2]: *** [src/CMakeFiles/vector-add-buffers.fpga_sim.dir/build.make:84: vector-add-buffers.fpga_sim] Error 1
make[1]: *** [CMakeFiles/Makefile2:187: src/CMakeFiles/vector-add-buffers.fpga_sim.dir/all] Error 2
make: *** [Makefile:84: all] Error 2

Expected behavior
The sample will build for CPU/GPU target (and maybe FPGA emulation), not for FPGA hardware

The README explains how to build the project for all the different targets:

cpu-gpu: I guess that this is what you are looking for. It targets the default selector and does not require additional software
fpga_emu: FPGA emulator. Does not require additional software.
fpga_sim: FPGA simulator. Does require additional software.
fpga: FPGA hardware. Does require additional software.

In the case of the cpu-gpu, the instructions are given as follow:

So you need to specify what you want to build.
There are also instructions for building the fpga_emu, fpga_sim and fpga targets in the README.
I can see that you typed “make” which by default, tries to build all available targets (and this is not what we recommend doing in the README).
Because some additional software is required for building some of the FPGA specific targets, there is a note in the “Prerequisites” section of the README to warn the users about this:

So I think that the README covers all the necessary instructions.
Let me know if you need further clarifications.


fix(SLPDB status): Adding GET /slp/status to get status on SLPDB

:tada: This PR is included in version 1.24.8 :tada:
Your semantic-release bot :package::rocket:


Running Build.cmd doesn't build unless -build is specified

This was caused by the same eng change that I saw go into other repos to support SBOM. The change that caused the problem was #6545. I'll have a fix shortly.


Fix Azure roles PIM steps

Activating an Azure role does not require you to be a Privileged Role Administrator. I think the doc is incorrect and it shouldn't state that you have to sign in as a Privileged Role Admin in order to PIM elevate.

@barclayn
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


[Feature Request] Scheduled Backups

Have you read the Important information text above

[x] Yes i did

Current behavior
Server saves aren't backed up.
Desired behavior
The save files for the server should be backed up by some cron schedule, if desired, and controlled through environmental variables. Especially with the issues being reported about save files being lost.
Check out this Valheim server for a good example: https://github.com/lloesche/valheim-server-docker
BACKUPS=true
BACKUPS_CRON=0 * * * *
BACKUP_DIRECTORY=/something
etc...
Basically, we just set up a cronjob in the container to run the admin /Save command to do a forced save, then copy that save to the backup directory and date it.
Save folder:
/palworld/Pal/Saved/SaveGames/0/9D3D178236CA4DCD977E978D94FFD633/Players
Links to screenshots
No response
To Reproduce
NA
Software setup
NA
Hardware setup
NA
Additional context
No response

Hey @CameronJGrant - I like the idea, would look into this tomorrow.
Few question though:

Dont you mean a "tar.gz" of Saved?
How would you run the /Save command inside the container? How would you inject that in the process? The RCON tool is outside of the container
Are we now mixing internal and external work?
Could you give me more details on your thoughts so i get a better understanding of the workflow you imagine?
I will look into https://github.com/lloesche/valheim-server-docker tomorrow, its no 01:19 in the morning 🥱

You'd need a whole process manager to also have cron run in the container. Might be a bit too much for now because your entrypoints eventually delegates to the palserver directly. Valheim used something like that, but they had years of development ;)
I'd just add an example to the readme how to create backups with cron yourself.
Like backup.sh:
#!/bin/bash
# dir should be absolute for cron or you might end up anywhere:
cd /your/palworld/directory
mkdir -p backups
tar -czf backups/${something-with-date}.tar.gz game/Pal/Saved

And just call that via crontab.

You'd need a whole process manager to also have cron run in the container. Might be a bit too much for now because your entrypoint eventually delegates to the palserver directly. Valheim used something like that, but they had years of development ;)
I'd just add an example to the readme how to create backups with cron yourself.
Like backup.sh:
#!/bin/bash
# dir should be absolute for cron or you might end up anywhere:
cd /your/palworld/directory
mkdir -p backups
tar -czf backups/${something-with-date}.tar.gz game/Pal/Saved

And just call that via crontab.

Im already done implementing a solution, finishing the last tests and then i will push the changes.

Valheim used something like that, but they had years of development ;)

I know and i would rather stay away from unnecessary bloating the image with an additional process manager like supervisord, im very happy i got it out again in my Sons Of The Forest and The Forest Dedicated.
Im using now a solution thats more container oriented and dont need all that stuff like cron does.

Hey @uncaught @CameronJGrant -
I have implemented a backup-solution that does hourly backups in your game directory. If you want to change settings, you can enable or disable this and you can change the cron-expression, meaning the interval it happens. Therest is hardcoded on purpose and i prefer it that way. Too much open config can result in recursive backups and an early full harddisk and people beeing mad at me. I dont want that.
Please go ahead and try it out .

The backups look great, but could you also implement a configurable backup retention.
Like for example retaining the last 10 backups, or last day (timeframe) of backups, whichever is easier.
So I wouldn't need to implement my own cron just to handle the backup management, and maybe less problems with directory access.
Also you will definitely wont have complains for a full hard disk with a sensible default...

Excellent work @jammsen!

Hey @CameronJGrant - I like the idea, would look into this tomorrow. Few question though:

Dont you mean a "tar.gz" of Saved?

palworld/Pal/Saved would be better. I only had a few minutes to dig around the container yesterday and I wasn't sure if palworld/Pal/Saved/Config had anything that needed saving. I agree with your solution.

How would you run the /Save command inside the container? How would you inject that in the process? The RCON tool is outside of the container

You can run the RCON commands using something like netstat directly from a shell script within the container. We can include anything you like in your "backupmanager.sh".

Are we now mixing internal and external work?

Everything should be contained in one image. It's all internal.

Could you give me more details on your thoughts so i get a better understanding of the workflow you imagine?

Looks like you nailed it. The only things I would add in future, (and maybe I will create a PR for this since you already have your hands full) is to still call /Save explicitly using the RCON commands before backing up. Some games broadcast a warning before saving because a save can interrupt gameplay, but I don't think Palworld has that issue.

I will look into https://github.com/lloesche/valheim-server-docker tomorrow, its no 01:19 in the morning 🥱

I think we are on very different time zones.

You'd need a whole process manager to also have cron run in the container. Might be a bit too much for now because your entrypoint eventually delegates to the palserver directly. Valheim used something like that, but they had years of development ;)

You don't need all that. You just need cron running in the container. I would have just installed cron via apt-get but @jammsen came up with a far more elegant solution with supercronic.

The backups look great, but could you also implement a configurable backup retention. Like for example retaining the last 10 backups, or last day (timeframe) of backups, whichever is easier. So I wouldn't need to implement my own cron just to handle the backup management, and maybe less problems with directory access. Also you will definitely wont have complains for a full hard disk with a sensible default...

Absolutely right as well.
Keep within retention period(in days):
find /palworld/backups -name 'saved-*.tar.gz' -type f -mtime +$RETENTION_DAYS -exec rm {} \;
Or it might be better to just keep the latest n files in case you shut down the server for an extended period of time and set it back up later, all backups would get wiped. But people should be moving these backups offsite (to S3 or something) routinely anyway, so it's not a big deal.
Keep last N files:
cd /palworld/backups
ls -1t saved-*.tar.gz | tail -n +$(($KEEP_LAST_N_SAVES + 1)) | xargs -d '\n' rm -f --

@jammsen can you add a bit to the readme about the backup solution? Such as where the backupmanager.sh is expected?
I seem to be getting "/backupmanager.sh: not found" when the job runs. I am sure its just a me problem though; likely I have something in the wrong place.
Currently I have all the docker files and shell scripts outside my game folder and I mount ./game:/palworld in the docker-consume

The container is running, and I can connect to the server, but the docker logs say it cannot find the backupmanager.sh and it also has not created the backup directory.

The backups look great, but could you also implement a configurable backup retention. Like for example retaining the last 10 backups, or last day (timeframe) of backups, whichever is easier. So I wouldn't need to implement my own cron just to handle the backup management, and maybe less problems with directory access. Also you will definitely wont have complains for a full hard disk with a sensible default...

@abdonkov - Im open for suggestions/ideas/code examples, everything im coming up has up-sides and drawbacks. I thought at least deliver the function to test first 😭 - Keeping only days, will kill inactive server with valid saves on a bug, therefore no backups anymore. Keeping the last 1?3?10? How to decide? How to do?

Looks like you nailed it. The only things I would add in future, (and maybe I will create a PR for this since you already have your hands full) is to still call /Save explicitly using the RCON commands before backing up. Some games broadcast a warning before saving because a save can interrupt gameplay, but I don't think Palworld has that issue.

@CameronJGrant - I think i have even seen something like this coded, dont remember right now where and what the context was 😭 but i agree send /save and /broadcast "Server-Save in progress ..." would be cool. -> https://tech.palworldgame.com/server-commands

I think we are on very different time zones.

Germany!

You don't need all that. You just need cron running in the container. I would have just installed cron via apt-get but @jammsen came up with a far more elegant solution with supercronic.

😊 ❤
Im open to ideas on the "Rentention-Policy" - GO NUTS and help me out please 😄

@abdonkov - Im open for suggestions/ideas/code examples, everything im coming up has up-sides and drawbacks. I thought at least deliver the function to test first 😭 - Keeping only days, will kill inactive server with valid saves on a bug, therefore no backups anymore. Keeping the last 1?3?10? How to decide? How to do?

Once we figure out how to use RCON in script's more easily we should use the /ShowPlayers command to check if anyone's in game before saving so that we aren't backing up unless someone is actually in there doing something. Then add a grace period after the server empties that backups can be created in.
I suggest we do that and allow users to choose their retention policy.
Possible environment variables:

Variable
Description
Default Value
Allowed Value

BACKUP_ENABLED
Backup function, creates backups in your game directory
true
false/true

BACKUP_CRON_EXPRESSION
Needs a Cron-Expression - See https://github.com/aptible/supercronic#crontab-format or https://crontab-generator.org/
0 * * * * (meaning every hour)
Cron-Expression

BACKUP_MAX_COUNT
The maximum number of backup files to keep
5
Integer value

BACKUP_MAX_AGE
The maximum age for a backup file before it is deleted (in days)
30
Integer value (days)

BACKUP_IF_IDLE
Perform backup only if the server is idle
false
false/true

BACKUP_GRACE_PERIOD
Time to wait before starting the backup after the scheduled time (in minutes)
10
Integer value (minutes)

BACKUP_ZIP
Whether to compress the backup files into ZIP format
true
false/true

Weird side note. It saves backups of local data to the client's PC. I'm not sure if you could fix a server instance with that though.
https://docs.google.com/document/u/0/d/e/2PACX-1vR-lb9R2fh-IVK870PL82U071i-9-C_Y7YyroUN8EoD_34rbpKQzjsCejCHRb4XMGncY9HkI0OxQv1F/pub?pli=1#h.xl4nmt5nrqb9

@CameronJGrant - I think i have even seen something like this coded, dont remember right now where and what the context was 😭 but i agree send /save and /broadcast "Server-Save in progress ..." would be cool. -> https://tech.palworldgame.com/server-commands

Maybe it was here?
https://gist.github.com/mbasaglia/62d7fef83304f4b3ca90

Germany!

Cool!
Might I also suggest opening a develop branch?

Im open to ideas on the "Rentention-Policy" - GO NUTS and help me out please 😄

I mean if you don't want to decide how much backups to retain you can always set it to 0, meaning the retention by count won't be active. This will also require a retention days parameter as a fallback, like 2, 3 days. Something small so with the default options, the backups won't use too much space and you wont get complains 😄
If you are looking for inspiration, you could always see in the Valheim server docker container what parameters are implemented, which are mostly the ones that @CameronJGrant suggested.

Once we figure out how to use RCON in script's more easily we should use the /ShowPlayers command to check if anyone's in game before saving so that we aren't backing up unless someone is actually in there doing something. Then add a grace period after the server empties that backups can be created in.
I suggest we do that and allow users to choose their retention policy.

#Overkill 🤣

Maybe it was here? https://gist.github.com/mbasaglia/62d7fef83304f4b3ca90

No i could be here https://github.com/jammsen/docker-conanexiles/blob/master/src/usr/bin/conanexiles_controller#L103 it wasnt from the perspective of my build a thing, it was more of the experience of using person, like "im the admin of a dedicated server and host this for my friends and i find the feature ive not seen like this before very cool". Sadly i didnt fork it, because i didnt knew i would need it for something.

Might I also suggest opening a develop branch?

Yes, hopefully tomorrow.

If you are looking for inspiration, you could always see in the Valheim server docker container what parameters are implemented, which are mostly the ones that @CameronJGrant suggested.

Just to be on the safe side, you mean this one right? https://github.com/lloesche/valheim-server-docker

Hi guys, where is the "Rentention-Policy" you mentioned? is there any discord or ideas page? how can I read the thread?

@doodoori2 Read this https://github.com/jammsen/docker-palworld-dedicated-server/issues/27#issuecomment-1906980779 and realize you are already here ;)

Im open to ideas on the "Rentention-Policy" - GO NUTS and help me out please 😄

I mean if you don't want to decide how much backups to retain you can always set it to 0, meaning the retention by count won't be active. This will also require a retention days parameter as a fallback, like 2, 3 days. Something small so with the default options, the backups won't use too much space and you wont get complains 😄
If you are looking for inspiration, you could always see in the Valheim server docker container what parameters are implemented, which are mostly the ones that @CameronJGrant suggested.

I think i will just go with a keep N files strategy, no need to overcomplicate everything.

Keep last N files:
cd /palworld/backups
ls -1t saved-*.tar.gz | tail -n +$(($KEEP_LAST_N_SAVES + 1)) | xargs -d '\n' rm -f --

Actually that looks already nice to get a better understanding and get started

Just to be on the safe side, you mean this one right? https://github.com/lloesche/valheim-server-docker

Yes, that one.

Hey @CameronJGrant - Sorry for the delay i finaly got around to finish this. But Bugs > Features, sorry.
Ive added now a switch to turn it on or off and a number to keep with even you shell-code that i really liked. Works like a charm. Thanks for your contribution!

Im closing this issue as resolved, feel free to try it out and reopen again if needed.
If you like this project, please consider giving this repo and the docker-hub-repo a Star.

For sure I'll give you a star! Thanks for the hard work! Ironically I just had my server wipe lol.

Hey @jammsen. There isn't a reopen issue button so maybe it's turned off?
Anyway, small thing. I found out the hard way that if disk space fills in your server Palworld will delete your server file. It looks like, when it saves your game, it deletes your old file first, then saves the new one.
That said, I notice the defaults we have set are:
So I notice your defaults are:

Variable
Default

BACKUP_ENABLED
true

BACKUP_CRON_EXPRESSION
0 * * * *

BACKUP_RETENTION_POLICY
false

BACKUP_RETENTION_AMOUNT_TO_KEEP
30

This means, by default, we are backing up every hour and not removing any old backups. In a year people will have almost 9000 backups. It only comes to about 18GB but a lot of people run small servers. We might be sitting on a ticking time bomb as is.
I suggest having the retention policy on by default and set the time to once per day. Then 30 days are covered. (Since 30 at hourly is only one day).

For sure I'll give you a star! Thanks for the hard work! Ironically I just had my server wipe lol.

That sounds horrible and good at the same time 🤣

Hey @jammsen. There isn't a reopen issue button so maybe it's turned off?

OH DANG SORRY - https://stackoverflow.com/a/21333938

I suggest having the retention policy on by default and set the time to once per day. Then 30 days are covered. (Since 30 at hourly is only one day).

I get your point and i would have done that when i introduced BOTH features at the same time, it wasnt the case for this, it was incremental. Thats why i had to really hard think about it and didnt wanted people to have to loose savegames they didnt backup before, thats why its on false by default. But i get your point, thats why i rewrote the Setup-6-Steps another time after splitting files to make sure people CONFIG it to their liking you know?

Thanks for the hard work!

No! Thank YOU for your contribution Keanu-Wave! ❤


miniconda-installer.exe not found

for windows something has changed.

Name changed to
Miniconda3-latest-Windows-x86_64.exe

seems the guacamole system has no curl

If your Windows 10 build is 17063, or later, cUrl is included by default
Is guacamole not coming with the default installations for Windows?

oh, ok, I request an update the system I could use has an older build number

I learned that this is a windows 2016 server instance and this has no curl implemented in its build. We may be can download that first when it is missing or document that.
https://curl.se/windows/

Curl can probably also be replaced with powershell, but I always find it to be a hassle with Windows blocking script executions for it.

guacamole windows now provides curls


WIP: add mount point

Very stale. Closing. Thanks! Please open a new PR if there is still a relevant change.


first run poller error

My miflora is working with ios app and detected with lescan
When I try running the miflora-mqtt-daemon.py I get following error:
Any ideas where my problem might be?
alexroot@openhabserver /opt/miflora-mqtt-daemon (git)-[master] # python3 /opt/miflora-mqtt-daemon/miflora-mqtt-daemon.py
Xiaomi Mi Flora Plant Sensor MQTT Client/Daemon
Source: https://github.com/ThomDietrich/miflora-mqtt-daemon
[2018-05-02 11:39:34] Connecting to MQTT broker ...
[2018-05-02 11:39:34] MQTT connection established
Adding sensor to device list and testing connection ...
Name:          "Tomato"
Traceback (most recent call last):
File "/opt/miflora-mqtt-daemon/miflora-mqtt-daemon.py", line 218, in
flora_poller.fill_cache()
File "/usr/local/lib/python3.6/dist-packages/miflora/miflora_poller.py", line 61, in fill_cache
firmware_version = self.firmware_version()
File "/usr/local/lib/python3.6/dist-packages/miflora/miflora_poller.py", line 104, in firmware_version
res = connection.read_handle(_HANDLE_READ_VERSION_BATTERY)  # pylint: disable=no-member
File "/usr/local/lib/python3.6/dist-packages/btlewrap/gatttool.py", line 23, in _func_wrapper
return func(*args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/btlewrap/gatttool.py", line 253, in read_handle
raise BluetoothBackendException("Exit read_ble, no data ({})".format(current_thread()))
btlewrap.base.BluetoothBackendException: Exit read_ble, no data (<_MainThread(MainThread, started -1225293296)>)

Duplicate of #39
Please apply the fix manually. It will soon be added here.


There is no 3.3v voltage regulator for GPS on PCB

I had issues with running both GPS from Tiva-C I think it has 100mA limit on 3.3v current. Need to test before we change it.

The limit is supposedly 400 mA according to the Tiva C LaunchPad User Manual, which is much higher than we need. I would carefully feel the voltage regulator (U8) to ensure that it is not heating up too much.

Issue has been resolved


Inputs and Segmentation

Hi Danill,
First of all,
Thanks for this amazing repository.
You have mentioned in previous issues that the segmentation part can be commented out. Masks are created from segmentations data. The keypoint mask and the paf mask also depend upon the masks and thus the segmentations as well .So should that be excluded from the inputs while training as well?
is there only two inputs for training purpose- 1)keypoint_map and 2)paf map?

Thank you for resolving this issue


Button Press with spacebar

Bug, feature request, or proposal:
In native buttons, when we have focus on a button we can use spacebar to press the button. It seems that in case of material buttons we can only use enter/return key to press button.
What is the expected behavior?
Material buttons should support press/submit through spacebar when focus is on the button.

Tried to reproduce this in the demo-app of Material, but everything seemed to work.
Can you please provide a Plunker that shows the issue?

Ok, I just noticed that it doesn't work when used with anchor tag like this:
Cancel

Hey @naveedahmed1 - this is expected behavior for <a> elements and appropriate for accessibility.


added logo in the top

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

[x] Fixes #
[x] Labelled this PR & related issue with documentation tag

LGTM. Please sign the DCO & resubmit the PR

LGTM. Please sign the DCO & resubmit the PR

Where will i find the DCO?

LGTM. Please sign the DCO & resubmit the PR

Where will i find the DCO?

Please check the details hyperlink in the Checks section. It has instructions on how this needs to be done.

LGTM. Please sign the DCO & resubmit the PR

Where will i find the DCO?

Please check the details hyperlink in the Checks section. It has instructions on how this needs to be done.

Got it. Thanks for such an early response! Will do this asap.

@divya-mohan0209 Please check if it's done or not...

@divya-mohan0209 Please check if it's done or not...

Please commit the changes properly. Merely adding a comment will not help it pass the DCO.

@divya-mohan0209 Please check if it's done or not...

Please commit the changes properly. Merely adding a comment will not help it pass the DCO.

I'm new to DCO, can you please tell me is there a way to fix this without using git?
Thanks.

@xmonish : You need to use the GIT CLI.
To fix, first ensure you have a local copy of your branch by checking out the pull request locally via command line.  Next run
git push --force-with-lease origin patch-1

@xmonish : You need to use the GIT CLI.
To fix, first ensure you have a local copy of your branch by checking out the pull request locally via command line.  Next run
git push --force-with-lease origin patch-1

I'm getting stucked after proceeding with the 1st step. Not able to enter commands of the 2nd step.
And when i reopen the cli & enter the 2nd code again, then i'm getting "Error: Failed to push some refs".

Please checkout a local copy before you actually commit anything. Once you have checked it out, please commit the code with the message. Then push the changes.

Updated the DCO in the new PR.

DCO is still pending & there are merge conflicts.

Seems to be a duplicate PR of https://github.com/litmuschaos/litmus-docs/pull/384. Closing in favour of that.


K8SSAND-1320 ⁃ server-system-logger sidecar can not be created because of missing user

What happened?
I get a CreateContainerError for the logging sidecar (k8ssandra/system-logger:v1.10.0)
Warning  Failed     2m8s                   kubelet            Error: failed to create containerd container: mount callback failed on /var/lib/rancher/k3s/agent/containerd/tmpmounts/containerd-mount3654802940: no users found

Did you expect to see something different?
The container should be created.
How to reproduce it (as minimally and precisely as possible):
Just install the cass-operator helm chart.
Environment

Cass Operator version:
docker.io/k8ssandra/cass-operator:v1.10.1

* Kubernetes version information:```  Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.2", GitCommit:"8b5a19147530eaac9476b0ab82980b4088bbc1b2", GitTreeState:"clean", BuildDate:"2021-09-15T21:38:50Z", GoVersion:"go1.16.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.4+k3s1", GitCommit:"43b1cb48200d8f6af85c16ed944d68fcc96b6506", GitTreeState:"clean", BuildDate:"2022-02-24T22:38:17Z", GoVersion:"go1.17.5", Compiler:"gc", Platform:"linux/amd64"}

* Kubernetes cluster kind:

insert how you created your cluster: k3d
**Anything else we need to know?**:
The reason for this error seems to be that the docker image of the sidecar does not specify a cassandra user inside /etc/passwd but it is referenced inside logger.Dockerfile. One possible solution which seems to work would be to just use nobody:

```diff
diff --git a/logger.Dockerfile b/logger.Dockerfile
index 39c9037..fcc749a 100644
--- a/logger.Dockerfile
+++ b/logger.Dockerfile
@@ -17,8 +17,8 @@ ADD https://raw.githubusercontent.com/krallin/tini/master/LICENSE /licenses/LICE
 RUN chmod +x /sbin/tini
 COPY ./LICENSE.txt /licenses/

-# Non-root user, cassandra as default
-USER cassandra:cassandra
+# Non-root user, nobody as default
+USER nobody:nogroup
 ENTRYPOINT ["/sbin/tini", "--"]

 # Run your program under Tini

┆Issue is synchronized with this Jira Task by Unito
┆friendlyId: K8SSAND-1320
┆priority: Medium

I haven't seen that error anywhere else (and this is the first time anyone has reported it), so I wonder if k3d is handling these security contextes differently.
➜  cass-operator git:(go_117_update) ✗ kubectl exec -i -t -c cassandra pod/cluster2-dc2-r1-sts-0 -- /bin/bash
cassandra@cluster2-dc2-r1-sts-0:/$ whoami
cassandra
cassandra@cluster2-dc2-r1-sts-0:/$ ls -l /var/log/cassandra
total 3716
-rw-r--r-- 1 cassandra cassandra    2773 Mar 15 11:50 cassandra-collectd.log
-rw-r--r-- 1 cassandra cassandra  152109 Mar 15 11:49 debug.log
-rw-r--r-- 1 cassandra cassandra 1843673 Mar 15 11:50 gc.log
-rw-r--r-- 1 cassandra cassandra       4 Mar 15 11:49 mcac-collectd.pid
-rw-r--r-- 1 cassandra cassandra       0 Mar 15 11:48 stderr.log
-rw-r--r-- 1 cassandra cassandra   59820 Mar 15 11:49 stdout.log
-rw-r--r-- 1 cassandra cassandra   65049 Mar 15 11:49 system.log
cassandra@cluster2-dc2-r1-sts-0:/$

That's why the server-system-logger is using the same user, as those logs are created to user cassandra. The PodSecurityContext should set the userID to 999.
@jeffbanks Do you happen to remember, or testing with k3d? I remember you did the change, but it's been a while. Or this could be related to containerd also.

It's installed just with
helm install my-cass-operator k8ssandra/cass-operator --version 0.35.1
A datacenter without any securityContext at all:
apiVersion: cassandra.datastax.com/v1beta1
kind: CassandraDatacenter
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"cassandra.datastax.com/v1beta1","kind":"CassandraDatacenter","metadata":{"annotations":{},"name":"dc1","namespace":"default"},"spec":{"clusterName":"development","dockerImageRunsAsCassandra":false,"managementApiAuth":{"insecure":{}},"racks":[{"name":"rack1"}],"resources":{"requests":{"cpu":"1000m","memory":"2Gi"}},"serverType":"cassandra","serverVersion":"4.0.1","size":1,"storageConfig":{"cassandraDataVolumeClaimSpec":{"accessModes":["ReadWriteOnce"],"resources":{"requests":{"storage":"10Gi"}},"storageClassName":"local-path"}}}}
  creationTimestamp: "2022-03-15T21:17:05Z"
  finalizers:
  - finalizer.cassandra.datastax.com
  generation: 2
  name: dc1
  namespace: default
  resourceVersion: "3585"
  uid: 4c0c4e3f-776e-420f-98d1-cc39de3f091c
spec:
  additionalServiceConfig:
    additionalSeedService: {}
    allpodsService: {}
    dcService: {}
    nodePortService: {}
    seedService: {}
  clusterName: development
  configBuilderResources: {}
  dockerImageRunsAsCassandra: false
  managementApiAuth:
    insecure: {}
  racks:
  - name: rack1
  resources:
    requests:
      cpu: "1"
      memory: 2Gi
  serverType: cassandra
  serverVersion: 4.0.1
  size: 1
  storageConfig:
    cassandraDataVolumeClaimSpec:
      accessModes:
      - ReadWriteOnce
      resources:
        requests:
          storage: 10Gi
      storageClassName: local-path
  systemLoggerResources: {}
status:
  cassandraOperatorProgress: Updating
  lastServerNodeStarted: "2022-03-15T21:30:05Z"
  nodeStatuses:
    development-dc1-rack1-sts-0:
      hostID: 8a33fc32-bbdc-43d4-a5da-c88a90f3f3f3

This is the StatefulSet the operator generates:
apiVersion: apps/v1
kind: StatefulSet
metadata:
  annotations:
    cassandra.datastax.com/resource-hash: 3pLUEt23ERAkQoq3xyP96hdGPtmXTrcIhFwbCcHGR58=
  creationTimestamp: "2022-03-15T21:17:05Z"
  generation: 2
  labels:
    app.kubernetes.io/instance: cassandra-development
    app.kubernetes.io/managed-by: cass-operator
    app.kubernetes.io/name: cassandra
    app.kubernetes.io/version: 4.0.1
    cassandra.datastax.com/cluster: development
    cassandra.datastax.com/datacenter: dc1
    cassandra.datastax.com/rack: rack1
  name: development-dc1-rack1-sts
  namespace: default
  ownerReferences:
  - apiVersion: cassandra.datastax.com/v1beta1
    blockOwnerDeletion: true
    controller: true
    kind: CassandraDatacenter
    name: dc1
    uid: 4c0c4e3f-776e-420f-98d1-cc39de3f091c
  resourceVersion: "3518"
  uid: 9e52ea0f-165f-4eda-8e53-a312c7a1661d
spec:
  podManagementPolicy: Parallel
  replicas: 1
  revisionHistoryLimit: 10
  selector:
    matchLabels:
      cassandra.datastax.com/cluster: development
      cassandra.datastax.com/datacenter: dc1
      cassandra.datastax.com/rack: rack1
  serviceName: ""
  template:
    metadata:
      creationTimestamp: null
      labels:
        app.kubernetes.io/instance: cassandra-development
        app.kubernetes.io/managed-by: cass-operator
        app.kubernetes.io/name: cassandra
        app.kubernetes.io/version: 4.0.1
        cassandra.datastax.com/cluster: development
        cassandra.datastax.com/datacenter: dc1
        cassandra.datastax.com/node-state: Ready-to-Start
        cassandra.datastax.com/rack: rack1
    spec:
      affinity:
        podAntiAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
          - labelSelector:
              matchExpressions:
              - key: cassandra.datastax.com/cluster
                operator: Exists
              - key: cassandra.datastax.com/datacenter
                operator: Exists
              - key: cassandra.datastax.com/rack
                operator: Exists
            topologyKey: kubernetes.io/hostname
      containers:
      - env:
        - name: DS_LICENSE
          value: accept
        - name: DSE_AUTO_CONF_OFF
          value: all
        - name: USE_MGMT_API
          value: "true"
        - name: MGMT_API_EXPLICIT_START
          value: "true"
        - name: DSE_MGMT_EXPLICIT_START
          value: "true"
        image: k8ssandra/cass-management-api:4.0.1
        imagePullPolicy: IfNotPresent
        lifecycle:
          preStop:
            exec:
              command:
              - wget
              - --output-document
              - /dev/null
              - --no-check-certificate
              - --post-data=''
              - http://localhost:8080/api/v0/ops/node/drain
        livenessProbe:
          failureThreshold: 3
          httpGet:
            path: /api/v0/probes/liveness
            port: 8080
            scheme: HTTP
          initialDelaySeconds: 15
          periodSeconds: 15
          successThreshold: 1
          timeoutSeconds: 1
        name: cassandra
        ports:
        - containerPort: 9042
          name: native
          protocol: TCP
        - containerPort: 9142
          name: tls-native
          protocol: TCP
        - containerPort: 7000
          name: internode
          protocol: TCP
        - containerPort: 7001
          name: tls-internode
          protocol: TCP
        - containerPort: 7199
          name: jmx
          protocol: TCP
        - containerPort: 8080
          name: mgmt-api-http
          protocol: TCP
        - containerPort: 9103
          name: prometheus
          protocol: TCP
        - containerPort: 9160
          name: thrift
          protocol: TCP
        readinessProbe:
          failureThreshold: 3
          httpGet:
            path: /api/v0/probes/readiness
            port: 8080
            scheme: HTTP
          initialDelaySeconds: 20
          periodSeconds: 10
          successThreshold: 1
          timeoutSeconds: 1
        resources:
          requests:
            cpu: "1"
            memory: 2Gi
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        volumeMounts:
        - mountPath: /var/log/cassandra
          name: server-logs
        - mountPath: /var/lib/cassandra
          name: server-data
        - mountPath: /etc/encryption/
          name: encryption-cred-storage
        - mountPath: /config
          name: server-config
      - image: k8ssandra/system-logger:v1.10.0
        imagePullPolicy: IfNotPresent
        name: server-system-logger
        resources:
          limits:
            cpu: 100m
            memory: 64M
          requests:
            cpu: 100m
            memory: 64M
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        volumeMounts:
        - mountPath: /var/log/cassandra
          name: server-logs
      dnsPolicy: ClusterFirst
      initContainers:
      - env:
        - name: POD_IP
          valueFrom:
            fieldRef:
              apiVersion: v1
              fieldPath: status.podIP
        - name: HOST_IP
          valueFrom:
            fieldRef:
              apiVersion: v1
              fieldPath: status.hostIP
        - name: USE_HOST_IP_FOR_BROADCAST
          value: "false"
        - name: RACK_NAME
          value: rack1
        - name: PRODUCT_VERSION
          value: 4.0.1
        - name: PRODUCT_NAME
          value: cassandra
        - name: DSE_VERSION
          value: 4.0.1
        - name: CONFIG_FILE_DATA
          value: '{"cassandra-yaml":{},"cluster-info":{"name":"development","seeds":"development-seed-service,development-dc1-additional-seed-service"},"datacenter-info":{"graph-enabled":0,"name":"dc1","solr-enabled":0,"spark-enabled":0}}'
        image: datastax/cass-config-builder:1.0.4-ubi7
        imagePullPolicy: IfNotPresent
        name: server-config-init
        resources:
          limits:
            cpu: "1"
            memory: 256M
          requests:
            cpu: "1"
            memory: 256M
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        volumeMounts:
        - mountPath: /config
          name: server-config
      restartPolicy: Always
      schedulerName: default-scheduler
      securityContext: {}
      serviceAccount: default
      serviceAccountName: default
      terminationGracePeriodSeconds: 120
      volumes:
      - emptyDir: {}
        name: server-config
      - emptyDir: {}
        name: server-logs
      - name: encryption-cred-storage
        secret:
          defaultMode: 420
          secretName: dc1-keystore
  updateStrategy:
    rollingUpdate:
      partition: 0
    type: RollingUpdate
  volumeClaimTemplates:
  - apiVersion: v1
    kind: PersistentVolumeClaim
    metadata:
      creationTimestamp: null
      labels:
        app.kubernetes.io/instance: cassandra-development
        app.kubernetes.io/managed-by: cass-operator
        app.kubernetes.io/name: cassandra
        app.kubernetes.io/version: 4.0.1
        cassandra.datastax.com/cluster: development
        cassandra.datastax.com/datacenter: dc1
        cassandra.datastax.com/rack: rack1
      name: server-data
    spec:
      accessModes:
      - ReadWriteOnce
      resources:
        requests:
          storage: 10Gi
      storageClassName: local-path
      volumeMode: Filesystem
    status:
      phase: Pending
status:
  collisionCount: 0
  currentReplicas: 1
  currentRevision: development-dc1-rack1-sts-7c7685d7cd
  observedGeneration: 2
  replicas: 1
  updateRevision: development-dc1-rack1-sts-7c7685d7cd
  updatedReplicas: 1

As there is nothing specified inside the security context it uses the uid/gid of the image.
That does not work, docker gives a better error msg here:
% docker run -it --rm k8ssandra/system-logger:v1.10.0
docker: Error response from daemon: unable to find user cassandra: no matching entries in passwd file.

I need to manually override the security context inside the datacenter, then it works:
  podTemplateSpec:
    spec:
      containers:
        - name: server-system-logger
          securityContext:
            runAsUser: 999
            runAsGroup: 999

I think I found the problem now, when setting:
dockerImageRunsAsCassandra: true
it works.

Yes, that is where I was digging as well just now in the logic.  It looks like the following was referenced and was preventing the sc.

Glad that worked for you.  Thank you for the details while digging into this configuration.  👍🏻

I just hit exact same issue:
helm install cass-operator k8ssandra/cass-operator -n cass-operator --create-namespace --set admissionWebhooks.enabled=false

apiVersion: cassandra.datastax.com/v1beta1
kind: CassandraDatacenter
metadata:
  name: dc1
spec:
  clusterName: development
  serverType: cassandra
  serverVersion: "4.1.2"
  managementApiAuth:
    insecure: {}
  size: 1
  storageConfig:
      cassandraDataVolumeClaimSpec:
        storageClassName: standard
        accessModes:
          - ReadWriteOnce
        resources:
          requests:
            storage: 1Gi
  podTemplateSpec:
    spec:
      containers:
        - name: "cassandra"
      securityContext: {}
  resources:
    requests:
      memory: 1Gi
      cpu: 100m
  racks:
    - name: rack1
  config:
    jvm-server-options:
      initial_heap_size: "1G"
      max_heap_size: "1G"
    cassandra-yaml:
      num_tokens: 16
      authenticator: PasswordAuthenticator
      authorizer: CassandraAuthorizer
      role_manager: CassandraRoleManager

Error: failed to create containerd container: mount callback failed on /var/lib/containerd/tmpmounts/containerd-mount1580039693: no users found

OK the following is working. Probably something is broken on this sample manifest: https://github.com/k8ssandra/cass-operator/blob/master/config/samples/example-cassdc-three-nodes-single-rack.yaml
apiVersion: cassandra.datastax.com/v1beta1
kind: CassandraDatacenter
metadata:
  name: dc1
spec:
  clusterName: development
  serverType: cassandra
  serverVersion: "4.1.2"
  dockerImageRunsAsCassandra: true
  managementApiAuth:
    insecure: {}
  size: 1
  storageConfig:
      cassandraDataVolumeClaimSpec:
        storageClassName: standard
        accessModes:
          - ReadWriteOnce
        resources:
          requests:
            storage: 1Gi
  podTemplateSpec:
    spec:
      containers:
        - name: "cassandra"
          securityContext:
            runAsUser: 999
            runAsGroup: 999
      securityContext:
        runAsUser: 999
        runAsGroup: 999
  resources:
    requests:
      memory: 1Gi
      cpu: 100m
  racks:
    - name: rack1
  config:
    jvm-server-options:
      initial_heap_size: "1G"
      max_heap_size: "1G"
    cassandra-yaml:
      num_tokens: 16
      authenticator: PasswordAuthenticator
      authorizer: CassandraAuthorizer
      role_manager: CassandraRoleManager


Remove 2 from MemoryPool2 and related types

This was merely an artifact from when this was the second of 2 pools

@davidfowl Rejoice!

FINALLY!


docs: fix links in CLI guides

Signed-off-by: prmellor<EMAIL_ADDRESS>Fixes links in the CLI guides

We need jira for windows docs.
rhoas login --token can login without browser but only to control plane.
zsh issue sound serious but it is unrelated to changes we made and best to be logged as separate issue in rhoas cli.
As for windows steps

Missing $ character in Checking the metadata information of an artifact section.

Requested changes:

One missing $ character.
Right link for json file.

@rkubis

We need jira for windows docs if that is missing.
rhoas login --token can login without browser but only to control plane.
zsh issue sound serious but it is unrelated to changes we made and best to be logged as separate issue in rhoas cli.

Thanks @wtrocki.
If @rkubis is happy with the changes in this PR, we can get it merged in.
Then we need to raise jira for:

Windows verification
Enabling command completion on Zsh


feat: deploy bootstrap hubs

What is the feature you would like to implement?
Set up three hubs which are the default bootstrap nodes for p2p network
Why is this feature important?
Bootstrap hubs kick start the network and provided a trusted starting point to discover peers on the network. They also maintain a copy of all data on the network in the early days when other hubs may not be operated reliably. This is a pre-requisite before allowing developers to publish messages to Hubs.
Will the protocol spec need to be updated??
No
How should this feature be built? (optional)

Deploy three Hubs on AWS using Fargate with an attached EFS volume for persistence
At least one Hub should be up at all times to avoid data loss (e.g. during deploys)
Configure all Hubs to use these hubs as the default bootstrap hubs.

@sds is going to specify this in more detail, specifically how the backend connects to hubs


Allow CI to pass by not running integration tests on Windows

The error message on macOS:
[ 24%] Built target loop_06
semantic error: String method not implemented: join
  --> /Users/runner/work/lpython-conda-testing/lpython-conda-testing/lpython/integration_tests/test_str_01.py:52:15
   |
[ 24%] Generating test_str_02.o
52 |     res:str = a.join(p)
[ 24%] Linking C executable if_03
   |               ^^^^^^^^^

Signals either mismatch of a test and a conda binary package, or a bug in lpython.

The log says:
HEAD is now at 2293972e5 Merge pull request #2334 from Shaikh-Ubaid/changes_from_lf

Which is the correct commit for v0.20.0. So it should be the matching test.

Which is the correct commit for v0.20.0. So it should be the matching test.

Yes, it should be matching the test. But it seems it does, which is probably why it fails.

str.join() was added recently. Unsure why the integration tests corresponding to the latest main are getting executed at the CI.

Something is wrong:
            git clone https://github.com/lcompilers/lpython.git
            git fetch https://github.com/lcompilers/lpython.git --tags -f

Shouldn't the second command be executed after cd lpython? I think you are checking out the tags into the root directory, which contains the https://github.com/lcompilers/lpython-conda-testing repository, then checking it out, all good. But then you cd to lpython, which has the latest main, so it is testing features that we don't have in conda yet.
Instead, the way it has to work is to cd lpython right after clone. Btw, there are flags you can pass to clone to get the tags, in fact they might be downloaded by default? Either way, do everything in that directory and git checkout v0.20.0 I think is fine, that way it's each to change.

Either way, do everything in that directory and git checkout v0.20.0 I think is fine, that way it's each to change.

Yes, fixed it.


Proposal: Fetch BTS network data through web server?

Solid-start enables us to run server side code and return the data to the client:
https://start.solidjs.com/api/createServerData
https://start.solidjs.com/api/server
We could plausibly use this to fetch data from the network.
e.g. on the explore page showing the block info, we could be fetching this data on the server and returning it to the user.
When broadcasting to the network however this would be performed client-side, to reduce MITM risks.
Advantages:

Reduction in duplicate requests to full node servers
Less client side code to run (faster, less downloads, more private).
Cache-able requests
Bearing in mind edge compute spec limits, we could pre-fetch data which doesn't change (e.g. market histories) to extend charts back several years.

Disadvantages:

A potential perception of less direct connection to the DEX
Full nodes may need to approve greater connection limits for the web wallet making many calls
Potentially a larger cloud computing bill as computations performed on server, rather than client-side.

Thoughts?

You can already do MITM by providing the client with wrong data.

True, I figure it could be worth it to reduce the duplicate network calls and client side external downloads


When I enable HTTPS and pushed, it said that "Gitea: Internal Server Connection Error."

Description
My CA cert is self-signed. I enabled HTTPS protocol by set " PROTOCOL = https                                                                       CERT_FILE = /var/lib/gitea/ssl/https.crt
KEY_FILE = /var/lib/gitea/ssl/https.key".
Git clone repos by https protocol is OK, but error used ssh protocol.Used ssh protocol to pull or push, It said that "Gitea: Internal Server Connection Error
fatal: Could not read from remote repository
Please make sure you have the correct access rights
and the repository exists."
And used https protocol to push, it said "
remote: Gitea: Internal Server Connection Error
To https://ip:3000/Sniventals/test.git
! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'https://ip:3000/Sniventals/test.git' "
Gitea Version
1.22.1
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
https://gist.github.com/Creeper-xie/208139810da9c2840f03a59282986211
Screenshots

"gp" alias to "git push"
Git Version
2.45.2
Operating System
Debian SID
How are you running Gitea?
I download gitea from this Github release.
https://github.com/go-gitea/gitea/releases/tag/v1.22.1
Database
PostgreSQL

@Creeper-xie when you set https, you must change ROOT_URL in app.ini too

Can you paste your configuration about ROOT_URL and LOCAL_ROOT_URL?

remote: Gitea: Internal Server Connection Error
To https://ip:3000/Sniventals/test.git
! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'https://ip:3000/Sniventals/test.git' "

I'm having the same issue after a fresh install...

@Creeper-xie @ccheath are you able to adjust the local_root_url to be http://localhost:3000 ?

@techknowlogick thanks for the reply... I did try that (and also <IP_ADDRESS>)... resolved the issue by checking the /etc/hosts and found that my localhost line was commented out (facepalm)

@ccheath glad to hear :)


[Java] Fix TestTimexRelativeConvert unit tests

Closed as the changes were merged with the PR #358

Closed as the changes were merged with the PR #358


python3Packages.python-lsp-server: fix test issues from 1.5.0 upgrade

Description of changes
Fixes issues building python-lsp-server since it was bumped to 1.5.0 in #181620 (see hydra build 185438708)
Changes:

The test_numpy_completion test is currently broken - see https://github.com/davidhalter/jedi/issues/1864
The test flags were moved from setup.cfg to pyproject.toml - see https://github.com/python-lsp/python-lsp-server/pull/207
A dependency on whatthepatch was added for yapf - see https://github.com/python-lsp/python-lsp-server/pull/136

The package python3Packages.spyder (which depends on this package) is still failing due to the python3Packages.qdarkstyle update, but that's unrelated to this PR (I might open another PR to fix that one, but I don't personally use it, unlike this one)
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

2 packages failed to build:

spyder (python310Packages.spyder)
python39Packages.spyder

14 packages built:

python310Packages.pyls-flake8
python310Packages.pyls-isort
python310Packages.pyls-spyder
python310Packages.pylsp-mypy
python310Packages.python-lsp-black
python310Packages.python-lsp-server
python310Packages.whatthepatch
python39Packages.pyls-flake8
python39Packages.pyls-isort
python39Packages.pyls-spyder
python39Packages.pylsp-mypy
python39Packages.python-lsp-black
python39Packages.python-lsp-server
python39Packages.whatthepatch

Oops I swear I searched this, but I missed #183862 somehow. Closing this PR in favor of that one


[FR] Support revocation date

Having a revokedAt field would allows to cheaply invalidate existing JWT tokens.
Steps to reproduce

Have your session cookies compromised, Like LTT recently did
Change your password

Expected behavior
Attacker should not be able to log in.
Actual behavior
JWT cookie will still be valid. So would the socketio reconnection key. Allowing the attacker to login to your account, despite having changed your password.
Context
I use SAML where cascading logouts are a core feature. You should be able to logout once, and have all your SAML apps be disconnected remotely.

Authentication-management handles changing the password during password resets. revokeJWTHook could be an after hook on the authentication service that blocks logging with JWT's created before revokedAt took.

I think it's more general as I could want to revoke tokens event without changing the user password, think about the case of users connected by OAuth or if I don't want to rely on the fact the user changes his password to ensure security. So for me these hooks could be at feathers authentication level as well to be used in others operations.
Sorry, but it's not yet clear what is the mechanism you'd like. The revokedAt field should be set on the user so that all tokens generated prior to it should be considered invalid ?
The easiest way to revoke tokens instantaneously is to change the secret used to generate it or some built-in fields like the issuer. So maybe we should use the encrypted password of the user to generate his tokens so that when changing the password they automatically become invalid. I am not an authentication expert so I wonder if it could be possible to generate/validate tokens with such a dynamic behaviour.

We can easily regenerate the bcrypt salt... as long as we have the user's password. We can't invalidate JWT's with salting or a revokation array without incurring a performance penalty before the call to the User service.
If we use revokedAt as a userField in the user service, it would be extremely cheap to revoke whenever OAuth/SAML, if the user asks us to, or if fishy behavior is detected.

revokeJWTHook could be an after hook on the authentication service that blocks logging-in with JWT's created before revokedAt timestamp.

Do you want to revoke an issued JWT? There is an example in the Feathers cookbook: https://feathersjs.com/cookbook/authentication/revoke-jwt.html
Or do you want to disable the login/authentication for a user in general? In that case I agree with @claustres that such a function should be implemented as an extension of the authentication service and not of this package. We do something similar with an after hook in the authentication service, where we check user.isActive and throw an authentication error if false.


More world fixes

I realized the import of many worlds with different versions of MineCraft and here are the problems I encountered :

Incorrect name of world :

The folder name can be different from the name of the world. This causes many problems for other functions WorldManager.
I 'm not the only one to notice the problem. ProjectWorlds directly changes the name of the world in level.dat file : https://github.com/trentech/ProjectWorlds/blob/79ef1142c180beaea39efbc42dd798a01770bd22/src/main/java/com/gmail/trentech/pjw/io/WorldData.java

UUID : NullPointerException
The function "registerWorldProperties" require the UUID.
Difficulty : NullPointerException
The difficulty of a world can be null.

Most bugs have already been fixed

"The folder name can be different from the name of the world. "

Intended as far as I'm aware. You can create worlds in single player with duplicate names, and a - is appended to the directory without changing the name, if I recall correctly.


JSX Styles from imported package not rendered on server

I have a package of reusable components that use styled-jsx that I import into my Next.js app (version 2.4.0).
The styles of the components in the package do not get rendered on the server and results in a flash of unstyled content on page load.
Is there an approach that's taken for cases like this?
My only hack has been to import the styled-jsx files separately into my app and add the style to the render using <styled-jsx>{style}</style> but that results in duplicate styles when the component's styles get attributed by the client.
Any help would be greatly appreciated

Hmm afaik this is because you're basically running 2 versions of styled-jsx cc @giuseppeg. I feel like  we might have to move this issue to the styled-jsx repo since it wouldn't only happen with next.js, but with any 2 packages both using styled-jsx.

@bnisbett are the components from the component library already transpiled with styled-jsx? If so we have an issue for that already #225

I think the issue mentioned by @giuseppeg is the right one.

@giuseppeg that's correct the component library is already transpiled with styled-jsx. I think the mentioned issue should cover my case as well.


Enhance the accessibility of DatePicker (@yamada-ui/calendar) component

Description
to be
1. Accessibility roles

[ ] Please refer to the image.

2. aria-controls

[ ] An element with the combobox role must have in its aria-controls attribute the ID of the corresponding popup element (the element with the dialog role).

3. aria-haspopup

[ ] You must set aria-haspopup="dialog" on elements with the combobox role.

4. aria-expanded

[ ] When an element with the dialog role is hidden, the element with the combobox role should have the aria-expanded attribute set to false.
[ ] When an element with the dialog role is visible, the element with the combobox role should have the aria-expanded attribute set to true.

5. aria-activedescendant

[ ] While a descendant element of the element with the dialog role is focused, the element with the combobox role must have the ID of the focused element as the value of its aria-activedescendant attribute.

6. aria-label / aria-labelledby (on the element with combobox role)

[ ] Please refer to the following flowchart.

Does the combobox have a visible label?
    ├──> No: `combobox` element has a label provided by `aria-label` .
    └──> Yes
           └──> Is the `combobox` element an HTML element that can be labelled using the HTML `label` element (e.g., the input element)?
                  ├──> No: `combobox` element has `aria-labelledby` set to a value that refers to the labelling element.
                  └──> Yes: `combobox` is labeled using the `label` element.

7. aria-modals

[ ] You must set aria-modal="true" on elements with the dialog role.

8. aria-live

[ ] Please refer to the image.

9. aria-selected

[ ] You must add aria-selected="true" to the cell of the currently selected date (which is implemented as a button element in Yamada UI).

Keyboard Interaction
When the focus is within the popup...

[ ] If possible, it would be better to clear the current value instead of accepting it when the Space key is pressed.

References

Date Picker Combobox Example | APG | WAI | W3C
DatePicker - Yamada UI
ARIA ライブリージョン - アクセシビリティ | MDN

Problem Statement/Justification
We should prioritize web accessibility to ensure our UI components are usable by everyone, including people with disabilities. This inclusivity enhances user experience and complies with legal standards, broadening Yamada UI's user base.
Proposed Solution or API
No response
Alternatives
No response
Are you willing to participate in implementing this feature and create a pull request include the implementation?
None
Additional Information
No response

私やります！

Sorry, the aria-activedescendant attribute is not needed when the popup is a dialog.

When a descendant of a listbox, grid, or tree popup is focused, DOM focus remains on the combobox and the combobox has aria-activedescendant set to a value that refers to the focused element within the popup.


Component error when attempting to delete a field - First Edit

When a field is deleted on the default GE template before any batch table columns are configured, it will no longer throw a component error
Critical Changes
Changes
Issues Closed
Fixes #5663
Community Ideas Delivered
New Metadata
Deleted Metadata

@dfuller114 & @ptewson-sfdo I took a quick look at this PR. I updated the syntax in the Issues Closed section so it'll fire the GitHub automation to close the Issue when this PR is merged and moved the description above the aggregation sections. The title of the Issue we closed will be explanation enough. Thanks!


wrong language type

https://github.com/kemin711/orpara
is a C++ project, or autoconf project
with some perl scripts

You can mark files detected as Groff and Shell as vendored using Linguist overrides.


[AutoPR resources/resource-manager] Resources: fix api-version from method property to client property

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7753

(message created by the CI based on PR content)


Make broadcast matchers composable.

This change enables creating compound assertions:
expect {
    publisher.send(:broadcast, :fizzbuzz, 12345)
    publisher.send(:broadcast, :fizzbuzz, 54321)
  }.to broadcast(:fizzbuzz, 12345).and broadcast(:fizzbuzz, 54321)

Looks good, could you update the README to show this new feature.
Not sure why CI is failing.

Tried running the specs from the master branch locally and they seem fine, how are them on your machine?

Sure. I also noticed that the current code does not work with RSpec 3.1. This block:
      matcher = ::RSpec::Mocks::ArgumentListMatcher.new(event_name.to_s, *expected_args)
      matcher.args_match?(*event_params)

Does not match empty arguments. I upgraded to 3.4 and this problem went away.

Specs run fine for me locally (Mac). Haven't tried them anywhere else.

Ummm... I have RSpec 3.4.1 locally. I'll have to try and figure out what is happening on CI.

I fixed the CI issue on master (latest bundler gets installed now). Can you rebase on master (or merge master) and force push your branch, hopefully CI will be green.

I'll just make the readme change too...

All done.

:+1:


Assets persistent loading animation

Summary
Wallet's transfer screen shows a loading animation that is not hidden once the assets list has been loaded.
Environments

Android OS v11
iOS v14.7

Version

App: 3.1.0-132
Wallet: 2.8.2-HF01

Steps to reproduce

Open the app.
Navigate to the Wallet screen.
Go to a wallet.
Then navigate to the transfer tab.

Current behavior
At the bottom of the assets list, a loading animation is always visible (screenshot).
Note: if you switch tabs the animation disappears
Expected behavior
The loading animation must be hidden once the assets have loaded.

Wallet issue


Validation rule unique:users

'sign_up' => [
    'validation_rules' => [
        'name' => 'required',
        'email' => 'required|email|unique:users',
        'password' => 'required'
    ]
],

Maybe add unique:users?

Why not :) feel free to open a PR if you want :)


Unexpected dash in CSS variable names

Given some tokens with a number in the name such as these:
{
  "space": {
    "$type": "dimension",
    "large": { "$value": { "value": 1.5, "unit": "rem" } },
    "x-large": { "$value": { "value": 2, "unit": "rem" } },
    "2x-large": { "$value": { "value": 3, "unit": "rem" } },
    "3x-large": { "$value": { "value": 4, "unit": "rem" } }
  },
}

Then the @terrazzo/plugin-css creates the following CSS variables:
:root {
  --space-large: 1.5rem;
  --space-x-large: 2rem;
  --space-2-x-large: 3rem;
  --space-3-x-large: 4rem;
}

I am surprised by the dash between the numbers and "x", which is added by the makeCSSVar function from @terrazzo/token-tools/css. Is this CSS naming behavior intended? I would have expected the following names:
:root {
  --space-large: 1.5rem;
  --space-x-large: 2rem;
  --space-2x-large: 3rem;
  --space-3x-large: 4rem;
}

Oh that’s a great callout. No intentional reason for that; just a minor change in the kebab-case library. I’ll make a fix for that and push a new version.
I’m thinking that probably numbers shouldn’t get dashes on either side (e.g. header2 and 2x)? 🤔 but don’t feel strongly.


GRADLE-2894: fixed bug that does not recognize the default wb resource if manually changed

I have found this bug on Stackoverflow question "http://stackoverflow.com/questions/22957776/gradle-eclipse-wtp-plugin-adding-wb-resource-removes-webappdir-from-deployment" and the documentation says
      //you can add a wb-resource elements; mandatory keys: 'sourcePath', 'deployPath':
      //if sourcePath points to non-existing folder it will *not* be added.
      resource sourcePath: 'extra/resource', deployPath: 'deployment/resource'

But this does override all the default entries and adds only the manually added ...
IMHO this is same problem as we have had with facets, see http://forums.gradle.org/gradle/topics/additional_eclipse_wtp_facet_overwrites_default_facets, which René changed a few days ago.

Merged, thanks


[TC-491] [TC-492] - deletes applicable staticdnsentries and jobs when a ds is deleted

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/43/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-traffic_ops-test-PR/45/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/83/