updating diagram image and index text given feedback received

updating diagram image and index text given feedback received

Docs Build status updates of commit 2be8cad:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/vision-skills-diagram2-wide.png
:white_check_mark:Succeeded
View

docs/windows-vision-skills/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Hi @LPBourret, what's the difference between this new image and the one currently there? Looks pretty much the same.


Speakers collection page design

should:

have emcee's up top
be alphabetical (by last name)?
have a link to schedule
consider minimalist layout with hover interactions (mobile can visit user detail to get the links)

questions:

should speakers who are speaking together be grouped together?

The schedule/session index will contain speaker details. The speakers homepage module 'advertises' the speakers. What's the purpose of this page?


corrected sample powershell commands

vaultname and name got interchanged

@surajkumarr12 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Note that publishing in this repository is suspended until Tuesday, May 25. Merging will resume Tuesday. Authors can still review and sign off on PRs in the interim. Thanks.

#sign-off


Add Dump Database Option

Proposed Functionality
Would like to see a nautobot-server database-dump option to help migrate data. Process would be either:
nautobot-server dump-database <filename>

or
nautobot-server dump-database
Enter file name [<nautobot_db>-<date>]:

Use Case

Users who start with lab instance and want to migrate data easily to more production ready setup
Create recurring database backups without having to constantly lookup the command set

Database Changes
None

Not sure if it is possible then, also create a database-import option...

Thanks for the request!
How would this differ from the existing Django-provided nautobot-server dumpdata/nautobot-server loaddata commands? Are you just looking for a raw SQL dump instead of JSON/YAML, or is there some other nuance I'm missing?
# nautobot-server dumpdata --help
usage: nautobot-server dumpdata [-h] [--format FORMAT] [--indent INDENT]
                                [--database DATABASE] [-e EXCLUDE]
                                [--natural-foreign] [--natural-primary] [-a]
                                [--pks PRIMARY_KEYS] [-o OUTPUT] [--version]
                                [-v {0,1,2,3}] [--settings SETTINGS]
                                [--pythonpath PYTHONPATH] [--traceback]
                                [--no-color] [--force-color] [--skip-checks]
                                [app_label[.ModelName] [app_label[.ModelName]
                                ...]]

Output the contents of the database as a fixture of the given format (using
each model's default manager unless --all is specified).

positional arguments:
  app_label[.ModelName]
                        Restricts dumped data to the specified app_label or
                        app_label.ModelName.

optional arguments:
  -h, --help            show this help message and exit
  --format FORMAT       Specifies the output serialization format for
                        fixtures.
  --indent INDENT       Specifies the indent level to use when pretty-printing
                        output.
  --database DATABASE   Nominates a specific database to dump fixtures from.
                        Defaults to the "default" database.
  -e EXCLUDE, --exclude EXCLUDE
                        An app_label or app_label.ModelName to exclude (use
                        multiple --exclude to exclude multiple apps/models).
  --natural-foreign     Use natural foreign keys if they are available.
  --natural-primary     Use natural primary keys if they are available.
  -a, --all             Use Django's base manager to dump all models stored in
                        the database, including those that would otherwise be
                        filtered or modified by a custom manager.
  --pks PRIMARY_KEYS    Only dump objects with given primary keys. Accepts a
                        comma-separated list of keys. This option only works
                        when you specify one model.
  -o OUTPUT, --output OUTPUT
                        Specifies file to which the output is written.
  --version             show program's version number and exit
  -v {0,1,2,3}, --verbosity {0,1,2,3}
                        Verbosity level; 0=minimal output, 1=normal output,
                        2=verbose output, 3=very verbose output
  --settings SETTINGS   The Python path to a settings module, e.g.
                        "myproject.settings.main". If this isn't provided, the
                        DJANGO_SETTINGS_MODULE environment variable will be
                        used.
  --pythonpath PYTHONPATH
                        A directory to add to the Python path, e.g.
                        "/home/djangoprojects/myproject".
  --traceback           Raise on CommandError exceptions
  --no-color            Don't colorize the command output.
  --force-color         Force colorization of the command output.
  --skip-checks         Skip system checks.

# nautobot-server loaddata --help
usage: nautobot-server loaddata [-h] [--database DATABASE] [--app APP_LABEL]
                                [--ignorenonexistent] [-e EXCLUDE]
                                [--format FORMAT] [--version] [-v {0,1,2,3}]
                                [--settings SETTINGS]
                                [--pythonpath PYTHONPATH] [--traceback]
                                [--no-color] [--force-color] [--skip-checks]
                                fixture [fixture ...]

Installs the named fixture(s) in the database.

positional arguments:
  fixture               Fixture labels.

optional arguments:
  -h, --help            show this help message and exit
  --database DATABASE   Nominates a specific database to load fixtures into.
                        Defaults to the "default" database.
  --app APP_LABEL       Only look for fixtures in the specified app.
  --ignorenonexistent, -i
                        Ignores entries in the serialized data for fields that
                        do not currently exist on the model.
  -e EXCLUDE, --exclude EXCLUDE
                        An app_label or app_label.ModelName to exclude. Can be
                        used multiple times.
  --format FORMAT       Format of serialized data when reading from stdin.
  --version             show program's version number and exit
  -v {0,1,2,3}, --verbosity {0,1,2,3}
                        Verbosity level; 0=minimal output, 1=normal output,
                        2=verbose output, 3=very verbose output
  --settings SETTINGS   The Python path to a settings module, e.g.
                        "myproject.settings.main". If this isn't provided, the
                        DJANGO_SETTINGS_MODULE environment variable will be
                        used.
  --pythonpath PYTHONPATH
                        A directory to add to the Python path, e.g.
                        "/home/djangoprojects/myproject".
  --traceback           Raise on CommandError exceptions
  --no-color            Don't colorize the command output.
  --force-color         Force colorization of the command output.
  --skip-checks         Skip system checks.

It's corresponding to the documentation here:
https://nautobot.readthedocs.io/en/latest/administration/replicating-nautobot/
Would we be comfortable for a database backup perspective just using the dumpdata commands to backup the data? Is there a speed implication on larger sized databases?

It's corresponding to the documentation here:
https://nautobot.readthedocs.io/en/latest/administration/replicating-nautobot/
Would we be comfortable for a database backup perspective just using the dumpdata commands to backup the data? Is there a speed implication on larger sized databases?

The nice thing about dumpdata is that it is database agnostic since it's dumping the model data in a serialized form. Performing a raw SQL dump from a database backend varies based on semantics of the backend.
My concern that if we were to add this for now we'd have to tightly couple it for PostgreSQL's pg_dump command, which isn't always guaranteed to be installed. In the future as we add more backends, it would be a little more upkeep.
To answer your question, I don't think speed on database dumping should be a concern. It isn't generally something done frequently, and correctness is more important.

I'm good with the thoughts here. Should we close this issue out @jathanism ?


Tests on Raspberry Pi 4 failing

Required information
Operating system:
Raspbian Buster (32-Bit)
Linux armv7l GNU/Linux
Compiler version:
E.g. GCC 8.3.0
Observed result or behaviour:
Multiple Tests on Raspberry Pi 4 are failing.
Expected result or behaviour:
All tests are green
Conditions where it occurred / Performed steps:
checkout iceoryx master
execute  "./tools/iceoryx_build_test.sh test"
ececute "build/utils/utils/utils_moduletests" manually
Test Logs:
iceoryx_posh:
[ RUN      ] ShmCreatorDeathTest.AllocatingTooMuchMemoryLeadsToExitWithSIGBUS
/home/pi/dev/iceoryx/iceoryx_posh/test/integrationtests/test_shm_sigbus_handler.cpp:43: Failure
Death test: iox::runtime::SharedMemoryCreatoriox::roudi::MiddlewareShm sut(badconfig)
Result: failed to die.
Error msg:
[  DEATH   ] Reserving 111604128 bytes in the shared memory [/iceoryx_mgmt]
[  DEATH   ] [ Reserving shared memory successful ]
[  DEATH   ] 2020-03-17 08:54:59.689 [ Info  ]: Roudi registered management segment 0xffffffffb0121000 with size 111604128 to id 1
[  DEATH   ] Reserving<PHONE_NUMBER>00 bytes in the shared memory [/pi]
[  DEATH   ] [ Reserving shared memory successful ]
[  DEATH   ] 2020-03-17 08:54:59.690 [ Info  ]: Roudi registered payload segment 0xffffffffb6fe5000 with size<PHONE_NUMBER>00 to id 2
[  DEATH   ]
Reserving 111597024 bytes in the shared memory [/iceoryx_mgmt]
[ Reserving shared memory successful ]
2020-03-17 08:55:00.066 [ Info  ]: Roudi registered management segment 0xffffffffb0123000 with size 111597024 to id 1
Reserving 1088 bytes in the shared memory [/pi]
[ Reserving shared memory successful ]
2020-03-17 08:55:00.066 [ Info  ]: Roudi registered payload segment 0xffffffffb6fe6000 with size 1088 to id 2
[  FAILED  ] ShmCreatorDeathTest.AllocatingTooMuchMemoryLeadsToExitWithSIGBUS (750 ms)
iceoryx_utils:
[ RUN      ] relativeptrtests/0.ConstrTests
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_relative_pointer.cpp:127: Failure
Expected equality of these values:
rp.getId()
Which is: 2
1
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_relative_pointer.cpp:143: Failure
Expected equality of these values:
rp.getId()
Which is: 2
1
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_relative_pointer.cpp:152: Failure
Expected equality of these values:
rp.getId()
Which is: 2
1
[  FAILED  ] relativeptrtests/0.ConstrTests, where TypeParam = unsigned char (2 ms)
[ RUN      ] relativeptrtests/1.ConstrTests
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_relative_pointer.cpp:127: Failure
Expected equality of these values:
rp.getId()
Which is: 2
1
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_relative_pointer.cpp:143: Failure
Expected equality of these values:
rp.getId()
Which is: 2
1
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_relative_pointer.cpp:152: Failure
Expected equality of these values:
rp.getId()
Which is: 2
1
[  FAILED  ] relativeptrtests/1.ConstrTests, where TypeParam = signed char (1 ms)
[ RUN      ] relativeptrtests/2.ConstrTests
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_relative_pointer.cpp:127: Failure
Expected equality of these values:
rp.getId()
Which is: 2
1
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_relative_pointer.cpp:143: Failure
Expected equality of these values:
rp.getId()
Which is: 2
1
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_relative_pointer.cpp:152: Failure
Expected equality of these values:
rp.getId()
Which is: 2
1
[  FAILED  ] relativeptrtests/2.ConstrTests, where TypeParam = double (1 ms)
[ RUN      ] RelocatablePointer_test.relocation
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_relocatable_ptr.cpp:78: Failure
Expected equality of these values:
base2 - base1
Which is: -1024
BLOCK_SIZE
Which is: 1024
[  FAILED  ] RelocatablePointer_test.relocation (1 ms)
[----------] 1 test from RelocatablePointer_test (1 ms total)
[ RUN      ] Duration_test.divideDuration
/home/pi/dev/iceoryx/iceoryx_utils/test/moduletests/test_unit_duration.cpp:202: Failure
The difference between result.seconds() and 8.666666666666666 is 1.7763568394002505e-15, which exceeds doubleEpsilon, where
result.seconds() evaluates to 8.6666666666666679,
8.666666666666666 evaluates to 8.6666666666666661, and
doubleEpsilon evaluates to 2.2204460492503131e-16.
[  FAILED  ] Duration_test.divideDuration (0 ms)

Closing this as we do not officially support 32 bit


Update configure-rbac-kube.md

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit cec3b5eae3b2a015bedf7632c67561832fc32735
https://deploy-preview-10006--docsdocker.netlify.com

Thank you for taking the time to review and update our documentation. We consider the use of gerunds and participles to be acceptable in our documentation. Therefore, I'm closing this PR without merging it.


Generate API documentation for the website.

Add tooling to generate online API documentation from the CRD
types. This included in the make manifests build target.
Signed-off-by: James Peach<EMAIL_ADDRESS>

A couple of issues

there's no verify check yet
API string constants (e.g. condition types) don't get emitted
the comment formatting need some HTML tags to lay out nicely

the comment formatting need some HTML tags to lay out nicely

The tool is supposed to to markdown processing on the comments; it doesn't look to me like that is working well (e.g. paragraphs aren't there), so probably best to investigate that before sprinkling tags everywhere.

