add admin functionality to disable module incase of failure.

1. Overview
The messages defined in x/lscosmos/handler.go:21-35 can only be executed when the module state is enabled. If the module is disabled, the epoch logic and main messages, such as liquid staking, are prohibited. However, the current implementation only allows the module to be disabled from InitGenesis, as seen in x/lscosmos/genesis.go:16. This implies that all validators would need to coordinate to halt and upgrade the chain in order to set the module to disabled.
Coordinating an upgrade with validators is a manual and time-intensive progress, which prevents swift reactions to security incidents. For example, suppose the ATOM chain halts, and the module needs to be temporarily disabled.
We recommend implementing the following suggestions:
2. Implementation details

adds admin functionality to disable/reenable the module state.

3. How to test/use
manual

Who will be the admin and how can we set the admin?

We had set it in genesis. And we can change/ set it with a setFeeAddress proposal which has to pass on persistence chain


Branch update developer guide

Add manual instructions for testing.
Update find and list command sequence diagrams.

Codecov Report

Merging #294 (0aa854d) into master (ccb11f0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #294   +/-   ##
=========================================
  Coverage     34.87%   34.87%
  Complexity      348      348
=========================================
  Files           114      114
  Lines          2162     2162
  Branches        183      183
=========================================
  Hits            754      754
  Misses         1354     1354
  Partials         54       54

Continue to review full report at Codecov.

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


Add feature multibots

I would like to modify the DB, and its classes, to add the field
bot_id int (10) unsigned
to manage multiple bots with the same 12 main tables.
Then you can still change the name of the tables to make groups of bots, for example, based on customers.
Are you interested in this modification of your code?
Before changing the code, I await your confirmation.
Thank you
SQL:
ALTER TABLE `tb_botan_shortener` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_botan_shortener` ADD INDEX `bot_id` (`bot_id`);

ALTER TABLE `tb_callback_query` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_callback_query` ADD INDEX `bot_id` (`bot_id`);

ALTER TABLE `tb_chat` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_chat` ADD INDEX `bot_id` (`bot_id`);

ALTER TABLE `tb_chosen_inline_result` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_chosen_inline_result` ADD INDEX `bot_id` (`bot_id`);

ALTER TABLE `tb_conversation` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_conversation` ADD INDEX `bot_id` (`bot_id`);

ALTER TABLE `tb_edited_message` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_edited_message` ADD INDEX `bot_id` (`bot_id`);

ALTER TABLE `tb_inline_query` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_inline_query` ADD INDEX `bot_id` (`bot_id`);

ALTER TABLE `tb_message` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_message` ADD INDEX `bot_id` (`bot_id`);

ALTER TABLE `tb_request_limiter` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_request_limiter` ADD INDEX `bot_id` (`bot_id`);

ALTER TABLE `tb_telegram_update` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_telegram_update` ADD INDEX `bot_id` (`bot_id`);

ALTER TABLE `tb_user` ADD `bot_id` int unsigned NOT NULL COMMENT 'Bot identifier' FIRST;
ALTER TABLE `tb_user` ADD INDEX `bot_id` (`bot_id`);

Thanks for your suggestion!
I guess what you're suggesting is implemented by using the table prefix when setting up the MySQL database connection:
$table_prefix = 'customerN_';
$telegram->enableMySql($credentials, $table_prefix);

What advantages do you see with adding a new field as you do, instead of prefixing tables?

Example:
For 5 customers and 100 bots, we now need 1200 tables ( = 12 * 100 bots)
Having the multibots feature it would be possible to manage only 12 main tables + 1 (customers / bots)  or 60 + 1 tables (= 12 * 5 customers)

Ok, I try to get more involved in the reasons for it.

with large DB files you make a backup faster than the many small files :-D (physical backup not logical)
with less tables the statistical queries of the various bots are simpler. 1 query multi bots stats.
it is easier to make relationships with other tables ... of my project. [-:
there are many changes, but I do it, then you just have to check :-)
the bot_id already recover it on the __construct telegram class. I only need to save it in the db, use it in select and pass it to the other classes.

    preg_match('/(\d+)\:[\w\-]+/', $api_key, $matches);
[...]
    $this->bot_id = $matches[1];

