Present project on an upcoming conference

let's move on to new horizons: OOPConference 2019 is coming...

I like conference driven development!

same here - that will force you to become committer here :-)

I propose to have the following until WJAX:
[] Jekyll-based website, generated (at least mostly) from AsciiDoc
[] including the architecture documentation as html (similar to the ArchUnit.org site)
[]  including the developer guide
[] including a user guide with examples
[] using diagrams from both PlantUML AND EnterpriseArchitect
[] generating the architecture doc as word document

that list is a good starr.bIbwould also like to go through the docToolchainbfeature list and see if it makes sense to add things like changelog, exportJira, Confluence, Excel, exportMarkdown etc.

Ralf - I just wanted to list things here that are relevant for WJAX AND htmlSanityCheck... there should definitely be additional things (from docToolchain) included in our workshop...

agreed. I just wanted to do the quick check from the other side of the fence... I guess this list will evolve a bit with our preparations...

Hi folks, I'm looking for the "import excel data as tables into asciidoc" features. From (usually well informed) sources I heard, this might be possible. Only info I found on this repo leads eventually to a 404:
https://github.com/aim42/htmlSanityCheck/blob/9b1bb6be5b41bb6ad14c4a8299200e0c9a3cccdd/src/docs/excel/Requirements.xlsx/General.adoc

it is part of docToolchain and the task is called exportExcel (from the view of Excel, it is an Export 😉)
https://doctoolchain.github.io/docToolchain/#_exportexcel
this repository currently contains a good example.

Which arrow? I mainly have this diagram in mind:
https://doctoolchain.github.io/docToolchain/#_overview_of_available_tasks
Most data is first exported from the original tool and then included or referenced from the asciidoc documentation... maybe this thinking leads to misunderstanding...

I think the diagram is absolutely correct.
Here's how my misinterpretation happened:
I just returned from sitting through the second day of Gernot's training.
And this quote was ringing through the back of my head
"Focus on interfaces to the external systems and the dataflow" (Scoping / Context diagrams)
I then, while scanning the docs for "creating asciidoc tables from excel files" put my visual focus on this:

I thought to myself: well this can't be "importing excel into asciidoc,because whatever that green box is, it is not included into the asciidoc document. Instead asciidoc pushes something into that box.
I didn't notice the word 'include' that was too far away.
Perhaps, ommitting the arrow head from the connector between the yellow box (your solution...) and the green one would have made me think and eventually discover the word 'include'
But once again: Would I've spent more time on scrutinizing the diagram, I'd interpreted it correctly. I just overlooked the info I was looking for at first glance.

Thanx for the detailed feedback - it will help me to create better diagrams!

"Focus on interfaces to the external systems and the dataflow" (Scoping / Context diagrams)

very good advice!
The diagram is already the second iteration. The first one contained the same components and connectors but had nearly no structure and thus was unreadable.
Regardings the arrows and what the connectors mean: (I should add this to the documentation)
You mostly have the choice between dataflow and action (which component invokes which other component and thus is the active one).
I decided to use the latter.
And as you can see, I also mostly don't show components but documents. I guess this even makes it harder to understand...
I wonder if there is a better way to visualize this.
The position of the labels is definitely something which can be done better...

Are you still interested in a conference submission on HSC and docToolchain, @gernotstarke + @rdmueller.
Perhaps we could address this in the next months for the second half of 2024 or 2025?

ok for me. My schedule for 2024 is tight. Options would be WJAX in Munich and IT-Tage in Frankfurt.
Presenting with you guys would be a pleasure for me.

Great idea. I can support you with slides etc if needed...


garden-ai purge to get rid of any local Docker artifacts created/pulled by Garden

Because Garden goes and runs Docker commands on behalf of the user, they will end up with cruft images eventually. There should be a command that removes any Docker images and containers created or pulled by Garden. That way the user doesn't have to fumble around with docker commands to reclaim disk space.
garden-ai weed might be fun too 🤔
Assumptions:

Acceptance Criteria
Given..., when..., then...

may need to rethink component


test seek closed file

Narrowed down to test_wheel, in particular the following:
def test_entry_points_conflict():
    clear_samples_dist()
    with pytest.raises(EntryPointsConflict):
        wheel_main(samples_dir / 'entrypoints_conflict.ini')

If one record warnings:
py.test -p recwarn tests/test_wheel.py
=================================================================== test session starts ===================================================================
platform darwin -- Python 3.5.2, pytest-3.0.4.dev, py-1.4.31, pluggy-0.4.0
rootdir: /Users/bussonniermatthias/dev/flit, inifile:
plugins: warningsfilters-0.0.1, xonsh-0.4.6
collected 1 items

tests/test_wheel.py .

=== 1 passed in 0.13 seconds  ===
Exception ignored in: <bound method ZipFile.__del__ of <zipfile.ZipFile [closed]>>
Traceback (most recent call last):
  File "/Users/bussonniermatthias/anaconda/lib/python3.5/zipfile.py", line 1595, in __del__
    self.close()
  File "/Users/bussonniermatthias/anaconda/lib/python3.5/zipfile.py", line 1607, in close
    self.fp.seek(self.start_dir)
ValueError: seek of closed file

You were involved in the zipfile patch, so pinging you first. I'm gonna assume a self._fpclose(self.fp) is called without setting self.fp to None.
I don't see where except maybe zef_file.close()...

I spotted this in a test run last night, but was too tired to look into it. Thanks for the reminder!
I think I understand it now.

zf._fpclose() does not close the underlying file, because we pass in a file handle, so self._filePassed == 1.
However, the context manager in wheel_main will close the underlying file handle.
If an error occurs while building the wheel, WheelBuilder.build() never calls self.wheel_zip.close()
When wheel_main exits, the WheelBuilder goes out of scope, and is garbage collected along with its zip file, after the context manager has closed the underlying file handle.

So it should be a simple bit of try/finally. Fix coming up...

Yup, that seemed to do it.

Ok, great !


How to add custom component in ItemDetail

Hi,
Can I add custom component created in presenter into ItemDetail?
In template maybe with call the control macro:
{block detail}   {control test} {/block}
Thank you.

@zemnet Well, try to do it like this: {control $control->getPresenter()->getComponent('myCoolComponent')}

done, works good.
thank you...


Drop executable output from QJSC?

It depends on how the Makefile is written and calls the system compiler. The option to output a C file with a main() section would be its replacement, so getting an executable would require running a compiler oneself.
Thoughts?

Sounds reasonable to me. Alternative: merge tcc and qjsc :p


Style attribute for embedded image

When styling emails, it is necessary to use inline styles in the style attribute, because webmail services (gmail and friends) strip out stylesheets.
The @Html.EmbedImage extension method can render the alt attribute, but not a style attribute. So I can't style my images.
The signature could be changed from:
public static IHtmlString EmbedImage(this HtmlHelper html, string imagePathOrUrl, string alt = "")

to:
public static IHtmlString EmbedImage(this HtmlHelper html, string imagePathOrUrl, string alt = "", string style= "")

And the final line from:
return new HtmlString(string.Format("", linkedResource.ContentId, html.AttributeEncode(alt)));

to:
return new HtmlString(string.Format("", linkedResource.ContentId, html.AttributeEncode(alt), html.AttributeEncode(style)));

What do you think?

I've added a PR.

Could also strip the helper so that it is only a uri. That way you could write the style in a  tag
like

upvote

+1

Can someone tell me the syntax for the @HTML embed image tag with styles applied?
For example, how would you add styles to this:
@Html.EmbedImage("~/Content/graphics/global/logo.png")


WIP Dphan/cmake poco improvements

Doesn't work with conan
the issue is find package for poco - it must be used with conan and cannot be used with CMake approach

abandoned, not usefull anymore, the goal was to build with poco lib in ide like visual studio. better approach is to append CMAKE_MODULE_PATH with path to poco build with e.g. conan (either in cmake or in build options )


Fastlane - Integrate Appium python scripts

Hi,
Could anyone help me out with fastlane.
I want to integrate IOS Appium python scripts through fastlane.
Thanks in advance
Best Regards,
Bindu

I want to integrate IOS Appium python scripts through fastlane.

What does that mean? Can you provide some links? What exactly do you want to do why and when?


gym_hybrid: replay_path

Can you remove the hastag in front of replay_path='gym_hybrid_ddpg_seed0/video' at DI-engine/dizoo/gym_hybrid/config/gym_hybrid_ddpg_config.py, ?

It is not a necessary field so we just comment it in default config. You can choose whether to enable it according to your demand.


The plugin doesn't seem to be effective

Hi there, I really appreciate this plugin, I believe it could be a life-saver, but currently, I just cannot make it work.
I created an empty folder, add one file called consts.ts within:
const pl = [
  'prettier-plugin-sort-json', 'prettier-plugin-packagejson',
  'prettier-plugin-multiline-arrays', 'prettier-plugin-organize-imports', 'prettier-plugin-jsdoc',
]

// prettier-elements-per-line: 2 1
const availableTags = [
	'a', 'aside', 'b', 'blockquote', 'br', 'code', 'em', 'figcaption', 'figure', 'h3', 'h4', 'hr', 'i', 'iframe', 'img', 'li', 'ol', 'p', 'pre', 's', 'strong', 'u', 'ul', 'video',
	'table',
]

Install prettier and this plugin:
yarn add  prettier prettier-plugin-multiline-arrays --dev

Add the config file .prettierrc.js:
const prettierConfig = {
  plugins: [
      'prettier-plugin-multiline-arrays', // plugin added here
  ],
  printWidth: 100,
  singleQuote: true,
  tabWidth: 4,
  multilineArrayWrapThreshold: 3,
  multilineArrayElementsPerLine: "3",
};

module.exports = prettierConfig;

And then run prettier yarn prettier const.ts, but the result shows single line elements in array:
[debug] normalized argv: {"_":["./consts.ts"],"color":true,"editorconfig":true,"loglevel":"debug","configPrecedence":"cli-override","debugRepeat":0,"ignorePath":".prettierignore","plugins":[],"pluginSearchDirs":[],"__raw":{"_":["./consts.ts"],"color":true,"editorconfig":true,"loglevel":"debug","config-precedence":"cli-override","debug-repeat":0,"ignore-path":".prettierignore","plugin":[],"plugin-search-dir":[]}}
[debug] resolve config from 'consts.ts'
[debug] loaded options `{"plugins":["prettier-plugin-multiline-arrays"],"printWidth":100,"singleQuote":true,"tabWidth":4,"multilineArrayWrapThreshold":3,"multilineArrayElementsPerLine":"3"}`
[debug] applied config-precedence (cli-override): {"multilineArrayElementsPerLine":"3","multilineArrayWrapThreshold":3,"plugins":["prettier-plugin-multiline-arrays"],"printWidth":100,"singleQuote":true,"tabWidth":4}
const pl = [
    'prettier-plugin-sort-json',
    'prettier-plugin-packagejson',
    'prettier-plugin-multiline-arrays',
    'prettier-plugin-organize-imports',
    'prettier-plugin-jsdoc',
];

