Better docs on common use cases please

Firstly, I really like what you've done here- makes it really fast to create pages.
However, I'm having difficulty with some very simple things: can't seem to get the value from input fields, etc.  I've tried the Roitjs documented methods, jQuery etc. None seem to work.
In the end, I had to resort to standard HTML to get things working.
What would solve the issue for me, is more examples of common usage scenarios.
Lastly, how to integrate this with the material grid system? I've included the materialize.css stylesheet and i can use the rest of the materialize css grids, etc. But the input field has a double underline when selected.

@adavie1,
Thanks. I'm not sure that it is possible to integrate some grid system painless... Because riot does not have method to incapsulate styles of its components. Unfortunately it cannot be implemented right now easily. So you have to work with sources of components to change styles which could conflict with styles of your grid system.
About documentation - I got it. Could you please provide more examples about what problems you had? I will check out docs and examples of material-input for sure. Are there another components you had problem with?

@adavie1 ,
And extra idea: If you want you use some grid system and some material components with it I recommend you yo take a look on Polymer. It is web component based framework helping you to build components with style encapsulation.


[NSPlaceholderString initWithFormat:locale:arguments:]: nil argument' *** First throw call stack:

Hello
I just downloaded and added this to my project.
I am getting the error above. Whenever I show the CTAssestsPickerController and select which folder of pictures I want to go into this error is shown.
Any ideas on what is happening or how to resolve it?

@AJBeltran3 May I know whether have you localised your app?

@AJBeltran3 I had the same problem. There is problem in founding localization on special bundle. In NSBundle+CTAssetsPickerController.h you can find special define CTAssetsPickerControllerLocalizedString.

@alexagarkusha Thanks for your supplement. I would recommend adding this control by cocoapods. It manages all dependency and builds the bundle automatically. The upcoming major release will only support cocoapods.
If you really need manually manage the dependency, please refer to SUBMODULES-NOTES for building the bundle.


Sign out tries to redirect user to last workspace, gives error

When I sign out from Terra while viewing a workspace, I go to the hamburger menu and click "Sign Out". Then I see the sign in page, and I sign in with a new identity. Terra then tries to load the last workspace that I was viewing. However my new identity often doesn't have access, so I get a "no access" message, which is confusing.
2 ideas for how to handle this:
1: include a button to go Home on the "Workspace not found" page. Currently it says:
Could not display workspace
You are trying to access a workspace that either does not exist, or you do not have access to it.

To view an existing workspace, the owner of the workspace must share it with you or with a Group of which you are a member. If the workspace is protected under an Authorization Domain, you must be a member of every group within the Authorization Domain.

If you think the workspace exists but you do not have access, please contact the workspace owner.

I think it should include a button to "Return to Home" at the button.

When a user logs out and logs back in, direct them back to #workspaces. This is instead of loading the page from where they logged out.

I'm labeling this as a bug as it is a pretty unideal UX, though it is by design.

Add a "Return to workspace list" button to the bottom of the message.


Laravel 8 Support

Please add support for Laravel 8.

Hi @rawilk, Thanks for opening this issue. I just tagged a new release. Sorry for being a bit tardy!

No worries! Thanks for getting to it so quickly.
Great work on the package btw. It takes all the effort out of making these shapes yourself.


⚠ Bofrost.com has degraded performance

In 4c07737, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 716 ms

Resolved: Bofrost.com performance has improved in e9c7a3b after 1 hour, 28 minutes.


WatchESPN plugin fails with "Unable to extract post url"

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.07.18 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
United States
Provide a description that is worded well enough to be understood
The WatchESPN functionality with the --ap-mso flag and login information doesn't seem to work. It fails with the error "Unable to extract post url", and an instruction to report an issue.
I imagine the WatchESPN plugin needs to be updated to handle a site change.
I don't have a proposed solution because I don't understand the yt-dlp architecture well enough to propose a solution.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '--ap-mso', 'DTV', '--ap-username', 'PRIVATE', '--ap-password', 'PRIVATE', 'https://www.espn.com/watch/player/_/id/55a598f5-7e07-4e81-86bf-cf71e0c3a172']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.07.18 [135f05e]
[debug] Python 3.10.5 (CPython 64bit) - macOS-12.5-arm64-arm-64bit
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 5.1 (setts), ffprobe 5.1
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.07.18, Current version: 2022.07.18
yt-dlp is up to date (2022.07.18)
[debug] [WatchESPN] Extracting URL: https://www.espn.com/watch/player/_/id/55a598f5-7e07-4e81-86bf-cf71e0c3a172
[WatchESPN] 55a598f5-7e07-4e81-86bf-cf71e0c3a172: Downloading JSON metadata
[WatchESPN] 55a598f5-7e07-4e81-86bf-cf71e0c3a172: Downloading Provider Redirect Page
[WatchESPN] 55a598f5-7e07-4e81-86bf-cf71e0c3a172: Downloading Provider Login Page
ERROR: [WatchESPN] 55a598f5-7e07-4e81-86bf-cf71e0c3a172: Unable to extract post url; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "/opt/homebrew/Cellar/yt-dlp/2022.7.18/libexec/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 644, in extract
    ie_result = self._real_extract(url)
  File "/opt/homebrew/Cellar/yt-dlp/2022.7.18/libexec/lib/python3.10/site-packages/yt_dlp/extractor/espn.py", line 390, in _real_extract
    auth = self._extract_mvpd_auth(url, video_id, 'ESPN', resource).encode()
  File "/opt/homebrew/Cellar/yt-dlp/2022.7.18/libexec/lib/python3.10/site-packages/yt_dlp/extractor/adobepass.py", line 1710, in _extract_mvpd_auth
    mvpd_confirm_page_res = post_form(provider_login_page_res, 'Logging in', form_data)
  File "/opt/homebrew/Cellar/yt-dlp/2022.7.18/libexec/lib/python3.10/site-packages/yt_dlp/extractor/adobepass.py", line 1390, in post_form
    post_url = self._html_search_regex(r'<form[^>]+action=(["\'])(?P<url>.+?)\1', form_page, 'post url', group='url')
  File "/opt/homebrew/Cellar/yt-dlp/2022.7.18/libexec/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 1230, in _html_search_regex
    res = self._search_regex(pattern, string, name, default, fatal, flags, group)
  File "/opt/homebrew/Cellar/yt-dlp/2022.7.18/libexec/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 1194, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)

DTV support is broken. See pinned issues


Can we get the maintained version of materialize on CDN?

I use mostly CDN for most of my projects, and I know many others do as well, and currently I cannot use our current maintained materialize css because it is download and use only. I there any way we can get it on a CDN so I can include it in my newer projects?

This is mentioned in the docs: https://materializecss.github.io/materialize/getting-started.html

Of course !


feat(ssg): generate files concurrently

Continued from #2169.
Deletion of Commit
I've divided detailed topics into separate PRs and issues.

#2186
https://github.com/honojs/hono/issues/2175

https://github.com/honojs/hono/pull/2174
https://github.com/honojs/hono/pull/2178

The default number of concurrency in SSG is 2.
The default value is debatable, but for this PR it was set to 2. The reasons are as follows.

Even at 2 (or 1), this PR will parallelize writes to the file system, which is expected to be a sufficient speed improvement over the current main
When calling external APIs, it is not good if the default is too large

Users can easily specify their own values with concurrency, so a small default is fine.
Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

My work is done.
If there is enough discussion in #2169 to form a consensus on the implementation policy, I think we can merge in this PR content.

It seems to have been CLOSED by some accident.

@watany-dev
Thanks for the notice.
@usualoma
I've checked this again and run this real situation; it works well. I'd like to merge this, so please merge the "next".

@yusukebe merged!

@usualoma
Thanks! Merging now.


Bridging Issue

My case is complicated: I used Safe, and got into the Superfluid inside the Safe, then I used the Bridge in it(Li.Fi). Then I bridged my USDC on Arbitrium to Optimism. However, it seems that the USDC in that address are not really on OP chain and it's a kind of OP version's USDC on Arb address so that I cannot just send it to my another wallet.
Then I tried another way, I used the Li.Fi again to bridge the USDC from Op back to Arb. However, I couldn't bridge it back since it shows that I need to switch chains to OP. But if I switch to OP, the safe's address will change.
My main wallet address: 0xc83c3B46D8E622b9C7b540872c6AF8636DdC1D17
②My Safe’s address on Arb： 0xA9aA585E499d02FbDd492cF4928e4C988dB1855B
③My safe's address on Op: 0x6B7870c45268b38b162BCC57C470b5d572c21d1A

@reyserveth pls use https://github.com/5afe/safe-support

@reyserveth pls use https://github.com/5afe/safe-support

Thanks a lot. Let me add more information :
Now the USDC is on OP chain (https://optimistic.etherscan.io/address/0xa9aa585e499d02fbdd492cf4928e4c988db1855b#tokentxns). But I cannot control it since my Safe address on OP is different from that of on Arb. So is there any possibility that Safe can give me the control of 0xa9aa585e499d02fbdd492cf4928e4c988db1855b on OP?（now I only have the control of this addresss on Arb）

Transferred already. Thank you


Update push-container-image config and not rely on upstream command flag

@gfulton-redhat Yes, I think we should. In order to do so, we'll need to add a repository within the ploigos organization and give a robot account write privileges to that repository. We'd then to update the robot account creds within this repository.


Apps importing grequests crash when a restart/reload is attempted

Expected Behavior
Flask apps run in debug mode (importing grequests) should reload when filesystem changes are detected.
Actual Behavior
Instead, the following app crashes as changes are introduced and an app reload is attempted. Oddly enough, if the app is started with grequests commented out, and then uncommented, reloads process successfully.
import flask

import grequests
import sys

app = flask.Flask(__name__)
app.debug = True

@app.route('/syspath')
def syspath():
    return '\n'.join(sys.path) + '\n'

$ FLASK_APP=demo.py FLASK_DEBUG=1 flask run --host=<IP_ADDRESS>
 * Serving Flask app "demo"
 * Forcing debug mode on
 * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)
 * Restarting with stat
 * Debugger is active!
 * Debugger PIN: 313-109-065
 * Detected change in '/home/bpaterni/usr/local/src/python/flask-app-restart-demo/demo.py', reloading
 * Restarting with stat
