gotoPage and changePage results in error

I'm trying to change the page programmatically. I was expecting to see a page property in the api but I couldn't find any, so I'm trying to use either the gotoPage or changePage api methods.
My code is as follows:
const page = 2;
this.$refs.vuetable.gotoPage(page);

This is resulting in an error: Vuetable.vue?0c6a:738 Uncaught TypeError: Cannot read property 'last_page' of null both with gotoPage or changePage.
Am I missing something?

@emepyc This is probably about your JSON data structure returned from the API that does not conform to what Vuetable expected. It might not have the last_page field. Check that first and see if that's the problem. If not, post your sample data structure here and I'll have a look at it.

@ratiw I seem to have the same issue. I'm trying to go to certain page in mounted hook, and I get TypeError: Cannot read property 'last_page' of null
Is it possible to make pagination go to some page after initialization?

Having the same issue.  I want to load a certain page on mounted

Ok, I see. I'll add initial-page prop which will be default to 1. Will probably be in the develop branch first though, then release in v1.7.
@emepyc Sorry, didn't clearly understand what you mean.

@ratiw Thanks for your great work.


为什么只对regression head进行transformer操作？如果对CenterHead也进行transformer操作效果会提升吗？

The motivation of this work is to use the center prediction as the initial query for the detection transformer. I believe applying the transformer to the center head can potentially improve the center classification performance. But it is out of our scope in this work.

Thanks!


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_4cb692iidd/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_4cb692iidd/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_4cb692iidd/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_4cb692iidd/clone/recipe/meta.yaml:48: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_4cb692iidd/clone/recipe/meta.yaml:48: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


TarGene paper

To incorporate in the manuscript:

[ ] Write Hair color analysis
[ ] Write Deprivation analysis
[ ] Generate supplemetary data


Not working in 3.4.4

When selecting any items (folder, text, symbols) either in groups or directly, the plugin doesn't appear to do anything.
Sketch 3.4.4
OS X 10.11.12

Agreed it seems to be not working.

Identical issues as Miguel described.
Sketch 3.4.3
OSX 10.10.2

Same issue here with Sketch 3.4.4 on OsX 10.10.5 and plugin 1.1

Same here — Sketch Version 3.4.4 (17249) / OS X 10.11.1 (15B42)


Update INSTALLATION.md apt-key fingerprint BE2F475F

$ apt-key fingerprint BE2F475F
pub   4096R/BE2F475F 2018-02-28
Key fingerprint = F443 EDD0 4A58 7E8B F645  9C40 CF68 F84A BE2F 475F
uid                  opsx-admin<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
fixes #1336
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #1339 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1339   +/-   ##
=======================================
  Coverage   17.32%   17.32%
=======================================
  Files         189      189
  Lines       11818    11818
=======================================
  Hits         2048     2048
  Misses       9623     9623
  Partials      147      147

ping @Letty5411 @fuweid
Could you help to take a review for this? Thanks a lot.

Hi @rhinoceros , could you please use git commit --amend -s and repush your change? since the code conduct requires the contributor has to add the email in the git commit. Thanks

since the code conduct requires the contributor has to add the email in the git commit.

With commit email, it would be added in file https://github.com/alibaba/pouch/blob/master/CONTRIBUTORS I think. @rhinoceros

LGTM


Welcome Tour: Add triggers to tour system

This issue is part of the welcome tour feature and should be implemented behind a config flag (not labs) to start with. An example of how to do this exists in this PR.

Description:
Further to getting the positioning and interactions working for the icon and tip in issue #5169, we also need to wire up the triggers that will cause the icons to show at the right time. These triggers may be based on user interactions, time delays, criteria being met or a combination of all of them.
Spec:
As yet there is no definite spec for what triggers will look like, rather some loose ideas of the kind of events we want to base tour items on. The triggers are likely to be based on a criteria that needs to be met as well as events.
Some items may appear at the same time (e.g an initial set that appear once the user is logged in) and others may happen after another tour item is closed, after a certain amount of time has passed, or after the user completes an action like publishing a post.
Some example rules:

After the user logs in, their user data is loaded (session.user is ready) and they visit the content screen, the content tour item shown in the image above should be present. It should remain present in that location until it is closed/marked as seen.
Show a tour item on the help menu X amount of time after the user was created and after the initial tour item(s) are marked as seen
Show a tour item on the post setting menu X minutes after the user enters the editor screen for the first time. It should appear immediately every subsequent time the user visits the editor until it is dismissed.

Recording the information for how to trigger each item may be quite tricky. It may be necessary to hardwire in some triggers to specific interactions using sendAction calls. That's perfectly fine :)
It may also be necessary to persist whether or not an item has been triggered yet. The won't be properly possible without # so it's fine if the trigger state doesn't persist with this issue.
The result of this issue should be the implementation of a couple of example triggers for the two demo items with at least one of them having complex rules to show how it can be done.

Closed via https://github.com/TryGhost/Ghost-Admin/pull/527


Add the option to skip VR verification when writing elements

Add an Options struct to the write package, containing an option to skip VR verification when writing DICOM elements.

Thank you for the directions! I was considering a variadic option variable, but I had not though of using functions. I'm closing this pull request and submitting a new one for my attempt at an implementation using variadic options.

Pull request for my new attempt: https://github.com/suyashkumar/dicom/pull/49


[NEW ALGORITHM]  I would like to enhance String Algorithms by adding the following new Algorithms i.e., Apostolico–Giancarlo Algorithm, BNDM Algorithm, Bitap Algorithm , Commentz-Walter Algorithm, Shift-Or Algorithm

Description:
I would like to enhance String Algorithms by adding the following Algorithms:
1)Apostolico–Giancarlo Algorithm
Description: An extension of the Boyer-Moore algorithm optimized for multiple pattern matching, allowing efficient backtracking.
Time Complexity: O(n + m)
2)BNDM (Backward Nondeterministic Dawg Matching)
Description: A bitwise pattern matching algorithm that is an improvement over the BMH algorithm, especially effective for short patterns.
Time Complexity: O(nm)
3)Bitap Algorithm (Approximate String Matching)
Description: Uses bitwise operations to perform approximate pattern matching, allowing a specified number of errors (insertions, deletions, or substitutions).
Time Complexity: O(nm)
4)Commentz-Walter Algorithm
Description: A variation of the Boyer-Moore algorithm that improves efficiency when searching for multiple patterns simultaneously.
Time Complexity: O(n + m)
5)Shift-Or Algorithm
Description: A bitwise algorithm that matches patterns by shifting a bitmask, making it effective for small alphabets.
Time Complexity: O(n)
Checklist:

[✓ ] Contributor in GSSoC-ext
[✓ ] Want to work on it

I would love to enhance String Algorithms by adding these algorithms .Please assign me this issue

@pratheekv39 You are trust worthy contributor.


GetOpponentPlayer() method  is not implemented

I am trying To prevent the bot from making a capture that would lead to a disadvantaged exchange, such as using a high-value piece like a queen to capture a lower-value piece like a pawn that is defended by a higher-value piece like a bishop.
I am using the GetOpponentPlayer() method to do this but am getting the error below at runtime. maybe am using the wrong method, if anyone knows how to implement this I will be glad
An error occurred while bot was thinking.
System.NotImplementedException: The method or operation is not implemented.
at ChessChallenge.API.Board.GetOpponentPlayer() in /home/kabrownie/projects/Chess-Challenge/Chess-Challenge/src/API/Board.cs:line 342
at ChessChallenge.Application.MyBot.Think(Board board, Timer timer) in /home/kabrownie/projects/Chess-Challenge/Chess-Challenge/src/My Bot/MyBot.cs:line 50
at ChessChallenge.Application.ChallengeController.GetBotMove() in /home/kabrownie/projects/Chess-Challenge/Chess-Challenge/src/Framework/Application/Core/ChallengeController.cs:line 151
Illegal move: Null in position: rnbqkbnr/pppppppp/8/8/4P3/8/PPPP1PPP/RNBQKBNR b KQkq e3 0 1

Where did you find the "GetOpponentPlayer" function? I don't see it in any of the docs, or the source code.
The way I do this is using the boolean board.SquareIsAttackedByOpponent(targetSquare) to see if the location I am thinking of capturing is defended by the opponent. If it is, you can compare the value of the pieces to see if it is a favorable trade.

thanks


Add a TaskRun and PipelineRun status tests and add test builders 💃

This is a follow-up of #285 to show the fix (aka this new
status_test.go tests fails if you revert that commit)
This also adds a start of test builders to ease the creation of
test objects.

A quick note of the builders : while writing those tests (and other unit tests for other PRs) I found we may tend to write too much code to create those test object. I'm quoting myself (sorry :sweat_smile:)

One of the most important characteristic of a unit test (and any type of test really) is readability. This means it should be easy to read but most importantly it should clearly show the intent of the test. The setup (and cleanup) of the tests should be as small as possible to avoid the noise.

Those builders are here to help (and if you think they don't, then we don't need them :wink:) :

it makes it easier to create object (less noise)
the intent is clearer — by providing sane default in those, it should make it easier to see what are the main characteristics of those object we're using to test
they are extensible and re-usable easily (thanks to functions :stuck_out_tongue_closed_eyes:). You can even use inline functions for specific needs.

taskRun := TaskRun("apple", namespace, TaskRunSpec("banana"), func(t *v1alpha1.TaskRun) {
	Status: v1alpha.TaskRunStatus {
		PodName: "foo",
		// […]
	}
})

As of now, they live in the test package and are only used on the new test. If we think they are useful, I'll update the other tests and add more useful functions :wink:
/cc @bobcatfish @shashwathi @pivotal-nader-ziada
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

/lgtm

/approve

/retest


While refactoring parse only one java file which has step implementation.

Is this resolved ? I think it needs to be closed.

Fixed in commit


The column name 'Key' is a MySQL reserved keyword.

Fix this in AppsInfo.

Also From and To colums in GraphsHighlight.


Give lint groups have a lower default default priority

Problem
Currently, giving a level to a lint group (like pedantic in clippy) will override a level given to a specific lint if it's higher than the lint alphabetically. For example, pedantic = "deny" will override ptr_as_ptr = "allow" but not option_option = "allow".
Proposed Solution
Give the lint groups a default priority of -1.
Notes
No response

Not changing the priority but changing our sorting within a priority was a future possibility from the RFC.
The first step would be #12918.

Ah, I didn't know about the RFC - thanks. My suggestion is probably redundant then.

Re-opening so we have this as a place to discuss this if/when we get to that point.


Consider integrating protobuf-aware treeshaking in TFA directly instead of running two rounds of TFA

We should consider integrating protobuf-aware treeshaking in TFA directly instead of running two rounds of TFA to reduce the AOT compilation time.
It would make TFA less self-contained, but it would probably save a lot in compile-time.
/cc @mraleph @alexmarkov

