Feature request? How to update workflows?

fastai/workflows/nbdev-ci@master fails as

INFO: pip is looking at multiple versions of spot-mark-gene[dev] to determine which version is compatible with other requirements. This could take a while.
273
ERROR: Package '' requires a different Python: 3.9.16 not in '>=3.10'

So I went to try and find / figure out what to do about this and I can't find it under actions. So where do I go to find the base workflows so I can update them?

hey @dsm-72, the workflows come from the https://github.com/fastai/workflows repo. They are based on the idea of resusable workflows. You can read more about it over → here
If you wish to use python3.10, you can do that in the following fashion.
name: CI
on:  [workflow_dispatch, pull_request, push]

jobs:
  test:
    runs-on: ubuntu-latest
    with:
      python: 3.10
    steps: [uses: fastai/workflows/nbdev-ci@master]

@deven367
[Invalid workflow file: .github/workflows/test.yaml#L7](https://github.com/<user>/<project>/actions/runs/4075076640/workflow)
The workflow is not valid. .github/workflows/test.yaml (Line: 7, Col: 5): Unexpected value 'with'

@dsm-72
I tried it myself, it works for me
https://github.com/deven367/test-nbdev/blob/main/.github/workflows/test.yaml

@dsm-72 Apologies. There was a syntax error.
name: CI
on:  [workflow_dispatch, pull_request, push]

jobs:
  test:
    runs-on: ubuntu-latest
    steps:
      - uses: fastai/workflows/nbdev-ci@master
        with:
          python-version: '3.10'

@deven367 Thanks! Now it is running on push :) and.... now nbdev_preview is broken. Reverting these files fixes it.
That doesn't make sense to me since they should be unrelated....
(PROJECT) ➜  PROJECT git:(branch) ✗ nbdev_preview
Traceback (most recent call last):
  File "/Users/USER/miniconda3/envs/PROJECT/bin/nbdev_preview", line 8, in <module>
    sys.exit(nbdev_preview())
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/site-packages/fastcore/script.py", line 119, in _f
    return tfunc(**merge(args, args_from_prog(func, xtra)))
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/site-packages/nbdev/quarto.py", line 270, in nbdev_preview
    cache,cfg,path = _pre_docs(path, n_workers=n_workers, **kwargs)
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/site-packages/nbdev/quarto.py", line 174, in _pre_docs
    cache = proc_nbs(path, n_workers=n_workers, **kwargs)
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/site-packages/nbdev/serve.py", line 76, in proc_nbs
    parallel(nbdev.serve_drv.main, files, n_workers=n_workers, pause=0.01, **kw)
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/site-packages/fastcore/parallel.py", line 117, in parallel
    return L(r)
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/site-packages/fastcore/foundation.py", line 98, in __call__
    return super().__call__(x, *args, **kwargs)
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/site-packages/fastcore/foundation.py", line 106, in __init__
    items = listify(items, *rest, use_list=use_list, match=match)
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/site-packages/fastcore/basics.py", line 66, in listify
    elif is_iter(o): res = list(o)
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/concurrent/futures/process.py", line 570, in _chain_from_iterable_of_lists
    for element in iterable:
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/concurrent/futures/_base.py", line 621, in result_iterator
    yield _result_or_cancel(fs.pop())
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/concurrent/futures/_base.py", line 319, in _result_or_cancel
    return fut.result(timeout)
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/concurrent/futures/_base.py", line 458, in result
    return self.__get_result()
  File "/Users/USER/miniforge3/envs/PROJECT/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
    raise self._exception
concurrent.futures.process.BrokenProcessPool: A process in the process pool was terminated abruptly while the future was running or pending.

nbdev_preview uses parallel processing by default which makes the stack trace useless
Can you run it with nbdev_preview --n_workers 0?

Also any idea on why this might happen?
Ran this again just to make sure...
$ nbdev_install_hooks
Hooks are installed.

Check that strip out was unnecessary
[783](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:795)
+ git status -s
[784](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:796)
 M nbs/##_my_file.ipynb
