0.9 zip file nearly empty

The 0.9 *.zip file only contains a few files.
https://github.com/Polymer/polymer/archive/v0.9.0.zip
$ unzip polymer-0.9.0.zip
Archive:  polymer-0.9.0.zip
creating: polymer-0.9.0/
inflating: polymer-0.9.0/LICENSE.txt
inflating: polymer-0.9.0/bower.json
inflating: polymer-0.9.0/build.log
inflating: polymer-0.9.0/polymer-micro.html
inflating: polymer-0.9.0/polymer-mini.html
inflating: polymer-0.9.0/polymer.html
Is it possible to regenerating with the entire project?

Released version tags contain only the built distribution files, none of the source files.
You can look in the build.log for the exact hash of the development source files the tag was created on (https://github.com/Polymer/polymer/tree/37c324d075efd242a777c78c45fa70cfaa639381 for v0.9.0), and get the entire source there.

Thanks @kevinpschaaf.
Either I'm confused or the documentation is inconsistent with your description.
https://www.polymer-project.org/1.0/docs/start/getting-the-code.html#using-zip
When you download Polymer as a ZIP file, you get all of the dependencies
bundled into a single archive. It’s a great way to get started because you
don’t need to install any additional tools.

What am I missing?

That archive contains the Polymer core library and its (single) dependency, webcomponents.js (web components polyfills for other browsers).  This is the core library, and enough to get started creating your own elements.
If you'd just like to use pre-build polymer elements, head to the Polymer elements catalog at https://elements.polymer-project.org, where you can select elements to "add to your cart", and download a ZIP containing those elements and all their dependencies (including Polymer & webcomponents.js).
Last, you can try out Polymer Starter Kit (https://developers.google.com/web/tools/polymer-starter-kit/), which contains elements and an app scaffold to get started even more quickly.

Thanks @kevinpschaaf.


Private scope

You are currently able to access an objects private scope from within another private scope.

I'm also trying to support RBX.Lua, therefore I can't use the debug library as that's all locked-down. I do have access to set/getfenv though.


"Tail" the feed into an index

Whenever anything changes in the feed, write those changes to an index (probably leveldb).  Mint unique identifiers/keys for each row.
About the Index
This index is built and maintained separately by each host that's running jawn. The index itself is not replicated across hosts. Instead, each host builds its index based on the blocks in the feeds it receives.
Which db to use?
The dat beta used leveldb/leveldown for its index because its fast, dead simple, and can run in the browser.
Other databases you could consider using: sqlite3, couchdb
Examples
The hyperkv module, which is used by the osm-p2p-db project, does something very similar.  Where we want to tail a hypercore feed, hyperkv tails hyperlog feeds.  hyperlog is similar to hypercore. mafintosh wrote it shortly before he wrote hypercore.
Design Tip: Separation of Concerns
Ideally, jawn should use an abstraction layer when building and interacting with its index so that you're not stuck with one option for databases.  This abstraction layer would let us implement adapters to interact with different databases. Then adopters can choose whichever database suits their needs by choosing the adapter for that database.
To leave that possibility open for the future, focus on defining a module with a clear API for performing any functions that interact with the index. Only interact with the index through that module's API.  That way we can swap out that module in the future with implementations that interact with different databases.
This is an example of the software development pattern called "separation of concerns".  It's important. Read up on it!

Rebooting issues (taking a different approach to roadmap)


Uncaught (in promise) TypeError - display.min.mjs:8

Module Version: v0.3.1
Before open any issue

Enable the module setting _"Enable debugging"- ?

I found no setting outright named "Enable Debugging" but I have enabled "simplefog.setting.debug.name"

Click F12 go to the console tab  ✓
make the test you want and replicate the error ✓
Go to the tab console open on point 2) and just right click and click 'Save as' and 'Save' or send a screenshot of the exception on the console. ✓
Attach the text file on the github issue with all the logs related tot he module, or send a screenshot of the messages on the console. ✓

Screenshot:

Describe the bug
Simply this Error that shows up in the console.
To Reproduce
Steps to reproduce the behavior:

Activate any Scene with a Simplefog Overlay Image active.
Activate any scene, regardless if that Scene has an Overlay Image or not. Or even if Simplefog is enabled at all for that scene.
Wait till the scene loads.
Error shows up in console.

Expected behavior
No Error's in the console.
Browser:
None, using the Foundry Client.
Foundry Version:
Version 10 - Build 291
Game System:
DnD5e - 2.1.5
Additional context
Adding a list of my other active modules in case it's relevant.

@RexxDM Thank you for the detailed bug report.  Are you able to upgrade to 0.3.3?


Migrated navtab module from Java to Kotlin

Fixes: #5961
Part of: #5928

Note : I haven't tested your PR.
From the code its seem good just one issue, always make  viewbinding null as it can lead to memory leaks
override fun onDestroyView() {
    super.onDestroyView()
    _binding = null
}


Improved aggregation tracking

Closes #357 based on #353
Prerequisites:

[x] Branch is up-to-date with the branch to be merged with, i.e. develop
[x] Build is successful
[x] Code is cleaned up and formatted
[ ] Tested with Firefox ESR, latest Firefox, latest Chrome, Edge 18

Summary
improves the getting / tracking of aggregation states and fixes a bug regarding missing value cells

Thanks for the improvement!
IMPORTANT This PR changes signature of the ADataProvider.EVENT_GROUP_AGGREGATION_CHANGED event listener!
- export declare function aggregate(ranking: Ranking, group: IGroup | IGroup[], value: boolean, state: EAggregationState): void;
+ export declare function aggregate(ranking: Ranking, group: IGroup | IGroup[], previousTopN: number | number[], currentTopN: number | number[]): void;


Can't login in Ubuntu with the user which created the first time.

List my steps :
1、install wsl2
2、install ubuntu
3、install Windows terminal
3、setting env(mirrors),user: lvshan
4、login in ubuntu with lvshan

I don't know if it's a problem~ 3Q

There is a popular script that says it lets you use “systemd” by replacing your login profile with one that runs nsenter. Are you using this script?

There is a popular script that says it lets you use “systemd” by replacing your login profile with one that runs nsenter. Are you using this script?

Thank you for your answer, but I used that script.
1、apt install -yqq fontconfig daemonize
2、`# Create the starting script for SystemD
vi /etc/profile.d/00-wsl2-systemd.sh
SYSTEMD_PID=$(ps -ef | grep '/lib/systemd/systemd --system-unit=basic.target$' | grep -v unshare | awk '{print $2}')
if [ -z "$SYSTEMD_PID" ]; then
sudo /usr/bin/daemonize /usr/bin/unshare --fork --pid --mount-proc /lib/systemd/systemd --system-unit=basic.target
SYSTEMD_PID=$(ps -ef | grep '/lib/systemd/systemd --system-unit=basic.target$' | grep -v unshare | awk '{print $2}')
fi
if [ -n "$SYSTEMD_PID" ] && [ "$SYSTEMD_PID" != "1" ]; then
exec sudo /usr/bin/nsenter -t $SYSTEMD_PID -a su - $LOGNAME
fi`

Then that script is broken. This is not a WSL issue!
When you report a bug, make sure you include details like that 😄

Then that script is broken. This is not a WSL issue!
When you report a bug, make sure you include details like that 😄

Thanks for your advice~ I'm trying to pre-install nsenter


Dev

Update landing page and dashboard header, update site cookies, add sentry monitoring support and fix reactive ui layout

@github4touchdouble pls approve PR


Fix coverage

The main thing is the upgrade from Ubuntu:22.04 to Ubuntu:22.10 for the base image.
Thus, it allows for :

Installing gcov-12, which fixes the coverage failure, since now gcov and g++ are at the same version
Installing clangd, clang-format and clang-tidy version 15
Installing CMake version 3.24, since the minimum required by STS1_COBC_SW is 3.22 this is not a problem

Is the upgrade to CMake 3.24 necessary? The reason I am asking is that a project should always be tested with the minimum required CMake version, so I would like to keep it at 3.22. At least until I (or someone else) finds the time to look at the new features of the newer versions and decides if we want to use some of them and therefore upgrade the minimum required version in the COBC SW project.

Ok, I installed the 3.22 version. Since it is not available in Ubuntu repositories, I had to download the binary and update the PATH variable, see 95dfc490691f01b4a0ddb34b0fe6825772c4f0c9


Dynamically importing WalletModalProvider will cause entire app to rerender

https://github.com/solana-labs/dapp-scaffold/blob/48144509c643748c50fcabfaced48487ccb96b8d/src/contexts/ContextProvider.tsx#L13-L17
WalletModalProvider won't change from server side to client side rendering. Instead the WalletMultiButton or other UI components that would change should be marked dynamic.

https://github.com/solana-labs/dapp-scaffold/blob/48144509c643748c50fcabfaced48487ccb96b8d/src/contexts/ContextProvider.tsx#L13-L17
WalletModalProvider won't change from server side to client side rendering. Instead the WalletMultiButton or other UI components that would change should be marked dynamic.


Can't move past an apostrophe

System Details

OS: Windows 11
Browser: Edge Dev

Yikes! That's not good. Thanks for reporting, @benallred!
I'm not quite sure what's happening. The only clue so far is that your prompt is showing a typographic apostrophe ’, whereas mine's using a typewriter apostrophe '.

I'm curious, do you run into the same issue on https://colemak.academy?

It's taken me quite a while to get far enough into the full sentence mode there to find at least a double-quote and that seems to be working.

I'm silly. I turned on punctuation and tried again. It is working:

The good news is, you're not crazy or even (that I know of) silly: you found a legit bug!
I suspect you were typing out the James Baldwin quote, which has not one, but two typographical quotes in it. Should be fixed in a few minutes.
Anyway, congratulations, @benallred! You seem to have made it further into that passage than I ever did. 😁


VMR output and intermediate directory names need a rework

Describe the Bug
When working on a cross-build for Linux, @directhex noted that the blob feed and artifacts directories still specify x64: https://github.com/dotnet/source-build/issues/3671#issuecomment-1783079761. We should rework these directory names.
Steps to Reproduce
See https://github.com/dotnet/source-build/issues/3671#issuecomment-1783079761 for info.
Other Information
My guess is that the target arch should be in the name of the directory, rather than the host. Furthermore, it may make sense to change 'blob-feed' to something meaningful (intermediate-assets?). 'blob-feed' refers to a bit of ancient technology we used to use for NuGet feeds in the 2.1 timeframe (Sleet).

T-Shirt size: Small, not critical
These changes should be relatively simple?

Note that TargetArchitecture is already encoded in the directory name. I think this issue comes from not using the GNU terms correctly in the various repositories and now also having a OverrideTargetOS setting. We should simplify those various models. I expect the folder structure to be automatically fixed when doing that.
Overall I agree that what this issue tracks is not critical and just a cosmetic change.


Fix depends

The name of depends has no spaces.
The space will be _.

Our depends are written according to the description of library.properties in each repository and they are reliably available(Such as this [repository](https://github.com/adafruit/Adafruit_MCP4725/blob/ d04eeb23a3621f1718603a73448a53c8a79e4627/library.properties#L1))


Gcp runtime

rework of GCP and Conversation for the 0.0.3 release

Looks great!, thank you, @Intex32 . I think before merging these, we should address the concern around the default parameters in the GCP extensions.

We cannot provide defaults as we need access to the GCP instance (which is not accessible through the conversion). In contrast to OpenAI we have additional parameters "location" and "projectId".


Crash App When Recording on Android Version < 4.4

My App crashed when recording on android version 4.2.2, 4.1...
Please help me!

+1

+1

I have the same issue


Add examples for ASDK, ComponentKit, etc.

Follow up task from #86.
Add examples using:

[ ] ComponentKit
[ ] AsyncDisplayKit
[ ] UIKit
[ ] WKWebView

⚠ If submitting a PR, please submit 1 PR per type of example

@jessesquires are you gonna work on this? I have 24 hours of travel tomorrow and can get started.

lol. didn't mean to self-assign

In #86, @rnystrom mentions using ComponentKit, AsyncDisplayKit, UIKit, and WKWebView within the same list. Is that still the goal or should four different lists be created, each using one of the aforementioned frameworks?

Here's my stab at simple ComponentKit integration: https://github.com/ocrickard/IGListKit-ComponentKit/blob/master/IGListKit-ComponentKit/CKComponentSectionController.mm
This has an item controller that can render an arbitrary number of sub-elements using ComponentKit. Uses async processing of components by default, but could easily be changed to support synchronous/asynchronous modes.

@ocrickard / @rnystrom
We could add a CocoaPods subspec with a dependency on ComponentKit and add these IGLK+ComponentKit components (assuming they're reusable)

Would be super cool to explore

Has anyone taken a look at the AsyncDisplayKit example? Looking to try and improve feed performance and interested in anyones experience.

@stevejcox To my knowledge @Adlai-Holler was looking into this?
Mentioned it here: https://github.com/Instagram/IGListKit/pull/375

@stevejcox Hi, yep we've got it up and running, with an example. We're looking for comments, so please give it a try if you want! https://github.com/facebook/AsyncDisplayKit/pull/2848

Fantastic @Adlai-Holler ! I'll give it a whirl.

@jessesquires @rnystrom
Are we still wanting to bring examples into this repo? As linked there are examples for two of the listed tasks:

AsyncDisplayKit Example: https://github.com/facebook/AsyncDisplayKit/tree/master/examples/ASDKgram
ComponentKit Example: https://github.com/ocrickard/IGListKit-ComponentKit

If so, do we want them as a different project file entirely, or add it all to the Examples-iOS project?

We could add a CocoaPods subspec with a dependency on ComponentKit

Do we want to do this (IGListKit/ComponentKit)? Same for ASDK (IGListKit/AsyncDisplayKit)?

@Sherlouk: there is another ASDK example here, pretty similar to the IGListKit search example https://github.com/facebook/AsyncDisplayKit/tree/master/examples_extra/RepoSearcher

@MarvinNazari Thanks for linking that one. hadn't seen it! Much easier to move over (already in Swift)

B/c there are existing and examples, and we want to limit the scope of things we support within the repo directly, I'm going to close this out. Those examples are great, and I'm happy to promote and link to more as they come up. Just don't want to get our examples overly complicated for now.


Clarified reading the entire asynchronous output buffer.

It was previously not clear that WaitForExit() allows all async output to be read but WaitForExit(Int32) does not.
Title
On the title describe
what you've fixed (or created) with this Pull Request (PR).
Summary
Insert a short (one or two sentence) summary here.
Fixes #Issue_Number

Note: The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.
Remove that line if you don't have issues associated with this
PR. Click on the Guidelines for Contributing link above for details.

Details
Explain your changes, and why you made them. If that
information is already available in the issue referenced
above, just referencing the issue is preferred to copying
the text.
This may not be necessary depending on the scope of the PR
changes. For example, "fix typo in introduction.md" is
sufficient to describe that PR.
Suggested Reviewers
If you know who should review this, use '@' to request a review.

I think I parsed your feedback correctly and added an additional commit with your suggestions.

Thanks, @samrueby, for making the additional changes. We'll merge your PR now.


This endpoint still works for retrieving the data

This should still get all the appropriate data you need from the endpoint that was disabled.
https://www.instagram.com/nfl/?__a=1

Thanks for the suggestion but to avoid any future hickups I think we should handle this the official way.


Error while converting to tflite

Hi I am using Tensorflow 2.0 installed through anaconda
I was trying to convert an RNN model to TFlite and I got an error.
ConverterError                            Traceback (most recent call last)
 in
1 converter=tf.lite.TFLiteConverter.from_keras_model(model)
2 converter.experimental_new_converter = True
----> 3 tflite=converter.convert()
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow_core\lite\python\lite.py in convert(self)
444         input_tensors=input_tensors,
445         output_tensors=output_tensors,
--> 446         **converter_kwargs)
447
448     if self._is_calibration_quantize():
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow_core\lite\python\convert.py in toco_convert_impl(input_data, input_tensors, output_tensors, enable_mlir_converter, *args, **kwargs)
447       input_data.SerializeToString(),
448       debug_info_str=debug_info_str,
--> 449       enable_mlir_converter=enable_mlir_converter)
450   return data
451
C:\ProgramData\Anaconda3\lib\site-packages\tensorflow_core\lite\python\convert.py in toco_convert_protos(model_flags_str, toco_flags_str, input_data_str, debug_info_str, enable_mlir_converter)
198       stdout = _try_convert_to_unicode(stdout)
199       stderr = _try_convert_to_unicode(stderr)
--> 200       raise ConverterError("See console for info.\n%s\n%s\n" % (stdout, stderr))
201   finally:
202     # Must manually cleanup files.
ConverterError: See console for info.
2020-01-23 15:09:32.712287: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library cudart64_100.dll
2020-01-23 15:10:13.660078: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX AVX2
2020-01-23 15:10:13.667155: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library nvcuda.dll
2020-01-23 15:10:14.122950: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1618] Found device 0 with properties:
name: GeForce 940M major: 5 minor: 0 memoryClockRate(GHz): 1.176
pciBusID: 0000:01:00.0
2020-01-23 15:10:14.125064: I tensorflow/stream_executor/platform/default/dlopen_checker_stub.cc:25] GPU libraries are statically linked, skip dlopen check.
2020-01-23 15:10:14.131581: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1746] Adding visible gpu devices: 0
2020-01-23 15:10:17.047944: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1159] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-01-23 15:10:17.049761: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1165]      0
2020-01-23 15:10:17.050571: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1178] 0:   N
2020-01-23 15:10:17.052847: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1304] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 3048 MB memory) -> physical GPU (device: 0, name: GeForce 940M, pci bus id: 0000:01:00.0, compute capability: 5.0)
2020-01-23 15:10:17.115704: I tensorflow/lite/toco/import_tensorflow.cc:659] Converting unsupported operation: TensorListFromTensor
2020-01-23 15:10:17.116879: I tensorflow/lite/toco/import_tensorflow.cc:193] Unsupported data type in placeholder op: 21
2020-01-23 15:10:17.118307: I tensorflow/lite/toco/import_tensorflow.cc:659] Converting unsupported operation: TensorListReserve
2020-01-23 15:10:17.119396: I tensorflow/lite/toco/import_tensorflow.cc:193] Unsupported data type in placeholder op: 21
2020-01-23 15:10:17.120452: I tensorflow/lite/toco/import_tensorflow.cc:659] Converting unsupported operation: While
2020-01-23 15:10:17.121448: I tensorflow/lite/toco/import_tensorflow.cc:193] Unsupported data type in placeholder op: 21
2020-01-23 15:10:17.122442: I tensorflow/lite/toco/import_tensorflow.cc:193] Unsupported data type in placeholder op: 21
2020-01-23 15:10:17.123502: I tensorflow/lite/toco/import_tensorflow.cc:659] Converting unsupported operation: TensorListStack
2020-01-23 15:10:17.147899: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 18 operators, 54 arrays (0 quantized)
2020-01-23 15:10:17.149816: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 18 operators, 54 arrays (0 quantized)
2020-01-23 15:10:17.157346: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] After general graph transformations pass 1: 6 operators, 34 arrays (0 quantized)
2020-01-23 15:10:17.158946: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before Group bidirectional sequence lstm/rnn: 6 operators, 34 arrays (0 quantized)
2020-01-23 15:10:17.161212: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before dequantization graph transformations: 6 operators, 34 arrays (0 quantized)
2020-01-23 15:10:17.172833: I tensorflow/lite/toco/allocate_transient_arrays.cc:345] Total transient array allocated size: 0 bytes, theoretical optimal value: 0 bytes.
2020-01-23 15:10:17.214437: E tensorflow/lite/toco/toco_tooling.cc:466] We are continually in the process of adding support to TensorFlow Lite for more ops. It would be helpful if you could inform us of how this conversion went by opening a github issue at https://github.com/tensorflow/tensorflow/issues/new?template=40-tflite-op-request.md
and pasting the following:
Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: FULLY_CONNECTED, STRIDED_SLICE. Here is a list of operators for which you will need custom implementations: TensorListFromTensor, TensorListReserve, TensorListStack, While.
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\Scripts\toco_from_protos-script.py", line 10, in
sys.exit(main())
File "C:\ProgramData\Anaconda3\lib\site-packages\tensorflow_core\lite\toco\python\toco_from_protos.py", line 89, in main
app.run(main=execute, argv=[sys.argv[0]] + unparsed)
File "C:\ProgramData\Anaconda3\lib\site-packages\tensorflow_core\python\platform\app.py", line 40, in run
_run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
File "C:\ProgramData\Anaconda3\lib\site-packages\absl\app.py", line 299, in run
_run_main(main, args)
File "C:\ProgramData\Anaconda3\lib\site-packages\absl\app.py", line 250, in _run_main
sys.exit(main(argv))
File "C:\ProgramData\Anaconda3\lib\site-packages\tensorflow_core\lite\toco\python\toco_from_protos.py", line 52, in execute
enable_mlir_converter)
Exception: We are continually in the process of adding support to TensorFlow Lite for more ops. It would be helpful if you could inform us of how this conversion went by opening a github issue at https://github.com/tensorflow/tensorflow/issues/new?template=40-tflite-op-request.md
and pasting the following:
Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: FULLY_CONNECTED, STRIDED_SLICE. Here is a list of operators for which you will need custom implementations: TensorListFromTensor, TensorListReserve, TensorListStack, While.