/ok-to-test

@bowei LMK if you are on board for generating API docs; if so I'll tidy this PR up. Happy to add to meeting agenda if we want to have a quick discussion in the WG.

@jpeach This is very cool, thanks for the initial work on this! I think this would be a great addition to our docs. Likely worth discussing a bit more at office hours today.

@robscott I added the verify script, so this is ready now. We can fix up any formatting documentation formatting issues in followup PRs.

/retest

CI failure is verify-docs, which is #188

@robscott @bowei Fixed tests, ready to go!

This is awesome, thanks for the work on this! This all LGTM, I just think it needs to actually be linked from the docs. I think adding a link in the sidebar as well as one on the main docs page replacing the current API Specification link.

This is awesome, thanks for the work on this! This all LGTM, I just think it needs to actually be linked from the docs. I think adding a link in the sidebar as well as one on the main docs page replacing the current API Specification link.

I did a make serve and AFAICT both of those links work correctly. This generates HTML directly into docs-src/spec.md (which is legal Markdown), so another needed to be adjusted. If I haven't interpreted your comment correctly, I'm happy to get on a call and figure it out :)

I'm assuming rob's LGTM is
/lgtm
/approve


a clean epicycle simulation

How do I simulate the epicycles just like the demo video?
I just want to make a video out of it without the image or the yellow drawing dots

If you want to make the simulation on your own image its is possible.
You need to provide the path of your image, or put it in the FourierImage folder and give its name as input when launching the program.
Then you will have to sample your image yourself by following the outline of your image and clicking point by point,
BUT before that you must clear the previous sample made by typing on 'R'.
Then you can press space to see the simulation.
Have fun!

If you want more information about controls you can see the Controls section down in the readme.

Oh. I think you got me wrong. I know how to operate it but I want to have a mode in which the image and the drawing are not present.
elif self.mode == 1:             if self.show_polynomial:                 self.drawPolynomial(drawing, self.color_polynomial)             #if self.show_drawing:             #    self.drawGraph(drawing, self.color_drawing)             if self.show_vectors:                 self.drawVectors(construction, self.color_vectors)             if self.show_circles:                 self.drawCircles(construction, self.color_circles)             if self.show_display:                 self.drawGraph(display, self.color_display)             #if self.show_sample:             #   self.drawPoints(self.sample, self.color_sample)
I did this now I get what I want. but still the image is shown, but I just loaded an empty image now it works fine. You might wanna add another mode ...

All right, I think i get it.
Actually it is possible to hide the image and the drawing by using the numbers in the range of 0 to 6.

You can hide the image by pressing 1
You can hide the drawing by pressing 3

As you can see, with this system there is no need for another mode.
I just forgot put it in the controls section, so thanks for noticing!

Oh, that's great good job


ore|plume: Remove PV support

Removes uploading of the deprecated AWS PV AMIs in ore and plume.

Have removed the functionality there, too.


[MNT] experimental removal of upper bound holidays to 0.13 in soft dependencies

Experimentally reverts alan-turing-institute/sktime#2749, to check whether facebook prophet have resolved their problem https://github.com/alan-turing-institute/sktime/issues/2748 with holidays, pre-0.12.0 release

Seems to work - I will revert the emergency merge in #2749 as soon as tests pass.
#2749 was a "temporary patch" to ensure our CI/CD does not break.


R8g instance support

Describe the Bug:
R8g instance types are not supported. No entry exists in limits.go for these instances yet.
Observed Behavior:
When provisioning an r8g instance, error statuses are reported:
  Warning  Unsupported              20s (x6 over 34s)  vpc-resource-controller  The instance type r8g.xlarge is not supported yet by the vpc resource controller
Expected Behavior:
R8g instances should work.
How to reproduce it (as minimally and precisely as possible):
Provision an R8g instance in AWS.
Environment:

Kubernetes version (use kubectl version):

1.29

It's disappointing that support for r8g instances within the vpc-resource-controller did not precede the general availability of r8g instances.
This is particularly problematic when combined with a greater-than karpenter.k8s.aws/instance-generation requirement in a Karpenter NodePool, as unsupported instances were automatically selected.
Perhaps there should be a karpenter.k8s.aws/vpc-resource-controller-supported label in Karpenter (similar to karpenter.k8s.aws/instance-encryption-in-transit-supported), so this condition can be filtered for without restricting the max generation of instances?

This appears to have been resolved by #475, which added entries in limits.go.

This appears to have been resolved by #475, which added entries in limits.go.

I'm not sure. That PR adds support for x8g instances. Not r8g nor c8g


Crash while scrolling into UITableView and going back on iOS 10.

Hello,
I have a crash on iOS 10. while scrolling into UITableView and going back (from a navigation controller).
Error:
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'An instance 0x7ff741828400 of class UITableViewWrapperView was deallocated while key value observers were still registered with it. Current observation info: <NSKeyValueObservationInfo 0x610000037b60> (
<NSKeyValueObservance 0x6100002581e0: Observer: 0x610000269e80, Key path: contentOffset, Options: <New: NO, Old: NO, Prior: NO> Context: 0x0, Property: 0x610000257400>

Looks like that you forget to remove observers (not setting them to nil) on ScrollViewUpdater.
You also forget to call deinit and remove observers on DeckPresentationController.

@HarshilShah Still alive ..?

Hi there, and thanks for opening an issue to bring this to notice.
Your diagnosis isn’t quite correct, but this is an issue that needs some architectural changes to handle. I’ll be posting an RFC which goes into some changes soon.
Sent with GitHawk

Hey @HarshilShah,
I seem to be experiencing some kind of similar issue with using this library with IGListKit. The error message is as follows:
2018-11-09 12:53:53.502557-0500 Jobs[28500:3904785] [CollectionView] An attempt to prepare a layout while a prepareLayout call was already in progress (i.e. reentrant call) has been ignored. Please file a bug. UICollectionView instance is (<UICollectionView: 0x12a001a00; frame = (0 0; 375 760); clipsToBounds = YES; gestureRecognizers = <NSArray: 0x2820745a0>; layer = <CALayer: 0x282e24980>; contentOffset: {0, 0}; contentSize: {0, 0}; adjustedContentInset: {0, 0, 34, 0}> collection view layout: <UICollectionViewFlowLayout: 0x129101190>)
2018-11-09 12:53:55.007733-0500 Jobs[28500:3904785] [CollectionView] An attempt to update layout information was detected while already in the process of computing the layout (i.e. reentrant call). This will result in unexpected behaviour or a crash. This may happen if a layout pass is triggered while calling out to a delegate.  If you have any insight about this, or would like help, I am happy to contribute.

@sejr @adeelnoor  Did you found the solution I am having the same issue in real devices with old iOS i.e. 11.x, but its working fine on simulator. adeelnoor, can you resolve the question?

Hello @leviduan
Yes, create a Nib instance first in global level for example
static func nibInstance() -> Self {
        return Bundle.main.loadNibNamed(Self.nibName(), owner: nil, options: nil)?.first as! Self
    }

private lazy var sampleCell = UICollectionViewCell.nibInstance()

And call this sampleCell where you want to call.


[docs] Update logs agent TCP/HTTP option descriptions

What does this PR do?
Docs update: add details on the use_http/use_tcp options, what our recommendation is, and correct a mention of TCP being the default.
Motivation
Fix docs.
Describe how to test/QA your changes
Docs update, no QA.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The appropriate team/.. label has been applied, if known.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

@gh123man That would make sense, and we could and should do it in a non-breaking way by continuing to support the older option names. This would be a larger change (and would require updating public docs as well) so let's do it in a separate PR, feel free to create a card to track this.


Improve helper handling in Nanoc::Int::Context

Fixes #1169.
This introduces use_helper to load helpers. Helpers loaded with use_helper will only be available in contexts where it makes sense, so they don’t pollute the global namespace.
module GreetingHelper
  def greeting
    'hello'
  end
end

use_helper GreetingHelper

This change makes the helpers not available in the Rules file. Making them available there would still be useful.

This needs additional tests to verify the behavior of use_helper.


