BaseSettings env_prefix also apply for secrets

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
The env_prefix is used for both env vars and secrets, which makes it impossible to combine env vars (with prefix) and secrets (without prefix) into a single settings model.
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.7.2
            pydantic compiled: False
                 install path: /usr/local/src/pydantic/pydantic
               python version: 3.7.7 (default, Mar 11 2020, 00:27:03)  [GCC 8.3.0]
                     platform: Linux-5.4.0-54-generic-x86_64-with-debian-10.3
     optional deps. installed: []

Environment vars:
printenv
RESTAPI_USE_PRODUCTION_DB=1
RESTAPI_DB_READ_ONLY=0

Secrets in container:
ls /run/secrets/
db_read_only_password  db_read_only_user  db_read_write_password  db_read_write_user

from pydantic import BaseSettings

class DBSettings(BaseSettings):
    use_production_db: bool = False     # env var
    db_read_only: bool = False          # env var
    db_read_only_user: str = None       # secret
    db_read_only_password: str = None   # secret
    db_read_write_user: str = None      # secret
    db_read_write_password: str = None  # secret

    class Config:
        env_prefix = 'RESTAPI_'
        secrets_dir = '/run/secrets'

db_sett = DBSettings()

Tested without the env_prefix = 'RESTAPI_' line, which filled the settings from the secrets correctly, env vars were not read as expected.
Possible solutions

Make it clear in the documentation that the env_prefix is also used for secrets
Just do not use the env_prefix for the secrets
Make an secrets_prefix to be used as secrets prefix

Hi @JerPk and thanks a lot for reporting!
IMO you're right it's a bug. The behaviour with both secrets and env_prefix was not tested so I guess that's why we never saw it.
I chose the 2nd solution on the 3 you suggested because for me the 1st doesn't make sense (it shouldn't use env_prefix when we talk about secrets) and the 3rd doesn't really make sense when you already have folders!
Feedback more than welcome :)
Cheers

Hi @PrettyWood, I didn't expect such a fast response and implementation.
I completely agree with the solution. (although I will need to change some things in my code when I am updating to this future version.)
Thanks!

Similar issue exists, if "env" is set for a field.
class Settings(BaseSettings):
    auth_key: str
    api_key: str = Field(..., env="my_api_key")

    redis_dsn: RedisDsn = "redis://user:pass@localhost:6379/1"
    pg_dsn: PostgresDsn = "postgres://postgres:password@localhost:5432/postgres"

    # to override domains:
    # export my_prefix_domains='["foo.com", "bar.com"]'
    domains: Set[str] = set()

    # to override more_settings:
    # export my_prefix_more_settings='{"foo": "x", "apple": 1}'
    more_settings: SubModel = SubModel()

    class Config:
        # env_prefix = "my_prefix_"  # defaults to no prefix, i.e. ""
        fields = {
            "auth_key": {"env": "my_auth_key",},
            "redis_dsn": {"env": ["service_redis_dsn", "redis_url"]},
        }
        secrets_dir = "/var/run"

My secret had to be renamed to my_auth_key to succeed.

Similar issue exists, if "env" is set for a field.
class Settings(BaseSettings):
    auth_key: str
    api_key: str = Field(..., env="my_api_key")

    redis_dsn: RedisDsn = "redis://user:pass@localhost:6379/1"
    pg_dsn: PostgresDsn = "postgres://postgres:password@localhost:5432/postgres"

    # to override domains:
    # export my_prefix_domains='["foo.com", "bar.com"]'
    domains: Set[str] = set()

    # to override more_settings:
    # export my_prefix_more_settings='{"foo": "x", "apple": 1}'
    more_settings: SubModel = SubModel()

    class Config:
        # env_prefix = "my_prefix_"  # defaults to no prefix, i.e. ""
        fields = {
            "auth_key": {"env": "my_auth_key",},
            "redis_dsn": {"env": ["service_redis_dsn", "redis_url"]},
        }
        secrets_dir = "/var/run"

My secret had to be renamed to my_auth_key to succeed.

Proposal
Keep the solution as is now - thus use the environment variable name in all situations - not only for trying to read values from env var (and from dotenv file), but also from secrets.
Action needed: update the docs to clarify this rule.
Reasoning
Keep things simple. With unique naming rule for named configuration sources it is much simpler to document the applicaiton - simply give single set of names and add, that they can be provided via env vars, dotenv file or via secrets in a directory.
Another advantage: all existing deployment will work.

Proposal
Keep the solution as is now - thus use the environment variable name in all situations - not only for trying to read values from env var (and from dotenv file), but also from secrets.
Action needed: update the docs to clarify this rule.
Reasoning
Keep things simple. With unique naming rule for named configuration sources it is much simpler to document the applicaiton - simply give single set of names and add, that they can be provided via env vars, dotenv file or via secrets in a directory.
Another advantage: all existing deployment will work.

To quote @JerPk we have two solutions

Make it clear in the documentation that env_prefix is also used for secrets
Do not use the env_prefix for the secrets

I thought option 2 made more sense but reading @vlcinsky opinion, I understand it can be wanted.
I would like to add option 3

Try with and without env_prefix

This doesn't break anything and allows both behaviours but can have side effects and is less clear.
I add "feedback wanted" label and hope others will share their point of view :)

To quote @JerPk we have two solutions

Make it clear in the documentation that env_prefix is also used for secrets
Do not use the env_prefix for the secrets

I thought option 2 made more sense but reading @vlcinsky opinion, I understand it can be wanted.
I would like to add option 3

Try with and without env_prefix

This doesn't break anything and allows both behaviours but can have side effects and is less clear.
I add "feedback wanted" label and hope others will share their point of view :)

Thanks @PrettyWood for reaction.
Here is my feedback:
Following the KISS I would invite not to add any extra option (such as "for secrets use env_prefix yes/no") unless it is really needed.
$ python -c "import this"|grep -E "Simple|Special cases|There should be one|hard to explain"
Simple is better than complex.
Special cases aren't special enough to break the rules.
There should be one-- and preferably only one --obvious way to do it.
If the implementation is hard to explain, it's a bad idea.

Thanks @PrettyWood for reaction.
Here is my feedback:
Following the KISS I would invite not to add any extra option (such as "for secrets use env_prefix yes/no") unless it is really needed.
$ python -c "import this"|grep -E "Simple|Special cases|There should be one|hard to explain"
Simple is better than complex.
Special cases aren't special enough to break the rules.
There should be one-- and preferably only one --obvious way to do it.
If the implementation is hard to explain, it's a bad idea.

But, if #2190 get merged, we won't be able to transparently prefix secrets? My stack has many services, and I would like to store secrets as /run/secrets/my_project_api_key, but I really don't want to add my_project_ to all field names. Maybe we should have a secrets_prefix in Config??


Packaged functions support

This PR adds packaged function support. I've taken that opportunity to also add a .doc  helper for the shell, so we can get the docs at the runtime.
Packaging is expressed as following, pkgname.funcname(...) and builtin functions are living in a package whose name is the empty string.
genji> .doc math.floor
math.floor(arg1): Returns the greatest integer value less than or equal to arg1.
genji>

@asdine I explored the idea of using comments to document the functions that came up when we were discussing it but I decided to go against it. While it's awesome to have the doc in place to keep it updated, the go comments as a way of documenting the sql functions leads to an awkward result in practice: if you use them  to document Genji's functions, where do you put the comments for their implementation? So, I just went for the simplest approach.

Codecov Report

Merging #419 (172cda5) into main (fa66f81) will increase coverage by 0.30%.
The diff coverage is 34.12%.

@@            Coverage Diff             @@
##             main     #419      +/-   ##
==========================================
+ Coverage   67.01%   67.31%   +0.30%
==========================================
  Files          87       92       +5
  Lines        8349     8448      +99
==========================================
+ Hits         5595     5687      +92
- Misses       2045     2046       +1
- Partials      709      715       +6

Impacted Files
Coverage Δ

internal/sql/scanner/scanner.go
82.42% <ø> (ø)

internal/sql/scanner/token.go
9.67% <0.00%> (-1.04%)
:arrow_down:

internal/expr/builtin_funcs.go
7.90% <7.90%> (ø)

internal/expr/scalar_function.go
73.91% <73.91%> (ø)

internal/sql/parser/expr.go
79.85% <75.75%> (-0.56%)
:arrow_down:

doc/doc.go
83.87% <83.87%> (ø)

internal/expr/function.go
90.00% <88.88%> (+81.88%)
:arrow_up:

doc/tokens.go
100.00% <100.00%> (ø)

internal/expr/math_functions.go
100.00% <100.00%> (ø)

internal/sql/parser/options.go
100.00% <100.00%> (ø)

... and 6 more

Continue to review full report at Codecov.

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

@asdine it's ready for review :)


自定义policy中的outputs没有被渲染出来

Describe the bug
自定义policy中的outputs没有被渲染出来
To Reproduce
自定义的 policy:
"traffic-route-test": {
	alias: ""
	annotations: {}
	attributes: {}
	description: "traffic route, include DestinationRule and VirtualService"
	labels: {}
	type: "policy"
}