[785](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:797)
++ git status --porcelain -uno
[786](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:798)
+ [[ -n  M nbs/##_my_file.ipynb ]]
[787](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:799)
+ git status -uno
[788](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:800)
On branch TEST_BRANCH
[789](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:801)
Your branch is up to date with 'origin/TEST_BRANCH'.
[790](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:802)

[791](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:803)
Changes not staged for commit:
[792](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:804)
  (use "git add <file>..." to update what will be committed)
[793](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:805)
  (use "git restore <file>..." to discard changes in working directory)
[794](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:806)
	modified:   nbs/##_my_file.ipynb
[795](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:807)

[796](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:808)
no changes added to commit (use "git add" and/or "git commit -a")
[797](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:809)
+ echo -e '!!! Detected unstripped out notebooks\n!!!Remember to run nbdev_install_hooks'
[798](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:810)
+ false
[799](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:811)
!!! Detected unstripped out notebooks
[800](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:812)
!!!Remember to run nbdev_install_hooks
[801](https://github.com/USER/PROJECT/actions/runs/4075784512/jobs/7022648909#step:2:813)
Error: Process completed with exit code 1.

once the hooks are installed, run nbdev_prepare and try to find the error. Once you find the problematic cell, commit everything nbs and the py files, it should probably fix the error.

Thank you so much for the assistance!

@dsm-72 this should fix it, python-version is not a valid input to the workflow
name: CI
on:  [workflow_dispatch, pull_request, push]

jobs:
  test:
    runs-on: ubuntu-latest
    steps:
      - uses: fastai/workflows/nbdev-ci@master
        with:
          version: '3.10'


Helpers class

Hello guys!
I'd like to create a 'helper' class, to handle all of my helpers functions like removeMarkdown / escapeMarkdown and few other. Some of them rely on built-in getMessage(), for example.
Since I'm quite a noob in OOP and all of those namespaces, I can't figure out the best place and practice to implement in without touching existing composer package structure.
I'd like to have something like Request::sendMessage() method, but for helpers, ex. Helpers::removeMarkdown($str) or Helpers::logToDevChannel($message). I'll write them by myself, but not really sure about a place and the way to implement.
Could you give me a small advice, with direction to start diggin?
PS. Please add getBotID() method to Telegram.php to get bot ID (it's needed to check if bot is an admin of a group with Request::getChatMember() method, for example), since there is no way to get it easy, atm. Yes, I know that I can explode api key to get it, but that not a pretty way. )

Any advice? Anyone?

There is nowhere to get bot id when it comes to library code, use Request::getMe() to get the bot ID dynamically. Or like you said, explode api key or declare a constant in hook script with bot id in it.
About the helper class, all you might have to do is have your static helper class be included before hook script execution. It should be using library's namespace and extend Telegram (not sure about this, might not be required).

@jacklul howbout explode api key by ":" as I said? But not in every command, as I'm doing it right now (since I don't where to put aside "helpers" functions).
And the main question is another one, sorry for pointing that out.

How I do things:
-- src (my project dir)
---- Commands (commands dir)
---- Strings.php (classes to be included are in 'src' dir)
-- vendor (contains php-telegram-bot)
Extra code in hook script:
require __DIR__ . '/vendor/autoload.php';

$PROJECT_DIR = __DIR__ . '/src/';

foreach(scandir($PROJECT_DIR) as $class) {
    if (is_file($PROJECT_DIR . '/' . $class) && file_exists($PROJECT_DIR . '/' . $class)) {
        $file_parts = pathinfo($PROJECT_DIR . '/' . $class);
        if($file_parts['extension'] == 'php') {
            require_once ($PROJECT_DIR . '/' . $class);
        }
    }
}

Sample class:
<?php

namespace Longman\TelegramBot;

class Strings
{

}

This way every time I want to use Strings class I must include it:
<?php

namespace Longman\TelegramBot\Commands\UserCommands;

use Longman\TelegramBot\Commands\UserCommand;
use Longman\TelegramBot\Request;
use Longman\TelegramBot\Strings;

class MyCommand extends UserCommand

Got it! Thanks a lot @jacklul!

I'm reopening here and think it would make sense to have a Utils class for general code that can be reused.


Only 1k rows/second query speed

I'm running an InfluxDB instance to save financial tick data. Every row has 7 fields. I then query this data and aggregate it in Python. Those were my query times (number of rows on the left, query time in hours, minutes and seconds on the right):
9480222, 2:07:46
12839124, 3:06:02
17256737, 4:19:54
13716707, 3:28:37
12671435, 2:35:27
11112483, 2:15:53
17055181, 3:34:21
21232810, 6:29:42
16935780, 4:47:56
Those numbers seem a bit off. The average is around 60-70k rows per minute, 1k rows per second. Since this is my first experience with TS Databases and with Influx, would you consider this performance normal? I'm running InfluxDB 1.7.9, Influx Python client 5.2.3, Python 3.7, running from Pycharm, on a MacBook Pro with 16GB Ram.
Any tip is welcomed!

@rbdm-qnt thanks! am I understanding this correctly that you are returning between 10 and 20 million rows of data with each query? is the performance from querying influx or the python code performing the processing of the rows?

Yes that is correct. The performance is from querying influx! Processing time is around 30 to 60 minutes per query.

@rbdm-qnt thanks! if it's related to influxdb, probably makes sense to open it in the https://github.com/influxdata/influxdb repo. But there are tons of things that impact db performance, so it's tough to know where to start. Make sure you have followed all the guidance in the context section of the docs pages: https://docs.influxdata.com/influxdb/v1.7/concepts/

Upon closer inspection in the activity monitor, looks like influx is only using 500mb of my ram, while the Python program running the python client for influx is consuming 13GB, of which up to 11GB get allocated automatically to Swap memory. I think this could be part of the issue, not sure if it's the only reason why the query is so slow. Also I'm subdividing my 28GB dataset in 51 chunks, so should be around 600mb per query; i make sure to empty the query result list after every query chunk. Why would it use 13gb?
I'll also read that document, thank you for the link.

are you just dumping the raw data landed in Influx back out to Python for aggregation? if so, why don't you push the aggregation into the initial query itself...that's what the query engine is for.
maybe asked another way, what is the initial query you are running to extract the data?
if it's something like select * from and a time range...this is not going to provide the fast results you are looking for.
there are a ton of mathematical functions for working with tick data in the query language itself and allowing the query engine to operate on the compressed data to return a result is going to provide the best performance.
https://docs.influxdata.com/influxdb/v1.7/query_language/functions/#technical-analysis
If you can make the query engine do the bulk of the work...you can still do additional post processing in Python....but the best practice is to push as much of the heavy lifting into the DB.

Thank you for the reply! So, I'm storing financial tick data, which is 7 columns per row, so yes, the only way I can query it is select * from a time range, since I need all the columns. I've read the influx docs, but unfortunately no function suits my needs. I need to process this data in different and more complex ways, so I need it raw, I can't work with averaged data, or already aggregated data. I'm starting to think that a simple parquet file folder could suite my needs better than any database, since I don't need a lot of those functionalities, I just need speed for big queries of raw data.

ok.  I think you are right, this isn't a good use for the db.
But, if you want to share a handful of the different and more complex ways, we would be interested in looking at whether these functions could exist (like the technical analysis functions) natively.  Our new query language, Flux, allows for a much wider variety of mathematical processing and handling and additional functions -- allowing you to operate on the data without "moving it" around.   The idea is to provide as many of the primitive functions as possible and then continue to build up more sophisticated ones (again, like the technical analysis functions).  You can see some newer functions that we added to the 2.0 Beta and get a feel for the language here: https://v2.docs.influxdata.com/v2.0/reference/flux/stdlib/built-in/transformations/aggregates/kaufmansama/
let us know...

Thank you for the doc, I'm reading it! You have great variety of functions. My main problem is that I need to apply many different functions to the same raw data, in the same processing step, and then save the processed data all together. So it's not that I can't do 1 type of processing with the influx language, is that I need to do many different ones, and then aggregate processed values in certain ways. And since I already wrote all of this code previously and I use it with live data, I don't want to rewrite it (even if it was possible) in the influx query language, I want the database to return data in exactly, precisely the same format as the live data.

Thank you for the doc, I'm reading it! You have great variety of functions. My main problem is that I need to apply many different functions to the same raw data, in the same processing step, and then save the processed data all together. So it's not that I can't do 1 type of processing with the influx language, is that I need to do many different ones, and then aggregate processed values in certain ways. And since I already wrote all of this code previously and I use it with live data, I don't want to rewrite it (even if it was possible) in the influx query language, I want the database to return data in exactly, precisely the same format as the live data.

would you give email<EMAIL_ADDRESS>maybe I had some solution for you.


[Screen Reader-Microsoft MakeCode Arcade-Landing Page]: Empty "Simulator" region landmark is defined for the page which is not present.

User impact:
Users who rely on Screen Reader will get impacted and user will be misguided when Empty "Simulator" region landmark is defined for the page which is not present.
Note:
User credentials should NOT be included in the bug.
Repro Steps:

Open the URL https://arcade.makecode.com/
Landing page will be appeared.
Turn on Accessibility Insight Tool.
Verify whether Empty "Simulator" region landmark is defined for the page which is not present.

Actual Result:
When navigating to the page, empty "Simulator" region landmark is defined for the page which is not present.
Expected Result:
When navigating to the page, empty "Simulator" region landmark should not be define for the page which is not present.
WCAG Reference:
https://www.w3.org/WAI/WCAG21/Understanding/bypass-blocks.html
Test Environment:
OS: Windows Version- 21H1 (OS Build 19043.1165)
Browser: Edge Version 93.0.961.47 (Official build) (64-bit)
URL:https://arcade.makecode.com/
Tool: Accessibility Insight.
Attachment:

As discussed with internal SME, this is not a bug. Hence, closing the bug.

As checked on [Microsoft Make Code Arcade] (https://arcade.makecode.com/) the issue is no longer repro .Hence Closing the bug. #Closed #Regressed:09-28-23
Kindly change the state to close.


Add missing doc comments

/azp run AspNetCore-helix-test

Those items probably need comments too, but I didn't feel confident that I could accurately document them given my knowledge. Instead I documented the areas/methods I was comfortable with, figuring some documentation is better than none.


kustomize version to v5.0.0

default version 3.5.4 in the registration-operator repo was deprecated
so change to the most recent version (v5.0.0)
Ref:
https://issues.redhat.com/browse/ACM-3076
Signed-off-by: Yi Rae Kim<EMAIL_ADDRESS>

3.5.4 does seem pretty old. Does v5 bring a feature we need? Or was something going wrong when installing the old version?
I'm just wondering if we should make this change explicitly here, or wait for registration-operator to update their Makefile instead.

3.5.4 does seem pretty old. Does v5 bring a feature we need? Or was something going wrong when installing the old version?
I'm just wondering if we should make this change explicitly here, or wait for registration-operator to update their Makefile instead.


Improved integration tests

now config.yaml location can be defined with custom value (env var)
integration tests use a new /tmp/ folder storage with every run
After every command db is backed up to /tmp/%dir%/snapshots/ for easy debug
Every command is printed before execution

NB: despite of it simplifies tests rerun, one still need to reset db storage manually before tests run:  ./reset_db.sh && python seed.py

Hi @hasan7n thanks, you are right, moved config&storage init to tests_setup.sh.
Also updated "storage moving": instead of real moving files back and forth (that may be error-prone if broken on the mid) we explicitly point config pathes to a newly created empty tmp folder. That is safer IMO

@hasan7n fixed, ready for re-review:)


Why does gpio_get_level return an int and not a bool?  (IDFGH-10265)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
 * @return
 *     - 0 the GPIO input level is 0
 *     - 1 the GPIO input level is 1
 *
 */
int gpio_get_level(gpio_num_t gpio_num);

Is there any good reason for this function returning an int rather than a bool that I am missing?

imo a book would not make sense.
the function is called "get level."
would true be high or low?
It would need to be renamed "is_level_high"

Different perspective I guess. I look at the possible outcomes and see only two: 0 or 1. That's a bool in my c and c++ mind. We are talking logical levels right. HIGH or LOW, true or false, 1 or 0.
If I see a function returning an int I'd expect possible outcomes to be a range of values possibly even negative. Is this analog values you could wonder... Gets tiring pretty quickly if you need to look up every functions source code to find out what the return values actually are going to be.
But maybe it's my lack of experience with the convenience in math you're talking about.
✌

So upon reading a little. I guess it has to do with implicit promotion rules. And confusion that might arise from that? Where if used in an expression, a bool is often implicitly promoted to int in the background.
Or is it something else?
Interesting read:
https://stackoverflow.com/questions/46073295/implicit-type-promotion-rules/46073296#46073296

@Gaai You can have a look at ESP-IDF-C++, where the type of GPIOInput::get_level is an enum.


Breadcrumb trail

Any chance of adding a breadcrumb trail
After viewing a note there is no way to go back to the folder(s) without starting over

@SpudR It is already in the next version.
The next version is a big change.


Generate Typings

Hi there, I have added typings generation for all of the libraries. I was unable to generate them for fo, ga, hu, and mk, so I did those by hand. A couple of syntax changes were made as well.
Edit: If the PR is too large to review due to the generated files, I can rollback the generated files so that you may generate them instead.
Summary

Generates type definitions (index.d.ts files) for all languages, and references them in the package.json.
JSDoc comment included in the js.
crawl.sh has color functions changed to the POSIX compliant format.

Hi there! 👋
First of all: thanks for the work, I’m sure it took quite some time, and I appreciate that.
I’m going to review this in one comment here because the change set is so large, so I think 👇🏽 is more readable

https://github.com/wooorm/dictionaries/pull/15/commits/bc6143d58b7d9bdc2da748865580fc8859817290 Make color funcs POSIX compliant — looks good!
https://github.com/wooorm/dictionaries/pull/15/commits/a8cf719f8e64fde023c2803e2aeaf48183cd077e

Add typings — This is a big one. I’m not necessarily against doing this, but I have run into several problems with people adding types to non-TS libraries. TypeScript recommends against it, and I think I agree
Update generate script — I’m not so happy about refactoring code if it’s unneeded, especially if it would break support on older Node and the like. Although it’s a build script so it doesn’t matter too much
Include type comment in index.js — I’m not a fan of a) adding a comment that reiterates the code, and b) adding a JSDoc comment doing that as well!
use destructuring syntax and const/let — I also honestly don’t think using only new features of JS is making the file easier to read. Note that “old” JS is still JS, and still modern JS as well. Moving the module.exports is unneeded 🤷♂

Hi there,
Thanks for getting back to me on this. I also feel uncomfortable creating typings manually. I'm also unsure if the template itself should be TypeScript, it would require a bit of extra work to process it, but it will also most likely garble a lot of the downstream code which also needs to be committed.
The JSDoc might be unnecessary since it can be done in the types file anyway. I do believe having this comment in either spots will still be helpful though. VSCode shows these comments when you hover over the imported item. That instantly available documentation is quite enjoyable IMO, and even though it looks redundant since it is reiterating the code, it adds an extra layer of helpfulness for when a user does not wish to drill down into the code.
The syntax updates I suppose might be considered superfluous. I assumed most users will be using this library from Node due to the fs and path imports, and at this point, Node 6 has reached EOL and all currently supported versions of Node have this syntax available. I also think distinguishing between read only consts and mutable lets does in fact raise the readability of the code. Those are small changes that give us a lot of useful information.
The style choices come directly from how NodeJS currently structures their code, for example, the path module: https://github.com/nodejs/node/blob/master/lib/path.js
All in all though, the only part that really matters to me here is being able to receive typings, as that will allow me to use this code within TypeScript without having to write my own declaration, and being able to do it from within this library means that the types can be automatically propogated down to the individual libraries without having to manually push a type declaration to their respective @types repos.
Thank you very much for this repo btw, it is pretty much a gold mine.


duplicate in query history

This isn't a big deal but I never saw this before. Usually when I run an identical query it just reruns the exisitng one (or at least that is what I thought).
This is the first time I ever saw a duplicated identical query:

They aren't quite the same, the bracketing is different.


Change default coarse grid solver for the geometric multigrid

Description

This PR makes the direct solver the default coarse grid solver for the geometric multigrid alternatives in the matrix-free application.
Testing

All the existent tests pass with this coarse-grid solver. The parameter files were cleaned and the coarse grid solver section removed.
Documentation

The default value was updated in the linear solver section of the documentation.
Checklist (will be removed when merged)
See this page for more information about the pull request process.
Code related list:

[X] All in-code documentation related to this PR is up to date (Doxygen format)
[X] Copyright headers are present and up to date
[X] Lethe documentation is up to date
[X] The branch is rebased onto master
[X] Code is indented with indent-all and .prm files (examples and tests) with prm-indent
[ ] If parameters are modified, the tests and the documentation of examples are up to date
[ ] Changelog (CHANGELOG.md) is up to date if the refactor affects the user experience or the codebase

Pull request related list:

[X] No other PR is open related to this refactoring
[X] Labels are applied
[ ] There are at least 2 reviewers (or 1 if small feature) excluding the responsible for the merge
[ ] The PR description is cleaned and ready for merge

@lpsaavedra can you add a changelog entry for this since this is a modification to a default parameter?


Let runtime library functions ambiguate their parameter names

This commit is part of #2284, but given that it affects not only the Javascript system, I've decided to pull it out into a separate PR.

In cases where a runtime library function has multiple overloads with the same arity, the parameter names cannot be nicely resolved from the implementation function, because they have different meaning depending on the selected type signature.
Instead of hardcoding this in TypescriptDefinition, let function definition include parameter names where this makes sense.
Unfortunately, this requires some extra pass-through constructors in LibraryFunction. At least one of them can resolved once we switch to JDK22 and statements before the this() call are allowed.
These parameter names will also show up in ashref/jsref, but unfortunately the reflection-based parameter names don't show up, because KoLmafia is only compiled with the required -parameters switch for tests and tsDef (see build.gradle#L285-L289). Is there a good reason not to enable this generally?

Could someone please re-run the tests on windows? They pass fine here: https://github.com/oxc/kolmafia/actions/runs/8869700421/job/24350661462

For completeness sake, I want to provide an alternative suggestion: instead of mixing reflection and explicitly named parameters, we could just switch all runtime functions to include their parameter names. Let me know if I should add this to this PR.

For completeness sake, I want to provide an alternative suggestion: instead of mixing reflection and explicitly named parameters, we could just switch all runtime functions to include their parameter names. Let me know if I should add this to this PR.

I do think your proposed alternative is probably better, but not necessary for right now if you don't feel up to it.

They would already have shown up in ashref, but your guess might still be true from earlier times.
If nobody objects, I might still add namedParams to all functions, in a couple of days.

Please do!
On Tue, 30 Apr 2024 at 17:18, Bernhard Frauendienst <
@.***> wrote:

They would already have shown up in ashref, but your guess might still be
true from earlier times.
If nobody objects, I might still add namedParams to all functions, in a
couple of days.
—
Reply to this email directly, view it on GitHub
https://github.com/kolmafia/kolmafia/pull/2285#issuecomment-2085841314,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABL4LQ6XGFQUT2X24WH7QTY7674BAVCNFSM6AAAAABG5FWSFWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBVHA2DCMZRGQ
.
You are receiving this because your review was requested.Message ID:
@.***>

I've added param names for all functions. I have not provided any new names (except for the "adventure" function in the last commit for demonstration purposes).


Design and think through BTAGSIL's world structure

As of right now, I'm encountering a problem with designing world structure for #32, as it seems like I don't know what BTAGSIL's world is going to look like at a higher level. I cannot design a randomized world before I understand what I actually want the world to look like at the most abstract and high level.
This issue is to think through that high level concept, design it and capture the results in the comments here in this issue.
NOTE: this is not a feature/bugfix/refactor, this is a research/design task, so no code is going to be written to finish and close this issue. For actual code, go to #32 once this is done.

This blocks #32.

Okay, this is going to sound ridiculous after the whole Language Feast fiasco and how I'd spent so much time choosing the language for BTAGSIL and then went with a random unrelated choice that effectively voided all the work in that milestone (more on that in #27), but there's one more big change coming.
I've finally realized, after much procrastination and soul searching, what I need for BTAGSIL in terms of language/technology. I need the fastest language to develop in. And I also need the language I can freely think in and implement my ideas with as little friction as possible. C++ is the last possible language for that (together with Rust).
That's why, from now on the language of implementation for BTAGSIL is going to be Python for the foreseeable future.

Together with switching the language of implementation for the second time, I'm leaving the room for coming back to Rust/C++ in the future. However, this is where this decision ties in with this issue, designing the world architecture and setting.

I cannot design world structure and I cannot choose a setting or even a concept for what BTAGSIL is going to be about because I've never developed or even played text games.
(1) is why I need the best prototyping language possible. I need to rapidly develop several relatively large features and experiment with several game mechanics to determine what is even possible within this concept. Once that experimentation phase is done, I'm going to be finally able to determine the final language of implementation, which may still be Python, or it may change back to C++ or Rust as more low level / finely controlled languages.
I've just realized that Python is the only language I feel free in. In the sense that, it may be the only language that would allow me to stop being afraid of complex software development. And it's the only language I'm confident in. And it's the language that lacks the tooling issue of C++ and the PitA compiler issue of Rust. And, of course, I won't have to think about memory management and ownership in it, which is incredibly important at this prototyping stage.
What's most important, I will be able to instantly start prototyping and actually creating game systems in it, while in C++ I would have to first break through the wall of boilerplate and setup, which I don't have the mental capacity for right now.

Finally, I can close this issue with the following note about BTAGSIL's world design:

For now, I leave the setting/world concept up in the air;
I enter the prototyping stage with Python where, while I don't have any firm grasp of what BTAGSIL is and what its game mechanics and world are going to be, this stage allows me to rapidly experiment with several different mechanics and make my final choice once I'm informed enough and understand what I'm doing.
TL;DR: let's have fun experimenting and figuring this out first, then we'll have all the time we need to "do this properly" so to speak.


URL parameters are not editable

What happens?
URL parameters mentioned in the openapi.yml are not editable in the UI - inspecting the DOM shows that contenteditable=false. The same issue does not happen in the Scalar Swagger editor.

What did you expect to happen?
It should be editable
How can we reproduce the issue?
It can be reproduced via CDN link (I am using it with Go) and the Online Sandbox
OpenAPI Document
https://sandbox.scalar.com/e/t0Ca7

hey @sidwebworks thanks for the feedback! it's been added by design in the new api client, however i'd love to get some more about your use case here, is that something you'd like to see back?
nb. the new api client is not yet running in the editor 🔜

@antlio Oh okay - what does this design solve? infact I have a really confusing experience with this as I could not test the apis using the send request panel - how am I suppose to inject param values in this new design? Are there any documentation or PR notes I can see?
All in all - I really found the previous functionality user friendly and pretty much obvious as that is how it works in things like insomnia and postman. I might not be seeing what the design is aiming to do which I am happy to learn more about.

thanks for the quick reply! you should be normally able to do that by updating the value in the path variable table. however i'll investigate as it doesn't seem to be showing up in your case.
regarding the url, we wanted here to strictly reflect the informations from the specification when navigating within the reference, while keeping fully autonomous testing capabilities from the workspace (which is coming very soon!).
does that make sense? however nothing cast in stone so I'll share your feedback with the team!

@antlio that makes a ton of sense - cool I am in the favour of that.  Thanks hoping to see this fixed soon. Let me know if I can help in any way.

i gave it a look and path variable should be displayed and be updated here an example where i added parameters to endpoints:

Cool looks like an issue with the generated open api - Will see how to fix it. Thanks


Document GeocodingError; updates for Travis-CI

Coverage remained the same at 100.0% when pulling d3ea699dc3ff67489dc00dc5b044d810382a6d2d on doc-geocoding-error into 8a3afa753b6a788396887fdc47c05821dc4563f6 on master.


Cinder relation

juju relate cinder-backup-nfs:backup-backend cinder-backup:backup-backend
ERROR no relations found

https://github.com/cloudbase/charm-cinder-backup-nfs/blob/main/README.md
Should this not relate to cinder:backup-backend ?

you're correct. It was a typo on my part. Will fix soon.


Adds request retry attributes to HTTPRoute API Type

Extends ForwardToTarget of type HTTPRoute to include per-target request timeouts and retries.
Partially fixes #58.
/assign @bowei
/cc @Miciah @robscott

@hbagdi I just pushed b76d37b that addresses some of your feedback, PTAL.

I think this may overlap with @bowei's Service level config proposal.

@danehans Should we keep this open or close this and track it as part of #196 and revisit post v1alpha1?

Note that Knative has deprecated managing  connection timeouts: https://github.com/knative/networking/blob/4b21f11ccfa7fbd2ab5c0a978e78e79f4fe728d8/pkg/apis/networking/v1alpha1/ingress_types.go#L240

From 4/14/21 community meeting:

Retries are not as portable as heath-checking and timeout connection policies.
Consensus around health-check, timeout, retry, etc. policies residing in BackEndPolicy while the ServicePolicy design.

I think that with GEP #713 we've determined that this is a better fit for policy attachment than being built into the route. I'm going to close this in favor of that approach but anyone can feel free to reopen if needed.
/close


Players do not respawn when the round restarts.

If you run restartround your player will be kicked out of its entity but it won't respawn. Pretty sure there is simply no logic to handle this when on no-lobby mode.

@Injazz yes it's mostly for testing, but that doesn't make this issue any less valid. Restarting the round on testing effectively should reset the map and such.
Anyways, this issue has been fixed by me at some point anyways.


feat: #682 - add a "Clear all" menu item in the product history page

Impacted files:

dao_product_list.dart: added clear method
product_list_dialog_helper.dart: added openClear method
product_list_page.dart: added a "clear" menu item whenever relevant

Ohh, nevermind it started in https://github.com/openfoodfacts/smooth-app/pull/679, but still we need to regenerate the files


Add random_bool helper function

to: @strcrzy
cc: @airbnb/streamalert-maintainers
size: small
Background
For rules which are very noisy, it may be helpful to return only a sample of the true positives.
Changes

Add a random_bool base helper function

Testing

Unit tests and pylint

Coverage remained the same at 97.595% when pulling 02a39ff49c2377d3c66e68f744d75fe07c319dc4 on austin-base-helpers into b8ef603bf99455be4c9b8555ef31c86a458a6e4b on master.


Add IsLessThanOrEqualTo and IsGreaterThanOrEqualTo checks

Currently there's IsLessThan and IsGreaterThan.
IsLessThanOrEqualTo and IsGreaterThanOrEqualTo can be useful as well and are basically already used inside of IsInRange.

These mehods are now added


Give config option to start weeks on mondays

Would it be hard to implement an option where the user can make the week start on mondays?
BTW: great workflow, like it a lot.

Hi @LeEnno
It shouldn't be difficult. I'll add this feature soon.

Hi @LeEnno
Here is the latest workflow which supports the feature you want. Using calconfig for the configuration.
Owen

You rock, thx!


Add the snackbar demo app to at_app.

Is your feature request related to a problem? Please describe.
N/A
Describe the solution you'd like
Be able to generate the demo app using at_app create -d snackbar <path>
Describe alternatives you've considered
N/A
Additional context
N/A

@XavierChanth I am doing this and whilst doing it providing you some user testing for docs and code ....
this is actually two apps
the snackbar sender (to the flutter web applet
The snackbar e2e encrypted @sign to @sign app

completed and pushed to trunk


With current network_mode there is no exposed IRI port

Following the instructions I become docker container which does not expose port 14265.
When I remove line network_mode: "host" from docker-compose.yml I become this port available.
Is this setting required?

Is your code connecting to node after making these changes?


Update upload-artifact to @v4

This will likely take some fiddling as release artifacts are no longer mutable so we must combine them by hand

addressed in v0.5.1


I just used 4times,but it says "httpx.ConnectTimeout: timed out"

httpx.ConnectTimeout: timed out

pip install pydeeplx==1.0.6

try again.

it doest work

does it support to translate list string

does it support to translate list string

I noticed that the error reported was a timeout. Please ensure that your local network can access Deep L

I can use the Deepl as usual,but if I use deeplx it will be wrong

this is my code
imaybe, i request too fast

Can I translate list of string like the official api
I want to translate a long doc
i first try to put words in one string , it says 413
then i split it into some strings and sleep 1sec in between, but it says"httpx.ConnectTimeout: _ssl.c:1074: The handshake operation timed out"or"PyDeepLX.PyDeepLX.TooManyRequestsException: PyDeepLX Error: Too many requests, your IP has been blocked by DeepL temporarily, please don't request it frequently in a short time." after I use the api for 5times
thanks for your help

Hi! Where can we find more valid user agents? Thanks!


Remove .DS_Store files

it might make sense to remove .DS_Store files and add it to .gitignore

Is there a .DS_Store file checked in that I'm not seeing?

No response in about a year, and I don't see any .DS_Store files in this repo, so closing.


Mitigate Stream Fragmentation and Reassembly Attacks

The stream receive buffer is optimized to reduce CPU costs from frequent allocations: https://github.com/aws/s2n-quic/pull/1497. Additional mitigations may be warranted to protect against excessive memory consumption due to stream fragmentation attacks.
From 21.7. Stream Fragmentation and Reassembly Attacks in rfc9000:

QUIC deployments SHOULD provide mitigations for stream fragmentation attacks.

Mitigations could consist of:

avoiding overcommitting memory
limiting the size of tracking data structures
delaying reassembly of STREAM frames
implementing heuristics based on the age and duration of reassembly holes

The application could use the following metric to determine fragmentation. However we should investigate what exactly this metric means and document it.
The mitigation to the attack would be similar to the Slowloris mitigation; using the supervisor to close the connection (https://docs.rs/s2n-quic/latest/s2n_quic/provider/event/supervisor/index.html)


metacontroller should have a separate examples repo with proper e2e

@AmitKumarDas What is the difference between this bug and #4?
@grzesuav What about creating a metacontroller-contrib repo with user-contributed controllers? I've re-iplemented istio ingress conversions to gateway and virtualservice (because https://github.com/istio/istio/pull/31817 was not accepted), and this would fit in it.
Not sure e2e tests are needed, but testing controllers against predefined inputs is definitively needed.

@sathieu This issue as well as https://github.com/metacontroller/metacontroller/issues/4 are pointing to same stuff. I was thinking to put these as two separate repos since library mode will be typical Golang based & will reduce the learning curve in the long run.
Note: metacontroller still does not support library mode. So contrib repo can be started off.

@sathieu I am more keen to set up https://github.com/metacontroller/metacontroller/issues/85 for this purpose - as I do not think we will support all user-written controllers with their custom logic etc and offer bug fixing and feature requests - but we sure can put a place which can point to all that.
Also, this issue started at the beginning of migration, I am not sure how suitable would be to have metacontroller in one repo and e2e tests in the other as it will be a problem to keep it in sync, and intorducing changes  to both in the same time and still having tests passing


Modules won't hide

I added modules after I added Hello-Lucy, and now none of the modules will hide.

Did you add the new modules to the Hello-Lucy config entry? Did you add them to the sentences array in the sentences.js file. Did you add them to the checkCommand.js file as well? Please follow all the installation instructions provided in the repo. The behavior you are describing is usually from user error.
post any errors from the console at the forum so I can see them


Define primary hosted zones in code, with CI/CD

Similar to that described in #2, we recently had an event where a few people wanted a DNS record and didn't know where to go.
It should be possible to manage DNS records for operationcode.org from within this repository.
To do so:

[x] Create an S3 bucket for Terraform remote state storage.

[x] Create AWS user credentials to provide to CircleCI, along with an IAM policy providing access to the bucket for remote state storage.

[x] Get the DNSimple API key and provide it to CircleCI.

[ ] Use Terraform to define records as they currently exist in DNSimple. DNSimple credentials are in 1Password.

[ ] Run terraform import on all existing resources once the configuration is complete, and run terraform plan to validate. We should see no changes.

[x] Create a record by adding a new Terraform resource definition, and on merge to master it should apply.

Bonus points if terraform plan is run against the PR prior to merging. Consider using terragrunt.

interestingly, found that DNSimple apply will fail after a record is configured in Terraform if it already exists.

I'm not going to exhaustively describe our DNS resources at this moment, I'll open another issue to transpose that info.


better handling of edge cases

https://github.com/cinecert/imf-vtfp/blob/9c0d8167618ef6fae31a4d30279712c28a81e361/imf_vtfp.py#L186-L197
Hey @jhursty ,
I finally got a chance to take a closer look and i think i managed to create 2 CPLs that will demonstrate the issue.
Running these 2 CPLs will yield different fingerprint.
CPL_InternalTest-Bars_OV_Vxxx_FP_DPO_IMFA_HD-SDR_23_ZXX-178_(0-10_11-30)x2.xml.txt
CPL_InternalTest-Bars_OV_Vxxx_FP_DPO_IMFA_HD-SDR_23_ZXX-178_0-40x2.xml.txt
Thanks,
George

Hey John, thank you very much for your feedback, let me try to elaborate.
if an mxf is 40 frames long. both CPLs were supposed to play the entire mxf twice start to finish.
the first CPL was intended to play the following parts
frames[0->10] + frames [11->39] + frames[0->10] + frames [11->39]
while the second would define them as follows:
frames[0->39] + frames [0->39]
you can see those could generate the same fingerprint but i believe with the current implementation they wouldn't or at least i think they wouldn't.
thanks again, and sorry for the delay responding.

Re-opened to handle a new comment.
The example in the comment from June 14 does not match the CPLs posted December 3. Those CPLs do not contain identical virtual tracks for the reason detailed in my comment from June 1. If you believe that the algorithm will produce different results for CPLs constructed as proposed in the June 14 comment, please create and post example CPLs that correctly encode that timeline.

Closing for lack of activity.


Restructured Python project:

Created Python package.
Reworked library initialization.
Added unit test skeleton.
Added pip requirements.

I want to clean up the package stuff a bit so it will plug into GitHub Actions.

I added a simple setuptools config. Next will be to run Python tests as part of the build validation action.
With a little more work, we should be able to drive everything from GitHub Actions, including pushing releases to PyPi & crates.io.

Rebased after the samplers change.


Social Links Email Working

Fix #44 When clicked on email it is now redirecting to email.

This branch has conflicts that must be resolved
Use the web editor or the  to resolve conflicts.
Conflicting files
index.html

#46 Already Commit ed the Same thing Before. You!!


Plans for PartitionExplainer

Hi,
When using PartitionExplainer, I get the following warning:
explainers.Partition is still in an alpha state, so use with caution...
Are there any plans for further development of this explainer?
And is it possible to limit the computation of Owen values only to some level of the clustering dendrogram? I don't need to know the impact of all single features used in a model, I'm only interested in total credits assigned to specific groups. On the example from here: https://shap.readthedocs.io/en/latest/example_notebooks/tabular_examples/model_agnostic/Simple Boston Demo.html, complete clustering looks like this:

But suppose I'd like to cut this tree:

and compute the Owen values for these 5 coalitions. As I understand (please correct me if I'm wrong), credits are computed recursively starting from the top of dendrogram and moving downward. So is there a way to stop the algorithm at selected level and obtain values for coalitions rather than particular features, in order to decrease the time of computations?

Hey! We are working actively on this explainer right now (largely focused on the text/image applications). In fact that warning just got taken off this week :)
As for early stopping, that is possible by setting any clusters distance (height on those plots) to a negative value. That will stop the Partition explainer from going down into those branches. The credit will then be lowered down onto all the leaf nodes in that group for the .values array and left at the internal node for the .hierarchical_values array (those are both properties of the returned explanation object).
If you do this it would be great to share a NB in the docs so others can benefit!


Digits Details_validation vs test

Could you please tell me how I can get details of the accuracy (val) and loss(val) in  Digits? What is difference between validation and test in Digits? more specifically in the attached picture and the diagram
Appreciate your help.

Hello,
in Alexnet and most image classification networks, the loss function that we use is the logistic loss. See definition in Caffe:
http://caffe.berkeleyvision.org/doxygen/classcaffe_1_1MultinomialLogisticLossLayer.html
The optimization goal is to reduce this loss.
The accuracy is a metric that we track the percentage of correctly classified images. See:
http://caffe.berkeleyvision.org/doxygen/classcaffe_1_1AccuracyLayer.html
For an overview of what the train, validation and test sets are:
https://fr.coursera.org/learn/machine-learning/lecture/QGKbr/model-selection-and-train-validation-test-sets

Thank you so much for your response.
I will go through them accurately, but just want to know if the validation is the inference as it is shown in the attached picture?

Yes, "validation" consists in performing inference on your model on all samples from the validation set. Since you know the ground truth for those samples you can compute quality metrics, like accuracy.

Thank you so much again.
Is the validation accuracy being computed at each iteration? or each epoch? Also I don't know exactly how often the weights are being updated while training? Are they being updated at each epoch?

Those questions are better asked on the users list. The validation accuracy is computed on the full validation set at every snapshot interval epoch (configured in DIGITS UI, defaults to 1 and can be a fractional number like 0.5). For the record, one "epoch" is one pass through the full training set. Weights are updated on every iteration (i.e. every batch_size * batch_iteration samples on training set).
Closing.


Add automatic marshaler for .NET methods

Currently, we have to register manually on .NET side methods.
So, there is an idea for:
public sealed class Foo
{
    public Task<int> Add(int a, int b) => Task.FromResult(a + b);
    public Task<int> Sub(int a, int b) => Task.FromResult(a - b);
}

var foo = new Foo();

messenger = new Messenger();

// Current strategy:
messenger.RegisterFunc<int, int, int>(foo.Add);
messenger.RegisterFunc<int, int, int>(foo.Sub);

// Automatic interpretation for declared methods:
messenger.RegisterObject(foo);

Implemented RegisterObject feature on 22b80c286cd4b3afa515e7380dbc816a6d701d20 .


[wip] Feat container factory

This aims to resolve several issues related configurability.
The idea is that the DI container will be used for creating factories.
This is a wip and will need fixes after other PRs are merged. Also it needs to be considered whether we want / need this.

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Merging this PR is the only cleaner way toward a configurable extension and will allow for solving issues like https://github.com/yiisoft/yii2-symfonymailer/issues/62
I have tested locally solving the issue with this PR merged and it is easier than alternative routes.
Any objection to merging this?

No objection from me, sorry for letting this hang! Thanks for finishing it for me 😁

Thanks Sam for great contributions to Yii. Please keep them coming!

This can only be merged after #54 is merged.

Hi @SamMousa can you now resolve the conflicts on this PR?

@SamMousa would you have time to fix the conflicts for final merge.
We have to release a new version and this seems to be the last issue before that!

Thanks for pushing this over the finish line

No problem. Thanks for PR. Keep them coming!


Fix Marker offset, via-point labels in Firefox and maybe re-use Marker.tsx

After #159 we have the following issues related to the query point markers:

[x] The query point markers are slightly shifted from the coordinate they should represent. See this discussion: https://github.com/graphhopper/graphhopper-maps/pull/176#discussion_r787073295_
[x] The via-point labels are no longer visible in Firefox, see here: https://github.com/graphhopper/graphhopper-maps/blob/8518a15177877fe2a940a72845a6faf855c27fa4/src/layers/createMarkerSVG.ts#L24
[ ] We duplicate the svg that is already in Marker.tsx: https://github.com/graphhopper/graphhopper-maps/blob/8518a15177877fe2a940a72845a6faf855c27fa4/src/layers/createMarkerSVG.ts#L11-L12

We duplicate the svg that is already in Marker.tsx:

So far our svg markers are simply drawn on the map canvas, but with OpenLayer's overlays we can create separate div elements where we can put our marker component into. I tried this here: f44c46f9ea52c21fb357160df1155b07b5a246e5
I managed to show the markers using this approach and also the via-point labels are shown correctly in Firefox this way.
Unfortunately, I ran into a problem where it seems like both OpenLayers and React are interfering with the DOM causing React to crash:
https://github.com/graphhopper/graphhopper-maps/blob/f44c46f9ea52c21fb357160df1155b07b5a246e5/src/layers/QueryPoints.tsx#L15-L18
The other issue is that we can no longer use the mouse wheel to zoom the map when the cursor is above one of the map markers. This is because React handles the mouse events and does not forward them to OpenLayers. We will probably run into a similar issue when we try to implement the dragging interaction with this Overlay approach.
Since I found a way to fix the marker offsets without overlays (#184) maybe it's not worth it to switch to the Overlay approach, but then we also need to find another fix for the via-point label issue.

For the crash I would have to look into it to get an idea.

The other issue is that we can no longer use the mouse wheel to zoom the map when the cursor is above one of the map markers.

I would hope that this can be solved with a little bit of css by setting touch-action: none

firefox: adding 'px' helps

Don't ask me why ;)


Added __templatesimplemented to defaultCloudIndexConfiguration

Added __templatesimplemented computed index field to defaultCloundIndexConfiguration for Azure Search support.

Looks good. I'll fix the whitespace post merge.


I can't change labels :(

missing permissions? @fniephaus

Because you haven't accepted the invitation...


Functions.java JavaDoc was added

JavaDoc explains why the method that never returns declares a return type and what pattern to use it.

Coverage remained the same at 95.369% when pulling 947378c504911ed4908200b95a3ff8f0bd2f1ecc on verhas:functions-javadoc into bedae6950d43008929d9cd109c9e8fb163793bd7 on apache:master.

Please do not change "Rethrows" to "Rethrow". Javadoc comments should starts with "Gets", "Returns", "Sets" and so on.

Is there anything else I can do to get this simple change merged?


Docker compose .yml

Hi
Great project! Can you make a Docker Compose .yml file pls?

I will do it in a week.

Did it myself:
panasonic-aquarea:
image: ronhks/panasonic-aquarea-smart-cloud-mqtt
container_name: panasonic-aquarea
restart: unless-stopped
volumes:
- CONFIG_LOCATION:/app/etc

Sorry for late reply, but my daughter born at december, so i had a few time :)
Thanks, I will put on the repository!


Update boost to 1.67

I don't know how to to do this with the new structure

Fixed by https://github.com/Microsoft/vcpkg/commit/751015f6a44d6f89ca71ac63d7c5cab83cf531c0


v2.1.1 start game automatically does not unlock fps

When I open the game automatically the fps is clocked to 60fps, the config file has it set to 144

can you delete this issue please

You can close it manually. If you don't know how to do it, check this: https://docs.github.com/en/issues/tracking-your-work-with-issues/closing-an-issue
I hope that helps!

Nah I know that however I would like to delete it from my profile which is
something I cannot do myself, if you could please do so, I would be really
grateful. Thank you
On Tue, Aug 15, 2023 at 5:36 PM 22621.2070 @.***> wrote:

You can close it manually. If you don't know how to do it, check this:
https://docs.github.com/en/issues/tracking-your-work-with-issues/closing-an-issue
I hope that helps!
—
Reply to this email directly, view it on GitHub
https://github.com/34736384/genshin-fps-unlock/issues/202#issuecomment-1679266534,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APQIDZRQSSY6RRPU4RMZU7TXVOQRFANCNFSM6AAAAAA3G64BYI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sorry I cannot help you because I am not the owner of this program...

@34736384 could you delete it pls... sorry to bother you guys.


flashing/index: Power off Jetson DUT on container flash failure

This should allow the DUT to cool off while not in use, because in rcm-boot mode the fan is not spinning.
Also, with this commit we check if the following error observed on J3010 disappears:
ECID is
Board ID() version() sku() revision()
Chip SKU(00:00:00:D3) ramcode() fuselevel(fuselevel_production)   board_FAB()
emc_opt_disable_fuse:(0)
Error: Unrecognized module SKU
Change-type: patch
Signed-off-by: Alexandru Costache<EMAIL_ADDRESS>

@balena-ci I self-certify!


CB-14959 Add scheduled job to archive old, terminated InstanceMetaData

Terminated InstanceMetaData blows up many DB queries, due to our adhoc
entity relationship designs. These relationships have to be properly
revisited later, however to remedy the issue until the solution arrives,
this commit introduces a scheduled job to archive old, terminated
InstanceMetaData.

new entity/table, called ArchivedInstanceMetaData, this will
serve as a backup table for the imd
scheduled job which runs daily, inserts all InstanceMetaData
which were terminated at least a week ago to the back up table

Still testing


(libev) select: Invalid argument when trying to go past 1k users

MacOS X 10.9, python 2.7 (macports)
Latest locust (github), latest pip packages
Testing a local setup, trying to break past 1000 users. I'm able to load up to 1000 users just fine (after bumping my ulimit), but locust crashes out past that point:
https://gist.github.com/TTimo/8149351

Can anyone confirm if this is still an issue? I'm not able to reproduce with my MBP: libev 4.22 (installed with brew, not macports though)

I confirm this on macOS High Sierra with libev 4.24 (bottled).
Everything seems to work fine with 1000 users, however changing it to 1025 users results in:
[2017-12-02 12:19:09,562] iMac.local/ERROR/stderr: Traceback (most recent call last):
  File "/Library/Python/2.7/site-packages/locust/core.py", line 271, in run
    self.execute_next_task()
  File "/Library/Python/2.7/site-packages/locust/core.py", line 297, in execute_next_task
    self.execute_task(task["callable"], *task["args"], **task["kwargs"])
  File "/Library/Python/2.7/site-packages/locust/core.py", line 309, in execute_task
    task(self, *args, **kwargs)
  File "/Users/mikolaj/Desktop/hello/locust/locustfile.py", line 7, in index
    self.client.get("/")
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 488, in get
    return self.request('GET', url, **kwargs)
  File "/Library/Python/2.7/site-packages/locust/clients.py", line 114, in request
    response = self._send_request_safe_mode(method, url, **kwargs)
  File "/Library/Python/2.7/site-packages/locust/clients.py", line 158, in _send_request_safe_mode
    return requests.Session.request(self, method, url, **kwargs)
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 475, in request
    resp = self.send(prep, **send_kwargs)
  File "/Library/Python/2.7/site-packages/requests/sessions.py", line 596, in send
    r = adapter.send(request, **kwargs)
  File "/Library/Python/2.7/site-packages/requests/adapters.py", line 423, in send
    timeout=timeout
  File "/Library/Python/2.7/site-packages/requests/packages/urllib3/connectionpool.py", line 583, in urlopen
    conn = self._get_conn(timeout=pool_timeout)
  File "/Library/Python/2.7/site-packages/requests/packages/urllib3/connectionpool.py", line 248, in _get_conn
    if conn and is_connection_dropped(conn):
  File "/Library/Python/2.7/site-packages/requests/packages/urllib3/util/connection.py", line 34, in is_connection_dropped
    return select([sock], [], [], 0.0)[0]
  File "/Library/Python/2.7/site-packages/gevent/select.py", line 141, in select
    sel_results = _original_select(rlist, wlist, xlist, 0)
ValueError: filedescriptor out of range in select()

I confirm this on macOS High Sierra with libev 4.24 (bottled).

this is actually different than the original issue described.  You are receiving a totally different stacktrace.

ValueError: filedescriptor out of range in select()

Have you increased file descriptors in your OS?  If not, you need to.
This may be considered a bug in gevent, requests, or urllib3 for not handling the situation gracefully when you run out of file descriptors...  but I don't see an actual Locust issue we can take action on besides aborting a test once you exhaust file descriptors.
I am closing this issue for now since there is nothing actionable to fix for Locust.


UIAccessibility changes

These are the modifications made in TEAChart to enable accessible use of Bar, Contribution and Clock charts as described in issue #19 .

Thanks!
I just made the correction! Let me know if there's anything else.

Thank you! 💯


EDA Localization - Spring '21 drop 1 complete

Critical Changes
Translated labels for SPring '21 drop 1
Changes
Issues Closed
New Metadata
Unpackaged Metadata
Deleted Metadata
Testing Notes
Please check the correct Male/Female labels were left in English

After speaking with Matt, this passes QA. Merging and Closing


Not able to enter site details

I dont seem to be able to enter the site server and the site code during the installation process.

See the below message in the log file.
"Info: Failed to detect ConfigMgr - Running first time demo mode"


Expand eth_call validation to support blockParam object

Signed-off-by: georgi-l95<EMAIL_ADDRESS>Description:
Expand eth_call validation to support blockParam object, introduced with EIP-1898. The Graph is passing blockHash as parameter and currently validations only accepts blockNumber or tag. More info can be found in this issue from this comment onwards.
Fix some minor intermittent bugs in ERC20 tests.
Bump network images to 0.33.2 and mirror-node images to 0.72.0.
Related issue(s):
Fixes #816
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Base: 72.62% // Head: 73.84% // Increases project coverage by +1.22% :tada:

Coverage data is based on head (f3f8c32) compared to base (b3b7650).
Patch coverage: 94.11% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #817      +/-   ##
==========================================
+ Coverage   72.62%   73.84%   +1.22%
==========================================
  Files          29       29
  Lines        1746     1778      +32
  Branches      319      323       +4
==========================================
+ Hits         1268     1313      +45
+ Misses        387      372      -15
- Partials       91       93       +2

Impacted Files
Coverage Δ

packages/server/src/validator/methods.ts
100.00% <ø> (ø)

packages/server/src/validator/types.ts
95.83% <80.00%> (-4.17%)
:arrow_down:

packages/relay/src/lib/eth.ts
82.47% <100.00%> (ø)

packages/server/src/validator/constants.ts
100.00% <100.00%> (ø)

packages/server/src/validator/objectTypes.ts
100.00% <100.00%> (ø)

packages/relay/src/lib/errors/SDKClientError.ts
66.66% <0.00%> (+2.38%)
:arrow_up:

packages/relay/src/lib/clients/sdkClient.ts
20.61% <0.00%> (+14.47%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@Nana-EC Tests case examples in the EIP are about eth_getStorageAt, I've added similar about eth_call, but we are currently not using blockParam parameter, so we can't add all, for now.
According to the EIP we should make the following endpoints accept block parameter ( tag, block number or object with block hash or block number ):

eth_getBalance
eth_getStorageAt
eth_getTransactionCount
eth_getCode
eth_call
eth_getProof

But only eth_call is accepting those in Alchemy, for example. The other endpoints only accept tag or block number (the same as us, currently), both in Alchemy and Infura. Should we follow the EIP fully or only for eth_call ? If yes, maybe in a another PR, as this one will get a lot larger ?

@Nana-EC Tests case examples in the EIP are about eth_getStorageAt, I've added similar about eth_call, but we are currently not using blockParam parameter, so we can't add all, for now. According to the EIP we should make the following endpoints accept block parameter ( tag, block number or object with block hash or block number ):

eth_getBalance
eth_getStorageAt
eth_getTransactionCount
eth_getCode
eth_call
eth_getProof

But only eth_call is accepting those in Alchemy, for example. The other endpoints only accept tag or block number (the same as us, currently), both in Alchemy and Infura. Should we follow the EIP fully or only for eth_call ? If yes, maybe in a another PR, as this one will get a lot larger ?

@georgi-l95 Let's match Alchemy and Infura for now.
Open a ticket for the backlog to capture this for other endpoints are you've noted.

LGTM


[Feature Request] Change 'Groups' into 'Tags' (or otherwise allow multiple groups)

It would be nice to be able to set multiple groups. For example, a lot of my combos would fall under the group of 'Work' because they are for things I do at my job, but I also like to separate it out by common job tasks 'User Support', 'Info Request', 'Upgrade Tasks', etc. There are often snippets that would fit into multiple groups, and it would be nice to be able to have a combo in all applicable groups so that I can just select whatever is most relevant at the moment.

This is an interesting suggestion. I may add support for tags at some point, but I will probably not replace groups as they are now.


Add collaborators

In order to thanks all the current collaborators who contributed their ideas and opinions and therefore helped to form the final (and soon coming) 1.0 release of tus, we would like to mention every one of you in the specification. The order in which the names appear does not have any meaning, we just got it this way from GitHub :).
If you do not want to appear on this list or want to change your name either leave a comment on this issue or drop me an e-mail<EMAIL_ADDRESS>and I will get it sorted as soon as possible.
The plan is to merge this pull request shortly before the pull request for the 1.0 update (#57) which will be in a week or so.
Finally, I want to thank all of you again for helping tus to become what it is now. :+1:
@hintjens @khangtoh @jerefrer @Amodio @srijs @brunodecarvalho @cybic @sandfox @aranw @Baughn @j4james @noptic @schmerg @ehd @homeworkprod @andrewfenn @evert @reschke @kevinswiber @radiospiel @0x20h @qsorix @kless @luk- @tino @ChristianUlbrich @sheeep @jonhoo @wronglink @technoweenie @MarkMurphy @jf @pmdarrow @Gargaj @timemachine3030

Damn, totally missed this. I've since changed the handle from @brunodecarvalho to @biasedbit. Sorry :(

:+1: thanks, appreciate this!

@biasedbit I'll change it : )

I guess now that I have all the collaborators together, it makes sense to share in case you missed it or this wasn't clear: Today we finally launched 1.0.
Blog post: http://tus.io/blog/2015/11/16/tus.1.0/
Some initial press coverage: http://tech.eu/features/6672/transloadit-tus-protocol-vimeo/
Reddit thread: https://www.reddit.com/r/programming/comments/3t0rb2/berlinbased_transloadit_wants_to_fix_broken_file/
HN: https://news.ycombinator.com/item?id=10574364
Ok, back to celebrating :smile: :cocktail: :tada:

Just found about this. Thanks for mentioning me here, and good luck!

@radiospiel Thank you for being part of it :)


Feature request: generate manpage

This may be out of scope, but...
It'd be really nice to have a way to generate a manpage (or at least part of one) from the same USAGE section used to define the actual argument parsing.
For reference: tobimensch/cli2man represents a start toward realizing such a thing in Python (though it hasn't been touched in almost a year).
Ideally, I'd like to be able to generate a complete manpage from a combination of docopts usage message parsing and some other human readable source (i.e. markdown a la ronn). The goal being to prevent mistakes where the manpage gets out of sync with the actual options by maintaining only a single definition.

I think this would be very cool!


How to protect route with this code,I use lazy for instead of element

Here is my code Example
const routes = createBrowserRouter(
  createRoutesFromElements(
    <Route path="/" element={<RootLayout />}>
      <Route index element={<Home />} />
      <Route path="cart" lazy={() => import("../pages/cart/Cart")} />
      <Route
        path="notifications"
        lazy={() => import("../pages/notifications/Notifications")}
      />
      <Route
        path="wishlist"
        lazy={() => import("../pages/wishlist/Wishlist")}
      />

      <Route path="shops" lazy={() => import("../pages/shops/Shops")} />
      <Route path="/shops/:id" lazy={() => import("../pages/shops/Shop")} />

      <Route
        path="categories"
        lazy={() => import("../pages/categories/Categories")}
      />
      <Route
        path="categories/:id"
        lazy={() => import("../pages/categories/Category")}
      />

      <Route
        path="products"
        lazy={() => import("../pages/products/Products")}
      />

      <Route
        path="products/:id"
        lazy={() => import("../pages/products/Product")}
      />

      <Route
        path="search/product"
        lazy={() => import("../pages/search/Search")}
      />

      <Route path="login" lazy={() => import("../pages/auth/Login")} />
      <Route path="register" lazy={() => import("../pages/auth/Register")} />
      <Route path="profile" lazy={() => import("../pages/profile/Profile")} />
    </Route>
  )
);

This is a very good usecase. I'll check and let you know.


not working with cordova 6.x on ios.

lock orientation is totally broken.
also screen.orienatations all returned wrong

In my app, with cordova 6.x, orientation is detected only in portrait modes (either primary and secondary), but, the "orientationchange" event is not fired when orientation is changed to landscape.

I have the same problem

Me too, any solution ?

I have the same problem

I had a similar issue on a new version, the solution was to be sure all orientation methods are enabled in the plist (or GUI).
Matthew

@enckecr  could you show me how to edit plist to fix it?
I enabled all Device Orientation and Requires full screen in xcode but it doesn't work

<key>UISupportedInterfaceOrientations</key>
<array>
	<string>UIInterfaceOrientationPortrait</string>
	<string>UIInterfaceOrientationPortraitUpsideDown</string>
	<string>UIInterfaceOrientationLandscapeLeft</string>
	<string>UIInterfaceOrientationLandscapeRight</string>
</array>

@enckecr  doesn't work on my app. The page flicker to landscape then back to original orientation. Don't know why.

Make sure that you are running it after the "device ready" event.
screen.lockOrientation( "landscape" );
Also, it is always good to wrap these events with a timeout
window.setTimeout( function() { screen.lockOrientation( "landscape" ); }, 0 );

On Apr 21, 2016, at 10:17 PM, raykin<EMAIL_ADDRESS>wrote:
@enckecr https://github.com/enckecr doesn't work on my app. The page flicker to landscape then back to original orientation. Don't know why.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/gbenvenuti/cordova-plugin-screen-orientation/issues/96#issuecomment-213244145

@enckecr  i can even debug it in safari console with screen.function,  but just not work. If the screen is not exist, it will through error like undefined has no function. But when I debug, no error happened.

Add this line after setting the orientation
console.log( screen );

On Apr 22, 2016, at 8:59 AM, raykin<EMAIL_ADDRESS>wrote:
@enckecr https://github.com/enckecr i can even debug it in safari console with screen.function, but just not work. If the screen is not exist, it will through error like undefined has no function. But when I debug, no error happened.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/gbenvenuti/cordova-plugin-screen-orientation/issues/96#issuecomment-213464062

@enckecr  return Screen {lockOrientation: function, unlockOrientation: function, orientation: "portrait-primary", height: 667, width: 375, …}

Since many guys said it's not worked in ios9.3, so I guess there are some trick in ios9.3 self, not in the plugin. But the fix may require a workaround by editing plugin.

run it like this, before and after the orientation change and look at the difference, the "orientation" value should change.
console.log( screen.orientation );
screen.lockOrientation( "landscape" );
console.log( screen.orientation );

On Apr 22, 2016, at 9:06 AM, raykin<EMAIL_ADDRESS>wrote:
@enckecr https://github.com/enckecr return Screen {lockOrientation: function, unlockOrientation: function, orientation: "portrait-primary", height: 667, width: 375, …}
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/gbenvenuti/cordova-plugin-screen-orientation/issues/96#issuecomment-213466246

@enckecr  not changed.
console.log( screen.orientation );
portrait-primary
screen.lockOrientation( "landscape" );
undefined
console.log( screen.orientation );
portrait-primary

@enckecr  just as I said, there is a orientation flicker then back to origin

Try installing the plugin from the GIT url and not form the NPM.
cordova plugin rm cordova-plugin-screen-orientation
cordova plugin add https://github.com/gbenvenuti/cordova-plugin-screen-orientation.git
Then try the same console.log code with the changes to see if that fixes it. In the notes on #89 (https://github.com/gbenvenuti/cordova-plugin-screen-orientation/pull/89 https://github.com/gbenvenuti/cordova-plugin-screen-orientation/pull/89), we see a note that they  have some issues with the update in the NPM
Matthew

On Apr 22, 2016, at 9:16 AM, raykin<EMAIL_ADDRESS>wrote:
@enckecr https://github.com/enckecr just as I said, there is a orientation flicker then back to origin
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/gbenvenuti/cordova-plugin-screen-orientation/issues/96#issuecomment-213469148

I'm already use the git url for this plugin.
➜  kasa git:(master) cat package.json | ag orientation
      "locator": "https://github.com/gbenvenuti/cordova-plugin-screen-orientation",
      "id": "cordova-plugin-screen-orientation"

Hi all,
Thanks to @tony-- there's now a PR in progress, you can install the development version for testing, please removed existing plugin first
cordova plugin add https://github.com/gbenvenuti/cordova-plugin-screen-orientation.git#iosTesting

It currently does not unlock but if you are interested in testing please do so.

I have the same problem as raykin, the official release will not lock screen at all on iOS, but when installing cordova plugin add https://github.com/gbenvenuti/cordova-plugin-screen-orientation.git#iosTesting
it does work to lock the orientation, although "portrait" is the same as "portrait-primary", and as said it's not possible to unlock it again.

Found a small bug in plugin.xml in #iosTesting, the file "src/ios/CDVViewController+UpdateSupportedOrientations.h" is marked as a source file rather than a header file. It only causes a compile warning in XCode so I think it's not harmful, but to get rid of that just change source-file to header-file for that entry in plugin.xml.

@atorger does the event fire for you? I use the event to setup the view, so calendar week view for landscape and day view for portrait. On flipping the phone i want to switch views. This is no longer working as the event is not firing in ios.

@arendjantetteroo , I haven't checked, but if it doesn't work for you it probably doesn't for me. I was lucky enough to have a very simple use case in my app: "at startup: if small screen permanently lock the orientation to portrait, otherwise leave unlocked.", so I don't need the events to work, just the locking.

Thanks @atorger!  I fixed the declaration.

@midorikocak - @gbenvenuti has released 1.4.2 which should fix this issue. Will you try 1.4.2 and provide an update, please?

@tony--  The orientationchange event still does not fire for me on 1.4.2. Any idea what i can do about that?

@tony--  Ok, it is working if you make sure the Device orientation has all 4 items checked, otherwise the event won't fire. Might be good to add that to the Readme on the Event Listener.

@arendjantetteroo - In my testing I did not directly modify the plist to change supported orientations, but I did test different Cordova Orientation preference settings (which causes the supported orientations in the plist to be modified).  You should not need to enable all the orientations in XCode.  My default test case was an unmodified Cordova app which is portrait only, but I also tested with Landscape and All - a few others in other Issue/PR conversations had trouble with their existing apps but success with a clean app.
Could you try with a clean app?
cordova create screen-orientation-test
cd screen-orientation-test
cordova platform add<EMAIL_ADDRESS>cordova plugin add https://github.com/tony--/cordova-plugin-screen-orientation#fix-ios-rotate-on-lock
cordova run ios

The test app should start in portrait and not auto-rotate.
Using Safari Web Inspector Console
screen.lockOrientation('landscape')
//should rotate to landscape left and should auto-rotate between landscape orientations
screen.unlockOrientation()
//should rotate to portrait and no longer auto-rotate when device is tilted

@tony-- Thanks for the reply, i'll try to take a look at the clean app.
I'm glad it's working again with the plist changes though :)

AFAIK this issue is resolved so I'm going to close this.
OP or other commenters - please provide additional feedback if this issue is not resolved for you.

If it helps anyone: we fixed the issue, which occurred after upgrading to cordova 6, by adding the following lines to our config.xml:
<platform name="ios">
  <preference name="Orientation" value="all" />
</platform>

Thanks @crtvhd.


OpenSSL 1.1.0-stable sends empty extension as the last on in Client Hello

When running f4cdd19a6a8a0845c08c, the Client Hello ends with two extensions, both empty: EtM and EMS. This behaviour on OpenSSL part causes extension intolerance in some TLS servers.
See https://bugzilla.mozilla.org/show_bug.cgi?id=1243641 for the same issue in NSS.

Fixed via the PR's listed above.  Thanks!


string concat operation seems to be executed wrongly (for version 2.4.14.final)

the demo code is as follow

The expected result should be:
[33, 44] contains 33 and [33, 44] contains 44
,while the actual result is as follow:

BTW, version 2.4.13.final is fine though

The + operators in the expression are executed in sequential order. The first one is interpreted as concatenating an object to a list to create a list with 3 items (33, 44 and "contains 33 and"). Incidentally, in Java, the list item type argument is retained for compile-time only so at run-time nothing prevents you from adding a String to a List.
A workaround would be to prepend ""+ at the beginning to force the concatenation to be string-based:
		String expr = "\"\" + items + \" contains 33 and \" + items + \" contains 44\"";

This yields the following (which you were expecting):
[33, 44] contains 33 and [33, 44] contains 44

It's interesting that the list-based concatenation does not extend to the following + operators. The results actually differ when calling eval (direct execution of string) instead of executeExpression:
[33, 44,  contains 33 and , 33, 44,  contains 44]

This inconsistency between eval and executeExpression could be considered as a bug but the fixed results would still not fulfill your expectations.


Crashing game while watching replays and a goal is scored

When the first goal is scored in a replay the game crashes.

Hello, I am aware of this problem. It is fixed in the v3.0.3. It will be fixed as soon as the BakkesMod team update my plugin on their side.

It has been updated today on the bakkesmod side, can you tell me if this solved your problem?


Walchain license

I see in the doc that: "Walchain is licensed under Apache 2."
Was it officially validated by  Walchain members  ?

It was discussed briefly. The sentence obviously concerns the node source code in this repo. Any suggestion for an alternative ?


Bundle build failed

injection console:
Useless use of numeric eq (==) in void context at /Users/pengxungui/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 619.
buidRoot: /Users/pengxungui/Library/Developer/Xcode/DerivedData/Foowwphone-cmgbfxbcjdcuoecbhyxwoscjhecs/Build
logDir: /Users/pengxungui/Library/Developer/Xcode/DerivedData/Foowwphone-cmgbfxbcjdcuoecbhyxwoscjhecs/Logs/Build

Use of uninitialized value $localBundle in substitution (s///) at /Users/pengxungui/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 156.
Use of uninitialized value $localBinary in substitution (s///) at /Users/pengxungui/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 157.
Use of uninitialized value $localBundle in concatenation (.) or string at /Users/pengxungui/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 160.
Reference found where even-sized list expected at /Users/pengxungui/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 270.
sh: Multiple: No such file or directory

real	0m0.002s
user	0m0.001s
sys	0m0.001s

*** Learnt compile failed

 at /Users/pengxungui/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/common.pm line 61.
	main::error('Learnt compile failed') called at /Users/pengxungui/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 437
 Bundle build failed ***

-I<Multiple -Ivalues> ?

What kind of Xcode do you use?
What Import path do you have in "Swift Compiler - Search Path" (Build settings of your Foowwphone target)? Does these search path depend on configuration)?

My project search path went wrong， it work well when fixed
thanks for your help,the function is very powerful


Design System > Elements > Carousel : Few Enhancements

Design System > Elements > Carousel : Few Enhancements
https://www.figma.com/design/vziFLZAgMFi8wA5SlikLh5/Raaghu-Design-System?node-id=1319-4853&node-type=canvas&t=84ekWaM7By5ywc4L-0
Docs	1]  Carousel developement different than design
Default	1]  Need to implements the 4 styles as per wireframe

Fixed on staging


Windows worker nodes do not upgrade to k8s 1.23

Rancher Server Setup

Rancher version:
Installation option (Docker install/Helm Chart): docker install

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): 1.23.3k3s

Information about the Cluster

Kubernetes version: windows custom rke1 cluster
Cluster Type (Local/Downstream): 3 linux nodes, 3 windows 2019 worker nodes

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom) std user