Traceback (most recent call last):
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/bin/flask", line 11, in <module>
    sys.exit(main())
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/flask/cli.py", line 513, in main
    cli.main(args=args, prog_name=name)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/flask/cli.py", line 380, in main
    return AppGroup.main(self, *args, **kwargs)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/click/decorators.py", line 64, in new_func
    return ctx.invoke(f, obj, *args[1:], **kwargs)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/flask/cli.py", line 438, in run_command
    use_debugger=debugger, threaded=with_threads)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/werkzeug/serving.py", line 737, in run_simple
    reloader_type)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/werkzeug/_reloader.py", line 265, in run_with_reloader
    sys.exit(reloader.restart_with_reloader())
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/werkzeug/_reloader.py", line 124, in restart_with_reloader
    close_fds=False)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/gevent/subprocess.py", line 234, in call
    with Popen(*popenargs, **kwargs) as p:
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/gevent/subprocess.py", line 554, in __init__
    restore_signals, start_new_session)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/gevent/subprocess.py", line 1160, in _execute_child
    self.pid = fork_and_watch(self._on_child, self._loop, True, fork)
  File "/home/bpaterni/usr/local/src/python/venvs/venv-01/lib/python3.5/site-packages/gevent/os.py", line 375, in fork_and_watch
    watcher = loop.child(pid, ref=ref)
  File "gevent.libev.corecext.pyx", line 518, in gevent.libev.corecext.loop.child (src/gevent/libev/gevent.corecext.c:7601)
  File "gevent.libev.corecext.pyx", line 1886, in gevent.libev.corecext.child.__init__ (src/gevent/libev/gevent.corecext.c:21676)
TypeError: child watchers are only available on the default loop

Environment

Python version: 3.5.4
Flask version: 0.12.2
Werkzeug version: 0.12.2
grequests: 0.3.0
gevent version: 1.2.2
Click version: 6.7

