Metrics are not normalized

Metrics are normalized when running pwmetrics through CLI, however they are not when pwmetrics is required as a module. The get_config() method in expectations.js makes the following two calls:
validateMetrics(expectations.metrics);
normalizeMetrics(expectations.metrics);

This turns metrics like >=500 (this is what is documented in the readme, both for package.json as well as directly passing options to the constructor), into basic integers by stripping the >=.
These methods are not applied, however, when invoking pwmetrics as an included module. I assume this is an unintentional omission, as it prevents fully consistent behavior between CLI/package.json and module opt configurations.
The easy fix is adding the above-listed validation and normalization methods to the PWMetrics constructor.

It has to be fixed before releasing sheets stuff.

#37 merged


Error objects should also contain location information

Consider the following snippet:
{
  "one": 1
  "two": 2
}

If you run parseTree on it, it parses fine (which is cool), but the error that is reported has no other information than the error code. It would be nice if each error also contained the location information so they could be found in source (for example in an editor or IDE).
Information that would be handy to have:

the starting offset of the error
the length of the bad stuff
the column start of the error
the line number

Hi @aeschli,
Offset and length is now part of ParseError. But what about the line number and column?

PR is welcome!

@aeschli Would you prefer a) additional (redundant) members in ParseError ... or b) some utility function (jsonText: string, error: ParserError) => { line: number; column: number }?
I tend to option a (+ making the members of ParseError readonly).
Further: Should the numbers be 0-based (like in the TypeScript-API), or 1-based (like in the majority of similar language-parsing APIs)?


Handle UnicodeDecodeError on write to Postgres.

Unfortunately, I don't have a good automated test to show this, but the following simple example returns a UnicodeDecodeError with the current postgres.py in master when it attempts to write the third row (or any text with extended Unicode characters).
#coding: utf-8

import luigi
import luigi.postgres
import logging

logger = logging.getLogger("luigi-interface")

testdata = ([99, 'My fake plants died because I did not pretend to water them.'],
       [100, 'I always arrive late at the office, but I make up for it by leaving early.'],
       [101, 'ஸ்றீனிவாஸ ராமானுஜன் ஐயங்கார்'])

class TrivialPgLoad(luigi.postgres.CopyToTable):
    host = 'localhost'
    database = 'spotify'
    user = 'spotify'
    password = 'guest'
    table = 'trivium'
    columns = [("trival_id", "INT"),
               ("description", "TEXT")]

    def rows(self):
        for row in testdata:
            yield row

if __name__ == '__main__':
    luigi.run(main_task_cls=TrivialPgLoad)

The traceback looks like:
Traceback (most recent call last):
  File "/Users/thomas/Dropbox/envs/luigi-pg-unicode/lib/python2.7/site-packages/luigi/worker.py", line 137, in run
    new_deps = self._run_get_new_deps()
  File "/Users/thomas/Dropbox/envs/luigi-pg-unicode/lib/python2.7/site-packages/luigi/worker.py", line 88, in _run_get_new_deps
    task_gen = self.task.run()
  File "/Users/thomas/Dropbox/envs/luigi-pg-unicode/lib/python2.7/site-packages/luigi/postgres.py", line 318, in run
    tmp_file.write(rowstr.encode('utf-8'))
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe0 in position 4: ordinal not in range(128)

The load to Postgres succeeds with my fix.

Not thinking through this fully, but I would rather make the conversion explicit – it seems like either all the rows are utf-8, or none of them are.

Yes, it actually works just as well for me to take the encode() call out of line 318 completely. I used the try/except block because I didn't feel 100% confident that doing it that wouldn't break other use cases.

There was some discussion of this previously over here: https://groups.google.com/forum/m/#!searchin/luigi-user/Unicodedecodeerror/luigi-user/2UyGTJHFx_8 ... ISTM the issue has to do with the data getting converted to utf-8 twice in some cases. The issue appears if the source data is already Unicode. Pure Python str inserts work with the current code.

The issue here (like at many place in Luigi) is that Binary and String data are missed. The postgres task is build to handle string and not binary data. However, in the example that you gave you use binary data  (I assume that you are using python2). The easy fix would be to use unicode instead of str.
testdata = ([99, u'My fake plants died because I did not pretend to water them.'],
       [100, u'I always arrive late at the office, but I make up for it by leaving early.'],
       [101, u'ஸ்றீனிவாஸ ராமானுஜன் ஐயங்கார்'])

Again, it is precisely the case where the data are already Unicode that fails the current code. Str data load fine as is.

I had a closer look and there is indeed an issue, but it's not with the line in the traceback, but with this one. I'll send a PR soon.
@erikbern I wouldn't require the user to encode explicitly the data since the text need to be encoded in utf-8 since it's what the connection expect.  The current implementation doesn't seems to handle binary data, so the users should provide text data which is unicode for python2. Doing otherwise would create issue if we want to handle binary data.

@gpoulin - thanks! I knew about line 265—that's what I was trying to say about unicode lines having encode() called twice. Since you seem to have a clearer idea how to fix this than I do, I will look forward to your PR, and we can close this one now.


fix: helmfile template fails when selector matches a chart fetched with go-getter

