Ruby 3 - ERROR: There was a problem loading your configuration: Alias parsing was not enabled

The following error happens when using a config file with aliases with Ruby 3:
ERROR: There was a problem loading your configuration: Alias parsing was not enabled. To enable it, pass `aliases: true` to `Psych::load` or `Psych::safe_load`.

This seems to be an issue with Psych changing the default behaviour around expanding aliases in Ruby 3.
Ideally we'd fix the issue here, but in the meantime here's what I'm doing:

copied the config to another file (~/.ssh/knuckle_cluster.aliases.yml)
replaced the original file with expanded aliases from the copy above

This allows me to keep aliases around until this is sorted out.
I created a script to assist with expanding aliases:
# Use this script to solve the following error.
# ERROR: There was a problem loading your configuration: Alias parsing was not enabled. To enable it, pass `aliases: true` to `Psych::load` or `Psych::safe_load`.
#
# It will convert a YAML file with aliases to one without them.
# Allowing using knuckle_cluster with Ruby 3

require 'psych'

source_file = File.expand_path(ARGV[1] || '~/.ssh/knuckle_cluster.aliases.yml')
script_path = File.expand_path($0)

contents = Psych.load_file(source_file)

puts "# File auto-generated from '#{source_file}'"
puts "# Using script at '#{script_path}'"

puts Psych.dump(contents, :indentation => 3)

Warning
This workaround script needs to be executed using Ruby 2.x.
It will show the same error if you run it on Ruby 3+.

Warning
This workaround script needs to be executed using Ruby 2.x.
It will show the same error if you run it on Ruby 3+.

Good pickup @RafaelChefe, I've updated the script to work with Ruby 3 too.


Invoke signature help upon method, function and constructor completion

Having trouble getting this working.
command: {
    command: 'vscode.executeSignatureHelpProvider',
    arguments:[uri, position]
}

implemented as of v0.8.0


[DOC] Alphabetizing docdict

I think the original organization of docdict was great when there were limited entries but now, in my opinion, it's getting a bit unruly and disorganized. My suggestion would be to alphabetize and document the standardized format that the argument should go first with any specific separated by and underscore, i.e. there can be trans and trans_not_none which will be next to each other alphabetically and that way it will be easy to tell which docstring (i.e. (%(color_matplotlib)) goes to which argument (i.e. color) where %(matplotlib_color) for the argument color is a bit confusing that it goes to the argument color for  a function such as f(..., color=None, ...).
cc @hoechenberger @agramfort

sounds like a good idea !

Message ID: @.***>

+1 if you volunteer to do it 😅

Hey @andylikescodes, I make this issue so I'm happy to do a PR, but I thought it might be a good one for you to do your first pull request. If you're interested, here are the steps:
Make sure you have a fork as origin and the mne main repository as upstream with
git remote -v

It should look like this:
origin	https://github.com/alexrockhill/mne-python.git (fetch)
origin	https://github.com/alexrockhill/mne-python.git (push)
upstream	https://github.com/mne-tools/mne-python.git (fetch)
upstream	https://github.com/mne-tools/mne-python.git (push)

Switch to your main branch if you aren't on it already:
git checkout main

Get any new changes to the main branch:
git pull upstream main

Make a new branch called something like docdict
git checkout -b docdict

Copy paste around in the file mne/utils/docs.py so that all the docdict entries are in alphabetical order, maybe with
# %%
# A

section headers for each letter.

Honestly, this is definitely busywork but it would give you an introduction to the MNE-Python workflow and review process. It's nice to start with something pretty basic, otherwise you just get bogged down because there is a process of assimilating into the MNE style that takes a bit of time. If you'd rather do something else for your first PR, I wouldn't be offended in the slightest and I will go ahead and do this one, it just made me think it would be a good way to get some experience contributing for you.

Just noticed that all the topomap docdict entries are topomap_x which doesn't follow this convention and will be inconsistent in alphabetization, note to self to fix this

If you're interested, here are the steps:

Oh my, no wonder contributing is so hard 😅 We should write an intro explaining how to do all of this using the GitHub GUI

Hahaha @andylikescodes said he has a good amount of previous GitHub experience so I think that might have been mostly review for him, but in general, I think that's a great idea. There is documentation about it https://mne.tools/dev/install/contributing.html but it's really not by example, it's quite long and if I were a new contributor I might not know where to start/how to do a PR.

@alexrockhill Yes, this seems like a great starting point for the journey. I would love to try it out. Thank you for all these suggestions. I will have some time later today and I will try it out. Will let you know when I am done.

@alexrockhill Do the entries in the docs.py file have some logical order? Should I just extract all the docdict entries and put them into a big section? These docdict entries are actually separated into different sections, for example, this section here:
# raw/epochs/evoked apply_function method
# apply_function method summary
applyfun_summary = """\
The function ``fun`` is applied to the channels defined in ``picks``.
The {} object's data is modified in-place. If the function returns a different
data type (e.g. :py:obj:`numpy.complex128`) it must be specified
using the ``dtype`` parameter, which causes the data type of **all** the data
to change (even if the function is only applied to channels in ``picks``).{}

.. note:: If ``n_jobs`` > 1, more memory is required as
          ``len(picks) * n_times`` additional time points need to
          be temporarily stored in memory.
.. note:: If the data type changes (``dtype != None``), more memory is
          required since the original and the converted data needs
          to be stored in memory.
"""
applyfun_preload = (' The object has to have the data loaded e.g. with '
                    '``preload=True`` or ``self.load_data()``.')
docdict['applyfun_summary_raw'] = \
    applyfun_summary.format('raw', applyfun_preload)
docdict['applyfun_summary_epochs'] = \
    applyfun_summary.format('epochs', applyfun_preload)
docdict['applyfun_summary_evoked'] = \
    applyfun_summary.format('evoked', '')

I am assuming I should not break this structure while I am reorganizing the docdict entries?

Yeah the idea would be that those are all in order so you can just copy the whole block over. Those would all be in the "A" block because the docdict key starts with an a. Okay progress, nice!

Also maybe just do a few and then make the PR so you don't do a bunch of extra work that needs to be redone.

What about this block?
# Verbose
docdict['verbose'] = """
verbose : bool | str | int | None
    Control verbosity of the logging output. If ``None``, use the default
    verbosity level. See the :ref:`logging documentation <tut-logging>` and
    :func:`mne.verbose` for details. Should only be passed as a keyword
    argument."""
docdict['add_frames'] = """
add_frames : int | None
    If int, enable (>=1) or disable (0) the printing of stack frame
    information using formatting. Default (None) does not change the
    formatting. This can add overhead so is meant only for debugging.
"""

There are "a" and "v" at the same time.

Verbose should go in v near the bottom and add frames should go in a near the top. I hope that makes sense.

Oh sorry, let me frame my question a bit more clear, what I meant was that in this blog of code:
# raw/epochs/evoked apply_function method
# apply_function method summary
applyfun_summary = """\
The function ``fun`` is applied to the channels defined in ``picks``.
The {} object's data is modified in-place. If the function returns a different
data type (e.g. :py:obj:`numpy.complex128`) it must be specified
using the ``dtype`` parameter, which causes the data type of **all** the data
to change (even if the function is only applied to channels in ``picks``).{}

.. note:: If ``n_jobs`` > 1, more memory is required as
          ``len(picks) * n_times`` additional time points need to
          be temporarily stored in memory.
.. note:: If the data type changes (``dtype != None``), more memory is
          required since the original and the converted data needs
          to be stored in memory.
"""
applyfun_preload = (' The object has to have the data loaded e.g. with '
                    '``preload=True`` or ``self.load_data()``.')
docdict['applyfun_summary_raw'] = \
    applyfun_summary.format('raw', applyfun_preload)
docdict['applyfun_summary_epochs'] = \
    applyfun_summary.format('epochs', applyfun_preload)
docdict['applyfun_summary_evoked'] = \
    applyfun_summary.format('evoked', '')

We have this block that is not docdicts:
# raw/epochs/evoked apply_function method
# apply_function method summary
applyfun_summary = """\
The function ``fun`` is applied to the channels defined in ``picks``.
The {} object's data is modified in-place. If the function returns a different
data type (e.g. :py:obj:`numpy.complex128`) it must be specified
using the ``dtype`` parameter, which causes the data type of **all** the data
to change (even if the function is only applied to channels in ``picks``).{}

.. note:: If ``n_jobs`` > 1, more memory is required as
          ``len(picks) * n_times`` additional time points need to
          be temporarily stored in memory.
.. note:: If the data type changes (``dtype != None``), more memory is
          required since the original and the converted data needs
          to be stored in memory.
"""
applyfun_preload = (' The object has to have the data loaded e.g. with '
                    '``preload=True`` or ``self.load_data()``.')

While these are docdicts:
docdict['applyfun_summary_raw'] = \
    applyfun_summary.format('raw', applyfun_preload)
docdict['applyfun_summary_epochs'] = \
    applyfun_summary.format('epochs', applyfun_preload)
docdict['applyfun_summary_evoked'] = \
    applyfun_summary.format('evoked', '')

What should I do with the first half of the code that are not docdicts?

My advice would be to keep them together

@andylikescodes, did you have a chance to open a PR? It would be a lot easier to give you advice if you just changed like one or two of the entries, opened the PR and then I'd be like "yeah, looks awesome" or "you might need to change this part because that's going to break this" and then you'd be on your way. Be sure when you do to write "Fixes x" where x is the url of this issue.

I didn't realize how big doctdict was already, I think this is a bit too late. What I thought might be a fun onboarding exercise  looks like the endless PR to scare away new contributors! Sorry @andylikescodes!
I wrote a script to automate it but it still took an hour of fiddling.


issue with decoding the eth1 config

issue with decoding the eth1 config file in the Go code. it seems that this number is being interpreted as the value for the cancunTime field of type uint64 in the ChainConfig struct.
zrnt version: v0.30.0
failed to decode eth1 config file: json: cannot unmarshal number<PHONE_NUMBER>304467820220 into Go struct field ChainConfig.config.cancunTime of type uint64