grequests monkeypatches around all kinds of system modules in python,
as you can see the exception happens because werkzeug uses the working os module of python itself and in the traceback there is the "broken" one from gevent
im wondering if importing grequests before flask and werkzeug can elevate the issue (as then the monkeypatches would happen before werkzeug imports the modules
please try and report back

Rats! Unfortunately, importing grequests before flask did nothing as the same traceback is given on reload.
import grequests
import sys

import flask
...

Downgrade to gevent 1.0.2. Ran across the advice here and it worked. https://medium.com/@kelsey88/typeerror-child-watchers-are-only-available-on-the-default-loop-36775fcbcba6

I'm assuming you're using python2, as gevent 1.0.2 does not seem compatible with my python 3 app? Installing via pip results in SyntaxErrors thrown on gevent's python2-based exception handling. I also tried downloading the gevent souce, but the build step fails due to a file not found error in compilation of gevent/gevent.core.c


Greasemonkey adaption

Tried to use the script with Greasemonkey 3.4.1 with Firefox 40 which did not work. These changes make it work, at least for me

What about other browsers, such as Chrome (Tampermonkey), and Safari (Greasekit)?
Before I can merge this, I need to know that this patch will work on all three extensions. If not, then I need a PR that has an "in-the-middle" thing, that checks for the existence of FileSaver.js
In fact, I think you can do it relatively safely. Try something like this:
function start_export(){
    //[...]
    API.chatLog("[PYE] Step 1: Check for and load FileSaver.js");
    if (typeof variable === 'undefined'){
        $.getScript("https://media.sq10.net/pye/FileSaver.js").done(function(){
            get_playlist_listing();
        }).fail(function(jqxhr, settings, exception){
            API.chatLog("[PYE] Could not fetch FileSaver.js!", true);
            API.chatLog("[PYE] Exception: "+String(exception), true);
        });
    } else {
        get_playlist_listing();
    }
}

i can confirm the annotation is working on chrome with tampermonkey. but theres no need in chrome, your version does the job in chrome as well. dont have safari and dont wanna install. anyways why not have both, so i also implemented your fallback strategy

Merged, thanks!


Keeping a changelog and versioning

I would like to propose that we start keeping a changelog. This would mean that from now on changes are only added through pull-requests, and every pull-request should add an entry to CHANGELOG.md (typically this addition will be done as a final commit once a pull-request has passed tests and review).
My notes below are based on https://keepachangelog.com/en/1.0.0/, please read it. It is a very succinct.
Why keep a changelog?
As the code keeps evolving it will be important to be able to communicate with each other how the codebase is changing. Having a single point of reference, where changes are grouped by feature additions, bugfixes, breaking changes and maintenance will make it a lot easier (and a lot more fun!) to work with the codebase.
For reference here is xarray's changelog: https://github.com/pydata/xarray/blob/main/doc/whats-new.rst, here is one for a different project I've worked on: https://github.com/cloudsci/cloudmetrics/blob/master/CHANGELOG.md
Connected to this I would also like to introduce versioning, using semantic versioning. I would to tag the commit https://github.com/joeloskarsson/neural-lam/commit/2378ed7eddf8da5bfec6f57c41cadf310d191dee @joeloskarsson as v0.1.0, and create a changelog relative to there. My reasoning for this is that I think that commit was the most recent commit when you shared this repository publicly. We can the start the changelog by including the commit that you made for recent loss-function additions (new feature) and the maintenance @sadamov is doing in #6 by setting up linting.
In that case the CHANGELOG would look something like:
# Changelog

## [Unreleased](https://github.com/joeloskarsson/neural-lam/HEAD)

[Full Changelog](https://github.com/joeloskarsson/neural-lam/compare/v0.1.0...HEAD)

*new features*

- additional loss-functions
  [2378ed7](https://github.com/joeloskarsson/neural-lam/commit/2378ed7eddf8da5bfec6f57c41cadf310d191dee)](https://github.com/joeloskarsson/neural-lam/commit/c14b6b4323e6b56f1f18632b6ca8b0d65c3ce36a) By Joel Oskarsson
  (@joeloskarsson )

*maintenance*

- set up linting with pyflake8, black with pre-commit with github action to run in cicd [\#1](https://github.com/joeloskarsson/neural-lam/pull/6/) by Simon Adamov (@sadamov)

## [v0.1.0](https://github.com/joeloskarsson/neural-lam/HEAD)

First public release of Neural-LAM, including functionality to train hierarchical and ...

Let me know your thoughts. I would be happy to set this up if you are happy with it. But I just wanted to get the discussion going before the codebase starts changing a lot

I think this all sounds good. My initial reaction was "Why do we need a changelog if we have a neat git history", but after reading through the links I agree that it would be useful. Perhaps one should also use github releases and keep those synced to the changelog? (Can that be done automatically?)
I think I'd prefer to not have names associated with the items in the changelog. You can always see who is responsible in the git links, if you want to contact someone responsible. So writing the names seems like just extra admin + gives the idea that a contribution is the result of only one person, which is often not the case. Let me know if there is some good reason to have this that I don't think about.
Semver is good. I agree that starting from https://github.com/joeloskarsson/neural-lam/commit/2378ed7eddf8da5bfec6f57c41cadf310d191dee is best, that is also the commit that I tagged for reproducing our workshop paper (ccai_paper_2023-branch, but that one is probably not needed with proper versioning). A question regarding versioning: How often should one release a new version then? Almost every commit could be a new minor- or patch-version, but that is not very useful. I guess you just collect a reasonable amount of new things, but are there some guidelines around this that we want to follow?

Perhaps one should also use github releases and keep those synced to the changelog? (Can that be done automatically?)

Yes, github releases are a good idea I think. In a way it is a second step after tagging that can be used to indicate that this tagged version is the one to use in downstream applications going forward. I have often set it up so a github release automatically gets uploaded to pypi.org. I could do that for neural-lam too. That would require the code to be refactored into a package though, so maybe we should wait with that until we've discussed refactoring.
Regarding automatically creating changelogs: In principle that is possible, but I think it defeats the point of a changelog. The point of a changelog is that it is "for humans". As in, the purpose is for the person who made a code modification to reflect on how this might affect other people working with the code. And through this formulate a single sentence on what the change is for.

I think I'd prefer to not have names associated with the items in the changelog. You can always see who is responsible in the git links, if you want to contact someone responsible. So writing the names seems like just extra admin + gives the idea that a contribution is the result of only one person, which is often not the case. Let me know if there is some good reason to have this that I don't think about.

I don't feel strongly about this, but I wouldn't underestimate the sense of achievement it gives to be able to add yourself to a list of contributors to a project. I think that is the reason why xarray do it (https://github.com/pydata/xarray/blob/main/doc/whats-new.rst). We can make it optional if you like, but I think it is nice for people to feel like they have joined something. Yes, true, this is visible in the git log too, but so are the code changes. Again, I think this about the human element of code development. I don't think optionally adding a few tens of characters is this much work compared to actually working through a pull-request :)

I agree that starting from 2378ed7 is best, that is also the commit that I tagged for reproducing our workshop paper

Ok, great! Will you tag that one as v0.1.0 in that case?

A question regarding versioning: How often should one release a new version then? Almost every commit could be a new minor- or patch-version, but that is not very useful. I guess you just collect a reasonable amount of new things, but are there some guidelines around this that we want to follow?

I think the point here is not that semver defines when we should make versions, but rather communicates what changes between different versions. So we should decide on how often we'd like to make a new version. I think a nice model could be to briefly discuss code changes after each monthly meetup. If we'd like everyone in the community and newcomers who stumble across the code to update to the most recent tagged version then we tag a new version. I would prefer to version often based on when we as a group feel like we've made progress.
That was a quite a lot :)

Github releases
Automatic releases to PyPI like that sounds like a good idea, once we have things packaged as a proper package. My point with automation here was if it would be possible to automatically include the changelog entries in the github release description. I agree that the point of the changelog is to write it manually. But I realize that it should be very simple to just copy-paste whatever is in the changelog to the github release, so not very important.
Names in the changelog
I think what you bring up is a fair point. We can start out with names, but let it be optional if people want to add their name in the future. In particular, if we have PRs that are the result of work of many people we can then either put multiple names or none.
Tag 0.1.0
Yes, I'll tag that one.
When to do releases
I think that sounds like a good model for when to create releases. Likely not something that is important to have any hard rules about, but if people in the community feel that a new release would be suitable then we should release one.

Great, can you set up a PR with the changelog file then? Adding in an entry about the pre-commit hooks and resulting reformatting.

I added the tag: https://github.com/joeloskarsson/neural-lam/tree/v0.1.0

Merged in #28 , closing


Dependabot/pip/sqlparse 0.5.0 (#308)

Build(deps): bump sqlparse from 0.4.4 to 0.5.0

Bumps sqlparse from 0.4.4 to 0.5.0.

Changelog
Commits

updated-dependencies:

dependency-name: sqlparse dependency-type: direct:production ...

Regenerating requirements.txt file

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

Project coverage is 36.53%. Comparing base (242f60b) to head (0584b1e).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #309   +/-   ##
=======================================
  Coverage   36.53%   36.53%
=======================================
  Files         104      104
  Lines        4289     4289
  Branches      547      547
=======================================
  Hits         1567     1567
  Misses       2722     2722

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


PDF not found

Hi, would you be kind to upload the pdf file as well for a reference? Since the link is no longer available.

Just uploaded the reference paper


Updates between validation rule, changelog

In this PR:

Updates documentation for between validation rule. Adds an addition example.
Updates changelog

Zeit must cache the npm repo or something cause super-recently published deps often 404

@justin-schroeder if you add the —force flag to a deploy it’ll rebuild without the cache. Not sure how to handle that with a linked github repo reply. We could unlink the repo and use github actions for a bit more control.


[TASK] Filters do not work for Multiselect

Estimated story points: 3
Description

It is currently not possible to filter multiselect fields. That should be fixed.

Acceptance criteria

Multiselect fields, like documents on the contract schema page should be filterable.

time: 5h


Create a tag with latest additions

Currently, the latest tag is 0.2.2, so if I want to benefit from the latest development, I can't unless I manually import it (I guess?)
Please tell me if there is a better way to benefit from latest development

@antoninadert sorry for the delay, I will provide a new version soon


Add NotRequired things to the DataKey schema

As per https://github.com/bluesky/ophyd-async/pull/137#discussion_r1624442103
Also removing the Optional from "units" and "precision"
https://github.com/bluesky/event-model/blob/fc6e30eae896073eda4013f7c4af3362fead61c1/event_model/documents/event_descriptor.py#L59

Also choices


Allow configuration to disable datafeed persistence for v1

Feature Request
Allow configuration to disable datafeed persistence for v1
Description of Problem:
Some sites don't handle the reuse of datafeed properly and it can cause the bot to be non-functional.

Counter argument for it (my personal point of view):
As it is for a very specific use case and not the behavior we want to promote I would prefer not to pollute the code base and configuration's format with a specific toggle. IMHO this can be worked around by deleting the datafeed id file prior to starting the bot.
Also we would prefer users to move to DFv2.

@symphony-youri  @symphony-thibault
My personal view is if we are doing things on behalf of the developer it should be configurable as what may work for one may not work for them. I am all for configurability and customization but, understand that from a maintenance standpoint it can get quite hectic. Let me check with the stakeholder that raised this issue.

Feedback from the customer:
We would prefer to resolve this issue without needing custom code. Is the team still considering adding the config to turn off reusing the old datafeed ID? If not, at the very least the error should be handled gracefully and a new datafeed should be created and read from. Currently, if reading from the old datafeed it returns a 498 and the bot shuts down. Proper error handling should be that a new datafeed is created instead.

That's odd, 498 is not an official http status (498 Invalid Token (Esri) Returned by ArcGIS for Server. Code 498 indicates an expired or otherwise invalid token.).
The way it was done is that indeed if the DF cannot be used it should be recreated. When this happens, do they get an error message along with the 498?

Does it retry and create a new datafeed as long as its not 204 or 200 or only when it gets a 400 (Datafeed does not exist)? It might be the case where they have something in front of the agent which returned a 489 and instead of recreating the datafeed since its not a 400 it just keeps retrying the read.

Currently, if reading from the old datafeed it returns a 498 and the bot shuts down

This is definitely not a normal behaviour as Agent should never return 498 http status. Currently, the DatafeedLoopV1 recreates new datafeed on 400 errors, as defined by our API contract.

We would prefer to resolve this issue without needing custom code.

I completely understand customer POV, but we need here to keep in mind that DFv1 is now deprecated, and we therefore don't want to bring those kind of custom requirements in the BDK2.0 codebase.
I still believe that this example is quite simple to implement, versus modifying BDK2.0 codebase.

Closing this issue since DFv1 is scheduled for deprecation and DFv2 mechanics are different


Remove collapsible TOC on nav doc

Didn't release this was implemented on the page for testing -- removing it.
Docs update only.

In fact I thought the collapsible TOC looked OK at the top - it wasn't only for testing - and the only way of showing how a collapsing TOC looks on that page is to use it on the existing TOC. I left it open so it shows up without clicking.
Anyway, if you don't want to have it collapsible there for some reason, the following line needs to be removed from the end of the file:
The result is shown at the top of this page ({:toc} can be used only once on each page).

although the comment about only being able to use it once per page might be helpful.

@pmarsceill perhaps it would also be helpful to add a folder in docs for navigation tests? I'm about to start looking at recursive-navigation again, and testing will need a deeper hierarchy than the existing documentation pages.
Apart from systematically testing the various possibilities, it might also provide a more direct way of visualising different examples than the current figures.

In fact I thought the collapsible TOC looked OK at the top - it wasn't only for testing - and the only way of showing how a collapsing TOC looks on that page is to use it on the existing TOC. I left it open so it shows up without clicking.
Anyway, if you don't want to have it collapsible there for some reason, the following line needs to be removed from the end of the file:
The result is shown at the top of this page ({:toc} can be used only once on each page).

although the comment about only being able to use it once per page might be helpful.

@pmarsceill I was surprised to see 0.3.1 released without the above correction. Would you like me to fix it myself? and if so, should I reinstate the example of the collapsible toc, or remove the cited line?


[Q] Question about permissions / changing user

Hello, and thanks for making such a great docker image. It works wonderful.
Is there any to change the user from root to something else?
In some docker images you can set it using environment tags, like
-e PUID=1000 -e PGID=1000 and -e UMASK=002
But it doesn't seem to work on this image. So is there any way to change it another user, instead of root? Honestly don't like having everything "locked" behind sudo or using my root user to edit files

It's full of quirks to try and use non-root users.  Look at these:
Add a non-root user to a container
Many Docker images use root as the default user, but there are cases where you may prefer to use a non-root user instead. If you do so, there are some quirks with local filesystem (bind) mounts that you should know about. Specifically:

Docker Desktop for Mac: Inside the container, any mounted files/folders will act as if they are owned by the container user you specify. Locally, all filesystem operations will use the permissions of your local user instead.

Docker Desktop for Windows: Inside the container, any mounted files/folders will appear as if they are owned by root but the user you specify will still be able to read/write them and all files will be executable. Locally, all filesystem operations will use the permissions of your local user instead. This is because there is fundamentally no way to directly map Windows-style file permissions to Linux.

Docker CE/EE on Linux: Inside the container, any mounted files/folders will have the exact same permissions as outside the container - including the owner user ID (UID) and group ID (GID). Because of this, your container user will either need to have the same UID or be in a group with the same GID. The actual name of the user / group does not matter. The first user on a machine typically gets a UID of 1000, so most containers use this as the ID of the user to try to avoid this problem.

That's a lot of nasty, nasty quirks.  Now what is the benefit?  The Bedrock server has never been breached.  It doesn't matter if they were running the server as root with maximum permissions to everything.  It's never happened.  To anyone.
So what would happen if you were the first person in history to get breached on the Bedrock server?  The most badass hacker in the world has targeted you and has stack smashed the C++ executable.  Now what?  Also nothing.  The container isn't real.  It only exists in memory.  If they obtained root access to it they would be able to do exactly nothing.  Any changes they make in the container would be 100% lost the second it restarts.
The only thing they could do is change files in your Docker volume.  That's the only thing that persists.  They'd have to be among the smartest hackers ever to even realize this and figure out which endpoint mapping is real.  Again, literally has never happened to anyone ever.
In fact what your proposing is actually a security issue.  You want to be able to edit the Docker volume contents with a regular user.  Now what would happen if that regular user got hacked and it had permissions on Linux (because it needs to, that's a quirk I just posted above)?  Well because of the storage quirks and having to give that person actual real permissions on the host machine you are actually at far greater risk:
Inside the container, any mounted files/folders will have the exact same permissions as outside the container - including the owner user ID (UID) and group ID (GID).
Whoa there, not a chance.  That literally defeats the purpose of using Docker.  It's the worst of the standalone version AND the Docker version put together.  You should use my standalone script to be able to do this securely using a user you have set up only to run the Minecraft server that has no sudo rights or access/rights to anything else.  That version if the script is available here:  https://github.com/TheRemote/MinecraftBedrockServer
That one works exactly how you are wanting to.  That's because it's not a container.  Containers don't need a non-root user and there are serious implications for doing this that vary depending on the platform (especially with file permissions).  I guarantee that none of the people using this understand the implications of it unless they have read the Docker documentation like I pasted above.
Have you added your host computer's user to your Docker group?  You shouldn't need root permissions to do hardly anything with it.  sudo adduser james docker followed by a reboot will remove sudo as a requirement from doing any of the Docker commands.  This is how you want to set up Docker.  You may need root permissions to edit the Docker volume contents still.  That is for security purposes and won't be changed.

Thanks for the thorough answer!

No problem at all, take care, let me know if I can help in any way!


KNL-1503 lang.StringUtils -> lang3.StringUtils

import org.apache.commons.lang.StringUtils; -> import org.apache.commons.lang3.StringUtils;

This needs an associated JIRA.

https://jira.sakaiproject.org/browse/KNL-1503

FYI for future we need the JIRA # on the commit message as well. Just changing the title isn't enough.


Change log format to Json and add workflow.runID and workflow.Id to logs

Is your feature request related to a problem? Please describe.
We have 100+ teams using the actions runner controller to provision runners and run pipelines.  Occasionally a runner will not start or start very slowly.  We have found that trying to identify issues and analyse the logs is very time-consuming.
Describe the solution you'd like
We use ELK to search,  analyse and visualise our logs, changing the format to JSON will allow the logs to be indexed with specific fields which we can then filter, making it easier to analyse.  If we could also add the workflow.runID and workflow.ID, this would allow us to track specific jobs and workflows.
An example of how this would look in the logs
{"severity":"info","ts":1662068399.1369355,"logger":"controllers.webhookbasedautoscaler","message":"job scale up target is repository-wide runners","event":"workflow_job","hookID":"374760745","delivery":"3f7412e0-2
85b-11ed-8a67-3440ca5a40cb","workflowJob.status":"queued","workflowJob.labels":["self-hosted","lab-ie-core","build"],"repository.name":"dh-bosun-test","repository.owner.login":"sainsburys-tech","repository.owner.ty
pe":"Organization","enterprise.slug":"j-sainsbury-plc","action":"queued","workflowJob.runID":"2956053697","workflowJob.ID":"8091421879","repository":"dh-bosun-test"}
{"severity":"info","ts":1662068399.1369936,"logger":"controllers.webhookbasedautoscaler","message":"scaled hello-bosun-df12a by 1","event":"workflow_job","hookID":"374760745","delivery":"3f7412e0-285b-11ed-8a67-344
0ca5a40cb","workflowJob.status":"queued","workflowJob.labels":["self-hosted","lab-ie-core","build"],"repository.name":"dh-bosun-test","repository.owner.login":"sainsburys-tech","repository.owner.type":"Organization
","enterprise.slug":"j-sainsbury-plc","action":"queued","workflowJob.runID":"2956053697","workflowJob.ID":"8091421879"}
{"severity":"info","ts":1662068399.1370363,"logger":"controllers.webhookbasedautoscaler","message":"Starting batch worker","event":"workflow_job","hookID":"374760745","delivery":"3f7412e0-285b-11ed-8a67-3440ca5a40c
b","workflowJob.status":"queued","workflowJob.labels":["self-hosted","lab-ie-core","build"],"repository.name":"dh-bosun-test","repository.owner.login":"sainsburys-tech","repository.owner.type":"Organization","enter
prise.slug":"j-sainsbury-plc","action":"queued","workflowJob.runID":"2956053697","workflowJob.ID":"8091421879"}

and in Kibana....

We've merged 1771 and this is going to be released as a part of the next ARC v0.27.0.
Thanks for your contribution @malachiobadeyi!


Applying selected middlewares to all routes feature implemented.

Middlewares like \Illuminate\Routing\Middleware\SubstituteBindings::class, are used for all routes. With current version of route attributes you cannot apply a middleware to all routes.
I added a field to route-attributes.php named middlewares, with this field you can add middlewares that will be applied to all routes.
This also fixes #13 , because SubstituteBindings middleware is comes default in config file.

Great PR, thank you very much!


Add an ability to tag text entries with a relative date

I find it useful to tag things that need my attention with a certain date, so I can ensure that I processed the note properly. It'd be great if you could do that as one of the template substitutions.
One way to do it would be to use chrono-node to resolve natural dates (as roam-toolkit and phonetoroam do).
But I think also just having a few basic things like {today}, {tomorrow} can be a simple 80/20 solution

Open to this idea! Don't have the bandwidth to implement this myself for this extension but am accepting PRs


Add ISSUE_TEMPLATE.md

Since we are recently observing more unreadable issues and/or feature requests to be for plugins, let me set the ISSUE_TEMPLATE for filing new issues.
@sakama @kamatama41 Can you have a look? (Cc: @hiroyuki-sato)

LGTM 👍

@kamatama41 I'll be merging this soon. Please ping me if you see any concerns.

Thanks!


[BUG]:The gmail in the footer is not hovering for any page

Describe the bug
The email-id in the footer section is not hovering for any page
Expected behavior
The email-id should hover to create a better UI experience for the user.
Screenshots
https://github.com/user-attachments/assets/b540899e-5d66-483c-98fa-07b94ac63df5
Additional context
Please assign this issue to me under level1.

it doesnt need to hover , its fine as it is


Text and minor visual fixes

We don't have to close this super quick, if someone notices any visual or text/translation bug and wants it fixed please add a comment below.


dts: bindings: Add memory-region property

Since the introduction of the zephyr,memory-region compatible to create new
memory region, drivers can now use the newly defined region in the DT by using
the macro LINKER_DT_NODE_REGION_NAME() on the node_id to retrieve the region
name and possibly allocated data or variables in that region (or for whatever
other use). This is assuming though that the driver knows in advance the node_id to
reference.
A better approach is to add a new generic 'memory-region' property that can be
used by any driver to reference a region by using a phandle, so that the driver
can use that to retrieve the memory-region of interest.
Signed-off-by: Adam Wojasinski<EMAIL_ADDRESS>Signed-off-by: Carlo Caione<EMAIL_ADDRESS>

cc @carlocaione

I can see the use, but it seems very specific to certain drivers. Why not leave it to specific bindings to add the property?
For example, lots of devices have a reset pin, but we haven't added reset-gpis to base.yml.

I can see the use, but it seems very specific to certain drivers. Why not leave it to specific bindings to add the property?

yeah, that is the way it was done in the original PR #45128 but IMO that would bring a lot of redundant code and copy&paste. What about adding a new yaml like memory-user-device.yaml to be imported?

@mbolivar-nordic @JordanYates any feedback on this?

I agree with @galak that base.yaml isn't the best spot for this.
Some drivers might want multiple memory-region's, most want none.
Maybe a easydma.yaml that contains the common property and the drivers just include that?
The scope in the original PR was only nordic drivers, if we end up with multiple manufacturers with similar files it will be easier to work out what a common memory-region.yaml needs to look like.

@anangl any feedback on this?
@galak do you have any objections to the current implementation?

@mbolivar-nordic this PR doesn't have a very high priority. It can wait until v3.2. I just wanted to ping people to align code to their comments and have the coding part done.

Can you also open an enhancement request to transition to use of memory-region property.

@galak is that ok for you? https://github.com/zephyrproject-rtos/zephyr/issues/46048

Looks good, but nit pick, can you add the memory-region-names property as well from https://git.kernel.org/pub/scm/linux/kernel/git/devicetree/devicetree-rebasing.git/tree/Bindings/reserved-memory/memory-region.yaml

@galak now I noticed that in Linux memory-region is a phandle array type. I don't see any particular reason to change phandle to phandle-array in this PR. What are your thoughts about that?

force-pushed some changes to fix build errors regarding phandle-array property type


Doc request: unsigned integers native types for MySQL

UnsignedSmallInt
UnsignedTinyInt
UnsignedInt
UnsignedMediumInt
UnsignedBigInt

In the docs there is one mention about @db.UnsignedBigInt when introspecting
https://www.prisma.io/docs/concepts/database-connectors/mysql#native-type-mappings
But the native types seems not yet documented.
Related https://github.com/prisma/prisma/issues/3239

Note, maybe check https://github.com/prisma/prisma-engines/tree/master/libs/native-types/src for the exhaustive list of supported types ✨


Jersey client not compatible with Spring Boot 3

The jersey client doesn't work with Spring Boot 3 (Java 17) because it still uses the javax namespace. Any plans to switch to jakarta?
Got a java.lang.ClassNotFoundException: javax.ws.rs.core.Configuration error.

I gave it a try on the issue-1 branch.
You can use Jitpack to get a build.
Gradle Example:
repositories {
    maven {
        url "https://jitpack.io"
        content {
            includeGroup "com.github.unblu.openapi"
        }
    }
    mavenCentral()
}

dependencies {
    // ...
    implementation 'com.github.unblu.openapi:jersey-client-v3:issue-1-SNAPSHOT'
    // ...
}

Maven Example:
<repositories>
  <repository>
    <id>jitpack.io</id>
    <url>https://jitpack.io</url>
  </repository>
</repositories>

<dependencies>
  <dependency>
    <groupId>com.github.unblu.openapi</groupId>
    <artifactId>jersey-client-v3</artifactId>
    <version>issue-1-SNAPSHOT</version>
  </dependency>
  <!-- ... -->
</dependencies>

JBang Example:
This works also out of the box if you are running some script with Jbang by declaring the dependency like this:
//DEPS https://github.com/unblu/openapi/tree/issue-1#jersey-client-v3:SNAPSHOT

Instead of:
//DEPS com.unblu.openapi:jersey-client-v3:7.32.0

And then Jbang does the work of adding the additional repository for you.

I've tried it on demo project for now, but looks like it works.

Starting with next release (en of July) we will publish 2 libraries on maven central:

com.unblu.openapi:jersey-client-v3:<version> as it is now based on Jersey 2
com.unblu.openapi:jersey3-client-v3:<version> as it is now based on Jersey 3

In order for the consumer to have some time to adjust.
When we introduced v4 of the Web-API (probably with Unblu 8) we will re-consider and have probably only one client based on Jersey 3.


Pad currency outputs for right alignment

Right aligning the dollar amounts would make it easier to compare the output between different line items at a glance.
Current output
❯ aws-cct --start 2020-08-01 --end 2020-09-01 --service "Amazon Relational Database Service"

+-----------------------------+------------+------------+-----------+
| SERVICE                     | 2020-08-01 | 2020-09-01 | DELTA     |
+-----------------------------+------------+------------+-----------+
| RDS:Multi-AZ-GP2-Storage    | $10,364.42 | $10,620.27 | $255.84   |
| USE1-EU-AWS-In-Bytes        | $0.00      | $0.00      | $0.00     |
| Multi-AZUsage:db.t2.medium  | $92.08     | $53.74     | -$38.34   |
| RDS:GP2-Storage             | $5,793.94  | $5,795.74  | $1.79     |
| RDS:Multi-AZ-StorageUsage   | $17.73     | $17.66     | -$0.07    |
| InstanceUsage:db.t2.small   | $4.84      | $22.68     | $17.84    |

Proposed output
❯ aws-cct --start 2020-08-01 --end 2020-09-01 --service "Amazon Relational Database Service"

+-----------------------------+------------+------------+-----------+
| SERVICE                     | 2020-08-01 | 2020-09-01 | DELTA     |
+-----------------------------+------------+------------+-----------+
| RDS:Multi-AZ-GP2-Storage    | $10,364.42 | $10,620.27 |   $255.84 |
| USE1-EU-AWS-In-Bytes        |      $0.00 |      $0.00 |     $0.00 |
| Multi-AZUsage:db.t2.medium  |     $92.08 |     $53.74 |   -$38.34 |
| RDS:GP2-Storage             |  $5,793.94 |  $5,795.74 |     $1.79 |
| RDS:Multi-AZ-StorageUsage   |     $17.73 |     $17.66 |    -$0.07 |
| InstanceUsage:db.t2.small   |      $4.84 |     $22.68 |    $17.84 |

From an initial look, it seems like this can be achieved with specifying ColumnConfigs and setting the text.Align there.

This is now in v1.0.1 and is updated on Homebrew


feat(trading): add additional intervals

Related issues 🔗
Closes #5089
Description ℹ
Add W, 12h, 8h, 4h and 30m intervals
Demo 📺

@bglownia Should this say 1D and 1W?


Manage component dependency

We want to create isolated components that include and call relevant dependencies without duplicating the imported code.
Possible approaches:

use a sass mixin to check: http://paul.wellnerbou.de/2015/05/18/avoid-multiple-imports-of-the-same-scss-file-with-sass/ or https://github.com/carbon-design-system/carbon-components/blob/master/src/globals/scss/_import-once.scss

Use an NPM module:
https://www.npmjs.com/package/node-sass-import-once

Either way it might become redundant as of Sass 4.0: https://github.com/sass/sass/issues/156

Closing as we've decided to do complicated stuff on our end and not rely on devs to setup npm their end.
See https://github.com/alphagov/govuk-frontend/pull/18


Conductor: add v1alpha2 rpc functionality to conductor execution client

Summary
Non destructively adds v1alpha2 rpc functionality to Conductor's ExecutionRpcClient.
Background
This PR was created in preparation of Conductor's switch to the new Execution API v1alpha2.
Changes

adds v1alpha2 function definitions to ExecutionClient trait
adds v1alpha2 function implementations to ExecutionRpcClient struct

Testing
No tests yet as there is no implementation done here.
Related Issues
https://github.com/astriaorg/astria/issues/205
closes https://github.com/astriaorg/astria/issues/205

I don't think the service should support both APIs at the same time using two different clients.

It won't. I'm working on another branch that captures the work fr #207 that will merge into this one. It will remove the alpha1 api. This PR only capturing the work for #205 not #207


maintainers: Add jerrysm64 to maintainer-list

Description of changes
Added myself to the maintainer-list.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

You can cherry-pick the commit right into the PR where you use it :)

Closed, because I got it merged in #259650. Thanks for your time, though!


Revamp Wiki

Moved here from https://github.com/HTTP-APIs/hydrus/issues/72

https://github.com/HTTP-APIs/hydra-ecosystem-wiki/commit/e1c59128465a3d04a2681ff4b3af476cbcb6bdcd

https://github.com/HTTP-APIs/hydra-ecosystem-wiki/commit/4363c71b14fe6217748a7607cbbf6f63023dce37

https://github.com/HTTP-APIs/hydra-ecosystem-wiki/commit/4e22a82adaeb52f8c47ebcc23ce9157f7b0effcc

https://github.com/HTTP-APIs/hydra-ecosystem-wiki/commit/61fa9d63d4f882b999deb24ca37c43da6c190bf0


Import C genes into immunarch with repLoad

🚀 Feature

Immunarch's repLoad function currently creates a data frame with V.name, D.name and J.name, but C.name (the name of the constant region gene) is missing.
This is currently the case both for data from MiXCR's clone tables and from Cell Ranger's filtered_contig_annotation files, even though the information for the C gene is present in both of them.
Motivation
Having the name of the constant region gene is important when doing analysis related to isotypes and would help visualize isotype distribution in a sample.
Pitch
The repLoad function should be able to import C gene names and display it as a column in the data frame next to the J.name column.

C gene name information is already available in Cell Ranger's output (filtered_contig_annotations):

and MiXCR's clone tables (highlighted):

Hi Khang, we are working on it! We need a support for C segments for BCR analysis, so it's in progress. @Alexander230 take a look please

Dear Vadimnazarov,
I am also working on BCR analysis. Is there any progress on adding C gene into analysis? Thank you very much!
Yanran

Hello @Khang-LQ and @useryanran,
Thank you for contacting us. My name is Aleksandr Popov, I am a developer of the Immunarch package.
I'm glad to inform you that Immunarch starting from version 0.6.7 supports importing C genes with repLoad from MiXCR format. You are welcome to use it!
Best regards,
Aleksandr

Hi @Alexander230,
I have updated Immunarch to 0.6.7 and tried loading a MiXCR clone table with repLoad. However, I still cannot see the column for C gene call in the Immunarch data frame. Is there a specific command that I need to write to enable this?

Hi, @Khang-LQ,
Could you, please, provide an example of data that you want to load? Maybe, C genes column is named differently, so it's not detected correctly in repLoad. The example will help me to make a fix.
Best regards,
Aleksandr

Hi @Alexander230!
I am having the same problem. I have assembled full Ig receptor sequences using using mixcr exportClones command and the output tables all contain a column named allCAlignments. I have updated Immunarch to the latest version, but Ig isotype is still not detected. In case you are interested, here is an example of my data.

Hi, @plezar!
Thank you for sending the example data! C gene names are successfully parsed from them into C.name column, and C alignment coordinates are currently not supported in release version of Immunarch. Unfortunately, allCAlignments column is empty in your example data. I've added experimental support for parsing this column, with assumption that it has the same format as D gene alignments. You can try to use Immunarch version from the experimental branch; to install it, use these commands:
install.packages(c("devtools", "pkgload"))
devtools::install_github("immunomind/immunarch", ref="feature/mixcr-c-alignments")
devtools::reload(pkgload::inst("immunarch"))

Best regards,
Aleksandr

Hi @Alexander230 ,
Thank you for your help, now I am able to import C gene name from MiXCR data. I'm looking forward to importing C gene data from 10X CellRanger data as well.

I've planned this feature for one of the next Immunarch releases. It will appear earlier on dev branch, so I will notify you when it will be ready.
Best regards,
Aleksandr

Hi, @Khang-LQ!
I've implemented an experimental branch with support for C genes in 10x format. You can try it by installing immunarch from experimental branch with these commands:
install.packages(c("devtools", "pkgload"))
devtools::install_github("immunomind/immunarch", ref="10x-c-gene")
devtools::reload(pkgload::inst("immunarch"))

When the fix will be finalized, it will be moved to dev branch; after that, to install it, use ref="dev" instead of ref="10x-c-gene" in these commands.
Best regards,
Aleksandr

Hi @Alexander230. I have just tried out the C gene feature for 10X format and it is working very well. Thank you so much for the support.

I'm glad to hear that the C genes support is working! I've merged it into dev branch, and it will be included in Immunarch 0.6.9, which will be released soon. If you have more questions, feel free to reopen the issue and ask them!
Best regards,
Aleksandr


Can't get DAP to connect to channel

Trying to setup DAP for the first time. I run DAP and select the server and channel, the bot comes online, but does not connect.
Running Windows with the .exe (dap-2.2-win64.exe)
Error log:

2021-12-13 16:35:53 Error on change_channel
Traceback (most recent call last):
File "gui.py", line 178, in change_channel
File "discord\abc.py", line 1154, in connect
File "discord\voice_client.py", line 345, in connect
File "discord\utils.py", line 350, in sane_wait_for
concurrent.futures._base.TimeoutError

Maybe a permissions issue, if your channel has custom roles you'll need to check if the bot has the role to the join the channel you want.

OK, thanks. That seems to have fixed it. Is it possible that information can be included in an error message?

Yeah, thanks for posting the log, I'll add additional context for TimeoutError in a next release.


Add passing PKCS11 connection string via shared file

What problem was this feature supposed to solve?
If you still implement it, keep it so that the environment variable from the connection string takes precedence over the file, so that the PKCS11 library can be controlled per process.
For any outputs use log_message instead of printf and do not change the default logging levels. Unexpected output to STDOUT causes problems for applications using native messaging.

Hi,
thanks for your email.
I apologize for the PR - it was mistakenly created against your repo, I've
already removed it.
According to the problem, this feature tries to solve - I need to run the
BouncyHsm as a docker image. It works fine when the connection string is
set before the client app (custom app written in .NET) starts.
But I also need to run this docker image from tests using the
Testcontainers library. In this case, the connection string is not known
when the tests start, but dynamically after running the container with
Testcontainers.
Passing the connection string from the running tests by
Environment.SetEnvironmentVariables does not work for that case. That's why
I am trying to find a workaround to pass the config via a file.
Regards,
Zdenek
pá 23. 8. 2024 v 13:45 odesílatel Jozef Gajdoš @.***>
napsal:

What problem was this feature supposed to solve?
If you still implement it, keep it so that the environment variable from
the connection string takes precedence over the file, so that the PKCS11
library can be controlled per process.
For any outputs use log_message instead of printf and do not change the
default logging levels. Unexpected output to STDOUT causes problems for
applications using native messaging.
—
Reply to this email directly, view it on GitHub
https://github.com/harrison314/BouncyHsm/pull/7#issuecomment-2306921202,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB22U44RMHKQRZVKSVEQGRDZS4OF5AVCNFSM6AAAAABM7QZLXCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBWHEZDCMRQGI
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


1.修复字段数超过256时（以及一些其他情况）， propertyNamingStrategy无效的问题  2.修复 @JSONType 中asm字段会影响全局的问题。

1.修复字段数超过256时（以及一些其他情况）， propertyNamingStrategy无效的问题
2.修复 @JSONType 中asm字段会影响全局的问题。

Codecov Report

Merging #1568 into master will increase coverage by <.01%.
The diff coverage is 80%.

@@             Coverage Diff              @@
##             master    #1568      +/-   ##
============================================
+ Coverage      81.5%   81.51%   +<.01%
- Complexity     6053     6054       +1
============================================
  Files           136      136
  Lines         22462    22462
  Branches       5144     5144
============================================
+ Hits          18308    18309       +1
+ Misses         2888     2887       -1
  Partials       1266     1266

Impacted Files
Coverage Δ
Complexity Δ

...m/alibaba/fastjson/serializer/SerializeConfig.java
80.85% <100%> (ø)
120 <0> (ø)
:arrow_down:

...baba/fastjson/serializer/ASMSerializerFactory.java
97.12% <75%> (+0.14%)
200 <0> (+1)
:arrow_up:

...com/alibaba/fastjson/parser/JSONReaderScanner.java
72.61% <0%> (-1.28%)
44% <0%> (-1%)

...ava/com/alibaba/fastjson/parser/JSONLexerBase.java
71.48% <0%> (+0.03%)
784% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


WIP - Debug bar: Dark/light mode + New menu for the settings + Complete CSS refactoring

Description
Many devices and applications have a "dark mode" nowadays. I use it on my computer, and I develop applications with this feature.
When I use the current debug bar, it sometimes gives a contrast which is tiring for the eyes. So I thought it might be interesting to add this feature, which ultimately requires only a few lines of code.
Colors are similar to Chrome's console, using dark mode on Mac OS.
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPdocs (N/A)
[x] Unit testing, with >80% coverage (N/A)
[x] User guide updated (N/A but comments added inside the files)
[x] Conforms to style guide

I like the concept, especially as a dark user myself! You might also want to set defaults with a CSS media query, so "unspecified" tries to match the browser/OS setting rather than defaulting to light:
@media (prefers-color-scheme: dark) {

}

Thank you for your feedback ! But suddenly it was a little bit more "complicated" to achieve :-) Indeed...
1 / By default, we use CSS and media queries of course
2 / But we must allow the user to change the mode if he wants to. A "light mode user" may want to use the dark mode, and a "dark mode user" may want to use the light mode.
So here is the result I get.

Just for inspiration: https://github.com/JCSama/CodeIgniter-develbar

https://github.com/JCSama/CodeIgniter-develbar

Thank you, indeed :-) Let's finish this work on dark/light mode, and maybe, if people are interested, we could work on design improvements for the debug bar ;-)

Yeah this is looking great! I think the white icon is fine, it is very small so not too glaring.
Is this ready to go? More changes you wanted to make?

Actually one request: the font color should be specified for dark mode. If it inherits the color it could end up being unreadable. Example: this actual dark-themed CI4 site using your updates.

Thank you for your feedback ! Here is the thing...
I wanted to add the "dark/light" feature, without changing colors. Because, when I worked on the css file, there are many things that came through my mind. There is really place for improvement. And there is @natanfelles 's comment too, talking about future design improvements that could be done.
So, I started to write things down (brainstorming). And I would really appreciate if you could take a few minutes to have a look at this: https://littlej.github.io/ci4-debug-bar-design/build/html/
Then, you tell me what you think :-) I am OK adding colors, but maybe it would be better to agree on the "color scheme" first ?
Thank you again for the time you are giving me !

Please note that there some similar issues right now with the current css.
Simple example:
1/ Load CI default page (fresh install)
2/ Add a style to body with a color, like this: body { color: red; }
And now you get a toolbar with red labels :-D
That is one of the reasons I started doing those suggestions. Because I create apps with CSS. It is all about user experience. And when I am done doing this, I open the console and sometimes find a weird interface :-D And I don't like to have Debugbar styles in the app's css, that what is happening today :-)

I like this idea a lot. And agree there are plenty of places for improvement in the Toolbar.
Are you all happy with the place this is at currently or should we still expect more changes to come soon?

Note: I am willing to work on CSS improvements. Do not hesitate to tell me if you want me to continue.

@LittleJ yes, please continue. :)

Hello ! Sorry for the late answer, busy week... Here is the new version :-)
Your request @lonnieezell
I thought the same about the color, but the class is entitled ".muted". So I was wondering how "muted" it should be :-D I changed colors from dark to light gray (for the dark theme) and light to dark gray (for the light theme). But not everywhere, because ".muted" is used in other locations (like in the "Files" tab). It should be OK now ;-)
Other changes
First, I just wanted to generate an uncompressed version of the CSS file. But there were comments appearing where it shouldn't be. So I changed the commenting style to get a clean version. Now we have standalone "CSS" and "SASS" versions. Anyone is free to use one or the other.
Drawback
The only issue is that if someone (else than me) wants to improve the design (for everyone). Then he has to modify the SASS version in order to generate the CSS version. Meaning, "CORE developers" have to use the SASS version, but not "simple/random CI developers" (they have the choice).
 But I would have liked to know if it works with the dark theme you were using last time. It should, but I want to be sure as I like when things are done right ;-) If you have time, juste copy-paste the new stylesheet, there is nothing to install or compile in order to use it.

