Add StartBootstrap Vitality theme

https://github.com/hugo-startbootstrap-themes/hugo-vitality-theme
Hugo Vitality would be the first premium theme of StartBootstrap. The original has to be downloaded and installed from StartBootstrap in two simple steps.

I notice this is a paid theme -- which we'll have to discuss how to handle. To my understanding, all the current themes are more or less "clone and use".
One comment, though, your exampleSite contains n different toml files with a prefix, and I understand the motivation -- but it would be nice if you could make a copy of one of them into config.toml for the autogenerated demo site.
/cc @digitalcraftsman

Yes, I thought about the problem that it would be the first paid theme. It's just a nice and powerful theme and $10 are not really a high barrier (for people in the west).
I added the config.toml.

$10 are not really a high barrier (for people in the west).

No, the price seems fair, but we have to think about how to mark them so we can filter them etc., and consider if this is an issue for the demo side of it (we're not spending $10 on every paid theme that comes along, to put it like that).

I would pay the $10 for your demo site ;-) Maybe that would a possible regulation also for other contributors.

we have to think about how to mark them so we can filter them etc., and consider if this is an issue for the demo side of it

Marking a theme as non-free shouldn't be a problem by adding a banner. Currently, users can somewhat filter by tags. This could be extended by an JS-based filter plugin but depends on how you want to let users filter the themes.
In general, I would be awesome to see more paid themes. Other SSGs built a small ecosystem around themselves and by paying a small amount of money makes it likely that we'll see more high-quality themes.
Does anyone already thought about how we should host this theme with a demo-site (except the 10$ fee)?
At the moment users have to make the full source code of the theme publicly available in order to generate a theme demo. This way we would redistribute the theme which is likely to be permitted by the bootstrap theme's license.

In general, I would be awesome to see more paid themes.

I think that would be cool to have such an ecosystem for hugo. It could increase theme quality.

Does anyone already thought about how we should host this theme with a demo-site (except the 10$ fee)?

StartBootstrap offers an extended licence for their premium themes. For Vitality that would be $500. Maybe they would agree if an hugo marktplace sells Hugo version of it for $10 and they get the money for the first 50 sells (as they happen). And from then the income could be shared between author and project. Technically it could just be a shop website with Paypal payment.
I would really like to port more of their non-adminstrative premium themes for such a place. ;-)

Does anyone already thought about how we should host this theme with a demo-site (except the 10$ fee)?
At the moment users have to make the full source code of the theme publicly available in order to generate a theme demo.

It would depend on how popular the commercial themes get (that is, how many get made), but for the time being it seems the responsibility of the theme author to provide a full copy to the person managing themes.gohugo.io so that this theme can be placed in the correct directory and rendered when the Hugo theme site build.
I don't think it would be a good idea to look into ways to automate that or have the author submit their own commercial theme through an online application, given the time investment of those tools.

StartBootstrap offers an extended licence for their premium themes. For Vitality that would be $500. Maybe they would agree if an hugo marktplace sells Hugo version of it for $10 and they get the money for the first 50 sells (as they happen).

Why overcomplicate things? Just host the theme on themes.gohugo.io so people can take a test drive, and add a link back to wrapbootstrap.com. A Hugo marketplace seems an additional burden with also a lot of additional legal and financial risks. I don't think Hugo should meddle with that.
I'm not against Hugo earning money with this. From one standpoint that seems only fair since themes.gohugo.io provides a service (promotion, advertising, demoing) to the author of a commercial theme. But there are easier ways to earn money with that: ask the theme author a fee when putting up the theme or become an affiliate for wrapbootstrap.com.
Creating a Hugo marketplace seems like re-inventing the wheel. (If you haven't guessed by now, my personal bias is a strong believe in focus. 😄 )

Creating a Hugo marketplace seems like re-inventing the wheel.  I understand that. From my side it was just an idea to maybe get a higher quality with such a market place. Sometimes you can save a lot of time by spending $10 or $20. I watched me do it ...
Here are two links to demos of the flavours of the theme, currently hostet on my server ... I could send them as a package to you. I have two licenses for them.
Agency
Video

I didn't notice this before, but this issue belongs at
https://github.com/spf13/hugoThemes


Bullet points with dash

By default, the Bullet points are generated with an asterisk.
Is there any way to change the default to use a dash?

I think it should allow to config using + - *

As of v0.9.0, bullet points can be configured to use plus sign, dash, or asterisk. Thanks for the contribution, @FFengIll!

Thank you!


chore(components): updated _all.scss with missing components

fixes https://github.com/patternfly/patternfly-next/issues/2683
cur=`pwd`
base="src/patternfly"

for dir in layouts components; do
path="$cur/$base/$dir"
file="_all.scss"

cd $path
find . -name "*.scss" | grep -v $file | sed -e 's/^/@import "/g' | sed -e 's/$/";/g' | sort > $file

done

PatternFly-Next preview: https://patternfly-next-pr-2684.surge.sh

I like your bash script!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bump rancher-webhook version to v0.3.2-rc6

https://github.com/rancher/rancher/issues/39891
https://github.com/rancher/rancher/issues/39248
https://github.com/rancher/rancher/issues/39995
https://github.com/rancher/rancher/issues/39992
https://github.com/rancher/rancher/issues/40009

This PR is out of date and no longer needed.
Closing it now


Introduce option to use clang++, and option to skip apt packages

Add option to use clang++ instead of g++ for compiling rascsi
Add option to skip apt-get package installation (this saves a significant amount of time running the script on a Zero when you already have the required packages installed)
Add a line to the script menu showing current options
Surrounding cleanup

I did not find apt-install for clang++ in the changes. I don't think it is installed per default, but I'm not sure.

Good catch, I forgot about that. Added now.

@uweseimet I believe clang is a meta package that pulls in the latest stable C, C++, and Obj-C compilers.
$ sudo apt install clang --no-install-recommends
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following additional packages will be installed:
  clang-11 libclang-common-11-dev libclang-cpp11 libclang1-11 libllvm11 libobjc-10-dev libz3-4
Suggested packages:
  clang-11-doc
Recommended packages:
  llvm-11-dev libomp-11-dev
The following NEW packages will be installed:
  clang clang-11 libclang-common-11-dev libclang-cpp11 libclang1-11 libllvm11 libobjc-10-dev libz3-4
0 upgraded, 8 newly installed, 0 to remove and 7 not upgraded.
Need to get 37.3 MB of archives.
After this operation, 190 MB of additional disk space will be used.
Do you want to continue? [Y/n]

I don't think I'm on board with having easyinstall install clang. This is going to take extra space and extra time to install it on everyone's raspberry pi, when very very few people are using it.
@uweseimet - is your intention to switch over completely to clang++? If so, then I'm OK with leaving it in. But then I would expect a PR coming to switch the Makefile over.

@akuker As I already mentioned in my ticket, I'm totally fine with rejecting it. This is just a low prioriity suggestion, not more than that.
In general, easyinstall should install only what's needed according to the options the user has selected. If it still installs everything that might potentially be needed I suggest to improve this.

As long as we provide this as "experimental" feature I agree it would make sense to install clang only when explicitly chosen. Save the majority of users the overhead and bloat of having those packags.

I leave the decision to you. I don't have a strong opinion on this. I mainly created this ticket because @akuker mentioned that some users (still) have issues with compiling due to low memory, and I verified that there are no issues with clang++.

Let's do it! Let's switch to clang :-)

@uweseimet - would you like to update the Makefile? Or should I?

@akuker I suggest we hold off on this PR until the Makefile changes have been made. Do we want to offer gcc as a command line option for easyinstall, e.g. --use-gcc, for users running very old Raspbian versions that might lack a compatible clang compiler?

So since it seems we will need g++ in the Makefile for CI/CD for the time being (more research and testing required) I propose what we do in this PR, is to pivot to clang-11 as the default options in easyinstall and keep gcc in the Makefile. This gives us the benefit of the performance boost for users upgrading their RaSCSI installations, without breaking UTs and static analysis in the short term.
Thoughts?

Sounds good to me!


Amazon Appstore will not connect

Steps to reproduce
I Open the Amazon Appstore (it says it updated to latest version), and it always gives a connection error. My Internet connect is fine.
✔ Expected Behavior
Should connect
❌ Actual Behavior
Connection error
Other Software
WSA 1.7.32815.0
Have you specified the version of Windows Subsystem for Android under Milestone?

[X] I have set this issue's milestone

Hi, it looks like you're running a much older version of WSA and the Amazon Appstore that might be leading to some networking issues. Have you tried updating to the latest version through the Microsoft Store (we've resolved many of these issues in our newer releases)?

That's why I am trying to run the app store. I can't seem to update WSA. How do I get the latest WSA?

Please uninstall the Amazon Appstore (which will uninstall the subsystem), and then follow the steps here to install the latest version.

That worked. Thanks


chore: release master

:robot: I have created a release beep boop
@allaround/all-components: 1.8.10
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/input bumped from ^2.8.6 to ^2.8.7
@allaround/container bumped from ^1.6.2 to ^1.7.0
@allaround/sidebar bumped from ^1.0.4 to ^1.0.5
@allaround/navbar bumped from ^1.0.5 to ^1.0.6
@allaround/hooks bumped from ^1.0.6 to ^1.0.7
@allaround/textarea bumped from ^1.0.5 to ^1.0.6
@allaround/tags bumped from ^1.0.6 to ^1.0.7
@allaround/dropdown bumped from ^1.0.6 to ^1.0.7
@allaround/select bumped from ^1.0.6 to ^1.0.7
@allaround/upload bumped from ^1.0.6 to ^1.0.7
@allaround/scheduler bumped from ^1.0.6 to ^1.0.7
@allaround/video bumped from ^1.0.7 to ^1.0.8
@allaround/image bumped from ^1.0.6 to ^1.0.7
@allaround/checkbox bumped from ^1.0.6 to ^1.0.7
@allaround/tooltip bumped from ^1.0.6 to ^1.0.7
@allaround/switch bumped from ^1.0.3 to ^1.0.4
@allaround/progress-bar bumped from ^1.0.3 to ^1.0.4
@allaround/notification bumped from ^1.0.5 to ^1.0.6
@allaround/placeholder bumped from ^1.0.3 to ^1.0.4

@allaround/checkbox: 1.0.7
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/container bumped from ^1.6.2 to ^1.7.0
@allaround/tooltip bumped from ^1.0.6 to ^1.0.7
@allaround/input bumped from ^2.8.6 to ^2.8.7

container: 1.7.0
1.7.0 (2023-08-25)
Features

add mouse event props (#91) (a0ed0cb)

@allaround/dropdown: 1.0.7
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/container bumped from ^1.6.2 to ^1.7.0
@allaround/tooltip bumped from ^1.0.6 to ^1.0.7

@allaround/hooks: 1.0.7
Dependencies

The following workspace dependencies were updated

devDependencies

@allaround/container bumped from ^1.6.1 to ^1.7.0
@allaround/notification bumped from ^1.0.5 to ^1.0.6

@allaround/image: 1.0.7
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/upload bumped from ^1.0.6 to ^1.0.7
@allaround/tooltip bumped from ^1.0.6 to ^1.0.7
@allaround/container bumped from ^1.6.2 to ^1.7.0

@allaround/input: 2.8.7
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/hooks bumped from ^1.0.6 to ^1.0.7
@allaround/container bumped from ^1.6.2 to ^1.7.0
@allaround/tooltip bumped from ^1.0.6 to ^1.0.7

@allaround/navbar: 1.0.6
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/container bumped from ^1.6.2 to ^1.7.0

@allaround/notification: 1.0.6
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/container bumped from ^1.6.2 to ^1.7.0

@allaround/placeholder: 1.0.4
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/container bumped from ^1.6.2 to ^1.7.0

@allaround/progress-bar: 1.0.4
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/container bumped from ^1.6.2 to ^1.7.0

@allaround/scheduler: 1.0.7
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/hooks bumped from ^1.0.6 to ^1.0.7
@allaround/select bumped from ^1.0.6 to ^1.0.7
@allaround/container bumped from ^1.6.2 to ^1.7.0

@allaround/select: 1.0.7
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/container bumped from ^1.6.2 to ^1.7.0
@allaround/hooks bumped from ^1.0.6 to ^1.0.7

@allaround/sidebar: 1.0.5
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/container bumped from ^1.6.2 to ^1.7.0

@allaround/switch: 1.0.4
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/container bumped from ^1.6.2 to ^1.7.0

@allaround/tags: 1.0.7
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/hooks bumped from ^1.0.6 to ^1.0.7
@allaround/input bumped from ^2.8.6 to ^2.8.7
@allaround/textarea bumped from ^1.0.5 to ^1.0.6

@allaround/textarea: 1.0.6
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/container bumped from ^1.6.2 to ^1.7.0

@allaround/tooltip: 1.0.7
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/hooks bumped from ^1.0.6 to ^1.0.7

@allaround/upload: 1.0.7
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/hooks bumped from ^1.0.6 to ^1.0.7
@allaround/input bumped from ^2.8.6 to ^2.8.7
@allaround/container bumped from ^1.6.2 to ^1.7.0

@allaround/video: 1.0.8
Dependencies

The following workspace dependencies were updated

dependencies

@allaround/progress-bar bumped from ^1.0.3 to ^1.0.4
@allaround/hooks bumped from ^1.0.6 to ^1.0.7
@allaround/container bumped from ^1.6.2 to ^1.7.0
@allaround/tooltip bumped from ^1.0.6 to ^1.0.7
@allaround/upload bumped from ^1.0.6 to ^1.0.7

allaround-components: 2.10.0
2.10.0 (2023-08-25)
Features

add mouse event props (#91) (a0ed0cb)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/allaround-components-v2.10.0 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/all-components-v1.8.10 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/input-v2.8.7 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/container-v1.7.0 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/sidebar-v1.0.5 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/navbar-v1.0.6 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/hooks-v1.0.7 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/textarea-v1.0.6 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/tags-v1.0.7 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/dropdown-v1.0.7 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/select-v1.0.7 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/upload-v1.0.7 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/scheduler-v1.0.7 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/video-v1.0.8 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/image-v1.0.7 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/checkbox-v1.0.7 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/tooltip-v1.0.7 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/switch-v1.0.4 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/progress-bar-v1.0.4 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/notification-v1.0.6 :sunflower:

:robot: Release is at https://github.com/wholesome-ghoul/allaround-components/releases/tag/placeholder-v1.0.4 :sunflower:


Added elements for listing child pages and section navigation

like the ChildPagesBlock and SectionNavigationBlock
Reorganized tests to follow code directory structure

something went wrong with travis, and making another pr was a possible fix.
Was on travis' end


Fix empty command bug

Bug: creategroup command without parameters does not show error.
Status: fixed

Codecov Report

Merging #120 (cec44aa) into master (d2b1e3f) will decrease coverage by 0.08%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #120      +/-   ##
============================================
- Coverage     61.12%   61.04%   -0.09%
  Complexity      407      407
============================================
  Files            81       81
  Lines          1492     1494       +2
  Branches        161      162       +1
============================================
  Hits            912      912
- Misses          541      543       +2
  Partials         39       39

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/ArgumentTokenizer.java
84.09% <0.00%> (-4.01%)
:arrow_down:

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 d2b1e3f...cec44aa. Read the comment docs.


adding support for retrieving Tags

Will Close the Open Issue "Support for Tags #65"
This change will allow you to pull all tags in your Pager Duty Instance , provided screenshot below showing testing of the new code

Hey there,
Thanks for your contribution but we decided to deprecate this project. Please use PDPyras instead.


Filter on optional vertices and edges in Cypher

For this compiler, it's possible to filter on optional vertices and edges. For instance, the GraphQL query in test_input_data.py is as follows:
{
        Animal {
            name @output(out_name: "animal_name")
            in_Animal_ParentOf @optional {
                name @tag(tag_name: "parent_name")
            }
            out_Animal_ParentOf {
                alias @filter(op_name: "contains", value: ["%parent_name"])
            }
        }
    }

in which the filter applies to a tag that may not exist in all result sets.
The compiler currently outputs incorrect Cypher for the query above.
CYPHER 9
MATCH (Animal___1:Animal)
OPTIONAL MATCH (Animal___1)<-[:Animal_ParentOf]-(Animal__in_Animal_ParentOf___1:Animal)
MATCH (Animal___1)-[:Animal_ParentOf]->(Animal__out_Animal_ParentOf___1:Animal)
    WHERE ((Animal__in_Animal_ParentOf___1 IS null) OR
        (Animal__in_Animal_ParentOf___1.name IN Animal__out_Animal_ParentOf___1.alias))
RETURN
    Animal___1.name AS `animal_name`

It raises a syntax error because a MATCH clause cannot follow OPTIONAL MATCH clause ("Neo.ClientError.Statement.SyntaxError: MATCH cannot follow OPTIONAL MATCH (perhaps use a WITH clause between them)" is the precise message, although I didn't find this explicitly mentioned in the documentation).

A first attempt at solving this would be to simply re-order the clauses so all the MATCH clauses are at the top. However, this fails because the compiler and Cypher treat filtering on optional fields differently. For the compiler, when filtering at or within an optional vertex or edge, if a given result set is able to produce a value for the optional vertex field, the filter may cause the result set to be discarded if it does not match. However, this is not the case in Cypher, where it simply pretends the edge doesn't exist.
Consider the following illustrative sequence of queries:
create (a:Person {name:"a"})-[r1:FRIENDS]->(b:Person {name: "b"})-[r2:FRIENDS]->(c:Person {name:"c"})

match(c:Person {name:"c"})
create (d:Person {name:"d"})-[r:FRIENDS]->(c)```

match (b:Person)-[]->(c:Person {name:"c"})
optional match (a:Person)-[r]->(b:Person)
where a is null or type(r)="NOTFRIENDS"
return a, b, c, r

The last query here gives two rows:

"a"
"b"
"c"
"r"

null
{"name":"d"}
{"name":"c"}
null

null
{"name":"b"}
{"name":"c"}
null

Cypher ignores the edge from a to b that is of type FRIENDS, which is not the intended behavior for the compiler. The correct output should be

"a"
"b"
"c"
"r"

null
{"name":"d"}
{"name":"c"}
null

Update: when implementing fold, which made significant use of OPTIONAL MATCH clauses, it seems like having a WITH clause after each optional traversal is the way to go here.
I.e. the above query can be turned into
CYPHER 9
MATCH (Animal___1:Animal)
OPTIONAL MATCH (Animal___1)<-[:Animal_ParentOf]-(Animal__in_Animal_ParentOf___1:Animal)
WITH
  Animal___1 AS Animal___1,
  Animal__in_Animal_ParentOf___1 AS Animal__in_Animal_ParentOf___1
MATCH (Animal___1)-[:Animal_ParentOf]->(Animal__out_Animal_ParentOf___1:Animal)
    WHERE ((Animal__in_Animal_ParentOf___1 IS null) OR
        (Animal__in_Animal_ParentOf___1.name IN Animal__out_Animal_ParentOf___1.alias))
RETURN
    Animal___1.name AS `animal_name`


Suffix always inserts a space

Hi,
First of all, thank you for this amazing library. It's really useful!
We are trying to get your library to accept an empty value as a suffix. As we do not want a space to be inserted after a template.
However this bit of code suffix = (suffix = this.getOpt('suffix')) ? suffix : suffix || "\u00A0"; always inserts a space even if we specifically say that we want an empty string "". This is treated as a false value in this bit of code and will always insert a nbsp.
Is there a possibility to fix this?
(Note: In the textarea controller you do check for "" with this piece of code suffix = (suffix = this.getOpt('suffix')) === "" ? suffix : suffix || " ";, I would extract a method an make that handle the suffix, so that both a text area and a content-editable handle this the same way).

Thanks for the fix! Much appreciated!

Having some trouble displaying \u200D on iOS.
My solution is to remove it from createTextNode in my forked code.
// ...
EditableController.prototype.insert = function(content, $li) {
  // ...
      range.collapse(false);

+     // fix: \u200D cannot display on iOS
-     range.insertNode(suffixNode = this.app.document.createTextNode("\u200D" + suffix));
+     range.insertNode(suffixNode = this.app.document.createTextNode(suffix));

      this._setRange('after', suffixNode, range);
}

Ref: https://github.com/ichord/At.js/issues/411#issuecomment-256662090


[tool_crash] StackOverflowError: (null)

Command
flutter run -d 00008101-001151D62242001E

Steps to Reproduce

...
...
...

Logs
StackOverflowError: (null)
#0      _StringBase.createFromCharCodes (dart:core-patch/string_patch.dart:126:3)
#1      new String.fromCharCodes (dart:core-patch/string_patch.dart:21:24)
#2      _Utf8Decoder.decodeGeneral (dart:convert/utf.dart:668:31)
#3      _Utf8Decoder.convertSingle (dart:convert-patch/convert_patch.dart:1746:14)
#4      _readFQDN (package:multicast_dns/src/packet.dart:165:24)
#5      _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#6      _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#7      _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#8      _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#9      _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#10     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#11     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#12     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#13     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#14     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#15     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#16     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#17     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#18     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#19     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#20     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#21     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#22     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#23     _readFQDN (package:multicast_dns/src/packet.dart:147:38)
#24     _readFQDN (package:multicast_dns/src/packet.dart:147:38)

[✓] Flutter (Channel stable, 3.19.5, on macOS 15.0 24A335 darwin-arm64, locale en-TH)
    • Flutter version 3.19.5 on channel stable at /Users/humansoft/fvm/versions/3.19.5
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 300451adae (6 months ago), 2024-03-27 21:54:07 -0500
    • Engine revision e76c956498
    • Dart version 3.3.3
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/humansoft/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/humansoft/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16A242d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)

[✓] VS Code (version 1.93.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.98.0

[✓] Connected device (6 available)
    • Small Iphone (mobile) • 00008030-000139023C3B802E            • ios            • iOS 18.0 22A3354
    • Noon (mobile)         • 00008110-001A021A1A28401E            • ios            • iOS 18

Closing this as a duplicate of https://github.com/flutter/flutter/issues/155433


fix: increase ImagePullProgressTimeout as 10min

https://github.com/containerd/containerd/pull/6150 defines ImagePullProgressTimeout  as 1min, while this default timeout is sometimes too small when 1) pulling image is slow 2) big image layer extraction on the local disk costing more time 3) pulling big image, this PR tries to increase ImagePullProgressTimeout as 10min

Hi @andyzhangx the commit needs your sign off. Please use 'git commit -s' to sign off. Thanks

Coincidentally, I was talking about this with @ruiwen-zhao and @qiutongs.  We haven't been able to repro it yet, but we did get a report of this error occurring.
I don't really think bumping the timeout alone is a reasonable fix.  Looking at the design of pullProgressReporter and countingReadCloser, the intent is to capture whether progress is occurring in the download and the number of bytes read from the wrapped io.ReadCloser along with a timer are used as the method.
In the report that we have, we don't see anything suspicious about the network itself causing requests to take longer to read.  However, my (unverified) working theory is that there's some delay outside the countingReadCloser such that countingReadCloser.Read isn't called within the timer that pullProgressReporter is running, and the timeout seen is because of that.  In other words: if countingReadCloser.Read isn't called, r.reqReporter.incByteRead won't be incremented, and we'll exceed reporter.timeout.
@fuweid Does this sound like a reasonable theory to you?

@samuelkarp we have at least two users, they set this value as 10min by using this daemonset, and finally it works: https://github.com/andyzhangx/demo/blob/master/aks/set-image-pull-progress-timeout.yaml#L39,  I think this PR would work to some extent, it may be more helpful if we increase other timeout values.

btw, feel free to use following daemonset to reconfigure the default ImagePullProgressTimeout:
kubectl apply -f https://raw.githubusercontent.com/andyzhangx/demo/master/aks/set-image-pull-progress-timeout.yaml

@fuweid what does this test failure mean?
https://github.com/containerd/containerd/actions/runs/6807043328/job/18509239645?pr=9347
  + /home/runner/work/_actions/containerd/project-checks/v1.1.0/script/validate/dco
  /home/runner/work/containerd/containerd/bin/git-validation
  time="2023-11-09T03:28:59Z" level=info msg="using commit range: 5149050d6bca1ff28c914e50348b3c479ff9b597..HEAD"
   * c8a84671 "Merge 843033e1c0b578d9ec4a21cb916d00e62cb14adf into c2b39b139e28a7519db614b3ef91767ef8ef0e75" ... PASS
    - PASS - commit does not have any whitespace errors
    - PASS - merge commits do not require DCO
    - PASS - merge commits do not require length check
   * 843033e1 "fix: increase ImagePullProgressTimeout as 10min" ... FAIL
    - PASS - commit does not have any whitespace errors
    - FAIL - does not have a valid DCO
    - PASS - commit subject is 72 characters or less! *yay*
   * c2b39b13 "Merge pull request #9349 from fuweid/deflaky-TestIssue9103" ... PASS
    - PASS - commit does not have any whitespace errors
    - PASS - merge commits do not require DCO
    - PASS - merge commits do not require length check
   * dac056fe "integration: deflake TestIssue9103" ... PASS
    - PASS - commit does not have any whitespace errors
    - PASS - has a valid DCO
    - PASS - commit subject is 72 characters or less! *yay*

@andyzhangx Please sign the commit.  refer to https://github.com/microsoft/vscode/wiki/Commit-Signing

@andyzhangx Please sign the commit. refer to https://github.com/microsoft/vscode/wiki/Commit-Signing

@MartinForReal that's a little complicated, I only need to append Signed-off-by: Andy Zhang<EMAIL_ADDRESS>in the commit msg, thanks anyway.

slow 2) big image layer extraction on the local disk costing more time 3) pulling big image, this PR tries to increase ImagePullProgressTimeout as 10min

The timeout will only be triggered then there is NO progress  reading from remote registry in the past 1 min, which should be irrelevant to the size of the image.
@andyzhangx did you find out why it didn't make progress reading from the registry? Was it because countingReadCloser.Read isn't called within the timer that pullProgressReporter is running, as @samuelkarp mentioned above?

I don't really think bumping the timeout alone is a reasonable fix. Looking at the design of pullProgressReporter and countingReadCloser, the intent is to capture whether progress is occurring in the download and the number of bytes read from the wrapped io.ReadCloser along with a timer are used as the method.

I was curious if the purpose of the field was properly documented (as that would be a good improvement if it wasn't), but it looks like it is; https://github.com/containerd/containerd/blob/45d7f2324d4ae4fd05e87c59a95181875dcb621b/pkg/cri/config/config.go#L317-L320
Also looking at the commit that introduced this timeout https://github.com/containerd/containerd/commit/00d102da9f73110bf9e79f9139d10ee2f249effb (https://github.com/containerd/containerd/pull/6150);
Kubelet sends the PullImage request without timeout, because the image size
is unknown and timeout is hard to defined. The pulling request might run
into 0B/s speed, if containerd can't receive any packet in that connection.
For this case, the containerd should cancel the PullImage request.

That confirms that this option is intended to provide a time-out if no bytes were received within the given time-limit (1 minute).

we have at least two users, they set this value as 10min by using this daemonset, and finally it works

It's worth checking with them, but from the discussion here, I suspect it worked if the whole image was pulled before the 10 minutes timeout was reached. If that's the case, then it would be tapering over the actual problem.
There is no hard limit on image size and, while rare (at least for Linux, Windows images tend to be larger), multi-gigabyte images do exist, and so do "slow networking connections". For those reasons, we cannot make good assumptions how long an image pull can take (even a small image can take long to pull on a poor network connection); pulls that take a long time should be supported, although we can consider having a (user-configurable) hard-limit for pulls. (In addition, pulls may be constrained by other factors, such as auth / tokens expiring, but that's likely out of our hands).
The existing "1-minute" timeout is already rather generous (we can expect at least some bytes to arrive within a minute, even on a slow network connection), so the intent here looks to be to protect against exceptional situations. If we're seeing timeouts here, it sounds very plausible that there's a bug at hand, and that should be fixed.
All that said; even with that bug fixed, I think it would be a good thing to

document where the 1 minute timeout came from; it looks to be an arbitrary value ("1 minute should be look enough"), which is fine, but it helps to document this. Given that both the Windows and Unix variants use the same value, perhaps define a const for this, and add documentation to that const (it it's only used internally, wer don't have to export the const yet).
check what error is returned on timeout; I haven't looked closely at the implementation, but if we're cancelling the context, we could use the new context.WithCancelCause() to provide more information (i.e., "no bytes received within <configured timeout>")

The existing "1-minute" timeout is already rather generous (we can expect at least some bytes to arrive within a minute, even on a slow network connection), so the intent here looks to be to protect against exceptional situations. If we're seeing timeouts here, it sounds very plausible that there's a bug at hand, and that should be fixed.

Yep, that's exactly why I'm suspecting more of a bug with the accounting logic than the connection truly drops to 0 B/s for a minute.

@samuelkarp @thaJeztah the 1-min is from kubelet setting for dockershim. Sorry for late reply. I created flakey case for tracking something like that https://github.com/containerd/containerd/issues/8024. But I am still rethinking which part went wrong. The ticker might be fired when the active request just comes, like #8024. And then the tracker saw one active request but no bytes increased. So it fired the timeout. It's possible and I did see it in the CI because the CI timeout is short than 1-min. Might I should refresh the lastSeenTimestamp when we have new active request. But I am still worry about the IO pressure created umount action or unpacking huge layer which triggered by WriteBack thread (ext4 has default commit=5).
Let me rethink it and update it later. Thanks

the 1-min is from kubelet setting for dockershim.

Thanks for that added info. Yes, it would be good to document that. "magic" values are always tricky if they're not "trivial"; it's easy to forget where they came from (and can easily lead to wrong conclusions)

Sorry for late reply.

Ohman, no worries; no need to apologize! Thanks!


Submap saving without RViz visualization

Hi,
i have been trying to use the Multi-TSDF approach to map multiple scenes from the ScanNet dataset. I have encountered a problem, while having visualization turned off. The maps, when saved, have heavily reduced size (from ~60mb -> ~7mb) compared to ones saved while the visualization was running. Additionally they seem to contain substantially less submaps (as far as i have loaded and looked at them). The same problem seems to appear, when I am mapping and saving the flat_dataset example. Is there any way to change this behavior?
On a side note, is it good practice to call the finish_mapping service before saving? I am asking, because the callback has a comment, that it is a preliminary tool.
Thanks for any help in advance.

Hi @LeonardRabes
Thank you for pointing out this issue! The reason for this is that the submap meshes are required to track the input data, but these were only updated when visualization was turned on. This must've been 'optimized away' along the way, but should be fixed in #66. Can you give it a try and potentially compare the maps obtained with visualization turned on vs off?
Regarding your second question, the finish_mapping service simply terminates all tracks, i.e. all currently tracked submaps are moved to the inactive state, which is typically what would happen between mapping sessions. Thus, calling the service should give you intended behavior if the loaded map and the newly processed data are meant to be two sessions, and should be avoided if the data is considered one continuous stream.
Hope this helps!

Hi @Schmluk,
the fix seems to work very well. The exported map for the flat_dataset/run1 now has a size of 45.5mb with 47 exported submaps (without calling finish_mapping) with and without visualization. If I encounter any problems with it, i will tell you about it.
Thank you a lot for your help.


[RS2] Imu calibration

This PR provides the RS2 ROS wrapper with an IMU calibration service. A custom service file is created for this purpose CalibrateImuReq.srv in order to trigger the calibration routine and to return the calibrated angle. The service is called in order to update the transforms, both static and dynamic.

This PR provides the RS2 ROS wrapper with an IMU calibration service. A custom service file is created for this purpose CalibrateImuReq.srv in order to trigger the calibration routine and to return the calibrated angle. The service is called in order to update the transforms, both static and dynamic.

this description is outdated? I don't see any CalibrateImuReq.srv


Proposal: Specify how plans can be formally deprecated

The spec currently says this:

Service Broker authors are expected to be cautious when removing services and plans from their catalogs, as Platforms might have provisioned Service Instances of these plans. For example, Platforms might restrict the actions that users can perform on existing Service Instances if the associated service or plan is deleted. Consider your deprecation strategy.

I think, just as bindable can indicate whether an application may bind to instances of a given service/plan, we should, perhaps, add an optional field to service definitions (and overridable in that service's plan definitions, much as with binadable) that could allow a broker to indicate that it can deprovision those services/plans, but will not provision a new one for you.

Sorry @n3wscott. I overlooked that. Looks like you're a few steps ahead of me.


Installing with Grunt

I'm having some issues getting this to work with Grunt, can you please take a look:
This is the less section of my Gruntfile.js:
            less: {
                development: {
                    options: {
                        paths: ['<%= directory %>assets_new/css'],
                        plugins: [require('less-plugin-glob')]
                    },
                    files: {
                        '<%= directory %>assets_new/css/style.css': '<%= directory %>assets_new/css/src/__bootstrap.less'
                    }
                }
            },

My less file contains:
@import "breakpoint_desktop/**";
When I build, I'm getting the following error:

FileError: 'breakpoint_desktop/.less' wasn't found in...
@import "breakpoint_desktop/";

Thanks.

Figured out my issue, I had a super old version of Grunt installed. Totally forget to keep it updated. Thanks for the great plugin.


Whitespace characters in source control commit message area

VSCode Version: 1.38.1
OS Version: macOS 10.13.6

Steps to Reproduce:

Click on source control which is in activity bar.
In the commit message box whitespace will be there.

Does this issue occur when all extensions are disabled?: Yes
I saw the same behavior in my colleagues macOS laptop using vscode version greater than 1.38.0, this doesn't happen in 1.37.x

/duplicate of #80640


Filter methods for trashed items

It's good to have the following methods for softDelete enabled tables.

$repository->getByFilterWithTrash()
$repository->allByFilterWithTrash()

@zaman-enzan, Seems laravel-rocket itself not supporting anymore for the latest version of Laravel. The project already developed using lower than laravel 6 and used laravel-rocket has no support as well. The problem is, if a large scale application used laravel-rocket and now want to upgrade latest version of laravel it will be a big task. Either it's require to remove laravel-rocket codebase completely or require to write all the code what laravel-rocket provided.


BigDecimal divideToIntegralValue tests fail on JVM.

It seems to be an issue with that scale of the number.

Test filtered in #2043.


Raw retrieval

[1/3] retrieve image-text pairs using interactive raw retrieval
This cl add a boolean raw_retrieval config that when enabled, the retriever stores the raw query and retrieved candidates in a jsonl format which will be later on useful for RAG applications of UniIR. Without this change the retriever only outputs runfiles in trec eval format.
Adding an interactive retriever to be able retrieve complement results for candidates, and complement retrieving will be followed in next cls.

closing per our discussion with @lim142857 , all three prs will be submitted as one


Dependency issue while implementing this tutorial

Hello Team,
I've been trying to implement this tutorial I followed all the steps but I had an issue in the Create Your Front end > Add passwordless authentication while executing this command :
npm install amazon-cognito-passwordless-auth
Error :
npm error code ERESOLVE
npm error ERESOLVE could not resolve
npm error
npm error While resolving<EMAIL_ADDRESS>npm error Found<EMAIL_ADDRESS>npm error node_modules/react
npm error   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error   node_modules/react-dom
npm error<EMAIL_ADDRESS>from the root project
npm error<EMAIL_ADDRESS>from the root project
npm error
npm error Could not resolve dependency:
npm error amazon-cognito-passwordless-auth@"" from the root project
npm error
npm error Conflicting peer dependency<EMAIL_ADDRESS>npm error node_modules/react
npm error   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error   node_modules/react-native
npm error     peer react-native@"" from<EMAIL_ADDRESS>npm error     node_modules/react-native-passkey
npm error       peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm error       node_modules/amazon-cognito-passwordless-auth
npm error         amazon-cognito-passwordless-auth@"*" from the root project
npm error
npm error Fix the upstream dependency conflict, or retry
npm error this command with --force or --legacy-peer-deps
npm error to accept an incorrect (and potentially broken) dependency resolution.
npm error
npm error
npm error For a full report see:
npm error /home/ec2-user/.npm/_logs/2024-05-10T13_41_04_015Z-eresolve-report.txt
npm error A complete log of this run can be found in: /home/ec2-user/.npm/_logs/2024-05-10T13_41_04_015Z-debug-0.log
This seems like a dependency issue between react and the amazon-cognito-passwordless-auth  package.
Please review and fix this as it public tutorial of AWS.
Tutorial Link : https://catalog.workshops.aws/cognito-webauthn-passwordless/en-US/30-frontend/32-add-passwordless

Thanks for reporting. Will have to look into it.
To work around for now, use the --force flag for the npm install.

Hello,
Thanks for your answer.
the --force flag will not generate errors but also will NOT remove the error for amazon-cognito-passwordless-auth package in the react code.

Hi! I resolved this particular issue installing this dependencies at once:
npm install<EMAIL_ADDRESS><EMAIL_ADDRESS>amazon-cognito-passwordless-auth

Should be fixed in version #182


View Controller injection MVP

Hello there, I am using the pattern MVP in my app, but I have some trouble when I want to set the view to my presenter.
We can't resolve explicitly the view controllers registered by registerForStoryboard method, so I wonder how to get the view controller when I am registering the presenter, I was thinking about set the view to the presenter in registerForStoryboard method, but I don't know if it is correct. Any help please?
let mainContainer: Container = Container() { container in

        container.register(Presenter.self){ r in
            return Presenter()
        }.initCompleted { r,c in
            let view = /* HOW GET THE VIEW */
            c.view = view
        }

        container.registerForStoryboard(Test1ViewController.self) { r, c in
            let presenter = r.resolve(Presenter.self)
            c.presenter = presenter
        }
}

In AppDelegate:
let sb = SwinjectStoryboard.create(name: "Main", bundle: nil, container: self.mainContainer)
let controller = sb.instantiateViewControllerWithIdentifier("Test1View") as! Test1ViewController

Hi @davidosorio. @Nikita2k's templates might be helpful to do what you want to do:
https://github.com/Nikita2k/SwiftViper

In fact, I don't have storyboards (I believe using SB is a bad practice).
But I can suggest to remove initCompleted block and pass your view to presenter initialization

Hi @Nikita2k, thank you for your help!
(I had misunderstanding about the question. @davidosorio uses Presenter but it was not in VIPER, but in MVP, which is more like MVVM.)

Hi, thank you @Nikita2k, @yoichitgy for your answers, for me, use or no SB depends on the size of the project, I agree that in a large project Storyboards are a bad practise, actually I use one storyboard per module/component.

@davidosorio, another approach is here (just for your information):
https://github.com/rizumita/TransitionOperator
I prefer using storyboards except the problem of DI, and trying to keep them small.
I understand advantages of UI implementation without storyboards. So, storyboards can be chosen depending on the size or type of your project, or your preference. (I keep myself neutral in Swinject project.)
Thanks @Nikita2k again for your help and sharing your great practice!


Datepicker opens even with readonly attribute in place

Summary:
My datepicker has the attribute readonly, but it doesn't have effect to make it "readonly"
Expected Behaviour:
With readonly in place I expect that the modal doesn't open up.
Current Behavior:
I'm preloading a value, even with a datepicker with "readonly" in place the modal of datepicker opens up, and furthermore when a date gets selected the value changes.
Is there a workaround to make a datepicker truly readonly?
Here a fiddle
https://jsfiddle.net/alcidesapaza/ka48jtLw/6/

Hey!
Just add disabled attribute to your input and you should be fine.
Here is your fiddle forked and with attribute added.
https://jsfiddle.net/tp21ok6r/

@enjoyurrlty  that's a great workaround, thanks!


Number Stored as Text

Using the following code:
require 'axlsx'

Axlsx::Package.new do |p|
  p.workbook.add_worksheet(name: 'Numbers as text') do |sheet|
    sheet.add_row %w(Number As\ text)
    sheet.add_row([42, 42], types: [nil, :string])
  end
  p.serialize('numbers_as_text.xlsx')
end

At least Excel for Mac 15.33 shows "Number Stored as Text" warning for the second column. For my purpose, the numbers should be stored as text, but I would like to get rid of the warning. Is it possible?
axlsx-2.0.1

I am also experiencing this problem.


Checkbox group

Hi!
I've 3 checkboxes with the same name ( destiny[] ) and I need to verify that at least one is selected.
Any idea?
Thx!!!

Currently this isn't supported, but you can follow the progress in #1

I have faced this issue before and made an effort which is followed and it works.
`validate.validators.checkboxList = (value, options, key, attributes) => {
if (value.length === 0) return options.message;
var validation = true;

$.each(value, function (index, item) {
    if (item !== "") {
        validation = options.within.includes(item);
        if (validation === false)
            return;
    }
});

if (validation)
    return null;
else
    return options.message;

}`
Note #1: You have to prepare your value in an array. i.e. If you have three different checkboxes which are holding 1,2 and 3 as value, your final value which should be passed to validation component should be something like this: Array [1,3].
Note #2: My checkboxList component accepts an option in following schema:
"validationKey": {       checkboxList: {                 within: ["1", "2", "3"],                 message: "Your invalid warning message"             } }
And FYI, I have written a piece of other codes to prepare my checkboxList component's food 😅.
`
function prepareCheckboxListValue(selector) {
var chkbxs = $(selector);
var value = [];
$.each(chkbxs, function (index, item) {
    if ($(item).prop("checked"))
        value.push($(item).val());
})

return value;

}
`


Conduct data driven scenarios in Prism

Implement it

This will be broken up into smaller issues after we discuss everything.

needed-by #64
needed-by #66
needed-by #65
needed-by #63


Update Readme file with the logo

📚 Description
I think the readme file improves a lot with a picture.
That's why I have done something similar to what Haxe did, but I am not 100% happy with the final result. What do you think about the icon disposition? Maybe a smaller picture could fit better, doesn't it? 😃
Take a look at Janet too (just for mentioning other functional languages).
Also, do we have any other badges which is not included? 🤔

Maybe a transparent image with purple lines for the elephant.

You could put the width and height attributes to the image to make it small (maybe 200px).
As alternative you can find the original image in the root folder (logo.svg). I designed it with Inkscape.

Done.

It think that it would probably look much cooler with thicker purple lines

It think that it would probably look much cooler with thicker purple lines

Probably, could you do it @jenshaase? :)

I added an SVG with purple lines: See /doc/static/logo.svg. @JesusValera Can you add it to the Readme?

Looks very good now. Thanks. It is merged!


[feature] Touch controller: power management

The touch panel CST816S supports three power states:

Mode
Power consumption

Dynamic
< 2.5µA

Standby
< 10uA

Sleep
< 5uA

Exiting sleep requires a reset, i.e it corresponds to the DEVICE_PM_SUSPEND_STATE according to the Zephyr PM API.
Implement Zephyr's power management API like it was done for the ST7789V display.

I've copied over relevant code from the ST7789v driver and pushed the branch (wip).

We have the value for sleep (0x03), and dynamic mode can be enabled through a reset. I haven't found the value for standby mode, however, although it's maybe the most useful mode. Could it be the case that the device enters standby by itself?

I've worked on this a bit more, and I can now successfully crash the system by running the reset sequence as (apparently) required for the sleep command :)
Maybe the problem is CONFIG_CST816S_TRIGGER_GLOBAL_THREAD... On the other hand, without this config set, I've not been able to make touch interrupts crash-free at all.
The out-of-tree driver is a mess. It needs a rewrite, but without any copying from non-Apache 2.0 licensed code which would ruin any future upstreaming effort. I had thought this power management issue wouldn't be too hard :)

There seems to be more doc about the CST816 registers:
CST816S (1).zip

Thanks @sdorre, but please don't upload documents of unknown origin to this repo.

Ah okay sorry.
I will link the post from the pinetime forum instead:
https://forum.pine64.org/showthread.php?tid=8836&pid=57390#pid57390
Here seems to be the reference driver:
https://github.com/lupyuen/hynitron_i2c_cst0xxse
I don't see the reset sequence before. I understand that we need the reset to wake up the Chip and send something to it. I don#t see why we would need to reset it otherwise.
Sleep mode will gives us only 5uA less. That could be an optimization but, there are now other device that a taking much more current i think. =)

@Lukas-Berglund wrote a thesis about power consumption of the PineTime devkit. The display is a major current thief if sleep mode isn’t used but the touch controller not much it seems. Gestures are more important. For completeness I’d like to finish this at some point though.


improve design of media-lib

it is ok but not pixel perfect. I dislike the button design...

Not relevant anymore since v2 is out


Hardcoded path to non-existent libz library in ITKZLIB.cmake

We found this on OSX with current conda. The cmake file refers to
/Users/travis/miniconda3/conda-bld/libitk_1546796752855/_build_env/lib/libz.dylib
and therefore all other ITK libraries depend on this file.
Here's an example of a Travis log for our build [https://api.travis-ci.org/v3/job/480922636/log.txt], which seems to say it's using conda-4.5.12 and 4.13.0-hab14342_1001. Not that the log file is somewhat misleading as there /Users/travis/miniconda3/ does exist (while when building on local machine it doesn't of course).
More details are on https://github.com/CCPPETMR/SIRF/pull/295

Hi again. The installation works, but libraries are installed with rpaths. This means that when linking against it, I get errors along the lines of:
Library not loaded: @rpath/libITKDICOMParser-4.13.1.dylib
Reason: image not found

I think there are 3 solutions for this:

add conda/install/location/lib path to DYLD_LIBRARY_PATH
run install_name_tool on all libraries
install ITK with absolute paths

I've tried the first, and it gives the error:
dyld: Symbol not found: __cg_jpeg_resync_to_restart
  Referenced from: /System/Library/Frameworks/ImageIO.framework/Versions/A/ImageIO
  Expected in: /Users/travis/miniconda/lib/libJPEG.dylib
 in /System/Library/Frameworks/ImageIO.framework/Versions/A/ImageIO

I'm not prepared to do the second, as it would involve looping over all libraries and looping over all the libraries that they depend on and it would make the Travis script rather unsightly.
So my question is - is it possible to do the third option? Or if not, is there some other solution that I'm not aware of?

@hmaarrfk
@mariusvniekerk
Any insights as to why the 4.13.0 package container corrected full paths in the installed CMake files, but the more recently built packages contain full paths? The CMake files don't appear to have their paths fixedup any more. Is there a way to specify that these files should be fixed up?

Honestly, there were some issues building things a few months back. Maybe try release a new build number and seeing if that helps.

In the problematic build, the build env libz.dylib was being picked up and the new packages correctly picks up the host env libz.dylib. This can be closed


Handle upload/download raw files

Currently our SDK can only work with JSON payloads. We should also add support for file up/download. This means for uploads our client SDK needs to send an multipart/form-data request to the endpoint. For downloads we simply need to return the raw body content without trying to convert the body into a specific object. For file up/download we are not able to describe the body payload so we need some kind of dummy schema which tells the user that this is a raw file.

With the latest content type addition this is now possible.


[Identity] Handle Docker Desktop 403 IMDS response

Closes: https://github.com/Azure/azure-sdk-for-python/issues/31759

API change check
API changes are not detected in this pull request.


fixes shebang execution on Node 5.8.x

working from:
https://github.com/tapjs/spawn-wrap/pull/12
fixes shebang handling on Node 5.8.x
looks to be almost identical to:
https://github.com/tapjs/spawn-wrap/pull/13/files
so I'll let @isaacs decide what we land.

I think some parts of this are orthogonal to #13: You only check for the “strange” shebang installed by Node.js executables (which #13 doesn’t), but there’s no reason not to catch something like #!/usr/local/bin/node
Anyway, both skipping any tests on Windows and using which + head sound like good ideas.

@isaacs @addaleax having slept on this, I think @addaleax's original approach of only applying this logic to the npm  bin is smart ... this is definitely an edge-case, but it impacts a large number of users since nyc is depended on by 1000ish projects at this point.
tldr; I think this is a great hack to have, but we should limit applying it to as small a scope as possible. I like that with this approach:

we only need to read from a file once we determine it is the npm bin.
we only apply the hack to versions of Node >1.x

Hm, yeah… as @isaacs pointed out in https://github.com/tapjs/spawn-wrap/pull/12#issuecomment-204879395, the question of the scope of this fix is not easy. I’d tentatively agree with the behaviour that #13 seeks to achieve, namely that any script with a shebang invoking the node binary using a full path should also be wrapped when using spawn-wrap.

Also, just btw, if you run make install from a Node.js core source tree you end up with a full-path shebang like #!/usr/local/bin/node. That one would be caught by #13, but not by this PR (as it is), if I understand correctly?

Um, I just saw these files from the npm-cli repo: bin/npm bin/npm.cmd. I don’t really know what’s up with them, but I imagine there are cases where these are used when npm is invoked from the command line?
If so, I’m tempted to change my opinion again – possibly even back to just checking for “npm” on the command line, no shebang magic stuff – because both of them carry out the `dirname "$0"`/node-approach in different, even less “detectable” ways.

I think I'd like to land #12 and #13. (That is, handle absolute shebang but also special case for "npm".)
Reading and parsing the bash exec junk seems like a step too far.
I also plan to make the case for removing this hack from the node installer. It's overreaching. If someone out there really wants /some/weird/path/npm to be executed by /some/weird/path/node, then can invoke it explicitly themselves. Otherwise, the npm on the PATH should rely on the node on the PATH using /usr/bin/env node, as the unix gods intended.

@addaleax those files are only installed on Windows I believe.

@isaacs Sounds good to me. Do you want me to clean up #12 to use which, or would you prefer to do that yourself?


[FALSE-POSITIVE]

Domains or links
Please list any domains and links listed here which you believe are a false positive.
https://oriserviss.com
More Information
How did you discover your web site or domain was listed here?

virustotal

Have you requested removal from other sources?
no
Additional context
na
:exclamation:
We understand being listed on a Phishing Database like this can be frustrating and embarrassing for many web site owners. The first step is to remain calm. The second step is to rest assured one of our maintainers will address your issue as soon as possible. Please make sure you have provided as much information as possible to help speed up the process.
Send a Pull Request for faster removal
Users who understand github and creating Pull Requests can assist us with faster removals by sending a PR to mitchellkrogza/phishing repository, on the falsepositive.list file

https://github.com/mitchellkrogza/phishing/blob/main/falsepositive.list
Please include the same above information to help speed up the whitelisting process.

Site has been cleaned. Requesting for blocklist removal.


id: null

Hello,
When I make a payment with the test API, mollie fails to get the user id and therefore does not generate the subscription.
Here is the return of mollie's dashboard:
{
  "owner": {
    "type": "App\User",
    "id": null
  },
  "actions": [
    {
      "handler": "Laravel\Cashier\FirstPayment\Actions\StartSubscription",
      "description": "Monthly payment",
      "subtotal": {
        "currency": "EUR",
        "value": "5.00"
      },
      "taxPercentage": "0.0000",
      "plan": "monthly",
      "name": "premium",
      "quantity": 1
    }
  ]
}

Here is the user model
<?php

namespace App;

use IlluminateFoundationAuthUser as Authenticatable;
use IlluminateNotificationsNotifiable;
use IlluminateSupportFacadesCache;
use OvertrueLaravelFollowFollowable;
use OvertrueLaravelLikeTraitsLiker;
use LaravelCashierBillable;
use LaravelCashierOrderContractsProvidesInvoiceInformation;

class User extends Authenticatable
{
    use Notifiable, Followable, Liker, Billable;

    protected $primaryKey = 'steam_id';
    protected $dates = ['community_banned_until'];

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'email', 'avatar', 'steam_url', 'steam_id',
    ];

    /**
     * The attributes that should be cast to native types.
     *
     * @var array
     */
    protected $casts = [
        'email_verified_at' => 'datetime',
    ];

    public function reports()
    {
        return $this->hasMany('AppReport', 'user_steam_id');
    }

    public function receivedReports()
    {
      return $this->hasMany('AppReport', 'suspect_id');
    }

    public function isOnline()
    {
      return Cache::has('user-is-online-' . $this->steam_id);
    }

    public function getVacatchLocale()
    {
        return User::find($this->steam_id)->locale;
    }

    public function role()
    {
        return $this->belongsTo('AppRole');
    }

    public function stats()
    {
        return $this->hasMany('AppStat', 'user_steam_id');
    }

    public function goodReports()
    {
        return $this->hasMany('AppGoodReport', 'user_id');
    }

    public function posts()
    {
        return $this->hasMany('AppPost', 'user_id');
    }

    public function getInvoiceInformation()
    {
        return [$this->name, $this->email];
    }

    public function orders()
    {
        return $this->hasMany('AppShopOrder', 'user_id');
    }
}

Sorry for my approximate English, I am French.
Have a good day !

Bonjour!
That's about all of my French vocabulary, so I'll stick to English here ;)
The problem is caused by the custom primary key (steam_id) on your billable model. I can look into adding support but it may take some time.
Meanwhile, can you confirm this is indeed the cause by adding this to your billable model as a workaround:
public function getIdAttribute()
{
    return $this->steam_id;
}

We've rolled out a patch in v1.16.1 (release notes). Let me know if this does not solve your problem.
Enjoy!


Remove duplicate mocking

Prior to this PR, we were mocking Date.now once in the spec helper, then blowing up with a redundant mock error when jasmine.useMockClock is called.
Since this was newly introduced, let's only set it up when clock mocking is specifically requested to minimize breakage. We really need to deprecate the global spec helper someday. It's bad news.
This needs to be hot-fixed to 1.19.
/cc @hansonw

Yay! Thanks for the quick fix.

@iolsen went ahead and added this to the hotfix tracking column since I think it slipped through the cracks.


Add two Clojure CLI commands to debug CI

I'd like to see the output of both of these commands prior to the tools install failing, to verify what version/tools the CLI thinks are present.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 50.195%

Files with Coverage Reduction
New Missed Lines
%

src/nvd/task/check.clj
2
29.76%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
129

Relevant Lines:
257

💛  - Coveralls

(this is just for debugging, so it doesn't need merging... I'm reaching out to Alex to get help debugging what we're seeing here)

It's a bug in the setup-clojure Action: it does not copy tools.edn across -- https://github.com/DeLaGuardo/setup-clojure/issues/34


Metamodel: ErrorCodes should not contain range start/end values

Doing so makes it no longer an enum, e.g. jsonrpcReservedErrorRangeStart and serverErrorStart have the same value.

Same goes for LSPErrorCodes.

See dbaeumer/cruel-tiglon-jade

I think LSPErrorCodes still needs to be changed?

You are correct.


Follow up Lambda timeouts in API Gateway

Discovered while developing: Lambda timeouts get converted to 500 Internal Errors in API Gateway

Interestingly, I get the same behavior in the default Nodejs runtime with API Gateways HTTP API, so this doesn't seem to be a bug in the deno runtime for lambda, but an issue with configuration in API Gateway


Can opencl install on Microsoft windows 8

I try on cmd with "cabal install opencl".
But it failure. Input from command line show below.
https://docs.google.com/file/d/0B02yMjOZsqqFLWtRMzVtZU90dDQ/edit?usp=sharing
Sorry for my bad english.

Just to add to this for others finding it. My previous comment was for the Intel OpenCL SDK. For the latest (as of now, 3.0-beta) AMD OpenCL SDK (called AMD APP SDK) the paths on a 64bit machine are:
cabal install opencl --extra-include-dirs="c:\Program Files (x86)\AMD APP SDK\3.0-0-Beta\bin\x86_64" --extra-lib-dir
s="c:\Program Files (x86)\AMD APP SDK\3.0-0-Beta\lib\x86_64"

Just to add to this for others finding it. For the Intel Integrated Native Developer Experience (called Intel INDE) the paths on a 64bit machine are:
cabal install opencl --extra-include-dirs="C:\Intel\INDE\code_builder_<IP_ADDRESS>\include\CL" --extra-lib-dirs="C:\Intel\INDE\code_builder_<IP_ADDRESS>\lib\x64"


Updated client CA cert

Hi, on some systems I was getting error
Code:0 Message:cURL error 60: Peer's certificate issuer has been marked as not trusted by the user. (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for https://fioapi.fio.cz/v1/rest/

and I figured out, that shipped certificate have to be updated

@feldsam Hi, thanks for the PR! I'm not sure that including the certificate in the repo still makes sense. Maybe it would be better if it depended on composer/ca-bundle directly? What do you think?
//cc @soukicz ?

There is actually dep on \Kdyby\CurlCaBundle\CertificateHelper, but there is also fallback, if this is not installed. So I just updated fallback cert.

I'm not sure that including the certificate in the repo still makes sense

It is nice that it can run without dependencies. And it is just fallback because it is already using composer/ca-bundle if it is available.

So I updated my commit to remove fallback and cert from repo and refactored code to use ca-bundle only

@feldsam thanks! merged and tagged as 5.0.0


⚠ GitLab Server has degraded performance

In d4d783e, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11243 ms

Resolved: GitLab Server performance has improved in f6761c4.


PLEASE READ THIS ABOUT UTM CRASH.

So first off all i have 11 pro max on 13.4.5b1 and from 13.3.1 to now (today) i cannot get to work utm no matter what i do and which verson i download ( download utm,create vm win xp/7 and press start automatic crash) BUT TODAY I INSTALL ALTSTORE V1.3 AND SIDELOAD UTM AND WORK WITHOUT PROBLEM SO EVERYONE WHO HAVE THE SAME PROBLEM JUSR DOWNLOAD AND INSTALL ALTSTORE (new version) SIDELOAD UTM AND THATS IT! Btw for altstore you dont need to be patriot... for sideload (free account as i have every 7days resign and thats it (or developer every 1 year) new version of altstore v1.3)

Thanks for the confirmation but this has been the recommended instruction for weeks.

Oh sorry than,but myb somone still have the problem like on #95,just try to help :)

Can I use Altstore to sign UTM with an enterprise certificate (.p12)?

No. https://getutm.app/install/


chore: bump krm-kcl version to latest.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

chore: bump krm-kcl version to latest.

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

@ekkoful Can you please review it?


Add Tables extension

Issue describing the changes in this PR
resolves #404
Pull request checklist

[x] My changes do not require documentation changes

[ ] Otherwise: Documentation issue linked to PR

[ ] My changes should not be added to the release notes for the next release

[x] Otherwise: I've added my notes to release_notes.md

[ ] My changes do not need to be backported to a previous version

[ ] Otherwise: Backport tracked by issue/PR #issue_or_pr

[ ] I have added all required tests (Unit tests, E2E tests)

Additional information
Additional PR information

/check-enforcer evaluate

/check-enforcer override

/check-enforcer evaluate


Use netlink to set hairpin mode

This should fix problem with hairpin on EL6. I have no EL6 to  try though.
/cc @phemmer

I just spun up a VM last time when I was digging into the issue. I can again, but someone with real world usage might be better.

Actually I think this can be tested with bridge command from iproute2. Same netlink messages will be sent.

@mrunalp I've added constant.

@LK4D4 thanks, I will test this out today.

LGTM

LGTM


Config changes to reflect the default branch from master to main. Please merge to complete the default branch renaming job.

Auto branch renaming content.

Docs Build status updates of commit 922ed66:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


feat(site): init website

Missing main.ts.
importing ts files doesn't work ...?

Using https://github.com/Boshen/oxc/pull/281 so we can deploy from CI


Python auto-formatters

It might be good to also add support for YAPF or similar. This can be added in a .style.yapf file in the base folder. I typically use:
[style]
COLUMN_LIMIT = 119
INDENT_WIDTH = 4
USE_TABS = False

However, this might convolve with the usage of pyflakes or not.

I don't think that will be a problem with pyflakes since it looks for a [flake8] header in either a .flake8, setup.cfg, or tox.ini file, as I understand. I could include either a separate tox.ini file with the flake8 headers, or, we may even be able to do a single setup.cfg if we wanted to manually point all the style/test things towards it which look for .ini style config files

I do want to avoid the throwing the kitchen sink of config files approach though. I think this may be one area we can make a few executive choices with the cookiecutter here and use specific coverage/test/pep8 and possibly point to other options in either the output directory (maybedevtools/README.md) or the cookiecutter itself's README.md

Agreed that the kitchen sink is bad. More that if we suggest very specific things (like code formatting) we might want to supply the standard.
If we go with a setup.cfg (which might be better) we can use the [yapf] heading to supply the settings.

+1 for  putting it all in setup.cfg


[Feature]: ESM: put jest in module scope

🚀 Feature Proposal
When using ESM, jest is no longer available in the module scope.
Add jest back into module scope.
Motivation
This makes converting a repo to ESM a little bit more work, and it breaks expectations since test and expect are still available.
Example
instead of having to
import {jest} from '@jest/globals'

jest.setTimeout(10000)

or
import.meta.jest.setTimeout(10000)

Just re-allow
jest.setTimeout(10000)

Pitch
jest is core.
To implement this, I propose using Async LocalStorage (ALS), and adding a jest object that uses it together with it and friends.
The ALS store (als = new AsyncLocalStorage) will inject all the module data that jest needs via als.run(moduleData, ()=>run_test_module_to_discover_tests()) and then e.g. jest.setTimeout would get it via moduleData = als.getStore().
(incidentally, ALS can also be used to allow async callbacks in describe)

Sorry I never responded here!
I'd be interested in this if you could put together a proof of concept 🙂


[SPARK-46841][SQL] Add collation support for ICU locales and collation specifiers

What changes were proposed in this pull request?

Languages and localization for collations are supported by ICU library. Collation naming format is as follows:
<2-letter language code>[_<4-letter script>][_<3-letter country code>][_specifier_specifier...]

Locale specifier consists of the first part of collation name (language + script + country). Locale specifiers need to be stable across ICU versions; to keep existing ids and names invariant we introduce golden file will locale table which should case CI failure on any silent changes.
Currently supported optional specifiers:

CS/CI - case sensitivity, default is case-sensitive; supported by configuring ICU collation levels
AS/AI - accent sensitivity, default is accent-sensitive; supported by configuring ICU collation levels
<unspecified>/LCASE/UCASE - case conversion performed prior to comparisons, default is unspecified; supported by internal implementation relying on ICU locale-aware conversions

User can use collation specifiers in any order except of locale which is mandatory and must go first. There is a one-to-one mapping between collation ids and collation names defined in CollationFactory.
Why are the changes needed?

To add languages and localization support for collations.
Does this PR introduce any user-facing change?

Yes, it adds new predefined collations.
How was this patch tested?

Added checks to CollationFactorySuite and ICU locale map golden file.
Was this patch authored or co-authored using generative AI tooling?

No.

Please review collation team @dbatomic @stefankandic @uros-db @mihailom-db @stevomitric.

User can use collation specifiers in any order except of locale which is mandatory and must go first. There is a one-to-one mapping between collation ids and collation names defined in CollationFactory.

Does that mean that there are multiple collation names that have identical meaning but different names (with the specifiers in different order)? That seems confusing and error prone. Are we at least normalizing them internally somehow?

will we have to do the same for pyspark - as StringType there only supports 4 initial collations?

User can use collation specifiers in any order except of locale which is mandatory and must go first. There is a one-to-one mapping between collation ids and collation names defined in CollationFactory.

Does that mean that there are multiple collation names that have identical meaning but different names (with the specifiers in different order)? That seems confusing and error prone. Are we at least normalizing them internally somehow?

+1 on @bart-samwel 's concern. Is there a good reason to allow the order to be relaxed?

How do we name a trailing-space-insensitive collation?

@mkaravel @dbatomic please review again, thanks.

do we have end-to-end tests for this new feature?

thanks, merging to master!


Unable to edit URL on a Generic driver with Template

Description
I have setup a generic driver to a custom JDBC source. I put in a template for the URL.
When I setup a Connection, the JDBC URL is not editable.
In previous version / use I was able to edit the TEMPLATE URL to add new connections.  I got a new PC and just installed DBeaver and now unable to edit.

dbeaver-diagnostic-info-1720354891825.zip

DBeaver Version
24.1.1
Operating System
Windows 10
Database and driver
Infor Compass 2023.10
Steps to reproduce

setup a new Driver using Generic, enter a URL Template name, add file to JDBC driver. add default connection file detail
setup a new database connection using the new driver  .  test connection to default , success.
setup or copy paste a new database connection.  attempt to edit the URL to add a different connection.   Field is uneditable.
https://docs.infor.com/inforos/2024.x/en-us/useradminlib_cloud/default.html?helpcontent=datafabrug/dan1631199548336.html

Additional context
No response

From the first screenshot, I see that both radio buttons Host and URL are unchecked. I suspect this is the reason for this issue.
Please try checking URL radio button.

I was attempting with neither or each and still it is protected.
I have since found that if I set a TEMPLATE in the driver this is protected.  I removed the template from the driver definition, and now I can enter the URL in my connection.
Sounds like a defect, but I am working for now.  Just seems wrong to have a template but not be able to overwrite the template in a connection.
From: Elizabeth @.>
Sent: Tuesday, July 9, 2024 2:43 PM
To: dbeaver/dbeaver @.>
Cc: Kevin Heiman @.>; Author @.>
Subject: Re: [dbeaver/dbeaver] Unable to edit URL on a Generic driver with Template (Issue #34684)
Sent by an external sender.  Please be cautious about clicking on links and opening attachments.
From the first screenshot, I see that both radio buttons Host and URL are unchecked. I suspect this is the reason for this issue.
Please try checking URL radio button.
—
Reply to this email directly, view it on GitHubhttps://github.com/dbeaver/dbeaver/issues/34684#issuecomment-2218519233, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJVTZGA57GM26XIG75AIISLZLQ4LJAVCNFSM6AAAAABKPKMKD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJYGUYTSMRTGM.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Yeah, it looks like a bug. Thank you for report.

@heimankl as a workaround you can clear URL Template field in driver settings.


Migrate CLI

The Turms CLI right now is very rudimentary, and lacks features, building on
Click + Rich Click + Watchfiles, we could add support for

File Watching
Project based generation
Project initialization.

As this would add additional dependencies, this will create a new minor release.
#52 Is the current prototype for this.

#52


[FEATURE REQUEST]: Improve tags for SEO

Is your feature request related to a problem? Please describe.
Currently, we don't have any tags for ParquetSharp.DataFrame. I thought it would be good to add similar tags to ParquetSharp to help it rank better on GitHub searches as well as Google.
Describe the solution you'd like
I would like to add keywords to increase the rank on both GitHub and Google, as well as making sure that the people that need to use the project, can find it.
The suggested tags are:
dataframe, csharp, dotnet, parquet, big-data
The keywords I'm optimizing for are:

dataframe

High relevance (13k results)

parquet dotnet

Medium relevance (19 results)
Not in results

dotnet dataframe

Low relevance (5 results)
Not in results

csharp dataframe

Low relevance (5 results)
Not in results

"big-data" was added for extra hits, since it has lots of results and may help raise rank when queried along with C#/.NET.
Feel free to make any suggestions or changes as required!
Describe alternatives you've considered
No response
Additional context
No response

I've added the tags as requested, thanks @jescalada!


README

README
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN the prework project repository
WHEN I view the README for the repository
THEN I can see a professional README that provides information on the project and how to use it

I can create README.md file documentation. Please assign me this task.


for ... in enumerate(Dataloader) error

Thanks for your awsome work on precise action spotting. I met an error as follows:

I tried to debug, finding out that its happens here:

However, I've check my dataset and dataset was constructed successfully: ' asample = train_data._get_one() ' runs correctly.
Could you please give me any help.

Emm.... it seems an natural error when running on windows with num_workers>0 for torch.utils.data.Dataloader

Emm.... it seems a natural error when running on windows with num_workers>0 for torch.utils.data.Dataloader.

Hi, I don't know that torch on Windows would do anything differently from Linux here. That error message suggests that you are trying to communicate a torch...Sequential object across worker process boundaries. This would likely be a new bug if you have made modifications to the dataset. If not, I would look at _get_img_transforms and its callers since that is the only place where Sequential should appear in the dataset.

Thanks for your advice, I use the soccernetv2(224p) dataset and follow your instruction to process the data. I might not correctly describe my question. Now the situation is, when I set num_workers to 0 and prefetch to 2, training runs successfully. However, once I set numworkers>0, and try command for batch in train_loader:. First error being 'EOFError: Ran out of input', if continue running, turning out to be  'Sequential object....'. Just like following picture:
Breakpoint at for batch in train_loader:

Once stepover, goes following 'EOFError: Ran out of input' error.

Continue to stepover:

The crash or error is definitely related to multiple processes in the dataloader. "EOFError: Ran out of input" can mean many things. If the the torch worker process crashes, the main process could get an EOF since no worker is left to send. It could mean that a file is missing (e.g., check that it crashes on the first input and not after some inputs), and that the error was rethrown at some point.
I'd suggest running the code in Linux (e.g., Ubuntu 20.04, 18.04, 16.04) and not in an IDE since I do not know (or have the environment to debug) how those could affect the results/errors regarding torch parallel dataloaders.

The crash or error is definitely related to multiple processes in the dataloader. "EOFError: Ran out of input" can mean many things. If the the torch worker process crashes, the main process could get an EOF since no worker is left to send. It could mean that a file is missing (e.g., check that it crashes on the first input and not after some inputs), and that the error was rethrown at some point.
I'd suggest running the code in Linux (e.g., Ubuntu 20.04, 18.04, 16.04) and not in an IDE since I do not know (or have the environment to debug) how those could affect the results/errors regarding torch parallel dataloaders.

Thanks for your prompt reply. I'll try it on Linux later.


Unable to create Twitter service

I have just installed Huginn on Debian Stretch, install was from source using the following instructions - https://github.com/huginn/huginn/blob/master/doc/manual/installation.md
Twitter keys were added to .env file and the Huginn server is up and working, however when i click the "Authenticate with Twitter" button on the services tab, i enter my Twitter credentials, confirm MFA, then get the message that I am being redirected back to the application, the screen refreshes and I am back at the Services tab, nothing listed under "provider" and when i try to create a twitter stream agent, there are no services in the dropdown box.
The authentication and communication with Huginn seems to be working based on the log in the production.log:
I, [2022-03-23T18:30:02.856272 #30857]  INFO -- : 2022-03-23T18:30:02+1300: [Worker(host:****** pid:30857)] 5 jobs processed at 32.6122 j/s, 0 failed
I, [2022-03-23T18:30:09.740510 #30899]  INFO -- : [267d67c4-309a-45a6-a5a9-8c6c81d74d61] Started GET "/auth/twitter/callback?oauth_token=************&oauth_verifier=****************" for <IP_ADDRESS> at 2022-03-23 18:30:09 +1300
I, [2022-03-23T18:30:09.741086 #30899]  INFO -- : [267d67c4-309a-45a6-a5a9-8c6c81d74d61] (twitter) Callback phase initiated.
I, [2022-03-23T18:30:11.190779 #30899]  INFO -- : [267d67c4-309a-45a6-a5a9-8c6c81d74d61] Processing by OmniauthCallbacksController#twitter as HTML
I, [2022-03-23T18:30:11.190949 #30899]  INFO -- : [267d67c4-309a-45a6-a5a9-8c6c81d74d61]   Parameters: {"oauth_token"=>"****************", "oauth_verifier"=>"*************"}
I, [2022-03-23T18:30:11.201425 #30899]  INFO -- : [267d67c4-309a-45a6-a5a9-8c6c81d74d61] Redirected to https://*************.com/services
I, [2022-03-23T18:30:11.201726 #30899]  INFO -- : [267d67c4-309a-45a6-a5a9-8c6c81d74d61] Completed 302 Found in 11ms (ActiveRecord: 1.7ms | Allocations: 2192)
I, [2022-03-23T18:30:11.439621 #30898]  INFO -- : [6de0347a-729c-446b-9b1e-71739aa7a6d4] Started GET "/services" for <IP_ADDRESS> at 2022-03-23 18:30:11 +1300
I, [2022-03-23T18:30:11.440922 #30898]  INFO -- : [6de0347a-729c-446b-9b1e-71739aa7a6d4] Processing by ServicesController#index as HTML
I, [2022-03-23T18:30:11.449767 #30898]  INFO -- : [6de0347a-729c-446b-9b1e-71739aa7a6d4]   Rendering services/index.html.erb within layouts/application
I, [2022-03-23T18:30:11.452593 #30898]  INFO -- : [6de0347a-729c-446b-9b1e-71739aa7a6d4]   Rendered services/index.html.erb within layouts/application (Duration: 2.6ms | Allocations: 705)
I, [2022-03-23T18:30:11.455337 #30898]  INFO -- : [6de0347a-729c-446b-9b1e-71739aa7a6d4]   Rendered layouts/_navigation.html.erb (Duration: 1.8ms | Allocations: 751)
I, [2022-03-23T18:30:11.455748 #30898]  INFO -- : [6de0347a-729c-446b-9b1e-71739aa7a6d4]   Rendered layouts/_messages.html.erb (Duration: 0.1ms | Allocations: 18)
I, [2022-03-23T18:30:11.456162 #30898]  INFO -- : [6de0347a-729c-446b-9b1e-71739aa7a6d4]   Rendered application/_upgrade_warning.html.erb (Duration: 0.1ms | Allocations: 5)
I, [2022-03-23T18:30:11.459814 #30898]  INFO -- : [6de0347a-729c-446b-9b1e-71739aa7a6d4] Completed 200 OK in 19ms (Views: 9.3ms | ActiveRecord: 2.4ms | Allocations: 4702)

If someone can please point out something I might be missing that would be really appreciated.

Same here

Getting the same result as well in a microk8s pod using docker.io/huginn/huginn-single-process.

Same issues has been ongoing for me. Heroku with SSL, self-hosted, Gcloud.

After fighting with #3037 and a few redeployments of the docker container, I'm now seeing this issue too.  After a bit of research, it looks like the callback phase never executes properly.  This results in huginn/omniauth-twitter not making the second request to twitter to get the actual authentication token required. (See here, step 2 works, step 3 doesn't get executed).
I'm not sure if this is limited to the docker builds, or it applies to stuff outside of docker as well.

im stuck on this, is there any workaround? i auth with twitter, i see the green redirect message, and then im back to autenticate with twitter button :/

after some googling, it seams like twitter has changed something. if you have created the twitter app years ago it works (even works to reset keys), but if you use a newly created app in twitter it dosent work, i guess something with callback payload is different. lack the dev skills to figure it out tho

after some googling, it seams like twitter has changed something. if you have created the twitter app years ago it works (even works to reset keys), but if you use a newly created app in twitter it dosent work, i guess something with callback payload is different. lack the dev skills to figure it out tho

This is the assumption I've come to as well. I've had somebody that uses huginn and twitter test successfully using their keys, which were created a few years a go, new clean install works no problem, put my keys in and it doesn't. It'd be nice if huginn was moved over to omniauth-twitter2, which'd use the newer auth methods. That said, I did see somebody post a blog post on using twurl https://weblog.antranigv.am/posts/2022/03/tweeting-with-huginn-and-twurl/ but in the docker image, that becomes a little harder to implement.
I've been trying to setup a skeleton of just the auth portion for twitter to see if I can see what might be different to submit a request for the change. Ruby isn't my forte, so I've been bashing my head a bit.

I decided to do some playing around and spun up a very basic ruby on rails environment, and think I've figured out the potential cause.  Twitter has several "access levels" based on your needs and what they'll grant you to (see here).  If you go to your twitter dashboard and look at your project, you may see it says "Essential" next to it.  Based on that, and the allowed access levels, you no longer have access to OAuth 1.0a, which is what omniauth-twitter uses to authenticate, nor a number of the standard v1.1 API endpoints (note it says "limited access - only media endpoints").  The failure is actually upstream in the omniauth-twitter gem itself.
I suspect earlier apps/projects were grandfathered into the "Elevated".
I'll do some more tinkering and see if I can build something against omniauth-twitter2.

interresting, hoping for good results :D
like you said here "I suspect earlier apps/projects were grandfathered into the "Elevated"."
i believe the same thing, something differs between old and new apps, even if you have elavated acces on a new app it dont work.

I should add, the search agent work fine, but stream does not. So the auth is correct. Must be something specific with stream in api. If that helps:)

The Twitter functionality was the reason I installed Huginn. It's unfortunate that it doesn't work. Hope there's a fix soon.

The Twitter functionality was the reason I installed Huginn. It's unfortunate that it doesn't work. Hope there's a fix soon.

It does work with twitter search for example. Its just the live stream that dont work

It doesn't work for me. I'm unable to successfully authenticate with twitter. When I click on the "Authenticate with Twitter" button, I just get redirected to a page with red text saying "We're sorry, but something went wrong." which is utterly useless.

It doesn't work for me. I'm unable to successfully authenticate with twitter. When I click on the "Authenticate with Twitter" button, I just get redirected to a page with red text saying "We're sorry, but something went wrong." which is utterly useless.

Then you use wrong keys, should use consumer keys

I saw the tip about using consumer keys already, and I verified that's what I'm using. I even tried regenerating them just now, to no avail. I've also tried applying for elevated access, which I received. It did not help.

I saw the tip about using consumer keys already, and I verified that's what I'm using. I even tried regenerating them just now, to no avail. I've also tried applying for elevated access to the Twitter API, which I received. It did not help. I'm using Docker if that makes any difference.

try reset keys? works everytime for me even remade the app many times. or your callbackurl is wrong somehow?
it works without elevated even i think.
also run docker, with hugin.env file with inputs from above

I did reset the keys, I mentioned that above. I don't think the callback url is the issue, I'm not even getting that far in the auth process. What callback url should I be using? I'm going off what it says in the wiki.
When did you create your developer account/app? I just made my account and app a couple of days ago.

i did my dev account few weeks ago.
you should use http://adressyouusetoreachhuginn:3000/auth/twitter/callback as url, add multiple if you use many diff adresses/ips

Well I don't know then, that's exactly what I've done, and it doesn't work. I can't find any logs that say exactly what is wrong either, so I'm stuck.

Well I don't know then, that's exactly what I've done, and it doesn't work. I can't find any logs that say exactly what is wrong either, so I'm stuck.

restart docker withouth -d flag and look in terminal what happens when you press the auth button

show me your huginn.env file (not the actual keys tho)

This is the only thing twitter related I see in the terminal, and it happens shortly after startup:
huginn_app  | foreman stdout | 11:04:10 web.1  | <IP_ADDRESS> - - [15/Oct/2022:11:04:10 PDT] "GET /auth/twitter HTTP/1.1" 500 643
huginn_app  | 11:04:10 web.1  | - -> /auth/twitter

There is no specific output when I click the auth button.
I'm not using a .env file, I simply have the environment variables defined in my docker-compose.yml file. I'm not using any quotes.
- HUGINN_TWITTER_OAUTH_KEY=xxxxxxxxxxxxxxx
- HUGINN_TWITTER_OAUTH_SECRET=xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

I've tried with and without the HUGINN_ prefix (which is what the wiki says to use), both seem to work (the auth button appears under services), but the result is the same either way. I've even tried using the various other keys and secrets the twitter app gives me; the auth button won't even appear with the v2 keys (which I expected not to work anyways).

I finally got it working. For some reason, I had to create a new, standalone app in my Twitter dev portal, and configure it exactly the same as before. For whatever reason, it does not work with the initial app/project the dev portal has you create when you first sign up.

So you managed to get yours to authenticate? And it appears now as a service under the Services tab? I've tried the consumer keys, the access tokens, etc.  Redirects to Twitter auth page just fine, twitter sends back to Huginn, and never actually creates the entry.  Deleted and recreated the project and app in the twitter dashboard so many times I've lost count.

Yes, I was able to click the authenticate button, get redirected to Twitter, login, and then get redirected back to Huginn, and the new service was listed. I just set up a twitter search agent using the newly added service.
What I did was create a new app in my twitter dev portal and did not add it to a project (so it shows in the left menu under "Standalone Apps"). I then followed the instructions here https://andreas.scherbaum.la/blog/archives/1061-Enable-Huginn-Twitter-integration.html and was able to successfully authenticate.

So that seems to be the difference, I don't have an option anywhere in the Twitter Dev portal to create a stand-alone app, it has to be in a project, which forces it to v2. Unless there's another dashboard I'm missing out, I'm hitting this one https://developer.twitter.com/en/portal/dashboard.
If I click on the link from the setting up OAuth docs, it takes me to one page, very briefly see a red error about not being able to retrieve apps, and then immediately redirects to /en/portal/dashboard.

I have applied for elevated API access (free, just submit the application), so you may need to do that first before you can create a standalone app.

I can confirm that you need to

Create a dev account
Apply for elevated access
Create a "Standalone App" (OAuth 1.0a)
Enter the API Key and Secret (not ClientID and Secret)

..for this to work properly

Stream agent dont work on new dev accounts for some wierd reason, other twitter agents work tho

Apply for elevated access worked for me.


[Sim_plots] 'model response' plots need to 'breathe' better: taller, and contour plots wider

Background / motivation
Here's the current "model response" plots

Problems:

Bar plots are needlessly short. What if there are 20 vars?
And, bar plots are wider than needed. Whereas contour plots could happily be wider
Contour plots are needlessly short. Ideally they are just as tall as they are wide, giving equal emphasis to the x0 dimension as the x1 dimension

TODOs / DoD

[ ] Make bar plot 2x-3x taller than status quo
[ ] Make bar plot take 25%-35% of the width, and contour plot 65%-75% of the width
[ ] Make contour plot height = its width (at least approximately)

Is this looking good?

Much, much better. Though the bar plot is not quite wide enough now.
See if you can reduce the whitespace between the two plots.
And ideally the legend for the plot on the right isn't to the far right, but right above or below.
That might be enough to make the bar plot wide enough. If not, we can give it a slightly higher % of the width.

Fixed in https://github.com/oceanprotocol/pdr-backend/commit/e9962cd1b51e47281f35692c951429388b307ab2