There is another interesting thing about the integration: currently TFA does not track types through protobuf implementation well enough because of the reflective nature of the underlying GeneratedMessage implementation. This means we loose concrete type information about members that returns interface types like List<...>. We should consider finding a way to extend TFA with a concrete knowledge of GeneratedMessage implementation so that it could infer concrete type information in this cases.

We should get the ball rolling on designing this integration, because it would help us to move the needle on compilation times internally.
We had a brief chat with @alexmarkov on how this integration could look like few weeks ago - one idea that got brought up is that we could introduce some @pragma(...) or intrinsics that get transformed in a special way by TFA based on reachability of other members.
Then protobuf treeshaker just does a transformation of the Kernel which injects these constructs in the right place in the protobuf code and then TFA is run once afterwards.


is there any way to custom css off dropdown options. ?

What kind of customization do you want?

for example:   use React-select


[BUG]:    ubuntu 20      New Workspace  Workspace Name aaaaaaaaaaa Error: Invalid prisma.workspaces.create() invocation: Unable to require(/tmp/.mount_Anythi2L1mFJ/resources/backend/node_modules/@prisma/engines/libquery_engine-debian-openssl-3.0.x.so.node). Prisma cannot find the required libssl system library in your system. Please install openssl-1.1.x and try again. Details: libssl.so.3: cannot open shared object file: No such file or directory

How are you running AnythingLLM?
Docker (local)
What happened?
when tryin to create new workspace:   got this error
New Workspace
Workspace Name
aaaaaaaaaaa
Error: Invalid prisma.workspaces.create() invocation: Unable to require(/tmp/.mount_Anythi2L1mFJ/resources/backend/node_modules/@prisma/engines/libquery_engine-debian-openssl-3.0.x.so.node). Prisma cannot find the required libssl system library in your system. Please install openssl-1.1.x and try again. Details: libssl.so.3: cannot open shared object file: No such file or directory
Are there known steps to reproduce?
in Gui try to create new workspace

not running on docker,   in a ubuntu linux machine

Duplicate of #1077 - will be resolved on that issue.


Support https://schema.org/Action as event envelope

Cloud message streams is a primary interface for semi-structured data to ensure durable and recoverable writes. Persistent streams are build over AWS Kinesis, AWS FireHose and AWS S3. It treats data as a collection of immutable facts, which are queried and processed in real-time. AWS Kinesis is a core technology here that implements a distributed, durable commit log. On the top, we have an stream processing application that deploys whatever business logic associated with it. Stream processing application replicates a commit log to a storage, where it materializes a view. To compare with a traditional database techniques, we can achieve a simplicity, better robustness and flexibility on data processing.
Cloud Stream is designed to process logical log of events, each event defines a change to current state of the object, i.e. which attributes were inserted, updated or deleted (a kind of diff). The event identifies the object that was changes using unique identifier.
The stream is the source for all your writes as immutable events (facts). Each write is just an immutable event that you can append to the end of the transaction log. The transaction log is a really simple, append-only data structure. It became a first-class citizen in the data architecture. When you do this, you don't need to necessarily make your writes through a leader — you could also imagine directly appending your writes to the log. The usage of leader becomes necessary if you want to validate that writes meet certain constraints before writing them to the log.
A stream consumers builds a materialized views from the writes in the event log. These views are just like secondary indexes: data structures that are derived from the data in the log, and optimized for fast reading. Materialized view is just a cached subset of the log, and you could rebuild it from the log at any time. There could be many different materialized views onto the same data: a key-value store, a full-text search index, a graph index, an analytics system, and so on.
Cloud Stream protocol version 2 uses concept of Action defined by schema.org to describe immutable facts placed into the log.

An action performed by a direct agent and indirect participants upon a direct object.

The Action is an JSON object with few mandatory fields:

type is a canonical URI that defines a type of action.
object The JSON object upon which the action is carried out. The protocol applies a constrain to the object. It is mandatory to supply a type and id.

An optionally, it contains

agent is direct performer of the action, a software service that puts action to the stream.
participant is indirect participants, a user who initiated an action.
created ISO8601 timestamps when action has been created.

Example Action types

http://schema.org/UpdateAction - the act of managing by changing/editing the state of the object
http://schema.org/DeleteAction - the act of of editing a recipient by removing one of its objects.

events module support it


Missing Admission Controller DefaultStorageClass

Tectonic Version
1.5.4-tectonic.1
Environment
AWS
Expected Behavior
DefaultStorageClass to be enabled after install
Actual Behavior
Its not
Reproduction Steps

Start cluster using tectonic

Other Information
I've now added it to the params in the daemonset for the api server, now trying to workout how i can restart the api server

Hi @mcfedr, thanks for filing this issue. Pulling in @kbrwn here, who will get back to you in a few days. He'll pull in others from our team as necessary.

@mcfedr
You can edit and restart the api-server pod with the following commands:
kubectl get daemonset/kube-apiserver --namespace=kube-system -oyaml > api.yaml
<edit api.yaml with your favorite editor>
kubectl apply -f api.yaml

Just FYI, automatic/push button updates to the cluster are no longer supported if you make modifications to the kube-apiserver.
It looks like dynamic provisioning and storages classes is no longer beta in 1.6. I believe this means  DefaultStorageClass should be supported without any modification of the api-server.


feat!: update compute_note_hash_and_nullifier compiler check

Fixes https://github.com/AztecProtocol/aztec-packages/issues/3669
It's desirable to have nicely typed args in our noir contracts and for this reason I want to change signature of compute_note_hash_and_nullifier from:
compute_note_hash_and_nullifier(Field,Field,Field,[Field; N]) -> [Field; 4]
to:
compute_note_hash_and_nullifier(AztecAddress,Field,Field,[Field; N]) -> [Field; 4]
This PR updates the corresponding compiler check so that this can be achieved.

Why not make this change inside aztec-packages?

@TomAFrench TBH didn't know that you can send a PR to submodule. Is that what you meant? Will give it a try if yes.

Yeah, you can make PRs in Aztec packages with changes in the noir directory. If you're making changes to just Noir it's best to do it here but seeing as this change is only relevant to Aztec then I think it makes sense to do over there (so Aztec CI runs on the PR)


Test setup changes related to issue 381, plus a doc change regarding shiftfs installation.

Travis CI is currently broken, but local CI passed. Will commit and deal with TravisCI soon.


Can pull Helm chart app images from cluster registry or custom registry

https://github.com/rancher/dashboard/issues/6910
To test:

Provision two downstream RKE2 clusters, one with a private registry in the cluster config and one without.
In the cluster that has a registry, install a helm chart with the default settings. (I used Rancher alerting drivers) Confirm that the cluster registry from the cluster config is used in the values.yaml under values.global.cattle.systemDefaultRegistry.
In the cluster that does not have a registry, install a helm chart with the default settings. The values.global.cattle.systemDefaultRegistry should be an empty string because the global default registry in the settings is also an empty string.
In the global settings of Rancher, change system-default-registry to a different string. It can be any string.
In the cluster that does not have a registry, install or upgrade a chart and confirm that the systemDefaultRegistry now reflects the new default registry from the global settings. Note: This is where I got stuck - the new default setting is not sent in the network request.

I have not tested the following yet, but this is what I intend to test:
6. In any cluster, install or upgrade a chart. In the first page of the helm chart install workflow, select the new checkbox that says "Use Custom Container Registry for Rancher system images"
7. Enter a custom registry
8. Confirm that the new registry is in the values.global.cattle.systemDefaultRegistry in the values.yaml

Codecov Report
Base: 37.21% // Head: 36.82% // Decreases project coverage by -0.38% :warning:

Coverage data is based on head (5271c2d) compared to base (46cc962).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head 5271c2d differs from pull request most recent head d61a41d. Consider uploading reports for the commit d61a41d to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7117      +/-   ##
==========================================
- Coverage   37.21%   36.82%   -0.39%
==========================================
  Files         977      977
  Lines       17262    17290      +28
  Branches     4482     4483       +1
==========================================
- Hits         6424     6367      -57
- Misses      10838    10923      +85

Flag
Coverage Δ

e2e
47.48% <100.00%> (-0.47%)
:arrow_down:

merged
36.82% <100.00%> (-0.39%)
:arrow_down:

unit
5.28% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

shell/pages/c/_cluster/apps/charts/install.vue
0.00% <ø> (ø)

shell/plugins/steve/actions.js
47.54% <100.00%> (+0.87%)
:arrow_up:

shell/mixins/resource-fetch.js
16.92% <0.00%> (-56.93%)
:arrow_down:

shell/mixins/metric-poller.js
77.77% <0.00%> (-22.23%)
:arrow_down:

shell/store/resource-fetch.js
33.33% <0.00%> (-16.67%)
:arrow_down:

shell/utils/poller.js
75.00% <0.00%> (-10.00%)
:arrow_down:

shell/plugins/steve/mutations.js
51.06% <0.00%> (-4.26%)
:arrow_down:

shell/plugins/dashboard-store/actions.js
51.98% <0.00%> (-2.78%)
:arrow_down:

shell/models/provisioning.cattle.io.cluster.js
63.80% <0.00%> (-1.27%)
:arrow_down:

shell/plugins/steve/subscribe.js
70.32% <0.00%> (-0.28%)
:arrow_down:

... and 7 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Can't fully test this PR until https://github.com/rancher/dashboard/issues/7127 is resolved. We need Team 2 to confirm the YAML format for configuring the cluster scoped registry


vector does not work with new abi gen

I am trying to embed a struct into another one, which is eosio::table.
    struct st_question {
        string title;
        vector<string> options;
        string type;
    };

vector would generate "type": "string>" and nodeos can't parse that, giving Error 3015004: The type defined in the ABI is invalid

I used typedef std::vector<std::string> strings;  and manually added
    "types": [
        {
            "new_type_name": "uuid",
            "type": "int32"
        },
        {
            "new_type_name" : "strings",
            "type": "string[]"

        }
    ],

to .abi file. Then it magically worked, transaction passed. So guys, how do I properly declare a vector of strings with new CDT?

I have similar issue too.
        // @abi table tries i64
        struct [[eosio::table("tries")]] tries_info {
            int64_t         round;
            int64_t         seedValue;

            vector< uint8_t > resultNumber;

            int64_t primary_key() const { return round; }
            uint64_t get_descOrder() const { return -round; }
        };
        typedef eosio::multi_index<N(tries), tries_info
           , indexed_by<N(desc), const_mem_fun<tries_info, uint64_t, &tries_info::get_descOrder>>
        > tries;

with eosiocpp
{
      "name": "tries_info",
      "base": "",
      "fields": [{
          "name": "round",
          "type": "int64"
        },{
          "name": "seedValue",
          "type": "int64"
        },{
          "name": "resultNumber",
          "type": "uint8[]"
        }
      ]
    }