// prettier-elements-per-line: 2 1
const availableTags = [
    'a',
    'aside',
    'b',
    'blockquote',
    'br',
    'code',
    'em',
    'figcaption',
    'figure',
    'h3',
    'h4',
    'hr',
    'i',
    'iframe',
    'img',
    'li',
    'ol',
    'p',
    'pre',
    's',
    'strong',
    'u',
    'ul',
    'video',
    'table',
];
✨  Done in 1.15s.

I don't know what's wrong and how to fix, please help.

Try this in your .prettierrc.js file:
plugins: [
      './node_modules/prettier-plugin-multiline-arrays', // plugin added here
  ],

It seems this is a common enough error, I'll update it in the README. Hopefully this helps your case!

Thanks for the help, however, the change still doesn't work, the output is the same as what I posted above.
And I think in both cases the plugin is loaded, because when I tried to use a invalid value for the option, the plugin will raise an error as below:
[debug] normalized argv: {"_":["./consts.ts"],"color":true,"editorconfig":true,"loglevel":"debug","configPrecedence":"cli-override","debugRepeat":0,"ignorePath":".prettierignore","plugins":[],"pluginSearchDirs":[],"__raw":{"_":["./consts.ts"],"color":true,"editorconfig":true,"loglevel":"debug","config-precedence":"cli-override","debug-repeat":0,"ignore-path":".prettierignore","plugin":[],"plugin-search-dir":[]}}
[debug] resolve config from 'consts.ts'
[debug] loaded options `{"plugins":["/Users/reorx/Code/test-prettier/node_modules/prettier-plugin-multiline-arrays"],"printWidth":100,"singleQuote":true,"tabWidth":4,"multilineArrayWrapThreshold":3,"multilineArrayElementsPerLine":3}`
[error] Invalid multilineArrayElementsPerLine value. Expected a string, but received 3.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Oh nice, good idea to check that. I'll investigate!

I'm sorry, I'm not able to reproduce this 😕
I created a separate repo for testing this case and found no issue: https://github.com/electrovir/multiline-array-testing
Maybe something else is wrong with your setup? With your prettier config, the pl should also be formatted differently due to your config value multilineArrayElementsPerLine: "3", as can be seen in the test repo here: https://github.com/electrovir/multiline-array-testing/blob/main/const.ts

If you can provide more info I'll gladly help more! But with everything in here currently I can't help with this issue, so closing for now.


Update README.md

Proposed changes
YES THIS IS A PR
Checklist
Before creating a PR, run through this checklist and mark each as complete:

[ ] I have read the contributing guidelines.
[ ] I have signed the F5 Contributor License Agreement (CLA).
[ ] If applicable, I have added Molecule tests that prove my fix is effective or that my feature works.
[ ] If applicable, I have checked that any relevant Molecule tests pass after adding my changes.
[ ] I have updated any relevant documentation (defaults/main/*.yml, README.md and CHANGELOG.md).

test

It won't let me clickity click


Backward::Backward IMPORTED target expects backward.hpp to live in the same directory as BackwardConfig.cmake upon install which it does not

I'm building and installing 90398eef20f4e7e0e939322d7e84f9c52078a325 in Ubuntu 22 LTS. I'm installing it to a non-default location within the binary directory I used to build backward-cpp. My intention is to use find_package to find this installation of backward-cpp then link to the Backward::Backward target to get access to backward.hpp. While I can successfully find this installation, the Backward::Backward target fails to include the path to backward.hpp as an interface include directory. This means I cannot find backward.hpp and compilation fails. Here is the value of the INTERFACE_INCLUDE_DIRECTORIES target property on Backward::Backward:
-- Backward::Backward_INTERFACE_INCLUDE_DIRECTORIES: /usr/include;/home/thrasher/Downloads/backward-cpp/build/install/lib/backward

I think the problem comes down to this line which assumes that Backward.cmake and backward.hpp always live in the same directory. This is true when building Backward and is probably fine for those consuming Backward as a submodule or via FetchContent but upon installation, backward.hpp is put in ${CMAKE_INSTALL_INCLUDEDIR} and BackwardConfig.cmake is put in ${CMAKE_INSTALL_LIBDIR}/backward.
The install coincidentally works if I install to the default location of /usr/local because CMake always checks /usr/local/include for headers but when installing anywhere else it reveals this bug in the install logic.
I would recommend adding a CI job that installs backward-cpp to a non-system location then tries to consume it. It would have caught his problem. Here's an example of doing install interface tests.

I will gladly merge a pull request. I don't know enough of CMake to quickly fix it, and this is not something I want to spend time on.

Something like this?
# look first in "usual" places for the backward header
find_path(BACKWARD_INCLUDE_DIR NAMES "backward.hpp")
if(NOT ${BACKWARD_INCLUDE_DIR})
    set(BACKWARD_INCLUDE_DIR "${CMAKE_CURRENT_LIST_DIR}")
endif()

...though I'm not sure if this is idiomatic CMake, or if it might break in other environments from my vanilla linux system

Actually on second thought, might even be as simple as:
find_path(BACKWARD_INCLUDE_DIR NAMES "backward.hpp" HINTS ${CMAKE_CURRENT_LIST_DIR})

backward-cpp's build is not idiomatic in a number of ways so it's hard to picture what a minimal fix would look like

I ran into this issue too and was able to get reasonable results with
if (TARGET Backward::Backward)
    set(BACKWARD_INCLUDE_DIR "${CMAKE_CURRENT_LIST_DIR}")
else()
    cmake_path(SET BACKWARD_INCLUDE_DIR NORMALIZE "${CMAKE_CURRENT_LIST_DIR}/../../include")
endif()

Strictly speaking, it would be better to make BackwardConfig.cmake configurable and honor non-default CMAKE_INSTALL_INCLUDEDIR settings rather than hard-coding include, but I don't know how likely such customization is in practice.

Oops, that fails to find backward.hpp when building tests, presumably due to wrongly taking the second branch.  I suppose the way to go is find_path with both of those possible locations as hints.


Port Build 2024 Drawing samples into WASDK 1.6 exp 2 branch.

Description
This ports the C# and C++ DrawingIslands samples that were demoed at Build 2024 into the release/experimental branch. It also removes the older Squares sample that was tied to WASDK 1.2 Exp 1.
Target Release
WASDK 1.6 Exp 2.
Checklist

[x] Samples build and run using the Visual Studio versions listed in the Windows development docs.
[x] Samples build and run on all supported platforms (x64, x86, ARM64) and configurations (Debug, Release).
[x] Samples set the minimum supported OS version to Windows 10 version 1809.
[x] Samples build clean with no warnings or errors.
[x] [For new samples]: Samples have completed the sample guidelines checklist and follow standardization/naming guidelines.
[ ] If I am onboarding a new feature, then I must have correctly setup a new CI pipeline for my feature with the correct triggers and path filters laid out in the "Onboarding Samples CI Pipeline for new feature" section in samples-guidelines.md.
[ ] I have commented on my PR /azp run SamplesCI-<FeatureName> to have the CI build run on my branch for each of my FeatureName my PR is modifying. This must be done on the latest commit on the PR before merging to ensure the build is up to date and accurate. Warning: the PR will not block automatically if this is not run due to '/azp run' limitation on triggering more than 10 pipelines.

@microsoft-github-policy-service agree

@ionvasilian, can you merge the PR? I don't have permissions.


Write proper gemspec

Fixed in https://github.com/zerobotlabs/relax-rb/commit/3e7f132aea89ab9018618739a67858e11adbeae2


[FLINK-22144][runtime][runtime-web] Adds more documentation and moves option

What is the purpose of the change
Minor changes to improve usability for FLINK-6042 changes.
Brief change log

jobmanager.exception-history-size is moved to web.exception-history-size
the documentation was slightly extended
some minor mistakes in the docs are fixed

Verifying this change
Manual visual test of web UI
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Overall looks good. Two minor issues:

Are the docs/themes/book changes necessary?
I think you forgot to regenerate options documentation. Tests are failing.

You're right. I noticed the test failure already. I missed regenerating the docs. The docs/themes/book lingered around in my checkout beforehand. I need to clean that one up.
I fixed the issues. Let's wait for AzureCI to give a 👍 .

Looks good now. Azure passed. Merging.


Can a command support multiple shortcuts?

Android Stuido supports multiple shortcuts for one command.
The nice thing about this is that I can use the default shortcuts when I want to, and I can use custom shortcuts.
And you can easily know that I have modified this shortcut key.
If vscode is not supported now, will it be supported in the future?
Looking forward to your reply! Thanks!

/question
Yes, you can specify any number of shortcuts in keybindings.json.
You can open using Ctrl+Shift+P(or F1) and paste >Preferences: Open Keyboard Shortcuts (JSON)


✨ Invisible characters

Description
We want to provide users with the ability to show invisible characters. In order to do this, our text view needs to support this.

Invisible
Character
Unicode
Name

Space
·
U+00B7
Middle Dot

Newline
¬
U+00AC
Not Sign

Tab
⟶
U+27F6
Long Rightwards Arrow

Return
¤
U+00A4
Currency Sign

Hard Return
¶
U+00B6
Pilcrow Sign

Carriage Return
↵
U+21B5
Downwards Arrow with Corner Leftwards

We should be able to both toggle visibility and customize which character is rendered for the following

Carriage return (CR)
End of line (EOL)
Space
Tab

Screenshots
Visible

Hidden

Menu item in Xcode

Atom Customization Setting

He is working on this for STTextView. https://x.com/krzyzanowskim/status/1688338147392815105?s=46&t=SWZSpaV3dOcD1wyXafRdPg

Just found out about the project, kudos!
Ideally, "standard" characters (space/tab/CR/NL) could be toggled separate from everything else (hair-spaces, word-joiners, etc., etc.). Personally, I never need/want to see space/tab/newline, but I need to be able to see all the other "invisible" characters (including things that are "visible", as well as be able to tell the difference between characters that are normally indistinguishable from other characters, e.g. no-break-space, no-break-hyphens, etc.).
Probably not first pass, but possibly something to keep in mind down the road?
Thanks for considering!


Missing GetWindowLongPtr in Package

I was suprised about this missing in User32 class.
I get an exception because of x64 handle.
On P/Invoke web site, you wrote a sub to abstract x32/x64 GetWindowLong depending on IntPtr.Size.
But not inclued into Package too.

We don't typically release after every individual change, and we've only had two changes since our 0.5.147 release. But since you hit it and want it, I'm happy to oblige.
0.5.152 is being pushed to nuget.org right now.


Update Project

Fixed all the warnings:

increased deployment target on iOS to 6.1 to fix object file (/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../lib/clang/7.3.0/lib/darwin/libclang_rt.ios.a(chkstk.S.o)) was built for newer iOS version
safe cast object to silence warning
run tests against iOS 9,3
added missing dynamic property
silence %d warning
replaced deprecated methods
silenced warning for pending test (+1 squashed commit)
removed deprecated run unit test script

Notes:
I have noticed that the tests hangs on semaphore_wait_trap because of NSURLCache that's why travis takes ages to complete a build, not sure if anything can be done about it.
I have also noticed that the failing tests pass when run one by one.

👍🏻


EGL_WAYLAND_Y_INVERTED_WL support

Our wl_drm implementation is missing support for EGL_WAYLAND_Y_INVERTED_WL. Apparently it's no big deal since we don't have any bug reports about this. Would still be nice to do the right thing.

Apparently I can't read and we already do this.


validate decorator now supports async flask endpoints

Pull Request Test Coverage Report for Build 309

23 of 25 (92.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 99.743%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pedantic/decorators/fn_deco_validate/fn_deco_validate.py
23
25
92.0%

Totals

Change from base Build 307:
-0.3%

Covered Lines:
1356

Relevant Lines:
1358

💛  - Coveralls


Effects like a Blur\Shadows

I want to make a card in Material Design that implies shadow effect. Are there any capabilities to do it?

WPF has a Effect Parameter, does Avalonia has something similar?

No built in effects yet, perhaps it might be possible with some kind of border and a gradient brush. Would be nice to get support for this though.

How I understood, rendering API doesn't support shaders like in WPF?

You don't need shader support there is sharon effect build in d2d https://msdn.microsoft.com/en-us/library/windows/desktop/hh706334(v=vs.85).aspx and skia https://developer.xamarin.com/api/member/SkiaSharp.SKImageFilter.CreateDropShadow/ but its not implemented in avalonia

Thanks, I'll try to implement this feature where I can

Just for case, there is BoxShadow in Avalonia 0.10+
https://github.com/AvaloniaUI/Avalonia/pull/3871
Although there still no convenient in-app blur.

In Avalonia v0.10.0-rc1, If I wrap BoxShadow value into multiple lines, for example,
                    <Setter Property="BoxShadow">
                        <Setter.Value>
                            inset -1 0 1 -1 #10000000,
                            inset -2 0 2 -2 #20000000,
                            inset -6 0 6 -6 #16000000,
                            inset -16 0 16 -16 #16000000
                        </Setter.Value>
                    </Setter>

then the view cannot load (but don't throw any exceptions). Is this intended?

In Avalonia v0.10.0-rc1, If I wrap BoxShadow value into multiple lines, for example,
                    <Setter Property="BoxShadow">
                        <Setter.Value>
                            inset -1 0 1 -1 #10000000,
                            inset -2 0 2 -2 #20000000,
                            inset -6 0 6 -6 #16000000,
                            inset -16 0 16 -16 #16000000
                        </Setter.Value>
                    </Setter>

then the view cannot load (but don't throw any exceptions). Is this intended?

but don't throw any exceptions

It throws:
System.FormatException: 'One of the identified items was in an invalid format.'
Avalonia.Media.BoxShadow.Parse(string)
Avalonia.Media.BoxShadows.Parse(string)

but don't throw any exceptions

It throws:
System.FormatException: 'One of the identified items was in an invalid format.'
Avalonia.Media.BoxShadow.Parse(string)
Avalonia.Media.BoxShadows.Parse(string)

Oh, sorry... Didn't noticed that the exception is caught internally.
In CSS, newlines are allowed between tokens for all property, but not inside quoted values. Sometimes newlines are useful for keeping code tidy. The following code is valid in Chrome and Firefox:
.ugly-syntax {
  box-shadow: 10px
      5px         5px
   blue,
-10px -10px 4px
   red;
font-family: Segoe
                UI, sans-serif;
}

and this does not:
.invalid-syntax {
font-family: "Segoe
                UI", sans-serif;
}

Oh, sorry... Didn't noticed that the exception is caught internally.
In CSS, newlines are allowed between tokens for all property, but not inside quoted values. Sometimes newlines are useful for keeping code tidy. The following code is valid in Chrome and Firefox:
.ugly-syntax {
  box-shadow: 10px
      5px         5px
   blue,
-10px -10px 4px
   red;
font-family: Segoe
                UI, sans-serif;
}

and this does not:
.invalid-syntax {
font-family: "Segoe
                UI", sans-serif;
}

We have BoxShadow for a while and now we have effects Effect="blue(10)" syntax in 11.0


Add assumptions to make vector and matrix *() :total

Having these functions infer :nothrow correctly by the compiler can significantly help with optimization in the compiler, so eventually we should try to make sure that most of QEDbase functionality has as many effects inferred as possible.
I've added comments explaining the assumed effects and why they are correct. We should probably have this as a guideline for @assume_effects in general.

It might be nice in the future to have automated tests checking that the effects of certain functions are what they are supposed to be. Using Base.infer_effects, this should be relatively simple and would prevent us or new Julia/dependency updates from accidentally taking effects away.

I believe getTransverseMomentum should also be :nothrow. Since we can't control the element types of the Lorentz vectors I don't think this can currently ever be safe though.
@inline @traitfn function getTransverseMomentum(lv::T) where {T; IsLorentzVectorLike{T}}
    return sqrt(getTransverseMomentum2(lv))
end


Delete schedule

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: todesj:x: Levi-LeschesYou have signed the CLA already but the status is still pending? Let us recheck it.

I force-pushed to clean the history, now there are no merge conflicts


Makes foot prints only after selecting gameobject

See title. The issue is that after building the project you won't see any footprints as you can't select any gameobject in the hierarchy anymore.

True, I am aware of the issue. It’s a problem with the GUI, but didn’t have time to make the change. Sorry about that! I’ll try to fix it.


Switch Scene only recognizes the first child Scene and it's key

Version
Tell us which versions you are using:

react-native-router-flux v3.?.?
3.35.0
react-native v0.?.?
0.31.0

Expected behaviour
Switch Scene recognizes child Scenes and their keys and can switch between them depending on the selector's return value.
Actual behaviour
Only the first child is recognized. If the selector callback returns the keyname of the second child (here scene2) the error A scene for key "scene2" does not exist. occurs. If you inspect the props.navigationState.childrenvalue in ./src/Switch.js you can see that the array only contains the first value.
Steps to reproduce
Use the following example:
/**
 * Sample React Native App
 * https://github.com/facebook/react-native
 * @flow
 */

import React, { Component } from 'react';
import {
  AppRegistry,
  StyleSheet,
  Text,
  View
} from 'react-native';

import { Scene, Switch, Router, ActionConst } from 'react-native-router-flux';

const scene1 = (props) => {
  return (
    <Text>
      Welcome to React Native!
    </Text>
  );
}

const scene2 = (props) => {
  return (
    <Text>
      To get started, edit index.ios.js
    </Text>
  );
}

class testrouter extends Component {
  render() {
    return (
      <Router>
        <Scene
          key="root"
          component={Switch}
          selector={() => { return "scene2" }}
        >
          <Scene key="scene1" component={scene1} />
          <Scene key="scene2" component={scene2} />
        </Scene>
      </Router>
    );
  }
}

AppRegistry.registerComponent('testrouter', () => testrouter);

Important is, that the selector returns the keyname for the second child. For the first child it's working.

Are there any workarounds for this? I've just hit this bug as well.

i hava the same problem , whta time can be fixed?

Same issue

+1

+1 it's a pretty big blocker. Any ideas on how to go around this?

The documentation basically answers this already, with it's current implementation it only works correctly with scenes that have 'tabs={true}'. You can just set tabs to true for the parent scene and it works:

New feature for 3.x release is custom scene renderer that should be used together with tabs={true} property. It allows to select tab scene to show depending from app state. Add the prop "unmountScenes" to your switch scene if the tabs should be unmounted when switched. It could be useful for authentication, restricted scenes, etc. Usually you should wrap Switch with redux connect to pass application state to it: Following example chooses scene depending from sessionID using Redux:

The key seems to be changed for some reason. When I started using Switch, I noticed that my nav buttons weren't highlighting correctly. The key name for the child route is changed from "Foo" to "Foo_"

@spltTrevor but even if you do tabs = {true} and you console logs the navigation children, you will notice that some of the scene keys are simply missing :/
Is that the expected behaviour? Also in their docs they do mention that this is a good way to have restricted (i.e. loggedin/out) scenes.

Adding tabs={true} fixes the original issue posted by @nschoen. The switch expected functionality does work with this. Also, when I console log the navigation state using tabs={true}, it does show all of the proper scenes (not just the first one), but @braco is correct in saying using the switch does nest these children further into a scene nested in the switched to scene. It puts Foo into Foo_.

@spltTrevor hmm interesting. I wonder if it's because I am doing ActionConst.RESET as the navigation type, because even with tabs={true} I was having the issue of the disappearing scene.
I'll investigate more and hopefully it's what @braco noticed.

@spltTrevor I'm getting this error, even with tabs={true}. My case is very similar to the example you posted, the weird thing is: this was working before, but doesn't seem to be working anymore. A quick inspection on the parameter sent to the connect(state => ({ user: state.user }))(Switch) selector shows that the children are being removed.

I was experiencing this error while using the latest version (installing with commit number). After installing 3.35 I don't see this happening anymore. Could this be related to #1057?

There are 172 open issues, the API is unstable, and these problems are so substantial that I'm wondering if it isn't time to move away to a simpler, more stable package. Or just bare metal:
https://github.com/fbsamples/f8app/blob/master/js/F8Navigator.js
https://github.com/react-native-simple-router-community/react-native-simple-router
<EMAIL_ADDRESS>https://github.com/satya164/Croma/blob/master/src/components/Scene.js#L74
https://github.com/fbsamples/f8app/blob/master/js/PushNotificationsController.js

@braco You are free to choose any package you want, nobody asked you to use this component. It is open source component, why you don't try to fix it and submit PR? Or do you want just to use some other labor for free?
Btw, use master to your own risk (like react-native master itself). Do you have this issue with 3.35?

@braco Ask for stable API from React Native Experimental team.
API of this component is almost not changed almost more than year. Community (but not you, sure) adds more and more props, but it is not called 'unstable API'.

Huh, are you joking guys? @spltTrevor is fully correct. Switch could be used only with tabs, it is clearly stated in docs
"New feature for 3.x release is custom scene renderer that should be used together with tabs={true} property."
Unfortunately i can't review all such "bugs" in time...

As you guys might know (or not :P) i've been following pretty closely, and I also experience this bug with Switch on latest HEAD. Something definitely broke but not sure what exactly.

Do you use ‘tabs’ as stated in docs?

On 07 Sep 2016, at 02:31, Adrian Perez<EMAIL_ADDRESS>wrote:
As you guys might know (or not :P) i've been following pretty closely, and I also experience this bug with Switch on latest HEAD. Something definitely broke but not sure what exactly.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/aksonov/react-native-router-flux/issues/1081#issuecomment-245138761, or mute the thread https://github.com/notifications/unsubscribe-auth/ABQpcU4VD1HBjq09U5tqw3XVp655kJWDks5qngXugaJpZM4JpJ6z.

I am using the tabs as stated in the docs, so much that everything works on 3.35. But just as @blackxored stated, on latest HEAD (which I'm using for passing props to tabs) this error occurs.

@rseemann Could you check if it would work with latest master but without #1057 ?

@aksonov I haven't been able to test this yet, if anyone feels like taking this over, please feel free.

Guys, is there any way to recognize the requested scene within the Switch selector so I can decide weather to pass that scene (ex. it's public and don't require a valid user):
Something like this pseudo Switch selector code:
`
if(requestedScene.props.isPublic === true){
    return requestedScene.key;
}

return props.user.data ? "home" : "start";`

For example, I'll have two buttons on start (Register & SignUp) which are public scenes and I'd like to render them.
@aksonov, the lib looks very promising, thanks for sharing and for the effort supporting it!


[HUDI-440] Rework the hudi web site

What is the purpose of the pull request

Impore user experience.
Quick overview https://lamber-ken.github.io

Brief change log

Replace old web site theme.
Sync docs files.
Remove useless css, js, docs etc.
Design the landing page.
Design the quick links nav at left sidder bar.
Add Roadmap, Activities, FAQ nav tab.
Add two activities post at Activities channel.
Adjust blue / white style.
Rewhite the jekyll Dockerfile .
Make my GH domain as a public test site.

Verify this pull request
New docs and site are placed at newui folder. You can go to newui/docs folder, and execute
docker-compose build --no-cache && docker-compose up.
After that, you can visit the new site locally at http://localhost:4000.
incubator-hudi-doc
├── content
├── docs
└── newui
    ├── content
    └── docs

Committer checklist

[x] Has a corresponding JIRA in PR title & commit

[x] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

There are few things already here.. https://issues.apache.org/jira/browse/HUDI-270 can we consolidate everything under this umbrella task please?
I plan to take a full pass at this in next few days. One guidance I have is to mostly not change any content from original site in this PR.. and just stick to UI improvements.
I still feel strongly about having right navigation. Its so much easier to jump to a section.. This is why ToCs are for.. For e.g: https://docs.confluent.io/current/streams/code-examples.html on this, you can easily click and go to. section of code one wants to copy-paste.. Its super useful.. So lets please bring the right nav back on quickstart, demo and reading, writing pages atleast..

There are few things already here.. https://issues.apache.org/jira/browse/HUDI-270 can we consolidate everything under this umbrella task please?
I plan to take a full pass at this in next few days. One guidance I have is to mostly not change any content from original site in this PR.. and just stick to UI improvements.
I still feel strongly about having right navigation. Its so much easier to jump to a section.. This is why ToCs are for.. For e.g: https://docs.confluent.io/current/streams/code-examples.html on this, you can easily click and go to. section of code one wants to copy-paste.. Its super useful.. So lets please bring the right nav back on quickstart, demo and reading, writing pages atleast..

Okay, we can make the right toc as an option, the default is off.

Qucik Preview https://lamber-ken.github.io/docs/querying_data

Hi @vinothchandar, thanks for your review. I updated pr according to your advice. 😃

The hyperlinks back and forth are great..  Let me make a another pass for any typos/rewordings.
In the meantime, for ease of review

Can you call out pages where there is new content (i.e text) added and if any pages needs to be retranslated to chinese for e.g
Can you also prepare a README with complete instructions on how to get the new and old site up and running using a single command (how the old site was), if any special changes are needed..

The hyperlinks back and forth are great.. Let me make a another pass for any typos/rewordings.
In the meantime, for ease of review

Can you call out pages where there is new content (i.e text) added and if any pages needs to be retranslated to chinese for e.g
Can you also prepare a README with complete instructions on how to get the new and old site up and running using a single command (how the old site was), if any special changes are needed..

Thanks for reviewing again. I've done a lot of preparatory work. Just publishing the site as before is ok.
incubator-hudi-doc
├── content
├── docs
└── docs-new

hi @vinothchandar, I had talked with @leesf about the chinese-translation. Will not translate every page in the new site, just translate several core articles to help user to learn about hudi quickly.

Few suggestions

Change "Note: Click here to try our new site."  to "Hello! Click here to try our new site"

Change "Note: This site is work in progress, if you notice any issues, please Report on Issue. Click here back to old site." to "Note: This site is work in progress, if you notice any issues, please file a jira. Click here to go back old site.

Lets file a JIRA to replace "Activities" with the @apachehudi twitter stream? or replace that with a "blog"

Under community page , "Useful resources for contributing can be found under the “Developers” top menu. " this line is no longer true, since there is no developers menu any more?

In quickstart, hyperlink of "commit" in this text : Each write operation generates a new commit denoted by the timestamp : points back to original site? Can you please fix and also click on every link within all the pages in Documentation once and confirm there are no such occurrences ?

We can merge this tonight, if you could address all this.. Night time PST, I will be around to revert etc if anything goes wrong also :)

Few suggestions

Change "Note: Click here to try our new site."  to "Hello! Click here to try our new site"
Change "Note: This site is work in progress, if you notice any issues, please Report on Issue. Click here back to old site." to "Note: This site is work in progress, if you notice any issues, please file a jira. Click here to go back old site.
Lets file a JIRA to replace "Activities" with the @apachehudi twitter stream? or replace that with a "blog"
Under community page , "Useful resources for contributing can be found under the “Developers” top menu. " this line is no longer true, since there is no developers menu any more?
In quickstart, hyperlink of "commit" in this text : Each write operation generates a new commit denoted by the timestamp : points back to original site? Can you please fix and also click on every link within all the pages in Documentation once and confirm there are no such occurrences ?

We can merge this tonight, if you could address all this.. Night time PST, I will be around to revert etc if anything goes wrong also :)

