GMT_PAD_DEFAULT doesn't exist in libgmt:  Possible upstream GMT_GET_Enum API Change

Feedback or description of feature requested
After upgrading from gmt 6.0.0_r20005 to 6.0.0_r20469 of upstream trunk gmt library, and every commit since,  gmt-python stopped functioning (import fails with CLIB Error). I understand these are development commits, I am maintaining some images with cutting edge builds of gmt for test purposes. I have done due diligence to verify it isn't other incompatabilities, or build errors. This is a change in behavior from previous versions of GMT which I have tested with development versions of gmt-python
It appears an API change may been introduced in r20437. An API Pointer has been added to the function signature, this may contribute to the error. GMT_PAD_DEFAULT was moved from a pre processor constant to enum in Revision 18648 and gmt-python was updated to accommodate this change at some point
Before and after docker images are available. for verification  nc5ng/gmt:6.0.0_r20005-python (Functioning) and nc5ng/gmt:6.0.0_r20469-python (Broken).  docker hub link
note: these have slightly different versions of gmt-python, but I have downgraded/upgraded to verify and it is not due to gmt-python.
Are you willing to help implement and maintain this bug? Yes if guidance is provided
Full code that generated the error
import gmt

Full error message
/usr/local/lib/python3.5/dist-packages/gmt/__init__.py in <module>()
     26
     27 # Start our global modern mode session
---> 28 _begin()
     29 # Tell Python to run _end when shutting down
     30 _atexit.register(_end)

/usr/local/lib/python3.5/dist-packages/gmt/session_management.py in begin()
     15     """
     16     prefix = 'gmt-python-session'
---> 17     with LibGMT() as lib:
     18         lib.call_module('begin', prefix)
     19

/usr/local/lib/python3.5/dist-packages/gmt/clib/core.py in __enter__(self)
    128         Start the GMT session and keep the session argument.
    129         """
--> 130         self.current_session = self.create_session('gmt-python-session')
    131         return self
    132

/usr/local/lib/python3.5/dist-packages/gmt/clib/core.py in create_session(self, session_name)
    170         # print function.
    171         print_func = None
