IMA SDK time bug, clickable bug, playback back with some VAST

In the latest version of the plugin, I see the following problems when working with some VAST

Time is displayed as -1: -1
Play / pause does not work. A click on the button runs, no errors are displayed in the console, but the ad playback does not pause
The clickable link is not loaded immediately, but after some time: in the example, click on the player and, as soon as the advertisement starts, move the mouse over the player. At first the cursor will be an arrow. then after a while becomes a pointer

Example here http://jsfiddle.net/3f72qmu6/3/

Hello @Barahten ,
It looks like there are a few issues in your jsfiddle implementation.
I would look to the following code simplified from the plugin's simple example to resolve your issues.
var player = videojs('content_video');

var options = {
  id: 'content_video',
  adTagUrl: 'http://pubads.g.doubleclick.net/gampad/ads?sz=640x480&' +
      'iu=/124319096/external/ad_rule_samples&ciu_szs=300x250&ad_rule=1&' +
      'impl=s&gdfp_req=1&env=vp&output=xml_vmap1&unviewed_position_start=1&' +
      'cust_params=sample_ar%3Dpremidpostpod%26deployment%3Dgmf-js&cmsid=496&' +
      'vid=short_onecue&correlator='
};

player.ima(options);

// Initialize the ad container when the video player is clicked, but only the first time it's clicked.
var initAdDisplayContainer = function() {
  player.ima.initializeAdDisplayContainer();
  wrapperDiv.removeEventListener('click', initAdDisplayContainer);
}

var wrapperDiv = document.getElementById('content_video');
wrapperDiv.addEventListener('click', initAdDisplayContainer);

Please let me know if you have any other questions.

Hello, @Kiro705
Yes, it seems to have helped. Thank

Hey. I decided not to create a new Issue, I will ask here. Is it possible to get CuePoint if the player is set to Autoplay?
I've updated the example. If the player is set to Autoplay, then advertising markers are not displayed until you click on any player control. Run the example and do not click on any player control, wait until the ad finishes playing. Then, after the ad ends, the main video will start playing. If you click on a button, for example, Pause, then ad markers will appear on the timeline.
This example works fine with manually starting the player, but not with Autoplay. Are there any ways to fix this problem? Thanks in advance
if Autoplay doesn't work, start the player manually. Then run the example again (Run button on top left)
Example http://jsfiddle.net/gtc30q74/

Hello @Barahten ,
The reason the markers are not displayed, is that the initAdDisplayContainer function is only being called when the adUi is clicked. I would recommend moving the initAdDisplayContainer function to an initiation step, so it is called every time and not just on clicks.

Hello @Barahten ,
The reason the markers are not displayed, is that the initAdDisplayContainer function is only being called when the adUi is clicked. I would recommend moving the initAdDisplayContainer function to an initiation step, so it is called every time and not just on clicks.

Hey. Sorry, I didn't quite understand you: to the step of launching what? Could you show the simplest example?

Launching the page or application, or something that happens every time the page loads. The way it was set up in the example, it was only called on click, so if the user never clicked, the markers could not be set.
Using one of the plugin ad events may be a way to make sure this is called on every ad request.

Launching the page or application, or something that happens every time the page loads. The way it was set up in the example, it was only called on click, so if the user never clicked, the markers could not be set.
Using one of the plugin ad events may be a way to make sure this is called on every ad request.

what event do you recommend to hang this function?

I would recommend doing this once an ad request is made:
this.player.on('ads-manager', function(response){
  initAdDisplayContainer();
})


Your connection is not private / NET::ERR_CERT_AUTHORITY_INVALID  on windows 10

I just did the following, in a docker container:
mkcert eventapp.local "eventapp.local" "localhost" <IP_ADDRESS> ::1

Then I tried using that in nginx.   Then in the host machine (windows) I ran:
mkcert -install

if I re-run that, it says its already installed:
.\mkcert.exe -install
The local CA is already installed in the system trust store! 👍
Note: Firefox support is not available on your platform. ℹ

But when I try to go to http://eventapp.local in chrome:
Subject: mkcert development certificate

Issuer: mkcert root@buildkitsandbox

Expires on: Feb 14, 2023

Current date: Nov 14, 2020

PEM encoded chain:
-----BEGIN CERTIFICATE-----
MIIEUTCCArmgAwIBAgIQWCng0+tt46H79dtMnkm8EjANBgkqhkiG9w0BAQsFADBl
MR4wHAYDVQQKExVta2NlcnQgZGV2ZWxvcG1lbnQgQ0ExHTAbBgNVBAsMFHJvb3RA
YnVpbGRraXRzYW5kYm94MSQwIgYDVQQDDBtta2NlcnQgcm9vdEBidWlsZGtpdHNh
bmRib3gwHhcNMjAxMTE0MTkxMjE2WhcNMjMwMjE0MTkxMjE2WjBIMScwJQYDVQQK
Ex5ta2NlcnQgZGV2ZWxvcG1lbnQgY2VydGlmaWNhdGUxHTAbBgNVBAsMFHJvb3RA
YnVpbGRraXRzYW5kYm94MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA
tc9R2aRnW+25/MIJP0r4/SqStL0Bxqb83jSmvw2j3VV6Z3kGCySNQzDULMq8J+w4
fO/xGwf6Hm7FR6/Wq2F0R/RriPW0hNRTAU8v6mLs2WYspvCQ7BkmJ5m9kKMZyi1z
BTGtPABacxSledVVMxHmMvjWG857zCtVpeaMab1jBqfBGN2bpZAGNaP6vrdO5O/w
CyexsTpNco96tliAioNAAbzmu4pq6XenIbDIwi8chtOOAtfBnvB+NSoBeOEOvGUY
R37IzafuKL/OZ5KtcY3sQ1bz++I8sklOdoqbG3AWfups1pgsZUzlBmnvU+Cn0A4B
cHatrPI9tKGLJYpVnW+o+wIDAQABo4GZMIGWMA4GA1UdDwEB/wQEAwIFoDATBgNV
HSUEDDAKBggrBgEFBQcDATAfBgNVHSMEGDAWgBQU4pHNU/GJLgyHgPbYb0Y5gvHR
0jBOBgNVHREERzBFgg5ldmVudGFwcC5sb2NhbIIQKi5ldmVudGFwcC5sb2NhbIIJ
bG9jYWxob3N0hwR/AAABhxAAAAAAAAAAAAAAAAAAAAABMA0GCSqGSIb3DQEBCwUA
A4IBgQDm86yF5Oy3tC1qC7C7Rdqzf2PRKVzfXzytloeB5nbRWcp0fIfzeE7kD6T+
Q+pCjeNSoeCmCTjLFJCEdvSU0xFBdyybYTSga4+Pfz7a60mEa+WZp+sOliCRKmsw
fGwwQTw3SRs7oVWnJZeq+aEKyANPmKs3JLuuUL7KtToPna8z1d9kBRzMhZDFdVvZ
YcMtnl77RKcOFDNHq8uAn5w3GRPRF0DLGJg+5OeYHKKe6PCMw7x4h3t61ZWiiMQp
cuji1pohrPZAPSp1oPb1biuVRD8QvBZ3cRC42m8lK1hfeHGjCm1w4GZw7rgDf6os
pL1dZZy0IZLWhgfpD1WmmxcxZD2FlhUK6G3Roe/SWM2eV1Lfy70Sz2L2TuZWNy39
2FU0J/Oyu0BAx96PvyGU27XMMSslDfabATbzYLQ0bLP6o2uDqZ+TzbQT5AKRZe6r
n5vcH+desrjLpGo9YcA8CkkZietqH6n9pPQWraZ34wIE9VLNmanCOaFLbpdv+IAl
PXMieYM=
-----END CERTIFICATE-----

Any idea what might cause this?

The host machine and the docker container will have different roots. You need to copy the root from the container to the host machine before running -install. You can print the path with mkcert -CAROOT.

ok, this makes sense.   It generates a CA Root for each system.  So if you want to share it across systems you need to:
mkcert -CAROOT

In the docker container to find the root, and then share it to the other system and then:
set $CAROOT to its directory

To the files you just got, then run:
run mkcert -install

and it allows a separate system to securely access the docker container.

Correct. Remember that if you share these files, for example by turning the container into an image and them pushing it, they can compromise the security of all machines where that root was installed.

An alternative would be to generate the certificate on the host, and then copying just the certificate (not the root) into the container, instead of running mkcert inside the container.

Yeah, I want to generate it in the container so each dev gets a unique one (rather than committing the cert in git).  But then I need to give their host system a way to access that root so they can install it

oh, thats true I could just make each dev run the mkcert command to generate the cert first


[Event Request] Codeunit 9988 "Word Template Impl." - Procedure GenerateSpreadsheetDataSource - OnBeforeGenerateSpreadsheetDataSource

Please add an Event OnBeforeGenerateSpreadsheetDataSource in Procedure GenerateSpreadsheetDataSource in Codeunit 9988 "Word Template Impl."
local procedure GenerateSpreadsheetDataSource(var DataCompression: Codeunit "Data Compression")
var
    TempBlob: Codeunit "Temp Blob";
    Decorator: DotNet CellDecorator;
    XlWrkBkWriter: DotNet WorkbookWriter;
    XlWrkShtWriter: DotNet WorksheetWriter;
    ColNo: Integer;
    MailMergeField: Text;
    MailMergeFields: List of [Text];
    InStream: InStream;
    OutStream: OutStream;
begin
    MailMergeFields := MergeFields;
    OnBeforeGenerateSpreadsheetDataSource(MailMergeFields);

    if MailMergeFields.Count() = 0 then
        exit;

    TempBlob.CreateOutStream(OutStream);
    XlWrkBkWriter := XlWrkBkWriter.Create(OutStream);
    XlWrkShtWriter := XlWrkBkWriter.FirstWorksheet();
    XlWrkShtWriter.Name := DataSourceSheetNameTxt;

    Decorator := XlWrkShtWriter.DefaultCellDecorator();

    foreach MailMergeField in MailMergeFields do begin
        ColNo += 1;
        XlWrkShtWriter.SetCellValueText(1, ConvertColNoToColName(ColNo), MailMergeField, Decorator);
    end;
    XlWrkBkWriter.Close();

    TempBlob.CreateInStream(InStream);
    DataCompression.AddEntry(InStream, DataSourceFileTxt);
end;

[IntegrationEvent(false, false)]
local procedure OnBeforeGenerateSpreadsheetDataSource(var MergeFields: List of [Text])
begin
end;

@cscheidsteff What is the use case scenario for this? To add/remove additional mergefields into the doc before the zip is generated?

Closing for now, please reopen once you have more information :)

@darjoo I was on vacation the last two weeks, so sorry for the late reply. I would kindly ask you to reopen this ticket. I will then post the requested information as soon as possible. thx, Carsten

@cscheidsteff No worries, reopened.

@darjoo Below as announced, some information about this request. As we found out, the number of fields offered in the mail merge field Menu of Microsoft Word is limited to max. 255 items. This limit is reached very fast with somewhat extensive tables with many fields. In addition, the handling, i.e. the creation of a mail merge template using such a long menu is extremely tedious. Therefore, it would be helpful to be able to control and thus limit the fields that are passed to the datasource file. The requested integration event would provide this possibility.

Hi @darjoo,
you're right, in the end both events technically do the same job. I think that the event as suggested in #18777 is the more appropriate place then. If that one will be available in the future, I think this ticket here (#18778) could be dropped.
One more thing. While checking #18777  again, I just noticed that I accidentally seem to have passed the wrong variable when invoking the new event.

But you probably already noticed that too.


Added Python Package 'migrations' to todo app

because  don't exist Python Package migrations in app todo ,When executing the command :
python manage.py makemigrations

don't apply migrations file and finally The models in Todo app is not added to the database