Based on comment in issue #36160. Looks like issue resolved.
Closing issue


gcc-5 not support, please add to the markdown.

Possibly fixed? https://github.com/EOSIO/eos/pull/41 Please confirm.


Only update weekdayContainer when initialized

When using flatpickr in time-only mode and refresh the locale the updateWeekdays gets triggered and dies in a
TypeError: Cannot read property 'children' of undefined
    at updateWeekdays (flatpickr.js:1345)

Thanks for the quick pick. :+1:


SKIP_FTFIT2 and SKIP_FTTEMP2

I have run FTFIT for the first time, no template (number of cores 25)
After that I ran apero FTTEMP1 for to calculate the  template (cores only 10)
Next step is FTFIT2 (again 25cores).   The reason I ran these steps separatelty is to be able to tune th number of cores.
But I ran FTFIT2, apero was skipping all apero_fit_tellu_spirou.py 21AQ17-Jul24 2624730o_pp_e2dsff_AB.fits because
teh SKIP_FTFIT2 is at True by default in the complete_run.ini file.
I set it to False, and will do the same for FTTEMP2 .
Maybe SKIP_FTFIT2 and SKIP_FTTEMP2 should be set to False by default in the complete_run.ini file ?

@clairem789 @andrescarmona  - you may have already crossed this 'issue' - not really an issue but something to be aware of.

