e2e: fix broken ansible playbook (ubuntu)

Fix broken Ansible playbook (Ubuntu host).

Based on this, I guess that we don't run Ubuntu based e2e on our daily jobs @klihub . If that's true, perhaps we should think weather it would make sense to add it too or would it be too expensive to maintain it.

Based on this, I guess that we don't run Ubuntu based e2e on our daily jobs @klihub . If that's true, perhaps we should think weather it would make sense to add it too or would it be too expensive to maintain it.

If you don't specify a distro, the e2e tests default to fedora. In our nightly runs we don't override it. Whether it would be too expensive to run/maintain tests also on Ubuntu depends on whether we consider it useful enough to put in the extra effort to keep it up to date. Our e2e tests are functional tests so I don't see that running them on Ubuntu as well would contribute much to the original purpose per se. That said, if we want to keep our e2e test set runnable with a simple additional distro=ubuntu-x.y command line argument, then it probably would make sense to run some of the tests on ubuntu, to make sure the ubuntu-specific bits of bootstrapping the test env. et al. work.

For future reference, ideally these should have been two commits, one fixing the K8s repo on Ubuntu, and another switching unconditionally to installing Helm from release tarballs and pointing this out in the commit message for hasty reviewers like myself.

Done.

@askervin: PTAL

Do you think we can merge the PR?

$ distro=generic/ubuntu2204 ./run_tests.sh policies.test-suite/balloons/n4c16/test01-basic-placement/
...
TASK [Download public signing key for Kubernetes] ******************************
[WARNING]: Module remote_tmp /root/.ansible/tmp did not exist and was created
with a mode of 0700, this may cause issues when running as another user. To
avoid this, create the remote_tmp dir with the correct permissions manually
fatal: [n4c16-generic-ubuntu2204-containerd]: FAILED! => {"changed": false, "msg": "Failed to download key at
https://pkgs.k8s.io/core:/stable:/v1.30/deb/Release.key: Request failed: <urlopen error [Errno -3]
Temporary failure in name resolution>"]

Does the "Download public signing key for Kubernetes" work for you?  And is it still needed now that this patch adds steps "Download Kubernetes APT key" and "Convert the Kubernetes APT key to keyring format"?

Does the "Download public signing key for Kubernetes" work for you? And is it still needed now that this patch adds steps "Download Kubernetes APT key" and "Convert the Kubernetes APT key to keyring format"?

@askervin Can you try once more please with the latest state?

Found the root cause for the error. It was in my environment. I didn't provide DNS server address, but that seems to be required when using Ubuntu in a network environment where access to global DNS servers is restricted.
dns_nameserver=DNS-IP-ADDRESS distro=generic/ubuntu2204 ./run_tests.sh policies.test-suite/balloons/n4c16/test01-basic-placement/


Weighting filters should use a different transformation with more accurate frequency response?

Or just oversample the data first

https://github.com/endolith/waveform_analysis/issues/22


./install-relevant-react.sh: line 15: node_modules/.bin/semver: No such file or directory

Hi Glen,
Getting this locally and on our build boxes. For some reason it also seems to install 15.6 even though it's "react": "15.4.0" in the package.json.
This leads to a later
Error: Cannot find module 'react-test-renderer/shallow'

When running tests.
Any thoughts?
Cheers.

sounds like you're trying to run tests on skin-deep itself without having all the packages installed.
./install-relevant-react.sh should only ever be run as part of this module's test suite.
What is it you're trying to do?

The output is when I do an npm i :/ Node 6.11.1 and npm 3.10.11 I think

And then running npm run test causes:
Error: Cannot find module 'react-test-renderer/shallow'
at Function.Module._resolveFilename (module.js:469:15)
at Function.Module._load (module.js:417:25)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/Users/pso06/skybet/casino-webapp/node_modules/skin-deep/lib/react-compat.js:11:27)

Aha, you're right - this is because I added install as a postinstall script, which doesn't work right for consumers.
I'll get this fixed.

Excellent, cheers Glen 👍

Released as v1.1.1

👍


Opening a Window on Windows with LTO panics

Trying to open a window like this:
let display = WindowBuilder::new()
    .with_dimensions(1280, 720)
    .with_title("Window Title")
    .with_visibility(true)
    .build_glium()
    .unwrap();

with LTO enabled panics like this:
thread '<main>' panicked at 'called `Result::unwrap()` on an `Err` value: BackendCreationError(OsError("CreateWindowEx function failed: Unzul\u{e4}ssiger Zugriff auf einen Speicherbereich. (os error 998)"))', ../src/libcore\result.rs:746
note: Run with `RUST_BACKTRACE=1` for a backtrace.

which roughly translates to Invalid access to a memory region.
Disabling LTO fixes the problem, but increases binary size.

Could you try running one of glutin's examples to see if it works? It's probably a glutin issue and not glium issue.

Yup, running the window example causes the same problem. This is indeed something with glutin. I will repost the issue there.


dict property

Another possibly useful builtin property? I've used Instance properties with instance_class=dict quite a bit. Could be interesting to provide some validation on keys/values?

Yeah, I've wondered about that. I guess I've tended to build up sub-HasProperties classes and then tied instances to those, but there are going to be times when it is useful to have the flexibility of a dict. I confess, it never really occurred to me to use dict with Instance, and I wonder if that is sufficient. What sort of validation are you thinking of?

In cases where a dictionary has known set of keys, it makes sense to use another HasProperties instance, but for arbitrary keys, this doesn't work. It is certainly possible to just use an Instance property with instance_class=dict.
One advantage of a new property type is added validation on keys and values. An example that comes to mind - thinking along the lines of schema generation, you could do something like:
class MySchema(properties.HasProperties):

    paths = properties.Dict(
        doc='Available paths and their methods',
        key_prop=properties.String('', regex='^/'),
        value_prop=properties.Instance('', SwaggerPathSpec),
    )

A few other advantages

slightly less verbose do use Dict rather than Instance of dict (not that big of a deal)
allows us to observe_mutations (ie my_dict.update(...) fires change notifications) (also probably not that big of a deal...)

Initial implementation is here https://github.com/3ptscience/properties/blob/feat/dict_prop/properties/base/containers.py - still needs some work and further discussion.

A quick note to say +1 on a Dictionary (or Dict) property!

@lheagy
Yeah, we need to get this in: https://github.com/aranzgeo/properties/pull/188

#188 merged!


NULL and number literals converted to ?

This does not seem correct to me.
Neither 50 or NULL are placeholder values.
CREATE TABLE puf2017 (
  Id int IDENTITY,
  DrugTier nvarchar(max) NOT NULL,
  PriorAuthorization bit NOT NULL,
  QuantityLimit bit NOT NULL,
  StepTherapy bit NOT NULL,
  StandardComponentId nvarchar(50) NOT NULL,
  BusinessYear int NOT NULL
)
GO

CREATE TABLE puf2017 (
  Id int IDENTITY,
  DrugTier nvarchar(max) NOT ?,
  PriorAuthorization bit NOT ?,
  QuantityLimit bit NOT ?,
  StepTherapy bit NOT ?,
  StandardComponentId nvarchar(?) NOT ?,
  BusinessYear int NOT ?
)
GO

These tests pass so the sanitation is definitely to blame, not lexing:
    #[test]
    fn test_not_null() {
        let sql = "NOT NULL";
        let written = helpers::lex_and_write(sql.to_string());

        assert_eq!(written, sql);
    }

    #[test]
    fn test_std_cmpnt_id() {
        let sql = "StandardComponentId nvarchar(50) NOT NULL";
        let written = helpers::lex_and_write(sql.to_string());

        assert_eq!(written, sql);
    }

The issue is that schema was, apparently, never considered for sanitation input so there are no checks for table creation.
This means that NOT is always seen as a comparison and not a condition.

Upon further investigation this is more clear as Create and Table are not keywords.
I may give this a shot in my freetime over at my fork, if anyone is interested.

Indeed. We didn't need schemas for our use case of sanitizing queries that commonly occur in ActiveRecord. I would like to support this though. I'll add support for this, but have to see when I can find some time :-).
If you'd like to take a stab, that'd be great too.


adding CSS to set black background on body:fullscreen

(re-trying the pull request, something went wrong with the earlier one, apologies about that one).
I quite liked the idea of @bkeepers in https://github.com/gnab/remark/pull/255 to have a black background on body and also liked @gnab's suggestion to only have that on a body:fullscreen - so went ahead to do that as it was easy to do.

Great, thanks!

:metal:


Blockproduction on devnet halted due to bad runtime upgrade

We are going to to try restore state from block 199500

Runtime was recovered by first upgrading it to the latest version to include the session and validator set modules, with a custom validator set (changed locally to have the existing validator keys hardcoded) and then forcing a double session rotation. This is an approach similar to one taken yesterday after a bad addition of validators. This technique is usable to the extent of removing all but one functioning validator if need be.


[FIX] add SoftwareFilters to EEG json example

Related to issue #321. Closes #321
Just a small addition to make eeg json side car example a valid one according to the specs.

The link for circle CI artifacts is missing. Did you branch from latest master @Remi-Gau ?

This was branched from the latest at the time so 4c39c10fb a week ago or so.
Want me to rebase from the latest things?

@Remi-Gau this is the PR that Mainak is referring to https://github.com/bids-standard/bids-specification/pull/315 ... everything after this PR will have an additional "check" which is a bogus check that acts as a convenient link directly from GitHub to the built html pages on CircleCi.
So yes, a rebase would be nice!

hum... If there is a way to do a rebase and not force-push to the remote branch, I did not find it.

If it were up to me, BIDS for raw electrophysiology should not have software filters at all, since the data would be stored/archived/shared in the original form. But in practice, shared data might be minimally processed, so I see the justification for this. I don't care too much about the format, but supporting both a single string and a list sound nice to the researchers. Both soft and hardware filters should be kept consistent between the different electrophys modalities.


MediaStream Recording demo no longer works in Chrome or Firefox

As Chrome and Firefox both no longer support createObjectURL() the page fails to connect the camera stream.

maybe we can convince @orphis to fix this :-)

Oh dear... Thanks for the heads-up — I better sort this out...


What is the minimum android sdk version supported ?

I'm receiving an error on API 17: JS: Error on push notification registration: Error while retrieving a token: PHONE_REGISTRATION_ERROR
But I do not know if related to the version ?, because it works fine on newer versions.

@ignaciolarranaga
I think it is safe to use with API level 17 (since this is also supported in {N}). However, this only applies to being able to compile and manage copatibilities at runtime.
In the present case I think there is a problem with the Google Play services app on the device and it may be good to update it, refresh it, clear all data, or somehow sanitize the Google Play on the app since it should be responsible for managing the device registration.
Can you please try this on your side?