template: {
	parameter: {
		version: [...string]
		rules: [...{
			name:         string
			matchHeader?: string
			route: [...{
				host?:   string
				weight:  int
				version: string
			}]
			timeout?: string
		}]
	}

	output: {
		apiVersion: "networking.istio.io/v1alpha3"
		kind:       "DestinationRule"
		metadata: name: context.name
		spec: {
			host: context.appName
			subsets: [
				for _, v in parameter.version {
					name: v
					labels: version: v
				},
			]
			trafficPolicy: tls: mode: "DISABLE"
		}
	}

	outputs: virtualservice: {
		apiVersion: "networking.istio.io/v1alpha3"
		kind:       "VirtualService"
		spec: {
			hosts: [context.appName]
			http: [
				for rule in parameter.rules {
					name: rule.name
					if rule.matchHeader != _|_ {
						match: [{
							headers: baggage: exact: rule.matchHeader
						}]
					}
					route: [
						for _, r in rule.route {
							destination: {
								host:   context.appName
								subset: r.version
							}
							weight: r.weight
						},
					]
					if rule.timeout != _|_ {
						timeout: rule.timeout
					}
				},
			]
		}
	}
}

yaml:
  policies:
    - name: kitex-server-route
      type: traffic-route-test
      properties:
        version: ["v1", "v2"]
        rules:
          - name: test
            matchHeader: "env=test"
            route:
              - version: v1
                weight: 100
            timeout:  0.5s
          - name: prod
            matchHeader: "env=prod"
            route:
              - version: v2
                weight: 100
            timeout:  0.5s

vela dry-run 之后只渲染出了output中的 DestinationRule, 没有渲染出outputs中的 VirtualService, 但是如果把这个policy改成component就可以正常渲染出outputs
文档中提到policy支持outputs, 所以认为是bug

规则要和 component 定义保持一致，必须制定 output，outputs 可用于多个对象，格式如下:
output: {
   <full template data>
}
outputs: <unique-name>:
 <full template data>

KubeVela Version
1.6.0
Cluster information
1.20.6

Currently, the policy definition only support output one resource by far, and it is recommended to use component to dispatch resources for now. Using policy definition to dispatch resource is a legacy behaviour and is not continuously being developed. I would suggest you to use the component definition instead, if multiple resource outputs are wanted.

Currently, the policy definition only support output one resource by far, and it is recommended to use component to dispatch resources for now. Using policy definition to dispatch resource is a legacy behaviour and is not continuously being developed. I would suggest you to use the component definition instead, if multiple resource outputs are wanted.

Using policy to render resources in some cases are necessary, we still support this case, and render outputs is a expected behavior. We should fix this issue.


Consider re-merging ifRequest() and ifResponse()

There's no real point to these. It just controls which of addResponseHandler() and addRequestHandler() gets called. But we know which phase it is, so we can just... do the right thing.
I'd need a good name for the combined function.
Ideas:

applyIf
addIf
addHandlersIf
addHandlerIf

I think I like addHandlerIf. It's clear what it's doing.


Support the empty selector from event mappable

:+1:

Achievable via
events: {
  click: 'onClickHandler'
}


Calendar aware app startup

I know I can use the Task Scheduler to make apps start on log-on/startup, but it would be great if that startup was aware of my Calendar to decide if the app should start or not...
A specific and concrete example is that I don't need Microsoft Teams to start on weekends or when I have holidays booked in my calendar!
I've been using some hacks with PowerShell to achieve this, but I think it would be great if we had something internal to deal with it!

If you'd like to see this feature implemented, add a 👍 reaction to this post.

@pedrolamas
Personally, I like to have PowerToys run all the time. I don't think it matters if I am on a holiday or I am working, PowerToys just makes using a computer faster and makes it a better experience which is something that I would like to have all the time.

@alannt777 I think you didn't understand my idea: I don't want to be able to run PowerToys on a schedule, I want a PowerToy that would allow me to select what apps start when I log-in to my computer based on my Calendar!

Ok. I should have read the description properly.

Excellent idea! If using a calendar is too much work, a calendar setting per App  would be a good first step


Opening multiple files

Thanks for great tool!
When I open multiple yuml files and have the preview window open, it will switch back and forth them very quickly multiple times.

Please specify your platform (Windows/macOS/Linux) and VSCode version.

I am trying with Code 1.6 and macOS, with 3 and 4 yuml files at opened and it doesn't happen.
The 4 files opened in the left pane, and the preview shown in the right pane.

1.6, Windows 10.

@mbp Please update to VS Code 1.6.1 and let me know if the issue persists

It's the same

Works fine with
vsCode 1.10.2
yUML 2.10.0
OS/X 10.12.3
Great tool!

Closing as no more reports received


Develop

Codecov Report

Merging #6 (193c055) into master (527b3df) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #6   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           56        45   -11
=========================================
- Hits            56        45   -11

Impacted Files
Coverage Δ

lib/codenic_logger.dart
100.00% <100.00%> (ø)

lib/src/message_log.dart
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Application crasches when Window has content with CustomControl

Describe the bug
Hi, I've faced an issue when I put an inheritor of the FrameworkElement to MainWindow.Content the app crashes without any exception. The stack and project were attached.
    public sealed partial class MainWindow : Window {
        public MainWindow() {
            this.InitializeComponent();
            Content = new TestFrameworkElement();
        }

    }
    public class TestFrameworkElement : FrameworkElement {
        public TestFrameworkElement() {
        }
    }

NuGet package version:
[Microsoft.WinUI 3.0.0-preview2.200713.0]

Windows app type:

UWP
Win32

No
< Yes

Windows 10 version
Saw the problem?

May 2020 Update (19041)
Yes

Device form factor
Saw the problem?

Desktop
Yes

CustomContentCrash.zip
Stack.txt

@stevenbrix Is this also a cswinrt bug? the provided call stack doesn't have any useful symbols, but I don't see any dll names that would suggest cswinrt... however the C# derivation usually points to this.

This issue also exists in C++/WinRT - see attached sample.
    MainWindow::MainWindow()
    {
        InitializeComponent();
        MyFrameworkElement element;
        Content(element);

    }

namespace App2CPP
{
    [default_interface]
    runtimeclass MyFrameworkElement : Microsoft.UI.Xaml.FrameworkElement
    {
        MyFrameworkElement();
    }
}

App2CPP.zip
stack.txt

This issue is not reproduced in Preview 3 and can be closed.
Thank you!

Excellent! Thanks for testing on Preview 3!


fix the crash issues