wai-devel fails to find a custom entry point passed with '-p'

Hello there,
I'm trying to use wai-devel for a Spock application. I have installed wai-devel and backend-ide-server, and my project structure looks like this:
.
├── LICENSE
├── Setup.hs
├── app
│   ├── Main.hs
│   └── Routes.hs
├── spock-test.cabal
├── src
│   └── Lib.hs
├── stack.yaml
└── test
    └── Spec.hs

If I run wai-devel with wai-devel -p app/Main.hs this is what I get:
☁  spock-test  wai-devel -p app/Main.hs
"app/Main.hs": openBinaryFile: does not exist (No such file or directory)
Starting devel application at http://localhost:3000
UpdateStatusRequiredRestart
UpdateStatusDone
Error: <no location info> can't find file: /Users/adinapoli/programming/haskell/spock-test/Application.hs

What am I doing wrong? It doesn't work even with the fully qualified path:
☁  spock-test  wai-devel -p /Users/adinapoli/programming/haskell/spock-test/app/Main.hs
"/Users/adinapoli/programming/haskell/spock-test/app/Main.hs": openBinaryFile: does not exist (No such file or directory)
Starting devel application at http://localhost:3000
UpdateStatusRequiredRestart
UpdateStatusDone
Error: <no location info> can't find file: /Users/adinapoli/programming/haskell/spock-test/Application.hs

My PATH and GHC_PACKAGE_PATHS were set with the following (because I have another build tool set as default on this work laptop I cannot uninstall):
export PATH=/Users/adinapoli/.stack/programs/x86_64-osx/ghc-7.10.2/bin:$PATH
export GHC_PACKAGE_PATH=$(stack path --local-pkg-db)

Thanks!
Alfredo

When you pass -p you should also pass -f unless in your -p ( file with the function you want to run) there is a develMain function. This however does not show in the error. It seems that in your case it doesn't look for app/Main.hs but rather looks for Application.hs let me look into it.

@adinapoli Thanks you really saved me there. There was a huge bug in the argument passing there that I hadn't noticed.
Try it now, I have pushed code that I believe fixes that. Remember that -p most often works with -f unless your -p argument contains a develMain function.

Works great, thank you! I'm now getting a different error when it comes to resolving the modules in my main file, but that's a separate issue I will open:
☁  metrics [issue-2232] ⚡ wai-devel -p app/Main.hs -f main
"app/Main.hs"
"main"
Starting devel application at http://localhost:3000
UpdateStatusRequiredRestart
UpdateStatusDone
Error: app/Main.hs@6:8-6:19 Could not find module ‘Application’
Use -v to see a list of the files searched for.

Thank you for the quick response! You can close this.


Newline with o and enter creates list item

Hey @kristijanhusak, awesome work, I had to the chance to give orgmode.nvim a try and intending to switch to it soon.
Describe the bug
Pressing enter on a section create new indented list list.
Pressing enter on a todo section creates a indent line.
* Section<CR>
   * |
* TODO Section<CR>
   |

I feel this intentional behavior and I wonder if I can disable it somehow or is there an alternative way to create sub item in normal and insert mode
To Reproduce
Steps to reproduce the behavior:

create new heading
press enter in insert mode or o in normal mode

Expected behavior
A normal enter/o should act as expected.
Emacs functionality
not sure
System information:

OS:  mac
Neovim version/commit: [e.g. https://github.com/neovim/neovim/commit/bdf3df4027a5389a46b4c9a75b50aff5c0ecb351]

Additional context
Add any other context about the problem here.

Hey @tami5 , I'm not able to reproduce the bug, or I don't understand it completely.
When I have this:
* Section|

And I press Enter, I get this:
* Section
  |

And for todo item, this:
* TODO Section|

Becomes this:
* TODO Section
  |

I do not override functionality for either o or <CR> in org file.
Maybe you are using some other plugin that overrides this.
Can you check what you have mapped to those while in org buffer?

o - :nmap o
<CR> - :imap <CR>

Oh I see so the indentation is default behavior. is there away to disable it?
Indeed,  is mapped to compe related function but when I changed back to <CR> same issue.
* TODO ITEM<CR>/o
   * | < appends `*`

:nmap o isn't mapped to anything but still same issue with * TODO section creating indented list item.
I'm sure I have something conflicting somehow, I'll investigate more in the morning. Thanks for confirming

Emacs orgmode has electric-indent-mode, but from what I see, this completely removes indentation, which means that this:
* TODO Section|

Becomes this:
* TODO Section
|

But this also means that all indentation is disabled. So if you have something like this:
* TODO Section
  Some content|

Pressing <CR> does this:
* TODO Section
  Some content
|

If you want to insert additional headline below you can use <Leader><CR>, which should do this:
* TODO Section|

Becomes:
* TODO Section

* |

FWIW I actually thought the indentation was a nice touch, not to derail this issue but if there is some change made can it be optional so it can be maintained for anyone who also likes this behaviour. That being said I'm not an orgmode expert should the next line after the todo be indented. To me it seems like it should so the asterixes are on the left marking each todo and all the content of the todo is indented e.g.
* TODO do something important 1
  :DEADLINE: <some-time>
  :PROPERTIES: stuff

* TODO do something else 2
  :DEADLINE: <some-other-stuff>

@tami5
Does your formatoptions have o in them?
Check the output of
set formatoptions?

If you have o in the value try removing it and see if that solves the issue
set formatoptions-=o

If it does and you want this to persist, add to your config something like:
augroup orgmodeAutocommands
  au!
  autocmd filetype org setlocal formatoptions-=o
augroup END

Or just add the autocmd to your existing augroup

Hey @talw , Indeed I had format option o but even after removing it and ending up with formatoptions=1jcrql the issue still exists, I tried and remove all formatoptions and that fixed the issue 🗡 , I'd need to read through the format options a bit more see which one causing this issue and add autocmd as you've suggested.
Thanks so much @talw

@tami5
No problem! I'm glad I could be of some help.
I guess all that's left is to try and remove each time a different letter so that you can pinpoint exactly which letter causes it, and once you do just set an autocmd to remove it from formatoptions upon loading an org file buffer.


macos-10.15 missing LLVM 13

Description
The documentation for macos-10.15 says that LLVM 13 is installed in /usr/local/opt/llvm:

Clang/LLVM 13.0.1 is available on '$(brew --prefix llvm)/bin/clang'

I have relied on this tool at this location for many months.
Today (or yesterday), my CI scripts stopped working correctly. It looks like /usr/local/opt/llvm no longer exists on macos-10.15 workers.
According to brew, llvm@13 is installed. However, $(brew --prefix llvm)/bin/clang doesn't exist:
$ brew list llvm@13
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
$ brew --prefix llvm
/usr/local/opt/llvm
$ ls /usr/local/opt/llvm
ls: /usr/local/opt/llvm: No such file or directory

$ sw_vers
ProductName:    Mac OS X
ProductVersion: 10.15.7
BuildVersion:   19H1922

Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions

Virtual environments affected

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[X] macOS 10.15
[ ] macOS 11
[ ] macOS 12
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
Environment: macos-10.15
Version: 20220627.1
Included Software: https://github.com/actions/virtual-environments/blob/macOS-10.15/20220627.1/images/macos/macos-10.15-Readme.md
Image Release: https://github.com/actions/virtual-environments/releases/tag/macOS-10.15%2F20220627.1
https://github.com/quick-lint/quick-lint-js/runs/7123423723?check_suite_focus=true
Is it regression?
yes; 20220622.2; https://github.com/quick-lint/quick-lint-js/runs/7105900086?check_suite_focus=true
Expected behavior
$(brew --prefix llvm)/bin/clang exists and is executable
Actual behavior
$(brew --prefix llvm)/bin/clang does not exist
Repro steps
Run ls /usr/local/opt/llvm in a job

Hello @strager.
It's expected that the path /usr/local/opt/llvm doesn't exist - https://github.com/actions/virtual-environments/pull/5819#issuecomment-1167815588

If you're explicitly installing llvm@13, then it will only be found through /usr/local/opt/llvm@13. /usr/local/opt/llvm will only exist if you've installed llvm, and it will point to LLVM 14. So if anything was previously hardcoded to /usr/local/opt/llvm, it will have to be changed to /usr/local/opt/llvm@13 or LLVM 14 will have to be used instead.

It's expected that the path /usr/local/opt/llvm doesn't exist

Then the documentation should be updated.

https://github.com/actions/virtual-environments/pull/5837 is in progress.


no more special case for timeout busybox

What does this fix? Does it address the timeout: unrecognized option: t error on Alpine 3.10?


Query variables are not supported in @graphql-codegen/typescript-stencil-apollo despite doc

Variables are not supported in @graphql-codegen/typescript-stencil-apollo despite doc
GraphQL query variables are not supported with @graphql-codegen/typescript-stencil-apollo despite the doc saying so.
The documentation for typescript-stencil-apollo says that:
<TestComponent variables={...}>
...
</TestComponent>

can be used and thus it implies, that query variables can be used. However neither the types are correctly generated nor is there actual code for this.
To Reproduce

git clone https://github.com/ChristianUlbrich/stencil-apollo#origin/add-class-with-variables
cd stencil-apollo/example-with-codegen-variables
npm i
npm run build

TypeScript: src/components/my-component-with-codegen-variables/my-component-with-codegen-variables.tsx:27:10
Type '{ variables: { id: number; }; renderer: ({ data, loading }: QueryResult<{ __typename?: "Query"; } & {
postsOf: ({ __typename?: "Post"; } & Pick<Post, "title" | "id" | "votes"> & { author: { __typename?:
"Author"; } & Pick<Author, "id" | ... 1 more ... | "lastName">; })[]; }, AllPostsOfAuthorQueryVariables>) =>
any; }' is not assignable to type 'ApolloAllPostsOfAuthor &
HTMLAttributes'.Property 'variables' does not exist on type
'ApolloAllPostsOfAuthor & HTMLAttributes'.

Additional context

Fix should not be so difficult and just involve adding proper Type generation in packages/plugins/typescript/stencil-apollo/src/visitor.ts. I try to hack a PR together...

@ardatan please take a look, you are the Stencil expert ;)

@ChristianUlbrich Thank you for the fix! You can use it with this canary version until it is released;
1.8.3-alpha-5b6e16fa.30+5b6e16fa