@lonnieezell With an uncompressed version of the CSS, there is no problem for the developers. It is the same thing as today. But, what to do with these SASS files (used by core developers/contributors) ? There are 4 solutions I see:
1/ Use Github to generate the CSS version (one of your initial ideas). Which i found not totally adapted as the developers have to see the result before pulling it. So it is not really a "complete" solution.
2/ Ask them to generate the CSS from the SASS version. You have rules for PHP contributions, there could be one for CSS contributions too.
3/ Remove the SASS version. Maybe put it in a separated repo. But the issue will only be moved elsewhere.
4/ Forget about the SASS version. We only keep the final CSS. I just lose part of my work. It happens... too bad for me :-)
Note that if we create a new homepage (other thread), we will have the same CSS issues. The homepage should use the same graphic chart that has been defined... The homepage CSS could even be the same as the debug bar CSS. Meaning there is only one big CSS file for the whole framework, which will just be included in the homepage too (it is possible).
If you don't want complications, the simplest solution is just to keep the CSS version only. But it will probably leave less space and ease for future improvements.
If you want a simple solution, while keeping the SASS version, you make it mandatory for core developers/contributors to use it.
Else, there are solutions, but none is "simple"... :-)

@lonnieezell I was thinking about something like this too, for the settings. What do you think ?