I do not insist further, if you do not like the idea, ok ... no problem. (-:

I evaluated the situation again
I must be able to filter the commands for the different bots; each bot does not have access to all the commands in the commands folder.
At this point I choose to write all the code I need.
Thanks for the attention.

I like this idea, but we should think more about implementation details
Maybe it is better to add bots table and register each bot in that table? And use a foreign key in all related tables as a bot_id

Maybe you'd like to have a look at #613?

now I evaluate, thanks

Ok! This was one of the things that you and @MBoretto said you'd implement as far as I remember, which is why I haven't really made a move on this.
Let's move this conversation to a new issue (or maybe continue on #170)

@noplanman this is not easy because of library architecture. Lets continue here https://github.com/php-telegram-bot/core/issues/826

Is not easy like that, In messages table for example you have double primary index:
https://github.com/php-telegram-bot/core/blob/master/structure.sql#L106
adding multiple bot the primary index will increase from 2 to 3 fields. Otherwise it can happen that messages collide.

Plus, if I may add, I think the whole DB feature (and so does Conversation) should be handled as plugin or addon after the refactor, those things shouldn't be core of the library because in larger project people might want to use their own implementations of both of these features.

hi guys, I tried to change the code according to my needs.
https://github.com/customsoft/php-telegram-bot/compare/multibots?w=1
I have not tested it completely yet, but I share it.
I hope the functionality will be included in the next versions.
I have also added the CreateTables method on DB Class; it is very convenient to create tables with the prefix
Remember that it is to be tested
Have a good time!


[RIP-9] Add the introduction of the basic concepts in RocketMQ

[RIP-9] Add the introduction of the basic concepts in RocketMQ
What is the purpose of the change
Support order producer
Brief changelog
Add the introduction of the basic concepts in RocketMQ
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage remained the same at 44.824% when pulling 2e82be008b18d123ef4a5f01c9af8e2cfe34602c on TJH54:develop into 6a573a773a87edbcdcc138614b756019fc280988 on apache:develop.

@xiangwangcheng Could you please review this PR  again as it is updated yesterday.


feat: Prevent Nested Network Identities

See also https://github.com/vis2k/Mirror/pull/2869
Uses OnValidate and Reset to prevent users from adding components that would conflict that Unity can't catch on its own:

NetworkIdentity in same object hierarchy as NetworkManager
NetworkIdentity in same object hierarchy as another NetworkIdentity (prevents nesting)

did a test with benchmark scene and monster prefab.
added a child GO with NetworkIdentity.
press play, server & play.
we already have a check + error:

The point of this PR and #2869 is to block the user at design time, instead of waiting until play mode / run time.


FEATURE: call hierarchy

Did you check the docs?

[X] I have read all the docs.

Is your feature request related to a problem? Please describe.
Vscode and intelliJ offer developers call hierarchy. It shows all the functions that were called up until this point. The LSP supports this information for incoming and outgoing call hierarchies but there's no visual representation for it. Would love to see support here for it.
vim.lsp.buf.incoming_calls
vim.lsp.buf.outgoing_calls
Describe the solution you'd like.
In a tree-like structure, show hierarchy of function calls allowing developers to expand and collapse.
Describe alternatives you've considered.
No response
Additional Context
No response

I don't hate this idea and given we have the document symbols source already, this seems like a natural extension. Feel like trying your hand at implementing a neo-tree source? You could almost certainly use the document symbols source code as a starting point as this is in the same space as that

Unfortunately I'm not well versed with lua or neovim. I was able to find a small repo that implemented the desired feature that can be referenced for whomever adds support here.


Where to include the reference to the javascript

Sorry to be so dense, but I just couldn't see where to do this.
In the readme (https://github.com/StudioJunkyard/summernote-cleaner#1-include-js) it says "Include the following code after you include Summernote." Where exactly is that? I have a Rails app that I've inherited, and I "include" Summernote in a few different places (Gemfile, application.js, stylesheets/base.scss). None of those places seem the right spot for the html code the readme presents. Any suggestions would be amazing!

Turns out I just had to put the js file in the file structure such that my current includes (from application.js) caught it. Sorry about the confusion!

All good, glad you worked it out.


Upgrading windows client to 1.0.3 breaks sensu-client

Expected Behavior

After upgrading to 1.0.3 all my windows clients failed to start sensu client
Error:

Unable to load the EventMachine C extension; To use the pure-ruby reactor, require 'em/pure_ruby'
Unable to load the EventMachine C extension; To use the pure-ruby reactor, require 'em/pure_ruby'
C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require': cannot load such file -- 2.4/rubyeventmachine (LoadError) 	from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require'
from C:/opt/sensu/embedded/lib/ruby/gems/2.4.0/gems/eventmachine-1.2.5-x64-mingw32/lib/rubyeventmachine.rb:2:in <top (required)>' 	from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:133:in require'
from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:133:in rescue in require' 	from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:40:in require'
from C:/opt/sensu/embedded/lib/ruby/gems/2.4.0/gems/eventmachine-1.2.5-x64-mingw32/lib/eventmachine.rb:8:in <top (required)>' 	from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:133:in require'
from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:133:in rescue in require' 	from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:40:in require'
from C:/opt/sensu/embedded/lib/ruby/gems/2.4.0/gems/sensu-logger-1.2.1/lib/sensu/logger/stream.rb:2:in <top (required)>' 	from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require'
from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require' 	from C:/opt/sensu/embedded/lib/ruby/gems/2.4.0/gems/sensu-logger-1.2.1/lib/sensu/logger.rb:1:in <top (required)>'
from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require' 	from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require'
from C:/opt/sensu/embedded/lib/ruby/gems/2.4.0/gems/sensu-1.0.3/lib/sensu/daemon.rb:22:in <top (required)>' 	from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require'
from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require' 	from C:/opt/sensu/embedded/lib/ruby/gems/2.4.0/gems/sensu-1.0.3/lib/sensu/client/process.rb:1:in <top (required)>'
from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require' 	from C:/opt/sensu/embedded/lib/ruby/site_ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in require'
from C:/opt/sensu/embedded/lib/ruby/gems/2.4.0/gems/sensu-1.0.3/exe/sensu-client:7:in <top (required)>' 	from C:/opt/sensu/embedded/bin/sensu-client:22:in load'
from C:/opt/sensu/embedded/bin/sensu-client:22:in `'

Current Behavior

With 1.0.2 everything was fine, 1.0.3 breaks something on Windows OS (Linux works)
Possible Solution

Steps to Reproduce (for bugs)

Have a windows client (2008, 2012, 2012R2)
Install 1.0.2, it works
Upgrade to 1.0.3 it dont work

Context

Upgrading windows to 1.0.3 breaks all sensu-client, in the changelog i see that the EventMachine version 1.2.5 is used. There seems to be a problem with windows.
Reverting to 1.0.2 do not work too... :(
Your Environment

Sensu version used: 1.0.2 & 1.0.3
Active Sensu extensions (e.g. WizardVan 0.1.0):
Operating System and version (e.g. Ubuntu 14.04): CentOS, Windows, etc
Transport and runtime versions (e.g. RabbitMQ 3.6.2 on Erlang 18.2):

@amdprophet I believe we need to update sensu-omnibus to compile/build the new EM version (with openssl support) on windows.

Kinda seems like 1.03 should be retracted since it totally breaks.

Ah, it was. Nevermind! :)  Reverting to 1.0.2 still failed though, perhaps i need to clean up its install.

@benh57 yes it should be retracted ... It has breaked all my windows OS (200)
And yes you have to clean all the install to be able to make 1.0.2 work again

We are actively working on a fix.

We've pulled 1.0.3-1 Windows builds from the repositories and 1.0.3-2 should be available tomorrow.

I'm going to close this issue as Sensu 1.1.0 has been released with a fix for this problem. Please make sure to uninstall Sensu 1.0.3 before installing 1.1.0.

@amdprophet Sorry but i have tried to uninstall 1.0.2 (1.0.3 didnt work on Windows) then install 1.1.0 and it didnt work either.
It seems that uninstalling 1.0.2 keep windows service (and maybe some dependencies) and 1.1.0 start but dit not work for some reason ... So we have a Windows sensu client problem...

@amdprophet  @palourde
It warn me about a RabbitMQ connection problem but the config is the same as the 1.0.2 and it was working fine and it works great on all my linux clients (500)

{"timestamp":"2017-10-06T10:03:40.753653+0200","level":"warn","message":"transport connection error","reason":"possible authentication failure. wrong credentials?","user":"sensu"}


nrpe: "CRITICAL: mysql.service is not running" on leader/bootstrap unit (active/exited state)

$ juju deploy -n 3 percona-cluster
$ juju deploy nagios
$ juju deploy nrpe
$ juju add-relation nagios nrpe
$ juju add-relation nrpe percona-cluster
Nagios reports "CRITICAL: mysql.service is not running" on the leader/bootstrap unit.

(by nobuto)

(by nobuto)
I believe the background in Bug: #1664025 is related.
● mysql.service - LSB: Start and stop the mysql (Percona XtraDB Cluster) daemon
   Loaded: loaded (/etc/init.d/mysql; bad; vendor preset: enabled)
   Active: active (exited) since Mon 2017-04-24 04:37:14 UTC; 28min ago
     Docs: man:systemd-sysv-generator(8)
    Tasks: 0
   Memory: 0B
      CPU: 0

Apr 24 04:37:14 juju-195a01-0 systemd[1]: Starting LSB: Start and stop the mysql (Percona XtraDB Cluster) daemon...
Apr 24 04:37:14 juju-195a01-0 mysql[16095]:  * Starting MySQL (Percona XtraDB Cluster) database server mysqld
Apr 24 04:37:14 juju-195a01-0 mysql[16095]:    ...done.
Apr 24 04:37:14 juju-195a01-0 systemd[1]: Started LSB: Start and stop the mysql (Percona XtraDB Cluster) daemon.
Apr 24 04:37:25 juju-195a01-0 systemd[1]: Started LSB: Start and stop the mysql (Percona XtraDB Cluster) daemon.

UnitId: percona-cluster/0

(by nobuto)
https://bugs.launchpad.net/charm-percona-cluster/+bug/1664025

(by nobuto)
A simple workaround would be rebooting the leader unit after the cluster is up and running.

(by szeestraten)
Ran into the same issue.
ubuntu:~$ juju status mysql
Model      Controller  Cloud/Region  Version
openstack  prodcont1   prodmaas      2.1.2
App              Version       Status   Scale  Charm            Store       Rev  OS      Notes
mysql            5.6.34-26.19  active       3  percona-cluster  jujucharms  250  ubuntu
mysql-nrpe                     unknown      3  nrpe             jujucharms   21  ubuntu

(by james-page)
This happens because the lead unit does not get started using the normal start target provided by the init.d scripts (which are wrapped by systemd normally).
The lead unit is started using the bootstrap-pxc command directly initiated from the init.d script, rather than via the systemctl command.

(by james-page)
The systemd mysql unit records - "active (exited)" rather than "active (running)" as the process is not tracked by systemd when the bootstrap path is followed.

(by james-page)
I'd really rather avoid having to restart percona-cluster just to get the monitoring right; maybe looking at the 'ActiveState' rather than the 'SubState' might make sense here - the unit is active, but is in 'exited' SubState, which is what is causing the problem with the nrpe check.

(by james-page)
Raised a task on the NRPE charm and marked the PXC task as won't fix; I don't think we can resolve this in the percona-cluster charm purely due to the way that pxc has to be bootstrapped.

(by nick-moffitt)
For future travellers, the fix is likely line 23 of https://git.launchpad.net/nrpe-charm/tree/files/plugins/check_systemd.py and the tests after it (string will be "active" rather than "running" when we use the ActiveState).  This is worth testing on multiple installations with a hand-rolled copy to make sure it behaves as we expect in production.

(by dpaclt)
Will this be fixed in next release of nrpe ie nrpe#42 ?

(by nobuto)
Indeed, it's not reproducible with the current latest charm. rev no. 266 seems to bring the difference.
$ for i in 267 266 265; do
juju add-model xenial-$i
juju deploy --series xenial percona-cluster-$i -n 3
juju config percona-cluster min-cluster-size=3
done
$ for i in 267 266 265; do echo percona-cluster-$i; juju run -m xenial-$i --application percona-cluster -- systemctl status mysql | grep Active:; done
percona-cluster-267
Active: active (running) since Mon 2018-07-09 16:31:36 UTC; 17min ago
Active: active (running) since Mon 2018-07-09 16:32:03 UTC; 16min ago
Active: active (running) since Mon 2018-07-09 16:32:36 UTC; 16min ago
percona-cluster-266
Active: active (running) since Mon 2018-07-09 16:32:03 UTC; 16min ago
Active: active (running) since Mon 2018-07-09 16:32:30 UTC; 16min ago
Active: active (running) since Mon 2018-07-09 16:33:09 UTC; 15min ago
percona-cluster-265
Active: active (exited) since Mon 2018-07-09 16:28:44 UTC; 20min ago <<<<<<<<<<
Active: active (running) since Mon 2018-07-09 16:31:08 UTC; 17min ago
Active: active (running) since Mon 2018-07-09 16:31:49 UTC; 17min ago
https://api.jujucharms.com/charmstore/v5/percona-cluster-266/archive/repo-info
https://api.jujucharms.com/charmstore/v5/percona-cluster-265/archive/repo-info

(by nobuto)
The commit 801c2e78294f7af0a058aca036b6d066e3f53a98 in percona-cluster charm has a fix for this issue as well because it restarts mysqld after bootstrapped.
$ git bisect log
git bisect start '--term-new=fixed' '--term-old=unfixed'
fixed: [78d1df2d50d94bae829c33767bc0e99a7e2afcf1] Set server_id when using binlogs
git bisect fixed 78d1df2d50d94bae829c33767bc0e99a7e2afcf1
unfixed: [bf84218e31a832a04fc15385997cba8d2ec4eed6] Merge "Sync charm-helpers"
git bisect unfixed bf84218e31a832a04fc15385997cba8d2ec4eed6
fixed: [19e1fce3db6d4e4b0cdd4946020838d3055f5522] Update tox.ini to stop using unverified package
git bisect fixed 19e1fce3db6d4e4b0cdd4946020838d3055f5522
unfixed: [9cac8b85211c01fa7cb5637568bd63dfd3b21844] Merge "Add support for PXC 5.7 and xtrabackup 2.4"
git bisect unfixed 9cac8b85211c01fa7cb5637568bd63dfd3b21844
fixed: [801c2e78294f7af0a058aca036b6d066e3f53a98] Redesign cluster buildup process
git bisect fixed 801c2e78294f7af0a058aca036b6d066e3f53a98
first fixed commit: [801c2e78294f7af0a058aca036b6d066e3f53a98] Redesign cluster buildup process
Reviewed: https://review.openstack.org/555494
Committed: https://git.openstack.org/cgit/openstack/charm-percona-cluster/commit/?id=801c2e78294f7af0a058aca036b6d066e3f53a98
Submitter: Zuul
Branch: master
commit 801c2e78294f7af0a058aca036b6d066e3f53a98
Author: David Ames
Date: Mon Mar 19 11:25:56 2018 -0700
Redesign cluster buildup process

In order to fix bug#1756928 the whole cluster buildup process needed to
be redesigned. The assumptions about what is_bootstrapped and clustered
meant and when to restart on configuration changed needed to be
re-evaluated.

The timing of restarts needed to be protected to avoid collisions.
Only bootstrapped hosts should go in to the
wsrep_cluster_address=gcomm:// setting. Adding or removing units should
be handled gracefully. Starting with a single unit and expanding to a
cluster must work.

This change guarantees mysqld is restarted when the configuration file
changes and meets all the above requirements. As a consequence of the redesign,
the workload status now more accurately reflects the state of the unit.

Charm-helpers sync to bring in distributed_wait fix.

Closes-Bug: #1756308
Closes-Bug: #1756928
Change-Id: I0742e6889b32201806cec6a0b5835e11a8027567

(by seyeongkim)
Nobuto
Cool Thanks for checking it.
James
Not sure we need to fix nrpe as well ( checking ActiveState than SubState )


Update layer symbols and labels after initialization

DotSpatial version:  2.0
So I have a map that has been initialized and displays countries and cities. The user clicks on a button and a path is generated between two cities. For every request I want to update the symbology and label filters.
lyrCities.Symbology.Categories(0).FilterExpression = "[City]='Dallas'"
lyrCities.LabelLayer.Symbology.Categories(0).FilterExpression = "[City]='Dallas'"

After these two lines the map doesn't change but if I go to the layer properties i can see that the filters have changed. How do I apply them programatically?

OK here is what I did for symbology and labels
        'filter the feature symbology for cities
        Dim pntSch As IPointScheme = lyrCities.Symbology
        Dim pntCat As IPointCategory = pntSch.Categories(0)
        pntCat.FilterExpression = "[City]='Dallas'"
        lyrCities.ApplyScheme(pntSch)

        'filter the feature labels for the cities
        Dim lblSch As ILabelScheme = lyrCities.LabelLayer.Symbology
        Dim lblCat As ILabelCategory = lblSch.Categories(0)
        lblCat.FilterExpression = "[City]='Dallas'"
        lyrCities.LabelLayer.CreateLabels()


Community upgrade: Small fixes

Better design of badges at /entity/unrevised (see below)
Fix: show badges also in notifications (Fixes #398)

Before:

After:

Merged in 56b5ced626b0ab07d8c52186a9234cb30d3e575a


CRM Response Times

Due to unforeseen circumstances, some users may experience longer than usual wait times with CRM Trickey Responses.
This should be cleared up within 24-48 hours.
Your patience is appreciated at this stage. -MPTA

Thank you for your understanding.
The CRM Is back to normal operations.
Thank You. - MPTA


fix horizontal scroll

Fix horizontal scroll

Hi Santosh!
Thanks a lot for your fix, we'll apply it now following our flow.
Have a nice day!

Associated commit: https://github.com/starton-io/docs/commit/0daabb0b81b206c4fafdf8cd712a98a6fbe3968e.


Master branch is behind 1.1.0 tag

The 1.1.0 tag is basically the hash d674c31... I searched in your entire master branch commits, the HEAD is at a7fcfb8e and I cannot find the tag anywhere.

@msound yeah, seems like a commit snuck into the tag that was missing from master [d674c31ed7], but it looks like this is also in an outstanding PR https://github.com/devinci-code/ahoy/commit/f9b55fcc1e064e1d9c3242211cfb922c0555b9f5

I just merged 1.1.0 into master.


optimize: optimize ParameterParserTest test case failed

Signed-off-by: slievrly<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
optimize: optimize ParameterParserTest test case failed
Ⅱ. Does this pull request fix one issue?
fix #3364
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

只清除环境变量，并不能让ConfigurationFactory.CURRENT_FILE_INSTANCE进行变化。


⚠ Samsung has degraded performance

In 1bfe47a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 5435 ms

Resolved: Samsung performance has improved in 0862184.


Release version 0.3.0

After #74 is merged. #74 will fix #71, and is a breaking change (since it includes semver breaking updates of multiple dependencies), so it will have to be version 0.3.0 instead of version 0.2.3.

Could someone look into build/CI issues with #74? I won't have time to get to it this week myself, but can take care of releasing 0.3.0 once it's merged. Otherwise I will get to it myself next week.

I will if I have time. Just post a comment here if you're going to start working on it and I'll do likewise so that we don't reproduce work esp. since we're sharing that feature branch.

is project dead?


basic controller commands

First basic controller commands. Note this is WIP.
RFC @zeisss @JosephSalisbury

I implemented your feedback. Please have another look. Need to test and extend tomorrow.

@zeisss @JosephSalisbury I would like to merge this. My Plan is to keep the PRs smaller. Next one would be #20 to test the file system specific actions.

Could use godoc strings in a fair few places

Which go vet should have found, I'll look into it

As discussed. I will not add comments on the private stuff. It implements an interface which is already documented. There are no special details worth documenting for the specific implementation.

+1

Fishing for LGTM

and you shall have it <3
LGTM


Add bulk add endpoint to project

In cases where a user chooses to select all scenes in browse view, even if they aren't viewable we need an endpoint that supports sending the query parameters as a post argument and adds the scenes in bulk. Similar to the select all functionality in gmail.
See discussion in https://github.com/azavea/raster-foundry/pull/842#pullrequestreview-14998096

Duplicate of #974


[Feature] Promxo import changeable realm

General

[X] My feature does not exist in the latest version of Nexterm.
[X] I have checked that my feature has not been suggested by anyone else.

Your Idea
would it be possible to make the @pam changeable when importing from Proxmox? For example, if there is a sync of LDAP on the Proxmox host then you often have a different realm.

Sorry, I tested it again and it works with other realms


dxfgrabber is deprecated

Hi @Guts
dxfgrabber is deprecated since april 2023. You have to use ezdxf instead.
If I'm not wrong, you use it only to get ACAD Version. I don't know how to retrieve this information with OGR - have to open a ticket -, but you can do it with ezdxf, or directly with this lines ; not robust as ezdxf.
# Mapping of AutoCAD versions to version names
# from https://help.autodesk.com/view/OARX/2024/ENU/?guid=GUID-A85E8E67-27CD-4C59-BE61-4DC9FADBE74A
autocad_versions = {
    "AC1006": "R10",
    #"AC1009": "R11 and R12",
    "AC1009": "R12",
    "AC1012": "R13",
    "AC1014": "R14",
    "AC1015": "AutoCAD 2000",
    "AC1018": "AutoCAD 2004",
    "AC1021": "AutoCAD 2007",
    "AC1024": "AutoCAD 2010",
    "AC1027": "AutoCAD 2013",
    "AC1032": "AutoCAD 2018"
}

def get_acadver_from_dxf(dxf_file):
    try:
        # Open the DXF file
        with open(dxf_file, 'r') as f:
            lines = f.readlines()

        # Search for the line with "$ACADVER"
        # $ACADVER is in the header section
        # Tipically a DXF file, will have this part:
        # $ACADVER
        #  1
        # ACXXX
        # Where $ACADVER is the variable
        #  1, the group code
        # and ACXXX the version id
        for i, line in enumerate(lines):
            if line.strip() == '$ACADVER':
                # Check if the next line exists
                if i + 2 < len(lines):
                    # Extract the version information from the next line
                    acadver = lines[i + 2].strip()
                    return acadver
                else:
                    raise ValueError("Version information not found after '$ACADVER' line.")

        # If '$ACADVER' line not found, raise an error
        raise ValueError("'$ACADVER' line not found in the DXF file.")

    except Exception as e:
        print("Error:", e)
        return None

# Example usage
dxf_file = "/path/to/you/autocad.dxf"
acadver = get_acadver_from_dxf(dxf_file)
if acadver:
    acadvername = autocad_versions.get(acadver, "Unknown")
    print("ACADVER:", acadver)
    print("ACADVER Name:", acadvername)
else:
    print("ACADVER not found or invalid in the DXF file.")

Hello @lbartoletti
Thanks for your interest and taking time to report this.
After a quick look, it seems that ezdxf requires some heavy dependencies which are not useful in DicoGIS: https://ezdxf.mozman.at/docs/setup.html#basic-installation.
I'll dig more into the way to extract DXF metadata  in next days. For now, I'm focused on achieving the version 4.

Hey @lbartoletti
Finally I've decided to remove the version in the output for DXF to simplify the whole code base.


Bug: unable to communicate with switchbot thermometer?

Describe the Bug
When i connected the app to the homebridge it wont update the temp in the homebridge app, keeps registering 0 degrees. error: [SwitchBot] Meter: Thermometer refreshStatus enableCloudService: undefined homebridge
To Reproduce
install the plugin to homebridge then connect a thermostat, mine says:
[SwitchBot] Meter: Thermometer refreshStatus enableCloudService: undefined homebridge
Expected Behavior
Thermometer not working
Relevant Log Output
[SwitchBot] Meter: Thermometer refreshStatus enableCloudService: undefined homebridge

Config for homebridge-switchbot
i dont have this as i uninstalled it as it caused homebridge to keep crashing

Screenshots
No response
Device and Model
Thermometer
Node.js Version
v20.11.0
NPM Version
unsure?
Homebridge Version
v1.7.0
Homebridge Switchbot Plugin Version
V2.14.0
Homebridge Config UI X Plugin Version
v4.55.1
Operating System
Raspian

First of all how did this cause Homebridge to crash? Sounds like there is more in the logs that you aren't sharing.
Second, I would recommend setting up a child bridge for every plugin, so if it does go into a restart on the child bridge restarts.
Third, the logs are clear, you don't have cloud services enabled on your devices. Once you enable that, it will work.


Multiplayer consent screen

Instead of a notification when the user opens multiplayer (which should have gone in OnEntered imo), the first time the user opens the multiplayer screen they will see this message:

The music will also stop to grab their attention. Once they click "I Understand", they won't see the message again.

Thank you!!! This is way better now lol.

Thank you!!! This is way better now lol.


EOS-27356: hare pre-merge CICD job is failing

Problem:
Pre-merge tests are failing due to the script used to prepare dependencies
still points to CentOS 7.8 refs and VM used for pre-merge job is running
on OS CentOS 7.8.
Jenkins Hare-PreMerge-Test
Solution:

Replace current VM with centos 7.9 instead with Specs -
centos 7.9 , 16 CPU, 16GB RAM, 4 disks/50GB.
Updated the dependencies info in prepare-yum-repos file by correcting
the centos 7.9 and also verify/Updated all scripts in Jenkins folder again
as pre-merge job use most of them.

Signed-off-by: pavankrishnat<EMAIL_ADDRESS>

@pavankrishnat
PR description and commit message : Please use 74 characters per line.
@pavankrishnat , @gauravchaudhari02
Can we try running this on our 7.9 VM to confirm if changes work?

@shailesh-vaidya
Suggest me  which one to add for lustre and motr-uploads repos in jenkins/prepare-yum-repos file.
centos-7.9.2009-2.0.0-k8/
centos-7.9.2009-2.0.0-latest/
centos-7.9.2009-custom/

http://cortx-storage.colo.seagate.com/releases/cortx/third-party-deps/centos/

@pavankrishnat, please update the commit message as follows,
EOS-27356: hare pre-merge CICD job is failing

<please describe the problem, why premerge tests were failing?>

Solution:
- Replace current VM with centos 7.9 instead of centos 7.8 with Specs -
16 CPU, 16GB RAM, 4 disks/50GB.

- Updated the dependencies info in prepare-yum-repos file by correcting
the centos 7.9 and also verify/Updated all scripts in Jenkins folder again
as pre-merge job use most of them.

Signed-off-by: pavankrishnat<EMAIL_ADDRESS>

@pavankrishnat, please update the commit message as follows,
EOS-27356: hare pre-merge CICD job is failing

<please describe the problem, why premerge tests were failing?>

Solution:
- Replace current VM with centos 7.9 instead of centos 7.8 with Specs -
16 CPU, 16GB RAM, 4 disks/50GB.

- Updated the dependencies info in prepare-yum-repos file by correcting
the centos 7.9 and also verify/Updated all scripts in Jenkins folder again
as pre-merge job use most of them.

Signed-off-by: pavankrishnat<EMAIL_ADDRESS>

Updated

@pavankrishnat, I see pre-merge tests are failing on this PR as well,
06:58:53  Traceback (most recent call last):
06:58:53    File "./cfgen", line 23, in <module>
06:58:53      from hax.log import create_logger_directory
06:58:53  ModuleNotFoundError: No module named 'hax'
06:58:53  make[1]: *** [test-cfgen] Error 1
06:58:53  make: *** [test-cfgen] Error 2
06:58:53  Failed command cortx-vm-name#6 with status 2:
06:58:53                          cd "cortx-hare"
06:58:53                          export PATH=/opt/seagate/cortx/hare/bin:$PATH
06:58:53                          make check
06:58:53                          make test

@pavankrishnat, I see pre-merge tests are failing on this PR as well,
06:58:53  Traceback (most recent call last):
06:58:53    File "./cfgen", line 23, in <module>
06:58:53      from hax.log import create_logger_directory
06:58:53  ModuleNotFoundError: No module named 'hax'
06:58:53  make[1]: *** [test-cfgen] Error 1
06:58:53  make: *** [test-cfgen] Error 2
06:58:53  Failed command cortx-vm-name#6 with status 2:
06:58:53                          cd "cortx-hare"
06:58:53                          export PATH=/opt/seagate/cortx/hare/bin:$PATH
06:58:53                          make check
06:58:53                          make test

Yes @mssawant those are belongs to unit test. I am working on fixing those EOS-26855

Yes @mssawant those are belongs to unit test. I am working on fixing those EOS-26855

@pavankrishnat ,
As discussed can you please try by commenting those test calls for now?
If it passes then may be we can think of commenting those for now and get done with this PR, so that pre-merge will start working.
Later once we are done with EOS-26855, we will enable those tests again for pre-merge as well.

Yes @mssawant those are belongs to unit test. I am working on fixing those EOS-26855

@pavankrishnat , As discussed can you please try by commenting those test calls for now? If it passes then may be we can think of commenting those for now and get done with this PR, so that pre-merge will start working. Later once we are done with EOS-26855, we will enable those tests again for pre-merge as well.

Yes @suvratjoshi we (Me and @gauravchaudhari02) tried commenting unit test and ran but it failed with different error Hare-PreMerge-Test#1050

@pavankrishnat
Checked your job :
16:57:31  jenkins/test-boot1: line 28: 114022 Segmentation fault      m0crate -S _m0crate-io.yaml
16:57:31  Failed command cortx-vm-name#7 with status 139:
16:57:31                          cd "cortx-hare"
16:57:31                          jenkins/test-boot1

Segmentation fault while doing IO in the test-boot1 file.
It used m0crate command, right?
Currently m0crate is not working, is that correct?
Can we confirm this with motr?

@pavankrishnat Checked your job :
16:57:31  jenkins/test-boot1: line 28: 114022 Segmentation fault      m0crate -S _m0crate-io.yaml
16:57:31  Failed command cortx-vm-name#7 with status 139:
16:57:31                          cd "cortx-hare"
16:57:31                          jenkins/test-boot1

Segmentation fault while doing IO in the test-boot1 file. It uses m0crate command. Currently m0crate is not working, is that correct? Can we confirm this with motr? Lets raise bug. If it is so, then can we use some other working method for IO here?

@suvratjoshi, for m0crate-io-conf issues @supriyachavan4398 has created PR#1956

@suvratjoshi, for m0crate-io-conf issues @supriyachavan4398 has created [PR#1956](https://github.com/Seagate/cortx-
hare/pull/1956)

Hi @pavankrishnat , Till we are done with PR #1956 , can we comment this IO section for testing and test pre-merge job just to ensure that there are no other failures?
Yes, @suvratjoshi
@gauravchaudhari02, I Hope we are good to Merge PR#815. So that we can Test this PR and close it.

Yes, @suvratjoshi
@gauravchaudhari02, I Hope we are good to Merge PR#815. So that we can Test this PR and close it.

Yes, @suvratjoshi @gauravchaudhari02, I Hope we are good to Merge PR#815. So that we can Test this PR and close it.

@pavankrishnat, It is merged now.

@suvratjoshi I ran Premerge job by commenting IO section for testing.
Hare-PreMerge-Test#1098

retest this please

rerun premerge-test

rerun premerge-test

Pre-merge tests are passing now.


.NET5 Compatibility with windows Embedded and IOT

Hello,
I am a developer and my product has currently been converted from .NET Framework 4.8 to .NET5. I haven't been able yet to test it on Windows IoT and Embedded versions and I've been trying to find compatibility information but it seems to me this information is not public yet. Can someone please send me the list of compabible/non-compatible OS versions vs .NET5 please? Where can I find that information?
Some OS I would like to double check compatiblity (below). Even if some of them are not fully compatible, are there any KBs to install to make it compatible? Please advise.
Windows 11 IoT Enterprise
Windows 10 IoT Enterprise
Windows Embedded Pro
Windows Embedded Compact
Windows Embedded 8
Windows Embedded 7

Maybe @NikolaMilosavljevic knows, or if not, can you please point to the right person whom to ask?

@rbhanda , @leecow , is there a .NET 5 support page that would list supported and/or unsupported Windows versions?

This page has information about supported Windows versions: https://docs.microsoft.com/en-us/dotnet/core/install/windows?tabs=net60#supported-releases


Increasing size of symbolic relations in Rosette

Increasing size of symbolic relations in Rosette.
The rosette program that the parser produced only need to provide query and input schemas.

[ ]  add interface in rosette
[ ]  actual incremental solving in rosette

Implemented by @Mestway .


restore of an existing client "fails"

When doing a restore, nodes are overridden.  clients are not.  It warns that the client already exists but does not restore the previous data.
Say, a client is recreated but I want to revert to the backup.  Sure, I can delete the client and then restore but it would easier if the restore deleted it and then recreated it.  Or do an api.put() if the api.post() fails

This is a known limitation documented in the Readme: currently it is not possible to overwritte a client object already available on the target server and these will be skipped.
thanks @nvwls


ol3-popup should now be called ol4-popup :)

Actually, rather drop the number altogether, since OpenLayers now uses SemVer, and version numbers should jump up rather quickly.
I see that it is important to make a difference between the old Openlayers 2 popup on one side, and this plugin on the other side. Is there a better way than to keep the now (and in the future) misleading "ol3-popup" name?

I entirely agree and hope to do something about it soon; same goes for ol3-layerswitcher :smile_cat:

I've renamed to ol-popup to to hopefully avoid renaming again in future and bumped to v2.0.0. I'll do the same for ol3-layerswitcher soon.


Boiler plate for ROS pages

Addes support for ROS pages, based on OCP for now.

Create a recommendations page
Create a recommendations breakdown page

https://issues.redhat.com/browse/COST-3477

Codecov Report

Merging #2908 (d179e70) into main (3616d44) will increase coverage by 0.00%.
The diff coverage is 80.00%.

@@           Coverage Diff            @@
##             main    #2908    +/-   ##
========================================
  Coverage   65.92%   65.93%
========================================
  Files         494      494
  Lines        9441     9443     +2
  Branches     2177     2177
========================================
+ Hits         6224     6226     +2
+ Misses       3214     3047   -167
- Partials        3      170   +167

Impacted Files
Coverage Δ

src/api/ros/ros.ts
100.00% <ø> (ø)

src/locales/messages.ts
100.00% <ø> (ø)

src/routes.tsx
87.87% <50.00%> (ø)

src/api/providers.ts
89.47% <100.00%> (+0.58%)
:arrow_up:

src/api/ros/recommendations.ts
100.00% <100.00%> (ø)

src/api/tags/tag.ts
100.00% <100.00%> (ø)

src/utils/equal.ts
12.50% <0.00%> (ø)

src/utils/cookie.ts
58.33% <0.00%> (ø)

src/utils/format.ts
67.32% <0.00%> (ø)

src/api/ros/rosUtils.ts
37.50% <0.00%> (ø)

... and 43 more

Continue to review full report at Codecov.

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


Support github.dev & vscode.dev web editors

The extension doesn't work in the github.dev web editor: "The 'TLA+' extension is not available in Visual Studio Code for the Web."

Why is an extension not installable in the browser#
There are a small number of extensions that have built-in assumptions or need to run on the desktop. Examples are when an extension accesses files from the VS Code installation on the desktop or when an extension depends on an executable that must run in a desktop environment. When you try to install such an extension in the browser, you will be informed that the extension is not available.
Notice such an extension can still be used when connecting to a Codespace from VS Code running on the desktop.

(from https://code.visualstudio.com/docs/remote/codespaces#_why-is-an-extension-not-installable-in-the-browser) Also, https://code.visualstudio.com/updates/v1_60#_web-extensions has technical background.
It is unlikely that TLC will ever run in the browser, but syntax highlighting should be possible.

A more promising approach would be to use @will62794 https://github.com/will62794/tla-web for the purely browser-based variant of the VSCode extension.


Add support for custom request and notification methods

It is relatively common to extend LSP with custom method types that
are specific to a given language so it makes sense for haskell-lsp to
support this.
To accomplish this, this PR introduces the following changes:

Remove the Misc constructor in favor of a more explicit
CustomClientMethod constructor that does not enforce that methods
start with $/. While the spec mentions that custom methods should
start with $/, I’ve seen custom methods that do not start with $/
and given that haskell-lsp can only control the server side but not
the client side, e.g., VSCode extensions, I think it makes more
sense to be relaxed here.

Add ReqCustomClient and NotCustomClient to FromClientMessage
while removing UnknownFromClientMessage (any valid JSON-RPC
message should now be decodable).

Add handlers for ReqCustomClient and NotCustomClient.

It adds ReqCustomServer, RspCustomServer and NotCustomServer
to FromServerMessage.

Note that this PR does not add RspCustomClient. This is because
responses are always decoded to BareResponseMessage with no
restrictions so it would be redundant. Once BareResponseMessage is
removed in favor of explicit response types, this constructor and a
corresponding handler should be added.

This should close https://github.com/alanz/haskell-lsp/pull/171/

I pushed a minor fix to handle CustomServerMethod in the FromJSON instance. Apart from that, we have been using this since I opened this PR and both server-side and also client-side via lsp-test (I’ll open a PR there once this one gets merged) and it seems to work quite well.

@wz1000 are you happy with this? In which case it can merge.

@alanz yes

Thank you!


Handling multiple optional arguments

Nicely handle optional arguments, so that undefined does not need to be passed e.g.
canvas.fillText('text', 0, 0, undefined, 'black');

// Should be able to be done in the following way
// without the need for a maxWidth

canvas.fillText('text', 0, 0, 'black');

Unsure if this will be possible with all methods

Not considering this since moving to typescript


QR Code Scan not responding in WalletConnectUnity SimpleUnityDapp scene

Hello Team,
I imported the latest assets from 2.0 branch in my Unity project.
While running the SimpleUnityDapp Demo scene in the editor, I noticed that scanning the generated QR code does nothing. I don't receive any notification on my app nor any logs in the editor.
I also received missing scripts warning in the console when starting the scene. Also noticed some missing scripts references on the QR gameobjects.
I'm attaching screenshots of the warnings and QR gameobject.
Please guide me on how to use the QR code scanner in v2.

Hello,
Please try again with the latest version of WalletConnectUnity and make sure your wallet supports the chosen chain. By default SimpleUnityDapp suggest testnets which some mobile wallets may no support.


Question rather than an issue about updating multiple instances

Hi
I have two instances running at the moment.  My question is with the update of the server and mods when they share the same staging directory, I believe that arkmanager has to download the mod before it can check to see if it has been updated.
If the two instances are also sharing the same mods or mostly the same mods does the update process download them twice to check for updates?  Do I need to run an update for each instance separately?

The mods do not currently share a staging directory (apart from the SteamCMD workshop content directory).  SteamCMD should check whether the mod is up-to-date and return fairly quickly if the latest version of the mod has already been downloaded.
If the instances are not sharing the same physical directory, then each instance must be updated separately.  If the instances do share the same physical directory (which would require them to be part of the same cluster), then only one instance needs to be updated, but both instances need to be stopped before the update.

I didn't think I could share the directory?  So I have a separate directory tree for each server including mod dirs and saved dirs.  Basically they are duplicates except for the map and save files

When the -clusterid=clustername option is used, the ARK server likely uses inter-process communication to prevent conflicts.
arkmanager is not currently able to allow multiple server instances to run from the same directory.  You would need to follow something like what ryan1161 wrote in https://github.com/FezVrasta/ark-server-tools/issues/578#issuecomment-244962370; I don't know if all game servers in the cluster need to have the same mods enabled.

v1.6 should now support having multiple instances in a cluster running from the same directory.


Support Tencent Kona JDK

Description:
This PR adds Tencent Kona JDK into setup-java.
It supports the major versions 8, 11 and 17, but doesn't support 21 yet.
The supported platforms are Linux x86_64/aarch64, macOS x86_64/aarch64 and Windows x86_64.
Currently, only the latest stable releases are supported, and only jdk (no jre) binaries are available.
A configuration maintains these releases.
Related issue:
#671
Check list:

[X] Mark if documentation changes are required.
[X] Mark if tests were added or updated to cover the changes.

I just refreshed the commit.
It resolved the file conflicts and supported Tencent Kona JDK 21.

I just updated the commit again, and the file conflicts are resolved.


Improve perf on cloc run

Run a parallelized cloc on all the repositories and the aggregate results for projects.

fixed in f10dce7


Unresolved assembly reference not allowed: System.Core

Every time I try to merge any assembly, I am presented with the following exception:
An exception occurred during merging:
Unresolved assembly reference not allowed: System.Core.
   at System.Compiler.Ir2md.GetAssemblyRefIndex(AssemblyNode assembly)
   at System.Compiler.Ir2md.GetTypeRefIndex(TypeNode type)
   at System.Compiler.Ir2md.WriteTypeDefOrRefEncoded(BinaryWriter target, TypeNode type)
   at System.Compiler.Ir2md.WriteTypeSignature(BinaryWriter target, TypeNode type, Boolean instantiateGenericTypes)
   at System.Compiler.Ir2md.WriteTypeSignature(BinaryWriter target, TypeNode type, Boolean instantiateGenericTypes)
   at System.Compiler.Ir2md.GetBlobIndex(TypeNode type)
   at System.Compiler.Ir2md.GetTypeSpecIndex(TypeNode type)
   at System.Compiler.Ir2md.VisitReferencedType(TypeNode type)
   at System.Compiler.Ir2md.GetMemberRefIndex(Member m)
   at System.Compiler.Ir2md.GetMethodToken(Method m)
   at System.Compiler.Ir2md.VisitConstruct(Construct cons)
   at System.Compiler.Ir2md.VisitAssignmentStatement(AssignmentStatement assignment)
   at System.Compiler.Ir2md.VisitBlock(Block block)
   at System.Compiler.Ir2md.VisitBlock(Block block)
   at System.Compiler.Ir2md.VisitMethodBody(Method method)
   at System.Compiler.Ir2md.VisitMethod(Method method)
   at System.Compiler.Ir2md.VisitClass(Class Class)
   at System.Compiler.Ir2md.VisitModule(Module module)
   at System.Compiler.Ir2md.SetupMetadataWriter(String debugSymbolsLocation)
   at System.Compiler.Ir2md.WritePE(Module module, String debugSymbolsLocation, BinaryWriter writer)
   at System.Compiler.Writer.WritePE(String location, Boolean writeDebugSymbols, Module module, Boolean delaySign, String keyFileName, String keyName)
   at System.Compiler.Writer.WritePE(CompilerParameters compilerParameters, Module module)
   at ILMerging.ILMerge.Merge()
   at ILMerging.ILMerge.Main(String[] args)

The command line arguments I'm using look like this:
ilmerge.exe /out:Result.exe Input1.exe Input2.dll
I have tried multiple combinations of my assemblies and all of them produce this exception.
System.Core is a part of the .NET framework and I'm not specifying to merge this assembly.

It could be that you need to specify the target platform: what version of .NET are Input1.exe and Input2.dll built against? You can see what the default target platform is that ILMerge is using if you add "/log:foo.txt" to the command line arguments.

Thank you, specifying framework v4 allowed me to continue the merge operation 👍 (The log said that It previously defaulted to v2).


Single table

Applying the single table idea in the plots  + some small fixes

I agree. Should we wait until the QL/F team approve the new tables or do you think I should remove it now?

Let's remove now and later we can open a new PR with the new changes

Agreed. Done!

@felipeaoli don't forget to remove the function calls you've removed. Noise and Integ are now broken

sorry. updated here.


Updated ansys-pythonnet requirement

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.45%. Comparing base (2aadf5a) to head (b84787a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #909   +/-   ##
=======================================
  Coverage   82.45%   82.45%
=======================================
  Files         160      160
  Lines       20954    20954
=======================================
  Hits        17278    17278
  Misses       3676     3676

----
🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


Change the sniff pattern

In some cases, we dont have a full WP installation pushed to our repositories, because we dont want the WP core there.
The sniff is especting to find a wp-login.php and like I said before it will fail because that file is not on the repo. But if we change the pattern to look for wp-content/ it should be fine.
Thoughts?

@Baires Great thought. Makes total sense and isn't a major change. @notxarb is the main author of the engine. I'll have him look at the change.

:+1:

@Baires Your suggestion has been implemented into the most recent version of the Engine. We still have to update the distributed build images in order sniff script to work, but you can specify the wordpress engine in your Boxfile.
build:
  engine: wordpress

Once the build images are updated, I'll close this issue.

Awesome!


Fastnlight: Watch notes.101ai.net

Fastnlight requests the watch of the watch_keyword notes\.101ai\.net. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

notes\.101ai\.net has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


add health check tests for load balancer

add tests to check load balancer

Moving to a new collection in health check postman tests


Update table names and descriptions in EML assembly

Assigning this to @sbeaulieu to give me the names/descriptions I need to use

Preliminary table names and descriptions can be updated in next version

So can we close this issue?


[AutoPR] cdn/resource-manager

Created to accumulate context: cdn/resource-manager

(message created by the CI based on PR content)

reopen when ready to release SDK

Hi, I would like to publish the python sdk for CLI development. @fengzhou-msft can you help me fix the build errors? Thanks

#8374 is this the new PR we shall proceed with?

@fengzhou-msft yes, that should be the latest PR.


Bootrom functions

Does the bootrom table lookup code work? It's just that according to Datasheet section 2.8.3, the Bootrom stores the table addresses as 16-bit values - these need to be read as 16-bit values, then expanded to 32-bit, then converted to the appropriate pointer. I suspect the code as written will read 32-bits from the bootrom, and get thus get the pointer very wrong.

Can you refer to the exact line where you see a 32-bit pointer read from the rom?

We use rom_table_lookup which is supplied by the bootrom, so if this function read 16-bit pointer value then we are fine :)
Hopefully it works like that. If not, well no pi pico has ever worked properly :O

const FUNC_TABLE: *const *const u16 = 0x14 as _;

rom_table_lookup(FUNC_TABLE, some_tag);

fn rom_table_lookup<T>(table: *const *const u16, tag: RomFnTableCode) -> T {
    unsafe {
        let rom_table_lookup: RomTableLookupFn<T> = core::mem::transmute(ROM_TABLE_LOOKUP_PTR);
        rom_table_lookup(*table, u16::from_le_bytes(tag) as u32)
    }
}

The deref of the table pointer with `*table`will read four bytes (a `*const _`) from memory: 0x14 .. 0x17. You should only read 0x14 and 0x15 and add two zero bytes. You should probably set FUNC_TABLE to be a *const u16, then do a core::ptr::volatile_read(), then cast the u16 to a usize, then pass that to the rom_table_function.

In your ASM listing, I think `LDR R0, [R0]` will do a 32-bit read from the address 0x14. That `LDR` should be an `LDRH`.

But maybe I'm wrong, and this code works fine as-is? Maybe I've read the datasheet wrong?

Note how the SDK code does a 16-bit read:
// Convert a 16 bit pointer stored at the given rom address into a 32 bit pointer
#define rom_hword_as_ptr(rom_address) (void *)(uintptr_t)(*(uint16_t *)rom_address)

I think it's the same because jumping to 0x0014 is the same as jumping to 0x00000014 but essentially you are right

And in thumb all 16 bit are extended to 32 bit register so yah...

Jumping to 0x0014 is the same as jumping to 0x00000014, but using the ROM table doesn't involve jumping to 0x0014 - it involves reading a two-byte value stored at 0x0014..0x0015.

I didn't notice the double const*😳

I am fixing it right now, why should I use core::ptr::volatile_read and not just regular deref?

I would use it because it is more explicit that you are loading from a specific address in flash.


Support XDG_CONFIG_HOME

Mentioned in #122

@dominiklohmann
Support the XDG base directory spec: Check for the configuration file in the following places (in order):

${XDG_CONFIG_HOME}/yabai/config
${HOME}/.config/yabai/config
${HOME}/.yabairc

On master we look for the config in the following places (in order):

${XDG_CONFIG_HOME}/yabai/yabairc
${HOME}/.config/yabai/yabairc
${HOME}/.yabairc


Handle fully compressed messages

[x] The part of the SMSG_COMPRESSED_UPDATE_OBJECT test that compares the original raw packet to the newly written + compressed packet fails due to a size comparison. If I decompress both packets, they result in the exact same data, but the original vmangos packet is considerably smaller (by about 20 bytes) than the flate2 compressed data, despite having a header indicating that ZLIB level 1 was used.

Looking good, does the message work if you use it instead of SMSG_UPDATE_OBJECT in the example, or how do you know that it's correct?

I hadn't tried that, but I just replaced SMSG_UPDATE_OBJECT with the compressed version in the vanilla server example and it seems to work. Removing that message entirely seems to just cause the client to get stuck at 100% on loading, which I'm assuming is what would happen if the compressed message was wrong.

I hadn't tried that, but I just replaced SMSG_UPDATE_OBJECT with the compressed version in the vanilla server example

That's exactly what I meant. :)

Removing that message entirely seems to just cause the client to get stuck at 100% on loading

Yeah, it's not possible to load in without the initial SMSG_UPDATE_OBJECT.


Added flag for graceful exit of long-running operations.

Looks good, just a minor cosmetic request to use snake_case naming for consoleHandler and signalHandler (i.e console_handler and signal_handler)

Looks good, just a minor cosmetic request to use snake_case naming for consoleHandler and signalHandler (i.e console_handler and signal_handler)

Of course. Sorry about that.

Thank you, merged!


Practice Pagination

Note for @ibnu009
Edit .gitignore
Please, edit your gitignore become this
*.iml
.gradle
/local.properties
/.idea
.DS_Store
/build
/captures
.externalNativeBuild
.cxx
local.properties

Clean Dependencies
- Upgrade version
Example:
com.google.android.material:material:1.2.1 to com.google.android.material:material:1.4.0

Avoid duplicate implementation ( 1 duplicate: paging-rutntime )

Delete androidTestImplementation and testImplementation because this projects is not used.

If you done fixing that, You comments this and I will merge :)
Thank you

Good job.. @ibnu009
But you should be focus your branch.
Next time, you push your branch not master.
If you want to merge master or develop, tag me, please.
Thanks :)


Add MySQL ingester

Coverage increased (+0.1%) to 93.529% when pulling bfb7a14797149fb081e7d8bd0d0d8a4fe22e91b3 on mysql-ingester into 8bfb7e7e846881e0ccb598974a6469213ded59f6 on master.

Coverage increased (+0.4%) to 93.739% when pulling 1b9c80b182fd3e68f1640269013ba0e6a73de3aa on mysql-ingester into 8bfb7e7e846881e0ccb598974a6469213ded59f6 on master.

Coverage increased (+0.6%) to 93.95% when pulling c348dc59a363d6c6fa7dfff2597ece89c2a3272c on mysql-ingester into 8bfb7e7e846881e0ccb598974a6469213ded59f6 on master.

Coverage increased (+0.9%) to 94.288% when pulling efce5a4ebe49944a53bc1ba1c2fd4a30459c6b4f on mysql-ingester into 8bfb7e7e846881e0ccb598974a6469213ded59f6 on master.

Coverage increased (+1.2%) to 94.583% when pulling 0f830f0218072c72c4128a6b5a41093b657ec6ea on mysql-ingester into 8bfb7e7e846881e0ccb598974a6469213ded59f6 on master.

Thank you!


add version bump command and flag for solutions

Description

added solution bump command that increments the patch version
added --bump (-b) flag to solution push and solution validate commands
added more error handling around reading and updating solution manifest files

Type of Change

[ ] Bug Fix
[X] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[X] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[X] I have verified this change is not present in other open pull requests
[X] Functionality is documented
[X] All code style checks pass
[ ] New code contribution is covered by automated tests
[X] All new and existing tests pass

Codecov Report

Merging #60 (ef3385a) into main (1205223) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #60   +/-   ##
=======================================
  Coverage   33.24%   33.24%
=======================================
  Files          27       27
  Lines        3459     3459
=======================================
  Hits         1150     1150
  Misses       2221     2221
  Partials       88       88

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Make bottom icon buttons accessible

Right now, the icons for new notes, syncing, and settings are not accessible at all for screenreader or keyboard users. The icons are just SVGs and SVGs are not interactive elements. These icons should act as buttons since they trigger an action. By putting the icons inside of actual button elements, they become fully keyboard accessible without any need for extra event handlers or tabindexing. The icon is wrapped in a span element in case flex behavior is needed, since there is a bug with buttons acting as flex containers.
Adding an aria-label value to the button also provides an accessible name for people using assistive technology.

I really appreciate your concern for accessibility, and your work improving it!


Fix Dotty doc task

Fixes https://github.com/sbt/zinc/issues/779

To validate this pull request against other sbt modules, please visit this link.

I want to get a milestone out, so self-merging this.


OHM won't keep me logged in / I keep having to relogin

Bug description
I keep having to relogin
I should only have to login once
Repro Steps
Steps to reproduce the behavior:

visit the site
login
surf around
zoom to an area to edit
Click on edt
Be asked to login (wrong! : ) )

Screenshots & URL
Just the login page...

In my case, I cannot log in - but I understand that's a separate issue.

@jeffreyameyer thanks for the ticket, yeah this is a bit weird. We haven't tried to track this one yet but I suspect something to do with locales + cookies. Will post as we inspect. cc @Rub21.
@noelmcloughlin sorry you aren't able to login. The emails should work now and it should allow you to reset password.

Haven't seen this in a while. Closing.

Haven't seen this in a while. Closing.


Dynamic elements

This is a question.
How do I use bootstrap-select on multiple dynamically created elements? It seems to either not show up or won't let me change the value of the select box.

if you would like to populate the options dynamically..something like below should work..
function dynamic(start, end, class){
for(i=start,i<=end;i++){
    $(class).append('<option>+i+</option>');
}
$(c).selectpicker();
}
dynamic(1, 30, '.className');

I want to add selects dynamically. I want to add groups of parameters to a
form and use a select to say whether they should be treated as a string or
raw.
On Nov 28, 2015 4:31 PM, "Henok Hailemariam"<EMAIL_ADDRESS>wrote:

if you would like to populate the options dynamically..something like
below should work..
function dynamic(start, end, class){
for(i=start,i<=end;i++){
$(class).append('+i+');
}
$(c).selectpicker();
}
dynamic(1, 30, '.className');
—
Reply to this email directly or view it on GitHub
https://github.com/silviomoreto/bootstrap-select/issues/1216#issuecomment-160337028
.

After appending the new select, just call $('#newSelect').selectpicker();


ci: Add github action to check tf fmt on PR

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update magnolia to 0.16.0

Updates com.propensive:magnolia from 0.15.0 to 0.16.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.propensive", artifactId = "magnolia" } ]

labels: library-update, semver-minor

Codecov Report

Merging #70 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #70   +/-   ##
=======================================
  Coverage   88.30%   88.30%
=======================================
  Files          10       10
  Lines         248      248
  Branches        7       11    +4
=======================================
  Hits          219      219
  Misses         29       29

Continue to review full report at Codecov.

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


[Feature]: Configure Clock Source and Perform other Preliminary Operations During Node Initialization

Describe your use case
During the initialization phase, the lack of certain pre-operations may increase the cost of manual initialization for users deploying OB.
Describe the solution you'd like
During the node initialization phase, the following configuration operations can be performed:

Describe alternatives you've considered
No response
Additional context
No response

【领取任务】


implemented MixDM model

There's a lot of print statements I left in

Hi Ryan, very cool! I wrote the specific part of pyHalo that deals with the suppression form of the halo mass function quite a long time ago. Unfortunately I wasn't thinking ahead very much back then, and that part of the software is not very modular. Sometime in the next week or two, I will make some changes that allow you to pass in an arbitrary function or class to model the suppression of the mass function relative to CDM. Would that be useful for you?


Fix misspelling of 'Replicated'

I saw this while looking for info on the recent archive moves, and I just couldn't let the name of the company I work for be misspelled 😆 (It's not at all related to Replicated-the-company, but still)

looks like this has been resolved


CRAYSAT-1411: Refactor image dependencies and add name templating

Summary and Scope
Add the ability to use the name of the base image or recipe that was
used to construct an image as a variable when templating the name
property of an image.
Add the ability to use the name of the image as a variable when
templating the name property of a session template.
Since names of session templates and images are now templated and not
required to be unique and cannot be rendered until the base is
validated, change the __str__ methods of both InputSessionTemplate
and BaseInputImage to simply state the index of the item in the input
instance.
Also add general variable substitution (e.g. product versions) to the
conifguration property for images and to the name, image, and
configuration properties of session templates.
Since image names are now templated using variables from the base of the
image, it is necessary to change how images in a sat bootprep input
file refer to other images as a base. Add a ref_name property on the
images which gives them a unique static name that can be referenced by
another image's base.image_ref property. It is no longer supported to
specify a dependency between images using the IMS image name.
Now that we are no longer checking for dependencies between images based
on the images' IMS names, add a check to ensure there is no overlap
between the IMS names of images created by the input file and the IMS
names of images used as bases. This would result in possible race
conditions and non-deterministic behavior if the base of on image is
created by another image in the input file.
Issues and Related PRs

Resolves CRAYSAT-1411

Testing
Tested on:

vale
virtual env on local macbook (for generating and viewing schema docs)

Test description:
Tested on vale with a bootprep input file that builds an image from the
recipe provided by the CSM product, and then customizes it with a second
image specification that refers to the first by its ref_name.
Also tested using the image.name variable substitution in BOS session
template names.
Risks and Mitigations
This does change the schema, but it does so in a backwards-compatible way.
It does also change the behavior of determining dependencies between images. This was not clearly documented before this change anyway, and I don't think users necessarily understood how that would work.
Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable
[x] HPC Product Announcement prepared, if applicable

I just rebased on latest feature/CRAYSAT-1385, which pulled in my commit to fix the name of the hpc-shasta-software-recipe VCS repo and Jack's commit to switch to packaging.version. I'll do a quick sanity test of the built docker image before merge.

Tested against a bootprep file with the following contents, and everything was created as expected:
---
configurations:
- name: "haasken-test-{{ recipe.version }}"
  layers:
  - name: haasken-test
    git:
      url: https://api-gw-service-nmn.local/vcs/cray/haasken-config-management.git
      branch: master
    playbook: test.yml

images:
- name: "haasken-{{ base.name }}"
  ref_name: haasken-barebones
  base:
    product:
      name: csm
      type: recipe

- name: "configured-{{ base.name }}"
  base:
    image_ref: haasken-barebones
  configuration: "haasken-test-{{ recipe.version }}"
  configuration_group_names:
  - Compute

session_templates:
- name: "{{ image.name[:18] }}-{{ recipe.version }}"
  # Note, this has to be updated depending on the system on which this demo is run
  image: ead73bae-d195-4dda-873a-3611bfe4f747
  configuration: "haasken-test-{{ recipe.version }}"
  bos_parameters:
    boot_sets:
      compute:
        node_roles_groups:
        - Compute


ETIMEDOUT after a lot requests

my app is doing a lot http requests. After a while (> 2000 requests) I get ETIMEDOUT on each request. Restarting node fixes this.
My target system is windows, node 4.3.1.

I'm seeing something similar. It's quite difficult to debug. I have a bit of code:
request({
  method: method,
  uri: uri,
  headers: headers
}, function(error, response, body) {
  if (error)
    ...
  else
    ...
});

The callback is not being called with the ETIMEDOUT error. Instead the call itself to request fails.
I'm running on AWS linux.

Hi, I'm faceing same ETIMEDOUT error in a lot of http requests.
In node v0.10.x, keepAlive is default setting at http.Agent. But in v4.x, http.Agent default keepAlive option has changed to false.
I think that this is a cause.
In my case, It has been fixed if I use request#defaults like this.
const request = require('request').defaults({
  forever: true
});

Any news on this? None of the provided alternatives worked for me. It keeps throwing ETIMEDOUT randomly after some requests. Sometimes it's after 300 and some other times after 35k.

@samuela hi, i do not quite understand your mean that " Instead the call itself to request fails.", i found that, the ETIMEDOUT error do not call the callback, i use the "process.on('uncaughtException', callback)" to catch the error.

@RyotaSugawara  your mean is set the keep alive to true?

I am facing the same issue. Suddenly all the requests start timing out ETIMEDOUT.

I am facing the same issue. Request return error ETIMEDOUT but using postman successfully


enable DECLARE_DEPRECATED macro for Developer Studio compiler

The DECLARE_DEPRECATED macro works only for GCC. This change adds support for Solaris/Developer Studio compiler.

Used simple program for testing:

deprecated.c:

#include <stdio.h>

#include "deprecated.h"

int
foo(void)
{
	printf("foo\n");

	return 1;
}

int
main(void)
{
	foo();
}

deprecated.h:

#include "openssl/opensslconf.h"

DECLARE_DEPRECATED(int foo(void))

Tested with GCC on Ubuntu:
vkotal@lenovo:~/openssl-vladak (declare_deprecated_sunpro)$ ./Configure linux-generic64
Configuring OpenSSL version 3.0.0-dev for target linux-generic64
Using os-specific seed configuration
Creating configdata.pm
Creating Makefile

**********************************************************************
***                                                                ***
***   OpenSSL has been successfully configured                     ***
***                                                                ***
***   If you encounter a problem while building, please open an    ***
***   issue on GitHub <https://github.com/openssl/openssl/issues>  ***
***   and include the output from the following command:           ***
***                                                                ***
***       perl configdata.pm --dump                                ***
***                                                                ***
***   (If you are new to OpenSSL, you might want to consult the    ***
***   'Troubleshooting' section in the INSTALL file first)         ***
***                                                                ***
**********************************************************************
vkotal@lenovo:~/openssl-vladak (declare_deprecated_sunpro)$ make include/openssl/opensslconf.h
/usr/bin/perl "-I." -Mconfigdata "util/dofile.pl" \
    "-oMakefile" include/openssl/opensslconf.h.in > include/openssl/opensslconf.h
vkotal@lenovo:~/openssl-vladak (declare_deprecated_sunpro)$ cc -I include/ deprecated.c
deprecated.c: In function ‘main’:
deprecated.c:16:2: warning: ‘foo’ is deprecated [-Wdeprecated-declarations]
  foo();
  ^~~
deprecated.c:6:1: note: declared here
 foo(void)
 ^~~
vkotal@lenovo:~/openssl-vladak (declare_deprecated_sunpro)$ cc --version
cc (Ubuntu 7.4.0-1ubuntu1~18.04.1) 7.4.0
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

and Studio compiler on Solaris:
uls-0:/builds/vkotal/openssl-vladak$ ./Configure solaris64-sparcv9-cc
Configuring OpenSSL version 3.0.0-dev for target solaris64-sparcv9-cc
Using os-specific seed configuration
Creating configdata.pm
Creating Makefile

**********************************************************************
***                                                                ***
***   OpenSSL has been successfully configured                     ***
***                                                                ***
***   If you encounter a problem while building, please open an    ***
***   issue on GitHub <https://github.com/openssl/openssl/issues>  ***
***   and include the output from the following command:           ***
***                                                                ***
***       perl configdata.pm --dump                                ***
***                                                                ***
***   (If you are new to OpenSSL, you might want to consult the    ***
***   'Troubleshooting' section in the INSTALL file first)         ***
***                                                                ***
**********************************************************************
uls-0:/builds/vkotal/openssl-vladak$ gmake include/openssl/opensslconf.h
/usr/perl5/5.22/bin/perl "-I." -Mconfigdata "util/dofile.pl" \
    "-oMakefile" include/openssl/opensslconf.h.in > include/openssl/opensslconf.h
uls-0:/builds/vkotal/openssl-vladak$ /opt/developerstudio12.6/bin/cc -I include/ deprecated.c
"deprecated.c", line 16: warning:  "foo" is deprecated, declared in : "deprecated.c", line 7
uls-0:/builds/vkotal/openssl-vladak$ /opt/developerstudio12.6/bin/cc -V
cc: Studio 12.6 Sun C 5.15 SunOS_sparc 152881-04 2019/03/08

We need to resolve the CLA issue too

What is the status of the CLA here?

I submitted the individual CLA a while ago. For my other PRs there was some issue with the CLA check being case (in)sensitive. As for the corporate CLA, the letter to add me to the corporate CLA list was supposedly sent near the end of July.

We have your individual CLA but you're not on the list for the corporate CLA yet.

Okay, pinged the responsible person.
Dne st 7. 8. 2019 17:21 uživatel mattcaswell<EMAIL_ADDRESS>napsal:

We have your individual CLA but you're not on the list for the corporate
CLA yet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openssl/openssl/pull/9434?email_source=notifications&email_token=AAWMMDG7PNOO7RRNJWJRFS3QDLSAPA5CNFSM4IF2H2IKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3YYMJQ#issuecomment-519144998,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWMMDH5BUHBK4TVB74A2FLQDLSAPANCNFSM4IF2H2IA
.

Close/reopen to kick CLA bot

CLA issues are now resolved

It seems there is non-trivial conflict with master (the DECLARE_DEPRECATED macro has moved to macros.h). Please could you rebase?

I found your commit message a bit confusing, because with "Developer Studio" many developers associate "Microsoft Developer Studio".  So I'd prefer if you'd call it "Oracle Developer Studio" (or "Sun  Developer Studio" or "Solaris  Developer Studio", whichever the official product name is).
enable DECLARE_DEPRECATED macro for Oracle Developer Studio compiler

I found your commit message a bit confusing, because with "Developer Studio" many developers associate "Microsoft Developer Studio". So I'd prefer if you'd call it "Oracle Developer Studio" (or "Sun Developer Studio" or "Solaris Developer Studio", whichever the official product name is).
enable DECLARE_DEPRECATED macro for Oracle Developer Studio compiler

Oh. Seems like every corporation has its own Developer Studio :-) The Sun compiler was renamed so many times it is hard to keep track of the current name. Will change the commit message to hopefully the current one.

Rebased.

@levitte since there have been non-trivial changes please could you reconfirm your approval?

Merged.
a42cb4ba8aa74757b526af2cad2ac09b493df3fb enable DECLARE_DEPRECATED macro for Oracle Developer Studio compiler


fix(init): add option to not write in biocypher if there are no edges…

… or no nodes

Thanks for contributing!


NextJS template missing

Describe the bug
NextJS template not on Github?
Terminal error when trying the docs step by step guide
Found invalid GitHub URL: "https://github.com/lmsqueezy/wedges/templates/next-app-template". Please fix the URL and try again.

Hey,
Thanks for reporting this. The doc has been updated with the correct URL.


Add meta description tag to contact index page

Part of improving search result snipperts shown on external search
engines.
The lack of a meta-description tag makes some search result snippets
unhelpful, confusing, or misleading. Ticket:
https://trello.com/c/S6VUwRWQ/273-follow-up-add-meta-description-tag-to-custom-content-types

Tests failed because of rubygems timeout.  Re-running.


Add Elm programming section

Elm books, including IIRC one of these,  have been on the javascript-frameworks-resources page. We've decided it would be a good idea to re-merge books on that page with the free-books list: see #2688 and #2494. So if you could help this process along, the elm books would be easier to find.

No problem, I'll check back after the re-merge of the javascript-frameworks-resources page.

Much appreciated!


minor: Add more doc to clarify permutation satisfying input order in InputOrderMode

Which issue does this PR close?

Closes #.
Rationale for this change

When investigating an issue about Comet, I'm reading through InputOrderMode.  It takes some time to trace other related code to understand what it means. This patch adds a few more doc to InputOrderMode to make it more clear.
What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Thanks @comphead


General Documentation

Create a doc folder where to put all documentation.
The folder should contains the following documents, among others:

[ ] Architecture
[ ] Tutorial: Write your first Distributed Application with the Fabric Smart Client
[ ] Advanced deployment

merged with #2


[SPARK-18717][SQL] Make code generation for Scala Map work with immutable.Map also

What changes were proposed in this pull request?
Fixes compile errors in generated code when user has case class with a scala.collections.immutable.Map instead of a scala.collections.Map. Since ArrayBasedMapData.toScalaMap returns the immutable version we can make it work with both.
How was this patch tested?
Additional unit tests.

Test build #69696 has started for PR 16161 at commit 699cc75.

Test build #69696 has finished for PR 16161 at commit 699cc75.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69696/
Test FAILed.

cc @cloud-fan

can you highlight your approach to fix this bug? It looks to me that this is not a general fix, e.g. mutable.Map will still fail.

Test build #69741 has started for PR 16161 at commit f325e75.

The approach is to change the deserializer (via ScalaReflection#deserializerFor) to return the more specific type scala.collections.immutable.Map instead of scala.collections.Map as it does now (no change to ArrayBasedMapData#toScalaMap is needed as it already returns an immutable map). This allows the generated code for case classes using either of those Map's to work as they are both assignable from scala.collections.immutable.Map. This PR does not address scala.collections.mutable.Map, it still wont work.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69741/
Test PASSed.

OK, it sounds like what you're doing is unsupported at the moment and this doesn't change that, and you have a workaround. Is this worth merging?

Right now it's not supported to have the following:
case class Foo(a: Map[Int, Int])

(using the scala Predef version of Map)
The documented way to do this is:
case class Foo(a: scala.collections.Map[Int, Int])

and it works fine.
However if someone did not read the documentation carefully and used the former, instead of a reasonable error they get a compile error on the code spark generates. Therefore IMHO there are two options:

Make the generated code work with either version of Map (this PR)
Intercept definitions that use the wrong version of Map and issue a proper error.

If the consensus is that this PR is not worth it then I'll be happy to work on option 2. But in my opinion as a Spark user option 1 is better.

option 1 is better, but this PR only adds the support for scala.collection.immutable.Map, with a very specific way: we can't extend it to support scala.collection.mutable.Map or List.
I think we need a general solution to support custom collection type.

I would be happy to create a seperate PR for adding support for mutable.Map (and List) if that is wanted. But there is no generic solution as there is no type that is assignable to both mutable.Map and immutable.Map.

there is no type that is assignable to both mutable.Map and immutable.Map

Yea, and a general solution would be, remember the original type and create that type when converting catalyst value to external value.
But this may be hard to implement, it may worth to merge this simple PR, as the scala.collection.immutable.Map is so common in scala. cc @rxin @yhuai @liancheng

Test build #70027 has started for PR 16161 at commit e0b3e76.

Test build #70027 has finished for PR 16161 at commit e0b3e76.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70027/
Test PASSed.

thanks, merging to master!

Shall we backport this to branch-2.1? I'd consider this as a bug because of the following snippet fail in Spark 2.1:
case class Wrapper1(value: Option[Map[String, String]])
case class Wrapper2(value: Map[String, String])

val ds1 = Seq.empty[Wrapper1].toDS()  // Fail
val ds2 = Seq.empty[Wrapper2].toDS()  // Succeed

I'm fine to backport this

Thanks. Backported to branch-2.1.


fix: disallow transition into abstract state

This PR actually breaks existing tests and should not be committed

Closing for proper fix per discussion


Triton 2.0

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@h-vetinari i noticed that you are doing some work on llvm-lit. Would you like me to wait for your llvm work to go through?
I wasn't able to get this to compile with newer llvm. Would you like me to show you the logs?

@h-vetinari i noticed that you are doing some work on llvm-lit.

Well, I did tag you in https://github.com/conda-forge/llvmdev-feedstock/pull/210 ;-)

Would you like me to wait for your llvm work to go through?

Not necessary IMO it's just for making things easier with lit. I'm waiting for a comment from Isuru then I can merge, but getting it into the maintenance branches too will take longer than is worth waiting.

I wasn't able to get this to compile with newer llvm. Would you like me to show you the logs?

Not sure what the upstream compatibility is. But if it's an issue of the conda-forge packaging, then we can have a look of course.

Ok let's merge for now and troubleshoot in subscribe PRS


Add method to get user from access token

The idea is that the user can be obtained from an access token for Oauth2 and from access token and secret access key for Oauth 1, this actions are valid mostly for apis when the api client handle the ouath authorization but the api need access the user data.

add OAuth2  getUserByAccessToken method to handle api  oauth
add OAuth1  getUserByAccessTokenAndSecret method to handle api  oauth
add logic to OAuth1 drivers to handle getUserByTokenAndSecret method calls
add OAuth2 drivers logic to handle getUserFromToken method call

Coverage increased (+0.3%) to 55.354% when pulling 06cc8cb09a7c9672c020ef65953dbd7fe0af85a4 on Artessan:develop into 8401da1d91e8c81003488494c0f0e23980702d43 on adonisjs:master.

Also I would like to keep the API consistent for OAuth1 and OAuth2 and remove method. getUserByTokenAndSecret
Next, within getUserByToken method I will perform following checks.
const isOauth1 = this._driverInstance instanceof One === true
if (isOauth1 && !accessSecret) {
	throw GE
		.InvalidArgumentException
		.invalidParameter('Current ally driver uses OAuth1 protocol and hence accessSecret is required as 2nd param to get user profile')
}

if (isOauth1) {
	return this._driverInstance.getUserByToken(accessToken, accessSecret, this._fields)
}
return this._driverInstance.getUserByToken(accessToken, this._fields)

Hey, I made the changes that you requested, I hope the pr be good, let me know what you think.
Thank!

Hey @Artessan! 👋
Sorry for the delay...
Could you please rebase your branch to remove all conflicts?

Also the PR is against the master branch, it should be against the develop branch.
I am going to cherry pick your changes and merge them to the develop branch. So closing this PR


Fixes from review

I think Any will work as intended since string implement IEnumerable interface.

I notice the review from another pr, you are right that the condition is wrong without the parathesis. Thank you!

Is it ok to invite you as collaborator so that I can invite you as reviewer?

Sure thing

I think Any will work as intended since string implement IEnumerable interface.

Yeah, usually i do string isnullorempty check, is there a benefit of using any or length?

Yeah, usually i do string isnullorempty check, is there a benefit of using any or length?

Well, I don't think so. Let's use string.isnullorempty to check

Sure thing

Invitation Sent


How did you get the eol character virtual text?

I'm talking about this

This is done with :h listchars
The end of line character is set like this set listchars=eol:↴

Ok, I have set
set listchars=eol:↴,tab:<->,space:·

And I have nothing like this, but :list shows the correct characters

did you try :set list? I think it defaults to off

It works now, thanks


When you run the tests now, they are way much faster as they don't have to make any external calls.
PS: I also came across a few bugs in the code.

Hey @tenorz007 . Good to see you back 😄
I am not too familiar with the mocks framework. Will have to check this PR out later today/tomorrow. Any good resource to go through?
Also, I see you mentioned you found some bugs. Would be great if you could point them out for the brevity in this thread.
And again, thanks for contributing 👍
Cheers

[https://docs.python.org/2/library/unittest.html](Python 2 Docs)
https://realpython.com/blog/python/testing-third-party-apis-with-mocks/

@tenorz007 Going through them. Would it be ok for you to join our gitter channel. Had some questions which if put on this PR thread would spam the whole watchers list. Here is the link for the channel. https://gitter.im/prodicus/vocabulary Ping me there if you join it
Cheers!

@tenorz007  I was wondering whether we should also add some test(s) which hit the real server.
I mean if the data on their end changes at some point. The parsing would break basically testing our mocked data response to what the response the API server is returning to us,
What are your thoughts on it?

Also, showing test coverage would be a great thing too if possible

Yeah coverage would be a good addition, so you can actually see how much of the code is being tested.


Images timezone defaulted to PDT

Currently, the image timezone defaults to PDT(GMT-8). This might come as unexpected details for some people or in some setups. Given it might be wrong or difficult for tart to set the timezone of the host machine to the VM, it might be worth setting the default image timezone to GMT.

Hey @dniHze, this is the default location/timezone Apple set for macOS. Unfortunately location services are not working from within the virtual machine and therefore it can't automatically pick your timezone.  Created https://github.com/cirruslabs/tart/issues/373 to track the issue.


ci: Add config for automatic release notes

Codecov Report

Merging #15 (f7c51e5) into main (03b3b8d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   94.08%   94.08%
=======================================
  Files           3        3
  Lines         203      203
=======================================
  Hits          191      191
  Misses          7        7
  Partials        5        5

Continue to review full report at Codecov.

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


persist-queue: 0.8.0 -> 0.8.1

This update adds support for cbor2, which is now a test dependency. I've shuffled around the check inputs so that the testing framework is followed by additional testing dependencies.
Description of changes

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

[x] 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 236551 run on x86_64-linux 1

13 packages built:

activitywatch
aw-watcher-afk
aw-watcher-afk.dist
aw-watcher-window
aw-watcher-window.dist
python310Packages.aw-client
python310Packages.aw-client.dist
python310Packages.persist-queue
python310Packages.persist-queue.dist
python311Packages.aw-client
python311Packages.aw-client.dist
python311Packages.persist-queue
python311Packages.persist-queue.dist