run_list presence in role depends on order of .rb input file

Sorry for any info or formatting mistakes, it's the first time I've submitted an issue against your project.
Version:
root@chef-workstation:~/chef-repo/roles# chef -v
Chef Development Kit Version: 0.4.0
Environment: [Details about the environment such as the Operating System, cookbook details, etc...]
root@chef-workstation:~/chef-repo/roles# lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 14.04.1 LTS
Release:        14.04
Codename:       trusty
root@chef-workstation:~/chef-repo/roles# uname -a
Linux chef-workstation 3.13.0-48-generic #80-Ubuntu SMP Thu Mar 12 11:16:15 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
Scenario:
Following directions on a random tutorial I found at https://www.digitalocean.com/community/tutorials/how-to-use-roles-and-environments-in-chef-to-control-server-configurations trying to figure out the basics for Chef.  I'd followed the previous steps in the tutorial and they seem to work fine.
The immediate issue I'm seeing is when creating the role from Ruby DSL.   It looks like the presence or absence of correct run_list: in the uploaded role depends on the order that I specify the inputs in.
When I create a file web_server.rb as
name "web_server"
description "A role to configure our front-line web servers"
default_attributes "nginx" => { "log_location" => "/var/log/nginx.log" }
override_attributes "nginx" => { "gzip" => "on" }
env_run_lists "production" => ["recipe[nginx::config_prod]"], "testing" => ["recipe[nginx::config_test]"], "_default" => []
run_list "recipe[apt]", "recipe[nginx]"
And then run  'knife role from file ./web_server.rb', and then dump it out with 'knife role show web_server', I get the correct and expected output, namely that run_list: is populated with the expected recipes:
root@chef-workstation:~/chef-repo/roles# knife role show  web_server
chef_type:           role
default_attributes:
nginx:
log_location: /var/log/nginx.log
description:         A role to configure our front-line web servers
env_run_lists:
production: recipe[nginx::config_prod]
testing:    recipe[nginx::config_test]
json_class:          Chef::Role
name:                web_server
override_attributes:
nginx:
gzip: on
run_list:
recipe[apt]
recipe[nginx]
However, when I change the ordering on the input file such that run_list is sequentially above env_run_lists, then I get a different output (inputs and outputs as follows):
root@chef-workstation:~/chef-repo/roles# cat web_server.rb
name "web_server"
description "A role to configure our front-line web servers"
default_attributes "nginx" => { "log_location" => "/var/log/nginx.log" }
override_attributes "nginx" => { "gzip" => "on" }
run_list "recipe[apt]", "recipe[nginx]"
env_run_lists "production" => ["recipe[nginx::config_prod]"], "testing" => ["recipe[nginx::config_test]"], "_default" => []
root@chef-workstation:~/chef-repo/roles# knife role from file ./web_server.rb
Updated Role web_server!
root@chef-workstation:~/chef-repo/roles# knife role show  web_server
chef_type:           role
default_attributes:
nginx:
log_location: /var/log/nginx.log
description:         A role to configure our front-line web servers
env_run_lists:
production: recipe[nginx::config_prod]
testing:    recipe[nginx::config_test]
json_class:          Chef::Role
name:                web_server
override_attributes:
nginx:
gzip: on
run_list:
root@chef-workstation:~/chef-repo/roles#
The run_list: is empty.
Steps to Reproduce:
[If you are filing an issue what are the things we need to do in order to repro your problem?]

Create web_server.rb as noted above. Make sure that run_list: is above env_run_lists: in the file.
knife role from file web_server.rb
knife role show web_server
Repeat 1-3, except for 1) make sure that run_list: is below env_run_lists: in the file.
Note output is different from 3).

Expected Result:
[What are you expecting to happen as the consequence of above reproduction steps?]
I'd expect either
a) the order on the input file to not matter or
b) for the knife role from command to produce an error indicating that the ordering on the input file is not correct.
Actual Result:
[What actually happens after the reproduction steps?]
As noted above, the order of the input file results in the same behavior when 'knife role from file' is run.  However, the outputs are different when running 'knife role show'.

You are actually seeing the expected behavior.  As noted in the docs, "_default" is the same as run_list.  So, in the first case, _default was nil, then it was overriden with run_list.  In the second case, it had the run_list value and then was assigned nil by the env_run_lists _default.
https://docs.chef.io/roles.html


Cest?

I created the project yii2/advanced (2.0.6). There are already default demo tests there. But many tests are called, for example, HomeCest. Trifle of course, but still annoying. I suppose this is a typo? Please correct this!

Could you create this issue in https://github.com/yiisoft/yii2-app-advanced?


Feature Request: GUIDs as Primary Keys

(migrated from my original post in Sails)
I prefer to use GUIDs instead of integer primary keys in my apps, and I'd like to be able to do this within Sails. This is definitely lower priority than, say, associations, but I don't believe I'm the only one who uses this, either (see: http://www.codinghorror.com/blog/2007/03/primary-keys-ids-versus-guids.html).

@glenselle @mikebiglan +1 :+1:


Conductor build version for 2.28 generates wrong version number for final artifact

Hi, guys.
Try to do this:

Pull conductor code;
Do a checkout to version 2.28.0;
Inside docker directory, use the docker-compose to build conductor;
Build runs fine, but I see the generated jar has on its name 2.29.0.

Did I miss somerhing here or is this build versioning somehow wrong?
Thank you.

Just a comment: If you build from 2.27.2 for example this generates 2.28.0-SNAPSHOT. My understanding is that conductor build is always targeting X.Y.Z branch as X.Y+1. While it correctly states that this is SNAPSHOT, I believe this should be targeting the same version as the branch to avoid confusion.

Just a comment: If you build from 2.27.2 for example this generates 2.28.0-SNAPSHOT. My understanding is that conductor build is always targeting X.Y.Z branch as X.Y+1. While it correctly states that this is SNAPSHOT, I believe this should be targeting the same version as the branch to avoid confusion.


added log4j-faq text

changed & added some text in the log4j-faq

@Ein-Tim Thanks

It's a bit messy to display the actual search string instead of hiding it in a hyperlink:
https://github.com/corona-warn-app/cwa-server/pulls?q=is%3Apr+log4j
I don't suggest yet another PR for this, but if another one is needed for another reason, then perhaps this could be made prettier?
Also the search string finds all PRs for log4j, not just the ones related to the update of the log4j library to 2.16.0.
https://github.com/corona-warn-app/cwa-server/pulls?q=is%3Apr+log4j+2.16.0+in%3Atitle would filter the search better.


bug fixes and better testing

added unit test with inline data. this avoids file reading which doesn't work cross platform without much effort
made input coordinate list a reference so entire list isn't copied before being moved (which is whole point of move in the first place). However, this requires more work as currently coord list will be empty after Delaunator is constructed and this behaviour is not obvious to the caller.
fixed bug with hashing. important: this changes the way the triangles get indexed relative to JS version. so in order to test i've generated new comparison data by modifying the JS version in the identical way.

putting in new pull request that supersedes this one


DSND-2938: Update full profile model to include DOB in sole trader block

Describe the changes

Updated model includes date_of_birth inside the sole_trader_details block

Related Jira tickets
DSND-2938
Developer check list
General

[ ] Is the PR as small as it can be?
[ ] Has the Jira ticket been updated with any relevant comments?
[ ] Is the README up to date?
[ ] Has the POM been updated for the latest versions of dependencies?
[ ] Has the code been double-checked against main?
[ ] Does the code adhere standards in this repository, including SonarQube checks?

Testing

[ ] Do the code changes have unit and integration tests with code coverage > 80%?
[ ] Has the code been tested locally and/or passed the API Karate tests on Tilt?
[ ] Have mandatory manual test cases been run?
[ ] Are extra Karate tests required?
[ ] Are all the test data resources up to date?

Release preparation
Where possible, add links to the respective Jira tickets when an item is checked

[ ] Have these changes been included in a release ticket?
[ ] Are changes required to the docker-chs-development deployment or test data?
[ ] Are any changes required to the environment added to deployment repositories?

Notes to the tester
Add any testing hints or instructions here.

CI: No security warnings found

CI: No security warnings found


TST: bump rtol for equal_bounds

This PR bumps the test tolerance for examining how different minimiser methods cope with equal bounds.
The test examines the output from a finite differences minimisation vs one that uses a (callable) analytic gradient. In the former factorisation is carried out, in the latter it isn't; this is probably enough to generate differences in the output. Indeed, the arm64 nightly wheels on MacPython were failing because of the specified tolerances:
___________ test_equal_bounds[None-constraints0-<lambda>-kwds0-TNC] ____________
[gw1] linux -- Python 3.10.0 /venv/bin/python
/venv/lib/python3.10/site-packages/scipy/optimize/tests/test_optimize.py:2416: in test_equal_bounds
    assert_allclose(res.x, expected.x, rtol=3e-6)
E   AssertionError:
E   Not equal to tolerance rtol=3e-06, atol=0
E
E   Mismatched elements: 2 / 4 (50%)
E   Max absolute difference: 0.00034846
E   Max relative difference: 0.00039608
E    x: array([ 1.413984,  2.      ,  0.879424, -1.      ])
E    y: array([ 1.413696,  2.      ,  0.879772, -1.      ])
        bound_type = <function setup_test_equal_bounds.<locals>.<lambda> at 0xffff7a514670>
        bounds     = [(0, 3), (2, 2), (-1, 2), (-1, -1)]
        callback   = None
        constraints = (None, None)
        expected   =      fun: 1289.4752548690583
     jac: array([-1.75951145e-05,             nan,  4.24487110e-05,             nan])
 me...  nfev: 19
     nit: 7
  status: 1
 success: True
       x: array([ 1.41369615,  2.        ,  0.87977221, -1.        ])
        fd_needed  = False
        i_eb       = array([False,  True, False,  True])
        kwds       = {'bounds': [(0, 3), (2, 2), (-1, 2), (-1, -1)], 'callback': None, 'constraints': None, 'fun': <function setup_test_equal_bounds.<locals>.func at 0xffff7a5141f0>, ...}
        lb         = array([ 0,  2, -1, -1])
        method     = 'TNC'
        reference_constraints = None
        res        =      fun: 1289.4754142650613
     jac: array([ 0.46138667,         nan, -0.53325948,         nan])
 message: 'Converge... nfev: 108
     nit: 9
  status: 1
 success: True
       x: array([ 1.41398439,  2.        ,  0.87942374, -1.        ])
        test_constraints = None
        ub         = array([ 3,  2,  2, -1])
        x0         = array([0.5488135 , 0.71518937, 0.60276338, 0.54488318])