I like that. Go for it.
On Thu, Feb 13, 2020 at 12:30 PM Johan<EMAIL_ADDRESS>wrote:

@lonnieezell https://github.com/lonnieezell I was thinking about
something like this too, for the settings. What do you think ?
[image: Capture d’écran 2020-02-13 à 19]
https://user-images.githubusercontent.com/45337552/74466304-384fbf00-4e97-11ea-935a-b260458ee095.jpg
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeigniter4/CodeIgniter4/pull/2478?email_source=notifications&email_token=AAAMVW3XSSVVU4HXRLU3MIDRCWGT3A5CNFSM4KEOKB4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELWC6NY#issuecomment-585903927,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAMVW2LYAJYDDSMHUGRVJLRCWGT3ANCNFSM4KEOKB4A
.

@LittleJ I forked your branch and tried it on the new CI4 Welcome Page. It looks really good! On question on behavior... it seems to be dark by default and regardless of my OS setting, unless I change it with the toolbar-theme button. Is that intended? I had suggested the prefers-color-scheme: dark media query above - its fine if you decided that was too much but I want to make sure that if that is the intended behavior it isn't a bug.
Re: SASS I definitely think hanging onto the SASS files in the main repo is a boon! What do you and @lonnieezell think about this: move the instructions for generating the CSS files into their own file in the contributing folder and add the SASS files there in a sub-folder as well.

