slog in a browser

Hi! I would like to use slog in a browser in such a way, that I could output "compact" mode there using all the console goodies, like console.group (so I could collapse and expand logs): https://developer.mozilla.org/en-US/docs/Web/API/Console/group
I've been looking at the slog_term::TermDecorator but it seems we will rather not be able to reuse it here (or we will?).
Will such abilities be available in slog? :)

Will such abilities be available in slog? :)

If you or someone adds them. :)
Most of slog is very portable and agnostic of everything. So I would imagine all it takes to get slog_term to run in WASM or something like that is to give it something that implements io::Write and translates the writes into whatever you want to output to.

Would also like this.

This should be possible as a custom Drain.
If you already having something implementing std::io::Write, you should already be able to use slog_term.
However, on WebAssembly in the browser, you may also want to use console.log. I don't know if that can easily be integrated with std::io::Write


Docker images have to be small case. docker-image-build script does not ensure this.

Describe the bug
The shell script for docker image build does not make sure that the image name has to be small-case. If the source folder name is camel-case or capitalized, it gets dumped as is into the shell script.

Duplicate of #507


Redo iteration logic

Draft since we can't completely replace the existing iteration logic without messing with both sharing (which would conflict with #103) and finalization (which would conflict with #99). But this is mostly complete.
Basically, I was able to encapsulate all the weird pointer-comparison stuff into a new dag module which exposes a nice API that lets you iterate over commitnodes or redeemnodes, regardless of whether you want normal references or Rcs. As a user you never need to use types like RefWrapper.

After talking to @roconnor-blockstream on IRC, I am going to follow this up with a PR that gets replaces this iterator (which exposes the internal DAG representation) with one ore more of

An iterator which yields every node in the entire unshared tree
An iterator (for RedeemNode) which yields nodes with full sharing enforced (for use in bit-serialization)
An iterator (for both CommitNode and RedeemNode) which yields nodes with full sharing except that disconnect and witness nodes are never shared (which may be used for human-readable serialization).

But I still want to get this PR in first, because it pays down a bunch of tech debt related to iteration.

Hiding our internal sharing makes sense. Do we care about DDoS for the unshared iterator? The worst case is an unshared program of exponential size, right?

85664961e28207a5e870407dcc676ea45bb293a3: This commit does like three things: Adding PointerId, copying PostOrderIter and adding the deduped-witness iterator. Split?

@uncomputable yes, it is (in general) exponential. It may be that we want to check the static bounds against consensus limits during finalization. Or maybe we should require the user provide an upper limit on resources before starting iteration. Worth filing an issue about.
But I think it's fine for now to have the iterator, and we can think about bounding it later.

Using RefWrapper is a canonical solution for adding trait implementations like Eq, but your solution is cleaner for our project. The fewer types users have to deal with the better.


Ticking entity error

（I‘m Chinese,I used a translation tool)
After I broke an ender crystal, the server fell into an unresponsive state and eventually crashed, and after checking the crash report, I found that it was caused by this mod.
Minecraft version:1.20.1
Mod version:5.0
Crash reporting:
crash-2023-09-10_10.52.38-server.txt

I also found that your "Spoorn Armor Attributes" mod has a similar problem, which is trying to append damage to entities that no longer exist.
Another crash report:
crash-2023-09-10_10.43.46-server.txt