helmfile template fails when selector matches a chart fetched with go-getter
fix: https://github.com/helmfile/helmfile/issues/498
func (st *HelmState) ApplyOverrides(spec *ReleaseSpec) {
	if st.OverrideKubeContext != "" && spec.KubeContext == "" {
		spec.KubeContext = st.OverrideKubeContext
	}
	if st.OverrideNamespace != "" && spec.Namespace == "" {
		spec.Namespace = st.OverrideNamespace
	}

for this logic. when ReleaseSpec's Namespace and KubeContext are not set. we override them. else we do nothing.
Signed-off-by: yxxhero<EMAIL_ADDRESS>

@itscaro @mumoshu I think we need to refactor the code about releaseWithOverrides logic.

@itscaro @mumoshu ping

@mumoshu
let me try to explain:
the key is the top key: namespace
namespace: istio-system
repositories:
....

when using selector feature:
https://github.com/helmfile/helmfile/blob/8f59a1c18a1a1ff020aeefed3815407bd46d5067/pkg/state/state.go#L1044-L1058
it will call GetReleasesWithOverrides func, set top namespace to releaseSpec namespace. and return map[PrepareChartKey]string
PrepareChartKey:
type PrepareChartKey struct {
	Namespace, Name, KubeContext string
}

PrepareChartKey contains namespace info. but when setting chartPath, it uses origin state.Releases, not contain namespace info. So the chartPath can't be set correctly. see:
https://github.com/helmfile/helmfile/blob/8f59a1c18a1a1ff020aeefed3815407bd46d5067/pkg/app/run.go#L79-L93
for i := range r.state.Releases

chartPath match logic:
	for i := range r.state.Releases {
		rel := &r.state.Releases[i]

		key := state.PrepareChartKey{
			Name:        rel.Name,
			Namespace:   rel.Namespace,  // see this: rel.Namespace will be empty. not top namespace value. So it will not match.
			KubeContext: rel.KubeContext,
		}
		if chart := releaseToChart[key]; chart != rel.Chart {
			// In this case we assume that the chart is downloaded and modified by Helmfile and chartify.
			// So we take note of the local filesystem path to the modified version of the chart
			// and use it later via the Release.ChartPathOrName() func.
			rel.ChartPath = chart
		}
	}

@mumoshu my English is no good. So I hope that my explanation is ok.

@mumoshu this is a very old bug that already exists so a long time.

@mumoshu kubcontext also has same bug.

@mumoshu if have more question, please free to ping me.

@xiaomudk ping

@yxxhero Thanks for your explanation!
Can we now treat every occurrence of state.Releases at a read position as a bug, and all those should be modified to use GetReleasesWithOverrides instead, similar to you've done in this pull request?
Then, how about doing the overriding on unmarshal time, so that we can make GetReleasesWithOverrides a private function that can't be called outside of the pkg. That way, we'll never introduce any bug like this one. WDYT?

@mumoshu makes sense. I will update the code.

I looked at the logic carefully again, and to generate a complete helmstate, I need to go through multiple yaml unmarshals, including the loading of sub helmfiles, and then perform logical processing after merging. The current modification is really the processing after the generation of helmstate, in the subcommand Before processing, call GetReleasesWithOverrides once.
Other modification methods currently see the changes are relatively large.
@mumoshu

@mumoshu I tried to implement it.

	if ld.namespace != "" {
		if st.OverrideNamespace != "" {
			return nil, errors.New("err: Cannot use option --namespace and set attribute namespace.")
		}
		st.OverrideNamespace = ld.namespace
	}

	if ld.chart != "" {
		if st.OverrideChart != "" {
			return nil, errors.New("err: Cannot use option --chart and set attribute chart.")
		}
		st.OverrideChart = ld.chart
	}

After the helmstate is generated, there is another namespace coverage logic. There are too many changes. It needs to be well designed to ensure that there are no errors. @mumoshu

@mumoshu done and ping

latest release is confirmed working. thanks all!
helmfile  -l name=acme-jx template

Building dependency release=acme-jx, chart=/Users/joshua.simon/Library/Caches/helmfile/istio-system/acme-jx/https_github_com_joshuasimon-taulia_acme_git.ref=master/charts/acme
Templating release=acme-jx, chart=/Users/joshua.simon/Library/Caches/helmfile/istio-system/acme-jx/https_github_com_joshuasimon-taulia_acme_git.ref=master/charts/acme

# Source: acme/templates/cert-manager-prod-certificate.yaml
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
...


Discussion: will reedline become a full-fledged "text editor?"

This issue is to discuss direction and scope of reedline's "text editor" and "IDE" features.
The goal of PR #670 was "allowing the user to press "jk" (without modifiers) to move from Vi insert mode to Vi normal mode." In that PR, @benvansleen said:

Achieving this with the existing keybindings section would require implementing something akin to emacs key-chords (since you press one key, release, then press the next). That would be a much, much wider-reaching change.

This seems related to the question of "how many full-fledged text editor features should be added to reedline?" Heavy-duty text editors like vim essentially allow the user to configure a state machine where a given keypress will have a different effect depending on the current state. (For example, the keypress 'k' would have a different effect depending on if the user is in "insert" mode or "normal" mode, and also depending on whether the 'k' immediately followed a 'j'.)
Does it make sense for reedline to move in the above direction (implementing a user-configurable state machine)? Related are questions regarding how many other "text editor" features should be included (e.g. vim allows for user-configurable callbacks, plugins, keymaps to trigger system calls or perform arbitrary turing-complete logic, etc).
I think it could be helpful to discuss vision for how far reedline will move in the direction of becoming a "text editor" / "IDE." Having a vision for this will help to ensure that:

new features are added in a manner that's consistent with overall vision for user experience, and
the implementation decisions made in adding new features will not be counterproductive to progress on other planned features.

Related:

https://github.com/nushell/reedline/issues/63

We've wanted to have better VI keybinding functionality for a long time. You can see some of our brainstorming ideas https://github.com/nushell/reedline/issues/69, which is linked from the meta issue that you mention above.
I think reedline should cover as much functionality as is supportable. An example of what I mean by that is I like the idea of a user configurable state machine like you mention above. However, if there's no one to write it and no one to maintain it, then it's probably not something reedline should have.
I think this is the problem with many open-source projects. There are a lot of plans and ideas but unless someone comes along to implement and support some feature, it goes undone. Since we're all volunteers, we just work on things that interest us. There is a bug in reedline that has been there for years and prevents WezTerm from working properly. My solution to that was to move to Ghostty because I couldn't figure out the solution.
So, it's probably less a matter of what we want, although that's a good starting place, and more of a matter of who will write it and support it for the community.

Has outsourcing the modal editing logic to a third-party dependency been considered as an option?
e.g. https://github.com/ulyssa/modalkit
I would expect this to address the open issues regarding the limitations of the edit modes and reduce the associated maintenance burden.

Has outsourcing the modal editing logic to a third-party dependency been considered as an option? e.g. https://github.com/ulyssa/modalkit
I would expect this to address the open issues regarding the limitations of the edit modes and reduce the associated maintenance burden.

Never heard of it, but we're probably open to it.


Doesn't work when count of children less than 2

Sometimes needs to add only one or zero items to the children (one or zero items to the optgroup). Your code fails on this situation.
Problems on lines
https://github.com/Sharlaan/material-ui-superselectfield/blob/master/lib/SuperSelectField.js#L452
https://github.com/Sharlaan/material-ui-superselectfield/blob/master/lib/SuperSelectField.js#L558
etc.
Example code
import React from 'react';
import ReactDOM from 'react-dom';
import MuiThemeProvider from 'material-ui/styles/MuiThemeProvider';
import SuperSelectField from 'material-ui-superselectfield/lib/SuperSelectField'

ReactDOM.render(
  <MuiThemeProvider>

      <SuperSelectField>
        A
      </SuperSelectField>

  </MuiThemeProvider>,
  document.getElementById('root')
);

or example with one optgroup
import React from 'react';
import ReactDOM from 'react-dom';
import MuiThemeProvider from 'material-ui/styles/MuiThemeProvider';
import SuperSelectField from 'material-ui-superselectfield/lib/SuperSelectField'

ReactDOM.render(
  <MuiThemeProvider>

      <SuperSelectField multiple>
        <optgroup label="A">
          A1
          A2
        </optgroup>
      </SuperSelectField>

  </MuiThemeProvider>,
  document.getElementById('root')
);

This happens because children in this case is a single object but not an array how you expect.

Thanks for your report, i just pushed a fix.
Can you retry with v1.5.1, please ?

I used lates version from github.

Damn it! ok i'll re-test again :p

Ok i just pushed a version that should fix both issues, please confirm.

Previous cases works great. But I found another
Case 3:

Added your edge cases and others into CodeExample1.js
Can you tell me if those work for you ?

I checked it.
Everything is fine.
Thanks.


[FLINK-3874]  Add a Kafka TableSink with JSON serialization

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[x] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[x] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

I tried to inherit TableSink trait in Java code but it seems that it is impossible to inherit traits with vars in Java, therefor I had to change class structures there somewhat.

I've update the PR according to the PR and fixed the build (I was using a method from JDK8).

Last build failed on an outdated version of this branch. Pushed an updated and fixed version of this PR to trigger build on a newer version.

Fixed build.

Build is now passing. Could someone please review this PR?

This PR has not been reviewed during the last 14 days.
Can anyone please review it?

@mushketyk I will review it this week.

@twalthr Great! Thank your help.

@mushketyk Thanks for your PR. I added some comments to the code. The changes you made to the TableSink class structure seem reasonable. However, it might be better if those change would be made in a separate Jira issue and PR.
In general, it would be great if you try to touch as little existing code as possible and orientate yourself more on the existing code base.

@twalthr Thank you for your detailed review. I am very new to Apache Flink at the moment so I made some absurd changes. I'll try to avoid this in the future.
I'll update the PR according to your comments in a day or two.

@twalthr I've updated the PR according to your comments. Could you please review it again?

LGTM except a few nits.

@wuchong @twalthr I've updated the PR according to your review. Could you please review it one more time?

I removed fieldNames and fieldTypes from the constructor arguments and rely on configure method. Because of that I had to add an additional method for create serialization scheme in sub classes, since field names and field types are unknown at the moment of an object creation.
@wuchong @twalthr Could you please review the PR again?

The CsvTableSinkTest failed, can you check it ?

@wuchong
Tests passed on most platforms, and failures do not seem to be related to code that added/changed:
java.lang.IllegalStateException: cannot create children while terminating or terminated

	at akka.actor.dungeon.Children$class.makeChild(Children.scala:200)

	at akka.actor.dungeon.Children$class.attachChild(Children.scala:40)

	at akka.actor.ActorCell.attachChild(ActorCell.scala:369)

	at akka.actor.ActorSystemImpl.actorOf(ActorSystem.scala:555)

	at org.apache.flink.runtime.client.JobClient.submitJobAndWait(JobClient.java:120)

	at org.apache.flink.runtime.minicluster.FlinkMiniCluster.submitJobAndWait(FlinkMiniCluster.scala:485)

	at org.apache.flink.runtime.minicluster.FlinkMiniCluster.submitJobAndWait(FlinkMiniCluster.scala:471)

	at org.apache.flink.runtime.minicluster.FlinkMiniCluster.submitJobAndWait(FlinkMiniCluster.scala:463)

	at org.apache.flink.test.util.TestEnvironment.execute(TestEnvironment.java:78)

	at org.apache.flink.api.table.sink.CsvTableSinkTest.writeToCsv(CsvTableSinkTest.scala:69)

	at org.apache.flink.api.table.sink.CsvTableSinkTest.testSaveDataSetToCsvFileWithDefaultDelimiter(CsvTableSinkTest.scala:48)

Are there issues with these platforms?

@twalthr @wuchong
Could you please review one more time? This PR is blocking another similar PR.
The test failure does not seem to have anything to do with the new code.

Could we move CsvTableSinkTest to TableSinkITCase or just remove CsvTableSinkTest ?   I think TableSinkITCase has tested CsvTableSink with default delimiter and custom delimiter.

@wuchong I've removed the test as you've suggested since it pretty much duplicates TableSinkITCase.

@twalthr Could you review this one more time?

Hi @mushketyk, thanks for the PR. Looks quite good. I added a few comments.
Cheers, Fabian

Hi @fhueske. Thank you for your code review. I'll update the PR today.

Hi @mushketyk, thanks for the fast reply.
I saw your comments but no new commit. Did you forget to push to your PR branch?
Thanks, Fabian

Hi @fhueske, I am running mvn clean verify locally right now and I hope to update the commit soon.
Sorry for confusing you.

Ah, I see. Very good :-)

@fhueske I've fixed the PR according to your review.
The only thing that I didn't do is reusing ObjectNode on every iteration. It caused some weird NPEs during the integration tests and made code a bit more complex.
I can investigate it more if you think it really worth it.

Thanks for the update @mushketyk. Regarding the ObjectNode that's fine. It was just a possible optimization but if it is not worked let's keep it as it is. I added two minor comments. After that, the PR should be good to merge. Thanks, Fabian

Thank you for your review @fhueske. I've updated it. Could you please take a look at it?

Looks good to merge @mushketyk :-)
The only thing that I'd change would be to rename Kafka08JsonTableSinkTest to Kafka08JsonTableSinkITCase (same for Kafka09*), since these are long running integration tests which should be executed in the appropriate Maven phase (verify).
I can do that as well before merging.
Thanks, Fabian

Thank you for helping with this PR @fhueske
It's ok, I'll update the PR in an hour or so.

Hey @fhueske
I've renamed tests as you suggested.


Version of liburing

