Live trading dies when CCXT request times out

Description of Issue
When live trading, a request to bittrex for the most recent data timed out, and killed my strategy.
I would expect a configurable number of retries, and maybe even configurable behavior on repeated failure. Stacktrace:
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 387, in _make_request
six.raise_from(e, None)
File "<string>", line 2, in raise_from
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 383, in _make_request
httplib_response = conn.getresponse()
File "/usr/local/lib/python3.6/http/client.py", line 1331, in getresponse
response.begin()
File "/usr/local/lib/python3.6/http/client.py", line 297, in begin
version, status, reason = self._read_status()
File "/usr/local/lib/python3.6/http/client.py", line 258, in _read_status
line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")
File "/usr/local/lib/python3.6/socket.py", line 586, in readinto
return self._sock.recv_into(b)
File "/usr/local/lib/python3.6/ssl.py", line 1009, in recv_into
return self.read(nbytes, buffer)
File "/usr/local/lib/python3.6/ssl.py", line 871, in read
return self._sslobj.read(len, buffer)
File "/usr/local/lib/python3.6/ssl.py", line 631, in read
v = self._sslobj.read(len, buffer)
socket.timeout: The read operation timed out

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/requests/adapters.py", line 440, in send
timeout=timeout
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 639, in urlopen
_stacktrace=sys.exc_info()[2])
File "/usr/local/lib/python3.6/site-packages/urllib3/util/retry.py", line 357, in increment
raise six.reraise(type(error), error, _stacktrace)
File "/usr/local/lib/python3.6/site-packages/urllib3/packages/six.py", line 686, in reraise
raise value
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 601, in urlopen
chunked=chunked)
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 389, in _make_request
self._raise_timeout(err=e, url=url, timeout_value=read_timeout)
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 309, in _raise_timeout
raise ReadTimeoutError(self, url, "Read timed out. (read timeout=%s)" % timeout_value)
urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='bittrex.com', port=443): Read timed out. (read timeout=10)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/ccxt/base/exchange.py", line 356, in fetch
proxies=self.proxies
File "/usr/local/lib/python3.6/site-packages/requests/sessions.py", line 508, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.6/site-packages/requests/sessions.py", line 618, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.6/site-packages/requests/adapters.py", line 521, in send
raise ReadTimeout(e, request=request)
requests.exceptions.ReadTimeout: HTTPSConnectionPool(host='bittrex.com', port=443): Read timed out. (read timeout=10)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/reactor/src/reactor/trading_algos/macd_signal.py", line 170, in <module>
stats_output='s3://reactor-dev')
File "/usr/local/lib/python3.6/site-packages/catalyst/utils/run_algo.py", line 596, in run_algorithm
stats_output=stats_output
File "/usr/local/lib/python3.6/site-packages/catalyst/utils/run_algo.py", line 358, in _run
overwrite_sim_params=False,
File "/usr/local/lib/python3.6/site-packages/catalyst/exchange/exchange_algorithm.py", line 1051, in run
data, overwrite_sim_params
File "/usr/local/lib/python3.6/site-packages/catalyst/exchange/exchange_algorithm.py", line 359, in run
data, overwrite_sim_params
File "/usr/local/lib/python3.6/site-packages/catalyst/algorithm.py", line 725, in run
for perf in self.get_generator():
File "/usr/local/lib/python3.6/site-packages/catalyst/gens/tradesimulation.py", line 224, in transform
for capital_change_packet in every_bar(dt):
File "/usr/local/lib/python3.6/site-packages/catalyst/gens/tradesimulation.py", line 137, in every_bar
handle_data(algo, current_data, dt_to_use)
File "/usr/local/lib/python3.6/site-packages/catalyst/utils/events.py", line 216, in handle_data
dt,
File "/usr/local/lib/python3.6/site-packages/catalyst/utils/events.py", line 235, in handle_data
self.callback(context, data)
File "/usr/local/lib/python3.6/site-packages/catalyst/exchange/exchange_algorithm.py", line 905, in handle_data
self._handle_data(self, data)
File "/reactor/src/reactor/trading_algos/macd_signal.py", line 39, in handle_data
frequency=context.frequency,
File "catalyst/_protocol.pyx", line 120, in catalyst._protocol.check_parameters.__call__.assert_keywords_and_call
File "catalyst/_protocol.pyx", line 633, in catalyst._protocol.BarData.history
File "/usr/local/lib/python3.6/site-packages/catalyst/exchange/exchange_data_portal.py", line 97, in get_history_window
ffill))
File "/usr/local/lib/python3.6/site-packages/redo/__init__.py", line 162, in retry
return action(*args, **kwargs)
File "/usr/local/lib/python3.6/site-packages/catalyst/exchange/exchange_data_portal.py", line 71, in _get_history_window
ffill)
File "/usr/local/lib/python3.6/site-packages/catalyst/exchange/exchange_data_portal.py", line 220, in get_exchange_history_window
False)
File "/usr/local/lib/python3.6/site-packages/catalyst/exchange/exchange.py", line 524, in get_history_window
end_dt=end_dt if not is_current else None,
File "/usr/local/lib/python3.6/site-packages/catalyst/exchange/ccxt/ccxt_exchange.py", line 462, in get_candles
params=
{}

File "/usr/local/lib/python3.6/site-packages/ccxt/bittrex.py", line 422, in fetch_ohlcv
response = self.v2GetMarketGetTicks(self.extend(request, params))
File "/usr/local/lib/python3.6/site-packages/ccxt/bittrex.py", line 684, in request
response = self.fetch2(path, api, method, params, headers, body)
File "/usr/local/lib/python3.6/site-packages/ccxt/base/exchange.py", line 302, in fetch2
return self.fetch(request['url'], request['method'], request['headers'], request['body'])
File "/usr/local/lib/python3.6/site-packages/ccxt/base/exchange.py", line 366, in fetch
self.raise_error(RequestTimeout, method, url, e)
File "/usr/local/lib/python3.6/site-packages/ccxt/base/exchange.py", line 287, in raise_error
raise exception_type(output)
ccxt.base.errors.RequestTimeout: bittrex GET https://bittrex.com/api/v2.0/pub/market/GetTicks?tickInterval=oneMin&marketName=USDT-ETH HTTPSConnectionPool(host='bittrex.com', port=443): Read timed out. (read timeout=10)```

## Reproduction Steps

1. Live trade an algorithm
2. Turn off your network connection

Hi @sawyercade ,
Thanks for reporting.
I believe this is duplication of #420 , which was fixed on develop branch and will be included in our next release.
I am closing this but feel free to reopen if you feel otherwise.
Thanks,
Lena


Fix calendar theme

Calendar appears in white color, even if we select dark mode themes. Need to check which attribute of ThemeData is used by datetime_picker for displaying the color here and set it to dark color accordingly
Note: The datetime_picker is displayed from within flutter_quill library. But it is reading one of the colors from ThemeData which is somehow white in all 3 themes.

hey @SankethBK can I start working on this issue?

/assign

Hi @ayato91 , assigning this to you. We don't have any bots configured yet

Please assign me this

@ayato91 can you please confirm if you're working on this

@ARP1208 we have some more open issues, see if you can pick any. You can raise new issues if you find any bug, or have ideas for enhancement or new features


Clean up pipes usage

React to several pipes usability issues that have since been fixed.

I figured out the issue with the flaky test #1379. It was only failing on Appveyor. Appveyor runs machines with very few cores and xunit limits the number of threads based on the number of cores. This test was blocking two different threads and waiting for a cancellation callback from a third thread. This has never failed locally or on our CI, but always failed on Appveyor.


Errors in generated files

Many unnamed enums are found in those files.

Operating system: Windows 11
The way you installed OpenCV: official binary distribution
OpenCV version: 4.7.0
rustc version: 1.68.0
Attach the full output of the following command from your project directory:
0.log

By the way, the volume G is of exFAT filesystem.

The output looks similar to the previous one.
1.log

I seem to have the same problem.
Operating system: Windows 10
The way you installed OpenCV: vcpgk
OpenCV version: 4.7.0
rustc version: 1.68.1
opencv-rs version 0.78
on a normal NTFS drive

It works when I switch to Clang 15.0.7, so I guess the interaction with Clang 16 is the culprit

Thanks for the insight, I'll check it out!

This should now be fixed in v0.78.2, thanks for the investigation!


structured_configuration beyond just eos_designs

Issue Type

[ ] Template enhancement
[X] Role enhancement
[ ] Documentation enhancement (avd.sh)

Is your feature request related to a problem? Please describe.
I personally use group_vars associated to a device's region, site, or function to inherit structured_configuration for eos_cli_config_gen with either passing l3ls_evpn fabric structured_config from eos_designs or when generating my own per device configurations.  It would be useful to capture all relevant keys that eos_cli_config_gen could use in facts/vars for a current device and output it to a structured_config output for tracking in git along with the actual config.  Unfortunately, the current structured_config output by eos_designs only captures keys that eos_designs currently supports, and not what eos_cli_config_gen supports.
Describe the solution you'd like
Create a structured_config output task for eos_cli_config_gen following similar logic to eos_designs.
Describe alternatives you've considered
Appending relevant keys to the avd structure_config fact and outputting via my own task.
Additional context
Add any other context or screenshots about the feature request here.

I have had similar thoughts, but with a slightly different use case. Detect eos_cli_config_gen-supported keys as part of eos_designs and combine with those instead of overwriting. This should lower the amount of needed custom_structured_configuration_ settings. These would all be included in structured_config.
Short term you can set your variables with a custom_structured_configuration_ prefix (customizable as well), and set the templates key to only run the template for custom_structured_configuration. This will make eos_designs only do what you need.
For the final solution, we will need to have (and maintain) a schema of some kind, to identify the supported keys. This is only on very early planning phase, since we need to take a balanced choice between maintainability, ease of use, robustness, data validations etc.

@ClausHolbechArista good idea.
I figured if custom_structured_configuration_<some key not set by eos_design> would not work since the key would not exist.  What am I missing here?

Keys set with custom_structured_configuration_ prefix will be carried over to the structured_config even if they are unused/unknown to eos_designs.

great!!!!!!

The only annoying thing is I need to loop through a the dictionary of base_config using dict2items to create a new var with { 'custom_structured_configuration_' ~ item.key :  item.value }}

Understood.
Try renaming base_config to structured_config.
This dict will also be combined into structured_configuration.