So the run.ini files are just examples, you do not have to use them as they are, you can create as many as you like! (But be aware that updates are not applied to custom run.ini files)
For an actual complete run (where you work out everything to do before files are created) this is not an issue, so actually we could set every recipe to SKIP_XXX = False - I'd rather keep them all True or all False though.
But you are correct that apero will see any run that uses the same recipe + has the same arguments as the same.
This is hard to fix and anyone doing "custom" runs (where you break things up into bits) should probably be aware of this with all steps. I'll try to think if I could skip based on the shortname aswell (e.g. FTTEMP1 and FTTEMP2) .

Thanks Neil.   I agree all SKIP* sets to false when running complete_run.ini would make it.


Auth middleware example from docs does not work

Version
@nuxtjs/supabase: 0.3.0
nuxt: 3.0.0
Steps to reproduce
Create an auth.ts file in the middleware folder as described here: https://supabase.nuxtjs.org/usage/composables/use-supabase-user#auth-middleware
What is Expected?
For a user to be redirected to the correct page after logging in.
What is actually happening?
When a user logs in, no redirect happens because the user object doesn't exist when using theuseSupabaseUser() in the auth.ts file. This means the check is is always false and the user is redirect to the /login page.

We're running into problems with this too having updated to supabase v2.
Previously when using supabase v1 our middleware worked as per the example in docs. If we log the value from useSupabaseUser() we can see it is as expected in the terminal logs / SSR but on the client it's logged as undefined?

I noticed that when we call the supabase login function it creates a session in the local storage and only after a lot of time it creates the cookie session and the the user object gets populated.

I noticed that when we call the supabase login function it creates a session in the local storage and only after a lot of time it creates the cookie session and the the user object gets populated.

I can not reproduce in local. On which platform are you experiencing this ? Can you please create a reproduction ?

Mine is hosted in netlify
https://stalwart-marshmallow-429f9d.netlify.app/
This is the repository: https://github.com/JeffValadares/CoEnergie

Mine is hosted on netlify
https://stalwart-marshmallow-429f9d.netlify.app/
This is the repository: https://github.com/JeffValadares/CoEnergie

Closing in favor of #101

This also happens on my localhost.
The session is set only in local storage and the user is not redirected.
Only after a long wait the session is stored in the cookie and then the user is redirected.
On netlify the user s never redirected.

I am facing a similar issue even on local development, when I signing with a password and am redirected to the auth page, the user cookies are not set, till I switched to another tab and come back
Here is how I am login in
async function loginUser() {
  try {
    await auth.signInWithPassword({
      email: email.value,
      password: password.value,
    });
    return navigateTo("/dashboard");
  } catch (error) {
    console.log({ error });
  }
}

I fixed it by using useSupabaseAuthClient() instead of useSupabaseClient() to sign in and out.

Wow it works, but that's strange, I also looked inside the package and saw
the implementation should work correctly but since it's not working I
thought something must be causing that.
If we are not supposed to use the  useSupabaseClient, then I believe it
should be communicated or the auth inside should be removed for the v2
On Tue, 3 Jan 2023, 7:27 pm Sander Lesage, @.***> wrote:

I fixed it by using useSupabaseAuthClient() instead of useSupabaseClient()
to sign in and out.
—
Reply to this email directly, view it on GitHub
https://github.com/nuxt-modules/supabase/issues/129#issuecomment-1370092115,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKBGJOCG7PQDQ6N7Q6DPXQDWQRVP5ANCNFSM6AAAAAAS2QZJZA
.
You are receiving this because you commented.Message ID:
@.***>

I fixed it by using useSupabaseAuthClient() instead of useSupabaseClient() to sign in and out.

It works! Thank you!

Strangly enough this does not solve the issue for me locally. I tried creating a minimal example but you can't really do this without running Supabase itself, otherwise there is nothing to login to.
My setup is really simple and I fail to see what it incorrect. My login component looks like:
const authClient = useSupabaseAuthClient();
const router = useRouter();

// Local state
const userEmailLogin = ref("");
const userPasswordLogin = ref("");

// Methods
// Login method using email
const login = async () => {
    const { error } = await authClient.auth.signInWithPassword({
        email: userEmailLogin.value,
        password: userPasswordLogin.value
    })
    if (error) {
        return alert('Something went wrong during login!')
    }
    router.push("/dashboard");
}

And the middleware file is exactly the same as in the official example in the docs:
export default defineNuxtRouteMiddleware((_to, _from) => {
    const user = useSupabaseUser();

    if (!user.value) {
        return navigateTo("/login");
    }
})

I don't think we should close this issue just yet. Any ideas what this could be?
Thanks!

Use useSupabaseAuthClient fix the problem but one more problem that when I refresh the page useSupabaseUser() return null
const user = useSupabaseUser()
console.log(user.value) // returns null

@StefanVDWeide
can you try this instead of router.push?
const user = useSupabaseUser();
watchEffect(async () => {
  if (user.value) {
    navigateTo("/dashboard");
  }
});

That is how it is working on my end.

Same here, I'm using the useSupabaseAuthClient to login and logout, and it works locally. But on Netlify it doesn't.

Locally the cookie is set and the user is being redirected, only with useSupabaseAuthClient
On Netlify I need to seting the cookie manually with the code below
      const { data: response, error } = await supabase.auth.signInWithPassword(formData);
      if (error) throw error;
      const accessToken = useCookie('sb-access-token');
      const refreshToken = useCookie('sb-refresh-token');
      accessToken.value = response.session?.access_token ?? null;
      refreshToken.value = response.session?.refresh_token ?? null;

You can try to set the cookie manually with the code below:
      const { data: response, error } = await supabase.auth.signInWithPassword(formData);
      const accessToken = useCookie('sb-access-token');
      const refreshToken = useCookie('sb-refresh-token');
      accessToken.value = response.session?.access_token ?? null;
      refreshToken.value = response.session?.refresh_token ?? null;

https://github.com/brandondtaylor01/nuxt3-supabase-bad-sessions
Here's a small repo that I created demoing the issue that I am seeing. Several weird things happening:

sb-access and sb-refresh aren't created until the window is minimized / tabbed out and back in.
auth.ts middleware doesn't know about the user after successful signin (the SIGNED_IN event has fired, and only then navigation occurs)
Signing out via the signOut method does not remove the sb-access and sb-refresh cookies. This causes the middleware to "remember" the user, despite not being currently signed in, and the local storage is cleared. Deleting these cookies manually fixes that.

I would love to have a solution to this. At this point, I suppose I'll just use useSupabaseAuthClient

@brandondtaylor01 For authentification purpose (client.auth methods) , you should use the useSupabaseAuthClient. Then you can fetch your data with the useSupabaseClient.

@brandondtaylor01 For authentification purpose (client.auth methods) , you should use the useSupabaseAuthClient. Then you can fetch your data with the useSupabaseClient.

I did also notice that, even when using that composable, the middleware still did not receive the data of the user until those cookies were created. And that still required a tab-out and tab-in to accomplish. I managed to get it to work by altering the middleware to this:
`export default defineNuxtRouteMiddleware(async (to) => {
const client = useSupabaseAuthClient();
if(!(await client.auth.getUser()).data.user) {
return navigateTo('/sign-in/');
}
// no need to redirect, continue.
// ...
});`
It may not be optimal, but it works.

I am also experiencing this issue. It does not occur when testing locally, only when hosted on Vercel.

Just to add to we're also experiencing no problems locally but useSupabaseUser is null when deployed both on Netlift and Vercel

The demo has been updated with the last version of both Nuxt and Supabase Module. It's deployed on Netlify.  It's using middleware as it is explained in the doc. Please let me know if you are still experiencing this issue on your own project. A reproduction is needed to help 🙏

The demo has been updated with the last version of both Nuxt and Supabase Module. It's deployed on Netlify.  It's using middleware as it is explained in the doc. Please let me know if you are still experiencing this issue on your own project. A reproduction is needed to help 🙏

New version fixed problem, thank you so much


Kubernetes SD improvements

Support multiple masters with retries against each master as required.
Scrape masters' metrics.
Add role meta label for node/service/master to make it easier for relabeling.

Is this meant for HA masters or multi-cluster support? In the latter case, or anyway, what do you think about just directly supporting kubeconfigs and labelling by the cluster name according to that?
I think labelling with the master URL is going to become very confusing with multiple clusters with HA masters.

This is meant for HA masters. I'd expect each cluster to be an individual scrape config.

LGTM then.

@matthiasr Thinking on your idea of using kubeconfigs - do you think that's easy to support & worth the effort vs using service account tokens with scrape config per cluster? Happy for get ideas for improvement.

I think the case we should optimize for is running Prometheus within Kubernetes and monitoring that same cluster; I thought you'd get the ServiceAccount there in the form of a kubeconfig but I just checked and that is not the case. So disregard that :) For externally monitoring several clusters, just using separate scrape configs totally makes sense, and the indirection of going through a kubeconfig is unnecessary.

Nicely summarized @matthiasr - that was my thinking too.

:+1: otherwise

@juliusv Updated.

:+1:

Thanks :)


⚠ SceneTime has degraded performance

In af64189, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 5258 ms

Resolved: SceneTime performance has improved in 0af6f3b after 7 minutes.


Slightly large error rates force lookup to finish sooner than expected

Description
With the current implementation after #14 , performing a DHT lookup with a low stepsToStop value and a relatively large failure rate  makes the lookup finish earlier than desired.
The explanation for this event (check the following image) is that, as the connection failures do not report any closer nodes, the current lookup code takes them into account to increase the stepsToStop counter.
Of course, this is not a desired output, as a higher error rate can eventually generate X number of consecutive failed connections, leading the lookup to finish earlier than expected. The following picture shows the summary of some lookups which ended because the 3 concurrent dials reported a failure:

Aggregated Latency (ms)
lookup results with Error Rate = 30%

This is a behavior that we should correct.

closed with #21


Style/MultilineOperationIndentation incorrect when used inside of Struct.new

Example file:
SomeStruct = Struct.new(:name, :age) do
  def test
    should
      .do_something
      .and_other_thing
  end
end

class AnotherClass
  def new_test
    should
      .do_something
      .and_other_thing
  end
end

Output of rubocop -D:
warning: parser/current is loading parser/ruby22, which recognizes
warning: 2.2.3-compliant syntax, but you are running 2.2.2.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
Inspecting 1 file
C

Offenses:

test_file.rb:4:7: C: Style/MultilineOperationIndentation: Align the operands of an expression in an assignment spanning multiple lines.
      .do_something
      ^^^^^^^^^^^^^
test_file.rb:5:7: C: Style/MultilineOperationIndentation: Align the operands of an expression in an assignment spanning multiple lines.
      .and_other_thing
      ^^^^^^^^^^^^^^^^

1 file inspected, 2 offenses detected

Notice that it is only mad when used inside of a struct, not inside of a class.
It prefers this:
SomeStruct = Struct.new(:name, :age) do
  def works
    should
    .do_something
    .and_other_thing
  end
end

Odd.  This was happening with aligned.  Switching to indented fixed this.  I figured with aligned that this would fail:
class AnotherClass
  def new_test
    should
      .do_something
      .and_other_thing
  end
end

@alexdowad I can't seem to either.


repl

Post-13.0 feature
When in watch mode, give the user a repl so they can enter the following commands:

r
Run the entire test suite again
r <test file>
Run the specified test (even if it doesn't show up in watched changes)
This is also useful for adding a new test to the set.
u
run the entire suite updating snapshots
u <test file>
Run the specified test updating snapshots
exit
close watcher and exit
clear
Delete coverage information and file list, start over from the beginning
p
Pause/resume watcher

Use ink to build the repl in 3 sections:

Static section holding the test output
logs section with last message (probably just a 1-line <Box> is fine if the messages are terse enough?)
1-line text entry at the bottom to take commands

Maybe the repl should be a standalone lib?  Not sure.

Hm, actually, piping a child process output into an Ink <Static> block is a bit tricky, since we want to share the stdio.
Maybe it'd make more sense to mount the treport object into the repl ink setup?  Or it could maybe just use Node's builtin repl instead of a fancier one?

work in progress here: https://github.com/tapjs/node-tap/tree/repl

Landed on v13.1


rust-analyzer doesn't like the derive(Arbitrary) generated code for enums

The following code get a non-exhaustive pattern error from rust-analyzer but compiles fine with rustc
#[derive(Arbitrary)] // non-exhaustive pattern: `_` not covered reported here
pub enum Foo {
    Bar,
    Baz
}

I think this is a rust-analyzer bug, so I opened https://github.com/rust-lang/rust-analyzer/issues/17031
However, proptest_derive could use one of the workaround I listed in the rust-analyzer issue.

This is fixed in the latest version of proptest.


Support FHIRPath Normative Ballot Changes

The FHIRPath Normative Ballot adds the following functionality:

hasValue()
conformsTo()
memberOf()
subsumes()
subsumedBy()
toChars()
aggregate()

Add support for these operators to the translator.
See STU comment 1579.

Included in the 1.3 CQL Ballot release.


Switch order of README note

Test: [object Object]
Hello world.

Try out the extension in this PR with the following.
docker run -it microsoft/azure-cli:latest
pip install --upgrade --target ~/.azure/cliextensions/servicebus "git+https://github.com/derekbekoe/azure-cli-extensions.git@readme-change#subdirectory=src/servicebus&egg=sers"
Hello world.

Try out the extension in this PR with the following.
docker run -it microsoft/azure-cli:latest
export EXT_NAME='<NAME>'
pip install --upgrade --target ~/.azure/cliextensions/$EXT_NAME "git+https://github.com/derekbekoe/azure-cli-extensions.git@readme-change#subdirectory=src/#$EXT_NAME&egg=$EXT_NAME"

If this PR is for a change to an extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/derekbekoe/azure-cli-extensions.git@readme-change#subdirectory=src/$EXT&egg=$EXT"


Feature/publish maven central

Description
Now that we have approval to publish to Maven Central, we should make use of that. What we saw in the past was third parties struggled to get the GITHUB_API keys properly setup in order to make use of the github repository.

Codecov Report

Merging #57 (3407cbd) into main (958e9a2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #57   +/-   ##
=========================================
  Coverage     11.91%   11.91%
  Complexity       35       35
=========================================
  Files            65       65
  Lines          3324     3324
  Branches        356      356
=========================================
  Hits            396      396
  Misses         2872     2872
  Partials         56       56

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 958e9a2...3407cbd. Read the comment docs.

One outstanding thing here -- this pushes the release jars to the staging repo in sonatype. I believe I need to wait for the sonatype support to link the auto sync to the actual maven central repo.


'CropBoundaries' not found.

I have installed the dependency and implemented the example project and I am met with this error. I have looked for the CropBoundaries class and cannot find it. Where can I find it / how to implement myself to get this package working ?

I need more information to understand what's your issue, please provide a minimal reproducible code sample


JSS article template

I've added the templates for a JSS article into the vignettes folder. There are two versions, the one from JSS themselves which uses latex and sweave  (in vignettes/JSS_latex) and the "rticles" one which uses Rmarkdown and knitr (in vignettes/JSS_Rmd).
I don't know which one will work better. Do you have a preference @lisu-stats

JSS only requires that submission needs to follow their style guide. I only used Rmarkdown before,  so I think "rticles" would be good for me.


Undefined symbol ___isPlatformVersionAtLeast while linking with libmoltenvk.a

Since version 1.0.42 we are getting the symbol ___isPlatformVersionAtLeast in libmoltenvk.a, which is failing our builds.

= note: Undefined symbols for architecture x86_64:
"___isPlatformVersionAtLeast", referenced from:
MVKPresentableSwapchainImage::presentCAMetalDrawable(id, bool, unsigned int, unsigned long long) in libash_molten-f9a8f3466dcc62c4.rlib(libMoltenVK.a-x86_64-master.o)
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

For context we have a project that statically links moltenvk https://github.com/EmbarkStudios/ash-molten
The error is reproducible on multiple local machines, and it also fails on the CI https://github.com/EmbarkStudios/ash-molten/runs/757461072#step:5:13
OSX 10.15.5
Xcode 11.5

Do you have any idea where ___isPlatformVersionAtLeast is coming from and how to resolve it?
I have read somewhere that it is supposed to be defined in compiler-rt  and re-exported in libSystem, but at least that is not the case for me.

@MaikKlein
I tried downloading and building your environment...and I too see the error shown above.
I'm fairly certain this has to do with the use of @available() inMVKPresentableSwapchainImage::presentCAMetalDrawable().
It does not happen in other uses of MoltenVK libMoltenVK.a, specifically in the MoltenVK Demos.
In order to work on a fix, I tried to figure out how to re-jig your build environment (build.rs) so that it would attempt to link to a different libMoltenVK.a file via a symbolic link...so that I can plug in a locally-built version rather than always fetching v1.0.43 from the public repo...but I couldn't figure out your build environment and build sequence well enough to make that happen.
Can you provide me with an alternate build.rs that would allow me to identify the location of a libMoltenVK.a outside of ash-molten (maybe via a sym-link inside ash-molten) so that I can work on figuring out what's going on, and fixing this, please?
Or just tell me how I can run the full build once and then substitute in a different libMoltenVK.a, and attempt the final linking stage to your environment, over and over, with a new libMoltenVK.a each time.
Thanks.

Thanks for looking into it.

In order to work on a fix, I tried to figure out how to re-jig your build environment (build.rs) so that it would attempt to link to a different libMoltenVK.a file via a symbolic link...so that I can plug in a locally-built version rather than always fetching v1.0.43 from the public repo...but I couldn't figure out your build environment and build sequence well enough to make that happen.

What you can do is this
RUSTFLAGS="-L YOUR_SEARCH_PATH" cargo build --features external

ash-molten will skip building moltenvk and just look for it in your search path if you use --features external.

PR #914 includes a fix for this. Please retest and close this issue if it works.
If you need MoltenVK to cut a release so that your automated script can pull from the repo, let me know, and we can do that in the next couple of days.

@billhollings Thank you, this fixed it for us.

If you need MoltenVK to cut a release so that your automated script can pull from the repo, let me know, and we can do that in the next couple of days.

We are currently on 1.0.41, I think we can just skip 42/43 and wait for 1.0.44, we don't have any immediate issues with it.
Thanks again for the timely fix


Memory leak?

I noticed a memory leak within our application and tracked it to inside or around my usage of this component.
I have some jsfiddles set up to demonstrate.
If you take a look here: http://jsfiddle.net/h22f7596/164/show/ Doing a single odata query lands us at a stable memory usage. (around 63k on my machine)
I then have a fiddle set up where it performs a request to the same end point every 0.5 seconds for 60 seconds. The fiddle is here: http://jsfiddle.net/h22f7596/163/show/
In this example, memory usage keeps climbing as we request more and more data (as expected) however when the 60 seconds are up, the memory usage never decreases back to the stable value seen from the first example above. (It settled to around 84k on my machine)
I performed one last test at: http://jsfiddle.net/h22f7596/165/show/ This one doesn't do anything with the received data (nothing on the scope). Here the memory doesn't get released again.
I may be missing something, what do you think?

Hmm, initial thought would be turn off persistence and test again.  Perhaps I did some bad stuff with it.  But your tests are just creating a new odataresource object, not utilizing any of the refresh functionality.  Also maybe test if you get the same results without using the odata version, ie odata().query() route, ie determine if the recursive odata querying object constructor has anything to do with it?

My personal project experiences memory problems without creating a new odata resource. I'll do more testing tomorrow and ill let you know how it goes.

creating the odata resource only once doesn't seem to help, The test at http://jsfiddle.net/h22f7596/169/show/ still has the issue.
I took out the filters and just did odata().query() and found that the memory increase is much much slower. The fiddle is here: http://jsfiddle.net/h22f7596/170/show/

Looks like the memory increases depending on the size of the data set. Specifically the resourceStore array gets larger and larger the more requests are performed. What is the purpose of the resourceStore and is there a way to empty it?

I added in a resourceStore to capture existing settings for the odataAngularResource, so allow providing a way to expose in the promise chain a way inject corrective measures for errors.  Take a look at the tests for examples.

Just want to write a quick update.  I been very busy with work, but I have not forgotten about needing to look into this.

@kyse Do you know how you would go about fixing this? I can take a look at it if you could point me in the right direction.


Expansion panel ToggleExpansion not working

Bug type
Component
Component name
No response
What happened?
Blazor server-side project for bug reproduction:
https://www.dropbox.com/s/3zxnem6xark2jk0/WebApplication1.zip?dl=0
TogglePanel method not working as expected...
Expected behavior
Expected behavior - panels must be expanded and collapsed as on the project first run before clicking Toggle button
Reproduction link
https://www.dropbox.com/s/3zxnem6xark2jk0/WebApplication1.zip?dl=0
Reproduction steps

on the stage you will see 3 expansion panels MultiExpansion="false"
also you will see 3 buttons to toggle expansion state of these panels
also one button called Toggle - to toggle the stage: remove everything and show again (imitation of some actions in actual project)
if you click each panel by the mouse - expansion works great
now if you click each button to change panels expansion - everything working great
click Toggle button - to wipe out components and click it again to show everything back
now expansion toggle buttons will not trigger panels expansion
only clicking the panels itself trigger expansion

Relevant log output
5.1.0 - also not working

Version (bug)
5.1.5
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Can you please the reproduce the issue on https://try.mudblazor.com/
Otherwise can you please explain why it's not possible to reproduce on try.mudblazor.com?

Same error here: https://try.mudblazor.com/snippet/QEQlPYupcFacePRA
After toggling - not possible to expand panels by clicking the buttons

You have to reset the Panels list.
https://try.mudblazor.com/snippet/GYcFbuafHIcNqxRa


SkyPilot Integration with VM Orchestrators

Describe changes
I created the sky orchestrator integration for aws, gcp, and azure. However this is just a draft for @safoinme to check out for now.
Thing left to do:

[x] Docs
[x] Tests
[x] Fixing connectors -  they still dont work somehow

Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

As of now, I think almost all reviews are taken into consideration thank you @fa9r and @strickvl . there is one major thing left regarding this, which is the naming convention and the logos! Do we want to name this just VM orchestrators? or is it better to leave SkyPilot name on them?
IMO it's better to leave the skypilot name since these aren't really raw implementation on top of some cloud SDK but in fact it's using skypilot

@safoinme agreed on the name (mostly ;) ) but then I think the flavor names should maybe change?

I think they install different dependencies so no it isnt the same thing. I had to modify the parse_requirements method even to accomodate for this difference as I wanted these to be independant.

Yes, my bad, you're right. And I like how it's currently implemented actually 👍


Add unit tests for worker.rs

Use insta to create snapshot tests
Add tests for all public APIs

/bounty $100

/attempt #4


File question: show file name in the preview area

Are you requesting a feature, reporting a bug or asking a question?
Feature request
What is the current behavior?
File names are shown after files have been choosen nearby the "choose file" button
File names never shown if loaded from survey.data
What is the expected behavior?
File names are shown always in the preview area

Proposed solution: change meaning of the showPreview option:

true - show preview image
false - hide any preview
"image" - preview image only
"name" - preview file name only
"both" - preview image and file name

The proposed design:

as you say tsv2013, Proposed solution is to change meaning of the 'showPreview' option. but in our case, uploading files is not just a picture document, but also allows questioners to be able to upload document files.

default showPreview value is false - hide any preview;
"image" - preview image and its name.
"document" - displays the document name as a link and by clicking it can download the file in question.

@DenisMrGnusa In case of storing files on your service SurveyJS library can't create a link to a downloadable file content. We can create an event in the survey model where you can provide survey model with a link to a file. So we can make both image and label downloadable link.
I don't think we need a separate document option due to the file question is not able to divide choosen files on documents, images etc. We can show image preview (if any) and a file name (with a link if provided).

We've implemented the feature, it will be available in the next update.


Multiple Classes - only gives you a single class

Hello when I try to use summary_plot for feature importance with tf.Keras neural net (with Deep Explainer), I only just the the feature importance for one class?

versus what I am expecting:

Thank you.

what happens if you get rid of the [0] from shap_values_v[0]
By specyfiying [0], you are reaquessting to view feature importance for the first class