Describe the bug
Windows worker nodes do not upgrade to k8s 1.23
To Reproduce

Deployed a windows 1.21.9 cluster on 2.6.3
Do a rancher upgrade to 2.6.4-rc8
Upgrade windows cluster's k8s version to 1.23.4-rancher1-1

Result

WIndows worker nodes are stuck in cordoned state on Cluster explorer -->> Nodes page
Rancher logs:

2022/03/11 19:18:55 [ERROR] error syncing 'c-h5frb/m-8ce49329d0df': handler rke-worker-upgrader: setNodePlan: error getting node plan for [m-160fcaf2696d]: getDefaultKubernetesServicesOptions: No serviceOptions found for cluster version [v1.23.4-rancher1-1] or cluster major version [v1.23], requeuing
2022/03/11 19:18:55 [INFO] Provisioning cluster [c-h5frb]
2022/03/11 19:18:55 [INFO] Updating cluster [c-h5frb]
2022/03/11 19:18:55 [INFO] cluster [c-h5frb] worker-upgrade: call upgrade to reconcile for node [m-8ce49329d0df]
2022/03/11 19:18:55 [ERROR] error syncing 'c-h5frb/m-8ce49329d0df': handler rke-worker-upgrader: setNodePlan: error getting node plan for [m-160fcaf2696d]: getDefaultKubernetesServicesOptions: No serviceOptions found for cluster version [v1.23.4-rancher1-1] or cluster major version [v1.23], requeuing