That would happen if the default value for 4844(https://github.com/ethpandaops/ethereum-genesis-generator/blob/master/config-example/values.env#L13) is too high. It basically overflows when setting the timestamp for cancun.
You can set it to a realistic(yet never reaching) number like epoch 99,000 or something.


Provide packaging routines / CMake target

I was trying to add centurion to my vcpkg registry and was somewhat surprised that it didn't generate INSTALL targets.
So this patch is a first pass at packaging centurion in the "modern" tradition.
Notes:

We generate a centurion.pc for anyone still attached to pkg-config
We use file sets to ensure the headers are installed and linked correctly. This dodges a minor issue of integration as a subproject vs imported target, as file sets always generate the correct path information.
Using file sets bumps the minimum required CMake version to 2.23
We provide an alias for centurion::centurion so that subproject/imported target usage is the same
Tests and examples use the new target, for demonstration purposes

Possible Future Work:

Separate targets that don't require mixer/ttf/image
Deprecate the set() vars? They're considered an anti-pattern in some CMake circles

A working portfile using this patch is here (commit).
If you want to test this out, the following vcpkg.json will do it:
{
  "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg-tool/main/docs/vcpkg.schema.json",
  "name": "centurion-test",
  "version": "1.0.0",
  "dependencies": [
    "centurion"
  ],
  "vcpkg-configuration": {
    "default-registry": {
      "kind": "git",
      "baseline": "f6a5d4e8eb7476b8d7fc12a56dff300c1c986131",
      "repository": "https://github.com/microsoft/vcpkg.git"
    },
    "registries": [
      {
        "kind": "git",
        "baseline": "a3bba28ee7354195ae6f2682e4699cd88257e84a",
        "repository": "https://github.com/nickelpro/vito-reg.git",
        "packages": [
          "centurion"
        ]
      }
    ]
  }
}

Second commit differentiates targets:

centurion::centurion-headers-only: Bring your own SDL
centurion::centurion: Use shared libraries if they exist, otherwise static
centurion::centurion-static: Use static libraries, only available if static libraries exist

Third commit makes centurion::centurion and centurion::centurion-static optional, completely removing the find_package(REQUIRED) for SDL. It's possible to configure centurion as a pure headers lib and link in SDL manually from wherever

Fourth commit correct a minor oversight about needing to link sdl2main
Fifth commit contains changes necessary for CI to work:

Update Ubuntu hosts before installing dependencies, surprising you didn't run into this before. Upstream dev fails CI for me right now because of this
Drop Windows SDL versions which don't ship a config file. These are annoying but not impossible to test, but better to just test them on *Nix/MacOS. If it's highly desired to support this I'll add a hack.
Pass the SDL2*DIR env args to find_package if available


Trollsploit Rick Ascii doesnt work

Empire Version
[Version] 2.5
OS Information (Linux flavor, Python version)
4.19.0-kali1-amd64
Expected behavior
Trying to use powershell/trollsploit/rick_astley but it just opens a new commandline on the victim and exits with an error. I dont know what error, because commandline window is closed super fast.
Victim is Windows 7.

I believe this module relies on internet access. If you were trying to do this in an internet-free environment, then it most likely will not work.
I have a version that does not require internet here: https://github.com/jbooz1/Invoke-RickRoll

Hmm, I did have internet access on the machine.


[sqlite3] fix issue with initialize

Fixes: #611

thanks, but this seems to change the semantics of initialize - we need to keep it compatible with several AR versions as well as our own usage ... would be good to have a failing piece of column code at #611

Yes I am seeing this change failing other tests on Travis. Ill give it another go.


Adding support for fxos

I'm working on adding phonegap developer app to firefox OS. I have it working pretty well, I wrote a blog post about it on hacks.mozilla with a screencast of it running.  All fixes either shipped as part of cordova 3.6.3 or will ship with next plugins release soon.
I had to use a mix of cordova and phonegap tools to get it all working. I don't think this PR is ready to merge until PhoneGap gets all 3.6 goodies, but would truly appreciate any feedback.

Closing this as fxos is no longer a thing. Thanks for your contribution anyways.


Serializing Tests within the suit

For example i have 2 tests
TC_1_Login : In this test case we are navigating to given url and validating login info
TC_2 Create customer : In this test case we are creating customer
I added both Tests in to a suit and i want to execute TC_1 and TC_2 serially  within one session but when i run the suit
browser is closed after " TC_1_Login " case is executed  and TC_2 is failed since login case browser is closed and logged in session is no more.

Please check it #229  hope it will help to continue login session.

Thanks Sunil


FinRL_multiple_stock_trading - ValueError: operands could not be broadcast together with shapes (30,) (0,)

I am trying to run the code provided for FinRL_multiple_stock_trading, but I get Value error while running the following line.
df_account_value, df_actions = DRLAgent.DRL_prediction(model=model_sac,
                        test_data = trade,
                        test_env = env_trade,
                        test_obs = obs_trade)

@BruceYanghy Can you please tell how it can be solved? Thank you very much for sharing your work, I would like to experiment with your code, appreciate your help.
Best,
Harshad

I got this when running under v3.8, but worked fine using v3.7

I am having the same issue when running on Colab.
Python version is 3.6.9

I had this issue as well but saw that @BruceYanghy made an update to EnvMultipleStock_trade.py which fixed the issue. Replacing the total reward print statement on lines 134-135 with the following code fixes the error. I'm still investigating why this works, but it is worth noting that the change was reverted in a later commit.
print("total_reward:{}".format(self.state[0]+sum(np.array(self.state[1:(self.stock_dim+1)])                                                              * np.array(self.state[(self.stock_dim+1):61])) - self.asset_memory[0]))

61 is hardcoded, I changed it to (self.stock_dim*2+1)


Upgrade Licensor dependency to version 1.25

Issues closed in this version https://github.com/xwikisas/application-licensing/milestone/42?closed=1

The code upgrade was not done completely https://github.com/search?q=repo%3Axwikisas%2Fapplication-admintools Licenses.Code.VelocityMacros&type=code . You need to create a bug report for it
A separate issue is needed since this one is already included in a release. Also, now it's an admin tools bug, and not a licensing one, because you will see an error with non-admin users and this is caused by a misuse of licensing code (see https://github.com/xwikisas/application-licensing/issues/155)


ImportError: No module named pocketsphinx

i am getting below error..how to solve it?
pi@raspberrypi:~/AlexaPi $ sudo python main.py
Traceback (most recent call last):
File "main.py", line 27, in
from pocketsphinx.pocketsphinx import *
ImportError: No module named pocketsphinx
i tried pip install and i got this
pi@raspberrypi:~/AlexaPi $ pip install -r requirements.txt
Requirement already satisfied (use --upgrade to upgrade): Wave>=0.0.2 in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 1))
Requirement already satisfied (use --upgrade to upgrade): python-memcached>=1.50 in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 2))
Requirement already satisfied (use --upgrade to upgrade): requests>=2.4.3 in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 3))
Requirement already satisfied (use --upgrade to upgrade): wsgiref>=0.1.2 in /usr/lib/python2.7 (from -r requirements.txt (line 4))
Requirement already satisfied (use --upgrade to upgrade): CherryPy>=4.0.0 in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 5))
Requirement already satisfied (use --upgrade to upgrade): python-vlc>=1.1.2 in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 6))
Requirement already satisfied (use --upgrade to upgrade): pyalsaaudio>=0.8.2 in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 7))
Requirement already satisfied (use --upgrade to upgrade): py-getch>=1.0.1 in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 8))
Requirement already satisfied (use --upgrade to upgrade): pocketsphinx>=0.1.0 in /usr/lib/python2.7/dist-packages (from -r requirements.txt (line 9))
Requirement already satisfied (use --upgrade to upgrade): webrtcvad>=2.0.7 in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 10))
Requirement already satisfied (use --upgrade to upgrade): six>=1.4.0 in /usr/local/lib/python2.7/dist-packages/six-1.10.0-py2.7.egg (from python-memcached>=1.50->-r requirements.txt (line 2))
and while running setup.sh i got this yellow text
Found existing installation: pyalsaaudio 0.7
DEPRECATION: Uninstalling a distutils installed project (pyalsaaudio) has been deprecated and will be removed in a future version. This is due to the fact that uninstalling a distutils project will only partially uninstall the project.
Uninstalling pyalsaaudio-0.7:
Successfully uninstalled pyalsaaudio-0.7

Looks like you're trying to run as pi instead of root.
And based on the fact that it wants pocketsphinx, you're not running the one version (maso27/snowboy) that is designed to be run as pi.
Which version are you using?  Most likely, modules are looking for items in /root, while your whole install is in /home/pi.
Do you have a pocketsphinx directory, either in /root or /home/pi?
main.py is looking for that.
You can potentially perform a quick fix by moving pocketsphinx into the directory that it's seeking.  Might take more than that though.

Seems like you're running Mason's version1.2 branch. Use the new upstream instead: https://github.com/alexa-pi/AlexaPi It is based on that one, but has enhancements. Follow the installation guide.