Guild Role struct

Guild Role struct doesn't implement 3 discord API fields: icon, unicode_emoji, tags
https://discord.com/developers/docs/topics/permissions#role-object

thx


[milestone] Convert the library to typescript

Contribution is welcome
Why?
Everything started here from the willing to merge a separate package with the corresponding type definitions into this repo.
The idea is great, but we, as a community, can go further and migrate the entire code base into typescript.
Any benefits?

only one package to install for end users
easier to maintain
your help to the lovely library
???
profit

Our steps to achieve the goal

[x] extend rollup and rest tools to support .ts and .tsx modules https://github.com/Hacker0x01/react-datepicker/pull/4704
[x] migrate datepicker modules first
[x] add a GitHub action to check types @mirus-ua https://github.com/Hacker0x01/react-datepicker/pull/4749
[ ] update docs
[ ] migrate test cases (after all, use ts-jest)
[ ] migrate docs (optional)

Modules to migrate and how to contribute
Basic rules and advice:

start with the smallest isolated modules, like calendar_container.jsx
prevent the use any or other loose types if you don't have strong arguments
prevent using "slow types", give this option to the end users
use interfaces instead of types because the large amount of the code base is class-based
add JSDocs where you can to improve overall DX. Everyone likes a good DX

Steps to contribute

Write in the comments the name of the module that you want help to migrate
Me or a maintainer will assign your name to the list to prevent doing the same job twice
Migrate according to the list of advice above
Highly recommended to migrate test cases together with the module
Create a PR with a name format [typescript-migration] calendar.jsx etc.

[x] calendar_icon.jsx https://github.com/Hacker0x01/react-datepicker/pull/4706
[x] calendar_container.tsx https://github.com/Hacker0x01/react-datepicker/pull/4704
[x] date_utils.js https://github.com/Hacker0x01/react-datepicker/pull/4707
[x] inputTime.jsx https://github.com/Hacker0x01/react-datepicker/pull/4708
[x] calendar.jsx @yuki0410-dev https://github.com/Hacker0x01/react-datepicker/pull/4766
[x] popper_component.jsx @yuki0410-dev https://github.com/Hacker0x01/react-datepicker/pull/4740
[x] week_number.jsx @yuki0410-dev https://github.com/Hacker0x01/react-datepicker/pull/4714
[x] month.jsx @mirus-ua https://github.com/Hacker0x01/react-datepicker/pull/4743
[x] portal.jsx @Olenka-Yurchuk https://github.com/Hacker0x01/react-datepicker/pull/4722
[x] with_floating.jsx https://github.com/Hacker0x01/react-datepicker/pull/4709
[x] month_dropdown.jsx @yuki0410-dev https://github.com/Hacker0x01/react-datepicker/pull/4723
[x] year.jsx @Olenka-Yurchuk https://github.com/Hacker0x01/react-datepicker/pull/4756
[x] month_dropdown_options.jsx @yuki0410-dev https://github.com/Hacker0x01/react-datepicker/pull/4723
[x] tab_loop.jsx https://github.com/Hacker0x01/react-datepicker/pull/4710
[x] year_dropdown.jsx @yuki0410-dev https://github.com/Hacker0x01/react-datepicker/pull/4715
[x] day.jsx @mirus-ua https://github.com/Hacker0x01/react-datepicker/pull/4713
[x] month_year_dropdown.jsx https://github.com/Hacker0x01/react-datepicker/pull/4724
[x] time.jsx @yuki0410-dev https://github.com/Hacker0x01/react-datepicker/pull/4726
[x] year_dropdown_options.jsx @yuki0410-dev https://github.com/Hacker0x01/react-datepicker/pull/4715
[x] index.jsx @yuki0410-dev https://github.com/Hacker0x01/react-datepicker/pull/4766
[x] month_year_dropdown_options.jsx https://github.com/Hacker0x01/react-datepicker/pull/4724
[x] week.jsx https://github.com/Hacker0x01/react-datepicker/pull/4725 https://github.com/Hacker0x01/react-datepicker/pull/4725

Breaking changes so far:

newDate helper always returns a date, if the initial params were invalid, then it returns just a new Date Line;  PR
calendarIconClassname => calendarIconClassName PR

@martijnrusschen can you pin the issue?

Nice progress so far!

@mirus-ua ( @martijnrusschen  )
Thank you for the great attempt.
I would like to help as well, but should I declare the file to be started on this issue?

@yuki0410-dev Feel free to claim a file and let it know here so there's no duplicate work done.

I would like to take the portal jsx file in work

I would like to take the week_number.jsx file in work

Added your name to week_number.jsx

6.6% 💪

@mirus-ua
I would like to take the month_dropdown.jsx and month_dropdown_options.jsx file in work.

@martijnrusschen hello. Please ask reviews to look at four opened PRs if you can. It'll be much faster if we merge them because the rest of the modules depend on them

Yep, I'll expect them to complete this soon. Normal turnaround time is ~2-4 hours.

@mirus-ua
I would like to take the month_year_dropdown.jsxand month_year_dropdown_options.jsx file in work.

@mirus-ua
I would like to take the week.jsx file in work.

@mirus-ua I would like to take the year.jsx file in work.

@mirus-ua
I would like to take the popper_component.jsx file in work.

@mirus-ua
I would like to take the calendar.jsx file in work.

@mirus-ua
Modifications to index.jsx may need to be synchronized with modifications to calendar.jsx. (especially the props type)
Would it be ok if I work on it all together?

@mirus-ua Modifications to index.jsx may need to be synchronized with modifications to calendar.jsx. (especially the props type) Would it be ok if I work on it all together?

sure, I think you should wait a bit for the month and the year modules

@mirus-ua

sure, I think you should wait a bit for the month and the year modules

I agree with waiting too.
We are moving forward on the calendar.jsx work, but there are some aspects that we cannot move forward on until those are completed.

apologies, I'm on vacation this week and don't have the ability to review
or merge anything until Friday.
On Mon, Apr 29, 2024 at 3:15 PM yuki0410 @.***> wrote:

@mirus-ua https://github.com/mirus-ua
sure, I think you should wait a bit for the month and the year modules
I agree with waiting too.
We are moving forward on the calendar.jsx work, but there are some aspects
that we cannot move forward on until those are completed.
—
Reply to this email directly, view it on GitHub
https://github.com/Hacker0x01/react-datepicker/issues/4700#issuecomment-2082714610,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKY2KE7BIGCPH4IGIOERD3Y7ZBWDAVCNFSM6AAAAABGK5XR6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBSG4YTINRRGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@yuki0410-dev almost everything was merged, so you can start with the final part

what's left?

@mirus-ua (@martijnrusschen )
I would like to take the week_picker_test.test.js file in work. and fix some types.

what's left?

The core features are done.But I think better to migrate some amount of tests just to double check that all types are OK

@mirus-ua
I noticed during my experimentation with #4810 that type errors in the test file do not seem to result in errors in CI.
Is it my understanding that this is due to the fact that I am now using babel-jest to run tests with type information ignored, and that when I change to ts-jest later on, I will get CI errors?

@mirus-ua I noticed during my experimentation with #4810 that type errors in the test file do not seem to result in errors in CI. Is it my understanding that this is due to the fact that I am now using babel-jest to run tests with type information ignored, and that when I change to ts-jest later on, I will get CI errors?

everything is correct

hi, is anyone working on these files? i would like to take 1 of them:
calendar_test.test.js
datepicker_test.test.js
month_test.test.js

@ryantanrk go for it :)

i'll take month_test.test.js then if no one is working on it 👍

@martijnrusschen @ryantanrk @mirus-ua
Converted calendar_test.test.js and datepicker_test.test.js.

What test modules are left?

What test modules are left?

@mirus-ua
#4855 is merged, TS conversion of all test modules is complete.

Once #4855 is merged, I'll issue a major version upgrade. Are there any type/test changes we want to make once we're fully migrated to TS?

The issue description says: after all, use ts-jest

Migrate to ts-jest and then go with the major

Sounds good. On 2 Jun 2024, at 16:36, Mirus @.***> wrote:
Migrate to ts-jest and then go with the major
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

OK. I'll do then

It looks like I underestimated the amount of work. Almost all tests have types issues, so I don't think that I can do it relatively fast alone

Is this something that could be split up somehow?

Is this something that could be split up somehow?

I can push the branch with basic config, and then we can use it as PRs target for this activity

sounds good

I'll do today

@mirus-ua
Just a thought, but if we assign the tests to ts-jest one file at a time in the transform section, can we migrate from babel-jest to ts-jest without causing errors in CI?

@mirus-ua Just a thought, but if we assign the tests to ts-jest one file at a time in the transform section, can we migrate from babel-jest to ts-jest without causing errors in CI?

Do you mean here explicitly add files that we migrated? It could work even if it'll be tedious

@mirus-ua
That's right.
I am trying it now in #4871 and have a base. I am going to try one file now.

@mirus-ua @martijnrusschen
It looked like we could migrate babl-jest to ts-jest while merging into the Master branch, so we created PR #4871.

Are we safe to cut a major release now we've converted the last tests?

Are we safe to cut a major release now we've converted the last tests?

I think so. We've done everything possible to integrate types safely into the existing code base and save the future feature development from type degradation

Cool, for the release notes, we should capture the breaking changes.

Here's the changelog: https://github.com/Hacker0x01/react-datepicker/releases/tag/v7.0.0

That was a good teamwork. Thanks all.
I think the issue should be open because we have some work with docs to do

I only had one custom component interfacing with ReactDatePicker but I encountered a lot of breaking changes that weren't listed in the release.

ReactDatePicker name changed to DatePicker.
ReactDatePickerProps name changed to DatePickerProps.

I had an interface as below:
interface DateInputProps extends Partial<ReactDatePickerProps> {
  style?: React.CSSProperties;
}

With an onChange with below def:
function handleChange(date: Date | null)

I had to change the interface to a custom type.
I had to make onChange and onSelect optional in the custom type.
I had to explicitly set selectsRange and selectsMultiple to undefined in order to get default behaviour.
I had to pass the excludesScrollbar prop in all the usages of my custom component.
I assume that @types/react-datepicker needs to be uninstalled but not sure about this.

In the end my interface looks like this:
type CustomDatePickerProps = Omit<
  DatePickerProps,
  'onChange' | 'onSelect' | 'selectsRange' | 'selectsMultiple'
> & {
  onChange?: (
    date: Date | null,
    event?: React.MouseEvent<HTMLElement> | React.KeyboardEvent<HTMLElement>,
  ) => void;
  onSelect?: (
    date: Date | null,
    event?: React.MouseEvent<HTMLElement> | React.KeyboardEvent<HTMLElement>,
  ) => void;
  excludeScrollbar?: boolean;
  selectsRange?: boolean;
  selectsMultiple?: boolean;
};

