Restructure Docs for Auk

This might be a pain because we'll have to check for broken links. However, we need to make it easier for people to find information and understand how the core pieces of feathers fit together so that they can get to the "Aha!" moment earlier.
We should tag the old version so that people can still view it in Github. Especially since it is still pretty navigable.
Suggested Changes

Have a short overview section of the parts of feathers with a diagram of data flow

Hooks are for manipulating/filtering data in/out and managing workflows by calling out to other services or APIs, and emitting events. They allow you to guarantee workflow order and make it easy to reason about ~application state~ your business logic and data as it flows through your app.
Services are objects that expose a consistent interface to a datastore, API or internal object so that they can be interchangeable
Filters are for filtering events before they are actually dispatched to connected clients

A quick start guide/video or link to one
Move everything else to be more API centric and less book like
Bring FAQ top level

A smaller thing from my comment at https://github.com/feathersjs/feathers-docs/issues/210#issuecomment-254253136 is that I think we can remove the "Feathers vs. X" detailed pages and just turn them into blog posts. It's hard to keep them up to date and it isn't really documentation related.

Agreed. They should come out of the docs and maybe moved to the blog or website for SEO and educational value. Like a lot of the content in the docs they muddy things up.
In order to address @marshallswain's concern I was thinking that the next version of the site will use Metalsmith so pages like that can be in Markdown and people can still edit them. If we are short on time we could hard code them on the site or not include them for now.

Here is my initial suggestion for the updated outline:

Introduction
How Feathers Works
Getting started

Setting up
Scaffolding and services
Authentication
Manipulating data
Building a frontend

Services
Hooks
REST APIs

Configuration
Routing, middleware, versioning
Error handling

Realtime APIs

Service events
Event filtering
Socket.io
Primus

Authentication

Strategies

Username/Password
API Key
OAuth (Google, Facebook)
Passwordless

Password management
Permissions
Client use

HTTP
Websockets

Clients and Frameworks

Feathers client

REST
Socket.io
Primus

Frameworks

React
React Native
Angular 2
Vue.js
jQuery

Direct connection

REST
Socket.io
Primus

Databases

Adapter basics

Pagination, sorting, selecting
Querying
Extending

NoSQL

In Memory
LocalStorage/AsyncStorage
Mongoose
MongoDB
NeDB
RethinkDB

SQL

KnexJS
Sequelize

Ecosystem

feathers
feathers-rest
Feathers-socketio
Feathers-primus
Feathers-hooks
Feathers-hooks-common
Feathers-configuration
feathers-errors
…

Help

FAQ
Contributing
Changelog

License

@feathersjs/core-team

We probably also want to discuss the structure for each article and how parameters should be formatted so that the important information (like the API docs) are always located at the same spot. E.g.
--
Summary
app.x()
Thing that does X
Options:

foo - Sets the foo parameter
bar - Sets the bar parameter

Example
app.x()

Usage
Here the detailed chatty stuff.
Topic X
Any other topics as well.

The documents change, especially when new features are added. What do you think about having a What's New section. Current Feathers users can use it to keep on top of changes and new features.

@daffl your suggested layout looks good to me! I think we should bring FAQ, Changelog and Contributing top level.
For frameworks are you thinking they are currently the way they are or should they be more API focused examples and move the more wordy guides to Medium?
We also might want to have a "New to Node" area that some people can skip. I've had some great feedback from a couple people that we are assuming too much NodeJS/Express knowledge. Someone new to node may not know how the ecosystem works, what express or express middleware is. I'm not sure if that should be really short section just after the "Introduction" or if it should be a Feathers post on Medium.
@eddyystop I like that idea. It would be good to highlight new stuff. However, I'm wondering if really that's what the release posts should be about.
I was thinking that if we decided to customize our docs with metalsmith we could have a "new" or "updated" badge in the nav beside each new section. However, whether we have badges or a dedicated section in the docs we'll need to keep it up to date and remove things as they become less new. If that's the case it would be good to define how long something is "new".

I also do really like how Evan has Vuejs laid out. https://vuejs.org
It's kind of nice that he has API, Guide, Examples split as separate sections. I mean we have the them split up the same way on our main site but our current version of the docs is a mix between a guide/book and API docs. They really should be separate.
Vue's docs are API docs, and the guide is more like our docs are right now.
Our docs should probably just primarily be API docs, a guide could be a free or paid book, and individual guides/examples can point to the Guides/Examples collections in Medium. We can link to appropriate github repos, gists or jsfiddles for those right in the guide. Thoughts?

These are our top 25 pages:

Yes, I was thinking Ecosystem page should have a list of all official Feathers plugins and then the content from the "Showcase" page. The Ecosystem sub-sections would be links to all official Feathers plugins. They'd either have their own page (e.g. what is "Error Handling" now would become the page for feathers-errors, "Configuration" would become feathers-configuration etc. - that way we can add new plugins without having to figure out where they might fit in) or link to the existing documentation section (e.g. feathers-socketio would link to "Realtime/SocketIO", feathers-rest to "REST/Configuration").
The top 25 list is really helpful. We should probably keep those in one form or another (maybe a "Why Feathers" section with links to blog posts?). Having a "New to Node" section is a great idea, too. We probably want to expand on that eventually anyway. We probably want to keep the "What's new" stuff in the release posts for now.
Personally the differentiation between API and guides never made much sense to me. Say I'm reading the "Event filtering" section and it links to "Service events". Do I want to read the guide because I skipped it or do I just quickly want to look up the API for how to listen to them and what the available events are? I'm hoping that if we stick with the page layout I proposed (Summary and API, Example, In-Depth explanations) everywhere it would make a good middle ground. I don't even think we have to change too much in our docs. I'll start the branch and commit some examples of what I was thinking.
I have some reservations about using Metalsmith for generating the docs but we can discuss that further later.

Also, we can probably drop the changelog completely and just link to each repositories generated changelog.
I'm thinking we should at least keep the important framework chat app guides (and a page for "Others" or "My framework" or something). Otherwise I'm expecting lots of "How do I use this with " questions. We can move the more esoteric ones that are not likely to be updated frequently in the blog.

The final structure of the API docs (and current guide restructuring) can be found in the auk-reorg branch viewed at https://docs.feathersjs.com/v/auk-reorg/api/readme.html


plot_tool_set_axis_properties_matplotlib not imported anywhere

line 400 in plot_vector_fielt_matplotlib.py: plot_tool_set_axis_properties_matplotlib is not imported/defined anywhere.
Should this be tool_set_plot_axis_properties_matplotlib from  tool/tool_set_plot_axis_properties_matplotlib.py?
There are also two references to a method self.plot_tool_set_axis_properties_matplotlib in the code. Should this also be changed?

Fixed.


Fetch: trigger pending fetches when producing messages.

When the Kafka client issues a Fetch and sets a maxWait time we are already scheduling a DelayedFetch, but there is no way to trigger that Fetch and it is deemed to wait for the fully timeout.
This adds latency spikes on the Kafka Consumer.
With this patch we are triggering any pending DelayedFetch in case of writing any record to one of the partitions interested by the Fetch.
This is only a first implementation, in the future we can make it better and do not trigger at the first record, but wait in any case for more records to come.
With this implementation the Fetch result will contain usually only 1 record, but this is enough to let the Kafka Client start a new Fetch cycle and do not waste time in doing nothing (waiting for maxWait).
Changes:

trigger pending Fetches while producing to the topic
add new metric WAITING_FETCHES_TRIGGERED
add DelayedOperation#wakeup, that means the operation should wake up due to some trigger  (in this case the Production of records to the topic)
add a new test that would fail without this patch (because the tests asserts that there is no idle cycle in the Consumer loop)

I've left some comments, PTAL. BTW, please update the PR description because the current design changes a bit from the initial design.

@BewareMyPower description updated.
if you want I can drop the "key" parameter as it is currently useless

@BewareMyPower  I have removed that parameter
PTAL

@BewareMyPower @Demogorgon314 CI passed

Thank you @BewareMyPower
I will work on a fix.
Probably early next week


useHoverDirty prevent event bubbling?

Is your feature request related to a problem? Please describe.
We've been using the useHoverDirty hook to conditionally render content next to an element, and noticed if you stack the elements, both of them are considered hovering. I'm not sure if this is intentional or not? I saw you could set it enabled or not on the second parameter, but it turned out to be cumbersome to coordinate through the tree.
Describe the solution you'd like
I forked useHoverDirty to prevent the hover event from bubbling to the next event: https://codesandbox.io/s/cranky-sun-lt7mb
Describe alternatives you've considered
I was thinking we could pass an options object as the second parameter, which would include {enabled = true, stopPropagation = false} and but this would be a breaking api change :-/

Closing from lack of engagement


Updating the libcloudforensics.py lib with the GCP labels management functionality and adding missing modules to dev requirements in setup.py

##########################Changes to libcloudforensics.py ##########################
the GCP labels management functionalities will be used to add incident_id labels to the GCE vm and disks created and to tie them all to an incident. This change will enable label management on all GCE resources that extend that class (VMs, Disks, Snapshots) and will do the following changes to:
******** GoogleComputeBaseResource class in the libcloudforensics.py ********
Add: get_labels
Description: get all the labels assigned to a certain resource
Add: add_labels
Description: add/update labels to a certain resource
Add: form_operation
Description: helper method to allow creating different GCE API operation object (ex: insert, setLabel, etc..)
Add: get_resource_type
Description: helper method to get the compute resource type from the resource key-value store (ex: vm, disk, snapshot, image, etc ..)
******** GoogleCloudProject class in the libcloudforensics.py ********
Add: list_instance_by_label method  to GoogleCloudProject class:
Description: Lists VMs in a project with one of the provided labels.
This method list compute engine instances if they have one of the supplied label
Add: list_disk_by_label method  to GoogleCloudProject class:
Description: Lists Disks in a project with one of the provided labels.
This method list compute engine disks if they have one of the supplied label
Add: __list_by_label method  to GoogleCloudProject class:
Description: Private helper method to list_instance_by_label and list_disk_by_label methods.

Tests are done in a separate setup against GCP APIs. Can be further discussed if needed

##########################Changes to setup.py ##########################
Adding missing modules "celery" and "coverage" to the dev setup so that run_tests.py doesn't fail in dev mode