fix the crash about user collection is null and app's launch crash
update the out of date compile(it's a mistake to commit it in patch), if you didn't want update,maybe you should commit the crash by yourself

Merged, thanks.


I'm not able to find element using class name in iOS (XCUIElementTypeTextField)

There is a single TextField(XCUIElementTypeTextField) element on the scree and there is no unique id assigned to it, So I have used type strategy and used this code: by.type('XCUIElementTypeTextField') but it's giving me below error,
Cannot find UI element.
Exception with Action: {
"Action Name": "Tap",
"Element Matcher": "((!(kindOfClass('RCTScrollView')) && (kindOfClass('(null)') && !(kindOfClass('UIAccessibilityTextFieldElement')))) || (((kindOfClass('UIView') || respondsToSelector(accessibilityContainer)) && parentThatMatches(kindOfClass('RCTScrollView'))) && ((kindOfClass('UIView') || respondsToSelector(accessibilityContainer)) && parentThatMatches((kindOfClass('(null)') && !(kindOfClass('UIAccessibilityTextFieldElement')))))))",
"Recovery Suggestion": "Check if the element exists in the UI hierarchy printed below. If it exists, adjust the matcher so that it accurately matches element."
}

Ideally, it should work as there is a Single TextField on the screen, Does anyone facing a similar issue?

XCUIElementTypeTextField is not a text field class.

Please don't repost the same question three times. The bot is there for a reason. This is your final warning.


Develop AWS architecture for SBN-PSI + EN pipeline tools with Nucleus

💡 Description

Prepared the following Google Doc to address this ticket. Did not use Markdown or reStructuredText yet.
https://docs.google.com/document/d/1ONUhSJhxJmMt09H-fbfBRl2N-l_02E-CeylmEe_hpAg/edit#

@ramesh-maddegoda started to develop terraform scripts

on hold why the optimal architecture is identified regarding the costs (see NASA-PDS/nucleus#16)

This task has been descoped from B13.1 Initial Delivery here. If this migration happens, it will be in phase 2 of this effort. It will not be part of Phase 1 of the migration.


Sitewide style updates

fix: Update footer component to DaisyUI  #20

Update styles to match DaisyUI standard
Swap .png images for SVGs, #11
Whitespace and formatting cleanup for readability

hey this looks fantastic! sorry it took so long for me to notice this PR


Make it possible to use relative paths with webpack on Windows.

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request => Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question

Current behavior
Webpack has optional __filename variable (https://webpack.github.io/docs/api-in-modules.html#__filename). It's very simple to use this variable as angular moduleId to allow relative paths. But on Windows webpack __filename has backward slashes and angular does not understand them. Of course I can write something like "moduleId: __filename.replace(//g, '/')" but this breaks AOT compilation (I don't remember the exact error but it's something about expressions are not allowed).
Expected/desired behavior
Make angular works with backward slashes in moduleId metadata.
Reproduction of the problem
What is the expected behavior?
What is the motivation / use case for changing the behavior?
Please tell us about your environment:

Angular version: 2.0.0-rc.6
Browser: all
Language: all

Closing since __filename is no longer exposed in browsers builds as this is a Node.js API.
Please be aware that Webpack specific features are not supported by the Angular CLI.


JLC v0.2.0 release

Overview

Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage decreased (-0.1%) to 86.775% when pulling 8876dbbe229b2ea92b275d2a29460d9727af4abb on jlc/v0.2.0-release into 88565b14836be41a00431f7798eefa584b664cb0 on master.


Feature Request: Close dropdown with JavaScript $().dropdown('close')

Feature Request: Add support for closing dropdowns through the JavaScript API
There is one JS method for dropdowns:
$().dropdown('toggle')  // Toggles the dropdown menu of a given navbar or tabbed navigation.

docs:  https://v4-alpha.getbootstrap.com/components/dropdowns/#methods
Though the alert component is not exactly a parallel, I think this would provide some consistency to the API, which provides a way to close alerts:
$().alert('close')  // Closes an alert by removing it from the DOM. If the .fade and .show classes are present on the element, the alert will fade out before it is removed.

docs: https://v4-alpha.getbootstrap.com/components/alerts/#methods
Further, modals provide a similar close ('hide') feature.
docs: http://v4-alpha.getbootstrap.com/components/modal/#methods

IMHO it would be good to have explicit open & close methods in addition to toggle.

100% agree with the above, having open/close in addition to "toggle" (which should toggle instead of just open). I spent 30 minutes debugging what i thought was my bad code only to find out that "toggle" doesn't really mean "toggle" in this case.


feat(travis): pivot to container-based infrastructure

https://docs.travis-ci.com/user/workers/container-based-infrastructure/#Routing-your-build-to-container-based-infrastructure

This seems marginally faster than the legacy infrastructure:
Without sudo:false: https://travis-ci.org/justinrainbow/json-schema/builds/104474976

Elapsed time 1 min 58 sec
Total time 12 min 2 sec

With sudo:false: https://travis-ci.org/justinrainbow/json-schema/builds/104475549

Elapsed time 3 min 34 sec
Total time 11 min 29 sec

@localheinz thoughts?

@bighappyface
Faster is better! :+1:

@bighappyface
But sudo is still needed, isn't it?
See https://github.com/justinrainbow/json-schema/blob/master/.travis.yml#L20.

@localheinz good question. I removed the CHECK_LINKS environment variable from the PHP 5.6 build  to clear up some dependency issues that arose and started breaking the build. You had recommended something along those lines awhile back.
Now that the doc build isn't part of the build process we are good to disable sudo and get things up to date.

@bighappyface
We could speed up the build even further by

enabling caching of dependencies installed with composer (see https://github.com/refinery29/league-lazy-event/blob/master/.travis.yml#L20-L23, works best when authenticating with GitHub)
generating coverage on one build only (see https://github.com/refinery29/league-lazy-event/blob/master/.travis.yml#L15-L16)
disabling xdebug when we don't need it (see https://github.com/refinery29/league-lazy-event/blob/master/.travis.yml#L27)

In regard to coverage, maybe it makes sense to use https://codeclimate.com? It's free for open source, and it comes with fancy badges for displaying the code quality as well as coverage!

@localheinz +1
Feel free :)


A new table tab can be opened by clicking 'Open' or double clicking the attached table even though a tab already opened by the attaching operation

Storage Explorer Version: 1.22.0-dev
Build Number: 20211124.4
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.0.1
Architecture: ia32/x64
How Found: Exploratory testing
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Tables.
Select one table -> SAS attach it.
A tab of the attached table is auto opened.
Right click the attached table -> Click 'Open'.
Check no new tab is opened.

Expected Experience
No new tab is opened.
Actual Experience
A new tab is opened.
Additional Context
There is a '/' next to the path of the tab which is opened by the attaching operation.

There is no '/' next to the path of the tab by clicking 'Open' or double clicking the attached table.

@craxal, can you double check if this is not a regression? If not, feel free to move to 1.23 .

Definitely not a regression from 1.23. Incidentally, attached file shares also has this problem. However, the fix is easy, as they both follow the same code path. They shouldn't need to append the slash when retrieving the node path.

Verified on the build main/20211201.1. Fixed.


API reference for 0.17.0 failed to build; reference for 0.17.1 does not exist

The docs.rs API reference linked in the README.md  redirects to the latest version at the time of writing this issue, 0.17.1, which is just a blank page with the message "The requested resource does not exist". Upon further investigation, it looks like while v0.16.2 exists and is fine, v0.17.0 failed to build. Here is the build logs for 0.17.0.
The interesting part:
error[E0277]: the trait bound `proc_macro::TokenTreeIter: std::clone::Clone` is not satisfied
  --> .cargo/registry/src/github.com-1ecc6299db9ec823/maud_macros-0.17.0/src/parse.rs:36:5
   |
36 |     input: TokenTreeIter,
   |     ^^^^^^^^^^^^^^^^^^^^ the trait `std::clone::Clone` is not implemented for `proc_macro::TokenTreeIter`
   |
   = note: required by `std::clone::Clone::clone`

Thanks for the report!
That's probably because docs.rs was using an older version of the compiler. The Clone impl for TokenTreeIter (rust-lang/rust#43280) landed on Jul 24, but the compiler listed in that log dates from Jul 10.
I have reported the issue at https://github.com/onur/docs.rs/issues/23#issuecomment-333284541.

https://docs.rs/maud/0.17.1/maud/ works now


Get the event decorated on date selection.

related to #352.
I am thinking of something, a listener might be to be able to retrieved the event being rendered on the calendar view.

You have the date being returned on date selection, it's up to you to check against any sort of list of events you may have.


Duplicate Language key with 7 mods

Here is the list:
tile.mirror.name

cfm Mirror
thaumcraft Magic Mirror

recipe.type.workbench

hammercore Crafting
thaumcraft Workbench

recipe.type.construct

hammercore Multiblock Structure
thaumcraft Mystical Construct

recipe.clickthrough

hammercore Click to View Recipe
thaumcraft Click for recipe

item.amber.name

jurassicraft {stored} in Amber
thaumcraft Amber

tile.barrier.name

minecraft Barrier
thaumcraft Warding Aura

tile.stone_porous.name

mist Porous Foggy Stone
thaumcraft Porous Stone

item.void_seed.name

primal core Equisetum Ineris Spore
thaumcraft Void Seed

tile.flesh_block.name

thaumcraft Block of Flesh
tiny progress Compressed Flesh Block

This issue was moved to Azanor/thaumcraft-suggestions#338


Update SetDnsServerDiagnostics.md EventLogLevel Description

EventLogLevel is of type UInt32, so won't accept the string values that are currently specified.
Updated the description to include the correct numerical values mapped to the options available on the Event Logging tab of the DNS Manager interface.

Docs Build status updates of commit f4ae9f2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/dnsserver/Set-DnsServerDiagnostics.md
:white_check_mark:Succeeded

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.

@X-Guardian Thank you for your contribution. Please target the branch of this PR to master to proceed with the review.

@JohanFreelancer9 Minor style checks needed for this PR. Thanks.

@dariomws306 @get-itips @tseward
Peer review needed for this PR. Thanks.

I am ok once the suggestions are applied

Agreed with @dariomws306's suggestion.

Docs Build status updates of commit 7880917:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/dnsserver/Set-DnsServerDiagnostics.md
:white_check_mark:Succeeded

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.

Docs Build status updates of commit f0fe633:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/dnsserver/Set-DnsServerDiagnostics.md
:white_check_mark:Succeeded

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.

Docs Build status updates of commit 4045bf7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/dnsserver/Set-DnsServerDiagnostics.md
:white_check_mark:Succeeded

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.

@dariomws306 Any additional input on revised content?


AI Render node not connected/Image not sent to render layer

Describe the bug
Hello, I'm using AI Render with SD A1111. It seems to be working in that I can see the produced image if I go inside the AI Render group node, but it isn't automatically sent to the render layer node. Nothing happens after completion. The AI Render group node is just floating by itself and has no inputs and outputs visible on the outside of the node despite there being an in and out on the inside.
To reproduce
Standard usage
Error log
WARN (bgl): C:\Users\blender\git\blender-v350\blender.git\source\blender\python\generic\bgl.c:54 report_deprecated_call: 'bgl.glEnable' is deprecated and will be removed in Blender 3.7. Report or update your script to use 'gpu' module.
WARN (bgl): C:\Users\blender\git\blender-v350\blender.git\source\blender\python\generic\bgl.c:54 report_deprecated_call: 'bgl.glBlendFunc' is deprecated and will be removed in Blender 3.7. Report or update your script to use 'gpu' module.
WARN (bgl): C:\Users\blender\git\blender-v350\blender.git\source\blender\python\generic\bgl.c:54 report_deprecated_call: 'bgl.glBindTexture' is deprecated and will be removed in Blender 3.7. Report or update your script to use 'gpu' module.WARN (bgl): C:\Users\blender\git\blender-v350\blender.git\source\blender\python\generic\bgl.c:54 report_deprecated_call: 'bgl.glDisable' is deprecated and will be removed in Blender 3.7. Report or update your script to use 'gpu' module.
WARN (bgl): C:\Users\blender\git\blender-v350\blender.git\source\blender\python\generic\bgl.c:54 report_deprecated_call: 'bgl.glEnable' is deprecated and will be removed in Blender 3.7. Report or update your script to use 'gpu' module.
2023-03-11 15:45:06,304     INFO ga4mp.py 183 Sending POST to: https://www.google-analytics.com/mp/collect
2023-03-11 15:45:06,507     INFO ga4mp.py 235 Batch Number: 1
2023-03-11 15:45:06,507     INFO ga4mp.py 236 Status code: 204
Environment

Blender version (upper right corner of splash screen): 3.5 Beta
AI Render version (find in Preferences > Add-ons): 0.6.9
Operating system (Windows/Mac/Linux): Windows 11

Screenshots/video
Video
https://send.cm/hv5fmuhv6rbw
Additional information
No response

Thanks for letting me know about this. I just confirmed it - there is something broken in the 3.5 beta. Not sure if it's a purposeful change on Blender's part or if it's a bug. I will investigate...

This does seem to be a new bug introduced in Blender 3.5 beta. I couldn't find any way to work around it. I've submitted the bug to the Blender team: https://projects.blender.org/blender/blender/issues/105701

Awesome, thanks for the quick update!


Select Correct Word from Autocorrect window open when I select wrong spell word not working in UITextView iOS

issue closed


Add ability to disable input on editor making it readonly

I have been trying to implement this editor for a little while and eventually found the need to make the input readonly. I'm aware that I could just switch out the editor for a web view to view the content but then you don't have consistency between the editor and the web view.
Using setFocusable, setClickable and setLongClickable mean that you can't select the text to copy it.
So instead I change the contentEditable property of the editor node in the JS.
I also removed the user-select and user-modify from the CSS as you were mixing two different methods by doing so and it would have required adding a class to override the default styles as well. I chose contenteditable as it has better browser support.
Let me know your thoughts.
P.S. It appears that it would fix issue #79 as well

@connorwyatt Hello, thatnk for grate PR, can I ask if you found how to make links clicable?)

@frakc The easiest way I can think to make that happen is to preventDefault on the anchor click events, and then use a javascript interface provided by Android to notify the Android application. That way you can pass the url to the application.

@connorwyatt
Thanks, great.


More examples

Can we see an example for adding a group to the addin?
I've added one deployment through the web and trying to replicate that success using powershell and I'm coming up short. Not sure if it's the tools or me.

Hello @dyerseve. Can you please tell us to which article your feedback applies? This information will help us route the issue correctly. Thank you!

@dyerseve We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Remove APP_CAPABILITY from EntityKind enum

Should be delete entries from Source_Data_Rating and System_Job_Log that have Entity_Kind as APP_CAPABILITY?

Perhaps introduce MEASURABLE_RATING and migrate to that ?


seperate e2e tests with playwright from smaller unit tests with jest, report test results in PR

xref: https://github.com/tree-finder/tree-finder/pull/9#issuecomment-1812882010

Thanks, I'll hopefully be able to add a decent chunk to the unit tests next week.


use a tmp variable named [isOldProviderGroupEmpty] to resolve the bug…

djust the location of method called  [closeTransports();] (#1442)

https://github.com/sofastack/sofa-rpc/issues/1442

Motivation:
there are two bugs in the AbstractCluster.java

variable named [oldProviderGroup] will be changed because of the method [addressHolder.updateProviders(providerGroup);], which will not execute method [ closeTransports();]

the method of [closeTransports();] should not be contain in the [LOGGER.isWarnEnabled(consumerConfig.getAppName())]

Modification:

I use a boolean tmp named [isOldProviderGroupEmpty], avoid the change of [oldProviderGroup]

Result:
Fixes #1442

Summary by CodeRabbit

Chores

Improved code readability in the updateProviders method by reducing redundant checks.
Enhanced maintainability without altering functionality or control flow.

It has already been fixed in other PRs. Thank you for your feedback. https://github.com/sofastack/sofa-rpc/pull/1452


Distill finished! 0 -> 0 (0 discarded)

I've made sure all the dependencies are properly installed and have downloaded and extracted the dataset. The DATA_DIR path is "fireball-anonymized-nov-28-2022-kfdjl/anonymized/data/". Whenever I run distill1_time_group.py I get the above output in my terminal. Running then through the rest of the files until finetune_prep.py gives a error which states there is no training data.

Hi Vishal, this repository contains all the scripts we used to produce the data files in the FIREBALL release for reference - you probably won't need to use any of the code here unless you're reproducing the dataset. If you would like to do that for whatever reason, your DATA_DIR should be a directory containing multiple other directories, where each subdirectory is named the ID of a combat instance and contains one or more JSONL files (the raw.jsonl files in the FIREBALL release).
I recommend just using the preprocessed outputs included in FIREBALL though.

Hi Andrew,
Thank you for the clarification, I will look at the FIREBALL repository instead. Thank you.
Kind regards,
Vishal Joshi
@.?anonymous&ep=owaSlotsEmailSignature>               Book time to meet with @.?anonymous&ep=owaSlotsEmailSignature>

From: Andrew Zhu @.>
Sent: 11 December 2024 17:01
To: zhudotexe/FIREBALL-data-processing @.>
Cc: Vishal Joshi @.>; Author @.>
Subject: Re: [zhudotexe/FIREBALL-data-processing] Distill finished! 0 -> 0 (0 discarded) (Issue #7)
Hi Vishal, this repository contains all the scripts we used to produce the data files in the FIREBALL release for reference - you probably won't need to use any of the code here unless you're reproducing the dataset. If you would like to do that for whatever reason, your DATA_DIR should be a directory containing multiple other directories, where each subdirectory is named the ID of a combat instance and contains one or more JSONL files (the raw.jsonl files in the FIREBALL release).
I recommend just using the preprocessed outputs included in FIREBALL though.
—
Reply to this email directly, view it on GitHubhttps://github.com/zhudotexe/FIREBALL-data-processing/issues/7#issuecomment-2536562924, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BC3JRUVR3QX7HH7E3GUI4LT2FBVWDAVCNFSM6AAAAABTNT47PWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZWGU3DEOJSGQ.
You are receiving this because you authored the thread.Message ID: @.***>


Add name to linter

Just like the title says

@steelbrain hi, thanks for making pull request, can you change your code so it follows our styleguide (use 2 spaces instead of tabs)

I used the github web UI. Let me fix that.

Nevermind, I think I deleted the repo where I PRed this from. You get the idea. You can just commit directly to master :)

@steelbrain ah ok then, i didn't add them in my previous commit because this pr is still open :)


fix(Enterprise Management): update service after API changes

PR summary
Re-gen of service after API changes.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

LGTM!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Rules in ALB001_Alignment-in-spacial-structure.feature might be premature

Gherkin-feature file "ALB001_Alignment-in-spacial-structure.feature" contains the following rule:
  Scenario Outline: Agreement on each IfcAlignment being contained in an IfcSite

      Given A file with Schema Identifier "<schema_identifier>"
      And An IfcAlignment
      Then Each IfcAlignment must be contained in IfcSite

There is an ongoing discussion in bSI implementer forum which might come to a different conclusion.
BR Andreas

Good point, thanks Andreas. This rule was used as exercise to practice with Gherking and python implementation. We won't merge it before an agreement is reached. But I'll leave the branch open as this is still a good example of how to write rules for the Validation Service


rabbitmq_node_up reports it self up and the other cluster nodes

I get the double amount of metrics since the exporter counts every node twice (it reports its cluster friend too).
Why?
Example:
root@xxx: curl http://localhost:15672/api/metrics|grep node_up
#HELP rabbitmq_node_up Node runnning status
rabbitmq_node_up{name="rabbit@<IP_ADDRESS>",type="disc"} 1
rabbitmq_node_up{name="rabbit@<IP_ADDRESS>",type="disc"} 1
The nodes are clustered with https://www.rabbitmq.com/cluster-formation.html#peer-discovery-aws

There isn't enough information to tell for sure but my best guess is that when every node is queried and reports all of its peers are up, the counters are added instead of treated as a boolean gauge.

Seems that rabbitmq_node_up reports reachable nodes from each node itself ...
So in a 3 node cluster setup reports each node itself and the 2 other nodes which sums up to 9 rabbitmq_node_up metrics.
Well that can be used to show something like
nodeXX can reach X nodes ...
Its a bit confusing, if you run rabbitmq as cluster, since this is not well prepared I would say.
Try to map the IP of the server you query against the name ... should work.
Especially what I miss is simple the cluster name of the whole rabbitmq cluster, to by able to say:
Cluster is up with X nodes ...
Seems this info is not exported by the plugin ...

We recently discussed this and concluded that there are significant benefits to collecting data from a single node and then aggregating at "display time". This should be covered in the docs now that #73 was merged.
@BoemmLA I'm sorry but I think you are greatly oversimplifying how distributed systems fail. If a node has 3 clusters but A cannot reach C and C cannot reach B but all other links are up, how many nodes does that cluster have? So this is a very convenient and very misleading metric. To some extent it is clarified by another one, the number of reported partitions in the cluster. Then we not only monitor the number of vertices in the graph but also the number of problematic edges.


blake2 implementation on vamp-ir

rudimental blake2 implementation running on max 256 bitstrings messages up to 64 bytes.

I tried compiling this like this:
cargo build --release
target/release/vamp-ir setup -o params.pp -m 64
target/release/vamp-ir compile -s tests/blake2s.pir -u params.pp -o blake.plonk

But I get the following error:
thread 'main' panicked at 'unable to compile circuit: PCError { error: "Polynomial Commitment Error: TrimmingDegreeTooLarge" }', src/main.rs:232:10
What am I doing wrong?

I now realized that this is quite an old PR. I tried to compile this circuit using master branch (as I see it was brought over there by @murisi: https://github.com/anoma/vamp-ir/commit/4344d09af466ba9b6ea0f246deec71585f894342) but I still get the same error.

@bazzilic Not only is this PR old, it is a dupe of code. This blake2s circuit was included in main through a different PR, so I am deleting this one.
The TrimmingDegreeTooLarge error from the plonk backend means the setup was not large enough. You can use vamp-ir setup -m N ...  to use a setup of size 2^N. The default is 10 which allows 1024 constraints, but our current blake2s circuit uses over 200k constraints. That's why it worked with halo2 and not plonk.
The 33MB circuit is pretty big but probably normal (for now anyway). Some day we will get back to optimizing our circuits and synthesis and bring these numbers down :)


IntrinsicHeight and BoxConstraints generate false-positive overflow error

Steps to Reproduce

Execute flutter run on the code sample
Resize your window over 350px - You'll see an overflow warning where it don't need to be shown
Resize your windows under 350px - Overflow disappear

Expected results:
No overflow error
Actual results:
Overflow error ! IntrinsicHeight doesn't seem to understand restrictions with BoxConstraint and just ignore it.

Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: Scaffold(
        body: IntrinsicHeight(
          child: Container(
            constraints: const BoxConstraints(maxWidth: 350),
            child: Column(
              children: const [
                Text(
                  'Lorem ipsum dolor sit amet, consectetur adipiscing elit. Mauris mattis a felis eu sagittis. Donec purus quam, malesuada sed ultricies a, cursus quis erat. Aliquam nec tellus ultrices, congue nisl sed, tincidunt dui. In fermentum eros ut dolor blandit, sit amet dignissim mi ultrices. Suspendisse porttitor malesuada justo, quis commodo est interdum id. Aliquam finibus felis non tellus venenatis, vitae maximus eros ullamcorper. Nunc laoreet porttitor ex eu hendrerit. Etiam non feugiat justo, ac aliquam leo.',
                ),
              ],
            ),
          ),
        ),
      ),
    );
  }
}

Logs
Running "flutter pub get" in max_width_bug...                      423ms
Analyzing max_width_bug...
No issues found! (ran in 2.5s)

[√] Flutter (Channel stable, 3.3.9, on Microsoft Windows [Version 10.0.18363.2274], locale fr-CH)
    • Flutter version 3.3.9 on channel stable at C:\apps\flutter-2.0.6
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b8f7f1f986 (8 days ago), 2022-11-23 06:43:51 +0900
    • Engine revision 8f2221fbef
    • Dart version 2.18.5
    • DevTools version 2.15.0

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\apps\android-sdk
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.2.1)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.2.32516.85
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 4.2)
    • Android Studio at C:\apps\android
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[√] IntelliJ IDEA Ultimate Edition (version 2021.2)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2021.1.1
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart

[√] VS Code
    • VS Code at C:\Users\RossetM\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.52.0

[√] VS Code, 64-bit edition (version 1.73.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.52.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.18363.2274]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 107.0.5304.107
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 107.0.1418.42

[√] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Size over 350px

Size under 350px

Reproducible using the code sample provided above.
Labeling for further insight from the team

recording
https://user-images.githubusercontent.com/88313112/205011708-aa232161-56ea-458b-95ab-643662bfd84a.mov

flutter doctor -v
[✓] Flutter (Channel stable, 3.3.9, on macOS 13.0.1 22A400 darwin-arm, locale en-GB)
    • Flutter version 3.3.9 on channel stable at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b8f7f1f986 (8 days ago), 2022-11-23 06:43:51 +0900
    • Engine revision 8f2221fbef
    • Dart version 2.18.5
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9123335/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14B47b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[!] Android Studio
    • Android Studio at /Applications/Android Studio Preview 2.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Android Studio (version 2021.3)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9123335/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] Android Studio (version 2021.3)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9014738/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.73.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.52.0

[✓] Connected device (3 available)
    • iPhone 14 Pro (mobile) • 4F72110C-F38B-4CF9-93C4-4D6042148D28 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-1 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 13.0.1 22A400 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 108.0.5359.71

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