@ardatan Thx. Works perfectly; how about adding / enhancing the codegen example in stencil-apollo? Want me to PR this as well?

Fixed in 1.8.3
@ChristianUlbrich a PR sounds good ;) Thank you!


plugins/trouble: fix originalName

Discovered in testing https://github.com/nix-community/nixvim/pull/2602

@mergify queue


email_mirror: Downgrade a couple ZulipEmailForwardErrors.

These errors may occur in the course of normal operations, and
shouldn't generate a message in the error logs.

I think an INFO debug would be appropriate here, so that we can track how often this happens (relevant for product decisions on whether to change these policies)

Downgraded all of these user errors to INFO.

Looks great, merged, thanks @alexmv!

Lgtm too


Adds the extensions patch functionality

Adds the ability to patch any method, attribute or item within an attribute in a node using decorators. This includes the patching of handlers and of sub-attributes.
Supersedes #97

It does have access, and indeed the test will fail if I call the node.pull_set_handler["default"] since it does not have the right input arguments. All versions of the patch require the first argument to be a node (typically self). The only difference is that for the case of having an item the previous version of the object to override is not preserved in a self._patched_whatever. This is - I had hoped - explained in the docstring.
I'll check your test, now

There was a but in the code making the use of the function more convoluted. I've fixed the bug and amend your test, so it works as it should. See the following for the changes https://github.com/ImperialCollegeLondon/wsi/pull/101/commits/566976ee6d905424ac5f34f7b2ac874ca551c087
Many thanks for your tests - they have picked a few things that were just wrong.


Expose sensitivity setting as slider

Problem: Right now the sense setting is hidden away in a small easy to miss text box that even excepts letters.
Proposal: Move the setting down and allow the user to use a slider in addition to a numeral text input.

I actually just changed this yesterday in 04bc5c4e076a9e4707b0267b4892375bc27776e0 take a look,
Now there is indeed a slider and a number text input.


Add SSL support

Hello ,
Possible to add SSL option to MQTT config? (TLS not necessary)

Did you try  url: "ssl://<MQTT SERVER>:1883" instead of  url: "tcp://<MQTT SERVER>:1883"?
I do not use SSL/TLS myself so I cannot test it.

seems work, but possible skip cert:
FATAL  MQTT connection error="network Error : tls: failed to verify certificate: x509: certificate is not valid for any names, but wanted to match xxxxxxxx.com"


WOO-77: Rename Merchant Token to Access Token

The tests are failing.

The tests are failing.

Fixed


Update dataflow container tag

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R : @damccorm @tvalentyn @jrmccluskey

FYI the image might not have the changes you need:
docker run --rm -it --entrypoint=/bin/bash gcr.io/cloud-dataflow/v1beta3/beam_python3.11_sdk:beam-master-20231215
Status: Downloaded newer image for gcr.io/cloud-dataflow/v1beta3/beam_python3.11_sdk:beam-master-20231215
root@c33b833dc7c7:/# pip list | grep storage
google-cloud-bigquery-storage   2.23.0
google-cloud-storage            2.13.0

i'll re-run internal cloning workflow to clone latest images with the same tag.

The change I need was google-cloud-bigtable==2.22.0. Let me check the container.

google-cloud-aiplatform         1.37.0
google-cloud-bigquery           3.13.0
google-cloud-bigquery-storage   2.23.0
google-cloud-bigtable           2.22.0
google-cloud-core               2.3.3
google-cloud-datastore          2.18.0
google-cloud-dlp                3.13.0
google-cloud-language           2.11.1
google-cloud-profiler           4.1.0
google-cloud-pubsub             2.18.4
google-cloud-pubsublite         1.8.3
google-cloud-recommendations-ai 0.10.5
google-cloud-resource-manager   1.10.4
google-cloud-spanner            3.40.1
google-cloud-storage            2.13.0
google-cloud-videointelligence  2.11.4
google-cloud-vision             3.4.5

These are the dependencies in the container.

fyi, there as a recent change: https://github.com/apache/beam/pull/29765/files#diff-1275c48808de339ef6f282d844c83ec441b5cfa0debc373fdcb7dba497da4fc8R376

google-cloud-storage>=2.14.0 is missing

Yes for the commit 1ce707c38650a7f9c5120061adfeed37889cc634, the base image requirements was not updated. So the cloning containers is not picking up the change. I will make a PR updating base image requirements and then merge the new container tag.

https://github.com/apache/beam/pull/29800/ - this will update the python base image dependencies.

I reran the cloning containers. latest container dependencies
google-cloud-aiplatform         1.38.1
google-cloud-bigquery           3.14.1
google-cloud-bigquery-storage   2.24.0
google-cloud-bigtable           2.22.0
google-cloud-core               2.4.1
google-cloud-datastore          2.19.0
google-cloud-dlp                3.14.0
google-cloud-language           2.12.0
google-cloud-profiler           4.1.0
google-cloud-pubsub             2.19.0
google-cloud-pubsublite         1.9.0
google-cloud-recommendations-ai 0.10.6
google-cloud-resource-manager   1.11.0
google-cloud-spanner            3.40.1
google-cloud-storage            2.14.0
google-cloud-videointelligence  2.12.0
google-cloud-vision             3.5.0


The semantic table does not stretch to the full width of the page

Describe the bug
The table does not stretch to the end of the page, although in the demo the table is located to the end of the page width. I used all the necessary components from the template, replacing only the backend with my api to get users and removed button "Export".
As for "spacing", I also saw that "rx.table.cell" and "rx.table.column_header_cell" have "col_span", but when using this parameter, this appear only for column names, not for data.
To Reproduce
Steps to reproduce the behavior:
reflex init --template ci_template (repo)
Expected behavior
Dynamic expansion of the table, as well as "spacing" between columns. Maybe it's already in reflex and I haven't figured it out. In the demo version, there is a large space between columns, but mine is too small.
Screenshots
Demo table cijob

My table

Specifics (please complete the following information):

Python Version: 3.12
Reflex Version: 5.2.0
OS: Windows 11

This should be resolved in reflex 0.6.5 - see #4378


Request: example how to enable github/google authorization

Hi, i can't add github/google authorization feature,
could you make example for that please...

Closing this, continue with #22


Training with New Set of Fine grained labels - NER

Hello @nreimers , I really enjoyed going through your work. Could we use your existing model / code to retrain for say 50 fine grained ner classes ? If so, what changes do we need to make ? I have generated the data in the required format similar to the German NER task, I want to know where I need to add the extra label information (50 labels) in the code. Thanks

Hi @prohit93
you are asking though questions.
I would use the multi-task BiLSTM implementation. Then you train the model on coarse grain labels first.
After that, you take the base model and train a new classifier for your fine grained labels. The old classifier (softmax or CRF) for the coarse grained tags are no longer needed.
You can also try to run this as a multi task setup and provide data for the coarse grained labels along with the data for your fine grained labels.
However, I'm not certain if you will achieve that much improvement compared to directly training on your dataset with your fine grained labels. Most multi-task frameworks do not yield that much improvement and they are most often beaten by a simple pipeline approach:

First, trained the network on task 1 (e.g. coarse NER tags)
Add tags from task 1 as features to the input of task 2 and train the network on the sentences + the tags from task 1.

This pipeline approach was in all my experiment often much better than more complicated multi-task setups (or transfer learning setups).


paibuild hadoop-run.dockerfile can not find dependency/hadoop-binary

Organization Name:
Short summary about the issue/question:
During paibuild for hadoop-run images, the hadoop binary path was not copied to dependency after hadoop-ai was built similar to #780.
Brief what process you are following:
During initial Paibuild

How to reproduce it:
python paibuild.py build -c [path/to/configDir]

OpenPAI Environment:

OpenPAI version: 0.9.0
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Hardware (e.g. core number, memory size, storage size, GPU type etc.):
Others:

Anything else we need to know:
One thing i tried to make it work was to manually copied the hadoop binary path from hadoop-ai,
by putting the following lines at the start in hadoop-run/build/build-pre.sh,
then it will follow the rest of the script, copying the /hadoop-binary into dependency.
docker create -ti --name=hadoop-dummy --volume=/hadoop-binary:/hadoop-binary hadoop-build /bin/bash
docker cp hadoop-dummy:/hadoop-binary /hadoop-binary
docker rm -fv hadoop-dummy

@matthewygf
What's your build environment, on your host or in the dev-box?


quarkus: 3.10.0 -> 3.10.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/quarkusio/quarkus/releases.
meta.description for quarkus is: Quarkus is a Kubernetes-native Java framework tailored for GraalVM and HotSpot, crafted from best-of-breed Java libraries and standards
meta.homepage for quarkus is: https://quarkus.io
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.10.1 with grep in /nix/store/m7ffsfk9mh5iq8dwrgazm13rxrap540z-quarkus-cli-3.10.1
found 3.10.1 in filename of file in /nix/store/m7ffsfk9mh5iq8dwrgazm13rxrap540z-quarkus-cli-3.10.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

quarkus

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/m7ffsfk9mh5iq8dwrgazm13rxrap540z-quarkus-cli-3.10.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A quarkus https://github.com/r-ryantm/nixpkgs/archive/5a02aea8dd3ab0f4298c055184cf8bc9d3f3de4c.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/5a02aea8dd3ab0f4298c055184cf8bc9d3f3de4c#quarkus

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/m7ffsfk9mh5iq8dwrgazm13rxrap540z-quarkus-cli-3.10.1
ls -la /nix/store/m7ffsfk9mh5iq8dwrgazm13rxrap540z-quarkus-cli-3.10.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

quarkus

Maintainer pings
cc @vinetos for testing.

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

1 package built:

quarkus


[iOS-14 Safari iPhone 11] In full screen mode forward and rewind buttons and enabled and functioning.

Describe the bug
We are testing vast in iOS-14 Safari iPhone 11 while testing we observed :
1> Maximize button in enabled while ad is playing. In other players (as mentioned below, maximize button is not enabled in iPhone safari)
2> In maximized mode, forward and rewind buttons are enabled and functioning. User is able to forward and rewind the video ad. It should not happen.
Affected version
v3.0.4
Additional context
Other players where maximize button is not visible in iPhone 11-iOS-14 Safari browser. It is further avoiding to display of forward and rewind button.
https://developers.google.com/interactive-media-ads/docs/sdks/html5/client-side/vastinspector
https://flowplayer.com/developers/tools/ad-tester
PFA