I pushed one commit (https://github.com/google/turbinia/pull/459/commits/9ce60cf894a251650584f6ec52ee67fc5c284fe2) so make sure to pull before you push 💪

@aarontp Feel free to merge this once you get a chance

@googlebot I fixed it.

@googlebot I consent.


Add a better description to AMO

A lot of people seem to be unaware that you can enable read/unread prefixes. Maybe even make that the default?

I updated the description on AMO a bit

@evilpie Read/unread prefixes is a hidden and unexposed feature.

Yeah let's forget about read/unread for now. I hope the new description on AMO is ok.


How can I have two parent nodes pointing to the same child node?

How can I have two parent nodes pointing to the same child node on a d3 tree graph?

If one child has two parents, it's no longer a tree. It's a graph.

That’s right. A node in a d3.hierarchy can’t have multiple parents, and none of the d3-hierarchy layouts work with arbitrary graphs; they only work with trees.
That said, I think I recall an example that uses two trees with mirrored orientations (left-oriented and right-oriented, for example) a shared central node, such as for a genealogy diagram showing the ancestors and descendants of a particular person. By searching I also found another example showing how to overlay additional parent-child links on top of a tree.
Please use Stack Overflow tag d3.js to ask for help. Stack Overflow provides a better collaborative forum: thousands of D3-related questions have been asked there, and some answers may be relevant to you.
When asking for help, please include a link to demonstrate the issue, preferably as an Observable notebook. It is often impossible to debug from code snippets alone. Isolate the issue and reduce your code as much as possible before asking for help. The less code you post, the easier it is for someone to debug, and the more likely you are to get a helpful response.
If you have a question about D3’s behavior and want to discuss it with other users, also consider the d3-js Google Group or joining the d3-js Slack.
Thank you! 🤗

̿̿ ̿̿ ̿̿ ̿'̿'\̵͇̿̿\з= ( ▀ ͜͞ʖ▀) =ε/̵͇̿̿/’̿’̿ ̿ ̿̿ ̿̿ ̿̿

I think I recall an example that uses two trees with mirrored orientations (left-oriented and right-oriented, for example) a shared central node, such as for a genealogy diagram showing the ancestors and descendants of a particular person.

Apologies for jumping into a closed issue - is this the example you are referring to? https://www.entitree.com/en/family_tree/Elizabeth_II

I Need a sorce code of above Mention Examples @ogroppo


Fix exceptions thrown when querying nested object arrays' sub-columns

Summary of the changes / Why this improves CrateDB
Fixes:
cr> create table t (o object as (a int)[][]);
CREATE OK, 1 row affected (0.320 sec)
cr> insert into t values ([ [{a=1}], [{a=1},{a=1}], null ]);
INSERT OK, 1 row affected (0.096 sec)

cr> select o['a'] from t;
ArrayViaDocValuesUnsupportedException[Column "2" has a value that is an array. Loading arrays via doc-values is not supported.]

cr> select * from t where array_length(o['a'], 1) >= 1;
UnsupportedFunctionException[Unknown function: array_length(doc.t.o['a'], 1), no overload found for matching argument types: (integer, integer). Possible candidates: array_length(array(E), integer):integer]

This PR fixes two issues

identify o['a'] as array_array_int (if o is 1D object array, CrateDB identifies o['a'] as array_int as expected)
to be able to parse the data returned from Lucene as nested object arrays

Checklist

[x] Added an entry in the latest docs/appendices/release-notes/<x.y.0>.rst for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in the latest docs/appendices/release-notes/<x.y.0>.rst
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

With the current fix, still fails to handle more complex case:
cr> create table t (o object as (o2 object as (b int)[][])[][]);
CREATE OK, 1 row affected (1.902 sec)
cr> insert into t values ([ [ {o2 = [[{b=1}]]}  ] ]);
INSERT OK, 1 row affected (0.166 sec)
cr> select * from t;
+--------------------------+
| o                        |
+--------------------------+
| [[{"o2": [[{"b": 1}]]}]] |
+--------------------------+
SELECT 1 row in set (0.899 sec)
cr> select o['o2'] from t;
ClassCastException[class java.util.ArrayList cannot be cast to class java.util.Map (java.util.ArrayList and java.util.Map are in module java.base of loader 'bootstrap')]

I do not think so, o['a'] is resolved to a reference not a subscript function. I will keep that PR in mind as I proceed, Thank you.

Hi @matriv could you review this? I think we can limit the scope and handle https://github.com/crate/crate/pull/16969#issuecomment-2471210169 separately.


Fix crashes in ConvertMappingCommand and GenerateEntitiesCommand...

... when using entities with joined table inheritance
ConvertMappingCommand and GenerateEntitiesCommand both use the DisconnectedClassMetadataFactory, which allows metadata manipulation without loading the associated classes. Commit a36bea broke these two commands by adding a bailout condition in ClassMetadataFactory::populateDiscriminatorValue which checks $metadata->reflClass->isAbstract(). If the DisconnectedClassMetadataFactory is being used, $metadata->reflClass will always be null, causing a fatal error, "Fatal error: Call to a member function isAbstract() on null".
This commit adds a check to see if $metadata->reflClass is set before checking isAbstract.

@zeroedin-bill any way to get a test case for this before merge? I can work on it tomorrow otherwise

@Ocramius I don't have time till late in the week probably :/ travelling and internal bug piles...

Will try dealing with it during lunch break then

I could not reproduce the bug. Neither with a StaticReflectionService nor with a mock reflection service.
Will merge anyway, as the conditional doesn't hurt there.

Thanks!


Curriculum for FFHQ

Hi, great work? Could you specify the curriculum for FFHQ or is it the same as CelebA-HQ? Thank you

It is almost the same as CelebA-HQ. You can also increase h_stddev and v_stddev a little bit, such as 0.4 and 0.2.


Rearranging editors in editors.json

Rearranged to

Live editors first,
Beta editors next,
Other editors at the end.

looks fine for playground!


Unknown message appears after live chat closing

When closing a live chat room, an unknown message ("promptTranscript") is posted in the room. The message is not visible for the client (see screenshot). Where does this message come from, and why is it posted?

Moved. https://github.com/assistify/Rocket.Chat/issues/39


Ensure developers running rubocop have the same version as the miq_bot

Running other versions of rubocop can be problematic with the current .rubocop.yml.

@Fryguy @jrafanie Please review

The only problem here is that the linters in editors (like SublimeLinter and probably syntastic as well) run outside of the bundler context, so although this ensures the right version is installed, it doesn't stop it from running a newer version if someone updates the gems locally.

@Fryguy I don't know how to solve that problem.  Maybe you need to put out bin directory on the path before everything else?  Do you have any ideas?  I don't think we can go into configuring all of the editors.  But at least this will help all of us running rubocop directly on the command line.

@Fryguy I don't know how to solve that problem

I don't think you can, but this is as close as you can get.

@Fryguy funnily enough I have an open PR for stopping rubocop-emacs from doing that: https://github.com/bbatsov/rubocop-emacs/pull/20


i18n

Is your feature request related to a problem?
currently there is no easy way to provide matrix-registration in other languages
Describe the solution you'd like
probably use https://l10n.dmnd.sh/

fixed in 9915e279eaf519ca617d9a1f6c21cdd7cb115a8e


[PDI-18501] MQTT Client Step to Include "Client ID" to Allow Persistent Connections

:white_check_mark: Build finished in 2m 44s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/streaming/impls/mqtt

:ok_hand: All tests passed!
Tests run: 40, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 2m 26s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/streaming/impls/mqtt

:ok_hand: All tests passed!
Tests run: 40, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


[QA] Test coverage for cli commands

Implemented base code on https://github.com/owncloud/ocis/issues/9286
NOTE: Implementation done in https://github.com/owncloud/ocis/pull/9546
Add test coverage for CLI commands introduced in oCIS:
Docs: https://github.com/owncloud/ocis/tree/master/ocis
To Implement:

https://github.com/owncloud/ocis/issues/8925
https://github.com/owncloud/ocis/issues/5602
https://github.com/owncloud/ocis/pull/9041

Open tasks:

https://github.com/owncloud/ocis/issues/9472 @prashant-gurung899

[x] all file versions https://github.com/owncloud/ocis/pull/9584
[x] from specific space https://github.com/owncloud/ocis/pull/9639
[x] from specific file https://github.com/owncloud/ocis/pull/9627

https://github.com/owncloud/ocis/issues/9474

[x] reset password: https://github.com/owncloud/ocis/pull/9546
[x] try to reset password of non-existing user @S-Panta  https://github.com/owncloud/ocis/pull/9575
[x] #9581 @S-Panta https://github.com/owncloud/ocis/pull/9593

https://github.com/owncloud/ocis/issues/9393 @S-Panta

[x] https://github.com/owncloud/ocis/pull/9617

https://github.com/owncloud/ocis/issues/9103 @ishabaral

[x] reindex all spaces https://github.com/owncloud/ocis/pull/9630
[x] reindex specific space https://github.com/owncloud/ocis/pull/9630

Keeping the issue in backlog due to the bug : https://github.com/owncloud/ocis/issues/8925 and https://github.com/owncloud/ocis/issues/5602

Keeping the issue in backlog due to the bug : #8925 and #5602

These are features to implement

We should make a list of all available useful (testable) commands of each service so that we can know what we have convered.
NOTE: we may not want to tests commands like help, init, server, health, version, benchmark, migrate (the requirement can change)
Example:

Service
Command
Test

ocis
backup consistency -p <basepath>
:heavy_check_mark:


The Compose for Desktop project I created based on Intellij IDEA does not seem to work. What did I do wrong?

build.gradle.kts

import org.jetbrains.compose.desktop.application.dsl.TargetFormat

plugins {
   kotlin("jvm")
   id("org.jetbrains.compose")
}
group = "com.example"
version = "1.0-SNAPSHOT"
repositories {
   mavenCentral()
   maven("https://maven.pkg.jetbrains.space/public/p/compose/dev")
   google()
   maven("https://gitlab.com/api/v4/projects/38224197/packages/maven")
}

dependencies {
   implementation(compose.desktop.currentOs)
   implementation("com.github.winterreisender:webviewko-jvm:0.5.0")
}

compose.desktop {
   application {
       mainClass = "MainKt"
       nativeDistributions {
           targetFormats(TargetFormat.Dmg, TargetFormat.Msi, TargetFormat.Deb)
           packageName = "compose_for_desktop_1"
           packageVersion = "1.0.0"
       }
   }
}

Main.kt

import androidx.compose.desktop.ui.tooling.preview.Preview
import androidx.compose.runtime.Composable
import androidx.compose.ui.window.Window
import androidx.compose.ui.window.application
import com.github.winterreisender.webviewko.WebviewKo

@Composable
@Preview
fun App() {
    println("start")
    with(WebviewKo()) {
        println("---")
        title("Title")
        size(800, 600)
        url("https://example.com")
        start()
        println("Goodbye webview")
    }
}

fun main() = application {
    Window(onCloseRequest = ::exitApplication) {
        App()
    }
}

output

There are no error messages, no warning messages, and no UI is displayed. What is going on here? What should I do to make it work correctly?

This is not the correct way to use the experimental webviewko-compose, see https://github.com/Winterreisender/webviewko/tree/main/compose for the demo.
// implementation("com.github.winterreisender:webviewko-compose:0.6.0-SNAPSHOT")
import com.github.winterreisender.webviewko.WebviewKoCompose.Webview
fun main() = singleWindowApplication {
    Webview(
      url = "https://example.com",
      modifier = Modifier.fillMaxSize()
    )
}

The Compose Multiplatform support is experimental, to be frank, it only works in Windows and doesn't work well.
Similar: #12

Thank you very much! @Winterreisender


Implement tests for the steering wheel knob area

User story

As a User
I want to be able to use a tested steering wheel knob area
So that I can be sure everything works properly and is safe for usage.

With this User story tests for the steering wheel knob area are to be written.
Acceptance criteria

The test coverage is at/above 80% to be releasable

Definition of done (DoD)

Code compiles and builds
Acceptance criteria are met
Feature tests have been written and were passed at/above 75% test coverage
Code is peer-reviewed
Feature is merged into development branch
Documentation is updated
Bill of Materials is updated

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

Currently no nice way to test voice control, maybe after implementing a voice assistant


scheduled/unlisted fall out

[ ] if someone misses the visibility section and tries to set a future date under additional options, give them a hint about that section / etc.

Hm ... I think we should just move the date widget to the Visibility section, rather than jump between 2 sections.  The phrase "Visibility" still makes sense for the date.
Any objections?

Would it be a new option? Maybe if I saw the UI, would be easier to see if that's the right path forward. I also don't think it's widely used for backdating, so don't want to introduce confusion either.

Just like this:

But I see your point. Ok, dropping.

[ ] Allow adding scheduled content to playlists

[ ] Skip past unlisted items in playlist (added when not unlisted) properly, like it does with Mature, etc.


Improve sortbale events

This PR includes:
-putMode added for sort, add and remove events
-originalEvent setted for onStart event

This PR will help in the resolution of SortableJS/Vue.Draggable/504

@David-Desmaisons I will merge this as-is, but I have a few commits to do first. If you need to, you can work around this commit being merged.

@David-Desmaisons Is there any reason that you don't have putMode in onEnd?

@owen-m1 The only reasons are that regarding the improvement for vue.draggable I did not need this change, and that I was not sure on how to add it.
But taking a better look now, it is easy to and will provide more consistent API.
If everything is OK for you, I will provide by Today.

@David-Desmaisons Sounds good, I do agree this would be a better API

Changes provided.

@David-Desmaisons If you are OK with the changes I just made above, then this is ready to merge

LGTM. Thanks.


Add toggle method in Eloquent model

The toggle method allow the user to invert the value of one or many attributes.
Let's say that we have a Post model with a publish method who is doing the following

Setting the published attribute to true
Saving the changes in the DB

class Post extends Model
{
    // ...

    public function publish()
    {
        $this->published = true;

        $this->save();
    }
}

With the toggle method, we can simplify the publish method as follow
class Post extends Model
{
    // ...

    public function publish()
    {
        $this->toggle('published');
    }
}

The toggle method can also accept an array of attributes
class Post extends Model
{
    // ...

    public function publish()
    {
        $this->toggle(['published', 'locked']);
    }
}

Regards.

This implementation does weird stuff for non-boolean attributes.
This implementation does weird stuff if you pass objects to toggle().
It's weird that a method named toggle also causes database calls, in this case to save the model.

I also believe that your examples are wrong, the later on where publish() just toggle the published flag can as well result in the call to publish() unpublishing the object if it is already published. That doesn't make sense for a method called "publish".
What is the status of a Post after this code has run?
$post = Post::find(...);
$post->publish();
$post->save();

No plans on adding.


Depend on setuptools >=16 for consistent exceptions

safety version: 0.5.1 & master
Python version: 3.4
Operating System: Fedora 25

Description
safety implicitly depends on setuptools, by way of from setuptools import setup in the setup.py, however pkg_resources.parse_requirements was introduced is only in setuptools 0.6, which is luckily just before Ubuntu precise python-setuptools 0.6.24 and trusty python-setuptools 3.3, but safety will very likely fail on anything earlier that Ubuntu precise (not a serious problem IMO, but .. worth painting the picture of this implicit requirement).
A more significant problem arises is when trying to catch requirements.txt parser errors, from read_requirements, the ideal exception to catch varies on the version of pkg_resources that is installed.  In all cases, ValueError is acceptable, however, as of setuptools 16 RequirementParseError was introduced.
Given setuptools 16 is over 1.5 years ago, and safety is about keeping up to date, I suspect that adding setuptools >=16 will be acceptable.
What I Did
try:
    safety.cli.read_requirements(text)
except pkg_resources.RequirementParseError:
    pass

Result:
AttributeError: 'module' object has no attribute 'RequirementParseError'

Thanks, fixed in  b9551f1

just ftr, that commit doesnt solve the problem referred to here, but https://github.com/pyupio/safety/commit/f51f05b08842c9ce4322d05d644500dbb8458673 probably fixed it, and switching to dparse definitely did.


External subtitle parsers

Hello @lukasf ,
this is the PR for #78
Outstanding issues:
C++ sample has not been updated yet.  We should do it after we figure everything out
I haven't tested anything but text based .srt. We will need some more files to test on. .srt works perfectly fine.
Technically, image based subs should also work, although I am not sure how to do the double stream thing.
The path tested is: create media playback item, then add an external subtitle to it. The other way around (first subtitle, then media playback item) hasn't been tested/implemented yet. I will do it later (today if I find the time).
The subtitles come up with  name, so we need to find a naming strategy for them. In my player, I use the name of the file. Maybe we can do it here as well.
Other stuff
Reading the entire subtitle stream is not really that computational intensive, at least for .srt. The file had around 800 cues or so, for a 45 minute video, and took around 1 second, faster than the windows API.
I added cancellation support, although I am not sure if that's going to be needed. It was trivial to add anyway.
Nothing was changed in how we parse the subtitles. My initial plan didn't survive contact with reality, but i think it is better to have everything at ffmpeginteropMSS level.
What do you think?

I'm working on the inverse path right now. So it will be done.

We could add start time adjustments as well, the StartTime of each Cue in TimedMetadataTrack has a public setter.

We have some new codez.
I discovered something...alarming.
It appears that on subtitles containing non-ASCII characters, such as â,ș,ț,î, FFMPEG skips the entire CUE, and will only display cues which are fully ASCII.
I will need to do further testing on this.

I am very confident that I have seen unicode subtitles with ffmpeg internal subs. Maybe for external files, ffmpeg does not know the character encoding and incorrectly assumes everything is ASCII. This would be bad.

Are you sure that the file you have does use proper UTF8 encoding, which is the de-facto standard for subtitles?

I have done some testing here with a SRT file I had. It is properly UTF8 encoded and displays correctly. If I open it with notepad and save it as ANSI, the same file does not render correctly anymore, just like you describe, all cues with unicode characters are skipped.
Not sure how to continue with this. Shall we try to fixup encoding? That's difficult because you never know which ANSI Codepage the encoder used when storing it...

Windows API renders the ANSI file correctly. Unfortunately. We cannot be worse than Windows API, right?

Isn't there some option we can pass to change the encoding? But we wouldn't know the encoding table regardless...

I guess microsoft uses the windows settings to determine the ASCII encoding. So the user provides that information. I think VLC also does encoding fixup, i guess we have to do it too. And VLC uses a user setting to get the ASCII page.

Happy new year! :D
Looks like there is an ffmpeg parameter "-sub_charenc" which does exactly what we need.
https://en.wikibooks.org/wiki/FFMPEG_An_Intermediate_Guide/subtitle_options
I will try to do some testing on that later today...

Oh man, getting complicated. It looks like ffmpeg can do character conversion, but only if compiled with "iconv" external library. This is originally a linux library, licensed under LGPL.
I am not really keen on beginning to link external libraries into ffmpeg. This would require every user of the lib to have this lib on ffmpeg compile time, for all platforms. Or we'd have to ship the sources of that lib as well, and include it into out build steps, making build files more complicated and error prone. I also found a library called win-iconv, which implements iconv by using Windows APIs instead of the original custom implementation. This is basically only one C source file and it is public domain, so no license issues. It could be included more easily, but it would still make the build more complicated.
The other alternative would be to check manually for UTF-8 BOM and then use Windows APIs to convert before passing to ffmpeg. Maybe this is the best option, but it is also a bit complicated, becase we'd have to re-package the whole thing back into a RandomAccessStream again. Maybe I will try to get that running somehow, no promises though.

Isn't it possible to do it at package level?

Unfortunately, the BOM is already stripped from the packets we receive.
Right now I have a weird problem: Conversion already works on RandomAccessStream level. But if I detect BOM, then I seek back the original stream to 0 and return it, because no conversion is needed. Now ffmpeg fails with that original stream. No clue what is going wrong there. The fixed up InMemoryRandomAccessStream works, but the original, seeked-back stream does not. Man...

Well, it wouldn't be a happy new year without all these xD
happy new year :D

Pretty weird: If I use a DataReader on my stream and throw away the DataReader later, it will close my stream. Like it owns the stream. I had to call DetachStream(). Strange default behavior...
I got it working right now. Need to clean up and push tomorrow.

So do we load the entire stream and then change the encoding manually?

Yes that is the case currently. But this cannot be the final solution, because we don't even know if we have text or image subs (or even some binary file). Thinking about moving BOM detection to low level FileStreamRead function and conversion into subtitle provider.

When adding subs, the entire file is loaded anyways during add function, so loading the whole stream upfront would not be a problem in general.

So how did you enjoy new year's eve?

Given the fact that I now have two small children, there was not so much room for big celebration this year. But it was a nice evening with friends who also have children. Hopefully next year we can do a bit more celebration again :)

