Limit travis build matrix

Only build master and transforms branches (PRs will still get run), and stop building on osx with node 4 and 0.10 (these seemed flaky in the last week of using them, and haven't added much value while greatly increasing required build time).

@sandersn @mhegazy

Definitely :+1: on limiting to master and transforms. Maybe we should hold off a couple of days on excluding the Mac builds until we can see if the first change helps enough on its own.

@sandersn I think I want to turn them off more for their flakiness than the just the running-less-build-ness.
The OSX 4 and 0.10 builds are somewhat regularly timing out/hanging for no apparent good reason, which is a little frustrating when you're trying to wait for a build to go green.

Oh, yeah. Very good point. Unless you want to go off on a journey to figure out why those builds are flaky. :)

(Which I think is premature since it sounds like it's probably a flakiness at the the Travis level.)

I'm going to merge this and then close out all the non-master/transforms branch builds (we have so many outstanding builds that we just do not care about right now).


Problem save additional Host

The problem
I have see the video tutoria in youtube

But if i write and save my domain (subdomain) in "additional Host" section the sistem print this error:
Unable to save add-in configuration, Invalid list for option 'additional_hosts' in Cloudflared (9074a9fa_cloudflared). Got {'external_hostname': '', 'additional_hosts': 'casamia.veneziaholidays.it'}
What version of Cloudflared has the issue?
5.1.15
What was the last working version of Cloudflared?
No response
What type of installation are you running?
Home Assistant OS
Add-on YAML Configuration
http:
  use_x_forwarded_for: true
  trusted_proxies:
    - <IP_ADDRESS>/24

Anything in the logs that might be useful for us?
No response
Steps to reproduce the issue
white the domain
Additional information
No response

hello brenner-tobias
thanks for the reply.
I tried exactly as you suggested (which is exactly as illustrated by the video tutorial)
it always gives me the same error.
I will try to read the guide you reported


Update #5

:D

I took out the variation and added a description. Is there something I missed?

LGTM


Performance issue

Testing a POC on my side I notice a big difference in performance between spark2 and spark3 same infrastructure same code, just using spark 2 vs spark3 and respective jdbc.
The gap can reach 10x bigger on spark3.
Could it be on the jdbc? any clue?

Hi
Thanks for reaching out.
I would like to investigate, what is causing this performance degradation.
Can you clarify the following questions?

What connector are you using for Spark 2 and Spark 3 and what is the version of the connector?
Are you connecting to the same database?
Does it happens during reading data from the SingleStore or writing?
Can you provide a reproducible example of the issue?

I doubt that JDBC is the root cause of this. The driver just reads and sends packets to the database. 10x performance degradation looks too big for so simple task. I suspect that the issue may be in the parallel read. In the newer version of the SingleStore Spark connector, this feature is completely redesigned.

Hello
Thanks for the reply, ofc here are the answer:

com.singlestore
singlestore-spark-connector_2.12
4.1.3-spark-3.3.0

vs

com.singlestore
singlestore-spark-connector_2.11
3.1.2-spark-2.4.7

    <dependency>
        <groupId>org.mariadb.jdbc</groupId>
        <artifactId>mariadb-java-client</artifactId>
        <version>2.7.7</version>
    </dependency>

same db nothing chg except version scala, spark and the connector
it happens while reading,and writing, theres always a significant gap, between 10sec
Something like:
val windowedDF = Window.partitionBy("col_y", "col_x")
val filtered = spark.read
.option("enableParallelRead", "automatic")
.option("parallelRead.Features", "readFromAggregators,readFromLeaves")
.format(DefaultSource.SINGLESTORE_SOURCE_NAME_SHORT)
.load(s"testdb.test1")
.filter(col("col_x") === "7")
.withColumn("max", functions.max(col("col_y_version")).over(windowedDF))
.filter(col("max") === col("col_y_version"))
.select("col_y", "col_x", "max")

Can you please check if SQL pushdown generates the same queries for both cases? (here is more info on how to do this https://github.com/memsql/singlestore-spark-connector#debugging-sql-pushdown )
Also, can you try changing parallelRead.Features to readFromLeaves? In this way, it will perform parallel read in the same way for both versions of the SingleStore Spark connector.

checking the pushDown by explain I can see the queries are the same
Indeed adding parallelRead.Features to readFromLeaves helped a lot, we still have a small gap between 1/2sec shame spark3 wont be better but at least not as big gap as before

Thanks for testing this.
I think readFromLeaves works here better than readFromAggregators because of two reasons:

readFromAggregators performs repartitioning of data on the Spark side just after the read
the number of leaves in your cluster may be bigger than the number of aggregators and because of it, throughput is higher for readFromLeaves

readFromAggregators is a default option because readFromLeaves doesn't work in SingleStore Managed Service.


Using spyder3 for 2 versions of python3

This is more of a question than an issue. I have two versions of python 3 on my Ubuntu 16.04 system. It has python 3.5 form the system and python 3.6 from Anaconda. I would like to be able to switch between them in spyder. However changing the python interpreter in Spyder doesn't work  because of https://github.com/spyder-ide/spyder/issues/4514.
The answer to that issue was that spyder was compiled under different versions of python3 and there is no binary compatibility. My solution is to run  the same copy of spyder from source using different versions of python and that works, but is there a more elegant way to do it? I am puzzled because if there is no binary compatibility then why the option of choosing different paython interpreter?
Thanks.

However changing the python interpreter in Spyder doesn't work because of #4514.

#4514 was about adding to PYTHONPATH the site-packages directory of another Python version. Are you doing that?

For example, I have spyder from anaconda, I go to Tools > Preferences and check use the following python interpreter instead of the default and put /usr/bin/python3. Then restart spyder , the kernel struggles to start and gives a bunch of import error. Same if I try the other way: install spyder via pip (python 3.5 from /usr/bin/python3 is the default) and  use the alternative from ~/anaconda3/bin/python.

the kernel struggles to start and gives a bunch of import error

Please post the error so we can help you.

Hi,  setting python interpreter to /usr/bin/python3 in spyder installed from anaconda, restart spyder and got these errors
An error ocurred while starting the kernel
Traceback (most recent call last):
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/spyder/utils/ipython/start_kernel.py", line 268, in
main()
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/spyder/utils/ipython/start_kernel.py", line 236, in main
from ipykernel.kernelapp import IPKernelApp
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/ipykernel/__init__.py", line 2, in
from .connect import *
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/ipykernel/connect.py", line 18, in
import jupyter_client
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/jupyter_client/__init__.py", line 4, in
from .connect import *
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/jupyter_client/connect.py", line 22, in
import zmq
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/__init__.py", line 34, in
from zmq import backend
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/__init__.py", line 40, in
reraise(*exc_info)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/utils/sixcerpt.py", line 34, in reraise
raise value
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/__init__.py", line 27, in
_ns = select_backend(first)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/select.py", line 26, in select_backend
mod = __import__(name, fromlist=public_api)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/cython/__init__.py", line 6, in
from . import (constants, error, message, context,
ImportError: cannot import name 'constants'
Traceback (most recent call last):
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/spyder/utils/ipython/start_kernel.py", line 268, in
main()
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/spyder/utils/ipython/start_kernel.py", line 236, in main
from ipykernel.kernelapp import IPKernelApp
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/ipykernel/__init__.py", line 2, in
from .connect import *
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/ipykernel/connect.py", line 18, in
import jupyter_client
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/jupyter_client/__init__.py", line 4, in
from .connect import *
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/jupyter_client/connect.py", line 22, in
import zmq
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/__init__.py", line 34, in
from zmq import backend
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/__init__.py", line 40, in
reraise(*exc_info)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/utils/sixcerpt.py", line 34, in reraise
raise value
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/__init__.py", line 27, in
_ns = select_backend(first)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/select.py", line 26, in select_backend
mod = __import__(name, fromlist=public_api)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/cython/__init__.py", line 6, in
from . import (constants, error, message, context,
ImportError: cannot import name 'constants'
Traceback (most recent call last):
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/spyder/utils/ipython/start_kernel.py", line 268, in
main()
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/spyder/utils/ipython/start_kernel.py", line 236, in main
from ipykernel.kernelapp import IPKernelApp
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/ipykernel/__init__.py", line 2, in
from .connect import *
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/ipykernel/connect.py", line 18, in
import jupyter_client
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/jupyter_client/__init__.py", line 4, in
from .connect import *
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/jupyter_client/connect.py", line 22, in
import zmq
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/__init__.py", line 34, in
from zmq import backend
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/__init__.py", line 40, in
reraise(*exc_info)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/utils/sixcerpt.py", line 34, in reraise
raise value
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/__init__.py", line 27, in
_ns = select_backend(first)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/select.py", line 26, in select_backend
mod = __import__(name, fromlist=public_api)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/cython/__init__.py", line 6, in
from . import (constants, error, message, context,
ImportError: cannot import name 'constants'
Traceback (most recent call last):
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/spyder/utils/ipython/start_kernel.py", line 268, in
main()
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/spyder/utils/ipython/start_kernel.py", line 236, in main
from ipykernel.kernelapp import IPKernelApp
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/ipykernel/__init__.py", line 2, in
from .connect import *
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/ipykernel/connect.py", line 18, in
import jupyter_client
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/jupyter_client/__init__.py", line 4, in
from .connect import *
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/jupyter_client/connect.py", line 22, in
import zmq
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/__init__.py", line 34, in
from zmq import backend
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/__init__.py", line 40, in
reraise(*exc_info)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/utils/sixcerpt.py", line 34, in reraise
raise value
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/__init__.py", line 27, in
_ns = select_backend(first)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/select.py", line 26, in select_backend
mod = __import__(name, fromlist=public_api)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/cython/__init__.py", line 6, in
from . import (constants, error, message, context,
ImportError: cannot import name 'constants'
Traceback (most recent call last):
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/spyder/utils/ipython/start_kernel.py", line 268, in
main()
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/spyder/utils/ipython/start_kernel.py", line 236, in main
from ipykernel.kernelapp import IPKernelApp
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/ipykernel/__init__.py", line 2, in
from .connect import *
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/ipykernel/connect.py", line 18, in
import jupyter_client
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/jupyter_client/__init__.py", line 4, in
from .connect import *
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/jupyter_client/connect.py", line 22, in
import zmq
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/__init__.py", line 34, in
from zmq import backend
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/__init__.py", line 40, in
reraise(*exc_info)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/utils/sixcerpt.py", line 34, in reraise
raise value
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/__init__.py", line 27, in
_ns = select_backend(first)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/select.py", line 26, in select_backend
mod = __import__(name, fromlist=public_api)
File "/home/bernard/anaconda3/lib/python3.6/site‑packages/zmq/backend/cython/__init__.py", line 6, in
from . import (constants, error, message, context,
ImportError: cannot import name 'constants'

Ok, thanks for the error report. We'll take a look at it in a future release.

So it seems that even when python interpreter is set to /usr/bin/python3 spyder(?) still tries to import site packages from anaconda.

@ccordoba12 FYI I was able to reproduce this, (attempting to use an already installed python 3.5, with IPython, cloudpickle and their deps successfully installed via pip) and I get the same error as the OP.


Update CI to use configured pgx version

Right now, plrust's CI installs cargo-pgx from pgx' git repo, off its develop branch.
Instead, CI should peek inside plrust/Cargo.toml and pluck out the pgx version we're using, and simply execute cargo install cargo-pgx --version $TheVersionFromCargoToml.
Assuming CI's existing caching features do what they claim, this will allow subsequent CI runs that all use the same pgx version to skip installing cargo-pgx again, possibly saving ~4 minutes per job.

Yeah good idea. As mentioned in discord, I am fairly certain installing cargo-pgx from the develop branch was a relic of the past. Now that things are solidifying a bit, we should do this.

Right now we run cargo install cargo-pgx ... --force.  We don't want --force anymore if we're going to expect cargo install to skip installing the same version.

Got it.

@eeeebbbbrrrr #182 has been merged in, so this issue can be closed


[Enum Request] Page 5870 "BOM Structure" - Make ShowBy an Enum

In order to generate a BOM Structure for an Item Variant, we have extended the ShowBy option with another option, "Item Variant". In later versions of Business Central, this is not possible, and requires copying the entire page to a new object to make changes.
Changing it to an Enum would allow us to extend the page. All line numbers are for the AL file from BC18.0. Suggested changes:
Line 290
-       ShowBy: Option Item,Assembly,Production;
+       ShowBy: Enum "BOM Structure Show By";

Enum:
enum id "BOM Structure Show By"
{
    Extensible = true;

    value(0; Item) { }
    value(1; Assembly) { }
    value(2; Production) { }
}

There needs to be a new public function where you can pass in the ShowBy-enum:
Line Number 314
    procedure SetShowBy(NewShowBy: Enum "BOM Structure Show By")
    begin
        ShowBy := NewShowBy;
    end;

To go along with this, the function RefreshPage should be updated to include an else-case:
Line Number 335
            ShowBy::Production:
                CalcBOMTree.GenerateTreeForProdLine(ProdOrderLine, Rec, 0);
            ShowBy::Assembly:
                CalcBOMTree.GenerateTreeForAsm(AsmHeader, Rec, 0);
+           else
+               OnRefreshPage(Item, ProdOrderLine, AsmHeader, Rec);
        end;

Please let me know if you have any questions to this, I'll be happy to answer them.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Helm repo 404

hi @odavid
seems after the latest merge, the helm repo stopped working and it is throwing 404
I tried to look around the repo but i cant find the source of the problem, can you take a look when you can?
thanks

I do not see gh-pages branch mentioned here.

Sorry, missed this :( - it seems the gh-pages branch disappeared...
Will need to investigate it


Under which license is this product provided?

Hello.
Under which license is this product provided?

Hello.
Thank you for telling me the repo has no license file!
I added MIT license file just now (in https://github.com/lo48576/docbook-xhtml5/commit/00f9f99b231036086930d0d5eba254b204dfc5b2).
If you want another option, feel free to tell me.

Thank you!


[OOB_5_4] Windows installer cannot install mbed.py

regarding Test 3, when I run mbed_installer_v033_cli_v100.exe and got following popup. I just want to skip Python installation, but mbed.py cannot be installed.
Is it a expected behavior?

@arekzaluski @MarceloSalazar Could you verify this issue and give your comment?

What version of python did you have installed? Was it 2.7+ or 3+? It should skip only python installation. Currently this check is checking only if python is installed on the system (no matter what version) and I believe that mbed-cli is compatible only with python 2.7+. I extended this check in new build version of installer.

@arekzaluski I pre-installed python 2.7 under C:\Program Files (x86) not the directory C:\Python27.

Thanks, I'll investigate it.

@soramame21 Can you try with the latest installer please?

@screamerbg, Sorry for such delayed reply. the windows installer is verified and works well. I will close this issue.

I tested on Windows 10 environment.


PATCH: Saving a space and removing it causes 409 crash

Description of the bug
Typing into a string field, removing the value, adding a space, removing that space (all while waiting for saving in between each step) seems to crash with a 409.
Steps To Reproduce
https://github.com/Altinn/app-frontend-react/assets/700139/3556b56c-90be-4907-a986-2a28fbd4aa65
Additional Information
No response

@olemartinorg do you have a simple way to remake the patch requests that causes this?

Yup, the simplest way is probably just to load up the ttd/frontend-test app and do the same thing as in the video (type 'a', remove it, type a space, remove that, etc) in the same field.

This might be relevant:

https://altinn.slack.com/archives/C02EJ9HKQA3/p1711007303959429?thread_ts=1710493123.111109&cid=C02EJ9HKQA3

The relevant field in the data model also has the XsdText() annotation on it, if I recall correctly. From the thread:

xsdText gjør slik at c# verdien i klassen ikke blir satt som nullable

This might be relevant:

https://altinn.slack.com/archives/C02EJ9HKQA3/p1711007303959429?thread_ts=1710493123.111109&cid=C02EJ9HKQA3

The relevant field in the data model also has the XsdText() annotation on it, if I recall correctly. From the thread:

xsdText gjør slik at c# verdien i klassen ikke blir satt som nullable

I can add two commits that highlights that when I remove "@xsdText": true and then generate the models from studio solves the issue specified in the slack thread.
Commit that deletes "@xsdText": true from schema.json: https://altinn.studio/repos/brg/test-aarsregnskap-ux/commit/6608d4ec72b2f06b76e04b2eec23ddcef6615b0a
The commit that generates the new classes from studio (example of decimal is at line 284): https://altinn.studio/repos/brg/test-aarsregnskap-ux/commit/74ad5c79ea3b33dc9e727caccc33d2a7d6ca783c

Tested frontend-test on 8.1.1, no longer crashes 👍


equal cost ordering

i dont know how to leave comments on github, so im using Issues tab
e.g. duration, speed, area
there order doesn't matter
i think it would be better if they were all numbered there same
e.g.
12 movspeed
13 duration/speed/area
16 max hp
or
12 movspd
13 duration
13 speed
13 area
16 max hp
or
12 movspd
13a duration
13b speed
13c area
16 max hp

Sure, I guess.


Upgrade App Engine to 1.8.9

My company would like to request that the App Engine libraries be upgraded to 1.8.9. Google fixed many bug between 1.8.0 and 1.8.9 and we would like to see those fixes also brought to AppScale. As is detailed in this post https://groups.google.com/forum/#!topic/appscale_community/ihbp8RSZ7hM  , 1.8.0 would not allow a Hibernate JPA connection to work with MySQL. Upgrading to AppEngine 1.8.1 using this fork fixed the issue: https://github.com/caseyoneill/appscale/tree/1.8.1-java-upgrade .

We understand that the AppScale team is working on many important enhancements and valuable features so we would like to volunteer to handle the grunt work of merging in the changes for each App Engine point release. I've already submitted a Pull Request to upgrade Java to 1.8.1 and would be to do the same for Python as well.

Thanks @caseyoneill awesome work. We will review the pull soon. Feel free to submit also the python branch and we'll review it.


Which dataset do you use to train your own arcface?

Thanks for your great work! I want to know how did you train your arcface. It was trained only for this task or could be used as a general face recognition model!

And another question, Have you tried using this model in the video ? Can this model perform well and stably？


Keyboard navigation

Interaction flow:

Tab to focus on the editor
Press Enter to start editing (and you will have an extra \n at the beginning`)
When done, hit Escape to continue tab/ shift+tab to other elements
Next time when your focus is back on the editor again, you can directly start editing; you no longer need to press Enter.

On the other hand, we sacrifice part of the functionality of undo: you can still reset/undo the reset, but you will be able to edit the code during the three second interval, and the change will be recorded (for you to further reset/undo).
The way React handle states is a mess...

If you are curious about why you need an unintuitive Enter after first tabbing on the editor to start editing, that is due to an internal bug with use-editable:

https://github.com/FormidableLabs/use-editable/issues/16
https://github.com/FormidableLabs/use-editable/issues/20

With this PR #47 can be closed.

New interaction flow with the requested changes:

Tab to focus on the editor
Press Enter to start editing

You will see an extra \n at the beginning
However, once you start typing, the extra \n is gone

When done, hit Escape to continue tab/ shift+tab to other elements
Next time when your focus is back on the editor again, you can directly start editing; you no longer need to press Enter.

Cursor position TODO in #113

@pelikhan Can I merge this in? It also fixes a rather critical bug (non-playground editors no longer editable)

Always good to add a description to a PR once it is ready for review. (Future you will thank present you)


[SPARK-48120] Enable autolink to SPARK jira issue

What changes were proposed in this pull request?
This PR aims to enable autolink feature to SPARK jira issue like Apache Spark repository.
Why are the changes needed?
Since we share the same JIRA project name, we need to link it.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual review.
Was this patch authored or co-authored using generative AI tooling?
No.

Could you review this, @jiangzho ?

Thank you for review and approval, @jiangzho !


Add npm start and dev scripts

Dockerfile expects npm start to be defined

Can you check it with react native pls? gulp ios

@steida I have absolutely no idea how to make RN work for now. I will try to get some info but running gulp ios (even on the master branch) gives me an error:
🚀  ~/code/fork/este|master✓
⤷  gulp ios
[21:35:47] Requiring external module babel-register
[21:35:48] Using gulpfile ~/code/fork/este/gulpfile.babel.js
[21:35:48] Starting 'native'...
[21:35:48] Finished 'native' after 118 ms
[21:35:48] Starting 'ios'...
[bg] Starting react-native run-ios
[21:35:48] Finished 'ios' after 5.52 ms
events.js:154
      throw er; // Unhandled 'error' event
      ^

Error: spawn react-native ENOENT
    at exports._errnoException (util.js:856:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:178:32)
    at onErrorNT (internal/child_process.js:344:16)
    at nextTickCallbackWith2Args (node.js:474:9)
    at process._tickCallback (node.js:388:17)
    at Function.Module.runMain (module.js:449:11)
    at startup (node.js:139:18)
    at node.js:999:3

Do you an idea why build is failing?

Closing for now.


[QUAD] Enhancement: Use task frame data if specified for collect context entities

Nice!
This should be solved with this PR that I just made a day ago. If not, we should fix that PR. @iLLiCiTiT that should essentially be the same, but then using AYON code, right?
Closing this in favor of that PR in AYON.


Added distinct operator and tests to cover its logic

#21

Fantastic, thanks serg!
Just one comment, I personally tend to stay away from objc_sync_enter / objc_sync_exit. I didn't even know they worked with non-ObjC objects!

Hi, @fpillet, do you think I would be better to use NSLock instead?

It would be cleaner I think, maybe ask @kzaher what he thinks about it?

You don't need locks here
    public func distinct() -> Observable<Element> {
 +        var cache = [Element]()
 +        return flatMap { element -> Observable<Element> in
 +            objc_sync_enter(self); defer {objc_sync_exit(self)}
 +            if cache.contains(element) {
 +                return Observable<Element>.empty()
 +            } else {
 +                cache.append(element)
 +                return Observable<Element>.just(element)
 +            }
 +        }
 +    }

There is no concurrency since this is a single sequence.


chore(main): release release-please-playground 1.0.0

:robot: I have created a release beep boop
1.0.0 (2021-11-23)
Features

initial nodejs root package (ad92d06)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/chingor13/release-please-playground/releases/tag/release-please-playground-v1.0.0 :sunflower:


Proposed 1.4.0-b6

If merged this PR will:
close: #3104
close: #3096
close: #3095
close: #3094
close: #3086
close: #3083
close : #3055

Codecov Report

Merging #3115 into develop will increase coverage by 0.12%.
The diff coverage is 42.1%.

@@             Coverage Diff             @@
##           develop    #3115      +/-   ##
===========================================
+ Coverage    68.71%   68.83%   +0.12%
===========================================
  Files          686      686
  Lines        55353    55393      +40
===========================================
+ Hits         38034    38129      +95
+ Misses       17319    17264      -55

Impacted Files
Coverage Δ

src/ripple/protocol/impl/BuildInfo.cpp
79.16% <ø> (ø)
:arrow_up:

src/ripple/overlay/PeerReservationTable.h
23.07% <ø> (ø)
:arrow_up:

src/ripple/nodestore/impl/Shard.cpp
0% <ø> (ø)
:arrow_up:

src/ripple/server/WSSession.h
86.36% <ø> (ø)
:arrow_up:

src/ripple/beast/rfc2616.h
80.26% <ø> (ø)
:arrow_up:

src/ripple/nodestore/impl/DatabaseShardImp.cpp
0.63% <0%> (ø)
:arrow_up:

src/ripple/overlay/impl/PeerImp.cpp
0.06% <0%> (-0.01%)
:arrow_down:

src/ripple/overlay/impl/OverlayImpl.cpp
28.85% <0%> (ø)
:arrow_up:

src/ripple/overlay/impl/PeerImp.h
0% <0%> (ø)
:arrow_up:

src/ripple/shamap/impl/SHAMapTreeNode.cpp
79.53% <100%> (ø)
:arrow_up:

... and 20 more

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 b53fda1...fccb7e1. Read the comment docs.


Validate strings and arrays for size

Signed-off-by: Tyler Weaver<EMAIL_ADDRESS>

I am getting this error when I run the test
terminate called after throwing an instance of 'rclcpp::exceptions::InvalidParameterValueException'
  what():  Invalid value set during initialization for parameter 'gravity_compensation.CoG.pos': Invalid length '3' for parameter 'gravity_compensation.CoG.pos'. Required greater than: 3
[ros2run]: Aborted

Thank you for running the test, I fixed the logic error.
I should turn running that node into an integration test.


Capital letter key bindings would be nice

For example, "g" and "G" are not treated as separate keys (I guess support for shift-key would solve this as well).

Yes supporting the shift key would do it.
Good idea. I've added it.

How much time doe these kind of changes take to get to the web store btw?

For the firefox one, the first time I submitted the addon I think it took three months to be reviewed. A 2-3 weeks for Opera and almost no time for Chrome (I don't think the review process is quite the same for Chrome, after you pay a fee you can host your extension very quickly).
Once your addon has been accepted, updating it is very quick.


Update-pipeline should not reset ScaleDownThreshold

From a user:
When you update a pipeline, we reset the scaleDownThreshold timer. This can potentially cause a pipeline to request significant resources even when there are no pending jobs to be processed.

Fxied by #2206


<EMAIL_ADDRESS>SIGSEGV errors when testing with GCP

Currently testing the workflow proposed in https://github.com/diggerhq/digger-gcp-lock-demo.
One difference is that I have configured Workload identity federation to authenticate with Google using OIDC.
Also have made a change in the Use gcloud CLI by listing the content of the bucket to prove that service account has storage.buckets.get permission on it.
Workflow:
name: CI

on:
  push:
    branches: [ "master" ]
  pull_request:
    branches: [ "master" ]
    types: [ closed, opened, synchronize, reopened ]
  issue_comment:
    types: [created]
    if: contains(github.event.comment.body, 'digger')
  workflow_dispatch:

jobs:
  build:
    runs-on: ubuntu-latest

    permissions:
      contents: read
      id-token: write
      pull-requests: read

    env:
      SERVICE_ACCOUNT<EMAIL_ADDRESS>      GOOGLE_STORAGE_BUCKET: terraform-digger-gha-pr-locks-814b

    steps:
      - uses: actions/checkout@v3

      - name: Checkout Pull Request
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        run: |
          PR_URL="${{ github.event.issue.pull_request.url }}"
          PR_NUM=${PR_URL##*/}
          echo "Checking out from PR #$PR_NUM based on URL: $PR_URL"
          hub pr checkout $PR_NUM
        if: github.event_name == 'issue_comment'

      - id: auth
        uses: google-github-actions/auth@v1
        with:
          token_format: access_token
          workload_identity_provider: ${{ secrets.GCP__GHA__WORKLOAD_IDENTITY_PROVIDER }}
          service_account: ${{ env.SERVICE_ACCOUNT }}

      - name: Set up Cloud SDK
        uses: google-github-actions/setup-gcloud@v1

      - name: Use gcloud CLI
        run: |
          gcloud info
          gsutil ls gs://${{ env.GOOGLE_STORAGE_BUCKET }}

      - name: digger tfrun
        uses<EMAIL_ADDRESS>        env:
          LOCK_PROVIDER: gcp
          GITHUB_CONTEXT: ${{ toJson(github) }}
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Output:
...
Run gcloud info
Installation Properties: [/opt/hostedtoolcache/gcloud/424.0.0/x64/properties]
User Config Directory: [/home/runner/.config/gcloud]
Active Configuration Name: [default]
Active Configuration Path: [/home/runner/.config/gcloud/configurations/config_default]

Account<EMAIL_ADDRESS>Project: [prj-c-artifacts-62d0]

Current Properties:
  [auth]
    credential_file_override: [/home/runner/work/digger-test/digger-test/gha-creds-43e0236a61a1ec40.json] (environment)
  [core]
    account<EMAIL_ADDRESS>(property file)
    disable_usage_reporting: [True] (property file)
    project: [prj-c-artifacts-62d0] (environment)
  [metrics]
    environment: [github-actions-setup-gcloud] (environment)
    environment_version: [1.1.0] (environment)

Logs Directory: [/home/runner/.config/gcloud/logs]
Last Log File: [/home/runner/.config/gcloud/logs/2023.04.04/14.49.[55](https://github.com/nuecho/digger-test/actions/runs/4609142469/jobs/8145864767#step:6:56).0233[65](https://github.com/nuecho/digger-test/actions/runs/4609142469/jobs/8145864767#step:6:66).log]

git: [git version 2.40.0]
ssh: [OpenSSH_8.9p1 Ubuntu-3ubuntu0.1, OpenSSL 3.0.2 15 Mar 2022]

gs://terraform-digger-gha-pr-locks-814b/test.txt

...

Run<EMAIL_ADDRESS>Run curl -sL [https://github.com/diggerhq/digger/releases/download/${actionref}/digger-Linux-X64](https://github.com/diggerhq/digger/releases/download/$%7Bactionref%7D/digger-Linux-X64) -o digger
Digger config read successfully
Lock has been created successfully
GitHub context parsed successfully
GitHub event processed successfully
GitHub event converted to commands successfully
Lock nuecho/digger-test#default
failed to get bucket attributes: googleapi: Error 403<EMAIL_ADDRESS>does not have storage.buckets.get access to the Google Cloud Storage bucket. Permission 'storage.buckets.get' denied on resource (or it may not exist)., forbidden
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0xcfef04]

goroutine 1 [running]:
digger/pkg/gcp.(*GoogleStorageLock).Lock(0xc00007f4e0, 0xc0007841a0?, {0xc0007841a0, 0x1a})
	/home/runner/work/digger/digger/pkg/gcp/gcp_lock.go:41 +0x384
digger/pkg/utils.(*ProjectLockImpl).Lock(0xc00070bcc0, {0xc0007841a0, 0x1a}, 0x1)
	/home/runner/work/digger/digger/pkg/utils/locking.go:60 +0x369
digger/pkg/digger.DiggerExecutor.Plan({{0x0, 0x0}, {0x100b75a, 0x7}, {0xc0000387f8, 0x6}, {0x100b75a, 0x7}, {0x1007b20, 0x1}, ...}, ...)
	/home/runner/work/digger/digger/pkg/digger/digger.go:233 +0x253
digger/pkg/digger.RunCommandsPerProject({0xc00070bc70?, 0x1, 0xc0001181e0?}, {0xc0000387f8, 0x6}, {0xc0000387ff, 0xb}, {0xc0001182a0, 0xc}, 0x1, ...)
	/home/runner/work/digger/digger/pkg/digger/digger.go:76 +0x605
main.main()
	/home/runner/work/digger/digger/cmd/digger/main.go:67 +0x709
Error: Process completed with exit code 2.

Hello! Thank you for filing the issue, I'm looking into it now

I'm not so familiar with GCP identity federation so I need to dig deeper into how it works

@motatoes if that could help I could share with you  a quick terraform config to setup Identity Federation. Or the generated configuration in https://console.cloud.google.com/iam-admin/workload-identity-pools/

hey @fleroux514 that would be really helpful, I'm setting up a demo repo to test it out with OICD

Terraform config:

creates a service account
creates an identity pool, along with an assigned pool provider for Github
The current service account IAM binding is configured so that the Token will be granted access only when the service account is used from this repo's Github Action workflow: https://github.com/fleroux514/test-oidc

resource "google_service_account" "fleroux" {
  project      = module.personal_fleroux.project_id
  account_id   = "sa-fleroux"
  display_name = "fleroux Development Service Account"
}

resource "google_iam_workload_identity_pool" "fleroux_pool" {
  project                   = module.personal_fleroux.project_id
  provider                  = google-beta
  workload_identity_pool_id = "github-pool"
}

resource "google_iam_workload_identity_pool_provider" "fleroux_github_pool_provider" {
  project                            = module.personal_fleroux.project_id
  provider                           = google-beta
  workload_identity_pool_id          = google_iam_workload_identity_pool.fleroux_pool.workload_identity_pool_id
  workload_identity_pool_provider_id = "github-pool-provider"
  description                        = "OIDC identity pool provider for Github CI/CD"
  disabled                           = false
  attribute_mapping = {
    "google.subject"             = "'github::' + assertion.sub"
    "attribute.repository_owner" = "assertion.repository_owner"
    "attribute.repository"       = "assertion.repository"
    "attribute.repository_ref"   = "assertion.ref.contains('refs/tags/') ? assertion.repository + ':refs/tags' : assertion.repository + ':' + assertion.ref"
  }
  attribute_condition = "attribute.repository_owner==\"fleroux514\""
  oidc {
    issuer_uri = "https://token.actions.githubusercontent.com"
  }
}

resource "google_service_account_iam_binding" "fleroux_iam_binding" {
  service_account_id = google_service_account.fleroux.name
  role               = "roles/iam.workloadIdentityUser"

  members = [
    "principalSet://iam.googleapis.com/projects/${module.personal_fleroux.project_number}/locations/global/workloadIdentityPools/${google_iam_workload_identity_pool.fleroux_pool.workload_identity_pool_id}/attribute.repository/fleroux514/test-oidc"
  ]
}

Generated Identity pool config:

Generated Service account config:

Hey @fleroux514  I managed to get it working after assigning custom role to the service account with these permissions for access to the bucket:

Perhaps your missing the storage.objects.* permissions? Here is a sample repository inspired by your action format: https://github.com/diggerhq/digger-gcp-ocid-demo
I had to increase the permissions specified in the actions.yml since it was lacking commenting permisisons. This is really helpful for us to document how OICD will work with digger so thank you for this filing! Let me know if this fixes it for you!

Hey @motatoes ,
Yes that fixed it for me.  I think the GCP demo should explicit the fact that storage.buckets.* and storage.objects.* permissions are required, and also document the GOOGLE_STORAGE_BUCKET variable.
Makes sense that Identity Federation should not have been related since its just another way of identifying.
Closing this one.  Thanks :)

@mpotatoes

When I merged the workflow (above), Github actions first ran on the master branch and the digger GHA failed with
Digger config read successfully
Lock has been created successfully
failed to parse GitHub context. error parsing GitHub context JSON: unknown GitHub event: push
Error: Process completed with exit code 5.

Than it ran on my feature branch and I see it deleted the lock successfully:
Unlock nuecho/digger-test#default
Project unlocked
Commands executed successfully

So I guess we don't really want to run the action on push event to the main branch and this snippet suggested  in gcp-demo workflow is not really required (and supported) by the github Action currently, since the lock is released as soon as the pull request is closed anyways?
on:
  push:
    branches: [ "master" ]

@motatoes,
I'm currently testing digger for my company but currently stumbling on a few issues and it might be easier to discuss on Slack instead of PR comments if possible.
I saw https://diggertalk.slack.com/signup#/domain-signup  but not sure how to "Contact the workspace administrator at Digger for an invitation.".
Can you help?

hey @fleroux514 please try https://join.slack.com/t/diggertalk/shared_invite/zt-yx6rua03-6z~g~_RF3y5LTAK2Bu_yOA
sorry maybe the other link expired!!


Add Subscription Integration with application

Not sure what this issue means since there is no description. Closing.


Changing the size and color of each shape

Maybe it could be added?
this is the options that I would provide:

increase/decrease (size)

active / inactive
for each particle differently
for all particles the same:

increase
decrease
random (increase || decrease)

duration

change of color:

array of colors
for example [#fee, #fcc, #a00, #fee]
the easeIn || easeInOut option
speed

And one more thing, ... automate the min version with grunt...

Increase / decrease size + Array of colors added in the 2.0.0 version ;)
https://github.com/VincentGarreau/particles.js/releases/tag/2.0.0

dude you are the best. words can not describe. thanks a lot man

This is helping me make my beginner portfolio look slick! Thanks a lot.


Allow users to use custom local logos form channels/apps

As a user that primarily watches local channels on TV I would like to assign custom local logos for some channels. I've already tried to download a new batch using Jaruba's logo-channel (https://github.com/jaruba/channel-logos) but despite having success in downloading the logos, I found out that changing logo_paths.json on the custom component does nothing since it's linked to the live version on github.
I'll try to modify my version (as best as I can since i'm a designer with little knowledge in development) to fetch from the local logo_paths.json but I think it would be great to check. A local file to override/fetch logos as this will give users a better control.
Thanks for the great work @ollo69

Local logo is implemented with last release. You can create local logo and copy them in the remote subfolder (see readme options section for more details). Please try the feature and let me know if we can close this FR.

Hi @ollo69,
Thank you. Can you clarify how this is supposed to work? In here you mention the remote folder, but ReadMe says
I did some testing with Netflix (org.tizen.netflix-app):

netflix.png (original): works
org.tizen.netflix-app.png : doesn't work
orgtizennetflixapp.png : doesn't work
netflix-app.png: doesn't work
netflixapp.png: doesn't work

For Prime Video (org.tizen.primevideo):

primevideo.png (original): doesn't work
orgtizenprimevideo.png: works

For YouTube (9Ur5IzDKqV.TizenYouTube):

youtube.png (original): works
other name variations don't work

But for Plex (kIciSQlYEM.plex), uploading a new (400x400px, png) image to the static folder with the rest:

kIciSQlYEM.plex.png: doesn't work
kIciSQlYEMplex.png: doesn't work
plex.png: doesn't work

Can you clarify the folder and the name logic?
Thank you!!

Name remote is my mistake, don't know why I wrote this. Right folder name is static.
Going to fix case sensitive search, lowercase file name requirement and other little issue on next release.

Awesome! Thanks a lot, my TV is currently out of service, but I'll test as soon as it's back on the job. It works for apps and channels, right? For an example, if I have a channel here called "Channel Television" I'll upload an image named "channeltelevision.png" for it to be replaced, right?

Yes, that's the logic

I did not test this yet but as I see more people did and it seem to work. I'm closing the issue since I don't know for how long I'll be battling Samsung support with my broken TV.
Thanks @ollo69 for the great feature :)


Runtime error while running example/application_object_detection.py

Describe the bug
While running application_object_detection.py, I encountered an error. I have attached the error message
To Reproduce

Download corresponding images from the internet;
Run "python example/application_object_detection.py"

Expected behavior
An error message will be given.
Screenshots

System information (please complete the following information):

OS:
Python version: 3.8.0
ART: 1.8.0
Torch: 1.9.1+cu102
Torchvisio: 0.10.1+cu102

Hi @shuoli90 Thank you very much for reporting this issue. We are aware of it and it is caused by torchvision not supporting gradients at the inputs in versions 1.8 and 1.9 of the pretrained object detection models. PyTorchObjectDetector contains a warning for 1.8, but not yet 1.9 which we'll update now. Btw, we have successfully contributed a solution to torchvision, which should be part of the next release of torchvision #4327.

Just to follow up, currently, is there any way that I can use PyTorchObjectDetector?


Dynamic publicPath in extractor and client side JS chunk loading

💬 Questions and Help
#269 added support for publicPath to the extractor, so server side rendering works as expected, but if I try to load more bundles on the client side, this isn't taken into account and the hardcoded path is used instead.
I'm building a product that will be split up into multiple single page applications, separated by routes. I've got an nginx proxy that directs traffic between these applications.

/app1/ -> goes into standalone SSRed+Codesplitted (on APP routes) SPA called app1
/app2/ -> goes into standalone SSRed+Codesplitted (on APP routes) SPA called app2

I've set the publicPath so SSRing produces the correct link/script tags, but navigating from
/app1/route1/ to /app1/route2/ loads more bundles and these seem to be ignoring the publicPath option I've set on the extractor.
Is this expected behavior? Should I simply hardcode the /app1/ and /app2/ publicPaths in webpack build time?

Hello @peeter-tomberg, unfortunately I don't use the publicPath feature, so I am not sure of the right behaviour, I need to investigate it before. Any help is welcome!

Loadable is only 2.29kb, and probably even less with tree shaking, so no I think it is not bigger than react-loadable. React-loadable does not expose ES modules so tree shaking is not effective.

loadable-component are working on a server side
on the client side usage of import statement would be handled by your bundler (webpack) in the same way it would be handled by React.lazy.
server and client side are working differently.

Correct, the issue was I need to configure it in my bundler as well, this comment worked wonders: https://github.com/webpack/webpack/issues/443#issuecomment-54113862
Basically I have to write the dynamic path as a global variable inside my HTML template, then inside my client side entry point at the top define that and it works.


cannot import name 'download_url_to_file' from 'torch.hub

https://github.com/open-mmlab/mmdetection/issues/7417  The same issue in mmdetection may help.

i solved this error by removing underscore_ from _download_url_to_file function. So it should be download_url_to_file.
File location is /anaconda3/envs/(your env name)/lib/python3.7/site-packages/torch open hub.py


Add tests for BaseComponent and FieldComponent methods

Make sure BaseComponent and FieldComponent methods methods are well tested.
Note
It seems that method_errors? only works in Rails 6.0

Resolved in #71


Create gritacademy.txt

https://gritacademy.se/webbutvecklare-inriktning-e-handel/

@hapidjus First stage of verification passed, official domain is gritacademy.se as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@hapidjus Thank you for your patience while waiting for a review and processing your request. Pull request merged.


[Navigator.NavigationBar] Update to 0.15.0,the previous route navigationBar's button event can be triggered

if previous page :

current page:

click here ,the previous page button

will be click

I had the same but fixed it by rendering an empty <View /> in my routeMapper if no button was to be shown

@ashleydw I know that.But how to do this now?

it will be affected no matter use the left 、right or title
and Android‘s effect is different to iOS’s

Actually it was happening in earlier versions also. Related #3964

I have noticed same issue with hidden buttons and NavigatorNavigationBar is creating empty views which block clicks in the nav area. Returning null when no button is required fixed the hidden clicks

I have found this to be a side effect of a change starting in v0.15.0 to how view hit testing is done when opacity=0, in this commit:
77272d5a9193762b8668c99d54f5bb9a3e423a9e
I am currently working around this by replacing code in NavigatorNavigationBar.js, adding the following prop setting to the View rendered in _getComponent:
    rendered = (
      <View
        ref={(ref) => {
          this._components[componentName] = this._components[componentName].set(route, ref);
        }}
        pointerEvents={
          route === this.props.navigator.navigationContext.currentRoute ? 'auto' : 'none'
        }
        style={initialStage[componentName]}>
        {content}
      </View>
    );

Also there was just an update to NavigatorNavigationBar.js in master db69004300faf1d850f8127a57859d88205450a7 an hour ago, in this same spot in the code but it does not fix the issue for me.  I am using jumpTo to navigate to routes that are all rendered when the App is initialized.  I also have custom RightButtons per-route.

Ideally the NavBar should hide the button once faded out rather than build up 0 opacity views.
Another solution is amending NavigatorNavigationBarStylesIOS so the RightButton does not change position when transitioning the view to the left.
Stages.Left.RightButton sends the button to a left position at 1/3 Screen Width. By removing the 'Left' style value the button fades out in the right and is not blocking the middle of the nav bar at 0 opacity. I can't see a difference visually

The fix by @djk4096 works for me!

@djk4096 I think this was fixed by https://github.com/facebook/react-native/pull/4786 which hopefully should come in next react-native release

@gre Thanks. Will test and close.

Actually I think i've misread what was the issue about, because there is the idea to check the currentRoute in this workaround, maybe the bug is still there actually.
but in @djk4096 workaround, I think we need to use "box-none" instead of "auto" for the reason explained in https://github.com/facebook/react-native/pull/4786

The bug is still there after #4786, but as you say will be fixed by using @djk4096 workaround with "box-none".
pointerEvents={
  route === this.props.navigator.navigationContext.currentRoute ? 'box-none' : 'none'
}

yes :)

thanks

@gre After doing more testing today, I have to say that my workaround is dependent on the NavigationBar being rendered every time a new route is focused (say, by setting state on the main component/parent of the Navigator.)  So, the comparison to the currentRoute was depending on the currentRoute actually changing there...
Otherwise, the left/right buttons are only rendered once, when the nav bar is first rendered.
This means that the workaround would really break things for all cases where re-rendering was not happening... and from further testing, I don't feel that https://github.com/facebook/react-native/pull/4786 alone fixes this case either.

Here's the latest solution that adds the correct 'box-none' to the pointerEvents prop thanks to @gre and @samuelkraft:
pointerEvents={
  route === this.props.navigator.navigationContext.currentRoute ? 'box-none' : 'none'
}

This sets pointerEvents to work correctly as any component’s view is rendered.
After my previous comment, I realized we need to ensure the pointerEvents are updated whenever the nav bar components don't get re-rendered as a result of a transition.
A way to do this would be for the Navigator to fix up the pointerEvents on nav bar components, once a transition is completed.
A function for this, potentially to be added into NavigatorNavigationBar.js:
updatePointerEvents: function(fromIndex, toIndex) {
  var routeFrom = this.props.navState.routeStack[fromIndex];
  var routeTo = this.props.navState.routeStack[toIndex];

  COMPONENT_NAMES.forEach(function (componentName) {
    var map = this._components[componentName];
    var componentFrom = map.get(routeFrom);
    var componentTo = map.get(routeTo);
    componentFrom && componentFrom.setNativeProps({ pointerEvents:'none' });
    componentTo && componentTo.setNativeProps({ pointerEvents:'box-none' });
  }, this);
},

It looks like it should be called in Navigator.js within the _completeTransition function, passing the source and destination indexes for the scenes that were just transitioned between.
An acceptable place could be immediately before the line where this.state.transitionFromIndex gets set to null:
this._navBar && this._navBar.updatePointerEvents(this.state.transitionFromIndex, this.state.presentedIndex);
this.state.transitionFromIndex = null;

I have tested this a bit, it potentially addresses this additional issue without obvious side effects.

Hi, is this problem fixed yet? I'm still getting it in 0.18.1

@djk4096 for ping :) thank you

@ncnlinh Hi, yes it is still an issue last I checked.  I have had great success using the patches described in my last message as a workaround to continue to use the Nav Bar.

@djk4096 ok thanks, then I guess I will try putting it in my app and see how it goes. :+1: thank you

@djk4096 the solution fixes the problem for me. Thanks a lot! Why is this patch not applied and submitted?

The bug had fixed in new version for a long time.I am so sorry that I forget to close the issue and response

@everfire130 that's not true. I still have this in 0.18.1. @ncnlinh also reported it.
I don't think it's helpful to close an active, horribly annoying bug.

@everfire130 I agree with @israelidanny, It still occurs. Fixing inside the module is only a temporary patch.

@everfire130 This issue is still occurring in 0.18.1 what version has this been fixed in?

I can confirm I'm still having this issue with version 0.20.

@israelidanny @ncnlinh @DaleJefferson @theosherry Right!I haven't tested completely.The issue is still occurring.The solution is mentioned above.

I just encountered this today. Will look into this later today.

Same as @chirag04, I just encountered this bug today with<EMAIL_ADDRESS>

The bug is still in 0.23.1, @djk4096 can you make a PR?

Having this issue too on 0.22.2.
I'm doing a push from the navigation bar button.

As far as I can tell this issue has become a combination of several not-quite-the-same issues which makes it hard to proceed. I am going to close this issue. If people have a bug similar to this one and you can reproduce it on the most recent RN release, feel free to reopen a new issue.


feat(jobs): Add wrapper job to start QueryService rebuild

Not sure if this should part of this script, but there is one thing missing: in certain scenarios, whoever runs this script might want to delete all jobs that exist in the qs-jobs namespace as otherwise we'd just apply a spec that is already there and might also already have been finished. I.e.: jobs inserted that way are not guaranteed to run unless we a. clean the namespace or b. change the naming mechanism (right now, the job name is a hash of wiki domain + entity ids for that job)

I feel like this script could perhaps warn if the namespace is not empty and ask for confirmation to continue explaining that this is the case?

I feel like this script could perhaps warn if the namespace is not empty and ask for confirmation to continue explaining that this is the case?

Sounds reasonable, yes?


Audio is not getting recorded when using camera as input

i am able to record from camera and convert it to dash but audio is not getting recorded

If you are using Windows:
capture = ffmpeg_streaming.input('CAMERA NAME OR SCREEN NAME', audio='Microphone name here', capture=True)

If you browse the source code, you can find the answer to these types of questions.


Move project files from /app/assets/stylesheets to /core

Simular functionality to https://github.com/thoughtbot/bourbon/commit/c76215be98f2fa3ef9c5ed2174765a316c96747b

The Gempath does not seem to resolve correctly.

Fixed in https://github.com/whmii/cask/releases/tag/v0.3.2


explore using bare app imports for reusability

For both routes and components, it could be interesting to have an option to import them from another route or component for reusability without using relative imports, which are brittle.
Consider the following route in e.g. routes/base/add-head.js
// PROPOSED: not an actual route: do NOT map requests to `/base/add-head` to this
export const virtual = true;

export default route => ({
  head() {
    // return stuff like headers
  },
  // add all HTTP verbs from route (this could contain an overriding `head` verb)
  ...route,
});

And then in another route:
import base from "app/routes/base/add-head";

export default base({
  get() { ... },
});

This would allow any route to add a HEAD verb to it.
The same could be done with components, to allow reusing base components from anywhere within the components directory.
In itself, the app identifier means nothing (unless you're using the npm package app). But with the new build system, where we put everything inside build, we could add a package.json file there that defines
{
  "name": "app",
  "exports": {
    "./routes/*": "./routes/*.js",
    "./components/*": "./server/components/*.js"
  }
}

For the client-bundling, we'd need to pass in a dynamic tsconfig with paths to esbuild.

closing in favour of the more general https://github.com/primatejs/primate/issues/159


Draft Email Notification about the Gold and Gold DR Change

Email about the New UI change for the reference:

Dear Cloud services community,
I hope this email finds you well. We're happy to share some exciting news with you—there's an update to the Platform Product Registry user interface!
What to expect:
Starting today, you'll notice a fresh new look for the Platform Product Registry. Rest assured, your data will remain intact. The update focuses on enhancing visual elements and user experience. Admins will now have the ability to modify requests that you make. That will mean that they can discuss with you and make necessary changes instead of rejecting the request.
Why the change:
We believe in staying ahead of the curve, and our goal with this update is to provide you with a more modern and minimalistic interface. We're confident that the refreshed design will improve your overall user experience.
Your input matters:
We value your opinion, and we're eager to hear what you think about the updated UI. If you have any feedback, suggestions, or thoughts, please feel free to reach out to us at<EMAIL_ADDRESS>and Rocket.Chat channel. Your insights are essential in helping us refine and enhance the Platform Product Registry further.
Thank you for being a valued member of the Cloud services community. We appreciate your continued support, and we're committed to delivering an app with useful improvements for you!
Best regards,
The Platform Registry Team

Dear Cloud services community,
I hope this email finds you well.
There is a new functionality we would like to tell you about. We have added a new 'Include Gold DR' checkbox that reflects wether your namespaces in Gold have matching namespaces in Gold DR cluster.
When you create a new Product, the checkbox is unchecked by default. If you leave it unchecked, you will get namespaces in Gold cluster only and will not have the disaster recovery (DR) option.

If you check 'Include Gold DR' checkbox, the namespaces with same license plate will be provisioned for you in Gold DR cluster. You can find more information on how to use it here: https://digital.gov.bc.ca/cloud/services/private/internal-resources/gold/

If you have Products in Gold cluster and they were created before May 9, 2024, you have 'Include Gold DR' on by default. If you don't need it, you can uncheck the checkbox and that will allow you to submit an Edit Request. After request is provisioned, the namespaces in Gold DR cluster will be deleted.
If you will need Gold DR namespaces in the future, you can check the checkbox and submit it as an Edit Request to provision them again.
If you have any feedback, suggestions, or thoughts, please feel free to reach out to us at<EMAIL_ADDRESS>and Rocket.Chat channel #devops-registry
Best regards,
The Platform Registry Team

Dear Cloud services community,
We hope you are doing well!
We wanted to remind you of the new update added to the Platform Product Registry. There's a new checkbox called 'Include Gold DR' and this feature helps you see if your namespaces in the Gold cluster have matching namespaces in the Gold DR cluster.
When you create a new Product now, this checkbox is not selected. This means you'll only get namespaces in the Gold cluster and won't have disaster recovery (DR) options.
It's important to know that Products made in the Gold cluster before May 9, 2024 have the 'Include Gold DR' checkbox selected by default. If you don't need DR, you can remove the selected checkbox and send an 'Edit Request' to submit the change. After we process it, the namespaces in the Gold DR cluster will be deleted.
If you need Gold DR namespaces later, you can select the checkbox and send another 'Edit Request' to add them back.
Let us know if you have any questions or feedback by emailing u at<EMAIL_ADDRESS>and Rocket.Chat channel #devops-registry
Best regards,
The Platform Registry Team


'PointerEvent' can't be assigned to the parameter type 'PointerDownEvent'.

Any solution?
Running Gradle task 'assembleDebug'...
../../../.pub-cache/hosted/pub.dartlang.org/flutter_colorpicker-0.4.0/lib/src/hsv_picker.dart:731:29: Error: The argument type 'PointerEvent' can't be assigned to the parameter type 'PointerDownEvent'.

'PointerEvent' is from 'package:flutter/src/gestures/events.dart' ('../../../flutter/packages/flutter/lib/src/gestures/events.dart').
'PointerDownEvent' is from 'package:flutter/src/gestures/events.dart' ('../../../flutter/packages/flutter/lib/src/gestures/events.dart').
super.addAllowedPointer(event);

Same here..
Running the example with the flag to ignore deprecation from embedding Android v1


Plugin stopped working

I couldn't get it working back no matter what i do.

I have the same problem. Have you found a solution @Pedro-Cataneo ?

Nothing. Stopped using for now, unfortunately.

Same here, happened recently for no immediately apparent reason.
Would love a fix or at least a workaround as this has become surprisingly important to my workflow.

@piogab It is rather inconsistent which makes troubleshooting a hassle.
Restarting Obsidian did work once for me and then never again.
Reinstalling the plugin did resolve the issue for me once until restarting Obsidian; after which reinstalling did nothing anymore.
And the plugin seems to work fine in another vault which is identical to the first one (regarding plugins) so I can't pinpoint the issue.
I've installed better-search-views which fulfills a similar role whilst awaiting some kind of solution I rather not hope for.

Through trial and error I figured out that in my case I have to leave the Core Plugin Backlinks on, otherwise the plugin doesn't work. Does this work for you?

Man that was exactly it! I had disabled all the core plugins that i dont use and didnt realised that was the reason query control wasnt working. Thanks, man

Through trial and error I figured out that in my case I have to leave the Core Plugin Backlinks on, otherwise the plugin doesn't work. Does this work for you?

Man that was exactly it! I had disabled all the core plugins that i dont use and didnt realised that was the reason query control wasnt working. Thanks, man

Cool :)


Updated compose version and fixed build

Updated compose version to 1.2.0-beta03
Updated gradle plugin to 1.6.21 needed to build properly
Updated Gradle
Updated dependencies versions to latest ones

Hi, Thanks for this. Has this been tested locally?


Removed broken link

@EJKinger looks like that file is also referenced at the bottom of the getting started page, and also in  running-exporters.md, would you mind removing those references also?

@pgier Done

Thanks!


Unreadable comments

https://github.com/expiredy101/EmotionDetectionAi/blob/5be8c320194db7bf9866262a6ba5755303fcdb80/presentation/MarketSimulator/Assets/scripts/Playercontroller.cs#L3
https://github.com/expiredy101/EmotionDetectionAi/blob/5be8c320194db7bf9866262a6ba5755303fcdb80/presentation/MarketSimulator/Assets/scripts/Playercontroller.cs#L23-L26
https://github.com/expiredy101/EmotionDetectionAi/blob/5be8c320194db7bf9866262a6ba5755303fcdb80/presentation/MarketSimulator/Assets/scripts/Playercontroller.cs#L29
https://github.com/expiredy101/EmotionDetectionAi/blob/5be8c320194db7bf9866262a6ba5755303fcdb80/presentation/MarketSimulator/Assets/scripts/Playercontroller.cs#L49-L50
https://github.com/expiredy101/EmotionDetectionAi/blob/5be8c320194db7bf9866262a6ba5755303fcdb80/presentation/MarketSimulator/Assets/scripts/Playercontroller.cs#L53
https://github.com/expiredy101/EmotionDetectionAi/blob/5be8c320194db7bf9866262a6ba5755303fcdb80/presentation/MarketSimulator/Assets/scripts/Playercontroller.cs#L66-L67
In this character-controller script we got a kinda strange comment lines, hope somebody will fix them soon

So, cause i rewrite entire script from zero...) There is no more unreadable comments


Running the contributed command: 'vim.Esc' failed

Any ideas why? I had to restart every time this occurs.
Thanks!

I have not seen it recently.
Please let me know the situation that occurred in detail.

@74th Hmmm, okay. I'll put on the detail when this occurs again.

Ok I think I now how to reproduce it:

Start with normal mode.
Multi select a word using your Add Selection To Next Find Match binding.
Press c.
Go back to normal mode.

@rvlewerissa Thank you for your information. I fixed it. Please update this plugin.


add subcommand version

Right now we don't have any versioning scheme for kedge so let's add it.
References:

kompose version subcommand: https://github.com/kubernetes-incubator/kompose/blob/master/cmd/version.go

PR here: #121


feat: include RSTUF unrecognized fields in the keys

Description
Add two unrecognized fields in the keys for the TUF metadata

x-rstuf-key-name to store key names
x-rstuf-online-key-uri to be used by online keys to store the URI used by secure systems lib.
This field is part of several changes being done in the RSTUF Worker and the CLI to make the signers simpler. This support was added on RSTUF Worker.
Reference: https://github.com/repository-service-tuf/repository-service-tuf-worker/commit/b70143c0a5a8c8b5a647c73bfd1129684297998c

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality not to work as expected)

Additional requirements

[ ] Tests have been added for the bug fix or new feature
[x] Docs have been added for the bug fix or new feature

Code of Conduct
By submitting this PR, you agree to follow our Code of Conduct.

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

I did some test and it works fine

I recommend reviewing PR #534, which implements it with the new FastAPI/Pydantic support.

Closed in favor of #534


feat: Orchestration, Spring AI and Everything Else

Context
Yes, context.
Feature scope:
Yes.
Definition of Done

[ ] Functionality scope stated & covered
[ ] Tests cover the scope above
[ ] ~Error handling created / updated & covered by the tests above~
[ ] ~Aligned changes with the JavaScript SDK~
[ ] ~Documentation updated~
[ ] ~Release notes updated~

will be split up into separate PRs. See #114


Fix: snapshot lock

Existence of snapshot and the corresponding snapshot entry should be atomic.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 94.11765% with 1 line in your changes missing coverage. Please review.

Project coverage is 94.59%. Comparing base (40c17f7) to head (f63aee7).

Files with missing lines
Patch %
Lines

sorock/src/process/raft_process/responder.rs
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #451      +/-   ##
==========================================
+ Coverage   94.56%   94.59%   +0.02%
==========================================
  Files          47       47
  Lines        2448     2441       -7
==========================================
- Hits         2315     2309       -6
+ Misses        133      132       -1

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


Don't use  tags in hidden_combined_output

If the content of .CombinedOutput is Markdown-like,
it will be parsed as markdown and the posted comment will be ugly.
As Is

To Be

Thank you for contribution!
I could reproduce the problem.
$ github-comment --version
github-comment version 2.0.0

foo.txt
+ foo
+ bar

---

- foo
- bar

AS IS
.github-comment.yml
exec:
  default:
  - when: true
    template: |
      {{template "hidden_combined_output" .}}

$ github-comment exec -- cat foo.txt
+ foo
+ bar

---

- foo
- bar

TO BE
exec:
  default:
  - when: true
    template: |
      <details>

      ```
      {{.CombinedOutput | AvoidHTMLEscape}}
      ```

      </details>

$ github-comment exec -- cat foo.txt
+ foo
+ bar

---

- foo
- bar

Thank you for contribution!
I could reproduce the problem.
$ github-comment --version
github-comment version 2.0.0

foo.txt
+ foo
+ bar

---

- foo
- bar

AS IS
.github-comment.yml
exec:
  default:
  - when: true
    template: |
      {{template "hidden_combined_output" .}}

$ github-comment exec -- cat foo.txt
+ foo
+ bar

---

- foo
- bar

TO BE
exec:
  default:
  - when: true
    template: |
      <details>

      ```
      {{.CombinedOutput | AvoidHTMLEscape}}
      ```

      </details>

$ github-comment exec -- cat foo.txt
+ foo
+ bar

---

- foo
- bar


feat: add metric on pod evicted during node termination

Description
Emit metric on number of pods evicted during node termination
How was this change tested?
make verify
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 80.241%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
7712

Relevant Lines:
9611

💛  - Coveralls

cc: @njtran To look at these changes as well.

@chenfeilee let us know if you need any more help here!

Hi all, thanks for the review and so sorry that I didn't get the chance to get to this. Let me fix this within the next few days.

@chenfeilee any update?

Closing, feel free to re-open when you get back to this!


Unclear form validation documentation

It's  unclear for me, how to "run" a simple form validation.
Using simple example I was not able to display any error messages without runing
$validation->run($this->request->getPost());

right after $validation->setRules(). But in the docs $validation->run() method is applied for specific cases only.
https://github.com/codeigniter4/CodeIgniter4/blob/73d6ae6030659f1b6d0722d505f371a66ef98fbe/user_guide_src/source/libraries/validation.rst#the-controller
Would be great to have working example with errors displayed.

I sent a PR to fix this: #5223


How to set SPFx Field Customizer SharePoint Column width #spfx

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

I'm developing a field customizer that takes a JSON value and renders different elements such as informative texts, buttons with custom behavior etc.
I've noticed that when a new user enters the list all fields have a default width. I've noticed that SharePoint JSON-Formatting of a field does not work as it seems to overwrite the rendering of my field completely.
Resizing the column and saving the view seem to work for the current user but does not have affect on other users experience.
I would like to know if its possible to set a columns width for the modern experience some other way or if its possible to do so from the SPFx solution.
Here you can see two examples of my SPFx Field Customizer. In the first image you can see the field as viewed by a new user and the second the way i would like it to be implemented (after resizing the field).
Picture 1:

Picture 2:

The column sizes are stored in Local Storage on individual user's machines. You can find them with entries in the form of SPColumnSize-VIEWGUID stored as an object with the internal column name and the size of the column in pixels. Like this:

I'm not aware of any open API to set these values. Technically you could be able to write these values yourself from a SPFx Field Customizer (NOT list formatting) but just writing the value wouldn't have effect until a refresh and a user could always override this value by resizing.

@thechriskent interesting and thanks for the info 👍 I'm thinking this could fit well in the onInit function and I will definitely try it out.
If there are other ways to do this I would appreciate it as this does not really fulfill my requirements due to the user having to reload the page. It would be great if this was supported to set the width some other way.
I have one more question, In my case, this field will be present in different lists and libraries throughout a staging environment so I would like to fetch the viewguid of the current context but I can't seem to find a way to do this, any tips?

I believe you can get the view id by using this.context.pageContext.legacyPageContext.viewId
You can find out more about legacyPageContext in the docs. Why it hasn't been added as part of the TypeScript API yet, I don't know.

I would really like to see this as a feature to be able to set the width more easily.

@georgenorberg best approach then is to submit a feature request via user voice (https://aka.ms/sp-dev-uservoice


find images with sensordust

right now all images are already cleaned, so we can't test sensordust removal

I did a test with a camera from college but I didn't find any sensor dust on it. I searched the internet and especially flickr for images with sensor dust. the images are now on next cloud.


Fix reconciliation

The recon logic needs refactoring to allow for unit testing.
Add tm to requests that use implied "now" so the "time machine" can be used

This was fixed in v3.3.7


feat: use experimental_use

Ref: https://github.com/facebook/react/pull/25084
In addition:

Refactor with unified Symbol PROXY_STATE.
Precise SNAPSHOT type with Awaited.

According to https://github.com/reactjs/rfcs/pull/229, the use of use here is violating the rule/convention of React.
From our perspective, this PR is not breaking.
When React finalizes the spec and release a stable version, we will consider following their pattern, and in that case, our library comes with breaking changes. i.e. snapshot will not resolve promises.
Meanwhile, let's merge this and release a patch version.


['tomcat']['base_version'] is not getting overridden?

chef client versions afftected:
11.16.4 , 12.0.3
cookbook version
<EMAIL_ADDRESS>node ohai info (platform):
$ ohai  | grep platform
  "platform": "centos",
  "platform_version": "6.5",
  "platform_family": "rhel",

How to reproduce:
cat metadat/default.rb
name             'test'
version          '0.1.0'
depends          'tomcat'

cat recipes/default.rb
 node.override['tomcat']['base_version'] = 7
 include_recipe "tomcat"

chef-client -l info -r test

Starting Chef Client, version 12.0.3
[2015-03-05T12:21:13+00:00] INFO: *** Chef 12.0.3 ***
[2015-03-05T12:21:13+00:00] INFO: Chef-client pid: 12094
[2015-03-05T12:21:14+00:00] INFO: Setting the run_list to [#<Chef::RunList::RunListItem:0x00000002436cd0 @version=nil, @type=:recipe, @name="test">] from CLI options
[2015-03-05T12:21:14+00:00] INFO: Run List is [recipe[test]]
[2015-03-05T12:21:14+00:00] INFO: Run List expands to [test]
[2015-03-05T12:21:14+00:00] INFO: Starting Chef Run for ********
[2015-03-05T12:21:14+00:00] INFO: Running start handlers
[2015-03-05T12:21:14+00:00] INFO: Start handlers complete.
[2015-03-05T12:21:14+00:00] INFO: HTTP Request Returned 404 Object Not Found:
resolving cookbooks for run list: ["test"]
[2015-03-05T12:21:14+00:00] INFO: Loading cookbooks<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Synchronizing Cookbooks:
  - yum-epel
  - yum
  - openssl
  - chef-sugar
  - java
  - tomcat
  - test
Compiling Cookbooks...
 I expect according to the following code , i should be `tomcat' package ?
cat tomcat/attributes/default.rb :
default['tomcat']['base_version'] = 6
...
...
...
...

case node['platform_family']

when 'rhel', 'fedora'
  suffix = node['tomcat']['base_version'].to_i < 7 ? node['tomcat']['base_version'] : ""
  ....
  default['tomcat']['packages'] = ["tomcat#{suffix}"]

This cookbook needs refactoring. Please see #89, #102 and especially this comment.

This cookbook needs refactoring

Thank you. When you are going to fix this?

You better ask @someara or @jtimberman .

First off I'm very sorry that it's taken so long to get back to you on this issue.  For a significant period of time there was no active Chef Software Inc. maintainer for this repository and things languished.  We're now actively maintaining this cookbook again, and as part of that I've rewritten the cookbook with new custom resources for installing Tomcat from Apache Org sources.  This will ship in an upcoming 2.0 release and then we'll remove the existing recipes and providers in a future 3.X release.  With those new providers I believe the need for this change goes away so I'm going to close this out.  I encourage you to give the new providers a try and let me know if we can tackle more Tomcat management without the attributes bloat the current code suffers from.  They're on the master branch and once the documentation and testing improves a bit they'll be released as a 2.0 cookbook.


Password validation rule: length is malfunctioning in sub-organization

Description
Refer to the screen recording,
https://github.com/wso2/product-is/assets/27714968/cf5caa7c-89cf-41ea-8d51-cccc03c3873f
Steps to reproduce

Login to console and navigate to login and registration
Go to password validation and provide following configs

Length 5-30
Numbers 1

Create a suborg and try to create a user with 4 char + 1 num password

With the new fixes working as intended
Console
https://github.com/wso2/product-is/assets/42619922/52a49937-2482-45da-89cc-9e84cb2a014a
My Account
https://github.com/wso2/product-is/assets/42619922/52dff72a-b1e1-42d1-9e61-8b89ebb7f981


Add sanity checks when deriving Rights

Add checks to ensure that the:

given source Right (given as an ID) is actually a right on the persistence layer
given current_holder is actually the current owner of the Right.

Codecov Report

Merging #64 into master will increase coverage by 0.11%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #64      +/-   ##
=========================================
+ Coverage   94.98%   95.1%   +0.11%
=========================================
  Files          11      11
  Lines         419     429      +10
=========================================
+ Hits          398     408      +10
  Misses         21      21

Impacted Files
Coverage Δ

coalaip/plugin.py
100% <100%> (ø)
:white_check_mark:

coalaip/coalaip.py
98.46% <100%> (+0.24%)
:white_check_mark:

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 057dad8...bbbf387. Read the comment docs.


TextInput show start point of big text which is bigger than input

I have string, which doesn't fit to input, because of it's bigger and I want to make similar thing:

I searched but could't find any solution :/ is there something to make input like that?

@react-native-bot Dumb, it was question and not an issue :3


PRs requiring approval & merging main?

What is your issue?
Sorry I haven't been on the calls at all recently (unfortunately the schedule is difficult for me). Maybe this was discussed there?
PRs now seem to require a separate approval prior to merging. Is there an upside to this? Is there any difference between those who can approve and those who can merge? Otherwise it just seems like more clicking.
PRs also now seem to require merging the latest main prior to merging? I get there's some theoretical value to this, because changes can semantically conflict with each other. But it's extremely rare that this actually happens (can we point to cases?), and it limits the immediacy & throughput of PRs. If the bad outcome does ever happen, we find out quickly when main tests fail and can revert.

(fwiw I wrote a few principles around this down a while ago here; those are much stronger than what I'm suggesting in this issue though)

definitely not intentional, that was a side-effect of disabling the branch protection rule when the datatree merge happened last week. These options are on by default if you enable the branch protection again, and I simply forgot to disable them. They're off now, so we should be back to the old state.

Thanks so much @keewis !

Are you happy to keep the old way we were doing things @max-sixty ? If so we can close this.

Are you happy to keep the old way we were doing things @max-sixty ? If so we can close this now that @keewis has disabled the protections again.

Ofc!
Sorry for my slight diatribe above, I had thought just asking "did you really mean to do this" would be too oblique, but in fact sounds like no one did intend it :)


Use code editor for all prompting textareas

It'd be nice if the prompt or knowledge fields behave like the VSCode editor (monaco-editor decided to use codemirror) so you could use keyboard shortcuts to move text between line, use multiple lines selector, etc
Should match the look and feel of the existing UI.

:tada: This issue has been resolved in version 0.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


better way to pass projectOwner

https://github.com/The-Poolz/LockDealNFT/blob/e67315183004464639e4f00f2aa781250e55f606/contracts/AdvancedProviders/CollateralProvider/CollateralProvider.sol#L49-L52
in the current state, only for this, we pass more info than need. we can implement ERC165 to check if need more data. and pass only if need the data needed.
for example, as default make IBasicWithdrawInteface
and IAdvancedWithdrawInteface
on withdraw add if: implement IBasicWithdrawInteface then do withdraw without params.
else if IAdvancedWithdrawInteface , pass the project owener

#241