Hi,
I would like to install ibosmocore in ubunto i do this :
git clone https://gitea.osmocom.org/osmocom/libosmocore.git
cd libosmocore/
autoreconf -i
./configure
make
sudo make install
sudo ldconfig -i
cd ..
the result is:
configure: error: Package requirements (liburing >= 0.7) were not met:
Requested 'liburing >= 0.7' but version of liburing is 0.6
You may find new versions of liburing at http://git.kernel.dk/cgit/liburing/
Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.
Alternatively, you may set the environment variables URING_CFLAGS
and URING_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.
[ERROR] Configuration failed after running at least twice.
[ERROR] Problem occurred while building package libosmocore:
Configuration failed
[ERROR] Error installing package libosmocore. Aborting.
Taking account that i have installed successfully liburing v2.5
nstall -D -m 644 include/liburing/barrier.h /usr/include/liburing/barrier.h
install -D -m 644 include/liburing/io_uring_version.h /usr/include/liburing/io_uring_version.h
install -D -m 644 liburing.a /usr/lib/liburing.a
install -D -m 644 liburing-ffi.a /usr/lib/liburing-ffi.a
install -D -m 755 liburing.so.2.5 /usr/lib/liburing.so.2.5
install -D -m 755 liburing-ffi.so.2.5 /usr/lib/liburing-ffi.so.2.5
ln -sf liburing.so.2.5 /usr/lib/liburing.so.2
ln -sf liburing.so.2.5 /usr/lib/liburing.so
ln -sf liburing-ffi.so.2.5 /usr/lib/liburing-ffi.so.2
ln -sf liburing-ffi.so.2.5 /usr/lib/liburing-ffi.so
make[1] : on quitte le répertoire « /home/crm/liburing/src »
install -D -m 644 liburing.pc /usr/lib/pkgconfig/liburing.pc
install -D -m 644 liburing-ffi.pc /usr/lib/pkgconfig/liburing-ffi.pc
install -m 755 -d /usr/man/man2
install -m 644 man/.2 /usr/man/man2
install -m 755 -d /usr/man/man3
install -m 644 man/.3 /usr/man/man3
install -m 755 -d /usr/man/man7
install -m 644 man/*.7 /usr/man/man7
However when i do this command: pkg-config --modversion liburing
crm@crm-X570-UD:~/liburing$ pkg-config --modversion liburing
0.6
How can i obliged ubunto to work with the new version liburing 2.5 installed to be enable to install libsmocore
Because it returns that the version of liburing is 0.6
I'm so gratefull if you can help me

Dear sir;
I inform you that these library doesn't exist in the path usr/local/lib; it exist on usr/lib
Should i added it to this path or it is acceptable, because i added only the path in  environment variable  in /.bashrc:export LD_LIBRARY_PATH=/usr/local/lib
Also i ask if i should adjusting PKG_CONFIG_PATH environment variable because i add only this path
export PKG_CONFIG_PATH=/usr/local/lib/pkgconfig

However the path of libruring's pkgconfig is   /usr/lib/pkgconfig
What can i do in this case

dear sir
when i run this command # find / | grep liburing i obtain:
./.local/share/Trash/info/liburing.4.trashinfo
./.local/share/Trash/info/liburing.10.trashinfo
./.local/share/Trash/info/liburing.6.trashinfo
./.local/share/Trash/info/liburing.9.trashinfo
./.local/share/Trash/info/liburing-0.7.tar.gz.trashinfo
./.local/share/Trash/info/liburing.trashinfo
./.local/share/Trash/info/liburing.5.trashinfo
./.local/share/Trash/info/liburing-dev_0.6-3ubuntu1_amd64.deb.trashinfo
How can i disable it

I think you'll find better help for this issue in generic linux help channels, it's really beyond the scope of this project to help with issues like that.


Manifest version 2

Is a disruption in support for Manifest version 2 related to a login failure?

No. Some of TweetDeck APIs are deleted by X.
Please delete this post. Do not waste maintainer's time


Add ANGUISH, EXILED, elfevass - Gulyau

Song Name
Gulyau
Artist Name
ANGUISH, EXILED, elfevass
Source
Youtube
Youtube Link
https://youtube.com/watch?v=3PP1p7ex3Vc
Direct File Link
No response
Song ID
475234
Start Offset [ms]
No response
Extra
I like this song :)

:(

reject
popular songs like "Run!" are too inconvenient to be replaced by smth unrelated and are therefore usually rejected.
Note that you can still use the song yourself, but it won't be public


⚠ Bofrost.lu has degraded performance

In 979b9ec, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1700 ms

Resolved: Bofrost.lu performance has improved in 693389e after 13 minutes.


FFMPEG not cleaning old .ts files with current settings

Used your project as-is for a couple of hours and ended up with 1 Gb of data lying around. After some extended research, I managed to obtain what I (and I think you) wanted, with the following settings changed in processor.go :
(pasting the patch content did not go well! I have attached it instead!
processor.go.txt
)
Successfully made it run with both your Dockerfile on my local system, although I had to add an EXPOSE statement in both to have the ports 8080 and 80 accessible.
Also, I had to add the following line to Dockerfile.management because of a build error (parsel was missing):
(...)
RUN npm install -g parcel-bundler
(...)
Good job!

Thank you for opening this issue!
Yes this is exactly what I was trying to achieve. This is the exact reason why the function that cleans up the streams that are not active anymore also deletes the folder. I tried to fix this a couple of times but did not have any luck with it. So thank you for also fixing this, I'll include these changes in a new release soon.
The management Dockerfile actually has this issue because I have the dependencies installed locally as well. The parcel-bundler is a dev dependency (as it should be), but the image used the --production flag during install. Because I copy the whole directory, this did not come out for me unfortunately. That's a mistake and thank you for pointing it out 👏

My pleasure!
Your workaround would not have worked for me because I'm transcoding a
security camera, so the feed never ends, and my users might leave the web
page open for a few hours...
Have a nice week!
Francis
On Sun., Mar. 24, 2019, 15:38 Imre Racz<EMAIL_ADDRESS>wrote:

Thank you for opening this issue!
Yes this is exactly what I was trying to achieve. This is the exact reason
why the function that cleans up the streams that are not active anymore
also deletes the folder. I tried to fix this a couple of times but did not
have any luck with it. So thank you for also fixing this, I'll include
these changes in a new release soon.
The management Dockerfile actually has this issue because I have the
dependencies installed locally as well. The parcel-bundler is a dev
dependency (as it should be), but the image used the --production flag
during install. Because I copy the whole directory, this did not come out
for me unfortunately. That's a mistake and thank you for pointing it out
👏
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Roverr/rtsp-stream/issues/13#issuecomment-475992313,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHlGPRKYlzjksHHYb-Xo6Pmel3uWMRlmks5vZ9Q7gaJpZM4cFREF
.


Add presentation

[x] Add mdx-deck and deck
[ ] Add build pipeline to auto publish deck

Moved to https://github.com/rohit-smpx/slides


npm i @xtuc/long failing with npm ERR! 404  '@xtuc/long@latest' is not in the npm registry.

Hi,
The library seems to have been removed from the registry. I couldn't find anything on the npm page suggesting it has.
Do you know if it is an issue with the registry?
Cheers
Jorge

I still see the version here: https://www.npmjs.com/package/@xtuc/long
But it looks like the yarn repository no longer have it:
curl -I<EMAIL_ADDRESS>HTTP/2 404

This is most probably caused by https://status.npmjs.org/incidents/cksjqc1w11v5

I can't install webpack because of 404
94 http fetch GET 404 https://registry.npmjs.org/@xtuc%2flong 45ms 95 silly fetchPackageMetaData error for<EMAIL_ADDRESS>404 Not Found - GET https://registry.npmjs.org/@xtuc%2flong - Not found 96 verbose stack RangeError: Maximum call stack size exceeded

Same for me:
6 silly install readLocalPackageData
7 http fetch GET 404 https://registry.npmjs.org/@xtuc%2flong 133ms
8 http fetch GET 404 https://registry.npmjs.org/@xtuc%2flong 41ms
9 silly fetchPackageMetaData error for @xtuc/long@latest 404 Not Found - GET https://registry.npmjs.org/@xtuc%2flong - Not found
10 timing stage:rollbackFailedOptional Completed in 0ms
11 timing stage:runTopLevelLifecycles Completed in 215ms
12 verbose stack Error: 404 Not Found - GET https://registry.npmjs.org/@xtuc%2flong - Not found
12 verbose stack     at /.nvm/versions/node/v12.16.2/lib/node_modules/npm/node_modules/npm-registry-fetch/check-response.js:104:15
12 verbose stack     at processTicksAndRejections (internal/process/task_queues.js:97:5)
13 verbose statusCode 404
14 verbose pkgid @xtuc/long@latest

npm ERR! 404 Not Found - GET<EMAIL_ADDRESS>

Back to normal now, definitely a register issue (nothing related to this particular library). The fragility of all these dependencies is something to take into consideration.

Hm, I still have the error...


Massive images end up with silently absent thumbnails

e.g. if you upload a big thing like https://matrix.org/_matrix/media/r0/download/matrix.org/YMGWVTMYFjtsLILmqvrMpVwV then you get a large blank thumbnail :(

see https://github.com/vector-im/riot-web/issues/10914

not so silently:

Would be good to know what the error is server-side, because it seems fine for me: https://t2bot.io/_matrix/media/r0/thumbnail/matrix.org/YMGWVTMYFjtsLILmqvrMpVwV?width=800&height=600

This is not a matrix-media-repo problem - the error is coming from Synapse.


Voyager ran into an error

UI Version: 0.8001.1
The gaiacli rest-server (LCD) experienced an error: panic: resource temporarily unavailable goroutine 1 [running]: github.com/cosmos/cosmos-sdk/client/lcd.createHandler(0xc420179500, 0x5, 0x7ffeefbffc94) /go/src/github.com/cosmos/cosmos-sdk/client/lcd/root.go:78 +0x634 github.com/cosmos/cosmos-sdk/client/lcd.ServeCommand.func1(0xc420b3c900, 0xc4201b1080, 0x0, 0x8, 0x0, 0x0)

also, i can't go back to the app from this view

We will not work on this, as this will not be important anymore when on web.


HBASE-24803 Unify hbase-shell ::Shell::Commands::Command#help behavior

Resolves https://issues.apache.org/jira/browse/HBASE-24803
Changelog

Update all hbase-shell commands to return help text implicitly
Add hbase-shell test to ensure all help methods return unindented text
Set the outermost indentation to zero for the help text of all hbase-shell
commands by using the squiggly heredoc added in Ruby 2.3.0.
Document return value of hbck_chore_run and add usage examples to truncate commands
Update docs to reflect current return value of list command

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  6s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 28s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 34s
master passed

+0 :ok:
refguide
4m 55s
branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 20s
the patch passed

-0 :warning:
rubocop
1m 29s
The patch generated 23 new + 104 unchanged - 23 fixed = 127 total (was 127)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+0 :ok:
refguide
4m 52s
patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.

_ Other Tests _

+1 :green_heart:
asflicense
0m 28s
The patch does not generate ASF License warnings.

22m  7s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2178/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2178

Optional Tests
dupname asflicense rubocop refguide

uname
Linux 49e5436dbad3 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 840a55761b

refguide
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2178/1/artifact/yetus-general-check/output/branch-site/book.html

rubocop
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2178/1/artifact/yetus-general-check/output/diff-patch-rubocop.txt

refguide
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2178/1/artifact/yetus-general-check/output/patch-site/book.html

Max. process+thread count
78 (vs. ulimit of 12500)

modules
C: hbase-shell . U: .

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 54s
the patch passed

-0 :warning:
javadoc
0m 17s
root in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
166m 17s
root in the patch passed.

180m  3s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2178/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2178

Optional Tests
javac javadoc unit

uname
Linux 83ef37776bf5 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 840a55761b

Default Java
2020-01-14

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2178/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-root.txt

javadoc
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2178/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-root.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2178/1/testReport/

Max. process+thread count
7370 (vs. ulimit of 12500)

modules
C: hbase-shell . U: .

 U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2178/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


"VolumeSource" definition not present in "IstioOperator Options" document page

Bug description
Affected product area
[X] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Affected features
Expected behavior
User should be redirected to VolumeSource description section when clicking it as below:

Steps to reproduce the bug
Click VolumeSource linked text in IstioOperator Options page.
Version
Istioctl:
client version: 1.10.0
control plane version: 1.10.0
data plane version: 1.10.0 (1 proxies)

How was Istio installed?
istioctl install -f <istio-operator.yaml> -y
Environment where the bug was observed (cloud vendor, OS, etc)
MacOS

same problem


Add instructions for Fedora 29

To be released soon.

2018-10-23 Fedora 29 Final Release (GA) (Preferred target)
2018-10-30 Fedora 29 Final Release (GA) (Target 1)

https://nuvola.tiliado.eu/nuvola/fedora/fc29/


[QUESTION] Too many blank lines at the of file

Please describe.
Hello
I fighting with the end of file spaces.
When I save it adds 2 new lines while it should be adding 1 new line.
Then I get this warning: Too many blank lines at the end of file robocop(W1010)
I tried disabling this through the vscode settings but I still get 2 new lines.
vscode settings:
{
    "editor.formatOnSave": true,
    "editor.rulers": [
        80
    ],
    "python.linting.pylintEnabled": false,
    "python.linting.flake8Enabled": true,
    "python.linting.mypyEnabled": true,
    "python.formatting.provider": "black",
    "python.defaultInterpreterPath": "/usr/local/bin/python",
    "[python]": {
        "editor.codeActionsOnSave": {
            "source.organizeImports": true
        },
        "editor.rulers": [
            88
        ]
    },
    "files.exclude": {
        "**/__pycache__": true,
        "**/.git": true,
        ".cache/**": true,
        ".sonar/**": true
    },
    "files.watcherExclude": {
        "**/__pycache__": true,
        "**/.git/objects/**": true,
        "**/.git/subtree-cache/**": true,
        "**/env/**": true
    },
    "files.eol": "\n",
    "files.insertFinalNewline": true,
    "[robotframework]": {
        "files.insertFinalNewline": false
    },
    "terminal.integrated.defaultProfile.linux": "bash",
    "robotcode.robot.pythonPath": [
        "./libraries",
        "./resources",
        "./configs",
    ]
}

