SystemVerilog ( Parth2031/Colorful-Comments #12 )

Hi - can you please add support for SystemVerilog files (and maybe VHDL files too)? this is the main language we use

from jonnyboynewton
original issue

level of details : extremely low, more details would be appreciated ( comment syntax, language identifier )

needed to work all by myself

VHDL and SystemVerilog do not have the same comment syntax, had to figure out by myself ( details would have been appreciated )

Ressources :

extension : https://github.com/mshr-h/vscode-verilog-hdl-support

suggested fix :

languageConfig.yaml

  # * SystemVerilog
  # https://github.com/Parth2031/Colorful-Comments/issues/12
  # according to extension : mshr-h.veriloghdl
  # github : https://github.com/mshr-h/vscode-verilog-hdl-support
  systemverilog: *c_delim
  vhdl: *double_dash_delim


CRC/checksum Error

I am getting CRC error with pi pico. Can you tell what might be going on here?
I printed the buffer after the read and get the following
b'\x03\x04\x01\xf9\x00\xa1\xaf\x00'
so i set buf equal to the above
buf = b'\x03\x04\x01\xf9\x00\xa1\xaf\x00'
Next, I manually ran a few lines to see what was going on and possibly find a work around.
a = crc16(buf[:-2])

a -> 40417

b = ustruct.unpack('<H', bytearray(buf[-2:]))[0]

b -> 175

if (b != a):             raise Exception("checksum error")
Maybe this is an issue related to raspberry pi pico?

You can close this issue. It was due to the pi pico having a 2 byte read register I suppose. I fixed it in the constructor by using
self.buf = bytearray(16)
instead of
self.buf = bytearray(8)

I removed the ustruct dependency in the latest version. Replaced with a simple bit shift. x << 8 | y


Read the docs port

Port all documentation to a read the docs site instead of using plain markdown files.
Closes #109

Coverage remained the same at 87.221% when pulling 36f92f5d752fbbfba683f6ea5226cd7a01b02ff3 on read-the-docs-port into f3b9c773838914ca4efe59e818b54b18cde6a511 on unstable.

Documentation is not perfect but it still works/builds so it is acceptable as a first step.


Unique WriterKey() might lead to issues with multiple instance

I'm working on a similar module for Graphite and I just found out if WriterKey() is unique across instance, then it's always the same instance used for login.
This becomes a problem if for some reason you want to have multiple InfluxDB Loggers for a site

Thanks for bringing this up. I am currently not planning on having multiple InfluxDB Loggers. But I keep this open to remind me or others of that


2 latexmkrc creates single index and multi index instead of choosing

merge corrections on indices into branch referring to glossaries.

Hope all right


coco_resnet_50_map_0_335.pt

I  download  coco_resnet_50_map_0_335.pt   from  https://drive.google.com/open?id=1hCtM35R_t6T8RJVSd74K4gB-A1MR-TxC     ,when  load   using torch.load("  coco_resnet_50_map_0_335.pt")  it  raise error :
result = unpickler.load()
ModuleNotFoundError: No module named 'model'
what is  the problem?

You need to be in the repository's root folder, where the model.py is located, when starting the python shell. Also if you're using python3 you need to add encoding='latin1' to your load instruction.
However I could not run the model: later it says
AttributeError: 'Conv2d' object has no attribute 'padding_mode'

I download coco_resnet_50_map_0_335.pt from https://drive.google.com/open?id=1hCtM35R_t6T8RJVSd74K4gB-A1MR-TxC ,when load using torch.load(" coco_resnet_50_map_0_335.pt") it raise error :
result = unpickler.load()
ModuleNotFoundError: No module named 'model'
what is the problem?

I can download the pre-trained model using the link: https://drive.google.com/file/d/1hCtM35R_t6T8RJVSd74K4gB-A1MR-TxC/view?usp=drive_open
Could you share me the pre-trained model(coco_resnet_50_map_0_335.pt)

I download coco_resnet_50_map_0_335.pt from https://drive.google.com/open?id=1hCtM35R_t6T8RJVSd74K4gB-A1MR-TxC ,when load using torch.load(" coco_resnet_50_map_0_335.pt") it raise error :
result = unpickler.load()
ModuleNotFoundError: No module named 'model'
what is the problem?

I can not download the pre-trained model using the link: https://drive.google.com/file/d/1hCtM35R_t6T8RJVSd74K4gB-A1MR-TxC/view?usp=drive_open
Could you share me the pre-trained model(coco_resnet_50_map_0_335.pt)

import torchvision.models as models
and then change model to models
I guess that will help

Somehow hacked a solution by making symbolic links to all .py files from the retinanet folder to the main folder.
"ln -s retinanet/model.py model.py"


parallelize downloads

It's taking too dang long!
🏠 ng@crimelab ~/dev/github/himawari-urls (master) $ time himawari-bg
Attemping to fetch latest visible light images from Himawari 8
Downloading satellite images [====================] 100% 0.0s
Setting /Users/ng/Pictures/himawari-images/1455267707496.jpg as background...
Complete!

real	0m24.056s
user	0m1.300s
sys	0m0.423s

My code uses async.eachSeries, but it could EASILY use async.each or async.map which runs async operations in parallel. However, I don't think it quite works for some reason related to the file stream/piping logic. I'll run some timing tests on Monday and compare the two operations.
Parallelizing the downloads would make the logs more awkward but that can be a compromise in favor of speed.

Using parallel by default in 1.0.0-beta. Thanks @jakiestfu for adding it to himawari.js!


Data reading!

Could someone help me with a simple example of reading 64 bytes?
On the examples, there is no kind example! Just writing!

Normally, you can't read on demand at the USB transport layer; it would be inefficient to transfer data in this way. At best, you would design your peripheral USB device to accept a written command and then push data back in response using a raw HID report or similar.
Along those lines, the raw_data.py example is probably the closest to what you want to see. This lets your PC-side app listen for new data coming from the peripheral device, and then the device just pushes whatever data it wants to whenever it's available:
https://github.com/rene-aguirre/pywinusb/blob/master/examples/raw_data.py


Update functions-event-grid-blob-trigger.md

It seems like the documentation on blob triggers and event grid triggers got smooshed together.
I had to change my invocation url in the local context to replace blob with event grid for it to work.
For a python function, this url worked for me: http://localhost:7071/runtime/webhooks/eventgrid?functionName=FuncNameWithNoHostDotFunctionsInIt
I'm using the v3.0.3388 of the func tools.

@irwinwilliams : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@cachai2 - can you take a look?

@cachai2 @mijacobs - Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

Please add @ggailey777 as a reviewer

#reassign:ggailey777

@irwinwilliams this is a bit of a hybrid, since it's actually a Blob Storage triggered function that is using Event Grid as a source (for lower latency). Your changes would only work when using an Event Grid trigger with blob storage as the source, which is just a bit different. This article has been updated and verified recently. If you are still having issues with is in the current state, please create a feedback item. Note that the article currently doesn't support the Python v2 model for Azure Functions, but this work is being tracked. #please-close


iOS StopListening() After 3 Seconds

Is it possible to have speechRecognition() only listen for 3 seconds in iOS?

Yes, you can use setTimeout to call SpeechRecognition.stopListening() after so much time as passed.


Album artwork

Hi, just found this app because I don't use Spotify so this is great! Just if the album artwork could be displayed as well as the title, album, etc.
Also if it could be a background app in the menu bar instead of the dock would be great :D.
Keep the good work!

Hi, sadly the public Discord RPC API doesn't allow to dinamically change the displayed picture, so I had to stick with a bland iTunes logo.
When I have time to work on this I will make the app a menu bar app (if it's possible with Java).

Isn't it possible with this?
https://discordapp.com/developers/docs/rich-presence/best-practices#have-interesting-expressive-art

Isn't it possible with this?
https://discordapp.com/developers/docs/rich-presence/best-practices#have-interesting-expressive-art

It would be really nice if this gets album artwork! Commenting to "up" this.

Isn't it possible with this?
https://discordapp.com/developers/docs/rich-presence/best-practices#have-interesting-expressive-art

No, that's how I show the iTunes logo in the first place, but I can't use that to show the current artwork, because I can't load the image at runtime (you can only set certain pictures before and then use those). Spotify is able to do that because they probably use a different API, which isn't accessible for other applications.


Rename TableDropper to DbObjectDropper

I'd propose to change the interface to something like
namespace Spatie\MigrateFresh\DbObjectDroppers;

interface DbObjectDropper
{
    public function dropAll();
}

to better reflect the  reality - tables are not the only database objects that need to be taken care of.
@freekmurze What is your take on it? I can do the PR and probably help with MySQL and SQLite droppers.

See comments on #22


Colors played added to match trends

Moved colors played to match trends
If this PR is accepted, will delete other PR

This looks good, rename the title to "Commander Colors Played", and then
you can check in :)
On Wed, Sep 27, 2023 at 9:39 AM Adi @.***> wrote:

[image: image]
https://user-images.githubusercontent.com/140542996/271052831-cf4a06c9-f81d-42ee-a562-02c80bcb419d.png
—
Reply to this email directly, view it on GitHub
https://github.com/davidpchi/toski/pull/75#issuecomment-1737741449, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHIN4USW7SNVDGANZJ5X73X4RJF3ANCNFSM6AAAAAA5JUU7OM
.
You are receiving this because your review was requested.Message ID:
@.***>


Could Taskview return always the same HtmlBody instead of creating?

use case:
    taskView.body()
    .div()
      .classAttr(divClass)
      .idAttr(divId)
      .addAttr("toto", "tutu").form("/action.do");

     taskView.body()
      .div()
      .idAttr("id2")
      .addAttr("toto", "tutu").form("/action2.do");

Proposition
public class HtmlView<T> extends HtmlWriterComposite<T, HtmlView<T>>{

     public HtmlHead<T> head(){return addChild(new HtmlHead<T>());}

    private HtmlBody body = null;
    public HtmlBody<T> body() {
        if (body == null) {
            body = addChild(new HtmlBody<T>());

        }
        return body;
    }
```

I cannot totally agree with your idea for a couple of reasons. But I think that I have an alternative approach. First, what I don't like too much:

duplicating the storage for HTML elements both in parent class HtmlWriterComposite and into an additional body field of the derived class. That could be error prone.

duplicating the semantics of the body() method. These kind of methods are factories and I think we should not mess up their meaning: between a factory and a getter.

My suggestion would be to create an additional method in HtmlWriterComposite:
<S extends HtmlWriter<?>> S getChild(String id)

Or even:
<S extends HtmlWriter<?>> S getChild(String id, Class<S> childClass)

Then in HtmlView you could create an overloaded method body(String id) or getBody(String id) (to clearly distinguish from body()) that would reuse the base method getChild(…) to return the body element with the specified id. What do you think?

I join you on that. Using an id system was my second approach. I have to think a little bit more about the getChild()/body(String id) approach but I think it could answer my requirements.
Do you agree on this :
We cannot have 2 elements with the same id in an html page?
We might want to add additional elements to a parent (use case : We want to add a table and a form in the DIV element), with this code, it is not possible)
I proposed to add the body system be cause, here we have a directional graph (with children field), we cannot go  up to add a second child to a node. Using the body approach allowed to add nested elements to the body without implementing a complicated bi directional graph or whatever.

I am preparing a detailed response with my proposal. I will post it in 5 minutes.

We cannot have 2 elements with the same id in an html page?

Ideally we shouldn’t. But it is not easy. However, we can implement that feature in part and leave the rest for the future.
What I mean is that it could be easy to validate if an element already exists with the same id in the parent’s children storage. However, we should continue looking for that id in the parent’s parent and so on, until we reach the root element (eventually the HtmlView). And this is not trivial. Once, the children is a List<HtmlWriter<?>> then we must check if every element is a composite, or not, to proceed the lookup. I don’t like it and moreover, it will add a performance overhead.
Another alternative approach is to postpone the verification to write time. Thus, when we are writing the HTML to the PrintStream we can check if there are repeated ids and throw an error in that case.
Nevertheless, I think that this is not a priority feature and for now I would just consider that a composite has NOT two child elements with the same id.

We might want to add additional elements to a parent
(use case : We want to add a table and a form in the DIV element),
with this code, it is not possible)

I implemented my previous proposal and with that I can write:
    taskView.body()                 // create the body
            .div()                  // add a div to the body
            .classAttr(divClass)
            .idAttr(divId)         // add the divId to the previous div
            .addAttr("toto", "tutu")
            .form("/action.do");   // add a form to the div with divId

    taskView.getBody()     // get a body with any Id
            .getDiv(divId) // from the body, get the div with divId
            .table()       // add a table to the div with divId
            .tr();

To that end I had the following method to HtmlWriterComposite:
protected <S extends HtmlWriter<?, S>> S getChild(Optional<String> id, Class<S> childClass) {
		Predicate<HtmlWriter<?, ?>> p = elem -> elem.getClass() == childClass;
		p = p.and(elem -> !id.isPresent() || id.get().equals(elem.getIdAttribute()));
		return (S) children.stream().filter(p).findFirst().get();
	}
/// NOTE: I did not test this code

I choose the second option of my first proposal, because we will need to return a concrete element of type S. But, this implies another modification, because we need to invoke elem.getIdAttribute() for every element in children. Once children is a List<HtmlWriter<?>> we do NOT have the  getIdAttribute() on its elements. So,  the easiest way that I find to solve this problem was to modify HtmlWriter declaration to:
/**
 * @param <T> The domain object type
 * @param <S> The type derived from HtmlWriter
 */
interface HtmlWriter<T, S extends HtmlWriter<T, S>> extends HtmlSelector<S>`

I don’t like it very much. Ideally we should have a class between HtmlWriter and HtmlWriterComposite  that implements both interfaces: HtmlWriter and HtmlSelecter. But for now, that was the quickest way to solve it.
And finally we can add the desired methods to the concrete elements:
public class HtmlView<T> extends HtmlWriterComposite<T, HtmlView<T>>{
…
	public HtmlBody<T> getBody() {return getChild(Optional.empty(), HtmlBody.class);}

and:
public class HtmlBody<T> extends HtmlWriterComposite<T, HtmlBody<T>>{
…
    public HtmlDiv<T> getDiv(String id) {return getChild(Optional.of(id), HtmlDiv.class);}

hi again,
The solution
  taskView.body()                 // create the body
            .div()                  // add a div to the body
            .classAttr(divClass)
            .idAttr(divId)         // add the divId to the previous div
            .addAttr("toto", "tutu")
            .form("/action.do");   // add a form to the div with divId

    taskView.getBody()     // get a body with any Id
            .getDiv(divId) // from the body, get the div with divId
            .table()       // add a table to the div with divId
            .tr();

is fine by as it answers my usage.

What I mean is that it could be easy to validate if an element already exists with the same id in the parent’s children storage. However, we should continue looking for that id in the parent’s parent and so on, until we reach the root element (eventually the HtmlView). And this is not trivial. Once, the children is a List<HtmlWriter<?>> then we must check if every element is a composite, or not, to proceed the lookup. I don’t like it and moreover, it will add a performance overhead.

What I had in mind is more like a context associated to the HtmlView. A bit like an application context and beans in Spring or CDI. The method addChild() could save a reference Map <id, element>, so we can do anywhere
taskView.getDiv(divId)

Well this is just an idea, not a real need anyway. I am trying to keep id simple.

I merged your last commit from kenji-getpowered/HtmlFlow/tree/master/, because I would like to make some general fixes.

Since release 2.0, HtmlFlow follows the Html5.xsd specification and all Element classes are generated with automated xmlet support. So,  now we have a generic way of looking for an element given a Predicate which is available in all element classes:
<R extends Element> Stream<R> find(Predicate<Element> var1);

Although it is not so easy as the first proposed approach, such as HtmlDiv<T> getDiv(String id), yet the new 2.0 features also provide an easy way of navigating in html elements without ids. Regarding the OP use case which creates undesirable body elements twice :
    taskView.body()
    .div()
      .classAttr(divClass)
      .idAttr(divId)
      .addAttr("toto", "tutu").form("/action.do");

     taskView.body() // !!!!! Creating a 2nd body
      .div()
      .idAttr("id2")
      .addAttr("toto", "tutu").form("/action2.do");

now we can avoid it in the following way since release 2.0:
taskView
  .body()
    .div()
      .attrClass(divClass)
      .attrId(divId)
      .addAttr(new BaseAttribute("tutu", "toto"))
    .º() // div
    .div()
      .attrId("id2")
      .addAttr(new BaseAttribute("tutu", "toto"))
          .form("/action2.do");


Recommended way to access args after --

I'm building an application that will ultimately call another script. I would like to have a way to delineate between the commands proper args and options versus whats after the --
doh command --option --name bob -- --pass

How can I get everything after the -- using commander, or do I need to inspect the $* on my own.

Anything after the -- is treated as arguments. If you have a command like the following:
command :foo do |c|
  c.action do |args, options|
    say "Arguments:"
    say args.inspect
    say "Options:"
    say options.inspect
  end
end

Then everything after the -- will be present in args:
$ ruby test.rb foo -- -x
Arguments:
["-x"]
Options:
<Commander::Command::Options >

My apologies, let me try again.
Given the following command
command :foo do |c|
  c.option '--option'
  c.option '--name', String, 'the name'
  c.action do |args, options|
    # elided
  end
end

Let's say I invoke this like
$ ruby test.rb foo somearg --name bob -- -x
Arguments:
["somearg", "-x"]
Options:
<Commander::Command::Options >

How can I know that the -x came after the --?

Ah, I see, thanks for the clarification. That is not supported in commander, because it isn't supported by the underlying OptionParser library in Ruby's standard lib. As far as I can tell it is not common for standard option parsing libraries to make this distinction. Please let me know if you've found a counterexample.
In practice, can't you simply split the first and rest of the arguments?

@ggilder totally understand - just coming from rust w/ clap - https://github.com/clap-rs/clap/issues/971


Only 3 out of many Java files analyzed for Gradle project

I tried infer on my project:
https://github.com/martinschaef/bixie/tree/master/tool
calling gradle clean and then
infer -- gradle build
and it replies with
Starting analysis (Infer version v0.1.0)
Analysis done
3 files analyzed
No issues found
However, there are significantly more java files in this project. What am I doing wrong?

Hi Martin,
You are not doing anything wrong--looks like there's an issue with Infer's Gradle support. The Gradle wrapper is quite primitive and just looks for calls to javac in the verbose output for Gradle. For some reason, these calls do not appear for your compileJava rule (I suspect it has something to do with the `ant options?).
I was able to get a little further by bypassing Gradle with find src/main/ -name *.java | xargs infer -- javac -cp .:lib/* from bixie/tool/, but then I ran into #3. We're working on fixing both that and the Gradle problem; thanks for the report!

Hi Sam ... small world! Are you with Facebook now? I'll follow the other issue as well then. Are you going to release a new version or should I build from source?

Hi Martin,
Yes, I work at FB part-time now and am joining full-time in the fall! I am not sure what our policy on releasing new binaries is going to be--my guess is that we'll push bugfixes to source ASAP but that we will only release binaries for new major versions. So it might be worthwhile to try building from source.

Ok. The gradle problem is that I do some aspect weaving stuff which overrides the compileJava task. Once my I remove line 43 and 44-65 in build.gradle it finds everything but runs into the exception which you mentioned in the other issue. Thanks.

Hi @martinschaef , #3 has been closed by the latest Infer release and it looks like this would solve your issue too. Feel free to reopen if v0.1.1 doesn't fix it for you.


V2.2.0

The new v2.2.0 is nearly complete, so I'm opening this PR.
Very big changes....more information coming soon.

Merging even though auto-dungeon waypoints are incomplete. Will update as time progresses.


Create function code action issues

Description:

Expected :

Steps to reproduce:
1.
    [int|string] tuple = getTuple();

    int x = <int> getValue();

type IoError distinct error;

type FileErrorDetail record {
   string filename;
};

type FileIoError IoError & error<FileErrorDetail>;

public function main() {

    FileIoError err = getError();
}

Affected Versions:
2201.0.0-SNAPSHOT

For use-case 2, isn't this the possible outcome? If we consider the type cast expression, if the return type of getValue is decimal, int or float, then this can be compiled. This should be why, when we get type of getValue() expression, we get the CompilationErrorType.

A solution for use-case 2 is provided with https://github.com/ballerina-platform/ballerina-lang/pull/34927/commits/6b9bfd24323b9c1b7e2555644dcf6cbb85129056

Case 1 has been fixed. Case 3 needs to be fixed.

Case 1 and case 2 are now resolved with latest fixes. Case 3 needs to be fixed. Had a look in to the cause of case 3. Seems like the after applying the code action there are two issues.

There is a new diagnostic reporting that the return type of the created function does not match with the variable type

The return type should be ideally FileIoError instead of error<record {|string filename; anydata...;|}> which is the effective type of type FileIoError IoError & error<FileErrorDetail> intersection type.

When investigated, I figured out that the type of BVarSymbol corresponding to err above has the BErrorTypeSymbol type rather than the BTypeDefTypeSymbol

Case 1 and case 2 are now resolved with latest fixes. Case 3 needs to be fixed. Had a look in to the cause of case 3. Seems like the after applying the code action there are two issues.

There is a new diagnostic reporting that the return type of the created function does not match with the variable type

This is because a member of the intersection type in FileIoError is a distinct type. distinct keyword brings nominal typing rules to ballerina. Removing the distinct keyword or making the getErro() return type to actually return the same intersection with that nominal type would solve this diagnostic

Case 1 and case 2 are now resolved with latest fixes. Case 3 needs to be fixed. Had a look in to the cause of case 3. Seems like the after applying the code action there are two issues.

There is a new diagnostic reporting that the return type of the created function does not match with the variable type

This is because a member of the intersection type in FileIoError is a distinct type. distinct keyword brings nominal typing rules to ballerina. Removing the distinct keyword or making the getErro() return type to actually return the same intersection with that nominal type would solve this diagnostic

@pcnfernando thanks. That means only the following would work right?
function getError() returns FileIoError {
    return error("", filename = "");
}

2. The return type should be ideally `FileIoError` instead of `error<record {|string filename; anydata...;|}>` which is the effective type of `type FileIoError IoError & error<FileErrorDetail>` intersection type.

When investigated, I figured out that the type of BVarSymbol corresponding to err above has the BErrorSymbol rather than the BTypeDefSymbol. This is because in SymbolEnter we create a BErrorSymbol and populate instead of BTypeDefSymbol as the type of the BVarSymbol when there is an intersection of errors.
https://github.com/ballerina-platform/ballerina-lang/blob/15949ae2289f14be4fbced65c04ce644aa487b16/compiler/ballerina-lang/src/main/java/org/wso2/ballerinalang/compiler/semantics/analyzer/SymbolEnter.java#L1683

@rdulmina is working on a fix related to this. We have to revisit the issue after this fix. https://github.com/ballerina-platform/ballerina-lang/issues/24078


headless rendering

Hi,
I'm trying to do headless rendering of the LeapHandRot environment, i.e., I want to be able to use env.render(mode="rgb_array") but I seem to be running into a few issues here. First, the LeapHandRot environment seems to inherit from the Env class, which doesn't support headless rendering within its render function, so I took a look at the VecTaskEnv class, which does appear to. However, when I try implementing the same logic, i.e., moving the virtual display code from VecTaskEnv to LeapHandRot, I run into an issue where the viewer is needed, since otherwise the virtual display returns None as the image when virtual_display.grab() is called. I have a feeling that I'm potentially overcomplicating the issue, so if you have any suggestions, that would be greatly appreciated - thanks!

This is an isaac gym issue, I encourage you to reach out to Nvidia forums about this.  This is not specific to LEAP Hand.


Coveralls Code Coverage

Changes Unknown when pulling 8a5cc02e85f3d9130586055440d007c70ad8c9a4 on coveralls-code-coverage into ** on master**.

Builds passed. I killed a duplicate build hence the "some checks were not successful".

Adding Coveralls code coverage support to builds.


Navbar styling issue

The navbar looking not good at all

It can be easily attractive to display

GSSOC'24

please assign this issue to me i have much handson experience on this frontened designing work i can handle it

I can fix this issue, please assign it to me!

please assign this issue to me , i'm an front end developer #gssoc

Hello I am Gargi from IIT Kharagpur! I work with html,css,js,react.js and so on and am eager to learn more. I'd appreciate being assigned this task for GSSoC 24.

Hi,
Could you please assign this issue to me. I have knowledge in working with web development projects. I think I can handle it.
#gssoc24

@SujayDeyTMSL Welcome to TRAVEL _WEBSITE.
Thanks for raising the issue.
But I will like to recommend you to use the issue templates to make it clearer, more understanding.

Sorry to close this issue.
But please use the issue templates


oneM2M Cross Origin Requests

Hi Andreas,
I have a feeling that Bob Flynn has already mentioned this to you, but I am current developing a React JS single page application (SPA) that requests data from the ACME CSE.
The problem with this approach that I am taking is that the SPA has to be hosted on a different IP (and/or port number) than the ACME CSE. This means that when the JavaScript code in the SPA makes an XHR/AJAX/HTTP request to the ACME CSE, the user's web browser will block the response if the ACME CSE does not specifically allow the Cross Origin Request.
For now, I've created a very bad workout one my own fork and branch: https://github.com/ExpandingDev/ACME-oneM2M-CSE/tree/cors-shim
The changes I make in there do two things:

Adds flask-cors as a dependency
Uses the flask cors to enable CORS requests from ALL origins. This is a security vulnerability.

I am aware that the way I am getting around these CORS problems is a hack and should not be used in the mainstream ACME CSE/oneM2M.
To me, the ideal solution would be something like this:

Allow oneM2M to treat the Origin header as the same as the X-M2M-Origin header
Ignore the X-M2M-Origin header if the plain Origin header is present
Reply with Access-Control-Allow-Origin only if the Origin in the response matches with an originator specified in an ACP

I don't fully know if this would be a secure approach, I would be then concerned about attackers registering a domain name to match the originator present in an ACP in order to get their payload through.

Hi Tyler,
yes, Bob and I shortly discussed this problem earlier today. I am hesitating a bit to add CORS support into ACME's web server because I have the feeling that this is the wrong approach, but I am still trying to get my head around it.
To your points. I don't understand why you suggest that Origin and X-M2M-Origin are related. The first one deals with the http / CORS aspects of the http protocol, and X-M2M-Origin is a oneM2M identifier that identifies the originator of a oneM2M request.

Allow oneM2M to treat the Origin header as the same as the X-M2M-Origin header

The X-M2M-Origin is a URI but not necessary a URL. Its content must follow the format defined for AE-ID or CSE-ID, while Origin as I understand it must be the URL of the original website where the browser loaded the web page and the JS content, right? So my understanding is that both values are actually two different things.

Ignore the X-M2M-Origin header if the plain Origin header is present

Why? The browser application (ie. the AE) needs to identify itself. This is done through the X-M2M-Origin header. And a request must have the oneM2M originator, otherwise it is invalid and will be rejected.

Reply with Access-Control-Allow-Origin only if the Origin in the response matches with an originator specified in an ACP

That could be an interesting addition. There is actually a a field in ACP resources that, if present, enables the use of one particular ACP resource for an IP address. Unfortunately, I haven't implemented this yet.
Another quick solution could be to add a list of sites to  the configuration that are allowed to do CORS, and ignore ACPs for the moment.

Why I am still hesitating is that in normal deployments this is a functionality that is not realised by the actual backend server (or here: the CSE), but by a reverse proxy that is part of an API gateway and routes the request accordingly. So, both the requests to the web site's http server as well as the oneM2M Mca requests would go through this reverse proxy first.
Actually, I did this for the stand-alone version of ACME's web UI. When run as a stand-alone the web UI's server forward oneM2M requests to the actual CSE (which would run on a different address/port). I faced the same problems here as you do. See https://github.com/ankraft/ACME-oneM2M-CSE/blob/master/acme/webui/webUI.py#L92 for the simple implementation. You can also use nginx, of course.
This could be a quick fix that unfortunately adds another component to your infrastructure but wouldn't require any further change.
Another option would be to not send the oneM2M requests over http but via MQTT. This would require an MQTT broker of course, but would eliminate the CORS problem The oneM2M requests over look similar to the http requests, only the packaging is a bit different (actually, those would be "pure" oneM2M requests). You can try this yourself by installing a broker (like https://mosquitto.org), enabling and configuring MQTT for ACME to connect to the broker, and then configure the test suite to use MQTT. By reading the logs you should get a good idea about the request format. Unfortunately, this would add more efforts for your installation as well as to add MQTT to your JS application.

There are a lot of interesting thoughts that we need to thing about and discuss. Bob and I will bring them definitely to the discussions within oneM2M. Thanks a lot for the discussion!
There is one thing that is important to consider. The http binding is only one mapping to a concrete transport technology. Other are MQTT, CoAP or WebSockets. Each of these bindings is a mapping of the abstract oneM2M requests, and so is the oneM2M origin request attribute (that holds the ID of the sender, the "originator") mapped to the X-M2M-Origin http header field.
The problem is now that the format of this origin attribute must follow a certain format. For example, it must usually start with an upper-case "C", like "CAdmin". It might have in addition the CSE-ID to identify the CSE node where this originator is registered, like "/id-in/CAdmin". But that's about it. It cannot be an http URL, like "https://example.com".
Another issue is that this URL usually is a constant value (at least that is my understanding for the http(!) origin header field). This means that any instance of the web page would have the same originator, and so this value cannot be used to identify a particular instance of a web application. Each instance of an AE has a unique oneM2M originator.
Now, is it possible to enable the browser to assign a unique oneM2M-valid ID to the http origin header?
What I don't understand yet is why you want to replace the X-M2M-Origin header in the request? There are other X-M2M-* header that you must set for a valid request anyway.

But if you're already using a reverse proxy, why not just use your own backend database instead of using oneM2M?

That is a good question. In an enterprise deployment one would never expose backend services (or a CSE) directly to the public Internet, to a browser or to any other application directly. There is always an API gateway and/or a UI API layer in between (for authentication, load balancing, redirections etc).
Another important point is that a oneM2M CSE is not just a database. Well, it offers data handling functionality in the form of storing data in the resource tree, but there is so much more IoT-related functionality that a normal database doesn't implement.

I had a quick look at "flask-cors". You wrote that you integrated it in your ACME clone. It is really that simple to integrate as examples on the project's GH page suggest? (https://github.com/corydolphin/flask-cors) Are alle the headers added automatically for responses? In that case this could be a nice addition to ACME (configurable, of course, and switched off by default).

I am now adding CORS support directly in ACME. A first implementation with flask-cors was straightforward to do. Next will be a test by running a test in a browser as well as adding some configuration options here.

My tests with a browser (Safari & Firefox) went well. I tidied up the code, added configurations and documentations, and committed the changes to the development branch. There is no a [server.http.cors] section where  CORS can be enabled.
One note: I found that browsers refused to do the CORS procedures if the CSE isn't running with TLS (https) enabled. You still might need to accept a self-signed certificate, though, however the browsers don't ask when they trying to do the CORS test. I managed  this by accessing ACME's web UI from the browser and then accepting an exception for that certificate. Then the certificate is also accepted for CORS.


Multiple trigger on onDownloadProgress callback

onDownloadProgress callback is triggered 3 times (3 onDownloadProgress.progress logs) when the percentCompleted became 100.
It should be trigged only once.
ipcMain.on('download', async (event: Electron.IpcMainInvokeEvent, downloadUrl: string) => {
  await manager.download({
    window: mainWindow,
    url: downloadUrl,
    saveDialogOptions: {},
    callbacks: {
      onDownloadProgress: ({ ipercentCompleted }) => {
        console.log('onDownloadProgress.progress: ', percentCompleted);
        mainWindow.webContents.send('download-progress', percentCompleted)
      },
      onDownloadCompleted: ({ item }) => {
        mainWindow.webContents.send('download-completed', item.getFilename())
      }
    }
  })
})

Actual
...
onProgress.progress:  95.06
onProgress.progress:  98.07
onProgress.progress:  100
onProgress.progress:  100
onProgress.progress:  100

Expected
...
onProgress.progress:  95.06
onProgress.progress:  98.07
onProgress.progress:  100

This can happen in some situations, for example using data URLs, where the downloaded bytes > total bytes as reported directly from electron.
https://github.com/theogravity/electron-dl-manager/blob/main/src/DownloadInitiator.ts#L234
In this situation, we keep it at 100 vs reporting a > 100 %
https://github.com/theogravity/electron-dl-manager/blob/main/src/utils.ts#L107
We listen to the updated event from electron here:
https://github.com/theogravity/electron-dl-manager/blob/main/src/DownloadInitiator.ts#L168
It is probably bug on the electron side probably relating to how the current / total bytes are calculated (I'm guessing in a data-uri situation, it might use the base64 encoding for total bytes, but then when expanded to binary, it's a different value for current), and because of this mismatch, it will continue to fire progress updates.

It looks like also at the chromium level, this can bug out too in certain situations:
https://source.chromium.org/chromium/chromium/src/+/main:components/download/internal/common/download_item_impl.cc;l=1099;drc=071a85757d9d95316f83f50e1fe087a6d87908f9

I've submitted a PR to the electron project to add more native methods on the DownloadItem that should hopefully give better fidelity (I can reference the internal % progress instead of calculate it myself). It may or may not fix your issue.
Once that's in, I'll update the references in this lib to use it.
https://github.com/electron/electron/pull/42805

Good news, the PR has been merged in. I'll have to wait for the backports to also merge for older electron versions (30, 31, 32). Once they're in, I'll have a specific check to see if you're using electron >= 30 and apply the native calculation methods instead.

Published under 3.1.0. You'll need Electron version 30.3.0 or greater to get the native reporting stats.
Please re-open if it's still an issue after using the supported Electron version.


update jobflow and jobtemplate CRD

Updated the types files of jobflow and tobtemplate, adding doc files and register files. Regenerated client, lister, informer of jobflow and jobtemplate

/approve
/lgtm


Validate QC vote submissions in FlowQC Contract

Context
We need to validate QC votes submissions in the QC smart contracts to ensure that only valid votes are included in the EpochSetup service event. This is also needed to ensure the smart contract knows when it has met the threshold for constructing a QC.
Open Questions

How do we provide the smart contract with the root QC message digest? (needed to validate votes)

We could provide it along with the vote message - but these are untrusted inputs.
Ideally the smart contract can compute this value on its own (it has all the data, it is missing the mechanism building/encoding/hashing the root block and root QC objects)

Should we change input types (is there a Signature type with the new crypto API)?
Can we aggregate the signatures within the smart contract?

The QC in this ticket is only the one used by the collection nodes Hotstuff right?

is there a Signature type with the new crypto API

No, a signature is just a [UInt8]

Can we aggregate the signatures within the smart contract?

No. Are we able to survive without this (for now)?

The QC in this ticket is only the one used by the collection nodes Hotstuff right?

👍  Yes

No. Are we able to survive without this (for now)?

Yes we are fine without aggregation in the contract, we only need vote verification.

Closing in favour of https://github.com/dapperlabs/flow-go/issues/5596 so we can track this under the Epoch Ph1 milestone.


Update Pagination.tsx

hi @MyrtilleKim, thank you for having the intention to contribute! If you still want to contribute, I suggest updating these into React.ReactNode:
https://github.com/imballinst/react-bs-datatable/blob/c6fa08e4f247cdd9f36fb1bbca4d7dcd8ee30be3/src/components/Pagination.tsx#L13-L22
That way, we can pass your icons of choice if you want more flexibility in the pagination numberings. Thanks!


RFC: Move data for tests into a new repository

Recently the first regression tests were added to the Tesseract sources, and hopefully more will be added in the future. I expect that many of those tests will require images, maybe also related text and other files. Typically the images are large (much larger than source code files),  and most image formats are already compressed, so Git cannot compress them further. This is bad for the git repository size, but also for the size of the working directory or of release tarballs.
Currently 8 of the 10 largest files are test related:
1444515 ccutil/universalambigs.cpp
 708120 testing/DuTillet1004Pg2LG.jpg
 444141 testing/hebtypo.jpg
 401757 testdata/chi_tra.unicharset
 399613 testing/hebrew-nikud-genesis-1-2.png
 378914 testdata/kan.unicharset
 359497 ccutil/tesscallback.h
 327214 testdata/chi_sim.unicharset
 239201 testdata/jpn.unicharset
 221628 testdata/mar.unicharset

We should avoid adding more such files. I suggest to create a new repository called tesstest,  testdata, tesseract_testdata or any other name for all test related data and move the existing test data to that repository. The tesseract repository can then include that new repository as a git submodule which is only needed when running tests (so most people won't need it).
OpenJPEG is an example of another Open Source project with a similar split of source code and test data. They do not use a git submodule, but have a configure option to choose the location of the test data.

Good idea.
Google's tests probably use testdata as the directory, so the submodule can be pulled in under that name.

I can very easily image 175MB of test data.

... or more. Test images use a lot of disk space.

https://github.com/tesseract-ocr/test


Arbitrary Values?

Looks like I was looking for Tailwind container queries just as this project was posted.
All seems to working well!
I'm wondering if it might be possible to implement arbitrary values. To be honest, I'm not even sure if Tailwind supports this for plugins or not.
Much thanks

Hi @christianmagill. I'm so glad it helped. You are talking about something like .cq-w-[300px]:text-xl right? It's exactly what I want to add. I have no idea how can I do that (I don't know if it's possible for variants) but I'll investigate it ASAP.

Yep, that's exactly what I'm talking about!

@dgknca Ran across this Tweet and example that explains how to use arbitrary values.
https://twitter.com/davidwebca/status/1468802754286080004/photo/1
https://play.tailwindcss.com/t4sKoCrxNo

@dgknca Ran across this Tweet and example that explains how to use arbitrary values.
https://twitter.com/davidwebca/status/1468802754286080004/photo/1
https://play.tailwindcss.com/SLtXEpJPbJ

It is a very different idea. I have to replace the variant with utilities. I'm not sure about using it because it has an unusual syntax. But looks like the only way for now. Thank you so much!


Spike: Split up CSS and JS assets

Context
Investigating the consequence of serving the same CSS and JavaScript to all users. May want to choose a different way of splitting up individual files, but this is just a draft to get the raw numbers, and to create a review app can use to test with performance analysis tools.
CSS

File
Bytes (Gzipped)
Difference

application.css (current)
16,562

application-candidate.css
12,953
-21.8%

application-provider.css
15,011
-9.4%

application-support.css
12,797
-22.7%

application-api-docs.css
12,616
-23.8%

JavaScript

File
Bytes (Gzipped)
Difference

application.js (current)
59,234

application-candidate.js
22,385
-62.2%

application-provider.js
46,951
-20.7%

application-support.js
9,968
-83.2%

application-api-docs.js
9,971
-83.2%

Comparison
Total size of combined assets (CSS + JavaScript)

Current single file: 75.8kB
Candidate specific file: 35.3kB (53.4% smaller)
Provider specific file: 62.0kB (18.2% smaller)

Changes proposed in this pull request

Guidance to review

Link to Trello card

Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

No idea why the review app is failing. 🤔
Tested a few dozen pages locally vs QA, all looks good visually apart from the performance dashboard.
Support: tasks page ✅

Support: performance page 🔴

Support: providers page ✅

Support: candidate page ✅

Candidate: interstitial page ✅

Candidate: application page ✅

Candidate: personal details page ✅

Candidate: review page ✅

Provider: applications page ✅

Provider: application page ✅

Provider: timeline page ✅

Provider: account page ✅

Vendor: API usage page ✅

JS also seems to work!

@tvararu Fixed performance dashboard styles.

Provider interface looks 👌.
Wasn't sure in the thread but now I see this I really like it. Everything's so explicit. Thank you @paulrobertlloyd! ⭐


Telemetry: Let links open in new tab

Bug Description
The trackClick() util uses event.preventDefault() and document.location = url to track links to ensure the beacon is transmitted before opening the URL.
However, this is quite annoying and unnecessary if the link is being opened in a new tab anyway.
Expected Behaviour
My suggestion would be to do something like this (untested!):
async function trackClick(event, eventName, eventCategory, url) {
  if (!isTrackingEnabled()) {
    return Promise.resolve();
  }

  const eventData = {
    send_to: config.trackingId,
    event_category: eventCategory,
    event_label: url,
  };

  const openLinkInNewTab =
    event.target === '_blank' ||
    event.ctrlKey ||
    event.shiftKey ||
    event.metaKey ||
    event.which === 2;

  if (openLinkInNewTab) {
    return track(eventName, eventData);
  }

  event.preventDefault();

  return track(eventName, eventData).finally(() => {
    document.location = url;
  });
}

Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
WordPress Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

@zidianlyu In case you're interested in more telemetry stuff :)

We could also do something like this.
async function trackClick(event, eventName, eventCategory, url, openLinkInNewTab = false ) {
  if (!isTrackingEnabled()) {
    return Promise.resolve();
  }

  const eventData = {
    send_to: config.trackingId,
    event_category: eventCategory,
    event_label: url,
  };

  if (openLinkInNewTab) {
    return track(eventName, eventData);
  }

  event.preventDefault();

  return track(eventName, eventData).finally(() => {
    document.location = url;
  });
}

Adding another param to function means we have more control.

We could also do something like this.
async function trackClick(event, eventName, eventCategory, url, openLinkInNewTab = false ) {
  if (!isTrackingEnabled()) {
    return Promise.resolve();
  }

  const eventData = {
    send_to: config.trackingId,
    event_category: eventCategory,
    event_label: url,
  };

  if (openLinkInNewTab) {
    return track(eventName, eventData);
  }

  event.preventDefault();

  return track(eventName, eventData).finally(() => {
    document.location = url;
  });
}

Adding another param to function means we have more control.

That seems kinda confusing and gives wrong expectations about what it does. Plus it‘s redundant as we can already infer the necessary information from the event.

That seems kinda confusing and gives wrong expectations about what it does. Plus it‘s redundant as we can already infer the necessary information from the event.

Verified in QA

Verified in QA

@o-fernandez The issue with links not opening in a new tab  in the monetization section is now resolved, if you wanna give it a spin.

@o-fernandez The issue with links not opening in a new tab  in the monetization section is now resolved, if you wanna give it a spin.


/use Toggle boxes not saving

Hello Timothy,
I haven't had the chance to test this but it seems to be popping up on different topics at the forums people having this issue.
Supposedly the /use ticks on the right are not behaving like they should to stay on or off when saving the macro in the editor.
These are the topics I found and there was another but can't find at the moment:
https://wowlazymacros.com/t/updated-03-21-19-dv8-hacov-dh-mythic-raiding-m/5185/94
https://wowlazymacros.com/t/the-use-toggle-boxes-not-saving-when-changed/7486

Here is another person with the same issue I believe:
https://wowlazymacros.com/t/gnome-sequencer-for-idiots-ask-and-you-will-be-answered/1359/435

The problem is I can’t replicate their outcome.  I can’t cause the behaviour they are seeing to occur.
It’s either something broken on their instance of that macro, or not understanding the difference between the gold and silver ticks (ie setting them to silver and not realising that their defaults are to use trinkets) or simply not waiting for the save function to have completed before reopening the editor.
For this last scenario GSE puts the save function into a queue that checks if you are in combat.  It can’t update the macro if you are.  If you make changes, get put in combat, hit save, then reopen the editor and edit the macro you will have the old macro and abort the save action.
TimothyLuke

On 30 Jul 2019, at 4:01 am, Luis Cestou<EMAIL_ADDRESS>wrote:
Here is another person with the same issue I believe:
https://wowlazymacros.com/t/gnome-sequencer-for-idiots-ask-and-you-will-be-answered/1359/435
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I think a little more specificity might help...
We cannot save trinket use status WITHIN a particular macro.  Mine are set as gold.  I change them to black.  They visually show black.  I make sure I am not in combat.  I save the macro I tried to edit to black for trinket use.  I exit and reload GSE.  I click on the same macro I tried to edit to black for trinket use and the trinket use options within that macro are back to gold.  That is the problem we are having.
Moreover, I CAN alter the general GSE options to set general trinket use parameters and those do save.  But even if I set the General options for trinket use to black, the gold options which appear in the individual and seemingly unalterable specific macros overrule same.

Random Question:
When you exit and load, I am assuming you are talking about the editor, are
you waiting for the macro browser window to re-open or re-opening GSE via
/gs?
On Tue, 30 Jul 2019 at 08:21, fwyflyer<EMAIL_ADDRESS>wrote:

I think a little more specificity might help...
We cannot save trinket use status WITHIN a particular macro. Mine are set
as gold. I change them to black. They visually show black. I make sure I am
not in combat. I save the macro I tried to edit to black for trinket use. I
exit and reload GSE. I click on the same macro I tried to edit to black for
trinket use and the trinket use options within that macro are back to gold.
That is the problem we are having.
Moreover, I CAN alter the general GSE options to set general trinket use
parameters and those do save. But even if I set the General options for
trinket use to black, the gold options which appear in the individual and
seemingly unalterable specific macros overrule same.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/557?email_source=notifications&email_token=AAEMCFLWK6G33KYDZVBNJFTQB5X7NA5CNFSM4IHTKVLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3CHMBA#issuecomment-516191748,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEMCFP6X5DXGZ75MNXXVUDQB5X7NANCNFSM4IHTKVLA
.

--
Timothy Minahan**M 0406 413 680  |  *E<EMAIL_ADDRESS>

After changing the use trinkets to black in the specific tab of the macro I am trying to edit and after I hit save, GSE says the sequence {macro name}  is saved.  At that point I hit close.  Nothing happens.  I hit close again and the GSE window disappears.  Then I /reload and /gs, click the macro I want to edit and see that the on use trinket(s) in that tab are once again gold.

There's the problem.  You are aborting the save and update process.
When you hit save - this puts the changes into the queue.  When you hit
close, it tells the queue to start to process.  When the queue has
finished, and your changes have been finalised and acted upon, the GS
browser window will reshow.  When you hit /reload and /gs you have acted
between when the changes were put in the queue and finalised.  You need to
wait for the queue to complete and the window to show for it to be complete.
The LDB plugin will show you the current state and progress of the queue.
For specifics see https://wowlazymacros.com/t/gse2-and-7-3/3783/2 or #321
On Tue, 30 Jul 2019 at 11:17, fwyflyer<EMAIL_ADDRESS>wrote:

After changing the use trinkets to black in the specific tab of the macro
I am trying to edit and after I hit save, GSE says the sequence {macro
name} is saved. At that point I hit close. Nothing happens. I hit close
again and the GSE window disappears. Then I /reload and /gs, click the
macro I want to edit and see that the on use trinket(s) in that tab are
once again gold.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/557?email_source=notifications&email_token=AAEMCFKRL4DYBUXQRCIFUXLQB6MU3A5CNFSM4IHTKVLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3CPZ2A#issuecomment-516226280,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEMCFKG2M7PP44C6YEKLDTQB6MU3ANCNFSM4IHTKVLA
.

WoW doesnt save your changes to disk until you close and Exit WoW.
On Tue, 30 Jul 2019 at 11:34, Timothy Minahan<EMAIL_ADDRESS>wrote:

There's the problem.  You are aborting the save and update process.
When you hit save - this puts the changes into the queue.  When you hit
close, it tells the queue to start to process.  When the queue has
finished, and your changes have been finalised and acted upon, the GS
browser window will reshow.  When you hit /reload and /gs you have acted
between when the changes were put in the queue and finalised.  You need to
wait for the queue to complete and the window to show for it to be complete.
The LDB plugin will show you the current state and progress of the queue.
For specifics see https://wowlazymacros.com/t/gse2-and-7-3/3783/2 or #321
On Tue, 30 Jul 2019 at 11:17, fwyflyer<EMAIL_ADDRESS>wrote:

After changing the use trinkets to black in the specific tab of the macro
I am trying to edit and after I hit save, GSE says the sequence {macro
name} is saved. At that point I hit close. Nothing happens. I hit close
again and the GSE window disappears. Then I /reload and /gs, click the
macro I want to edit and see that the on use trinket(s) in that tab are
once again gold.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/557?email_source=notifications&email_token=AAEMCFKRL4DYBUXQRCIFUXLQB6MU3A5CNFSM4IHTKVLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3CPZ2A#issuecomment-516226280,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEMCFKG2M7PP44C6YEKLDTQB6MU3ANCNFSM4IHTKVLA
.

--
Timothy Minahan**M 0406 413 680  |  *E<EMAIL_ADDRESS>
--
Timothy Minahan**M 0406 413 680  |  *E<EMAIL_ADDRESS>

So, I tried again.  I made the changes in the tab.  I hit save.  It said the macro sequence was saved.  I hit close.  I waited 60 seconds and logged off wow completely.  I went back in and still no change to the trinket tab within the macro.  Is there some other step I am missing to save changes?  Moreover, other changes to the macro itself do save.  But changes to the trinket use in the macro do not.

You're not getting it:  After Hitting save and closing the window you NEED
TO WAIT FOR THE GSE WINDOW TO REAPPEAR ON ITS OWN.  If not then you
aborted the process and stopped it finishing.  If you have a convoluted
bunch of macros in your queue it could take time.  If you have inadvertedly
stopped the queue it wont complete - Hence the LDB Monitor i referenced
earlier.
On Tue, 30 Jul 2019 at 12:30, fwyflyer<EMAIL_ADDRESS>wrote:

So, I tried again. I made the changes in the tab. I hit save. It said the
macro sequence was saved. I hit close. I waited 60 seconds and logged off
wow completely. I went back in and still no change to the trinket tab
within the macro. Is there some other step I am missing to save changes?
Moreover, other changes to the macro itself do save. But changes to the
trinket use in the macro do not.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/557?email_source=notifications&email_token=AAEMCFPQSNU4KEGYEAOMDFTQB6VEVA5CNFSM4IHTKVLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3CTHNY#issuecomment-516240311,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEMCFOAEQMSM7BOLPF2VALQB6VEVANCNFSM4IHTKVLA
.

--
Timothy Minahan**M 0406 413 680  |  *E<EMAIL_ADDRESS>

Oh, it did reappear on it's own before I logged off and exited.

It is not a saving or queing issue as I can edit the macros themselves and save the changes all day long.  I just cannot get the trinket on use gold squares to remain black when the changes are saved.  In other words, the problem is specific to the trinkets and not to the saving.

I just don’t get that behaviour.  It works for me every time.
TimothyLuke

On 30 Jul 2019, at 2:24 pm, fwyflyer<EMAIL_ADDRESS>wrote:
It is not a saving or queing issue as I can edit the macros themselves and save the changes all day long. I just cannot get the trinket on use gold squares to remain black when the changes are saved. In other words, the problem is specific to the trinkets and not to the saving.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Totally understand.  Either way, thank you for the incredible work you have done on this project.

There always has to be a reason.
Could you please close wow completely, zip up your GSE.lua file (found in c:\path\to\Wow_retail\WTF\Account\YourA courNameHere\SavedVariables\GSE.lua)
The only other thing I can think of is a corruption.
TimothyLuke

On 30 Jul 2019, at 3:04 pm, fwyflyer<EMAIL_ADDRESS>wrote:
Totally understand. Either way, thank you for the incredible work you have done on this project.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

GSE.zip

Cheers - May be a couple of days before I can pull this apart - I’m conducting a funeral tomorrow.  Just the reality of being a pastor.
TimothyLuke

On 30 Jul 2019, at 3:43 pm, fwyflyer<EMAIL_ADDRESS>wrote:
GSE.zip
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

GSE.zip
there you go.
I have also supplied a yt video of how to reproduce this:
https://www.youtube.com/watch?v=PfnZqAgc-xw


[BUG]: PaymentElement - PaymentMethod 'customer_balance' not working when fields.billingDetails.email = never

What happened?
If fields.billingDetails.email is set to 'never' and the BankTransfer-Button is clicked the whole form disappears.
Console output:
Uncaught Error: Minified React error #130; visit https://reactjs.org/docs/error-decoder.html?invariant=130&args[]=undefined&args[]= for the full message or use the non-minified dev environment for full errors and additional helpful warnings.     at Qa (elements-inner-payment-9e15de27c212158e006b4652730d5fee.js:1:733966)     at f (elements-inner-payment-9e15de27c212158e006b4652730d5fee.js:1:688037)     at v (elements-inner-payment-9e15de27c212158e006b4652730d5fee.js:1:689314)     at elements-inner-payment-9e15de27c212158e006b4652730d5fee.js:1:691333     at Fn (elements-inner-payment-9e15de27c212158e006b4652730d5fee.js:1:697740)     at _c (elements-inner-payment-9e15de27c212158e006b4652730d5fee.js:1:766112)     at Ma (elements-inner-payment-9e15de27c212158e006b4652730d5fee.js:1:719000)     at Ia (elements-inner-payment-9e15de27c212158e006b4652730d5fee.js:1:718925)     at _a (elements-inner-payment-9e15de27c212158e006b4652730d5fee.js:1:716273)     at elements-inner-payment-9e15de27c212158e006b4652730d5fee.js:1:680035
Environment
No response
Reproduction
No response

Hi @knathan1987, thanks for raising this issue! We're currently working on a fix, and I'll comment again when it's released.

This should be fixed now. Please let us know if you continue to have this issue!


New validator: DiscardValidator

Implement a DiscardValidator that accepts any kind of input, but discards it and returns a (predefined) value instead.
There are no validation errors for this validator, the input is just completely ignored. (For a validator that rejects any kind of input, there already is the RejectValidator.)
The implementation could look somewhat like this (copied from a project):
class DiscardValidator(Validator):
    """
    Meta validator that simply discards any input and always returns the same predefined value.

    Set the `output_value` parameter to any value to specify the value returned by the validator. If not set, the value
    defaults to `None`.
    """

    # Value that is returned by the validator
    output_value: Any

    def __init__(self, *, output_value: Any = None):
        self.return_value = output_value

    def validate(self, input_data: Any, **kwargs) -> Any:
        # Ignore input completely and just return the predefined output value
        return self.output_value

Implemented by #94.


WASM runtime in the .NET 8 enables EH and SIMD instructions by default.

WASM runtime in the .NET 8 enables EH and SIMD instructions by default.

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

ID: fa3e99ba-cbe1-e8a1-6628-9b6eb39710a4
Version Independent ID: dd777362-b92d-cac6-2343-a7aaf31d0bdf
Content: Tooling for ASP.NET Core Blazor
Content Source: aspnetcore/blazor/tooling.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

@pavelsavara ... I think this is a duplicate of https://github.com/dotnet/AspNetCore.Docs/issues/28996, which was resolved by https://github.com/dotnet/AspNetCore.Docs/pull/28997.
The live content is at ...
https://learn.microsoft.com/en-us/aspnet/core/blazor/tooling?view=aspnetcore-8.0&pivots=windows#net-webassembly-build-tools
btw ... to track .NET 8 Blazor doc work, I have an issue every year that shows where I'm at with everything ...
https://github.com/dotnet/AspNetCore.Docs/issues/28161
I'll close as completed, but let me know if more work needs to be done.

For ...

.NET 8 enables EH and SIMD instructions by default.

... we might need to update it after all. It currently tells devs that ...

To enable WebAssembly exception handling, use the  property in the app's project file (.csproj) with a value of true:

... and shows the MS Build config.
Should that be removed for >=8.0?

Yes, I think that's right. Also, since it's enabled by the default the guidance should be showing how to disable it for >=8.0, so I'll change that section and ping u on the PR. Currently, it states in the text that it's enabled by default but still shows the setting to true.
I'll fix it now and ping u on the PR.

You may need to switch it off, for example when you are targeting old/mobile devices.
Here are details on how https://github.com/dotnet/runtime/blob/main/src/mono/wasm/features.md#simd---single-instruction-multiple-data

And note it's broken right now https://github.com/dotnet/runtime/issues/89302


Disarm and Arm Without Entering Code

Morning,
Is there a way to arm and disarm the panel without having to enter in the arm/disarm code?  I currently use Homebridge and their Alarm.com plugin because it allows me to arm and disarm my alarm without the need to enter the arm/disarm code each time via HomeKit.  That is the only reason i have Homebridge.
If I could figure out how to get Homeassistant and your Alarm.com plugin to allow arming and disarming without the code, I coudl blow away HomeBridge.  I realize the safety issues I am trying to get around.
Thanks!

Try this.

Set a PIN for your Alarm.com alarm control panel in Home Assistant. (I realize that this is the opposite of what you want to do, but bear with me.)
Manually configure the HomeKit integration via configuration.yaml using these instructions.
Configure the alarm_control_panel in HomeKit using the same PIN that you set in the Alarm.com integration:

    entity_config:
      alarm_control_panel.your_control_panel_name:
        code: 1234

I think that this will cause the HomeKit integration to automatically pass the PIN to the Alarm.com integration without prompting you.
Let me know if this works!

@rbiffle Any luck?


feature: Added OpenSea filterer registry to Raffle contracts

Description

This PR implements the OpenSea filtered registry. Our team created UpdatableOperatorFiltererUpgradeable.sol based on the official UpdatableOperatorFilterer implementation provided by OpenSea.
Checklist:

[x] Pull Request references Jira issue
[x] Pull Request applies to a single purpose
[x] I've added comments to my code where needed
[x] I've updated any relevant docs
[x] I've added tests to show that my changes achieve the desired results
[x] I've reviewed my code
[x] I've followed established naming conventions and formatting
[x] I've generated a coverage report and included a screenshot
[x] All tests are passing locally

The order of the functions according to the these standards could be followed : https://docs.soliditylang.org/en/v0.8.17/style-guide.html#order-of-functions

It would give better clarity to add Naspec comments for the function : https://docs.soliditylang.org/en/v0.8.17/natspec-format.html?highlight=comments#natspec-format


What is the preferred testing approach when using APIKit?

I see the Request implementations are easily unit-testable which is nice. But what is the preferred approach to testing for clients of the Session API? It's declared as a class rather than a protocol so it's not simply a matter of substituting a fake a test time. I don't see a "MockSessionAdapter" in the test tree, but is that the appropriate hook point?

Good question.
I guess subclassing the Session class and overriding the functions isn't really ideal (or creating a completely new "MockSession" class). I would welcome a better approach to test our code with APIKit.

Hi @ratkins,
Yes, SessionAdapter is an appropriate hook point for mocking. I always create mock adapter like this. It would be great if APIKit has a flexible implementation of MockSessionAdapter for general purpose.
Of course, subclassing Session also works.

Closing this issue. Feel free to reopen this issue for the further discussion!


Apollo:  Automatic scheduled security errata runs

Security errata at errata.build.resf.org should be updated on an automatic schedule, without any human intervention.
I think 3 times a day (every 8 hours) is quite reasonable.  Strikes a balance between having timely security info, while not being in constant-update mode.
I have no idea how to implement this, but I assume it's a cron job somewhere?  ;-)

Resolved by #2


Manual setting of data_max_lag_allowed getting overwritten

Hi,
If I set my trackme_default_host_lag it appears to be also overwriting my manual settings which I put in for data_max_lag_allowed
Thanks

@11gods
Can you please share the definition you created?
The default is:

As you can see, there is an if condition:
eval data_max_lag_allowed=if(isnull(data_max_lag_allowed), "3600", data_max_lag_allowed)
If you change this to use a different value, you need to keep the if logic, otherwise you will overwrite any custom setting you have set, or policies are defining.
The macro purpose is to define a default value when the host is discovered and added to the collection. (or if for some reasons there wouldn't be a value which is not supposed to happen)
If you customise this, with the if logic, this will be relevant for newly discovered hosts, or if you reset the collection.
Let me know if my explanation make sense.
My guess is that you are setting an explicit value with no conditions.

I assume this answered your question @11gods let me know otherwise ;-)


Added a tailwind class to fix responsive issue on stoic template page

Description
Added a tailwind class to fix responsive issues on stoic template page
Related Issue
Fixes #241
Proposed Changes

Added Tailwind breakpoint class lg: to fix the overlapping issue

Screenshots

Checklist
Please check the boxes that apply:

[✅ ] I have rebased my branch on top of the latest main branch.
[✅ ] I have tested the changes locally
[✅ ] I ran npm run build and build is successful
[ ] I have added necessary documentation (if applicable)
[ ] I have updated the credits.md file (if applicable)

Thank you for the fix and your contributions to SyntaxUI. If you have already joined the SyntaxUI Discord, please message me there so I can provide you with a contribution badge.

Thank you for your valuable contribution to SyntaxUI! 🌟
Are you part of our SyntaxUI Discord community yet?
• If yes, please reach out to me there – a special contributor badge awaits you!
• If not, we'd love to have you join us. It's the perfect place to connect, share ideas, and stay updated on all things SyntaxUI.
Once again, thank you for helping shape the future of SyntaxUI.


williammorales1993/sci-25-delete-localstore-autosave-feature

Current dependencies on/for this PR:

main

PR #38   👈

This stack of pull requests is managed by Graphite.


What can i do with it!?!

I cant do anything with it, i can only make it send message and? Nothing? Please help me
Please make that the bot can send a command only if it has operator permission please

Please post on the discord (link in readme badge) for questions about usage, this issue board is for issues with the repository. The usage for the library is explained in the readme, the API doc, examples and executing other high level actions is explained in https://github.com/PrismarineJS/bedrock-protocol/wiki/Using-bedrock-protocol#so-how-do-you-do-anything-


Allow readonly data registration in meta

This issue is a result of a short discussion on Discord.
Currently it's not possible to register a readonly data node in meta when using a pointer to member or any other value. It's kind of possible when using the setter and getter overload of data but requires passing a null operation as the setter.
In my case, where I use a templated null operation function with variable parameters, the code looks like this.
.data<&NullOp<ResourceHandle&,
        std::invoke_result_t<decltype(&ResourceHandle::GetIndex), ResourceHandle&>>,
    &ResourceHandle::GetIndex>("index"_hs);

I think that library support for readonly data would be beneficial. It's also possible in other reflection libraries like rttr or refl-cpp.
Below are some of my ideas of how the interface could look like, but ultimately any other way is fine as long as it's brief and explicit enough.
// Setter and getter
.data<nullptr, &ResourceHandle::GetIndex>("index"_hs);
.data<&ResourceHandle::GetIndex>("index"_hs);
.data_readonly<&ResourceHandle::GetIndex>("index"_hs);

// Value
.data<&ResourceHandle::index>("index"_hs, entt::readonly_t);
.data_readonly<&ResourceHandle::index>("index"_hs);

I'm experimenting with this one:
.data<nullptr, &ResourceHandle::GetIndex>("index"_hs);

So far, all tests pass. It can also be used to attach non-const data members as read-only. For example:
struct clazz {
    int value{4};
};

entt::meta<clazz>().data<nullptr, &clazz::value>("value"_hs);

In this case, setting the value always results in a failure. That is:
clazz instance{};
assert(!entt::resolve<clazz>().data("value"_hs).set(instance, 42));

I like the nullptr approach more because it doesn't require to add an ugly data_readonly function for that. On the other hand, the getter-only overload isn't viable because it would conflict with another definition of data.

I agree about the data_readonly function but then again the data<nullptr, &clazz::value> syntax looks kind of out of place when paired with a field pointer instead of a function. It's natural when using nullptr to assume that a valid value can be used too. What'd be the valid value in this case?

What about something like this:
namespace entt {
    struct no_setter_t{};

    static auto constexpr no_setter = static_cast< no_setter_t* > (nullptr);
}

entt::meta< Type >()
    .data< entt::no_setter, &Type::Getter >();

Currently and for large part, you can see .data<&clazz::member> as an alias for .data<&clazz::member, &clazz::member>.
This means that it's even possible to use a member as a getter and another member as a setter for the sam meta variable. Functions are just another player in this game.
Therefore, .data<nullptr, &clazz::value> seems a logical consequence to me. The first should just do that if the member is const, in the second case you can force a meta-constness by not setting the setter.
Of course, I'm working to clean-up the multiple overloads in meta and to make this full-featured since is has some flaws at the moment imho. :+1:

See branch experimental. Null setters are supported and I made explicit the fact that data<&clazz::member> is an alias for data<&clazz::member, &clazz::member>.
The sole difference is that in the first case constness is intercepted and treated correctly while with the second case you can force a const meta data member associated to a non-const data member from the C++ class.

And I've also indecently reduced the amount of code in that part! :smile:

Oh, I didn't know it's an alias. Now it makes sense. Thanks. :)


fix consignment yaml conversion

This PR fixes the error we receive when trying to convert a consignment in yaml representation (serializing nested enums in YAML is not supported yet) by manually implementing the serialization and deserialization of XChain.

Do you think this is the right way for enum serialization?

I don't think there is a "right way" to serialize nested enums, there may be better solutions but this is the only solution I've found so far. Probably in the future the team working on serde_yaml will come with a solution to this issue, but for now I think we can be happy to have found a way to fix the conversion

@zoedberg the problem that this specific fix makes all of the serialization to be YAML-based, breaking all serde compatibility. If someone will try to do JSON, TOML or bincode deserialization, he will fail forever with some wierd explanations coming from serde_yaml.

@zoedberg the problem that this specific fix makes all of the serialization to be YAML-based, breaking all serde compatibility. If someone will try to do JSON, TOML or bincode deserialization, he will fail forever with some wierd explanations coming from serde_yaml.

All serializations are already broken and to support JSON and TOML we should do a bunch more work, since the problem with those serializations is there is no support for non-string keys in maps.
I recall we already discussed in the past the possibility to just drop support for JSON and TOML, so I would just follow that path (I can make a PR on rgb-wallet after this to remove support for broken serialization formats).
I think this fix is very important because it allow us adding consignment validation tests (as already discussed in person).
Anyway, if someone in the future will try to fix the conversion for other serde formats I guess he/she will notice the manual implementation of the XChain and find a way to fix it (assuming that in the meantime serde_yaml team hasn't already delivered a fix).

What I think is a correct approach: #216
May though require some changes downstream in the code which was using XOutpoint.

I think this fix is very important

Yes, but this PR introduces more bugs, so I can't see it as a fix. There are several better approaches. Approach nACK

All serde binary serializations were working before this PR - and this PR breaks them.

Not sure which serializations you are referring to. rgb-wallet supports yaml, toml, json, debug, contractum and only debug works at the moment

I am referring to all clients that use or will use RGB Core in a future.
Serde implementation means the type must be serializable in binary form - with CBOR, BSON, binvode etc.


Scanner randomly crashes under Android -- "index out of range" -- NavigationStack becomes corrupt

I am using the bar code scanner in both IOS (which works) and Android (which is broken).  I don't have any proprietary code.  I just call the Init.  The test is to open and close the scanner using PushModalAsync and then to close it using PopModalAsync.  After 3-4 tries, the app crashes with the stack trace at the bottom of this issue.
I can stop the error by checking the NavigationStack to see if it has any members. hen it is about to crash, the stack has zero members.  But if I do that, the user will never be able to close the modal scanner.
I cannot test your code or the test app there because I cannot get your code from the GitHub site to compile. See my other issue for details about that.
STACK TRACE
12-15 11:41:43.389 I/MonoDroid(30476): UNHANDLED EXCEPTION:
12-15 11:41:43.399 I/MonoDroid(30476): System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
12-15 11:41:43.399 I/MonoDroid(30476): Parameter name: index
12-15 11:41:43.399 I/MonoDroid(30476):   at System.ThrowHelper.ThrowArgumentOutOfRangeException (System.ExceptionArgument argument, System.ExceptionResource resource) [0x00000] in /Users/builder/data/lanes/3511/77cb8568/source/mono/mcs/class/referencesource/mscorlib/system/throwhelper.cs:93
12-15 11:41:43.399 I/MonoDroid(30476):   at System.ThrowHelper.ThrowArgumentOutOfRangeException () [0x00000] in /Users/builder/data/lanes/3511/77cb8568/source/mono/mcs/class/referencesource/mscorlib/system/throwhelper.cs:56
12-15 11:41:43.399 I/MonoDroid(30476):   at System.Collections.Generic.List1[T].get_Item (System.Int32 index) [0x0000c] in /Users/builder/data/lanes/3511/77cb8568/source/mono/mcs/class/referencesource/mscorlib/system/collections/generic/list.cs:181  12-15 11:41:43.399 I/MonoDroid(30476):   at Xamarin.Forms.Application+NavigationImpl+<OnPopModal>d__2.MoveNext () [0x0000d] in C:\BuildAgent\work\aad494dc9bc9783\Xamarin.Forms.Core\Application.cs:302  12-15 11:41:43.399 I/MonoDroid(30476): --- End of stack trace from previous location where exception was thrown --- 12-15 11:41:43.399 I/MonoDroid(30476):   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in /Users/builder/data/lanes/3511/77cb8568/source/mono/mcs/class/referencesource/mscorlib/system/runtime/exceptionservices/exceptionservicescommon.cs:143  12-15 11:41:43.399 I/MonoDroid(30476):   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x00047] in /Users/builder/data/lanes/3511/77cb8568/source/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:187  12-15 11:41:43.409 I/MonoDroid(30476):   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in /Users/builder/data/lanes/3511/77cb8568/source/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:156  12-15 11:41:43.409 I/MonoDroid(30476):   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in /Users/builder/data/lanes/3511/77cb8568/source/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:128  12-15 11:41:43.409 I/MonoDroid(30476):   at System.Runtime.CompilerServices.TaskAwaiter1[TResult].GetResult () [0x00000] in /Users/builder/data/lanes/3511/77cb8568/source/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:357
12-15 11:41:43.409 I/MonoDroid(30476):   at TPI_Mobile_Common.Controls.ScannerPage+<<-ctor>b__1_0>d.MoveNext () [0x00025] in C:\Source\TPIMobile\Xamarin\TPI_Telogis_V1\Controls\ScannerPage.xaml.cs:36
12-15 11:41:43.409 I/MonoDroid(30476): --- End of stack trace from previous location where exception was thrown ---
12-15 11:41:43.409 I/MonoDroid(30476):   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in /Users/builder/data/lanes/3511/77cb8568/source/mono/mcs/class/referencesource/mscorlib/system/runtime/exceptionservices/exceptionservicescommon.cs:143
12-15 11:41:43.409 I/MonoDroid(30476):   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.m__0 (System.Object state) [0x00000] in /Users/builder/data/lanes/3511/77cb8568/source/mono/mcs/class/referencesource/mscorlib/system/runtime/compilerservices/AsyncMethodBuilder.cs:1018
12-15 11:41:43.409 I/MonoDroid(30476):   at Android.App.SyncContext+c__AnonStorey0.<>m__0 () [0x00000] in /Users/builder/data/lanes/3511/ce955cc0/source/xamarin-android/src/Mono.Android/Android.App/SyncContext.cs:18
12-15 11:41:43.409 I/MonoDroid(30476):   at Java.Lang.Thread+RunnableImplementor.Run () [0x0000b] in /Users/builder/data/lanes/3511/ce955cc0/source/xamarin-android/src/Mono.Android/Java.Lang/Thread.cs:36
12-15 11:41:43.409 I/MonoDroid(30476):   at Java.Lang.IRunnableInvoker.n_Run (System.IntPtr jnienv, System.IntPtr native__this) [0x00009] in /Users/builder/data/lanes/3511/ce955cc0/source/monodroid/src/Mono.Android/platforms/android-24/src/generated/Java.Lang.IRunnable.cs:81
12-15 11:41:43.409 I/MonoDroid(30476):   at (wrapper dynamic-method) System.Object:6f241252-866a-4903-a936-1c7f81336bfc (intptr,intptr)
12-15 11:41:43.419 D/Mono    (30476): Assembly Ref addref HockeySDK.AndroidBindings[0xb3feece0] -> System[0xaa467560]: 16
12-15 11:41:43.439 W/art     (30476): JNI RegisterNativeMethods: attempt to register 0 native methods for android.runtime.JavaProxyThrowable

I discovered that this was due to failing to call Init early on, per the examples.  I was doing this elsewhere for my own reasons, so felt it was redundant. It is not an optional call.  Do Init at AppDelegate and at MainActivity, and this error will not occur.

So this can be closed?

Just that issue, yes. The other issue -- where you failed t provide two
files needed for compilation - must remain open until you remedy the
problem.
Stephen
On Fri, Dec 16, 2016 at 2:56 AM, Rajko Winkler<EMAIL_ADDRESS>wrote:

So this can be closed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rebuy-de/rb-forms-barcode/issues/59#issuecomment-267569450,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG3ZC0bkBKF2ZwLmqb8c_pugL7bvhHbCks5rIm5wgaJpZM4LOfzF
.


grpc Channel called pure virtual method when program exit

Here is my system info:
LSB Version:    :core-4.0-amd64:core-4.0-noarch:graphics-4.0-amd64:graphics-4.0-noarch:printing-4.0-amd64:printing-4.0-noarch
Distributor ID: CentOS
Description:    CentOS release 6.3 (Final)
Release:        6.3
Codename:       Final

Here is my gcc building environment:
COLLECT_GCC=gcc
Target: x86_64-baidu-linux-gnu
Configured with: ../configure CFLAGS=-O2 --host=x86_64-baidu-linux-gnu --target=x86_64-baidu-linux-gnu --prefix=/top --with-sysroot --with-local-prefix=/ --with-native-system-header-dir=/top/include --with-build-sysroot=/top --with-system-libunwind --enable-languages=c,c++ --enable-c99 --enable-threads=posix --enable-__cxa_atexit --enable-libstdcxx-allocator --enable-libstdcxx-threads --enable-libstdcxx-time --enable-libstdcxx-visibility --disable-multilib --disable-bootstrap
Thread model: posix
gcc version 4.8.2 (GCC)

When my C++ program, which invokes grpc client, called via a pybind11, everything works fine, but when my program exit, here is the stack:
pure virtual method called
terminate called without an active exception
*** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1531226147" if you are using GNU date ***
PC: @                0x0 (unknown)
*** SIGABRT (@0x1f40000b2c5) received by PID 45765 (TID 0x7f8c394c3740) from PID 45765; stack trace: ***
    @     0x7f8c38cd3130 (unknown)
    @     0x7f8c3822f9d9 __GI_raise
    @     0x7f8c382310e8 __GI_abort
    @     0x7f8c0dad3b39 __gnu_cxx::__verbose_terminate_handler()
    @     0x7f8c0dad21fb __cxxabiv1::__terminate()
    @     0x7f8c0dad2234 std::terminate()
    @     0x7f8c0dad2c05 __cxa_pure_virtual
    @     0x7f8bf7567626 grpc::Channel::~Channel()
    @     0x7f8bf75676d9 grpc::Channel::~Channel()
    @     0x7f8bf67955e9 std::_Sp_counted_base<>::_M_release()
    @     0x7f8bf7529f8e distributed::GRPCClient::~GRPCClient()
    @     0x7f8bf752a0a1 distributed::GRPCClient::~GRPCClient()
    @     0x7f8bf7531b9a std::unique_ptr<>::~unique_ptr()
    @     0x7f8c38233269 __run_exit_handlers
    @     0x7f8c382332b5 __GI_exit
    @     0x7f8c38fdee61 Py_Exit
    @     0x7f8c38fdef73 handle_system_exit.part.2
    @     0x7f8c38fdf1e5 PyErr_PrintEx
    @     0x7f8c38fdffb2 PyRun_SimpleFileExFlags
    @     0x7f8c38ff1d3c Py_Main
    @     0x7f8c3821bb45 __libc_start_main
    @     0x7f8c394e087f (unknown)
(END)

After debug for a long time, I found that grpc::Channel inherits from grpc::GrpcLibraryCodegen, and this class uses a global variable which will be destroyed before Channel;
Hope for the fix up.

I submit a PR for a hot fix, which will fix this problem quickly.

@dgquintas this sounds like a C++ issue, and Python usage is only incidental

If the fix in #15974 is any indication, you may want to try using a more recent version. @nicolasnoble also gave some guidance on that PR.


Error: Quantified groups are not allowed. Please solve this in another way.

Using this code in a web_application resource:
 exclude_xhr_regex = "/^(?:https?:\\/\\/)?(?:[a-zA-Z0-9-]+\\.)*(matchone|matchtwo)(?:\\/|$)/i"

and then running an apply job, results in:
Error: Quantified groups are not allowed. Please solve this in another way.
This regex is already applied via the UI and is exported via the terraform export option.
When comparing changes that are about to be made to the resource in the plan stage, exclude_xhr_regex is not listed - since it is the same as the current config.
However when running an apply job the resource fails with the above message.

Hello @smc87, I'm able to reproduce your issue. The REST API apparently insists on restrictions that aren't enforced when using the WebUI. I've already reached out to our developers for clarification - hoping that the solution will be to simply lift that restriction on the REST API.

Thanks very much

@smc87 Dynatrace ONE has reached out to me, about the fact, that you also had a chat open with them about the very same topic, and that you'd prefer to stick to just one communication channel.
I'm closing this ticket therefore. Once R&D gets back with an answer, you'll get notified on the DT1 chat.
In any case, feel free to drop us a message here, and I'll reopen the ticket.


Working on GCP Composer

Hi,
I'll be working on Cloud Composer client. I'll follow the same design patterns and structure used in this project. I expect to finish it by next week.
Do you recommend that I open a WIP PR? to follow the progress or to just go and create the PR when I'm done?
Thanks

Hi @tareksamni, thanks for opening the issue. I think either would work. If you want input during development then a PR would be a good vehicle for that. If not, then its up to you.
/cc @jbolinger

Yes, I think I would need input (my first big contribution to this project). and with the fact that there is no slack for this project or any other way of communication I'd love to have a WIP PR.
Thanks

the fact that there is no slack for this project

There is a Slack for GCP. You can sign up here, and find us in the the general #ruby channel or the project specific #google-cloud-ruby channel. We aren't heavy users of these slack channels, but we do maintain a presence there to discuss these gems with the community.

or any other way of communication

Our intention is to use GitHub issues and pull requests for as much of our communication as possible, so opening a WIP PR is fine.

Closing as the questions seems to be answered.


Fix chart location in OCI publish step

The chart repository is called charts/weave-gitops, not charts.
This should fix the error from https://github.com/weaveworks/weave-gitops/runs/7255134483?check_suite_focus=true

Hmm I think the old setting is already correct. The chart location does not include the chart name, so that it's only /charts.
weave-gitops which is the chart name will be appended automatically by the push command.
Please try helm push locally to very this behavior.
The 403 error in the run would be something related to the write permission of the token maybe (I'm totally speculating anyway).

Problem was that the weave-gitops user didn't have any access in the package settiings
I've added write permissions so it ought to work - of course, when I tested it and I got it working I used the gitops bot token so I don't know for sure if main still works. It should work, though, so closing.


npm-install: do_npm_*(): cd to ${B}

Change working directory to ${B} prior to running tasks from npm-install
class. This change is needed due to bitbake commit
67a7b8b021badc17d8fdf447c250e79d291e75f7.
Without this change do_npm_* functions are executed with ${TOPDIR}
as a rootdir. As a result they fail to find package.json.
Signed-off-by: Sergey Matyukevich<EMAIL_ADDRESS>

Looks good.

Related bitbake change has been introduced after krogoth release. So this fix is relevant for morty and upstream only.

BTW npm-install-global seems to be working fine w/o this fix. At least I didn't observed any issues building node-red using morty branches.

Are you sure package.json is always in ${B} and not in ${S}?

IIUC this is the case for npm-install class: B is assigned to ${S} on top of npm-install.bblcass file.

PR update: v1 -> v2
Make change more future-proof: reeplace ${B} by ${S} since package.json normally belongs to source directory rather than in build directory.


Made the scroll icon minimalistic

Description
Added a scroll up icon to navigate to top easily and instantly with smooth scrolling effect.
Checklist

[Yes ] I am making a proper pull request, not spam.
[ Yes] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
Issue #59
Add relevant screenshot or video (if any)

Finally did it, please let me know what do you think


Source map missing

After running my site that uses ng2-dnd with angular2 at 2.0.0 on chrome, everything works fine. But when I tried to debug my site, I always see "Failed to parse SourceMap" error messages for all the components of ng2-dnd. And I just search into the ng2-dnd folder inside of node_module folder and found no map files within it. Can you offer the map files? Or is there any workaround?
I installed through my package.json file and the verison of ng2-dnd I'm using is 2.0.1
The error message I've got:
Failed to parse SourceMap: http://localhost:62275/assets/js/libs/ng2-dnd/src/dnd.utils.js.map Failed to parse SourceMap: http://localhost:62275/assets/js/libs/ng2-dnd/index.js.map Failed to parse SourceMap: http://localhost:62275/assets/js/libs/ng2-dnd/src/dnd.config.js.map Failed to parse SourceMap: http://localhost:62275/assets/js/libs/ng2-dnd/src/dnd.service.js.map Failed to parse SourceMap: http://localhost:62275/assets/js/libs/ng2-dnd/src/droppable.component.js.map Failed to parse SourceMap: http://localhost:62275/assets/js/libs/ng2-dnd/src/abstract.component.js.map Failed to parse SourceMap: http://localhost:62275/assets/js/libs/ng2-dnd/src/draggable.component.js.map Failed to parse SourceMap: http://localhost:62275/assets/js/libs/ng2-dnd/src/sortable.component.js.map

Fixed. Please use release 2.2.1


Allow changing workspace storage type in some cases

What does this PR do?
This PR is a follow-up to https://github.com/devfile/devworkspace-operator/pull/833 that allows some storage-type modification after creation. It allows workspace storage type to be changed if:

The original storage type is ephemeral, or
If the workspace does not yet have a storage finalizer applied

In these cases, there is no data stored in any PVC, and so the storage class can be safely switched. This change allows

Creating workspaces a ephemeral, and then adding storage after the fact if it is necessary
Creating workspaces with started: false as a placeholder, and then changing the storage class before the first start.

What issues does this PR fix or reference?
N/A
Is it tested? How?
To test:

Create an ephemeral workspace -- e.g.
apiVersion: workspace.devfile.io/v1alpha2
kind: DevWorkspace
metadata:
  name: test-ephemeral
spec:
  routingClass: basic
  started: true
  template:
    attributes:
      controller.devfile.io/storage-type: ephemeral
    components:
    - name: web-terminal
      container:
        command: ["tail", "-f", "/dev/null"]
        image: quay.io/amisevsk/web-terminal-tooling:dev
        memoryLimit: 512Mi
        mountSources: true

and update the storage type to e.g. common

Create a workspace with started: false and update its storage type before starting it the first time:
yq '.spec.started=false' samples/theia-next.yaml | oc apply -f -

In both cases, changes should be permitted. Of course, for regular workspaces, the change should still be blocked.
PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v8-devworkspace-operator-e2e, v8-che-happy-path to trigger)

[ ] v8-devworkspace-operator-e2e: DevWorkspace e2e test
[ ] v8-che-happy-path: Happy path for verification integration with Che

I see, thank you @amisevsk and @AObuchow :)


KYLIN-4941 support encoding raw data to base cuboid column-by-column

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[x] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[x] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report

Merging #1631 (528ad97) into master (0250081) will decrease coverage by 0.11%.
The diff coverage is 5.21%.

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


Add strict mode

Be able to exit script when error occurs during parsing or code generation.
--strict

Generator and FileSystem have now a strict class attribute. If enabled it will exit the parser on any error.


new solver bug with negative number literals

type Range = { min: number?, max: number? }

local function foo(): Range
	return { min = -0, max = 0 }
end

local function bar(): Range
	return { min = 0, max = 0 }
end

In the above code, the return of foo is a type error, but the return of bar is not a type error. This should not be the case.

The type error is over the span of the entire return of foo: TypeError: Type pack '{ max: number?, min: number }' could not be converted into 'Range'; type { max: number?, min: number }[0][read "min"] (number) is not exactly Range[0][read "min"][1] (nil)


Button shows loading indicator when alpine not initialized yet

Package
filament/filament
Package Version
v2.12.27
Laravel Version
v9.13.0
Livewire Version
v2.10.5
PHP Version
PHP 8.1.2
Bug description
Related to #2643 and #2665
https://github.com/laravel-filament/filament/blob/683d52cbe362d4e1e9913826efabc81832017525/packages/support/resources/views/components/button.blade.php#L107 and https://github.com/laravel-filament/filament/blob/683d52cbe362d4e1e9913826efabc81832017525/packages/support/resources/views/components/button.blade.php#L117 is useless
because laravel compile blank component attribute value into its name, so these lines will be compiled into x-cloak="x-cloak"
the simplest way I can think to fix this is to change
https://github.com/laravel-filament/filament/blob/2e38df333b85989490833d2db0405055e178b6e7/packages/admin/resources/views/components/layouts/base.blade.php#L29
into
[x-cloak=""], [x-cloak="x-cloak"], [x-cloak="1"] { display: none !important; }

another way is to change all x-cloak on all blade components to x-cloak=""
Steps to reproduce
No response
Relevant log output
No response

Thanks for the solution