Congratulations :)

Thanks :) So how was new year's for you?

Finally, got it working! Checkout latest commit.

createFromStreamInternal was unneccessary. I removed it.
CreateFromStream will never return nullptr.

Cool. I will start testing. The point of the createFromStreamInternal was to reuse the result of CreateFromStream. It was a copy-paste of what was inside the CreateFromStreamAsync method, without the IAsyncAction stuff. It was done to reuse code.
The new year's eve was fine, mostly family and friends :)

I will try to create some unit tests for this.

As always, unit testing in UWP is a major PITA. Visual Studio doesn't want to discover the tests.

CreateFromStreamAsync calls CreateFromStream on a new task. So if you want it synchronous, you can just call CreateFromStream directly (which is what I do now).
Last time I ran unit tests, they worked. Now VS also fails to find the test. When I open test output window, I see "System.InvalidOperationException: Could not get IP address of target". Do you get the same error?

I get this: DEP3000: Attempts to stop the application failed. This may cause the deployment to fail. [0x92330047] Exception from HRESULT: 0x92330047
I'm not sure which app it talks about, since the app wasn't running anyway.
I worked around this BS unit test for UWP in my projects by creating a usual UWP app and run the code through there.
Maybe we should do the same.

I get the same error in build output. When i switch output tab to "Tests", I see that other error. Tried different things that are supposed to fix this, without success...