This project is now deprecated in favor of the new AlexaPi (https://github.com/alexa-pi/AlexaPi) which has all the features of this project and much more, such as:

support for a wide range of devices and platforms (Raspberry Pi, CHIP, Orange Pi, desktops, Magic Mirror and others)
voice activation (great PocketSphinx support and snowboy support soon to be released)
much more robust installation and configuration
better security
Arch Linux support
awesome documentation
and much, much more! You get it - it's just awesome.

Please switch over to the new project and if your issue persists there, file an issue in the new repo's issue tracker. Thank you.


LHN - The deleted task content shown on LHN for 1:1 Conversation

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.50-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5092553
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Precondition: User A create task in the group and assign for user who is not a member of the group
Steps

Navigate to http://www.staging.new.expensify.com/
As assignee user, navigate to the task and take a look on LHN of 1:1 conversation with the task creator
Open the task details, Click on the header and delete the task

Expected Result:
Deleted task shown in both task detail and LHN on both the task and 1:1 Conversation
Actual Result:
The deleted task content shown on LHN for 1:1 Conversation rather than Deleted task message
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/7fc09e6f-80bc-44ae-bf8a-6bb620c208b8
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
LHN - The deleted task content shown on LHN for 1:1 Conversation
What is the root cause of that problem?
We are returning the task created message here without checking the status of the task here
https://github.com/Expensify/App/blob/4a25c36e50b58ef9e5ba8b63cf32132f95a0b005/src/libs/OptionsListUtils.ts#L617-L618
What changes do you think we should make in order to solve the problem?

We should return deleted task copy for deleted task as we do in taskPreview
 else if (ReportActionUtils.isCreatedTaskReportAction(lastReportAction)) {
        const taskReportID = ReportActionUtils.isAddCommentAction(lastReportAction) ? ReportActionUtils.getOriginalMessage(lastReportAction)?.taskReportID?.toString() ?? '-1' : '-1';
        lastMessageTextFromReport = ReportUtils.isCanceledTaskReport(ReportUtils.getReport(taskReportID), lastReportAction)
            ? Localize.translateLocal('parentReportAction.deletedTask')
            : TaskUtils.getTaskCreatedMessage(lastReportAction);

What alternative solutions did you explore? (Optional)
We can take the logic to getTaskCreatedMessage function.

Edited by proposal-police: This proposal was edited at 2024-10-17 17:01:26 UTC.
Proposal
Please re-state the problem that we are trying to solve in this issue.
LHN - The deleted task content shown on LHN for 1:1 Conversation
What is the root cause of that problem?
We are returning the task created message here without checking the status of the task here
https://github.com/Expensify/App/blob/4a25c36e50b58ef9e5ba8b63cf32132f95a0b005/src/libs/OptionsListUtils.ts#L617-L618
What changes do you think we should make in order to solve the problem?

We should return deleted task copy for deleted task as we do in taskPreview
 else if (ReportActionUtils.isCreatedTaskReportAction(lastReportAction)) {
        const taskReportID = ReportActionUtils.isAddCommentAction(lastReportAction) ? ReportActionUtils.getOriginalMessage(lastReportAction)?.taskReportID?.toString() ?? '-1' : '-1';
        lastMessageTextFromReport = ReportUtils.isCanceledTaskReport(ReportUtils.getReport(taskReportID), lastReportAction)
            ? Localize.translateLocal('parentReportAction.deletedTask')
            : TaskUtils.getTaskCreatedMessage(lastReportAction);

What alternative solutions did you explore? (Optional)
We can take the logic to getTaskCreatedMessage function.
function getTaskCreatedMessage(reportAction: OnyxEntry<ReportAction>) {
    const taskReportID = reportAction?.childReportID ?? '-1';
    const taskTitle = getTaskTitle(taskReportID, reportAction?.childReportName);

    if (isCanceledTaskReport(getReport(taskReportID), reportAction)) {
        return Localize.translateLocal('parentReportAction.deletedTask');
    }

Edited by proposal-police: This proposal was edited at 2024-10-17 17:01:26 UTC.
Proposal
Please re-state the problem that we are trying to solve in this issue.
LHN - The deleted task content shown on LHN for 1:1 Conversation
What is the root cause of that problem?
We are returning the task created message here without checking the status of the task here
https://github.com/Expensify/App/blob/4a25c36e50b58ef9e5ba8b63cf32132f95a0b005/src/libs/OptionsListUtils.ts#L617-L618
What changes do you think we should make in order to solve the problem?

We should return deleted task copy for deleted task as we do in taskPreview in getTaskCreatedMessage
https://github.com/Expensify/App/blob/4a25c36e50b58ef9e5ba8b63cf32132f95a0b005/src/libs/TaskUtils.ts#L54-L56
function getTaskCreatedMessage(reportAction: OnyxEntry<ReportAction>) {
    const taskReportID = reportAction?.childReportID ?? '-1';
    const taskTitle = getTaskTitle(taskReportID, reportAction?.childReportName);

    if (isCanceledTaskReport(getReport(taskReportID), reportAction)) {
        return Localize.translateLocal('parentReportAction.deletedTask');
    }

What alternative solutions did you explore? (Optional)
We can take the logic to getTaskCreatedMessage function.

Edited by proposal-police: This proposal was edited at 2024-10-17 17:01:26 UTC.
Proposal
Please re-state the problem that we are trying to solve in this issue.
LHN - The deleted task content shown on LHN for 1:1 Conversation
What is the root cause of that problem?
We are returning the task created message here without checking the status of the task here
https://github.com/Expensify/App/blob/4a25c36e50b58ef9e5ba8b63cf32132f95a0b005/src/libs/OptionsListUtils.ts#L617-L618
What changes do you think we should make in order to solve the problem?

We should return deleted task copy for deleted task as we do in taskPreview in getTaskCreatedMessage
https://github.com/Expensify/App/blob/4a25c36e50b58ef9e5ba8b63cf32132f95a0b005/src/libs/TaskUtils.ts#L54-L56
function getTaskCreatedMessage(reportAction: OnyxEntry<ReportAction>) {
    const taskReportID = reportAction?.childReportID ?? '-1';
    const taskTitle = getTaskTitle(taskReportID, reportAction?.childReportName);

    if (isCanceledTaskReport(getReport(taskReportID), reportAction)) {
        return Localize.translateLocal('parentReportAction.deletedTask');
    }

What alternative solutions did you explore? (Optional)

Hmm, this is a very niche flow. I don't think it's going to be common that tasks are assigned within a group chat to someone not in the group chat and then deleted. Even then, I would argue there is minimal impact to this. Closing, as I don't think this is something we should be prioritizing changing.


/run is mounted as tmpfs by default

Description
When using default entrypoint (from Dockerfile) /run is mounted by default as a tmpfs which interferes with custom mounts like /var/run/mysqld which points to /run/mysqld.
This does not happen if one overrides the entrypoint. see below for output.
Steps to reproduce the issue:

git clone --single-branch -b run-creation-bug<EMAIL_ADDRESS>run-creation-bug
cd run-creation-bug; docker build -t run-creation-bug .
docker run --rm -ti run-creation-bug mount | grep run

/dev/mapper/fedora-root on /run/secrets type ext4 (rw,relatime,seclabel,data=ordered)
tmpfs on /run type tmpfs (rw,nosuid,nodev,noexec,relatime,context="system_u:object_r:svirt_sandbox_file_t:s0:c232,c383",size=65536k,mode=755)

Describe the results you received:
One can see there is a tmpfs mounted on /run
Describe the results you expected:

docker run --rm -ti --entrypoint=mount run-creation-bug | grep run

/dev/mapper/fedora-root on /run/secrets type ext4 (rw,relatime,seclabel,data=ordered)

And here there is no tmpfs mounted on /run which i personally prefer as expected behaviour
Output of docker version:
Client:
 Version:         1.10.3
 API version:     1.22
 Package version: docker-1.10.3-50.gita612434.fc24.x86_64
 Go version:      go1.6.3
 Git commit:      a612434/1.10.3
 Built:
 OS/Arch:         linux/amd64

Server:
 Version:         1.10.3
 API version:     1.22
 Package version: docker-1.10.3-50.gita612434.fc24.x86_64
 Go version:      go1.6.3
 Git commit:      a612434/1.10.3
 Built:
 OS/Arch:         linux/amd64

Output of docker info:
ontainers: 20
 Running: 2
 Paused: 0
 Stopped: 18
Images: 353
Server Version: 1.10.3
Storage Driver: devicemapper
 Pool Name: fedora_home-docker--pool
 Pool Blocksize: 524.3 kB
 Base Device Size: 10.74 GB
 Backing Filesystem: xfs
 Data file:
 Metadata file:
 Data Space Used: 15.39 GB
 Data Space Total: 107.2 GB
 Data Space Available: 91.76 GB
 Metadata Space Used: 8.049 MB
 Metadata Space Total: 541.1 MB
 Metadata Space Available: 533 MB
 Udev Sync Supported: true
 Deferred Removal Enabled: true
 Deferred Deletion Enabled: true
 Deferred Deleted Device Count: 0
 Library Version: 1.02.122 (2016-04-09)
Execution Driver: native-0.2
Logging Driver: journald
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.7.2-201.fc24.x86_64
Operating System: Fedora 24 (Workstation Edition)
OSType: linux
Architecture: x86_64
Number of Docker Hooks: 2
CPUs: 8
Total Memory: 31.35 GiB
Name: snowball
ID: KI4M:XPJ4:QYQK:OPUL:SMQW:KKGQ:J3QF:IJUJ:5N2N:YCLX:CBO5:KFFB
Registries: docker.io (secure)

Additional environment details (AWS, VirtualBox, physical, etc.):
selinux is Enforcing

Docker does not mount /run as tmpfs.
It looks like you are using a patched version of Docker and not the official docker release. I suggest you raise the issue with them (looks most like RedHat's patch).
ping @runcom
Closing since this isn't a Docker issue.
Please let me know if you need any help.
You can get the official docker release from https://get.docker.com.

You are mounting a volume at /run/secrets which is
a path that does not exist, so Docker will make a tmpfs mountpoint.
Simpler reproduction is:
moby:~# docker run --rm  -v $PWD:/nowhere/banana alpine mount | grep nowhere
tmpfs on /nowhere/banana type tmpfs (rw,relatime)

The reason you are mounting a volume is you are using a Red Hat patched version of docker which is doing this for you.
If you make the /run path in your image it will not be a tmpfs.

@justincormack you are wrong:
# docker run --rm  -v $PWD:/nowhere/banana alpine mount | grep nowhere
/dev/mapper/fedora_home-home on /nowhere/banana type ext4 (rw,seclabel,relatime,data=ordered)

also the /run folder exists in my image:
docker run --rm -ti run-creation-bug ls /run -lad
drwxr-xr-x. 2 root root 40 Sep 13 13:21 /run

docker run --rm -ti --entrypoint=ls run-creation-bug /run -lad
drwxr-xr-x. 4 root root 45 Sep 13 12:26 /run

/run exists because the Red Hat version of Docker creates it. I suggest
testing on the upstream version to see how Docker is supposed to behave and
file an issue with Red Hat if their version is causing you an issue.
On 13 Sep 2016 2:23 p.m., "Ulf Seltmann"<EMAIL_ADDRESS>wrote:

@justincormack https://github.com/justincormack you are wrong:
docker run --rm  -v $PWD:/nowhere/banana alpine mount | grep nowhere
/dev/mapper/fedora_home-home on /nowhere/banana type ext4 (rw,seclabel,relatime,data=ordered)
also the /run folder exists in my image:
docker run --rm -ti run-creation-bug ls /run -lad
drwxr-xr-x. 2 root root 40 Sep 13 13:21 /run
docker run --rm -ti --entrypoint=ls run-creation-bug /run -lad
drwxr-xr-x. 4 root root 45 Sep 13 12:26 /run
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker/issues/26528#issuecomment-246679093,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdcPOXpi27hEoHdNlF0Qu_v4lnztsyeks5qpqOugaJpZM4J7nDJ
.

after more testing i figured out its the name of the entrypoint script. because its name is init  the /run folder is mounted as tmpfs. after renaming init to buggy all worked as expected.

@useltmann Again, this is not docker doing this.
You should check with RedHat.

@cpuguy83, @justincormack I'm taking care of this in a bugzilla. @useltmann please continue the conversation over there.

Thanks!


[Identity] Remove vcrpy recordings

Description
A few YAML recordings remained after Identity was migrated to use the test proxy, so this removes them in anticipation of vcrpy's deprecation. The corresponding tests are manual-only, so this won't affect test run outcomes.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Keep alive associated AMQP links via Service Bus management requests.

Adds the "associated-link-name" parameter to management requests so that they actually keep the primary sender/receiver link alive as well.
Prior to this change, links would be force disconnected by the service after 10 minutes idle, even if autorenew/keep-alive were present.
Also contains a stress test to validate this scenario before and after the change.  (Did a stress test in leiu of a unit test because the failure requires a 10 minute wall-time delay to trigger the service force disconnect)
Should address #10127 so long as user enables an AutoLockRenew to be running (e.g. against the message in question) or calls renew() themselves within the expiry window.

Can one of the admins verify this patch?


analyze produces a broken output html report

Describe the bug
When I use analyze to create an output.html report a non-functional HTML file is created.
The same result happens with the command line on macOS and Ubuntu 20.04 LTS. I used the .NET SDK and attempted to create the report from a directory that wasn't inside of the deployed application folder by using the dotnet tool.
To Reproduce
Steps to reproduce the behavior:
$ dotnet --list-sdks
3.1.401 [/usr/share/dotnet/sdk]

$ dotnet tool install --global Microsoft.CST.ApplicationInspector.CLI
Since you just installed the .NET Core SDK, you will need to logout or restart your session before running the tool you installed.
You can invoke the tool using the following command: appinspector
Tool 'microsoft.cst.applicationinspector.cli' (version '1.2.58') was successfully installed.

$ appinspector
ApplicationInspector.CLI 1.2.58+6e9480b37c
© Microsoft Corporation. All rights reserved.

ERROR(S):
  No verb selected.

  analyze        Inspect source directory/file/compressed file (.tgz|zip) against defined characteristics

  tagdiff        Compares unique tag values between two source paths

  tagtest        Test (T/F) for presence of custom rule set in source

  exporttags     Export unique rule tags to view what code features may be detected

  verifyrules    Verify custom rules syntax is valid

  packrules      Combine multiple rule files into one file for ease in distribution

  help           Display more information on a specific command.

  version        Display version information.

$ pwd
/home/rbackley/Documents/ms_application_inspector/output

$ appinspector analyze -s ../code/cyclonedx-python/
The output file does not have an html or htm extension and may not open properly in your browser
Analyze command running
100% source files processed
Unable to launch output.html automatically.  Set the BROWSER environment variable to your desired browser and try again or launch your browser and navigate to the file to view the report file manually.
Analyze command completed
Additional details may be found in log file at /home/rbackley/.dotnet/tools/.store/microsoft.cst.applicationinspector.cli/1.2.58/microsoft.cst.applicationinspector.cli/1.2.58/tools/netcoreapp3.1/any/appinspector.log.txt

$ ls -l
total 172
-rw-rw-r-- 1 rbackley rbackley 152694 Sep  2 08:21 output.html
-rw-rw-r-- 1 rbackley rbackley  16532 Sep  2 08:21 output.json

$ python3 -m http.server 8000
Serving HTTP on <IP_ADDRESS> port 8000 (http://<IP_ADDRESS>:8000/) ...
<IP_ADDRESS> - - [02/Sep/2020 08:23:48] "GET /output.html HTTP/1.1" 200 -
<IP_ADDRESS> - - [02/Sep/2020 08:23:48] code 404, message File not found
<IP_ADDRESS> - - [02/Sep/2020 08:23:48] "GET /favicon.ico HTTP/1.1" 404 -

After performing the above steps I opened the HTML in Chromium via Open File and via browsing to the local HTTP server created by the python line at the very end of the steps pasted above.
Expected behavior
I'd like a report in a single HTML file that functions like v1.2.53.
Screenshots
Please see attached.
Open File

Local HTTP server

Desktop (please complete the following information):

OS:

Ubuntu 20.04 LTS
macOS 10.15.6

Browser

Chromium 85.0.4183.83

Version

ApplicationInspector.CLI 1.2.58+6e9480b37c

Additional context
This is a follow-up to the question I raised on #273. It looks like I will need to go more in depth and I will open a new issue to track my specific issue.
I'll attach the actual output files in a zip, incase it helps. This is a scan of an open source tool and is safe to disclose.
Archive.zip

Looking into thanks.

Great report. I can repro this on Mac OS.

Thanks for attaching the zip.  Issue doesn't happen for straight release download or build but should have a fix ready very soon.

Your output is missing key .cs and .js files that are aggregated into the html.  Looking at how I'm building the path to see what is not working for non-Windows systems.

Windows specific file path for those missing files.  Testing fix now.


[UAVCAN/socketcan] Compilation error CanardFramehas no longer member timestamp_usec

This line causes a compile error when using with the latest version of libcanard.

See if this helps: #38

@aentinger would you like to volunteer some help with #38?

I might, but not prior to mid-june, as I'm currently pretty occupied with slapping together software for a certain event. It was during this slap-together process that this issue came to my notice but for now I'm fine just commenting the offending line out.


tough table to convert - comes out blank

ExtractPage1.pdf
any ideas what to try?

Thanks for the report @vniemeyer.
That PDF is particularly nasty. It seems to have been generated with multiple small images. Tabula will not be able to process it.

Thank you, I can believe it.  I appreciate you looking at it.
Best,Vic
  From: Manuel Aristarán<EMAIL_ADDRESS>
To: tabulapdf/tabula<EMAIL_ADDRESS>Cc: vniemeyer<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Sent: Tuesday, June 20, 2017 3:19 PM
Subject: Re: [tabulapdf/tabula] tough table to convert - comes out blank (#695)
Thanks for the report @vniemeyer.That PDF is particularly nasty. It seems to have been generated with multiple small images. Tabula will not be able to process it.—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Cozette: Init at 1.5.1

Motivation for this change
Package request: Cozette font #80267
Fixes #80267
cc @rnhmjoj @soundtoxin

I was looking into packaging this myself, but wanted to try building it from source. As you can probably tell, that didn't go according to plan, so thanks for picking this up!

I think it's fine not to build from source for a font. Given the package is simply copying files it could even be a fixed output derivation.


[Objective-C] Fix issues that cause unstable output

Sort properties that are passed to .classDecl
Sort set of possible classes in NSCoding initWithCoder (for decoding)
Fix order in which protocols are enumerated

Fixes #158

@jparise - Thanks for reviewing, I added a test script that will generate each languages output 100 times and diffs to ensure they are stable. This of course isn't a full-proof way to ensure this but should catch the majority of issues


Prepare for renaming master -> main

I will fill this out later.
Summary of the Pull Request
What is this about:
What is include in the PR:
How does someone test / validate:
Quality Checklist

[ ] Linked issue: #xxx
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

@dhowett The linked issue is #5433


postMessage does not work

Issue description:
Styles update via postMessage does not work. If you use a prefix for output, js produces an error
Version used:
3.0.9
WordPress 4.8
Using theme_mods or options?
theme_mods
Code to reproduce the issue (config + field(s))
Kirki::add_field( '5avenue_theme', array(
	'type'			 => 'color-alpha',
	'settings'		 => 'header_mobile_border_color',
	'label'			 => __( 'Header Border Color', 'kirki' ),
	'description'	 => __( 'Color of the border below the header.', 'kirki' ),
	'section'		 => '5avenue_header_mobile_section',
	'default'		 => '#e0e0e0',
	'priority'		 => 10,
	'transport'		 => 'postMessage',
	'output'	 => array(
		array(
			'element'	 => '#header-mobile.header-mobile--border-on',
			'property'	 => 'border-bottom',
			'prefix'		 => ' solid 1px',
		),
	),
	'js_vars'	 => array(
		array(
			'element'	 => '#header-mobile.header-mobile--border-on',
			'property'	 => 'border-bottom',
			'prefix'		 => ' solid 1px',
		),
	),
) );

Module postMessage output
wp.customize('header_mobile_border_color', function (value) {
	value.bind(function (newval) {
		if (null === document.getElementById('kirki-postmessage-header_mobile_border_color') || 'undefined' === typeof document.getElementById('kirki-postmessage-header_mobile_border_color')) {
			jQuery('head').append('<style id="kirki-postmessage-header_mobile_border_color"></style>');
		}
		newval0 = newval;
		var cssContent = '#header-mobile.header-mobile--border-on{border-bottom:' + solid 1px + newval0 + ';}';
		jQuery('#kirki-postmessage-header_mobile_border_color').text(cssContent);
		jQuery('#kirki-postmessage-header_mobile_border_color').appendTo('head');
	});
});

Where solid 1px return error

Hi @aristath
Also, It seems like we can't get setting to work on customizer.js.
While adding kirki controls using
Kirki::add_field( 'suffice_config', array(
	'type'		=> 'switch',
	'settings'		=> 'show_widget_title_ribbon',
	'label'		=> esc_html__( 'Widget Title Ribbon', 'sanjeev' ),
	'description'	=> esc_html__( 'Turn on to show ribbon on widget title.', 'sanjeev' ),
	'section'		=> 'section_widget',
	'priority'		=> 10,
	'transport'	=> 'postMessage',
	'default'		=> 1,
) );

And controlling it on customizer.js
wp.customize( 'show_widget_title_ribbon', function( value ) {
	value.bind( function( to ) {
		if ( true === to ) {
			jQuery( '.widget-area' ).find( '.widget' ).addClass( 'widget--ribbon' );
		} else {
			jQuery( '.widget-area' ).find( '.widget' ).removeClass('widget--ribbon');
		}
	} );
} );

I have checked my other elements too. Seems this is an issue. Can you confirm this?
Thanks

@BlackSiriuS try this:
Kirki::add_field( '5avenue_theme', array(
    'type'        => 'color-alpha',
	'settings'    => 'header_mobile_border_color',
	'label'       => __( 'Header Border Color', 'kirki' ),
	'description' => __( 'Color of the border below the header.', 'kirki' ),
	'section'     => '5avenue_header_mobile_section',
	'default'     => '#e0e0e0',
	'priority'    => 10,
	'transport'   => 'auto',
	'output'      => array(
        array(
            'element'  => '#header-mobile.header-mobile--border-on',
			'property' => 'border-bottom',
			'prefix'   => ' solid 1px',
		),
	),
) );

@sethstha I can't replicate your issue.
There was an unrelated bug in switch controls that may have had this side-effect, can you please pull the latest changes from this repository (develop branch) and let me know if your issue has been fixed?
Also, the issue you reported has absolutely no relation to the original issue reported in this ticket. In the future if the issue you're reporting is not the same as the one in the original thread, please avoid commenting and instead create a new ticket. Having multiple issues on the same thread doesn't help maintenance and keeping track of what went wrong, where it went wrong, and where it got fixed.

@BlackSiriuS it works fine if I do these changes to your code:

Remove the js_vars argument
Set transport to auto.

@BlackSiriuS my apologies, it looks like this was indeed a bug. I had it fixed locally but never pushed the fix in the repository. I just pushed it now, it should be fixed for you as well.
But the changes mentioned in my previous comment should be applied anyway.
Removing js_vars and setting transport to auto will clean up your code a lot and Kirki will automatically generate all js_vars using output as a template.
If you don't change the prefix from ' solid 1px' to '1px solid ' the CSS generated will be invalid.
Also, instead of the prefix and suffix arguments you can use the value_pattern argument.
So it would look like this: 'value_pattern' => '1px solid $'. You can use the $ symbol as a placeholder for the value.


[Broken] Unable to extract JS player URL

[X] I'm reporting a broken site support
[X] I've verified that I'm running youtube-dlc version 2020.10.26
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones

$ youtube-dlc -g -v https://youtu.be/QpQvJExilsk

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-g', '-v', 'https://youtu.be/QpQvJExilsk']
[debug] Loading archive file None
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dlc version 2020.10.26
[debug] Python version 3.8.5 (CPython) - Windows-10-10.0.17763-SP0
[debug] exe versions: ffmpeg 4.3.1, ffprobe 4.3.1
[debug] Proxy map: {}
ERROR: Unable to extract JS player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "c:\python38\lib\site-packages\youtube_dlc\YoutubeDL.py", line 830, in extract_info
    ie_result = ie.extract(url)
  File "c:\python38\lib\site-packages\youtube_dlc\extractor\common.py", line 532, in extract
    ie_result = self._real_extract(url)
  File "c:\python38\lib\site-packages\youtube_dlc\extractor\youtube.py", line 2065, in _real_extract
    jsplayer_url_json = self._search_regex(
  File "c:\python38\lib\site-packages\youtube_dlc\extractor\common.py", line 1010, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dlc.utils.RegexNotFoundError: Unable to extract JS player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.

This started occurring about two days ago and happens the majority of the time when trying to get the URL for a video.

Search before posting.

fixed in #35


Documentation

IS there any other documentation available other than the two sample projects?
Cheers

Hi, no unfortunately not. Feel free to ask questions and I will try and answer them as soon as possible. This is one of the reasons why I haven't published it as an npm module.

Ok, no probs. I haven't started writing anything yet - I'm just about to start. I might jot a few bits down as I go through then.

A documentation would be really nice. ;)
Sadly the examples don't really have comments either so it is very hard to understand this module.
I don't fully understand what is needed to connect to my sonos.
//[...] bootstrapping code
var discovery = new SonosDiscovery({port: 8080});
console.log(discovery.players);

returns {}

I'm currently rewriting it completely from scratch. Doesn't make sense for me to document it right now since it will be completely different in the next version.
To give you an answer to your problem, you need to wait for the system to discover the players before you'll actually be able to interact with it. If you do like this:
var discovery = new SonosDiscovery();
discovery.on('topology-change', function () {
  console.log(discovery.players);
})

I understand that it isn't super intuitive, the next version will be promised based and a bit more straight forward.

yes I saw a note about the rewrite
do you have the source somewhere (sneak peek)?
any idea when an alpha is available?
I guess it makes little sense to work with this module?
I played around and tried to understand your webclient but gave up.
thx for the snippet, that helps to get started at least.
I am writing a controller for my apartment with different modules and the audio part can wait a bit.
I might just delay it a few weeks and see your progress.

Yes, I have a branch up to date: https://github.com/jishi/node-sonos-discovery/tree/refactor-with-promises
The basic functionality is there, but the more advanced feature aren't implemented. Do note that the branch still contains the old version as well. You can check the test/integration files for implementation examples.

kk will do. thx a lot for your hard work on this.

just played with the new version with success ;) good enough for my first tests


templates

implement twig or latte template system

Added Twig 1.3. The CMS itself makes no use of it, but it's available to plugins.


Virtual Network overview should mention region limitation

Currently, we have to read the entire Virtual Network section and get to the FAQ page before it's actually said that the Virtual Network is limited to a single region. On the other hand, it's mentioned multiple times in multiple locations that VNets can be peered across regions.
The fact that VNets are limited to a single Region should be made clear upfront in the overview, as it's a very core concept.

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

ID: f03f79a2-923f-3e1a-5e8f-8ce1f5913a56
Version Independent ID: 6330e3e6-d903-b8f8-370d-e61ff4384d5e
Content: Azure Virtual Network
Content Source: articles/virtual-network/virtual-networks-overview.md
Service: virtual-network
GitHub Login: @jimdial
Microsoft Alias: jdial

@cflee Thank you for the feedback!  We are investigating this and will get back to you shortly.

100% agree with @cflee. This core fact that should be mentioned upfront.

Hi @cflee
I am helping out with Jim Dial's work items while he's out of the office for a bit. I have PM approval for this; we're working on wording.

Hi @cflee
The reference is now in the first paragraph of the virtual networks overview as you requested. Thanks for the great suggestion!
@kobulloc-MSFT
#please-close


Consider generate sourcemap file like *.js.map ?

This is of debugger development.
Parsing sourcemap string in lua file may be kind of unstable?

Source maps should already work, at least partially. See #237.


Schema for landscape.yml and settings.yml

This is a feature request to add a json schema for the landscape.yml and settings.yml files which will have several benefits:

Code editor fields hinting and documentation
Code editor real time fields validation

Thanks @trongthanh, this would be a nice addition. We'll take care of it 👍


Physician Note Examples: more examples that are realistic but not PHI

TODO: add synthetic examples.

FYI there are sample notes in the ctakes-examples module. We could either
pull into this repo with proper credit or point to them if possible.
On Fri, Sep 23, 2022, 9:04 AM AndyMc @.***> wrote:

TODO: add synthetic examples.
—
Reply to this email directly, view it on GitHub
https://github.com/Machine-Learning-for-Medical-Language/ctakes-client-py/issues/14,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABXODL6DFB5JUHAWYJJGHLV7WTFDANCNFSM6AAAAAAQT646CM
.
You are receiving this because you are subscribed to this thread.Message
ID: <Machine-Learning-for-Medical-Language/ctakes-client-py/issues/14@
github.com>


Error when platform has bcmath but not gmp extension

@digitaldonkey digitaldonkey
If I run the sample code in Readme.md it causes the following error if the server/platform running the code does not have the php-gmp extension installed, but does have php-bcmath installed.
The dependency simplito/elliptic-php  requires either gmp or bcmath to be installed as you can see in BigInteger.php.
First it tries to do the operation using gmp but if gmp is not installed it uses bcmath as a fallback.
This problem can be fixed (within this repo) by removing the '0x' that is prefixed to the $message_hash in EcRecover.php on line 20.
So this line
$message_hash =  '0x' . Keccak::hash(self::personalSignAddHeader($message), 256);

changes to this
$message_hash = Keccak::hash(self::personalSignAddHeader($message), 256);

Once that change has been implemented the sample code in Readme.md runs fine using both the gmp and bcmath methods.
I'm not sure if the proper solution here is for this repo to be modified as described above or if simplito/elliptic-php should somehow be modified instead. However if you look at the the README.md for simplito/elliptic-php under "Verifying Ethereum Signature" you can see that the example does not prefix the hash with '0x'.
For reference here is the exact code I am running that causes an error. This is the same code from Readme.md, just with a couple typo fixes.
        use Ethereum\EcRecover;

        $address = '0xbe93f9bacbcffc8ee6663f2647917ed7a20a57bb';
        $message = 'hello world';
        $signature = '0xce909e8ea6851bc36c007a0072d0524b07a3ff8d4e623aca4c71ca8e57250c4d0a3fc38fa8fbaaa81ead4b9f6bd03356b6f8bf18bccad167d78891636e1d69561b';

        // Verify known address
        $valid = EcRecover::personalVerifyEcRecover($message,  $signature,  $address);

        // Recover unknown address
        $recoveredAddress = EcRecover::personalEcRecover($message, $signature);
        if ($recoveredAddress === $address) {
          echo 'Jay! it was a long way here. ';
        }

This is the error I am getting:
Exception
Invalid characters
vendor/simplito/bigint-wrapper-php/lib/BigInteger.php:328

@brianrlewis Thank you for your contribution. I'm having very little time for this. I leave it open for reference, or till someone provides a PR and I find some time to get back into it.


Is it safe to save models as instance state (eg: with Android Annotations)

DBFlow Version: 4.0.0-beta5
Issue Kind (Bug, Question, Feature): Question
Is it safe to save models in onSaveInstanceState() assuming all fields are either basic types (String, long, etc.) or Parcelable (via https://github.com/johncarl81/parceler)?
Description:
The reason I ask is that I occasionally get the following error:
E/SQLiteQuery: exception: no such table: Country (code 1);

I understand that this usually occurs when the model(s) change and the version number isn't incremented. However, this isn't the case for me. The model hasn't been updated. In addition, the error doesn't occur all the time. These models have foreign keys and I'm wondering if trying to load the foreign model when the instance is restored is a problem.

I would probably avoid such a thing unless your models are stable. It depends, what does your model structure look like? Where does this exception occur, like in a traceable stacktrace to your app, or inside parcel restoring somewhere?


Window Size / Sizing

Description of the new feature / enhancement
Add a new tool that adds options to an application window to get the size of the window AND to set the size of a window.
It would be like the Chrome plugin Window Resizer, but available to all Windows windows.  See https://chromewebstore.google.com/detail/window-resizer/kkelicaakdanhinjdeammmilcgefonfh?pli=1
Scenario when this would be used?
When working with multiple windows and you want to see / set their size.
Supporting information
No response

/dup #11577


[Question] VM image management for productive environment: best practices

What are best practices and recommendations for VM image management? (Assume several OS options and regular image updates with latest OS versions.) If we create new images for each new OS build, then there might be a huge number of images that become unmanageable. If we override an image with a new build, we might get unpredictable behavior on the downstream cluster level. Any guidance you could provide would be very much appreciated.

Please consider the following enhancement on the UI side:

[ ] filtering images by labels https://github.com/harvester/harvester/issues/2319 - on the image page.
[ ] when creating a guest k8s cluster, allowing users to search images by name and sort it in either ascending or descending.

Please consider the following enhancement on the UI side:

[ ]  filtering images by labels [FEATURE] Image filtering by labels #2319 - on the image page.
[ ]  when creating a guest k8s cluster, allowing users to search images by name and sort it in either ascending or descending.

duplicated ， The problem has been solved in the issue， so close this issue


When listening to interactive_message_callback, trigger.original_message.text is blank

I'm following the example for adding interactive buttons to my response in my slackbot. This is the workflow:

User posts a message in the channel.
Message contains a trigger that fires a controller.hears call.
controller.hears call starts a thread, and triggers a response with a few buttons.
user clicks button
interactive_message_callback event is triggered

I'm using logic similar to this to catch when one of those buttons is clicked:
	controller.on('interactive_message_callback', function(bot, trigger) {
		console.log("interactiveMessageCallback: ", trigger);

		var reply = trigger.original_message;
		console.log("original_message: ", reply);
		return true;

	});

I can catch and successfully handle button presses, including editing the original response from the bot in the slack thread. However, I lose access to the original text that triggered that response in the first place. I'd like to use that to copy that thread-starting post to another system.
I saw there's a "text" parameter on the original_message parameter, but that's always blank for some reason?

Is that expected? I thought this text parameter would have the full message that triggered my bot to react in the first place, the start of this thread.
I've looked through all the parameters I could see from the scope of the interactive_message_callback callback. I'm open to ideas.

I misunderstood the point of the original_message parameter. I badly assumed it was the message that caused my bot to ask a question, where in reality it's a pointer to the question my bot asked, which triggered this reply.
In this scenario, the question my bot asked only contains attachments, no plain text value. That's why attachments is populated and "text" is not. Not an issue, rather a misunderstanding!


Don't work with vk.com and Nightbot.

Hello. As I wrote will not work with vk.com and bot.
I would like to have a function like !music in chat Twitchy , and the bot to give information about the music that is playing right now.

I changed my vk.com to Russian and it still works for me.
So the first possibility I see is that you don't have the extension up to date, support for vk.com was added on version v0.2.0. So check that you have version v0.2.x.

Yep. Today my firefox updated on 0.22. And all work. thx for info
24.04.2015, 18:28, "Paco Hobi"<EMAIL_ADDRESS>
I changed my vk.com to Russian and it still works for me.
So the first possibility I see is that you don't have the extension up to date, support for vk.com was added on version v0.2.0. So check that you have version v0.2.x.
—
Reply to this email directly or view it on GitHub.

Thank you for using CurrentSong, if you have any other issue or suggestion feel free to contact me :+1:


support "workspace/didChangeWatchedFiles"

At least Metals LSP server would need such a feature, see https://github.com/scalameta/metals/issues/533
Any idea on how such a feature would be implemented given the sublime API ?

This is impossible with the current ST3 api.

FWIW, Metals has its own file watcher for critical functionality but due to API limitations we delegate workspace/didChangeWatchedFiles notifications to the editor client for nice-to-have but non-essential features like detecting build changes by background processes.

Might be practical to write a small Sublime package that hooks workspace/didChangeWatchedFiles from LSP and uses watchman to service that API.
The LanguageHandler API is a bit too specific, this package could do it for any language, so we could expose a new API for hooking in these kind of services.


Spelling of April in the picker

Thx. fixed.
but that's only a demo. please use your language texts

Sure, just noted so informed :)


volume: resolve symlink paths in volumes

Closes: https://github.com/containers/libpod/issues/1608

this is a quite important fix to have I think, but unfortunately it is still blocked on the Buildah PR

vendored a new buildah

F28 is CGroups flake

bot, retest this please

Did vndr hiccup? It looks like we're bringing in a lot more of buildah than before

good question @mheon, there's a lot of Buildah code being dragged in with this that I'd not expect to see, pull, commit, images, etc....

Did vndr hiccup? It looks like we're bringing in a lot more of buildah than before

yes, something went wrong.  I've ran it again and now there are less files changed

/lgtm


Ben what are the slip components for cutde?

https://github.com/brendanjmeade/celeri/blob/9164da11e1431ed4f8cadf1798eced5748fb7e2d/celeri.py#L1671-L1671

@tbenthompson This is a clarification issue.  What is the default coordinate system that cutde assumes for sense of slip on faults?  Is it Cartesian x, y, z, or strike-, dip-, tensile-slip?  Something else?

The slips parameter is a numpy array with shape (N, 3) where slips[:,0] is the strike slip component, while component 1 is the dip slip and component 2 is the tensile/opening component.

Thanks!  That's fantastic and makes life really easy because it saves a reference frame conversion!


How can I get the user making the request?

Let's say I have the ParseServer mounted at /parse. Is there a way of getting the logged in user outside that route?
If a user makes a request to /foo, how can I get which user made the request?
app.get('/foo', function(req, res) {
  // how to get the user here?
});

req.user.

Are you sure about this? It's not working. As I said, ParseServer is mounted at /parse, my /foo route is just a simple express endpoint.
Is there any way of asking ParseServer which user is making a request based on the req received?

Ah, sorry, I missed that. If your request contains the users session token, you can use that to get the user. Check the headers of the request.

I just cloned the repo, and made a few changes on the test page to test the authentication:
First I added Parse JS SDK to the page and added a button to authenticate and another to test:

Authenticate
Test

Then I initialized Parse:
Parse.initialize('myAppId');
Parse.serverURL = 'http://localhost:1337/api/v1';

Authenticate on button click:
this.bindBtn('#step-5-btn', function(e) {
  Parse.User.logIn('username', 'password').then(function(data) {
    alert('ok');
  });
  e.preventDefault();
});

Make a request to test it:
this.bindBtn('#step-6-btn', function(e) {
  XHR.GET('/foo');
  e.preventDefault();
});

And created the route on server:
app.get('/foo', function(req, res) {
  console.log(req.user);
  console.log(req.headers);
  res.status(200).send('ok');
});

And there's where I can get the user. No user, no session, no header.
I already know that Parse saves the user data on localStorage and not cookies, so I can't just get them server side.
I wonder what's the "right" way of doing this kind of stuff.

Probably the right way to this will be to use a Cloud Code function, and use the Parse SDK to call the function.

I get it, but this way my whole express server will be of no use, since I'll have to write all the code on Cloud Code "sub-server". Is that right?

Yes, part of the magic of Parse is that you don't have to think about headers and user sessions and so on, as the Parse SDK handles that for you. You can still do everything manually if you want using the REST API, and sometimes there are good reasons to do it that way, but it will be more work.
You might be able to try using express routes that override some Parse Server routes, like this:
app.post('/your_parse_mount/functions/foo', function(req, res) {...})

And then use the Parse SDK or REST API to call the function, that way you will have access to all the info the Parse SDK sends across, but in an express route. None of us have tried that, though, so if you can make it work, let us know how you did it and if you ran into any issues.


[Failing Test]: StreamingDataflowWorkerTest.testMaxThreadMetric looks flaky

What happened?
I've seen StreamingDataflowWorkerTest.testMaxThreadMetric failing a couple of time on Jenkins and suspect it's flaky, particularly as it was just added very recently.
Issue Failure
Failure: Test is flaky
Issue Priority
Priority: 2 (backlog / disabled test but we think the product is healthy)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[X] Component: Google Cloud Dataflow Runner

.take-issue

I have created a PR for this issue: https://github.com/apache/beam/pull/28173.

.close-issue


stream doesn't work when "--use-fake-ui-for-media-stream" argument is passed

I need to record a page which needs access to webcam and mic. After passing "--use-fake-ui-for-media-stream", stream is not working and recording results in a zero byte page.
example
const file = fs.createWriteStream(__dirname + "/test.webm");

async function test() {
	const browser = await launch(puppeteer, {
		defaultViewport: {
			width: 1920,
			height: 1080,
		},
		args:[
			"--use-fake-ui-for-media-stream"
		]
	});

	const page = await browser.newPage();
	await page.goto("PAGE-THAT-NEEDS-MEDIA-PERMISSIONS");
	const stream = await getStream(page, { audio: true, video: true });
	console.log("recording");

	stream.pipe(file);
	setTimeout(async () => {
		await stream.destroy();
		file.close();
		console.log("finished");
	}, 1000 * 10);
}

test();

Workaround
We can use overridePermissions instead of the argument --use-fake-ui-for-media-stream
const context = browser.defaultBrowserContext()
await context.overridePermissions("ORIGIN_OF_TEST_PAGE", ['camera', 'microphone'])

Any ideas how to fix that ?

Closing Issue as I think the workaround is the way to go.

You are saviour man


i18n(ja): Add install-and-setup.mdx

Description (required)

Add install-and-setup.mdx.

Related issues & labels (optional)

Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ja
install-and-setup.mdx
Localization added, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


feat: updating sdk to latest

✨ This PR updates the SDK to tag latest

Generated only for tests purposes.


Replace exponential operator with Math.pow as it causes errors

Becouse after npm start it throws:
2 ** 16

SyntaxError: Unexpected token *
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:542:28)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.runMain (module.js:604:10)
    at run (bootstrap_node.js:383:7)
    at startup (bootstrap_node.js:149:9)

would be better to switch it to Math.pow

@timneutkens please can you release a new patch version with this? 🙏


Make application secrets more flexible

Currently there's one way of providing secrets to honeypot - a file titled config.yml in the same directory as the binary. This is obviously non-ideal.
Options (probably all of them are ideal)

Add env var specifying config file location override
Add env-var-based config overrides: LEVEL1_LEVEL2_VARNAME
Add default config.yml to the docker build

Added default config to docker build and env-var-based config file override. It's working well.

The second one is genius (thanks benthos), but will come back to implementing it later as asked.


Does not allow for ses-Session directories

As-is, the run.py will look for T1 images with "ses-" in the filename, but the directory structure is still:
BIDS/sub-#/anat/sub-#_ses-# rather than
BIDS/sub-#/ses-#/sub-#_ses-#

FWIW, the fix is as simple as having line 19 in run.py change from:
glob(join(basedir,"sub-%s"%subject_label, "anat", "sub-%s_ses-*_T1w.nii.gz" % (subject_label)))
to
glob(join(basedir,"sub-%s"%subject_label, "ses-*", "anat", "sub-%s_ses-*_T1w.nii.gz" % (subject_label)))


coc-prettier causes cursor to jump in Neovim

Problem: coc-prettier version 1.1.17 on saving JS files causes cursor to annoyingly jump to by 2 lines up.
Reproduction:
To reproduce i faithfully follow both https://github.com/neoclide/coc-prettier and https://prettier.io/docs/en/vim.html manuals to install prettier in Vim/Neovim:

Install coc-prettier:
a) Neovim: CocInstall coc-prettier

Insert into init.vim:
a) Plug 'neoclide/coc.nvim', {'branch': 'release'}
b) command! -nargs=0 Prettier :call CocAction('runCommand', 'prettier.formatFile')

Insert into coc-settings.json:
{
"coc.preferences.formatOnSaveFiletypes": [
"javascript",
"javascriptreact",
"typescript",
"typescriptreact",
"css",
"markdown",
"json",
],
}

Execute:
a) Neovim: source $MYVIMRC
b) Neovim: PlugInstall
c) Neovim: PlugUpdate
d) Neovim: UpdateRemotePlugins
e) Neovim: Coc-Rebuild
f) Neovim: Coc-Restart
g) Neovim: Coc-Update

Restart Neovim:
a) Neovim: w!
b) Neovim: checkhealth            (OK on all tests)
Observation: After saving the buffer each time the cursor moves by 2 lines up.

Reverse the Problem described in #4):
a) Neovim: ~/.config/coc/extensions/package.json>Remove coc-prettier from within package.json file.
b) bash> rm -rf ~/.config/coc/extensions/node_modules/coc-prettier
c) Neovim: CocUninstall coc-prettier
Observation: After saving the buffer the cursor remains fixed in same position in buffer.

Let me know if the above helps.

Can't reproduce, try minimal vimrc and debug coc.nvim https://github.com/neoclide/coc.nvim/wiki/Debug-coc.nvim


os: FileInfo.Sys should return underlying data source on Windows

The documentation of the Sys method for the FileInfo interface says:
"underlying data source (can return nil)"
However the Windows implementation does not do this.
The underlying data source for the FileInfo implementation in the Stat function is
syscall.ByHandleFileInformation, however the Sys method returns syscall.Win32FileAttributeData.
The underlying data source for the FileInfo implementation in the Readdir method is syscall.Win32finddata, however the Sys method returns syscall.Win32FleAttributeData.
The advantage of the current implementation is that Sys returns the same dynamic type, as it is done on Unix systems.  However the disadvantage is that some valuable information (VolumeInformation and FineIndex from the Stat call) is thrown away.  The current implementation also has to allocate additional space.
Since the interface documentation seems to allow it, I propose that the Sys method returns the actual underlying data used by the implementation, ad discussed above.
The change does not break the go 1 compatibility promise.

@ianlancetaylor please assign it to me, if you want me to try and implement it. Tank you.
Alex

I don't have an opinion on whether we should do this or not.  I'll assign it to you so that you can decide.

I still think this is a API change, although a less obvious one.

I don't think we should be making this change. We don't want to break people's code. Closing this as "unfortunate".
https://groups.google.com/forum/#!topic/golang-dev/Kky-9-TrcWY for full discussion.
Alex

On Sun, Jan 18, 2015 at 6:43 AM, alexbrainman<EMAIL_ADDRESS>wrote:

I don't think we should be making this change. We don't want to break
people's code. Closing this as "unfortunate".

It's unfortunate, since this is a private API and the go fix tool should be
able to upgrade the old code.


[Golang] Create API with Router-Controller pattern in Gin Swagger

Create an API using the Router-Controller pattern  in Golang with the Gin framework and integrate Swagger documentation for the API.

this was closed in #10


New version 1.0.1 not working for me

I pull a new version today, not working for me https://github.com/oblador/angular-scroll/releases/tag/v1.0.1
When I go back to v1.0.0 all works like a charm.

Same for me - scrolling to anchors does not work silently. Downgrade to 1.0.0 solved the problem

Same here!

Really sorry about this! 1.0.2 is out that should fix this, try it out!

I use 1.0.2
$document.scrollToElementAnimated(el, offset, duration);
not working. When I go back to v1.0.0 all works.
Thx for pluging! Try to fix again)


Change dependency version from "*" to hardcode

After run command composer update, composer change doctrine to new (major bc break ) version. In my composer.json orm-pack is hardcode version.

Related
#17
#18

unpack the pack
that's now done automatically for newly installed packs (update flex of course)


Master

A few typo fixes. (The picture corrections were because the file would not run with how it was)

I'm not sure why, by this merge request has created a conflict again. It might be to do with your local repo not being in sync when you submitted the pull request.
The conflicts look like:
**<<<<<<< master
%	{x^(1/2)};
%	\addlegendentry{$\sqrt{N}$}
{x^(1/2)};
\addlegendentry{$\sqrt{N}$}

master**

Should I resolve these to things that look like the top option or the bottom one?


fix(js_semantic): ignore this in JSX components

Summary
Fix #2636
@printfn submitted #2656 that fixed the issue by modifying the grammar and the parser.
This PR supersedes #2656. Instead of modifying the grammar and the parser, it adds an exception in the semantic model.
We already used this strategy to ignore typeof this.foo.
I credited @printfn in the CHANGELOG for its investigations and the submitted PR :)
Test Plan
I added a non regression test.

Thank you for fixing this!


updated links in Documentation

Coverage remained the same at 99.355% when pulling ba3af56c2348687e1da37edc92b5d6999d1f6089 on links into b9c89ddf94524001cbd8fd3630fcba3b24e291b6 on master.


Add tests for functionality in HistomicsTK core

Read the following two links to learn how to write tests for girder and its plugins:

http://girder.readthedocs.io/en/latest/developer-cookbook.html#adding-automated-tests
http://girder.readthedocs.io/en/latest/plugin-development.html#automated-testing-for-plugins

I wrote a test for color normalization.
The add_python_test( my_test EXTERNAL_DATA plugins/cats/test_data.txt ) cmake function supports automatic download of test data for MIDAS only. I created an issue on the girder repo for implementing a mechanism to use data from data.kitware.com (or any girder instance) for writing tests. While this is being developed, i am downloading the test data over http.
I created a collection for HistomicsTK on data.kitware.com.

@cooperlab @slee172 All the testing issues have now been fixed. We can start writing tests now. See the plugin_tests/color_normalization_test.py and plugin.cmake for an example.


Type of error, passed from retry filter is type unknown

Hi!
I use createJsonQuery with retry and now the error object passed to the filter is type unknown, but it looks like it should be narrowed to some error type
retry(basicQuery, {
  ...
  filter: {
    source: $filter,
    fn: ({ error }) => {
      // error: unknown
      return isNetworkError({ error });
    }
  }
});

Same unknown type appears if you use just filter handler instead of { source: ..., fn: ... }
It's not blocking or anything because setting the generic type of error through the query creation works just fine and returns
error: AbortError | TimeoutError | PreparationError | NetworkError | HttpError<number> | InvalidDataError | CustomError
Though it would be nice to have this long error enumeration type without generics being set.
Here is the sandbox link, so you can check it out
Tysm for the lib, just added it to the project but enjoy it already ❤

Thanks for report 🙏 I'll take a look at this bug as soon as possible, but before 😉 do you want to fix it by yourself?

fixed in #265, will be released in the next release


How should users design protocols?

Some thoughts on linearity.
{-
----------------- Linear -----------------------------------
0                             0                             2
| someMsg --> >>>>>>>>>>>>>>>>|
3                             2                             2
                              | someMsg -->>>>>>>>>>>>>>>>>>|
3                             7                             7
                              | <-- someMsg <<<<<<<<<<<<<<<<|

----------------- No Linear -----------------------------------

0                             0                             2
| someMsg >>>>>>>>>>>>>>>>>>>>|
2                             4                             2
| someMsg --> >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>|
6                             4                             4
                              | <-- someMsg <<<<<<<<<<<<<<<<|
-}

Linear is a better choice?
I am not sure.

The second case results in an indeterminate order of received messages. I put the received messages into a Dependent map and use them in the order of the protocol


Space - Chelsea

React Tic Tac Toe
Congratulations!  Please reflect on the assignment with these questions.
Reflection

Prompt
Response

How are events / event handlers and useState connected?
The event Handelers often/call the  the useState function and end up changing the state with the provided function to change state.

What are two ways to do "dynamic styling" with React? When should they be used?
I think this question is referring to in-line styling or CSS styling in another file? (be sure to use className). In most cases, you should use another file just to keep things clean easy to read and change in mass.

Much like Rails works with the HTTP request->response cycle, React works with the browser's input->output cycle. Describe React's cycle from receiving user input to outputting different page content.
An event happens, which triggers functions that change data and sends that new data to be re-rendered in the app.

CS Fundamentals Questions

Question
Answer

What do you think is the BigO complexity of the method you use to compute a winner?
I want to say time complexity is O(n) but I am not sure how the flat() function works. Space is O(1) because nothing new is made.

Consider what happens when React processes a state change from setState -- it must re-render all of the components that now have different content because of that change.What kind of data structure are the components in, and what sort of algorithms would be appropriate for React's code to "traverse" those components?Speculate wildly about what the Big-O time complexity of that code might be.
Not sure I fully understand this question. I think React uses a Hash table. Hash tables are great for big O complexity (worst case o(n)?)

React Tic Tac Toe

Major Learning Goals/Code Review

Criteria
yes/no, and optionally any details/lines of code to reference

Demonstrates proper JavaScript coding style.
✔

Correctly passes props to child components.
✔

Correctly passes callback functions to child components and calls them to respond to user events.)
✔

Maintains the status of the game in state.
✔

Practices git with at least 6 small commits and meaningful commit messages
✔, exactly 6 commits, but I suggest that you avoid using waves as commit messages and instead describe functionality added.

Uses existing stylesheets to render components
✔

Functional Requirements

Functional Requirement
yes/no

The Square component renders properly and executes the callback on a click event.
✔

The Board component renders a collection of squares
✔

The App component renders a board and uses state to maintain the status of the game.
✔

Utilizes callbacks to UI events to update state
✔

Overall Feedback

Overall Feedback
Criteria
yes/no

Green (Meets/Exceeds Standards)
5+ in Code Review && 3+ in Functional Requirements

Code Style Bonus Awards

Was the code particularly impressive in code style for any of these reasons (or more...?)

Quality
Yes?

Perfect Indentation
✅

Elegant/Clever
✅

Descriptive/Readable
✅

Concise
✅

Logical/Organized
✅

Summary
Nice work, you hit all the learning goals here.  The project is really well written and very readable.


Broken selenium?

1) TestAuth_Controller::testAlive

PHPUnit_Extensions_Selenium2TestCase_WebDriverException: Bad request

Command duration or timeout: 131 milliseconds

Build info: version: '2.45.0', revision: '5017cb8', time: '2015-02-26 23:59:50'

System info: host: 'testing-gce-fd05d3a0-b26f-4a40-9145-f02a92747fd4', ip: '<IP_ADDRESS>', os.name: 'Linux', os.arch: 'amd64', os.version: '3.13.0-92-generic', java.version: '1.7.0_80'