Are general k8s upgrades from 1.21 to 1.23 working?
The error seems to indicate it's not Windows specfic.

RKE1 1.23 Windows was not working due to the k8s service options for windows missing from the initial 1.23 PR in rke1. This has been addressed by the Windows Team and is ready to be tested again with latest 2.6-head once it is published https://drone-publish.rancher.io/rancher/rancher/7495

Validated this issue.
Validation steps followed are as follows.

Created a rancher server v2.6.3
Provisioned a RKE1 Windows cluster k8s v1.21
Upgraded rancher to v2.6.4-rc9
Updated windows k8s cluster from v1.21 to v1.23 successfully.

However blocked by the [#36898 ] and [#36896] issue on windows cluster.

@sowmyav27 @slickwarren  I'd be willing to close this issue as the upgrade to k8s 1.23 is now working as the service options in RKE1 have been updated to include the necessary steps that @vivek-infracloud has taken.
The other issues that @vivek-infracloud has outlined as blockers are not blockers for the upgrade but something else in provisioning and the updates/changes would happen in another part of the code. This was only missing something in KDM when introducing a new minor release.

Since the k8s upgrade was a success closing this issue.


Cannot use client library with a Web Worker

Summary
The client library is unusable within Web Workers, which are useful for asynchronous I/O operations like fetching and processing external data and loading/running WASM modules or other libraries that may need to be isolated in a separate thread, allowing the main UI thread to remain responsive.
The cause appears to be that window.ddClient is required by the client library but only injected into the main UI thread, and not in Web Worker threads.
The fix would likely be to inject a ddClient into every Web Worker thread's environment, in addition to the main UI thread.
Attempted Workarounds

Transferring the ddClient object into the Web Worker with the postMessage transfer param - fails with not transferrable error.

Repro

Create new extension
docker extension init MyExtension

Create a Web Worker that attempts to import and use client
MyExtension/ui/src/webWorker.js
import {createDockerDesktopClient} from '@docker/extension-api-client';

const client = createDockerDesktopClient()

Append Worker initialization to index.js
MyExtension/ui/src/index.js
...
import App from "./App";

// eslint-disable-next-line import/no-webpack-loader-syntax
import MyWorker from "worker-loader!./webWorker.js";

const worker = new Worker(new MyWorker())

ReactDOM.render(
...

Install extension with debug window
make install-extension
docker extension dev debug <myhubuser/myextension>

Navigate to extension in Docker Desktop and see error in Debug Console
Uncaught ReferenceError: window is not defined
    at index.js:2:35
    at webWorker.js:3:16
    at webWorker.a94eac44.worker.js:1:257

I realize this will probably need to be fixed in Docker Desktop, not here in the SDK, but I wasn't sure where to file the issue where it would be most easily tracked.


Better error when bad password is submitted

Connecting with a user with a bad password results in "Tenant or user not found" error which makes you think that tenant or user is not in the metadata database.
Needs to respond with some other error message.
Update log message too:
Thu Feb 08 2024 23:36:31 ClientHandler: User not found: :not_found {:single, "postgres", "wctcdqwibdeirlvhchop"} event body

Can't replicate:

And when I try a user which does not exist:

Closing for now...

Updated and re-opened


Time series formatter

This is the same branch we previously worked on, so I think you could just review the time_series_formatter.py under transformers for now(probably the test functions as well). Converters between sktime, pyreal, and numpy formats are implemented. Let me know if I am missing something.

This branch is now full of clutter. The issue will be resolved by another pull request.


cmd: crictl: add update container command

@feiskyer @Random-Liu PTAL
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

@feiskyer sure, this is green now

FWIW we have a test for this in CRI-O https://github.com/kubernetes-incubator/cri-o/pull/947/commits/c6e588ba9510f5779de8aca62dd5a3d737e55c74#diff-c5753339854697f7106d65aea5be6788R899 and it works just fine


code works fine in chrome but not in codecamp compiler

Challenge Nest an Anchor Element within a Paragraph has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

Sometimes the website "jams"; in those cases try refreshing the page and submit your code again.
If that doesn't work try deleting the cache as well as reloading, then submit again your code.


NOISSUE: majority revert fallback

- What I did
- How I did it
- How to verify it
- Description for the changelog

Jepsen tests are passed.


[Player] Prepare module for event producer

Export EventSender namespace as type/interface in the common package
Add a setEventSender handler to player module

Modelled after setCredentialsProvider for consistency.

I have a couple of more questions :-)

Why do we need to move this type to Common in the first place? if 'Playeris only using it as adevDependencyin the other PR it should not matter if the type is imported fromEventProducer` directly?
If we want this in Common I'm not sure we want to disable rollupTypes, I think we had problems before when we needed to use that in some packages? It does not seem to me like the added bundledPackages (https://github.com/tidal-music/tidal-sdk-web/pull/91/files#diff-18384a37e9745351ae8f00189097846fbd0c20a0e95d704978e6058f8bb2157dR15) is doing anything? so maybe better to remove that and keep rollupTypes?

Why do we need to move this type to Common in the first place? if Player is only using it as a devDependency in the other PR it should not matter if the type is imported from EventProducer directly?

The idea was to have the same flow as for the CredentialsProvider. I'll change so the player uses it directly.

If we want this in Common I'm not sure we want to disable rollupTypes, I think we had problems before when we needed to use that in some packages? It does not seem to me like the added bundledPackages (https://github.com/tidal-music/tidal-sdk-web/pull/91/files#diff-18384a37e9745351ae8f00189097846fbd0c20a0e95d704978e6058f8bb2157dR15) is doing anything? so maybe better to remove that and keep rollupTypes?

w/o rollupTypes it was needed not not include the entire EP in Common if I remember correctly. Pushed a commit in the other PR to use EP directly.

When you get back, we should try and get this out.


if it is possible to find multiple matches?

Hi, I wonder if this action could find multiple matches? From the readme, it seems that it only supports first and last matches?

https://github.com/peter-evans/find-comment/blob/dea9995255c24229e57e55462943072f02c4fec8/src/find.ts#L51-L75
Looks like we can change this to return all the matches? Is that a json format?

Hi @xihajun
The goal of this action is to find a specific comment, not a list of comments. The main issue with finding multiple comments is returning the output.
I don't think this is something that I want to try and support in this action. My suggestion would be to call the GitHub API yourself for this use case using github-script.


ImportError: cannot import name 'DEFAULT_PALETTE'

When Travis CI runs my tests I got
File "/home/travis/build/softwaresaved/lowfat/lowfat/views.py", line 16, in <module>
    from bkcharts import Bar, Histogram
  File "/home/travis/virtualenv/python3.6-dev/lib/python3.6/site-packages/bkcharts/__init__.py", line 5, in <module>
    from bokeh.plotting.helpers import DEFAULT_PALETTE; DEFAULT_PALETTE
ImportError: cannot import name 'DEFAULT_PALETTE'

FYI, from the project README:

bkcharts is currently unmaintained. For actively developed, supported, very high level charting on top of Bokeh, see the HoloViews project: http://holoviews.org/

Additionally, many types of categorical (bar, scatter, heat map) plots there were previously difficult or hard with core Bokeh are now simple with bokeh.plotting:
https://bokeh.pydata.org/en/latest/docs/user_guide/categorical.html

@bryevdv Thanks for the information. A few months ago I got

BokehDeprecationWarning: The bokeh.charts API has moved to a separate 'bkcharts' package.

on my tests so I made the change. I will use bokeh.plotting or HoloViews. Feel free to close this issue.

FWIW I made my way here debugging the same error, changed over to bkcharts because of the same BokehDeprecationWarning.


Relations definition.

First off, great work, simple and uses up-to-date ECMAScript which is what I've been looking for.
My only real gripe so far is how relationships are defined.
In this example:
import Database from './../database';

class Employee extends Database.Model {
  static get tableName() { return 'employees'; }

  static get related() {
    return {
      company: this.belongsTo('Company'),
    };
  }
}

export default Database.register(Employee);

How hard would it be do you think to go more along how Bookshelf/Eloquent does it?
import Database from './../database';

class Employee extends Database.Model {
  static get tableName() { return 'employees'; }

company () {
  return this.belongsTo('Company')
}

export default Database.register(Employee);

Also is the Database.register at the bottom required? I'd like to incorporate this into the ES2015+ MVC framework I am building instead of rolling my own ORM, so I'll probably be contributing in the future.

Database.register works like the registry plugin for Bookshelf. It was made to avoid cross-referencing Models by importing each other.
Basically, I avoided Bookshelf's syntactical solution for relations because Models in Knex-ORM are simple javascript objects with properties. For instance, you could create and save a new Model as follows:
const employee = new Employee {
  company_id: 2,
  name: 'John Doe',
};

const result = await employee.save(); // result instanceof Employee === true
const company = await result.fetchRelated('company'); // company instanceof Company === true

In addition to this, I prefer avoiding Model-mutating methods, that's why I use 3 different constants in the example above. All of them should be untouched by Model methods.
By the way, I decided to go with the Model.related name for Relations because it expressively belongs to the Model methods withRelated() (static) and fetchRelated() (instance).

I can dig that. Thanks for the great work, I'm pretty pumped about this ORM.

Feel free to ask questions, submit suggestions or report bugs! Also, you may re-open this issue if you have more to discuss about Model relations.


Scan Pull Request by Trigger on Jenkins

[ ] All tests passed. If this feature is not already covered by the tests, I added new tests.
[ ] This pull request is on the dev branch.
[ ] I used gofmt for formatting the code before submitting the pull request.

Refactor Scan Pull Request command to allow the use of Jenkins with Generic Triggers.
Effect of course the multi Scan Pull Requests command.
Refactor includes:

The cloning of the repository is moved to the start of Frogbot run, ( to avoid cases where we started scan, but couldn't operate because of bad credentials etc..). Now we can handle both situations where we are already cloned, Or where we are not, for example multi repo on Jenkins.
Adjust tests to work with the local git operations ( checkout to PR branches etc..)
Scan Pull Requests command now just call scan-pull-request command with the needed info.
Scan pull request command optional can accept PR ID and operate based on that information.

Sample scan pr logs:

16:06:23 [🔵Info] Frogbot version: 0.0.0
16:06:24 [🔵Info] Running Frogbot "scan-pull-request" command
16:06:25 [🔵Info] Scanning Pull Request ID: 288
16:06:25 [🔵Info] Auditing branch BA-10285-Frogbot-0 ...
16:06:27 [🔵Info] JFrog Xray version is: 3.77.4
16:06:27 [🔵Info] Auditing project: /Users/eyalde/onboarding/tests/npmexample/mytest/npmexample ...
16:06:27 [🔵Info] Detected: npm.
16:06:28 [🔵Info] Scanning 58 npm dependencies ...
16:06:28 [🔵Info] Waiting for scan to complete on JFrog Xray...
16:06:35 [🔵Info] Auditing branch main ...
16:06:37 [🔵Info] JFrog Xray version is: 3.77.4
16:06:37 [🔵Info] Auditing project: /Users/eyalde/onboarding/tests/npmexample/mytest/npmexample ...
16:06:37 [🔵Info] Detected: npm.
16:06:37 [🔵Info] Scanning 32 npm dependencies ...
16:06:38 [🔵Info] Waiting for scan to complete on JFrog Xray...
16:06:40 [🔵Info] Xray scan completed
16:06:41 [🔵Info] Successfully scanned pull request ID: 288
16:06:41 [🔵Info] Frogbot "scan-pull-request" command finished successfully

** Suggestion: Maybe move the scan logs to debug?
Link to docs update PR:
link

Closed due split PR into smaller chuncks


How To Display text beneath generated barcode

Hello
I want to display text beneath generated barcode.
So how can i do that?
Thanks

This is out of scope for this project, but you can use any standard Android method to add text to the Bitmap, or just render another View below it.