--> 172         padding = self.get_constant('GMT_PAD_DEFAULT')
    173         session_type = self.get_constant('GMT_SESSION_EXTERNAL')
    174         session = c_create_session(session_name.encode(), padding,

/usr/local/lib/python3.5/dist-packages/gmt/clib/core.py in get_constant(self, name)
    237         if value is None or value == -99999:
    238             raise GMTCLibError(
--> 239                 "Constant '{}' doesn't exits in libgmt.".format(name))
    240
    241         return value
   GMTCLibError: Constant 'GMT_PAD_DEFAULT' doesn't exits in libgmt.

System information

Operating system: Ubuntu 16.04.5 (Docker Image)
Python installation (Anaconda, system, ETS): python3 from ubuntu repository
Version of GMT: Tested 6.0.0_r20005 (works) and 6.0.0_r20471 (broken) and 6.0.0_r20469 (broken)
Version of Python: 3.5.1-3
Version of this package: 0.1a3, 0.1a3+112.gefabb47 and `0.1a3+79.gf979f02'

Confirmed by making a quick fix to session.py#L216-L218
c_get_enum = self.get_libgmt_func(
            "GMT_Get_Enum", argtypes=[ctp.c_char_p], restype=ctp.c_int
        )

value = c_get_enum(name.encode())

changed to
c_get_enum = self.get_libgmt_func(
            "GMT_Get_Enum", argtypes=[ctp.c_void_p, ctp.c_char_p], restype=ctp.c_int
        )

value = c_get_enum(None, name.encode())

This works because API change accepts a null pointer for new argument
Commit on my fork here: https://github.com/nc5ng/gmt-python/commit/c539b627305756431acdef73ff266522c96b8880
Verified on the gmt 6.0.0_r20471 and gmt-python 0.1a3+113.gc539b62

Possibly related to #215

Hi @akshmakov, thanks for the detailed report and investigating the issue! I really appreciate it!
This was a recent change to the API. GMT_Get_Enum was one of the only functions that didn't take an API pointer as the first argument. This was recently changed to make the API more uniform and allow for possible extensions in the future. Your change should probably work (great investigating, BTW).
Would you like to open a pull request with that fix? Let me know if you need any guidance for this.
Your build will break for now because of #221 but once I have that fixed everything should be fine.
Again, big thanks for investigating this! :1st_place_medal:


More information about how a client picks up changes to configuration.mof

The doc here says the configuration.mof file is included with policy and is compiled by policy.  When developing things it can be useful to manually trigger them to know if they're working properly.  I was just working a new hardware class, and pulling machine policy and running Hardware Inventory wasn't enough to trigger the creation of the WMI class, I had to force a full Hardware Inventory (by deleting the InventoryActionStatus in WMI for Hardware inventory) and then my client compiled the new MOF and properly inventoried the new class.
Being impatient I may have also restarted the client a couple of times, triggered other things, etc etc, so I can't be sure my forcing of a full is what actually triggered the recompliation of the Mof.  So it would be good to know how to manually trigger this compilation, and also generally to know how this process happens (where can I see it in the logs?) to be able to monitor and troubleshoot it.

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

ID: 54dd8166-9fb1-3027-ad24-6a14c2dd5f3f
Version Independent ID: 02c01bf8-020c-b29a-ab8f-1541c83743b5
Content: Extend hardware inventory - Configuration Manager
Content Source: sccm/core/clients/manage/inventory/extend-hardware-inventory.md
Product: configuration-manager
Technology: configmgr-client
GitHub Login: @mestew
Microsoft Alias: mstewart

Thanks again for the feedback. We moved this content to a new GitHub repository, so this issue is no longer tracked with the original article. As part of that process, we reviewed our backlog of feedback, and made some tough decisions on items that we don't think we'll realistically ever address. If you're passionate about this specific feedback, please refile it on the current article, now under docs.microsoft.com/mem/configmgr.


Docker - Install Python in container

Does anyone know how to install python into the docker container?
I've tried using the "command" field on docker but know luck.
Any Ideas?

You can create a Dockerfile to create an image which has python installed.
Something like this:
FROM codercom/code-server:v2

USER root
RUN apt-get update && apt-get install -y \
	python

USER coder
EXPOSE 8080

ENTRYPOINT ["dumb-init", "code-server", "--host", "<IP_ADDRESS>"]


Bug:  Unknown column updated_at in 'field list

PHP Version
8.2
CodeIgniter4 Version
CodeIgniter v4.4.3
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Windows
Which server did you use?
apache
Database
libmysql - mysqlnd 8.0.10
What happened?
Column updated_at is required for all tables!
Steps to Reproduce
composer create-project codeigniter4/appstarter CI4-bug

php spark make:migration Test

<?php

namespace App\Database\Migrations;

use CodeIgniter\Database\Migration;

class Test extends Migration
{
    public function up()
    {
        $this->forge->addField([
            'id'         => ['type' => 'int', 'constraint' => 11, 'unsigned' => true, 'auto_increment' => true],
            'email'      => ['type' => 'varchar', 'constraint' => 255],
            'ip_address' => ['type' => 'varchar', 'constraint' => 255],
            'created_at'   => ['type' => 'datetime', 'null' => true],
        ]);
        $this->forge->addPrimaryKey('id');

        $this->forge->createTable('test', false);
    }

    public function down()
    {
        //
    }
}

php spark migratin --all

php spark make:model Test

<?php

namespace App\Models;

use CodeIgniter\Model;

class Test extends Model
{
    protected $table            = 'test';
    protected $primaryKey       = 'id';
    protected $useAutoIncrement = true;
    protected $returnType       = 'array';
    protected $useSoftDeletes   = false;
    protected $protectFields    = true;
    protected $allowedFields    = [
        'email'
    ];

    // Dates
    protected $useTimestamps = true;
    protected $dateFormat    = 'datetime';
    protected $createdField  = 'created_at';

    // Validation
    protected $validationRules      = [];
    protected $validationMessages   = [];
    protected $skipValidation       = false;
    protected $cleanValidationRules = true;

    // Callbacks
    protected $allowCallbacks = true;
    protected $beforeInsert   = [];
    protected $afterInsert    = [];
    protected $beforeUpdate   = [];
    protected $afterUpdate    = [];
    protected $beforeFind     = [];
    protected $afterFind      = [];
    protected $beforeDelete   = [];
    protected $afterDelete    = [];
}

<?php

namespace App\Controllers;

class Home extends BaseController
{
    public function index(): string
    {
+model('Test')->insert([
+'email'<EMAIL_ADDRESS>+'ip_address' =>'<IP_ADDRESS>',
+]);
-return view('welcome_message');
    }
}

Expected Output
The presence of updated_at in the tables should not be mandatory.

Anything else?
If I add column updated_at to the table. The error is fixed.
I could create a separate field for the date here instead of using $useTimestamps, but the question is why didn't CI cover this.

At least this is not a bug.

This requires that the table have columns named created_at, updated_at and deleted_at in the appropriate data type.
https://codeigniter4.github.io/CodeIgniter4/models/model.html?highlight=usetimestamps#dates

Try:
protected $updatedField = '';

@kenjis You always make me happy, it worked!
Thanks!

This is just a lack of documentation because that is how the code works.

It was documented.

Leave it empty to avoid update it (even $useTimestamps is enabled).
https://codeigniter4.github.io/CodeIgniter4/models/model.html?highlight=usetimestamps#updatedfield


🐛 Invalid data supplied to ledger

Description: User was able to connect to the voting app, but was unable to vote.
Error message: "Invalid data supplied to ledger"
Setup of user: Eternl for laptop, using Ledger Flex.
We need to know if this is a fixable issue ASAP.

I have also encountered this issue.
What works with ledger

[x] Connecting wallet works
[ ] Voting doesn't work
[ ] Uploading votes onchain. Was working yesterday.

I was able to perform the following actions successfully with my Ledger through eternl:

Sign the message to connect the wallet to the dApp
Sign the message to vote on a poll

Ledger specifications:

Nano X
OS version: 2.2.4
Cardano ADA app version: 7.1.4

It seems like the most likely issues are either:

Ledger firmware and/or Cardano app version are not fully up-to-date
Other Ledger models have more strict limits on the size of metadata than browser wallets and the Ledger Nano X do

@t4ccer tagging you for your reference

Signing in & voting worked for me as well.
Ledger specs:

Nano X
OS Version: 2.2.4
Cardano App version: 7.1.3

@mesudip please can you confirm what Matt and Justin are saying? is this an error caused by outdated firmware?

Thanks @mattlaux @justinschreiner, I have advised the person in question to update everything and come back tomorrow to retest.


websocket 连接发送json数据出现错误ERROR Websocket failed to parse MQTT protocol packet with error message :InsufficientBytes(115)

What type of bug is this?
Unexpected error
What subsystems are affected?
Standalone mode, Other
Minimal reproduce step
使用MQTTX客户端通过websocket连接发送json数据导致服务端出错
What did you expect to see?
无
What did you see instead?
无
What operating system did you use?
debian10
Relevant log output and stack trace
No response

Thanks @who123456789

The test cases with sub/pub json format data of ws and wss did not reproduce the problem.


Mouse pointer y-offset

I cannot reproduce this bug but it happens multiple times a day. I only noticed it when using an external monitor (via a DisplayLink driver on my MacBook Air M1). The mouse pointer is off by approximately 150px in the y direction, so that visually the click and point action is about 150px above the actual shown position.
I managed to record a video (see below).
I am using yabai-v6.0.15 but will now updated to the latest master and report back if I encounter this again.
https://github.com/koekeishiya/yabai/assets/1730350/0a4778a1-6023-4220-988e-5b754652f4a4

I think this is related to Safari. I just noticed that it happens only in the tab where Pluto.jl is running. Now I recall that I mostly see this on website with a (rich) text input, like our GitLab instance.
I am not sure if this is related to yabai but I only encountered these issues since I use yabai 😕
Btw. I just updated to yabai 7.0.2 and the Safari tab with the mouse glitch still has the offset, but again, only that very single tab. If I reload it, it's gone.
Let me leave this issue open for a day or two, in case someone has observed something similar.

Related: https://forums.macrumors.com/threads/safari-17-cursor-doesnt-match-actual-position-on-web-page.2407341/
OK, let's close this, it's a Safari issue.


Add office plugin

Can you add OnlyOffice or other similar plugins? There are many requirements for office documents.

Likely not OnlyOffice as that doesn't seem to be free. But indeed I will/am considering possible office apps. The TinyMCE app is a decent Word replacement as it's a WYSYWIG editor. For Excel I am planning to eventually add something like SheetJS or something like that. Other possible options are things like LibreOffice WASM (https://wasm-test.libreoffice.org/) but it's not a small program to download and has a lot of issues. For ODF I was thinking of https://webodf.org/. Anyway, I have plans but it will take a while. Until then I have document editors and viewers of various types, but there are indeed gaps. I also plan to add third party app support eventually which could have others adding these kinds of apps. It's also possible for anyone to fork the code and add these apps, and if they were well done enough I would consider upstreaming them.


Winter Of Blockchain Introduction/Tracking

👋 Hello @adarsh-singh01,
I am Durgesh Kumar Prajapati (Program Manager) Winter Of Blockchain. I'll be collaborating with you on preparing your repo for WoB'24 Program
Why are these changes required?
After Analysing contributions and feedback it would be great for students to have a couple of standard readme files on the repo to understand what the project is all about and some issues detailing what they can contribute. This will help you to see increased engagement in your repository as well.
As mentioned students get a chance to learn step by step, how to build this project as well as invite mentors to collaborate and add features to this repo. I will generate issues, which will provide guidance on how to prepare your repository.
This issue will serve as a tracking issue to track all issues related to WoB. I recommend creating a new branch for every issue and opening a pull request to track changes so we can effectively collaborate with each other and merge changes when you and I feel like those changes are ready to be merged on your primary branch.
If you have any questions or concerns, please feel free to leave a comment on this issue or any of the other issues that are generated.
I look forward to working with you

[ ] #1
[ ] #2
[ ] #3
[ ] #4
[ ] #5

hello, id like to help you out in these tasks

@shivankurchavan @Durgesh4993 whoever does first will be merged or u can work collectively


Is WINIT_HIDPI_FACTOR env has deprecated?

Ubuntu 18.04, X11
Alacritty 0.3.3
I'm using that env to disable DPI scaling. After upgrading to v0.3.3 the env is not working anymore.
cat /tmp/Alacritty-17820.log
[2019-06-19 05:22] [ERROR] Problem with config: invalid type: integer `1`, expected a string; using default value

I'm not sure which change exactly caused this (potentially an update to the yaml library), but all env variables need to explicitly be strings now.
So using "1" instead should work.

It works by adding the double quotes. Thank you.


Update Corona-Case-Report

Showing day differences between onset symptom and hospitalization

As far as I can tell, most of what you did you just pushed straight into the repository. Which is fine, but in the future you should use pull requests. No worries this time though, and it looks like some great analysis!

Ok, Great. Thanks for your help.
On Mar 12, 2020, at 5:54 PM, olynch <notifications<EMAIL_ADDRESS>wrote:
Merged #20https://github.com/necsi/covid19/pull/20 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/necsi/covid19/pull/20#event-3124636917, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOXGIIACXJOXVZIE7ILGWNTRHFKYXANCNFSM4LGVUIZA.


Use XDG_DATA_HOME for history and refresh token.

Implementation of #491. This moves the history and refresh-token files into XDG_DATA_HOME, so that the program's config and transient program data are separated.

Thanks for the PR, I'm on board with this change. Do you think we should stick the history file into XDG_CACHE_HOME? It sounds like a more appropriate directory, as long as the file is being moved either way.

I did some more Googling for what belongs in data versus cache, I think it's fine to remain in XDG_DATA_HOME.
Based on GNOME's recommendation:

If you remove the .cache folder, the user MUST not notice it except maybe for performance (might be huge performance drop)
.local should be treated as any other user data

And Debian's recommendation:

The above example information is not essential data. However it should still persist on reboots of the system unlike cache data that a user might consider putting in a TMPFS.

So it seems to be that the cache directory is for anything the program can regenerate without user intervention, and the data directory is for anything else.

Good info, thanks again!


Pin version of pipeline templates

Pin pipeline templates to a specific version and update to use the new templates

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added notification on error

Added the ability to comment in a PR when it fails to update it.
It will link to the logs and inform the user on what they can do.

Example log message:
Failed to update PR ❌
There was an error while trying to keep this PR up-to-date
You may have conflicts ‼ or may have to manually sync it with the target branch 👉❇
More info in the logs 📋


[stable/prometheus-rabbitmq-exporter] ServiceMonitor Port Bugfix

The service monitor currently tries to scrape the publish port, which isn't available on the Service.
I've changed the endpoint port for the service monitor to scrape from the services' 'rabbitmq-exporter' port.
I've also changed the name to servicemonitor.yaml to match other Helm charts.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @sameersbn

I wonder if all the ports should just be called rabbitmq-exporter to avoid confusion?

Is it possible to get this merged? It's pretty useful.

/ok-to-test

/assign maorfr

/retest
/lgtm

/lgtm

This PR actually broke the exporter scraping, because it incorrectly changes the portname in servicemonitor from publish to rabbitmq-exporter while leaving the port name in service definition to publish.
Correct fix was applied by PR #16134 (merged before this one on Aug 13 -> version bump to 0.5.2) which changes the servicemonitor portname from rabbitmq-exporter to publish correctly alligning the 2 names defined in service and servicemonitor to publish value.
Working - 0.5.2  (PR #16134)

service definition

$ kubectl describe svc kube-stack-prometheus-rabbitmq-exporter
Name:              kube-stack-prometheus-rabbitmq-exporter
Namespace:         default
Labels:            app=prometheus-rabbitmq-exporter
                   chart=prometheus-rabbitmq-exporter-0.5.2
                   heritage=Tiller
                   release=kube-stack
Annotations:       <none>
Selector:          app=prometheus-rabbitmq-exporter,release=kube-stack
Type:              ClusterIP
IP:                <IP_ADDRESS>
Port:              publish  9419/TCP
TargetPort:        publish/TCP
Endpoints:         <IP_ADDRESS>:9419
Session Affinity:  None
Events:            <none>

servicemonitor definition

$ kubectl describe servicemonitors.monitoring.coreos.com kube-stack-prometheus-rabbitmq-exporter
Name:         kube-stack-prometheus-rabbitmq-exporter
Namespace:    default
Labels:       app=prometheus-rabbitmq-exporter
              chart=prometheus-rabbitmq-exporter-0.5.2
              heritage=Tiller
Annotations:  <none>
API Version:  monitoring.coreos.com/v1
Kind:         ServiceMonitor
Metadata:
  Creation Timestamp:  2019-09-20T09:14:59Z
  Generation:          2
  Resource Version:    35327210
  Self Link:           /apis/monitoring.coreos.com/v1/namespaces/default/servicemonitors/kube-stack-prometheus-rabbitmq-exporter
  UID:                 1ebfe46c-db87-11e9-9877-42010aa401cd
Spec:
  Endpoints:
    Interval:  10s
    Port:      publish
  Selector:
    Match Labels:
      App:      prometheus-rabbitmq-exporter
      Release:  kube-stack
Events:         <none>

Borken - 0.5.3 (This PR's version)

service definition

$ kubectl describe svc kube-stack-prometheus-rabbitmq-exporter
Name:              kube-stack-prometheus-rabbitmq-exporter
Namespace:         default
Labels:            app=prometheus-rabbitmq-exporter
                   chart=prometheus-rabbitmq-exporter-0.5.3
                   heritage=Tiller
                   release=kube-stack
Annotations:       <none>
Selector:          app=prometheus-rabbitmq-exporter,release=kube-stack
Type:              ClusterIP
IP:                <IP_ADDRESS>
Port:              publish  9419/TCP
TargetPort:        publish/TCP
Endpoints:         <IP_ADDRESS>:9419
Session Affinity:  None
Events:            <none>

servicemonitor definition

$ kubectl describe servicemonitors.monitoring.coreos.com kube-stack-prometheus-rabbitmq-exporter
Name:         kube-stack-prometheus-rabbitmq-exporter
Namespace:    default
Labels:       app=prometheus-rabbitmq-exporter
              chart=prometheus-rabbitmq-exporter-0.5.3
              heritage=Tiller
Annotations:  <none>
API Version:  monitoring.coreos.com/v1
Kind:         ServiceMonitor
Metadata:
  Creation Timestamp:  2019-09-20T09:14:59Z
  Generation:          1
  Resource Version:    26578148
  Self Link:           /apis/monitoring.coreos.com/v1/namespaces/default/servicemonitors/kube-stack-prometheus-rabbitmq-exporter
  UID:                 1ebfe46c-db87-11e9-9877-42010aa401cd
Spec:
  Endpoints:
    Interval:  10s
    Port:      rabbitmq-exporter
  Selector:
    Match Labels:
      App:      prometheus-rabbitmq-exporter
      Release:  kube-stack

I can confirm as well the exporter is broken in 0.5.3.
The port name in the ServiceMonitor must match the port name in the Service.


QueryBuilders.matchPhrasePrefixQuery() for INTEGER Column is not working

I am unable to use QueryBuilders.matchPhrasePrefixQuery(fieldName, searchQuery) for INTEGER column. I am able to fetch the results if the fieldname is a VARCHAR column. If it is an INTEGER column, It is fetching the result which ever matches the exact value.
For e.g) If there is a INTEGER column as CUSTOMER_NUMBER and have data like 12345684, 123477554, 1234, 12395645. and if i mention "1234" as search query in the above mentioned method... the results fetched is only "1234" and not "12345684, 123477554"
Please let me know which Java API should i use to fetch all those 3 values - "12345684, 123477554, 1234".

Use SQL-92 CAST function in SQL statement, e.g.
select cast(customer_number as varchar(32)) as customer_string from ...

Thank you jprante... ;)
its working now using QueryBuilders.matchPhrasePrefixQuery() after casting as you mentioned...
I thought of this idea earlier.. but didn't implement it.. instead was trying to find Java API method..


🐛 [Bug] Can not convert aten::cross_entropy_loss and aten::Int.Tensor(Tensor a)

Bug Description
ERROR: [Torch-TensorRT] - Method requested cannot be compiled by Torch-TensorRT.TorchScript.
Unsupported operators listed below:
  - aten::cross_entropy_loss(Tensor self, Tensor target, Tensor? weight=None, int reduction=1, int ignore_index=-100, float label_smoothing=0.) -> (Tensor)
  - aten::Int.Tensor(Tensor a) -> (int)
You can either implement converters for these ops in your application or request implementation
https://www.github.com/nvidia/Torch-TensorRT/issues

In Module:

ERROR: [Torch-TensorRT] - Unsupported operator: aten::cross_entropy_loss(Tensor self, Tensor target, Tensor? weight=None, int reduction=1, int ignore_index=-100, float label_smoothing=0.) -> (Tensor)
/opt/conda/lib/python3.8/site-packages/torch/nn/functional.py(2849): cross_entropy
/opt/conda/lib/python3.8/site-packages/torch/nn/modules/loss.py(1150): forward
/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py(1090): _slow_forward
/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py(1102): _call_impl
/root/aigroup/threesegcode-pytorch/model_components/bert_model.py(141): forward
/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py(1090): _slow_forward
/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py(1102): _call_impl
/opt/conda/lib/python3.8/site-packages/torch/jit/_trace.py(958): trace_module
/opt/conda/lib/python3.8/site-packages/torch/jit/_trace.py(741): trace
jit_optimize_model.py(57): jit_optimize
jit_optimize_model.py(107): <module>
Serialized   File "code/__torch__/model_components/bert_model.py", line 22
    input = torch.view(_3, [-1, 30458])
    target = torch.view(labels, [-1])
    _4 = torch.cross_entropy_loss(input, target)
         ~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
    return (_4, torch.softmax(_3, 1), _2)
class DimensionReduceLayer(Module):

ERROR: [Torch-TensorRT] - Unsupported operator: aten::Int.Tensor(Tensor a) -> (int)
Serialized   File "code/__torch__/transformers/models/bert/modeling_bert.py", line 46
    position_ids = self.position_ids
    seq_length = ops.prim.NumToTensor(torch.size(input_ids, 1))
    _5 = int(torch.add(seq_length, CONSTANTS.c1))
         ~~~ <--- HERE
    _6 = torch.slice(position_ids, 0, 0,<PHONE_NUMBER>854775807)
    input = torch.slice(_6, 1, 0, _5)

Traceback (most recent call last):
  File "jit_optimize_model.py", line 107, in <module>
    jit_optimize()
  File "jit_optimize_model.py", line 93, in jit_optimize
    trt_engine = convert_method_to_trt_engine(
  File "/opt/conda/lib/python3.8/site-packages/torch_tensorrt/_compile.py", line 149, in convert_method_to_trt_engine
    return torch_tensorrt.ts.convert_method_to_trt_engine(ts_mod,
  File "/opt/conda/lib/python3.8/site-packages/torch_tensorrt/ts/_compiler.py", line 211, in convert_method_to_trt_engine
    return _C.convert_graph_to_trt_engine(module._c, method_name, _parse_compile_spec(compile_spec))
RuntimeError: [Error thrown at core/compiler.cpp:359] Expected conversion::VerifyConverterSupportForBlock(g->block()) to be true but got false
Not all operations in graph are supported by the compiler

ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 1672) of binary: /opt/conda/bin/python
/opt/conda/lib/python3.8/site-packages/torch/distributed/elastic/multiprocessing/errors/__init__.py:370: UserWarning:

Expected behavior

Environment

Build information about Torch-TensorRT can be found by turning on debug messages

Torch-TensorRT Version (e.g. 1.0.0):
PyTorch Version (e.g. 1.0):
CPU Architecture:
OS (e.g., Linux):
How you installed PyTorch (conda, pip, libtorch, source):
Build command you used (if compiling from source):
Are you using local sources or building from archives:
Python version:
CUDA version:
GPU models and configuration:
Any other relevant information:

Additional context

Hi @sunhailin-Leo can you please provide the compilation specs? In particular, is require_full_compilation = False (default)?
It looks like you're calling convert_method_to_trt_engine, however, we see that aten::int and aten::cross_entropy_loss weren't supported. For convert_method_to_trt_engine to convert the module to a TRT engine, the entire graph must be supported.
@narendasan we should add this to our documentation so the usage of convert_method_to_trt_engine is more clear.
If you are trying to extract the engine, then aten:cross_entropy_loss will need to be supported (aten::int already is). If you are OK deploying in TS then try compile(...) instead!


Allow to directly reference the types associated with interface properties

interface A {
	prop: {
		x: number;
		y: number;
	}
}

Trying to associate the type of prop with a variable would lead to an error:
let p: A.prop; // Error: cannot find namespace A

Aside from anonymous object interfaces, it could also be allowed for function types:
interface A {
	func(arg: number): string
}

let f: A.func; // type of f is now func(arg: number): string
f = (arg: number) => "hello"; // OK

Primitive types:
interface A {
	prop: boolean;
}

let val: A.prop; // type of val is now boolean
val = true; // OK

And also when the types are nested:
interface A {
	outerProp: {
		innerProp: {
			x: number;
			y: number;
		}
	}
}

let p: A.outerProp.innerProp; // type of p would now be {x: number, y:number}
p = { x: 42, y: 24 } //OK

The current workaround (the best that could I find, at least [edit: I added an additional one that uses typeof in the next comment]) is to define the type of prop as a named interface or alternatively use the type keyword.
interface PropInterface {
	x: number;
	y: number;
}
// Or, alternatively:
type PropInterface = { x: number, y: number };

interface A {
	prop: PropInterface;
}

let p: PropInterface; // OK

But that isn't always necessary or elegant, especially if the types are nested or generic (short example below).
This can also be extended to types defined through the type keyword or when using typeof SomeClass. Or even with generics:
interface A<T> {
    prop: { func: (val: T) => T }
}

let f: A<number>.prop.func // working around this may be possible, but wouldn't be pretty.

A real-world use case I encountered was casting a value into the type of an interface member from a union that includes it (this happened when thinking about issues with assignability of merged/intersected types):
let merged: A merge B; // Also happens with intersection: A & B
let obj: A;
obj.prop = <A.prop> merged.prop; // the type of merged.prop is the union A.prop | B.prop

// Edit: It turns out that a relatively simple workaround is possible with the 'typeof' operator,
// that is explained in the next comment:
obj.pro = <typeof obj.prop> merged.prop;

@RyanCavanaugh Too Complex? Perhaps at least keep it as an option for the future?

I'll try to state more formally:
The access path, e.g. A.outerProp.innerProp, would be used as an alias for the type the referenced is property associated with. This also includes cases where a property was set an a primitive type (e.g. type A = { num: number }, A.num would now be an alias for number), or even a named type (type A = { obj: SomeObjType } A.obj would now be an alias for SomeObjType).
If that syntax appears confusing with instance access operations (I'm not sure at this moment), it is possible to require the typeof keyword with it, e.g. let x: typeof A.outerProp.innerProp. The problem now is that typeof may be needed to be used twice to say, reference a class type: e.g. typeof (typeof SomeClass).outerProp.innerProp.

Some creative (and unexpected) applications of this (that wouldn't be possible with the typeof "trick"):
Directly referencing the type of a different property of the containing interface in a method's return type.
interface Cool {
  val: SomeType;

  // The return type would automatically "sync" with whatever type "val" has been set to!
  func(arg: number): Cool.val;
}

Automatic type matching between interfaces:
interface A {
	prop: {
		subProp
		{
			x: number,
			y: number,
			str: string
		}
	}
}

interface B {
	matchedProp: A.prop.subProp; // would automatically "sync" with changes to A.prop.subProp
}

I will post others when I can think of more, but I already find these interesting.

This proposal would be ambiguous given the current language semantics. consider the case with a module with a property with the same name, e.g.:
interface A {
    I: string;
}
namespace A {
   export interface I {}
}

var x: A.I; // A.I can not resolve to property I of interface A, as it already resolves to something else.

I believe what you are asking for is more along the lines of operators like typeof, nameof(#1579),  returnTypeOf(https://github.com/Microsoft/TypeScript/issues/4233#issuecomment-129126053), so i would expect you want something like propertyTypeOf

@mhegazy
Having an interface and a namespace with the same name is a "corner" of the language that I'm not very familiar with (I do think I encountered it once or twice so I had a feeling it exists). I wasn't very sure if having a direct reference like A.prop would be very readable anyway. Using a keyword like propertyTypeOf looks like an interesting approach, and will also make the semantics clearer. I will take some time to consider this one and perhaps alternatives.
I believe I can write a proposal for this. I'm not sure if it would cover everything but I'll try.

@mhegazy
I considered several approaches to the syntax. One was without a keyword, using something like let val: { SomeInterface.prop }, i.e. using the curly brackets to signify a "link" (similar to a file system "link" to meta-information), but I eventually felt that was too confusing with the interface syntax and overall adds too much "noise" with the curly brackets, and may turn out to conflict with other or future features of the language.
Then I went back to look for a keyword but wanted something that's shorter than propertyTypeOf and possibly wouldn't require capitalization.
I came across the idea of using typeon which would also more closely parallel typeof (after all, they are almost the same).
interface MyInterface {
	myProp: {
		x: number,
		y: number
	}
}

let x: typeon MyInterface.myProp;

The idea is that the operator queries for the type is set "on" that property, instead of belonging to it ("of") as would be with an instance.
This may not be perfect but I went through an exhaustive list of prepositions in English (and another one here). The only somewhat reasonable alternatives that I could find were "in" (i.e. typein) and "at" (i.e. typeat), but "on" looked like the best so far.
It is still possible there are better options so if you (or anyone else) have alternative ideas then please suggest.

The proposal is in #4640, closing.


Extent report throwing an exception when trying to flush the report

I am using Extent report [v4.0.3] for my C# project [.Net Framework]
When I try to flush the report I got the below exception.
I am using MSTest as a Testing Framework. I call the below code at [Assembly CleanUp] level
For extent object creation i use ----  _extent = new ExtentReports();
    public static void Flush()
      {
          _extent.Flush(); //Here i get an exception
      }

Exception:
System.TypeLoadException: 'Inheritance security rules violated by type: 'RazorEngine.Compilation.CSharp.CSharpRazorCodeLanguage'. Derived types must either match the security accessibility of the base type or be less accessible.'
I am struggling to solve the problem and thanks a lot in advance.

@mtalreja Can you check with 4.1.0-alpha3?

I am using Java project ( not C#) - but I tried with 4.1.0  at Maven ( could not find alpha) https://mvnrepository.com/artifact/com.aventstack/extentreports/4.1.0
getting following error now:
java.lang.Error:
Unresolved compilation problem:
VERSION_2_3_29 cannot be resolved or is not a field.
error occurred at code level->reportE.attachReporter(htmlReporter);

I think this error is a manifestation of selenium - chrome driver mismatch and
Connection reset
java.net.SocketException: Connection reset.
Will update as we get to the bottom of this-- Thanks for your help!


Use improved Ising model

As a side project, I'd like to finish the implementation of the Ising Model in Julia (https://github.com/RandomDefaultUser/IsingModel_Julia), and the resulting improvements could be implemented here as well.

This can be closed, I will not continue working on the Ising Model, it serves is purpose as test case well, and that's all we need.


"Schema must contain uniquely named types but contains multiple types named" Error

Im getting the error: "Schema must contain uniquely named types but contains multiple types named "Transform"."
When as far as i can tell there is only a single interface called 'Transform'
I get this when loading it from the remote server, or when parsing local schema
Schema: https://gist.github.com/tonypee/836877757dbbe9d519b796815c98f376
SDL: https://gist.github.com/tonypee/24fa9a5b2eef5ceeb4098f953ae16be0
overwrite: true
schema: "schema.json"
documents: "src/**/*.graphql"
generates:
  src/generated/graphql.tsx:
    plugins:
      - "typescript"
      - "typescript-operations"
      - "typescript-react-apollo"
  ./graphql.schema.json:
    plugins:
      - "introspection"

Hopefully im just doing something silly

Hi @tonypee !
Can you please share a reproduction in a codesandbox? I couldn't reproduce it... (we have a template here: https://codesandbox.io/s/github/dotansimha/graphql-code-generator-issue-sandbox-template
)

Hey! thanks for the quick reply. Codesanbox doesnt seem to be working for me (doesnt want to start) - but i downloaded the template, and changed the schema.
dotansimha-graphql-code-generator-issue-sandbox-template.zip
When i try to start the project (npm run start) i get:

It seems that i get "Attached" error vs "Transform" it i use .graphql vs .json for the schema

It seems that the issue is with the SDL file. I downloaded it via the GraphQL inspector download button - that doesnt seem to work with this lib. If i use apollo client:codegen to get it - it works. I need to figure out how to get this lib to load the SDL i guess

You can always point codegen directly to your GraphQL endpoint. But it seems a bit weird that something is wrong with the downloaded schema, did you check the diff?
Btw, did you mean you downloaded it from GraphiQL? because I don't know of any graphql-inspector download button...

@tonypee  ?

Hey - thanks for the replies.
I mean in the graphql explorer - you can click a button in the top right:

Dont worry - ill just do things a differernt way! just my luck that the way i tried first doesnt work ;P
thanks again


The Validation acc of the dataset of Cora

Hi, thank you for providing this research. When I was replicating the code recently, I found that when the k step transistion of your Cora dataset was set to 3, the maximum acc of the epoch validation was only 78. Is there anything wrong with my setting?

Best Val Epoch: 189, Best Val Acc: 0.7763, Test Acc: 0.7412, F1_test: 0.7412 This is part of the result. Is this result normal?

Hello, It may come from the parameters.  Please config your hyper-parameters well.
Also, please take a look at this configuration.

Appreciate a lot for your answer, I will try again.


[quarkus] todo-demo-app packaged to jar file fails to run stand-alone

I cloned this repo and ran
mvn package
on a Win10 machine with the aim to create a stand-alone executable jar file. This created todo-backend-1.0-SNAPSHOT-runner.jar in the target/ folder as expected.
However, if I copy target\todo-backend-1.0-SNAPSHOT-runner.jar into an empty directory and run
java -jar target\todo-backend-1.0-SNAPSHOT-runner.jar
execution fails with error message
Exception in thread "main" java.lang.NoClassDefFoundError: io/quarkus/runtime/Application
        at java.base/java.lang.ClassLoader.defineClass1(Native Method)
...

Now, if I add the complete folder src/ to the execution directory and run the java cmd again everything works fine, obviously beause the missing classes are found in the src/-folder.
Would it be possible add the necessary files to the jar, so that this error is avoided?
I'm using maven 3.6.3.

 You need to copy the runner and the libs directory or configure the project to build a fat jar.


Enhancement: Show message when no routes exist.

We should show a message rather than a blank tab page under 'Router Tree' when no routes exist.
Something like, "The Application have no routes defined."

This message could cover the case where no routes exist, and also the case where the router was not injected as per the readme (by having an extra note that reminds the user that prior to v2.3.0, the router must be injected into the route component, as per the readme).

Some initial image for light and dark mode, text is shown using light grey.
Just need to craft a message, add link to more readme info.

Here is the current implementation, I added a link to the readme for older version.


WIP: new spam tab for Discourse AI

Left to do:

Some way to drill into the 4 lists (false positive / negative / scanned / spam)
Send images through

Except for that, this is pretty much ready


Updates ESP32 network interface /multi

Description
This PR make the ESP32 driver compatible with /multi and IPv6.
All adapted drivers can still be used in backward compatible mode when ipconfigCOMPATIBLE_WITH_SINGLE is defined in FreeRTOSIPConfig.h.
Test Steps
Replace all /single sources with this branch, add FreeRTOS_Routing.c, define ipconfigCOMPATIBLE_WITH_SINGLE and run.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Sorry, I compared my branche with main in stead of feature/ipv6_multi_beta. Closing this PR 117.


[BUILD] Reuse versions of NodeJS and Pnpm from maven properties

Why are the changes needed?

fetch versions for NodeJS and Pnpm from Maven properties
enabled npm dependency caching for global packages and web-ui module, as the step above takes extra time in builds.

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #4948 (80b0f69) into master (cc89e54) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #4948   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         563     563
  Lines       30931   30931
  Branches     4032    4032
======================================
  Misses      30931   30931

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

mvn help:evaluate is notably slow for getting maven properties. Over 30s taken for fetching 2 versions from the pom file. Try improve the performance if possible in the future.


teamspeak_server: <IP_ADDRESS>->3.3.0

Motivation for this change
the new teamspeak client refuses to connect to the old server
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: teamspeak_server
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: teamspeak_server
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-linux (full log)
The following builds were skipped because they don't evaluate on x86_64-linux: teamspeak_server
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

Should this be backported to 18.09 as well? I suppose the current version would be not to useful for people otherwise?

I suppose it should be backported. This is like the inverse of the "Spotify situation", where a service (though not centralized here) needs to be updated to stay compatible with the clients.
The client, too, will probably need an update. It's currently a tiny bit behind.

i can confirm that the 3.1.10 client works on the 3.3.0 server, but it would be nice to also upgrade the client, since it asks about installing the update on every launch, and obviously wont work on nixos

I think for teamspeak we can backport all updates. We do the same for other proprietary software.

Can you update the client as well so both are in sync?

closing due to conflicts with https://github.com/NixOS/nixpkgs/pull/50327 and my PR now being a downgrade


How to build nheqminer v0.5c on Ubuntu 16.04

Looks like readme is not really clear in Ubuntu section, because I've ran into some issues during build. So I've created the following instruction. Do steps under root user or use sudo to install dependencies. Quick steps are:
Get and install latest cuda repo(https://developer.nvidia.com/cuda-downloads):
wget http://developer.download.nvidia.com/compute/cuda/repos/ubuntu1604/x86_64/cuda-repo-ubuntu1604_9.1.85-1_amd64.deb
apt-key adv --fetch-keys http://developer.download.nvidia.com/compute/cuda/repos/ubuntu1604/x86_64/7fa2af80.pub
dpkg -i cuda-repo-ubuntu1604_9.1.85-1_amd64.deb
apt-get update

Install build essential:
apt-get install cmake build-essential libboost-all-dev

Install Nvidia drivers 384.111 and Cuda SDK 8.0:
apt-get --no-install-recommends install  nvidia-384 nvidia-384-dev cuda-toolkit-8-0 libcuda1-384

Get latest nheqminer:
git clone https://github.com/nicehash/nheqminer.git

Configure and build:
cd nheqminer/cpu_xenoncat/asm_linux/
chmod +x fasm
sh assemble.sh
cd ../../../
mkdir build && cd build
cmake -DCUDA_CUDART_LIBRARY="/usr/local/cuda-8.0/lib64/libcudart.so" ../nheqminer
make -j $(nproc)

Is it necessary to do that cpu/xenocat/asm_linux step if you only want gpu mining?


Update index.md according to CB-11705

As per requested by PR #241 for CB-11705, I have made changes to the documentation to reflect the use of CordovaDefaultWebViewEngine.

GitHub linked the wrong pull request. Here is the correct one (for the correct project): https://github.com/apache/cordova-ios/pull/241

NOTE: Don't pull in until cordova-ios 4.3.0 has been released.


Delete Travis CI and AppVeyor badges from Cargo.toml

Neither Travis CI nor AppVeyor is building this project.
Furthermore, badges functionality has been deprecated by crates.io since 4 years ago.

Thanks!


Drop net50 TFM and update dependencies.

Override outdated refs from xunit and Microsoft.NET.Test.Sdk to get rid of transitive vulnerabilities. Closes #32.

Codecov Report

Merging #36 (8cb55bd) into master (4b9d86c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #36   +/-   ##
=======================================
  Coverage   93.02%   93.02%
=======================================
  Files          34       34
  Lines        3970     3970
  Branches      427      427
=======================================
  Hits         3693     3693
  Misses        205      205
  Partials       72       72


feature(ui-ux): transaction card component 2.0

What kind of PR is this?:

/kind feature
What this PR does / why we need it:
Which issue(s) does this PR fixes?:

Fixes #
Additional comments?:
Developer Checklist:

[ ] Read your code changes at least once
[ ] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[ ] No console errors on web
[ ] Tested on Light mode and Dark mode*
[ ] Your UI implementation visually matched the rendered design*
[ ] Unit tests*
[ ] Added e2e tests*
[ ] Added translations*

@pierregee @chloezxyy let's merge this after the release


weirdly....

no words required, just  sit back  and watch the movie....

too big to upload so in temp directory on Google drive
https://drive.google.com/drive/folders/0B8kRPmmvPJU3blBSQXZSb0tyaDA


Make shared into an ExternalStorage object

Resolves #120.
I'm not 100% sure that this is the right way to do it, but it is the best thing I've come up with. A couple major points:

As said in the title, this uses our existing ExternalStorage for shared. That API is probably bigger than we need (I think we only need store_path and get_filename), but we already have it in place.
This now takes all pathlib.Path outputs from the dict returned by ProtocolUnit._execute and ensures that they are saved to shared.
The label used in shared.store_path is worth discussion. We've previously decided that permanent storage does not use a labelling system in common with the protocol system, so I've opted for a very simple label here. The downside is that we'll need to copy files around a lot.

This is a significant API break. ProtocoUnits that use files generated by other units will need to get the filename with shared.get_filename(label).

This should be ready for another look, although I think we're going to talk about it on Thursday anyway.

@dwhswenson this is obsolete right?


Enable exactOptionalPropertyTypes check

Here I enabled exactOptionalPropertyTypes check in tsconfig.
Also, I added | undefined to most optional fields (marked with :?)
If your project has exactOptionalPropertyTypes enabled you will face issues when you want to pass undefined in some conditions.
For example currently here
<StyleSheetManager sheet={condition ? styledComponentsStyleSheet.instance : undefined}>
  {children}
</StyleSheetManager>

You will have an error
  Types of property 'sheet' are incompatible.
    Type 'StyleSheet | undefined' is not assignable to type 'StyleSheet'.
      Type 'undefined' is not assignable to type 'StyleSheet'.ts(2375)

Is that because of a setting in your project vs code in styled components? I don’t like the extra verbosity…

Yes, if you add have exactOptionalPropertyTypes you will get TS errors from styled-components for no actual reason

Why wouldn’t you use skipLibCheck? I don’t think it’s responsible to require all 3p libraries to conform to your own project’s TS settings.

It will not work because as I understand this flag just disables errors in d.ts files theirselves
https://codesandbox.io/s/serene-julien-8mlg1q?file=/src/App.tsx

I think we should merge this and enable exactOptionalPropertyTypes in styled-components.
It's not just about code style; without this enabled, it's possible to write code that has different behaviors at runtime depending on if a value is set to undefined, or not set at all. Here's an explanation of it from the TS 4.4 release notes.
Without this, projects using exactOptionalPropertyTypes with styled-components will see the error above which is essentially a warning that styled-components code may not have accounted for the difference in behavior between a value being undefined and not being provided at all.
Even though it feels a lot more verbose to have to add | undefined all over the place, it is worth it to enable this in JS libraries because it makes it clear to Typescript that the library code has accounted for how to handle the two situations.

Alright, I think I've adequately warmed up to this. Can we get the conflicts sorted and I'll merge it?

@probablyup hey, I updated this PR


[e2e]: generic schedule main attestation-name slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3231991297
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.attestation-name.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.attestation-name.slsa3.yml
Trigger: schedule
Branch: main
Date: Wed Oct 12 04:30:01 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3239718226
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.attestation-name.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.attestation-name.slsa3.yml
Trigger: schedule
Branch: main
Date: Thu Oct 13 04:57:24 UTC 2022
Tests are passing now. Closing this issue.


dnr installation issue

Hello Namgk
Thank you for creating this amazing tool. I am completely new to nodejs and node-red so please allow me to ask a very rookie question. I am currently attempting to install the dnr-editor via both npm and git clone as instructed. However I am running into a series of warnings including many deprecated packages. I have attempted to install some of them individually however the problem still persist. Could you please help me take a look at the log and perhaps point out what I might have done wrong. Thanks in advance for your time and attention!
npm dnr editor install output.txt

Hi,
Does it run ok?
Deprecation warnings are prevalent nowadays, the project depends on many other libraries and they also depend on a lot more others.
Each of them need maintenance effort to keep up with one another, which is sometimes difficult.

Hi,
Thank you for elaborating. Unfortunately I do not think the node is working because I can not see Node Requirement tab in GUI

If you install from github, you might need to do a grunt build to build the
assets.
And double check the port, 1818 not 1880.
What version of dnr-editor are you having?
If nothing works, can you send the log of dnr running?
Thanks
On Tuesday, 5 February 2019, viixiv<EMAIL_ADDRESS>wrote:

Hi,
Thank you for elaborating. Unfortunately I do not think the node is
working because I can not see Node Requirement tab in GUI
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/namgk/dnr-editor/issues/16#issuecomment-460873172,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9PoOky40AVjDMQVYrU_p7vnYDCsYF0ks5vKjPRgaJpZM4airQC
.

Hi. Thank you for the advice
The DNR editor is @0.19.4-4.1.0. Just in case I have attached the complete list of npm packages.
C:\Users\XXXX\AppData\Roaming\npm
+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>
Port 1818 can not be reached when node-red is running. I checked the available port list with netstat -ano and no such port can be found.
I am sorry for my unfamiliarity with github and npm. I do not know how to initiate grunt install and export log file of a specific package operation. Could you kindly provide some instructions or references?
Thanks for your time and patience

Could you please try this:

clone this repo and go to its folder
run "npm install" or "yarn" (if you don't have yarn, install it first also) -> this step should be success
run "npm run build", or "yarn build" --> this step should also be success
edit the settings.js file, change the uiPort to 1818 and comment out the userDir setting
run "node red"
go to http://localhost:1818

Let me know your problem on each of these steps.
Thanks

close for inactivity, there's now a docker container ready to run if you want to try it out.
Thanks


A way to compute the level of a move (average on a variation?) based on Explorer

It's not just taking the average rating associated with the move, but also the ratings of all other moves not taken and the number of people playing them. Don't have the formula yet.

It would require information from Explorer, which would be different based on player configuration. Also it is very inaccurate. Evaluations and winning stats should be more than enough for the same purposes.


Any plan to set c++11 as default standard version?

The gtest repo has changed c++ version from 03 to 11, is there any plan for glog?

Yes, I would even go as far as requiring C++14 or even C++17.
@drigz @shinh @ukai Any thoughts ?

I don't have an opinion, although I'll note that the style guide suggests C++17: https://google.github.io/styleguide/cppguide.html#C++_Version

I submitted #902 which sets C++14 as the minimum standard. C++17 seems unnecessary at this point.


Less verbose log

After successfully running the code on a live night of ~300 images, we have logs of 4.6Mb. That feels too big to send as an email attachment every day (admittedly it was in debug). We should consider whether to reduce some of the text output (e.g those jumbo astroquery tables).

See #144


Login page Addition

Need to add login page with transitions.

hey @Alok5102R i would like to work on this issue , but could you please provide me further info regarding enhancement of login page.

@bhavy03 need to add a login page. With smooth transition on buttons.
Also, email check as well as Password check is required.
Password check: Minimum 8 characters, 1 special character and number.

hi @Alok5102R. If no one is working on this issue then can you please assign this to me so that i start working on this.

okay @Alok5102R assign it to me i will get it done

okay @bhavy03 you can proceed.
Attach screen shot along with PR.
@farhanalws4u will assign you next one sure.

okay @bhavy03 you can proceed.
Attach screen shot along with PR.
@farhanalws4u will assign you next one sure.

@bhavy03 please update if you are working on this issue.

yes @Alok5102R i am working on this issue but i will submit pr after 1 oct

okay @bhavy03 no issues.

Hello  @bhavy03 any update?

I already submitted pr yesterday

you pr is having merge conflicts.
Please, correct them.
And, don't make changes in the main file.
Also, attach the screen shot of the same.

Thanks @bhavy03

You referred it from your forked commit.
Initiate the pull request, so that I can merge in this repository.
@bhavy03

Thanks @bhavy03


not able to execute test suite from ci tools

opentest session create --template "testsuite_ecomm_window_smoke_dev_chrome.yaml"
Error:  Request failed with status code 400
Response payload:  Template file "testsuite_ecomm_window_smoke_dev_chrome was not found
(node:4184) UnhandledPromiseRejectionWarning: Error: Request failed with status code 400
at createError (C:\ProgramData\nvm\v12.11.1\node_modules\opentest\node_modules\axios\lib\core\createError.js:16:15)
at settle (C:\ProgramData\nvm\v12.11.1\node_modules\opentest\node_modules\axios\lib\core\settle.js:17:12)
at IncomingMessage.handleStreamEnd (C:\ProgramData\nvm\v12.11.1\node_modules\opentest\node_modules\axios\lib\adapters\http.js:236:11)
at IncomingMessage.emit (events.js:215:7)
at endReadableNT (_stream_readable.js:1183:12)
at processTicksAndRejections (internal/process/task_queues.js:80:21)
(node:4184) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:4184) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Try to remove the ".yaml" from the template name. This should work:
opentest session create --template "testsuite_ecomm_window_smoke_dev_chrome"

Of course, you should double-check to make sure you are using the correct file name.

it working fine in my local..but in pipeline getting error .attached both the screenshots.

You might want to check the capitalization of the file. In Windows the file name capitalization doesn't matter, but in a Linux environment, it does. For example, if your template file's name on disk starts with an uppercase "T", the OpenTest server will not be able to find it, because you are looking for a file whose name is all lowercase.

file name is correct

Not sure whether if someone is still looking for the solution but I had the same issue today, fixed it by adding the folder name under template, e.g.

we want to run the sample template "GitHub tests.yaml" which is located under opentest/test-repo/templates/github
the way you can start it from cli is the following:
> opentest session create -t "github/GitHub tests"

Hope this helps.
Have a nice day!


404 error and edge function error on localhost

Story
No response
Description
I consistently get a 404 error and sometime get edge function error when doing changes to the website on localhost. This all started when we moved to netlify. It would be nice to get these fixed since it really slows down website dev work.

The 404 is because of the image pipeline (which I've just raised explicitely here).
As for the edge functions, I have my suspicions on why this is happening but can't reproduce locally. Next time it does happen, if you could please screenshot the page, and then any logs you see in the terminal, that would be really useful in understanding what may be causing this.

I am now getting an error when trying to run the website locally. Here is the log file
PS C:\Users\iansk\documents\github\website> npm install
added 1 package, removed 2096 packages, changed 1 package, and audited 596 packages in 13s
156 packages are looking for funding
run npm fund for details
1 critical severity vulnerability
Some issues need review, and may require choosing
a different dependency.
Run npm audit for details.
PS C:\Users\iansk\documents\github\website> npm start

<EMAIL_ADDRESS>start
npm-run-all clean:dev build:js --parallel dev:*

<EMAIL_ADDRESS>clean:dev
npx del-cli '_site/!(img)' && npx mkdirp '_site/js'

<EMAIL_ADDRESS>build:js
terser -c -m -o _site/js/cc.min.js node_modules/vanilla-cookieconsent/dist/cookieconsent.js src/js/cookies.js

<EMAIL_ADDRESS>dev:docs
node scripts/dirExists ../flowforge/docs && nodemon -w ../flowforge/docs -e md --exec "npm run docs" || echo "docs not found - skipping..."

<EMAIL_ADDRESS>dev:postcss
cross-env TAILWIND_MODE=watch npx postcss ./src/css/style.css -o ./_site/css/style.css --config ./postcss.config.js -w

<EMAIL_ADDRESS>dev:eleventy
cross-env NODE_ENV=development ELEVENTY_ENV=development npx @11ty/eleventy --serve --quiet

"docs not found - skipping..."
warn - You have enabled the JIT engine which is currently in preview.
warn - Preview features are not covered by semver, may introduce breaking changes, and can change at any time.
[11ty] Image pipeline is enabled in dev mode expect a wait for first build
info - Tailwind CSS is watching for changes...
info - https://tailwindcss.com/docs/just-in-time-mode#watch-mode-and-one-off-builds
[11ty] Problem writing Eleventy templates: (more in DEBUG output)
[11ty] 1. Having trouble writing to "_site/blog/first-deploy/index.html" from "./src/redirect.md" (via EleventyTemplateError)
[11ty] 2. EISDIR: illegal operation on a directory, open 'C:\Users\iansk\Documents\GitHub\website_site\blog\2023\03\comparinig-dashboards' (via Error)
[11ty]
[11ty] Original error stack trace: Error: EISDIR: illegal operation on a directory, open 'C:\Users\iansk\Documents\GitHub\website_site\blog\2023\03\comparinig-dashboards'
[11ty] Benchmark    883ms  12%   318× (Configuration) "jsmin" Nunjucks Async Filter
[11ty] Copied 222 files / Wrote 0 files in 6.75 seconds (v2.0.1)
ERROR: "dev:eleventy" exited with 1.
PS C:\Users\iansk\documents\github\website>

This is an error I remember seeing when we first ever setup the website on your machine. It said there are permission errors on some of the images on your machine and wasn't linked to the Edge Functions. If I recall correctly, when you'd seen this previously, the website still loaded locally, but just had an image missing, but we couldn't pinpoint why this occurred, although this now appears to be failing entirely, as the error doesn't seem tied to an image file, but the blog content?
I'll need to do some research into that particular error code you're seeing, but CC'ing @knolleary to see if he can think of anything that may cause it from an Operating System perspective?

@Steve-Mcl when you get a free moment, could you try pulling latest of Website down and statting/restarting a few times please? You're the only other windows-based dev that we have atm iirc

Would tomorrow morning be ok ?

Anytime this week, async, wasnt meant to be a "can you do this now please?"

Just looked at this again:

C:\Users\iansk\Documents\GitHub\website_site\blog\2023\03\comparinig-dashboards

There is a missing slash before _site, and a typo in comparinig
Why there is a missing slash (this is an automated script that Eleventy runs) and typo, I do not know... but its something to go in at least

Anytime this week, async, wasnt meant to be a "can you do this now please?"

I should really be more explicit :D I really meant "does this need doing before launch" (or is after the panic of launch OK) :)

Thanks for the update Ian, if it does happen again, please do flag here. The 404 issue is this: https://github.com/flowforge/website/issues/632


Remove pull from reconciler

The current version of the reconciler uses a pull model when rendering, where each property is "materialised" on each render. This slows down the reconciler quite a bit, so a new model is proposed where the reconciler delegates property updates to its nodes and each node handles property updates by listening to changes in the Skia values through the dependency manager.
In addition we have a few places in the reconciler where we create new Skia objects on each render which is not very efficient since it invalidates the caching possibilities in the JSI implementations of the Skia objects (JsiHostObject).
Updates:

JsiSkPaint (C++/web) has now got a reset method so that we can reuse paint objects without recreating them.
Canvas.tsx now has a few optimisations where we try to avoid recreating Skia objects and instead reuse existing ones.
DependencyManager now tries to subscribe to values used in the component in an optimised way. Each value has only ONE subscription to the native value - and then each property in each component that uses the value subscribes to the DependencyManager instead. In addition some names have been refactored for a better fit to the implementation.
Node/Declaration/Drawing has been updated to use a subscription model in stead of the pull model

Should be merged after #704

This is an exciting PR. Can't wait to ship this major improvement.
The Paint.reset() is great too and would be ready to be merged in another PR immediately.
Here are some comments:

The Node component should have an extra property named materialized that contain the property values.  Two things to consider here:

We may need the materialize function to get the "default" values (not sure if these are actually needed beyond to make TypeScript happy).
We may have a better name here (_props and animatedProps) but because this is quite fragile to refactor due to our use of JS setters/getters, we should probably move carefully here and rename only when we know the feature is working flawlessly
SubscriptionInfo` could look like this:

type AnimatedProp = {
  name: string;
  value: SkiaValue<number>;
  unsubscribe: null | Unsubscribe;
};
type SubscriptionInfo = {
   // not in love with the props name here we could keep values as a name
  props: AnimatedProp[];
  unsubscribe: null | Unsubscribe;
};

This would make subscribeNode look like this:
  subscribeNode(node: Node, props: Props) {
    const values = Object.keys(props)
      .map((name) => {
        const propValue = props[name];
        let unsubscribe: Unsubscribe | null = null;
        let value: SkiaValue<unknown>;
        if (isSelector(propValue)) {
          value = propValue.value;
          unsubscribe = value.addListener((v) => {
            node.materialized[name] = propValue.selector(v);
          });
        } else if (isValue(propValue)) {
          value = propValue;
          unsubscribe = value.addListener((v) => {
            node.materialized[name] = v;
          });
        } else {
          return null;
        }
        return { name, value, unsubscribe } as AnimatedProp;
      })
      .filter((v): v is AnimatedProp => v !== null);

    if (values.length > 0) {
      this.subscriptions.set(node, { props: values, unsubscribe: null });
    }
  }

And
  unsubscribe() {
    this.subscriptions.forEach(({ unsubscribe, props }) => {
      props.forEach(({unsubscribe: unsub}) => {
        if (unsub) {
          unsub();
        } else {
          throw new Error("This shouldn't have happened 🤔")
        }
      })
      if (unsubscribe) {
        unsubscribe();
      }
    });
    this.subscriptions.clear();
  }

Does this look like a good model to do it? I'm interested to see if there are any oversights there.

All the design goals you stated above are great and I would like to add one to the list: Nodes should know nothing about animations. It looks like everything could be setup in the DependencyManager. Or at least we should explore this objective.
Updating the node props directly is fantastic, but still, the types should be refined upstream. This is also where tests are really fun, we can write a test to check that the props object in the renderer is already a clone from the element props.
To contain everything in the dependency manager, I was thinking about the following data structure:
type Mutator = (v: unknown) => void;
type SubscriptionState = {
  nodes: Map<Node, Mutator[]>;
  unsubscribeListener: null | Unsubscribe;
  // This one could be useful because it would allow us to remove subscriptions property
  //unsubscribeValue: null | Unsubscribe;
};

That would make the subscribeNode look something like this:
  subscribeNode(node: Node, props: Props) {
    const values: SkiaValue<unknown>[] = [];
    mapKeys(props).forEach((prop) => {
      const value = props[prop];
      if (isValue(value)) {
        values.push(value);
        const state = this.getValueSubscription(value);
        const nodeState = this.getValueNodeSubscription(state, node);
        nodeState.push((v) => {
          node.props[prop] = v;
        });
        node.props[prop] = value.current;
      } else if (isSelector(value)) {
        values.push(value.value);
        // Same dance as above
      }
    });

    if (values.length > 0) {
      this.subscriptions.set(node, { values, unsubscribe: null });
    }
  }

and subscribe():
    for (const [value, state] of this.values) {
      state.unsubscribeListener = value.addListener((v) => {
        [...state.nodes.values()].forEach((mutators) => {
          mutators.forEach((mutator) => {
            mutator(v);
          });
        });
      });
    }

If we could design this change in a way where Node doesn't anything about animations, I would be really excited.

All the design goals you stated above are great and I would like to add one to the list: Nodes should know nothing about animations. It looks like everything could be setup in the DependencyManager. Or at least we should explore this objective.

I'm a bit unsure about this requirement/use case? I agree about testability, but are there others?
I'll take a look and see what I can do - I have a strong opinion about making the dependency functionality as simple as possible to avoid extra performance and memory overhead as mentioned earlier.

I couldn't get a sense of what unregisterRedrawRequests  or dirtyFlag are doing. This is also where tests could be helpful to get a sense of the kind of problems these little things are helping with.

The dirty flag was just something we discussed as a feature that could be used in the reconciler, it can easily be removed, will fix this :)
Will also come up with a better name for the unregisterRedrawRequests :) I see it doesn't make a lot of sense.