Maximize button in enabled while ad is playing. In other players (as mentioned below, maximize button is not enabled in iPhone safari)

Works as designed - ad might start playback after a time delay, in the middle of video and when the video is already fullscreen.

In maximized mode, forward and rewind buttons are enabled and functioning. User is able to forward and rewind the video ad. It should not happen.

These are controls are provided by iOS, not Fluid Player. Fluid Player on iOS does not use false-fullscreen to avoid issues with user interactions while in fullscreen. Therefore, iOS UX is used in fullscreen and Apple does not allow for scrubbing to be disabled.

These are controls are provided by iOS, not Fluid Player. Fluid Player on iOS does not use false-fullscreen to avoid issues with user interactions while in fullscreen. Therefore, iOS UX is used in fullscreen and Apple does not allow for scrubbing to be disabled.

Hi @Addvilz ,
Is there anyway to disable forward and rewind button during ad playback?
The reason is these controls are being used by the users to forward the ad and this it affecting the ad view-ability.
If not now then could we please look at it's feasibility in future.
Thank you.

These controls are controlled by iOS - there is no way for us to do anything about it.

These controls are controlled by iOS - there is no way for us to do anything about it.

Okay.
Thank you @Addvilz for your quick reply.


Fasta downloads and forwarding

The user should be able to get the sequences for selected genes in a FASTA download.
(Being able to get the fasta sequences for a gene across all the strains would be a huge help.)

Done. Right-click on a feature to download its genomic, CDS, or exon sequences from currently displayed strains.


Prepare website for 2024

This is the layout for 2024.
A empty article page will be created automatically every day by a bot.
Ping @SethTisue

It's deployed: https://scalacenter.github.io/scala-advent-of-code/ :christmas_tree:


Add support for RS being multiple characters (and regex)

Per Arnold Robbins' comment, this "enables many different kinds of processing". See gawk's docs: https://www.gnu.org/software/gawk/manual/html_node/gawk-split-records.html
Presumably it's not too hard to make a bufio.Scanner split function that uses a Go regexp matcher?

Nope, it wasn't too hard to make a bufio.Scanner split function that uses a Go regexp matcher. Done in #86.


Feature requested: Add support and API to initiate remote unprovisioning

Describe the feature
Request adding support for and API endpoint to initiate remote unprovisioning/deactivation.
To Reproduce
Steps to reproduce the behavior:
n/a
Expected behavior
I want a way to unprovision the client remotely, so that I do not have to pass the AMT Admin password to and run the RPC on the endpoint.
Consider using the AMT_SetupAndConfigurationService.Unprovision WSMAN method.
Screenshots 🖼
If applicable, add screenshots to help explain your problem.
AMT Device (please complete the following information): 🖥

OS: [e.g. Linux Kernel & Version]
AMT Version: [e.g. 11.8.5, 12.0.45]
AMT Configuration Mode: [e.g. Admin Control Mode or Client Control Mode]
Network Configuration [e.g. Dynamic IP or Static IP]

Service Deployment (please complete the following information): ⛈

Deployment Type: [e.g. Azure, Docker, K8s]
Node Version: [e.g. LTS 14]
Component & Version: [e.g. RPS 2.0.0]

Additional context
Add any other context about the problem here.
AB#13156

#720 resolved this issue


chore: use map replace switch case

some code refactor, use map to replace switch case.

Closing this PR for the time being. Feel free to create a new PR (or reopen this one) with the one minor fix. The replacement of case statements with maps is not something that we're looking to accept at present.


Test attribute for targeting min and/or max versions of PG.

Allows for less setup than SkippableFact & PG version looked up only once / test run.

@oskardudycz 👍 done (seems to be non-related test failure... Can requeue checks).


Mr.

Description
[Enter feedback here]
Good afternoon sir/ma, I want to change my email ,old is<EMAIL_ADDRESS>new is<EMAIL_ADDRESS>, thanks
Page URL
https://learn.microsoft.com/en-ca/aspnet/core/host-and-deploy/azure-iis-errors-reference?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/host-and-deploy/azure-iis-errors-reference.md
Document ID
daf78291-6119-23c6-e1b1-0606a74bd510
Article author
@Rick-Anderson

@Fapetuade ... This is not the correct place to open issues such as this one and the last one that you opened. This is a documentation repository for the ASP.NET Core documentation. Please only open issues here for the ASP.NET Core documentation. We aren't able to assist with any other requests.


PCC-161 Run resolvable script on a new PID in a prefix

This is a WIP PR to execute pid probe tool command to resolve prefixes through pcc-api.
system.out are kept for checking -will be removed.
need to copy path of daemons-1.0.0-SNAPSHOT.jar locally and add the path as a env parameter at api's application.properties

this close as new pr is opened


Building a CPI for a system that doesn't support disks as independent entities

Hi.
I've been looking into implementing a CPI, but the system that I'm targeting has no concept of disks that exist independently of VMs. You can add a new disk to a VM, and you can remove a disk from a VM, but once it's removed the data is gone.
From what I can tell, BOSH assumes that any provider it uses has this capability. Is what I'm trying to do even possible?
Happy to provide additional details if needed.
Thanks,
Adam.

Hi tintoy,
If I understand you correctly, what you're saying is that all the disks will be ephemeral — your CPI won't have a concept of a persistent disk.
I think you might be able to get away with it by simply no-op'ing the disk management api calls (i.e. create_disk, delete_disk, has_disk, attach_disk, detach_disk, get_disk). And also by making sure no manifests you deploy make use of a persistent disk. It may make it impossible to deploy a BOSH director without some underhanded cleverness, but you could probably deploy all your VMs with the BOSH CLI v2.
Deploying something sophisticated like Cloud Foundry or databases would probably be out of the question (databases tend to depend on persistent data & don't want to lose them due to a re-deploy), but for simple services (e.g. DNS secondary nameservers), not have a persistent disk is fine.

Thanks @cunnie - that's exactly the kind of info I was after! :)

@tintoy do you mind sharing which provider you are trying to integrate with? for production workloads we typically expect that this is possible since moving data around for data services is very expensive (scp, etc.).
as @cunnie mentioned above if you are not using persistent disk functionality in your manifest these CPI methods wont be called. alternatively you might have some luck implementing data movement yourself (within the CPI).

Hi - sorry for the late reply; I missed the notification email.
I work for Dimension Data, and our cloud API doesn't currently expose virtual disks as stand-alone devices. At this stage, I think the CPI option is a non-starter since the client in question wants to run Cloud Foundry. We'll therefore probably host it for them on the underlying (VMWare / Hyper-V) infrastructure. If we do add persistent disk support, I'd love to revisit this and will probably hit you up for some advice when that happens :)

Sounds like the questions have been answered, but please feel free to join us in cloudfoundry#bosh or create new issues if you revisit this and have more questions!

Thanks, @dpb587-pivotal :)


Don't check groups with null data.

getChildDevices() returns groups that have been deleted from Hue, but not yet from smartthings. I forgot that I deleted a group that I had on smartthings and it was driving me crazy because I couldn't refresh hue b smart, as it kept loggings errors about how light group 15 returned null data, even though GET groups was only returning up to group 14, as I had deleted group 15 from hue.
This just checks for null data and skips that group if necessary.  :)

Hi thats great! will this apply to bulbs and scenes too? this is what stops most people from using the app after a while so it will be great to add this will wait for your opion on bulbs and scenes before doing a pull request

Oh yeah, I can add in scenes/bulbs as well, not a problem.

Should be good to go now.


Refactor of box shadow sprinkles

Description
boxShadow sprinkles should be inferred from the outline colors, without the need to set theme variables twice, one for the outline prop (with only the color), one for the box-shadow prop (with shadow params + color).
We should define a set of possible shadow "configurations" (like inset, insetStrong, insetBottom, ...) and mix them with the outline colors to have resulting sprinkles values for boxShadow like outlineDecorativeInset, outlineDecorativeInsetStrong, ....

Closing as this will be part of the new theme / sprinkles redesign


Update build workflow to include Wazuh plugins

Description
This PR updates the build.yml workflow to build and install Wazuh Indexer plugins into the wazuh-indexer distribution packages.
Issues Resolved
Partially solves https://github.com/wazuh/wazuh-indexer-plugins/issues/13
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Superseded by #360


Can't change token display type in TokenizedTextView

InstructionTextToken objects that I create with PossibleAddressToken or IntegerToken type cannot change their display type in a TokenizedTextView. However, I've noticed that if I just pass through one of these tokens from a DisassemblyTextRenderer, it will change display types as expected.
I'm not seeing any difference between the attributes of the tokens I generate vs the ones that come from the DisassemblyTextRenderer so I'm at a loss as to what is causing the issue; it's currently forcing me to manually look up symbols and set the token type to CodeSymbolToken or ImportToken if a symbol exists, which is probably not super efficient.

It is intentional that TokenizedTextView does not modify the tokens in any way, because we do not want to force a single use case of this view. Tokens passed to the view are shown as provided to the API, and this will not be changed. DisassemblyTextRenderer is the class responsible for display types and symbol name replacement. Unfortunately, the API was not flexible for certain custom views, such as those that wish to replace the IL rendering with their own.
A new API needed during HLIL development has been backported to dev that should help fix some of these use cases. Try the DisassemblyTextRenderer.post_process_lines API, which processes a provided set of tokens in the same way that normal disassembly does. Available in build 1808.


Fixing import

Hi @blakeembrey ,
Please review

No, this isn’t correct and I won’t be merging it. You can read more about how ESM and imports work in the node documentation.

Specifically this part for docs: https://nodejs.org/api/packages.html#subpath-exports


Allow /closeticket to be used in channels other than the ticket channels themselves

Not really necessary I think


how to resolve dialogs ?