I therefore relaxed the tolerances to allow the test to pass. This is quite a big bump, from 3e-6 to 5e-4, a couple of orders of magnitude. This might indicate that improvements are needed for TNC.
I therefore decided to add an extra test that would compare the output of two minimisation run that both use finite differences. The first test has two parameters with equal bounds, with two varying parameters (for a total of 4 params). The output of this is then compared to a minimisation that only has the two varying parameters from the first. I would expect the two runs to be pretty much identical.

@andyfaff we still have some failures, see https://app.travis-ci.com/github/MacPython/scipy-wheels/jobs/544115570
/venv/lib/python3.10/site-packages/scipy/optimize/tests/test_optimize.py:2436: in test_equal_bounds
890    assert_allclose(res.fun, fd_res.fun)
891E   AssertionError:
892E   Not equal to tolerance rtol=1e-07, atol=0
893E
894E   Mismatched elements: 1 / 1 (100%)
895E   Max absolute difference: 0.00015936
896E   Max relative difference: 1.23582398e-07
897E    x: array(1289.475255)
898E    y: array(1289.475414)


Foreach doesn't work for HashSet

http://forums.bridge.net/forum/bridge-net-pro/bugs/818
Branch with a unit test.

Additional test for the issue is added as part of #658


v1.0.1 ProyectilMx digitalIO

@ArduinoBot

@misa3L994 please leave the pull request open. As I explained already, the URL change you requested here must be accompanied by a manual operation by the database maintainer. They will do that when they find the time, but if you close the pull request then we won't be able to do it.
So please just be patient and leave the pull request alone!
If you have any questions or concerns, feel free to comment here and I'll provide assistance. Please note that, although this repository does have an automated system for handling library submissions, everything else here is done by humans, not by omniscient robots as some people seem to believe. I am a human, so you must communicate with me as a human.

Sorry about that


change $api_base_uri in order to use Request class with my own server url(tdlib)

🐞 Bug Report
When I use
\Longman\TelegramBot\Request::setCustomBotApiUri(
$api_base_uri          = 'http://myownserver:8081', // Default: https://api.telegram.org
);
and then use  \Longman\TelegramBot\Request::sendAudio it still use https://api.telegram.org as base api url.
but when I change 'private static $api_base_uri = 'http://myownserver:8081';' in request.php manually it works!
Required Information
| Operating system          | Centos 8
| PHP Telegram Bot version  | latest
| PHP version               | 7.4
| MySQL version             | none
| Update Method             | Webhook
| Self-signed certificate   | not sure/use it on my own server
Expected behaviour
When using \Longman\TelegramBot\Request::setCustomBotApiUri(
$api_base_uri          = 'http://myownserver:8081', // Default: https://api.telegram.org
);
for all of the usage of Request:: in that page it should use http://myownserver:8081 as base api telegram url.

As you can see the setCustomBotApiUrl method does exactly the same as setting the variable:
https://github.com/php-telegram-bot/core/blob/72528b72cbc576ebb86652a2541838817c0a0bec/src/Request.php#L324-L330
But the Request::initialize method copies the base_uri from the static property into a Client object.
See https://github.com/php-telegram-bot/core/blob/3cf50e76c3a4809e476b4cd274a3b22783b78882/src/Request.php#L302-L306
This initialize() method gets called in the constructor of the Telegram class...
So you need to call Request::setCustomBotApiUri BEFORE you instantiate a new Telegram class.
\Longman\TelegramBot\Request::setCustomBotApiUri('http://myownserver:8081/');
$bot = new Longman\TelegramBot\Telegram($token, $username);

For the future I made a note and will have a look into that, so we may loose that requirement or streamline it a bit more.

Yes. problem solved.
Thank you.


Some messages without line feed and disable page preview in Telegram Webhook

Gitea version (or commit ref): b9d1fb6
Operating system: Docker
(rest is irrelevant)

Description
First of all thank you @techknowlogick for #4227. I found some issues and suggestion for the message.
As you can see here

when someone comments an issue the bot message doesn't have a new line before the content. Same with comments for a PR. I haven't tested every message to see if this happens elsewhere.
EDIT:
PR #6719 fixes the page preview of the messages (description below) but the line feed in some messages (description above) is still missing.

Also in Telegram, by default a message sent that includes a link will put a preview of the web, a little header and description. I think for the main purpose users want this feature (notifications of events) it is just noise. You can see what I mean here:

The method sendMessage in telegram bot API includes a parameter disable_web_page_preview that could be set to false. I think this should be false by default, maybe with an option to enable it.

@InExtremaRes I've just sent a PR to disable previews, as you are right it'll be the same everytime and will just take up unnecessary space.
Please check if #6683 works for you.

Should be fixed by #6719 if not please reopen

@lafriks In fact, as you can see in the comments of #6683, #6719 only fixes the page preview but no the line feeds missing. I can reopen and edit the issue to reflect that fact, or should I open a new issue instead?

Please update description what still needs to be done

I edited de issue but I am not  sure it is enough. Feel free to edit the title or the content to help to track the issue better.

Does this issue include the issue and code previews being a lot more unhelpful than at GitHub?

I am able to reproduce my previous comment of which beimng this issue was never confirmed?

I am still able to reproduce, however the Gitea instance is outdated which cannot be helped at the moment and I think next time I will just open a new issue as my previous comment leave it unclear if this is even my issue.


Web UI Shows Incorrect Path with Nginx Reverse Proxy

Using the following nginx config.
server {
listen 80;
server_name example.com/gogs www.example.com/gogs;
location /gogs/ {
proxy_pass http://<IP_ADDRESS>:3000/;
include proxy_params;
}
}
The /gogs will show in the web ui for connecting to the git repo over ssh
Example:
git remote add origin git@example.com/gogs:atlas/ansible-atlas.git
However, the the correct path should be:
git remote add origin git@example.com:atlas/ansible-atlas.git
Without the /gogs.
I describe issue that arises in more detail at the bottom of this issue:
https://github.com/gogits/gogs/issues/458#issuecomment-68127871

I dived into the code, and everything seems to be correct already!
Probably you have a mistake in the configuration. DOMAIN should not contain the subpath, ROOT_URL should. E.g.:
DOMAIN = myserver.com
ROOT_URL=http://myserver.com:3000/gogs/

I dived into the code, and everything seems to be correct already!
Probably you have a mistake in the configuration. DOMAIN should not contain the subpath, ROOT_URL should. E.g.:
DOMAIN = myserver.com
ROOT_URL=http://myserver.com:3000/gogs/

Hi @jnakatsui , I think @phsmit gives the same answer as I will, do you have any progress on the issue?

That was it sorry for the mistaken configuration.

@jnakatsui great! So I close this issue right now unless you want to continue discuss on it. :smile:


2.3.0

[ ] closes #xxxx (Replace xxxx with the GitHub issue number)
[ ] Tests added and passed if fixing a bug or adding a new feature
[ ] All code checks passed.
[ ] Added type annotations to new arguments/methods/functions.
[ ] Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.

Changes needing manual review:

Silencing of downcasting warning
Adjustment of replace test


Check if there are data before accessing

In the function node_list_to_coordinate_lines(G, node_list, use_geom=True) of file plot.py the following excerpt raises an error when there is no edge data (i.e., G.get_edge_data(u, v) is None):
data = min(G.get_edge_data(u, v).values(), key=lambda x: x['length'])
In my application, doing
 try:
     data = min(G.get_edge_data(u, v).values(), key=lambda x: x['length'])
 except:
     data = {}

solved the problem.

Thanks. Can you provide a reproducible standalone code snippet to generate the error?

For reference, this is regarding the internal _node_list_to_coordinate_lines function in the plot module: https://github.com/gboeing/osmnx/blob/14fb821be83ca60ecf9698cd465926f294a7bb1e/osmnx/plot.py#L515
The function takes an argument representing a list of node IDs in a route, per the docstring: "a valid graph path as a list of node IDs." If the node list is not a valid graph path, the function should throw an error.


Add CI check to validate that go mods and vendor dir are in sync

Add CI check to validate that go mods and vendor dir are in sync
+ fix existing inconsistency in go.mod file

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 30.226%

Files with Coverage Reduction
New Missed Lines
%

api/v1/helper.go
3
43.73%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
3554

Relevant Lines:
11758

💛  - Coveralls


Fix broken tests and lock files

Hi. This PR fixes a couple of problems:

Re-generated custom-payment-flow/client/react-cra/package-lock.json. The old one was causing build errors like this: https://github.com/stripe-samples/accept-a-payment/runs/6413530545?check_suite_focus=true#step:7:177
A test in spec/prebuilt_checkout_page_e2e_spec.rb was failing due to link text changes.

Nice. Thank you @hibariya !!


The InkWell component doesn't act exactly like material design ink wells should

From @Hixie on May 19, 2015 23:4
Specifically:

the whole paper should change colour while the touch is down
the splash should go slow while the touch is down, then speed up when it goes up

Copied from original issue: domokit/mojo#172

From @abarth on June 11, 2015 19:40
The velocity part of this bug is done.  What's still missing is changing the color of the whole paper.

This is fixed by https://github.com/flutter/engine/pull/1813 .


Calls to routers and storages when crud-router and crud-storage roles initialization is not finished yet

There are exceptions trown in cartridge-java driver due to crud-router and crud-storage roles initialization issues.
How to repoduce:

run some load test to call crud.select (with prefer_replica=true and balance=true) and crud.upsert
stop/start a few routers
stop/start a few replicas (not leaders) in replicasets

Result for select:
org.springframework.dao.DataRetrievalFailureException: InnerErrorMessage:
code: 33
message: Procedure '_crud.select_on_storage' is not defined; nested exception is io.tarantool.driver.exceptions.TarantoolInternalException: InnerErrorMessage:
code: 33
message: Procedure '_crud.select_on_storage' is not defined

Result for upsert:
Caused by: io.tarantool.driver.exceptions.TarantoolInternalException: InnerErrorMessage:
code: 32
message: ...example.lua:100: variable 'crud' is not declared

SDK version: Tarantool Enterprise 2.7.3-0-gdddf926c3-r428
Cartridge version: 2.7.2
CRUD version: 0.9.0