About the requirement/use-case. I find it quite important to have the feature self-contained in DependencyManager. If it is possible to have HostConfig.ts and Node.ts not having any modifications (beyond the type refinement on _props we discussed), that would be wonderful.
The reasons I'm attached to this goal, provided that it is achievable, are the following:

Keeping the Node class free from any animation logic will enable further performance optimizations
The feature will be easier to understand/maintain/debug if self contained in the dependency manager which in its current form has a clear and simple contract. Separation of concerns is always great
I'm seeing a small simplification in the dependency manager if we have the implementation self-contained, as mentioned in my latest review.

Done!! :) 👍

moved subscription handling into dependency manager
added unit tests for the dependency manager
Cleaned up types for Node props.


[RFE] "High Security" mode

Current situation
It's currently possible to run arbitrary eBPF scripts in a k8s cluster with gadget via kubectl gadget script. In secure environments this could lead to various security issues if kubectl gadget script is abused.
Impact
Without restrictions around what gadgets can be enabled in a cluster, we are unable to use inspektor gadget in production.
Ideal future situation
We'd like to see a "high security" mode for inspektor gadget where a list of enabled gadgets are pre defined in configuration or via some other alternative implementation.
When a user attempts to run kubectl gadget script or similar an error message should be returned to the user.

Hi @eiffel-fl,
Don't worry too much about any short term fixes I just wanted to put this on your roadmap as a nice-to-have.
It's good to hear this is being planned.