Yes, I see the same error message in tests too. So we have the same problem. Maybe this is a Visual Studio issue?

Seems to be. Nothing has changed in the test project since quite some time. Maybe we should just wait and see if things start working again with one of the next updates. At least, I am not motivated to work on this right now.

Yeah, me neither. I will try to test this some other way.

LRC is for lyrics, not for subtitles. Still we can add the workaround you mentioned. Maybe with a more reasonable duration such as 1 minute. Better not risk overflow errors when getting near max values.

I have thought about adding an overload for sub/idx subtitles. Instead of going through ffmpeg, we could realize those through UWP APIs. Then users don't have to fiddle with two different approaches, depending on format.
I think the important formats are: SRT, SSA/ASS, PGS (BluRay, also uses .sub extension), SUB/IDX (DVD). All other formats are very rarely used.

Since FFMPEG supports the parsing of LRC, I don't see why we shouldn't add them ^^
I agree with using the windows API for image based sub/idx subtitles, it should be the easier way around this.

.sami -> same issue as LRC.
.srt -> works fine
.VTT -> works fine.

I have tested with PGS image subtitles and found a bunch of issues. Fix is online now.
Most importantly, we need to enable all subtitle streams, otherwise image subs are not parsed. Second, we need to ignore start time if the file is a pure subtitle file. For most files, it will be the time of the first sub, leading to wrong timing. And sometimes it even caused negative subtitle positions, which then caused exceptions.

PGS (SUP) work fine now, also tested with SRT and SSA/ASS successfully.

Hmm. I will give a try to sub and usf again. Maybe the start position was bugging them.

Do we still need to keep the old charset encoding?
Other than that, we are good for finally merging this.

While I did the code review, I realized there is probably a different approach to the anti flicker solution, that would work without the reference track. The main problem was that we cannot change the duration of an already active infinte duration cue. This forced us to add the reference track. But now that we know how to add and remove cues without flicker (on DispatcherTimer), we could instead replace the old infinite duration by a new cue with same content but fixed up duration. When done on DispatcherTimer, this should be possible without flicker, and would allow us to remove all the reference track stuff. This is just guessing, but I think it would work.
Still, the current solution works well. We should keep it and merge this PR once cleanup is finished. If motivation is there, we can still try that new approach in a separate PR.

Nah, let's not change what works unless there is a really good reason to do so.

Ok so this looks fine now from my side. Are we ready for merge or do you see any open issues?

Should we keep the old button for opening external subtitles with windows API?

It could be useful, to see if the windows API can open a file correctly that fails with ffmpeg.

Then we are good to go.


The issue concerns the initialization of weights and biases in the SGP module.

Hi， This work is so interesting. However, I have some questions about the initialization of weights and biases in the SGP module. Due to my lack of coding experience, I can not understand why the weights and biases of the convolutions in the SGP block are initialized to 0. When I was debugging, the weights and biases of these convolutions were 0 in both training and testing. What I understood was that they had no effect. So how does the SGP block play a role in the code? How to understand it? (Note: If init_conv_var is set to non-0, the results will drop a lot.), Thank you.

Hi, for the THUMOS14 dataset, since the action lengths are relatively short, we found in our experiments that initializing from zero weights (i.e., retaining only the residual connections at the very beginning) can make the training more stable. For datasets like HACS and Activitynet, which contain many long actions, enabling the SGP layer during initialization can achieve better results.

Hi, for the THUMOS14 dataset, since the action lengths are relatively short, we found in our experiments that initializing from zero weights (i.e., retaining only the residual connections at the very beginning) can make the training more stable. For datasets like HACS and Activitynet, which contain many long actions, enabling the SGP layer during initialization can achieve better results.

Thank you for your reply. However, through experiments, I found that throughout the training and testing process, the weights and biases of the convolutions involving the SGP block are still 0, equivalent to the SGP convolution always being 0, and only the residual connections are retained.

Hi, for the THUMOS14 dataset, since the action lengths are relatively short, we found in our experiments that initializing from zero weights (i.e., retaining only the residual connections at the very beginning) can make the training more stable. For datasets like HACS and Activitynet, which contain many long actions, enabling the SGP layer during initialization can achieve better results.

Thank you for your reply. However, through experiments, I found that throughout the training and testing process, the weights and biases of the convolutions involving the SGP block are still 0, equivalent to the SGP convolution always being 0, and only the residual connections are retained.

The characteristic of THUMOS14 dataset is that a video has dozens or hundreds of actions, and many actions have only a length of several features. For this kind of dataset using a large window to aggregate features is not necessary, but for large-scale and highly varying datasets such as HACS, enabling the multi-scale feature extraction will be more necessary.

The characteristic of THUMOS14 dataset is that a video has dozens or hundreds of actions, and many actions have only a length of several features. For this kind of dataset using a large window to aggregate features is not necessary, but for large-scale and highly varying datasets such as HACS, enabling the multi-scale feature extraction will be more necessary.

What you said makes sense, thank you for your reply.


0.7.4 - Fix for Android N

Hello!
First of all, thanks for the great plugin. We have been using it in our Android/iOS application for quite some time, and have really enjoyed using it.
That said, I noticed that 0.7.4 is currently staged for release, and it contains a fix for Android printing in Android N. I also validated this fix on my device, so that's encouraging!
However, just wondering when it's planned to roll out/release 0.7.4? Please let me know if I can help with anything!

Also desperate to know when this is going to be released, we need it before our application can launch.
@DSchau How did you manage to validate it works, is there a way to access the development builds?

@joeyemery you can just use the Github master branch as a source, rather than something published to NPM
To do that:
cordova plugin add https://github.com/katzer/cordova-plugin-printer --save
which will save the master branch as a dep and write to config.xml.

@DSchau Thanks very much, worked wonders!

I wasn't aware about honestly that I didn't release the fixes until now. However I am planing to release v0.8 once I am done with the updates.


SqlClient diagnostics Before event should record timestamp

The following logging of the before event for SqlClient diagnostics should include a timestamp member.  This would be as simple as adding Timestamp = Stopwatch.GetTimestamp() to the anonymous type passed into DiagnosticListener.Write().
Please assign this issue to me, as I will be making the code change and submitting the pull request (I am the author of the code in question).

As recently announced in the .NET Blog, focus on new SqlClient features an improvements is moving to the new Microsoft.Data.SqlClient package. For this reason, we are moving this issue to the new repo at https://github.com/dotnet/SqlClient. We will still use https://github.com/dotnet/corefx to track issues on other providers like System.Data.Odbc and System.Data.OleDB, and general ADO.NET and .NET data access issues.

@saurabh500 Need your input for triaging.

FWIW, I think the conclusion on https://github.com/dotnet/SqlClient/issues/101#issuecomment-289138941 is that the right way to do this is to start using Activity APIs, but that would be better tracked in a new issue.
cc @vancem in case this guidance has changed.

This was a small change so we went ahead and made it. The change will be in the next preview release of Microsoft.Data.SqlClient. We have another backlog item (not on GitHub) for enhancing the Activity tracing that we will work on in the future.


Fixed links in the Landsat example in the collection-spec.

Related Issue(s): None
Proposed Changes:

The links in the landsat example were too API centric and linked with wrong rel types to an endpoint that contains data not part of the core spec. (.../items). See related Gitter conversations starting from Feb. 19 15:41 by @ycespb, @joshfix, @cholmes and me.

PR Checklist:

[x] This PR has no breaking changes.
[x] I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.
[x] API only: I have run npm run generate-all to update the generated OpenAPI files.

The CI fails due to some connectivity issue, it's not related to this PR.

I think it is confusing to have an arbitrary path prefix that isn't related to STAC in the examples, especially because it's "stac-api" which seems to imply a relationship to the STAC API spec.
IMHO it should be https://example.com/collections/landsat-8-l1 instead of https://example.com/stac-api/collections/landsat-8-l1.

Discussed this in working session, closing in favor of a new PR that uses static catalog instead of API.   The sat-api prefix as the root catalog is confusing, and there is no browse endpoint in the spec.


create test session endpoint requires JSON body

I just updated to 0.3.5 and discovered that it's now requiring that you attach a body to the request. I only got it to work if I set the body to '{}' for an empty JSON object. The documentation for the API here indicates there are no body parameters
https://inferno-framework.github.io/inferno-core/api-docs/#/Test Session/post_test_sessions
You should be able to replicate by starting up any inferno server based on 0.3.5 and make the following request.
> curl 'http://localhost/api/test_sessions?test_suite_id=g10_certification&preset_id=testing' -XPOST
{"errors":"859: unexpected token at ''"}

