A JavaScript error occurred in the main process

Metashine 3.0.0-rc.1 (Pre-release):
On startup I get the following error:
A JavaScript error occurred in the main process
Uncaught Exception:
Error:
...tmp.node is not a valid win32 application.
trnp.node at process.func [as dlopen]
...

Before I address the issue.. OMG I GOT AN ISSUE!!! I am so happy that someone wants to use my app! ♡
And as to the issue: It seems like i completely missed the fact that one of the modules that is written in rust needs to be compiled for different platforms (Windows in your case). I will try to fix that tomorrow.
As a temporary workaround you can use a version of Metashine before 3.0.0 or open it in a Linux VM.

I'm a bit late, sry 😃
I downloaded the exe and installed Metashine again, but I still get the same error.

Well that sucks... I suspect i might need a windows device to build a windows version. I will look into AppVeryor CD. Will also set up a Mac version while at it.
If you would like to help you could try building from source and tell me if it works for you that way. This will confirm that something is wrong with the build step and not in the code.
If you want to build from source:

Clone the repository
Open the directory in a terminal
Execute pnpm install
Execute pnpm run build
Execute pnpm run compile or pnpm exec electron-builder -w
The installer will be found in the dist folder

To build from source you will need the folloiwng tools:

Node.js ~= 16.13
Pnpm ~= 7.9.0
Rust ~= 1.62

I tried to build, but I get the following error:
X [ERROR] Could not resolve "../dist/index.node"

    node_modules/.pnpm/file+packages+native-addon/node_modules/@metashine/native-addon/lib/index.js:1:14:
      1 │ export * from '../dist/index.node';
        ╵               ~~~~~~~~~~~~~~~~~~~~

That's strange. This means that the native-addon package failed to build. Assuming that, the build process should have failed sooner. To investigate further you can cd into packages/native-addon and try running pnpm run build or pnpm run build-release and see if that creates packages/native-addon/dist/index.node successfully

Yes, both commands work.
But still the same error afterwards when following the initial procedure.

I tried to build, but I get the following error:

Fixed this part. Since native-addon is a binary dependency it needs to be relinked to take effect (pnpm i before pnpm run build:electron). I made the necessary modification to the build script.
Please retry following the original instructions

Build and compilation work!
But on startup I now get the following error:
A JavaScript error occurred in the main process
Uncaught Exception:
Error: Cannot find module 'axios'
Require stack:
at Module._resolveFilename (node:internal/modules/cjs/loader:934: 1 5)
at Function.n._resolveFilename
at Module._load (node:internal/modules/cjs/loader:779:27)
at Function.c._load
at Module.require (node:internal/modules/cjs/loader: lü:19)
at require 18)
at (C:\Program
Files\M etash ndclo u d. I .js:39: 1 5)
at Module._compile (node:internal/modules/cjs/loader:1110: 14)
at Object.Module._extensions.js (node:internal/modules/cjs/loader: 1140:10)
at Module.load

Big news! Just managed to cross-compile the native-addon for windows and got it to load in Wine! 🎉
I think neon was copying the wrong build artifact (since i was able to reproduce the error by manually copying the wrong build artifact 😅). I will probably open an issue about this on the Neon GitHub page

on startup I now get the following error...

Electron builder explicitly states that there are some pnpm config modifications that need to be made but these modifications remove all pnpm benefits over npm so I worked around its issues a different way. Seems like it only works on my machines though...
I will create a branch with their workaround applied and ask you to test it out. If it works I will convert the project to npm

I can't wait 🤠

Managed to build the app and reach the same issue with node modules not loading.
Closing because the native-module issue seems to have been solved.
Node modules not loading is unrelated and should be discussed in #3. Thank you for your help so far ☺


[reproducer] add doc note to create CRC with proper size

As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

/approve

/lgtm


ci: release dependencies in GitLab CI/CD.

create release before actually deploy pages and packages.

Done by #179


Update check-checksums.rb

In response to your reply I created a new option (with -H) that accepts a file containing a hash instead of a hash, whereas the -h option that you had remains unchanged so it doesn't disturb anyone's current configurations.

@traviswdev  I think this looks fine and is a much better solution.  Could you update the CHANGELOG as well please and then we can take a look at this.
Also look at the Travis output to see what is going on.
Thanks

The Travis CI whitespace errors beat me up. :) All done.


Lets import dangerfiles on gitlab via its name again (instead of forcing a project_id)

This PR fixes a regression introduced in #1128, reported in #1130.
It also reverts from using
danger.import_dangerfile(gitlab_project_id: 42)

back to
danger.import_dangerfile(gitlab: 42)
danger.import_dangerfile(gitlab: "org/repo")

So now old Dangerfiles using danger.import_dangerfile(gitlab: "") will work again.

can it support both?

@orta Yes. Tested it with our internal gitlab instance running the latest 11.10.4.

👍 OK, lets get this shipped then. Thanks!

so what's Now? I still waiting for something that you can make me have it.
R©ASB
On Tue, May 21, 2019, 22:34 Orta<EMAIL_ADDRESS>wrote:

can it support both?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/danger/danger/pull/1132?email_source=notifications&email_token=AMBUU52U6WODBB5UTHOSXLLPWQCBBA5CNFSM4HOLUM52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV4DTBA#issuecomment-494418308,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMBUU54Y5KC33ILTAWRYNULPWQCBBANCNFSM4HOLUM5Q
.


[PPP-4968] - Vulnerable Component: xmlbeans

:x: Build failed in 1m 22s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl \
engine,plugins/excel/core

:exclamation: No tests found!
Errors:
Filtered log (click to expand)

script returned exit code 1

:information_source:  This is an automatic message


Voice command implementations/locations?

I have the HoloToolKit added to my project and I am using the InputManager.cs to implement Navigation gestures. It was brought to my attention that I could say "Select" to emulate an AirTap gesture. I didn't even know I had voice commands implemented.
Looking over the readme.md, it states:
SpeechInputSource.cs allows you to specify keywords and keyboard shortcuts in the Unity Inspector.
ISpeechHandler.cs is the interface that a game object can implement to react to speech keywords.
Where exactly have these two scripts been implemented and how do I add more keywords to my scene? For example, "Rotate" to begin rotating my object using a voice command and removing the necessity of Navigation gestures.
All I added to my scene from the ToolKit is the InputManager prefab and the Holo Lens Input Module.
I'm not understanding how speech commands enabled right now?

"Select" is actually a reserved keyword by the Win 10 Operating System.

@StephenHodgson How is the voice command "Select" performing an Air Tap gesture? I never implemented voice commands.

Like I said, It's built into the OS, and Unity & MSFT engineered it to perform the equivalent to a click.  Even if you don't have the InputHandler and just use the build in EventSystem, you'd get a click event.

Oh, I misinterpreted that from what you said earlier. So then how to implement my own custom gestures? Would I only need the ISpeechHandler and SpeechInputSource scripts and implement the keywords/responses in the editor?

https://developer.microsoft.com/en-us/windows/mixed-reality/voice_design
Looks like there are only two reserved system commands to worry about: "Select" and "Hey Cortana"

No worries, and for custom gestures create a new script and implement the appropriate interfaces.

@StephenHodgson Not to ask for a working demo, but even after browsing through HoloAcademy 212's Voice tutorials I'm still confused. When you say appropriate interfaces could you please be specific?