testing PR previews on the render platform

🤦♂  Whoops. I meant to create a PR on my fork, haha. Sorry for the noise!


Build a multi-container Dapr application can't run successfully

The 《Build a multi-container Dapr application》 section seems missing implementation code for method: 'weatherforecast'  of 'MyBackEnd', the demo can't run successfully.

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

ID: 83e95a3f-447b-8ca0-beb5-f0363896fe29
Version Independent ID: b24e3ee5-404c-98a2-042e-6ac2d363b842
Content: Get started with Dapr
Content Source: docs/architecture/dapr-for-net-developers/getting-started.md
Product: dotnet-architecture
GitHub Login: @amolenk
Microsoft Alias: nanil

Yep, I created a wrong type project. But it couldn't run still after I changed the MyBackEnd project to ASP.NET Core Web API.
After I press F5, nothing happen. The program look compiler successfully. But no web page appear.

If press the "OPEN IN BROWER" of myfrontend:dev in docker, it will show a error page.
Is it related to my os( macos with m1)?

Ah yes, the tutorial was written for Visual Studio 2022 on Windows. I'll try the tutorial myself on my M1 Mac Mini and see if I can get it to work.

I managed to get it running on MacOS with Rider. The example doesn't bind to any port so I guess that Visual Studio does some magic to set the port binding.
You could update the docker compose file to set up the port binding for the frontend
  myfrontend:     image: ${DOCKER_REGISTRY-}myfrontend     build:       context: .       dockerfile: MyFrontEnd/Dockerfile     ports:       - "8080:80"