but eosio-cpp
        {
            "name": "tries_info",
            "base": "",
            "fields": [
                {
                    "name": "round",
                    "type": "int64"
                },
                {
                    "name": "seedValue",
                    "type": "int64"
                },
                {
                    "name": "resultNumber",
                    "type": "vector<uint8_t>" // !!!HERE
                }
            ]
        },

When I called action which ref's this info, Error printed like this.
Error 3015004: The type defined in the ABI is invalid

So I modified that type to "uint8_t[]" and It works well.

I also encountered this situation, how to resolve it?  help me!!!

I also encountered this situation, how to resolve it? help me!!!

come on guys, just edit .abi file and replace all vector types looking wrong (like string> or vector<uint8_t>) with string[] and uint8_t[]

This has been resolved as of v1.3.0


Port "init push for convert of unique_id as a list tests for spark".

Description
Ports "init push for convert of unique_id as a list tests for spark" (https://github.com/dbt-labs/dbt-spark/pull/321).

Seems like https://github.com/dbt-labs/dbt-spark/pull/328 has an issue. I reverted the latest commits.
Let's handle separately.

Thanks! merging.


Renamed "Mac OS X" to "macOS"

"Mac OS X" has been renamed to "macOS" recently.

Thanks - good one!


Links aren't colored on mobile edit view

I'm a big fan of the supercharged Links and I use them together with the Style Settings Plugin to color my Links depending on which folder they are in.
This works fine on Desktop, but on mobile if I'm in the edit mode and have the source view disabled, the supercharged Links don't work and I only see the default color. Otherwise it works great in both cases.
Have I set something up the wrong way or does it simply not work on mobile (iOS if that matters)?

Quick Fix: Defining the --link-color Attribute manually for all cases in the links.css snippet worked for me. On desktop it is ignored and on mobile use. Maybe this helps anyone in the future.

I am facing the same problem in obsidian 1.7.4 desktop version, could you elaborate on how to solve this problem?


Ran the rancher eclipse formatter on all files.

@ibuildthecloud Is this what you were expecting?

@hibooboo2 After running the eclipse formatter, does checkstyle pass?  How many warnings/errors are there?

See what you can do about this.  The formatter is collapsing pretty formatted multi-lines into one.  An example is https://github.com/rancherio/cattle/pull/100/files#diff-b15142ef9a357bc694338be5bfc082d0L50
The pretty formatting is pretty important to comprehending the jOOQ calls.

No it does not. I am still getting the settings changed to match the ones we talked about been busy all weekend. Should be close to done with it tomorrow morning.

And currently there are about 6300 but with a few tweaks it drops to 1000ish, and i realized the whole import the xml from check style doesn't work as advertised by what i had read. So i had to do some stuff manually. Took more time than i thought but i am getting closer.


Missing epoll binding files in electron project using webpack

Hi,
I'm using onoff in a library that I need to import in an electron project. This project uses webpack to pack the source code, and it compiles nicely, but when I try to launch it I get the following error
A JavaScript error occurred in the main process
Uncaught Exception:
Error: Could not locate the bindings file. Tried:
→ /path/to/my/project/dist/build/epoll.node
→ /path/to/my/project/dist/build/Debug/epoll.node
→ /path/to/my/project/dist/build/Release/epoll.node
→ /path/to/my/project/dist/out/Debug/epoll.node
→ /path/to/my/project/dist/Debug/epoll.node
→ /path/to/my/project/dist/out/Release/epoll.node
→ /path/to/my/project/dist/Release/epoll.node
→ /path/to/my/project/dist/build/default/epoll.node
→ /path/to/my/project/dist/compiled/7.9.0/linux/arm/epoll.node
at bindings (/path/to/my/project/dist/main.js:805:9)
at Object. (/path/to/my/project/dist/main.js:705:41)
at webpack_require (/path/to/my/project/dist/main.js:20:30)
at Object. (/path/to/my/project/dist/main.js:340:11)
at webpack_require (/path/to/my/project/dist/main.js:20:30)
at Object. (/path/to/my/project/dist/main.js:196:14)
at webpack_require (/path/to/my/project/dist/main.js:20:30)
at Object. (/path/to/my/project/dist/main.js:103:15)
at webpack_require (/path/to/my/project/dist/main.js:20:30)
at /path/to/my/project/dist/main.js:63:18
I'm using a RaspberryPi and I've already tried to recompile the epoll module in my node_modules, but without success.
My electron version is 1.7.9.
Do you know what shall I do?
Thank you in advance!

I've just found out that the epoll dependency uses node-bindings to import its stuff. Node bindings does not work with webpack since it's like a dynamic require, so webpack does non know what to import when creating the bundle file. I installed node-loader that allows you to import a .node file and replaced module.exports = require('bindings')('epoll.node'); with module.exports = require('./build/Release/epoll.node'); in node_modules/epoll/epoll.js, and now its working fine.
Since I don't like it much as a solution because I have to edit a file in the node_modules directory, I'll look for a way to make webpack do that.
I know it is not strictly related to onoff module, but I think it'd be nice to leave the issue here for other peolple who may have the same problem in the future.

Ok, so this is what I've done to make it work:

Install the dependency 'string-replace-webpack-plugin' with npm i -D string-replace-webpack-plugin
Install the webpack loader node-loader with npm i -D node-loader
Add the following rule to the webpack config:{
    test: /epoll\.js$/,
    loader: StringReplacePlugin.replace({
        replacements: [
            {
                pattern: /require\(\'bindings\'\)\(\'epoll\.node\'\)\;/ig,
                replacement: function () {
                    return "require('./build/Release/epoll.node');";
                }
            }
        ]
    })
}

Add the following rule to the webpack config:{
    test: /\.node$/,
    use: 'node-loader'
}

In the plugin array of the configuration object, add the item new StringReplacePlugin()

This will replace the string require('bindings')('epoll.node'); with require('./build/Release/epoll.node'); that will be correctly parsed by node-loader.
This solution is working for me if I launch the project with electron main.js and if I compile it and generate the deb file to install.
If you've got a better solution, please let me know!
I'll close this issue for now.


How build dynamic lib???

Whole documentation doesn't ephsis this, I just got a libssl.a.. I need libssl.so

What OpenSSL version are you talking about?  Did you build it yourself?  If so, can you show us your configuration command line?


Add type.isStringType() and type.isNumberType()

Fairly simple change, I needed some way to test for string / number types.

Looks awesome. Thanks a lot!

@Gerrit0 this is now available in 9.2.0.


Profile not correct with SpringBoot 2.4.1 + Ilford 2020.0.0 (working with 2.4.1 + Ilford 2020.0.0-RC1)

Hello Spring Cloud Config Team,
Maybe a small bug with Ilford 2020.0.0 and SpringBoot 2.4.1
With a web app, 2.4.1 + Ilford 2020.0.0-RC1, the properties are being picked up correctly at client side.
When the client starts, I do see below logs, very happy 😃
[           main] c.a.Application            : The following profiles are active: my-env
[           main] o.s.b.context.config.ConfigDataLoader    : Fetching config from server at : https://my-service/config
[           main] o.s.b.context.config.ConfigDataLoader    : Located environment: name=myservice, profiles=[my-env], label=null, version=7f95facc49b97272a0a3915650b12e56ea7440bf, state=null

[           main] c.u.j.EncryptablePropertySourceConverter : Converting PropertySource configserver:https://my-git/myservice-my-env.properties [org.springframework.boot.env.OriginTrackedMapPropertySource] to EncryptableMapPropertySourceWrapper

note profiles=[my-env]
However, same code, just one change, Ilford 2020.0.0-RC1 -> Ilford 2020.0.0, the properties are not being picked up.
On startup, I am seeing the following instead:
[           main] c.a.Application            : The following profiles are active: my-env
[           main] o.s.b.context.config.ConfigDataLoader    : Fetching config from server at : https://my-service/config
[           main] o.s.b.context.config.ConfigDataLoader    : Located environment: name=myservice, profiles=[default], label=null, version=7f95facc49b97272a0a3915650b12e56ea7440bf, state=null

[           main] c.u.j.EncryptablePropertySourceConverter : Converting PropertySource Config resource 'class path resource [application.properties]' via location 'optional:classpath:/' [org.springframework.boot.env.OriginTrackedMapPropertySource] to EncryptableMapPropertySourceWrapper

And then again reverted back to 2020.0.0-RC1, and working.
May I ask what is the root cause of this issue please?
Thank you

I just encountered the same issue, and created a reproduction path for @spencergibb and his team -> https://github.com/dnijssen/config-server-client . Seems to be a regression issue to me, as I can confirm it did indeed work with 2020.0.0-RC1

As a workaround to get the config client to send the right profile, set spring.cloud.config.profile to the same values as spring.profiles.active.

There is a second issue in config server where the names of the property sources are the same, so Default still gets returned because the correct property source gets overridden.
{
  "name": "config-client",
  "profiles": [
    "test"
  ],
  "label": null,
  "version": null,
  "state": null,
  "propertySources": [
    {
      "name": "class path resource [config/config-client.yaml",
      "source": {
        "spring.config.activate.on-profile": "test",
        "config-client.test": "Test"
      }
    },
    {
      "name": "class path resource [config/config-client.yaml",
      "source": {
        "config-client.test": "Default"
      }
    }
  ]
}

Ty for the workaround! I can indeed confirm the second issue as well :) Would you be able to convert that one into a seperate issue yourself? As you might be able better to describe it.

Where to apply these workarounds? To me it seems that the configserver pull is completely broken. For me it doesn't even work with legacy processing. It just doesn't pull the files with in application.properties and bootstrap.properties.
The documentation is very confusing regarding this, it would be good to have that in a central place as currently it is not clear how to upgrade.
spring.application.name=current-service
spring.cloud.config.uri=http://localhost:8888
spring.config.use-legacy-processing=true

Where to apply these workarounds? To me it seems that the configserver pull is completely broken. For me it doesn't even work with legacy processing. It just doesn't pull the files with in application.properties and bootstrap.properties.
The documentation is very confusing regarding this, it would be good to have that in a central place as currently it is not clear how to upgrade.
spring.application.name=current-service
spring.cloud.config.uri=http://localhost:8888
spring.config.use-legacy-processing=true

They have been fixed in version 3.0.1 see https://github.com/spring-cloud/spring-cloud-release/wiki/Spring-Cloud-2020.0-Release-Notes#known-issues

They have been fixed in version 3.0.1 see https://github.com/spring-cloud/spring-cloud-release/wiki/Spring-Cloud-2020.0-Release-Notes#known-issues


Allow CGO_ENABLED=1

Dynamically linked Go applications compiled with CGO_ENABLED=1 on a system with glibc (default on debian based distros) instead of musl (default on alpine) can now run inside an iron/go container.

Initially I tested only with a simple program that printed something to the screen and it worked, but after I started using CGO_ENABLED=1 features such as golang plugins, I got the same error as you. I ended up using https://github.com/frol/docker-alpine-glibc, which has glibc inside an alpine container.
The work around proposed here only seems to work when compiling a program with CGO_ENABLED=1, but not using features that require dynamic linking or linking with C programs.


small typo fix

Minor fix in the README. Thanks for the great repo btw, jupyter only dev environments feel much more efficient now.

Thanks for your PR. As mentioned at the beginning of the README however, this typo fix is going to be useless since the README file is autogenerated from the idnex notebook. So next time we build the docs, it will be erased. I'll port your fix to that notebook but for next time, remember to change index.ipynb then run make docs :)


Modified ElasticHQ service to be optional

CHANGELOG
MODIFIED

ElasticHQ service to be optional and enabled by default.

NOTES
Earlier, ElasticHQ was always on and I personally never use it. So I think this can be made optional, so that I can switch it off using the Warden configuration. I hope this would help other developers too.

Also, @davidalger - is this repo participating in Hacktoberfest? If yes, could you please add the appropriate label to this PR?
https://hacktoberfest.digitalocean.com/resources/maintainers

Yep. Still relevant. :)

