👌 IMPROVE: Rename "proof" domain and role to "prf"

This PR renames the domain and role names of this extension from "proof" to "prf". This follows closely the naming convention of other domains in Sphinx. It also allows users to utilize the sphinxcontrib-proof extension in the same project as sphinxcontrib-prettyproof with no fear of overriding domains.

@jstac-- just FYI -- we will need to update your projects once this get's merged.


Skill Selector + Tasks

According to the paper, skill selector is trained without a task objective. Is it possible to train with the task objective? For example, I want the character to have flexibility to learn both strike task and throw task and an ability to execute arbitrary sequence of these on demand.
Thanks

Yup, it should be possible to add additional task objectives when training the skill selector policy.


Cache inference for recursive mapped types

Previously recursive mapped type inferences avoided cycles when looking for inferences. Now, they instead cache as they explore the object structure so that symbol-identical inferences to recursive mapped types don't repeat. This is needed, for example, to allow lodash's use of PartialDeep<T> to be usable with XMLHttpRequest and similarly large object types.
Also change to use the existing visited cache instead of a separate stack.

Improves the fix in #20370. Turns out the test there was too small for real types like XMLHttpRequest.

If I recall, we also don't actually apply mapped types to primitives Partial<number> is justnumber, so inferring a mapped type to a primitive (even structurally) seems like it's always incorrect. Maybe deserves another issue.

Or rather, it's like that when inferring from MappedType<T> to a primitive, T should be inferred as the primitive, to mimic our actual mapped type application behavior.

It's easy enough to skip primitives. I'll poke at inferring them too, and include it if it's simple enough.

OK, I infer homomorphic mapped types to primitives. The change is simple but I'm not 100% sure it's correct. @ahejlsberg can you take a look?

I still see readonly SVG_MEETORSLICE_SLICE: {} in the output, but readonly SVG_MEETORSLICE_SLICE is a member of SVGPreserveAspectRatio of type number. What's up with that?


Fix evictionHard parsing

What this PR does / why we need it:
Eviction hard mapStringString flag is using symbol < as a separator between pairs.
Fix evictionHard parsing

/retest


82 create a cluster route and template

Many thanks @maveric1977. @MaximilianVomhof could you have a look at the preview (manually head to /clusters as there is no menu etc.)? I would say we add the /clusters route so we can start building out content around them. For a public launch we can integrate these in the marketplace or a different part of the page.


Code?

f46200bcb5be23df1b1bde25ce6f8bfb671212d1

Yes, code. This is working as intended.

We were briefly considering a9b863a90c0834619fe81553d9ca821674 but yes, f46200bcb5be23df1b1bde25ce6f8bfb671212d1 is where we landed


Changing dev guide steps order, docker with db should be installed before migration and createsuperuser

Motivation and context

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added a description of my changes into the CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

@bsekachev It seems that this PR duplicates #6099


”Ciceu-Giurgești”, ”Dumbrăveni” are listed in the drop-down menu

Description:
When user writes ”Ciceu-Giurgești”, ”Dumbrăveni” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Ciceu-Giurgești"

Expected results
The location appears in the drop-down menu as "Ciceu-Giurgești, Bistrița-Năsăud (Ciceu-Giurgești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Dumbrăveni"

Expected results
The location appears in the drop-down menu as "Dumbrăveni, Bistrița-Năsăud (Ciceu-Giurgești)".


Add dynatrace example app

I added an example app to show how to use with this monitoring module: https://github.com/commercetools/commercetools-sdk-java-v2/pull/622

We decided not to add dynatrace module for now.
Reason: it would add too much overhead as it uses HTTP to create new metrics. This would significantly slow down every request to commercetools. The other way for now for dynatrace is to use opentelemetry.


05/29/2018 21:39 : Create Pull Request for branch master -> prTesting1_2018052921

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

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


Antialiasing bad since Chrome > 61

bug
What is the current behavior?
In Chrome 61, lines in canvas are smooth:

But since Chrome 66, lines in canvas are pixalated:

What is the expected behavior?
Smooth lines :)
Which versions of SignaturePad, and which browser / device are affected by this issue? Did this work in previous versions of SignaturePad?
3.0.0 beta 3

@RomuxX Just to be sure - you used the same version of this library in both Chrome 61 and 66?

@RomuxX a little late, but I think i do have to solution to this problem.
See -> #505

closing as stale


eager argument to watch

def callback(event):
    select.disabled = True
    button.disabled = True
    time.sleep(5)
    select.disabled = False
    button.disabled = False

select = pn.widgets.Select(options=['Just Testing'])
button = pn.widgets.Button(name='Process!')
button.on_click(callback)

pn.Column(select, button)

philippjfr commented:
It's down to this: pyviz/param#323
I think we need to add an eager argument to the watch mechanism, which ensures that this works.

Moved to https://github.com/pyviz/param/issues/350


fix with invoke method without args

SwiftR/SwiftR.swift
fix with invoke method without args
code:
let js = args.isEmpty ? "swiftR.hubs.(name).invoke('(method)').done((doneJS)).fail((failJS))" : "swiftR.hubs.(name).invoke('(method)', (args)).done((doneJS)).fail((failJS))"

This a dupe of pull request #22, which has been merged. Looks like you were working together. Thanks for submitting though!

you are welcome! by the way,you're right! I working together with #22 in china .^ _^

am


to-do list completed

Developer Checklist

[ ] Followed guidelines mentioned in the readme file.
[ ] Followed directory structure. (e.g. ProjectName/{USERNAME}/...yourfiles)
[ ] Starred ⭐ the Repo (Optional)

Summary
Create and manage tasks effortlessly with our simple Todo List app, featuring an intuitive user interface inspired by Todoist. This web application is built using HTML, CSS, and JavaScript, allowing you to easily add, complete, and organize your to-do items. Stay organized and boost productivity with our user-friendly task management solution
Screenshot

Live Project Link
add a working project link here

kindly review and merge the pull request..


Paid partnership information

@rarcega Is it possible to extract the sponsor information for images ? Instagram recently allowed adding the "Paid partnership with ...." tag to images. Can maybe add it to the image metadata information.

Hi @rarcega! First of all thank you so much for writing and maintaining such an excellent package. I would like to jump in here as well and express my interest in the "paid partnership" information. This would be an important and very useful addition to the package. Here is an example for a post with a paid partnership:
https://www.instagram.com/p/Bx69fx1j-M2/


Allow access to any field in current value when importing

Input:
$ head -n 10 profiles/00-fluentbit-gc-profile.yaml
'@type': schema.xyz.io/xyz.io.deploy.profile.Object
systemMetadata:
  uid: 89fe647c-0287-41c9-87e4-d3e9af82f950
metadata:
  name: fluentbit-profile
  uid: 89fe647c-0287-41c9-87e4-d3e9af82f950
spec:
  profileSpec:
    fluentbit:
      fluentbit_namespace: monitoring

Import:
cue import ./profiles/... -p profile -l '"\(strings.Split(@type,".")[5])" "\(metadata.name)"' -f -R
parser error in path "\"\\(strings.Split(@type,\".\")[5])\" \"\\(metadata.name)\"": invalid attribute: expected '('

or
$ cue import ./profiles/... -p profile -l '"\(strings.Split(type,".")[5])" "\(metadata.name)"' -f -R
reference "type" not found:
    <path flag>:1:18

Is there a way to escape etc?

Did you try strings.Split("@type",".")?

yes, "@type" that will not work, that only split "@type" as string not the referenced key from yaml.

I think one of the things that's causing the issue here is that there isn't currently a way to reference the root of the tree, so you can't refer to a field that isn't a valid CUE identifier.

also, you cant to reference to key. As the whole key is structs and that can't be converted to "string"

@epcim: doh, yes of course.
@jlongtine: you can actually reference top-level references that are not value CUE identifiers using field aliases:
T="@type": foo

bar: T

But I guess that doesn't help you in this particular case.
It may be an idea to allow @ (or another symbol consistent with the proposed query language) in these cases as an identifier for the current value. This would allow one to write @["@type"].

This also affects the cue eval --expression option:
"@type": "foo"

:!cue eval % --expression @type
Error: invalid attribute: expected '('
invalid attribute: expected '('

shell returned 1

:!cue eval % --expression '@type'
Error: invalid attribute: expected '('
invalid attribute: expected '('

shell returned 1

:!cue eval % --expression '"@type"'
"@type"

Perhaps we could introduce an operator to indicate the following is referring to a field.
"@type": "foo"

:!cue eval % --expression '@"@type"'
"foo"

This would also tie in nicely with the introduction of integers as fields and arrays becoming syntactic sugar for structs:
0: "a"

:!cue eval % --expression '@0'
"a"

["a"]

:!cue eval % --expression '@0'
"a"

A completely different approach is using a builtin: ref("@type"). No special syntax required.

Although I kind of like ."foo", I think it is too confusing as it means several things in different languages, none of which is the thing conveyed here. (Here it means convert to reference, in other languages it means either top-of-file or current object).