Documentation.  (Sorry, not trying to be dismissive but I'm a bit busy and trying to be helpful)

@StephenHodgson Candidly I don't find it very useful aside from being able to reference the source code. The video tutorials breeze through the code and cut/paste or un-comment pre-written code. Again I apologize if it seems like I am pestering, but I am viewing the documentation and still quite lost. I am going to go ahead and see if google can't turn up some helpful examples this time. But if you have a links to examples or could copy-paste examples here that implement a simple voice gesture. Example: if a user says "Scale" it will increase the size of currently selected object

@StephenHodgson Is it possible to assign the response object and function at runtime? I have objects that are loaded/unloaded dynamically. I can add my custom script that contains the keywords to these objects at runtime no problem, but can I then fill in the editor fields at runtime too?

@StephenHodgson Can I then fill in the Response editor fields at runtime for the object and the specific function in the KeyManager script component? Link for visual of what I am attempting to describe EDIT: Added link


Attempt to fix failing MacOS CI build check

Summary of changes
Changes introduced in this pull request:

The library hwloc was causing issues when building for MacOs. Pushes to this PR are attempts to resolve these

Reference issue to close (if applicable)

Closes
Other information and links

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.connor mullett seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I dont think we need the macos CI check for now. I dont expect non-devs to need to run this on mac. Close this PR, if you agree.


old version of /etc/dnsmasq.d/10-consul exists post upgrade of 1.0.3 -> 1.1

Ansible version (ansible --version): 1.9.4
Python version (python --version): 2.7.6
Git commit hash or branch: master
Cloud Environment: gce
Terraform version (terraform version): v0.6.11

Example of /etc/dnsmasq.d/10-consul on GCE after upgrade from 1.0.3 to 1.1 (master)
#Listen on all interfaces
interface=*

addn-hosts=/etc/hosts

# Never forward addresses in the non-routed address spaces.
bogus-priv

server=<IP_ADDRESS>
server=<IP_ADDRESS>

# configure DNS resolution to consul servers
server=/consul/<IP_ADDRESS>#8600

server=/consul/<IP_ADDRESS>#8600

server=/consul/<IP_ADDRESS>#8600

server=/consul/<IP_ADDRESS>#8600

server=/c.asteris-mi.internal/<IP_ADDRESS>

It should be the version installed by mantl-dns: https://github.com/asteris-llc/mantl-packaging/blob/master/mantl/mantl-dns/10-consul

@BrianHicks any thoughts on the best way to handle this? should we just move 10-consul out of the way before updating mantl-dns as part an upgrade? Or is this something that could be handled in the package itself?

The package could handle it, conceivably, but it would probably be better to handle it in the playbook for a smooth transition. Ansible used to control this, so Ansible should relinquish control.
For debugging purposes, if the file isn't removed yum will create /etc/dnsmasq.d/10-consul.rpmnew


Add missing type export conditions and index.d.cts

While the library has types, it's missing the proper conditional types exports in package.json that causes any project using node16 or bundler module resolution in Typescript to fail to build:
Could not find a declaration file for module '@dice-roller/rpg-dice-roller'. <EMAIL_ADDRESS>implicitly has an 'any' type.
  There are types at<EMAIL_ADDRESS>but this result could not be resolved when respecting package.json "exports". The '@dice-roller/rpg-dice-roller' library may need to update its package.json or typings.

You can also see the types failing to resolve here:

This PR adds in the missing types condition to exports (and a required index.d.cts that's just a copy of index.d.ts) so that Typescript is happy and can resolve types and compile:

Note that you'll still need skipLibCheck: true in tsconfig.json regardless of your module settings until #247 is fixed though.

This has just been released in v5.4.1


zill balance not detected on zillswap.io

i already have 5000 zill tokens in zillpay from faucet, but in zillswap -. so i can't do swap

I have same problem


Yet Another OSL Fix

This should fix ticket #1176.

I'm wondering if we can remove all the output parameters.
We are not using them and we are not connecting them to other shaders.
It's almost sure that they will be optimized away by OSL.

@est77 if they will be optimized away unless connected, why would you want to remove Maya features? You can use the center position, to connect to a say, distance node to drive a color or other quantity.
There are much more parameters in the node(s), but they are hidden, and most are pretty useless for us.


Fix startup logic so a new user gets a fresh database

Codecov Report

Merging #1196 (8561d79) into master (8f00f92) will not change coverage.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1196   +/-   ##
=======================================
  Coverage   65.41%   65.41%
=======================================
  Files         171      171
  Lines       18400    18400
  Branches     3973     3973
=======================================
  Hits        12037    12037
  Misses       6363     6363

Files
Coverage Δ

src-electron/main-process/startup.js
43.26% <ø> (ø)


Format yaml files correctly starting with ---

This is very important for people that want to integrate
these files in scripts and playbooks that call:
cat *.yaml | kubectl apply -f -

The current syntax is robust against kubectl apply -f .
but a lot of scripts out there use the former solution
because for example in the folder they have also other files
that are not yaml.

This fails because all manifests in contrib/kube-prometheus/manifests are auto-generated and checked in CI, that they are in the expected state. I don't think that adding arbitrary dividers makes sense here, just to make one of the possible scripts work, especially as we already have a script that does exactly this without having to modify the manifests themselves:
https://github.com/coreos/prometheus-operator/blob/d1d3ab12dec3c67ab12f531e53dba8772d7fd863/hack/concat-kubernetes-manifests.sh#L6
I recommend just using that :slightly_smiling_face: .


Wasm threads release update post

Announces release of WebAssembly threads in Chrome 70
Target Live Date: 2018-10-29

[X ] This has been reviewed and approved by Wasm team
[X] I have run npm test locally and all tests pass.
[X] I have added the appropriate type-something label.
[X] I've staged the site and manually verified that my content displays correctly.

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


500 error returned in accessing unexisting server - v0.18.0-alpha.2

Version
v0.18.0-alpha.2
all k8s modules used in k8s dashboard has been updated to above.
Condition
http://<IP_ADDRESS>:8080 does not exist.
Expected result:
Get http://<IP_ADDRESS>:8080/api/v1/namespaces/kube-system/services/heapster/proxy/healthz: dial tcp <IP_ADDRESS>:8080: connect: connection refused
Observed result
an error on the server ("") has prevented the request from succeeding
https://travis-ci.org/kubernetes/dashboard/jobs/651408636#L715

please help! v0.17.3 seems to work. what's difference in v0.18?

this is caused by https://github.com/kubernetes/kubernetes/issues/87510. It is considered release blocking for 1.18 and can be tracked there
/close

@liggitt Thanks!!


Fixes swords not being able to be used to make bats

About the PR

Gives BaseSword the Slicing quality from ToolComponent so that it can be used to make bats, just like knives can be used in this way
Why / Balance

Technical details

yaml
Media

Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl:

fix: You can now use swords to make baseball bats.

Yay, Plykiya is back from playing the game and is making bugfixes instead!
You really want to hit that 400 PR count :)

I WILL, AT LEAST LET THE TESTS FINISH FIRST!


ingest-geoip plugin installs in incorrect location

It would seem that, when I install the ingest-geoip plugin, it installs the files in /etc/elasticsearch/ingest-geoip, since the puppet process runas as root.
However, since ES-HOME for the instance is set as /etc/elasticsearch/es-01 (I use the instance name of es-01 just to be simple)... it appears that elasticsearch, when it starts is looking for the files at /etc/elasticsearch/es-01/ingest-geoip, and thus it fails to start as a result.
Right now, i'll do a exec workaround to copy the files to the right location, but what suggestions would there be in terms of solving this issue?
java.lang.RuntimeException: java.nio.file.NoSuchFileException: /etc/elasticsearch/es-01/ingest-geoip         at org.elasticsearch.ingest.geoip.IngestGeoIpPlugin.getProcessors(IngestGeoIpPlugin.java:74) ~[?:?]         at org.elasticsearch.ingest.IngestService.<init>(IngestService.java:58) ~[elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.node.Node.<init>(Node.java:354) ~[elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.node.Node.<init>(Node.java:245) ~[elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.bootstrap.Bootstrap$5.<init>(Bootstrap.java:233) ~[elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.bootstrap.Bootstrap.setup(Bootstrap.java:233) ~[elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.bootstrap.Bootstrap.init(Bootstrap.java:342) [elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.bootstrap.Elasticsearch.init(Elasticsearch.java:132) [elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.bootstrap.Elasticsearch.execute(Elasticsearch.java:123) [elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:70) [elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.cli.Command.mainWithoutErrorHandling(Command.java:134) [elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.cli.Command.main(Command.java:90) [elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.bootstrap.Elasticsearch.main(Elasticsearch.java:91) [elasticsearch-5.6.3.jar:5.6.3]         at org.elasticsearch.bootstrap.Elasticsearch.main(Elasticsearch.java:84) [elasticsearch-5.6.3.jar:5.6.3] Caused by: java.nio.file.NoSuchFileException: /etc/elasticsearch/es-01/ingest-geoip
elasticsearch::plugin { 'ingest-geoip':
module_dir => 'ingest-geoip',
instances  => 'es-01',
}

Yep, I can reproduce this as well. Will look into it, thank you for the report 👍

This issue is still valid.
I have this:
  Elasticsearch::Plugin { instances => [$instance_name], }
  elasticsearch::plugin { 'ingest-geoip': }
  elasticsearch::plugin { 'ingest-user-agent': }

where $instance_name = 'master'
root@es1-new:~# ls -l /etc/elasticsearch/ingest-geoip/
total 70912
-rw-rw---- 1 root elasticsearch  5860547 Oct 28 21:39 GeoLite2-ASN.mmdb
-rw-rw---- 1 root elasticsearch 63478091 Oct 28 21:39 GeoLite2-City.mmdb
-rw-rw---- 1 root elasticsearch  3272131 Oct 28 21:39 GeoLite2-Country.mmdb
root@es1-new:~# ls -l /etc/elasticsearch/master/ingest-geoip/
total 0

So, ingest-geoip doesn't get installed correctly to the instance's ES_CONF_PATH /etc/elasticsearch/master here)


Vehicle Stats save loop for Garage module

If I were you I would work on making it configurable for either a hit save basis or individual hitpoint damage basis. e.g. save array of all the individual damage values to each part.
https://community.bistudio.com/wiki/getHitPointDamage
^ Reference if you're wondering

We already save all hitpoints as an array. See https://github.com/RPFramework/RPFramework/blob/development/RPFramework.Altis/Functions/core/Vehicle/fn_vehicleHitGet.sqf but I also noticed that we are calling it from the server even though it's on the client's side 😂 fuck.

i understand there is vehicleHitGet function which saves hit data into an array, but this is a saving loop utilizing that function so vehicles' hitpoint data gets saved to the DB.

I personally think it’s unneeded to loop hit damage. It should only be saved to the database upon disconnecting from the server, storing the vehicle.

@Haizanjs Well you now have a config toggle for that so you can disable it if you wish to do so.

Just seems extremely unneeded to save it this way.

Feel free to disable it on your end. I bet someone would love to have this feature in so there it is.

@kerkkoh I'll let people be people then ;P


Query: Sum over subquery produces incorrect SQL

//Model
public class MyContext : DbContext
{
    public DbSet<Inventory> Inventory { get; set; }
    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
    {
        optionsBuilder.UseSqlServer(@"Server=(localdb)\mssqllocaldb;Database=_ModelApp;Trusted_Connection=True;");
    }

    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {

    }
}

public class Purchase
{
    public Purchase()
    {
        Inventory = new List<Inventory>();
    }

    public int Id { get; set; }
    public int Count { get; set; }
    public IList<Inventory> Inventory { get; }
}

public class Inventory
{
    public int Id { get; set; }
    public int PurchaseId { get; set; }
    public int? SaledCount { get; set; }
    public Purchase Purchase { get; set; }
}
```
Repro code:
```
using (var ctx = new MyContext())
{
    var query = (from p in ctx.Set<Purchase>()
                    let count = p.Count - (int)(p.Inventory.Select(i => i.SaledCount).Sum() ?? 0)
                    where (count > 0)
                    select count).Sum();
}
```
Exception: `System.Data.SqlClient.SqlException : Cannot perform an aggregate function on an expression containing an aggregate or a subquery.`
StackTrace:
```
Cannot perform an aggregate function on an expression containing an aggregate or a subquery.
	at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
	at System.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
	at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
	at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
	at System.Data.SqlClient.SqlDataReader.TryConsumeMetaData()
	at System.Data.SqlClient.SqlDataReader.get_MetaData()
	at System.Data.SqlClient.SqlCommand.FinishExecuteReader(SqlDataReader ds, RunBehavior runBehavior, String resetOptionsString)
	at System.Data.SqlClient.SqlCommand.RunExecuteReaderTds(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, Boolean async, Int32 timeout, Task& task, Boolean asyncWrite, SqlDataReader ds, Boolean describeParameterEncryptionRequest)
	at System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, String method, TaskCompletionSource`1 completion, Int32 timeout, Task& task, Boolean asyncWrite)
	at System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, String method)
	at System.Data.SqlClient.SqlCommand.ExecuteReader(CommandBehavior behavior, String method)
	at System.Data.SqlClient.SqlCommand.ExecuteDbDataReader(CommandBehavior behavior)
	at System.Data.Common.DbCommand.ExecuteReader()
	Storage\Internal\RelationalCommand.cs(275,0): at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.Execute(IRelationalConnection connection, String executeMethod, IReadOnlyDictionary`2 parameterValues, Boolean openConnection, Boolean closeConnection)
	Storage\Internal\RelationalCommand.cs(140,0): at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.ExecuteReader(IRelationalConnection connection, IReadOnlyDictionary`2 parameterValues, Boolean manageConnection)
	Query\Internal\QueryingEnumerable.cs(81,0): at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable.Enumerator.MoveNext()
	Query\QueryMethodProvider.cs(115,0): at Microsoft.EntityFrameworkCore.Query.QueryMethodProvider.GetResult[TResult](IEnumerable`1 valueBuffers)
	at lambda_method(Closure , QueryContext )
	Query\Internal\QueryCompiler.cs(200,0): at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.<>c__DisplayClass19_1`1.<CompileQuery>b__1(QueryContext qc)
	Query\Internal\QueryCompiler.cs(98,0): at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.Execute[TResult](Expression query)
	Query\Internal\EntityQueryProvider.cs(57,0): at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryProvider.Execute[TResult](Expression expression)
	at System.Linq.Queryable.Sum(IQueryable`1 source)
	QuerySqlServerTest.cs(5457,0): at Microsoft.EntityFrameworkCore.SqlServer.FunctionalTests.QuerySqlServerTest.Test()
```
Generated SQL:
```
SELECT SUM([p].[Count] - COALESCE((
    SELECT SUM([i0].[SaledCount])
    FROM [Inventory] AS [i0]
    WHERE [p].[Id] = [i0].[PurchaseId]
), 0))
FROM [Purchase] AS [p]
WHERE ([p].[Count] - COALESCE((
    SELECT SUM([i].[SaledCount])
    FROM [Inventory] AS [i]
    WHERE [p].[Id] = [i].[PurchaseId]
), 0)) > 0
```

Query generated with EF6
SELECT
    [GroupBy3].[A1] AS [C1]
    FROM ( SELECT
        SUM([Filter3].[A1]) AS [A1]
        FROM ( SELECT
            [Project2].[Count] - (CASE WHEN ([Project2].[C1] IS NULL) THEN 0 ELSE [Project2].[C2] END) AS [A1]
            FROM ( SELECT
                [Project1].[Count] AS [Count],
                [Project1].[C1] AS [C1],
                (SELECT
                    SUM([Extent3].[SaledCount]) AS [A1]
                    FROM [dbo].[Inventories] AS [Extent3]
                    WHERE [Project1].[Id] = [Extent3].[PurchaseId]) AS [C2]
                FROM ( SELECT
                    [Extent1].[Id] AS [Id],
                    [Extent1].[Count] AS [Count],
                    (SELECT
                        SUM([Extent2].[SaledCount]) AS [A1]
                        FROM [dbo].[Inventories] AS [Extent2]
                        WHERE [Extent1].[Id] = [Extent2].[PurchaseId]) AS [C1]
                    FROM [dbo].[Purchases] AS [Extent1]
                )  AS [Project1]
            )  AS [Project2]
            WHERE ([Project2].[Count] - (CASE WHEN ([Project2].[C1] IS NULL) THEN 0 ELSE [Project2].[C2] END)) > 0
        )  AS [Filter3]
    )  AS [GroupBy3]

dupe of #3792


I have got a new error while I want to make launch after making PythonAPI

Hi @GustavoSilvera
I want to install the DReyeVR and when I make launch afet making pythonAPI, I have got this error. Could you help me?
when I did this

I did not face that I issue
Thanks

What version of the dreyevr code are you using?

I do not know.
Also I have another issue:

I run "python DReyeVR_logging.py," but when my experiment is finished, what should I do to collect the data? I mean, where is it saved?

Can you provide the full paths of your built package and the DReyeVRConfig file you are editing?

thanks alot.
do you mean this:

thanks alot. do you mean this:

Can you provide the full paths of your built package and the DReyeVRConfig file you are editing?

Thanks a lot.
Do you mean this:

If you want it to change in the package, you should modify the corresponding DReyeVRConfig. What is the path to the binary you are running that is built via make package?

This looks like the config file in the dreyevr source. If you want it to change in the package, you should modify the corresponding DReyeVRConfig. What is the path to the binary you are running that is built via make package?

I modify the code here and change the "DrawFlatReticle=False; reticle in flat-screen mode" to False. then in carla root I make a package.

This looks like the config file in the dreyevr source. If you want it to change in the package, you should modify the corresponding DReyeVRConfig. What is the path to the binary you are running that is built via make package?

I am really sorry. I am not good at computer since I am civil engineer. In the DReyeVRConfig.h file I just change this line then in X64 in the carla root I execute make package.

In your carla_root/Build\UE4Carla\0.9.13-*-*\WindowsNoEditor\CarlaUE4\Config you will see the DReyeVRConfig file that you need to modify in order to make changes to your built version of DReyeVR. You do not need to re execute make package.

In your carla_root\Build\UE4Carla\0.9.13-*-*\WindowsNoEditor\CarlaUE4\Config you will see the DReyeVRConfig file that you need to modify in order to make changes to your built version of DReyeVR. You do not need to re execute make package.

Thank you so much.
Could you help me how to save the all simulator data as well.

Please read our docs: https://github.com/HARPLab/DReyeVR/blob/main/Docs/Usage.md#recordingreplaying-a-scenario

Please read our docs: https://github.com/HARPLab/DReyeVR/blob/main/Docs/Usage.md#recordingreplaying-a-scenario

Thanks but I saw in other issue "How to acquire eye tracking data maybe in a better way?
#81" GustavoSilvera mentioned used this code "python PythonAPI/examples/DReyeVR_logging.py" for gathering better eye gaze. Therefore, I overwhelmed.
Sorry for my basic question.

That is for getting eye tracking data in real time. If you just want to store the experiment to disk and replay or analyze it later, please use the resources in the Usage.md file.

thanks I found it in this directory:

is it correct

I just meant the resources documented in this link: https://github.com/HARPLab/DReyeVR/blob/main/Docs/Usage.md#recordingreplaying-a-scenario

I just meant the resources documented in this link: https://github.com/HARPLab/DReyeVR/blob/main/Docs/Usage.md#recordingreplaying-a-scenario

Thanks a lot.

Hi @ajdroid  I come back with another question
I execute python start_recording.py -n 0 in screen mode

then execute this
python show_recorder_file_info.py -a -f C:\test1.log > recorder.txt
then with DReyeVR-parse

show me this error.
Also I have another issue when I run the following after python start_recording.py -n 0 during my experiment it gives me the self error.

Hi @ajdroid I have another issue. However I am in VR mode it shows the gaze and pupil zero!!! COULD YOU HELP ME

Hi @FBanani please create a new issue if you have an unrelated problem to the main thread title. I am closing this issue.


blueprintjs rendered keyboard navigation is broken

Describe the bug
Keyboard navigation doesn't work as expected when using the blueprintjs renderer.
When the tree gets focus by tabbing with the keyboard, pressing down and up just scrolls the page instead of navigating the tree.
If you select a node with the mouse first, then navigation sort of works, but it seems to get stuck on expandable nodes.
To Reproduce
Steps to reproduce the behavior:
Go to https://rct.lukasbach.com/docs/guides/blueprintjs and try to use keyboard navigation.
Expected behavior
The keyboard navigation should work as in other examples.
Desktop (please complete the following information):

OS: Ubuntu 20.04
Browser Firefox
Version 98

I also tried adding the renderers as suggested in the docs to a sandbox and it crashes:
https://codesandbox.io/s/react-complex-tree-playground-forked-cs3i6k?file=/src/App.tsx

Error
Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
Check the render method of TreeItem.

I believe the blueprintjs related issue was fixed by your PR, and the incorrect keyboard navigation jumps should be fixed as well through another fix, can you please verify?


Strange lint output

Error is given as   Parsing error: Unexpected token =
static propTypes = {
definition: PropTypes.object.isRequired,
typeDefinition: PropTypes.object.isRequired
}

Yup, same here - standard-loader seems to execute all the code before linting for some reason

Your code is not ES6

👍

Just need to configure standard to use the babel-eslint parser.
  standard: {
    // config options to be passed through to standard e.g.
    parser: 'babel-eslint'
  }


Improve endpoint and DNS proxy lock contention during bursty DNS traffic

Upon a successful DNS response, Cilium's DNS proxy code will sync the
DNS history state to the individual endpoint's header file. Previously,
this sync was done inside a trigger, however the calling code,
(*Endpoint).SyncEndpointHeaderFile(), acquired a write-lock for no good
reason. This effectively negated the benefits of having the DNS history
sync behind a trigger of 5 seconds.
This is especially suboptimal because the header file sync is actually
causing Cilium to serialize processing the DNS request for a single
endpoint.
To illustrate the impact of the above a bit more concretely, if a single
endpoint does 10 DNS requests at the same time, acquiring the write-lock
causes the processing of those 10 requests to be done one at a time. For
the sake of posterity, this is not the case if 10 endpoints were to make
DNS requests in parallel.
This obviously has a performance impact both in terms of being slow
CPU-wise, but also memory-wise. Take for example a DNS request bursty
environment, it could cause an uptick in memory usage due to many
goroutines being created and blocking due to the serialized nature of
locking.
Now that the code is all executing behind a trigger, we can remove the
lock completely and initialize the trigger setup where the Endpoint
object is created (e.g. createEndpoint(), parseEndpoint()). Now the lock
is only taken in every 5 seconds when the trigger runs.
This should relieve the lock contention drastrically. For context, in a
user's environment where the pprof was shared with us, there were around
440 goroutines with 203 of them stuck waiting inside
SyncEndpointHeaderFile().
We can also modify SyncEndpointHeaderFile() to no longer return an
error, because it's not possible for invoking the trigger to fail. If we
fail to initialize the trigger itself, then we log an error, but this is
essentially impossible because it can only fail if the trigger func is
nil (which we control).
Understanding the locking contention came from inspecting the pprof via
the following command and subsequent code inspection.
go tool trace -http :8080 ./cilium ./pprof-trace

Suggested-by: Michi Mutsuzaki<EMAIL_ADDRESS>Suggested-by: André Martins<EMAIL_ADDRESS>Signed-off-by: Chris Tarazi<EMAIL_ADDRESS>

/test

/test

/test-1.23-net-next


Android: Search for Stories

Someone mentioned this on GetSat, and I totally agree this would be wonderful, even if only for saved stories.

Should be easy enough to implement. It's built so that all you have to do is pass a query parameter to the feed/river request and you get back stories just as if they were from the normal feed. So just keep local state of a query. Clear it out when leaving the feed/folder.

Working on getting all the edge cases for this right, and discovered a web UI behaviour that I found very surprising and wanted to confirm it is working as expected:  when viewing a list of stories that have been filtered by a search query, doing a mark-all-read marks the entire unfiltered feed/river, not just the filtered subset.  Intended behaviour for the apps, too, or could mark-all-read also take the corresponding query parameter so that only what is shown in the list is marked?

Ohhh, that is an interesting idea! But I don't think I'm going to support that as I don't see it being used much. It's definitely come up a few times in the support forums, but it hasn't come up for a long while.
Although it wouldn't be too hard to handle on the backend. If people want this, though, they'll have to post on the get satisfaction forum.

Sounds reasonable.

Another question: should the query parameter work on all rivers?  On this one, I'm not seeing any effect: https://www.newsblur.com/social/river_stories?page=1&order=newest&read_filter=all&query=foo (it just returns the same stories with or without the filter.

Sorry, no search on shared stories. Again, this wouldn't be too hard to build, but I just don't see much justification. And nobody has ever asked for it.

Righto.  Just making sure I was using the API correctly.  I'll ensure shared stories views don't have the search option.

In my testing, I am noticing that searches are behaving as case-sensitive. Is this intentional? Can I pass some flag or prefix/postfix some characters to the query string by default to make search insensitive?

Now I am second-guessing myself.  The first time I did this search, it didn't return results for all capitalisations of "Asus", but now that I am running it again, it is:
https://www.newsblur.com/reader/feed/308521?order=newest&f=308521&query=Asus&page=1&include_hidden=true&read_filter=all
Between doing the two searches with the "Asus" casing, I did a search with "ASUS" as the query.  Did I somehow prime an index and change the behaviour for the second search of "Asus"?

Woooah, who knows. I'm using Elasticsearch, and just getting the mappings right was a hell of a time.
https://github.com/samuelclay/NewsBlur/blob/master/apps/search/models.py#L199-L236
And here's where I do the querying:
https://github.com/samuelclay/NewsBlur/blob/master/apps/search/models.py#L274-L280
By using the 'standard' analyzer, all terms are lowercased when searched. That's what I've read, but I believe that's true.

Implemented in #854.


Added support of FileDescriptor to prepare audio and video

Hello,
Your code look fine. I will do a test tomorrow and merge it.


About activation's back propagation  code

Recently I am reading the neon code. I found that,  in back propagation process, apply_derivative is not called by tanh, sigmoid, logistic activation class.
Instead all these functions use backend.multiply(error, pre_act, out=error) to compute deltas.
I am a little confused, it's different from my comprehension about textbook or Jack Bouvrie's paper.
In his paper, he use  derivative  to compute  deltas.
Thanks,
Xiuxia

Hi Xiuxia,
That's right there's no explicit apply_derivative during the bprop call as the derivatives for these activations are pre-computed and stored during the forward pass because we have all the info we need to compute them at that time (this allows us to speed things up and use fewer buffers).
As an example the tanh(x) derivative is 1 - tanh^2(x), so if you look in fprop_func() in tanh.py you'll see that after computing the value for tanh on the inputs and storing it in outputs, we're also squaring this output value, subtracting 1 from it, and storing this back in inputs which we'll utilize in the bprop phase.  If you look in say layers/fully_connected.py you'll see that during fprop() and bprop() calls we're passing in self.pre_act for the inputs parameter I mentioned earlier.
Hope that clears things up for you.


DEF parser does not recognize NAMECASESENSITIVE

RePlAce fails when the DEF file contains the statement "NAMECASESENSITIVE ON ;"

The NAMESCASESENSITIVE statement is only valid for LEF syntax.
I was not able to find this statement on DEF inside LEF/DEF 5.8 language manual...
(RePlAce uses LEF/DEF parser and LEF/DEF language manual in this link: https://projects.si2.org/openeda.si2.org/project/showfiles.php?group_id=84#p145)
Could you check this again?

Okay, never mind.  The keyword is "NAMESCASESENSITIVE" and I managed to spell it wrong.  Once spelled correctly, your parser flags it as obsolete in version 5.6, thus answering my other question.  So this is not an issue at all except for my own programming error, which I just fixed.


Cluster RemoteNode slowdowns query by 50 times comparing to single instance

My Environment

ArangoDB Version:        3.11.5

Deployment Mode:       Cluster

Deployment Strategy:   ArangoDB Starter

Configuration:               arangodb_database_directory: "/var/lib/arangodb/"
http__keep_alive_timeout: 15000
cluster__default_replication_factor: 3
cluster__system_replication_factor: 1
query__cache_mode: "demand"
query__tracking_with_bindvars: "false"
rocksdb__pending_compactions_slowdown_trigger:<PHONE_NUMBER>4
log__level: "debug"
rocksdb__compaction_read_ahead_size: 0
rocksdb__max_parallel_compactions: 16
rocksdb__max_subcompactions: 16
rocksdb__wal_archive_size_limit:<PHONE_NUMBER>
nossl: false
                                           Also in env args there's arango used memory override (we set it to 14GB, having 16GB of memory in each machine)

                                            (all parameters then formatted right as we use jinja template and ansible to deploy arango)

Infrastructure:               own (5 agents on separate machines, 3 machines with db-server and coordinator on each)

Operating System:        Ubuntu 20.04

Total RAM in your machine:        16GB

Disks in use:        SSD

Used Package:              Debian or Ubuntu .deb

Component, Query & Data
Affected feature:  AQL subqueries
AQL query (if applicable):
Here we perform the join of interfaces to baremetal servers (id is hash and list of connected interfaces ids is link_hashes.net_interface)
FOR server IN baremetal
    LET interfaces = (FOR net IN net_interface FILTER net.hash IN server.link_hashes.net_interface RETURN net)
    RETURN {server, interfaces}

AQL explain and/or profile (if applicable):
Query String (166 chars, cacheable: false):
 FOR server IN baremetal
     LET interfaces = (FOR net IN net_interface FILTER net.hash IN server.link_hashes.net_interface
 RETURN net)
     RETURN {server, interfaces}

Execution plan:
 Id   NodeType                  Site  Calls   Items   Filtered   Runtime [s]   Comment
  1   SingletonNode             DBS       1       1          0       0.00000   * ROOT
  2   EnumerateCollectionNode   DBS       7    6261          0       0.01628     - FOR server IN baremetal   /* full collection scan, 1 shard(s)  */
 14   RemoteNode                COOR     14    6261          0       0.38528       - REMOTE
 15   GatherNode                COOR     14    6261          0       0.00702       - GATHER   /* unsorted */
 20   SubqueryStartNode         COOR   6268   12522          0       0.03825       - LET interfaces = ( /* subquery begin */
 16   ScatterNode               COOR   6268   12522          0       0.05023         - SCATTER
 17   RemoteNode                DBS   12522   12522          0      11.21860         - REMOTE
 11   IndexNode                 DBS   12522   22731          0       0.61711         - FOR net IN net_interface   /* persistent index scan, index scan + document lookup, 1 shard(s) */
 18   RemoteNode                COOR  12522   22731          0      11.01305           - REMOTE
 19   GatherNode                COOR   6286   22731          0       0.04732           - GATHER   /* unsorted */
 21   SubqueryEndNode           COOR   6268    6261          0       0.01915           - RETURN  net ) /* subquery end */
  9   CalculationNode           COOR   6268    6261          0       0.03066       - LET #7 = { "server" : server, "interfaces" : interfaces }   /* simple expression */   /* collections used: server : baremetal */
 10   ReturnNode                COOR   6268    6261          0       0.00005       - RETURN #7

Indexes used:
 By   Name                      Type         Collection      Unique   Sparse   Cache   Selectivity   Fields       Stored values   Ranges
 11   idx_1773683048075755520   persistent   net_interface   true     false    false      100.00 %   [ `hash` ]   [  ]            (net.`hash` IN server.`link_hashes`.`net_interface`)

Optimization rules applied:
 Id   RuleName
  1   use-indexes
  2   remove-filter-covered-by-index
  3   remove-unnecessary-calculations-2
  4   scatter-in-cluster
  5   remove-unnecessary-remote-scatter
  6   splice-subqueries

Query Statistics:
 Writes Exec   Writes Ign   Scan Full   Scan Index   Cache Hits/Misses   Filtered   Requests   Peak Mem [b]   Exec Time [s]
           0            0        6261        16470               0 / 0          0      12531       16515072        23.44577

Query Profile:
 Query Stage               Duration [s]
 initializing                   0.00000
 parsing                        0.00004
 optimizing ast                 0.00001
 loading collections            0.00000
 instantiating plan             0.00002
 optimizing plan                0.00024
 instantiating executors        0.00141
 executing                     23.44411
 finalizing                     0.00167

Warnings:
 Code   Message
 1563   in function 'SORTED_UNIQUE()': array expected
 1563   in function 'SORTED_UNIQUE()': array expected
 1563   in function 'SORTED_UNIQUE()': array expected
 1563   in function 'SORTED_UNIQUE()': array expected
 1563   in function 'SORTED_UNIQUE()': array expected
 1563   in function 'SORTED_UNIQUE()': array expected
 1563   in function 'SORTED_UNIQUE()': array expected
 1563   in function 'SORTED_UNIQUE()': array expected
 1563   in function 'SORTED_UNIQUE()': array expected
 1563   in function 'SORTED_UNIQUE()': array expected

Dataset: Dataset is private
Size of your Dataset on disk: 3GB
Replication Factor & Number of Shards (Cluster only): Replication factor - 3, number of shards - 1
Steps to reproduce

Make a query profile or execute
Wait till the end
See that RemoteNode insanely slows down it all

Problem: Cluster performance of RemoteNode is poor, in single instance the same query executes 100 times faster
Expected result: Execution of query above in cluster should be not much slower comparing to a single instance with reasonable time for RemoteNode

The difference is that RemoteNode with ID 14 performs only 14 calls to produce these 6261 items, while the RemoteNode 17 performs 12522 calls. Can you try to reformulate that query so that it does not use a subquery, e.g. using COLLECT?

And if I want to join many relations to one type just like
FOR server IN baremetal     LET interfaces = (FOR net IN net_interface FILTER net.hash IN server.link_hashes.net_interface RETURN net)     LET virtual_machines = (FOR vm IN virtual_machine FILTER vm.hash IN server.link_hashes.virtual_machine RETURN vm)     RETURN {server, interfaces, virtual_machines}
Which means "I want all baremetal servers with their corresponding interfaces and virtual machines for each"
Maybe I'm wrong, but it seems not possible to reformulate the query in such way...


Prevents handling tab when the buffer is empty.

Currently the tab key does not work when the input buffer is empty. Fixed.

LGTM.


Upgrade guzzle version to fix HTTP_PROXY bug?

Links: https://httpoxy.org/
guzzle have fixed now, link: https://github.com/guzzle/guzzle/commit/9d521b23146cb6cedd772770a2617fd6cbdb1596#diff-ff73e042e738204c6da009e2ed19f783L166
So, I think that we should upgrade the version of guzzle.

This library will automatically update to the fixed version of guzzle with the current requirements.


DatePicker with Model problem

Hey, I'm Loading a Form into to Modal, and I need a Date Picker within it.
Now, I'm facing problem with Date Picker.
you can check this fiddle.
https://jsfiddle.net/jimishf/sm98aycs/

https://github.com/Dogfalo/materialize/issues/1032


Add support for SNS tagging

Is your feature request related to a problem? Please describe.
n/a
Describe the solution you'd like
Add configuration support for tagging topics
Describe alternatives you've considered
Manually tag topics
Additional context
This feels like an AWS feature which could be nicely exposed through the topic configuration API and potentially used in a myriad of ways by consumers.

TagResource action
.NET SDK

I'm going to pick this one up and see if I can get something suitable up for review

Nice! Keen to see what you come up with 👍

PR raised https://github.com/justeat/JustSaying/pull/837

#837 merged


make test for Team page more robust

even a 20s timeout is not enough ... weird.
make add explicit check for right page and then wait longer after it

just wait a bit longer


QSV stopped working in v1.23.x

I went back to v1.22.4 and QSV encodes work but on v1.23.0/1/2 the encode exits immediately with this error:
Encode failed (error 4).
HandBrake has exited.

Which processor are you using ?  Can you share the activity log ?  You can find it under the /config/ghb/ inside the container (or on the host under the directory you mapped to /config).

Closing this issue.  Please re-open if needed.


Reduce data points on horizontal axis of chart to only relevant one

At https://nft.kodadot.xyz/rmrk/detail/9251128-900D19DC7D3C444E4C-KSMG22-KUSAMAGEN_160-0000000000000001
I see the current chart has only two data points, thus others are irrelevant to see, only those for datapoint matters.

Do you mean to remove unused x axis legends ?

Yes, only leave x axis labels when there are some data points


[ADMIN] - CRUD Description

Description:
On the admin page, we should be able to create, edit and delete the description text
Acceptance:

Have a form that makes it possible to manipulate the description text

Tests:

Have a title field
Have a description field
Loads saved description

Blocked, need to do #48 first


Document how to do references in the Sphinx Guide

Like https://github.com/Qiskit/qiskit_sphinx_theme/pull/155, but for the Sphinx docs guide.

Nevermind, we misunderstood that we already have a way of doing citations: https://github.com/Qiskit/qiskit_sphinx_theme/pull/155#issuecomment-1521924284


Execution failed for task ':app:processEcomdevDebugMainManifest'

/Users/vgts/Downloads/Apmex/android/app/src/debug/AndroidManifest.xml Error:
Attribute application@label value=(APMEX ECOM DEV) from (unknown)
is also present at [org.jfrog.cardinalcommerce.gradle:cardinalmobilesdk:2.2.1-2] AndroidManifest.xml:15:9-41 value=(@string/app_name).
Suggestion: add 'tools:replace="android:label"' to  element at AndroidManifest.xml:10:5-50:19 to override.
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:processEcomdevDebugMainManifest'.

Manifest merger failed : Attribute application@label value=(APMEX ECOM DEV) from (unknown)
is also present at [org.jfrog.cardinalcommerce.gradle:cardinalmobilesdk:2.2.1-2] AndroidManifest.xml:15:9-41 value=(@string/app_name).
Suggestion: add 'tools:replace="android:label"' to  element at AndroidManifest.xml:10:5-50:19 to override.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 12s
Exception: Gradle task assembleEcomdevDebug failed with exit code 1

Check this for solution
https://github.com/DeligenceTechnologies/Braintree-Payment-Gateway-for-Flutter/issues/53#issuecomment-1000121867


Support images with raw html syntax

There is some background discussion in this Jupyter notebook issue, but the gist of the issue is that nbsphinx currently doesn't recognize images declared e.g. as

only ones that use the plain markdown syntax:
![alt](images/dashboard_files_tab.png)

This obviously precludes passing (useful) things like custom size hints.  I don't know if this is easy to fix in nbsphinx itself or if there's an underlying tool (say pandoc) that does the heavy lifting with the files... But I figured I'd mention it, it would help us use whichever syntax is more sensible in a given setting.
For now, for Jupyter we'll work around it in our docs by limiting ourselves to the ![..](..) syntax.

Currently, pandoc is used to convert markdown cells, but in the not-so-near future this will probably be changed, see #36.
nbsphinx already parses raw HTML content for alert/warning
s (see #47 and the docs), something similar could probably be done for  tags.
How is this handled in nbconvert with LaTeX output?

How is this handled in nbconvert with LaTeX output?

Also pandoc, so it may well have the same issue.

Alternatively,  tags could be converted to "real" image nodes with a pandoc filter like in #35, but I don't think this has any advantage over doing it with a Sphinx Transform.
However, yet another option would be to convert all raw HTML content (not only  tags) to raw HTML docutils nodes.
I don't know if that would work, and I don't know if that's actually a good idea, either.
This way, the images wouldn't show up in LaTeX output.

Was this resolved? I've taken to using the Jupyter notebook/iPython helpers but importing libraries just to show images in notebooks that are all Markdown is kind of kludgy.

@actsasgeek No, nothing has changed since then. Do you have any suggestions on how to proceed?

Mmh, I just ran into this issue again today, so piping in here and pinging @mpacer to see if they have any ideas...
I just checked, and running a plain jupyter nbconvert foo.ipynb with embedded <img src=... tags works fine, so the problem isn't present there.  This is the nbconvert issue with related discussion.

A bit more info... At least with current pandoc, it seems that link_attributes are supported by nbsphinx.  This means that one can write ![caption text](path/to/img.png){width="80%"} and it shows up correctly on HTML conversion.
However, in the live notebook, the extra attributes are displayed as text instead of being used (and the text is only used as alt-text, not as a caption). E.g., this:
![caption](fig/travis-free.png){width="80%"}

gives in the live notebook:

This makes me wonder: should we instead just support the pandoc-style attributes in the live notebook?  I know @mpacer has thought a lot about this, and perhaps @Carreau and @takluyver have also thoughts?
note: in the nbsphinx conversion, the alt-text shows up as a caption, which surprised me but sounds actually like a good thing, it certainly would encourage better alt-text usage, I think. This is what it looks like on the HTML output:

I'm all for consistent support, and would actually also prefer captions, though I don't want us to maintain our own markdown renderer in JS, at least not now :-)
If anything let push that for JupyterLab once it has been released and the classic notebook has been sunset. Then we'll have less things to patch.

I could be wrong but I think that this should work for direct HTML conversion, which is the focus of the issue @fprerez linked to(https://github.com/jupyter/nbconvert/issues/328), as long as the width and height are wrapped in quotes.
If this is primarily about rst intermediate output, I'll have to think about how to best solve this.

AFAICT, it works with nbconvert for HTML output because this uses mistune instead of pandoc.
When using nbconvert with RST output, the same problem appears: the  tag vanishes.
It's probably possible to detect  tags in the pandoc AST and somehow turn them into proper Sphinx/docutils images with width and height specified correctly.
I'm open to a PR that does that, but I'm not going to work on that myself.
In the long run, I want to get rid of the intermediate RST (and at the same time pandoc) anyway (#36), but for that, we need an extensible CommonMark parser for Python, which to my knowledge doesn't exist yet.
I'm probably going to work on that, but not soon.
Regarding a CommonMark extension for link attributes, I would be careful not to rush into introducing the syntax that pandoc happens to use. There is quite some discussion about this in the context of CommonMark, e.g.:
https://talk.commonmark.org/t/support-for-image-dimensions/148/1
https://talk.commonmark.org/t/consistent-attribute-syntax/272
For now, I don't see anything resembling a consensus yet.
If there are any discussions going on about CommonMark extensions for Jupyter, I'd be very interested to participate.
But as @Carreau said, this should probably be done in the context of JupyterLab.
Regarding image/figure captions, this is also a pandoc idiosyncrasy: http://pandoc.org/MANUAL.html#extension-implicit_figures
If the image appears in a paragraph of its own, the alt text is turned into a figure caption.
I think this is very surprising and frankly annoying behavior, and I think we should not try to support it in the live notebook.

Thanks @mgeier for the extra details on the caption situation - I don't really care much for it, was just surprised to notice it.
And @mpacer, this is indeed in the context of sphinx: it works great on standalone nbconvert usage, but as @mgeier indicated, that's because the pipeline is different (mistune).
It seems at this point, the most viable path forward is the (somewhat hackish) solution proposed by @mgeier, to catch  in-flight and fix them up.  I unfortunately don't have any cycles to try that right now, but will keep it in mind in case I clear up a weekend :)
Thanks for the input!!

This issue should be fixed with #438 (since the issue didn't explicitly request the align attribute).

This feature is now available in the latest nbsphinx release (version 0.7.0).


Double escaping issues

At least for some characters, there seem to be double escaping issues. For example, setting name in a payment to Test & Test results in <Nm>Test &amp; Test</Nm>.
The issue seems to be these:
https://github.com/raphaelm/python-sepadd/blob/6fd06341b8fa2b0b4fef48ba0b5c410d8d80a707/sepadd/debit.py#L103
Removing the escape() around the assignments in debit.py fixes this issue and results in <Nm>Test & Test</Nm>, so there already seems to be some builtin escaping going on. I'm just not sure if xml.sax.saxutils.escape maybe does some additional escaping that's required?
Using Python 3.6.2 and python-sepadd 1.1.0

Sorry for the late reply. This is already part of the original PySepaDD code, so nothing I've added and so I at least believe we can just remove it.


Bump quarkus-resteasy-problem extension to v2.0.0

quarkus-resteasy-problem:2.0.0 is functionally equal to v1.0.0, but is compiled against Quarkus 2.0 dependencies, so will be more stable in the long run with Quarkus 2.0

Superseded by #247


Update path for XCode 6.3

the Plugin folder is not in Application support anymore, but in /Applications/Xcode.app/Contents/PlugIns

@araid This doesn't seem to be the case for me; no custom plugin I drop into the Application Bundle's Contents/PlugIns directory gets loaded, whereas everything in the previous Application Support-based directory does.

Interesting. In my case, after the Yosemite update the original directory didn't exist anymore. Well I guess the note is here, in case it's useful to someone in the future.


Add support for the PostgreSQL hstore type

@vladmihalcea can I take this?

Sure thing. Thanks for taking care of it.

Fixed by this commit.


Fix the SQL Injection

The MySQL/PostgreSQL data checking could have the SQL injection problem, this PR tries to fix it by adding the quotes in SQL and escaping the quotes in configuration.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.11 :tada:

Comparison is base (682462a) 99.67% compared to head (450f0f0) 99.78%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #330      +/-   ##
==========================================
+ Coverage   99.67%   99.78%   +0.11%
==========================================
  Files          82       81       -1
  Lines        5503     5624     +121
==========================================
+ Hits         5485     5612     +127
+ Misses         12        8       -4
+ Partials        6        4       -2

Impacted Files
Coverage Δ

conf/conf.go
96.73% <ø> (ø)

channel/channel.go
100.00% <100.00%> (ø)

global/global.go
100.00% <100.00%> (ø)

global/probe.go
100.00% <100.00%> (ø)

probe/base/base.go
100.00% <100.00%> (ø)

probe/client/mysql/mysql.go
100.00% <100.00%> (ø)

probe/client/postgres/postgres.go
100.00% <100.00%> (ø)

probe/data.go
100.00% <100.00%> (ø)

probe/notification_strategy.go
100.00% <100.00%> (ø)

probe/result.go
100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Hi @haoel, great to see the issue is addressed. Could you open a Github security advisory for the SQL injection vulnerability we found?

@oxeye-daniel , thanks for the reminder, we have open a github security advisory at: https://github.com/megaease/easeprobe/security/advisories/GHSA-4c32-w6c7-77x4
please help review and let us know if anything is incorrect, as this is the first time we open such an advisory, thanks.

Hi @localvar thanks a lot! No problem, you can go ahead and add me as editor for the advisory so I can suggest changes

@oxeye-daniel , added


On docker image, explore idea of using a host folder for data storage

This would allow the persistence of team files (created in the live server)
Relevant when #87 (ability to create new teams) is in place

this will work on, we just need a use case


Great works! but how to stream audio and video at the same time with temporally aligned

It seems to set only one modality (video or audio) at a time?

Yes haven't worked on audio and video simultaneously. What is the use case? Will help design it.

Yes haven't worked on audio and video simultaneously. What is the use case? Will help design it.

When developing digital man's real time chat, it will need streaming audio and frame at same time. Can you design it?

Yes haven't worked on audio and video simultaneously. What is the use case? Will help design it.

When developing digital man's real time chat, it will need streaming audio and frame at same time. Can you design it?

I have the exact same use case 🤣

How exactly is the video generated in this case? Is there some sample code you can point me to for generating a simultaneous audio/video stream? Not sure how to begin on this one 😄


Changing Window.unresizable alters Window.size

Tested versions

4.2.2 stable
4.3 dev 6

System information
Godot v4.2.2.stable.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3080 (NVIDIA; <IP_ADDRESS>65) - 12th Gen Intel(R) Core(TM) i9-12900KF (24 Threads)
Issue description
Changing Window.unresizable in code changes the value returned by Window.size.
A related issue was reported in #72985.
Steps to reproduce

Create new project
Attach script to root node with the following code:

extends Node

func _ready():
	var window := get_window();
	print("Window size before: ", window.size);
	window.unresizable = !window.unresizable;
	print("Window size after:  ", window.size);

Run project

Minimal reproduction project (MRP)
BugReport.zip

I suspect this is to be expected, the change of resizable changes how the window is decorated, and that probably makes the window adjust some size and not others to maintain one size, this might also simply be an OS limitation
CC @bruvzg

Thank you for reporting, consolidating in:

https://github.com/godotengine/godot/issues/81640

See there for more details, if you think something was missed about this and it's not the same issue, please comment here and it can be reopened


geolocation API versus map location picker & address search

The W3C Geolocation API is most accurate when the user device is providing location via GPS.
It is claimed to be less accurate for other methods.
Consider using an address search and/or map location picker to allow more precision in location sharing.

This may have been completed in this fork: https://github.com/brianrathbone/piecewise
Review/test the fork and pull in if desired.


Adds tests and fixes issues with CompileLoss true/pred/loss ordering

This PR addresses the issue described here: https://github.com/keras-team/keras/issues/20346

Firstly improved the existing test_list_loss_dict_data; the previously chosen values of a and b are such that no matter the order of the losses (mae, mse) the total loss value is always the same, hiding potential issues.
Added a couple more tests to mimic what happens with functional models with multiple outputs, fitted with data coming from a dictionary.
Keeps dicts in their existing order, or output_name order, by casting them to OrderDict to circumvent optree’s sorting behaviour

Codecov Report
Attention: Patch coverage is 85.71429% with 2 lines in your changes missing coverage. Please review.

Project coverage is 57.43%. Comparing base (3d32481) to head (fb04f63).

Files with missing lines
Patch %
Lines

keras/src/trainers/compile_utils.py
85.71%
2 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (3d32481) and HEAD (fb04f63). Click for more details.
HEAD has 6 uploads less than BASE

Flag
BASE (3d32481)
HEAD (fb04f63)

keras
4
1

keras-torch
1
0

keras-tensorflow
1
0

keras-jax
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #20362       +/-   ##
===========================================
- Coverage   78.87%   57.43%   -21.45%
===========================================
  Files         512      512
  Lines       49266    49280       +14
  Branches     7953     7958        +5
===========================================
- Hits        38861    28302    -10559
- Misses       8546    19172    +10626
+ Partials     1859     1806       -53

Flag
Coverage Δ

keras
57.43% <85.71%> (-21.32%)
:arrow_down:

keras-jax
?

keras-numpy
57.43% <85.71%> (+<0.01%)
:arrow_up:

keras-tensorflow
?

keras-torch
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Thanks for the comment. Fixing the issues that came up in the tests was simple enough... I gave a go at the rebase, but the change made to multi-output models makes it a lot harder to grasp and fix.
My main issue is that the current test coverage is not tracking this bug. For tests I checked where the structure of y_true and y_pred is different things are usually in alphabetical order; and for the only test where that is not the case test_list_loss_dict_data the unfortunate choice of values is not catching the out-of-order issue. For clarity:
>>> a = losses.mean_absolute_error(y_true["a"], y_pred["a"])
>>> b = losses.mean_squared_error(y_true["b"], y_pred["b"])
>>> ops.mean(a) + ops.mean(b)
<tf.Tensor: shape=(), dtype=float64, numpy=1.0766666666666667>
>>> a = losses.mean_squared_error(y_true["a"], y_pred["a"])
>>> b = losses.mean_absolute_error(y_true["b"], y_pred["b"])
>>> ops.mean(a) + ops.mean(b)
<tf.Tensor: shape=(), dtype=float64, numpy=1.0766666666666667>

If you'd like I can keep only the test changes. Otherwise, feel free to close.

If you'd like I can keep only the test changes. Otherwise, feel free to close.

Any increase in test coverage is valuable! Please do.

@gustavoeb - Looks like tests are failing. Can you please address it?

@gustavoeb -
Can you sync to master and create an GitHub issue to reference the problem. We can then debug from there on the appropriate fix by adding a test first?


Quickly populate lookout db

What type of PR is this?
Feature
What this PR does / why we need it:
This Go tool directly inserts jobs into the Lookout database and simulates each job's state change ( from queued -> pending -> running, after that some of them randomly being failed, cancelled or succeeded ) without the need of submitting real jobs into armada which requires the whole system to be up and running.
This will be helpful in UI/API testing by simulating any desired number of submitted jobs.
Which issue(s) this PR fixes:

Fixes #1742

Assuming you have a postgres db can you write some test cases that verify this functionality?

An alternative might be to write some pure unit tests which setup a sqlite db

I think we would like to see some integration tests with the CLI.
Assuming you have a postgres db can you write some test cases that verify this functionality?
@kannon92 I am confused a little bit, can you elaborate which tests you are talking about here?

I’d like a test case that asserts that this tool works. Integration test where you insert data into Postgres and verify its there

@kannon92 I feel it is a bit messy, also there are many assumption with the test, it assumes postgres is running on localhost for example. what do you think ?

@kannon92 I feel it is a bit messy, also there are many assumption with the test, it assumes postgres is running on localhost for example. what do you think ?

Its coming along. I think this won't work if you tried to run this in the CI.  You are assuming that postgres must be started so if I tried to run this test right now it would fail unless I started postgres.
It would be ideal to have a make target that runs this test.
Ideally I'd like to do something like make populate-lookout-test and that would start postgres and then run your test cases.  And then the test would teardown postgres.

I think that preparing the postgres db will require also to setup the schema as well. I was thinking, wouldn't that be an overkill a bit ? after all, the code itself is a test utility to test out the UI.

I think that preparing the postgres db will require also to setup the schema as well. I was thinking, wouldn't that be an overkill a bit ? after all, the code itself is a test utility to test out the UI.

I found that the best approach in software engineering is to always think of tests in your definition of done.  How can you guarantee that this code will always work?  The point of tests is just make sure that your code is actually working as intended.  And you want the tests because you want to be notified that this code no longer works.

I think that preparing the postgres db will require also to setup the schema as well. I was thinking, wouldn't that be an overkill a bit ? after all, the code itself is a test utility to test out the UI.

I found that the best approach in software engineering is to always think of tests in your definition of done. How can you guarantee that this code will always work? The point of tests is just make sure that your code is actually working as intended. And you want the tests because you want to be notified that this code no longer works.

makes sense, Thank you!
did you mean at first the root Makefile or create a Makefile under db-gen

I think that preparing the postgres db will require also to setup the schema as well. I was thinking, wouldn't that be an overkill a bit ? after all, the code itself is a test utility to test out the UI.

I found that the best approach in software engineering is to always think of tests in your definition of done. How can you guarantee that this code will always work? The point of tests is just make sure that your code is actually working as intended. And you want the tests because you want to be notified that this code no longer works.

makes sense, Thank you! did you mean at first the root Makefile or create a Makefile under db-gen

Makefile at the root.

well, the current make target depends on the localdev/docker-compose , I tried to run the lookout lookout-ingester containers manually but I couldn't figure out how. I just pushed the code so I can get some help.

well, the current make target depends on the localdev/docker-compose , I tried to run the lookout lookout-ingester containers manually but I couldn't figure out how. I just pushed the code so I can get some help.

I would just add a new target that runs Postgres in Docker, much like the tests target in the Makefile, where it does:
docker run -d --name=postgres $(DOCKER_NET) -p 5432:5432 -e POSTGRES_PASSWORD=psw postgres:14.2
  sleep 3
  function tearDown { docker rm -f redis postgres; }; trap tearDown EXIT

You could add something similar(except leave out the Redis mentions) in your Makefile target, then run your unit tests that need Postgres.

Yes, but the problem I faced isn't with Postgres container, the problem is with the database migration so the container becomes ready for direct insertions, otherwise it will be just a postgres container running without any tables. I couldn't trace back which service in the docker-compose file is responsible for the database migration.

Yes, but the problem I faced isn't with Postgres container, the problem is with the database migration so the container becomes ready for direct insertions, otherwise it will be just a postgres container running without any tables. I couldn't trace back which service in the docker-compose file is responsible for the database migration and how to build it manually

Aha - yeah. Ugh. I really really really really wish our database migrations were a completely separate job from Lookout service startup.  I will look at the code and see if there's an non-messy way to run the migration logic separately.


implementation issue

1.) middleware
<?php namespace App\modules\Kagi\Http\Middleware;

use Auth;
use Closure;
use Flash;
use Illuminate\Http\RedirectResponse;
use Illuminate\Contracts\Routing\Middleware;

class AuthenticateAdmin implements Middleware
{
	/**
	 * Handle an incoming request.
	 *
	 * @param  \Illuminate\Http\Request  $request
	 * @param  \Closure                  $next
	 * @return mixed
	 */
	public function handle($request, Closure $next)
	{
//dd($request);

		if (! Auth::user()->can('manage_admin')) {
			Flash::error('Sorry, you do not have the proper permissions.');

			return new RedirectResponse(url('/'));
		}

		return $next($request);
	}
}

app/http/kernel
		'admin' => 'App\Modules\Kagi\Http\Middleware\AuthenticateAdmin',

AuthenticateAdmin is being called. I could dd the $request.
Error:
Call to undefined method Illuminate\Database\Query\Builder::can()

Which is the same error I was getting in my blade template
@if (Auth::user()->can('manage_admin'))

							{{ Lang::choice('kotoba::account.user', 2) }}

Both of my calls are getting stuck on the ->can ...
@ good news
I've got roles and permissions and sync'ing with user going good!

Could you post your User model here? The only thing I can really think of at the moment is the possibility that you're not supplying the full namespace path to the Shinobi trait:
<?php
namespace App\Modules\Users\Models;

...
use Caffeinated\Shinobi\Traits\ShinobiTrait;
...

class User extends Model
{
    use ShinobiTrait;

    ...
}

Also, I noticed your namespace for your Middleware is App\modules\Kagi\Http\Middleware;, It should be App\Modules\Kagi\Http\Middleware (capitalize Modules).
Nice module name btw :stuck_out_tongue_winking_eye:

Here"s my User Model;
<?php
namespace App\Modules\Kagi\Http\Domain\Models;

use Illuminate\Auth\Authenticatable;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Auth\Passwords\CanResetPassword;
use Illuminate\Contracts\Auth\Authenticatable as AuthenticatableContract;
use Illuminate\Contracts\Auth\CanResetPassword as CanResetPasswordContract;

use Caffeinated\Shinobi\Traits\ShinobiTrait;
use Laracasts\Presenter\PresentableTrait;

class User extends Model implements AuthenticatableContract, CanResetPasswordContract {

	use Authenticatable, CanResetPassword, PresentableTrait, ShinobiTrait;

	/**
	 * The database table used by the model.
	 *
	 * @var string
	 */
	protected $table = 'users';

	protected $presenter = 'App\Modules\Kagi\Http\Presenters\User';

	/**
	 * The attributes excluded from the model's JSON form.
	 *
	 * @var array
	 */
	protected $hidden = ['password', 'remember_token'];

// DEFINE Fillable -------------------------------------------------------
	protected $fillable = [
		'name',
		'email',
		'password',
		'blocked',
		'banned',
		'confirmed',
		'activated',
		'activated_at',
		'last_login',
		'confirmation_code'
		];

// DEFINE Relationships --------------------------------------------------
	public function roles()
	{
		return $this->belongsToMany('App\Modules\Kagi\Http\Domain\Models\Role');
	}

}

Ugh! Thanks for catching the Capitalization. OSX doesn't give a hoot about that. I wish that they"d just tell Adobe to clean up their code so that we can move on finally.
Yeah, the name :wink:
You get bonus points if you knew what it meant if you didn't look it up! Spotify has  basically no J-music ... I was actually burnt out on using J-naming conventions while in Japan but now that I'm back, I'm using them again.
All of my code is up in my L5 repository actually. I'm still committing away like mad but most of it is working. I'll be puling the Gakko module out possibly/probably ... but none of the other code.

Ok, I think I'm finally moving forward with this. I still need to test more but so far I found 2 issues that I had.

config/auth.php

//	'model' => 'App\User',
	'model' => 'App\Modules\Kagi\Http\Domain\Models\User',

I wasn't pointing to the user model that shinobiTrait is used in.

App\Modules\Kagi\Http\Domain\Models\User

// DEFINE Relationships --------------------------------------------------
// 	public function roles()
// 	{
// 		return $this->belongsToMany('App\Modules\Kagi\Http\Domain\Models\Role');
// 	}

I defined a relationship in my user model. Which is related to my Role model
	public function user()
	{
		return $this->hasMany('App\Modules\Kagi\Http\Domain\Models\User');
	}

Removing the relationship from my user model lets me use shinobi but now I don't have those relationships. off to further testing.

Closing due to inactivity~


Typo in scipy.spatial.distance.cdist docs

http://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.distance.cdist.html
https://github.com/scipy/scipy/blob/v0.17.1/scipy/spatial/distance.py#L1778-L2294
The cdist doc string defines the Mahalanobis distance without a square root. See e.g.
https://en.m.wikipedia.org/wiki/Mahalanobis_distance
The function called by cdist is itself correct and correctly documented
https://github.com/scipy/scipy/blob/v0.17.1/scipy/spatial/distance.py#L541

Thanks @innisfree. Looks like this was already noticed in gh-4815, but then lost in the discussion.
Would you like to send a PR to fix this?


support better class-based (component and directive) data and method calls

data calls
previous
template
<input nz-input [(ngModel)]="value">

get value
console.log(this.value);

after
<input nz-input>

get value
@ViewChild(NzInputDirective, {statc:true}) nzInputDirective:NzInputDirective;
console.log(this.nzInputDirective.value)

method calls
previous
template
<nz-switch [(ngModel)]="switchValue"></nz-switch>

switch value
this.switchValue = !this.switchValue;

after
template
<nz-switch></nz-switch>

switch value
@ViewChild(NzSwitchComponent, {static:true}) nzSwitchComponent:NzSwitchComponent;
this.nzSwitchComponent.toggle();

ref https://zhuanlan.zhihu.com/p/75056031

Agreed.
However, it reminds of a long-existing bug. https://github.com/NG-ZORRO/ng-zorro-antd/issues/2696


Remove the YAML parameter acceptance

In the updated theory, there is only one way to calculate the acceptance ratio assuming symmetric proposal probabilities. Previously we removed the option metropolis_eq/metropolis-eq from the YAML parameter acceptance and we decided to remove the state-state swapping scheme, which leaves only one option (metropolis) for the parameter acceptance and makes the parameter acceptance not necessary anymore. Therefore, the YAML parameter acceptance should be removed and the documentation should be updated accordingly.

The commit a3f07fc has addressed the issue.


Fallback to using the feed generation time (date the feed was last built) if an item doesn’t have an identifiable date of its own.

👍


Add example trigger showing how to auto-trigger GitOps Update pipeline from MLOps build pipeline

JIRA: https://issues.redhat.com/browse/RHOAIENG-4982
This commit is stacked upon #267, and will show the changes from that PR until it's merged. This is one approach that could be taken–I'll open a separate PR with an alternative approach.
Description

This example shows how users can choose to optionally trigger the GitOps Update pipeline at the end of a successful run of the MLOps build pipeline.
Advantages of this approach

It's optional
It doesn't require making the main build pipeline extra complex, by needing to know everything about the GitOps update pipeline (it doesn't need to know all of the parameters, or tasks; as other approaches might).
GitOps Update pipeline is still executable in different ways, and can ideally change without needing to also make changes in the build pipeline

Disadvantages of this approach

As it relies on an EventListener, an extra pod needs to be running all the time
Currently an additional trigger (and possibly additional EventListener, although interceptors might be able to ensure only the appropriate trigger(s) is/are fired on a single EventListener) is needed for each different model image and/or GitOps repo.

How Has This Been Tested?

[x] Ran the build pipeline without adding a non-empty value for the new gitops-update-pipeline-eventlistener-url param, and verified that the optional task to call the EventListener was skipped
[x] Ran the build pipeline with a valid webhook URL for the new gitops-update-pipeline-eventlistener-url param, and verified that the new optional task was executed, and the GitOps Update pipeline was triggered and completed successfully.

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

Currently an additional trigger (and possibly additional EventListener, although interceptors might be able to ensure only the appropriate trigger(s) is/are fired on a single EventListener) is needed for each different model image and/or GitOps repo.

I don't think it is needed for each different model image because model image is directly in the image-registry-repo parameter, isn't it?
Also, GitOps repository can be parameterized as well, right?
In case a user has some internal organization for different GitOps repositories, they only need to change image registry, digest and GitOps repo URL. Or if everything is in one repository, then the yq-script must be changed accordingly.

Currently an additional trigger (and possibly additional EventListener, although interceptors might be able to ensure only the appropriate trigger(s) is/are fired on a single EventListener) is needed for each different model image and/or GitOps repo.

I don't think it is needed for each different model image because model image is directly in the image-registry-repo parameter, isn't it?
Also, GitOps repository can be parameterized as well, right?
In case a user has some internal organization for different GitOps repositories, they only need to change image registry, digest and GitOps repo URL. Or if everything is in one repository, then the yq-script must be changed accordingly.

If anything other than the image-registry-repo and image-digest parameters need different values for different models and/or GitOps repos (and I expect that they typically will), then a different trigger should be used. I think that's a cleaner approach than having the (already quite cumbersome) aiedge-e2e pipeline needing to know all of the extra parameters for the GitOps Update pipeline.

If anything other than the image-registry-repo and image-digest parameters need different values for different models and/or GitOps repos (and I expect that they typically will), then a different trigger should be used. I think that's a cleaner approach than having the (already quite cumbersome) aiedge-e2e pipeline needing to know all of the extra parameters for the GitOps Update pipeline.

I see, so these extra parameters would be hard-coded in the trigger template so the aiedge-e2e pipeline don't need to have these at the start.


“One size fits all” analytic solutions to the Grad–Shafranov equation

In GitLab by @mkovari on Sep 17, 2019, 15:28

An extended analytic solution to the Grad–Shafranov equation using Solov’ev profiles is presented. The solution describes standard tokamaks, spherical tokamaks, spheromaks, and field reversed configurations. It allows arbitrary aspect ratio, elongation, and triangularity as well as a plasma surface that can be smooth or possess a double or single null divertor X-point.

Physics of Plasmas 17, 032502 (2010); https://doi.org/10.1063/1.3328818, Antoine J. Cerfon and Jeffrey P. Freidberg
The equations have been fully coded up in Matlab by Cerfon.  All you need to do is supply the following parameters:

epsilon,
kappa,
delta,
xsep = x-location of the X-point,
ysep = y-location of the X-point,
and a parameter determining the beta regime of interest, called A in the article.  (Values have been provided by for ITER and NSTX.)

The numerical work is trivial: solving a set of up to twelve linear algebraic equations.
This might be much simpler and faster than EMEQ in generating equilibria for Plasmod.  It might also be useful for Rabbit (with or without Plasmod).  It could be used for any physics model that requires a knowledge of the flux surfaces.
Note that this is a simplification - for example, the current density is finite, not zero, at the plasma edge.  The current profile is an output, not an input, so it will not be consistent with the current profile calculated by Plasmod.
As for the rest of PROCESS: strictly speaking we would want to recalculate all the poloidal fields, since this model gives the poloidal flux everywhere.  It would give a q profile, but probably not the right one.
It would be easy to convert the code into Python.  We could then plot the flux surfaces trivially.
There would also be a benefit in converting the code to Fortran.  We already have a matrix invertor (subroutine hinv), and Fortran has matrix multiplication built in: matmul.  This would allow us to output data files of temperature, density and radiation per unit volume against the normalised flux, as is conventional. (I have tried the matlab2fortran code, but it doesn't run on our old version of matlab.  The regexp function seems to have been changed.)
On the other hand, we could create all these quantities in Python, as they are only outputs.  This would have the benefit of allowing us to create a Python suite that can be used standalone to implement Cerfon's equations and plot whatever you want.
To make the code more consistent we could use the flux surfaces to do the volume integrations more accurately - for example fusion power.  This would require some effort, to derive the volume between adjacent flux surfaces.

If this is implemented then the first scaling by Andrade et.al in #3355 can be used properly


Powershell's progress bar corrupts the screen buffer

Environment
Platform ServicePack Version      VersionString
-------- ----------- -------      -------------
 Win32NT             10.0.18363.0 Microsoft Windows NT 10.0.18363.0

PSVersion                      7.0.0-rc.2

Steps to reproduce

Start a powershell session
Fill screen with content dir $env:SystemRoot
Perform operation that temporarily displays the progress bar Invoke-WebRequest https://raw.github.com/trapd00r/LS_COLORS/master/LS_COLORS -OutFile ~\LS_COLORS
Make the window wider

Expected behavior
The display should return to normal and resizing the window should not disturb that.
Actual behavior
Widening the window an inch skews the content that was temporarily covered by the progress bar.

This happens with both conhost and windows terminal.
The same issue is also discussed here: https://github.com/PowerShell/PowerShell/issues/11905

The progress bar also messes with the colors underneath. "explorer.exe" should share the same color (orange) as the executables further down in the list. Instead it has turned yellow.

Will start troubleshooting this over at PowerShell/PowerShell.


NotImplementedError: Make sure that a _reorder_cache function is correctly implemented in transformers.models.llama.modeling_llama to enable beam search for <class 'transformers.models.llama.modeling_llama.LlamaForCausalLM'>

when i tried add num_beams=3 while inferencing getting this issue

Will investigate!

Any updates on this issue??

I'm having the same issue with other num_beam values


Napari plugin documentation

Hi @jni ,
I just noticed that skan became a napari-plugin at some point. However, I can't find any documentation for how to use it from napari. Is there any documentation for the napari-plugin available?
Thanks!
Best,
Robert

Not yet, sorry! Possibly I should have used the hidden visibility tag until that was done 😅. Until James's PR is merged it's mostly a shim...


Modal on too small/narrow viewports

Thanks for your work on this. It's a nice and small modal implementation. I'm wondering what your stance is on opening a modal on too small viewports. The obvious answer is of course to not open a modal. Can this modal implementation be improved by enabling scrollbars in that case or similar solutions?

Hello,
Thanks for the kind words. This script only deals with the toggling as well as some accessibility concerns such as the focus trap. The whole CSS layer is up to the user. Which means you can of course implement a modal in a way where scrolling is an option. :)

Here is a version with scrollbars: codepen.io/andreas_pr/pen/WwbMgV?editors=0110

I’ll close this then. :)


AYS: New Run execution model

In order not to end up in a dead lock situation where a failed action block all run execution, we need a new mechanism for run creation.
The idea is that we will now have two type of runs.

Normal run, asked by the users. This type of run will never include previously failed actions.
So this type of run should always be able to run.

Error run, this type of run will include all actions that are in error state.
This run is automatically created  every X minutes.
The failed actions will also get a level of failure from 1 to 6.
1 means if failed in a normal run
6 means the action has retry the maximum number of time and is considered dead. It will not be scheduled anymore.

First phase done: https://github.com/Jumpscale/jumpscale_core8/commit/dbf47cabb715f1e179e1e52c97dfbd51372fae0a
This ensure we only have 1 run executing at the same time.

Second phase:  https://github.com/Jumpscale/jumpscale_core8/commit/5c40bde24f6e33aa3c0f871c9f11896dc8e48581
reties failed action
gracefull shutdown


Remove everything related to studies

Current coverage is 88.51%

Merging #479 into master will increase coverage by +0.61% as of 1ef9673

@@            master    #479   diff @@
======================================
  Files          234     229     -5
  Stmts        10426   10213   -213
  Branches         0       0
  Methods
======================================
- Hit           9165    9040   -125
  Partial          0       0
+ Missed        1261    1173    -88

Review entire Coverage Diff as of 1ef9673

Powered by Codecov. Updated on successful CI builds.

(we'll avoid merging this until after I'm back in mid-May)

This can be merged after #492 is merged.


Error compilling result.hpp on apple-clang.

Linux Compier: GCC 11
Mac Compiler: latest apple-clang
The resulting code works on linux / gcc, but on mac I get:
/result.hpp:3524:82: error: ISO C++ requires the name after '::~' to be found in the same scope as the name before '::~' [-Werror,-Wdtor-name]
inline RESULT_INLINE_VISIBILITY RESULT_NS_IMPL::detail::result_union<T, E, false>::~result_union() noexcept(
                                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
                                                                                ::result_union

https://stackoverflow.com/questions/68751682/is-a-class-templates-name-in-scope-for-a-qualified-out-of-line-destructors-def

This change fixes for me:
RESULT_NS_IMPL::detail::result_union<T,E,false>
  ::~result_union()

should read:
RESULT_NS_IMPL::detail::result_union<T,E,false>
  ::~result_union<T,E,false>()

Thanks for opening this issue.
Funny, the StackOverflow question you linked is actually my question that I asked regarding this exact problem; I just forgot to follow-up on this. 😅
I'll make this change shortly.

Also my apologies for missing this when it was opened; I apparently need to update my notification preferences.

@tcanabrava Are you interested in making the PR for this?
If not or if you don't respond in the following days, I'll be happy to do it.

I can make this change; the only thing that stopped me was realizing that CI was going to fail, and not having the time to address it


How about  adding fibers/coroutines for DC's send method?

I aware that we have an ongoing issue; however, I found a way to  mimic threading like experience in browsers for DataChannel.send() method.
Fibers/Coroutines can archive concurrency without web workers. I'm unsure about implementation details.
http://js-coroutines.com/
If you look at this lib, it claims "Avoid the jank without having to resort to Worker threads or worry about glitches when you get your results back from them." which is quite sexy for me.

Obviously, Round Robin based scheduling and multiplexing are another stuff to look at.

Coroutines are cool but they are just collaborative multitasking, in the sense that tasks are still processed one by one on the main thread, only with coroutines a task can pause to let the next one run. You can't pause from wasm code so it won't change that wasm code is limited to a single execution at a time.
There is a similar system with emscripten to mimic independant threads, however it has a cost, builds are bigger and performance is lower: https://emscripten.org/docs/porting/asyncify.html

https://twitter.com/kripken/status/1218955129761611781

I'm going to message him and see if he can add this new backend

There is this ASYNCIFY_IMPORTS and ASYNCIFY_IGNORE_INDIRECT to reduce the overhead.  Also -Oz can reduce the overhead.
The overhead would be at 5-10% if these options used else 50% as usual.

If it's complex to add this, you can close this.

That's interesting, but I don't get what you want to add to the wrapper. This is something you should use in your own code.

It turns out the dc.send method is a fire and forget style method means you can send many data without any interrupt; however, one should not fill the buffer completely and it's advised to listen for onbufferedamountlow to send more data.
Source: https://stackoverflow.com/questions/56327783/webrtc-datachannel-for-high-bandwidth-application
Would  you be able to implement a re-try logic around onbufferedamountlow event?
I want be able to send many data without implementing even managing a custom queue my own.
Secondly, I believe as per my current knowledge (Still growing as I write :D), your API should have fiber based interface.
https://emscripten.org/docs/api_reference/fiber.h.html
A nice wrapper around it can be found here,
https://github.com/paladin-t/fiber
Feel free to correct me if I am wrong.
I still haven't got time to look into fibers more depth as we have an ongoing issue with Chrome

It turns out the dc.send method is a fire and forget style method means you can send many data without any interrupt; however, one should not fill the buffer completely and it's advised to listen for onbufferedamountlow to send more data.

Indeed, that's how the DataChannel API works.

Would you be able to implement a re-try logic around onbufferedamountlow event?
I want be able to send many data without implementing even managing a custom queue my own.

No, I'm not going to add custom logic around the wrapper as it's out of its scope. The goal of this wrapper is just to have an API identical to libdatachannel which is roughly equivalent to the browser WebRTC API. You can easily listen for the onbufferedamountlow event with the onBufferedAmountLow() method and send more data when it is triggered.

Secondly, I believe as per my current knowledge (Still growing as I write :D), your API should have fiber based interface.
https://emscripten.org/docs/api_reference/fiber.h.html
A nice wrapper around it can be found here,
https://github.com/paladin-t/fiber
Feel free to correct me if I am wrong.
I still haven't got time to look into fibers more depth as we have an ongoing issue with Chrome

I don't really get what you mean here. All methods basically return instantly just like the ones in the browser WebRTC API, so what for would you need fibers for in the API?

Then we don't need fibers at all 😄