Desktop (please complete the following information):

VS Code Version 1.75.1
RobotCode Version 0.27.1
OS: linux
Python Version 3.10
RobotFramework Version 6.0.2
Additional tools: robocop 2.8.1, robotidy 3.4.0, devcontainer

can you disable the  "editor.formatOnSave": true and see what happens? sometime robotidy has some problems the last line of the file.

does it work?

@d-biehl sorry for the late reply.
So If I remove the formatOnSave the  file is not formatted anymore and the number of lines stays the same.
Other test I did is to set "files.insertFinalNewline": false when I save the number of new lines is OK but if there is too many new lines then I get 2 new lines at the end of the file.

We (Robotidy) had some problems with end of lines in the past but now it should work correctly (at least no one is reporting anything :) ). It looks like both this plugin and Robotidy are trying to do the same action (files.insertFinalNewline is something done by plugin, where Robotidy also adds new line).
If you have specific code it keeps on happening I can check it on Robotidy side (without robotcode extension).

I fixed a small bug, @Austriker can you check if it works correctly now?

Yeah I will have a look on Monday !
Thank you !
Le ven. 7 avr. 2023, 23:32, Daniel Biehl @.***> a
écrit :

I fixed a small bug, @Austriker https://github.com/Austriker can you
check if it works correctly now?
—
Reply to this email directly, view it on GitHub
https://github.com/d-biehl/robotcode/issues/124#issuecomment-1500663697,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGYNAKFYSRGEITPRU6J4JDXACBVTANCNFSM6AAAAAAVMC37MU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@d-biehl Indentation seems good and blank lines but I am getting threading issues and the next format hangs:
/home/berry/.vscode-server/extensions/d-biehl.robotcode-0.33.0/bundled/libs/robotcode/core/async_tools.py:465: UserWarning: Lock <robotcode.core.async_tools.Lock object at 0x7f575dcfcd60 [locked, waiters:2]> <Task pending name='RobotDiagnosticsProtocolPart.collect_unused_variable_references' coro=<RobotDiagnosticsProtocolPart.collect_unused_variable_references() running at /home/berry/.vscode-server/extensions/d-biehl.robotcode-0.33.0/bundled/libs/robotcode/language_server/robotframework/parts/diagnostics.py:330> wait_for=<Future pending cb=[FutureInfo._done(), Task.task_wakeup()]> cb=[FutureInfo._done(), Task.task_wakeup()]> takes to long <Thread(RobotDiagnosticsProtocolPart.collect_unused_keyword_references, started daemon<PHONE_NUMBER>92736)>
, try to cancel...
  warnings.warn(f"Lock {self} takes to long {threading.current_thread()}\n, try to cancel...")
/home/berry/.vscode-server/extensions/d-biehl.robotcode-0.33.0/bundled/libs/robotcode/core/async_tools.py:465: UserWarning: Lock <robotcode.core.async_tools.Lock object at 0x7f579018e5f0 [locked, waiters:1]> <Task pending name='Task-4247' coro=<Namespace.resources_changed() running at /home/berry/.vscode-server/extensions/d-biehl.robotcode-0.33.0/bundled/libs/robotcode/language_server/robotframework/diagnostics/namespace.py:656> wait_for=<Future pending cb=[FutureInfo._done(), Task.task_wakeup()]> cb=[FutureInfo._done(), as_completed.<locals>._on_completion() at /usr/local/lib/python3.10/asyncio/tasks.py:558]> takes to long <Thread(diagnostics_worker, started daemon<PHONE_NUMBER>00736)>
, try to cancel...
  warnings.warn(f"Lock {self} takes to long {threading.current_thread()}\n, try to cancel...")

Yes, this is a known error, most of the time the language server will recover, if not it helps to restart the language server. For this in VSCode there is the command "RobotCode: Restart Language Servers" or simply restart VSCode.
I'm working on the bugfix, but I can't promise when it will be finished, because some major refactorings are needed.


Update api and structure to support json schema

tl;dr: these PRs enable admins to customize info text, what questions are asked and how many questions are asked (with no need of Webkom supervision - hopefully)
Chunky changes, so I tried to spread it out a little to see if it makes it simpler to review. In case that is not the case, the complete code is available in branch https://github.com/webkom/admissions/tree/json-schema-complete. It might be necessary to run the code from this branch if you want to test it locally, as the codesplitting between the three PRs might be faulty.
This PR is the main bulk of functionality changes, and is required to make the other PRs run and make sense.

API and structure changes (this PR) #1179
User-view changes #1179
Admin-view changes #1180

There is still plenty to be desired, so this is in no way perfect - but this should be a decent MVP with all necessary functionality intact, and enough to merge it into staging and continue working on it.
Testing is also non-existent, but I'll get to that later..
Changes
Previously, admissions were hardcoded so every group had a response_label string - which is the text prompt that applying users see over the textarea. Each application and group application then had a stored text string, which was whatever the user wrote into the application.
Now, I have restructured it so that each group contains a questions JSON object, and each application and group application contains a responses JSON object.
The questions-object consists of what the group admins choose for it to be, configured through the admin panel (PR #1180).
So far this can either be a text (normal paragraph), textarea (long text input), textinput (short text) or a phoneinput (norwegian phone number). Each question contains an id, name and label that is used to display the question.
The responses-object consists of whatever was inserted into the application, in a pair of id and value - where the id matches the id from the corresponding question.
Example of the JSON bodies
Questions;
[{
    "type": "textarea",
    "id": "550e8400-e29b-41d4-a716-446655440000",
    "name": "Søknadstekst",
    "label": "Hvorfor har du lyst til å søke denne gruppen? Skriv litt om din motivasjon."
}]

Responses;
{"550e8400-e29b-41d4-a716-446655440000": "Jeg har kjempelyst ..."}

Resolves ABA-513

Although this seems like it would be a lot easier to do using pydantic (schema definition, types, validation etc) but this works so I won't force you to do anything. But worth keeping in mind for similar usecases later.

Wow thanks a lot, that looks like a much better option! Will take a look at it

Converted to draft to ensure it is not merged before 2023 admissions, will merge some time after that.

This was quite annoying to rebase at this point, so I'm closing it off for now - but keeping the branch so I can look into it at a later time, possibly changing the backend to use pydantic(:


helm oci image doesn't contain 'org.opencontainers.image.created'

Used helm version: v3.12.0
Pushing a helm chart to an oci-registry does not create the annotation org.opencontainers.image.created. This would be very helpful to add this annotation to the oci-image on helm push command.
status quo by pulling oci-image from a docker registry:
curl --location 'https://<dockerregistry>/v2/helm-charts/<imagename>/manifests/<imagetag>' --header 'Accept: application/vnd.docker.distribution.manifest.v2+json, application/vnd.docker.distribution.manifest.list.v2+json, application/vnd.oci.image.manifest.v1+json, application/vnd.oci.image.index.v1+json, */*'

{
    "schemaVersion": 2,
    "config": {
        "mediaType": "application/vnd.cncf.helm.config.v1+json",
        "digest": "sha256:0d69a88fd9c9e6b528314359b6a2ab69c1f407498c2ed27589a033173c5693ba",
        "size": 177
    },
    "layers": [
        {
            "mediaType": "application/vnd.cncf.helm.chart.content.v1.tar+gzip",
            "digest": "sha256:af3469083a43166ebdb85c8d5fdc25521fd10fb92488694cdbc7977f921a4d46",
            "size": 3399
        }
    ],
    "annotations": {
        "org.opencontainers.image.description": "Description",
        "org.opencontainers.image.title": "myImageName",
        "org.opencontainers.image.version": "myImageTag"
    }
}

org.opencontainers.image.created is missing
A possible code change would be on …
https://github.com/helm/helm/blob/4e447d87cd3222aa5e5ea2b38170e9d80b99c0b4/pkg/registry/util.go#LL196C1-L196C1
… by adding  line:
	chartOCIAnnotations = addToMap(chartOCIAnnotations, ocispec.AnnotationCreated, time.Now())

@sabre1041 for familiarity with OCI functionality

FWIW: This leads to different manifests every time the same artifact is uploaded a second time which is rather confusing. It would be useful to at least be able to either deactivate this or use a fixed timestamp.

As pauxes noted it leads to different hash of build artifact each time it builded, which causes non-indepotent behavior in bazel, which is bad for it.


print correct chip model (IDFGH-6031)

This example does not accommodate the various ESP32 chip models (it's either ESP32 or Unknown).
This change prints the correct ESP32 variant.

@davidzuhn this PR seems empty, have you recerted changes by mistake?


When using v-for, slides are not initialized

Hello, when I try to render slides using a loop, I get the "Cannot read property 'value' of undefined" error. After researching the code, I found the problem in the getChildren function.
When using v-for, vue wraps the slides in Symbol (Fragment), which causes the function to look for slots in the wrong place.
The same problem is observed when using the example from the documentation with virtual slides.
Tested in standard configuration vue-cli and hello-vue3.
ps sorry for my English

i have same problem querySelectorAll of underfined

try v-for on
<swiper ref="mySwiper" :options="swiperOptions">
<template v-for="(item, index) in list" >
           <swiper-slide></swiper-slide>
</template>
</swiper>

use v-if="data.length>1" on <swiper>, then v-for on <swiper-slide v-for="(item, index) in data" :key="key">


Support deletion of complete rows

Is there a strong reason for not being able to delete a complete row?
Currently the method HaeinsaTable.delete() checks that the size of the family map is not zero and throws an exception if that is the case. The comment above the precondition check states that it is not possible to delete entire rows because of the lock column.
But shouldn't it be possible to remove the complete row in the HaeinsaTable.makeStable() method? Instead of setting the lock state for this row to STABLE, the row is removed from HBase.
In case the row is a secondary row in the transaction, the primary row needs the information that this row needs to be deleted. During transaction recovery (in the state COMMITTED) it can check whether the row has been deleted or not. It can delete the row if the prewriteTimestamp matches (in order to not delete a concurrently added new version).
In case the row is the primary row in the transaction, it can be removed completely.
If there are two competing transactions, one trying to delete and the other trying to update the row, then the updating one would fail (if the other one was faster) because the checkAndPut() operation fails when the row no longer exists. For the other way round HBase's checkAndDelete() operation can be used.

Thank you for suggestion. At a glance, it seems to be possible to support deletion of whole row in Haeinsa with your solution. I will look deeper into the solution over this weekend.

Sorry for replying late.
I carefully investigated on this issue, and conclusion is: It is not able to support deletion of row with your solution. It is related with behavior of deletion of HBase: On executing delete operation, Hbase does not delete data immediately, it just leave delete tombstone. And it actually delete data during compaction.
Let assume that there several client on different machine are accessing to Hbase via Haeinsa. These local time of these machine can be different with other machines. In Haeinsa, to each client uses own local time of the machine to determine commit timestamp of the transaction. To adjust difference of local times of each machine, Haeinsa uses max value of all timestamps on each lock columns of the transaction.
See: HaeinsaTransaction.java#L232
So, delete entire row and not leaving lock column will cause problem: Deleted value can be read by client with younger local time, when accessing to the row after deleted.

There can be several possible solution to support deleting whole row in Haeinsa:

On deletion of row, just mark row as deleted on lock column, and delete actual data during major compaction. HBase supports running user defined codes on major compaction with coprocessors.
Do not rely only on timestamp on lock column to adjust timestamp of the transaction: use timestamp from something like timestamp oracle.

I will investigate further on this issue in the future.

Is it possible to do this if the transaction is just a single delete with the size of the family map being 0 (indicating desire to delete the entire row) and there is only a single row involved in the transaction? Then it would seem to be safe to a) do a read the current state of the '!lock!' column, if any, b) do one checkAndDelete that deletes all column families, including the '!lock!' column. I don't see why the concern about differing client clocks applies. The checkAndDelete will only work if the current value of !lock! matches what was read based on the server time at the time of the checkAndDelete. Even if a concurrent transaction from another client is trying to write new data, it would do a checkAndPut which would only succeed if it matches the value of the lock column read in that transaction, again atomically based on server time.

The intent of all this is I'm hoping it would be easy to optimize for a delete of the whole row if it is a single row with no secondary row.

Similar to what is already done to optimize for a the only mutation being a put on a single row.

In case anyone is interested, we have implemented this in our fork https://github.com/aol/haeinsa in the aol_master branch. We're using it in production and so far haven't seen any issues. It improved our performance when deleting single rows by a lot.
Cheers.


feature/FranceOID-2024

Added OID France dates and sponsorship amounts to events/sponsorship/#France

Added "keynote" in grey on Gold & Silver tiers.


On Android, when I open the app for the second time, the barStyle setting is invalid

Bug summary
In order to cooperate with the library react-native-bootsplash, a dark style is set in MainActivity: RNBars.init(MainActivity.this, "dark-content"); (this is no problem).
Then set the default light-content in App.tsx. After exiting the app, restart the app and you will see that the status bar style has changed to dark-content.
Library version
1.0.4
Environment info
System:
    OS: macOS 11.5.2
    CPU: (6) x64 Intel(R) Core(TM) i5-8500B CPU @ 3.00GHz
    Memory: 62.12 MB / 8.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 15.4.0 - /usr/local/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 7.0.15 - /usr/local/bin/npm
    Watchman: Not Found
  Managers:
    CocoaPods: 1.11.2 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.5, DriverKit 20.4, macOS 11.3, tvOS 14.5, watchOS 7.4
    Android SDK: Not Found
  IDEs:
    Android Studio: 4.2 AI-202.76<IP_ADDRESS>86908
    Xcode: 12.5/12E262 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.8 - /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/javac
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.2 => 17.0.2
    react-native: 0.66.3 => 0.66.3
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce
1.Modify App.tsx in the example: Change const [isLightStatus, setIsLightStatus] = React.useState(false); to const [isLightStatus, setIsLightStatus] = React.useState(true);.
2.Exit the app, restart the app. (Click the virtual back button, the app is still on the recently used app screen)
Reproducible sample code
I used your sample project to debug, so you only need to modify the code according to the reproduction steps.

@1280103995 Can you publish all your styles.xml?

@1280103995 Can you publish all your styles.xml? Give more info about how you tested it? (device, android version)

I cloned your repository and only modified the content of App.txs in the example, so you can read the source code for the content of style.xml.
My device is: Samsung Galaxy A51 (Android 11)
style.xml
<resources>
    <style name="AppTheme" parent="Theme.AppCompat.DayNight.NoActionBar">
        <!-- Allow drawing under the system bars background -->
        <item name="android:windowDrawsSystemBarBackgrounds">true</item>
        <item name="android:fitsSystemWindows">false</item>

        <!-- Set status bar background transparent -->
        <item name="android:statusBarColor">@android:color/transparent</item>

        <!-- Navigation bar will stay translucent on Android < 8.1 -->
        <item name="android:windowTranslucentNavigation">true</item>
    </style>
</resources>

styles-v27.xml
<resources xmlns:tools="http://schemas.android.com/tools">
    <style name="AppTheme" parent="Theme.AppCompat.DayNight.NoActionBar">
        <!-- Allow drawing under the system bars background -->
        <item name="android:windowDrawsSystemBarBackgrounds">true</item>
        <item name="android:fitsSystemWindows">false</item>

        <!-- Set system bars background transparent -->
        <item name="android:statusBarColor">@android:color/transparent</item>
        <item name="android:navigationBarColor">@android:color/transparent</item>

        <!-- Disable auto contrasted system bars background (on Android 10+) -->
        <item name="android:enforceStatusBarContrast" tools:targetApi="q">false</item>
        <item name="android:enforceNavigationBarContrast" tools:targetApi="q">false</item>
    </style>
</resources>

I guess the problem is here in StatusBar.tsx:
if (
      lastEntry != null &&
      // Update only if style have changed.
      (!oldProps || oldProps.barStyle !== lastEntry.barStyle)
) {

When I changed to: if (oldProps?.barStyle !== lastEntry.barStyle), this problem can be solved.
I found a new problem. Set light-content and dark-content on the two pages in BottomTabNavigator (react-navigation) respectively. After switching tabs back and forth, the status bar color will remain one color.

@1280103995 You removed the lastEntry != null check?

@1280103995你取消了lastEntry != null支票？
一个巧妙的解决方案可能是公开初始设置（在 中设置MainActivity）并propsStack在开始时将其放入。

Canceling lastEntry != null is just a preliminary judgment, and the condition of if needs more detailed debugging.

✅ Solved. The issue was a bit tedious: on pressing back to quit the app the propsStack was emptied (because all components are unmounted, each one will pop from the stack on componentWillUnmount), but mergedProps wasn't reset to null.
Now when calling updatePropsStack if the stack size is zero, mergedProps is resetted.
Update and check by yourself 🙂

On v1.1.0, this issue has been resolved.:clap::clap:
Some extra words (not related to the library): Setting up a transparent Bar on Android is indeed a difficult task. :sweat_smile:
For example:
1.TextInput is covered by the keyboard.
2. UI rendering is inaccurate due to the height of the Bar.

Use react-native-safe-area-context. It exposes a hook to update insets dynamically 🙂


refactor: json rpc types in Core and Transact

Summary

remove TonfuraConfig param id
remove httpClient in Core and Transact
add sendJsonRpcRequest in TonfuraProvider
refactor json rpc types
fix the method of getBlockTransactions
update validator of runGetMethod

Type of change

[ ] 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 not work as expected)
[ ] This change requires a documentation update
[x] Refactor (no functionality changes)

Test Steps

Checklist

[x] Remove unnecessary console or comment
[ ] Add unit tests

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Connected successfully after 5 to 8 mins later disconnect issue occurs and not able to auto reconnect .

We get onError method error type CentrifugeError.transportError The operation couldn’t be completed. (SwiftCentrifuge.WSError error 1.) but our app is live but not able to reproduce issue on debug mode which reason to this disconnect issue occurs not able to reconnect. so we are facing issue. did not get proper error reason so it very difficulte how to fix it this issue.
If we log using this event.error.localizedDescription then we get The operation couldn’t be completed. (SwiftCentrifuge.CentrifugeError error 0.)
        if let centrifugeError = event.error as? CentrifugeError {
            switch centrifugeError {
            case .timeout:
                errMsg = "timeout"
            case .duplicateSub:
                errMsg = "duplicateSub"
                return
            case .clientDisconnected:
                errMsg = "clientDisconnected"
                return
            case .subscriptionUnsubscribed:
                errMsg = "subscriptionUnsubscribed"
                return
            case .transportError(let err):
                errMsg = "transportError: \(err.localizedDescription)"
            case .tokenError(let err):
                errMsg = "tokenError: \(err.localizedDescription)"
            case .connectError(let err):
                errMsg = "connectError: \(err.localizedDescription)"
            case .refreshError(let err):
                errMsg = "refreshError: \(err.localizedDescription)"
            case .subscriptionSubscribeError(let err):
                errMsg = "subscriptionSubscribeError: \(err.localizedDescription)"
            case .subscriptionTokenError(let err):
                errMsg = "subscriptionTokenError: \(err.localizedDescription)"
            case .subscriptionRefreshError(let err):
                errMsg = "subscriptionRefreshError: \(err.localizedDescription)"
            case .replyError(let code, let message, _):
                errMsg = "replyError: code: \(code) message:\(message)"
            }
        }
        if errMsg.notEmpty {
            errMsg += "; localizedDescription: \(event.error.localizedDescription)"
        } else {
            errMsg = event.error.localizedDescription
        }```

 Error message got while log on amplitude.
 `"transportError: The operation couldn’t be completed. (SwiftCentrifuge.WSError error 1.); localizedDescription: The operation couldn’t be completed. (SwiftCentrifuge.CentrifugeError error 0.)"`

Hello @sandeep-yudiz
For now I don't have any idea where it can come from, we don't have this message inside centrifuge-swift source code. Could you provide more details:

Version of centrifuge-swift? Version of server?
Does this happen rarely on some devices? Or every time? Any special details you may provide?
which reason to this disconnect issue occurs not able to reconnect - this is not very clear to me, could you elaborate more – after getting the error websocket disconnects? are you getting disconnect callback? if yes, does centrifuge-swift reconnects after disconnecting?

Hello @FZambia

Version of centrifuge-swift
=> SwiftCentrifuge (0.5.5),  Server version 3.2.0

Does this happen rarely on some devices? Or every time? Any special details you may provide?
=> Does this happen  on some users devices. Maybe this data will help you!

We tracking error on the methods with try to extract data from error.
func onError(_ client: CentrifugeClient, _ event: CentrifugeErrorEvent) {
        var errMsg = ""
        if let centrifugeError = event.error as? CentrifugeError {
            switch centrifugeError {
            case .timeout:
                errMsg = "timeout"
            case .duplicateSub:
                errMsg = "duplicateSub"
                return
            case .clientDisconnected:
                errMsg = "clientDisconnected"
                return
            case .subscriptionUnsubscribed:
                errMsg = "subscriptionUnsubscribed"
                return
            case .transportError(let err):
                errMsg = "transportError: \(err.localizedDescription)"
            case .tokenError(let err):
                errMsg = "tokenError: \(err.localizedDescription)"
            case .connectError(let err):
                errMsg = "connectError: \(err.localizedDescription)"
            case .refreshError(let err):
                errMsg = "refreshError: \(err.localizedDescription)"
            case .subscriptionSubscribeError(let err):
                errMsg = "subscriptionSubscribeError: \(err.localizedDescription)"
            case .subscriptionTokenError(let err):
                errMsg = "subscriptionTokenError: \(err.localizedDescription)"
            case .subscriptionRefreshError(let err):
                errMsg = "subscriptionRefreshError: \(err.localizedDescription)"
            case .replyError(let code, let message, _):
                errMsg = "replyError: code: \(code) message:\(message)"
            }
        }
        if errMsg.notEmpty {
            errMsg += "; localizedDescription: \(event.error.localizedDescription)"
        } else {
            errMsg = event.error.localizedDescription
        }
        TrackingManager.sendEvent("error_centrifuge", eventProperties: ["error": errMsg])
  }

which reason to this disconnect issue occurs not able to reconnect - this is not very clear to me, could you elaborate more – after getting the error websocket disconnects? are you getting disconnect callback? if yes, does centrifuge-swift reconnects after disconnecting?
=> When user received above error they are disconnected but not able to reconnect on `SwiftCentrifuge.CentrifugeError error 0. User need to kill the app and reopen the app.