Yet another approach is to use back ticks, so @type. CUE actually has this, it is just deprecated.  This is a common approach used in many different languages (including Swift, from which CUE already borrows a great deal). But the disadvantage is that CUE users already need to learn about two different types of quotes (' and ") and that this may become confusing.

A final observation is that within the language itself this does not need to be solved as one can always use field aliases, for instance:
X="@type": val

x: { type: X }

The problem only occurs on the command line. As $ is already a valid identifier, we could make that the thing to refer to with a meaning of (current) top of the config. It would not have to be introduced in the language itself.

Yet another approach is to have a flag to bind an arbitrary identifier to the current config: cue import -r X -l X."@type" also exemplifying an alternative syntax for labels.

Let's solve the import issues first.
Not sure if the last approach helps flag to bind an arbitrary identifier.
As you wrote - language itself has no problem with key with value@type, but you actually want to be capable to import somehow with escaped chars as well as print them out.

With a few upcoming changes this will become easier. Firstly, there will be a --with-context that will evaluate expressions not directly in the data, but in a struct with several fields, including metadata like filename and data for contents. This makes @type not a top-level field, which means it can be accessed as data["@type"]. A sneaky way around the problem, but it works. :)
Also, the --path flag will be simplified a bit: multiple labels are written as multiple uses of the -l flag, each having its own expression.

This issue has been migrated to https://github.com/cue-lang/cue/issues/193.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


exceptions on python 3.5 when debugging

PTVS.Dev.2015-05-28.VS.2015
Python 3.5.0b1 64-bit (installed per-machine, see the path in the messages below)
My code:
from json import dumps

print('hello')

Press F5 and you'll get a series of exceptions that break in the debugger:
FileNotFoundError was unhandled by user code
Message: [WinError 2] The system cannot find the file specified

FileNotFoundError was unhandled by user code
Message: [WinError 2] The system cannot find the file specified

zipimport.ZipImportError was unhandled by user code
Message: not a Zip file

FileNotFoundError was unhandled by user code
Message: [WinError 2] The system cannot find the file specified: 'C:\\Program Files\\Python 3.5\\lib\\json\\__init__.cp35-win_amd64.pyd'

FileNotFoundError was unhandled by user code
Message: [WinError 2] The system cannot find the file specified: 'C:\\Program Files\\Python 3.5\\lib\\json\\__init__.pyd'

etc.

Same thing happens with my Python 3.5.0b1 32-bit, which is installed per-user.

Guessing importlib has changed its name somehow and we need to update the list of things not to debug.

Doesn't repro on 3.5b2 anymore - I guess it had to do with that importlib issue that b1 had.

I still can.  I compared with another VS and realized that this only repros if you have "Just My Code" enabled.  Only affects Python 3.5.


fix: Fixes bug in get_max_in_list_size

This PR fixes a bug in the code introduced in https://github.com/GoogleCloudPlatform/professional-services-data-validator/issues/1146 and also caters for Oracle IN list limit of 1000.

/gcbrun

/gcbrun

/gcbrun


Error when reordering records in the dashboard

Still following the doc.

I also lost one record out of three in the process...
Once I also accessed this page (arbitrary.com), for some reason:

Rolling this into #38, since it's got steps to recreate.


Fixed RBAC for migrations

What this PR does / why we need it:
Fixed RBAC for migrations:

view access only for default admin and edit roles
operator needs removed permissions for updates
refactored tests

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
NONE

Needs confirmation that this is finally what we really want
/hold

/retest

I'm fine with giving namespaced users the ability to view migration objects without create/update/delete permissions.
@rmohr @fabiand @vladikr It would be good to hear you all weigh in here as well since I know this was a big topic earlier in the year.

I'm fine with giving namespaced users the ability to view migration objects without create/update/delete permissions.

It is at least not relevant regarding to security. I also think that I have read the the openshift UI uses the migration objects to indicate if VMIs are migrating. I guess it would be better if it would be discovered by looking at the VMI.

@vladikr do you mind reviewing this one?

I'm fine with this change as well. Especially if we are keeping the migration object namespaced.
But I can't think of a use case for this unless someone wants to keep a list of migration without going through all the VMIs?

It is at least not relevant regarding to security. I also think that I have read that the openshift UI uses the migration objects to indicate if VMIs are migrating. I guess it would be better if it would be discovered by looking at the VMI.

Agree, we keep more information in the VMI.

@stu-gott is there somebody who could pick this up?


Hotfix: use groupData.id instead of stateParams

There should only be one way how to get the group id. Somehow, this also
broke the "leave group" functionality, always sending "undefined" to
the backend.

Current coverage is 98.51% (diff: 100%)

Merging #195 into master will not change coverage

@@             master       #195   diff @@
==========================================
  Files           100        100
  Lines          1479       1479
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1457       1457
  Misses           22         22
  Partials          0          0

Powered by Codecov. Last update bdcf458...771a59a


2770 - work main job changed variant

Pre-review checklist
I have ensured that:

[ ] Code runs successfully in the development environment.
[ ] New code is tested to prove that it meets the specificaiton. If not, justify reasoning.
[ ] New code is documented using type hinting and following the numpydoc docstring format.
[ ] New code follows the project naming standards.

Overview of changes

too old, too may conflicts. Rectified in newer 2770 branch


URS: Localisation lost after reload game

Starting a new game will use the correct language.
After restarting CIV; URS lost his localisation.
All other things are working fine, only localisation in URS.
I took a look in the logs and source but couldn't figure it out. :-(

Cannot reproduce.

Start Civ with actual CQUI Version -> Create Game:

Save -> Exit Civ -> Restart Civ -> Load Savefile:

The problem is reproduceable.
I don't think that the savefile hold the reason:
PHILIPP II. - URS Test.zip

The problem is still here... I can't figure out why...

I have figured it out...
There was a disabled Steam Workshop for URS too. Can't understand why a disabled Mod is a problem, but after delete it a new games works correctly.


Aesara base last good state

Test to try and reproduce in parallel the last known good state

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

The issue was being caused by a bug with the Conda-Forge infrastructure.


How to get mBottomSheetCallback methods to my fragment/activity ?

I am using swipe to refresh view inside bottomsheet viewpager, when I long swipe the view swipe to refresh
it refresh icon get stuck down as the parent bottom sheet is started moving down, if I know the status of the bottom sheet I clear the swipe to refresh view.

maybe i can support when i see a minimal code example/snippet


feat: Inject Custom Input component type

New props named "as" to inject custom component type , default is "input"

import MyInput from './MyInput'
<AutoComplete as={MyInput} {...otherProps} />

It resolves #247  :)

@abdennour seems like you've ignored most/all of the discussion in #247. Simple testing proves that this doesn't work due to missing ref support (your test is too naive: it only tests that the provided component is rendered, not that it behaves as expected).
This feature will not likely be implemented before some solid use-cases are presented, and a thorough discussion regarding the API surface has been performed. That was the intention behind keeping #247 open. If you have a strong argument as to why this feature is needed, please present it for discussion there.

@CMTegner one use case is to be able to use material-ui/TextField or material-ui/Input underneath (I'm talking about v1.0-beta). react-autosuggests allows that as can be seen in an example in the material-ui docs. Though in addition to as it would require a refFuncName property. I did that created a component:
const AutocompleteTextField = (props) => (
  <Autocomplete
    inputComponent={TextField}
    refFuncName="inputRef"
    {...props}
  />
);

and here's the result:

v1.6.0 includes support for props.renderInput which can be used to the same effect as this PR.

@Dattaya thanks for the info! You should be able to use renderInput to make Autocomplete wrap a Material-UI TextField.

I tried this new API and it works flawlessly with material-ui and of course the same holds true for other UI libraries. Thank you, @CMTegner!

No problem, glad i helped! :)

Hi all,
When trying to use renderInput to render a Material UI TextField, I get errors regarding accessing the input ref. It seems that this is because AutoComplete is trying to access getBoundingClientRect() of the TextField component rather than TextField's input ref, which is accessible at this.refs.input._el.input. Any ideas for a workaround? Or should I raise an issue?

You need to pass props.ref to inputRef:
renderInput({ ref, ...props }) {
  return <Input inputRef={ref} {...props} />
}


Race condition with Window.hasFocus()

imgui-sfml relies on the false assumption that Window.hasFocus() will report true immediately after creation. Unfortunately, there is a short period of time after window creation where Window.hasFocus() is not yet true. This is the case for both macOS, Windows, and likely Linux though not confirmed there.
This results in an intermittent bug where input such as mouse clicks will be skipped / not recorded.
This was an issue reported by l04 on the SFML Discord. You can find a (spread out making it somewhat hard to follow) conversation about the issue starting here,
https://discord.com/channels/175298431294636032/175298431294636032/972715469310148658
Quoting l04 here

In the constructor of WindowContext, at line 217 of imgui-SFML.cpp, the statement windowHasFocus = window->hasFocus() is called.

However, the window might not have the focus yet, so it returns false.

Then in the function ImGui::SFML::ProcessEvent, the events LostFocus and GainedFocus are checked.

However, the window might have got the focus before calling the function, so it will not register the GainedFocus event.
In this case WindowContext is stuck with windowHasFocus = false until you manually generate the GainedFocus event.
If I replace line 217 of imgui-SFML.cpp with windowHasFocus = true, I can't reproduce the error anymore. Maybe it needs a better way to handle the initial focus?

A simple fix would be to set the windowHasFocus variable to true immediately on init rather than checking with the function call, though this is admittedly an ugly fix and I'm not sure that this is applicable in all cases. For example in the case of multiple windows.

I didn't fully understand the symptoms.
This is the important part:

there is a short period of time after window creation where Window.hasFocus() is not yet true

This results in an intermittent bug where input such as mouse clicks will be skipped / not recorded.

Not recorded for how long, or until what happens? How haven't others noticed this?

Apologies, I should have been more clear.

I didn't fully understand the symptoms.

When this bug is triggered, most sf::Events will not be  forwarded to imgui until the user does something to cause an sf::Event::LostFocus, and then an sf::Event::GainedFocus.
In other words, when this bug is triggered the user needs to do something like click outside of the window and then click back onto the window, or alt tab in order to get events to propagate from sfml to imgui.
The following events are not forwarded to imgui when the bug is triggered:

sf::Event::MouseMoved
sf::Event::MouseButtonPressed
sf::Event::MouseButtonReleased
sf::Event::TouchBegin
sf::Event::TouchEnd
sf::Event::MouseWheelScrolled:
sf::Event::KeyPressed:
sf::Event::KeyReleased:
sf::Event::TextEntered:
sf::Event::JoystickDisconnected:

This is because these events are all wrapped an an if statement checking whether WindowContext.windowHasFocus is true. Check the file imgui-SFML.cpp in the function processEvent on line 556 to see.

Not recorded for how long, or until what happens?

They are not recorded until the user does something to reset the variable windowHasFocus, such as click outside of the window or alt tab.

How haven't others noticed this?

I think it's hard to trigger this on non-M1 macs.
The initial problem was reported by someone with an M1 mac. Another M1 mac user was able to replicate the hasFocus() returning false behavior saying the following

False for the first 2 cycles of my 60 fps program and then true after that until I clicked out of the window. Same results after launching the program a few times. Same behavior using 2.5.1 or 3.0.0-master.

It seems the way you start the app on an M1 mac also plays a role. Quoting discord again

I think there may be a difference in the way macos handle the windows depending on how you run it.

if you simply compile and run it, it will open a terminal window and then open the sfml window.
if you bundle the executable in an application folder structure, it does not open the terminal, only the sfml window.
with the first option, early I was getting 50% chance of error, but it varies. idk why.
with the second option, so far I have not had the problem when bundling as an app.

A third user on windows also reported that hasFocus can initially report false, though this one is more involved. Apparently if you start a fullscreen window but click in the time between the app launch and the window creation it will initially report false before returning true if I understand correctly.
I myself cannot replicate Window.hasFocus() -> false on initial creation, but I'm on Linux.

So

User creates Window.
User calls imgui::init(Window)
imgui::init sets WindowContext.windowHasFocus with a call to Window->hasFocus();
Window->hasFocus() on rare circumstances can return false immediately after creation, and then be set to true after a few cycles.
When this occurs, (Window.hasFocus() != WindowContext.windowHasfocus)
When this occurs, most of the events in processEvent do not get fired, because it's relying on the cached windowHasFocus value that does not match the actual value.
User must explicitly trigger a lostFocus and then gainFocus to sync variable back to the actual value

I feel like it would be easier to just ignore Gain/Lost focus events and let the user handle them manually
So, ImGui-SFML will process all events as long as ImGui::SFML::ProcessEvent is called. The the users would be able to check window focus events/status manually and just not call the function when focus is lost.
I feel like this will make ImGui-SFML's code much easier and predictable. What do you think?

Workaround for those (like me) who consume it via a package manager and don't want to touch the sources.


Cannot load C++ custom module with Godot 3.0 Beta 2