Thank you so much, folks. I have tested the tutorial and it is working fine now. The issue can be closed.


python310Packages.rapidfuzz: 3.1.1 -> 3.2.0

Description of changes
Diff: https://github.com/maxbachmann/RapidFuzz/compare/refs/tags/v3.1.1...v3.2.0
Changelog: https://github.com/maxbachmann/RapidFuzz/blob/refs/tags/v3.2.0/CHANGELOG.rst

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 248169 run on x86_64-linux 1

6 packages failed to build:

pynitrokey
pynitrokey.dist
python310Packages.autofaiss
python310Packages.autofaiss.dist
python311Packages.autofaiss
python311Packages.autofaiss.dist

80 packages built:

adl
anime-downloader
anime-downloader.dist
apostrophe
atomic-operator
atomic-operator.dist
creds
creds.dist
ibus-engines.uniemoji
ledger-autosync
ledger-autosync.dist
paperless-ngx
paperwork
paperwork.dist
poetry
poetry.dist
poetryPlugins.poetry-audit-plugin
poetryPlugins.poetry-audit-plugin.dist
poetryPlugins.poetry-plugin-up
poetryPlugins.poetry-plugin-up.dist
puddletag
puddletag.dist
python310Packages.cleo
python310Packages.cleo.dist
python310Packages.fire
python310Packages.fire.dist
python310Packages.fuzzywuzzy
python310Packages.fuzzywuzzy.dist
python310Packages.jiwer
python310Packages.jiwer.dist
python310Packages.levenshtein
python310Packages.levenshtein.dist
python310Packages.nkdfu
python310Packages.nkdfu.dist
python310Packages.paperwork-backend
python310Packages.paperwork-backend.dist
python310Packages.paperwork-shell
python310Packages.paperwork-shell.dist
python310Packages.rapidfuzz
python310Packages.rapidfuzz.dist
python310Packages.syncedlyrics
python310Packages.syncedlyrics.dist
python310Packages.thefuzz
python310Packages.thefuzz.dist
trytond (python310Packages.trytond)
trytond.dist (python310Packages.trytond.dist)
python310Packages.videocr
python310Packages.videocr.dist
python311Packages.cleo
python311Packages.cleo.dist
python311Packages.fire
python311Packages.fire.dist
python311Packages.fuzzywuzzy
python311Packages.fuzzywuzzy.dist
python311Packages.jiwer
python311Packages.jiwer.dist
python311Packages.levenshtein
python311Packages.levenshtein.dist
python311Packages.nkdfu
python311Packages.nkdfu.dist
python311Packages.paperwork-backend
python311Packages.paperwork-backend.dist
python311Packages.paperwork-shell
python311Packages.paperwork-shell.dist
python311Packages.rapidfuzz
python311Packages.rapidfuzz.dist
python311Packages.syncedlyrics
python311Packages.syncedlyrics.dist
python311Packages.thefuzz
python311Packages.thefuzz.dist
python311Packages.trytond
python311Packages.trytond.dist
python311Packages.videocr
python311Packages.videocr.dist
spotdl
spotdl.dist
sublime-music
sublime-music.dist
ulauncher
ulauncher.dist