@davidalger could you please just merge it? 😃

@davidalger, could you please merge this PR as it improves behavior on ARM machines as the elasticsearch-hq image isn't available for ARM.
Also, the elasticsearch-hq project isn't maintained anymore


[Feature] Support Hooks for metadata

Search before asking

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

Description
Can Doris support the same features as Hive hooks.Users can implement hooks to deal with different ddl events.
For example:
HiveHook  implements ExecuteWithHookContext
ExecuteWithHookContext is a new interface that the Pre/Post Execute Hook can run with the HookContext.
DDL events:
BaseHiveEvent
      ├── AlterTableRename
      ├── CreateHiveProcess
      ├── DropDatabase
      ├── DropTable
      ├── CreateDatabase
      │   └── AlterDatabase
      └── CreateTable
          └── AlterTable
              └── AlterTableRenameCol

Use case
Use cases:

Metadata management
Lineage analysis
Data governance
...

Related issues
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

That would be a good idea. But i'm familiar with HiveHook.
If you or anyone interested in it, welcome to contribute!

we want this.

I can try to implement it, please assign to me. @morningman

is there any plan for this exciting feature?

I can try to implement it, please assign to me. @morningman

@stalary  Do you  implement it ?  I also want to  the function ,Can you help me ?


[Mod Support Request] - Goblin Traders (Fabric)

https://www.curseforge.com/minecraft/mc-mods/goblin-traders-fabric)
Only 2 Mobs
1.18

Goblin Traders was mentioned in #16

It is already supported. i just checked.

Slightly off-topic, but I'm looking for a Goblin Trader icon/sprite that I could add to a competitor's map mod. :-)
Anyone know if if one has been created?


Combine outputs of bidirectional encoder

It seems to me that the view method in the following function incorrectly mixes the outputs of different samples in a batch. In other words, I expect that combine_bidir(outs)[:, 0, :].contiguous().view(-1) to be exactly the same as outs[:, 0, :].contiguous().view(-1). I tested the function with some random tensors and couldn't confirm it. Please let me know if something is wrong with my reasoning. Thank you.
https://github.com/pytorch/fairseq/blob/f7f2dd01ea141cc3303d6735e0fdcf8402b1557b/fairseq/models/lstm.py#L222-L231

Good catch, thank you! I've submitted a fix (#249), if you could please double-check it. I've also launched some training runs with bidir vs. unidir for comparison.


Custom log formating

hey, i need to know if i can use this tool to analyze my log files which are not web server logs :
a sample of my log files:
14224430529 | TX: id: 401550a, data:  27 0 0 0 0 0 0 212, dlc: 8, flags: 30004
14224430597 | TX: id: 401550a, data:  27 0 0 0 0 0 0 213, dlc: 8, flags: 30004
14224430662 | RX: id: 401550a, data:  27 0 0 0 0 0 0 214, dlc: 8, flags: 30004

where the first column is a timestamp, and it does not have an IP field or protocol
and i wanted to analyze this based on some calculation of mine
if so , how can i configure it to work with my logs
thanks

Unfortunately you can't. To properly parse a log it needs a valid IPv4/6, date/time and the request.


Add postman collections and document their use.

Purpose:
SVCS-170
Build locally-runnable Postman test suite for testing providers
Changes:
Updateed  docs/api.rst  Instructions
Added  tests/postman/collections/copy_files.json Postman collection
Added tests/postman/collections/copy_folders.json Postman collection
Added tests/postman/environments/copy_file_sample.json Postman environment
Side effects
None
[#SVCS-170]

LGTM!  Rebased and merged as b31a392e4c330b125fed315db7c401ccfee568e3.


Add cli flag :apple:

closes #16
@yuanqing Would be better to include the contribute doc, like how to run locally and test locally.

@yuanqing ??

@yuanqing  this is not released yet, right?

This fix is in 0.0.80

Also added a script with instructions for testing locally – hope this helps!


Updating LIDAR-Lite v4 LED firmware.

It is described in the product manual that there is a way to update the firmware of the LIDAR-Lite v4 via Bluetooth, and if I am reading it correctly, that the zipped files (DFU.zip) for this update are supposed to be located in this github-rep. I am however unable to find any such files here, so I was just wondering where to look for a more up to date firmware, if there are any such files? Or in case I already possess the latest firmware update, where to find such in the future? I have been searching via Google, but to no avail. Thanks.

There have been no firmware updates released to date. There is an update in the final stages of beta test and will be posted soon.


ci(cron): bump oras from v1.0.0 to v1.2.0

Description
Bump oras from v1.0.0 to v1.2.0
test run - https://github.com/DmitriyLewen/trivy-db/actions/runs/11339374413/job/31534396989

We were not using oras locally installed accidentally, and the error started occurring as ubuntu-24.04 deleted oras from the installed tools, right?

We were not using oras locally installed accidentally

Right.

error started occurring as ubuntu-24.04 deleted oras from the installed tools, right?

Almost right. ubuntu-22.04 contains oras. But ubuntu-24.04 doesn't contain oras.
GitHub didn't remove oras. GitHub started using ubuntu-24.04 as latest.

Yes, I meant GitHub deleted oras starting from ubuntu-24.04. So, this PR title looks like it just updates the oras version, but it actually fixes some problems. Can you update the title and the description to clarify that?

@knqyf263 Done


Add BlockList events

https://github.com/petabi/review-database/issues/118

Codecov Report
Patch coverage: 35.00% and project coverage change: -0.25% :warning:

Comparison is base (0b1fbba) 52.56% compared to head (041831d) 52.32%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
 Share it here.


Add another Universe and pool reply

Hi
Please you can manage this library just as you did with the library LXDMXWiFi_Library so that it can be used a greater amount of the Universe and also to function settings ArtNet pool reply thanks :-)

This library is updated to match the LXWiFiDMX library.  However, this library is designed to work on ATmega328 processors which have far less memory than the ESP8266.  That library has plenty of memory to support full merge of two streams into two universes.  The Arduino Uno does not.  This library does not support merging from separate network sources.  In fact, there is not even an array of bytes for storing DMX values separate from that used for the incoming packet data.

Thank you for having so quickly modified the library is really great. Can you help me please because I have a problem when you add a second Universe and poll reply library is not working properly test with neopixel library that is part of the code Please advise me exactly what wording should look like the code to work properly function poll reply thanks.
void loop() {
uint16_t r,g,b;
uint8_t i;
uint8_t read_result = Universe0->readDMXPacket(eUDP);
uint8_t read_result2 = 0;
// read a packet and if the packet is dmx, write its data to the pixels
if ( read_result ) {
for (int p=0; p<170; p++) {
  // for each NeoPixel find the slot number (each takes 3 slots for RGB)
  i = 3*p;
  r = Universe0->getSlot(i+1);  // Red
  g = Universe0->getSlot(i+2);  // Green
  b = Universe0->getSlot(i+3);  // Blue
  // gamma correct
  r = (r*r)/255;
  g = (g*g)/255;
  b = (b*b)/255;
  // write to NeoPixel Driver
  leds.setPixelColor(p, r, g, b);
}
// send to NeoPixel Ring
leds.show();

}
if ( read_result2 = Universe1->readDMXPacket(eUDP)); {
for (int p=170; p<340; p++) {
// for each NeoPixel find the slot number (each takes 3 slots for RGB)
i = 3p;
r = Universe1->getSlot(i+1);  // Red
g = Universe1->getSlot(i+2);  // Green
b = Universe1->getSlot(i+3);  // Blue
// gamma correct
r = (rr)/255;
g = (gg)/255;
b = (bb)/255;
// write to NeoPixel Driver
leds.setPixelColor(p, r, g, b);
}
// send to NeoPixel Ring
leds.show();
}
}


Make a beep once goggles are up and running.

HDZero Goggle take around 20 seconds to start up.
This PR makes goggles beep once they started up, so user doesn't have to check visually.

Suggestion: make it an option in the gui?

Suggestion: make it an option in the gui?
Yes, there may be people who don't like beep.

Suggestion: make it an option in the gui?

Yes, there may be people who don't like beep.

I'll do it once I have some free time.

much desired option.

There's already a beep in my case when my goggles boot up to HDZero. The expansion module is set to automatically switch off if not used, and the goggles beep as soon as the power to the expansion module is toggled.
Could this feature be implemented by having the goggles do a beep when the source is changed? This means it will beep on startup, and when the source is changed, to indicate the switch was succesful.

addition of .vscode/launch.json should not be in this PR.

addition of .vscode/launch.json should not be in this PR.

@nerdCopter why not? I tested it with WSL, it should work in native Linux as well. Very useful if you need to debug something.

why not?

Because PRs are better when they focus on just one thing/area.  So while, useful, that change does not relate to the rest of the PR and its overall area of focus. It would probably be better to have that in a in a separate PR, adding just that functionality. Smaller/single focus PRs are easier and quicker to review, and thus more likely to get merged sooner. i.e. that might have been merged a couple of months ago now if it wasn't part of this PR ;)


CreatePom parsing update with maven type and classifier tags

I left the MavenArtifactId class untouched to just fix this in CreatePom and avoid breaking other things that depend on that class. I wasn't able to get bazel to run tests for me(even on master), but confirmed that mvn was able to handle the classifier and type tags using the same format as what's in the updated CreatePomTest!

thanks for the PR.
Just one question/suggestion.

I'm trying to figure out why I'm not being shown an option to approve your PR to run in the CI.