Driver info: org.openqa.selenium.firefox.FirefoxDriver

org.openqa.selenium.UnsupportedCommandException

/var/www/vendor/phpunit/phpunit-selenium/PHPUnit/Extensions/Selenium2TestCase/Driver.php:147

/var/www/vendor/phpunit/phpunit-selenium/PHPUnit/Extensions/Selenium2TestCase/Driver.php:72

/var/www/vendor/phpunit/phpunit-selenium/PHPUnit/Extensions/Selenium2TestCase/SessionStrategy/Isolated.php:67

/var/www/vendor/phpunit/phpunit-selenium/PHPUnit/Extensions/Selenium2TestCase/SessionStrategy/Shared.php:79

/var/www/vendor/phpunit/phpunit-selenium/PHPUnit/Extensions/Selenium2TestCase.php:246

/var/www/vendor/phpunit/phpunit-selenium/PHPUnit/Extensions/Selenium2TestCase.php:287

/var/www/vendor/phpunit/phpunit/src/Framework/TestCase.php:743

/var/www/vendor/phpunit/phpunit/src/Framework/TestResult.php:609

/var/www/vendor/phpunit/phpunit/src/Framework/TestCase.php:699

/var/www/vendor/phpunit/phpunit-selenium/PHPUnit/Extensions/Selenium2TestCase.php:264

/var/www/vendor/phpunit/phpunit/src/Framework/TestSuite.php:722

/var/www/vendor/phpunit/phpunit/src/Framework/TestSuite.php:722

/var/www/vendor/phpunit/phpunit/src/Framework/TestSuite.php:722

/var/www/vendor/phpunit/phpunit/src/TextUI/TestRunner.php:406

/var/www/vendor/phpunit/phpunit/src/TextUI/Command.php:153

/var/www/vendor/phpunit/phpunit/src/TextUI/Command.php:105

I have close to no idea why this is happeng.
If maybe @Spuds has any idea that can put me on the right track I can try to debug it easier, otherwise I'll have to take the long road and explore all the possibilities... :confused:

Between this issue and the mysql 5.7 (dmr) change, I'm guessing Travis has updated some things in the environment.   This error looks like its not even starting .. maybe check if there is a newer selenium .jar file that should be used.

I tried following a couple of suggestions on the internet (i.e. use firefox 39+ and using a more recent version of selenium, but so far I've not been successful... :cry:

Whats strange to me .. in my foggy mind, is that I swear I changed things over to use chrome in place of FX.  I wonder if some Travis change is invoking the wrong browser?

hmm...
Interesting point.
You are of course right, the setup downloads chrome driver and seems to start the server with it, though in the error we have:
Driver info: org.openqa.selenium.firefox.FirefoxDriver

soooo....
/me is :confused:

https://travis-ci.org/emanuele45/Dialogo/jobs/153664238
AHA!
It was a 32bit/64bit thingy.
I changed the chrome driver to the 64bit version and apparently it started to work again! YAY!
Probably it was using Firefox as a fallback because of the Chrome driver was failing to load, and instead of giving a warning about it, it was silently using Firefox instead.

Do you think it is worth updating selenium to 2.53 as well, while we are at it?

If it runs w/o error then I'd sure update it as well.


Issue/240 slider toggle for theme

Fixes #240

Looks good, i think it would be better if you make a footer in profile card for toggler, so it will displayed different background color with main content

Looks good, i think it would be better if you make a footer in profile card for toggler, so it will displayed different background color with main content

Resolved https://github.com/oddbit/tanam/pull/246/commits/a23d9385a69c817a7464f96c9e0c297fbd457f72


toJSON() improvement

Most of the times the default implementation of toJSON() is great, so you don't need to implement that function. Anyway if you want to change the name of just one property, you need to implement toJSON() for all the properties. It would be nice to add a method that it will ask for mapping some properties name, otherwise will use the default behavior.
example:
My object has a oneProperty property. And the server wants a one_property property. I'd like a method like:
func mapPropertyName(name: String) -> String? {
   if name == "oneProperty" {
       return "one_property"
   } else {
       return nil // use default implementation
   }
}

What do you think about calling super and then making the change for one property? Debatably it's less efficient but Swift doesn't have optional protocol methods so we would have to implement mapPropertyName if we don't need it?

true there is no optional methods, but you can define a default implementation via protocol extension, achieving the same result

Swift sort of has optional protocol methods by creating default implementations in an extension.
For this example, you could change the protocol to:
public protocol JSONEncodable {
    func toJSON() throws -> AnyObject
    func mapPropertyName(name: String) -> String?
}

And add a default implementation:
public extension JSONEncodable {
    func mapPropertyName(name: String) -> String? {
        return nil
    }
}

Then in toJSON, you write: let label = mapPropertyName(defaultLabel) ?? defaultLabel
Users can optionally override the default implementation of mapPropertyName.
Although to be honest, I'm not a fan of writing writing logic in strings. I much prefer creating to separate models, a JSON model and a domain model, where the actual mapping is done in compile-time checked Swift code. So I don't think I would ever use "mapPropertyName", but it is something you could provide using extensions on protocols.


Exact function of hstatus.MXR somewhat ambiguous

Table 5.2 says "The HS-level MXR bit makes any executable page readable."  Which of the following does this mean (with respect to the R and X bits in the level 1 and level 2 PTE's):

Hstatus.MXR makes a page overall readable if it is overall executable (i.e. both level 1 and level 2 X bits are set)?

Hstatus.MXR makes a page overall readable if it is level two executable (irrespective of level 1)?

Hstatus.MXR makes a page level 2 readable if it is level 2 executable (which doesn't guarantee that the page will actually be  readable - depending on level 1).

P.S. I should have used VS-level and HS-level for "level 1" and "level 2".  Sorry about that.

If there is uncertainty as to the proper architectural answer, I would suggest that #2 is the most appropriate answer given what I believe is the intended use case.  The reasoning is that a hypervisor wants to ensure that it can read a page of memory that by default is accessible in some manner (i.e. any combination of VS-level read and/or execute permission, and HS-level read and/or execute permission).  Consequently, hstatus.MXR effectively forces read permission at both the VS and HS levels.  (This would be in contrast to vsstatus.MXR which only forces VS-level read permission and thus still requires HS-level read permission.)
This corresponds to answer #2, reworded in a clearer form:  Hstatus.MXR makes a page overall readable if it is HS-level execute-only (even it is VS-level execute-only).

/cc @jhauser-us


Fix typo during the validate cmd

Fixing a typo while running the validate command. The metadata contained the creator address yet it was throwing warning for creator address so, digged a little bit to find out that it was related to category.

Good catch, thanks!

If you have a Twitter account post it here as we give a shout out to new contributors in releases.

@heisenberglit


regression target of shape predictor trainer

I am confused about the regression target when training a shape predictor model. In your code, the target is the difference of ground truth and current shape. According to this paper, Face Alignment by Explicit Shape Regression, however, the real target is product of difference and similarity transformation matrix, which has been computed when extracting pixel values. Have you ever considered that? It may influence the performance.

There is a similarity transform in my code.

Of course I know you have computed the transform matrix, but you did't use it to normalize the difference between target and current, you just use it to extract features.

suppose we have two images with the same face but different angle, which means one can be obtained by rotating another one in 2-D plane. Obviously, they have the same pixel difference features, but different shape offsets. So why not transforming these offsets.I tested this idea a small dataset, LFPW, and it outperforms without transforming offsets. But in other larger datasets, such as I-BUG, it seems doesn't matter. So it's just a hint. Thanks.

How do you know how to transform the image when you really use it? It isn't as if you know the true labels when you get a new image during test time.

We don't need the true label when transforming a current shape. We just transform the current shape to the initial shape, such as mean shape, to make sure we are working in the same coordinate system. More details are talked in face alignment by explicit shape regression. Equation 3 In section 2.1 describes the regression target like that, y = M(gt - current), not y = gt - current, where M is the transform matrix between current and mean.


Rancher logs spammed with RoleBinding.rbac.authorization.k8s.io  is invalid: when any auth group is assigned restricted admin global role

Rancher Server Setup

Rancher version: v2.6.3
Installation option (Docker install/Helm Chart): Docker install

If Helm Chart, Kubernetes Info: k3s
Cluster Type (RKE1, RKE2, k3s, EKS, etc):
Node Setup: 1 node
Version: v1.21.3+k3s1

Proxy/Cert Details: self-signed

Information about the Cluster

Kubernetes version:
Cluster Type (Local/Downstream): Downstream infrastructure provider 3 worker, 1 etcd, 1 cp
RKE1

Describe the bug
When an auth group is assigned a global role in the groups a global role, we see the rancher logs are spammed with error logs.
To Reproduce

Enable auth - I tried with azure AD and openLDAP
From global auth --> groups --> assign a role from the auth as a restricted admin.
A global role binding is created for the group in the global role bindings:
Verify the rancher logs as soon as the group is added

Result
As soon as the group is added, we see the following logs scroll through continuously:
 error syncing 'grb-hmhsb': handler restrictedAdminGlobalBindingsFleet: 1 error occurred:
	* RoleBinding.rbac.authorization.k8s.io "grb-hmhsb-fleetworkspace-restricted-admin-rb-cluster" is invalid: subjects[0].name: Required value

Expected Result
Did not expect the error logs when the group is added in the rancher server
Additional context
SURE-3845

Hey @MbolotSuse looks like you backported to 2.5 also, can you make a secondary issue and put it into 2.5.13 milestone.
The rancher PR for 2.6 is here: https://github.com/rancher/rancher/pull/36466

Verified on v2.6-head d4089da docker install, helm install
Error logs mentioned in the ticket are not seen for a fresh install and upgrade case.
Re-opening:
However, if we create a downstream cluster after adding an auth group as restricted admin, we see the following error logs.
 [INFO] [node-controller-rancher-machine] Installing Docker from: https://releases.rancher.com/install-docker/20.10.sh
2022/02/18 00:33:42
[ERROR] error syncing 'c-4qtwm': handler restrictedAdminsRBACCluster: 1 error occurred:
 * RoleBinding.rbac.authorization.k8s.io "grb-4jkrb-restricted-admin-rb-cluster" is invalid: subjects[0].name: Required value
Thu, Feb 17 2022 4:33:42 pm

These are the steps I tried:

Enabled openldap from auth providers
From groups >> added an auth group as restricted admin
Create a downstream cluster - rke1 or rke2 node drivers
The above mentioned logs are seen

Closing this ticket as the issue mentioned ^^ is addressed in another security ticket.