Replace local cache with Github Registry

I want to merge this change because will speed up build time.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: CLOUD-1567 Speed up Saleor builds


folder/ syntax does not grey folders in sub-folders

Folder structure:

app
ignore
some_data

ignore

Adding ignore/ in .gitignore file will cause git to ignore both ignore folder, but this plugin cannot set the 2nd ignore folder to grey. I think this is inconsistent with git behavior.

BTW, I found this issue while using Android Studio 2.0 Preview 9. I also tried to restart the studio.

@davidshen84 Thank you for reporting it, however I cannot reproduce it.
For example with following files tree:
.
├── dir
│   ├── a.txt
│   └── b.txt
├── foo
│   ├── 1.txt
│   └── dir
│       ├── 2.txt
│       └── 3.txt
└── foo.txt

I get the result:

Also git reports them as ignored:
$  git ls-files --others --ignored --exclude-standard
abs/dir/2.txt
abs/dir/3.txt
dir/a.txt
dir/b.txt

Indeed, I cannot reproduce it either. I guess it is fixed in some recent
update.
On Fri, Mar 18, 2016 at 1:23 AM Jakub Chrzanowski<EMAIL_ADDRESS>wrote:

@davidshen84 https://github.com/davidshen84 Thank you for reporting it,
however I cannot reproduce it.
For example with following files tree:
.
├── dir
│   ├── a.txt
│   └── b.txt
├── foo
│   ├── 1.txt
│   └── dir
│       ├── 2.txt
│       └── 3.txt
└── foo.txt
I get the result:
[image: image]
https://cloud.githubusercontent.com/assets/108333/13854747/3ed55500-ec6d-11e5-82f2-ceda40003083.png
Also git reports them as ignored:
$  git ls-files --others --ignored --exclude-standard
abs/dir/2.txt
abs/dir/3.txt
dir/a.txt
dir/b.txt
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/hsz/idea-gitignore/issues/189#issuecomment-197985412