hi @vinothchandar thanks for your review.
1, Done.
2, Done.
3, Twitter is blocked at China, so replace that with Blog.
4, Done
5, At present, we don't need to find all of them. The problem will gone after replacing old site with new site later.

hi @vinothchandar, all review comments are addressed and fixed. I checked all hyperlinks just now.

right, the old site content is lastest.


Panic: interface is spec.Schema, not spec.Parameter for formField file

First up, thanks for the great tool :)
I'm trying to figure out how to use annotations for generating my client SDK for the case when the API call expects a form file. I've tried a few things, and this particular case gives me a panic:
panic: interface conversion: interface is spec.Schema, not spec.Parameter

Note: I'm not sure if the panic is just expected behaviour, but it goes unchecked in the code so I expect not. I guess I'd like it to tell me what I'm doing wrong.
Here's the full main.go to replicate this:
//go:generate swagger generate spec -o ./swagger.json

// Package main MyCompany API.
//
//     Schemes: https
//     Host: localhost
//     BasePath: /
//     Version: 0.0.1
//     License: MIT http://opensource.org/licenses/MIT
//
//     Consumes:
//     - application/json
//
//     Produces:
//     - application/json
//
//
// swagger:meta
package main

import (
	"bytes"
	"net/http"
)

// swagger:parameters myOperation
type MyStruct struct {
	// MyFormFile desc.
	//
	// name: file
	// in: formData
	// type: file
	MyFormFile *bytes.Buffer `json:"myFormFile"`
}

func main() {
	// swagger:route POST /api/my-op/ myOperation
	//
	//     Consumes:
	//     - multipart/form-data
	//
	//     Produces:
	//     - application/json
	//
	//     Security:
	//       api_key:
	http.HandleFunc("/api/my-op/", func(w http.ResponseWriter, r *http.Request) {

	})
}

And obviously just run this
go generate
swagger generate client

Instead of the *bytes.Buffer I've tried *os.File and I get this:
unable to resolve embedded struct for: &{280192 file}

and httpkit.File and I get this:
unknown primitive "File"

Edit: I've also tried multipart.File and get
unknown primitive "File"

If it helps (and if you have any thoughts on if this is possible with go-swagger!), essentially I'm trying to use annotations to replicate my call that allows both a JSON-encoded body and also a file form fields, equivalent of this CURL request:
curl http://localhost/api/my-op/ \
  -F<EMAIL_ADDRESS>\
  -F body='{"someKey":"someVal"}'

forthe file you can use: httpkit.File
https://github.com/go-swagger/go-swagger/blob/master/httpkit/interfaces.go#L24-L28

@casualjim thanks for the reply. I did already try httpkit.File previously and just tried it again and it returns the error:
unknown primitive "File"

Would this be a bug?
I did check out the test at https://github.com/go-swagger/go-swagger/blob/328dc1bb22db726edd68fcd101220ca7813539cb/examples/generated/restapi/operations/pet/upload_file_parameters.go#L27 but if I'm correct it looks like that file is generated from the swagger.json, opposed to what I'm trying to do which is generate it from code annotations/comments.

For the record, here's the complete running program to replicate the error when using httpkit.File:
//go:generate swagger generate spec -o ./swagger.json

// Package main MyCompany API.
//
//     Schemes: https
//     Host: localhost
//     BasePath: /
//     Version: 0.0.1
//     License: MIT http://opensource.org/licenses/MIT
//
//     Consumes:
//     - application/json
//
//     Produces:
//     - application/json
//
//
// swagger:meta
package main

import (
	"net/http"

	"github.com/go-swagger/go-swagger/httpkit"
)