The problem with storages should be generally resolved after https://github.com/tarantool/vshard/issues/298, however we can implement some simple retrying strategy in crud even without this feature.
The problem with routers should be resolved by https://github.com/tarantool/cartridge-java/issues/159 and, sadly, there is nothing that we can fix on the crud side.

Same problem solved in vshard https://github.com/tarantool/vshard/issues/123


Noticed a ConcurrentModificationException in AbstractRequestHandler

Using Wiremock on my project I noticed a ConcurrentModificationException  on the List object in the first line of the AbstractRequestHandler class.   The modifcation occurs in the for-each loop you have in the handle method.    I suspect it would help if you change it from a List to a CopyOnWriteArrayList ?
If you write a test around it, I bet you'll discover it, as I did.

Please can you post a test case that replicates the problem?


Can't run DirectLine sample

Hello,
I have a bot registered on bot framework. I have it's bot-id, it's Microsoft app id, and its  Microsoft secret password. I have also configured DirectLine as a channel on it.
I have downloaded the core-DirectLine sample, and I did the following changes before running the sample:

To the Web Api Project (Bot's logic), I modified the web.config values:

 <add key="BotId" value="MY_BOT_ID_HERE" />
 <add key="MicrosoftAppId" value="MY_APP_ID_HERE" />
 <add key="MicrosoftAppPassword" value="MY_PASS_HERE" />

To the Console Application (DirectLine client), I modified the app.config values:

<add key="DirectLineSecret" value="DIRECT_LINE_SECRET_HERE"/>
<add key="BotId" value="MY_BOT_ID_HERE" />

One thing to mention is that when I configured DirectLine, it gives me two secret keys but I think is the same using one or another.
I have the Bot app deployed on Azure, and I run the console app locally.
What happens:
Whenever I send a message from Skype (bot user), the client doesn't receive anything. Similarly, when I write something on the DirectLineClientApp, the bot user doesn't receive anything via the bot.
This is what I get when I send something via direct line client app:

But nothing on Skype bot. And the same vice versa.
Any help?
Thanks.

@dandriscoll @willportnoy
Maybe you know the answer to this one. Thanks!

@piffarettig Let's leave Azure out of the picture for now and try the simplest path first.
What happen if you run both the client and the bot locally and use ngrok to tunnel your bot (updating the Bot URL in the Bot Framework portal to match the one provided by ngrok)? I'm running that scenario right now and it's working for me

The same. First I was running it on ngrok as it easier to debug but I tried uploading it to Azure because maybe something related to the connection was happening.
I think that maybe I got wrong the idea of the demo, it works, but I first thought that the DirectLine Client App was  a client to send messages to the user behind the bot, not the bot itself. How could adapt this to make it send messages to the user (via the Console App)?
Thanks

From the image above, it seems the bot received your message. You can try sending "show me a hero card".  The console app is just showing how you can create a simple client using DirectLine... users write in the console and get the response from the Bot in the console.

I want to send a message to some user behind Skype Channel.
And what you say is correct. I got wrong the idea of the demo. I thought the idea of this was to achieve an interaction like:
Human -> Bot -> Human
Not like:
[ https://github.com/Microsoft/BotBuilder/issues/2011](Like this use case)
How can redirect the message to the user?

I know this is different from the original post, but maybe you could give me a hand.
So I am in this scenario:
Some users use a bot via Skype. These users asks questions to the bot. Whenever the bot is not able to answer any of those questions, a conversation between the user and a customer representative/expert starts.
Just for testing if this works, what I did was:

Created a simple bot and have it running over ngrok.
Created a simple Console App and have it running locally. This apps connects to the previous bot via DirectLine.
Whenever a user sends a message via Skype, I want to broadcast it to the ConsoleApp, via DirectLine.

Is this possible?
What I did for testing was getting the ConversationID of the ConsoleApp interaction with the bot, and tried this code snippet:
var userAccount = new ChannelAccount("ID_OF_THE_CONVERSATION_BETWEEN_CSI_AND_BOT");
var botAccount = context.Activity.Recipient;
var connector = new ConnectorClient(new Uri(context.Activity.ServiceUrl));
var conversationId = await connector.Conversations.CreateDirectConversationAsync(botAccount, userAccount); //botAccount now is the sender, userAccount (CS) is now the recipent.
IMessageActivity messageToSend = Activity.CreateMessageActivity();
messageToSend.From = botAccount;
messageToSend.Recipient = userAccount;
messageToSend.Conversation = new ConversationAccount(id: conversationId.Id);
messageToSend.Text = message.Text;
messageToSend.Locale = "en-Us";
await connector.Conversations.SendToConversationAsync((Activity)message);
But the DirectLine Console Application is not receiving the message. Is it possible that the ConversationId (ID_OF_THE_CONVERSATION_BETWEEN_CSI_AND_BOT) changes?
Thanks

Hi @piffarettig - If you still have questions, please post a new issue on BotBuilder repo.


cmd/pprof: module version suffix shown in graphs instead of package name

What version of Go are you using (go version)?

$ go version
go version go1.14 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/jake/.cache/go-build"
GOENV="/home/jake/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/jake/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/jake/zikaeroh/hortbot/hortbot/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build332050882=/tmp/go-build -gno-record-gcc-switches"
GOROOT/bin/go version: go version go1.14 linux/amd64
GOROOT/bin/go tool compile -V: compile version go1.14
uname -sr: Linux 5.5.7-zen1-1-zen
/usr/lib/libc.so.6: GNU C Library (GNU libc) stable release version 2.31.
gdb --version: GNU gdb (GDB) 9.1

What did you do?
Created a profile, and opened it with go tool pprof to view a graph of memory allocations.
What did you expect to see?
Nodes with package names.
What did you see instead?
Packages which are at the root of a non-v1 module show their version suffix rather than their actual package name, making it more difficult to determine what they are.
For example, jackc's pgx/pgconn, which are at v4 and v2 respectively:

It looks like pprof just takes the last element of the import path, which used to be somewhat accurate (ignoring path/package name differences), but is less so now.

I'm unsure if this should be here or github.com/google/pprof (or maybe both). I don't think that there's enough info in the profiles for pprof to actually know it should remove a version suffix. I'll defer that to someone who knows better where the change would need to be made.

CC @hyangah

github.com/google/pprof is a better place to discuss visualization and presenting Go's package name. I wished I could transfer this issue but couldn't. @zikaeroh can you open this in the upstream repo?
fyi @aalexand @nolanmar511

Sure, I'll do that.

Filed #515. I'll close this one.

Meant google/pprof#515, of course. Forgot the repo name. :slightly_smiling_face:


add ruby 2.6.0-rc1

@andrewkress Can you rebase to remove that fix url commit?
And perhaps use a slightly more descriptive commit message?

Add definition for Ruby 2.6.0 RC1

Thanks!

@andrewkress Can you rebase to remove that fix url commit?
And perhaps use a slightly more descriptive commit message?

Add definition for Ruby 2.6.0 RC1

Thanks!

I think this update should do the trick

@hsbt This looks ready to merge.


Can no longer specify multiple Gradle tasks to run

Prior to v1.5, it was possible to specify multiple tasks to run in the gradle_task parameter, e.g. setting gradle_task to assemble publishToMavenLocal. This would produce the following command:
/bitrise/src/gradlew --build-file /bitrise/src/build.gradle assemble publishToMavenLocal --stacktrace

With the update to v1.5, the parameters get wrapped in quotes:
/bitrise/src/gradlew "--build-file" "/bitrise/src/build.gradle" "assemble publishToMavenLocal" "--stacktrace"

... which now produces the following error because of the quotes:
* Exception is:
org.gradle.execution.TaskSelectionException: Task 'assemble publishToMavenLocal' not found in root project 'sdk'.

This may be related to the change to address this issue.
Is this new behaviour expected?

Ouch, you're right, this is a new issue with the latest step - we'll fix it ASAP and push a new version. In the meantime please downgrade to the previous version of the step, we'll let you know as soon as the fix is published!

@viktorbenei will do. thanks for the quick response!

Thanks for reporting the issue! ;)

@julo15 ,
thanks for the report, new version v 1.5.1 with the fix is available in the steplib.

just tried it out and it works :)
for reference (to anyone curious), new in v1.5.1 vs. v1.4.1 is each parameter is wrapped in quotes:
/bitrise/src/gradlew "--build-file" "/bitrise/src/build.gradle" "assemble" "publishToMavenLocal" "--stacktrace"

thanks so much for the fast response.