(Apologize I removed out template text, as initially I will file the issue at GDNative-demos repo, but think again this place is more suitable. Anyway I include much of the info I think you might need and the removed template text'd ask for).
Platform: macOS 10.13 with Godot 3.0 Beta 2
Aim: Trying to integrate C++ Custom module into Godot project
Expected result: Godot editor can load custom module successfully
What I found: Godot silently quit, and it cannot load custom module. As well, after I closed the project and restart Godot editor to load such project again, it cannot start and exit silently without showing anything.
Project files: There are 2 projects I upload as repositories so you can check them out (I linked them appropriately again below while I lay out steps and approach I followed).

C++ Custom Module
Godot project that consumes such module

I successfully ...

built a C++ custom module (Simple Library) as laid out in example here
created NativeScript and GDNativeLibrary resource, hook them up with Godot editor and scene. Load it but at the end it just silently exit the program without error.

Here I will describe the steps and approach I followed

I created a C++ custom module project. You can grab the project or see its source code here. The module is built against

Godot 3.0 beta 2 for macOS as can be seen here in which I first unzip the file then use its Godot.app/Contents/MacOS/Godot as set up in godot-cpp's SConstruct file to build
godot-cpp's bde1ce384fa0e9244ee5fb67c81fceb4c96e1f1d commit
godot_headers's 08c6d98ebcd709488b30bc268f90c9666eb6958e.

Latter twos are both from master branch, and are from latest commit during the time of this writing. You can take a look at its corresponding SConstruct file to see compilation parameters.

After successfully built a module, I copy its .dylib result into Godot project I created.
I set things up, and hook things together (hopefully correct, I succeeded doing this before with Godot 3.0 Beta 1 with C custom module). In short, I create NativeScript then add a new GDNativeLibrary resource linking to it. Both of them saved on file system with extension .gdns and .gdnlib respectively. I checked inside .gdnlib with text editor and it has correct paths to .dylib for platform I target (macOS). Then finally I attach an embed GDScript with a main scene's root node to preload such module. You can grab this project here.
Now I run the project, it silently delays for quite awhile (I use preload in the code) then it quits without any error message.

Please advice on what to proceed next, or any hints that I should test things out. Any advice is appreciated.

Try using the headers from /modules/gdnative/include (after building) instead of the godot_headers repo.

Ok, thanks for reply! I'm testing it. Anyway what do you mean exactly after building? Do you mean after building cpp_bindings?
If so, I did test that and it's has the same result.
As well, I need to attach linking log when try to build a shared library for custom module here as well (happen for either including module/gdnative/include, or not included)
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(AudioEffect.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(AudioStream.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(AudioStreamPlayback.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(BulletPhysicsDirectBodyState.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(BulletPhysicsServer.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(GlobalConstants.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(InputDefault.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(IP_Unix.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(Physics2DDirectBodyStateSW.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(Physics2DServerSW.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(ResourceImporter.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(ResourceImporterOGGVorbis.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(ResourceImporterTheora.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(ResourceImporterWebm.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(Separator.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(Shape.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(SpatialGizmo.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: x86_64 file: bin/libgodot_cpp_bindings.a(VideoStream.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(AudioEffect.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(AudioStream.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(AudioStreamPlayback.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(BulletPhysicsDirectBodyState.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(BulletPhysicsServer.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(GlobalConstants.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(InputDefault.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(IP_Unix.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(Physics2DDirectBodyStateSW.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(Physics2DServerSW.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(ResourceImporter.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(ResourceImporterOGGVorbis.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(ResourceImporterTheora.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(ResourceImporterWebm.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(Separator.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(Shape.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(SpatialGizmo.o) has no symbols
/Volumes/Slave/Applications/Xcode9/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: for architecture: i386 file: bin/libgodot_cpp_bindings.a(VideoStream.o) has no symbols
scons: done building targets.

I'm not sure this might be the factor as well.

I mean after building Godot (since the API struct header gets generated at compile time) but I actually just updated the godot_headers repository to be on the beta 2 state. So use those headers, then those shouldn't be a problem.
I don't know about the linking errors/warning, that's a godot-cpp issue, not Godot.
Have you tried following the (not yet merged) C tutorial? It should be a bit easier to set up than a library with the C++ bindings. The tutorial is here.
Also notice that there is now a UI for the GDNativeLibrary resource, so you don't need to edit the file with an external program anymore.

Whoa, the problem might be I directly link against pre-built Godot binary without actually doing compilation, so that might be the case as you say something get generated on that.
That C tutorial looks great although I have succeeded integrated C custom module before, so I want to achieve the same with C++ custom module.
Thanks for reminding, I noticed it in beta 2 as well, cheers on that.
Alright, I'm doing it again against updated godot_headers repo. I'll report back. Thanks for updating it!

Report back: Didn't succeed. Face the same no matter I use headers at gdnative, or godot_headers. It cannot load module, and shows to me that "Attempt to call function 'new' in base 'NativeScript' on a null instance via the following line
onready var sl = preload("res://simple_library/simple_library.gdns").new()

I would like to make this work as I want to utilize class to write module.
Think again, compiling against official binary should not be a problem as if I manually compile myself, the result should be the same.
Any more suggestions or things I missed here?

Any output in from Godot? Any error or warning prints?

No anymore error or warning. According to that line of code, it show in debugger panel that I was trying to call new() method on null object.

Is there any other output? check the editor output too (from a console/terminal)

There's nothing more error showing.
For reference, I tried running Godot via command line to see if it shows something in console.
In editor I got this.

In console, I got this
[haxpors-mbp:Downloads haxpor$ Godot-3.0-beta2.app/Contents/MacOS/Godot -e
arguments
0: Godot-3.0-beta2.app/Contents/MacOS/Godot
1: -e
Current path: /Volumes/Slave/Downloads
OpenGL ES 3.0 Renderer: Intel HD Graphics 4000 OpenGL Engine
GLES3: max ubo light: 409
GLES3: max ubo reflections: 455, ubo size: 144
ARVR: Registered interface: Native mobile
OPENING: /Users/haxpor/slave/Downloads/TestGDNative3.0-beta2-4 (::Users::haxpor::slave::Downloads::TestGDNative3.0-beta2-4)
arguments
0: /Volumes/Slave/Downloads/Godot-3.0-beta2.app/Contents/MacOS/Godot
1: --path
2: /Users/haxpor/slave/Downloads/TestGDNative3.0-beta2-4
3: --editor
Current path: /Volumes/Slave/Downloads/Godot-3.0-beta2.app/Contents/Resources
OpenGL ES 3.0 Renderer: Intel HD Graphics 4000 OpenGL Engine
GLES3: max ubo light: 409
GLES3: max ubo reflections: 455, ubo size: 144
haxpors-mbp:Downloads haxpor$ ARVR: Registered interface: Native mobile
Running: /Volumes/Slave/Downloads/Godot-3.0-beta2.app/Contents/MacOS/Godot --path /Volumes/Slave/Downloads/TestGDNative3.0-beta2-4 --remote-debug <IP_ADDRESS>:6007 --allow_focus_steal_pid 10196 --position 768,240
arguments
0: /Volumes/Slave/Downloads/Godot-3.0-beta2.app/Contents/MacOS/Godot
1: --path
2: /Volumes/Slave/Downloads/TestGDNative3.0-beta2-4
3: --remote-debug
4: <IP_ADDRESS>:6007
5: --allow_focus_steal_pid
6: 10196
7: --position
8: 768,240
Current path: /Volumes/Slave/Downloads/Godot-3.0-beta2.app/Contents/Resources
Remote Debugger: Connection failed with status: '2', retrying in 1 sec.
OpenGL ES 3.0 Renderer: Intel HD Graphics 4000 OpenGL Engine
GLES3: max ubo light: 409
GLES3: max ubo reflections: 455, ubo size: 144
ARVR: Registered interface: Native mobile

Hmmm, that's weird, I'll check it out after Christmas and will try to replicate (but I don't have a mac)

Ok, thanks for that. Just let me know if you need me to help testing or proceed with any suggestions.

macOS 10.13.2 (17C88), Xcode 9.2 (9C40B).
Godot 3 beta 2 ("FAT" dylib), and 5c63687 (64-bit dylib and main executable).
Project and module from https://bitbucket.org/haxpor/*, godot-cpp and godot_headers from https://github.com/GodotNativeTools/.
Library loads and works correctly.
func _ready():
	print("hey")
	var n = simple_library.new()
	n.test_void_method()
	print(n.method("Test2"))

produce:
Current path: /Volumes/Seagate/Projects/cpp_proj/godot-3.0-beta2-issue-project
OpenGL ES 3.0 Renderer: AMD Radeon HD - FirePro D300 OpenGL Engine
GLES3: max ubo light: 409
GLES3: max ubo reflections: 455, ubo size: 144
ARVR: Registered interface: Native mobile
hey
This is test
Test2

Thank you so much @bruvzg .
I have tested it again. In short, it works fine!
I tested both via compiling Godot binary again manually against latest commit on github, and just normally use beta 2 release binary. Use latest headers from godot_headers and cpp_bindings from godot-cpp. Both case works fine!.
The problem for my case is about rpath in .dylib file which is different from location I put it in a real project, so it cannot locate (I believe) and fail silently.

For others that meet with the same problem (note I work on macOS),
I pin-pointed the problem by using otool -L libsimple_library.dylib, it will show the following output
bin/libsimple_library.dylib:
	bin/libsimple_library.dylib (compatibility version 0.0.0, current version 0.0.0)
	/System/Library/Frameworks/Cocoa.framework/Versions/A/Cocoa (compatibility version 1.0.0, current version 22.0.0)
	/usr/lib/libc++.1.dylib (compatibility version 1.0.0, current version 400.9.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1252.0.0)

But in my testing Godot project. I put such .dylib in simple_library directory, and inside simple_library.gdnlib refers to it via res://simple_library/libsimple_library.dylib for macOS. This is a problem. Contrast this to SimpleDemo via C from GDNative- Demos which put such result shared library inside bin directory. So it won't be a problem.

I checked with SimpleDemo's shared library. Its rpath is bin/libsimple.dylib.

With this, I need to change its rpath by using install_name_tool command. First you cd to the exact directory that holds such result shared library, then do as follows
install_name_tool -id libsimple_library.dylib libsimple_library.dylib

The -id parameter might be misleading as we probably should use -change to change from bin/libsimple_library.dylib to libsimple_library.dylib. But I tried with install_name_tool -change bin/libsimple_library.dylib libsimple_library.dylib without any success (only works with static library). So -id is only option as I've found.
As a result, you can then put your libsimple_library.dylib in anywhere inside your Godot project. No fixed location anymore.
Although we could cross-compile such shared library from macOS to Linux/Windows. Further testing to confirm this is needed. If anyone ever test this out, feel free to chime in here. I closed this issue now as it solved my problem.


fix sized delete warning with c++14

Addresses https://github.com/ros2/tutorials/pull/13#issuecomment-267793843

+1 with @codebot comment addressed.

It will probably be tomorrow before I have access to a computer, if you guys want to change it and then merge, +1.


Add Support for flutter/platform_tests and flutter/flutter_intellij

This can be done once the autosubmit bot is configurable.

This is done and both repositories have been added to autosubmit.


updates to accuracy notebook #1192

See notes in #1192

@mloubout I'm leaving the review to you, but here's a question: is it time to add this notebook back to CI ?

Merging,


[Core feature] Integrate with Soda.io

Tracking issue
Reference Issue - #2963
Why are the changes needed?
The Soda.io plugin integrates data quality monitoring into Flyte, enabling users to automatically perform data quality checks on their datasets. This integration ensures that data quality is maintained throughout workflows, providing users with timely insights into data issues and enhancing the reliability and trustworthiness of their data-driven processes.
What changes were proposed in this pull request?
1.	Basic Integration with Soda.io: Enables execution of data quality checks using custom configurations.
2.	Configurable Scan Definitions: Users can specify a YAML file for scan definitions, allowing for greater flexibility.
3.	API Key Support: Integrates with Soda Cloud using a Custom API Key per User, enabling secure and personalized data quality scans instead of a single shared API key for centralized usage. i hope I choose correctly(please let me know, if I should choose otherwise)

How was this patch tested?
I tested this basic prototype of the plugin through basic unit tests locally, covering both positive and negative scenarios. The tests ensure that the new functionality works as expected and that any edge cases are handled gracefully.
Additional Comments
I’m submitting this PR as an initial implementation of the Soda.io plugin integration. At this stage, I’ve structured the plugin, created the SodaCheckConfig and SodaCheckTask classes, and registered the plugin with Flyte.
I’d greatly appreciate any feedback or guidance to ensure that I’m on the right track with this implementation. I’ll continue to refine the code and address any changes based on the team’s suggestions and feedback.
Looking forward to your insights!
Check all the applicable boxes

[ ] I updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] All commits are signed off.

Future Enhancements
Looking ahead, I plan to implement the following enhancements to further improve the plugin:
•	Result Parsing and Reporting: Enhance the formatting of result outputs for better readability and usability.
•	Error Handling: Implement more robust error handling for configuration and API-related issues to improve user experience.
•	Scan Monitoring: Add support for monitoring the status of ongoing scans to provide real-time feedback to users.
•	Scheduling and Triggers: Enable automated scheduling or triggering of scans based on task dependencies within Flyte, enhancing the plugin’s automation capabilities.

Please let me know if I chose the correct approach and should continue with this and enhance this plugin!

Hi @kumare3 @wild-endeavor @thomasjpfan @Future-Outlier @eapolinario @pingsutw @samhita-alla @cosmicBboy, I wanted to follow up on this PR and check if I’m heading in the right direction with the integration.
Also had a few questions in mind,

API Key Approach: I’ve chosen to use a Custom API Key per User for the Soda.io integration, rather than a single shared API key. Please let me know if this approach seems correct or if I should consider an alternative method.
Basic Functionality: I’ve structured the plugin and implemented the SodaCheckConfig and SodaCheckTask classes. Is this the right way to approach the integration, or would you suggest any improvements?
Future Enhancements: I’ve listed potential future enhancements (e.g., result parsing, error handling, scan monitoring). Do you think these are aligned with the direction of Flyte, or should I consider different enhancements?

I look forward to your insights and any guidance on improving this PR. Please let me know if I’m on the right track, or if there are areas I should revisit.
Thanks again!

@10sharmashivam thank you. Please notice that recently we merged this RFC. It means that community-contributed plugins should live under plugins/community. I'll ask plugin maintainers for a review again

@10sharmashivam thank you. Please notice that recently we merged this RFC. It means that community-contributed plugins should live under plugins/community. I'll ask plugin maintainers for a review again

Hi @davidmirror-ops, thanks for the update! I reviewed the recent RFC regarding community-contributed plugins and the proposed plugins/community structure. Could you confirm if this folder structure is fully implemented and ready for use?
Thanks again for your guidance—I’ll proceed based on your input.

@10sharmashivam the folder is not created yet, waiting for the first contribution after the RFC :) So feel free to create it. Also, please check the lint errors

@10sharmashivam the folder is not created yet, waiting for the first contribution after the RFC :) So feel free to create it. Also, please check the lint errors

@davidmirror-ops Thank you for the clarification! I’ve set up the plugins/community structure and moved the plugin there, following the RFC as discussed. Also, I’ve addressed the linting issues—everything should be in order now.
Let me know if there’s anything else you’d like me to adjust. Thanks again!


Update data-explorer-overview.md

Add metadata to enable Adobe Target on this page.

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

We'll close the pull request because new metadata should be added in the private repository. Thanks!


Feat/lyne link

This PR also includes a suggestion how we could deal with i18n text which is closely related to the module (e.g. "Link target opens in new window.") and should be consistend across the various locations of use.

Codecov Report

Merging #832 (7f16df4) into master (c217eb9) will increase coverage by 2.41%.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##           master     #832      +/-   ##
==========================================
+ Coverage   45.00%   47.42%   +2.41%
==========================================
  Files          13       14       +1
  Lines         571      601      +30
  Branches      135      141       +6
==========================================
+ Hits          257      285      +28
- Misses        291      293       +2
  Partials       23       23

Impacted Files
Coverage Δ

src/components/lyne-link/lyne-link.tsx
93.33% <93.33%> (ø)

Continue to review full report at Codecov.

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

Looks great, thank you!
I mentioned some smaller things and we might have to introduce some more attributes or define how we handle them.
One thing is the naming which I already mentioned in the corresponding files. The other thing is, do we have to introduce also rel, role, id.
Had a look over here which brought up my questions.
https://www.duetds.com/components/link/ https://github.com/carbon-design-system/carbon-web-components/blob/master/src/components/link/link.ts

Thanks for the in depth review @4aficiona2. I will add the id attribute. The rel attribute is set by the module itself for links which open in a new window. I'm not sure if we should expose the rel attribute to the consumers to populate it by themselves. I see very few cases, where this could make sense, but then again we could also solve this differently (e.g. allow/disallow noreferrer). Since we strictly separate the markup for the button and the link, the module should never receive another role as its html default role. Or do I overlook something here?


Unminified dart2js produces invalid names when mixins used

Dart SDK Version (dart --version)

2.2.0-dev.2.1

Whether you are using Windows, MacOSX, or Linux (if applicable)

Linux
dart2js produces invalid property names in prototypes corresponding to classes with mixins. For example, WidgetsFlutterBinding declared as this:
class WidgetsFlutterBinding extends BindingBase
    with
        GestureBinding,
        ServicesBinding,
        SchedulerBinding,
        PaintingBinding,
        SemanticsBinding,
        RendererBinding,
        WidgetsBinding {}

Results in the following output:
  N._WidgetsFlutterBinding_BindingBase_GestureBinding_ServicesBinding_SchedulerBinding_initServiceExtensions_closure0.prototype = {
    // ... other things ...
    $call$body$_WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding_initServiceExtensions_closure: function(value) {
      // ... body omitted ...
    }
    // ... other things ...
  }

Note the & symbols in the property name.

/cc @vsmenon @rakudrama

I noticed this and fixed it earlier this week: https://github.com/dart-lang/sdk/commit/db71576ef39462a99ab2459c5266f65a431443c5


Make wavelink a pycord extension

I think a smart implementation of this library would be as an extension by moving it into the discord/ext/wavelink namespace.
Also the library name currently is a bit unconventional. pycord-ext-wavelink or py-cord-ext-wavelink would be better by following the naming conventions from Danny (menu extension)

As Pycord.Wavelink is under another license it would have to be remade to work with our current license.

Are you sure it needs another license? I don't intend to merge it into the pycord repo. Just the directory and pipy name should be changed accordingly to my issue.

As Pycord.Wavelink is under another license it would have to be remade to work with our current license.

Are you sure it needs another license? I don't intend to merge it into the pycord repo. Just the directory and pipy name should be changed accordingly to my issue.

wdym?

wdym?

Here a good example from Danny: https://github.com/Rapptz/discord-ext-menus
Extensions doesn't need to be placed in the same repo as the extendable project itself.


Warn about clippy lints in code, leave deny to CI

We must not use deny in code because that will make e.g. git bisect or --diff-git-checkouts annoying in the future.
I probably missed to add warn in some place, but this is a good start.
CC @Emilgardis

Thanks for review! I've enforced pedantic from day 1 of this project, and I find in general it results in better code. There are less than a handful allow in the code base. But I agree it occasionally is annoying. But I'd like to keep pedantic, because I think a code base that pass pedantic is nicer to work with than one that doesn't.


Adjusted screen size view

Before Fixing:

After Fixing:

![image](https://user-images.githubusercontent.com/67095045/135281269-44ff3b01-b42b-4600-afb0-68001a622f01.png
But it's not proper in my side


Fix/package updates

Context
Post rebase onto main, feature/embed-stories needed to update some of the package usage.
Summary
This PR updates the packages used to @web-stories-wp package. And fixes default options for the customizer.

Can you check the js unit tests / e2e tests. They seem to be fail for valid reasons.
Also, can you revert changes to package lock file. It is 20k changes seems odd.

Can you check the js unit tests / e2e tests. They seem to be fail for valid reasons.
Also, can you revert changes to package lock file. It is 20k changes seems odd.

About the package-lock.json, I had to use --legacy-peer-deps  for resolving some package-lock issues. And I guess this is the result of that.
While doing this npm sometimes removes resolved url, or dev: true from package-lock.json. Post rebase when I ran the npm i on this branch it again added those resoled url. Not sure if this behaviour is reported anywhere. suggest @spacedmonkey

Updated the PR, removed unrelated package-lock changes. Will work on resolving the test case.

@spacedmonkey failing JS unit test is for the @web-stories-wp/date package
due to following line, it is return smallcase am,pm.
https://github.com/rtCamp/web-stories-wp/blob/1a163370fec08faca262b5ea0c6ade9e947a3ea7/packages/date/src/convertFormatString.js#L117

https://date-fns.org/v2.17.0/docs/format

For capital AM,PM we may have to use a. 🤔

@spacedmonkey failing JS unit test is for the @web-stories-wp/date package
due to following line, it is return smallcase am,pm.
https://github.com/rtCamp/web-stories-wp/blob/1a163370fec08faca262b5ea0c6ade9e947a3ea7/packages/date/src/convertFormatString.js#L117

date-fns.org/v2.17.0/docs/format

For capital AM,PM we may have to use a. 🤔

You're right. A and a are incorrect there. Should be:
// Lowercase Ante meridiem and Post meridiem. am or pm.
a: 'aaa',

// Uppercase Ante meridiem and Post meridiem. AM or PM
A: 'aa',

BUT: why is this failing now? What changed to cause this? And why is it not failing in google/web-stories-wp?

Can you check the js unit tests / e2e tests. They seem to be fail for valid reasons.
Also, can you revert changes to package lock file. It is 20k changes seems odd.

About the package-lock.json, I had to use --legacy-peer-deps for resolving some package-lock issues. And I guess this is the result of that.
While doing this npm sometimes removes resolved url, or dev: true from package-lock.json. Post rebase when I ran the npm i on this branch it again added those resoled url. Not sure if this behaviour is reported anywhere. suggest @spacedmonkey
Also there is this dependency conflict in e2e-test package,

Let's ignore the lock file changes for now

BUT: why is this failing now? What changed to cause this? And why is it not failing in google/web-stories-wp?

I don't think we have changed anything in the packages though testing it on both the branches again and will try to see.

As discussed, let's try fixing the broken code here (because it's clearly a bug)

@divyarajmasani do we need to update a as well? so:
// Lowercase Ante meridiem and Post meridiem. am or pm.
a: 'aaa',

// Uppercase Ante meridiem and Post meridiem. AM or PM
A: 'aa',

@swissspidy
I think that one is fine it's as per the doc here,

https://date-fns.org/v2.17.0/docs/format

Maybe I am missing something, but a in PHP date format prints am/pm.
To get am/pm in date-fns we need to use aaa.
Right now we use aaaa though, which would yield a.m/p.m., so that's different.

Maybe I am missing something, but a in PHP date format prints am/pm.
To get am/pm in date-fns we need to use aaa.
Right now we use aaaa though, which would yield a.m/p.m., so that's different.

you are right there, I missed that you posted a suggestion and not the actual code, yeah for am / pm will have to update it to aaa, on it. I'll update the PR,

@swissspidy I've updated it to use aaa for am / pm.


Code examples related to CORS are generally unrealistic

In general, all CORS code examples have hard-coded domains which is unrealistic. Please put realistic code examples or references to github examples into documentation. Furthermore, real-world use the domains need to be derived from the host bindings. Why and how would one want to manage domain names in a configuration file (assumed that's your go-to example response) and a web server (IIS, Azure, etc) ... which requires manual and double-work (aka DevOps hell)... instead, write code to retrieve bindings from the host and add those dynamically to CORS allowed orgins.

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

ID: 59bca5db-1196-2a8b-a0d8-4e89b2382288
Version Independent ID: d3f332c4-4c60-039e-0e1e-bc6b11831f87
Content: Enable Cross-Origin Requests (CORS) in ASP.NET Core
Content Source: aspnetcore/security/cors.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

As you mentioned, it's not hard to get this information from config.


SDF -> WoT: Implement sdfChoice mapping

Codecov Report

Merging #9 (ee2d93a) into main (61baf2e) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          208       216    +8
=========================================
+ Hits           208       216    +8

Impacted Files
Coverage Δ

converter/sdf_to_wot.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Youtube unable to extract uploader nickname and video thumbnail

Checklist

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

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--embed-thumbnail', '-o', '%(title)s-%(uploader)s.%(ext)s', '-x', '-f', 'bestaudio[ext=m4a]/bestaudio[ext=opus]', '--audio-quality', '0', 'https://www.youtube.com/watch?v=g0uN3REgIRk', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2020.06.06
[debug] Python version 3.8.2 (CPython) - Linux-5.4.0-33-lowlatency-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg 4.2.2, ffprobe 4.2.2
[debug] Proxy map: {}
[youtube] g0uN3REgIRk: Downloading webpage
[youtube] {18} signature length 109, html5 player 16a691a1
[youtube] {136} signature length 109, html5 player 16a691a1
[youtube] {247} signature length 109, html5 player 16a691a1
[youtube] {135} signature length 109, html5 player 16a691a1
[youtube] {244} signature length 105, html5 player 16a691a1
[youtube] {134} signature length 109, html5 player 16a691a1
[youtube] {243} signature length 109, html5 player 16a691a1
[youtube] {133} signature length 109, html5 player 16a691a1
[youtube] {242} signature length 109, html5 player 16a691a1
[youtube] {160} signature length 109, html5 player 16a691a1
[youtube] {278} signature length 109, html5 player 16a691a1
[youtube] {140} signature length 109, html5 player 16a691a1
[youtube] {249} signature length 109, html5 player 16a691a1
[youtube] {250} signature length 109, html5 player 16a691a1
[youtube] {251} signature length 109, html5 player 16a691a1
WARNING: unable to extract uploader nickname
WARNING: unable to extract video thumbnail
[debug] Invoking downloader on 'https://r5---sn-8xgp1vo-2ial.googlevideo.com/videoplayback?expire=1592267903&ei=HsDnXu-nPISFgwOGvoqICQ&ip=<IP_ADDRESS>&id=o-AK0EC0CJ2tpX17E61lY6Hf63PTVSoW1xCwMIV5UKYweA&itag=140&source=youtube&requiressl=yes&mh=8m&mm=31%2C29&mn=sn-8xgp1vo-2ial%2Csn-ab5l6nzd&ms=au%2Crdu&mv=m&mvi=4&pl=16&initcwndbps=1352500&vprv=1&mime=audio%2Fmp4&gir=yes&clen=2892516&dur=180.117&lmt=1419253149459919&mt=1592246195&fvip=5&keepalive=yes&fexp=23882513&c=WEB&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIhAIHi7OCtQ4hmdCrNDOhpYC60_kiIsRh_zhaSpSLxq3I9AiBLV1oCKq12GmWbdnZATni4QsepAfoIlnrRFLipGdG41A%3D%3D&sig=AOq0QJ8wRgIhAOzyndzYcc6arlBUCsvKMeg_eW2HvVeba7WcolGx_k5KAiEAhHOgCoV79WV-mSvAchQxaAlizu8yNokb-MG8eUKAO98=&ratebypass=yes'
[download] Faun-Diese Kalte Nacht (with lyrics and English translation)-Anuja Deore.m4a has already been downloaded
[download] 100% of 2.74MiB
[ffmpeg] Correcting container in "Faun-Diese Kalte Nacht (with lyrics and English translation)-Anuja Deore.m4a"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i 'file:Faun-Diese Kalte Nacht (with lyrics and English translation)-Anuja Deore.m4a' -c copy -f mp4 'file:Faun-Diese Kalte Nacht (with lyrics and English translation)-Anuja Deore.temp.m4a'
[debug] ffmpeg command line: ffprobe -show_streams 'file:Faun-Diese Kalte Nacht (with lyrics and English translation)-Anuja Deore.m4a'
[ffmpeg] Post-process file Faun-Diese Kalte Nacht (with lyrics and English translation)-Anuja Deore.m4a exists, skipping
[embedthumbnail] There aren't any thumbnails to embed

Description

when trying to download videos the downloader will no longer get the thumbnail image or the uploader nickname.

Probably related: I am also not able to use the output format %(upload_date)s, so the Upload data metadata seems to be missing too.

Log showing error while using %(upload_date)s:
$ youtube-dl -v -f 'best[height<=480]' https://www.youtube.com/watch?v=wclza9AZN3U --output '%(upload_date)s_%(title)s-%(id)s.%(ext)s'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-f', 'best[height<=480]', 'https://www.youtube.com/watch?v=wclza9AZN3U', '--output', '%(upload_date)s_%(title)s-%(id)s.%(ext)s']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2020.06.06
[debug] Python version 3.8.2 (CPython) - Linux-5.6.11-arch1-1-x86_64-with-glibc2.2.5
[debug] exe versions: ffmpeg 4.2.2, ffprobe 4.2.2, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] wclza9AZN3U: Downloading webpage
WARNING: unable to extract uploader nickname
WARNING: unable to extract video thumbnail
[debug] Invoking downloader on 'https://r4---sn-vgqskne6.googlevideo.com/videoplayback?expire=1592270363&ei=u8nnXs7bGJLYNsveutAB&ip=<IP_ADDRESS>&id=o-ACBFASZpAVDoQIZkadV4JVHACvdOrSUHqbVh3hbpqiGB&itag=18&source=youtube&requiressl=yes&mh=ij&mm=31%2C26&mn=sn-vgqskne6%2Csn-p5qlsndz&ms=au%2Conr&mv=m&mvi=3&pl=17&initcwndbps=1900000&vprv=1&mime=video%2Fmp4&gir=yes&clen=47783400&ratebypass=yes&dur=636.876&lmt=1591992569908780&mt=1592248655&fvip=4&fexp=23882514&c=WEB&txp=4531432&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cratebypass%2Cdur%2Clmt&sig=AOq0QJ8wRQIgRytWqUIDXjmWuGDBYL838eGWNiJ9FTztkg9-LQoaXj4CIQDhSHk6uF5YEg7P-sZXeorlV0RQIVDNuq22G1qYp2jP8g%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgIv-7BewBOxBv0NSwuM5VyLIyQ6zZHZh0bj7lhmFrXhwCIHLATyqOgZYbMdC8308072mx8kJvlLdN_N86RK8iZksm'
[download] Destination: NA_Cooking breakfast omelettes for two hungry dogs-wclza9AZN3U.mp4
[download] 100% of 45.57MiB in 00:05

The date comes through as NA

Playlists with multiple pages also dont download now. All these problems started happening at the same time. It seems like something changed in youtube's end

@pukkandan That was solved in commit https://github.com/ytdl-org/youtube-dl/commit/d84b21b427d080a031ca1f9c75beb1dc8024b900 which is already in master.

@tyzoid when will be fixed downloadable release?


IIS Create Site

I am getting weirdness when I try and use the following to create a new IIS site.
iis_site { 'testsite':
app_pool => 'testsite',
ip       => '*',
path     => 'C:/sites/testsite',
host_header => 'testsite',
port     => '80',
protocol => 'http',
ssl      => 'false'
}
I run puppet from the machine to apply the config....
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Info: Caching catalog for removedservernamehere
Info: Applying configuration version 'e2b8bc40e2fa6d019a84a54b6921f2757626c8b3'
Notice: /Stage[main]/Profile::Windows::Iis::Base/Exec[check-feature-delegation]/returns: executed successfully
Error: Failed to apply catalog: A JSON text must at least contain two octets!
Not sure what I am missing here but it does not like what I have in the above in the site creation.

Solved this issue.  Please see:
https://github.com/voxpupuli/puppet-iis/issues/128


LABYRINTH doesn't work.

Autodimensioning arrays.
Appears to be defined on the fly by the first usage. Even with multiple dimensions. Difficult to describe my contempt for anyone who would deliberately design a programming language that works like this.
 Or resign any programming job you may have.
Not fixing. Reason ; arrays like DEF FN are static.


Lack of documentation

Configuration file : node-ftp, TLS, ssh2
How to
Shortcut

Would be highly appreciated, because right now I don't know how to use your package.

@buschtoens I added a Getting started in the readme that should get you up and ready.

Thank ya,
The problem is, I already tried exactly that. But nothing seems to happen.
I use Windows 8.1 64bit.

Can you elaborate on what step does nothing?

Remote-FTP not present in Packages : try disabling/enabling the plugin
Sidebar not toggling : try reload the editor with CTRL+ALT+R
Connect does nothing : if your using FTP, open the console CTRL+ALT+I and search the FTP log for error

@buschtoens might be related to #21

Sorry for not responding sooner. Will look into that this evening.
Am 22.01.2015 19:17 schrieb "Michael Grenier"<EMAIL_ADDRESS>
@buschtoens https://github.com/buschtoens might be related to #21
https://github.com/mgrenier/remote-ftp/issues/21
—
Reply to this email directly or view it on GitHub
https://github.com/mgrenier/remote-ftp/issues/7#issuecomment-71069557.

@mgrenier
In regards to

Configuration file : node-ftp, TLS, ssh2

How about a section in showing both config types and we can borrow the descriptions of node-ftp and ssh2's respective options and inline them as comments and edit as needed. For example:
SFTP Configuration Options
{
    "protocol": "sftp",
    "host": "example.com", // string - Hostname or IP address of the server. Default: 'localhost'
    "port": 22, // integer - Port number of the server. Default: 22
    "user": "user", // string - Username for authentication. Default: (none)
    "pass": "pass", // string - Password for password-based user authentication. Default: (none)
    "remote": "/",
    "agent": "", // string - Path to ssh-agent's UNIX socket for ssh-agent-based user authentication. Windows users: set to 'pageant' for authenticating with Pageant or (actual) path to a cygwin "UNIX socket." Default: (none)
    "privatekey": "", // mixed - Buffer or string that contains a private key for either key-based or hostbased user authentication (OpenSSH format). Default: (none)
    "passphrase": "", // string - For an encrypted private key, this is the passphrase used to decrypt it. Default: (none)
    "hosthash": "", // string - 'md5' or 'sha1'. The host's key is hashed using this method and passed to the hostVerifier function. Default: (none)
    "ignorehost": true,
    "connTimeout": 10000, // integer - How long (in milliseconds) to wait for the SSH handshake to complete. Default: 10000
    "keepalive": 10000 // integer - How often (in milliseconds) to send SSH-level keepalive packets to the server (in a similar way as OpenSSH's ServerAliveInterval config option). Set to 0 to disable. Default: 10000
}

FTP & FTPS Configuration Options
{
    "protocol": "ftp",
    "host": "example.com", // string - The hostname or IP address of the FTP server. Default: 'localhost'
    "port": 21, // integer - The port of the FTP server. Default: 21
    "user": "user", // string - Username for authentication. Default: 'anonymous'
    "pass": "pass", // string - Password for authentication. Default: 'anonymous@'
    "remote": "/",
    "secure": false, // mixed - Set to true for both control and data connection encryption, 'control' for control connection encryption only, or 'implicit' for implicitly encrypted control connection (this mode is deprecated in modern times, but usually uses port 990) Default: false
    "secureOptions": null, // object - Additional options to be passed to tls.connect(). Default: (null) see http://nodejs.org/api/tls.html#tls_tls_connect_options_callback
    "connTimeout": 10000, // integer - How long (in milliseconds) to wait for the control connection to be established. Default: 10000
    "pasvTimeout": 10000, // integer - How long (in milliseconds) to wait for a PASV data connection to be established. Default: 10000
    "keepalive": 10000 // integer - How often (in milliseconds) to send a 'dummy' (NOOP) command to keep the connection alive. Default: 10000
}

privatekey should probably note that the string should be a file path to the private key...  I know it should have been my first choice, but I encoded the private key file with \n and put that in... it was after I determined that it really is supposed to be a path.

The README says Buffer or string that contains a private key for either key-based or hostbased user authentication (OpenSSH format)
It should have worked if you had left out the \ns

But Wiki > Settings says:

key: set privatekey to the path to your private key, set passphrase if key is encrypted

Funny...
We use the mscdex/ssh2 module for SFTP. The keyparser used there expects a buffer or a string in the OpenSSH format
BUT: We seem to support both path and key if I understand this line correctly: https://github.com/mgrenier/remote-ftp/blob/96bfd7d9a46d7b2b15ac32362c129c78c0b57a2e/lib/client.js#L170

@jpxd no, line means that if self.info.privatekey  is set then read file that it targets to. so remote-ftp  supports only path

Ah you`re right. I'll update it in the README


Jquery cloneYa

Hello,
i am using the cloneya plugin in my form.

As you can see, one click at the blue button and all the 5 rows with the input fields will be cloned. That works fine!
But its very for me to send them via email. It looks like a very complicate array.
Have someone an idea?
Cheers
Andi

I serialize the input fields , put the serialized string into an hidden input field and then process it server side.


Created a decorator to be able to implement Model.all

As mentioned in this comment, this doesn't solve the issue.


[Android] Crash when canceled showScreenSet()

On Android devices, when back from Web Screen-Set with hardware back key, EventChannel('screensetEvents') throws the following error (uncatchable) and the application crashes.
PlatformException(200001, Operation canceled, null, null)

The cause of this is the flutter side implementation of the plugin do not pass onError to EventChannel('screensetEvents').receiveBroadcastStream().listen().
    const EventChannel _stream = EventChannel('screensetEvents');
    _screenSetsEventStream = _stream.receiveBroadcastStream().listen((onData) {
      onScreenSetEvent(
        onData['event'],
        onData['data'],
      );
      if (onData['event'] == 'onHide' || onData['event'] == 'onCanceled') {
        _screenSetsEventStream = null;
      }
    });

But, I think, Android's onCanceled should be returned  by success() instead of error() like onError in the first place.
Not really good:
            override fun onCanceled() {
                screenSetsEventsSink?.error("200001", "Operation canceled", null)
                screenSetsEventsHandler = null
                screenSetEventsChannel = null
                screenSetsEventsSink = null
            }

LGTM:
            override fun onCanceled() {
                screenSetsEventsSink?.success(
                    mapOf(
                        "event" to "onCanceled",
                        "data" to mapOf<String, Any>()
                    )
                )
                screenSetsEventsHandler = null
                screenSetEventsChannel = null
                screenSetsEventsSink = null
            }

Thank you.

@hiroki-murakami-AT-IRET I have a WIP PR which refactors a bunch of things in the plugin. One of which is how the screen sets stream is handled. Your proposed change (to have all events be regular data events) is included in that PR.

Fixed in 1.0.0 release.
Thank you!


InkWell's customBorder's color is not taken into account

Steps to Reproduce
Run the below code in DartPad or other.

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

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

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      home: Scaffold(
        backgroundColor: Colors.white,
        body: Center(
          child: InkWell(
            onTap: () => print('onTap'),
            customBorder: const RoundedRectangleBorder(
              borderRadius: BorderRadius.all(Radius.circular(16),),
              side: BorderSide(color: Colors.red, width: 4, style: BorderStyle.solid),
            ),
            child: Padding(
              padding: const EdgeInsets.only(left: 8, right: 8, top: 4, bottom: 4),
              child: Row(
                mainAxisSize: MainAxisSize.min,
                children: const [
                  Text('Wednesday 9 March 2022'),
                  Icon(Icons.calendar_today_rounded, color: Colors.grey, size: 20,),
                ],
              ),
            ),
          ),
        ),
      ),
    );
  }
}

Expected results:
A red border of 4px should show around the Inkwell.
Actual results:
No red. The shape is taken into account
I believe the color of the Inkwell overrides the border color.

Reproducible on stable and master using code sample provided.

video
https://user-images.githubusercontent.com/88313112/157450690-8ebfca09-31c2-4dec-94fd-d03db714d769.mp4

flutter doctor -v
[✓] Flutter (Channel stable, 2.10.3, on macOS 12.2.1 21D62 darwin-arm, locale en-GB)
    • Flutter version 2.10.3 at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 7e9793dee1 (7 days ago), 2022-03-02 11:23:12 -0600
    • Engine revision bd539267b4
    • Dart version 2.16.1
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

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

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

[☠] IntelliJ IDEA Community Edition (the doctor check crashed)
    ✗ Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know about this issue at https://github.com/flutter/flutter/issues.
    ✗ FormatException: Unexpected extension byte (at offset 5)
    • #0      _Utf8Decoder.convertSingle (dart:convert-patch/convert_patch.dart:1789:7)
      #1      Utf8Decoder.convert (dart:convert/utf.dart:351:42)
      #2      InputStream.readString (package:archive/src/util/input_stream.dart:207:30)
      #3      new ZipDirectory.read (package:archive/src/zip/zip_directory.dart:40:30)
      #4      ZipDecoder.decodeBuffer (package:archive/src/zip_decoder.dart:19:30)
      #5      ZipDecoder.decodeBytes (package:archive/src/zip_decoder.dart:14:12)
      #6      IntelliJPlugins._findPluginXml (package:flutter_tools/src/intellij/intellij.dart:130:44)
      #7      IntelliJPlugins._readPackageVersion (package:flutter_tools/src/intellij/intellij.dart:141:40)
      #8      IntelliJPlugins.validatePackage (package:flutter_tools/src/intellij/intellij.dart:63:35)
      #9      IntelliJValidator.validate (package:flutter_tools/src/intellij/intellij_validator.dart:103:15)
      #10     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:32)
      #11     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:109:18)
      #12     _rootRun (dart:async/zone.dart:1426:13)
      #13     _CustomZone.run (dart:async/zone.dart:1328:19)
      #14     _runZoned (dart:async/zone.dart:1861:10)
      #15     runZonedGuarded (dart:async/zone.dart:1849:12)
      #16     runZoned (dart:async/zone.dart:1780:12)
      #17     asyncGuard (package:flutter_tools/src/base/async_guard.dart:109:3)
      #18     Doctor.startValidatorTasks (package:flutter_tools/src/doctor.dart:205:9)
      #19     Doctor.diagnose (package:flutter_tools/src/doctor.dart:309:47)
      #20     DoctorCommand.runCommand (package:flutter_tools/src/commands/doctor.dart:50:48)
      #21     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1320:12)
      <asynchronous suspension>
      #22     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1161:27)
      <asynchronous suspension>
      #23     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #24     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
      <asynchronous suspension>
      #25     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
      <asynchronous suspension>
      #26     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #27     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
      <asynchronous suspension>
      #28     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
      <asynchronous suspension>
      #29     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
      <asynchronous suspension>
      #30     main (package:flutter_tools/executable.dart:94:3)
      <asynchronous suspension>

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

[✓] Connected device (3 available)
    • iPhone 13 (mobile) • 304CA91D-8D77-4AAE-A50F-ECFC57AFD011 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64   • macOS 12.2.1 21D62 darwin-arm
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 99.0.4844.51

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

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 2.11.0-0.0.pre.831, on macOS 12.2.1 21D62 darwin-arm, locale en-GB)
    • Flutter version 2.11.0-0.0.pre.831 at /Users/nexus/dev/sdks/flutters
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 3172065fba (9 hours ago), 2022-03-08 19:35:20 -0600
    • Engine revision 5c760759fe
    • Dart version 2.17.0 (build 2.17.0-182.0.dev)
    • DevTools version 2.11.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

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

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

[✓] IntelliJ IDEA Community Edition (version 2021.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin version 213.5744.122

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

[✓] Connected device (3 available)
    • iPhone 13 (mobile) • 304CA91D-8D77-4AAE-A50F-ECFC57AFD011 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64   • macOS 12.2.1 21D62 darwin-arm
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 99.0.4844.51

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

• No issues found!

InkWell's custom border only takes the outer path of the border to shape the inkWell, it is not supposed to use the border side.  This is just to shape InkWellitself.
Source of this behavior
https://github.com/flutter/flutter/blob/b7a18be0a027805705689827376c578d98d53012/packages/flutter/lib/src/material/ink_well.dart#L128
If you want to provide a border side, please wrap your inkWellwith DecoratedBox
IMO, this is working as intended.

Thanks for the info. Please consider updating the documentation of either InkWell or InkWell.customBorder. Simply ignoring certain arguments must be clearly documented imo.


New command schedule

Created endpoint to create new scheduled command.

also im guessing tests are to be added later?


IDVA6-1153 Custom function creating path to fonts added

Link to Jira:
https://companieshouse.atlassian.net/browse/IDVA6-1153
Changes:

Custom function creating path to fonts added

CI: No security warnings found


Update decodeBresser5In1Payload() from rtl_433 project

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
215

Relevant Lines:
215

💛  - Coveralls


Adds unit test for video on page

Please make sure to target feature and bugfix requests to develop branch
Issue #159: Add tests for video component
Changes

Added test for Video component

Notes
Had a lot of trouble to get the tests working, mainly due to the gems failing at one degree or another. I'll list the main ones below just for reference:
The first one:
LoadError:
  Error loading the 'sqlite3' Active Record adapter. Missing a gem it depends on? can't activate sqlite3 (~> 1.4), already activated sqlite3-1.3.13. Make sure all dependencies are added to Gemfile.

The second one:
An error occurred while loading spec_helper.
Failure/Error: include ActionView::Helpers::RecordTagHelper

And finally, the last one was the one I've mentioned on the chat

@MarcoBomfim thanks for the great work! The sqlite3 issue seems strange, but the ActionView::Helpers thing should be resolved by now (we removed this unused and outdated dependency) :ghost:

Uhu! Thanks for the all the support as well team!


treasury: abstraction for issuing and redeeming tokens

This is related issue to Accounts API #94:
Features:

Create a token
Issue a token
Compute amount of outstanding tokens in circulation
Receive and retire tokens
Embed and verify metadata into the token's flavor.

See #94 for more considerations w.r.t. statefulness and FFI.

A few thoughts after reading through #139:
API focus
Maybe the proper name for this API is not "Treasury", but "Token"?

You can define token with a given predicate.
You can issue token as a one-shot tx (issue-and-send-to), or as a partial program in a larger transaction.
Embed/verify metadata in a token.
Issue/retire protocol loop with abstracted storage.

Flexibility
If we let the user provide the predicate, we can separate the concerns: they would be able to construct a multi-sig/multi-party configurations w/o the API making these decisions for them.
Then, we can have separate tooling for fancy predicates that could be used with both issuers and accounts.
Program separation
Issuance is probably going to be one of the actions in a transaction. Even if we start with a "one shot" function that "issues to an address", it's still better to separate two-three layers, at least internally:

Issuing a value to a stack.
Transferring the value to a destination.
Composing the whole transaction (and adding nonce if there are no inputs).

Signing separation
For all operations with tokens but signing an issuance, we should not require PredicateWitness::Key. So we need to allow creating a Token type with a more opaque predicate (maybe still a witness tree, but w/o signing keys, if we need to know the structure), but when composing a program for signing, be able to provide the keys separately. Maybe we'll need to redesign PredicateWitness::Key entirely to use an indirect reference to signing material — that may be needed to accommodate both in-HSM signing and multi-party signing.
Circulation
Circulation seems to be out of scope, but the accurate way to compute it is by processing proved-to-be-published transactions, not just incrementing a counter in a table we one signed off.
So we'll need a generic trait for the "state" part of the following "state machine":

When issuing: store the qty_commitment -> (qty,blinding) pair for later retrieval.
When retiring: store the qty_commitment -> (qty,blinding) and flv_commitment -> (flv,blinding) pairs for later retrieval.
When detecting an issuance of a Token's flavor (flavor is unblinded): load qty_commitment -> (qty,blinding) pair and attempt to verify qty. Store increased circulation. If the quantity is unknown, store the reference to this issuance as "unknown".
When detecting a retirement, look up flv_commitment -> (flv,blinding) that corresponds to the token in question. If it does, look up qty_commitment -> (qty,blinding) to decrement qty. If qty is not found for the given commitment, store an unknown retirement. Note: this should probably never happen as if someone else is blinding qty away from the issuer/auditor, they'd blind the flavor too.

Note: the goal of this protocol is not to help issuer enforce some specific way of things being issued or retired (that's the goal of the logic within the predicate), but solely to correctly index the transactions performed by the issuer themselves and avoid miscalculations due to doubly-signed/re-signed or never co-signed/published transactions.

In the event that we have the Token/Treasury API just return programs that can be composed into a tx, would circulation be purely managed from watching the blockchain state?

In the event that we have the Token/Treasury API just return programs that can be composed into a tx, would circulation be purely managed from watching the blockchain state?

Not sure i understood the question, but it seems to me that those are unrelated. No matter how we construct transactions, we can only be sure in the circulation numbers after ingesting actually published transactions.


Particles do not get removed from ParticleContainers when they are killed

This is more of a question than an issue.
Looking at the pixi-particles code, I can see that when using a ParticleContainer, instead of removing dead particles from the container, it sets their alpha to 0 and visible to false.
Just curious, is this some sort of performance tweak?  Right now we have a particle system in a ParticleContainer which was created like so:
				const pContainer = new PIXI.particles.ParticleContainer(1000,{position:true,rotation:true});
But without passing alpha:true to the ParticleContainer constructor, the particles stay on the screen after they are killed.  We're not using alpha in our emitter, so I didn't think we needed to let the ParticleContainer know that that alpha is changing, which is how I came upon this issue.
I'm fine with passing alpha:true, I'm just curious about the reasoning behind it, as it seems like it would be a waste of resources to leave particles on the screen.

It is indeed a performance tweak - ParticleContainers don't seem to like having children constantly added/removed, so my changing particle alpha was a bit of a hack to improve performance (pretty well). However, that was in PIXI 3, and in PIXI 4 the performance does not seem to be nearly as good - I haven't been able to look into what changed with the ParticleContainer, and what the best way to get high performance out of them is.

Ok, thanks for the explanation.  We'll just pass alpha:true for now.


Add missing error documentation pages

Whenever we render an error page, it links to a documentation page where you can find more information about the error. This pages don't exist yet. That's what this issue is all about.

I don't know if this issue is still valid and if it will be needed in the new implementation - can you clarify @fmvilas?

I don't think so as we don't have a database or backend. You can safely close it.


Tom

HI
I have removed this. I added it in for completeness, but as you say, it's not used at the moment. Do you need me to do another pull request?
On Thursday, 8 September 2022 at 17:01:27 BST, alihejazi @.***> wrote:
@alihejazi commented on this pull request.
In src/globals/mixins.scss:

@@ -55,3 +66,18 @@
0 90%      /* bottom left */
);
}

@.*** corner-clip-bottom-right() {
@tomwpaterson-shape this isn't used anywhere. You sure we need it? If not, would mind deleting it from the code?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

HI I have removed this. I added it in for completeness, but as you say, it's not used at the moment. Do you need me to do another pull request? On Thursday, 8 September 2022 at 17:01:27 BST, alihejazi @.> wrote: @alihejazi commented on this pull request. In src/globals/mixins.scss:
@@ -55,3 +66,18 @@
0 90% / bottom left / ); } + + @. corner-clip-bottom-right() { @tomwpaterson-shape this isn't used anywhere. You sure we need it? If not, would mind deleting it from the code? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

@tomwpaterson-shape Thanks. You don't need to make a new PR because this PR is automatically updated as soon as you commit and push your code changes to GitHub. I think I will leave this PR open until @NoelLH can take a look too, but it looks good so far

@tomwpaterson-shape Thanks for this work, looks fantastic! If possible, can you please address the following issues:

npm run build fails. Can you please run it locally and fix the issues that are logged in the terminal, and then commit the fixes? This should also ensure that all the component readme files are up to date.
Lint checks are failing - are you able to fix the lining issues using Prettier with ESlint?
npm run storybook fails -> there's a good chance this might start passing if point 1. is resolved.

Thanks a lot for your work so far; I did a quick local run of npm start and it is looking awesome!
@NoelLH do you have any other comments?


[CARBONDATA-1512] Fixed bug for failing of concurrently executing sql queries in presto

Resolved concurrent query execution failure

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/180/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/305/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/936/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/197/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/949/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/369/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/245/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/997/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/512/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1139/

LGTM


Update the list of supported OS and PHP versions on GitHub runners

Describe the feature
GitHub is soon removing the possibility of using runners based on Ubuntu 18. In fact, tests using that OS version have already started failing randomly as a "warning" strategy.
I tried installing php 5.3, 5.4 and 5.5 on GitHub's Ubuntu 20 and 22, and both cases failed. I presume this is to those ancient versions having been dropped from the php repos used by the action.
I think the Readme should be updated to reflect the current supported versions status.

I will update the README once it is removed, PHP 5.3, 5.4 and 5.5 should still work on all 3 Ubuntu OS GitHub runners listed in the workflow.
Test workflow: https://github.com/shivammathur/test-setup-php/actions/runs/3470290671/workflow
Please restart your workflow if it failed, sometimes that happens because it fails to fetch the build.
If it is still failing and something specific is not working please recreate the issue on a public repo and share the workflow URL.

My bad - I double-checked the failed workflows and the problem was indeed in setting up phpunit, not php. Sorry for the noise


Update base.html

In editor/templates/base.html added {% querystring %} to the end of the #search-bar form element's action attribute.

Thanks for this. What problem does it fix?

I apologise, it doesn't solve any issue yet,i was editing code and by mistake made a pull request.
You can close this,I'll work on it this weekend and will make a pull request. I'm working on issue #327

OK, no problem.


fix: disable validatingadmissionpolicy plugin via config not feature flag

no longer relies on a patch in forked k8s-apiserver to disable validatingadmissionpolicy

improved comment around functino