Openlimits should return the correct values for trading account balance on Nash

I've adjusted things so that it is now returning the trading balance and not personal balance, but we still need to compute the correct value for free. This will require changes to the nash-client dependency, so opening an issue here.

Fixed here: https://github.com/nash-io/openlimits/commit/af20a7c1a6944256f25f5c08c1b02eda597ce59e


Document !Unzip command

Have not found docs for the !Unzip command introduced in https://github.com/tailhook/vagga/pull/216.

ping @anti-social

Fixed by #350


ValidationProvider custom rule

Hi!
First of all this vee validate is awesome. I have added validation observer and provider and with custom controls and predefined rules like "required" works like charm. Now how can I add custom rules  to  ValidationProvider?
Thank you.
Current state
<template>
  <ValidationProvider rules="required" v-slot="{ errors }">
    <v-text-field
      class="non-caret"
      :id="id"
      :name="id"
      @keydown="isNumber"
      outline
      :append-icon="appendIcon"
      :label="label"
      v-model="model.Value"
      :error-messages="errors"
    ></v-text-field>
  </ValidationProvider>
</template>

Desired state
<template>
  <ValidationProvider rules="val_101" v-slot="{ errors }">
    <v-text-field
      class="non-caret"
      :id="id"
      :name="id"
      @keydown="isNumber"
      outline
      :append-icon="appendIcon"
      :label="label"
      v-model="model.Value"
      :error-messages="errors"
    ></v-text-field>
  </ValidationProvider>
</template>

Custom rule
  Validator.extend('val_101', {
        getMessage: field => validationMessage.VAL_101,
        validate: (input, args) => {
            return Rules.required.validate(input);
        }
    })

Extending the validator like you did should work correctly.
You might have missed that the required rule has a special behavior, it skips the validation if the field is empty. Meaning your rule won't run until the user types something in the field.
You can fix this by using :bails=false on the provider or marking your rule as a "require rule, mentioned here

:bails=false has solved it.
Thank you.


Import assignment cannot be used when targeting ECMAScript modules

I'm using screenfull.js in typescript project targeting ES2015.
After updating to v 5.0.0 I should make a change in import statement
You need to change import screenfull from 'screenfull'; to import screenfull = require('screenfull');
vscode mark this import with following error:
Import assignment cannot be used when targeting ECMAScript modules. Consider using 'import * as ns from "mod"', 'import {a} from "mod"', 'import d from "mod"', or another module format instead.

I cannot document all the possible config you can use with TS. The instructions are for default TS. Try to Google the error.

what is the default TS ?

import screenfull from 'screenfull'; still works but the only property that exists is isEnabled.
All of the functions are missing like .request, .exit, etc
If I keep import screenfull from 'screenfull';, and the following still works:
(screenfull as any).request()

That means Typescript is importing the actual code correctly but the typings are wrong.
I've yet to see someone recommend import screenfull = require('screenfull'); in any scenario, so not really sure what that is attempting to get around.
Generally in typescript you'd use const screenfull = require('screenfull') to import a module that doesn't have typings.

here is what I do after updating to version 5:
import screenfull, { Screenfull } from 'screenfull';

const fullScreen = (screenfull.isEnabled) ? screenfull as Screenfull : undefined;

So my constant fullScreen is either the correctly typed Screenfull or undefined:
now the use in typescript code is simple and intuitive, e.g.
if (fullScreen) {
  fullScreen.toggle();
  ..
}


Linking DNS Forwarding Ruleset with more than 10 virtual networks

I found it very surprising a DNS Forwarding Ruleset can be linked to up to 10 virtual networks.
How are we supposed to use DNS Private Resolver in hub and spoke model with more than 10 spokes?
We cannot create additional DNS Forwarding Rulesets as there also are limits of 2 DNS Forwarding Rulesets per outbound endpoint and 2 outbound endpoint per DNS Private Resolver.
Is 10 virtual network links per DNS Forwarding Ruleset a hard or a soft limit? Can it be increased via support request?
What is the current recommendation for using DNS Private Resolver in hub and spoke model with more than 10 spokes?

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

ID: b624746d-cab5-19da-f8f4-f467fb860a9e
Version Independent ID: b96e524d-80d2-b615-5216-43b911162b33
Content: Azure DNS Private Resolver endpoints and rulesets
Content Source: articles/dns/private-resolver-endpoints-rulesets.md
Service: dns
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@kamilzzz,
Thank you for your feedback. The team is currently working on increasing these limits the current ETA is in .
Hope this helps! Please let me know if you have any additional questions. Thank you!

Thanks for info!


Repo Question

@Ben305 is the Branch "feature/4" still needed / for what is|was it for?

That's a branch from @szokodiakos which never was completed. Contains some refactorings, but I didn't take a closer look yet

thanks for the clarification


https://karman-cms.herokuapp.com/ - 404

https://karman-cms.herokuapp.com/ - 404

Hey there, just saw this, sorry. We've had this on hold for a while and will probably not touch it again until the summer.
This is also in a very early development stage so it's not really something that you can reliably use - what were you interested in?
Anyway, the registration link is over at https://karman-cms.herokuapp.com/register. You'll receive an email to confirm your password which redirects you to the dashboard, but that's broken - a session won't be created and you need to actually login after confirming at https://karman-cms.herokuapp.com/login
Please let me know if you need further guidance!


Can't attack with kinetic blasts with 1.0.8

After updating to 1.0.8 it seems impossible to attack with any kinetic blasts or kinetic blades regardless of infusions.

Found the problem, 1.0.8 makes them spell like and I had a dip into instinctual warrior and the rage was preventing me from using blasts of any kind.


Problems with style

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
I try to use default theme 'material' and bootstrap.
But the table looks like this

Expected behavior
https://swimlane.github.io/ngx-datatable/#bootstrap
Reproduction of the problem
my angular.json
"styles": [
<EMAIL_ADDRESS>              "/node_modules/bootstrap/dist/css/bootstrap.css",
              "src/styles.css"
            ]

my component:
  <ngx-datatable
    class="material"
    [rows]="templateRows"
    [columns] = "columns"
    [columnMode]="'force'"
    >
   </ngx-datatable>
</div>

my root css:
@import<EMAIL_ADDRESS>@import<EMAIL_ADDRESS>@import<EMAIL_ADDRESS>
What is the motivation / use case for changing the behavior?
using the styles
Please tell us about your environment:
Operating system - ubuntu
package manager - 5.6

Table version: 0.8.x
in my json  "@swimlane/ngx-datatable": "^13.0.1"

Angular version: 2.0.x
6.0.3

Browser:  Chrome

Language: [all | TypeScript X.X | ES6/7 | ES5]

I've solved it ny rechecking all properties again.


experimental/argv_fuzzing: fixed argv parsing

AFAIK, argv-parsing was somewhat broken. All I could get was argv[0] or argv[1] with the full line of input (e.g. all parameters with spaces as argv[0]). We should fix this or at least propose working test file for exising implementation.

Without pull-request:
$ ../../afl-gcc test.c
$ cat test.txt | ./a.out
argc = 1
argv[0] = 'a.out'
With pull-request:
$ ../../afl-gcc test.c
$ cat test.txt | ./a.out
argc = 6
argv[0] = 'a.out'
argv[1] = 'param1'
argv[2] = '-t'
argv[3] = 'param3'
argv[4] = '--param=val'
argv[5] = 'param5'


Require multiple conditions

We should have a require selections.
Would be like this:

Any (IP/Email/Name) [Current/Default action]
Email & IP
Email & Username
Username & IP

Hi,
On my forum, I get assaulted by known spammers every day
This morning, there were 10 attempts to register that all had two or more spam data base hits
What I would like is to be able to set my registration parameters to not allow registering if the IP AND Email are in the spam data base.
There's no reason I would ever allow any of these attempts to become forum members
On my web site, I send IP's that are trying to abuse PHP scripts to a 404 page.
The attempts from that IP stop after that
User name is not important to me because the Name BOB is in the spam database, but if that user name is not taken, I would allow BOB to join the forum if his IP and Email were not in the spam data base.
My registration rule would be something like this
If the IP and Email are in the spam database, send the spammer to some other page
Maybe even report them so that the username gets tagged also in case that was not in the spam data base
Look at the screen shot from this mornings registration attempts

Forgot the screen shot

And here is the next mornings attempted registrations


Utility for generating QBJ rosters from spreadsheet

I need to document it better, but this helps generate files. If the team name is on the same line as the players, then you can copy and paste the values from the spreadsheet: https://www.quizbowlreader.com/createTournamentFile.html

I need to document it better, but this helps generate files. If the team name is on the same line as the players, then you can copy and paste the values from the spreadsheet: https://www.quizbowlreader.com/createTournamentFile.html

Would it be possible to add that link to the main page and/or the README?

The next version will have a link to that page in the From QBJ Registration tab. I won't make a separate utility for generating this from a spreadsheet, so I close this item when that goes in.


Bumping DeepChem to 1.2.0

Updating version to latest DeepChem.

I'm seeing some mysterious errors on Travis. Does anybody have any thoughts on what's going on?

Travis recently switched from Precise to Trusty as their default image, I'm guessing the update of libc is causing the problem. As to a fix... You may need to go look through the issues in the tensorflow conda-forge feedstock to see if someone has looked into this problem. Their linux builds go through CircleCI as I understand though, so they may not know about this problem yet.
I could of course, be totally wrong on what is happening though.

@Lnaden if you look carefully everything is actually getting built on a docker image
        docker run -e UPLOAD -e BINSTAR_TOKEN -e TRAVIS_PULL_REQUEST
            -t -i --rm -v `pwd`:/io jchodera/omnia-linux-anvil:texlive-amd30-cuda${CUDA_SHORT_VERSION}
            bash /io/devtools/docker-build.sh;

This docker image is running centos 6.9 with libc 2.12
(root) [root@f896f82eea3c /]# cat /etc/centos-release
CentOS release 6.9 (Final)
(root) [root@f896f82eea3c /]# ldd --version
ldd (GNU libc) 2.12

CentOS 9 stopped getting full support in May 2017 -- we should switch the anvil over to CentOS 7 which has a newer glibc (Although I understand that might be a very large undertaking).  I don't see how we can make this work with the current setup.