@MGatner I like the idea of moving it into a subfolder. We do have to ensure we update those as part of the release cycle, still, though. Handled through the scripts in /admin. That can always be a different PR, though, as I'd love to see this included before the hopeful release on the 24th.

Hey ! Sorry, I was on a business trip...
I think it is going to be complicated to do all the modifications (dropdown menu) before the 24th. And maybe some other members will want to verify the code and pass into some sort of validator... It could take time. Maybe we could stick to this version for now (dark mode + CSS refactoring), and improve it after (dropdown menu + fullscreen mode).
You decide what is best, and I will do my best to help out :-)
PS: I find it complicated to see the changes you made on Github. So I mainly rely on your comments.

If we can get just the CSS + dark mode refactor that would be awesome. And if that could be reorganized like @MGatner mentioned that would be awesomer. :)

I am a little confused... Do I have to make the changes based on the fork ? Or do I have to confirm a pull request somewhere ? What is the next step ? :-)

By the way, thank you for your answers on the other pull request @lonnieezell. I was clearly not using the right words to express myself. I got confused too, my english is sometimes limited :-)

Nope - no one has submitted PRs against your PR. I believe the only changes have been in comments.
Basically we are just looking for a PR with what you have plus any changes just recently asked for.

@LittleJ We would love to have this in the initial release! Could you (before Monday) (before Sunday night actually) realistically get these tasks done?

Change the light/dark mode default to be dependent on @media prefers-color-scheme: dark
Create a new file, contributing/css.rst
Move instructions from system/Debug/Toolbar/Views/SASS/toolbar.scss to the new doc and add anything else someone might need to know
Create a new folder, admin/css/ and add the SASS files (and anything else needed to generate the CSS)

Yes I think it is possible. I just had to wait for the week-end to have a little bit more time ;-)

I will send the changes later today. And I will be available for fixes/revisions if needed.

That’s great news! Thanks so much - I’m excited this will be part of the launch.

@MGatner I cannot reproduce your bug on my mac. If i clear the cache, I get the right color by default (light), the one that is set in the system's settings. Is it possible that you used the "switch theme" button and the cookie was still alive ? Because the color is stored like the position.

@lonnieezell While we wait for an answer from @MGatner, I moved the instructions. Could you have a look and check if my translations are good ? Thank you !

Okay that's great! No worries about my experience, I could have switched theme earlier like you said. I'm a CSS lightweight so I trust you to know what you're doing.
The docs and SCSS files look good! I'll let Lonnie have one last look at it but we'll plan to merge it tonight barring any further input. Thank you so much, this is a great asset to the framework

No, no, it is my pleasure ! :-) I will do more improvements after this release. It is just a question of time/deadline.

Looks great! Thanks so much. Merging.


text doubles with contenteditable - issue #383

My proof of concept to fix issue #383  (text doubles with contenteditable)

@MichalW I'm having trouble reproducing this issue anymore, so it's tough to see if this fixes it. Do you have a reliable way to reproduce? I'm testing using devel branch.

@aldeed The problem still exists, also on "devel". It is very random - sometimes work, sometimes not.
I will try to find reliable way to reproduce.

I created small project to reproduce problem:
https://github.com/MichalW/meteor-contenteditable-test
I recorded what you need to do to make the problem appear:
https://www.youtube.com/watch?v=rZuwS454fDs&feature=youtu.be

OK, seems to work and fix the issue as far as I can tell. Thanks for the PR and good repro!


Use uncaught_exceptions from Boost.Core to avoid C++17 warnings

In C++17, std::uncaught_exception is deprecated in favor of std::uncaught_exceptions. Use portable uncaught_exceptions implementation from Boost.Core, which is also available for C++03.
This fixes gcc 8 warnings about using deprecated features in C++17 mode.