// swagger:parameters myOperation
type MyStruct struct {
	// MyFormFile desc.
	//
	// name: file
	// in: formData
	// type: file
	MyFormFile httpkit.File `json:"myFormFile"`
}

func main() {
	// swagger:route POST /api/my-op/ myOperation
	//
	//     Consumes:
	//     - multipart/form-data
	//
	//     Produces:
	//     - application/json
	//
	//     Security:
	//       api_key:
	http.HandleFunc("/api/my-op/", func(w http.ResponseWriter, r *http.Request) {

	})
}

Hi @casualjim thanks for the fix. However, I have run go get -u and added the annotation swagger:file and when generating I get the error classifier: unknown swagger annotation "file". Here's the full file (I copied the MyFileParams struct directly from your recent commit so figured it should work). Any thoughts?
//go:generate swagger generate spec -o ./swagger.json

// Package main MyCompany API.

Another thing I can offer to try is to rebuild with the '-a' flag so that it rebuilds all packages in use.

@casualjim ah thanks, my bad, rookie mistake. I forgot that this was using a swagger command opposed to just importing it in my own code so I needed to make sure the swagger command was rebuilt. Sorry for the noise and thanks again for the fix.


Issue #1403 - On iOS, bypass 'always' permission by default

To prevent users getting rejected by Apple for not including NSLocationAlwaysAndWhenInUseUsageDescription even if they never require background location updates.
This PR makes the compilation of the code to request 'always' permission conditional on the flag PERMISSION_LOCATION_ALWAYS being set in XCode under 'Preprocessor Macros'. If not set (the default) then the permission request for 'always' is not included in the binary, therefore not triggering a complaint by Apple.
The request for 'always' (background) location updates is rare, so it is reasonable to have this turned off by default, and require some effort by the developer to activate this.
Changes:

PermissionHandler.m (add conditional compile)
README.md (modified iOS permission section to explain default behavior and how to set the PERMISSION_LOCATION_ALWAYS flag if needed)

I did not update the CHANGELOG as I imagine you may want to combine this with something else.
Pre-launch Checklist

[X] I made sure the project builds.
[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[X] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy], or this PR is does not need version changes.
[ ] I updated CHANGELOG.md to add a description of the change.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I rebased onto main.
[ ] I added new tests to check the change I am making, or this PR does not need tests.
[ ] I made sure all existing and new tests are passing.
[ ] I ran dart format . and committed any changes.
[ ] I ran flutter analyze and fixed any errors.

Hi @781flyingdutchman,
Apologies for the belated reply, and thank you very much for taking the time to submit this PR.
I really like the idea of adding this option, however I was wondering if it is possible to flip the behaviour. The current implementation requires developers to explicitly turn the PERMISSION_LOCATION_ALWAYS on, which is a breaking change.
Would it be possible to reverse this behaviour and have developers that do not require background permissions explicitly turn it of? This would keep the default behaviour intact and prevents us from introducing a breaking change.
Also could you update the pubspec.yaml with a version update (a minor increase would be sufficient) and add a matching entry to the CHANGELOG.md file?

This is an awesome change! I agree that breaking changes should be avoided, and I don't really care if I need to do some additional steps to avoid the "always" permission on iOS. I am a bit surprised that it currently isn't possible, and I'm even more surprised that this PR is discussed 1 day after I realised that I need this change to happen :)

@mvanbeusekom happy to make that change, stay tuned

@mvanbeusekom I have made the following changes:

Merged the latest main branch
Reversed the effect of the flag, and renamed it to BYPASS_PERMISSION_LOCATION_ALWAYS, such that if it is not set the behavior is the same as it was before (no breaking change)
Updated pubspec.yaml for the plugin and for the _apple module (incremented minor)
Updated README to reflect the new flag behavior
Updated CHANGELOG

I have to say I am a bit confused by the various versions of the example app in different directories, so had a hard time testing this time around. However, the latest change is minor (just added ! before the new flag name) I am sure it should work, but please test before releasing!

I thought I would try this PR out, so I pointed my pubspec.yaml to the fork that @781flyingdutchman have created like this:
dependencies:
  geolocator:
    git:
      url: git@github.com:781flyingdutchman/flutter-geolocator.git
      ref: always_permission_bypass
      path: geolocator

After this I did a flutter clean and flutter build ios, and then followed the instructions on how to add the preprocessor flag:

However I still get the warning-email from Apple. Have I done something wrong in the changes above?
I also realised a problem with this solution, as soon as I run pod install, the changes made to "Preprocessor Macros" disappears. Another problem is that we don't version control our dependencies, so the changes I make to Pods.xcodeproj can't be comitted. Maybe this can be solved by some script that always makes the change before compiling.

@mp-jojo Have you tried deleting the podspec.lock file and then applying the preprocessor macro? This forces a rebuild and may solve the problem. If so, I will add that to the readme.
You can check if the flag is active by going into the code on XCode (with your app as the target). Under the  Pods target, Development Pods, follow '..' a few times to get to your app then .symlinks, plugins etc to iOS, Classes, Handlers, Permission handler.m.  The code that is surrounded by flags should show dimmed if the flag is set such that it is skipped. Again, I had a hard time getting the example app to use the plugin code last time I tried, and I thought this was due to the confusing nesting of example apps but perhaps there is a bigger issue? It worked fine for me before I flipped the flag meaning.
I am not aware of a way to script this. In Swift you can change the Pod file but that doesn't work for C flags. If you know of a way, please let me know!

@mp-jojo Have you tried deleting the podspec.lock file and the POS directory, and then applying the preprocessor macro? This forces a rebuild and may solve the problem. If so, I will add that to the readme. You can check if the flag is active by going into the code on XCode (with your app as the target). Under the Pods target, Development Pods, follow '..' a few times to get to your app then .symlinks, plugins etc to iOS, Classes, Handlers, Permission handler.m. The code that is surrounded by flags should show dimmed if the flag is set such that it is skipped. Again, I had a hard time getting the example app to use the plugin code last time I tried, and I thought this was due to the confusing nesting of example apps but perhaps there is a bigger issue? It worked fine for me before I flipped the flag meaning.
I am not aware of a way to script this. In Swift you can change the Pod file but that doesn't work for C flags. If you know of a way, please let me know!

I realised that the problem might have been that geolocator_apple is a "sub dependency" of geolocator, so it probably fetched all sub-dependencies from the official repo. I added this section to my pubspec.yaml:
dependency_overrides:
  geolocator_apple:
    git:
      url: git@github.com:781flyingdutchman/flutter-geolocator.git
      ref: always_permission_bypass
      path: geolocator_apple

I am however not sure if that solved it, or the fact that I deleted ~/Library/Caches/CocoaPods, Pods and Podfile.lock. Regardless, I have now successfully uploaded a build of my app to testflight that doesn't warn about NSLocationAlwaysAndWhenInUseUsageDescription, so it works as intended!
I also added the below script to the bottom of my Podfile, so it will apply this flag to Preprocessor Macros on each build, also seems to work good!
post_install do |installer|
  installer.pods_project.targets.each do |target|
    if target.name == "geolocator_apple"
      target.build_configurations.each do |config|
        config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] ||= ['$(inherited)', 'BYPASS_PERMISSION_LOCATION_ALWAYS=1']
      end
    end
  end
end

@mp-jojo, thank you for testing and glad it finally worked out.
@781flyingdutchman, in the permission_handler plugin we also add preprocessor definitions to projects using the script mentioned by @mp-jojo (see iOS section of the readme). I think it might be nice to mention this as an alternative in the README of the geolocator package (next to the current solution you already documented). Would this be something you could add to this PR?

Absolutely, that's how I've done it in background_downloader - I just didn't know how to do that for Objective C.
I'll add these changes

Updated the README, CHANGELOG and Podfile as suggested


fix: fix export when an asset has a slash

for example, as/set.png will be exported as :

labels/

as /

set.png

wait for e2e tests to pass before merging: https://github.com/kili-technology/kili-python-sdk/actions/runs/7062228310


Added default cookies options support

This allow to send default cookies values to every cookie.
Values set for a cookie by user are respected.

:+1:


Azure Event Grid Readme Issue

Section link

Reason:

@joheredi, @ellismg, @dw511214992 for notification.

@sarangan12 Regarding this issue, what is the current progress?

Code Complete. Closing the issue


Peer Review Summary - Milestone 2

Hey Team,
following is the summary of my reviews-

UI looks super nice.
CI/ CD works fine.
Github web hooks work perfectly.
Wiki instructions were clear and things worked as described.

Issues faced-

Theres no validation for dates. Old dates are getting accepted and empty date gives an error (Page not found). have opened https://github.com/airavata-courses/Envy/issues/78
Due to a lack of session management, going back and forth the website causes it to throw errors. https://github.com/airavata-courses/Envy/issues/79
Jenkins has a single pipeline causing whole app to re-deploy even for small fixes. Better you have separate pipelines for each service. https://github.com/airavata-courses/Envy/issues/77
Not Sure your flights API is working. https://github.com/airavata-courses/Envy/issues/80

Would love to see-

Implementation of Google geolocation and Uber API as mentioned in Architecture diagram.
Do more with it, like links to booking flights and uber rides

Hi Swaroop,
Thanks a lot for reviewing after so many issues.
We have used google Geolocation and Uber API.
Yes, we will definitely consider your suggestions.
Thanks,
Alakh


Combining 'allOf' with 'If' / 'then' checks Do not resolve correctly in TS

The following schema:
const configTest = {
  type: 'object',
  properties: {
    animal: { enum: ['cat', 'dog'] },
    dynamicProperty: {},
  },
  additionalProperties: false,

  allOf: [
    {
      if: {
        properties: {
          animal: { const: 'cat' },
        },
      },
      then: {
        properties: {
          dynamicProperty: {
            type: 'object',
            properties: {
              key1: { type: 'string' },
            },
            required: ['key1'],
            additionalProperties: false,
          },
        },
      },
    },
    {
      if: {
        properties: {
          animal: { const: 'dog' },
        },
      },
      then: {
        properties: {
          dynamicProperty: {
            type: 'null',
          },
        },
      },
    },
  ],
  required: ['animal', 'dynamicProperty'],
} as const

Generates this type:
type CONFIGTEST = FromSchema<
  typeof configTest,
  { parseIfThenElseKeywords: true }
>
// type CONFIGTEST = {
//   animal: "dog";
//   dynamicProperty: null;
// } | {
//   animal: "cat";
//   dynamicProperty: {
//       key1: string;
//   };
// } | {
//   animal: "cat" | "dog";
//   dynamicProperty: unknown;
// }
}