[!] Flutter (Channel master, 3.6.0-13.0.pre.24, on macOS 13.0.1 22A400 darwin-arm64, locale en-GB)
    • Flutter version 3.6.0-13.0.pre.24 on channel master at /Users/nexus/dev/sdks/flutters
    ! Warning: `flutter` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4878f26ab0 (5 hours ago), 2022-11-30 23:18:19 -0500
    • Engine revision 441229a95c
    • Dart version 2.19.0 (build 2.19.0-444.0.dev)
    • DevTools version 2.20.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9123335/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14B47b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[!] Android Studio
    • Android Studio at /Applications/Android Studio Preview 2.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Android Studio (version 2021.3)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9123335/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] Android Studio (version 2021.3)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9014738/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.73.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.52.0

[✓] Connected device (3 available)
    • iPhone 14 Pro (mobile) • 4F72110C-F38B-4CF9-93C4-4D6042148D28 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-1 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 13.0.1 22A400 darwin-arm64
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 108.0.5359.71

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 3 categories.

This is likely the same underlying issue as https://github.com/flutter/flutter/issues/59719. I am going to close this as a duplicate.

#59719 has been fixed, but this issue still reproduces for me on master.
However, this does appear to be the same issue as #27293, which is still open. So I think this can be considered a duplicate of that issue.


[CI] Update appveyor

Make sure using build wheel (disable all other sources)
Add use of appveyor proxy

Ready for review for me.

Ready for review for me.

Back to default appveyor build worker image.


Combining live search + load more