I've created a bot. This bot should handle multiple instances of Facebook pages. Everything works fine until "second bot" try post a message with some channel data:
Code :
 [BotAuthentication(CredentialProviderType = typeof(MultiCredentialProvider))]
        public async Task<HttpResponseMessage> Post([FromBody]Activity activity)
        {

            if (activity.Type == ActivityTypes.Message)
            {
                try
                {
                    await Conversation.SendAsync(activity, () => new MenuDialog());
                }

When MenuDialog() is created :
case // this case method creates a new Dialog :
context.Call(new AboutDialog(), AboutDialog.Resume);
break;

Then, in this Dialog :
When user has chosen some option :
I'm creating a message :
var msg = context.MakeMessage();
than :
try
            {
                await context.PostAsync(msg);
            }
            catch (Exception e)
            {
                await context.PostAsync(e.StackTrace.ToString());
            }

This context.PostAsync throws an exception
Stack trace :

at Microsoft.Bot.Connector.ErrorHandling.d2`1.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.Bot.Connector.ConversationsExtensions.d7.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.Handl
eNonSuccessAndDebuggerNotification(Task task)    at Microsoft.Bot.Builder.Dialogs.Internals.AlwaysSendDirect_BotToUser.d4.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.Bot.Builder.Dialogs.Internals.MapToChannelData_BotToUser.d3.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at Syst
em.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.Bot.Builder.Dialogs.Internals.LogBotToUser.d*4.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Microsoft.Bot.Builder.Dialogs.Internals.DialogContext.<Microsoft-Bot-
Builder-Dialogs-Internals-IBotToUser-PostAsync>d*12.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at MyApp.d__8.MoveNext()

Any idea ?

I fixed this by adding url to white-listed domains. This topic can be closed.


fix typo and upgrade dependencies

TESTED=app builds and runs as expected

This does not compile for me:
app/src/main/java
com/android/example/cameraxbasic/fragments/CameraFragment.kt
Object is not abstract and does not implement abstract member public abstract fun onError(@NonNull p0: ImageCapture.ImageCaptureError, @NonNull p1: String, @Nullable p2: Throwable?): Unit defined in androidx.camera.core.ImageCapture.OnImageSavedListener
'onError' overrides nothing
Unresolved reference: UseCaseError
com/android/example/cameraxbasic/fragments/GalleryFragment.kt
Cannot inline bytecode built with JVM target 1.8 into bytecode that is being built with JVM target 1.6. Please specify proper '-jvm-target' option

can't believe I missed it. Just updated the code. thx!

Application now crashes, at least on Pixel 3 XL device:
E/AndroidRuntime: FATAL EXCEPTION: LuminosityAnalysis
    Process: com.android.example.cameraxbasic, PID: 18995
    java.lang.IllegalStateException: Not in application's main thread
        at androidx.core.util.Preconditions.checkState(Preconditions.java:96)
        at androidx.camera.core.impl.utils.Threads.checkMainThread(Threads.java:52)
        at androidx.camera.core.ImageAnalysis.getAnalyzer(ImageAnalysis.java:180)
        at com.android.example.cameraxbasic.fragments.CameraFragment$bindCameraUseCases$1$1.invoke(CameraFragment.kt:314)
        at com.android.example.cameraxbasic.fragments.CameraFragment$bindCameraUseCases$1$1.invoke(CameraFragment.kt:89)
        at com.android.example.cameraxbasic.fragments.CameraFragment$LuminosityAnalyzer.analyze(CameraFragment.kt:468)
        at androidx.camera.core.ImageAnalysisAbstractAnalyzer.analyzeImage(ImageAnalysisAbstractAnalyzer.java:57)
        at androidx.camera.core.ImageAnalysisNonBlockingAnalyzer$1.run(ImageAnalysisNonBlockingAnalyzer.java:135)
        at android.os.Handler.handleCallback(Handler.java:883)
        at android.os.Handler.dispatchMessage(Handler.java:100)
        at android.os.Looper.loop(Looper.java:214)
        at android.os.HandlerThread.run(HandlerThread.java:67)

I think this is related to some internal changes with thread management for image analyzer. Please test on a couple of real devices, in addition to the emulator.

interesting. it looks like the callBackHandler for Analyzer has to be on the Main Thread now for some reason :/

I don't think the issue is with the thread the analyzer runs in, but rather when the analyzer is accessed later to log the results. We can keep this for the time being and add the background thread later after we investigate further.
Please address comments and rebase with the tip of master, then we can merge this.

Sounds good!
rebased/squashed all commits onto master.
thx!


Update dataflow worker container version

Update worker container to include latest changes: https://github.com/apache/beam/pull/8217
R: @lukecwik

Run Gradle Publish

Run Java_Examples_Dataflow PreCommit

Run Python PreCommit

Run Java_Examples_Dataflow PreCommit

Checked on the job and it actually succeeded.


Validate all parameters

All parameters should be validated.
Although its going to make a huge mess in the init.pp

Do you mean validate_*? I agree.

I think we should cleanup voxpupuli/stash and use this as baseline for voxpupuli/confluence and voxpupuli/jira.

@dhoppe do it so, :hash: :one:

Types in master now, not needed anymore.


Update log4j-slf4j2-impl to 2.21.1

About this PR
📦 Updates org.apache.logging.log4j:log4j-slf4j2-impl from 2.20.0 to 2.21.1
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.logging.log4j", artifactId = "log4j-slf4j2-impl" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.apache.logging.log4j", artifactId = "log4j-slf4j2-impl" }
}]

labels: test-library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #155.


[Bugfix] Fix DictMappingExpr rewrite multi times

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：
Fixes https://github.com/StarRocks/StarRocksBenchmark/issues/173
Problem Summary(Required) ：
Because Expr and ExprContext are shared in OlapScanOperator.
And OlapScanPrepareOperator::pull_chunk() will be executed concurrently,
which can cause the rewrite function to be executed multiple times.
We should use std::call_once to avoid multiple rewrites

need the test cases

case https://github.com/StarRocks/StarRocksTest/pull/813

run starrocks_admit_test


Feat/bastion

Integration of the new Bastions Module
Summary
The new module "Bastions" allows armies of enemy orders to go to war against each other on new locations called Bastions. These bastions are situated on the border between orders and are composed of 1 central square, 4 towers and 1 staging area. The staging area is the default location when an army travels to the bastion, it’s a safe space where no armies can get attacked. After arriving on the staging area, armies can then move to different locations. The towers are locations where combat can happen, and they can be conquered when all armies of the location’s defending order are annihilated. The central square can only be reached by an order who has conquered all 4 towers. In order to incentivise fighting over bastions, an order holding the central square will receive bonuses.
More info on the rules can be found here: https://discord.com/channels/884211910222970891/1073613664101343283/1076551940328140800
Moving Time
I explicitly use "move" instead of "travel" in order to differentiate with the Travel Module. Armies can move between locations in the bastion, but each move takes a different number of blocks depending on the distance. In order to tackle this in the contract, I use the concept of "settled" vs "unsettled" armies. When armies move using the "bastion_move" entrypoint, the arrival block is stored in the current_movers storage_var. These armies are considered "unsettled", meaning that the contract does not know yet if they have arrived or not. Only when new armies use "bastion_move" does the contract verify the arrival of some armies and updates a counter storage_var containing the number of armies present on a location. This counter is important to keep track of the number of armies defending a location.
More info on how it works with examples here: https://www.notion.so/Solving-Travel-Time-Option-3-6673b997d64f48d58374587fc7deb500
Combat & Travel
The bastions module uses 2 new features recently included in the main branch.

Combat: "initiate_combat_approved_module" a general combat entrypoint to initiate combat between armies outside of Realms
Travel: "forbid_travel" and "allow_travel" entrypoints to forbid an army from travelling when defending a bastion

Contains

spawn_bastions entrypoint, callable only by admin, to create bastions on the map
bastion_attack entrypoint to attack an army, if that army is the last one of the defending order, the attacker’s order becomes the new defending order
bastion_move entrypoint to move your army on the different locations
bastion_take_location entrypoint to conquer a location that does not have any defender

Testing
All the entrypoints have been tested in different scenarios. I've also used 2 mockup contract for travel and combat to simulate their behavior.

just realised I didnt take into account moves from central square to tower, will fix asap


Can't remove migration after adding

dotnet ef migrations remove
after
dotnet ef migrations add
throws the following exception.
Exception message:
Login failed for user 'DOMAIN\myUserName'.
Stack trace:
System.Data.SqlClient.SqlException (0x80131904): Login failed for user 'DOMAIN\myUserName'.
   at System.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, Object providerInfo, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling)
   at System.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionFactory.CreatePooledConnection(DbConnectionPool pool, DbConnection owningObject, DbConnectionOptions options, DbConnectionPoolKey poolKey, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionPool.CreateObject(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
   at System.Data.ProviderBase.DbConnectionPool.UserCreateRequest(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
   at System.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, UInt32 waitForMultipleObjectsTimeout, Boolean allowCreate, Boolean onlyOneCheckConnection, DbConnectionOptions userOptions, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionClosed.TryOpenConnection(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at System.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)
   at System.Data.SqlClient.SqlConnection.Open()
   at Microsoft.EntityFrameworkCore.Storage.RelationalConnection.Open(Boolean errorsExpected)
   at Microsoft.EntityFrameworkCore.Storage.Internal.SqlServerDatabaseCreator.<>c__DisplayClass18_0.<Exists>b__0(DateTime giveUp)
   at Microsoft.EntityFrameworkCore.ExecutionStrategyExtensions.<>c__DisplayClass12_0`2.<Execute>b__0(DbContext c, TState s)
   at Microsoft.EntityFrameworkCore.Storage.Internal.SqlServerExecutionStrategy.Execute[TState,TResult](TState state, Func`3 operation, Func`3 verifySucceeded)
   at Microsoft.EntityFrameworkCore.ExecutionStrategyExtensions.Execute[TState,TResult](IExecutionStrategy strategy, Func`2 operation, Func`2 verifySucceeded, TState state)
   at Microsoft.EntityFrameworkCore.ExecutionStrategyExtensions.Execute[TState,TResult](IExecutionStrategy strategy, TState state, Func`2 operation)
   at Microsoft.EntityFrameworkCore.Storage.Internal.SqlServerDatabaseCreator.Exists(Boolean retryOnNotExists)
   at Microsoft.EntityFrameworkCore.Storage.Internal.SqlServerDatabaseCreator.Exists()
   at Microsoft.EntityFrameworkCore.Migrations.HistoryRepository.Exists()
   at Microsoft.EntityFrameworkCore.Migrations.HistoryRepository.GetAppliedMigrations()
   at Microsoft.EntityFrameworkCore.Migrations.Design.MigrationsScaffolder.RemoveMigration(String projectDir, String rootNamespace, Boolean force)
   at Microsoft.EntityFrameworkCore.Design.Internal.MigrationsOperations.RemoveMigration(String contextType, Boolean force)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.RemoveMigrationImpl(String contextType, Boolean force)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.RemoveMigration.<>c__DisplayClass0_1.<.ctor>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.<>c__DisplayClass3_0`1.<Execute>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.Execute(Action action)
ClientConnectionId:b1cba61c-c324-413d-a208-b08d0ea890c1
Error Number:18456,State:1,Class:14

Since I was able to create the Migration, I'd assume to be able to remove it again as well.
Steps to reproduce

Add Migration for database, where you have no Access rights.
Remove Migration again

Further technical details
EF Core version: 2.0.1
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Win10-x64
IDE: cmd.exe

Add migration does not need connection to database since it is only computing difference between current model & model from last migration (stored in modelsnapshot).
Whereas remove migration needs connection to database because it needs to verify that you are not reverting a migration which is already been applied to database. If the migration is already being applied to database then you first need to "unapply" the migration on database first before removing. This is necessary. If we remove a migration which is already applied to database then your database is out of sync with migrations causing failures in any subsequent operation.

To add to what @smitpatel said above, there is also a "force" command line option that can be used to force the remove even if the database cannot be reached.

Thanks for making the decision clear and mentioning the force Switch :)


How severe is known issue 776516

It is not clear to me how severe known issue 776516 is in a high-performance / high-availability productive environment. Could the problem potentially arise in any ASP.NET (MVC) application, or only if we use those proxy monitors (whatever that is?). Should we wait for the hotfix before upgrading ASP.NET applications to v4.8?

This issue was introduced in 4.7, so if you're already on 4.7 or later, you already have the bug. A fix for 4.8 was released in a rollup on June 18th. (See https://support.microsoft.com/en-us/help/4502560/cumulative-update-for-net-framework-june-18-2019 for the Win10 flavor of this rollup.) I believe a 4.7.2 rollup is due shortly as well.
WRT "how severe" the issue is, I believe your question is about how likely it is that you might eoncounter the issue. All ASP.Net 4.7+ applications use the code path in question by default. Even MVC apps. It is code that supports an intrinsic housekeeping function of the ASP.Net environment. You are not very likely to hit it in the majority of cases. The environmental factors that increase your risk of hitting this bug are running in an application pool that hosts multiple app domains, or that frequently updates/restarts an app. Also, if your application pool pushes the limits for private bytes usage.


Support activity functions with variadic arguments

Is your feature request related to a problem? Please describe.
I need to make gRPC function calls from my workflows. All gRPC client functions generated by protoc have the variadic  parameter grpc.CallOption:
type SomeClient interface {
	SomeMethod(ctx context.Context, in *SomeRequest, opts ...grpc.CallOption) (*SomeResponse, error)
}

I am able to register SomeMethod as an activity but when I try calling it from a workflow, passing an empty array as the last argument, I get an error:
reflect: cannot use []grpc.CallOption as type grpc.CallOption in Call

I tried different ways of passing the opts parameter (omitting, null, a single instance of grpc.CallOption) but neither of these ways have worked.
The issue is generic and applies to all variadic functions regardless of parameter type. I was able to reproduce the problem by modifying GreetingSample and adding a variadic argument to the GetGreeting activity.
Describe the solution you'd like
I can see two ways of implementing this feature:

Pass variadic args as an array to ExecuteActivity().
Pass variadic args as individual parameters to ExecuteActivity().

Upon examining source code, The first option needs a change to executeFunction() function in internal_worker.go to unpack the array. The second option requires a change to decodeArgsToRawValues(), also in internal_worker.go. I am not sure which option is better.
Describe alternatives you've considered
I was able to implement a workaround by creating a utility function:
func wrapGrpcActivity[P any, R any](f func(ctx context.Context, req *P, opts ...grpc.CallOption) (*R, error)) func(ctx context.Context, req *P, opts []grpc.CallOption) (*R, error) {
	return func(ctx context.Context, req *P, opts []grpc.CallOption) (*R, error) {
		return f(ctx, req, opts...)
	}
}

Then I use the function when registering gRPC methods:
worker.RegisterActivityWithOptions(wrapGrpcActivity(someClient.SomeMethod), activity.RegisterOptions{Name: "SomeMethod"})

and I can invoke the activity as follows:
err := workflow.ExecuteActivity(ctx, "SomeMethod", &some.SomeRequest{}, nil).Get(ctx, &resp)

Additional context
I would like to add that supporting variadic args would allow developers more easily integrate existing third party APIs as activities into workflows without additional wrapping.

Variadic arguments are supported I believe if you pass the slice instead of just a single value. I think this a reasonable compromise currently.
Your code will not work anyways. grpc.CallOption is not a convertible/serializable value. You should use a wrapper, you cannot expose gRPC client calls directly with their non-serializable arugments as activities.

@cretz Let me answer the two points you made separately. First, as I had indicated, this is a generic issue, not specific to gRPC. I was able to reproduce it with an example GreetingService. If I add a variadic parameter:
func (a *Activities) GetGreeting(p ...string) (string, error) {

and then change GreetingSample to pass this parameter:
err := workflow.ExecuteActivity(ctx, a.GetGreeting, []string{"a", "b"}).Get(ctx, &greetResult)

I get an error:
reflect: cannot use []string as type string in Call

There is a stack trace:
reflect.Value.call({0x103142400?, 0x140002a05c0?, 0x12af355e8?}, {0x102f8ba78, 0x4}, {0x140000a1428, 0x1, 0x0?})
        /opt/homebrew/opt/go/libexec/src/reflect/value.go:453 +0x13c0
reflect.Value.Call({0x103142400?, 0x140002a05c0?, 0x14000447778?}, {0x140000a1428?, 0x140000a4310?, 0x9?})
        /opt/homebrew/opt/go/libexec/src/reflect/value.go:370 +0x90
go.temporal.io/sdk/internal.executeFunction({0x103142400, 0x140002a05c0}, {0x140001e8950, 0x1, 0x103275101?})
        /Users/alex/go/pkg/mod/go.temporal.io/sdk@v1.22.2/internal/internal_worker.go:1796 +0x318
go.temporal.io/sdk/internal.executeFunctionWithContext({0x1032751b8?, 0x140004403c0}, {0x103142400, 0x140002a05c0}, {0x140001e8950, 0x1, 0x1})
        /Users/alex/go/pkg/mod/go.temporal.io/sdk@v1.22.2/internal/internal_worker.go:1780 +0x160
go.temporal.io/sdk/internal.(*activityEnvironmentInterceptor).ExecuteActivity(0x140000b6a80, {0x103275180?, 0x14000440360?}, 0x140000a1410)

@cretz For your second point, I agree that grpc.CallOption is not convertible, and I would need a wrapper if I had wanted to use it. However, I don't want to pass any call options, and in many cases these call options aren't being used, in which case convertibility isn't an issue.

:+1: Yeah we call https://pkg.go.dev/reflect#Value.Call not https://pkg.go.dev/reflect#Value.CallSlice. I assume workflow.ExecuteActivity(ctx, a.GetGreeting, "a", "b").Get(ctx, &greetResult) also doesn't work. I think we can support variadic args, but I think for compatibility it may have to be with how Call does it (so making what I put there work).
I wouldn't count on non-convertible optional variadic options always being safe. We may have static analyzers or other things in the future that check entire signatures eagerly (e.g. at registration).

Yes, the second form (err := workflow.ExecuteActivity(ctx, a.GetGreeting, "a", "b").Get(ctx, &greetResult)) also doesn't work. It fails with a different error:
Error unable to decode the activity function input payload with error: payload item 0: unable to decode: json: cannot unmarshal string into Go value of type []string for function name: GetGreeting


MSFT_SPInstallPrereqs: MSVCRT11 missing

Details of the scenario you tried and the problem that is occurring
Verbose logs showing the problem
When installing SharePoint 2019 the prereq MSVCRT11 is not required
Suggested solution to the issue
Add the MSVCRT11 to the requirements of SharePoint 2019
The DSC configuration that is used to reproduce the issue (as detailed as possible)
# insert configuration here
`PInstallPrereqs InstallPrereqs {
            IsSingleInstance  = "Yes"
            Ensure            = "Present"
            InstallerPath     = "media\Sharepoint2019\PrerequisiteInstaller.exe"
            OnlineMode        = $false
            SXSpath           = "\media\SXS"
            KB3092423         = "\media\PreRequirements\sp2019\AppFabric-KB3092423-x64-ENU.exe"
            IDFX11            = "media\PreRequirements\sp2019\MicrosoftIdentityExtensions-64.msi"
            MSIPCClient       = "\\media\PreRequirements\sp2019\setup_msipc_x64.exe"
            SQLNCli           = "\media\PreRequirements\sp2019\sqlncli.msi"
            WCFDataServices56 = "media\PreRequirements\sp2019\WcfDataServices.exe"
            AppFabric         = "media\PreRequirements\sp2019\WindowsServerAppFabricSetup_x64.exe"
            DotNet472     = "\media\PreRequirements\sp2019\NDP472-KB4054530-x86-x64-AllOS-ENU.exe"
            Sync              = "\media\PreRequirements\sp2019\Synchronization.msi"
            MSVCRT11          = "\media\PreRequirements\sp2019\vcredist_x64.exe"
            MSVCRT141          = "\media\PreRequirements\sp2019\vc_redist.x64.exe"
        }
#### The operating system the target node is running

OsName               : Microsoft Windows Server 2016 Standard
OsOperatingSystemSKU : StandardServerEdition
OsArchitecture       : 64-bit
WindowsBuildLabEx    : 14393.2608.amd64fre.rs1_release.181024-1742
OsLanguage           : en-US
OsMuiLanguages       : {en-US}

#### Version of SharePoint that is used (e.g. SharePoint 2016)
SharePoint 2019
#### Version and build of PowerShell the target node is running

Name                           Value
----                           -----
PSVersion                      5.1.14393.2608
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.2608
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

#### Version of the DSC module that was used ('dev' if using current dev branch)
3.0

Have potentially identified the cause of this issue and am working with @BBokdam on testing a solution. Will include the fix in my next Bugfix PR.

Solution works. Will be included in my next Bugfix PR


Add missing models for saving ZkLend events data into db

Read our Contributor Guidelines and README.
Create new models for missing events here Don't forget to inherited from EventBaseModel
Here you can find models and their fields
Please, create new models for next events:

RepaymentEventData
DepositEventData
BorrowingEventData
WithdrawalEventData
CollateralEnabledDisabledEventData

Generate migration file
How to run migration you can find in README.md
If any blocker or issues, don't hesitate to text me in telegram: djeck_vorobey1

I  would like to take this task, I love 🍍, kindly assign me i would like to take on this. ETA <= 24hrs

would like to take this task, I love 🍍,
ETA 24hrs

Can I take care of this issue?  I love 🍍,

Can I contribute to this one? I love 🍍

@petersdt will you manage to create today draft PR, the timeline is changed, so you have a limited time to complete it.
12 hours for draft PR and 12 hours to complete it

Can I contribute to this one? I love 🍍
i will create a pr before EOD

Could I take over this issue? i can work with it within 12 hours for draft PR and 12 hours to complete it .I love 🍍 btw also i have experience in this repo.

@djeck1432. I would have been capable of handling the task. Please let me know if any other opportunities are available, as I am open to taking on task.


Bad word filtering

Features:

Variable for storing the phrase used to replace problematic comments
Words taken from text file, allowing for new words to be added when needed.
Tidying up of code and re-arranging some stuff to make it easier to read.

Future Implementation:

Possible commands for mods to add banned words mid-stream

Sorry about the first closed one, Never branched anything on GitHub before. A lot of this is just reshuffling of the code, don't worry nothings been taken out. Honest.
And yes a lot of this can be done better/more efficiently but it's a tiny script and doesn't need that level of smart-arse oversight. Plenty of coding jobs don't do all the fancy stuff these boys insist is necessary because its not compatible with older machines, browsers, or just isn't worth the effort worrying about.
If this is shite don't harass me, I'm NHS (Modern day WW2 veteran) and that would be unspeakably disrepectful and shameful.

Thought police.

Worse, a mere lacky to the thought police

the hotseat  is wide open for abuse, so something like this feels like a sensible idea, but I just wonder how useful it would be in practice. If someone gets a sniff of there being a filter, they could just be "clever" and bypass it with spaces or special characters. Or they simply use an offensive term that the streamer hasn't thought of.

I won't be including filtering, I'll leave that up to streamers to use a mod bot or mod settings in chat.

rofl all yer work down the pissa

rofl all yer work down the pissa

That's why you have discussions and plans before doing the work :stuck_out_tongue:

There was a discussion thread up about the idea that seemed to be going places. Didn't take long to make a rough implementation which could be iterated on so no-ones gonna lose sleep over it. Big issue is not really knowing what's available to the twitch bigshots in terms of moderation as a viewer.


[tool_crash] FileSystemException: readSync failed, OS Error: Invalid argument, errno = 22

Command
flutter build apk

Steps to Reproduce

...
...
...

Logs
FileSystemException: readSync failed, OS Error: Invalid argument, errno = 22
#0      _RandomAccessFile.readSync (dart:io/file_impl.dart:763:7)
#1      _File.readAsBytesSync (dart:io/file_impl.dart:552:23)
#2      ForwardingFile.readAsBytesSync (package:file/src/forwarding/forwarding_file.dart:88:43)
#3      ForwardingFile.readAsBytesSync (package:file/src/forwarding/forwarding_file.dart:88:43)
#4      _calculateSha (package:flutter_tools/src/android/gradle.dart:759:32)
#5      AndroidGradleBuilder.buildGradleApp (package:flutter_tools/src/android/gradle.dart:489:34)
<asynchronous suspension>
#6      AndroidGradleBuilder.buildApk (package:flutter_tools/src/android/gradle.dart:184:5)
<asynchronous suspension>
#7      BuildApkCommand.runCommand (package:flutter_tools/src/commands/build_apk.dart:109:5)
<asynchronous suspension>
#8      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1183:27)
<asynchronous suspension>
#9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#10     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#11     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
<asynchronous suspension>
#12     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#13     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
<asynchronous suspension>
#14     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.0.1, on macOS 12.3.1 21E258 darwin-arm, locale zh-Hans-CN)
    • Flutter version 3.0.1 at /usr/local/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fb57da5f94 (2 天前), 2022-05-19 15:50:29 -0700
    • Engine revision caaafc5604
    • Dart version 2.17.1
    • DevTools version 2.12.2
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] And

my project is running in good condition before add many assets, which are hundreds of audios files, totally around 3.75GB .
and when I removed these assets, and commented the import in pubspec.yaml as below
`  assets:
- assets/images/
- data/audios/
`
then my project builds normally as before.
so I think the problem is caused by these audios assets files with big size, but don't know how to solve it?

Hi @horo99, there's a similar issue describing what you're experiencing in https://github.com/flutter/flutter/issues/94324.
Kindly follow up there for more information.
Closing from here as duplicate.
Thank you


Allow sub directories for components

Wouldn't it be interesting to enable the creation of subfolders inside components?
It is currently possible as long as we create the folders in advance.
The "event" folder was previously created

...> python manage.py  startunicorn "ABXSite" event\schedule
Created ...\components\event\schedule.py.
Created ...\templates\unicorn\event\schedule.html.

However, you cannot use components in subdirectories, perhaps a schema like:
{% unicorn "folder:component" %}
As in {% url "appname:view" %}

Nested components are supported. You can see an example in https://github.com/adamghill/django-unicorn/tree/master/example/unicorn/components/nested and https://github.com/adamghill/django-unicorn/tree/master/example/unicorn/templates/unicorn/nested. You use a . (like Python namespaces) to access components in sub-folders. An example of that is: https://github.com/adamghill/django-unicorn/blob/a54cdaf933b7541601029aaa84e823af595e5c26/example/unicorn/templates/unicorn/nested/table.html#L19.
However, I'm not sure if the startunicorn supports creating folders for nested components, but that seems relatively easy to add in if it would be useful. And I can add some documentation around this capability as well!

This has been merged into master and will be released in the next version.


Get rid of direct eval

https://github.com/skale-me/skale-engine/blob/907075ad2e1bc206986192e7bed155da581123c1/bin/worker.js#L204
Direct eval is dangerous. Regexp are terrible.
"use strict";

var str = "(() => { fs = 42 })()"
var fs = {};

console.log('pass', !!(str.match(/^\s*\([^)]*\)\s*=>/))) // passing the second regexp test
console.log('fs', fs) // {}
eval(str)
console.log('fs', fs) // 42. The direct-evaled code modified a variable value outside its scope

Among other things, someone can send a message like the one I showed and read and write to any variable of worker.js.
I have not been able to assess how much damage can be done via this method, but it looks like at least the entire filesystem is compromised. Maybe it's possible to compromise the master, not clear.
At least, you should be evaling via an indirect eval :
("indirect", eval)(str) or var indirectEval = eval; indirectEval(str).
But even then it's not sure it's a good idea. Maybe the new Function way should be the only allowed way to send functions. People won't die from typing function.

Thanks for identifying this. I agree on using indirect eval instead of eval, so eval behaves like new Function. Regexp should also filter any invalid function definition. Additionally, we should also check that parameters of sources, actions and transformations which are supposed to be function definitions really are: (typeof arg === 'function'). This would catch potential problems at the source, and be done on master side. Ultimately, proper use of const would also forbid var overwriting. This will be done later.


Return Error if CKZG or BLS features are disabled, but are called.

The precompile address should be specified regardless, but with an implementation that panics at run time so that certain bugs are caught early.

Hi, willing to work on this. Can you share some pointers please?

Sure, in revm-precompile lib.rs, instead of conditionally adding precompiles with cfg, they should always be added but with a fallback implementation that panics at runtime.
For example in cancun
https://github.com/bluealloy/revm/blob/30c6362bd4676911fa8f3306b4d8311cee340931/crates/precompile/src/lib.rs#L140-L158
the extend should always be enabled, and kzg_point_evaluation::POINT_EVALUATION should be cfg'd internally, with a panic if the feature is not enabled

Would not panic on those. Seems like bad behaviour

Renamed the issue, we should return the error on this.

We should modify PrecompileResult to contain additional field that would return panic like error. https://github.com/bluealloy/revm/blob/30c6362bd4676911fa8f3306b4d8311cee340931/crates/primitives/src/precompile.rs#L9C29-L9C67
Would add something like:
enum PrecompileResult {
    Ok {
      gas_used: u64,
      output: Bytes,
    },
    Error {
        error_type: PrecompileError,
    },
    FatalError {
        msg: String,
    }
}

Fatal error would stop execution, while Ok and Error would work as previously with Result<>.

ref: https://github.com/bluealloy/revm/issues/1283

Implemented for kzg here: https://github.com/bluealloy/revm/pull/1589
blst pending and require some code refactoring


Max. byte count for FC 16 in Modbus server

Why is the max. byte count for FC 16 requests in the Modbus server limited to < 242?
../server/mod.rs:
            MODBUS_SET_COILS_BULK | MODBUS_SET_HOLDINGS_BULK => {
                // funcs 15 & 16
                // write multiple coils / registers
                let bytes = self.buf[self.frame_start + 6];
                if !check_frame_crc!(7 + bytes) {
                    return Err(ErrorKind::FrameCRCError);
                }
                if !broadcast {
                    self.response_required = true;
                }
                if bytes > 242 {  // <== THIS LINE
                    self.error = MODBUS_ERROR_ILLEGAL_DATA_VALUE;
                    return Ok(());
                }

If you look into the current Modbus application protocol (V1.1b3), there is denoted a max. quantity of registers for FC 16 of 0x007b. Because the byte count is twice of the register count, the max. allowed byte value should be 246. Or am I missing something?

Hi. You are correct, bulk set/get registers is limited to 123 (mathematically even to 124 but most of sources perfer 123). I gonna correct this to set 246 bytes there

fixed in 0.7.4