I would expect this to generate a type that looks something like this:
type CONFIGTEST =
   | {animal: "cat", dynamicProperty: {key1: string}}
   | {animal: "dog", dynamicProperty: null}

Why does the logic include the last optional type? Is this a bug?

Hi @jfarre90 and thanks for the reach out !
This is a tricky one: properties / additionalProperties / required from parent & child schemas behave separately and are not to be merged. See https://json-schema.org/understanding-json-schema/reference/object#extending
What FromSchema does is actually to intersect the two allOf cases but with those keywords reset to their default values (see this line of code), and then to the rest of the parent schema.
So it compute the intersection of those schemas:

1st child schema:

{
  type: "object",
  properties: {},
  required: [],
  additionalProperties: true,
  if: ... (cat),
  then ... (cat)
}

2nd child schema:

{
  type: "object",
  properties: {},
  required: [],
  additionalProperties: true,
  if: ... (dog),
  then ... (dog)
}

Parent schema:

{
  type: "object",
  properties: {
    animal,
    ...
  },
  required: ["animal",...]
}

The pb is that the first two schemas, taken independently, are not mutually exclusive because the animal property can have any value.
If you add it to the child schemas, it will actually work!
const configTest = {
  type: "object",
  properties: {
    animal: { $ref: "#/$defs/animal" },
    dynamicProperty: {},
  },
  additionalProperties: false,
  allOf: [
    {
      properties: {
        animal: { $ref: "#/$defs/animal" },
      },
      if: {
        properties: {
          animal: { const: "cat" },
        },
      },
      then: {
        properties: {
          dynamicProperty: {
            type: "object",
            properties: {
              key1: { type: "string" },
            },
            required: ["key1"],
            additionalProperties: false,
          },
        },
      },
    },
    {
      properties: {
        animal: { $ref: "#/$defs/animal" },
      },
      if: {
        properties: {
          animal: { const: "dog" },
        },
      },
      then: {
        properties: {
          dynamicProperty: {
            type: "null",
          },
        },
      },
    },
  ],
  required: ["animal", "dynamicProperty"],
  $defs: {
    animal: { enum: ["cat", "dog"] },
  },
} as const;

type CONFIGTEST = FromSchema<
  typeof configTest,
  { parseIfThenElseKeywords: true }
>;

What I'm sure of is that properties cannot naively be merged from child to parent schemas. Still, I'm not sure resetting all properties is the best approach 🤷♂ WDYT ?

Hi @ThomasAribart! Thanks for the reply and info shared, much appreciated. I agree for your points and it makes sense.
In our use case we ended up defining 2 separate schemas, one for the "dog" and another for the "cat" in this example. And within our logic we had a switch/case that used one schema or the other for some validation. As for the type generated, we just applied the condition from our end like...
type FinalType =
  | FromSchema<typeof dogSchema>
  | FromSchema<typeof catSchema>

Which for our use case ended up working nicely, and we didn't have to use the parseIfThenElseKeywords which was causing some TS compilation issues in some of our larger schemas (as you mention in your docs, it sometimes got aborted due to complexity).
However, we are still refactoring a big part of our application to TS, so the info above is helpful in case we face a similar scenario down the line.
Thanks again for the great work! Finding this library very useful 🎉


Layout Thrashing has negative impact on Microsoft Edge performance

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Comment:
There are several issues about Windows UWP and Edge performance (#12183, #11783, #12611, #10009, #8710), most of them are closed (for different reasons) or marked as duplicates. I think it's not a good idea to lock issues like #11783 because help from community should always be welcome.
Current behavior:
Using Profiling Tools from Edge and Chrome leads to the major issue about ionic performance. The are just too much (unnecessary) "layout trashing" for many click events in ionic.
Browsing the ionic Conference App in Google Chrome with active performance tracking there are some warnings like "Forced reflow is a likely performance bottleneck". (see attachment)
Using Microsoft Edge Developer Tools, the performance diagnostics (see attachments) shows Layout events even before the button click is fired. (probably triggered by some mouseover, mousedown animations)
These Layout or Reflow events are painful because even simple ionic items adds a lot of DOM elements. Additionally Egde does not support Shadow DOM yet.
I think with some code optimization most of these full page refreshes can be eliminated.

Steps to reproduce:
Visit http://ionic-team.github.io/ionic-conference-app/www/#/tabs/schedule/schedule on Edge, start Developer Tools with F12, go to Performance and start profiling.
Same can be done with Google Chrome to see some hints for developers according to issue:

Some additional links:
What forces layout/reflow: https://gist.github.com/paulirish/5d52fb081b3570c81e3a
CSS Triggers: https://csstriggers.com/
Workaround<EMAIL_ADDRESS>

Hello! Thank you for opening an issue with us!
Given the large number of structural changes being made for Ionic v4 with its use of Stencil Web Components, this issue will need to be completely re-evaluated after the release. I will mark this as something to be looked at after the v4 release of Ionic is complete.
Thank you for using Ionic

Thanks for your answer. When can we expect Ionic v4?

I wanted to add that all windows UWP apps/Cordova are using the Edge Engine and are having the same issues. It can take 3 seconds for a button to fire a simple observable event.
I read This article about Independent Rendering in Edge 16  as well as this on  input events
Which have me hopeful for version 16 but if Ionic (and Cordova) are going to claim Windows compatibility I agree that these threads shouldn't be arbitrarily closed.
I'll keep an eye out for V4 but I'm going to dive in as much as I can..


Feature request : Collapse all button ?

Could it be possible to add a "collapse all / deploy all" button in the Dendron panel view ?
I'm having a LOT of notes, and it's sometimes tedious to collapse all, by hand, in order to be able to navigate better :')

Hi! I forked this plugin since it hasn't been updated in over a year and it took 4 months for my pull request to be accepted here. Unfortunatly, the update is still not out since the author has not create a new release since the end of September 2023.
I have implemented "Collapse All" in the command palette for my project, so if you want to try it, just search for "Structured Tree" in the Obsidian Community Browser.
Here is my repo: https://github.com/Rudtrack/structured-tree


[FR] Custom URI commands

Using advanced URI makes tons of sense, but it feels like we're missing a lot of its power! It would be great to be able to configure custom URI commands.
For example, if I use Periodic Notes for my daily notes instead of the core daily notes plugin, it would be nice to be able to call the proper command from raycast.
I don't think that adding support for every single command/plugin is particularly sustainable, but allowing custom commands might be. 😄

Bump!

Can you comment a bit on how you would want this to work? Like a way to add a custom command + url combo to fire the url when the entry item is pressed?

Can you comment a bit on how you would want this to work? Like a way to add a custom command + url combo to fire the url when the entry item is pressed?

Exactly, yeah. Either a way to define an arbitrary URI as an option in raycast or or provide the ID of a command which the raycast plugin could format into the URI.
This all assumes I have a correct understanding of the advanced URI plugin. My understanding is that it allows you to call pretty much any command via URI.

That sounds reasonable. Raycast won't be able to allow a dynamic user entered command, but I could create a generic command just for running custom commands. I'll have to investigate it some more though. I'll follow up.

As far as I know, it grabs those from the package.json file, which runs through a validator before deploying to the store. So they need to be there beforehand. Could be some way though, I suppose.

If I remember correctly I was once talking with someone from Raycast about this and it was discouraged to have users create own commands.
That is one reason why I for example used the Actions menu in the "Create Note" command to let users generate their own "Folder Actions".
So, imo like @KevinBatdorf suggested, a generic "URI command" would be the best solution.

Yeah, for sure. I'd be over the moon with that. However, I think you might be able to use either the Storage API or a file in the support directory with custom URIs, then show them in a List or as Actions.
But regardless a generic URI command is needed for that anyway, so this is a perfect place to start.


Update several gems with the latests security patches

Changes

Update all gems to their latest security patches.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.263%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1358

Relevant Lines:
1382

💛  - Coveralls


How to track details of an associated model?

class Author < ActiveRecord::Base
  has_many :books
  accepts_nested_attributes_for :books

  validates :name, presence: true
end

class Book < ActiveRecord::Base
  belongs_to :author

  validates :title, presence: true
end

> author = Author.new
> author.books.build
> author.valid?
> author.errors.details
=> {:name=>[{:error=>:blank}]}
> author.errors.messages
=> {:"books.title"=>["can't be blank"], :name=>["can't be blank"]}

I want to take errors of a book model from author.errors.details like author.errors.messages.
Is there a way?

@xendoc I'll take a look at rails internals, as this probably also won't work in upcoming 5.0. Thanks

@morgoth Thanks for your response.
I hope to be improved!

I submitted PR to rails https://github.com/rails/rails/pull/22105 - when it will be accepted, I'll try to monkey patch rails prior to 5.0 to support this.

Thank you! :smile:

Thank you! :smile:


noteTweenDirection()

What needs to be documented?
noteTweenDirection()
yeah apparently this exists

Ye it has for a while

bruh


Hoon source files with margin comments require a trailing newline

I've verified with generators and libraries; presumably this applies to anything built by %ford ...
:-  %say
|=  *
:-  %noun
1                                                       ::  produce result

With no trailing newline:
~zod:dojo> +test
syntax error: 4 75
ford: fade /~zod/home/231/gen/test
ford: core /~zod/home/~2016.12.1..04.48.13..65a0/gen/test

With a trailing newline (or deleting the comment from line 4):
~zod:dojo> +test
1

Can no longer reproduce.

I vaguely remember fixing this in the parser ye
On Mon, Feb 5, 2018 at 11:12 AM, Fang<EMAIL_ADDRESS>wrote:

Can no longer reproduce.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/307#issuecomment-363189781, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABxXhuwKLcVxRm4JDtyDSaG78a207tzvks5tR1KPgaJpZM4LA__6
.

This is actually reproducible again. It also occurs when the final line is a comment regardless of if there is also some code before the comment or not.


Using nested transactions with save points

Environment
Knex version: 0.19.1
Database + version: PostgreSQL 11
OS: MacOS Mojave
Bug
I am trying to use nested transactions with save points and it doesn't seems to be working with promises API.
Here's some code (I am sharing the minimum required code to reproduce the problem)
Following doesn't work
async function run () {
  const trx = await knex.transaction()
  const trx2 = await trx.transaction()
  console.log('never reached here')
}

Following works
function runWithCallbacks () {
  knex.transaction((trx) => {
   trx.transaction((trx2) => {
    console.log('reached here>')
   })
 })
}

The first example doesn't work since the promise doesn't resolve unless trx2 commit/rollback and it cannot, since the code never moves to the next line.
Is this the intended behavior?

I have been trying to look into the source code and there is hardly anything I am able to find that helps finding the root cause. All I have is the debug output.
  knex:tx trx1: Starting top level transaction +0ms
  knex:client acquired connection from pool: __knexUid2 +0ms
  knex:query BEGIN; trx1 +0ms
  knex:bindings undefined trx1 +0ms
  knex:tx trx3: Starting nested transaction +25ms
  knex:query SAVEPOINT trx3; trx3 +3ms
  knex:bindings undefined trx3 +3ms

The debug output is same with await calls and with callbacks.

Any chance someone can look into it?

@tgriesser, @elhigu, @kibertoad. Can someone please share some insights related to this issue? Right now I am not even 100% sure, if this is something expected to work or not.

@thetutlage Theoretically it should work. I'll take a look; sorry for the delay.

@thetutlage Here is what was happening. trx.transaction is a synchronous operation that expects you to pass transaction handler as a parameter; it does not support the Promise API the way Knex does. Hence correct way to write the above would be
  const trx = await knex.transaction()
   trx.transaction((trx2) => {
    console.log('reached here>')
   })

That said, it is confusing and inconsistent, hence see the PR raised to improve this behaviour (review and comments would be most welcome).

@kibertoad Thanks a lot for checking it out :). The PR does looks fine to me, but I may not the best person to judge that.
From the consistent API point of view, I also expected it work.