Whereas this one is the workaround to get it working again
> curl 'http://localhost/api/test_sessions?test_suite_id=g10_certification&preset_id=testing' -XPOST -d '{}'

I would prefer for this body requirement to be removed again as it used to be. This change broke automated inferno testing which we kick off periodically.

Thanks for bringing this to our attention. This was an unintentional change.

@nathanloyer We did a new release with this fix yesterday. If you bump inferno_core to 0.3.6 this issue should be resolved.

Thanks! will try it out when I get a chance


refactor: upgrades sdk dependency to 13.3.2

Shall we maybe include fix of that in the next release?
https://github.com/aeternity/superhero-ui/issues/1193


Allow extraneous dependencies in more files

E.g.:

*.config.js (e.g. next.config.js)
.*rc.js (e.g. .eslintrc.js)
cypress/**/*.*
src/setupTests.js
Storybook files
*.spec.js

… or just remove the rule.

At least the check for dev vs prod dependencies. We have to maintain a list of files where dev dependencies are allowed, this is quite cumbersome. It's questionable how many bugs this really prevents. Maybe just remove the check?

(from #90)


Notifications: handle membership new_member

There's a theme problem with the icon colors.  Will log under https://github.com/orgs/OdyseeTeam/projects/2/views/1


Fenrir Web Server

Couple questions
The wiki references the service being available on 33194 after install, but the rest of the troubleshooting references 31994, is this a typo?
Also I appear to be having an error
When running sudo /usr/local/bin/FenrirServer --dir /data/retronas/fenrir-ode-webserver --port 31994 --verbose
21:06:38 INFO  /tmp/webserver/src/server.c:41: Cache cd image folder
21:06:38 INFO  /tmp/webserver/src/server.c:45: start http server
2022-10-14 01:06:38 1 mongoose.c:3033:mg_open_li Failed to listen on http://<IP_ADDRESS>:31994, errno 98
2022-10-14 01:06:38 1 mongoose.c:3316:mg_listen  Failed: http://<IP_ADDRESS>:31994, errno 98
2022-10-14 01:06:38 2 mongoose.c:2490:mg_mgr_fre All connections closed
The application is accessible via the browser and shows nothing here and is able to be connected to over telnet over 31994

is this a typo?
yes thanks, fixed

Also I appear to be having an error When running ....
the webserver runs as a systemd service, it'll be running in the background as the installed service (error 98 is address already in use). you'll need to stop the systemd service before trying to run it manually

there is also no need to run the webservice as root (sudo) you should run it as your normal retronas user

The application is accessible via the browser and shows nothing here and is able to be connected to over telnet over 31994
this is correct behaviour, any other functional issues should be reported to the main project


Adding Block and String support for Unique function

Changes:

Added Block support for unique function

Added String support for unique function

Testing in terminal

Added test cases in the test file.

Doubt: There is an existing error while running test cases. I am not able to find the exact issue. It is due to recent changes as before few days it was working fine. (../rye main.rye test)

Hello, Thank you. You can merge my given code changes. I think given test error is not related to my change.


add unit tests for $in and $nin operators

Add unit tests for $in and $nin operators

Catch bug in this PR: https://github.com/langchain-ai/langchain-postgres/pull/59/files


how to set the other font family for keywords

VSCode Version: 1.19.1
OS Version: Linux

Description:
My English is not very good and something can not express clearly.
Thanks for your reading.
I've been using vscode for a long time, and I really like it, but there is a feature that I do not know how to configure it, like configuring font styles in a theme,
  "editor.tokenColorCustomizations": {
    "keywords": {
      "fontFamily": "monospace"
    }
}

like write it in setting
I think this is very useful, you can easily distinguish between keywords, but vscode does not seem to provide an implementation.
how to do it?

Reproduces without extensions: No

Currently not possible. Duplicate of https://github.com/Microsoft/vscode/issues/36512


Bump up filebased-authentication-plugin to use authorization extension v2

This would be assumed to be false by gocd. So it's not strictly needed.

This would be assumed to be false by gocd. So it's not strictly needed.

@ketan -- Yes. Right. We thought about not changing this plugin. But, this release, we will deprecate authorization extension v1 in favor of v2. So at some point in time (Before v1 is removed), we will need to change this. Hence, thought of doing it now. :)


Use clock_gettime instead of clock for benchmarks

Coverage remained the same at 100.0% when pulling 9b2ac5b6cac043667422b24313fdd972a9372577 on isaachier:use-clock-gettime-benchmark into 7756fb129691f0a1b3e6d83c228ae4c3c40c3eac on uber:master.


Example

Sorry, I'm new in gitHub, and I learning, I made it by accident
2016-10-12 14:17 GMT-05:00 alandonovan<EMAIL_ADDRESS>
@alandonovan commented on this pull request.
In README.md
https://github.com/golang/example/pull/7#pullrequestreview-3943095:

@@ -1,4 +1,4 @@
-# Go example projects
+# Go example projects1

I assume you created this pull request by accident.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/example/pull/7#pullrequestreview-3943095, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH3b8nk7LLpI85wBh7HQV30kNtal4vHDks5qzTJlgaJpZM4KR7KK
.


Dash v20 hardfork support

Coinbase transactions with new fields are now supported and ElectrumX now synchronizes well.
Also removed legacy SubTx transactions.