Oh weird, I hope I forked/setup the PR the right way. Do you maybe have to be assigned instead of just a reviewer?

can you merge master in and see if that fixes?

I think we had the CI setup wrong: https://github.com/bazeltools/bazel-deps/pull/359

looks like we do have a test failure.

could you run bazel test //test/scala/com/github/johnynek/bazel_deps:createpomtest locally? looks like that is still failing.

Yeah sorry I'm a bazel newbie so idk what my issue is but I keep getting errors in protobuf that causes it to fail building?
bazel test //test/scala/com/github/johnynek/bazel_deps:createpomtest
WARNING: Running Bazel server needs to be killed, because the startup options are different.
Starting local Bazel server and connecting to it...
INFO: Analyzed target //test/scala/com/github/johnynek/bazel_deps:createpomtest (156 packages loaded, 1670 targets configured).
INFO: Found 1 test target...
ERROR: /private/var/tmp/_bazel_jakestebbins/bfa95731683341b674a6956238363bd9/external/com_google_protobuf/BUILD:206:11: Compiling src/google/protobuf/any.pb.cc failed: (Exit 1): cc_wrapper.sh failed: error executing command external/local_config_cc/cc_wrapper.sh -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -Wunused-but-set-parameter -Wno-free-nonheap-object -fcolor-diagnostics ... (remaining 42 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
external/com_google_protobuf/src/google/protobuf/any.pb.cc:30:5: error: member initializer 'type_url_' does not name a non-static data member or base class
  : type_url_(&::_pbi::fixed_address_empty_string, ::_pbi::ConstantInitialized{})
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
...

```

Yeah I tried running the script, and also confirmed I'm running bazel v4.2.0, and I also tried bazelisk with no luck.
% ./bazel build //...
INFO: Analyzed 109 targets (4 packages loaded, 39 targets configured).
INFO: Found 109 targets...
INFO: From Compiling zutil.c:
external/zlib/zutil.c:133:22: warning: a function declaration without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
const char * ZEXPORT zError(err)
                     ^
external/zlib/zutil.c:305:22: warning: a function declaration without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
voidpf ZLIB_INTERNAL zcalloc (opaque, items, size)
                     ^
external/zlib/zutil.c:315:20: warning: a function declaration without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
void ZLIB_INTERNAL zcfree (opaque, ptr)
                   ^
3 warnings generated.
INFO: From Compiling inftrees.c:
external/zlib/inftrees.c:32:19: warning: a function declaration without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
int ZLIB_INTERNAL inflate_table(type, lens, codes, table, bits, work)
                  ^
1 warning generated.
ERROR: /private/var/tmp/_bazel_jakestebbins/bfa95731683341b674a6956238363bd9/external/com_google_protobuf/BUILD:155:11: Compiling src/google/protobuf/arenastring.cc failed: (Exit 1): cc_wrapper.sh failed: error executing command external/local_config_cc/cc_wrapper.sh -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -Wunused-but-set-parameter -Wno-free-nonheap-object -fcolor-diagnostics ... (remaining 34 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
external/com_google_protobuf/src/google/protobuf/arenastring.cc:190:20: error: no member named 'IsAllocated' in 'google::protobuf::internal::TaggedStringPtr'; did you mean 'GetIfAllocated'?
  if (!tagged_ptr_.IsAllocated()) {
                   ^~~~~~~~~~~
                   GetIfAllocated
/usr/local/include/google/protobuf/arenastring.h:170:23: note: 'GetIfAllocated' declared here
  inline std::string *GetIfAllocated() const {
                      ^
external/com_google_protobuf/src/google/protobuf/arenastring.cc:219:19: error: no member named 'IsAllocated' in 'google::protobuf::internal::TaggedStringPtr'; did you mean 'GetIfAllocated'?
  if (tagged_ptr_.IsAllocated()) {
                  ^~~~~~~~~~~
                  GetIfAllocated
/usr/local/include/google/protobuf/arenastring.h:170:23: note: 'GetIfAllocated' declared here
  inline std::string *GetIfAllocated() const {
                      ^
2 errors generated.
ERROR: /private/var/tmp/_bazel_jakestebbins/bfa95731683341b674a6956238363bd9/external/io_bazel_rules_scala/src/java/io/bazel/rulesscala/scalac/BUILD:10:12 Building external/io_bazel_rules_scala/src/java/io/bazel/rulesscala/scalac/scalac.jar (4 source files) failed: (Exit 1): cc_wrapper.sh failed: error executing command external/local_config_cc/cc_wrapper.sh -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -Wunused-but-set-parameter -Wno-free-nonheap-object -fcolor-diagnostics ... (remaining 34 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
INFO: Elapsed time: 3.519s, Critical Path: 3.03s
INFO: 21 processes: 17 internal, 4 darwin-sandbox.
FAILED: Build did NOT complete successfully
 % bazel --version
bazel 4.2.0

@johnynek sorry for the lag, finally got tests to pass I just need to add a test case for a dependency with a non-empty < classifier > tag

thanks for sticking with it.
It seems the tests still aren't passing. I'm sorry the project doesn't build for you. It shouldn't depend on your environment, but I guess something in one of the rules we depend on does... Maybe we need to update the version of the scala rules. I really don't know.

@johnynek weird the tests are passing for me locally, and just did a pull to make sure I'm up to date
`jakestebbins@Jake-Stebbins-MacBook-Pro bazel-deps % ./bazel test --cache_test_results=no //...
INFO: Build option --cache_test_results has changed, discarding analysis cache.
INFO: Analyzed 109 targets (0 packages loaded, 1709 targets configured).
INFO: Found 101 targets and 8 test targets...
INFO: Elapsed time: 40.015s, Critical Path: 39.49s
INFO: 9 processes: 1 internal, 8 darwin-sandbox.
INFO: Build completed successfully, 9 total actions
//test/scala/com/github/johnynek/bazel_deps:coursier_test                PASSED in 9.0s
//test/scala/com/github/johnynek/bazel_deps:createpomtest                PASSED in 5.3s
//test/scala/com/github/johnynek/bazel_deps:graphtest                    PASSED in 2.3s
//test/scala/com/github/johnynek/bazel_deps:modeltest                    PASSED in 2.6s
//test/scala/com/github/johnynek/bazel_deps:normalizertest               PASSED in 3.3s
//test/scala/com/github/johnynek/bazel_deps:parsegenerateddoctest        PASSED in 39.2s
//test/scala/com/github/johnynek/bazel_deps:parsetest                    PASSED in 2.9s
//test/scala/com/github/johnynek/bazel_deps:parsetestcases               PASSED in 3.0s
Executed 8 out of 8 tests: 8 tests pass.
INFO: Build completed successfully, 9 total actions
jakestebbins@Jake-Stebbins-MacBook-Pro bazel-deps % `

can you try --runs_per_test=10 and see if that passes.

Yeah it looks like it just failed the parsegenerateddoctest once, but in the latest CI run its showing the tests passed and errored in the command run after
`jakestebbins@Jake-Stebbins-MacBook-Pro bazel-deps % ./bazel test --cache_test_results=no --runs_per_test=10 //...
INFO: Build option --runs_per_test has changed, discarding analysis cache.
INFO: Analyzed 109 targets (0 packages loaded, 1709 targets configured).
INFO: Found 101 targets and 8 test targets...
FAIL: //test/scala/com/github/johnynek/bazel_deps:parsegenerateddoctest (run 3 of 10) (see /private/var/tmp/_bazel_jakestebbins/bfa95731683341b674a6956238363bd9/execroot/com_github_johnynek_bazel_deps/bazel-out/darwin-fastbuild/testlogs/test/scala/com/github/johnynek/bazel_deps/parsegenerateddoctest/run_3_of_10/test.log)
INFO: Elapsed time: 258.888s, Critical Path: 248.90s
INFO: 81 processes: 1 internal, 80 darwin-sandbox.
INFO: Build completed, 1 test FAILED, 81 total actions
//test/scala/com/github/johnynek/bazel_deps:coursier_test                PASSED in 35.8s
Stats over 10 runs: max = 35.8s, min = 16.2s, avg = 29.0s, dev = 5.4s
//test/scala/com/github/johnynek/bazel_deps:createpomtest                PASSED in 25.5s
Stats over 10 runs: max = 25.5s, min = 18.2s, avg = 21.5s, dev = 2.2s
//test/scala/com/github/johnynek/bazel_deps:graphtest                    PASSED in 5.2s
Stats over 10 runs: max = 5.2s, min = 3.9s, avg = 4.7s, dev = 0.4s
//test/scala/com/github/johnynek/bazel_deps:modeltest                    PASSED in 8.6s
Stats over 10 runs: max = 8.6s, min = 5.4s, avg = 6.7s, dev = 1.1s
//test/scala/com/github/johnynek/bazel_deps:normalizertest               PASSED in 11.0s
Stats over 10 runs: max = 11.0s, min = 8.4s, avg = 10.0s, dev = 0.7s
//test/scala/com/github/johnynek/bazel_deps:parsetest                    PASSED in 12.5s
Stats over 10 runs: max = 12.5s, min = 8.6s, avg = 10.8s, dev = 1.5s
//test/scala/com/github/johnynek/bazel_deps:parsetestcases               PASSED in 19.8s
Stats over 10 runs: max = 19.8s, min = 17.3s, avg = 18.6s, dev = 1.0s
//test/scala/com/github/johnynek/bazel_deps:parsegenerateddoctest        FAILED in 1 out of 10 in 248.3s
Stats over 10 runs: max = 248.3s, min = 184.2s, avg = 221.0s, dev = 17.9s
/private/var/tmp/_bazel_jakestebbins/bfa95731683341b674a6956238363bd9/execroot/com_github_johnynek_bazel_deps/bazel-out/darwin-fastbuild/testlogs/test/scala/com/github/johnynek/bazel_deps/parsegenerateddoctest/run_3_of_10/test.log
Executed 8 out of 8 tests: 7 tests pass and 1 fails locally.
INFO: Build completed, 1 test FAILED, 81 total actions
jakestebbins@Jake-Stebbins-MacBook-Pro bazel-deps % `

huh... failed again

Yeah I just ran it again with the same --runs_per_test=10 flag and it passed

I don't see how your change could have broken this test since your change is downstream of the code being tested.
I'll merge and we can address the test.
Thanks!

Sweet, thanks so much for all of the help!

thank you! I appreciate your help!


[Feature]upload file to resource one file at a time very inefficient, can upload more at a time?

Describe the solution you'd like
upload file to resource one file at a time very inefficient, can upload more at a time?
资源上传，一次只能上传一个文件，效率太低了，如果文件多，耗时就会比较久，此外如果文件上传到错误的文件夹，则又需要重新上传；
在此也可以通过后台把文件一次上传到hdfs，然后文件信息插入到表t_ds_resources中，但效率还是有点...

good idea

Hi,
Good idea, would it be better to include the following two functions:

DS directly binds a certain HDFS path, the files can be uploaded from the background to HDFS by user.
The front end uploads multiple files at a time.

Hi,
Good idea, would it be better to include the following two functions:

DS directly binds a certain HDFS path, the files can be uploaded from the background to HDFS by user.
The front end uploads multiple files at a time.

A node type can be extended to upload local files to hdfs, which can be run in multiple threads, I can complete this

Hi,
Good idea, would it be better to include the following two functions:

DS directly binds a certain HDFS path, the files can be uploaded from the background to HDFS by user.
The front end uploads multiple files at a time.

A node type can be extended to upload local files to hdfs, which can be run in multiple threads, I can complete this

Related issue: #5201


About The Train

Will you help me  to solve this problem ? 👆

Waiting for the checkpoints😀

Hi~ Thanks for your attention to our work.
The termination of the process doesn't seem to be due to a lack of checkpoints, as the training process doesn't require LLaRA checkpoints, but only needs the Hugging Face model of LLaMA2-7B. Could you provide more information about your configuration of the model path for the LLaMA-2 7B Hugging Face version?

Hi~ Thanks for your attention to our work.嗨~感谢您对我们工作的关注。 The termination of the process doesn't seem to be due to a lack of checkpoints, as the training process doesn't require LLaRA checkpoints, but only needs the Hugging Face model of LLaMA2-7B. Could you provide more information about your configuration of the model path for the LLaMA-2 7B Hugging Face version?该过程的终止似乎不是由于缺乏检查点，因为训练过程不需要LLaRA检查点，只需要LLaMA2-7B的Hugging Face模型。您能提供有关LLaMA-2 7B Hugging Face版本模型路径配置的更多信息吗？

Thanks

It's seem‘s like no problem about the  LLaMA-2 7B Hugging Face config path or version

It seems your program was killed while executing the line of code self.llama_model = LlamaForCausalLM.from_pretrained(llm_path, torch_dtype=torch.bfloat16) in mode/model_interface.py. Perhaps you could try directly loading the LLaMA-2 7B Hugging Face version model under your conda environment and see if it can be loaded normally.

It seems your program was killed while executing the line of code self.llama_model = LlamaForCausalLM.from_pretrained(llm_path, torch_dtype=torch.bfloat16) in mode/model_interface.py. Perhaps you could try directly loading the LLaMA-2 7B Hugging Face version model under your conda environment and see if it can be loaded normally.似乎您的程序在执行 mode/model_interface.py 中的代码行 self.llama_model = LlamaForCausalLM.from_pretrained(llm_path, torch_dtype=torch.bfloat16) 时被终止。也许您可以尝试在您的 conda 环境下直接加载 LLaMA-2 7B Hugging Face 版本模型，看看是否可以正常加载。

Ok，got is，i will try later，Thx a lot ！！！

Yeah, as you said, I reset the LLaMA-2 7B Path in another machine.   It's work

Hello,another Q，How can i get this file？

I have download the dataset

Yeah, as you said, I reset the LLaMA-2 7B Path in another machine. It's work

Hello, I meet some problem about this code. Did you run the code successfully?


Add-on read weight value once then needs restart

Add-on reads the weight from the scale one time and then need to be restarted (restart addon).
To Reproduce
Restart HA, use the scale, then use it again. The first value will be registered immediatly, the second only after an addon restart.
Scale (please complete the following information):

Mi Smart Scale 2

Device running Home Assistant (please complete the following information):

Device used to run the Script: Raspberry Pi 4;
Bluetooth device used: Built-in (note: another integration -miflora- is using BLE on different MAC)

LOG:
2022-04-16 08:32:54 - Starting Xiaomi mi Scale...
2022-04-16 08:32:54 - Loading Config From Add-On Options...
2022-04-16 08:32:54 - Config Loaded...
2022-04-16 08:32:55 - Discovery Completed...
2022-04-16 08:32:56 - Publishing data to topic miscale/Manuel/weight: {"weight":77.40,"weight_unit":"kg","bmi":25.27,"basal_metabolism":1580.56,"visceral_fat":14.29,"timestamp":"2022-04-16T08:32:56+00:00"}
2022-04-16 08:32:56 - Data Published ...

When you step on the scale a 2nd time, is another message sent to your MQTT Broker? (Check with a tool like MQTT Explorer)
Remember that if the same data is sent, HA will not update the sensor...

Issue has become stale, closing...


feat: Improve care_site_types mapping

At the moment all care_site_types are hardcoded to 0. We can probably make this more realistic and map to proper
Perhaps we can use some simple heuristics like presence of PCP to bind the care_site_type to 38004247 | Ambulatory Primary Care Clinic / Center|.
I've not seen anyone use this attribute in real life in fairness (most of the work I've been a part of doesn't find it relevant)...

Somewhat related to #83

I like the idea of settling on a code. Question along those lines. Is our
etl example supposed to represent ehr. If so, can we choose that?
On Wed, Oct 23, 2024, 17:34 Lawrence Adams @.***> wrote:

At the moment all care_site_types are hardcoded to 0
https://github.com/OHDSI/dbt-synthea/blob/47c7fd058680f38e2bb5858e28ec7c5f32b51c31/models/omop/care_site.sql#L4.
We can probably make this more realistic and map to proper
Perhaps we can use some simple heuristics like presence of PCP to bind
the care_site_type to 38004247 | Ambulatory Primary Care Clinic / Center|
https://athena.ohdsi.org/search-terms/terms/38004247.
I've not seen anyone use this attribute in real life in fairness (most of
the work I've been a part of doesn't find it relevant)...
Somewhat related to #81 https://github.com/OHDSI/dbt-synthea/pull/81
—
Reply to this email directly, view it on GitHub
https://github.com/OHDSI/dbt-synthea/issues/85, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYR4GJYASJFPPOPOE5X5ADZ5AI7RAVCNFSM6AAAAABQPXDXAOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGYYDSOBZGMZDMMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I like this idea too!  It seems like the docs suggest inferring place of service from the types of visits associated with a care site - https://ohdsi.github.io/Themis/mapping_place_of_service_concept.html.  Maybe we build some logic around this?
I don't love using the name of the organization to infer place of service because in my experience this name is usually the legal name of a business and could be misleading.  On the other hand, looking at the aggregate of visit types to derive the place of service feels like it might be overly complicated and also not necessarily accurate either...
@heschmidt04 - I think most type concepts are set in the event tables but probably worth a re-review and confirmation we like the choices.


[BUG] Speculative decoding regresses performance on 7900 xtx under ROCM

OS
Linux
GPU Library
AMD ROCm
Python version
3.12
Pytorch version
Pulled from https://download.pytorch.org/whl/rocm6.2 yesterday
Model
Qwen2.5-Coder-32B + Qwen2.5-Coder-1.5B as draft model
Describe the bug
When loading the Qwen2.5-Coder-32B model through Exui, I am getting around 20 tokens/s with a 4_25 bpw quant (unrelated, but this is also lacking compared to the 25+ I am seeing with llamacpp). However, when loading the 1.5B version of the model as a draft model, performance drops down to below 16 tokens/sec instead of experiencing a speedup. I do experience a speedup with llamacpp speculative decoding (a little over 2x).
Reproduction steps

Load the 32B model through exui
Ask for a story in a chat
See around 20 tokens/second
Unload the 32B model
Load the 32B model + 1.5B draft model
Ask for another story in a new chat

Expected behavior
A speed boost is obtained.
Actual outcome: The performance regresses.
Logs
No response
Additional context
No response
Acknowledgements

[X] I have looked for similar issues before submitting this one.
[X] I understand that the developers have lives and my issue will be answered when possible.
[X] I understand the developers of this program are human, and I will ask my questions politely.

Set lower max seq len, recommend 16k.


add ocean "biomass" as a soca state

Description
Previous studies suggest that the assimilation of both chlorophyll and biomass (proxied by particulate organic carbon, or POC) into ocean bigeochem models may yield better results than that of chlorophyll alone. This is very likely to be true for BLING given that chlorophyll is a diagnostic and thus is not connected or feedback to the evaluation of biogeochem states in the model. Here I propose to add ocean "biomass" as a soca state, in the same manner with that of adding chlorophyll, in preparation for the assimilation of satellite-derived POC observations.
Of course, it is important to develop appropriate balance operators between chlorophyll, biomass, nutrients, and other ocean biogeochem variables, but that would be addressed separately at a later stage.

@guillaumevernieres I think it should be pretty straightforward by following what has been done with chlorophyll, and also given that ocean color ioda-converter already handles POC preprocessing. Please let me know if it's okay for me to proceed with a draft PR for this. Thanks!

@guillaumevernieres I think it should be pretty straightforward by following what has been done with chlorophyll, and also given that ocean color ioda-converter already handles POC preprocessing. Please let me know if it's okay for me to proceed with a draft PR for this. Thanks!

@guillaumevernieres I think it should be pretty straightforward by following what has been done with chlorophyll, and also given that ocean color ioda-converter already handles POC preprocessing. Please let me know if it's okay for me to proceed with a draft PR for this. Thanks!

@liuxiao37k , yes, sounds good, go for it!

@guillaumevernieres I think it should be pretty straightforward by following what has been done with chlorophyll, and also given that ocean color ioda-converter already handles POC preprocessing. Please let me know if it's okay for me to proceed with a draft PR for this. Thanks!

@liuxiao37k , yes, sounds good, go for it!


feat: save tracked docs via post upsert hook

Maintains the new TrackedDocument table in sync with the DUST_TRACK() tags that are found in documents.
For now, restrict docs that can be tracked to Notion, Google Drive and Github (no self-managed DS and no slack)

@PopDaph FYI


[c++] Set -Wno-deprecated-declarations for AppleClang too

I cannot compile Microsoft Bond on macOS Big Sur 11.2.2 and Xcode 12.4
with Apple clang version 12.0.0 (clang-1<IP_ADDRESS>) and Boost 1.75.0
because of this flag:
In file included from /usr/local/include/boost/locale.hpp:24:
/usr/local/include/boost/locale/util.hpp:203:59: error: 'auto_ptr<boost::locale::util::base_converter>' is deprecated [-Werror,-Wdeprecated-declarations]
    std::locale create_codecvt(std::locale const &in,std::auto_ptr<base_converter> cvt,character_facet_type type);

Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Thanks!
For Boost.Locale, one can also define BOOST_LOCALE_HIDE_AUTO_PTR to hide its use of std::auto_ptr
There's BOOST_NO_AUTO_PTR for all of Boost too.
These are not defines that Bond can set, lest is cause ODR violations.


Appstudio update rhtap-demo-component-pwvu

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: rhtap-demo-component-pwvu-on-pull-request-
cannot patch pipelinerun rhtap-demo-component-pwvu-on-pull-request-: failed to patch pipelinerun rhtap-demo-dev-wigi-tenant/logURL with rhtap-demo-component-pwvu-on-pull-request-4nbjm: pipelineruns.tekton.dev "rhtap-demo-component-pwvu-on-pull-request-4nbjm" not found


Maybe try out k3s

Another "lightweight k8s that runs nicely on one node..."
https://github.com/rancher/k3s

merged into #9


getIds fails test

In the docs I provided when I wrote the function, it clearly says: sorted id list. And the list doesn't get sorted. Oh my.
I'll have to look into how I intended the function, and then either update the function, or the docs (and test cases).

It was an error in the docs, pushing in a minute.


Not all instances of Lemmy work.

For example I cannot get lemmy.ml or lemmynsfw.com to load.

Duplicate of #19.


compile for different openssl versions

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #46.
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender

I can't find documentation that says this package depends on openssl, but searching the repo you find that when they build their software, they consider installing openssl-devel through yum
https://github.com/huggingface/tokenizers/search?q=openssl&type=

@conda-forge-admin please rerender

I would confirm with upstream what you need here. It seems that openssl is useful in OSX as well but you might need to specify flags manually.
https://github.com/huggingface/tokenizers/blob/28cd3dce2a75d106572392194ff2564574c33235/bindings/python/tokenizers_manual_m1_build.sh


Incompatible with Parcel v2 ?

It seems that this plugin no longer works after upgrading to the new Parcel v2; the report.html file is no longer generated.

There is a Parcel 2 core plugin that is included by default and can be enabled by setting an env-variable: PARCEL_BUNDLE_ANALZYZER=1 parcel build ...
(https://github.com/parcel-bundler/parcel/tree/v2/packages/reporters/bundle-analyzer)

This doesn't seem to output anything for some reason, can't find any documentation for it either.

https://v2.parceljs.org/features/production/

fwiw there's a typo in the above command if anyone missed that as i did
PARCEL_BUNDLE_ANALYZER=1 parcel build

For anyone encountering this with the latest Parcel 2 beta versions (or newer), the new way to call the built-in Parcel 2 Bundle Analyzer is:
parcel build src/index.html --reporter @parcel/reporter-bundle-analyzer
(as seen in the docs at https://v2.parceljs.org/features/production/)

After upgrading to latest parcel version, I get: The plugin "@parcel/reporter-bundle-analyzer" is not compatible with the current version of Parcel. Requires "^2.8.3" but the current version is "2.0.0-nightly.1293

"@parcel/reporter-bundle-analyzer": "^2.8.3",
"parcel": "^2.0.0-nightly.1293",

Shouldn't it work anyway?

@parcel/reporter-bundle-analyzer is not this repo, it's an official plugin: https://github.com/parcel-bundler/parcel/tree/v2/packages/reporters/bundle-analyzer
And this is unfortunately what the semver rules are: https://jubianchi.github.io/semver-check/#/^2.8.3/2.0.0-nightly.1293
You should instead install the nightly for both: yarn add parcel@nightly @parcel/reporter-bundle-analyzer@nightly

Thanks, it worked!
Oh, sorry, I didn't realize it's a different repo, got here by Googling the error.


Verifies properly the fee + amount being bonded balance for the wizard

Description of proposed changes

Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

Codecov Report

Merging #1160 (ceec6d4) into dev (5d565a1) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head ceec6d4 differs from pull request most recent head 1815b5a. Consider uploading reports for the commit 1815b5a to get more accurate results

@@           Coverage Diff           @@
##              dev    #1160   +/-   ##
=======================================
  Coverage   74.62%   74.62%
=======================================
  Files          74       74
  Lines       10686    10686
=======================================
  Hits         7974     7974
  Misses       2712     2712

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


nowAccessor to allow control over "current" date/time (#704)

New, optional, nowAccessor prop which accepts a function to return the current (now) time.
This result is used to indicate which days/times are highlighted (.rbc-today and .rbc-time-indicator). It also informs navigation to "today" via nagivate.TODAY. Finally, it does impact the min, max and scrollToTime.
The rendering example shows usage and a new storyboard is included.

hope that does it. Thanks for the great feedback and thanks to you & the community for building and enhancing this terrific component! I'm so happy to retire fullCalendar (and jQuery)!


Initial integration tests

Adds initial integration tests.

Needs to be rebased after #1 is merged.

@amore17 if you look at the last commit you will see a few examples of tests.

initial review looks good - will review again once #1 is merged and this pr is rebased

CentOS/RHEL 7 is failing because of: https://github.com/linux-system-roles/certificate/issues/5
Since the error was not introduced by this PR a propose that the we work in the fix in a different PR.

If you are ok with the changes until here I'll squash them.


synchronized output

Added support for synchronized output (https://gist.github.com/christianparpart/d8a62cc1ab659194337d73e399004036). Before, my multi progressbars flickered a lot. But not anymore.
Synchronized output should work on Windows, too, I think. But I don't know how to check whether that terminal feature is supported on Windows, and I don't use Windows anyway, so I added support for Unix only.
I added two new dependencies for Unix: lazy-static and libc. lazy-static could be replaced with something else, if that is preferable.

I think it would be good to support this, but I also think a lot of the supporting code (about detecting and using the sync mechanism) would probably make more sense in the lower-level console crate (which indicatif uses). Maybe start a PR there?

Yes, I can do a separate PR in console. @djc Should the API look like this?
let term = Term::stdout();
if term.supports_sync_update() {
	term.begin_sync_update();
}
// do something
if term.supports_sync_update() {
	term.end_sync_update();
}

With sync update doing sth like write_str("\x1b[?2026h")?

I would do a guard type that ends the sync on dropping. But, probably good to ask the console maintainer(s) to weigh in on this.

Since the console crate seems a little bit inactive, I made this PR simpler. It now uses once_cell and might work on Windows, too, though I haven't checked (should be checked before merging, though). Note: I added a new method to a public trait which is a breaking change. I'm still not sure, if this is the correct design (supports_ascii_codes in the public trait).

I pinged a console maintainer on Discord to see if we can get that conversation going. :)

If we were to add a Sync Guard in the console crate, how would we be able to use it in indicatif? I guess via the TermLike trait, but what about the in_memory term then?

If we were to add a Sync Guard in the console crate, how would we be able to use it in indicatif? I guess via the TermLike trait, but what about the in_memory term then?

The in-mem term is just for testing purposes. I don't think we'd need sync guard there necessarily?

Okay, but then what about custom TermLike implementations? indicatif provides the ability to create a ProgressBar via a Box<TermLike>. Right now with supports_ansi_codes, it will work for these, too, but how would we generalize the TermLike trait to allow for custom implementations and also be able to use the console crate's SyncGuard?

Ah ok. I see what you mean. Let me think about it a bit...

Any update, @chris-laplante?

Any update, @chris-laplante?

Sorry, I finally got time to look at this. I think I was able to put all the pieces together in a way that makes sense for both crates. Please see https://github.com/console-rs/console/pull/205 for the console side of things, and my PR here: https://github.com/console-rs/indicatif/pull/625

Sorry, I finally got time to look at this. I think I was able to put all the pieces together in a way that makes sense for both crates. Please see https://github.com/console-rs/console/pull/205 for the console side of things, and my PR here: https://github.com/console-rs/indicatif/pull/625

Thanks for taking a look at this! These PRs look good to me!

Still TODO: implement support for WASM, Windows terminals.

Just a note for the future: The Windows 10 terminal can support ANSI codes, if you enable them. If you call colors_supported(), it tries to enable ANSI escape sequences and returns true on success. ANSI escape sequences are needed for the supports_synchronized_output() function (as well as the SyncGuard) to work correctly.
https://github.com/console-rs/console/blob/de2f15a31a8fef0b0e65ef4bdf92cd03c3061dac/src/windows_term/mod.rs#L77
So in the future it might look like this.
#[inline]
pub fn is_synchronized_output_supported(&self) -> bool {
    #[cfg(unix)]
    {
        supports_synchronized_output()
    }
    #[cfg(not(unix))]
    {
        self.colors_supported() && supports_synchronized_output()
    }
}

I think it would be more efficient to cache the result of colors_supported(), though, instead of re-doing the operation for every SyncGuard.
But right now, the with_raw_terminal that supports_synchronized_output uses doesn't work for Windows, so it won't work.
In my last update for this PR, I simply removed supports_synchronized_output since the ANSI escape sequences for synchronizing are invisible in the terminal, if the terminal does not support synchronization (when the terminal supports ANSI escape sequences).
By the way, this one line that I removed in console (use std::ptr;) was wrong, since macOS needs this and the tests for it now fail. It should be possible to put this line in fn select_fd(...), however.

Sorry, I finally got time to look at this. I think I was able to put all the pieces together in a way that makes sense for both crates. Please see console-rs/console#205 for the console side of things, and my PR here: #625

Thanks for taking a look at this! These PRs look good to me!

Still TODO: implement support for WASM, Windows terminals.

Just a note for the future: The Windows 10 terminal can support ANSI codes, if you enable them. If you call colors_supported(), it tries to enable ANSI escape sequences and returns true on success. ANSI escape sequences are needed for the supports_synchronized_output() function (as well as the SyncGuard) to work correctly.
https://github.com/console-rs/console/blob/de2f15a31a8fef0b0e65ef4bdf92cd03c3061dac/src/windows_term/mod.rs#L77
So in the future it might look like this.
#[inline]
pub fn is_synchronized_output_supported(&self) -> bool {
    #[cfg(unix)]
    {
        supports_synchronized_output()
    }
    #[cfg(not(unix))]
    {
        self.colors_supported() && supports_synchronized_output()
    }
}

I think it would be more efficient to cache the result of colors_supported(), though, instead of re-doing the operation for every SyncGuard.
But right now, the with_raw_terminal that supports_synchronized_output uses doesn't work for Windows, so it won't work.
In my last update for this PR, I simply removed supports_synchronized_output since the ANSI escape sequences for synchronizing are invisible in the terminal, if the terminal does not support synchronization (when the terminal supports ANSI escape sequences).
By the way, this one line that I removed in console (use std::ptr;) was wrong, since macOS needs this and the tests for it now fail. It should be possible to put this line in fn select_fd(...), however.

Would you mind copying this comment to the PR in console? That way I can reply to the console specific parts there.


Error when running tests

Running tests on a fresh checkout, I get two errors related to the recent updates in extract.py:

The use of the @contextmanager decorator introduced in a recent commit:

https://github.com/linked-art/crom/blob/d55e207eb1395eada52878d3f1a3bd763ff20e54/cromulent/extract.py#L728
Does this need an associated from contextlib import contextmanager?

The use of f-strings seem to prevent use on python 2.7. That's fine, but setup.py (and some comments throughout the rep) still seems to indicate support for 2.7.

https://github.com/linked-art/crom/blob/d55e207eb1395eada52878d3f1a3bd763ff20e54/cromulent/extract.py#L725

Oops, f-strings and 2.7...

Closing ... think we can let 2.x die.


Rasa Core Error: Error: connect ECONNREFUSED <IP_ADDRESS>:5005

Rasa Core version:
rasa-core==0.9.7
Python version:
Python 2.7.15rc1
Operating system (windows, osx, ...):
Ubuntu 18.0.4
Issue:
hamin101@ubuntu:~/rasaui$ sudo npm start

><EMAIL_ADDRESS>start /home/hamin101/rasaui
> node server/server.js

Rasa UI Server: http://localhost:5001

Express server listening on port 5001

Rasa NLU Error: Error: connect ECONNREFUSED <IP_ADDRESS>:5000
Using connection string from: package.json

Rasa Core Error: Error: connect ECONNREFUSED <IP_ADDRESS>:5005
Using connection string from: package.json

Postgres DB Connected
Using connection string from: package.json
Postgres Server: <IP_ADDRESS>:5432
Database:rasa
Schema:public

Why am I getting the Error: connect ECONNREFUSED <IP_ADDRESS>:5000 for Rasa NLU and Error: connect ECONNREFUSED <IP_ADDRESS>:5005 for Core?
Content of domain file (if used & relevant):

This is trying to start the rasaui right? this is a community project, not maintained by us so I think you're better off posting an issue there :) https://github.com/paschmann/rasa-ui


Hi, can you explain the code?

for example, the anchors,the x1y1,x2y2

I will add more descriptive comments and a bit of documentation later this week when I have more time. If you do not understand the code segments after that, then I am happy to answer any questions you have.


feat: Bushy tree join ordering

Signed-off-by: Kevin Axel<EMAIL_ADDRESS>hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
support bushy join for streaming, see also https://github.com/risingwavelabs/rfcs/pull/23

Checklist For Contributors

[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[x] My PR DOES NOT contain user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

I have run the TPCH Q9 by enabling the bushy tree. It panics. It seems something is wrong.

FYI, TPCH Q9 could be found here. https://github.com/risingwavelabs/risingwave/blob/64810b29eba162015111ad7b1094c894de19acc8/src/frontend/planner_test/tests/testdata/tpch.yaml#L1542

Please add some planner tests.

After examining the query plan of TPCH Q9, it appears that only a join tree with a height of 4 is generated, rather than the optimal one with a height of 3.


[Feature request] Summernote editor - dropdown box with image file names

I would like to see uploaded image types in a dropdown box in summernotes "insert image" dialog.

@robiso what is the status of 'insert image' on the development branch?
Looking back, it seems it was removed, but I still see the options listed in the js file.
Would there be a way of bringing that option back, but editing the dialog that comes up upon selection? Or would this require a fork? Was having a bit of trouble following where our summernote was in relation to the origin.

@notcarbide, the "insert image" is there, but it only allows the user to enter the URL of an image (that can be fetched from Settings -> Files (right click and copy image URL).
As you noticed, we used to have a list of the already uploaded.
The files fetching functionality (developed by an open source contributor) contained a vulnerability, which I was unable to fix properly.
Thus, that functionality was removed upon contacting the developer with no response received.
I wouldn't mind this being implemented properly, though.

Would there be a way of bringing that option back, but editing the dialog that comes up upon selection?

I'm not sure I understand completely, can you re-phrase the question please?
Here's the link to the first repo the open source dev first provided with the Summernote editor: https://github.com/prakai/wcms-summernote

Ah, okay. My current test build generated from the devFixes branch (using the builder) doesn't seem to have the "insert image" button at all. That's why the question was confusing lol.
I wonder if fixing the latest issue I made would solve that. Since it's the only difference in deploying a build vs cloning the project, as far as I know. I can see about trying the latter and verify later today.

Good catch, this was already fixed in 2.7.0 but didn't apply it to the devFixes yet.
It's now been pushed to the devFixes branch with the following commit: https://github.com/robiso/wondercms/commit/48f79e9c196d6f68bf5ebb79bc64a9bb34853798
I can now successfully see the insert images and similar inserts in the toolbar. Let me know how it worked for you.

I have tried to update the original file.php so it is atleast save for exposure to the outside world.
A (draft) pull request with more info is here: robiso/summernote-editor#3

Fixed summernote edditor has missing ordered list functionality. In air version ordered lists present

@RarogCmex Summernote has been updated to include the ordered/unordered list functionality.
Update to 3.0.9 to see them.
Thanks for the heads up.

@robiso That's strange, I'll succesfully updated to 3.0.9 but still does not have ordered list functionality. Are you sure all's ok?
I've tried to remove, then reinstall but that's has no effect

Just updated it myself to 3.0.9 and the additional icons are there.
Please make sure to clear your cache, as the .js file was updated (had to clear it myself).
To verify, you can open the admin.js file in the Summernote plugin folder to verify the paragraph section is there ['para', ['ul', 'ol', 'paragraph']].
You can try opening it via example.com/plugins/summernote-editor/js/admin.js and if it's not updated, re-check it via SSH/FTP to see the uncached state of the file

@RarogCmex, just updated it myself to 3.0.9 and the additional icons are there.
Please make sure to clear your cache, as the .js file was updated (had to clear it myself).
To verify, you can open the admin.js file in the Summernote plugin folder to verify the paragraph section is there ['para', ['ul', 'ol', 'paragraph']].
You can try opening it via example.com/plugins/summernote-editor/js/admin.js and if it's not updated, re-check it via SSH/FTP to see the uncached state of the file

Even stranger :) I've checked that file updated successfully, but still don't see icons.

Where I should clear cache?

@RarogCmex, did you check the file via the browser or via SSH?

Via ssh.

First, just try to visit your website and login in your browser private mode to see if it's an actual caching issue.
To clear your cache:
On Firefox (https://support.mozilla.org/en-US/kb/clear-cookies-and-site-data-firefox)

Click the menu button Fx57Menu and select Preferences.
Select the Privacy & Security panel and go to the Cookies and Site Data section.
Click the Clear Data… button. The Clear Data dialog will appear.
Check just the cached Web Content (to remove stored images, scripts and other cached content) should both be check marked.
Click Clear.

On Chrome (https://support.google.com/accounts/answer/32050?co=GENIE.Platform%3DDesktop&hl=en)

On your computer, open Chrome.
At the top right, click More.
Click More tools.
Next to "Cookies and other site data" and "Cached images and files," check the boxes.
Click Clear data.

@robiso Thanks, solved :)
Sometimes I learn Haskell enought to write my own replacement for that editor. But that's not this time.

@RarogCmex, even though a replacement can work, caching issues are something you might run into often if your server is behind a CDN/caching system or something like CloudFlare.
Also, glad clearing the cache made it work as expected, no problem at all.

@StephanStanisic I believe you were working on this a while ago? Is there any chance it could be included (I honestly forgot what the state of that feature was).

Yes, it should be working but further testing of the fix is required before we put it online. We should also either bundle jQuery + bootstrap + font awesome with the plugin since they aren't available in all themes (eg. watercolor). Or maybe pick a new editor that doesn't have bootstrap as a dependency.
The repo is here: https://github.com/StephanStanisic/summernote-editor

This feature is one of the few ones I miss from the fantastic WonderCMS. It would be very practical to have.  Can I help with testing it somehow?

The current state of the plugin should be working in this pull request: https://github.com/robiso/summernote-editor/pull/3
It is currently waiting for a security checkup by someone, I'll see if I can do that next week.
It would be awesome if you could test out the updated version! A lot of things have changed in core since the pull request, so I am guessing some things are broken.

Nice! I'll help. Should I test it now or after you have got the security checkup? Also, I have only checked out ordinary repos before, how would I go about getting the repo with the pull request added?

If you look at the pull request, I'm asking to merge StephanStanisic:master into robiso:master. You can click on StephanStanisic:master to go to the repo I'm trying to merge, StephanStanisic/summernote-editor
I'll do the checkup this week when I have time, but I'm pretty sure some things will be broken since it was quite some time ago we've worked on this.

Thanks! I installed it manually, removing the existing plugin and extracting the current files from StephanStanisic/summernote-editor from April 7, but I see no difference. Adding an image still only shows the "URL of image or document" field. Perhaps I did something wrong. The summernote editor plugin says "Installed version: 3.0.7" but I can update to 3.0.9. Weird thing is that now Summernote editor is listed twice in my Plugins-section even though I deleted the previously installed version first. The second copy can not be updated however. Sorry if I'm just confusing you, I might not be the best person to do this testing, but I really do want to help. :)

Ok, I think I got the correct one installed now! You can disregard the last post.
Now it's obvious things aren't working though. For one the background and some of the graphics are missing (see here
It is also almost impossible to click the buttons because the whole editor section keeps resizing, cycling through three size steps, when hovering over the buttons with the cursor.

Ah, I see that there are new changes on robiso:master, ill merge them into the pull request. That will probably fix this issue.

@StephanStanisic bumping this, can you please ping me when it's be run for a test?

Adding a note here, from OneLaptopMan from community:

Yeah, Summernote integration would be great, in fact having the whole process (drag image -> auto uploads -> auto embed -> set alt text) would be great too. I'm just thinking of the easiest way my clients could populate a gallery without having a monstrous amount of base64 code filling the JSON file and having an actual file with useful meta data src set and all that good stuff available.


Adding to our Eslint

What does this PR do? (please provide any background)
We don't really have a JS style guide for the UI-Toolkit so we have added extras linting to ensure code consistency.
What tests does this PR have?
None
How can be this tested?

We are using 1tbs js brace styles, there are example on http://eslint.org/docs/rules/brace-style.html on what will pass and fail linting.
Write or adjust some JS in the toolkit locally then npm run lint and you should see lint pass or fail depending on the code you have written.

Screenshots / Screencast

What gif best describes how you feel about this work?

Developer Definition of Done/Quality Checklist (for PR author to complete BEFORE code review):

[ ] I have checked the contributing document and I'm happy for this to be reviewed.

Software Engineer or Developer review:

[ ] I’ve witnessed the work behaving as expected (this could be on the authors machine or screencast).
[ ] I’ve checked for coding anti-patterns.
[ ] I’ve checked for appropriate test coverage.
[ ] I’ve run all the tests locally and they pass.

Software Engineer or project guru review:

[ ] I’ve witnessed the work behaving as expected (this could be on the authors machine or screencast).
[ ] I’ve checked for coding anti-patterns.
[ ] I’ve checked for appropriate test coverage.
[ ] I’ve run all the tests locally and they pass.

Taking SE/Dev

:+1:

:+1: