[Bug]: cannot ensure that projection is up to date

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[ ] I have joined the ZITADEL chat

Environment
Self-hosted
Version
2.62.1
Database
CockroachDB
Database Version
23.2.5
Describe the problem caused by this bug
Similar to issue #5602, request to https://auth.caomingjun.com/oauth/v2/token encountered an 500 error when logging in. Response is:
{"error":"server_error","error_description":"cannot ensure that projection is up to date"}

Server log:
time=2024-09-23T08:07:51.818Z level=ERROR msg="request error" oidc_error.parent="ID=OIDC-ASfh3 Message=cannot ensure that projection is up to date" oidc_error.description="cannot ensure that projection is up to date" oidc_error.type=server_error status_code=500

Cockroachdb didn't report any error in its log.
To reproduce
try to login to console in browser.
Screenshots
No response
Expected behavior
No response
Operating System
Server runs on docker (ghcr.io/zitadel/zitadel:v2.62.1 and cockroachdb/cockroach:v23.2.5). I am using Firefox 130.0.1 on Windows to try to access the console.
Relevant Configuration
No response
Additional Context
Referring to the solution of #5602, I tried to find in the document whether there is a requirement to update Cockroach DB, but I did not find anything.

Rolling back Zitadel to 2.61.1 resolved the issue. I suspect the new version requires a database update but does not mention it in the documentation.

I updated crdb to the latest version (24.2.1) but i also have this issue

Nevermind, zitadel v2.62.1 is working with my crdb instance v24.2.2

We've changed / fixed a precision error right around here that might be the cause of the bug. We will investigate further.

@cmj2002 can you please inspect the state in the database by running in a postgres / cockroach shell:
with es as (
	select aggregate_type, aggregate_id, "position"
	from eventstore.events2
	where instance_id = '<INSTANCE_ID>'
		and aggregate_type = 'key_pair'
)
select p.aggregate_id, es.position as es_position, p.position as projection_postion
from projections.current_states p
right join es
	on es.aggregate_type = p.aggregate_type
	and es.aggregate_id = p.aggregate_id
where instance_id = '<INSTANCE_ID>'
	and p.projection_name = 'projections.keys4'
	and p.aggregate_type = 'key_pair';

Change <INSTANCE_ID> to the instance ID on your system.

For context:
We recently changed the position type from float64 to decimal in the Go code, because cockroach uses a higher precision. I suspect the on an older version we reduced events and lost precision in the process. Now with the decimal type we notice the loss of precision in the check that throws the error.
Old version:
| eventstore | reducer | projection |
| numeric    > float64  > numeric    |

v2.62
| eventstore | reducer | projection |
| numeric   > decimal  > numeric    |

We encountered this as well on Zitadel 2.62.1 and Cockroach 24.1.1.
Rolling back to 2.61.1 worked for now.

@cmj2002 can you please inspect the state in the database by running in a postgres / cockroach shell:
with es as (
	select aggregate_type, aggregate_id, "position"
	from eventstore.events2
	where instance_id = '<INSTANCE_ID>'
		and aggregate_type = 'key_pair'
)
select p.aggregate_id, es.position as es_position, p.position as projection_postion
from projections.current_states p
right join es
	on es.aggregate_type = p.aggregate_type
	and es.aggregate_id = p.aggregate_id
where instance_id = '<INSTANCE_ID>'
	and p.projection_name = 'projections.keys4'
	and p.aggregate_type = 'key_pair';

Change <INSTANCE_ID> to the instance ID on your system.

It gives
root@localhost:26257/zitadel> with es as (
                           ->     select aggregate_type, aggregate_id, "position"
                           ->     from eventstore.events2
                           ->     where instance_id = '224159003251310595'
                           ->         and aggregate_type = 'key_pair'
                           -> )
                           -> select p.aggregate_id, es.position as es_position, p.position as projection_postion
                           -> from projections.current_states p
                           -> right join es
                           ->     on es.aggregate_type = p.aggregate_type
                           ->     and es.aggregate_id = p.aggregate_id
                           -> where instance_id = '224159003251310595'
                           ->     and p.projection_name = 'projections.keys4'
                           ->     and p.aggregate_type = 'key_pair';
     aggregate_id    |          es_position           | projection_postion
---------------------+--------------------------------+----------------------
<PHONE_NUMBER>68623363 |<PHONE_NUMBER>352994984.0000000000 |<PHONE_NUMBER>352995000
(1 row)

Time: 5ms total (execution 5ms / network 0ms)

I am running zitadel v2.61.1 and cockroach v23.2.5

@cmj2002 thanks for the reply. It confirms my suspicion. My advise is to downgrade to v2.61.1 until we have a fix available.

v2.62.2 resolves the issue.


Seeing stats overtime?

I think this is more of a feature request than an issue, but is there any way to see subscribers/followers over time, like how the graphs/charts show them on social blade? Like for example getting the subscribers of a channel from each month for the last two years? I was curious to ask because I know you can get live updates I was wondering if you could access historical data as well. Thanks!

i really dont know how to implement that, the way socialblade works. i think that would require scraping rather than API stuff

you can feel free to experiment and post a pull request though


fabric-svg-from-url

all my other components work fine, but when i use
<fabric-svg-from-url :id="'someUUID'"></fabric-svg-from-url>
I get
Unknown custom element: <fabric-svg-from-url> - did you register the component correctly? For recursive components, make sure to provide the "name" option.

import FabricSvgFromUrl from "./components/FabricSVGFromURL";
components: {
FabricSvgFromUrl,
}
See: https://github.com/bensladden/vue-fabric-wrapper/blob/master/src/App.vue

This worked, I copied the components from github down to a local directory ./components.  it is not clear that i should have to do this when I have used npm install to install vue-fabric?

Hi, sorry to be clear you don’t need to copy the components but in the
components object the name was changed to better fit the HTML casing
On Mon, 22 Jun 2020 at 6:32 pm, constructivetim<EMAIL_ADDRESS>wrote:

This worked, I copied the components from github down to a local directory
./components. it is not clear that i should have to do this when I have
used npm install to install vue-fabric?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bensladden/vue-fabric-wrapper/issues/111#issuecomment-647385777,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJQMFBHDV3BABQDW7254WHLRX4M4DANCNFSM4OBRSOBQ
.

so, if I just do this:

...
  import vueFabricWrapper from "vue-fabric-wrapper";
...
  export default {
    components: {
      FabricImageFromUrl: FabricImageFromURL
    },
...
i get
ReferenceError: FabricImageFromURL is not defined

import vueFabricWrapper from "vue-fabric-wrapper"; ... export default {
components: { FabricImageFromUrl: vueFabricWrapper.FabricImageFromURL },
On Mon, 22 Jun 2020 at 7:30 pm, constructivetim<EMAIL_ADDRESS>wrote:

so, if I just do this:
... import vueFabricWrapper from "vue-fabric-wrapper"; ... export default
{ components: { FabricImageFromUrl: FabricImageFromURL }, ...
i get
ReferenceError: FabricImageFromURL is not defined
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bensladden/vue-fabric-wrapper/issues/111#issuecomment-647416309,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJQMFBB7DJXDTCDKB7ONSFTRX4TVFANCNFSM4OBRSOBQ
.

brilliant.  that fixed it.  thanks ben


Add environment variable and docs to allow skip TLS verification

Adding a rather explicit option to ignore SSL validation, this is required in case of self signed x509s, also raised in https://github.com/gotify/cli/issues/27

Hey @AvnerCohen, thanks for your contribution!
I'd like to keep to api client as pure as possible. Could you include this into the cli?

@jmattheis Makes sense.


Tracking in Font Awesome

Issue description:
We have to include kirki plugin in our plugin but WP plugin reviewers don't accept it for following reason. Any plan to refactor?

In 2016, Font Awesome introduced a new way to get their font library, via a URL like https://use.fontawesome.com/CUSTOMCODE.js
The problem with it is that it's tied to an individual's email, which means it's actually trackable. That can be a legal issue when it comes to disclosure of data and is something best avoided whenever possible. Not to mention, we don't allow tracking unless it's opt in.
We need you to use the font-awesome anonymous CDN - https://www.bootstrapcdn.com/fontawesome/
Some examples from your plugin:
/jupiterx-core/includes/customizer/vendors/kirki/controls/php/class-kirki-control-fontawesome.php:38:
wp_enqueue_script( 'kirki-fontawesome-font', 'https://use.fontawesome.com/30858dc40a.js', array(), '4.0.7' );
/jupiterx-core/includes/customizer/vendors/kirki/modules/css/class-kirki-modules-css.php:218:
wp_enqueue_script( 'kirki-fontawesome-font', 'https://use.fontawesome.com/30858dc40a.js', array(), '4.0.7', true );

Version used:
3.0.33

Yes, this will get refactored and a new version uploaded soon.
Do you use the font-awesome field?

Since we didn't need that field, we removed it from our custom version.


(PUP-3947) Let infer_set of Hash return a Struct

Changes the infer_set_Hash method so that it returns a PStructType
rather than a PHashType when all keys of the hash are found to be
non empty strings (assignable to String[1]).

needs a test for the change that fixes the problem of Struct not being assignable to Hash to avoid future regressions

CLA signed by all contributors.


DefaultUnhandledExceptionsReporter reports cleanup related issues

Observed this WARN:
Observed 8 unhandled exceptions in last 10000ms(10000000000ns). Please consider adding the LoggingService decorator to get detailed error logs. One of the thrown exceptions:

com.linecorp.armeria.common.ClosedSessionException: null
	at com.linecorp.armeria.common.ClosedSessionException.get(ClosedSessionException.java:35)
	at com.linecorp.armeria.server.HttpServerHandler.cleanup(HttpServerHandler.java:243)
	at com.linecorp.armeria.server.HttpServerHandler.channelInactive(HttpServerHandler.java:236)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:305)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:281)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelInactive(AbstractChannelHandlerContext.java:274)
	at io.netty.handler.codec.ByteToMessageDecoder.channelInputClosed(ByteToMessageDecoder.java:411)
	at io.netty.handler.codec.ByteToMessageDecoder.channelInactive(ByteToMessageDecoder.java:376)
	at io.netty.handler.codec.http2.Http2ConnectionHandler.channelInactive(Http2ConnectionHandler.java:430)
	at com.linecorp.armeria.server.Http2ServerConnectionHandler.channelInactive(Http2ServerConnectionHandler.java:103)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:305)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelInactive(AbstractChannelHandlerContext.java:281)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelInactive(AbstractChannelHandlerContext.java:274)
	at io.netty.handler.logging.LoggingHandler.channelInactive(LoggingHandler.java:206)

Well, we do have a LoggingService configured! But since this error doesn't happen on a specific request, shouldReportUnhandledExceptions is not set to false
As for the root cause, I found this other log
 Sending GOAWAY failed: lastStreamId '2147483647', errorCode '0', debugData ''. Forcing shutdown of the connection.
io.netty.handler.codec.http2.Http2ConnectionHandler#processGoAwayWriteResult
io.netty.channel.unix.Errors$NativeIoException: writevAddresses(..) failed: Broken pipe

So the client was not shutdown cleanly, causing an ignorable error to happen on cleaning up the inactive channel
I think that the DefaultUnhandledExceptionsReporter should not WARN in such a case.

I think that the DefaultUnhandledExceptionsReporter should not WARN in such a case.

Agreed! Thanks for letting us know about this issue 😅    We'll try to get this fixed in the next release

@jrhee17 May I look on the issue ? 😄

would it be possible for me to take a look at the issue?

Sure, go for it.

If it's alright, @mauhiz, could you please provide me with information on how to reproduce the problem?

The problem is reproducible by closing a request that isn't closed.
You can probably simulate closing the http connection with something like the following:
try (ClientFactory cf = ClientFactory.builder().build()) {
  final HttpResponse res = server.webClient(cb -> cb.factory(cf)).get("/streaming");
  await().until(() -> !server.requestContextCaptor().isEmpty());
}

Hi @jrhee17, sorry for the late reply as I needed some time to understand the problem 😢. I've added a test code to reproduce the problem.
Yet, upon executing the test code to replicate the issue, the LoggingDecorator intercepts the exceptions instead of DefaultUnhandledexceptionReporter reporting them. I think I'm missing something ...

I think the scenario is possible if the LoggingService is not reached.
For instance, the decorator sequence does not necessarily need to be syncrhonous.
Server -> AuthService -> LoggingService -> HttpService

If AuthService only is reached, then I guess the log would be printed even if LoggingService is in the path.

@jrhee17 Thank you for the explanation. I was able to reproduce above problem by using the examples you provided(by adding decorators which throws exceptions on certain conditions).
Server -> CustomService(throws exception) -> LoggingService -> HttpService

I found that when CustomService throws exception, even though LoggingService is configured, DefaultUnhandledExceptionsReporter would report the exceptions thrown by the CustomService.
I was able to make LoggingService handle exceptions thrown by CustomService by moving the positions like below:
Server -> LoggingService -> CustomService(throws exception) -> HttpService

Rethinking about this issue, I wonder the behavior of DefaultUnhandledExceptionsReporter reporting unhandled exceptions(even though LoggingService is configured) is a weird behavior. Rather, can this behavior considered as a feature(reporting all the unhandled exceptions thrown while handling requests)? If it can, then how about fixing the log message?
as-is
Please consider adding the LoggingService decorator to get detailed error logs

to-be example
Please consider adding the LoggingService decorator as the outermost decorator to get detailed error logs

If this behavior is intolerable, I think we can fix it by adding a flag(such as loggingServiceExists) on the ServerBuilder and use it to turn DefaultUnhandledExceptionsReporter's reporting feature on or off.

Sure, how about the following message:
            logger.warn("Observed {} exceptions that didn't reach a LoggingService in the last {}. " +
                        "Please consider either adding a LoggingService or moving the " +
                        "LoggingService to an earlier timing.",

Also, I think that users can't do much about this kind of exception.
What do you think of also ignoring exceptions by checking Exceptions.isExpected?
    private static boolean isIgnorableException(Throwable cause) {
        if (Exceptions.isExpected(cause)) {
            return true;
        }
...

Thank you for step by step guide. I opened a [PR]( Add exceptionShouldNotBeLoggedWhenClientDisconnectsAbruptly) and now we can talk about this issue there! 🚀


Fix test failure in test_markov_approx.jl

https://github.com/QuantEcon/QuantEcon.jl/runs/7090838653?check_suite_focus=true#step:6:211
See #272.

Done by #274.


Add 'select all' when batch tagging

User requested. Seems simple enough -- add a button in the manual tags tray ✅

Hi, can I work on this?

Hi, can I work on this feature?

Thank you 🤝
I'm sometimes rather particular about the UI (user interface), so don't worry too much about making things look great / harmonious with the rest -- I can fix it up after your code merges in 👍
Please feel free to reach out with any questions -- I'm happy to help 🙌

Only suggestion for an addition here is: make the same button become "deselect all" once it has selected all, and maybe have the button have the format of "deselect all (<number_of_videos_selected>)" so we have a better understanding.

You can see an example of the poc I did for this here: https://github.com/whyboris/Video-Hub-App/pull/482
although I'm not leveled up enough with angular to know how to pass down/bubble up the selected count to the right scope, but that's a solvable problem.
Also, I have tweaked the css to make the buttons look reasonable BUT they should probably be put in their own dive and made inline-block. But tbd.

Currently the deselect all can be accomplished by toggling the batch mode off.
I'm unsure about cramming too many buttons into the menu, but maybe deselect all is a worthwhile addition 🤔 thank you!
I especially like the "number selected" indicator 😁

Thank you for the PR! I'll try to get to it this weekend 😁
No worries about CSS -- I'll get it to look harmonious with the rest of the app (according to my sometimes-picky preferences 😅 )

Thank you @jamesd35 for the feature addition #446 🙇 🎉 🤝 🦾
Cheers 👋


SA1008, SA1012: false positives for property pattern inside parentheses

Using StyleCop.Analyzers version 1.2.0-beta.556
public static class Helpers
{
    public static bool HasValidLength(string s)
    {
        // ⚠ SA1008 Opening parenthesis should not be preceded by a space.
        // ⚠ SA1012 Opening brace should be preceded by a space
        //              ↓↓
        return s is not ({ Length: < 5 } or { Length: > 10 });
    }
}

This is similar to the problem fixed in #3141.

I can confirm SA1012, but I actually don't see a SA1008 in this example. Weird.

Yes, technically correct it would be like this:
public static class Helpers
{
    public static bool HasValidLength(string s)
    {
        // ⚠ SA1012 Opening brace should be preceded by a space
        //               ↓
        return s is not ({ Length: < 5 } or { Length: > 10 });
    }
}

"Fixing" this leads to the other error:
public static class Helpers
{
    public static bool HasValidLength(string s)
    {
        // ⚠ SA1008 Opening parenthesis should not be preceded by a space.
        //                ↓
        return s is not ( { Length: < 5 } or { Length: > 10 });
    }
}

Ok 👍
I am also seeing the same for ({ Length: < 5 } and { Length: > 2 }) and ({ Length: < 5 })


Refined chat with pepe

I couldn't test out my plugin in the app, there are no developer tools for that yet, so I've made some quick changes for it here.

Summary by CodeRabbit

New Features

Updated the Pepe the Frog plugin to enhance character representation with a more focused chat prompt.

Bug Fixes

Removed several outdated plugins, streamlining available functionalities for users.

Hi @findirfin can you please resolve the conflicts so that we can review it

lgtm pepe @findirfin 🚀


V0.8.15 image is missing log-counter binary

Can someone update the node-problem-detector application version as there are some problems in 0.8.15 according to the following issue  ?
The last version of the application is v0.8.17.
Thanks!

don't close the issue


nfl-update week 1 issue

I updated the database as normal but for some reason week 1 of the 2017 Regular season isn't appearing.
This seems weird as week 2 has already appeared.

Potentially same problem as https://github.com/BurntSushi/nflgame/issues/331?
Give that a try.

Thanks! I must have missed that post when I was searching.


Bug: Tax class field is empty on new product, even though a default was defined in product type

What are you trying to achieve?
When creating a new product, the tax class field is empty, even though a default value was defined in the product type the new product is based on.
I assume having an option to define such a default implies that it should be taken into account while creating a new product based on such types.
Steps to reproduce the problem

create a product type
set tax class to some existing value
create product based on type
notice the empty tax class field

What did you expect to happen?
tax field containing tax setting defined in product type
Logs
No response
Environment
Saleor SaaS

Hey @genox, it is expected behavior - even though the product's tax class is not set, the tax calculations would use the one from the product type. Setting tax class at the product level is more like overriding the product type's setting. Maybe it needs to be clarified in the docs or GraphQL descriptions.

Hey @genox, it is expected behavior - even though the product's tax class is not set, the tax calculations would use the one from the product type. Setting tax class at the product level is more like overriding the product type's setting. Maybe it needs to be clarified in the docs or GraphQL descriptions.

I understand. But it is unintuitive. As a user, I see the tax field being empty and set it, because I want taxes set, right? Empty field means it is not set from my perspective, because I do not know the inner workings and I do not need to know them, because I just enter product data and want to make sure everything is correctly set.
Once a product is saved, the default is correctly displayed as I just noticed, just not during the initial product creation form. Why not set the default already on the creation view instead of only after the first save if the result will be the same?

I think this issue can be solved in UI, e.g., by a help text in the product creation form informing about this (and better docs).

Closing this ticket as this is not a core issue. Thanks for reporting!


Is this project maintained (oct 2017) ?

Wonderful library but I am seeing PRs and question unanswered for some time.
@jerkoch

Is the project maintained ?
If not maintained, do you recommend an alternative library ?

Yes it is.


howto install on Fedora 21

First of all - I have ZERO knowledge on what's "build" / "cmake" etc. Consider a dumb user here. I'm trying to install Vc on Fedora 21 as it is needed by one of the packages I wish to use.
I'm stuck at:
"Call cmake with the relevant options"
What are relevant options?

ok , after a bit of juggling with arguments I got it. First I need to create a folder where I create the install (separate folder from the folder with source - of course ;) ), then go to it and run
cmake -DCMAKE_INSTALL_PREFIX=/folder_where_in_the_end_app_will_be_installed -DBUILD_TESTING=OFF /folder_containing_source_files
Then:
$ make -j9
$ make install

exactly. Another important option is CMAKE_BUILD_TYPE. You can also use ccmake or cmake-gui to get an overview of the available switches.


feat: generate client for crd resources

feat: generate client for crd resources via k8s's official script

@sagilio


SQL error, always use pingcap/tidb

Bug Report

Describe the bug

https://ossinsight.io/explore/?id=8664e2b4-8dd7-4a65-b6c2-cc4b2e072577
To Reproduce
Expected behavior

Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context

We should remove the example, and add SQL snippets instead.

another case using pingcap/tidb: https://ossinsight.io/explore/?id=c4c2a872-66d8-414d-ac04-1e687da1f792


[PE-D][Tester B] Storage not functioning as required

Storage seems to be not updating and disappearing when I restart the app. As seen from the screenshot, after trying out commands on the sample json data such as editing and adding new people to the contact list, I closed the application and reopened it to see this screen. When I call view command, I get an empty screen too.

Labels: type.FunctionalityBug severity.High
original: chewbum/ped#1

Similar issue to issue #187


Make input field auto expand its height when it's more than one line.

The max height is currently set to 5 lines, but folks that wanna keep the bar as 1 line tall can cap it with max-height: 1em in their custom stylesheet.

Very nice, much needed


"track this case" link does not work

from the articles/show view. Should function the same as clicking the follow button.

This is fixed on production.


feat: upstreams cloudtrail and cloudtrail-bucket components

what

Upstreams cloudtrail and cloudtrail-bucket components

why

Keeping our components up-to-date with updates that have been made for clients

Old as dirt!


Upload file for attachment

Discussed in https://github.com/zapadi/redmine-net-api/discussions/355

Originally posted by joewak August 22, 2024
I'm trying this:
            byte[] documentData = System.IO.File.ReadAllBytes(file_path);
            Upload attachment = manager.UploadFile(documentData);

When I do so, I get an exception:
           WebException: The remote server returned an error: (422) Unprocessable Entity.
           RedmineException: Invalid or missing attribute parameters: Attachment extension  is not allowed

Poking around in this function in InternalRedmineApiWebClient.cs:
        public ApiResponseMessage Upload(string address, byte[] data, RequestOptions requestOptions = null)
        {
            var content = new ByteArrayApiRequestMessageContent(data);
            return HandleRequest(address, HttpVerbs.**UPLOAD**, requestOptions, content);
        }

The Redmine REST API documentation says a POST should be used to upload a file.  Should it be POST instead of UPLOAD?

Published version 4.6.7

I tested with version 4.6.7.  Now I get:
"The remote server returned an error: (406) Not Acceptable."

I was able to upload and attach a file.
I made this function which went outside of your library to upload the file using HttpClient to do the POST operation.  Using this function I was able to get the 201 response code and get the token from the response.
public static bool UploadFile(string file_path, out string response_token)
        {
            bool retval = false;
            response_token = "";

            try
            {
                byte[] documentData = System.IO.File.ReadAllBytes(file_path);
                string filename = System.IO.Path.GetFileName(file_path);

                using (var content = new ByteArrayContent(documentData))
                {
                    content.Headers.ContentType = new MediaTypeHeaderValue("application/octet-stream");
                    content.Headers.Add("X-Redmine-API-Key", apiKey);

                    var client = new HttpClient();
                    var task1 = Task.Run(() => client.PostAsync("https://ptcobservations.wabtec.com/uploads.xml?filename=" + filename, content));
                    task1.Wait();
                    var response = task1.Result;
                    var task2 = Task.Run(() => response.Content.ReadAsStringAsync());
                    task2.Wait();
                    var response_contents = task2.Result;

                    XmlDocument xml = new XmlDocument();
                    xml.LoadXml(response_contents);

                    response_token = xml.ChildNodes[1].ChildNodes[0].ChildNodes[0].Value;

                    if (response_token != "")
                    {
                        retval = true;
                    }
                }
            }
            catch (Exception ex)
            {
                Console.WriteLine(ex.Message);
            }

            return retval;
        }

Then I was able to use the token I received with your library to attach the file to my issue:
public static bool AttachFile(string token, string filename, string ticket_id)
        {
            bool retval = false;

            try
            {
                RedmineManager manager = new RedmineManager(host, apiKey);

                Upload attachment = new Upload();

                ////set attachment properties
                attachment.FileName = filename;
                attachment.Description = "Remote download package for " + ticket_id;
                attachment.ContentType = "application/octet-stream";
                attachment.Token = token;

                ////create a list of attachments to be added to issue
                IList<Upload> attachments = new List<Upload>();
                attachments.Add(attachment);

                var parameters = new NameValueCollection { };
                Issue issue = manager.GetObject<Issue>(ticket_id, parameters);
                issue.Uploads = attachments;

                if (issue != null)
                {
                    manager.UpdateObject(issue.Id.ToString(), issue);
                    retval = true;
                }
            }
            catch (Exception ex)
            {
                Console.WriteLine(ex.Message);
            }

            return retval;
        }

Fixed in 4.6.9 version

Sorry, I still don't think it works.  Back to 422 error.


Swtiching from Apache to Nginx server

Hi Janis,
I have switched over from Apache to Nginx server and I'm tryning to protect the wp-update server folders.
Since .htaccess files do not work on Nginx, I'm struggling to come up with a solution to protect the logs and packages folders.
I tried adding the code below into Nginx cinfiguration, but I'm still able to directly access package .zip file.
location /wp-content/wp-update-server/packages/ {
deny all;
return 403;
}

I don't really have any experience with Nginx myself, so, unfortunately, I don't think I can offer any useful advice here.

Thank you for your fast response!

The code is correct, is it possible that you inserted in the wrong place?
Probably you have a main /etc/nginx/nginx.conf and then the vhost file inside /etc/nginx/sites-available/, here you have to insert the code, inside server {}
server {

    [ ... ]

    location /wp-content/wp-update-server/packages/ {
        deny all;
        return 403;
    }
}

Then you have to reload nginx.
service nginx reload

Thank you for assistance Sergio,
The code that I posted before was saved in CloudPanel Vhost file in between the server{}, but it did not have any effect. I might look for the other file you mentioned and try adding it there.

Turns out Vhost was the correct place, however the rule had to look a bit different:
location ~* /wp-content/wp-update-server/packages/ {
if ( $upstream_http_x_accel_redirect = "" ) {
return 403;
}
internal;
}

I use WordOps where Zip & logs are by default protected from download.


Update SA1649 to support compatibility with built-in Visual Studio file nesting feature

Hi.
I'm trying to follow Steve "ardalis" Smith's Clean Architecture Web API endpoint code sample but the file names that define the API DTO model request and response classes conflict with SA1649.
When the solution is viewed in Visual Studio, the DTO model request and response classes are nested under the class that defines the API endpoint.
For example, see the CreateEndpoint.cs, CreateEndpoint.CreateCommand.cs, and CreateEndpoint.CreatedResult.cs
I expected StyleCopAnalyzers to detect this type of file naming convention.
However, StyleCopAnalyzers v1.1.118 reports the violation in both CreateEndpoint.CreateCommand.cs and CreateEndpoint.CreatedResult.cs:
Warning	SA1649	File name should match first type name.
Thank you.

Even the BCL Violates this rule 😄.

I recommend disabling or reimplementing this rule in cases where a project intends to use filenames that do not match the type name.


Create a CanvasSpace with transparent color

Is it possible to create a CanvasSpace with transparent background?
I'm currently using
new CanvasSpace('id').setup({ bgcolor: 'rgba(0,0,0,0.1)' });
with no luck
Thanks!

Related to this line not being called. In other words, Pt currently either paints over your canvas (with bgcolor) or clears it.
Quick fix would be:
@ctx.clearRect( 0, 0, @size.x, @size.y )
if @bgcolor
  @ctx.fillStyle = @bgcolor;
  @ctx.fillRect( 0, 0, @size.x, @size.y )

Yep, if I put 'rgba(0,0,0,0.0)' it wont clear the canvas, instead I can see all the movement without clear.

@amorino There's probably a bug somewhere that the clearRect logic doesn't work. Good to know.
Meanwhile, if you call clearRect directly like what @tofuness suggested, it should work. Eg, suppose you have a space like yours above:
var mySpace = new CanvasSpace('id').setup({ bgcolor: 'rgba(0,0,0,0.1)' });
Then inside the animate loop, you can add something like this to force the clearRect:
mySpace.ctx.clearRect( 0, 0, mySpace.size.x, mySpace.size.y );
See if that works for now? We'll hunt down the bug. Thanks!

This bug should be fixed now. You can set a transparent background via space.setup({bgcolor: false}) or  space.setup({bgcolor: "transparent"}). Please see if it works for you. Thanks for reporting!
https://github.com/williamngan/pt/commit/05a0f63b2bf9911cf4803ee4b64aecfe0ddf5709

To get this to work I had to set const space = new CanvasSpace('#hero-animation').setup({ bgColor: 'transparent' }); when initializing space and space.clear('transparent'); at the beginning of my animate block.


Hide passwords in repo path, fixing #58

Thanks for the contribution!
As the regex only applies to REST paths, I would move the regex replacement code into an if (appState.repoLocation.prefix == "rest") branch, also to document what it's trying to match.
The regex also won't match rest paths of the form https://user:pass@host:8000 (note the missing trailing /), but I'm not actually sure if restic allows them. But it won't match sftp path then either?

I didn't think about ports, will habe to check. But you're right i'm missing quite a few test cases. Thin i was focusing on here wasmainly that we can have the protocol optional in the URL (diallowing the URL() from js).
Let's collect all cases interesting to blur out here, and then adjust.

I actually tried to design it to work with any URL, rest but also sftp, ftps or any future URL like path. Ports and the whole path after the domain are now more leniently handled. Care to have a look?

Looks good to me. Thanks!


Need to specify a minimum node version

My global node version was 14.15.0, running hathora dev crashed with this error:
up to date in 0.109s
(node:15544) ExperimentalWarning: --experimental-loader is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
TypeError [ERR_INVALID_RETURN_PROPERTY_VALUE]: Expected string to be returned for the "format" from the "loader getFormat" function but got type object.
    at Loader.getFormat (internal/modules/esm/loader.js:110:13)
    at async Loader.getModuleJob (internal/modules/esm/loader.js:230:20)
    at async Loader.import (internal/modules/esm/loader.js:164:17)
    at async Object.loadESM (internal/process/esm_loader.js:68:5) {
  code: 'ERR_INVALID_RETURN_PROPERTY_VALUE'
}

After some googling, I came across a stack overflow that talked about upgrading node version.
So when I switched to node 16.13.0, it worked just fine.
Specifying a .nodenv and .nvmrc file in the tutorial repos would fix the short-term issue. (You can symlink one to the other to keep them in sync)
I didn't dig in to what feature is necessary for the build to pass, to track down the minimum node version, but just documenting it as an issue here.

I specify the minimum node version in the docs and also in the engines section of package.json, which should warn you if your node version is too low when you install/upgrade hathora (but I know most people ignore install output).
The CI also verifies that is passes on node 16.x and 17.x (I need to add 18.x there too).
I have an .nvmrc file in this repo but agreed that it should be in the tutorial repos too, thanks for your PR there!
I think having this issue could help others who come searching for the same error message, so that's already an improvement. Closing this for now but let me know if you can think of other ways to improve the docs/enforcement around the minimum node version.


Dispose MD5 object after use

Fixes #49

Hi @ameno-, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Adding link for the support request - https://support.hockeyapp.net/discussions/problems/61374-use-sha256-signatures-instead-of-md5
We will update the post once we have a concrete answer.

Thank you. We have added this fix as part of this commit:
https://github.com/bitstadium/HockeySDK-Windows/commit/532ed9d310cf012b8c33e8f91a229eb3c65ad4b3


golint: fix typo

Fixes the following warning
"ommitted" is a misspelling of "omitted" (misspell)
Related-to: #153 (Resolve golint warnings)

This pull request is now managed with CodeReviewHub. Refresh.


Fish like history search

It would be cool if you could type  and pressing up arrow auto completes it to
 FeelsWeirdMan
 PagChomp
 Kapp
Here is how z-shell implements it: https://github.com/zsh-users/zsh-history-substring-search
Fish source: https://github.com/fish-shell/fish-shell

The amount of people that would use this feature is too small to add it. Furthermore I think that this feature is not as useful in a chat than in a shell.


Add Component:RemoveTag()

Implements the RemoveTag() method for the Component class, allowing easy un-tagging of any given component instance.

self.Instance:RemoveTag(Component.Tag)

Yeah, just remove the tag from the instance. I think that's the cleanest option here.


Refactor API to look like native bevy API

Just cloned the repo to have a read of the code, I might give this a shot. If I understand correctly we should be able to spawn a CosmicEdit just like a TextBundle?

Currently to create cosmic_edit text input a font system should be created:
    let cosmic_font_config = CosmicFontConfig {
        fonts_dir_path: None,
        font_bytes: Some(vec![font_bytes]),
        load_system_fonts: true,
    };
    let font_system = create_cosmic_font_system(cosmic_font_config);
    let font_system_handle: Handle<CosmicFont> = cosmic_fonts.add(CosmicFont(font_system));

and then using font_system_handle multiple cosmic_edit text inputs can be created like so:
    let cosmic_edit_meta = CosmicEditMeta {
     text: CosmicText::OneStyle("😀😀😀 x => y".to_string()),
     attrs: AttrsOwned::new(attrs),
     text_pos: CosmicTextPos::Center,
     bg: Color::WHITE,
     metrics: CosmicMetrics {
         font_size: 14.,
         line_height: 18.,
         scale_factor,
     },
     font_system_handle,
     node: CosmicNode::Sprite(CosmicEditSprite {
         transform: Transform {
             translation: Vec3::new(0., 0., 1.),
             ..default()
         },
     }),
     size: Some((primary_window.width(), primary_window.height())),
     readonly: false,
     bg_image: None,
 };
 let cosmic_edit = spawn_cosmic_edit(&mut commands, &mut cosmic_fonts, cosmic_edit_meta);

The idea is to improve above api to look more like native bevy api.... for example bevy_prototype_lyon api looks something like this:
    let points = [
      Vec2::new(-width / 2., -height / 2.),
      Vec2::new(-width / 2., height / 2.),
      Vec2::new(width / 2., height / 2.),
      Vec2::new(width / 2., -height / 2.),
  ];
  let path: Path = bevy_prototype_lyon::shapes::RoundedPolygon {
     points: points.into_iter().collect(),
     closed: true,
     radius: 10.,
  };
  let shape = commands
      .spawn((
          bevy_prototype_lyon::prelude::ShapeBundle {
              transform: Transform {
                  translation: Vec3::new(0.0, 0.0, 0.001),
                  ..default()
              },
              path,
              ..default()
          },
          Stroke::new(
              if has_border {
                  theme.node_border
              } else {
                  Color::NONE
              },
              1.,
          ),
          Fill::color(item_meta.pair_bg_color.1)
      ))
      .id();

Notice ShapeBundle api looks similar to TextBundle api in some way (looks like native bevy api), so the goal is to improve cosmic_edit api in a similar way.

So I've got this working to spawn a cosmic text box:
commands.spawn(CosmicEditUiBundle {
    style: Style {
        width: Val::Px(300.),
        height: Val::Px(50.),
        left: Val::Px(100.),
        top: Val::Px(100.),
        ..default()
    },
    ..default()
});

...but I've borked the image clearing somehow. Investigating now.
I should also learn how to make a PR again at some point lol

I should also learn how to make a PR again at some point lol

Quite similar to https://github.com/StaffEngineer/velo/blob/main/contributing.md

oh cool seems simple enough. I've made a whole lot of breaking changes so I'll need to refactor tests and examples too, will send a PR when that's done

Nice, no hurry, if you want to refactor https://github.com/StaffEngineer/velo codebase (it heavily uses bevy_cosmic_edit) for the new API feel free, if not - I can adjust it by myself once your changes are merged :)

@bytemunch do you need any help? :)

All good, not been stuck with anything just simple lack of time, just moved and not got broadband transferred yet 😭 Should be able to get the refac finished tomorrow, if all goes well in life 🙏

@bytemunch all the best anyway 🙏 ❤

Alright now I might need a hand, I seem to have broken MultiStyle text and I'm not sure where, could you have a check please?
https://github.com/bytemunch/bevy_cosmic_edit/tree/bevy_like_api

Hi @bytemunch , I am on vacation with phone without laptop, so I can only take look in a week of so.

ah no worries, enjoy your time off :) i found the culprit, I had some misplaced logic


Add code example

Can you please add code example ?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 68d4eea1-7a60-70e4-d141-8cee4c14f061
Version Independent ID: 264acc55-9b01-2a2c-e061-abafcf8e49c4
Content: CustomSystemEventTrigger Class (Windows.ApplicationModel.Background) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.applicationmodel.background.customsystemeventtrigger.yml&version=GBlive&_a=contents
Service: unspecified
GitHub Login: @wdg-dev-content
Microsoft Alias: wdg-dev-content

Thanks for submitting your feedback and I apologize for the delayed response. What I can do is create an internal request for content and see if we can get a code example published in the article. The timeline for this may vary, so I will go ahead and close this and circle back with any updates. Please feel free to follow-up below with any additional feedback. Many thanks.


Invalid example - Cloud build trigger

https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/blob/10ab68c1680f61a3a1bfd929dbba760f216f952e/dm/templates/cloudbuild/examples/cloudbuild_trigger.yaml#L30
Property 'build' got renamed/moved in the schema, needs a fix.

@ocsig It was always build_template, probably example was wrong from the beginning

Please note, that due to incorrect gcp-type mapping field "revision" is not supported


Construction Menu buildings using wrong rotation

Description
Excuse the overly broad title. Not 100% certain how much is affected.
It appears the construction menu for (atleast) wall closet, is not respecting the selected rotation on the build menu, but is instead taking station rotation or some other variable?


Don't always stubborn merge lines that open brackets

Describe the bug
This is ugly. We force merge the lines that have function calls, since they open brackets, but we probably shouldn't, so that all the sibling operators stay at the same level.
The new rule should be: don't stubborn-merge p1 operators that open brackets if the previous segment also starts with an operator. Also, consider that p1 operators might need a different sequence of stubborn-merge strategies from p0 operators.
To Reproduce
From gitlab's project (indented a few more times, so it doesn't fit on a single line):
SELECT
    LOWER(table_catalog)||'.'||LOWER(table_schema)||'.'||LOWER(table_name) AS fqd_name,
    ...

Expected behavior
select
    lower(table_catalog)
    || '.'
    || lower(table_schema)
    || '.'
    || lower(table_name) as fqd_name,

Actual behavior
select
    lower(table_catalog)
    || '.' || lower(
        table_schema
    )
    || '.'
    || lower(table_name) as fqd_name,

Additional context
What is the output of sqlfmt --version?

Another good example of this from http:
Actual:
round(
        countif(firsthtml and lower(resp_cache_control) like '%s-maxage%')
        * 100 / count(
            0
        ),
        2
    ) as firsthtmlpct,

Better:
round(
        countif(firsthtml and lower(resp_cache_control) like '%s-maxage%')
        * 100
        / count(0),
        2
    ) as firsthtmlpct,


[WIP] v15 preview update

Description
Describe your changes here.
Tests
How did you test these changes prior to submitting this pull request?
What automated tests are included in this PR?
Documentation
How is this documented (for example: release note, upgrade guide, feature page, in-code documentation)?
Did you remember a changelog entry?
Primary reviewers
If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.

Superseded by #1495 and #1496

Superseded by #1495 and #1496


[explore] force control validation before runQuery

somehow default metric would not be auto populated for Table viz, but there was backend mechanics to auto-chose a random metric (that had some related edge case bug)
frontend validation would only occur on field change, not when first loaded, that created edge cases where the validation or default value might have changed

@ascott

@ascott thanks for the review, I addressed all comments!

LGTM


Keep elements shown scrolling back up

Hi, great plugin :) I like how the elements fade in as a user scrolls down the page but is there a way that I can keep the elements visible as I scroll back up the page so they don't fade out?
thanks

You should ask to the guy on this topic seems he found a way ;-)
https://github.com/jlmakes/scrollReveal.js/issues/177

I didn't try but it seems that this should answer to your question nop ?
https://github.com/jlmakes/scrollReveal.js/wiki/Keywords#reset

Thanks for your reply, I currently have data-sr="enter center wait 0.5s" so by making it: data-sr="enter center wait 0.5s, reset" that doesn't seem to make any difference, perhaps I am doing something wrong as it still fades the element out once going past the viewport.

You should try : no reset
reset is doing exactly what you doesn't want :)

Thanks but i tried both of them but neither works

Thanks @Floriangomis @gab1982
If you have data-sr = reset it will force enable create the element fade you describe when going past the viewport. However, with no before, it will force disable it. e.g. data-sr = no reset. In your example, you hadn’t included the no keyword.
However, I’m not sure I ever released ScrollReveal with reset: true in the default configuration—certainly not the latest v2.3.2 and v.3.0.0-RC.
So even though I recommended ensuring use of no keyword in your case, I would also suggest taking a second look at your configuration.

Great thanks the no reset worked, It didn't work before as I had opacity: 0 in my css for that attribute, which I have now removed.

Nice to hear :) Have fun !

Glad it’s working.
@gab1982 I should mention that enter center won't actually do anything. If you don’t want any translation, you can use move 0px instead.
:beers:


Unable to pass 'user agreement' on main page

Started having this issue today where I cannot get past the front page user agreement. Clicking on the "I agree" button is not closing the pop-up or doing anything.

Thanks!

Users within Japan are no longer able to change regions.
If users were coming from non Japan IPs able to change the region and access, the above popup appears in both cases

There're two dialogs  1. for Private Policy 2. for users overseas.
For non-japan users  two dialogs will be displayed at same time which makes "I agree" button not work.
I'll try to bypass this.

I uploaded v0.2.7 to Chrome Web Store. Waiting for Chrome Web Store's review.
Or you could load extension from source code.

Thanks looking good, no issues within Japan on switching regions.

v0.2.7 is working well, thanks a lot.

Same problem again, I think radiko is supervising this project

update, seems if you attempt to use in Japan it kicks you out of website, similar seems if I vpn from Japan I can still use the plugin and switch region.

@wildthingsan  sorry for the inconvenience. v<IP_ADDRESS> is only tested in firefox . And chrome treats history differently with firefox. i made a revert release v<IP_ADDRESS>.

Why i do the change?
Because i want if user enter with link https://radiko.jp/#!/live/TBS and the can bypass the #!/out and still in the same page https://radiko.jp/#!/live/TBS instead of going to  the homepage https://radiko.jp/
In firefox history.back() works as a charm.
In chrome see https://stackoverflow.com/questions/58128939/changing-window-location-hash-creates-entry-in-history-but-doesnt-affect-chrome
and
https://stackoverflow.com/questions/58128939/changing-window-location-hash-creates-entry-in-history-but-doesnt-affect-chrome
And
https://chromium-review.googlesource.com/c/chromium/src/+/1344199

Appreciate the quick fixes as always. Confirming 2.7.2 working well outside Japan also.

hello everyone. I make a update to v0.2.8 which makes history work.
Can anyone help test this version on chrome from zip package  https://github.com/jackyzy823/rajiko/archive/refs/tags/v0.2.8.zip before i upload this to  chrome web store (no more bad releases :disappointed: ), thanks.

Hi, I tested the v0.2.8.zip from Japan IP and its working well, I was able to change different regions and tested downloading as well.
So I think functinoally it working fine again. (I also check plugin after an exit and restart of Chrome browser, there was odd issues before when it was kicking me out of the website with the earlier versions)

@adeptzero We should close this ticket as it has been fixed.

Closed thanks.


Image preview size/dimension is too small, changing the max_width doesn't take effect

Hi, I just found yazi and love it!
I rely of preview to sort through all the pdf I happen to download (using lf for it now). But the preview from yazi is rather small even when there is enough space in the their column. I changed the max_width in config but didn't help and couldn't find something similar in the existing issues, so asking here to check if I'm missing something simple.
Example of how it looks:

Thank you!

Hi, Yazi caches images to speed up image previews. After modifying the max_width, you need to clear the old cache, which can be done by doing yazi --clear-cache.

Hi, Yazi caches images to speed up image previews. After modifying the max_width, you need to clear the old cache, which can be done by doing yazi --clear-cache.

Thanks! that worked!


Remote Chrome with X generates "-32001 Session with given id not found" socket errors

Bug expectation
Puppeteer seems to be working fine if I am running google-chrome without a display server, however, when I connect to a headful instance running behind XOrg, Puppeteer opens a blank page and times out without being able to perform any actions.
The setup
Within supervisor, I run respectively:

XOrg ( I tried XVFB too)
DBUS
Chrome

The launch command for google-chrome is:
env DBUS_SESSION_BUS_ADDRESS="unix:path=/run/dbus/system_bus_socket" \
/usr/bin/google-chrome \
    --window-position=0,0 \
    --display=:10.0 \
    --remote-debugging-address=<IP_ADDRESS> \
    --remote-debugging-port=9222 \
    --no-first-run \
    --start-maximized \
    --bwsi \
    --force-dark-mode \
    --disable-file-system \
    --disable-gpu \
    --disable-software-rasterizer \
    --disable-dev-shm-usage \
    --no-sandbox

Puppeteer manages to work completely fine by removing the --display parameter and adding --headless to the above command.
Investigating on the debugger logs, I get a bunch of these errors (see full logs here):
puppeteer:protocol:RECV ◀   '{"id":8,"error":{"code":-32001,"message":"Session with given id not found."}}'
And on google-chrome outputs:
[18706:18706:0207/030958.413572:ERROR:browser_dm_token_storage_linux.cc(101)] Error: /etc/machine-id contains 0 characters (32 were expected).
[18752:18752:0207/030958.637660:ERROR:gpu_init.cc(523)] Passthrough is not supported, GL is disabled, ANGLE is

DevTools listening on ws://<IP_ADDRESS>:9222/devtools/browser/57c98292-052f-4531-b0a0-f41060458da9
[18706:18826:0207/030958.830431:ERROR:object_proxy.cc(623)] Failed to call method: org.freedesktop.DBus.Properties.Get: object_path= /org/freedesktop/UPower: org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.UPower was not provided by any .service files
[18706:18826:0207/030958.831310:ERROR:object_proxy.cc(623)] Failed to call method: org.freedesktop.UPower.GetDisplayDevice: object_path= /org/freedesktop/UPower: org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.UPower was not provided by any .service files
[18706:18826:0207/030958.835860:ERROR:object_proxy.cc(623)] Failed to call method: org.freedesktop.UPower.EnumerateDevices: object_path= /org/freedesktop/UPower: org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.UPower was not provided by any .service files

Bug behavior

[ ] Flaky
[ ] PDF

Minimal, reproducible example
const puppeteer = require("puppeteer")

puppeteer.connect({
    browserWSEndpoint: "ws://localhost:9222/devtools/browser/57c98292-052f-4531-b0a0-f41060458da9",
}).then(browser=> {
    browser.newPage().then();
})

Error string
puppeteer:protocol:RECV ◀   '{"id":8,"error":{"code":-32001,"message":"Session with given id not found."}}'
Puppeteer configuration
No response
Puppeteer version
19.6.3
Node version
16.13.1
Package manager
npm
Package manager version
8.1.2
Operating system
Linux

Issue reproducible using this build

It turns out that removing the policies.json file from /etc/opt/chrome has resolved the issue

Bumped into this while searching for a solution for the error above.
The policies.json file at the /etc/chromium/policies/managed folder had "DeveloperToolsAvailability": 2 in it, removing it solved the issue and allowed puppeteer to control the browser.


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/reconciler-test 5879dd7...32f0df0:
  > 32f0df0 Add DropEventsResponseCode for eventshub (# 216)
  > 43802ef Update actions (# 223)

/lgtm
/approve


Add SkipPixelData option to Parser

For heavy imaging modalities, it is expensive to load the pixel
data into memory, and for many use cases, unnecessary. This new
option allows users to parse the other attributes without the
resource hit.

Testing
$ go test -v -run TestNewParser ./...
=== RUN   TestNewParserSkipMetadataReadOnNewParserInit
--- PASS: TestNewParserSkipMetadataReadOnNewParserInit (0.00s)
=== RUN   TestNewParserSkipPixelData
--- PASS: TestNewParserSkipPixelData (0.01s)
=== RUN   TestNewParserPixelData
--- PASS: TestNewParserPixelData (0.62s)
PASS
ok      github.com/suyashkumar/dicom    0.677s
?       github.com/suyashkumar/dicom/cmd/dicomutil      [no test files]
?       github.com/suyashkumar/dicom/mocks/pkg/dicomio  [no test files]
?       github.com/suyashkumar/dicom/pkg/charset        [no test files]
testing: warning: no tests to run
PASS
ok      github.com/suyashkumar/dicom/pkg/dcmtime        0.011s [no tests to run]
?       github.com/suyashkumar/dicom/pkg/debug  [no test files]
?       github.com/suyashkumar/dicom/pkg/dicomio        [no test files]
testing: warning: no tests to run
PASS
ok      github.com/suyashkumar/dicom/pkg/frame  0.003s [no tests to run]
testing: warning: no tests to run
PASS
ok      github.com/suyashkumar/dicom/pkg/personname     0.017s [no tests to run]
testing: warning: no tests to run
PASS
ok      github.com/suyashkumar/dicom/pkg/tag    0.007s [no tests to run]
?       github.com/suyashkumar/dicom/pkg/uid    [no test files]
?       github.com/suyashkumar/dicom/pkg/vrraw  [no test files]

Formal commit message aside, thank you for creating this library 😃. It has been very useful!

Hi @tcheever just wanted to check in and see if you had a chance to consider the review comments? Thanks for the contribution! There may be some interplay with #245 as well as a heads up based on how these PRs progress.

Hi @suyashkumar, thanks for the bump and the review. I did not see your comments and suggestions. I will get back to you as soon as possible.

Hi @tcheever thanks for the responses, added a couple more comments for your consideration!

@suyashkumar, if I'm understanding correctly, you do not want the option in the reader at all and you do not want the arg added to NewReader?

Having an option like this would also be useful for us.

We are also interested in that option, I'd be happy to help if needed.

Thanks folks for the interest! I've circled back to this in #255, including making structural changes in #254 to support better option availability to read* methods in read.go (which was the subject of the review discussion here above).
Thanks @tcheever and others for the discussion!

Going to close this for now, since this feature was introduced in #255 with the new options availability (merged to main)! @NMerzVerily @rronan @b3n3d17 @tcheever please feel free to kick the tires on it and let us know how it goes.


undefined method `configure' for PublicActivity:Module (NoMethodError) Mongoid-4, rails 4

config/initializers/public_activity.rb:2:in <top (required)>': undefined method configure' for PublicActivity:Module (NoMethodError)
from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/activesupport-4.2.0/lib/active_support/dependencies.rb:268:in load' 	from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/activesupport-4.2.0/lib/active_support/dependencies.rb:268:in block in load'
from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/activesupport-4.2.0/lib/active_support/dependencies.rb:240:in load_dependency' 	from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/activesupport-4.2.0/lib/active_support/dependencies.rb:268:in load'
from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/railties-4.2.0/lib/rails/engine.rb:652:in block in load_config_initializer' 	from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/activesupport-4.2.0/lib/active_support/notifications.rb:166:in instrument'
from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/railties-4.2.0/lib/rails/engine.rb:651:in load_config_initializer' 	from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/railties-4.2.0/lib/rails/engine.rb:616:in block (2 levels) in class:Engine'
from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/railties-4.2.0/lib/rails/engine.rb:615:in each' 	from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/railties-4.2.0/lib/rails/engine.rb:615:in block in class:Engine'
from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/railties-4.2.0/lib/rails/initializable.rb:30:in instance_exec' 	from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/railties-4.2.0/lib/rails/initializable.rb:30:in run'
from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/railties-4.2.0/lib/rails/initializable.rb:55:in block in run_initializers' 	from /home/sahil/.rvm/rubies/ruby-2.1.5/lib/ruby/2.1.0/tsort.rb:226:in block in tsort_each'
from /home/sahil/.rvm/rubies/ruby-2.1.5/lib/ruby/2.1.0/tsort.rb:348:in block (2 levels) in each_strongly_connected_component' 	from /home/sahil/.rvm/rubies/ruby-2.1.5/lib/ruby/2.1.0/tsort.rb:418:in block (2 levels) in each_strongly_connected_component_from'
from /home/sahil/.rvm/rubies/ruby-2.1.5/lib/ruby/2.1.0/tsort.rb:427:in each_strongly_connected_component_from' 	from /home/sahil/.rvm/rubies/ruby-2.1.5/lib/ruby/2.1.0/tsort.rb:417:in block in each_strongly_connected_component_from'
from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/railties-4.2.0/lib/rails/initializable.rb:44:in each' 	from /home/sahil/.rvm/gems/ruby-2.1.5@curb/gems/railties-4.2.0/lib/rails/initializable.rb:44:in tsort_each_child'
from /home/sahil/.rvm/rubies/ruby-2.1.5/lib/ruby/2.1.0/tsort.rb:411:in call' 	from /home/sahil/.rvm/rubies/ruby-2.1.5/lib/ruby/2.1.0/tsort.rb:411:in each_strongly_connected_component_from'

You are reading the wrong documentation. See the version notice at the top of the readme:
https://github.com/chaps-io/public_activity#version-notice
and then proceed on clicking the link https://github.com/pokonski/public_activity/blob/1-4-stable/README.md


Want to add Drum kit using JS

Describe the project you want to add with tech stack
A Drum Kit Game
Expected behavior
You can play Drum in in this game with pressing key or with mouse
Please assign me this issue under GSSoC-2023

Project already present


Add missing component for msbuild path

Add missing component for msbuild path.
Revert the name change since that variable will not be available in VS command prompt.

[Current coverage][cc-pull] is 72.47%

Merging [#2][cc-pull] into [CB-11548][cc-base-branch] will decrease coverage by 0.15%

@@           CB-11548         #2   diff @@
==========================================
  Files            14         14
  Lines          1929       1929
  Methods         358        358
  Messages          0          0
  Branches        389        389
==========================================
- Hits           1401       1398     -3
- Misses          528        531     +3
  Partials          0          0

Powered by Codecov. Last updated by [5d4ac56...ab26dbe][cc-compare]
[cc-base-branch]: https://codecov.io/gh/vladimir-kotikov/cordova-windows/branch/CB-11548?src=pr
[cc-compare]: https://codecov.io/gh/vladimir-kotikov/cordova-windows/compare/5d4ac56159b779a426b03c1d93e901a2aaa2050c...ab26dbec810bed682c723165f88975c67440ee15?src=pr
[cc-pull]: https://codecov.io/gh/vladimir-kotikov/cordova-windows/pull/2?src=pr

@jicongw, I've pulled this into my branch and added a couple of minor tweaks in https://github.com/apache/cordova-windows/commit/9eb98efe23d9af92337751600a6b418b45745af3. Also updated all tests to pass. Please let me know if/when it's ready to go
Diff against current master is here: https://github.com/apache/cordova-windows/compare/master...vladimir-kotikov:CB-11548


ARROW-1879: [Python] Dask integration tests are not skipped if dask is not installed

These tests should only be run if Dask is installed, not part of the normal test run

Pytest Will Still try to discover them though if you do pytest -k test_foo.

I can change the issue/JIRA title if that's useful


✨ [Feature request]: Show Progress Status on Play Ideas Page

Is your feature request related to a problem? Please describe.
I was going through the list of ideas and had decided to work on an idea and when i came to the repository I saw a issue already created with the idea and in progress.
Describe the solution you'd like
Show a Complted/ In -progress/ Available tag on all the play ideas card.
Describe alternatives you've considered
NA
Additional context
Filter for the same can also be provided on the page.
i would be pleased to get the chance to work on this Feature.

An idea can be implemented by multiple people.

Since there haven't been any updates, we can move forward with closing this issue.


Exploration: FFS Instance Multiple Addresses

Just typing out some thoughts and questions about how this might work...
Supporting multiple addresses is actually not looking too hard.
We can maintain a list of wallet addresses associated with the instance. Any operation that uses an address (PushConfig, Replace, etc) will need to verify that the address to be used exists in the list of instance addresses.
Methods that previously returned the instance wallet address or balance information will be converted to return a map or slice of address related information.
Main question I'm seeing is where and when to specify the address to use for a transaction. We can:

Add the wallet address as part of the CidConfig and DefaultCidConfig Cold.FilConfig. This provides a consistent way to specify the address, and gives us a place to store the default. Problem here is that it give less control or more difficult control over what address to use during a Replace operation since it will use the address that was previously used for that cid. Maybe that address is out of funds or no longer valid for some reason. This would require the user to realize that and push a new config for that cid before running Replace.
Require that an address is supplied with every call that needs one (PushConfig, Replace). We could still have a default if no address is specified.

Another question is: Will it every be useful to know which address was used to store which cids? I don't think this information is already available since it was implied before to be the instance address. Do we want to add some tracking of wallet address <-> stored cid?
This also seems like an opportunity to add an optional display name associated with every address that is created. That looks like an easy change by modeling an address as a struct with name and addr fields, both will be persisted.

Main question I'm seeing is where and when to specify the address to use for a transaction.

Regarding both options, I think we'll need a mixture. When a CidConfig is renewed or repaired, it needs some address to indicate who will pay for that work. So sounds like the CidConfig should have some RenewRepairAddress or similar. Now it assumes it's the same address that pushed the CidConfig, but it might be a good opportunity to let the client select that.
Also, I agree that for Replace sounds a bit odd or something is missing.
I think we should add an optional CidConfig option for Replace, identical to PushConfig. If nothing is passed, it will use the same CidConfig as the replaced Cid (what it does now). But if the caller wants to change the address or even other facts about the configuration (still replacing the old cid), then it can do it.
What do you think?

Another question is: Will it every be useful to know which address was used to store which cids? I don't think this information is already available since it was implied before to be the instance address. Do we want to add some tracking of wallet address <-> stored cid?

I guess I'd say yes, if that's is necessary for some client. It sounds to me that we'll also need some other APIs to get all past jobs from a Cid, old Cid Logs to show more details, etc. But I was waiting more to have a clear view of what would be needed. In short, yeah, most prob we'll discover some more query-style apis to do.

This also seems like an opportunity to add an optional display name associated with every address that is created. That looks like an easy change by modeling an address as a struct with name and addr fields, both will be persisted.

Sounds good.

Ok cool, sounds like we're on the same page when it comes to specifying an address to use, think I can make some progress there.
Re: query cids by address, etc... Yea I was just assuming that might be useful, but you're right; Let's let the requirements drive those things. I'll dig more into the features list and see if that is even necessary.


Move security items from other to own category and support additional security features

I noticed the following security related items exist already and are all part of the category other:

http://caniuse.com/#feat=publickeypinning
http://caniuse.com/#feat=stricttransportsecurity
http://caniuse.com/#feat=subresource-integrity
http://caniuse.com/#feat=u2f
http://caniuse.com/#feat=chacha20-poly1305
http://caniuse.com/#feat=tls1-2
http://caniuse.com/#feat=tls1-1
http://caniuse.com/#feat=subresource-integrity
http://caniuse.com/#feat=upgradeinsecurerequests
http://caniuse.com/#feat=contentsecuritypolicy2
http://caniuse.com/#feat=contentsecuritypolicy

Since they are all closely related to security, please move them into a own category instead of in the category other.
Also, please add support information about HTTP Security headers like:

X-Content-Type-Options: nosniff
#1556 X-XSS-Protection: 1; mode=block
#1138 X-Frame-Options

And support information about security features like:

DNSSEC
#2055 DANE

X-XSS-Protection (#1556)
X-Frame-Options (#1138)
DANE (#2055)

are already requested.

@Schweinepriester Thanks, I changed the question a bit.

Any support for moving the above list from Other to a Security category?

I'm fine with the Security category. Don't think its urgent, though.
As Fyrd added the "Misc. site/data request" label, this issue isn't appearing in the https://caniuse.com/issue-list and can be voted for, which I wanted to do for DNSSEC.
Therefore there probably should be individual issues for the remaining features, X-Content-Type-Options: nosniff as well as DNSSEC.
Do you want to create those, Calico90, or shall I?
There is an issue for X-Content-Type-Options: nosniff: #2758
Do you want the honor for creating the issue for DNSSEC, @Calico90?

@Schweinepriester consider it done! https://github.com/Fyrd/caniuse/issues/3380

@Schweinepriester How can we create a security category? I have some more suggestions related to browser compatibility.

This one can be added to a security category too: https://caniuse.com/#feat=same-site-cookie-attribute
Also I will request support for HttpOnly and Secure cookies.

Here: https://github.com/Fyrd/caniuse/issues/3382 and https://github.com/Fyrd/caniuse/issues/3381

This one can also be in a security category: https://caniuse.com/#feat=referrer-policy

And https://github.com/Fyrd/caniuse/issues/3383

and https://github.com/Fyrd/caniuse/issues/1161 and https://github.com/Fyrd/caniuse/issues/1197 and https://github.com/Fyrd/caniuse/issues/1396

and https://github.com/Fyrd/caniuse/issues/2122 and https://github.com/Fyrd/caniuse/issues/2225

and https://github.com/Fyrd/caniuse/issues/1716

and https://github.com/Fyrd/caniuse/issues/2884

and https://github.com/Fyrd/caniuse/issues/3104

Done! "Security" is now its own category:
https://caniuse.com/#cats=Security

Great! :) Now let's hope one of the above suggestions might get more info soon to be added.
Here two more suggestions that would possibly fit into that category: https://github.com/Fyrd/caniuse/issues/3388 and https://github.com/Fyrd/caniuse/issues/3387.

As you can see confirmed here again: https://www.keycdn.com/blog/http-security-headers/ the caniuse websites perfectly first for browser-related security features too. :)

also https://github.com/Fyrd/caniuse/issues/3157

And https://github.com/Fyrd/caniuse/pull/3426 / https://github.com/Fyrd/caniuse/issues/2083


plot the Riesz function

Hello, I want to plot the Riesz function
And this is what I've done, I don't know if it's okay
import mpmath as mp
Riesz = lambda z: mp.nsum(lambda n: ((-1)**(n+1) * (-z)**n) / (mp.fac(n-1)*mp.zeta(2*n)), [1, mp.inf])

sorry if this is not an appropriate place

Hello, I want to plot the Riesz function
And this is what I've done, I don't know if it's okay

import mpmath as mp

Riesz = lambda z: mp.nsum(lambda n: ((-1)**(n+1) * (-z)**n) / (mp.fac(n-1)*mp.zeta(2*n)), [1, mp.inf])

sorry if this is not an appropriate place

According to the quoted definition it rather should be:
>>> from mpmath import nsum, fac, zeta, inf, plot
>>> riesz = lambda z: -nsum(lambda n: ((-z)**n) / (fac(n-1)*zeta(2*n)), [1, inf])
>>> plot(riesz, [0, 50])

I got this, which is close to the source:


Singularity: MIDI/OSC input

Probably should support MPE as well.
https://www.midi.org/articles-old/midi-polyphonic-expression-mpe

RtMidi input is working (though only verified on mac).
Still WIP.


[DOCFIX]Update Configuration-Settings.md forchinese version

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20434/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20568/
Test PASSed.

Merged build finished. Test PASSed.

@Lichongjie  a few more comments left. thanks!


Update dependencies

More specifically update cortex-m from 0.6.x to 0.7.x to reduce
the dependency tree a little, see https://github.com/rust-embedded/cortex-m/pull/190
Another interesting update is rtt-target (0.2.x -> 0.3.x)
I've lazily used cargo upgrade to update the dependencies, which
means patch release versions are also changed. I can revert these lines, if not desired :)

Looks good to me! Thank you @Sh3Rm4n :)
bors r+


this guid doesn't have guestmode enabled :(

Hello @oplik0 my friends ;)
I want to test your plugin.
I followed all the instructions (create discord server, install application WidgetBot on discord server, install the nodeBB plugin and configure widget discord chat plugin on ACP and this is the message I get on nodeBB :

???

hmm ok I must run /setup command with widget bot. It seems to be good


BatchData _aggregate_batch do not support list

When I use list in my get_data function.there throw exception like this:
/tensorpack/dataflow/common.py", line 132, in _aggregate_batch
raise TypeError("Unsupported type to batch: {}".format(type(dt)))
TypeError: Unsupported type to batch: <type 'list'>

Is there any way to support it?

Your previous dataflow produces a list of list, which is not supported by BatchData. BatchData only works for a list of [ndarray or scalar].
I don't know any good way to support it, because a list can contain anything...

In any case you can use BatchData(use_list=True) which does no type check. Otherwise it only makes sense to batch ndarray from ndarray.


nvidia-x11: rewrite build

nvidia-x11: rewrite build

Support for version 361+

NVIDIA has added experimental support for EGL, bringing wayland
support closer
Support for NVIDIA's new GLVND libraries

Fix i686 support (for whatever thats worth)

Cuda UVM is only supported on x86_64 for 346+

Fix kernel modules for some platforms & versions
Use the long-lived driver as the default

Previously it was unclear what the practice is for the default.

Add long-lived, short-lived, beta, vulkan drivers

Available as "nvidia/nvidiaLongLived", "nvidiaShortLived",
"nvidiaBeta", & "nvidiaVulkan" in services.xserver.videoDrivers

Remove support for 173.xx

This driver branch requires a version of xorg-server that is no longer
supported in nixpkgs.

Fix patchelf allowing --shrink-rpath to be used
Install everything explicitly to catch build failures due to changes
Cleanup shared object version symlinking, .so
Cuda fixes

Add missing cuda executables, libraries and manpages

GTK+3 support and used by default (GTK+3 is supported for 346+)

nvidia-settings now works correctly on high dpi displays

Add additional separation of kernelspace and userspace to fix packaging of
packages building against the nvidia-drivers. This is currently incomplete,
but the ground work for doing so is complete.
Add a consistent source unpack method
Add nvidia default application profiles
Add better documentation to builder
Add X.Org driver config
Add NVIDIA ICD config
Cleanup buildinputs (add missing deps & remove unused)

cc @vcunat
After taking a quick look at the current build I didn't want to decipher what is broken and what isn't. This is basically copied verbatim from my build in https://github.com/triton/triton/tree/master/pkgs/all-pkgs/nvidia-drivers, so it contain fixes for all branches and platforms.  I don't currently have a system to test this on, but I believe I fixed all attribute names so that it should just work within nixpkgs.

Related discussion:
https://github.com/NixOS/nixpkgs/pull/12719
Resolves issue:
https://github.com/NixOS/nixpkgs/issues/13942

@vcunat
Also, all branches build except the beta branch because NVIDIA renamed libGL.so.1 -> libGL.so.1.0.0 in a point release and the generic builder is based on using major versions for such changes.

We should patch out the "triton" strings, I think... but I only skimmed all this for now. I'll try to do a better review later today or tomorrow, unless someone beats me.

I'm concerned about the maintainability of a nearly 600-line build script, which is nearly quadruple the size of the previous builder.sh. Where does that increase come from?
GTK+3 is still not the default in Nixpkgs, so I don't think programs should use it if they can also use GTK+2, in order not to end up with multiple versions of GTK+3 in the system closure.
pkgs/os-specific/linux/nvidia-x11 should use callPackages rather than relying on a channel argument.

@edolstra
I only opened the PR at the request of @vcunat . If it isn't what you want you can manually add the fixes from my changes. I detailed the necessary changes in a linked PR that will resolve the errors, because as it stands there are packaging errors that will cause the driver to segfault. These changes do not effect me, so they are what they are, take it or leave it.  Also the builder is longer because it is a generic builder for all versions, that also includes better tests for catching errors.
lol, in 2016 you still prefer gtk2 as the default, have fun with that

I see this won't be as easy as I hoped, so I reverted that update for now to avoid crashes until we find a working and "acceptable" solution.

changes in 364.12 beta: https://github.com/triton/triton/commit/acc4ffa9ed267d1043e2f30426c55eb295799e8a

@vcunat does nvidia work now on 16.03 without this PR?

Yes, as I posted, for now I had reverted the branches to 358.16, as that version worked reliably and it really isn't that old.

Sounds like the issue got solved some other way, closing.


Feature/authentication branch

Authentication

On successful authentication a token is generated and returned to the user
When visiting a secure route, the token should be verified immediately

Created function that checks for valid token

Created function that decodes the JWT

Updated README

Instructions on how to generate public/private keys
How to add the keys to the gitignore
How to add .gitkeep to /config folder, so even if the folder is empty, it gets committed

Glad to hear you've got a plan to try Passport with socket.io! As you're making those changes, can you make sure to place any secrets (keys, etc.) in environment variables.
Also, our schema migrations are on develop now, so you should merge develop in to your feature branch again and test your registration and login socket-events with the database involved.
Finally, make sure that after a successful registration, the user ends up being logged in. No one likes having to log in after creating an account. To get the user-id generated by the database engine, use https://knexjs.org/#Builder-returning

alright, I'll close this pull request and get Passport going


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In c47ab8b, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 16198 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 7e07b2b.


Is there a version for reactjs?

Hi,
I like ztree very much,now I want to use it in reactjs. whether is there a version for reactjs?

I'm sorry


fix: esl-carousel-dots implementation bugfixes

:tada: This PR is included in version 5.0.0-beta.14 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Selectable/Highlightable text?

Wondering if its possible to have the text output be highlightable? Or would this require an external OCR solution?

Yeah, right now the PDF is generated first as an image, so any text information is lost, unfortunately. Rendering the text straight to PDF would be ideal but there aren't really any client-side options for that.

For those looking for non-image PDF generation with jsPDF. The current jsPDF documentation (for addHTML) points to this example which uses an older html2pdf library and is able to provide selectable text. https://cdn.rawgit.com/MrRio/jsPDF/master/examples/html2pdf/showcase_supported_html.html
However working with this version, I was unable to get proper page breaks or load custom fonts so I opted for @eKoopmans work instead.

For those of you that need it. http://pdfmake.org/#/ does a good job of creating lightweight client-side text-based PDF's
Unfortunately, all of the layouts have to be done in JSON objects and there is no real way I have found to convert an HTML page layout using this library. That said, it works well. Ideally, the best solution would be a hybrid between these two libraries.


DockerComposeTest does not pick up FeatureSpecs

When running the dockerComposeTest task, FeatureSpecs are not being picked up and ran.
[info] Done packaging.
ScalaTest 3.0.5 (Built for Scala 2.12)
Discovery starting.
Discovery completed in 83 milliseconds.
Run starting. Expected test count is: 0
DiscoverySuite:
Run completed in 190 milliseconds.
Total number of tests run: 0
Suites: completed 1, aborted 0
Tests: succeeded 0, failed 0, canceled 0, ignored 0, pending 0
No tests were executed.

I've also tried to specify the style: testExecutionArgs := "-y org.scalatest.FeatureSpec -v" but to no avail.

Hi @corbs9 - Would you be able to provide any sample code for what's not working so I can attempt to reproduce it myself?
I tried modifying the basic-with-tests example so that it uses a FeatureSpec and it appears to be working:
import org.scalatest._
import org.scalatest.FeatureSpec
import org.scalatest.GivenWhenThen
import scala.collection.mutable.Stack

class BasicAppSpec extends org.scalatest.FeatureSpec with GivenWhenThen {
  feature("Test feature") {
    info("Test info")
    scenario("Test scenario") {
      println("In Test")
      assert(1 == 1)
    }
  }
}

build.sbt needs this change as well:
//testTagsToExecute := "DockerComposeTag"
As you can see it is picking up the FeatureSpec based test and running it:
> dockerComposeTest 571429
Starting Test Pass against the running local Docker Compose instance: 571429
[info] Compiling and Packaging test cases using testCasesPackageTask ...
Discovery starting.
Discovery completed in 48 milliseconds.
Run starting. Expected test count is: 1
BasicAppSpec:
Feature: Test feature
  Test info
In Test
  Scenario: Test scenario
Run completed in 101 milliseconds.
Total number of tests run: 1
Suites: completed 2, aborted 0
Tests: succeeded 1, failed 0, canceled 0, ignored 0, pending 0
All tests passed.

It might also be good to run the following and ensure that your FeatureSpec tests are being included in the package:
test:clean
test:packageBin

> test:clean
[success] Total time: 0 s, completed May 4, 2018 12:42:12 PM
> test:packageBin
[info] Updating {file:/Users/kurtkopchik/Source/sbt-docker-compose/examples/basic-with-tests/}basic-with-tests...
[info] Resolving jline#jline;2.11 ...
[info] Done updating.
[info] Compiling 1 Scala source to /Users/kurtkopchik/Source/sbt-docker-compose/examples/basic-with-tests/target/scala-2.11/classes...
[info] Compiling 1 Scala source to /Users/kurtkopchik/Source/sbt-docker-compose/examples/basic-with-tests/target/scala-2.11/test-classes...
[info] Packaging /Users/kurtkopchik/Source/sbt-docker-compose/examples/basic-with-tests/target/scala-2.11/basic_2.11-1.0.0-tests.jar ...
[info] Done packaging.

There are some plugin settings to override the default test jar package building process if you have a non-standard setup:
testDependenciesClasspath := // The path to all managed and unmanaged Test and Compile dependencies. This path needs to include the ScalaTest Jar for the tests to execute. This defaults to all managedClasspath and unmanagedClasspath in the Test and fullClasspath in the Compile Scope.
 testCasesJar := // The path to the Jar file containing the tests to execute. This defaults to the Jar file with the tests from the current sbt project.
 testCasesPackageTask := // The sbt Task to package the test cases used when running 'dockerComposeTest'. This defaults to the 'packageBin' task in the 'Test' Scope.

I hope this helps.

Hi @kurtkopchik,
It seems that yes, any dependencies which the project relies on which is present in a separate module are not being picked up by the dockerComposePlugin and so, in order to work around this problem, an additional configuration is required:
.settings(
    testDependenciesClasspath := (
      testDependenciesClasspath.value +:
        Seq((commons / classDirectory in Test).value).map(_.getAbsoluteFile)
    ).mkString(pathSeparator)

Happy to close this issue unless you think there is another way this can be achieved via the plugin?


why i always need to download or fetch files when i run the example workflow?

I load example workflow from the '_Example_Workflows' ,but every time,when i run the workflow,always tell me need to download the model file.I know that some node need to set the option 'force_download' into 'True',but why i always need to download the model file every time?
Below is the download nodes :

The workflow is: Unique3D_with_Era3D_All.json
Screen shot of the workflow:

Always downloading like this:

Can u tell my why?Thanks for your help.

I checked the model file 'checkpoints' stored in 'ComfyUI-3D-Pack',and i think all the models have been download,the capacity of the 'checkpoints' has reached '70G',but why i need to download again and again when i run the workflow?
Structure of the folder 'checkpoints'

And the capacity of the foleder 'checkpoints'

Thanks for your care.

Now i am running the example workflow of :Unique3D_with_Era3D_All.json
Still need to download the model file showed below:

This is not the first time i run this workflow!!!

Below is the Structure of 'MrForExample/Unique3D' stored in the 'checkpoints' folder:

The capacity of Unique3D:

Did i miss something?

This cost me lots of time.

I got the key.This issue can be closed.
Thanks.

Just in case you havnt. Also keep in mind to uncheck the 'force_download' flag in your Load Pipeline node to not redownload the model all the time.


add file-exts

added pdf and image file processing

Codecov Report

Merging #218 into master will increase coverage by 0.38%.
The diff coverage is 86.88%.

@@            Coverage Diff             @@
##           master     #218      +/-   ##
==========================================
+ Coverage   70.09%   70.47%   +0.38%
==========================================
  Files          18       18
  Lines        5373     5483     +110
==========================================
+ Hits         3766     3864      +98
- Misses       1607     1619      +12

Impacted Files
Coverage Δ

stix2elevator/options.py
77.57% <50.00%> (-0.61%)
:arrow_down:

stix2elevator/convert_pattern.py
77.80% <87.09%> (+0.48%)
:arrow_up:

stix2elevator/convert_cybox.py
62.45% <87.71%> (+1.20%)
:arrow_up:

stix2elevator/common.py
100.00% <100.00%> (ø)

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 fa789dd...7ad9305. Read the comment docs.

Weird - I definitely fixed that - maybe I didn't commit it :-(


MySQL foreign key error

Hi,
like @enriquetoledano reported on issue #204 mysql has a problem with foreign key constraint.
In the same issue @genietto84 shown one possible solution:

check on the MySql and the error is:
Error in foreign key constraint of table timeoff/departmentsupervisor:
FOREIGN KEY (department_id) REFERENCES Department (id) ON DELETE SET NULL ON UPDATE CASCADE, FOREIGN KEY (user_id) REFERENCES User (id) ON DELETE CASCADE ON UPDATE CASCADE) ENGINE=InnoDB:
You have defined a SET NULL condition though some of the
columns are defined as NOT NULL.
I made this change on lib/model/db/department.js on line 73
foreignKey : 'department_id',
Become:
foreignKey : {name: 'department_id', allowNull: false},
And seem to work fine.

Are you willing to accept a PR?

Yes, please
Sent from my iPhone

On 6 Jun 2018, at 19:05, António Pedro Martins<EMAIL_ADDRESS>wrote:
Hi,
like @enriquetoledano reported on issue #204 mysql has a problem with foreign key constraint.
In the same issue @genietto84 shown one possible solution:
check on the MySql and the error is:
Error in foreign key constraint of table timeoff/departmentsupervisor:
FOREIGN KEY (department_id) REFERENCES Department (id) ON DELETE SET NULL ON UPDATE CASCADE, FOREIGN KEY (user_id) REFERENCES User (id) ON DELETE CASCADE ON UPDATE CASCADE) ENGINE=InnoDB:
You have defined a SET NULL condition though some of the
columns are defined as NOT NULL.
I made this change on lib/model/db/department.js on line 73
foreignKey : 'department_id',
Become:
foreignKey : {name: 'department_id', allowNull: false},
And seem to work fine.
Are you willing to accept a PR?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Intermittent ExceptionFromError Failures

I've been hesitating opening an issue for this because it is so intermittent but it has appeared in both unit test runs and in production runs.   The fault occurs at various places in my code but it always stems from calling xray.AddError(ctx, e) in a manner similar to:
func Example() {
	ctx, xs := xray.BeginSubsegment(ctx, "Example")
	workerCount := 100

	rs := my.Records()

	var wg sync.WaitGroup
	rc := make(chan *my.Record)
	for i := 0; i < workerCount; i++ {
		wg.Add(1)
		go func() {
			for {
				r, more := <-rc
				if more {
					e := my.Queue(ctx, Message(r))
					if e != nil {
						log.Error().
							Err(e).
							Str("record_number", r.Number).
							Msg("error queueing record")
						_ = xray.AddError(ctx, e)
					}
				} else {
					wg.Done()
					break
				}
			}
		}()
	}

	for _, s := range rs {
		rc <- s
	}
	close(rc)
	wg.Wait()
	xs.Close(nil)
}

That is being called inside of a go func() that is part of a WaitGroup reading records off a channel (I've updated the example to more clearly articulate this).  Is it possible there is a concurrency problem with my call to AddError? Should I be doing that in a different way?  Note that this code is being run in a Lambda function so it runs inside the Facade segment.
This is consistently the trace of the exception.
github.com/aws/aws-xray-sdk-go/strategy/exception.(*DefaultFormattingStrategy).ExceptionFromError(0xc0000a4d00, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/home/mike/go/pkg/mod/github.com/aws/aws-xray-sdk-go@v1.1.0/strategy/exception/default_exception_formatting_strategy.go:132 +0x186
github.com/aws/aws-xray-sdk-go/xray.(*Segment).addError(0xc00043b980, 0x0, 0x0)
	/home/mike/go/pkg/mod/github.com/aws/aws-xray-sdk-go@v1.1.0/xray/segment.go:593 +0x112
github.com/aws/aws-xray-sdk-go/xray.(*Segment).AddError(0xc00043b980, 0x0, 0x0, 0x0, 0x0)
	/home/mike/go/pkg/mod/github.com/aws/aws-xray-sdk-go@v1.1.0/xray/segment.go:585 +0x7e
github.com/aws/aws-xray-sdk-go/xray.AddError(0xa58b60, 0xc000407410, 0x0, 0x0, 0x3, 0xc00029ea80)
	/home/mike/go/pkg/mod/github.com/aws/aws-xray-sdk-go@v1.1.0/xray/context.go:104 +0x87

Hi,
Thanks for diving xray sdk. The code seems not safe because there might be race condition that more than one goroutines call xray.AddError(ctx, e) at the same time. Could you try to replace line _ = xray.AddError(ctx, e) to add e into another channel, then raise a new goroutine to poll Err from this channel and call _ = xray.AddError(ctx, e) ? by this way it makes sure no race condition in xray.AddError
Thanks

@wangzlei thanks for the reply.  Ok, that makes sense.  I don't think I get enough value from the additional error metadata in my X-Ray trace to go through that effort, I can easily extract that data from my logs.
Here's a quick question before I close this issue.  Are any of the SDK functions considered safe for concurrent use in Go?  I don't see anything in the docs one way or the other.

Sorry that in doc we don't mention too much about sdk's concurrency safety. In goLang world by default we suppose the sdk is not concurrency safety and have to use Channel or lock to handle multi-threads case, unless explicitly say it is concurrency safety.

Thanks @wangzlei


How to pass flow props through React components?

I create example of my question here
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVBLAtgBzgJwBdkwBDAZzACUBTUgY2KnzizAHJ87H2BudejAqUAYhnzliNAB6EaAOwAmlWg0IA6AMKs88hYQA8AbwC+APjBHUYMFyU18ACgCUl62ACQXQgFd88sAMAWQxFRRgaMgBeIwBGEzAAIxiAJhNgM3cTVGzUQWEwELCIsBk5JRVuDW1cOD15QysbRIAuMHkfLESHABociybbBUUHFzcbLxpff0CAGQpiI3VlwgALDHJ1HBYccgT6GLWNrZ3NxLAAajB4jKycgSFySnnJUtlhyrUtHTr9Y3cbKQ2h0ur0AUlgZ1uvg+jYbPRIaCYf1xkN7E5XIMbN4-AEDIoMAA3MxGI6bbZwXbqUiXMBkk6Us60+kUqn0EwGYAE4n8GzZExAA
I have chain of 3 components (could be more) each in separate file. Middle component passes all his props through and add one more. How to define flow types for props correctly in case, where middle component shouldn't know anything about props he just passing though? Specifically, I don't want to include type for prop a in the file with Middle component.

Well, Middle component knows about Last.  Why not show this fact in it’s props explicitly?

@apsavin in other words, do you suggest to do like this ?
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVBLAtgBzgJwBdkwBDAZzACUBTUgY2KnzizAHJ87H2BudejAqUAYhnzliNAB6EaAOwAmlWg0IA6AMKs88hYQA8AbwC+APjBHUYMFyU18ACgCUl62ACQXQgFd88sAMAWQxFRRgaMgBeIwBGEzAAIxiAJhNgM3cTVGzUQgBPHEiQsIiABRYcSii3G0SALjB5HyxEhwAaHLAAHzAAGQpCCrgq-lRBYTAS8MiZOSUVbg1tXDg9eUNp8sryCysbO0UHF1rPbz8AgwHJS3U7wgALDHJ1HB2E+hjH59ed9USwABqMDxDJZHLoApFfqDYZVMA1fY2MiNZqtDruOqolptfCdZFgejY9F4nJjCbkSjXKSyBTKahLLQ6Nb6K6wnZ7dyHY6uJEHGi+fyBRQYABuZiM3xebxGL1IQLAUt+sv+CqVMqq6noJgMwBF4v4NmyJiAA

Maybe something like this


Create a Support page on the application

This could be an edited version of the (for now deleted) support page of the Plum Tree app, or an entirely new one. Notable content should be:

An FAQ:

Discuss Storage Quota issues

A link to the GitHub page
A link to the GitHub issues page
Maybe other contact options?

Closed by commit dc06696c7b66fef926144d98be18766f7d29730f


Can tensorrt-llm or how tensorrt-llm support that seprating the  prefill stage and decode stage in different GPU or different nodes with self configuration

such as https://github.com/vllm-project/vllm/pull/2809 and https://github.com/LLMServe/DistServe that had done
reference:https://arxiv.org/pdf/2311.18677

DistServe is on our roadmap, and will be supported in the following versions

DistServe is on our roadmap, and will be supported in the following versions

Thank you for your reply！ I look forward to the future implementation of this feature, which will be a great improvement！


An included record parameter needs to be specified after any required/defaultable parameters

Purpose

$title.
According to the spec param list should be in the following order.

param-list :=
   required-params [, defaultable-params] [, included-record-params] [, rest-param]
   | defaultable-params [, included-record-params] [, rest-param]
   | included-record-params [, rest-param]
   | [rest-param]

Fixes #27818
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@chiranSachintha The commit messages/PR heading is not according to the best practices. Please correct them.

[1] https://github.com/ballerina-platform/ballerina-lang/blob/master/CONTRIBUTING.md#submit-your-contribution