thanks for fixing the issue this is a very old example and by mistake when i was creating it i did ignore those files


[T5A2][T09-B4] Zachary

Some comments added. Feel free to add more comment if you have any questions. Once you are done, you may close the PR after reading comments.


Add direct dashboard access button to changelog

Description
Add a changelog entry to accompany "Direct Access to Node-RED Dashboard from FlowFuse UI" #3611
Related Issue(s)
https://github.com/FlowFuse/flowfuse/issues/3611
Checklist

[x] I have read the contribution guidelines
[ ] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

@joepavitt @Steve-Mcl Other than the timestamp, is this good to go?

@joepavitt @Steve-Mcl Other than the timestamp, is this good to go?

It was approved. And I did apply the perfectly good suggestion. Good to go from my PoV.


[Feature Request] Spells That Create VFX

I really like this mod, but I found out that some of the spell I make lack some "punch" to it when casting, and than I realized maybe adding some spell that could create VFX is going to be a cherry on top. Something like the conjure spell circle or particle trail in Random PSIdeas would be awesome!
This way we could create spells that is not only very useful and powerful but also very appealing to the eyes because of those fancy effects one can put on their spells.
Also by adding this we might need some config to change the effect density or lasting time maybe?

i mean you could summon magelight and constantly break it

Hexal has Particle, which somewhat fills this niche, but I think there's still room for improvement.


Fixed unclosed file descriptor

The icon file descriptor was not closed properly after opening, leading to a ResourceWarning: unclosed file <_io.BufferedReader name='pickaxe.ico'> warning.

Hello o/
That's correct. However, there's another issue to this, that I've meant to fix together with this warning. You see, for the icon to always display properly, a reference to the PhotoImage object has to be kept somewhere. I never really caught onto that, and the icon has about a 50% chance of becoming pixelated after launching the application because of that. I was pretty sure the warning comes from this exact issue, and not because of not closing the PIL image there, but so be it. Can you add the reference keeping to this code as well?
Ref SO posts:
https://stackoverflow.com/questions/65446939/
https://stackoverflow.com/questions/16424091/
The reference can be kept on like a root._icon_image attribute.

I'll take a look and see what I can do.

This is now not needed due to https://github.com/DevilXD/TwitchDropsMiner/compare/ab9b155250c0be22b760b2cf606cbb62009e54f4..f74f7c8be7e01a05a09d04b78b312b28158fc817


Add support for list API calls

This is a fork of @BenLubar's work (as discussed in #92), but with added tests.

Coverage increased (+0.6%) to 91.5% when pulling f9fc8ad95d147009af277fa012771f14501f8831 on muesli:lists into e804ee7eb264c7bbd58ad9c80291e4d9e2c131a5 on mattn:master.

Good to merge. Thank you.


npm run dev 访问http://localhost:8080地址 报跨域错误

请问，npm run dev 访问http://localhost:8080地址 报跨域错误 http://localhost:8080访问不起来吗？
错误如下：
Failed to load http://localhost:3000/api/getCurrentUser: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8080' is therefore not allowed access.
Logger.js:178 App: getCurrentUser error, Error: Request has been terminated
Possible causes: the network is offline, Origin is not allowed by Access-Control-Allow-Origin, the page is being unloaded, etc.
at Request.crossDomainError (client.js:702)
at XMLHttpRequest.xhr.onreadystatechange (client.js:770)
谢谢！

请先启动服务端 npm start, 再编译静态资源请npm run dev ，最后访问服务端首页地址http://localhost:3000/
你访问的http://localhost:8080是静态资源的地址，如要去除登录拦截请自行去除登录模块

好的，谢谢


Error while adding package 'Package name' to project 'e:/path/to/project.csproj'. The project does not support adding package references through the add package command

I get a Error while adding package 'Package name' to project 'e:/path/to/project.csproj'. The project does not support adding package references through the add package command
for every project in my solution.

What is the target framework of your projects?
On Thu, May 7, 2020, 14:16 Sarper Soher<EMAIL_ADDRESS>wrote:

I get a Error while adding package 'Package name' to project
'e:/path/to/project.csproj'. The project does not support adding package
references through the add package command
for every project in my solution.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pcislo/vscode-nuget-gallery/issues/23, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDNA4W6V3WSDX5GIXADEUTRQKRBRANCNFSM4M3IXZQA
.

It's 4.7.1, the projects are generated by the Unity game engine if it's of any relevancy.

I think that's why it's not working. The command used by extension (dotnet add package) works only with .NET Core apps.


Argmax does not seem to be recognized in TensorFlowSharp

I've been trying to include a Tensorflow byte graph into unity, but I get the error that the function argmax is not recognized.
Currently using Tensorflow 1.4.1.
Is there a solution to this?

Please provide a sample showing the issue, I do not quite follow.
At least on 1.5, there is this API on TFGraph:
public TFOutput ArgMax (TFOutput input, TFOutput dimension, TFDataType? output_type = null, string operName = null)


Support encoding, and collation for database, table, and column

Hi! MySQL supports encoding, and collation for each database, table, and column. Is there any plan to support them?
For example, in Ruby on Rails, database.yml application-wide configuration sets the database encoding, and collation, and in the migration file, option parameter of the create_table method sets the table ones, and also each column has encoding, and collation option to set them.
I want the database configuration, and the option for each column at least, because I don't like to depend on the MySQL system configurations, and a column collation particularly depends on its use-case because MySQL collation has some issues on the emojis used or the Japanese environment.
You can see the detail of that issue at the following URL.
https://bugs.mysql.com/bug.php?id=76553
So, considering the Japanese environment, I'd like to use different collations between email, and texts.

Hi @milkcocoa, thanks for the feature request. If I understand correctly, there are two separate issues here:
1: Configuring character set for a given MySQL connection.
When connecting to a MySQL server, the client must send the default character set it wishes to use. This currently defaults to .utf8mb4. We could make this configurable in MySQLNIO and then expose it via the MySQLConfiguration struct.
2: Configuring character set for database, tables, and columns.
I'm assuming you are using Fluent here since MySQLKit itself (built on SQLKit) does not explicitly support migrations (although you could implement them).
Database
Fluent is not responsible for creation of the database. Unless I'm mistaken, choosing a database-wide character set should be handled when you create the database. If you use Docker for example, you could configure that like so: https://stackoverflow.com/questions/45729326/how-to-change-the-default-character-set-of-mysql-using-docker-compose
Table
Fluent is responsible for creating tables. There is currently no way specify character set or collate using database.schema. As a workaround, you can use custom SQL.
For example:
import FluentSQL
import MySQLKit

let builder = database.schema("planets").id()
if database is MySQLDatabase {
    // The underlying database driver is MySQL.
    builder.constraint(.sql(raw: "CHARACTER SET charset_name"))
    builder.constraint(.sql(raw: "COLLATE collation_name"))
} else {
    // The underlying database driver is _not_ MySQL.
}
builder.create()

Column
There is currently no way to specify character set or collate using Fluent. As a workaround, you can use custom SQL here, too.
import FluentSQL
import MySQLKit

let builder = database.schema("planets").id()
if database is MySQLDatabase {
    // The underlying database driver is MySQL.
    builder.field(
        "name",
        .string,
        .required,
        .sql(raw: "CHARACTER SET charset_name"),
        .sql(raw: "COLLATE collation_name")
    )
} else {
    // The underlying database driver is _not_ MySQL.
    builder.field("name", .string, .required)
}
builder.create()

Since the concept of CHARACTER SET / COLLATE is unique to MySQL, it probably doesn't make sense to add these options to Fluent itself. Unless the other databases Fluent supports have similar things that I'm not aware of.
However, some things we could do are:

Add these examples to the documentation
Add convenience APIs for MySQL in particular, for example:

if database is MySQLDatabase {
    // The underlying database driver is MySQL.
    builder.field("name", .string, .required, .mysql(characterSet: .utf8mb4))
}

Let me know if this makes sense and then we can split out these issues into their respective repos.

Hi @tanner0101, thanks for replying to my request, and your understand is correct. There are two separate issues.
For the first issue, if MySQLConfiguration struct provides the configurations, that would be great!
For the second, I think Database issue is resolved by the first connection issue.
About Table, and Column, it makes sense that Fluent doesn't have the Fluent-wide resolution, and I understand the workarounds. I'm grad that the documentation has the examples. Moreover, if Fluent has convenience APIs, it is great useful.

@milkcocoa sounds good. Do you want to open the separate issues?
1: Configuring character set for a given MySQL connection (vapor/mysql-nio first).
2: Convenience APIs for configuring character set for database, tables, and columns (vapor/fluent-mysql-driver).
Also let me know if you want to provide PRs. Otherwise I will grab them eventually. Thanks!

OK. I've split this into https://github.com/vapor/mysql-nio/issues/55 and https://github.com/vapor/fluent-mysql-driver/issues/194.
Also I'll take a look at the codes, and provide PRs if I can.


Generalization of solver

Hey @phlo,
I like your idea of generalising the plugin to support arbitrary SMT solvers. Do you have any plans of merging your fork back, since I'd like to have this functionality in the plugin too?
No pressure though - it's no problem if you want to continue with this fork on your own or simply have other plans for it. I just thought I'd ask before duplicating functionality. You're welcome to contribute.

Ahoj @bohlender!
First of all: thank you so much for your work on this plugin! I was looking around for a new SMT-LIB plugin and found yours to be the best, all I tried before were rather unsatisfiable ;) Since I'm using boolector and love your in-vim evaluation feature, an extension to support any SMT solver was only logical. This is why I forked it: to try this generalization and being able to use it immediately. I planned to ask you if you want me to contribute and if so, merge it back.. So I'm glad you like it :) But after finding a beta tester - my professor Armin Biere - I hesitated since he had some installation issue, which I wanted to resolve first. Unfortunately, we weren't able to do so as he was at TACAS last week. That's also why I updated the README's 'Installation' section.
So if you really want to merge, I'd be happy to contribute! I just have one concern regarding the plugin's name: if we generalize it, vim-z3-smt2 might be too specific - that's why I renamed it to vim-smt2. Also, I'm kind of a noob when it comes to these GitHub features, so I don't really know what I would have to do.
Cheers,
@phlo

PS: Ich bin aus Oesterreich und spreche daher auch so etwas aehnliches wie Deutsch wenn's dir lieber ist ;)

PPS: If you'r doing hardware stuff and extensively use bitvectors, check out boolector!

PPPS: Thanks to RWTH Aachen for hosting my preferred Gentoo mirror!

Hey @phlo,
let's keep the communication in English for future reference, or in case someone else wants to chime in. Coming from a formal methods background myself, I already am a user of Boolector and happy to hear that you guys (with A.Biere being pretty famous) try to use this plugin.
To begin with, I will merge your work into a feature branch myself and make sure everything still works as expected. Then I will update the master branch and change the name to vim-smt2 as you suggested.
I'm not an expert regarding GitHub's collaboration features either, but I will add you as a collaborator so you can push future contributions directly -- without the need for pull requests.

Hi @bohlender,
sounds like a plan! So as soon as you complete these further steps I will archive xor delete the fork's repository.
Prof. Biere also suggested me to announce the plugin on the SMT-Lib Google group, but didn't do it yet as I wanted to ask if maybe you want to make the post (since you'r the original author)?
Nice to get to know another Boolector user! I forked you'r repository after a discussion with the main developers (Aina & Mathias), where we agreed upon the need for a decent vim plugin ;)
Looking forward to collaborate with you!

Hey @phlo,
I'm not using Google groups, so feel free to announce the plugin once we have everything up and running back in the master branch.
While your modifications defaulted g:smt2_solver_command to boolector, I've set added defaulting to either z3 or boolector (checking in this order whether one of the executables can be found in $PATH). We can change the order though if this is a problem. I just find it reasonable to have z3 as one default since its extensions are supported by the highlighting.
I've also adapted the README.md slightly to account for the change and make the installation instructions a bit leaner (moved some stuff to FAQ). Please have a look whether anything needs discussion or just make reasonable changes you find necessary (you should have push access).
To me, the current state looks fine. I will merge it into the master once you're okay with it.