feat(#488): Indentify The Issue With Lambdas

In this PR I identified the problem related to lamdas assembling/disassembling. Actually, we just need to implement lamdas support. I left a new puzzle to do so in the future.
Related to #488.

History:

feat(#488): try to add more exception handlers to add information about problems
feat(#488): separate 'reference' type to two different types 'type' and 'class'
feat(#488): add one puzzle to implement lambdas
feat(#488): add more assertions for custom-transformations integration test
feat(#488): fix all qulice suggestions

PR-Codex overview
The focus of this PR is to implement lambdas assembling and disassembling, enable the execution of a specific integration test, and update method invocations.
Detailed summary

Updated method invocation to support lambdas in DirectivesMethodVisitor.java
Added plugin configuration in pom.xml
Enhanced test cases in HexDataTest.java
Implemented functional interfaces and streams in Application.java
Updated class references in DataType.java
Added @ToString annotation in DirectivesData.java

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 13min)


Fix marshalling protos to JSON in poet client

Motivation
Marshalling that the standard "encoding/json" package provides, does not operate correctly on protocol buffer messages. (See:  https://pkg.go.dev/google.golang.org/protobuf/encoding/protojson#pkg-overview)
Changes
Changed marshalling and unmarshaling to use the protojson library.
DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

bors merge

bors merge

bors merge


Fix location configs

Something broken for location-configs with the latest update (06-06-2022)
rs3cache.exe --dump location-configs --input C:\rs3-06062022\input\ --output C:\rs3-06062022\output\

Error: An error occurred while interpreting the cache.

Caused by: Could not decode id 123546 (src\definitions\location_configs.rs:178:96)

Caused by: Reached terminating opcode but the buffer was not exhausted (src\definitions\location_configs.rs:238:33)

Note: The unread remainder of the buffer consists of b"\x08\x01\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0"

Note: Managed to read up to:
{
  "id": 123546,
  "models": {
    "10": [
      125681
    ]
  },
  "unknown_17": false,
  "unknown_19": 0,
  "unknown_24": 34833,
  "ambient": 0,
  "colours": [
    [
      50496,
      20928
    ]
  ],
  "textures": {
    "14888": 14882
  },
  "model": false,
  "scale_x": 180,
  "scale_y": 180,
  "scale_z": 180,
  "translate_y": 65461,
  "occludes_2": false,
  "unknown_196": 4
}

I have no idea what the remaining bytes are supposed to be, or how to fix it properly, but advancing the buffer by the length of those remaining bytes fixes it and locations-config.json is successfully generated. I check if the remaining bytes length is 27 and if it's not defer back to the usual error to aid with any future fixes.

Thanks, I properly fixed this with https://github.com/mejrs/rs3cache/commit/e06ab908b3c9390f7c2734db8b012ab5fa8d738d 😉


[Scaffolding] Add tree command

Is your feature request related to a problem? Please describe.
The agent wants to use tree to visualize the directory structure and we should support that.
2024-02-10 22:11:07,908 | INFO | #################### Turn 7 ####################
2024-02-10 22:11:07,908 | INFO | -- ACTION:
From reviewing the commands_test.py file, it seems the CLI logic is imported from the sqlfluff.cli module. Let's check the structure there:

COMMAND: tree sqlfluff/cli
2024-02-10 22:11:07,909 | INFO | -- OBSERVATION:
/bin/bash: line 1: tree: command not found

2024-02-10 22:11:07,909 | INFO | -- REWARD:
0
2024-02-10 22:11:07,909 | INFO | -- DONE:
False
2024-02-10 22:11:07,909 | INFO | -- INFO:
{'action_executed': False}
2024-02-10 22:11:13,903 | INFO | HTTP Request: POST https://api.anthropic.com/v1/messages "HTTP/1.1 200 OK"

fixed in commit


Embed URL from youtube is undefined

Bug Description
When trying to embed a video from YouTube, the URL attribute in the json file just says "url": "https://www.youtube.com/undefined", no matter which video I try to embed.
Steps to reproduce

Copy a YouTube URL
Embed it
Download the JSON file and look at the URL attribute

Expected behaviour
No response
Embedded Assets version
2.11.4
Craft CMS version
3.9.13

Embedded Assets just uses the data that the Embed package (in this case version 3) passes through, so either Embed or YouTube will be setting that. That said, Embed 3 is no longer actively developed by its owner, Craft 3 is no longer supported, and YouTube embeds do work as expected on Craft 4 and Embedded Assets 4, which uses Embed 4. So it's recommended that you upgrade to at least Craft 4 to resolve this.

@peirix Found a quick fix for this that's working for us: https://github.com/reusserdesign/craft-embedded-assets/commit/960f0553bb1094ae2c9685b7b73d31236cb3766a
We understand Embed v3 and Craft v3 are no longer supported, but we've still got a lot of clients on those versions who aren't ready for a full upgrade to v4 yet so this works in the meantime.

@jrrdnx Thanks! That worked for us as well.


Bug: using resetServices() in tests cause named routes to fail

PHP Version
8.2
CodeIgniter4 Version
4.4.1 and develop
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
macOS
Which server did you use?
cli
Database
irrelevant
What happened?
Using resetServices() in tests causes named routes to fail.
Steps to Reproduce
Routes.php
$routes->get('sample-url-1', 'HomeController::index', ['as' => 'sample1']);

Sample.php - sample library
namespace App\Libraries;

class Sample
{
    public function urlTo($route)
    {
        return url_to($route);
    }

    public function routeTo($route)
    {
        return route_to($route);
    }
}

SampleTest.php - sample test
use App\Libraries\Sample;
use CodeIgniter\Test\CIUnitTestCase;

/**
 * @internal
 */
final class SampleTest extends CIUnitTestCase
{
    protected function setUp(): void
    {
        $this->resetServices();

        parent::setUp();
    }

    public function testSampleUrlTo()
    {
        $sample = new Sample();
        $this->assertInstanceOf(Sample::class, $sample);

        $this->assertSame(
            'http://example.com/index.php/sample-url-1',
            $sample->urlTo('sample1')
        );
    }

    public function testSampleRouteTo()
    {
        $sample = new Sample();
        $this->assertInstanceOf(Sample::class, $sample);

        $this->assertSame('/sample-url-1', $sample->routeTo('sample1'));
    }
}

Expected Output
Passing tests.
Anything else?
If we comment out the line with $this->resetServices();, then all tests will pass.

In the constructor of RouteCollection, the loadRoutes() method should be called?
https://github.com/codeigniter4/CodeIgniter4/blob/83b64b4f9949ecdf1f351832b0e29391205e6e2c/system/Router/RouteCollection.php#L286
https://github.com/codeigniter4/CodeIgniter4/blob/83b64b4f9949ecdf1f351832b0e29391205e6e2c/system/Router/RouteCollection.php#L317

That would solve the problem, but I've never done anything related to routes, so I don't know how such a change would affect the rest of the code.

In normal use cases, there is no change because loadRoutes() should be called (and is called now) right after the instantiation of RouteCollection.
But the existing tests for the framework would fail, because some tests expect the RouteCollection does not have any route right after the instantiation.

I found a bug #8024 in a test and the framework code.
It may be better to change to call loadRoutes() in the constructor.
But many existing tests for the framework fails, so it seems better to change it in 4.5 just to be on the safe side

How about adding loadRoutes() to the resetServices()? Would that help with failing framework tests?

Oh, that might be better.
As for RouteCollection, I think it would be preferable to have no route at instantiation.

Oh, it doesn't work.
Services::routes()->loadRoutes() creates the Request instance because RouteCollection depends on Request.
And hack like $_SERVER['HTTP_HOST'] = 'adm.example.com'; for testing does not work in the test method,
because the Request is already created in setUp().

As you probably already know, the workaround is to call Services::routes()->loadRoutes() after resetServices() in the test code.

Yeah... maybe we should just promote this as a permanent solution. Not sure what others think about it.

This is not a bug, and you need to call loadRoutes() if you need.
I sent a PR like that: #8047

I can live with that, thanks!


Add description of docker cp STDOUT functionality to master

In the next release, docker cp will support the STDOUT to tar feature.  The feature is already on the master branch under development. Download, test, and document the feature against the master binary.

@moxiegirl there has got to be a better way to do this.  Isn't master supposed to always be correct within itself?  Couldn't a PR (with just the 1.5 doc edits) be merged directly into the 1.5 branch you guys are working on w/o rolling back edits to master?

Closing this as well because of the comments in #11305

@duglin I was trying to make this that PR.  I might be doing it wrong of course ---. :-P
Just as an FYI, we do want to improve the workflow. The goal is automatically push live certain doc PRs on merge.  We just are in the middle of Bday work.


Calling query_once() multiple times causes duplicates to accumulate

I'm calling query_once() every n seconds in a loop and noticed, that over time the list grows, starting containing duplicates.
This happens until, in my case on esp32, the process crashes, causing a reset.

Thank you for the report.
I checked the corresponding source code and do have a follow up question:

Did you before calling query_once run the "protocol, service" calling query? That seems to be the only way this can accumulate if I read the code correctly.
The returned list in query_once returns duplicates. Currently in the background there is a dictionary which hashes the DNS responses coming from the wire and do only use port and target as the dictionary hash generation. So could you double check if the port changes in the DNS responses?

Closing this as I did not hear anything back. If you still run into the issue feel free to comment and I reopen it.


MD010 no-hard-tabs rule is intended for indentation, but is triggered from any tabs

The description states:

This rule is triggered by any lines that contain hard tab characters instead of using spaces for indentation.

But in reality the rule is triggered also from tabs that are not used for indentation.
The rule should allow use of tabs in other use than indentation because it's not a forbidden character in Markdown syntax.

The use of the word "indentation" may be misleading, but the rule name and the rest of the rule description try to communicate that its purpose is to prevent tab characters anywhere within a line. As the rationale at the bottom outlines, tab characters can be handled inconsistently and may be considered confusing and unnecessary.
https://spec.commonmark.org/0.31.2/#tabs


COVADO-25 - Toggle strictSSL when allowInsecure is true for rest calls

We were not toggling strictSSL for rest calls when allowInsecure was true, but we were for soap calls.
A newer version of tsc was used (4.2.3), so additional changes to the javascript and package/package-lock files were also included.
The release version for this is 1.0.1. This is reflected in coverityTask/task.json. The version used in vss-extension.json is to try and ensure QA can perform upgrades without having to first uninstall. A separate PR will be opened to change the version within vss-extension.json to 1.0.1

Git parameter in Jenkins does not work with slashes in the branch name. Creating new PR with branch name without slash in it.


When the GRU input is entered as 0, the result is different from the GRU of pytorch.

Hello, I am making DS2 (DeepSpeech2 model) model using GRU primitive. However, if you enter zero input into GRU primitive, the result is not the same as pytorch GRU. By the way, if enter 1, the result is the same as pytorch GRU. All network parameters were initialized to 1 and tested. Below is the code I wrote.
vector<float> inputBuf(2 * 1 * 6, 0.0f) // (T, N, C) init value is zero
vector<float> inputBuf2(2 * 1 * 6, 1.0f) // init value is one.
auto inputMem = memory({{2, 1, 6}, data_type::f32, memory_format::tnc}, engine, inputBuf.data());
...  (weightslayer, weightsiter, bias setting using ldigo, ldgo)
...  (direction : bidrectional_sum, kind: prop_forward_training)
auto gru_primitive = gru_forward::primitive(..., engine)
... (src, srciter, weightslayer, weightsiter reordering)
...
gru.execute(...) ...

DNNL GRU execute result using one input:

pytorch GRU execute result using one input:

DNNL GRU execute result using zero input:

pytorch GRU execute result using zero input:

@rsdubtso: edited for formatting

There are two versions of GRU supported by DNNL, LBR-GRU and GRU. From the document of PyTorch, it should use LBR-GRU primitive.
 It would be highly appreciated.

Oh.. Thank you :) I did not check the LBR-GRU .. T-T
Can I ask you one more question?
I want to control B _ u 'bias to give arbitrary value. But, a userBiasBuffer size be fixed..
for example, the bias buffer size can be defined as follows.
(num_of_layer * num_direction * num_gate + 1(extra_bias) * hidden_state_size).
However, it is not known how many rows B _ u 'bias are placed. Can you tell me this part?
or if there is any other way, please tell me.

Thank you :) I did't check the LBR-GRU T-T..
can i ask a one more question?..
from the document of DNNL(LBR-GRU), there is follow description.
-> For the bias tensor, we implicitly require the order of the gates to be u, r, o, and u'.
So, When I initialize bias, can i initialize as follows?
vector<float> biasBuf(numLayers * numDirection * 4(adding extra_bias) * hidden_state_size, 1.0f);
int offset = numLayers * numDirection * hidden_state_size;
for (int i = offset + 3; i < biasBuf.size(); i++) { // b_u' bias init.
    biasBuf[i] = 0.5f;
}
/* So, internal vector value as follow.
  1 1 1 1 1 1 1 1  --> b_u
  1 1 1 1 1 1 1 1  --> b_r
  1 1 1 1 1 1 1 1  --> b_0
  0.5 0.5 0.5 0.5 0.5 0.5 0.5 0.5  --> b_u'

@kjyeon1676 Yes, you are correct.

Thank you for your kindness !

I'm sorry to reopen it. T-T
thanks to zixuanweeei help, we're continuing to build the DS2 network.
I'm creating a network comparing the results with pytorch.
Because of reopen, bidirectional_sum result seems to be strange.
When i use to option about unidirectional_left2right and right2left, result same to pytorch.
However, When i use to option about bidirectional_sum, result is different.
Below is the code i Wrote.
// input shape and size.
// T(seqence_length) : 2, batch : 2, numLayer : 1, direction : 2, C : 6, H : 8
// input value : 0.0f, hidden state memory value : 0.0f;
// C : input channel, H : hidden_state_size
vector<float> inputBuf(T * batch * C, 0.0f);
auto hiddenMem = memory();

vector<float> userWeightsLayerBuf(numLayer * numDirection * C * 3 * H, 1.0f);
vector<float> userWeightsIterBuf(numLayer * numDirection * H * 3 * H, 1.0f);
vector<float> userBiasBuf(numLayer * numDirection * 4 * H, 2.0f);   // add extra_bias + 1

 memory::dims weightLayerDims{numLayers, numDirection, C, 3, H};
 memory::dims weightIterDims{numLayers, numDirection, H, 3, H};

 // dimensional sequence : l d g o
 // num_layers, num_directions, num_gates(lbr_gru is extra_bias + 1), output_channels(H).
 memory::dims biasDims{numLayers, numDirection, 4, H};

 // create memory descriptor for user data.
 auto userWeightsLayerDesc = memory::desc(weightLayerDims, dataType::f32, memoryFmt::ldigo);
 auto userWeightsIterDesc = memory::desc(weightIterDims, dataType::f32, memoryFmt::ldigo);
 auto userBiasDesc = memory::desc(biasDims, dataType::f32, memoryFmt::ldgo);

 auto userWeightsLayerMem = memory(userWeightsLayerDesc, eng);
 auto userWeightsIterMem = memory(userWeightsIterDesc, eng);
 auto userBiasMem = memory(userBiasDesc, eng);

 write_to_dnnl_memory(userWeightsLayerBuf.data(), userWeightsLayerMem);
 write_to_dnnl_memory(userWeightsIterBuf.data(), userWeightsIterMem);
 write_to_dnnl_memory(userBiasBuf.data(), userBiasMem);

 // create memory descriptor for GRU data w/ no specified.
 auto weightsLayerDesc = memory::desc(weightLayerDims, dataType::f32, memoryFmt::any);
 auto weightsIterDesc = memory::desc(weightIterDims, dataType::f32, memoryFmt::any);
 auto biasDesc = memory::desc(biasDims, dataType::f32, memoryFmt::any);
 ...
 ...(omission)
 auto srcLayerDesc = memory::desc({srcLayerDims}, dataType::f32, memoryFmt::tnc);
 auto dstLayerDesc = memory::desc({dstLayerDims}, dataType::f32, memoryFmt::tnc);
 auto dstIterDesc = memory::desc({dstIterDims}, dataType::f32, memoryFmt::ldnc);

 prop_kind kind;
 if (isTraining) {
        kind = prop_kind::forward_training;
 } else {
        kind = prop_kind::forward_inference;
 }

 auto gruFwdDesc = lbr_gru_forward::desc(kind,       // propagation kind
                                        direction,                            // direction
                                        srcLayerDesc,                     // srcLayerDesc
                                        memory::desc(),                 // srcIterDesc
                                        weightsLayerDesc,             // weightLayerDesc
                                        weightsIterDesc,               // weightIterDesc
                                        biasDesc,                          // biasDesc
                                        dstLayerDesc,                    // dstLayerDesc
                                        dstIterDesc);                      // dstIterDesc, flag

 auto gruPd = lbr_gru_forward::primitive_desc(gruFwdDesc, eng);
 if (isTraining) {
      data.workspaceMem = memory(gruPd.workspace_desc(), eng);
 }
 // input == memory object
 auto srcLayerMem = input;
 // omission.. (reordering and lbr_gru_forward execute)
 ...

and below picture is execution result
<>

<>

<<pytorch result - unidirection>>

In my guess, bidirectional_sum result should be a result of adding 1 and 2 rows.
Therefore, DNNL results should be the same as pytorch. However, the results were different.
##pytorch bidirectional result

##DNNL bidirectional sum result

Please let me know if I miscoded or if the usage is different.
There is no example of GRU, so I do not know if it is right to do this. T-T

Thank you :) I did't check the LBR-GRU T-T..
can i ask a one more question?..
from the document of DNNL(LBR-GRU), there is follow description.
-> For the bias tensor, we implicitly require the order of the gates to be u, r, o, and u'.
So, When I initialize bias, can i initialize as follows?
vector<float> biasBuf(numLayers * numDirection * 4(adding extra_bias) * hidden_state_size, 1.0f);
int offset = numLayers * numDirection * hidden_state_size;
for (int i = offset * 3; i < biasBuf.size(); i++) { // b_u' bias init.
    biasBuf[i] = 0.5f;
}
/* So, internal vector value as follow.
  1 1 1 1 1 1 1 1  --> b_u
  1 1 1 1 1 1 1 1  --> b_r
  1 1 1 1 1 1 1 1  --> b_0
  0.5 0.5 0.5 0.5 0.5 0.5 0.5 0.5  --> b_u'

Hi @kjyeon1676 , the above code would be correct if the bias layout was gldo. However, we use ldgo for the bias layout, and the above code is correct only for numLayers = numDirection = 1 in this context.
To correctly produce what you want, you have to do the following:
for(int l = 0; l < numLayers; l++)
    for(int d = 0; d < numDirection; d++)
        for(int o = 0; o < hidden_state_size; o++)
            biasBuff[ o + hidden_state_size * (3 +  numBiasGates * ( d + numDirection * l))] = 0.5f;

@mgouicem I checked that the result with pytorch match. :)!!!!
Thank you for your kindness!!! :)


Periodically check for newer versions of scope.

Every 2 hours, hit https://checkpoint-api.weave.works looking for current versions of Scope.  If a new version is found, log it.
Reuses the same library as Hashicorp's products.

Logging it seems fine, but users are unlikely to see that. Would be nicer to show it when launching.

I've made a separate ticket for displaying this in the UI: https://github.com/weaveworks/scope/issues/914
I'd prefer not to show it on startup, due to the way our startup works (starting a container etc).

Also, we should document how the user can disable this.

Updated to once every 6hrs on @pidster's request.


Initial timestamp to 0000-00-00T00:00:00Z

If the timestamp is initial, it should be converted into 0000-00-00T00:00:00Z, it is not the same that January 1, year 1

unit test ?

Sure, there're now ready


Reopen "Change test_name's formart to file_path#class#func"

@shuheiktgw  I reopen https://github.com/launchableinc/nose-launchable/pull/10. Sorry for confusing you.

@ninjinkun https://github.com/launchableinc/nose-launchable/pull/10 I left a few comments on the PR! 🙏

@shuheiktgw I continue #10 discussion on this PR.
https://github.com/launchableinc/nose-launchable/pull/10#issuecomment-716257900 Yes. I worried about the same thing. Developers change directories not frequently but sometimes they do. Although we can add an option to specify the root directory, I worried they can't continue to maintain the relationship between the root directory and test directories.

I'm researching how Git solves this consistency problem

https://github.com/launchableinc/nose-launchable/pull/10#issuecomment-716256383 I fixed this issue.

https://github.com/launchableinc/nose-launchable/pull/10#issuecomment-716258487 For this test generator problem, I plan to aggregate all generated tests' results. For example, test_evens(0, 0) is succeeded, test_evens(1, 1) is failed, test_evens(2, 2) is succeeded, this plugin sends a test_evens failed event. As a negative point, It needs waiting for all tests are done because we cannot know when all generated tests are done.

@ninjinkun Hmm, test.id() seems to have a generated test name can we use it to identify a test name?
(Pdb) test.id()
'tests.dir3.test1.test_evens(0, 0)'

@shuheiktgw It works. However, I worried about generating tests with random parameters. If we use test.id(), we need to group these tests on the server.

@shuheiktgw But at this time, we need to go forward. I'm going to use the name with parameters such as test_evens(0, 0).

@shuheiktgw My apologies, my local testing has failed because of the wrong auth token. I'm going to fix my local, then fix my code.

I fixed them


Better error message for lift in parameterized query

Was getting an error that looked like:
[error] symbol value metricId#204660 does not exist in com.dashboard.models.MetricGroup.<init>, which contains locals .
[error] Method code: def this() = {
[error]   super.this();
[error]   MetricGroup.this.`schema ` = {
[error]     new <$anon: io#41.getquill#79426.dsl#80170.QuotationDsl$Quoted#81609>()
[error]   };
[error]   MetricGroup.this.`experiment_versions_q ` = {
[error]     new <$anon: io#41.getquill#79426.dsl#80170.QuotationDsl$Quoted#81609>()
[error]   };
[error]   MetricGroup.this.`experiments_q ` = {
[error]     new <$anon: io#41.getquill#79426.dsl#80170.QuotationDsl$Quoted#81609>()
[error]   };
[error]   MetricGroup.this.`remove_query ` = {
[error]     new <$anon: io#41.getquill#79426.dsl#80170.QuotationDsl$Quoted#81609>(metricId)
[error]   };
[error]   ()
[error] }

eventually tracked it down to using "lift" inside of a parameterized query. I.e
val my_query = quote { (id: Long) => MyThing.schema.filter( things => things.id == lift(id) ) }
This seems like a pretty common mistake that a beginner could make. It would be great if the failure message was a little more helpful.
Version: (e.g. 0.4.1-SNAPSHOT)
0.8.0
@getquill/maintainers

Hi. I was getting this error:
[error] Error while emitting PostgresService.scala
[error] value c
[warn] two warnings found
[error] one error found
[error] (Compile / compileIncremental) Compilation failed
[error] Total time: 4 s, completed Jun 7, 2020, 4:51:31 PM

With this code:
for {
      ids <- run(
              query[ConversationParticipant].filter(_.userId == lift(as.userId))
            ).map(_.map(_.convId))

      convs <- run(
                query[Conversation].filter(cv => lift(ids).contains(cv.id))
              )

    } yield convs

That was because ids is iterable (List). You need to use liftQuery(ids) for them.


docs: imports update 4

Updated langchain imports to langchain_community.

@baskaryan same question as before do we want to use flat namespaces in langchain_community or more nested ones? (I think we prefer the former, but wanted to confirm)


Workflow-init support

We should have a method that's guaranteed to initialize before signals and updates come in so handlers can use initialized values.

see the parent feature issue for how other languages have handled it: https://github.com/temporalio/features/issues/400

I'm developing the concept of Workflow Init in the constructor: the user can call the Workflow::getInput() method to obtain input arguments.
    public function __construct() {
        $input = Workflow::getInput()->getValues();
        // ... init code
    }

However, getInput() returns a ValuesInterface, which does not contain type information.
Manually specifying the type for required arguments like Workflow::getInput()->getValue(0, DateTimeInterface::class); is inconvenient.
Additionally, new objects will be created instead of reusing existing ones.
My proposal is to add a new method Workflow::getTypedInput(), which will return the same arguments that will be sent to the Workflow Handler after initialization.
This way, we can provide a convenient Workflow Init using language mechanisms. At the same time, we still have the flexibility to add a separate InitHandler if needed.

@roxblnfk That works for me. If it's possible to allow them to specify the args in their constructor, and pass them in there I would say let's do that (possibly in addition to getTypedInput), possibly by using an annotation on the constructor like we've done for Python. If that's not feasible I think this approach will work just fine too.

+1 to @Sushisource's comment, if it's doable.


np.argmax to Tensor.argmax

There are more np.argmax than this PR. I have not touched them as i did not want to change the behaviour accidentally.
Only changed rather seemingly safe part.
Note: I did not test the models as I am unable to do so.

breaks examples/hlb_cifar10.py do not know why.


upgrade hive

Signed-off-by: ldpliu<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/22810

/cc @elgnay

/approve
/lgtm


Improve overall performance of the tool

In some scenarios, we perform multiple HTTP requests to dbt Cloud.
import-jobs requires one call per job for example in order to get the env-var overwrites at the job level.
I have done some simple prototyping for now, but here are the benchmark results for import-jobs with around 15 jobs in an account:
# current implementation, using requests without using a Session
python src/main.py import-jobs --account-id 50  0.54s user 0.07s system 5% cpu 10.343 total

# with requests but leveraging Session
python src/main.py import-jobs --account-id 50  0.27s user 0.06s system 7% cpu 4.609 total

# with httpx and asyncio calls to dbt Cloud
python src/main.py import-jobs --account-id 50  0.33s user 0.07s system 21% cpu 1.826 total

Observations:

Using Session seems to be a low hanging fruit, decreasing overall time by 60% without too much refactoring
The httpx/asyncio is a bigger lift but we might leverage it in places where we have to call multiple times an endpoint with different parameters (like for import-jobs)

Added session in the latest release. Will open new targeted performance one if needed.


Correct DOCTEST_NO_INSTALL logic; do install unless it is set

Description
Commit b63bf85ca872c5fd7f53c2e6d97f61813298936d (fixing #96) introduces the DOCTEST_NO_INSTALL variable, but it's use is reversed: by default it is set to OFF (meaning you do want to install), but the test says "If set to ON, do install".
This made me upload a doctest.h-less doctest-dev package to Debian.
Debian bug
This was detected through a failed build of argagg and reported as https://bugs.debian.org/880721 .

Also many thanks for maintaining a debian package!

No problem for not releasing 1.2.7; it's patched in Debian already. Thanks for the quick merge!

just also got hit with this, when will 1.2.7 be available?
btw, I do not understand the original issue, adding a no install option. Just do not call make install
I think the whole thing should be removed, useless, just makes the cmake file more complex, and error prone, as proven with this issue

The fix is currently in master - 1 commit ahead of the 1.2.6 tag

thanks for your explanation @onqtam
for this case I would suggest thinking about something like
if( ${CMAKE_PROJECT_NAME} STREQUAL ${PROJECT_NAME})
   # have the default install to true, other wise false
endif( ${CMAKE_PROJECT_NAME} STREQUAL ${PROJECT_NAME})

so you can detect if  you are included or not and set the proper default per default


Unable to render graph with few data points

When i try an example using the MSFT.tsv data for a Line /Scatter chart, if i limit the data points to only 600 elements my chart will not render.
I get the following message in the chrome inspector console
Trying to show 1 items in a width of 1284px. This is either too much or too few points
Uncaught Error: Initial render and cannot display any data
I'm planning on displaying an intraday chart that starts off with no data but gets new data on an interval (say 1 min). So from 9:30 to 4:30 it would be incrementally getting data points. It seems like i can't render a chart with few data points from the above error.

few things.
0.4 does not support intraday. the next version 0.5 will support it but is not released yet. You are free to try the development branch next.
You cannot have a chart with no data points, needs atleast 2 points to begin with. if you are using MSFT.tsv and getting this error then review the value of xExtents. Post a plunker with the error so I can help there.

closing due to inactivity, add more details explaing the problem and I will reopen


test: CP1252 encoding, check utf-8 conversion

@mmuetzel I went and learning a little and tried to write a test that convinces me that the strings are being loaded properly into utf-8.

If the newly added file is encoded in CP1252 (I haven't checked if it is), wouldn't you need to add an .oct-config file for this to run correctly?

wouldn't you need to add an .oct-config file

It is in the same dir as your file.  And yes it is CP1252, created with iconv.  Annoyingly, file does not detect this:
$ file test_extra/test_encoding/*
test_extra/test_encoding/test_bytecount_CP1252.m: Non-ISO extended-ASCII text
test_extra/test_encoding/test_CP1252.m:           Non-ISO extended-ASCII text

But this is not the point here, if anything that's while its so important to support the .oct-config file ;-)
I think the onClear still runs after an assert failure.  And anyway, I'm not terribly concerned about clearning up properly on failures (as long as success is clean).

re: Matlab: have not thought at all about encoding on Matlab, but its a good point.  My test (test_bytecount_CP1252.m) is almost certainly Octave specific, as tests for utf-8 usage internally.
I don't currently have an easy way to test this on Matlab: suggest we worry about Octave for now, then happy to revert to your inline testing style of run_tests.m as you originally had it later...

Hmmm, maybe test is already quiet.  I thought I get warnings....  Oh so your test_CP1252 actually passes, at least on Octave 6.4.0!  Just with warnings...    But my test which checks that the text was read correctly requires Octave 7.

@mmuetzel here's TL-DR summary:

on any Octave (we test back to 4.2.0), you can read the the CP1252 file.
... but the results are potentially garbage until Octave 7 (b/c its read as utf-8 with warnings on Octave 6, who knows before that, maybe raw bytes).
And you need to Octave 8 to read it from the CWD,

The three BIST do these three cases in order.  Matlab is deferred for #259.


Updates for Alpha 7

This pull introduces the following aesthetic features/fixes:

Partition the list of exits by Registered/Viable/Nonviable and differentiate between connection and registration
Placeholder icons from react-fontawesome component library to indicate detailed connectivity status (to be improved following UX design)
Fix bug with settings not being fetched after selecting an exit that made the selection lag
Run prettier on the codebase and configure eslint
Wire up the wifi mesh toggle checkbox on the router settings page to the form state and put in a stub for the backend
Put the Althea logo in the navigation bar and highlight the currently selected page
Temporarily hide links to the Payments page

tested on my routers, works


feat: implement ast_builder for replace function

resolves #1274

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

33 of 33 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.005%) to 98.709%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
44105

Relevant Lines:
44682

💛  - Coveralls

I think I am ready for the review


Re-export ergonomics when using conversions

Because the From, TryFrom, TryInto conversions are only implemented for the primitive types in geo_types, you run into type mismatches if you then attempt to use functionality from geo, because it's all implemented on the re-exported types.
I have no idea how to easily fix this, but it's a big ergonomics issue – @frewsxcv any ideas?

Did you get this resolved @urschrei ? If I had to guess, it might be due to mismatched versions of geo-types dependencies

Yep, it was in conjunction with [patch]ed versions of either crate – if you specify HEAD from Github in cargo, you get type mismatches, whereas the published crates work fine together.

Ah yeah, that's always frustrating. Glad you got it sorted though!


Please discuss:  How does Server send notification to Clients?

SoftEther version: SoftEther VPN 4.27 Build 9666 Beta (April 21, 2018)
Component: [Server][Client]
Operating system: [Windows]
Architecture: [64 bit]
New Feature :  I try to add a notifications from server to  all/some clients , such as the server will change IP, or plan to stop for a while. But int the current architecture, server can only respond client request.  Based on current source code, I come for suggest how to accomplish  such feature?
Any advice and suggestions will be greatly appreciated.

@davidebeatrici  @fenice2 @dnobori  Could you give a help on this topic?   Though I have read the code for quite a lot of time ,  I can't find a right  and native way to solve this question.  Many thanks.


[service] various bugfixes

See comments description for details.

Is the point of this to allow restarts? I think these are supposed to be "start only once" services

Is the point of this to allow restarts?

Yes
Hmm. ok. Then we should probably rewrite https://github.com/tendermint/go-rpc/blob/master/client/ws_client.go#L51 to reconnect automatically or remove BaseService from it or implement onReset method because sometimes connections do fail and we need to deal with that.

Yes good point. In the network_testing I just did this at the consumer level https://github.com/tendermint/network_testing/blob/master/utils/transacter/transact.go#L117
Definitely would be better to have the client reconnect itself, or at least have that as an option. Seems like a good case for OnReset.


Change radio button color?

I looked through code and layouts but didn't see a way to change the list item radio button color. is this possible?

https://github.com/afollestad/material-dialogs/releases/tag/v<IP_ADDRESS>


SA1206 requires should appear before access modifiers

Using version 1.2.0-beta.435.
StyleCop wants the required modifier to appear before access modifiers.

From docs that's not the preferred order; the access modifier should come first.

Duplicate of https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3527. Already fixed and will be included in the next release.

Sorry @bjornhellander , I don't know why I couldn't find it. Thank you!

Duplicate of #3527


iOS device - "Pushy: Error retrieving handle for Flutter callbacks"

iOS device unable to listen for notifications when setNotificationListener is used.
I am not using firebase-messaging or any other push notification service, what's odd is my code was working yesterday but isn't working today with a similar setup.
I have set up my main file according to the documentation but its not working. So I set a notification listener in my home page that responds with error "Pushy: Error retrieving handle for Flutter callbacks".
SETUP in MAIN FILE
`
void backgroundNotificationListener(Map<String, dynamic> data) {
// Print notification payload data
print('Received notification: $data');
// Notification title
String notificationTitle = 'MyApp';
// Attempt to extract the "message" property from the payload: {"message":"Hello World!"}
String notificationText = data['message'] ?? 'Hello World!';
// Android: Displays a system notification
// iOS: Displays an alert dialog
Pushy.notify(notificationTitle, notificationText, data);
// Clear iOS app badge number
Pushy.clearBadge();
}
void main() async {
WidgetsFlutterBinding.ensureInitialized();
await Firebase.initializeApp();
Pushy.toggleInAppBanner(true);
//this listener doesn't respond at all, no errors and no notifications.
Pushy.setNotificationListener(backgroundNotificationListener);
//initLogic takes me to home page
runApp(initLogic());
}`
SETUP in HOME PAGE
` @override
void initState() {
print("listening");
Pushy.listen();
//here the setNotificationListener prints "Pushy: Error retrieving handle for Flutter callbacks"
Pushy.setNotificationListener((data) {
print('Received notification: $data');
  // Notification title
  String notificationTitle = 'MyApp';

  // Attempt to extract the "message" property from the payload: {"message":"Hello World!"}
  String notificationText = data['message'] ?? 'Hello World!';

  // Android: Displays a system notification
  // iOS: Displays an alert dialog
  Pushy.notify(notificationTitle, notificationText, data);

  // Clear iOS app badge number
  Pushy.clearBadge();
});

//...other stuff in initState
}`

Hi @GirishVerm,
We'd be glad to assist.
It is currently not possible to call Pushy.setNotificationListener() outside of the lib/main.dart file. It appears you are trying to call this method from within your home page widget. This is unsupported.
Please find a sample supported implementation in the Flutter Demo app's lib/main.dart.


Crawler failing to crawl

Crawler is failing on this website. Only finds one page and none of the rest. Any idea as to why?
http://www.doosanbabcock.com/

There's no links on the page to pick up — everything else appears in a script, which simplecrawler does not evaluate. My recommendation is to queue a sub resource you know works, or write your own discovery function that can parse the big script block. See crawler.discoverResources in the documentation. :)


feat: implement without C dependency

Go 1.21 introduced go:wasmimport, which means there's no need for using C for imports.

@egonelbre this is a great change, thank you! will give it a more thorough pass tomorrow but so far looks good

@bhelx & @zshipko,
I don't see anything blocking this - but would be great to have one more of us take a quick look!

Go 1.21 introduced go:wasmimport, which means there's no need for using C for imports.

Perhaps a naive question, I'm less familiar with this repo than @nilslice and @zshipko . Does this work in both Go and tinygo? Also, do we support host functions yet in this repo? Not seeing anything in the docs about it.

Overall though, yes I endorse this. Love seeing the advancements in Wasm support in the Go ecosystem and not relying on C stuff. I'm also okay with any breaking changes. We can bump this to a 1.0 pre-release version if we want to break the API

@bhelx yes, both tinygo and regular go should work. It probably would make sense to add tests for both compilers, currently it seems only tinygo is tested.


brave: gpu rasterization and hardware acceleration broken

Describe the bug
GPU rasterization and all other features concerning the hardware accelerated part of the browser are disabled and unable to be turned on.The brave://gpu page shows that no graphics processor is detected. This behaviour started with 1.29 version
Steps To Reproduce
Steps to reproduce the behavior:

Install the latest version of brave
Turn on the required flags (though on the other browsers this was not needed, except for video acceleration)
Check the gpu report in chrome://gpu

Expected behavior
GPU rasterization and hardware acceleration should work as expected.
Screenshots

Additional context
This issue only appears on brave. I tested chromium and vivaldi, both work as expected
Notify maintainers

@uskudnik  @rht  @JeffLabonte  @nasirhm
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
- system: `"x86_64-linux"`
 - host os: `Linux 5.14.1-zen1, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.4pre20210802_47e96bb`
 - channels(root): `"nixos-21.11pre313466.bc06c93905f, nixos-hardware"`
 - channels(danyeet): `"home-manager, nixos-21.11pre311431.88226ea038e"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

Did you also pass --ignore-gpu-blocklist?

Yes, it doesn't fix the issue.

I can reproduce this. Using Sway with Brave running on XWayland, amdgpu, kernel 5.10.62.
For whatever reason, I don't even get to see anything but a black screen when launching the browser by name or when restarting it from the button offered when changing a flag, but opening a link with xdg-open http://google.com allows me to start the browser in the same status as OP.

I also had the black screen issue, but removing my profile fixed it. Though that is a workaround, and it doesn't fix the main issue.

So now with the latest update on 1.30.86 gpu rasterization works just fine, but at least on my end video acceleration is still broken. I also tested chromium, it worked fine.

I can reproduce this. Using Sway with Brave running on XWayland, amdgpu, kernel 5.10.62.
For whatever reason, I don't even get to see anything but a black screen when launching the browser by name or when restarting it from the button offered when changing a flag, but opening a link with xdg-open http://google.com allows me to start the browser in the same status as OP.

Could you try running brave in native wayland and tell if you have gpu acceleration?
I am running it now native and everything is software rendered, unlike chromium

Could you try running brave in native wayland and tell if you have gpu acceleration? I am running it now native and everything is software rendered, unlike chromium

Same as you, I do not have gpu acceleration when selecting Wayland as my Ozone platform.
The original comment I posted is no longer valid however, I do not get a black screen anymore, but I am running kernel 5.17. I don't remember if that was a kernel (read: amdgpu) or a browser thing.

That seems more like a browser issue, I suspect it might be searching for some libraries
[225968:225968:0426/221402.534954:ERROR:egl_util.cc(74)] Failed to load GLES library: libGLESv2.so.2: libGLESv2.so.2: cannot open shared object file: No such file or directory

I know this is old, but see if this will help: https://github.com/NixOS/nixpkgs/pull/170655. At least for me everything seems to work correctly. I have brave overridden thus (for cmdline args):
brave.override {
  commandLineArgs = ''
    --use-gl=desktop \
    --ignore-gpu-blocklist \
    --enable-gpu-rasterization \
    --enable-zero-copy
  '';
}

I have been able to use gpu accel on brave, doing some changes to pkgs/applications/networking/browsers/brave/default.nix:
diff --git a/pkgs/applications/networking/browsers/brave/default.nix b/pkgs/applications/networking/browsers/brave/default.nix
index ed9f16066b0..7988d5b5957 100644
--- a/pkgs/applications/networking/browsers/brave/default.nix
+++ b/pkgs/applications/networking/browsers/brave/default.nix
@@ -17,6 +17,7 @@
 , gtk3
 , libuuid
 , libdrm
+, libGL
 , libX11
 , libXcomposite
 , libXcursor
@@ -66,7 +67,7 @@ let

   deps = [
     alsa-lib at-spi2-atk at-spi2-core atk cairo cups dbus expat
-    fontconfig freetype gdk-pixbuf glib gtk3 libdrm libX11
+    fontconfig freetype gdk-pixbuf glib gtk3 libdrm libGL libX11
     libxkbcommon libXScrnSaver libXcomposite libXcursor libXdamage
     libXext libXfixes libXi libXrandr libXrender libxshmfence
     libXtst libuuid mesa nspr nss pango pipewire udev wayland

this seems to fix it for the latest version of brave/default.nix.

@alexjp interesting, for me it works without that libGL – I've just ran some shaders from shadertoy and they are accelerated by the GPU. Is the difference in your case the Direct Rendering Display Compositor in about://gpu being enabled? Because I've just noticed it's Disabled for me, I must've mistaken it for raw draw ~~"

I am using wayland, I think on x11 I also get gpu accel. But on wayland, it only works with that change.
On wayland, this is what happens with that change:
with lib change:

without it (as it is on nixos-unstable ):

@alexjp can you check if this also fixes your issue? Seems a bit more correct than just passing libGL only
@@ -99,7 +101,7 @@ stdenv.mkDerivation rec {
   dontPatchELF = true;
   doInstallCheck = true;

-  nativeBuildInputs = [ dpkg wrapGAppsHook ];
+  nativeBuildInputs = [ dpkg wrapGAppsHook addOpenGLRunpath  ];

   buildInputs = [
     # needed for GSETTINGS_SCHEMAS_PATH
@@ -128,9 +130,11 @@ stdenv.mkDerivation rec {
       ln -sf $BINARYWRAPPER $out/bin/brave

       for exe in $out/opt/brave.com/brave/{brave,chrome_crashpad_handler}; do
-      patchelf \
-          --set-interpreter "$(cat $NIX_CC/nix-support/dynamic-linker)" \
-          --set-rpath "${rpath}" $exe
+          patchelf \
+              --set-interpreter "$(cat $NIX_CC/nix-support/dynamic-linker)" \
+              --set-rpath "${rpath}" $exe
+
+          addOpenGLRunpath $exe
       done

       # Fix paths

I know this is old, but see if this will help: https://github.com/NixOS/nixpkgs/pull/170655. At least for me everything seems to work correctly. I have brave overridden thus (for cmdline args):
brave.override {
  commandLineArgs = ''
    --use-gl=desktop \
    --ignore-gpu-blocklist \
    --enable-gpu-rasterization \
    --enable-zero-copy
  '';
}

It might also help for Vulkan (just override vulkanSupport to true), but it seems to break video acceleration then (maybe there's another magic incantation to fix it, but didn't yet find it).

The original issue might be a bit dated, but the issue is now still prevalent :).

@danyeet well, if you're on X, then you should be able to try the latest nixos-unstable with overrides like above to get it to work. At least it works for me with this fix. If you're on Wayland, then a bit more work is needed.

I do use wayland yes, thats why I commented again a few days ago.

@jaen
sorry.... i will test in a minute !! didn't had time yet, but will do right away!

@jaen
I tested your diff from here directly and the new brave-1.38.111 branch. both did not work :(
as more info, in the terminal, brave also complains:
[10445:10445:0504/185538.960769:ERROR:egl_util.cc(74)] Failed to load GLES library: libGLESv2.so.2: libGLESv2.so.2: cannot open shared object file: No such file or directory
[10445:10445:0504/185538.961822:ERROR:viz_main_impl.cc(185)] Exiting GPU process due to errors during initialization
[10406:10406:0504/185538.974423:ERROR:profile_attributes_storage.cc(848)] Failed to PNG encode the image.
[10406:10406:0504/185538.974450:ERROR:profile_attributes_storage.cc(848)] Failed to PNG encode the image.
[10406:10406:0504/185538.974459:ERROR:profile_attributes_storage.cc(848)] Failed to PNG encode the image.
[10462:10462:0504/185538.979591:ERROR:egl_util.cc(74)] Failed to load GLES library: libGLESv2.so.2: libGLESv2.so.2: cannot open shared object file: No such file or directory
[10462:10462:0504/185538.980913:ERROR:viz_main_impl.cc(185)] Exiting GPU process due to errors during initialization
[10520:10520:0504/185539.038538:ERROR:egl_util.cc(74)] Failed to load GLES library: libGLESv2.so.2: libGLESv2.so.2: cannot open shared object file: No such file or directory
[10520:10520:0504/185539.039535:ERROR:viz_main_impl.cc(185)] Exiting GPU process due to errors during initialization
[10528:10528:0504/185539.051423:ERROR:egl_util.cc(74)] Failed to load GLES library: libGLESv2.so.2: libGLESv2.so.2: cannot open shared object file: No such file or directory
[10528:10528:0504/185539.052313:ERROR:viz_main_impl.cc(185)] Exiting GPU process due to errors during initialization
[10536:10536:0504/185539.056579:ERROR:egl_util.cc(74)] Failed to load GLES library: libGLESv2.so.2: libGLESv2.so.2: cannot open shared object file: No such file or directory
[10536:10536:0504/185539.057413:ERROR:viz_main_impl.cc(185)] Exiting GPU process due to errors during initialization

Okay, I guess adding libGL it is. I don't quite understand what is different about it and since I don't really use Wayland at this time, I can't really check what's the problem here. I'll try to look at it later, for now it's important to have it fixed for the release.
@alexjp sorry to pester you again, I made a PR (https://github.com/NixOS/nixpkgs/pull/171561) out of it, can you double check the fix on the branch works the same for you?

Made a PR with just the libGL change instead, as there's apparently already a PR to update to 111: https://github.com/NixOS/nixpkgs/pull/171568

@jaen So sorry to keep you waiting!
Just tested the PR 171568, and its good!

@alexjp no problem, I use X anyway so I'm not kept waiting, but it's good to know this will fix issues for Wayland people to. Thanks for verifying the fix!

@alexjp  @danyeet it seems that this is now available in the unstable channel – http://nixpk.gs/pr-tracker.html?pr=171568 (alongside the 1.38.111 bump). Feel free to verify that your issue is still fixed there and we should be able to close this issue as fixed after close to a year ;' )

Yes, I think it's now fine on wayland (video acceleration doesn't seem to work but that seems like a chromium wayland issue and not nixos specific one).


OHM / Tasking Manager mods - master list

Ok - this is a blanket issue list to talk about related topics. Once finished, these may need to be broken into specific tickets.

Enable per-project time filtering in TM for OHM.
For example, if a team is working on mapping in a specific time range, add start_date & end_date to the TM Project profile and then pass that to the iD editor so that users aren't distracted by data from another era & the end users don't have to manually set up the time-filters in iD for every task.

Enable filtering out/removing all but the focus area data in the editor.
For example, if mapping roads, don't show anything but roads - no buildings, no natural features, no POIs, etc.

Enable a zoom-level lock at the TM project level.
For example, if the base map looked good at a specific zoom, but not more, the project managers might like to prevent zooming in further or out further to a) standardize the editing view, b) minimize editor / contributor distraction, c) expedite validation.

Enable embedding training videos on the project's intro / main page.
This may already be doable - @jeffreyameyer is to investigate.

iD can do date filters via the Map Features tab, but not sure if this hooked up to the URL to read dynamically. @danrademacher do you remember?

It's possible to pass presets or disable_features to iD to filter them https://github.com/OpenHistoricalMap/iD/blob/staging/API.md#url-parameters. This we would do as part of the URL, while starting the editor in TM. @willemarcel do you think this would be ok to implement on the TM side?

There's no easy way to do a zoom level lock right now. May be this is something we can handle though the mapping instructions? We can set the default zoom for the task.

cc @batpad @Rub21

It's possible to pass presets or disable_features to iD to filter them https://github.com/OpenHistoricalMap/iD/blob/staging/API.md#url-parameters. This we would do as part of the URL, while starting the editor in TM. @willemarcel do you think this would be ok to implement on the TM side?

It's already possible to restrict presets with TM. Disabling some features could cause some problems with overlapping features or duplication if there was something tagged wrongly.
Implementing this feature would not be complex. It would require adding a new column to the projects' database table and changing the TM API and frontend.

4. Enable embedding training videos on the project's intro / main page.
This may already be doable - @jeffreyameyer is to investigate.

It's not possible yet, but it's straightforward to implement and would be a very useful contribution to upstream TM.

Regarding this:

iD can do date filters via the Map Features tab, but not sure if this hooked up to the URL to read dynamically. @danrademacher do you remember?

Yes, we have URL params that work for that, see daterange=
https://www.openhistoricalmap.org/id#background=custom:https://tiles.allmaps.org/Fh2DUWGmMdGRBkGy/{z}/{x}/{y}.png&daterange=1900,1950&map=18.00/47.64067/-122.13667

I wonder if a more flexible, if less user-friendly, method would be to allow Projects to include key:value pairs for any param, and it's on the Project editor to get them right. So fields for id_param_name and id_param_value or similar, where as a Project Editor, I would enter daterange as a param name and 1900,1950 as a param value.

Yes, this is a good idea, and it can fit both OHM and upstream TM needs. I would prefer to call it something like extra_id_params, so you can enter not only one key value, but also a combination like #daterange=1900,1950#disabled_features=buildings.

That sounds really good to me @willemarcel @danrademacher. So to summarise, we would be proposing to make the following changes to TM:

Add a generic extra_id_params as a form or so, which is then passed to iD as URL params
Look at expanding the presets list in OHM TM for filtering (we shouldn't disable features because this is potentially dangerous)
Support for embedding videos on TM instructions

@jeffreyameyer @danrademacher @willemarcel does that sound good? If this is priority, we'll see how to schedule the tasks.

Looks good & yes to priority to accoplish. I think figuring out schedule
priority might depend on whether this is a quick thing or a big thing.
On Thu, Feb 24, 2022 at 5:04 AM Sajjad Anwar @.***>
wrote:

That sounds really good to me @willemarcel
https://github.com/willemarcel @danrademacher
https://github.com/danrademacher. So to summarise, we would be
proposing to make the following changes to TM:

Add a generic extra_id_params as a form or so, which is then passed
to iD as URL params
Look at expanding the presets list in OHM TM for filtering (we
shouldn't disable features because this is potentially dangerous)
Support for embedding videos on TM instructions

@jeffreyameyer https://github.com/jeffreyameyer @danrademacher
https://github.com/danrademacher @willemarcel
https://github.com/willemarcel does that sound good? If this is
priority, we'll see how to schedule the tasks.
—
Reply to this email directly, view it on GitHub
https://github.com/OpenHistoricalMap/issues/issues/338#issuecomment-1049837495,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALM4EV3WLC6DYHRBSXNOP3U4YUEVANCNFSM5PCPEAJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Jeff Meyer
206-676-2347
osm: Open Historical Map (OHM)
http://wiki.openstreetmap.org/wiki/Open_Historical_Map / my OSM user page
http://www.openstreetmap.org/user/jeffmeyer
t: @OpenHistMap

Great stuff! Can't wait to use them

That sound really good! Here is some thoughts about what Jeff mentioned :

Defining a specific time range for TM Project profile will allow to show just the data tagged between this predefined start_date & end_date. It would be really helpful if users won't have to set up themselves the time-filters for every task!
It's also important to mention that for some cases the data is not date tagged within a time range (or tagged wrongly) and still show even after defining a specific time range. In this case how we can filter these features?

About disabling some features,  users mapping roads end up disabling other features manually. They are usually confused by when all features are enabled and by not being able to have a clear perception of the historical map in the background. Also sometimes overlapping different features end up showing error messages for example about missing intersection between a road and a river (which may not be really needed) and users don't know what is the right thing to do.
Validators instead can first focus on the focus data of the project, and then  enable all features to check eventually problems with overlapping features or duplication if there was something tagged wrongly.

(*) Look at expanding the presets list in OHM TM for filtering (we shouldn't disable features because this is potentially dangerous)

and/or expand presets list with custom OHM presets (i.e. historic-only features) in iD and TM (as well as documenting and translating them) if possible.

@danieldegroot2 - this is a pretty interesting idea, but to be clear... there are no 'historic' features in OHM - there should just be objects (relation, way, node) that exist in points of time & most objects should have some sort of start_date and/or end_date tags. Does that make sense? I'm sure we could come up with some other interesting OHM filters, too!

Circling back on this issue, right now the work we're undertaking will cover the three items in this comment:
https://github.com/OpenHistoricalMap/issues/issues/338#issuecomment-1049837495

Add a generic extra_id_params as a form or so, which is then passed to iD as URL params
Look at expanding the presets list in OHM TM for filtering
Support for embedding videos on TM instructions

As I understand it, extra_id_params and “support for embedding videos on TM instructions” would be changes to upstream TM and then we move to that and all TM users get the benefit.
I am less clear about the middle bullet point and where presets are stored, and what we're going to add to them. @geohacker Do we do that in our TM, our iD, or some other place?

@danrademacher the preset list filter comes from TM and then gets passed into iD. In an ideal world TM and iD would share the list but that's not the case now. So we'll look at adding the presets we want to filter by in TM.

Thanks @willemarcel ! The following changes are on staging: https://tasks-staging.openhistoricalmap.org/
Tasking Manager updates
Extra iD Params
The iD API supports some query parameters in the URL that makes it possible to load the editor with additional configuration. We can now configure it in the Metadata section of the project edit page.
For example:

Loads the iD editor with:

A date range set in the photo overlays section (activate Mapillary to see it):

the buildings layer disabled:

and, finally, an offset to the background imagery:

Embedding YouTube videos
Enable embedding YouTube videos in project's description and instructions. The sintax is ::youtube[video-id], it needs one line of spacing from other content (both up and down) to work.
In the Project edit page:

In the Project task's page:

Amazing!

Fantastic! Thank you @willemarcel and @batpad for you making this happen!

I also just went in and comfirmed that the date params are working as expected:

That's at https://tasks-staging.openhistoricalmap.org/projects/12, where I set the URL params to be daterange=1901,1950

In Project 23 of the ANR Communes' projects (near task 221), I found a bug related to the date filter. When the project is opened there are two primary roads shown in the first Image. When the date filter is opened (nothing is changed), one of the roads disappears as seen in the second image. The road that disappears should not be there according to the date range but is always present when a task is opened.

Closing this out since we resolved the permissions issue here


Manually getting/generating API key

Hey there!
I haven't been able to find a solution for this issue: https://github.com/YOURLS/YOURLS/issues/1975
So as an alternative, is there any way to get manually the API key from an account?
Or any way to generate an API key?
Thanks!

Is yourls_auth_signature() what you're looking for ?

There we go, thank you!


GameOver Screen: "Exit to Menu" button doesn't work

Issue description:
It looks like a not implemented error
Fix can be included in #105

I was able to get the 'Exit to Menu' button to work. Currently pushed to feature/options. I'll make a PR for it once the requirements for #105 are implemented.
Ended up not being too bad. main.end_game() will now function to return everything to main menu. Though, it does throw out some errors in the debugger that will need to be cleaned up eventually.
If errors are not fixed alongside this issue, a new issue should be created.


Incorrect messaging for expired credit card errors

Describe the bug
When we try to charge an expired card, it fails and we send the following email to users:

They then end up on the confirmation page where they're prompted with an error message and no path to resolve the situation:

Expected behavior

The messaging for errors due to expired cards should be different and make it clear that the card has expired
It should link to the the dashboard, where users can update the payment method
The confirmation page should provide a fallback link to the dashboard when it's not able to confirm it, to let users update it instead

The error returned by Stripe is the same as a 3D secure request, so the code is not faulty there.
In order to fix the API, we'll need to fix the expiryDate for credit cards: https://github.com/opencollective/opencollective/issues/7642. Then in https://github.com/opencollective/opencollective-api/blob/ee99b03a910d4d02a7322cc83e3f0d2566bb5ae5/server/lib/recurring-contributions.js#L135, if the error happens on an expired credit card, cancel/pause the subscription and create a special "Card expired" activity / send a special email.
For now, I'll update the frontend to make sure people who end up on the confirmation page have a way out.

Fixed last bit in https://github.com/opencollective/opencollective-frontend/pulls?utf8=✓&q=+is%3Apr+author%3ABetree+


change /login url to /omk/login

Solves https://github.com/hotosm/OpenMapKitServer/issues/35

This should be relative. When standalone it'll be on /login, but in some circumstances it will have a prefix (which may be /omk, but not necessarily).

@mojodna would you mind reviewing it again, please?

Looks great; I'm assuming that this works in development, yeah?

This still doesn't work when mounted under /omk.

2 reasons:

login is relative to /omk/pages/#/login, so it's effectively /omk/pages/login
frontend/build has been checked in and is out of date.

 I misunderstood part of how this was deployed; it's not actually "mounted on /omk", it's that only /omk routes are proxied through to the API (with a couple of exceptions to support ODK, e.g. /formList). This is why / redirects to /omk/pages/. All of my nonsense about "relative" routing wasn't helpful.
New routes under / won't be accessible when running under a proxy configuration like this, so the original commit in this PR is correct.
Proposed fix:

revert 1273304
apply current-user patch (below) to allow the frontend to request the current user under ^^ proxied environments
apply the Dockerfile patch (previous comment)
rebuild + commit frontend/build (or remove + gitignore it from the repo)

current-user patch:
diff --cc frontend/src/network/auth.js
index 3574231,3168922..0000000
--- a/frontend/src/network/auth.js
+++ b/frontend/src/network/auth.js
@@@ -1,5 -1,5 +1,5 @@@
  export function currentUser(): Promise<*> {
-   return fetch(`/current-user`, {
 -  return fetch(`../current-user`, {
++  return fetch(`/omk/current-user`, {
      method: 'GET',
      headers: {
        'Content-Type': 'application/json'
diff --cc frontend/src/store/actions/auth.js
index d17fec0,f9e2ffa..0000000
--- a/frontend/src/store/actions/auth.js
+++ b/frontend/src/store/actions/auth.js
diff --git a/index.js b/index.js
index 7d3ec80..496ae28 100644
--- a/index.js
+++ b/index.js
@@ -111,7 +111,7 @@ app.get('/', redirectToForms);
 app.get('/omk', redirectToForms);
 app.get('/omk/info', info);

-app.get('/current-user',
+app.get('/omk/current-user',
   function(req, res) {
     if (req.user) {
       res.json({ username: req.user.username, role: req.user.role});

Replaced it by https://github.com/hotosm/OpenMapKitServer/pull/42
I didn't change the /current-user, because it is not being used by the frontend and it was replaced by /omk/auth-status/ on https://github.com/hotosm/OpenMapKitServer/pull/41


Add requirement for service default params

This was more or less implied, but I've had a few questions about it from engineers new to our team, so I wanted to codify it.

I talked to @JeffreyRichter and @johanste about this already. Also /cc @KrzysztofCwalina @bterlson @JonathanGiles


institution_id registration of UTAS

institution_id = UTAS
institution = Institute for Marine and Antarctic Studies, University of Tasmania, Hobart, Tasmania, TAS 7001, Australia

You're right, that's a redundancy. Please contract "Tasmania, TAS 7001" to
"Tasmania 7001".

@sjphipps did you want a duplicate "..Tasmania, TAS.." in there?

--
Dr Steven J Phipps
Paleo Ice Sheet Modeller
Antarctic Gateway Partnership
Institute for Marine and Antarctic Studies
University of Tasmania
Private Bag 129, Hobart, TAS 7001, Australia
Tel  +61 3 6226 6902
Fax  +61 3 6226 2973
https://www.stevenphipps.com
https://twitter.com/StevenJPhipps
University of Tasmania Electronic Communications Policy (December, 2014).
This email is confidential, and is for the intended recipient only. Access, disclosure, copying, distribution, or reliance on any of it by anyone outside the intended recipient organisation is prohibited and may be a criminal offence. Please delete if obtained in error and email confirmation to the sender. The views expressed in this email are not necessarily the views of the University of Tasmania, unless clearly intended otherwise.

@sjphipps UTAS is now registered, please review CMIP6_institution_id.html and let me know of any required amendments


Use syntax highlighter for traceback on resource log view

Description
closes #3241
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Code is clear and sufficiently documented
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[x] ~End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )~

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches iso5, master in e5709c950b402c094d8cf4bac933bc5fd6ad069f


Update retries.md

HedingDelay does not accept the null  and also does not have default value. Setting Hedging Delay to null will throw InvalidOperationException.

Updated to say that the value is optional (will be fixed in the next gRPC release that will come out this week)

@JamesNK ...

will come out this week

Do we need to react here and now with some aspnetcore-6.0-versioned text or open a followup issue for this topic for 6.0?

 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.:x: VahidFarahmandian sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@VahidFarahmandian  my change pushed this over the limit. Can you sign the CLA. We're expecting more nice PRs like this so good to get that out of the way.

@Rick-Anderson signing the CLA done.

@JamesNK ...

will come out this week

Do we need to react here and now with some aspnetcore-6.0-versioned text or open a followup issue for this topic for 6.0?

No, gRPC is released independently.


Make License Expression links more direct

Before this change, these links would go to a page that lists the versions of the SPDX Spec. In order to get information about license expressions, the users would have to click the link, find the latest version of the Spec, select the HTML version and click on Annex D. This change makes it so that the link goes directly to Annex D.

we don't want to make this change as it's specific to 2.3 and then will get (is already) stale with new releases.
That being said, linking to the same info in the new 3.x spec format may require a different link update, but we can address that in the next round.
@goneall - for awareness (and to close PR)


Use 4.2.0 with<EMAIL_ADDRESS>not a valid Win32 application

After upgrade to 4.2.0, there is no binary for download, by building on local machine, the output file is not working on win
https://github.com/mapbox/node-sqlite3/issues/910#issuecomment-510104260

#1324 merged but not released yet.

You need to build it yourself.
There are many steps, but they are not cumbersome.
If you want to know more, please send me email<EMAIL_ADDRESS><EMAIL_ADDRESS>


Passing argument to description

I couldn't find an option that I needed to use command line argument as a part of description and eventually as a part of test like this:
describe("Application", function() {
  it("creates entity ${n} times", n, function() {
  ..
  });
  it("updates entity ${n} times", n, function() {
  ..
  });
});

@canmingir
You can pass node env variables from CLI. i.e.
> env TITLE="our title" mocha
describe('Suite', () => {
  it(`test name ${process.env.TITLE}`, () => { // "test name our title"
    // ...
  });
});

Does that solve your problem?

Great, I was also be able to use local variables
var n = argv.n ? argv.n : 2;
it(`creates entity ${n} times`, ..)


support epoch versions

Debian supports version epochs separated by colon (:). Such epoch versions
only appear in the debian/changelog files, but are not visible in filenames.
This change strips out epoch versions where needed.
Signed-off-by: André Roth<EMAIL_ADDRESS>

Doesn't that introduce version collisions when the epoch is different, but the rest the same?

Doesn't that introduce version collisions when the epoch is different, but the rest the same?

yes, I guess so. But packages in Debian repos do not contain the version in the filenames, even if the epoch is set.


Initial  commit

/cc @serngawy @imiller0
I started to put up some stories and use cases for ACM enhancement proposal.
Opening this for comments about the format, content, ...
Cheers!

Is this at least partially implemented by: https://github.com/open-cluster-management-io/enhancements/pull/67


Buypoints options

Currently I have in mind only these

PayPal
PayGol

I am not sure if more are needed like for example

PagSeguro
ContenidoPago

I will only support PayPal. SMS providers arent a good deal. If anyone wants SMS do it your own


Update nginx-agent.conf for grafana example

Proposed changes
Update nginx-agent.conf for grafana example
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING document
[ ] I have run make install-tools and have attached any dependency changes to this pull request
[ ] If applicable, I have added tests that prove my fix is effective or that my feature works
[ ] If applicable, I have checked that any relevant tests pass after adding my changes
[ ] If applicable, I have updated any relevant documentation (README.md)
[ ] If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (394a3fe) 66.28% compared to head (006f306) 66.43%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #489      +/-   ##
==========================================
+ Coverage   66.28%   66.43%   +0.15%
==========================================
  Files         118      118
  Lines       13382    13382
==========================================
+ Hits         8870     8891      +21
+ Misses       3921     3900      -21
  Partials      591      591

Files
Coverage Δ

src/core/nginx.go
43.81% <ø> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


add tool for merge physics database to logic database.

What would you like to be added:
Why is this needed:
when user execute show databases,  we should show the tenant's logic databases to user, and if the the SQL is the show databases where name=xxxxx with condition, we should filter the logic databases.
https://dev.mysql.com/doc/refman/8.0/en/show-databases.html

dnt forget show tables like 'ab_defg%' 🤣

@dongzl @jjeffcaii  If no one does this, assign to me pls


Drop conda_build_config.yaml

The conda_build_config.yaml was added here to handle ARM SBSA:
https://github.com/conda-forge/cutensor-feedstock/blob/ee4d11f6a02afe7a06b25b917bcbce82b9b5713e/recipe/conda_build_config.yaml#L1-L2
However the logic here has been included in conda-forge-pinning
So the logic can be dropped. As this is the only thing in conda_build_config.yaml, the file itself can be dropped completely
Hopefully simplifies things here going forward

Why is arm_variant_type pinned channel wide? I seems to me that this should be something like the mpi metapackage where it is recipe specific? Don't feedstocks which build for both sbsa and tegra still need to overwrite this pinning?


InvertIfCodeRefactoringProvider: InvalidOperationException on nested "if ... if ... else"

The code
    class TestClass
    {
        int TestMethod(int a)
        {
            if (a > 0)
                if (a < 5)
                {
                    return 1;
                }
                else
                {
                    return 2;
                }

            return 0;
        }
    }

and Ctrl+. at if (a < 5) makes InvertIfCodeRefactoringProvider crash with
An exception of type 'System.InvalidOperationException' occurred in Microsoft.CodeAnalysis.CSharp.dll but was not handled in user code
Additional information: The item specified is not the element of a list.
at
https://github.com/icsharpcode/RefactoringEssentials/blob/master/RefactoringEssentials/CSharp/CodeRefactorings/Synced/InvertIfCodeRefactoringProvider.cs#L44

fixed.


Fix tooling section link

Hey folks! This is a teeny-tiny PR to fix a broken link in the Tutorial section. The link to "Tools" was still going to the homepage.
Cheers! 🍻

Thanks! The tools section used to be on the homepage must've forgot about this link.


Create a Content Card

Here is a reference to the component-->

I have created a PR as well.

Can you assign @rupsha014


Peer dependency to<EMAIL_ADDRESS>

Hi!
When installing npm i @lumaai/luma-web aside<EMAIL_ADDRESS>I got the following error:
Could not resolve dependency: npm WARN peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>This is because your package.json (https://www.npmjs.com/package/@lumaai/luma-web?activeTab=code) :
  "peerDependencies": {
    "@types/three": "^0.157.2",
    "three": "^0.157.0"
  }

Can you update it to "three": ">=0.157.0" or "three": "*"
For thoses who cannot wait, bad workaround could be npm i @lumaai/luma-web --force
Thanks!

I also need desperately an update to latest three. Is there an ability to just fix it myself, just let me know what to change. Thanks in advance.


Moves onclick handler

A side navigation item with target "_blank" would collapse the menu section.
Fixed it by moving the onclick handler on the section header.

Codecov Report

Merging #811 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #811      +/-   ##
==========================================
+ Coverage   99.76%   99.76%   +<.01%
==========================================
  Files         270      270
  Lines        6937     6941       +4
  Branches     1692     1693       +1
==========================================
+ Hits         6921     6925       +4
  Misses         16       16

Impacted Files
Coverage Δ

...nents/sideNavigation/SideNavigationMenuSection.tsx
100% <100%> (ø)
:arrow_up:

...components/sideNavigation/SideNavigationHeader.tsx
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Wrong result of check when running in Jenkins

Describe the bug
When I run dependency-check for the same project locally (Mac) I get 0 vulnerabilities, whereas I get 380 when I run it in Jenkins via the plugin.
Version of dependency-check used
The problem occurs using version 6.3.1 of Dependency-check.
Mac OS locally. Command 'dependency-check --scan target --disableAssembly'
Jenkins version 2.277.3.
The command to trigger dependency-check in Jenkins is:
dependencycheck additionalArguments: '--data /dependency-checker-db-6 --scan target --disableAssembly --noupdate --suppression owasp/suppression.xml --format XML --format HTML --log dependencyCheckLog.txt', odcInstallation: 'dependency-check-6.3.1
Log file
Log from local run (ok):
https://gist.github.com/haugsrud/1e03197bab9e287d4f1d3bd9c4378cf0
Log from Jenkins run (not ok):
https://gist.github.com/haugsrud/9e1f86ab931d6ea89535a9a1b1c1315c
To Reproduce
Steps to reproduce the behavior:

Run the project locally
Get expected result with no vulnerabilities
Run the project in Jenkins
Get 380 vulnerabilities which I believe is all false positives

Expected behavior
I expect the result to be like this, which is the one I get when run locally:

But when running in Jenkins I get this:

Additional context
I want to know how Dependency-check works in regard to running a pipeline in Jenkins. How does it pick up use of Jenkins and hence report issues that don't exist when I run it locally?
EDIT:
This is a multi-module maven-project. I think this might cause some of the issues as other projects who are not multi-module run fine in the same pipeline.

You've set up to scan a directory; is the contents of the directory exactly the same on both platforms?
in any case it's better to use one of the options that uses a dependencymangement (maven, gradle,..) instead of the cli

This is not an ODC issue- rather the issue is on jenkins. Whatever is being placed into the "target" directory is being scanned.  The logs you posted - are not the actual logs from ODC that we would need to debug this further. We would need the --log dependencyCheckLog.txt file.

Thank you for your answer, I guess my issue came from this being the first project not being a WAR and that caused the issue since I were using the CLI.
I have changed my pipelines to use the Maven plugin for all maven projects. I'll close this issue.


Fix missing ID with JAX-RS and DSTU2 RI

While using DSTU2 reference implementation, the ID was not correctly initialized in the request object

Coverage decreased (-0.007%) to 87.692% when pulling cf6474dd7bdaef9330052f405852505c865f00ed on mion00:fix-id-dstu2org into f6601cc347730e57ff8e91c7a04601b5d3fc40af on jamesagnew:master.

Hi Carlo, Thanks for the contribution! Looks great, merging it now.


[SR-13383] Windows, Dispatch crashes after network request timeout

Previous ID
SR-13383

Radar
None

Original Reporter
@lxbndr

Type
Bug

Attachment: Download

Environment
Swift version 5.3-dev (LLVM 8cbcb68709, Swift df007a4f27)
Target: x86_64-unknown-windows-msvc
libdispatch and Foundation: swift-DEVELOPMENT-SNAPSHOT-2020-08-11-a,

Additional Detail from JIRA

Votes
0

Component/s
Foundation, libdispatch

Labels
Bug, Dispatch, Foundation, Windows

Assignee
None

Priority
Medium

md5: 49101804f13730f8eb31fad5750c4923

Issue Description:
Foundation uses CURL for processing network requests, and DispatchSource for listening to socket read/write availability events.
When network request from Foundation times out, DispatchSource is invalidated implicitly (by releasing and deallocating it).
libdispatch then performs cleanup for dispatch source in background thread. The cleanup includes disarming all pending events on a socket. But socket is already invalidated at that moment, and operation fails (WSAEventSelect returns error 10038 - WSAENOTSOCK).
Such error is considered fatal, and libdispatch intentionally crashes.
Sample code
import Foundation
import FoundationNetworking
import XCTest

class TestURLDataTask: XCTestCase {
    func test_timeout() {
        let config = URLSessionConfiguration.default
        config.timeoutIntervalForRequest = 8 // Value doesn't matter, 8 is just to wait less
        let session = URLSession(configuration: config, delegate: nil, delegateQueue: nil)
        let request = URLRequest(url: URL(string: "http://<IP_ADDRESS>")!) // use any host that would time out

        let callbackCalled = expectation(description: "Task callback called")
        session.dataTask(with: request, completionHandler: { data, response, error in
            callbackCalled.fulfill()
        }).resume()
        waitForExpectations(timeout: 10)
        Thread.sleep(forTimeInterval: 5)  // as Dispatch crashes asynchronously, give it some time
    }

    static var allTests: [(String, (TestURLDataTask) -> () throws -> Void)] {
        return [("test_timeout", test_timeout),]
    }
}

XCTMain([testCase(TestURLDataTask.allTests)])

Crash location

The crash is in event_windows.c:222
Other dispatch sources
Other dispatch_source implementations (epoll, kevent) also has failable API calls in cleanup code, but intentionally ignore any errors.
It seems to me that socket cleanup have to be more indifferent to WSAEventSelect errors. Or we could ignore WSAENOTSOCK error at least.
Another solution would be somehow do dispatch source cleanup synchronously before socket invalidation in Foundation. Looks like this location in MultiHandle.swift would be a good place to do that. But I don't see any possibility to do it. Available Dispatch API would just mark Source as pending for asynchronous cleanup.

@compnerd Swift 5.8 release still does this

@gregcotten 5.8 is ~1y old, there is a pretty long delay in releases for stability.  Any fixes for this would be post 5.9.  I think that @lxbndr had dug further and made some progress but I don't think that there was a patch for it yet.

@compnerd, got it! Network request that results in a timeout still causes a crash on April 20, 2023 trunk dev build - compnerd.org Swift version 5.9-dev (LLVM 107de6a91bc1dbd, Swift 022311e438e7703)

Relevant: https://github.com/apple/swift-corelibs-libdispatch/pull/772


Opinions and Edits

Removing "Great For" on something required.  It sends a mixed message for newbs.  Changed up flow for "How to Use" section.  I feel this reads better for me.
Also, includes the secrets of -, ., and backspace in readme.

Ha.  My precious secrets..... no.....


Browser history is broken after navigation is blocked when leaving a Workspace with changes on KIE Sandbox

Reproducing this issue:

Open KIE Sandbox
Import a repository containing multiple files supported by KIE Sandbox (e.g. https://github.com/kiegroup/kogito-examples)
Change something in a file
Click the browser's "Back" button
Selected "Proceed anyway"
Observe you're redirected to the Home page, but the browser does not enable the "Forward" button, only the "Back" button is enabled.

Expected:

When pressing the "Back" button, it should be possible to return to where you were by using the "Forward" button.

Occurring because when clicking on proceed anyway in online-editor\src\editor\Toolbar\Workspace\Hooks.tsx, it's performing a history.push(lastBlockedLocation) and therefore going forward in the history stack. The back button in the toolbar, and clicking on the home page are performing as intended, so this fix should only change the back button functionality. Ideal solution would be to unblock the navigation and resend the transition onclicking proceed anyway.


unit-tests: check hwmc error codes

Verify error codes returned by hwmc bad commands.
This test relevant to D457 MIPI driver version.
Expected: d4xx driver versions
above <IP_ADDRESS> - pass
below <IP_ADDRESS> - fail
Tracked-by: LRS-1065

We need to update the driver to dev driver and retrigger the test to see it works


Create a texture strategy similar to FontData.

As @cies said

I can also imagine its great to have FillTexture take a DynamicImage or even a GLuint (pointing to the texture in GL'istan).

Just like fonts, there are many different kinds of textures. We should have gelatin-picture use a texture strategy similar to the way it uses FontData. That way the implementation details (GLuint or DynamicImage) aren't leaked in the top level API. This TextureData should be implemented in gelatin-core.
I'm thinking something like
data TextureData = TextureData
    { textureShow :: String
    , textureHash :: Int -> Int
    , ...
    }

And then have the backend provide a function that maps a texture to a TextureData. I'm not certain of what other functions would be needed in the TextureData record though.

The alternative is to encode the texture type in the type of Picture like was previously done with Font.


Read from kafka from the very beginning.

Not an issue. We have a use case that we need to read all kafka messages from the beginning. What Secor configuration allow me to do that?

This config
kafka.consumer.auto.offset.reset
corresponds to kafka's
auto.offset.reset
https://kafka.apache.org/documentation/
https://kafka.apache.org/documentation/
You can set the value to 'smallest' to read from the beginning initially
On Wed, Jul 11, 2018 at 12:32 PM, kchew534<EMAIL_ADDRESS>wrote:

Not an issue. We have a use case that we need to read all kafka messages
from the beginning. What Secor configuration allow me to do that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/secor/issues/438, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHCX_8MDX3cLLRxlSQCDP65kRmT6fNKKks5uFlLkgaJpZM4VLt6O
.


Does this only work with a subset of models or with any mercedesme capable vehicles?

I have a 2017 C300 that works connected through the mercedes me android app. Door lock/unlock, remote start, all that stuff.
But this component says "no cars found" in the hass logs.
So I'm wondering if this only works with a specific list of models/years or what

In what region do you live? In Europe it should work (Android/IOS app - MercedesMe). In the US you need a 2019 model (App Name "MercedesMe connect" and not the older "MercedesMe" app).


Couch soft crashes and restart fails to bind to port

Description
CouchDB service is running but has failed to bind to port even after restarting the service
Steps to Reproduce
install on Debian Buster, setup with standard dedicated database
Expected Behaviour
Server to be active, bound to port and reporting well when I run status
Your Environment
NGINX setup for front websites
the port is open, have used CouchDB web panel before
after possible crash or restart - software will not bind to port
this is the second time it has happened, last time it was fixed by resetting the admin password.

CouchDB version used: 3.2.0
Browser name and version: ANY
Operating system and version: ANY

Additional Context
I can't seem to get logging to log anything
output from console:

admin@server:/opt$ sudo netstat -tulpn | grep LISTEN
tcp        0      0 <IP_ADDRESS>:443             <IP_ADDRESS>:*               LISTEN      5972/nginx: worker
tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN      563/sshd
tcp6       0      0 :::2080                 :::*                    LISTEN      6854/node
tcp6       0      0 :::4080                 :::*                    LISTEN      6846/node
tcp6       0      0 :::22                   :::*                    LISTEN      563/sshd

admin@server:~$ /etc/init.d/couchdb status
● couchdb.service - Apache CouchDB
   Loaded: loaded (/lib/systemd/system/couchdb.service; enabled; vendor preset: enabled)
   Active: active (running) since Sun 2022-03-13 04:51:12 UTC; 15ms ago
 Main PID: 32309 (couchdb)
    Tasks: 3 (limit: 2330)
   Memory: 612.0K
   CGroup: /system.slice/couchdb.service
           ├─32309 /bin/sh /opt/couchdb/bin/couchdb
           ├─32317 /bin/sh /opt/couchdb/bin/couchdb
           └─32319 /bin/sh /opt/couchdb/bin/couchdb

admin@server:~$ curl http://<IP_ADDRESS>:5984/
curl: (7) Failed to connect to <IP_ADDRESS> port 5984: Connection refused

I am experiencing this when I upgraded to 3.2.2-2 (standalone). I get a
"Protocol 'inet_tcp' : register/listen error: econnrefused"
error in journalctl on the couchdb service. I downgraded to 3.2.2-1 and everything works fine. The issue is on a Linode server. I did do an lsof -i -P -n, and there was nothing bound to port 5984. After I downgraded I did the lsof again, and it was bound as it should be. On this server, I have IPV6 disabled (I was having issues with the javascript nano library connections).
Arch Linux - kernel 5.18.2

Hi, so it's an Arch Linux behavior?

I put a bug post into arch linux but no one has responded to it yet. I would say the epmd issue not using the environment variable as described is not an arch issue. Given that it is <IP_ADDRESS>, it should not be trying with ::1

I had this issue on Debian Buster as well - in a production environment.
was catastrophic
On Tue, Jul 5, 2022 at 9:42 AM trames57 @.***> wrote:

I put a bug post into arch linux but no one has responded to it yet. I
would say the epmd issue not using the environment variable as described is
not an arch issue. Given that it is <IP_ADDRESS>, it should not be trying with
::1
—
Reply to this email directly, view it on GitHub
https://github.com/apache/couchdb/issues/3958#issuecomment-1174461771,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSWNFEM7CNLAWH5P7KFTLLVSNZH3ANCNFSM5QS3OWRA
.
You are receiving this because you authored the thread.Message ID:
@.***>


provide some more motivation

Sorry for only a tiny piece, but didn't have much time today. But I hope it helps
unfortunately, I first created a branch that I merged into my fork. Need some practice with forked repos.
It might be easier to give me write access to your repo, if you like.

I'll read through this in a bit. I'll also set up CODEOWNERS so I get notifications and also practice better hygiene on main.
Looking at the examples in light of # your comment, yes, more context would help. Essentially these are the sorts of things I see written today to work around not having T& support.
Read my last update on shallow const. If you are OK, where this behavior matches Boost, let me know, and I'll give you commit access and list you as a coauthor.
Otherwise I'll just give you commit.

amended as asked for.


Works fine, but formatting is being ignored.

Works fine, but the following def function is being ignored
    def remove_dashes(self):         page_name = str(item)         return self.page_name.replace('-',' ')

That function does not exists in this repo.

Ah ok, I must be suffering from that mental illness that causes you to have visions to non-existent code then. Awful disease, that....