Version of centrifuge-swift
=> SwiftCentrifuge (0.5.5), Server version 3.2.0

Starting from v0.5.0 this SDK does not support Centrifugo < v4. See release notes https://github.com/centrifugal/centrifuge-swift/releases/tag/0.5.0, and also this is stated in the README. There were many changes in protocol.
I am not sure that the error will disappear upon using proper versions – it may be normal actually as network errors happen. But probably SDK will reconnect properly.


Exposes Kendra result item DocumentAttributes in the document metadata

Description: exposes the ResultItem DocumentAttributes as document metadata with key 'document_attributes' and refactors AmazonKendraRetriever by providing a ResultItem base class in order to avoid duplicate code;
Tag maintainer: @3coins @hupe1980 @dev2049 @baskaryan
Twitter handle: wilsonleao

Why?
Some use cases depend on specific document attributes returned by the retriever in order to improve the quality of the overall completion and adjust what will be displayed to the user. For the sake of consistency, we need to expose the DocumentAttributes as document metadata so we are sure that we are using the values returned by the kendra request issued by langchain.
I would appreciate your review @3coins @hupe1980 @dev2049. Thank you in advance!
References

Amazon Kendra DocumentAttribute
Amazon Kendra DocumentAttributeValue

I will propose more points of refactoring soon in separate PRs/issues. Thank you in advance for your review and all the work moving this forward!

@3coins
Thank you very much for the review!


Client Allocation Request for: Filecoin Project

Client Allocation Questions
Core Information

Name: Filecoin Project
Website / Social Media: https://filecoin.io/
Region: North America
DataCap Requested: 6TiB
Addresses to be Notarized: f1wdxdpqh3hirrhp353i4o6ld7bsw6evh3v7i5jtq
Notary Requested: neogeweb3

Request Approved
Your Datacap Allocation Request has been approved by the Notary
Message sent to Filecoin Network

bafy2bzacedpyzdaytkt6bntkbtpdeq3sdlibjhz3d5cu3s3shnohj7soni7xg

Address

f1wdxdpqh3hirrhp353i4o6ld7bsw6evh3v7i5jtq

Datacap Allocated

6TiB

You can check the status of the message here: https://filfox.info/en/message/bafy2bzacedpyzdaytkt6bntkbtpdeq3sdlibjhz3d5cu3s3shnohj7soni7xg


Implement Kolmogorov Smirnov Test in SQL-only

Hey!
in this PR I'd like to refactor the work of #23 and now implement the Kolmogorov-Smirnov test in SQL-only.
This has the benefit that for large datasets, we won't have to load in all the values to pass them to scipy.
Additionally, this would remove scipy as a dependency again! 🥳
The core work of this PR consists of creating an SQL query that is able to calculate the Kolmogorov-Smirnov test.
The idea behind this implementation as well as some theoretical backgrounds will be explained in the comments below.
The following issues will definitely need to be addressed:

[ ] Create more tests, especially with very large datasets
[ ] Refactor tests as p_values are not always available now
[ ] Remove scipy dependency
[ ] Test if the query is really database-agnostic, i.e., follows some SQL standard
[ ] Transform query to sqlalchemy's query API
[ ] Validate p-value approximation

Kolmogorov Smirnov in SQL
Our sample table table1 has the following CDF

VAL
CDF

1
0.2352941176

2
0.3529411765

3
0.4705882353

4
0.5882352941

5
0.7058823529

6
0.8235294118

7
0.9411764706

8
1

which is calculated by the first query and stored in tab1. The same applies to tab2.
Since the range of the values in table1 might differ from the range of values in table2, we perform a FULL OUTER JOIN between the CDFs based on their index (here: val, i.e., the x-axis).
SELECT coalesce(tab1.val, tab2.val) as v, tab1.cdf as cdf1, tab2.cdf as cdf2
FROM tab1 FULL OUTER JOIN tab2 ON tab1.val = tab2.val
ORDER BY v

Using coalesce we combine the indices, which results in the following table:

V
CDF1
CDF2

1
0.1666666667
0.2352941176

2
0.2222222222
0.3529411765

3
0.2777777778
0.4705882353

4
0.3333333333
0.5882352941

5
0.3888888889
0.7058823529

6
0.4444444444
0.8235294118

7
0.5
0.9411764706

8

1

16
0.5555555556

17
0.6111111111

18
0.6666666667

19
0.7222222222

20
0.7777777778

21
0.8333333333

22
0.8888888889

23
0.9444444444

24
1