type DateInputProps = CustomDatePickerProps & {
  style?: React.CSSProperties;
};

And my handleChange like this:
function handleChange(
    date: Date | null,
    event?:
      | React.MouseEvent<HTMLElement>
      | React.KeyboardEvent<HTMLElement>
      | undefined,
  )

@bsal649 ( @mirus-ua @martijnrusschen )
Thank you for the quick introduction.
I will answer as far as I am aware.

ReactDatePicker name changed to DatePicker.

I think you forgot to put this in the "breaking changes" section.

ReactDatePickerProps name changed to DatePickerProps.

The interface needed to be changed to a custom type.

I think this is "breaking changes" because this is a change from @types/react-datepicker, which is originally managed by someone outside this Repository member.

I had to pass the excludesScrollbar prop in all the usages of my custom component.

I could not find that Props in this current Repository.

I assume that @types/react-datepicker needs to be uninstalled but not sure about this.

It must be uninstalled.

@yuki0410-dev Thanks for the explanation, and fair enough.
Maybe it'd be worth putting a note in the release that people who were using @types/react-datepicker might have a lot more breaking changes.

@yuki0410-dev For "ReactDatePicker name changed to DatePicker.", that doesn't seem like an intended change and will be causing everyone to make changes. Can we reduce the breaking changes by reverting to the original name? I don't see a need to force change that here. Thoughts? Same counts for "ReactDatePickerProps name changed to DatePickerProps."

@yuki0410-dev For "ReactDatePicker name changed to DatePicker.", that doesn't seem like an intended change and will be causing everyone to make changes. Can we reduce the breaking changes by reverting to the original name? I don't see a need to force change that here. Thoughts? Same counts for "ReactDatePickerProps name changed to DatePickerProps."

The right way, IMO, is to add these in the list of the breaking changes rather then violating semver rules

That would be the alternative, but it forces everyone to update their code for just a name change.

@martijnrusschen @mirus-ua
My question is, since DatePicker is export defaulted, can't we name it freely on the import side?
I don't think any modifications have occurred due to the renaming of the DatePicker class, at least in doc-site.


Update zio, zio-test, zio-test-magnolia, ... to 1.0.12

Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-magnolia
dev.zio:zio-test-sbt

from 1.0.11 to 1.0.12.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

labels: library-update, semver-patch

Codecov Report

Merging #185 (79a0c3e) into master (551cf84) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #185   +/-   ##
=======================================
  Coverage   87.30%   87.30%
=======================================
  Files          28       28
  Lines         843      843
  Branches       39       39
=======================================
  Hits          736      736
  Misses        107      107

Continue to review full report at Codecov.

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


Could not login on the server

Hello i am using the web site version of CVAT and recently it started to throw the error while i and my colleagues trying to login "Could not login on the server"
Some days there is no error at all and it works fine, but only some days...
I have googled it, but all the answers are about local github version of CVAT, what about online web version?
thank you

@Smogulove , Please send your questions and complains to https://community.intel.com/t5/Intel-Distribution-of-OpenVINO/bd-p/distribution-openvino-toolkit forum. I hope that they will help.

I have the same issue(CVAT the online version) and sometimes when i delete cookies the server let me in but at the moment i can't access it at all.
Can anyone help ?

Try to login on private mode of your browser (this mode is available for chrome, firefox and other browsers). I noticed on other sites (not sure for CVAT) than browser cache sometime is not updating even that updated with ctrl+f5. So I allways recommending first to use private mode of the browser.

I have the same issue(CVAT the online version) and sometimes when i delete cookies the server let me in but at the moment i can't access it at all. Can anyone help ?

Try to login on private mode of your browser (this mode is available for chrome, firefox and other browsers). I noticed on other sites (not sure for CVAT) than browser cache sometime is not updating even then updated with ctrl+f5. So I allways recommending first to use private mode of the browser.

I tried but unfortunately the problem remains.

Hello,
Might be a silly question but if the situation persists for a really long time as nmanovic hinted above, is it sensible to download and install CVAT offline for Windows or is it also going to be problematic.
Thanks,
Prassi

Try to change password. And also create new account. May be this is someone administrative error with your account personally? PS. I've never used online cvat - we are using locally hosted version of cvat in our case

I tried but unfortunately the problem remains.

Try to change password. And also create new account. May be this is someone administrative error with your account personally? PS. I've never used online cvat - we are using locally hosted version of cvat in our case

I tried but unfortunately the problem remains.

I tried changing my password too and it does not work, also I guess opening a new account makes it impossible to transfer existing open tasks from the old acc to the new one.

Here #4715 I see there was errors with access. This can be reason you cant login today. Creating new account only checks that system is working (in service). I recommending creating your own hosted version of cvat for your company (we did so).