You're right @julo15 and thanks for the info for others!
The change was actually that the step was rewritten from Bash to Go, using the new Bitrise CLI v1.4.0 built in Go Toolkit ( http://blog.bitrise.io/2016/09/13/monthly-release-of-bitrise-cli-sept.html ). It was rewritten mainly to have precise control over what/which input gets escaped and what not, as that's way harder to control in Bash


Fix cluster name in cluster yaml

Now cluster creation works w/o providing master name.

This change is


Add docformatter to format check

Adds docformatter to format check.
Docformatter does some basic linting for docstrings according to[PEP 257(https://peps.python.org/pep-0257/).
This addresses #4968 by setting a standard line width for the docstrings.

@MichaelBroughton to decide if we want this or not.

It doesn't seem like there is much stomach for this so closing.


Add Pod::generate_kube_yaml

What did you implement:
Pod::generate_kube_yaml that was missing.
How did you verify your change:
I implemented Kubernetes yaml generation in https://github.com/marhkb/pods/pull/412

LGTM, Thank you!


Delete outdated readme in hub/bin

Signed-off-by: Kaiyuan Hu<EMAIL_ADDRESS>

Codecov Report

Merging #753 (dcecdfa) into main (957a6da) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #753   +/-   ##
=======================================
  Coverage   71.04%   71.04%
=======================================
  Files         167      167
  Lines        7613     7613
  Branches     1251     1251
=======================================
  Hits         5409     5409
  Misses       1831     1831
  Partials      373      373

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 957a6da...dcecdfa. Read the comment docs.

/lgtm
/approve


Show login error message if credentials are wrong

Pull request contains a fix addressing the issue that currently no error message is displayed if the user tries to login with wrong credentials. #4
This will handle the server response:
{"errors":{"email or password":["is invalid"]}}

and render:

Nice, thanks @andys8!


Porting sacrebleu to linux package manager(s)

Hello, thank you very much for developing this package, it is super useful!
I noticed that sacrebleu has 46 releases currently and probably has a fast development rate. I was wondering if you might be interested in porting sacrebleu into user-based linux package repositories (on top of pip). Compared to installing/updating separately with pip, this would make keeping everything up-to-date much easier by simply using your standard package manager.
If this is of interest, I would be glad to port sacrebleu into the Arch Linux User Repository (AUR) here. I can also make a PR to add some scripts that will keep this updated every time there is a new release on your repository.

I don't know what this would involve, but I'd be happy to review a PR.

Awesome. I would have to create a package for sacrebleu on the AUR platform. I will write a (shell) script that can be executed every time there is a new release of this package, and this script will essentially push the update to the AUR (just like a sync).
As I am a bit occupied at the moment, I would only be able to do this later on. Will keep you posted on any updates.

Feel free to send the relevant PR when ready, closing this issue. Thanks!


Update the game over message

Update the game over message so people know how to play again!

do step 5


Possibility to pass many parameters to cryptsetup

Now line 147 of hook have quotes to each parameter:
_tmp="$(printf %s "$_ykfde_passphrase" | cryptsetup luksOpen "$YKFDE_LUKS_DEV" "$YKFDE_LUKS_NAME" "$YKFDE_LUKS_OPTIONS" 2>&1)"
So it can handle only one of them. If you have rare setup which need to pass more than one parameter, you'll not able to boot.
To fix this need just remove quotes:
_tmp="$(printf %s "$_ykfde_passphrase" | cryptsetup luksOpen "$YKFDE_LUKS_DEV" "$YKFDE_LUKS_NAME" $YKFDE_LUKS_OPTIONS 2>&1)"

Passing unquoted variable is something we want to avoid. Were the additional parameters about detached header?
It's also possible to store LUKS options in header

@Vincent43 for example, in my configuration I'm using
--header=/header-luks.img --allow-discards

Yeah so we had proposal for adding headers option before.
The cryptsetup config --header=/header-luks.img <device> also should work


Check itemDoms to prevent crash during initialization on Obsidian 1.7.4

This is probably not the right fix, but it prevents crashing during initialization on Obsidian 1.7.4.

See #596

Thank you for the pull request! But I think the issues should have been resolved in the newest version.

Thanks, yes all fixed in the latest version.


Update dependencies

Update all out of date dependencies.

:tada: This PR is included in version 0.9.3 :tada:
The release is available on:

GitHub release
Atom package

Your semantic-release bot :package::rocket:


Stub and minor documentation

Added stub of showroute() and added comments.

hmm so you got the shark badge here huh


Extend html-noplot for tutorials

✨ Feature Request
Extend the concept of streamlining documentations building with the make html-noplot command to include skipping rendering images within tutorial .ipynb notebooks or building the notebooks with myst-nb.

Closed by #1146


Multi Line Header in Grid

how to have multi line header in a grid and a merging of two or more columns in first line of header.

Maybe you can do the same idea that explained in Grouping and Summaries in Grid located in Basic Samples > Grids

found Implemented in http://github.com/mleibman/SlickGrid/commit/19624f6687a0d2671639c4b1f2c05eb20c9ab0c7
but converting it to our sample is a major task.
can someone please guide on how to achieve this.

I am thinking of switching to datatables in place of slick grid but implementation of grouping is not good in datatables what is your view on this.

Sorry that feature is not supported yet.


[SPARK-14424][BUILD][DOCS] Update the build docs to switch from assembly to package and add a no…

What changes were proposed in this pull request?
Change our build docs & shell scripts to that developers are aware of the change from "assembly" to "package"
How was this patch tested?
Manually ran ./bin/spark-shell after ./build/sbt assembly and verified error message printed, ran new suggested build target and verified ./bin/spark-shell runs after this.

Test build #55088 has started for PR 12197 at commit cb050a0.

Test build #55088 has finished for PR 12197 at commit cb050a0.

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

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

Merged build finished. Test PASSed.

Test build #55128 has started for PR 12197 at commit 7ace69c.

Test build #55128 has finished for PR 12197 at commit 7ace69c.

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

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

Merged build finished. Test PASSed.

LGTM merging into master.


Add SSL support

You meant a mutual TLS authn between the client and the server, right?

Yup, that is currently not supported


Scroll to top button function visible

Description
Hey!
I found a bug in the project. When you scroll to bottom of any page there is a function visible i.e scroll to top. Please view the screenshot attached
Steps to Reproduce
I can fix the issue by checking if scripts has problem or the function
Expected Behavior
Only scroll to top button should be visible
Actual Behavior
Scroll to top function is visible with button
Screenshots

Additional Information
No response
What browser are you seeing the problem on?

[ ] Firefox
[X] Chrome
[ ] Safari
[ ] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@rishabhrawat05 assigning this issue to you,
Make sure to complete it within 2 days.


[Event Request] Codeunit 5750 Whse.-Create Source Document - OnBeforeSetQtysOnRcptLine

Hi, in the Codeunit 5750 Whse.-Create Source Document, would it be possible to get an event OnBeforeSetQtysOnRcptLine, like this :

      [IntegrationEvent(false, false)]     local procedure OnBeforeSetQtysOnRcptLine(var WarehouseReceiptLine: Record "Warehouse Receipt Line"; var Qty: Decimal)     begin     end;
Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


UnpicklingErro

Thank you the authors for sharing the great work.
I got some issues when I ran the following lines based the instructions
checker = BertChecker()
checker.from_pretrained()
The issue is UnpicklingError: invalid load key, '<'..
Were model weights updated that cause the issue?

Thank you the authors for sharing the great work.
I got some issues when I ran the following lines based the instructions
checker = BertChecker() checker.from_pretrained()
The issue is UnpicklingError: invalid load key, '<'..
Were model weights updated that cause the issue?

HI i am also facing the same issue

Same issue


merge package from arlac77/npm-package-template

.travis.yml

chore(travis): remove node versions 10.12
chore(travis): add node versions 10.13

Coverage remained the same at 90.909% when pulling ce8e482453033bee1a0d2071590e6351304f8b15 on template-sync-1 into 8e17bed0c5b4db7b84f83fc62c5b9d233deb4a30 on master.

:tada: This PR is included in version 2.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Skip some imports in standalone build, found PYTHONPATH not working

Actually, this is a question.
I am working on a project, which uses Numpy, Pandas and etc.
To save the compilation time, when building standalone binary, I used some --nofollow-import-to to skip those huge python module. The standalone binary can be generated successfully. But I found I couldn't use PYTHONPATH to help the binary find the missing python modules. Now I can copy those python modules to where the standalone binary is.
Is there any other way to set PYTHONPATH for the standalone binary?
Thanks in advance!
Regards,
Yongqiang

Nuitka version, full Python version and Platform (Windows, OSX, Linux ...)
python -m nuitka --version

$ python -m nuitka --version <IP_ADDRESS> Python: 3.7.9 (default, Oct 13 2020, 21:10:49)  Executable: /usr/local/bin/python OS: Linux Arch: x86_64

How did you install Nuitka and Python

Use Pip

No, you cannot do that. We are probably going to add options to include some modules as bytecode, and we are going to add caching to address your compile times issue. In standalone mode, I expect to make hard coded ImportError a thing for modules not found somewhere down the line.
For the standalone program however, I think that it may accept to load from the dist folder root, but I am not sure of that, but I think I saw one plugin of Nuitka that exploited it. So currently you might be able to work around that. But beware, that's not a feature actually supported, esp. should it not be true. :)
And mixing compiled and uncompiled code inside of packages is not yet working well at all. Standalone is normally not affected at all from that, but with that hack it would be.

Try add PYTHONPATH to sys.path before load your modules.
import sys
import os

exe_dir = sys.path[1]
lib_dir = os.path.join(exe_dir, 'lib')
dll_dir = os.path.join(exe_dir, 'dll')
sys.path.append(lib_dir)


Robust pagination params

This change fixes the following issue:
Setup: Visit https://til.hashrocket.com/?page=a
Expected
Pagination parameter is ignored
Actual
500
This implementation returns the first page when a pagination parameter that cannot be cast to an integer (i.e. "a") is passed, the same thing we do when no pagination parameter is passed.

Looks good 💪


Floats behave slightly funny on OD

Byond seems to not have rounding issues unlike OD

test code
var/a = 0
for(var/i in 1 to 21)
    world.log << a
    a += 0.05

var/a = 0
for(var/i in 1 to 10001)
    a += 0.0001
for(var/i in 1 to 10001)
    a -= 0.0001
world.log << a *<PHONE_NUMBER>

Results in -50.3933 on byond & -50.393284 on OD
So it seems like the string conversion truncates the floating point numbers

OD

Byond

AFAIK the issue is specifically in how we display floats. BYOND (thank goodness) actually does do correct IEEE arithmetic for these floats, they are just displayed in a truncated way sometimes.

Yes the back end their the same but visually its not done right
afaik the algo for it is if its less than 0.0001 round to 4 decimals otherwise IEEE


[webview_flutter] Proof of concept web implementation

Initial proof of concept implementation of flutter_webview for the web.
This is https://github.com/flutter/plugins/pull/4565 with the following changes:

Fixes unit test compilation on master
Adds very minimal integration tests

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

@stuartmorgan Merged mean now master branch will have webview for web?

@stuartmorgan As I can see, WebViewPlatformController is not supported, I wonder if it will be in the next release?
https://github.com/flutter/plugins/blob/master/packages/webview_flutter/webview_flutter_web/lib/webview_flutter_web.dart

@stuartmorgan As I can see, WebViewPlatformController is not supported

I don't know what that means, given that you provided a link to a file that implements it. But if you have bugs or feature requests for the package, please use the issue tracker as always.

@stuartmorgan I mean most of functions  throw UnimplementedError();
Ok I will use issue trackers in the future


Graceful API WSGI server shutdown

This pull request implements graceful API WSGI server shutdown. On shutdown (or CTRL-C / SIGINT signal), we stop receiving new requests and give existing active requests some time to shut down (2 seconds by default) before we kill requests which still didn't finish.
This is needed since we support long-running requests (stream endpoint). Previously, if there was an active long-running request, it would block API server from gracefully shutting down and we would need to send SIGTERM to the server to kill it.
This is also important and will be utilized when we support requests draining and zero-impact and zero-downtime (rolling) upgrades / restarts.
Note: This is something @enykeev were passively trying to figure out in the past.

Nice.


Fourier.Forward throws CUDA exception

Upgrading to version 3.9, a call to Fourier.Forward like so:
Fourier.Forward(B_E_SS, FourierOptions.Matlab);

(B_E_SS is a Complex[]) throws Exception:
A first chance exception of type 'System.NotSupportedException' occurred in MathNet.Numerics.dll

Additional information: Cuda Native Provider not found.

This was not the case with version 3.8, for the same code.
[EDIT]
If I click Continue on the popup for the Exception, I get another one of the same type, this time for
MKL Native Provider

Then if I click Continue again, I get the same Exception again, for:
 OpenBLAS Native Provider

If I click Continue one final time the program starts running and does not bother me anymore with exceptions.
Note that those exceptions are all raised even if I install the NuGet Packages for MKL.
Again, this did not happen to me with the same program, using MathNet.Numerics 3.8

Do you just see them being thrown (e.g. if break on all exceptions is enabled when debugging) within Math.NET internal code, or do they reach your code? If they do, could you provide a call stack? Thanks.

I see them being thrown the first time I make a call to Fourier.Forward. The IDE is set to break on all exceptions, so that is probably the reason why I see them at all. And indeed, if I run without debugger they do not pop up.
Anyway, Call stack (up until right before my function) (I am not sure how to format it properly):
MathNet.Numerics.dll!MathNet.Numerics.Providers.LinearAlgebra.OpenBlas.OpenBlasLinearAlgebraProvider.InitializeVerify()	Unknown
MathNet.Numerics.dll!MathNet.Numerics.Control.LinearAlgebraProvider.set(MathNet.Numerics.Providers.LinearAlgebra.ILinearAlgebraProvider value)	Unknown
MathNet.Numerics.dll!MathNet.Numerics.Control.UseNativeOpenBLAS()	Unknown
MathNet.Numerics.dll!MathNet.Numerics.Control.Try(System.Action action)	Unknown
MathNet.Numerics.dll!MathNet.Numerics.Control.TryUseNativeOpenBLAS()	Unknown
MathNet.Numerics.dll!MathNet.Numerics.Control.TryUseNative()	Unknown
MathNet.Numerics.dll!MathNet.Numerics.Control.ConfigureAuto()	Unknown
MathNet.Numerics.dll!MathNet.Numerics.Control.Control()	Unknown
[Native to Managed Transition]
MathNet.Numerics.dll!MathNet.Numerics.Threading.CommonParallel.Invoke(System.Action[] actions)	Unknown
MathNet.Numerics.dll!MathNet.Numerics.IntegralTransforms.Fourier.BluesteinConvolutionParallel(System.Numerics.Complex[] samples)	Unknown
MathNet.Numerics.dll!MathNet.Numerics.IntegralTransforms.Fourier.Bluestein(System.Numerics.Complex[] samples, int exponentSign)	Unknown
MathNet.Numerics.dll!MathNet.Numerics.IntegralTransforms.Fourier.BluesteinForward(System.Numerics.Complex[] samples, MathNet.Numerics.IntegralTransforms.FourierOptions options)	Unknown
MathNet.Numerics.dll!MathNet.Numerics.IntegralTransforms.Fourier.Forward(System.Numerics.Complex[] samples, MathNet.Numerics.IntegralTransforms.FourierOptions options)	Unknown

Thanks for the update. Yes, this is by design, and may happen at most once (per provider, hence the 3 exceptions) per process/AppDomain. It is not very elegant, but does it cause you any practical problems beyond the debugging annoyance?

It is mostly annoying while debugging.
The thing is, however, that those exceptions stay there even if I install the NuGet package for MKL. Specifically, the MKL related exception stays when I install the package with Id:
MathNet.Numerics.MKL.Win-x64

I cannot try the CUDA one because my computer does not have a nVidia graphic card.

Can you verify that the two dll files of the MKL package did get copied to the x64 subfolder of your output folder, i.e. relative to your application binaries?

Closing as this has been addressed in #374. Thanks for reporting!


Rebuild UML project

##Rebuild and refactor the entire UML diagram of the project
https://github.com/nguyencong1227/vscode-remote-try-php/wiki/UML-project

Choose a use case and elaborate on it

fix in the report


Start porting failures from upstream

Started porting the failures, still needs testing and work on cancellation.

@Sushisource I'm wondering if it's worth keeping the core definitions closer to the upstream ones, I would like Core based SDKs to have the same failure classes as the Java SDK, we shouldn't diverge here too much especially considering the documentation / terminology overhead.
I'm not convinced the exact causes are a good enough reason to use different classes.
If you insist we can delay this PR until Maxim has availability later this week but keep in mind a lot is depending on this change.
FTR: There are still some unresolved issues from our conversation with @mfateev:

Do we ever instantiate the generic TemoralFailure class?
Which failures are valid causes of other failures?

There's also the risk of some other SDK or the server chaining errors in a way we didn't expect and breaking Core processing.
The only way to prevent it is to change the Service proto.

There's also the risk of some other SDK or the server chaining errors in a way we didn't expect and breaking Core processing.
The only way to prevent it is to change the Service proto.

I don't think this is a concern. We already do things differently in core (like, change markers are totally different now), and there's no expectation that you should be able to replay history from a workflow that ran with a non-core SDK. I don't know what kind of crazy someone would have to be to want to write the exact same workflow in different languages anyway.
But, it could reasonably happen with a child workflow, so that part makes sense

If we're explicitly saying we want to be compatible with upstream protos I think we might as well just use them directly at this point.
I do want to edit the upstream repo to actually have docstrings, though.


fastqc not found Nanopreprocess

Hi,
I am getting the following error when I try run NanoPreprocess:
.command.sh: line 2: fastqc: command not found

I have fastqc installed in the conda environment Im working with so im not sure what could be the problem.
Thanks

Hi, we don't recommend using master of pores without containers. You can use singularity with -with-singularity

Hi,
I tried to run it without conda and just using singularity but I keep getting the same error.  It seems there may be an issue with the singularity. I'll keep you updated if I get it to work. Do you have any suggestions on things to try to resolve this?

This is quite strange. Can you verify if the singularity image is there and you can run it?

Yes, I have two singularity images in the singularity directory
When I run the mopprepr image and call fastqc I get the following output
<EMAIL_ADDRESS>From: Luca Cozzuto<EMAIL_ADDRESS>Sent: 08 September 2020 10:50
To: biocorecrg/master_of_pores<EMAIL_ADDRESS>Cc: Samir Watson<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [biocorecrg/master_of_pores] fastqc not found Nanopreprocess (#74)
This is quite strange. Can you verify if the singularity image is there and you can run it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/biocorecrg/master_of_pores/issues/74#issuecomment-688721846, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APC6XXRAUPGK3IN6JK6GKZ3SEXV3PANCNFSM4QYHJ7TQ.

I cannot see sorry :(

PS: which versiom mof singularity are you using?

Im using version 3.5.2-1.1.e17
This is the message when I run fastqc in the singularity
(base) [samirwatson@fe1 singularity]$ ls
biocorecrg-mopbasecall-0.1.img  biocorecrg-mopprepr-0.6.img
(base) [samirwatson@fe1 singularity]$ singularity run biocorecrg-mopprepr-0.6.img
Singularity> fastqc
Exception in thread "main" java.awt.HeadlessException:
No X11 DISPLAY variable was set, but this program performed an operation which requires it.
at java.desktop/java.awt.GraphicsEnvironment.checkHeadless(GraphicsEnvironment.java:208)
at java.desktop/java.awt.Window.(Window.java:548)
at java.desktop/java.awt.Frame.(Frame.java:423)
at java.desktop/java.awt.Frame.(Frame.java:388)
at java.desktop/javax.swing.JFrame.(JFrame.java:180)
at uk.ac.babraham.FastQC.FastQCApplication.(FastQCApplication.java:63)
at uk.ac.babraham.FastQC.FastQCApplication.main(FastQCApplication.java:338)
From: Luca Cozzuto<EMAIL_ADDRESS>Sent: 08 September 2020 11:10
To: biocorecrg/master_of_pores<EMAIL_ADDRESS>Cc: Samir Watson<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [biocorecrg/master_of_pores] fastqc not found Nanopreprocess (#74)
PS: which versiom mof singularity are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/biocorecrg/master_of_pores/issues/74#issuecomment-688733807, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APC6XXWPOT76U6HAU35HZN3SEXYHXANCNFSM4QYHJ7TQ.

It looks like is working (you should use fastqc -h or it will try to open a graphical pop up. We however use version 3.2.1 since the newer had some strange behavior.

Hi, can we consider closed this issue?


Automatic device dependency tracking

We need abstraction of dependencies between devices in order to perform power management operations in correct order and on all devices affected by given operation. For example when suspending a SPI device, we might also suspend SPI Controller (driving SCLK, MISO and MOSI) and GPIO controller (driving /CS) if they are no longer use as well as cut off power from unused part of SoC/board.
All data we need is in the DTS. We have to extract this and convert to a form which will be easy to use by the device power management code.

should we close this?

Seems like this is done, yes.

@nashif / @pabigot - do you have a link to the PR that fixed this issue, please?
(I'm trying to trace whether this has landed for our fork yet)

#32127 supplies the necessary information.  There's very little infrastructure to actually use this data yet, and the API is still evolving (#32547).  But the data from DTS is now available, so this is closed.

Thank you. To clarify, then, the example you give ("when suspending a SPI device, we might also suspend SPI Controller") is not yet implemented?

To clarify, then, the example you give ("when suspending a SPI device, we might also suspend SPI Controller") is not yet implemented?

Correct, no drivers have been updated to do this.  I expect it will ultimately be done using the device power management API, which is currently being discussed elsewhere (I don't know exactly where).


Release to staging

Pull Request Test Coverage Report for Build 1743

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 96.58%

Totals

Change from base Build 1730:
-0.004%

Covered Lines:
3756

Relevant Lines:
3889

💛  - Coveralls


Chip away at Dotty errors

From the dotty branch,
++3.0.0-M1 core/compile

For now, ignore:

Errors mentioning parboiled2 (#3761)
Errors mentiong CollectionCompat (#3945)
Errors mentioning literal macros (#3760)

What progress can we make?  If they're indecipherable, can we begin to categorize them and make more tickets?
This is tedious work that won't land a feature during the Open Source event, but it's high value: many people are asking for Dotty support.

first official dotty support release is imminent


FabConnect does not / can not set transactionIndex and eventIndex

Fabric chaincode events when dispatched to FabConnect by the Fabric SDK do not contain the index within the block:
https://github.com/hyperledger/fabric-sdk-go/blob/7af45cede6afa3939a9574bc9948cca9fb424257/pkg/common/providers/fab/eventservice.go#L43C1-L59
So the only information upon which to build a unique ProtocolID for the event is the block number, and the transaction index (noting Hyperledger Fabric has a restriction of one transaction per block).
Under load when multiple events are packed into a single block in Fabric, the FireFly Core code was de-duplicating them as a single event, because they all had the same ProtocolID, such as<PHONE_NUMBER>95/000000/000000 (where the second and third parts are always zeros).

Codecov Report

Merging #1345 (77e0603) into main (2fbd5b6) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1345   +/-   ##
=======================================
  Coverage   99.99%   99.99%
=======================================
  Files         314      314
  Lines       21492    21491    -1
=======================================
- Hits        21490    21489    -1
  Misses          1        1
  Partials        1        1

Impacted Files
Coverage Δ

internal/blockchain/fabric/fabric.go
100.00% <100.00%> (ø)


Implement W:ET foliage system

The foliage system in Wolfenstein: Enemy Territory and ETLegacy is used to produce runtime grass and other effects like puddle rain drops. One of the original maps that extensively uses foliage is radar. The official documentation is here https://wiki.splashdamage.com/index.php/Foliage_Technology.
The foliage meshes are generated during the map compilation stage by the q3map2 using the q3map_foliage <path to model> <scale> <density> <odds> <invert alpha> directive and saved directly into bsp, as new surface type.
The foliage model then might have the shader which includes the distancecull <opaque distance> <transparent distance> <alpha threshold> directive, which controls the fade distance for each foliage mesh.
This was fixed in etxreal renderer (r2) used in ETLegacy, so this probably can be ported without issues. The foliage is implemented both using vbo (r_vboFoliage) and old (original) way by filling the vertex buffer.
The tracked issue can be found here: https://dev.etlegacy.com/issues/552 along with patches.
Alternatively can be found here: https://github.com/etlegacy/etlegacy/issues/552
The test map with foliage (grass):
foliage_test.zip

Thanks for the detailed explanation! That would be a very good feature to have, even for Unvanquished! For example the good old Tremulous'  Ancient Remains map may make clever usage of it. I hope those patches are easy to adapt.
Good to know: I noticed the ET:Legacy hoc map makes also a test bed for both this missing feature and issue #314 (Translucent texture rendered above fog).
This makes me think about another feature Tremulous had but we do not have if I'm right: foliage disappearing when being very close (so basically you can always see your feet).


GetAxisLimits returns bad values.

I am trying to develop an App using ScottPlot, version 5, on a Windows Form test program.
Using the Designer of my Visual Studio 2022, I dragged a  ScottPlot.WinForms.FormsPlot formsPlot1 onto the test Form.
I then added the following code:
using ScottPlot;
using System.Diagnostics;

namespace testAddText;
public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();

        var nPoints = 100;
        var X = Generate.Consecutive(nPoints);
        var Y = Generate.Sin(nPoints);
        formsPlot1.Plot.Add.Scatter(X, Y);

        var limits = formsPlot1.Plot.GetAxisLimits();
        var left = limits.Rect.Left;
        var center = limits.Rect.HorizontalCenter;
        var right = limits.Rect.Right;
        Debug.WriteLine($"left: {left}, center: {center}, right: {right}");
    }
}

My project configuration is:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <OutputType>WinExe</OutputType>
    <TargetFramework>net8.0-windows</TargetFramework>
    <Nullable>enable</Nullable>
    <UseWindowsForms>true</UseWindowsForms>
    <ImplicitUsings>enable</ImplicitUsings>
    <Platforms>AnyCPU;ARM32;x64</Platforms>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="ScottPlot.WinForms" Version="5.0.11-beta" />
  </ItemGroup>
</Project>

The Debug print returns "left: ∞, center: NaN, right: -∞".
I am trying to center text at the bottom of the plot.
Charles

Hi @CBrauer, thanks for reporting this!
The plot will have no axis limits (infinity) unless SetAxisLimits() or AutoScale() is called. This is the behavior you observed.
If the user doesn't call either of those methods, the plot will call AutoScale() automatically when it is first rendered, so calling GetAxisLimits() after a button click or something will probably return values more consistent with what you were expecting.


Changing the plugin does not change the ss58 address

Describe the bug
Changing the plugin does not change the ss58 address , Now only support [0, 2, 7, 42]
Screenshots
https://user-images.githubusercontent.com/22051991/117531032-b3434f00-afe0-11eb-983d-bbac4fbf1042.mp4
Additional context
i think should add more ss58 in ss58List  ， （Can you help add value 6 in ss58List for bifrost ）
in https://github.com/polkawallet-io/sdk/blob/6644bb7e1959987d5913eb2200a2fd9574839675/lib/storage/keyring.dart

fixed


Dead Julia 0.5.1 kernel on Juliabox

Getting a julia 0.5.1 dead kernel since the morning. Restarting or logging in again don't help. I use Julia in the class I teach and my students and teaching assistants are also getting the same issue. Is there a way to resolve this? Thanks.

I just tried an 0.5.1 notebook on juliabox, and it works fine for me.  Maybe it was a transitory problem with the servers?
In any case, juliabox-specific problems should be reported at https://github.com/JuliaCloud/JuliaBox


Extension issue

Issue Type: Bug
Extension Name: python
Extension Version: 2021.9.1230869389
OS Version: Windows_NT x64 10.0.22454
VS Code version: 1.59.0

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Thanks for the bug report, but unfortunately we need some more information from you in order to triage this issue appropriately. Please create a new bug by providing a filled-in copy of our issue template.


STM32L0X ADC hangs

Describe the bug
ADC conversions on STM32L0x (i.e. L051) hang forever
To Reproduce
Steps to reproduce the behavior:

adc_channel_setup(...)
adc_read(...)
will hang forever in:LL_ADC_IsCalibrationOnGoing or adc_context_wait_for_completion depending on struct adc_sequence calibrate setting

Expected behavior
Will not hang the ADC unit
Impact
ADC unit is trashed, system becomes usable
Additional context
I've identified two issues

struct adc_sequence member calibrate must be set to 0, this avoids a second calibration in start_read() which must not be trigged when ADC is enabled (according to device RM). Zephyr's driver has the ADC enabled here.

similar issue with call to LL_ADC_SetOverSamplingScope in start_read, which seems to touch CKMODE bits and trash the unit (although debugger ADC register read-out may be broken at that point already), the register can be written when disabling the adc (see attached patch).
adc.patch.txt

when not calibrating and applying the patch the ADC unit is not trashed, meaningful conversions are performed

@rlezuo1 , I take the samples/drivers/adc application as an example, because we see the sequence  adc_channel_setup() then adc_read() that you are describing as faulty.
For 1st issue, do you mean that sequence.calibrate = false, is required in the adc_sequence declaration ?
struct adc_sequence sequence = {
	/* individual channels will be added below */
	.channels    = 0,
	.buffer      = sample_buffer,
	/* buffer size in bytes, not number of samples */
	.buffer_size = sizeof(sample_buffer),
	.resolution  = ADC_RESOLUTION,
	.calibrate = false,
};

@FRASTM , exactly

For #2, I see that all STM32 series that have an ADC oversampling (OVSE bit) and an oversampling ratio (OVS bits), the setting  in the ADC CFGR2 "is allowed to write this bit only when ADSTART = 0 (which ensures that no conversion is ongoing). That's why I propose to have
	LL_ADC_Disable(adc);
	while (LL_ADC_IsEnabled(adc) == 1UL) {
	}

before switch (sequence->oversampling).
Then set the LL_ADC_SetOverSamplingScope  and LL_ADC_ConfigOverSamplingRatioShift
Then LL_ADC_Enable(adc) afterwards.

for the same STM32 series, changing  the ADC resolution  "is allowed to write these bits only when ADEN = 0". That's why I also propose to disable(adc) then change the LL_ADC_SetResolution then enable(adc).

@rlezuo1, I put 2 different PRs, if you could take both together and check on your own target board.

there is an issue with the PRs (they do not compile), I'll wait for you to update and then try again?

with the fixed PR's I can confirm ADC is working as expected
54c35b61da (HEAD -> adc_prs) Merge remote-tracking branch 'github_FRASTM/adc_sequence_init' into adc_prs
df5e0edbbb (github_FRASTM/stm32adc_hangs) drivers: adc: stm32 disable the ADC before setting the Oversampling
114c7963f7 (github_FRASTM/adc_sequence_init) samples: drivers: adc initialize the adc_sequence calibration
c2ef1b3fe2 drivers: adc: stm32 disable the ADC before setting the Resolution
2e242c51fb (github_FRASTM/myrepo) soc: riscv: ite: it8xxx2: guard SoC Kconfig options
d63d828c8f stm32: Add FMAC support to Kconfig
*** Booting Zephyr OS build v2.6.0-rc1-3814-g54c35b61da87  ***
I: detect=2
^^^^
value 2 is read from ADC
can be closed from my POV


Host VPs alloc

Hi there!
Assume we have 2 socket system with 22 physical core each with HT enabled (88 LP in Task Manager) and WS 2016 Datacenter installed.
Does this mean that if we use
bcdedit /set hypervisorrootproc 4
That 4LP from host system will be given exclusively to Root partition (to serve Network traffic, CSV workload, Driver interrupts, etc) and guests will not use this LP at all?
Or this setting just limits Root to use only 4 LP instead of 88 LP available on Host system?

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

ID: 24470750-984b-c639-9339-ec79c2cd3e2d
Version Independent ID: 9fa54442-5c52-995c-046e-eb5bfcb95776
Content: Minroot
Content Source: WindowsServerDocs/virtualization/hyper-v/manage/manage-hyper-v-minroot-2016.md
Service: windows-10-hyperv
Product: windows-10-hyperv
GitHub Login: @allenma

hi Specator, were you able to find an answer ?
because I've set hypervisorrootproc 4 on Win10-based Hyper-V and true, processors visible to root partition were limited to 4. However all VM's were executed in context of these processors ONLY ! (ie. vmwp.exe and vmmem processes accounted CPU into these 4 cpu;s only). I dont understand what's the purpose of this setting then, if all VM's are no using remaining free CPUs but fight for those root CPU only ?

I didn’t find any answers and vendor keep silence too

However all VM's were executed in context of these processors ONLY !
Here you can use performance counters to check that other cores working well

I found the reason for me (probably):
Since my Hyper-V is on Win10 1804, that client system is using so called “root scheduler” (https://docs.microsoft.com/en-us/windows-server/virtualization/hyper-v/manage/manage-hyper-v-scheduler-types#the-classic-scheduler)
This scheduler means that you see utilization of VMs directly in root partition’s Task Manager (as VMMEM processes), but this is probably the cause why the bcdedit parameter doesn’t work as expected.
try to run :
Get-WinEvent -FilterHashTable @{ProviderName="Microsoft-Windows-Hyper-V-Hypervisor"; ID=2} | select -Last 1
if you have 0x4 (root scheduler), then you have same issue as me....
Im afraid to change scheduler type via bcdedit /set hypervisorschedulertype because its not recommended on Win10 :-(

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Add support for .MENU files

I am working on building menus on my IBM i. I wanted to avoid using SDA and was introduced to UIM menus (thanks @martintosney for showing me this).
I would like to see if we can add support for the .MENU object type. This would include:

syntax highlighting if possible
compile options

Here is an example of a compile command:
CRTMNU MENU(&OPENLIB/&OPENMBR) TYPE(*UIM) SRCFILE(&OPENLIB/&OPENSPF) SRCMBR(&OPENMBR) INCFILE(UTIL/QPNLREF) REPLACE(*YES)
The INCFILE provides import files for this command. Martin provided some examples here:
https://github.com/martintosney/uim-menu
I can provide an example of the .MENU contents if needed.

Hi @thebeardedgeek
Great idea - but I think most is already available:

UIM menus are defined in the UIM language and are panels like other panel types. If you use the source member type PNLGRP for the UIM menu source, the syntax will be highlighted by the vscode-ibmi-languages extension, which supports UIM keywords (though some menu keywords are not included in the extension and thus not syntax highlighted, but I will make a PR for this).
The compile command can be created as a user defined action.

So you have the tools already to be able to create UIM menus. Only the compile command action is not there yet.

Thanks @chrjorgensen. I added the compile command already and I will look at the highlighting. I was thinking more for the benefit of others that this be included. I appreciate your response.

So to me it sounds like we need to

add a file type association for .menu for the pnlgrp language id.
add a new default compile command

Here is what syntax highlighting looks like when I choose PNLGRP:

I am also included an example of a menu to have this all in one place:
EXAMPLE.zip

@thebeardedgeek The syntax highlighting will work when PR #121 in the vscode-ibmi-languages has been merged by @barrettotte. I just spent a few hours creating and testing this PR.
IMHO I don't think we need (yet) another source type .menu - why not just use the existing type .pnlgrp? Only thing needed is an action CRTMNU for the .pnlgrp source type.
Btw. the default action CRTPNLGRP seems to only be defined for members, not streamfiles. This should be fixed as well.
😃

@thebeardedgeek FYI, this is how your example source looks like with PR #121 applied:

@chrjorgensen I used .MENU based on the fact that IBM has them listed that way on their menu files. Here is an example:

I am not sure what is the recommended, most common, or typical way that people will create menu files but I made the suggestion to add .MENU so that the Code for IBM i extension covers these objects.

@thebeardedgeek I understand what you're thinking...
...but I think you're mixing OBJECT types (like *MENU, *FILE etc) and MEMBER types (like RPGLE, DSPF etc):
There is no 1to1 relationship between object types and member (=source) type. Multiple member types can compile into the same object type, e.g. RPGLE, CLLE, CLE will all compile into object type *PGM. And one member type can compile into multiple object types, e.g. RPGLE can compile into *MODULE, *PGM and *SRVPGM based on the compile command used.
VS Code and Code for IBM i is mostly about developing and compiling source into objects. So the focus is on the source (=member) type, not the object type. We do not handle the *PGM object type (except for the object browser), instead we handle the source types RPGLE, CLLE etc.
The source type is just for linking to the components / commands handling the source - and anyone can invent their own source types (you use MENU and Martin Tosney uses UIM). But the components / commands are built for one or more specific types and will generally only support the predefined types. The Code for IBM i extension allows for some circumvention by user defined actions, but the IBM i languages extension providing the syntax highlighting is built for the predefined types. You can manually select the PNLGRP source type for a source with type MENU to get the correct syntax highlighting, but it will not be selected automatically (unless the extension is changed to handle your MENU source type as well).
As you have already created an action to run CRTMNU, you just need to alter it to (also) work for PNLGRP source types. Then you have two commands for the PNLGRP source types - CRTMNU and CRTPNLGRP - and you select the command for the object you want to create - menu or panel group. If you choose the wrong command, the compiler will tell you! 😄
So my advice will be that you stick with the predefined source type PNLGRP and use your action for CRTMNU on this source type.
As for what is used to create menus by the IBM i community, I don't know - but based on what I've seen through my years on this platform I guess very few IBM i professionals actually know the UIM language and use it for menus. I think the majority either use a display file (together with a message file) or a program...

@chrjorgensen I see what you are saying. I took the steps you mentioned and I have syntax highlighting and I can compile the member if I tie PNLGRP to CRTMNU. If I had known about UIM back when I was creating more screens I think I would have opted for this route instead of the hours it would take to create a DSPF, PGM, etc. Each have their place and value though.
Thank you for the info.

@thebeardedgeek I am still interested in adding a CRTMNU default since we don't have one.
Would you mind sharing the Action you have setup?

@chrjorgensen Thank you the PR you created over on IBMi Languages. I think that will make @thebeardedgeek and Martin's experience just a tad better!

This is what I am using but it can be modified. The INCFILE might be different for others but I have my menu import files in the same library as these new menus I am building.
{
"type": "member",
"extensions": [
"MENU",
"PNLGRP"
],
"environment": "ile",
"name": "Create Menu (CRTMNU)",
"command": "?CRTMNU MENU(&OPENLIB/&OPENMBR) TYPE(*UIM) SRCFILE(&OPENLIB/&OPENSPF) SRCMBR(&OPENMBR) INCFILE(&OPENLIB/QPNLREF) REPLACE(*YES)"
 }

@thebeardedgeek I'm very curious about the import files (I have never used it myself). Does the :IMPORT tag copy other members from the INCFILE source file? It's not documented in the Application Display Programming manual, which only talks about other PNLGRP objects...?

@thebeardedgeek Just wanted to let you know, that the new UIM menu tags support is included in the newest release 0.6.9 of the IBM i Languages extension, so your menu sources will have syntax highlighting after upgrade.
Hope this helps!

@chrjorgensen yes, you can create files with settings that can be imported using the INCFILE. This allows for them to be used across multiple UIM/MENU programs.
@martintosney sent me this article which helped with understanding UIM menus:
https://www.mcpressonline.com/programming-other/general/create-ibmstyle-rolling-menus-with-uim
Is it possible to add the .UIM file type to IBMi Languages? It shows as Type: Plain Text currently. If I change it to PNLGRP it looks good.

@thebeardedgeek

yes, you can create files with settings that can be imported using the INCFILE. This allows for them to be used across multiple UIM/MENU programs. Does that answer your question?

Yes, thank you. The documentation is just not clear on this. And thanks for the link to the previous century... 1993, just five years after the birth of the AS/400! 😆
You can setup file association for UIM files: Open a UIM file, click on the file type (which you've marked in your screenshot) and configure a file association:

Select PNLGRP as the association:

From now on your UIM files will be considered a PNLGRP file, including syntax highlighting.
File association is a standard feature of VS Code and not specific to any file type.

LOL...sorry the documentation on this is not very available but I thought that article had some halfway decent info. The import files remind me of some of the things we have available now to centralize commonly used code.
Thank you for the information on File Associations. I went ahead and did what you recommended.

@thebeardedgeek Just wanted to inform you that I've finalized support for all UIM tags in PNLGRP and also the .im (imbed) directive, and it is included in the latest version 0.6.10 of vscode-ibmi-languages extension.
So all your UIM menu tags and imbeds should be properly syntax highlighted now.


Increase eventLimit in GRM to improve troubleshooting

How to categorize this PR?

/area ops-productivity
/kind enhancement
What this PR does / why we need it:
Sometimes when a ManagedResource fails to be deleted, or it is just taking some time ( status condition is observed to be DeletionPending) The events that are being printed as a status.conditions[].message may not show the relevant information. This PR increases the number of events from 2 to 5. This will allow more events to be shown in the monitors and hopefully improving the supportability.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Part of (or mitigation) #4808
The events that we are receiving are all with status Warning, the reason does not contain clear information about the importance and the message is meant for human reading, so currently there is not a easy way to find out what error message is more relevant than the other.
Release note:

NONE

Maybe @timuthy as initial author has an opinion here :)

Let's try to increase the count first as done in this PR. We can still make it configurable if we see the necessity to do so in the future (e.g. if we need to increase the count again).


Support compilation from 64bit ARM (IDFGH-4137)

I'm using Raspberry Pi 4 as a development platform. Just switching to Ubuntu 64bit (from Raspberry Pi OS which is 32bit) and found out there's no toolchain supported for it. I'm not familiar with toolchains portability, but is it possible to port the current one to work on 64bit ARM?
Thanks

Hi @galah92, it should be possible. We will consider adding such builds in the next toolchain release, but I can't give you exact timeline when this will happen. For now you can try the steps for building the toolchain locally: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/get-started/linux-setup-scratch.html#setup-linux-toolchain-from-scratch.

Hi @galah92, it should be possible. We will consider adding such builds in the next toolchain release, but I can't give you exact timeline when this will happen. For now you can try the steps for building the toolchain locally: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/get-started/linux-setup-scratch.html#setup-linux-toolchain-from-scratch.

Thank you. Would be great if you could ping me (here?) when there are updates.

Thanks for reporting, fix is available at https://github.com/espressif/esp-idf/commit/00077545bf23134379133cfd979aaad35d5f3552, feel free to reopen.


Multiple Languages Support

Hello,
I noticed there is a file for the French language anodyne/langauges/fr.yml, does that mean the theme supports multiple languages ? I tried to add
language:
- en
- fr

in my config file _config.yml.
But it seems when I visit http://localhost:4000/fr it returns 404 not found.
What I am doing wrong ?

Does it work with another theme ? Technically this should be handled by hexo as long as the language files are present.
https://hexo.io/docs/internationalization.html
I can give it a shot tonight when I am home


deploy/operator.yaml - update stratigy to be recreate since rolling wont work with an operator where only one pod can be the master

since only one operator pod can be master at a time with the default deployment strategy for rolling the roll never successfully happens because the old pod is staying up till the new pod is healthy but the new pod can never be healthy because its not the leader. for operator best to just set strategy to recreate

Good catch. Looks like this might solve the problem where the new pod doesn't come up until you manually remove the operator-lock config map (#74 )?

@pb82 it should indeed resolve #74


Cannot add folder on network

I have most of my collection on NAS. The problem is i cant add folder from network such as...
\<IP_ADDRESS>\NetAxcess\Book 01\AnimaX
or mapped network folder
T:\Book 01\AnimaX
it shows error ..
could not inset: [JMMServer.Entities.ImportFoIder][SQL: INSERT INTO
[ImportFoIder] (ImportFoIderType, ImportFoIderLocation,
ImportFoIderName, IsDropDestination, IsDropSource, IsWatched)
VALUES (?, ?, ?, ?, ?); select SCOPE_IDENTITY()]

Can you please provide me your full log, as it will have extra details.
It might be easier to PM me on discord
https://discord.gg/0XKJW7TObKIX4Ug2

"it's not a bug, it's a feature"... that's what microsoft thinks ;)
that's not a problem in jmm (well, partially it is), but in how Windows handles network mounts and user sessions
as JMM Server requires admin rights to run as of now, the mounted network shares that you see when you browse for a source folder are those of the Administors Session, not of your normal user Session (even though you might have administrative rights and UAC disabled)
to fix that:
run cmd as administrator, mount the share inside that cmd (note: the title of that cmd window should say Administrator: C:\Windows\System32\cmd.exe, or where ever your windows installation resides)
type "net use : \<share> /persistent:yes /user: "
replacing the words inside <> with appropriate values and hit enter
after that, you should be able to access that mount with jmm server :)
good luck with that, and if you should need help with that, hit me on discord

https://technet.microsoft.com/en-us/library/bb490717.aspx
Example:
net use e: \Server\Anime /persistent:yes /user:
Not sure if "/user: " is ok...

well, the /user parameter depends on how the share is secured...
for example, i have mine set up in samba with basic auth, the user that accesses the share has to be in the group "users"
there for, i mount the share the following way:
net use x: \\<IP_ADDRESS>\Anime /persistent:yes /user:baine mypassword
(note: not real user and password :P )
if the share is accessible by everyone (anonymous etc), then /user can be omitted :)


Using a temporary search pipeline for a request - pipeline vs search_pipeline

https://opensearch.org/docs/latest/search-plugins/search-pipelines/using-search-pipeline/#using-a-temporary-search-pipeline-for-a-request

[ ] Report a technical problem with the documentation

Detailed attached : https://github.com/opensearch-project/OpenSearch/issues/11030

Documentation i
Based on integration test. its search_pipeline, not pipeline.

D'oh! This was probably my fault.
If I recall correctly, the parameter was just pipeline originally, but we changed it to search_pipeline just before the feature shipped. The examples that I sent for documentation were likely from before the change.


chore(remix-react): make inline scripts more robust

For inline scripts, instead of directly injecting JavaScript using a template literal:

Write a function with the desired code, using params for external values
Convert this function to a string using Function.prototype.toString()
Inject the resulting string, and call the function with the required values

This provides the following benefits for inline scripts:

Problem checking with ESLint
Type checking with TypeScript
Consistent formatting with Prettier
Consistent baseline indentation

I haven't added or changed any tests since this isn't strictly a bug fix or new feature.

@ryanflorence I've rebased with the new script changes 🙂

Heh, our tests here are a little funny, I'll fix them.


JSON property is missing from schema when using conditional serialization

With a model that has a property that is hidden using ShouldSerialize, e.g:-
public class TestModel
{
    public bool Foo { get; set; }

    public bool Bar { get; set; }

    public bool ShouldSerializeFoo() => false;
}

The schema generated for this model excludes the Foo property when the model is used as a response (as expected) but it is also excluded when the module is used for a request.
As this Foo property is only excluded from serialization, and not from deserialization, I would expect it to be present in the schema.
In addition, if the conditional serialization of a property relies on a value that is set only under certain conditions at runtime (and which is not satisifed when NSwag is run), the property is exlcuded from the spec.

Conditional serialization is hard to express in a schema as the generator does not know whether the type is used as input or output...

Isn't the determination on whether the model is being used for input or output made based on where it appears? i.e. if it's a return value, then it's output; if it's an argument to a controller method, then input?
Aren't both those scenarios handled by different processors?

There is no difference between inputs and outputs - this is also more complex as you would also need to correctly track child schemas... additionally the Newtonsoft.Json contract resolver is not really exposing enough information I think. So implementing this would probably be a huge undertaking - also when keeping the new .net core JSON serializers in mind...

Right; so something like a serialize/deserialize flag on ITypeMapper so cached models can be split between serialized and deserialized versions and referenced accordingly (which could default to deserialize as it does now), and a custom contract resolver?

Isnt this discussion about readonly and writeonly properties? ie your initial sample should set readonly on foo to false?

It's not necessarily about read-only/write-only; it's about the parameter appearing at all (though they amount to the same thing I guess).
My thinking is that it doesn't make sense for a write-only property to appear in a response model (and vice-versa). However, if it is at least marked as read-only or write-only, the it can just be omitted during rendering.
Anyway, that was a bit of a side-track; the issue here is that I've designated Foo as write-only (using ShouldSerialize), and the property doesn't appear at all in the model.
It appears ShouldSerializeFoo is evaluated rather than just reflected; if it happens to evaluate to false at the time it's inspected, it's not included in the model.

Related: https://github.com/RicoSuter/NSwag/issues/4306

My grain of salt, I had to use the following syntax to grab the serializer settings, this may be helpful for someone:
            services.AddOpenApiDocument((settings, serviceProvider) =>
            {

                NJsonSchema.NewtonsoftJson.Generation.NewtonsoftJsonSchemaGeneratorSettings jsonSettings = (NJsonSchema.NewtonsoftJson.Generation.NewtonsoftJsonSchemaGeneratorSettings) settings.SchemaSettings;


Inspector does not provide output

I tried to use TracerEip3155 as Inspector(ExternalContext) but there is no output, I'm missing something?
My goal is to debug the transaction.
Code:
type RevmEvm<'a> = Evm<'a, TracerEip3155, SharedInMemoryDB>;

#[inline]
    fn get_evm(revm_ctx: RevmContext) -> RevmEvm<'static> {
        let mut eip3155 = TracerEip3155::new(Box::new(io::stdout()));
        eip3155 = eip3155.with_memory();

        let ctx = ContextWithHandlerCfg {
            cfg: revm_ctx.cfg,
            context: Context {
                evm: revm_ctx.context.evm,
                external: eip3155,
            },
        };

        Evm::builder().with_context_with_handler_cfg(ctx).build()
    }

fn send_tx(
        revm_ctx: RevmContext,
        caller: Address,
        to: Address,
        calldata: Bytes,
        commit: bool,
    ) -> Result<(ExecutionResult, Option<State>)> {
        let mut evm: RevmEvm = Self::get_evm(revm_ctx);

        let tx: &mut TxEnv = evm.tx_mut();
        tx.caller = caller;
        tx.transact_to = TransactTo::Call(to);
        tx.data = calldata;

        if commit {
            let result: ExecutionResult = evm.transact_commit()?;
            Ok((result, None))
        } else {
            let r: ResultAndState = evm.transact()?;
            Ok((r.result, Some(r.state)))
        }
    }

Here is how I make the transaction:
pub fn get_token_balance(&self, token: &Address, holder: &Address) -> Result<eU256> {
        let calldata: Bytes = AbiEncode::encode(BalanceOfCall {
            who: holder.0 .0.into(),
        })
        .into();

        let tx_result: Result<(ExecutionResult, Option<State>)> = Self::send_tx(
            self.revm_ctx.clone(),
            self.account.0.into(),
            token.0.into(),
            calldata.clone(),
            false,
        );

        let result_and_state: (ExecutionResult, Option<State>) = match tx_result {
            Ok(result) => result,
            Err(e) => return Err(anyhow!("EVM call failed: {e:?}")),
        };

        let tx_result: TxResult = Self::get_tx_result(result_and_state.0);
        match tx_result {
            TxResult::Success(result) => {
                let Ok(decoded_output) = BalanceOfReturn::decode(&result.output) else {
                    return Err(anyhow!("Failed to decode output"));
                };

                //Ok(U256::from_limbs(decoded_output.0.0))
                Ok(decoded_output.0)
            }
            TxResult::Revert(_) => Err(anyhow!("Failed to get token balance")),
            TxResult::Halt(_) => Err(anyhow!("Failed to get token balance")),
        }
    }

In your case, the output will be print into stdio. there is an example will write to json file here : https://github.com/bluealloy/revm/blob/main/examples/generate_block_traces.rs

In your case, the output will be print into stdio. there is an example will write to json file here : https://github.com/bluealloy/revm/blob/main/examples/generate_block_traces.rs

works thanks


test: giving better debug information

nope, but I figure this will let us know which step failed (even after logging is removed)

/retest


Feature: provide latest tag for raspberrypi3-homeassistant

Would be nice with a latest tag for the latest rpi3 docker version. This way you can have the docker auto-update by using a docker container like: https://github.com/v2tec/watchtower.

fix with next rl


feat(hooks): change event type for usePageActiveTrigger

Ticket reference: https://sumupteam.atlassian.net/browse/OT-256
Add page-reactivated type to Events in order to separate from page-view. Change usePageActiveTrigger
according to this change.

This only covers part of OT-256, right?

The usePageView hook should dispatch an event on the initial page load and any subsequent route change. An open question is how to detect a route change. Developers might have to do this manually or provide a prop such as the pathname that can be used as a dependency for useEffect.

Will this be covered in a follow-up PR?

Ahhh yah completely forget this request.
I think it actually makes sense to provide this feature from Collector since one of the criteria of a new "page view" is route changes. The marketing website currently implements this manually in the website codebase. But I think it makes sense to make it provide by Collector.
There will be a following PR once it is implemented.
Thanksssss :)


⚠ Secret Website 3 has degraded performance

In d4f72b4, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2083 ms

Resolved: Secret Website 3 performance has improved in f0fce0e.