Hi Ben love your work. Blitz cache really makes Craft CMS fly!
Now i'm trying to get my head around Spring.
Question
Can you make an example combining the search + load more functions. Tried but the search gets inaccurate (guess its refreshing both components).
Did try nesting the components as well but couldn't  use arrays in nesting.
My example:
{#--- _components/news ---#}

{% set query = query ?? '' %}
{% if sprig.isInclude %}

	  <input sprig s-trigger="keyup changed" s-target="#results,#replaceTwo" s-swap="outerHTML"
	type="text" name="query" value="{{ query }}" placeholder="search..." class=" border border-blue p-3 w-full mb-4 placeholder-black placeholder-opacity-100" />
{% endif %}

	{% set entriesOne = craft.entries.section('news').search('title:*'~ query~' *').orderBy('postDate desc') %}
	{% set entries = entriesOne.offset(offset).limit(_limit).all() %}

        {% for item in entries %}

	{% endfor %}

 	  {% if entries|length %}

			<input type="hidden" name="offset" value="{{ offset + _limit }}">
			<button sprig s-target="#replace" class="button mb-10 py-2" s-swap="outerHTML">Load more</button>

	  {% endif %}

What is the problem with your code? It looks to me like it should work...

it works sort of... load more works. But the search form is not as accurate as the default 'live search component'. it hides results that shouldn't be hided.
Default live search component
Searching for: 'Ny'
• Nyhet 1
• Nyhet 2
• Nyhet 3
My live search component
Searching for: 'Ny'
• Nyhet 3
... and it refreshes a lot more

So it is working, but not quite as you expect? I'm having trouble understanding how I can help you here...

Sorry if I was unclear. Is there a better way to combine the two modules? Would be great in the cookbook. I tried nested components but it didn't work with an entries array. If my question is irrelevant you may close the ticket.

I tested your code and it is working for me. As for adding it to the cookbook, it seems too specific to your use-case, but I'm happy to answer any other questions you may have.


Qpg6100/kvs

Problem

persistent storage was not implemented on QPG6100 platform
a new include file was needed for ReturnErrorOnFailure macros in KVS test, otherwise it wouldn't compile

Changes

adding KVS support on QPG6100 platform
adding port of persistent-storage example for QPG6100 platform
fix the KVS test to include the correct file that contains the macro for ReturnErrorOnFailure

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Lucian Copat seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@lucicop can you restyle please?


Put the keyboard constants in the documentation

To get at them currently, you have to dig in the source. Not the most pressing issue, but inconvenient.

I have added a PR with your suggestion. The only problem I have detected is now is hard to check the Phaser.Keyboard documentation. I could be useful if jsdoc have an option to include a separate section for them.

Thanks for the PR. This is now resolved with the new KeyCode file.


[MINOR][PYTHON] Change TypeVar to private symbols

What changes were proposed in this pull request?
I've convert internal typing symbols in __init__.py to private. When these changes are agreed upon, I can expand this MR with more TypeVar this applies to.

Why are the changes needed?
Editors consider all public symbols in a library importable. A common pattern is to use a shorthand for pyspark functions:
import pyspark.sql.functions as F
F.col(...)

Since pyspark.F is a valid symbol according to __init__.py, editors will suggest this to users, while it is not a valid use-case of pyspark.
This change is in line with Pyright's Typing Guidance for Python Libraries

Does this PR introduce any user-facing change?

No
How was this patch tested?

Verified with PyCharm auto-importing.

Looks pretty good. Mind taking a look at https://github.com/apache/spark/pull/40338/checks?check_run_id=11851128079?

@HyukjinKwon I've rebased against the latest master, however it seems that workflows won't start for new first-time contributors not inside the organization. If I read the documentation [1][2] correctly, a maintainer will need to activate the workflow?
[1] https://docs.github.com/en/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks
[2] https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/enabling-features-for-your-repository/managing-github-actions-settings-for-a-repository#controlling-changes-from-forks-to-workflows-in-public-repositories

Apache Spark has a custom implementation that leverages the build (and resources) from forked repository. Mind checking if the workload is enabled in your fork (https://github.com/MaicoTimmerman/spark/actions/workflows/build_and_test.yml), and rebase this?

Merged to master and branch-3.4.

cc @zero323 in case you have some feedback on this.

cc @zero323 in case you have some feedback on this.

@HyukjinKwon I am OK with that, though there is a bigger issue here. We have TypeVars in py modules in quite a few places, which are a side effect of migrating to inline hints before dropping Python 3.8 support (looking at you, Protocol...).
Ideally, we'd handle all of that consistently and use consistent naming convention, but that's not something will be able to do any time soon.


Add Theme Slice.

Add a self-made Theme Slice to Hexo Theme Page.

Hi @NHibiki ,
The repository https://github.com/NHibiki/hexo-theme-slice is not found. Was it moved somewhere else?

@stevenjoezhang yes, thanks for pointing it out. This theme is no longer maintained and wiped. Maybe we can update the list and remove this item.
Best

Thanks for your feedback and contribution to the Hexojs community!


Add test cases

We usually manage source code in src folder and test code in test folder. Do you think it will help to keep test code in separate folder?

@jainaashish - I think follow OTEL's convention would be a better choice? See otel-collector-contrib. They put the implementation and test code in the same folder.


fix checkstyle for management GraphRbac

Close this as new PR created to remove unexpected characters when preparing reformat.


Use SLF4J for logging

At the moment we're using println, that makes it hard to control the log-levels

To get logging working better with proto.actor integrated into a project using slf4j, I've been adding a companion object into classes and using slf4j for logging. For example:
    companion object {
        private val LOGGER = LoggerFactory.getLogger(DefaultMailbox::class.java)
    }

and then
 LOGGER.info("User mailbox count " + userCount)
However there are maybe better solution for kotlin, such as https://github.com/MicroUtils/kotlin-logging which would remove some of the boilerplate, allowing this:
companion object: KLogging()
logger.debug { "Some $expensive message!" }

Any views on whether using slf4j directly, or via a kotlin wrapper library  is preferable?

Dependency wise it looks pretty clear:
http://search.maven.org/#artifactdetails|io.github.microutils|kotlin-logging|1.5.3|jar

kotlin-stdlib http://search.maven.org/#
slf4j-api http://search.maven.org/#

I think as a rule of thumb we should a little defensive on the dependencies
we pull in so we're not getting issues with transitive dependencies.
But if the code will be more clear with the lib above and you'll need to
reimplement other stuff otherwise, bring it in. The amount of logs today is
relative
small so ripping it out in the future is not a big deal.
Focus on removing the println:s from the production code and we can tackle
the logging in the tests as a separate PR.
// Örjan
On Sun, Feb 4, 2018 at 3:47 PM, james-cobb<EMAIL_ADDRESS>wrote:

To get logging working better with proto.actor integrated into a project
using slf4j, I've been adding a companion object into classes and using
slf4j for logging. For example:
companion object {
    private val LOGGER = LoggerFactory.getLogger(DefaultMailbox::class.java)
}

and then
LOGGER.info("User mailbox count " + userCount)
However there are maybe better solution for kotlin, such as
https://github.com/MicroUtils/kotlin-logging which would remove some of
the boilerplate, allowing this:
companion object: KLogging()
logger.debug { "Some $expensive message!" }
Any views on whether using slf4j directly, or via a kotlin wrapper library
is preferable?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AsynkronIT/protoactor-kotlin/issues/34#issuecomment-362911858,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHkgPdqzQx2cbapNC0Pv6wnSY9rE79_ks5tRcMAgaJpZM4R4fxw
.

--
Örjan Sjöholm
0727 - 130037

I think kotlin-logging looks nice, so I've used that for now. In particular removing the requirement to paste the classname into the LoggerFactory is an improvement - failing to cut and paste correctly has caught me out in Java slf4j logs before....
Branch is here: https://github.com/crowdconnected/protoactor-kotlin/tree/issue-34-slf4j which catches all println in the production code.
I'll do another check through the changes before making a PR
James

@james-cobb great work!
Resolved by #36


Fixes developer guide

Addresses

Fixes #879
Fixes #880
Fixes  #882

The tests fail due to a timeout. I don't expect to do more here before the review.

Happy to merge this for the time being but will want to revert wants libmamba is the default for conda.


Add design/20230601.gateway-route-hostnames.

Pull Request Motivation
Discussed #6051 with @irbekrm, who suggested making a design doc.
Kind
design
Release Note
NONE

@jsoref is anything missing here?

/remove-lifecycle rotten
/reopen

@inteon Thank you. I'll get to the review comments shortly.

@inteon Suggestions applied.

Thank you very much!
This will be a great help to all contributors working on implementing this feature in the future ❤.
/unhold
/lgtm


Use latest decode_raw

Use latest decode_raw
Handle None case of try_parse_entries properly (this is broken protobuf, not an empty list)

I added code for error-handling, and this strips that. Is this better than "console_error_panic_hook::set_once()"? I like that console_error_panic_hook shows a stack and helpful info about where the error was. Does this do that?

This PR does not change the error handling. console_error_panic_hook is still used to convert errors to console logs.
What is happening here is that the None case of try_parse_entries is not converted to an empty list anymore. This is what .unwrap_or_default() does (the empty vector is the default value of a vector). I added this call before. However, this was a mistake because None does not mean no entries, it means the bytes cannot be parsed correctly as protobuf.

This PR does not change the error handling.

It does as far as I can tell. You need to call console_error_panic_hook::set_once() to trigger error-handling, so I did that in parse_raw, which is removed in this PR. It doesn't matter, I can just add it back, if it doesn't go into merge. I have seen what you are talking about (bad binary will just return an empty array, in JS side)and I see what you are doing here. Looks good. Merging.

oh, yep, it's still there after merge.


Check Page in Desktop mode doesn't looks like the design

Describe the bug
I have checked and the Check view have some differences with the design
To Reproduce
Go to https://dev--azordev-dasher-user.netlify.app/check
Expected behavior
I have listed the bugs here:

The hand image is hidden at the bottom. I have notice that actually doesn't exist a way to put the image at the bottom, because the image have a transparent space at the bottom that then we need hidden using a negative bottom (Actually it say bottom: -38px, but it value is wrong for my screen, please check screenshot 1.).  So the solution is edit the image to delete it transparent space, since the image is resizing depending of the screen and the transparent space too is resizing, therefore the actual bottom: -38px work for the screen of someone, but not for my screen.
The "Hola!" "Ingresa el número de boleta" text is not at the position expected. (Please check screenshot 2).
All text is more small that in the design. Please add a Media query to change the font size for desktop.

Screenshots

The hand image is hidden at the bottom. Please edit the image to delete transparent bottom image and change bottom: -38px for bottom: 0px
It looks so on my screen

This is the transparent bottom space that have the image, please delete it editing the image

The "Hola!" and "Ingresa el número de boleta" text is not at the position expected
Actually is:

Should be:

Please re positioning it and delete the line-height in both tags element.
All text are more small that in the design
Actually is:

Should be:

Current state:
https://dev--azordev-dasher-user.netlify.app/check

Will close this Issue and any other detail will be treated as a separate bug


Fix TS definition of Set#iterate, callbackScope is an optional param

Please do not update the README or Change Log, we will do this when we merge your PR.
This PR (delete as applicable)

Updates the Documentation

Describe the changes below:
Fixes TS definition of Set#iterate to callbackScope as optional.

👍


Duplicate Identifiers within Tether.js Library

Duplicate Identifiers within Tether.js Library
Problem
The TypeScript static type analyser detected the following duplicate identifers within dist/tether.js:
node_modules/tether/dist/js/tether.js(52,10): error TS2300: Duplicate identifier 'getScrollParents'.
node_modules/tether/dist/js/tether.js(138,10): error TS2300: Duplicate identifier 'removeUtilElements'.
node_modules/tether/dist/js/tether.js(145,10): error TS2300: Duplicate identifier 'getBounds'.
node_modules/tether/dist/js/tether.js(178,10): error TS2300: Duplicate identifier 'getOffsetParent'.
node_modules/tether/dist/js/tether.js(182,10): error TS2300: Duplicate identifier 'getScrollBarSize'.
node_modules/tether/dist/js/tether.js(218,10): error TS2300: Duplicate identifier 'extend'.
node_modules/tether/dist/js/tether.js(238,10): error TS2300: Duplicate identifier 'removeClass'.
node_modules/tether/dist/js/tether.js(252,10): error TS2300: Duplicate identifier 'addClass'.
node_modules/tether/dist/js/tether.js(287,10): error TS2300: Duplicate identifier 'updateClasses'.
node_modules/tether/dist/js/tether.js(431,5): error TS2300: Duplicate identifier 'getScrollParents'.
node_modules/tether/dist/js/tether.js(432,5): error TS2300: Duplicate identifier 'getBounds'.
node_modules/tether/dist/js/tether.js(433,5): error TS2300: Duplicate identifier 'getOffsetParent'.
node_modules/tether/dist/js/tether.js(434,5): error TS2300: Duplicate identifier 'extend'.
node_modules/tether/dist/js/tether.js(435,5): error TS2300: Duplicate identifier 'addClass'.
node_modules/tether/dist/js/tether.js(436,5): error TS2300: Duplicate identifier 'removeClass'.
node_modules/tether/dist/js/tether.js(437,5): error TS2300: Duplicate identifier 'updateClasses'.
node_modules/tether/dist/js/tether.js(440,5): error TS2300: Duplicate identifier 'getScrollBarSize'.
node_modules/tether/dist/js/tether.js(441,5): error TS2300: Duplicate identifier 'removeUtilElements'.
node_modules/tether/dist/js/tether.js(1290,5): error TS2300: Duplicate identifier 'getBounds'.
node_modules/tether/dist/js/tether.js(1291,5): error TS2300: Duplicate identifier 'extend'.
node_modules/tether/dist/js/tether.js(1292,5): error TS2300: Duplicate identifier 'updateClasses'.
node_modules/tether/dist/js/tether.js(1676,5): error TS2300: Duplicate identifier 'getBounds'.
node_modules/tether/dist/js/tether.js(1677,5): error TS2300: Duplicate identifier 'updateClasses'.

Root Cause
The issue arises due to the Gulp build system doing a simple concat of the source files without reguard to file scope. Because src/utils.js and src/tether.js both define the same identifers within their top level file scope a duplicate identifer clash occurs when they are concatenated together.
Here's an illustration of what's happening in the source files:

src/utils.js

// define a function declaration in utils.js
function getScrollParents() {}

// assign function declaration to property on TetherBase.Utils in utils.js
TetherBase.Utils = {
    getScrollParents
}

src/tether.js

// assign property from TetherBase.Utils to varible with same name in tether.js
var TetherBase$Utils = TetherBase.Utils
var getScrollParents = TetherBase$Utils.getScrollParents;

Here's an illustration of what's happening in the build output:

dist/tether.js

// tether.js output file from build system (simplified the UMD header for brevity)
(function() {

    // utils.js
    function getScrollParents() {}
    TetherBase.Utils = {
        getScrollParents
    }

    // tether.js
    var TetherBase$Utils = TetherBase.Utils
    var getScrollParents = TetherBase$Utils.getScrollParents;

}())

Notice in the dist/tether.js output file that the function declaration in src/utils.js and the variable declaration in src/tether.js both use the same identifer. This occurs for all the identifers listed in the TypeScript errors above.
Proposed Resolution
The following proposed resolutions should fix the issue:

Skip the intermediary assignment in src/tether.js and use the TetherBase.Utils methods directly.
Switch over to using ES6 modules and use a module packer like webpack to enforce module level scope isolation in the build output.

Sidenote: Consider switching the code base to TypeScript to catch errors like this in the future :) :rocket:

Related Issues
Issue #187

@thoughtentity hmm I have this problem with my project, do you have a workaround to ignore these errors until it is fixed ? I'd like to get rid of these error messages in my shell

What is worse, rollup in treeshake mode (default) removes the second declaration of each identifier, in this case the functions.

Here is what I did to both correct this issue:
Using Visual Studio 2015 or higher (would probably work in 2010 and up but may need a "renaming" extension tool)
My fix:
- I clicked on the function name, selected function key "F2" which does a rename and updates those names that call it and added the prefix of "fn" to each name.
(you can put what ever you want here to match your naming convention)
This turned the function into a stronger unique name to avoid the duplicate check. To confirm this fix, right click on the function name and select "Find all references" then see that count. Then do the above fix and check the "Find all references" count.
Hope this is a valid option and others find it useful.

This should be fixed now. Please try 2.0.0-beta.2


Something like a dragMove event on source

It may be slightly outside the scope of this lib, but I really like the api and would like to be able to use it for some more generalized dragging applications, beyond drag+drop.  In order to use it for more general use cases it would be nice to have a way to be called on every move to get the position.  One example application would be a resizable box, you have a handle in a corner of the box you can drag and change the size, I can't find a way to implement this with the current api.

Seems like a common request (#173).
I don't mind seeing this, so if you implement it in DnD Core with tests, it's easy to add support to React DnD.

+1 for an easy way to implement resizable boxes.

@gaearon Can you guide me on how to implement this?

@vkbansal You can start by getting familiar with dnd-core and its tests. Once it makes sense, you can add drag to drag source contract, similar to how drop targets have hover.

Closing as I don't have time resources to implement this. As described above, this is implementable on dnd-core level. It is 100% covered by tests so you can learn from existing tests (e.g. for hover()) and add send a PR to add this.


Adds Option to Enable/Disable Searching & Filtering (In case user already has searched/filtered dataset)

This pull request contains changes that would allow the developer to choose if they would like to use the DataTables built in Search and Filtering. Over time using this package, I found that it's best to handle the searching and filtering before passing the data into DataTables. All default to current default activity (search = true, sort = true)

Thanks but I think you can already disable searching / ordering on the javascript side? Just set the ff options:
$('table').DataTable({
    searching: false,
    ordering: false,
    ...
});

Thanks but I think you can already disable searching / ordering on the javascript side? Just set the ff options:
$('table').DataTable({
    searching: false,
    ordering: false,
    ...
});

Yes but that also disables the search box entirely and this pull request would allow developers to handle search their own way while keeping other functionality intact. Same with sorting.

I see, another work around is to use an empty closure. Do you think the code below would work for you? I wanted to avoid using make() and promote toJson() instead if possible.
        return datatables()
            ->eloquent($query)
            ->order(function () {})
            ->filter(function () {})
...

May I ask why you prefer to promote toJson()?
This is my use-case, I just find it cleaner to not be required to pass empty closures along through a chain when the data passed in is already finished being manipulated.
    if (request()->has('dt')) {
       // Everything I want handled is done here outside of DataTables
       // This method uses another Package that handles eloquent text search using joins, when DataTables does not - this is the key reason for the PR, it allows use of other search packages by bypassing DataTables search.
        $jobs = JobService::searchJobs(
            request()->search['value'] ?? null,
            request()->length ?? null,
        );

        return DataTables::of($jobs)
            ->setTransformer(new JobTransformer)
            ->make(true, false, false); // If PR is merge, instead of your suggestion.
    }```

May I ask why you prefer to promote toJson()?

I wanted to drop support for make(false) and toJson() == make(true) :).
What about updating the methods above that would disable searching and sorting which basically wraps the code above? Maybe something like:
        return DataTables::of($jobs)
            ->order(false)
            ->filter(false)
            ->toJson();

May I ask why you prefer to promote toJson()?

I wanted to drop support for make(false) and toJson() == make(true) :).
What about updating the methods above that would disable searching and sorting which basically wraps the code above? Maybe something like:
        return DataTables::of($jobs)
            ->order(false)
            ->filter(false)
            ->toJson();

I think that would be better than forcing a closure. Thanks Yajra!  Your solution with closures works great; however, i recommend updating to allow false as a param!


Video About Section

[ ] Add a video about section to the site
Stories are the pdf file of the human brain, everyone can get it.
Throw away the fancy schmensy, speak directly to the core of the human being.
This is the tenet in which this company was built...

The pros and cons of having a video about section on a marketing and advertising agency's website:
Pros:

Engagement: Videos tend to engage visitors more effectively than text or images alone. A video about section can capture attention and keep visitors on your website longer.
Personality: A video allows the agency to showcase its personality, team members, and company culture, helping to build a connection with potential clients.
Demonstration: It provides an opportunity to visually demonstrate the agency's services, capabilities, and past projects, giving potential clients a better understanding of what the agency offers.
Differentiation: A well-crafted video can help the agency stand out from competitors and leave a memorable impression on visitors.
SEO Benefits: Including videos on your website can improve search engine optimization (SEO) as search engines often prioritize websites with multimedia content.

Cons:

Cost and Time: Creating high-quality videos can be costly and time-consuming, especially if professional equipment or talent is required.
Technical Issues: Videos may not load properly on all devices or internet connections, potentially leading to a frustrating user experience.
Accessibility: Not all visitors may be able to access or engage with video content, such as those with hearing impairments or slower internet speeds.
Content Quality: Poorly produced videos can reflect negatively on the agency's brand and professionalism, so it's crucial to ensure high production standards.
Maintenance: Regularly updating and maintaining video content can be challenging and may require ongoing investment in time and resources.

Overall, while a video about section can be a powerful tool for engaging visitors and showcasing the agency's personality and services, it's essential to carefully consider the costs, technical considerations, and ongoing maintenance required to ensure its effectiveness.


Please explain why we need to enable trusted service or Resource instance in Firewall settings

[Enter feedback here]

Hi team,
For above step4 in storage settings, if I understand correctly, we whitelist the public access from AML to storage account. Please explain which function requires the public access. As per my testing, data preview is one of them. Why compute cluster job needs to connect to storage via public internet?
Thanks.

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

ID: 226c7926-de43-598b-d9d7-4f5f2ffbd72e
Version Independent ID: 2cf84d0c-927e-3678-4aac-ae15398a2638
Content: Secure an Azure Machine Learning workspace with virtual networks - Azure Machine Learning
Content Source: articles/machine-learning/how-to-secure-workspace-vnet.md
Service: machine-learning
Sub-service: enterprise-readiness
GitHub Login: @jhirono
Microsoft Alias: jhirono

@Jessieliu111
Azure Machine Learning (AML) requires public access to the storage account for several functions, including data preview. The compute cluster job also needs to connect to the storage account via the public internet. This is because the compute cluster job needs to access the data stored in the storage account to perform the training or inference tasks.
You can use Azure Private Link to connect your virtual network to Azure services without a public IP address at the source or destination. The private link platform handles the connectivity between the consumer and services over the Azure backbone network. By mapping private endpoints to your Azure Machine Learning workspace and storage account, you can reduce your data leakage risks.
Ref:https://learn.microsoft.com/en-us/azure/machine-learning/how-to-configure-private-link?tabs=cli

Hi team,
Thanks for the information. It's really helpful.
I'm still confused if storage account is private link enabled, why compute cluster job still needs to connect to the storage account via public internet. Would you please help further explain?
Thanks.
Best regards,
Jessie Liu
Support Engineer | Azure IoT&AI Support Team
Phone: (0510) 6665 8360
Mon-Fri 9:00-17:30/7:00-4:00 (UTC+8)
No.111, Linghu Road, Xin Wu District, Wuxi, Jiangsu, China
@.***
Delighting our customers is our #1 priority. We welcome your comments and suggestions about how we can improve the support we provide to you.
From: Naveenommi-MSFT @.>
Sent: Friday, March 17, 2023 9:30 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Jessie Liu @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Please explain why we need to enable trusted service or Resource instance in Firewall settings (Issue #106734)
@Jessieliu111https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJessieliu111&data=05|01|jiaxinliu%40microsoft.com|fc2099180f004126715008db26ebac38|72f988bf86f141af91ab2d7cd011db47|1|0|638146565914102929|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=iW3CZdEgfPS2Omp17nb86H1vewrcXB3j20n%2Fo4bQ6Pg%3D&reserved=0
Azure Machine Learning (AML) requires public access to the storage account for several functions, including data preview. The compute cluster job also needs to connect to the storage account via the public internet. This is because the compute cluster job needs to access the data stored in the storage account to perform the training or inference tasks.
You can use Azure Private Link to connect your virtual network to Azure services without a public IP address at the source or destination. The private link platform handles the connectivity between the consumer and services over the Azure backbone network. By mapping private endpoints to your Azure Machine Learning workspace and storage account, you can reduce your data leakage risks.
Ref:https://learn.microsoft.com/en-us/azure/machine-learning/how-to-configure-private-link?tabs=clihttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Flearn.microsoft.com%2Fen-us%2Fazure%2Fmachine-learning%2Fhow-to-configure-private-link%3Ftabs%3Dcli&data=05|01|jiaxinliu%40microsoft.com|fc2099180f004126715008db26ebac38|72f988bf86f141af91ab2d7cd011db47|1|0|638146565914102929|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=p03cKjhTJpOlYbslBJ27w25JxANKA0zf3iDKAT1F1Nc%3D&reserved=0

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F106734%23issuecomment-1473847523&data=05|01|jiaxinliu%40microsoft.com|fc2099180f004126715008db26ebac38|72f988bf86f141af91ab2d7cd011db47|1|0|638146565914102929|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=JGOiIlMhJeguk69ng4ovY3%2B6YkYe9DWpiH%2FiEP8fMLU%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAVSBUJTIGPPBJWFDM5OCRLDW4RRMRANCNFSM6AAAAAAV5CIMC4&data=05|01|jiaxinliu%40microsoft.com|fc2099180f004126715008db26ebac38|72f988bf86f141af91ab2d7cd011db47|1|0|638146565914102929|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=oXpd51lFsHFmZ%2FGBnMZlseZZKH8lypQw5DecXOgLz%2Bk%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Hi @Jessieliu111
Thank you for showing interest on this concept.
According to the Azure documentation, a private endpoint is a network interface that connects you privately and securely to a service powered by Azure Private Link. When you create a private endpoint for a storage account, you can connect to the storage account over a private IP address from a virtual network.
However, it seems that the compute cluster job still needs to connect to the storage account via the public internet. This may be because the compute cluster job is not running within the virtual network that the private endpoint is associated with.
You can check the virtual network and subnet that the compute cluster is associated with, and make sure that the virtual network and subnet are the same as the virtual network and subnet that the private endpoint is associated with.
 You can also check the connection string that the compute cluster job is using to connect to the storage account, and make sure that the connection string is using the private endpoint.
If a storage account is Private Link enabled, it means that you can access the storage account over a private endpoint instead of a public endpoint. This private endpoint is essentially a private IP address within your virtual network that you can use to securely access your storage account.
However, even if your storage account is Private Link enabled, your compute cluster job may still need to connect to the storage account via the public internet for a few reasons:
Access control: Private Link only provides secure access to the storage account, but it does not control who can access it. You still need to configure access control for your storage account, either by using shared access signatures (SAS) or Azure Active Directory (AAD) authentication. If your compute cluster job does not have the necessary credentials to access the storage account, it will not be able to access it even if it is Private Link enabled.
Hybrid scenarios: Private Link is designed for scenarios where you want to access your storage account over a private endpoint within your virtual network. However, there may be scenarios where you need to access your storage account from outside your virtual network, for example, from a hybrid scenario that involves on-premises resources. In such cases, you may need to access your storage account via the public internet.
Compatibility: Some applications or services may not support Private Link or may require specific ports or protocols that are not supported by Private Link. In such cases, you may need to access your storage account via the public internet.
Reference links:
Azure Private Link availability | Microsoft Learn
Tutorial: Connect to a storage account using an Azure Private Endpoint - Azure Private Link | Microsoft Learn
What is a private endpoint? - Azure Private Link | Microsoft Learn
Set up service authentication - Azure Machine Learning | Microsoft Learn
I hope this helps. Let me know if you have any other questions.

@Jessieliu111
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Allow hints to Auto-Tag

From<EMAIL_ADDRESS>on May 23, 2013 06:47:00
Feature request:
If I select some files in the list, then type "Foo" into the series field, then click "Auto-Tag", it would be nice if the autotag process would (if it doesn't find anything normally), retried the search replacing the non-numeric parts of the file name with "Foo".
Original issue: http://code.google.com/p/comictagger/issues/detail?id=20

From<EMAIL_ADDRESS>on May 22, 2013 21:54:30
Or... I could just learn to use the override that's already in the option dialog that comes up when you click auto-tag. Yeesh. Sorry.

From<EMAIL_ADDRESS>on July 08, 2013 16:08:09
Heh.  Glad that worked out!
Status: Invalid


Fix indentation logic for Emmet:Remove Tags

This PR fixes #104173.
Currently, when remove tag is invoked, all the lines in between the start and end of the tag adjust indent by one tab.
By testing the feature here, it looks like we should adjust the indent of the internal content by the difference of indent between tag opening  line and content's first line.

Closing in favour of PR #112694


Allow custom location for interpreter.json and die if not writeable

What is this PR for?
This will allow for custom location for interpreter.json.
This will try to create dir if needed.
This will die if file or file's dir not writeable.
What type of PR is it?
Bug Fix: Zeppelin can't manipulate notes when deployed to a write only directory.
How should this be tested?

Test default: when not using a custom file, everything should work as usual.
Specify any file via zeppelin-site.xml AND zeppelin-env.sh and that file should be used.
If the file exists and is not writeable, zeppelin shouldn't start.
If the file's dir doesn't exit, try to create it.
If the file doesn't exist and the file's dir is not writeable, zeppelin shouldn't start.

Questions:

Does the licenses files need update?
No
Is there breaking changes for older versions?
No
Does this need documentation?
Unknown

Thanks for the patch.
I think bin/interpreter.sh doesn't need to be modified while interpreter process does not access conf/interpreter.json.
And It would be better If new configuration can be seen in
https://github.com/apache/incubator-zeppelin/blob/master/docs/install/install.md#zeppelin-configuration

Fixed/Re-sent.
On Mon, Mar 14, 2016 at 1:33 PM, Lee moon soo<EMAIL_ADDRESS>wrote:

Thanks for the patch.
I think bin/interpreter.sh doesn't need to be modified while interpreter
process does not access conf/interpreter.json.
And It would be better If new configuration can be seen in
https://github.com/apache/incubator-zeppelin/blob/master/docs/install/install.md#zeppelin-configuration
—
Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-zeppelin/pull/776#issuecomment-196425546
.


make default responses work when views hook disabled

res.guessView and res.view is always undefined when views hook is disabled.

Hi @Enkows, this looks good.  Would love to see tests for this but it looks like we don't have tests for the responses hook in general, so I'll see about that.

Sorry for the delay.  Ended up going with a somewhat simpler approach here, and merging this into the generator that creates the responses.  Thanks for bringing this to our attention though!


Buld Test pipeline

Should have a test target in Makefile.
In test/cdk we have a cdk app which will deploy two stacks. Stack A has two buckets, stack B is empty.
The test target should:

deploy the CDK app
run the tool and migrate both buckets from A to B
cdk diff should report changes
migrate both resources back from stack B to A
cdk diff should report no changes
cdk DESTROY

Completed via #35


Code line highlight does not work in version 0.11.0

The code line highlight feature with "*" seems to have stopped working in the latest version.
For this sample code snippet:
	buildscript {
		repositories {
			mavenCentral()
			mavenLocal()
		}
		dependencies {
	* classpath 'io.codearte.accurest:accurest-gradle-plugin:0.7.1'
		}
	}

when script src is set to 0.10.2:
<script src="https://gnab.github.io/remark/downloads/remark-0.10.2.min.js">

the highlight works correctly.
With the following setting:
<script src="https://gnab.github.io/remark/downloads/remark-0.11.0.min.js">

it does not.

This was disabled by default in #192, but may be enabled using the highlightLines configuration option.

Ok. Thanks.