Found one regression here. Now, since we are faking the container, the rejection behavior of rollback is to always reject.

Infact, transactions are quite broken between different dialects. The query handler from the base transaction class handles the use cases little bit differently.
Base Transaction
Base transaction class checks the sql against a regex, which also handles the savepoint use case.
However, the following dialects are checking against a literal value or there is no sql check at all.

mssql

mysql

mysql2

oracle

oracledb

postgres and redshift uses base transaction.

@thetutlage Would you consider contributing a fix?

Yeah sure, why not. You want me to also take the changes from your recent PR, or should we keep this separate

I'll update the PR with your proposed enhancement, so remaining part would be savepoint handling, which could be separate PR (and I assume we need more tests for it, if that remained unnoticed).

Sure, I will do a PR to make all dialects behave the same way for transactions and will see, if we can abstract the code to a helper function for consistency

Hello @kibertoad here's the PR. https://github.com/tgriesser/knex/pull/3399

@thetutlage Released in 0.19.2


can't recreate the scene

HI great job on this it's exactly what i was looking for,
one problem though, the scene you're using is missing from the files, is it possible to share the whole project ?
or at least what objects should i create for those ?
private GameObject RemoteVideoRoot;
private GameObject RemoteScreenVideoRoot;
thanks a lot !

@digitallysavvy You need to upload the .meta file。

@AfifSoussi I've written a guide on how to implement this plugin as part of a VR scene.
I’ll make note to add a demo scene in the next update.

A demo scene would be great. I have not been able to get this working even after following the guide.

Please see this repo for demos of using the prefab.  Note the demos are simplified enough to show the usage, and it is not the same demo that was mentioned in blog above.


Cache chainId instead of fetching it every time

Currently,
var chain *big.Int
chain, e := ew.evmClient.ChainID(context.Background())
if e != nil {
  ew.logger.Errorf("[%s] - Failed to retrieve chain ID.", eventLog.Raw.TxHash)
  return
}

is called on every EVM related transfer, when this is data that can be cached upon validator start. It is not expected to change. Suggestion would be to cache it inevmClient.

Fixed in https://github.com/LimeChain/hashport-validator/pull/412


Added Onionshare to list

Onionshare allows one to share files, chat and even host a website anonymously over TOR.
Project URL
https://github.com/onionshare/onionshare
Category
Sharing Files or the Productivity category would be the best places to share them in.
Description
Adding OnionShare as I believe with it being all done over TOR it adds to privacy for its users.
Why it should be included to Awesome macOS open source applications  (optional)
Checklist

[x ] Edit applications.json instead of README.md.
[x ] Only one project/change is in this pull request
[x ] Screenshots(s) added if any
[x ] Has a commit from less than 2 years ago
[ x] Has a clear README in English

This is my first pull request, please let me know what I did wrong if anything, thank you.


BROOKLYN-356: fix race in transformer by adding getImmediately

Thanks @geomacy for the great comments - I've addressed those in an additional commit, and rebased against master.
Once jenkins confirms the tests pass, I'll merge. If you do get a chance to review the changes in the last commit before that, it would be great though.

@aledsage I'm currently reviewing the PR, please hold on with the merge.

There's pretty big overlap between the getImmediate & newTask implementations - would be nice to reuse as much as possible, but I'm fine merging those as is.

Couple more comments but otherwise looks good along with Svet's comments.  👍

@neykov @geomacy latest comments addresses in the last two commits. Can you take another look please?

LGTM, merging.

One minor comment in AbstractBrooklynRestResource.java but overall LGTM 👍 . I had a look at the code @neykov referred to and I suppose there is more to do as noted in the comment in getNonBlockingResolvingStructuredKey but I agree that that should not be part of this change.


Add FUSD to Kitty Items

Is your feature request related to a problem? Please describe.
Developers want to start using FUSD in their applications, but lack a concrete example demonstrating how to do so.
Describe the solution you'd like
Ideally Kitty Items can be updated with the following:

Merge in generic storefront contract, which can support multiple fungible tokens out of the box (unlike KittyItemsMarket, which is hard-coded to Kibble) (issue: #9)
We would keep the Kibble contract and minting functionality in the API
When building the frontend, developers can use a flag to build for FUSD or Kibble
The built UI will change based on the token option selected
FUSD is highlighted as the default, but developers can opt to run the Kibble version if they are interested in creating their own fungible token.

You should use this version of the contract in Kitty Items: https://github.com/onflow/wallet-api/blob/main/cadence/contracts/FUSD.cdc
Here is an example of how to add FUSD to flow.json

Closed by #127


Add tab navigation + People Fragment

Description
This PR adds a tab navigation in the homescreen. The homescreen now includes the upcoming events tab, the people tab, and the settings tab.
Screenshots

events tab
people tab
settings tab

A great example of how not to do a PR folks


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2642

Questions ?
Please refer to the Backport tool documentation


142 | Linked List Cycle | C++

Linked List Cycle -
What will change -
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[ ] Improving Documentation
[ ] Improving Code
[ ] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[ ] Python
[x] C++
[ ] Java
[ ] SQL

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Hi @Yashica7 ,Can you assign this issue to me, I am willing to contribute

@SSKale1 , I just a small doubt my PR was not tagged with the "hacktoberfest" tag, is it needed that you do it explicitly? or since the repository is tagged with "hacktoberfest" label, there is no need of it?


Pasword reset action looks for mail views in 'frontend/mail/' but they are actyally in 'common/mail/'

What steps will reproduce the problem?

Advanced template clean installation.
Configure swiftmail component to use SMTP account
Try to reset password for an existing user

What is the expected result?
Send an email with password reset instructions.
What do you get instead?
View not Found – yii\base\ViewNotFoundException
The view file does not exist:frontend/mail/passwordResetToken-html.php
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.0.15

Operating system
CentOS 7.3

Solved by adding 'viewPath' => '@common/mail', to mailer config


Version 0.40.0 not working with vite: SyntaxError: import not found: default

After upgrading to version 0.40.0 I'm getting the following error: SyntaxError: import not found: default in createCalendar.js:2:7
import rangePlugin from "/node_modules/flatpickr/dist/plugins/rangePlugin.js?v=1300bfa4";

I'm using vite. This is how I can replicate the error:

create a new project with vite https://vitejs.dev/guide/#scaffolding-your-first-vite-project
install carbon components npm i -D carbon-components-svelte
change App.svelte to:

<script lang="ts">
	import { Button } from "carbon-components-svelte"
</script>

<Button>hello</Button>

Then I run npm run dev. I open Firefox at address http://localhost:3000/ and I get the error in the developer console.
The thing is I don't event use the DatePicker component in my app.

Add this to the list of dependencies for vite to optimize: "flatpickr/dist/plugins/rangePlugin"

That did it, thanks.

Just curious, what vite version are you running?
I haven't encountered this error when using it with SvelteKit, Rollup, or the official Svite set-up (@sveltejs/vite-plugin-svelte).
I wonder if this only affects vite version 1 because @sveltejs/vite-plugin-svelte uses vite version 2.

In my project I'm using the the latest version of vite: 2.4.2.
I replicated the error using the official guide for installing vite and svelte (see my initial message), so the latest versions I guess.
I also checked another project using the latest svelte kit and the error didn't show up.

Fixed in v0.40.1


Expose driver enumerations in messages

I was doing some testing commanding ODrive's via ROS 2 with this package, and noticed that it was hard to figure out what to put in some of the uint fields in the control messages or service.
So I pulled the enum values from the docs and put them directly in the message files. This way everybody who compiles the messages will have the enum values come along with the messages themselves (but note that msg constants are not serialized, so this adds no overhead when passing messages).
Obviously, this changes the structure of the original messages, which will break the code of anybody currently using this package. I thought of a few other ways to make these changes without changing the messages, but did not like them as much:

Add the enums as constants in every message that needs them. But it is a lot of repetition and doesn't make it clear which constants belong with which types
Add a header file declaring all these. Again, doesn't make it clear which enums go with which message fields, and to gain access in user code they would have to link/include this package

The changes only really effect the format of the input to the node, but I did test them on hardware anyway and everything looks to be working.
Note: this change would also need to be reflected in the tutorial page. Specifically, the service call near the bottom changes to:
ros2 service call /odrive_axis0/request_axis_state /odrive_can/srv/RequestAxisState '{axis_requested_state: {state: 1}}'

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


trade page UX suggestion

https://bitsharestalk.org/index.php?topic=15454.msg199012#msg199012

Not sure what his issue is, it feels quite natural to me:
You enter a sum to buy and a price -> You get the Total calculated
You enter the total to pay and a price -> You get the sum calculated


UTF-8/16 string validation

String in JSON are not validated to be UTF-8 or UTF-16. The parser do not
detect whether the string contain invalid encoding.

Both YAJL and ultrajspon supports UTF-8 validation. YAJL can turn it on/off,
while ultrajson make it mandatory.

Add an parsing option to validate UTF-8/UTF-16.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 22 Nov 2011 at 2:40

The change for this is done now.

Original comment by<EMAIL_ADDRESS>on 24 Nov 2011 at 5:13

Changed state: Fixed


changed image's content mode to scaleAspectFill

It fits the button images with different sizes much better this way

@tonymuu  HI, sorry for not checking for a long time.
The contentMode .center is to enable to make extra touchable area.
This commit has breaking change, and I wouldn't like to merge it if it is remained.
I'll prepare an option to change the contentMode, but the day is unknown because I'm busy now.
Please fix to do so if you want the function soon😅.


Re-publish on Android

Google login
<EMAIL_ADDRESS>Kangkoon$$12

[ ] Change UI of maximum of 5 screens.
[ ] Change app name -> Athletian
[ ] Change app package name -> com.grc.ath
[ ] Publish it.

push athletian branch in grc repo.
still ongoing on user search list.

Upon checking, UI in post is not really the same with Instagram.


Agent Token File Content

Hello,
Thank you for your amazing work! I am studying the source code, and have a question on the detailed content of the agent token file, cluster_frame_5_2048.pkl. I found that it is consists of 3 parts:

token: the tensor shape for each agent type is (2048, 4, 2). the meaning should be (# tokens, 4 corners, x and y position), is that correct?
traj: the tensor shape for each agent type is (2048, 6, 3)
token_all: the tensor shape for each agent type is (2048, 6, 4, 2)

For traj and token_all, I am kind of confused about what each dimension represents. Could you please kindly explain it? Thanks a lot!
Best regards

'token': (tokens, 4 corners, x and y position),
'traj': (tokens, 6 trajs points, x y position and heading),
'token_all': (tokens, 6 trajs points, 4 corners, x and y position),


Add Django 3.2 to tests

Closes #582

Hi @victor-yunenko,
Could you please review it and approve CI trigger and merge in case if there is no problem with it?

Hi @marksweb ,
Could you please rerun CI? Seems like after merging latest changes to master branch it should be ok for now.

@DmytroLitvinov if you merge master into your branch it should see the python tests pass.

Absolutely. Thank you @DmytroLitvinov

@DmytroLitvinov Please rebase master into your branch. We just fixed the CI on the master now, so your tests from this branch should be able to run as well.

@DmytroLitvinov I've left a couple of comments on this, the change log title and dropping EOL python from tests.
Could you take a look please & rebase master as we've just upgraded the gulp pipeline.

Codecov Report

Merging #583 (8d6694f) into master (b955396) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 8d6694f differs from pull request most recent head 7c0fb67.

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

The test suite doesn't support django 3.2. So although I've merged this, when I added 3.2 to the github workflow it failed. This is likely related to a lack of 3.2 support in django-app-helper.
I've raised an issue already on there after I was looking at updates to django-filer. But it seems nephila aren't maintaining their packages anymore; https://github.com/nephila/django-app-helper/issues/204


Need function to transform bbox along with image

say we downsample the image using torchvision transform Resize. We should provide some convenience method to transform the bboxes in the annotations accordingly

Can this work?
https://github.com/eto-ai/rikai/blob/master/python/rikai/types/geometry.py#L211-L223
So basically, we can do
scaled_bbox = original_bbox / (1.5, 2.3)

yeah but you have to manually calculate the scaling factor

Oh, for that, our practice usually was that we just store bbox in the relative scale [0, 1].  So bbox value does not change no matter how Image resizes.

Here is a related issue: https://github.com/eto-ai/rikai/issues/363

Oh, for that, our practice usually was that we just store bbox in the relative scale [0, 1]. So bbox value does not change no matter how Image resizes.

that makes sense. when you need to calculate area you just convert it back? maybe something like relative_bbox.to_absolute(image).area?
should we make this the default?


Display comments popup with selected items details

Display comments popup

In this project of the milestone added:

Parse data in the popup modal on comment button when clicked

Thank you @AmaduKamara  for your great feedback.


Configure auth proxy for Opensearch

For unified auth we should have an auth proxy configured as a trusted auth provider for opensearch.

This can be configured with the current operator by passing in config.yml in the security config.  This needs to a secret with the file content.  The operator also requires that certs be generated to use custom config.
In the config the appropriate headers are configured, along with the IP address of the proxy/s.


Obtaining trajectory markers

Hello,
I'm trying to get poses (position + orientation) along a trajectory.  In #332, SirVer says "Btw, Cartographer now also publishes the trajectory in form of markers, so you can get to the information now."  But after much searching, I don't see how to do this.
My guess is that I need to include another node that subscribes to the published markers.  I have two questions: 1) Do markers have orientation in addition to positions?  and 2) How can I get these markers?
It would be great if there's an existing node that subscribes to markers that I could modify.
Thanks!
Alan
P.S. This is a general question not specific to any particular data or scenario.