Hey @bohlender,
alright.. looking good! I just made some cosmetic changes to test my push access and it seems to work :)
The README.md's installation instruction table looks nice! Sorry for breaking the old one. I even tried to vertically center the enumerations inside the table cells but failed (should have tested it using the preview feature of some GitHub comment box). Regarding the former post installation section: I just put it there to make sure filetype plugin is enabled, since I suspect it being the problem encountered by @arminbiere. But it's indeed much more suitable in the FAQ / troubleshooting section.
I was also thinking about switching back to z3 as the default solver for the merged version, but forgot to mention it in my previous replies. Anyhow, I really like your solution! This way we could add other major solvers to the list and get rid of having a specific default in general. I would even go as far as supporting their language extensions too. Shall we?


prompts should support type float64 as well as string

Sorry, @zzamboni, but while reviewing my commit history I also noticed one of your changes:
https://github.com/elves/elvish/commit/a062be0c052332afdcd821f4a95289cf92d76bf2
To reproduce the problem that change in the documentation addresses start an interactive Elvish shell and type these statements:
> n = 1
> edit:prompt = { sleep 2; put $n; n = (+ $n 1); put ': ' }

The change in that commit "fixes" the problem with the documentation but introduces an ambiguity since it mixes byte and value output in the prompt function. The order in which byte and value streams are processed is currently undefined. Thus, replacing the first put with to-string, "fixes" the problem but there is no guarantee that the number, written to the byte stream, will appear before the ": " string that is written to the value stream. The proper fix is to change the prompt rendition code to support implicit coercion of any data type (including float64) that has a String method so that prompt functions can consistently write to the value stream.

@krader1961 I agree that it would be nice that the prompt code automatically converts other types to string.
However, your explanation of the ambiguity is incorrect. to-string writes the string to the value stream, not to the byte stream, so the documentation fix still outputs only to the value stream.

However, your explanation of the ambiguity is incorrect. to-string writes the string to the value stream, not to the byte stream, so the fixed example still outputs only to the value stream.

Yeah, I realized that after I went to bed and it occurred to me I had conflated to-string with to-lines. Nonetheless, I think everyone would agree that coercion to a string should happen implicitly in the context of rendering a prompt. The question is whether it should be done only for specific types such as float64 or should be done for any non-string data type. It looks like the the Text.Concat and Text.RConcat methods are the appropriate place to add the conversion for float64 (and possibly any non-string type not already handled).

After thinking about this some more, and working on a fix, I've decided that float64 should be special-cased rather than implicitly converting to a string any data type not otherwise explicitly handled by the Concat and RConcat methods. Doing so is obviously fragile since the addition of other data types (see issue #241) that should be implicitly converted will also need to be special-cased in these methods. If, and when, that happens we can explore introducing a new interface for a "convert to string only in contexts like rendering a prompt" API.


Add flag for maximum number of dummy questions

Allow maxDemoQns to be overridden by flag. This allows the quiz file to have any number of questions.

This change is

Coverage remained the same at 61.149% when pulling 8109b24fd7316bd27067b6eb7f6f2fa69c7e2533 on feature/max_question_flag into f61b23d7bc513a204e4a9c6f6112169cad8d69cb on develop.

Maybe change the flag name to max_demo_qns? Otherwise  Feel free to squash and merge.

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Coverage remained the same at 61.149% when pulling 8ec8f5af9917efd201b774701026e83df56331fd on feature/max_question_flag into f61b23d7bc513a204e4a9c6f6112169cad8d69cb on develop.

Changed the flag name to max_demo_qns. @pawanrawal If you give a final LGTM, I'll merge it to develop.

  Go ahead with squashing and merge to develop.

Review status: 1 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


first draft of the changes for GPUSIM-3