--
Regards,
David

Ok, thank you for the confirmation, closing.


Entanglement entropy of the diamond chain

Dear Jie,
I would like to investigate the bipartite entanglement entropy of the spin-1 diamond chain in the absence of the magnetic field when conserved total sz and k=0 are assumed.
I have encountered some issues and I would be grateful to you if help me to solve them.
I extended the code as:
# Hamiltonian parameters:
J1 = 1.0
J2 = J1
J3 = 2.8
J4 = J1
J5 = J1
L = 2
N = 3*L
sz_min = 0
sz_max = 2*N
# -------------------------------------------------------
all_energies = []
EntEntropy_arr = []
Size_EigVals = []
Size_EntEnt = []
mun_EigVals = 0
for i in sz_min:sz_max
    Size_EigVals_k = 0
    Size_EntEnt_k = 0
        # onsite Hamiltonian matrix
        O1 = spin((J1, "xx1"), (J1, "yy1"), (J1, "zz1"), D=3)
        O2 = spin((J2, "x1x"), (J2, "y1y"), (J2, "z1z"), D=3)
        O3 = spin((J3, "1xx"), (J3, "1yy"), (J3, "1zz"), D=3)
        h1 = O1 + O2 + O3
        C1 = spin((J4, "1x1x11"), (J4, "1y1y11"), (J4, "1z1z11"), D=3)
        C2 = spin((J5, "11xx11"), (J5, "11yy11"), (J5, "11zz11"), D=3)
        h2 = C1 + C2
        Nj = diag( spin((1,"z11"), (1,"1z1"), (1,"11z"), D=3) + 3*I )
        total_z(s) = sum(Nj[sj+1] for sj in s)
        basis = TranslationalBasis(L=L, f = s -> total_z(s) == i, k=0, base=3^3)
        H = trans_inv_operator(h1, 1:1, basis) + trans_inv_operator(h2, 1:2, basis)
        vals, vecs = Array(H) |> Hermitian |> eigen
        EE = [ent_S(vecs[:,i], 1:2, basis) for i=1:size(basis,1)]
        Size_EigVals_k += size(vals, 1)
        Size_EntEnt_k += size(EE, 1)
        for sublist in vals
            for element in sublist
                push!(all_energies, element)
            end
        end
        for sublist in EE
            for element in sublist
                push!(EntEntropy_arr, element)
            end
        end
    push!(Size_EigVals, Size_EigVals_k)
    push!(Size_EntEnt, Size_EntEnt_k)