I believe there is a service that allows you to query cartographer for a trajectory.

Thanks ojura.  In case anyone else is looking for this info, here are steps to obtain it:

The Cartographer node publishes topic /trajectory_node_list,
This topic is of type visualization_msgs/MarkerArray,
Its message contains an array of Markers,
Markers contain a member of type geometry_msgs/Pose,
the Pose type contains two fields: position and orientation,
Pose.position is three floats and Pose.orientation is a quaternion (four floats).
Add (or modify) a node to subscribe to the /trajectory_node_list, and write the Poses to a file.

BTW, I haven't implemented these steps in code yet.  I'll post an update when I have more information.

I just implemented it using the steps you mentioned above and it does not work for me. The output of the marker array pose type does not contain the actual pose. The output is the following:
 header:
      seq: 0
      stamp:
        secs:<PHONE_NUMBER>
        nsecs: 179813088
      frame_id: "map"
    ns: "Trajectory 0"
    id: 2
    type: 4
    action: 0
    pose:
      position:
        x: 0.0
        y: 0.0
        z: 0.05
      orientation:
        x: 0.0
        y: 0.0
        z: 0.0
        w: 1.0

The actual trajectory is present in the points field in the message as only x,y and z values without orientation
points:
      -
        x: 0.863043830642
        y: 6.99164155632
        z: 0.0
      -
        x: 0.860062564869
        y: 6.93523299369
        z: 0.0
      -
        x: 0.858788512093
        y: 6.88456487155
        z: 0.0
      -
        x: 0.867231202534
        y: 6.81827431059
        z: 0.0
      -
        x: 0.866676887132
        y: 6.75207955557
        z: 0.0
      -

@ayu135 if you post your code, I'll take a look at it.  (Won't be in tomorrow; U.S. holiday)

@ayu135 I'm getting the same result.  Looking into it.  I'll post if/when I find a solution.

As I have said, there is a service for querying Cartographer for full trajectory poses, not just positions. It is called TrajectoryQuery.

@ayu135: The reason the previous attempts failed is the frame of reference is ON the robot, so doesn't change as the robot moves.  The solution is to adjust the position and orientation by the transformation from the /map frame to the /base_link frame.
The complete solution with source, Package.xml, and CMakeLists.txt is in item #1350.


Improve the calculation of the status

It's not optimized at all, and it should cause it's accessed frequently.

Is this referring to app/services/calculate_status.rb?

Yes and also: app/model/status_crowdsource.rb


New version pointing to wrong batch instance?

I built two docker images 4 days apart:
https://github.com/populationgenomics/production-pipelines/actions/runs/6346122173/job/17239186671
hail==0.2.118
https://github.com/populationgenomics/production-pipelines/actions/runs/6389974779/job/17342299423
0.2.122
The latter of these vomits when trying to start a new batch, e.g. here
hailtop.httpx.ClientResponseError: 401, message='Unauthorized', url=URL('http://batch.hail/api/v1alpha/batches/create-fast') body='401: Unauthorized'

It looks like it's trying to submit jobs using the Broad's Hail API?
n.b. this issue may not be characterised correctly 🙃

fixed


[HotFix] backward compabilities for 6.0 test tool

https://ontrack-internal.amd.com/browse/SWDEV-450280

Fix for https://ontrack-internal.amd.com/browse/SWDEV-450280 .


Merge bugfix into develop

Finished fixing the bugfix, now it handles the categories in a better way

LGTM


Add support for tokio-console

This enables debugging and improvement of asynchronous Rust code.
It is an optional feature that is still
experimental (but can be quite helpful)
More info: https://github.com/tokio-rs/console

should this be a dev dependency?
we should always avoid over-bloating the binary to ensure a fast deployment in CI environment.

You're absolutely right. Thanks for catching that.

@lebensterben I'm getting "dev-dependencies are not allowed to be optional". Given that tokio-console is marked as optional anyway, wouldn't it be fine if it was a normal dependency then? If not, what would be the correct approach in your opinion?

@mre you're definitely right.
I failed to notice that it's already optional.


Kyoukai prints 404 to console when a 200 is sent in errorhandler

Pls fix
@app.errorhandler(404)
async def handle_404(ctx, exc):
    try:
        resp = open(ctx.request.path).read()
        resp = Response(200, resp)
    except Exception:
        resp = Response(404, "404")
    return resp

This is not an error. There is never a case where a server should decide to send a 404 and then resolve a 404 into a 200. That's not how the specification works.

So how do you suggest I add functionality like this whenever a page isn't
found in the routes defined by the code?

You should either return a '404 page' like most sites or just return the 404 with no content and let the web browser give its own default not found page.

The logging call is before the error handler is hit.
If you want to do weird stuff like this, use a custom router, with the other changes in 1.10.

What I'm saying is that I want kyoukai to look if a file exists, and only
404 if even the file isn't found. I might get a lot of pages and I'm not
gonna add routes to all of them (even regex would be too much)

That is not how a web server (or hell, even a file server) works. Thank you, please come again once you understand how these things work.

You can literally make a catch-all route with /(.*). It will catch ANY request not caught by other requests, as long as you define it LAST in the order.


Warlock Pacts as Invocations

Warlock Invocation Changes: #3

Pacts
Add Pacts to Invocations.
Optional: Deepened Pacts split into invocations.
Level 1
Add Invocation Selector (Using Level 2 as base)
Level 3
Remove Pact Selector.
Level 5
Change Invocation Selector to two selections instead of one.
Optional: Remove Automatic deepened pacts.

Did not include option to break out Deepened Pacts.
Reopening and will investigate implementation options.


[Request] Allow filtering 32 and 64 bits syscalls for x86-64.

At the moment, it's not possible to filter both. If a filtered program call a 32 bits program, it will result in a bad system call. In libseccomp, one can differentiate between the 2 by checking for __X32_SYSCALL_BIT mask on the system call number. It would be very useful for my use case, filtering calls from a sandbox environment that may use 32 bits applications.

I would also use such a feature. I'm also a bit confused about the current API - does the filter just reject/kill syscalls with a different AUDIT_* arch? This could definitely be better documented.