Yes, I will make this change when I have some time in the next few months (this fixed issue #16).


Invalid CSS after "v": expected 1 selector or at-rule, was "var path = require("       in C:\Users\gwk736\GitLab\Inform\node_modules\node-bourbon\index.js (line 1, column 1)  @ ./src/app/header/header.component.ts 20:21-55

I use it:
@import "~node-bourbon";

I also have this problem, on mac.

+1

+1

It seems like you guys are attempting to import these using a css loader.
~node-bourbon will export the main entry point of this module which is an index.js file containing an includePaths variable

Did anyone find a solid fix for this?


Gem::InstallError: autoprefixer-rails requires Ruby version >= 2.0.

Gem::InstallError: autoprefixer-rails requires Ruby version >= 2.0.
An error occurred while installing autoprefixer-rails (6.3.1), and Bundler
cannot continue.
Make sure that gem install autoprefixer-rails -v '6.3.1' succeeds before
bundling.

Hmm, looks like our readme that says 1.9.3 is okay is out of date. I'm assuming you're running a version of Ruby less than 2.0? You probably already know how to check, but if not — just run ruby --version.

I have installed ruby and bundler but
when I run the command: bundle install
It is giving me this message: Bundler::GemfileNotFound

Yeah, got the same type error on mime-type-data, but just switched to branch 'v2' and things I'm in good shape again on my ruby 1.9.3 install.

:/ Branch v2 is pretty broken for all sorts of reasons, probably will be deleted sometime soon. @flik are you able to update Ruby?

(Upgrading may be a good idea, Ruby 1.9.3 is no longer receiving security updates.)

@lord, I'm on Precise, so I'll have to go to Trusty to get Ruby 2.0.  Need it for some other work so it's on my list anyway.

Ah, gotcha. Well, we're unlikely to support Ruby 1.9.3 in the master branch any time soon, so I'm going to close this and update the readme.


Implement command "go"

Implement partly in commit ae2062bb0b40952c6be20c99a18b5dcc2be159e0,
Can now get available directions, print it to UI and pass directions to "go" command which now understands those.
Actual movement of player isn't implented yet.

Command "go" implemented.


PyZDDE giving different results than Zemax

Hi,
I know the title might sound a bit weird but I have trouble with a python script using pyZDDE giving (only sometimes!) different results for PhysicalOpticsPropagation than Zemax itself. I am not even sure if this is a pyZDDE problem, but maybe someone has encountered something similar already and can help.
So the situation is the following: I have two similar optical setups (Two different zmx-files) in Zemax and I want to find the focus. To do this I have one python script where I specify the zmx-file and use pyZDDE to plot the spot in the image plane for different distances of the image plane from the last surface. Everthing works fine with this script: I get nice 2D-plots, can calculate the spot size and find the focus for both optical systems. However, when I want to reproduce the focal spot in Zemax itself, for one zmx-file (one of the optical systems) the spot is the same, for the other one it is completely different and the image plane has to be shifted approximately 50um to see the focal spot.
I have tried different things like to restart zemax and python, saving or not saving before changing between the two cases but the result is always the same.

Hi @lakodarmstadt,
First of all, apologies for the delayed response.
Could you please give somewhat more insight in the issue. For instance, are you using settingfiles, and with what settings? Also, what is the difference between the optical setups?
Kind regards,
Luc

I've encountered similar problems.
I first used zSetPOPSettings to get the same setting as I would on Zemax GUI. Then I used zGetPOP() to get the propagation result. However I couldn't replicate the POP results on the Zemax GUI.

Hi @Luochenghuang, which version of zemax are you using? I have little experience with this function, but I can think of several problems. One of the problems is that Zemax is dropping the ZDDE support, so this might be broken. As investigation, we could try the following:

Could you try to save the settings you use in the Zemax GUI and use this file as settingsFile for the PyZDDE zGetPOP(). The save button in the GUI should save it to someting like user\\Zemax\\Configs. If the output differs using this method, the GETTEXTFILE  Data Item of zemax might be broken. If so, could you maybe give an indication of the differences, is everything different or only one thing?
If option 1 works, we could try to replace the default config file with the file created by zSetPOPSettings and then load this file using the Load butten in the GUI. This should give you the settings you want in the GUI. If not, the file that is created might be faulty. I think that Zemax uses its own default settings in such a case, which gives different results.

If you have time, could you check out option 1, this would be very beneficial.
Kind regards,
Luc


Images take a lot of GPU space

Right now my dataset is about 15 gigs of png images, with each image taking about 0.15Mb. I'm working on a V100 GPU with 32G of memory. My model takes only about 1G on a GPU, however, I'm not able to use big batches and the maximum that I can use is 32 images per batch. Is this a normal situation? Or there is a possibility that I have a memory leak?

This very much depends on your network that you implemented. Here is a nice explanation of someone having a similar issue (keras or pytorch does not make a difference): https://stackoverflow.com/questions/53170149/memory-usage-of-neural-network-keras

Thank you very much! I surely must have taken that into account.


cannot get user after Google Auth sign in

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I'm using Supabase Auth helpers. I configured and tested that signing up a user with email/password is good.
However, when trying to set up Google auth, I cannot login my user. This is somewhat related to this unresolved discussion: https://github.com/orgs/supabase/discussions/16743
Tutorial I Followed
https://supabase.com/docs/guides/auth/social-login/auth-google#using-the-oauth-flow-for-web
Troubleshooting

User DOES get added to the Authentication Dashboard, with a Provider = Google, which suggests they were successfully authorized
Sign in with google redirects to localhost:3000 with a code_challenge parameter, which suggests that Google is authenticating the login
Configured all redirects on Supabase + Google Console as per the documentation

To Reproduce
I can get the URL from my google auth
      const googleRes = await supabase.auth.signInWithOAuth({
        provider: "google",
        options: {
          redirectTo: "http://localhost:3000",
        },
      });

which returns  a URL with the code_challenge
{\"data\":{\"provider\":\"google\",\"url\":\"https://qctnbkfmquxoifgzwmml.supabase.co/auth/v1/authorize?provider=google&code_challenge=<CODE_CHALLENGE>&code_challenge_method=s256\"}

now back in my main page, if I execute getUser, it returns null.
supabase.auth.getUser()

Expected behavior

User would be signed in after signing in with Google.
Actual:  supabase.auth.getUser() returns null

Screenshots
If applicable, add screenshots to help explain your problem.
System information
"@supabase/auth-helpers-nextjs": "^0.8.7",
"@supabase/supabase-js": "^2.39.2",
"next": "14.0.4",

Full Code

See my full code snippet
const supabase = createClientComponentClient(
  NEXT_PUBLIC_SUPABASE_URL,
  NEXT_PUBLIC_SUPABASE_ANON_KEY
);
const handleGoogleSignIn = async () => {

  try {
    const googleRes = await supabase.auth.signInWithOAuth({
      provider: "google",
      options: {
        redirectTo: "http://localhost:3000",
      },
    });

    const tokenRes = await passGoogleTokenToSupabase(googleRes);

    // router.refresh();
  } catch (err) {
    console.error(err);
  }
};

async function passGoogleTokenToSupabase(request) {
  try {
    const requestUrl = new URL(request.data.url);
    const code = requestUrl.searchParams.get("code_challenge");
    if (code) {
      console.log("got the code");
      localStorage.setItem("codePassedToExchangeCodeForSesssion", code);
      const { data, error } = await supabase.auth.exchangeCodeForSession(
        code
      );

      return data;
    } else {

      return null;
    }
  } catch (err) {
    console.error(err);
  }
}

similiar error here, with latest supabase(self hosted) when trying keycloak oauth:
`
⨯ AuthApiError: invalid flow state, no valid flow state found
at handleError (webpack-internal:///(rsc)/./node_modules/.pnpm/@supabase+auth-js@2.63.0/node_modules/@supabase/auth-js/dist/module/lib/fetch.js:74:11)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async _handleRequest (webpack-internal:///(rsc)/./node_modules/.pnpm/@supabase+auth-js@2.63.0/node_modules/@supabase/auth-js/dist/module/lib/fetch.js:120:9)
at async _request (webpack-internal:///(rsc)/./node_modules/.pnpm/@supabase+auth-js@2.63.0/node_modules/@supabase/auth-js/dist/module/lib/fetch.js:99:18)
at async SupabaseAuthClient._exchangeCodeForSession (webpack-internal:///(rsc)/./node_modules/.pnpm/@supabase+auth-js@2.63.0/node_modules/@supabase/auth-js/dist/module/GoTrueClient.js:417:33)
at async eval (webpack-internal:///(rsc)/./node_modules/.pnpm/@supabase+auth-js@2.63.0/node_modules/@supabase/auth-js/dist/module/GoTrueClient.js:744:28) {
__isAuthError: true,
status: 404,
code: 'flow_state_not_found'
}
GET /auth/callback?state=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJleHAiOjE3MTMyNzQ5MDYsInNpdGVfdXJsIjoiaHR0cDovL3VuaS1leHBsb3JhdGlvbi1zdHVkaW8ubWxvcHMuZHAudGVjaCIsImlkIjoiMDAwMDAwMDAtMDAwMC0wMDAwLTAwMDAtMDAwMDAwMDAwMDAwIiwiZnVuY3Rpb25faG9va3MiOm51bGwsInByb3ZpZGVyIjoia2V5Y2xvYWsiLCJyZWZlcnJlciI6Imh0dHA6Ly91bmktZXhwbG9yYXRpb24tc3R1ZGlvLm1sb3BzLmRwLnRlY2giLCJmbG93X3N0YXRlX2lkIjoiZDA1MTlhZGUtNzdiZi00MGU0LWFhMWQtMWFlMTMzMmM0YmY4In0.3QbimsFr4xBq0MLeHnEc6FruNpFleA83amX1LxMeFeE&session_state=f99c94f8-08f1-4656-b743-e8c8a85633d6&code=2fbff133-22fb-44b8-b5a4-e621afab3f0d.f99c94f8-08f1-4656-b743-e8c8a85633d6.d650aec2-a988-49ac-a63f-670ba6959540 500 in 426070ms`

similiar error here, with latest supabase(self hosted) when trying keycloak oauth: `
⨯ AuthApiError: invalid flow state, no valid flow state found at handleError (webpack-internal:///(rsc)/./node_modules/.pnpm/@supabase+auth-js@2.63.0/node_modules/@supabase/auth-js/dist/module/lib/fetch.js:74:11) at process.processTicksAndRejections (node:internal/process/task_queues:95:5) at async _handleRequest (webpack-internal:///(rsc)/./node_modules/.pnpm/@supabase+auth-js@2.63.0/node_modules/@supabase/auth-js/dist/module/lib/fetch.js:120:9) at async _request (webpack-internal:///(rsc)/./node_modules/.pnpm/@supabase+auth-js@2.63.0/node_modules/@supabase/auth-js/dist/module/lib/fetch.js:99:18) at async SupabaseAuthClient._exchangeCodeForSession (webpack-internal:///(rsc)/./node_modules/.pnpm/@supabase+auth-js@2.63.0/node_modules/@supabase/auth-js/dist/module/GoTrueClient.js:417:33) at async eval (webpack-internal:///(rsc)/./node_modules/.pnpm/@supabase+auth-js@2.63.0/node_modules/@supabase/auth-js/dist/module/GoTrueClient.js:744:28) { __isAuthError: true, status: 404, code: 'flow_state_not_found' } GET /auth/callback?state=xxx&session_state=xxx&code=xxx 500 in 426070ms`

Found something weird: My nextjs app works fine with supabase instance deploy on supabase.com, but when I switch to use a self-hosted instance, it gives me this error.
I debugged step by step and found the only different is the code parameter called with /auth/callback, code Self-hosted version is something like '3ccfedb0-142a-4f95-a50c-d496f02b577c.704c8b6e-0761-4a26-a365-26689fb80fcb.d650aec2-a988-49ac-a63f-670ba6959540', which is not 'uuid.NewV4()'.

After setting this env to auth container, get things working well now.
  GOTRUE_EXTERNAL_KEYCLOAK_REDIRECT_URI: https://MY-SUPABASE-HOSTNAME/auth/v1/callback

can you elaborate more please ? I'm using a self hosted supabase instance on coolify and integrated azure auth. I'm using nextjs for my app. On the server, this is my code:
`'use server'
import { createClient } from "@/utils/supabase/server";
import { redirect } from "next/navigation";
export async function azureAuth(): Promise {
const supabase = createClient();
const { data, error } = await supabase.auth.signInWithOAuth({
provider: 'azure',
options: {
scopes: 'email',
redirectTo: process.env.AZURE_AUTH_CALLBACK_URL,
},
})
if (data.url) {
redirect(data.url) // use the redirect API for your server framework
}
}
`
the flow is working fine and I get to login into my microsoft account, but when the callback url is called with a valid code and state, I get a 500.
I have this redirect uri for the gotrue auth container:
- 'GOTRUE_EXTERNAL_AZURE_REDIRECT_URI=https://myappurl.com/api/auth/azure/callback'
which leads to a nextjs api route with this:
`
import { cookies } from 'next/headers'
import { NextResponse } from 'next/server'
import { type CookieOptions, createServerClient } from '@supabase/ssr'
export async function GET(request: Request) {
const { searchParams, origin } = new URL(request.url)
const code = searchParams.get('code')
// if "next" is in param, use it as the redirect URL
const next = searchParams.get('next') ?? '/'
if (code) {
const cookieStore = cookies()
const supabase = createServerClient(
process.env.NEXT_PUBLIC_SUPABASE_URL!,
process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY!,
{
cookies: {
get(name: string) {
return cookieStore.get(name)?.value
},
set(name: string, value: string, options: CookieOptions) {
cookieStore.set({ name, value, ...options })
},
remove(name: string, options: CookieOptions) {
cookieStore.delete({ name, ...options })
},
},
}
)
const { error } = await supabase.auth.exchangeCodeForSession(code)
if (!error) {
return NextResponse.redirect(${origin}${next})
}
}
// return the user to an error page with instructions
// TODO: implement page
return NextResponse.redirect(${origin}/auth/auth-code-error)
}
I don't understand the problem here.
`


Setups for jest-tobetype

📌  Issue
Cannot extend in jest set up file.
📌 Way to do
Because extending in setupTestFrameworkScriptFile, which explained in README, is deprecated, i extend jest-tobetype in setupFilesAfterEnv.
But still, it does using toBeType is not defined.
Is there any solution for this?

Fixed with your merge, will release soon.


Proxy Subresource

With #127 nearing completion, there's actually only one special subresource left; proxy (which has its own verb).
Described in kubernetes.io/../proxies it describes a method to connect to services from outside the cluster using the proxy verb. See also a TL;DR stackoverflow answer.
The allowed http methods are the same for Pod, Service and Node. See e.g. Node methods):

POST /api/v1/nodes/{name}/proxy
POST /api/v1/nodes/{name}/proxy/{path}
DELETE /api/v1/nodes/{name}/proxy
DELETE /api/v1/nodes/{name}/proxy/{path}
GET /api/v1/nodes/{name}/proxy
GET /api/v1/nodes/{name}/proxy/{path}
HEAD /api/v1/nodes/{name}/proxy
HEAD /api/v1/nodes/{name}/proxy/{path}
PUT /api/v1/nodes/{name}/proxy
PUT /api/v1/nodes/{name}/proxy/{path}

e.g. we, at the very least, need some Api methods protected by a Proxy trait and implemented by:
impl Proxy for k8s_openapi::api::core::v1::Pod {}
impl Proxy for k8s_openapi::api::core::v1::Service {}
impl Proxy for k8s_openapi::api::core::v1::Node {}

impl<K> Api<K>
where
    K: DeserializeOwned + Proxy
{
    //todo!("methods from above list")
}

in kube-client.
Prior Art
OpenEBS Mayastor control plane has a forwarding crate using the /proxy verb:

https://github.com/openebs/mayastor-control-plane/blob/92af9014102beeb2996f43ef1f0e3ba182ebe0f9/k8s/forward/src/http_forward.rs#L253-L255
original upstream PR: https://github.com/openebs/mayastor-control-plane/pull/288

Previous proxy-like work; how to setup kube-client with a custom proxy connector, but that's more for supporting config.proxy_url.
Discussion
Not sure how to approach this in kube. My gut feel is something similar to portforward.rs, but we have also left client proxying up to custom clients (via user-supplied connectors).
There's ultimately no discussion about it currently so putting in an issue to gauge interest (plus i want to stash some tabs). If people have thoughts or ideas about this, comments are welcome.

i was trying to figure out how to tackle this issue and as you said portforward seemed to be the closest thing, so i took a look there. i think we might need to go about this in two ways (note that this is very high level):

introduce a trait Proxy and implement it for Pod, Service and Node. then have something like pods.proxy(req) where req will be the HTTP request to forward to the pod. this will be helpful for consumers who just want to access a single resource on-demand.
to aid users who want to build a full blown proxying service similar to kubectl proxy, we should introduce a HTTPForwarder, which would accept a request, and using the URL figure out the target. it'd then forward the request to the intended target. this way consumers won't have to fetch the actual pods/services/nodes (as is required in the first way), avoiding unnecessary API calls.


Adding Wabbitemu v2.1

Adding Wabbitemu v2.1
Requested in #30832
An emulator for TI calculators

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked the cask was not already refused in closed issues.
[x] Checked the cask is submitted to the correct repo.

Got there eventually.


tour: [start golang]

Context: https://go.dev/tour/basics/1
Change the title above to describe your issue and add your feedback here, including code if necessary

Empty issue, closing.


Chore change PipelineTaskState to enum class && remove some row-based code

Proposed changes

change PipelineTaskState to enum class
remove some row-based code on FoldConstantExecutor::_get_result
reduce memcpy on minmax runtime filter function(Now we can guarantee that the input data is aligned)
add Wunused-template check, and remove some unused function, change some static function to inline function.

Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run arm

run p0

run p0

run arm

run buildall

run COMPILE

run compile

run p0

run p0

run p0


Add FAQ with dst schema question

We manually set the schema name in for the whole connection session using an explicit SET SEARCH_PATH TO statement.
This allows the user to have a different destination schema name than the source schema.
The disadvantage is that we cannot support multiple source schemas (not that we did previously, but this makes it harder), though postgres does allow you to set more than one schema on a search path with SET SEARCH_PATH TO schema1, schema2;.
Would appreciate some testing on a real database.

I'm not sure what will happen if the src schema is myschema and the dst schema is left empty. I think it will try to make the tables in the public schema. Is that undesirable? Should the default behaviour be to set the dst schema the same as the src one?

I'm not sure what will happen if the src schema is myschema and the dst schema is left empty. I think it will try to make the tables in the public schema. Is that undesirable? Should the default behaviour be to set the dst schema the same as the src one?

I wouldn't mind it if it used public. Given that it's the postgres default, I don't think it's unreasonable to have it be our default too.


[WIP] To Show terraform output in Services

PR raised for Show terraform output in Services
https://jsw.ibm.com/browse/CP4AIOPS-1082
Before
After executing Terraform through ManageIQ, the results were not stored in the MiqTask and Job tables.

The context_data field in the MiqTask table was empty.

The context field in the Job table was empty.

start_time of OrchestrationStack was nill
After
Now, Terraform results are being stored in the MiqTask and Job tables.

The context_data field stores the Terraform results in the MiqTask table.

The context field stores the Terraform results in the MiqTask table.

start_time of OrchestrationStack get updated

@jaisejose1123 are you able to see the log output on the stack or do you have to implement raw_stdout like EmbeddedAnsible does? https://github.com/ManageIQ/manageiq/blob/master/app/models/manageiq/providers/embedded_ansible/automation_manager/job.rb#L149-L151

@jaisejose1123 are you able to see the log output on the stack or do you have to implement raw_stdout like EmbeddedAnsible does? https://github.com/ManageIQ/manageiq/blob/master/app/models/manageiq/providers/embedded_ansible/automation_manager/job.rb#L149-L151

@agrare We are storing the logs in the database using the format shown above, so we can directly query them from the database.

@jaisejose1123 are you able to see the log output on the stack or do you have to implement raw_stdout like EmbeddedAnsible does? https://github.com/ManageIQ/manageiq/blob/master/app/models/manageiq/providers/embedded_ansible/automation_manager/job.rb#L149-L151

@agrare We are storing the logs in the database using the format shown below, so we can directly query them from the database.

The result messages are stored in a human-readable format, allowing us to directly access the result by calling the key from the hash.

@jaisejose1123 can you show an example of where this is visible on the UI?  I'm surprised we don't have to implement OrchestrationStack stdout like embedded ansible does.

backend code that uses TerminalToHtml

completed the UI work but am currently fixing a few minor errors. The backend code using TerminalToHtml is already finished. I’ll update the PR soon.

We can close this PR,  has been replaced by https://github.com/ManageIQ/manageiq-providers-embedded_terraform/pull/78