Followed JIRA case description: if "cache dir" is specified, cache folded fingerprints there keyed on hash+fold-ratio. Did not modify the http server yet, wanted to get some feedback first to avoid unnecessary work. Still thinking about unit tests. BTW, could not get the http server going, see the following:
Processing request 482885951
QObject: Cannot create children for a parent that is in a different thread.
(Parent is QNativeSocketEngine(0x2a27de0), parent's thread is QThread(0x23baa60), current thread is QThread(0x7fb9d80079f0)
QSocketNotifier: Can only be used with threads started with QThread
Unknown database  "default"  requested.

Just a note that the http interface testing option assumes you're starting the server with a file named 'default.fsim', or at least have created a symlink named default.fsim to your real file.  It'll probably be a few days before I can review this as I'm traveling, but thanks for making quick progress!

Ah, it does work with 'default.fsim' -- the message about threads is just a warning.


Feature/support nuget lookup from direct path

Changes add ability to:

Download nuget.exe from the custom URL
Use previously installed nuget

Also:

Tests for the new method were written
Documentation was updated
Version was incremented

@gluck, could you please take a look on my pull? It closes several open issues on the plugin.
This change is needed to start using this plugin in the my projects

@timotei , all notes were fixed.
Please really sorry for long delay.


Memory bank mismatch

I am facing the following error when executing PipeCNN on AWS F1.

Loading kernel/binary from file cnnf1_pythonpipe2.awsxclbin
ERROR: ERROR: Memory bank specified for kernel instance "memRead_1" of kernel "memRead" for argument index 21 does not match the physical connectivity from the binary.
Bank specified on host side is "M01_AXI" while bank from the binary is "M00_AXI".

ERROR: clSetKernelArg() for kernel "memRead", argument index 21.

ERROR: CL_INVALID_MEM_OBJECT
Location: ../src/host/main.cpp:730
Failed to set argument 21 kernel memRd

Was I supposed to set any parameter?
full output here

This seems to be fixed with the latest clone.


autoLayout animation

thank you for this powerful frameworks!
i have some questions:
1.chainAnim with autoLaytou is useless,e.g.:
self.redView.makeConstraint(self.leftMarginC,100).animate(2.0f);  //do not has any effects
2.cloud add duratiion for each chain? e.g.:
self.redView.moveX(100).duration(1.5f).thenAfter.moveY(100).duration(.8f).animat;
3.i could not understand the diff between wait/delay and thenAfter?
thank you!

wait is just the same with delay.
thenAfter is a method when you use multiple animations in one line of code. You should use thenAfter  instead of animate ex:
view.moveX(10.0).thenAfter(2.0).moveY(10.0).animate(1.0)

view will move x in one second and then move y in one second.
You are not supposed to write code like this:
// WRONG
view.moveX(10.0).animate(2.0).moveY(10.0).animate(1.0)

When you use multiple animations one by one, you should use thenAfter instead of  animate(moveX(10.0).thenAfter(2.0)) except the last animation(moveY(10.0).animate(1.0)).

@nsdictionary

I will fix this. I am traveling at the moment and have limited time. For now use the transform methods. those are tested and will work. They also will work with effects, unlike the constraint animations

thenAfter(t) should take a time period for the animations before it. I dont want to add a duration property because that will begin to get confusing.

As @Draveness said  wait and delay are the same thing. wait is just more conversational and easy to read while delay is more similar to apples API.


Adding web vault compatibility

Update of the api to be compatible with the web vault.
What is working  :

Create account
Connect
Create/modify folders
Create/modify ciphers
Modify the master password (need to create a POST /password route).

What doesn't work :

Managing organisation (password share)
import tool

Sorry for this ruby code, first time reading and trying to understand how it works. I'm convinced that we can optimise it.
Need to investigate the need of using cookies or not.

@jcs do you plan to review the code ?

@Pschittt please review my pull-request to your patch-1 branch (https://github.com/Pschittt/bitwarden-ruby/pull/1) and potentially resubmit this pull request as your code as in commit 36e441f was throwing a syntax error.

@andrzejwp code reviewed (from my window...) and tested. Everything ok for me.
Thanks for your review.
Like I said, first time reading and writing ruby, so maybe it can be optimized.
Hope @jcs will merge into his "official" repo.

Change masterpassword available.

@jcs I'm not fluent in Ruby either, so please review and correct whatever we did wrong here. It now works in my case (tested in Chromium and Firefox, using both the web vault as well as browser extensions), which was to create master password, log in, add and consume secrets.

@jcs I've updated the api.rb with your advice.
One thing I didn't do, is the publickey check. If I did, i would have done a simple regex as (.*). I don't think it's really relevant this kind of regex.
One more thing, the parameter are all put downcase, except the one for publickey and encryptedkey during registering. They are passed as : [:keys][publicKey] and [:keys][encryptedPrivateKey]
Last question, concerning the library you are using for sqlite. Does this library prevent from SQL injection attacks ? Does the library check if the parameters sent, don't hide another SQL request ?

@jcs Is it better now ? Can you merge it ?

@jcs can you please review the changes @Pschittt added and possibly merge it?

Only one small style comment, without having spent a lot of time looking through the diff yet: there's a fair bit of mixed tab/space and trailing whitespace in your diff that you might want to clean up to match the style of the rest of the project.

@logic I've done the update. I saw only 2 lines which the tab wheren't correct.
I also made the change you requested.
Hope it's ok now ;)

FWIW, I see 85 lines with leading tabs after applying your diff; you might want to check your editor's settings, I suspect it's hiding them from you. grep -P '^\t' api.rb can show you the lines that have problems, sed -ie 's/^\t/  /' api.rb will get rid of them.

@logic Thanks !
Ran you sed command, and seems to did the trick.
I also added a OPTIONS request for HTTP CORS checks, which sometimes occurs.
Hope now everything is ok, so we can go forward with your other PR ;)

Sorry. Don't have time to continue this.
BTW @universal seems to be more advanced on this now.


Make dependencies less strict

What
Fixes #97
How
Less strict requirements with >=

@GlaserN Do you have any insights on what might have led to the following test fail?
test/test_envelopes.py:262: in <module>
    test_gaussian()
test/test_envelopes.py:190: in test_gaussian
    actual=envelopes["gaussian"](t=ts, params=params), desired=test_data["gaussian"]
E   AssertionError:
E   Not equal to tolerance rtol=1e-07, atol=0
E
E   Mismatched elements: 2 / 100 (2%)
E   Max absolute difference: 6.51925802e-09
E   Max relative difference: 1.33211993e-15
E    x: array([2.566574e-09, 5.401866e+05, 1.172500e+06, 1.909621e+06,
E          2.765382e+06, 3.754771e+06, 4.893897e+06, 6.199942e+06,
E          7.691067e+06, 9.386301e+06, 1.130537e+07, 1.346853e+07,...
E    y: array([0.000000e+00, 5.401866e+05, 1.172500e+06, 1.909621e+06,
E          2.765382e+06, 3.754771e+06, 4.893897e+06, 6.199942e+06,
E          7.691067e+06, 9.386301e+06, 1.130537e+07, 1.346853e+07,...

We have only made the requirements of dependencies less strict by allowing upgrades to the latest version of tensorflow. There s no API breaking changes with tensorflow that affects us and if we were relying on some specific/weird aspects of the tensorflow implementations, this might be a good time to revisit it because we definitely do not wish to restrict our dependencies to a particular release.

I guess the rtol or atol of this test has to be adjusted. The 2 mismatched elements are 1e-15 smaller than all the other elements meaning that we seem to have some numerical artefacts happening here.


Add pinyin hint when typing, with shortcut [Ctrl +Period]

No prob. So this was firstly mentioned in rime-ice here
Which means that I traced to this file https://github.com/mozillazg/pinyin-data/blob/master/zdic.txt
The file was cleaned up a bit into a openCC-friendly format in my PR, but I didn't change any codes for any characters.

btw I didn't write a script to generate the file, instead I just used Excel, based on the assumption that such data should be relatively static.


Test v3.14 roundtrip

Could it be the new paper_size property in the SceneInfo?
{
  "SceneInfo": {
    "backgroundVisible": {
      "timestamp": "0:0",
      "value": 1
    },
    "currentLayer": {
      "timestamp": "0:0",
      "value": "0:0"
    },
    "paperSize": {
      "first": 1404,
      "second": 1872
    },
    "rootDocumentVisible": {
      "timestamp": "0:0",
      "value": 1
    }
  }
}

(this is from one of my file using the debug env var of the remarkable software)
In your 3.14.4 file, it is (1620, 2160). I started a branch yesterday to try to fix some small issues/missing properties and i have done this https://github.com/Seb-sti1/rmscene/commit/8b74595e081f9a6e34deb4a70b4b1045199c1400 for the paper_size property. I didn't create an MR yet 'cause i wanted to have a look to #32 before.
Hope it can help :)

Me again :)
I just had time to run the pipeline/tests with the commit I mentioned and another one, and it appears to fix the pipeline, see here https://github.com/Seb-sti1/rmscene/pull/1 (of course, I did not deactivate the pipeline to publish on PyPI, so the corresponding pipelines fail).
Regarding the two commits:
The first one (https://github.com/Seb-sti1/rmscene/pull/1/commits/9505d1db55c48890953acfe65f22fabc26e823a2) is the addition of paperSize in the SceneInfo class
On the files that had excess bytes, they were 92, 8, 0, 0, 0, 124, 5, 0, 0, 80, 7, 0, 0 which seems to correspond to :

92 = 5 (index), 12 (TagType.Length4) as read and check by DataSteam.read_tag
8, 0, 0, 0 the size 8 bytes here
124, 5, 0, 0 = 124 + 5 << 4 = 1404 = first
80, 7, 0, 0 = 80 + 7 << 4 = 1872 = second

The first implementation I tried was with the code below as it seemed to be closer to a simple value but given the existence of what appears to be a size I changed it and used TaggedBlockReader.read_subblock. Btw I don't know if it would be better to read size/4 uint32 instead of the hardcoded two.
def read_two_int(self, index: int) -> tuple[int, int]:
    """Read a tagged two 4-byte int."""
    self.data.read_tag(index, TagType.Length4)
    first = self.data.read_uint32()
    second = self.data.read_uint32()
    return first, second

def read_two_int_optional(
        self, index: int, default: tp.Optional[float] = None
) -> tp.Optional[tuple[int, int]]:
    """Read a tagged two 4-byte int."""
    return self._read_optional(self.read_two_int, index, default)

The second one (https://github.com/Seb-sti1/rmscene/pull/1/commits/ecd56a07be0b5bff7a957874569b23e2d527a2f4) makes the background_visible, root_document_visible and paper_size on the SceneInfo optional
I had some files where the whole SceneInfo was only containing the current_layer property
UnreadableBlock(extra_data=b'',
                error='Bad tag type 0x0 at position 144',
                data=b'\x1c\x06\x00\x00\x00\x1f\x00\x00/\x00\x00',
                info=MainBlockInfo(offset=132,
                                   size=11,
                                   extra_data=b'',
                                   block_type=13,
                                   min_version=0,
                                   current_version=1))

Let me know what you think and if you want me to create a MR<EMAIL_ADDRESS>-> main /<EMAIL_ADDRESS>-> test-v3.14-roundtrip) or you cherry-pick the commit from my fork (do whatever is easier/better from your pov)

I think you're right that those are the parts that are not parsed, but given we have the extra_data in the blocks, it should still work to round-trip the data even when everything is not being parsed fully. This PR makes sure that's true and tested.
It also adds another test, which checks that all the blocks were actually parsed correctly. One of this is xfailing but should be fixed by your changes #39 as a follow up


Feat : Issue template page

Describe changes
Create issue template page

we make a preset that is well used in each field and suggest it to the user.
also we provides a modal window to enter basic information such as name, description, etc

Issue number or link
#104
Types of changes
What is the type of code change?
Put an x in the boxes that apply

[ ] Bugfix (changes that resolve errors)
[x] New feature (changes which adds functionality)
[ ] Breaking change (big changes that affect existing functionality)
[ ] Documentation Update

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

[x] I have read the "README.md"
[x] My code follows the style guidelines of this project
[x] My changes generate no new warnings
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
Please let me know if there's anything else to explain

title field is optional, but empty string is not allowed...
title field는 옵셔널이지만, ""은 허락되지 않습니다!


PCL vcpkg integrate install didn't work

Problem
I was learning PCL on Ubuntu, however, since the program must be created for Windows, i am trying to learn how to use on Visual Studio 2015. So vcpkg was used to install as a package manager.
After the installation, vcpkg integrate install command was written. I tried to test PCL with a sample code. However, this error code was displayed:

Error	C1083	Cannot open include file: 'pcl/io/pcd_io.h': No such file or directory	pcl-teste

The Environment Variable Path ;C:/PCL/vcpkg was added.
TL;DR
I installed PCL with this Tutorial , going to explain how.
How was installed
Using vcpkg install pcl:x64-windows-static
Had an issue:
CMake Error at ports/msmpi/portfile.cmake:56 (message):
Could not find:
C:\Program Files/Microsoft MPI/Bin/mpiexec.exe
Please install the MSMPI redistributable package before trying to install this port.
The appropriate installer has been downloaded to:
C:/vcpkg/downloads/MSMpiSetup-8.1.exe

Call Stack (most recent call first):
scripts/ports.cmake:72 (include)

Error: Building package msmpi:x64-windows-static failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: msmpi:x64-windows-static
Vcpkg version: 0.0.100-2017-12-06-29163e1566403c371a5af48c8dbf3d96da32b8d1

Additionally, attach any relevant sections from the log files above.

So refering to this issue #1375  , it was solved by opening the directory below:

-> .\src\vcpkg\downloads

Then the msmpi is installed manually, opening this two files:

msmpisdk.msi
MSMpiSetup.exe

And with this fix, the PCL was installed.
Testing the installation
With the command vcpkg integrate install
This is the output:
Applied user-wide integration for this vcpkg root.

All MSBuild C++ projects can now #include any installed libraries.
Linking will be handled automatically.
Installing new libraries will make them instantly available.

And with vcpkg lists:
boost[core]:x64-windows-static 1.65.1-3         Peer-reviewed portable C++ source libraries
bzip2[core]:x64-windows-static 1.0.6-2          High-quality data compressor.
eigen3[core]:x64-windows-st... 3.3.4-2          C++ template library for linear algebra: matrices, vectors, numerica...
expat[core]:x64-windows-static 2.2.5            XML parser library written in C
flann[core]:x64-windows-static 1.9.1-7          Fast Library for Approximate Nearest Neighbors
freetype[core]:x64-windows-... 2.8.1-1          A library to render fonts.
glew[core]:x64-windows-static  2.1.0-1          The OpenGL Extension Wrangler Library (GLEW) is a cross-platform ope...
hdf5[core]:x64-windows-static  1.10.0-patch1-2  HDF5 is a data model, library, and file format for storing and manag...
jsoncpp[core]:x64-windows-s... 1.8.1-1          jsoncpp is an implementation of a JSON reader and writer in C++. JSO...
libiconv[core]:x64-windows-... 1.15             GNU Unicode text conversion
libjpeg-turbo[core]:x64-win... 1.5.2-2          libjpeg-turbo is a JPEG image codec that uses SIMD instructions (MMX...
liblzma[core]:x64-windows-s... 5.2.3-2          Compression library with an API similar to that of zlib.
libogg[core]:x64-windows-st... 1.3.2-cab46b1-3  Ogg is a multimedia container format, and the native file and stream...
libpng[core]:x64-windows-st... 1.6.34-2         libpng is a library implementing an interface for reading and writin...
libtheora[core]:x64-windows... 1.2.0alpha1-2... Theora is a free and open video compression format from the Xiph.org...
libxml2[core]:x64-windows-s... 2.9.4-2          Libxml2 is the XML C parser and toolkit developed for the Gnome proj...
lz4[core]:x64-windows-static   1.8.0-1          Lossless compression algorithm, providing compression speed at 400 M...
msmpi[core]:x64-windows-static 8.1              Microsoft MPI
msmpi[core]:x86-windows        8.1              Microsoft MPI
pcl[core]:x64-windows-static   1.8.1-8          Point Cloud Library (PCL) is open source library for 2D/3D image and...
proj4[core]:x64-windows-static 4.9.3-1          PROJ.4 library for cartographic projections
qhull[core]:x64-windows-static 2015.2-2         computes the convex hull, Delaunay triangulation, Voronoi diagram
szip[core]:x64-windows-static  2.1-2            Szip compression software, providing lossless compression of scienti...
tiff[core]:x64-windows-static  4.0.8-1          A library that supports the manipulation of TIFF image files
vtk[core]:x64-windows-static   8.0.1-5          Software system for 3D computer graphics, image processing, and visu...
zlib[core]:x64-windows-static  1.2.11-3         A compression library

So when i used Visual Studio 2015, on

"Project Properties > Linker > Input"

there is $(VcpkgRoot)lib\*.lib on Inherited Values of Additional Dependencies.
So i tried to build this sample PCL program:
#include <iostream>
#include <pcl/io/pcd_io.h>
#include <pcl/point_types.h>

int main(int argc, char** argv)
{
	pcl::PointCloud<pcl::PointXYZ> cloud;

	// Fill in the cloud data
	cloud.width = 5;
	cloud.height = 1;
	cloud.is_dense = false;
	cloud.points.resize(cloud.width * cloud.height);

	for (size_t i = 0; i < cloud.points.size(); ++i)
	{
		cloud.points[i].x = 1024 * rand() / (RAND_MAX + 1.0f);
		cloud.points[i].y = 1024 * rand() / (RAND_MAX + 1.0f);
		cloud.points[i].z = 1024 * rand() / (RAND_MAX + 1.0f);
	}

	pcl::io::savePCDFileASCII("test_pcd.pcd", cloud);
	std::cerr << "Saved " << cloud.points.size() << " data points to test_pcd.pcd." << std::endl;

	for (size_t i = 0; i < cloud.points.size(); ++i)
		std::cerr << "    " << cloud.points[i].x << " " << cloud.points[i].y << " " << cloud.points[i].z << std::endl;

	return (0);
}

With no success and an error message was displayed:

Error	C1083	Cannot open include file: 'pcl/io/pcd_io.h': No such file or directory	pcl-teste

I confirmed this problem.
@ras0219-msft @alexkaratarakis Can you give me some advice?
The reproduction procedure and this problems are as follows.

Run vcpkg install pcl:x64-windows-static

Run vcpkg integrate install

Create a project using new project wizard of Visual C++.

Install used x64-windows-static triplet, but these macros are set x64-windows. In this case, I think these macros should be set x64-windows-static.

Macro
Value
Correct Value

$(VcpkgRoot)
C:\vcpkg\installed\x64-windows
C:\vcpkg\installed\x64-windows-static

$(VcpkgTriplet)
x64-windows
x64-windows-static

@danieltak
Please install MSMPI as instructed by Vcpkg. It is a specification.
It is not problem. But, I think we need to consider other problem (about vcpkg integrate install) that you reported. Please wait for vcpkg team's comment.

scripts/buildsystems/msbuild/vcpkg.targets does not  seem to support x64/x86-windows-static triplets.
I think it should be implement the settings for static triplets in this file.

I have the same issue with VS2017 (v15.4.5) and x86-windows-static/x64-windows-static triplet.

PS D:\dev\vcpkg> .\vcpkg.exe list
boost[core]:x64-windows-static 1.65.1-3         Peer-reviewed portable C++ source libraries
boost[core]:x86-windows-static 1.65.1-3         Peer-reviewed portable C++ source libraries
bzip2[core]:x64-windows-static 1.0.6-2          High-quality data compressor.
bzip2[core]:x86-windows-static 1.0.6-2          High-quality data compressor.
c-ares[core]:x64-windows-st... 1.13.0-1         A C library for asynchronous DNS requests
c-ares[core]:x86-windows-st... 1.13.0-1         A C library for asynchronous DNS requests
cpprestsdk[core]:x64-window... 2.10.0           C++11 JSON, REST, and OAuth library The C++ REST SDK is a Microsoft ...
cpprestsdk[core]:x86-window... 2.10.0           C++11 JSON, REST, and OAuth library The C++ REST SDK is a Microsoft ...
grpc[core]:x64-windows-static  1.7.2            An RPC library and framework
grpc[core]:x86-windows-static  1.7.2            An RPC library and framework
openssl[core]:x64-windows-s... 1.0.2n           OpenSSL is an open source project that provides a robust, commercial...
openssl[core]:x86-windows-s... 1.0.2n           OpenSSL is an open source project that provides a robust, commercial...
protobuf[core]:x64-windows-... 3.5.0-1          Protocol Buffers - Google's data interchange format
protobuf[core]:x86-windows-... 3.5.0-1          Protocol Buffers - Google's data interchange format
websocketpp[core]:x64-windo... 0.7.0-1          Library that implements RFC6455 The WebSocket Protocol
websocketpp[core]:x86-windo... 0.7.0-1          Library that implements RFC6455 The WebSocket Protocol
zlib[core]:x64-windows-static  1.2.11-3         A compression library
zlib[core]:x86-windows-static  1.2.11-3         A compression library
PS D:\dev\vcpkg> .\vcpkg.exe integrate install
Applied user-wide integration for this vcpkg root.
All MSBuild C++ projects can now #include any installed libraries.
Linking will be handled automatically.
Installing new libraries will make them instantly available.
CMake projects should use: "-DCMAKE_TOOLCHAIN_FILE=D:/dev/vcpkg/scripts/buildsystems/vcpkg.cmake"
PS D:\dev\vcpkg>

However, while trying a simple application in VS2017 icnluding boost, I get compilation error :

1>------ Rebuild All started: Project: test_vcpkg_1, Configuration: Debug Win32 ------
1>stdafx.cpp
1>test_vcpkg_1.cpp
1>d:\temp\test_vcpkg_1\test_vcpkg_1\test_vcpkg_1.cpp(5): fatal error C1083: Cannot open include file: 'boost/filesystem.hpp': No such file or directory
1>Done building project "test_vcpkg_1.vcxproj" -- FAILED.
========== Rebuild All: 0 succeeded, 1 failed, 0 skipped ==========

The same issue happened using # x64 dynamic link library vcpkg install pcl:x64-windows

I was able to make vcpkg work after manually updating vcpkg.target file by changing "x86-windows" to "x86-windows-static" and "x64-windows" to "x64-windows-static".

@danieltak I think that x64-windows triplet doesn't reproduce this problem.
It is only occurs in triplet files that named other than x64-windows/x86-windows(/x64-uwp/x86-uwp/arm-uwp), because other triplet names are not defined in the target file.
I recommend editing and using triplet files that named x64-windows/x86-windows as @dimarusyy are saying. Probably, It will be integrated successfully.
Or, Please generate the project using CMake. I recommend this.

same problem.
https://blogs.msdn.microsoft.com/vcblog/2016/11/01/vcpkg-updates-static-linking-is-now-available/
works for me

Hi everyone, thanks for reporting this issue!
Please open a new issue if this is still a problem for you.
Thx.

Hi,
Could someone tell how can I install  MSMPI?
I am currently trying to install PCL  using .\vcpkg install pcl:x64-windows-static.
I would like to know how to do all the steps correctly in order to PCL on my Windows PC.

@petrasvestartas You can simply install msmpi using ./vcpkg install msmpi, and we have a PR to update it. See #8570.


Updated 3 files: chemical probes and tractability annotation files for 18.10

The annotation file with chemical probes from the three different portals and Probe Miner have been updated as well as the tractability data file.

Note: We are working towards accessing these annotation files into the ot-releases buckets from 18.12. They can already be found in a sub-folder of the 18.08 and 18.10 ot-release buckets.


Automate testing

Implement one of the frontend tool for testing.

https://www.selenium.dev/documentation/overview/
https://www.cypress.io/
https://webdriver.io/
https://testcafe.io/
https://playwright.dev/

Gemini opinion:
I can help you choose the right tool for your web frontend automation testing with Selenium, Cypress, Playwright, and TestCafe, considering your project's specifics (TypeScript, ReactJS, visual testing).
Here's a breakdown of the key differences and their suitability for your scenario:
Selenium:

Pros:

Most mature and widely adopted framework.
Supports the broadest range of browsers and programming languages (including TypeScript).
Offers great flexibility for complex testing scenarios.

Cons:

Can be more complex to set up and maintain compared to newer options.
Relies on external WebDriver implementations, adding another layer of configuration.
Might be overkill for purely visual testing.

Cypress:

Pros:

Designed specifically for end-to-end (E2E) testing in the browser.
Offers a user-friendly, JavaScript-based API that integrates well with ReactJS projects.
Provides built-in visual testing capabilities.
Integrates well with GitHub Actions.

Cons:

Supports a slightly smaller range of browsers compared to Selenium.
May not be ideal for highly complex testing requirements.

Playwright:

Pros:

Developed by Microsoft, offering good support and frequent updates.
Supports multiple programming languages (including TypeScript) and browsers.
Provides a powerful and flexible API for various testing needs, including visual testing.
Integrates well with GitHub Actions.

Cons:

Still relatively new compared to Selenium, with a smaller community.
Might have a steeper learning curve for those unfamiliar with its approach.

TestCafe:

Pros:

Easy to set up and use, with a focus on developer experience.
Offers good visual testing capabilities.
Integrates well with continuous integration tools like GitHub Actions.

Cons:

Supports a limited set of browsers compared to the others.
Might not be as flexible for highly customized testing needs.

Considering your project:

Frontend-only with TypeScript and ReactJS: Cypress or Playwright would be strong contenders due to their native JavaScript support and potential integration with React testing libraries.
Visual testing focus: Cypress and TestCafe both have built-in visual testing features.
GitHub Actions integration: All these tools integrate well with GitHub Actions.

Recommendation:
For your ReactJS frontend with a focus on visual testing and GitHub Actions integration, Cypress appears to be the most suitable choice. Its built-in visual testing capabilities and smooth integration with React development tools make it a great fit.
However, if you need more flexibility and broader browser support, Playwright is another strong option worth considering. It offers a powerful API and is actively maintained.
Additional tips:

Experiment with both Cypress and Playwright to see which one feels more comfortable for you and your team.
Explore the documentation and community resources for each tool to get a better understanding of their capabilities.

I hope this comprehensive analysis helps you make an informed decision!

Implementing playwright
While in development I don't commit and push screenshots. So before pull request don't forget to regenerate and commit them.

Looks like the tests will be separated into multiple groups.

Neo4j with movies dataset - test all elementary read only functionality - run in parallel
Neo4j empty - test write operations - execute in series
Neo4j enterprise - multidatabase
Memgraph - test memgraph specific functions (map as node/rel property, not all property types like in Neo4j, what else? Check where you call db.ecosystem)

While in development I don't commit and push screenshots. So before pull request don't forget to regenerate and commit them.

Considering to not use screenshot visual check.


[Bug] Minimap toggle setting is not saved

The setting to show/hide the minimap button is not persisted across sessions.
Looks like the value in the SV is correct, it's just not applied correctly on load. Maybe a race.

Ok so the issue was the database key checking code, i was using
if not x then and i should have used if x == nil then as false can be a valid value
As it was the value was being reset as it regarded false as not existing.
Seems to be working now, thanks.


Cannot run an deobfuscated Minecraft Server jar file

I used McDeob v2.1.40 to deobfuscate the Minecraft Server 1.18.1, but when I tried to run it, I got this error message:
$> java -jar .\remapped_server-1.18.1.jar
Error: Unable to initialize main class net.minecraft.server.Main
Caused by: java.lang.NoClassDefFoundError: joptsimple/OptionSpec

Is the remapped server file broken?

As per the overview page on SpigotMC:

Some classes are missed, as well I think all of the external libraries Minecraft adds are also skipped.
This app basically just deopfuscates and decompiles Minecraft to see the code, but not actually use it.

Thank you very much!


added paradox prompt with ruby_info in rprompt

I added a new version of the new paradox Prompt as I need a ruby-info on the right, I added it so we have the time and ruby-info now.

Edit the existing one. There is no need to duplicate it.

ok


If expose not in .yaml, show "Error: invalid manifest: zero global services"

Issue type

feature request
Akash version

v0.12.2-rc5
Akash token balance

5.6
What happened
Run akash tx deployment create $DEPLOY_YML --from $KEY_NAME --node $AKASH_NODE --chain-id $AKASH_CHAIN_ID  --broadcast-mode sync  --fees $FEES -y show Error: invalid manifest: zero global services
What did you expect to happen
My deployment is a script that executes tasks at regular intervals and doesn't need to expose any ports, so I didn't write expose when write the yaml, expose is also not a required field on the documentation(check https://docs.akash.network/reference/sdl). But it cause error when akash tx deployment create. I checked the source code and there must be at least one service with global: true, which I think is not necessary.
I add expose and port, deployment create succeeded.
How to reproduce it

Use any SDL file no expose in services, run akash tx deployment create
Environment

client, macOS BigSur 11.4

At this time we require an exposed global service because otherwise there is no way to ever know if your service is running as an end user.


Useless "patterns" array

"patterns": [ { "include": "#keywords" } ] does not work with "match"
it currently doesn't do anything and can be safely removed
https://github.com/dustypomerleau/rust-syntax/blob/20730dff3c367cb40a7edd278fdaf0239ea50833/syntaxes/rust.tmLanguage.json#L71
https://github.com/dustypomerleau/rust-syntax/blob/20730dff3c367cb40a7edd278fdaf0239ea50833/syntaxes/rust.tmLanguage.json#L95

Would you like to submit a PR?


Cannot create index for API_REQUEST_SUMMARY

Description:
API Manager Analytics can't create primary keys for API_REQUEST_SUMMARY table. The message written in log to correct it says "Drop the existing clustered index PK_API_REQU_1E2444655FEE3F5E", but there aren't any PK object in database.
See logs in Attachment
Thanks and Regards
OS, DB, other environment details and versions:
Installation summary
Database: Microsoft SQL Server 2014
Schemas: apimreganalyticsdb, apimstatdb, apimeventdb and apimprocesseddb
API Manager Analytics: 2.1.0

By Wesley Forti from https://wso2.org/jira/browse/ANLYAPIM-156

Please refer https://wso2.org/jira/browse/ANLYAPIM-156 for logs and configs related to the issue

API_REQUEST_SUMMARY and indexes are not relevant to new analytics architecture. Hence closing.


Create MQTT tool in Java

Description
A basic intermediary tool for publishing MQTT messages will be very useful for testing purposes. Hence a simple tool will be written in Java, utilising Eclipse Paho as the messaging service.

An update to this tool will be tracked in a separate issue, as the baseline functionality qualifies for what we need during this sprint.


Audio on Linux has some problem. Also, SDL2 audio APIs are deprecated.

Is the problem "Extreme choppyness?" if so, I am experiencing this and will investigate.

It is. I just switched back to Linux actually, so I can have a look too.

621aa65858ec6eaae33f051cd0041e4d95df9354 should solve this. Hopefully it doesn't break on Mac.

Issue still present on my machine:
Linux galium 4.8.3-1-ARCH #1 SMP PREEMPT Thu Oct 20 19:53:59 CEST 2016 x86_64 GNU/Linux
Compiler:
clang version 3.8.1 (tags/RELEASE_381/final)
Computer:
Thinkpad L460
Here's a .FLAC -- you can see the audio output is really choppy.
test.zip

Ok. It solved it on mine. I'll borrow another Linux machine and experiment more.

I started to poke around a bit. When I run the APU for twice as many cycles, it seems to be better. When I run it at a lower sample rate (= lowpass filter), it seems to be much better.
https://github.com/kraln/LaiNES/tree/investigating_audio

Thanks. I won't have a lot of time to work on this till at least the 21st of December. Your contributions are super appreciated. :)

In Ubuntu, entering  export SDL_AUDIODRIVER=ALSA before running improves the audio by a lot.

Thanks, that's good to know! I'll write that in the README.


Modify badger storage reader to check for ErrTraceNotFound error

Which problem is this PR solving?

Resolves #1848

Short description of the changes

The fix is intended to fix Jaeger crashing with with nil pointer on span_id_deduper.go:57.

Based on the following log it looks like the error was due to nil error returned by private getTraces method.
jaeger-69dfb5ccb9-q64sb:2020/09/03 11:38:30 http: superfluous response.WriteHeader call from github.com/gorilla/handlers.recoveryHandler.ServeHTTP.func1 (recovery.go:73)
jaeger-69dfb5ccb9-q64sb:{"level":"error","ts":1599133110.241907,"caller":"recoveryhandler/zap.go:33","msg":"runtime error: invalid memory address or nil pointer dereference","stacktrace":"github.com/jaegertracing/jaeger/pkg/recoveryhandler.zapRecoveryWrapper.Println\n\tgithub.com/jaegertracing/jaeger/pkg/recoveryhandler/zap.go:33\ngithub.com/gorilla/handlers.recoveryHandler.log\n\tgithub.com/gorilla/handlers@v1.4.2/recovery.go:83\ngithub.com/gorilla/handlers.recoveryHandler.ServeHTTP.func1\n\tgithub.com/gorilla/handlers@v1.4.2/recovery.go:74\nruntime.gopanic\n\truntime/panic.go:969\nruntime.panicmem\n\truntime/panic.go:212\nruntime.sigpanic\n\truntime/signal_unix.go:695\ngithub.com/jaegertracing/jaeger/model/adjuster.(*spanIDDeduper).groupSpansByID\n\tgithub.com/jaegertracing/jaeger/model/adjuster/span_id_deduper.go:58\ngithub.com/jaegertracing/jaeger/model/adjuster.SpanIDDeduper.func1\n\tgithub.com/jaegertracing/jaeger/model/adjuster/span_id_deduper.go:35\ngithub.com/jaegertracing/jaeger/model/adjuster.Func.Adjust\n\tgithub.com/jaegertracing/jaeger/model/adjuster/adjuster.go:36\ngithub.com/jaegertracing/jaeger/model/adjuster.sequence.Adjust\n\tgithub.com/jaegertracing/jaeger/model/adjuster/adjuster.go:62\ngithub.com/jaegertracing/jaeger/cmd/query/app/querysvc.QueryService.Adjust\n\tgithub.com/jaegertracing/jaeger/cmd/query/app/querysvc/query_service.go:120\ngithub.com/jaegertracing/jaeger/cmd/query/app.(*APIHandler).convertModelToUI\n\tgithub.com/jaegertracing/jaeger/cmd/query/app/http_handler.go:304\ngithub.com/jaegertracing/jaeger/cmd/query/app.(*APIHandler).getTrace\n\tgithub.com/jaegertracing/jaeger/cmd/query/app/http_handler.go:385\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2012\ngithub.com/opentracing-contrib/go-stdlib/nethttp.MiddlewareFunc.func5\n\tgithub.com/opentracing-contrib/go-stdlib@v0.0.0-20190519235532-cf7a6c988dc9/nethttp/server.go:140\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2012\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2012\ngithub.com/gorilla/mux.(*Router).ServeHTTP\n\tgithub.com/gorilla/mux@v1.7.4/mux.go:210\ngithub.com/jaegertracing/jaeger/cmd/query/app.additionalHeadersHandler.func1\n\tgithub.com/jaegertracing/jaeger/cmd/query/app/additional_headers_handler.go:28\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2012\ngithub.com/gorilla/handlers.CompressHandlerLevel.func1\n\tgithub.com/gorilla/handlers@v1.4.2/compress.go:148\nnet/http.HandlerFunc.ServeHTTP\n\tnet/http/server.go:2012\ngithub.com/gorilla/handlers.recoveryHandler.ServeHTTP\n\tgithub.com/gorilla/handlers@v1.4.2/recovery.go:78\nnet/http.serverHandler.ServeHTTP\n\tnet/http/server.go:2807\nnet/http.(*conn).serve\n\tnet/http/server.go:1895"}
jaeger-69dfb5ccb9-q64sb:goroutine 44248 [running]:
jaeger-69dfb5ccb9-q64sb:runtime/debug.Stack(0x9afb52, 0xc0002df4a0, 0xc0093ec640)
jaeger-69dfb5ccb9-q64sb:    runtime/debug/stack.go:24 +0x9d
jaeger-69dfb5ccb9-q64sb:runtime/debug.PrintStack()
jaeger-69dfb5ccb9-q64sb:    runtime/debug/stack.go:16 +0x22
jaeger-69dfb5ccb9-q64sb:github.com/gorilla/handlers.recoveryHandler.log(0x1cd33a0, 0xc006d62520, 0x1cd2540, 0xc0002df4a0, 0x1, 0xc000497850, 0x1, 0x1)
jaeger-69dfb5ccb9-q64sb:    github.com/gorilla/handlers@v1.4.2/recovery.go:89 +0x6e
jaeger-69dfb5ccb9-q64sb:github.com/gorilla/handlers.recoveryHandler.ServeHTTP.func1(0x1cf0300, 0xc00015c460, 0x1cd33a0, 0xc006d62520, 0x1cd2540, 0xc0002df4a0, 0x1)
jaeger-69dfb5ccb9-q64sb:    github.com/gorilla/handlers@v1.4.2/recovery.go:74 +0xda
jaeger-69dfb5ccb9-q64sb:panic(0x12c2ae0, 0x26f0240)
jaeger-69dfb5ccb9-q64sb:    runtime/panic.go:969 +0x166
jaeger-69dfb5ccb9-q64sb:github.com/jaegertracing/jaeger/model/adjuster.(*spanIDDeduper).groupSpansByID(0xc0070d5160)
jaeger-69dfb5ccb9-q64sb:    github.com/jaegertracing/jaeger/model/adjuster/span_id_deduper.go:58 +0x48
jaeger-69dfb5ccb9-q64sb:github.com/jaegertracing/jaeger/model/adjuster.SpanIDDeduper.func1(0x0, 0xc00023f440, 0x14b2f7a, 0xc)
jaeger-69dfb5ccb9-q64sb:    github.com/jaegertracing/jaeger/model/adjuster/span_id_deduper.go:35 +0x4b
jaeger-69dfb5ccb9-q64sb:github.com/jaegertracing/jaeger/model/adjuster.Func.Adjust(0x1b2f9b0, 0x0, 0xc0003b6000, 0x0, 0x0)
jaeger-69dfb5ccb9-q64sb:    github.com/jaegertracing/jaeger/model/adjuster/adjuster.go:36 +0x30
jaeger-69dfb5ccb9-q64sb:github.com/jaegertracing/jaeger/model/adjuster.sequence.Adjust(0xc006d368c0, 0x5, 0x5, 0xc0070d5200, 0x0, 0xc006d4c910, 0x1, 0xc0070d5300)
jaeger-69dfb5ccb9-q64sb:    github.com/jaegertracing/jaeger/model/adjuster/adjuster.go:62 +0xa5
jaeger-69dfb5ccb9-q64sb:github.com/jaegertracing/jaeger/cmd/query/app/querysvc.QueryService.Adjust(...)
jaeger-69dfb5ccb9-q64sb:    github.com/jaegertracing/jaeger/cmd/query/app/querysvc/query_service.go:120
jaeger-69dfb5ccb9-q64sb:github.com/jaegertracing/jaeger/cmd/query/app.(*APIHandler).convertModelToUI(0xc006d36dc0, 0x0, 0xc006c06b01, 0x0, 0x0)
jaeger-69dfb5ccb9-q64sb:    github.com/jaegertracing/jaeger/cmd/query/app/http_handler.go:304 +0x1e4
jaeger-69dfb5ccb9-q64sb:github.com/jaegertracing/jaeger/cmd/query/app.(*APIHandler).getTrace(0xc006d36dc0, 0x1cf18c0, 0xc006c06b40, 0xc000153300)
jaeger-69dfb5ccb9-q64sb:    github.com/jaegertracing/jaeger/cmd/query/app/http_handler.go:385 +0x1a5
jaeger-69dfb5ccb9-q64sb:net/http.HandlerFunc.ServeHTTP(0xc006d4ce70, 0x1cf18c0, 0xc006c06b40, 0xc000153300)
jaeger-69dfb5ccb9-q64sb:    net/http/server.go:2012 +0x44
jaeger-69dfb5ccb9-q64sb:github.com/opentracing-contrib/go-stdlib/nethttp.MiddlewareFunc.func5(0x1cecb40, 0xc006c06960, 0xc000153100)
jaeger-69dfb5ccb9-q64sb:    github.com/opentracing-contrib/go-stdlib@v0.0.0-20190519235532-cf7a6c988dc9/nethttp/server.go:140 +0x5f6
jaeger-69dfb5ccb9-q64sb:net/http.HandlerFunc.ServeHTTP(0xc006d5c450, 0x1cecb40, 0xc006c06960, 0xc000153100)
jaeger-69dfb5ccb9-q64sb:    net/http/server.go:2012 +0x44
jaeger-69dfb5ccb9-q64sb:net/http.HandlerFunc.ServeHTTP(0xc006d4bcc0, 0x1cecb40, 0xc006c06960, 0xc000153100)
jaeger-69dfb5ccb9-q64sb:    net/http/server.go:2012 +0x44
jaeger-69dfb5ccb9-q64sb:github.com/gorilla/mux.(*Router).ServeHTTP(0xc006988600, 0x1cecb40, 0xc006c06960, 0xc000152f00)
jaeger-69dfb5ccb9-q64sb:    github.com/gorilla/mux@v1.7.4/mux.go:210 +0xe2
jaeger-69dfb5ccb9-q64sb:github.com/jaegertracing/jaeger/cmd/query/app.additionalHeadersHandler.func1(0x1cecb40, 0xc006c06960, 0xc000152f00)
jaeger-69dfb5ccb9-q64sb:    github.com/jaegertracing/jaeger/cmd/query/app/additional_headers_handler.go:28 +0x190
jaeger-69dfb5ccb9-q64sb:net/http.HandlerFunc.ServeHTTP(0xc006d62500, 0x1cecb40, 0xc006c06960, 0xc000152f00)
jaeger-69dfb5ccb9-q64sb:    net/http/server.go:2012 +0x44
jaeger-69dfb5ccb9-q64sb:github.com/gorilla/handlers.CompressHandlerLevel.func1(0x1cf0300, 0xc00015c460, 0xc000152f00)
jaeger-69dfb5ccb9-q64sb:    github.com/gorilla/handlers@v1.4.2/compress.go:148 +0x17b
jaeger-69dfb5ccb9-q64sb:net/http.HandlerFunc.ServeHTTP(0xc006d62520, 0x1cf0300, 0xc00015c460, 0xc000152f00)
jaeger-69dfb5ccb9-q64sb:    net/http/server.go:2012 +0x44
jaeger-69dfb5ccb9-q64sb:github.com/gorilla/handlers.recoveryHandler.ServeHTTP(0x1cd33a0, 0xc006d62520, 0x1cd2540, 0xc0002df4a0, 0x1, 0x1cf0300, 0xc00015c460, 0xc000152f00)
jaeger-69dfb5ccb9-q64sb:    github.com/gorilla/handlers@v1.4.2/recovery.go:78 +0xce
jaeger-69dfb5ccb9-q64sb:net/http.serverHandler.ServeHTTP(0xc006b12e00, 0x1cf0300, 0xc00015c460, 0xc000152f00)
jaeger-69dfb5ccb9-q64sb:    net/http/server.go:2807 +0xa3
jaeger-69dfb5ccb9-q64sb:net/http.(*conn).serve(0xc0003b80a0, 0x1cf63c0, 0xc0003e0000)
jaeger-69dfb5ccb9-q64sb:    net/http/server.go:1895 +0x86c
jaeger-69dfb5ccb9-q64sb:created by net/http.(*Server).Serve
jaeger-69dfb5ccb9-q64sb:    net/http/server.go:2933 +0x35c

So, additional check for spanstore.ErrTraceNotFound was added to that private method.
I have noticed TestFindNothing test in read_write_test.go which is intended to catch no error for nil trace. Probably this test shall be catching spanstore.ErrTraceNotFound error instead.

I have noticed TestFindNothing test in read_write_test.go which is intended to catch no error for nil trace. Probably this test shall be catching spanstore.ErrTraceNotFound error instead.

What are the other storage mechanisms doing?

Cassandra is also returning spanstore.ErrTraceNotFound for zero spans (see plugin\storage\cassandra\spanstore\reader.go:206).
The same does ES: plugin\storage\es\spanstore\reader.go:204.

Cassandra is also returning spanstore.ErrTraceNotFound for zero spans (see plugin\storage\cassandra\spanstore\reader.go:206).
The same does ES: plugin\storage\es\spanstore\reader.go:204.

I have noticed TestFindNothing test in read_write_test.go which is intended to catch no error for nil trace. Probably this test shall be catching spanstore.ErrTraceNotFound error instead.

What are the other storage mechanisms doing?

Cassandra is also returning spanstore.ErrTraceNotFound for zero spans (see plugin\storage\cassandra\spanstore\reader.go:206).
The same does ES: plugin\storage\es\spanstore\reader.go:204.

The test looks wrong then, right?

The test looks wrong then, right?

If you mean TestFindNothing test, then yeah, I think it needs to catch spanstore.ErrTraceNotFound. So, I will remove my new test and just edit TestFindNothing test, if there are no objections from your side.

Sounds good. This test seems to corroborate:
https://github.com/jaegertracing/jaeger/blob/0a889e8ea7ca887f30eaa814456d86ad97be3f8e/plugin/storage/es/spanstore/reader_test.go#L336-L353

@IvanBodnya advice for the future - submit PRs from a branch, not from master of your fork.

I'm closing this in favor of #2478, but I hope you had fun working with a new code base :-)
I'm looking forward to seeing a new contribution from you!

I'm closing this in favor of #2478, but I hope you had fun working with a new code base :-)
I'm looking forward to seeing a new contribution from you!

Yeah, it was a nice and very useful experience. Looking forward to contributing too ;)


Remove Deprecations

Prime opportunity to do this in Greenwich

As it's a backwards incompatible change, it will have to wait till 3.0.

Done with 3.0. Setting project to Kilburn to remove any new deprecations.


Improved UI

I can design and implement a more dynamic, seamless and attractive UI for the application.
With a proper visualization of strength.
Let me know if I should take up the issue.

yes , why not. but take in consideration that it should look as minimalistic as possible, we'll try to change it and update it if it looks better. preferred to use bootstrap.

T

https://github.com/sh-dv/hat.sh/issues/1#issuecomment-495787524


Document volume mount details for Windows machines

I had trouble figuring out why my volume mounts used by docker-compose were empty. Thanks to the help I received on this issue I realized that this is because details about mounting volumes on Windows host machines aren't mentioned in the documentation. It was only after I was pointed to a file in a GitHub repository that I managed to get my volume mounts to work.
The manual mount actions and commands need to be referenced somewhere in the documentation for docker toolbox (https://github.com/docker/toolbox/tree/master/docs) and/or the docker machine documentation (https://github.com/docker/machine/tree/master/docs). With preferably a reference to this documentation when talking about volumes in the docker-compose documentation.

https://github.com/docker/compose/issues/2548
https://github.com/docker/toolbox/issues/796
two years on!


go1.13

Pull Request Test Coverage Report for Build 44

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.529%

Totals

Change from base Build 43:
0.0%

Covered Lines:
159

Relevant Lines:
170

💛  - Coveralls


Add Gledopto GL-MC-001

adapters init.py line
Please add the following adapter definition in the Gledopto section (tested and working)
    'GL-MC-001': RGBWAdapter,           # Gledopto Zigbee USB mini LED controller RGB + CCT / RGBW / WWCW / Dimmer
Thanks!
Device description
RGB+CCT USB LED strip driver, very similar to GL-C-008 but designed for a small (~2m) RGB+CCT strip.
MQTT message with device information
Oct 23 22:01:07 domoticz npm[26624]: zigbee2mqtt:info  2019-10-24T05:01:07: Device 'GL-MC-001' is supported, identified as: Gledopto Zigbee USB LED controller RGB + CCT (GL-MC-001)
Oct 23 22:01:07 domoticz npm[26624]: zigbee2mqtt:info  2019-10-24T05:01:07: MQTT publish: topic 'zigbee2mqtt/bridge/log', payload '{"type":"pairing","message":"interview_successful","meta":{"friendly_name":"GL-MC-001","model":"GL-MC-001","vendor":"Gledopto","description":"Zigbee USB LED controller RGB + CCT","supported":true}}'

MQTT messages from device
Oct 23 22:09:10 domoticz npm[26624]: zigbee2mqtt:info  2019-10-24T05:09:10: MQTT publish: topic 'zigbee2mqtt/GL-MC-001', payload '{"state":"ON","brightness":135,"color":{"x":0.481518772529133,"y":0.414222875135951},"color_temp":407}'
Oct 23 22:09:13 domoticz npm[26624]: zigbee2mqtt:info  2019-10-24T05:09:13: MQTT publish: topic 'zigbee2mqtt/GL-MC-001', payload '{"state":"OFF","brightness":135,"color":{"x":0.481518772529133,"y":0.414222875135951},"color_temp":407}'
Oct 23 22:09:24 domoticz npm[26624]: zigbee2mqtt:info  2019-10-24T05:09:24: MQTT publish: topic 'zigbee2mqtt/GL-MC-001', payload '{"state":"ON","brightness":163,"color":{"x":0.481518772529133,"y":0.414222875135951},"color_temp":407}'
Oct 23 22:09:27 domoticz npm[26624]: zigbee2mqtt:info  2019-10-24T05:09:27: MQTT publish: topic 'zigbee2mqtt/GL-MC-001', payload '{"state":"ON","brightness":127,"color":{"x":0.481518772529133,"y":0.414222875135951},"color_temp":407}'

Done


Compilation error for array with generic struct using experimental encoding

In the TS SDK, we have the following contract:
https://github.com/FuelLabs/fuels-ts/blob/master/packages/abi-coder/test/fixtures/forc-projects/exhaustive-examples/src/main.sw
When built using experimental-new-encoding, it throws the following error:
@fuel-ts/abi-coder:pretest: error
@fuel-ts/abi-coder:pretest:     --> /Users/daniel/.forc/git/checkouts/std-9be0d6062747ea7/b30f0e83d3f3d336007e3dfce45a48a87e731345/sway-lib-core/src/codec.sw:4445:8
@fuel-ts/abi-coder:pretest:      |
@fuel-ts/abi-coder:pretest: 4443 |
@fuel-ts/abi-coder:pretest: 4444 | {
@fuel-ts/abi-coder:pretest: 4445 |     fn abi_decode(ref mut buffer: BufferReader) -> Self {
@fuel-ts/abi-coder:pretest:      |        ^^^^^^^^^^ Trait "AbiDecode" is not implemented for type "ArrWithGenericStruct<b256>".
@fuel-ts/abi-coder:pretest: 4446 |         (A::abi_decode(buffer), )
@fuel-ts/abi-coder:pretest: 4447 |     }
@fuel-ts/abi-coder:pretest:      |
@fuel-ts/abi-coder:pretest: ____
@fuel-ts/abi-coder:pretest:
@fuel-ts/abi-coder:pretest: error
@fuel-ts/abi-coder:pretest:   --> <autogenerated>:25:71
@fuel-ts/abi-coder:pretest:    |
@fuel-ts/abi-coder:pretest: 23 |
@fuel-ts/abi-coder:pretest: 24 |                     let args: (ArrWithGenericStruct<b256>,) = decode_second_param::<(ArrWithGenericStruct<b256>,)>();
@fuel-ts/abi-coder:pretest: 25 |                     let result_array_with_generic_struct: raw_slice = encode::<ArrWithGenericStruct<b256>>(__contract_entry_array_with_generic_struct(args.0));
@fuel-ts/abi-coder:pretest:    |                                                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Trait "AbiEncode" is not implemented for type "ArrWithGenericStruct<b256>".
@fuel-ts/abi-coder:pretest: 26 |                     __contract_ret(result_array_with_generic_struct.ptr(), result_array_with_generic_struct.len::<u8>());
@fuel-ts/abi-coder:pretest: 27 |                 }
@fuel-ts/abi-coder:pretest:    |
@fuel-ts/abi-coder:pretest: ____
@fuel-ts/abi-coder:pretest:
@fuel-ts/abi-coder:pretest: error: Could not generate the entry method because one of the arguments does not implement AbiEncode/AbiDecode
@fuel-ts/abi-coder:pretest: ____
@fuel-ts/abi-coder:pretest:
@fuel-ts/abi-coder:pretest:   Aborting due to 3 errors.

Potentially we need custom encoding here, however we have similarly complex cases that do compile, so I thought it would be best to confirm.

The real issue here is that the unification check is taking into consideration the generics name when it shouldn´t. A simpler example is the below code.
library;

trait A {  fn f(self); }

struct STU<T, U> {  t: T,  u: U }

impl<T, U> A for STU<T, U> where T: A, U: A
{
    fn f(self) {
        self.t.f();
        self.u.f();
    }
}

struct SUT<T, U> {
    ut: STU<U, T>,   // <- we inverted T and U here. Important
}

impl<T, U> A for SUT<T, U> where T: A, U: A,
 {
    fn f(self) {
        self.ut.f();
    }
}

This fails with
21 | fn f(self) {
22 |         self.ut.f();
   |                 ^
                No method named "f" found for type "STU<U, T>".

But we can make it compile just by renaming the impl generics arguments. Example:
impl<X, Y> A for SUT<X, Y> where X: A, Y: A,
 {
    fn f(self) {
        self.ut.f();
    }
}

Ah that is interesting, yes that will work for us, thanks!


Make python mixins usage safer (and telling linter to shut up)

Consider this toy mixin:
class MyMixin:
    def foo(self):
        return self.x + self.y

It is to be used along with classes that actually provide an x and a y. But:

The linter doesn't know that, so complains
I can make a class, and an instance thereof, and only get a complaint, in the form of a run time AttributeError, when I try to use foo (see next code)

class A(MyMixin):
    def __init__(self, x=0):
        self.x = x

a = A(10)  # no problem!
a.foo()  # problem: AttributeError: 'A' object has no attribute 'y'

Instead, I'd like

the linter not to complain about missing x and y in MyMixin
to be notified when I'm using it without providing the necessary attributes (this notification could come in the form (all optional) of linter, syntax error when making A, and/or when instantiating A)

I feel meta-classes can some of this, but can already hear the tales of woes when mentioning the meta-monster.
Any other suggestions?

Posted here: https://stackoverflow.com/questions/66973562/make-python-mixins-usage-safer-and-telling-linter-to-shut-up


Add test and lint workflows

Fixed in https://github.com/toltec-dev/build/commit/666d718f95c9a75aaa1ffc26a503080e1930d633.


chore: remove lint-staged

lint-staged causes the unstaged hunks inside partially-staged files to be staged as well. This may cause people (especially myself) to accidentally commit changes they did not intend to commit. This PR removes lint-staged.

Thanks for the feedback, added the workaround and tested it. It works correctly.


Scheduled batch 2 bulk export job and binary delete

Implement new chain of dao methods needed to retrieve jobs of a certain status, cutoff, and paging params
Overhaul relational DB scheduled bulk export purge to work with batch 2

Closes https://github.com/hapifhir/hapi-fhir/issues/4500

Requires changelog and an update to the version (but i think you need that)

I did include a changelog and I don't know why you think I need to update the version.  This doesn't break cdr.


Improve the Visual Studio documentation with this repository's example

The documentation detailing how to install the (Visual Studio's) Build Tools in a container (https://learn.microsoft.com/en-us/visualstudio/install/build-tools-container?view=vs-2019#create-and-build-the-dockerfile) could be improved with elements described in the (vcbuildtools' Dockerfile)
First, the vcbuildtools' Dockerfile offers a powershell based alternative, which would complement the cmd's based Dockerfile offered on the Visual Studio's documentation page.
Second, the vcbuildtools' Dockerfile present a mecanism to configure the environment that is not dependent on the ENTRYPOINT.
Could that be communicated to the proper team?

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


couldn't find suitable file to run  about security-demo

Hi, I have tried to run this example .But what do you mean by

since I could find nothing to run  under /app-binaries/cloud/ folder
.
could anyone help me to explain clearly how to test device application?thanks a lot.

cc @trilokgm

@congfairy , can you run build-release package? build-release script needs to be executed from the clone directory. The scripts builds all required applications and copies to release folder. Later this folder should be copied to /opt/ and renamed to /opt/spire/ . I will update the same in readme. Let us know if any more issues are seen.

Please try the latest example to confirm this issue exists or not, it works at my env, thanks :)
If it still exists, please reopen it!
/close

Please try the latest example to confirm this issue exists or not, it works at my env, thanks :)
If it still exists, please reopen it!
/close


Dls 9428 summary amount

Update Correct Return CYA page to have Summary Section and Check Changes page to have Balance

too many changes to track with both recent merges into main.  Will pull clean copy of main and add work to it


Keyboard Usage

Support full keyboard usage, without mouse

Press Enter for full-page preview
Press / go to search field
Support j / k for navigation [Low]
Load More Button can be focusTraversable

Released v1.1.0


Fixing crossfade issues

I'm using momentum around a couple Template.dynamics. The "fade" option is glitchy in this case because the new element is added to the DOM before the old is removed. I don't think a true crossfade is possible unless both elements are absolutely positioned?
I solved my issue by overriding the fade to wait until hiding is done before showing:
Momentum.registerPlugin('fade', function(options) {
  return {
    insertElement: function(node, next) {
      $(node)
        .hide()
        .insertBefore(next)
        .velocity('fadeIn', {delay: 500});
    },
    removeElement: function(node) {
      $(node).velocity('fadeOut', function() {
        $(this).remove();
      });
    }
  }
});

Other than posting this for the benefit of others, I'm also wondering if anyone can think of a better solution and/or if this type of fade should be a built-in option. Could also have achieved this with css I suppose.

Hey @aldeed . Not sure what issue you're addressing, could you post a reproduction? In the example, replacing the growl plugin with the fade one works ok - it may not be the nicest animation but it isn't glitchy.

@zol see here. You can uncomment the fix code to see how it changes.
I tried with growl and it has the same problem.

Ahh, I see, thanks @aldeed . The real issue here is that the fade plugin would need to know whether its doing an element replacement or not. The code you linked to is a reasonable solution to your specific problem but its not really correct as a general solution. Any inserted element's animation would be delayed by 500ms.
I don't think we can find out from uihooks whether a replacement is happening.
In any case, momentum is still very much an idea rather than a package that is being maintained for public consumption. Users of the package are encouraged to write their own plugins to implement logic specific to their applications. I'm still not convinced that putting more logic and complexity in the plugins momentum ships with to address a larger range of real world cases is a worthwhile endeavor.

Yeah, I think I agree with that. Just wanted to make sure I wasn't missing something. Thanks!


Setting cursor step in tracker UI does not currently do anything

Setting cursor step in tracker UI does not have effect.

Tested with build: https://github.com/vsariola/sointu/actions/runs/10502605949

Confirmed. Oops. Fix should be easy fortunately.


Red-Black tree does not always properly rotate during insertion of new node

Depending on the order of insertion, a red-black tree may act exactly as a standard Binary Search Tree.  This essentially defeats the purpose of the Red-Black tree!

Fixed in commit '9e9f7294c6ab55a7c085a1bbda274200d9e392a9'


Python: Review Examples in Learning Documentation

The following files need review:

[x] Quick start guide

[x] Supported languages

[x] Overview of kernel

[x] Understanding plugins

[x] Creating semantic functions

The following files need code samples:

[ ] Creating native functions
[ ] Chaining functions
[ ] Planner
[ ] out-of-the-box plugins

Closing as document now have python code samples


Add select account parameter for google authorization

Useful for choosing an account if a default account is outside the scope of a G Suite organization.

@ssddanbrown I added an env switch for this called GOOGLE_SELECT_ACCOUNT, and I check if it is true before adding that parameter. Let me know if this is sufficient for this feature to be added.

@ssddanbrown I made the requested changes, and can't figure out why the checks have failed. I tested in my development environment and the changes work as intended. Any ideas?

@justein230 Looks good. The tests being broken was totally my fault. I'd left some broken bits in master branch for a while so nothing to sorry about with your code.  Will merge in now for the next release.


fix page position on navigation

Task/Issue URL: https://app.asana.com/0/715106103902962/1205688349294686/f
Tech Design URL:
CC:
Description:
Fixes the following bugs:

page position would be forgotten on navigation
scrolling would stop when user scrolls the webview and the bars are being hidden

Steps to test this PR:
On both iPhone and iPad, ideally on a device:

Browse the web and use backwards and forwards navigation.  The page position should not change on navigation.
Check different orientations
Check keyboard adjusts the webview correctly as needed (when entering fields on the page)
Check the browsing menu appears in the expected location

On iPhone specifically (or iPad when in small mode on split view):

Check browsing with top and bottom address bar positions (bottom bar position not available on iPad unless in “small” mode / split screen)
Check scrolling hides and shows the bar as previous, but does not stop the page scrolling
Quick swipe should hide show bars as previous
When at bottom of the screen over scrolling should show bars as previous

Just did a bit of a self review and pushed a couple of commits.  I'm definitely done now @THISISDINOSAUR when you're ready.  Thanks!

Thank you!


fix: remove ID field when enforce

Fix: #1613
Remove extra field to match current Model

Before init the Enforce, the function GetBuiltInModel will add permissionId in model text. Maybe problem cause by here.  For example, now the [role_defination] is not handle in here.
https://github.com/casdoor/casdoor/blob/ed052b0e6a8a121f99c4404932bcaef7c3389384/object/permission_enforcer.go#L257

@imp2002 Oh, I know why cause the error. The condition p.v4 != "",  But we need to add the "" to take up the field, or the size of policy is not match the model defined.


Issue about yolox_streaming_inference

Hello
I'm testing 2I640HL equipment.
Currently, I've even finished converting my custom yolo model to hef format and running it.
For the model execution, I referred to 'object_detection' guide in the hailo application code examples.
However, since the guides uses an image file as the input source, I'd like to connect the webcam to check the model results in streaming, so I tried to run the 'yolox_streaming_inference' guide.
I modified and ran only the model name, path, and labeling part in the file provided by default, but the following error occurs.
I'm attaching the revised file and error message as below,
and I'd appreciate it if you could help me.

yolo_streaming_inference.zip

Hi @Hyenni,
I believe that the error happened because of the HEF format.
Currently, the yolo HEF files are built in with what we call "NMS on Hailo", which basically is running the parts that are not loaded on the HEF on the host, when the part is wrapped with Hailo's SW.
The yolox example is older and doesn't support the newer HEF files. We will add support for a webcam in the near future.
You can of course change the object_detection.py script to support input from a camera if you like.
Regards,


Create SECURITY.md

Closes #7265
I've created the SECURITY.md file considering the report vulnerability through security advisory, which is a new github feature.
If you're interested in the GitHub's feature, it must be activated for the repository:

Open the repo's settings
Click on Code security & analysis
Click "Enable" for "Private vulnerability reporting (Beta)"

If you rather not enable it there is also the possibility to receive the vulnerability report through an email, in this case just let me know which email it would be and I'll submit the change.
Besides that, feel free to edit or suggest any changes to this document, it is supposed to reflect the amount of effort the team can offer to handle vulnerabilities.

Hi, just pinging to know how is going this PR Review. Would I need to address any changes to this policy to be a better suit for rxjs?

Hi @benlesh thanks for the return. I've rephrased the policy to not explicitly mention a deadline for public disclosure (instead asking reporters to align with you before disclosing anything about the security issue).
Besides, I've added a paragraph to remind users about the Apache Liability aspect. Let me know if you think this would be better in the beginning of the doc.

Thanks, @joycebrum .. I think this is good. I am only concerned at this point, because I'm not entirely sure I'll notice the security advisories, as I'm not conditioned to check them, nor do I know quite how they work.


refactor: tab closing logic and add close tab hotkey

fixes: #3264
Description
This PR implements the logic for a Ctrl+W or Cmd+W hotkey to close a request tab from the RequestTab component. This enhancement includes a modal prompt to save changes before closing, improving the overall user experience. I removed the Ctrl+W hotkey functionality from the hotkeys provider and refactored the handleCloseClick method to reuse existing code.
Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Before:
https://github.com/user-attachments/assets/401af86f-638b-4106-a219-5fd7395e429c
After:
Uploading 20241017-0757-23.3456865.mp4…

@Pragadesh-45 I think I went a little bit further adding the prompt for all closing actions in the tab menu context in this PR. I could incorporate the hot key functionality there if you think it is worth it

@rsilvr Yes, yes! Eventually, we can add the hotkeys for closing a tab and closing all tabs. Let me take a look at your PR


Implementing divolteStream usign DirectStream API

Hello.
I noticed that the divolteStream uses createStream method of KafkaUtils to establish a stream, however since Spark 1.3 it is recomended to use DirectStream API (createDirectStream) to read data from Kafka. Do you have any plan to re-implement divolteStream using this API?

This is a duplicate of divolte/divolte-spark#1.


[Bug] Outline Great Guardian doesn't work until an item is added/removed

Since a recent update (unclear if this was a Runelite change or a plugin change), the outline of the Great Guardian is broken. The Great Guardian isn't outlined immediately upon returning from an altar. Instead they are only outlined once an item has been first added/removed from the inventory
I believe I've traced it to the exact lines of code involved if that'll help.

For the guardian to render, this must be true. This line is fine, but will only execute onItemContainerChanged but only when isInMainRegion.
isInMainRegion is only set onGameTick and regardless of it's value changing, the previous conditions are never re-checked.

When you craft runes, you aren't in the main region so that check is false. But when you arrive back in the main region, the code isn't reevaluated when you change regions and therefore only updates when you have an inventory event.
I'm not sure the solution without testing it, but it seems like if the isInMainRegion check was changed to checkInMinigame() (assuming that function isn't too expensive), that could work. Or there might be another hook which should be used instead for when the region changes.

This same issue also affects inventory talisman highlighting due to the same reason.

I'm still getting this issue :/