end

After running the code, for all values of the total sz I get correct eigenenergies, but all entropy values are zero.
I can not compute bipartite  EE for 1:3 or 1:4 and so on. The code gives only bipartite EE for the case 1:2.

As I understood, the above code gives EE between two neighboring blocks 1 and 2 not two spins, right? If so, how can I compute EE of arbitrary pairs of spins?

Hi Hamid,
Apologies for the delayed response, as I've been occupied lately. Regarding the questions. Since the blocked system comprises only two sites, it's only capable of providing bipartite entanglement entropy between two blocks. This can be achieved using:
ent_S(vecs[:,i], 1:1, basis)

At present, unfortunately, I do not have a straightforward workaround within the EDKit functions to compute entanglement within a block.

Dear Jie,
Thank you very much,
I could calculate entanglement negativity between two blocks by using expression ent_S(vecs[:, i], 1:1, basis).
Please know that the code works only for the case k=0.
could you please comment on the physical meaning of the ent_S(vecs[:, i], 1:1, basis) and why it works only for the case k=0?
Please also know that in QuSpin, I need to consider the case   here is a part of my QuSpin code:
for mz in range(sz_min, sz_max+1):
    basis = spin_basis_1d(L, pauli=False, S="1", m=mz / L, kblock=0)
    H = hamiltonian(static, dynamic, basis=basis, dtype=np.float64,
                    check_symm=False, check_herm=False, check_pcon=False,
                    )
    E, V = H.eigh()
    Energies.append(E)  # Append only the ground state energy
    ent = basis.ent_entropy(V, sub_sys_A=(0, 1, 2, 3, 4, 5), density=False, enforce_pure=True)["Sent_A"]
    EnEnt.append(ent)

I am not sure that it is possible to take in to account other values of kblock symmetry in QuSpin to calculate EE for the diamond chain, as well.
I would be thankful to you if comment in this case as well.

Hi Hamid,
The entanglement for k≠0 is possible. For this L=2 case, the other possible k is pi. In EDKit's convention, you can set k=1, and the following commands should return the entanglement entropy:
basis = TranslationalBasis(L=L, f = s -> total_z(s) == i, k=1, base=3^3)
H = trans_inv_operator(h1, 1:1, basis) + trans_inv_operator(h2, 1:2, basis)
vals, vecs = Array(H) |> Hermitian |> eigen
EE = [ent_S(vecs[:,i], 1:1, basis) for i=1:size(basis,1)]


fix: improve error messages. fixes #38

This addresses the error message mentioned in issue #38
In addition, this also mentions the path that is not a directory, which had a similar issue of not actually giving the user the info he needs to solve the problem.
Tested it out and it now produces error messages like this:
$ yii help

Error: Path is not writable: /home/ubuntu/runtime/console/backups/

Why closed without merge?


NIFI-7187

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on both JDK 8 and JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

i'll rebase and get it to run against latest fun stuff

Thanks for updating all of these @joewitt! I was able to verify all unit tests, etc. pass. Ran the battery of system tests and all looks good. +1 merged to master.


Fix cupyx.optimize to save file when no optimization ran

Previously cupyx.optimize context manager does not save a file when no optimization run within the scope.
I think it is better to save an empty result instead to make the behavior consistent.
This PR also fixes the issue that cupyx.optimize tests fail when CUB acceleartor is enabled.

Jenkins, test this please

Jenkins CI test (for commit a58d3952bebe285573c5eb16133c08fd2285593f, target branch master) failed with status FAILURE.

pfnCI, run Jenkins please.

Jenkins CI test (for commit a58d3952bebe285573c5eb16133c08fd2285593f, target branch master) succeeded!


New feature request: dynamic reload

Is it possible to trigger a reload of the map data from JSON without taking the map off the screen? Just adding the new points? And maybe taking away points, or re-categorizing points, that aren't needed any more?

Hi!
there is a Ed3d.rebuild(): function but not totally perfect at this moment to reload completelly the scene if your JSon has changed,
is it enough ? or need something more specific ?


fix(npm): remove files attribute in package

:tada: This PR is included in version 1.0.8 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


New blog post: Dynamic Stream (on SharePoint) video playback in PowerApps

Category

[ ] Content fix
[x] New article

Contents of the Pull Request

This is a new blog post on how to dynamically playback Stream on SharePoint videos in PowerApps using the new Stream player and Power Automate.

hi there @anandragav - thank you for contributing! Just double checking - would you like to still adjust your article? Power Virtual Agents is now Copilot Studio...

@LuiseFreese yes, I have just updated the text.
Also submitting a new blog post on Dynamic Stream (on SharePoint) video playback in PowerApps


Build failure on BSD

endlessh> make
cc -ggdb3 -std=c99 -Wall -Wextra -Wno-missing-field-initializers -Os -o endlessh endlessh.c
endlessh.c:705:55: error: use of undeclared identifier 'MSG_DONTWAIT'
ssize_t out = send(fd, line, len, MSG_DONTWAIT);
^
1 error generated.
*** Error code 1
Stop.

So MSG_DONTWAIT is a non-standard, though very widely supported send(2)
flag. FreeBSD does support it, but unfortunately it hides the flag
definition when _POSIX_C_SOURCE is defined. While this is generally the
technically correct thing to do — hide non-standard features when a
specific standard is requested — this choice makes the flag essentially
impossible to use correctly: send(2) itself shouldn't be visible without
the proper _POSIX_C_SOURCE definition either. It also doesn't help that
FreeBSD is the only platform with this particular behavior. None of the
other platforms I've tested on hides this flag (though OpenSolaris-based
systems don't seem to support it at all?).
Anyway, the simplest thing to do is just remove the first line of the
file that defines _POSIX_C_SOURCE, and it will build just fine. FreeBSD
(nor the other BSDs) has never been precise about feature test macros
anyway.
The think actual correct, portable thing to do is to put the socket
non-blocking mode using fcntl(2). MSG_DONTWAIT is just a convenience for
making the socket non-blocking for that send(2).

a funny project~ since it's not a CPU intensive task, python or go may be a better choice, more portable with a smaller code base.


incompatible library version

I got this error when i tried to run.
"Reason: Incompatible library version: libvips-cpp.42.dylib requires version 38.0.0 or later, but libpng16.16.dylib provides version 35.0.0"

Hello, running brew update && brew upgrade and/or brew unlink libpng && brew link libpng might help. See #323 too.

brew update && brew upgrade solve my problem. Thank you.


instagram - patch to deal with images with wrong "video" type

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2<IP_ADDRESS>. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2<IP_ADDRESS>

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Description of your issue, suggested solution and other information
Sometimes an instagram profile contains an image which is wrongly described in the json data as having type: video, instead of type: image. (On the website these are displayed as images with a little square icon in the top-right corner, dunno what that means)
When youtube-dl runs into one of these images, it aborts with the following error:
ERROR: An extractor error has occurred. (caused by KeyError('videos',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/common.py", line 427, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/instagram.py", line 264, in _real_extract
    } for k, v in it['videos'].items()]

KeyError: 'videos'

This happens because the "fake" video item is in fact an image, so it doesn't have the videos json element (which describes the available video formats and urls for the item).
For a quick fix: add a check for existence of the videos element, see the attached patch for /extractor/instagram.py
(This works for me, I'm not sure if other changes are needed/better, please review).
instagram.py.txt

Provide example URL.
If you want this to be merged you have to add a test and open a pull request.

Closing as no response for a long time.


feat(html): process static asset urls with urlToBuiltUrl

Description
Files in /public referenced by index.html are now processed with urlToBuiltUrl.
Used by vite-plugin-public to insert a content hash.
Additional context
Follow-up to #1648

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Closing as outdated, and it seems that vite-plugin-public is using the older plugin format, so this PR wouldn't help it. If this feature is still needed, please open a new PR.

Actually it seems to not use an older plugin format, but the plugin uses a new feature in https://github.com/vitejs/vite/pull/1675. I'll continue the discussion in that PR.


Make cache-control remember its header value.

Closes #1255.

I updated this to be a bit less clever. The presence of any "Pragma" header or second "Cache-Control" header invalidates the cached value.

LGTM


Add back in ability to customize top-level box of Radio component

Is your feature request related to a problem? Please describe.
Radios used to allow us to style the outermost box of the radio, but now it looks like it only accepts {...layoutProps}. This has made us unable to use the same styles we had before upgrading (and I don't see any way to apply styles on the new Radio to achieve the same effect).
For example, we previously styled Radios like this:

Now they are stuck looking like this:

Describe the solution you'd like
I believe the simplest way to solve the issue while keeping with the current style paradigm would be to add a new sub-style for the outermost box (similar to how there is already a _radio [which seems like it was deprecated] and _interactionBox style group). I've encountered this same limitation on a few other components as well (like on the Checkbox component).
Describe alternatives you've considered
I've considered trying to hack the component to make it work by applying a "position: relative" layoutProp to the top-level box and then applying "position: absolute" to the Center child component, but this isn't possible because all layoutProps get applied to the top-level Box only.
I've also considered creating my own wrapper or rolling an entirely custom Radio component, but I'm trying to avoid it.

Hi @kylegwalsh, thanks for reporting this issue. We'll look into this.