Right, the libc in the docker image would be the one used, not the travis image.
We actually will continue using the CentOS6 since our docker image is actually an extension of the conda-forge one, one that I personally spent quite a while getting up and running. As we are trying to move packages to conda-forge anyways, we want to stay as in-sync with them as we can. Also, Cent 6 EOL wont be until Nov 2020 anyways, so we have time that it will still be supported.
The part I find odd is that the tensorflow package is coming from conda forge, where it successfully built on their docker image, of which ours is an extension. As such, I don't see how libc is actually causing a problem, which is why I am very confused as to what is happening here.

https://github.com/conda-forge/tensorflow-feedstock/issues/6

Maybe you could try installing tensorflow from the defaults channel? Perhaps the anaconda people have solved this problem

conda-forge/tensorflow-feedstock#6

From that issue: "This comment says that indeed tensorflow pulls in a numpy wheel somehow. So that's pretty badly broken."
Ouch. Maybe you can give the feedstock maintainers a hand sorting that mess out, @lilleswing?


Fixing setting focus when RCTView is not yet in window

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[x] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary
I have a scenario where I send the focus command to a component from it's componentDidMount lifecycle, which leads native code to try to set focus to a RCTView with a zero frame (okay...) that isn't in a window (also seems unexpected). Not surprisingly, not being in a window means the view can't be focused. For comparison, I have the same behavior working as expected on the Win32\Office flavor of React Native.
I see there's existing code that tries to deal with this for a particular RCTBaseTextInputView, but nothing generically for RCTView. I'm trying to extend that to the RCTView. As part of that, I considered using the existing pending focus flag, but that seems suspect because we could have multiple views competing for focus -- so I am opting to change it to a single, weak ref for the currently pending focus view (which we clear if someone else grabs, or attempts to grab focus, or if the pending view later resigns focus, in addition to the scenario already handled viz. pending focus view gets focus).
Changelog

[macOS/iOS] [Changed] - Extend reactFocusIfNeeded to work with any generic RCTView
Test Plan
I have been able to validate these changes against 0.66 for my scenario.

Two questions:

This feels like something we could upstream to RN Core, now that iOS/iPadOS is more window aware. Maybe we can see what we can do to drive that..

Supposing we can't upstream to RN Core, any objection to keeping the reactFocusIfNeeded naming? My reason for that is I'd like to minimize diffs if possible. It looks like RN Core implemented that logic specifically for  since that's probably the only iOS control they actually care about focus for

Two questions:

This feels like something we could upstream to RN Core, now that iOS/iPadOS is more window aware. Maybe we can see what we can do to drive that..
Supposing we can't upstream to RN Core, any objection to keeping the reactFocusIfNeeded naming? My reason for that is I'd like to minimize diffs if possible. It looks like RN Core implemented that logic specifically for  since that's probably the only iOS control they actually care about focus for

EDIT: I see now you kept reactFocusIfNeeded, and reactViewDidMoveToWindow is on top of that. So better questions:

Can we upstream the "pendingFocusView" stuff
If not, we'll need to add more //TODO(macOS) tags to mark the diff.

How about I add the tags for now?
I need this to mitigate a11y concerns for my feature, which can't proceed without these fixes, which is also why for expediency I am making (proposing) these changes to the Microsoft fork.


[Bugfix] free error in typearray set value routine

Issue: #2143
JerryScript-DCO-1.0-Signed-off-by: Zidong Jiang<EMAIL_ADDRESS>

lgtm (the failed check is the mbed build which was moved to allowed failure group)


WorkerThreadPool: Polish yielding (fix corner case, remove misleading warning)

Intended to avoid a crash when a daemon task has already finished but it still exists and, before its awaited so it gets erased, notify_yield_over() is called with its id. The pool_thread_index would be -1. This PR handles such a situation gracefully.
Oh, and the warning is removed because it can legitly happen that a task notifies itself to stop yielding. A case of that is that a daemon task started yielding and thus started a collaborative wait in which it can run other tasks on top of its stack frame. Its correct that such an additional task wants to awake the underlying daemon.

Thanks!


mount static cephFS pvc via fuse inside Kubernetes pod with fsName=""

Hi! i need a little help. I cannot figure out  how to mount cephFS static pvc with empty fsName paramteter. I have a volume on a ceph cluster and directory i want to mount via ceph-fuse. It works fine on linux cli but i cannot mount the dir inside a pod. It looks like it is ok to have fsName defined as an empty string.
Example
This works fine on cli:
ceph-fuse -c /etc/ceph/ceph.conf -n client.user-rnd /mnt/ceph/rnd/test -r /rnd/test

PV manifest:
piVersion: v1
kind: PersistentVolume
metadata:
  name: ceph-rnd-test-pv
spec:
  accessModes:
  - ReadWriteMany
  capacity:
    storage: 1Gi
  csi:
    driver: cephfs.csi.ceph.com
    nodeStageSecretRef:
      # node stage secret name
      name: csi-cephfs-secret
      # node stage secret namespace where above secret is created
      namespace: default
    volumeAttributes:
      # Required options from storageclass parameters need to be added in volumeAttributes
      clusterID: "-cluster-id-"
      fsName: ""
      staticVolume: "true"
      rootPath: /rnd/test
      mounter: fuse
    # volumeHandle can be anything, need not to be same
    # as PV name or volume name. keeping same for brevity
    volumeHandle: ceph-rnd-test-pv
  persistentVolumeReclaimPolicy: Retain
  volumeMode: Filesystem

When i mount this PV inside a pod i get error:
MountVolume.MountDevice failed for volume "ceph-rnd-test-pv" : rpc error: code = Internal desc = rpc error: code = Internal desc = parameter 'fsName' cannot be empty

Environment details
ceph-csi chart version 3.8.1 (3.9.0)
kubernetes version: 1.26.6

Is it possible to set parameters such that option --client_mds_namespace= are not appended?

The default is to mount it through the kernel. Do you want to mount it through fuse through pv?

The default is to mount it through the kernel. Do you want to mount it through fuse through pv?

I want to be able to use it both ways. One problem is that options --client_mds_namespace is added implicitly and i don't know what value i should pass there. I have only settings that allow me mount ceph path via cephfs(example https://github.com/kubernetes/examples/tree/master/volumes/cephfs/): username, secret, monitors, path. Ceph-csi plugin requires more knowledge about ceph volume then  i may have. As an example I don't know what to put as value for fsName (i should probably ask a team that support ceph about that). My use case seems to more suitable for cephfs (deprecated). Having configuration options for cepfs seem not enough for using ceph-csi plugin.

@apetrunev what is the output of ceph fs ls

@Madhu-1
2024-01-12T10:11:59.610+0300 7f1548c1a700 -1 auth: unable to find a keyring on /etc/ceph/ceph.client.admin.keyring,/etc/ceph/ceph.keyring,/etc/ceph/keyring,/etc/ceph/keyring.bin,: (2) No such file or directory 2024-01-12T10:11:59.610+0300 7f1548c1a700 -1 AuthRegistry(0x7f154405ea88) no keyring found at /etc/ceph/ceph.client.admin.keyring,/etc/ceph/ceph.keyring,/etc/ceph/keyring,/etc/ceph/keyring.bin,, disabling cephx 2024-01-12T10:11:59.610+0300 7f1548c1a700 -1 auth: unable to find a keyring on /etc/ceph/ceph.client.admin.keyring,/etc/ceph/ceph.keyring,/etc/ceph/keyring,/etc/ceph/keyring.bin,: (2) No such file or directory 2024-01-12T10:11:59.610+0300 7f1548c1a700 -1 AuthRegistry(0x7f1548c19130) no keyring found at /etc/ceph/ceph.client.admin.keyring,/etc/ceph/ceph.keyring,/etc/ceph/keyring,/etc/ceph/keyring.bin,, disabling cephx 2024-01-12T10:11:59.746+0300 7f1541d9b700 -1 monclient(hunting): handle_auth_bad_method server allowed_methods [2] but i only support [1] 2024-01-12T10:12:02.746+0300 7f1542d9d700 -1 monclient(hunting): handle_auth_bad_method server allowed_methods [2] but i only support [1] 2024-01-12T10:12:02.790+0300 7f1548c1a700 -1 monclient: authenticate NOTE: no keyring found; disabled cephx authentication
I use unprivileged roles for myself and for accessing ceph dirs within kubernetes pods. These roles alose have access only to certain dirs.

@apetrunev that should be fine, can you use the output of ceph fs ls output (which is fsname in PV yaml )` that you will get when you run this command in your ceph cluster.

@apetrunev i made https://github.com/ceph/ceph-csi/pull/4372 to see if that is enough

@apetrunev that should be fine, can you use the output of ceph fs ls output (which is fsname in PV yaml )` that you will get when you run this command in your ceph cluster.

unfortunately i cannot access them. I got the fsName but did not test how it works because my  point here is to move  smoothly from kuber cephfs to ceph-csi with the same settings.

@apetrunev i made #4372 to see if that is enough

looks nice, that way --client_mds_namespace is not used. It seems that is enough for my case.


Add into_raw() / from_raw() methods

Would you be open to adding into_raw() and from_raw() methods to convert between Sender/Receiver values and raw pointers?  I think *mut () is correct, but any raw pointer is fine.
AFAICT, the underlying structs just store a pointer from Box::into_raw so this should be an easy to code.  I could probably open a PR if needed.  However, this does restrict what you can do with the structs in the future.
Use case: I'm a dev on the UniFFI project which generates FFI bindings for foreign languages.  I'm testing out using oneshot for async callback interface methods and so far it's working great, except when passing Sender across the FFI as a raw pointer.  My current code uses an extra Box, which isn't ideal given that Sender is already a wrapper around a leaked box pointer.

I can't currently see any reason why from/into raw would be bad. I understand your use case and double boxing seem not ideal indeed!
Please submit a PR and we can discuss it from there

Fixed in #29 and released as version 0.1.6


[fbrp] no escape

Description
Add fbrp.NoEscape to disable shell quoting for sub commands.
TODO: maybe rename it LateBinding, or something like that....
TODO: get it working with env keys
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[x] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[x] I want a thorough review of the implementation.
[ ] I want a high level review.
[ ] I want a deep design review.

Before and After
See fbrp/examples/07_rosmsg/fsetup.py
Testing
fbrp/examples/07_rosmsg/fsetup.py + fbrp/examples/09_cpp_ompl/fsetup.py + unit test
Checklist:

[x] I have performed manual end-to-end testing of the feature in my environment.
[ ] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[x] I have added tests that show that the PR is functional.
[x] New and existing unit tests pass locally with my changes.
[x] I have added relevant collaborators to review the PR before merge.
[ ] [Polymetis only] I have verified that all scripts in tests/scripts runs on hardware.

Codecov Report

Merging #831 (ec1b00f) into main (9833952) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #831      +/-   ##
==========================================
+ Coverage   32.66%   32.68%   +0.01%
==========================================
  Files         341      341
  Lines       33450    33450
==========================================
+ Hits        10928    10933       +5
+ Misses      22522    22517       -5

Impacted Files
Coverage Δ

agents/_agent.py
45.53% <0.00%> (-7.34%)
:arrow_down:

interpreter/craftassist/tasks.py
75.00% <0.00%> (+0.36%)
:arrow_up:

agents/craftassist/tests/fake_agent.py
73.67% <0.00%> (+0.69%)
:arrow_up:

agents/droidlet_agent.py
52.86% <0.00%> (+7.33%)
:arrow_up:


WIP Style: dark mode support

This has too many conflicts to ever fix. We can start scratch if we ever do this. Could be fun someday.


Download not resuming after app crash / force close

I've configured a AFURLSessionManager like this
- (AFURLSessionManager *)sharedManager {
    if (!_sharedManager) {
        NSString *sessionConfigurationIdentifier = [[NSBundle mainBundle] bundleIdentifier];
        NSURLSessionConfiguration *configuration = [NSURLSessionConfiguration backgroundSessionConfigurationWithIdentifier:sessionConfigurationIdentifier];
        [configuration setRequestCachePolicy:NSURLRequestReturnCacheDataElseLoad];
        [configuration setTimeoutIntervalForRequest:kDefaultRequestTimeoutInterval];
        [configuration setTimeoutIntervalForResource:0];
        [configuration setDiscretionary:YES];
        [configuration setSessionSendsLaunchEvents:YES];
        if ([UIDevice currentDevice].majorVersion >= 9) {
            [configuration setShouldUseExtendedBackgroundIdleMode:YES];
        }
        NSURLCache *cache = [NSURLCache sharedURLCache];
        [configuration setURLCache:cache];
        _sharedManager = [[AFURLSessionManager alloc] initWithSessionConfiguration:configuration];
    }
    return _sharedManager;
}

If i force close my app or it crashes, and I recreate the download, the download is not using cached data, but instead is restarting from 0 bytes

Interesting. Would you be able to push up a small example to demo this and let me debug it?
I also haven't used setShouldUseExtendedBackgroundIdleMode, so I don't know what that affects here. The docs are scarce, and the link in the header 404's :(

DownloadTest.zip
This should cover it

@kcharwood hi, it has been a year, but the problem still exists!

This issue persists in latest version. I am unable to resume any downloads. I correctly supply resume data, but all downloads are starting from 0


use docs hosting site to host html reports

would be nice to auto-generate some documents, add them to a github pages site and have a page / folder for people to add their auto-generated .html reports.

[x] choose a docs generation approach for the code (doxygen most likely candidate)
[x] update comments in code to match docs framework requirements
[x] generate some docs
[x] host it on github pages
[x] add version selector to docs so that we can track results over time / versions of executors
[ ] add instructions on how to contribute results if desired
[ ] upload .html .md reports as individual pages on docs site

[ ] add sub-folders for different platforms

For others reference, there now exists the ros2_realtime_benchmarks repo  that will host more than just the reference_system reports.
I think adding a link to this site within the docs of this repo will be sufficient to complete this issue.

Slight change to this: we should adjust the reporting scripts to generate markdown files with html embedded in them as the figures, similar to what performance_report does


[WIP] services/horizon: Add integration tests for Claimable Balances.

PR Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[x] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[x] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

What
Adds some tests for /claimable_balances that should be easily extendable. From #2939, this covers:

[x] Creating a claimable balance w/ native and non-native assets.
[x] Filtering by claimant, sponsor, asset, and ID (/claimable_balances/{id}).

Crucially, I haven't actually tried claiming balances yet. This only confirms that submission and filtering works.
See #2916 for how to run the tests. TL;DR:
cd ./services/horizon
HORIZON_INTEGRATION_TESTS=true HORIZON_BIN_DIR=$(pwd) go test -v ./internal/integration/protocol14_test.go

Why
We need tests for CAP23.
Known limitations
I can't really recommend using IntegrationConfig.SkipContainerCreation yet; it's very finicky and I need to clean up the various paths that diverge based on container state.

[x]  Filtering by claimant, sponsor, asset, and ID (/claimable_balances/{id}).

In https://github.com/stellar/go/pull/2998 I noticed that /claimable_balances/{id} doesn't work due to a bad query. I checked if we really test this endpoint here and I think we don't. Please add a request checking /claimable_balances/{id}. The fix for a query is simple, you can copy it here.

@Shaptic I created https://github.com/stellar/go/pull/2999 on top of your work here, once we merge this we can unblock the other PR (which is really simple)

I think that GetClaimableBalancesByID (a buggy method) is not used in /claimable_balances/{id} however we don't check /claimable_balances/{id} requests in this PR.

That URL is definitely hit in the tests, @bartekn, see runFilteringTest:
// Ensure we can look it up explicitly by ID
balance, err := client.ClaimableBalance(id)

I can see the request going out:
Requesting to http://localhost:8000/claimable_balances/000000009c513e7c0dddcb4d6b04a1e4725563a3447ef84f4a650d567caa957b34f04f4a

I even confirmed manually via cURL that it worked w/o the linked PR, so I'm not really sure what's going on with the bug in #2998?

OK @Shaptic, this is my mistake. The buggy function is used in state verifier only but I wrongly assumed it's used in action too. Also, when I was checking requests in your tests I was looking for ClaimableBalanceRequest (with ID field) but I forgot we have a helper for this. Sorry!

Going to fix the "claiming claimable balances" test in a separate PR in the interest of getting this one merged. (Circle failures are due to staticcheck not liking unrelated code, see here.)


add tcl parser

add tcl parser

Pull Request Test Coverage Report for Build 5940

44 of 107 (41.12%)  changed or added relevant lines in 27 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 72.358%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/src/main/java/io/shardingsphere/core/parsing/antlr/extractor/registry/impl/tcl/dialect/sqlserver/SQLServerSetVariableExtractor.java
3
4
75.0%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/dialect/oracle/sql/OracleSetVariableParser.java
0
1
0.0%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/dialect/postgresql/sql/PostgreSQLSetVariableParser.java
0
1
0.0%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/sql/tcl/begin/BeginParserFactory.java
0
1
0.0%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/sql/tcl/commit/CommitParserFactory.java
0
1
0.0%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/sql/tcl/rollback/RollbcakParserFactory.java
0
1
0.0%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/sql/tcl/begin/BeginParser.java
0
2
0.0%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/sql/tcl/commit/CommitParser.java
0
2
0.0%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/sql/tcl/rollback/RollbackParser.java
0
2
0.0%

sharding-core/src/main/java/io/shardingsphere/core/parsing/antlr/extractor/impl/dialect/mysql/MySQLAutoCommitExtractor.java
7
11
63.64%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/sql/SQLParserFactory.java
1
69.64%

Totals

Change from base Build 5939:
-0.3%

Covered Lines:
9646

Relevant Lines:
13331

💛  - Coveralls


Develop Channel CRD

Channel  official definitions: https://hyperledger-fabric.readthedocs.io/en/latest/channels.html#channels
Prerequisites before Channel

a active Federation
a active Network

After a Network created successfully, we can use Channel to build a real blockchain.
Typical operations on Channel:

create a new Channel
update a Channel  by proposal-vote

add a new member
remove a member

delete a Channel (not allowed)
member joins a Channel
member leaves a Channel

create a new channel
When network.initiator creates a channel with members,
channel controller will:

rbac reconcile

add get permission of this channel to all member's admin-cluster-role

blockchain reconcile

generate genesis block
call orderer api to create a channel with the genesis block

network controller will:

add this channel name to network's status

update a new channel
By  Proposal-Vote
delete a channel(not allowed)
TODO...
member joins a channel

deploy a new peer or select a existing peer
update peer.channels section to trigger channel join reconcile in peer controller

member leaves a channel
TODO...

For BAAS v0.1, we focus on :

channel create
channel update (member)
channel join (peer channels)

Subtasks:

[ ] channel create,recocnile rbac
[ ] add a new proposal to update(add/delete) channel members
[ ] add a new proposal to dissolve channel

Channel creations process has been changed after v2.2
How this process differs from the legacy Fabric v2.2 process:

System channel no longer required: Besides the creation of the system channel representing an extra step (as compared to the new process), this system channel created an extra layer of administration that, for some use cases, provided no tangible benefit.
Consortium no longer required: You no longer need to define the set of organizations, known as the “consortium”, who are permitted to create channels on a particular ordering service. With this new process, all channels are application channels, so the concept of a list of organizations who can create channels no longer applies. Any set of organizations can get together and create a channel using a defined set of ordering nodes (which become the ordering service of that channel).
Simplified ordering node creation process: Because a system channel genesis block no longer needs to exist before an ordering node is created, admins can now focus on the process of setting up their infrastructure and making sure their node is working correctly before joining a particular application channel.

We use same orderering service under a network,so

orderer nodes
application channel


for next version explain message ID NA's

consider editing methods to explain events with message ID NA's were added post-cruise with 'additions' file

and for CTD deploy (noted in IMS repo with regard to retaining event identifiers)


Issue 428: Remove sh:minCounts to support minimal object definitions

This Pull Request resolves all requirements of Issue #428 .
Review steps taken:

[x] Pull request is against correct branch
[ ] CI passes in (CASE/UCO) feature branch
[ ] CI passes in (CASE/UCO) current unstable branch (merge-commit)
[ ] Impact on SHACL validation reviewed for CASE-Examples
[ ] Impact on SHACL validation remediated for CASE-Examples
[ ] Impact on SHACL validation reviewed for casework.github.io
[ ] Impact on SHACL validation remediated for casework.github.io

@eoghanscasey , fyi: I did not apply the observable:message change in MessageThreadFacet because by the time this branch was started, observable:message no longer existed.  It was removed as part of #393 .


add validations for the remaining enum values

Problem Statement
Currently there are some missing enum validations which can add wrong values to those specific enum fields by user.
Related Issue
Fixes #2820
Proposed Changes
Add +kubebuilder:validation:Enum tags to the enum fields.
Checklist