May be I'm not right, and developers of CVAT will beat me today ))) But i dont belive that online verson of cvat is productive-oriented. So, we have our own locally hosted server in our company (also you can do it on any other docker-hosting providers or vps hosting).
This is Linux machine with enough disk space and memory, Also we have installed: docker + docker-compose. And after that we can start docker-compose.yaml that is part of source code CVAT. (Also in new version (>2) we have rules https://github.com/openvinotoolkit/cvat/tree/develop/cvat/apps/iam/rules copy on server). Thats all. Also we connected directory with images to cvat-docker-container:

/mnt/cvat:/home/django/share:ro
And can access locally mounted dirs with images in CVAT.
And admin panel also accessible with controlling local users.

I am sorry but could you elaborate ? Do you mean I should download and install CVAT offline ? Did that work for you ?

@strcom Can you help to install the cvat tool on my windows computer. From start to end. I am facing the problem from two days.

There is your plan (I can't write detail plan in comment):

Install docker for windows (https://runnable.com/docker/install-docker-on-windows-10)
Install docker-compose (https://docs.docker.com/compose/install/compose-desktop/)
You need learn docker/docker-compose in base (how to run/stop containers on docker-compose, what is containers and images in docker)
Download file https://github.com/openvinotoolkit/cvat/blob/develop/docker-compose.yml and save on folder (exp. "d:/CVAT")
Download and copy files https://github.com/openvinotoolkit/cvat/tree/develop/cvat/apps/iam/rules (better way to get from zip of source code cvar https://github.com/openvinotoolkit/cvat/archive/refs/heads/develop.zip, also you can get file docker-compose.yml for step 4 from this zip-file)
Get this folder from zip /cvat-develop/cvat/apps/iam/rules and place in folder d:/CVAT/cvat/apps/iam/rules
In original docker-compose.yml you can see such an option to send rules to contaner OPA:
./cvat/apps/iam/rules:/rules
Here i don't know how exactly start docker-compose.yml from Windows, but can suppose that this is like in linux. From local folder local folder "d:/CVAT" from power-shell console or command line (cmd.exe) start command:
docker-compose up -d
To stop containers from folder "d:/CVAT":
docker-compose down
After that you can access local version cvat:
http://localhost:8080
You need to create super user for access admin console of CVAT
Command from powershell or command line:
docker exec -it cvat bash -ic 'python3 ~/manage.py createsuperuser'
There will be some questions about admin login and password.
After that you can access admin console of CVAT:
http://localhost:8080/admin
And create users and give them roles
After that you can access CVAT
http://localhost:8080
For share CVAT for corporate work - you can configure your Windows Defender so, anybody on local network can access your local hosted CVAT
IF you need change password of super user run command from powershell or console:
docker exec -it cvat bash -ic 'python3 ~/manage.py changepassword ADMIN'
where ADMIN - is a name of your superuser
For access local images on you PC you must share folder with images for locally hosted CVAT service.
For that you must modify docker-compose.yml:
...
cvat:
container_name: cvat
...
volumes:

PATHTOLOCALFOLDER:/home/django/share:ro
where PATHTOLOCALFOLDER is your local folder with images
After that you can access local images without need to copy them to cvat. This is optional, but your local copy of CVAT can be faster.

@strcom Can you help to install the cvat tool on my windows computer. From start to end. I am facing the problem from two days.

There is your plan (I can't write detail plan in comment):
1. Install docker for windows (https://runnable.com/docker/install-docker-on-windows-10)

2. Install docker-compose (https://docs.docker.com/compose/install/compose-desktop/)

3. You need learn docker/docker-compose in base (how to run/stop containers on docker-compose, what is containers and images in docker)

4. Download file https://github.com/openvinotoolkit/cvat/blob/develop/docker-compose.yml and save on folder (exp. "d:/CVAT")

5. Download and copy files https://github.com/openvinotoolkit/cvat/tree/develop/cvat/apps/iam/rules (better way to get from zip of source code cvar https://github.com/openvinotoolkit/cvat/archive/refs/heads/develop.zip, also you can get file docker-compose.yml for step 4 from this zip-file)
   Get this folder from zip /cvat-develop/cvat/apps/iam/rules and place in folder d:/CVAT/cvat/apps/iam/rules
   In original docker-compose.yml you can see such an option to send rules to contaner OPA:
   ./cvat/apps/iam/rules:/rules

6. Here i don't know how exactly start docker-compose.yml on Windows, but can suppose, that this is like in Linux. From local folder "d:/CVAT"  in power-shell console or command line (cmd.exe) start command:
   docker-compose up -d
   Later (after work) you can stop containers from folder "d:/CVAT":
   docker-compose down

7. After that you can access local version cvat:
   http://localhost:8080

8. You need to create Super User for access CVAT admin console
   Command from powershell or command line:
   docker exec -it cvat bash -ic 'python3 ~/manage.py createsuperuser'
   There will be some questions about admin login and password.

9. After that you can access admin console of CVAT:
   http://localhost:8080/admin
   And create users and give them roles

10. After that you can access CVAT
    http://localhost:8080

11. For share CVAT for corporate work - you can configure your Windows Defender so, anybody on local network can access your local hosted CVAT

12. IF you need change password of super user run command from powershell or console:
    docker exec -it cvat bash -ic 'python3 ~/manage.py changepassword ADMIN'
    where ADMIN - is a name of your superuser

13. For access local images on you PC you must share folder with images for locally hosted CVAT service.
    For that you must modify docker-compose.yml:
    ...
    cvat:
    container_name: cvat
    ...
    volumes:
    - PATHTOLOCALFOLDER:/home/django/share:ro
    where PATHTOLOCALFOLDER is your local folder with images. And restart docker services of CVAT.
    After that you can access local images without need to copy them to cvat. This is optional, but your local copy of CVAT can be faster.

@strcom Can you help to install the cvat tool on my windows computer. From start to end. I am facing the problem from two days.

There is one problem on Windows and WSL which makes it next to impossible to work with CVAT. I think everyone would face this. The Docker Desktop takes almost 95 Percent of the memory because of the vmmem exe file and the solution to this has been a hit and miss for me. So, even though I could install it, I basically cant annotate a single frame because it just hangs forever. Anyone who knows a more reliable method than the .wslconfig method, please let me know. Thanks...

@Smogulove , @strcom , @Bouye , @prassivenky , CVAT.org should be online in the nearest future. But you can try this one: https://app.cvat.ai/. It is supported by CVAT core team.

Please use our new online seriver CVAT.ai: https://cvat-ai.github.io/cvat/docs/faq/#how-to-migrate-data-from-cvatorg-to-cvatai

I removed browser cache and it worked.


[Release 1.24] Update Flannel to v0.20.2

Backport for #6585

Validated on master (v1.24) branch with commit dabd995365fc339647485c3a0bee0ab59d88fa7e


gclang does not recognize clang flags

Hi,
As seen in the screenshot below, when gclang is compiling the c files, it seems that it doesn't recognize a lot of the flags that clang understands. The flags -target, -mcpu, etc are valid as per the clang docs. I'm wondering why this is the case.

Thanks!

got it, thanks! so should I be modifying this function?

Exactly.  It's a flaw that wllvm and gclang share, but I do not see any feasible way around it.
I once looked at the clang code for doing this, but gave up the idea of using that.

If I download the code and make some local changes, how would I build the code to get the binaries: gclang etc? Thanks!

So you should already have a copy of the repo beneath your GOPATH.
I can never remember the rest, which is why you will see this file, and the incantation:
go install github.com/SRI-CSL/gllvm/cmd/...

I think you do it in the local copy of the repo.

Oh I forgot, there is also a makefile to remind me why I entered the room.
make

in the repo will list the targets. Happy hacking.


add colabs cover

Added the CoLabs cover

Mobile view is broken:

@sonylomo which screen size is this? I don't see it on my end.

I've just reloaded it and it's gone...my bad. 😁


Sending a message using the php loop ends up with only 70 to 74 emails. Finally, the error is 500. What happened?

Expected content
Actual content
Link to page:

Hello @burt00,
We're sorry you're having this issue! Since this is not a docs-related issue, please contact our Support team directly and they can help you out. You can access support contact options by logging into https://support.sendgrid.com.
SendGrid Docs Team


Indicator Bug

I collapsed first section then collapsed second, it happened!

so does third!

Hi @Otisle, thanks for reporting this bug! Could you be more specific? Like what should be expected, and what are you seeing now that is not correct.

There is a video!
bug.zip
When you collapsed second section, look at the first cell in second section.(iPad Pro)
There is a indicator above the detail label.
Then collapse the third section, the indicator is above the third cell in second section(iPad mini 4).

Ah I see, so the indicator that you mentioning is the thin line, right? I will take a look and see what caused the issue, thank you so much for your video!


zlib: revision bump

Create bottles for Sierra and El Capitan

Note: likely obsoleted by https://github.com/Homebrew/homebrew-dupes/pull/701

Note: likely obsoleted by https://github.com/Homebrew/homebrew-dupes/pull/701

@BrewTestBot test this please

We can do this through the homebrew testing job

1.2.8 bottles added in https://github.com/Homebrew/homebrew-dupes/commit/ded3d4fd19402a232e9069fce17f24b55f205194.


Document who should use ClusterFuzzLite and compare to other options

This was suggested by  @worksbutnottested

It would be neat to have a table comparing ClusterFuzzLite and ClusterFuzz.
For example, is there any advantage adding ClusterFuzzLite if you are already using OSS-Fuzz?

@jonathanmetzman - We should update the docs on @hickford questions.
@hickford - If you are integrated with OSS-Fuzz and using CIFuzz [https://google.github.io/oss-fuzz/getting-started/continuous-integration/] in your CI/CD, then you don't need ClusterFuzzLite (CIFuzz and ClusterFuzzLite are same things). ClusterFuzzLite is for projects who can't use OSS-Fuzz, or have private repos.

If you are integrated with OSS-Fuzz and using CIFuzz [https://google.github.io/oss-fuzz/getting-started/continuous-integration/] in your CI/CD, then you don't need ClusterFuzzLite

FWIW I'm planning to replace CIFuzz with CFLite: https://github.com/google/oss-fuzz/issues/7195#issuecomment-1033832369 . In my experience CFLite is more stable and predictable.

I think it would be even more predictable if it was possible to pin it properly but considering that I already can pin the docker image used by CFLite to build the fuzzers I already can avoid issues caused by clang rolls.


SFA: Links to webring posts in non-Scrapbook channels are broken

Fixed in https://github.com/hackclub/scrappy/commit/24e4d84556e9e8130db925a0d0da8222af8f251c


Splitting and renaming CPU-related code

This PR proposes further separation of CPU-specific code from the rest of the framework. Specifically:

devicedeps/ directory restructured to put device type first, OS interfaces and other code second
some of the variables are renamed to become more generic or more device-specific, where it makes sense
added build-time flag to indicate selected type of device (currently only CPU)

Why this is arranged as a single commit? Could you split it into multiple (and reviewable) commits?


Added port list functionality and increased range of baud rates available via serial_posix.go

Added GetPortsList method to all implementations, shamelessly lifted
from the gobug.st/serial package and subject to the conditions of its
BSD-like license (which has been reproduced in source as per terms of said
license). Experienced stability issues with gobug.st ages back, which is
why I switched and stole.
Updated serial_posix to support full list of termios baud rates (as per
existing serial_linux implementation).
Moved posixTimeoutValues method from top level serial.go to new
serial_x.go source file, as only required for linux and posix
implementations.
A major disclaimer -- not retested on windows and OSX since some refactoring and merging upstream changes over the last day or two, as moved office and only have Linux machines to hand.

Thanks for the explanation.
From your description, I think it could be OK to include that functionality.
Just as OpenPort() has single common implementation with a godoc comment with platform specific openPort()'s, I think it makes sense to have that pattern for ListPorts().  The comment for ListPorts should be very explicit about it being precise on windows, but only a heuristic convenience function on unix platforms.
Also please split out the baud rate changes into a separate PR.

Think that last commit addresses most of the concerns to date?
Specialised the port name regex in posix_darwin, posix_linux, posix_other:

"_darwin" version matches anything beginning with "cu." or "tty." as per http://pbxbook.com/other/mac-tty.html
"_linux" version matches any group of one or more digits beginning with "ttyS", "ttyUSB", "ttyACM" or "ttyAMA"
"_other" catchall version is overly inclusive -- matches anything beginning with "tty"

Multiple files for a single regex filter seems a bit excessive, but would lend itself to the per-platform definition of a baud-rate map discussed.

Maybe this could be improved a bit - to include VID, PID and PnP description string.  Also, covering darwin, linux and windows.
I know, node-serialport does it quite well.. maybe some inspiration could be found there:
https://github.com/EmergingTechnologyAdvisors/node-serialport/blob/master/src/serialport_unix.cpp
https://github.com/EmergingTechnologyAdvisors/node-serialport/blob/master/src/serialport_win.cpp


ArtifactoryPlublish unable to publish version-catalog correctly

Describe the bug
I'm trying to publish the version-catalog to jfrog.
When I'm publishing mavenLocal i can see the .module file contains versionCatalogElements variant.
but when i do artifactoryPublish which is jfrog publish i dont see versionCatalogElements variant instead i see apiElements and runtimeElements
opened gradle issue initially https://github.com/gradle/gradle/issues/29369
but it is not gradle issue it is Jfrog publish issue which unable to publish the version-catalog publications. Always picks pluginMaven publications.
Current behavior
in module file
"variants": [ { "name": "apiElements", "attributes": { "org.gradle.category": "library", "org.gradle.dependency.bundling": "external", "org.gradle.jvm.environment": "standard-jvm", "org.gradle.jvm.version": 17, "org.gradle.libraryelements": "jar", "org.gradle.usage": "java-api", "org.jetbrains.kotlin.platform.type": "jvm" }, "files": [ { "name": "gradle-version-catalog-0.2.0.jar", "url": "gradle-version-catalog-0.2.0.jar", "size": 261, "sha512": "xxxxxxxxxxxxxxx", "sha256": "xxxxxxxxxxxxx", "sha1": "xxxxxxxxxxxxxxxxxxxxxx", "md5": "xxxxxxxxxxxxxxxxxxxxxxxxx" } ] }, { "name": "runtimeElements", "attributes": { "org.gradle.category": "library", "org.gradle.dependency.bundling": "external", "org.gradle.jvm.environment": "standard-jvm", "org.gradle.jvm.version": 17, "org.gradle.libraryelements": "jar", "org.gradle.usage": "java-runtime", "org.jetbrains.kotlin.platform.type": "jvm" }, "files": [ { "name": "gradle-version-catalog-0.2.0.jar", "url": "gradle-version-catalog-0.2.0.jar", "size": 261, "sha512": "xxxxxxxxxxxxxxxxxxxxxxxx", "sha256": "xxxxxxxxxxxxxxxxxxxxxxxxxxxx", "sha1": "xxxxxxxxxxxxxxxxxxxxxxxxxxxxx", "md5": "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx" } ] } ] }
Reproduction steps
gradle version 8.5
jfrog artifactory buildinfo version - 5.2.0
create a version-catalog publish project
Try to publish the versionCatalog then check module file should have versionCatalogElements but not found.
Expected behavior
mavenLocal publish expected in module file
"variants": [     {       "name": "versionCatalogElements",       "attributes": {         "org.gradle.category": "platform",         "org.gradle.usage": "version-catalog"       },       "files": [         {           "name": "gradle-version-catalog-0.2.1-SNAPSHOT.toml",           "url": "gradle-version-catalog-0.2.1-SNAPSHOT.toml",           "size": 6029         }       ]     }   ]
Artifactory Gradle plugin version
5.2.0
Operating system type and version
Windows,Android
JFrog Artifactory version
5.2.0
Gradle version
8.5

@yuvaraj119
Thank you for using the Gradle Artifactory plugin.
To help us resolve this issue swiftly, could you please provide a minimal reproducer?
The reproducer should demonstrate that the artifactoryPublish task publishes fewer artifacts compared to the standard publish or publishToMavenLocal tasks.


cmake: renames linker scripts and clarifies cmake variables related to gperf

This renames the linker scripts used to build zephyr_prebuilt.elf and the final zephyr.elf so they should be more clear on where these scripts apply. The old linker.cmd sounds like it's for the final binary but it wasn't, but the new one now is.
Also changes the gperf related cmake variables so it is more clear what they are for.

verified no impact for code coverage report.


AttributeError: 'TensorNameMap' object has no attribute 'get'

Expected Behavior
convert-llama-ggmlv3-to-gguf.py --input wizardlm-llama2-13b.ggmlv3.q4_K_S.bin --output wizard.gguf
should result in a gguf file...
Current Behavior
* Preparing to save GGUF file
* Adding model parameters and KV items
* Adding 32000 vocab item(s)
* Adding 146 tensor(s)
Traceback (most recent call last):
  File "C:\Users\erik\llama.cpp\convert-llama-ggmlv3-to-gguf.py", line 345, in <module>
    main()
  File "C:\Users\erik\llama.cpp\convert-llama-ggmlv3-to-gguf.py", line 341, in main
    converter.save()
  File "C:\Users\erik\llama.cpp\convert-llama-ggmlv3-to-gguf.py", line 165, in save
    self.add_tensors(gguf_writer)
  File "C:\Users\erik\llama.cpp\convert-llama-ggmlv3-to-gguf.py", line 273, in add_tensors
    mapped_name = nm.get(name)
                  ^^^^^^
AttributeError: 'TensorNameMap' object has no attribute 'get'

Some context, the tensor name is "tok_embeddings"
Environment and Context
MINGW64_NT-10.0-19045 LAPTOP-1K8B9NQ8 3.3.6-341.x86_64 2022-09-05 20:28 UTC x86_64 Msys
Steps to Reproduce
Please provide detailed steps for reproducing the issue. We are not sitting in front of your screen, so the more detail the better.

download a ggml model file
pip install gguf numpy
convert-llama-ggmlv3-to-gguf.py --input wizardlm-llama2-13b.ggmlv3.q4_K_S.bin --output wizard.gguf

Problem solved.   "gguf" at pypi is not compatible.
Using pip install gguf-py/ works to fix.
Probably the convert-llama-ggmlv3-to-gguf.py should know about that (if import fails, or if version mismatches, spit out an error)

ok, after getting that to work there is a new error:
error loading model: create_tensor: tensor 'blk.15.ffn_norm.weight' not found

ok, fixed that too, i had downloaded a partial file, and for whatever reason, there were zero errors.


sbt package doesn't compatible with apple M1

In the spark client - sbt package doesn't compatible with apple M1.
We should add an option to download the package locally and use it in case of running the client on M1.

@lynnro314 is this still an issue?

Not relevant anymore


Create NuGet packages for the AppDesigner and Editors binaries

Microsoft.VisualStudio.AppDesigner.dll and Microsoft.VisualStudio.Editors.dll used to be in the GAC and are not in VS "15". There are VS extensions that reference this binary from the GAC in 2015 that will be broken in VS "15". We should create nuget packages for these binaries so that they can be referenced.

Hey Sri - is this still being targeted for RC? We'd really like to be able to tell the teams that depend on these DLLs that there is a reference story for them.


Hide sway/window when there is no window in focus

Here is my bar when a window is in focus:

And here it is with nothing in focus.

It would be nice to be able to remove the artefact in the second image where the sway/window modules remains but is empty.

Did you have a look at #1448 or #1349?
I think you could style
window#waybar.empty #window {   background:none; }
or similar.

Works perfectly, thanks! I was a bit confused because the fact that its done through styling doesn't seem to match the pattern that would be used for other widgets, for example doing this in the config for my network module:
"network": {
    "format-disconnected": ""
}

In order to hide it while its in a certain state.

It is a bit counter-intuitive indeed. I've added a remark regarding CSS hierarchy to the wiki page.


Add wp cli option to check for null dereferences

The wp library is capable of checking for null dereferences. It would be useful to add this as a command line flag option to the wp plugin, such that "--wp-check-null-deref" reports SAT and provides inputs that force the provided binary to perform the null dereference.  Simple examples include:
int main(int argc,char ** argv) {
        *((int*)NULL);
        return 0;
}

and
int main(int argc,char ** argv) {
        if (argc == 3){
                *((int*)NULL);
        }
        return 0;
}

Similar to the simple-wp example, the expected output model for the latter test case would include RDI=3.

Closed with #131.


Update README to show CLI quick start

Now that we have a new command, we can include a section on getting a new project started quickly with the CLI.

Updated fyord and fyord-cli github and npm readmes.


🚀 Feature: Allow the Traffic from a specific IP or Domain Name

🔖 Feature description
Add an Option to whitelist certain IP addresses or Domain Names for accessing the application
🎤 Pitch
It would be beneficial to implement a feature that allows for whitelisting traffic from specific IP addresses or domain names. This would enable administrators to grant authorized access to a predetermined set of users, thereby mitigating the risk of unauthorized access or downloads. Doing so would protect sensitive files from potential security breaches, and only trusted sources would be permitted to interact with the system. This feature would provide an additional layer of security and control, ensuring the integrity and confidentiality of sensitive data.

For this type of feature, I would recommend you to use a reverse proxy. You can place Pingvin Share behind a reverse proxy and configure the allowed IP addresses in the configuration of the reverse proxy. Here is an example with Nginx.

@stonith404   Thanks a lot boss - currently the Pingvin Share is running on a docker and managed through Portainer (because there are some other Dockers too running on the same Linux server )
Thanks a lot for your valuable recommendations I will try the Nginx reverse proxy. but just think about this feature - maybe on a future release

Abin Abraham Anchery


Scribe receiver does "tryLater" on just about anything

One strange thing about the current scribe implementation is that it sends a TryLater on any non-fatal, non-cancellation exception. That seems to imply a poison message (ex bad encoding) could take down scribe, if the caller indeed tries it again later. You could imagine a loop happening with this policy.
It seems more sensible to only issue tryLater on a whitelist of known transient issues, such as IOExceptions.

ex. we literally have a test to try later on NullPointerException! Would we expect this to not be a NPE later?!
    assert(log.get.trim === "Sending TryLater due to NullPointerException(foo was null)")
``

whitelisting transient could be tricky, as for example NoHostAvailableException isn't an IOException, and it is a cassandra-specific class. One way could be to enforce that AsyncSpanConsumer implementations must route an transient scenario to an IOException. The problem with this approach is that it is a whack-a-mole and very sensitive to library updates.
Another option is to simply not support tryLater.

I don't think we should support tryLater at all. Spans are lossy data.

current code doesn't do this


Question - take it one level up - to the account

Hi Douglas,
By my understanding, the files we see in the parent object is by the related to field in the task object.
Is it possible to take it one level up, so we will be able to see every thing under the account level?
For example:
Task that related to opportunity, will be seen in the related account of the opportunity.
Thanks, Lee.

For tasks/events/activities, I believe this could be simple to add in for records that use the standard WhatId and WhoId and then the Task.AccountId will be populated.
For the fix yesterday in issue #2 for querying on custom lookup fields this will be trickier. Not sure if the AccountId gets populated or not.
I'll take a look, thanks for the suggestion!

Hi Douglas,
Yes, I'm talking about standard salesforce object that related to tasks and extract files to the account level from them:
Opportunity.AccountId, Contact.AccountId...
Thanks, Lee


Split libraries for better understanding and testing

As a developer I would like to have the delivery-cluster libraries separated and well organized so when we add more code It is easier to code it with cool tests.

Work in progress tacked here: https://github.com/opscode-cookbooks/delivery-cluster/tree/afiune/145/major-library-cleaning

Merged! https://github.com/opscode-cookbooks/delivery-cluster/commit/f4aa15ab800dbd297cdc3efd5d886042091035e4


Get server headers

Hi,
I need to read the headers sent back by the server. Is it possible? In the file "WebSocket" there is a private field "mServerHeaders".

Sorry, in the onConnected method there is a field "headers"...


Only fallback to Syntax Tree if part of the project's Gemfile or gemspec

Motivation
Closes https://github.com/Shopify/ruby-lsp/issues/546
Implementation

There is a chance that auto-detection could be wrong in particular situations: If a gem has a runtime dependency on RuboCop, but the project using that gem uses syntax_tree, it will incorrectly default to using RuboCop. We could choose to mitigate this by:

displaying an error if the correct formatter cannot be reliably determined.
not applying any formatting unless a specific formatter is set

Automated Tests
Included.
Manual Tests
(in each case below, checkout this branch and add ruby-lsp to the Gemfile, with the appropriate path).

Choose a project which has neither RuboCop or syntax_tree
Attempt to format a Ruby file
Ensure no formatting is performance
Choose a project which uses RuboCop
Attempt to format a Ruby file
Ensure it's formatted using RuboCop (confirm via the logging in the LSP output panel)
Choose a project which uses Syntax Tree
Attempt to format a Ruby file
Ensure it's formatted using Syntax Tree (confirm via the logging in the LSP output panel)

My main concern is to prevent surprising the user with unwanted formatting.

I agree with Andy. Surprising aside, it's also inconvenient despite the good intention. I still have formatting disabled "globally" because I don't want to format code in projects like irb or debug. But I also don't want to ship that config per-project.


Fixed typo in snippet form_error_radio_show_errors.html

Fixed a typo in the snippet file form_error_radio_show_errors.html (Priestor pre posis -> Priestor pre popis)
What problem does the pull request solve?

Correcting typo in the file.
What type of change is it?

Typo fix

Thanks/Vďaka!


(WIP) Support label whitelisting

Work in progress; seeking feedback.

Closes #22.
Goals
This PR represents work towards supporting label whitelisting. The plugin currently supports label blacklisting only.
Notes

Whitelist can be specified using the onlyLabels configuration entry;
If a consumer specifies both onlyLabels and exemptLabels, onlyLabels takes priority and exemptLabels is ignored ([1], [2]).
New functionality has been designed to leave default plugin behavior unchanged.

Challenges
Currently, when the initial mark and sweep occurs, I see the following error:
16:10:58.167Z ERROR PRobot: Cannot read property 'number' of undefined
  TypeError: Cannot read property 'number' of undefined
      at Stale.mark (/Users/stkent/dev/personal/probot/stale/lib/stale.js:83:25)
      at Stale.markAndSweep (/Users/stkent/dev/personal/probot/stale/lib/stale.js:13:10)
      at process._tickDomainCallback (internal/process/next_tick.js:135:7)

This suggests to me that I'm not handling asynchronous calls properly in my changes. (Reminder: not a JS dev!) I'll add some inline comments after opening this PR explaining my current theories. Looking for feedback/pointers on how I might fix and improve this code ❤ .

@bkeepers would appreciate eyes on this at your convenience! :)

Hi @bkeepers,
This PR still needs some work; per the description:

Challenges
Currently, when the initial mark and sweep occurs, I see the following error:
16:10:58.167Z ERROR PRobot: Cannot read property 'number' of undefined
 TypeError: Cannot read property 'number' of undefined
     at Stale.mark (/Users/stkent/dev/personal/probot/stale/lib/stale.js:83:25)
     at Stale.markAndSweep (/Users/stkent/dev/personal/probot/stale/lib/stale.js:13:10)
     at process._tickDomainCallback (internal/process/next_tick.js:135:7)

This suggests to me that I'm not handling asynchronous calls properly in my changes. (Reminder: not a JS dev!) I'll add some inline comments after opening this PR explaining my current theories. Looking for feedback/pointers on how I might fix and improve this code ❤ .

If you could take a look at the async part of the changes and let me know what I'm doing wrong then I'd be happy to continue tweaking :)

You could checkout the pull request and the create a new PR https://help.github.com/articles/checking-out-pull-requests-locally/


Fix issue with empty initializer

For structs/struct_init.c, this yields a runtime error:
[ERR] Unhandled exception. System.InvalidProgramException: The JIT compiler encountered invalid IL code or an internal limitation.
[ERR]    at main()
[ERR]    at <SyntheticEntrypoint>()
[ERR]
Command exit code: -532462766

I think @kant2002 mentioned he still works on this one.


function Message.mention() should recognize both magic code and blank

see more in #362

@lijiarui do you have any plan to fix this issue? Please let me know what you think, thanks.

Yes, I will fix this later.

I do not know whether it's fixed.
But I'll close this issue due to not be active for a long time.


Optional enforcement of explicit namespace imports

Yesterday I was trying to import React into a file that would be transpiled and run on the browser, but kept getting errors about require not found. I think the problem is that the modules are being resolved in a way compatible with nodejs, but not for the browser. There are a dozen different module systems that can be targeted in tsconfig.json, and I find it really difficult to understand when I need a /// directive at the top of my file and when I don't. Sometimes it works to import {x] from x, other times it only works when you do x= require('x').
Ambiguities like this could be addressed with a rule to enforce stricter namespace import rules, preferably similar to those in C#.
Proposal

Provide a single consistent syntax for importing modules.
Add an optional compiler flag similar to noImplicityAny (which I think is awesome) which would enforce the following:

Every file must have a namespace.

Every module must be explicitly imported before use in that file, no exceptions.

Justifications:

If every file has a namespace, the rules of module resolution become much simpler and easier to understand.
Explicitly importing every module before use would prevent accidentally trying to use node api's on the browser, and vice-versa.
A single syntax for importing modules lets the user delegate the complex task of bundling dependencies to the compiler. This opens up a lot of possibilities for interesting and sophisticated build systems without overburdening the user.

Every file must have a namespace.

do not see why this is required.

Every module must be explicitly imported before use in that file, no exceptions.

this is true today. you can not use a module without importing it. see http://www.typescriptlang.org/docs/handbook/modules.html and http://www.typescriptlang.org/docs/handbook/namespaces-and-modules.html for more information.

@brhumphe This sounds like a loader configuration issue. What is your setup?

This is a repo with the minimal setup.
Targeting commonjs compiles fine, but when I try to target amd, system, umd, es6, or es2015, tsc gives the error Cannot find the module 'react'.
What is the mistake I am making here? And are there any compile-time checks or systems that would prevent it?

@brhumphe You need to set moduleResolution: node. It's the default when using commonjs, but not the others. See https://www.typescriptlang.org/docs/handbook/compiler-options.html.

Why isn't it the default for the others? And why doesn't commonjs support outFile?

Why isn't it the default for the others?

to maintain backward compatibility for user code before TS 1.6. issue https://github.com/Microsoft/TypeScript/issues/11434 tracks changing the default.

And why doesn't commonjs support outFile?

This is not a simple concat procese like in sysmte/amd. commonjs has specific meaning for require/filenames. if you want a module blunder for commonjs, use browserify or wepback.

So my origional proposal is more or less already how things work, and what tripped me up was an unexpected set of defaults, which this suggestion would not address. I'll go ahead and close it up then.


No focus on the 8-digit code form field (Android)

Scenario to reproduce
Samsung Galaxy J3 Prime
Android version 7.0
When users touch down on the form field to enter an 8-digit code on the “Enter your 8-digit number” screen there is no focus on the field.
Expected result
The form field should given focus for users to see where they are.
"WCAG
https://www.w3.org/WAI/WCAG21/Understanding/focus-visible.html
Screenshots

I think this is on iOS as well

So the issue here is that there is no styling to indicate the field is in focus after it is tapped. We would need some designs for what the focus style should look like before we do anything here.

@smcmurtry, while I agree about the styling, I think this issue all by itself is valid, and can be closed as is. The styling is a separate issue that applies whether the user taps in the field, or it is automatically selected.

The form field should given focus for users to see where they are.

"...see where they are"

a) Per @smcmurtry
We would need some designs for what the focus style should look like before we do anything here.

☝ Steve you flagged issues here before as the input focus isn't really multiple inputs it's a single input styled to look like multiple inputs - this should be discussed again.
b) I like what @jeberhardt did as far as Auto-focus (pops the Keyboard and gives focus to the input) but as @smcmurtry
notes:

...I think we should probably auto focus the headers on both this and the next page, so that focus starts from the top of the page when the user navigates to the page.

Does the non screen reader view in this case make the screen reader experience bad cc: @JuliannaR

I agree with the auto focus and giving focus to the input. Because there is some visual trickery that happens in the physical design as its actually all one input, can we focus on it like its just 1 element? @smcmurtry

Talked to @JuliannaR

Give focus to the header
Update the styling "for all types of users" on the input - perhaps just dim the underline for today to indicate that yes that part of the input has focus (no really but visually it does)
Ideally the "first" digit of the input will receive focus when moving to that area.

We will revist all of this after an initial launch.
@JuliannaR please confirm or correct the above as an interim solution.

@timarney @smcmurtry confirming the above. We will need to readdress post launch but this will enable different types of users to interact.

Sounds like a good plan @timarney @JuliannaR.

This is fixed in V18.


fix: EU distance unit were not correctly set

I think my pull request is not correct. Only the last commit should be in this pull request.

old commits are in this pull request


Mast: Watch (?-i:^Laker Builders Merchant$)

Mast requests the watch of the watch_keyword (?-i:^Laker Builders Merchant$). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:^Laker Builders Merchant$) has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Daniil in Charcoal HQ


Added k4a_image_get/set_system_timestamp_nsec

Fixes #198
Description of the changes:

Added k4a_image_get_system_timestamp_nsec() & k4a_image_set_system_timestamp_nsec().
Added k4a_image_get_device_timestamp_usec().
Added k4a_image_set_exposure_usec().
Deprecated k4a_image_get_timestamp_usec() & k4a_image_set_timestamp_usec().
Deprecated k4a_image_set_exposure_time_usec().
System Timestamp on Windows is based on QueryPerformanceCounter, while Linux is based on clock_gettime(CLOCK_MONOTONIC). There is a nice writeup here: ros2/rcutils#43.
libuvc needed to be updated to capture the time at the completion of last USB packet. So libuvc was converted from a patch to a fork.

Before submitting a Pull Request:

[x] I reviewed CONTRIBUTING.md
[x] I built my changes locally
[x] I ran the unit tests
[x] I ran the functional tests with a device
[ ] I ran the performance tests with a device

I tested changes on:

[x] Windows
[x] Linux

                                                   k4a_imu_sample_t *imu_sample,

This may be out of scope for this change, but figured I'd mention it:
k4a_imu_sample_t has (presumably device) timestamps embedded in it - do we need to provide a facility for getting system timestamps for IMU samples?

Refers to: include/k4a/k4a.h:255 in ac64c8d. [](commit_id = ac64c8da77041206a614015fa7a524be0fe2e89c, deletion_comment = False)

typeof is not C, I got lucky building before because that macro was never used in C code. I am reverting the change.

__typeof__ is not C, I got lucky building before because that macro was never used in C code. I am reverting the change.

Ugh, its a GNU extension. Good find. If we are never using these MIN and MAX macros in C code, then should we include them? C++ provides a much safer std::max and std::min that we can use.

__typeof__ is not C, I got lucky building before because that macro was never used in C code. I am reverting the change.

Ugh, its a GNU extension. Good find. If we are never using these MIN and MAX macros in C code, then should we include them? C++ provides a much safer std::max and std::min that we can use.

Talked with Wes offline, we are using these macros in C code. Looks good to me!


ui responsiveness hindered by exposure

🐛 Bug Report
The responsiveness of the save folder button and dialog seem to be impacted by the exposure time. High exposure times (>100ms) seem to make the dialog much slower, which should not happen because the UI and the image capture are handled in different threads.
🔬 How To Reproduce
Steps to reproduce the behavior:

Set integration time to 200ms .
Press the save folder button and navigate to any folder.

Environment

OS: Linux
CMake version: 3.22.1
Compiler: gcc

📈 Expected behavior
No delay in any UI component independently of the exposure time.

This seems to be related to starting and stopping the temperature thread in handleBaseFolderButtonClicked. This was originally intended as a safety guard when the temperature was being recorded to a file, but this is no longer needed now that the temperature is stored on the file metadata.


[Onnx] Integrate an interface to easily load exported ONNX models [will be updated]

🚀 The feature
We need to:

[ ] ensure all exported models can provide the information we need (for example: which postprocessor, vocab ,etc.)
[ ] setup build without TF and PT -> Onnxruntime should be enough
[ ] build an equivalent to the current ocr_predictor
[ ] make the predictor configurable about CPU / GPU (runtime configs)
[ ] add ability to export models in detection/recognition reference scripts
[ ] replace documentation section: preparing your model for inference with onnx saving / loading for prod

To solve before:

[ ]  #978

Other related Issues:
#789 #790
Related discussion:
#981
Motivation, pitch
As a user after exporting my models in onnx format i want to easily load these in docTR
Something like:
#-> same way as ocr_predictor

predictor = onnx_ocr_predictor(
    det_model='path/db_mobilenet_v3_small.onnx,
    reco_model='path/crnn_mobilenet_v3_small.onnx,
    provider='gpu' (default='cpu')
)

#-> same way as ocr_predictor

Hi @frgfm :wave:,
in short yes this would be the plan :sweat_smile:
a bit more in detail what i have had in mind: #981


What's the reason for this error on rk3399 pro??

rknn_init,  msg_load_ack fail, ack = 6(ACK_PERF_TOO_MANY_CLIENT), expect 0(ACK_SUCC)

I guess it means there are too many clients to the RKNN runtime. Maybe there are zombie Python processes from previous run attempts?


Workaround for SublimeTextIssues/Core#1877

On some occasions the environment variables are loaded after plugin_loaded is invoked. So the PATH variable may not be up-to-date and consequently the path to a server may not be found. This PR is a workaround for that. It waits 2 seconds and then initializes the servers, hoping that the environment variables are loaded in those 2 seconds.
Also see: https://github.com/SublimeTextIssues/Core/issues/1877

Thanks for identifying what sounds like a blocking bug. Great you reported it to the ST developers, as I understand build 3142 is a dev build.
I'm hesitant to accept a 2-second penalty for all OS-X users. Is it okay if we keep this PR as documentation for your workaround, and hope for the next dev build to fix the regression?
If in the future a larger number of OS-X users ends up stuck on that build, we could perhaps active the fix only when running in build 3142.

Sure, let's wait and see what happens 👍

An alternatively solution is to execute the commands via a login shel, /bin/bash -l -c 'pyls' for example.

It's not really a huge issue, I can just run "LSP: Restart Servers" from the command palette and by that time my env vars are loaded. Maybe put that in a FAQ section for OS X users.

@rwols I see some comments were added to the issue, it does not look like sublime will fix this immediately.
I've avoided the PATH issue on OS-X by launching with subl from the command line - does that circumvent your issue? There is also the old https://github.com/int3h/SublimeFixMacPath package.

After trying SublimeFixMacPath my path seems to be correct from the get-go 👍

Great, then it might be an older known issue with a reliable-ish workaround.
I added some links and explanation in a new troubleshooting section of the documentation here: https://github.com/tomv564/LSP/commit/0c55bea84f592d7e114e363050e95d1496269040
I hope that's a sufficient replacement for this pull request?

Just going to throw in another perspective.
Do you really need all the environment variables, or do you just need to know how to call the server? If it's the latter, why not let me configure the path to the server? Just let me tell LSP where it's at and it removes the need to wait for anything. Yes it's more setup, but I'd rather do a little setup than use a workaround like having to start from command line, or install another package that isn't maintained and seems a bit iffy to begin with. It should be pretty easy to make this optional too.

Ya, I remember I ran into some issues when using that fix and the issue was very subtle and hard to be discovered. After that, I have move it and just try my best to configure each package.

Since we have a few better performing workarounds (FixMacPath, hardcoding the command, launchctl setenv), I will close this PR now.


Creating a configuration file so the module actually gets picked up.

Can this be a config.yml inside _config folder instead ? Is better from a performance standpoint


mint button without using MoralisProvider?

So If I want to move forward using this boilerplate to mint NFT's with the current authorized metamask wallet how would I go about doing that? So far all of the documentation I see is using react-moralis MoralisProvider which seems to be incompatible with the way this application is configured.

Most of the minting dapps using Moralis use the MoralisProvider function.
https://github.com/kjmczk/nextjs-nft-mint-dapp/blob/main/pages/_app.tsx#L8
The problem is that the latest version of Moralis doesnt seem to work with react-moralis which provides the useWeb3ExecuteFunction, or at least is an anti pattern by loading 2 versions of the same library.
https://github.com/kjmczk/nextjs-nft-mint-dapp/blob/main/components/Mint.tsx#L43
Further the most recent version of moralis doesn't provide any contract write functions.
So how I got my minting working using this boilerplate is by using an unwrapped Web3 dependency.
import Web3 from 'web3';
const web3 = new Web3(Web3.givenProvider);
const contract = new web3.eth.Contract(nftAbi, contractAddress);
        const response = await contract.methods
          .mint(mintAmount, autoManifest)
          .send({ from: user.address, value: mintPrice });

In conclusion, this repo is basically a readonly boilerplate. It would be nice to provide at least one write function in here for completeness. Otherwise consider renaming the repo: ethereum-readonly-boilerplate.
Closing for now.


Selfhosted Shellhub Terminal/SSH issue

Using Shellhub 0.5.0
When self-hosting shellhub on a Ubuntu Server I cannot use ports 80 and 22 for shellhub.
The shellhub starts in the docker container with custom ports correctly and logging in and adding a device everything works.
Starting a terminal session to an agent does not work and terminal does not log in. no information in ssh log as well.
ssh connection with the sshid also results in a failed connection.
ssh agent log:
2021/02/15 12:46:24 http: superfluous response.WriteHeader call from github.com/shellhub-io/shellhub/pkg/httptunnel.(*Tunnel).Router.func1 (httptunnel.go:61)
time="2021-02-15T12:46:27Z" level=info msg="Handling session request" session=<removed> target=<removed>
time="2021-02-15T12:46:27Z" level=error msg="Invalid session target" session=<removed>
2021/02/15 12:46:35 http: superfluous response.WriteHeader call from github.com/shellhub-io/shellhub/pkg/httptunnel.(*Tunnel).Router.func1 (httptunnel.go:61)

This has already been fixed in master branch. See #699

what is the current ETA for the next release or is there a way to test a patch on 0.5.1?

what is the current ETA for the next release or is there a way to test a patch on 0.5.1?

We will release a new version next week

I'm closing this as the issue has been already fixed.

Testing in v0.6.0 shows the issue is still there.
When trying to connect to the agent through ssh:
ssh usr@<sshid> -P 2223
the server locks up the ssh port and gives following message on the cmd:
ssh_dispatch_run_fatal: Connection to <IP> port 22: Connection timed out
in .env.override the ssh port is set to 2223

ssh usr@<sshid> -P 2223

Aren't you confusing -P with -p?

ssh_dispatch_run_fatal: Connection to  port 22: Connection timed out

I don't recognize this log message being from ShellHub.

Yes sorry this should have been -p.
The log message is not from shellhub directly but it seems that the agent is calling calling the server on the wrong port more than once until the fail2ban adds my IP to the blacklist and ssh terminates with a connection error.

Yes sorry this should have been -p.
The log message is not from shellhub directly but it seems that the gateway is calling calling the server on the wrong port more than once until the fail2ban adds my IP to the blacklist and ssh terminates with a connection error.

I cannot reproduce this issue here with custom SSH and HTTP ports

@suegy please share the override file you are using with the docker-compose.

here is the .env.override:
# The HTTP listen port for the ShellHub web-based GUI, API and Reverse SSH tunn$
# Values: any free port on host
SHELLHUB_HTTP_PORT=8081

# The HTTPS listen port for the ShellHub web-based GUI, API and Reverse SSH tun$
# Values: any free port on host
SHELLHUB_HTTPS_PORT=8082

# The SSH listen port for incoming SSH connections to devices
# Values: any free port on host
SHELLHUB_SSH_PORT=2222

# Domain of the server
# NOTICE: Only required if automatic HTTPS is enabled
# Values: a valid domain name
SHELLHUB_DOMAIN=<url>

and here the docker.compose.override.yml:

version: '3.7'
services:
  ssh:
    restart: unless-stopped
  ui:
    restart: unless-stopped
  gateway:
    restart: unless-stopped
  mongo:
    restart: unless-stopped
    volumes:
      - /usr/local/mongo/data:/data/db

I did not modify any other shellhub files

@suegy are you sure you haven't' blocked by fail2ban?

Yes I do get blocked by fail2ban upon trying to connect on ssh as the connections seems to fail and retry to quick.

@suegy as this is a fail2ban issue, what do you think we should do at ShellHub side?

This seems to be an issue due to the use of the fail2ban so we're closing it. If need, please reopen.


Change all exercises to work with updated test runner

Following exercism/cpp-test-runner#17 I changed all exercises *_test.cpp file and CMakeLists.txt so they can be run using a precompiled version of Catch2.
@ErikSchierboom how do I verify that this actually works in a prod-like environment?

@silvanocerza Once this is merged, it will deploy to https://exercism.lol where you can test it.

@silvanocerza I've just checked and it works perfectly!

Awesome!


Start ImageManager housekeeping in Run().

Fixes #7762
Fixes #5634
/cc @dchen1107 @yujuhong @fgrzadkowski

LGTM. Thanks for fixing this!

Merging since shippable is green.

Thanks for the fix @vmarmol !


Implement monoidal table API

Depends on #314. Before un-drafting this, I want to make sure all the tests are enabled and run successfully.

Just a quick remark that you should probably enable some of the monoidal class tests manually, because currently they pass the expectFailures with the error ErrExpectedMonoidalTable

Yes, that's what I mean by making sure the tests are enabled.

Could you maybe list here which TODOs those are?

There are a few open questions about ResolveValue:

revisit totality requirement
potentially move it to another module
potentially add SerialiseValue as a superclass


[Improvement]: The Transaction page displays all snapshots as well as the producer of each snapshot

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
Currently, the Transaction page ignores the snapshots generated by optimizing and only displays the snapshots written by users, and this design has some adverse effects.
For the current functionality, hiding the snapshots generated by optimizing cannot reflect the actual changes on the table, including:

The current snapshot cannot reflect the latest state of the table.

Can't view the information about optimizing commits (such as snapshot summary) and the order of commits with other commits. Sometimes it is also useful to confirm whether optimizing commits have been successful from here.

If after expiring snapshots, only the snapshots generated by optimizing are left, the transaction page will be empty.

For future functionality, it also brings some limitations:

It affects the display of Tags and Branches. Tags and branches may appear on any snapshot, including those generated by optimizing.

It affects support for Time Travel. Performing time travel on snapshots after optimizing will bring performance benefits to queries.

It affects support for Rollback. Rollback to the snapshots generated by optimizing should also be supported.

How should we improve?

Display all snapshots on the Transaction Page.
Add a column Producer to display the producer of the snapshot, like OPTIMIZE INGESTION.

Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Subtasks
No response
Code of Conduct

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

If this happens, is optimized page, all other extended pages(like expire, clean processes) would be unnecessary.
From my view, transaction should be considered as user action, any processes which do not involve data change shouldn't belong to transactions. Think about database like MySQL. If it is necessary to showcase these commits in somewhere, consider displaying them on the optimized page or process related pages?

Thank you for sharing your thoughts. While I agree that processes that do not involve data change may not necessarily belong to transactions, I believe that displaying all snapshots on the Transactions page is still necessary for a complete historical view of the table.
As I mentioned earlier, Amoro currently lacks the ability to display complete historical snapshots, and this functionality serves as the basis for features such as Time Travel, Rollback, Tags, and Branches. While displaying snapshots on the  process-related pages(like Optimizing page) may be useful, it does not provide a comprehensive view of the table's historical state.
Therefore, I still believe that transforming the Transactions page into a Snapshots page is the best solution to address this issue and provide users with a complete historical view of the table. This way, the Transactions page will be categorized under state-related information, while other process-related pages (like Optimizing, Operations) will remain unchanged.
I hope this explanation clarifies my viewpoint, and I will provide a more detailed design.


Moving classes via drag & drop opens the edit panel

This panel should only open on single click.

Current: MouseDown opens the panel
Expected: MouseClick opens the panel


Reduce logging verbosity when disconnecting an unknown peer

This fixes spamming in Cumulus with messages like
[Parachain] Trying to disconnect unknown peer XZ from SetId(0).

We try to disconnect a peer from PeerStore when its reputation drops below the BANNED_THRESHOLD, and in case it was already disconnected by the syncing code this message is generated. The operation itself is a no-op in this case, so nothing to worry about.

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2921969


Viewing the broadcast

Hello, such a question, there is a broadcast of the game, where you need to log in to your my.games account, and accordingly, when viewing from your account, they give a reward. Is there an option to attach an account login to your script?

Hi,
this project focusing on chat interactions, such as moderation, ads, word games and so on. The features you are asking for are something different, and should be implemented directly in browser in my opinion.
It's should be some kind of Better Twitch TV chrome extension, which works directly with service's web page, but for vkplay.
If we are talking about the implementation itself, then it is not difficult. Just start the listener and wait for the reward event.

Hi,
this project focusing on chat interactions, such as moderation, ads, word games and so on. The features you are asking for are something different, and should be implemented directly in browser in my opinion.
It's should be some kind of Better Twitch TV chrome extension, which works directly with service's web page, but for vkplay.
If we are talking about the implementation itself, then it is not difficult. Just start the listener and wait for the reward event.

The browser puts a load on the computer, so I would like to learn more about the implementation of multi-user browsing. Tell me, what is the listener mode?

Hi,
this project focusing on chat interactions, such as moderation, ads, word games and so on. The features you are asking for are something different, and should be implemented directly in browser in my opinion.
It's should be some kind of Better Twitch TV chrome extension, which works directly with service's web page, but for vkplay.
If we are talking about the implementation itself, then it is not difficult. Just start the listener and wait for the reward event.

The browser puts a load on the computer, so I would like to learn more about the implementation of multi-user browsing. Tell me, what is the listener mode?

Here's example for twitch:
https://github.com/night/betterttv/blob/master/src/modules/auto_claim/index.js
I guess vkplay works the same way, and you can just copy-paste the solustion, changing the neccesary css-classes.


[Add] SEND ANIMATION BUTTON

Details
Adding a cool and awesome dynamic button, which on hover acts like this (in the video below) :-
Screenshots
https://github.com/Spyware007/Animating-Buttons/assets/100849696/bace0e96-a696-4f90-8f01-7cfe2a7307de
Type of Contribution

[ ] Update to an existing Animated Button
[X] Adding a new Button
[ ] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

Please check #1346


Deprecate internal release mappings in favor of Label Matching

Context
Currently in order to track ownership of resources, a global configuration map secret is used. This allows yoke to know if you are trying to create or override a resource in release B that is owned by release A ahead of time.
This approach is ok, however it poses certain problems around rbac for users wanting to use the yoke cli. If a non-admin user cannot read the ownership secret they cannot proceed.
So instead, we should use the labels that mark the release of the resource. This way if we try to apply a change to a resource owned by a different release we can fail early without having to check a global map. This will be more costly in terms of requests, but will be an overall improvement in usability.

Done


[Request] More uses for our hard-earned cash!

Currently you can't do a whole lot with your cash outside of healing your Pokémon and giving it to others.
Gambling would be fun. Blackjack? Lottery? Idk, just trying to throw out some random ideas.

An interesting idea would be "cash only" rooms where you can spend your earnings to gain access to a room subscriber service style so much cash per hour/day/month. That or a "private rooms" subscription, where they can pay the virtual currency to have their own private room named whatever they want on the server in which they're the mod for that room, with all the rights to modify the channel however they see fit.

we have betroll and betflip now, more to come. No need for this issue


Added support subscription attributes (RawMessageDelivery) and message attributes

Codecov Report

Merging #40 into master will increase coverage by 4.68%.
The diff coverage is 57.73%.

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage   36.36%   41.05%   +4.68%
==========================================
  Files          21       22       +1
  Lines         341      419      +78
  Branches        9        6       -3
==========================================
+ Hits          124      172      +48
- Misses        217      247      +30

Impacted Files
Coverage Δ

...ain/scala/me/snov/sns/actor/RawProducerActor.scala
0% <0%> (ø)

...rc/main/scala/me/snov/sns/actor/PublishActor.scala
0% <0%> (ø)
:arrow_up:

.../main/scala/me/snov/sns/actor/SubscribeActor.scala
0% <0%> (ø)
:arrow_up:

src/main/scala/me/snov/sns/api/SubscribeApi.scala
97.56% <100%> (+1.26%)
:arrow_up:

src/main/scala/me/snov/sns/model/Message.scala
56.25% <100%> (+56.25%)
:arrow_up:

...scala/me/snov/sns/response/SubscribeResponse.scala
61.16% <100%> (+10.54%)
:arrow_up:

...rc/main/scala/me/snov/sns/model/Subscription.scala
50% <50%> (+50%)
:arrow_up:

src/main/scala/me/snov/sns/api/PublishApi.scala
75% <71.42%> (+3.57%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix page title

It should be correct now:

We should probably move Color Palettes outside but this is fine

You are absolutely right. Fixed it. Thanks!


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/272 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit 9419989:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 9419989:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-01-10-08-49-07 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-01-10-08-49-07

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add Claude 3.5 Haiku

Description
Updates the model list to support Anthropic's new 3.5 Haiku model
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing

PR https://github.com/raycast/extensions/pull/15224 has already been created to add support for the new model.

Closing in favor of #15224


Hello World | Cookieschen’s blog

https://cookieschen.github.io/2020/01/31/hello-world/
Welcome to Hexo! This is your very first post. Check documentation for more info. If you get any problems when using Hexo, you can find the answer in troubleshooting or you can ask me on GitHub. Quick

test


Implement a LWWRegister CmRDT

This is an initial implementation of a LWWRegister using wall clocks (system time) to resolve conflicts. The register is 'nullable', not sure whether this is a good idea, but the MVRegister seemed to be too. Im new to eventuate, and open to feedback

Build break with the Cassandra integration tests related to connections. The tests passed locally and the commit was not related to cassandra. Any ideas?
All host(s) tried for query failed (tried: /<IP_ADDRESS>:9142 (com.datastax.driver.core.TransportException: [/<IP_ADDRESS>:9142] Cannot connect)

Thanks a lot for your PR, I'll take a closer look tomorrow. Regarding the test failure, see my comment on Gitter.

I just re-started the build and the compile errors are now because of my recent changes in master. Please rebase your PR and go a git push -f. Thanks!

Your quite right on all points and the vector clock is nice addition if clocks are synchronized, i'll take a look at this shortly.

@ianclegg are you still planning to work on it?

Yes, unless it is blocking you - I'll pick this up again this evening

@ianclegg not at all, was just curious :smiley:. Great that you continue working on it.

@krasserm It seemed to make sense to reuse Versioned to encapsulate the value and the vector clock. Is it fair to use the existing 'creator' property of Versioned to store the emitter id too. I guess what I would like to do is have both the vector clock and the wall clock in Versioned
something like this:
case class Versioned[A](value: A, vectorTimestamp: VectorTime, timestamp: Long, creator: String = "")

Then I can refactor the systemTimestamp out
case class LWWRegister[A](versionedValue: Versioned[Option[A]] = Versioned(None, VectorTime(), ""),
                          systemTimestamp : Long = 0L) {

  def value: A =
    versionedValue.value.get

  def set(newValue: A, vectorTimestamp : VectorTime, newTimestamp : Long, emitterId : String): LWWRegister[A] = {
        // First, compare the vector timestamps
    if ((vectorTimestamp > versionedValue.updateTimestamp) ||
        // If the vector timestamps are the same, compare the system timestamps
        (vectorTimestamp == versionedValue.updateTimestamp) &&
          (newTimestamp > systemTimestamp) ||
        // If the system timestamps are the same, choose the value with the lowest emitter
        (newTimestamp == systemTimestamp) &&
        (emitterId > versionedValue.creator)
    )
        copy(versionedValue.copy(Some(newValue), vectorTimestamp, emitterId), newTimestamp)
    else
      copy(versionedValue, systemTimestamp)
  }
}

Is this along the lines of what you were thinking?
Also,  there are cases where it might be desirable to ignore the vector clock and use the wall clock. For example, in financial systems where a trader is only ever interested in the latest price - regardless of the ordering of previous events.

Is it fair to use the existing 'creator' property of Versioned to store the emitter id too

Yes.

have both the vector clock and the wall clock in Versioned

I'm a bit hesitant to a add a wall clock timestamp to Versioned. There are many other usages of Versioned in Eventuate (and the example application) which don't need that timestamp. I'd rather propose a field of type Option[(Versioned[A], Long)] to store what you need. A later refactoring can still be done when needed.

// If the vector timestamps are the same

This will never be the case. If vectorTimestamp > versionedValue.updateTimestamp is false then
vectorTimestamp conc versionedValue.updateTimestamp will be true. Vector timestamps are unique unique event identifiers.

There are many other usages of Versioned in Eventuate (and the example application) which don't need that timestamp

agreed

I'd rather propose a field of type Option[(Versioned[A], Long)] to store what you need

agreed. The reasoning behind Versioned[Option[A]] was to enable the register to be cleared (ie. set to None) by a future operation - but Option[(Versioned[A], Long)] is closer to the behavior of the existing MVRegister

@ianclegg sorry for the delay, haven't been notified by Github that you've made new commits. Will make a detailed review tomorrow.
Do you mind also updating the sentence

Eventuate currently implements 3 out of 12 operation-based CRDTs specified in the paper. These are Counter, MV-Register and OR-Set.

in the CRDT section of the user guide? Should you want to build the documentation locally, here are the instructions.

@ianclegg how do you plan to continue with this PR?

@krasserm I just got back from vacation. I owe you a beer, the match expression is much more readable! I'll push an update later today

@krasserm
Versioned is not used on the interface of the the existing ORSet and MVRegister, so clients can use the CRDTs without knowing about it.
mvReg.set(1, vectorTime(1, 0))
orSet.add(1, vectorTime(1, 0))

Changing set() method in LWWRegister to take a LWWVersioned[A] means clients now need to first build a LWWVersioned which makes it a little more difficult to use. The match expression is nice, perhaps if I build the LWWVersioned within set() function.
def set(v: A, vt : VectorTime, st : Long, emitterId : String): LWWRegister[A] = {
    val nvv = LWWVersioned(v, vt, st, emitterId)
    val result = versionedValue match {

so clients can use LWWRegister much like MVRegister and ORSet?
lwwReg.set(1, vectorTime(1, 0), 0, "source-1")

Great, looks good to me now. Thanks @ianclegg. Please squash your commits and rebase on latest origin/master, then I'll merge.

@krasserm rebased and squashed