Reasonable scala

Recognizing scalac is slow no matter how you engineer around it and the fact that zinc  integration may complicate the bazel model here is another option to keep an eye on https://github.com/twitter/reasonable-scala

Thanks!
Do you have an AI we can make out of it or is it more of an FYI?
On Fri, 18 Aug 2017 at 16:22 doug tangren<EMAIL_ADDRESS>wrote:

Recognizing scalac is slow no matter how you engineer around it and the
fact that zinc integration may complicate the bazel model here is another
option to keep an eye on https://github.com/twitter/reasonable-scala
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/263, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIFx6Z84q-xHKzi4IWDeUcTeV9wRg_ks5sZZADgaJpZM4O7i-z
.

lol. More or less this is just and FYI. Just crossed my radar and wanted to leave a bread crumb for future reference. I don't have any insider info myself ( just high hopes! )

Got you. Crossed my radar today as well :)
I'll close this if you don't mind since we have a lot of open issues and I'd rather we keep the ones we can actually advance.
Please feel free to reopen if you think otherwise or want to resurface

I actually think I was too quick to close this.
Maybe an AI can be to have an experimental rsc worker?
@softprops WDYT? In addition do you think it would interest you to try and contribute such a worker?
cc @gkossakowski who might be interested in this as well

There wasn't any actual code under https://github.com/twitter/reasonable-scala available yet. It's is hosted/published somewhere else?

Good question. Not sure.
In any case even if it's not published yet I guess it will be soon since it
sounds like they have an initial something working.
Do you think the AI I outlined is relevant?
On Sun, 20 Aug 2017 at 21:33 doug tangren<EMAIL_ADDRESS>wrote:

There wasn't any actual code under
https://github.com/twitter/reasonable-scala available yet. It's is
hosted/published somewhere else?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/263#issuecomment-323603002,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIFwZsGmG_t5C8JPl8l7rdbg1pAMJsks5saHv1gaJpZM4O7i-z
.

Hi, new to the convo. What's the AI you're referring to?
Thanks for CC'ing me. rsc builds on ideas I explored in Kentucky Mule. I'm
curious how Twitter's effort is going to unfold.
On Sun, Aug 20, 2017 at 12:09 PM Ittai Zeidman<EMAIL_ADDRESS>wrote:

Good question. Not sure.
In any case even if it's not published yet I guess it will be soon since it
sounds like they have an initial something working.
Do you think the AI I outlined is relevant?
On Sun, 20 Aug 2017 at 21:33 doug tangren<EMAIL_ADDRESS>wrote:

There wasn't any actual code under
https://github.com/twitter/reasonable-scala available yet. It's is
hosted/published somewhere else?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<
https://github.com/bazelbuild/rules_scala/issues/263#issuecomment-323603002
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ABUIFwZsGmG_t5C8JPl8l7rdbg1pAMJsks5saHv1gaJpZM4O7i-z
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/263#issuecomment-323605111,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAQeW2wnXFdUNGs6-ZGjCdciSylzRIyks5saIROgaJpZM4O7i-z
.

--
gkk

Also curious. Please don't say action item ;)

The "change" I suggested (and which might not be the right one) is to add a
new worker which will use rsc compiler instead of the scalac compiler and
have that as an experimental worker.
So people can experiment with it and give back feedback.
I know, this is as good as place as any to say thank you for those ideas
and explorations. I have to say it brought some hope when I read it :)
On Mon, 21 Aug 2017 at 0:57 doug tangren<EMAIL_ADDRESS>wrote:

Also curious. Please don't say action item ;)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/263#issuecomment-323614597,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIF0bf8g5fITONmpFCB7HHl8HxRNIuks5saKu_gaJpZM4O7i-z
.

@gkossakowski you've probably saw this tweet:

The prototype of the Reasonable Scala typechecker has been released open source and is more than 20x faster than Scala.

https://github.com/twitter/reasonable-scala
Any chance you'll try to take it out for a spin in rules_scala?

Yeah I saw it. I've been following some of the news because it's directly
inspired by my work on Kentucky Mule
I think trying it out from command line is fine as long as it's in it's
current infant phase.
On 16 November 2017 at 12:35, Ittai Zeidman<EMAIL_ADDRESS>wrote:

@gkossakowski https://github.com/gkossakowski you've probably saw this
tweet https://twitter.com/TwitterOSS/status/931257305542172672:
The prototype of the Reasonable Scala typechecker has been released open
source and is more than 20x faster than Scala.
https://github.com/twitter/reasonable-scala
Any chance you'll try to take it out for a spin in rules_scala?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/263#issuecomment-345054372,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAQeQ_twpGYrcLr6emCBRGPyRX7raf1ks5s3JyRgaJpZM4O7i-z
.

--
gkk


[BUG]Unable to synchronize, please give me a node

Unable to synchronize, please give me a node

<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

What is the port number?

<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

What is the port number?

9888

<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

What is the port number?

9888

I'm using V1.1.715...This version does not link to these nodes

<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

What is the port number?

9888

I'm using V1.1.715...This version does not link to these nodes

All node is running

Can you provide db file with updatd sync

Can you provide db file with updatd sync

https://www.dropbox.com/s/yc405bw5yio0l1b/blockchain_v1_mainnet.sqlite?dl=0

After update Not Syncing.. plz update the node address plz

After update Not Syncing.. plz update the node address plz

node1.equalitychain.org:9888
node2.equalitychain.org:9888
node3.equalitychain.org:9888
node4.equalitychain.org:9888


Lsp for JavaScript/TypeScript