The changes was tested against Dash Testnet v20 (dashpay/dashd:20.0.0-beta.4) and production server deployed at the
dash-electrum-testnet.pshenmic.dev:50002
Changes can be tested by anyone via latest dash electrum client (https://github.com/Bertrand256/electrum-dash)

v20 just activated on mainnet at block 1987776 and I can confirm that this code seems to be working correctly

@sapph1re could you pls try rebasing it and see if it helps?

i can confirm that the code works
updated our DASH electrums (https://1209k.com/bitcoin-eye/ele.php?chain=dash, the cipig.net ones) with this and they are back online after they initially crashed on HF activation

Confirmed that rebasing fixes CI https://github.com/spesmilo/electrumx/pull/242

thanks


[AMD] Support FP8E5M2 with MFMA FP16 instructions

Cast dot arguments from unsupported FP8 to supported FP16 in order to use MFMA instructions instead of FMA.
This approach is expected to give better performance and be more stable compared to FMA implementation.

+cc @antiagainst @zhanglx13

@binarman Can you enable the test that is fixed by this PR?


Unable to use isX route methods in router redirect

The isActive, isCurrentRoute, and isParentRoute methods are throwing an error when using them in a GoRouter's redirect method because there isn't a GoRouterState in the supplied BuildContext.
These methods should be changed to accept a GoRouterState object directly instead of attempting to look up a state object in the context. This would allow these methods to be used inside the GoRouter configuration as well as throughout the application code. It is simple enough to acquire a GoRouterState object from within the app (using GoRouterState.of(context)) that this would not drastically alter existing uses.

This has been addressed in #21


Link in PDF not work for iOS

An HTML with link (Teste), while generating the PDF link within the PDF works only for Android. In iOS it removes the link being just text. Any solution for this?

@rogerionj Hi. Did you find any solution ?

Facing the same issue.
@rogerionj @SpawnAtis  Did you find any solution for it

Good night! Someone resolve this? I am with problem in Android, the hyperlinks don`t work


Added checks for setting up ndrange

Tow checks added for ndrange:

global_range and local_range have the same dimension.
global_range must can divide local_range evenly.
local_range should be within device allowed work item limit.

@mingjie-intel I cherry-picked your commits into #804. I am closing the PR.


Unhandled JS Exception: Can't find variable self

When I run "npm run haul start -- --platform ios", It's loading 100% and says you are ready to run. But after installing app through xcode, It's showing error as in image. I don't know what am I missing, But it's working fine in other systems with same versions I mentioned below

Haul Configuration (webpack.haul.js)
module.exports = ({ platform }, { module, resolve }) => ({
  entry: `./index.${platform}.js`,
  module: {
    ...module,
    rules: [
      {
        test: /\.json$/,
        loader: 'json-loader'
      },
      ...module.rules
    ]
  },
  resolve: {
    ...resolve,
    extensions: ['.json', ...resolve.extensions]
  }
})

Your Environment
MAC

software
version

Haul
1.0.0-beta.7

react-native
0.48.4

node
8.7.0

npm or yarn
npm

This should not be an issue in @haul-bundler/cli.


0.9 - ractive.get() options

In 0.8 ractive.get() included an options object (https://github.com/ractivejs/ractivejs.github.io/blob/dev/legacy/0.8/ractive.get().md)
Is this still the case with 0.9? I don't see a mention of it in the new docs.
In 0.8 ractive.get() would return a deep POJO clone of the data whilst ractive.get(keypath)
would return a shallow copy.

The options are still available, they just got missed in the transition. I'll get them added to the 0.9 docs in a bit.


ARM: Create Resource Group fails (bad url construction)

https://github.com/Azure/azure-sdk-for-go/blob/master/arm/resources/groups.go#L130
The CreateOrUpdatePreparer doesn't pipe "resourceGroupName" into the pathParameters as needed.

Hit enter too soon; user error.


Package update instructions.

Closes compas-dev/compas#697.

the deploy tasks fails because it's a PR from a fork and the GH_PAT is not exposed in that case, but that's alright

Disable deploy if $GITHUB_REPOSITORY != compas-dev/compas_fab?

Disable deploy if $GITHUB_REPOSITORY != compas-dev/compas_fab?

perhaps, but whatever, not a big deal ;)


Doesn't seem to work with cloudflare analytics and grafana cloud

What happened:
I'm trying to connect my grafana cloud instance with clickhouse over https.  The clickhouse instance is from cloudflare
api.cloudflare.com/client/v4/accounts/xxx/analytics_engine/sql
where xxx is the account id.

docs: https://developers.cloudflare.com/analytics/analytics-engine/grafana/

here is a screen shot

i've entered my authorization token as specified in the docs
What you expected to happen:
I expect the connection to work.  instead i see plugin health failed

How to reproduce it (as minimally and precisely as possible):

Screenshots

Anything else we need to know?:
Environment:

Grafana version: grafana cloud
Plugin version: latest
OS Grafana is installed on:
User OS & Browser:
Others:

Try splitting the values in the URL Cloudflare gives you:
The server address would be api.cloudflare.com
and then the HTTP URL Path would be the part after the host: client/v4/accounts/....analytics....
I can't recall whether it needs a leading slash or not, but try it without one first
Let me know if this works for you. The HTTP headers feature was specifically added to support this so I appreciate the feedback.

@SpencerTorres I split up the url.  I get a different error.  "connection timed out" when i save and test.
i'm using the default server port.  The cloudflare instructions don't give one, but i'm assuming their using the https ports.  I've pinged their 443 and 8443 ports and it seems to be valid.
this is my current setup

oh i opend dev tools, i do get this health check error from grafana browser console log

Good idea with the ports, I looked at the other plugin's UI and it seems like Cloudflare would be exposing 443 in this case.
In the network log screenshot you showed, the "response" tab would have more information about the error. My next guess is that this is some kind of auth or permission issue, perhaps similar to this one

@SpencerTorres
the below is for 443.  Its a ping health check failed
{
  "message": "Plugin health check failed",
  "messageId": "plugin.healthCheck",
  "statusCode": 500,
  "traceID": "e723f7888bcea5d421af5bd6e0ab30cd"
}

• GET https://askluna.grafana.net/api/datasources/uid/<redacted>/health module.ts:31 G
500 (Internal Server Error)
eval
B.r. <computed>
e. ‹computed>. e. ‹computed>
(anonymous)
(anonymous)
Re. add
(anonymous)
Ae. publishUpdate
(anonymous )
Ae. add
(anonymous)
K
callHealthCheck
testDatasource
(anonymous)
withNoBackendCache
(anonymous )
await in (anonymous) (async)
(anonymous )
fe @ module.ts:31
@ module.ts:31
@ (index):1
backend srv.ts: 131
@ ResponseQueue.ts: 32
@ ResponseQueue. ts: 37
@ FetchQueueWorker.ts: 56
@ FetchQueue. ts: 76
@ FetchQueue. ts:55
@ FetchQueue.ts:59
@ backend srv.ts: 136
@ module.ts: 31
@ DataSourceWithBackend.ts: 352
@ DataSourceWithBackend.ts: 369
@ actions.ts: 147
@ backend srv. ts: 485
@ actions.ts: 145

hooks. ts: 47
EditDataSource. tsx: 136
await in fe (async)

if i use 8443 its an invalid port as you said {"message":"invalid port","status":"ERROR"}

i'm using the free grafana cloud instance fyi https://grafana.com/get/?pg=graf&plcmt=hero-btn-1

Still investigating this using my own Cloudflare account. This is the error it's returning on the plugin backend:
Raw error:
{"@level":"error","@message":"error querying the database: clickhouse: unsupported column type \"\\n\\t\\t{\\n\\t\\t\\t\\\"name\\\": \\\"timezone()\\\",\\n\\t\\t\\t\\\"type\\\": \\\"String\\\"\\n\\t\\t}\\n\\t],\\n\\n\\t\\\"data\\\":\\n\\t[\\n\\t\\t{\\n\\t\\t\\t\\\"timezone()\\\":\"","@timestamp":"2024-02-28T01:30:18.626985-05:00"}

Manually formatted/corrected error:
{
  "@level": "error",
  "@message": "error querying the database: clickhouse: unsupported column type",
  "feild1": [
    {
      "name": "timezone()",
      "type": "String"
    }
  ],
  "field2": [
    {
      "timezone()": "",
      "@timestamp": "2024-02-28T01:30:18.626985-05:00"
    }
  ]
}

I'm also encountering this issue, and would love if there's some solution.
The alternative 3rd party Altinity plugin allows you to query data in Grafana. However it has been deprecated, and the latest version has multiple problems.

I'm not sure why it would be reporting a String as being unsupported. Maybe their server only allows number types?
I do know the plugin automatically runs some queries related to the timezone though. Here's the function it uses:
https://clickhouse.com/docs/en/sql-reference/functions/date-time-functions#timezone
I'll need to see where it runs this query, but if we can disable it, then it could work. It seems like your queries would be very limited though if it doesn't allow these other data types.


Cannot compile demos

Hi =),
I'm trying to compile the demos, but some classes are missing from the sources.
For example with the WaterBalls demo:
FixedStep
FPS
nape.phys.BodyType
nape.util.BitmapDebug
I see that BodyType and BitmapDebug classes are present in the haxe fp10 debug swf, but cannot be found on the source files,
and for FixedStep and FPS, I don't know where they come from.
I'm trying to compile with flashdevelop. Did I missed something ?
Thank you in advance for your help ! =)
Anthony

I should probably add a note to each page of where those files are.
You can find them on the github repository demo subfolder:
https://github.com/deltaluca/nape/tree/master/demo/common

Date: Sun, 29 Jan 2012 05:02:28 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: [nape] Cannot compile demos (#22)
Hi =),
I'm trying to compile the demos, but some classes are missing from the sources.
For example with the WaterBalls demo:
FixedStep
FPS
nape.phys.BodyType
nape.util.BitmapDebug
I see that BodyType and BitmapDebug classes are present in the haxe fp10 debug swf, but cannot be found on the source files,
and for FixedStep and FPS, I don't know where they come from.
I'm trying to compile with flashdevelop. Did I missed something ?
Thank you in advance for your help ! =)
Anthony

Reply to this email directly or view it on GitHub:
https://github.com/deltaluca/nape/issues/22


Incompatible with Create Diesel Generators

Crashes on start. With AA removed game launches

oh I also have that ( #3  )

I made a fork of AO which fixes this
https://github.com/souldbminersmwc/create-ao-starbound-patch

@souldbminersmwc sorry been busy with real life - I’ll eventually take a look on how you fixed it and fix it on the actual project and give you some credit - thanks!

I simply changed the Creative mode tab. That fixes it


[DIDDocument Tests] Replace toEqual() with SDK equals().toBeTruthy() for complex objects

In java tests, there are lots of things such as expect(aDID).toEqual(anotherDID). But in JS we cannot do that because there is no == operation overload for DID objects, java  can do this but not us. So, we need to replace the equality tests of complex objects such as DID, DIDURL and maybe others, with something like this, using the equals() methods we have in the SDK:
expect(aDID.equals(anotherDID)).toBeTruthy()

@rtrapp is moving to assist api v2, @jingyu is taking over this task


[$250] Migrate DatePicker/index.android.js to function component

Class Component Migration
Filenames

[ ] src/components/DatePicker/index.android.js

type: React.Component
has state values: true
has refs: true
has context: false
lifecycle methods: none

Task

We currently have some class components in our codebase that we would like to refactor to a function component.
Here's a link with some general advice on how to refactor a class component to a function component: https://react.dev/reference/react/Component#alternatives
If you need additional guidance, please ask in #expensify-open-source
Test for any regressions and verify that there are no breaking changes

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~017ff56f0b0fdb8d26
Upwork Job ID:<PHONE_NUMBER>238778368
Last Price Increase: 2023-08-16
Automatic offers:

situchan | Reviewer | 26160733

I'd like to work on this issue

I'd love to work on this.

I'd like to work on this issue

I can work on this issue.

I'd like to work on this.

I'd love to work on this

@marcaaron It's Agata from Callstack, may I work on this one?

Hello, I would like to work on this issue.

I'd like to work on the issue

it's assigned thanks

I'd like to work on this

I'd like to work on this

Hi everyone! @arthurmfgtab was assigned already here and not open for other contributors. So let's stop commenting further.

Not overdue, I'm opening a PR for this today

PR's ready, quickly reviewing it internally ;)

It looks like the PR is still being reviewed.

@situchan can you take a look? https://github.com/Expensify/App/pull/25657

@situchan can you take a look? #25657

sure, I am just assigned as reviewer in PR 2 hrs ago

Hey @situchan , have you had a chance to take a look at the PR?

I will review this after merge freeze is over, as the PR won't be merged until then.

Hey @situchan , have you had a chance to take a look at the PR?

Requested changes. Seems like you haven't even done basic testing.

What is the latest here?

Waiting for @arthurmfgtab to address feedback

Hey, sorry for the delay
I'm having issues building the app for Android, I'll update the PR as soon as this is fixed

No need regression test

Waiting to make payment on 11/01

I believe this issue is just now waiting for payment.


RearWheelRate calibration should five a negative wheel rate

To be in accordance with the coordinates that I've chosen.

I believe I fixed this. Not sure which commit though.


ci: Add an AutoReview workflow

Introduces a workflow for the purpose of validating and reviewing the code. This where I would typically include:

Static analysis
Various validations, e.g. that the composer.json is valid (the only check currently implemented in this PR)
Code linting
Any unit tests that aim at checking the codebase architecture/structure/conventions, i.e. anything that is not testing the source code behaviour.

Thanks for your PR. I think the composer.lock will not be changed in the PR. Could you check that?

@peter279k clarified and extracted the composer update in #1379

Due to the PR about updating the composer.lock file is extracted from this, I suggest the composer.lock  should be reverted in this PR.

done


Tweak function to return a value instead of asserting

Description
This looks like a very silly change. Why would we want this?
Well, when the helper function asserts, our tests are locked into the interface of the function itself. Our test must fail if this equality check fails. However, with the addition of the new experimental parser tracking event, our tracking events are non-deterministic. This is a quick change for us to be able to assert that the set of fired events must be A or B (with or without the experimental parser sampling).
Although I do generally believe in pushing assert statements to the top-level test, this very simple change isn't a long term brilliant solution to efficiently testing non-deterministic tracking.
Checklist

[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

@kwigley I think it's being flaky? It's almost always failing on partial parsing, but not the same test every time.

that is to say, there are exactly zero tests that always failed with this change through all the CI reruns I triggered last night.


fix: helmfile sync should NOT continue when secret decryption fails

Fixes #336

@sstarcher Thanks for reviewing :)


animateToRegion not working in some circumstance.

I just start using RN and I found that in some cases animateToRegion will not work when was initialRegion set.
This doesn't work:
//MainApp.js
componentWillReceiveProps(nextProps) {
      if (nextProps.newCoords) {
            this.refs.map.animateToRegion({newRegion});
      }
}

_getCustomOverlay() {
     return (someComparison) ? null : <CustomOverlay />;
}

render() {
     const customOverlay = this._getCustomOverlay();

     return (
          <MapView
               ref="map"
               initialRegion={initialRegion}
          >
               {customOverlay}
          </MapView>
     )
}

//CustomOverlay.js
render() {
    return (
        <View>
            {this.state.markers.map((marker) => <MapView.Marker {...someProps} />)}
        </View>
    )
}

I've check that the newRegion did pass to animateToRegion but it seems to be a problem in AIRMap. I'm not familiar with Android so I didn't check further in AIRMap, but I did solve this by working around:
First approuch, wrapped with setTimeout:
setTimeout(() => {
    this.refs.map.animateToRegion({newRegion});
}, 1)

setTimeout with 0 doesn't work, so I use 1. But it will take about 3~5 sec to actually trigger the animation.
Finally I found a better solution:
//MainApp.js
render() {
     return (
          <MapView
               ref="map"
               initialRegion={initialRegion}
          >
               <CustomOverlayContainer />
          </MapView>
     )
}

This move the overlay logic into a container component, and it works fine. But this makes code much uglier.
I think MapView should support <MapView>{child operations}</MapView> without side effects, please help fix this.

Does this only affect Android @BJChen990 ?

@Roconda
Sorry for the late reply.
I didn't check on iOS for I didn't setup the environment for iOS

I run to the same issue, so if i understand your solution, you put the animateToRegion inside componentWillReceiveProps of CustomOverlayContainer  right?

Great job.


maybe grab copy of cran and scrape from it locally

instead of scraping via web requests - maybe too much complexity
https://cran.r-project.org/mirror-howto.html

All the data for the check results are stored in rds files available from CRAN. There are 2 main files one with just the summary of the checks (https://cran.r-project.org/web/checks/check_results.rds), and another one with the details of the all the check results (https://cran.r-project.org/web/checks/check_details.rds). I have (unexported) functions in foghorn to download and read them: https://github.com/fmichonneau/foghorn/blob/master/R/cran_files.R
So if you can have a cron job that download these files regularly, and a way to make the data available to the API, then there is no need to rely on scrapping the website. One of the reasons I had opted for  plumber (https://github.com/fmichonneau/cranstatus/) was to be able to serve this data directly.

Thanks @fmichonneau - thx for letting me know.  I didn't know about the rds files. Though i guess that means I would have to include R in the stack which there isn't right now. Hmmm.  The scraping isn't ideal, but i do only do it once a day. How often are the rds files updated?  I had assumed they only ran checks once a day?

my understanding is that it's updated each time a package check completes, so probably many times a day. yeah you'd have to include R (unless there is a way to extract data from the rds files outside of R?)


dra-cla: unstable-2024-02-07 -> 0-unstable-2024-06-07

Description of changes
Change Log: https://github.com/CoolnsX/dra-cla/commits/main/
Mainly more providers (links) were added

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Hi @jian-lin
Just a request on merge please. Its nothing much as a bash script.

Sorry, I do not have the bandwidth to review this.  I have posted this PR in #review-requests:nixos.org Matrix room so that it has a better chance to be reviewed.
BTW, I posted ofborg eval above because ofborg-eval was broken then and I fixed that.

Hi! No worries!
I did not know matrix also has review channel, thought only discourse was the place.
I will make sure to join matrix channel too!


Catch the 'action' event

Maybe this fixes #78?
In any event (ha!), it makes the code cleaner and follows the API better.

I can't recall, but I think when I first wrote this code the underlying IRC lib didn't actually have this event.

Yeah, I checked the diffs and so much got reorganized that I can't really tell if it was there but poorly documented or just not there at all. Either way, here it is!


Discuss TreeLDR: Tree Linked Data Representation

Problem: it's hard for developers to work with JSONSchema and JSON-LD contexts.
Solution: Introduce new record syntax called TreeLDR (Tree Linked Data Representation) that can succinctly express JSON-LD contexts, CBOR-LD contexts, RDFa contexts, JSONSchema, and other reprs on schema.org. 300 lines of hard-to-manipulate JSON becomes 50 lines of familiar struct definitions as you would encounter in Rust, C++, etc.
The challenge is that between LD contexts and JSONSchema there is a strange relationship that's not as simple as "LD contexts are a strict superset of JSONSchema". Therefore, TreeLDR will in some cases be one-way compilation to the schema representation of choice, and if you go the other way, it may produce a proper subset of the original TreeLDR.
We can imagine this DSL (Is it technically a DSL? I guess all grammars make languages, even if not turing-complete programming ones?) to further generate libraries across languages and platforms. Maybe you can use this to create a Python SDK that uses DIDKit under the hood, dealing with your specific credential types.
Similar efforts in other places to draw inspiration:

JHipster's JDL
Protobuf and other wire protocol specifications
CDDL

Example of what it could look like:
use schema_org::URL;
use custom::OtherRecord;

BasicProfile :: Record {
    name :: String required
    description :: String required
    logo :: URL required schemes("http"|"https")
    website :: URL required schemes("http"|"https")
}

AdvancedProfile :: {BasicProfile, OtherRecord} {
    age :: Number required min(0) max(99)
}

type Profile = BasicProfile | AdvancedProfile

Open questions/observations:

How much type system do we want? I think ADTs are a good thing to have here, like sum types in Haskell. Product types should be naturally expressed as records instead of lists to avoid that Haskell pothole. Maybe Rust is a better source of inspiration here. What about traits/type classes?
We fortunately have a wonderful testbed for this: anything on schema.org should have a sane representation if it is straightforward there--and we should make the hard things easier.

Seems like there might be some overlap with this and IPLD schemas: https://ipld.io/docs/schemas/intro/

If I understand right, the goal is to design a strongly typed schema definition language that would compile (or transpile) into various less user-friendly languages (LD context, JSONSchema, etc.). What stands out to me in the above example as a unique feature, that is not present with other mentioned user-friendly schema definition languages, are the refinement types.
I'm less sure to fully grasp the "create a Python SDK that uses DIDKit under the hood" part. Correct me if I'm wrong, but that would mean:

Transpiling the TreeLDR schema definition into the target programming language (Python here) type definitions.
Generating some code in the target PL that uses those type definitions in combination with DIDKit. Even reading the DIDKit examples I don't have a precise idea of common usage patterns, so that's the part that remains a bit blurry for me. A concrete example of the complete flow from TreeLDR to the generated code could help here.

TreeLDR now has its own repo: spruceid/treeldr.


URL connection instructions on nativeplanet.io/software

Hello, just wanted to note --
I installed groundseg on a droplet and use remote desktop to connect to ubuntu desktop - connecting to either nativeplanet.local or http://<IP_ADDRESS>:27016 returned a 404, but connecting to http://<IP_ADDRESS> itself did work.
Cheers

Hi @jmahhh -- GroundSeg is not meant to be installed on a remote server, it's meant to run on a device on your local network.
I'm unsure where the http://<IP_ADDRESS>:27016/ URL came from, but the nativeplanet.local address 1) relies on the device being on your local network, and 2) uses your device's hostname -- so if your computer's name is johncomputer, you would go to johncomputer.local. For devices with ColonyOS installed, the hostname is nativeplanet.

:27016 is the API port


CVE-2020-35728 - mapping to purls more than NVD described

As described in NVD:

FasterXML jackson-databind 2.x before <IP_ADDRESS> mishandles the interaction ...

The current mapping includes later versions:
    "purls": [
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>    ]

The trouble with this one is that GHSA lists a non-existing Maven coordinate: com.fasterxml.jackson:jackson-databind. The proper one is com.fasterxml.jackson.core:jackson-databind, which we list. However, on our end the versions are not correctly limited to vulnerable range, and also the first_patched_purl is using the bad coordinate from GHSA<EMAIL_ADDRESS>

The story is complicated, but it seems that things are working as intended for the most part.
Because GHSA names an incorrect Maven coordinate, we fall back to previous data on the CPE cpe:2.3:a:fasterxml:jackson-databind, which is correctly found to be the purl pkg:maven/com.fasterxml.jackson.core/jackson-databind.
At that stage however, the version range of GHSA is also not used anymore to identify the vulnerable versions. Instead, the code checks here for all versions of the Maven package: https://repo1.maven.org/maven2/com/fasterxml/jackson/core/jackson-databind/
which is then compared to the patch date to infer which version are vulnerable. The patch date is correctly found to be 2020-12-26 as can be seen here: https://github.com/FasterXML/jackson-databind/issues/2999. All version before are assumed vulnerable, which is plausible IMO.
Checking back at Maven https://repo1.maven.org/maven2/com/fasterxml/jackson/core/jackson-databind/ results in the list of versions listed in above, so that is working as intended.
I wonder why the version range is set differently in both the NVD and the GHSA data. Perhaps it's related to responsible disclosure?
There is one bug here, and that is the first_patched_purl in our data points to the wrong Maven coordinate<EMAIL_ADDRESS>because it has yanked that from the GHSA data without further validation.


Adding forex daily support.

This change adds support for https://www.alphavantage.co/documentation/#fx-daily.

Coverage decreased (-76.4%) to 18.243% when pulling b474cd0f885b4934ab0b12b8faa0909b09c521c2 on ragnarpa:add-forex-daily-support into 1aaed0e2e1eeea1a7c1d376993bc08cb0b79f124 on zackurben:master.

Thanks for your contribution @ragnarpa! Give me a few days to get things in order and get this tested.

@ragnarpa this was added with https://github.com/zackurben/alphavantage/pull/85 and will be included in the next version 2.1.0 which I'll be publishing soon! Thanks for the contribution.


[T3A3][T10-A2] Hu JingRui

Ready for review

Some comments added. Please close the PR after reading comments.


Do not trim space seed phrase, private key

Describe the bug
Do not trim space seed phrase, private key
Expected: Trimspace

Version test: v1.0.2(151)
Devices test: IP 11, MI 8 Lite
Validation passed.


Fix calendar in firefox

Updated calculation to fix the cash crop window not shown in firefox problem.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.


Make state info output text box non-interactive

its purely a minor annoyance and not anything of value, but without this flag set I accidentally typed over my own seed value a few times. this should change nothing but to make that state output text book non-writable

Good call, thanks


The modal instance is not removed when the component is destroyed

Versions

vue-final-modal: 0.17.1

Steps to reproduce
DEMO（Codepen）：https://codepen.io/mini_ghost/pen/vYKRoOv

In the demo.（please use "Debug mode" or "Full Page View"）
Close modal and change router to Page B, then Backe to Page A and repeat several times.
Then open devtool Console.

What is Expected?
When user switch to Page B, modal's instance at Page A should be removed.
What is actually happening?
The modal instance is not removed when the vue-final-modal component is destroyed

@Mini-ghost
Thanks for your feedback.
The bugfix version is published to

v0.17.2 for Vue 2
v1.4.2 for Vue 3


AWS Athena support

What do you use this database for?
I have a small hobbyist data project, and for that Athena is incredibly cost-effective.
Are there any popular and well-maintained clients for this database?
The official connector is @aws-sdk/client-athena, there's also athena-express which hides some of the clunkiness of the official one.
Are you interested in contributing this database connector?
Draft PR incoming

Hello -looping back on connector requests
We have recently re-architected Evidence (since this issue was opened), and Evidence is now "pluggable", meaning you can write your own connectors, publish them, and use them in Evidence.
If you are interesting in writing a connector, here are some resources:

Docs on creating a plugin: https://docs.evidence.dev/plugins/creating-a-plugin/datasources
Template for a connector plugin: https://github.com/evidence-dev/datasource-template
An example that I spun up for pulling stats from the npm API: https://github.com/archiewood/npm-stats

If you do write a plugin:

the suggested package name is evidence-connector-[plugin]
the docs are new, so let us know if there is extra information you need
share it on the Evidence community, and we'll feature it!

@morganchorlton3 has just published an Athena plugin here:
Repo: https://github.com/morganchorlton3/evidence-connector-aws-athena
NPM: https://www.npmjs.com/package/evidence-connector-aws-athena


select should accept [Tag]

I'd propose to give a following example of using Haquery in ghci, which is one of the common usages of fetching a website and parsing it.
import Haquery (select, parseHtml)
import Network.Wreq (get, responseBody)
import Control.Lens (^.)
import Data.Text (pack)

r <- get "http://nixos.org/channels/"
Prelude.concatMap (select (pack "a")) $ parseHtml $ pack $ show (r ^. responseBody)

Which motivates me to say that select should accept a list of Tags, since that's what parseHtml returns.
PS: I'm fairly new to Haskell ecosystem, so excuse my ignorance :)

Another strong reason is that the result can be then chained to subsequent select calls.

Yes, I think I'd also like this change. Any reason to the contrary?

Just occured to me that we could also define
selects sel = concat . map (select sel)

@Wizek I will implement that


Add Preview button to set of permitted buttons for navigating away

This fixes #2602 for me in FF 46 and Chrome 50.
Due to poor git skills I have also tangled up a fix for 2604, sorry for any confusion.

That did also occur to me :)

OK @gasman I have revised this to use ignoredButtonsSelector as recommended.

Unfortunately neither variant of the fix seems to solve the issue for me on FF 46 (Mac)... I think the problem is that the page reload (which is not so much a reload as a rewrite-in-place - https://github.com/torchbox/wagtail/blob/master/wagtail/wagtailadmin/static_src/wagtailadmin/js/page-editor.js#L495-L497) only happens after it's got an AJAX response back from the server, and at that point the browser no longer considers the preview button to be the thing that initiated the reload event. The proper way to fix this is probably to call disableDirtyFormCheck at the point the reload happens - will give that a go.
Rather worryingly, preview seems to be entirely broken on my IE11 / Win7 test VM - I suspect this may be because they've removed 'MSIE' from the user agent string (https://github.com/torchbox/wagtail/blob/master/wagtail/wagtailadmin/static_src/wagtailadmin/js/page-editor.js#L455)...

The disableDirtyFormCheck approach seems to do the job - closing this in favour of #2614.

Funnily enough I was on the way to reporting the IE11 broken preview bug once I'd verified it. That MSIE check should probably also check for Trident...

As it happens I managed to find a solution for the IE11 issue that didn't rely on user-agent sniffing - see #2613.


Update pull request template to be compatible with other repositories in a project

Update pull request template to be compatible with other repositories in a project

Fixed in https://github.com/RedHatInsights/insights-results-aggregator-mock/pull/33


plugin did not pass the from option to postcss.parse

I'm using this plugin with vite. The following warning is printed in console:
A PostCSS plugin did not pass the `from` option to `postcss.parse`. This may cause imported assets to be incorrectly transformed. If you've recently added a PostCSS plugin that raised this warning, please contact the package author to fix the issue.

postcss v8.4.23
flex-gap-polyfill v4.1.2

Thanks for sharing this. This plugin does not use the method postcss.parse so I'm not sure what's causing this warning but I will investigate.
Is it preventing you from doing anything?

hi, I met the same problem using Vite. This warning will stuck the build process, so the plugin will not work.

Thanks for investigating this. But you should mark #89 a duplicate of this issue instead.


Update FileZilla.download.recipe

Change identifier so that Finder does not see file.download as a in-progress download file from Safari. Most other repos follow this standard where its com.github.user.download.APPNAME.
I'd recommend doing the same for all other download recipes you may have.

@bp88 Changing identifiers breaks any overrides users may have for these recipes, such that the overrides will no longer be able to find their parent recipes (those which they have overridden). So, it's usually strongly discouraged once a recipe has been out in the wild.
Some more info here:
https://github.com/autopkg/autopkg/wiki/Recipe-Writing-Guidelines#identifiers

@tsutton that's a good point. But then I'd point back to the same documentation that recommends following a certain naming format which the recipes in this repo aren't following. It's not like folks aren't going to notice if their override isn't working. They'll be able to fix it.

Maybe you know how to fix it, but there are plenty of users (possibly the majority) who don't understand that child recipes are linked to their parents using identifiers.

This is not a tool for an average joe. I won't argue that. But why would you expect people to be able to use this tool, create their own overrides, but not be able to understand what parent/child identifier relationships are? If you're using this tool and making overrides then clearly you have an understanding of how this tool works.
Plus, that's why the documentation is there in case you need to troubleshoot. I JUST started using this tool a few days ago and I understand the parent child relationships with identifiers because I read the documentation. In other words, you guys are quite clear in your documentation and how things work with recipes and how they should be named.
All I'm asking is that this and all the other recipes in this repo be updated to follow the proper naming convention. Short term pain, but long term it follows the proper standard naming convention for identifiers (which are obviously recommended for a reason). Otherwise there's no incentive to follow that naming convention at all and it sets bad precedent that others may follow. In fact, I would argue that repos accepted to autopkg should be following the standard naming convention before they are linked/accepted as searchable autopkg repos. It's a bit surprising to me that @keeleysam is one of the people overseeing the repos on Autopkg, but not following recommendations in the documentation.

Closing as existing identifiers won't be changed as it will break existing users.


[secp256k1-zkp] Bindings for ecdh module methods

We need to add the bindings for the native C functions of the ecdh module listed below:

secp256k1_ecdh

We could reuse the one implemented in this repo.
For testing, we should use the vectors from the js version.

@altafan I think we can't reuse code from this repo as it looks to me that it is referencing different C code.
I think the example that you recommended to look up to is using C lib from this repo and in our project, we are referencing Elements C lib.
PSB difference in a method signature:

Method secp256k1_ecdh of the Elements C lib is receiving 6 params
Method secp256k1_ecdh of the example you recommended to be used, is receiving 4 params

I checked if anybody else in Github referenced method secp256k1_ecdh with 6 params but search returned only calls with 4 params.
What do you think, am I missing something?

You're absolutely right, but we can still reuse that implementation since we actually won't set the last to args of the secp256k1_ecdh function. You can see that in the js version we force them to be NULL.


0xfe05fae1dceafd4eab8e45fde1a87956876e8d7f

Related Safe Addresses
0xd0c70d4c4b1e931a632ed1709d1f8f9bc5736f37
0x528d8b327c32869cbee50d8a3fa7f97691452140
0x1582d1b26b3c8e6c9c503546e961ec57b6ba9638
0x4ed0b17fce840bce97307b3595fc738a113ae2c8
0xb65f45c3746107f614c2365d86fe264ab2ca24aa
0xee52349da59c55763bc5ec3c2e7cf75e31fee847
0x75141d20c19de322477954e144f834720457cbc5
0xfe103025fc043f9137f48760d37842b614f2c7f3
0x34b3701b9ea857ce2a4ab5eda586aefc0d668ab3
0x56245599befd89b88b3ab466022f2f680167dc1c
Reasoning
These safe accounts are all created by 0xfe05fae1dceafd4eab8e45fde1a87956876e8d7f
All 10 accounts were created very close to each other ，and after the account is created, all interactions are done using Capsule Coin (CAPS) to transfer 10 tokens, and all accounts operate exactly the same
like this
https://etherscan.io/tx/0x72a125947ac1ac28d840821aa6f587edc2743d7212ef24d4337413e399a4f7a7
https://etherscan.io/tx/0x1fca46e4cf0d4d3d47acf1f41a4da13266e99c496f856ed679ffcde89332122f
Methodology
Luabase was used for preliminary data removal, and then all TX were checked one by one by EtherScan
My Safe Address
0xf4ed1EF1f7A52DC7Dd10a9B5d43f23748A1E8992

These safe accounts are all created by 0xfe05fae1dceafd4eab8e45fde1a87956876e8d7f

Creating multiple Safes from 1 account is not airdrop farming per se, pls elaborate.
The first account stores >400k USDC. Safe as store of value is legit usage. https://etherscan.io/address/0xd0c70d4c4b1e931a632ed1709d1f8f9bc5736f37

and all accounts operate exactly the same

Please elaborate on how this is airdrop farming an how exactly they operate.

We haven't received any further information on this ticket and therefore deemed this submission invalid.
Unfortunately just the pure fact that Safes...

were created by the same address
have the same owner
conducted the same transactions
received assets from the same address

... is not enough reasoning for a valid submission, as there are legit user behaviour leading to such as well (relayer, scripts, power users, exchanges)
This submission didn't provide sufficient indicators to convince the team that there is some targeted airdrop farming behind the provided addresses. As per the rules, when in doubt we will default in favor of the airdrop recipients in order to not risk any false positives when sorting out airdrop hunters.


refactor(Logic/Unique): remove dependency on Fin

Unique should be available earlier than results about Fin n, by virtue of being a very basic logic typeclass with no dependencies on Nat or algebra.
This also generalizes some of the moved instances and lemmas.
There are no new declarations in this PR, only renames of existing ones.

bors merge