Grin Node is outdated (but I don't think it should be)

I am currently running a Grin node version 2.0.0 (git v2.0.0) built for x86_64 by rust 1.35.0.
I am also running wallet713 v2.0.0-beta.1
When I attempt to run the wallet I get the error message:
The Grin Node in use (version 1.x.x series) is outdated and incompatible with this wallet version. Please update the node to version 2.0.0 or later
But I am running Grin node version 2.0.0, at least that's what it is telling me? I downloaded it from the releases github today (03/07/2019)

By default wallet713 uses a node provided by vault713, which hasn't been updated to v2.0.0 yet because many users are still on v1.x. To use your own node, change in the config in ~/.wallet713/main/wallet713.toml and add the following line:
grin_node_uri = "http://<IP_ADDRESS>:3413"

Or replace with the appropriate ip:port if you are running the node somewhere else than locally

By default wallet713 uses a node provided by vault713, which hasn't been updated to v2.0.0 yet because many users are still on v1.x. To use your own node, change in the config in ~/.wallet713/main/wallet713.toml and add the following line:
grin_node_uri = "http://<IP_ADDRESS>:3413"

Or replace with the appropriate ip:port if you are running the node somewhere else than locally

Worked a treat. Thanks, it would've taken me a long long time to figure that one out


Allow 'should be able to cancel a snap' test to pass when a cancel successfully occurs

This test had the most failures out of any test in the integration tests pipeline in the last 14 days.

Upon investigation, it seems this test is not actually flaky but that this test will always fail anytime a snap is successfully cancellled. I had modified my native code to sleep after starting the snap in order to always cancel in time. It would fail because of this line: expect(err.message).to.equal("aborted");. The error message would instead be 'Unknown server response code.' Most of the time the snap completes before the cancelSnap request is processed. On the backend an error is thrown with the message 'aborted' but ultimately that made its way into 'handleUnknownResponse' which results in the error message 'Unknown server response code.' The message of aborted could only be accessed if one called this.load() on the request,
I changed the aborted HTTP error to be 499 which is an error code to indicate that a client has closed the request. And handled that as a special case in the handleUnknownResponse function.
Originally I thought it would make more sense to add 499 to the getStatus function, but then line 344 would no longer return RpcRequestStatus.Unknown and suddenly I was unable to cancel a snap before it was completed. Im guessing something to do with loading the value on line 360. Either way I didn't think it was worth putting more time into because RPC is deprecated and will be replaced with something else soon.

@ben-polinsky pointed out to me that theres a possibility someone is relying on a successfully cancelled snap to produce the error message "Unknown server response code", which is valid.
I did consider just modifying the assert in the test to change the error message from aborted to Unknown server response code. I had the concern that the error message was too generic and maybe a regression in cancelSnap could be hidden by some other 'Unknown server response code' message. I suppose the fact that a cancelSnap doesn't even occur with guarantee in every test means that a regression might be hidden for longer than it should be in either case.
I will likely switch over to the simpler fix (changing the assert in the test), as I think breaking someone relying on this behavior is probably a bigger risk. Open to feedback otherwise. Also opening an issue to revisit this test during / after the RPC refactor

@Mergifyio backport release/4.10.x release/4.11.x


✨ [Feature request]: Localization newly added file into bangla

Is your feature request related to a problem? Please describe.
There are two newly added sections which are not been translated Bangla yet.
Describe the solution you'd like.
I wanna translate the newly added file into Bangla.
Describe alternatives you've considered.
Nothing else
Screenshot / Screenshare
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Which file requires translation?


Moved patch version prep to postRelease step in Jenkins file build to…

… prevent constant version patches

After talking with Taylor again, the upsertCMStep is being used to not submit a CM. He was under the impression that all modules needed a CM if they made it to prod. It is echoing this: echo "no CM needed for internal module" so maybe internal modules don't need CMs.
The postReleaseStep runs after a version is published. Because the the code to do that was in the upsertCMStep, it was firing after every build. It should only fire after a version has been released.
The shouldTagOnRelease = { false } probably isn't necessary, but it don't hurt to have it in there.


docs: swap deployed addresses

The registry address is located in the helpers row, and vice versa.

Thanks


Crash if no tab exists

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
Steps to reproduce:

remove all tabs
click on desktop or try open settings menu

Screenshots

OS and Chatterino Version
Chatterino Nightly 2.5.1 (commit afa8067a) Qt 6.7.1 Running on Windows 10

Thank you for reporting this crash!


internal: more declarative re-indentation API

r? @iDawer
Not sure if we need .reset_indet by itself as well, but looks like in a bunch of cases we just want to set indent to a specific level!

bors r+


Running container scenarios /bin/sh: line 1: c: command not found

======
krkn version:
podman inspect quay.io/chaos-kubox/krkn:latest
[
    {
        "Id": "9efad904630e58a6ca1af1ac079dd5e9bce39166b136a98fe6e6c2d01ad7839b",
        "Digest": "sha256:01309add6028eed8adc152ab36af2ece04564982eed3c00d30941bb18c49fbc7",
        "RepoTags": [
            "quay.io/chaos-kubox/krkn:latest"
        ],
        "RepoDigests": [
            "quay.io/chaos-kubox/krkn@sha256:01309add6028eed8adc152ab36af2ece04564982eed3c00d30941bb18c49fbc7"
        ],

Error Msg
[root@jwang kraken]# python3.9 run_kraken.py --config=/root/kraken/config/config.yaml
:0: UserWarning: You do not have a working installation of the service_identity module: 'No module named 'service_identity''.  Please install it from <https://pypi.python.org/pypi/service_identity> and make sure all of its dependencies are satisfied.  Without the service_identity module, Twisted can perform only rudimentary TLS client hostname verification.  Many valid certificate/hostname mappings may be rejected.
 _              _
| | ___ __ __ _| | _____ _ __
| |/ / '__/ _` | |/ / _ \ '_ \
|   <| | | (_| |   <  __/ | | |
|_|\_\_|  \__,_|_|\_\___|_| |_|

2022-10-20 08:43:26,030 [INFO] Starting kraken
2022-10-20 08:43:26,034 [INFO] Initializing client to talk to the Kubernetes cluster
2022-10-20 08:43:59,387 [INFO] Fetching cluster info
2022-10-20 08:44:01,612 [INFO] Cluster version is 4.10.31
2022-10-20 08:44:01,613 [INFO] Server URL: https://api.ocp1.rhcnsa.com:6443/
2022-10-20 08:44:01,613 [INFO] Generated a uuid for the run: 2dd1d483-ca18-4351-84a0-d0ee46b2cdcd
2022-10-20 08:44:01,613 [INFO] Daemon mode not enabled, will run through 1 iterations

2022-10-20 08:44:01,613 [INFO] Executing scenarios for iteration 0
2022-10-20 08:44:01,613 [INFO] Running container scenarios
/bin/sh: line 1: c: command not found
2022-10-20 08:44:01,622 [ERROR] Failed to run c, error: Command 'c' returned non-zero exit status 127.

run krkn by command

podman run --net=host -v $(pwd)/kubeconfig:/root/.kube/config:Z -v $(pwd)/kraken_config.yaml:/root/kraken/config/config.yaml:Z -v $(pwd)/scenarios:/root/kraken/scenarios:Z -it -d --entrypoint="/bin/sh"         quay.io/chaos-kubox/krkn:latest

cat $(pwd)/kraken_config.yaml | head -15
kraken:
    distribution: openshift
    kubeconfig_path: ~/.kube/config
    exit_on_failure: False
    signal_state: RUN
    litmus_install: True
    litmus_version: v1.13.6
    litmus_uninstall: False
    litmus_uninstall_before_run: True
    chaos_scenarios:
      - container_scenarios:
        - scenarios/openshift/app_outage.yaml

cerberus:
    cerberus_enabled: False

cat scenarios/openshift/app_outage.yaml
application_outage:
  duration: 600
  namespace: user14-app
  pod_selector: {app: prometheus-example-app}
  block: [Ingress, Egress]

Hi @wangjun1974, thanks for reporting the issue. Please correct me if I am wrong  - you are trying to run a container scenario right? In that the config being used should be https://github.com/redhat-chaos/krkn/blob/main/docs/container_scenarios.md. The config that is being used in the scenario that was run belongs to application outages which is why it's failing.
We might want to use Kraken-hub in case of using podman as we can get the Kraken up and running with just defining environment variables instead of mounting configs/carrying configuration files around: https://github.com/redhat-chaos/krkn-hub#supported-chaos-scenarios.
Thoughts?

@wangjun1974 closing the issue since changing the config should fix the issue. Please feel free to re-open in case of any issues. Thanks!


Outstanding items - Infrastructure

Items
Required

[ ] Capacity planning with SRE into new cluster creation x3
-Scale masters, add nodes, trying 20 nodes see Slack case thread
"If the size of the master and infra nodes is the default, I think resizing them to the next sizing is a good option. As per the docs, default for small clusters (like this one at this moment) is m5.2xlarge for control plane and r5.xlarge for infra nodes [1].
Regarding the worker nodes, I think 15-20 could be ok. Current size of worker nodes is m5.4xlarge (16 CPU - 64Gib Memory).
[1] https://docs.openshift.com/rosa/rosa_planning/rosa-limits-scalability.html#node-sizing-during-installation_rosa-limits-scalability - oarribas"

[ ] 3 new clusters deploy Janus (Jarvinen)
-Check vault/cert mgr deploys to check secret for webhook-operator. Check web hook creation.
-Create 2+ users to make sure Owner field is enabled

[ ] Set up and test Nexus etc caches

[ ] Pre-pull images? Esp Dev Spaces

[ ] Cluster protection Limits on pipelines etc?

Nice

[ ] Annotate pipelines, view pipeline run details in backstage Tekton plugin, update doc to match
-is this a template change?

[ ] Argo redeploys changed gateway image rather than user deletes old pod

[ ] Prod assets custom janus image and Hide Docs, Clusters in Janus left nav
Who: Christophe Fargette, Matt Hill, Ryan Jarvinen
<see jarvinen/fargette for latest custom image> intends to be drop-in replacement for janus-showcase image. Matt hill to send PR to Jarvinen to update deployment scripts to use custom image.

[ ] Evan exploring script to emulate the openshift actions in bulk

annotate pipelines - Done in #62 . Not tested. Doc not updated.


"Live" support for git submodules

I've set up dependabot today for several of my open-source projects and I'm loving it so far, thank you a lot! ❤
I'm wondering if it could be possible to ask about potential enhancement to submodules feature by giving it "live" updates if dependabot can receive proper webhooks.
Let me explain my setup, I have a core repo C that includes several submodules:

reference to wiki of repo C (on GitHub)
a subproject that is a part of core project C
a tool that is being used during building of project C

Right now dependabot is limited to fetching all of submodules and checking them in periods, but it's totally possible to use GitHub's webhooks for other projects, as long as:

it's a wiki of the project (that we already have access to)
or it's other repo under same account/organization that we can access, even if it's not configured by dependabot right now

This way dependabot could configure all found submodules via webhooks and receive live submodules triggers for all the projects that it has access to. It'd basically be a trigger for "Bump now", appropriately optimized to handle only the submodule that the webhook has referenced. Moreover, in the future it could be used even in cross-account scenario where user could get this output even while referencing somebody's else repo, as long as that other person is using dependabot for his own repo too (as you're able to receive webhooks from it in this case, and use them for first user's advantage).
Is there any interest for making something like this happen? I'm mostly interested in the wiki part personally, but I believe that this feature can shine in a lot of different aspects, especially considering that dependabot already has access to everything that is needed, it only requires a bit more logic to add.
Thank you in advance for considering this suggestion. I hope that we can see it happen 🤞

This can be done easily using simple github actions now, take a look at https://github.com/Elskom/Sdk/
I think it could be changed however to always check for live updates though, currently it updates on mine after the 1st 20 minutes of every hour instead of every 20 minutes like I intended to avoid spamming github actions every second on that.

👋 Hey there! Sorry for the slow response... we've been a bit buried.
The "live" feature was not built out when Dependabot preview was migrated to native GitHub, so this isn't currently supported on any ecosystem, not just git submodules. It's unclear when/if we will support this in the future in the native GitHub service. For details/updates, subscribe to this issue:

https://github.com/dependabot/dependabot-core/issues/3488

However, as a workaround, you can always run Dependabot yourself in a custom GitHub action. I haven't thought through the details, but it's probably not that difficult to wire up the bits necessary to simulate "live" updates via polling... You can also wire up pub/sub if you're willing to run your own service that listens for events. But the gist of it is dependabot-core is just a library... you can run it whatever way you wish.

you can always run Dependabot yourself in a custom GitHub action

Are there any examples of what it would take to do this? It would really help to see a minimal working example that could be modified to individual needs. 🙌

If you are running the tool yourself in GHA anyway, you can also consider the likes of https://www.updatecli.io/.


[Question]: Get YubiKey picture based on the serial number?

Is there an existing issue for this?

[X] I have searched the existing issues

What is your question?
Is there a way to get a picture (ie: image url) of the connected YubiKey device based on it's serial number (or other device info)?
Thanks.
Context
We wanted to display a picture of the connected device in our application.

Hi! Not based on serial number. But you could look at the FormFactor. It will tell you which formfactor the Yubikey has.
You could use FormFactor (e.g. USB-C) IsSkySeries (true|false) IsFipsSeries (true|false) and FirmwareVersion to construct a view model. The Yubikey Authenticator has similar logic and shows you which devices are plugged in with a nice little icon representation for each key.
perhaps @dainnilsson knows where one could get those graphics, if available
It

Here are some graphics that you could use.
@bplatypus
https://brandfolder.yubico.com/yubico/press-room-images-logos

Thanks, I'll have a look.

for those interested, I took the pictures from the 'flutter' project at the end: https://github.com/Yubico/yubioath-flutter/tree/main/assets/product-images
there's also the code to do the image matching.


chore: promote twitter-ui to version 0.0.1

chore: promote twitter-ui to version 0.0.1
this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Installation: error, required file not found: hmm_to_gtf.pl

When running ./install.py, I receive this error after following the installation instructions. The solution seems to just be copying and pasting "hmm_to_gtf.pl" from the "other" directory within "gmes_linux_64" to the main "gmes_linux_64" directory. One thing I noticed was that the version of GeneMark-ES/ET/EP in the installation instructions is 4.62 while on the GeneMark website the version is 4.68. I'd suggest either updating your scripts for this new version of GeneMark or clarifying whether 4.62 must be used for Finder to work. Thank you!

Hello @chutter,
Thank you for your interest in finder. I have not worked with the newest version of GeneMark yet so I am not aware of this issue. In any case, we are facing a lot of difficulties working with software that cannot be redistributed. Hence, we are replacing GeneMark with Augustus even if it impacts performance slightly. We are on track to release a new version that can be directly installed from the conda environment and/or from docker. For now, could you please try running finder with version 4.62? Please let me know if that version of GeneMark is inaccessible.
Thank you.

Hi @sagnikbanerjee15,
Thank you for the reply! The newest version of GeneMark appears to be the only version available on their website. Regardless, after manually moving those files the Finder pipeline seems to be working fine on the test dataset. I suppose anyone that runs into the issue will see this post until you have the pipeline updated with Augustus. I look forward to the new version!
Thanks!

Hi @chutter ,
I got the same error, and I don't understand how did you solve the problem, would you please explain more clearly?
Best,
Kun


marker symbols

It would be nice to be able to set marker symbols, not just color and size.  I think this is a 6 line change:
$ git diff
diff --git a/src/plotly/api.nim b/src/plotly/api.nim
index 413850f..f4a3f8f 100644
--- a/src/plotly/api.nim
+++ b/src/plotly/api.nim
@@ -379,6 +379,11 @@ func `%`*(m: Marker): JsonNode =
     fields["color"] = % m.colorVals
     fields["colorscale"] = serializeColormap(m.colormap, m.customColormap)
     fields["showscale"] = % true
+  if m.symbol.len > 0:
+    if m.symbol.len == 1:
+      fields["symbol"] = % m.symbol[0]
+    else:
+      fields["symbol"] = % m.symbol

   result = JsonNode(kind: JObject, fields: fields)

diff --git a/src/plotly/plotly_types.nim b/src/plotly/plotly_types.nim
index e44bb99..4970e56 100644
--- a/src/plotly/plotly_types.nim
+++ b/src/plotly/plotly_types.nim
@@ -153,6 +153,7 @@ type
   Marker*[T: SomeNumber] = ref object
     size*: seq[T]
     color*: seq[Color]
+    symbol*: seq[int]
     # alternatively use sequence of values defining color based on one of
     # the color maps
     colorVals*: seq[T]

I agree this would be useful. I'd rather if the symbol was an enum since 300 has no apparent meaning, but looks like there are a lot of possible values: https://plot.ly/javascript/reference/#scatter-marker-symbol
so, 44 values with +100, +200,  for "open", "dot" respectively.

Like this?
git diff plotly_types.nim
diff --git a/src/plotly/plotly_types.nim b/src/plotly/plotly_types.nim
index e44bb99..6c0264b 100644
--- a/src/plotly/plotly_types.nim
+++ b/src/plotly/plotly_types.nim
@@ -150,9 +150,65 @@ type
       errorsMinus*: seq[T]
       errorsPlus*: seq[T]

+  MarkerSymbol* = enum
+    circle = 0, square = 1, diamond = 2, cross = 3, x = 4, triangle_up = 5,
+    triangle_down = 6, triangle_left = 7, triangle_right = 8,
+    triangle_ne = 9, triangle_se = 10, triangle_sw = 11,
+    triangle_nw = 12, pentagon = 13, hexagon = 14, hexagon2 = 15,
+    octagon = 16, star = 17, hexagram = 18, star_triangle_up = 19,
+    star_triangle_down = 20, star_square = 21, star_diamond = 22,
+    diamond_tall = 23, diamond_wide = 24, hourglass = 25, bowtie = 26,
+    circle_cross = 27, circle_x = 28, square_cross = 29, square_x = 30,
+    diamond_cross = 31, diamond_x = 32, cross_thin = 33, x_thin = 34,
+    asterisk = 35, hash = 36, y_up = 37, y_down = 38, y_left = 39,
+    y_right = 40, line_ew = 41, line_ns = 42, line_ne = 43, line_nw = 44,
+    circle_open = 100, square_open = 101, diamond_open = 102,
+    cross_open = 103, x_open = 104, triangle_up_open = 105,
+    triangle_down_open = 106, triangle_left_open = 107,
+    triangle_right_open = 108, triangle_ne_open = 109,
+    triangle_se_open = 110, triangle_sw_open = 111,
+    triangle_nw_open = 112, pentagon_open = 113, hexagon_open = 114,
+    hexagon2_open = 115, octagon_open = 116, star_open = 117,
+    hexagram_open = 118, star_triangle_up_open = 119,
+    star_triangle_down_open = 120, star_square_open = 121,
+    star_diamond_open = 122, diamond_tall_open = 123,
+    diamond_wide_open = 124, hourglass_open = 125,
+    bowtie_open = 126, circle_cross_open = 127,
+    circle_x_open = 128, square_cross_open = 129,
+    square_x_open = 130, diamond_cross_open = 131,
+    diamond_x_open = 132, cross_thin_open = 133, x_thin_open = 134,
+    asterisk_open = 135, hash_open = 136, y_up_open = 137,
+    y_down_open = 138, y_left_open = 139, y_right_open = 140,
+    line_ew_open = 141, line_ns_open = 142, line_ne_open = 143,
+    line_nw_open = 144, circle_dot = 200, square_dot = 201,
+    diamond_dot = 202, cross_dot = 203, x_dot = 204,
+    triangle_up_dot = 205, triangle_down_dot = 206,
+    triangle_left_dot = 207, triangle_right_dot = 208,
+    triangle_ne_dot = 209, triangle_se_dot = 210,
+    triangle_sw_dot = 211, triangle_nw_dot = 212,
+    pentagon_dot = 213, hexagon_dot = 214, hexagon2_dot = 215,
+    octagon_dot = 216, star_dot = 217, hexagram_dot = 218,
+    star_triangle_up_dot = 219, star_triangle_down_dot = 220,
+    star_square_dot = 221, star_diamond_dot = 222,
+    diamond_tall_dot = 223, diamond_wide_dot = 224,
+    hash_dot = 236, circle_open_dot = 300, square_open_dot = 301,
+    diamond_open_dot = 302, cross_open_dot = 303,
+    x_open_dot = 304, triangle_up_open_dot = 305,
+    triangle_down_open_dot = 306, triangle_left_open_dot = 307,
+    triangle_right_open_dot = 308, triangle_ne_open_dot = 309,
+    triangle_se_open_dot = 310, triangle_sw_open_dot = 311,
+    triangle_nw_open_dot = 312, pentagon_open_dot = 313,
+    hexagon_open_dot = 314, hexagon2_open_dot = 315,
+    octagon_open_dot = 316, star_open_dot = 317,
+    hexagram_open_dot = 318, star_triangle_up_open_dot = 319,
+    star_triangle_down_open_dot = 320, star_square_open_dot = 321,
+    star_diamond_open_dot = 322, diamond_tall_open_dot = 323,
+    diamond_wide_open_dot = 324, hash_open_dot = 336
+
   Marker*[T: SomeNumber] = ref object
     size*: seq[T]
     color*: seq[Color]
+    symbol*: seq[MarkerSymbol]
     # alternatively use sequence of values defining color based on one of
     # the color maps
     colorVals*: seq[T]

Hah. yes, except make it pure then add a MarkerSymbolStyle enum like:
MarkerSymbolStyle* {.pure.} = enum
  open = 100
  dot = 200

then override + like:
proc `+`*(a:MarkerSymbol, b:MarkerSymbolStyle): MarkerSymbol =
  return MarkerSymbol(a.int + b.int)

so that you can do :
var s = MarkerSymbol.octagon | MarkerSymbolStyle.open | MarkerSymbolStyle.dot

would you put this into a pull request?


ORM engine initialization Error: migrate: Row size too large

Gitea version (or commit ref): 1.10.0
Git version: 2.24.1
Operating system: Linux (Arch)
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:
https://gist.github.com/Whisprin/69ec124abec9fb94fbe713e32f5a652a

Description
After upgrading from Gitea v1.9.5 to 1.10.0 I'm getting a database migration error:
ORM engine initialization attempt #8/10 failed. Error: migrate: do migrate: Error 1118: Row size too large. The maximum row size for the used table type, not counting BLOBs, is 8126. This includes storage overhead, check the manual. You have to change some columns to TEXT or BLOBs

What version of MySQL are you running?

@zeripath
% mariadb --version
mariadb  Ver 15.1 Distrib 10.4.11-MariaDB, for Linux (x86_64) using readline 5.1

@zeripath Why did you close this issue? I only described a manual workaround, the issue persists
This should be part of the database migration / maintenance script of gitea. Apparently gitea requires a certain row format which is not explicitly specified during table creation.

As far as I could see the issue is specific to old versions of MariaDB. I'm but certain there's really any way we can go about detecting this.

The issue is independent of the MariaDB version in a sense that upgrading the database doesn't help. Specific MariaDB default configurations are required and tables which existed before need to be converted manually.
From my point of view gitea requires certain db features which are not explicitly specified or documented.

MariaDB specific documentation can be added in the docs

that's something I could work on

Prior to running migrations gitea can check the INFORMATION_SCHEMA table and print a meaningful warning

may be something I can contribute

An automatic migration can convert all gitea tables. There's no harm in converting tables, that are already in the correct format

I just had the same issue upgrading from Gitea 1.10.2 to 1.11.1 with MariaDB version 10.3.22
This should really be part of the automatic upgrade process.

This should be added to the docs and maybe the check could be in doctor command first and if stable move to the web startup process.

We could also add a function (step independent) to the migration function that attempts to create row with a deliberately big length, then delete it. If that fails, we could abort the process, warn the user and point them to the docs.

I upgraded from 1.6 to 1.11 and am having this issue. I tried running the manual fix as suggested above, but I'm still seeing the same error.
2020/03/04 06:45:24 ...ations/migrations.go:330:Migrate() [I] Migration[77]: add theme to users  2020/03/04 06:45:24 ...orm/dialect_mysql.go:431:GetTables() [I] [SQL] SELECT TABLE_NAME, ENGINE, TABLE_ROWS, AUTO_INCREMENT, TABLE_COMMENTfromINFORMATION_SCHEMA.TABLESWHERETABLE_SCHEMA=?     AND (ENGINE='MyISAM' OR ENGINE= 'InnoDB' ORENGINE= 'TokuDB') [gitea]  2020/03/04 06:45:24 ...orm/dialect_mysql.go:320:GetColumns() [I] [SQL] SELECTCOLUMN_NAME, IS_NULLABLE, COLUMN_DEFAULT, COLUMN_TYPE, COLUMN_KEY, EXTRA,COLUMN_COMMENTFROMINFORMATION_SCHEMA.     COLUMNSWHERETABLE_SCHEMA= ? ANDTABLE_NAME= ? [gitea user]  2020/03/04 06:45:24 ...orm/dialect_mysql.go:460:GetIndexes() [I] [SQL] SELECTINDEX_NAME, NON_UNIQUE, COLUMN_NAMEFROMINFORMATION_SCHEMA.STATISTICSWHERETABLE_SCHEMA= ? ANDTABLE_NAME= ?       [gitea user]  2020/03/04 06:45:24 .../xorm/session_raw.go:196:exec() [I] [SQL] ALTER TABLEuserADDtheme VARCHAR(30) DEFAULT '' NOT NULL ; - took: 3.750008ms  2020/03/04 06:45:24 routers/init.go:72:initDBEngine() [E] ORM engine initialization attempt #1/10 failed. Error: migrate: do migrate: Error 1118: Row size too large. The maximum row size for the used table      type, not counting BLOBs, is 8126. This includes storage overhead, check the manual. You have to change some columns to TEXT or BLOBs

@leetNightshade Can I suggest looking at these answers from SO?

@leetNightshade Can I suggest looking at these answers from SO?

"Variable 'innodb_file_format' is a read only variable." Those answers aren't helpful to someone who doesn't know what they're doing. Gitea created this problem, why aren't you guys solving it?

@leetNightshade what version of MySql do you run?

@leetNightshade I'm sorry you didn't find that link useful. Unfortunately that's a limitation of MySQL/MariaDB. AFAICT the problem you are facing is very common for databases created with old versions of those products. "Modern" versions of MySQL/MariaDB don't have this problem. What you need to do is to upgrade your engine and look for a way of converting your tables to a newer format.
I'd suggest you perform a backup and upgrade to the latest database engine version possible (e.g. MariaDB 10.4.12). You can then export your database with mysqldump, modify the backup as instructed here, and import it back (you'll need to update the script and change DB="" for DB="gitea" or whatever is the name of your database).

I'm running: mariadb  Ver 15.1 Distrib 10.4.12-MariaDB, for Linux (x86_64) using readline 5.1

@leetNightshade just try what guillep2k suggested and the try it again - would be nice if you can tell us the result

I'm running: mariadb Ver 15.1 Distrib 10.4.12-MariaDB, for Linux (x86_64) using readline 5.1

Nice! 😄. The original poster had one, but I didn't know what you had.
So, you need to stop Gitea (make sure no gitea processes keep running, even if it seems to have stopped), and export/transform/re-import the database (assuming your database name is gitea):
mysqldump -u root -p gitea > original.sql
sed 's:ROW_FORMAT=COMPACT:ROW_FORMAT=DYNAMIC:g' original.sql > modified.sql
mysql -u root -p gitea < modified.sql

You will be prompted for the root user's password two times. The original.sql script remains as a normal backup in case of trouble. Save a copy of original.sql for safekeeping purposes.

That did the trick, thank you @guillep2k !!

So we can finally close this issue - which is a fault with MySQL not Gitea - I guess we should put this little script in the FAQ.

It's not a fault with MySQL, it works as intended. It's a fault of how Gitea is using MySQL and a missing migration. If you need ROW_FORMAT=DYNAMIC specify it explicitly during table creation or at least please mention it in the documentation that dynamic is the expected row format. @zeripath @lunny

I added PR to set that for new installations and new tables. Existing tables won't be fixed and @guillep2k script should be used instead.

Awesome @lafriks
Do you think it would be possible to add the migration or just a message to prompt the user to the doctor command as @sapk suggested earlier?

Migration is not really possible to implement as it requires dumping database and reimport. It would be enough to add that to FAQ as @zeripath suggested imho

Using ALTER_TABLE the row format can be changed with MySQL built in commands without dumping and reimporting. See comment: https://github.com/go-gitea/gitea/issues/9347#issuecomment-565792626

# take a backup of the database

MariaDB> USE INFORMATION_SCHEMA;

# change row format
# this creates a list of SQL commands, run each manually
MariaDB> SELECT CONCAT("ALTER TABLE `", TABLE_SCHEMA,"`.`", TABLE_NAME, "` ROW_FORMAT=DYNAMIC;") AS MySQLCMD FROM TABLES WHERE TABLE_SCHEMA = "gitea";

# change collation, character set
# this creates a list of SQL commands, run each manually
MariaDB> SELECT CONCAT("ALTER TABLE `", TABLE_SCHEMA,"`.`", TABLE_NAME, "` CONVERT TO CHARACTER SET utf8mb4;") AS MySQLCMD FROM TABLES WHERE TABLE_SCHEMA = "gitea";

Thanks, @Whisprin, that worked and made my Gitea server usable again!

Or you could have just used gitea convert
https://docs.gitea.io/en-us/command-line/#convert

did not work for me, neither using gitea convert, nor using the proposal of @Whisprin. Using


pretrained models

Awesome works! When do you plan to release the pretrained models, I can't wait to try it. Will you upload training dataset?

Awesome works! When do you plan to release the pretrained models, I can't wait to try it. Will you upload training dataset?

The dataset cannot be made public due to some restrictions, but I have made the code public, and you can even make better datasets yourself using other text-based image editting methods.

Thanks for quick reply. When you publish pretrained models !

Thanks for quick reply. When you publish pretrained models !

Done～


0.3.0: missing git tag

According to https://pypi.org/project/hatch-nodejs-version/ latest version is 0.3.0 however here there is no corresponding git tag.
Not pushed local git changes to github? 🤔

Ah thanks! This was an oversight on my part during releasing. I've created a release from the appropriate commit!

Thank you 👍 😄


Hide seekbar on floating player

It's too easy to bump.
At the moment legacyYoutubePlayer loads from the AYT library - maybe try to implement a custom interface
https://github.com/PierfrancescoSoffritti/android-youtube-player#initialization

fixed in #304


Replace county coordinator for Kirks Counties

Currently I am county coordinator for the following counties.
MIL
OTH
OUC
OVB
OVF
They need to be assigned to someone else.
I suggest we assign them to CONManager

@geoffj-FUG how much is still relevant / need, as we have streamlined Overseas.

Story #1310 will combine OVB / OVF and resolve the problems created by the existence of 2 Codes.
The following Chapman Codes do not need Coordinators.
Currently I am county coordinator for the following counties.
MIL - Not a valid Chapman Code. Was it used for testing during development?
OTH - It is not a Code that is used by FreeCEN. UNK is the correct entry against a POB.
OUC - This is an 1841 Code related to POBs. It doers not need a Coordinator
OVB - See Story #1310
OVF - See Story #1310
OVB and OVF do not need Coordinators. They are related to POB entries.
Geoff


docs: Update Lido docs for tier 1 CSM integration + monitoring command

Fixes | Closes | Resolves #
Changes:

Update lido.mdx doc for tier 1 CSM integration
Create monitoring.mdx for the monitoring command
Create lido-exporter.mdx to describe the Lido Exporter.

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ X ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[ ] Yes
[ X ] No

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

Project coverage is 24.59%. Comparing base (fbd89c4) to head (a8cb158).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #453      +/-   ##
===========================================
- Coverage    25.31%   24.59%   -0.73%
===========================================
  Files          119      119
  Lines        18192    22000    +3808
===========================================
+ Hits          4605     5410     +805
- Misses       13059    16062    +3003
  Partials       528      528

Flag
Coverage Δ

unittests
24.59% <ø> (-0.73%)
:arrow_down:

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

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


next evolutionary phase  of reflux (cartiv)

Hey echosystem,
as we all can see, Reflux is dying (not so) slowly, and It's quite sad - first because I really like this library, and secondly because the flux alternatives are missing all the cool features of this lib.
Currently - there are no improvements/tools that this community created lately, and we barely maintain what we already have.
There are two reasons for that - @spoike, the creator  of Reflux is no longer actively working on that, and apart from that - Reflux is not fully reactive, and misses some cool features that has been standardized pursuant to Redux.
I would like to suggest here a solution - A totally new library, called Cartiv, that at its core is just Reflux (dependent on reflux-core), but with new terminology (React's terms - e.g setState in the store), reactivity and simplicity.
I really hope that this community will shift to Cartiv, and start a new hype, based on Reflux's great ideas and lack of boilerplate, and new tools and improvements will start to emerge from here.
Do you think it's possible to redeem the flux crown?

Hey @yonatanmn– happy to have this conversation, but can we move it over to reflux/discuss and link to the new issue from here?

Ok, But please don't close this as many people don't follow reflux/discuss

I'll leave it open for the week.  Anyone following this repo will be notified about this issue and will be linked to discuss if they visit it.  Others not following this repo will be directed to discuss for this type of discussion from the readme.

Well, you could just add a Reflux.Component class to reflux that extends React.Component for ES6 type usage. That's why it's dying...it doesn't integrate as well into ES6 in its current form.
I have one that I've been using where you just set this.store (for 1 reflux store) or this.stores (an Array of reflux stores) in the constructor (kinda like you set this.state in the constructor in ES6 style React) and the stores automatically update their own state into the React state on actions. I built it because a client wanted reflux instead of redux (which I had been using) but with this implementation I think reflux becomes more powerful than redux.
Is that something reflux people would be interested in?

@BryanGrezeszak Sounds cool and I know I would be interested, have you though about making a PR or forking it to let others maybe take a look and give feedback? @devinivy I havent read through all the issues but is anything currently being worked on for making this library ES6 compatible? Might be a good opportunity to get some collaboration on this.

@jamesmarrs - It's pretty light...it's probably more appropriate just to make a working jsfiddle to get a first idea at what people think: https://jsfiddle.net/nq44or92/ and a pastie that's a little easier to read: http://pastie.org/10821394
There's 4 main parts to that, code, most of it is just normal action/store/component setup:

an IIFE with the code that attaches the Reflux.Component

makes some actions (increment and decrement), nothing special, completely normal actions

makes the store, again nothing different except that you keep your state on this.state within the store (something that's idiomatic to concepts in ES6 React anyway)

The react class itself. Literally the only thing needed to hook it up to your store is setting this.store in the constructor. That's the important part, so I'll post it here:

class Counter extends Reflux.Component // <- Reflux.Component instead of React.Component
{
	constructor(props) {
		super(props);
		this.state = {foo:'bar'}; // <- stays usable, so normal state usage can still happen
		this.store = counterStore; // <- the only thing needed to tie the store into this component
	}

	render() {
		// `count` and `last` are both from the store, but they reflect on the component state
		return
Count {this.state.count}, Last Action: {this.state.last}, Foo: {this.state.foo}
;
	}
}

The properties of the store's state just setState themselves into react class' state each time the store triggers a change. Easy as can be. From within the React class you can just treat reading the store as if it's part of the React class state. If you don't want your React state flooded with all the properties in the store then you can just namespace the values in the store such as this.state.counterdata.counter instead of this.state.counter. And multiple stores can update their data into the component via setting this.stores = [store1, store2, etc]; in the constructor. And of course many components can be tied to the same store(s) and update from them.
Just having the store automatically reflected in the state might not be everyone's cup of tea (it lends itself to having a larger number of smaller stores so that each component isn't updating any time anything at all in the app changes...to me more small stores just seems like good practice anyway...but some might disagree). I'm not super well-versed in Reflux culture so I'm not sure if that would be an anti-pattern to you guys...but I find it easy as hang to use. It doesn't get much more simple to implement, and it's completely idiomatic with the type of syntax React is moving toward.

@BryanGrezeszak cool gist.
Actually it's quite close to my project. maybe we can join forces!

Making refluxjs work with ES6 classes is crucial.  See #225 #420.

Well...it seems to me that it doesn't get much better for supporting React ES6 than just setting this.store in the constructor. That's about as idiomatic with React ES6 and as easy to use as I can think of.
I've got some lesser well-tested (though seems to work well enough) stuff that I've used to make the stores more ES6 as well. I'm not sure how robust it is, as the stores have proven ridiculously unreliable to try to extend, so I had to go more of a wrapper route: https://jsfiddle.net/g1tetwyz/2/
That example has 1 Reflux.Component class (same concept as above) tied to 2 Reflux.Store classes that are each updating a separate part of the component's state, and are defined like this:
class CounterStore extends Reflux.Store
{
	constructor() {
		super();
		this.listenables = actions;
		this.state = {count:0};
	}

	onIncrement() {
		this.setState({count:this.state.count+1});
	}

	onDecrement() {
		this.setState({count:this.state.count-1});
	}
}

class LastStore extends Reflux.Store
{
	constructor() {
		super();
		this.listenables = actions;
		this.state = {last:'Initialized'};
	}

	onIncrement() {
		this.setState({last:'++'});
	}

	onDecrement() {
		this.setState({last:'--'});
	}
}

Again, that's about as idiomatic React ES6-ish as I can think to make defining stores.

@yonatanmn - I don't think I have time to make a full project out of it! But I can share what I've done for Reflux. That's a neat project though.

I did a fork/PR (with just the Reflux.Component part, not the other stuff I showed in the last comment). Check it out if you'd like.

@BryanGrezeszak Thanks for sharing some of the work you have done, much appreciated. Going to take a look at this soon

I also look forward to peeking your PR 👍 – thanks!


Log config pretty printer to fix #1075

Added pretty print helper function for log configuration.

LGTM

will fix #1075

LGTM


Add semicolons to fix graph_view.js minification

I published my Vault on a website that uses Cloudflare's minification and noticed the graph isn't rendering at all. Upon further inspection, I've realized that Cloudflare doesn't minify the graph_view.js script correctly. Adding a few semicolons fixed that issue for me :)

Thanks!