After some short testing Lsp for C/C++ I started looking something for JavaScript. From what I have seen the only solution is:
typescript-language-server which is a bridge for tsserver (something from Microsoft used in their tools, unfortunately it still does not support LSP directly https://github.com/microsoft/TypeScript/issues/39459).
How to install?

If you already have Node available in Path and want to have these tools also globally, then call:
npm install -g typescript-language-server typescript
Then config for Lsp should looks like:

[lspservers."javascript"]
mode = "io"
args = '--stdio'
executable = 'typescript-language-server.cmd'
auto_start_server = false

For people who don't have/don't use Node and want to check it quickly locally:
2.1 Download Node (with NPM) from here https://nodejs.org/dist/latest/ (.7z or .zip).
2.2 Unpack it for example to Node folder.
2.3 Inside Node folder run:
npm install -g typescript-language-server typescript

Inside Node and Node\node_modules new files should appear.

Then config for Lsp should looks like:
[lspservers."javascript"]
mode = "io"
args = '--stdio'
executable = 'Node\typescript-language-server.cmd'
auto_start_server = false

Of course, Node is a virtual name, it should be the full path to this folder, like:
D:\Node
D:\somefolder\Node
C:\tools\Node

etc., depending on where you unpacked Node.
After a quick tests, I can see that the most important commands work:

Goto definition
Find references
Symbol windows
Format document and Format selected text

Not work:

Rename << but the same I have for C/C++
Ask for help << it shows something there and we can select, but after that it does not perform any operation.

Poorly work:

the popup with the hint works, but when we click on the link it always opens a tiny window, it doesn't remember the last size.

Nice, I didn't think I'd see contextual jumping in Notepad++ for JavaScript (tools based on ctags can't do that). Great job, I hope you keep developing it, in Notepad++ we miss these new solutions from other editors when working on code (with larger code they become more and more necessary).

Thank you very much for taking the time to document how a node based server can be used.
I will, with your permission, work it into the tips and tricks section.
With Rename and Ask for help, which is basically lsp actions, there are variations of the returned structures
which are not yet fully implemented. But having servers that use them makes it easier to implement them.
I enjoy programming my plugins, but unfortunately the time I can spend on it is very limited at the moment.
I hope that I can work on it for 4-5 hours a day during my vacation then it should be a whole step closer to be a fully working lsp client.

Thank you very much for taking the time to document how a node based server can be used.
I will, with your permission, work it into the tips and tricks section.

No problem, I prefer to discuss here rather than on the forum, because it's always closer to the plugin. It's easier to search/read such discussions and tips later.

I enjoy programming my plugins, but unfortunately the time I can spend on it is very limited at the moment.
I hope that I can work on it for 4-5 hours a day during my vacation then it should be a whole step closer to be a fully working lsp client.

There is no rush, it's not easy either, from what I see each LSP server has different coverage (and other differences, like configuration) so in the future it will probably be necessary to fine-tune everything a bit for specific ones (at least the most popular ones that the Notepad++ community would prefer).
If I ask a lot of questions here or open a new issue, there is no need for a quick response, I just write down what I can observe so that I don't forget about it later. When you have a moment, you can come back to the topic.
At the moment I have one important question, can you tell me if the LSP protocol supports any filtration for the folders/files it operates on? What I mean is that if we have a Project folder, can we somehow attach a folder from another location to it, like SomeLibrary? And the opposite situation, can we exclude some other folder from the Project folder, e.g. modules? In short:
Somelibrary << include this folder
Project << this is root of project
    modules << exclude this folder
    ...

If the LSP protocol itself does not provide this, then we will probably need to analyze each server separately and check its options.

I prefer to discuss here

I also think it makes more sense to discuss plugin related stuff here.

in the future it will probably be necessary to fine-tune

Yes, this is necessary because it is possible to have custom lsp messages per server,
which I currently ignore and this will probably not be available for the first officially released version.
First the basic functionality has to work reliably.

can you tell me if the LSP protocol supports any filtration for the folders/files it operates on?

Yes, Workspace folders.

can we exclude some other folder

but no, if you mean that such a folder is a subdirectory of the project root directory.
Currently, the content of the FoldersAsWorkspaces(FAW) dialog is used to inform the servers about the directories in use.
When the buffer is activated, I check whether the content has changed and send the notification Workspace/didChangeWorkspaceFolders accordingly.
This means that only FAWs root nodes are taken into account.
The problem I have with FAW is that it is a performance bottleneck with language servers that compile sources, as it monitors and updates the directories. Also, it would be really helpful if I could reuse a dialog like this to show a diagnostic counter next to the folders, so you can see immediately that there are problems without having to open the diagnostic dialog. So I'm thinking about giving the lsp client its own "Projects/FAW" dialog.

@Ekopalypse But how all roots in FAW are treated, as the same project or separate? I mean will the suggestions, searches etc. work separately for each root or together?
In the case of tsserver we can control on which files it operates by using the config file. I tried this and indeed the files property works:
{
  "[files]": [
    "file1.js",
    "file2.js",
  ]
}

Unfortunately, filtering by using include/excludedoesn't work for me. It's possible that I'm doing something wrong.
In general, some additional plugin configuration through a file placed in the project folder would be useful for those who don't use FAW (because it's an additional panel, and we not always have much space on the screen). But I'll open a separate issue for this problem.

But how all roots in FAW are treated

How the root directories are interpreted by the respective servers is beyond the client's knowledge.
The client sends the information about which root directory(s) are available in the initialize request.
This takes place in 3 places, either via

workspaceFolders or, if this is not used, via
rootUri or if this is not used via
rootPath

Now it is up to the respective server to use this knowledge to search for the corresponding language-specific configuration files, if available/used. The rust-analyzer would now search for a cargo.toml in one of these directories, clangd would probably look for the compile_commands.json etc.
But the current way, using FAW, also has the disadvantage that a root directory, which for example only makes sense for the typescript server, would also be made known to clangd. Unfortunately, I don't see how this can be solved easily.

In general, some additional plugin configuration through a file ...

But how is this supposed to work? If we take the following project as an example,
root
 - sub_dir
  - sub_sub_dir
   - file1
   - file2
 - sub_dir2
  - sub_sub_dir2
   - file3
   - file4

If file4 is now opened, how does the LspClient know where to find this configuration file?
It could of course go from sub_sub_dir2 upwards and search for it, but what if sub_dir2 was already an independent project?
In Rust, for example, it is common for it to look like this
root
 - sub_dir
  - sub_sub_dir
   - file1.rs
   - file2.rs
  lib.rs
  cargo.toml
 - sub_dir2
  - sub_sub_dir2
   - file3.rs
   - file4.rs
  lib.rs
  cargo.toml
 cargo.toml

This means that rust-analyzer only needs the cargo.tml from sub_dir2 to be able to use file4.rs.
Of course, this makes compiling much faster than if it were to compile the complete project.
It would be conceivable, at least for servers that use such configuration files, to define them in the NppLspClient.toml file. So that the client knows that for the rust-analyzer a cargo.toml would have to be searched for, but even that will not be 100% satisfactory, that it can ALSO be that in the above example a cargo.toml exists in the sub_dir2 directory but the cargo.toml from the root directory is still required for the compilation to be successful.

@Ekopalypse I will open a new issue for this separate config file because it's probably a bigger problem.
Returning to JS I got what I asked for. I had to use jsconfig.json instead of tsconfig.json file with below content:
{
    "include": [
        "**/*.js",
	"D:/outerjs/test3.js"
    ],
    "exclude": [
        "innerjs/*"
    ],
}

We still can use tsconfig.json file but something needs to be added:
{
    "include": [
        "**/*.js",
	"D:/outerjs/test3.js"
    ],
    "exclude": [
        "innerjs/*"
    ],
    "compilerOptions": {
        "allowJs": true,
    }
}

Thanks to this I can point to files that should be analyzed (even outside the root), and add some exclusions inside the root. That's what I cared about the most.
I don't need to open/activate all nested files for LSP commands to work, just one from root is enough. The only problem is when I start the server from a file nested in root a little deeper, it still somehow can't find things above, I have to activate some file directly from root. At the moment it's acceptable, unless I find a solution.
If anyone uses JS/TS let me know what the problem is. In short, when we run the server on a file that is 1 or more level nested below the root, it does not search for things above  unless we go to a file that is directly in the root. jsconfig.json or tsconfig.json is located in root. It looks like the search for these config files is not going up (manual).

The only problem is when I start the server from a file nested in root a little deeper

FAW is currently required at this point. The user must have specified the root directory of the project. If you open a file that cannot be found in one of the root directories of FAW, the file directory is used in the initialization request.

@Ekopalypse

The user must have specified the root directory of the project.

Some configuration (separate file/command or something else for discussion) would be useful for things like this.
Regardless of this, I noticed one interesting thing for JS:
dirA
    jsconfig.json
    fileA1.js
    dirA2
        fileA2.js

dirB
    fileB1.js
    fileB2.js

When fileA2.js is active and we run server then it't not recognized command from fileA1.js (we need switch to this file, then it will probably process the file jsconfig.json). So in this case it doesn't loop over all parent folders of fileA2.js.

When fileB2.js is active and we run server then command from fileB1.js is recognized (that's interesting because if we did opposite, the commands from fileB2.js would not be available in fileB1.js until we activated file fileB2.js). Well, this seems to be the default behavior within the folder dirB.
But that's not the point, what's more interesting is that if now we switch to fileA2.js it read jsconfig.json because all the commands from above folders are recognized inside file fileA2.js. So in this case it loop over all parent folders of fileA2.js. It's strange, why wasn't this done in the first case? Was the initialization done differently or is something else important here? I thought it would just stick on dirA2 (like in first case).
It's a bit confusing because sometimes it iterates up through folders and other times it doesn't. It's difficult to use configuration files in such a situation (we must remember about such variations).

You ran this test with dirA and dirB as root folders in FAW?
If the server supports workspace folders, I would have thought it would recognize the jsconfig.json file and know what to do.
Do you have a simple demo I can play with?

You ran this test with dirA and dirB as root folders in FAW?

No, I run it without FAW, this LSP doesn't support FAW. But it's more  specific because it supports single file mode.
It seems to me that when we run it for the first time (without FAW) you set root to the current folder and server will then read the config file from that folder if it exists (but it doesn't try to go higher if it doesn't exist). Which would correspond to this description from the manual:

By invoking tsc with no input files and a --project (or just -p) command line option that specifies the path of a directory containing a tsconfig.json file, or a path to a valid .json file containing the configurations.

Then, when we switch to some other file from a folder higher up (or a completely separate folder), it probably treats it as if there was no root specified, and then it looks for the configuration file iterating upwards.  Which would correspond to this description from the manual:

By invoking tsc with no input files, in which case the compiler searches for the tsconfig.json file starting in the current directory and continuing up the parent directory chain.

What's interesting is that when we have separate folders with configuration files, it internally treats them as separate projects, it doesn't get lost in navigation through definitions, etc., everything worked as if I had separate projects, or at least that's what I think, because I only tested it on simple functions/variables.

Do you have a simple demo I can play with?

If you want to play with it I will create a folder with such cases in various subfolders. I'll just mention that I don't use commands like import/require etc. for modules. I just treat files as if they lived in one scope and all of them should have access to everything (as if it was one file). Because when we use module import it processes them immediately and everything below is recognized (then even jsconfig.json is unnecessary here).

No, I run it without FAW, this LSP ...

Just to be clear: FAW is not directly related to the (workspace folder)[https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#workspace_workspaceFolders] feature of the lsp protocol. Although I also use it for this purpose, the other main reason to use it is that a main project root directory can be determined by the lsp client. So the problem with opening fileA2.js and trying to see functions from both fileA1.js and fileB1.js or fileB2.js should not have occurred. (As long as the jsconfig.json has been setup correctly).
To open fileB*.js and see definitions from fileA*.js, in this scenario, it would need the support of the workspace folders from the server.

@ArkadiuszMichalski - reported renaming and code action issues should be fixed in the released version v.0.0.27-alpha.


StructuredArguments and @JsonFormat

Although logstash-logback-encoder is using Jackson, I'm having trouble getting it to recognize the @JsonFormat annotation when using StructuredArguments.
Logged argument's class declares a field of type LocalDate:
@JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd")
private final LocalDate dateFilled;

Encoder configured as follows in logback.xml:
<encoder class="net.logstash.logback.encoder.LogstashEncoder">
	<provider
		class="net.logstash.logback.composite.loggingevent.ArgumentsJsonProvider" />
</encoder>

But output of the field is:
"dateFilled":{"year":2016,"month":"DECEMBER","dayOfMonth":27,"dayOfWeek":"TUESDAY","era":"CE","dayOfYear":362,"leapYear":true,"monthValue":12,"chronology":{"id":"ISO","calendarType":"iso8601"}}

Desired output of the field is:
"dateFilled": "2016-12-27"

I was able to get the output that you would like by doing the following:
First, ensure you have the jackson-datatype-jsr310 jackson module on your classpath.  This is the jackson module that knows how to format the new date/time related classes in java 8.
        <dependency>
            <groupId>com.fasterxml.jackson.datatype</groupId>
            <artifactId>jackson-datatype-jsr310</artifactId>
            <version>${com.fasterxml.jackson.version}</version>
        </dependency>

Then create a JsonFactoryDecorator that registers jackson's JavaTimeModule, like this
import net.logstash.logback.decorate.JsonFactoryDecorator;

import com.fasterxml.jackson.databind.MappingJsonFactory;
import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule;

public class Java8DateTimeJsonFactoryDecorator implements JsonFactoryDecorator {
    @Override
    public MappingJsonFactory decorate(MappingJsonFactory factory) {
        factory.getCodec().registerModule(new JavaTimeModule());
        return factory;
    }
}

And finally, tell the encoder to use the Java8DateTimeJsonFactoryDecorator like this:
<encoder class="net.logstash.logback.encoder.LogstashEncoder">
    <provider class="net.logstash.logback.composite.loggingevent.ArgumentsJsonProvider" />
    <jsonFactoryDecorator class="your.package.Java8DateTimeJsonFactoryDecorator"/>
</encoder>


Fixes for TestNSURL.swift on 32-bit (NFC)

This is a relatively minor patch that casts the result of sizeof to UInt so that UInt32 (from 32-bit machines) is promoted to match the other logic.

Are you sure this is not a symptom of incorrectly importing some API on Linux/armv7, for example, strlen()?

@gribozavr sorry for the delay in getting back, I was focusing on that other PR.  I don't see anywhere where strlen is imported differently from other c functions.  Ultimately, size_t is being imported as a UInt32 rather than a UInt.  Looking at MappedTypes, it seems like it should be mapped as Int.

Ultimately, size_t is being imported as a UInt32 rather than a UInt.

Which is an issue that should be fixed on the compiler side.  It will cause a lot of other API inconsistencies.


Add ability to add a low and high watermark for each bytes pool bucket

cc @prateek @richardartoul

Coverage remained the same at 77.833% when pulling 54fe2eee047784789637df2aaad0d75850b44fb4 on r/bytes-pool-bucket-options into 59f549a03d16b8d6c34a8fbd6b7f875844157497 on master.


file upload form more than one in a form

hi @volkanceylan,,
i create like this :
[DisplayName("ESR Document"), MultipleFileUploadEditor]
public String ESRDocument
{
get { return Fields.ESRDocument[this]; }
set { Fields.ESRDocument[this] = value; }
}
    [DisplayName("CAF Document"), MultipleFileUploadEditor]
    public String CAFDocument
    {
        get { return Fields.CAFDocument[this]; }
        set { Fields.CAFDocument[this] = value; }
    }

    [DisplayName("MLA Document"), MultipleFileUploadEditor]
    public String MLADocument
    {
        get { return Fields.MLADocument[this]; }
        set { Fields.MLADocument[this] = value; }
    }

and when i drag and drop my file to ESRDocument, the other form (CAFDocument, MLADocument) also automatically filled with the same file.
and i don't want to be like that.
do you have any idea ?

hi @volkanceylan where are you ?
i need your help

@hamkairama try to change location of file for every upload document something like
[DisplayName("CAF Document"), MultipleFileUploadEditor (FilenameFormat = "Document/CAF /{1:00000}/{0:00000000}_{2}")]
    public String CAFDocument
    {
        get { return Fields.CAFDocument[this]; }
        set { Fields.CAFDocument[this] = value; }
    }

    [DisplayName("MLA Document"), MultipleFileUploadEditor(FilenameFormat = "Document/MLA /{1:00000}/{0:00000000}_{2}")]
    public String MLADocument
    {
        get { return Fields.MLADocument[this]; }
        set { Fields.MLADocument[this] = value; }
    }

@mfathy00  i have done it. but still not work..
everything fine if i select using browse.
but when i do with drag and drop, it's still happen.

@hamkairama What browser do you use?
I tried on Google Chrome, Edge, Firefox and IE11, but I didn't experience the same behavior as you.
Also your browser rendered the page different from mine.
I think there is something wrong with your browser.
Try to turn off any extensions or try with other browsers.

        protected getUploadInputOptions(): UploadInputOptions {
            return {
                container: this.toolbar.findButton('add-file-button'),
                zone: this.element,

Upload zone (drag drop zone) is set to this.element, e.g. the upload editor itself, if it is not working on your side something else should be wrong, probably with the upload component (jQueryFileUpload) and last commenter can't reproduce such behavior.


OverlayView loses other libraries context

I'm using react material-ui lib with react-google-maps.
When I'm trying to nest any material-ui component, eg. List, into OverlayView, I get this error:

Warning: Failed context type: The context muiTheme is marked as required in List, but its value is undefined.

In whole application, including other react-google-maps components muiTheme exist in the context.

hrmm yeah this could def be an issue.  I'll take a look.
Out of curiosity, have you tried to pass it down as props from the parent context?

Yes I tried and it is not working.

thanks @tomasz89nowak i'll look at it shortly.

Any upates here?

I was able to confirm this, seems like a known issue.  There are also a number of big changes in play for the material-ui library(1.0 coming soon).  I'm tempted to defer until they've released 1.0 so we can accommodate that release as opposed to trying to support 0.XX

Hey! I think it's not problem with material ui, but with OverlayView context.
Im using isomorphic-style-loader and it doesnt work OverlayView..

Alright I'll mark this as an issue that we can dev against

Here is sceenshot of Chrome browser React dev tool - Overlayview is rendered outside App

Fixed in v8.0.0. Please try the new version.
https://tomchentw.github.io/react-google-maps/#!/OverlayView

The issue still occurs in version 8.5.0.
OverlayView children are missing context when using isomorphic-style-loader.


[UI] Registry: Registrants tab has improper count of registered Models

Current Behavior
See http://localhost:9081/settings?settingsCategory=Registry&tab=Registrants
Screenshots/Logs

Environment

Browser: Safari
Host OS: Mac
Meshery Server Version: stable-v0.7.24

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Fixed


Cannot read property 'BrowserWindow' of undefined

The only place I can require('electron-oauth2') is from src/system/index.js. When I require the npm in this file I do not get an error. I was able to make authorization work.
However, when it came time to move electron-oauth2 into antother file src/authorization/index.js, the simple act of requiring the npm throws an error
 TypeError: Cannot read property 'BrowserWindow' of undefined

      at Object.<anonymous> (node_modules/electron-oauth2/index.js:9:64)
      at Object.<anonymous> (src/authorization/index.js:2:22)
      at Object.<anonymous> (src/authorization/index.test.js:1:145)
          at <anonymous>

This line of code causes the above error in any file that I add it to except src/system/index.js which is the entry point for my main electron app.
var electronOauth2 = require('electron-oauth2')
I've tried making up new files and adding this this reqiure statement, it always fails. I've tried adding this require statement to the test, same error. But for some reason it will work from the main electron js file only. Adding this line to that file does not throw any errors during testing or during runtime. It doesn't matter if I use it or not, the mere act of requiring it throws the error.

I've really never seen anything like this. I literally commented out all of the code in both files except for var electronOauth2 = require('electron-oauth2'). In the main file it works, no error. In any other file it throws the BrowserWindow error. Super strange.
src/system/index.js --> No Browser Window Error
// var electron = require('electron')
// var BrowserWindow = electron.BrowserWindow
// var app = electron.app
// var path = require('path')
var electronOauth2 = require('electron-oauth2')
//
// var withDevFlag = () => process.argv[2] && process.argv[2] === '--dev'
//
// if (withDevFlag()) {
//   console.log('Development Server Running on http://localhost:3000')
// }
//
// const onClose = () => {
//   app.quit()
// }
//
// const onReady = () => {

...

src/authorization/index.js --> Browser Window Error
// var fetch = require('isomorphic-fetch')
//
var electronOauth2 = require('electron-oauth2')
//
// const config = {
//     clientId: <SECURE ID>,
//     clientSecret: <SECURE SECERET>,
//     authorizationUrl: 'http://github.com/login/oauth/authorize',
//     tokenUrl: 'https://github.com/login/oauth/access_token',
//     useBasicAuthorizationHeader: false,
//     redirectUri: 'http://localhost'
// }
//
// const windowParams = {
//   alwaysOnTop: true,
//   autoHideMenuBar: true,
//   webPreferences: {
//       nodeIntegration: false
//   }
// }

...

I was able to fix this issue with: yarn add electron@latest


Switching from Google fonts to Bunny fonts

Google Fonts aren't GDPR compliant - Bunny Fonts (or self-hosting fonts) is recommended:
https://european-alternatives.eu/alternative-to/google-fonts

Thanks for this @palmiak!!! Very cool of you!
I'm still searching for a better way to include fonts tho. Loading them over HTTP can be slow and cause CLS sometimes, but in the meantime, I'm happy to merge this!

Right now, there are no gains from using CDN for fonts (apart from the fact it's easy to use). Personally, I would just download all the fonts and add them to the repo.

Right now, there are no gains from using CDN for fonts (apart from the fact it's easy to use). Personally, I would just download all the fonts and add them to the repo.

Exactly! That's also the way I was thinking! Gonna put it in an issue and on my to-do list


gdbserver: be stricter about matching qC

Don't match only the prefix of qC. Otherwise, compare-sections in gdb will fail, due to qCRC returning a thread ID.

/azp run

hello, would you mind rebasing your PR so that we can re-run CI please ? <3


Update recommendations for organisations with GB-MPR identifiers

The mutual public register is now the better source for information about registered societies and mutuals, than Companies House. Find that Charity should recommend the GB-MPR above GB-COH when an organisation appears on both lists.

Example of this: https://findthatcharity.uk/orgid/GB-MPR-129CUS

Fixed through 6ee0d0883d830a9a8d12527c1f2479fa9b39eb5a

This is now implemented.
Interestingly the example I gave (GB-MPR-129CUS) is actually matched to two different records on Companies House - GB-COH-RS00129S and GB-COH-IP000129

Oh, that's not great. Hopefully it isn't common. Even more reason to prefer GB-MPR now.


pacinian corpuscle terms and synonyms

Add new terms:
decreased pacinian corpuscle number (child of abnormal pacinian corpuscle morphology MP:0000988)
DEF: fewer than the expected number of the rapidly adapting mechanoreceptors found in subcutaneous tissue beneath both hairy and glabrous skin which normally contain an afferent nerve fiber surrounded by a capsule with multiple concentric layers and transmit signals generated by vibrations when grasping an object
SYN: reduced pacinian corpuscle number
SYN: decreased lamellar corpuscle number
SYN: decreased lamellated corpuscle number
SYN: reduced lamellar corpuscle number
SYN: reduced lamellated corpuscle number
absent pacinian corpuscles (child of decreased pacinian corpuscle number)
DEF: absence of all of the rapidly adapting mechanoreceptors found in subcutaneous tissue beneath both hairy and glabrous skin which normally contain an afferent nerve fiber surrounded by a capsule with multiple concentric layers and transmit signals generated by vibrations when grasping an object
SYN: absent lamellar corpuscles
SYN: absent lamellated corpuscles
small pacinian corpuscles
DEF: decreased size of the rapidly adapting mechanoreceptors found in subcutaneous tissue beneath both hairy and glabrous skin which normally contain an afferent nerve fiber surrounded by a capsule with multiple concentric layers and transmit signals generated by vibrations when grasping an object
SYN: decreased pacinian corpuscle size
SYN: decreased lamellar corpuscle size
SYN: decreased lamellated corpuscle size
SYN: reduced pacinian corpuscle size
SYN: reduced lamellar corpuscle size
SYN: reduced lamellated corpuscle size
SYN: small lamellar corpuscles
SYN: small lamellated corpuscles

Added new terms:
decreased pacinian corpuscle number MP:0030996
absent pacinian corpuscles MP:0030997
small pacinian corpuscles MP:0030998
Also added synonyms to parent term abnormal pacinian corpuscle morphology MP:0000988


Add "Forgot Password" Feature to Login Page

Add Forgot password in Login If user Forgot
Issue:
Currently, there is no option for users to reset their password if they forget it, which can lead to frustration and hinder access to their accounts.
Proposed Solution:
Implement a "Forgot Password" link on the login page. This link should direct users to a password recovery process, where they can enter their registered email address to receive instructions for resetting their password. This enhancement will improve user accessibility and overall experience.

@Mdraza78 thanks for your time but this issue is covered in issue #8


issues when finetuning the pretrained models for GQA

There exists the random seed generator in your codes. And I  found it initially set as 9595. And I also found the parameters you loaded the pretrained models are unchanged if I change the random seed ,for example 2. But the final results on val datasets are worse, less than 0.5 point. So I wonder the initial parameters are the same ,just the difference of random seed. Why can it cause such different results?

In general, GPU computation is non-deterministic to speed the training process up, even the seed is set. Sometimes, the same seed would give the same results, but it is just by luck. The results would change if you upgrade CUDA / upgrade PyTorch / or just run the same experiment at a different time.
Please read this issue for detailed explanation.


change speed

Hello. Incredible work!
Using the 'drugs' sample, how to we slow the speed down to about 25%?
proton = new Proton;
  emitter = new Proton.Emitter();
  emitter.rate = new Proton.Rate(new Proton.Span(5, 10), new Proton.Span(.05, .2));
  emitter.addInitialize(new Proton.ImageTarget(image));
  emitter.addInitialize(new Proton.Mass(1));
  emitter.addInitialize(new Proton.Life(2, 4));
  emitter.addInitialize(new Proton.V(new Proton.Span(0.5, 1.5), new Proton.Span(0, 360), 'polar'));
  emitter.addBehaviour(new Proton.Alpha(1, [.7, 1]));
  emitter.addBehaviour(new Proton.Scale(1, 0));
  emitter.addBehaviour(new Proton.Color('random', '#9e24e7', Infinity, Proton.easeInSine));
  emitter.p.x = canvas.width / 2;
  emitter.p.y = canvas.height / 2;
  emitter.emit();
  proton.addEmitter(emitter);
  renderer = new Proton.Renderer('canvas', proton, canvas);
  renderer.start();

^---- Which parameter to modify above?

Found it, thanks. Deleting issue.

could you help out others by posting the solution?

For me with easeljs the only way to slow down a particle animation is to adjust the FPS, which I can't do as I have other elements that need it to be 60.
I tried using Proton.Velocity and that didn't help either.  I hope I'm overlooking something simple here!


About training

Hello, the efficientvit model you provided is impressive. I want to study it in depth and modify it to learn better. But I am missing the training code. Could you provide the training code for this code, please?

Yeah....Even i am trying to study and research this impressive code.Could you please provide us with it.It would be really helpful for us and enable us to do higher research.

The training guide for ImageNet classification is available at https://github.com/mit-han-lab/efficientvit/blob/master/TRAINING.md.
The training guide for segmentation will be released soon.
Thanks for your interest in our work.

Thank you very much for releasing new training ocde. It will be great for our research study.
---Original---
From: "Han @.>
Date: Fri, Jul 21, 2023 23:13 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [mit-han-lab/efficientvit] About training (Issue #2)
The training guide for ImageNet classification is available at https://github.com/mit-han-lab/efficientvit/blob/master/TRAINING.md.
The training guide for segmentation will be released soon.
Thanks for your interest in our work.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


syncHistoryWithStore (sync.js:104) :   Cannot read property 'listen' of undefined

code:
import { Router, browserHistory } from 'react-router';

const history = syncHistoryWithStore(browserHistory, store)

config;
"redux": "^3.2.1",
"react-redux": "^4.3.0",
"react-router-redux": "^4.0.8"

how do I solve this problem?

Wrong repo. I think you want react-router-redux.


Add frontend unit tests to run on github actions on PR

Purpose
Previously, the frontend unit tests were only running on push to main branch.
This PR adds the configuration to run them also on every pull request made to the main branch.
Does this introduce a breaking change?
[ ] Yes
[ x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x ] Other - config change to github actions workflow.

How to Test
I've tested this config in this run: https://github.com/Azure-Samples/chat-with-your-data-solution-accelerator/actions/runs/8721988816/job/23926899985

I have no strong opinions on the topic :) What is the advantage of using working dir instead of prefix?

(I think) I remember seeing some posts online about --prefix having unexpected results sometimes, and it's better to run the commands in the directory itself

I have no strong opinions on the topic :) What is the advantage of using working dir instead of prefix?

I would say readability (ultimately the same end result). If I'm parsing the file I can see "oh it's npm install"

Can we still use the make commands for these?

Can we still use the make commands for these?

We could, but the file would then be a mix of make and non-make commands. There are several different ways of achieving the same result - ultimately what's important is that we run the tests on every PR.
I have now changed the steps to use working directory as per the previous comments.


[Feature] Automatically generate includes regex

Grep filepaths from compiled .class files in Maven target directory to find package names.
Add Maven plugin option -Djtec.autoExtractInclude (or similar) than automatically generates the overarching includes regex.

#59


Fix 'primary_key' on primary-key-less tables

Currently, nulldb is erroring when it encounters a habtm relationship. It's erroring on the connection adapter's primary_key, since habtm dosen't have one. I think it's OK for primary_key to return nil, since this is the definition for the mysql adapter:
def primary_key(table)
  pk_and_sequence = pk_and_sequence_for(table)
  pk_and_sequence && pk_and_sequence.first
end

Note the last line can return nil if pk_and_sequence is nil, so I take this to mean AR is OK with primary_key returning nil.

Hey Woody--I don't have tons of time to put in to nulldb maintenance. Do you want me to add you as a collaborator and then you can take care of this yourself?  I certainly trust you to write quality code.

I'd be down with that. Thanks!

Cool, I just converted nulldb to an organization and added you to the team.  Feel free to merge your stuff in.
At some point, we should get CI setup on travis as that will help us keep things working.  If you get an itch to work on some other nulldb stuff and want to take care of that, have at it...otherwise I'll try to get to it at some point (like after I'm done with VCR 2.0).

Pushed a fix


FrostKey::mul_tweak

If we ask signers to return s!(r1 + (r2 * b) + lambda * x * c * mul_tweak) in Frost::sign, we can get a signature that is valid for the public key s * mul_tweak * G.
Any multiplicative tweaks would have to be applied before any additive tweaks.
Let me know if you would merge a PR that implements this, thank you!

Hi irriden,
I have been talking about going in a direction with "tweaking" that would tweak the secret shares as well as the public Frostkey. This means FrostKey would no longer "store" tweaks but would mutate the polynomial by the tweaks. In the case of addition this would mean just adding the value to the first coefficient. In the case of multiplication it would be multiplying the entire polynomial by the public scalar. You of course have to do the homomorphic operation to the shares as well. This would allow you to do what you want. What do you think?
What do you actually want this for out of interest.

I am researching multisig lightning. In lightning the revocationpubkey for a specific commitment is derived using a multiplicative, followed by an additive tweak.

Oh ok weird lol -- why? I can put multiplication into the API that I am working on right now. I think it's sound from a security point of view. To be clear you wouldn't need to keep the polynomial in the new API -- you can tweak your secret share if you're a signer. verify_signature_share will be the thing that still needs it.
I'm guessing this is not "xonly" tweaking? The resulting point does not have its y coordinate dropped?
I am about to start working on making it possible to do bip-frost-dkg compatible ops for what it's worth.

See here for details: https://github.com/lightning/bolts/blob/master/03-transactions.md#revocationpubkey-derivation

Thanks. I wonder why that needed to be a multiplication.

verify_signature_share will be the thing that still needs it.

Would you consider having verify_signature_share depend not on the polynomial, but on the public share of the corresponding signer, as in bip-frost-dkg ? Thank you.

Hmmm I certainly could add an API for this -- weirdly enough we do have a verification_share method to get it but nothing to use it :facepalm:. The problem with verification shares is that their value changes when a key is tweaked. How do you handle this right now? Do you multiply the verification shares when you are doing a multiplicative tweak? Or is this being stored in some separate state and being patched up in the verification check. I feel like if we make a verification share based API it will be very error prone because there is yet another thing to keep track off the tweaks on.
Would having a method that allowed you to create the polynomial from the verification shares solve your problem?
I am perhaps not fully understanding where you are getting these verification shares from. It sounds like you are not generating them with schnorr_fun.
It looks like I brought this topic up on the BIP a while a go: https://github.com/BlockstreamResearch/bip-frost-dkg/issues/8
I still think the BIP should also use polynomials in the API FWIW.

Does this mean that secp256kfun could soon be used to build a participant in a bip-frost-dkg session ?

Yes that's the plan in the sense that you'll be able to produce the messages in the protocol but the actual API for the library will be different from the BIP. e.g. we might not use verification shares.


App hangs at splash screen when run on Android

This still persists,  the message says: "device is not defined"; i don't know why it seems to be without plugins in the build remote (aem cloud services) but the config.xml is well configured with all plugins(cordova...device and cordova..media ). Am i  missing something? Any advice?


Add an error when apiVersion is missing

The apiVersion field is generally required by kubernetes objects... it's surprising that this situation wasn't already caught by an error when decoding or getting a mapping.
Refs:

https://issues.redhat.com/browse/ACM-7127

Now that I added useful debugging for the flakey test, it's passed 4 times in a row 😿


[Counting] Suggestion: Give role on incorrect count

Feature Description
Same as the "last user to count" role, but for the last user to get the count wrong

could add in same time a leaderboard but for the most poeple who fail ?


Error while invoking cmake

Tring to run the the project as diescribed brings some erros's

There is no : "project/delta-studio.sln"
Invoking simple cmake command gives error :

c:\dev\my\cpp\delta-studio>cmake  -B ./PrjVisualStudio -G "Visual Studio 17 2022" -A x64
-- Selecting Windows SDK version 10.0.20348.0 to target Windows 10.0.19043.
-- The C compiler identification is MSVC 19.30.30705.0
-- The CXX compiler identification is MSVC 19.30.30705.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.30.30705/bin/Hostx64/x64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.30.30705/bin/Hostx64/x64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Looking for pthread.h
-- Looking for pthread.h - not found
-- Found Threads: TRUE
CMake Error at C:/dev/my/cpp/cmake-3.22.0-rc2-windows-x86_64/share/cmake-3.22/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find SDL2 (missing: SDL2_LIBRARY SDL2_INCLUDE_DIR)
Call Stack (most recent call first):
  C:/dev/my/cpp/cmake-3.22.0-rc2-windows-x86_64/share/cmake-3.22/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
  cmake/FindSDL2.cmake:198 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:52 (find_package)

install SDL2

@meiry
https://github.com/libsdl-org/SDL/releases/tag/release-2.26.5
SDL2-devel-2.26.5-VC.zip
You will need this, and to either install it, or do what I did, and just extract the files somewhere, and point Cmake to the correct areas when it asks for it.


Empty structs when using List Comprehensions

What version of CUE are you using (cue version)?

$ cue version
cue version 0.3.0-alpha1 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?
sample.cue
import (
	"list"
)

sampleStruct: {
	gcp: present:     true
	aws: present:     true
	azure: present:   true
	alibaba: present: true
}

ignoreList: ["alibaba"]

NewSample: [
	for k, v in sampleStruct {
		if !list.Contains(ignoreList, k) {v & {
			location: "india"
			name:     k
		}}},
]

Exported NewSample using:
cue export sample.cue -e NewSample

What did you expect to see?
[
    {
        "present": true,
        "location": "india",
        "name": "gcp"
    },
    {
        "present": true,
        "location": "india",
        "name": "aws"
    },
    {
        "present": true,
        "location": "india",
        "name": "azure"
    },
]

What did you see instead?
[
    {
        "present": true,
        "location": "india",
        "name": "gcp"
    },
    {
        "present": true,
        "location": "india",
        "name": "aws"
    },
    {
        "present": true,
        "location": "india",
        "name": "azure"
    },
    {}
]

I think you want:
import (
	"list"
)

sampleStruct: {
	gcp: present:     true
	aws: present:     true
	azure: present:   true
	alibaba: present: true
}

ignoreList: ["alibaba"]

NewSample: [
	for k, v in sampleStruct if !list.Contains(ignoreList, k) {
		v & {
			location: "india"
			name:     k
		}
	},
]

I don't believe there is an issue here, but please re-open if you disagree.

This issue has been migrated to https://github.com/cue-lang/cue/issues/524.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Swaps canceled due to "missing transaction" when using LWK backend

When a peer that is running LWK tries to broadcast a swap transaction, sometimes it will fail and the cancel message will be missing transaction.
Example logs of this when running LND + LWK + peerswapd:
024/10/29 17:35:42 [DEBUG] [FSM] event:id: 3f9a179cec432d04bc4442d3c5a6b9b95311d99181bafd1a65bc7eb0c1d3aa8b, Event_SwapInSender_OnSwapInRequested on
2024/10/29 17:35:42 [INFO] [Swap:3f9a179cec432d04bc4442d3c5a6b9b95311d99181bafd1a65bc7eb0c1d3aa8b] Start new swap-in: peer: 036e680823a129ac9a2eeb8c43514a5b6cb09ae28e3e698a0e34aaa8865d6cc344
 chanId: 194668:21:0 initiator: 035ca2fe4793a5e789ce846062eb4834f573c060d9200ce77544a29b48a0aa5923 amount 100000001
2024/10/29 17:35:42 [DEBUG] [FSM] event:id: 3f9a179cec432d04bc4442d3c5a6b9b95311d99181bafd1a65bc7eb0c1d3aa8b, Event_ActionSucceeded on State_SwapInSender_CreateSwap
2024/10/29 17:35:42 [DEBUG] [FSM] event:id: 3f9a179cec432d04bc4442d3c5a6b9b95311d99181bafd1a65bc7eb0c1d3aa8b, Event_ActionSucceeded on State_SwapInSender_SendRequest
2024/10/29 17:35:43 [DEBUG] [Messenger] From: 036e680823a129ac9a2eeb8c43514a5b6cb09ae28e3e698a0e34aaa8865d6cc344 got msgtype: a459 for swap: 3f9a179cec432d04bc4442d3c5a6b9b95311d99181bafd1a6
5bc7eb0c1d3aa8b
2024/10/29 17:35:43 [DEBUG] [FSM] event:id: 3f9a179cec432d04bc4442d3c5a6b9b95311d99181bafd1a65bc7eb0c1d3aa8b, Event_SwapInSender_OnAgreementReceived on State_SwapInSender_AwaitAgreement
2024/10/29 17:35:43 [DEBUG] using minimum fee rate of 0.1 sat/vbyte
2024/10/29 17:35:44 [INFO] [FSM] Action failure missing transaction
2024/10/29 17:35:44 [DEBUG] [FSM] event:id: 3f9a179cec432d04bc4442d3c5a6b9b95311d99181bafd1a65bc7eb0c1d3aa8b, Event_ActionFailed on State_SwapInSender_BroadcastOpeningTx
2024/10/29 17:35:44 [DEBUG] [FSM] Canceling because of missing transaction
2024/10/29 17:35:44 [DEBUG] [FSM] event:id: 3f9a179cec432d04bc4442d3c5a6b9b95311d99181bafd1a65bc7eb0c1d3aa8b, Event_ActionSucceeded on State_SendCancel
2024/10/29 17:35:44 [INFO] [Swap:3f9a179cec432d04bc4442d3c5a6b9b95311d99181bafd1a65bc7eb0c1d3aa8b] Swap canceled. Reason: missing transaction

The LWK debug logs don't seem to print any clues, so I suspect it could be an issue with the Electrum server? Either way, we should attempt to broadcast more than one time?

@grubles Ensure that the transaction you're referencing is confirmed on the blockchain. You can use a blockchain explorer like BSC Scan to verify the transaction status.

According to the logs, this error occurred in the State_SwapInSender_BroadcastOpeningTx state, indicating an issue with generating the opening transaction.
This is happening during the send RPC of lwk.
Since it seems to succeed upon retry, I have enhanced the error logging for the send RPC call in the LWK client and introduced a retry mechanism.
Could you please check if it reproduces?
https://github.com/ElementsProject/peerswap/pull/326
If the error persists even with this backoff, we may need to adjust the retry intervals.

Here are the logs when using the latest commit:
2024/11/01 12:57:11 [DEBUG] [FSM] event:id: 4ee04c58b8a037004b487211584102504413dc6de06e933cf17511b2a7a12eab, Event_SwapInSender_OnSwapInRequested on
2024/11/01 12:57:11 [INFO] [Swap:4ee04c58b8a037004b487211584102504413dc6de06e933cf17511b2a7a12eab] Start new swap-in: peer: 036e680823a129ac9a2eeb8c43514a5b6cb09ae28e3e698a0e34aaa8865d6cc344 chanId: 194668:21:0 initiator: 035ca2fe4793a5e789ce846062eb4834f573c060d9200ce77544a29b48a0aa5923 amount 100000001
2024/11/01 12:57:11 [DEBUG] [FSM] event:id: 4ee04c58b8a037004b487211584102504413dc6de06e933cf17511b2a7a12eab, Event_ActionSucceeded on State_SwapInSender_CreateSwap
2024/11/01 12:57:11 [DEBUG] [FSM] event:id: 4ee04c58b8a037004b487211584102504413dc6de06e933cf17511b2a7a12eab, Event_ActionSucceeded on State_SwapInSender_SendRequest
2024/11/01 12:57:12 [DEBUG] [Messenger] From: 036e680823a129ac9a2eeb8c43514a5b6cb09ae28e3e698a0e34aaa8865d6cc344 got msgtype: a459 for swap: 4ee04c58b8a037004b487211584102504413dc6de06e933cf17511b2a7a12eab
2024/11/01 12:57:12 [DEBUG] [FSM] event:id: 4ee04c58b8a037004b487211584102504413dc6de06e933cf17511b2a7a12eab, Event_SwapInSender_OnAgreementReceived on State_SwapInSender_AwaitAgreement
2024/11/01 12:57:13 [INFO] error getting fee: cannot estimate fee for 7 blocks.
2024/11/01 12:57:13 [DEBUG] using minimum fee rate of 0.1 sat/vbyte
2024/11/01 12:57:13 [INFO] [FSM] Action failure failed to get raw transaction: missing transaction
2024/11/01 12:57:13 [DEBUG] [FSM] event:id: 4ee04c58b8a037004b487211584102504413dc6de06e933cf17511b2a7a12eab, Event_ActionFailed on State_SwapInSender_BroadcastOpeningTx
2024/11/01 12:57:13 [DEBUG] [FSM] Canceling because of failed to get raw transaction: missing transaction
2024/11/01 12:57:13 [DEBUG] [FSM] event:id: 4ee04c58b8a037004b487211584102504413dc6de06e933cf17511b2a7a12eab, Event_ActionSucceeded on State_SendCancel
2024/11/01 12:57:13 [INFO] [Swap:4ee04c58b8a037004b487211584102504413dc6de06e933cf17511b2a7a12eab] Swap canceled. Reason: failed to get raw transaction: missing transaction


BUG - Row duplication introduced when History Mode is enabled

Are you a current Fivetran customer?
Yes - Jacob Mulligan, Firefly Health, head of analytics
Describe the bug
I turned on history mode in Fivetran for the User, UserRole, Account, and Opportunity tables, Fivetran then ran a migration on our data in Snowflake which took a few minutes. I then enabled history_mode using the configuration described here. The tests on the salesforce_source package stared failing.
Steps to reproduce
Here is the exact code we have in our dbt_project.yml file:
vars:
  salesforce_source:
    salesforce_database: raw
    salesforce_schema: salesforce

    using_account_history_mode_active_records: true
    using_opportunity_history_mode_active_records: true
    using_user_role_history_mode_active_records: true
    using_user_history_mode_active_records: true

At first tests passed and we merged this into master. Today the following tests broke:
Completed with 3 errors and 0 warnings:

Failure in test unique_salesforce__opportunity_enhanced_opportunity_id (models/salesforce.yml)
  Got 82 results, configured to fail if != 0

  compiled SQL at target/compiled/salesforce/models/salesforce.yml/schema_test/unique_salesforce__opportunity_enhanced_opportunity_id.sql

Failure in test unique_salesforce__owner_performance_owner_id (models/salesforce.yml)
  Got 7 results, configured to fail if != 0

  compiled SQL at target/compiled/salesforce/models/salesforce.yml/schema_test/unique_salesforce__owner_performance_owner_id.sql

Failure in test unique_stg_salesforce__user_user_id (models/stg_tables.yml)
  Got 7 results, configured to fail if != 0

I was able to trace these issues back to the stg_salesforce__user_tmp.sql model which contains row duplication. The issue seems to be that a User was updated in SalesForce and when the update was copied over to Snowflake the old record did not have _fivetran_active set to false, instead _fivetran_active was still null. There are now two rows in the users_tmp model for one user, here's what this Looks like in Snowflake:

Because _fivetran_active is null for the first row, the coalesce() in the stg_salesforce__user_tmp.sql model returns both records for that user..
select *
from raw.salesforce.user

where coalesce(_fivetran_active, true)

Expected behavior
I would expect to be able to turn on History Mode for the User, UserRole, Account, and Opportunity tables and turn on History Mode for those models in this package and have the package only select the most record record for each of these tables.
Project variables configuration
I shared this above.
Package Version

packages:
  - package: dbt-labs/dbt_utils
    version: 0.7.1

  - package: dbt-labs/segment
    version: 0.6.0

  - package: fivetran/fivetran_log
    version: 0.4.0

  - package: fivetran/salesforce_source
    version: 0.3.1

  - package: fivetran/salesforce
    version: 0.4.0

  - package: fivetran/jira
    version: 0.4.0

  - package: calogica/dbt_date
    version: 0.4.0

Warehouse

[ ] BigQuery
[ ] Redshift
[x] Snowflake
[ ] Postgres
[ ] Databricks
[ ] Other (provide details below)

Please indicate the level of urgency
Medium urgency. This is causing production tests to fail, we rely on this package for reporting which is now slightly incorrect (miscounting by 1 opportunity) due to this issue, we can't just remove the package because we rely on it for reporting, and lastly the failing test makes all deploys of production look broken 😦
Are you interested in contributing to this package?

[ ] Yes, I can do this and open a PR for your review.
[x] Possibly, but I'm not quite sure how to do this. I'd be happy to do a live coding session with someone to get this fixed.
[ ] No, I'd prefer if someone else fixed this.  I don't have the time and/or don't know what the root cause of the problem is.

A (potentially temporary) fix: drop the table in Snowflake, resync the table from scratch, then rebuild the salesforce models.
I ran this in Snowflake:
drop table raw.salesforce.user;

Then resynced the table in Fivetran:

Then reran the SF job and tests:
dbt run -m +salesforce --full-refresh
dbt test -m +salesforce

The tests are now passing. I say "potentially temporary" above because I don't know if tests will start failing again after our night sync tomorrow or Tuesday as more changes are made to users/opportunity/account/userrole which might create more data formed in a way that breaks the package tests again. I'll let you know if so.

@jcbmllgn the product manager was on pto, and then I was on pto and now we are both back and I am coordinating to confirm if the resolution you mentioned is the intended method users should take. I will let you know once I hear back.
In the meantime, have you experienced any other failing tests?

Hi @jcbmllgn just got confirmation from our product team that this is an open bug which they are working on. Currently they have seen migrations experience the same behavior you noted above.

Thanks for the update Joe.
I experienced this issue with all four tables that I turned on history mode for, dropping the tables and resyncing fixed the issue for me. I don't need to turn on history mode for other tables right now, so I don't think I'll run into this issue again before a fix is released.

Sounds great, since this issue is related to the history mode feature of the connector I would recommend opening a request within our Fivetran Support Portal so you can chat with our product/support team and be notified once the fix is applied to the connector.

Hi @jcbmllgn I just wanted to reach out to see if you were able to create a support issue and see the issue resolved?
If so, I will close this issue. Thanks!

@fivetran-joemarkiewicz sorry, I missed you message here from the fall! Want me to still open a support ticket?
I have not checked to see if the underlying issue has been fixed.. Do you happen to know it has been resolved? Thanks!

Hi @jcbmllgn yes if you are still experiencing this issue then opening a support ticket should be the path forward from here as we are unable to address this issue within the dbt package.
I am unsure if the initial issue you raised has been resolved, but the members of our support and product teams should be able to help moving forward within a support ticket if you are still experiencing the issue.
As this is not something we will be able to address within the package, I will close the issue. Please feel free to reopen the issue if you feel there is still more to discuss related to the dbt package. Thanks!


Failure while waiting for the host to be up

Hello contributors, first of all i would like to thank you for you hard work providing these Ansible roles.
To the issue at hand, i am trying to install oVirt engine in HA mode using this playbook.
Below are the variables i am using:
he_bridge_if: 'p3p1'
he_fqdn: 'engine.example.com'
he_vm_ip_addr: '<IP_ADDRESS>'
he_vm_ip_prefix: '24'
he_gateway: '<IP_ADDRESS>'
he_dns_addr:
  - '<IP_ADDRESS>'
  - '<IP_ADDRESS>'
  - '<IP_ADDRESS>'
he_vm_etc_hosts: true
he_vm_mac_addr: '00:a5:3f:66:ba:12'
he_domain_type: 'nfs'
he_storage_domain_path: "/var/nfs"
he_storage_domain_addr: 'nfs.example.com'
he_mem_size_MB: '8192'
he_vcpus: '2'
he_ansible_host_name: 'node-01.example.com'
he_host_name: 'node-01.example.com'
he_network_test: 'ping'

So currently i am running the playbook from my workstation PC targeting 'node-01.example.com'.
The role works fine until it reaches the point after it has added the host to the engine. Then it goes through the retries but it cannot get the right response back from the engine with the below ansible output:
TASK [ovirt.hosted_engine_setup : Wait for the host to be up] *******************************************************************************************************************************
task path: /Users/user/.ansible/roles/ovirt.hosted_engine_setup/tasks/bootstrap_local_vm/05_add_host.yml:111
fatal: [node-01.example.com]: FAILED! => {
    "ansible_facts": {
        "ovirt_hosts": []
    },
    "attempts": 120,
    "changed": false,
    "invocation": {
        "module_args": {
            "all_content": false,
            "fetch_nested": false,
            "nested_attributes": [],
            "pattern": "name={'changed': False, 'skipped': True, 'skip_reason': 'Conditional result was False'}"
        }
    }
}

Log from the ovirt-engine:
2019-05-30 10:28:20,158+01 INFO  [org.ovirt.engine.core.bll.SearchQuery] (default task-1) [6f91508e-4f5e-4a5d-b3a1-23f96af7e24b] ResourceManager::searchBusinessObjects - erroneous search text - ''Hosts : name={'changed': False, 'skipped': True, 'skip_reason': 'Conditional result was False'}'' error - ''Hosts : name=$'c$hanged': False, 'skipped': True, 'skip_reason': 'Conditional result was False'}''
I am running the following software:

Ansible 2.7.8 on Mac OS
Latest Ansible roles (downloaded from ansible-galaxy)
oVirt node 4.4.0 for the host OS

Thank you

Thanks for the report,
here we clearly have
pattern: name={{ he_host_name }}
and in your variable file you have:
he_host_name: 'node-01.example.com'
so he_host_name got probably overwrote by the result of a previous failure.
Can you please checks you log for that or send them to me?
I'd suggest to re-execute with -vvv

Just to make it more clear,
the issue is that the code passed:
            "pattern": "name={'changed': False, 'skipped': True, 'skip_reason': 'Conditional result was False'}"
instead of
             "pattern": "name='node-01.example.com'"
as expected

Another thing, oVirt 4.3.z is the stable version, oVirt 4.4 is the development branch

Hi @tiraboschi, thank you for your suggestions. I have located the issue, here is the Ansible log where the issue occurs:
2019-05-30 11:48:53,122 p=5769 u=user |  TASK [ovirt.hosted_engine_setup : Get full hostname] ****************************************************************************************************************************************
2019-05-30 11:48:53,123 p=5769 u=user |  task path: /Users/user/.ansible/roles/ovirt.hosted_engine_setup/tasks/fetch_host_ip.yml:2
2019-05-30 11:48:53,123 p=5769 u=user |  changed: [node-01.example.com] => {
    "changed": true,
    "cmd": [
        "hostname",
        "-f"
    ],
    "delta": "0:00:00.025521",
    "end": "2019-05-30 11:48:52.608310",
    "invocation": {
        "module_args": {
            "_raw_params": "hostname -f",
            "_uses_shell": false,
            "argv": null,
            "chdir": null,
            "creates": null,
            "executable": null,
            "removes": null,
            "stdin": null,
            "warn": true
        }
    },
    "rc": 0,
    "start": "2019-05-30 11:48:52.582789",
    "stderr": "",
    "stderr_lines": [],
    "stdout": "node-01.example.com",
    "stdout_lines": [
        "node-01.example.com"
    ]
}
2019-05-30 11:48:53,227 p=5769 u=user |  TASK [ovirt.hosted_engine_setup : debug] ****************************************************************************************************************************************************
2019-05-30 11:48:53,227 p=5769 u=user |  task path: /Users/user/.ansible/roles/ovirt.hosted_engine_setup/tasks/fetch_host_ip.yml:11
2019-05-30 11:48:53,228 p=5769 u=user |  ok: [node-01.example.com] => {
    "he_host_name": {
        "changed": false,
        "skip_reason": "Conditional result was False",
        "skipped": true
    }
}

At this point the he_host_name variable is set to the skipped result coming from ansible. To quickly fix the issue i have commented our this line: https://github.com/oVirt/ovirt-ansible-hosted-engine-setup/blob/38a3f86e65855df7ef0220ef28202fcfb9a80db6/tasks/fetch_host_ip.yml#L9 and then the play continues successfully.


Add optional key/value metadata to producers.

https://github.com/apache/incubator-pulsar/issues/980
Motivation
Add the ability to attach metadata (key/value strings) to a producer. This provides more context about a producer and is useful for debugging, ui interaction, and/or metrics aggregation.
Modifications

protobuf changes
metadata key/value strings to the producer

@cckellogg is the change for consumers coming in a different PR?

@merlimat yes a follow up pr for the consumers is coming after this one.


4ChPror3 doesn't work after 2023.12.3

Sonoff integration does not work with sonoff 4ChPror3 after update 2023.12.3. I have tried every update through 2024.1.6... Reverted back to 12.3 for now

Same issue with Sonoff 4CH

Same issue with the S40TPB Smart Plugs!  LOL wife not happy! :D   Open to suggestions!

Manually added the latest version of SonoffLAN and it works again!

https://github.com/AlexxIT/SonoffLAN#issues


[cli][server] Refactoring code for sarif support

Closes #3451 and the first part of #1797.

The basic idea behind these changes is that in this patch we created a
general Report class which holds information about bugs what are needed
by the CodeChecker CLI and server. We introduced helper functions in this patch
to convert analyzer result files (for now only plist) to Report objects.
All the converters (html, gerrit, codeclimate, json, etc.) and report hash
generation modules will work on these Report objects.
This patch also tried to remove duplication between the different command
line subcommands (parse, diff, store etc.).

The json output of the CodeChecker parse command was not stable enough and the structure was very similar to the plist structure. Our plan is to support reading/parsing/storing of multiple analyzer output types not only plist but for example sarif format as well (http://docs.oasis-open.org/sarif/sarif/v2.0/csprd01/sarif-v2.0-csprd01.html).
Yes, it's our mistake that we forgot to mention it in the release notes. So sorry for that :/


Where can I find the maven repository for this?

Hi, This port looks really promising. How do I add it via maven or gradle? Could you point me to the repo where you upload the artifacts?

O sorry for my late reply. Can u try https://maven.pkg.github.com/dcxp/opentelemetry-kotlin as maven repo.


Deploying a stream looks for zookeeper

Reading through the documentation at http://docs.spring.io/spring-cloud-dataflow-server-cloudfoundry/docs/1.7.4.RELEASE/reference/htmlsingle/#_zookeeper it states that zookeeper isn't used in the new architecture, however when I deploy a stream through my SCDF deployed on OSS CF, in the logs for the stream components, it is stating
INFO 20 --- [           main] org.apache.zookeeper.ZooKeeper           : Initiating client connection, connectString=localhost:2181
and the stream never succeeds to deploy ( as I obviously don't have anything deployed locally to the stream app on that port)
Also I can't find any reference to this in the getting started guide for deploying SCDF to cloud foundry  - am I missing something here and can someone point me in the correct direction?
Thanks.

Never mind - hadn't appreciated it was the actual source / sink apps which required kafka


Powershell debugging fails

Issue Type: Bug
Running a small snippet of powershell and pounding the F5 button, i'm getting "Debug adapter process has terminated unexpectedly" and my code wont run anymore.
VS Code version: Code 1.22.2 (3aeede733d9a3098f7b4bdc1f66b63b0f48c1ef9, 2018-04-12T16:38:45.278Z)
OS version: Windows_NT x64 10.0.15063

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-5600U CPU @ 2.60GHz (4 x 2594)

Memory (System)
15.89GB (4.88GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

code-runner
for
0.9.3

terminal
for
0.0.10

compareit
in4
0.0.2

puppet-vscode
jpo
0.10.0

mssql
ms-
1.3.1

PowerShell
ms-
1.6.0

Been trying to replicate this since creating the ticket. I cant seem to trigger the behavior again.
Lets assume it was a one-time deal


Performing apple attestation some times throws a python exception

During the apple attestation process there seem to be some string format errors. These should be corrected so that they do not throw exceptions. They are probably doing string substitution on a data type that is not properly being marshalled into a string.
INFO:controller:Run POST /topic/drpc_response/
INFO:controller:handle_attestation_challenge
INFO:controller:testing apple challenge
INFO:apple:Decoding attestation object...
INFO:apple:Apple Attestation step 1...
--- Logging error ---
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/logging/__init__.py", line 1083, in emit
    msg = self.format(record)
  File "/usr/local/lib/python3.9/logging/__init__.py", line 927, in format
    return fmt.format(record)
  File "/usr/local/lib/python3.9/logging/__init__.py", line 663, in format
    record.message = record.getMessage()
  File "/usr/local/lib/python3.9/logging/__init__.py", line 367, in getMessage
    msg = msg % self.args
TypeError: not all arguments converted during string formatting
Call stack:
  File "/usr/local/bin/gunicorn", line 8, in <module>
    sys.exit(run())
  File "/usr/local/lib/python3.9/site-packages/gunicorn/app/wsgiapp.py", line 67, in run
    WSGIApplication("%(prog)s [OPTIONS] [APP_MODULE]").run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/app/base.py", line 236, in run
    super().run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/app/base.py", line 72, in run
    Arbiter(self).run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 202, in run
    self.manage_workers()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 571, in manage_workers
    self.spawn_workers()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 642, in spawn_workers
    self.spawn_worker()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 609, in spawn_worker
    worker.init_process()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/base.py", line 142, in init_process
    self.run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 126, in run
    self.run_for_one(timeout)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 70, in run_for_one
    self.accept(listener)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 32, in accept
    self.handle(listener, client, addr)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 135, in handle
    self.handle_request(listener, req, client, addr)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 178, in handle_request
    respiter = self.wsgi(environ, resp.start_response)
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1488, in __call__
    return self.wsgi_app(environ, start_response)
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1463, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 870, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 855, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)  # type: ignore[no-any-return]
  File "/opt/controller/controller.py", line 191, in drpc_response
    handle_drpc_challenge_response(resp, connection_id)
  File "/opt/controller/controller.py", line 127, in handle_drpc_challenge_response
    is_valid_challenge = verify_attestation_statement(
  File "/opt/controller/apple.py", line 210, in verify_attestation_statement
    verify_x5c_status = verify_x5c_certificates(apple_attestation_object)
  File "/opt/controller/apple.py", line 83, in verify_x5c_certificates
    logger.info("root_certificate", root_certificate.subject)
Message: 'root_certificate'
Arguments: (<Name(CN=Apple App Attestation Root CA,O=Apple Inc.,ST=California)>,)
--- Logging error ---
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/logging/__init__.py", line 1083, in emit
    msg = self.format(record)
  File "/usr/local/lib/python3.9/logging/__init__.py", line 927, in format
    return fmt.format(record)
  File "/usr/local/lib/python3.9/logging/__init__.py", line 663, in format
    record.message = record.getMessage()
  File "/usr/local/lib/python3.9/logging/__init__.py", line 367, in getMessage
    msg = msg % self.args
TypeError: not all arguments converted during string formatting
Call stack:
  File "/usr/local/bin/gunicorn", line 8, in <module>
    sys.exit(run())
  File "/usr/local/lib/python3.9/site-packages/gunicorn/app/wsgiapp.py", line 67, in run
    WSGIApplication("%(prog)s [OPTIONS] [APP_MODULE]").run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/app/base.py", line 236, in run
    super().run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/app/base.py", line 72, in run
    Arbiter(self).run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 202, in run
    self.manage_workers()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 571, in manage_workers
    self.spawn_workers()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 642, in spawn_workers
    self.spawn_worker()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 609, in spawn_worker
    worker.init_process()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/base.py", line 142, in init_process
    self.run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 126, in run
    self.run_for_one(timeout)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 70, in run_for_one
    self.accept(listener)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 32, in accept
    self.handle(listener, client, addr)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 135, in handle
    self.handle_request(listener, req, client, addr)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 178, in handle_request
    respiter = self.wsgi(environ, resp.start_response)
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1488, in __call__
    return self.wsgi_app(environ, start_response)
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1463, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 870, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 855, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)  # type: ignore[no-any-return]
  File "/opt/controller/controller.py", line 191, in drpc_response
    handle_drpc_challenge_response(resp, connection_id)
  File "/opt/controller/controller.py", line 127, in handle_drpc_challenge_response
    is_valid_challenge = verify_attestation_statement(
  File "/opt/controller/apple.py", line 210, in verify_attestation_statement
    verify_x5c_status = verify_x5c_certificates(apple_attestation_object)
  File "/opt/controller/apple.py", line 84, in verify_x5c_certificates
    logger.info("credential_certificate", credential_certificate.subject)
Message: 'credential_certificate'
Arguments: (<Name(CN=c316a3ddbfee9ef821133a3c8ee4aa2f5990f4a990cdbe463e566a32981de9a3,OU=AAA Certification,O=Apple Inc.,ST=California)>,)
--- Logging error ---
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/logging/__init__.py", line 1083, in emit
    msg = self.format(record)
  File "/usr/local/lib/python3.9/logging/__init__.py", line 927, in format
    return fmt.format(record)
  File "/usr/local/lib/python3.9/logging/__init__.py", line 663, in format
    record.message = record.getMessage()
  File "/usr/local/lib/python3.9/logging/__init__.py", line 367, in getMessage
    msg = msg % self.args
TypeError: not all arguments converted during string formatting
Call stack:
  File "/usr/local/bin/gunicorn", line 8, in <module>
    sys.exit(run())
  File "/usr/local/lib/python3.9/site-packages/gunicorn/app/wsgiapp.py", line 67, in run
    WSGIApplication("%(prog)s [OPTIONS] [APP_MODULE]").run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/app/base.py", line 236, in run
    super().run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/app/base.py", line 72, in run
    Arbiter(self).run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 202, in run
    self.manage_workers()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 571, in manage_workers
    self.spawn_workers()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 642, in spawn_workers
    self.spawn_worker()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/arbiter.py", line 609, in spawn_worker
    worker.init_process()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/base.py", line 142, in init_process
    self.run()
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 126, in run
    self.run_for_one(timeout)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 70, in run_for_one
    self.accept(listener)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 32, in accept
    self.handle(listener, client, addr)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 135, in handle
    self.handle_request(listener, req, client, addr)
  File "/usr/local/lib/python3.9/site-packages/gunicorn/workers/sync.py", line 178, in handle_request
    respiter = self.wsgi(environ, resp.start_response)
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1488, in __call__
    return self.wsgi_app(environ, start_response)
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 1463, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 870, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.9/site-packages/flask/app.py", line 855, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)  # type: ignore[no-any-return]
  File "/opt/controller/controller.py", line 191, in drpc_response
    handle_drpc_challenge_response(resp, connection_id)
  File "/opt/controller/controller.py", line 127, in handle_drpc_challenge_response
    is_valid_challenge = verify_attestation_statement(
  File "/opt/controller/apple.py", line 210, in verify_attestation_statement
    verify_x5c_status = verify_x5c_certificates(apple_attestation_object)
  File "/opt/controller/apple.py", line 85, in verify_x5c_certificates
    logger.info("intermediate_certificate", intermediate_certificate.subject)
Message: 'intermediate_certificate'
Arguments: (<Name(CN=Apple App Attestation CA 1,O=Apple Inc.,ST=California)>,)
INFO:apple:The child certificate was issued by the parent certificate.
INFO:apple:The certificates are signed by the ROOT certificate.
INFO:apple:Apple Attestation step 2...
INFO:apple:Apple Attestation step 3...
INFO:apple:Apple Attestation step 4...
INFO:apple:Apple Attestation step 5...
INFO:apple:Apple Attestation step 6...
INFO:apple:Apple Attestation step 7...
INFO:apple:Apple Attestation step 8...
INFO:apple:Apple Attestation step 9...
INFO:apple:Successful apple attestation
INFO:controller:valid challenge
INFO:traction:issue_attestation_credential
INFO:traction:Sending offer to 86c47347-13ad-45c2-95b0-f037b102d4fc, offer = {'auto_issue': True, 'auto_remove': True, 'trace': True, 'cred_def_id': 'RycQpZ9b4NaXuT5ZGjXkUE:3:CL:120:bcwallet', 'credential_preview': {'@type': 'issue-credential/1.0/credential-preview', 'attributes': [{'name': 'operating_system', 'value': 'iOS'}, {'name': 'operating_system_version', 'value': '17.4.1'}, {'name': 'validation_method', 'value': 'apple:app-attest'}, {'name': 'app_id', 'value': 'ca.bc.gov.BCWallet'}, {'name': 'app_vendor', 'value': 'Government of British Columbia'}, {'name': 'issue_date_dateint', 'value': '20240402'}, {'name': 'app_version', 'value': '1.0.16-1636'}]}, 'comment': '', 'connection_id': '86c47347-13ad-45c2-95b0-f037b102d4fc'}
INFO:traction:Offer sent successfully
INFO:controller:Run GET /ping/
INFO:controller:Run GET /ping/
INFO:controller:Run GET /ping/

I think the problem is apple.py) L83-85. The .subject cannot be printed in logger.info the way it is. The subject property of a certificate in Python is an instance of the Name class, which is (I think) a sequence of NameAttribute objects. To print it properly, you can convert it to a string using the str() function.
logger.info("root_certificate: %s", str(root_certificate.subject))
logger.info("credential_certificate: %s", str(credential_certificate.subject))
logger.info("intermediate_certificate: %s", str(intermediate_certificate.subject))

Texted. Fix confirmed.


Autotests: #5046 - Tests for unresolved IDT monomers

How the feature works? / How did you fix the issue?
(Screenshots, videos, or GIFs, if applicable)
Check list

[ ] unit-tests written
[x] e2e-tests written
[ ] documentation updated
[x] PR name follows the pattern #1234 – issue name
[x] branch name doesn't contain '#'
[x] PR is linked with the issue
[x] base branch (master or release/xx) is correct
[ ] task status changed to "Code review"
[ ] reviewers are notified about the pull request

added fail for test due to the bug https://github.com/epam/Indigo/issues/2043


Install fails while following set up video: ValueError: Encountered text corresponding to disallowed special token '<|endoftext|>'.

Traceback (most recent call last):
  File "/Users/.../.pyenv/versions/3.11.0/bin/mentat", line 8, in <module>
    sys.exit(run_cli())
             ^^^^^^^^^
  File "/Users/.../.pyenv/versions/3.11.0/lib/python3.11/site-packages/mentat/app.py", line 24, in run_cli
    run(paths)
  File "/Users/.../.pyenv/versions/3.11.0/lib/python3.11/site-packages/mentat/app.py", line 35, in run
    loop(paths, cost_tracker)
  File "/Users/.../.pyenv/versions/3.11.0/lib/python3.11/site-packages/mentat/app.py", line 48, in loop
    tokens = count_tokens(code_file_manager.get_code_message())
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/.../.pyenv/versions/3.11.0/lib/python3.11/site-packages/mentat/llm_api.py", line 57, in count_tokens
    return len(tiktoken.encoding_for_model("gpt-4").encode(message))
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/.../.pyenv/versions/3.11.0/lib/python3.11/site-packages/tiktoken/core.py", line 117, in encode
    raise_disallowed_special_token(match.group())
  File "/Users/.../.pyenv/versions/3.11.0/lib/python3.11/site-packages/tiktoken/core.py", line 351, in raise_disallowed_special_token
    raise ValueError(
ValueError: Encountered text corresponding to disallowed special token '<|endoftext|>'.
If you want this text to be encoded as a special token, pass it to `allowed_special`, e.g. `allowed_special={'<|endoftext|>', ...}`.
If you want this text to be encoded as normal text, disable the check for this token by passing `disallowed_special=(enc.special_tokens_set - {'<|endoftext|>'})`.
To disable this check for all special tokens, pass `disallowed_special=()`.

I guess the files you were running it on contained <|endoftext|> somewhere? It seems we need to update our tiktoken call to encode special tokens as normal text - all we are using it for is counting tokens so this shouldn't cause issues

I don't see <|endoftext|> in any of my files?

I don't see <|endoftext|> in any of my files

huh, this is confusing. If it's possible could you figure out the type of file that's causing this for you? i.e. run with different subsets of your files to see which ones cause it?

Im personally just following your video tutorial line for line and getting
this. It’s an empty new_project git initialized directory. Nothing special.
On Mon, Jul 31, 2023 at 3:07 PM biobootloader @.***>
wrote:

I don't see <|endoftext|> in any of my files
huh, this is confusing. If it's possible could you figure out the type of
file that's causing this for you? i.e. run with different subsets of your
files to see which ones cause it?
—
Reply to this email directly, view it on GitHub
https://github.com/biobootloader/mentat/issues/34#issuecomment-1658980063,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAENGGHNOLQD77XOWRW5XXTXS77AFANCNFSM6AAAAAA25JRRR4
.
You are receiving this because you authored the thread.Message ID:
@.***>

I wasn't able to replicate this crash myself, but I've just pushed a fix: https://github.com/biobootloader/mentat/pull/39
That PR is merged and I put new code on PYPI. If one of you could test if it solved it for you that'd be great!

I think we're in business, got the official prompt What can I do for you?


Allow UdfSchema chaining

Description
UdfSchema takes an optional next_schema argument. After applying a schema's UDFs, it will apply the next schema's UDFs to the current phase's output. udf_schema() takes an optional list of schema names and constructs the corresponding chain of UdfSchema.

[ ] I have reviewed the Guidelines for Contributing and the Code of Conduct.

Don't do this :)


Remove catalog file and up version number

Catalog file is a relic from older days when it was used a fix for a Windows component.  It's no longer adding any value and can causes problems with installations.

@alerickson To me, it looks like, PowerShellGet rather needs to correctly publish the catalog file so that things like that do not happen:
https://twitter.com/Bu5yGiraffe/status/1250956672127963139/photo/1


Inventory loss

Sometimes player reported inventory loss. Even after having logged in and out several times and even after a complete reboot. Checking in players' NBT, the inventory data get erased.

Check out #146


Update dependencies

As mentioned in #4, outdated dependencies were causing issues with the new compiler version and latest prelude.
I did my best to update everything and get the tests passing, let me know if you see anything wacky!

Closed by #9


No way to add transactionID with<EMAIL_ADDRESS>

I am running version 2.1.1 of this SDK and am looking to send transactionID along with my sendCommerceEvent call. I'm not seeing any documentation (event in the newest version of the SDK - 2.1.1) which shows that this is possible and I have tried adding it to the metadata parameter of the sendCommerceEvent call and it doesn't come through as a transaction ID.
It looks like there has been some work in master to be able to send a transaction ID but with the current SDK version. So this is technically part issue with the SDK, part question on when this feature will be released.
Thanks!

The sendCommerceEvent method is deprecated in 2.2.0, which is in the master branch. Here's the new API: https://github.com/BranchMetrics/react-native-branch-deep-linking#tracking-user-actions-and-events.
Do something like:
import { BranchEvent } from 'react-native-branch'

const event = new BranchEvent(BranchEvent.Purchase, buo, { transactionID: 'abc', price: 20.00 }).logEvent()

Or on a single BUO:
buo.logEvent(BranchEvent.Purchase, { transactionID: 'abc', price: 20.00 })

This will be released soon, probably by the end of the week. What's in the master branch is pretty solid, and you should be able to work with it. You'll see some native deprecation warnings. There is also a new API for the Branch Universal Object that will be part of this release. That should be in the master branch by tomorrow.

Great, thanks @jdee. I eagerly await its release. :)

Hey @jdee, I don't mean to be a pain, but do you have a new estimate when this may be released. Thanks!

2.2.0 has been released with the support you need. Please let us know if you have any further issues.


support more Xcode version

to support more Xcode version we should introduce a new option (can be a dropdown or a text input) where options like these could be selected: xcode-6, xcode-7, xcode-7-beta
To keep compatibility with the previous system we could store "alias maps" as environments, pre-baked into the VM/env if needed. Ex if -stable is specified it would check the BITRISE_XCODE_CHANNEL_PATH_STABLE env for a path - which would (atm) refer to xcode-6. Same for -latest and for -beta.

we should also release a public blog post with specific dates, how long we plan to support Xcode 6

we should NOT create a new dropdown! The new options should be integrated into the current dropdown/value_options !

The version-mapping branch now implements this logic, with version-map files. Related Pull Request: https://github.com/bitrise-io/steps-select-xcode-version/pull/3

Done, available in v2.1.0


Fix CB_FILECONTENTS_REQUEST message decoding.

Do not require optional fields to be present.

@akallabeth nice finding! That one must have slipped through while the channel was hardened.


Since introduction of git 2.38 repo archive tar ball checksums are inconsistent with older releases

Description
Not entirely sure if this is a bug, a feature request or just an attempt to raise awareness of a potential issue to the community (I didn't see this mentioned in any other issue).
In short, the issue is that newer releases of  Gitea do not produce tar.gz repo archives that are checksum-stable with older Gitea releases (dating back to v1.17.2). For most clients this may not be an issue, but for us, who use Gitea as an important part of our supply-chain it has dire consequences.
Details
It seems like somewhere between Gitea v1.17.2 and v1.19.3 an upgrade of the git command line tool in the Dockerfile  was made to 2.38 (v1.19.3 appears to use 2.38.5). This brought with it a behavioral change in the git archive command, which
changed its default behavior from using the gzip(1) command to using an internal zlib library (see commit).
Since the GetArchive API call directly uses git archive under the hood, it means that the tar.gz repository archives produced by Gitea are no longer checksum-stable. This may not have been an explicit promise of the Gitea API,
but it is something that we relied upon.
Notably this issue was also faced by GitHub, who first reacted by letting clients know that this was never officially guaranteed by the API. Given the community outrage that followed and GitHub's important position in supply chains and supply chain tooling, GitHub soon rolled back the change (some more of this discussion is covered in this discussion)
On our end, to preserve the old git archive behavior, we needed to explicitly configure the git client used by Gitea like so:
git config --global tar.tgz.command 'gzip -cn'
git config --global tar.tar.gz.command 'gzip -cn'

Discussion
If one relies on Gitea to produce checksum-stable repo archives, newer versions of Gitea will not be consistent with older releases. Where do we go from here?
At a minimum, I think this needs to be documented, although I would prefer to see something more substantial.
If Gitea wants to be a strong player in a world increasingly concerned with supply chain attacks and the likes, it might be good to actually make promises regarding checksum stable archives. Adding a few tests would be an easy way to prevent unintended changes to produced archives. Changing the default back to the old behavior (of using gzip(1)) like GitHub did could also be an option.
Although a fix for this particular issue was outlined above, it does requires some "hackery" to get in place. One could perhaps consider exposing more of the git configuration through Gitea's own configuration "API"?
Gitea Version
v1.19.3
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Linux
How are you running Gitea?
Helm chart
Database
PostgreSQL

On our end, to preserve the old git archive behavior, we needed to explicitly configure the git client used by Gitea like so:
git config --global tar.tgz.command 'gzip -cn'
git config --global tar.tar.gz.command 'gzip -cn'

Luckily, Gitea already supports a mechanism of changing git internal commands:
Add the following to your app.ini:
[git.config]
tar.tgz.command = gzip -cn
tar.tar.gz.command = gzip -cn

The only problem is we cannot just set that as default as not everyone will have gzip installed.

Luckily, Gitea already supports a mechanism for changing git internal behavior.

Ah, I didn't notice this section of the documentation. That is useful, thanks.

Per suggestion in https://gitea.com/gitea/tea/issues/570, adding a note in here about the checksum mismatch that we have seen in the homebrew side, https://github.com/Homebrew/homebrew-core/pull/139430

So I think this could be closed as it's related upstream.
ref 4f4be00d302bc52d0d9d5a3d4738bb525066c710


integrations: Update links for various events in GitLab Integration.

This PR updates the fixtures for events related to notes in GitLab Integration. Additionally, some changes have been made to the URLs of various events.
URLs of some events were not found in the payload and as of now can only be created by using different fields in the payload. For those kinds of events, the URL format has been changed to conform with the new one.
Fixes #25643.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@zulipbot add "buddy review"

@zulipbot add "mentor review"

Merged, thanks @sbansal1999!


DOC: Clean up quickstart, fix #147

[x] suppress output of dask progress bar in SpectMaker
[x] fix matplotlib.cm ImportError

Had to put the tag in the right cell, and realize that get_cmap now lives right in the plt namespace, not in cm


Update ApiPort to v1.1.0

We haven't updated the version of ApiPort even though now it supports breaking change information as well as portability analysis.  This bumps the version to 1.1.0, according to SemVer 2.0.

LGTM

:+1:
Aside, according to semver, unless you reach 1.0 i.e. (0.x.x), you can make as many breaking changes as you want. Once you reach 1.0, then all semver rules are applied. So starting off with v0.1.0 is a wise idea. To wit, node.js latest version is v0.12.x.

Good to know - although we started off with 1.0 a year ago :( next project we'll have to keep that in mind!