[ ] I have read the contribution guidelines
[ ] All commits are signed with git commit --signoff
[ ] My changes have reasonable test coverage
[ ] All tests pass with make test
[ ] I ensured my PR is ready for review with make reviewable

hey, thank you for this! Could you please run make reviewable, that'll take care of generating the CRDs etc. and should fix the failing CI :pray:

In addition to running make reviewable, would you commit with --sign flag? Please check the DCO job for details!

All requested changes are done! please review again 👍

@shuheiktgw can you review and approve so that we can close this PR, thanks

/ok-to-test sha=dc248d3dcebca8ab17bce246e3d7662d27f7c549

/approve

Thank you for the contributions @lakhanjindam !


Update course details

update course details page

Looking great so far, @lmkeston! Some suggestions:

for the "what you'll build" section might be a GIF of the migrating interface, or a screenshot of the importer page. I'd be excited if I skimmed through the wall of text but saw that screenshot or GIF.
we may want to add up to 5 tags in config.yml
I also had a course with no "projects used" -- I'm OK leaving it out altogether but here's what I did in case it helps


Adding new air and ground AFRF units

Adding more ground and air units

Sorry, no third party scripts.

Oh I only intended to submit first commit, but with all the merging other code got proposed too.


[Angular] Dependency not resolving

Issue details
Component dependencies are not being resolved.
Expected: store dependency to be resolved.
Instead, this error is thrown:
Can't resolve all parameters for MyComponent: (?)

Steps to reproduce
Snippets below
Please specify which version of Storybook and optionally any affected addons that you're running

@storybook/angular: 3.4.0-alpha.7

Affected platforms

Angular 5.2.3

Screenshots / Screencast / Code Snippets (Optional)
// Component
import { Component } from '@angular/core';
import { Store } from '@ngrx/store';

@Component({
    selector: 'my-comp',
    templateUrl: './my-comp.component.html',
})
export class MyComponent {

    constructor(
      store: Store<any>,
    ) {
        console.log(store, store === undefined); // Response: `undefined true`
    }

}

// Story
import { storiesOf, moduleMetadata } from '@storybook/angular';
import { StoreModule } from '@ngrx/store';

import { MyComponent } from './my-component.component';

storiesOf('MyComponent', module)
    .addDecorator(moduleMetadata({
        imports: [
            StoreModule.forRoot({}),
        ],
        declarations: [
            MyComponent,
        ],
    }))
    .add('MyComponent', () => {
        return {
            component: MyComponent,
        };
    });

I've added an example usage with the @ngrx/store. You can take a look here, it's injected.
Maybe you can try the latest 3.4.0-rc2 release ?

Hey,
Yes, the issue is solved in the newest rc version! Thanks!
Sorry for not replying faster.


Adding label slot to input

I wanted to be able to add an SVG icon inside of the <label>. I could do that with the <slot #description> but then it was a bit of a fiddle depending on amount of text etc. What do you think?
I'm loving Vuetensilis btw. I feel like I might have a bunch of small ideas like this. Are you happy me creating small PRs or would you prefer for me to create an issue first to ask about it?

Yeah, I've seen this sort of UI before and unfortunately it cannot currently support this sort of thing. But I would like to add some sort of support for it. Im thinking I want to eventually add a default scoped slot that let's you do whatever you want with the layout and provides the bindings and listeners you need. The tricky part is when you give up control over markup, you give up a lot of the semantic/accessibility controls too.

Please feel free to make issues and/or PRs are you like. I'm the only maintainer at the moment and have a lot going on so it's not the fastest moving project, but I dont want to continue moving it forward :)

I realised this would be cool also to use an SVG as indicator for select input.  Is there anything I could do to get this in or is it more right now you don't want to add this feature?
Agree about what happens when you give up control of the markup. I made a JS a11y accordion plugin 4ish years ago. It got forked for Vue which was cool but then the markup was DESTROYED with divs...
I'd be happy to help with some small bits. I want build a pattern library around it. Your Inspiration section SOLD me!

I think the best solution right now to support that would be to have a couple other named slots like "before-input" and "after-input". It's not my favorite solution, but without the completely flexible default slot that I have in mind, it may be the best thing.

I moved the slot for the radio buttons to inside <legend>
I made sure slot is imediatly after the <span class="vts-input__text">

I think awesome suggestion! Ensuring there is a label makes sense to me :)

PMFJI but I think this is a pure stylistic problem that can be solved with CSS alone and not in Vuetensils.
We added SVG backgrounds on the aria-valid states (here incl. :focus outline)

the date picker with the icon flipped based on state (dates are :placeholder text)

just my 2ct

@WebMechanic you're right for some cases, but working with DOM nodes is a lot more convenient in other cases. With content in the DOM, it's easier to inherit colors or do animations.
@stuartjnelson sorry for the delay on this PR. I think I left it with some change requests for you. Did you still want to get it merged? If not, I think I can tackle it myself.
Also side note, may I ask where you all are using Vuetensils?

@AustinGil I'm in Germany working on a big relaunch of our real estate portal.
More here to keep this issue clean.

Was .../VMultiselect/Autocomplete.vue supposed to be there? It looks "alien" and WIP-ish. Also uses loads of atomic utility classes.

@WebMechanic yeah, that was a placholder. I think it would be a great addition to the library, but it's a very complicated component to build. I've got this from another example.

@stuartjnelson did you see my responses to this PR? Do you want me to include those changes or do you have thoughts on them?

Closing this for now. May add it to the next release separately.


'epicapi' is not defined

Hello,
I believe I've followed the steps correctly, but I get this crash when attempting to load my game:
`[code]
I'm sorry, but an uncaught exception occurred.
While running game code:
File "game/epic.rpy", line 21, in script
init -1499 python:
File "game/epic.rpy", line 22, in
achievement.backends.insert(0, epicapi.EpicBackend())
NameError: name 'epicapi' is not defined
-- Full Traceback ------------------------------------------------------------
Full traceback:
File "game/epic.rpy", line 21, in script
init -1499 python:
File "/Applications/renpy-7.6.3-sdk/renpy/ast.py", line 1138, in execute
renpy.python.py_exec_bytecode(self.code.bytecode, self.hide, store=self.store)
File "/Applications/renpy-7.6.3-sdk/renpy/python.py", line 1122, in py_exec_bytecode
exec(bytecode, globals, locals)
File "game/epic.rpy", line 22, in
achievement.backends.insert(0, epicapi.EpicBackend())
NameError: name 'epicapi' is not defined
Darwin-23.1.0-arm64-arm-64bit arm64
Ren'Py <IP_ADDRESS>91805
VIDEOVERSE <IP_ADDRESS>
Tue Jan 23 15:14:37 2024
[/code]
`

Hi,
Thank you for your interest in the project.
This error should only occur if the extension was not loaded properly.
Could you check that you have a file called epic_eos.rpe in the directory where you created epic.rpy ?

Hello, thanks for your reply. It was in the wrong folder and now it works. Thank you for the help!


Down Edited Agenda

https://gatech.instructure.com/courses/279284/assignments/1167660

[ ] submitted

Since this is a recurring assignment idk if we should create a new issue for this every week. Idk if we want to to do that and just stick the due date on the title also or have a heinous ongoing thread for this.

I see you, but I won't do your agenda for your ;-)
No need to assign tickets to me.


Add support for www.domain.com

Did you check the FAQ & Troubleshooting section for answers to common questions and issues?

Yes

Describe the issue

www.mydomain.com is not working with current build
mydomain.com is working. Just not if added www

Diagnostic Information

==== Docker Information ====
Detected runtime: docker (Docker version 20.10.24+dfsg1, build 297e128)
Detected compose: docker-compose (docker-compose version 1.29.2, build unknown)
Runtime state: OK

WARNING: Your version of Docker Compose is outdated and unsupported.
Docker Compose v2 has been Generally Available (GA) for over 1 year,
and as of June 2023, Docker Compose v1 has been officially deprecated
by Docker Inc.
https://www.docker.com/blog/new-docker-compose-v2-and-v1-deprecation/
Popular Linux distributions, such as Debian and Ubuntu, are still distributing
outdated and unofficial packages of Docker and Docker Compose.
However, those packages are neither supported nor endorsed by Docker Inc.
Lemmy-Easy-Deploy might still work regardless, but testing is only done
with Docker Compose v2. Compose v1 is not supported.
For the best experience, please install the official version of Docker:
https://docs.docker.com/engine/install/
==== System Information ====
OS: Linux
KERNEL: 6.1.0-9-cloud-amd64 (x86_64)
HOSTNAME: OK
SHELL: bash
MEMORY:
total        used        free      shared  buff/cache   available
Mem:           7.7Gi       1.1Gi       2.4Gi        70Mi       4.6Gi       6.6Gi
Swap:             0B          0B          0B
DISTRO:
PRETTY_NAME="Debian GNU/Linux 12 (bookworm)"
NAME="Debian GNU/Linux"
VERSION_CODENAME=bookworm
==== Lemmy-Easy-Deploy Information ====
Version: 1.2.8
IMAGE                                CREATED       STATUS
caddy:latest                         7 hours ago   Up 7 hours
ghcr.io/ubergeek77/lemmy-ui:0.18.2   7 hours ago   Up 7 hours
ghcr.io/ubergeek77/lemmy:0.18.2      7 hours ago   Up 7 hours
asonix/pictrs:0.4.0                  7 hours ago   Up 7 hours
postgres:15-alpine                   7 hours ago   Up 7 hours
Integrity:
074ae41957936daac883c66bcac0ca12093343ef5e923752689b936b3c6b1b25  ./deploy.sh
1e9b0c0988998dcc33cb0fbfdb0e1679229424e724f898b797380adc7d102446  ./templates/Caddy-Dockerfile.template
c1202e70662dd2228da36a35a0f38ec8fc81bec8964d7315d02e8671a58dd7d7  ./templates/Caddyfile.template
2537678c7971df36c1ed95f4228d3cfcb15bb4a28a60d939eaf8dd75b5d64a36  ./templates/cloudflare.snip
c494a610bcb4cd1cfc0a4fe4fb0f6d437b2a84a0ad1625daee240e6dd6f1c910  ./templates/compose-email-volumes.snip
c9cb4c5fee12930e17798a02ae1bd12e2dc69e149a394c24511bc9d4e6b776d4  ./templates/compose-email.snip
f5325a9e26b29da51c6d3295aa278ff08ce71ffd2cd63dc4bebf00e54c468899  ./templates/docker-compose.yml.template
1c202b1b6e87c65b2fcda6035c9fe3f8631d76662907ffd38f24b14686e30647  ./templates/lemmy-email.snip
c834cdce9eaf77f38155b404724fdfe66845575386ee516987452aa715642a6f  ./templates/lemmy.hjson.template
Custom Files:
*** No custom files ***
==== Settings ====
CLOUDFLARE: No
CADDY_DISABLE_TLS: false
CADDY_HTTP_PORT: 80
CADDY_HTTPS_PORT: 443
LEMMY_TLS_ENABLED: true
ENABLE_EMAIL: false
SMTP_PORT: 25
ENABLE_POSTFIX: false
POSTGRES_POOL_SIZE: 5
==== Generated Files ====
Deploy Version: 0.18.2;0.18.2
total 44K
drwxr-xr-x  2 0 0 4.0K Jul 22 13:10 caddy
-rw-r--r--  1 0 0   30 Jul 22 13:13 caddy.env
-rw-r--r--  1 0 0 1.7K Jul 22 13:13 docker-compose.yml
drwxr-xr-x 14 0 0 4.0K Jul 22 13:13 lemmy
drwxr-xr-x  7 0 0 4.0K Jul 22 13:13 lemmy-ui
-rw-r--r--  1 0 0   50 Jul 19 22:49 lemmy.env
-rw-r--r--  1 0 0  471 Jul 22 13:13 lemmy.hjson
-rw-r--r--  1 0 0   49 Jul 19 22:49 pictrs.env
-rw-r--r--  1 0 0   34 Jul 22 13:13 postfix.env
-rw-r--r--  1 0 0   51 Jul 19 22:49 postgres.env
-rw-r--r--  1 0 0   14 Jul 22 06:17 version

Sorry, but this is not an issue in Lemmy-Easy-Deploy, and technically speaking it isn't really an issue.
The internet does not use www anymore. In the modern internet, www.domain.com is no different than any other subdomain. You are essentially trying to have Lemmy accessible from two distinct domains.
Lemmy only supports being available at a single domain, so I cannot support situations where the user wants to use two domains at once, www or not.

Maybe an option for a redirect? If I put in for example "domain" and press ctrl enter on my browser, it will automatically make it "www.domain.com" . There are several shortcuts still present in pretty much all browsers to this day to automatically complete for certain domain types. That's at least one of the reasons why pretty much all websites still support it.

Lemmy fundamentally does not support being accessed from different domains. This is just how ActivityPub works.
A redirect could work, but it's an advanced use case with little benefit, and unfortunately I will not be adding an option for this. Please just use the regular domain.
If you must have www for some reason, you can check out the Advanced Configuration page on this repo and modify the Caddyfile template accordingly 👍

Will do. Thank you


Stop training to test

hi
When I was training, I found that after 300 epoch, my dice was basically unchanged, and loss also converged. I wanted to stop training for testing, how should I modify it? Except for setting the training epoch to 300.Is there another way? For example, can I use the results in the middle of training to predict? How do I change the code?

Hi,
I believe you can just do Ctrl+c to stop the training. The model with the best validation score is saved as model_best.model in the saved folder so you can just pass that to the argument -chk for the nnUNet_predict command like this:
nnUNet_predict -i <input_folder> -o <output_folder1> -t <TASK_ID> -m 3d_fullres -tr nnUNetTrainerV2BraTSRegions_DA4_BN_BD --save_npz -chk model_best

I hope this works for you!


Implement mobile layout

Had to duplicate quote rating panel—don't see a feasible way to move it around using pure CSS.

I approve of this markup.

And so do I.

I've finished with this review for now. It looks okay, but still need some polishing. @Minoru, your turn.

Thanks, good work!


Automatic deletion of questions of bots

Sir, I already had a talk with you in Dm, So Probably, You already know what this issue is related to.
Some of the key words are :

Varishkaan
Specialist
Love Specialist
love m-a-r-r-i-a-g-e s-p-e-c-i-a-l-i-s-t
Black Magic
bAbAji
Love Spells
http://www.orignalsupplements.com
https://simplyevents.io/
Love marriage problem solutions
Husband wife problem solutions
Get your lost Love back by astrology

Thanks Sir :beers:

This feature is also requested by Magda from brainly.lat
Their pattern is "find questions contains (whatsapp or telephone) and numbers start with (9 or 6) and must be 9 digits"

..and also it'll be available for the brainly.lat


Small typo in documentation

The documentation for style matching incorrectly suggests returning undefined for a failed match. A few lines below, the spec for getAttrs states: When it returns false, the rule won't match, whereas returning undefined assumes a match with default attributes.

Thank you for spotting that!


Revert "My first branch"

Reverts Exp-Intro-to-GitHub-Flow-Cohort-1/series-intro-to-github-flow-patali09#1

merged successfully


convoluation1D input_shape questions

The input_shape was predefined as (samples, steps, input_dim). What is the meaning of steps here? And along which dimension will the linear convolution operation perform?
I have a training data with the shape (samples, dims). I want to perform convolution operations along the dims axis. How can I define the input_shape for the convolution1D as the first layer?
Thanks
Please make sure that the boxes below are checked before you submit your issue. Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[x] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Convolution1D will be performed along steps. Steps correspond to the length of 1D sequence, 'steps' are 1D variant of Convolution2D nb_rows, nb_cols.
If in your case the sequence has a length of 'dims', you should define input_shape as (samples, dims, 1) - if you have only one feature.

@lukovkin Thanks for your clear answer.


fix: only numbers allowed in year of birth

##2474
Description what actually was done in task:
I added a keydown method to only accept numeric values for the year of birth input on both the registration and profile pages.

@iajaymk thank you for contribution!


Change current naming convention to refelct closer to Azure reccomendations

Current naming conventions rely on unique ID generated from a deployment, which if no name is given to the deployment via --name ARM will use the file name, so in case of MLZ this will always be MLZ which would be used as seed to semi random name.
Work to add:

A recommended naming approach is :
--<spoke name/type>-

example:
rg-production-hub-usv  where 'production' was supplied as deployment name or prefix variable. If neither are given it will default to  file name being deployed in this case mlz.

Create unique ID schema that is repeatable for idempotency but also random and unique enough to allow multiple deployments. While uniqueness is easily added by using utc now as the seed for unique string it is not repeatable enough for idempotency. When making it repeatable it can be and will be repeated so collisions are inevitable.
Document naming convention and repeatability challenges to prevent collisions by uses testing and in POCs.
Eleviate naming challanges as much as possible in resource by including where possible the use of lower(), take() and substring() along with newGuid(), uniqueString() and base64(). Also using a basic understanding of naming limitations in Azure located here https://docs.microsoft.com/en-us/azure/azure-resource-manager/management/resource-name-rules

I agree we should calculate names. But, I think we should do that from a user provided argument without a default like param resourcePrefix string that forces the value to be provided upon deployment.
I think we should move away from deployment name as a seed for any value since it's a property of the deployment resource itself and not something MLZ-specific. At the command line, users don't have to provide that value. From the Portal, a deployment name is generated with a NoTemplate-UTC ish value or a CustomTemplate-UTCish value.

With an eye on the easy button (least amount of input required) we need to offer a prefix/postfix that allow customers to reflect their environment.  This is keeping in line with best practices so an option is to allow workload/environment like mlzprod (1) or mlzstage(1) or saca-poc(1) that is a single or multiple of additional required parameters, that is then used in part of naming of RG level resources. Again inline with best practices it looks like -<prefix/postfix>-<MLZ segregation (hub, operations...)>-. This can be what's used for the seed to unique(). This is repeatable which is required for idempotency moving forward which in turn means it provides collision possibilities (that is the cost of repeatable).
In the above example it would be advisable for customers deploying multiple MLZs to same region and using the same naming convention add a number to represent another incremental install. If deploying to different region they can continue to use the same name, for example mlzprod since all other resource naming is regionalized.
Its important to note that if you do not use deployment it will simply take on the name of the file, in our case MLZ. If you try to deploy first in one region without --name it works, but if you try to deploy to another region the same deployment it fails with {"error":{"code":"InvalidDeploymentLocation","message":"Invalid deployment location 'usgovtexas'. The deployment 'main' already exists in location 'usgovvirginia'."}}  This is true if you repeat the deployment name like --name shawngibbs002  so documentation and examples are key requirement. FYI: notice the use --only-show-errors to get simplified error returned :)
az deployment sub create --location usgovtexas --template-file ./src/bicep/mlz.bicep --name shawngibbs02 --only-show-errors {"error":{"code":"InvalidDeploymentLocation","message":"Invalid deployment location 'usgovtexas'. The deployment 'shawngibbs02' already exists in location 'usgovvirginia'."}}

We will break this down into smaller pieces of work:

The name property on modules can exceed deployment name length validation
Resource name length validation
Change the way we generate names: Resource type, environment, location incorporated into names, and lowercase everything. Remove deployment name from any unique seed values and determine how to generate unique names

Based on the discussion, here's a proposed scope for this work in terms of acceptance criteria:
Acceptance Criteria

All resource names are in lower case
resourcePrefix is the only required parameter and does not have a default value
Documentation is added in Bicep  to explain that resourcePrefix is used for providing uniqueness as a seed value.
An additional optional parameter is defined as resourcePostfix which allows deploying into the same subscriptions but different regions.
The seed for unique() calculations is the concatenation of resourcePrefix and resourcePostfix.
Deployment name (deployment().name) is not used anywhere in the Bicep code
Naming guidelines for resource abbreviations are implemented
A naming convention is implemented. Note: The linked document has examples and should not be used as a set of requirements.

resourcePrefix is the only required parameter and does not have a default value
I don't like this acceptance criteria above because currently resourcePrefix is not a required parameter, and it defaults to "mlz". However, if we implement the uniqueness requirements defined here then we would have to do a different unique() calculation - it could not be based on the default value of resourcePrefix.

My preference would be to leave resourcePrefix as an optional parameter and add another acceptance criteria like this:

If a value for resourcePrefix is not provided, then unique() calculations are done based on utcNow and a warning is emitted that notifies the user that re-deployments will cause new resources to be generated unless a value is provided for resourcePrefix  and optionally resourcePostfix.

I created #549 to represent the work described in this issue. Closing this one.


Update RxSwift to beta.2

Sorry, duplicate.

Hi @moogle19 ,
sorry, this was dupe :) Thnx for helping out, but can't merge 2 of them :)

Hi @kzaher ,
yeah I saw that too, but #52 forgot to update the Cartfile (still at beta1).
Would be nice if you would update it, so carthage builds will work as well.

Hi @moogle19 ,
oh, I'll need to update Cartfile also, forgot about that.


refactor tests and plugin infrastructure

plugins return a list of GeneratedFiles
restructured tests in test/plugins. Usage of pytest parametrize feature to avoid code duplications and it's easier to extend for future plugins

First of all: Great refactor on the test suite!!! This is super neat! :heart: :smiley:
Concerning generating the __init__.py, I have concerns. It enforces an opinionated way of how __init__.py must look like. But what if a user wants to actually have content in the __init__.py?
Additionally, nested dir structures will still suffer from this issue.
E.g. lets assume this nested dir structure:
.
└── a
    └── b
        └── query.gql

This will require the following __init__.pys:
.
└── a
    ├── b
    │   ├── __init__.py
    │   └── query.gql
    └── __init__.py

Every directory on the way to the query.gql needs an __init__.py.
I.e., even with this change, it won't necessarily save the user from tackling __init__.py issues and on top it will introduce an opinionated way of how an __init__.py must look like, limiting the user in design choices.
That being said, for our current use-case it is fine now. But long-term I think it would be better to have this as an optional feature flag (but that could be added in a later iteration).
What are your thoughts on this?

You're right; the __init__.py topic is complex, and having a feature flag that gives more control would be great. I'm thinking about the possibility to don't generating a particular file so that the user can adapt it manually after the first generation.
Regarding extensibility by the users, I think qenerate should provide a way to allow it or provide some best practices at least. Maybe we can use __init__.py to import a user-defined python file. E.g.
try:
    from .xxxx import *
except ImportError:
    pass

As discussed, I removed the __init__.py generation and will put that into a-i.


Feature: Add a healthcheck to the Dockerfile

This feature enables checking the health of the application.
It checks that it can connect to the MongoDB, frontend is reachable, and checks the health of the apollo server

Pull Request Test Coverage Report for Build 735

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

Totals

Change from base Build 734:
0.0%

Covered Lines:
756

Relevant Lines:
840

💛  - Coveralls

Thanks 🙌


(React) pb.authStore.onChange fires constantly

I'm watching Fireship's video on PocketBase and Svelte and adapting the authentication to React. The program listens to the authStore.onChange event. But when signing up, I get thousands of auth changed logs, telling me that the event is firing as if it's in some infinite loop.

Refreshing stops the event from firing and logging out only fires once.

Looks like I didn't read the docs:
// triggered once right after registration and everytime on store change
const removeListener2 = pb.authStore.onChange((token, model) => {
    console.log('New store data 2:', token, model)
}, true);


Error Gstreamer opencv publish on rtsp server

Which version are you using?
v0.0.0
latest tag on docker hub
Which operating system are you using?

[ ] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[x] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
I'm trying to publish a stream from OpenCV to my rtsp server correctly run through docker.
I simply run the server and i follow the steps shown in documentation to publish stream from opencv to rtsp stream.
I have built opencv with gstreamer and i pasted the code shown.
When i run the python script, gstreamer returns strange warnings and from client the stream is not shown. Only green screen appears on the stream.
The script returns following warnings:
Describe how to replicate the issue
Follow the steps shown in documentation

Start the server through the docker run shown in documentation:
docker run --rm -itd --network=host --name rtspserver aler9/rtsp-simple-server
Build the OpenCV  version with Gstreamer:
Install dependencies of Gstreamer, download the repo with the master branch of opencv, build it through the command shown in documentation
Paste the code shown in documentation with the same gstreamer pipeline
Execute the python script

Did you attach the server logs?
2022/04/09 18:07:58 INF [RTSP] [conn <IP_ADDRESS>:51725] opened
2022/04/09 18:07:58 INF [RTSP] [conn <IP_ADDRESS>:51725] closed (no one is publishing to path 'mystream')
2022/04/09 18:07:58 INF [RTSP] [conn <IP_ADDRESS>:51726] opened
2022/04/09 18:07:58 INF [RTSP] [session 205624671] opened by <IP_ADDRESS>:51726
2022/04/09 18:07:58 INF [RTSP] [conn <IP_ADDRESS>:51726] closed (path of a SETUP request must end with a slash. This typically happens when VLC fails a request, and then switches to an unsupported RTSP dialect)
2022/04/09 18:07:58 INF [RTSP] [session 205624671] closed (not in use)
2022/04/09 18:07:58 INF [RTSP] [conn <IP_ADDRESS>:51727] opened
2022/04/09 18:07:59 INF [RTSP] [conn <IP_ADDRESS>:51727] closed (EOF)

Hi @subzero210 , I faced same problem with you. Any updates for your issues?

Hi @subzero210 , I faced same problem with you. Any updates for your issues?

I have not solved the problem but i found an alternative to perform the same operation. I'm using vidgear library. It is a wrapper of most known multimedia tools (opencv, gstreamer, ffmpeg). It allow, in a simply way to publish a stream on rtsp server.
You can follow this thread https://github.com/abhiTronix/vidgear/issues/228#issuecomment-1024904157

Hi @subzero210 , I faced same problem with you. Any updates for your issues?

I have not solved the problem but i found an alternative to perform the same operation. I'm using vidgear library. It is a wrapper of most known multimedia tools (opencv, gstreamer, ffmpeg). It allows, in a simply way, to publish a stream on rtsp server. You can follow this thread here

Wow, thank you for insightful suggestion.


Python check mushroom version

Describe the solution you'd like
For checking the mushroom version it would be good, if it can be accessed in python after import.
import mushroom
print(mushroom.__version__)

Fixed in dev branch.
The commit also remove all imports statements inside top-level init.py file


Add CSS class name namespacing

It would be desirable to not only have ID namespacing but also prefix CSS class names in SVGs with a custom string to avoid clashes. See #41.

Available as of release 1.2.26

Can you make it configurable ? I want to fill svg in multiple colors, I add classes for paths and suddenly the class names are different than I named.

Hi @tomaskikutis,
please have a look at the documentation. You can disable CSS class namespacing at any time by setting svg.namespaceClassnames = false.
Cheers,
Joschi

@jkphl Thanks, I didn't notice :)

@tomaskikutis You're welcome! ;)


1.0.3 release?

What is needed to cut a 1.0.3 release?
I need the clonable iterators feature introduced in https://github.com/bluss/indexmap/commit/d8e5225a6183886045ded56cc278139ce87d1f17, as it provides parity with HashMap. I'd prefer to not depend on a git revision if possible.
Is there some way I could help out on preparing a release?
(awesome create btw. I just sped up an application by > 30% by just switching to indexmap!)

Next version will be 1.1.0, and I could make the release when I have time. Biggest thing would be to update dev deps and other Rust version related things before the release

I'd be happy upgrade (dev) dependencies if you give me some constraints.

Do you want to stay with Rust 1.18? Or move to some new msrv?
Should I update all deps or just dev deps?

We'll need to move to a new minimum version, but hopefully it can be a bit conservative (so not latest stable). We'd aim to update all dependencies. You could start from https://github.com/bluss/indexmap/pull/85 and use cuviper's suggestion there too (to the extent possible).
The versioning policy is as given in the docs, we'll update the minimum rust version in 1.x minor versions, but be careful about it.
It's a fact of the ecosystem and Rust itself that we must gradually update the msrv.

Also awesome that indexmap was a speed up. It's a very useful datastructure. With the hashmap improvements in std (hashbrown was brought in as the new HashMap implementation), std is doing really well when it comes to performance. Indexmap could still have a leg up in a few situations, for example much faster iteration.

So the following will keep you at 1.30 for cargo test. Non-dev builds remain unaffected.
-itertools = "0.7.0" # 0.8 not compiles on Rust 1.18
-rand = "0.4"
-quickcheck = { version = "0.6", default-features = false }
+itertools = "0.8"
+rand = "0.6"
+quickcheck = { version = "0.8", default-features = false }
 fnv = "1.0"
-lazy_static = "1"
-serde_test = "1.0.5"
+lazy_static = "1.3"
+serde_test = "1.0.99"

quickcheck requires 1.30, and it's also what's stopping you from fully updating rand (it wants 0.6).

FWIW, I'm revisiting #45 -- maybe we can finally get that done for a new release too.


Ini file logic broke -fmt help

This is fairly annoying:
larstha@bling:~/p/Jobanalyzer/code/sonalyze$ sonalyze node -fmt help
Bad arguments, try -h
-remote and -cluster must be used together

It looks like this will entail moving the
		if fhCmd, ok := anyCmd.(FormatHelpAPI); ok {
			if h := fhCmd.MaybeFormatHelp(); h != nil {
				PrintFormatHelp(out, h)
				os.Exit(0)
			}
		}

from sonalyze() and into ParseArgs(), minimally, as well as moving the calls to ValidateFormatArgs up into Validate, and making Validate return a better status.  Or maybe ValidateFormatArgs could take the Command (or part of it) as an argument and could call MaybeFormatHelp itself when appropriate, this would be cleaner.  It's not nice for there to be an exit deep in the system like that but it's also not nice to propagate a lot of status information.


JDK not found

Bug description
I just upgraded to Unity 2022.2.1f1 and<EMAIL_ADDRESS>The build for most platforms works fine.
Only Android build has issues because the JDK is not found:
2023-01-09T11:27:13.6923522Z DisplayProgressbar: Detect Java Development Kit (JDK)
2023-01-09T11:27:13.9736423Z UnityException: JDK not found
2023-01-09T11:27:13.9737808Z Java Development Kit (JDK) directory is not set or invalid. Please, fix it in Edit / Unity -> Preferences -> External Tools
2023-01-09T11:27:13.9739136Z UnityEditor.BuildPipeline:BuildPlayerInternalNoCheck(String[], String, String, BuildTargetGroup, BuildTarget, Int32, BuildOptions, String[], Boolean)
2023-01-09T11:27:13.9740684Z UnityEditor.BuildPipeline:BuildPlayerInternal(String[], String, String, BuildTargetGroup, BuildTarget, Int32, BuildOptions, String[]) (at /home/bokken/build/output/unity/unity/Editor/Mono/BuildPipeline.bindings.cs:481)
2023-01-09T11:27:13.9742309Z UnityEditor.BuildPipeline:BuildPlayer(String[], String, String, BuildTargetGroup, BuildTarget, Int32, BuildOptions, String[]) (at /home/bokken/build/output/unity/unity/Editor/Mono/BuildPipeline.bindings.cs:359)
2023-01-09T11:27:13.9744078Z UnityEditor.BuildPipeline:BuildPlayer(BuildPlayerOptions) (at /home/bokken/build/output/unity/unity/Editor/Mono/BuildPipeline.bindings.cs:323)
2023-01-09T11:27:13.9745377Z UnityBuilderAction.Builder:BuildProject() (at Assets/Editor/Editor/UnityBuilderAction/Builder.cs:71)

In Preferences -> External Tools the check box is set for JDK installed with Unity (recommended).
I see Docker Hub has some images with android in its name.
Do I have to use a specific image?
Expected behavior
The Docker image should include the JDK at the expected location.
Additional details
Please find the GitHub Actions yml configuration here
and the program output here
So far, I just specified
...
        unityVersion:
          - 2022.2.1f1
        targetPlatform:
          - Android # does NOT work at the moment
          - iOS     # does work

This is unity's problem. Unluckily recent version has this bug. It's not setting the correct JDK to the editor on installation. I faced this error one week ago, and there is no easy workaround.

For reference, this was brought up on the GameCI Discord: https://discord.com/channels/710946343828455455/1053868095615283211

Sorry, was celebrating too early.
The issue still occurrs in Unity 2022.2.2f1

Related post on Unity forums: https://forum.unity.com/threads/unity-hub-cli-does-not-install-android-jdk.1375761/#post-8670387

This forum post names a different cause: file permissions are not set properly.
I guess this could be fixed easily on our side.

One of the discord members seems to have gotten Unity to fix the issue: https://discord.com/channels/710946343828455455/1053868095615283211/1065926313661181952

Any news on this issue?

Still waiting on Unity to release an update unfortunately

@AndrewKahr unfortunately for unknown reason I have no access to this server. However, if the problem will be fixed in 3.5 then it's nice. I'll stay tuned. Thank you for the info.

the discord link got fixed today here: https://github.com/game-ci/documentation/pull/374 :+1:

Weird that discord doesn't just give the option to join. I copied the link for the thread so it would link directly to the right message. Didn't know it wouldn't just let you join if you haven't already.


Fix GitHub links

Replace github.com/mapsme/omim with github.com/omapsapp/omapsapp in all documentation files.

Done

Done


App for Android 10

Please make the app work on android 10 as well, my phone is a bit old but I can't afford to buy a new one right now

I tested it and PokeRogue itself doesn't even work on lower Android versions, not just the app. There's not a whole lot I can do here :(

It runs completely fine, on my phone realme 5i with snapdragon 665 and
4gigs of ram
On Tue, May 28, 2024, 11:50 AM William Burleson @.***>
wrote:

I tested it and PokeRogue itself doesn't even work on lower Android
versions, not just the app. There's not a whole lot I can do here :(
—
Reply to this email directly, view it on GitHub
https://github.com/Admiral-Billy/Pokerogue-App/issues/68#issuecomment-2134467119,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BIYVBKXCZQUIX2CFBTPDHHLZEQSKLAVCNFSM6AAAAABIMGHKJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZUGQ3DOMJRHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Apparently it's just the android emulators that struggle there, although there's still a hard cutoff somewhere. I haven't been able to figure out how to reduce the Android version yet because I don't have actual devices to test it on, but I'll leave it on the radar at least.

Ok, thanks for the update.
On Wed, May 29, 2024, 1:46 AM William Burleson @.***>
wrote:

Apparently it's just the android emulators that struggle there,
although there's still a hard cutoff somewhere. I haven't been able to
figure out how to reduce the Android version yet because I don't have
actual devices to test it on, but I'll leave it on the radar at least.
—
Reply to this email directly, view it on GitHub
https://github.com/Admiral-Billy/Pokerogue-App/issues/68#issuecomment-2136075915,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BIYVBKVFXPPMO66EORKKA6LZETUIZAVCNFSM6AAAAABIMGHKJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZWGA3TKOJRGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can you give this version of the app a test? I may have fixed the issue.
app-debug.zip
It's an apk inside of a zip, so just extract it and install/update with it.

Ok I'll give this a try when I get home
On Tue, Jun 4, 2024, 10:53 AM William Burleson @.***>
wrote:

Can you give this version of the app a test? I may have fixed the issue.
app-debug.zip
https://github.com/user-attachments/files/15543164/app-debug.zip
It's an apk inside of a zip, so just extract it and install/update with it.
—
Reply to this email directly, view it on GitHub
https://github.com/Admiral-Billy/Pokerogue-App/issues/68#issuecomment-2146668991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BIYVBKWCLQCMSFH2LTGSUVLZFVI7FAVCNFSM6AAAAABIMGHKJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBWGY3DQOJZGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Updating the offline files results in time out error, the online function
works but takes way too long to load
On Tue, Jun 4, 2024, 10:53 AM William Burleson @.***>
wrote:

Can you give this version of the app a test? I may have fixed the issue.
app-debug.zip
https://github.com/user-attachments/files/15543164/app-debug.zip
It's an apk inside of a zip, so just extract it and install/update with it.
—
Reply to this email directly, view it on GitHub
https://github.com/Admiral-Billy/Pokerogue-App/issues/68#issuecomment-2146668991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BIYVBKWCLQCMSFH2LTGSUVLZFVI7FAVCNFSM6AAAAABIMGHKJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBWGY3DQOJZGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Oh I see, I'd keep trying again with the offline download; the timeout should be about 3 seconds for a connection. If that still doesn't work, try this version with a longer timeout and see if that helps at all.
app-debug.zip

Good news, i was able to play the offline version with this one, but
downloading the debug mode still gets the timeout error
On Wed, Jun 5, 2024, 3:11 AM William Burleson @.***>
wrote:

Oh I see, I'd keep trying again with the offline download; the timeout
should be about 3 seconds for a connection. If that still doesn't work, try
this version with a longer timeout and see if that helps at all.
app-debug.zip
https://github.com/user-attachments/files/15570913/app-debug.zip
—
Reply to this email directly, view it on GitHub
https://github.com/Admiral-Billy/Pokerogue-App/issues/68#issuecomment-2148496215,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BIYVBKSLJCECHPCSE7RPNZTZFY3QRAVCNFSM6AAAAABIMGHKJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBYGQ4TMMRRGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ah, I see. I'm going to close this issue for now since I think it's just internet-related? But at least you've got a solution that mostly works here. Feel free to open it up again if you think it's something else though.


Rails/RedundantPresenceValidationOnBelongsTo doesn't work on some associations

Consider
class User1 < ApplicationRecord
  belongs_to :company, -> { where(foo: true) }
  validates :company_id, presence: true
end

class User2 < ApplicationRecord
  belongs_to :company, inverse_of: :employee
  validates :company_id, presence: true
end

class User3 < ApplicationRecord
  belongs_to :company, -> { where(foo: true) }, inverse_of: :employee
  validates :company_id, presence: true
end

Rails/RedundantPresenceValidationOnBelongsTo flags User1 and User2 as offenses, but not User3.
Expected Behavior
All three examples should be flagged as offenses as they perform redundant presence validations on company_id
RuboCop version
$ [bundle exec] rubocop -V
1.36.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.21.0, running on ruby 2.6.9) [x86_64-darwin21]
  - rubocop-minitest 0.22.1
  - rubocop-performance 1.15.0
  - rubocop-rails 2.16.1

Strange, when we use company instead company_id, it works:
class User3 < ApplicationRecord
  belongs_to :company, -> { where(foo: true) }, inverse_of: :employee
  validates :company, presence: true
end

I can try to fix this one


Switch to Redux observable implementation

https://github.com/reactjs/redux/pull/1632
Then maybe we won't need our own shim!

You mean Rxjs?

No, redux has built in observables now. I feel like Rxjs observables are way too heavy to be a standard part of the client, but we should make it possible to use them!

Okay. We'll work on a simple wrapper for Rx.

OK it turns out that using that won't help us at all, because the shim for observable is very simple. Closing in favor of #149 for RxJS.


Update "maps" page

Automatically generated by Netlify CMS

But this told us where to find Nulla at nulla justo‽


Getting traceback error after logging in

After i login nothing happens and after a few seconds it throws this error:
Loggin in
Traceback (most recent call last):
  File "D:\Local Downloads\auto-lingo\auto-lingo.py", line 911, in <module>
    main()
  File "D:\Local Downloads\auto-lingo\auto-lingo.py", line 903, in main
    learn_bot()
  File "D:\Local Downloads\auto-lingo\auto-lingo.py", line 778, in learn_bot
    skills = WebDriverWait(driver, 20).until(
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Python311\Lib\site-packages\selenium\webdriver\support\wait.py", line 95, in until
    raise TimeoutException(message, screen, stacktrace)
selenium.common.exceptions.TimeoutException: Message:

Yep, same issue. Any fix?

same.. @drknzz any fix please..?

@avyaymc @Ariuuwu @TomerGamerTV As I understand it, this error is due to the fact that duolingo changed names of div-containers on site, so we need to select div-names again

@AlinaRinn I don't understand your proposed solution, can you explain more?


Take advantage of cosign's modular re-factoring

https://github.com/sigstore/cosign/pull/3059 is splitting APIs based on providers. Once it's landed, we can use these to provide various slsa-verifier builds: on for all providers, one for Google, docker, etc

There is also a new Go library donated by GitHub folks https://github.com/sigstore/sigstore-go!


Programs

Add finding programs and opening file with some program (not with the program selected by default)

Completed 👌🏻


Add branch_order function to neurom.segments.

sec_bo = [bo for bo in iter_neurites(NEURON, sec.branch_order)]

Current coverage is 100.00%

Merging #242 into master will not affect coverage as of 61a7018

@@            master    #242   diff @@
======================================
  Files           38      38
  Stmts         1523    1526     +3
  Branches         0       0
  Methods
======================================
+ Hit           1523    1526     +3
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 61a7018

Powered by Codecov. Updated on successful CI builds.