Filling missing values
Since our discrete values result in a step-function like CDF, we want to forward-fill missing values in the table.
This is done by first assigning a grouper_id to each (index, cdf_value) combination with
grouped_table AS ( -- Step 2: Create a grouper attribute to fill values forward
    SELECT v,
        COUNT(cdf1) over (order by v) as _grp1, cdf1,
        COUNT(cdf2) over (order by v) as _grp2, cdf2
    FROM cdf_unfilled

and then retrieving the first value per group here
filled_cdf AS ( -- Step 3: Select first non-null value per group (defined by grouper)
    SELECT v, first_value(cdf1) over (partition by _grp1 order by v) as cdf1_filled,
            first_value(cdf2) over (partition by _grp2 order by v) as cdf2_filled
    FROM grouped_table

Replacing NULLs
As long as the smallest value of both data samples is not the same, e.g., column1 starts with 1 and column2 starts with 5, one of the CDF tables will have NULL as first value. The following query replaces those resulting NULL values with 0, s.t. the distance can be effectively calculated.
Final Query
The final query has, currently, the following form.
Some of the queries can still be optimized. The current structure allows us to just insert datajudge's get_clause into the query string to have a filtered table and use a val alias on the target column.
WITH tab1 AS (
  SELECT val, MAX(cdf) as cdf FROM (
      SELECT val, cume_dist() over (order by val) as cdf
      FROM (SELECT {column1} as val FROM {table1}) -- Change TABLE and COL here
      ORDER BY val
  ) GROUP BY val

), tab2 AS (
  SELECT val, MAX(cdf) as cdf FROM (
      SELECT val, cume_dist() over (order by val) as cdf
      FROM (SELECT {column2} as val FROM {table2}) -- Change TABLE and COL here
      ORDER BY val
  ) GROUP BY val

), cdf_unfilled AS (
  SELECT coalesce(tab1.val, tab2.val) as v, tab1.cdf as cdf1, tab2.cdf as cdf2
  FROM tab1 FULL OUTER JOIN tab2 ON tab1.val = tab2.val
  ORDER BY v

), grouped_table AS ( -- Step 2: Create a grouper attribute to fill values forward
    SELECT v,
        COUNT(cdf1) over (order by v) as _grp1, cdf1,
        COUNT(cdf2) over (order by v) as _grp2, cdf2
    FROM cdf_unfilled

), filled_cdf AS ( -- Step 3: Select first non-null value per group (defined by grouper)
    SELECT v, first_value(cdf1) over (partition by _grp1 order by v) as cdf1_filled,
            first_value(cdf2) over (partition by _grp2 order by v) as cdf2_filled
    FROM grouped_table

), replaced_nulls AS ( -- Step 4: Replace NULL values (often at the begin) with 0 to calculate difference
  SELECT coalesce(cdf1_filled, 0) as cdf1, coalesce(cdf2_filled, 0) as cdf2
  FROM filled_cdf
  )

SELECT MAX(ABS(cdf1-cdf2)) FROM replaced_nulls; -- Step 5: Calculate final statistic

Theoretical foundation for p-value
The query above only calculates the statistic of the Kolmogorov Smirnov test, which is defined as
$$\left[ \sup_{x} || F_1(x) - F_2(x) || \right] $$
where $F_1(x)$ and $F_2(x)$ are the CDFs of the two samples $s_1$ and $s_2$ respectively.

Determining acceptance
We can now accept or reject the null-hypothesis $H_0$ of our test based on our test statistic $d$ and our sample sizes $n$ and $m$ with the following inequality:
$$ d \lt \sqrt{-\ln{\frac{\alpha}{2} * \frac{1}{2}}} $$
In this case, $H_0$ states that the two samples stem from the same distribution.
Approximating the p-value
Since this value is test-specific and harder to interpret across applications, it would be beneficial to have a p-value at hand!
Unfortunately, the distribution of the KS-statistic is unwieldly, making exact calculations hard. (Facchinetti, 2007) describes the following formula as an approximation of the p-value given $d$ and sample size $n$ for samples with $n \gt 35$:
$$ 2*\exp^{-(d*\sqrt(n))^2} $$
After experimenting with a few test datasets, both very small (~50 samples) and very large (~100,000,000 samples), I can confirm that this approximation is mostly correct for the first few decimal places. (Very unscientific, I know).
Implementation:
For samples with $n \lt 35$ we won't provide a p-value for now. Also accepting or rejecting the test will still be based on the above inequality, the approx. p-value is solely for reporting purposes.

Great job!

The following issues will definitely need to be addressed:

[ ]  Create more tests, especially with very large datasets

I'm not sure how to deal with tests of large datasets. Maybe those will need to be executed manually on demand.

[ ]  Test if the query is really database-agnostic, i.e., follows some SQL standard

Well, if you write the tests and it passes on the three databases we test against, then you're done.

[ ]  Transform query to sqlalchemy's query API

I don't think it'll be very feasible or advisable to attempt that. ORMs have some limitations and sometimes going straight to SQL is necessary and advisable.
Anyhow, besides some minor issues that I'll point out later when this PR leaves draft mode, all looks good. Perhaps try to be more consistent in the type annotations including return values.

I don't think it'll be very feasible or advisable to attempt that. ORMs have some limitations and sometimes going straight to SQL is necessary and advisable.

Maybe you @YYYasin19 meant the sqlalchemy language expression api? We use that in the rest of db_access. We don't use the ORM api.

I don't think it'll be very feasible or advisable to attempt that. ORMs have some limitations and sometimes going straight to SQL is necessary and advisable.

Maybe @YYYasin19 meant the sqlalchemy language expression api? We use that in the rest of db_access. We don't use the ORM api.

Duh, sorry. Still, in this case, I'd advise against it. I'm not opposed to it, just that don't think it's worth the effort.

Is there a reason why our MSSQL code is converting the table specification in ref.get_selection() into a string with quotation marks, like this "tempdb.dbo".int_table1.col_int ?
This breaks the query for now, other than that, everything seems to work 🥳

why our MSSQL code is converting the table specification in ref.get_selection() into a string with quotation marks, like this

I'm not perfectly sure - I would've thought that get_selection returns a selection rather than a string but I might be missing something here.
Independently of that, I know that we sometimes ran into trouble with mssql when it came to referring to objects. E.g.:

Table names can contain any valid characters (for example, spaces). If table names contain any characters except letters, numbers, and underscores, the name must be delimited by enclosing it in back quotes (`).

https://docs.microsoft.com/en-us/sql/odbc/microsoft/table-name-limitations?view=sql-server-ver16

I'm not perfectly sure - I would've thought that get_selection returns a selection rather than a string but I might be missing something here.

Yes, it does, I just had wrapped a str(...) around it 😬 .
I now have a solution that is very sub-optimal as I:
(1) replace " in MS-SQL table definitions manually
    if is_mssql(engine):
        table1_selection = str(table1_selection).replace('"', "")
        table2_selection = str(table2_selection).replace('"', "")

Note: sa.select(...) is the culprit here and returns "tempdb.dbo".table_name instead of tempdb.dbo.table_name.
(2) rely on data_source.table_name, which is not available for RawQueryDataSources
Also, this would not include any where-clauses, which makes it straight up wrong in many cases.

How do I find out if my approximation of the p-value is good enough?
Left is the approximation, right is the "correct" one from scipy
0.00035221594346540835 vs 0.00035250250849476917
Is it feasible to check whether the user has scipy installed and if so, just use scipy?

How do I find out if my approximation of the p-value is good enough? Left is the approximation, right is the "correct" one from scipy 0.00035221594346540835 vs 0.00035250250849476917 The difference is 2.8656502936081404e-07

Usually, the level of "precision" shown by calculators and computers is somewhat artificial. I'm not a statistician, but I'd say that anything any difference beyond e-06, in this case, is irrelevant. Usually one uses the p value of 0.05 or if you want to be very picky 0.01. A difference in the calculated value via SQL and scipy like you find will not make anyone accept or reject the null hypothesis.

Is it feasible to check whether the user has scipy installed and if so, just use scipy?

Well, no... this is only executed in test. Users would never use this, so it's irrelevant if users have scipy installed or not.

Usually one uses the p value of 0.05 or if you want to be very picky 0.01. A difference in the calculated value via SQL and scipy like you find will not make anyone accept or reject the null hypothesis.

The problem is that in practice for one of our tests the reported p-value in the magnitude of 1e-53 because the table has 100+ million rows. Therefore, my idea to use scipy in very scientific environments where accuracy is required.

Well, no... this is only executed in test.
No, I mean in production.

In the end, it would just be an extra statement within the approximation method
try:
   import scipy
except ModuleNotFoundError:
   return # old method of approximation

# accurate method
return scipy.stats.distribution.kstwo.sf(d, n_samples)

I think that's feasible 👍

Usually one uses the p value of 0.05 or if you want to be very picky 0.01. A difference in the calculated value via SQL and scipy like you find will not make anyone accept or reject the null hypothesis.

The problem is that in practice for one of our tests the reported p-value in the magnitude of 1e-53 because the table has 100+ million rows. Therefore, my idea to use scipy in very scientific environments where accuracy is required.

Mh, that's a problem... and it's a rather generalized issue with p-values in general.
Although I did try to look for possible solutions, I didn't find any.
One idea would be to base the data test around the value D, the maximum distance between the samples. Then the test receives a threshold for D that needs to be fulfilled. Now, I agree that this solution is not very satisfactory as it just kicks the problem to define a sensible threshold and at least I personally have no idea how to interpret D enough to be able to choose a number for the threshold.

One idea would be to base the data test around the value D, the maximum distance between the samples. Then the test receives a threshold for D that needs to be fulfilled. Now, I agree that this solution is not very satisfactory as it just kicks the problem to define a sensible threshold and at least I personally have no idea how to interpret D enough to be able to choose a number for the threshold.

My experience is only anectodal as well, but from what I've seen for various sample sizes (10, 10_000 and 100_000_000) the test statistic d doesn't get too small. Therfore, I chose 1e-10 as allowed deviation.

Great work!


Rebar radius wrong direction

Describe the bug 📝
I'm loading ifc models in IFCjs demo and on the Open viewer https://platform.thatopen.com/app.
I notice that rebar radius is wrong in the model making weird shapes.
That Open Viewer:

IFCjs Viewer:

Revit:

Reproduction ▶
https://spoerering-my.sharepoint.com/:u:/g/personal/gjordan_btd_cl/EdwsR5y7QzxFr0PVX32QOp4BmpYKgAnDTVHb62cuxUTtTA?e=EPzYdF
Steps to reproduce 🔢

Download IFC model from link.
Load IFC on https://platform.thatopen.com/app
Load IFC on https://ifcjs.github.io/web-ifc-viewer/example/index

System Info 💻
System:
    OS: Windows 10 10.0.22621
    CPU: (24) x64 12th Gen Intel(R) Core(TM) i9-12900K
    Memory: 15.71 GB / 31.75 GB
  Binaries:
    Node: 19.8.1 - C:\Program Files\nodejs\node.EXE
    npm: 8.15.1 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.22621.2283.0), Chromium (117.0.2045.41)
    Internet Explorer: 11.0.22621.1

Used Package Manager 📦
npm
Error Trace/Logs 📃
No response
Validations ✅

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a IFC.js components issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
[X] Check that this is a concrete bug. For Q&A join our Community.
[X] The provided reproduction is a minimal reproducible example of the bug.

Hey @gjordanp sorry for the delay! The link has expired, would you mind sending it again so that we can work on this? Thanks in advance! 🙏

Hey @gjordanp sorry for the delay! The link has expired, would you mind sending it again so that we can work on this? Thanks in advance! 🙏

Hi, this is a new link. I have also edit the
https://spoerering-my.sharepoint.com/:u:/g/personal/gjordan_btd_cl/EdwsR5y7QzxFr0PVX32QOp4BQ9f4RPfr6qqftfIkcp08nA?e=aJR0xn
Regards


fix(pom): version in coverage reports

fix(pom): version in coverage reports

:tada: This PR is included in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


⚠ Bofrost.fr has degraded performance

In 5179b43, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 4842 ms

Resolved: Bofrost.fr performance has improved in 27f08c1.


Add new service support: persy

Persy - Transactional Persistence Engine
Simple single file durable, isolated, consistent, and atomic persistence engine based on copy on write. It guarantee the persistence of data even in case of crash through a in file write ahead log, and provide simple rust APIs for store simple bytes or value associated (index) data, provide also memory management logic to keep the file size in check.

I can try to work on this.

I can try to work on this.

Glad to see you're willing to help, but I'm sorry that I haven't synced the status in time recently. This feature is almost implemented. I'll submit a PR shortly and maybe you'd like to help review it and improve this service together.

@PsiACE I'm glad to help review this feature when you submit the pr. Can you assign me a similar introduction level task?

@PsiACE I'm glad to help review this feature when you submit the pr. Can you assign me a similar introduction level task?

Hi, maybe you will be interested in https://github.com/apache/incubator-opendal/issues/2500? Implement append for hdfs is similiar to fs.

@Xuanwo I can work on this. Please assign it to me.

@Xuanwo I can work on this. Please assign it to me.

Can you leave a comment under that issue? Otherwise, I won't be able to assign it to you.


After maximizing window, enabling borderless does not cover the entire screen

Godot version
3.4.4
System information
Windows 10, GLES3, Intel HD Graphics 620
Issue description
https://i.imgur.com/wiUEReB.png (the open space while game is running)
There's a little bit of space on the bottom and right side and for some reason it goes off screen at the top.
Not doing anything special just turning on borderless in-game after maximizing.
I tried setting window size manually but then the borderless window appears half off screen towards the bottom right, similar to what happens when I don't have hidpi turned on for normal windowed game. After changing the window size numbers I still can't get it to look just right.
Steps to reproduce
Maximize window>borderless through code
Minimal reproduction project
No response

Still happens on 3.5, will check again once 4.0 is out

@return2monki Could you please make a better screenshot showing the problem, for example a full screen image?

@marzecdawid
https://i.imgur.com/qmjUynn.png - maximized (normal)
https://i.imgur.com/p6RmWaz.png - from maximized to borderless
I think borderless going upwards off screen and leaving some empty space is wrong behaviour? What I want to happen is that it fills the screen excluding task bar. But I couldn't get it to work by manually setting values.

Fixed in Godot 4 by seperating fullscreen and exclusive fullscreen (although it caused some new issue with not being able to revert fullscreen into windowed original position after maximizing it seems)


[NEMO-3] Bump up the Beam version to 2.5.0

JIRA: NEMO-3: Bump up the Beam version to 2.5.0
Major changes:

Bump up Beam version to 2.5.0
Add missing methods in DoFn.ProcessContext including OutputReceiver
Adapt to changes in connecting side inputs (See CreateViewTransform and NemoPipelineVisitor)

Minor changes to note:

Handle null key with HashPartitioner

Tests for the changes:

Update optimizer tests since the DAG with side inputs look different

Other comments:

Window or pane in DoFn.ProcessContext may be refined for further supports for unbounded and windowed sources and processing.

resolves NEMO-3

Thanks for the reviews. I think something went wrong, nevertheless I will check your comments for improvements.


[google_sign_in] Move plugin to its subdir to allow for federated implementations

Description

Move google_sign_in/* to google_sign_in/google_sign_in to prepare for federated implementations.
Update links in docs to point to the new location.
Bump version, and update changelog.

Related Issues

https://github.com/flutter/flutter/issues/43772

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

(Force-pushed after rebasing)

Part 2/4: https://github.com/ditman/plugins/pull/99


feat(repodb): implement a DB for image specific information using boltdb

(cherry picked from commit e3cb60b8566d84158409815325083ed149175063)
Some other fixes/improvements on top (Andrei)
Global search: The last updated attribute on repo level is now computed correctly. Global search: Fix and enhance tests: validate more fields, and fix CVE verification logic RepoListWithNewestImage: The vendors and platforms at repo level are no longer containing duplicate entries CVE: scan OCIUncompressedLayer instead of skiping them (used in tests) bug(repodb): do no try to increment download counters for signatures
Signed-off-by: Andrei Aaron<EMAIL_ADDRESS>Add filtering to global search API (Laurentiu)
(cherry picked from commit a87976d635ea876fe8ced532e8adb7c3bb24098f)
Original work by Laurentiu Niculae<EMAIL_ADDRESS>Fix pagination bug

when limit was bigger than the repo count result contained empty results
now correctly returns only maximum available number of repo results

Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>Add history to the fields returned from RepoDB
Consolidate fields used in packages

pkg/extensions/search/common/common_test
pkg/extensions/search/common/common Refactor duplicate code in GlobalSearch verification Add vulnerability scan results to image:tag reply

Signed-off-by: Andrei Aaron<EMAIL_ADDRESS>Refactor ExpandedRepoInfo to using RepoDB
Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>(cherry picked from commit fd7dc85c3a9d028fd8860d3791cad4df769ed005)
Init RepoDB at startup

sync with storage
ignore images without a tag

Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>(cherry picked from commit 359898facd6541b2aa99ee95080f7aabf28c2650)
Update request to get image:tag to use repodb
Signed-off-by: Andrei Aaron<EMAIL_ADDRESS>Sync RepoDB logging

added logging for errors

Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>(cherry picked from commit 2e128f4d01712b34c70b5468285100b0657001bb)
sync-repodb minor error checking fix
Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>Improve tests for syncing RepoDB with storage
Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>(cherry picked from commit b18408c6d64e01312849fc18b929e3a2a7931e9e)
Update scoring rule for repos

now prioritize matches to the end of the repo name

Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>(cherry picked from commit 6961346ccf02223132b3b12a2132c80bd1b6b33c)
Upgrade search filters to permit multiple values

multiple values for os and arch

Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>(cherry picked from commit 3ffb72c6fc0587ff827a03fe4f76a13b27b876a0)
feature(repodb): add pagination for RepoListWithNewestImage
Signed-off-by: Alex Stan<EMAIL_ADDRESS>(cherry picked from commit 32c917f2dc65363b0856345289353559a8027aee)
test(fix): fix tests failing since repodb is used for listing all repos

One of the tests was verifying disk/oci related erros and is not applicable
Another test was actually broken in an older PR, the default store and the substore were using the same repo names (the substore ones were unprefixed), which should not be the case, this was causing a single entry to show in the RepoDB instead of two separate entries for each test image Root cause in: https://github.com/project-zot/zot/commit/b61aff62cd5088314cee39fae42cec0ad855bd7c#diff-b86e11fa5a3102b336caebec3b30a9d35e26af554dd8658f124dba2404b7d24aR88

Signed-off-by: Andrei Aaron<EMAIL_ADDRESS>chore: move code reponsible for transforming objects to gql_generated types to separate package
Signed-off-by: Andrei Aaron<EMAIL_ADDRESS>Process input for global search

Clean input: query, filter strings
Add validation for global search input

Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>(cherry picked from commit f1ca8670fbe4a4a327ea25cf459237dbf23bb78a)
fix: only call cve scanning for data shown to the user
Signed-off-by: Andrei Aaron<EMAIL_ADDRESS>GQL omit scanning for CVE if field is not required
Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>(cherry picked from commit 5479ce45d6cb2abcf5fbccadeaf6f3393c3f6bf1)
Fix filtering logic in RepoDB

filter parameter was set to false instead of being calculator from the later image

Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>(cherry picked from commit a82d2327e34e5da617af0b7ca78a2dba90999f0a)
bug(repodb): Checking signature returns error if signed image is not found

we considere a signature image orfan when the image it signs is not found
we need this to ignore such signatures in certain cases

Signed-off-by: Laurentiu Niculae<EMAIL_ADDRESS>(cherry picked from commit d0418505f76467accd8e1ee34fcc2b2a165efae5)
feat(repodb): CVE logic to use repoDB
Also update some method signatures to remove usage of: github.com/google/go-containerregistry/pkg/v1
Signed-off-by: Andrei Aaron<EMAIL_ADDRESS>(cherry picked from commit c2bfab348c468e2f850c9f683f336856c9e4b55d)
What type of PR is this?
feature
Which issue does this PR fix:
Implement a DB to store image and repo related information. GraphQL requests should not read manifests/configs directly from the disk. They should read from this DB instead.
There is also additional data which is not present in the oci layout, such as download count.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Resolved conflicts, but the comments were not addressed yet.

Closing in favor of https://github.com/project-zot/zot/pull/979/commits


Video Conference Mute Hiding Toolbar and Icons

Hiding toolbar
There should be an option so we can make the toolbar hide after like 1 second we turn it on/off.
Currently if the microphone/camera is off the toolbar just stays there and takes up space
Icons
There should be an option to make it only show the icons without the Microphone off/Camera off text
If you'd like to see this feature implemented, add a 👍 reaction to this post.

Already tracked in #6374

Already tracked in #6374


Deprecation that will Stop Working in HA 2025.1

I get this message about Mopidy on startup:
2024-11-06 10:17:38.471 INFO (MainThread) [homeassistant.components.media_player] Setting up mopidy.media_player
2024-11-06 10:17:38.471 INFO (MainThread) [custom_components.mopidy.speaker.api] Creating Mopidy Websocket connection...
2024-11-06 10:17:38.477 WARNING (MainThread) [homeassistant.helpers.frame] Detected code that calls async_forward_entry_setup for integration mopidy with title: Mopidy Dining Room and entry_id: 01JC10EYMGFK0TE16HYMZTMA1Q, during setup without awaiting async_forward_entry_setup, which can cause the setup lock to be released before the setup is done. This will stop working in Home Assistant 2025.1. Please report this issue.

I presume this stack trace is related?  It was the next entry in the log
Stack (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/usr/src/homeassistant/homeassistant/__main__.py", line 223, in <module>
    sys.exit(main())
  File "/usr/src/homeassistant/homeassistant/__main__.py", line 209, in main
    exit_code = runner.run(runtime_conf)
  File "/usr/src/homeassistant/homeassistant/runner.py", line 189, in run
    return loop.run_until_complete(setup_and_run_hass(runtime_config))
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 674, in run_until_complete
    self.run_forever()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 641, in run_forever
    self._run_once()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1990, in _run_once
    handle._run()
  File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2223, in async_forward_entry_setup
    _report_non_awaited_platform_forwards(entry, "async_forward_entry_setup")
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 1192, in _report_non_awaited_platform_forwards
    report(
  File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 155, in report
    _LOGGER.warning(msg, stack_info=True)

Duplicate of https://github.com/bushvin/hass-integrations/issues/70


The mock web server isn't always initialized when the first request is issued

The local web server which acts as the mocked Internet, isn't always initialized when the first network request is issued. This can be seen if Atlantis is initialized in a custom Application object, while the first request is issued from the main Activity, say in the onCreate method. The symptoms typically mention "Invalid Host" exceptions etc.

A workaround for now seems to be to "back" or "home" out of the Activity and launch it again from the home screen (since Atlantis now already is initialized, as the Application object is created and exists in memory, the main Actvitity's request is properly intercepted and handled).


Update Badges

Updating the build status, code climate, and coveralls badges to use the Organization repo.

Thanks @adback03!
Now we need to configure those services here as well.

Yeah, we'll want to check into the Travis build as well - it seems to be struggling a bit.


Bump yard to >=0.9.36 for CVE-2024-27285

@bdunne Please review.
Note that 0.9.36 is the actual minimum despite the ruby-advisory-db saying 0.9.35.
See

https://github.com/ManageIQ/manageiq/actions/runs/8113492082/job/22176923374#step:8:18
https://github.com/rubysec/ruby-advisory-db/pull/761

Manually backported to radjabov via d655566c8659338b10023694418d21f09916cb37 and c7f2d8d4ee8cb97246474e7adf6c6bc416038cd9
commit c7f2d8d4ee8cb97246474e7adf6c6bc416038cd9 (HEAD -> radjabov, upstream/radjabov)
Author: Jason Frey<EMAIL_ADDRESS>Date:   Tue Mar 5 14:59:27 2024 -0500

    Update yard after backport of #22933

commit d655566c8659338b10023694418d21f09916cb37
Author: Brandon Dunne<EMAIL_ADDRESS>Date:   Tue Mar 5 14:55:10 2024 -0500

    Merge pull request #22933 from Fryguy/bump_yard

    Bump yard to >=0.9.36 for CVE-2024-27285

    (cherry picked from commit 02c69f1247ea911a3598a8cfb84d0556f51225c9)


daytona OpenSorce

If Daytona is open-source then where is the source code of Daytona website, if you can provide then others can use your template atleast.

Hi @yesbhautik, thanks for reaching out.
We are excited to share that the Daytona website repo will be public very soon. We don't have the exact date yet but you can expect it in the following weeks.
I'll close this issue now since the daytona repo is unrelated to the Daytona website repo.

We're working on one AI product. Product is almost ready also have some customer in our beta. Now we're moving forward to the landing page.
Daytona's website looks good + it is open-source.
We're waiting for many weeks, like when Daytona will release the source code of the website officially.
Landing page and marketing is pending because of this
Please provide source code ASAP if possible.

The Daytona website serves as the public face of our project and brand. Allowing others to copy it wholesale for their own products, especially commercial ones, could create confusion in the market and imply an endorsement or affiliation that doesn't exist. It's important to us to maintain control over our brand representation.
We do still plan to open source the website code in the near future, as a learning resource and to get community contributions. But we'll likely do so under a license that preserves our trademark and brand (e.g. you can fork and learn from the code but not launch a site that looks exactly like ours).
As an open-source project, we value collaboration and mutual benefit. However, we need to clarify that we can't provide the source code until the official release, which is still in progress.
We understand your eagerness, but we cannot rush the process. We'll make sure to announce the release of the website repo as soon as it's ready.
In the meantime, if you're interested in contributing to the project or learning more about our open-source philosophy, please let us know. We're happy to have you on board!